Build failure when running esy build under Ubuntu 18.04.1 LTS

$ esy build
info esy build 0.4.9 (using package.json)
info building reason-gl-matrix@0.2.1@d41d8cd9
error: build failed with exit code: 1
  build log:
    # esy-build-package: building<EMAIL_ADDRESS>    # esy-build-package: pwd: $HOME/.esy/3__________________________________________________________________/b/reason_gl_matrix-0.2.1-7d45b189
    # esy-build-package: running: 'refmterr' 'dune' 'build'
             gcc src/glm_stubs.o (exit 1)
    (cd _build/default/src && /usr/bin/gcc -I $HOME/.esy/3__________________________________________________________________/i/ocaml-4.6.9-584f7b7f/lib/ocaml -O2 -fno-strict-aliasing -fwrapv -fPIC -I ./../../../include/glm -fno-exceptions -fno-rtti -lstdc++ -x c++ -o glm_stubs.o -c glm_stubs.cpp)
    In file included from ./../../../include/glm/detail/type_int.hpp:6:0,
                     from ./../../../include/glm/fwd.hpp:6,
                     from ./../../../include/glm/glm.hpp:61,
                     from glm_stubs.cpp:9:
    ./../../../include/glm/detail/setup.hpp:456:100: note: #pragma message: GLM: GCC older than 4.6 has a bug presenting the use of rgba and stpq components
     # pragma message("GLM: GCC older than 4.6 has a bug presenting the use of rgba and stpq components")
                                                                                ^
    In file included from ./../../../include/glm/detail/type_vec4.inl:968:0,
                     from ./../../../include/glm/detail/type_vec4.hpp:453,
                     from ./../../../include/glm/vec4.hpp:6,
                     from ./../../../include/glm/glm.hpp:70,
                     from glm_stubs.cpp:9:
    ./../../../include/glm/detail/type_vec4_simd.inl: In static member function 'static glm::tvec4<float, (glm::precision)5> glm::detail::compute_vec4_div<float, (glm::precision)5, true>::call(const glm::tvec4<float, (glm::precision)5>&, const glm::tvec4<float, (glm::precision)5>&)':
    ./../../../include/glm/detail/type_vec4_simd.inl:156:11: error: 'struct glm::tvec4<float, (glm::precision)5>' has no member named 'data'
        Result.data = _mm_mul_ps(a.data, _mm_rcp_ps(b.data));
               ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl:156:31: error: 'const struct glm::tvec4<float, (glm::precision)5>' has no member named 'data'
        Result.data = _mm_mul_ps(a.data, _mm_rcp_ps(b.data));
                                   ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl:156:50: error: 'const struct glm::tvec4<float, (glm::precision)5>' has no member named 'data'
        Result.data = _mm_mul_ps(a.data, _mm_rcp_ps(b.data));
                                                      ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(T) [with T = float; glm::precision P = (glm::precision)5]':
    ./../../../include/glm/detail/type_vec4_simd.inl:367:3: error: class 'glm::tvec4<float, (glm::precision)5>' does not have any field named 'data'
       data(_mm_set1_ps(s))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(T) [with T = float; glm::precision P = (glm::precision)4]':
    ./../../../include/glm/detail/type_vec4_simd.inl:372:3: error: class 'glm::tvec4<float, (glm::precision)4>' does not have any field named 'data'
       data(_mm_set1_ps(s))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(T) [with T = float; glm::precision P = (glm::precision)3]':
    ./../../../include/glm/detail/type_vec4_simd.inl:377:3: error: class 'glm::tvec4<float, (glm::precision)3>' does not have any field named 'data'
       data(_mm_set1_ps(s))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(T) [with T = int; glm::precision P = (glm::precision)5]':
    ./../../../include/glm/detail/type_vec4_simd.inl:399:3: error: class 'glm::tvec4<int, (glm::precision)5>' does not have any field named 'data'
       data(_mm_set1_epi32(s))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(T) [with T = int; glm::precision P = (glm::precision)4]':
    ./../../../include/glm/detail/type_vec4_simd.inl:404:3: error: class 'glm::tvec4<int, (glm::precision)4>' does not have any field named 'data'
       data(_mm_set1_epi32(s))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(T) [with T = int; glm::precision P = (glm::precision)3]':
    ./../../../include/glm/detail/type_vec4_simd.inl:409:3: error: class 'glm::tvec4<int, (glm::precision)3>' does not have any field named 'data'
       data(_mm_set1_epi32(s))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(T, T, T, T) [with T = float; glm::precision P = (glm::precision)5]':
    ./../../../include/glm/detail/type_vec4_simd.inl:431:3: error: class 'glm::tvec4<float, (glm::precision)5>' does not have any field named 'data'
       data(_mm_set_ps(d, c, b, a))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(T, T, T, T) [with T = float; glm::precision P = (glm::precision)4]':
    ./../../../include/glm/detail/type_vec4_simd.inl:436:3: error: class 'glm::tvec4<float, (glm::precision)4>' does not have any field named 'data'
       data(_mm_set_ps(d, c, b, a))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(T, T, T, T) [with T = float; glm::precision P = (glm::precision)3]':
    ./../../../include/glm/detail/type_vec4_simd.inl:441:3: error: class 'glm::tvec4<float, (glm::precision)3>' does not have any field named 'data'
       data(_mm_set_ps(d, c, b, a))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(A, B, C, D) [with A = int; B = int; C = int; D = int; T = int; glm::precision P = (glm::precision)5]':
    ./../../../include/glm/detail/type_vec4_simd.inl:447:3: error: class 'glm::tvec4<int, (glm::precision)5>' does not have any field named 'data'
       data(_mm_set_epi32(d, c, b, a))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(A, B, C, D) [with A = int; B = int; C = int; D = int; T = int; glm::precision P = (glm::precision)4]':
    ./../../../include/glm/detail/type_vec4_simd.inl:453:3: error: class 'glm::tvec4<int, (glm::precision)4>' does not have any field named 'data'
       data(_mm_set_epi32(d, c, b, a))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(A, B, C, D) [with A = int; B = int; C = int; D = int; T = int; glm::precision P = (glm::precision)3]':
    ./../../../include/glm/detail/type_vec4_simd.inl:459:3: error: class 'glm::tvec4<int, (glm::precision)3>' does not have any field named 'data'
       data(_mm_set_epi32(d, c, b, a))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(A, B, C, D) [with A = int; B = int; C = int; D = int; T = float; glm::precision P = (glm::precision)5]':
    ./../../../include/glm/detail/type_vec4_simd.inl:465:3: error: class 'glm::tvec4<float, (glm::precision)5>' does not have any field named 'data'
       data(_mm_castsi128_ps(_mm_set_epi32(d, c, b, a)))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(A, B, C, D) [with A = int; B = int; C = int; D = int; T = float; glm::precision P = (glm::precision)4]':
    ./../../../include/glm/detail/type_vec4_simd.inl:471:3: error: class 'glm::tvec4<float, (glm::precision)4>' does not have any field named 'data'
       data(_mm_castsi128_ps(_mm_set_epi32(d, c, b, a)))
       ^~~~
    ./../../../include/glm/detail/type_vec4_simd.inl: In constructor 'glm::tvec4<T, P>::tvec4(A, B, C, D) [with A = int; B = int; C = int; D = int; T = float; glm::precision P = (glm::precision)3]':
    ./../../../include/glm/detail/type_vec4_simd.inl:477:3: error: class 'glm::tvec4<float, (glm::precision)3>' does not have any field named 'data'
       data(_mm_castsi128_ps(_mm_set_epi32(d, c, b, a)))
       ^~~~
    In file included from ./../../../include/glm/detail/../detail/func_exponential.inl:144:0,
                     from ./../../../include/glm/detail/../detail/func_exponential.hpp:103,
                     from ./../../../include/glm/detail/../detail/func_geometric.inl:4,
                     from ./../../../include/glm/detail/../detail/func_geometric.hpp:113,
                     from ./../../../include/glm/detail/../geometric.hpp:6,
                     from ./../../../include/glm/detail/func_matrix.inl:4,
                     from ./../../../include/glm/detail/func_matrix.hpp:149,
                     from ./../../../include/glm/detail/type_mat2x2.inl:4,
                     from ./../../../include/glm/detail/type_mat2x2.hpp:182,
                     from ./../../../include/glm/mat2x2.hpp:6,
                     from ./../../../include/glm/glm.hpp:71,
                     from glm_stubs.cpp:9:
    ./../../../include/glm/detail/../detail/func_exponential_simd.inl: In static member function 'static glm::tvec4<float, (glm::precision)5> glm::detail::compute_sqrt<glm::tvec4, float, (glm::precision)5, true>::call(const glm::tvec4<float, (glm::precision)5>&)':
    ./../../../include/glm/detail/../detail/func_exponential_simd.inl:28:11: error: 'struct glm::tvec4<float, (glm::precision)5>' has no member named 'data'
        result.data = glm_vec4_sqrt_lowp(v.data);
               ^~~~
    ./../../../include/glm/detail/../detail/func_exponential_simd.inl:28:39: error: 'const struct glm::tvec4<float, (glm::precision)5>' has no member named 'data'
        result.data = glm_vec4_sqrt_lowp(v.data);
                                           ^~~~
    In file included from ./../../../include/glm/detail/func_matrix.inl:399:0,
                     from ./../../../include/glm/detail/func_matrix.hpp:149,
                     from ./../../../include/glm/detail/type_mat2x2.inl:4,
                     from ./../../../include/glm/detail/type_mat2x2.hpp:182,
                     from ./../../../include/glm/mat2x2.hpp:6,
                     from ./../../../include/glm/glm.hpp:71,
                     from glm_stubs.cpp:9:
    ./../../../include/glm/detail/func_matrix_simd.inl: In function 'typename glm::detail::outerProduct_trait<T, P, colType, rowType>::type glm::outerProduct(const vecTypeA<T, P>&, const vecTypeB<T, P>&) [with T = float; glm::precision P = (glm::precision)5; vecTypeA = glm::tvec4; vecTypeB = glm::tvec4; typename glm::detail::outerProduct_trait<T, P, colType, rowType>::type = glm::tmat4x4<float, (glm::precision)5>]':
    ./../../../include/glm/detail/func_matrix_simd.inl:67:27: error: 'const struct glm::tvec4<float, (glm::precision)5>' has no member named 'data'
       glm_mat4_outerProduct(c.data, r.data, *reinterpret_cast<__m128(*)[4]>(&m[0].data));
                               ^~~~
    ./../../../include/glm/detail/func_matrix_simd.inl:67:35: error: 'const struct glm::tvec4<float, (glm::precision)5>' has no member named 'data'
       glm_mat4_outerProduct(c.data, r.data, *reinterpret_cast<__m128(*)[4]>(&m[0].data));
                                       ^~~~
    ./../../../include/glm/detail/func_matrix_simd.inl:67:79: error: 'glm::tmat4x4<float, (glm::precision)5>::col_type {aka struct glm::tvec4<float, (glm::precision)5>}' has no member named 'data'
       glm_mat4_outerProduct(c.data, r.data, *reinterpret_cast<__m128(*)[4]>(&m[0].data));
                                                                                ^~~~
    ./../../../include/glm/detail/func_matrix_simd.inl: In function 'typename glm::detail::outerProduct_trait<T, P, colType, rowType>::type glm::outerProduct(const vecTypeA<T, P>&, const vecTypeB<T, P>&) [with T = float; glm::precision P = (glm::precision)4; vecTypeA = glm::tvec4; vecTypeB = glm::tvec4; typename glm::detail::outerProduct_trait<T, P, colType, rowType>::type = glm::tmat4x4<float, (glm::precision)4>]':
    ./../../../include/glm/detail/func_matrix_simd.inl:75:27: error: 'const struct glm::tvec4<float, (glm::precision)4>' has no member named 'data'
       glm_mat4_outerProduct(c.data, r.data, *reinterpret_cast<__m128(*)[4]>(&m[0].data));
                               ^~~~
    ./../../../include/glm/detail/func_matrix_simd.inl:75:35: error: 'const struct glm::tvec4<float, (glm::precision)4>' has no member named 'data'
       glm_mat4_outerProduct(c.data, r.data, *reinterpret_cast<__m128(*)[4]>(&m[0].data));
                                       ^~~~
    ./../../../include/glm/detail/func_matrix_simd.inl:75:79: error: 'glm::tmat4x4<float, (glm::precision)4>::col_type {aka struct glm::tvec4<float, (glm::precision)4>}' has no member named 'data'
       glm_mat4_outerProduct(c.data, r.data, *reinterpret_cast<__m128(*)[4]>(&m[0].data));
                                                                                ^~~~
    ./../../../include/glm/detail/func_matrix_simd.inl: In function 'typename glm::detail::outerProduct_trait<T, P, colType, rowType>::type glm::outerProduct(const vecTypeA<T, P>&, const vecTypeB<T, P>&) [with T = float; glm::precision P = (glm::precision)3; vecTypeA = glm::tvec4; vecTypeB = glm::tvec4; typename glm::detail::outerProduct_trait<T, P, colType, rowType>::type = glm::tmat4x4<float, (glm::precision)3>]':
    ./../../../include/glm/detail/func_matrix_simd.inl:83:27: error: 'const struct glm::tvec4<float, (glm::precision)3>' has no member named 'data'
       glm_mat4_outerProduct(c.data, r.data, *reinterpret_cast<__m128(*)[4]>(&m[0].data));
                               ^~~~
    ./../../../include/glm/detail/func_matrix_simd.inl:83:35: error: 'const struct glm::tvec4<float, (glm::precision)3>' has no member named 'data'
       glm_mat4_outerProduct(c.data, r.data, *reinterpret_cast<__m128(*)[4]>(&m[0].data));
                                       ^~~~
    ./../../../include/glm/detail/func_matrix_simd.inl:83:79: error: 'glm::tmat4x4<float, (glm::precision)3>::col_type {aka struct glm::tvec4<float, (glm::precision)3>}' has no member named 'data'
       glm_mat4_outerProduct(c.data, r.data, *reinterpret_cast<__m128(*)[4]>(&m[0].data));
                                                                                ^~~~
    error: command failed: 'refmterr' 'dune' 'build' (exited with 1)
    esy-build-package: exiting with errors above...

  building<EMAIL_ADDRESS>esy: exiting due to errors above

