GCSFileSystem.walk broken with kwarg detail=true

A project I recently joined recently upgraded from v0.2.3 to v0.3.0. I've been hunting a bug related to a change in the behavior of GCSFileSystem.walk described in some detail below. Before that, I'll say up front that my ultimate question is:
What is the best way in v0.3.0 to get the same "path" and "md5Hash" information given by walk(path,detail=True) in v0.2.3?
Before: v0.2.3
Before upgrading, my the project code called walk(path,detail=True). For reference, see what this returns when called on one of the public GCP datasets:
>>> fs = gcsfs.GCSFileSystem()
>>> w = fs.walk('gs://gcp-public-data-landsat/index.csv.gz', detail=True)
>>> pprint.pprint(w)
[{'bucket': 'gcp-public-data-landsat',
  'contentType': 'application/octet-stream',
  'crc32c': '6p/Z1w==',
  'etag': 'CNT6gcy4ouQCEAE=',
  'generation': '1566887980858708',
  'id': 'gcp-public-data-landsat/index.csv.gz/1566887980858708',
  'kind': 'storage#object',
  'md5Hash': 'wvqmRKytddbPHfXC/ygYhg==',
  'mediaLink': 'https://www.googleapis.com/download/storage/v1/b/gcp-public-data-landsat/o/index.csv.gz?generation=1566887980858708&alt=media',
  'metageneration': '1',
  'name': 'index.csv.gz',
  'path': 'gcp-public-data-landsat/index.csv.gz',
  'selfLink': 'https://www.googleapis.com/storage/v1/b/gcp-public-data-landsat/o/index.csv.gz',
  'size': 601644439,
  'storageClass': 'STANDARD',
  'timeCreated': '2019-08-27T06:39:40.858Z',
  'timeStorageClassUpdated': '2019-08-27T06:39:40.858Z',
  'updated': '2019-08-27T06:39:40.858Z'}]

The project only uses the information keyed by 'path' and 'md5Hash'.
After: v0.3.0
However, in version 0.3.0 the same method call returns a generator instead of a list:
>>> fs = gcsfs.GCSFileSystem()
>>> w = fs.walk('gs://gcp-public-data-landsat/index.csv.gz', detail=True)
>>> pprint.pprint(w)
<generator object AbstractFileSystem.walk at 0x7f0e9e005938>

If you try to convert it to a list, an exception is raised:
>>> list(w)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ahmed/COLGATE/env/lib/python3.6/site-packages/fsspec/spec.py", line 338, in walk
    listing = self.ls(path, True, **kwargs)
TypeError: ls() got multiple values for argument 'detail'

Weirdly enough, in contrast, conversion to a list via list comprehension gives an empty sequence:
>>> [x for x in w]
[]

As does converting to a tuple:
>>> tuple(w)
()

The documentation for v0.3.0 is updated to no longer list an optional detail argument for walk. Still, I need functionality equivalent to the v0.2.3 version of walk(path, detail=True) i.e. I need the "path" and "md5Hash" information for each file below path.
I'm not sure if the new ls is functionality equivalent to the old walk, but I've tried instead using ls(path,detail=True):
>>> l = fs.ls('gs://gcp-public-data-landsat/index.csv.gz', detail=True)
>>> pprint.pprint(l)
[{'bucket': 'gcp-public-data-landsat',
  'contentType': 'application/octet-stream',
  'crc32c': '6p/Z1w==',
  'etag': 'CNT6gcy4ouQCEAE=',
  'generation': '1566887980858708',
  'id': 'gcp-public-data-landsat/index.csv.gz/1566887980858708',
  'kind': 'storage#object',
  'md5Hash': 'wvqmRKytddbPHfXC/ygYhg==',
  'mediaLink': 'https://www.googleapis.com/download/storage/v1/b/gcp-public-data-landsat/o/index.csv.gz?generation=1566887980858708&alt=media',
  'metageneration': '1',
  'name': 'gcp-public-data-landsat/index.csv.gz',
  'selfLink': 'https://www.googleapis.com/storage/v1/b/gcp-public-data-landsat/o/index.csv.gz',
  'size': 601644439,
  'storageClass': 'STANDARD',
  'timeCreated': '2019-08-27T06:39:40.858Z',
  'timeStorageClassUpdated': '2019-08-27T06:39:40.858Z',
  'type': 'file',
  'updated': '2019-08-27T06:39:40.858Z'}]

It looks like this is returning the desired info, where the "md5Hash" key is the same and there is a "name" key instead of "path". Is this correct?

I don't believe that walk should work like this, since the builtin version explicitly returns an iterator of tuples. You could propose this change, instead, for how find() works, which is like the previous version of walk.


Fix getting ClassCastException on DesiredCapabilities to IOSCapabilities

IOSDriver is the only driver that takes a non DesiredCapabilities
object.  Therefore we need to create the capabilities as an
IOSCapabilities instance, not a DesiredCapabilities instance.

merged.


Sortable: Restrict multi list re-order by default

The sortable directive must be single list sortable by default unless configured i.e. Items should be re-orderable with the list only. Right now this is not the default behavior.
Kinda like scopes. Not sure how we should approach this. I am open to ideas.

Hi @ObaidUrRehman, @cah-alexsukhodolsky
Our team has been waiting to the sortable/re-orderable feature for so long. We hope it is still on the early/next release. Thanks again to this great work.
Congrats

@calebeaires I've been busy lately. But I am going to restart this feature now. Thanks for being patient :-)


Support local source code in Shipwright

Idea:
Provide support for local source code.  But adding here some key points on adopting this feature. EP for feedback.
Context

IBM folks demo this back in February 2021 @xiujuan95 @mattcui
Tekton currently does not support this feature.

Requirements

The implementation to make this work will rely heavily in Shipwright/CLI.
For this repository(Shipwright/Build) we need to find the right way to signalise to our controllers, the desire of using local source code from users. API changes and code changes when creating a TaskRun are required, in order to make this functional.

Implementation details

As previously mentioned, we would like to propose an approach where we abuse of the usage of container registries ( you need one to build an image, you probably have one if you are running in k8s ), so that the local source code is pushed into a bundle image that self-extracts on runtime.
The above is just one approach of many, CLI should be generic enough to extend to further approaches, like relying on s3 to store code or other ways of storing.

Examples

@mattcui @xiujuan95 demo this repo https://github.com/mattcui/s2i-local.
@qu1queee have a prototype with shp, see:

Changes in Shipwright/Build, see branch
Changes in Shipwright/CLI, see branch
Note: The changes in CLI include a pinned module, please modify this to your local one.

Adding this to the v0.6.0 milestone because this is a feature we would like to have soon. Getting agreement in the enhancement proposal (merged and marked "implementable") is a prerequisite to completing this feature.


Add tag for scodix textures

Fixes https://tfs.farheap.com/ONP/overnightprints.com/_workitems/edit/31755?fullScreen=false

Confirm that the boolean flag can be removed, ProFIT will check the presence of Scodix coating on the side of given order item by looking at the variable_uv_file_url parameter on the XML.
Sorry for the inconvenience.

Agreed.  Need hashes.  Is this what the ProFIT team implemented?  Why is the UV uppercased on one of the XML nodes?

Currently ProFIT does not check hashes for any types of images ONP sends there (CMYK or SpotUV), same is for Scodix files - ProFIT is not going to validate hash for them for now.


Add "Mock API" action to dashboard

As a user i would like to mock an API defined in my OpenAPI definition - I know I can do this by going through the existing deployment wizard and/or by adding the corresponding x-kusk extension, but it would be nice to have a "shortcut" action for this which only prompts me for the OpenAPI definition and nothing else.
The implementation would

add the x-kusk.mocking.enabled extension at the root level
generate a unique API name from the name of the API in the definition - prefixed with "mock"
deploy the API in the kusk namespace

PR - https://github.com/kubeshop/kusk-gateway-dashboard/pull/40


Remove 'express' dependency

It seems like Express isn't used anywhere and it's a huge dependency taking a lot of time to install.

Sorry, no need to rush this, but if you have time please let me know if removing the Express dependency is acceptable.

@blitzprog, thanks for the PR! Sorry that it took me so long to merge this. I missed the email notifications from Github for this and only noticed this PR when you pinged us on Twitter.


Full BigQuery load errors are neither logged nor displayed for dbt seed

Issue description
Upon running dbt seed for a specific CSV file, the load to BigQuery fails. dbt prints the output as follows:
  Error while reading data, error message: CSV table encountered too many errors, giving up. Rows: 1; errors: 1. Please look into the error stream for more details.

Unfortunately, this message doesn't provide the root cause of the error (i.e. the issues encountered loading the CSV file). I don't see any additional detail in dbt.log beyond what's printed.
I've used the BigQuery Python SDK before for load jobs, and this looks like the output from job.result(). However, the job object has an additional attribute called errors, which contains the detailed description of the load error. In this past I've handled this with a try/catch block on job creation that raises the full error when a generic API exception is caught. I believe this is what should be logged or surfaced to the user. Happy to attempt a PR if you can point me roughly to where this code lives in dbt!
System information
The output of dbt --version:
installed version: 0.11.0
   latest version: 0.11.1

The operating system you're running on: Mac OS Mojave
The python version you're using (probably the output of python --version) Python 3.4.3

Thanks for the report @joshtemple!

@drewbanin, would you mind pointing me to where this code exists in dbt? I'd be happy to attempt a fix, I think it would be relatively quick.

@joshtemple this is the python code that loads csvs into bigquery: https://github.com/fishtown-analytics/dbt/blob/dev/guion-bluford/dbt/adapters/bigquery/impl.py#L604-L621
That error message must come from bigquery itself, I don't see it anywhere in dbt.

Cool, thanks @beckjake. I've submitted #1079 to address this.


Executing MobileVRInterface.initialize crashes Godot

Godot version
4.0.beta.custom_build. d3fc9d9e4
System information
Ubuntu 22.10 - AMD RX 470, Gnome shell 43 X11
Issue description
When executing
extends Node
func _process(delta):
	var temp_variable4236 = MobileVRInterface.new()
	temp_variable4236.initialize()

Godot crashes:
ERROR: Trying to unreference a SafeRefCount which is already zero is wrong and a symptom of it being misused.
Upon a SafeRefCount reaching zero any object whose lifetime is tied to it, as well as the ref count itself, must be destroyed.
Moreover, to guarantee that, no multiple threads should be racing to do the final unreferencing to zero.
   at: _check_unref_sanity (./core/templates/safe_refcount.h:173)

================================================================
handle_crash: Program crashed with signal 4
Engine version: Godot Engine v4.0.beta.custom_build (d3fc9d9e416560d228a7914a82902118ce911a4d)
Dumping the backtrace.
-- END OF BACKTRACE --
================================================================

This example was found by Godot fuzzer - Qarminer, so it is quite unlikelly that this code could be used in real project, but still this should be handled gracefully.
Memory leaks or asan backtraces are visibe when using Godot build with sanitizers support - https://github.com/qarmin/GodotBuilds/actions (linux -> linux-editor-sanitizers)
Steps to reproduce
Above
Minimal reproduction project
N/A

I think this is already fixed in master, could you please double-check?

Still crashes with c3539b456


⚠ Fortnite has degraded performance

