Behavior of fd_write changed, seems to not be printing entire supplied array

I recently updated my local version of wasmtime and noticed that the behavior of fd_write seems to have changed. I am supplying two strings to print: a supplied string, and a newline character. This used to print the newline character, which is what I expect, but seems to no longer do so. I bisected the behavior change back to PR #2444, but I didn't see this behavior change explicitly called out.
Basically, I'm wondering:

Is this new behavior intended, or a bug?
If this new behavior is intended, what am I doing wrong in the example program?

Thanks for reading!
Test Case
(module
  (import "wasi_unstable" "fd_write" (func $fd_write (param i32 i32 i32 i32) (result i32)))
  (memory 1)
  (export "memory" (memory 0))
  (data 0 (i32.const 16) "\n")
  (data 0 (i32.const 18) "world on a new line!")
  (data 0 (i32.const 47) "Hello,")
  (func $print (param i32) (param i32)
    (i32.store (i32.const 0) (local.get 0))
    (i32.store (i32.const 4) (local.get 1))
    (i32.store (i32.const 8) (i32.const 16))
    (i32.store (i32.const 12) (i32.const 1))
    (call $fd_write (i32.const 1) (i32.const 0) (i32.const 2) (i32.const 0))
    drop
  )
  (func $main
    (export "_start")
    i32.const 47
    i32.const 6
    call $print
    i32.const 18
    i32.const 20
    call $print
  )
)

Steps to Reproduce

wasmtime repro.wast (contents above)
Observe that the newline is not printed

Expected Results
Hello,
world on a new line!

Actual Results
Hello,world on a new line!

Versions and Environment
I first observed this behavior on wasmtime 0.29.0. I bisected the issue back to PR #2444, which seems to be where the behavior changed.
Operating system: Windows 10
Architecture: x86

fd_write returns the amount of bytes written. You have to call it in a loop until all bytes are written. This matches the posix write syscall. From the man page of the write syscall:

The  number  of bytes written may be less than count if, for example, there is insufficient space on the underlying physical medium, or the RLIMIT_FSIZE resource limit is encountered (see setrlimit(2)), or the call was interrupted by a signal handler after having written less than count bytes.  (See also pipe(7).)

Higher level api's generally do this loop for you.

fd_write returns the amount of bytes written. You have to call it in a loop until all bytes are written. This matches the posix write syscall. From the man page of the write syscall:

The  number  of bytes written may be less than count if, for example, there is insufficient space on the underlying physical medium, or the RLIMIT_FSIZE resource limit is encountered (see setrlimit(2)), or the call was interrupted by a signal handler after having written less than count bytes.  (See also pipe(7).)

Higher level api's generally do this loop for you.

Thank you, this makes a lot of sense! Sorry for the false report


tensorflow版本

楼主你好，看了你的代码，predict.py中还存在清session的代码, utils.py里面还存在错误（eg: wraning找不到），麻烦确认一下是否为tf2.0版, 还是在原来tf1.0的版本改过来的时候漏改了，谢谢！

漏改了

tf2.2版本

utils.py的错误应该是你环境配置上的小问题，因为我是有单独的tf2环境的，里面没有任何keras和tf1的内容

楼主您好，感谢您的回答，刚才看了一下，utils.py中是由于没有import warnings所以在使用warnings的时候会报错。望修正，谢谢！
另外有个问题想请教一下。
跑训练的时候，报了以下错误，麻烦您能帮忙看一下吗？谢谢！
数据集用的VOC2007
Train for 4510 steps, validate for 501 steps
Epoch 1/25
2020-07-27 18:20:03.244965: I tensorflow/core/profiler/lib/profiler_session.cc:184] Profiler session started.
Traceback (most recent call last):
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 681, in on_epoch
yield epoch_logs
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 324, in fit
total_epochs=epochs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/training_v2.py", line 123, in run_one_epoch
batch_outs = execution_function(iterator)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/training_v2_utils.py", line 86, in execution_function
distributed_function(input_fn))
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/eager/def_function.py", line 457, in call
result = self._call(*args, **kwds)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/eager/def_function.py", line 503, in _call
self._initialize(args, kwds, add_initializers_to=initializer_map)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/eager/def_function.py", line 408, in _initialize
*args, **kwds))
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/eager/function.py", line 1848, in _get_concrete_function_internal_garbage_collected
graph_function, _, _ = self._maybe_define_function(args, kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/eager/function.py", line 2150, in _maybe_define_function
graph_function = self._create_graph_function(args, kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/eager/function.py", line 2041, in _create_graph_function
capture_by_value=self._capture_by_value),
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/framework/func_graph.py", line 915, in func_graph_from_py_func
func_outputs = python_func(*func_args, **func_kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/eager/def_function.py", line 358, in wrapped_fn
return weak_wrapped_fn().wrapped(*args, **kwds)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/training_v2_utils.py", line 73, in distributed_function
per_replica_function, args=(model, x, y, sample_weights))
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/distribute/distribute_lib.py", line 760, in experimental_run_v2
return self._extended.call_for_each_replica(fn, args=args, kwargs=kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/distribute/distribute_lib.py", line 1787, in call_for_each_replica
return self._call_for_each_replica(fn, args, kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/distribute/distribute_lib.py", line 2132, in _call_for_each_replica
return fn(*args, **kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/autograph/impl/api.py", line 292, in wrapper
return func(*args, **kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/training_v2_utils.py", line 264, in train_on_batch
output_loss_metrics=model._output_loss_metrics)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/training_eager.py", line 311, in train_on_batch
output_loss_metrics=output_loss_metrics))
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/training_eager.py", line 252, in _process_single_batch
training=training))
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/training_eager.py", line 127, in _model_loss
outs = model(inputs, **kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/base_layer.py", line 847, in call
outputs = call_fn(cast_inputs, *args, **kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/network.py", line 708, in call
convert_kwargs_to_constants=base_layer_utils.call_context().saving)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/network.py", line 860, in _run_internal_graph
output_tensors = layer(computed_tensors, **kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/base_layer.py", line 847, in call
outputs = call_fn(cast_inputs, *args, **kwargs)
File "/Users/liuyubin/anaconda3/envs/deep_learning/lib/python3.6/site-packages/tensorflow_core/python/autograph/impl/api.py", line 292, in wrapper
return func(*args, **kwargs)
File "/Users/liuyubin/Documents/MyWorkspace/Python/ai_projects/bilibili_up/tensorflow2_ssd/nets/ssd_layers.py", line 111, in call
step_x = img_width / layer_width
TypeError: unsupported operand type(s) for /: 'int' and 'NoneType'
说的是，在获取先验框的时候，无法取得各个有效特征层的宽高。debug了一下，确实是在调用call方法时，参数x的值得shape为(None, None, None, 512), 这里应该是net['conv4_3_norm']的shape(None, 38, 38, 512), 不知道为什么, (38,38)无法取得，取得的是(None, None) 麻烦您，谢谢！

要么你装个2.2试试？

楼主你好，果然是版本问题, 开始我在mac上跑的，查了查，mac最大支持到2.0，换了个环境，就可以了，添麻烦了，多谢！
2020-07-27 11:07:26.007587: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1561] Found device 0 with properties:
pciBusID: 0000:00:04.0 name: Tesla T4 computeCapability: 7.5
coreClock: 1.59GHz coreCount: 40 deviceMemorySize: 14.73GiB deviceMemoryBandwidth: 298.08GiB/s
2020-07-27 11:07:26.007629: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-07-27 11:07:26.009482: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
2020-07-27 11:07:26.011165: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcufft.so.10
2020-07-27 11:07:26.011528: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcurand.so.10
2020-07-27 11:07:26.013472: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusolver.so.10
2020-07-27 11:07:26.014621: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusparse.so.10
2020-07-27 11:07:26.018172: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-07-27 11:07:26.018307: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-27 11:07:26.018918: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-27 11:07:26.019461: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1703] Adding visible gpu devices: 0
2020-07-27 11:07:26.644546: I tensorflow/core/platform/profile_utils/cpu_utils.cc:102] CPU Frequency:<PHONE_NUMBER> Hz
2020-07-27 11:07:26.644750: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x1ce6bc0 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-07-27 11:07:26.644789: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-07-27 11:07:26.755111: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-27 11:07:26.756071: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x1ce6d80 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
2020-07-27 11:07:26.756107: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Tesla T4, Compute Capability 7.5
2020-07-27 11:07:26.756343: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-27 11:07:26.757222: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1561] Found device 0 with properties:
pciBusID: 0000:00:04.0 name: Tesla T4 computeCapability: 7.5
coreClock: 1.59GHz coreCount: 40 deviceMemorySize: 14.73GiB deviceMemoryBandwidth: 298.08GiB/s
2020-07-27 11:07:26.757278: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-07-27 11:07:26.757346: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
2020-07-27 11:07:26.757380: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcufft.so.10
2020-07-27 11:07:26.757401: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcurand.so.10
2020-07-27 11:07:26.757420: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusolver.so.10
2020-07-27 11:07:26.757439: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusparse.so.10
2020-07-27 11:07:26.757458: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-07-27 11:07:26.757537: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-27 11:07:26.758208: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-27 11:07:26.758755: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1703] Adding visible gpu devices: 0
2020-07-27 11:07:26.758805: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-07-27 11:07:27.283300: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1102] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-07-27 11:07:27.283370: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1108]      0
2020-07-27 11:07:27.283400: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1121] 0:   N
2020-07-27 11:07:27.283598: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-27 11:07:27.284199: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-07-27 11:07:27.284773: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1247] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 13970 MB memory) -> physical GPU (device: 0, name: Tesla T4, pci bus id: 0000:00:04.0, compute capability: 7.5)
2020-07-27 11:07:28.733087: I tensorflow/core/profiler/lib/profiler_session.cc:159] Profiler session started.
2020-07-27 11:07:28.733161: I tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1363] Profiler found 1 GPUs
2020-07-27 11:07:28.733974: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcupti.so.10.1
2020-07-27 11:07:28.835520: I tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1479] CUPTI activity buffer flushed
Epoch 1/25
2020-07-27 11:07:40.762919: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-07-27 11:07:45.915494: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
2020-07-27 11:07:51.277061: I tensorflow/core/profiler/lib/profiler_session.cc:159] Profiler session started.
1/281 [..............................] - ETA: 0s - loss: 4.00132020-07-27 11:07:51.477306: I tensorflow/core/profiler/internal/gpu/cupti_tracer.cc:1479] CUPTI activity buffer flushed
2020-07-27 11:07:51.477640: I tensorflow/core/profiler/internal/gpu/device_tracer.cc:216]  GpuTracer has collected 693 callback api events and 693 activity events.
2020-07-27 11:07:51.511243: I tensorflow/core/profiler/rpc/client/save_profile.cc:168] Creating directory: logs/train/plugins/profile/2020_07_27_11_07_51
2020-07-27 11:07:51.525389: I tensorflow/core/profiler/rpc/client/save_profile.cc:174] Dumped gzipped tool data for trace.json.gz to logs/train/plugins/profile/2020_07_27_11_07_51/6abc6edfae8e.trace.json.gz
2020-07-27 11:07:51.532754: I tensorflow/core/profiler/utils/event_span.cc:288] Generation of step-events took 0.244 ms


