dotnet-sdk: Allow multiple packages

Motivation for this change
Colleagues have switched some projects to version 3.0 of .NET Core. I want my development environment to work with all versions.
This changes allow to combine multiple SDKs and runtimes.
in configuration.nix
  dotnetCombined = with dotnetCorePackages; combinePackages {
    cli=sdk_3_1_preview;
    withSdks = [ sdk_2_2 sdk_3_0 sdk_3_1_preview ];
    withRuntimes = [ aspnetcore_2_1 sdk_3_0 sdk_2_2 sdk_3_1_preview  ];
  };

This setup worked so far. I am open for suggestions.
Using the runtime packages, it should also be possible to build smaller docker images for deployment.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @kuznero
Related issues
https://github.com/NixOS/nixpkgs/issues/73193
https://github.com/NixOS/nixpkgs/pull/72300
https://github.com/NixOS/nixpkgs/pull/69392
https://github.com/NixOS/nixpkgs/pull/69382
https://github.com/NixOS/nixpkgs/pull/69391

I do like the idea of being able to compose many sdk's. I will have to take a closer look when I get back from work

I can confirm this is something a Dot Net developer could need : I'm currently working on some legacy projects (2.2) and some new projects that are working with 3.0 only features (Like some Generic Hosts extensions).
I'll will give a try to this PR this week.

I've been using this work in a project of mine with no issues on Ubuntu, Windows WSL2, and a modified version of it for macOS. Excellent work and IMO exactly what I think the dotnet package should look like. @baracoder thank you!

I've only tried it with v3.1 (stable) of the SDK yet but that has been working without any issues for quite a few project.
Since 3.1 is stable by now it should also be added to the list.
To save you some time this is what I've used:
sdk_3_1 = buildNetCoreSdk {
  version = "3.1.100";
  sha512 = "0hvshwsgbm6v5hc1plzdzx8bwsdna2167fnfhxpysqs5mz7crsa4f13m4cxhrbn64lasqz2007nhdrlpgaqvgll6q8736h884aaw5sj";
};

Thank you all for the suggestions. I have updated the branch with the changes and replaced sdk_3_1_preview with sdk_3_1.
When using Rider, I had to run dotnet clean and invalidate cache to catch up.

Great! Maybe update the default version in pkgs/top-level/all-packages.nix too?
-  dotnet-sdk = dotnetCorePackages.sdk_2_2;
+  dotnet-sdk = dotnetCorePackages.sdk_3_1;

I am not sure about this change. This could break some of the packages, which depend on the current version of dotnet-sdk:
% grep -irnl dotnet-sdk
..
pkgs/applications/blockchains/wasabiwallet/default.nix
pkgs/development/python-modules/dotnetcore2/default.nix
pkgs/development/tools/build-managers/msbuild/default.nix
pkgs/servers/nosql/eventstore/default.nix
pkgs/servers/jellyfin/default.nix
pkgs/top-level/python-packages.nix

I am not sure about this change. This could break some of the packages, which depend on the current version of dotnet-sdk:
% grep -irnl dotnet-sdk
..
pkgs/applications/blockchains/wasabiwallet/default.nix
pkgs/development/python-modules/dotnetcore2/default.nix
pkgs/development/tools/build-managers/msbuild/default.nix
pkgs/servers/nosql/eventstore/default.nix
pkgs/servers/jellyfin/default.nix
pkgs/top-level/python-packages.nix

Ah, right. Never mind then - I suppose updating (which includes checking each of these and possibly fixing them to 2.2) warrants a separate issue in any case.

@worldofpeace do you see anything out of the ordinary? it LGTM
the reason why such a construction is necessary is:
dotnet makes the assumption that it's able to write to a directory relative to the executable, which ends up being in the nix store, but if the sdk/runtime is already available, then it skips this step. By allowing the sdks/runtimes to be composed through the dotnetPackages.combinePackages function.

@baracoder I still haven't forgot this PR, I just keep getting sidetracked with other things :(

the problem with linking to sdk's as "runtimes" is that you don't really get any closure benefit, you bring in the whole other closure once you do that.

Thank you for testing and the input.
with dotnetCorePackages; combinePakcages { cli=sdk_3_1; withRuntimes=[ sdk_3_1 ]; withSdks=[ sdk_3_1 ]; }

Using the same sdk package in runtimes won't increase the closure size, as the directories are just linked.
I had the wrong assumption that explicitly specifying the runtime would decrease closure size for built packages, but this is false, because the cli from the combined package would be used and pull in all SDKs.
combinePackages is only be useful for development environments.
I will update the PR tomorrow.

also, I don't think the cli argument is necessary, could order the sdks and runtimes first by sdk, then by version, then just choose the first one.

I did not have time to update the pull request, but stumbled today on a nice nixpkgs function buildEnv. Which makes it look like I have tried to reinvent the wheel:
  latestCombined = buildEnv {
    name = "dotnet-combined-latest";
    paths = [ sdk_3_1 sdk_2_2 sdk_3_0 sdk_3_1 ];
    pathsToLink = [ "/host" "/packs" "/sdk" "/shared" "/templates" ];
    ignoreCollisions = true;
    postBuild = ''
      cp ${sdk_3_1}/dotnet $out/dotnet
      mkdir $out/bin
      ln -s $out/dotnet $out/bin/
    '';
  };

It works a little bit different but simplifies the process a lot.

This change pleases me greatly.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/why-cant-i-install-dotnet-in-my-rust-build-environment-shell/5786/1

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/what-are-your-goals-for-20-03/4773/16


Typeahead only works on arrays of strings

It would be nice if Typeahead allowed arbitrary data sources without needing to redefine sorter/matcher/highlighter.
This could be accomplished with 2 additional options:

get(): would get passed an item, and it returns the value to match & sort against.  The built-in would just return the item
getDisplay(): would get passed an item, and it returns the value to display.  The built-in would just return the item.

The built-in matcher & sorter functions would then use this.get(item) rather than just item.
The built-in highlighter function would use this.getDisplay(item) rather than just item

Related to issue #1751

adding this would make everything slow. We definitely don't want to be calling a function for every item we're matching against.


Crash from jvp(jit(vmap(custom_jvp(in-place-update-on-size-zero-array))))

Description
Yes, I know what that title sounds like. You don't know want to know how long this has taken me to track down.
First of all, a repro:
import jax
import jax.numpy as jnp

@jax.custom_jvp
def f(x):
    y = jnp.zeros((1,) + x.shape)
    return y.at[0].set(x)

@f.defjvp
def f_jvp(primals, tangents):
    (x,) = primals
    (t_x,) = tangents
    out = jnp.broadcast_to(x, (1,) + x.shape)
    t_out = jnp.broadcast_to(t_x, (1,) + t_x.shape)
    return out, t_out

x = tx = jnp.zeros((1, 0))
jax.jvp(jax.jit(jax.vmap(f)), (x,), (tx,))

