Cannot create bottle for tcl-tk with a newer brew than 3.3.11

brew config output
> brew config
HOMEBREW_VERSION: 3.3.13-13-g62b82c0
ORIGIN: https://git.balabit/macos/homebrew.git
HEAD: 62b82c0792a3ad21584e6b1070214d2b1beaeded
Last commit: 12 seconds ago
Core tap ORIGIN: https://git.balabit/macos/homebrew-repo.git
Core tap HEAD: 88ac7fce91b3f1785d8d51767045ba5d8aa15253
Core tap last commit: 23 hours ago
Core tap branch: upgrade-formulas
HOMEBREW_PREFIX: /opt/bb-brew
HOMEBREW_CACHE: /opt/bb-brew/Caches
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 4
HOMEBREW_NO_ANALYTICS: set
HOMEBREW_NO_AUTO_UPDATE: set
Homebrew Ruby: 2.6.8 => /opt/bb-brew/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.8/bin/ruby
CPU: quad-core 64-bit kabylake
Clang: 11.0.3 build 1103
Git: 2.24.3 => /Applications/Xcode.app/Contents/Developer/usr/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 10.15.7-x86_64
CLT: <IP_ADDRESS>.29
Xcode: 11.7

brew doctor output
> brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some taps are not on the default git origin branch and may not receive
updates. If this is a surprise to you, check out the default branch with:
  git -C $(brew --repo homebrew/core) checkout master

Warning: Your Xcode (11.7) is outdated.
Please update to Xcode 12.4 (or delete it).
Xcode can be updated from the App Store.

Verification