In 4d52ec1, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1438 ms

Resolved: Fortnite performance has improved in 1bba63f.


Dark Mode Toggle

Is your feature request related to a problem? Please describe
This Projects Needs a Dark Mode Toggle ,which should be on the top right corner
Describe the solution you'd like

Describe alternatives you've considered
We can Create a Dark Mode toggle using JavaScript in this Project
Additional context
No response
Are you working on this?
No

Hi, I'm interested in contributing to add the dark mode toggle on this project. Please assign me the relevant issue and I'll be happy to start working on it. Thank you!

Hi @kumarankit999 I have made a pull request for this issue.

Hey @kumarankit999, is this issue still open? I would love to work on it.

Hey,is anyone working on this? If not I want to work on this.


Use callback param instead of script onload

Fixes #26
Motivation
See #26. This work is a bit speculative - I haven't verified that the failure mode that I described in #26 is actually what's happening, but this is the approach recommended by the google maps docs.
Changes

Add onGoogleMapsLoad callback to window at the beginning of load
Put callback logic into that callback
Add callback parameter to params object, placing it before the destructuring so that a consumer could override it if they wanted to for whatever reason
Only run script load callback if it's an error

Testing
Code review. See that tests pass. Tell me if there's another test that you'd like to see.

Codecov Report

Merging #27 into master will decrease coverage by 8.23%.
The diff coverage is 63.63%.

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
- Coverage   89.18%   80.95%   -8.24%
==========================================
  Files           2        2
  Lines          37       42       +5
  Branches        7        8       +1
==========================================
+ Hits           33       34       +1
- Misses          3        6       +3
- Partials        1        2       +1

Impacted Files
Coverage Δ

src/loadGoogleMapsSdk/index.js
80% <63.63%> (-12%)
:arrow_down:

Continue to review full report at Codecov.

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

Anecdotally, a similar change in our codebase resulted in a lower error rate.

I'm not done with this, but I've pushed where I've gotten so far.
On master, the error behaviors aren't actually tested. The expect statements inside of the setTimeouts are never exercised - the test goes right past them to done() and calls the test good.
My guess for why the tests are considered "covered" is that setting up the callback involves reading through those lines, and the coverage counter considers them "covered" because of that.
With the auth error test, there's some weird behavior on the part of the Google Maps SDK. It runs the SDK load and the auth concurrently and asynchronously, and while the SDK loads in less than a second, the auth takes up to 6s to execute. This means that the onLoad callback is triggered before the auth error callback. I'm not sure if this means that there will need to be changes in the code that triggers the callback handed to loadGoogleMapsSdk.
With the network error test, the easiest way to get it to fail in the scope of the test is to change the URL that it's referencing. This is why I added a third argument to loadGoogleMapsSdk which defaults to the SDK URL. It's not intended to be a part of the public API, just a test convenience.
Finally, the network error test is still failing. It passes if you add test.only to that test and make it the only executed test in the module. This points to the module-level state (i.e. googleMaps and state) not getting reset between test runs, so the loadGoogleMapsSdk call short-circuits because state === LOADED.
I attempted to address this by adding the require in beforeEach, which should re-execute the module and reset the module's state before each test. However, jest runs tests in parallel by default, which means that there's a race condition, and the tests still fail. The simplest way to address this (short of a pretty major rework of loadGoogleMapsSdk that takes advantage of memoization or some other way of not relying on module scope) is to make the tests run in sequence. The docs for that are here, but I ran out of time to figure that out and implement it today. I'll see if I can return to this on Monday.

... The fact that tests passed is concerning to me. I don't think they should have.

Good investigation about this 👏🏻Running tests in sequence could be a good improvement

@tstirrat15 What is the status of this ?

Sorry, it very much fell through the cracks. I'll see if I can have another go at it in the next week.

Closing because there is no update


voice noise doesn't stop if skipped

pressing "ctrl" to skip dialogue while a voice noise is happening causes it to continue playing in navigation screens.

to be fair, when you implemented this, we didn't have the nav screens, so i dont blame you or anyone for not realizing this would happen


issue while using LSTM in functional mode

System information

Have I written custom code (as opposed to using example directory):  yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Windows 10 Pro
TensorFlow backend (yes / no):  yes
TensorFlow version:  1.13.1
Keras version:  2.2.4
Python version: 3.7.3
CUDA/cuDNN version:  NA
GPU model and memory:  NA

Describe the current behavior
When creating a LSTM layer using the functional moded below error is thrown
ValueError: Input 0 is incompatible with layer lstm_1: expected ndim=3, found ndim=2
Describe the expected behavior
LSTM layer is created successfully
Code to reproduce the issue
import keras
from keras.layers import Input, LSTM, Dense
from keras.models import Model

import numpy as np

def build_lstm(lstm_input, hidden_dim=256):
    out = LSTM(hidden_dim, input_shape=(1, 129))(lstm_input)
    return out

def test_lstm():
    lstm_input = Input(shape=(129, ), name='lstm_input')
    out = build_lstm(lstm_input)

    predictions = Dense(256, activation='softmax')(out)
    model = Model(inputs=lstm_input, outputs=predictions)
    model.compile(loss='categorical_crossentropy',
        optimizer='rmsprop',
        metrics=['accuracy'])

    model.summary()

    return

Error Log
(NPI_Arch_Keras) C:\workspaces\NPL\project\NPI_Arch_Keras\npi>python npi_lstm_test.py
Using TensorFlow backend.
Traceback (most recent call last):
  File "npi_lstm_test.py", line 35, in <module>
    test_lstm()
  File "npi_lstm_test.py", line 21, in test_lstm
    out = build_lstm(lstm_input)
  File "npi_lstm_test.py", line 15, in build_lstm
    out = LSTM(hidden_dim, input_shape=(1, 129))(lstm_input)
  File "C:\MyProgramFiles\Anaconda3\envs\NPI_Arch_Keras\lib\site-packages\keras\layers\recurrent.py", line 532, in __call__
    return super(RNN, self).__call__(inputs, **kwargs)
  File "C:\MyProgramFiles\Anaconda3\envs\NPI_Arch_Keras\lib\site-packages\keras\engine\base_layer.py", line 414, in __call__
    self.assert_input_compatibility(inputs)
  File "C:\MyProgramFiles\Anaconda3\envs\NPI_Arch_Keras\lib\site-packages\keras\engine\base_layer.py", line 311, in assert_input_compatibility
    str(K.ndim(x)))
ValueError: Input 0 is incompatible with layer lstm_1: expected ndim=3, found ndim=2

The below version of the code which uses the Sequential model runs successfully
import keras
from keras.models import Sequential
from keras.layers import LSTM, Dense

import numpy as np

def build_lstm(hidden_dim=256, input_shape=None):
    model = Sequential()
    model.add(LSTM(hidden_dim, return_sequences=True, input_shape=input_shape))
    model.add(LSTM(hidden_dim))
    return model

def test_lstm():
    model = Sequential()
    model.add(build_lstm(input_shape=(1, 129)))
    model.add(Dense(256, activation='softmax'))

    model.compile(loss='categorical_crossentropy',
              optimizer='rmsprop',
              metrics=['accuracy'])

    model.summary()

    return

Success log
(NPI_Arch_Keras) C:\workspaces\NPL\project\NPI_Arch_Keras\npi>python npi_lstm_working.py
Using TensorFlow backend.
WARNING:tensorflow:From C:\MyProgramFiles\Anaconda3\envs\NPI_Arch_Keras\lib\site-packages\tensorflow\python\framework\op_def_library.py:263: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Colocations handled automatically by placer.
_________________________________________________________________
Layer (type)                 Output Shape              Param #
=================================================================
sequential_2 (Sequential)    (None, 256)               920576
_________________________________________________________________
dense_1 (Dense)              (None, 256)               65792
=================================================================
Total params: 986,368
Trainable params: 986,368
Non-trainable params: 0
_________________________________________________________________

I was able to execute both of the code snippets successfully in TF 1.13.1 using google colab. From the stack trace I suspect that the reported error arises from different line of code.

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


convert_language_name function missing

Expected Behavior
Launch the app.py without any errors
Current Behavior
Unable to launch the application
Steps to Reproduce

pull down latest repo
remove virtual environment and recreate
try to run python app.py --host <IP_ADDRESS>

Error:
Traceback (most recent call last):
File "/home/user/lollms-webui/app.py", line 30, in
from lollms.utilities import AdvancedGarbageCollector, reinstall_pytorch_with_cuda, convert_language_name
ImportError: cannot import name 'convert_language_name' from 'lollms.utilities' (/home/user/lollms-webui/lollms_core/lollms/utilities.py)
Context
If change app.py line:
from lollms.utilities import AdvancedGarbageCollector, reinstall_pytorch_with_cuda, convert_language_name
to
from lollms.utilities import AdvancedGarbageCollector, reinstall_pytorch_with_cuda #, convert_language_name
launch the app with python app.py --host <IP_ADDRESS> and then remove the comment from above the app will launch again
Also, looks like there might be some work being done around this recently: https://github.com/ParisNeo/lollms-webui/blame/bb43498c89c185eab2d67af8d86a984cfdb80568/api/init.py#L41

Hi, this function is in lollms_core. It has been added lately. It is in the repo, so maybe you just need to co to the folder lollms_core and do a git pull.

That would be it! Thank you and happy holidays!

You are welcome


As a head nurse, I can delete contacts of patients who have been discharged

Existing Delete function can satisfy this condition.


WIP: implementation of malloc hooks using LD_PRELOAD

This is an implementation of library hooking malloc, realloc, calloc and free using LD_PRELOAD. It is still a work in progress because the given results are, in some cases, different from those given by ltrace.
For example using the following test code:
#include <stdlib.h>

int    main(int argc, char **argv)
{
  (void) argc, (void) argv;

  void *a;
  int i;

  for (i = 0; i < 10; ++i)
    {
      a = malloc(i * 1000);
      if (i % 2)
	free(a);
    }

  return 0;
}

will produce the same output as ltrace.
However, when I tried to run it with the ls command, the result were a bit different.
For testing purpose, you can use the Makefile:
make

and launching a program is done by doing:
LD_PRELOAD=/path/to/mallhook.so command

Does not compile cleanly on my system. Could you fix the errors bellow?
gcc mallhook.c -shared -fPIC -o mallhook.so -W -Wall -Wextra -ansi -O3 -pipe -ldl
/usr/include/stdlib.h: In function ‘malloc’:
mallhook.c:90:8: warning: ignoring return value of ‘write’, declared with attribute warn_unused_result [-Wunused-result]
   write(outfile, buf, strlen(buf));
        ^
/usr/include/stdlib.h: In function ‘calloc’:
mallhook.c:110:8: warning: ignoring return value of ‘write’, declared with attribute warn_unused_result [-Wunused-result]
   write(outfile, buf, strlen(buf));
        ^
/usr/include/stdlib.h: In function ‘realloc’:
mallhook.c:133:8: warning: ignoring return value of ‘write’, declared with attribute warn_unused_result [-Wunused-result]
   write(outfile, buf, strlen(buf));
        ^
/usr/include/stdlib.h: In function ‘free’:
mallhook.c:149:8: warning: ignoring return value of ‘write’, declared with attribute warn_unused_result [-Wunused-result]
   write(outfile, buf, strlen(buf));
        ^

