Edit Cloud-Init after first boot

Hi,
I don't know if this is even possible but I tried to edit the user-data under /boot after the very first boot of hypriot, but it didn't take effect...is there another way? Because I want to test multiple cloud-init possibilities without always set up a new machine...

Found it out! With the command: "sudo cloud-init clean" you can tell cloud init to run at next reboot =)

Thanks @bd8392 for sharing the solution!

@StefanScherer no problem :-) but be careful when using it. When I got it right, it doesn't revert the configs that the previous Cloud-init made, but applies the new ones over it...so you maybe get mixed up with settings when rerunning cloud-init...


Read location data from csv file and plot markers on map. Info screens open on click.

Primary: Derrill
Screenshot of map features: OGTQZ2n74Wz

Was unable to make the ImmutableList final and initialize in constructor. I got this error b7Q1BMJqBNK when I tried to do so.


Fix ServiceWorker registration failed

Fixes: #24490
I explain what I'm doing here:

I change the destination of 'workbox-sw' from 'assets/js/vendor' to 'assets/js/sw'. The change of destination is because I want to include the folder in 'keep_files' key of '_config.yml'.

I include './sw.js' in the files to exclude during build, since this file only serves as a template (it should not be copied to gh_pages, as it would overwrite the correctly generated file).

Then:

After docs-compile, the workbox files are copied and the manifest is generated in gh_pages.
During docs-serve, Jekyll deletes all files, but keeps the files needed for the service worker.

As the current Bootstrap documentation mentions, npm run docs must be run at least once for the service worker files to be generated.

Why can't we just keep the file in vendor?

Also, with your solution, how will the precache files be updated during the serve command?

Look at the changes @XhmikosR . This works both during npm run docs-compile and during npm run docs-serve.

I'm not sure I like it. Tying things more to Jekyll. Plus the Travis build doesn't even run for your PR so I'll need to check it out locally.
At any rate, I think, I'd rather skip this kind of solution and just live with the local warning, until we have a proper solution.


Update label logic to piggyback on XCActivityRecord

Motivation
By piggybacking our label() functionality on top of XCActivityRecord instead of _XCINFLog(), we can synchronously grab a screenshot of the device via the [XCAXClient_iOS sharedClient] instance.

Updated.


Add save and restore to file

Clarification and motivation
Hello, i'm interested in testing the performance of your cache in my application.  Currently I use theine.  Only thing I can see that I am missing from the API is the ability to save and restore the cache from file, so warm restarts are possible (very important for my app).
Acceptance criteria
Important that the file object is e.g. a generic io.ReadCloser, as we use afs (abstract file system) to use any cloud object store.
Thanks!

Hi, you're not the first with such a request, which clearly shows the need for this feature, but there are a few nasty problems.

By design, otter was supposed to be responsible for in-memory caching and do it as best as possible. Interaction with files is unfortunately not included in this domain. But okay, we can make an exception.

Theine saves not only keys, values, ttls and costs, but also meta information about eviction policy. I'm not willing to allow this, because it obliges the library to guarantee that it will never change the structure of its eviction policy in the future, because changing it would cause very nasty errors when updating the library. And otter is extremely likely to make changes to the eviction policy.

Because of the previous two points, I was betting on the Range function, planning to just prompt users to save key-value pairs to a file themselves and load the cache from the file, but I'm not sure about this solution and am open to discussion.

Basically, you can continue to use theine. It is written quite well. There are only a few things I don't like about it:

poor cost synchronization, which can lead to panics when using cost-based eviction
high per entry memory overhead, which is somewhat hidden by the excellent implementation of count-min sketch (not important for the vast majority of applications). The memory consumption analysis I wrote a while back

And just for fun, why is cold start so scary to you?

Hi @maypok86 , actually having the range function is all I need really, I missed that.  I just need the keys and values to be restorable, ttls and costs etc can be recreated.
For my app cold (re)starts can be problematic because I use the cache to store results of particularly expensive DB queries that can be eventually consistent in a highly concurrent system.  If all workers are simultaneously restarted without caches, the DB will have a masssive load spike which can hurt performance and stability.
So I think this request just becomes: perhaps it is worth having a convenience function that ranges over the cache to directly produce the file.  Your call really, i'm happy to do that on my side.

I guess I won't add saving to the file for now after all. It seems better to let users do it the way they want.
Maybe it will be added, but only after otter closes all the problems with its domain. Which will be much later :)


ci: Add GraphQL schema syncing job

Stops the schema from becoming out-of-date over time.
Test Plan
n/a

@bobheadxi I've tagged you in case you have better ideas on how to accomplish this. 😅

The alternative would be to have sourcegraph/sourcegraph trigger an update, e.g. what @davejrt did for Zoekt: https://github.com/sourcegraph/zoekt/blob/master/.github/workflows/pr.yml
If this works it works though :)

Oh yeah, that seems like a better approach, because that does it on push, instead of a periodic basis. It also uses auto-merging instead of requiring manual approval.


[11.x] Prevent implicit route model binding parameter name mismatches in controllers

