Max. recursion depth with nested structures exceeded at /usr/lib64/perl5/vendor_perl/Storable.pm

[postgres@perf-psql13vm log]$ /usr/local/bin/pgbadger postgresql-Fri.log   -j 4 -o /tmp/pg-13-AUG-21.html
I am receiving an error while trying to analyze large log file:
Max. recursion depth with nested structures exceeded at /usr/lib64/perl5/vendor_perl/Storable.pm line 328, at /usr/lo                                       cal/bin/pgbadger line 15114.
can not load incompatible binary data, binary file is from version < 4.0.
LOG: Ok, generating html report...
[postgres@perf-psql13vm log]$
[postgres@perf-psql13vm log]$ /usr/local/bin/pgbadger --version
pgBadger version 11.5

OK, I have solved my problem. I re-installed the Storable module using CPAN and added the following 2 lines to the pgbadger script:
$Storable::recursion_limit=-1;
$Storable::recursion_limit_hash=-1;
The error is still thrown but the HTML report is usable.
[postgres@perf-psql13vm log]$ /usr/local/bin/pgbadger postgresql-Fri.log   -j 4 -o /tmp/pg-13-AUG-21.html
Max. recursion depth with nested structures exceeded at /usr/lib64/perl5/vendor_perl/Storable.pm line 328, at /usr/local/bin/pgbadger line 15115.
Max. recursion depth with nested structures exceeded at /usr/lib64/perl5/vendor_perl/Storable.pm line 328, at /usr/local/bin/pgbadger line 15115.
can not load incompatible binary data, binary file is from version < 4.0.
can not load incompatible binary data, binary file is from version < 4.0.
LOG: Ok, generating html report...

Here is the final file which no longer throws errors.
pgbadger.txt
[postgres@perf-psql13vm log]$ /usr/local/bin/pgbadger postgresql-Fri.log   -j 4 -o /tmp/pg-13-AUG-21.html
[========================>] Parsed<PHONE_NUMBER> bytes of<PHONE_NUMBER> (100.00%), queries:<PHONE_NUMBER>, events: 22
LOG: Ok, generating html report...
[postgres@perf-psql13vm log]$

Whet does perl -E 'use Storable qw(dclone); say "version=",$Storable::VERSION; say "limit=",$Storable::recursion_limit; my @tt; for (1..1_000_000) { my $t = [[[]]]; push @tt, $t } dclone @tt' returns?

[postgres@perf-psql13vm log]$ perl -E 'use Storable qw(dclone); say "version=",$Storable::VERSION; say "limit=",$Storable::recursion_limit; my @tt; for (1..1_000_000) { my $t = [[[]]]; push @tt, $t } dclone @tt'
version=3.15
limit=512
Not a reference at -e line 1.
[postgres@perf-psql13vm log]$

hum, you should not have the error message, I have the same result. If you can give me a link on my private email  [ gilles AT darold DOT net ] to download your log file so that I can try to reproduce the issue this will be helpful. If you can't, no problem I understand.

Sorry Giles, the file is already gone. I would have uploaded it, there is no confidential information in there, but it was cleaned up after I upgraded to 13.4.

Ok, if you have the same error again do not hesitate to send me the log file. Thanks.


Fixing Wait For Navigation fails: 'str' object has no attribute 'name

#2081

@allcontributors please add @luchoagomezt for code.


Make ToTokens more lenient when generating tokens for invalid ASTs

This is my first pass at making our ToTokens implementations more lenient when it comes to generating tokens for invalid ASTs. I have some comments in the patch in the places which I both a) noticed as I read through all of our ToTokens implementations, and b) didn't fix.
This patch contains a few large if conditionals, and I'm really not sure how to nicely format those in the style used by syn, so I just... gave up? I tried running rustfmt for inspiration, but the style syn uses is so far from rustfmt's suggestions right now that it felt out of place. I'd love suggestions as to how to fix them to not be so ugly.
Fixes #177

Thanks for merging this @alexcrichton. I didn't have enough time to fix it up :).


make run doesn't  work on mac

Following the README instructions on mac, I ran:
make -f Makefile build && make -f Makefile run

build step completes. And the it runs:
cd ../../models && \
        podman run -it -d -p 8001:8001 -v ./mistral-7b-instruct-v0.1.Q4_K_M.gguf:/locallm/models/mistral-7b-instruct-v0.1.Q4_K_M.gguf:ro -e MODEL_PATH=/locallm/models/mistral-7b-instruct-v0.1.Q4_K_M.gguf -e HOST=<IP_ADDRESS> -e PORT=8001 quay.io/ai-lab/model_servers/llamacpp_python:latest