Thanks for the report @shaform ! This should be addressed by a fix in reason-gl-matrix here: https://github.com/bryphe/reason-gl-matrix/pull/16
I just picked up that update in this PR here: https://github.com/revery-ui/revery-quick-start/pull/15
So should be fixed now in the latest master. Let me know if that's not the case.

I've found another issue when running esy build on Ubuntu 18.04 LTS
I'm not sure where to report this. If in other repo (e.g. revery), in a separate issue or here.


sed error: invalid reference \1 on `s' command's RHS

The latest version (cd9c9f40aba1c6d2ced89b8aca8f0d0313512e5d) doesn't build for me in Arch Linux:
==> Starting package()...
make: Entering directory '/mnt/WD4e/makepkg/pacaur-git/src/pacaur'
Generating documentation...
Installing...
sed: -e expression #1, char 50: invalid reference \1 on `s' command's RHS
make: *** [Makefile:54: install] Error 1
make: Leaving directory '/mnt/WD4e/makepkg/pacaur-git/src/pacaur'

Sorry about that, it should be fixed now. I'm not sure why, but GNU sed doesn't seem to enable extended regex for commands before specified after the -E flag, which I don't think makes sense given that the flag is supposed to enable extended regex for the whole command/script. Either way I believe it works now.


HttpResponse implementations should throw if writes are not awaited