Also sed -i 's/[ \t]*$//' "$1".
-2 norme (inside joke)

Is there a reason tu use snprintf + write instead of fprintf?

I had a segmentation fault using fprintf, didn't really look into it, I think it comes from the hooks, but it works with snprintf + write anyway. But yeah, I agree that it is not that beautiful.


Update README and AboutUS

Under README: Changed the Java CI to the correct one and removed the build tag
Under AboutUS: Fixed a small typo

LGTM


[BUG]OPENSEARCH_JAVA_HOME env is not working

Hi,
I am creating this issue from the response of this
Describe the bug
I am using Opensearch-1.2.3 tar which ships Java 15 with it. But I want to use Java 11. So I referred Compatibility - OpenSearch documentation which stated to “set OPENSEARCH_JAVA_HOME environment variable to the Java install location” . I tried it but Opensearch installation used bundled jdk-15 but not jdk-11 which I want to use. Logs are as below.
[opensearch-1.2.3]$ export OPENSEARCH_JAVA_HOME=/etc/alternatives/jre_openjdk
[opensearch-1.2.3]$ ./bin/opensearch
[2022-01-07T11:47:12,157][INFO ][o.o.n.Node               ] [chai-node] version[1.2.3], pid[16189], build[tar/8a529d77c7432bc45b005ac1c4ba3b2741b57d4a/2021-12-21T01:36:21.407473Z], OS[Linux/3.10.0-957.10.1.el7.x86_64/amd64], JVM[AdoptOpenJDK/OpenJDK 64-Bit Server VM/15.0.1/15.0.1+9]
[2022-01-07T11:47:12,162][INFO ][o.o.n.Node               ] [chai-node] JVM home [/home/cloud-user/chaitra-opensearch/opensearch-1.2.3/jdk], using bundled JDK [true]
[2022-01-07T11:47:12,162][INFO ][o.o.n.Node               ] [chai-node] JVM arguments [-Xshare:auto, -Dopensearch.networkaddress.cache.ttl=60, -Dopensearch.networkaddress.cache.negative.ttl=10, -XX:+AlwaysPreTouch, -Xss1m, -Djava.awt.headless=true, -Dfile.encoding=UTF-8, -Djna.nosys=true, -XX:-OmitStackTraceInFastThrow, -XX:+ShowCodeDetailsInExceptionMessages, -Dio.netty.noUnsafe=true, -Dio.netty.noKeySetOptimization=true, -Dio.netty.recycler.maxCapacityPerThread=0, -Dio.netty.allocator.numDirectArenas=0, -Dlog4j.shutdownHookEnabled=false, -Dlog4j2.disable.jmx=true, -Djava.locale.providers=SPI,COMPAT, -Xms1g, -Xmx1g, -XX:+UseG1GC, -XX:G1ReservePercent=25, -XX:InitiatingHeapOccupancyPercent=30, -Djava.io.tmpdir=/tmp/opensearch-12173583538575190589, -XX:+HeapDumpOnOutOfMemoryError, -XX:HeapDumpPath=data, -XX:ErrorFile=logs/hs_err_pid%p.log, -Xlog:gc*,gc+age=trace,safepoint:file=logs/gc.log:utctime,pid,tags:filecount=32,filesize=64m, -Dclk.tck=100, -Djdk.attach.allowAttachSelf=true, -Djava.security.policy=/home/cloud-user/chaitra-opensearch/opensearch-1.2.3/plugins/opensearch-performance-analyzer/pa_config/opensearch_security.policy, -XX:MaxDirectMemorySize=536870912, -Dopensearch.path.home=/home/cloud-user/chaitra-opensearch/opensearch-1.2.3, -Dopensearch.path.conf=/home/cloud-user/chaitra-opensearch/opensearch-1.2.3/config, -Dopensearch.distribution.type=tar, -Dopensearch.bundled_jdk=true]