This PR is based on a question I asked on X, where I somewhat frequently scratch my head as to why a route model binding isn't working and a new model instance is tossed in via dependency injection. It's obvious when you show the routes together with the controller method (as you'll see below), but it's an easy mistake to make when you're in one file working with models named two or more words (PhoneNumber, CompanyReservation, ActivityLog, etc.):
https://twitter.com/ste_bau/status/1788976209943986419
Route::name('show')->get(
    'phone-numbers/{phoneNumber}',
    [PhoneNumberController::class, 'show']
);

// ...

public function show(PhoneNumber $number) // Named incorrectly - new instance created
{
    $number->exists; // false ❌
}

public function show(PhoneNumber $phoneNumber) // Named correctly - existing model instance given
{
    $phoneNumber->exists; // true ✅
}

This PR implements a way to enable throwing an exception when a model (similarly to Model::shouldBeStrict()) binding is missed:
\Illuminate\Routing\ImplicitRouteBinding::preventModelParameterMismatch();

Then, then the first scenario above occurs, developers will immediately see the exception with clarity:
public function show(PhoneNumber $number) // ❌ throws ModelParameterMismatchException
{
    // ...
}

ModelParameterMismatchException: Route parameter name 'number' does not match expected model binding name 'phoneNumber'.

This PR doesn't affect nullable controller parameters:
public function show(PhoneNumber $number = null); // ✅

It also does not affect explicit model bindings (as these are performed prior to implicit):
Route::model('number', PhoneNumber::class); // ✅

Let me know your thoughts! No hard feelings on closure ❤ .

I like the PR, nice work! It can definitely be very confusing that the container sometimes generates empty Eloquent models that don't exist in some places that are implicit. I would even argue that this setting could go on the container, as a way of telling the container "never instantiate an empty non existent Eloquent model". The chances that instantiating such a model is erroneous are much higher than that a developer is actually doing app(User::class)->fill(...)->save() to name sth. Everyone would call Eloquent models statically User::create(...).

I loved this PR. I have personally bitten by it several times.

This should definitely exist. Sometimes, you get trapped in this situation, and after hours of debugging, you realize it was just a wrong naming..

It should be added. At first, I didn’t know the name needed to be the same! I thought the model name would be enough. After a few incidents, I just stopped using route model bindings.

Should this be tied in with Model::shouldBeStrict() I feel like it's related.

This might be of interest: I'm using spatie/route-attributes, which mostly avoids this issue by having the route declaration next to your controller method declaration

What is the current status of this PR?

How does it know that the incorrect variable name is not what the user actually wants?  While I would say it's very unlikely the user intentionally mis-named the variable, this feels like it could cause some unexpected or unwanted issues.

@browner12 It knows because after enablement, it makes the assumption that you will only want existing model instances to be provided in controller endpoint methods. If you want a new model instance to be provided, you must be explicit.
For example, this would throw a ModelParameterMismatchException exception:
public function store(PostRequest $request, Post $post)
{
    // ...
}

And this would not:
public function store(PostRequest $request, Post $post = new Post)
{
    // ...
}


when I compile mxnet with cmake, an error occured

CMake Warning at D:/opencv3/build/OpenCVConfig.cmake:166 (message):
Found OpenCV Windows Pack but it has no binaries compatible with your
configuration.
You should manually point CMake variable OpenCV_DIR to your build of OpenCV
library.
Call Stack (most recent call first):
CMakeLists.txt:59 (find_package)
CMake Error at CMakeLists.txt:59 (find_package):
Found package configuration file:
D:/opencv3/build/OpenCVConfig.cmake

but it set OpenCV_FOUND to FALSE so package "OpenCV" is considered to be
NOT FOUND.

You are probably using a 32bit OpenCV and trying to compile a 64bit mxnet,
or vice versa. Make sure your OpenCV binary fits your chosen platform.
On Thu, Dec 8, 2016 at 10:24 AM, poorfriend<EMAIL_ADDRESS>wrote:

CMake Warning at D:/opencv3/build/OpenCVConfig.cmake:166 (message):
Found OpenCV Windows Pack but it has no binaries compatible with your
configuration.
You should manually point CMake variable OpenCV_DIR to your build of OpenCV
library.
Call Stack (most recent call first):
CMakeLists.txt:59 (find_package)
CMake Error at CMakeLists.txt:59 (find_package):
Found package configuration file:
D:/opencv3/build/OpenCVConfig.cmake
but it set OpenCV_FOUND to FALSE so package "OpenCV" is considered to be
NOT FOUND.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/4146, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABFI4QsZI0UPMatZDE5bKszJGfUEhxgQks5rF2p4gaJpZM4LHXjT
.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Adding 80RATE/20WETH pool to weighted allowlist

Description
Please include a summary of the change and if relevant which issue is fixed. Please also include relevant motivation and context.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[ ] Other

How should this be tested?
This PR is adding the 80RATE/20WETH  pool to the weighted pool allow list. Test by ensuring the UI allows provisioning liquidity to it.
Please provide instructions so we can test. Please also list any relevant details for your test configuration.

[x] Test A: Add liquidity to the 80RATE/20WETH pool, assert all works as intended.

Visual context
Please provide any relevant visual context for UI changes or additions. This could be static screenshots or a loom screencast.
Checklist:

[x] I have performed a self-review of my own code
[x] I have requested at least 1 review (If the PR is significant enough, use best judgement here)
[x] I have commented my code where relevant, particularly in hard-to-understand areas
[x] If package-lock.json has changes, it was intentional.
[x] The base of this PR is master if hotfix, develop if not

Hello sirs, opening this PR to whitelist the RATE/WETH pool to the UI. Kindly request a review @garethfuller , @timjrobinson . Let me know if any questions. RATE is the governance token for TAI (tai.money), the goal is to bootstrap liquidity for it. Thank you!

Thank you @garethfuller !


DirectionalLight missing directional_shadow_max_distance in editor

Godot Engine v3.2.2.stable.official
To change maximum directional_shadow_max_distance you have to script it's value.

I can see it on Godot 3.2.2:

Oh! Why it is not in Shadow's tab?

It's not in the Shadow tab because it's a property specific to DirectionalLight. (DirectionalLight, OmniLight and SpotLight all inherit from a base Light class.)


Added Table of Contents in the Readme file

Description
Added Table of Contents section to the README file to enhance navigation and provide a more user-friendly experience for end users.
Fixes: #15

@Devasy23 I have added the Table of Contents section under #15 issue. Please merge it.


Delete "user left" notifications from #lobby

The so and so "left" the channel messages aren't currently deleted by @prophetorpheus.
Demo: https://youtu.be/n0B2lmgX81I?t=2m10s

https://www.youtube.com/watch?v=n0B2lmgX81I&feature=youtu.be&t=3m42s

^ that's the correct timestamp to see the issue.

We don't have a lobby channel now, I don't think this is relevant anymore


Install issues , transformer module, involves pandas

Is there an existing issue for this?

[X] I have searched the existing issues and found nothing

Bug description
Tested two installation, following the description in README. Windows 10 and have problems with molfeat.trans, similar, yet different errors:
WSL Ubuntu 22: using mamba, and a new environment, Py3.10  the import attempt for from molfeat.trans import MoleculeTransformer leads to following error shown in error window.
Did a  pip ... [all] just in case, but get the same error.
On a Windows conda with py3.10 I also get an error for Pandas, but not equally "bad", the code runs ok, but still gives error (or, well, warning) when I import.
upon from molfeat.trans import MoleculeTransformer I "only" get this:
Failed to find the pandas get_adjustment() function to patch
Failed to patch pandas - PandasTools will have limited functionality

then after the line of the example code mol_trans(data):
Failed to find the pandas get_adjustment() function to patch
Failed to patch pandas - PandasTools will have limited functionality
....multiple times...

How to reproduce the bug
WSL Windows 10 Ubuntu 22 install. Fresh conda installation, mamba ontop of that, creation of new environment, then followed the instructions here, then run the example code.
The Win10 environment: conda install (no mamba) and pip directly.

Error messages and logs
Error from WSL Ubuntu
 File "<stdin>", line 1, in <module>
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/molfeat/trans/__init__.py", line 1, in <module>
    from molfeat.trans.concat import FeatConcat
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/molfeat/trans/concat.py", line 15, in <module>
    from molfeat.trans.fp import FPVecTransformer
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/molfeat/trans/fp.py", line 12, in <module>
    from molfeat.trans.base import MoleculeTransformer
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/molfeat/trans/base.py", line 31, in <module>
    from molfeat.utils.cache import _Cache, FileCache, MPDataCache
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/molfeat/utils/cache.py", line 35, in <module>
    class MolToKey:
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/molfeat/utils/cache.py", line 39, in MolToKey
    "dm.unique_id": dm.unique_id,
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/datamol/__init__.py", line 186, in __getattr__
    mod = importlib.import_module(obj_mod)
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/datamol/mol.py", line 33, in <module>
    from .convert import to_inchikey_non_standard
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/datamol/convert.py", line 15, in <module>
    from rdkit.Chem import PandasTools
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/rdkit/Chem/PandasTools.py", line 653, in <module>
    InstallPandasTools()
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/rdkit/Chem/PandasTools.py", line 622, in InstallPandasTools
    PandasPatcher.patchPandas()
  File "/home/a/mambaforge/envs/molstuff/lib/python3.10/site-packages/rdkit/Chem/PandasPatcher.py", line 263, in patchPandas
    if getattr(pandas_formats.format, get_adjustment_name) != patched_get_adjustment:
AttributeError: module 'pandas.io.formats.format' has no attribute 'get_adjustment'

Environment

Current environment
#- Molfeat version (e.g., 0.1.0):
#- PyTorch Version (e.g., 1.10.0):
#- RDKit version (e.g., 2022.09.5):
#- scikit-learn version (e.g.,  1.2.1):
#- OS (e.g., Linux):
#- How you installed Molfeat (`conda`, `pip`, source):

Additional context
No response

We had a similar issue in Polaris! This seems to be an upstream issue. get_adjustment() was moved to pandas.io.formats.printing in this PR. It was fixed in RDKit in this PR.
I believe this could be fixed by either setting an upper limit for pandas <2.2.0 or by setting a lower limit for rdkit > 2023.09.6. I am not sure which one should be preferred.

Since these two releases are fairly recent, you probably break less by setting an upper limit for now?


Automated Latest Dependency Updates

This is an auto-generated PR with latest dependency updates. Please do not delete the latest-dep-update branch because it's needed by the auto-dependency bot.

Closed by #2678


Client Routing not working

on upgrade-react-router example, client-side routing not working.

Thanks for letting me know @ParishKhan
I fixed it by updating package.json and saving all dependencies to its exact version. The error was coming from<EMAIL_ADDRESS>being installed instead of<EMAIL_ADDRESS>Feel free to pull and install to get the app working again.

Client-side routing still not working.


Mac version always opens file with default app instead of DBeaver

Whenever I use File -> Open and try to open an SQL file, DBeaver doesn't open it itself but triggers the default app associated with this file type on the OS level.
I would expect that DBeaver always opens files itself.

Seems to be MacOS specific

Actually I can't reproduce this on MacOS. It opens all unrecognized file types using system default application but *.sql files should be opened by DBeaver itself.
Moreover it is configurable in preferences. Please check it:


WFLY-19232 Update ElytronOidcSubsystemSchema to only add the Scope attribute to Preview:2.0 schema version or later

Issue: https://issues.redhat.com/browse/WFLY-19232

More information about the wildfly-bot[bot]

Thanks @PrarthonaPaul and @fjuma


Multiple improvements to performance and responsiveness of restreamer

when requesting the same very large playlist over and over.
Main highlights:

generating the hls playlist is now a streaming result
if multiple calls for the same time range come in concurrently, we only run it once
we yield to the event loop in more places to avoid locking up other requests

Doing clean up of open PRs. I did some self-review, and I figure that "me, 5 months later" is sufficiently removed to count as a second set of eyes.


Added output option to process (and rename) single file

"-o/--output" option allows to preprocess a single file and provide the output filename.

launch jenkins


tablineat() support for ctrlspace tabline

This request adds clickable buffers and tabs when the CtrlSpace variant of the tabline is used.

thanks!


Segfault When Initiating HDFS Connection

Hi,
I'm getting a segfault when trying to create a connection using the HDFileSystem constructor. The code that I'm running is:
from hdfs3 import HDFileSystem
hdfs = HDFileSystem([HOSTNAME], port=[PORT-NUM])

(with HOSTNAME and PORT-NUM filled in, ofc.)
When I run the script through GDB (for debugging purposes), I get:
Starting program: /home/jovyan/.conda/bin/python hdfs-test.py
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".

Program received signal SIGSEGV, Segmentation fault.
strlen () at ../sysdeps/x86_64/strlen.S:106
106	../sysdeps/x86_64/strlen.S: No such file or directory.
(gdb) backtrace
#0  strlen () at ../sysdeps/x86_64/strlen.S:106
#1  0x00007ffff3ff83ee in std::char_traits<char>::length (__s=0x1 <error: Cannot access memory at address 0x1>)
   from /home/jovyan/.conda/lib/python3.6/lib-dynload/../../libhdfs3.so
#2  std::string::assign (__s=0x1 <error: Cannot access memory at address 0x1>, this=0x555555c27fa0)
    at /feedstock_root/build_artefacts/libhdfs3_1526441785297/work/libhdfs3/src/client/FileSystem.cpp:1131
#3  std::string::operator= (__s=0x1 <error: Cannot access memory at address 0x1>, this=0x555555c27fa0)
    at /feedstock_root/build_artefacts/libhdfs3_1526441785297/work/libhdfs3/src/client/FileSystem.cpp:555
#4  Hdfs::FileSystem::FileSystem (this=0x555555c27fa0, conf=..., euser=0x1 <error: Cannot access memory at address 0x1>)
    at /feedstock_root/build_artefacts/libhdfs3_1526441785297/work/libhdfs3/src/client/FileSystem.cpp:148
#5  0x00007ffff400f117 in hdfsBuilderConnect (bld=0x555555bf8c40,
    effective_user=0x1 <error: Cannot access memory at address 0x1>)
   from /home/jovyan/.conda/lib/python3.6/lib-dynload/../../libhdfs3.so
#6  0x00007ffff6607ec0 in ffi_call_unix64 () from /home/jovyan/.conda/lib/python3.6/lib-dynload/../../libffi.so.6
#7  0x00007ffff660787d in ffi_call () from /home/jovyan/.conda/lib/python3.6/lib-dynload/../../libffi.so.6
#8  0x00007ffff681cdee in _ctypes_callproc ()
   from /home/jovyan/.conda/lib/python3.6/lib-dynload/_ctypes.cpython-36m-x86_64-linux-gnu.so
#9  0x00007ffff681d825 in PyCFuncPtr_call ()
   from /home/jovyan/.conda/lib/python3.6/lib-dynload/_ctypes.cpython-36m-x86_64-linux-gnu.so
