🐛 BUG: D1 doesn't handle upper/lower case mismatches in queries properly

Which Cloudflare product(s) does this pertain to?
D1, Wrangler
What version of Wrangler are you using?
latest
What operating system are you using?
Mac
Describe the Bug
Sqlite is case insensitive, so creating a table called COMMENTS, then inserting into comments should Just Work, but it doesn't:
https://github.com/cloudflare/wrangler2/issues/2360

This issue is fixed in the new d1 backend, closing this issue.


Provide an updated Java dictionary by traversing the JDK12 accessible public symbols

feat(Java dictionary): provide JDK12 wordlist & add reserved words

Provide an updated wordlist for JDK12 (previously JDK8) generated by traversing all public accessible symbols of OpenJDK 12 using a new script      leveraging classgraph project. This drops a few private and protected symbols and adds many new JDK9+ symbols.
Manually add Java reserved words copied fromt he official Java Tutorial
Add true, false and null literals to dictionary

Tested using yarn run build and yarn run cspell-link.

Following my comments on #23 I tried to generate and add a new updates wordlist. Took some puzzling but finally achieved the desired result.
Also added the reserved word, mostly instanceof was previously not detected as correct word.
My two questions would be:

Currently the VS plugin of cspell seems to come with a few predefined dictionaries like rust etc. but not java. Can we have it included by default and what would have to be done for that?
Is there a unit test infrastructure to assure the dictionary? I only did a manual quick test as described above think I saw a few test cases here, no?

Thank you!


Concatenate input model data from different CMIP5 experiments

Several namelists combine input data from different experiments to create long timeseries.
For example, to cover the time-period 1850-2100, the model data are read from the historical experiment for 1850-2005 and from one of the future projections (say rcp45) for 2006-2100.
In v1.0, this was realized with a very inelegant workaround, just by creating soft-links to the future projections in the historical data directory, something like:
ta_Amon_HadGEM2-CC_historical_r1i1p1_200512-203011.nc -> <path-to-rcp45-exp>/ta_Amon_HadGEM2-CC_rcp45_r1i1p1_200512-203011.nc
ta_Amon_HadGEM2-CC_historical_r1i1p1_203012-205511.nc ->  <path-to-rcp45-exp>/ta_Amon_HadGEM2-CC_rcp45_r1i1p1_203012-205511.nc
...

In v2.0 we need to implement a more flexible solution, also because some user may not be allowed to create links in their input data directory.
One option could be specifying the two experiments to be concatenated as a list in the exp key of the models section of the namelist:
- {model: HadGEM2-CC,  project: CMIP5, exp: [historical, rcp45], ensemble: r1i1p1, start_year: 1850, end_year: 2100}

I could start working on implementing this, but have some questions:

Is combining two experiments the only use case, or do you want this to work more generally?
What to do if the experiments specified overlap in time? Or does this never happen?