which raises the following error from within JAX:
  File ".../jax/_src/lax/lax.py", line 2849, in _broadcast_in_dim_abstract_eval
    shape = _broadcast_in_dim_shape_rule(  # error checking
  File ".../jax/_src/lax/lax.py", line 2686, in _broadcast_in_dim_shape_rule
    raise TypeError(msg.format(broadcast_dimensions, operand_ndim))
jax._src.traceback_util.UnfilteredStackTrace: TypeError: broadcast_in_dim broadcast_dimensions must have length equal to operand ndim; got broadcast_dimensions (1, 2) for operand ndim 3.

Second of all, the root cause:
import jax
import jax.numpy as jnp

def g(x):
    y = jnp.zeros((1,) + x.shape)
    return y.at[0].set(x)
z = jnp.zeros((0,))
print(jax.make_jaxpr(g)(z))
#{ lambda ; a:f32[0]. let
#    b:f32[1,0] = broadcast_in_dim[broadcast_dimensions=() shape=(1, 0)] 0.0
#    _:i32[1] = broadcast_in_dim[broadcast_dimensions=() shape=(1,)] 0
#  in (b,) }

This snippet demonstrates that in-place updates to size-zero arrays are ignored, which implies that the output has no data dependency on the input, and the batch tracer is not promulgated.
However, returning to our original example: the JVP rule implementation does propagate batch tracers through size-zero arrays.
As a result the primal and tangent branches of the vmap(custom_jvp) get different shapes, and JAX crashes.
For completeness, here's the full traceback:
Traceback (most recent call last):
  File "main.py", line 20, in <module>
    jax.jvp(jax.jit(jax.vmap(f)), (x,), (tx,))
  File ".../jax/_src/traceback_util.py", line 166, in reraise_with_filtered_traceback
    return fun(*args, **kwargs)
  File ".../jax/_src/pjit.py", line 212, in cache_miss
    outs, out_flat, out_tree, args_flat = _python_pjit_helper(
  File ".../jax/_src/pjit.py", line 159, in _python_pjit_helper
    out_flat = pjit_p.bind(*args_flat, **params)
  File ".../jax/_src/core.py", line 2628, in bind
    return self.bind_with_trace(top_trace, args, params)
  File ".../jax/_src/core.py", line 383, in bind_with_trace
    out = trace.process_primitive(self, map(trace.full_raise, args), params)
  File ".../jax/_src/interpreters/ad.py", line 315, in process_primitive
    primal_out, tangent_out = jvp(primals_in, tangents_in, **params)
  File ".../jax/_src/pjit.py", line 1431, in _pjit_jvp
    jaxpr_jvp, is_nz_tangents_out = ad.jvp_jaxpr(
  File ".../jax/_src/interpreters/ad.py", line 699, in jvp_jaxpr
    return _jvp_jaxpr(jaxpr, tuple(nonzeros), tuple(instantiate))
  File ".../jax/_src/interpreters/ad.py", line 709, in _jvp_jaxpr
    jaxpr_out, avals_out, literals_out = pe.trace_to_jaxpr_dynamic(f_jvp, avals_in)
  File ".../jax/_src/profiler.py", line 314, in wrapper
    return func(*args, **kwargs)
  File ".../jax/_src/interpreters/partial_eval.py", line 2150, in trace_to_jaxpr_dynamic
    jaxpr, out_avals, consts = trace_to_subjaxpr_dynamic(
  File ".../jax/_src/interpreters/partial_eval.py", line 2172, in trace_to_subjaxpr_dynamic
    ans = fun.call_wrapped(*in_tracers_)
  File ".../jax/_src/linear_util.py", line 188, in call_wrapped
    ans = self.f(*args, **dict(self.params, **kwargs))
  File ".../jax/_src/core.py", line 229, in jaxpr_as_fun
    return eval_jaxpr(closed_jaxpr.jaxpr, closed_jaxpr.consts, *args)
  File ".../jax/_src/core.py", line 447, in eval_jaxpr
    ans = eqn.primitive.bind(*subfuns, *map(read, eqn.invars), **bind_params)
  File ".../jax/_src/core.py", line 380, in bind
    return self.bind_with_trace(find_top_trace(args), args, params)
  File ".../jax/_src/core.py", line 383, in bind_with_trace
    out = trace.process_primitive(self, map(trace.full_raise, args), params)
  File ".../jax/_src/interpreters/ad.py", line 315, in process_primitive
    primal_out, tangent_out = jvp(primals_in, tangents_in, **params)
  File ".../jax/_src/lax/lax.py", line 2796, in _broadcast_in_dim_jvp_rule
    y = broadcast_in_dim_p.bind(operand, *dyn_shape, shape=shape,
  File ".../jax/_src/core.py", line 380, in bind
    return self.bind_with_trace(find_top_trace(args), args, params)
  File ".../jax/_src/core.py", line 383, in bind_with_trace
    out = trace.process_primitive(self, map(trace.full_raise, args), params)
  File ".../jax/_src/interpreters/partial_eval.py", line 1854, in process_primitive
    return custom_staging_rules[primitive](self, *tracers, **params)
  File ".../jax/_src/lax/lax.py", line 2770, in _broadcast_in_dim_staging_rule
    return trace.default_process_primitive(broadcast_in_dim_p, (x,), params)
  File ".../jax/_src/interpreters/partial_eval.py", line 1859, in default_process_primitive
    out_avals, effects = primitive.abstract_eval(*avals, **params)
  File ".../jax/_src/core.py", line 416, in abstract_eval_
    return abstract_eval(*args, **kwargs), no_effects
  File ".../jax/_src/lax/lax.py", line 2849, in _broadcast_in_dim_abstract_eval
    shape = _broadcast_in_dim_shape_rule(  # error checking
  File ".../jax/_src/lax/lax.py", line 2686, in _broadcast_in_dim_shape_rule
    raise TypeError(msg.format(broadcast_dimensions, operand_ndim))
jax._src.traceback_util.UnfilteredStackTrace: TypeError: broadcast_in_dim broadcast_dimensions must have length equal to operand ndim; got broadcast_dimensions (1, 2) for operand ndim 3.

What jax/jaxlib version are you using?
HEAD
Which accelerator(s) are you using?
No response
Additional system info
No response
NVIDIA GPU info
No response

Wow, thanks for diagnosing this, Patrick! Sorry it took so long to diagnose but... I really appreciate you doing it!
And thanks for taking JAX's composable transformations seriously :P

Happy late birthday to this issue!
Because I'm not going to fix this now (sorry), a note to future self: I think we need DynamicJaxprTrace.process_custom_jvp_call to run both fun and jvp (rather than delaying the latter), and for BatchTrace.process_custom_jvp_call to prepare that case (i.e. not only ready for just one or the other running, as it is now, but also being ready for both being run) and join the output batching as appropriate. This is essentially the logic that @dougalm thought of for a final-style cond back in 2018. (The reason we delay running jvp in a thunk is to avoid an infinite mutual recursion in case jvp calls fun internally... we'll have to sort that out.)
The best temporary workaround I can think of is to ensure f and f_jvp have the same primal-input-to-primal-output data dependencies. An easy way to do that is to compute the primal outputs in f_jvp just by calling f.
We could also catch this error earlier; the earliest we can at the moment is above this line in DynamicJaxprTrace.process_custom_jvp_call. That's when we should realize the JVP function's output primal types don't match those of the primal function's. But at that point it's too late to fix things.
We could patch up this one case by making y.at[0].set(x) have a data dependence on x even in the case where x and/or y are size 0. That fix feels bad to rely on but at least it would've saved Patrick a lot of time.


Feature/x fabric static response

eature: x-fabric-static-response
based on feature/x-fabric-employee-access

👍

:+1:


How to render html element when exporting to pdf

{
"components": [
{
"label": "HTML",
"className": "",
"attrs": [],
"content": "Please fill all the details below.",
"refreshOnChange": false,
"mask": false,
"tableView": true,
"alwaysEnabled": false,
"type": "htmlelement",
"input": false,
"key": "html"
},
{
"label": "Email",
"tableView": true,
"alwaysEnabled": false,
"type": "email",
"input": true,
"key": "email2"
},
{
"label": "Phone Number",
"allowMultipleMasks": false,
"showWordCount": false,
"showCharCount": false,
"tableView": true,
"alwaysEnabled": false,
"type": "phoneNumber",
"input": true,
"key": "phoneNumber2"
},
{
"type": "button",
"label": "Submit",
"key": "submit",
"disableOnInvalid": true,
"theme": "primary",
"input": true,
"tableView": true
}
]
}
above is the component structure, I tried but couldn't find out, can you guys please help me to render the html data, 1st object in the above array. Thanks.

I am having same issue. Please provide a solution

Hello, is there any update on this? I am having issues rendering these components when exporting to PDF as well.

@airarrazaval Could you help me out with this issue please?

Hey @airarrazaval , could you give me right to create a branch, I just add support to HTMLElement to your package. You can review my PR once its done.
Thanks

@marcelcruz1 check out this repository, it is created from @airarrazaval repo but they added html and content support.
https://github.com/multinerd/formio-export

@0LeoG , you can find the implementation at https://github.com/marcelcruz1/formio-export

@marcelcruz1 do you know how I could get an object from a form submissions so I can export a form with real values?


Fixed broken links in deprecation assertions for framework and surface modules

Description of changes
When importing deprecated modules microsoft/surface/default.nix and framework/default.nix, a link is shown to a markdown document guiding the user to import the new modules. These links are broken because they point to a non-existent OLD-BEHAVIOUR-DEPRECATED.md. This patch fixes them to point to the existing OLD-BEHAVIOUR-DEPRECATION.md documents.
Things done

[x] Tested the changes in your own NixOS Configuration (attempted to build to display the assertion and make sure they link to the correct files).
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/error-cached-failure-of-attribute-nixosconfigurations-framework-config-system-build-toplevel/35451/4

@mergify queue


Bugfixes for spectrogram functions

Which issue(s) are closed by this pull request?
Closes #255
Changes proposed in this pull request:

remove duplicate spectrogram plot functions from plot.line and plot._line
move dB handling from plot.spectrogram to dsp.spectrogram
update test_plot for removed spectrogram plot function
add tests for dsp.spectrogram

I deleted C._t (VSCode keeps generatin it, never looked up, what it is)
I am in favor of removing the dB option from dsp.spectrogram. However this would break backwards compatability. Let's discuss at the weekly...


Add support for a new 'process' action type

What would you like to be added:
Add support for a 'process' action type, which would allow users to actually run simple shell/bash commands
Why is this needed:
In some case, users might have no other choice than to run bash commands, for example to install a self-signed certificate in a worker's isolated computing unit.

@tsurdilo @ricardozanini What do you guys think? If I've your green light, I'll work ASAP on a PR to address that ;)

What are the security implications? Adding a process action can result in a security breach. Some users might not want to have it enabled by default. I can see implementations not implementing or having to add an option to disable process actions. This way, we might lose the interoperability "no, process action we don't work with." I'm okay with this assumption, btw. We need to clarify.

+100, I totally agree. Security implications are however minimal in a Docker context, for example.

This action sounds like an infrastructure requirement. Are we going down this path?

No, absolutely not! If I ever do, please shoot me out of my misery 😆 I just had a couple of real-life scenario where such this would have saved the team from developing a dedicated FaaS.
But yeah, it's probably a very bad idea nonetheless, mainly for cross-platform concerns. As a matter of fact, most commands are different in Windows, Linux and Mac. Hell, even escaping rules are different in Windows if you run in Powershell or in classic Cmd.
Just wanted to put it out there, in case we can find a way to do that in a proper, clean and sustainable way.

Would it make sense to use functions type "custom" to run scripts?

@tsurdilo WDYT?

@cdavernas i kinda like what Argo is doing: https://github.com/argoproj/argo-workflows/blob/master/examples/scripts-bash.yaml#L29

Yeah looks nice!
Are you proposing to use the image part of it, too?

@cdavernas the question is, where is this command going to run?

Well, imo should run in something containerized, so we can address both the security and cross compatibility concerns at once. In that case, 'process' could be understood as 'container' execution result. I'm not sure, though, if that's not putting too much weight on implementations, as well as make the spec a bit too technical.
However, I defintly can imagine something of the like:
name: MyProcess
kind: process
operation: ubuntu:latest:wget

And call it like:
...
functionRef:
  refName: MyProcess
  arguments:
    - "http://test.com"
    - "--no-cache"


Clustered Conduit

The problem to solve is High Availability. Should one node go down, another will need to take it's place. The goal is not disaster recovery. We will need to develop a separate solution for that.

Random thought about this: we should think about how to work with standalone plugins when Conduit is clustered. The plugin executables need to be found on all instances (if we want to make it possible that any Conduit instance in a cluster can work on any pipeline).

I think this is still pretty far out in the future @neovintage? Especially if there's a conduit Kubernetes operator which can help with orchestration.

Closing, the Conduit Kubernetes operator work will address running Conduit in a distributed manner.


Shadowrun Sixth World v.56 - 3 Bugfixes and 2 New Features

[x] The pull request title clearly contains the name of the sheet I am editing.

[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).

[x] The pull request makes changes to files in only one sub-folder.

[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

[x] I have followed the Character Sheets Standards when building this sheet.

Changes / Description

Bugfix - There were 10 repeating sections that didn't use "name" for the name field.  Fixed this to make other automated tasks easier to achieve in the future.
Bugfix - Cleaned up non-PC sheets to show condition monitors better (including the max)
Bugfix - Fixed how display of matrix device and matrix roll buttons work.
New Feature - added Composure roll button to Grunts
New Feature - normalized the styling on non-PC sheets a little bit to help mark separate sections better.

Character Sheet Info Roll20 Internal Use only.


Application info not fetched when using TouchID

When using TouchID login/signup, the application info is not fetched (through calling fetchAppInfoWithSuccess). This is unlike normal login/signup which does fetch it (in A0LockViewController.m /  A0LockSignUpViewController.m.
This means the sign-up flow for TouchID fails when it attempts to call signUpWithUsername, as this depends on application being set on the API client object.
I'm by no means an Obj-C expert so please advise if I can give any more useful info on this.

By default it's using a connection named Username-Password-Authentication which is the default connection you get when creating your Auth0 account. If you wish to use a different connection just specify so in the property authenticationParameters of A0TouchIDLockViewController like
    A0Lock *lock = [A0Lock sharedLock];
    A0TouchIDLockViewController *controller = [lock newTouchIDViewController];
    A0AuthParameters *parameters = [A0AuthParameters newDefaultParams];
    parameters[A0ParameterConnection] = @"YOUR_CONNECTION_NAME";
    controller.authenticationParameters = parameters;

The only restriction is that the connection MUST be a DB connection

Thanks for getting back to me -- that was the name of my DB connection. I'm using this via React and it turns out the magic incantation is below.
I think the bug is that if you don't manually specify the authParams.connection value, this library falls back to trying to pull it from application and as mentioned above that is unset.
this.state.lock.show(
      {
        connections: [
          'touchid',
        ],
        closable: true,
        authParams: {
          scope: "openid email offline_access",
          connection: 'Username-Password-Authentication',
        },
      },
      (err, profile, token) => {
        console.log('err', err);
        console.log('profile', profile);
        console.log('token', profile);
      }
    );

It's not trying to pull the info at all since it only does for non-passwordless connections. Also it should honor the default if you don't specify one. Probably something wrong in RN bridge

Okay -- will add a PR to that project to fix it at that end. Thanks for your help!

@shauns double checked and it's an issue in this lib. Just submitted and issue for tracking it https://github.com/auth0/Lock.iOS-OSX/issues/220


Example of how to reroute all vehicles [DO NOT MERGE]

Adds an optional param that lets you reroute every vehicle if it is on its final edge.

The checking of the final edge is not valid for certain OSM maps (e.g. this). For most vehicles the route is a single element list.

You can use self.k.vehicle.get_position(veh_id) to figure out the vehicle position and self.k.network.edge_length(edge) to get the edge length and figure out when you are near the edge. Adding that check for single edge routes should solve this issue.

Something like this?
            edge = self.k.vehicle.get_edge(veh_id)
            self.k.vehicle.get_position(veh_id)

            pos=self.k.vehicle.get_position(veh_id)
            length=self.k.network.edge_length(edge)

            eta=length*0.01

            # check if its on the final edge
            if edge == route[-1] and pos+eta>=length:
                type_id = self.k.vehicle.get_type(veh_id)
                # remove the vehicle
                self.k.vehicle.remove(veh_id)

Moreover I get this weird situation in which the previous if statement
 if edge == route[-1] and pos+eta>=length
is False but the vehicle id is in
sim_obs[tc.VAR_ARRIVED_VEHICLES_IDS]  (here)
therefore it is removed.

Yes that's the idea! One possible issue could be your value of eta is too small; if one of your edges is not too long and your speed is high you could travel off the edge before this condition catches it. Could you check if this is what's hapening?
A trick we used to use for this situation (that you could try) is to add long fake edges to every edge in the network that way there's zero chance of this happening. This is why there are weirdly long edges at the end of the grid example.

Using
eta=length*0.1
Seems to solve the problem, I'll try with higher horizons (currently using 10).
Since this method seems a little forced in my opinion, I was wondering if there is a way to extract routes from an OSM map programmatically.

Cool! There isn't one that I know of, at least because it's an ill-posed problem. Technically any route is valid but you wouldn't take some routes (very long ones, routes that contain cycles). We have some work in progress to add this feature but it's not ready yet. If you want to contribute it it would be very appreciated!

I would love to contribute, but my project is due to next Sunday and I don't think I have the time.

Np. Hope it helps and let me know if anything comes up.

I found out that using the add method, you do not reintroduce the agent in the rl_ids. This causes the samples to have inconsistent length, thus leading to the error.
I tried using a custom add funtion:
    def add_rl(self, veh_id, type_id, edge, pos,sumo_obs):

        """See parent class."""
        if veh_id in self.master_kernel.network.rts:
            # If the vehicle has its own route, use that route. This is used in
            # the case of network templates.
            route_id = 'route{}_0'.format(veh_id)
        else:
            num_routes = len(self.master_kernel.network.rts[edge])
            frac = [val[1] for val in self.master_kernel.network.rts[edge]]
            route_id = 'route{}_{}'.format(edge, np.random.choice(
                [i for i in range(num_routes)], size=1, p=frac)[0])

        self.kernel_api.vehicle.addFull(
            veh_id,
            route_id,
            typeID=str(type_id),
            departLane="random",
            departPos=str(pos),
            departSpeed=str(0))

        self._add_departed(veh_id, type_id)

        # modify the number of vehicles and RL vehicles
        self.num_vehicles = len(self.get_ids())
        self.num_rl_vehicles = len(self.get_rl_ids())

But for some reason is not working as expected. Is there a way to reintroduce an RL vehicle immediately after the rerouting so that the next update will consider its observations?

There will be occasions where it’s not possible to reinsert the vehicle immediately. I’d just track the IDs and output a zero vector for that vehicle for the timestep it is missing. That shouldn’t be a problem as long as the amount of time it’s missing is small.

What are such occasions? Why wouldn't it be possible to reroute the agent immediately?

Where you want to insert them is blocked for example.


Microservices to supply authentication token to other EdgeX microservices when security is enabled

🚀 Feature Request
Description
This is a sub-story of https://github.com/edgexfoundry/edgex-go/issues/1950.
Depends on #2456
In order to enable authentication of microservice callers, EdgeX microservices, when calling other EdgeX microservices when security is enabled, should query the EdgeX secret store for their authentication token and embed it in the outgoing request.  The client microservice must ensure that its token is reasonably fresh (e.g. > 15 minutes TTL remaining) else refresh its token before issuing the request.

Archiving this story as this presumes a particular implementation and will leave #1950 open.  Will reopen if decide to go this route.


Standard function naming for directive, controller, service / factory, etc.?

Naming the implementation of the controller as ctrlFn, service as serviceFn, directive as directiveFn, etc.
Good idea or not?
Good reason:

Easier to templatize
One less name to remember and it's always the same for whatever service you created.

Bad probably because of the same reason...

harder to use Find since everything is named the same thing? But then again, you can still search by using the angular name..., if it is one per file, might not be bad?

Example...
//fooService
(function() {
  'use strict';

  angular.module('...')
    .factory('fooService', serviceFn);

  serviceFn.$inject = ['$http','$log',...];
  function serviceFn($http, $log, ...) {
    var service = {
       bar: bar
    };

    return service;

    function bar(...) {
       return $http.get('...');
    }
  }
}());

templates for tooling can adjust to custom names. i am not a fan of changing styles for tooling, i like the other way around.
all being the same name can be more difficult to debug too, when the error shows the function name 'serviceFn'
I prefer my code to follow readability, too.
does it work? sure. but i'm not a fan personally.


use LF instead of CRLF for wider compatibility

fixes #393
fails with postfix on CentOS 6 otherwise

Fix is from #549 - Hopefully fix the tests that are failing in #549

Or maybe PHP_EOL should be used? http://stackoverflow.com/a/6360550

Postfix expects EOL to be LF when picking up mail from sendmail on unix and replaces that with CRLF when sending. When it gets CRLF it still replaces the LF and we get CRCRLF.

This would explain the seen behavior. Maybe Swiftmail should use LF if it uses sendmail with postfix :see_no_evil:

Or maybe PHP_EOL should be used?

That would lead to inconstant behaviour across platforms.

@GrahamCampbell Do you have a good idea how to fix the broken behavior on postfix?

Hmmm, maybe just change the implode statement?

Hmmm, maybe just change the implode statement?

To what?

Headers must follow the standards, which require the usage of CRLF!
https://tools.ietf.org/html/rfc5322#section-2.2

Header fields are lines beginning with a field name, followed by a
colon (":"), followed by a field body, and terminated by CRLF.

If Sendmail expects something else, the sendmail transporter should be fixed instead.

It turns out that the is an issue with the sendmail version -> close


Add hic-tre.dundee.ac.uk CA cert

Will do a CM for the cert installs in shared services and I'll dm you a link to the docs for our CA cert

Now it is optional 😁


PeerJS doesn't work if peers are using different browsers

Caused by https not allowing getUserMedia. Forgot to enable bypass http for the other browser. lol


Support for capitalised file extensions

We have some image URLs ending in .JPG. This results in Invalid image format: JPG.
It looks like it's coming from https://github.com/DarthSim/imgproxy/blob/master/image_types.h and https://github.com/DarthSim/imgproxy/blob/f5f6b24d4b19a20dffd08205181a67e7b3dde3e5/processing_options.go#L424 .
Would lowercasing the extension in this function be enough to add support for this?
Thanks

Hi,
I'm not a fan of modifying processing options without significant need. Is there a reason why you can't use lowercased extension in processing options? Extension of the source image and resulting one have nothing common.

Hi,
For our particular situation, we have the images hosted on s3, uploaded through strapi (a cms). Since we can't control what a user's browser sends as the filename on upload, we can either modify strapi to lowercase the files before sending to s3, or modify imgproxy to not fail when it receives a request with an uppercase filename.
I'm not sure whether the location in processing_options I've mentioned is the correct place to make a change like that (and I'm not a go developer).
Since as you say "Extension of the source image and resulting one have nothing common." it seemed odd that it would fail processing an image with a .JPG extension and like the fix would be within imgproxy rather than strapi. If you'd prefer not to for whatever reason, we'll find a workaround though.
Thanks!

Ah, yeah. I misunderstood how this was working (I did think it was strange imgproxy was relying on the extension like this).
In this case, we can certainly fix this ourselves.
Thanks for your help.


Add: Las Américas Institute of Technology

Official website: https://itla.edu.do
Academic offer: https://itla.edu.do/carreras
Software engineering associate degree program: https://itla.edu.do/software
Contact info: https://itla.edu.do/contactos/
Email domain for students/teachers proof:

@DarkRoku12 Pull request merged. Thank you!


🤷♂ What quality should we use when storing deliberation video recordings?

There is a tradeoff between the cost of storage and the resolution/fidelity of the recording.

Higher resolutions will be more expensive to store long term.
Lower resolutions will make it harder to pull out facial expressions, gestures, transcribe audio, etc.

I don't know how expensive storing videos will be, nor how beneficial it is to have the best recordings possible.
To figure this out, lets:

[ ] Estimate the cost per month on S3 for storing a minute of video at the full resolution we get from daily.
[ ] Estimate the cost to stream a minute of video to S3 (if there is a separate data ingestion cost?)
[ ] Estimate the monthly cost for 10,000 deliberations with an average of 5 participants, each 15 minutes long, a different video for each participant
[ ] Research the benefit of storing at high resolution for doing audio transcription, gesture and expression analysis.

Talking it over with the lab group as well, we should record the highest quality that Daily streams to us.


can sb please help the UE5.2 project wont open build failed and when i try to build thats the error

[Enter feedback here]

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

ID: 4cb5d86f-c142-b522-aab1-03282ea5e23c
Version Independent ID: 4cb5d86f-c142-b522-aab1-03282ea5e23c
Content: MSB3073: The command 'command' exited with code 'return code'.  - MSBuild
Content Source: docs/msbuild/errors/msb3073.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

### Tasks

Severity	Code	Description	Project	File	Line	Suppression State
Error	MSB3073	The command "S:\UE5_sorcecode\UnrealEngine-5.2\Engine\Build\BatchFiles\Build.bat -Target="bax_studio_V2Editor Win64 Development -Project="S:\Unreal Engine\Projects\bax_studio_V2 source\bax_studio_V2.uproject"" -Target="ShaderCompileWorker Win64 Development -Quiet" -WaitMutex -FromMsBuild" exited with code -532462766.	bax_studio_V2	S:\visual studio\MSBuild\Microsoft\VC\v170\Microsoft.MakeFile.Targets	44

@hosseinfps -- thank you for your feedback.  This repo is solely for documentation issues, and this is implementation-related.  Please consider these resources:

Ask a question or find an answer on the Microsoft Q&A page for Visual Studio.
Check the Visual Studio MSDN Forum.
Stack Overflow
Reddit

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


terraform-providers.kubernetes-alpha: 0.3.3 -> 0.5.0

Motivation for this change
New release!
Things done

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 129666 at a394a96e run on aarch64-linux 1

2 packages built successfully:

terraform-full
terraform-providers.kubernetes-alpha


Clearing labels and titles for edges

Hello @Thomaash,
I had an issue last year about clearing label and titles, with edges in vis-network graph. It was added to DataSet the possibility to delete properties via updateOnly operation, given the symbol DELETE. This works great, the property gets removed and all is well. However, it does not clear out the label, or the title in vis-network, they still remain and are displayed. I pointed to the code section where the magic happens. When after clearing (deleting) e.g. the label property, the said code does nothing with the label in the graph (since the label is now undefined) so it remains visible.
Experimenting with the code, and modify it with the following:
// Only copy label if it's a legal value.
if (isValidLabel(newOptions.label)) {
  parentOptions.label = newOptions.label;
} else if (!isValidLabel(parentOptions.label)) {
  parentOptions.label = undefined;
} else if (newOptions.label === undefined) {
  parentOptions.label = undefined;
}

Do the magic. The label is cleared out. However now two clustering tests are failing:

1 ) properly opens 1-level clusters
2 ) properly opens clustered clusters

As the tests are relying on the text in the label for the test outcome, and the label is now undefined with the code above. It seems that the code is assuming that an undefined value for the property means no change. And that's okay, however, how are we to propagate that a label was cleared to the network when the label was deleted with updateOnly and symbol DELETE with DataSet, so that the label can be cleared with the network too?
Thanks

I've been trying to figure out how to use the DELETE symbol with no luck.
Could you give a minimal jsfiddle (or similar) example.
Is this only possible in a node environment or similar?
Looks as if one needs to import the symbol - which isn't really doable in inline script on a html page (unless I'm wrong)

@jan-johansson-mr
As to your problem - I can now say that it seems to be a general problem (not just related to your labels).
My specific use-case is that I hide and then unhide a specific edge - and then later I want to hide/unhide all edges (via network options).
But that specific edge is now "stuck" on being shown - as in the network the "hidden" property has not been deleted.
So for now it seems the DELETE functionality is somewhat broken.
It correctly removes a property from the dataset object - but it remains in the network object.
The failure mode seems to be that subscribers to the "update" event (this._trigger("update", .......)) themself to not check oldData for the presence of the DELETE symbol.
And this then causes all sorts of side effects....

Just stumbled upon the same problem, when I try to delete the label of an edge, it doesn't actually remove the text.


vllm is not supported on 4 bit quantized gemma2 9b  model i tried to work on it but it gave me this error

ValueError                                Traceback (most recent call last)
 in <cell line: 18>()
16 #llm = LLM(model="Raviksh/gemma-2-9b-unsloth-merged",disable_xformers=True)
17 #llm = LLM(engine_args=engine_args)
---> 18 llm = LLM(model="Raviksh/gemma2_9b_4_bit")
19
20 def evaluate_responses(prompt, response_a, response_b):
20 frames
/usr/local/lib/python3.10/dist-packages/vllm/attention/backends/xformers.py in init(self, num_heads, head_size, scale, num_kv_heads, alibi_slopes, sliding_window, kv_cache_dtype, blocksparse_params, logits_soft_cap)
387                 "XFormers does not support block-sparse attention.")
388         if logits_soft_cap is not None:
--> 389             raise ValueError(
390                 "XFormers does not support attention logits soft capping.")
391         self.num_heads = num_heads
ValueError: XFormers does not support attention logits soft capping.

You will require bitsandbytes to make it work


flag/config: Allow setting resource parameters for the next operations

Currently, we have to specify flags at each command which isn't great for UX of the cli. This change introduces viper based configuration logic which uses following precedence:

flag
env
config

It also introduces a config command to set/unset/show config
Fixes #11
Testing done
$ kubectl get nodes
NAME                                STATUS   ROLES   AGE   VERSION
aks-agentpool-77471288-vmss00000u   Ready    agent   9h    v1.24.9
aks-agentpool-77471288-vmss00000v   Ready    agent   9h    v1.24.9
aks-agentpool-77471288-vmss00000w   Ready    agent   9h    v1.24.9

# env testing
$ kubectl az check-apiserver-connectivity
Error: specify either --node or --id or VMMS instance information (--subscription, --node-resource-group, --vmss and --instance-id)
$ export NODE=aks-agentpool-77471288-vmss00000v
$ kubectl az check-apiserver-connectivity # works fine

# config testing
$ unset NODE
$ kubectl az config set-node --node aks-agentpool-77471288-vmss00000v
$ kubectl az check-apiserver-connectivity # works fine

# precedence testing
$ kubectl az config set-node --node foobar
$ kubectl az config show # verify config
$ kubectl az check-apiserver-connectivity
Error: failed to retrieve Azure resource ID of node foobar from API server: nodes "foobar" not found
$ NODE=aks-agentpool-77471288-vmss00000v  kubectl az check-apiserver-connectivity # works fine
$ NODE=foobar  kubectl az check-apiserver-connectivity --node aks-agentpool-77471288-vmss00000v  # works fine

I think we found a great solution. I added some comments and an error I found after a quick test.

Thanks for the great review. I think we are getting there. We can also introduce a config import command in future that should take all the Kubernetes nodes via kubeconfig and write them in config as:
current-instance: aks-agentpool-77471288-vmss00000x
instances:
    aks-agentpool-77471288-vmss00000x:
        subscription: mySub
        node-resource-group: myNRG
        vmss: myVMSS
        instance-id: myInstanceID
    aks-agentpool-77471288-vmss00000x:
        subscription: mySub
        node-resource-group: myNRG
        vmss: myVMSS
        instance-id: myInstanceID
    aks-agentpool-77471288-vmss00000x:
        subscription: mySub
        node-resource-group: myNRG
        vmss: myVMSS
        instance-id: myInstanceID

Should improve populating the config. What do you think?

Should improve populating the config. What do you think?

Love it!

Should improve populating the config. What do you think?

Love it! I was also thinking that maybe if you use it once with the API server up, we can store the values of the VMSS instance for a future use when the API server may be down

Perfect, Yeah it will work well  in that use-case. I think this PR is ready to go in. I will open a separate PR for handling config import.

Thanks for awesome reviews!


unable to initialize pay by bank

when i select bank transfer method it does not initialize any ui

Please, can you take a look at your log? The plugin should spit out some errors as to why the UI failed to display.

ok will do. many thanks

@ibeninc  Please, can you confirm that this issue has been fixed?


[GEARPUMP-273] Remove java in travis language setting

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the commit message is formatted like:
[GEARPUMP-<Jira issue #>] Meaningful description of pull request
[ ] Make sure tests pass via sbt clean test.
[x] Make sure old documentation affected by the pull request has been updated and new documentation added for new functionality.

R: @huafengw

tests passed on my fork https://travis-ci.org/manuzhang/incubator-gearpump/builds/202219303 while stuck here

+1


NTR: dIMP salvage

Please provide as much information as you can:

Suggested term label:
dIMP salvage

Definition (free text)
Any process which produces a dIMP from derivatives of it, without de novo synthesis.

Reference, in format PMID:#######
PMID:8692979

Parent term(s)
cellular metabolic compound salvage (GO:0043094) AND has_primary_output SOME 2'-deoxyinosine 5'-phosphate(2−) (CHEBI:61194)

Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?

Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)

Cross-references

For enzymes, please provide RHEA and/or EC numbers.

Can also provide MetaCyc, KEGG, Wikipedia, and other links.

Any other information
@hdrabkin see @deustp01 and my notes for more detail

+id: GO:0106385
+name: dIMP salvage


Race condition on multiple threads calling logger_register_message_callback

Describe the bug
If two threads simultaneously call logger_register_message_callback, there is a possible race where one thread will allocate g_user_logger_cb_info, and another thread will attempt to access that global before it is allocated.
Thread 1

Calls logger_register_message_callback
increments g_user_logger_cb_info_ref (count_of_registered_callbacks is 1)
Thread gets context switched away before completing g_user_logger_cb_info = new …

Thread 2

Calls logger_register_message_callback
increments g_user_logger_cb_info_ref (count_of_registered_callbacks is 2)
Crashes dereferencing NULL during g_user_logger_cb_info->callback

To Reproduce
Get very unlucky in timing when starting a multi-threaded process that attempts to initialize logging on multiple threads at the same time.
Expected behavior
App doesn't crash 😊
Logs

Screenshots

Desktop (please complete the following information):

OS with Version: Windows 1903
SDK Version:  1.1.0
Firmware version:  N/A

Additional context

@Brent-A definitely a bug, but why is logger_register_message_callback being called twice from different threads? Did you find this through code inspection? I would like to better understand your usage pattern. The function will fail if you try to use different callback functions.

I'm wrapping K4A in a class that gets activated by an external framework on multiple threads. As the class author, I have no control over how often, and on what threads, I get activated.

Fix here is to spin when multiple registrations are happening at the same time to ensure initialization completes.


Improve quality of data logged to Azure App Insights

Many of our logs are difficult to correlate with support requests and/or other log statements. Add additional information to logs where possible - things like:

can events, exceptions, and traces include organization ID?
can our audit logs be linked to the user sessions (via session ID or via operation ID?) in such a way that we can see things like page views, exceptions, and custom events in the same session as the audit logs?

we do have a session ID on our audit logs already, but we don't always populate it

Notes
The App Insights trackException and trackExceptions methods can take additional custom properties where we can add details we might find useful. TelemetryService.ts I think allows us to log to AI every time frontend code logs to the console, so updating the code there might be a good place to start.
Maybe this should be a spike until we have a clearer picture of what we can and can't do.
There should be some way to intercept all logging and tack information on to that.

@mehansen has this been addressed by #4305?

@mehansen has this been addressed by #4305?

@lg-king we had a team discussion on improving logging across the app overall and this ticket is supposed to address that. I actually call out in #4305 that "this is not meant to address https://github.com/CDCgov/prime-simplereport/issues/4257, which is a larger effort that might even require some spike work first"
I don't know if it was ever very high priority so I think it's safe to close it if we want, or maybe it should be tech debt and go on the eng health board backlog instead


How to DISABLE the printing of zsh PACEHOLDERS like !$

mkdir tmp
cd !$
cd tmp # why printing it while bash or zsh with grml rc just run it like below

cd !$
tmp >

got it
don't use !$ its bash like
use $_ instead


[Feature Request] the width of the recommended tips with autocomplete

Existing Component
是
Component Name
el-autocomplete
Description
带提示的输入框在弹出提示的时候由于提示信息太长不能全部显示，希望能够动态改变提示框的长度,谢谢。
when I use the autocomplete with the recommended tips, the tips are too long to show all of them, I hope the width of the tips box can change automatically, thanks a lot.

https://jsfiddle.net/leopoldthecuber/sphqza5b/


Package icon in search results doesn't match actual icon

<EMAIL_ADDRESS>wrote:
The package has wrong icon in the package list (on the site and in the UI in Visual Studio).
Package icon https://habrastorage.org/files/99f/a59/fa7/99fa59fa760f4762a86fed46302b2d30.png
Icon in the package list https://habrastorage.org/files/c66/481/299/c664812990004f4b9a56edd5b2e81770.png

Seems like the fallback icon is used in the second case, may be an SSL error in the browser?

Saw no SSL errors. There wasn't even an attempt to load the icon from it's URI at https://raw.githubusercontent.com/bem/bem-identity/master/sign/_theme/sign_theme_spots.png
Checked the search results, and IconUrl is not part of the search results for this package. Do we need to rebuild the index?

Try unlist / list cycle. Should update the index.

I executed the 'list/unlist' commands and the package icon is displayed correctly.
Thank you!


[DO NOT MERGE, TESTING ONLY] Release Qualification

Trigger release qualification jobs
none

@chxchx: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-release-qualification.sh
15a23be7fce7cc0081ba3e84464873d22d680f51
link
/test daily-release-qualification

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Client side validations

Hello
I'm using client_side_validations(3.2.0.beta.3) and client_side_validations-formtastic (2.0.0.beta.3) (https://github.com/bcardarella/client_side_validations)
On the form I have nested fields that is added with help of the cocoon gem and validations throw an error 'validators is undefined'
Can you help please?

IMO it is the problem with client-side-validations gem not with the cocoon. If anyone would like to see my workaround it is described here: http://stackoverflow.com/questions/13329248/client-side-validations-on-nested-field-not-generate-valiadtion/29490646#29490646

Thanks @jmarceli. Now I can close the ticket.

I'm not seeing the OP's error but I am seeing that nested fields added by cocoon to the DOM don't validate with the client_side_validations gem. Using the callback as @nathanvda suggested above does seem to work in the most recent versions of the gems:
$('body').on 'cocoon:after-insert', (event, insertion) ->
  insertion.closest('form').validate()

Now when the user submits the form the validations trigger correctly for the cocoon-added elements.


Trying to use environment variables within custom uaa.yml

I was trying to use environment variables within the uaa.yml (e.g. ${DB_URL" within the jdbc url). This did not seem to be working. Does uaa.yml support environment variables?

The UAA does support environment variables, but not as mentioned above. You can't insert an environment variable into the uaa.yml file.
Instead, you can omit the field in the uaa.yml, and the runtime will pick it up directly from the system environment.


[DOC]Add a new argument to Makefile for making docs without computing

Description
As we want to simplify the process of compiling the docs and website manually on a laptop machine without GPU (e.g. MacBook Pro), we want a command that can disable computing python scripts in the docs.
This pull request is a standalone change to the interface of Makefile that supports converting without computing python scripts.
This change is compatible with all the existing CI and Makefile. It accepts existing commands and enables computing on default.
After this change, you can make the docs without computing the python scripts by using the following command:
make docs MD2IPYNB_OPTION =-d

What's more, you can still use the default command make docs to run the python scripts as before.
This change relies on the previous change #1191
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

cc @dmlc/gluon-nlp-team

Job PR-1192/1 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1192/1/index.html

@leezu

Job PR-1192/5 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-1192/5/index.html

@eric-haibin-lin This pr is ready. Thanks!


feat: sort keys of ApiObject manifests

Fixes #17
Using json-stable-stringify to sort output alphabetically and recursively.
Signed-off-by: campionfellin<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Nice work - thanks @campionfellin


refactor(interactive): Refactor StopService API for Interactive SDKs

As titled.

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

Project coverage is 58.78%. Comparing base (a8d0975) to head (dfa8ae9).
Report is 1 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #4271       +/-   ##
===========================================
+ Coverage   34.33%   58.78%   +24.45%
===========================================
  Files         126       12      -114
  Lines       13235      592    -12643
===========================================
- Hits         4544      348     -4196
+ Misses       8691      244     -8447

see 126 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


Will llrt be a WinterCG Runtime?

https://wintercg.org/
Many cloud providers and other groups in the industry are aligning on standards for new non-node based runtimes.
Will llrt attempt to follow specifications created by the WinterCG?
Library authors are starting to need to consider differences between many Javascript runtimes, aligning on standards for these new runtimes will help ensure consistency and compatibility.

The long term goal for LLRT is to become Winter CG compliant. Not every API from node will be supported.
From API.md.

That's the plan!

Wouldn't it be smarter for AWS to support other runtimes as well like Bun or Deno besides Node and LLRT? One thing is that it is very fast to start like LLRT, but also that many of us are doing projects with Bun and Deno and we can no longer use AWS to deploy and you will lose large volume of customers, and Bun is already a thousand laps away from the speed of Node.


Notify teams about community PRs

What does this PR do?
Notifies teams by slack when a new community PR is open.
details.

Motivation

Additional Notes

Here is the message preview (:pr: will be replaced by the emote)

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

I got a rebase problem, do not pay attention to this PR for the moment. Sorry for the notifications 😅

/merge

/merge

/merge


Added the SiteMaps

Related Issue
@Shruti3004 @Ankita297 ... I have added the SiteMap.html and Sitemap XML.

Closes:  #47
Description of the changes I've made
I have added the sitemap to the website so that it will be helpful for the visitors, and the XML file can be useful for SEO purposes.
NA there is no unusual behaviour of the code.
Checklist:
- [x] Added the sitemap.html

- [x] Added the sitemap.xml

- [x] Added the sitemap.css

- [x] Added the Sitemap link in the footer description of pages
Screenshot

Steps to add the sitemap.xml on Google Search Console

Step 1.  Open Google Search Console and login with your google id.
Step 2.  Add your website on Google Search Console
Step 3.  Click on Sitemap, given on the sideNavBar
![image](https://user-images.githubusercontent.com/36589853/110593361-79e99100-81a1-11eb-85fb-11fa5bbe46d7.png)
Step 4.  Add the link of your sitemap on the textbox and submit.
![image](https://user-images.githubusercontent.com/36589853/110593586-c634d100-81a1-11eb-8283-a5c80d6af809.png)

How to create a new sitemap.xml after changing the structure of website?

Step 1.  Open https://www.xml-sitemaps.com/ in order to create a sitemap.xml
Step 2.  Replace the new sitemap.xml with the current one (this file should be the root folder)

How to create a new sitemap.html after changing the structure of the website?

Step 1.  Open https://www.mysitemapgenerator.com/start/free.str.html in order to create sitemap.html
Step 2.  Replace it with current sitemap.html
Step 3.  Link (sitemapstyles.css) on the head section.

Note: You can only generate sitemap.html, if you have a proper URL of the site, you can't add the path to generate the file. For your current project, if you want to add the new link, then you have to add it separately because your site URL has a path to open.

@Shruti3004 @Ankita297 I hope the description will help your further process.

@Shruti3004 @Ankita297 It will be great if you check out my changes soon so that I can work on new issues.

@harikesh409 I have made the changes as per your request. Thank you for your feedback. It will be great to see the fast replies.


Redesign the home page

**Problem : **

The current home page lacks visual appeal , with an unengaging design.
The color scheme inconsistency hampers the overall cohesiveness of the page.
The game cards are also not effective enough.

**Proposed Solution : **
Redesign the home page by enhancing its visual appeal through cohesive color schemes and also making the game cards effective by using relevant pictures.
Screenshots of the current home page

I would like to work on this issue under JWOC

Okay can you give some designs about how you are envisioning the changed look?

Okay I will provide you the same by tomorrow.
On Sun, 21 Jan, 2024, 20:10 Spandan-Bhattacharya, @.***>
wrote:

Okay can you give some design or idea about how you are envisioning the
changed look?
—
Reply to this email directly, view it on GitHub
https://github.com/Spandan-Bhattacharya/Solve_it/issues/79#issuecomment-1902658324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A47BSTSNRZP4GO53M4E2VITYPUSG3AVCNFSM6AAAAABCD2OMVCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBSGY2TQMZSGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

The navbar , footer and game cards below, with a comparable colour palette,
are what I'm considering adding.Let me know your opinion.
On Sun, 21 Jan, 2024, 20:37 Aadhya Sharma, @.***>
wrote:

Okay I will provide you the same by tomorrow.
On Sun, 21 Jan, 2024, 20:10 Spandan-Bhattacharya, <
@.***> wrote:

Okay can you give some design or idea about how you are envisioning the
changed look?
—
Reply to this email directly, view it on GitHub
https://github.com/Spandan-Bhattacharya/Solve_it/issues/79#issuecomment-1902658324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A47BSTSNRZP4GO53M4E2VITYPUSG3AVCNFSM6AAAAABCD2OMVCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBSGY2TQMZSGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


 Not the happiest person with this fix -- it's a little unclear why the #include "util.h" isn't working. Don't have access to xcode currently to find a better fix, though.

Superseded by #2566


Some export categories tests fail on older WordPress version

The following tests are failing on WP 3.6 and 3.8, even though the export categories endpoint seems to be working correctly:

ExportCategoriesTest::test_export_categories_returns_correct_if_page_and_rows_are_given
ExportCategoriesTest::test_export_categories_returns_empty_and_page_and_rows_as_parameters_in_json_if_page_too_high

The error message is "Object of class stdClass could not be converted to int" from wp-includes/taxonomy.php.

Solved by 62a74622ba9f6e8e9e344c5ab42fbe52cf817dc7


[Lifecycle] OSM: Missing logo in adapter chip in /settings

Description
The generic Meshery logo is used here.
Expected Behavior
The OSM logo should be presented instead.
Screenshots/Logs

Would love to take this up!

Excellent. 😄

@flozender Are you working on this?
If not I would love to take this up

Not having heard from @flozender in Slack/the community for a little bit, you might have a go at this @adithyaakrishna.
It's possible that @flozender will circle back to pick this up.  We do need it completed this week.

This issue is open for business (open for a contributor to pick it up).

@igallup, you might be interested here.

I am!  I'd be happy to work on it tomorrow night, if that's not too late.

Perfect. That will work well, @igallup

I'm afraid I'm going to need to hand this one back.  I think I found where the bug is happening, but I can't get Ruby to stop throwing a tantrum about your adapters.md file long enough to see where case "osm" isn't getting passed in.

I'm afraid I'm going to need to hand this one back.  I think I found the source of the bug, but couldn't verify it.  Sorry!

Oh, gotcha. Ok, no sweat. Depending on what issue was cropping up, you might find http://rvm.io of help.

@TeezyOg or @AmjadNoor interested?

I can definitely try my hand at it but I might only be able to get to it on the weekend!

@sestissi16 still working on this?

Oh, sorry. I'm just seeing that this is unassigned. Yes, @adithyaakrishna, if you can knock out this item, that would be great.

I can take a look at it tonight.
On Thu, Apr 1, 2021, 8:44 AM Lee Calcote @.***> wrote:

Oh, sorry. I'm just seeing that this is unassigned. Yes, @adithyaakrishna
https://github.com/adithyaakrishna, if you can knock out this item,
that would be great.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/layer5io/meshery/issues/2558#issuecomment-811881675,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASHBICJXTOIAVOENDWJTK7TTGRTDXANCNFSM4ZHLF4CA
.

@leecalcote
I've fixed this issue and had one doubt, was the text-align: left on purpose?
Because, I fixed their margins to make them not stick together and aligned them on right instead of left
Do lmk, if I can go ahead with this change or else will push the fix for this issue particularly 👀
Screenshot:

Also to add to the above comment, there are two logos missing on the System Dashboard Page also,
Screenshot:

Good eye. The Meshery logo shows up when Meshery server doesn't have an active connection to a given adapter.
I'm not sure that I have the right details with regard to current formatting / alignment. If you have improved the formatting, yes, let's do it.

Okay got it, Will put these changes to the PR too :)


Support ios8

it's perfect ,but i expert it's location can use in ios8 foreground and background

yes,please support iOS8

Just add a new row into the app's plist file:
key: NSLocationWhenInUseUsageDescription
value: text with a description why location services is needed.
For instance: "Location Services is used to calculate the movement of the device."
Finally, in SOLocationManager add:
[self.locationManager requestWhenInUseAuthorization];

right after:
self.locationManager.delegate = self;

in the init method.
iOS 8 needs these adjustments.


When will the Release APIs be supported in the .NET client library

It appears they are in preview, but having those preview APIs in the .NET client library would be helpful. Any timeline for this?
Specifically, I'm looking for two things that will be set off from a 3rd party service:

Start a Release deployment
Approve/deny a Release deployment

I tried to create a Release using the REST API, but I keep getting 404s. See gist.


Some modules might not be available in some DICOM files

Coverage decreased (-0.01%) to 2.489% when pulling 392e020199c6619727b292e070ffe596647d90d4 on campanflaviu:master into 400526a91918afcf88c662edae82e04714b13198 on chafey:master.

Thanks!


BUG (#20) Not Localization's in Map

related to #20
Steps to reproduce

Go to https://greencityclient.herokuapp.com
Click on some places which are tied with Google Maps (e.g. "Готель Львів")

Actual Result

Appears tooltip in Russian Languages

Expected Result

All elements in Map must be translated according to mock up

To display in English you need to use https://www.google.com/maps/place/ %/=en
To display in Ukrainian you need to use https://www.google.com/maps/place/ %/=uk

Please check localization your browser or laptop, somewhere  you use russian localization

Please check localization your browser or laptop, somewhere you use russian localization

I check localization browser and  laptop:

my comp:  browser - English , Win - Russian  https://prnt.sc/paw15o
RD comp:  browser - English , win - Eng  https://prnt.sc/paw2fo

Errors are identical on different computers and different settings.
Accordingly, it does not extend from Environment
BUG in RD comp:  https://prnt.sc/paw5f6
BUG  in  my comp: https://prnt.sc/paw4vc

It depends of custom setups in Google Maps (not in OS)
Go to the settings in your browser. Scroll down to the link Advanced. Choose Language and add Ukrainian to the list (if it was not added). And choose the option View Google Chrom in Ukrainian
https://i.piccy.info/i9/4e8b54a2cdd8bc121de18ec22c7898b1/1580722188/112998/1358686/language.jpg


Milestone 4: Public recipe list [1pt]

[ ] Should display a list of all public recipes ordered by newest as in the wireframe.
[ ] Should lead to recipe details.
[ ] If the user is the owner of the recipe, should allow the user to delete it.

Hi @MargaretTechworld and @2004-okari,
While you made a great effort in this project, I cannot proceed to review your code.
Invalid Code Review Request
Kindly submit the other issue where the previous reviewer left a review.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


[Docs] Missing agentgroups module

Fixed in https://github.com/scottwestover/Unofficial-LiveEngage-API-Wrapper/commit/b31bbffb9a9d3bf5045699a5c45014a94f7dd92b


feat(whale): migrate packages/whale-api-wallet

What this PR does / why we need it:
Migrate whale-api-wallet from whale to jellyfish repo in consolidation efforts of the whale api.
Which issue(s) does this PR fixes?:

Partial #977
Additional comments?:
This PR follows up which is a precursor and dependency of this package:

https://github.com/DeFiCh/jellyfish/pull/1166

Wallet has a direct dependency on whale-api-client so this trails after that.

This PR will be in draft until #1166 is merged to main and will be followed up by:

https://github.com/DeFiCh/jellyfish/pull/1168

Btw your filename wasn't renamed on Git, likely due to your OS settings.

Btw your filename wasn't renamed on Git, likely due to your OS settings.

Thanks for the heads up, will fix.

@fuxingloh should be corrected now


Sorl thumbnail / swift storage

Hi @blacktorn,
did you try implementing this with soul-thumbnail ? It return a 404 error for image.
It return url of the website instead of the swift url .
without sorl image is returned correctly.
Is it possible to fix that ?
Thank you

@masterdubs: It will never works. With Sorl-thumbnails, django generates images at the time of serving, with swift storage the files are not served by django but media server, so the thumbnails must be generated previously and uploaded to media server. These are two different ways of working.
https://sorl-thumbnail.readthedocs.io/en/latest/operation.html


how to download stories only ?

instaloader --latest-stamps --login myname --stories --no-video-thumbnails --no-captions --no-metadata-json --no-compress-json targetID

it still download all post, stories ,highlights of targetID.
so how to download stories only

You can add --no-posts to not download posts.
Highlights shouldn't be downloaded unless you use --highlights; make sure you don't have that argument.


[Contributing] Thoughts on adding a contrib folder for dataclass support?

Hi @bhch , I'm working on a project which requires django-jsonform + dataclasses-jsonschema.
I feel this project could be quite useful to the Django community. So I want to make this functionality public.
Would you be OK if I make PRs to add a contrib/dataclasses folder?
E.g., from django_jsonform.contrib.dataclasses import DataclassJSONField

Could you please describe the use case for DataclassJSONField?

In the Python code, we'd work with dataclasse, rather than Dicts.
In the DB, it gets stored as a JSON.
Basically dataclass with a JSON schema get serialized/deserialized as JSON.

This makes good sense for django-jsonform because when you add a json schema, then you expect structure. Then the Dataclass would provide structure and utility for the Python side.

Pseudo code example:
@dataclass
class Foo:
    x: int

class MyModel:
    foo = DataclassJSONField(Foo)
    foo_list = DataclassJSONField(Foo, many=True)

m = MyModel.objects.first()
assert isinstance(m.foo, Foo)
assert isinstance(m.foo_list, list)
assert isinstance(m.foo_list[0], Foo)

That does sound useful. Alright, I'm open to include this in the library. Kindly open a PR. Thank you.

I am @ghost


Exploring CUDA Acceleration: Testing HYPRE and MFEM Integration with CUDA 12.2 on GTX 3080 Ti

Hello everyone,
I have compiled HYPRE and MFEM with CUDA (CUDA version 12.2; GTX 3080 Ti).
I would like to test this CUDA-supported version. Do we have any examples to test it?
Best regards,
Sungho

Example 1 (and its parallel version) support CUDA, you can try out some of the sample runs.
https://github.com/mfem/mfem/blob/541f10f9b44fd52bacde793a08dc19e53a8972ec/examples/ex1p.cpp#L31-L33
Many of the other examples support CUDA as well, look for the --device flag or consult the list here

@pazner Thank you so much!


Download a video with 0 bytes

I was trying the examples, and doesen't download the video in a correct way
videoID := "BaW_jenozKc"
	client := youtube.Client{}

	video, err := client.GetVideo(videoID)
	if err != nil {
		panic(err)
	}

	formats := video.Formats.WithAudioChannels() // only get videos with audio
	stream, _, err := client.GetStream(video, &formats[0])
	if err != nil {
		panic(err)
	}

	file, err := os.Create("video.mp4")
	if err != nil {
		panic(err)
	}
	defer file.Close()

	_, err = io.Copy(file, stream)
	if err != nil {
		panic(err)
	}

output:
-rw-rw-r-- 1 pai pai    0 nov 29 22:54 video.mp4

I can fixed changing formats[0] to formats[2]

this is about the video`s quality.
I changing formats[0] to formats[1] it works, formats[2] is only audio.

What is the version of library you are using?


conv{bin,img,font}: init at {3.0,8.3,1.0}

Motivation for this change
Package conv* utilities to eventually package the CE C Toolchain
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 102017 run on x86_64-darwin 1

3 packages failed to build:

convbin
convfont
convimg

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

3 packages built:

convbin
convfont
convimg

gcc: command not found

gcc is hardcoded in the makefiles...

@SuperSandro2000 now it should work on darwin

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

3 packages built:

convbin
convfont
convimg

Result of nixpkgs-review pr 102017 run on x86_64-darwin 1

1 package failed to build:

convimg

2 packages built:

convbin
convfont

Result of nixpkgs-review pr 102017 run on x86_64-darwin 1

1 package failed to build:

convimg

2 packages built:

convbin
convfont

@luc65r I would be fine with disabling it on darwin.


DRILL-6498: Support for EMIT outcome in ExternalSortBatch

@parthchandra / @Ben-Zvi - please help to review.

@Ben-Zvi, @parthchandra any chance of reviewing this soon?

@Ben-Zvi / @parthchandra  - I have written a description in the JIRA to give idea about current implementation of Sort, motivation behind current approach for supporting EMIT outcome and what future enhancements can be done. Please let me know if it looks good.


XmlText Constructor not recognised?

Trying to follow the most basic code from the docs and what's wrong with this code that's it's producing error?
import 'package:xml/xml.dart' as xml;

void main(){
var bookshelfXml = '''<?xml version="1.0"?>
    <bookshelf>
      <book>
        <title lang="english">Growing a Language</title>
        <price>29.99</price>
      </book>
      <book>
        <title lang="english">Learning XML</title>
        <price>39.95</price>
      </book>
      <price>132.00</price>
    </bookshelf>''';
var document = xml.parse(bookshelfXml);

print(document.toString());
print('-------------------');
var textual = document.descendants
    .where((node) => node is XmlText && !node.text.trim().isEmpty)
    .join('\n');
print(textual);

//the following works fine.
// var titles = document.findAllElements('title');
// titles
//     .map((node) => node.text)
//     .forEach(print);
}

the error produced is;
Unhandled exception:
'file:///home/xxxxx/dart/xmlXSLT/bookxml.dart': malformed type: line 21 pos 30: type 'XmlText' is not loaded
    .where((node) => node is XmlText && !node.text.trim().isEmpty)
                             ^

#0      _TypeError._throwNew (dart:core-patch/errors_patch.dart:65)
#1      main.<anonymous closure> (file:///home/iveagh/dart/xmlXSLT/bookxml.dart:21:30)
#2      WhereIterator.moveNext (dart:_internal/iterable.dart:439)
#3      Iterable.join (dart:core/iterable.dart:303)
#4      main (file:///home/iveagh/dart/xmlXSLT/bookxml.dart:22:6)
#5      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:261)
#6      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)

Oups, that's a bug in the docs. The package is imported as xml, so you need to prefix all references with xml., i.e. xml.XmlText. I'll update the docs in a second.

wow that was quick!
thanks so much, renggli

Fixed in 3b3e355d71d83b61d5e38b57dd7303ba05bbd165.
Please let me know if you encounter other problems!

Hi renggli,
no other probs yet but if i wanted to ask you simple questions about Dart, Xml and Xslt and using and Xml database where would be the best place to do that?
would it be stackoverflow, dartlang.slack (or perhaps some other slack) or just gmail||G+ you?
if you don't have time that's ok.....

Preferably ask on StackOverflow, I think you can reach the widest community there.

thanks, renggli. later....


Composability via component nesting

I was working on a class-based library for hyperHTML but this project beat me to the punch, so I figure I may drop my work and just share notes with you.
The main difficulty I've encountered in experimenting with hyperHTML is composability, which is the entire purpose of the component based approach.
There needs to be an easy way to declare nested components. It's trivial if the nested components are already instantiated; you'd be able to do the following in the parent's render function:
function render() {
    return this.html`

            ${childComponentInstance.render()}

    `
}

The above is only possible if the child element is already declared and instantiated. In React, this is not an issue because child elements are automatically identified and rendered via React.createElement (JSX transpiles to createElement), which instantiates and then tracks the child elements in its VDOM tree upon rendering. But if there's no VDOM and no createElement, then something is certainly missing!
Proposal:
I think the leanest approach here is a hypercomponent.prototype.child function that wraps the render function provided by hyperHTML.wire and augments it in the following way:

Instantiates any nested classes and places them in parent.children under a specific key
Transparently returns the render output of a child component

Hypothetical example of what this would look like:
/* Parent class
...
...
*/
function render() {
    return this.html`

            ${this.child(ChildClass, props)}

    `
}

hypercomponent.prototype.child aka this.child would basically create an instance of type ChildClass and return its render output directly in the parent's render template.
Starting from the first render pass, this.children would contain references to any nested components.
Does this make sense?

@nodefish thanks so much for the info in this post. It's great to see a few of us are coming to the same conclusions here, and again I really appreciate you sharing your experience.
As luck would have it, I've been thinking on this same problem recently also, and basically have the same as you've proposed unstaged locally.
As I've been toying with the notion of taking inspiration from CustomElements, where you have HyperComponent.prototype.child I've arrived at HyperComponent.prototype.slot.
The way I envisage this working is effectively this:
HyperComponent.prototype.slot(Component, name?) returns a ComponentProxy that's effectively a factory function, exposing an interface to both constructor(...args) and the render(...args) functions. eg:
class Component extends HyperComponent {
  render () {
    return this.html`
${[
      this.slot(ChildComponent),
      this.slot(ChildComponent, 'named-slot'),
      this.slot(ChildComponent, 'call-render').render(...args)
      this.slot(ChildComponent, 'call-constructor')(...args),
      this.slot(ChildComponent, 'call-constructor-and-render')(...args).render(...args)
    ]}
`
  }
}

The nice thing about an interface like this, is we'll be able to abstract caching component instances with basically a WeakMap, similar to how hyperHTML.wire works. 👍
Would love to hear you thoughts on the above. cc/ @WebReflection

Sounds promising. One of my concerns (in general, not specific to your approach) is adding overhead to nesting child elements in the parent template because we should be able to have thousands of child elements (e.g. in a dynamic list), but your approach seems lean enough.
What's the rationale behind exposing the nested component's constructor? Perhaps standardizing the constructor is the way to go, in which case we don't need to expose it, i.e. it just needs props. I'm drawing heavily from React here, where React.createElement has the signature (ClassName, props, children). I think JSX is the perfect amount of verbosity, and the user works as if the custom component is an HTML tag, meaning just ClassName and props (HTML attributes) are specified and the rest is taken care of automatically.
Another random thing that comes to mind which may or may not be helpful: in other frameworks, the slot or yield keyword is for transclusion, as in, defining dynamic child content within a child tag, from the parent scope. Here's how I view the terminology:
Nested tag
function render() {
    return this.html`

            ${this.child(ChildClass, props, null)}

    `
}

Nested tag with transclusion
function render() {
    return this.html`

            ${this.child(ChildClass, props, [
                                                            '
',
                                                            this.child(AnotherChildClass, OtherProps, null),
                                                            '

                                                        ])}

    `
}

Essentially, this is a rip off of React internals and the React.createElement signature, where the 3rd parameter allows the equivalent of other frameworks' yield or slot.

That's a really good point which I hadn't considered fully slot/yield != children.
I think in order to adopt a createElement style interface, I'd expect the children argument to mirror types as supported by hyperHTML: https://github.com/WebReflection/hyperHTML/blob/master/DEEPDIVE.md#good
Which given your above examples could effectively result in the following API:
class Parent extends HyperComponent {
  function render () {
    return this.html`
${
      this.child(Child, { id: 'child1' }, this.wire()`
${[
        this.child(Child, { id: 'subchild1' }, this.wire()`<div>woah!</div>`),
        this.child(Child, { id: 'subchild2' }, this.wire()`<div>dude!</div>`)
      ]}
`)
    }
`
  }
}

class Child extends HyperComponent {
   function render () {
    return this.html`
${
      this.props.children
    }
`
  }
}

The challenge with the above is to manage calls to this.wire() (which just proxies hyperHTML.wire) as we really want to ensure we're reusing wires with every subsequent render.

I still have a couple of blind spots in my understanding of wires, but this looks pretty good to me! I have a suspicion some quirks are going to appear at some point during actual usage, but nonetheless this looks like the right direction.

@nodefish ah-ha! I just realised managing wires with this.wire is actually dead easy - thanks to the following: https://github.com/WebReflection/hyperHTML/#new-in-011.
Because wires can now be bound to the same object via an id we can simply have this.wire(this, ':child') and it will be automagically be managed internally by hyperHTML. Is there anything @WebReflection hasn't thought of here? ;)

Awesome!
One thing to note is that :type notation for wire seems to work on html tags that are part of the final output e.g li or div but our child item would not have a <child> tag in the markup, as it would be replaced by its render output.
I wasn't sure what was the best thing to be doing with the wires here. They're essentially persistent document fragments, and there are a lot of options for what to do with them. I was thinking since we want this.child to instantiate the child element upon the first render call, and add it to a keyed list that the parent can track (parent.children), we have different things that can be done, that are not mutually exclusive:

Encapsulate the whole wire functionality in each component instance so when we call this.child, (childInstance).render is called and it would transparently render to its own wire contained in the instance

Have a wire containing multiple wires in order to handle lists (no idea if this works, the hyperHTML docs are sparse here, probably needs some experiments)

Have the 3rd argument of this.child automatically call wire on the array elements

Really happy to share ideas on this, as this is quite challenging to navigate but the possibilities are pretty exciting.

Hrmmm, I'm not entirely sure the type argument in wire(obj, type) has to be anything at all related to the final output. Looking over the code I suspect it's more of a convention to have :li for a wire managing a list items than anything else. Happy to be told otherwise /ping @WebReflection
@nodefish did you happen to see the latest in #7, I've just pushed a working draft of what we've discussed above. Would love feedback if you're able.

I just gave my feedback on the draft. It's looking good and I can't think of much to say. It's pretty much what I think it should be at this stage. Need to test it to be able to say more. Is it stable enough to give it a try?

type can be 'html' or 'svg' or 'async' , by default is 'html'. Type can contain any ':id' which is used to weakly relate a wired content to the same context. 'html:any-id' is the same as ':any-id', the colon is there just to delimit type from I'd.
Type is needed mostly to figure out how the wire will be used: some HTML content, some svg which require different handling, some async content which will be resolved at distance and it will figure out if where it's resolved should be HTML or svg.

Understood. I've read the hyperHTML docs but I need to get my hands dirty to truly understand it, as it's a lot to take in (and the author's style is a bit erratic :) )

I just need time to write a better documentation. Right now the situation is too poor

No worries, it's a huge amount of stuff and I'm surprised you did it all on your own in the first place.

I'm going to close this one off, as the API discussed here made it's way into v3 via component(Component, props, children). Feel free discuss further if required, or open another issue entirely.


changed hover color of category-info in dark theme

Description

Category

[ ] Resource Addition
[ ] Documentation
[ ] Codebase
[ ] User Interface
[ ] Feature Addition

Related Issue

Fixes #XX
Checklist

[ ] I have followed the CONTRIBUTING guide
[ ] The name of the resource is spelled correctly (if applicable)
[ ] The link to the resource is working (if applicable)
[ ] The resource is added in the correct format (if applicable)
[ ] I have tested changes on my local computer (if applicable)

initially on hover the category-info turned white which not good when the surrounding colors are dark, it defeats the purpose of the dark theme which requires minimal bright colors to work in dark lighting

Valid point, @Greyash-Dave.
After checking the card hover state with some background colors (including yours) and without any background color, I've decided to not go with any color, i.e., keeping it 'black' as it looks better for me that all the colors that I've tried.
Plus, the cards are getting lifted on hover, which adds interactivity.
So, let's change the codebase to keep the background color 'black' on hover.

The 'dark blue' buttons on hover look better when the card background is 'white'. However, the 'white' buttons (like the 'Get Started' button) look better when the card background is 'black'.
So, let's change the color of the buttons to 'white' on hover.


HHH-13303: fix some LGTM alerts

Fix some of the alerts raised by LGTM.

Array index out of bounds in TupleBuilderTransformer and JoinedSubclassEntityPersister
Potential output resource leak in CacheableFileXmlSource
Useless comparison test in AbstractEntityPersister, FromElement, AbstractHANADialect
Useless null check in AbstractSchemaMigrator, GroupedSchemaMigratorImpl (2 instances), IndividuallySchemaMigratorImpl (2 instances)

The alerts raised by LGTM are here: https://lgtm.com/projects/g/hibernate/hibernate-orm/alerts/?mode=tree

@gbadner thanks for reviewing. I have addressed your comments and pushed the change.

@1m2c3t4, thanks for the update.
I've been seeing the same failures without your fix as shown here. I'd like to get that issue fixed so we can get a clean build.

@gbadner I have rebased

@gbadner Is there any reason to not include this?

many thanks @1m2c3t4 ! And sorry for the delay, it's now rebased and merged.

Thanks for reviewing and merging, @gbadner and @Sanne


remove Hmisc dependency please

Hi @ShozenD
It can be tricky to install Hmisc as installation runs into OMP support issues. I believe we only use binconf. Could you replace all instances with prop.test from the basic stats package?
https://stackoverflow.com/questions/21719578/confidence-interval-for-binomial-data-in-r
Huge thanks!!
olli

Ohhh, I found the explanation for the OMP issues,
https://stackoverflow.com/questions/53014306/error-15-initializing-libiomp5-dylib-but-found-libiomp5-dylib-already-initial
The Intel MKL functions (e.g. FFT, LAPACK, BLAS) are threaded with the OpenMP technology.
But on macOS you do not need MKL, because the Accelerate Framework comes with its own optimization algorithms and already uses OpenMP. That is the reason for the error message: OMP Error #15: ...
Workaround:
You should install all packages without MKL support:
conda install nomkl


[11.x] Dynamic Model Attribute Mutators

This PR adds the ability to dynamically register model attribute mutators without having to declare them as methods.
See #50759 for a scenario where this would be useful.
Ideally, on the boot method of a model, or an external trait, these mutators could be added:
class User extends Model
{
    public static function boot()
    {
        /** @var array<string,Attribute> */
        $attributes = [
            'example' => Attribute::make(...),
            /* ... some dynamic source ... */
        ];

        foreach ($attributes as $name => $attribute) {
            static::addDynamicAttribute($name, $attribute);
        }
    }
}

// echo $user->example;
// $user->example = 5;

The failing check (tests / PHP 8.2 - PHPUnit 10.5 - prefer-stable) appears unrelated:

@driesvints, per your comment on #50760, I have created a new PR with passing tests.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

Seems for now, we aren't going to pursue to further. Sorry @tylernathanreed.


File storage quota

It may be implemented in this repo but modifying 'Impress/lib/impress.files.js' will be better

Outdated


RTL Direction for all products

Summary:
Hebrew, Farsi and Arabic are written in the opposite direction, we need to adjust the entire UI experience (Not only the mobile app but also the website) to support RTL directionality.
Steps to reproduce:
Switch to Hebrew, Farsi or Arabic.
Expected behavior:
When Hebrew, Farsi or Arabic is selected the app should flip horizontally.
As an example (Firefox 13 for iOS):
Hebrew - https://wopr.norad.org/~fxios/screenshots/fxios/v13/he/he/
English - https://wopr.norad.org/~fxios/screenshots/fxios/v13/en-US/en-US/
Hebrew:

English:

Observed behavior:
The text was changed to Hebrew but the direction remained the same making the sentences seem incomprehensible or misleading.
Device and Android version:
Irrelevant.
Screenshots:
(Added example screenshot of desired solution above).

For the translations you can use Crowdin  to correct them. @yarons

Lol, translation is not the issue, I'm the only Hebrew translator anyway.


Screen jumps up on Android when hiding status bar

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
When opening an image in a full screen modal, we are hiding the status bar using the status bar plugin as follows:
if (window.StatusBar) {
                        if (window.StatusBar.isVisible) {
                            window.StatusBar.hide();
                        }
                    }

On Android, the whole screen jumps up shortly to fill the new available space as the status bar isn't there anymore before it is resized to the new screen dimensions without the status bar.
This actually happens when not opening any modal window. Just hiding the status causes the screen to jump.
What behaviour are you expecting?
There should be no jumping of the screen.
Steps to reproduce:

In any Ionic app using the status bar plugin, hide the status bar and you will see the issue described

if (window.StatusBar) {
                        if (window.StatusBar.isVisible) {
                            window.StatusBar.hide();
                        }
                    }

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
I'made a video of this issue occuring on a Samsung Galaxy S6 which can be downloaded from here: https://dl.dropboxusercontent.com/u/22622235/android_status_bar.mp4
The app is using Crosswalk, animatable preference set to false.
Which Ionic Version? 1.x or 2.x
1.2.4
Run ionic info from terminal/cmd prompt: (paste output below)
$ ionic info
Your system information:
Cordova CLI: 5.4.1
Gulp version:  CLI version 3.9.0
Gulp local:   Local version 3.9.0
Ionic Version: 1.2.4
Ionic CLI Version: 1.7.14
Ionic App Lib Version: 0.7.0
ios-deploy version: 1.8.4
ios-sim version: 5.0.6
OS: Mac OS X El Capitan
Node Version: v0.12.7
Xcode version: Xcode 7.2.1 Build version 7C1002

I have the same problem. Even I am using a different method to hide the status bar. I add this line to config.xml:
<preference name="Fullscreen" value="true" />
And screen jumps up to go full screen.

Try using LAYOUT flags available through either window manager or system ui visibility flags, to avoid such jumping or resizing.

This issue was moved to driftyco/ionic-v1#161


Prevent tooltip flickering

Steps to reproduce the issue

Open the demo
Hover over any of the three <button> elements.
While doing so, move your cursor toward the arrow sticking out from the tooltip. Here's a photo illustrating where to hover:

Results
The tooltip will get confused and flicker quickly. This strobe-like effect could cause seizures and violates the Operable tenet of WCAG 2.0.
Expected results
The tooltip should not flash.
Potential (not ideal) fix
Making .simpletooltip have a negative z-index prevents the flicker in your demo. I changed line 172 of your CSS file to z-index: -10.
This z-index change is not ideal because if you have multiple tooltips right after one-another (for example: a row of small icons), the tooltips won't show in front of the adjacent elements.
The root of this problem could likely be in the javascript.
Environment

OSX
Tested in latest versions of Chrome, Firefox, and Opera.

PS: Thanks for making this!

By ading pointer-event: none; on the ::before pseudo-element of  the <span class="simpletooltip js-simple-tooltip"> it work.
I can send a PR if you're ok with this solution.

@yoanmalie hey, this is a good idea and seems to work from my side. 👍
@adamjohnson do you confirm that it fixes the problm on your side?

Fine, I've merged the PR. Other comment on Twitter told me that the flickering comes from the style for the triangle, it might be useful to create it with a transform (and not using border as I did)
Thanks to all ;)

Looking good folks! I added this to my project.
Something of note: the tooltip will still flicker in the demo if you shrink your window to a mobile size where the text in the button is on two lines. This can be alleviated by adding display: block; to .simpletooltip_container, .minimalist-simpletooltip_container, .minimalist-left-simpletooltip_container. Not sure if adding this is wise or not, but I thought I would mention it for future visitors.

Thanks for the quick response. I hadn't used pointer-events before, so that's educational.


feat(agents-api): Add label lookup for inputs and outputs

closes #791

[!IMPORTANT]
Add label lookup for inputs and outputs in agents API by updating StepContext and workflow execution methods.

Behavior:

Add labels attribute to StepContext in tasks.py for label lookup in inputs and outputs.
Update prepare_for_step in tasks.py to merge labels with inputs and outputs.

Workflow Execution:

Modify run method in __init__.py to handle previous_labels for task execution.
Update continue_as_child in helpers.py to accept previous_labels parameter.

This description was created by  for 9a3cfd07f5668c743561f61c5f0fccd397cd80ac. It will automatically update as commits are pushed.

Stale since PR #959


[WIP] torch.compile applied to ray sampling

This PR applies torch.compile to pixel sampling and ray generation.
We could theoretically see some speedup.
NOTE: I haven't tested all configurations.

On a simple scene without camera distortions, I am getting 166 train rays/s with torch compile vs 160 rays/s without -> roughly 4% speedup overall.

Note: this is currently not ready for merging for the following reason:
When typeguard is enabled, the torch.compile fails for some reason (str has no _module attribute). This has to be resolved first.


fix: move type libs to devDeps

This PR moves the type libraries to devDependencies, as they are only needed for development and not for production.

heh thanks :muscle: this is superseded by #69 now, though