#10 0x00005555556631bb in _PyObject_FastCallDict ()
#11 0x00005555556f0d3e in call_function ()
#12 0x000055555571519a in _PyEval_EvalFrameDefault ()
#13 0x00005555556ea7db in fast_function ()
#14 0x00005555556f0cc5 in call_function ()
#15 0x000055555571519a in _PyEval_EvalFrameDefault ()
#16 0x00005555556e99a6 in _PyEval_EvalCodeWithName ()
#17 0x00005555556eb108 in _PyFunction_FastCallDict ()
#18 0x000055555566339f in _PyObject_FastCallDict ()
#19 0x0000555555667ff3 in _PyObject_Call_Prepend ()
#20 0x0000555555662dde in PyObject_Call ()
#21 0x00005555556bdf6b in slot_tp_init ()
#22 0x00005555556f0f27 in type_call ()
#23 0x00005555556631bb in _PyObject_FastCallDict ()
#24 0x00005555556eacfa in _PyObject_FastCallKeywords ()
#25 0x00005555556f0d3e in call_function ()
#26 0x0000555555715eb1 in _PyEval_EvalFrameDefault ()
#27 0x00005555556eb529 in PyEval_EvalCodeEx ()
#28 0x00005555556ec2cc in PyEval_EvalCode ()
#29 0x0000555555768af4 in run_mod ()
#30 0x0000555555768ef1 in PyRun_FileExFlags ()
#31 0x00005555557690f4 in PyRun_SimpleFileExFlags ()
#32 0x000055555576cc28 in Py_Main ()
#33 0x000055555563471e in main ()

I installed hdfs3 via conda (conda install -c conda-forge hdfs3).
Here are the package versions installed by conda-forge:
    boost-cpp:    1.66.0-1         conda-forge
    bzip2:        1.0.6-1          conda-forge
    curl:         7.59.0-1         conda-forge
    hdfs3:        0.3.0-py36_0     conda-forge
    icu:          58.2-0           conda-forge
    krb5:         1.14.6-0         conda-forge
    libgcrypt:    1.8.2-hfc679d8_1 conda-forge
    libgpg-error: 1.31-hf484d3e_0  conda-forge
    libgsasl:     1.8.0-2          conda-forge
    libhdfs3:     2.3.0-2          conda-forge
    libiconv:     1.15-0           conda-forge
    libntlm:      1.4-1            conda-forge
    libprotobuf:  3.5.2-0          conda-forge
    libssh2:      1.8.0-2          conda-forge
    libuuid:      1.0.3-1          conda-forge
    libxml2:      2.9.8-0          conda-forge

I don't think there's anything wrong with the Hadoop setup, since I'm still able to access it using hadoop fs commands.
Any thoughts on what could be going wrong?

libhdfs3-2.3.0-2 was just released, can you try with libhdfs3-2.3.0-1 and see if this a new problem?
conda install -c conda-forge libhdfs3=2.3.0=1

@sk1p , ideas?

@martindurant that worked! Ran this in a Jupyter Notebook and things are working again: ! conda install -c conda-forge libhdfs3=2.3.0=1 hdfs3 --yes. Hopefully they're able to get that fixed before the issue pops up for more people.

Thanks for testing, @lucashu1

Thanks for tagging me. That's weird: effective_user=0x1 - do you have anything related to effective_user in your configuration file?

@sk1p Nope, not seeing any instances of effective_user in /etc/hadoop, which should contain the Hadoop config files.

If I remember, effective_user arises in situations something like you have logged in with a kerberos principal other than your local username. Are you using kerberos?

Since the last release was 9 months ago, a whole load of versions have been updated, but the latest build release was with the older code and newer versions, so that is likely not the problem. I think that there must still be bad code from the big merge https://github.com/ContinuumIO/libhdfs3-downstream/pull/3 which is beyond my c++ ability. You'll notice that there were conversations around how to make that right, but it was never used in a conda build until now (the recipe still pointed to the concat branch)

