Formatter doesn't respect formatter.exs while your code is still compiling

Environment

Elixir & Erlang versions (elixir --version): Erlang/OTP 23 [erts-11.1.7] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:1]

Elixir 1.11.3 (compiled with Erlang/OTP 21)

VSCode ElixirLS version: 0.7.0
Operating System Version: darwin 20.3.0

Troubleshooting

[x] Restart your editor (which will restart ElixirLS) sometimes fixes issues
[x] Stop your editor, remove the entire .elixir_ls directory, then restart your editor

NOTE: This will cause you to have to re-run the dialyzer build for your project

https://www.dropbox.com/s/llgnf7ham9d0hf8/Screen Recording 2021-04-07 at 8.28.49 AM.mov?dl=0
You can reproduce this by deleting the .elixir_ls directory, and then saving a file to trigger the LS to start recompiling, then run the formatter. You observe that it is not respecting the formatter configuration because it is adding parens to Ecto functions/macros that shouldn't have parens.
The screen recording I added above using VSCode, but I have also reproduced it with Neovim builtin language client.
I personally have never run into this race condition, but my coworkers constantly are pushing up code that is clearly formatted incorrectly, and I traced it down to this issue.
Thanks!

Up. I'm facing this issue too. When I run 2 successive commands to format file, formatter.exs is ignored.

This is caused because when compiling code, mix changes the current working directory, and the code that is used to fetch the .formatter.exs and compare file paths against it is dependent on the current working directory. This is related to https://github.com/elixir-lsp/elixir-ls/issues/402

Ahh gotcha. Feel free to close this in favor of the earlier reported issue if that makes things easier for you.

Commenting here because this seems like the central place to address it. This problem was bugging me for a while and I stumbled on https://github.com/elixir-lsp/elixir-ls/pull/394 (the ETS cache fix), which I had checked out and compiled locally. That PR seemed to fully fix the race condition, but looks like it's been dead for a bit.
I'm happy to pick that up and try to get it across the finish line, but I also stumbled on a couple other approaches: https://github.com/elixir-lsp/elixir-ls/pull/687 (blocking until project reload) and @axelson you work on the Elixir core PR.
Is there a preferred approach here? It seems like the Elixir core approach is most robust but this ETS cache fix might be faster to implement? Would love to help in any way I can.

@MrGrinst at ElixirConf this year I've made plans to start some regular pairing sessions with @zachdaniel on the Elixir core PR that I have started but that got stalled out. Hopefully we'll have something to report soon!

Any progress or an idea of the direction to take? I can take it over the line otherwise, I need this kind of small win in a project that is not mine right now...

@DianaOlympos I think this would count as a large win! The way I've been thinking about this is there's two parts to this.
First is a PR to Elixir that changes Mix.Tasks.Format.formatter_opts_for_file/2 to not rely on the current working directory (most likely by passing the root directory into the function). I discussed this with José a while ago and he stated that he is open to the PR: https://groups.google.com/g/elixir-lang-core/c/r5FA-4_Fu9o/m/rZiFgGOJBAAJ
The second is temporarily vendoring Mix.Tasks.Format.formatter_opts_for_file/2 within ElixirLS until ElixirLS can depend on a version of Mix.Tasks.Format.formatter_opts_for_file/2 that does not rely on the current working directory.

Maybe it would be easier to run formatter as a separate OS process instead

Hi all, has anyone found a viable short-term workaround to prevent this from happening (even if it means just ignoring formatting during compilation?) - really anything short of entirely disabling the auto-formatting. It's a huge pain point in my day-to-day.

I have a branch where I did just that with the dot_formatter option:  https://github.com/robmerrell/elixir-ls/tree/formatter_project_dir that I have been using for a couple of months.
It does solve this formatting issue, but I have run into a bug a few times where it can't find the import_deps in my formatter.exs that a quick restart of the lsp server fixes. It hasn't happened often enough for me to be able to reproduce it reliably or make a guess at what might be causing it.

@skbolton @robmerrell while dot_formatter seemingly resolves the issue with changing cwd during build it still has race conditions. See e.g. https://github.com/elixir-lang/elixir/blob/da8d89438271ee042a3295d3723efb04e330288b/lib/mix/lib/mix/tasks/format.ex#L371 the formatter task is making some changes to Mix.Project stack or calls mix compile task https://github.com/elixir-lang/elixir/blob/da8d89438271ee042a3295d3723efb04e330288b/lib/mix/lib/mix/tasks/format.ex#L308. Normal elixirLS build is also triggering mix compile and pushing/popping on Mix.Project stack.

Ok so proper fixing options seem to be

Spawn a new OS process. No idea what would be the impact and/or problems there or how we would handle it
Make Mix.Tasks.Format.formatter_opts_for_file/2 in Elixir take a root option and inject the directory into it. Then wait for elixir to catch up
Vendor Mix.Tasks.Format.formatter_opts_for_file/2 and make it take the root option.
Do both 3 and 2.

Intuitively I want to go for 3 first, to fix the damn problem, then upstream with 2. 1 would be a nice fix, but I really do not want to deal with wiring up everything.
What do you think @lukaszsamson ? I may have time to do 2 today.

Go ahead. It would be best if we could solve the problem upstream

So after a dive, I will just upstream it. This function got far more complex recently, probably due to plugins, which would make vendoring a pain and a lot of code.
Once upstreamed, I will come back to implement it here, in particular because there are at least 5 different ways we call it and validate the correct version to call in the LSP codebase, so I will try to consolidate this a bit

Opened the PR above with elixir, we will see how it goes.

FWIW, I had this issue while I was working with a very large codebase. For me, formatter must be fast and reliable (even if code can not compile) as I added to "on-save" callback.
As I am using Emacs, I just created a library to run an inferior shell iex -S mix run --no-start --no-compile at the project root. Then I would send Mix.Task.rerun("format", ["%s"]) command on file save. This was way faster since we are starting on OS process only once (especially on macos). Emacs has a lot of tooling with similar approach for other languages (CIDER, SLiME etc.) so it is not that hard to build these tooling there.
Interestingly, this approach helped me to speed up running test suite as well. Start iex shell, load test helper code, start all OTP applications required for test, and run recompile() && Tester.run(["%s"]) to test. Since applications are started only once when shell startes, it will be much faster than starting applications every time we run tests. There are some corner cases but it works for common cases. I also created utility tools like json-to-map & map-to-json which converts the selection and copy the output to clipboard.

Ok so this is now merged.
https://github.com/elixir-lang/elixir/pull/12541
I will look at bringing it to lsp.
I quickly grepped the codebase, it seems we have multiple places where we call the formatter, with different styles and using different ways to check which call we should do/elixir version.
Is there an interest in unifying this while i am at it?
If yes do you prefer a version check or a "if function exported" ?
I suppose we would path the project path as root?

Is there an interest in unifying this while i am at it?

Sure

If yes do you prefer a version check or a "if function exported" ?

I prefer version check, it's easier to find and remove them when we bump min version required. Here we'd need to check version anyway as the fun is exported but the new opts will be >= 1.15 only

I suppose we would path the project path as root?

Exactly. By default elixir-ls looks for mix.exs under workspace root but that can be overwritten by settingelixirLS.projectDir. The function ElixirLS.LanguageServer.Server.set_project_dir combines workspace root with  elixirLS.projectDir, calls File.cd! and sets the resulting dir as project_dir property in server state.

Is there an interest in unifying this while i am at it?

Sure

If yes do you prefer a version check or a "if function exported" ?

I prefer version check, it's easier to find and remove them when we bump min version required. Here we'd need to check version anyway as the fun is exported but the new opts will be >= 1.15 only

I suppose we would path the project path as root?

Exactly. By default elixir-ls looks for mix.exs under workspace root but that can be overwritten by settingelixirLS.projectDir. The function ElixirLS.LanguageServer.Server.set_project_dir combines workspace root with  elixirLS.projectDir, calls File.cd! and sets the resulting dir as project_dir property in server state.

Here we'd need to check version anyway as the fun is exported but the new opts will be >= 1.15 only

Note that this would not be a problem @lukaszsamson because the option is optional. We can set it up pre 1.15 and it will just be ignored and works as it does today.


Icon Request: X (fka Twitter!)

Describe the requested icon
X, fka Twitter!
Additional context
Twitter's logo is obsolete now and needs to be updated to X.

Closing as duplicate of #309


Fix timeline section

Trello card: https://trello.com/c/pVhEejgd

@demiwu96  there is a conflict in AboutUs/TimelineSection.js

@demiwu96 there is a conflict in AboutUs/TimelineSection.js

fixed


MSL: OpGroupNonUniformAllEqual erroneously calls simd_broadcast_first()

Hi, thanks for fixing the first error. I have now found something else:
Setup: same specs as in #1343
Consider OpGroupNonUniformAllEqual and its specification:

Evaluates a value for all active invocations in the group. The result is true if Value is equal for all active invocations in the group. Otherwise, the result is false.

This generates two functions. The SPIR-V to generate this is:
    OpMemoryModel Logical Vulkan
    OpEntryPoint GLCompute %main "main" %invocation_id %_
    OpExecutionMode %main LocalSize 1024 1 1

...

    %main = OpFunction %void None %11
    %16 = OpLabel

        %inv = OpAccessChain %wg %invocation_id %uint_0

        %nil = OpGroupNonUniformAllEqual %bool %uint_3 %float_0
        %newv = OpSelect %float %nil %float_1 %float_0
        %this = OpAccessChain %_ptr_Uniform_float %_ %uint_0 %inv
        OpStore %this %newv

    OpReturn
    OpFunctionEnd

Yet, spirv-cross with --msl --msl-version 020200 generates MSL:
template<typename T>
inline bool spvSubgroupAllEqual(T value)
{
    return simd_all(value == simd_broadcast_first(value));
}

template<>
inline bool spvSubgroupAllEqual(bool value)
{
    return simd_all(value) || !simd_any(value);
}

kernel void main0(device _7& _4 [[buffer(0)]], uint3 gl_GlobalInvocationID [[thread_position_in_grid]])
{
    _4._m0[gl_GlobalInvocationID.x] = float(spvSubgroupAllEqual(0.0));
}

Focus is at float(spvSubgroupAllEqual(0.0));. This seems to erroneously call the first declaration of spvSubgroupAllEqual(T value). Indeed, consider fulling the buffer with the following (with f32 types, my subgroup size is 64):
64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 64 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0
The result is:
[1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0]
This, to me, confirms that the wrong function is a) generated and b) called.
Thanks for the project, as always.