Antiwirus/Malwarebytes problem

Boonwin do you want to tell us something? Kappa
https://imgur.com/a/RJ7aUUG
i need to put those files into quarantine

i dont know what does that mean, its just a libary from microsoft, that i use to be able to use updates automatically instead of doing it youself. i think malewarebytes doesnt like this in gernal because you could use it for some bad stuff but i didnt...


META-INF/javax.annotation.processing.Processor is included empty when shading is enabled - log4j

Hello,
to reproduce I create a simple sbt project with.
sbt new scala/hello-world.g8
Any sbt-assembly version >= 1.0.0 has the issue.
In the build.sbt:
scalaVersion := "2.12.12"

name := "hello-world"
version := "1.0"

libraryDependencies += "org.scala-lang.modules" %% "scala-parser-combinators" % "2.1.1"
libraryDependencies += "org.apache.logging.log4j" % "log4j-core" % "2.17.1"

assembly / assemblyMergeStrategy := {
  // As an example, keep services but discard all the rest
  case PathList("META-INF", "services", xs @ _*) => MergeStrategy.first
  case PathList("META-INF", xs @ _*)             => MergeStrategy.discard
  case x                                         => MergeStrategy.first
}
// Randomly shade something - if your remove this, the issue is not re-produced
assembly / assemblyShadeRules := Seq(
  ShadeRule.rename("scala.collection.compat.**" -> "shadecompat.@1").inAll
)

In the produced hello-world-assembly-1.0.jar file \META-INF\services\javax.annotation.processing.Processor is empty.
To my point of view this is bug, because that file is not empty in the originating log4j-core-2.17.1.jar.
To "fix" this issue, I can either change the plugin version to an older one, like 0.14.10, or remove the shading rules.
Additonal files: In my project that produces an 160MB .jar file, I have the same issue with META-INF\org.apache.logging.log4j.util.PropertySource. Interestingly, other javax.* or org.apache.logging.log4j.* files are correctly included.

I'm having the same issue. Note that this only happens on Windows. Building on Linux produces the correct files in the jar.

Any progress on this issue?


Update BUTTON_IDEAS.md

A social button to follow the creator's social handles like twitter, github, etc.

Issue:

Great thank you!
@all-contributors please add @Archis19 for idea


Enhance and update documentation

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Update documentation and changelog highlighting the main features and bugfixes.
Which issue(s) this PR fixes:

Part of #222
Special notes for your reviewer:

/assign @ahg-g
cc @cortespao

/label tide/merge-method-squash
/hold for reviews

/lgtm

Updated and squashed

/hold cancel

/lgtm


Fixed crash when calling checkForDecideResponseWithCompletion:useCache:

We get around 20 crashes a day with the following call stack. We debugged the issue and found an incorrectly initialised property. This fix prevents those crashes.
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '*** -[NSSet initWithSet:copyItems:]: set argument is not an NSSet'
*** First throw call stack:
(
	0   CoreFoundation                      0x04a4ca84 __exceptionPreprocess + 180
	1   libobjc.A.dylib                     0x04443e02 objc_exception_throw + 50
	2   CoreFoundation                      0x0497cf4f -[NSSet initWithSet:copyItems:] + 1231
	3   CoreFoundation                      0x0498e504 +[NSSet setWithSet:] + 68
	4   Mixpanel                            0x00cbe9ab __58-[Mixpanel checkForDecideResponseWithCompletion:useCache:]_block_invoke + 6347
	5   libdispatch.dylib                   0x05672377 _dispatch_call_block_and_release + 15
	6   libdispatch.dylib                   0x056959cd _dispatch_client_callout + 14
	7   libdispatch.dylib                   0x05679650 _dispatch_queue_drain + 2227
	8   libdispatch.dylib                   0x05678b04 _dispatch_queue_invoke + 570
	9   libdispatch.dylib                   0x0567b7bb _dispatch_root_queue_drain + 550
	10  libdispatch.dylib                   0x0567b58e _dispatch_worker_thread3 + 115
	11  libsystem_pthread.dylib             0x059b5270 _pthread_wqthread + 1050
	12  libsystem_pthread.dylib             0x059b2f82 start_wqthread + 34
)
libc++abi.dylib: terminating with uncaught exception of type NSException

@SebasDesimone What an incredible oversight. Thank you so much for finding this and testing it. We will be sure to include this in the next release.
Please don't hesitate to submit another PR if you find any other issues with our SDK. Thanks again!


Feature request: Support for synchronous applications

Feature request to add new method verify_sync or verify_blocking to AccessTokenVerifier and IDTokenVerifier that will have the same behavior as verify but not be async. I see the README has some instructions on how to make sync calls but does not address how to get the JWK synchronously.
We are an Okta customer with a synchronous Python application written in Django. The async implementation for AccessTokenVerifier.verify and IDTokenVerifier.verify is causing issues for us. Our application is not written with coroutines and due to other implementation issues, we don't want to use run_until_complete. We can't rewrite our entire application to be async at this time.
This change would involve a lot of duplicated code to make async methods sync / blocking, but it would allow support for Python code that doesn't use coroutines.

Is there any update on this?

Workaround, making sure to use a new loop if there could already be a running event loop for the Python process:
loop = asyncio.new_event_loop()
asyncio.set_event_loop(loop)
loop.run_until_complete(jwt_verifier.verify(token))
loop.close()


Need to Skip test cases under a fixture If previous test of the same fixture failed

What is your Scenario?
I am working with a situation where I have the following scenario/steps to Test

Create
Edit
Delete

I have the test cases ready but I need a way to skip the Edit and Delete TCs if the Create TC fails. In other words I need to chain them together so the system understands that if Create was not successful then skip the Edit and Delete test cases (Conditional Skipping).
Is it possible with any existing process like hooks/test Context?
Is there any logic I can use to make this happen?
Thanks in Advance.
Expecting something like below
What are you suggesting?
test('Verify Create New Account Functioning Successfully', async (t: TestController) => {
    // if this fails, I can set a variable x=fail
});

test('Verify Edit Account Functioning Successfully', async (t: TestController) => {
    // Skip this as create test case failed
    if(x == fail) {
        test.skip(t.this test);
    }
});

What alternatives have you considered?
Could not find any material regarding this in testcafe documentation.
Additional context
No response

Hey, try something like this
import { Selector } from 'testcafe';

let shouldSkip = false;

fixture `My Fixture`
    .beforeEach(async t => {
        if (shouldSkip) {
            await t.skip();
        }
    })
    .afterEach(async t => {
        // Check if the current test failed
        const testFailed = t.testRun.errs.length > 0;

        if (testFailed) {
            shouldSkip = true;
        }
    });

test('Test 1', async t => {
    // Your test logic for Test 1
    // Uncomment this line to simulate a test failure
    // throw new Error("Simulated failure in Test 1");
});

test('Test 2', async t => {
    // This test will be skipped if Test 1 fails
    // Your test logic for Test 2
});

test('Test 3', async t => {
    // This test will also be skipped if any previous test in the fixture fails
    // Your test logic for Test 3
});

In TestCafe, you cannot skip a test from inside or use the beforeEach hook for this purpose because the test queue is created at the compilation stage.
Unfortunately, we cannot take this enhancement into work since we are currently focused on other tasks.


Use Stellar Classic test Assets in Testnet