But when I set JAVA_HOME to my java installation directory like below, it picked up the java version which I want to use.
[opensearch-1.2.3]$ export JAVA_HOME=/etc/alternatives/jre_openjdk
[opensearch-1.2.3]$ ./bin/opensearch
OpenJDK 64-Bit Server VM warning: Option UseConcMarkSweepGC was deprecated in version 9.0 and will likely be removed in a futur
[2022-01-07T11:49:17,898][INFO ][o.o.n.Node               ] [chai-node] version[1.2.3], pid[16544], build[tar/8a529d77c7432bc453b2741b57d4a/2021-12-21T01:36:21.407473Z], OS[Linux/3.10.0-957.10.1.el7.x86_64/amd64], JVM[Red Hat, Inc./OpenJDK 64-Bit Server 11.0.13+8-LTS]
[2022-01-07T11:49:17,902][INFO ][o.o.n.Node               ] [chai-node] JVM home [/usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.8-1.el, using bundled JDK [false]
[2022-01-07T11:49:17,902][INFO ][o.o.n.Node               ] [chai-node] JVM arguments [-Xshare:auto, -Dopensearch.networkaddres=60, -Dopensearch.networkaddress.cache.negative.ttl=10, -XX:+AlwaysPreTouch, -Xss1m, -Djava.awt.headless=true, -Dfile.encoding=a.nosys=true, -XX:-OmitStackTraceInFastThrow, -Dio.netty.noUnsafe=true, -Dio.netty.noKeySetOptimization=true, -Dio.netty.recyclityPerThread=0, -Dio.netty.allocator.numDirectArenas=0, -Dlog4j.shutdownHookEnabled=false, -Dlog4j2.disable.jmx=true, -Djava.loers=SPI,COMPAT, -Xms1g, -Xmx1g, -XX:+UseConcMarkSweepGC, -XX:CMSInitiatingOccupancyFraction=75, -XX:+UseCMSInitiatingOccupancyO.io.tmpdir=/tmp/opensearch-3512813336349669626, -XX:+HeapDumpOnOutOfMemoryError, -XX:HeapDumpPath=data, -XX:ErrorFile=logs/hs_eg, -Xlog:gc*,gc+age=trace,safepoint:file=logs/gc.log:utctime,pid,tags:filecount=32,filesize=64m, -Dclk.tck=100, -Djdk.attach.allf=true, -Djava.security.policy=/home/cloud-user/chaitra-opensearch/opensearch-1.2.3/plugins/opensearch-performance-analyzer/pansearch_security.policy, -XX:MaxDirectMemorySize=536870912, -Dopensearch.path.home=/home/cloud-user/chaitra-opensearch/opensearDopensearch.path.conf=/home/cloud-user/chaitra-opensearch/opensearch-1.2.3/config, -Dopensearch.distribution.type=tar, -Dopensed_jdk=true]

Hence OPENSEARCH_JAVA_HOME doesn't seem to work.

OpenSearch doesn't use the environment variable OPENSEARCH_JAVA_HOME as mentioned on https://opensearch.org/docs/latest/opensearch/install/compatibility/. Closing this ticket, since we already have another issue opened to fix the documentation.

Should we turn this into a feature request? @chaitrahegde115 is there a reason why you'd want an ENV-set value vs. say replacing binaries in the distribution or consuming a no-JDK min version?

To be clear, OpenSearch does support specifying a JDK with an environment variable, it's just that the environment variable is JAVA_HOME... the documentation is wrong in this case.

@dblock @jcgraybill that is correct, but it could be useful for certain cases (migration fe), Elasticsearch has this feature https://github.com/elastic/elasticsearch/pull/68954.
+1 to turn it into feature request, I actually do have an example from the field where it is useful. There are often several JVM colocated apps / services running on the same host which use different JDKs (fe apps go on edge with JDK-17 while services are still conservative on JDK-8/11). In this case, setting JAVA_HOME would not work, but introducing OPENSEARCH_JAVA_HOME would actually make such deployments / migrations much simpler.

I'll reopen as a feature request.

@dblock if there are no objections, you could assign it to me, thx!

@chaitrahegde115 original issue was as follows
Describe the bug
I am using Opensearch-1.2.3 tar which ships Java 15 with it. But I want to use Java 11. So I referred Compatibility - OpenSearch documentation which stated to “set OPENSEARCH_JAVA_HOME environment variable to the Java install location” . I tried it but Opensearch installation used bundled jdk-15 but not jdk-11 which I want to use. Logs are as below.
[opensearch-1.2.3]$ export OPENSEARCH_JAVA_HOME=/etc/alternatives/jre_openjdk
[opensearch-1.2.3]$ ./bin/opensearch
[2022-01-07T11:47:12,157][INFO ][o.o.n.Node               ] [chai-node] version[1.2.3], pid[16189], build[tar/8a529d77c7432bc45b005ac1c4ba3b2741b57d4a/2021-12-21T01:36:21.407473Z], OS[Linux/3.10.0-957.10.1.el7.x86_64/amd64], JVM[AdoptOpenJDK/OpenJDK 64-Bit Server VM/15.0.1/15.0.1+9]
[2022-01-07T11:47:12,162][INFO ][o.o.n.Node               ] [chai-node] JVM home [/home/cloud-user/chaitra-opensearch/opensearch-1.2.3/jdk], using bundled JDK [true]
[2022-01-07T11:47:12,162][INFO ][o.o.n.Node               ] [chai-node] JVM arguments [-Xshare:auto, -Dopensearch.networkaddress.cache.ttl=60, -Dopensearch.networkaddress.cache.negative.ttl=10, -XX:+AlwaysPreTouch, -Xss1m, -Djava.awt.headless=true, -Dfile.encoding=UTF-8, -Djna.nosys=true, -XX:-OmitStackTraceInFastThrow, -XX:+ShowCodeDetailsInExceptionMessages, -Dio.netty.noUnsafe=true, -Dio.netty.noKeySetOptimization=true, -Dio.netty.recycler.maxCapacityPerThread=0, -Dio.netty.allocator.numDirectArenas=0, -Dlog4j.shutdownHookEnabled=false, -Dlog4j2.disable.jmx=true, -Djava.locale.providers=SPI,COMPAT, -Xms1g, -Xmx1g, -XX:+UseG1GC, -XX:G1ReservePercent=25, -XX:InitiatingHeapOccupancyPercent=30, -Djava.io.tmpdir=/tmp/opensearch-12173583538575190589, -XX:+HeapDumpOnOutOfMemoryError, -XX:HeapDumpPath=data, -XX:ErrorFile=logs/hs_err_pid%p.log, -Xlog:gc*,gc+age=trace,safepoint:file=logs/gc.log:utctime,pid,tags:filecount=32,filesize=64m, -Dclk.tck=100, -Djdk.attach.allowAttachSelf=true, -Djava.security.policy=/home/cloud-user/chaitra-opensearch/opensearch-1.2.3/plugins/opensearch-performance-analyzer/pa_config/opensearch_security.policy, -XX:MaxDirectMemorySize=536870912, -Dopensearch.path.home=/home/cloud-user/chaitra-opensearch/opensearch-1.2.3, -Dopensearch.path.conf=/home/cloud-user/chaitra-opensearch/opensearch-1.2.3/config, -Dopensearch.distribution.type=tar, -Dopensearch.bundled_jdk=true]

But when I set JAVA_HOME to my java installation directory like below, it picked up the java version which I want to use.
[opensearch-1.2.3]$ export JAVA_HOME=/etc/alternatives/jre_openjdk
[opensearch-1.2.3]$ ./bin/opensearch
OpenJDK 64-Bit Server VM warning: Option UseConcMarkSweepGC was deprecated in version 9.0 and will likely be removed in a futur
[2022-01-07T11:49:17,898][INFO ][o.o.n.Node               ] [chai-node] version[1.2.3], pid[16544], build[tar/8a529d77c7432bc453b2741b57d4a/2021-12-21T01:36:21.407473Z], OS[Linux/3.10.0-957.10.1.el7.x86_64/amd64], JVM[Red Hat, Inc./OpenJDK 64-Bit Server 11.0.13+8-LTS]
[2022-01-07T11:49:17,902][INFO ][o.o.n.Node               ] [chai-node] JVM home [/usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.8-1.el, using bundled JDK [false]
[2022-01-07T11:49:17,902][INFO ][o.o.n.Node               ] [chai-node] JVM arguments [-Xshare:auto, -Dopensearch.networkaddres=60, -Dopensearch.networkaddress.cache.negative.ttl=10, -XX:+AlwaysPreTouch, -Xss1m, -Djava.awt.headless=true, -Dfile.encoding=a.nosys=true, -XX:-OmitStackTraceInFastThrow, -Dio.netty.noUnsafe=true, -Dio.netty.noKeySetOptimization=true, -Dio.netty.recyclityPerThread=0, -Dio.netty.allocator.numDirectArenas=0, -Dlog4j.shutdownHookEnabled=false, -Dlog4j2.disable.jmx=true, -Djava.loers=SPI,COMPAT, -Xms1g, -Xmx1g, -XX:+UseConcMarkSweepGC, -XX:CMSInitiatingOccupancyFraction=75, -XX:+UseCMSInitiatingOccupancyO.io.tmpdir=/tmp/opensearch-3512813336349669626, -XX:+HeapDumpOnOutOfMemoryError, -XX:HeapDumpPath=data, -XX:ErrorFile=logs/hs_eg, -Xlog:gc*,gc+age=trace,safepoint:file=logs/gc.log:utctime,pid,tags:filecount=32,filesize=64m, -Dclk.tck=100, -Djdk.attach.allf=true, -Djava.security.policy=/home/cloud-user/chaitra-opensearch/opensearch-1.2.3/plugins/opensearch-performance-analyzer/pansearch_security.policy, -XX:MaxDirectMemorySize=536870912, -Dopensearch.path.home=/home/cloud-user/chaitra-opensearch/opensearDopensearch.path.conf=/home/cloud-user/chaitra-opensearch/opensearch-1.2.3/config, -Dopensearch.distribution.type=tar, -Dopensed_jdk=true]

Hence OPENSEARCH_JAVA_HOME doesn't seem to work.

@dblock could you please add this one as well https://github.com/opensearch-project/performance-analyzer-rca/issues/127

@dblock I think we are done with it?


First example on GitHub project main page is not working

Did cargo init winapitest

Did copy&paste code from this project´s main page (Cargo.toml and main.rs)

Did cargo run or cargo build

Output:

error[E0277]: the `?` operator can only be applied to values that implement `Try`
  --> src\main.rs:16:9
   |
16 |         SetEvent(event)?;
   |         ^^^^^^^^^^^^^^^^ the `?` operator cannot be applied to type `BOOL`
   |
   = help: the trait `Try` is not implemented for `BOOL`

error[E0277]: the `?` operator can only be applied to values that implement `Try`
  --> src\main.rs:17:9
   |
17 |         WaitForSingleObject(event, 0)?;
   |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the `?` operator cannot be applied to type `WIN32_ERROR`
   |
   = help: the trait `Try` is not implemented for `WIN32_ERROR`

error[E0277]: the `?` operator can only be applied to values that implement `Try`
  --> src\main.rs:18:9
   |
18 |         CloseHandle(event)?;
   |         ^^^^^^^^^^^^^^^^^^^ the `?` operator cannot be applied to type `BOOL`
   |
   = help: the trait `Try` is not implemented for `BOOL`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `winapitest` due to 3 previous errors

Assumption: Result<()> is a type alias for Result<(), Error> whereas some stuff inside the used methods not fullfills the Error trait. Since i´m a bit in hurry at the moment, i had no time to check it. But nonetheless imo a simple example on the main page should work out-of-the-box.

The GitHub repo is always primed for the next release. You can for example look at the readme for the current release here:
https://github.com/microsoft/windows-rs/tree/0.48.0
Alternatively, you can find the readme for the current release here:
https://crates.io/crates/windows

Hey @kennykerr,
thx for your fast reply! But i don’t really understand what you wanna say, tbh. The example on the frontpage combines example code and version into a single block which should match), since the version 0.4.8 is declared in the .toml file, which is part of the example. So the example‘s source code should match the crate‘s version, ofc.
Maybe you can further explain what you try to say, or what should best the reason why the example not works.
Best regards

It's a little confusing I understand. The samples in the readme are checked by CI against the latest in github but that isn't able to validate the toml code. That gets bumped when I prep for the next release and update all the version numbers.
Anyway, if you need an example to get started go here:

https://crates.io/crates/windows

If you need more samples go here:

https://github.com/microsoft/windows-rs/tree/0.48.0/crates/samples

For general questions about Rust go here:

https://stackoverflow.com/search?q=windows-rs

It's a little confusing I understand. The samples in the readme are checked by CI against the latest in github but that isn't able to validate the toml code. That gets bumped when I prep for the next release and update all the version numbers.
Anyway, if you need an example to get started go here:

https://crates.io/crates/windows

If you need more samples go here:

https://github.com/microsoft/windows-rs/tree/0.48.0/crates/samples

For general questions about Rust go here:

https://stackoverflow.com/search?q=windows-rs

Maybe we have some miscommunication here. 😄 I totally understand that the examples in the READMEs, and the READMEs itself, are released together and fit each other.
But what i try to tell you: Your example code is not working.
The example in https://crates.io/crates/windows or the frontpage (it’s the same code and both target 0.4.8) is simply not working.
Steps to produce:

Go to https://crates.io/crates/windows
Copy first example
Run
Error

Best regards

Hey @MBODM. The GitHub Readme and the crates.io Readme are not the same. I've verified the crates.io Readme examples work. Maybe you mixed code between the two by mistake? (The GitHub Readme is meant to be used against master, not 0.48.)
Sorry for the confusion; we're currently working to improve the Readme situation.

Does this matter even at all, when i use a crates.io example code, together with the crate that is listed on crates.io ?
I mean, i use

example code of crates.io site
corresponding crate from crates.io site (cargo command of right side menu)
example code says 0.4.8
actual cargo version used is 0.4.8 (cargo command of right side menu on crates.io)

and it does not work.
I understand what you said about master branch and example code on GitHub page. But when i go to crates.io, use the example code there, together with the given crate install command on right side menu…. should it not work then?

@MBODM Can you provide your main.rs and Cargo.toml so we can get a better idea of what's going on? Thanks!

Btw, You say GitHub‘s example and example of crate.io differs.

Where? Am i blind?

Here's the GitHub Readme before today's changes:
https://github.com/microsoft/windows-rs/blob/97efa322720a1aa84bbd437bfc84c3c984e978dc/docs/readme.md
Here's the crates.io Readme:
https://crates.io/crates/windows
Here's the diff before today's changes:

Here's the GitHub Readme after today's changes (https://github.com/microsoft/windows-rs/pull/2538):
https://github.com/microsoft/windows-rs/blob/master/docs/readme.md
Here's the crates.io Readme:
https://crates.io/crates/windows
Here's the diff with today's changes:

As you can see, in both scenarios, the Readmes do not match. The current changes between the two Readmes are not breaking.
If you provide your Cargo.toml and main.rs, we can verify you have the correct code in place. Hopefully moving forward (with today's changes) this will be less confusing!

As we already learned in Breaking Bad: „All bad things must come to an end.“
This seems what happened:

I copied the very first example code yesterday from the GitHub page. (That code had NO ok entries.
I ran the example and it failed.
I did the conversation with @kennykerr here, see above.
@kennykerr changed the GitHub page Readme file, to make it the same as at crates.io, without stating that fact here.
AFTER that happened i compared the crates.io and Readme file to be sure, what i had tested is the same on both. Not realizing that the code i am using is no longer the same code as 8 hours before.
I just made sure GitHub code and crates.io code is the same. And at that point in time (after @kennykerr‘s changes) it indeed was. Ofc. But ofc i assumed the code i had at that time, was the same as on crates.io, since i checked both and they were same (without realizing @kennykerr changed it). That’s why i said i use the code from crates.io. Sinve crates.io and GitHub was completely the same (at THAT time, but not 10 hours before, when i got it).

That’s what happened and why the confusion started to grow, foremost for @riverar
So, you can close the issue. The new ok() version works now (ofc) and all is fine.
But let me state one last thing here, especially targeted at @kennykerr:
After 22 years of being a developer i came to the conclusion, It is NOT funny to add silent Ninja changes to the code (commit https://github.com/microsoft/windows-rs/commit/4c88d904e76125cf9576c240b283e87de531981f), without stating that while we are discussing. It just confuses peoples like me or @riverar
But nonetheless a happy end. Thx for your patience and best wishes!

Happy coding and let us know if you run into other issues or if anything else is confusing!

Apologies it wasn't clear enough. I specifically tagged this issue in #2538 so that it would show up in this timeline:

Anyway, glad you figured it out!

Great! Thx a lot!


encoding a lacuna that stretches until the end of a text

@ajaniak — how should Aditia encode the fact that in Siksa Kandang Karesian, the witness A breaks off a few paragraphs before the end of the text, to avoid an empty § as in this screenshot? Should he create an <app> at the very end of the text edition for no opther reason than to be able to encode the corresponding <lacunaEnd>? Could we perhaps add some attribute to such cases of <lacunaStart>? (But are any attributes allowed on this element?)

I am not exactly sure... because in theory all  shoud end with a .
If we add a final app such as <app><lem wit="#EdASD #B">palipurna.</lem><rdg wit="#A"><lacunaEnd/></rdg></app>at the end of the closing chapter.

We will resolve the app entry 1061, but it will generate an unsatisfactory entry for this  note.

But we could trick this solution adding an attribute @rend="hide" on the last app.
<app rend="hide><lem wit="#EdASD #B">palipurna.</lem><rdg wit="#A"><lacunaEnd/></rdg></app>
so that it is not display, but the anchoring for the  in made and so the display expected for the note 1061 still works.

I kind of like this solution since it keeps the logic of our model, and we achieve the display without adding anything new nor special.
I also though of adding a fallback the add an anchor for a <lacunaStart/> at the end of the text, but I am afraid it will cause issue and lack the abilty to adapt to different context.

@ajaniak — I entirely agree with your proposal. Can you use the elements of your response to make the necessary additions to EGC to address this kind of scenario?
@aditiagunawan — Please make the necessary changes in your xml file(s) and respond to this comment when you are done. We will wait for Axelle to be done with the EGC before closing this issue.

@ajaniak
Done!
Thank you
Aditia

Thnaks @aditiagunawan I have just corrected the double final dot.
A draft have been added in the encoding guide, so I am closing this


B #3556: Fix IP spoofing filters

Signed-off-by: Venko Moyankov<EMAIL_ADDRESS>

Thanks!. Yes this is in fact the approach used in the other rules


error on execution

Hi,
Thanks for your awesome library.
I have executed this line on RPI 4 (buster 64bit-debian 10) after downloading a model:
echo 'Welcome to the world of speech synthesis!' |
./larynx --model blizzard_lessac-medium.onnx --output_file welcome.wav
but there is a error relating to GLIBC as follows:
./larynx: /lib/aarch64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by /home/pi/speech/larynx/larynx/libespeak-ng.so.1)
I tried to upgrade my GLIBC from 2.28 to 2.29, but I could not find this version in this website: https://packages.debian.org/buster/
How can I solve the issue?
Best,

Larynx 2 is built on Debian bullseye, which has a newer GLIBC. You will need to build from source on buster, which I found to be a major pain because gcc was out of date (no C++20).
If any C++ wizard could help out here, it would be much appreciated.

FYI, I re-built the Larynx binaries on an older Linux distro with GLIBC 2.20. Please re-download and try again.


Failing to download 9anime links with 2captcha key

The following errors appear when trying to download from 9anime.to.

Revisiting the website, and restarting the downloader fails to fix the issue.

To download from 9anime, you need to solve recaptcha for 9anime url manually and provide the captcha answer token in the given text area to download. There is a guide on how to do this in the README file's FAQ section.
Please refer the README before posting such issues

Isn't that only applicable without the 2captcha API key (which I have)?

Are you sure you have enough 2capthca credits?
Have you been downloading from 9anime before successfully (with the key)?
How many times did you retry after failing?

Also, please title and describe the issue in detail, like I re-opened the issue since now you mentioned you have the 2captcha key; otherwise I thought this was invalid

I do indeed have enough 2captcha credits.
Unfortunately, I only found this program last night and have had no success with 9anime as of yet.
I've tried numerous times using both the subbed links and dubbed links, as well as different animes and multiple different settings.

Seems there's a problem indeed. I myself haven't tried 9anime for a while for downloading and still I was using it with the captcha answer manually.
I'm currently working on the new version v2.0.0 and I will definitely look into this within a day and let you know of the results.
For the meantime, try giving captcha answer manually and most probably it will work.
Also on another note, since you're downloading from 9anime, use twist.moe (if they have the anime you're looking for) since their file sizes are almost same as 9anime's but with higher bit rate. So it's better quality

@connorlawd https://github.com/Zebraslive/Mass-Anime-Downloader


Background not fully filled on KitKat

Not sure if it's a bug in the lib or my code. It works fine (of course) on Lollipop. On KitKat I get the attached screenshot. Here is my code (custom ViewGroup so no XML layout). Also, the theme is derived from Holo and I am not used appcompat activities or fragment
        mSelfieFab = new FloatingActionButton(context);
        mSelfieFab.setImageDrawable(
                getResources().getDrawable(R.drawable.sidenav_icon_photo_finish));
        mSelfieFab.setType(FloatingActionButton.TYPE_MINI);
        mSelfieFab.setColorNormal(getResources().getColor(R.color.basis_red));
        mSelfieFab.setColorPressed(getResources().getColor(R.color.basis_red_pressed));

It turns out to be a bug in my layout code. I assumed that the size of the button would always be 56dp (normal) or 40dp. However, in Kitkat, the actual size is bigger because of the shadow added when implementing the elevation.


Show state wise testing data

Great work man! Looks really good!
I have fixed the deployment previews. So you can check them using the URL above. The netlify.app domain was not white-listed. Seems like they recently switched to .app instead of .com.
See if we can add some colors, though even without them this looks good!


Take impl AsFd instead of BorrowedFd

This is more general, in particular the following will still compile.
let fd = something_implementing_as_fd;
let fd: &std::os::fd::BorrowedFd<'_> = &fd.as_fd();
ashpd::any_method_using_a_fd(fd).await?;

Something annoying is that with this you cannot just use HashMap::new  when one of generics is impl AsFd, see the docs change.

Needs a rebase

Done.


DRES 2.0.0 RC3 cannot log when more than one evaluation is running

If more than one evaluation is active (i.e., with a running task) the query-log and result-log is not working. It returns 409 (conflict). You can currently test this with vbs.videobrowsing.org, where we have two evaluations running.

Logging across multiple evaluations with overlapping participants is an edge-case not currently supported. Would this be needed for VBS'24?

It is not VERY important for VBS 2024, but still it would be good to fix this soon, because even now we use several evaluations at the same time to test different task types.

Ok, we'll see what we can do. This will necessitate an API change, though.

Thank you! The API change should not be a big issue, I hope.

This has been addressed in RC4


Adding Portuguese Steak with Beer Sauce recipe

Could you remove salt and pepper as ingredients?
I haven't been including them because it'd just be two lines in literally every page. I'd like that to be the standard.

Yes, will do!


Missing directive_defaults from Cython.Compiler.Options?

I am using directive_defaults in my packages and I suddenly get an import error. I could not find anything in the changelogs, in fact, Cython uses this import line too in its setup.py. Does anyone have a clue what's going on here? I tried the latest release (0.25) in a fresh virtualenv with Python 2.7.12 and 3.5.2.
In [2]: from Cython.Compiler.Options import directive_defaults
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-2-43ee860b87f9> in <module>()
----> 1 from Cython.Compiler.Options import directive_defaults

ImportError: cannot import name directive_defaults

Sorry, this should have been in the release notes. Now you must call
Cython.Compiler.Options.get_directive_defaults()
On Wed, Oct 26, 2016 at 12:56 AM, Tamas Gal<EMAIL_ADDRESS>wrote:

I am using directive_defaults in my packages and I suddenly get an import
error. I could not find anything in the changelogs, in fact, Cython uses
this import line too in its setup.py. Does anyone have a clue what's going
on here? I tried the latest release (0.25) in a fresh virtualenv with
Python 2.7.12 and 3.5.2.
In [2]: from Cython.Compiler.Options import directive_defaults
ImportError                               Traceback (most recent call last)
 in ()
----> 1 from Cython.Compiler.Options import directive_defaults
ImportError: cannot import name directive_defaults
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cython/cython/issues/1497, or mute the thread
https://github.com/notifications/unsubscribe-auth/ATqe4pG7C05j1ra-c2hvOEQFM4oFm46xks5q3wekgaJpZM4Kg3TC
.
--
You received this message because you are subscribed to the Google Groups
"cython-github-notifications" group.
To unsubscribe from this group and stop receiving emails from it, send an
email to<EMAIL_ADDRESS>To post to this group, send email to cython-github-notifications@
googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/
msgid/cython-github-notifications/cython/cython/issues/1497%40github.com
https://groups.google.com/d/msgid/cython-github-notifications/cython/cython/issues/1497%40github.com?utm_medium=email&utm_source=footer
.
For more options, visit https://groups.google.com/d/optout.

Many thanks!

Hey, how do I get the directive_defaults attribute into Cython.Compiler.Options using Command Prompt for Windows?


Support for Guzzle / generic promise adaption

Guzzle promises implement A+ promises but are no react promises. How about supporting them as well or more generic all vendor-promises that implement "then()"? Instead of $result instanceof ReactPromise you would write e.g. isVendorPromise($result). adapt() handles any object that implements done() or then() anyway.
Any opinions?

Not everything having a then() or done() method will be a promise, which might be problematic if such instances are returned from a coroutine.
For Guzzle specifically, interoperability with its promise library would be easy to add. However, interoperability with Guzzle itself isn't really sanely possible.

Of course I meant Guzzle promises, not Guzzle as a whole
It depends on how you interpret the possible yielded values. Maybe I missed something but currently nothing but promises or arrays of promises are supported anyway. So why is it wrong to assume that anything that comes from a yield (within a coroutine) should be treated as a promise. This of course is a philosophic decision between strict typing and duck-typing principles

I fully agree that it's a philosophic decision for yield. I'm rather worried about the return case, that also supports returning a promise just as yield, where instances that aren't promises won't error and instances that are promises are subscribed to, so the coroutine is only resolved once the returned promise resolves.

Generic solution:

define interface PromiseAdapter
provide registry for PromiseAdapters
PormiseAdapters::adaptVendorPromise() checks available adapters instead of instanceof ReactPromise + adapts in case of a match
Check instanceof \Amp\Promise for final validity check

That creates a generic, type safe interface for promise adaption + it decouples adaption from actual adapter implementation. It also makes it extensible for further custom adapters.

What do you think about that? If you basically agree on that idea, we could provide a PR for further review.

If we added such a registry, it would need to be used not only in coroutines, but anywhere that we currently allow React promises. Mostly this would be fine, the only place I'd be concerned about checking for vendor promises is where @kelunik alluded to: resolving promises with adaptable promises, this line in particular. It could be changed to \is_object($value) && PromiseAdaptors::isVendorPromise($value). I don't think the impact of that would be significant enough to be a blocker here.
That being said, I wonder if it's worth baking support for further adaptations for something that seems rather niche, which already has a viable solution. You can use Promise\adapt() with Guzzle promises or any other A+ promise. As far as I'm aware, only Guzzle itself uses that promise library, through an API that doesn't play nice with an event loop. Is this true? Are there other promises that can be adapted?

@brstgt What if two libraries define a different adapter for a specific implementation? Or do you plan that only applications make use of this new feature?
I wonder whether it's worth, given the problems above. But it made me thinking that we should maybe forbid returning promises from coroutines?

Of course we had to change every occurence.
Even if react + guzzle are the only candidates at the moment it is IMHO better to have an approach that is not that tightly coupled / dependent on vendor code or if so not in so many places.
I personally don't like to have instanceof ReactPromise scattered over roughly 20 places.
@kelunik To avoid clashes, libraries must not implement adapters on their own. Instead they could depend on an adapter package.
If all that is too bloated, it would still be helpful to gather all the adaption code in one place and implement e.g. \Amp\Promise\isAdaptable() that does the instance checks and maybe support Guzzle as well.
Guzzle Promises:
We need them for async AWS client. See here: https://github.com/noerosell/awsyncronic.
Of course there are workarounds for everything but I thought the cleanest thing would be to have a generic way to support any kind of promises.

@kelunik Removing support to return promises from coroutines probably breaks a lot of existing code. But I agree that it can be really confusing to allow it.

I don't like the scattering either, but for one implementation it seemed acceptable. Having libraries depend on an adapter package instead of including their own adapter just moves the problem, no?
When will this be called?

@kelunik I am not really sure when this will be called. I didn't deal with that lib, yet, that's up to @bzikarsky - my co-worker

@kelunik AwsLoopMiddleware::runGuzzleQueue has to be called once by the application, the naming is not perfect. Middleware::tap registers a synchroneous callback whenever an AWS command/request gets queued into Guzzle. This is required to make sure Guzzle starts all requests and invokes the custom handler in AwsHttpHandler.

@brstgt It's not a problem, but at that point we can just build it into Amp itself instead of having separate packages, just like we do now for ReactPHP's promise library.
@bzikarsky As far as I can tell you need another runGuzzleQueue() call in a Loop::defer() after that then to invoke the callbacks registered on the promise returned from the HTTP handler?

@kelunik Actually I am totally open to any solution that helps. My 2 cents would just be:

generic approach, no scattered vendor-related code
if we change it anyway, we can make it extensible
non-baked-in extensions could be managed by packages

Now it's up to you:

do you want to support guzzle promises only
do you support an extensible approach
do you want us to create a PR or do it on your own
you don't like the concept at all and ask people to use Promise\adapt() manually in application code with vendor-promises

I'd love to see a generic promise adapting approach, compatible with anything we'll do in future. The individual adapters then shall be separate from this project.

@bwoebi We already have a generic adapting approach via Amp\Promise\adapt(). If we keep that separate (not integrated), then there's no reason for the adaption to be generic.

I too was never thrilled about having instanceof ReactPromise littered throughout the code, a generic approach seems much better, so 👍.
@brstgt If you want to provide a PR, please do.

Umm @kelunik this is not really generic, doing instanceof ReactPromise there. And obviously I mean internal adapting, so that you can just yield these (like the React Promises).

@bwoebi There's no instanceof ReactPromise inside adapt().
I'd like an approach that's open to future extension but doesn't allow users to register their own adapters, because changes to the supported yielded objects affect all coroutines in all packages.
Another point I just noticed we must handle: Currently neither ReactPHP nor Guzzle enforce \Throwable as rejection reason. Guzzle simply accepts and documents mixed. Currently that will result in a TypeError when calling Deferred::fail(). As Guzzle doesn't provide any done() method, we'll call then(). The promise implementation will invoke the callback, which results in the TypeError and the error will go unnoticed, because it just fails the promise returned from then().

I'm not talking about adapt(). I'm talking about the locations calling adapt() internally (e.g. in coroutines). It forces the user to call adapt() himself.
Also, allowing users to register their own adapters is no problem, as long as you register individually per class. And if you register something which is nothing promisy, that's your own fault. Nothing we shall do there.

For what it's worth: react does a similar thing as Amp\adapt inside the React\resolve function:
https://github.com/reactphp/promise/blob/0845d291435b862f9a22bb5971ba18f50a00da09/src/functions.php#L24-L43
React supports guzzle promises, since they implement a then function:
https://github.com/guzzle/promises/blob/60d379c243457e073cff02bc323a2a86cb355631/src/Promise.php#L31-L53
Both react's and guzzle's then function return another promise.
Meaning this issue can be resolved by changing the the Amp\adapt() function to support the then method returning another promise-like object.
Currently the Amp\adapt function knows the promise is resolved, but doesn't pass through the return value:
https://github.com/amphp/amp/blob/eb2f325586bc6ebb12d27834fc779fa140c38a57/lib/functions.php#L336-L338
This could be changed to something like:
if (\method_exists($promise, 'then')) {
    $promise->then(
        function ($result) use ($deferred): void {
            $deferred->resolve($result);
        },
        function ($failure) {
            $deferred->fail($failure instanceof \Exception ? $failure : new SomeException);
        }
    );
}

(we could add some safeties as you can see inside the guzzle and react codesnipptes I linked to above)
It would also open up amp to be able to pass around non-async promises (e.g. Success or Failed promise), like these:
https://github.com/azjezz/psl/blob/develop/src/Psl/Result/ResultInterface.php#L91
More info: https://github.com/azjezz/psl/pull/65#discussion_r498151955, https://github.com/azjezz/psl/issues/66
Would you be interested in a PR that contains the fix as suggested above?

Other than the check for a non-exception on failure, your code snippet doesn't do anything different, particularly with a return value as you seem to suggest.
If the Amp promise is being resolved with null, then that's what the adapted promise is passing to the first callback given to then().

Woops, looks like I was confused. It indeed works as expected now that I use onResolve(fn ($exception, $value) => ..) instead of setting the $value as first argument. ... Thanks!

We don't plan to support this for now.


Uncaught TypeError: Cannot read property 'style' of null

[Enter steps to reproduce below:]

Removed a project folder;
Clicked the Atom window

Atom Version: 1.2.4
System: Ubuntu 15.04
Thrown From: atom-material-ui package, v1.0.1
Stack Trace
Uncaught TypeError: Cannot read property 'style' of null
At /home/awc/.atom/packages/atom-material-ui/lib/bindings.coffee:33

TypeError: Cannot read property 'style' of null
    at HTMLDivElement.<anonymous> (/home/awc/.atom/packages/atom-material-ui/lib/bindings.coffee:33:24)

Commands
Throws an error when the project folder is removed and the window is clicked again.
Config
{
  "core": {
    "ignoredNames": [
      "*.class"
    ],
    "disabledPackages": [
      "linter-hlint",
      "minimap-git-diff",
      "linter-pep8",
      "linter-tidy",
      "linter-javac",
      "js-refactor",
      "linter-jsonlint",
      "omnisharp-atom",
      "linter-pep257",
      "linter-pylama"
    ],
    "themes": [
      "atom-material-ui",
      "cinzo-syntax"
    ],
    "autoHideMenuBar": true
  },
  "atom-material-ui": {
    "fonts": {},
    "colors": {
      "abaseColor": {
        "red": 86,
        "green": 131,
        "blue": 127,
        "alpha": 1
      }
    },
    "ui": {},
    "tabs": {},
    "treeView": {
      "compactList": true
    }
  }
}

Installed Packages
# User
.git, vundefined
atom-alignment, v0.12.1
atom-html-preview, v0.1.13
atom-material-ui, v1.0.1
atom-slime, v1.7.2
atom-ternjs, v0.10.3
atom-yeoman, v0.3.15
autocomplete-bibtex, v0.6.1
autocomplete-clang, v0.8.8
autocomplete-paths, v1.0.2
autocomplete-python, v1.0.2
autocomplete-ruby, v0.1.0
cinzo-syntax, v0.2.0
color-picker, v2.0.14
file-icons, v1.6.13
haskell-ghc-mod, v1.1.6
hidpi, v0.7.0
highlight-line, v0.11.0
javascript-snippets, v1.0.0
json-schema, v0.1.15
language-haskell, v1.5.1
language-latex, v0.6.1
language-lisp, v0.1.2
latex, v0.28.2
latexer, v0.2.7
linter, v1.11.3
linter-bootlint, v0.0.6
linter-chktex, v1.2.3
linter-clang, v3.3.0
linter-clojure, v1.1.1
linter-cpplint, v1.0.0
linter-csslint, v1.1.0
linter-htmlhint, v0.2.1
linter-jscs, v3.2.0
linter-jshint, v1.2.1
linter-less, v2.3.1
linter-pylint, v1.2.0
linter-ruby, v1.2.0
linter-shellcheck, v1.0.2
lisp-paredit, v0.5.1
minimap, v4.16.2
minimap-color-highlight, v4.1.4
minimap-find-and-replace, v4.3.2
node-debugger, v1.3.0
package-cop, v0.2.10
ruby-block, v0.3.5
tabs-to-spaces, v1.0.1

# Dev
No dev packages

I believe this happens when you open a window without a tree-view. It's been fixed in the development version. 1.0.x was unpublished and 0.8.0 is the latest version once more. If you wish to downgrade just reinstall the theme:
apm uninstall atom-material-ui
apm install atom-material-ui

However, if you wish to continue using 1.0.x, clone this repository and link it with APM.
apm uninstall atom-material-ui
git clone git@github.com:silvestreh/atom-material-ui.git
cd atom-material-ui
apm link
apm install

Thanks!

All good, thanks!


Support per directly local settings

Following the pattern of rbenv and jenv it would be awfully convenient if gvm had the ability to store local configuration options that take precedence over the global ones on a per project basis.

+1 for .rvm like commands. Because the point of GVM and the like is to manage multiple version of development tools, doing so transparently seems like the next step in evolution. The alternative is to assume the developer manually switch to get their project to work, which if thats mandatory, why do we have to wait for a compile or other exception to notify us of what we should have done, when such change could have been done on our behalf if a particular file (e.g. .gvm-versions) exists there, otherwise using the default. Such I have found useful in the ruby world,  though I suppose a case can be made for the existing alternative, Im just not sure of technical benefits anymore than I'm privy to the alternative ramifications, etc. Just the same, thanks for the great work!

Not sure, but are you confusing us with the Go Version Manager?
This is the SDKMAN, formerly GVM, the Groovy enVironment Manager.

Forgive the confusion, I'm using GVM (now sdkman) for grails development primarily, though it supports multiple sdks and desired such support generally speaking. I think I may have referenced this in the go version manager issue, but admittedly didn't realize it was different. My mistake :)

No prob, you are in the right place then.
The following issue was raised a few days ago which relates directly to this:
https://github.com/sdkman/sdkman-extensions/issues/9


Updated yellowOrangePurple scale and removed lightness correction

Updated yellowOrangePurple scale with slightly different colors
Updated yellowOrangePurple scale to have exponential shift applied to colors
Removed lightness correction in domainArray since this should be done when creating a color scale to use within visualizatoins and not in the consumption of the color scale itself.
Removed adjustTanh function from data (transformation should be done to color scale and not to data itself -- exponential shift on scale accomplishes this)
Made yellowOrangePurple the default color scale as it provides a greater range of intensity levels for data to be visualized.

Love the idea but needs a little cleanup; mainly some unused variables.npm run lint -- --fix will autofix a couple things for you, but you need to manually delete the unused vars.

@camjc I've removed unused vars & functions, however linting displays 204 warnings with files I have not made any modifications to on this branch/PR (primarily in the events/ directory)


FR: Migrate to Apollo Server v4

Apollo Server v4 has been released in Nov, 2022.
Migration guide:

https://www.apollographql.com/docs/apollo-server/migration/
https://www.apollographql.com/docs/apollo-server/integrations/integration-index/
https://www.apollographql.com/docs/apollo-server/integrations/building-integrations/

#10 - https://github.com/kimyvgy/apollo-server-integration-cloudflare-workers


Automated Frontend Tests

AFAIK, we currently don't run our Jasmine tests anymore, because there was in issue with Travis #632. A nice way of running frontend tests is to use headless chrome and chromeless. A sample test can be seen here. The nice thing would be that we are able to fully test our frontend and make sure that the frontend works with the latest GH changes.
We could spin up an actual GH instance and then open our front end on headless chrome (go to localhost:8989). Enter start and end coordinates and see if the route calculation works as expected. Chrome headless seems to be supported on Travis.
I think this sounds like a very usable addition to GH and I could try to setup a proof of concept to show that everything works as expected?

Not sure if this is worth the effort at the moment as there is no deep development happening there and manually checking this is sufficient and simple enough.

I had the same thought. I am not sure if it's worth the overhead or not. I think this will give us a good overview of the different functions of the frontend and make sure that we don't break anything and that the frontend is always up to date with the backend.
My long-term dream would be also to further improve GH maps to also use our js client, maybe remove jQuery, etc. It would be good to have this setup before changing anything, so that we can make sure that we don't introduce a regression.


Similar bug in polynomial function

Also here, the str is trying to be called.

:white_check_mark: Build SMT 0.3.550 completed (commit https://github.com/SMTorg/SMT/commit/87638bd8cf by @jbussemaker)

Coverage remained the same at 77.615% when pulling a05065820c0d85e59db5b6302867558e7203e0ca on jbussemaker:jbussemaker-patch-1 into c00b9185fd5936eee8cb95159cfb76e4b8a55f78 on SMTorg:master.

:white_check_mark: Build SMT 0.3.552 completed (commit https://github.com/SMTorg/SMT/commit/3be84c0c22 by @relf)


implement removing headers

this is done https://github.com/google/tamperchrome/commit/c7afde44d82a6fc1d5db0e2b2cdd02c8a346cd37


Remove legacy parenthesized label parsing; Remove redundant PnLabel parse tree node and handling

This PR includes two changes:

Remove legacy parenthesized label parsing, follow up to #2115
Simplify label parsing by removing the label parse tree node type and corresponding bifurcation on the buildAST mode

In the latter change, use the NoAST label tracking logic for both NoAST and buildAST modes.  The label parse tree nodes were unused after parsing, so no need to create and keep them.
This is a post v1.4 change.

Looks good :)

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

@ianwjhalliday if you have a minute, could you make the bot happy?


2351 CLI sets exit-code 1 if the command supplied was not parseable

See issue #2351.
Sorry, I haven't found tests for the bin/cli.js file.
Reference to a stack exchange question, that is somewhat related ☺.

Sorry, I haven't found tests for the bin/cli.js file.

There hasn't been tests for cli.js so far. Someone has to be the first one to write them 😅


ioBroker startup does not work after raspberry restart

Hello!
After a power failure I needed to restart my router and raspberry pi with ioBroker installed on it.
Unfortunately, I was not able to start ioBroker again. In Docker everything seems to be healthy but the logs show the following
------------------------------------------------------------
-----          Step 5 of 5: ioBroker startup           -----
------------------------------------------------------------

Starting ioBroker...

Recived termination signal (SIGTERM).
Shutting down ioBroker...

I tried to recreate the container (with portainer) and used the latest image which worked already before the restart. After using docker exec -it iobroker bash I tried to find some logs and in the /log/iobroker.current.log file I found the following:
2021-04-17 21:08:22.323  - info: host.iobroker iobroker.js-controller version 3.2.16 js-controller starting
2021-04-17 21:08:22.329  - info: host.iobroker Copyright (c) 2014-2021 bluefox, 2014 hobbyquaker
2021-04-17 21:08:22.330  - info: host.iobroker hostname: iobroker, node: v12.22.1
2021-04-17 21:08:22.330  - info: host.iobroker ip addresses: <IP_ADDRESS> fd00::49e3:736d:24d1:9550 2a02:908:4b13:dc80:7ca:4ab1:de69:46a7 fe80::1273:d624:88b2:f7c1 <IP_ADDRESS> fe80::42:a1ff:fe7a:79ae <IP_ADDRESS> fe80::70c8:4bff:fe80:8780 <IP_ADDRESS> fe80::54be:17ff:fe9d:7e55
2021-04-17 21:08:22.431  - error: host.iobroker Cannot read system.config: null (OK when migrating or restoring)
2021-04-17 21:08:22.578  - info: host.iobroker connected to Objects and States
2021-04-17 21:08:22.610  - info: host.iobroker added notifications configuration of host
2021-04-17 21:08:22.649  - error: host.iobroker Cannot find view "system" for search "instance"
2021-04-17 21:08:22.650  - error: host.iobroker Could not add notifications config of this host: Could not get notifications setup from instances: Cannot find view "system"
2021-04-17 21:08:22.718  - info: host.iobroker Plugin sentry Sentry Plugin disabled for this process because sending of statistic data is disabled for the system
2021-04-17 21:08:22.725  - error: host.iobroker Cannot find view "system" for search "host"
2021-04-17 21:08:22.737  - error: host.iobroker Cannot find view "system" for search "state"
2021-04-17 21:08:22.738  - error: host.iobroker Cannot find view "system" for search "instance"
2021-04-17 21:08:22.739  - error: host.iobroker Could not collect system.host.iobroker states to check for obsolete states: Error: Cannot find view "system"
2021-04-17 21:08:22.740  - error: host.iobroker _design/system missing - call node iobroker.js setup

On the host system I didn't change anything and of course I have a volume on the host which is mapped to the container path /opt/iobroker on which I didn't change anything either.
I'm not understanding what happened since I'm using your image for a long while with many raspberry restarts.
I would appreciate any help and can provide more information at any time.
Thanks in advance!
Mike

I have a few updates and managed to fix it, but maybe my findings will help you for further development or other people with the same issue.
The solution to fix the issue was taken from https://<IP_ADDRESS>/topic/42816/gel%C3%B6st-iobroker-nach-stromausfall-nicht-mehr-erreichbar/29 but the iobroker forum is currently in maintenance, therefore I used the Google cache with the mentioned link.
The trick was to override the objects.json and states.json in /opt/iobroker/iobroker-data/ with older ones from /opt/iobroker/iobroker-data/backup-objects/.
After overriding those files and restarting the docker container I found the following in the logs:
hash changed or no sources cached => force download of new sources
I remembered that something similar has been stated in the logs before I recreated the container, as mentioned in the issue description. Therefore, I thought about the following...
The raspberry restart happened after a power failure and therefore my router needed to restart as well. I guess that ioBroker tried to download something without internet connection after the router restart/power failure. Maybe that lead to corrupted states.json and objects.json files.
I'm not sure if that is really the case, but maybe it will help you in the future to keep that case in mind for reliable restarts.
Nevertheless, thanks for this amazing project!

Hi Mike,
my solution would have been to deploy a new container and restore your last backup. In some rare cases it could happen that you get a corrupt file or something after a power failure. So its always a good thing to have an actual backup. :)
Thank you for sharing your solution so others can find it when googling.
Regards,
André


unix: squelch -Wcast-function-type warning

CI: https://ci.nodejs.org/job/libuv-test-commit/1434/

AFAIK, using union to accomplish what C++'s reinterpret_cast does will eventually end up with undefined behaviour when entry() gets call, just like the current casting.
How about a more unintrusive way to get around the warning using test macro + GCC's pragmas?
#if defined __GNUC__ && __GNUC__ >= 8
#pragma GCC diagnostic push
#pragma GCC diagnostic ignored "-Wcast-function-type"
#endif
  err = pthread_create(tid, attr, (void*(*)(void*)) entry, arg);
#if defined __GNUC__ && __GNUC__ >= 8
#pragma GCC diagnostic pop
#endif

@ktrongnhan Thanks for the suggestion but I decided to stick with the union, it looks a little more idiomatic to me.


♻ compute parallelism feature as described in arXiv:2202.11045

As discussed in #895, this PR refactors the implementation of the parallelism feature to align with the description from the original paper https://arxiv.org/abs/2202.11045. The implementation takes extra care to avoid corner cases where divisions by zero could happen.
Fixes #895

Hi @burgholzer, do you have any updates on this PR?

Hi @burgholzer, do you have any updates on this PR?

Sorry for the long wait here. Somehow the message above got lost in a flood of emails.
Just added the necessary tests and rebased on main.

@burgholzer Looks like 1 more line needs to be covered by the test.

@burgholzer Looks like 1 more line needs to be covered by the test.

Yeah.  Do you happen to know if there is any way to do that?
In Qiskit, I would just do QuantumCircuit(2) and that would be it and it would cover the remaining missing line.

Create a test in converters_test.py that passes a Qiskit circuit into compute_parallelism_with_qiskit to get that line covered.

Create a test in converters_test.py that passes a Qiskit circuit into compute_parallelism_with_qiskit to get that line covered.

Alright. That should do it. Didn't wanna add tests in that file at first as there were basically no related tests in that file.


Generating opaque tokens - Not matching to the document

Description:
As per the documentation [1] when we try to change the token type expecting bellow UI,

But what i find in 3.2.0 - RC 1
is as follows.

Why is this difference
[1] https://apim.docs.wso2.com/en/next/learn/api-security/oauth2/access-token-types/opaque-tokens/#secure-apis-using-oauth2-opaquereference-access-tokens
Affected Product Version:
3.2.0-RC1

We have removed the token type selection in 3.2.0. We are tracking the documentation issue via https://github.com/wso2/docs-apim/issues/1601

This was fixed
https://github.com/wso2/docs-apim/issues/1601


Unlimited Approval Threshold

The reason to make this change is to make 0xff00...0000 a valid unlimited approval.
Currently 0xffff...ffff is a valid unlimited approval, but if you can approve with 0xff00...0000 instead that is less gas from calldata due to the gas difference between nonzero and zero bytes.
Changes
Change eq to gte
Rename MAX_UINT to UNLIMITED_APPROVAL to reflect usage
Set UNLIMITED_APPROVAL to 0xff00...0000

Another benefit of this is that the trailing zero-bytes can be safely truncated, since your approve doesn't enforce calldatasize

Good idea, although EVM doesn't support lte / gte opcodes. Implemented your suggestion here: https://github.com/Philogy/meth-weth/commit/9012790f9b1bf2670b24d1382fc50e5f7a38fcab


Dupes are opening with physical error.

I use much constrained physical props with ball socket and axis. constraint is about 297.
Sorting constraints check menu is unchecked.
But when I spawn saved dupes, It spawns with physical error. For example, prop with axised is not spawning with axis.
How can I solve this problem.
ps. I'm not influent at enlgish, so I hope you understand.

can't investigate without the dupe file

How I send dupe to you?

I sent dupe from E-mail.

The axis is working for me. Maybe an addon you have is causing it.


Long TXT records are automatically split on add which marks them as changed when checking state

Long TXT records are automatically split into multiple strings on insert which is fine.
However, when retrieving the current state, these are not merged together before comparing them against the string in the terraform file.
Examples for this are long DKIM keys.
Would be nice if the provider could merge them together before comparing them.

done


Reducer dev 2 (opened on behalf of Umur)

If this does what I expect it to do it may accomplish the unrevert as well as give opportunity for more commits.

strange... may not be what I want, especially since it doesn't show diff.

Dialogue pertaining to this PR is taking place on the https://github.com/QURIresearch/squiggle/pull/68 wall

There is a general problem of writing monad code. sth1 >> ( \a -> (sth2 a >> (\b -> sth3) ) ) problem.  I was putting extra lines and indentation so that I could explain the monad code easier. However, the standard code formatter killed that opportunity.
I know what I write and I know it looks cryptic. This is a headache beyond this project.


Update layout.tsx

Maige, what is this PR about?

Maige what is this PR about?

Maige what is this PR about?

Maige what is this PR about?

Maige what is this PR about?

Maige what is this PR about?

Maige what is this PR about?

Maige what is this PR about?

Maige what is this PR about?

Maige what is SEO?

Maige what is SEO in the PR?


Fix some doc typos

@bors r+

:pushpin: Commit 441c9692939ed33705e21c714c17e557d1a2cd80 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 441c9692939ed33705e21c714c17e557d1a2cd80 with merge 74f8d70bdf6d2fe337360a70c4a999c299305db5...

:boom: Test timed out

There is an ongoing incident at GitHub: https://www.githubstatus.com/incidents/qfby3shb6r3w

@bors retry
The GitHub outage should be resolved.

:hourglass: Testing commit 441c9692939ed33705e21c714c17e557d1a2cd80 with merge aaf08a07cccd2e83b8a79636a80c2fdb1c6521e6...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing aaf08a07cccd2e83b8a79636a80c2fdb1c6521e6 to master...


Aliased Install fails with "Is a directory" when trying to install directories

This issue was originally created at: 2007-10-10 17:41:40.
This issue was reported by: benmwebb.
benmwebb said at 2007-10-10 17:41:40

Given a simple SConscript:

Install('/tmp/scons-test/', 'testdir')
Alias('install', '/tmp/scons-test/')

where 'testdir' is a directory, the following fails every time with the v0.97.0d20070918.r2446 build:

$ rm -rf /tmp/scons-test/; scons install
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
Install directory: "testdir" as "/tmp/scons-test/testdir"
scons: *** [install] /tmp/scons-test/testdir: Is a directory
scons: building terminated because of errors.

However, the directory is in fact successfully installed. "scons install" also reports no errors if the target /tmp/scons-test/ directory already exists.
Oddly, specifying the target explicitly works just fine:

$ rm -rf /tmp/scons-test/; scons /tmp/scons-test
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
Install directory: "testdir" as "/tmp/scons-test/testdir"
scons: done building targets.

"scons install" also works just fine using scons v0.96.96.D001.

stevenknight said at 2008-05-27 14:15:49

Might be related to issue #2015 and/or #1741.

gregnoel said at 2008-05-30 13:10:28

Bug party triage.  May be related to issue 2015.

gregnoel said at 2008-12-26 13:29:09

Adjust triage of issues.

stevenknight said at 2009-11-10 18:00:19

stevenknight => issues@scons

ajf58 said at 2014-07-07 14:56:43

Works for me (SCons 2.3.0 on Linux).
$ scons -Q install
Install directory: "testdir" as "/tmp/scons-test/testdir"
Suggest mark as resolved (fixed).

It's working with recent SCons as well, so closing as resolved.


Adding backgroundColor to ListItem style not showing touch animations.

Hi,
I am trying to change ListItem's default backgroundColor from #808080 to #fff. But, I am not able to see touch animation if I do so.

https://github.com/GeekyAnts/NativeBase/blob/master/ISSUE_TEMPLATE.txt


fix: import path for KeyManager

Description
The current develop branch fails when building the contracts using npm run build.
Tasks

[x] fix import path

this should be fixed in
https://github.com/lukso-network/lsp-smart-contracts/pull/18


Getting Uncaught DOMException: Failed to execute 'removeChild' on 'Node' when rerendering component

Hi,
I am getting this error: Uncaught DOMException: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node, when my react component is rerendered after the drag is completed.
It is probably because the DOM was changed without telling React, and it can no longer find the dragged item in the right place. Is there a workaround for me for this issue? Thank you.

Same problem. I think react-dragula needs to provide a method to cancel everything it has done.
I have tried drake.cancel();, but not working.

Same problem here. Can we check what should be done for this ?

drake.cancel(true); works fine.

You saved my day :).
Thanks a lot :)

