Crash on startup

I've tried different Forge versions and VS2 versions, but my game seems to crash whenever Hexxy Skies is enabled. It generates no crash log so I've left the console log from the Prism Launcher. I did not have the same issue on the Fabric version of Hexxy Skies.
https://pastebin.com/tpGUKt9x
Modlist

found the issue
I was accidentally exporting my EntityClipVSKt under the wrong path

fixed in dev!


feat: add filter support to listUsers

What kind of change does this PR introduce?
There is already support for a filter=xyz query param for the listUsers endpoint, but it is not exposed on the client. This PR adds support for the filter, which allows users to filter the respond of listUsers by a partial or full email address.
This PR also fixes pagination metadata for page values >9. This was originally proposed in #793, but changes were requested and that PR is now stale so I have implemented those changes here too.
What is the current behavior?
It is not possible to filter users via the SDK.
What is the new behavior?
You can now filter users.
Additional context
https://github.com/supabase/gotrue/issues/880

@kangmingtay any change you could review this?
I came from https://github.com/supabase/auth/pull/1330#issuecomment-1848007008, but discovered there is no way to actually use this endpoint from the SDK

hi @JKFSOM, thanks for taking the time to contribute but i don't think we'll be adding this endpoint in - the GET /admin/users endpoint actually allows you to filter for a user's email address already:
for example:
curl -X GET "http://localhost:9999/admin/users?filter=<user_email_address>" -H "Authorization: Bearer ADMIN_JWT"


Baselines does not work for algorithmic env with mlp network

executed python -m baselines.run --alg=ppo2 --env=Copy-v0 --num_timesteps=35e6 --activation=tf.nn.selu --gamma=0.99 --num_env=8 --save_path=~/models/test_copy
got the following error:
Training ppo2 on algorithmic:Copy-v0 with arguments
{'activation': <function selu at 0x0000000006D4F8C8>, 'gamma': 0.99, 'network': 'mlp'}
Traceback (most recent call last):
File "C:\Users\adomingues\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "C:\Users\adomingues\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 85, in _run_code
exec(code, run_globals)
File "C:\Users\adomingues\Desktop\ML\OpenAI\baselines\baselines\run.py", line 243, in
main(sys.argv)
File "C:\Users\adomingues\Desktop\ML\OpenAI\baselines\baselines\run.py", line 210, in main
model, env = train(args, extra_args)
File "C:\Users\adomingues\Desktop\ML\OpenAI\baselines\baselines\run.py", line 84, in train
**alg_kwargs
File "C:\Users\adomingues\Desktop\ML\OpenAI\baselines\baselines\ppo2\ppo2.py", line 108, in learn
max_grad_norm=max_grad_norm)
File "C:\Users\adomingues\Desktop\ML\OpenAI\baselines\baselines\ppo2\model.py", line 34, in init
act_model = policy(nbatch_act, 1, sess)
File "C:\Users\adomingues\Desktop\ML\OpenAI\baselines\baselines\common\policies.py", line 175, in policy_fn
**extra_tensors
File "C:\Users\adomingues\Desktop\ML\OpenAI\baselines\baselines\common\policies.py", line 47, in init
self.pdtype = make_pdtype(env.action_space)
File "C:\Users\adomingues\Desktop\ML\OpenAI\baselines\baselines\common\distributions.py", line 289, in make_pdtype
raise NotImplementedError
NotImplementedError

Hi @axeldomingues! I confirm the issue; this happens because "Copy" environment has tuple action space that is not currently supported by baselines.

On the high level what needs to be implemented is subclasses of PdType and Pd classes (in baselines/common/distributions.py) to describe probability distribution over the tuple space, and methods such as mode, neglogp, kl, entropy.  The simplest approach could be to assume independent probability distributions for each component of the tuple. This way all methods can be expressed through methods of distributions that correspond to sub-spaces. For instance, if your space is Tuple(Discrete(5), Discrete(5)) then corresponding distribution (say, TuplePd) will wrap two CatergoricalPd's. Then mode() will return (first.mode(),  second.mode()), and entropy() will return first.entropy() + second.entropy().
Does that make sense?

Hi @pzhokhov ,
Thanks for your reply. It makes sense. Will try to work this around and see.
Thank you!


shotwell: add glib-networking

Motivation for this change
Without this shotwell is unable to export pictures to Google Photos.
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.

We should probably backport too.


fixes for main

A few small fixes:

When dump json with model run, use indent=4 to print multi-line results for reading
When throw out error, add block_number information. Good for debugging Contract
Add a local in-memory cache for contract.meta. When creating the same token for different swap protocols during querying token.prices, it's slow and hitting DB each time if without cache,
Optimize Token._load(), move each property (i.e. symbol, decimals, etc) retrieval code to each property. It may take up to 1.x sec from usual a few micro-seconds. When doing historical query, this could slow things down.
Add alternative web3 event filtering for some node server.
Fix for loading instance()/functions, they only return non-None.
To allow fungible token query with address, to solve the current token loading issues, mostly for upgradeable tokens.
Various fixes for linting and pylance pdate Address class
Passed tests against all current models.

@leafyoung You don't need to update this PR. We'll copy over some of the features into separate PRs.

json formatting and error message detail for exceptions during model run are added in framework version 0.8.3.

Close this PR as parts of it has or will be merged to main.


ProjectSettings.save() does nothing

Godot version:

3.2.3.stable
OS/device including version:

Windows 10 Home 1909
Issue description:

I wanted to save the ProjectSettings which I configured earlier with ProjectSettings.set_setting("application/run/main_scene", "res://title_screen/title_screen.tscn"), which works because when I do print(ProjectSettings.get_setting("application/run/main_scene"), I get "res://title_screen/title_screen.tscn" printed, with ProjectSettings.save(). I even disabled safe save in EditorSettings and my antivirus. For days I ask myself where the problem is.
Steps to reproduce:
in script: change any ProjectSetting and then save it.
Minimal reproduction project:

Bug_REPRODUCTION.zip

Seems like you can't modify ProjectSettings from within running project. It works when you add tool to your script. Probably should be documented.

But why does the method exist then? To me it seems like a bug.

After disconnecting from my Android phone I got
E 0:10:43.140   _save_settings_text: Couldn't save project.godot - project.godot.
  <C++ Error>   Condition "err != OK" is true. Returned: err
  <C++ Source>  core/project_settings.cpp:755 @ _save_settings_text()
  <Stack Trace> tutorial.gd:36 @ next()
                tutorial.gd:29 @ _input()

. I don't know if this helps.

For me, it does not work with tool.

This was fixed by #45880

For me, it does not work with tool.

You need to call it from the editor. It won't work in the running project.

This was fixed by #45880

It's unrelated.

Sorry, I forgot to show my findings. :sweat_smile:
It looks like ProjectSettings.save() works; the issue is that, as soon as the editor is re-focused, it saves the project settings, thus overriding any modifications done externally

Ah you are right, although the PR that fixed this is #45968
I wonder if it works in exported project though.

That should be documented!

No need to document something that's already fixed @MaaaxiKing

Ok so it works in exported project too, but the modified settings can't be loaded. This might be worth mentioning in the doc.

This bug needs to be merged into v3.x aswell.
My inital plugin setting are not saved ;)


Question: CUDNNv4 SpatialBatchNorm momentum

I noticed that cudnn.h includes this rather cryptic comment for the cudnnBatchNormalizationForwardTraining() function:
/* MUST use factor=1 in the very first call of a complete training cycle.
    Use a factor=1/(1+n) at N-th call to the function to get
    Cumulative Moving Average (CMA) behavior
    CMA[n] = (x[1]+...+x[n])/n
    Since CMA[n+1] = (n*CMA[n]+x[n+1])/(n+1) =
    ((n+1)*CMA[n]-CMA[n])/(n+1) + x[n+1]/(n+1) =
    CMA[n]*(1-1/(n+1)) + x[n+1]*1/(n+1) */
double                              exponentialAverageFactor,

In the current version of the code, this is just the momentum parameter which defaults to 0.1. Is it worth implementing the proper averaging behavior? The advantage is that this would allow us to capture the mean and variance across the entire training set.

It's not worth it. The proper averaging behavior on training set has to be computed freezing training, and takes way too long, and there's no accuracy gain for doing so


fix: support module resolution nodenext so type module works in host

Since we are using ESM build so when compiling to js using tsc the imports don't have .js extension due to which it fails to resolve in host and throws error (The below error is when integrating in @excalidraw/excalidraw docs)
Module not found: Error: Can't resolve './graphToExcalidraw' in '/excalidraw/dev-docs/node_modules/@excalidraw/mermaid-to-excalidraw/dist'
Did you mean 'graphToExcalidraw.js'?
BREAKING CHANGE: The request './graphToExcalidraw' failed to resolve only because it was resolved as fully specified

Hence I am upgrading typescript version and using nodeNext as module resolution which makes sure all imports have .js extension as that's recommended by typescript team - https://github.com/microsoft/TypeScript/issues/42151 and https://github.com/microsoft/TypeScript/issues/16577
To test install the preview version
yarn add @excalidraw/mermaid-to-excalidraw@preview

The fact TS vehemently rejects compiling ./file (or ./file.ts) -> ./file.js and instead forces us to write .js when importing TS files is nuts.


fix FC22 buildbot (hopefully)

http://buildbot.shogun-toolbox.org/builders/FC22 - libshogun/builds/150/steps/test/logs/stdio
Avoid using REGEXP for untangling filenames in meta examples, but rather leave this to cmake. Hoping this fixes the problem on buildbot

lets see if it worked :)


[api] updating publish_check_result to set client signature when present

Description
When events are deleted via the API, the publish_check_result helper is used to issue check result which resets the event's status to 0. Currently this mechanism does not include  the client signature when one is set. As a result, the API will return a 202 response code, as if the event were successfully deleted, when in reality the event is not deleted because the check result lacks the required signature.
This change updates the publish_check_result helper used by the API when deleting events to ensure that the published check result includes the client signature, provided that a signature for that client is present in the client registry.
Related Issue
Closes #1435
Motivation and Context
Failure to delete an event under the circumstances described above constitutes a bug.
How Has This Been Tested?
Added unit test to validate that events are actually removed from redis.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

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

Thank you very much!


feat(elixir): add influxdb telemetry push and system telemetry polling to hub

Proposed Changes

Add TelemetryInfluxDB and :telemetry_poller packages
Configure TelemetryInfluxDB as a child of Ockam.Hub
add config section for :telemetry_influxdb
add config section to releases.exs so it expects environment variables at runtime.

Environment Variables
INFLUXDB_HOST
INFLUXDB_PORT
INFLUXDB_BUCKET
INFLUXDB_ORG
INFLUXDB_TOKEN

In a follow-up PR I will  add metric recording for TCP send and receive, and flattening of metrics so we can get everything we want into influxdb in the correct format.