maybe you could use classic stellar assets for the test assets so that way you can mint, send all of them and add the trustlines in a single transaction (that way we don't need to wait)

Added stellar classic assets on the backend and api.
Trustlines pending form the frontend

Now:

We can add a list of Stellar Classic assets in the API
We can see the Balances:

Next:

Add Liquidity / Remove Liqudiity / Swap

Add support to receive a Stellar Classic asset (Trustline)

#216


Visual Studio Code & node.js: Debugger exits without error message on "utils.format"

Node 5.3, VS Code 0.10.6, Windows 8.1

node.js application logging debug output to the console.
Running it as "node myApp.js" works fine: The app runs and all the "console.log" statements are correctly printed to  stdout resp. stderr.
Running it as "node debug myApp.js" worsk fine, too.
When I launch the same "myApp.js" in VS Code, the debuger exits (shuts down the app) on the first "console.log('Logging a complex object: %O', myComplexObject)". It does not print any error messages.
Narrowing down the error, I found out that "util.format()" with a complex object causes the problem, not the logging itself.

@mindruptive that seems to be a problem in node 5.3. Node 5.3 is crashing on this program:
var myComplexObject = { name: "foobar" };
console.log('Logging a complex object: %O', myComplexObject);

I could reproduce this with node-inspector too. Run the program from the command line:
node --debug-brk bug2484.js

and then attach node-inspector. Stepping over the console.log will make it crash.
This does not happen with node 5.5 (but the program will not terminate either).
Closing because this issue is not VS Code related.

Thanks weinand,
it happens with node 5.5, too but regardless of VSC. Could reproduce it even by starting the commandline debugger and stepping through with "next".
`var util = require("util");
var obj = { 1: "fasdadsaas", 2: "safsdafdsafsadfad" };
var message = util.format("Format the object: %O", obj);
var i = 1;`
Thanks for the hint!
mind

@mindruptive since you have a small reproducible snippet, I suggest to file a bug against node: https://github.com/nodejs/node/issues

Thanks, I will.
Best
m


Unicode Calendar Events don't work.

Describe The Bug:
If you setup an event match with unicode characters the regex here:
https://github.com/uamanager/homebridge-calendar-scheduler/blob/master/src/configs/event.config.ts#L33
Will strip them out, possibly leaving an empty string and putting homebridge into a restart loop.
By changing safeEvent to eventName here:
https://github.com/uamanager/homebridge-calendar-scheduler/blob/master/src/calendar.handler.ts#L100
Everything works.
Plugin Config:
{
            "caseInsensitiveEventsMatching": false,
            "debug": false,
            "calendars": [
                {
                    "calendarName": "Calendar",
                    "calendarUrl": "webcal://p101-caldav.icloud.com/...",
                    "calendarUpdateInterval": 60,
                    "calendarUpdateButton": true,
                    "calendarTriggerOnUpdates": false,
                    "calendarTriggerOnAllEvents": true,
                    "caseInsensitiveEventsMatching": true,
                    "calendarEvents": [
                        {
                            "eventName": "😎",
                            "eventTriggerOnUpdates": false,
                            "caseInsensitiveEventsMatching": false,
                            "calendarEventNotifications": [
                                {
                                    "notificationName": "TurnOn",
                                    "notificationStartOffset": -45
                                }
                            ]
                        }
                    ]
                }
            ],
            "platform": "CalendarScheduler",
            "_bridge": {
                "username": "0E:32:8A:D1:92:1A",
                "port": 54101
            }
        }

Hey, thanks for reporting this bug!
Seems like it's really removing unicode symbols and leaving empty string. The idea was to remove everything that could be wrong from homekit standpoint, but it caused such issue.
Let me think on how it can be fixed without impact on those who doesn't use emojis in name.

Thanks. Were there specific characters that were causing issues in HomeKit?

HomeKit accessory name can not contain special characters or emoji, so I assume that id's also should not contain them.

With the change I did above, it created an accessory with the emoji no problem.

Sure, It creates accessories using Homebridge, but it doesn't mean that such symbols are not allowed. Try to change accessory name from Home application.
I'll think about fix and notify you as soon as it'll be released.

I see what you mean now. You can't modify the name in Home (or say the Eve) app. But Homebridge via your plugin can create those accessories.

Sure, but I'd prefer not to go against rules even if I can in such workaround way.

I've added additional config property unsafeEventNames on each level of configuration, so you can enable it globally, for calendar or for concrete event. It'll allow you to use unicode symbols and won't touch event name at all.

Thanks - I'll give that a try.


Can't move created steps inside of switch loops

Describe the Bug with repro steps

select the " create an item" sharepoint step
Create a switch case

Try to put the ceate an item inside the switch loop

What type of Logic App Is this happening in?
Consumption (Portal)
Are you using Preview Designer or GA Designer
Preview
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Chrome
Additional context
No response

Hey @tyki21, Thanks for the report. This one is a known issue and we'll have a better drag-drop solution out in our next release. About 2 weeks. Sorry for the inconvenience currently.


Add Docker container definition

Added Dockerfile to create a containerized environment with pre-installed dependencies for running the plugin.
Updated runPluginStarter.sh to add default values for easier usage with the container.
Updated GitHub Actions workflow (build.yml) to include building and pushing of Docker images to the GitHub Container Registry. Moved .github folder to the repository root to make it visible to GitHub Actions.

Nicely done!


Running jupyter notebook tutorials locally requires installing pytest

Hi thanks for putting together the tutorials!
I got started following https://github.com/microsoft/QuantumKatas#running-the-katas-locally-, the jupyter notebook tutorials use pytest for evaluation but it was not mentioned in the set up process. It was in the Dockerfile though.

True, our instructions for running the katas are rather outdated, I will be taking a closer look at them next week. (You might have a similar problem with matplotlib for "Exploring Grover" tutorial)
I wonder which flavor of the installation instructions did you follow when installing Q#? I think my Python came from Miniconda install which already included a lot of Python tools, but that was the previous version of Q# install instructions, the current one at https://docs.microsoft.com/en-us/quantum/install-guide/ doesn't suggest Anaconda.

I'm on Mac OS, I installed Python3 via Homebrew, it is located at /usr/local/bin/python3 and version 3.7.4. For Q# I followed the first two sections of https://docs.microsoft.com/en-us/quantum/install-guide/index?view=qsharp-preview#develop-with-python. I have been using jupyter notebook for the tutorials. Hope it helps.

You're right, I just got to VisualizingGroversAlgorithm and I had to install matplotlib manually, thanks for the heads up.

Ran into the same issue before realizing I needed to install pytest and also have a similar setup: Mac OS, Python3 via Homebrew

Thank you for confirming! To my great embarrassment, I still have to carve out the time to update our installation instructions...


bump: upgrade go redis version

https://github.com/arana-db/kiwi/issues/35

Summary by CodeRabbit

New Features

Upgraded the Redis client library, potentially enhancing performance and introducing new features.

Bug Fixes

Included improvements from the latest version of the Redis client library.

CI 问题看下

为啥要升级呢？用原来的有啥问题

等 CI 没问题再合并


fix header installation

this fix https://github.com/aminya/cpp_vcpkg_project/issues/4

Thank you! I added your changes as part of #5


docs(ts): update TS sdk docs (input/output removed in PR #34)

https://github.com/langfuse/langfuse-js/pull/34 removed the input and output params from the *.generation() parameters.

Thank you for spotting this, the change to input/output slipped through when we updated the sdk docs as the v2 release of the JS SDK is a bit delayed (pr). The SDK will catch up with the docs tomorrow hopefully.

Decided to merge to ensure consistency until v2 is merged


GitHub Repository Chooser

Intention

I'm submitting a feature proposal

Using the same principle as Monterey Package Chooser (minimal user input needed to get access to desired resource), I propose to create and maintain the collection of GitHub Organizations of interest as a part of user preferences.
This collection is used as the first level choice (pop-down combobox) - the selected organization is used to dynamically fetch all repositories in that organization - and so on ...

Instead of grouping by github organizations, we could also choose to show all repositories and include search functionality, similar to SourceTree:

One idea of mine is to create "Accounts" in Monterey's settings, one of which could be a Github account. This account can then be used to authorize requests to the GitHub API, and to retrieve all github repositories associated with the accounts. Note that Accounts would be something abstract, so that we can allow users to associate other accounts as well (docker hub account for example).

Having a (modifiable) list of organizations as the basis for subsequent searches is an optimization for reducing the domain of the search - so, we should offer both via radio-button stated preference). I do want to ensure that the user is not expected to go search for the needed repository URL somewhere outside Monterey.
I like your idea of "Monterey's account structure abstraction", since then the GitHub organization has immediately has found its place in that structure. Very good concept!

I initially did not even think of such approach out of concern for the sheer amount of GitHub repositories and the length of the such search (issue that I am at the moment addressing with the new npm search in a related situation for MPC)

I do not have this concern, as typing in a few letters is enough to find what you are looking for. Just to confirm, I am talking about showing all repositories associated with the GitHub user account, and the repositories of the organizations this user account is a part of. So I am not suggesting to show all github repositories

So I am not suggesting to show all github repositories

This is too narrow approach (if the concept of your abstract  account is narrower than GitHub organization)
To eliminate this possibility for misunderstanding, I will define the new term kraka (completely new word in any language that I know of) and say that kraka is the reference to the second component of the path to a Github repository (monterey-famework) on the example below:

So, kraka is either reference to an user account (as in github.com/adriatic) or to an organization (as in github.com/aurelia. I want to be able to quickly select any of repositories define by my own list of krakas - a search process that is a lot quicker than searching through the whole set of GitHub public repositories.
Use case scenario
Very often I am interested in looking at details of several repos belonging to the specific kraka - like https://github.com/electron to find out news about Electron. Rather than browsing the repo with browser, I would like to clone each repo of interest, and then build it and run - all within Monterey.
I am pretty sure that there are even more appealing use cases, but mine should suffice for this discussion.

MPC and GRC common idea
In both cases the first step of the chooser is very quick:

in MPC (Monterey Package Chooser) we use autocomplete element being fed from npm registry - process that takes less than a second in the most complex case - entering the first character. Once the name of the package is defined (by a click in the automplete pop-down list), we issue the second API call to npm, which returns all data of interest to to the application Create / Update process.

in GRC (GitHub Repository Chooser) we do not have the same luxury as with npm - which is why I am inventing the help from kraka as explained above. The rest of the process should be obvious.

Very often I am interested in looking at details of several repos belonging to the specific kraka - like https://github.com/electron to find out news about Electron. Rather than browsing the repo with browser, I would like to clone each repo of interest, and then build it and run - all within Monterey.

Seems like a good use case to me, but that does not remove the need for being able to see my repositories and the repositories of all my organizations in one, searchable list. So we should make sure that both use cases are covered


Make xpub and xpriv identifiers match rust-bitcoin

Recently in rust-bitcoin we agreed that 'xpub' and 'xpriv' are well known terms in the Bitcoin vernacular so, being "words" should be written as Xpub and Xpriv when translated to Rust identifiers.
Make the descriptor key identifiers match rust-bitcoin.

Jeeebus, I didn't realize that in descriptors we use xprv not xpriv - what a mess.

This isn't a descriptor thing. xprv and xpub come from the base58 encoding of these objects, as defined in BIP32 in 2012 https://github.com/bitcoin/bips/blob/master/bip-0032.mediawiki#serialization-format

I don't mind changing the capitalization here.
But FWIW I don't think anybody uses the rust-bitcoin types if they have access to the rust-minscript types. So it's not the end of the world if they're inconsistent.


README.md: adding a note about yarn create

README.md clearly describes how to use npx, but doesn't provide an example with Yarn.
This PR adds a note about Yarn usage in Getting Started.

Thanks, @robert-zaremba !


Adding bug report and feature request issue templates

@TMCamp Just wanted to check: is this PR still in progress? Seems stale

We'll revisit this later.


Problem CollisionShape2D disabled (3.1)

Godot version:
v3.1.beta.custom_build.33b64ec
OS/device including version:
Ubuntu 16.04
Issue description:
Godot 3.0.6 works great CollisionShape2D.disabled = bool
Godot 3.1 fix pliz =)

E 0:00:26:0494   Can't change this state while flushing queries. Use call_deferred() or set_deferred() to change monitoring state instead
     servers/physics_2d/physics_2d_server_sw.cpp:737 @ body_set_shape_disabled()
 WorldMap.gd:22 @ fsignalstairs()
Area2DSignalReturn.gd:16 @ _on_Area2DSignalReturn_body_entered()

Steps to reproduce:
Godot 3.0.6 ===>

Godot 3.1 ====>

Minimal reproduction project:
PsychiatricHospital2D_test.zip

Could you provide a minimal project that reproduces the issue?

already posted file

Not sure this is a bug, was probably undefined "works by accident" behaviour in 3.0 instead. The error message is very specific and clear in what you need to do, otherwise you'd end up telling the Area "While you're detecting a collision, you're no longer an active collider" in the same physics frame.

well this is an bug that cannot be missed wall, I have come up with some other option  .set_collision_layer_bit(id, bool) works no problem, only .disabled = bool does not work

Can't change this state while flushing queries.
Use call_deferred() or set_deferred() to change monitoring state instead
WorldMap.gd:22 @ fsignalstairs()
… only .disabled = bool does not work

This is a layer 8 problem. Just use the friendly suggested
arrsignalstairs[i].set_deferred("disabled", true), etc
and it will work.

Just use the friendly suggested
arrsignalstairs[i].set_deferred("disabled", true), etc
and it will work.

Indeed, that's an expected change in 0b1e93cc to fix bugs with changing physics properties during physics callbacks. You are now expected to use set_deferred as advised by the error indeed.

Hello, I have the exact same issue for the same functionnity : going up and down stairs.
I put
stairs_shape.set_deferred("disabled", "true")
But that doesn't work at all. Any idea ?


[DOC] Add more examples to docs

Suggested examples:

In configuration reference (/docs/0.8.0/advanced-resources/settings/), if applicable, each default value should be included in addition to the example values.
In settings, add a full example YAML file of a longhorn deployment configuration
Multiple disk support: Configuration section should have an example YAML
The example setup in "recovery volume after unexpected detachment" should be in the best practices and in the settings/configuration reference
Multiple disk support: Need to make sure configuration information is in the settings/configuration reference
In "Create NGINX ingress controller with basic authentication" It says "It’s important the file generated is named auth (actually - that the secret has a key data.auth), otherwise the ingress-controller returns a 503" I think it would help to provide an example YAML of the auth secret
Need to make sure volume expansion configuration information/example yaml is in the settings/configuration reference.

Yes, it would be great to have an improved documentation. I'm trying to figure out how to create a YAML configuration for multiple disks but so far I've failed.
@catherineluse Have you been able to find anything? Do you mind to share what you may have if you don't mind?

@aureq You meant https://longhorn.io/docs/0.8.1/advanced-resources/default-disk-and-node-config/ ? Notice the node annotation only applies if there is no disk on the node yet.

[ ] Add default values in references/settings.md
[ ] Provide a full example YAML file/Update the YAML file in advanced-resources/deploy/customizing-default-settings.md
[ ] Add multi-disk instructions  in advanced-resources/deploy/customizing-default-settings.md and put the related references in references/settings.md.
[ ] Add a reference of high-availability/recover-volume.md in best-practices.md.
[ ] Add more details in deploy/accessing-the-ui/longhorn-ingress.md

Merged.


TypeError (no implicit conversion to float)

In development I use ruby 2.1.2, and have no issues with this gem but in production I use rubinius 2.5.2 (2.1.0) and I get the following error:
TypeError (no implicit conversion to float)
I'm getting this error in a few different PDFs I'm trying to generate, any ideas?

Thanks for trying it with Rubinius! I should add it to Travis :grin:
Meanwhile… could you send the entire stack trace of the error, so I can get an idea of where it is happening? Thanks!

I think I found the error!  See #45
https://travis-ci.org/Fullscreen/squid/jobs/110049740#L282

I don't :( sorry, and thanks so much for the quick response.  I upgraded to RBX 3.14 and am now getting the following error:  **NameError (uninitialized constant ActiveSupport::NumberHelper::NumberConverter) **,   this looks to be an issue with RBX and not squid.


Supply google-services.json and GoogleService-Info.plist on Cloud Build

Can use KMS to supply the files

No longer needed, thanks @arnoid


Fix error on export

This is not #792.

Should we even call makeEditable if we don’t pass an element?

Oh I see, because we use ref hack it gets passed null when the component unmounts


ARROW-6314: [C#] Implement IPC message format alignment changes, provide backwards compatibility and "legacy" option to emit old message format

Porting the fix for ARROW-6314 to the C# library.
/cc @chutchinson @pgovind

Thanks @eerhardt!

Does this patch account for the EOS change under discussion on the mailing list?

Does this patch account for the EOS change under discussion on the mailing list?

Yes, partially. The reader handles the first 4 bytes being 0 to mean stop reading after 4 bytes. And if it encounters a continuation token 0xFFFFFFFF and then a 0 it stops.
On the writer side, the C# API doesn't have a "Write end of stream" function. But maybe now it is important to add that method, since the EOS can take 2 formats depending on the options.
I can add that new API, if you think it makes sense.

Yes, that sounds good to me!

@pgovind @chutchinson - FYI I implemented a new method on ArrowStreamWriter - WriteEndAsync(), which will write the EOS signal. I also moved ArrowFileWriter's WriteFooterAsync to instead just use WriteEndAsync from the base class. This means anyone using ArrowFileWriter will have a slight change - WriteFooterAsync => WriteEndAsync.
Please take another look.


No module named randomcolor

cagcak@xps:~$ sudo python G213Colors.py
Traceback (most recent call last):
  File "G213Colors.py", line 30, in <module>
    import randomcolor
ImportError: No module named randomcolor

Related to #6

Try running: sudo python3 G213Colors.py

did not react to my question in https://github.com/SebiTimeWaster/G213Colors/issues/6


Theme 004 is missing a lighter color for the focused workspace text

Hello.
I love your work so much I wanted to make an account to contribute. As mentioned in the title - I was missing a white color for the text of the active workspace. I managed to fix it, though!
In .config/polybar/config - change
label-focused-foreground = #1d1e26
to
label-focused-foreground = #ffffff
Thank you for your work, I hope this helps someone!

Hello sir,
Help is always welcome, you can fork the project, make the changes and then create a Pull Request (so that your contribution is visible to others if you may). I will merge the pull request as soon as I see it :)

I'm closing this (for now), any other improvement you want to make you can always create a new Pull Request and I'll check it out.


Millions of followers

Hi, Thanks for sharing. This is not an issue, but a question. What if a user has a very long list of followers, eg millions of followers like twitter or facebook? How we will deal with it?

The config to use is the one that scales best, which is where edges (relationships) are stored separately from nodes (users).  This is discussed in the presentations we've done about this.  Embedding is not a good choice for this type of application for that exact reason.


Update WHAM TAGS INFO field with renamed sample ID

Recent changes to re-write sample IDs missed the fact that WHAM encodes the sample name in the TAGS INFO field. This caused the VCF standardization script for WHAM files to exclude all variants, leaving the files empty. This change ensures that the TAGS field is re-written with the specified sample ID in addition to the VCF header.
Tested by running the single sample pipeline on NA12878 with different sample identifier.

@cwhelan  What should to do with results generated by the Whamg.wdl in gatk-sv-0.8.2-beta? Thanks a lot!

Hi @LNGDingj -- have you been trying to run the single sample pipeline (GATKSVPipelineSingleSample.wdl)?
First I'd verify that this bug was affecting you. I believe that it would only affect you if you were running the pipeline with a different sample identifier input (sample_id) than the sample ID that is present in the bam or cram file you are using in the @SM header. The issue that this PR was trying to fix resulted in standardized wham VCF files that are returned from Module00c in the std_wham_vcf output but contain no records. This was also causing downstream fatal errors in the SRTest subworkflow in some samples.
If you think you might be affected, I'd probably recommend re-running the workflow with the v0.8.3-beta release. If you've already run on lots of data, I could try to walk through other solutions for recovering your data.

Hi, @cwhelan -- Thanks so much for your quick reply. I already used Whamg.wdl through Module00aBatch, and generated results of over 8000 samples at GCP using same sample IDs as those presenting in .cram and .bam files. And plan to continue to run Module00b and following Modules in the gatk-sv pipeline. Thanks again!

@LNGDingj  If you'd like to double-check that your files are OK, you can make sure that the TAGS info field within your wham output VCFs matches the sample ID listed on the header line of the VCF file and the expected sample_id that you gave as an input for the workflow. If you do run into issues let me know (file a ticket on this repo, probably) and we could help you figure out how to fix the raw wham output files if necessary.

@cwhelan I did not change sample IDs (same as in .cram and .bam), when I was using cohort mode to run Module00aBatch.wdl, I noticed that in README.md it states  "Currently, sample IDs can be replaced again in Module 00c". May I change sample IDs for those with "-" in Module 00c? Thanks so much!

@LNGDingj You should be able to proceed to 00b. However, I would wait on 00c until we resolve this issue: https://github.com/broadinstitute/gatk-sv/issues/23

@mwalker174 Sure, thanks a lot!

@mwalker174
Hello,
Following your reply on Jul 10, I noticed that issues #26 (Replace remote tabix with GATK) was closed yesterday, so it is ready to proceed to Module00cBatch with all scripts currently deposited at broadinstitute/gatk-sv on github?
Regarding to "Currently, sample IDs can be replaced again in Module00c", in addition to replacing all "-" in sample IDs with "_" for "Module00c.samples" in input json file, what else should I do?
Thanks a lot!


⚠ Adacta-fintech.com has degraded performance

In 5ff4363, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 200
Response time: 1559 ms

Resolved: Adacta-fintech.com performance has improved in ea7de75 after 7 minutes.


Test: make sure a provider doesn't get dispatched content it's already storing

@gallexis are you gonna add a test for UpdateRef? Just to check that it merges the keys correctly?

@gallexis are you gonna add a test for UpdateRef? Just to check that it merges the keys correctly?

Just made it :)