Can you expand on how drake.cancel(true); was correlated with the drag and dropping in your code in order to remove the error and still render successfully?

The problem was that the actual DOM structure changed after drag and drop but the virtual DOM (react DOM) doesn't aware of that. So after drag and drop if I will set the state with this.setState() which will be going to change the DOM. It gave me error i.e children dom structure changed or mismatch (don't remember exactly error).If we do drake.cancel(true); the dragula doesn't change the actual DOM and setState works fine and mimic the same effect if I will change my state in the right way just after drake.cancle :)
Hope it helps you to understand it :)

This works but creates an effect where the item jumps back to its original location and then to the new target location. Is there a recommended way to fix that?

If anybody else comes across this, I used the following in my drop callback, which helped a lot:
el.setAttribute("style", `${el.style.cssText};display: none;`);
cardDrake.cancel(true)

Before canceling, I add the display: none; style to the el. This has the effect of hiding the original element so it doesn't pop back into its original place until React re-renders. It is much less noticeable now. It does have the downside that if the logic to update the data on the backend fails, the user has to refresh before they see it again. I suppose that could be fixed in a failure callback though.

One more update...I just got super confused when moving my item caused it to disappear lol! You only want to hide it if you are moving the item to a new container. If it is staying in the same container, it will remain hidden since react won't render a new element, it just moves it instead. Back to the original problem... (or remove the hide in a callback, which is what I will probably do).