merged 55a9a861da05906828069402a4eca47ee6051aed and 73d8cdc9a3c357f5e83b154ccec2fc177435230f


I want to add a mail to link at the footer section on email address.

Description
I want to add mail to link on the email address so the user can directly get an option for dropping an email. This will make them easy to contact with us. And I want to make the footer more interactive
Screenshots

Any additional information?
No response
What browser are you seeing the problem on?
Chrome
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@Kratika-Solanki assigned

im interested please assign me, ill share my proposed solution as soon as u ping me


Observers can use crafting tables

@iEli2tyree011 Isn't fixed

@TiTo-418 it's fixed for me, maybe you were using an older version?

@TiTo-418 or @twizmwazin can close this, fixed in #845


[RFC] Soft real-time items

Need something like this to keep up with newer frameworks and web application development trends. The technologies below all allow the client to receive unsolicited updates from the server, while WebSockets allow the client to also send data back to the server on the same connection.
Technologies involved

Server-sent Events
WebSockets
Long polling (Comet)

Items needed

[ ] Sugar.Events
[ ] Integration with Plug
[ ] Sugar.Router.socket/4
[ ] Sugar.Controller.socket/2
[ ] Modify controller plugs to allow socket handlers to be called

Notification/event layer
This would allow notifications/events to be published by producers (data layer, api endpoints, etc.) and subscribed by consumers (clients). Would most likely be built upon GenEvent or something similar.
Interaction layer
This would interface the notification layer with the clients by means of the above technologies, e.g. WebSockets. A behaviour would most likely be created to allow for the differing technologies to talk to the clients via the current Plug adapter (web server) in a consistent manner.
API additions
I already have some macros thought out from working on another project the would remove cruft from a WebSockets handler, which should be easy to expand to include the necessary bits for long polling and SSE.

I had some thoughts today about what the end result of an implementation could look like for a developer wanting to use WebSockets in Sugar.
Router
In the router, it should be fairly simple and should remain consistent with the other available macros.
defmodule My.Router do
  use Sugar.Router

  get "/", My.Controllers.Main, :index
  socket "/", My.Controllers.Main, :notifications, name: :notifications
  socket "/chat", My.Controllers.Chat, :index, name: :chat
end

Routes should be able to be sent to standard controllers, allowing a developer to blend functionality as he/she sees fit.
Controller
Inside of a controller, socket/2 will help define socket handlers.
defmodule My.Controllers.Main do
  use Sugar.Controller

  def index(conn, []) do
    # GET handler for index
    render conn
  end

  socket :notifications do
    def handle("notify", socket, data) do
      # Do something on "notify" events
      socket
    end
  end
end

defmodule My.Controllers.Chat do
  use Sugar.Controller

  socket :index do
    def handle("message", socket, data) do
      socket
        |> broadcast(data["message"])
    end

    def handle("message:" <> user, socket, data) do
      socket
        |> send(user, data["message"])
  end
end

I've not seen implementations like this before. Typically, there is a clear separation between socket handlers and standard actions (i.e. different modules), so there will need to be some testing to see if this is a possibility.
Events
With the :name option on the routes, modules outside of a set of socket handlers in a controller can still send messages for events.
defmodule My.Queries.User do
  import Ecto.Query
  alias Sugar.Events

  after_insert :notify

  def create(data) do
    # Build query
    My.Repos.Main.insert(query)
  end

  def notify(changeset) do
    Events.broadcast(:notifications, "new:user", changeset)
  end
end

This is a rough idea, so I'd love to see any feedback on this direction.

I think the 'handle' functions should be at the same level than the regular actions inside the controller. Actually in that scenario the 'socket' macro in the controller is not needed (and the first rule of macros is...).
Pattern matching can be used to determine the right 'handle' function for a given 'socket' route.
Something like 'def handle(: notifications, "notify", socket, data)'.
That would keep controllers from getting messy when they grow as they would always be one level deep functions. Also allow simpler testability.
To free a controller from some code one can always put it on other module and 'use' it from the controller.
About 'Events' , I can't avoid feeling uncomfortable sending data to a client straight from models, or outside of a controller at all. I have no constructive proposal for this right now, but then again, it's up to the developer to do it that way. So I guess it's ok.
Thanks for asking anyway.
I'm glad sugar keeps growing!

Thanks for the feedback, @rubencaro! I definitely agree that macros should only be used when necessary, and in this case, I'm thinking there will be some work needed at compile time. For things like Sugar.Events.broadcast/3, handler names will most likely need to be registered so those functions will know what will actually do the handling.
There's always a good possibility that things can be simplified, especially since my first passes in my head aren't always the best plan of attack. I always want to keep things as simple as possible.
The example with the after_insert Ecto callback was just the first thing that popped into my head. I would probably never do that as well, but I wanted something to clearly demonstrate what that module could be used for.
If you have any other criticisms, please don't hesitate to lay them out.

I'd personally lean toward keeping as much socket-specific stuff in the router (and outside the controller) as possible; it seems more appropriate (and more natural and more intuitive and less confusing) as a routing concern in general (including whatever compile-time stuff is necessary), reserving the controller(s) for performing actions upon sockets without needing to worry nearly as much about the semantics of how/why those actions were called.
In other words, I'd probably prefer something structured like so (though probably not exactly like so; my experience with WebSockets is admittedly not very developed compared to that of more typical HTTP requests, so my imagination of this might be atypical relative to what a more experienced WebSockets guru would expect):
defmodule My.Router do
  use Sugar.Router

  get "/", My.Controllers.Main, :index

  # magic happens here
  socket "/" do
    # magic also happens here
    handle "notify", My.Controllers.Main
  end

  socket "/chat" do
    # Not very DRY in this example, but the point is clarity and intuition
    handle "message", My.Controllers.Chat
    handle "message:*", My.Controllers.Chat
  end
end

defmodule My.Controllers.Main do
  use Sugar.Controller

  def index(conn, _args) do
    conn |> do_something
  end

  # socket handler defaults to "handle"; uses pattern matching to figure out
  # what to handle
  def handle("notify", socket, data) do
    socket |> do_something_with(data["notification"])  # or somesuch
  end
end

defmodule My.Controllers.Chat do
  use Sugar.Controller

  # same deal here; default to "handle/3", use pattern matching
  def handle("message", socket, data) do
    socket |> broadcast(data["message"])
  end

  def handle("message:" <> user, socket, data) do
    socket |> send(user, data["message"])
  end
end

Basically, if we're going to do before-compile magic in order to get our sockets/handlers setup, I think it should be in the router (where we've already piled on a bunch of macros and where the addition of these particular macros would make the most sense) rather than the controller (which is currently pretty magic-free (other than the automagical view rendering, perhaps) and probably shouldn't be burdened with what really amounts to routing concerns).
I don't know how feasible this will be in practice, but I reckon it should be similar to the way Phoenix.Channel (for example) works, but with more intuitive terminology in use.

Putting a little more thought into my own suggestion, I think something like
defmodule My.Router do
  use Sugar.Router
  # ...
  socket "/chat", My.Controllers.Chat
end

would probably also be sufficiently terse without sacrificing clarity if it's documented that the router will direct to the specified controller's handle/3 by default (the above recommendation could still work if different events on the same socket should be handled by different controllers).
Of course, if more explicitness is required, we could expand this to accept an optional atom representing a function name, like so:
defmodule My.Router do
  use Sugar.Router
  # ...
  socket "/chat", My.Controllers.Chat, :chat
end

defmodule My.Controllers.Chat do
  use Sugar.Controller
  # ...
  def chat("message:" <> user, socket, data) do
    socket |> send(user, data["message"])
  end
end

Of course, coming up with ideas is the easy part.  No idea how well this will translate to reality :)

+1 for all that!

:thumbsup: I'm going to start poking around, probably within the week, to see what's possible. I'm hoping we can do most of this without being limited too much by getting Cowboy/Plug to play nicely when it comes to WebSockets.

Started in on this tonight and got a demo going with Plug and a WebSocket handler. Documenting here:

Cowboy WebSocket handler:
defmodule WsHandler do
  @behaviour :cowboy_websocket_handler

  ## Init

  def init(_transport, _req, _opts) do
    {:upgrade, :protocol, :cowboy_websocket}
  end

  def websocket_init(_transport, req, _opts) do
    {:ok, req, :undefined_state}
  end

  ## Handle

  def websocket_handle({:text, msg}, req, state) do
    {:reply, {:text, "responding to " <> msg}, req, state, :hibernate}
  end

  def websocket_handle(_any, req, state) do
    {:reply, {:text, "whut?"}, req, state, :hibernate}
  end

  ## Info

  def websocket_info({:timeout, _ref, msg}, req, state) do
    {:reply, {:text, msg}, req, state}
  end

  def websocket_info(_info, req, state) do
    {:ok, req, state, :hibernate}
  end

  ## Terminate

  def websocket_terminate(_Reason, _req, _state) do
    :ok
  end

  def terminate(_reason, _req, _state) do
    :ok
  end
end

Standard Plug:
defmodule Router do
  use Plug.Router

  plug :match
  plug :dispatch

  get "/" do
    conn
      |> send_resp(200, "index")
  end
end

Application callback:
defmodule WebSocket do
  use Application

  def start(_type, _args) do
    import Supervisor.Spec, warn: false

    children = []
    plug = Router
    opts = []
    dispatch = build_dispatch(plug, opts)

    Plug.Adapters.Cowboy.http plug, opts, [dispatch: dispatch]

    opts = [strategy: :one_for_one, name: WebSocket.Supervisor]
    Supervisor.start_link(children, opts)
  end

  defp build_dispatch(plug, opts) do
    opts = plug.init(opts)
    [{:_, [ {"/ws", WsHandler, []},
            {:_, Plug.Adapters.Cowboy.Handler, {plug, opts}} ]}]
  end
end