Today, Kestrel (I haven't tried this yet on InProc or HttpSysServer) will not throw if you call HttpContext.Response.Body.WriteAsync() multiple times in a row without awaiting for the response. Ex:
app.Run(context =>
{
    context.Response.Body.WriteAsync(buffer1);
    context.Response.Body.WriteAsync(buffer2);
    return Task.CompletedTask;
});

It's possible that the first call to WriteAsync returns a completed Task, but even if it doesn't, Kestrel will allow the app to write again immediately.
To make this work on top of a PipeWriter, we've had to use a special pipe flushing logic, and even that seems like it might have issues.
Instead, we should throw an InvalidOperationException if the app attempts to write to the response body prior to the previous write's completion. This of course would be a breaking change, and would need to be done ASAP to make it into 3.0.

Might also want to throw if the request ends/control is returned to server before write completion? To also catch
app.Run(context =>
{
    context.Response.Body.WriteAsync(buffer1);
    return Task.CompletedTask;
});

Not awaiting the last write is an interesting case. The server can't just throw an InvalidOperationException since user code is no longer on the stack. I think logging a warning makes sense. That feels similar to how we log a warning when the app doesn't consume the entire request body.

Log an error? It does mean a coding error.

What about reading from the request without awaiting?
var t1 = context.Request.Body.ReadAsync(byteMemory1);
var t2 = context.Request.Body.ReadAsync(byteMemory2);
await Task.WhenAll(t1, t2);

What about reading from the request without awaiting?

Should also 💥
Unfortunately in most cases they will instantly complete; so these will only be best endeavour errors?

Parallel reads already throw. @benaadams is right that the example would not throw if the request body data was already buffered.
This highlights a concern of mine about making this breaking change to HttpResponse: in most cases, bad code won't throw.
The unawaited write has to return an incomplete Task before any InvalidOperationExceptions are thrown. And writes really only return incomplete Tasks for large responses.

This highlights a concern of mine about making this breaking change to HttpResponse: in most cases, bad code won't throw.

Large responses likely will hit the return control without completing barrier though? e.g. if you single shot a large response.
app.Run(context =>
{
    context.Response.Body.WriteAsync(bufferOf1MB);
    return Task.CompletedTask;
});

Not sure what the default threshold is where it starts giving back pressure?

Not sure what the default threshold is where it starts giving back pressure?

Kestrel configures the pipe to start inducing back pressure after buffering 64KB by default. OSs also wait for their buffers to fill before introducing back pressure. OSs buffer a lot more than Kestrel itself does. Just take a look at our MaxRequestBufferSizeTests and you'll see that OSs don't start inducing back pressure until several MBs of data get buffered.

Kestrel configures the pipe to start inducing back pressure after buffering 64KB by default.

Could pull down the default if in Development mode? Would that help?

HttpSys & HttpListener have explicitly supported this for a very long time. There was a legit benchmark showing this was required in order for a response to saturate a 1gb connection. Something about copying large VHDs between datacenters.
There's no reason to go back and remove this where it's not causing an issue. In Kestrel's case there are functional reasons to remove it. I would be curious to try the same benchmark with Kestrel though, to see how well it works with the pipeline infrastructure.

HttpSys & HttpListener have explicitly supported this for a very long time. [...] There's no reason to go back and remove this where it's not causing an issue.

Seems fair.
You can do overlapped writes on a socket at the OS level; but also the OS will return the write as completed almost instantly (as MaxRequestBufferSizeTests shows); however Pipelines does not support overlapping writes, though also has a margin for instant completion. Pipelines coalesces writes to a certain degree (via dispatch) instead of overlapping.
So its a question of doing a lot of extra work to fake overlapping writes in Kestrel (as they won't actually happen) or denying it. Denying it seems easier to get correct; and there is no advantage to supporting it other than extra code complexity an testing.

There was a legit benchmark showing this was required in order for a response to saturate a 1gb connection. Something about copying large VHDs between datacenters. [...] I would be curious to try the same benchmark with Kestrel though, to see how well it works with the pipeline infrastructure.

Guessing that would be max data throughput on a single connection and single (large) request?

Guessing that would be max data throughput on a single connection and single (large) request?

Yes, max throughput on a single response.

Triage decision: We think the behavior should be consistent across HttpSys and Kestrel, so if we are to do this we have to feel confident we can do it in HttpSys without regressing perf. We don't feel like we can do that in 3.0. Closing for now, we'll revisit in the future.

Re-opening.


Add support for PowKiddy RGB20 Pro

Describe the Issue (If applicable)
Can you add support for the PowKiddy RGB20 Pro? It uses the RK3566 chipset along with a 3.2" 1024*768 screen.

I may add that unit.  I'm just waiting for my unit to arrive.

I may add that unit. I'm just waiting for my unit to arrive.

When did you place your order?
I’m also waiting like you and hope that when the device arrives, I’ll be able to install updates from you.
Thanks

Just got mine last week.  Haven’t spent much time with it yet as I’m spending time with family at the moment.

I think about 2 to 3 weeks before your request here. Just got mine last week. Haven’t spent much time with it yet as I’m spending time with family at the moment.

Thanks for the update! Looking forward to it, but please take your time.

I have a build of ArkOS running on the RGB20 Pro.  It does require that the unit be opened and the battery unplugged for 10 seconds or it will not be able to safely shutdown.  It's been a known issue when running a mainline kernel based OS like ROCKNIX then switching to a BSP Kernel based OS like ArkOS.

My RGB20Pro hasn’t been delivered yet, but I am currently using both the RGB30 and RGB20SX. This issue only occurs on the RGB30 and does not affect the RGB20SX at all. My RGB20SX is from the latest batch manufactured in 2024. I’m quite surprised by this as well. Hopefully, this information will be helpful to you.

RGB20 Pro support is now available.  You can download a compatible image from the Download links section of the wiki.  It will be labelled specifically for the Rgb20 Pro
Important Note!
If your unit came with ROCKKNIX installed or you've run ROCKNIX on your units, you must open your unit and unplug the battery for 10 seconds or it will not be able to safely shutdown.  The unit will just reboot.
As a temporary workaround, if you shutdown the unit in ROCKNIX using the the proper shutdown process through the start menu, you will be able to shutdown safely in ArkOS, however, the solution is temporary because if you do a system reboot, the issue will return.


Update faq.md

Added "Tesla API Token Generator" Python script. Local tool guaranteeing privacy of data, as I don't like using third party which could steal my secrets.

Closing as no control over the linked repo. If wish to submit a PR /w the python code, that would be different.


N6 GenAI more fine tuning, and Hiding of API Keys

More fine tuning of GenAI capability and the removal of API keys from codebase.

LGTM


Allow sourcemaps to be generated for production

Adding always parameter to generate separate file sourcemaps when in production mode.
Coming from elixir which had the elixir.config.sourcemaps = true; option to always generate sourcemaps even in production was useful for us, since we use Sentry for providing insights into errors in JS Code. :)

This will be a nice addition. I am using bugsnag js in my project and bugsnag recommend have source map available even in production.
It would be good if we can generate hidden-source-map source maps.
This is the recommended by webpack.

It looks like you can use hidden source maps with Sentry as well, so seems like a good option, but not sure making it the default is the "right" decision.
I wonder whether a better option would be to have something like
mix.options({
    productionSourceMaps: true // for standard inclusion, 'hidden' for hidden, or allowing the full text, e.g. 'cheap-source-map'
});

Giving the developer the options of what to use.

This is a great addition, but IMO the default for always should be true since I believe source maps in general are mostly useful and pretty common in production environments. Thinking you won't need source maps for tracking down errors manually or with tools such as sentry once you ship your code is very misguided, I think.
With regards to using hidden-source-map optionally, you can already do this anyways using webpackConfig. I'm currently doing the following:
const production = process.env.NODE_ENV === 'production'
const devtool = production ? 'source-map' : 'inline-source-map'

mix
  .webpackConfig({
    devtool
  })
  ...

I mean, the last thing you want is for laravel-mis to have an API as complex as webpack 😄.
The default for production should definitely be source-map.

@rhyek I love your solution to the problem there, I may have to adopt that.
I've been holding off merging my elixir to mix conversion, because I don't want to lose sourcemaps in production, and without something like the above, it certainly would be a show stopper for all the sites I use elixir on.
Considering #223 there appears to be some divide in the community on what should be the standard.  I agree of course that source-maps in production should exist by default.  However, I understand that there are some out there that use the minification as a way of protecting their code.
I would imagine the 80% use case that mix aims for is probably more in the camp for generating in production rather than not.

Lets disable the sourcemaps in production and inline in dev mode by default.
Let anyone override default behaviour in this way -
// Default behaviour for new users
mix.sourceMaps();

// For more advanced user
mix.sourceMaps('any-devtool-option-here');

one can pass any valid string from here

Why should the default for production be to disable source maps when I, as a developer, am specifically calling sourceMaps()? IMO, a less "advanced user" wouldn't even be calling sourceMaps(), and those who have a very specific reason to disable them only for production are probably less than those who would like them available for all environments as the default behavior for the method. So, use the option for those less common scenarios, no?

I think in general, the expectation when using sourceMaps() is that they're just always generated. I actually spent a couple minutes figuring out why they were being generated locally, but not on my server.
The option should be made available for people who are actively trying to acomodate different scenarios on their end.

I agree with @rhyek that the expectation would be for them to be generated when calling the function.
And I went the route of trying to minify in development first, which of course didn't work as the source maps were still in-line creating larger than expected files.
elixir didn't have a "sourceMaps" function, it just did it on dev, and not on prod unless you specified elixir.config.sourcemaps = true;, so calling the function here is also akin to stipulating that you want sourceMaps.

Any progress ?

 I know people will try npm run dev and wonder why it's not working. :/


Azure KeyVault 2.0.0 preview libraries on NuGet ?

Is there any specific ETA on when Azure KeyVault libraries for .NET Core in the AutoRest branch  will get published on NuGet ?

@prajaybasu how about now: https://www.nuget.org/packages/Microsoft.Azure.KeyVault/2.0.1-preview


INFO Will use root access: write NetworkManager config in /etc/NetworkManager/conf.d/crc-nm-dnsmasq.conf

crc setup
INFO Checking if oc binary is cached
INFO Checking if podman remote binary is cached
INFO Checking if CRC bundle is cached in '$HOME/.crc'
INFO Checking if running as non-root
INFO Checking if Virtualization is enabled
INFO Checking if KVM is enabled
INFO Checking if libvirt is installed
INFO Checking if user is part of libvirt group
INFO Checking if libvirt is enabled
INFO Checking if libvirt daemon is running
INFO Checking if a supported libvirt version is installed
INFO Checking if crc-driver-libvirt is installed
INFO Checking for obsolete crc-driver-libvirt
INFO Checking if libvirt 'crc' network is available
INFO Checking if libvirt 'crc' network is active
INFO Checking if NetworkManager is installed
INFO Checking if NetworkManager service is running
INFO Checking if /etc/NetworkManager/conf.d/crc-nm-dnsmasq.conf exists
INFO Writing Network Manager config for crc
INFO Will use root access: write NetworkManager config in /etc/NetworkManager/conf.d/crc-nm-dnsmasq.conf
[sudo] password for abc:

@Amitninja12345 This is expected, not sure what is the issue you are facing?

FATA Failed to write NetworkManager config file: /etc/NetworkManager/conf.d/crc-nm-dnsmasq.conf: Failed writing to file as root: /etc/NetworkManager/conf.d/crc-nm-dnsmasq.conf: Sorry, user abc is not allowed to execute '/bin/tee /etc/NetworkManager/conf.d/crc-nm-dnsmasq.conf' as root on abc.dd.ttt.
: exit status 1

it worked now.


Impossible to use inject inside RouteMeta or CanActivateFn

Please provide the environment you discovered this bug in.
Analog 1.7.2, angular 18.1.3
Which area/package is the issue in?
Don't know / other
Description
Hi,
I'm trying to create a route guard but can't seem to use inject() inside it.
export const AuthGuard: CanActivateFn = async () => {
  const session = await inject(SupabaseService).getSession();

  if (!session.data.session) {
    inject(Router).navigate(["/auth"]);
    return false;
  }

  return true;
};

export const routeMeta: RouteMeta = {
  canActivate: [AuthGuard],
};

@Component({
  standalone: true,
  imports: [
    RouterOutlet,
    HeaderComponent,
  ],
  selector: "mkp-layout",
  template: `

      <mkp-header />
      <router-outlet />

  `,
})
export default class LayoutComponent {
}

Please provide the exception or error you saw
Error: NG0203: inject() must be called from an injection context such as a constructor, a factory function, a field initializer, or a function used with `runInInjectionContext`. Find more at https://angular.dev/errors/NG0203

Other information
It seemed possible in Angular docs => https://angular.dev/guide/di/dependency-injection-context#stack-frame-in-context
I would be willing to submit a PR to fix this issue

[ ] Yes
[X] No

Ok Apparently it's only the inject(Router) that causes the error.

Took a quick look and it has something to do with the async/await. Changing the order of injection works for some reason.
export const AuthGuard: CanActivateFn = async () => {
  const router = inject(Router);
  const session = await inject(SupabaseService).getSession();

  if (!session.data.session) {
    router.navigate(["/auth"]);
    return false;
  }

  return true;
};

export const routeMeta: RouteMeta = {
  canActivate: [AuthGuard],
};

@Component({
  standalone: true,
  imports: [
    RouterOutlet,
    HeaderComponent,
  ],
  selector: "mkp-layout",
  template: `

      <mkp-header />
      <router-outlet />

  `,
})
export default class LayoutComponent {
}

Need to look more into why.

For reference, this also fails within a regular Angular application, so it's not Analog-specific. If you open the devtools and click on "Test" it throws the error.
https://stackblitz.com/edit/stackblitz-starters-uonwng?file=src%2Fmain.ts

This behavior is working as intended, as explained by @SanderElias.
Anything after the await gets moved into the next browser macrotask, which would be after instantiation. The injection context is only available during instantiation, which is why putting all the inject(s) above the await works.


Modules no longer stackable

Plethora modules (laser, sensor, scanner, etc.) are no longer able to be stacked. This is a deviation from 1.12 behavior, where, if I recall correctly, they had a maximum stack size of 64. This makes storing large quantities of modules slightly inconvenient, since each individual module item needs its own slot in an inventory. For example:

Are you able to tell if those specific stacks have any NBT?

list on the chest peripheral should show the NBT hashes


[docs] Change Fedora OS dependencies

CATEGORY
Choose one

[ ] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[x ] Documentation

SUMMARY
Fixes #7681.
I have also made some very minor tweaks to formatting on the table and capitalization.

Codecov Report

Merging #8496 into master will increase coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #8496      +/-   ##
=========================================
+ Coverage   66.44%   66.6%   +0.16%
=========================================
  Files         449     449
  Lines       22608   22608
  Branches     2366    2366
=========================================
+ Hits        15022   15059      +37
+ Misses       7448    7411      -37
  Partials      138     138

Impacted Files
Coverage Δ

superset/views/core.py
72.17% <0%> (+0.21%)
:arrow_up:

superset/models/core.py
81.46% <0%> (+0.62%)
:arrow_up:

superset/dataframe.py
94.48% <0%> (+1.57%)
:arrow_up:

superset/db_engine_specs/mysql.py
92.59% <0%> (+51.85%)
: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 fda2333...a28a462. Read the comment docs.


renovatebot(deps): update amazonsdk to v1.11.552

This PR contains the following updates:

Package
Update
Change
References

com.amazonaws:aws-java-sdk-clouddirectory
patch
1.11.551 -> 1.11.552
homepage

com.amazonaws:aws-java-sdk-cloudwatch
patch
1.11.551 -> 1.11.552
homepage

com.amazonaws:aws-java-sdk-cognitoidp
patch
1.11.551 -> 1.11.552
homepage

com.amazonaws:aws-java-sdk-core
patch
1.11.551 -> 1.11.552
homepage

com.amazonaws:aws-java-sdk-dynamodb
patch
1.11.551 -> 1.11.552
homepage

com.amazonaws:aws-java-sdk-logs
patch
1.11.551 -> 1.11.552
homepage

com.amazonaws:aws-java-sdk-s3
patch
1.11.551 -> 1.11.552
homepage

com.amazonaws:aws-java-sdk-secretsmanager
patch
1.11.551 -> 1.11.552
homepage

com.amazonaws:aws-java-sdk-sns
patch
1.11.551 -> 1.11.552
homepage

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.null seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


sys/linux/socket_netlink_route_sched: add missing tc qdisc/class/filters

This patchset add all missing tc qdisc/class/filters

Codecov Report

Merging #1478 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1478      +/-   ##
==========================================
- Coverage   56.36%   56.31%   -0.05%
==========================================
  Files         140      140
  Lines       26162    26162
==========================================
- Hits        14745    14732      -13
- Misses      10712    10728      +16
+ Partials      705      702       -3

Impacted Files
Coverage Δ

prog/target.go
55% <0%> (-7.15%)
:arrow_down:

prog/prog.go
81% <0%> (-2.5%)
:arrow_down:

prog/rand.go
89.27% <0%> (+0.36%)
: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 b602d64...3e41d7c. Read the comment docs.

Cool! Eager to see if it will uncover new bugs. Thanks.


LMS does not work for tranducer network

Hi,
While doing some experiments on espnet, an asr framework with pytorch backend,  the memory of gpu is not enough. So I installed the LMS for pytorch, and it works well  except the tranducer network. It seems that the error is about lms, the exception is as following:
asr_train.py --config conf/tuning/transducer/train_transducer.yaml --preprocess-conf conf/specaug.yaml --ngpu 8 --backend pytorch --outdir exp/train_960_pytorch_train_transducer_specaug/results --tensorboard-dir tensorboard/train_960_pytorch_train_transducer_specaug --debugmode 1 --dict data/lang_char/train_960_unigram5000_units.txt --debugdir exp/train_960_pytorch_train_transducer_specaug --minibatches 0 --verbose 0 --resume --n-iter-processes 8 --train-json dump/train_960/deltafalse/data_unigram5000.json --valid-json dump/dev/deltafalse/data_unigram5000.json
Started at Mon Oct 19 01:32:06 UTC 2020

/usr/local/miniconda3/lib/python3.8/site-packages/chainer/backends/cuda.py:142: UserWarning: cuDNN is not enabled.
Please reinstall CuPy after you install cudnn
(see https://docs-cupy.chainer.org/en/stable/install.html#install-cudnn).
warnings.warn(
2020-10-19 01:32:07,749 (asr_train:561) WARNING: Skip DEBUG/INFO messages
2020-10-19 01:32:09,098 (asr:481) WARNING: batch size is automatically increased (16 -> 128)
terminate called after throwing an instance of 'c10::Error'
what():  state_ == State::kActive INTERNAL ASSERT FAILED at ../c10/core/LargeModelSupport.h:94, please report a bug to PyTorch.  (unpin at ../c10/core/LargeModelSupport.h:94)
frame #0: c10::Error::Error(c10::SourceLocation, std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&) + 0x6c (0x7f5f893c340c in /usr/local/miniconda3/lib/python3.8/site-packages/torch/lib/libc10.so)
frame #1:  + 0x398b173 (0x7f5f8eecf173 in /usr/local/miniconda3/lib/python3.8/site-packages/torch/lib/libtorch.so)
frame #2:  + 0x6101da8 (0x7f5f91645da8 in /usr/local/miniconda3/lib/python3.8/site-packages/torch/lib/libtorch.so)
frame #3:  + 0x5960519 (0x7f5f90ea4519 in /usr/local/miniconda3/lib/python3.8/site-packages/torch/lib/libtorch.so)
frame #4:  + 0x58e2b71 (0x7f5f90e26b71 in /usr/local/miniconda3/lib/python3.8/site-packages/torch/lib/libtorch.so)
frame #5:  + 0x5960519 (0x7f5f90ea4519 in /usr/local/miniconda3/lib/python3.8/site-packages/torch/lib/libtorch.so)
frame #6:  + 0x44cc4f (0x7f5fa384cc4f in /usr/local/miniconda3/lib/python3.8/site-packages/torch/lib/libtorch_python.so)

frame #40: __libc_start_main + 0xe7 (0x7f5fb3f54b97 in /lib/x86_64-linux-gnu/libc.so.6)
Accounting: time=6 threads=1
Ended (code 134) at Mon Oct 19 01:32:12 UTC 2020, elapsed time 6 seconds
Python version is 3.8.3, and pytorch version is 1.4.0, cuda version is 10.0

Very interesting work!
I notice the line
/usr/local/miniconda3/lib/python3.8/site-packages/chainer/backends/cuda.py:142: UserWarning: cuDNN is not enabled.
Please reinstall CuPy after you install cudnn

Which is perhaps something to explore. What is chainer for in your configuration?
You may also try moving to a newer version of CUDA. CUDA 10.0 is fairly old and likely predates LMS altogether. :)

@jayfurmanek  The warning message is about "chainer" backend of espnet, but I use "pytorch" backend. So it does not matter. I also upgrade the cuda to 10.2, and compile pytorch 1.5 with it, but it still not work.  The error message is same.


Local-storage add-on : Reset all sections button functionality

resolves #39
Hi @katherinepeterson , as we had discussed in issue #39 , that we will implement a clear all/reset button to clear all sections to start afresh after local storage is implemented .  Now that i implemented local-storage in PR #80 , I have implemented reset functionality as below and its live on preview url  for you to check .

I have also added translations for the word 'Reset' per language (best of my efforts via Google Translate :) , if this require any minor change someone can work on this via a separate issue . )
Request you to consider to merge this PR first , as its a feature enhancement & will avoid any minor conflicts with any translations PR .
Once you approve this PR , i will look at the issue #81 .
Thanks !!

@katherinepeterson any comments on this MR, or do i need to change something ? Let me know, as i see conflict in one file on this MR  , but resolve conflicts button is disabled for me . Thanks

@saurabhchaturvedy can we reset it back to initial state of only Title and Description slug?

@katherinepeterson thanks for the review , i can resolve the conflicts , just wanted to know if the comments that you have given , should i resolve them in this same PR or work of them as separate issue, as this MR was pending for some time  ?


Improve Error Message for Parquet Schema Mismatch

@dain @electrum

We have lots of nested schema parquet files, eg. struct of structs. And new fields are added/removed/reordered into nested structs. Currently Presto will report error when metastore schema does not match parquet schema. In our case, it is quite often that our metastore has the most updated schema, but the parquet files only contain some of the fields. The first commit is giving more hints when query fails. The second commit is trying to support schema evolution in nested parquet structs, say, using struct field name to access the corresponding parquet fields. If metastore could not find the fields in parquet schema, just return NULL.
@dain @electrum any comments and suggestions are appreciated.

Merged, thanks!


DEC-627: Remove exhibit and exhibition

Depends on https://github.com/ndlib/honeycomb/pull/283
Now that everything has been changed to not use these objects, I deleted the models, and any leftover references to them within services, views, etc.
In addition to this cleanup, I found a bug during testing with the url handling in external collections. SaveCollection will now add http:// to an address if it is not given so that links are always treated as absolute.

:+1:

Just had one question about the routes, but apart from that :+1:


Adding proxy voting to custodians

This change includes proxy voting and associated tests. The changes should be backwards compatible with the current voting as deployed on main-net but since it includes quite a lot of internal logic changes and refactoring it should be thoroughly tested on the test nets first.

This was continued work from an earlier PR that was still open to improve the tests. This still relies on the most changes added to Lamington. While the changes have merged back into the Lamington master these changes have not been released as an NPM module change yet so it still needs to installed locally and run through npm link lamington. I'll follow up with the Lamington guys about the public npm module update.


upgraded LoadScene and OpenScene calls to use 5.3beta API

Some minor updates I made as a result of using Unity 5.3beta. Probably can't be merged into master due to backward compatibility. Perhaps can be put aside in a branch for now.

Thanks.  I wonder if we could handle this with #ifdefs ?  Is there a UNITY_5_3 define?  I haven't upgraded yet so can't check

Ah of course, didn't think of that. Will try it out.


[INLONG-10876][SDK] Transform support RAND() function

Fixes https://github.com/apache/inlong/issues/10876
Motivation
add RandFunction class to support flinkSQL common function RAND() and RAND(INT). Also, add the corresponding unit test codes.
Rand()--Returns a pseudo-random double precision value in the range [0.0, 1.0)
Rand(Integer)--Returns a pseudo-random double precision value in the range [0.0, 1.0) with an initial seed of Integer.

Modifications

Verifying this change
(Please pick either of the following options)

[ ] This change is a trivial rework/code cleanup without any test coverage.

[x] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

plz rebase the master to resolve the conflict

plz rebase the master to resolve the conflict

done


[New Feature]:  Add play/pause button

🎮 feature Request
feature logic and basic description
The pause/play button button will pause the timer
Point down the features
feature points
a button so that timer can be stopped.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@ayush-t02 please assign me this issue


Correction for raw affiliation School of Social Work, King's University College at Western University, London, ON, Canada

Correction needed for raw affiliation School of Social Work, King's University College at Western University, London, ON, Canada
raw_affiliation_name: School of Social Work, King's University College at Western University, London, ON, Canada
new_rors: ;02grkyz14
previous_rors: 048f5b183;02grkyz14
works_examples: W4322494177;W4380626987
contact: aedeba6afc902cc94b1af369c7150758:de6366cbe83b229c @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


I think  c.Limit  also can  be used to filter unmatched urls  . Should support regex  before request   in anywhere.

The purpose is to filter out the mismatched urls
for example
package main
import (
"github.com/asciimoo/colly"
"fmt"
"time"
)
func main() {
urls := []string{"https://httpbin.org/hello", "https://httpbin.org/123", "https://httpbin.org/12xyz"}
// Instantiate default collector
c := colly.NewCollector()

// when visiting links which domains' matches "*httpbin.*" glob
c.Limit(&colly.LimitRule{
	DomainGlob: "*https://httpbin.org/[a-z]+",
})
c.OnResponse(func(r *colly.Response) {
	fmt.Println("Finished", r.Request.URL, time.Now())
})

for _, i := range urls {
	c.Visit(i)
}

 c.Wait()

}
Starting https://httpbin.org/hello",
Starting "https://httpbin.org/123",
Starting "https://httpbin.org/12xyz
Finished  https://httpbin.org/hello

@tx991020 this would be a useful feature, however LimitRule works a bit different than in your example. Currently it specifies restrictions to the matching domains and you want to allow URLs with these rules. I'm not sure if extending LimitRule is the correct way to achieve this functionality.

@tx991020 I could imagine this as an independent feature - e.g. c.AddURLFilter(urlGlob string) or something similar. Thoughts?

yes.

In my understanding Limit it's something what limits resources used by a program. You need to extract URLs in code of your program anyway.
Isn't your code a good place to add some regexp?

I would rather add c.URLFilters []*regexp.Regexp to make sure before crawling that regexps are correct.


[ENH] Theme:  Logo Visibility and Consistency in Light/Dark Themes

Issue
Logo Visibility and Consistency in Light/Dark Themes
Objective
Ensure that the logo used in navigation and pages is updated and remains clearly visible when switching between light and dark themes.
Description:
To maintain a cohesive user experience, the logo must adapt to both light and dark themes without losing visibility or brand identity. This issue aims to establish guidelines and implementation steps for ensuring the logo's design and placement are optimized for both themes.

Looking into docfx I ran into
https://github.com/dotnet/docfx/discussions/9576 to switch images in dark mode
which reference:
https://github.com/orgs/twbs/discussions/38776
an issue on github relating to bootstrap created by @leigh-pointer
and points to a js solutoin
https://github.com/orgs/twbs/discussions/38793
So another option is we create a shadow behind the white logo so it works in both dark and light themes as done in the Oqtane Framework repository readme page.
We can create this if needed as well.
Looks like the Guides WIP image performs this on switch, we just need to get the logo to do this which may take altering the main.js file or?

#91 fixes this and can at least put the logo part to bed.  The content can use this for reference and we should update our DOCS documentation for working with images so they are seen in dark/light themes.


Update German translations

Coverage remained the same at 59.648% when pulling 305e547be4266cd6e2364581a9979f1f2640df88 on der-lyse:german-translations into be755569058b862a540cdba996dac401b4a09ebd on newsboat:master.

Thank you! And thanks for reviewing other i18n PRs too :)