I've added the updateRef unit test 👍


[Feature Request] Melting Pot Integration

Motivation
Melting Pot: https://github.com/deepmind/meltingpot is a great MARL evaluation tool for MARL research.
Solution
Integrate Melting Pot into Env Pool
Additional context
Melting Pot was built on top of Lab2D: https://github.com/deepmind/lab2d
Checklist

[x] I have checked that there is no similar issue in the repo (required)

It uses Lua for core simulation. Seems not so easy to be directly integrated into envpool.


fix: idea default dir can't use git

@microsoft-github-policy-service agree

This seems specific to where you mount or clone your git repositories with IDEA which is a bit too specific of a scenario to include here. It would fit better in the devcontainer.json you use to spin up your dev container.


Allow SegmentAnalyzer to read columns from StorageAdapter, allow SegmentMetadataQuery to query IncrementalIndexSegments on realtime node

Allows SegmentAnalyzer to create an analysis from a StorageAdapter; this is used during the SegmentMetadataQuery to allow the query to read from IncrementalIndexSegments wrapped by a StorageAdapter.
This allows the metadata query to read columns from the most recent hydrant on the realtime node.
Size calculation is not performed in this alternate path.

:+1:

:+1:


 When I put 'hoodie.cleaner.commits.retained': 1 in hudi_options I still have two last commits. One that is being written and the previous one. What I want to achieve is to have only last change and last parquet file.