Biggest point to note there is the use of build_dispatch/2. By default, the Cowboy adapter for Plug will use only {:_, Plug.Adapters.Cowboy.Handler, {plug, opts}} in the dispatch table passed to Cowboy. When sending a custom dispatch table (which is needed in order to add the customer WebSocket handler, we have to add {:_, Plug.Adapters.Cowboy.Handler, {plug, opts}} on our own as a catch-all for the Cowboy dispatcher to use because the adapter will only use what we pass it.
At compile time, we'll need to build a {"/ws", WsHandler, []} for each use of the socket/3 macro in the router to pass our controllers and handle functions to future Cowboy.WebSocket.Handler, which will take the place of the above WsHandler.

For anyone interested in the development, you can follow along here: https://github.com/slogsdon/plug-web-socket

Alright, I have a good, extremely simple base ready for review: https://github.com/slogsdon/plug-web-socket. If things go well, I'll start on the Event layer or the socket/4 macro.
Tear this up as much as possible. I won't feel bad. Thanks!
/cc @YellowApple @rubencaro
Walkthrough
WebSocket.Cowboy.Handler acts as a general Cowboy handler to accepting connections. It should be used in dispatch tables, getting passed a module and function name responsible for handling WebSocket messages.
The socket/4 macro should be fairly straightforward. All it really needs to accomplish is accumulate the list of routes for the dispatch table, and before the router is compiled, the dispatch table just needs to end up in the format in the demo, ready to be called in the run function of the router.
There's nothing needed in a controller except plain functions. Currently, the handling of websocket_init and websocket_terminate are required in the controller using atoms instead of strings for the first argument, :init and :terminate respectively. These will eventually be optional. The functions for handling the WebSocket messages can only handle text-based payloads at the moment, which covers more use cases than binary data, but binary data will eventually be supported. Also, some helper functions should be created to simplify building proper replies that the Handler knows how to use when responding to clients.
At the moment, the socket argument is a slightly modified Plug.Conn, with the scheme updated to be either :ws or :wss. This will most likely need something more, but for the time being, it works (kinda). A big plus is that information about the connected client is available, just as with HTTP clients.
By default, WebSocket messages are to be a WebSocket.Cowboy.Handler.Message, carrying an event name and a data payload to and from a connected client. This allows for the separation of the type of message and the data for the message. This is similar to Phoenix's notion of topics. There is an option to disable this (see the Echo test for an example), allowing for the ability to parse the raw message in the controllers, but I'm not sure how I feel about that. It adds a fair amount of needed complexity to handle the messages for use cases beyond a simple echo server, so I'm thinking no one would actually use this.

I went ahead with a basic Events layer: WebSocket.Events. Public API looks like this:
defmodule WebSocket.Events do
  use GenEvents

  def start_link(ref) do
    case GenEvent.start_link(name: ref) do
      {:ok, pid} ->
        GenEvent.add_handler(ref, __MODULE__, [])
        {:ok, pid}
      {:error, {:already_started, pid}} ->
        {:ok, pid}
      otherwise ->
        otherwise
    end
  end

  def join(ref, pid) do
    GenEvent.notify(ref, {:add_client, pid})
  end

  def leave(ref, pid) do
    GenEvent.notify(ref, {:remove_client, pid})
  end

  def broadcast(ref, event, originator) do
    GenEvent.notify(ref, {:send, event, originator})
  end

  def broadcast!(ref, event) do
    broadcast(ref, event, nil)
  end

  def stop(ref) do
    GenEvent.stop(ref)
  end
...
end

Ok, last update. Worked on the macro, but it's not all that it can hopefully be.
defmodule WebSocket.Router do
  use Plug.Router
  use WebSocket.Macro

  socket "/topic", WebSocket.TopicController, :handle
  socket "/echo",  WebSocket.EchoController,  :echo

  def run(opts) do
    plug = __MODULE__
    opts = plug.init(opts)
    # build_dispatch/3 is from WebSocket.Macro
    dispatch = build_dispatch(plug, ws_routes, opts)
    Plug.Adapters.Cowboy.http plug, opts, [dispatch: dispatch]
  end

  # ...
end

I want to be able to define run/1 function during __before_compile__/1, but the plug.init(opts) bit creates trouble because since the module is still being compiled, the compiler complains about the init/1 function not existing. The nicest (only only working) work around I've found so far is to do the above, create run/1 manually, which will most likely cause issues with Sugar.Router as it creates a run/1 during the __before_compile__/1 call. This might require some rethinking of how/where run/1 is defined.


Login Shells?

Would be nice to have login shells in lite and execute whatever command you want, instead of executing a pre-defined script.

Full terminal emulation would be a huge undertaking and is far outside the scope of the project

I guess I will close this, then.


Make navbar chosen tab change colour

If the user is on the about page, currently the navbar text has no color change, so the user can get confused about which page they're on.
Expected behaviour should be that on whichever tab the user is on, the text on the navbar for that page should change color (the green that the rest of pages uses).
When user hovers over a button on the navbar, the color of the text should also turn green.

Working on this

@SHADOW-LIGHTS ty, what's ur discord name?

"Light" is the username


MKVToolNix Directory selection makes the extraction of subtitles impossible

Combining path of pMkvMerge variable and string literal "mkvmerge.exe" doesn't work properly because of missing \ at the end of pMkvMerge

I have a rewrite of the whole program in the works, which will include a CLI, to finally work on Linux too. I wouldn't bother with the codebase myself, which will soon be completely replaced. But thank you for your message and if you have ideas for improvement for the next version, I will try to implement them.


Yatin: Watch ivanovation.ro?software

Yatin requests the watch of the watch_keyword ivanovation\.ro?software. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

ivanovation\.ro?software has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Rejected by Jeff Schaller in Charcoal HQ. Reason: 'Rejected at author's request'


UI changes

I like the immersive UI effect that makes the status bar transparent.
I've also added shadow boxes on buttons and text fields, along with some other minor UI changes.

lgtm, try to keep commits smaller in the future though. Its hard to see whats changed when so many files have been changed

yeah sorry it caused too many conflicts, try push again


x/website: blog/comparable: bad punctuation

https://go.dev/blog/comparable

introduced with Go 1.18. comparable is an interface type

Period after "1.18".

Also:

But this is ok: we do like to know that

"ok" should be OK or okay.

CC @griesemer
I think the punctuation is intended and correct: comparable starts the next sentence, and since it is a Go identifier it is correct not to change its capitalization.

"ok" should be OK or okay.

Sure, I think that's right. But IMO this is benign, and it's OK (or okay :)) for the blog to have some typo's / misspelling / grammar issues. I suspect there are more!  With that said, I defer to @griesemer as to whether or not this particular typo should be fixed.

I think the punctuation is intended and correct: comparable starts the next sentence, and since it is a Go identifier it is correct not to change its capitalization.

I see it now, I agree.


X11 Client Black Border around remoteapp window that appears on all workspaces

Describe the bug
Black border renders above (most) other windows and appears on all workspaces. Only occurs during remoteapp connection to Windows 10 host (Education) running Visual Studio 2017.
To Reproduce

RemoteApp Visual Studio 2017 setup on Windows 10 with RemoteAppTool
Execute command line
Splash screen has no border
Slightly after the main application window appears, the black border appears
Drag other windows onto border and border overlaps them
Switch to another work space and the black border is still rendered (not consistent, sometimes the remoteapp window minimizes/disappears on workspace switch)
Mouse can still interact with border (aka resize) when it is overlapping another window or on a different desktop.

Expected behavior
No black border that renders above all other windows on all workspaces
Screenshots

Linux Environment

Distro: Manjaro (latest)
Kernel 4.17
Window manager: bug occurs on both i3 and MATE

Application details

Affected Version of FreeRDP
FreeRDP version 2.0.0-rc3 (n/a)
FreeRDP version 2.0.0-dev4 (c2e2eb910) [latest as of now]
Command line used
xfreerdp /u:user /app:"||vs17" /v:hostname
output of /buildconfig
Not sure where this is.
OS version connecting to
Windows 10 Education Version 10.0.16299.611
log level trace
https://gist.github.com/mj-sakellaropoulos/3099b0c1b6bddc0850ccc79074bc25ff

xwininfo & xprop of window
xwininfo: Window id: 0x3c0015b "Start Page - Microsoft Visual Studio "
  Absolute upper-left X:  1803
  Absolute upper-left Y:  19
  Relative upper-left X:  0
  Relative upper-left Y:  0
  Width: 1258
  Height: 2085
  Depth: 24
  Visual: 0x20
  Visual Class: TrueColor
  Border width: 0
  Class: InputOutput
  Colormap: 0x22 (installed)
  Bit Gravity State: ForgetGravity
  Window Gravity State: NorthWestGravity
  Backing Store State: NotUseful
  Save Under State: no
  Map State: IsViewable
  Override Redirect State: no
  Corners:  +1803+19  -779+19  -779-56  +1803-56
  -geometry 1058x1885+1803+19

~ >>> xprop
WM_NORMAL_HINTS(WM_SIZE_HINTS):
		program specified minimum size: 200 by 200
		program specified maximum size: 3854 by 2174
		program specified resize increment: 1 by 1
_NET_WM_DESKTOP(CARDINAL) = 0
WM_STATE(WM_STATE):
		window state: Normal
		icon window: 0x0
_NET_WM_NAME(UTF8_STRING) = "Start Page - Microsoft Visual Studio "
WM_NAME(STRING) = "Start Page - Microsoft Visual Studio "
_NET_WM_PID(CARDINAL) = 10948
_NET_WM_WINDOW_TYPE(ATOM) = _NET_WM_WINDOW_TYPE_NORMAL
_MOTIF_WM_HINTS(_MOTIF_WM_HINTS) = 0x3, 0x1, 0x0, 0x0, 0x0
WM_PROTOCOLS(ATOM): protocols  WM_DELETE_WINDOW
WM_HINTS(WM_HINTS):
		Client accepts input or input focus: True
WM_CLASS(STRING) = "RAIL", "RAIL:000202EC"

xwininfo & xprop of border
xwininfo: Window id: 0x3c001b1 (has no name)
  Absolute upper-left X:  1922
  Absolute upper-left Y:  710
  Relative upper-left X:  1922
  Relative upper-left Y:  710
  Width: 1908
  Height: 9
  Depth: 24
  Visual: 0x20
  Visual Class: TrueColor
  Border width: 0
  Class: InputOutput
  Colormap: 0x22 (installed)
  Bit Gravity State: ForgetGravity
  Window Gravity State: NorthWestGravity
  Backing Store State: NotUseful
  Save Under State: no
  Map State: IsViewable
  Override Redirect State: yes
  Corners:  +1922+710  -10+710  -10-1441  +1922-1441
  -geometry 1798x65516-10+710

~ >>> xprop
WM_NORMAL_HINTS(WM_SIZE_HINTS):
		program specified minimum size: 110 by 29
		program specified maximum size: 3854 by 2174
		program specified resize increment: 1 by 1
_NET_WM_NAME(UTF8_STRING) =
WM_NAME(STRING) =
_NET_WM_PID(CARDINAL) = 10948
_NET_WM_WINDOW_TYPE(ATOM) = _NET_WM_WINDOW_TYPE_POPUP
_NET_WM_STATE(ATOM) = _NET_WM_STATE_SKIP_PAGER, _NET_WM_STATE_SKIP_TASKBAR
_MOTIF_WM_HINTS(_MOTIF_WM_HINTS) = 0x3, 0x1, 0x0, 0x0, 0x0
WM_PROTOCOLS(ATOM): protocols  WM_DELETE_WINDOW
WM_HINTS(WM_HINTS):
		Client accepts input or input focus: True
WM_CLASS(STRING) = "RAIL", "RAIL:00040210"

