[C#] _ExitTree is not called at the main scene node after QueueFree

Godot version
4.0.beta4
System information
Windows 10, 64bit
Issue description
After changing a scene with the following code, the _ExitTree method is not called under C#. In GD-Script, on the other hand, it is. However, Dispose is called in C#.
_ExitTree is called on all nodes contained in the scene except on the root node. (I don't mean the tree node "/root")
C#:
public static void SwitchScene(string newScenePath)
{
    var tree = (SceneTree)Engine.GetMainLoop();
    Viewport root = tree.Root;
    using Node CurrentScene = root.GetChild(root.GetChildCount() - 1);

    CurrentScene.QueueFree();

    Node newCurrentScene;
    using (var nextScene = GD.Load(newScenePath) as PackedScene)
    {
        newCurrentScene = nextScene?.Instantiate();
    }

    if (!IsInstanceValid(newCurrentScene))
    {
        GD.PushError("[SceneManager] Scene instance not valid.");
        return;
    }

    tree.Root.AddChild(newCurrentScene);
    tree.CurrentScene = newCurrentScene;
}

GD-Script:
func switch_scene(new_scene_path : String) -> void:
	var tree : SceneTree = Engine.get_main_loop()
	var root : Viewport = tree.root
	var current_scene : Node = root.get_child(root.get_child_count() - 1)

	current_scene.queue_free()

	var new_scene : Node = load(new_scene_path).instantiate()

	if !is_instance_valid(new_scene):
		push_error("[SceneManager] Scene instance not valid.")

	tree.root.add_child(new_scene)
	tree.current_scene = new_scene

Edit: Works with the code normally using for changing the scene:
var tree = (SceneTree)Engine.GetMainLoop();
tree.ChangeSceneToFile(newScenePath);

And works on 3.5: Here _ExitTree is called as expected.
Steps to reproduce
After starting the test scene, click once on the "Switch scene to scene 2" button and then on the "Switch scene to scene 1" button. Then the following output is generated:
under C#:
[Scene1] : _EnterTree
[Scene1] : _Ready
[Scene2] : _EnterTree
[Scene2] : _Ready
[Scene1] : Dispose
[Scene1] : _EnterTree
[Scene1] : _Ready
[Scene2] : Dispose

under gd-script:
[Scene1] _enter_tree
[Scene1] _ready
[Scene2] _enter_tree
[Scene2] _ready
[Scene1] _exit_tree
[Scene1] _enter_tree
[Scene1] _ready
[Scene2] _exit_tree

Minimal reproduction project
Godot4B4_GD_Test_ExitTree.zip
Godot4B4_CS_Test_ExitTree.zip

using Node CurrentScene = root.GetChild(root.GetChildCount() - 1);

You are using using statement so the csharp instance had been disposed before the node exited tree (QueueFree will defer deletion to the end of MessageQueue).
You can remove using statement, or remove child manually:
using Node CurrentScene = root.GetChild(root.GetChildCount() - 1);

root.RemoveChild(CurrentScene); // will call _ExitTree
CurrentScene.QueueFree();

Thank you for your explanation. I had a misconception here.
Of course it works great without the using.
Can be closed, sorry.


The problem about the  feature matching

Hi. I downloaded your code and paper.
As you paper says: “Siamese-based feature extraction and matching”.How to do feature extraction and matching in paper? Can you explain it in detail?
Look forward to your replay. Thx.

It's same to most Siamese tracker, eg. SiamRPN++, SiamBan SiamFC++...


Windows icon broken

It doesn't show up as the icon on windows for some reason

Now it's back. What?

Maybe it only happens when you open Qtcord in the proper directory

Seems to work fine with wine. 🤷🏻♂


Allow to await for the dialog show

Problem statement
I use the progress dialog to keep the user informed of a blocking operation.
So the progress dialog must be showed and hidden after an operation, but if the operation is async but terminates very quickly, the dialog is not hidden, because the showDialog method (used internally in the .show()) does not have finished showing the dialog.
Describe the solution you'd like
I would like to be allow to await the .show() method, like this
ProgressDialog dialog = ...;
await dialog.show();

// do some work ...

await dialog.hide();

good idea, but  call funciton require async.

good idea, but call funciton require async.

Yeah sure, it is the main point of doing it.
Obviously you could also use the .then() method and chaining the Futures, but I think it is more readable this way.

Until this issue is resolved, you can use
await Future.delayed(Duration(seconds: 1));
for allowing the dialog to pop up

Could we get a new release with the fixed version?
By the way (for the impatient), a workaround until then:
pubspec.yaml
dependencies:
  progress_dialog:
    git: https://github.com/fayaz07/progress_dialog.git

Update v1.2.1 is live, please checkout


go-jsonnet: New package v0.13.0

Motivation for this change
The go version of jsonnet is faster at compiling. It's nice to have it as an option.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] 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)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

You also need to add all meta (license, description, ...) information.

Thanks for the review. I've updated this PR accordingly.

Would you like to become the maintainer of this package? In this case, you could add yourself in the maintainer list.

Only if I have to...

It's not mandatory for a package to have a maintainer but it's really better! If you are the maintainer, it is easier for you to do upgrades because we merge more quickly upgrades from maintainers. You are also notified on upgrades.
So, it's up to you ;)

I'm going to wait for upstream to cut a new release and point to that tag. I'll add myself as a maintainer at the same time.

@nlewo Upstream has tagged a new release, and I've updated this PR accordingly. Please review again when you have a moment.

@GrahamcOfBorg build go-jsonnet

Thx!


Support for numpy<2?

I'm really excited about this project, but it's currently pretty difficult to install in any non-minimal environment because Numpy 2.0 is so recent. How essential is your use of 2.0 features? Is there any chance of this being relaxed?

Hey! We're using numpy 2.0 for the new variable-length string dtype, as a good way to efficiently store arrays of paths to files.
Context here:
https://github.com/zarr-developers/VirtualiZarr/issues/33
https://github.com/zarr-developers/VirtualiZarr/pull/107#discussion_r1637241715
If you really want backwards-compatibility we can accept a PR that generalizes this, but its always going to be less efficient to use a fixed-length string dtype.

Thanks a lot for the explanation! Relying on a new dtype seems like fairly essential use of 2.0, and it'd probably be a significant amount of work to maintain a parallel backwards-compatible implementation.
I tracked down one of my current bottlenecks which is xagg→pytables→numpy<2, and happily that's being actively worked on :rocket:!
I think my current strategy will be to build a separate container for the kerchunking step of my pipeline so that I can make use of VirtualiZarr until numpy 2 is widely supported.
Thanks so much for the quick response, it's really helpful!

Sounds good! Let us know how you get on using VirtualiZarr.

With the PyTables release just now, there are no more dependency constraints blocking me from upgrading to NumPy 2 and using VirtualiZarr in my main environment. :rocket:


Vue runtime-only build

First off, thanks for this amazing project!
I'm working through the demos within my existing app. When I use the TodoItemNode, maybe others I'm getting:
[Vue warn]: You are using the runtime-only build of Vue where the template compiler is not available. Either pre-compile the templates into render functions, or use the compiler-included build.

Is the runtime-only build not supported?
Did I miss in the setup or docs where it talks about the Vue build type?
Thanks!

I'm getting a little farther. Will have to pick back up in the morning. This is what I have so far to extend.
import { TodoItemNode as TipTapItemNode } from 'tiptap-extensions'

export default class TodoItemNode extends TipTapItemNode {
  get view () {
    return {
      props: ['node', 'updateAttrs', 'editable'],
      methods: {
        onChange () {
          this.updateAttrs({
            done: !this.node.attrs.done,
          })
        },
      },
      render (h) {
        h(
          'li',
          {
            attrs: {
              'data-type': 'todo_item',
              'data-done': this.node.attrs.done.toString(),
            },
          },
          [
            h('span', {
              staticClass: 'todo-checkbox',
              attrs: {
                contenteditable: 'false',
              },
              on: {
                click: this.onChange,
              },
            }),
            h('div', {
              ref: 'content',
              staticClass: 'todo-content',
              attrs: {
                contenteditable: this.editable.toString(),
              },
            }),
          ],
        )
      },
    }
  }
}

When I click the button, it removes all of my highlighted test and has the following errors in the console:
[Vue warn]: Error in event handler for "click": "TypeError: node.getBoundingClientRect is not a function"
found in
...

TypeError: node.getBoundingClientRect is not a function
    at coordsAtPos (index.js?703f:345)
    at EditorView.coordsAtPos$1 [as coordsAtPos] (index.js?703f:4391)
    at EditorView.updateState (index.js?703f:4290)
    at VueComponent.dispatchTransaction (tiptap.esm.js?cd42:861)
    at EditorView.dispatch (index.js?703f:4484)
    at eval (schema-list.js?7a87:94)
    at eval (tiptap.esm.js?cd42:464)
    at Array.forEach (<anonymous>)
    at Object.assign.obj.(anonymous function) (webpack-internal:///./node_modules/tiptap/dist/tiptap.esm.js:488:15)
    at VueComponent.invoker (vue.runtime.esm.js?2b0e:2023)
logError @ vue.runtime.esm.js?2b0e:1737
globalHandleError @ vue.runtime.esm.js?2b0e:1728
handleError @ vue.runtime.esm.js?2b0e:1717
Vue.$emit @ vue.runtime.esm.js?2b0e:2536
e.(anonymous function) @ backend.js:1
trigger @ QBtn.js?c7ec:73
click @ QBtn.js?c7ec:82
invoker @ vue.runtime.esm.js?2b0e:2023
fn._withTask.fn._withTask @ vue.runtime.esm.js?2b0e:1822

Hey, yeah… runtime-only build is not supported at the moment because on the TodoItemNode. I like templates 🙃. But as you already found out: you are free to use your custom nodes for that. In your example you did just a small mistake. you have to return h() in your render function.

How do I solve this issue if I build on top of vue-cli? Running vue-cli-service build then serve dist doesn't solve the issue TypeError: i.getBoundingClientRect is not a function

For anyone else hitting this who is using vue-cli. You can enable the runtime compiler in Vue with the following option in your vue.config.js file.
module.exports = { runtimeCompiler: true, }

@philippkuehn Please consider re-opening this. The runtime-only build is the only CSP-compliant Vue build. It will become a major blocker as proper CSP headers get more and more popular (and are already pretty much a must-have for large scale or enterprise apps)... 😕
Or is a custom node the "official" workaround?

Linked to pull request #576. I know this is closed, but this is highly relevant to anybody looking to have a CSP compliant app, and it would be pretty silly to avoid it when the solution is so simple :)

@philippkuehn what about #576?


To Do list: list structure review

Used webpack to bundle JavaScript.
Created an index.html file and set it as a template using the HTML Webpack Plugin.
Created index.js and set an array of objects for the to-do tasks.
Created a function to iterate over the tasks array.

Adjusted linter errors


no function clause matching in Veil.verify/2

Hi.
I followed veil's official instruction an this to adding it in my project but for verification section it throws this error
no function clause matching in Veil.verify/2
an I tried create a new phoenix project for testing veil and I do it without any additional work! but I gave this error another time
so this is phoenix error page details:
%Plug.Conn{adapter: {Plug.Cowboy.Conn, :...}, assigns: %{}, before_send: [#Function<0.41104819/1 in Plug.CSRFProtection.call/2>, #Function<2.64052486/1 in Phoenix.Controller.fetch_flash/2>, #Function<0.54074652/1 in Plug.Session.before_send/2>, #Function<0.110189767/1 in Plug.Telemetry.call/2>, #Function<0.126136900/1 in Phoenix.LiveReloader.before_send_inject_reloader/2>], body_params: %{}, cookies: %{"_project_key" => "SFMyNTY.g3QAAAABbQAAAAtfY3NyZl90b2tlbm0AAAAYMDZ5NktPU0l6NUxFRGdVX0J1TmxuZzd1.Ka1I86vHSWu3oi2ZW9mvUO1KaLzwsuwKqp4l6pQcDOc", "session_unique_id" => "OTHCZF3R6PCBTTKTKOKQOSKQJPCXHMHCI53TUQVLP5BUIJ42GF7Q===="}, halted: false, host: "localhost", method: "GET", owner: #PID<0.32758.0>, params: %{"request_id" => "56VOVSQX4TCBUHPI45GVWOTWVKXUVRPB4QPQ5OMPLXYMMUKXUIZQ===="}, path_info: ["veil", "sessions", "new", "56VOVSQX4TCBUHPI45GVWOTWVKXUVRPB4QPQ5OMPLXYMMUKXUIZQ%3D%3D%3D%3D"], path_params: %{"request_id" => "56VOVSQX4TCBUHPI45GVWOTWVKXUVRPB4QPQ5OMPLXYMMUKXUIZQ===="}, port: 4000, private: %{ProjectWeb.Router => {[], %{}}, :phoenix_endpoint => ProjectWeb.Endpoint, :phoenix_flash => %{}, :phoenix_format => "html", :phoenix_router => ProjectWeb.Router, :plug_session => %{"_csrf_token" => "06y6KOSIz5LEDgU_BuNlng7u"}, :plug_session_fetch => :done}, query_params: %{}, query_string: "", remote_ip: {127, 0, 0, 1}, req_cookies: %{"_project_key" => "SFMyNTY.g3QAAAABbQAAAAtfY3NyZl90b2tlbm0AAAAYMDZ5NktPU0l6NUxFRGdVX0J1TmxuZzd1.Ka1I86vHSWu3oi2ZW9mvUO1KaLzwsuwKqp4l6pQcDOc", "session_unique_id" => "OTHCZF3R6PCBTTKTKOKQOSKQJPCXHMHCI53TUQVLP5BUIJ42GF7Q===="}, req_headers: [{"accept", "text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3"}, {"accept-encoding", "gzip, deflate, br"}, {"accept-language", "en-US,en;q=0.9"}, {"connection", "keep-alive"}, {"cookie", "_project_key=SFMyNTY.g3QAAAABbQAAAAtfY3NyZl90b2tlbm0AAAAYMDZ5NktPU0l6NUxFRGdVX0J1TmxuZzd1.Ka1I86vHSWu3oi2ZW9mvUO1KaLzwsuwKqp4l6pQcDOc; session_unique_id=OTHCZF3R6PCBTTKTKOKQOSKQJPCXHMHCI53TUQVLP5BUIJ42GF7Q===="}, {"host", "localhost:4000"}, {"sec-fetch-mode", "navigate"}, {"sec-fetch-site", "cross-site"}, {"sec-fetch-user", "?1"}, {"upgrade-insecure-requests", "1"}, {"user-agent", "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/76.0.3809.87 Safari/537.36"}], request_path: "/veil/sessions/new/56VOVSQX4TCBUHPI45GVWOTWVKXUVRPB4QPQ5OMPLXYMMUKXUIZQ%3D%3D%3D%3D", resp_body: nil, resp_cookies: %{}, resp_headers: [{"cache-control", "max-age=0, private, must-revalidate"}, {"x-request-id", "Fbl--3GwlqVY6UwAAAYi"}, {"x-frame-options", "SAMEORIGIN"}, {"x-xss-protection", "1; mode=block"}, {"x-content-type-options", "nosniff"}, {"x-download-options", "noopen"}, {"x-permitted-cross-domain-policies", "none"}, {"cross-origin-window-policy", "deny"}], scheme: :http, script_name: [], secret_key_base: :..., state: :unset, status: nil} {:ok, %Project.Veil.Session{__meta__: #Ecto.Schema.Metadata<:loaded, "veil_sessions">, id: 3, inserted_at: ~N[2019-08-10 07:25:18], ip_address: "<IP_ADDRESS>", phoenix_token: "SFMyNTY.g3QAAAACZAAEZGF0YWECZAAGc2lnbmVkbgYAkddqemwB.PV8Kcnq8qhFBsoyv2Cjuq68O4Dz7TSw9-HEmtagJKkc", unique_id: "OTHCZF3R6PCBTTKTKOKQOSKQJPCXHMHCI53TUQVLP5BUIJ42GF7Q====", updated_at: ~N[2019-08-10 07:25:18], user: #Ecto.Association.NotLoaded<association :user is not loaded>, user_id: 2}}

what should I do?
thanks

@thantez I just ran into this, I think something changed with Veil.get_session/1 in an update. Here's a quick workaround:
Open Veil.UserId and change line 14 from {:ok, session} <- Veil.get_session(session_unique_id) to {:ok, {:ok, session}} <- Veil.get_session(session_unique_id).

@thantez I just ran into this, I think something changed with Veil.get_session/1 in an update. Here's a quick workaround:
Open Veil.UserId and change line 14 from {:ok, session} <- Veil.get_session(session_unique_id) to {:ok, {:ok, session}} <- Veil.get_session(session_unique_id).
In looking closer at the error message, I noticed that the 2nd arg is a tuple instead of a %Rodinia.Veil.Session{} or %Rodinia.Veil.Request{} struct.

thanks. worked for me.


[Packaging] Fix #22278: Build RPM for Fedora

Description

Fix #22278. Current RPM builds are not fully compatible with Fedora, and some issues have been created such as #22235.
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Packaging


Fix package name of generated Res file when project is building for JsTarget (#4295)

Fix package name of generated Res file when project is building for JsTarget (#4295)

Thank you for the contribution. You are supposed to provide tests cover the issue

@terrakok Thanks for review. I have added a test case that verifies the issue is fixed. Let me know if it works or if I need to add some more test cases

Reworked tests: https://github.com/JetBrains/compose-multiplatform/pull/4300


Make ops-files and vars-files inputs optional

What is this change about?
Make ops-files and vars-files inputs optional

They are not used if the pipeline user is using default cf-deployment setup
This causes users to add dummy / unused resources and adds pipeline cruft

Please provide contextual information.
https://vmware.slack.com/archives/C01ERQAVD2L/p1631131092013600
Please check all that apply for this PR:

[ ] introduces a new task
[x] changes an existing task
[ ] changes the Dockerfile
[ ] introduces a breaking change (other users will need to make manual changes when this is released)

Did you update the README as appropriate for this change?

[ ] YES
[x] N/A

How should this change be described in release notes?
Something brief that conveys the change and is written with the component author audience in mind.
What is the level of urgency for publishing this change?

[ ] Urgent - unblocks current or future work
[x] Slightly Less than Urgent

Tag your pair, your PM, and/or team!
@cloudfoundry/runtime

Hi @rroberts2222,
Could you please sign the CLA? You will need to close and reopen the PR to get the check to rerun.
Thanks,
Dave


Feature: On Off Commands for Switch

Would it be possible to get a custom command to be run when turning off the switch?
like say i want one switch for bluetooth so i can turn it on and off whit the same switch instead of one button for on and another for off
If its already integrated and i just cant find it, im sorry for wasting youre time

Hm not directly, you could create an automation in HA that runs the command when the switch turns off.


Splash screens on iOS aren't put in the proper directory

The splash screens are generated, but not stored in platforms/ios/[App]/Resources/splash. Or am I missing something here?

You just have to configure the paths for the images in the 'config.xml' as stated in the readme
Configuring Splash


Changing of device class after initial setup is ignored

Is this a bug or a feature request?
Bug
What did you expect?
I created a device with class-a and class-c, which only supports class-a. After discovering my error, I disabled class-c. Hence, downlink messages should be queued and only be delivered later.
What happened?
The downlink message was signed with delivery "Immediately", therefore Loraserver tried to deliver the message without waiting for the next uplink.
What version are your using?
3.0.?
How can your issue be reproduced?

create a device with class-a and class-c
after setting it up, disable class-c
send a downlink to the device. It will be delivered immediately, instead of waiting for the next slot.

Could you share your log output?
loraserver_1  | 2019-06-28T11:40:06.872807892Z time="2019-06-28T11:40:06Z" level=warning msg="device-queue item discarded due to timeout" dev_eui=[...] device_queue_item_fcnt=12

After changing the device-profile, you might need to re-activate the device (OTAA or ABP). This is needed as the Device Profile contains the boot parameters of the device, but these might change over time (and are stored in the device-profile). E.g. LoRa Server might decide to re-configure the device to use different parameters. Also, the device could have indicated that it changed from Class-A to Class-C using mac-commands.
I've added this note: https://github.com/brocaar/lora-app-server/commit/ff43d6784b13020d7b5a57d7db6183a159e1a323.


[APP] Parallel PR-Nibble

Implemented Parallel PR-Nibble.
This should be substantially simpler than the original gunrock version or @neoblizz's work in progress https://github.com/gunrock/essentials/pull/69
Note: This matches the reference implementation from SDH, which is parameterized slightly different than the HIVE version.  I need to dig up the HIVE reference code to make sure this is right, but it's definitely close.  Maybe need to change the formulation slightly.
Observations:

Uses the batch operator, but I think maybe we're doing lots of small kernel launches.  The essentials version actually isn't any faster than a single threaded (!) CPU implementation on jhu.mtx, with the parameters that are hardcoded now.  I think this is because the algorithm doesn't actually do a whole lot of work (in Pagerank terms, it doesn't spread mass to enough nodes).  If you reduce alpha to 0.05 or 0.01 runtime goes up (less strongly local / more global diffusion) and the essentials version beats the CPU version.

Reference: Fig 5 + 6 in this paper -- https://arxiv.org/pdf/1604.07515.pdf

Gonna merge this as is and then look to see how we can improve the batch operator for the inner-loop.


[102404592] Refactor to CloudFoundry Branding extension point to allow programatic contributions for Cloud Server Urls.

This is the description of the pull request:
Major changes include:

Adding a new property "urlProviderClass" to the service child of org.cloudfoundry.ide.eclipse.server.core.branding which is a class that returns programatically Cloud Server Urls.
Make the static extensions (defaultUrl and cloudUrl) to be both optional.
Added a new public interface ICloudFoundryUrlProvider that every contributor of the class mentioned in #1 must extend.
Created a new AbstractCloudFoundryUrl class that every Cloud Server Url must extend. This class provides all methods that identify a server.
Switched all possible references of the internal CloudServerURL class to use instances of AbstractCloudFoundryUrl, then deprecated all methods that were exposing that internal class.
Created a new class CloudServerUIUtil that exposes the new methods that return AbstractCloudFoundryUrl
Enhanced CloudUIUtil to publicly expose the methods that are real API (using the new AbstractCloudFoundryUrl)
Improved extension point reading, since now that dynamic Urls can be provided dynamically they can be read from a network call, so all UI places that use these are receiving a IRunnableContext so those operations can run forked and report progress.

Thanks for these changes. I've reviewed the code and it looks fine for merging, but I'll test with our scenarios first to make sure nothing is broken. If all is fine with the testing, I'll merge.


Enable toggling dark mode, light mode

I think a flag --dark-mode can be added, since some people prefer viewing content in dark mode


Output of mix sentry.send_test_event has no environment name

Environment
What version are you running? Etc.

OTP 26.2.3
Elixir 1.16.2
sentry-elixir 10.2.1

Steps to Reproduce

Configure sentry (set DSN and environment_name to staging)
Sending a test event via mix sentry.send_test_event

Expected Result
Display current environment_name (staging) in the output of the mix command.
Actual Result
root:/opt/app# mix sentry.send_test_event
Client configuration:
server: <url>
public_key: <public_key>
secret_key:
current environment_name: ""
hackney_opts: [pool: :sentry_pool]

Although the current environment is set to staging the output shows "". In the iex I get the correct value for environment_name:
iex(5)> Sentry.Config.environment_name()
"staging"

Hi @krns thanks for the issue.
Are you running iex in the same Mix env as mix sentry.send_test_event? Where are you configuring the environment?
I cannot reproduce this within this repo, for example:
→ mix sentry.send_test_event
Client configuration:
current environment_name: "dev"
hackney_opts: [pool: :sentry_pool]

Event not sent because the :dsn option is not set (or set to nil)

We are setting the dsn and environment_name in the runtime config to distinguish between staging and production environment.

Can you try a patch in another branch I just pushed up by doing
{:sentry, github: "getsentry/sentry-elixir", branch: "al/loadconfig"}

in your deps instead of getting :sentry from Hex?

Just tried you branch, but the output is the same:
root:/opt/app# mix sentry.send_test_event
Client configuration:
current environment_name: ""
hackney_opts: [pool: :sentry_pool]

Event not sent because the :dsn option is not set (or set to nil)

@krns do you have a way to reproduce this with a minimal app, so I can give it a try?

I will try to isolate the problem and get back to you.

Here is the minimal app for reproduction: https://github.com/krns/sentry_test
To reproduce you have to setup the project, mix deps.get and set sentry env via export SENTRY_ENV='staging'. When you execute the mix task via mix sentry.send_test_event the environment_name should be "".
⋊> ~/c/sentry_test on main ◦ export SENTRY_ENV='staging'                                                                                13:29:28
⋊> ~/c/sentry_test on main ◦ mix sentry.send_test_event                                                                                13:29:29
Client configuration:
current environment_name: ""
hackney_opts: [pool: :sentry_pool]

Event not sent because the :dsn option is not set (or set to nil)

⋊> ~/c/sentry_test on main ◦ iex -S mix                                                                                13:29:32
Erlang/OTP 26 [erts-14.2.1] [source] [64-bit] [smp:10:10] [ds:10:10:10] [async-threads:1] [jit]

Interactive Elixir (1.16.2) - press Ctrl+C to exit (type h() ENTER for help)
iex(1)> Sentry.Config.environment_name()
"staging"

@krns I was able to reproduce it, thanks. #708 fixes it, can you give that a try?

Great. We'll get a release going in the next few days.


About the training speed

Hello. I am trying to reproduce the fine-tuning experiments of SD1.5. But it is found that the training speed is very low. Each step of online sampling takes close to 10 minutes. For this estimate, the entire training process requires ((4000/40)(step) * 10(min) * 10(epoch) / 60 = 166h for online sampling alone. This is far from the 12h training time stated in the paper.
Therefore, I would like to ask what might be causing my training speed to be abnormal. I use 4 A800-80G gpus. The batch size of each gpu is 10. I do not change any other hyperparameters.
Looking forward to your reply.

I change the sample_batch_size from 10 to 5, and then the training speed gets right. Thanks!


Removes the dependency on OpenVanilla

Yes, we don't use any function in OpenVanilla anymore.
Let's say goodbye to 2006.

A second. I will fix that.


Fancy, Embeddable Quotes

We should be able to generate fancy, embeddable quotes that feature in press releases, but are also searchable within a Leaflet Bomb instance.

This will allow us to catch user attention and allow the media to easily embed a quote in their story that looks good.
In order to spread awareness of our service, we could put a little bomb in the bottom-left corner.
Also, putting a beacon in the quote will allow marketers to track stories where their quote has been featured.

https://oembed.com/


Bump<EMAIL_ADDRESS>

Closes #466 with https://github.com/nteract/transformime/pull/72

Looks like we have to wait a bit for travis: https://www.traviscistatus.com/


[Bug]: An error occurs when the partition plan is delivered using a version earlier than ob3.x

ODC version
ODC 4.2.4-bp1
OB version
OB 1479，ob2.x
What happened?
An error occurs when the partition plan is delivered using a version earlier than ob3.x

What did you expect to happen?
The partition plan task is successfully delivered.
How can we reproduce it (as minimally and precisely as possible)?
Use OB1479 and OB2.x versions to deliver partition plan tasks.
Anything else we need to know?
No response

pass


Create codeql-analysis.yml

Description
Added Code-QL workflow
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Updated the dataset

Checklist:

[x] I checked that there's no dataset update (can be done by running npm run update-dataset)
[x] npm test passes on my machine
[x] npm run lint passes on my machine

Codecov Report

Merging #79 (b803ffa) into develop (3fe8e06) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop       #79   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           20        20
  Lines          173       173
  Branches        33        33
=========================================
  Hits           173       173

Continue to review full report at Codecov.

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


[5.6] blocking pop from redis queues

Currently we pop and reserve jobs off the queue using redis pop and zadd operations inside a LUA script. This PR adds an alternative algorithm that uses blpop instead of pop. It enables the Redis queue driver to wait for the jobs for a specific number of seconds when the queues are empty and server the jobs as soon as they arrive. As a result we can safely run artisan queue:work with no sleep (--sleep=0) without killing the CPU power and improve the wait time of low traffic queues from sleep/2 seconds on average to zero.
The new blocking pop algorithm has a drawback: blpop cannot be used in Lua scripts, therefore, there is a tiny chance of the jobs being lost if the PHP script crashes between pop and reserve operations. Another consideration is that there is no way to do blocking pop from delayed and reserved queues in redis. Therefore, this PR keeps the current pop algorithm in place and lets the user to choose by setting config('queue.connections.{redis_connection_name}.timeout).
PS: I have used blpop for a while via my package, https://github.com/halaei/bredis, that I would like to deprecate if this PR is acceptable.

There's some weird naming; timeouts in the queue system usually refer to how long time a job can execute. It also seems that this implementation disallows me to block forever. Perhaps it would be easier to just add a "blpop": { "enabled": true, "timeout": 1337 } setting in the config instead of trying to calculate if blpop should be used based on the timeout specified.

You are right about the naming, I just coppied the argument name from redis documentation.
I don't think blpop forever is a good idea. For one thing, it will be in conflict with --timeout param of queue:work. Similar systems, like long-polling in sqs and telegram, don't allow for large values, let alone waiting forever.

Any suggestion on what should timeout be renamed to?

blockFor?

Closing pending inactivity. Feel free to re-open with blockFor --block-for changes.


cj: Fix deadlock in SignFinalTransaction

POTENTIAL DEADLOCK DETECTED
Previous lock order was:
 cs_deqsessions coinjoin/coinjoin-client.cpp:141 (in thread )
 ::mempool.cs coinjoin/coinjoin-client.cpp:561 (in thread )
 (1) cs_coinjoin coinjoin/coinjoin-client.cpp:561 (in thread )
 (2) cs_wallet wallet/wallet.cpp:315 (in thread )
Current lock order is:
 cs_deqsessions coinjoin/coinjoin-client.cpp:980 (in thread )
 cs_main coinjoin/coinjoin-client.cpp:788 (in thread )
 mempool.cs coinjoin/coinjoin-client.cpp:788 (in thread )
 (2) mixingWallet.cs_wallet coinjoin/coinjoin-client.cpp:789 (in thread )
 (1) cs_coinjoin ./coinjoin/coinjoin.h:369 (in thread )

Introduced in #4463

rebased after #4475


Configuration should be a required property

🎊 My diff is ready for review at http://codereview.cc/D1950


DOC: add Numpy import to examples

Reference issue
https://github.com/scipy/scipy/issues/16759
What does this implement/fix?
Added import numpy as np to docstring examples in every file as per the script https://gist.github.com/WarrenWeckesser/f81e1d6453a639c745da50746d7078fb
Additional information

In cases where the example has code like this
    >>> from numpy.random import default_rng
    [...]
    >>> rng = default_rng()

I agree with @melissawm's suggestion to adjust to code to have just one numpy import.  The code would become
    >>> import nump as np
    [...]
    >>> rng = np.random.default_rng()

This is consistent with most other uses of default_rng throughout the SciPy examples.
The one other case like this involves scipy.signal.max_len_seq, which has the import
    from numpy.fft import fft, ifft, fftshift, fftfreq

I think we can leave that as is in this pull request.  In a separate pull request, we could change that to import from scipy.fft instead of numpy.fft.

The 3.11-dev job installs numpy from the main development branch.  The failures are the result of https://github.com/numpy/numpy/pull/22385, and not related to this pull request.

Merged.  Thanks @BassCoder2808!


params: (prev) => ({ ...prev, bar: "hello" }) causes a TS error when there are multiple params in prev

Describe the bug
When passing a function to params (per the docs), the params contained within the prev argument are string | undefined, which doesn't look right to me.
      onClick={() => {
        navigate({
          to: '/test/$foo/$bar',
          params: (prev) => ({ // <-- params has an error (see screenshot below)
            ...prev,
            bar: 'new-bar',
          }),
        });
      }}

Your Example Website or App
https://stackblitz.com/edit/github-xazqaz?file=src%2Froutes%2Ftest.%24foo.%24bar.tsx,src%2Froutes%2F__root.tsx,src%2Froutes%2Findex.tsx
Steps to Reproduce the Bug or Issue

Open the stackblitz project
Open test.$foo.$bar.tsx
Observe the TS error

Expected behavior
I don't expect a TS error:
    Call signature return types '{ bar: string; foo?: string | undefined; }' and '{ foo: string; bar: string; }' are incompatible.
      The types of 'foo' are incompatible between these types.
        Type 'string | undefined' is not assignable to type 'string'.
          Type 'undefined' is not assignable to type 'string'.(2322)

Instead I expect the type foo to be string, not string | undefined.
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome
Version: 126.0.6478.115 (Official Build) (arm64)

Additional context
No response

You need to set the from field to get strict typing.
Like this:
const Foo: React.FC = () => {
  const navigate = useNavigate({ from : '/test/$foo/$bar' });
  return (
    <button

Gotcha, thanks.
How about const navigate = Route.useNavigate(), shouldn't that be equivalent to useNavigate({ from: '/test/$foo/$bar' }) then?


Add Authorization header if userinfo is present

Summary
Add Authorization header if userinfo is present
Full changelog

Add Basic Authorization to http-log plugin

Added tests and fixed indentation. CI failure doesn't seem to be related to the changes though.

@thefosk do we really want to log secrets?

@amir can you rebase this against next to resolve the merge conflict?

merged through #2432. Thanks for your contribution @amir 👍


problem with static files

thank you for sharing OXAR!
However, I have one issue: static files.
APEX cannot access them, so e.g. logo, which is in static files, is not visible etc.
There are 3 accounts APEX_LISTENER, APEX_PUBLIC_USER, APEX_REST_PUBLIC_USER,
all unlocked and i can connect to database using them.
In tomcat log i found info as below:
"SEVERE: *** The pool named: apex_pu is reporting no APEX workspaces are enabled for RESTful Services,
but the pool named: apex_al is reporting 2 workspaces are enabled.
This implies APEX was installed after Oracle REST Data Services. Please run the following command and restart Oracle REST Data Services:
java -jar ords.war validate --database apex
APEX based RESTful Services, including those used to serve static resources will not function until this has been done."
i have run java -jar ords.war validate --database apex two times, log from this command does not contain anything wrong, restarting ords.war, tomcat or whole machine gave no result.

Thanks for your feedback. Can I please check with you which version of ORDS are you attempting to install?


Filter Widgets for Event_Site

@nursix  Please Review

I think this needs proper use-case research before hacking in just anything. The budget-related filters for event-sites don't quite make sense to me yet.
Filtering by site name and status is fine, however - that's more obvious.
Also, the fact that the deployment has budgets enabled doesn't necessarily mean that is used for event-sites (it's an obvious use-case, but not a mandatory one). We may need to develop the requirements for this linking first.
The budgeting module has basically just been salvaged, and the requirements for it's use are still rather rudimentary. We do need practitioner input on that.

@nursix So should we create a deployment setting for the use of budget in event-site???

As I say: before we go and implement just anything, we should conduct a proper use-case research, and develop appropriate requirements for this. Right now this looks very much like a "shoot first, ask later" strategy.
We can of course speculate how this case might be, but I seriously doubt that this will lead to a practically useful feature set. It would be much better to get some domain input first (i.e. from potential users).

@nursix I see, so i will keep the list fields same as before and add only the filters for site_id and status. And add a todo to add filter widget for budget filtering

@nursix Removed the filter widget for the budget, still displaying all the fields as it were (including budget). Added Todos for budget filters.
Also added filter for event_human_resource and event_asset
Please review

@nursix Hey, So this code still needs re-factoring and also the widgets for budget, but logic wise, is this the implementation that u wanted?


Precision on PFP setting

Description
We need to create a better screen when someone sets his PFP because right now you know that you dond't have whitelisted NFTs but you don't know how to get one, which collection are whitelisted ...

PROPOSED TODO

[x] @Kevils make some propositions and we define a design
[ ] @irisdv or @ayushtom use the constant file to directly integrate the design

Loom : https://www.loom.com/share/df236c1a6639487a90cef9d99d4243fa
Figma file : https://www.figma.com/file/S1UKYgWewNqNHZFAaBUilG/Starknet-ID?type=design&node-id=1926%3A19449&mode=design&t=j9jw2Ye7KGBb9P31-1


Switch from CSS to SCSS

This merge would switch CoursePro from CSS to SCSS. Which, from my experience, is far easier to maintain and build off of.
Benefits

SASS allows variable use which allows for easier changes both by developers and, in the future, users. This includes maps which allow for the creation of variable groups.
SASS has more intuitive nesting that reflects the structure of the HTML/Pug.
SASS can use simple loops which help remove repetitive CSS.
SASS supports basic logical statements.
SASS's SassScript has several built-in functions that automate many styling changes, such as color changes.
SASS using SassScript allows functions to be created by the developer.
SASS using the SCSS form is completely backward compatible with plain CSS.
It allows developers to more easily navigate Bootstrap's source code.

Misc. Changes

The README has been updated to reflect the additional step required to compile SASS.
The README has had various spelling errors and style inconsistencies removed.
The README has had a "Built With" section added to provide future developers easier access to learning resources.
The README has been restructured significantly to increase readability.

I heavily considered adding the Sass compilation to the start script but decided not to ultimately to keep startup times the same.
However, it only takes about 200ms to compile because of how fast LibSass is, so I would be completely for adding that to npm start if you think it is a good idea.

Yeah, I agree. I think we're better off leaving it out and letting the developer make the decision on whether or not to constantly rebuild in their local environment.


WIP: fixing multiple references in an analysis

This should pass all test and could be deployed in the main site. However, I'm adding as WIP as I would like to discuss internally the decisions made here via email with other devs. Email should be out in a few minutes.

closing in favor of https://github.com/biocore/qiita/pull/1678


'idf.py build' appears to be single threaded (IDFGH-4589)

When I run idf.py build, my machine is largely idle and only a single core is used.  For large builds, this unnecessarily extends the build time to the point of frustrating waste of time.
Please update idf.py to be multi-threaded or multi-processed so that more machine resources are utilized, reducing build time and frustration.

Hi @tpoppe, idf.py uses CMake to generate the build script. The build process is done by Ninja (if available) or GNU Make.
Ninja is multithreaded by default, it will detect the number of processors on the computer.
When GNU Make is used, idf.py detects the number of processors and sets concurrence via -j argument:
https://github.com/espressif/esp-idf/blob/2bfdd036b2dbd07004c8e4f2ffc87c728819b737/tools/idf_py_actions/constants.py#L32
Some information requested by the issue template is missing from your report, could you please fill it in? Operating system, ESP-IDF version, and other information about your environment would be helpful to diagnose the problem.

Hi @tpoppe, idf.py uses CMake to generate the build script. The build process is done by Ninja (if available) or GNU Make.
Ninja is multithreaded by default, it will detect the number of processors on the computer.
When GNU Make is used, idf.py detects the number of processors and sets concurrence via -j argument:
https://github.com/espressif/esp-idf/blob/2bfdd036b2dbd07004c8e4f2ffc87c728819b737/tools/idf_py_actions/constants.py#L32
Some information requested by the issue template is missing from your report, could you please fill it in? Operating system, ESP-IDF version, and other information about your environment would be helpful to diagnose the problem.

Hi @tpoppe. How these experiments of yours relate to https://github.com/espressif/esp-idf/issues/6407? Have you enabled Windows defender bypass in the installer or not?

Hi @tpoppe. How these experiments of yours relate to https://github.com/espressif/esp-idf/issues/6407? Have you enabled Windows defender bypass in the installer or not?

I'm closing this for inactivity. Please let us know if this is still an issue.


Leveling from 2 to 3 forces a 4th level up as well.

AS-1.1.1
DnD5e-3.1.2
Foundry-11.315
https://github.com/user-attachments/assets/b8f06853-585f-41a2-af52-acb411e83b5c

This is not so simple. The problem is that the dnd5e system actually requires the class from the compendium, not the class as stored in the character. For some reason, the SRD classes, when added to the character, record their original compendium uuid in a flag: item.flags.core.sourceId. However, for the AS-test classes, that flag is missing.
@ChobitsCrazy maybe you can figure out why the AS-test classes are different in this way?

nvm... somehow that flag was there and now I can't find it again, so leave it with me


Add Resource Request: metaBUS

What is the resource called?:
metaBUS
What is the link to the resource?:
http://metabus.org/
Please provide a short description of the resource (1-3 sentences):
A cloud-based research synthesis platform sitting atop the world's largest collection of curated social science research findings.
Which step of the research cycle does the resource belong to?:
I don't know
If possible, please provide the exact path where you want this resource to go?:
NA

suggestions of where to place it? Dissemination --> Research Sharing ?

Unsure but makes sense, @helenahartmann what's your opinion?

How about Project Start –> Literature ?

Bumping so we won't forget: @helenahartmann, we need your input :)

I like project start - literature or we make an extra point somewhere about type of study - meta analysis? Didn't we have other meta resources already?

Ugh that's a nice point. Maybe in the future we have some more "meta-resources". Then creating a new subgraph for this in "type of study" makes sense


fix Github action

(test) deactivate sftp protocol
Signed-off-by: Michael Geiger<EMAIL_ADDRESS>

Codecov Report

Merging #289 (895a198) into master (53e57d9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #289   +/-   ##
=======================================
  Coverage   40.81%   40.81%
=======================================
  Files           4        4
  Lines          49       49
=======================================
  Hits           20       20
  Misses         29       29

Continue to review full report at Codecov.

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


Codegen Event switch case ordering

CodeGen doesn't seem to generate reproducible code from run to run. In the switch case, the order of case changes every time you do codegen. Have yet to look into the codegen code but perhaps sorting the IEvent generic type before generating the switch-case block might be a solution?
Note that this is an annoyance for the developers as they now need to manually exclude the auto generated files from being committed or you end up with file changes that are superfluous in your git history.

Closing this issue as the code that does the sort is in Marten - https://github.com/JasperFx/marten/issues/2996


New KB article

New KB article describing how to add both ports 80 and 443 to a self
service load balancer.

your numbers do not increment after 1-2.  check your code you need to put a couple spaces (like you did on item 1) on the image code or it doesn't increment properly (in atom its just a tab)


Code works in deployment environment but malfunctions after packaging - Need a way to package in deployment mode

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
6.3.0
Electron version
19.0.11
Operating system
macOS 13.2
Last known working Electron Forge version
No response
Expected behavior
My code works correctly in the deployment environment, but it malfunctions after being packaged. I want to package it in the deployment mode. How should I do that?
Actual behavior
My code works correctly in the deployment environment, but it malfunctions after being packaged. I want to package it in the deployment mode. How should I do that?
Steps to reproduce
My code works correctly in the deployment environment, but it malfunctions after being packaged. I want to package it in the deployment mode. How should I do that?
Additional information
No response

I want to package it in the deployment mode.

I'm assuming you mean in dev mode. This sounds like a workaround for the actual problem, which is that something in your build/packaging pipeline is broken. I don't think you would actually want to do this.

Do you have any alternative solutions

I want to package it in the deployment mode.

I'm assuming you mean in dev mode. This sounds like a workaround for the actual problem, which is that something in your build/packaging pipeline is broken. I don't think you would actually want to do this.

Do you have any alternative solutions

I'm developing a desktop application with Electron and ethers.js. I encounter an error '[cause]:
Error: could not detect network (event="noNetwork", code=NETWORK_ERROR, version=providers/5.7.2)'.
It works perfectly when launched directly via Electron, but the error emerges when initiated through Electron Forge

If you're unable to use that library but are able to connect to the network in other ways in your packaged app, I would guess the problem would lie in the library rather than application packaging causing network errors.
Either way, a minimal repro (that ideally does not involve a specific third-party library unless) would help you debug the situation further.

If you're unable to use that library but are able to connect to the network in other ways in your packaged app, I would guess the problem would lie in the library rather than application packaging causing network errors.
Either way, a minimal repro (that ideally does not involve a specific third-party library unless) would help you debug the situation further.

This is a demo project: https://github.com/infra403/forge-demo. Thank you for the help.


Use same Fastly Key env var as the Fastly Terraform provider

This provider uses FASTLY_KEY, while the Fastly Terraform provider uses FASTLY_API_KEY.
It'd be helpful if the two providers used the same environment variable!

Hi @sjparkinson,
v2.0.0 now supports this feature.
https://github.com/signalsciences/terraform-provider-sigsci/pull/189


[TODO] items tripled in sidebar

Given a file with contents like:
...
** [TODO] Send testing asciidocs to FTP
** [TODO] Call Jeff
...
I get each [TODO] item in triplicate in the Todo Items sidebar.

Marking an item as [DONE] removes all entries for that item.

Whenever I cmd+S save my .org file I am getting the same (in duplicate, not triplicate)
Mac OS 10.14
Atom 1.31.2


fix(cli): correct providers list in prisma init

Closes #15149

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


capture haml feature needed

I've tried (and failed) to do this with
    - var some_haml = Haml(" \
      .some-haml \
         ... \
      ");

what exactly are you trying to do here?


Added neo.mjs

Added neo.mjs to the list of frameworks.
neo.mjs is a webworker driven UI framework using ES8

I'd suggest taking a look over the contribution guidelines (https://github.com/sindresorhus/awesome/blob/master/pull_request_template.md) as this does not appear to be an Awesome list. Perhaps you'd like to create one?


modal don't work in ie11

help me please
modal don't work in ie11
https://codepen.io/Oltica/pen/JpZZPp?editors=1010

@Oltika use the forum, the Discord server or StackOverflow for questions

Please actually read the responses. If you keep opening issues while ignoring our policies we will have to block you.


KAFKA-3066: Demo Examples for Kafka Streams

@ymatsuda

Make streams-examples a separate package.
Three examples:
a. very simple "pipe".
b. "wordcount" comparing KStream and Processor layers.
c. "pageview" comparing typed and untyped programming in KStream.

@ewencp @ymatsuda Could you take a look (it involves using KC to ingress data to Kafka)? Jenkins failure seems not related.

@ewencp @ymatsuda comments addressed; add one more layer to the state directory to include job IDs.

rebased on trunk.

LGTM

Thanks @ewencp , I will address these comments in a follow-up PR.


client_secret required or optional for OAuth2 authorization code flow

https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-auth-code-flow#request-an-access-token says that client_secret is optional and should not be used for native apps. However, I cannot get it to work without the client secret.
For example, the code at https://github.com/jongiddy/oauth2-rs/blob/pkce/examples/msgraph.rs using a client secret works. If I change https://github.com/jongiddy/oauth2-rs/blob/1f8f0dbd98560ff79fa9302ce8540a8d6c106ed9/examples/msgraph.rs#L61 from Some(graph_client_secret) to None, then no client_secret is passed in the token exchange request. I have verified that this is the case using mitmproxy.
However, I get the response:
Err(ServerResponse(ErrorResponse { error: invalid_client, error_description: Some("AADSTS70002: The provided value for the \'client_secret\' parameter is not valid.\r\nTrace ID: 3b7cdbfb-fb25-42c3-8b63-b9dbbf400c00\r\nCorrelation ID: 51b6471c-574b-4c45-97bc-b9684950f5b6\r\nTimestamp: 2018-09-01 15:05:54Z"), error_uri: None }))

Note that https://docs.microsoft.com/en-us/onedrive/developer/rest-api/getting-started/graph-oauth#step-2-redeem-the-code-for-access-tokens provides the same OAuth2 endpoints and workflow, but states that the client_secret is a required parameter.
Is there an app configuration that makes client_secret optional for native apps?

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

ID: fef99ca3-72bb-2c48-a62b-9ff9c314d72b
Version Independent ID: e58fd3a1-0af8-6e16-3d78-3dc748992593
Content: Azure AD v2.0 OAuth Authorization Code Flow
Content Source: articles/active-directory/develop/v2-oauth2-auth-code-flow.md
Service: active-directory
GitHub Login: @CelesteDG
Microsoft Alias: celested

@jongiddy Thanks for the feedback! We are currently investigating and will update you shortly.

@jongiddy We will proceed to close this thread. Please let us know if you have any further query in this regard.

@jongiddy We will now proceed to close this thread. If there are further questions regarding this matter, please open a new issue and we will gladly continue the discussion.

I think this bug is still there.
see this comment https://github.com/MicrosoftDocs/azure-docs/issues/11781#issuecomment-566158538
this is almost the same problem I get
I'm making a cloud save system for my game on Unity. I use an url like this
https://login.microsoftonline.com/common/oauth2/v2.0/authorize?client_id=myclientid&response_type=code&redirect_uri=http%3A%2F%2F" + localhost + "%3A" + port + "&code_challenge=" + codeChallenge + "&code_challenge_method=S256&response_mode=query&scope=user.read%20files.readwrite.appfolder%20files.readwrite%20files.readwrite.all
Since I use PKCE I expect to be able to do this post
    `

form.Add("client_id", myclientid);
form.Add("code_verifier", codeVerifier);
form.Add("scope", "https://graph.microsoft.com/files.readwrite.all");
form.Add("code", token);
form.Add("redirect_uri", "http://" + localhost + ":" + port);
form.Add("grant_type", "authorization_code");
using (UnityWebRequest www = UnityWebRequest.Post("https://login.microsoftonline.com/common/oauth2/v2.0/token", form)) {
           without        form.Add("client_secret", myclientsecret);`
But if I try to do that I get
HTTP/1.1 401 Unauthorized {"error":"invalid_client","error_description":"AADSTS70002: The provided request must include a 'client_secret' input parameter
I also get the same error with a SPA, not only with a webapp. That make no sense.
The same flow made with dropbox works, and of course with PKCE the implicit flow should be obsolete.


adding random delay between pokemon capture & transfer. #774

Short Description:
random delay between capture & transfer.

As with #1205, this should be customizable within the settings for people who want lightning fast botting.

@douglascamata this makes the bot more humanlike, can we get a couple approvals? Looks good to me.

:+1:  has my approval

:+1:

This seems kinda silly. There is no way that this is going to be the way niantic uses to catch people botting. I feel like this will just slow down the bot for no gain. :-/

@TheSavior tracking request frequency is definitely a method to detect bots. You can virtually disable this in the configs if you feel this is a waste.


1.20.1 support

Describe the Suggestion
since the 1.20 version for forge dropped and i don't see a branch for 1.20 development yet, can you start on it?
Screenshots and Videos
No response
Additional Context
No response

Tropheus has been porting the Forge branch of Create to 1.20, and it has been literally a day since release. Be patient.


ci(master): replace  DOCKER_BUILD_BASE_IDR => DOCKER_BUILD_BASE_DIR

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

/kind bug
What this PR does / why we need it:
replace  DOCKER_BUILD_BASE_IDR => DOCKER_BUILD_BASE_DIR
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

/lgtm
/approve


[admin] merge to live

Docs Build status updates of commit addeecf:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

officedocs-dev-exchange-ews-proxy/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Update index.html

there are some improvements you can make for better practices and efficiency:

Meta Tags: The meta tags should be corrected to ensure compatibility and define the character set correctly.

Favicon: Use the  tag to specify the favicon.

Font Link: The  tag for fonts should be used correctly to load the desired font.

Developer Checklist

[x] Followed guidelines mentioned in the readme file.
[ ] Followed directory structure. (e.g. ProjectName/{USERNAME}/...yourfiles)
[ ] Starred ⭐ the Repo (Optional)

Summary
add a summary here
Screenshot
attach screenshots/gifs here
Live Project Link
add a working project link here

please check it and merge it if possible.


Support for Java 9 and above?

Looks like the latest version in EclipseExecutionEnvironment is still JavaSE18.

I'm not exactly sure what the executionEnvironment field is used for. It looks to me like if you don't set it and use the default value of None then everything generally works, which is probably why no one else has reported problems with Java 9. I'm curious if you can leave it unset or if you can help explain what it's used for?

You are right, leaving it unset works. So why does that field exist if it's not doing much? I originally assumed that this project didn't support Java 9 because that field was capped at Java 8.
Anyway, I'll close the ticket.

I don't know. That was there before I started helping with this project
My guess is that if you have multiple JREs installed that field allows you to specify a different one to use


Nmelhado master

Whoops, I'm a scrub, sorry.

No worries! I've made far worse mistakes on GitHub before


Error prone lift

Version: 4.8.1
Module: quill-sql
Database: postgres
Expected behavior
double lift should be detected at compile time.
Actual behavior
It fails at runtime with
java.lang.RuntimeException: Unquotation can only be done from a quoted block.
	at io.getquill.EagerPlanter.unquote(DslModel.scala:99)

Steps to reproduce the behavior
https://scastie.scala-lang.org/nQ7pVp4ZQXarCegBoMmIyg
With postgres driver it compiles fine.
Workaround
Remove lift.
@getquill/maintainers

https://scastie.scala-lang.org/CCFCK1XNQS6nCAl1x0TiPg


modified client put to use no content-type header

According to the REST API documentation (https://msdn.microsoft.com/en-us/library/dn631831.aspx) in put methods, the content-type header should be blank).
Otherwise when sending a put with a body in the request of a file, the content-type is given and the response is a 415 (Invalid Media Type)

Good work. Didn't realize this issue. Merging


time setting on Europe

where is the time zone line ???

Edit your time difference information here.
https://github.com/Xinyuan-LilyGO/T-Display-S3/blob/dcc62d861dafd3a89aabd1b4dd48da5bcea1d2e6/example/factory/pin_config.h#L10

What is obtained is UTC time. I'm not aware of time differences in all regions. But I know that all time differences are adjusted according to UTC time. I think users can correct the displayed time according to their own time difference information.

Yeah, it's just very unclear what to set. If you set a location then it is clear.
For example, here in the UK we use GMT, which depending on time of year has 1 hour offset or not, so the UTC offset is zero, but presumably the DST is 1hour (3600secs), but is that true at all times of the year or do you need to recompile to change the daylight savings. Also some countries switch the Daylight Saving time at different dates compared to other countries, so how the heck can that work if you don't set a location.

I have an idea to modify the time difference by accessing the current IP address information after networking. In this way, the correct time can be obtained without the need for the client to modify it.

I need to find a free and globally accessible API support.

This is a risky tactic, but probably acceptable to 95% of people. I've used a few but as you know most need api keys, this seems legit and open https://www.ipify.org/ (scroll down for usage). I'm probably glossing over my memories as I was convinced I'd set the location as a string like Europe/London, but I can't see that anywhere except some old esp issues, but it looks like the https://github.com/nayarsystems/posix_tz_db/blob/master/zones.csv zones file was what I used for lookup, combined with setenv and tzset like https://randomnerdtutorials.com/esp32-ntp-timezones-daylight-saving/
One user even keeps the whole zones file in spiffs and parses if required (user change)...madness but works.

I updated the method to get the timezone automatically. In theory, users anywhere can access local time. But batch verification is missing. I tested it locally and it works fine.

I like it, works well. I've added a custom override with simple #ifdef check, not sure you want it, but stuck in a pull request in case you do.

If you have any questions, please open it again


Bottom Radio button on growth parameters clears text next to button

Described in detail here by @MattHJensen:
https://github.com/OpenSourcePolicyCenter/webapp-public/issues/87#issuecomment-151159872

The problem is in paramradio.html
which generates this html:

I doubt this form is working correctly. This should probably be hotfixed soon.
@talumbau can you think of a quick fix here? If not, I can dig deeper.

Is this not the proper default for a radio button? Anyway, I am responsible for the radio button but it's all pretty hokey because that radio button is not tied to the PersonalExemptionForm ModelForm. I think the real answer is a more mature Django form field or something from a JS toolkit because @MattHJensen mentioned that it would be desirous to have features like:

if I start typing in one of the boxes, that radio button becomes selected
if a radio button is selected and text is in the field next to it, if you click the other radio button, the text in the field next to the previously selected button disappears

there could be other useful interactions. The button does work in the sense that a selected button and selected text are propagated to the worker nodes and the "Edit Parameters" button will show the persisted state (if text is entered and a button is selected). If you could implement the two features listed above and have that make sense in some way with the Django ModelForm class, that would be great. H already implement a "growth_fixup" function to handle some of the ugliness here.

Just want to chime in that the whole purpose of the radio button here is to keep the user from modifying both of the growth options at the same time. (An alternative, if it is easier, would be to just erase one when the user modifies the other and provide some sort of notification for why this happened.)


Species are not removed from the legend once deselected

Selecting a species to filter on creates a legend on the map view. When a species is deselected it remains in the legend, when all species that have been selected are deselected (i.e. the species filter option is empty) the legend can still be seen.

Fixed, tested 23/01


fix(remix-architect): fix binaryTypes

Closes: #4040
That's a tiny typo in the remix-architect/binaryTypes.ts at line 36. I suppose there should be two types audio/wav and audio/webm.

I'm not sure should that goes to the discussion section or that could be done here, but I've compared types from that file with types from mime-types (thanks to @wingleung for his note in his fork of Architect Handler). The library has all needed types except:

audio/x-midi
application/x-gzip
application/x-java-archive
application/x-zip

I've not found reliable info about those types. I'm not sure if they are valid, and it seems okay to check binary types by the mime-types library instead of the custom method. What do you think about that?

I'm not sure should that goes to the discussion section or that could be done here, but I've compared types from that file with types from mime-types (thanks to @wingleung for his note in his fork of Remix Architect). The library has all needed types except:

audio/x-midi

application/x-gzip

application/x-java-archive

application/x-zip

I've not found reliable info about those types. I'm not sure if they are valid, and it seems okay to check binary types by the mime-types library instead of the custom method. What do you think about that?

this list started as a form of the list that architect keeps, but i'm down to looking into using mime-types if we can

I'm not sure should that goes to the discussion section or that could be done here, but I've compared types from that file with types from mime-types (thanks to @wingleung for his note in his fork of Remix Architect). The library has all needed types except:

audio/x-midi
application/x-gzip
application/x-java-archive
application/x-zip

I've not found reliable info about those types. I'm not sure if they are valid, and it seems okay to check binary types by the mime-types library instead of the custom method. What do you think about that?

@kamtugeza 👍  yes, I wrote a todo about it because I couldn't find a isBinaryMimeType() kind of helper in that library. I had a closer look now and I see the library is more of a tool to figure out mime-types instead of identifying if the mime-type is binary or not.
I see now in the AWS docs that ALB and both api gateway versions already pass a property called isBase64Encoded so I think, for the aws integration, we should use that property instead.
And for the other adapters like netlify, we need to check if they have a similar property to this.

ALB doc
APIGW doc

Here is my silly question: do we have a situation where we should have an empty body, and a response should be "binary"? 🙃
I ask that because we check a response encoding by the Content-Type header and if it's base64 then we pass nodeResponse.body property into the readableStreamToString  method where we expect body is ReadableStream<Uint8Array> and we don’t perform here any type checks or transformation (e.g. string to stream).
So why don’t we do a simple check, does the nodeResponse.body has an instance of ReadableStream?


Adding a Model for categorising complaints

🚀Feature Request
I have a suggestion (and may want to implement it 🙂)!
I want to add a model to The complaints Section for the accountant profile side that will classify the complaints into critical and low that will tell the accountant that which complaints need immediate attention and action and which need to be resolved first that will also save the time of accountant.
I am going to build and Train the Model using Hugging face Transformers , NLTK.

Yeahh It is a great idea but to categrize the complaints I have used voting feature which itself reflect the most voted complaints is the critical one and since it's a MERN based project so personallly I don't want this it u can fork the repo and can implement these features in your own.


nginx 1.13.4 fails to build

cc -c -pipe  -O -W -Wall -Wpointer-arith -Wno-unused-parameter -Werror -g -I/usr/include/apr-1 -DNDK_SET_VAR -DNDK_UPSTREAM_LIST      -I/usr/include/apr-1       -I/usr/include/apr-1 -I/usr/include     -I/usr/include     -I/usr/include/libxml2     -DWITH_LUA -I/usr/include     -DWITH_PCRE_STUDY -DMODSEC_PCRE_MATCH_LIMIT=1500 -DMODSEC_PCRE_MATCH_LIMIT_RECURSION=1500  -DREQUEST_EARLY  -DCACHE_LUA     -DPERFORMANCE_MEASUREMENT            -DWITH_APU_CRYPTO -DWITH_REMOTE_RULES     -I/usr/local/include -DPCRE_STATIC     -DWITH_YAJL -I/usr/include/yajl     -DWITH_SSDEEP -I/usr//include  -D_GLIBCXX_USE_CXX11_ABI=0 --std=c++11  -I src/core -I src/event -I src/event/modules -I src/os/unix -I /usr/work/ngx_devel_kit/objs -I objs/addon/ndk -I /usr/work/modsecurity/nginx/modsecurity -I /usr/work/modsecurity/nginx/modsecurity/../../standalone -I /usr/work/modsecurity/nginx/modsecurity/../../apache2 -I /usr/work/pcre -I /usr/work/openssl-1.1.0f/.openssl/include -I objs -I src/http -I src/http/modules -I src/http/v2 -I src/http/modules/perl -I /usr/work/ngx_devel_kit/src -I /usr/work/ngx_devel_kit/src -I /usr/work/ngx_devel_kit/objs -I objs/addon/ndk -I /usr/work/ngx_pagespeed/psol/include -I /usr/work/ngx_pagespeed/psol/include/third_party/chromium/src -I /usr/work/ngx_pagespeed/psol/include/third_party/google-sparsehash/src/src -I /usr/work/ngx_pagespeed/psol/include/third_party/google-sparsehash/gen/arch/linux/x64/include -I /usr/work/ngx_pagespeed/psol/include/third_party/grpc/src/include -I /usr/work/ngx_pagespeed/psol/include/third_party/protobuf/src/src -I /usr/work/ngx_pagespeed/psol/include/third_party/re2/src -I /usr/work/ngx_pagespeed/psol/include/out/Release/obj/gen -I /usr/work/ngx_pagespeed/psol/include/out/Release/obj/gen/protoc_out/instaweb -I /usr/work/ngx_pagespeed/psol/include/third_party/apr/src/include -I /usr/work/ngx_pagespeed/psol/include/third_party/aprutil/src/include -I /usr/work/ngx_pagespeed/psol/include/third_party/apr/gen/arch/linux/x64/include -I /usr/work/ngx_pagespeed/psol/include/third_party/aprutil/gen/arch/linux/x64/include -I /usr/work/ngx_pagespeed/psol/include/url
-o objs/addon/src/ngx_pagespeed.o
/usr/work/ngx_pagespeed/src/ngx_pagespeed.cc
/usr/work/ngx_pagespeed/src/ngx_pagespeed.cc: In function ‘char* net_instaweb::{anonymous}::ps_merge_srv_conf(ngx_conf_t*, void*, void*)’:
/usr/work/ngx_pagespeed/src/ngx_pagespeed.cc:1164:53: error: ‘class net_instaweb::RewriteOptions’ has no member named ‘standby’
if (!cfg_s->server_context->global_options()->standby()) {
^~~~~~~
/usr/work/ngx_pagespeed/src/ngx_pagespeed.cc: In function ‘ngx_int_t net_instaweb::{anonymous}::ps_preaccess_handler(ngx_http_request_t*)’:
/usr/work/ngx_pagespeed/src/ngx_pagespeed.cc:3023:31: error: ‘ngx_http_core_try_files_phase’ was not declared in this scope
while (ph[i + 1].checker != ngx_http_core_try_files_phase &&
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
objs/Makefile:1737: recipe for target 'objs/addon/src/ngx_pagespeed.o' failed
make[1]: *** [objs/addon/src/ngx_pagespeed.o] Error 1
make[1]: Leaving directory '/usr/work/nginx-1.13.4'
Makefile:8: recipe for target 'build' failed
make: *** [build] Error 2

a little update: patch from https://patch-diff.githubusercontent.com/raw/pagespeed/ngx_pagespeed/pull/1453.diff and net/instaweb folder from https://github.com/pagespeed/mod_pagespeed did the trick - it works!

@netsafe what commands did you run (and where did you find them)?
The following error makes me think you are building ngx_pagespeed/master against mod_pagespeed/<IP_ADDRESS>:
{anonymous}::ps_merge_srv_conf(ngx_conf_t*, void*, void*)’:
/usr/work/ngx_pagespeed/src/ngx_pagespeed.cc:1164:53: error: ‘class net_instaweb::RewriteOptions’ has no member named ‘standby’
if (!cfg_s->server_context->global_options()->standby()) {

I also get the error. at first, I was getting the exact issue described in issue #1451, so I applied the fix(same diff file as @netsafe mentioned), which resulted in the
‘class net_instaweb::RewriteOptions’ has no member named ‘standby’
issue
I'm following a basic guide to compile nginx with pagespeed, which basically guides me to download this:
https://github.com/pagespeed/ngx_pagespeed/archive/v<IP_ADDRESS>-stable.zip
which gets extracted(in my case, to /usr/local/src), then this:
wget https://dl.google.com/dl/page-speed/psol/<IP_ADDRESS>-x64.tar.gz
extracted to the same directory
using the example ./configure options, I simply add this to the end:
--add-module=/usr/local/src/ngx_pagespeed-<IP_ADDRESS>-stable
this is the point where I get the first error, I apply the patch to the src/ngx_pagespeed.cc file, which then produces the second error.

@kronflux so when I extract https://github.com/pagespeed/ngx_pagespeed/archive/v<IP_ADDRESS>-stable.zip and open up ngx_pagespeed.cc line 1164 does not look like if (!cfg_s->server_context->global_options()->standby()) {.  So I am a bit confused about that..
(On the master branch that line does correspond with the error though: https://github.com/pagespeed/ngx_pagespeed/blob/master/src/ngx_pagespeed.cc#L1164 )
Can you point me to the url of the guide you followed? Maybe I can spot what is going on and suggest a correction there.
The official one is here: https://modpagespeed.com/doc/build_ngx_pagespeed_from_source

You seem to have completely missed the part about the patch xD
The file is now changed to match the fix, the whole file is here:
https://github.com/pagespeed/ngx_pagespeed/blob/6ba8658cf9c8ea8c7f0c043483927dac905846a1/src/ngx_pagespeed.cc

Actually, that is the guide I'm following(ish)
I didn't use the variable bits in the commands (NPS_RELEASE_NUMBER, etc)
I simply worked out the correct URLs and used those directly - also extracted files manually.
I skipped the bits using "scripts/format_binary_url.sh", and used the x64 link directly.
As for nginx, I've compiled it many times, so I'm going by memory, however originally I used the official compile guide for nginx.
I'm doing nothing special, except I'm manually grabbing the files.
Perhaps the patch depends on additional changes which are not present in the stable zip?

ah yes. I'm just not too bright.
The patch is specifically for the current(?) branch 1.13.4, the stable release zip is <IP_ADDRESS>
<IP_ADDRESS> was where I was experiencing: ngx_http_core_try_files_phase’ was not declared in this scope
assumedly there's no easy way to use the fix for 1.13.4 with this release, so I'll just either have to switch to the latest branch, or wait until the next stable release.

Found the issue!
https://github.com/pagespeed/mod_pagespeed/pull/1482
This is not present in the current PSOL download.

@kronflux that's correct, pagespeed/mod_pagespeed#1482 isn't included in <IP_ADDRESS>-stable

Would replacing the instaweb folder from the psol download with the one from mod_pagespeed be sufficient? or would this more likely lead to additional issues?
I suppose one could simply add the standby code to the current psol file.
Just trying to find the best way to use the stable release files, without having to wait for a new PSOL release.

Same problem, only path to openssl is different
root/addons/nginx-1.13.4
ngx_cache_purge-2.3
ngx_slowfs_cache-1.10
ngx_pagespeed-<IP_ADDRESS>-stable
ngx_cache_purge-2.3
ngx_slowfs_cache-1.10
ngx_coolkit-0.2
ngx_headers-more-0.32
Ubuntu 16.04.3 with Plesk Onyx 17.5.3 by 1and1. Normal Plesk built-in nginx 1.11.10.
OpenSSL is here, --with-openssl=/usr/src/openssl-1.0.2l \
not in /usr/local/src/
Greets

I think something was wrong with my build environment. I was able to get it to build on a fresh Ubuntu 16.10 installation perfectly using the patch mentioned previously.
All is good here with 1.13.4 and latest stable pagespeed(with patch applied)

Fixed via https://github.com/pagespeed/ngx_pagespeed/pull/1453

not fixed for me (
centos 6.8
psol from https://dl.google.com/dl/page-speed/psol/<IP_ADDRESS>-x64.tar.gz
git log
commit ef3899515e40fc4fd7dd553fdef0048e937db7f4
               ngx_pagespeed-<IP_ADDRESS>-stable/src/ngx_rewrite_driver_factory.cc ngx_pagespeed-<IP_ADDRESS>-stable/src/ngx_pagespeed.cc: In function 'char* net_instaweb::{anonymous}::ps_merge_srv_conf(ngx_conf_t*, void*, void*)': ngx_pagespeed-<IP_ADDRESS>-stable/src/ngx_pagespeed.cc:1165:53: error: 'class net_instaweb::RewriteOptions' has no member named 'standby'        if (!cfg_s->server_context->global_options()->standby()) {                                                      ^ make[1]: *** [objs/addon/src/ngx_pagespeed.o] Error 1 make[1]: *** Waiting for unfinished jobs.... make[1]: Leaving directory/root/rpmbuild/BUILD/nginx-module-pagespeed-1.13.4'
make: *** [modules] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.6H95Ck (%build)
`

@LuckySB ef38995 is the head of the current ngx_pagespeed master.
That should have been f3c64ca8a251f05f228ad32947d7c68c89308f6a (a.k.a. tag v<IP_ADDRESS>-stable)

Still got the same issue thought the psol is updated
objs/Makefile:1429: recipe for target 'objs/addon/src/ngx_pagespeed.o' failed make[1]: *** [objs/addon/src/ngx_pagespeed.o] Error 1 make[1]: Leaving directory '/usr/local/src/nginx-1.13.5' Makefile:8: recipe for target 'build' failed make: *** [build] Error 2


VX30 Weightings Are Significantly Off VXX Current Holdings

Since VXX follows the VX30 index, I would expect he holdings to be close to the calculated VX30 weightings.

did you able to download vix-utils features, it's looks like M2 broken for now.

weights are definitively off
import vix_utils as vu

vix_dates = vu.vix_futures_dates.vix_futures_trade_dates_and_expiry_dates()
weights = vu.vix_futures_dates.vix_constant_maturity_weights(vix_dates)

I am just starting to look at this now.  Anyone already looking at it?

  I have some unit tests I am working on to identify these cases.

there are many cases for 2024

the issue should be around this line of code (before it actually)
https://github.com/dougransom/vix_utils/blob/1b231b2c7fc44ae84584496940ac068139a2e55b/src/vix_utils/vix_futures_dates.py#L184
but i have not been able to pin point it
I don't think that roll period calendar days is the main issue. There are plenty of good weights with it being zero

Working on this in branch https://github.com/dougransom/vix_utils/tree/weights.
So far some refactorings, so we can debug the logic that generates the weights on a small subset of dates.

The orginal code was far off the documentation.  I think I am close to resolving …

Closer still, it might a week or two though before I complete it.  There is an edge case on Wednesday of Vix  Future Expiration.  That is the front month (at least I think it is), but on those days, for the purpose of  S&P 500 VIX Short-Term Futures Index, the front month is the following expiration.

I think it is working now, try vixutil -w w.xlsx.
Note the weights are for three expirations, since there are a couple days where the front month may not be included in VXX etc. (instead it is the following two months).  It happens the day before and day of expiration.

The latest on PyPi has this fix now 0.1.7.

thanks a lot!
so now front month goes to zero the friday before expiration. it makes sense to me.


Bug: NEWS file out of track between main and devel

What happened?
@thomas-neitmann i noticed you added 0.6.2 and 0.6.3 updates around CRAN release, but devel doesn't have these as we had the placeholder for 0.7.0 after 0.6.1. i'll merge these to fix.
Session Information
N/A
Reproducible Example
N/A

@thomas-neitmann feel free to close this issue if you agree? as think you were going to make all the devel updates that you made to main for CRAN release in one go.


Hexagon requires layout

For an unknown reason, the Hexagon component gets its points from a "LayoutContext".  Thus, a Hexagon requires a Layout tag to appear.  Really strange will resolve asap.

ref: https://github.com/aaronzshey/react-hexgrid/blob/405978ca9848dc99c5445a7e3b83c9bb2dd4d0ea/src/Layout.tsx#L114
calculateCoordinates should be used directly in the Hexagon tag.  The Layout tag should only be used for grouping - it should be turned into a simple  tag.

Finally figured out why we need Layout for a Hexagon.  It looks more and more like the original author just copied code from https://www.redblobgames.com/grids/hexagons/
Layout.tsx creates LAYOUT_FLAT:
https://github.com/aaronzshey/react-hexgrid/blob/405978ca9848dc99c5445a7e3b83c9bb2dd4d0ea/src/Layout.tsx#L18
and LAYOUT_POINTY
https://github.com/aaronzshey/react-hexgrid/blob/405978ca9848dc99c5445a7e3b83c9bb2dd4d0ea/src/Layout.tsx#L29
However, LAYOUT_FLAT is hardcoded in and is the only layout ever used:
https://github.com/aaronzshey/react-hexgrid/blob/405978ca9848dc99c5445a7e3b83c9bb2dd4d0ea/src/Layout.tsx#L47
Adding a flat or pointy option is low priority.  For now, I will hardcode all the values and put them straight into the Hexagon component.

see branch no-layout: 73b8798dcfb5c4419c312f6a9ee8f57ac8263f9e
While it works, as it turns out Layout exposes a prop to switch from flat to pointy.  I will add this to the Hex tag

#41


fix(SelectItemGroup): label as defaultProps (il8n)

Related to #107

👍

Bluemix toolchain: Delivery Pipeline deployed carbon-components-react to Bluemix Design System, including commit 6b647a8bc9a22d79bb06e8c75ed6d7a3936da4d7


There is no code to make a figure from RT-qPCR data.

We need some code to

Import RT-qPCR data
Organise it into a pandas DataFrame
Perform the appropriate normalization
Generate publishable figures

Would anyone in the @HuangLabPopulation/dna-probes team like to help with this?

I should add to help with generate python code to Import and Analyze data

@shuqin19 Great idea. The best way to do that would be

Fork this repository to your account
Upload your code
Make a pull request explaining what you are adding

I will review your pull request (check your code) and then integrate it with this master repo

Please see the new pull request, I added a script for data processing

You can reference a pull request using the # symbol or clicking the star bookmark icon above the comment window. This #3 should referenc your pull request.


Speed up sub-format check

This avoids making lots of requests to get expanded details, and
just checks base_path the finder's base path, as that's all we need.

(Paired completely with @tommyp on this.)

LGTM. :ship:


info verb in virtual env docker image returns null version

Describe the bug
info verb in virtual env docker image returns null version
Steps to reproduce
* setup local virtual environment using virtualenv:dev_env image
* connect to any secondary from virtual environment and run the info verb
* @alice🛠@info
data:{"version":null,"uptimeAsWords":"1 minutes 34 seconds","features":[{"name":"noop:","status":"Beta","description":"The No-Op verb simply does nothing for the requested number of milliseconds. The requested number of milliseconds may not be greater than 5000. Upon completion, the noop verb sends 'ok' as a response to the client.","syntax":"^noop:(?<delayMillis>\\d+)$"},{"name":"info:","status":"Beta","description":"The Info verb returns some information about the server including uptime and some info about available features. ","syntax":"^info(:brief)?$"}]}

Expected behavior

info verb should the correct version

Screenshots
No response
Smartphones
No response
Were you using an atApplication when the bug was found?
No response
Additional context
No response

@cpswan @XavierChanth
Please review this PR.
https://github.com/atsign-foundation/at_server/pull/1006

Will this docker file be used to build virtual env docker image
https://github.com/atsign-foundation/at_server/blob/trunk/tools/build_secondary/Dockerfile ?
how do I verify the fix in the PR?

Fixed by #1006
@info
data:{"version":"3.0.25+gha12345","uptimeAsWords":"39 seconds","features":[{"name":"noop:","status":"Beta","description":"The No-Op verb simply does nothing for the requested number of milliseconds. The requested number of milliseconds may not be greater than 5000. Upon completion, the noop verb sends 'ok' as a response to the client.","syntax":"^noop:(?<delayMillis>\\d+)$"},{"name":"info:","status":"Beta","description":"The Info verb returns some information about the server including uptime and some info about available features. ","syntax":"^info(:brief)?$"}]}

Fixed by #1006
@info
data:{"version":"3.0.25+gha12345","uptimeAsWords":"39 seconds","features":[{"name":"noop:","status":"Beta","description":"The No-Op verb simply does nothing for the requested number of milliseconds. The requested number of milliseconds may not be greater than 5000. Upon completion, the noop verb sends 'ok' as a response to the client.","syntax":"^noop:(?<delayMillis>\\d+)$"},{"name":"info:","status":"Beta","description":"The Info verb returns some information about the server including uptime and some info about available features. ","syntax":"^info(:brief)?$"}]}

server functional tests were still returning null. Worked after i made the below change
https://github.com/atsign-foundation/at_server/pull/1008


PFX file is corrupted

When trying to create a new certificate, I get the following error message "An error occurred installing the certificate. Certificate file may not be a valid." A PFX file gets created but it is corrupt. The file size is 0 KB and I get an invalid file message when trying to open it. Since the file is corrupted, it is unable to be imported/installed.
I think I tracked down the issue though. In the VaultManager.CreateCertificate method there is a call to PowershellManager.SubmitCertificate. The SubmitCertificate has force set to true. If I change force to false, the PFX is successfully created and installed.
What's the purpose of using force? Any idea why it won't work on my machine?
Here is the log for reference:
[4:39 PM] Powershell: Initialize-ACMEVault -BaseURI https://acme-staging.api.letsencrypt.org/
[4:39 PM] Powershell: New-ACMERegistration -Contacts<EMAIL_ADDRESS>[4:39 PM] Powershell: Update-ACMERegistration -AcceptTOS
[4:39 PM] Powershell: New-ACMEIdentifier -Dns example.com -Alias example.com -Label Identifier:example.com
[4:39 PM] Powershell: Complete-ACMEChallenge -Ref example.com -ChallengeType http-01 -Handler manual  -Regenerate
[4:39 PM] Powershell: Submit-ACMEChallenge -Ref example.com -Challenge http-01
[4:39 PM] Powershell: Update-ACMEIdentifier -Ref example.com
[4:39 PM] Powershell: Update-ACMEIdentifier -Ref example.com
[4:40 PM] Powershell: New-ACMECertificate -Identifier example.com -Alias cert_example.com -Generate
[4:40 PM] Powershell: Submit-ACMECertificate -Ref cert_example.com
[4:40 PM] [Error]: System.Management.Automation.CmdletInvocationException: asset file already exists ---> System.IO.IOException: asset file already exists
   at ACMESharp.Vault.Providers.LocalDiskVault.CreateAsset(VaultAssetType type, String name, Boolean isSensitive, Boolean getOrCreate)
   at ACMESharp.POSH.SubmitCertificate.ProcessRecord()
   at System.Management.Automation.CommandProcessor.ProcessRecord()
   --- End of inner exception stack trace ---
   at System.Management.Automation.Runspaces.PipelineBase.Invoke(IEnumerable input)
   at System.Management.Automation.PowerShell.Worker.ConstructPipelineAndDoWork(Runspace rs, Boolean performSyncInvoke)
   at System.Management.Automation.PowerShell.Worker.CreateRunspaceIfNeededAndDoWork(Runspace rsToUse, Boolean isSync)
   at System.Management.Automation.PowerShell.CoreInvokeHelper[TInput,TOutput](PSDataCollection`1 input, PSDataCollection`1 output, PSInvocationSettings settings)
   at System.Management.Automation.PowerShell.CoreInvoke[TInput,TOutput](PSDataCollection`1 input, PSDataCollection`1 output, PSInvocationSettings settings)
   at System.Management.Automation.PowerShell.Invoke(IEnumerable input, PSInvocationSettings settings)
   at Certify.PowershellManager.InvokeCurrentPSCommand() in D:\Development\Source\Repos\Certify\src\Certify.Core\Management\PowershellManager.cs:line 94
[4:40 PM] Powershell: Update-ACMECertificate -Ref cert_example.com
[4:40 PM] Powershell: Get-ACMECertificate -Ref =7b2dadb2-d9cb-4a90-b977-7364e1e30536 -ExportPkcs12 C:\ProgramData\ACMESharp\sysVault\99-ASSET\7b2dadb2-d9cb-4a90-b977-7364e1e30536-all.pfx -Overwrite
[4:40 PM] [Error]: System.Management.Automation.CmdletInvocationException: error:0B080074:x509 certificate routines:X509_check_private_key:key values mismatch ---> OpenSSL.Core.OpenSslException: error:0B080074:x509 certificate routines:X509_check_private_key:key values mismatch
   at OpenSSL.Core.Native.ExpectNonNull(IntPtr ptr)
   at OpenSSL.X509.PKCS12..ctor(String password, CryptoKey key, X509Certificate cert, Stack`1 ca)
   at ACMESharp.PKI.Providers.OpenSslLibBaseProvider.ExportArchive(PrivateKey pk, IEnumerable`1 certs, ArchiveFormat fmt, Stream target, String password)
   at ACMESharp.PKI.Providers.OpenSslLibProvider.ExportArchive(PrivateKey pk, IEnumerable`1 certs, ArchiveFormat fmt, Stream target, String password)
   at ACMESharp.POSH.GetCertificate.ProcessRecord()
   at System.Management.Automation.CommandProcessor.ProcessRecord()
   --- End of inner exception stack trace ---
   at System.Management.Automation.Runspaces.PipelineBase.Invoke(IEnumerable input)
   at System.Management.Automation.PowerShell.Worker.ConstructPipelineAndDoWork(Runspace rs, Boolean performSyncInvoke)
   at System.Management.Automation.PowerShell.Worker.CreateRunspaceIfNeededAndDoWork(Runspace rsToUse, Boolean isSync)
   at System.Management.Automation.PowerShell.CoreInvokeHelper[TInput,TOutput](PSDataCollection`1 input, PSDataCollection`1 output, PSInvocationSettings settings)
   at System.Management.Automation.PowerShell.CoreInvoke[TInput,TOutput](PSDataCollection`1 input, PSDataCollection`1 output, PSInvocationSettings settings)
   at System.Management.Automation.PowerShell.Invoke(IEnumerable input, PSInvocationSettings settings)
   at Certify.PowershellManager.InvokeCurrentPSCommand() in D:\Development\Source\Repos\Certify\src\Certify.Core\Management\PowershellManager.cs:line 94
[4:40 PM] Powershell: Update-ACMEIdentifier -Ref example.com

Sorry, I'd have to dig into ACMESharp but I think this was an issue overwriting the certificate in the vault.

Resolved.


More detailed errors when resource not found

Quick fix for #1846
Got scared of the other "Requested resource does not exist" error at line 395 in system_param.rs, under impl<'a, T: Component> SystemParamFetch<'a> for ResMutState<T> {. Someone with better knowledge of the code might be able to go in and improve that one.

Got scared of the other "Requested resource does not exist" error at line 395

I believe you can make the panic! message be the same as the other one you implemented. (e.g, utilizing system_state.name)

bors r+

@therealstork please respond in #2373 for the relicense to MIT/Apache 2.0. Thanks!


Add support for Dlk-Services

https://github.com/DalekIRC/Dalek-Services/

(closing this to avoid notification spam while I'm debugging it)

currently blocked by https://github.com/DalekIRC/Dalek-Services/issues/28


Update FAQ article vac_cert_privileges_travel

@MikeMcC399 thanks for your help! I added all of your suggestions and removed the typos.

@larswmh

thanks for your help! I added all of your suggestions and removed the typos.

You can mark each of the suggestions as resolved. I can't do that myself, since I'm not the author of the PR.

@MikeMcC399 @Ein-Tim @larswmh Many thanks.


Add hashbrown as option for serde_json::Value::Map

Using hashbrown for serde_json::Value gives a up to 10% improvement
in serialization/deserialization performance. On the flip side it
looses the option of a double ended itterator. This adds it as a
feature flag allowing people to choose it if it should suite their
needs.

The benchmarks ignore that applications do not simply parse and stringify json, they do something with the data.

I think the difference you saw is based on the system allocator, runniung it w/ jemalloc returns the results:
target/release/json-benchmark-hashbrown
                                DOM                  STRUCT
======= serde_json ======= parse|stringify ===== parse|stringify ====
data/canada.json         220 MB/s   350 MB/s
data/citm_catalog.json   340 MB/s   370 MB/s
data/twitter.json        260 MB/s   530 MB/s

heinz@Schrodinger ~/Projects/json-benchmark (master) $ target/release/json-benchmark-btree
                                DOM                  STRUCT
======= serde_json ======= parse|stringify ===== parse|stringify ====
data/canada.json         220 MB/s   360 MB/s
data/citm_catalog.json   310 MB/s   310 MB/s
data/twitter.json        230 MB/s   480 MB/s


How long does the run_texture_optimization.lua takes to run??

When I try to run the run_texture_optimization.lua, it repeats the same thing over and over again. It say synthesis finished at resolution 3 and repeats itself. How do I know when to stop the program.
Thank you.

It should only takes 3 iterations and it will stop. Did you check the synthesis result? Is it running multiple images?

I am not sure about that. How do we check the synthesis results? Since I am running it on a server.
Thank you.

Check the file transfer_CNNMRF_wrapper.lua. In line 574:
params.output_folder = string.format('examples/%s',params.save_name)
params.save_name=save_name.."_"..style_name.."TO"..content_name
so the synthesized images should be saved under examples/demo_...TO.../

Thank you, the issue has been resolved. As the results generated are for the example images, what changes should I make to the code, so that I can run for my own image and mask.
Thank you.

@DixithR19  have the same question with you.  I also want to modify the code to make it suit for my own mask, and I'm still not sure should just modify the texture loss is enough?

You need to have initialization which could be anything (in our paper from the content network).

We accelerated the algorithm now it only takes 30s to run.


Improve search algorithm

Currently the search-feature uses a (case-INsensitive) RegEx-matching algorithm like:
If <search input> can be found as-is ANYWHERE in the Answer- OR Question-fields of a Q&A-set.
So for example:

"abc xyz" would match on a Question/Answer of: "ABC xyz" or "123aBc XyZ ... 78910"
"abc xyz" would NOT match on a Question/Answer of: "abcdef" or "abcdefghijklmnopqrstuvw xyz"

Maybe more intuitive would be to match ANY keyword (by splitting the input on spaces) found anywhere in the Question/Answer.
So for example:

"abc xyz" would match on a Question/Answer of: "test-xyz test test abc test"

See #286
See #433

Also:
Add a "relevance"-score to each result, so we can sort the results on it.
For example a level of relevance could be: ("the number of matches in the answer" * 10) + ("number of matches in the question" * 1) =
Other things to consider:

names/descriptions of (Sub-)Categories
Sub-Question's influence on parent Question's relevance?
More recent result more relevant?
Show results from the same (Sub-)Category grouped together?

There are fancy semantic search models that can be deployed without re-training them, e.g. https://www.sbert.net/examples/applications/semantic-search/README.html
Much more robust than RegEx. But need to be hosted, cannot run on client-side

Note to self: See PowerPoint-file: "HIA FAQ UT 2 presentation 26092022"

Also an option:
Using a (commercial) third-party service.
Like: https://www.algolia.com/

Other options for third-party search tools: https://patrickposner.dev/tutorials/search-on-a-static-site/

Use https://fusejs.io/ ?

Possibly, to create a "chatbot-as-a-search"-feature: https://azure.microsoft.com/en-us/products/cognitive-services/question-answering/

Another option, maybe: https://lunrjs.com/

@elwinschmitz we were approached by Deloitte who's looking for a data science project as part of their CSR, see this convo. The "chatbot-as-a-search" might be part of it, although the use case needs to be refined with @JLijftogt .

For Future Reference:
When going the static-generator route: Use https://pagefind.app


Use playbook_dir instead of ansible_user_dir

This will keep the host clean by storing the artifacts in the playbook dir
instead of users home directory.

Thanks for the review @mohit-sheth.


A Changelog would be nice

As a user I only see, that the version number increased but I have no idea what is new.
I would recommend to add a Changelog.md.
And maybe a post for a new release to places like reddit.com/r/rust, so that others see this great project!

This suggestion is very good, I will consider adding CHANGELOG.md from the next release. 😁

This suggestion is very good, I will consider adding CHANGELOG.md from the next release. 😁

Great idea! I want to chime in and recommend this: https://keepachangelog.com/en/1.0.0/
It is used all over by a lot of different projects that I have stumbled across and is genuinely a great guideline!

Great idea! I want to chime in and recommend this: https://keepachangelog.com/en/1.0.0/
It is used all over by a lot of different projects that I have stumbled across and is genuinely a great guideline!

Something like this could really help automate the process, not sure if it exists for Rust projects though: https://github.com/semantic-release/semantic-release#how-does-it-work

Ye, there are tools that already do that. There was a cli on crates called "conv" I think. And there is @conventional-commits-rs. They all work. I use the changelog generator myself. But they're far from complete (and don't get me started on code quality). I sadly had no time lately to work on them for a longer amount of time.

CHANGELOG.md has been added, so I will close this issue, if you have any questions, please feel free to open it again.