Hi Guys,
Is it possible to retain only last commit? When I put 'hoodie.cleaner.commits.retained': 1 in hudi_options I still have two last commits. One that is being written and the previous one. What I want to achieve is to have only last change and last parquet file.

@kirkuz  providing some context. Cleaning and compaction happen in the background (asynchronous to ingestion itself). When the cleaner kicks in it would get rid of the older commit. If there is an ongoing write, generally there could be two possibilities -

the write would succeed. in which case based on hoodie.cleaner.commits.retained the cleaner would get rid of the old version when it triggers.
the write would fail for some reason - in this case the cleaner would later get rid of the failed commit and retain the other  version (which is the last succeeded one)

This is why you are seeing two commits. This should not affect the queries. Can you please elaborate on what you were looking for in terms of use case/ performance concern etc to help us understand better ?

Hi @bhasudha, thanks for this information. Now, it's clear how it's working. My use case is as following: I want to have only last change in parquet files, because when I try to read it with AWS Athena it sees doubled record with different _hoodie_commit. In ideal world I want to have one S3 bucket with only last change (to not force users to deduplicate it in AWS Athena query) and the second bucket with all commits to have whole history.
Can you recommend me sth?

@kirkuz : AWS Athena support for Hudi is just out : https://aws.amazon.com/about-aws/whats-new/2020/07/amazon-athena-adds-support-querying-apache-hudi-datasets-amazon-s3-based-data-lake/
With this your query should not see any duplicate records. The duplicate records could only happen if the table is not defined properly with the correct Input-format.  The reason behind keeping at-least 1 previous version is to prevent queries from failing when concurrent write is happening.

Hi @bvaradar thanks for that. Does it mean that it was released on AWS yesterday? Should I use the latest EMR cluster release?

Yes, @kirkuz . ccing @umehrot2 who can also chime in

Thanks guys! I'll test it out.

@kirkuz yes the AWS Athena support was just released yesterday. Please try out the official support and if you face this issue open a support case with AWS Support, and ping on this thread as well. I will try to get someone from Athena to check it out.

Hi @umehrot2, I can see that it works as it should for now. I'm just wondering if it's possible to create hudi tables in Athena via AWS Glue crawler not only by running CREATE TABLE statement with hudi input format (mentioned here: https://docs.aws.amazon.com/athena/latest/ug/querying-hudi.html)

btw. do you have any community slack channel?

btw. do you have any community slack channel?

Please add your id to https://github.com/apache/hudi/issues/143 and we will add you to slack

Hi @umehrot2, I can see that it works as it should for now. I'm just wondering if it's possible to create hudi tables in Athena via AWS Glue crawler not only by running CREATE TABLE statement with hudi input format (mentioned here: https://docs.aws.amazon.com/athena/latest/ug/querying-hudi.html)

@kirkuz AWS Glue does not officially support Hudi. So this may not be possible right now.

@kirkuz : Kindly reach out to AWS support directly. I am closing this ticket for now.


Storage service collaboration with CUL and NLS

Link to feature: https://wellcomecollection.productboard.com/feature-board/planning/features/7879725


[FR] Craft 4 Compatibility

The CraftCMS 4 pre-upgrade check says this plugin is "not ready" — do you have plans to upgrade this to work with Craft 4?
Cf. https://craftcms.com/docs/4.x/extend/updating-plugins.html#rector

Thanks for the input. ❤ There aren't any plans, yet. That's not because I am not interested in keeping the plugin up to date, but because of my limited spare time to work on projects like this one. Due to other projects I am unfortunately very busy.
Though I have to admit that you made me curious. I haven't thought about Craft 4 at all so far – up to this moment. I can't promise anything right now, but I will try to have a look soon-ish.

Just to chime in, we've added this to our Craft 3 migration project, as people are keen to see this plugin on Craft 4.
I know you mentioned you're still keen to make the update at some stage, so just wanted to put the offer out there for Verbb to take over this plugin and continue development (and migrate to Craft 4). @dueddel

Thanks, @engram-design ! Would be nice to see someone else doing my job. 😅

@dueddel If you want to transfer this repository to verbb or myself engram-design (you might have to add me as a collaborator first, then promote to admin), we'd be happy to continue development!

@dueddel Just wanted to check in about transferring this repo, so we can kick things off?

+1

+1 … we are in deep trouble now as I used this plugin for a huge museum / cultural institution archive, where loads and loads of footnotes (soon in the thousands) are used. this means we can not upgrade to Craft 4, which is now about to become a disaster. anyone experiencing the same? how do you cope with your project?

FYI, dear plugin users, I just made @engram-design an admin of the project to take over the maintenance. Let's cross fingers and hope that he's able to manage upgrading the plugin soon. 😉
By the way, thank you so much, Josh! – Go, Josh, go! 😘

@dueddel Thank you for making this plugin in the first place and being so generous as to share the code with the Verbb folks. I'm eager to upgrade a site that relies on this to Craft 4. Thanks so much!

Added support in 4.0.0


how check if PR already assigned to avoid run the job?

@AugustoCalaca
I am also in trouble.
Did you solve it?

I'm 95% sure it's not possible, it would have to go in the on block to run any machine - and those are limited to workflow events, not other data

I'm trying something like:
jobs:
  assign-author:
    if: ${{ !contains(github.event.pull_request.assignees, '') }}
    runs-on: ubuntu-latest
    steps:
      - uses<EMAIL_ADDRESS>


MHD source

Hooked a generic MHD source updater into the moment app
Dedner's GLM source, simple version

ch is set to max speed from wave solvers
first way of using the source: specify glm_alpha, see Mignone & Tzeferacos, JCP (2010) 229, 2117, Equation (27)
second way of using the source: specify a constant glm_cp

Powell's eight-wave scheme
Orsag-Tang test updated to newer Athena version

results do not look great with either eight-wave or glm

TODOs

more consistent calculation of divB (for eight-wave and glm) and B.grad(psi) (for glm)

right now, simple central difference is used for both divB and grad(psi), assuming uniform Cartesian grid
for nonuniform Cartesian grid

to compute divB one may use face-center B_normal due to linear interpolation, and compute integral of div(B) use Green's theorem (sum Area * Bn/Volume)
to compute B.grad(psi), one may change it into div(B*psi)-(divB)psi, and compute div(Bpsi) and divB using the surface integrals
for GLM/8-wave, since B_normal is indeed changed by the Riemann solver, it might be better to use some proper B_normal value after limiting; but this is not straightforward to compute in the wave-propagation method (in the flux difference method, one may use the value in the obtained numerical flux due to limited reconstruction)

Our design goal is never to exposed internals of equation objects in public-facing APIs. This allows a clean use of the various equation objects via well-defined APIs. Hence, please do not move the struct to the header.
Further, please remember that all public-facing structs and functions MUST be prefixed with the gkyl_ prefix. Private with only local scope need not be prefixed like that.
Once you remove the struct we can review the code and then accept the PR if it looks good.

This code is not ready for merge yet. First, you should not store pointers to eqn objects in this manner, but need to acquire() a pointer. Our eqn objects are reference counted and so storing pointers by simple assignments is very dangerous and can lead to hard-to-find memory leaks.
Second, we can't restrict the code to uniform Cartesian. The wave_prop solver works on any mapped grid and so putting a limitation on the GLM source to only work on uniform Cart grid is not correct. Basically, immediately someone will try a flow in another geometry (like me! I want to do theta-pinch in R-theta geometry, or the spinning mirror). So we need to reconsider carefully how to compute the needed terms properly, as done for rest of the FV solver.
What I suggest is to close this PR without merging at present. Let us talk on Tuesday after my geometry tutorial and we will disucss how to proceed.

Closing this PR without merging. Overall, this is good work and in the correct direction. However, we need to discuss some important (and not trivial to fix like geometry) issues before we can merge this.

This code is not ready for merge yet. First, you should not store pointers to eqn objects in this manner, but need to acquire() a pointer. Our eqn objects are reference counted and so storing pointers by simple assignments is very dangerous and can lead to hard-to-find memory leaks.
This will be fixed.
Second, we can't restrict the code to uniform Cartesian. The wave_prop solver works on any mapped grid and so putting a limitation on the GLM source to only work on uniform Cart grid is not correct. Basically, immediately someone will try a flow in another geometry (like me! I want to do theta-pinch in R-theta geometry, or the spinning mirror). So we need to reconsider carefully how to compute the needed terms properly, as done for rest of the FV solver.
Nonuniform Cartesian is easy. But general geometry needs more thinking.
Nevertheless, overall the results do not look great in Orsag-Tang. It's not clear what I am missing.
What I suggest is to close this PR without merging at present. Let us talk on Tuesday after my geometry tutorial and we will disucss how to proceed.

Let us talk in-person on Tuesday to see where we are and how to resolve. Maybe two of us looking at this together will help.


support more than one parameter

I've tried to create a command with more than one parameters however the library seems not to support it.
code:
vantage
  .command('dbs create <name> <username> <password>', 'creates database')
  .action(require('./cmds/dbs-create'));

help output:
    dbs create <name>>  creates database

action args:
[{ options: {}, 'name>': 'a' }]

Thanks for your input.

Thanks for your help.
I took your pull request and used the code, and just finished it up. It needed some changes in a few more places + tests.
Variadic args are fully supported now! :fire:


Hardened Eth 1.0 voting strategy

Addresses #722.
(Todo: find a clean way to tie-break the max by distance.)

Is there anything left on this @JustinDrake ?

The only thing left is the tie breaker for return max(valid_votes, key=valid_votes.count, default=get_eth1_data(ETH1_FOLLOW_DISTANCE)). Favouring the vote with the smallest distance (as previously specified) makes sense to me.

@protolambda Do you feel like adding a test for get_eth1_vote?

@JustinDrake We don't quite have an executable validator spec, but I'm testing the 50% and 51% cases for eth1 voting in in the beacon-spec, in addition to a full block-processing based eth1 voting test.


Issue of scrolling with SEO sitemap plugin

We are using sitemaps plugin for CMS12. As seen in the picture, it is not possible to scroll the righthand pane and you thus can't access all information.

I believe this is due to the UI changes in the CMS as part of update 461: https://world.optimizely.com/releases/optimizely---pdate-461
The display container for this plugin will probably need some CSS like this:
{
height: calc(100vh - 40px);
overflow: auto;
}

Yes, plus - if you have multiple sitemaps, it's impossible to scroll the list of sitemaps :(

Hi, is there any solution for it? I have multiple sitemaps and it's getting really impossible to use this plugin.

and I have to scroll out to load the page, and add the sitemap :(

have any one found the solution, please share it.

Hi Oleksa,

I don't see this file in entire solution, how should I make this changes?

Is there any alternate way we can add scroll functionality to sitemaps?


archive project / search for maintainers

It seems there areopen issues.
Currently there are way too many custom scrollbar solutions on GitHub and no real collaboration between projects and everyone reinvents the same. Would be great to see a few great solutions instead of many with not enough support by the community.

Thanks for your insight Daniel.
As you can see on top of the README of this project, I  recommend to use https://github.com/utatti/perfect-scrollbar which is the most popular library and has very few issues.
However it seems like Simplebar is still helping some people. Also it exists since 2013 so if there are too many on Github it's probably not my fault (especially as you can see there isn't any pull requests on this repo as oppose to what you say).
I'm open to maintainers and to merging this project to another if authors of other libraries are up for it (that's something I tried without success a long time ago).

I'm open to maintainers and to merging this project to another if authors of other libraries are up for it

That would be good. So far the

Also it exists since 2013 so if there are too many on Github it's probably not my fault (especially as you can see there isn't any pull requests on this repo as oppose to what you say).