Update: Apparently each edge of the "border" is a window. Running xdotool windowunmap $border_part_win_id removes that edge of the border.

Cause: The bug is caused by the border windows being designated as "override redirect" windows. The only way to solve this is to properly detect these border windows and not apply the "override redirect" designation. ref /client/X11/xf_window.c function xf_SetWindowStyle.
I have dug as far as I can with my little experience, the rest is up to someone else.

Cause for being drawn on top of everything:

The 4 VisualStudioGlowWindow (s) have the WS_EX_TOOLWINDOW style applied to them, which cause freerdp to set them as "override redirect" (xf_window.c line 648).
They also have the style WS_POPUP, which triggers xf_SetWindowUnlisted to add the _NET_WM_STATE_SKIP_PAGER hint.
Cause for the border being larger than expected: I suspect those windows normally have an alpha applied to them on Windows, causing only the 1px purple/grey stip to be visible. The windows always have a 9px width, they are just normally transparent.
I hope my sleuthing can help resolve this.

@mj-sakellaropoulos Thank you for the analysis, hope to find time to fix that

I've been able to repro this on the official windows remoteapp client, the black border will appear on any window that has the "Metro Border" aka drop shadows, glows, etc. The only bug unique to FreeRDP is the border rendering above other windows. The broken alpha channel seems to be common to many platforms.

I've tried to fix it with #4891 but don't know if all corner cases are fixed (or just my setup)
could anyone verify that?

Note, the fix should only remove the fact that the window is above the currently active one, the transparency issue is a harder beast.

No improvements on my end
Not sure I'm doing this right? I'm testing (same build config) on the "rail_fixes" branch on the fork referenced by PR. I'm also getting intermittent crashes opening notepad with rail and this message is getting printed repeatedly:  [INFO][com.freerdp.client.x11] - Property 337 does not exist. I can also no longer interact with the visual studio window (clicking, etc).

I'm experiencing the exact same issue as @mj-sakellaropoulos described. The black borders are rendered on top of other windows.
This happens for me on both the release version of FreeRDP on Ubuntu 18.04 as well as in the nightly build.

The issue only occurs when Visual Studio is windowed -- the 1-pixel focus border is not drawn when maximized.
Windows 10 version 1809
Visual Studio 2017 & Visual Studio 2019

$ /opt/freerdp-nightly/bin/xfreerdp --version
This is FreeRDP version 2.0.0-dev5 (f867c90e4)

@akallabeth should I create a new issue or would you re-open this one?

I can confirm that I'm having the exact same issue as described by @warreq
Windows Server 2016
Visual Studio 2017

$ /opt/freerdp-nightly/bin/xfreerdp --version
This is FreeRDP version 2.0.0-dev5 (4a6b843f1)

I can also confirm the crash issue @mj-sakellaropoulos mentioned still persists as of latest FreeRDP version available in the git.
In my case I am using the remoteappmode of FreeRDP in order to open Microsoft Outlook application remotely. The outlook application opens normally but then we have an authentication window pop up and as a result both the authentication window and the outlook application crash. When looking at the output in the terminal I see the message “[INFO] [com.freerdp.client.x11] Property 304 (_NET_WM_STATE) does not exist” (I added the atom property name in the brackets by printing it from the code).
This issue was tested on Ubuntu 16.04 + 18.04 , Arch Linux 5, Mint 18 and CentOS 6+7 but was only occurring on Ubuntu 18.04 and CentOS 7.
This issue happens in both the latest FreeRDP release (v 2.0.0 rc4) and in the master version (v 2.0.0-dev5).
@akallabeth Do you have any suggestion for us on how to solve this issue? I can also create a new issue if required.
Thanks!

Still have this issue on 2.1.2

Yes, 2.2.0 with this bug as well

Might be time to reopen this issue as it still exists with 2.3.2 on i3wm.

Issue still persists.
Guest: Windows 10.
App: Visual Studio 2022 Community
Host: Ubuntu 22.04, Wayland
XFreeRDP: 2.6.1
Command:
xfreerdp /u:$RDP_User /p:$RDP_Password /v:$RDP_IP /gdi:hw +clipboard /dynamic-resolution /bpp:32 /cert:ignore /app:"C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\devenv.exe"
Only when running Visual Studio does it leave a border when a native window is brought into the foreground and VS 2022 sent to background.
Any other app running on the guest seems fine.

Issue still persists.
Guest: Windows 10. App: Visual Studio 2022 Community
Host: Ubuntu 22.04, Wayland XFreeRDP: 2.6.1
Command: xfreerdp /u:$RDP_User /p:$RDP_Password /v:$RDP_IP /gdi:hw +clipboard /dynamic-resolution /bpp:32 /cert:ignore /app:"C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\devenv.exe"
Only when running Visual Studio does it leave a border when a native window is brought into the foreground and VS 2022 sent to background.
Any other app running on the guest seems fine.

Also having this issue on same setup. Any news?


Not able to install Azure Dependency Agent on Azure RedHat 9.4.

Type of issue
Other (describe below)
Feedback
Not able to install Azure Dependency Agent on Azure RedHat 9.4.

Page URL
NA
Content source URL
Na
Author
Abhijeet Pramanik
Document Id
Na

@abhijeet4022
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

@PesalaPavan Please find the below URL.
https://learn.microsoft.com/en-us/azure/azure-monitor/vm/vminsights-dependency-agent-maintenance

@abhijeet4022
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@abhijeet4022, thank you for your feedback. Dependency Agent doesn't support Red Hat Linux 9+, see Dependency Agent Linux support.
If you have any further questions, please reach out to us via one of the following support channels. #please-close
Microsoft Azure Monitor Q&A question page:
https://docs.microsoft.com/answers/topics/24223/azure-monitor.html
Get support for your Azure services:
https://azure.microsoft.com/support/create-ticket/


Local dev server

Can I suggest https://github.com/lambci/docker-lambda
I would also say that a local dev server is out of scope for the project


Branch at forked repo with actions enabled fails publish-java-snapshot

Behaviour is the expected, as the workflow is running in a separate repo, but, could the execution of publish-java-snapshot excluded when running in forked repo?

not really needed, cancelling


Refactor loadBalance some code

Feature Request

Is your feature request related to a problem? Please describe
A clear and concise description of what the problem is. Ex. I have an issue when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen. Add any considered drawbacks.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Please read the Contribution Guideline before submitting the PR

Does a related pr already exist? @yu199195


[chip] Incorrect PC value after synchronous ISR

Related to #12754
After applying #12782, the return from ISR works. However, right after return from the software ISR, the static variable address is mis-calculated.
software_intr_triggered address should be 0x10000064. However, the core accesses 0x0FFFFD3C for the address.
The disassembled code shows the code uses AUIPC and ADDI to access the variable:
20001d38:>  efffe417          > auipc>  s0,0xefffe
20001d3c:>  32c40413          > addi>   s0,s0,812 # 10000064 <software_intr_triggered>
20001d40:>  4008                >   lw> a0,0(s0)

So, it seems that the PC used in AUIPC instruction seems to be incorrect. This line is right after the exit from the software isr.

As compute_mepc_on_synchronous_irq manipulates MEPC to point to the next instruction, the ADDI never executed. These instructions are right next to SWI.

Need to check what is the proper step to call software interrupt...

Closing this as fixes are merged.


Ensure website is mobile-friendly

Most of our development has been done on desktop. It's entirely possible everything works well out of the box on mobile, but in either case we should check and confirm this.

Mike says it's ready for production, although with one caveat:
Only issue is that the sidebar, while it opens smoothly, just closes abruptly when you collapse it. No immediately apparent easy way to solve this, but it's such a minor issue that trying to solve it is likely more trouble than it's worth.

all done!


Feature/quiz

Hi team,
I'm pleased to share with you the latest version of the quiz feature. This version is fully functional and ready for review.
Changes in this version:

Implemented functionality to retrieve and display quiz progress based on stored data in local storage.
Refactored code to improve readability and maintainability, including the use of dynamic rendering for progress bars.
Added error handling for cases where no data is retrieved from local storage.

This version has been thoroughly tested and should not cause any merge conflicts. However, I'm open to any feedback or suggestions for further improvements.
Please review the changes and merge the pull request if you're happy with the current implementation. I may make further edits based on your feedback, but for now, I'd like to keep this version as it is since it meets the functional requirements.
Thank you for your attention to this pull request.
Kate :)
npm install react-hot-toast react-circular-progressbar

I have deleted the part that was causing conflict (=====main====features/quiz).


Importing JSON datetime columns

New to Danfo, very clean and easy to work with, great job. :+1: I do have some problems understanding dates though.
In my JSON:
[ { "recordedAt": "2017-02-16T00:00:00.000Z", "value": 300 }]

In my browser Danfo 0.1.2 script:
df = new dfd.DataFrame(myJson)
df.ctypes.print()

The dtype for the recordedAt column is a string, this makes sense, JSON does not have dates. I tried appending columns where I convert the date string to a JavaScript date or a timestamp but since Danfo does not have a datetime dtype I can't explain Danfo the column data is representing a date. The examples all use date ranges but what if I already have dates in my input file?
Thanks!

@steveoni what do you think. Probably our date parser is not robust enough

@steveoni what do you think. Probably our date parser is not robust enough

We don't have datetime as a ctypes or so. But the date parser is robust enough.
myjson = [ { "recordedAt": "2017-02-16T00:00:00.000Z", "value": 300 }]
df = new dfd.DataFrame(myjson)

dateSeries = df["recordedAt"].dt

table(dateSeries.month_name())

We don't have datetime as a ctypes or so. But the date parser is robust enough.
myjson = [ { "recordedAt": "2017-02-16T00:00:00.000Z", "value": 300 }]
df = new dfd.DataFrame(myjson)

dateSeries = df["recordedAt"].dt

table(dateSeries.month_name())

I see. I didn't realize I had to call dt to convert the column to a date. I taught the DateFrame constructor would convert the values to dates internally.
df = new dfd.DataFrame(myjson)
df.set_index({ key: "readAt", inplace: true })
df.plot("plot_div").line()

It works now!

I see. I didn't realize I had to call dt to convert the column to a date. I taught the DateFrame constructor would convert the values to dates internally.
df = new dfd.DataFrame(myjson)
df.set_index({ key: "readAt", inplace: true })
df.plot("plot_div").line()

It works now!

It looks like the info on this page is no longer accurate: https://danfo.jsdata.org/api-reference/general-functions/danfo.to_datetime
Following the example code with toDateTime() throws an error where as using .dt appears to works swimmingly.

Correct @chapmanderek ! I have the same experience!  The api reference seems to be is out of date.


Use DynamoDB autoscaling when we're reindexing in a pipeline

So we'll get a big pile of provisioned capacity when everything goes through the matcher, but when we're waiting for the ingestor it'll scale away. 💸 💸 💸