[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
I would like to update the formulas we are using to build our product. To release the binaries another git branch was created which is reseted to the homebrew-core repo's master.

Error: new ID must be a String
Do not report this issue until you've run `brew update` and tried again.
/opt/bb-brew/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/ruby-macho-3.0.0/lib/macho/macho_file.rb:318:in `change_dylib_id'
/opt/bb-brew/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/ruby-macho-3.0.0/lib/macho/tools.rb:26:in `change_dylib_id'
/opt/bb-brew/Homebrew/Library/Homebrew/os/mac/keg.rb:10:in `change_dylib_id'
/opt/bb-brew/Homebrew/Library/Homebrew/extend/os/mac/keg_relocate.rb:26:in `block (2 levels) in relocate_dynamic_linkage'
/opt/bb-brew/Homebrew/Library/Homebrew/extend/pathname.rb:341:in `ensure_writable'
/opt/bb-brew/Homebrew/Library/Homebrew/extend/os/mac/keg_relocate.rb:23:in `block in relocate_dynamic_linkage'
/opt/bb-brew/Homebrew/Library/Homebrew/extend/os/mac/keg_relocate.rb:22:in `each'
/opt/bb-brew/Homebrew/Library/Homebrew/extend/os/mac/keg_relocate.rb:22:in `relocate_dynamic_linkage'
/opt/bb-brew/Homebrew/Library/Homebrew/keg_relocate.rb:130:in `replace_placeholders_with_locations'
/opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:501:in `block (2 levels) in bottle_formula'
/opt/bb-brew/Homebrew/Library/Homebrew/utils.rb:399:in `ignore_interrupts'
/opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:499:in `ensure in block in bottle_formula'
/opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:499:in `block in bottle_formula'
/opt/bb-brew/Homebrew/Library/Homebrew/keg.rb:334:in `block in lock'
/opt/bb-brew/Homebrew/Library/Homebrew/lock_file.rb:35:in `with_lock'
/opt/bb-brew/Homebrew/Library/Homebrew/keg.rb:330:in `lock'
/opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:390:in `bottle_formula'
/opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:105:in `block in bottle'
/opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:104:in `each'
/opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:104:in `bottle'
/opt/bb-brew/Homebrew/Library/Homebrew/brew.rb:110:in `<main>'

After a short investigation it seems an empty id is generated for the lib at Library/Homebrew/extend/os/mac/keg_relocate.rb:25
==> FILE: /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/tcltls1.7.22/tcltls.dylib
==> FILE.DYLIB_ID shared-tcltls.dylib
==> ID:
With version 3.3.11 brew can build the bottle successfully. With a newer version I could reproduce the issue.
What did you expect to happen?
The json config is generated to create a bottle afterwards.
Step-by-step reproduction instructions (by running brew commands)
> brew install --build-bottle tcl-tk
> brew bottle --json tcl-tk
> brew bottle --merge tcl-tk*.json --write

Finally finished with the git bisect.
It seems this commit brought in the issue:
568bc94f3ee66bb62a31ae9d9c7f15de8c60cc92 is the first bad commit
commit 568bc94f3ee66bb62a31ae9d9c7f15de8c60cc92
Author: Carlo Cabrera<EMAIL_ADDRESS>Date:   Wed Jan 19 00:34:39 2022 +0800
os/mac/keg: use `MachOFile#delete_rpath` instead of `MachO::Tools`

This will allow us to avoid keeping track of the number of `RPATH`s
while trying to delete duplicates.

See discussion at #12745.

Library/Homebrew/extend/os/mac/keg_relocate.rb | 10 +---------
Library/Homebrew/os/mac/keg.rb                 |  2 +-
Library/Homebrew/os/mac/mach.rb                |  7 +++++++
3 files changed, 9 insertions(+), 10 deletions(-)

Thanks. That's the only closely related change recently, so it makes sense. On the other hand, I don't see how that affects what currently looks like unrelated code...

Can you show the output of brew bottle --verbose --debug --json tcl-tk?

brew bottle --json --verbose --debug tcl-tk
/opt/bb-brew/Homebrew/Library/Homebrew/brew.rb (Formulary::FormulaLoader): loading /opt/bb-brew/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/tcl-tk.rb
==> Determining tcl-tk bottle rebuild...
/opt/bb-brew/Homebrew/Library/Homebrew/brew.rb (Formulary::FormulaContentsLoader): loading /opt/bb-brew/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/tcl-tk.rb
/opt/bb-brew/Homebrew/Library/Homebrew/shims/shared/git --version
==> Bottling tcl-tk--8.6.12_1.catalina.bottle.tar.gz...
==> Changing install name in /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/bin/tclsh8.6
from /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/libtcl8.6.dylib
to @@HOMEBREW_CELLAR@@/tcl-tk/8.6.12_1/lib/libtcl8.6.dylib
==> Changing install name in /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/bin/wish8.6
from /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/libtk8.6.dylib
to @@HOMEBREW_CELLAR@@/tcl-tk/8.6.12_1/lib/libtk8.6.dylib
==> Changing install name in /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/bin/wish8.6
from /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/libtcl8.6.dylib
to @@HOMEBREW_CELLAR@@/tcl-tk/8.6.12_1/lib/libtcl8.6.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/itcl4.2.2/libitcl4.2.2.dylib
from /opt/bb-brew/opt/tcl-tk/lib/itcl4.2.2/libitcl4.2.2.dylib
to @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/itcl4.2.2/libitcl4.2.2.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/itk4.1.0/libitk4.1.0.dylib
from /opt/bb-brew/opt/tcl-tk/lib/itk4.1.0/libitk4.1.0.dylib
to @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/itk4.1.0/libitk4.1.0.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/libtcl8.6.dylib
from /opt/bb-brew/opt/tcl-tk/lib/libtcl8.6.dylib
to @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/libtcl8.6.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/libtk8.6.dylib
from /opt/bb-brew/opt/tcl-tk/lib/libtk8.6.dylib
to @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/libtk8.6.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/sqlite3.36.0/libsqlite3.36.0.dylib
from /opt/bb-brew/opt/tcl-tk/lib/sqlite3.36.0/libsqlite3.36.0.dylib
to @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/sqlite3.36.0/libsqlite3.36.0.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/tcllibc/macosx-x86_64-clang/tcllibc.dylib
from /opt/bb-brew/opt/tcl-tk/lib/tcllibc/macosx-x86_64-clang/v3118_00000000000000000000000000000154.dylib
to @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/tcllibc/macosx-x86_64-clang/v3118_00000000000000000000000000000154.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/tcltls1.7.22/tcltls.dylib
from shared-tcltls.dylib
to
==> Changing install name in /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/bin/tclsh8.6
from @@HOMEBREW_CELLAR@@/tcl-tk/8.6.12_1/lib/libtcl8.6.dylib
to /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/libtcl8.6.dylib
==> Changing install name in /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/bin/wish8.6
from @@HOMEBREW_CELLAR@@/tcl-tk/8.6.12_1/lib/libtk8.6.dylib
to /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/libtk8.6.dylib
==> Changing install name in /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/bin/wish8.6
from @@HOMEBREW_CELLAR@@/tcl-tk/8.6.12_1/lib/libtcl8.6.dylib
to /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/libtcl8.6.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/itcl4.2.2/libitcl4.2.2.dylib
from @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/itcl4.2.2/libitcl4.2.2.dylib
to /opt/bb-brew/opt/tcl-tk/lib/itcl4.2.2/libitcl4.2.2.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/itk4.1.0/libitk4.1.0.dylib
from @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/itk4.1.0/libitk4.1.0.dylib
to /opt/bb-brew/opt/tcl-tk/lib/itk4.1.0/libitk4.1.0.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/libtcl8.6.dylib
from @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/libtcl8.6.dylib
to /opt/bb-brew/opt/tcl-tk/lib/libtcl8.6.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/libtk8.6.dylib
from @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/libtk8.6.dylib
to /opt/bb-brew/opt/tcl-tk/lib/libtk8.6.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/sqlite3.36.0/libsqlite3.36.0.dylib
from @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/sqlite3.36.0/libsqlite3.36.0.dylib
to /opt/bb-brew/opt/tcl-tk/lib/sqlite3.36.0/libsqlite3.36.0.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/tcllibc/macosx-x86_64-clang/tcllibc.dylib
from @@HOMEBREW_PREFIX@@/opt/tcl-tk/lib/tcllibc/macosx-x86_64-clang/v3118_00000000000000000000000000000154.dylib
to /opt/bb-brew/opt/tcl-tk/lib/tcllibc/macosx-x86_64-clang/v3118_00000000000000000000000000000154.dylib
==> Changing dylib ID of /opt/bb-brew/Cellar/tcl-tk/8.6.12_1/lib/tcltls1.7.22/tcltls.dylib
from shared-tcltls.dylib
to
Error: new ID must be a String
/opt/bb-brew/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/ruby-macho-3.0.0/lib/macho/macho_file.rb:318:in change_dylib_id' /opt/bb-brew/Homebrew/Library/Homebrew/vendor/bundle/ruby/2.6.0/gems/ruby-macho-3.0.0/lib/macho/tools.rb:26:in change_dylib_id'
/opt/bb-brew/Homebrew/Library/Homebrew/os/mac/keg.rb:10:in change_dylib_id' /opt/bb-brew/Homebrew/Library/Homebrew/extend/os/mac/keg_relocate.rb:26:in block (2 levels) in relocate_dynamic_linkage'
/opt/bb-brew/Homebrew/Library/Homebrew/extend/pathname.rb:341:in ensure_writable' /opt/bb-brew/Homebrew/Library/Homebrew/extend/os/mac/keg_relocate.rb:23:in block in relocate_dynamic_linkage'
/opt/bb-brew/Homebrew/Library/Homebrew/extend/os/mac/keg_relocate.rb:22:in each' /opt/bb-brew/Homebrew/Library/Homebrew/extend/os/mac/keg_relocate.rb:22:in relocate_dynamic_linkage'
/opt/bb-brew/Homebrew/Library/Homebrew/keg_relocate.rb:130:in replace_placeholders_with_locations' /opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:501:in block (2 levels) in bottle_formula'
/opt/bb-brew/Homebrew/Library/Homebrew/utils.rb:399:in ignore_interrupts' /opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:499:in ensure in block in bottle_formula'
/opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:499:in block in bottle_formula' /opt/bb-brew/Homebrew/Library/Homebrew/keg.rb:334:in block in lock'
/opt/bb-brew/Homebrew/Library/Homebrew/lock_file.rb:35:in with_lock' /opt/bb-brew/Homebrew/Library/Homebrew/keg.rb:330:in lock'
/opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:390:in bottle_formula' /opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:105:in block in bottle'
/opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:104:in each' /opt/bb-brew/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:104:in bottle'
/opt/bb-brew/Homebrew/Library/Homebrew/brew.rb:110:in `'

I can reproduce this, but I'm still not quite sure why it's happening. Just to double-check: what is the output of brew install --debug tcl-tk?

console_log.log

My last theory was that this is a regression in ruby-macho, but I, unfortunately, have not been able to reproduce the problematic behaviour using just ruby-macho.
The problem is that tcltls.dylib has an install name of shared-tcltls.dylib (i.e. basename-only). Normally, after the install, fix_dynamic_linkage will turn this into an absolute path (i.e. something like /usr/local/opt/tcl-tk/lib/tcltls1.7.22/tcltls.dylib).
For some reason, fix_dynamic_linkage fails to do this, but doesn't complain about it. This only proves to be a problem when we try to bottle tcl-tk, and the call to change_dylib_id here chokes because the new ID is for some reason an empty string.
The right fix is probably to make sure fix_dynamic_linkage gets the dylib ID change right the first time (this is probably dylib_id_for).
But that still leaves open the question of how this even got broken in the first place. You could try reverting my commit from the git bisect above, but I would be very surprised if it helped. (Maybe there's some edge case I'm not seeing.)

CONTRIBUTING.md

@carlocab, I can confirm that grpc correctly bottles after reverting https://github.com/Homebrew/brew/commit/568bc94f3ee66bb62a31ae9d9c7f15de8c60cc92. Here's the brew install --build-bottle and stool logs before reverting:
grpc.build-bottle.v1.log
grpc.otool.v1.log
and after reverting:
grpc.build-bottle.v2.log
grpc.otool.v2.log
Note that pre-revert, even though the log says:
==> Changing dylib ID of /opt/homebrew/Cellar/grpc/1.43.2/lib/libgrpc++_test_config.1.43.2.dylib
  from @rpath/libgrpc++_test_config.1.43.dylib
    to /opt/homebrew/opt/grpc/lib/libgrpc++_test_config.1.43.dylib

the change doesn't seem to be applied:
/opt/homebrew/opt/grpc/lib/libgrpc++_test_config.1.43.dylib:
	@rpath/libgrpc++_test_config.1.43.dylib (compatibility version 1.43.0, current version 1.43.2)

One thing I am confused about: https://github.com/Homebrew/brew/commit/568bc94f3ee66bb62a31ae9d9c7f15de8c60cc92 only affects the rewriting of RPATHs, but the problem in brew is about the rewriting of dylib IDs. Why does the implementation of RPATH rewriting affect dylib ID rewriting?

If I'm reading the commit correctly, it replaces an open-edit-write sequence on a dylib filename (MachO::Tools.delete_rpath) with an edit-write sequence on an already-open dylib (Pathname.delete_rpath). Perhaps there's a cache overwrite or some similar issue in the latter logic?

I still don't see how that's related to our reading of dylib IDs (and subsequent re-reading and replacing) from the cache, though.

I'm not sure either, and I'm definitely not familiar with the codebase, but since that's the only effective logic difference, that suggests a failure to commit existing changes somewhere, or overlapping changes.
Some years back, I wrote some code that ending up doing something like the following:

read file F into buffer A
make change in A
read file F into buffer B
make change B
flush A to file
flush B to file

Needless to say, the changes in A disappeared without a trace.

This was surprisingly subtle. (Though maybe I should know better than to be surprised.)
Thanks for the help figuring this out, @gromgit, and thanks for helping track down the offending commit, @imresteiner.
Should be closed by #12864.


Document what rules are in each collection

In this patch I just added the list under the description and left them where they are in the docs. I think it's likely they'll be moved to a new location in the docs, but that can be done later.

Codecov Report

Merging #295 (95bbf42) into main (4f2dbbc) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #295   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           62        62
  Lines         1738      1738
=========================================
  Hits          1738      1738

Extracted from #287 so it can be merged sooner.

Needs a rebase.


Fix image vector overwrite description

What this PR does / why we need it:
mount path should point to the directory
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Fix image vector overwrite description

/hold not working yet

It should probably be something like /charts_overwrite instead of /charts (this dir already exists in the image)

Thanks! Please squash the commits into a single one.


Missing processed NER data/ files

Hi!
I see that in the README.md it says that processed NER data can be found in the data/ folder, but I cannot find it. I tried to recreate it from other sources but I think I have not been able to recreate it properly.
Could you add it to the repo or provide a link to it? I am working on modifying EMEA for a binary classification task and it would help me a lot to debug the code.
Thanks,
Melanie

Hi,
Sorry I forgot to add it. You should be able to find it now!


IE 64 bit issue with jssigner

Code is not working on 64 bit IE

It got broken also on firefox with recent updates. Waiting for a standardized API

Is it an IE 64 bit issue, or that Microsoft deprecated CAPICOM and doesn't include it after Windows 7?

If they deprecated CAPICOM, then it will no longer work, yes


make n_procs in cice_in.nml match ncpus in config.yaml

CICE will exit if n_procs in cice_in.nml doesn't match ncpus in config.yaml.
It would it be better for payu to modify n_procs in cice_in.nml to match ncpus in config.yaml.

Somewhat related to this issue: https://github.com/payu-org/payu/issues/97
I agree this is very annoying and we should not have to suffer it.
I recall the debate was whether to just silently fix it or to warn the user and/or prevent submission.  The challenge with the latter option is that we do far too much setup work inside the submission, when we probably ought to move that work prior to submission.  I think this would actually solve a great many problems, since setup is where we see our errors.


[Question] Inference Code

Question
Thank you for the excellent work! I am curious if LLaVA is exclusively available for CLI inference. Is it possible to perform LLaVA inference using code without relying on the CLI? The reason for my inquiry is that using CLI inference is not convenient for generating text for a large number of images for research purposes. Thank you!

i'm wondering the same thing! what's your solution?


Following: handle removed channels better

Argh, the previous PR handled blocked channels, but not deleted channels.
Currently, deleted ones will appear in the sidebar but not in the Manage Page -- should be the other way around, so users can decide whether to remove or let it be.

Is there any way to remove the deleted channel from sidebar/following? @keikari can you try some things?


Build emergency/recovery PROD dispatch workflow

Given a set of PROD-approved images and a release, allow for a workflow dispatch that redeploys that.  Tests should still be included, but can run after recovery.

It's only taking PR numbers for now, but this is finally addressed!  Man, I'm slow...


Lost connection with Neo4j instance if run with multiprocess package

from neo4j.v1 import GraphDatabase
neo4j = GraphDatabase.driver()
graph = neo4j
import multiprocessing

def main(query):
    def run(query, return_):
        with graph.session() as session:
            query_result = session.run(
                query
            )
        return_["answers"] = [rec for rec in query_result]

    manager = multiprocessing.Manager()
    return_ = manager.dict()
    p = multiprocessing.Process(target=run, name="run_query",
                                args=(query, return_))
    p.start()
    p.join(2)  # timeout after 2s
    p.terminate()

query = "MATCH (n) RETURN n LIMIT 1"
# works good
main(query)
# raise error:
# neo4j.exceptions.ServiceUnavailable: Failed to read from defunct connection Address(host='<IP_ADDRESS>', port=7687)
main(query)

My question is why the second run of the main function would raise the error. It seems related to the
p.join(2)
Any idea about it would be appreciated!
Thank you!

There are currently no plans for the driver to support multiprocessing. This module is often used to improve the performance of CPU-bound work, whereas multithreading is often used instead for I/O-bound work, such as database access.
It's non-trivial allow Driver to work multiprocess as sockets cannot be pickled and the Driver hosts a connection pool.
Therefore multithreading may be a good choice in your case. If not, your only option is to move Driver construction inside the process:
def run(query, return_):
        graph = GraphDatabase.driver(...)
        with graph.session() as session:
            query_result = session.run(
                query
            )
        return_["answers"] = [rec for rec in query_result]


Errors should be shown when it is not possible to login

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug Fix

What is the current behavior? (You can also link to an open issue here)
Did not show error notification when trying to log in

What is the new behavior (if this is a feature change)?
Shows error notification if login or password is wrong

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Is there any issue related to this PR in other repository? (such as dojot/dojot)
dojot/dojot#1771

You need add more tests

The tests should be done and create in a New PR!
I will merge this by request @bizetti.


[Tracking] Sentence Embedding Model

Overview
This is a global tracking issue to bring generic sentence embedding models to MLCEngine.
Action Items

[ ] Add support for mistral based sentence embedding

Links to Related Issues and PRs

context https://github.com/mlc-ai/mlc-llm/issues/1744

bge-m3 is a good candidate since it also supports sparse embedding model. In the e2e pipeline, we found the process turning text into embedding took most of the time. (text->embedding through openai API costs 100ms+ while vector search part only needs 10ms). It would be nice to have an efficient embedding model at local


remove xfail https://github.com/apple/swift/issues/60627

Pull Request Description

Remove Xfail https://github.com/apple/swift/issues/60627 from Doggie.

This compiler crash has been fixed. I built the swift toolchain using main and rested things locally

@swift-ci test

Will close the associated issue once I merge this

Compiler crashed again. This seems to be an intermittent issue. Will do some further investigating

Turns out this issue only plagues the project when built with release config. Thus the solution is just to add that to the xfail

@swift-ci test


GH-34105: [R] Provide extra output for failed builds

Rationale for this change
Providing extra output for R package builds where the C++ build fails
What changes are included in this PR?
Update the system call to save output when building Arrow C++ from the R package and output it if it's failed
Are these changes tested?
No
Are there any user-facing changes?
Sure

Closes: #34105

Those three suggestions look correct to me @nealrichardson, thanks for catching them.
@thisisnic I sent you a PR on your fork at https://github.com/thisisnic/arrow/pull/10 with @nealrichardson 's suggestions and also one other fix.
To test this out, I needed a way to make the build fail which I did by setting the libarrow install dir (DEST_DIR) to a non-existent directory. This makes the libarrow build fail at the install step. I didn't test my re-creating other types of common build issues but I think since we're printing everything this is enough of a test for this PR.
The output of that run looks good to me and can be seen in my gist.

@thisisnic requested I just make a fresh PR so this one can be closed in favor of https://github.com/apache/arrow/pull/37727.

Thanks @amoeba


Upper right method inheritance hierachy pane empty

This pane used to let you select the level of inheritance of method visibility.... but is now empty space.

Removed in 3.0.31


add gzip support with config

Coverage increased (+4.5%) to 87.536% when pulling 318128fab66909b94e72ba6b941f30700d9275eb on support-gzip-with-config into 134fd8a4a09223e61acee3afafefc351e3689750 on master.


Close the popover on mousedown instead of click

I needed to change the event, with which the popover component is closed when the user clicks outside. Instead of listening to click, with this change it listens to mousedown now.
The reason for that is that if you open the popover, then drag the window with the scrollbar, it will only close the popover after releasing the scrollbar.

I have same requirement, any updates?

I feel this kind of makes sense to have as the default behavior, maybe we should do that /cc @Akryum?

Will replicate this in the current branch

https://github.com/Akryum/v-tooltip/commit/89258a98f375e5e17bd53de587c697052db41573


Support ID list deserialization with custom Ids

Hi,
We ran into this problem when we tried to use plural custom id-s.

CreateListFactory for custom types creates a string list instead of a custom type list. From this, the type converter can pick up it and create the correct type.

I used one of the recommendations from this PR to solve the issue.
Closes #5114

 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.

Let me check why the type conversion does not work with typed lists ... I think the change should not be needed.

It doesn't work, because the createList creates a list with concrete type. In this case: new List()
but the id serializer serializes the CustomId with (d)efault type; when it deserializes it, string type is used. This results in error.
The type conversion happens after the ID deserialization, so that can't help. The old PR tried to bring in the conversion, but that would change the public API.

Hi Michael,
If you have time, please consider whether my implementation helps improve HotChocolate.
As far as I can tell, CustomId parsing in List<> is only possible this way since the conversion happens later.
I improved my solution. Now it works even if the serialized ID is not the default type.
I investigated the serializing side also, and there the non-default serialization only happens if a custom resolver is created, which transforms the value to one of the basic types.

Hi Michael,
As I see this is solved in this PR: https://github.com/ChilliCream/graphql-platform/pull/7026 so I closed.


RTM: parse /proc

First decision was to parse top. But it shows more information then we need. Also it's harder to parse because of differences between systems.
So we will parse /proc
For average count we will parse /proc/loadavg.
For network usage - /proc/net  , etc.

done


How to use the web-based authentication flow with a conversational bot?

The section of the documentation states the following:

If you want to use the web-based authentication flow with a conversational bot, you'll need to use a task module with a bot.

Yet, as a comment on this StackOverflow question: https://stackoverflow.com/questions/62182009/is-it-possible-to-signin-from-task-module-in-teams-bot-framework-4, the Microsoft representative claims the following:

Sigincard is not supported in Task Module

In the first quoted sentence above, where you state that if developers want to use web-based authentication flow with a conversational bot, they will need to use a task module, instead of linking to https://docs.microsoft.com/en-us/microsoftteams/platform/task-modules-and-cards/task-modules/task-modules-bots (Using task modules from Microsoft Teams bots) which doesn't provide any help on the topic, you should have linked to a documentation page or a sample explaining how to use the web-based authentication with a conversational bot.

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

ID: a6416300-1ddf-3ffd-10d2-99ed1f4eb08c
Version Independent ID: 335d9588-866d-bdde-f1de-6a093a091e2a
Content: Authenticate a user in Teams - Teams
Content Source: msteams-platform/concepts/authentication/authentication.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

@dejancg , Currently we don't have sample for Auth in task module, here is the sample for authentication.
Please let us know if you need any help on this.

@VaraPrasad-MSFT thank you for your response.
I know about those examples. However, they don't show how to start the authentication using that specific scenario, using the task module, which your documentation clearly references. You need to fix the documentation to clearly show how to use the web-based authentication flow from bot messages (not messaging extensions).


Improve automatic folder detection for iCloud email addresses

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.800
Where did you get the app from?
Google Play
Android version
12
Device model
Motorola Edge+ (2020)
Steps to reproduce
In the initial setup, configure an iCloud email address. This requires a previously configured app-specific password on the Apple ID website.
Expected behavior
All folders should be detected automatically, for iCloud and all major email services, considering the larger Thunderbird release that is to come and the new user base that it will bring.
Here's the full list of native folders in iCloud:

With that in mind, this is how the folders should be configured by default in K-9:

Actual behavior
Only 2 of the folders are detected automatically, being the "Sent" and "Trash" folders. The other ones are set to "None".
I would share the screenshot but I deleted it by mistake; it can easily be replicated by configuring an iCloud email address.
Logs
No response

With my iCloud test account I get the following during account setup:

There's an IMAP extension to communicate which folders are associated with special roles. The iCloud IMAP server does support this extension, but (in my case) only returns information about the Sent folder. This is behavior that should be fixed on iCloud's side.
We currently have no plans to add provider-specific non-standard ways to find special folders. You should file a bug against iCloud.

Thank you for your response!
I have submitted a feedback with Apple. I believe it would have been in your team's best interest to file a ticket on your end tho, since the average user who's looking for an Android email client will make its decision on what works best "out of the box".


Export error with sketch 52 and 53

Hi, I'm receiving this error. It started after I updated my sketch libraries. I already tried with version 2.7.7 and 2.7.9 of the plugin and with sketch 52 and 53, but the same thing happens.
If someone could help me, I would greatly appreciate it.

Tanks!

@caroGit13  Please give this a try: #536.
Remember to uninstall the plugin, restart sketch and install the downloaded version, just to make sure there's no cached plugin information.
Also, make sure you're using Sketch v53.2 (72643).
Download from sketch-measure-latest
Still not working?
Follow the steps below:

[ ] Uninstall the plugin
[ ] Quit Sketch
[ ] Follow the steps bellow to remove the PluginsWarehouse folder
[ ] Download the latest zip file from the fork
[ ] Reinstall the plugin
[ ] Test the Mark Properties / Spec Export again

Remove the PluginsWarehouse folder that lives in ~/Library/Application Support/com.bohemiancoding.sketch3/. This is where we cache plugin downloads, and if you’ve been testing different versions of your appcast, you probably have some old stuff there that’s worth cleaning.

Go to the Finder of Mac OS
From the “Go” menu navigate down to “Go to Folder” or...
Hit Command+Shift+G from the Mac OS X desktop or a Finder window
Paste the folder location ~/Library/Application Support/com.bohemiancoding.sketch3/
Remove/Delete the PluginsWarehouse folder

I hope it helps  :)

It working now. Thanks a lot guys!

Follow the @ellunium instructions, it works!! Thanks.


RPC Generator needs to be updated to generate correct Android X imports

Bug Report
The RPC generator generates imports for the old Android support libraries. Now that the project is using the Android X libraries, the generator needs to be updated to generate the correct imports.
Reproduction Steps
1- Generate RPC for one of the responses for example AlertResponse
2- Look at the annotation import
Expected Behavior
import androidx.annotation.NonNull;

Observed Behavior
import android.support.annotation.NonNull;

Fixed with #1445


⭐ [Enhancement]: Add authentication.provider synonyms.

Update runtime.host.authentication.provider with synonyms for future planning.

These are already accounted for in the application_name specification.

Values

Current
New Synonyms

StaticWebApp
None

AppService

AzureAd
EntraId, OAuth

Simulator

Tasks

[ ] Create synonyms
[ ] Ensure application_name compatibility
[ ] Update documentation

None
Some Data API implementations don't require any security at all. There is no option for None, although StaticWebApp handles this in most cases. The truth is, this is slightly more like Simulator except it behaves the opposite—none is treated as authenticated.
EntraId
Azure Active Directory was renamed to Microsoft Entra Identity. We will retain the previous AzureAd moniker for backward compatibility, but it will be marked as deprecated in the documentation once this synonym is available.
OAuth
Currently, AzureAd is just a simple OAuth implementation, and any custom JWT implementation can use it to handle claims. However, it’s far from ideal to make customers choose AzureAd or EntraId when their underlying authentication technology is not from Microsoft. This also opens up a future direction should we need to handle special cases with EntraId.

Introducing synonyms like EntraId and OAuth definitly provides flexibility, also it sounds more intuitive.
Some Questions:

@seantleonard , what complexities does it add in the underlying implementation?
do we plan to completely deprecate AzureAd in future?
Can you add some more explaination on None being similar to Simulator but treated as authenticated?

Do we plan to completely deprecate AzureAd in future?
Deprecate? Yes. Remove? Probably not. @abhishekkumams
Can you add some more explanation?
Typo corrected: none will treat everything as anonymous.

Should permissions:* be optional in an entity?


CNCF Member Engagement Quarterly Emails

Deliverable: Develop quarterly member engagement email.
Detailed requirements are documented in this google doc.
Technical requirements to execute using SFMC and SFDC:

[x] Build a custom data extension to populate dynamic content blocks

[x] Build and manage SQL queries to automatically populate DE every quarter

[x] Fix any mapping issues/logic with the DE and dynamic content blocks as needed

[x] Create new fields in SFDC to be able to populate the DE as needed

[ ] Build a custom report and dashboard to track engagement with the email

[ ] Build email in such a way that when we get feedback from the member EG: I do not want to become a KCSP or KTP. Ensure this content is not displayed in the next quarter.

Where we are today November 12, 2019
Email template with dynamic content blocks is created
Need to validate content is correct
Add member desk links to content
DE spreadsheet template is built, but not populated
SQL script is needed to automatically populate the DE
Webinars are populated in SFDC for the past year
Blogs are in progress, will populate in SFDC week of 11/11/19
KCSP and KTP are up to date
Logo URL is up to date
Interviewing contractors to complete the above work.

@dankohn and @KimMcMahon We've made great progress in building the DE. @dheerajkottali has the DE almost complete which contains all of the data needed to populate the email correctly. He should have this ready for testing soon and is beginning to build out the email template today. We are working to test the email later this week with an internal list. Then we'll move to test with a small member subset to validate the data is pulling correctly.  Our goal is to send the email to all of our members (primary, marketing, and technical contacts on file) on January 6th, once everyone has returned from the holidays.

Documentation of locations of custom DEs, SQL queries and email template in SFMC is here: https://docs.google.com/document/d/1NubMrf8DuZMCDbtDP6RCzhbd-T-qIDtONJwjhn9VMic/edit?usp=sharing

Update: The first member engagement email went out on February 12, 2020 to 706 contacts! We did uncover an issue with subscriber keys, which caused 188 contacts to not receive our email. Marijana is working to fix this as it's a global SFMC issue. This ticket is where we are tracking the progress: https://jira.linuxfoundation.org/browse/DMO-77
Luckily, Dheeraj pulled all reports needed last Friday to determine who received our email and who did not. Today, all subscriber keys were deleted and with this, the tracking on an individual level. We still have overall open and click rates, just no ability to see individually who engaged or did not engage with the email.
Dheeraj is creating a report and DE to capture the contacts that we missed. However, we are in a holding pattern until the subscriber key issue is fixed to send out to the missing 188 contacts.

Planning for Q2 email:
Frequency:
Email 1: Delivered 2/12/2020 showing data from 6/1/2019 to 12/31/2019
Email 2: Est delivery 4/13/2020 showing data from 10/1/2019 to 3/31/2020
Email 3: Est delivery 7/13/2020 showing data from 1/1/2020 to 6/30/2020
Email 4: Est delivery 10/12/2020 showing data from 4/1/2020 to 9/30/2020
Planned Items to include in Q2:

[ ] Add “branding opportunity” language in email to drive updated logos + promote those that are private to go public with their membership
[ ] Add “event contact field” to email to capture primary sponsorship contacts in SFDC, not just in our heads
[ ] 20 bounces - develop a MixMax campaign to follow up with the alternate contacts of those that bounced to ask for a new contact
[ ] 395 did not open - develop a journey to remind people to open over period of time
[ ] End user member inclusion discussion scheduled 3/2
[ ] Explore ways to automate submissions of blogs with an SFMC Cloud Form to enable automated tracking and approval in SFDC

Still waiting for KCSP, KTP and Certified Kubernetes drop down fields to be created in SFDC. See Jira ticket: https://jira.linuxfoundation.org/browse/PG-546

Closed as Q1 was completed, see Q2 for the next phases of member engagement email quarterly. #16 #15


fix: change target property on church levels to last swell target

Description
This PR changes the target property on churches to use the last swell target

Types of changes

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

How Has This Been Tested?
This has been tested on my localhost

Screenshots/Videos (if appropriate):
https://www.loom.com/share/245ed5ecdad04f37ad01ade12066b051

@dabick14 Waiting for your final changes on this

@dabick14 where are the changes?

@dabick14 where are the changes?

@jaedag no change yet. Responses are in earlier comments.

@dabick14 where are the changes?
The ones I requested for

@jaedag final changes pushed please. One rounding up throughout, and component Target Ids added to the aggregated targets


new version still broken

https://github.com/camenduru/stable-diffusion-webui-colab/issues/35#issuecomment-1376905439

Generated images are worse quality, people are deformed, crippled, with extra limbs even if putting negative promts...

Can confirm, the error I keep getting is this -
Traceback (most recent call last):
File "/content/stable-diffusion-webui/modules/call_queue.py", line 56, in f
    res = list(func(*args, **kwargs))
  File "/content/stable-diffusion-webui/modules/call_queue.py", line 37, in f
    res = func(*args, **kwargs)
  File "/content/stable-diffusion-webui/modules/img2img.py", line 148, in img2img
    processed = process_images(p)
  File "/content/stable-diffusion-webui/modules/processing.py", line 479, in process_images
    res = process_images_inner(p)
  File "/content/stable-diffusion-webui/modules/processing.py", line 572, in process_images_inner
    p.init(p.all_prompts, p.all_seeds, p.all_subseeds)
  File "/content/stable-diffusion-webui/modules/processing.py", line 941, in init
    image_masked.paste(image.convert("RGBA").convert("RGBa"), mask=ImageOps.invert(self.mask_for_overlay.convert('L')))
  File "/usr/local/lib/python3.8/dist-packages/PIL/Image.py", line 1731, in paste
    self.im.paste(im, box, mask.im)
ValueError: images do not match

How do we turn back to v1.6?

go back to the old version
!git clone https://github.com/camenduru/stable-diffusion-webui
to this
!git clone -b v1.6 https://github.com/camenduru/stable-diffusion-webui

go back to the old version !git clone https://github.com/camenduru/stable-diffusion-webui to this !git clone -b v1.6 https://github.com/camenduru/stable-diffusion-webui

Thank you!!


LLM as a judge notebook

As discussed, let's take a multi-step approach. 1/ the use case notebook, 2/ how to evaluate against a human created ground truth, 3/ using an LLM as a judge to evaluate without a ground truth.

This will remain as a POC, and the content won’t be merged.


Support eslint 7

ESLint 7 (or 6) removed the start field on node, it actually triggers this error:
Error: Use node.range[0] instead of node.start
Also I assume they changed whether the start is inclusive or not since everything is offset by 1 now, hence the random + 1.
The parser option of RuleTester now requires absolute paths so I added require.resolve.
Fix error in compat check in getLocFromIndex. It checks for getSourceCode but it should be getLocFromIndex.
Test plan
Tested by running tests on eslint 5 and 7.
Also wanted to test eslint 3.5 as it is mentionned in a comment but tests failed for other unrelated reasons. Not sure if supporting 3.5 is still relevent now.

Just verified that this fixes the location reporting with ESLint 7 (on a TS project) 👍
cc @kassens


Error installing app that includes Prometheus

I get this error when running:
helm install myapp-api --set EnvType=dev --set BuildVersion=0.1.0-1590642144668 --namespace=api-dev .
I get this error:
Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "ServiceMonitor" in version "monitoring.coreos.com/v1"
the service monitor is defined in a templates folder (service_monitor.yaml:

apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
name: {{ template "myapp.name" . }}-api
labels:
monitored: "true"
spec:
selector:
matchLabels:
app: {{ template "myapp.name" . }}-api
endpoints:

port: {{ template "myapp.name" . }}-api
interval: 15s

How can I fix this ?

Output of helm version:
version.BuildInfo{Version:"v3.1.2", GitCommit:"d878d4d45863e42fd5cff6743294a11d28a9abce", GitTreeState:"clean", GoVersion:"go1.13.8"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.10-dispatcher", GitCommit:"f5757a1dee5a89cc5e29cd7159076648bf21a02b", GitTreeState:"clean", BuildDate:"2020-02-06T03:33:34Z", GoVersion:"go1.12.12b4", Compiler:"gc", Platform:"windows/amd64"}
Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.6", GitCommit:"72c30166b2105cd7d3350f2c28a219e6abcd79eb", GitTreeState:"clean", BuildDate:"2020-01-18T23:23:21Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
(this is my dev env, this code runs in docker built by jenkins, I hope it's the same)

At first glance it looks like the chart is relying on an API version (monitoring.coreos.com/v1) that does not exist in a vanilla kubernetes cluster; the CRD must be registered first.

https://helm.sh/docs/chart_best_practices/custom_resource_definitions/#install-a-crd-declaration-before-using-the-resource


Disable Lint/UnusedMethodArgument

We don't agree with this rule. See discussion in https://github.com/bundler/bundler/pull/3922.

@homu r+

:pushpin: Commit ed7d822 has been approved by segiddins

:hourglass: Testing commit ed7d822 with merge 4b0db7f...

:sunny: Test successful - status


adding a handful of linux fanotify data types.

close #3688

I think this looks good, but can you confirm that fanotify_event_info_pidfd and fanotify_event_info_error are GNU-only? Also needs a rebase.

@rustbot author

I think this looks good, but can you confirm that fanotify_event_info_pidfd and fanotify_event_info_error are GNU-only? Also needs a rebase.

yes I confirm no presence in other libcs

I think this should be
@rustbot ready
If you have a link to the headers / docs that would be great, trying to get them linked in every PR so we can find them again when needed.

@rustbot author

@rustbot review

:umbrella: The latest upstream changes (presumably #3480) made this pull request unmergeable. Please resolve the merge conflicts.


Swiss flag is a rectangle, not oblong

Please check all that apply

[X] visual bug
[ ] functional bug
[ ] able to consistently reproduce

Expected Behavior
Swiss flag should be a rectangle.
Current Behavior
Swiss flag is oblong.

Hi @adrianegger, thanks for submitting this visual bug. I've added the bug to the backlog. We'll keep you posted in this issue.

Hi @adrianegger, thanks for submitting this visual bug. I've added the bug to the backlog. We'll keep you posted in this issue.

@adrianegger After discussing this with the team we won't be adding exceptions like these in the near future. This will be really intensive to maintain.
You can do this by yourself of course😄! Every flag component contains a property that you can add your own class, so you can add your own exceptions/customizations. If you've any further questions, let us know.

@adrianegger After discussing this with the team we won't be adding exceptions like these in the near future. This will be really intensive to maintain.
You can do this by yourself of course😄! Every flag component contains a property that you can add your own class, so you can add your own exceptions/customizations. If you've any further questions, let us know.


Proxy.jsp response HTML body with sensitive information

When issuing a request like ".../proxy.jsp?http://fakeserver/arcgis/rest/services/some_service/MapServer/" and passing along a valid Referer header to get a response, it will contain HTML which expose private or sensitive information, like the name of the intranet server, example, http://agserver.intranet.local:6080/....
This shouldn't be possible, unless the user really wants it but it's not the case.
To avoid it, there could be a sanitization of the input url by appending a f=json parameter or adding a config option to the proxy.config file blocking this.
This may be an issue for other proxy versions, like PHP and Dot.Net.

This is certainly an issue with the .Net proxy too. It clearly shows the username that the proxy logged in as.

Hi @gustavopalminha @lancegosby,
The resource-proxy does not look inside the proxied content. This is by design.
You might want to investigate other options than resource-proxy, see for example https://developers.arcgis.com/documentation/core-concepts/security-and-authentication/

The proxy here works well otherwise. It would be a shame if we had to redesign our applications.
The proxy wouldn't even need to look inside the content. @gustavopalminha had a good suggestion of validating the "f" parameter. For my limited scenarios, a proxy config setting of "allowHtmlResponse" could work. Then the proxy could ensure that none of the requests use f=html. I'm sure some of the endpoints use html as the default when the "f" parameter is missing so it would need to handle that too.

Hi @lancegosby - thanks for explaining in more details.
If this is just to avoid the "html" being sent back, then there's a server setting for that - "Disable the Services Directory" that might achieve the same result:
http://enterprise.arcgis.com/en/server/latest/administer/linux/disabling-the-services-directory.htm
This still allows the REST API to work.  See for example https://traffic.arcgis.com/arcgis/rest/services and https://traffic.arcgis.com/arcgis/rest/services?f=json.

Hi @bsvensson ,
I'm not sure if we can disable the services directory for all services. For most of our services, it is fine for anyone to browse them. We even browse them ourselves internally when troubleshooting so find it useful to be turned on. We have a couple of secured map services that we host publicly so only use the proxy to access them.
Would you be open to me submitting a PR with the following changes, or you could suggest some way to make it more universal.

add config setting "preventHtmlResponse" with default value of false so as not to force anyone to change their config. probably on the serverUrl node.
if preventHtmlResponse == true, then check the "f" param and update as needed

The only problem is I wouldn't have any way to develop/test the PHP or Java versions. I only have a .NET environment.
Lance


Implement emoji painting support in Skia painter

DisplayListPlayerSkia::draw_glyph_run() should not ignore DrawEmojis in a glyph run and blit them from a bitmap, like it already does for DrawGlyph. You would probably want to look at how Gfx::Painter handles emojis and replicate its behavior using Skia APIs.
Just to clarify, Skia has its own text rendering capabilities, and we would like to start using them in the future. Currently we are not using them, because there is not way to construct SkTypeface from Gfx::Typeface because raw font binary is gone by the time we reach painting. Fixing that would require larger refactoring, but we can totally get there eventually.
HTML that is currently not rendered currently with --enable-skia-painting:
<!DOCTYPE html>

😊

IMO replicating the extremely ad-hoc Gfx::Painter machinery just to get ugly pixelated emojis with Skia would be a poor use of time. I would rather see us directly moving to SkTypeface and friends, and then getting good-looking system emojis.

IMO replicating the extremely ad-hoc Gfx::Painter machinery just to get ugly pixelated emojis with Skia would be a poor use of time.

sure, although it would be cool to get Skia painter on par in terms of completeness, so we could make it a default.
hackish idea how to get Skia text rendering without big refactoring: make Gfx::Typeface always carry a raw binary, so we can construct SkTypeface right before rendering.

IMO replicating the extremely ad-hoc Gfx::Painter machinery just to get ugly pixelated emojis with Skia would be a poor use of time.

sure, although it would be cool to get Skia painter on par in terms of completeness, so we could make it a default.

I think it's good enough that we could make it the default right now :)

Solved in https://github.com/LadybirdBrowser/ladybird/pull/1297


Disable Konflux on release-next branches

Disabling Konflux on release-next branches, as the branches change to often and it's just noise ATM

LGTM, do you want to add the logic to delete the .tekton and  .konflux on main so that we don't have to do it manually ?

LGTM, do you want to add the logic to delete the .tekton and .konflux on main so that we don't have to do it manually ?

Good point. Can you check on 2e493294106f122ec79c50689e25edb2cf18ebfe?

/cc @pierDipi


2011-02-12-0.json.gz third entry repo.name is "/" ?

This also lacks a repo.id:
{"repo":{"url":"https://api.github.dev/repos//","name":"/"},"type":"PushEvent","public":true,"created_at":"2011-02-12T00:00:10Z","payload":{"shas":[["4c65fb00df1005d9e2f6d7d7d23018349bc710d5","d677aaa2417cc877d5126f07f47fde466d1099f5@frd-linux.(none)","When truncating we now ensure the empty directories are pruned.\nAlso each time a directory is created or deleted the listeners are informed.\nThe MonitoringBlobListener in the diskquota now monitor the space taken by the directory indexes by adding one block size per directory created.\nDisable the function that was readjusting the quota after a certain amount of updates.","frd"]],"repo":"francoisregis/geowebcache","actor":"francoisregis","ref":"refs/heads/master","size":1,"head":"4c65fb00df1005d9e2f6d7d7d23018349bc710d5","actor_gravatar":"fd79ce8b2ca0a31bd5f796e2298e7f41","push_id":24643030},"actor":{"gravatar_id":"fd79ce8b2ca0a31bd5f796e2298e7f41","id":601242,"url":"https://api.github.dev/users/francoisregis","avatar_url":"https://secure.gravatar.com/avatar/fd79ce8b2ca0a31bd5f796e2298e7f41?d=http://github.dev%2Fimages%2Fgravatars%2Fgravatar-user-420.png","login":"francoisregis"},"id":"1127195568"}

We're at the whim of GitHub API updates and evolution, unfortunately. This means that the schema, especially when you rewind way back, may not have 100% coverage relative to newer data.
For the particular case you shared there is the "repo":"francoisregis/geowebcache" field, which you could special case and check if id is missing. It's an annoying solution, I know.. but unfortunately not something we can fix easily.


Question about augmentation

Hi, I'm a little confused about the data augmentation.

How did you set img_aug when img_moco=True? It seems that we need an 'img_pipeline' in 'simipu_kitti.py', right?
For 3D augmentation, it seems that it is done in this line. So the 3D augmentation is done based on the point features instead the raw points, right? If I want to try moco=True, how to set 3D augmentation? should I do this in the dataset building part?
https://github.com/zhyever/SimIPU/blob/5b346e392c161a5e9fdde09b1692656bc7cd3faf/project_cl/decorator/inter_intro_decorator_moco_better.py#L394

Looking forward to your reply. Many thanks.

For moco, we follow the implementation of mmselfsup. Please remind we do not use mono in our image branches (no contrastive loss in intra-image), so we do not need img_pipeline in  simipu_kitti.
Yes, given the raw point cloud p. We use feature extractor f to get p_f, meanwhile, we can get p_f' with f and augmented point cloud p'. The p_f and p_f' are with location l_p and l_p'. When implementation, we adopt the same transformation on (p_f, l_p), which is what this line is doing.

Ok, got it. Thanks a lot.


If i open NnnPicker and then close it, the statusline is messed up.

https://user-images.githubusercontent.com/17055027/136293218-4ac197f1-b864-4b0f-a85b-b57480713cdc.mp4
And also, NnnExplorer doent work. It opens but i have an error. And none of the files open if i click enter on it.
https://user-images.githubusercontent.com/17055027/136293368-c1b3c6ff-f935-48f3-864e-413852b97295.mp4
Here i clicked enter or double click on the file and it just won't open.

You will get that error if nnn is started with the -a flag. The plugin removes a from the NNN_OPTS environment variable but you might have added it yourself to the cmd override?
I'll see if I can improve the docs to discourage adding it or mention it with error detection.

I got a similar error as @hrqmonteiro when calling :NnnExplorer (the drawer on the side does not even get show up) - maybe it has to do with the /tmp/nvimsSEKVQdirectory name.
Error executing luv callback:
...l/share/nvim/site/pack/packer/start/nnn.nvim/lua/nnn.lua:87: Error opening pipe for reading:E
NOENT: no such file or directory: /tmp/nvimsSEKVQ/3-explorer
 Avoid running nnn with the -a flag!

I have freshly installed n^3 using sudo apt install nnn. Thanks for any help!

Closing due to lack of response, please do let me know if you still have any errors.

Hi, thanks for reporting.
nnn.nvim doesn't touch the statusbar so not sure what we can do here. My statusbar(feline) also hides when I open NnnPicker but the statusline is restored when NnnPicker is toggled/closed.

And also, NnnExplorer doent work. It opens but i have an error. And none of the files open if i click enter on it.

Are you using a custom cmd override and or NNN_OPTS? If so could you share those. Haven't been able to reproduce the error.
Can confirm mouse support is not working in the explorer split though, I'll look into that, not sure if it's possible though. Mouse pass-through seems to work in floating windows but can't find anything on enabling it in regular splits.

I was using NNN_OPTS but i cleared it all, and i removed any passable flags of nnn i had, and i am still having the same error when opening NnnExplorer, even after i updated

And i am using galaxyline.nvim, and the statusline is still being messed after i close NnnPicker. Didnt try with feline though cause i dont have a config for that.

What version on nnn  and neovim are you on?
Does pressing ENTER on that message give you any more errors?

Could you check if latest commit fixes the issue?

Hi, i updated.
It stills behaves the same way.
I realized that if i use Bash instead of Fish, the NnnExplorer issues resolve. I can use the explorer and open files without that ENOENT error.
But the NnnPicker messing with galaxyline still occurs.

Like I said, that error is expected if nnn.nvim is starting nnn with the -a flag somehow I guess that is what must be happening in your fish shell for some reason.
I can't reproduce your statusline issue, tried lualine and galaxyline.

The only other possible explanation I can think of is that fish is significantly slower resulting in nnn not having created the fifo yet when read_fifo is being called.
Could you check if this branch fixes it?


[WIP] *: rename Arrow to chunk

Signed-off-by: wshwsh12<EMAIL_ADDRESS>
What have you changed?
Please explain in detail what the changes are in this PR and why they are needed:
In tidb, use arrow instead of chunk in some places. This can be misleading.
In this pr, I change all arrow to chunk.
Don't assume reviewers understand the original issue.
What is the type of the changes?
How is the PR tested?
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
Does this PR affect tidb-ansible?
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Any examples? (optional)

Actually I prefer Arrow than the Chunk, because there are too many places using chunk to refer another thing.. while Arrow is a very unique name without ambiguity.

Actually I prefer Arrow than the Chunk, because there are too many places using chunk to refer another thing.. while Arrow is a very unique name without ambiguity.

Arrow(in fact, it it chunk) will be confused with Apache Arrow.  Chunk is not exactly the same as Apache Arrow.   In TiDB,  Chunk is just for one thing - data storage structure.  @breeswish

/run-all-tests

/run-integration-copr-test copr-test=pr/10

Mostly LGTM. But I worry now we don't have any definition of the chunk format. People may be confused when seeing "chunk format". Can we comment about it?

Or should I add the comment back?
/// `Chunk` stores multiple rows of data in Apache Arrow format.
/// See https://arrow.apache.org/docs/memory_layout.html

Now we only have one place that describes chunk in the blog https://pingcap.com/blog-cn/tidb-source-code-reading-10/  and it only have Chinese version.

@wshwsh12 However as you said it is actually not the Arrow format, so this kind of comment is also confusing.. I think we don't have good way to describe this format for now, because the specification is missing.

I think we can merge this pr now.

I think we can merge this pr now.

OK.
/merge


Local Functions unknown command: node

Describe the bug
I'm trying to use nhost dev and run a cloud function. I am using asdf as nodejs package manager.
The whole nhost development system is stuck when browsing to the function locally, and the page in the browser is loading indefinitely:
unknown command: node. Perhaps you have to reshim?

nhost dev -d output:
[nhost] [prefix] Fetching containers
Wrapping containers into environment
[path] Watching .git/HEAD
Fetching local git branch
git HEAD not found
Parsing app configuration
[nhost_hasura] [configuration] Initializing
[nhost_auth] [configuration] Initializing
[nhost_storage] [configuration] Initializing
[nhost_mailhog] [configuration] Initializing
[nhost_postgres] [configuration] Initializing
[nhost_minio] [configuration] Initializing
[watcher] Activated
Server logs will be deleted on shutdown.
Checking for required Nhost container images
Configuring services
[data] Mounting data from  .nhost/main
[nhost] [network] Preparing
[nhost] [network] Fetching
[nhost_postgres] [container] Starting
[nhost_auth] [container] Starting
[nhost_storage] [container] Starting
[nhost_mailhog] [container] Starting
[nhost_hasura] [container] Starting
[nhost_minio] [container] Starting
[nhost] [prefix] Fetching containers
Wrapping containers into environment
Starting health check
[nhost_storage] [service] Health check successful
[nhost_auth] [service] Health check successful
[nhost_hasura] [service] Health check successful
Initializing Hasura client
Applying migrations
Applying metadata

[proxy] [functions] http://localhost:7777 --/v1/functions/
[proxy] [nhost_hasura] http://localhost:9263/v1/graphql --/v1/graphql
[proxy] [nhost_hasura] http://localhost:9263/v2/query --/v2/query
[proxy] [nhost_hasura] http://localhost:9263/v1/metadata --/v1/metadata
[proxy] [nhost_hasura] http://localhost:9263/v1/config --/v1/config
[proxy] [nhost_auth] http://localhost:9023 --/v1/auth/
[proxy] [nhost_storage] http://localhost:8753 --/v1/storage/
[proxy] [console] http://localhost:9306 --/
[proxy] [GET] /
[proxy] [GET] /v1/functions/integrations/greenhouse/webhook
[functions] [NodeJS] Building /integrations/greenhouse/webhook.ts
unknown command: node. Perhaps you have to reshim?

I can execute node -v in all dirs (outside of the project) and it is the correct version.
Desktop (please complete the following information):
Using:  asdf (installed via brew) as package manager,

OS: Mac OSX,
Browser chrome
Version Nhost CLI v0.6

Do you have Node and NPM installed?
Please install them and try again.

It seems like he do give this:

I can execute node -v in all dirs (outside of the project) and it is the correct version.

And npm comes bundled with Node. So I would guess he has both Node.js and NPM installed.

Yes, using asdf thank you will look into it

I'm closing this issue. Please feel free to open it if you can't solve it, @mirko314!


Allow parsing MyST-flavored Markdown

The Sphinx and Jupyter-Book projects both make use of MyST, a Markdown variant specialized for formatting documentation and interactive books (https://myst-parser.readthedocs.io/en/latest/index.html). In particular, MyST extends Markdown with a few concepts from reStructuredText (an alternative plain-text format historically used by Sphinx), such as allowing plugins to add new element and block types. For example, the following MyST markup instructs the Sphinx toctree plugin to typeset a table of contents with depth 2, and include several pages in the generated ToC.
:::{toctree}
---
maxdepth: 2
---

parser
generator
jit
:::

Currently, markdownlint seems to interpret the above as a section called :::{toctree} denoted with setext-style heading markers, and raises warnings about the use of setext versus atx:
Warning: docs/api-reference/index.md:3 MD003/heading-style/header-style Heading style [Expected: atx; Actual: setext]
Warning: docs/api-reference/index.md:3 MD022/blanks-around-headings/blanks-around-headers Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Below] [Context: ":::{toctree}"]
Warning: docs/api-reference/index.md:5 MD003/heading-style/header-style Heading style [Expected: atx; Actual: setext]
Warning: docs/api-reference/index.md:5 MD022/blanks-around-headings/blanks-around-headers Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Above] [Context: "maxdepth: 2"]

If possible and reasonable, it would be really helpful to be able to provide an option that allows for markdownlint to ignore MyST-style syntax.
Thank you for an awesome project! 💕

The general way to do this is to reference a markdown-it plugin that supports the particular syntax you want. This is how support for math blocks is added, for example. I don't see a plugin for MyST, but perhaps you can find one or write one. I want to keep the core library as close to CommonMark as possible for the broadest utility.
https://www.npmjs.com/search?q=keywords:markdown-it-plugin

markdownlint-cli does not support markdown-it plugins, but markdownlint-cli2 does. Documentation is here and there are examples in the repo: https://github.com/DavidAnson/markdownlint-cli2#markdownlint-cli2jsonc


export idle and unidle function for consumption

Not required now, different direction was taken and aergia is now an all in one unit with external controls via labels


fix: Avoid disposing watchers if there is no env to update

I discovered that watchers were cleared when the extension host was restarted. I traced this to calling direnv.dump() twice when the extension host was restarted, and the 2nd time it returned nothing which just results in an empty map.
Since the empty map wasn't guarded, this resulted in the watchers being disposed and no new watchers were setup.

Thanks!

@mkhl Can we get a patch release containing this fix? I've begun rolling direnv stuff out to my repos and this fix makes a big difference in dev workflows.

Of course, sorry.I'll be back at my desk on Monday and release the fix then.On 21. Apr 2023, at 21:09, Blaine Bublitz @.***> wrote:
@mkhl Can we get a patch release containing this fix? I've begun rolling direnv stuff out to my repos and this fix makes a big difference in dev workflows.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


defrag: Implement a IPv4 defragmenter (EXPERIMENTAL)

This is an experiment, I've made during a cloudy weekend ;)
It is based on the DefragIPv4 interface suggested by Graeme some months ago.
I am submitting it for  reviewing purpose and get extensive comments from the GoPacket Authors
This implements the DefragIPv4 interface, which embed IPv4
defragmentation, and modify packets in place. It returns false
if the packet is fragmented, and caller will need to give subsequent
packets. It returns true, if the packet is in its final state (defragmented
or just pristine).

Initial set of comments is mostly stylistic and API stuff (what to export, how API should look, etc).  I haven't done a deep review of the actual logic yet, but hope to in the next day or so.

OK thanks for the initial comments.
Waiting yoru other review round about logic and so one,  to make all fixes at once, and submit it again

First round of implementation comments all done.  PTAL.
On Mon, Apr 20, 2015 at 10:56 AM, Laurent Hausermann <
<EMAIL_ADDRESS>wrote:

OK thanks for the initial comments.
Waiting yoru other review round about logic and so one, to make all fixes
at once, and submit it again
—
Reply to this email directly or view it on GitHub
https://github.com/google/gopacket/pull/71#issuecomment-94507862.

I think we're mostly down to very small comments/nits, so I'd expect we'd be good to merge after the next commit.
Could you add ip4defrag to .travis.cfg in the toplevel dir so it's tested automatically?

Mmm, I did not find any .travis file ?

Well, how should we use it inside the examples/* ?
Add it as a non-default option of dumpcommand at the begining of the main loop ?

Travis config is here:  https://github.com/google/gopacket/blob/master/.travis.yml
I just added it a few days ago, so you may need to pull from master to get it.
I like the idea of adding some example code to dumpcommand.

Also, I think this is in a fine state for an initial push to master.  I've added you as a collaborator, so you should be able to merge as you see fit after the .travis.yml addition.  Please merge whenever you feel comfortable with it, or let me know if you have additioanl questions.

Quick ping on this.  Feel free to merge it whenever you think it's ready.

Hi Graham,
I've commited the dumpcommand part.
There is still an issue : when I defrag packet, it looks like the packet I am writing as "out", is still considered as a Fragment... I suppose it's because we dont re-decode it..
How would like to handle this issue ? The IPv4 defrag should call the decode routine once again ?

I've also added a secondary email address to my CLA...

Well...
I've try to understand how to handle that decoding stuff.. I've done what you asked (ie rebuilding the layer), and more (calling once again decode), but I still have incoherent decoding...
On one side:
$ ping -c1 -s2000 google.com
On the other
$ sudo ./pcapdump -i=wlan0 -defrag=true icmp
Using BPF filter "icmp"
Starting to read packets
Decoding re-assembled packet: ICMPv4
PACKET: 562 bytes, wire length 562 cap length 562 @ 2015-06-13 00:20:14.902541 +0200 CEST

Layer 1 (14 bytes) = Ethernet {Contents=[..14..] Payload=[..548..] SrcMAC=7c:7a:91:7d:7c:74 DstMAC=f4:ca:e5:4e:e1:46 EthernetType=IPv4 Length=0}
Layer 2 (20 bytes) = IPv4     {Contents=[..20..] Payload=[..528..] Version=4 IHL=5 TOS=0 Length=548 Id=18251 Flags= FragOffset=185 TTL=64 Protocol=ICMPv4 Checksum=39123 SrcIP=<IP_ADDRESS> DstIP=<IP_ADDRESS> Options=[] Padding=[]}
Layer 3 (528 bytes) = Fragment        528 byte(s)
Layer 4 (08 bytes) = ICMPv4   {Contents=[..8..] Payload=[..2000..] TypeCode=EchoRequest(0) Checksum=58624 Id=8864 Seq=1}
Layer 5 (2000 bytes) = Payload        2000 byte(s)

Decoding re-assembled packet: ICMPv4
PACKET: 562 bytes, wire length 562 cap length 562 @ 2015-06-13 00:20:14.950885 +0200 CEST

Layer 1 (14 bytes) = Ethernet {Contents=[..14..] Payload=[..548..] SrcMAC=f4:ca:e5:4e:e1:46 DstMAC=7c:7a:91:7d:7c:74 EthernetType=IPv4 Length=0}
Layer 2 (20 bytes) = IPv4     {Contents=[..20..] Payload=[..528..] Version=4 IHL=5 TOS=0 Length=548 Id=20433 Flags= FragOffset=185 TTL=55 Protocol=ICMPv4 Checksum=39245 SrcIP=<IP_ADDRESS> DstIP=<IP_ADDRESS> Options=[] Padding=[]}
Layer 3 (528 bytes) = Fragment        528 byte(s)
Layer 4 (08 bytes) = ICMPv4   {Contents=[..8..] Payload=[..2000..] TypeCode=EchoReply(0) Checksum=60672 Id=8864 Seq=1}
Layer 5 (2000 bytes) = Payload        2000 byte(s)

Hey cla/google, I am one person with two emails ! Both are in the CLA site, and my Github account !


Wrong postcondition in example

I couldn't help but note that the postcondition in https://github.com/codemix/contractual/blob/master/examples/src/both.js is probably wrong, given the preconditions do not prevent a<=0 or a===NaN or b===NaN or b<0 to be passed to the function. Was that a test? ;-)

@valioDOTch (sorry for the delay) yes you're right, I was never happy with the examples. This module is now deprecated, all the features (and more) are implemented in https://github.com/codemix/babel-plugin-contracts

I see...
Thanks for pointing that out and updating the Readme.md ^^


[TVMScript] Avoid visiting repetition tensor in SetCommonPrefix Visitor

This commit fix the issue about:
[Bug] [TVMScript] Progress hang on lowerTE during build dynamic Relay IR caused by endless ReprPrintTIR. https://github.com/apache/tvm/issues/15048

Hi, @Hzfengsy @junrushao . All test cases passed, please help to review this patch.

Yeah, you are right.


Add Skip Navigation link to the Website Header

Reason/Context
AsyncAPI website needs a skip link for bypassing or 'skipping' over repetitive web page content which is the navigation menu.
This will help screen reader users and keyboard-only users navigate to the main content of each page on the website. they can skip the navigation area and jump right into the main content of the page instead of repeating and going over the large navigation area again
The motivation for this is to improve the accessibility of AsyncAPI's website and be more inclusive.
Description
The main change is an addition of the skip-link code to the header/navigation section of the website and the styling for it in the css.
Below is an example of github skip-link.

@magicmatatjahu and @mcturco, I am tagging you folks in this a11y issue because it affects the /website repo code and styling.
@Mcnoble1, @magicmatatjahu helps as one of our main dev contributors to the /website repo, and @mcturco is one of our top design contributors, so these folks will prob need to be at a minimum, reviewers on the PR for this issue.

@magicmatatjahu and @mcturco, I am tagging you folks in this a11y issue because it affects the /website repo code and styling.
@Mcnoble1, @magicmatatjahu helps as one of our main dev contributors to the /website repo, and @mcturco is one of our top design contributors, so these folks will prob need to be at a minimum, reviewers on the PR for this issue.

Noted, Thanks.

Hey I can see that already a skip link is present in the website.

Uploading async API proposal.pdf…

@ChandniJha630 kindly don't upload your proposal over here as it is open for all and can be copied by anyone.

Hey I can see that already a skip link is present in the website.

Hey @pranavmadhu01, sorry for the late response. Actually, the button exists in the website but it doesn't work properly. So the motive of the issue is to implement the functionality of the button properly.

can you specify to which section should the button  redirect when navigated

can you specify to which section should the button redirect when navigated @akshatnema

It's not a redirecting button. It allows the keyboard only users to skip the navigation buttons of website and directly start pointing out to Read the Docs button.

ok sure i will make a pr

ok sure i will make a pr

Sorry, this issue is under the Winter of Code program, so the assigned mentee is allowed to contribute to the issue. Thanks for showing interest in this. You can look into other issues for contributing to the organization.

ok sure i will make a pr

Sorry, this issue is under the Winter of Code program, so the assigned mentee is allowed to contribute to the issue. Thanks for showing interest in this. You can look into other issues for contributing to the organization.

Iam a participant of Winter Of Code program and I will submit a proposal for the issue

sure

hey can I work on the issue?

hey can I work on the issue?
@akshatnema

Sorry, this issue is under Winter of Code, so the contributor whose proposal gets accepted will work in this.


Not seeing autocompletion

I am not seeing the autocompletion but I am seeing highlighted white warnings for Core library definitions if I don't use a .merlin file.

When I add PKG core to my .merlin file the highlighting goes away. So I assume it's working in that area. But still no autocompletion.
I do see autocompletion and snippets for builtin keywords for OCaml but it's kind of poor so I assume it's part of Sublime's standard package for OCaml and not this plugin working there either?
Also I am not seeing any type information in the status bar or warnings there either. Not sure if I am suppose to... But otherwise not much happens with this plugin versus without it except for the highlighting white borders for unknown functions.
What am I doing wrong?

Hi @hnry,
Can you share which OS you use, and the versions of sublime text, sublime-text-merlin, and merlin on your system?
Thanks!

$ opam show merlin
             package: merlin
             version: 2.3.1
   installed-version: 2.3.1 [4.02.3]

OS X 10.11.3
Sublime 3103
sublime-text-merlin is whatever the latest is from package control , not sure how to check. But I installed it last night, and re-installed just moments ago.

Installed Fedora 23 VM to try this out. I'm also not seeing autocompletions on there either.
Sublime 3103, also tried out Sublime 2 2221
merlin 2.3.1 [4.02.1]
In all 3 situations (OS X, Sublime 3103) (Fedora23 Sublime 3103) (Fedora23 Sublime 2221) I can not get autocompletion to work. They all report warning / syntax error highlighting for unknown symbols. They all correctly show errors when I don't have core pulled in by merlin (like the picture in the original post), and then remove the errors when I do have it pulled in. They all will show ocamlmerlin running in the background correctly.
Except in Fedora23 (Sublime 2 or 3) the jump to definition seems to work correctly. This does not work on OS X for me, it says it can't find the ML files.
But like I said no autocompletion...
And as noted previously I am able to get autocompletion working in atom just fine on my OS X system. (So I can't imagine it being an issue with my merlin install.)

Ok the problem is with whatever is on Package Control, when I clone this repo for install, it works.
Side note:
This does not feel right?
If I type "whi" while does not show up.
If I type "whil" while shows, but not if I start over the 2 "with" functions are gone?

Alright, I finally updated the plugin on Package Control. It should be available soon!
As for the autocompletion (while not showing up), it seems like it is the standard behavior of merlin. Indeed, Lis shows List but whi does not show while.

Merlin only report semantics information, not concrete syntax. So while is never reported by merlin.
It seems to be a sublime-text behavior to fall back to another completion source in absence of results. As for the results staying after you erased characters, this is again a sublime-text behavior (I would hazard a guess that it is over optimistic caching).


Firebase.readEvent() returns "type" instead "put" while getting data from Firebase

Hi,
I am new in Arduino coding and trying to learn how to control Node MCU 1.0 ESP12E using Firebase DB data.
I am not able to get the "put" event type, from Firebase.readEvent(), using Deprecated DB Secret.
I am able to get the "Streaming Success" message.
Please let me know if I have to change anything, in below code.
below is my code:
#include <ESP8266HTTPClient.h>
#include <FirebaseArduino.h>
#include <ESP8266WiFi.h>
// Set these to run example.
#define WIFI_SSID "SSID"
#define WIFI_PASSWORD "Password"
void setup() {
Serial.begin(115200);
// connect to wifi.
WiFi.begin(WIFI_SSID, WIFI_PASSWORD);
Serial.print("connecting");
while (WiFi.status() != WL_CONNECTED) {
Serial.print(".");
delay(500);
}
Serial.println();
Serial.print("connected: ");
Serial.println(WiFi.localIP());
Firebase.begin("xxxxxxxx.firebaseio.com", "xxxxxxxxxx");
Firebase.stream("/automation");
if (Firebase.success()) {
Serial.println("Streaming success");          //I am able to get the Success
}
}
void loop() {
if (Firebase.failed()) {
Serial.println("streaming error");
Serial.println(Firebase.error());
}
if (Firebase.available()) {
FirebaseObject event = Firebase.readEvent();
String eventType = event.getString("type");
eventType.toLowerCase();
 Serial.print("event: ");
 Serial.println(eventType);   // this return "type" instead of "put"
 if (eventType == "put") {     //If I write "type" instead" "put" then path is "/" and No data was returned.
   Serial.print("data: ");
   Serial.println(event.getString("data"));
   String path = event.getString("path");
   Serial.println(path);
   String data = event.getString("data");
 }

}
}

i have the same problem

I had the same problem and proposed the following fix: https://github.com/firebase/firebase-arduino/pull/374

Should be fixed with #374. Thanks @kptdobe !


Pairing HERO(2018) without PIN to PC-Wifi (solved)

Problem:
The Hero4 Commands describe a pairing with PIN entering. But in the Hero(2018) no Pin is needed. So no command is given in the DOC
Details:
I used PacketCapture for Android and scanned the WIFI Traffic from the GoPro App.
For Adding the Computer to the Device List in the GoPro(2018) the make folowing procedure:
GET: http://<IP_ADDRESS>/bacpac/cv
GET: http://<IP_ADDRESS>/gp/gpControl/command/wireless/pair/complete?success=1&deviceName=WIN10
as deviceName you can choose your own device name.
I tried it with the computer and it worked perfectly. Please add it to the docs.
nice regards
Gregor Schulze

https://github.com/KonradIT/goprowifihack/commit/2511520748a4818d049a1e37a4510f030b8bc43e


Support Custom Types API

Add methods to fetch Custom Type schemas via the Custom Types API.
These methods could be:

getCustomTypes
getCustomTypeByID
getCustomTypeByLabel

The result should match the API response, including the id and label, even if they were provided in the method call. By doing it that way, we can include auxiliary data, like repeatable.
interface CustomType {
  id: string
  label: string
  repeatable: boolean
  json: PrismicSchema
}

To properly support this, a new customTypesApiEndpoint option should be added to client instantiation, similar to the endpoint parameter. The endpoint is standard across all Prismic repositories, but there could be use cases where the endpoint must be customized.

Closing this issue in favor of creating a separate client for the Custom Types API.
See: https://github.com/prismicio/prismic-custom-types-client


GPL-915 Update 'Root Sample ID' in data stores

User story
Milton Keynes performed an investigation on their manual cherry picking and detected that 317 samples had the wrong root sample Id shared with us. They came up with a list of corrected samples IDs, which is attached.
RT: /Ticket/Display.html?id=713901
Who are the primary contacts for this story
Cristina A
Acceptance criteria
To be considered successful the solution must allow:

[ ] work out why querying Sequencescape sample_metadata based on root sample id / description gives 326 rows (rather than 317 (@KatyTaylor)
[ ] create an update script to update mongo
[ ] create an update script to update MLWH (@pjvv)
[ ] create script in SS rails console to update root sample id for all 317-ish (will also propagate to MLWH sample table)
[ ] create script to update Events WH for the incorrect events ?? (Beckman-fired events will include the root sample id in subject_friendly_name)

Dependencies
This story is blocked by the following dependencies:

#<issue_no.>
sanger/#<issue_no.>

Additional context
Add any other context or screenshots about the feature request here.

Script and log of when run here - https://ssg-confluence.internal.sanger.ac.uk/display/PSD/Root+Sample+Id+fix+March+2021