why  I use  drake.cancel(true)  it do not work

One more update...I just got super confused when moving my item caused it to disappear lol! You only want to hide it if you are moving the item to a new container. If it is staying in the same container, it will remain hidden since react won't render a new element, it just moves it instead. Back to the original problem... (or remove the hide in a callback, which is what I will probably do).

I was experiencing exactly the same problems that @brandoncc was experiencing. While drake.cancel(true) in the drop listener did prevent that error message to pop up, the next problem was that the dragged element goes back to the original container before switching to the next container. I tried to do some modification with styling as well but I was also experiencing the element disappearing.
What worked for me was I did optimistic ui and in this case we already assume that the dragged element will stay onto the targeted container the moment we know what element is being dragged and what container is being targeted. So by having to setState of dragged element to somehow be under the targeted container category in the drop listener callback right before drake.cancel(true) is called, will keep the element in place. This allows the page to re-render and keep the dragged element to stay in its targeted container, prevent the error message to pop up and even stop the page from crashing (at least in my case).
Example
const drake = Dragula(containers, { revertOnSpill: true });

drake.on('drop', (el, target) => {
  // ** sets optimistic ui **
  this.setState({
    draggedElement: { id: el.id, group: target.id }
  })

  /**
  * add any action here
  */

  drake.cancel(true);
}


lovr.timer.getTime only changes at updates on Oculus Mobile.

I was trying to implement a rough CPU timer with lovr.timer.getTime() as a first pass and was constantly getting 0. It seems like it's set to only change at events. This should at least be documented, and preferably be changed to give a live version of the time if possible (although it's currently doing stuff to handle pausing and things, so I imagine there's a good reason it is the way it is that I'm not aware of).