Maybe add a comment somewhere obvious about how often this can (or rather, can't) be changed?

There's a comment a bit further up the file, is that good enough?

Ah yes, so there is 👍


Update request library

Closes #12 and closes #8. Use the axios library which works on the client and server with unchanged browser compatability.

@alanshaw can we update the version of node that Travis CI is using to run tests?

@treyhuffine also yes, feel free to update the travis node version - 4 & 6?

:thumbsup: thank you!


Profile layout

Summary
Implemented components in:

Avatar.tsx
Banner.tsx
ProfileTemplate.tsx

Implemented

profile.tsx

Testing

Great job guys! Kindly remove yarn.lock and package-lock.json from the pr

yarn.lock and package-lock.json removed! :)


feat: support light mode

Is it possible to support light mode?

Yes, I'll work on it!

I'll work on it soon!


Code review and improvements

Check the code for best practices:

OOP
SOLID
Design Patterns
Docstrings
Overall formatting

Found a minor bug in the reset endpoint


add unimaat school file

Url: UNIMAAT

@EduardoMay First stage of verification passed, official domain is unimaat.mx as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://www.unimaat.mx/nosotros

@EduardoMay You might miss my previous comment. Please react.

@EduardoMay I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


ViewComponent attribute bugs

Passing a variable previously defined doesn't work ✅
<?php
$meta = MetaType::CONSOLE;
?>

<x-base title="Console" :meta="$meta">

attribute name casing problem
<x-base title="Console" :metaType="abc">

// $metaType now known in parent

Attributes passed to parent are only accessible via $this ✅
<?php $meta ?> // doesn't work
<?php $this->meta ?> // works

<x-slot></x-slot> does not work
Only <x-slot /> does

Hey @brendt, while I was working in the upload stuff, I saw that attributes aren't passed to the component so, I reported it in #381.
Could you take a look because I updated the #381, and let know if I need to close it!

Should be fixed once this is merged :)

I made an issue about the casing conventions, we need to document them: 458


added new project

I added a beginner friendly open source project - coala, to the helpful material page.
Also added my name to the contributors page.

Great addition


Auto-refresh didn’t happen after wake from sleep in the morning

I woke up my laptop in the morning. An hour later it hasn’t refreshed yet. (My setting is to refresh feeds every hour.)

This one has me puzzled.  I checked and we are still getting the wake notification on Big Sur and it is calling fireOldTimers.  This was the fix for this previously, so I'll have to dig in a little deeper here.

I can't recreate this.  I'm syncing fine after coming back from sleep.  I think we'll need more criteria to figure out what caused this.

I wonder if this is a result of my running a Debug build.

If you have the debug menu enabled, I think you have to hit the refresh once to get the timer going.

Closing this. We’ll see if other people report it or if it’s jut me running a Debug build.


Add guide to add auth to Vercel with open source Ory

This PR adds a guide which explains how to add authentication with login, user management, registration, ... to NextJS deployed on Vercel. We use the Ory Kratos open source identity server as the user backend.

@lfades anything I can do to help get this merged? :)

@aeneasr Hey! Can you add some context to the PR? It's empty now. I'll take a look.

Of course! I updated the opening comment :)

Works pretty good! Could you add the same UI as the rest of the examples? https://edge-functions-bot-protection-datadome.vercel.sh/ ? LMK if you need some help there.

Cool :) Yes, it would be great if you had 1 or 2 pointers!

Cool :) Yes, it would be great if you had 1 or 2 pointers!

Hey @aeneasr thanks for the PR! For using the common layout you can see:

_app.tsx
line 3 and 62 here

If you have any question just let us know :)

Thank you! :) Done :)

Sorry, I am a bit confused-the commenter above said to use the edge template, should I now remove it again?
The code was following other examples in this repo, such as the Clerk one. Also what do you mean with „new folder“?

Gentle ping @lfades :) I think there are a lot of benefits of combining Ory with Next.js and Vercel. We are getting quite a lot of questions around integrating Nextjs with Ory, and our recent blog post about it was on HN front page for two days.
I think our two ecosystems (Ory is the biggest open source auth ecosystem with 45M docker pulls and 30k+ stars) fit well together and Nextjs users will benefit quite a lot from this example and accompanying docs!

Hey @aeneasr! Sorry about the confusion.

To add some clarifications: It's not using _middleware therefore it's not using edge functions.
What we mean by adding it to another folder is because it's misleading to add it under edge-functions if it's not using edge functions (Check the other examples using _middleware and go here for more information https://vercel.com/features/edge-functions).

Don't worry about this one. After merge I can add it to the right folder. 👍

If you check this example: https://edge-functions-bot-protection-botd.vercel.sh/ or this one https://edge-functions-ip-blocking-datadome.vercel.sh/ it shows an interactive landing where it explains what the examples does and creates a flow between buttons. If you think this is done already and showcases everything for Ory then I'll move forward and merge it in a couple of hours if I don't hear back.

Thanks for your contribution! Big :fan: of Ory!

Got it, thank you! :) I'll try to address the feedback today :)

To add some clarifications: It's not using _middleware therefore it's not using edge functions.
What we mean by adding it to another folder is because it's misleading to add it under edge-functions if it's not using edge functions (Check the other examples using _middleware and go here for more information https://vercel.com/features/edge-functions).

Got it :) I saw the new example pages and I think "auth" would be the best suited category :)
I've updated the pages according to your feedback! I tried to mimic the design from the others. Let me know if this works!

Happy new year, and a gentle ping @okbel :)

Happy new year! This looks amazing can @goncy or @lfades take a look? (LGTM)

Wohooo thank you! :)


Size marker images to be larger than the marker itself

This doesn't appear to be as easy as we thought.
When we add a marker:
//Create the THREEAR BarcodeMarker object for this marker
let barcodeMarker = new THREEAR.BarcodeMarker({
  barcodeValue: marker.barcodeValue,
  markerObject: markerGroup,
  size: 1,
  minConfidence: 0.2
});

Changing size to a larger value, like 5, makes it smaller. Changing it to a fraction, like 0.5, causes errors and doesn't show anything.

When I put it to 0.5, it looks the same size as 1 but it sticks to the center of the marker layer until it becomes undetected.

Is this basically closed?
Paul's solution posted on Slack:

let geometry1 = new THREE.PlaneBufferGeometry(2,2, 1,1)

Oh, you know, we actually closed this one with #81!


.bl__body should not be flexbox

As of now, the .bl__body class looks like this:
.bl__body {
  display: flex;
  flex-grow: 1;
}

flex-grow: 1 is needed to make the body fill the space between the header and footer, but display: flex accomplishes nothing other than give bl__body a row format.
Instead of adding a flex-direction: column we can just drop display: flex all together (but keeping the flex-grow: 1).
Another benefit of removing display: flex is that the content of bl__body, like buttons and divs won't fill the whole width of bl__body.

This issue will be fixed in version 1.1.0.


Have a way to easily avoid performing semantic transformations

I execute rustfmt a lot while writing code.
I hate that it removes the mut from variables / function arguments just because I don't need mut yet.
I appreciate a tool that fixes / lints on these issues at the points in the development process in which I want them fixed, but there is nothing more annoying than writing a function prototype or block with some mut variable, hitting tab for it to get automatically indented using rustfmt, and seeing the mut from a variable removed just because I haven't written the line that mutates it yet... Having to either undo or retype it again is annoying.
A flag would probably be enough, although rustfmt --format, rustfmt --format-only, rustfmt --please-change-the-format-only sound creepily tautological too me.
I've argued this before but in my opinion the root of the problem is that rustfmt tries to solve many (hard) problems as if they were one intertwined problem instead of solving single hard problems separately (and as a consequence does a poor job in solving some of them).
A way healthier tooling ecosystem would be built on top of:

a formatting-only tool (that ensures code is well indented),
a rewrite engine (that ensures that code transformations compile),
an AST matching engine (to detect things to be rewritten), and
an AST transformation engine (to specify how the AST should be transformed).

Tools like clippy, the semantic parts of rustfmt, some other rustfix tool, a tool that rewrites variables/types/traits names to conform to coding conventions, extra static analysis,  and many others, could then easily be written on top of these primitive tools that do one hard thing and one hard  thing only (maybe one can write a tooling framework on top of these primitives as well to make writing tools even easier).

I'm surprised that rustfmt does this at all. Could you give an example of code where the mut modifier is removed?

Am I dreaming? I would swear this happened to me yesterday a couple of times and today but I cannot reproduce it so I'm closing this. Weird.


FEATURE adding concatenation transformer #20.

@ivanyu what do you think of this proposed Pr?

@willyborankin  or @ivanyu would it be possible to get a review on this? Thanks!

@willyborankin  or @ivanyu would it be possible to get a review on this? Thanks!

Thank you very much for your PR. It looks OK, but besides comments I and @ivan added could you please provide documentation for this feature as well?

Yep I'll add in the documentation as well. Should have an update this afternoon

Thank you very much for your PR. It looks OK, but besides comments I and @ivan added could you please provide documentation for this feature as well?

Yep I'll add in the documentation as well. Should have an update this afternoon

@willyborankin I've applied all the requested changes except for the Map<String, ?> with that I was receiving the following error
/Users/brown/git/aiven-kafka-connect-transforms/src/main/java/io/aiven/kafka/connect/transforms/ConcatFields.java:104: error: incompatible types: String cannot be converted to CAP#1
            newValue.put(config.outputFieldName(), outputValue.toString());
                                                                       ^
  where CAP#1 is a fresh type-variable:
    CAP#1 extends Object from capture of ?
Note: /Users/brown/git/aiven-kafka-connect-transforms/src/main/java/io/aiven/kafka/connect/transforms/ConcatFields.java uses unchecked or unsafe operations.

I also switched to using StringJoiner at the suggestion of IntelliJ, but can change that if you'd like.

@willyborankin I've applied all the requested changes except for the Map<String, ?> with that I was receiving the following error
/Users/brown/git/aiven-kafka-connect-transforms/src/main/java/io/aiven/kafka/connect/transforms/ConcatFields.java:104: error: incompatible types: String cannot be converted to CAP#1
            newValue.put(config.outputFieldName(), outputValue.toString());
                                                                       ^
  where CAP#1 is a fresh type-variable:
    CAP#1 extends Object from capture of ?
Note: /Users/brown/git/aiven-kafka-connect-transforms/src/main/java/io/aiven/kafka/connect/transforms/ConcatFields.java uses unchecked or unsafe operations.

I also switched to using StringJoiner at the suggestion of IntelliJ, but can change that if you'd like.

@willyborankin I've applied all the requested changes except for the Map<String, ?> with that I was receiving the following error
/Users/brown/git/aiven-kafka-connect-transforms/src/main/java/io/aiven/kafka/connect/transforms/ConcatFields.java:104: error: incompatible types: String cannot be converted to CAP#1
            newValue.put(config.outputFieldName(), outputValue.toString());
                                                                       ^
  where CAP#1 is a fresh type-variable:
    CAP#1 extends Object from capture of ?