Retrieve artifact metadata with non-existent blob

When a new blob is added to an artifact ie. a
snapshot, the blob UUID is stored in the artifact
before the blob is created in the DB. This causes
an error when retrieving non-existent metadata for
an artifact.
A future fix should resolve this by only storing UUID's
in the DB that refer to objects already in existence.

I think even with only storing uuids for objects which already exist we'd want this as defence in depth against some sort of DB inconsistency issue. LGTM, thanks.

CI failures are all unrelated (my slow internet), so merging.


-user (User) option/config to downgrade privileges after binding to ports

Most daemon support starting as root and downgrading to a specific user/group after start
e.g.
https://httpd.apache.org/docs/2.4/mod/mod_unixd.html#user
Shouldn't envoy should do the same - or at least document the workarounds
(CAP_NET_BIND_SERVICE to bind to privileged ports and wrappers like setuidgid)

Note that the Envoy process doesn't actually have the "daemon nature". It runs in the foreground and doesn't do the other things daemons usually do to dissociate themselves from the context they were started in. You can use a wrapper to achieve some of this behavior.

the wrapper can't do the binding of privileged ports though; it's also additional overhead to require a wrapper

👎 personally. Too much code complication.

FTR I am not a fan of building this functionality into Envoy either.

it's been solved by all/most httpd servers since the first CERN one
https://www.w3.org/Daemon/Implementation/HTDaemon.c
but it's your call obviously.
but shouldn't we have a documented workaround on how users would bind on port 80 on say trusty and xenial ?