I got something the same issue with a manual install, on master.
Here are the complete steps to build latest libhdfs3 master (commit 41fd50d from May 7, PR#12) on fedora 28 and reproduce this (I don't believe an actual kerberos connection is needed, just the standard hdfs-client with auth set to kerberos):
dnf install -y krb5-workstation protobuf-devel gtest-devel gmock-devel libcurl-devel libgsasl-devel libgasl
git clone https://github.com/ContinuumIO/libhdfs3-downstream
cd libhdfs3-downstream/libhdfs3
mkdir build
cd build
../bootstrap --prefix=/usr/local
make -j5
sudo make install
export LD_LIBRARY_PATH=/usr/local/lib:$LD_LIBRARY_PATH
pip3 install --user hdfs3  # 0.3.0
gdb python3
import hdfs3 ; f = hdfs3.HDFileSystem('some.net', port=8020)

Thanks to @martindurant I was able to work around it by building the following libhdfs3 commit:
git checkout 7842951    # august
git cherry-pick 069d774 # gcc-7

(then same steps as above, it should at least get you to point where it tries to connect and fails with  ERROR Failed to setup RPC connection to "some.net:8020")

I could not reproduce this when I used a debug build. That may mean there is unitialized variable

It is probably important to be sure the python hdfs3 code is using:
hdfsBuilderConnect.argtypes = [ct.POINTER(hdfsBuilder), ct.c_char_p]
I also could not reproduce this with non debug build. I noticed the connect call looks like:
fs = _lib.hdfsBuilderConnect(o)
instead of:
fs = _lib.hdfsBuilderConnect(o, ensure_bytes(self.effective_user))
so there might be incompatability between python lib and C++ lib

Here is the line from the python side, having the extra argument https://github.com/dask/hdfs3/blob/master/hdfs3/core.py#L147 , but this is not in the released conda/pip package.

might be why there are issues if people are using older pip package without those changes.

I was definitely using pypi latest, v0.3.0 as stated above
On Fri, 15 Jun 2018, 15:33 Brett Rosen<EMAIL_ADDRESS>wrote:

might be why there are issues if people are using older pip package
without those changes.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/hdfs3/issues/163#issuecomment-397621357, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACpOGSDQbwkQFQAQCtUa6gbUIz0mdid1ks5t87engaJpZM4UCIqM
.

I don't think so, the libhdfs3 conda package pull from my repo. not from ContinuumIO's for this reason; I could not get a build of libhdfs3 that worked reliably on the various systems, and that's why hdfs3 has remained unreleased since. It would be great, but apparently beyond my ability without considerable investment of time.
Since arrow's libhdfs implementation now does most of the job, there is less incentive to push on here.

Do you get same error if you use the unreleased version from github instead of pypi? By older I meant the version that was out of sync with C++ changes.

conda install -c conda-forge libhdfs3=2.3.0=1 hdfs3 --yes
fails to install proper version of libcrypto.so.1.0.0
this shows up in ldd $minicondda_home/lib/libhdfs3.so

Sorry, @petacube - the build difficulties here are indeed what led to a stop in the efforts on our part to try to get everything smooth. Arrow's hdfs does not suffer from them, so long as you have the java-native libraries present (which is normally true on any hdfs edge-node). For connection from outside the cluster, I would not recommend my implementation of webhdfs or one of the other python implementations out there.

@martindurant i see..
so the webhdfs requires hdfs admin to enable/install webhdfs interface on hdfs machines right? or can it be used against plain hdfs cluster without any changes?

Right, webhdfs needs to be enabled, and may need its own kerberos principal, etc. That's fairly common, though. You probably do not need direct access to the data nodes, if a proxy has been suitably setup - but there are many complicated options there.

can it be installed in ubuntu 18.0.4..if so how? ty

@HarshaRagyari , have you tried pip? Generally I would recommend basing your whole python environment on conda, but I understand there are a range of user cases. A debian package called python-fsspec exists, that might be available on ubuntu and might be recent enough for you - I don't know the details on that.


Typescript support?

Does redux-orm have typescript support?

There are type definitions for it here: https://www.npmjs.com/package/@types/redux-orm

See also #168.


updated Microsoft Azure credit card requirement

Credit card info is required to create the free account

@varsubham thanks a lot for your PR and clarification


Update landscape.yml

Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[ ] Have you picked the single best (existing) category for your project?
[ ] Does it follow the other guidelines from the new entries section?
[ ] Have you added your SVG to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[ ] Does your project/product name match the text on the logo?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
No cached entry, and Sangfor EDS has issues with logo: sangfor.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/sangfor.svg'
Empty twitter for Mail.Ru Cloud Solutions (member): https://twitter.com/mailru
Empty twitter for Mail.Ru Cloud Containers: https://twitter.com/mailru
No cached entry, and Sangfor EDS has issues with twitter: https://twitter.com/SangforUSA, Request failed with status code 404
Empty twitter for Spotinst Functions: https://twitter.com/spot_hq
Empty twitter for Karmada: https://twitter.com/karmada_io
Item Sangfor EDS has no image_data


Idea: Seaweed at bottom of page

I could put seaweed at the bottom of the page to give it more of an oceanic feel. idk how complicated that would be tho considering I would want it to move about

Done :+1:


feat: adding support for new products in Checkpoint Log Exporter R81.20

Added support for new products generated from Checkpoint Log Exporter R81.20

Product
New source

Check Point GO Password Reset
checkpoint:audit

Database Tool
checkpoint:audit

FG VPN-1 & FireWall-1
checkpoint:firewall

QoS
checkpoint:firewall

cpmidu_update_tool
checkpoint:audit

query-database
checkpoint:audit

Anti Phishing
checkpoint:email

Anti-Spam and Email Security
checkpoint:email

:tada: This PR is included in version 3.17.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat(gkehub): generate library

fixes #8010

This change is

Sorry, I snuck #8013 in ahead of this PR. Can you regenerate the *_connection.cc file here and in the retail PR? I think generate-libraries-ci will break if we don't do this.

Thanks. I am happy. I will let @coryan have the final say


Switch default load balancing backend to ALB

I'm going to tag this for the 1.0 release, but may or may not end up there.
Benefits

http/2
Host port mappings are managed dynamically, and allows ECS to schedule multiple web procs on the same instance.

Considerations

http/https only. No tcp/ssl listeners.
http only health checks. We should document recommendations for default health checks and/or add health check configurations to the extended procfile.
ALB's don't inherit tags from the stack :(

I'll close this for now. Switching entirely to ALB is not a clear win, and there's various tradeoffs and considerations (e.g. http/2 is not supported from ALB -> backend).
I think the right thing to do is to continue to support both ELB and ALB.


#20935 do not use old URL pattern

Please check the BigQuery connection in PRO versions (cloud and usual new connection)

verified for cloud and usual connections with default credentials and key file


1.8.0 {results:[]} returning empty

Steps to reproduce
const google = require('googlethis');
const response = await google.search('googlethis', {page: 0});
console.log(response);
Failure Logs
{"results":[],"videos":[],"knowledge_panel":{"type":null,"title":null,"description":null,"url":null,"metadata":[],"books":[],"tv_shows_and_movies":[],"ratings":[],"available_on":[],"images":[],"songs":[],"socials":[],"demonstration":null,"lyrics":null},"featured_snippet":{"title":null,"description":null,"url":null},"did_you_mean":null,"weather":{"location":null,"forecast":null,"precipitation":null,"humidity":null,"temperature":null,"wind":null,"image":null},"time":{"hours":null,"date":null},"location":{"title":null,"distance":null,"map":null},"dictionary":{"word":null,"phonetic":null,"audio":null,"definitions":[],"examples":[]},"translation":{"source_language":null,"target_language":null,"source_text":null,"target_text":null},"top_stories":[],"unit_converter":{},"people_also_ask":[],"people_also_search":[]}

Expected behavior
I am expecting the {results[]} to return an array of webpages consist of urls, descriptions and favicons of the webpages.`
Current behavior
It returned empty {results: []}
Version
Default
Anything else?
No response
Checklist

[X] I am running the latest version.
[X] I checked the documentation and found no answer.
[X] I have searched the existing issues and made sure this is not a duplicate.
[X] I have provided sufficient information.

Same error here. It was working perfectly but this week it started to return an empty list

I can see only the knowledge panel and the "people also ask" section. Everything else appears to be empty/null.

Have done a quick hack replacing SELECTORS with below in constants.js since Google has done some changes.
SELECTORS: { // Organic Search Results TITLE: 'div.q8U8x.oewGkc.LeUQr', DESCRIPTION: 'div.MUxGbd.yDYNvb', URL: 'a.C8nzq.BmP5tf',
This is for if you are just scraping the pages. For other elements, obviously have to do deeper research and do changes.

Thanks it works!

Have done a quick hack replacing SELECTORS with below in constants.js since Google has done some changes.
SELECTORS: { // Organic Search Results TITLE: 'div.q8U8x.oewGkc.LeUQr', DESCRIPTION: 'div.MUxGbd.yDYNvb', URL: 'a.C8nzq.BmP5tf',
This is for if you are just scraping the pages. For other elements, obviously have to do deeper research and do changes.

how can i change the file and work?pls.

This fix worked for me #73
Updated description selector (Constants.SELECTORS.DESCRIPTION) in file /lib/utils/constants.js to div[data-snc] > div:nth-child(2) > div

Nothing worked for me

The issue is related to changed selectors, try my fork - https://github.com/CROSP/google-this/tree/main
Or use the following selectors by directly changing the contants.js file
  SELECTORS: {
    // Organic Search Results
    TITLE: 'a[role="presentation"][ping][data-ved] div[aria-level="3"][role="heading"]',
    DESCRIPTION: 'div[data-sncf="1"] div',
    URL: 'a[role="presentation"][ping][data-ved]',


Clear All Filters does not reset the sort by option

This didn't seem intuitive to me. Is this okay this way or should it reset the sort by option too?

tagged with wishlist and move this to recordset feature list.


Error installing Realm with CocoaPods

I alway get this error when trying to install Realm with CocoaPods.
My XCode version is 6.2 but it didn't work with 6.1.1 either.
Installing Realm (0.91.1)
[!] /bin/bash
set -e
sh build.sh cocoapods-setup
: command not found
: command not found
: invalid option namet: pipefail
: invalid option4: set: -
set: usage: set [--abefhkmnptuvxBCHP] [-o option] [arg ...]

Installing other cocoapods (like AFNetworking) works fine

What version of CocoaPods are you running?

I'm running the latest version of cocoapods 0.36.0, but the problem also existed with the previous version 0.35.0

I can't seem to reproduce this particular error... Does it happen consistently for you?

Yes, I have tried it several times to install it also with other projects and I always get this error.

Realm uses Bash's set -o pipefail as part of its installation script, which was introduced in Bash 3.0 (released 10 years ago). So I doubt that this is failing because your bash version is too old.
Is it possible that you have a custom shell installed (zsh, fish, etc) and that the Realm installation script is using that shell instead of system bash?

Also, what is the output of which sh on your machine?

No I'm using the original shell installed with the system
bash-3.2$ bash --version
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin14)
Copyright (C) 2007 Free Software Foundation, Inc.
bash-3.2$ which sh
/bin/sh

I'm at a loss for other possible causes of this problem. Are there any other aspects of your system's configuration that you feel may be relevant? OS version, ruby version, etc? We won't be able to fix this until we can reproduce it on some machine where we can troubleshoot.

@beowulfdk I'm going to close out this issue, since we haven't been able to reproduce the issue nor have we seen other reports of this happening. If there's any further information that might help us track this down, we'll re-open the issue. Thanks!

I have this is issue with realm 1.1.0


Panic less with S3

This PR turns many s3 unwrap() calls into try!(). We were running into panics (which we couldn't handle, unlike errors) due to intermittent network failures.

Tests succeeded locally. Overflow appears to be a recurring issue right now?

Yup, overflow is https://github.com/rusoto/rusoto/issues/302 .  Haven't gotten to wrestling with that one yet.
Thanks for the PR, will review tonight.

Since the try blocks return different errors, they aren't automatically converted into our AwsError type.  You can see the compiler error when you run cargo build --features s3.
I think the simplest solution is to add a conversion function for AwsError.  We've got some existing ones in https://github.com/rusoto/rusoto/blob/master/src/error.rs .  As an example, you could add this for the ParseBoolError type:
use std::str::ParseBoolError;

(snip)

impl From<ParseBoolError> for AwsError {
    fn from(err: ParseBoolError) -> AwsError {
        AwsError::new(format!("{}", err))
    }
}

The AwsError class will also need std::num::ParseIntError added in a similar manner.
Let me know if you want a hand making these changes.

👍 thanks!

Thank you!


Exposure notifications - reading from the stream in a loop

Hi, stream.read(byte[] buffer) does not make an effort to read exactly buffer.size bytes, it just tries to read at least 1 byte, and may return with a smaller number of bytes read:
https://docs.oracle.com/javase/7/docs/api/java/util/zip/ZipInputStream.html#read(byte[], int, int)
This was a bug also present in original Google implementation, which resulted in header validation error on some devices.

Thanks!


403 forbidden on existing shortened links

I've been using YOURLS for years and recently experienced an issue where I suddenly couldn't log in using admin and existing shortened links stopped resolving.
After some investigation it seemed that I needed to change my database SQL hostname in config.php to an internal hostname for scripts running on our webserver which then allowed me to log in as admin.
However existing links still do not work - new ones do.  I cannot edit links and get the YOURLS error message - Error, URL not found when I attempt to.  The original link however is valid and original open when clicked (just not the shortened link).
I've tried the three database options as suggested in troubleshooting first steps and turned on debugging.  I'm getting nothing back and changing database engines hasn't made any difference.
I'm stumped.  We have hundreds of links that no longer work and am inundated with complaints from customers trying to use them.
I'd be really grateful if anyone could point me in the right direction to resolve this.
Steps to reproduce the behavior:

Go to any existing shortened link
See error (screenshot)

Expected behavior
To be able to edit, delete, share and visit preexisting shortened links.
Please note that NEW shortened links are fine and behave as expected.
Screenshots
Error when attempting to edit a pre-existing link

Error when attempting to delete a preexisting link:

Environment (please complete the following information):

YOURLS version: 1.7.2

Plugins enabled: none

PHP version: 7.1

Thanks in advance of any help you can give me.

After some investigation it seemed that I needed to change my database SQL hostname in config.php

Get in touch with your host. This makes it sound like they changed something about database hosting and perhaps something went wrong during the data migration.
Based on the information provided, I presently doubt that it's an issue with YOURLS.

I have and they claim the hosting is fine and the database is accessible by phpmyadmin where I can perform edits, deletes etc. Which is true.  Because only scripts relating to YOURLS are not working they told me to check my files.
Any html or php that is not related to YOURLS works fine.
If I delete a record using phpmyadmin and then create a new link using YOURLS then everything is good but I've got thousands and can't do that every one of them.

After my updates and installs my plug in for hypens in short links needed reactivating!  As most of our links have hyphens this made it appear that none work working and could not be edited.


Voiceover iOS Safari - Not focusing modal

The screen reader is not focusing the modal when there is a <select> in it.
Example: https://codesandbox.io/s/pkzl2v2mkj

Hey @DarudeGarystorm, there are some updates released on 3.6.1. Can you please check if it helps?

@diasbruno bug is still present on 3.6.1

There's a bug with Safari/iOS with VoiceOver where a modal dialog hidden with display:none will not get focus moved to it. See more at https://developer.paciellogroup.com/blog/2018/06/the-current-state-of-modal-dialog-accessibility/. Could be that a dialog that is getting entirely constructed and inserted on the fly is suffering the same effect? Does having the modal container already in the DOM and hiding/showing it with visibility:hidden|visible solve the issue?


bom tries to pull down the image from a registry even if it exists in local image cache

What happened:
I built the image, then tried to generate an SBOM for that image but bom tool tried to pull an image from a registry even if it exists locally.

What you expected to happen:
It should generate an SBOM for an image that exists in the local image cache.
How to reproduce it (as minimally and precisely as possible):
Try to build an image then run bom generate -i <img>
Anything else we need to know?:
$bom version
______  ________  ___
| ___ \|  _  |  \/  |
| |_/ /| | | | .  . |
| ___ \| | | | |\/| |
| |_/ /\ \_/ / |  | |
\____/  \___/\_|  |_/
bom: A tool for working with SPDX manifests

GitVersion:    devel
GitCommit:     unknown
GitTreeState:  unknown
BuildDate:     unknown
GoVersion:     go1.17.8
Compiler:      gc
Platform:      darwin/amd64

Environment:

Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release): Darwin
Kernel (e.g. uname -a): Darwin C02DM1U3MD6R 20.6.0 Darwin Kernel Version 20.6.0: Wed Jun 23 00:26:31 PDT 2021; root:xnu-7195.141.2~5/RELEASE_X86_64 x86_64
Others:

I'm willing to work on this, would you mind assigning it to me?

/assign @developer-guy

thanks for finding this issue and willing to fix it! 🎉


Fix config typings

Before submitting a pull request, please verify the following:

[x] If you've added code that should be tested, please add tests.
[ ] If you've modified the API (e.g. added a new config or public method), update the docs and TypeScript declaration file.
[x] Ensure your code lints and the test suite passes (npm test).

Hey guys 👋
I thought I'd provide a pull request for the typings covering the fact that raven-js can take a falsy dsn and be disabled, as these lines suggest it in the source code.
https://github.com/getsentry/raven-js/blob/5d12cc1bc92842e3b2a49c64f1d20aaa79a06a8f/packages/raven-js/src/raven.js#L155-L162
Thanks!

Thanks for the PR!
SDK won't work at all without the dsn (only captureX calls will work, but they won't send anything and will throw an error anyway due to isSetup call). It should rather be changed in the code itself, not in the typing.
However, to change this, we'd have to bump major version and we are already working on the new SDK, to I don't think we should merge it now.

@kamilogorek Thanks for the review. That makes total sense, I thought that was an intented behaviour, that's why I've decided to fix the typings.
I'll close this then 👍


Add an example of a quite minimalistic server

Just for show, you know :sunglasses:

@homu r+

:pushpin: Commit 51e25f4 has been approved by Ogeon

:hourglass: Testing commit 51e25f4 with merge c15c562...

@homu retry

:hourglass: Testing commit 51e25f4 with merge b336216...

@homu retry

:hourglass: Testing commit 51e25f4 with merge d319b7f...

Third time the charm... @homu retry

Third time the charm...
@homu retry

Third time the charm...
@homu r+

:pushpin: Commit fac3714 has been approved by Ogeon

:hourglass: Testing commit fac3714 with merge e783b8b...

:sunny: Test successful - status


speed up the setting up of defaults

fixes: https://github.com/kiali/kiali/issues/6132

just an experiment to see what this would do. Haven't done any examination of the performance data other than quick smoke testing -- seems this makes things faster but not as fast as I was expecting.

I did a quick test. I extracted only the code that is changing - first into set-defaults-original.yml
and then the updated code that uses raw jinja: set-defaults-jinja.yml
I then ran with ansible.cfg that enabled the profiler:
[defaults]
callbacks_enabled = profile_tasks
callback_whitelist = profile_tasks

The results:
The original code (ansible-playbook set-defaults-original.yml):
PLAY RECAP *******************************************************************************************************************************
localhost                  : ok=19   changed=0    unreachable=0    failed=0    skipped=19   rescued=0    ignored=0

Thursday 29 June 2023  15:49:41 -0400 (0:00:00.042)       0:00:01.448 *********

The jinja code that does the same only with jinja expressions, not ansible tasks (ansible-playbook set-defaults-jinja.yml)
PLAY RECAP *******************************************************************************************************************************
localhost                  : ok=4    changed=0    unreachable=0    failed=0    skipped=3    rescued=0    ignored=0

Thursday 29 June 2023  15:50:15 -0400 (0:00:00.040)       0:00:00.341 *********

So the good news is this PR speeds up the replaced code by a whopping 80% (from 1.5s to 0.3s).
The not-so-good news (not that it is bad) is that really, from a absolute time perspective, we only just shave off 1 second from the total processing time of the operator so it doesn't really do a whole lot.
That said, we DO get a speed improvement (albeit 1 second). So this could be merged if we want to eek out some more improvement.

up to you

I'm going to put this ready for review and merge when accepted.
It is faster (albeit in a minor way), the code is just as easy to maintain, and I did confirm it works -- all molecule tests pass.
=====================
=== TEST RESULTS: ===
=====================

              accessible-namespaces-test... success [8m 32s]
     affinity-tolerations-resources-test... success [3m 40s]
                cluster-wide-access-test... success [9m 9s]
                      config-values-test... success [2m 56s]
                  default-namespace-test... success [2m 2s]
                            grafana-test... success [3m 45s]
                        header-auth-test... success [1m 54s]
                      instance-name-test... success [2m 34s]
                             jaeger-test... success [2m 50s]
                            metrics-test... success [3m 59s]
                     null-cr-values-test... success [1m 50s]
                only-view-only-mode-test... success [2m 45s]
                             openid-test... success [1m 58s]
                     openshift-auth-test... skipped
                   os-console-links-test... skipped
                 read-certs-secrets-test... success [1m 51s]
                              roles-test... success [5m 51s]
                    rolling-restart-test... success [2m 46s]
                              token-test... success [2m 11s]


Failed to load options java.lang.NullPointerException

One of my testers came across an interesting bug. He was testing an install build and the options set in DefaultSettings did not load - nothing but vanilla configs loaded. The result was none of my default configs loaded, so there was no custom menu, no quests, no custom settings, and most important, he loaded into a vanilla overworld and not my custom skyblock world. He is able to reproduce this issue, but I have not. I tried 4 or 5 times, but my installs were fine.
Minecraft: 1.19.2
Forge: 43.2.3
DefaultSettings: 1.19.x-2.8.7
This is how he reproduced the failed install:

using whatever launcher go into create new instance/profile from zip file
select Create_Skyblock_Forge_1.19.2-0.9.0.zip (This is an unpublished version for testing)
wait for it to install
copy over an existing options.txt to the instance folder (see
troubleshoot-0_9_package.zip also contains log files, and the pack's default configs)
launch the pack
observe problem: dev menu, no quests, normal world
close everything, manually delete the options.txt file
re-launch the pack
observe everything is fixed because default settings applied

The player believes it is step 4 is the cause. In troubleshoot-0_9_package.zip is his latest.log file and that is where I found the error Failed to load options java.lang.NullPointerException
I also did a file comparison between his options.txt (on left) and the one that comes with the pack in config/defaultsettings/Default (on right) and it looks like all of his keybindings are in his file.

i'm the one that originally reported this to @ClaudiusMinimus
OS is windows 10 if that matters.
The fact that there is a lot in this options file kind of makes sense because i bring it with me over from pack to pack (mostly because of things like fov, ui scale, volume settings, quest keybinds)

Thank you very much for reporting that bug! It really is a strange one.
Especially the NullPointerException is odd.
To mitigate this you might wanna swap step 4 and 5 (also stop the game after step 5). Under certain circumstances the mod doesn't really know what to do when the options.txt already exists on the first startup (end-user's side of the pack). It expects it to not be there.
Does it work if you do it this way?
Next month I'll dive deeper into the mod to iron out all those weird problems that have been there for some time. Including this one ofc.

To mitigate this you might wanna swap step 4 and 5 (also stop the game after step 5 has finished). Under certain circumstances the mod doesn't really know what to do when the options.txt already exists on the first startup (end-user's side of the pack). It expects it to not be there.

yes, as I described in the reproduction steps, even after the problem has happened, if I just delete options.txt and re-launch, everything works.
so this isn't really a blocker for anyone, just costs a game reload.

Yeah, you pointed out the exact problem. That check via the options.txt is fine in most cases but inappropriate in others.
I'll substitute that with a custom solution that isn't dependent on any Minecraft files :) It should be fixed by the next major release in Feb/Mar 2023.


Fix Overflow of Batch Events in Case of Send Failure

Closes #190

Closes #190

Hi @nbetten,
Your changes have conflicts with existing codebase.
Could you please resolve all the conflicts and push your changes so that we can review your changes ?


运行日志提示警告

在dock运行日志里有下面的提示信息
`> node ./build/index.mjs
(node:18) [DEP0040] DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.
(Use node --trace-deprecation ... to show where the warning was created)
Server is running on port 3002
(node:18) NOTE: We are formalizing our plans to enter AWS SDK for JavaScript (v2) into maintenance mode in 2023.
Please migrate your code to use AWS SDK for JavaScript (v3).`
这个是否会影响运行？谢谢

不影响 就是一些提示


Project not deleted even after the wait period

The project is not fully deleted even after the wait period
[mrinaldas@localhost odo]$ ./odo project delete myproject
Are you sure you want to delete project myproject? [y/N] y
Deleting project myproject...
(this operation may take some time)
Deleted project : myproject
You are not a member of any projects. You can request a project to be created using the `odo project create <project_name>` command
[mrinaldas@localhost odo]$ ./odo project create myproject
project.project.openshift.io "myproject" already exists
[mrinaldas@localhost odo]$ ./odo project create myproject
New project created and now using project : myproject

working fine for me though :sweat_smile:
$ odo project delete myproject
Are you sure you want to delete project myproject? [y/N] y
Deleting project myproject...
(this operation may take some time)
Deleted project : myproject
You are not a member of any projects. You can request a project to be created using the `odo project create <project_name>` command

$ odo project list
You are not a member of any projects. You can request a project to be created using the `odo project create <project_name>` command

$ odo project create myproject
New project created and now using project : myproject

$ odo project list
ACTIVE   NAME
  *      myproject

@mik-dass , can you please check once again

@surajnarwade It's a flaky issue. It occurs rarely. I even gave a quick look over the code and it seems fine to me. I guess it's the API which has the problem.

@mik-dass , so it's not in scope of odo then, let's close this one, feel free to reopen if you find this again :+1:


[T6A3][W15-C3]Jazlyn Ang

@turtle96
Some comments added. Please ack & close the PR after reading comments.


Make working with the SaltStackClientConfig more typesafe

This replaces the current stringly typed SaltStackClientConfig with a statically typed key/value store that knows the type of each key. This shifts the burden of parsing the configuration values from the consumers side to the producers side. So everything querying the config for values does not have to deal with parsing and error handling of invalid values like "hello" as a port value. This also results in those invalid values being detected earlier (for example when the config is loaded from a file instead of when a value is used).
I would also suggest to generally make parameters stricter and ask for what is really needed instead of something that could potentially be parsed to what is needed. An example of this is the change here asking directly for a URI object instead of a String.
This reduces the overall error handling needed inside the API and and shifts it to the user of the API which should be fine since most argument parsing errors would probably result in an error that bubbles up to the user anyway.
I also experimented with a version that can prevent different sub types of KeyValueStore from mixing keys with each other but left it out for now since there is only one Configuration class right now. But it could be useful in the future.

We discussed this with @isbm and @renner and generally like the underlying ideas, thanks for submitting this PR.
What we currently don't like is the design of classes, specifically, the fact that KeyValueStore is separate and abstract. After your changes it is only used to implement SaltStackClientConfig, which then becomes a mere collection of constants, moreover not implementing any abstract method.
We think it would be better to merge KeyValueStore code into SaltStackClientConfig to keep design simple, at least until we have an actual need of reusing KeyValueStore somewhere else. That way SaltStackClientConfig will have its inner Key class with default value, which sounds correct, and will internally use HashMap<Key, Object> as an implementation detail.
What do you think?

The design mostly comes from my experiments with multiple sub classes and a second type parameter to prevent mixing keys. You are right that this does not make much sense if we only have one sub class.
I will merge those too together and rebase this PR onto the latest master so it is mergeable without conflicts.
Out of curiosity, what would you do differently in regards to sub classes only being used as a namespace for keys, if there were multiple sub classes of KeyValueStore? I personally don't mind using them that way but i'm open for suggestions.

what would you do differently in regards to sub classes only being used as a namespace for keys, if there were multiple sub classes of KeyValueStore? I personally don't mind using them that way but i'm open for suggestions.

I am not sure I understand your question - are you wondering what to do with inner classes like KeyValueStore.Key when the outer class is subclassed?
In that case, I would evaluate how strong the inner and outer class "link" is, semantically. If it is not too strong, I would simply move the inner class in a new separate one - after all inner classes are supposed to be so highly coupled with their outer ones to be basically part of them.

The question was more about SaltStackClientConfig being just a collection of constants and not having anything else. If we had the super type KeyValueStore and two sub classes A and B that each contain only their key constants would that be okay or would there be a better way of doing this?

Ok i think it should be fine now i addressed all the points you mentioned.

If we had the super type KeyValueStore and two sub classes A and B that each contain only their key constants would that be okay or would there be a better way of doing this?

OK, I understand now. I don't think using classes as mere constant containers is a great design idea in general, it certainly doesn't sound very object-oriented. Depending on the situation I would probably try to design classes differently so I can stick constants to related classes that also perform some kind of function. That said the idea might prove valuable if a very high number of constants is needed for some reason, even if I haven't stumbled in such a case yet.
PR is good, merging, thanks for your contributions so far.


gRPC JSON transcoding: Move openAPIto gRPC HTTP API doc and link out

Internal Review
Fixes: #25837
Adding a note on gRPC HTTP API and linking out to its gitHub readme.

@JamesNK , I kept this short, but in a note where it would not be missed.  Alternatively, I could add that gRPC JSON transcoding was introduced with .NET 7.0 and that gRPC HTTP API continues to be available for previous versions of .NET, or something of that sort.


Make the blog publish RSS/ATOM feed

More inspiration can be found here: https://pydanny.blogspot.com/2009/04/show-me-your-open-source-django-blog.html

Fixed with 5863ace257f6a0116c5facffb3bf2b2cd2c3f8f3.


200 OK even if all summaries are empty due to timeout between first and second protocol phase

Describe the bug
<GET /search/?query=sars&yql=select%20title,%20id%20from%20doc%20where%20userQuery()&timeout=0.01&ranking.softtimeout.enable=true HTTP/2
<HTTP/2 200
< date: Tue, 10 Jan 2023 19:00:41 GMT
< content-type: application/json
< content-length: 1503

{"timing":{"querytime":0.007,"summaryfetchtime":0.0,"searchtime":0.007},"root":{"id":"toplevel","relevance":1.0,"fields":{"totalCount":92795},"coverage":{"coverage":69,"documents":673334,"degraded":{"match-phase":false,"timeout":true,"adaptive-timeout":false,"non-ideal-state":false},"full":false,"nodes":2,"results":1,"resultsFull":0},"errors":[{"code":12,"summary":"Timed out","message":"Summary data is incomplete: Timed out waiting for summary data. 2 responses outstanding."}],"children":[{"id":"index:content/1/b4dcfdf99fc4f469b08aef6f","relevance":0.36776325146483624,"source":"content"},{"id":"index:content/1/ed8a721d6d283ae7a5319a24","relevance":0.36637411121832414,"source":"content"},{"id":"index:content/1/80ff4d1faa345d294ddec0cd","relevance":0.3656431806441757,"source":"content"},{"id":"index:content/1/98b2d8205f724b1fe2d33354","relevance":0.3656431806441757,"source":"content"},{"id":"index:content/1/7b1174edd754bdb67a961bed","relevance":0.36529888546717887,"source":"content"},{"id":"index:content/1/00e02962f3eded0fb143abc3","relevance":0.36409955723740106,"source":"content"},{"id":"index:content/0/8ae169d8b5a6cdd13ec7e7a6","relevance":0.36409955723740106,"source":"content"},{"id":"index:content/0/b58f0d05bdb8b63d8ca77931","relevance":0.3634864920866355,"source":"content"},{"id":"index:content/1/c2b596e5ac3e0346aaff963e","relevance":0.36335079529424297,"source":"content"},{"id":"index:content/1/fe8dc3ee573171877d437cc5","relevance":0.36335079529424297,"source":"content"}]}}

This is reproduced by setting a low timeout and disabling softimeout.

Don't count unfilled hits towards success: https://github.com/vespa-engine/vespa/pull/25662


Add support for the LongEmbed benchmark

Checklist for adding MMTEB dataset

Reason for dataset addition:

[x] I have tested that the dataset runs with the mteb package.
[ ] I have run the following models on the task (adding the results to the pr). These can be run using the mteb run -m {model_name} -t {task_name} command.

[ ] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[ ] intfloat/multilingual-e5-small

[x] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[x] I have considered the size of the dataset and reduced it if it is too big (2048 examples is typically large enough for most tasks)
[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.
[ ] I have added points for my submission to the POINTS.md file. (i will add these information when the arxiv version of our pape is out)

@isaac-chung, if you want to take charge of some of these reviews that would be great. If so I can add you to the list of reviewers for MMTEB (see overview issue)

Thanks to @KennethEnevoldsen @isaac-chung for your timely review! I have updated the PR, where I added some model scores and meta data. Specifically, I have included results on all six datasets except for Passkey and Needle, which requires an additional parameter of context_length, and results of different context_length just overwrites each other. For the meta data, I have added some key information such as domains and forms. However, I have also find some fields very confusing. For example, what do you mean by saying the socioeconomic_status of a dataset?

many thanks to @KennethEnevoldsen  for the detailed elaboration on each parameter for meta data. I have filled them all, and added results on Passkey and Needle using 512  context length.

This looks amazing! If you also want a leaderboard tab for your benchmark feel free to open a PR here: https://huggingface.co/datasets/mteb/results with the result files. We could add it under Retrieval similar to https://huggingface.co/spaces/mteb/leaderboard?task=retrieval&language=law

This looks amazing! If you also want a leaderboard tab for your benchmark feel free to open a PR here: https://huggingface.co/datasets/mteb/results with the result files. We could add it under Retrieval similar to https://huggingface.co/spaces/mteb/leaderboard?task=retrieval&language=law

Thanks!

merging into branch to add points and then merge

Excellent work!

Tom Aarsen

hi, I'm ready to add points for this submission, may I ask what kind of information should I provide? I think I'm still a little bit confused about what to fill in these fields: @KennethEnevoldsen
{
    "GitHub": "GitHubUser1",
    "New dataset": 2-6,  # 2 points for the dataset and 4 points for the task
    "New task": 2, # e.g. a new style of task (e.g. classification, or retrieval)
    "Dataset annotations": 1, # 1 point for each full dataset annotation
    "Bug fixes": 2-10, # depends on the complexity of the fix
    "Running Models": 1, # pr model run
    "Review PR": 2, # two points pr. reviewer, can be given to multiple reviewers
    "Paper Writing": NA,
    "Ideation": NA,
    "Coordination": NA
}


DefaultValue For Macro Generated Case Classes

in the following example, I expect "John Doe" and "42" as default values.
 case class Person(name: String = "John Doe", age: Int = 42)

 object Person {
   implicit val schema = DeriveSchema.gen[Person]
 }

 assert(Person.schema.defaultValue == Right(Person("John Doe", 42))) // Is this a right expectation?!

But it seems the DeriveSchema.gen doesn't encode the default values.

/bounty $150

/attempt #585


Change paths to star rating

Fixes: https://github.com/tastyigniter/TastyIgniter/issues/756

Cool - updated.


Umbraco V8 Version

Hi Ryan,
Are you looking to release a version to run on Umbraco V8?
If not, I might look at writing something myself.

Hi @dinc5150, did you get a reply from Ryan, or did you find an alternative yet?

Sorry, no @rbottema . I was going to fork the repo and update if it was no-longer maintained, but haven't had the chance as yet. I'll post back here if I get the chance to update

@dinc5150 @rbottema apologies for the lack of replies.
I no longer use this package or Umbraco, so this package is no longer maintained. I would be happy to link to any forks/alternatives in the README if you were to provide any.

All good, thanks @ryanlewis . I'll let you know if I get the chance to update.


[.NET] Dutch DateTime Duration support

All test cases pass.
Some cases were updated (e.g. cases containing "dgn" instead of "dagen" or "jr" instead of "jaar" because these abbreviations do not seem to be used)
Two cases were updated in DateParser because they used "couple of" instead of "few" which has a different resolution.

@iMicknl, can you take a look this please?


Update install-consul logic to determine CPU arch which makes it run on Graviton based EC2 instances

Changes

Include logic to determine binary architecture from returned value from uname -m

Verification

Tested on a c6g.* EC2 instance running Amazon Linux 2

$ uname -m
aarch64

Tested on a Raspberry Pi 3B running Raspbian

$ uname -m
armv7l

Tested on some good 'ol amd64 Linux

$ uname -m
x86_64

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.

There was a transient test failure on the first run, but on re-run, all tests passed. Merging now!

https://github.com/hashicorp/terraform-aws-consul/releases/tag/v0.9.3


install falied using mysql database with utf8 charset

time="2017-10-27T09:38:02Z" level=info msg="MySQL dialed" max_idle_connections=30
time="2017-10-27T09:38:02Z" level=fatal msg="Error initializing datastore." error="Error 1071: Specified key was too long; max key length is 767 bytes"

hello @heww ,
Assuming you are following mysql docs .
It is difficult to diagnose a problem with so little information, but it seems your problem is about indexing a UTF8 field in MySQL. I would advise checking stackoverflow for similar problems.  If you have identified this as a IronFunctions specific problem, please share some information.
Thanks !


[Bug]: Memory Leak / No way to kill running captcha instances

Bug report requirements

[X] I have confirmed that this is a bug and not a configuration/user error.
[X] I have searched for existing issues and confirmed that my issue has not been reported before.

IconCaptcha version
4.0.3
PHP version
8.0
Browsers affected
Google Chrome
Expectation
When captcha is completed or deleted, it should not continue to send request regularly.
Actual behaviour
If the option DisplayInitialMessage is set to false,
Although the captcha is completed or deleted, it continues to send requests regularly in background.
After one captcha it's no big deal, but after completing multiple captchas, it starts to send as many regular requests so it quickly becomes a problem...
Please, just add a way to remove a captcha so this behaviour stops.
Relevant log information
No response
Additional information
No response

Hi @Quetlo! Could you please elaborate your issue, perhaps with a screen recording? In my environment, the captcha does not make requests on its own once a captcha has been completed - even when setting the displayInitialMessage option to false.
The only case the captcha makes requests on its own is if no data could be loaded from the server, e.g. when receiving a 4xx or 5xx status code. It will retry to fetch a challenge if it encounters an error. There is no limit on how many times it can retry, which might be something I could implement to prevent server spamming in this particular case. However, retrying on a 5xx status indicates something is incorrectly setup on your IconCaptcha backend which should be addressed.

Hi @Quetlo! You can ignore my previous notification, as I now see what you mean. The issue seems to be related to the 'invalidation' feature, which automatically invalidates a captcha after a certain time. Somehow, this invalidation process still takes place even after completing a captcha - which obviously shouldn't happen. I'll look into it and release a fix later!

Hi again. In my previous comment, I forgot about the 'completion invalidation' feature in the PHP config (see here). The behavior you mention in your issue, although maybe a bit strange combined with displayInitialMessage, is expected behavior and not a bug.
By setting the value of the PHP 'completion invalidation' option to anything other than 0, the captcha widget will automatically invalidate any completed challenge after the set amount of time - both in the client and on the server (a request).
Because the displayInitialMessage option is set to false, the client will immediately request a new challenge from the server after the completed one was invalidated. Assuming you deleted the widget from the screen (by deleting the DOM), this will not stop the JavaScript from running its cycle of invalidating and requesting a new challenge, and you end up with infinite periodic requests in the background without seeing the widget itself reset.
challenge loaded -> completed -> invalidated (completion expired) -> challenge loaded -> invalidated (inactivity) -> challenge loaded -> etc...

My advise is to not delete the widget at all as the scripts are not intended to work this way. Deleting the DOM might even break the submission of the form due to it containing hidden input fields which are now missing. You may also want to disable the invalidation options in the PHP config, as they kind of clash with the displayInitialMessage option. This way it wont request a new challenge after X time of inactivity.
If my assumption is incorrect, please let me know.


Add ability to check account details during refresh grant

Hi,
I'm using Oauth2 with JWT as Access Token. At the moment I try to find a way to invalidate a “Refresh Token” before it expires. I use a JdbcUserDetailsService with the possibility to enable/disable a user.  Requesting a new Access Token with a disabled user by password grant leads to an error with the message “User is disabled”. So far, so good. Now I would expect that Access Token requests via Refresh Token by users that got disabled during their “session” (maybe because of bad behavior) will also lead to an error, but that is not the case. A disabled user is able to get new tokens as long as the Refresh Token does not expire (and this new Token may also contain a new Refresh Token that is able to create even more Access Tokens and so on). I think even if I delete the user from database it would have no effect because there is no kind of user validation.
My question is, is this kind of validation missing and valid for Refresh Tokens or is my approach wrong?

Good catch. Arguably this should also be a feature of the DefaultTokenServices (i.e. you shouldn't have to rely on the TokenStore being up to date to decide on whether a user account is still active). I would also like to have been able to do something in an OAuth2RequestFactory, but it looks like the user id is not directly available there.

I’m glad that I was able to help.
To fix the “disabled user” problem, I added some additional lines to the DefaultTokenServices in refreshAccessToken(…), but unfortunately I used some of my own custom classes, so I can’t deliver my solution to you.
Anyway there is one more question that I have very similar to the one above. My JWT stores some additional information about the user in it (e.g. e-mail address etc.). These information are added to the Access Token in the enhance(…) method of a class that overrides the JwtAccessTokenConverter. Here I have an issue similar to the one above. User data that got changed during the “Access Token session” won’t be updated in the JWT, if I use the refresh_token grant, because the user information authentication.getUserAuthentication().getPrincipal() comes from the Refresh Tokens content.
So I need two different enhance flows. One for the refresh_token grant that reloads the user data from database and another one for all other grant types using the user data loaded by the UserDetailsService.
My Problem is that I have no information about the current grant type in the AccessTokenConverter. I think this information gets lost in createRefreshedAuthentication(…) in DefaultTokenServices.
At the moment I temporary store the grant type in the additional information of the new created Access Token so I can access it in the enhance flow. But I don’t think that this is the way to go.
Is there any other way to access the grant type in AccessTokenConverter or is reloading user data there (via UserDetailsService) placed wrong?

Anyway there is one more question that I have very similar to the one above.

In general, can you ask questions about usage in Stackoverflow (or create a new issue here if it's not appropriate on SO)?
The grant type is usually available OAuth2Request.getGrantType(). If it is getting lost in createRefreshedAuthentication() please open a new issue.

One for the refresh_token grant that reloads the user data from database and another one for all other grant types using the user data loaded by the UserDetailsService.

What's the difference? Isn't the UserDetailsService a database?

The grant type is usually available  OAuth2Request.getGrantType() . If it is getting lost in  createRefreshedAuthentication()  please open a new issue.

Done.

One for the refresh_token grant that reloads the user data from database and another one for all other grant types using the user data loaded by the  UserDetailsService .

What's the difference? Isn't the  UserDetailsService  a database?

The UserDetailsService is a database, but the refresh_token grant needs to reload the data from database (to load potential updates) whereas the other grant types already loaded the data from database. So if I don’t check for the grant type, I would end up loading user data twice from database (e.g. for the password grant).

So if I don’t check for the grant type, I would end up loading user data twice from database (e.g. for the password grant).

So that's just a symptom of you having chosen the TokenEnhancer as the place to reload the user details. I still think this issue is worth fixing, but I'm not sure if we had the right hooks for reloading users you would need to do it that way.

Having added this feature for 2.0.7 I remembered what we did in Cloudfoundry (back in 1.0), so you can still do that in 2.0.6 I reckon: use a physical ApprovalStore and link it to user account changes (e.g. so all approvals are revoked when an account is disabled).

Thanks for providing this information, but the workaround I implemented for 2.0.6 works quite smooth. So I will wait for release 2.0.7 to replace it with your great “AuthenticationManager” solution. But anyway, it’s good to know.

Re-opened because of some issues with Spring Boot (see JDBC integration test project).

Are there any ways to skip this AuthenticationManager check? Maybe it should have an option to turn it off?

Everything is possible, but that sounds like a bad idea, so I wouldn't look too hard for a solution if I were you.

My authentication server uses only external identity providers as in your example https://github.com/spring-guides/tut-spring-boot-oauth2 and issues its own JWT access tokens (using authorization_code flow) that I want to refresh using refresh_token flow - authenticationManager  can't find (and therefore - authenticate) the users that came from external identity providers. I am not sure whether my approach is wrong or I am missing something?

That might be a new feature in the sample. Best not to discuss it here. Why not open a new issue in the tutorial and we can discuss it there until we decide if a new feature is also needed here?

where should we authenticate for refresh token?because I am using customauthentication for generating access token but when I generate access token from refresh token It is saying UserDetailsService is required.So how to write custom authentication for using refresh token ?


Add pywikibot-like bots

Continues #3
Create a bot object so that a user can read / update / delete a page more easily. (close #2)
Page
A user can read a page by the following code:
const site = Bot.site(SiteUrl.wikipedia("en"));  // English wikipedia site
// const site = Bot.site("https://en.wikipedia.org/w/api.php");  // This is also available
const page = await site.page("Software_testing");
console.log(page.text);  // Print the content of "Test_Page" page

If a user want to edit / delete a page, they can login via bot password to edit / delete a page. (Login via user's name and password hasn't implemented yet)
const site = Bot.site("https://test.wikipedia.org");
await site.login("Bot_username", "Bot_password");
const page = await site.page("Test_Page");

// Edit
page.text = "Hello World";  // Change content of page into "Hello World"
await page.save("Update it!!");  // Save

// Delete
await page.delete();

Category
A user can get the pages that belong to the specific category.
const site = Bot.site("https://test.wikipedia.org");
const cat = await site.category("Cat");
console.log(cat.articles); // ["American Shorthair", "Burmese cats", "Cymric cat", ...]

Done

[X] User can read a page
[X] Support bot login
[X] Bot can edit / delete a page with login status

TODO (Some trivial ones might not listed)

[x] Fix "undefined" string problem. When a user doesn't pass argument to the function, the relevant argument would become "undefined" string.
[X] Support category lookup.
[ ] Support login via user's name and password. (It seems that clientlogin API should work with authentication. The example in official API page is different from its documentation.)
[X] Predefined the API URL of the wikis, so that users don't need to find what's the API URL
[X] Error handling. When an HTTP request isn't successful, shows the messages to users.
[X] Add testing
[x] Add instruction (or document) of bot into README.md.

Thanks for opening the PR again, sorry for the inconvenience. I will make some minor modifications, and talk to you about larger modifications. Once everything is done, we’ll release it.

I am concerned with the name bot.

How come? Is the name too common? Or its name is same as the folder?

And don’t you think Site and Bot should be merged into a single class?

Yes, we could merge them together. The reason why pywikibot initial a Site instance by calling a static function is to cache the Site object. But I think it's not necessary in this repo.

How come? Is the name too common?

The name bot to me feels like something that automates things, but no problem, I think we’ll be adding more methods that will complete the name in the future.

Yes, we could merge them together. The reason why pywikibot initial a Site instance by calling a static function is to cache the Site object. But I think it's not necessary in this repo.

Can you merge them then?

I've merged them. Also update the testing and README.md.

Oops, sorry I misunderstood your meaning...

No, problem. It’s only me and you in the universe working on this.

I've rename it...! Also the variable names are changed from site to bot.


About the positional encoding

Hi, thanks for sharing this great work.
I wonder if this an implementation mistake in the position encoding part in line 21 of  position_encoding.py.
div_term = torch.exp(torch.arange(0, d_model//2, 2).float() * (-math.log(10000.0) / d_model//2))
Should we change (-math.log(10000.0) / d_model // 2)  to (-math.log(10000.0) /  (d_model // 2) )? Otherwise this part ends up with -1.0 which means very high temperature.

Hi, sorry for the late reply. And thanks for pointing out this implementation mistake! 👍 We'll fix this in this week!

We have trained a new model on ScanNet with this bug fixed, and it leads to a minor performance increase as shown below:

model
auc@5
auc@10
auc@20

paper
22.06
40.8
57.62

bug-fix
22.5
41.23
57.77

I would conclude that this bug does not affect the results of experiments in our paper. However, thanks again for pointing this out!


Update README.md

edits

Docs Build status updates of commit 984677c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

README.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


small fix get_uptime() to eliminate compiler's warning on armv6 architecture

According to man sysctlbyname(3) populates struct timeval instead of struct timpespec for KERN_BOOTTIME
On armv6 architecure long has 4 bytes length but time_t has 8, so the compiler generates warning about integer precision loss. Use explicit cast.

Just reviewed the changes. OK
Merging in the code.
Thanks for the work @dorjechang!


Add XAML Code to sample string helper

In the Debug build, we could add a helper on the settings page, which you can paste the ordinary XAML code and it gives the sample code in the format we currently use for the samples, that is with the special character codes for "<" and ">".
If we include it through conditional compile, it won't affect the store version of the app, yet make it easier to contribute and update XAML sample code.

Is this still an issue with the latest version?

@jay-o-way Still would be nice to have IMHO.

Closing since it's unlikely this will be implemented.


How can I get the editor object outside the on-image-upload event?

Hello.
I like angular-simmernote and I have a question.
I can upload a image using 'editor' object. 'editor' object is on-image-upload's parameter.
This object can be used only inner on-image-upload.
I'd like to use 'editor' object outside the on-image-upload too.
Thanks.

Did you mean second parameter, editor, in onImageUpload?
If so, I just followed summernote's API.
Can you provide the use case for use editor object outside the callback?

I added a button which is irrelevant to summernote's picture toolbar to upload a image.
If click this button, upload a image to server and return URL which is accessible to image, from server.
I'd like to append image URL at summernote's editor.
So, I have to access editor object outside the onImageUpload.

In such case, can you use ngModel?
I think you can add image url into editor if you add it into ngModel.

function DemoController($scope) {
  $scope.text = "Hello World";
}

This sample code is reset "Hello World" to $scope.text.
Is it possible insert a image using ngModel in cursor position?
I have a time and searched this issue at Summernote.
https://github.com/summernote/summernote/issues/282

var editor = $.summernote.eventHandler.getEditor();

Look at HackerWins's comment on comment list. Summernote provide plugin and can access editor object. editor object can use functions insertText, insertImage, createLink....
I think angular-summernote needs that function. What do you think?

@Pyohwan
Sorry for late. I understand the problem.
<summernote editor="editor">
How about do you think the attribute name?
Actually, I already used <summernote editable="editable"> for callbacks.
But I worried about the attribute name is too general. And editor is also general.
And editor could be confuse with editable.
Do you have a idea fot it?

How do you think adding prefix like ng- of Angular Core module?

sm means Summernote.
But it's up to you.

@Pyohwan
I digging the issue. but summernote return editor object globally, not per summernote instance.
so, $.summernote.eventHandler.getEditor() return always same object.
Therefore, developer can get Editor via summernote instead angular-summernote.
And then, they use it with editable obtain from angular-summernote
DId I missed something? How do you think?

That's ok.
Thank you for your help.


Is port number correct?

https://github.com/jbyuki/one-small-step-for-vimkind/blob/fa0df0ade3402c44190147da7d0f766254ebcbeb/doc/osv.txt#L71
port number is 8088
But in README.md;
8086.
It worked correctly with 8086.

Good catch, that's an oversight.
If you can PR that, it would be great. It would credit you as the contributor.


MH-13266 Start date cross link does not work correctly

Can you add a short description to this pull request and fix the quoting issue?
The patch looks good otherwise.


Add start/duration options to the defined activity configuration

Describe desired feature
Currently when you are defining specific activity you can only define an interval and amount. It would be helpful to be able to pick a start time and duration so that some nodes are sending transactions at different times throughout the simulation.
Use case for feature
For example:
This activity definition would tell Alice to send 2000 msats every second, starting 10 seconds after the simulation starts and ending after 30 seconds. In other words, Alice would send Bob 30 transactions between 10 and 40 seconds in the simulation.
  "activity": [
    {
      "source": "Alice",
      "destination": "Bob",
      "start": 10,
      "duration": 30
      "interval_secs": 1,
      "amount_msat": 2000
    }
  ]

Would you like to contribute code for this feature?
I am interested and hope to contribute toward this feature.

I think this would be a very nice feature!
"start": 10,
"duration": 30

Small user-experience related question: WDTY about count rather than a duration so that we can save the end user some calculation? I would imagine that people know how many of this type of payment they want and then calculate the duration from that anyway.  But defer to your pref since this is a feature that you want/ I know no better!
One thing we'd need to think about in the context of the project is handling shutdown if we have a set of these activitiesf and they all finish. Currently doing some refactoring in #164 to neaten up all of our shutdown mechanics, so it shouldn't be difficult (it should just be handled because the producer/consumer will drop, but just flagging it because we've got tasks flying all over the place and it's easy to lose track).

I will start looking into implementing this in conjunction with the performance testing I am doing on LND.

Sounds great!

I am going to try to make this feature backwards compatible with older sim.json files. In other words.. the start and count values will not have to be defined in the sim.json file.
The count attribute will be an Optional<u64> where None means that the payment activity will continue for the full length of the simulation. If count is not provided in the sim.json file it will have a default value of None.
The start attribute will be a Duration and its default value will be 0, meaning the payments will start as soon as the simulation starts.
@carlaKC Any issues with that implementation? Trying to avoid making users re-create their simulation files.

Any issues with that implementation?

SGTM!


Add possibility to disable changelog, description and credits validation

Description of the Change

PR adds the possibility to disable changelog, description and credits validation
Closes #76
How to test the Change

Changelog Entry

Added - Possibility to disable changelog, description and credits validation

Credits

Props @senhordaluz
Checklist:

[ ] I agree to follow this project's Code of Conduct.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my change.
[ ] All new and existing tests pass.

Closing in favor of #84.
@senhordaluz Thanks a lot for your contributions. Due to conflicts here, I moved your work from here to the separate PR and added an additional pass regex feature there to handle both features in that PR. I appreciate your time and efforts on this project.
Thanks

Hi @iamdharmesh sorry for the delay
I've just saw your PR improving the initial idea. Great work.
Thanks


Today button not being translated

When using locale: '{my_language_code}', the 'today' button is not translated. Everything else is in the specified language.

The locale option has no effect on the today button. To change the text, use the buttonText option.


Contributions for 3rd round

This PR contains the contributions for the 3rd round of the call for abstracts.

the pdf generation failure is likely related to https://github.com/rseng/pdf-generator/issues/6, so let's wait for this getting fixed


Use createPromiseCapability in /web files and avoid unnecessary PDFPageView.setPdfPage calls

Slightly easier reviewing with https://github.com/mozilla/pdf.js/pull/8295/files?w=1.

/botio-linux preview

Looks really good. Thanks!


feat: port Data.Fin.Fin2

bors merge

I've marked this as synchronized in https://github.com/leanprover-community/mathlib/pull/17039


chore: update examples for traffic shaping

Purpose of this pull request
PR updates examples and add new section regarding traffic shaping for tracing.
Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions, updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

@mat-rumian, I went ahead and merged the PR.


Story: Geburtstag

Kiwi

most tasks are already quite poetic, they only need a little help to become fitting


Custom Mute Times

I would like these to be added as I think it gives moderators the tools to deal with rule offenses that are small enough to not warrant a 1-hour mute, but large enough to make a 7-minute mute too short for the offense. (Examples of these kinds of offenses would be extreme toxicity, excessive swearing, and second offense immediately after a 7-min mute)
I also simplified the mute constants to not include operators, as I think that it may make minor differences in performance. I left the values with operators in comments.

It's intentional that we've never supported this, because bikeshedding is bad, narcissism of small differences, etc.

I also simplified the mute constants to not include operators, as I think that it may make minor differences in performance. I left the values with operators in comments.

something something premature optimization; if you care about perf that much, we have lots of comments and indentation and whitespace that could be removed for additional lexer performance!

Thanks for the feedback.
Also, the second comment made me kind of laugh at how you made fun of me. You just took a simple action I took and then made me look like I'm against code formatting, which I just find funny, because being against formatting is ridiculous. Admittedly, I shouldn't have made changes to the variables now that I think about it. It sacrifices readability for an unnoticeable difference. That's not what you want with open source software.
My apologies for wasting your time.

I mean, it was kind of flippant, but the serious idea is that "sacrificing readability for an unnoticeable performance difference" is exactly what things like removing comments and indentation would do.


Conventions & rules for additional object types

It's been about 2.5 years since this article was last updated. Can you please update it to cover the following:

Triggers
Activities
Mapping data flows
Wrangling data flows
Any other object types that are yet to be covered

Thanks!

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

ID: 65f81ce0-8915-2390-2c8e-53eb8a93180b
Version Independent ID: 49fb5956-019b-0f5a-ebff-1ae19fe87c80
Content: Rules for naming Azure Data Factory entities - Azure Data Factory
Content Source: articles/data-factory/naming-rules.md
Service: data-factory
GitHub Login: @djpmsft
Microsoft Alias: daperlov

@VitalyMCT Thank you for your feedback, our team will look into it and get back to you at the earliest.

Until the document is updated, here are the rules collected from UI messages.
Data flow name: only letters, numbers and '_' are allowed.
Activity name: 55 characters, only letters, numbers, '-' and '_' are allowed.  Spaces are also only permitted in the middle
Trigger name: name can be up to 140 characters and only letters, numbers, '-' and '_' are allowed. Spaces are also only permitted in the middle.
Wrangling Data Flow (Preview): only letters, numbers and '_' are allowed.
Thank you for showing us where we can improve, @VitalyMCT .

I have opened a documentation work item on this. As its currently being tracked internally, I am going to go ahead and close this issue.
#please-close