Are you sure the wrong function is being called? The second overload is for bool arguments; that is, when the value passed to OpGroupNonUniformAllEqual is itself a bool, which it isn't in this case.
What this operation does is check if all the invocations in the group have the same value for its parameter. There's no native operation for this MSL, so I implemented this by comparing every value to the first invocation's value. If all the comparisons return true, then by the transitive property of equality, they should all be equal to each other. Here you passed a constant 0.0 here, so of course it returned true.

Well, this is embarrassing, you are certainly correct. Somewhere I made a logical mistake and assumed I was using a different opcode: I was supposed to use OpGroupNonUniformAll, which takes in a predicate, but then I must have made an error and accidentally switched the command in my source code to OpGroupNonUniformAllEqual, still assuming I was using the logic of OpGroupNonUniformAll. This bias even passed me so badly I did not notice it when filing for the issue -- the initial idea was comparing a register value to a zero-value float.
Also, thanks for the explanation of the code generation -- I was quite surprised when the code generated two functions, but now I see what is the idea behind it.


background :)

not an issue, but could i have the background image used in the screenshot in the README?

Hey there. It's from Mikael Gustafsson. You can find his art here: https://dribbble.com/MikaelGustafsson


[Test] Require asserts in Sema/implicit-import-in-inlinable-code.swift

One of its run lines includes -swift-version 6 which is only allowed in asserts builds currently.
rdar://96341389

preset=buildbot,tools=R,stdlib=RD
@swift-ci please test with toolchain and preset

@swift-ci please smoke test

preset=buildbot,tools=R,stdlib=RD
@swift-ci Please test with preset macOS


Assert that Commands Were Called with the Right Args

Currently we only check if the command returns Stdout properly and uses exec.Command but we do not assert that the command was called with the right args. Adding the ability to check what is passed to exec.Command would be another way to ensure we are properly testing these functions.

Eh, this is a nice to have but I'm not going to spend time on it.


[BUG] Inflow/Outflow report should not include on-budget transfers

I was very confused by this graph at first because I didn't realize that it was including a transaction that moved money between accounts.
E.g. If I transfered $5k from my checking account to my savings account, then for that month, both the inflow and the outflow would increase by $5k.
I personally don't feel like this is expected behaviour, and it seems to go against the ynab philosophy of separating WHERE a dollar lives with the JOB of a dollar.
I also don't like that if I were to simply transfer money between accounts over and over again (I'm not sure why this would happen, but regardless) then that month will have artificially inflated inflows and outflows.
Thoughts?

I totally agree with you. This made the graph very confusing and obscures the relevant information. In case the current behavior should be kept, it's probably best to introduce a checkbox.

Ah this is a good catch and isn't something I caught when it was implemented. I'm going to mark this one as a bug.

#2580 is similar, it ended up being reverted, but if instead of changing default behaviour we add an extra config I think it should be merged again


add mdTooltipClass to allow CSS customization of tooltips

Bug, feature request, or proposal:
The ability to specify a CSS class for a MdTooltip.
What is the expected behavior?
Developers should be able to customize some tooltips with CSS rules.
What is the current behavior?
Developers cannot apply CSS to a portion of tooltips because they are created in the cdk-overlay-container. Their only option is to apply CSS globally using .mat-tooltip.
What is the use-case or motivation for changing an existing behavior?
A number of feature requests could probably be resolved if developers had mdTooltipClass
#3389

Developers would still need to disable the view encapsulation to reach to the tooltip overlay element.
Or is the main reason to be able to differentiate between different tooltips?


Problem with jax when installing multinerf

Description
I am trying to install the multinerf framework on my ubuntu 20.04 machine but it seems that there is a problem with JAX. When I try to confirm that all unit tests pass I get the following error:
(multinerf) mypc@pc0005:~/user/multinerf$ ./scripts/run_all_unit_tests.sh
.
----------------------------------------------------------------------
Ran 1 test in 2.074s

OK
....
----------------------------------------------------------------------
Ran 4 tests in 13.416s

OK
...............................Mean Error = 0.0803162083029747, Tolerance = 0.1
.Mean Error = 0.08638705313205719, Tolerance = 0.1
........................
----------------------------------------------------------------------
Ran 56 tests in 90.169s