Hm, that seems bad.  It's returning the headset's predicted display time and using that as the implementation of getTime in the platform layer (which is what lovr.timer.getTime queries).  The headset interface has a separate getDisplayTime function where the predicted display time is meant to be returned.
I don't fully understand the pause stuff going on but maybe this could be tracked as an "epoch" that is separate from the various timers?  cc @mcclure for more info, I think the pause stuff was tricky/buggy and interacts with android lifecycle stuff...

This… was… me misunderstanding what lovr.timer.getTime was supposed to do, I think.
The complicated pausing stuff is MOSTLY to work around a particular nasty OS bug from early in the Oculus Go's lifespan. That bug has since been fixed so I can simplify the pause logic now.
Here is a question: Say you pause the app, and then you resume it 20 seconds later (ie you go to the main menu, and then you return). Should lovr.timer.getTime() indicate 20 seconds passed?
If you're basing dynamics on lovr.timer.getTime(), a jump of 20 seconds could make you do… bad things.

I think it would advance by 20 seconds because timer.getTime is supposed to be wallclock time.  But maybe you could hook into lovr.focus events to detect when the pauses happen if you need to stop your game's timers?  Hm

It's going to be challenging to emit focus events on android because if we were to lovrEventPush(EVENT_FOCUS, false) when android tells us we are paused, no Lua code would be around to poll that event until after resuming...