Closing this as won't fix as I haven't seen anyone else ask for this. We can reconsider if we get more requests for this feature.

@mattklein123
I'm using envoy as an edge reverse proxy for its udp capabilities (something haproxy can't do).
However, I'm mapping to port 53 and am getting the same permission problem.
I run it in docker (seems like the simplest way to run Envoy) and could do a port mapping instead of using the host network, but in our case the dns proxy is a single point of failure (dictated by our extremely limited number of available public ips... we run on a private Openstack operated by a state agency) so we want to make it as performant as possible (hence host network is preferable).
Getting the performance benefit of running directly on the host network with the reduced privilege of a non-root user would be ideal (it is after all an internet-facing proxy, I'm sure the codebase of envoy is very good given its reputation, but every extra bit of assurance helps).
Granted, this is an uncommon problem (a projet running on AWS, Google Cloud, Azure and al. with their near-infinite elasticity, plentiful public ips and managed load balancers will not experience these kinds of limitations, but some projets with strict locality requirements can't go in that direction), but it might happen more often than you think.


Number of images (dat files) used is not clear

I have the SiW-M dataset with me but it has a dat file corresponding to each video. That means it has (No. of frames) * (2562563+64*64+1) bytes. So did you extract all frames from each dat file and trained on all of them?

Can you share it with me?  I am a university student and suffer from no data set for experimenting. My email is<EMAIL_ADDRESS>thank you very much

Hi together,
Could you share the dataset with me also, I'm unable to download it
My email is<EMAIL_ADDRESS>Thank you,

@parthagar Can u Confirm me the labels for spoof and live images.

@Epimenides7 @AnhNguyen1805 sorry I can't share it, its License prohibits sharing.
@rathodx The labels are provided in the respective dat file itself.


RTATestBundle: per-task noisiness threshold values

This implements what @douglas-raillard-arm and I discussed offline.
Briefly tested on a trace from last week's mainline integration.

FYI on a previousled UNDECIDED test I got this output:
2019-04-08 15:00:30,738 INFO    : lisa.tests.scheduler.eas_behaviour.TwoBigThreeSmall : Ignored PIDs for noise contribution: 2265 (big_0), 2264 (small_2), 2263 (small_1), 2262 (small_0), 2266 (big_1), 0 (swapper/4), 86 (sugov:2), 84 (sugov:0)

PASSED: noisiest task={duration (abs)=0.004312999999967815 s, comm=sshd, pid=175, duration (rel)=0.4276368626314915 %}, energy threshold=1572.1895813953488 bogo-joules, estimated energy=1517.9204315027 bogo-joules

@douglas-raillard-arm Done

LGTM


try to lookup pgp-keys when sending new email

Expected behavior
write new email to "sender"
when pgp provider like openkeychains is active, k-9 should ask openkeyschain to lookup "sender" and check if there is a key stored online.
if yes -> download key and use it
if no -> just send email normally
Actual behavior
write new email to "sender"
emails is just signed (k-9 does not check if a key for "sender" can be found online)
Steps to reproduce
see above
Environment
K-9 Mail version: 5.114 (f-droid)
Android version: 6.x
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

I think we just pass off to OpenKeychain to determine whether there is a key available. So this may actually be first a change to OpenKeychain's behaviour to launch an intent suggesting searching online key repositories.
@dschuermann @Valodim what do you think?

it would be great if this would work as seemless as possible, to make more users transition to pgp encryption

the less user interaction is needed, the more widespread pgp will be used

Firstly, the is the privacy issue of looking up keys potentially every time, when users write an email to an unknown address. Secondly, what do we do if there is multiple keys?

it should be the same as with receiving an encrypted email.
openkeychains should automatically be opened if multiple keys are found

k9 already informs you about the existence of recipient public keys when adding the recipients to the message. I would suggest that when you click on the recipient showing the "no public keys" icon, it should provide an option to start the key obtaining process for that recipient with openkeychain.

it should be seamless to use for any user without any technical knowledge.
so when you have "encrypt if possible" set, it k-9 should actually check if encryption is possible, by using the pgp provider to get the keys needed. (call the same intent as when receiving an encrypted email where the key is not available)
why should the user take a sperate action, when "encrypt if possible" was already set?
the result now is that email is send unencrypted, but user has selected to encrypt.

No. It should only go get keys IF the user ACTUALLY tells it to.
User interaction IS MANDATORY in order for a key to be trusted. Without ACTUALLY taking manual steps to verify the key, no security is being added BY encrypting the message. You might as well send it plain text.
k9mail makes it VERY clear when the message will or will not be encrypted to all recipients. That is the appropriate opportunity to ALLOW the user to perform appropriate key management functions.
It should NEVER go out on the internet and find some random key to use to encrypt something.
Key verification requires an actual communication with the owner of the key, for instance, READING the fingerprint over a voice phone call, or in the very least, sharing the fingerprint over a different protocol. Without that, you DO NOT KNOW who generated the key.

Prompting should be a decision taken by the PGP app. This is an issue that can be debated on OKC's app as to whether a prompt for internet lookup is required (or perhaps configurable - 'lookup always, prompt, never'?)
It's not for K-9 to make decisions about key sources. We should fire an intent, OKC should display a prompt activity if required, do any lookups it wants to and then return us the keys for emails that are available).

@philipwhiuk yep, exactly what i mean. the pgp provider should take care of security, K-9 should fire an intent without any button presses from the user.

also it's clear because:
you have 1 pgp-provider app, and many apps that use pgp.
so deceisions should be made at a single point, the pgp proverder app.

I don't think it's that clear. There are use cases for pgp that do have very different requirements that lead to different trust models - think e-mail communication vs package signing.
Mid term I plan to have an inband-exchange of keys with a TOFU trust model that works with zero user interaction. I'm not sure yet where that information will be stored, and how it will look exactly though :)

i hope some version of that will make it into the next release.
seemless pgp integration in k-9 (with openkeychains) would most likely boost pgp usage a lot.

Wow.. I do think folks should be very careful with this.  Don't spoil a good Email app by messing up the encryption. For social graph revealing apps we already have whatsapp. Some of the beauty of PGP is the fact that it is, because it's highly configurable, suitable to all use cases. Something probably Stallman would applaud. I'd love to ask him what he thinks of this recent developer attitude a la Valodmin, Moxie etc. but I'd rather not hang around for the answer. :)
Out with the old.. Down with user decision making. Control them all!

you missunderstand. k-9 should pass it seamless to the pgp provider.
the provider can show a popup to get the user consent.
no need for the user to say "go" in k-9 and then say "go" again in openkeychains.
also most ordinary users will not even know of the existance of pgp keys of the other party.
and it's email so every time you write email you already leak lots of meta data, so asking a keyserver for the key is only a small amount more of that.

You know, I think this is all down to configurable options. Everything is fine as long as the user has a choice about the behaviour. When you lock stuff down there will always be a use case you don't support.
You can say that's not a valid use case, but the beauty of open source thinking used to be total configurability by the end user.  I guess re my previous comment, Stallman would just say "fork it". Actually he wouldn't have such a massive fail as an Android device in the first place. :)
I might like to have a default of "encrypt if possible", but not have to either:

click the icon move the slider
dismiss something in the pgp provider.
rather, I might like k-9 to just send the mail.
Maybe it's actually about getting the design right. I quite liked the easy access to the sign and encrypt check boxes before.
Right now, it's annoying me to have to move the slider on every email that I'm not encrypting. But I believe that's fixed now in master. :)
I will probably step back from posting on this now, as although I mean to be light hearted, I fear I may offend someone.
Anyway, I'd prefer to contribute code rather than comments, but there's only so many hours in the day, and I haven't looked at Android development in well over a year.

re configurable: I no longer agree that configurability and empowering the user to do what they want is a reasonable prime directive, particularly not in crypto, and particularly not if it means to support every conceivable way of twisting the openpgp data format into workflows that make no sense on a larger scale. See also: misfeatures, feature creep, bloat. We'll always be totally configurable in the Apache 2.0 sense though :)
Your complaint relates to #1790, and should be fixed there.
If you'd like to contribute code, we'd be happy to have you. How about that message source display activity or image attachment resizing? Or if you prefer bug hunting, we have tons of those. Drop by the irc/matrix channel if you want real time guidance :)
(other than that - back to topic please)

"forking" or "options" will have the effect that ordinary users can not use pgp. k-9 has the chance to widen pgp usage (just look at the download numbers in playstore).
so some good default settings are essential.


Add darts exercise

Ha, gdlint had a point about the unneeded elifs. I've definitely given the same advice before. :)

So, you're saying that it actually sometimes gives a good advice? :stuck_out_tongue:


create-react-app integration instructions are out of date

The new version of create-react-app no longer uses a .babelrc file in the root of the project, with the configuration instead existing in the package.json. This lead to confusion when spinning up a new project. The new version of CRA also has a more robust linting configuration, and will not compile without further modification.
I went ahead and created a working version of an ejected CRA, available here https://github.com/SydBal/cra-ejected-pug which demonstrates the modifications I needed to make in order to get the integration working. (spinning off of this stack overflow answer https://stackoverflow.com/questions/56513346/how-to-use-pug-templating-engine-with-reactjs/62051614#62051614)
The README should be updated to reflect the new setup.

use craco

I just set this up with react-app-rewired and customize-cra and will have a PR shortly

I’ve been using react-app-rewired without issues for prod apps since the beginning of 2020. Can post configs if y’all need


For red edge, let's make Override Fork Point action visible but only in context menu

Not in toolbar.
Let's make it possible (i.e. via context menu) to override the FP for the red edge case, but not suggested (i.e. via toolbar).

You probably meant enabled (because it is already visible from context menu)

Right, enabled indeed, changing the title


feat: switch to mill and publish to maven

Adds upstream rocketchip and chisel as dependency
Use @sequencer code for mill

Haha, I think these are from sequencer/arithmetic. I'm refactoring chisel mill build flow, after finishing that, I will go back to this.

Thanks!


Mongo fetch fixes

This fixes two problems with fetching and updating elements from Mongo, described in #218.

Codecov Report

Merging #221 into dev will decrease coverage by 0.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##                dev     #221      +/-   ##
============================================
- Coverage     23.07%   23.05%   -0.02%
  Complexity      103      103
============================================
  Files            62       62
  Lines          2405     2407       +2
  Branches        216      218       +2
============================================
  Hits            555      555
- Misses         1817     1819       +2
  Partials         33       33

Impacted Files
Coverage Δ
Complexity Δ

...l/taui/controllers/RegionalAnalysisController.java
9.21% <ø> (+0.06%)
2 <0> (ø)
:arrow_down:

...om/conveyal/taui/controllers/BrokerController.java
19.64% <0%> (-0.55%)
6 <0> (ø)

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 582dcda...abcb1b9. Read the comment docs.

LGTM but I think this issue needs to be addressed further. See https://github.com/conveyal/analysis-backend/issues/218#issuecomment-490341999

LGTM but I think this issue needs to be addressed further. See #218 (comment)

This is really just a bugfix to make staging more useable in the mean time. Maybe we can bundle a few more related fixes into this PR (per #218).


ability to deserialize arrays through IDictionary<string, object>

Added ability for JsonUtil.Deserialize<IDictionary<string, object>>(json) to properly deserialize arrays to support a scenario where you want to deserialize json for which you don't know the schema at compile time.
fixes #812

Thanks for proposing this. I've looked into it, and unfortunately I don't think this is the desired way to proceed.
The trouble with this approach is that the items in the array aren't necessarily objects (i.e., can't necessarily be represented by an IDictionary<string, object>). For example, this change would cause cause an InvalidCastException for JsonUtil.Deserialize<IDictionary<string, object>>("{\"array1\":[\"something\"]}") (which previously wouldn't return a useful result, but at least wouldn't crash, making it it safe to have JSON of this form as part of a larger deserialization process). In general the IDictionary<string, object> contract implies that the dictionary entries can be anything, which could include numbers, strings, arrays, other dictionaries, and so on.
Instead of doing this, if you really want to work with an unknown JSON schema, an easier way to proceed is just to call JsonUtil.Deserialize<object>(someJson). Then you can inspect the resulting type and perform the necessary runtime casts (e.g., to IDictionary<string, object> or IList<object> depending on what you receive at runtime).
For example, the following unit test would already pass without any implementation change:
[Fact]
public void SupportsDictionaryArrayDeserialization()
{
    // Arrange/Act
    var result = JsonUtil.Deserialize<object>("{\"array1\":[{\"key1\":\"value1\"},{\"key1\":\"value2\"},\"another\"]}");

    // Assert
    var array1 = (result as IDictionary<string, object>)["array1"] as IList<object>;
    Assert.Equal("value1", (array1[0] as IDictionary<string, object>)["key1"]);
    Assert.Equal("value2", (array1[1] as IDictionary<string, object>)["key1"]);
    Assert.Equal("another", array1[2]);
}

I'm aware that SimpleJson's built-in handling for the specific IDictionary<string, object> case isn't hugely useful, but trying to cover all scenarios is the difference between SimpleJson and a full JSON library like Json.NET. At least there are workarounds like using Deserialize<object>.

Fair and agreed. Thanks for investigating it, and keep up the good work, Steve.


Aouter helper 插件 添加 支持kotlin的导航跳转。

支持

ARouter.getInstance().build("/xx/xxx").navigation(context)
@Autowired(name = "/yourservicegroupname/hello")
与
@route(path = "/xx/xxx")
@route(path = "/yourservicegroupname/hello", name = "test service")
之间的相互跳转。（只支持kotlin之间的跳转

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.liwei seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


op-star prevents match

https://regex101.com/r/2V42EM/1
b* on text= abcd is supposed to match the b.
Same if the patten is b*c* or (\b|\B)b* or x?b* on text= abcd
But it returns empty.
First Charset is "", which is correct: after all, if there was no "b" in the expression, then it would match empty at the start.
But it should not give up, after matching empty.