You are right. Sorry I've used a text which fit for all repos but 2 not.
Generally it makes sense to bundle the knowhow and take a look at the features (feature comparison tables), different implementations and solutions and try to bring other solutions like perfect-scrollbar forward together but perfect-scrollbar should generally move to an org account to increase the truck factor, improve team collaboration and manage this better using teams with different rights.

feature comparison tables
This would be really useful and I'll try to take the time to do this. That would help me to check again the different solutions on the "market" as well and see how Simplebar positions itself in 2018 and potentially do a move.

Let me know if you need any help and I will try to invest some time to help with this.

I recently chose simplebar above alternatives like perfect-scrollbar because most of them are using non-passive event listener (scroll-block) which has a negative performance impact.
I hope this helps when setting up a feature comparison table.

because most of them are using non-passive event listener (scroll-block) which has a negative performance impact.

This should be easy to fix. Probably with a new option to optin using passive event listeners or use them by default.

This should be easy to fix. Probably with a new option to optin using passive event listeners or use them by default.

You would say so but for example perfect-scrollbar won't implement... https://github.com/utatti/perfect-scrollbar/issues/560

won't implement

It's just because it is not so easy but it would be still possible imo. So it should be possible in general.

This is a very simple solution, and I agree that it has the potential.
Also I just opened a PR #141

How is this related to the issue here and what does your PR solve @neo?

I was commenting that this project is a very nice solution and I think it has potential to become a community project.
The PR is just a side note saying that I'd like to contribute

Ah ok. That was not quite clear.

glad you understood now :D

open / closed issue / PR ratio makes more sense than just the number of open ones, and also mumber of contributors.


fix: Fix cdklabs/cdk-hyperledger-fabric-network#6

Fixes #6
Added bundling using docker as esbuild doesn't resolve the path on fabric-proto npm package.