Note: /Users/brown/git/aiven-kafka-connect-transforms/src/main/java/io/aiven/kafka/connect/transforms/ConcatFields.java uses unchecked or unsafe operations.

I also switched to using StringJoiner at the suggestion of IntelliJ, but can change that if you'd like.

Then it should be Map<?, ?>

@willyborankin I've applied all the requested changes except for the Map<String, ?> with that I was receiving the following error
/Users/brown/git/aiven-kafka-connect-transforms/src/main/java/io/aiven/kafka/connect/transforms/ConcatFields.java:104: error: incompatible types: String cannot be converted to CAP#1
            newValue.put(config.outputFieldName(), outputValue.toString());
                                                                       ^
  where CAP#1 is a fresh type-variable:
    CAP#1 extends Object from capture of ?
Note: /Users/brown/git/aiven-kafka-connect-transforms/src/main/java/io/aiven/kafka/connect/transforms/ConcatFields.java uses unchecked or unsafe operations.

I also switched to using StringJoiner at the suggestion of IntelliJ, but can change that if you'd like.

Then it should be Map<?, ?>

Ok I've switched to Map<?, ?> for the cast. I still have to keep the return type as Map, making it Map<?, ?> caused
> Task :compileJava FAILED
/Users/brown/git/aiven-kafka-connect-transforms/src/main/java/io/aiven/kafka/connect/transforms/ConcatFields.java:103: error: method put in interface Map<K,V> cannot be applied to given types;
            newValue.put(config.outputFieldName(), outputValue.toString());
                    ^
  required: CAP#1,CAP#2
  found: String,String
  reason: argument mismatch; String cannot be converted to CAP#1
  where K,V are type-variables:
    K extends Object declared in interface Map
    V extends Object declared in interface Map
  where CAP#1,CAP#2 are fresh type-variables:
    CAP#1 extends Object from capture of ?
    CAP#2 extends Object from capture of ?
1 error

FAILURE: Build failed with an exception.

Let me know any other changes you'd like and I'll knock them out, thanks!

Awesome I'll work through these comments in the next day or two. Thanks!

@ivanyu addresses all of your feedback.

@ivanyu I have addressed your most recent feedback, plus I've added a fix for the broken implementation of the Hash transformer

the broken build on integration seems to be caused by com.github.dockerjava.api.exception.NotFoundException: {"message":"No such image: quay.io/testcontainers/ryuk:0.2.3"}

@ivanyu I was able to fix the broken integration tests by bumping the test containers version.

Thank you to the fix in Hash, @brbrown25! May I ask you to make a separate PR for this?

May I ask you to make a separate PR for this?

Yes I'll do that this morning and then remove the commit from this pr!

Flattened and I moved the hash changes into a separate pr here https://github.com/aiven/aiven-kafka-connect-transforms/pull/31

@ivanyu I went ahead and removed the conditional logic from this one as well per your comment on the hash transformer.


[BUG] when using Set-PnPFolderPermission to set permissions on a folder, the folder is not showing up in the user's "Shared" view in the onedrive.

Notice
Many bugs reported are actually related to the PnP Framework which is used behind the scenes. Consider carefully where to report an issue:

Are you using Invoke-PnPSiteTemplate or Get-PnPSiteTemplate? The issue is most likely related to the Provisioning Engine. The Provisioning engine is not located in the PowerShell repo. Please report the issue here: https://github.com/pnp/pnpframework/issues.
Is the issue related to the cmdlet itself, its parameters, the syntax, or do you suspect it is the code of the cmdlet that is causing the issue? Then please continue reporting the issue in this repo.
If you think that the functionality might be related to the underlying libraries that the cmdlet is calling (We realize that might be difficult to determine), please first double check the code of the cmdlet, which can be found here: https://github.com/pnp/powershell/tree/master/src/Commands. If related to the cmdlet, continue reporting the issue here, otherwise report the issue at https://github.com/pnp/pnpframework/issues

Reporting an Issue or Missing Feature
Please confirm what it is that your reporting

When giving permission to the folder the folder is not showing up in user's "Shared" view in onedrive.

Expected behavior
Please describe what output you expect to see from the PnP PowerShell Cmdlets

Based on the permissions folders/files should show up in the "Shared" view in the "OneDrive"

Actual behavior
Please describe what you see instead. Please provide samples of output or screenshots.

Nothing is showing up in the "Shared" view. The user has to search for the folder with name to view the contents.

Steps to reproduce behavior
Please include complete script or code samples in-line or linked from gists

Use Set-PnPFolderPermission to give permissions on a folder or a file. Once the permissions are giving the file or folder is not showing in the "Shared" view of the OneDrive.

What is the version of the Cmdlet module you are running?
(you can retrieve this by executing Get-Module -Name "PnP.PowerShell" -ListAvailable)

2.12.0

Which operating system/environment are you running PnP PowerShell on?

[* ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

Hi @AshokChava , this is as expected, not a bug. The user needs to search for the folder to access it.
If you want to "share" it, maybe also consider using this cmdlet :
https://pnp.github.io/powershell/cmdlets/Add-PnPFolderSharingInvite.html


Expose rate_limited_as_resource_exhausted in Ambassador configuration

Please describe your use case / problem.
We've configured Ambassador to send rate limit requests to a 3rd party service, and noticed that all 429 HTTP responses code are translated to UNAVAILABLE, as per gRPC mapping document. This isn't particularly helpful. Per Google's recommendation, 429 response shall be translated to RESOURCE_EXHAUSTED.
Envoy added the support to explicitly specify whether we want 429 responses to be mapped to RESOURCE_EXHAUSTED while UNAVAILABLE remains default; this is configurable through a flag named rate_limited_as_resource_exhausted.
It doesn't seem like Ambassador supports this configuration.
Describe the solution you'd like
We'd like Envoy option rate_limited_as_resource_exhausted to be exposed via Ambassador configuration.
Describe alternatives you've considered
Not at this moment.
Additional context
Reference:

https://cloud.google.com/apis/design/errors#generating_errors
https://github.com/envoyproxy/envoy/pull/4879

Hate to necro this thread but... is there still no way to do this?
UNAVAILABLE gRPC status codes for rate-limited responses isn't cutting it for us...

This was addressed in 3.8.2.  Please let us know if issues persist.


[Form Validation] 2.2.8 inputs with type = radio are missing from onFailure callback field parameter

When using form validation inputs with type = radio are validated, but are not included in the onFailure callback "field" parameter.
https://jsfiddle.net/kmd1970/qsfcwnxw/

Fixed in 2.2.11


Use connection bound to model instead of default

Also can you please use Query/Builder or Eloquent instead of raw sql queries.

@rafis I've attempted to at least remove the hard coded SQL here https://github.com/EspadaV8/ClosureTable/blob/master/src/EspadaV8/ClosureTable/Models/ClosureTable.php#L63
It's not as clean as I would like, but it is working across databases now (tested on Postgres).
If you can think of any way to clean them up more I'd be happy to accept some pull requests.

@EspadaV8, thanks for your work! Could you make the same changes to L4 branch (if you haven't yet) and make a pull request to my repository?

@rafis I'm sorry for being so late... I'm accepting your PR!


Only shift players that are still in the game with money

First of all, nice project. Structure looks pretty neat and good to understand.
Update:
Skip the players that are out of money at rotating the order for a new hand

Nice catch! Thank you! :-)


Option to group namespace specifiers

This PR adds the importOrderGroupNamespaceSpecifiers config variable. When true it groups import * as lib from 'lib' imports to the top of the group.
Closes #104

Hi @Mattinton, The build is failing. Could you please check it.

Hi @Mattinton, The build is failing. Could you please check it.

Sorry should have checked this before the PR. Should be passing now.

Is there a standard release schedule for PR's like this?

Is there a standard release schedule for PR's like this?

We will try to release the PR this week.

Hi @Mattinton , we had merged your changes to v3.2.x branch. We will try to release it as soon as possible along with some new changes. Thank you so much for your contribution ❤

Hi, is there any progress on a release for this?
Cheers!

Sent again, let me know if any issues :D

Hi @Mattinton, I still have not received the mail. I checked the mail systems and they are working fine. Could you please drop the information to<EMAIL_ADDRESS>and please cc<EMAIL_ADDRESS>

Sent again again 😆. Hoping it goes through this time!

I received it. Thank you.


Add OpenMPI version configuration

@bradmiro Could you review this patch please. Thanks

putting PR on hold

@medb FYI


1.9 composite export shap bug

1.9 composite export this pic like:

1.8 compose export right

@wushiwein119 Can you share a reproducible example?

psd_10.zip

@wushiwein119 Can you share a reproducible example?

by the way,i have another question. for example: 'pic = layer.compsite()' .How do I change pic into base64 string, that can be use in html; i try to use tobytes() and base64.b64encode()，but i failed, It doesn't work

@wushiwein119

by the way,i have another question.

This is a off topic, but you can use PIL's save() method to export in whatever image format, and encode in base64.


migrate and binject don't work on idle processes

The migrate and binject commands don't work on idle processes.
User: lee | Computer: test | OS: Windows 10 (x64) | Secure: Yes | Ping: 1s
shad0w(lee@target) ≫ getpid
[i] Beacon (27b2499bf0f3386ba90ab1830860d292) received task
PID: 10744
Image: rundll32.exe

shad0w(lee@target) ≫ ps
[i] Beacon (27b2499bf0f3386ba90ab1830860d292) received task
PID     PPID    Arch    User                              Name
===     ====    ====    ====                              ====

0       0                                                 [System Process]
4       0                                                 System
120     4                                                 Registry
636     4                                                 smss.exe
964     744                                               csrss.exe
672     744                                               wininit.exe
676     592                                               csrss.exe
936     672                                               services.exe
5592    1188    x64     ACME-CORP\lee                     SystemSettings.exe

shad0w(lee@test) ≫ migrate -p 5592
[i] Beacon (27b2499bf0f3386ba90ab1830860d292) received task

shad0w(lee@test) ≫ getpid
[i] Beacon (27b2499bf0f3386ba90ab1830860d292) received task
PID: 10744
Image: rundll32.exe

Same results with the binject command.
Note: I tried migrating to a process that I know was active (Chrome) and it worked.

im having similar issues with shinject


Adds ParticipationCard for Election Registration

Adds the Election participation flow to participate and cancel participation. My next iteration will be to detect if the password is present or not and merge the set encryption password before confirming the election.

One more note; we should make sure that when determining whether to prompt the user for their password or not, we also check the validity of their password/private key in the browser against the public key on chain...make sure they match. If they do not, the user should be asked to re-enter their password. (If we're already doing this check, I don't see it.)
This goes for both the check in the registration flow and the banners that appear elsewhere.


Create flakehub-publish-tagged.yml

We can publish our Nix flake to flakehub when there is a release.

Pull Request Test Coverage Report for Build 1819

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 7.039%

Totals

Change from base Build 1818:
0.0%

Covered Lines:
2574

Relevant Lines:
36566

💛  - Coveralls


NWPS-1082: Resource/document link rewrite for hee:fileLink node type

Rewrites (resource) index on document/resource links with path (forward slash) separator, but without square brackets to avoid the 404 error returned by brCloud for index based resources/documents.
Example: Rewrites /content/documents/lks/common/file-links-cards/downloads/downloads/hee:fileLinks[2]/hee:file as /content/documents/lks/common/file-links-cards/downloads/downloads/hee:fileLinks/2/hee:file (for rendering resource URLs) and vice versa (for processing the resource requests).

@smanivasag @FranciscoRuizHEE
I am closing this PR as it isn't needed now


Alerts/Warnings not available.

Alerts and warnings are not showing currently on 0.4.3.

Unsure if this is an AllisonHouse issue but I recommend COD as they have the same warning format as AllisonHouse and/or allowing users to pick between the two if one happens to go down.

[GR2 Above.]

[Supercell Below]

Yes, the AllisonHouse server is down.

Option 1: Use a placefile
Option 2: Login to GitHub, and download the development snapshot (https://github.com/dpaulat/supercell-wx/actions/runs/8842657471). In settings, change your warnings server to https://warnings.cod.edu

I suggest having wX installed on your phone and receiving the alerts that way. Plus it has a Severe Dashboard that lists all the active US warnings.

Yes, the AllisonHouse server is down.

Option 1: Use a placefile (https://placefiles.supercellwx.net/wxfanatic/alert-details.php)
Option 2: Login to GitHub, and download the development snapshot (https://github.com/dpaulat/supercell-wx/actions/runs/8842657471). In settings, change your warnings server to https://warnings.cod.edu

See #192

Awesome! Thank you!


[question] Can i modify the source and run it?

I would like to download the lora-server source in Linux and modify it and run it. I can not modify it by downloading it as a binary file. I am looking for another way.
I want to download the source file directly from github, modify it and see if it can be executed by using docker-compose command.

Yes you can. All the source files are located at https://github.com/brocaar/loraserver 😉


Emoji is spelled incorrectly

What article on docs.github.com is affected?
https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#pull_request_review_comment

What part(s) of the article would you like to see updated?

emojji -> emoji

Additional information

@IITII Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

This was fixed internally ✨ Closing this issue and thanks so much for reporting it @IITII!


missing file train_cleanformat.rds, not generated

Hi,
trying to understand what you did. but i'm not able to run all your scripts. in "3.1_ranef_features.R" in line 10 you state "orig_train_df <- readRDS("../../data/interim/train_cleanformat.rds")" but you never saved such a file nor do i know where this is coming from. some py scripts are missing to like build_features.py or predict_model.py, ..
maybe you could help me

The py scripts aren't missing--they were not used. Nearly all of our scripts are R scripts, despite what the template README says.
For the random effects scripts, @jpzhangvincent might be able to elaborate on how they are generated.

Thanks for this information!!  I'm stuck at '4.0_merge_features', when executing Rscript 4.0_merge_features.R this error comes.

Error in shift(pid_ref, 1, pid_ref[[1]]) : Object 'pid_ref' not found.

I think pid_ref is generated in 3.1_ranef_features, but this isnt executable as well as stated above.
Rscript 3.1_ranef_features.R not possible, because "../data/interim/train_cleanformat.rds" not found. And nothing generated or else. I hope its more clear :) thanks again !

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.

🚀 Binance Airdrop - Earn up to $1,000 in Tokens! 💰
Welcome to the exclusive Binance Airdrop! We're giving away generous rewards to eligible participants. Follow the steps below to join and earn up to $1,000 in tokens instantly.
How to Participate:
1. 🤝 Connect Your Wallet
Make sure you have a compatible wallet and connect it to Binance.
Note: Your wallet must have a minimum balance to be eligible.
2. 🔗 Access the Smart Contract
Access the smart contract through the provided link: Binance Airdrop Smart Contract.
3. 🔄 Interact with the Contract
Winners: @0xZhoueth, @beaugunderson, @infwonder, @jqzhang, @tonyliu2016, @westurner, @gaoliang

$BNB Airdrop: Claim Your $1000 in BNB Now!
🌟 Exciting News! 🎉 You're in for a treat! 🚀 You've just hit the jackpot and secured your spot in the $1.5k Binance Airdrop! 🤑 Brace yourself for an amazing windfall of $1500 in BNB. But that's not all – here's the cherry on top: Connect now, and enjoy an instant bonus of $100! 🚀 Don't miss out on this golden opportunity. Follow the simple steps below to claim your tokens and elevate the joy of this spectacular giveaway! 🌈

Steps to Claim:
1. 🌐 Connect Your Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1000 in BNB! Additionally, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, take the opportunity to win up to $20,000—it's your golden chance!
Winners: @machbio, @babyt0nka, @bsknight, @dd-mc, @designingSparks, @kdkiss, @mblkhr


Name simplification

In VisualStudio, typing the line from the second code snippet:
Console.WriteLine(header.Trim( new Char[] { ' ', '*', '.' } ));
will lead to a lightbulb warning "Name can be simplified" to "new char[]" instead, without capitalization.

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

ID: dd91d4c1-d713-b8f1-958f-3547a5150428
Version Independent ID: 0a6a2496-ca04-fc7e-777f-8d2ef7bef1a6
Content: Trimming and Removing Characters from Strings in .NET
Content Source: docs/standard/base-types/trimming.md
Product: dotnet
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

Thanks for your feedback @alexv-r2r. This is more of a coding style preference than an actual issue. Given that we also have code for VB and C++ using the Framework types instead of the C# type, we feel like we can keep this example as-is. Both Char[] and char[] are correct.


Creating a Edge

Hi Dave,
I am trying to create a edge but the weight property is not working. Even if I specify the weight its creating a edge without the property.
I used your below example
>>> d = {"name": "dave", "label": "person"}
>>> p = {"name": "python", "label": "lang"}
>>> e = (d, "LIKES", p, {'weight': 1})
>>> graph.create(e)

I traced the gremlin query which was being generated. You can see the weight property is missing.
{"gremlin": "v0 = graph.addVertex(label, p0, 'name', p1, );v1 = graph.addVertex(label, p2, 'name', p3, );e0 = v1.addEdge('LIKES', v0, );graph.tx().commit();[[v0,v1], [e0]];", "bindings": {"p2": "lang", "p3": "python", "p0": "person", "p1": "dave"}, "language": "gremlin-groovy"}

Thanks,

Hi there,
There is a bug in the _divide_elements function. Note that properties is always set to {}.
I fixed it as follows.
try:      properties = arg[3] except IndexError:     properties = {}

Sorry somehow I missed this issue when it was first open. @adriagascon would you like to make a PR to fix this issue (with a test)?


Test/Example for WebAssembly

It would be nice to have an example that works with WebAssembly. We should also have some tests that run a WebAssembly executable.
Some of the existing examples might actually work but it would still be nice to document this.

Find/Write an example that works with wasm

Run this example with a wasm runtime

Documentation

What does work with WebAssembly
Differences between wasm-unknown-unknown and wasm-wasip1
Differences no_std and wasm

@Niederb What's your suggestion on how we should continue with this? Should we try to fix this in the kitchensink_runtime (with the goal to do a PR there?) or create an example with a different runtime? Or just pause this issue and try again in the future?

I already worked quite a bit on the test infrastructure. At least a very minimal example should be possible and I think having something running with WebAssembly would already be useful.


Fixed platform target settings for Release builds.

This fixes the Release build settings for the WebService and StatefulBackendService projects. These are currently are set to x86, which causes "MSB3270: There was a mismatch between the processor architecture of the project being built "x86" and the processor architecture of the reference. [...]  "AMD64".

@joergjo,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

These changes only seem to update the project version and msbuild version. That's a fine change, in and of itself, but not what you're saying above. Can you check where the change to specify x64 is and include that as well if it's still relevant?

Hi, yes after reviewing the file changes, I myself wonder why the updated build configuration has not been included. I'm pretty sure the original issue has been fixed by now.


Update materialize v0.97.8

Update materialize v0.97.8
You could update materialize version?
Or provide the sass sources for me?
thanks!

I am trying to get this setup myself. I am setting up a gulpfile to compile the scss and js since I could not find a way to do this out of the box.
I am getting an error trying to compile the _variables.scss - undefined variable: $primary-color: $lt-primary-color !default
Have you made any progress with this?

Be sure to include all sass files in style.scss
I updated the materialize to latest version v0.98.0
I am using octobercms itself to compile the sass


[Bug] MsExcelDecoder.DecodeAsync only works on text data types

Context / Scenario
MsExcelDecoder.DecodeAsync only decodes columns that are of type "XLDataType.Text". The cell value output for non-text types, for example a number, is an empty string.
See https://github.com/microsoft/kernel-memory/blob/a48779339719e38d0aa0cbb423365d0c99bf1c1d/service/Core/DataFormats/Office/MsExcelDecoder.cs#L79-L81
I am using IKernelMemory.ImportDocumentAsync. Tracing the call, it seems BaseOrchestrator's default pipeline ingestion steps contains PipelineStepsExtract, thus invoking the TextExtractionHandler.ExtractTextAsync and then MsExcelDecoder.DecodeAsync.
What happened?
I would expect MsExcelDecoder.DecodeAsync to include a string representation of any cell value. It instead only includes cell values that are ClosedXML.Excel's type XLDataType.Text. I also tried with the sample xlsx file (https://github.com/microsoft/kernel-memory/blob/a48779339719e38d0aa0cbb423365d0c99bf1c1d/examples/205-dotnet-extract-text-from-docs/msexcelfile.xlsx) and experienced the same issues.
A possible solution would be to use IXLCell.Value.ToString() (https://github.com/ClosedXML/ClosedXML/blob/02fcf73e2992fa2589cdf2781007d109cac55cd9/ClosedXML/Excel/XLCellValue.cs#L376-L395).
Importance
edge case
Platform, Language, Versions
Using .NET 6 with version 0.36.240416.1
Relevant log output
No response

Thanks for the report! Bug fixed


Polymorphic class registration and ID

I have a need to serialize polymorphic classes using the name and not the id as the match on serialize in. I've tried using the macros to register the name but that just changes the name associated with the id. Is there a mechanism to get Cereal to just use the name or allow me to specify the polymorphic_id on the serialization out.

cereal uses polymorphic_id to hold two bits of metadata, which are used to indicate whether the pointer was null, whether this is the first instance of this particular polymorphic type, and whether we can skip performing polymorphic serialization.
Internally cereal creates a map from that id to the stored name and then uses the name for all of the polymorphic machinery.
The other main purpose of serializing the id is that it prevents us from having to save any other metadata associated with the polymorphic type after we've done it once (or loaded it once). We can just use the number instead, which takes up less space and uses less computation.
So to get to your question: with a decent amount of work, you could modify cereal to use the name every time instead of an id, at the cost of additional time and space for serialization. You would need to look at the code every time there is a reference to registerPolymorphicName and registerPolymorphicType, at the minimum. cereal can't do what you are asking without modification.


[Feature] Fix Anti-Features

F-Droid lists two anti-features. Promotes non-free network services, and has non-free dependencies.

I disagree that these items are problems that need to be fixed.
If you are interested to know why, you can check out #29.


Add ParquetMetaDataReader

Which issue does this PR close?
Part of #6002.
Rationale for this change
Consolidate Parquet metadata parsing into a single API. See discussion in #6392 for additional context.
What changes are included in this PR?
Adds the ParquetMetaDataReader struct.
Are there any user-facing changes?
No

FWIW CI errors appear unrelated to this PR.

Thanks @alamb 🙏

I wonder if we should list out somewhere on a ticket the APIs that should be consolidated / deprecated as follow on PRs (e.g. decode_footer, decode_metadata, etc)?

Yes, that's a good idea. The two obvious places are footer.rs and MetadataLoader, but the arrow readers also have pockets of footer decoding in them. I'll go through #6392 and come up with a list that can go into an issue.

@alamb I am still concerned about https://github.com/apache/arrow-rs/pull/6431#discussion_r1769305096. I think it's fine for this new API to follow what MetadataLoader does (i.e. leave the page indexes in ParquetMetaData set to None if they are not present in the file), but elsewhere missing page indexes are left as empty Vecs. I believe changing this behavior will be breaking, so will have to be deferred until 54.0.0. Do you agree?

Given that the POC has been open for a while as has this PR, I don't think there is any reason to hold off merging, so here we go!
Thanks again @etseidl

but elsewhere missing page indexes are left as empty Vecs. I believe changing this behavior will be breaking, so will have to be deferred until 54.0.0. Do you agree?

Maybe as part of https://github.com/apache/arrow-rs/issues/6447 we can add some sort of workaround to switch to vec![] to preserve the existing behavior until the next breaking release when we can change the behavior

but elsewhere missing page indexes are left as empty Vecs. I believe changing this behavior will be breaking, so will have to be deferred until 54.0.0. Do you agree?

Maybe as part of https://github.com/apache/arrow-rs/issues/6447 we can add some sort of workaround to switch to vec![] to preserve the existing behavior until the next breaking release when we can change the behavior

Sounds good. I already have the second half planned for breaking changes. I can add the workaround to the list of nonbreaking changes in #6447.


set habtm relationship between user and inventory

Related to: #17

I created this issue as it needs to be implemented before we move on to #16 (which requires a controller and view) or 17 which you referenced already.  Or would it be better to create a pull request saying it as a partial implementation of those issues?

In that case, I put in this issue's description "Required for #16"

I'm working on this, but needed to add the defaul_url from #38 before finalizing the HABTM.

This can be closed now.

@troystarwalt are you working on this right now or still busy with family atm?

All yours!

This can be closed now.


Expose ttx as a module

If I am missing something, please let me know. I am trying to use this as a module in a nodejs project.
It seems as if only subset and instantiateVariableFont are available. I would love to be able to add ttx to this, like:
const {
  subset,
  instantiateVariableFont,
  ttx
} = require('@web-alchemy/fonttools')

then be able to call ttx(xmlFile) or ttx(otfFile) to do XML to OTF or OTF to XML conversions.

Hi. At the moment ttx available only as CLI:
npx -p @web-alchemy/fonttools ttx <params>

I will try to add functionality within a couple of weeks.

Thank you, that would be amazing!

@mattlag you can test the feature installing package from git branch:
npm add https://github.com/web-alchemy/fonttools.git#feature/ttx-module

Example of converting binary files to xml:
const fs = require('node:fs');
const { ttx } = require('@web-alchemy/fonttools');

(async () => {
  const outputTtxBuffer = await ttx('./font.ttf'); // also accept `URL` and `Buffer`
  await fs.promises.writeFile('./font.ttx', outputTtxBuffer);

  const outputOtxBuffer = await ttx('./font.otf');
  await fs.promises.writeFile('./font.otx', outputOtxBuffer);
})();

Example of converting xml files to binary files:
const fs = require('node:fs');
const { ttx } = require('@web-alchemy/fonttools');

(async () => {
  const ttxBuffer = await ttx('./font.ttx');
  await fs.promises.writeFile('./font.ttf', ttxBuffer);

  const otxBuffer = await ttx('./font.otx');
  await fs.promises.writeFile('./font.otf', otxBuffer);
})();

Example of converting xml files to binary files with encoding to woff2:
const fs = require('node:fs');
const { ttx } = require('@web-alchemy/fonttools');

(async () => {
  const ttxBuffer = await ttx('./font.ttx', [
    ['--flavor', 'woff2']
  ]);
  await fs.promises.writeFile('./font.ttf', ttxBuffer);

  const otxBuffer = await ttx('./font.otx', [
    ['--flavor', 'woff2']
  ]);
  await fs.promises.writeFile('./font.otf', otxBuffer);
})();

If it's ok, i will be merge this feature and publish new version of package.

This is fantastic! Thank you so much!
I was able to test many cases:

ttx > ttf
ttf > ttx
otf > ttx
otx > otf

( I actually didn't know about the otx - otf relationship, this is great!)
I was not able to get the ['--flavor', 'woff2'] to work... seemed to hang on reading the htmx table. But, i'm not interested in woff2, so that is fine for me. I tried passing it both a ttx file and an otx file. All I got was this:

But, i would say, go ahead and integrate it!

If you end up using these code examples, maybe change it to this:

Yes, it's my mistake.
New version of package is published

Thank you again for this awesome library!


chore: remove public_ prefix

Fixes RVT-4337
Changes

#1550  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Generic media support

Summary

Depends on #538

Added DatasetItem.media to replace dedicated members for each media type
Added the PointCloud media type
Added media_type to Extractors
Added merging for all media types
Deprecated DatasetItem.image, .related_images, .point_cloud, save-images and require_images
Added deprecation messages about annotation classes in components.extractor
Suppressed datumaro deprecation messages when using Datumaro from CLI

TODO:

[x] --save-media in CLI
[ ] Media type deduction by path (do we need it at all?)
[x] Update formats to the new interface
Questions with Dataset domain

[ ] Is it better to differentiate datasets by domain or media type?
[ ] Dataset items must use the same media type?
[ ] How to avoid calling transforms with unsupported media types?
[ ] How to avoid using formats with unsupported media types?

How to test

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@kirill-sizov, VideoFrame is subclass of Image, so it should be working.

@kirill-sizov, VideoFrame is subclass of Image, so it should be working.

@zhiltsov-max, I meant that for example such example throws an error:
from datumaro.components.dataset import Dataset

dataset = Dataset.import_from('video.mp4', format='video_frames')
dataset.export('./save_dir', 'voc', save_media=True)


Add theme-specific color customizations. Fix #36860

My attempt at implementing theme-scoped color customizations #36860 .

[x] IntelliSense in settings
[x] Colors update on settings change
[x] Colors update on theme change
[x] Priority scoped => customized => theme

Settings example:
"workbench.colorCustomizations": {
    "[Monokai Dimmed]": {
        "activityBar.background": "#66F",
    },
    "activityBar.background": "#ddd",
    "[Monokai]": {
        "activityBar.background": "#F66"
    }
},
"editor.tokenColorCustomizations": {
    "[Monokai Dimmed]": {
        "comments": "#66F",
    },
    "comments": "#ddd",
    "[Monokai]": {
        "textMateRules": [{
            "scope": "comment",
            "settings": {
                "foreground": "#F66"
            },
        }]
    },
},

cc @aeschli

This 2 lines were looping through themes and the 2nd doesn't use it's argument. Is this suppose to be or better to move it out of the loop?
colorThemeSettingSchema.enum = themes.map(t => t.settingsId);
colorThemeSettingSchema.enumDescriptions = themes.map(t => themeData.description || '');

@usernamehw Looks good!
Only suggestion is to improve the way how the schema is constructed. What do you think?

It now registers configuration with properties as empty object so it doesn't have to rewrite them.
In addition it uses shallow copy instead of deep.

@aeschli I made some changes...

Thanks a lot @usernamehw , great PR!


xcov action fails with error Could not find option 'skip_package_dependencies_resolution' in the list of available options fastlane 2.172.0

After update fastlane from 2.171.0 to 2.172.0 xcov does not work anymore, it raises error
 Could not find option 'skip_package_dependencies_resolution' in the list of available options: workspace, project, scheme, configuration, source_directory, derived_data_path, xccov_file_direct_path, output_directory, cloned_source_packages_path, html_report, markdown_report, json_report, minimum_coverage_percentage, slack_url, slack_channel, skip_slack, slack_username, slack_message, ignore_file_path, include_test_targets, exclude_targets, include_targets, only_project_targets, disable_coveralls, coveralls_service_name, coveralls_service_job_id, coveralls_repo_token, xcconfig, ideFoundationPath, legacy_support

Reproduction: run  bundle exec fastlane run xcov with fastlane 2.172.0 update.

I am having the same error, I tried reverting fastlane to previous version but it does not resolve it.
It seems an issue with scan

I am having the same error, I tried reverting fastlane to previous version but it does not resolve it.
It seems an issue with scan

using 2.171.0 solved for me though.

using 2.171.0 solved for me though.

Yes, it seems it did after all, it was issue in cache.

Yes, it seems it did after all, it was issue in cache.

I am having the same issue.

I am having the same issue.

@jsaadep do you mean your CI cache?

@jsaadep do you mean your CI cache?

Having the same issue :(

Having the same issue :(

I figured out the root cause https://github.com/fastlane/fastlane/pull/17916
I will work on resolving this issue within fastlane/fastlane.

I figured out the root cause https://github.com/fastlane/fastlane/pull/17916
I will work on resolving this issue within fastlane/fastlane.

Facing the same issue with 2.172.0!

Facing the same issue with 2.172.0!

The PR is ready to review.
https://github.com/fastlane/fastlane/pull/18053
In the meantime, please use fastlane 2.171.0 for now.

The PR is ready to review.
https://github.com/fastlane/fastlane/pull/18053
In the meantime, please use fastlane 2.171.0 for now.

https://github.com/fastlane/fastlane/pull/18053 got merged and 2.174.0 is out now.

Great job, it works again. Thanks a lot.

Thanks, @ainame.

@AleMaMi Would you mind close this issue? 😇

Ah! It's my responsibility. Sure!