The container instantly crashes with:
llama_model_load: error loading model: failed to open /locallm/models/mistral-7b-instruct-v0.1.Q4_K_M.gguf: Permission denied
llama_load_model_from_file: failed to load model
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/opt/app-root/lib64/python3.11/site-packages/llama_cpp/server/__main__.py", line 88, in <module>
    main()
  File "/opt/app-root/lib64/python3.11/site-packages/llama_cpp/server/__main__.py", line 74, in main
    app = create_app(
          ^^^^^^^^^^^
  File "/opt/app-root/lib64/python3.11/site-packages/llama_cpp/server/app.py", line 138, in create_app
    set_llama_proxy(model_settings=model_settings)
  File "/opt/app-root/lib64/python3.11/site-packages/llama_cpp/server/app.py", line 75, in set_llama_proxy
    _llama_proxy = LlamaProxy(models=model_settings)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/app-root/lib64/python3.11/site-packages/llama_cpp/server/model.py", line 31, in __init__
    self._current_model = self.load_llama_from_model_settings(
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/app-root/lib64/python3.11/site-packages/llama_cpp/server/model.py", line 138, in load_llama_from_model_settings
    _model = create_fn(
             ^^^^^^^^^^
  File "/opt/app-root/lib64/python3.11/site-packages/llama_cpp/llama.py", line 314, in __init__
    self._model = _LlamaModel(
                  ^^^^^^^^^^^^
  File "/opt/app-root/lib64/python3.11/site-packages/llama_cpp/_internals.py", line 55, in __init__
    raise ValueError(f"Failed to load model from file: {path_model}")
ValueError: Failed to load model from file: /locallm/models/mistral-7b-instruct-v0.1.Q4_K_M.gguf

I think that the it might be an issue with the BIND_MOUNT_OPTION in the model_severs common Makefile.

Update:
This works as long as I download the model with make download.

Update:
Thanks to @Gregory-Pereira help, we were able to figure out that this was ultimately a user error on my part.
Using the Z bind mount option when running the model server, which I've been doing for the last few months, adds a security.selinux tag to all of the files on the localhost that are part of the mounted directory. This prevented the use of the ro option enforced by the makefile.
Looks like this issue can be avoided, by only using ro and not Z when using volume mounts on Mac.


Modify default label for historic range base year

Currently blank, as default is to not send the parameter.
Text to substitute for blank string as yet undecided.
See comment here.

When we don't send the historic base year the server ends up deciding which one to use it for us, so we should just nix the default and force the user to always pick one.


PnP PowerShell March 2019 Updates

OPS Build status updates of commit 656317f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/office-docs-powershell, working directory: W:\o2ni-s, target branch: master, target branch commit: ab6ab59086bc78fd43b6f4c096ebd5d5417ba8a0, source branch: eb98490187e7496e08284c71e816c381/PnPPSMarch2019, source branch commit: .

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


Change webpack setup for faster builds

Development speed times are currently slow due to slow builds, which makes it hard to test code against the examples pages.
This PR speeds up development build times for the draggable library, by separating between development and production builds. The development bundle will be placed at<EMAIL_ADDRESS>which will be within the examples scope and therefor make path lookups for its build time a lot faster.
The new setup also outputs untranspiled ES6 code by default now and creates an optional transpiled ES5 counterpart bundle namespaced under es5/.
Import examples
// ES6 defaults
import {Draggable} from '@shopify/draggable';
import Sortable from '@shopify/draggable/lib/sortable';
import Droppable from '@shopify/draggable/lib/droppable';

// ES5 counterparts
import {Draggable} from<EMAIL_ADDRESS>import Sortable from '@shopify/draggable/lib/es5/sortable';
import Droppable from '@shopify/draggable/lib/es5/droppable';
// etc

The examples pages can now use Draggable via @shopify/draggable module

🎉


Exported project on android crashes immediately in master

Godot version:
Master
OS/device including version:
Android 10
Google Pixel
Issue description:
Running anything exported from a master build causes the app to crash immediately.
Reverting all the commits in https://github.com/godotengine/godot/pull/33910 makes the issue go away CC @Faless @m4gr3d
Steps to reproduce:

Build from master
export to android

Output from adb logcat:
log.txt
The following line appears to be the most prominent error
java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/data/app/org.godotengine.test3d-0O2E8CAMe9vWVpzNZ2brXw==/base.apk"],nativeLibraryDirectories=[/data/app/org.godotengine.test3d-0O2E8CAMe9vWVpzNZ2brXw==/lib/arm, /data/app/org.godotengine.test3d-0O2E8CAMe9vWVpzNZ2brXw==/base.apk!/lib/armeabi-v7a, /system/lib, /vendor/lib, /system/product/lib]]] couldn't find "libgodot_android.so"

I have same issue but got this in adb:
12-17 14:13:27.070: E/AndroidRuntime(904): 	at org.godotengine.godot.utils.PermissionsUtil.hasManifestPermission(PermissionsUtil.java:172)
12-17 14:13:27.070: E/AndroidRuntime(904): 	at org.godotengine.godot.utils.GodotNetUtils.<init>(GodotNetUtils.java:50)
12-17 14:13:27.070: E/AndroidRuntime(904): 	at org.godotengine.godot.Godot.initializeGodot(Godot.java:507)
12-17 14:13:27.070: E/AndroidRuntime(904): 	at org.godotengine.godot.Godot.onCreate(Godot.java:673)

As you can see, a error message says GodotNetUtils.java:50 which does seem to have been changed in https://github.com/godotengine/godot/pull/33910.


Scan this code with your HomeKit App on your iOS device to pair with Homebridge:

    ┌────────────┐
    │ 031-45-154 │
    └────────────┘

[1/19/2017, 9:15:09 AM] [websocket] Plugin - DidFinishLaunching
[1/19/2017, 9:15:09 AM] [websocket] Number of Accessories: 0
[1/19/2017, 9:15:09 AM] Homebridge is running on port 51826.
[1/19/2017, 9:15:09 AM] [websocket] url {"address":"::","family":"IPv6","port":4050}

I didn't use homebridge-websocket for months, so I have to install it on my mac and do some tests. ...
Are your accessories stored in the file /.homebridge/accesories/cacheAccessories?

Are your accessories stored in the file /.homebridge/accesories/cacheAccessories?

[]

empty. :(

any backups? macOS with Time Machine?

Maybe I need to better understand how your plugin works...
How is it that a link is established between an accessory added via the plugin and the actual real life accessory added to homebridge?  I've looked dover the code and can't figure out how you're doing that.

any backups? macOS with Time Machine?

Well I'm not sure it has ever populated the cache, but homebridge has functioned properly for a long time for me regardless.

Closing for now.

I'm trying to understand how this plugin works. Should it discover existing accessories added to config.json of Homebridge or does it only discover accessories added via web socket?

Well, same problem here. HomeBridge websocket plugin worked for a long time and then suddenly, Number of Accessories: 0 is reported by the websocket plugin...


Server hangs forever

I am using the same setup as described in #100 and the request just hangs indefinitely. Any idea what I can do?
Also, but unrelated to this issue: is it possible to stop the server using ctrl+c? I always have to delete the container through Docker Desktop...

PS: it gives me 404 when I access https://localhost/ and hangs when I access https://localhost/index.php.
In the server log, only the request to https://localhost/ is logged.

Can you try https://localhost/worker.php?

Still hangs. Errata: my file was named index.php (and also the command used index.php instead of worker.php). Same with worker.php as filename.

Am I as the user supposed to rewrite all requests to the worker.php in production? Or does frankenphp do this automatically?


Revise building-p4-programs.md

Practically speaking, this document is primarily about installing the ES2K P4 compiler and tools. This is a separate task from compiling the P4 program and deploying its artifacts. It's something you would do once, as apposed to the many times you would repeat the compile-prepare-deploy cycle once the tools were installed.
I would recommend making sections 1-3 as the basis of an "Installing" document and using sections 4-5 as the foundation of a separate "Building and Deploying" document.


Create Openalpr Camel component

Done - https://rhiot.gitbooks.io/rhiotdocumentation/content/cloudplatform/camel_components/camel_openalpr.html .


Govee dual plug

I added new govee dual plugs and I can not control each outlet individually it only turns on the entire plug.

Came here to say the same thing hopefully this can be fixed

I looked at the api response for dula plug devices. This is how it looks, under 'deviceName' we have 'name' for each outlet and value is 1_0, 1_1 for outlet 1, 0_0, 0_1 for outlet 2.
{
"data": {
"devices": [
{
"device": "0F:6E:D4:AD:FC:XX:XX:XX",
"model": "H5082",
"deviceName": "Room Plug",
"controllable": true,
"properties": {
"toggle": {
"options": [
{
"name": "Outlet 1 ON",
"value": "1_1"
},
{
"name": "Outlet 1 OFF",
"value": "1_0"
},
{
"name": "Outlet 2 ON",
"value": "0_1"
},
{
"name": "Outlet 2 OFF",
"value": "0_0"
}
]
}
},


Add PUCESD University to domains

PR for adding PUCESD domain to list
University url: https://pucesd.edu.ec/

@andresandoval I am sorry for a delay with processing your request.
Pull request merged. Thank you!


Flexibility in configuration of directories to better support rails/webpacker?

I'm considering a couple of options for React on Rails regarding directory configuration to work better with rails/webpacker.

Client Side Code: All files related to generation of client side bundles, including JS, CSS, images, webpack configuration, node modules, etc.

/client/node_modules
/client/package.json

rails/webpacker directory structure:

/app/config/webpacker.yml
/bin: binstubs of webpackandwebpack-dev-server`
/package.json
/node_modules

In my branch for 9.0, I've added this:
  config.node_modules_path = '' # Pre 9.0.0 used /client

And I'm going to change the default installer to use a top level /node_modules.

can you also add support for webpacker.yml.erb in case ruby code needs to be injected into the yml?

@nratter you sure can! try it and let me know how it works out.
This issue is closed as the flexibility to do any setup is just based on:
this for 8.0 and prior default:
config.node_modules_location = "client"

or for the 9.0 default:
config.node_modules_location = ""


dynamic ports

Rancher Version:
v0.63.1
rancher/server           latest              f2584280e144        10 days ago         768.1 MB

Docker Version:
Docker version 1.10.1, build 9e83765

OS:
rancheros 0.4.3

Steps to Reproduce:
CREATE A MULTI-CONTAINER APPLICATION USING RANCHER COMPOSE (follow the steps in [quick start guide](http://docs.rancher.com/rancher/quick-start-guide/)
need set dynamic ports on mywordpress containers, and loadbalancer (wordpresslb) can connnect to that dynamic ports automatically

Results:
not sure if Rancher compose support dynamic ports or not.

Expected:
Start services with dynamic ports, Docker Datacenter (UCP) has this function already.

I'm not sure what the question is here, but the load balancer connects to target services using the overlay network and does not need the target port to be published at all, whether dynamically or statically assigned.

seems Rancher support random ports
http://docs.rancher.com/rancher/rancher-ui/infrastructure/containers/


coredns not clear

why fork from non original repo
how do we know what got changed on this repo?
how do we keep it up to date?

➤ Christophe de Carvalho commented:
So the reason we fork CoreDNS was because they didn't support to connect to ETCD that was password protected.
So we did implement this on our fork: https://github.com/threefoldtech/coredns/commit/8feb641500ffa726847ddada677f7debf05519c9
Our CoreDNS template ( https://github.com/threefoldtech/0-templates/tree/development/templates/coredns ) and SAL ( https://github.com/threefoldtech/jumpscale_lib/tree/development/JumpscaleLib/sal_zos/coredns ) uses an flist build from this repository.
Since then I re-implemented our fix in a cleaner way and proposed it upstream: https://github.com/coredns/coredns/pull/2442
My patch has been merge already.
What left to be done is to modify our code to deal with the version from upstream again.
Once this is done we can get rid of our fork since it won't contain anything specific for us anymore.


add Dependabot

https://github.blog/2020-06-01-keep-all-your-packages-up-to-date-with-dependabot/

Coverage increased (+0.3%) to 92.625% when pulling a37835461054af3a5e9a06d3cde9b3cd2cdafd6f on sullis:add-dependabot into 9a90624aa0a50f1785d6a10e1fcb2a296ce468f7 on vdurmont:master.

LGTM @vdurmont

ping @vdurmont


updated README to display size badge

in reference to https://github.com/ApoorvSaxena/lozad.js/issues/63


add Riot and Domrender to the 'Extreme Benchmark'

Thanks! I'll check this out as soon as possible..

Are you using DOM reordering with Domrender?

No, I am not using DOM reordering. If this benchmark is supposed to only test DOM reordering, I am fine removing the pull request. :)

Yeah, it is.. ;) Of course replacing textContent will be faster.. You can do that with FRZR by removing the key attribute (with just view.reset(data))

I think Riot also doesn't use reordering by default no more..

Got it. Thank you.

Thanks anyway! If you could modify them to use reordering, would be great to see the performance! Riot is actually using almost similar reordering, since I'm one of the contributors..

Cool. I will consider adding reordering. Thank you for the feedback.


Fix 483: string readers

This PR fixes a bug discovered in #483.

Current coverage is 88.40%

Merging #485 into master will not affect coverage as of db89c88

Powered by Codecov. Updated on successful CI builds.


Error when using the class_weight parameter in the fit function in tensorflow.python.keras

OS Platform and Distribution: Linux Ubuntu 16.04
TensorFlow version: 1.13.1
Python version: 3.6

I wanted to test my network on a small set of records with two imbalanced classes (0 and 1). I was going to use the class_weight parameter in the fit function to improve the balance, but unfortunately there were problems probably associated with tensor support. Without the class_weight parameter, the fit function works correctly. I also tried to use tensorflow.python.keras.layers.Input as an input to the Xception model, but with the same effect.
import tensorflow as tf
from tensorflow.python.keras.layers import Dense, Dropout
from tensorflow.python.keras.applications.xception import Xception, preprocess_input
from tensorflow.python.keras.models import Model
from tensorflow.python.keras.optimizers import Adam

# parsing images and labels from TFRecords
def parse_function(proto):
    example = {'image_raw': tf.FixedLenFeature([], tf.string), 'label': tf.FixedLenFeature([], tf.int64)}
    parsed_example = tf.parse_single_example(proto, example)
    image = tf.decode_raw(parsed_example['image_raw'], tf.uint8)
    image = tf.reshape(image, [HEIGHT, WIDTH, DEPTH])
    image = preprocess_input(tf.cast(image, tf.float32))
    return image, parsed_example['label']

def get_data(filepath, schuffle_size=32, batch_size=8, prefetch=1, repeat=None, num_parallel_calls=1):
    dataset = tf.data.TFRecordDataset(filepath)
    if schuffle_size != 0:
        dataset = dataset.shuffle(schuffle_size)
    dataset = dataset.repeat(repeat)
    dataset = dataset.map(parse_function, num_parallel_calls=num_parallel_calls)
    dataset = dataset.batch(batch_size)
    dataset = dataset.prefetch(prefetch)
    iterator = dataset.make_one_shot_iterator()
    return iterator

def build_model(number_of_neurons_in_dense_layer, dropout, learning_rate):
    base_model = Xception(weights='imagenet', include_top=False, pooling='avg', input_shape=(HEIGHT, WIDTH, 3))
    for layer in base_model.layers:
        layer.trainable = True
    x = base_model.output
    x = Dropout(dropout)(x)
    x = Dense(number_of_neurons_in_dense_layer, activation='relu')(x)
    x = Dropout(dropout)(x)
    logits = Dense(NUMBER_OF_CLASSES, activation='softmax')(x)
    model = Model(inputs=base_model.input, outputs=logits)
    model.compile(optimizer=Adam(lr=learning_rate), loss='sparse_categorical_crossentropy', metrics=['categorical_accuracy'])
    return model

global NUMBER_OF_CLASSES, HEIGHT, WIDTH, DEPTH
NUMBER_OF_CLASSES = 2
...
CLASS_WEIGHTS = {
        0: 1,
        1: 7
       }
model = build_model(64, 0.4, 0.001)
train = get_data(..., 8, 2, num_parallel_calls=8)
val = get_data(...., 0, 4, num_parallel_calls=8)
model.fit(train, validation_data=val, epochs=3,steps_per_epoch=8//2,
           validation_steps=8//4, shuffle=False,
           class_weight=CLASS_WEIGHTS)

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/numpy/core/fromnumeric.py", line 51, in _wrapfunc
    return getattr(obj, method)(*args, **kwds)
AttributeError: 'Tensor' object has no attribute 'reshape'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "./model.py", line 137, in main
    model.fit(train, validation_data=val, epochs=3, steps_per_epoch=8 // 2, validation_steps=8 // 4, shuffle=False, class_weight=CLASS_WEIGHTS)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/training.py", line 776, in fit
    shuffle=shuffle)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/training.py", line 2432, in _standardize_user_data
    feed_sample_weight_modes)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/training.py", line 2431, in <listcomp>
    for (ref, sw, cw, mode) in zip(y, sample_weights, class_weights,
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/training_utils.py", line 758, in standardize_weights
    y_classes = np.reshape(y, y.shape[0])
  File "/usr/local/lib/python3.6/dist-packages/numpy/core/fromnumeric.py", line 279, in reshape
    return _wrapfunc(a, 'reshape', newshape, order=order)
  File "/usr/local/lib/python3.6/dist-packages/numpy/core/fromnumeric.py", line 61, in _wrapfunc
    return _wrapit(obj, method, *args, **kwds)
  File "/usr/local/lib/python3.6/dist-packages/numpy/core/fromnumeric.py", line 41, in _wrapit
    result = getattr(asarray(obj), method)(*args, **kwds)
TypeError: __index__ returned non-int (type NoneType)

@jvishnuvardhan any updates?

@imarkiew This looks like due to the fact that code in training_utils.py of tf.keras is still wrongly assuming input data to be numpy arrays. Try enabling eager execution. It goes away for me if I do that but then it also uncovers other bugs.

@dd1923 Honestly, I changed my approach some time ago and completely switched to Kerasa (so I don't use TFRecords and Datset API). But I also think that there is something broken underneath between the Tensors and numpy arrays.

@imarkiew @dd1923 There were lots of improvements over the last two releases. Can you please check with TF2.0 or tf-nightly and let us know whether the issue persists in the newer TF versions. Please provide a standalone code reproduce the issue. Thanks!

@imarkiew Is this still an issue? If it not, please close the issue. Thanks!

@jvishnuvardhan I tested this for 2.0.0b1 version and it still seems that the problem exists (but with another error). Unfortunately, due to some problems I wasn't able to test tf-nightly.
Traceback (most recent call last):
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 2326, in get_attr
    c_api.TF_OperationGetAttrValueProto(self._c_op, name, buf)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Operation 'dropout_1/cond' has no attr named '_XlaCompile'.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/ops/gradients_util.py", line 344, in _MaybeCompile
    xla_compile = op.get_attr("_XlaCompile")
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 2330, in get_attr
    raise ValueError(str(e))
ValueError: Operation 'dropout_1/cond' has no attr named '_XlaCompile'.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/framework/importer.py", line 427, in import_graph_def
    graph._c_graph, serialized, options)  # pylint: disable=protected-access
tensorflow.python.framework.errors_impl.InvalidArgumentError: Node 'dropout_shape_dense_identity' expects to be colocated with unknown node 'dropout_1/cond'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "main.py", line 52, in <module>
    model.fit(train, validation_data=val, epochs=3, steps_per_epoch=8//2, validation_steps=8//4, class_weight=CLASS_WEIGHTS, shuffle=False)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py", line 643, in fit
    use_multiprocessing=use_multiprocessing)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/keras/engine/training_generator.py", line 694, in fit
    steps_name='steps_per_epoch')
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/keras/engine/training_generator.py", line 264, in model_iteration
    batch_outs = batch_function(*batch_data)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py", line 917, in train_on_batch
    self._make_train_function()
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py", line 1974, in _make_train_function
    params=self._collected_trainable_weights, loss=self.total_loss)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/keras/optimizers.py", line 476, in get_updates
    grads = self.get_gradients(loss, params)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/keras/optimizers.py", line 91, in get_gradients
    grads = K.gradients(loss, params)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/keras/backend.py", line 3568, in gradients
    loss, variables, colocate_gradients_with_ops=True)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/ops/gradients_impl.py", line 158, in gradients
    unconnected_gradients)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/ops/gradients_util.py", line 677, in _GradientsHelper
    lambda: grad_fn(op, *out_grads))
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/ops/gradients_util.py", line 349, in _MaybeCompile
    return grad_fn()  # Exit early
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/ops/gradients_util.py", line 677, in <lambda>
    lambda: grad_fn(op, *out_grads))
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/ops/cond_v2.py", line 100, in _IfGrad
    true_graph, false_graph = _get_func_graphs(if_op)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/ops/cond_v2.py", line 268, in _get_func_graphs
    return (_get_func_graph_for_branch(op.get_attr("then_branch")),
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/ops/cond_v2.py", line 258, in _get_func_graph_for_branch
    fdef, input_shapes)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/framework/function_def_to_graph.py", line 62, in function_def_to_graph
    importer.import_graph_def(graph_def, name="")
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/util/deprecation.py", line 507, in new_func
    return func(*args, **kwargs)
  File "/home/user/PycharmProjects/TfRecordsTestV2/venv/lib/python3.6/site-packages/tensorflow/python/framework/importer.py", line 431, in import_graph_def
    raise ValueError(str(e))
ValueError: Node 'dropout_shape_dense_identity' expects to be colocated with unknown node 'dropout_1/cond'

@imarkiew Can you please try TF2.0 or create a simple standalone code to reproduce the issue. Thanks!

@jvishnuvardhan It seems that the change form 2.0.0b1 to 2.0.0 has helped. I don't see this issue anymore.

@imarkiew Thanks for the confirmation. We will close this issue as it was resolved. Thanks!


Restore Wii-like aiming (to use it with gyro)

Describe the feature you'd like to see added
Proof of concept
I think this could be a great adition.
If you guys are able to restore it, all the user would have to do is map the gyro to work as a mouse (like I did on dolphin), though I assume you would have to find a way to allow both Xinput and K&M at the same time.
My idea is to make it work in a "hybrid" mode, like have both the normal controls and the Wii cursor working at the same time, so you can still control the camera normally with the right stick when you're not aiming.

I'll try to record some gameplay, so far it seems to have a lot of potential.

[https://youtu.be/ECUi3vfCYPY?si=PNOIacCIXz37yn0E](Here's some bad gameplay)

Here's some gameplay


feat: introduce customizers for configuring host env variables

Introduces set of ContainerCustomizerFns to load host env variables to dagger containers
Depends on #59

:slightly_smiling_face:  I know I should have written some test for these functions :/

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 8.125%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
13

Relevant Lines:
160

💛  - Coveralls


fire

add one or more types of fire atom. fire rises and also turns flammable things, such as plant and oil, into more fire

resolved by 166f9b5b0f007e09e6f61f0e7cca707fd1a72771


Add Data Acceleration and Data Freshness Policy support

Added support for Data Acceleration and Data Freshness Policy REST API.
Mainly adding functionalities to workbook.update & workbook.get_by_id

@ltiffanydev I closed & reopened this PR to force the CLA bot to run again and it looks happy now.
One change you'll need to make on this PR is target it for the development branch instead of master.

@ltiffanydev reminder when you merge this to use the "squash and merge" option to bring all these commits into one. I think that might be the default anyways.
 It's happening to all PRs on TSC at the moment.


,bbbhnhm,ghfdgsfaafFFAWFWFSTSGHLASDFGHL'[[

Bug reporting
A good bug report has some very specific qualities, so please read over our short document on reporting bugs before submitting a bug report.
To ask a question, go ahead and ignore this.

@keyankay ?

This was by mistake. Please ignore. My apologies


5.2.2 Table 2 FIA_PMG_EXT

Add "See FIA_PMG_EXT.1" to the Notes cell for Management Function 12, Ability to change default authorization factors

Added to client.

Added to client.


godot will crash if it is built with gcc6

see issue #31584
Motivation for this change
I was experiencing a crash
Things done
I tested this on the 17.09 branch with nix-shell -i nixpkgs=~/nixpkgs -p godot --pure
With that godot binary I no longer experienced the  crash outlined in issue #31584 .
It should be noted that without the --pure option godot still seemed to link against gcc-6 and crashed...

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

merged in https://github.com/NixOS/nixpkgs/commit/61b20e24156e5653df2b5e8d316331cf89cd4ee4


Cleanup SPN in deployment

There is the following SPN's:
$mountSpnName                 = "AtcMountSpn"
$dbDeploySpnName              = "AtcDbSpn"
$cicdSpnName                  = "AtcGithubPipe"

Do we need them all?

See #200. All SPN's now have a clear purpose.


Enable access of the Contants object in the shaka-player npm package

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
2.5.5
Can you reproduce the issue with our latest release version?
Yes, because 2.5.5 is the latest.
Can you reproduce the issue with the latest code from master?
Yes
Are you using the demo app or your own custom app?
No
If custom app, can you reproduce the issue using our demo app?
N/A
What browser and OS are you using?
Google Chrome on MacOS Siera
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
N/A
What are the manifest and license server URIs?
N/A

What did you do?

In my react component file...
import shaka from '../../../node_modules/shaka-player/dist/shaka-player.ui';
shaka.ui.Constants.SEEK_BAR_BASE_COLOR = 'rgb(0,0,255, 0.3)';

What did you expect to happen?
I expected the seek bar to have a light blue color based on the rgba value provided
What actually happened?

Application fails and I receive this error in the console
TypeError: Cannot set property 'SEEK_BAR_BASE_COLOR' of undefined

This was never our intent.  Changing a constant makes it not constant.  :-)
We had always intended for style to be overridden via CSS.  But the seek bar colors are constructed in code using these constants, so it's not necessarily possible to change the colors and keep the same gradient effect purely in CSS.
I see three options:

We could accept PR #2204 to allow you to change these constants.  It's the least work, but also a little strange/hacky.
We could reimplement the seek & volume bars as nested, colored divs instead of a range input element.  That would allow each div to have a separate class whose color you could change purely in CSS, but the element would no longer be semantically correct.  This would cause issues with screen readers and require us to do additional work to make the elements accessible.
We could offer a configuration for the volume & seek bar colors.  This would be more work than option 1, but would be in keeping with the design of Shaka Player.  You would change the colors via ui.configure() instead of by overwriting a constant.

I'm inclined toward option 3, which seems like the best balance to me.  While you're waiting for us to implement option 3, you could certainly continue using option 1 in your fork.  Would that work for you?

@joeyparrish thank you very much for getting back on this issue :). Option 3 seems like the best solution and I could use my fork in the meantime. I think it would also be nice if it's stated somewhere in the docs that with the npm package it's not "possible" to override those color constants with the current version 2.5.5.


Submit a complaint on /complaint should go to /complaint/getting-started

Current behavior

When on /complaint "Submit a Complaint" link is prominent on the upper-right, but the "Start a complaint" button may be scrolled out of view. This could lead a user clicking "Submit a Complaint" in the upper-right and expect to be taken to /complaint/getting-started (as the button further down on the page does). However, that link goes to the same page the user is on, so it appears the link is not working.

Expected behavior

Perhaps change "Submit a Complaint" link to point to /complaint/getting-started OR remove the link on /complaint OR disable the "Submit a Complaint" link in the upper-right when on /complaint.

Steps to replicate behavior

Visit https://www.consumerfinance.gov/complaint/ and click "Submit a Complaint" in the upper-right.

Screenshots

OR have it go to /complaint/getting-started IF you're already on /complaint?

And then there's the question of what to do with that link when on https://www.consumerfinance.gov/complaint/getting-started/ :)

@schaferjh How do you think this should work?

This is a great detailed observation but we have not recieved any indication that it's an issue or blocker and I would prefer not to create a unique behavior here given that the link in question is part of the global header. I think this issue can be closed.


Hero flickers on transition (Android & IOS)

So on previous versions of flutter I didn't have this issue, but after upgrading to the latest version of flutter it makes heros flicker while they transition.

Steps to Reproduce

https://github.com/littlemarc2011/FlutterTodo
Use the latest version of flutter (master branch)
Build to either IOS or android, happens on both.

Logs
There are no logs/no errors
Flutter Analyze
Analyzing todo...

   info • The top level variable '_kUnspecifiedArgument' isn't used • lib/CustomAppBar.dart:7:14 • unused_element
   info • This method overrides a method annotated as @mustCallSuper in 'State', but does not invoke the overridden method • lib/main.dart:77:8 • must_call_super
   info • The operator x ~/ y is more efficient than (x / y).toInt() • lib/main.dart:85:18 • division_optimization
   info • The value of the local variable '_ratioW' isn't used • lib/main.dart:123:18 • unused_local_variable
   info • The value of the local variable '_ratioH' isn't used • lib/main.dart:126:18 • unused_local_variable
warning • Element 'TextStyle' from SDK library 'ui.dart' is implicitly hidden by 'text_style.dart' • lib/main.dart:184:44 • conflicting_dart_import
   info • Avoid empty statements • lib/main.dart:511:5 • empty_statements
   info • Unused import: 'dart:collection' • lib/todo.dart:5:8 • unused_import

8 issues found. (ran in 1.3s)

Flutter Doctor
[✓] Flutter (Channel master, v0.6.1-pre.44, on Mac OS X 10.13.6 17G65, locale en-US)
    • Flutter version 0.6.1-pre.44 at /Users/bigmarco/Repositories/flutter
    • Framework revision 980c8a737b (3 hours ago), 2018-08-20 14:33:27 -0700
    • Engine revision c23690f0d7
    • Dart version 2.1.0-dev.0.0.flutter-be6309690f

[!] Android toolchain - develop for Android devices (Android SDK 28.0.2)
    • Android SDK at /Users/bigmarco/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.2
    • Java binary at: /Library/Java/JavaVirtualMachines/jdk1.8.0_171.jdk/Contents/Home/bin/java
    • Java version Java(TM) SE Runtime Environment (build 1.8.0_171-b11)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.4.1, Build version 9F2000
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✗] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[✓] IntelliJ IDEA Ultimate Edition (version 2018.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 27.1.3
    • Dart plugin version 182.3911.37

[!] VS Code (version 1.26.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected devices (2 available)
    • SM G950U • 9887bc48414134574b                   • android-arm64 • Android 8.0.0 (API 26)
    • iPhone X • 0A1678FE-CBF9-4948-8D87-E5AE8E31BF4C • ios           • iOS 11.4 (simulator)

! Doctor found issues in 3 categories.

Did you check in profile or release mode?
Performance behavior in debug mode is not representative.

@zoechi I just ran it on my s8 in release mode and in profile mode and it is still flickering. (flutter run --release and flutter run --profile).

All the examples and GIFs are directed towards the text "Custom"
Previously I had the text wrapped in a material widget, and this is what happens if I do it without:

Code before:
new Hero(
  tag: todoObject.uuid +"_title",
  child: new Material(
    color: Colors.transparent,
    child: new Text(
      todoObject.title,
      style: new TextStyle(
        fontSize: 30.0
      ),
    ),
  ),
)

Code after: (What is shown in first GIF)
new Hero(
  tag: widget.todoObject.uuid + "_title",
  child: new Text(
    widget.todoObject.title,
    style: new TextStyle(fontSize: 30.0),
  ),
)

This would tell us that it is not related to material and more so text.
With a little more digging around, I set the size of the material (I reverted back to the before code) with a SizedBox and it no longer flickers as can be seen here:

The code is: (140.0 and 30.0 are just some random numbers that happened to fit in this case)
new Hero(
  tag: todoObject.uuid + "_title",
  child: new SizedBox(
    width: 140.0,
    height: 30.0,
    child: new Material(
      color: Colors.transparent,
      child: new Text(
        todoObject.title,
        style: new TextStyle(
          fontSize: 30.0
        ),
      ),
    ),
  ),
)

I assume something with how Hero's handle the size of text has changed (since a month ago this was working fine). It would be nice if I wouldn't have to specify the width and height of the text but this should help narrow down the issue.

Hello,
Any luck with fixing the issue? It happens to me as well :/

Facing the same issue. Please help!

@kundan9595 add a Material parent between your hero and your text, should fix it.

Linking #30647 and #12463 to this issue.

Any update on this issue?

Any update on this issue?

I still have this issue even though I used Material between Text and Hero.

Dup of #36220. This issue is older but since I assigned #36220 to myself I'm going to close this instead. This should be fixed once #36220 is closed.


Memory leak with email validation in zod forms

Current Behavior
The browser tab hangs and crashes
Expected Behavior
Email validation works
Steps To Reproduce

basically start writing in the email field with characters and browser tab will grind to a halt (enter at least 50 characters)

Link to Reproduction / Stackblitz
https://stackblitz.com/edit/github-zxmrcw?file=src%2Froutes%2F%2Bpage.svelte,src%2Froutes%2F%2Blayout.svelte
More Information
I did some testing and this only happens when validating the email constraint -> and I have no running hypothesis why is that the case.

Wow, this is definitely a weird one, especially considering it only occurs when validating the email constraint 🤔.
Does this happen when using Superforms by itself, or only when using it with Formsnap? If it was happening for all the validation cases then it would make more sense, but the fact that it's only happening for this one is peculiar.

There's this issue here that might be related: https://github.com/colinhacks/zod/issues/2580

Going to close this as it isn't an issue with Formsnap itself, but rather an issue with Zod and complex email addresses.
As a solution in the meantime, you can stop using .email() and instead create your own custom one which someone recommended in another Zod issue regarding this:
const schema = z.object({
	email: z.regex(/^[a-zA-Z0-9.!#$%&’*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\.[a-zA-Z0-9-]+)*$/)
})

Thank you so much! So weird that this is yet to be actionable by zod. 😄


sbt should validate maven model before publishing

I downloaded all Scala 2.11 poms from maven central (~50 000 poms). Almost 10% fails to load because they are invalid poms.
For example: you cannot duplicate license tags, but sbt permits that https://github.com/magneticio/vamp/issues/612
Maven model builder can validate a pom. Usage example
steps
create an invalid pom (for example https://github.com/magneticio/vamp/issues/612)
publish
problem
sbt does not validate the model
expectation
I expect sbt to validate the pom before publishing

or get rid of pomExtra and provide a dsl for it

pomExtras is a pretty blunt tool, but I'm not sure if it should be validate by default. Perhaps this could be a community ticket for an sbt plugin to validate it.

well since publish is part of sbt I think it should be in the core of sbt

test is part of sbt, but sbt doesn't come built in with scalacheck.. It's also abiut adding another/more dependencies..

what if we copy paste this model: https://github.com/MasseGuillaume/super-winner/blob/master/src/main/scala/ch.epfl.scaladex/maven/Pom.scala#L14 in sbt

Also an sbt plugin could eventually be folded into sbt.
Or we could add it as a feature, off by default, or on by default with a flag to disable it..

An easier, partial solution, at least for the magneticio case, is that instead of building the pom from sbt settings first, and then appending pomExtra, you start with pomExtra and then you append nodes in the XML. So when you generate the XML for the licenses key, for instance, if the licenses node exists your append in it.

Would a Java dependency to Maven Model Builder be acceptable, or would an implementation need to be Scala in order to be acceptable?
Does anyone know if XSD validation would suffice?


Updates SCMB examples replacing legacy code

Description
Replaces non-functional legacy code in scmb examples
Issues Resolved
Fixes #332
Check List

[x] New functionality includes testing.

[x] All tests pass for Python 2.7+ & 3.4+($ tox).

[x] New functionality has been documented in the README if applicable.

[x] New functionality has been thoroughly documented in the examples (please include helpful comments).
[x] New endpoints supported are updated in the endpoints-support.md file.

[ ] Changes are documented in the CHANGELOG.

@fgbulsoni  Yes and No.
Yes we already do have a resource for RabbitMQ certificates.
No, we don't want to add any other action to RabbitMQ resource. First because the methods are simply handling certificates and there are other methods creating connections using amqp. Also, the scmb examples README makes it clear that it is an example on how to connect the SDK to an external tool.

Weirdly, I'm also getting these other issues on the ov_to_sm example:
Warning	Flake8	F821 — undefined name 'http'	43:20
Warning	Flake8	F821 — undefined name 'http'	54:16
Warning	Flake8	F821 — undefined name 'time'	57:13
Warning	Flake8	F821 — undefined name 'common'	115:20

Is it perhaps my environment? I see travis's flake8 did not hit those. But at the same time I don't see where those are being declared.

AH, travis is ignoring examples/scmb/, I'd suggest fixing those up and checking line 14 on tox.ini:
exclude = tests.py, hpOneView/__init__.py, examples/scmb/, examples/scripts
I believe examples/scripts has already been dropped, examples/scmb/ should be passing with this PR and I haven't seen tests.py, so those three could likely be removed from the exclude

Fixes #332


Can I use ipv6 in network.trr.bootstrapAddress?

Thanks!

Why not just try it? I think it works. If not, ask the good people at Mozilla.


how to run unittest ?

Describe the issue:
Environment:

NNI version: 2.9
Training service (local|remote|pai|aml|etc): local
Client OS:
Server OS (for remote mode only):
Python version:  3.7
PyTorch/TensorFlow version:  TF2.9
Is conda/virtualenv/venv used?:  yes
Is running in Docker?:  yes

Configuration:

Experiment config (remember to remove secrets!):
Search space:

Log message:

nnimanager.log:
dispatcher.log:
nnictl stdout and stderr:

I  have builded the nni  whl package  from source code successfully,  and i find the test folder in nni source  code, so how to run  the unnitest  for nni ? can you give me the command or relevant  path. Thanks.
How to reproduce it?:
I  have builded the nni  whl package  from source code successfully,  and i find the test folder in nni source  code, so how to run  the unnitest  for nni ? can you give me the command or relevant  path. Thanks.

can you answer it ? @liuzhe-lz

You can check the commands used by our pipeline https://github.com/microsoft/nni/blob/master/pipelines/fast-test.yml.
The python UT is run by cd test && python -m pytest ut.
It may requires extra dependencies defined in develop.txt and recommended.txt


Add note about embedding of view-chains and the interaction with db-extra-search-path

Following up on https://github.com/PostgREST/postgrest/pull/1625#issuecomment-728357780.
What do you think @steve-chavez?

What about this? Better that way?

I did not bother to add something to releases/upcoming.rst, yet, given that most of the changelog is still missing. Tell me again, how were we going to proceed with this? You had something better in mind, didn't you?

@wolfgangwalther Main idea was to move the CHANGELOG to docs: https://github.com/PostgREST/postgrest-docs/issues/340. But still not sure about the implications..


[Divider] Becomes thicker/thinner depending on browser zoom level

While developing using the Divider component, I noticed in Firefox that zooming in and out would cause some Dividers to become invisble, and some to become thicker. After some investigation, I found that this could be caused by subpixel calculation.
Related google search results:

https://stackoverflow.com/questions/12547580/borders-disappear-in-chrome-when-i-zoom-in
https://stackoverflow.com/questions/17730975/border-disappears-on-zoom-out

I fixed this locally by replacing the 'height' with a border with the "thin" thickness, and have created a PR that does the same

[X] The issue is present in the latest release.
[X] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Zooming in and out changes the height of the Divider, sometimes resulting in double-thickness, sometimes resulting in being invisible
Expected Behavior 🤔
Divider component should render consistently at all zooms
Steps to Reproduce 🕹
Add a multitude of Dividers to a page, then zoom in and out
Codesandbox
Your Environment 🌎
Latest clone of Material UI Project

I have submitted a PR that changes the Dividers UX to use a Border with a thin thickness which is consistent at all zoom levels


[CARBONDATA-3829] Support pagination in SDK reader

Why is this PR needed?
Please refer the design document attached in the JIRA.
Carbondata SDK now currently doesn't support pagination.
What changes were proposed in this PR?
a) Support pagination from java SDK with LRU cache support.
b) Support pagination in python SDK by calling JAVA SDK.
Does this PR introduce any user interface change?

No [Added new interfaces]

Is any new testcase added?

Yes

@jackylk , @xubo245 : please check

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1329/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3049/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1339/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3059/

@xubo245 @VenuReddy2103 @akashrn5

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1398/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3122/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3123/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1399/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3128/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1404/

retest this please

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3130/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1406/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1417/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3141/

retest this please

please merge @akashrn5 , @xubo245 @jackylk

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3161/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1437/

@akashrn5 , @xubo245 , @jack86596 : reminder !

retest this please

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1483/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3210/

retest this please

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3393/

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1653/

retest this please

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1657/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3399/

need one more LGTM

LGTM

Thanks for your contribution!


add optional vendored-openssl feature

context:
https://github.com/mozilla/dump_syms/blob/bf7f6bd855eb6ecb233b52bfa7b2c975b7026540/Cargo.toml#L40-L45
https://searchfox.org/mozilla-central/rev/6deb8b6af57a8b5b6b1bcb143ea498e566475d8d/taskcluster/scripts/misc/build-dump-syms.sh#52

@gabrielesvelto do you know the historical context for why our linux builds for dump_syms do this? Currently just assuming it's necessary. The build on taskcluster does fail for rust-minidump if I don't do this:
[task 2021-11-16T23:09:47.421Z] The following warnings were emitted during compilation:
[task 2021-11-16T23:09:47.422Z]
[task 2021-11-16T23:09:47.422Z] warning: build/expando.c:2:10: fatal error: 'openssl/opensslconf.h' file not found
[task 2021-11-16T23:09:47.422Z] warning: #include <openssl/opensslconf.h>
[task 2021-11-16T23:09:47.422Z] warning:          ^~~~~~~~~~~~~~~~~~~~~~~
[task 2021-11-16T23:09:47.422Z] warning: 1 error generated.
[task 2021-11-16T23:09:47.422Z]
[task 2021-11-16T23:09:47.422Z] error: failed to run custom build command for `openssl-sys v0.9.63`
[task 2021-11-16T23:09:47.422Z]

@gabrielesvelto do you know the historical context for why our linux builds for dump_syms do this?

It's an issue that predates dump_syms, I think it's best explained in this comment and the current version of things first happened with sccache IIRC (see this).

Ok so tl;dr the docker images we build inside of use a weird symbol/path for OpenSSL which would cause portability/build issues, and the simplest solution we found was to statically link it and not worry about it. Seems ~fine.

I agree, if this gets you unstuck then it's fine. The only thing that should be using openssl in this crate is the symbol-fetching-via-HTTPS code in breakpad-symbols, and that's just transitively by way of native-tls.


ecl: make sure boehmgc is available to gcc/linker

Motivation for this change
Since #131719, maxima-ecl is built with strictDeps = true. When ecl is built with useBoehmgc = true, libgc's headers, which were previously found by virtue of boehmgc being in propagatedBuildInputs, no longer get appended to the include path. A similar thing happens for linker flags.
This PR fixes the Sage build.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 132269 run on x86_64-linux 1

5 packages built:

ecl
eql
maxima-ecl
sage (sagemath)
sageWithDoc

Result of nixpkgs-review pr 132269 at 75b4fccf run on aarch64-linux 1

3 packages built successfully:

ecl
eql
maxima-ecl


Fixed issue #486

🛠 Fixes Issue
Closes #486
👨💻 Changes

Fix shadow fades away during hovering (due to having same color)

Before changes

After changes

Fix This section has not any shadow effect like other sections. It looks like a static faded outline.

Before changes

After changes

Fix Shadow is too much spread out and with hover effect make it worse.

Before changes

After changes

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Great Work! @front-runner-sd


Domain statistics response incorrect

The "Response" section shows "If domain does not exist - 404 Not Found"; however, it appears to return 200 OK with an prevalence of 0 and first and last seen null. This was tested by using registered domains not in any alerts on the tenant, unregistered domains that could be legitimate if registered, and domains that had TLDs that are invalid. All returned 200 OK.

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

ID: 2eae16f9-5929-ae70-8356-4bcea2380be0
Version Independent ID: a8c5ee2a-6c0c-0e1d-18e1-471fa7b37085
Content: Get domain statistics API
Content Source: microsoft-365/security/defender-endpoint/get-domain-statistics.md
Product: m365-security
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@MastaDisasta Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/6023
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @MastaDisasta Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


feat: simplifies data fetching model

The data fetching model as it currently stands was the first thing I implemented and borrowed heavily from the DD model. This part of the app has disgusted me, but I actually like the solution now.
This replaces the fetchers with a react-query implementation which is simple but powerful. https://react-query.tanstack.com/overview
Future work would be to eliminate the global cache entirely by refactoring QueryChart into a functional component which can use hooks.
Still having trouble changing git branch without a page reload. Even after invalidating cache
Sorry for the big PR but many files were touched after I consolidated the Extension state
This build of the diagram can be found here: https://demoexpo.looker.com/extensions/lookml_model_diagram::dev_ext/models/thelook_adwords/explores/events

I'm really displeased with this size of this PR. Seeking approval for merge conditional on these follow up PRs:

reload data not page, remove location.reload() usage
remove sqlTableName overloading


Update README.md

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (e09c478) 27.48% compared to head (7b508e9) 27.48%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   27.48%   27.48%
=======================================
  Files           4        4
  Lines        1721     1721
=======================================
  Hits          473      473
  Misses       1248     1248

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


bind text to container on ipad - accessing context menu is difficult

probably would be better to add an action button to the element properties panel, similar to add hyperlink.
https://user-images.githubusercontent.com/14358394/159355871-7ac101a0-4985-4ee3-86fc-34386d50775f.MOV

I'm not sure. The usual principle of "minimize buttons in the UI" applies, even though this is a contextual button. We're also planning to do an editor redesign, and it's unclear how/where we'll display contextual buttons/actions — it might be better to wait for the redesign so that people don't get used to a button that would be cut in the process.

@zsviczian now that we allow binding text to containers via clicking anywhere on filled containers and on stroke otherwise - this shouldn't be a problem anymore.

@zsviczian can you confirm on this ?


Index Btree Benchmark

test btree types:
1 FixedLenBtree
2 VarKeySizeBtree
3 VarValueSizeBtree
4 VarObjSizeBtree
5 PrefixIntervalBtree
we can run benchmark from command line with different parameters to simulate different case , for example:
./bin/index_btree_benchmark --num_threads=5 --num_fibers=5 --operation_list=put:100

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (02b16cc) 70.07% compared to head (8528625) 68.46%.
Report is 3 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
- Coverage   70.07%   68.46%   -1.61%
==========================================
  Files          97       96       -1
  Lines        7932     8017      +85
  Branches     1018     1053      +35
==========================================
- Hits         5558     5489      -69
- Misses       1881     1992     +111
- Partials      493      536      +43

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

test failure is caused by iomgr, not the test case itself


Add a question for ALPeopleList[i].attorney

It should prompt for bar number and office name, and it should allow a user to load information from a stored profile when we implement that feature eventually.

It may make sense to have two separate screens-one for name and organization name, and one for the other details.

Actually should consider making this plural - .attorneys

I think attorneys is the right attribute name to make.


About split_pcl2pcl.txt

Thanks for your good work，Can you provide the file "split_pcl2pcl.txt" in dataloader? Thanks a lot!

Thank you for your attention. This file is now in "dataset" folder. https://github.com/a4152684/KT-Net/blob/main/dataset/split_pcl2pcl.txt

Thanks！It is of great helpful!


Agent uid 1000 is problematic

Is it possible to change the agent's user ID from 1000 to something less likely to collide, e.g. 1100 ? It's useful for the agent ID to match a controlled user on the docker host, but user ID 1000 is often taken already in a given machine image.

For sure, I think we could change this user ID. I don't see any problems with that right now.

It's useful for the agent ID to match a controlled user on the docker host

Could you say more about why it's useful? I guess perhaps you're thinking about mounting files from the Docker host inside the container.

Yes it's a bit easier to reason about file ownership when they're mounted in the container. Not essential though.


Loading the unicode-math package breaks mhchem math

Description
Loading unicode-math breaks the math-mode typesetting of chemical formulae provided by the mhchem package.  Link to the stackexchange discussion (provided below) gives a more detailed description.
Check/indicate

[x] Relevant for XeTeX
[x] Relevant for LuaTeX
[x] Issue tracker has been searched for similar issues? Yes
[X] Links to <tex.stackexchange.com> discussion if appropriate. Please see here for StackExchange  and here for Libertinus github repo issue

Minimal example demonstrating the issue
%! TEX program = lualatex
    \documentclass[varwidth=true, border=10pt, convert={size=640x}]{standalone}
    \usepackage[version=4,arrows=pgf]{mhchem}
    \usepackage{unicode-math}
    \setmathfont{Libertinus Math} % doesn't matter. All unicode math font behaves the same way
    %\setmathfont{texgyrepagella-math.otf} % doesn't matter. All unicode math font behaves the same way

    \begin{document}

    \begin{center}
    \ce{A_x <=> B + y C} \\
    \end{center}

    \begin{align}
        \ce{A_x &<=> B + y C} \\
        \ce{X + Y &-> Z}
    \end{align}

    \begin{align}
        \ce{A_$x$ &<=> B + $y$ C} \\
        \ce{X + Y &-> Z}
    \end{align}

    \end{document}

Further details

The problem is that in math mhchem surrounds everything with \mathrm. And this change the  \Umathcode.
\documentclass{article}
\usepackage[version=4,arrows=pgf]{mhchem}
\usepackage{unicode-math}

\begin{document}

$\ce{A_{\symit{x}}}$

\makeatletter
\def\mhchem@hook@beforeItalicMath{\symit}

$ \ce{A_x} \ce{A_{xxxx}} $

 %not a solution but only to demonstrate that removing the \mathrm avoids the problem:
\def\mhchem@option@mathFont{}%{\mathrm}

${\ce{A_x}}$

\end{document}

Imho it would be very useful to have a official way command to "break out " of a \mathrm as this is used internally by packages (see also https://github.com/wspr/unicode-math/issues/438).

So, is this a bug with mhchem and not a unicode-math bug?

@krishnakumarg1984  What is your reasoning why $\mathrm{\text{\ensuremath{x}}}$ should render differently when unicode-math is loaded?

@mhchem, a variant of @u-fischer 's example removes amsmath showing it's really a unicode-math issue.
\documentclass{article}

\ifx\Uchar\undefined\else
\usepackage{unicode-math}
\fi
\begin{document}

$\mathrm{\mbox{$abc$}}$

\end{document}

this gives roman in unicode-math and italic with pdftex

@davidcarlisle But I wonder if unicode-math can really avoid it. I mean it would have to reset the Umathchar codes at every math to make sure to nullify some \mathXX command.

@u-fischer  not sure it seems to me that setting \fam should be enough.

@davidcarlisle I don't think that \fam1 as in the tex.sx answer works. It actually uses the cmmi-fonts:
\documentclass{article}

\usepackage{unicode-math}

\setmathfont{TeX Gyre Termes Math}
\begin{document}
$A_x$

$\mathrm{\mbox{\ensuremath{\fam1 A_x}}}$

$\mathrm{\mbox{\ensuremath{\csname__um_switchto_normal:\endcsname A_x}}}$

\end{document}

@u-fischer  I guess you are right, perhaps
$\mathrm{\mbox{\ensuremath{\mathnormal{A_x}}}}$
or higher level way to get unicode-math to put things back in nested contexts

There is so much setting and reseting of mathcodes that I get a bit lost sometimes :)
The reason using \fam doesn’t work is that the mathcodes for ascii need to be set to plane 1 for regular maths to work. It looks here, though, like I should perhaps patch \mbox to call \mathnormal if it’s already inside maths mode?

@wspr  patching \mbox wouldn't work in most cases eg ams\text (which is actually what gets used here) use \hbox in \mathchoice, you could patch \everymath but that seems a bit costly as well.  as nested math in a \mathxxx is probably rare, an alternative plan would be to document that unlike the classic setup the setting is inherited by a nested math construct (which to be honest is I would guess what most people expect from \mathrm{\hbox{$x$}}   and then @mhchem  could use \ensuremath{\mathnormal{... to normalize the nested math constructs back to math italic.
perhaps..

which means that my \fam=-1 answer at stackexchange is wrong so I'll delete (mentioning here for those who can't see deleted tex.sx posts)

Can the Unicode-math team officially announce using chemformula as a viable workaround, until  @mhchem and unicode-math team are able to figure out a solution? I am a bit surprised this hasn't been reported so far. Aren't there any chemistry folks using Unicode fonts?

@krishnakumarg1984 I'm a chemist but (1) use chemformula and (2) don't use unicode-math (the latter isn't really needed for most synthetic chemistry)

I wish I could stick with  traditional fonts, but my PhD supervisor is a bit more idealist and requires me to use Unicode fonts for text and math throughout

@krishnakumarg1984  I don't understand your problem. The work-around I suggested at the tex.sx question should work fine.
\makeatletter
\def\mhchem@hook@beforeItalicMath{\symit}

@u-fischer  I am sorry if I misunderstood, but I thought that the solution you proposed worked only in certain cases, i.e. your answer said "But it doesn't work if the argument doesn't consist of a single char".
Secondly, such kind of low-level kernel-hacks looking at document stack-trace defeats the purpose of the mhchem package. I did a lot of readings on these packages. The developer of the chemformula package once wrote a tugboat article, in which Martin Hensel, the author of mhchem package summarised these two packages. I am paraphrasing them as "mhchem```` is more hands-off.  There is very little customisability, but the focus is on ease-of-use. chemformula``` on the other hand gives the user a lot more control and customisability. "
For an electrical engineer like me, looking to describe a few chemical equations in my thesis and move on, mhchem does not work, when used with modern font technologies.  This is a let-down. Sure, you could hack up a workaround. I am not even questioning whether your code works (since I am well aware of your TeX-related prowess), but I am certainly questioning whether someone else like me will immediately find this bug report or that Stack-exchange post.
On the other hand, chemformula does work as advertised.  As a non-expert, you read the package documentation, and try the examples and voila, things work.  Until things work like what it it says on the mhchem package documentation without such low-level hacks, it might help those using unicode-math, if the unicode-math team have a footnote listing incompatibilities with mhchem in their PDF documentation.  This should help a lot of people.

My solution doesn't work for more than one char as mhchem actively sets another font in such cases, this is independant from unicode-math. It is not the task of unicode-math to maintain long lists about problematic package -- it is the documentation of mhchem which should mention this.
Beside this: your bug report is one day old and you already have a work-around, people analyzed the problem and also the maintainer of mhchem got involved in the discussion, so be a bit more patient.

@u-fischer  Sorry about that. I have anyway moved on to my thesis. Sorry for the noise.

@u-fischer I tried to update mhchem in a way so that it works with unicode-math and without. At first, \mathnormal seemed very promising, but then I found the reason why I did not use it. The numbers render as old-style figures. So, \mathnormal{...} does not really escape \mathrm. \text{\ensuremath{...}} does, but it does not work with unicode-math. Do you have any more ideas?

@mhchem I would need an example document to play around. And could you say which fonts \ce{A_x} should use when used inside math? I do find it a bit odd that you switch to math mode, isn't a chemical notation done in the text font?

@u-fischer mhchem uses the text font when called from a text environment. And it uses the (upright) math font, when called from a math environment. For layout (superscript and subscript) it uses the math mode (and switches to \mathrm and \text respectively.
However, not all things are upright in a chemical formula. Some things are mathematics like a subscripted variable x. For this, I need a way to switch from \mathrm to "normal" math. \mathnormaldoes not work properly. I found that \text{\ensuremath{...}} works well, but this is exactly what brakes with unicode-math.
So, I am looking for an alternative for $\mathrm{upright\text{\ensuremath{x-1}}upright}$. I thought, there would be a generic solution and already asked at https://tex.stackexchange.com/q/428999/73371. (The first answers indicate that there is no generic solution that works without unicode-math-specific code.)

I did understand what mhchem is doing now. But does it switch to mathrm because of a logical reason, or only "to get the correct output"? As an example lets assume that the text font, mathrm and the math font are really different looking fonts (lmodern, Arial and Cambria in the example below). Which output do you want to get in such a case?
\documentclass{article}
\usepackage{mhchem}
\usepackage{unicode-math}
\setmathfont{Cambria Math}
\setmathfontface\mathrm{Arial}
\begin{document}
some Text \ce{A_x}

\( \symup{A}_x \quad \mathrm{A}_x \quad \text{\ce{A_x}} \)

\sffamily some sans serif Text \ce{A_x}

\( \symup{A}_x \quad \mathrm{A}_x \quad \text{\ce{A_x}} \)

\end{document}

Good question.
When used in running text, \ce will take the current text font to blend it as best as it can. This is what your examples show. But this has nothing to do with this thread.
When used in a math environment, like $m(\ce{NO_x})$, \ce will take the math font. Only in that case, \mathrm is used and the problem of this thread occurs.

@mhchem  perhaps a solution to the issue could be using a standard if....else...endif equivalent in TeX programming to check if unicode-math package has been loaded and act accordingly?

"the math font" doesn't make sense. \mathrm is not "the" math font, it is one of a variety of math alphabets. And you didn't answer the question why do you use in math \mathrm instead of \symrm or switching to text mode with \text. But imho this is no longer an issue for unicode math and you should perhaps move the discussion to a better place.

"the math font" means: the set of mathematical alphabets, as you call them.
Scientific typography follows certain rules. Variables are printed in italics, operators etc. are not. This is the reason, why the sinus operator is not $sin$, but printed in upright font. It is the same for chemical elements, they are printed using an upright font. For the same reason \sin uses the upright math font and not \text, mhchem uses the upright math font and not \text. For that, I use \mathrm because that was – and to my understanding still is – the universal way to get the upright math font. \symrm is unicode-math-specific, isn't it?
I still think, it is a unicode-math-related issue, because it is this package that introduces this inconsistency and breaks the behavior of existing documents and packages that now are forced to adapt.

I still think, it is a unicode-math-related issue, because it is this
package that introduces this inconsistency and breaks the > behavior of
existing documents and packages that now are forced to adapt.

Well not really, as I mentioned on tex.sx the underlying mechanisms are
completely different and unicode-math hides the differences as far as
possible but in this case not enough for you not to notice.
in classic tex math usually uses 16 different fonts and \mathrm switches to
one of them and 9at the primitive tex level) that switch is not seen by
nested math contests so nested math starts off again in italic.
when using a Unicode Math font then (normally) all teh symbols come from a
single font and math italic is not just the normal ascii slots in a math
italic font the characters are translated up to the plane 1 1Dxxx Unicode
Math range, so at the primitive level this is inherited in nested math
contexts.
So the difference is not due to the unicode-math package it is because you
are using a Unicode math font and Unicode math layout engine.

Yes. I used align environment to align the two reactions at their reaction arrows (as per the mhchem manual)

I find it strange to have a different font for chemistry depending whether you are in a math environment or not…

@davidcarlisle, I think we got lost in the fine-details of language. Sorry, my English is a foreign language for me. I did not mean to say that unicode-math does anything wrong. I just wanted to say the issue we discuss here is related to unicode-math, because, to a user, it looks like "when I load unicode-math, my documents look different than before".
@krishnakumarg1984 Currently, there is no option to use \begin{align} and the text font. Hmm. I always thought short chemical names (like \ce{H2O}) should appear in the running text with the respective text font. And dispayed, aligned equations should use the math font. If a document should look very consistent, then the setup should use a math font and a text font that match very well.
Anyway, I will update mhchem to detect the presence of unicode-math.
@u-fisher thanks for pointing me to the issues with beamer. It really does a strange setup.
@ArchangeGabriel The possibility to use \ce{H2O} in text while it adapts to the current font, is one of the main features of mhchem (i.e. main body, section heading, page header, toc, image caption, etc.).

Fixed with mhchem v4.08 (2018-06-22), uploaded to CTAN a few minutes ago.


change to reference variable

use reference variable to make sure keypoints is updated.

I signed it!

Nagging Assignee @protoget: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 45 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 60 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 75 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 75 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 90 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 20 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 35 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 50 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @protoget: It has been 65 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


State-level Sector edits

Set of small changes to pre-processed scaled impact and scalar data for a few sectors.

Merged into branch state_code_addSectors with merge #91. Then merged into branch state_code with merge #92.


Stop sending any emails to my email address

I have asked to stop sending any emails to my public GitHub email address, but I got another message:

For your information, this goes against CAN-SPAM Act and GDPR.
Further email messages will be reported to the police.
Kind regards.

Hey @pipboy96 sorry about that. There's an unsubscribe link in the email you can click to exclude yourself from all future emails. I will triple-check that you are already excluded from any marketing emails.


chore: Improve code style

Nice change! Weighing in a bit, any thoughts on automating this thing? Seems that Prettier 1.4 has TS support - Does thing worth considering? Thanks!


Wordpress stack: As a new stacks user, I'd like an example of a stack with a simple controller

Related to #572
What seems to be the problem?

As part of #572, we want to have a simple, realistic example stack with a simple controller. We are imagining that we should start by creating a wordpress stack which uses default resource classes to support multiple providers with a single app definition. We are imagining that it lives in its own repository. This will be an example that people can look at to understand how stacks work. The reason we're using wordpress is because it's become an idiom representing a simple application in the Kubernetes world.
What could Crossplane do to help?
Because this is the first real-ish stack for running an application, we want to start with the simplest possible stack to write. We are currently imagining that this would be a stack with a simple controller which watches for the creation of wordpress instance objects, and spins up wordpresses in response to that, by generating a series of objects similar to what can be found in the wordpress workloads examples.
What does this look like when it's done?

We have a stack which can be used to create wordpresses which has been tested with GCP and AWS
We are imagining that the stack should have a controller, but the idea is that this is the simplest possible stack to write at this time
The stack lives in its own repo
The stack can be built, published, and installed easily

I'm working on this in a branch in a personal repo while we think about where we want this to live in the long term.

Currently, I have a blank kubebuilder2-based project with a single API, and with some extra tooling to make developing a stack nice, which means some Make recipes for project initialization, stack bundling, and local installation. I've been taking some extra time as I also spend some time to become more familiar with our existing thoughts on Stacks and kubebuilder.
Now that I have a workflow running end-to-end with an empty controller, I'll be working on adding the functionality to press wordpresses. 🗞 🗞 📰

The work is now being tracked in this Crossplane org repo. I'm going to open a new ticket to break spinning up the project into a separate work item so we can track progress at a finer level.

I've got a proof of concept in a branch currently working that I'm going to make more robust.

I'm currently working on cleaning this up and making it nicer to use, but it's going to roll over into the next sprint.

I've smoothed out the proof of concept a bit. Now it can create uniquely named wordpresses, and has some light usage documentation. The changes are in the master branch. I now plan to shift focus over to some of the CLI-like commands for managing some of the development lifecycle phases of a stack, and then I'll return to the stack itself later. Some people will be playing with the stack in the meantime.

This was ready a while back, but wasn't tested in AWS yet. Now, there is some work to update the wordpress stack to use the latest crossplane features and changes, but I'm going to close this ticket and open a new one to track the upgrade work and the additional testing.


[terra-button] Fixes focusing issue on button long press, and focus not received on closing modals.

Summary
Removed timeout from a previous change which failed wdio tests if the focus of the button was tested immediately after closing modals.
Removing timeout also fixes a issue where long pressing the button enabled click focus-styles.
closes : #2790 #2789

Terra-button cannot have a dependency on terra-abstract-modal. You'll have to find another way to test these changes.

updated here 716688cc6b76a7f8f6ba276ed52158a5779e87f5

After doing further investigations offline with @lokesh-0813 what we came to understand are the following:

handleMouseDown

Previously, setTimeout(() => { this.shouldShowFocus = true; }); here was needed to reset this.shouldShowFocus back to true to prevent the button from getting the focus style when manually clicked because handleFocus would get called immediately after handleMouseDown (only in Chrome). However, with event.preventDefault() added, handleFocus would no longer get called automatically in Chrome. Instead, handleFocus would get called from handleClick for the need to programmatically set focus on the button without the focus style. Given that we have event.preventDefault(), it is safe to remove setTimeout(() => { this.shouldShowFocus = true; });` since it is no longer needed.

handleClick

Button in Chrome automatically gets focus after clicked. However, FF, IE, and Safari do now. For this reason, we need to programmatically set focus on the button in handleClick. Additionally, since this is a manual click we do not want to apply the focus style on the button. Therefore, it is needed to set this.shouldShowFocus to false, before calling .focus() and setting this.shouldShowFocus back to true after it is done.


Particles have strange vertical lines

I’m seeing some strange vertical (pixel-like) lines in the particle system (also other strange stuff as well). This happens when I zoom in.
Here is the project: PlayCanvas | HTML5 Game Engine 3
Also, there is a discussion in the forum: https://forum.playcanvas.com/t/particles-have-strange-vertical-lines/28192
@leonidaspir and @mvaligursky confirmed it.
Here are the images;

Not sure what is happening, it seems related to the Depth Softening … if I set this to 0, the issue goes away.
@GSterbrant  - could you please have a look? It might possibly be related to the changes to the linear depth we encode / decode?

Last working engine version is 1.53.4
Project with the camera pre set up: https://playcanvas.com/project/1003680/overview/f-fireball-particle-issue
Launch tab with 1.53.4: https://launch.playcanvas.com/1578471?debug=true&use_local_engine=https://code.playcanvas.com/playcanvas-1.53.4.js
Launch tab with 1.54.0: https://launch.playcanvas.com/1578471?debug=true&use_local_engine=https://code.playcanvas.com/playcanvas-1.54.0.js

I think this issue might be due to an incorrect capture of the scene depth, as demonstrated here:

This is caused by not running:
    this.entity.camera.requestSceneDepthMap(true);

On the camera, which means we never end up running the grab pass for the depth.

this warning when using a debug engine is the reason:

If I create a new script like this, and add it on the scene camera entity, all is good, as the depth for the camera is available
var Test = pc.createScript('test');

// initialize code called once per entity
Test.prototype.initialize = function() {
    this.entity.camera.requestSceneDepthMap(true);
};

Shouldn't the particle system enable that automatically since it's required? Thinking non coders will have trouble working in editor with particles.

We considered this when changing the way the depth is requested. Initially, the depth would be enabled on all cameras when a particle needed it, which is costly. Now user can only enable it on a camera that needs it / that renders soft particles. Engine tracking to see if a particle system is on a layer the camera uses is turning out to be a bit tricky, so it has not been done (yet at least). And similarly tracking if a material needs depth texture - that would need tracking to see if any mesh instance using this material is on a layer of the camera.
Short term plan is to add a tickbox to the camera component, allowing user to generate camera depth texture without a script.


Overwritten recipes mariculture.Anvil

Hello,
with the newest version (9.1) I got some error messages at login. Before that version all "addRecipe" where added side by side (as intened from my side). Now it seems that every line overwrite the gravel recipe. In game the only avabile recipe is the last cobbleblock from UBmetamprophic.
MSG:
WARNING: [ModTweaker2] Overwritten Mariculture Anvil Recipe for minecraft:gravel * 1
SCRIPT:
////Imports
import mods.mariculture.Anvil;

//Vars
val cobblestone = <minecraft:cobblestone>;
val cobblestoneUBigneous = <UndergroundBiomes:igneousCobblestone:*>;
val cobblestoneUBmetamorphic = <UndergroundBiomes:metamorphicCobblestone:*>;

//Blacksmith's Anvil Recipes
//InputStack, OutputStack, NumberOfHits
Anvil.addRecipe(cobblestone, <minecraft:gravel>, 10);
Anvil.addRecipe(cobblestoneUBigneous, <minecraft:gravel>, 15);
Anvil.addRecipe(cobblestoneUBmetamorphic, <minecraft:gravel>, 20);
Anvil.addRecipe(<minecraft:gravel>, <minecraft:sand>, 40);

The handler looks up the ore ID from the item you are providing. If all 'cobblestones' have the same ore id, then they are considered the same input. This behavior hasn't changed during the last versions, but you are now getting warning messages if they overwrite each other. All inputs should.
Maybe we can change this behavior, so you can specify if you want an item or an ore dict entry as input.

Looked at the code of Mariculture, nothing we can do. If we add the items without the oredict name, then Mariculture won't be able to fetch the recipe, because it will always look up the oredict name for the input.
The only solution i could imagine is to remove the items from the oredict before adding the recipes. This way neither the ModTweaker2 handlers nor Mariculture won't be able to lookup the oredict name.

Ok I see the problem, i think your workaround is good for me. Thank you for this solution :smile:


Remove things to lighten support burden

Just trying to simplify here.
Removes:

Blogging demo sql. This should be maintained in docs not here.
Debian packaging scripts. The PostgREST distribution is a single binary file, very easy to install. Hope to keep it that way!
Old docs placeholder that points at docs repo -- postgrest.com now points at docs.

One thing from the deleted files is valuable and should be moved somewhere, init script (although it needs some work).
It's a non trivial thing to properly install postgrest as a system service (and not just a binary you manually start)

Good point. We can research and document ways to make the server a linux service.


Adding security context to milvus-helm

I need to deploy milvus on kubernetes cluster that is being run through rancher. To do that I need to modify the security context both for the pod and the containers in the following way (below is an example values.yaml file):
apiVersion: apps/v1
kind: Deployment
metadata:
    name: example
spec:
  replicas: 1
  selector:
    matchLabels:
      app: example
  template:
    metadata:
      labels:
        app: example
    spec:
      securityContext: # Pod security context
        fsGroupChangePolicy: OnRootMismatch
        runAsNonRoot: true
        seccompProfile:
          type: RuntimeDefault
      containers:
      - image: ubuntu
        name: example
        securityContext: # Container security context
          runAsUser: 1000
          allowPrivilegeEscalation: false
          capabilities:
            drop:
            - ALL

However, I am not exactly sure, how I should modify the sections of default values.yaml and which sections require these modifications. If there are any tips on that, they would be greatly appreciated.
Thank you very much in advance!

This may require a patch to milvus https://github.com/milvus-io/milvus. Milvus image is now run in root by default.


Fixed Room Import Bug, lenient_late pairing bug

Can you re-trigger the travis run? Other than that lg2m
:+1:

actually I can kick it


UsePathBase does not disable root path

When configuring the base path of the application, the root path continues to work, for example:
        public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
        {
            app.UsePathBase("/basepath");

            app.Map("/ping", map => map.Run(async
                ctx => await ctx.Response.WriteAsync("pong")));

            app.UseMvc();
        }

With this configuration both /basepath/ping and /ping work. I would expect /ping to return a 404.
This is the same behavior demonstrated by setting ASPNETCORE_URLS to http://+:5000/basepath though I believe setting the base path this way has been depreciated.

It doesn't feel right, but it looks like it probably is to me.
For every request there's two paths, the Path and the PathBase. AFAIK, most middleware just uses the Path to determine whether it should execute.
Both the PathBaseMiddleware and MapMiddleware operate in a similar way - they move segments that match an expected value from the Path to the PathBase. The main difference is that the MapMidddleware runs the inner pipeline if the segment matches, whereas the PathBaseMiddleware always just runs the rest of the pipeline as normal.
So imagine you have a request for /basepath/ping. With the pipeline you defined, you get the following sequence of events:

At the start of pipeline, Path = "/basepath/ping", PathBase = ""
PathBaseMiddleware sees the /basepath prefix on Path so moves it from Path to PathBase.
(Path = "/ping", PathBase = "/basepath")
MapMiddleware sees the /basepath prefix so moves it from Path to PathBase.
(Path = "", PathBase = "basepath/ping")
As the MapMiddleware matched the provided segment, it runs the inner "pong" pipeline.

That's the behaviour you expect I believe. So now lets look at a request for /ping.

At the start of pipeline, Path = "/ping", PathBase = ""
PathBaseMiddleware sees that the Path variable is not  prefixed with /basepath, so it doesn't modify Path or PathBase
(Path = "/ping", PathBase = "")
MapMiddleware sees the /ping prefix so moves it from Path to PathBase.
(Path = "", PathBase = "basepath/ping")
As the MapMiddleware matched the provided segment, it runs the inner "pong" pipeline.

I believe, what you really want is to have the first scenario only, and for the second scenario to return a 404. In which case, you probably want something closer to this, with a nested MapMiddleware:
public void Configure(IApplicationBuilder app)
{
    app.Map("/basepath", mainapp =>
    {
        mainapp.Map("/ping", map => map.Run(async
            ctx => await ctx.Response.WriteAsync("pong")));

        mainapp.UseMvc();
    });
}

This ensures that if the initial /basepath doesn't match, then the pipeline is skipped.

I understand the expectation, but it is working as intended. UsePathBase is primarily about getting those segments out of your way because they're a deployment detail, and if they stayed it would mess up your routing.
Consider the alternative. If it were to disable the root path, how would it do so? A 404 isn't really appropriate, presumably that path is available on a separate instance of your site hosting the root. You could use Map as shown above if you really wanted the 404.
IIS/Http.Sys uses a 503 for this scenario, which we could do, but you wouldn't have an opportunity to provide content that fit the theme of your site. The IIS/Http.Sys 503 page is pretty stark. If you wanted a 503 you could use Map and then put a proper error page on the other branch.