@vponselvan the change itself looks good, but the build here on Github is failing, likely need to add some flag to the .projenrc.js file to ensure the generated github action files allow docker (https://docs.github.com/en/actions/creating-actions/creating-a-docker-container-action)

Yep. I saw the failure. Looking into.


How to maintain a uniform sphere of gas?

Hi, I'm trying to get a problem to work in spherical coordinates that currently works fine in cartesian.   In spherical there's either an issue with the inner boundary or I'm missing some trick.
I can reproduce the issue with the sog (sphere of gas) test attached.  I configure with
python configure.py --prob=sog  --eos=isothermal --coord=spherical_polar
sog_test.zip

Can you attach a plot here?

No matter what boundary condition I try, I'm losing total energy conservation.

I think gravity is on by default in spherical polar. I think you need to add GM = 0 to <problem>.

Just tried that.  Same plot as above.

Hi,
I'm not sure it'll make a difference, but have you tried second order spatial reconstruction to see if it makes a difference? Also have you checked how things change with resolution?
Another thing you could try would be using a different Riemann solver, so e.g. configuring with --solver roe.
From your plots, it also seems that \rho is not 1 initially, as you are trying to set it in the ProblemGenerator. Don't think that this is the issue, but I would change the '1' to '1.0' in line 57 in sog.cpp
Just a few guesses, hope at least one of them helps.

Thanks for these suggestions.  I just tried a few different lower order methods to no avail.  This test problem is definitely sensitive to the domain size and resolution, but this issue remains at higher resolution.

@msbc gravity is not active, by default

Also, I just verified that cylindrical coordinates handles this problem just fine with outflow BCs.

...no issue either with all of the various versions of my user BC.

If you symmetrize the theta bounds about pi/2, e.g. with x2min = 1.4726215563702154, x2max = 1.6689710972195777, the problem seems to run much better. I can get it to a time of 4 before it blows up.
(Tested with order = 2, integrator = vl2. Note you were compiling with 2 ghost cells but had 3rd order in the input file. Not sure what the code thinks it's doing in that case.)

Thanks, this was the trick I needed (and if I remember you pointed out this symmetry breaking to me years ago).  With the llf flux, the solution seems to be maintained indefinitely.

Hi Tim,
Good to hear from you; hope you are staying safe/healthy~
Can you try the attached athinput (athinput.sog.txt) and pgen (sog.cpp.txt) with the following configure:
python configure.py --prob=sog --eos=isothermal --coord=spherical_polar -mpi --flux=roe
I believe this maintains a uniform sphere of gas to your pre-prescribed <time>/time=100.  Here is the corresponding movie for the density (https://tinyurl.com/y7khvbcr), which just might be the most exciting media you have seen in all of 2020 thus far.
Interestingly, I find this test fails without specifying --flux=roe.  @felker and @msbc, we have seen some funny behavior with HLLD (isothermal) Riemann solver before, right? (I believe for us it was in the context of STS and general EOS)
Best,
Patrick

Hey Patrick, I'm doing well, hope you are too.
It is so exciting to see such boring evolution.  Thanks for having a look!  So now I confirm the roe flux also works, but only with the symmetric bounds around pi/2.  Did you use symmetric bounds?

Wait, you're doing full 0 to pi I see.  I got an exception thrown that the origin had to extend to 0 for that.  So your domain here is r = [0, 10], theta = [0,pi] with polar BCs?

Sorry, I just noticed that you included your pgen and input file.  Okay, so I'm in business.  Thank you everyone for your help!

Glad that it is solved now. But I just took a look into the problem and the underlying problems in the HLLE solver not computing the fluxes correctly.
In lines 121-122 of hydro/rsolvers/hydro/hlle.cpp, al and ar are redeclared such that the al and ar in line 72 are not properly initialized.
Removing Real from lines 121-122 should fix the problem in case someone wants to use the HLLE solver.

Since this is after all the simplest hydro problem there is, I had to tweet about this particular experience
https://twitter.com/thatTimWaters/status/1252383533479297027?s=20
@kahoooo, good catch!  Although that must not be all, as I'm still not able to get HLLE to work.

For completeness, I'm not sure what the history is, but the bug found by @kahoooo isn't in the development branch, so the next public release will fix it.

@trwaters That's weird since it works for me.
Did you also compile with nghost=3 and symmetrize x2min and x2max about pi/2 as @c-white suggested?
Another note is that user boundary conditions should never touch the active zone although it doesn't break the code in this problem.

Let me try again then, I didn't know you verified it worked.
Your comment about active zones is interesting because for years I've been setting these in spherical coordinates.  I could not get the code to maintain the Parker and Bondi analytic solutions otherwise.

I just confirmed that my previously attached athinput and pgen combination are stable to <time>/tlim=100 when using all isothermal Riemann solvers in Athena++ development version.  Therefore, the issue must be isolated to HLLE in public version -- which should be fixed in the next public release.

I also confirmed that my version of sog (with x1min = 1 and x2min/x2max symmetric but not extending to the poles) works for all combinations of vl2, rk2, rk3 and 2nd and 3rd order reconstruction with both Roe and HLLE on the dev version I was granted access to.


Test failures using Python 3.11 and Mypy 0.981

Hi!
Tests are failing on Python 3.11 due to a Mypy error and I'm not sure why. :thinking:
I forked pytest-mypy-plugins and added 3.11-dev Python version to CI tests: https://github.com/Delgan/pytest-mypy-plugins/commit/27f9037897507026519c67fcf016758bd2b3d7a6
You can see the failing test here: https://github.com/Delgan/pytest-mypy-plugins/actions/runs/3166219555/jobs/5155788328
E     ../../opt/hostedtoolcache/Python/3.11.0-rc.2/x64/lib/python3.11/site-packages/mypy/typeshed/stdlib/builtins.pyi:1865: error: Overloaded function signatures 1 and 2 overlap with incompatible return types (diff)
E     ../../opt/hostedtoolcache/Python/3.11.0-rc.2/x64/lib/python3.11/site-packages/mypy/typeshed/stdlib/builtins.pyi:1885: error: Overloaded function signatures 1 and 2 overlap with incompatible return types (diff)

When I manually run mypy on the code base no error is reported, so I'm a bit confused about why it's failing when tests are ran through pytest and this plugin.

3.11 is not supported by mypy yet.

Oh, I see. That could explain the issue, indeed.
Sorry for the noise, then. Feel free to close the ticket.
It could be reconsidered when https://github.com/python/mypy/issues/12840 is completed, I guess.

Let's leave it open, we would need to inlcude tests for 3.11
Thanks for the report! 😄

It appears to be resolved with mypy==0.982 :)


ebrasca/fs Update

Fix small errors in fat32 , cache
Don't display . and .. in directory
Refactor cache , disk

I agree, block-device-read/write-sector look like they're just helper functions for the disk file systems. In fact, block-device-write-sector seems completely redundant and callers could be updated to directly call block-device-write.

The helper functions are specific to the supervisor disk block interface implementation, so I suggest moving them down past the API definition. Also, you have (flet ((read/write-sector ... which calls the (defun read/write-sector. This seems a bit confusing. Maybe use (flet ((%read/%write-sector instead so it's clearer?

Removed flet read/write-sector

Also, changing (defun block-device-read/write-sector to (defmethod block-device-read/write-sector breaks all other block device implementations (that is, breaks USB). These two functions were added to support FAT32. Maybe they should be changed back to functions and moved to FAT32? They really shouldn't be a part of the block device API - I don't think the block device code should be allocating buffers for the user.

Are you sure it does break USB?


Fix casing for DiplomacyActionView loads with Exp1

The diplomacyactionview_CQUI_expansion1.lua file and diplomacyribbon_CQUI_expansion1 file use Camel casing for the "include" statements to load the CQUI game files
update the casing to match how they appear in the file system, as it likely affects Linux

There are several instances of:
include("Civ6Common");
and the file is civ6common.lua.
Can you also fix that quickly?

Actually, it is better to remove this file. As the comment says:
--  CQUI: The ONLY change to this file from the base version (besides this comment) is the include("CQUICommon") declaration below
And it is true, I just checked.
Basically add where necessary cquicommon (also lowercase it, just in case) and make it this "common functions stuff" from another issue.


Undesired Auto-stretch of images to Max Screen Width

This functionality should be overriden, somehow. It is possible
Desired (default) display of images in Obsidian => about 40% width of screen, 60% vertical height

Display for Minimal theme (far too wide, and then image takes up 200% vertical height due to width stretch:

Potential Solutions:

Not stretching
Stretching only to 50% max (if stretched):
https://forum.obsidian.md/t/resize-embedded-content/877/13
Alt tag overloading for custom sizes (taken as the only working option for Obsidian from this markdown article).

Note, I've gotten this (#3) desired functionality by disabling the Minimal theme (minimal overrides custom CSS), and using the following CSS Snippet:
.markdown-preview-view img[alt="sm"] {
  max-width:30%;
  max-height:auto;
}
.markdown-preview-view img[alt="md"] {
  max-width:50%;
  max-height:auto;
}
.markdown-preview-view img[alt="lg"] {
  max-width:70%;
  max-height:auto;
}

This is activated using the following notation:
![sm](Pasted%20image%2020210727084710.png)
![md](Pasted%20image%2020210727084710.png)
![lg](Pasted%20image%2020210727084710.png)

If this is something Obsidian minimal theme would like to support, I'd be very open to opening and submitting a pull request / support any breaking changes that Obsidian pushes (I'm an insider - https://forum.obsidian.md/u/mochsner/summary)
Thanks!
Marc

Obsidian has an existing feature for this — see Obsidian docs
You can define the dimensions via this notation ![[image.png|100x100]]
If you want to disable maximizing the width of images you can also write a snippet like
.markdown-preview-view .image-embed img:not([width]) {
  width:50%;
}

@kepano this CSS override is perfect, and simple! thank you. While obsidian's implementation isn't % based (my desire to use % is something I'm pushing for official obsidian support of), Obsidian's width and width/height override is a great feature to use as well.
In regards to the obsidian-minimal theme setting, this default of auto is exactly what I need to be able to comfortable use the obsidian-minimal theme again. I imagine others might like it as well, because one wouldn't really expect a "theme" (UI) to have a direct impact on how something like images function in the app itself - in particular for folks who paste screenshots of small items (such as UI components) to paste into the editor.
Question: would it be significant development to add this as a toggle-based setting in obsidian-minimal-settings, if there's demand? I'd be happy to help with development for that (preferably in a few weeks) if you think it might be useful!

This is done. See 3.2.2

@kepano you're the man! :beers:


module "Ths" is not installed

Hi, I can't reproduce this bug, would you mind provide more information?

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!
kfeng123, @rsheftel, @geduardcatalindev, @JasonWei512, @spawn88, @PerkinTahmaz, @ellolee, @clayne, @laztb9, @erdemkulunk, @yorickshan, @liuhaoXD, @Surly-maid, @travis103, @laurent-pck, @willowj, @jiebiantiaoqu, @ntzw, @Rjerk, @Imperador-RIC, @miumiuplus, @Tan4ek, @jiaowochunge, @StephenSorriaux, @hayyas49, @nigdaemon, @softgoto, @MarcusXavierr, @holld, @SimingK, @AndyLiamano, @sierra-tang0, @TheFox, @cdroad, @stiw-ef, @rakino, @wsk2007, @broker, @dandanes7, @monomania, @ZeroKill1024, @horacioh, @Mhowser, @yureribeiro, @ArtScInfo1, @seb-10, @xucongyong, @PSheon, @yangjian102621, @gyuri-lajos


Bundle debugging tools

bpf-tools should include:

lldb
lldb-vscode
obj2yaml

This requires -DLLVM_ENABLE_PROJECTS=clang;lldb

taking this one

How about bundling these tools in separate tarballs. They're not required to build on-chain programs. However, cargo-build-sbf spends significant amount of time downloading sbf-tools tarball, and making the tarball bigger would slow it down even more.

I added lldb to sbf-tools tarballs. The tarball sizes for Linux and macOS increased by about 10-15%. Not sure it's worth separating the debugger binaries in an optional tarball.


Restore previous behavior when posting a message

Description
We want the message to disappear immediately from the input bar as before.
Risk
Low
Deploy Plan
Deploy front

Ooooh why ooc? Liked the behavior.

Ooooh why ooc? Liked the behavior.

It makes sense on the new conversation, but in an existing conversation it was breaking the magic of the optimistic UI ✨.

Makes sense. Should/can we keep it for new conversation?

Makes sense. Should/can we keep it for new conversation?

@spolu I kept the new behavior for new convo.
BTW, for replies, we need to enhance our current implementation as if the reply fails, the message (& files) will be lost. I was thinking of tackling it when looking into this


Updated wind direction icon

Updated wind direction icon with text based on currentWeather.wind_from_direction
Working example: https://codepen.io/MarkBoots/pen/ExRZbML

Erm... I just saw.
Why is North pointing down ? Shouldn't it be pointing up instead ?


Better error handling and msg if waiting for block (EOF)

Purpose of Changes and their Description
Better error handliing and msg in case of transactions whose result is not known due to connectivity lost while waiting for next block, so not confirmed whether tx was successful.
Use a boolean to handle such cases specifically, to improve ux.
Link(s) to Ticket(s) or Issue(s) resolved by this PR
PROTO-2561
Are these changes tested and documented?

[ ] If tested, please describe how. If not, why tests are not needed. -- not tested yet, though it is a trivial change.
[X] If documented, please describe where. If not, describe why docs are not needed. -- trivial change, not needed, though if the error messages are documented in detail, then it should be.
[ ] Added to Unreleased section of CHANGELOG.md?

it's unfortunate we have to tell these things by error string, which clearly has multiple meanings, but it will have better UX so... nice!

indeed... ugly but I don't see a better way without a code...


Implement Client#stw

I don't know if that's all, but it's a good start

stw news api has changed (Discord, from @LeleDerGrasshalmi)

Oh I accidentally messed it up

I'll fix it

Let's handle the stw news issue in a separate PR


the editor has the app background and the profile pop-out mixed up

the app background in the editor is trying to change the profile pop-out rather than the actual app background. the profile background works fine however.
sorry if this is the wrong place to put this or if it doesn't go here at all.

Same Here, and even adjusting the positioning it moves the profile pop out.


Minimal number of characters typed before showing noResult

Hi,
I'm using vue-multiselect with ajax to populate the options list, and in order to avoid useless queries the method doesn't actually send a query until the user has typed at least 3 characters. So until that, the options lists stays empty.
The problem with this is that the "noResult" message is shown after the first letter typed in the search bar. Is there a parameter to tell "noResult" not to show up until the user has typed n characters? If not, I think it would be useful to create it.

Yes, I've noticed that, but it takes a few more characters to search the data and populate the drop-down selection box, and I don't know if that's a BUG or not

You have 2 options here:

Use the show-no-results prop with a computed val that sets it to false until the user typed in 3 characters.
Use the noResult slot to display your customized message ( eg. search.length < 3 ?  'Type more characters:No results` )

Yes, I do use it that way

I think this is solved and can be closed @shentao

Thanks @akashrajum7!

i have added the noResult slot like below , and it is not working

test here


chore: update supported versions in the welcome message

Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - [learn more]

/run-e2e aws*


cmake: Export compile_commands.json for all generated code

Presently, if you include the following setting in a CMakeLists.txt file in the project root, the targets listed in the compile_commands.json file generated by CMake are limited to the targets specified in the CMakeLists.txt file.
set(CMAKE_EXPORT_COMPILE_COMMANDS ON)

I would prefer that the compile_commands.json file also include all the compiled code for Zephyr (the drivers, kernel, subsys, etc).  There reason is that I can configure MS VSCode to read the compile_commands.json file and configure its intellisense feature to highlight coding errors.
I tried editing the CMakeLists.txt files in the following Zephyr tree subfolders:
arch/arm
boards
drivers
kernel
soc
subsys
This improved things quite a bit and many of the Zephyr targets now appear in the compile_commands.json file.  However, there are a few that still do not appear.  For instance, sam0_rtc_timer.c is not included as a target.  Neither is wdt_sam0.c.  I know these are being compiled because my application uses them and I have debugged them.  There are likely a few other files that are not included as well, I have not yet performed an exhaustive analysis.
Is there another way to inform CMake so that a complete set of targets will be included in compile_commands.json?

Is there another way to inform CMake so that a complete set of targets will be included in compile_commands.json?

There are several examples in the west documentation that I believe do what you are asking about:
https://docs.zephyrproject.org/latest/guides/west/build-flash-debug.html#permanent-cmake-arguments

Thank you @mbolivar
I was able to get export setting to stick by adding
-DCMAKE_EXPORT_COMPILE_COMMANDS=1

to the west command line.
There is still a VSCode CMake-Tools issue where resolving the location of a header file can result is multiple paths.  I have noticed this happens when there are headers with no target.  That is, there is a <kernel.h> but there is not target "kernel.c".  CMake-Tools attempts to look up "kernel.c" when I try to local <kernel.h>  Since there is nothing to look up in the compile_commands.json, it defaults to some other search process and comes up with multiple paths.


Question: DE with discrete integer variables

Most of these algorithms works with continuous variables. But is it possible for DE (for example) to works with discrete integer variables?
I want to learn more about the variant parameter of 'DE/rand/1/bin'. Would you be able to provide any information about all possibles values?
I am working on a 9 variables problem with a single objective, For the F and CF parameters, can I provide an array of 9 elements for each of these two params?
Thanks!!

Sorry for the delay.

Generally, DE has not been proposed for integer variables. (there might be some variants for other variable types, though)
Please consult the current literature. There are a LOT of DE variants, and pymoo only has a few. I had someone recently contributing a few DE methods, and we had a chat. Let me know (shoot me an email) if you are interested in talking to him.
F and CR are usually defined for all mating procedures in a whole run (thus simply a float variable)

Hi, @zollen!
Regarding your first question, you can create objective functions or constraints that interpret real valued variables as discrete by using operators such as int, round, floor, or ceil inside the functions. When doing so, the output would still be in the real valued domain, so be careful to apply the same rounding operations before interpreting results.
In this article, I present an engineering design example of how to do so  https://link.medium.com/6KLd1v5Hrrb
I was using scipy implementation of DE back then, but same idea.


Correct spelling modelt to model #196

Thanks. Please add a note to the API changes section of the 0.4.0 whatsnew file. Also add your name or username to the contributors list.

Added changes to whatsnew

Thanks, @mangecoeur!
The appveyor build is broken again, but it's not important for this. Closes #196.


Fix exception Invalid LatLng object: (NaN, NaN) with the firefox debu…

…gger in the mobile simluator mode
Fix a problem with the Firefox debugger in simulated mobile mode who pointerdown event received have two touches with the same coordinate which leaflet does not handle correctly.
Fix bug #7566

@mourner have you closed this? Or is this automatically happened with deleting the master branch? There are many other PRs too

PRs which are auto-closed:

#7567
#3955
#6066
#6248
#7364
#6676
#6935

maybe more?

Or is this automatically happened with deleting the master branch? There are many other PRs too

Yeah, it appears this happened automatically, I didn't anticipate this. Let's open an issue and track closed PRs we're still interested in.


Remove information regarding PHP

Openly showing which PHP version a site is running does actually help "bad guys" more than it helps the good ones.
I suggest that this information is removed.
"This webserver is running PHP 5.5.9-1ubuntu4.19."

The site is intended to be visited only by moderators.  I left that there to make submitting bug reports easier.
Anyway, I'll do something about that.  Regardless, people should be using the latest version of PHP whenever possible.

The web app no longer gives PHP information. (Commit at close: e5b36f36d79bfcea6891ca0b2f95266ead01b0fd)


Trouble scraping deck.tk/deckstats.net

For example:
import asyncio
from ruia import Request

async def request_example():
    url = "https://deck.tk/07Pw8tfr"
    params = {
        'name': 'ruia',
    }
    headers = {
        'User-Agent': 'Python3.6',
    }
    request = Request(url=url, method='GET', params=params, headers=headers)
    response = await request.fetch()
    json_result = await response.json()
    print(json_result)

if __name__ == '__main__':
    asyncio.get_event_loop().run_until_complete(request_example())

This simply hangs without resolution. That is, the request is never resolved, and I must Ctrl-C out of it. Scrapy handles this without issue, but I was hoping to transition to ruia. Any ideas?

Also:
poetry add ruia-shell
ruia_shell "https://deck.tk/07Pw8tfr"

never resolves, and must be terminated by Ctrl-C.

This simply hangs without resolution. That is, the request is never resolved, and I must Ctrl-C out of it. Scrapy handles this without issue, but I was hoping to transition to ruia. Any ideas?

Just add a aiohttp session:
import aiohttp
import asyncio
from ruia import Request

async def request_example():
    url = "https://deck.tk/07Pw8tfr"
    params = {
        'name': 'ruia',
    }
    headers = {
        'User-Agent': 'Python3.6',
    }
    async with aiohttp.ClientSession() as session:
        request = Request(url=url, method='GET', params=params, headers=headers)
        response = await request.fetch()
        json_result = await response.json()
        print(json_result)

if __name__ == '__main__':
    asyncio.get_event_loop().run_until_complete(request_example())

never resolves, and must be terminated by Ctrl-C.

I'll fix this issue soon~
Thanks for your report. :)

This did not seem to resolve the issue for me. It still hangs without resolving the request.

@Triquetra
Notice this line: request = Request(url=url, method='GET', params=params, session = session , headers=headers)
Sorry, I just forgot to pass in session parameters to request

Thank you for your help. Including request_session=session as a Request argument solved it.

never resolves, and must be terminated by Ctrl-C.

Fixed, upgrade ruia-shell to 0.0.4:
pip install -U ruia-shell
# or
pip install git+https://github.com/python-ruia/ruia-shell.git


: backtrace:

undefined

@BlueXMarv3l, can you please elaborate on your intention behind this issue?

Closing this issue. If there is some input it can be re-opened


[WIP] chore: add S3 test model

Issue #, if available:
Description of changes: work in progress - adds a TestModel for S3 in the aws-sdk TestModels directory
I've uploaded this code for now to give an easy repro for https://github.com/smithy-lang/smithy-dafny/issues/350.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I am closing this PR in favor of https://github.com/smithy-lang/smithy-dafny/pull/461;
as we are focusing on Python first for S3.


Minor update of python/setup.py and documentation of installation with conda

This PR provides a minor update of how to install alm python module along under conda environment.

Merged! Thank you.


Codelense on light theme is less than 4.5:1.

GitHubTags:#A11y_VSCode;#A11ySev3;#A11yTCS;#A11yWCAG;#DesktopApp;#Visual Studio Code Client;#ColorContrast;#WCAG1.4.3;#DesktopApp;#Win10;#VisualStudioCode-Win32-Apr2021;#Linked:Bug<<2717>>;
Environment details:
Application Name: Azure Stream Analytics Tools for Visual Studio Code
Windows Version: Win10
Visual studio code version : -1.55.1(User Setup)
Repro Steps:

Launch Visual Studio code application with installed Azure Stream Analytics Tools extension .
Tab to "Explorer" section in activity bar.
Press control +shift +P and edit bar will appear.
Type "ASA:Create new project" and press enter to create project.
New project will get created.
Tab to Input item in Project Tree Section
Right click on input and navigate to "ASA : Add Input" using arrow key and press enter.
Select IoT hub from available option using enter key.
Select "select from azure subscriptions" from available option using enter key.
Type any name for input file and press enter to create Input json file.
Tab to "Select Subscription" button.
Start AI4D tool and check the color contrast ratio of select subscription text.

Actual Result:
The color contrast ratio of the text "Select Subscription" button available in input json file is less than 4.5:1.
Note: - This issue is also observed with "Select IoT Hub", "Select Shared Access Policy name" text
Expected Result:
The color contrast ratio of the text "Select Subscription" button available in input json file should be greater than or equal to 4.5:1.
User Impact:
The low vision users may not able to see the "Select Subscription" text available in the input json file.

Can you please upgrade your VS Code to the latest version, we just launched 1.59 and you are on 1.55, as we fixed this in a previous version.
See https://github.com/microsoft/vscode/issues/126899 for the duplicate issue that was fixed.

@misolori I have checked the color contrast in latest version ,ratio is 3.151:1. The color contrast ratio of the text should be greater than or equal to 4.5:1.


[REVIEW] Reseter.css

Project Link:
https://github.com/krishdevdb/reseter.css
You can also join the discord community here: Did it already!
Feel free to check out other cool repositories of EddieHub Community here Did Already. Will Check For Something New!
Thanks for reviewing our project. Thanks, @eddiejaoude. I watch your videos on youtube. They inspired me to get into open source and contribute to it. Your Videos are great!

But, Couldn't understand 1. There is no redundant information. Could you give examples?

Yeah the issue count Label and the GitHub stars are visible in github itself don't think this is needed in the readme as well hope this will explain this a little bit more

Thank you for sharing, I will review on a live stream soon 👍

Reviewing on a live stream https://www.youtube.com/watch?v=w0ELpv3Z4Wk
Looks good, some thoughts:

README is amazing, maybe a bit too long, could remove things like the license as it is displayed already by GitHub
suggest moving the wiki to the docs folder in the repo, so that others can contribute and it can easily be reviewed via PRs etc for knowledge transfer

Reviewing on a live stream https://www.youtube.com/watch?v=w0ELpv3Z4Wk
Looks good, some thoughts:

README is amazing, maybe a bit too long, could remove things like the license as it is displayed already by GitHub
suggest moving the wiki to the docs folder in the repo, so that others can contribute and it can easily be reviewed via PRs etc for knowledge transfer

@eddiejaoude how about I setting up a action using the git subtree command, so that the wiki can be pushed from the docs folder on a push. Meaning that wiki and the docs folder can work simutaneously together? I'll fix in the readme rn. Thanks


Icons don't exist for all default sizes

FILER_ADMIN_ICON_SIZES defaults to '16', '32', '48', '64', but default icons don't exist for all those sizes. I haven't done a full pass, but just saw an error about plainfolder_16x16.png not being found.

Changing the setting to [32, 48, 64] seems to resolve the issue

This is related to #613

@ipmb and @vstoykov Icons are implemented in every size. New Filer Version is already released