OK
.........2022-12-27 13:00:39.973523: E external/org_tensorflow/tensorflow/compiler/xla/pjrt/pjrt_stream_executor_client.cc:2163] Execution of replica 0 failed: INTERNAL: Failed to execute XLA Runtime executable: run time error: custom call 'xla.gpu.cholesky' failed: cuSolver internal error.
E..PE of degree 5 has a maximum error of 2.5369226932525635e-06
.PE of degree 10 has a maximum error of 6.4849853515625e-05
.PE of degree 15 has a maximum error of 0.002378210425376892
.PE of degree 20 has a maximum error of 0.11622805148363113
.PE of degree 25 has a maximum error of 1.999955415725708
.PE of degree 30 has a maximum error of 1.9999704360961914
....
======================================================================
ERROR: test_integrated_pos_enc (tests.coord_test.CoordTest)
tests.coord_test.CoordTest.test_integrated_pos_enc
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/user/multinerf/tests/coord_test.py", line 254, in test_integrated_pos_enc
    samples = random.multivariate_normal(key, mean, cov, [num_samples])
  File "/home/mypc/anaconda3/envs/multinerf/lib/python3.9/site-packages/jax/_src/random.py", line 625, in multivariate_normal
    return _multivariate_normal(key, mean, cov, shape, dtype, method)  # type: ignore
  File "/home/mypc/anaconda3/envs/multinerf/lib/python3.9/site-packages/jax/_src/traceback_util.py", line 162, in reraise_with_filtered_traceback
    return fun(*args, **kwargs)
  File "/home/mypc/anaconda3/envs/multinerf/lib/python3.9/site-packages/jax/_src/api.py", line 623, in cache_miss
    out_flat = call_bind_continuation(execute(*args_flat))
  File "/home/mypc/anaconda3/envs/multinerf/lib/python3.9/site-packages/jax/_src/profiler.py", line 314, in wrapper
    return func(*args, **kwargs)
  File "/home/mypc/anaconda3/envs/multinerf/lib/python3.9/site-packages/jax/interpreters/pxla.py", line 2136, in __call__
    out_bufs = self.xla_executable.execute_sharded_on_local_devices(
jax._src.traceback_util.UnfilteredStackTrace: jaxlib.xla_extension.XlaRuntimeError: INTERNAL: Failed to execute XLA Runtime executable: run time error: custom call 'xla.gpu.cholesky' failed: cuSolver internal error.

The stack trace below excludes JAX-internal frames.
The preceding is the original exception that occurred, unmodified.

--------------------

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/user/multinerf/tests/coord_test.py", line 254, in test_integrated_pos_enc
    samples = random.multivariate_normal(key, mean, cov, [num_samples])
  File "/home/mypc/anaconda3/envs/multinerf/lib/python3.9/site-packages/jax/_src/random.py", line 625, in multivariate_normal
    return _multivariate_normal(key, mean, cov, shape, dtype, method)  # type: ignore
jaxlib.xla_extension.XlaRuntimeError: INTERNAL: Failed to execute XLA Runtime executable: run time error: custom call 'xla.gpu.cholesky' failed: cuSolver internal error.

----------------------------------------------------------------------
Ran 21 tests in 27.995s

FAILED (errors=1)
......
----------------------------------------------------------------------
Ran 6 tests in 5.412s

OK
..
----------------------------------------------------------------------
Ran 2 tests in 4.408s

OK
.
----------------------------------------------------------------------
Ran 1 test in 0.325s

OK
.
----------------------------------------------------------------------
Ran 1 test in 1.427s

OK
....../home/user/multinerf/internal/math.py:28: RuntimeWarning: overflow encountered in cast
  return fn(jnp.where(jnp.abs(x) < t, x, x % t))
/home/mypc/anaconda3/envs/multinerf/lib/python3.9/site-packages/jax/_src/numpy/lax_numpy.py:1090: RuntimeWarning: overflow encountered in cast
  return _where(condition, x, y)
.
----------------------------------------------------------------------
Ran 7 tests in 4.480s

OK
..........................................
----------------------------------------------------------------------
Ran 42 tests in 37.426s

OK

I've installed jaxlib with GPU support:
pip install -U jax[cuda11_cudnn82] -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html
What jax/jaxlib version are you using?
jax 0.4.1
jaxlib 0.4.1+cuda11.cudnn82
Which accelerator(s) are you using?
GeForce GTX 1080 Ti
Additional system info
Python version 3.9, Ubuntu 20.04
NVIDIA GPU info
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2021 NVIDIA Corporation
Built on Mon_Oct_11_21:27:02_PDT_2021
Cuda compilation tools, release 11.4, V11.4.152
Build cuda_11.4.r11.4/compiler.30521435_0

Can you paste the output of the following command? This will make clear which jax/jaxlib versions you have installed:
$ python -m pip list | grep jax

Hi @jakevdp first of all: thank you for your help! I've updated the question. The versions for jax/jaxlib are:
jax 0.4.1
jaxlib 0.4.1+cuda11.cudnn82

Does anyone have any idea what could be the problem here?

I'm not sure what might be causing this. Can you try running the JAX test suite? if that passes, then perhaps test_integrated_pos_enc is exercising some corner case that is not covered by JAX's tests. Otherwise, maybe updating the cusolver version could help.

I have the same problem. How can I solve it?
jax                          0.4.3
jaxlib                       0.4.3+cuda11.cudnn86


feat: enable linking multiple modules via the manifest

Enable linking multiple modules via the manifest; walk from the main
module to all dependencies. Notable changes:

Modules are preferred over host imports. (If the main module requests "foo",
and there's another manifest module named "foo", the main module will get the
exports of that module, even if the host also provides "foo".)
Only modules reachable via the main module are instantiated. Other modules are
ignored. (Previously, all modules would be instantiated.)
Modules can share memories, tables, globals, and functions. Circular imports are
resolved for functions -- a trampoline is inserted by the runtime -- but all other
circular import types fail.

I have some open questions:

What should our module initialization behavior be for sub-modules with hs_init/initialize/_start functions?
Should we initialize command modules once per import request? (Wasmtime appears to do this, but I need to verify.)
Should we call wasi.initialize once per manifest or many times within the dep tree?

Fixes https://github.com/extism/js-sdk/issues/29.

Super excited about this!

What should our module initialization behavior be for sub-modules with hs_init/initialize/_start functions?

This is tricky - I was just exploring this the other day and noticed that Haskell sub-modules sometimes don't get initialized correctly. In wasmtime this would require each sub-module to be instantiated, so we're only calling those initializers for the main module there. I'm still investigating that path a little bit, but as of now only the main module gets initialized in Rust - I think this is how the Go SDK works too, although the implementation is closer to this one so we would probably have more flexibility there (cc @mhmd-azeez).
Once we exercise some of the linking abilities more we may find that we do really need to initialize all the sub-modules, it could make sense to go through and update all the runtimes to match at that point. Definitely worth some further discussion!

Should we initialize command modules once per import request? (Wasmtime appears to do this, but I need to verify.)

Is this about importing the _start function from a linked command module? My understanding is that once _start returns any initialization should be considered invalid.

Should we call wasi.initialize once per manifest or many times within the dep tree?

This is only done once for the main module in the other runtimes, but I'm not sure how wasi.initialize differs from wasmtime_wasi::add_to_linker so it may be necessary to call it multiple times here.

This is tricky - I was just exploring this the other day and noticed that Haskell sub-modules sometimes don't get initialized correctly. In wasmtime this would require each sub-module to be instantiated, so we're only calling those initializers for the main module there. I'm still investigating that path a little bit, but as of now only the main module gets initialized in Rust - I think this is how the Go SDK works too, although the implementation is closer to this one so we would probably have more flexibility there (cc mhmd-azeez).

Yes, we only initialize the main module in the Go SDK too. Once this is merged, can apply the same logic there too

OK, latest update: I have to initialize a WASI instance for every module in the tree that requests it. (This is the only way I could get the WASI interface to work for side modules that use WASI.) The good news is that enabling WASI is free for modules that don't use WASI now. The bad news is that you might end up with multiple WASI instances for complicated manifests – and I'm not sure the degree to which they'll "agree" on the state of the world (does a fd from one WASI instance map to a fd from the other WASI instance?) but it's the closest I could get the behavior to match wasmtime, I think. The neutral news is that I don't think many users will run into this in practice (Fingers crossed 😅🤞🏻)


Improve docs on ENV file

Overview
As the recommendation is to move the system folder, "the root of your ExpressionEngine install", as stated on docs, isn't helpful information.

Resolves #NN.
Nature of This Change

[ ] 🐛 Fixes a bug
[ ] 🚀 Implements a new feature
[x] 🛁 Rewrites existing documentation
[ ] 💅 Fixes coding style
[ ] 🔥 Removes unused files / code

Related Application Change

Application Pull Request: https://github.com/ExpressionEngine/ExpressionEngine/pulls/NNN

Closing this one in favor of #600


Update FindSDL2.cmake to search for SDL2d.dll

Fix for #147 and #148

Also I think the AppVeyor check is broken.

Sorry, I'm installing SDL2 via Vcpkg, but i can´t find where is the 'FindSDL2.cmake' script. Do you know what folder of vcpkg may contain it? Should I merge it wiht the 'portfile.cmake' script that is in vcpkg\ports\sdl2 folder ?
Thanks!

Sorry, I'm installing SDL2 via Vcpkg, but i can´t find where is the 'FindSDL2.cmake' script. Do you know what folder of vcpkg may contain it? Should I merge it wiht the 'portfile.cmake' script that is in vcpkg\ports\sdl2 folder ?
Thanks!


typo: machinelearningservices/resource-manager/Microsoft.MachineLearningServices

configugation -> configuration
creadentials -> credentials

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-java

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Can one of the admins verify this patch?


Can search still work without tokenization or stemming?

If I don't care about tokenization or stemming, is it still possible to make search work for languages like Korean, or Chinese.  I thought some kind of simple string match would take place if tokenization and stemming are not used.  But that doesn't seem to be the case when I search for Korean and Chinese.  I didn't get any result back.
Is it just a matter of adding a new language in lunr-languages?

Please see my response in #46 ; basically we are dependent upon lunr-languages and the tokenization supports that they provide. If the built-in tokenization does not work for you, then I recommend using normal map/reduce, splitting the string yourself, and emit()ing each token.


[BUG] Issues during the installation process

Hi @Singosgu,
First of all, I would like to express my gratitude for this impressive system.
However, when I was running the installation process, following the guide you provided, I encountered a problem at the "Create Database" step - python manage.py makemigrations with the following issue:
in 'module'
from pandas.core.util.hashing import ( # noqa:F401
ValueError: source code string cannot contain null bytes.
I would appreciate your assistance and any insights on how to fix it.
Thank you.

please use windows cmd .
don't use powershell

Hi @Singosgu,
I have already used Windows Command Prompt, but the same error still appears, which is: "ValueError: source code string cannot contain null bytes" along with the capture.
Please help me.
Thank you.

I think you use xlrd already，you should use xlrd verson less than 2.0.0

I have already used xlrd version less than 2.0.0, specifically version 1.2.0, and have reinstalled it. However, I still encounter the same error.


Use environment variables in the CI pipeline, rather than generating a .env file

Given we have to create the necessary strings and files during build time, it does not seem to be feasible with true environment variables. Instead, we can stick with what we have to create a .env file during build time using the https://pub.dev/packages/dotenv package.


/v2/events behaviour

Did some basic playing with the event stream and found some rough edges.

http --verbose GET https://marathon.mesosphere.com/v2/events

This returns a 405 Method Not Allowed. This appears incorrect: a GET is an allowed method. Surely what we really mean is 406 Not Acceptable, "The resource identified by the request is only capable of generating response entities which have content characteristics not acceptable according to the accept headers sent in the request."

i.e. We want the client to specify Accept:text/event-stream.
Note, "the response SHOULD include an entity containing a list of available entity characteristics" - if we can give the user a hint on the correct Accept header inline (without going to docs), this is even better.

The response has many blank lines for some reason [1].

http --verbose GET https://marathon.mesosphere.com/v2/events/ (note trailing slash) returns a 404. This should work consistently with 1 above. Compare to /v2/info which works identically with and without trailing slash.

 Reason:

    HTTP method GET is not supported by this URL

Powered by Jetty://

Did our Jetty and JSON upgrades in 0.11 make any of this better?

@air: no. Thanks to the command that you supplied that was very easy to check ;)

+1
also, because of the bug, the api doc is incorrect and the http helper there fails to send get request.

@air I consider that isn't a bug, maybe you just should specify the Accept header to "text/event-stream", just like:
curl -H"Accept: text/event-stream" https://<IP_ADDRESS>:8080/v2/events
If you typed like above, you'll receive the events stream.

@anrs it is a bug because "Accept: /" includes text/event-stream as an acceptable response.  I just ran into this myself.  If I tell someone I'll accept anything, they should tell me their response in any format they like, including text/event-stream.  They shouldn't say "you need to ask me more specifically"

@benpillet got it, thank for your explanation.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2691. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


404 error on https://docs.raku.org/language/language/operators.html#Operator_associativity

Problem
broken link https://docs.raku.org/language/language/operators.html#Operator_associativity   yields 404 error
Suggestions
If someone knows an alternate page point at it, otherwise if you know about this please write a page for it

Hi! Where did you find this url? It looks very bogus and shuold be fixed, the correct URL is https://docs.raku.org/language/operators#Operator_associativity

Looks like the commits addressed this, closing.


Russian language

A huge part of the articles began to appear in English, although they have a version in Russian!
I ask you to correct this.

Сведения о документе
⚠ Не вносите правки в этот раздел. Это необходимо для связывания страницы сайта docs.microsoft.com с вопросом на GitHub.

ID: f9a5cd25-def9-18f4-71f4-ecdfa83d03d6
Version Independent ID: acfde4a4-e53b-0adb-d1fc-179f9c055553
Content: Store and retrieve settings and other app data - UWP app developer
Content Source: windows-apps-src/design/app-settings/store-and-retrieve-app-data.md
Service: unspecified
GitHub Login: @mijacobs
Microsoft Alias: mijacobs

Thank you @lxgdark for your feedback! I'm checking on the status of the Russian version and I will let you know what I find out.

Hi @lxgdark, currently, the site navigation menu is translated to Russian, but the article contents are not. We've passed your feedback on to our localization team.


Metalsmith · [metalsmith-sass] Error: invalid top-level expression -> :1:1

My sass file has
@import './components/bower_components/sassarch/sass/style';
My metal script jason has
"metalsmith-sass":     {         "files":["*.css"],         "includePaths":["app/components/bower_components/sassarch/sass"],         "outputDir":"css",         "watch":"*.scss",         "indentedSyntax": false     },
and when I run the metalsmith script I get

Metalsmith · [metalsmith-sass] Error: invalid top-level expression -> main.scss:1:1

I'm not sure what is wrong though...I am new to metalsmith and sass
Any thoughts?

You've got the wrong repository, try it at: https://github.com/stevenschobert/metalsmith-sass

sunnufah

see this though https://github.com/stevenschobert/metalsmith-sass/issues/29 it refers to your plugin

Doesn't seem to be a metalsmith-layouts related issue.


Order fails by hits / path etc.

Is there any plan to add possibility to order failure's list by other columns then ID?
It might be useful to solve issues with most hits first.

sorry i dont understand your question. my redirects plugin just resolves the redirect but not track any hits (like the retour plugin does).
where exactly do you have that ID column? in the panel section of this plugin? in the resolving code?

sorry, I have commited this to wrong repository :-( You are right, it is about retour plugin. My apologies.

16. 10. 2022 v 13:15, Bruno Meilick @.***>:
sorry i dont understand your question. my redirects plugin just resolves the redirect but not track any hits (like the retour plugin does).
where exactly do you have that ID column? in the panel section of this plugin? in the resolving code?
—
Reply to this email directly, view it on GitHub https://github.com/bnomei/kirby3-redirects/issues/21#issuecomment-1279948354, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABWQ6VLYC4BMCGA2B3BUKJDWDPPUVANCNFSM6AAAAAAQT3CCZM.
You are receiving this because you authored the thread.


Add affliction chat template

This can be done with effects

If I create affliction object and try to convert to message with method toMessage() I got error
Error: ENOENT: no such file or directory, open '..../FoundryVTT/Data/systems/pf2e/templates/chat/affliction-card.hbs'

Afflictions are not available for production use.


Word filtering doesn't seem to filter edited messages

if a user sends a message then edits it to contain a filtered word, nadeko doesn't delete it.

I don't think its possible at the moment for a bot to monitor an edited message.

@ItsFriday it is. it's even possible to have nadeko log message edits already with .log MessageUpdated.

Just fixed


[Bug]: NameError: free variable 'steps' referenced before assignment in enclosing scope

Project Version
3.2.6
Platform and OS Version
Windows 11
Affected Devices
Acer Nitro V 15
Existing Issues
No response
What happened?
Traceback (most recent call last):
File "C:\Applio-3.2.6\rvc\train\train.py", line 1030, in
main()
File "C:\Applio-3.2.6\rvc\train\train.py", line 296, in main
edit_config(model_config_file)
File "C:\Applio-3.2.6\rvc\train\train.py", line 285, in edit_config
config_data["train"]["log_interval"] = steps
NameError: free variable 'steps' referenced before assignment in enclosing scope
Steps to reproduce
Train
Sampling Rate 48000
Model Architecture V2
Preprocess
Dataset Path C:/... (1h .wem audio files)
Audio cutting (check)
Noise Reduction (check) Strength 0,5
Process effects (check)
Extract Features
Pitch Guidance (check)
Pitch extraction algorithm rmvpe
Embedder Model contentvec
CPU Cores (auto)
GPU Number (auto)
GPU Information (auto)
Train
Batch Size (auto)
Save Every Epoch (2)
Total Epoch (20)
Save Only Latest (check)
Sync Graph (check)
Save Every Weights (check)
Cache Dataset in GPU (check)
Pretrained (check)
Pitch Guidance (check)
Overtraining Detector (check)
Overtraining Threshold 5
GPU Number (auto)
GPU Information (auto)
Index Algorithm (auto)
Expected behavior
Train, with default settings the issue do not happen
Attachments
No response
Screenshots or Videos
No response
Additional Information
No response

My bad:
The sample rate of the pretrain doesn't match the selected one


has no parent node when collecting docker log by log-pilot

I have a docker container running, which I hope its log can be collected by log-pilot, and from the log I can see the log lable is attached and log-pilot is handling this entry, but I got  daotian "no parent node error ", Could you help provide some clue on this error? what this error means?
2018-04-28 08:35:15 +0000 [info]: #0 fluentd worker is now running worker=0
time="2018-04-28T08:46:18Z" level=debug msg="Process container start event: 345b70d00a3f904675a604331654bf3568cc6fcf0c318a3b09b7ed5084de4532"
time="2018-04-28T08:46:18Z" level=error msg="fail to process event: {start 345b70d00a3f904675a604331654bf3568cc6fcf0c318a3b09b7ed5084de4532 idockerhub/daotian/my-daotian container start {345b70d00a3f904675a604331654bf3568cc6fcf0c318a3b09b7ed5084de4532 map[image:idockerhub/daotian/my-daotian name:mesos-3a86cc63-f295-4122-866a-26b43b31561e]}<PHONE_NUMBER><PHONE_NUMBER>844394022},  daotian has no parent node"

https://github.com/AliyunContainerService/log-pilot/search?q=has+no+parent+node&unscoped_q=has+no+parent+node
In my case, I create a deployment in k8s, and put a log-pilot DaemonSet there already. But one of deployments can't shift logs to aliyun sls. I do something as below,

put the deployment instance in the same k8s cluster and namespace
change the env aliyun.logs name (like aliyun.logs_huaweicloud -> aliyun_logs_huaweicloud-sso)

it's hard to catch the issue with no tool. But recommend to check log-pilot's log for more information.
Not sure if there is a switcher to open debug level for more information


feat: skip unsafe contracts calls

https://github.com/warp-contracts/warp/issues/263

1.2.34-beta.3
Example deployment with a manifest:
const {contractTxId, srcTxId} = await warp.deploy({
      wallet,
      initState: initialState,
      src: jsContractSrc,
      evaluationManifest: {
        evaluationOptions: {
          unsafeClient: 'skip',
          internalWrites: true
        }
      }
    });

NOTE: the evaluation options set for the contract that we want to read must match the evaluation options of the manifest.
So, in order to read the above contract state:
const contract = warp.contract<any>(contractTxId)
      .setEvaluationOptions({internalWrites: true, unsafeClient: 'skip'})
      .connect(wallet);


Applied to 'Address line x' format and modified guidance

Applied generic Address line x format:

Replaced Building and street with Address line 1 and Address line 2. This simplifies things because it makes screenreader interface the same as the visible interface. It also resolves some issues raised in the github page for the address pattern
Replaced Town or City with Address line 3.

Removed County because it adds no value.

You can preview this change here:
Built without sensitive environment variables with commit d6f603c1b637d6878cc6a29d0dba85b80009c478
https://deploy-preview-1369--govuk-design-system-preview.netlify.app

Hi @terrysimpson99 thanks for your contribution. We think this is a significant enough change to the pattern to be worth running it by the Design System working group. The next review is in 2 weeks' time. Would you be okay with that?
We will link to this PR and the preview link. Is there any further rationale you'd like to add?

@christopherthomasdesign Yes, I'm ok. I'll add more rationale in the next few days.

@christopherthomasdesign Here are my further comments. Happy to elaborate on anything if required.
I'll add that the design & guidance has been discussed by several people at https://github.com/alphagov/govuk-design-system-backlog/issues/31, 'https://github.com/alphagov/govuk-design-system/issues/1190' and possibly other places. I hope people will read those pages at least.
I'll add the following for the 'Address line x' format:

It has a label for each field, unlike the current design.
It has no hidden label, unlike the current design. It's better to design the UI so the visible interface is is suitable for screenreaders. In pure design terms, explanations can be found by searching for 'not all screenreader users are blind'. In pragmatic terms, quality control of the visible interface is more extensive and continues through even small design changes. Visible labels are examined by more people, more often than hidden labels.
It supports international addresses. Thus the caveat "Only use multiple text inputs when you know which countries the addresses will come from and can find a format that supports them all." can be eliminated.
It is no more complex than it needs to be - it eliminates the cognitive & physical burdens for the user to match their address to specific labels. There is no statement of the need for specific labels on the UI. It may have been an arbitrary decision. It may have been an unproven belief that users cannot input an address capable of being used by Royal Mail without specific labels on the UI. It may have been an unproven belief that address database functions require data (such as town) in a specific field on the UI (such as the third field) and will not function correctly otherwise.
It no longer enforces the county field. This reduces user effort and eliminates one field on the UI. The Royal Mail address/postcode finder does include county is (see: https://www.royalmail.com/find-a-postcode) nor do address lookup tools. This has been the case for about ten years (see: https://www.poweredbypaf.com/wp-content/uploads/2017/07/Latest-Programmers_guide_Edition-7-Version-6.pdf).

That's great, thanks @terrysimpson99 !

Any further thoughts on this?

@hannalaakso asked me summarise the evidence here. I'm not sure if evidence is the right word. I assume people will read what I said on 10 October 2020 in this same pull request (https://github.com/alphagov/govuk-design-system/pull/1369#issuecomment-706534770).
This can be split into multiple decisions, each somewhat independent of the other:
Decision 1: Remove county field. This reduces user effort and eliminates one field on the UI.
I've heard people suggest county can be useful if the postcode is absent AND the postal town shares a name with a non-postal town AND the intended destination is the non-postal town. However, that is an edge case. Neither that combination of circumstances nor any other error scenario has been mentioned by Royal Mail or by the PAF advisory board (who made the decision to delete counties from the PAF).
The term 'county' is ambiguous anyway. It leads to users repeating the town name in the county field ('Bristol', 'London'). Users give inconsistent responses and may give the: 'former postal county', 'historic county', 'traditional county', 'preserved county', 'administrative county', or the adjacent county (this is common in cities that overlap boundaries).
This isn't just a human failing, some machines have similar failings, for example Argos thinks Manchester is in Lancashire (it isn't). The Royal Mail address/postcode finder does not mention county (see: https://www.royalmail.com/find-a-postcode) nor do address lookup tools. This has been the case for about ten years (see: https://www.poweredbypaf.com/wp-content/uploads/2017/07/Latest-Programmers_guide_Edition-7-Version-6.pdf). Royal Mail doesn't use or recommend any of the three variants of county as a means of delivering post in the absence of postcode. See also https://en.wikipedia.org/wiki/Postal_counties_of_the_United_Kingdom
Decision 2: Ensure each field has a visible label and eliminate the hidden label. It's better to design the UI so the visible interface is is suitable for screenreaders. In pure design terms, explanations can be found by searching for 'not all screenreader users are blind'. More designers and stakeholders check visible labels than check hidden labels. Errors in hidden labels can go undetected for longer than they would with visible labels.
Decision 3: Apply the 'Address line x' format. This makes the design scalable - designers simply need to choose how many lines they want. It also means the same basic design is suitable for UK only addresses and to international addresses.  By the way, this is a reason why some people want to use text area for address input.
I hope that helps. Let me know if you need me to do anything else.

Split into two separate PRs:
https://github.com/alphagov/govuk-design-system/pull/2046 and https://github.com/alphagov/govuk-design-system/pull/2045

Closing this because we split the work into https://github.com/alphagov/govuk-design-system/pull/2046, https://github.com/alphagov/govuk-design-system/pull/2045 and https://github.com/alphagov/govuk-design-system/issues/2130


⚠ API - Lens has degraded performance

In a9fb34b, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 503
Response time: 329 ms

Resolved: API - Lens performance has improved in fb1eb54 after 8 minutes.


What databases are supported?

Hi, I would like to know what databases can be used with Ent, previously I've been using it with Postgres and works perfectly, but know I would like to try it with a graph database such as ArangoDB or NEO4J.
Being Ent a graph oriented orm, it could be good to describe what databases can be used.

Hello @ltns35, you can navigate through the available drivers from the dialect sub-package of the project. For instance: https://github.com/ent/ent/blob/a27f753bc800ce0a20d3158fc1cdb4d284e9b068/dialect/dialect.go#L18-L23


Update the root URL

Is there a way to update the root URL after crawling it by adding the new news articles instead of re-crawling the root URL again?
Thanks

or just crawling today's news articles

or just crawling today's news articles

I got that but it did not work when I modified the config file as shown below:
# It is also possible to only define one date, assigning the other variable the value 'None' to create an half-bounded
# interval.

start_date = '2021-01-13 00:00:00'
end_date = 'None'

# If 'True' articles without a publishing date are dropped.
strict_mode = True

I got that but it did not work when I modified the config file as shown below:
# It is also possible to only define one date, assigning the other variable the value 'None' to create an half-bounded
# interval.

start_date = '2021-01-13 00:00:00'
end_date = 'None'

# If 'True' articles without a publishing date are dropped.
strict_mode = True


feature(notifier): ability to set foreground colors for notifications individually

Did you check the docs?

[X] I have read all the snacks.nvim docs

Is your feature request related to a problem? Please describe.
When a warning message pops up, the foreground color is set to white (depends on the current theme), so is the case for every other type of notification.
Describe the solution you'd like
a opt section with a function to edit specific highlight groups (like noice) would be really nice.
Describe alternatives you've considered
Noice.nvim is probably the only alternative I'd use.
Additional context
No response

You can inplement your own style. Check the docs / code


Added support for multi-column table rendering (sharing the header)

Coverage increased (+0.8%) to 45.037% when pulling ede37c40828394f718e4001d66a0316a850a8cf4 on multicolumns into 7954b1d17ecad510fba5156201c38bfc40b30dfb on master.


Fix apt tasks so they work on Bionic behind a proxy

It seems that there is a bug when adding an apt repo key in Bionic, one of the packages involved (apt-key, gpg or software-properties) no longer honours HTTP proxy settings:
https://bugs.launchpad.net/ubuntu/+source/software-properties/+bug/1433761
To get around this issue we can use a lookup as described here:
ansible/ansible#31691 (comment)

Closed by #13


ARM build

mediatracker The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested  0.0s
For your information, I could not use docker-compose as there is no arm64/v8 build.
Keep up the great work!

Now there are amd64, arm/v6, arm/v7 and arm64 builds available for both stable and unstable releases

Thank you sir!


config ionic_www_dir

ionic_www_dir default is www,
for some reason,i config it to another folder,
then i do 'sudo ionic cordova platform add ios',
i found there was no www files in 'platforms-ios-www' folder.
how can i solve it?
all those will all right when i delete my ionic_www_dir config.

This issue was moved to ionic-team/ionic-app-scripts#1409


Revenue and Vat checklist Report display error

Hello,
I'm having an issue while generating the report "Revenue and Vat Checklist" on the Cronus BE Company.
Environment Information: Business Central Saas
Version: 15.0.36560.0
Localisation: Belgium (BE)
The pdf or excel is not generating, and while previewing it i got the error message below:

Regards,
Rishta

This area is not suitable for reporting bugs, please use technical support or create ticket online.


How to open contacts app when adding a contact?

Hello guys! I want to save a contact. I don't want to save it 'directly' to the address book, but to open the contacts app, filling the form fields with the sent data. How can I do this? And if is not possible with this library, do you know any other option?
Look at this image: https://i.stack.imgur.com/2ryDu.png
Any help will be greatly appreciated!

I don't think this library does this. Instead you'd want to launch the native contacts app. In android I think you'd use an intention. I'd search through android studio's samples and skim https://developer.android.com/reference/android/content/Intent.html#CATEGORY_APP_CONTACTS This vid is a good overview https://www.youtube.com/watch?v=95AARxHoupA I'd also check https://developer.android.com/guide/components/intents-filters.html#ExampleExplicit
and lesson 3 of https://www.udacity.com/course/new-android-fundamentals--ud851
feel free to ping here if you think the issue should remain open


[18.09] Bump SwarmKit to b0bc4017ad110cd20898d8c44be03c1e78e4e979

Includes: https://github.com/docker/swarmkit/pull/2892

Edited TestServiceWithDefaultAddressPoolInit to validate
dynamic ingress network subnet

Signed-off-by: Arko Dasgupta<EMAIL_ADDRESS>
- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

If this is still a desirable backport, it unfortunately needs to be rebased.

closing this as this has already been integrated with https://github.com/docker/engine/commit/e06f07ef337ab890f211397d6b408b75a2512dc5


feat: track token usage in Langfuse (and fix memory leak)

I suggest to track token counts in Langfuse as they are captured by openwebui
cc @samuelpetermoshi who suggested this
thanks to @mardet for sharing his implementation with me privately

LGTM, Thanks!

thanks for the quick review/merge


Double brackets around array filtered by JsonPath.read

With a given JSON string
{     "CONFIGURATION": {         "GLOBAL_PARAMETER": {             "SOURCE": {                 "SYSTEM": "JMS",                 "APPLICATION": "SAP PI",                 "COMMUNICATION": {                     "ADAPTER": "SAPEIB",                     "PARAMETER": [{                         "VALUE": "WBBDLD",                         "NAME": "MESTYP"                     },                     {                         "VALUE": "",                         "NAME": "SNDPRN"                     }]                 }             }         }     } }
this statement
String paramArr = JsonPath.read(headerJsonString, "$..PARAMETER").toString()
produces a String
[[{"NAME":"MESTYP","VALUE":"WBBDLD"},{"NAME":"SNDPRN","VALUE":""}]]
with double brackets around, an array within an array.

That's expected. The outer array is the array that represents the results from your query. In this case, there is one result, represented by the inner array.


Private Command decoding calculates command length incorrectly, causes Exception

>>> import threefive
>>> cue = threefive.Cue("/DAdAAAAAAAAAP/wDP8AvGFOZGVhZGJlZWYAABizs+c=")
>>> cue.decode()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/davide/work/oss/scte35-threefive/threefive/cue.py", line 90, in decode
    crc = hex(int.from_bytes(bites[0:4], byteorder="big"))
                             ~~~~~^^^^^
TypeError: 'NoneType' object is not subscriptable

The current method in PrivateCommand.decode  doesn't work because when it is called self.bites contains the remainder of the cue unparsed up til that point ie it has the descriptor loop length, descriptors and crc. The length is set incorrectly, and this causes the remaining decoding to fail.
As an aside, the identifier length is also incorrect (should 32 bits).

Firstly, where did you get the Base64 Cue?
Secondly, give me a minute to figure it out.... :)

Does your patch fix this one?

Sadly not. I genuinely don't know how to solve this other than by trusting the splice_command_length. But even then it can be set to 0xFFF and then you're out of luck again.

first , you rarely see private commands,
second, I don't many 0xFFF lengths  these days.
Here's what I'm thinking,
In cue.py I will set it  from info_section for all the Splice Commands BEFORE they run decode, and then after decode I will set the info_secrtion command length from the command command length, since all the rest of the commands can calculate their own length.
So in PrivateCommand don't set a command_length, will just keep the info_section length.
def _set_splice_command(self, bites):
"""
Cue._set_splice_command parses
the command section of a SCTE35 cue.
"""
sct = self.info_section.splice_command_type
if sct not in command_map:
return False
self.command = command_mapsct
self.command.command_length=self.info_section.splice_command_length  # <---- set it from info section
self.command.decode()
del self.command.bites
self.info_section.splice_command_length = self.command.command_length # <---- set it from command
bites = bites[self.command.command_length :]
return bites

Love it :+1:

It's close, the extra 6 bytes on the end is probably descriptor loop length and crc
a@fu:~/build/SCTE35_threefive$ pypy3
Python 3.9.16 (7.3.11+dfsg-2+deb12u2, May 20 2024, 22:08:06)
[PyPy 7.3.11 with GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>>> from threefive import Cue, PrivateCommand
>>>> pc = PrivateCommand()\
....
....
>>>> pc.identifier= int.from_bytes(b'ABCD',byteorder="big")
>>>> pc.private_bytes=b'AdrianofDOOM'
>>>> cue=Cue()
>>>> cue.command =pc
>>>> cue.encode()
'/DAhAAAAAAAAAP/wEP9BQkNEQWRyaWFub2ZET09NAAA9zmva'
>>>> cue.xml()
b'AdrianofDOOM\x00\x00=\xcek\xda'
<SpliceInfoSection xmlns="https://scte.org/schemas/35" ptsAdjustment="0" protocolVersion="0" sapType="3" tier="4095">
   <PrivateCommand identifier="1094861636">
      <PrivateBytes>b'AdrianofDOOM\x00\x00=\xcek\xda'</PrivateBytes>
   </PrivateCommand>
</SpliceInfoSection>

Now it works .
>>>> from threefive import Cue, PrivateCommand
>>>> pc = PrivateCommand()
....
....
>>>> pc.identifier= int.from_bytes(b'ABCD',byteorder="big")
>>>> pc.private_bytes=b'AdrianofDOOM'
>>>> cue=Cue()
>>>> cue.command =pc
>>>> cue.encode()
'/DAhAAAAAAAAAP/wEP9BQkNEQWRyaWFub2ZET09NAAA9zmva'
>>>> cue.xml()
b'AdrianofDOOM'
<SpliceInfoSection xmlns="https://scte.org/schemas/35" ptsAdjustment="0" protocolVersion="0" sapType="3" tier="4095">
   <PrivateCommand identifier="1094861636">
      <PrivateBytes>b'AdrianofDOOM'</PrivateBytes>
   </PrivateCommand>
</SpliceInfoSection>

Now, it works.
>>>> from threefive import Cue, PrivateCommand
>>>> pc = PrivateCommand()
>>>> pc.identifier= int.from_bytes(b'ABCD',byteorder="big")
>>>> pc.private_bytes=b'AdrianofDOOM'
>>>> cue=Cue()
>>>> cue.command =pc
>>>> cue.encode()
'/DAhAAAAAAAAAP/wEP9BQkNEQWRyaWFub2ZET09NAAA9zmva'
>>>> cue.xml()
b'AdrianofDOOM'
<SpliceInfoSection xmlns="https://scte.org/schemas/35" ptsAdjustment="0" protocolVersion="0" sapType="3" tier="4095">
   <PrivateCommand identifier="1094861636">
      <PrivateBytes>AdrianofDOOM</PrivateBytes>
   </PrivateCommand>
</SpliceInfoSection>

Almost! Something weird is happening if you put a descriptor on the end - see the CUEI:
>>> import threefive
>>> cue = threefive.Cue()
>>> cue.command = threefive.PrivateCommand()
>>> cue.command.identifier = 12345678
>>> cue.command.private_bytes = b'test'
>>> d = threefive.descriptors.AvailDescriptor()
>>> d.provider_avail_id = 9876
>>> cue.descriptors.append(d)
>>> cue.xml()
<SpliceInfoSection xmlns="https://scte.org/schemas/35" ptsAdjustment="0" protocolVersion="0" sapType="3" tier="4095">
   <PrivateCommand identifier="12345678">
      <PrivateBytes>test
CUEI</PrivateBytes>
   </PrivateCommand>
   <AvailDescriptor providerAvailId="9876"/>
</SpliceInfoSection>

That is weird...... hmmmm it has to be coming of the avail descriptor , I got to run out for  while , I'll hit when I get back.

The bites are not aligned correctly, that is happening when a cue is encoded. I'll sort it out tonight, hopefully .
We are super close , great job man.
>>>> ad.provider_avail_id=1223435
>>>> ad.encode()
b'CUEI\x00\x12\xab\x0b'
>>>> cue.descriptors.append(ad)
>>>> cue.encode()
'/DArAAAAAAAAAP/wEP9BQkNEQWRyaWFub2ZET09NAAoACENVRUkAEqsLlX8/mw=='
>>>> cue.xml()
b'AdrianofDOOM\x00\n\x00\x08CUEI\x00\x12'
<SpliceInfoSection xmlns="https://scte.org/schemas/35" ptsAdjustment="0" protocolVersion="0" sapType="3" tier="4095">
   <PrivateCommand identifier="1094861636">
      <PrivateBytes>AdrianofDOOM
CUEI</PrivateBytes>
   </PrivateCommand>
   <AvailDescriptor providerAvailId="1223435"/>
</SpliceInfoSection>

Boom goes the dynamite.
from threefive import Cue,PrivateCommand,AvailDescriptor

cue=Cue()

pc = PrivateCommand()
pc.identifier= int.from_bytes(b'ABCD',byteorder="big")
pc.private_bytes=b'yoyo'

ad =AvailDescriptor()
ad.provider_avail_id=37

cue.command=pc
cue.descriptors.append(ad)

cue.encode()
cue.show()
print(cue.xml())

<SpliceInfoSection xmlns="https://scte.org/schemas/35" ptsAdjustment="0" protocolVersion="0" sapType="3" tier="4095">
   <PrivateCommand identifier="1094861636">
      <PrivateBytes>yoyo</PrivateBytes>
   </PrivateCommand>
   <AvailDescriptor providerAvailId="37"/>
</SpliceInfoSection>

Nice one! One last thing I was thinking about over night: what if private_bytes needs to be something like
cue.command.private_bytes = b'\x00\x01\x02\x03'

I think this will currently result in <PrivateBytes></PrivateBytes>?
Maybe it isn't worth all this effort for a little used feature ...

Oh no, we're not stopping now, we are in way too deep. :)
I just did another commit, a found a few more bugs.
The good news is  I've fixed a bunch of problematic code in cue.py
that hadn't been touched in years and was kind of janky
Do  you want to take over PrivateCommand for a while  and sort the private_bytes?
I need to take a shower.
By the way, thanks again for helping out, I really appreciate it.
We are doing well. pylint score is 9.77 and  cyclomatic complexity is  at 2.06
git pull

Really, Private Command wasn't the problem, today none of my changes were in commands.py, it was mostly cue.py.

There is still something wrong with cue.bites, the descriptors are off when I parse streams


boot2docker init downloads latest ISO, regardless of b2d version

I installed v1.6.2 of boot2docker and wanted to use that version, but it downloaded the latest ISO:
$  boot2docker up
error in run: Failed to get machine "boot2docker-vm": machine does not exist (Did you run `boot2docker init`?)
$  boot2docker init
Latest release for github.com/boot2docker/boot2docker is v1.7.0
Downloading boot2docker ISO image...
Success: downloaded https://github.com/boot2docker/boot2docker/releases/download/v1.7.0/boot2docker.iso
	to /Users/carmstrong/.boot2docker/boot2docker.iso
Generating public/private rsa key pair.
Your identification has been saved in /Users/carmstrong/.ssh/id_boot2docker.
Your public key has been saved in /Users/carmstrong/.ssh/id_boot2docker.pub.
The key fingerprint is:
e0:af:c8:61:ea:76:ae:86:fa:1d:b8:43:5a:56:04:1f<EMAIL_ADDRESS>The key's randomart image is:
...
$ boot2docker version
Boot2Docker-cli version: v1.6.2
Git commit: cb2c3bc

Yeah, I ran into this too when trying to roll back to 1.6.2. Theoretically, you should be able to use --iso-url to specify the ISO you want. But I agree, boot2docker should get the ISO version for the version of boot2docker

But I agree, boot2docker should get the ISO version for the version of boot2docker

:+1:

Sorry, this repository is long-since deprecated in favor of Docker Toolbox (whose usage is now also discouraged in favor of Docker for Windows and Docker for Mac).


[Feature request] Add matcher to test asynchronous handlers

Currently, I have to write spec like this:
before do
  publisher.subscribe(Listener, async: true)
  publisher.broadcast(:event, args)
end

specify { expect(Wisper::ActiveJobBroadcaster::Wrapper).to have_been_enqueued.on_queue('default').with('Listener', 'event', args_matchers).exactly(:once) }

It would be nice if we have a shorter version of these specs, for example (I'm not good at naming stuffs):
specify { expect { publisher.broadcast(:event, args) }.to enqueue_listener(Listener, 'event', args_matchers).exactly(:once) }
specify { expect { publisher.broadcast(:event, args) }.to enqueue_subscriber(Listener, 'event', args_matchers).exactly(:once) }

# Or
before { publisher.broadcast(:event, args) }
specify { expect(Listener).to have_been_enqueued_from_broadcast('event', args_matchers).exactly(:once) }

I'm guessing have_been_enqueued is part of rspec-rails? If so, it couldn't be added to this gem as it is not framework specific.

I understand. I will write a custom matcher for this.


Tenant links are no longer replaced in comments

Tenant links are no longer recognized in comments and replaced by a relative version.

Assigning to @mrvisser for fix

Believe it or not, this only happens if the tenant host contains the letter s or n in it.... our tests cover link replacing with cambridge.oae.com
:(

PR is submit here: https://github.com/oaeproject/Hilary/pull/1258

Closing in favour of oaeproject/Hilary#1258


Create GoQuorum/Besu Clique Network

this may require a way of making 2 different genesis files (if they aren't the same across besu/quorum)

Tests need to include:

Sending/mining transactions
Turning nodes on/off & mining continues

Do not need to test:

full gammut of json rpc - i.e. we already know that the validator lists are valid (proven in Besu/Quorum AT)


Sat active database missing

Set active database is missing, Can't change database to execute sql. My version is 6.3.3 and database is posgresql.

Already implemented.


docs: add note about mat-app-background to theming guide

We used to have this note in the old theming guide, but it looks like it was lost in-between the rewrites.
Fixes #24090.

The feedback has been addressed.


Wrong css propery for td.today:before

Browsers don't show corner for today with this short syntax:
.bootstrap-datetimepicker-widget table td.today:before {
    ...
    border: 0 0 7px 7px solid transparent;
    ...
}

Must be corrected. Like this:
.bootstrap-datetimepicker-widget table td.today:before {
    ...
    border-width: 0 0 7px 7px;
    border-style: solid;
    border-color: transparent;
    ...
}

Or this:
.bootstrap-datetimepicker-widget table td.today:before {
    ...
    border: 7px solid transparent;
    border-top-width: 0;
    border-right-width: 0;
    ...
}

Thanks. This was fixed in the dev branch LESS files. It will make it into the master soon.


tests: revert to previous linter

Definition of done (v0.0.2)

[ ] Automated integration test

feel free to merge once tests passed :)


Task 14

Extract data from Vajrang.org and saved it in a excel file

SourceCode/2023/Oct2023/Basics/Khantil Desai/Task 14/project.json
NoFile

Score: 1/2 Automation Hub URL is missing from the project settings. To rectify this, please go to the "Project Settings" and add in the URL.

Score 2/2 - The UiPath.Mail.Activities package is attached but not used. This is unnecessary and could slow down execution. Please, clean up.

Score 2/2 - The UiPath.Testing.Activities package is also not used. For the same reasons stated above, kindly remove this unused package.

Main.xaml

Score: 1/2 - The variable names of your DataTable do not comply with the standard convention which starts with 'dt_'. Please, rename your variable to follow this convention. For example, 'dt_ExtractDataTable'.

Code completion: 3/6
Execution: 4/6
Annotations and Comments: 2/6
Take this review into consideration and work on the pointed areas.

@Khantil12
Task -14
code verified.


Organisation details page - bug fixes : Testing GithubProjects

Related to Pivotal: https://www.pivotaltracker.com/story/show/131749453

When typing in website field it displays message “please enter a web address valid.” > should be “please enter a valid web address”
Tooltip for ? icon appears to be for template creation area and/or accompanied the Description text box that we removed. It doesn’t make sense here > need to delete Tooltip
Organisation type field (at bottom of page for editing org details) should have a drop-down list or radio buttons to select from: Funder, Institution, others?

I fixed issues 1 and 2 here @stephaniesimms
Do we actually want to allow the organizational admin to change their own org type? The 'funder' type will change the behavior of their templates and allow them to be customized by external users if I understand the workflow properly.


A note to the mods

Ok, guys and gals and mods and modesest, hear me out. Please onboard more contributors that will have the right to merge with the main branch. The last merge was 3 days ago and some users are complaining that their pull has not been merged for two weeks.  I also noticed that the practice was skipping pull requests. This is purely biased and unacceptable. Maybe you should change the REDME.md file header from...
Ordinals Collections Standards
to
Our Collections Standards

i kind of thought this was a global repo too that would help to promote all ordinals. I am sad to see i can't even update my collection anymore. I know its a lot of work but yet...  We can see updates for promoted collections. If this is not supposed to be a global repo to help ordinals grow, let us know ordinals wallet. Thank you

Hello @StellarStoic , thank you for your feedback. Please bear with us while we are working on everything. I personally do have much love for smaller collections as well. Thank you for your understanding. And let me know if you have a current issue open for one your collections.


cant find attachments in query even with filled ATTACHMENTS_DIR

I have set "Linked Attachment Base Directory" in Zotero, and all my attachments can be launched in Zotero directly.
but in ZotHero, only the items with "URL link" and "File" are displayed as owning attachments, what we want is defiantly "File link", but none of the file link attachments is shown on my query.
I have tried to set the ATTACHMENTS_DIR,  it makes any difference.
My file links are generated by ZotFile, and my attachments are mainly stored in the subdirectory (or even more "sub") of the "Linked Attachment Base Directory" in Zotero(because I define the subdirectory name as collection path in Zotero with ZotFile)
Thanks for your workflow, I hope my message will help, maybe the issue is related to the paths of file links related to the subfolder?

I have set "Linked Attachment Base Directory" in Zotero

Then ATTACHMENTS_DIR must be set to the same path or the workflow cannot find the attached files.
I can't tell you what the problem might be, I'm afraid, because I can't replicate your problem from this information.
Please check the log file (enter zotconf in Alfred and choose Open Log File), and search for "attachment". There should be a warning message for each attachment the workflow can't locate telling you what the problem is.

like

10:50:57 zotero.py:324 WARNING  [zotero] can't access attachment "Functional Materials/Thermoelectric/Concept/Structure/Organic/Huang et al_2016_Bismuth Interfacial Doping of Organic Small Molecules for High Performance.pdf": attachments directory is unset

related logs all end with "attachments directory is unset", no matter I leave  ATTACHMENTS_DIR blank or set it as the abs path of "Linked Attachment Base Directory"

but it is fixed after I filled the ATTACHMENTS_DIR and Reload Zotero Data, Thanks.
I think it will be nice if you mentioned this extra reload if the user doesn't set the right ATTACHMENTS_DIR before the first query.


Default location of data-directory should be documented

Neither in the Database config nor  the CLI documentation mentions what the "default" location of the databse directory is.
As it seems to turns out, it is:
data.directory='[JAR]/data'
thus, the h2-db files reside in the .m2-repository at

[.m2-repository]/org/owasp/dependency-check-data/3.0/

Would have saved me an hour if I found this documented somewhere.
If you point me to the right location (is the documentation checked in?), I'm willing to update it myself.
Keep up the good work.

The documentation is in the maven site; for this the best place to document it would likely be: https://github.com/jeremylong/DependencyCheck/blob/master/dependency-check-maven/src/site/markdown/configuration.md

I don't have ~/.m2 folder and do you know where can I find the directory?

Are you using the CLI, Ant, gradle, maven????

I am using the SBT dependency check: https://github.com/albuch/sbt-dependency-check

I found it. For the SBT one, the data directory by default is: ~/.ivy2/cache/org.owasp/dependency-check-utils/jars/data


Windows support

Description of the changes
Please provide a clear and concise description of the changes you made.
Added support for windows using the GetExtendedTcpTable, GetExtendedUdpTable, OpenProcess, and TerminateProcess I used the windows-sys crate for implementing this
Related issue(s)
If this PR is related to an existing issue, please link to it using the Fixes #issue_number or Closes #issue_number syntax.
Closes #12
Type of change
Please select one or multiple of the following options:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code cleanup (refactoring or improving code quality)
[x] Documentation update (adding or updating documentation, updating README)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Additional information
Add any other information or screenshots about the pull request here.
Sorry about the Cargo.toml assets formatting my editor refused to save without formatting it

This is incredible work. Thank you so much ❤, I will give you a proper review soon.

I am not sure why this didn't work...

That's strange I'll have a look when I get the chance to see if I can find out why that's happening

Oh okay I found the issue, looks like I need to write some extra code to give the current process the correct permissions to access other user processes. I will get to writing this extra code, ill also brush up the error handling to make it more clear of what actually went wrong.
Will bump this pull request once I've implemented the changes

Actually I don't think that was the issue, from the looks of it nginx has two processes and this program only manages to kill the first process which immediately gets started again by the first process
As for that system http listener I think that process becomes protected by windows as its refusing to let me open a process handle to it. As for other apps I have no problem terminating them I've tested on a few different servers of mine and haven't had any issue.
Looks like these problems might be specific two these two specific servers?

@jacobtread In the nginx case do you know if we could find the parent process and kill it as well?

I've produced a working version which collects the parents processes and from my experience it seems as though its able to successfully kill nginx I will add this commit to the pull request shortly after finalizing the code for it

Changes are up 👍🏻


Enhancing StreamDiffusion's Efficiency for High-Resolution Image Generation

Dear StreamDiffusion Development Team,
I hope this message finds you well. I am reaching out to discuss a potential enhancement to the StreamDiffusion pipeline, particularly concerning the generation of high-resolution images. As an avid user and admirer of your innovative work, I have been utilising StreamDiffusion for various projects and have been thoroughly impressed with its performance and capabilities.
However, I have observed that when scaling up to generate images of higher resolution, there is a noticeable increase in computational demand, which in turn affects the real-time interactivity that is a hallmark of StreamDiffusion. While the current pipeline is exceptionally efficient, I believe there is an opportunity to optimise it further for high-resolution output.
To address this, I propose the following considerations:

Adaptive Resolution Scaling: Implementing a mechanism that initially generates images at a lower resolution and progressively enhances them could maintain interactivity without compromising on detail.

Distributed Processing: Exploring the possibility of distributing the computation across multiple GPUs could significantly reduce the time required for generating high-resolution images.

Model Pruning: Investigating the effects of model pruning on the diffusion models to reduce the number of parameters, which could lead to faster computation times while maintaining image quality.

Advanced Caching Strategies: Enhancing the pre-computation for KV-caches to handle higher resolution images more effectively, potentially by utilising a hierarchical caching system that prioritises the most impactful features.

I am curious to hear your thoughts on these suggestions and whether they align with the future roadmap for StreamDiffusion. I believe that by addressing the challenge of high-resolution image generation, StreamDiffusion can set a new standard for real-time interactive pipelines in the field.
Thank you for your time and consideration. I look forward to the possibility of contributing to the evolution of this remarkable tool.
Best regards,
yihong1120

I'm delighted to see your enthusiastic use of StreamDiffusion. Your excellent suggestion is greatly appreciated! In our future updates to the pipeline, I will try to integrate these elements to explore further enhancements in performance.


Smart Wallets -> Account Abstraction

PR-Codex overview
This PR updates terminology from "Smart Wallet" to "Account Abstraction" across various files and components to align with the new terminology.
Detailed summary

Replaced "Smart Wallet" with "Account Abstraction" in multiple file names, functions, and descriptions.
Updated text and titles to reflect the change from "Smart Wallet" to "Account Abstraction" for consistency.
Adjusted descriptions and links related to "Smart Wallet" to now refer to "Account Abstraction."
Renamed components, pages, and templates related to "Smart Wallet" to "Account Abstraction" for clarity and consistency.

The following files were skipped due to too many changes: pages/sponsored-transactions.tsx, components/wallets/ConnectWalletPlayground/Playground.tsx, pages/dashboard/connect/account-abstraction.tsx, pages/account-abstraction.tsx, contract-ui/tabs/code/components/code-overview.tsx

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

📝 Changed routes:

/account-abstraction
/connect

(view full size)
(view full size)

…ct/account-abstraction
…nnect/embedded-wallets

(view full size)
(view full size)

/deploy
/embedded-wallets

(view full size)
(view full size)

And 7 other routes:

/engine
/smart-contracts
/solutions/gaming
/solutions/web2-onboarding
/sponsored-transactions
/support
/templates

Commit 5af49c24567cc4d38927f63189bc8f96cd637509 (https://thirdweb-g06gq19ak.thirdweb-preview.com).


No playbutton on mobile-connection

Yes - very strange problem.
The Play button does not appear on mobile connection. No problem on Wifi connection with the same device.
Testet on different Android devices with 3 different Browsers.
I was able to fix the problem temporarily when i set preload to none.
Yes, the type of network connection has nothing to do with the content and so on...... but wifi on: playbutton. Wifi off and reload: no playbutton. But the player (time...) appear on both.  Also the mp3-file is only about ~100kb so this is no slow connection problem.
It's also happened on the online demo.
Maybe its a Browser Problem, idk if browsers check the art of connection and replace things, but the player shows up without playbutton so something is strange there.

https://github.com/greghub/green-audio-player/blob/master/src/js/main.js#L391
The function on this line might give you a clue as to what's going on.
It toggles the visibility of the play button.


Simplify reload of modeler

Problem you would like to solve
As a user I'm always frustrated by our existing reload behavior (used to re-discover global element templates).
I'd assume I can only use a keyboard shortcut (Slack has CTRL+R / CTRL+SHIFT+R) to reload without the need to open the dev tools. Instead I need to open the dev tools and press the shortcut to work.
Proposed solution

[ ] Provide a dedicated shortcut (CTRL/CMD+SHIFT+R) to force reload of modeler that works without devtools being open

Alternatives considered
None.
Additional context
This is relevant in the context of extension and element template development.
While you could argue devtools shall be open anyway this is not strictly necessary.

My proposal is, let us do it!


Adds the _java_runtime attribute to fix toolchain use in Scala.

When executing non-locally, the wrong version of Java would be used,
referencing a path that didn't exist. This change adds the
_java_runtime attribute to have to locate the java runtime on the
machine where the Scala code is being executed, and selects the
correct configuration for the host_javabase.
As a minor cleanup, this also removes the now no-longer-necessary
_java attribute.
TESTED=Ran unit tests and also ran a custom Scala unit test through:
previously, the Scala unit test failed with an incorrect java path
error, now it passes.
Change-Id: I9c2829bded04ec2d9d014094bd9e72cd984fe6e6

Ok, sorry for the delay...after further discussion with lberki and trying out an example Scala test with remote execution, we don't believe the comment (nor the _java_toolchain entry) is necessary, so I've removed them from the two locations where I found them.

Test this please (ci bot)

seems like the google ci isn't responding...
@ittaiz I'm happy to merge as is. What do you think?

Ditto
On Fri, 2 Feb 2018 at 21:11 P. Oscar Boykin<EMAIL_ADDRESS>wrote:

seems like the google ci isn't responding...
@ittaiz https://github.com/ittaiz I'm happy to merge as is. What do you
think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/409#issuecomment-362677698,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIF3WtrcFvEJCE7wQosxPtAtiozruQks5tQ13TgaJpZM4RynQn
.


[WIP] [DX11] fix crash when window is minimized

Closes #1156
I fixed the minimization crash by ignoring DXGI_STATUS_OCCLUDED, so the window will be resized to (0, 0).
I called update_views in gfx_app and I'm not sure if device.clear_state() is necessary.

I used a generic to manage the two devices of DX11 (Deferred and Device)

By the way, the examples are not releasing the targets so they all raise the error.

Thanks!
Is this still WIP?

I think it's ready to be merged

@homu r+

:pushpin: Commit e6d4007 has been approved by kvark

:hourglass: Testing commit e6d4007 with merge 784247d...

:sunny: Test successful - status

Hi. When this fix will be on crates.io?


Parallelize application

Find all for loops that do not need to be run serially, and make app parallel.
Optional: show benchmarks (before-after times)

While there are areas where we can add concurrency (possibly true parallelization) to for loops, it will not lead to any significant performance gains.


ddccontrol-db: 20210505 -> 20210804

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ddccontrol/ddccontrol-db/releases.
meta.description for ddccontrol-db is: "Monitor database for DDCcontrol"
meta.homepage for ddccontrol-db is: "https://github.com/ddccontrol/ddccontrol-db"
meta.changelog for ddccontrol-db is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 20210804 in filename of file in /nix/store/i9xxp3gx1rwbngd0zvc1qd17npdk0yyf-ddccontrol-db-20210804

directory tree listing: https://gist.github.com/be6dc82aff429c5d6c61284bc38e55fe

du listing: https://gist.github.com/b2e59c0fb7dea2c0290283798744648e

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
ddccontrol
ddccontrol-db

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i9xxp3gx1rwbngd0zvc1qd17npdk0yyf-ddccontrol-db-20210804 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ddccontrol-db https://github.com/r-ryantm/nixpkgs/archive/24ba262fbe335f6cb853f9c66b48968b0acf6dfb.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i9xxp3gx1rwbngd0zvc1qd17npdk0yyf-ddccontrol-db-20210804
ls -la /nix/store/i9xxp3gx1rwbngd0zvc1qd17npdk0yyf-ddccontrol-db-20210804/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

ddccontrol
ddccontrol-db

Maintainer pings
cc @pakhfn for testing.

Result of nixpkgs-review pr 132733 at 24ba262f run on aarch64-linux 1

2 packages built successfully:

ddccontrol
ddccontrol-db


Tanks forgetting connections to EIO on reload

Summery:
Every time i unload a world, such as quitting the game, then reload it EIO does not reattach to DeepResonance tanks until i change the input\output mode of one of the tank's faces.

Issue:
DR tanks are rendered defunct to anything other than DR blocks after world reload until manually reset.
Other DR blocks, such as Smelters, are seemingly unaffected.

Steps to reproduce:

Get two empty DR tanks, either crafted or obtained by other means
Place the two tanks vertically, with at least 1 block space between them
place a fluid conduit from EIO as to bridge the divide
set the Input/output mode of the tanks via right-clicking with empty hand
quit to title
a.or restart Minecraft, makes no difference
Reload world, observe
a. This occurs regardless to if the world was freshly generated or existing.

Useful information:
DR version 1.1x-1.4.5 built against 1.11/ 0254505c4d2d063fecbb6846d5134c09e2b6e243
mcjtylib version1.1x-2.3.7.jar (from Curse)
EnderIO version 1.10.2-3.1.168 (from Curse)
Minecraft version 1.10.2
Forge Version <IP_ADDRESS>1

It is possible that this bug will be fixed next release


major(graphcache): stop buffering operations while the cache hydrates

Resolves #1512
Summary
This enables graph-cache to forward operations while hydration is ongoing, this benefits application in various ways

they can benefit from synchronous rehydration due to the ssrExchange
does not cause an extra delay for empty caches or missing operations

Set of changes

remove buffering of operations

I'm wondering whether we'll still have to issue reexecutes for them in case the hydrated data disagrees with the cache here 🤔
The dedupe would then take care of filtering them out anyway if they're not needed. Gotta test this

This was intended to be a draft so we could do testing soz :D valid point though!


Temperature units

Hi,
First of all thanks a lot for your job, I love your work.
I've just install the new UNI and I see in Signalk DataBrowser that the units of the temperature sent from the uni has shown in Kelvin.
Inside the UNI I select centigrade degree and also inside your config.
I don't understand if it is my issue or not.
Thanks for any help.

Temperature in signalk is always Kelvin. So it looks good to me.

We use SI units. It's up to the clients/displays to convert to the users preferred units.

Oh, ok! So I see K units but it is only a label that I have to manage, but the temperature has shown in decree Celsius as I configured in the UNI and your setup. Ok. No problem. Thanks Da: Scott Bender @.***>Inviato: Mercoledì 21 Aprile 2021, 10:25A: sbender9/signalk-shellyCc: Max Terragni; AuthorOggetto: Re: [sbender9/signalk-shelly] Temperature units (#15)
We use SI units. It's up to the clients/displays to convert to the users preferred units.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.

It should show as Kelvin in the Data Browser. If not, I made a mistake.

It should show as Kelvin in the Data Browser. If not, I made a mistake.

In DataBrowser the value is Celsius


Added scroll progress indicator bar

Fixes Issue🛠

Closes #1597
Description👨💻
I have added a progress bar on the top of the navbar which progresses as the page scrolls up or down.
Type of Change📄

[x] New feature (non-breaking change which adds functionality)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

https://github.com/user-attachments/assets/5e3ee77e-213e-40d2-8b8b-3c3eddd9b921

Please add the same in all the pages such as About, Contributors, etc.
Also, use the color that matches the home page UI nicely @meetarora10

Sure @Rakesh9100

@Rakesh9100 now I have added in all the pages and also changed the UI as you said .


fix: Hanle the error of vdisk.GetIStorage in syncCloudDisk

这个 PR 实现什么功能/修复什么问题:
问题：客户的某个环境，vcenter的一些宿主机失去连接，所以宿主机下的虚拟机的storage拿不到了，
这里没有handle error 导致 触发panic，同步任务异常退出。

是否需要 backport 到之前的 release 分支:

release/3.2
release/3.1
release/3.0
release/2.13

/area region
/cc @zexi @swordqiu

Codecov Report

Merging #6786 into master will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #6786   +/-   ##
=======================================
  Coverage    7.11%    7.12%
=======================================
  Files         654      654
  Lines      110462   110464    +2
=======================================
+ Hits         7857     7868   +11
+ Misses     102000   101990   -10
- Partials      605      606    +1

Flag
Coverage Δ

#aFlag
7.12% <0.00%> (+<0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/compute/models/disks.go
0.53% <0.00%> (-0.01%)
:arrow_down:

pkg/cloudcommon/cronman/cronman.go
53.95% <0.00%> (+5.11%)
:arrow_up:

Continue to review full report at Codecov.

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


[Automation] Generate Fluent Lite from iothub#package-2021-03

[Automation] Generate Fluent Lite from iothub#package-2021-03

My team is interested in making this a non-beta release since the swagger it is based on has long since GA'd. Is there any reason we should hold off on doing this, though? For example, will the generator of this code change in the future such that this SDK would get breaking changes upon regeneration?

@timtay-microsoft
You can make it a GA release as 1.0.0 (as usual change the version_client.txt, etc.).
However note that once it GAed, it will be much more difficult for your spec to make breaking changes (as you can see from CHANGELOG there is multiple breaking changes since 2020-03 to 2021-03).
We would be very reluctant to have breaking change within say a year and then need to release 2.0.0.

@weidongxu-microsoft we plan on preventing the service team from making breaking changes to the control plane moving forward. With that in mind, my team would like to do this release as 1.0.0 rather than another beta

@timtay-microsoft
Sure, you can make it 1.0.0. Codegen change will likely not affect future releases.


Bower package

Good work !
can i suggest you to register a bower package ?
MORE HERE

waiting for merge


bug:community form text says profile will be editable, it is not at this point

community form text says profile will be editable, it is not at this point so we should either fix it so it is, or change text to something like "contact X to make changes."

button gone, but happening elsewhere and waiting on feedback from support team.

ken reaching back out to form Helpdesk


[opt](inverted index) Add troubleshooting logs

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:

Occasionally encounter 'No segments* file found' issue, need to add troubleshooting logs.

Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall


Selector on CircleImageView

I need to display a check sign on long press on CircleImageView.
How to do this

@dwivedi Is this done? Should I look into it, to help you?

Yes,
Override on setSelected(Boolean flag) of Image view and alter some code
snippet on onDraw method.
Please provide your valuable suggestions on same.
Should I share you my code?
Regards
Ashish
On Tue 14 May, 2019, 7:28 PM MonikaJethani<EMAIL_ADDRESS>wrote:

@dwivedi https://github.com/dwivedi Is this done? Should I look into
it, to help you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hdodenhof/CircleImageView/issues/331?email_source=notifications&email_token=AAGIFFUFYJDOD2KNGNROFTTPVLAQ7A5CNFSM4HLCCFHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVLR72A#issuecomment-492249064,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGIFFVAX3YYSHFOD22QSNTPVLAQ7ANCNFSM4HLCCFHA
.

I doubt that this is still relevant, but you could have a look at https://github.com/hdodenhof/CircleImageView/issues/153#issuecomment-249692049 and adapt that method to display a checkmark.


All commands which work with accounts should accept --account= argument.

The web based lbry wallet will use lbrynet accounting / publishing commands by passing a different --account= to specify on which users behalf the command should be run; therefore all common commands which require blockchain interaction should accept a --account= argument.

[ ] publish
[ ] claim_list_mine
[ ] ... add checkboxes for all of the other commands that need this update ...

completed in #1456, closing


BREAKING CHANGE

I'm about to land PR #1294 which contains a breaking change. While I understand that is very annoying, the fix is extremely straight forward and it does fix a couple of nasty bugs.
In essence it means that now all option struct fields are pointers. This was already true for most of the fields but not for slices. That is now corrected and so it means that everywhere where you set assign a slice value to a field in an option struct, you now have to add a & in front of it to take the address of the slice.
I've created tag v0.53.0 just before landing the PR and will create v0.54.0 once the PR is merged. Sorry for the inconvenience!

Closing this issue as the PRs are merged and the tags are set.


Kill button stays Active

In an online game session after the player has already failed a murder attempt, the kill button is still marked as active allowing them to attempt a second murder attempt.

The issue appears to have been addressed by Kren, I have tested in a local and online game. Unable to reproduce the bug after the latest commit.
https://github.com/arevi/Kill-Doctor-Lucky-Web-Port/commit/7a0a14f9943991486e2105d4e5120487e6927a29

Corrected this issue; kill button should no longer stay active.

Looks good!


No github links for csharp

Genarating docs for a solution with both F# and C#, the github links for F# are generated correctly, but there are no github links for the C# project. Is this by design, or should they be there?

I think this is not supported. Happy to accept PR that adds this :)

I think this needs to be added to FCS and is not trivial :)

@matthid I think we should close out all issues involving using FSharp.Formatting for formatting C# assemblies.  What works works, what doesn't seems out of scope.  What do you think?

I think good C# support could have been a good way to attract C# developers at the time. I'd say that time has passed and they use different tools nowadays. While it might still be a good idea to support C#, there is probably no benefit in keeping the issues open.

@matthid Yes agreed

AFAIK we still have users in mixed language projects.
Don Syme<EMAIL_ADDRESS>schrieb am Sa., 13. Apr. 2019, 01:54:

@matthid https://github.com/matthid Yes agreed
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/FSharp.Formatting/issues/413#issuecomment-482755812,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNAxgDtlKMRO5vs_yfqkARMiWRuhjks5vgRzPgaJpZM4JETIn
.

I am closing this as it isn't a goal to be a perfect c# doc generator. Someone can contribute it I guess but it's not a bug or even a feature on the roadmap


Update Socket to work around sending secure HTTPS requests with PHP < 7.1.4

PHP < 7.1.4 (and PHP < 7.0.18) suffers from a bug when writing big chunks of data over TLS streams at once. We try to work around this by limiting the write chunk size to 8192 bytes for older PHP versions only. This is only a work-around and has a noticable performance penalty on affected versions. Please update your PHP version.
Resolves / closes #60
Refs https://github.com/reactphp/socket/issues/105

Note that the included tests are a bit fragile and have repeatedly failed here. Restarting the builds fixed the tests, so it appears that the Travis matrix issues too many parallel requests.
I've updated the tests to be spread across eu.httpbin.org and httpbin.org at least and I'm open for suggestions :+1:

@clue I've updated the repository settings on travis to only run 1 concurrent build