Actually, the website got me there. It has opt-global on by default.
The behaviour is only if global is turned on.
TRegEx does not have that yet (yet TRegEx has ExecPos to do what global would do.
So how to handle?
Do we need the global flag?

I am not sure that b* must match the b, it can match empty string and it is doing this. looks like other engines have some option for this case? and we don't.

I am not sure how much we need the 'global flag to fix this'. which other RE engines have it? only 1 or 2 from totally 10?

Actually you are right...
I either had a severe issue with my vision, or they update the page.
Looking at it today, it shows that b* does match the empty, but as it is global, it matches again (and for an empty pattern the next match is at "previous end +1" (which for TRegEx is up to the user-code, and that's fine).
So we actually do the correct thing.

thanks. I dont know what is 'global match' at the www site.


adding mysql database integration

Closing this in favor of https://github.com/sourcejump/discord-bot/pull/11.


Small segment sizes for empirical line calculation

This finalizes our fix to the spatial interpolation artifacts (using a small segmentation size and many neighbors for the empirical line stage).  This is much slower than with large segments, but it eliminates any spatial artifacts so I think it is more appropriate for a default setting.

Addendum: It's OK to squash this commit history.

Looks good to me, thanks for the work tuning this up.  As a note to any users who might be looking at more homogenous scenes, these parameters in apply_oe can always be reverted for faster evaluations.


[BUG] There was a problem generating

Background
User testing of the webapp revealed an issue with the viewer where some forms fail to display. This could be an issue with the underlying XML or with the transform. This occurs in 4 out of 30+ of the tax forms in the Consolidated Group XML forms.
Steps to Reproduce

Navigate to Beta MeF Viewer
Open Consolidated Group XML provided by Joe
Attempt to open 1118 Schedule J form, this should produce the error: "There was a problem generating 1118 Schedule J." at the top of the webapp.

Acceptance Criteria
Fix is merged and deployed to the beta site.

Seeing the following errors in the console:

TypeError: XSLTProcessor.importStylesheet: Argument 1 is not an object.

render transform.js:332
generateAndDisplayForm transform.js:251

Root cause is an issue with the Schedule J 1118 template, see https://github.com/newjersey/taxation-mef-viewer/blob/main/mef/Stylesheets/2020/IRS1118ScheduleJ.xsl#L977.


Enable other LR schedulers and expose loss history in LabelModel.fit()

Is your feature request related to a problem? Please describe.
I was working on a text classification problem and have been calling LabelModel.fit() with the default parameters. Since the fit() method does not print out the loss by default, I did not realize that training loss was not converging. I have been debugging the labels assigned by snorkel to some samples and during this process I wanted to look at loss curve. I dug deep into the fit method and noticed that logging of the loss is being done with logging.info() (see here and here).
I had to import logging and set the level to info to get the losses printed. This showed me that loss has not yet converged and I had to increase the number of epochs and play with the learning rate a bit to get it converged.
I suggest printing the losses by default or (or add some verbose parameter to control it) with some reasonable default value for log frequency. That way the user will be able to see if the loss converged or not and take appropriate actions.
I would also suggest returning the loss history to the user in the form of an array to enable plotting of the loss curve. Looking at the loss curve gives a sense of whether the learning rate is low or high or just right.
Also, I noticed here that only four types of learning rate (LR) schedulers are supported -- ["constant", "linear", "exponential", "step"]. I would suggest exposing other schedulers in torch.optim.lr_scheduler. Especially, a scheduler like ReduceLROnPlateau  which requires less tuning may be very useful here.
I am curious if there has been any experiment comparing the performance of different LR schedulers and optimizers for fitting the LabelModel. And if so, what emerged as a recommended approach that worked in most cases.

Hi @cdeepakroy, thanks for the suggestions! We're looking into a way to provide a better default logging level when training the label model. As far as optimization goes: we've found SGD with the provided LR schedulers to be a reliable approach in most cases. We've also seen some gain from using Adam in certain cases. Unfortunately, there's no one-size-fits all here, so we recommend trying out different settings (as it sounds like you're doing already!).


Add functional interfaces for creating DSLs to core-api

Java land the typical () -> Unit lambda will force a return of Unit.INSTANCE
MyThing.Builder(b -> {
    // TODO
    return Unit.INSTANCE;
});

By using functional interfaces, the return can be eliminated
fun interface ThisBlock<in T: Any> {
    operator fun T.invoke()
}

inline fun <T: Any> T.apply(block: ThisBlock<T>): T {
    with(block) { invoke() }
    return this
}

class MyThing private constructor(
    val stuff: List<String>
) {

    companion object {
        @JvmStatic
        fun Builder(block: Block<ActualBuilder>): MyThing {
            return ActualBuilder().apply(block).build()
        }
    }

    // ...
}

Now from Java
MyThing.Builder(b -> {
    // TODO
});

Actually, no... kmp-tor-resource will not utilize this as there is nothing to configure, nor will core-resource because that's all "internal". Leave it in kmp-tor module runtime-api


Issues with detection of Python on codespaces

##Test environment
Create a codespace from https://github.com/github/codespaces-jupyter
Problem 1: Two instead of 1
The following two envs are the same, curren is a symlink to 3.10.13
Problem 2 : Incorrect sys.prefix
The exes /usr/local/python/current/bin/python and /usr/local/python/current/bin/python3
are symlinks to the same exe /usr/local/python/3.10.13/bin/python3.10
However they produce different sys.prefix values.
Problem 3:  We are spawning
[2024-06-28T04:24:37Z WARN  pet::locators] Unknown Env ("/home/codespace/.python/current/bin/python") in Path resolved as ResolvedPythonEnv { executable: "/home/codespace/.python/current/bin/python", prefix: "/usr/local/python/3.10.13", version: "3.10.13.final.0", is64_bit: true, symlink: None } and reported as GlobalPaths
Environment (GlobalPaths)
   Executable  : /home/codespace/.python/current/bin/python
   Version     : 3.10.13.final.0
   Prefix      : /usr/local/python/3.10.13
   Architecture: x64
   Symlinks    : "/home/codespace/.python/current/bin/python"
               : "/home/codespace/.python/current/bin/python3"
               : "/home/codespace/.python/current/bin/python3.10"
[2024-06-28T04:24:37Z TRACE pet_python_utils::env] Executing Python: /home/codespace/.python/current/bin/python3 -c import json, sys; print('093385e9-59f7-4a16-a604-14bf206256fe');print(json.dumps({'version': '.'.join(str(n) for n in sys.version_info), 'sys_prefix': sys.prefix, 'executable': sys.executable, 'is64_bit': sys.maxsize > 2**32}))
[2024-06-28T04:24:37Z WARN  pet::locators] Unknown Env ("/usr/local/python/current/bin/python") in Path resolved as ResolvedPythonEnv { executable: "/usr/local/python/current/bin/python", prefix: "/usr/local/python/3.10.13", version: "3.10.13.final.0", is64_bit: true, symlink: None } and reported as GlobalPaths
Environment (GlobalPaths)
   Executable  : /usr/local/python/current/bin/python
   Version     : 3.10.13.final.0
   Prefix      : /usr/local/python/3.10.13
   Architecture: x64
   Symlinks    : "/usr/local/python/current/bin/python"
               : "/usr/local/python/current/bin/python3"
               : "/usr/local/python/current/bin/python3.10"

Solution for Problem 2

Find the real file of the exe (e.g. /usr/local/python/current/bin/python or /usr/local/python/current/bin/python3)
If the file is still in the same directory, then we know what the real exe is, in this case it is /usr/local/python/current/bin/python3.10
Use this real exe to run Python and get the version and sys prefix information

No one else has reported this, hence closing this as wont fix


Why did you remove option to see invocation history?

It was very useful for our team to be able to look at it as regular function app. Please bring it back. Now I only see this message. Yes, I have app insight set up and they are working (Still this message).

Hi @eduards-vavere  you should be able to configure appInsights later or during the create time. What issue are you facing here?

I'm talking about the invocation history! What was there before this popup!

No it was not removed , here is the screenshot of a function app invocations

OK, thanks. I will see if somehow I can get it work.

I still don't see the invocation history. Only prompt to attach AI. I have app insight attached. It only offers me to change the app insight to different one.
I am using:
{
name: 'APPLICATIONINSIGHTS_CONNECTION_STRING'
value: appInsightConnectionString
}
And in UI it shows correctly as attached.

Can you please let me know which trigger is this?

Checked all these triggers - Service Bus, Queue, Http and timer.  I checked all my 3 function apps - all have the warning in the picture.

It was working before in March.
Under function app / Application Insights
"Your app is connected to Application Insights resource: name-of-my-app-insight"

Hi @eduards-vavere , there is an issue with invocations tab in the portal. The fix is being rolled out and ETA is May 20. If this is urgent, please email me -<EMAIL_ADDRESS>, we can unblock you.

Thanks for info and reply! :)

It started to work again today. Thanks

Should be stable now. Please let me know the if you are seeing any errors


Cache directory fingerprint as a hash of file fingerprints

Closes #68.

Codecov Report

Merging #70 (7e0bce9) into master (775daf0) will decrease coverage by 0.41%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
- Coverage   92.61%   92.19%   -0.42%
==========================================
  Files           3        3
  Lines         406      410       +4
  Branches       50       51       +1
==========================================
+ Hits          376      378       +2
  Misses         18       18
- Partials       12       14       +2

Impacted Files
Coverage Δ

src/fscacher/cache.py
90.44% <100.00%> (+0.28%)
:arrow_up:

src/fscacher/tests/test_cache.py
92.85% <0.00%> (-0.76%)
:arrow_down:

Continue to review full report at Codecov.

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

superseded by #71


Direct links to vcredist files for VC++ 10.0 SP1 are out of date

Direct links to vcredist files for VC++ 10.0 SP1 are out of date. The links point to version 10.0.40219.1, while the download page at https://www.microsoft.com/en-ca/download/details.aspx?id=26999 has version 10.0.40219.235

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

ID: 75e6432b-c6ec-e208-5f1c-bbc4fde9929b
Version Independent ID: 9cbb940e-cddd-f82f-e8bb-ee62b17af794
Content: Latest supported Visual C++ Redistributable downloads
Content Source: docs/windows/latest-supported-vc-redist.md
Product: visual-cpp
Technology: cpp-windows
GitHub Login: @corob-msft
Microsoft Alias: corob

@amandeepmand
Thanks for the report. An updated version of the article is available now.


New theme: Blogpaper

Theme submission
Please make sure that you have read the theme submission guidelines before submitting a theme. The guidelines provide all relevant information and requirements that have to be fulfilled before the submission. Please tick the relevant boxes for your theme in the checklist below:
Link to my theme repository: https://github.com/NormandErwan/Blogpaper
I made sure that...

[x] the repository contains a good README.md describing my theme
[x] an open source license has been added to LICENSE.md
[x] all metadata have been added to theme.toml
[x] screenshots have been added in the images/ folder with the required dimensions
[x] in case I'm using a customized demo via the exampleSite folder that

[x] https://example.com is set as base url in exampleSite/config.{toml, yaml, json} to avoid the abuse of unused domains.
[x] I tested that my theme's demo works with the content directory of gohugoio/HugoBasicExample

[x] I tested my theme against the gohugoio/HugoBasicExample

[x] I've checked the developer tools' console in my browser for error messages

[x] in case my theme is using Hugo Pipes features like toCSS and PostCSS that I have committed the /resources directory with all generated assets, for my theme to work in the basic version of Hugo

N.B. By submitting a theme to the Hugo Themes Showcase you understand that you need to maintain your theme. You will need to keep an eye on whether your theme's demo generates on the Hugo website, whenever there is a new release. We are a very small team and we just cannot keep up opening issues in the various theme repositories to notify authors. Whenever a theme demo breaks and remains broken then at some point it will be removed from the list without prior warning. If you no longer wish to maintain a theme please let us know.
Question: I don't understand "copy the resources folder". The latest themes added to this repo I checked don't tick this checkbox.
I'm using toCSS. Do I need to copy the generated .content and .json on the resources/_gen/assets/scss/css/ folder, when I run hugo serve?
When I run hugo, a CSS is generated CSS on the public/. Do I need to put it on a public/resources folder instead and copy it?
Note: Instead of https://example.com, I set https://normanderwan.github.io/Blogpaper/ as base url to have a working Github Pages.

Hello @NormandErwan
First of all when I test your theme with the Hugo Themes Build Script I get the following ERROR:
 ==== PROCESSING  Blogpaper  ======
Building site for theme Blogpaper using default content to ../themeSite/static/theme/Blogpaper/
Error: Error building site: failed to render pages: render of "home" failed: execute of template failed: template: index.html:3:3: executing "index.html" at <partial "head" .>: error calling partial: "/home/alex/work/src/hugoThemes/Blogpaper/layouts/partials/head.html:49:36": execute of template failed: template: partials/head.html:49:36: executing "partials/head.html" at <resources.Concat>: error calling Concat: resources in Concat must be of the same Media Type, got "text/x-scss" and "text/css"

The above is quite descriptive so when you look into it please let me know.

Regarding your questions:

I don't understand "copy the resources folder". The latest themes added to this repo I checked don't tick this checkbox.
I'm using toCSS. Do I need to copy the generated .content and .json on the resources/_gen/assets/scss/css/ folder, when I run hugo serve?

No. You need to commit the entire resources directory with all generated resources contained within preferably under the `/exampleSite/ directory.
Also note that whenever you perform changes to your stylesheets or JS you will always need to re-generate and commit the updated resources again.

When I run hugo, a CSS is generated CSS on the public/. Do I need to put it on a public/resources folder instead and copy it?

No. You do not need to copy anything from the /public/ folder. The resources are generated automatically and you do not need to do anything to theme. Also note that you should not commit the /public/ directory in your theme's repo, since it is not needed.

Instead of https://example.com, I set https://normanderwan.github.io/Blogpaper/ as base url to have a working Github Pages.

The requirement is due to the abuse of unused domains.
However since you are the owner of https://normanderwan.github.io/Blogpaper/ and you need this for a functional GitHub Pages demo I suppose that we can make an exception as long as @digitalcraftsman also agrees.

However since you are the owner of https://normanderwan.github.io/Blogpaper/ and you need this for a functional GitHub Pages demo I suppose that we can make an exception as long as @digitalcraftsman also agrees.

I'm fine with this decision.

Hello,
Thanks both of you for your quick replies!
I committed the exampleSite/resources folder. The error you got is related to this folder missing I think. I tested the example site with both standard Hugo and Hugo extended binaries. It works well now for both versions. Please tell me if it's ok with your build script.

Hello @NormandErwan
I am still getting the ERROR on my end.
Which Hugo version do you have installed locally?
It doesn't seem like it is a recent one, because the resources directory does not have the folder structure created by recent versions.
Please do the following:

Delete the existing resources directory.
Download and install the latest Hugo Extended
Regenerate the resources by executing hugo server at the root of the exampleSite directory
Re-commit the new resources folder in its entirety i.e. with its generated folder structure (it is important to keep it as is).

And let me know so that I can review your theme further.

I'm using the latest version (0.62.1). I've just tried reinstalling Hugo Extended, deleting the resources and regenerating them with hugo server. I'm having the same result on exampleSite/resources.
I also tried to hugo server with Hugo standard on a fresh clone of the example site of the theme, it worked without errors.
Is the error identical or is it another one?

The ERROR is identical to the one I mentioned above.
When I generate the theme's resources on my own I get the following folder structure and your theme's demo generates when I execute the Build Script even with the Basic version of Hugo:
resources
└── _gen
    ├── assets
    │   └── scss
    │       ├── blogpaper
    │       │   └── css
    │       │       ├── blogpaper.scss_570f86a3dfadba483721b04b00f4d7f2.content
    │       │       └── blogpaper.scss_570f86a3dfadba483721b04b00f4d7f2.json
    │       └── css
    │           ├── blogpaper.scss_570f86a3dfadba483721b04b00f4d7f2.content
    │           └── blogpaper.scss_570f86a3dfadba483721b04b00f4d7f2.json
    └── images

The resources that you have provided in the theme repo have the following different folder structure:
resources
└── _gen
    ├── assets
    │   └── scss
    │       └── blogpaper
    │           └── css
    │               ├── blogpaper.scss_570f86a3dfadba483721b04b00f4d7f2.content
    │               └── blogpaper.scss_570f86a3dfadba483721b04b00f4d7f2.json
    └── images

As a result of this different folder tree Hugo fails to find the generated resources and the demo does not generate with the Basic version of Hugo.

I'm using Windows x64.
I tried to find across themes how SCSS is used: I've only found themes converting manually from SCSS and committing the converted CSS.
It seems to me that the Hugo's SCSS feature is not stable yet. Since the development of my theme is done, I removed the SCSS and converted it to CSS.
Can you try this new version?

I've only found themes converting manually from SCSS and committing the converted CSS.

Have you read the theme submission guidelines?

If your theme makes use of the Hugo Pipes methods toCSS and / or PostCSS, you need to take additional steps to make your theme compatible with the basic version of Hugo:
From the root of a Hugo project that uses your theme execute the command hugo server -this will generate the /resources/ folder under the project root-.
Or if you prefer to build the project simply run the command hugo -this will generate the resources under public/resources-.
Copy the generated resources folder and commit it directly either under the root of your theme's repository or under exampleSite.
Whenever you make any changes in your Hugo Pipes assets, you will need to regenerate and commit the theme's resources again.

There are plenty of themes that use Hugo Extended and commit their resources, like for example the Academic Theme
I will try the new version tomorrow and let you know.

Ok I found the problem. It's related to the baseURL variable in exampleSite/config.yml.
On my theme it's set to https://normanderwan.github.io/Blogpaper/. Running hugo server with Extended creates a web server at http://localhost:1313/Blogpaper and generate CSS files to exampleSite\resources\_gen\assets\scss\blogpaper\scss.
But your building script is using another baseURL set to http://localhost:1313. And Hugo Basic looks for generated CSS files in exampleSite\resources\_gen\assets\scss\scss.

Ok. The resources no longer cause any problems and the demo looks fine
However I have noticed the following minor CSS issue:
-In the content file under posts/emoji-support/ the text in lines 30-32 overflows the container. To make the text wrap you need to use the CSS mentioned over here

Great news!
I've just added text wrap on <pre> like you mentioned and pushed the new version of the theme.

@NormandErwan
Ok. I added your theme to the repo in ec3d421

@digitalcraftsman
This is a graphical newspaper like blog theme for Hugo.

I've promoted your theme on Hugo's official Twitter account.

Thank you very much, both of you!


On Windows, use c++20 mode as correctly needed for designated initial…

…izers.
Designated initializers is technically only supported starting with c++20, and the compiler on Windows points that out with an error.

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

Comparison is base (cfcbb82) 92.84% compared to head (0196432) 92.84%.
 Consider uploading reports for the commit 2fecf38 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2049   +/-   ##
=======================================
  Coverage   92.84%   92.84%
=======================================
  Files         357      357
  Lines       26396    26396
=======================================
  Hits        24507    24507
  Misses       1889     1889

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


Update Nginx config file generator and add support for connection upgrade

Adds support for HTTP 1.1 and connection upgrades, as well as improving formatting and better use of the StringBuilder
The fix for the NPE when no SSL config is provided has now been merged as #59

@grkvlt what tests should we run to confirm that this hasn't broken previous use-cases, and that it  now supports the use-case that motivated you to make this change?

@aledsage the existing Nginx tests are sufficient to verify there are no regressions, however the use-case for this code is no longer as urgent. It is still a useful change, since it supports HTTP 1.1 and connection upgrades properly now, and the code is also more readable.
The change to the config file that adds this is the addition of these two groups of lines:
    map $http_upgrade $connection_upgrade {
      default Upgrade;
      ''      close;
    }

    proxy_http_version 1.1;
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection $connection_upgrade;

@grkvlt the server.conf used with NginxTemplateConfigGenerator follows roughly the config from NginxDefaultConfigGenerator. Can you update it as well.
I think we should just migrate to the template configuration and deprecate NginxDefaultConfigGenerator altogether as its easier to support.

@grkvlt Can you take a look at this please to see if it is still relevant, and address the comments above if appropriate
Thanks

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/brooklyn-library-pull-requests-pipeline/7/

P.S. Can you also rebase if this is still valid


testing

Add test for reading from recording extractor and writing
Use https://github.com/SpikeInterface/spikeextractors/tree/master/spikeextractors/example_datasets for generating fake data
Use actions to set up continuous integration similar to NWBWidgets

@Armin12 can you help with this?

Sure!

and let me and/or @mvdm know if you would like some real data for neuralynx systems, we could provide! ;)

What's the best way to integrate real data with CI?
On Mon, Apr 6, 2020, 10:35 AM Yaroslav Halchenko<EMAIL_ADDRESS>wrote:

and let me and/or @mvdm https://github.com/mvdm know if you would like
some real data for neuralynx systems, we could provide! ;)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/catalystneuro/nwb-conversion-tools/issues/24#issuecomment-609832960,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGOEEXDYF4WI2E5EAGMFHLRLHSBJANCNFSM4MCKFDWQ
.

I guess it all depends on the sizes etc.  We could setup some datalad datasets (could even store data within github lfs while sizes allow for free storage, see datalad handbook for an example), and use datalad to get desired files from the cloudy cloud.

here is e.g. one example in heudiconv where we do some regression testing using data we fetch using datalad (from my server I believe): https://github.com/nipy/heudiconv/blob/ca402a9fa701551bf2c74f72d77223aaad7e5d68/heudiconv/tests/test_regression.py


aws-sam-translator 1.86.0

aws-sam-translator 1.86.0
Destination channel: defaults
Links

Upstream repository
Relevant dependency PRs:

cfn-lint

Explanation of changes:

Update version number
Add pydantic
--no-build-isolation

It seems like you last commit a849462 is undoing the previous changes.

Apologies! rectified this


network/domainspec: Fix lint errors

What this PR does / why we need it:
Fixed import entry errors that reported:
File is not `goimports`-ed with -local kubevirt.io/kubevirt (goimports)

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
NONE

/test pull-kubevirt-code-lint

/test pull-kubevirt-e2e-k8s-1.23-sig-network

/lgtm

/approve

Resolved through #8050


Release 0.14.0

hi there, it'd be super useful to have another release (particularly including the libc export fixes #1055) so things work on ARM again.
thanks,
ryan

Yep, sure would be nice.

🤣 is there anything blocking this happening or could i just open a PR with a version bump?

In progress as #1061 .

Done!

thanks!


Razer not detecting chroma app

so i am not sure if this is an issue on your end, or razer synapse, but it will NOT detect aurora as a chroma app, which means i can not have any effects on my keyboard. it worked like a month ago, but then my pc had some issues, and i had to re install windows, and wipe all my previous apps and data. ever since then this will not work for razer products.

Did you follow the setup instructions for Razer?

yes, i did. i also tried un-installing aurora, and razer synapse. i tried toggling the chroma apps off, then back on, and it didn't work.
and also, since i didn't mention it in my original post, i am running everything as administrator,
and now after trying to re-install aurora again i am getting the following error message (it re-occurs every time i try): http://imgur.com/a/gRPWJ
as for the error i am convinced i am just being dumb and doing something wrong lol.

For that last issue, you'll need to delete %appdata%/Aurora/Config.json. It's a known issue when going back to v0.6.0b after using a dev release.

ok yeah i figured the error was me doing something wrong lol.
after re installing it, it still wont show as a chroma app in synapse, and wont allow lighting control over my keyboard or headset.

Can you have a look at what is shown in the Devices section for Razer? Could you also make sure that the SDK has been enabled and post the log file if you can't get it working?

ok this time a re install of synapse seems to have worked.
i don't know why it worked this time, but not last time. most likely some form of user error, but god only knows with razer synapse lol.

Okay, I'm glad you got it sorted.


pull request for e2e test at 11/08/2016 02:56:10

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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


Tracking Issue for multilingual clients document catching up

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
We have a Client Libraries page to host all official client libraries documents. Despite the Java client's document being relatively completed, documents for multilingual clients are outdated and less maintained.
This issue tracks the effort to document what we already implemented for multilingual clients, focusing on the C++ client, the Python client, and the Node.js client - all based on the core C++ client library.
Solution
The catching-up plan is based on:

Java client's document for the overall structure.
Feature Matrix for what we already implemented.

Layout
Top-level layout (continuously updating):

Brief introducation
Installation
Connection URLs
Client
Producer
Consumer
Reader
Table View
Schema
Authentication

Common

[x] #17303
[ ] #17307

C++

[ ] Simplify the Install section and promote a top-level "Install" title

Python
Node.js
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

@RobertIndie @shibd when you merge some patches requiring docs, you can cc me in this thread and I'm glad to track them and help on the docs side.

Closed as halted and partially delivered. We moved out the C++/Python client from the main repo, and I've worked on the API docs, which are delivered as #17917.
Sooner or later we can restructure the client library in the proposed structure above, but it will be a new turn.