The thing I want to do here is:

lovr.timer.getTime returns wallclock time on android/oculus_mobile
When the android activity is paused/resume, focus events are emitted.  Due to the structure of the android main loop and the Lua coroutine, both of these events will be consumed after the activity is resumed.  This is just going to be documented as a quirk/known limitation of the design.
To avoid huge dt values after resuming, when the android activity is resumed it will ping the timer module and tell it to reset the delta.

@bjornbytes i'd make sure to use the headset predicted time for the deltas passed to update when available, it's a lot more consistent (iirc, oculus documentation even specifically recommends doing this) than regular wall clock will be

There is lovr.headset.getTime that returns the display time, but the default run loop hasn't been converted to use it yet.  It makes sense for animation and stuff but I haven't been able to grok if it's okay to use for things like physics timesteps whatnot (you'd probably want a separate fixed timestep or something).  Either way, it's a separate issue from this one.

I think you want to use predicted-frame-time for deltas. The point of the update function is to prepare a frame for display. Actual wallclock time is something that would be good to expose but it's only useful for certain special applications.
Fixed physics timesteps are for many purposes a good feature but the user Lua code can emulate it itself by looking at the dt and saying "oh that's 30 ms, run my physics timestep function 3 times). The only reason I see for Lovr getting involved in providing a fixed timestep event is so the user can do work "between frames" instead of leaving the CPU idle. However in order for this to be useful we would have to be already meeting and beating our deadlines consistently (such the CPU is idle in the first place)… is that the case in practice now?
Not sure this is clear but the predicted display time currently freezes while the application is suspended on Android.


Apache Drill driver not working since 6.1.0

Windows 10 x64 v1903
DBeaver 6.1.3
Included JRE version
Apache Drill JDBC 1.16
org.apache.drill.jdbc.Driver
Since 6.1.0 release, connections using JDBC Drill driver fail with one of two errors:

Unexpected driver error occurred while connecting to database
com/sun/management/OperatingSystemMXBean
com/sun/management/OperatingSystemMXBean
com.sun.management.OperatingSystemMXBean
com.sun.management.OperatingSystemMXBean

or

Unexpected driver error occurred while connecting to database
Could not initialize class oadd.org.apache.drill.exec.memory.RootAllocator
Could not initialize class oadd.org.apache.drill.exec.memory.RootAllocator

To reproduce, just open new connection for Apache Drill and click "Test Connection".  Nothing needs to be populated to test.

We missed jdk.management module in redist JDK.
Thanks for the bug report.

You can test the fix in Early Access version (https://dbeaver.com/files/ea)

verified


Fail to compile file with dot in the name

Package fails to compile files with dot in the name. For example, for a file called test.file.tex I get the following error:
> pdflatex -synctex=1 -interaction=nonstopmode -file-line-error test

This is pdfTeX, Version 3.14159265-2.6-1.40.19 (TeX Live 2019/dev/Debian) (preloaded format=pdflatex)
 restricted \write18 enabled.
entering extended mode
! I can't find file `test'.
<*> test

(Press Enter to retry, or Control-D to exit)
Please type another input file name
! Emergency stop.
<*> test

!  ==> Fatal error occurred, no output PDF file produced!
Transcript written on texput.log.

Seems that the extension is trimmed incorrectly from the file name.
atom-latex version [0.8.6] 2019-07-01

Hmm this is an artefact of https://github.com/James-Yu/Atom-LaTeX/pull/97 - where everything after the first period are stripped out of the filename to support for non default file extensions (.tikz,.Rnw).
This was done to handle cases such as file.tex.tikz (https://github.com/James-Yu/Atom-LaTeX/pull/89#issuecomment-321505504)
I am unsure how to strip filenames to incorporate your use case as well here.


Win and lose conditions

Detect when the player has lost or won the battle and show appropriate animations and screen effects (i.e. "Game Over" and "You Won" messages).

Implemented as of 6a0dc8882bb7fa4aac490991f40ae0b23d803e48