I think exp is the only use case, I can't think about any other model key that could be combined in a time-series (like ensemble or mip)
I would not exclude this case, it also depends on how the data are structured on the specific system (remember for example #314). In this case I would give priority to the first experiment in the list and this should be reported in the log.

I tag @bettina-gier who asked me about this issue a few days ago and might have further suggestions.

I think this may be useful also for observations, but in this case probably we will need lots of flexibility, as I can imagine wanting to concatenate very different datasets. It is not required now, but if you see that adding more flexibility is not more difficult it can come in handy in the future

A problem that might arise with the concatenation is that files from different CMIP experiments will have different global attributes (for example, expereiment_id) and Iris will complain about it.
This is related to #538 and we should probably address these two issues together.

@mattiarighi Could you please explain how we should eventually work with concatenation? I understand using a list for the exp, e.g. [historical, rcp85]?

I understand we should follow #729 right?

Yes, just specify a list of experiments, e.g. exp: ['historical', 'rcp85']. Note that this fails if the different experiments have overlapping data (for example in the time).


Wexflow core version is throwing "Invalid culture" error while run the app.

Wexflow core version is throwing "Invalid culture" error while launching the app using run.bat file.

Can you post the stack trace and details about the error?

I'm using .Netframework version now (https://github.com/aelassas/Wexflow/releases/download/v5.3/wexflow-5.3-windows-x64.exe), Just got the below logs from eventlogs:
Application: Wexflow.Server.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.Globalization.CultureNotFoundException
at System.Globalization.CultureInfo..ctor(System.String, Boolean)
at LiteDB.LCID.GetCulture(Int32)
at LiteDB.Collation..ctor(Int32, System.Globalization.CompareOptions)
at LiteDB.Engine.EnginePragmas.<.ctor>b__36_7(LiteDB.BufferSlice)
at LiteDB.Engine.EnginePragmas..ctor(LiteDB.BufferSlice, LiteDB.Engine.HeaderPage)
at LiteDB.Engine.HeaderPage.LoadPage()
at LiteDB.Engine.LiteEngine..ctor(LiteDB.Engine.EngineSettings)
at LiteDB.ConnectionString.CreateEngine()
at LiteDB.LiteDatabase..ctor(LiteDB.ConnectionString, LiteDB.BsonMapper)
at Wexflow.Core.LiteDB.Db..ctor(System.String)
at Wexflow.Core.WexflowEngine..ctor(System.String)
at Wexflow.Server.WexflowServer..cctor()
Exception Info: System.TypeInitializationException
at Wexflow.Server.WexflowServer.StartThread()
at System.Threading.ExecutionContext.RunInternal(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object)
at System.Threading.ThreadHelper.ThreadStart()

@srinathgnath The problem is related to the database system. Just use another database for the moment and it should be fine. Try MongoDB for example.
Are you using Windows 10?
Also, what is your culture? Try Get-Culture command on powershell.

I'm using windows 7. System culture:

@srinathgnath Ok thanks. Did you try MongoDB or another database system different from LiteDB?

@srinathgnath Can you go to C:\Wexflow\Database, compress that folder and send me the .zip to<EMAIL_ADDRESS>please. So I can figure out what's the problem with your database.

Sent. Check your mailbox.

@srinathgnath Thanks. I've connected my dev environment to the db you sent me but couldn't reproduce the issue. It worked fine. However, I opened an issue on LiteDB repo. I am using Windows 10. Maybe that's why I couldn't reproduce it.
Can you download LiteDB studio, and open your database with it and see if it works please.

@srinathgnath I found the bug. It is related to the collation of the database. Can you try with the following database please?  Just copy the new Wexflow.db in C:\Wexflow\Database and tell me if it works.
Wexflow.zip

I changed db to sqllite and its working fine.

@srinathgnath Thanks. Can you switch back to LiteDB and test with the database file that I sent you please? I need you to this so I can fix the collation bug please.

yes, LiteDB also works fine now. Thanks

@srinathgnath Thanks! If you want to use the  .net core version on Windows, here is the database file you need to copy in C:\Wexflow-dotnet-core\Database
Wexflow.zip


Simplified statements & removed unrequired returns

Hi, apologies for the slow reply @L4bb3rs , thanks for the submission.
We're really trying to have this code super-obvious, so people can do their own security review extremely easy. So that's why we'd be inclined to keep the multi-line if/else formulations, rather than go with one-line options. Similarly, having lots of comments.
Would you like to try coding a "name register" command? feel free to ping me on Twitter or Keybase and we can discuss further... Thank you


Support nested points in Juttle

[ ] correctly handle nested points when grouping
[ ] correctly handle nested points in sort
[ ] correctly handle nested points in all backends
[ ] correctly handle nested points in views
[ ] correctly handle nested points in all remaining procs

Details for each task:
correctly handle nested points when grouping
Support correct grouping by fields containing arrays or objects. This means that:

Group keys should be tested for equality using the relation defined in values.js.
Group keys which are arrays or objects should not be stringified when emitted as part of emitted points.

correctly handle nested points in sort
Support sorting by fields containing arrays or objects. This means field values should be compared using relations defined in {{ops.js}}.
correctly handle nested points in all backends
correctly handle nested points in views
Make sure that views handle points with fields containing arrays or objects correctly. More specifically, this means:

Field values should be stringified using the function defined in values.js.
Wherever a field value is used for equality testing, comparison, sorting, etc. it should be either handled using functions defined in values.js/ops.js or unhandled value types should be rejected with understandable error message. This includes e.g. values passed in -keyField of incoming points.

correctly handle nested points in all remaining procs
In no proc should a field containing array or object cause an internal error or incorrect behavior. In general, values should be handled using functions defined in values.js/ops.js (which may throw type errors in certain situations — this is fine). Where this is impossible, unhandled value types should be rejected with understandable error message.
This is a "catch-all" issue for all cases not covered by the above.

@bkutil does this mean we will support writes of nested points too?

Yes, where possible.

I think it makes sense to split this issue according to the tasks (they are themselves pretty big) and maybe keep this one for overall tracking (as an “epic” in scrum terminology). Leaving this to @bkutil.

[ ] nested point with array value
[ ] nested point with list of objects value (that's supported right?)
[ ] reduce by two non-scalar fields
[ ] other "by" invocations such as head by
[ ] sort on non-scalar field
[ ] join on non-scalar field
[ ] pluck non-scalar field
[ ] unique on non-scalar field
[ ] the four above called without parameters on data containing non-scalar

Handle nested points in core views:

[x] file
[x] text
[x] table

Views for juttle-viz are tracked in a separate issue: https://github.com/juttle/juttle-viz/issues/17

Handle nested points in adapters - ensure that nested points get serialized and written out correctly, or that appropriate error is raised when the adapter does not support that.

[ ] influx
[ ] elastic
[ ] gmail
[ ] graphite
[ ] sql
[x] twitter (no write support)

Check nested points in remaining procs:

[ ] skip
[ ] sequence
[ ] split
[ ] remove
[ ] keep

The remaining work to be done is tracked in the juttle viz repo: https://github.com/juttle/juttle-viz/issues/17, closing.


Proposed fix to support DotNet Core 3 preview 4 (#111)

This is my proposed starting point for supporting DotNet Core 3.0 (at least the preview release 4).
It appears that the location of the System.Runtime.Serialization.FormatterServices class has been moved and the GetUninitializedObject made public again.
I'm proposing that a netstandard2.0 target be added so that the original CoreCLR workaround is compiled for netstandard1.6, but that netstandard2.0 and net45 reference the public member directly.

I've expanded the range of DotNet runtimes that the unit test run against and discovered some more issues that need resolving for DotNet Core 3.0. Given the future direction of both Framework and Core being merged into .NET 5 by Microsoft, I suspect that this type of thing will happen at least once more.

Given that netcoreapp3.0 is still in preview, I've removed it as a build target for the tests. Producing the netstandard2.0 package with my additional changes still fixes the operation under the 3.0 preview so I think that it's still useful to keep.

superseeded with #119


Not focusing input fields

Removed autofocus from input fields to prevent automatic scrolling and to avoid the keyboard opening immediately on mobile devices.

This looks good for me. From my side, this PR can be merged.

I changed it and am using the long regex expression from https://stackoverflow.com/a/11381730.
Testing on iPhone and Mac it works

This is not only an issue on mobile. I noticed the issue on my desktop on a longer question. I generally don't mind if a field is focused but under no circumstances should the page scroll down.

I agree that scrolling is annoying too, apparently it can be prevented easily: https://stackoverflow.com/a/48500276

I didn't know the preventScroll option. This should then be used.

I added preventScroll to ExcercieTextInput and FormInputField.
But I think it has a quite strange behavior sometimes. When I load the page it does not scroll down anymore (even when the field is not visible). But when I zoom far in, it still scrolls down (but this also happens if I don't focus any input fields).
Does this work for you?

Currently, if I zoom in, so the text input is not in view, the site does not scroll down. So it looks good on my side. I used this question
tcs.uni-frankfurt.de/algo-learn-testing/pull_request_36/en/recurrence/input/7x3jgce
in firefox.


Disable std::filesystem on macOS when compiling C++17 but targetting <10.15

macOS flat out refuses to compile if you touch std::filesystem::path when targetting macOS < 10.15 (to be able to deploy a binary to older macOS versions) with the -mmacosx-version-min=10.14 (or earlier) compiler flags.
If you try to compile code with ::path (even if you don't actually call it) the Apple clang gods smite you with:
foo.cpp:8:26: error: 'path' is unavailable: introduced in macOS 10.15
        std::filesystem::path p;
                         ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/usr/include/c++/v1/filesystem:738:24: note: 'path' has been explicitly marked unavailable here
class _LIBCPP_TYPE_VIS path {
                       ^

This is reportedly caused by Apple's C++ team being somewhat incompetent at actually updating libc++ when they push out new releases, so libc++ stl features (in this case, to get at the std::filesystem exceptions from libc++) often lag behind the compiler by years, and Apple never releases libc++ updates for older mac versions because they don't care because C++ isn't Swift and only Swift is great and good.
</rant>
Anyway, this PR works around the issue by disabling the std::filesystem::path support when in C++17 mode when targetting macOS before 10.15.
I also broke up the hairy macro conditions because it was already hard to follow and would have gotten worse if kept in a single condition.

Please, can you check and merge?

Yes, nice, thank you!


Publish helm chart also as an OCI package (e.g. on GHCR registry)

Could you push the packaged Helm chart also on ghcr.io, so we can pull/install the helm chart like this:
helm install xxx oci://ghcr.io/xxxxxxxx --version x.y.z

Why? What's the difference?

@max-rocket-internet the difference: #554 and #606


Mark test class single-threaded

Will try to detect these automatically -- https://github.com/prestosql/presto/pull/5412


routing-key-expression for default producer property gets exception on application startup

Why:
To be able to set a routing-key-expression to all producers in the application.
When:
add the following propert to applciation.properties
spring.cloud.stream.rabbit.default.producer.routing-key-expression=headers.routingKey
Expected:
sets the routing-key-expression to all producers in the application.
Actual:
***************************
APPLICATION FAILED TO START
***************************

Description:

Failed to bind properties under 'spring.cloud.stream.rabbit.default.producer.routing-key-expression' to org.springframework.expression.Expression:

    Property: spring.cloud.stream.rabbit.default.producer.routing-key-expression
    Value: headers.routingKey
    Origin: class path resource [application.properties]:37:68
    Reason: No converter found capable of converting from type [java.lang.String] to type [org.springframework.expression.Expression]

What version of spring-cloud-stream you are using?

Sorry I forgot to put it there
3.0.1.RELEASE
The one included in spring-coud Hoxton.SR1.

FWIW can you upgrade to Hoxton SR3?
Meanwhile I'll take a look

Thanks, I cannot upgrade to Hoxton SR3 ath this moment, because of a classpath problem.
Caused by: java.lang.ClassNotFoundException: org.springframework.cloud.gcp.security.firebase.FirebaseTokenValidator
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:581)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
	... 31 common frames omitted

Just an update the case is also reproducible for non-default definitions when used such as
headers.['routing-key']

Similar to #1932 this only happens in multiple binder or environment customisation scenarios due to not linking parent/child context between the app and the binder.


enable junit 5

Signed-off-by: lai<EMAIL_ADDRESS>Description
This pr enables junit 5 test framework with advanced testing features while maintaining backwards compatibility with current test cases in junit 4.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #164 (1611f2d) into main (6843b29) will increase coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #164      +/-   ##
============================================
+ Coverage     70.30%   70.40%   +0.09%
- Complexity     2970     2975       +5
============================================
  Files           268      268
  Lines         14051    14051
  Branches       1409     1409
============================================
+ Hits           9878     9892      +14
+ Misses         3554     3542      -12
+ Partials        619      617       -2

Flag
Coverage Δ

plugin
70.40% <ø> (+0.09%)
:arrow_up:

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

Impacted Files
Coverage Δ

...port/SearchAnomalyDetectorInfoTransportAction.java
60.00% <0.00%> (-4.45%)
:arrow_down:

.../opensearch/ad/cluster/ADClusterEventListener.java
90.90% <0.00%> (+1.81%)
:arrow_up:

...earch/ad/transport/GetAnomalyDetectorResponse.java
70.52% <0.00%> (+5.26%)
:arrow_up:

.../handler/IndexAnomalyDetectorJobActionHandler.java
35.00% <0.00%> (+7.14%)
:arrow_up:


Fix #93: true/false being mis-parsed when part of an attribute name.

cc @jwiegley

With this change hnix /home/deepfire/nixpkgs/pkgs/development/haskell-modules/hackage-packages.nix works.


Will TensorFlow 2.2.0 support CUDA 10.2?

Hi :) I am going to use neural networking and TensorFlow.
I'm trying to install different versions of tensorflow and tensorflow-gpu using pip (for example, 2.1.0 both tensorflow and tensorflow-gpu, 2.2.0-rc0 both tensorflow and tensorflow-gpu) and in Python (3.7) I get error about loading cudart64_101.dll, like this:
>>> import tensorflow as tf
2020-03-31 03:30:42.120394: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'cudart64_101.dll'; dlerror: cudart64_101.dll not found 2020-03-31 03:30:42.134395: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
I copied cuDNN files, also I set CUDA_HOME env. to value of CUDA_PATH env. My hardware meets the requirements.
As far as I understand, TensorFlow 2.1.0 should work fine with CUDA 10.1. But I don't want to use CUDA 10.1 unless emergency, I just install 10.2 and don't want to reinstall it to reinstall back to 10.2 again in future.
I ready to wait for 2.2.0 release, if that makes sense in my case. So my question is: Will TensorFlow 2.2.0 support CUDA 10.2?

@Farxial, To use CUDA 10.2 with Tensorflow 2.2. Please build the Tensorflow from source.
Follow the instructions mentioned here. Thanks

CUDA 10.2 should be compatible with CUDA 10.1. We are building the official pips with CUDA 10.1 as we already changed infrastructure a lot to enable Python3.8 pips. Next release will have infrastructure changed for newer CUDA versions.
Until then, you can try compiling from source, or symlinking the libraries.

Symlinking works.
Nice :)
Thanks for answers :)

@Farxial, Closing since the issue is resolved. Thanks!

The symlink works for me too, details below (installed on Ubuntu 20.04):

actual 10.2 libcudart code is in /usr/local/cuda-10.2
the tensorflow code looks in a number of places (and fails to find in all of them)

strace -o test1.log /usr/bin/python .../quick_tour.py
...
openat(AT_FDCWD, "/home/peter_v/.local/lib/python3.8/site-packages/tensorflow/python/../libcudart.so.10.1", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/home/peter_v/.local/lib/python3.8/site-packages/tensorflow/python/libcudart.so.10.1", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/home/peter_v/.local/lib/python3.8/site-packages/tensorflow/python/../libcudart.so.10.1", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = 20
fstat(20, {st_mode=S_IFREG|0644, st_size=83403, ...}) = 0
mmap(NULL, 83403, PROT_READ, MAP_PRIVATE, 20, 0) = 0x7fb8ad602000
close(20)                               = 0
openat(AT_FDCWD, "/lib/x86_64-linux-gnu/tls/libcudart.so.10.1", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/lib/x86_64-linux-gnu/libcudart.so.10.1", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/usr/lib/x86_64-linux-gnu/tls/libcudart.so.10.1", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/usr/lib/x86_64-linux-gnu/libcudart.so.10.1", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/lib/libcudart.so.10.1", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
openat(AT_FDCWD, "/usr/lib/libcudart.so.10.1", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)

Somewhat at random, I decided to symlink from /usr/lib/x86_64-linux-gnu/ to the libcudart.so.10.2 file.
sudo ln -s /usr/local/cuda-10.2/targets/x86_64-linux/lib/libcudart.so.10.2 /usr/lib/x86_64-linux-gnu/libcudart.so.10.1

Just to confirm, symlink idea works on Windows too. I symlinked C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.2\bin\cudart64_102.dll as cudart64_101.dll in the same folder.

In Ubuntu 20.04 you don't have to symlink, nor build from source. You just need to modify the installation steps in the TensorFlow documentation at https://www.tensorflow.org/install/gpu to match the new Cuda 10-2 package names.
Here are the modifications to the https://www.tensorflow.org/install/gpu instructions that worked for me:
# Download the 10-2 packages
wget https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64/cuda-repo-ubuntu1804_10.2.89-1_amd64.deb
sudo apt-key adv --fetch-keys https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64/7fa2af80.pub
sudo dpkg -i cuda-repo-ubuntu1804_10.2.89-1_amd64.deb
sudo apt-get update
wget http://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64/nvidia-machine-learning-repo-ubuntu1804_1.0.0-1_amd64.deb
sudo apt install ./nvidia-machine-learning-repo-ubuntu1804_1.0.0-1_amd64.deb
sudo apt-get update

# Install the ubuntu drivers, if not done so already
sudo ubuntu-drivers autoinstall

# Install the 10-2 versions of packages
apt-get install -y --no-install-recommends \
cuda-10-2 \
libcudnn7=<IP_ADDRESS>-1+cuda10.2  \
libcudnn7-dev=<IP_ADDRESS>-1+cuda10.2 \
libnvinfer7=7.0.0-1+cuda10.2 \
libnvinfer-dev=7.0.0-1+cuda10.2 \
libnvinfer-plugin7=7.0.0-1+cuda10.2\
cuda-cudart-10-1

This works for a clean install.
For pre-existing configurations you may need to uninstall previous Cuda 10-1 packages beforehand.

Just to confirm solution by @palisadoes works.
Just to make sure you have installed everything (developer version) and then you can run:
sudo apt-get install cuda-cudart-10-1

I had to install libnvinfer-plugin-dev also

Expanding on the Windows fix for people who aren't familiar (like myself) with symlinks and just want it to work.
As admin, in cmd, paste:
mklink /H "C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.2\bin\cudart64_101.dll" "C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.2\bin\cudart64_102.dll"

Alternatively and more clearly written, navigate to the directory and do the same thing:
cd "C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.2\bin"
mklink /H cudart64_101.dll cudart64_102.dll

I'm quite surprised that there doesn't exist out-of-the-box support for CUDA 10.2 yet. I mean, CUDA 11 is out.

after trying most every solution I could find  for windows
turned out tensor flow could not find any dll's .even with setting the system path
"C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.2\bin."
the solution that worked for me with python 3.8 was

import os
os.add_dll_directory("C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v10.2/bin");

all the dlls could be imported and everything works

After many months, CUDA 10.2 still cannot work with TF 2.2?

TF2.2 won't be patched to support newer CUDA versions. We can only bring new versions of CUDA with newer versions of TF (likely 2.4)

Where does tensorflow look for cudcudart? I tried symlinking 10.2 to /usr/lib/x86_64-linux-gnu/libcudart.so.10.1 as per  @petervandenabeele suggestion with no success

I am desperate for 10.2 support! - my company has bought me a graphics card and I can't get it to play with Cuda desite all the above suggestions. I have tried the nightly build of Tensorflow (which is 2.4) - but seems it still looks for 10.1.
Has anybody produced a build that supports 10.2 ?

Please try conda install -c anaconda tensorflow-gpu=1.15.0. Anaconda built TF under CUDA 10.2

Please try conda install -c anaconda tensorflow-gpu=1.15.0. Anaconda built TF under CUDA 10.2

thanks very much.
I did give this a shot, but no luck so far. For one thing it seems the max conda version of tensorflow is 2.1 - actually I need at least 2.2 for what I'm doing (Tensorflow.Net). Also - I am seeing the following error which I can't make much sense of:
The following specifications were found to be incompatible with your CUDA driver:

  - feature:/win-64::__cuda==10.2=0
  - feature:|@/win-64::__cuda==10.2=0

Your installed CUDA driver is: 10.2

I think that in this case the best solution is to try building on the target machine with the 10.2 CUDA headers. NVidia claims compatibility between 10.1 and 10.2 so it should be possible to compile from source and have something working

on my windows machine with RTX2060, symlink works again for the cudnn.
cd "C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.2\bin"
mklink /H cudnn64_7.dll cudnn64_8.dll

Compiled v2.3.1 for Cuda 10.2 in my fork:
v2.3.1-cuda10.2

In Ubuntu 20.04 you don't have to symlink, nor build from source. You just need to modify the installation steps in the TensorFlow documentation at https://www.tensorflow.org/install/gpu to match the new Cuda 10-2 package names.
Here are the modifications to the https://www.tensorflow.org/install/gpu instructions that worked for me:
# Download the 10-2 packages
wget https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64/cuda-repo-ubuntu1804_10.2.89-1_amd64.deb
sudo apt-key adv --fetch-keys https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64/7fa2af80.pub
sudo dpkg -i cuda-repo-ubuntu1804_10.2.89-1_amd64.deb
sudo apt-get update
wget http://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64/nvidia-machine-learning-repo-ubuntu1804_1.0.0-1_amd64.deb
sudo apt install ./nvidia-machine-learning-repo-ubuntu1804_1.0.0-1_amd64.deb
sudo apt-get update

# Install the ubuntu drivers, if not done so already
sudo ubuntu-drivers autoinstall

# Install the 10-2 versions of packages
apt-get install -y --no-install-recommends \
cuda-10-2 \
libcudnn7=<IP_ADDRESS>-1+cuda10.2  \
libcudnn7-dev=<IP_ADDRESS>-1+cuda10.2 \
libnvinfer7=7.0.0-1+cuda10.2 \
libnvinfer-dev=7.0.0-1+cuda10.2 \
libnvinfer-plugin7=7.0.0-1+cuda10.2\
cuda-cudart-10-1

This works for a clean install.
For pre-existing configurations you may need to uninstall previous Cuda 10-1 packages beforehand.

Is this valid only on Ubuntu 20.04? I'm curious if it works for Ubuntu 18.04 as well.


[Event Request] codeunit 1250 "Match General Journal Lines" - OnPrepareCustLedgerEntryForApplicationOnBeforeSetAmountToApply

Describe the request
Hello, can you please add a new Event OnPrepareCustLedgerEntryForApplicationOnBeforeSetAmountToApply in codeunit 1250 "Match General Journal Lines"?
    procedure PrepareCustLedgerEntryForApplication(var CustLedgerEntry: Record "Cust. Ledger Entry"; var GenJournalLine: Record "Gen. Journal Line")
    begin
        CustLedgerEntry.CalcFields("Remaining Amount", "Remaining Amt. (LCY)");
        OnPrepareCustLedgerEntryForApplicationOnBeforeSetAmountToApply(CustLedgerEntry);
        CustLedgerEntry."Applies-to ID" := GenJournalLine."Document No.";
        if Abs(CustLedgerEntry."Remaining Amt. (LCY)") < Abs(GenJournalLine."Amount (LCY)") then
            CustLedgerEntry."Amount to Apply" := CustLedgerEntry."Remaining Amount"
        else
            CustLedgerEntry."Amount to Apply" := -GenJournalLine.Amount;

        CODEUNIT.Run(CODEUNIT::"Cust. Entry-Edit", CustLedgerEntry);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnPrepareCustLedgerEntryForApplicationOnBeforeSetAmountToApply(var CustLedgerEntry: Record "Cust. Ledger Entry")
    begin
    end;

Additional context
We have additional values to calculate the Remaining Amount and need to recalculate it before being allocated and the record being modified.
Internal work item: AB#547389

Availability update: We will publish a fix for this issue in the next update for release 25.
Build ID to track: 26242.


Need more details for versions

We are all looking forward to use this awesome project, since the release of the paper
When trying to build the code we - as a software team - encountered several issues.
However, we could fix one by simply using an other tag (version of Sophus. We used v.0.9.5 , the error was in rigid.h and was gone after this fix.
Now there are many errors coming from Cuda Kernels in relationship with Eigen - so it would be really nice to know which versions / tags etc. were used for the Dependencies, like:

Tensorflow
CUDA
Linux-Version
Sophus
Eigen
Nanoflann
Pangolin

Thank you really much for your help.

+1

Hello,
Here is my versions for the dependencies:
Tensorflow 1.2.0
CUDA 8.0
Ubuntu 16.04 LTS
Sophus SHA1 ID: 341346e306d657ac8acaf052939ffd85dacd8f82
Eigen 3.2.92
Nanoflann 1.2.2
Pangolin 0.5
If you see errors in calling Eigen like "calling a host function from a device function", please download and use our modified Eigen from https://drive.google.com/open?id=0B4WdmTHU8V7VTDFIdU5IWGxkMGM
For Sophus, if you do "git checkout 341346e306d657ac8acaf052939ffd85dacd8f82", should get the same version I used. It seems that the new version removes the SE3Group class.

For Sophus, I see your pull request, and merged it. Thanks!

Thank you very much.
We highly appreciate your fast response!

Thank you for your help,
The Eigen package helped a lot, however we found another dependency "suitespace", which includes the "cholmod.h" - this dependency can be easily installed with:

sudo apt-get install libsuitesparse-dev

Furthermore:

sudo apt-get install libmetis-dev

Also we encountered a linke error refering to ATLAS, which I guess comes from  ATLAS-Bug Ubuntu - but should be fine on Versions greather than 14.04
This bug could be fixed with :

sudo apt-get install liblapack-dev


Allow setting an account for requests impersonification when using service accounts

The https://github.com/googleapis/google-api-php-client library allows to authenticate using Service Accounts keys in JSON format and also impersonificate another user within a domain (which requires domain-wide delegation to be enabled). This is simply done as:
$client = new Google_Client();
$client->setSubject('address_to_act_as@example.com');

Could we please have this functionality as well in this library? From what I've seen there is a class \Google\Auth\Credentials\ServiceAccountCredentials with a constructor's 3rd argument for setting impersonation subject but this is never used/referenced. What's more - hardcoding a valid address to impersonate also fails with the permission denied for provided property id - error code 7.

If you look at the code in Google\Client.php, it uses the same class you're referring to, ServiceAccountCredentials, to set the subject. So using that class and supplying the subject to its constructor would hav the same efect:
$credentials = new Google\Auth\Credentials\ServiceAccountCredentials($scope, $credFile, $subject);
// the configuration option may be different depending on the client
$bigquery = new Google\Cloud\BigQuery\BigQueryClient(['credentialsFetcher' => $credentials]);
$analytics = new Google\Cloud\AnalyticsData\V1\BetaAnalyticsDataClient(['credentials' => $credentials]);

Be sure to check the constructor options for your clients to see which parameter (credentials or credentialsFetcher) it accepts

Thanks @bshaffer for clearing this out. It works perfectly with code snippet you provided

this helped me to thank you
https://github.com/googleapis/google-cloud-php/issues/7257


Improve window

This branch has only a small change to improve the efficiency of the window functions.  The previous had a built in inefficiency when windowing long signals using a copy constructor from the parent. Changes now only create a workspace of the needed size for the returns.  There is a cost as the code is much more fragile now as it has some dependencies on implementation details.  I left a MAINTENANCE node in comments to help all be aware of that issue in the future

Ouch, the new code seg faults.  Suspect I know why, but will take some debugging to find out if my hypothesis is correct.

Codecov Report

Merging #313 (7709e90) into master (e4ee331) will decrease coverage by 0.73%.
The diff coverage is 5.39%.

@@            Coverage Diff             @@
##           master     #313      +/-   ##
==========================================
- Coverage   51.31%   50.57%   -0.74%
==========================================
  Files         123      124       +1
  Lines       17586    17885     +299
==========================================
+ Hits         9024     9046      +22
- Misses       8562     8839     +277

Impacted Files
Coverage Δ

python/mspasspy/db/matcher.py
0.00% <0.00%> (ø)

python/mspasspy/db/database.py
80.10% <28.57%> (-0.09%)
:arrow_down:

cxx/src/lib/algorithms/slice_and_dice.cc
28.15% <100.00%> (+2.90%)
:arrow_up:

cxx/include/mspass/seismic/BasicTimeSeries.h
94.33% <0.00%> (+3.77%)
:arrow_up:

cxx/src/lib/seismic/Seismogram.cc
66.12% <0.00%> (+19.35%)
:arrow_up:

cxx/include/mspass/seismic/TimeSeries.h
100.00% <0.00%> (+23.52%)
:arrow_up:

Continue to review full report at Codecov.

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

I fixed the window code but when I look over this branch I think it got tangled up with some experimental changes in the file mspasspy/db/matcher.py.   I think it may not be too hard to untangle as I am pretty sure all the relevant changes are in slice_and_dice.cc.   Not sure how to proceed.


Postgres role errors on GitHub Actions CI shutdown

I'm seeing a lot of this in our CI runs, even for successful ones:
FATAL:  role "root" does not exist

My first guess is that this happens because the user name is not set for the health check command. Need to verify this.

Fixed.


breaking(Reveal): change prop name

This PR renames effect prop to animated for consistency with Button component.

Current coverage is 99.48% (diff: 100%)

No coverage report found for master at 24e9490.

Powered by Codecov. Last update 24e9490...259ea58

Thanks for the update!


api(queue): add job queue api definition.

Ⅰ. Describe what this PR does
we will implement job-level queue soon and introduce api definition into master branch first.
II. Does this pull request fix one issue?

III. Special notes for reviewers if any.

Codecov Report

Merging #247 (2866e7e) into master (c1f315a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #247   +/-   ##
=======================================
  Coverage   30.19%   30.19%
=======================================
  Files          80       80
  Lines        5219     5219
=======================================
  Hits         1576     1576
  Misses       3427     3427
  Partials      216      216

Flag
Coverage Δ

unittests
30.19% <ø> (ø)

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

Continue to review full report at Codecov.

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


why in windows build so slowly, and in ubuntu build fast

What's the way? to build fast in windows?
Look forward to your reply!

This is not a bug. Please come to the IRC channel, the discord channel, or the users mailing list.
https://discordapp.com/invite/bXVpWAy
#scons on freenode.net
or
https://pairlist4.pair.net/mailman/listinfo/scons-users


Introductory App features

This includes the starting page and a few extra dialogue pages. More details in the figma url.

.


refactored /daemonstatus

refactored /daemonstatus to contain node_name, status and the number of pending, running and finished jobs, tested that it shows the actual load

Not sure why the test is failing now, but would be good to look into.

Restarted the test, and is fine. Hope this happens too often :+1:

Ah, good one, about the iterator! Didn't realise it either when reading :o:

Ah, please mention the issue number when opening a PR :) #23 here
(And as a sidenote, we could also make a list of just the state, not requiring the whole objects in memory; but I'm not sure if that improves the code (readability), I was happy to merge it as it is.)


project name too long

It's very good, but i think the name of this project is too long.

Since I had some technical problems with DUB, I have renamed the project.


Synchronous API?

Is there a way to use grpc-rs without futures, i.e. just synchronously? I ask because I'm dealing with !Sync types, and it's not clear to me whether it's possible to use them with grpcio::RpcContext::spawn.

seem you can use future wait to act as a sync call.

Here's the best I could chalk up with that in mind:
fn transaction(&self, ctx: grpcio::RpcContext, stream: grpcio::RequestStream<request::TransactionRequest>, sink: grpcio::DuplexSink<response::TransactionResponse>) {
    let datastore = self.datastore.clone();
    let trans = datastore.transaction().unwrap();

    for result in stream.wait() {
        match result {
            Ok(request) => {
                let response = match build_response(&trans, request) {
                    Ok(response) => response,
                    Err(err) => build_error_response(&err)
                };

                sink.send((response, grpcio::WriteFlags::default()));
            }
            Err(err) => {
                let response = build_error_response(&err);
                sink.send((response, grpcio::WriteFlags::default()));
            }
        }
    }
}

This doesn't work because grpcio::DuplexSink::send consumes self. Also, I'm guessing grpcio doesn't setup a thread per request, so wouldn't this bottleneck request anyways?
(For context, datastore.transaction() wraps a postgresql transaction, which is not Sync.)

/cc @BusyJay

Hi @ysimonson

Is there a way to use grpc-rs without futures, i.e. just synchronously?

Yes, just as @siddontang s suggestion.

I ask because I'm dealing with !Sync types, and it's not clear to me whether it's possible to use them with grpcio::RpcContext::spawn.

Yes, it can, e.g.,
diff --git a/tests/cases/alarm.rs b/tests/cases/alarm.rs
index 53d8009..01ed67e 100644
--- a/tests/cases/alarm.rs
+++ b/tests/cases/alarm.rs
@@ -27,6 +27,7 @@ struct GreeterService {

 impl Greeter for GreeterService {
     fn say_hello(&self, ctx: RpcContext, mut req: HelloRequest, sink: UnarySink<HelloReply>) {
+        let mut c = ::std::cell::Cell::new(1); // <- It's `!Sync`.
         let (tx, rx) = oneshot::channel();
         let tx_lock = self.tx.clone();
         let name = req.take_name();
@@ -38,6 +39,7 @@ impl Greeter for GreeterService {
             .and_then(move |(greet, _)| {
                 let mut resp = HelloReply::new();
                 resp.set_message(format!("{} {}", greet, name));
+                *c.get_mut() += 1;
                 sink.success(resp)
                     .map_err(|e| panic!("failed to reply {:?}", e))
             });

You can apply this diff and cargo test --all.

This doesn't work because grpcio::DuplexSink::send consumes self.

It does consumes self, but also returns Self. I think you can:
let sink = sink.send(...).wait().unwrap();

Yes, that did it, thank you @overvenus! If anyone else gets stuck on this, this is what I ended up with:
fn transaction(&self, ctx: grpcio::RpcContext, stream: grpcio::RequestStream<request::TransactionRequest>, mut sink: grpcio::DuplexSink<response::TransactionResponse>) {
    let datastore = self.datastore.clone();
    let trans = datastore.transaction().unwrap();

    for result in stream.wait() {
        let response = match result {
            Ok(request) => build_response(&trans, &request).unwrap_or_else(|err| build_error_response(&err)),
            Err(err) => build_error_response(&err)
        };

        sink = sink.send(response).wait().unwrap();
    }
}

(With the usual caveat that unwrap is only used here to make the example more terse.)


Setting up OTBR on standalone Raspi is failing

Following this provided link to setup my Raspi-4 as OTBR
https://groups.zigbee.org/wg/chip-cert-sg/document/23206
Device Details :
Raspi - 4
Ubuntu - 21.04
Cloned "https://github.com/openthread/ot-br-posix" and ran  ./script/bootstrap but I see "libjsoncpp-dev" package is not found
Below is the error

sudo apt-get install --no-install-recommends -y libjsoncpp1 libjsoncpp-dev
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
E: Unable to locate package libjsoncpp1

I also tried "sudo apt-get install libjsoncpp1" but didn't help

Found the bug at https://bugs.launchpad.net/ubuntu/+source/libjsoncpp/+bug/1928695
I think we only need to install libjsoncpp-dev on Ubuntu since it will automatically install the correct libjsoncpp<Version> package.
Ubuntu 21.04:
# apt install libjsoncpp-dev
The following NEW packages will be installed:
  libjsoncpp-dev libjsoncpp24

Ubuntu 18.04:
# apt install libjsoncpp-dev
The following NEW packages will be installed:
  libjsoncpp-dev libjsoncpp1

@ramakrishnanandyala As a temporary workaround, you can remove libjsoncpp1 from the install script.

@simonlingoogle with temporary workaround it worked


FINERACT-557: Readme Improvement

Description
Please refer https://issues.apache.org/jira/browse/FINERACT-557
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Commit message starts with the issue number from https://issues.apache.org/jira/projects/FINERACT/. Ex: FINERACT-646 Pockets API.

[ ] Coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions have been followed.

[ ] API documentation at https://github.com/apache/fineract/blob/develop/api-docs/apiLive.htm has been updated with details of any API changes.

[ ] Integration tests have been created/updated for verifying the changes made.

[ ] All Integrations tests are passing with the new commits.

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the list.)

Our guidelines for code reviews is at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide

Thanks very much for the PR @thesmallstar . Keep up the good work. Are  you a GSoC applicant?

@awasum yes, I will be attempting for GSOC :)
Currently I am understanding the code base and use cases, trying to solve small issues.
Do you have any suggestions?

@awasum yes, I will be attempting for GSOC :)
Currently I am understanding the code base and use cases, trying to solve small issues.
Do you have any suggestions?

Good initiative. Yes here


GitGuardian has uncovered 2 secrets following the scan of your pull request

libs/dh/shared/feature-authorization/src/lib/permission.service.spec.ts and libs/dh/shared/feature-authorization/src/lib/permission.service.spec.ts triggers GitGuardian. This is a properly false-positive, so we should have these files ignored or change the specs, so the secrets aren't triggering GitGuardian.

This has been fixed


Executing some tests fail

Hi,
I want to port your Adaptive Radix Trie to my little database project (https://github.com/sirixdb/sirix) and make it a persistent datastructure + write it to and read it from disk :-)
Currently some tests fail, as attached because of missing default constructors. I have to admit it's not obvious to me how they can be run at all.

Another thing is that the system needs fine granular, fast, random reads to read the variable sized nodes (so, hopefully Optane Memory will be the thing). Furthermore the leaf nodes of the trie can be versioned (storing from the parent node/page a bunch of references to retrieve partial nodes or page-fragments in the SirixDB terminology. I maybe will try to store them within the last InnerNode and version the InnerNodes which then have embedded leaf nodes.This way, only changed records will be stored + some records which are outside of a sliding window (so, if only one value, that is record has been added, it will only store this one and fetch others from previous revisions).
I'm also in contact with Robert Binna for a HOT version, but I think I won't have that much time to start completely from scratch and I'm not really familiar with C++, so reading his Open Source version is not easy for me.
Kind regards and thanks for the great work.
Have a great weekend
Johannes

Hi @JohannesLichtenberger, your project sounds interesting! I have some questions but I'll ask them later after learning more about your project.
The tests you mentioned are not to be run directly. They are used as base classes for other JUnit3 style tests present in com.github.rohansuri.art.acc (acc = Apache Commons Collections). For example the ARTStringTest that inherits AbstractNavigableMapTest. I will package them better to separate JUnit4 tests from JUnit3 tests. Both style of tests exist in the project because I inherited some of them from ACC which are in JUnit3 style.
The error message correctly complains:
junit.framework.AssertionFailedError: Class com.github.rohansuri.art.NavigableKeySetStringTest has no public constructor TestCase(String name) or TestCase()

which is a requirement for JUnit3 style tests. All the tests in *.acc have the required ctor and those can be directly run.
For some reason, the command gradle test testJUnit4 seems to correctly classify them and not run them as a top level test. I'd suggest you use gradle for the time being.

Sadly it's going to be executed with JUnit 4 with gradle test or gradlew clean build --info --stacktrace --refresh-dependencies. Thanks for your interest in the project, that's great :-)


Added username template function.

Added a template function that returns the full name of the user (for example 'Tamer Tas').

@tmrts can I help in any way to merge this PR? Are there any objections?

@adarnimrod sorry for the late response, LGTM :+1:


Automatically perform prune operations on Restic

Description
Currently prune must be performed out-of-band, this issue is to sync delete and prune operations. It needs to check, if this can be performed with active locks on the repo.
In this issue we'll also improve the current locking recovery of the restic repo.
Use case
Ease operations on backup repos
Interface Changes
None
Additional Context
None

Progress Status

[x] Code committed
[x] Testing - QA
[x] Documentation (Release notes - resolved issues, compatibility, known issues)

Fixed as part of #6029


fixed extra requirements

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[x] Bug fix
[ ] New feature
[ ] Other, please explain:

What changes did you make? (Give an overview)
I updated _doGetReqs function to parse transitive dependencies for optional dependencies specified in requirement list avoiding the optional dependencies not specified.
Which issue (if any) does this pull request address?
Fixes #52
Is there anything you'd like reviewers to focus on?
I added a few unit tests to check the behavior of my implementation.

Please merge this ASAP
We have the exact same problem a #52 in our gitlab-ci, this is a very big problem

Thanks very much for this!


Standardize the database connection

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/748
Originally created by @mbergeron on 2019-06-27 13:58:18

Context
Databases should be treated as a Meltano concept, typically defined as an entry in the meltano.yml.
As Meltano uses SQLAlchemy behind the hood, it is natural to use the dialect[+driver]://user:password@host/dbname[?key=value..] scheme as a naming convention across all database connections.
databases:
  - name: warehouse
    namespace: sqlite
    settings:
      - name: dialect
        value: sqlite
      - name: user
        value: warehouse
      - name: password
        value: warehouse
      - name: host
        value: 'localhost:5432'
      - name: dbname
        value: warehouse

Iteration path

Refactor database connection as a Meltano plugin
Refactor the model connection to use the Database definition
Generate loader configuration using Databases, via the kind. For instance, target-sqlite could have these settings:

name: target-sqlite
namespace: sqlite
settings:
  - name: database
    kind: sqlite

We'll have to think carefully where we want the mapping (my guts tells me namespace => kind) to happen, but this would alleviate much of the UX pain for the loaders.

Expose the meltano system database?

View 12 previous comments from the original issue on GitLab


PRP-52 Consider combining the Eligibility Screener and PRP codebases

Ticket
https://wicmtdp.atlassian.net/browse/PRP-52
Changes

What was added, updated, or removed in this PR.

Add ADR for whether to combine the Eligibility Screener and the PRP codebases

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers. Explain how the changes were verified.

Testing

Screenshots, GIF demos, code examples or output to help show the changes working as expected. ProTip: you can drag and drop or paste images into this textbox.

@microwavenby @laurjpeterson I propose that we keep these two codebases separate. Any thoughts or objections?

@microwavenby @laurjpeterson I propose that we keep these two codebases separate. Any thoughts or objections?

We can always merge later. 🤷🏻♀


Questions about Registering Schema

Hi, I just wanted to ask about what internal state does the producer take after failing to register the Schema because its incompatible( Line 592 in ProducerImp ) . From what I have understood the state does not change but then wouldn't be logical to just close the producer. Since going back to Ready state will just result in a deadlock of an endless loop of trying to register the schema.
I hope this is the right place to ask such a question.

If the schema is incompatible, creating a producer should fail. Did you see any problems?

I am writing my thesis about Pulsar and a part of it is to model producers and consumers as state machines. When the reading the code I couldn't figure out what state the producer assumes when going from Ready to RegisteringSchema and then the registering fails. Can you please refer to where in the source code is the state changed to "Failed" when the function tryRegisteringSchema catches an IncompatibleSchemaException ?
I am sorry if this is not the appropriate place to discuss this.
My email<EMAIL_ADDRESS>If you deem it better to discuss this privately.

ClientCnx#sendGetOrCreateSchema will send the request to broker.
ClientCnx#handleGetOrCreateSchemaResponse will handle the response.

ClientCnx#sendGetOrCreateSchema will send the request to broker.
ClientCnx#handleGetOrCreateSchemaResponse will handle the response.

So what does ProducerImpl#tryRegistringSchema does ?


Cross compilation stop working if host OS binary is built

Hi,
I just integrated https://github.com/fatih/images/ to gb.
This works:
env GOOS=linux GOARCH=amd64 gb build
env GOOS=windows GOARCH=amd64 gb build
env GOOS=darwin GOARCH=amd64 gb build

however the following only creates for darwin and skips linux and windows:
env GOOS=darwin GOARCH=amd64 gb build
env GOOS=linux GOARCH=amd64 gb build
env GOOS=windows GOARCH=amd64 gb build

So if the host binary is available inside bin/ folder it skips building for linux and windows. For now I just starting building for linux and windows and then move to darwin:
https://github.com/fatih/images/blob/master/Makefile#L25

Thanks for the report, I know what that's happening, isStale is not taking into account the suffix.

On 4 Sep 2015, at 08:33, Fatih Arslan<EMAIL_ADDRESS>wrote:
Hi,
I just integrated https://github.com/fatih/images/ to gb.
This works:
env GOOS=linux GOARCH=amd64 gb build
env GOOS=windows GOARCH=amd64 gb build
env GOOS=darwin GOARCH=amd64 gb build
however the following only creates for darwin and skips linux and windows:
env GOOS=darwin GOARCH=amd64 gb build
env GOOS=linux GOARCH=amd64 gb build
env GOOS=windows GOARCH=amd64 gb build
So if the host binary is available inside bin/ folder it skips building for linux and windows. For now I just starting building for linux and windows and then move to darwin:
https://github.com/fatih/images/blob/master/Makefile#L25
—
Reply to this email directly or view it on GitHub.

@fatih this should be fixed, but I had some trouble verifying the initial report. Could you please test and let me know if it's still broken. Thanks.

This has been indeed fixed and works as intended. Thanks Dave. Btw this is just a question totally related. the binary names are in the form of:
-rwxr-xr-x   1 fatih  staff  16432868 Sep  8 00:47 images*
-rwxr-xr-x   1 fatih  staff  16528840 Sep  8 00:47 images-linux-amd64*
-rwxr-xr-x   1 fatih  staff  16358400 Sep  8 00:47 images-windows-amd64.exe*

isn't it better if  name images as images-darwin-amd64 when called explicitly with the envs GOOS=darwin GOARCH=amd64? This is something very minor, I'm just curious. Thanks again.

isn't it better if name images as images-darwin-amd64 when called
explicitly with the envs GOOS=darwin GOARCH=amd64?

If we can reliably detect the difference between the value being set, and
being inherited by default from runtime.{GOOS,GOARCH}, I agree. Please log
an issue for the cross compilation milestone.
On Tue, Sep 8, 2015 at 7:50 AM, Fatih Arslan<EMAIL_ADDRESS>wrote:

This has been indeed fixed and works as intended. Thanks Dave. Btw this is
just a question totally related. the binary names are in the form of:
-rwxr-xr-x   1 fatih  staff  16432868 Sep  8 00:47 images*
-rwxr-xr-x   1 fatih  staff  16528840 Sep  8 00:47 images-linux-amd64*
-rwxr-xr-x   1 fatih  staff  16358400 Sep  8 00:47 images-windows-amd64.exe*
isn't it better if name images as images-darwin-amd64 when called
explicitly with the envs GOOS=darwin GOARCH=amd64? This is something very
minor, I'm just curious. Thanks again.
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/337#issuecomment-138383090.

Awesome. Here is the issue: https://github.com/constabulary/gb/issues/346 I couldn't add it to the milestone though (it's forbidden for non collaborator)


Nativefier doesn't work on Windows 7

I've successfully installed nativefier by executing this command:
npm install nativefier -g
When I try to use nativefier I get this error:
> nativefier
ReferenceError: nativefier is not defined
    at repl:1:1
    at ContextifyScript.Script.runInThisContext (vm.js:50:33)
    at REPLServer.defaultEval (repl.js:240:29)
    at bound (domain.js:301:14)
    at REPLServer.runBound [as eval] (domain.js:314:12)
    at REPLServer.onLine (repl.js:441:10)
    at emitOne (events.js:121:20)
    at REPLServer.emit (events.js:211:7)
    at REPLServer.Interface._onLine (readline.js:282:10)
    at REPLServer.Interface._line (readline.js:631:8)
>

This error appears only on Windows 7. On win 10 it works.
My OS is Windows 7
npm 5.5.1
node 8.9.1
How to run nativefier on Windows 7?

I've fixed the issue by restarting my PC.


Make licenses hidden by default

The intent here is to make it easier for contributors to propose new licenses be added, without multiple rounds of back-and-forth to correct metadata, or extensive review by maintainers. In other words, let robots do the work, rather than humans.
There should be no change to the site output or metadata. Instead, this PR does three things:

Makes hidden: true the default for licenses in _config.yml
Adds hidden: false to previously shown licenses, and removes hidden: true from hidden licenses
Adds a test to ensure the number of shown licenses does not expand unless we explicitly update the test.

Fixes https://github.com/github/choosealicense.com/issues/323.

I don't think we should merge requests until they have correct metadata, which robots aren't going to correct, so I'm not sure I understand your first paragraph.
Whether explicit property is set on hidden or shown licenses doesn't make any difference to me; if it does to you and you think flipping a good idea, then I agree. :)
Test looks good.

I don't think we should merge requests until they have correct metadata, which robots aren't going to correct, so I'm not sure I understand your first paragraph.

Ahh. I meant that a contributor could get immediate feedback on whether the license metadata beyond the legal properties (e.g., license name, hidden/not-hidden ,etc.) was at least in line with out contributor guidelines. There have been a few requests to add licenses that I had to say "can you please add hidden: true to the license (for less adopted licenses). This way, the contributor doesn't need to take that step, and if they do add hidden: false, the test should err out, or if they update the test, we'll easily see that in the diff. We'll obviously still have to review the diff and legal properties, but this will at least offload some of the basic "does it meet our guidelines" checks to Travis.

:cool:


command line args aren't passed through to brooklyn.sh

Attempting to start clocker on the command line with brooklyn persistence args fails, there is a ${PERSISTENCE_FLAGS} envvar passed through to brooklyn but not populated by the clocker.sh script.
# ./bin/clocker.sh --persist auto
Cannot find blueprint for network auto
Supported network options: weave, calico, host, localhost

See the ./bin/persist.sh script for an example of PERSISTENCE_FLAGS variable usage

Ah, thanks @grkvlt, I'd just been setting the var in my local env when starting clocker.sh.


Fix for "Multiple App level Alerts have incorrect rendering on low resolution screen" Adding left margin to alert-actions for smaller screens

Signed-off-by: Abhishek Srivastava<EMAIL_ADDRESS>

As far as signing the commit ... Once we have the correct value for margin-left, as @mathisscott said, all you have to do is squash your commits into the one that is signed.


Memory Leak on route navigation

Environment

Operating System: Linux
Node Version:     v16.14.2
Nuxt Version:     3.0.0-rc.12
Nitro Version:    0.6.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://v3.nuxtjs.org/getting-started/introduction
Describe the bug
repeatedly switch tabs from e.g. get started to guide and watch the memory increase.
Fresh load

After 30 swaps

Additional context
For bigger pages this results in 100s of leaked MBs within only a handful of route changes which in turn makes switching routes take seconds and is really bad user experience.
Logs
No response

I believe I have a similar memory leak
I have created this reproduction with nuxt 3.7.4

Initially (on page /) the website takes up 2.4mb of ram.
After navigating to /page/first or /page/second the ram is 4.2mb.
After navigating back to / and forcing gc in chrome devtools, the ram is 4.3mb

I don't think the issue is related to awaiting anything in script setup.
/page/second has the same issue, but it doesn't have any async gaps.
I have tried disabling page transitions, but they don't reduce the issue for me.
The issue probable is somewhere in router, and probably due to suspense.
Removing <NuxtLayout /> seems to fix the issue.
The issue happens because there is an Object retaining a few megabytes of ram somewhere.

Screenshot after GC

Memory inspector in Chrome points to context in t in entry.js:
function xu(e, t) {
  // context t is pointing around here
  const n = (r) => {
    if (!r._vts) r._vts = Date.now();
    else if (r._vts <= n.attached) return;
    Me(Pu(r, n.value), t, 5, [r]);
  };
  return (n.value = e), (n.attached = Tu()), n;
}

It seems to be retaining something in createInvoker()
Perhaps, something keeps holding onto VDOM nodes in onClick listeners somewhere?
On some runs memory inspector points to buildAssetsUrl from path.js, but I'm not sure why.
Also sometimes after leaving a tab a few minutes running some of the ram allocated for
/page/first is cleared (not by gc).

we are facing the same issue. Between routing the dom nodes & memory increase. A bunch of detached elements are still kept in memory.
Picture is from node inspector, while running a local build of our app.

We are also struggling with terrible memory leaks.

At some point the service crashes and is being restarted.

@waldi Hey, have u found some solution for this? I'm at the same situation for now, on prod with every request total heap size increased and doesn't go back for some reason

@waldi @vhovorun If you guys have a reproduction I'd highly recommend opening up a dedicated issue as this one is more related to client side navigation

@waldi Hey, have u found some solution for this? I'm at the same situation for now, on prod with every request total heap size increased and doesn't go back for some reason

@vhovorun I wasn’t really able to fix it, but for a while I was using pm2 auto-restart and memory limits to counter the problem. But the solution still wasn’t great and in the end we fixed it by migrating to cloudflare workers.
No server = No memory leaks

Nuxt is using vue-router transitions - that is what pageTransition enables. You can always disable Nuxt's integration and implement it yourself to test the issue.
More generally, would you see if you can reproduce this in pure Vue SSR without Nuxt (reproduction sandbox), and if so, raise there? 🙏

I’m encountering this memory leak as well only when switching pages. At first I thought it was a problem with the library I was using (vue-pdf) however I’m beginning to think it’s a nuxt issue. This example here helps illuminate the issue very well as the pdf is large and switching pages leaves a ton of detached nodes. It is a minimal repro please take a look. There’s a video in the repo as well showing the memory increasing exponentially.
https://github.com/chronicadventure/tato30-vue-pdf-mem-leak
I also tested this in plain vue 3 and could not get the detached nodes to increase per navigation event; however, it does leave one set of detached nodes which I believe shouldn’t happen in the first place. But the nuxt app continues to leave sets left over after each route change.
https://github.com/chronicadventure/vue-pdf/tree/debug/memory-issues

I've done a minimal reproduction using plain nuxt with page routing. If you spin up this project, navigating from home to "go to large dom" nuxt link creates 2000 detached nodes each time you navigate away from it.
https://github.com/chronicadventure/tato30-vue-pdf-mem-leak/tree/leak-without-pdf

I will have a look. However the last time I did so I found this was reproducible without nuxt, using a combination of Vue, Vue Router, Suspense and transitions - and it might be worth seeing if you can reproduce that way.

I will have a look. However the last time I did so I found this was reproducible without nuxt, using a combination of Vue, Vue Router, Suspense and transitions - and it might be worth seeing if you can reproduce that way.

Do you have any recommendations on disabling suspension and transitions? I don't see anything in the docs hinting at this.


ISSUE: MDI Amazon being removed.

Icon mdi:amazon was removed from MDI, please replace this icon with an other icon in your config, it will be removed in version 2021.12.
Environment (please complete the following information):

OS: Hassio
Type of device: NUC
Home Assistant version: core-2021.10.0

Logs
Icon mdi:amazon was removed from MDI, please replace this icon with an other icon in your config, it will be removed in version 2021.12.
Not sure if moving to mdi:truck-delivery to align with other delivery services.

where can I get the complete list of icons in use (default and custom icons) in my HA installation?

No idea, sounds like a great question for the HA forums.

estion is: where can I get the complete list of icons in use (default and custom icons) in my HA installati

Check out<EMAIL_ADDRESS>


NSG Flow Logs Improvements

Add traffic analytics support for a pre-existing LA workspace
Add support for tags

PR Checklist

[x] I have added example(s) inside the [./examples/] folder
[x] I have added the example(s) to the integration test list for normal (~30 minutes) or long runner >30 minutes
[x] I have checked the coding conventions as per the wiki
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change?

Description

Does this introduce a breaking change

[ ] YES
[x] NO

Testing

cd /tf/caf/examples
az login
terraform init
terraform plan -var-file networking/virtual_network/203-nsg-flow-logs-with-existing-la-and-storage/configuration.tfvars
terraform destroy-var-file networking/virtual_network/203-nsg-flow-logs-with-existing-la-and-storage/configuration.tfvars

Hi @arnaudlh my apologies I have just spotted https://github.com/aztfmod/terraform-azurerm-caf/pull/986 which does some of what this PR does. This one also allows you to specify the region if you're using cross-region flow logs, and it adds support for tags. Let me know what you want me to do in relation to https://github.com/aztfmod/terraform-azurerm-caf/pull/986. I'm happy to wait until that's merged then adjust this one to fit if you like.


Edited row in table does not compre in diff

Hello, I've tried to clone a database, edit a single value in a table row, and launch the program.
The edit passed unnoted, and no diff file was generated.
Conversely, editing the primary key of the table is noted and the diff file is generated.
The databases reside on the same (local VM) server.

Yeah, I've got the same issue. But there are some tables with working diffs...
With a test-table I figured out, that if there are (only) nullable rows, diff doesn't work.

Hi @stevevg,
Please can you post the exact command you're giving to DBDiff?
Also if possible could you post the two tables/databases which you're comparing and which is failing for you?
Thanks

Dear All,
Excellent program. LUV it! Also encountered the same problem with a row that had NULLs. If I update any field from a field that had a NULL value then the diff don't pick it up. Here is a screen shot of the two rows that don't work. Try it...
Let me know if there is any questions. Great program!!!
Ruan Cilliers

Also here is the schema definition of the table. Once I change the "end" field to a value which is not NULL then dbDiff picks up the difference, but if I keep it NULL it does not recognize the diff at all and ignores that line...

I fixed the problem in my pull request.
A workaround is to set one of the servers to "another" host.
Example:
server1-host = localhost
server2-host = <IP_ADDRESS>

Just ran into that issue myself, fixed it similar to dev-jan's PR, changing
https://github.com/DBDiff/DBDiff/blob/0a0021e2e1dd4abea78209b1a4e6df8a6af70c8b/src/DB/Data/LocalTableData.php#L109
to
                return "CAST(IFNULL(`{$p}`.`{$el}`,'') AS CHAR CHARACTER SET utf8),IF(`{$p}`.`{$el}` IS NULL, '+', '-')";

(which is ugly, but got the job done, so..)


Request header parameters not being sent

Hi Yuka,
I could able to successfully configure Swagger ui for my Azure functions. However, I am facing weird issue header parameter(Authentication) value not being sent due to this I am getting 401 response.
Please find below json for reference
{
"openapi": "3.0.1",
"info": {
"title": "Sample Azure Services",
"description": "Azure functions",
"version": "v1"
},
"servers": [
{
"url": ""
}
],
"paths": {
"/SampleService": {
"post": {
"tags": [
"SampleService"
],
"summary": "",
"parameters": [
{
"name": "Authorization",
"in": "header",
"required": true,
"schema": {
"type": "string"
}
}
],
"requestBody": {
"description": "",
"content": {
"application/json": {
"schema": {
"type": "array",
"items": {
"$ref": "#/components/schemas/SampleServiceModel"
}
}
}
}
},
"responses": {
"200": {
"description": "Success",
"content": {
"application/json": {
"schema": {
"type": "array",
"items": {
"$ref": "#/components/schemas/FIleModel"
}
}
}
}
}
}
}
}
}
My function header:
[FunctionName("SampleService")]
[ProducesResponseType(typeof(List), (int)HttpStatusCode.OK)]
[Produces("application/json")]
[RequestHttpHeader("Authorization", isRequired: true)]
public static async Task Run(
[HttpTrigger(AuthorizationLevel.Anonymous, "post", Route = null)] [RequestBodyType(typeof(SampleServiceModel[]), "Sample Service request")] HttpRequest req,
ILogger log)
{
}
CURL:
curl -X POST "" -H "accept: text/plain" -H "Content-Type: application/json" -d "{"input":"string","output":"string","searchFor":"string","replaceWith":"string","searchDirection":"Forward","searchType":"sample","distance":0,"actionType":"actiontype"}"
Can you please help me to resolve issue?

Same for me. Every custom header specified works as expected, except Authorization


Update cache for the command line command.

Provide a flag to force fetch the views and tables to update the cache.
Tested: on staging.
Reviewers:

@mistercrunch , @vera-liu , @ascott

LGTM


Feature Proposal - Pass options to downstream agents to reduce redundancy

Goal
It should be easy to have similar agents to go through the same list of operations but with slightly different options specified by the initial agents.
For example, if I have to monitor a number of websites' (say 10) HTTP status and I would like to send an email notification to its owner when one of the websites changes HTTP status. I can use 10 Http Status Agent to monitor each site's status. After that, I might use Attribute Difference Agent, Email Agent to process and send the email for each of the Http Status Agent simply because the email destinations are not the same.
Unless there is a better way, I have to use 3 agents for each task. It seems to be very difficult to maintain.
Proposal
We can let some types of agents to be able to specify options for its downstream agents. The downstream options should be stored in the generated events and passed down to the next agent. The next agent can consume the downstream options if the GUID matches. Otherwise, it passes the options to the next agent.
In this way, we only need to create 10 Http Status Agent,  one Attribute Difference Agent and one Email Agent by specifying email destination in these Http Status Agent.

Hi!
That's an interesting idea. I think the main problem with implementing would be that many Agents have predefined options and down allow to add custom fields. What could solve your problem is to add the ability to merge incoming Events with the output of any Agent. We have that for a few Agents, but if that would be an option for all Agents you could have one Agent emitting an Event that both contains the URL to be checked an the email that will be used by the EmailAgent.
What you could do right now is using a JavascriptAgent to inject the email into the Event created by the HttpStatusAgent based on the checked url.

Hi @dsander
Thank you very much for your opinion. I still want to add this to Huggin and probably try a PR. Please help me validate the following implementation:

Agents that can_create_events has many downstream_options. A user may select an agent of the same senario and add some options to it. The use may select more agents for customization of other agents.

Hmm I am still not convinced to be honest, that looks like a lot of work and complication of the UX for a very special use case (which I still think is very valid). I don't want to stop you from implementing it, I just want to note that it's a pretty big change that needs to be accepted by maintainers and users.
Did you think about adding a merge option for all Agents that would merge the incoming Event with the newly created one? I still think that this and the use of Liquid in downstream Agents would solve your use case and be easier to implement, but I might be routine-blinded. 😄


Key validity should be 30 days.

This page says "This key is valid for 90 days."  It should say "This key is valid for 30 days."

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

ID: 13f85bf6-849d-020a-7c1b-fc949e102eee
Version Independent ID: 26ce2d38-6da1-59f2-88dd-f386ba61f2e1
Content: StoreContext.GetCustomerPurchaseIdAsync(String, String) Method (Windows.Services.Store) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.Services.Store/StoreContext.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @GrantMeStrength
Microsoft Alias: jken

Reaching out to Store team.

We are changing the way we track issues filed from our documentation pages: please see this blog post for more details. The Microsoft Store documentation has been updated several times since your issue was reported, and there's a good chance your problem has been addressed. Right now, this issue is being closed but if you feel it is still valid and has not been addressed, please re-open it with any more available information. Please note that if you question is more general and not related to documentation, Microsoft Q&A is a great way to get questions answered.


React 19 Actions + RAC Form aren't working properly due to automatic form reset

Provide a general summary of the issue here
React 19 actions has behavior (that was added in canary cycle after Form component in RAC and documentation around it) that it automatically resets form after submission to mimic default browser behavior - PR Link.
I spotted two things that it's breaking while using RAC Form:

This form reset sets the value to defaultValue that was initially passed, not to the current defaultValue one:

https://github.com/adobe/react-spectrum/blob/455cbf45d0641cc4efa1fbe43a68ef858bc84442/packages/%40react-aria/utils/src/useFormReset.ts#L22-L25
This isn't working correctly with the recommended pattern by React that defaultValue represent canonical state from the server.

Errors passed from action state to validationErrors prop on Form aren't properly used because the Form validation state is discarded with form reset event. I'm not 100% sure but I feel like it's here in code:

https://github.com/adobe/react-spectrum/blob/455cbf45d0641cc4efa1fbe43a68ef858bc84442/packages/%40react-aria/form/src/useFormValidation.ts#L46-L48
This breaks the pattern where action returns some additional errors that can be created only from the server. This is also the pattern used in the RAC docs.
🤔 Expected Behavior?
React 19 patterns where you use actions to drive both canonical form state and form errors works correctly. You use defaultValue with the formData to prevent the values from being reset and you can pass errors to Form component and they're properly displayed.
😯 Current Behavior
React 19 automatic reset behavior breaks both form state and errors state
💁 Possible Solution
I'm not sure here
🔦 Context
I'm building an SPA app with React 19 RC version and noticed that actions doesn't really work properly with RAC form handling
🖥 Steps to Reproduce
I created a reproduction:
https://stackblitz.com/edit/vitejs-vite-edgrdn?file=src/App.tsx,package.json
There're three forms rendered:

Using actions without RAC, example of how React actions should be used. The error is properly displayed after submission, the value is being keep because of the defaultValue being updated. Obviously this misses RAC a11y stuff and all the goodies
Using actions with RAC, you can observe that the error isn't properly being displayed after submission and the value is being reset despite defaultValue being updated
RAC without actions, using plain state, just to confirm that it works in React 19 and how +- it's supposed to behave

Version
RAC 1.3.1
What browsers are you seeing the problem on?
Firefox, Chrome, Safari, Microsoft Edge
If other, please specify.
No response
What operating system are you using?
MacOS Sonoma 14.5,  Windows 11
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

Thanks for the issue and all the detailed information. Someone will look into this soon. Thank you for your patience.

Had a little look into this today. In terms of reset and defaultValue, it's even more complex. Fields such as NumberField which actually take a number as their value, cannot be restored by React since the form data is all strings, and it's unlikely a user would be able to always know how to parse those strings, which is why we'll sometimes recommend a hidden input that you maintain with the value you want to go to the server.
We listen for reset already in our form elements, as you noted on useFormReset. Right now that directly calls setState with the first value that the field had. We'd need to get the value off corresponding ref to the input, then apply whatever transformation to get it into a state we understand.
We can't rely on tracking the defaultValue as users may update it too much, so we wouldn't want to be responsive to it. But on the flip side, if we try to only track the defaultValue in a ref, then we have to do that in a layout effect, which would be too late for the reset event. We would always be one state behind.
As for the discarded error on reset, I'm really not sure what to do about that one. I don't quite follow how React envisions getting rid of the error or what the purpose of a Reset button in a form would now be. I tried adding a Reset button to your example, it doesn't do anything. It doesn't clear the errors and it doesn't reset to the actual default of empty string.


Documentation Changes for Contributing.MD

Work Item ID
Description

Added documentation that allows a maintainer or contributor to manually add Database support for any existing dev container definition. This will make it easier for contributors to add DB support for any language pair as needed.
PR Checklist

Use the check-list below to ensure your branch is ready for PR.  If the item is not applicable, leave it blank.

[X] I have updated the documentation accordingly.
[X] I ran the lint checks which produced no new errors nor warnings for my changes.
[X] I have checked to ensure there aren't other open Pull Requests for the same update/change.

Does this introduce a breaking change?

[ ] Yes
[X] No

Other information or known dependencies

This Documentation change is primarily as a result of learnings from adding PostgreSQL DB support to Go and Java.

Yes. 369


Add recommendations for procedures with branches

Say in step three I want my reader to behave one way if they are wearing rainbow socks and another way if they are wearing any other color of socks. How would y'all recommend presenting this? Here are a couple examples:
How I've done this at my past job

Open the box.
Take the sandals out of the box.
Do one of the following:

If you are wearing rainbow-colored socks, put on the sandals over the socks.
If you are any other type of socks, take them off and put on the sandals over your bare feet.

Go out in public.

How I've seen it done at my current job

Open the box.

Take the sandals out of the box.

If you are wearing rainbow-colored socks, put on the sandals over the socks.
If you are any other type of socks, take them off and put on the sandals over your bare feet.

Go out in public.

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

ID: 39b5e3aa-271d-e55e-5c7c-fd31ec3ae360
Version Independent ID: 18c8323f-fa16-669e-2d58-f959b8b772d7
Content: Writing step-by-step instructions - Microsoft Style Guide
Content Source: styleguide/procedures-instructions/writing-step-by-step-instructions.md
Product: non-product-specific
GitHub Login: @pallep
Microsoft Alias: pallep

Hi, This is a good question that others have raised, too.
It's something that we're looking at for future revisions.


StackOverflowError when large number of requests sent

I'm load testing Copycat by sending a large number of small, asynchronous requests. After a certain number of requests accumulate, the system throws a StackOverflowError. I've pasted the stack trace below. If anyone wants to work on this, I'll generate a test case.
2016-12-14 17:35:50 DESKTOP-KVRE8GD io.atomix.catalyst.concurrent.SingleThreadContext 31283 ERROR An uncaught exception occurred
java.lang.StackOverflowError: null
at java.io.ObjectOutputStream.writeNonProxyDesc(ObjectOutputStream.java:1286)
at java.io.ObjectOutputStream.writeClassDesc(ObjectOutputStream.java:1231)
at java.io.ObjectOutputStream.writeNonProxyDesc(ObjectOutputStream.java:1294)
at java.io.ObjectOutputStream.writeClassDesc(ObjectOutputStream.java:1231)
at java.io.ObjectOutputStream.writeNonProxyDesc(ObjectOutputStream.java:1294)
at java.io.ObjectOutputStream.writeClassDesc(ObjectOutputStream.java:1231)
at java.io.ObjectOutputStream.writeOrdinaryObject(ObjectOutputStream.java:1427)
at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1178)
at java.io.ObjectOutputStream.defaultWriteFields(ObjectOutputStream.java:1548)
at java.io.ObjectOutputStream.writeSerialData(ObjectOutputStream.java:1509)
at java.io.ObjectOutputStream.writeOrdinaryObject(ObjectOutputStream.java:1432)
at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1178)
at java.io.ObjectOutputStream.writeObject(ObjectOutputStream.java:348)
at io.atomix.catalyst.serializer.util.JavaSerializableSerializer.write(JavaSerializableSerializer.java:36)
at io.atomix.catalyst.serializer.Serializer.writeById(Serializer.java:896)
at io.atomix.catalyst.serializer.Serializer.writeObject(Serializer.java:885)
at io.atomix.copycat.server.storage.entry.CommandEntry.writeObject(CommandEntry.java:79)
at io.atomix.catalyst.serializer.util.CatalystSerializableSerializer.write(CatalystSerializableSerializer.java:51)
at io.atomix.catalyst.serializer.util.CatalystSerializableSerializer.write(CatalystSerializableSerializer.java:45)
at io.atomix.catalyst.serializer.Serializer.writeById(Serializer.java:896)
at io.atomix.catalyst.serializer.Serializer.writeObject(Serializer.java:885)
at io.atomix.catalyst.serializer.Serializer.writeObject(Serializer.java:820)
at io.atomix.copycat.server.storage.Segment.append(Segment.java:311)
at io.atomix.copycat.server.storage.Log.append(Log.java:294)
at io.atomix.copycat.server.state.LeaderState.applyCommand(LeaderState.java:516)
at io.atomix.copycat.server.state.LeaderState.lambda$sequenceCommand$169(LeaderState.java:493)
at io.atomix.copycat.server.state.ServerSessionContext.setRequestSequence(ServerSessionContext.java:267)
at io.atomix.copycat.server.state.LeaderState.applyCommand(LeaderState.java:524)
at io.atomix.copycat.server.state.LeaderState.lambda$sequenceCommand$169(LeaderState.java:493)
at io.atomix.copycat.server.state.ServerSessionContext.setRequestSequence(ServerSessionContext.java:267)
at io.atomix.copycat.server.state.LeaderState.applyCommand(LeaderState.java:524)
at io.atomix.copycat.server.state.LeaderState.lambda$sequenceCommand$169(LeaderState.java:493)
at io.atomix.copycat.server.state.ServerSessionContext.setRequestSequence(ServerSessionContext.java:267)
at io.atomix.copycat.server.state.LeaderState.applyCommand(LeaderState.java:524)
at io.atomix.copycat.server.state.LeaderState.lambda$sequenceCommand$169(LeaderState.java:493)
at io.atomix.copycat.server.state.ServerSessionContext.setRequestSequence(ServerSessionContext.java:267)
at io.atomix.copycat.server.state.LeaderState.applyCommand(LeaderState.java:524)
at io.atomix.copycat.server.state.LeaderState.lambda$sequenceCommand$169(LeaderState.java:493)
at io.atomix.copycat.server.state.ServerSessionContext.setRequestSequence(ServerSessionContext.java:267)
(Last three lines repeat several hundred more times.)

I think the right solution here is to add some kind of backpressure. When you call copycatClient.submit(operation), it should block temporarily until the system is ready to accept more requests.

Hmm... so the problem here is in sequencing on the leader. Under high load, if a request is lost and a sequence number is skipped, a queue of requests grows fairly large. Once the lost request is resent with the missing sequence number, it causes the queue to be drained into the log which causes a huge stack. We should probably reject requests with sequence numbers x larger than the last in sequence request to prevent memory consumption issues from the queued requests anyways, and that should reduce the stack size when the queue is emptied. There may also be a way to empty the queue without recursion.

This was fixed in #267


Implement basic nf_EventHandler for AdguardNetApi.dll

I suggest to implement it in AGWinProxyServer, but with external dependency on AdguardNetApi.
It should redirect to proxy port in tcpConnectRequest and store original destination address / appName in temporary map.
Before calling onNewConnection_cb in AGWinProxyServer, this destination address may be resolved from this map.
If this can't be implemented without moving AdguardNetApi code to CoreLibs, let's move it.

Done


pypi release step

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


Issue with T8161 and T8134 Cameras Motion Detection

Thank you for your work on this integration. Currently, I have two cameras: one is a T8161 (Firmware: <IP_ADDRESS>), and the other is T8134 (Firmware: <IP_ADDRESS>). Neither of them is triggering the motion detection sensor. I am aware that most of the logic is handled by the Eufy WS add-on. I'm able to turn the cameras on and off, but I feel like that's the extent of what's working. Have you done any testing with these camera models? I have some python coding experience, so I'm willing to help and not just offload my issue to the community. If you have suggestions on where to look in the code, that would be extremely helpful. I don't see any errors in the logs.

Same issue here with both T8210 doorbell and T8410 Pan&Tilt camera.
When using the switch for activating and deactivating Motion Detector nothing happens. But other switches, such as turn on/off camera works fine.
I have tried to restart the add-on and my Home Assistant, but to no avail.

It would seem like a restart of the camera and homebase fixed my issues with the switch buttons not working.
So I suggest you try and do the same.
Br

I restarted the cameras and homebase, but that did not resolve my issues. I'm still unable to receive any motion sensor event data.

add-on should send the the motion event to integration and add-on relies on push notifications to be enabled to receive these events.


Adding mapper could fail under JPMS

Under JPMS (Java Platform Module System), ClassLoader#getResources() could throw FileSystemException it seems.
Should fix #2598

Coverage remained the same at 87.309% when pulling ced797363fbb9bb31c1b404c8fa3d39cfcfda8e2 on harawata:jpms-file-system-exception into afc439a9d3624c1cb4d7c129e3c3c7bda6aa8d8f on mybatis:master.


Issue with k8s.io/docs/tutorials/stateless-application/hello-minikube/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
Great Minikube tutorial. All steps smoothly working for me until I got to this step:
minikube addons open heapster

Documented (expected) output:
Opening kubernetes service kube-system/monitoring-grafana in default browser...

But actual output:
This addon does not have an endpoint defined for the 'addons open' command
You can add one by annotating a service with the label kubernetes.io/minikube-addons-endpoint:heapster

(Page accessed just now, 2018-03-23 @ 3pm Pacific time.)
Proposed Solution:
Page to Update:
https://kubernetes.io/docs/tutorials/stateless-application/hello-minikube/#enable-addons

sounds good, thanks @zacharysarah


Engine will not build.

Describe your bug here.
I downloaded the 0.5 version of PsychEngine, and tried to build it for the first time.
I got this as an error

As a note, I did not TOUCH the storymenustate for the mod, so I just got a random error that is keeping me from building the engine.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
Nope. Nothing.

Update Haxe

Update Haxe

Do I do it through command prompt commands or just from the website?

Update Haxe

Do I do it through command prompt commands or just from the website?

I recommend the website

Update Haxe

Do I do it through command prompt commands or just from the website?

I recommend the website

Uhhhhhhhhhhh don't think that really fixed anything

Was already fixed on main repo.


Error, When building AOT with ng build --prod --aot

This issue is happening on ubuntu, when trying to build AOT build.
ERROR in /usr/src/app/my-app/src/$$_gendir/app/auth/auth.component.ngfactory.ts (597,168): Property 'required' does not exist on type '{ [key: string]: any; }'.
Depending upon which validation I used, Property changes to email, range length or e.t.c.
I am using "@angular/cli": "^1.0.0-rc.1" and     "ng2-validation": "^3.1.3".
Thanks

@NomiJ Have you tried the latest version? 3.9.1 is the latest version of ng2-validation.

Thanks, upgrading the version. I don't see errors any more. On the side note, I see this
+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>I am not sure if it actually mean that ng2-validation does not support angular 4.0.1 or not. Kindly let me know.
Thanks

Don't worry. It works well.


API differences with filippo.io/edwards25519

filippo.io/edwards25519 was a later attempt at building an API, and differences emerged in two parts.
First, instead of having identity and generator setters (called Base and Zero here, which I don't love), we have two separate New methods. I am not positive the new pattern is better, but I like the names better, and it avoids the "what does NewElement return?" question.
func NewElement() *Element
func (e *Element) Base() *Element
func (e *Element) Zero() *Element
// vs
func NewIdentityPoint() *Point
func NewGeneratorPoint() *Point

Then, the serialization and deserialization functions. This package has a Decode that can't be chained, a FromUniformBytes that panics, and an append-like Encode. The names map to the spec so they should probably stay (although I like the "set" verb better than the "from" preposition), but everything else is suboptimal. Decode should chain, Encode should be outline an allocation, and FromUniformBytes should return an error (matching Decode).
func (e *Element) Decode(in []byte) error
func (e *Element) Encode(b []byte) []byte
func (e *Element) FromUniformBytes(b []byte) *Element
// vs
func (v *Point) Bytes() []byte
func (v *Point) SetBytes(x []byte) (*Point, error)

The version is still v0, do we feel like breaking the API? Do we want to make the jump to v2?

Similarly, Scalar has an extra func (s *Scalar) Zero() *Scalar setter, and the same serialization differences.
func (s *Scalar) Encode(b []byte) []byte
func (s *Scalar) Decode(x []byte) error
func (s *Scalar) FromUniformBytes(x []byte) *Scalar
// vs
func (s *Scalar) Bytes() []byte
func (s *Scalar) SetCanonicalBytes(x []byte) (*Scalar, error)
func (s *Scalar) SetUniformBytes(x []byte) *Scalar

I do slightly prefer FromUniformBytes here, since that's a derivation process (wide reduction) and not an assignment process. Was there a reason we changed it in fio/edwards25519? Consistency with SetCanonicalBytes, which I do think is correct?

Consistency with SetCanonicalBytes, as well as keeping verbs in front for things that change the receiver: Add, Negate, Set... From?

Ah, good point about the verb idiom. ReduceUniformBytes? I don't feel strongly about it, but the Set- prefix suggests to me that the input isn't significantly transformed. Won't hold it up over this if you disagree or don't think it matters.

I do like ReduceUniformBytes, but I am not sure it's worth deprecating SetUniformBytes in fio/edwards25519 for... and I like that all the various ways to set a value end up next to each other in the docs.


clangd does not identify basic headers (iostream , vector , etc.)

Hi, I really don't know if it's a bug or a simple misconfiguration of mine, but coc-clangd doesn't identify headers like , , etc.

Can you share a screenshot of your code(header) or your config file to make it easier?

https://clangd.llvm.org/troubleshooting#cant-find-standard-library-headers-map-stdioh-etc
Sorry, there's no actionable info in this report.


Add TS_EVENT_AIO_DONE, deprecate TS_AIO_EVENT_DONE (8.0.x).

(cherry picked from commit 8b40c578c77d8a31ee4fb57a912f269ad9a23aec)

@bryancall I think this was a clean cherry-pick so you don't really need this branch.  It should go into 8.  It would not have a 9.0.0 milestone, this was cherry-picked from master.


[Feature Request]: Box Furnace funnel to exhaust generated gas

Email (Optional)
No response
Problem
Currently, if we synthesize gas-generating chemical spaces, the gas can fill the A-Lab room.
Example chemical spaces:

Na-Sn-P
Na-Sn-Mg-P
Na-Mg-P

Proposed Solution
Box furnace funnel is decided in conjunction to structure to hold the exhaust on top of the box furnace. However, we need funnels that is not glued. These funnels have been ordered since November but delayed each month for some reason. We need another vendor or solution. Maybe mounting smaller tubes is ok as well.
Alternatives
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

All components are purchased from mcmaster. The funnel from VWR is a dead end.

Check if pick and place box furnace D will have issue with the new exhaust installation

Tested.

Sorry need to check pick and place box furnace D.


[WIP] Human APP backend - 2nd milestone

Description
This PR adds the code from 2nd milestone for Human App backend-for-frontend

Summary of changes

oracles discovery
jobs discovery
job assignment
statistics
error handling refactoring

How test the changes
There are unit tests for all the features. To run them please execute yarn test

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: macnablocky:white_check_mark: MWBlocky:x: MicDebBlockyYou have signed the CLA already but the status is still pending? Let us recheck it.


set larger haproxy maxconn for scale out tests

the original maxconn value (2048) is too small. Increase it to 500K per backend.
As doc online, a haproxy is able to process 2M connections.
Reference https://www.freecodecamp.org/news/how-we-fine-tuned-haproxy-to-achieve-2-000-000-concurrent-ssl-connections-d017e61a4d27/#:~:text=HAProxy Configuration,-This is probably&text=The maxconn setting allows us,support overall (one way).&text=The max open files has,being set at 2 million.

/approve

/lgtm


1.0.0a3 release

Current dependencies on/for this PR:

main

PR #144

PR #145   👈

This comment was auto-generated by Graphite.


SQL Saturday Haiti 2024

Event Name: SQL Saturday Haïti 2024
Event Date: 27 Apr 2024
Details in email

Event complete


Feature/i18n enhancement

Extending i18n support to include plural translations.

Added this rebased and with a simplified diff to a local branch.

See #1290.


entity things

knocking out some unmapped entity-related classes. never mind. this became a 100% of entity AI
refactors:
run -> create: these methods create a TaskControl, not run a task
ArmadilloWhatever -> MobSensor: the mojang name is MobSensor, which indicates that it will in the future be used for things other than armadillos. it's also possible for modders to use this to sense any entity. best to rename now!

this is why we need to make enigma drop invalid parameters :p
I'll remove them!

confirmed that both parameters are out of bounds and removed