nlohmann::to_json method not acting as expected with nlohmann::adl_serializer specialization.

I could not get the nlohmann::to_json method to work consistently with specializations of the nlohmann::adl_serializer which I assumed would work from the way described in the readme.
What is the issue you have?
I was using the How do I convert third-party types explanation and I specialized the adl_serializer in the nlohmann namespace with my third party types, Eigen's Vectors and Quaternions, I also did for some of my own types because they were move-constructible only. It sort of works. I would attempt to call nlohmann::to_json on my class's to to_json them. Because that appeared to work. (I looked at the how to make your own serializer for a small amount of guidance on implementing the functions) I did the nlohmann::to_json method because it was intuitive to me but it doesn't compile with nlohmann::to_json(json, Quaternion{}), things appear to compile when I do json = Quaternion{} though so I'm going to use that for now.

Please describe the steps to reproduce the issue.

Define a specialization of the nlohmann::adl_serializer with some type
Call the nlohmann::to_json on a type defined like this.
Coin flip if it will compile or not.

Can you provide a small but working code example?
This is a repository I set up very quickly to show off the smallest case I had of this occurring. It should be cross platform in terms of files. It's simply a main file that includes your include directory and eigen's include directory.
https://github.com/NovaSmoof/Eigen-nlohmann-conflict

What is the expected behavior?
that I can call nlohmann::to_json after specializing the adl_serializer.

And what is the actual behavior instead?
That I could not.

Which compiler and operating system are you using?

Compiler: MSVC C++ from Visual Studio Community 2019 version 16.11.10
Operating system: Windows 10

Which version of the library did you use?

I test both. Neither worked for MSVC.

[x] latest release version 3.10.5
[ ] other release - please state the version: ___
[x] the develop branch

I tested with both and it failed with both.
If you experience a compilation error: can you compile and run the unit tests?

[ ] yes
[x] no - please copy/paste the error message below

I didn't get a compilation error, but my project's settings are really weird and it wouldn't build your unit tests right and your instructions didn't work. I tried it for an hour. And if you would like me to I'd be more than happy to attempt again, but as of right now I can't get many of them to run. I had difficulties downloading the data.

You're not supposed to call to_json or from_json directly. That is, instead of calling
nlohmann::adl_serializer<Thing>::to_json(json["Thing"], thing);
nlohmann::to_json(json["Thing"], thing);
json["Thing"] = thing;

call
json["Thing"] = thing;

Making these changes let me compile your example code. As the main is empty, I have no idea what to expect running it though.

@NovaSmoof Do you need further assistance?

Note that Eigen had some issues with versions 3.10.4 and 3.10.5 (#3267). I think it had to do with the string type constraints, i.e., a similar issue as #3602.


Hsearch 2494 tika bridge bug

https://hibernate.atlassian.net/browse/HSEARCH-2494
The issue is caused by to problems:

ExtendedBridgeProviderContext#getReturnType returns only the contained type for arrays and collections but the TikaBridgeProvider expects the complete type.

TikaBridge provider should overrides the default bridge we use for arrays.

I've also added a test to check the support for URI.

@DavideD It seems checkstyle is not happy. Could you please fix the code formatting errors?

@yrodiere I think it is fixed now

@DavideD I have some doubts related to naming, see above. Apart from that, it looks good!

Renamed the way you suggested

Rebased and merged. Thanks!

Backported to 5.5.


Update reqparse.rst

Update import of RequestParser

The documentation is already valid, why do you think it needs to be changed?

Nevermind, I see it now... Thanks!


Remove t.deepEqual from enhanced patterns

This is required to make magic assert display expected output.

It should be released as a major release, so that current AVA installations won't pick it up, right?

It was introduced just a week ago in master: https://github.com/avajs/ava/commit/ad5122d507bfaa044fc3a2e51e467238e16dba11#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R91 So no AVA version is using this. But it might cause test failures in master, so better to just do a major bump to be safe.

Done, published 2.0.0.


Made what matrix utils I trivially could be generic

@shivaram

LGTM. Thanks @tomerk ! This should make some things a little better
Merging this


explore: optimize Node#at_css and #at_xpath

Currently, #at_css and #at_xpath execute the entire XPath query with multiple results, creates the NodeSet and wraps each result as a Ruby object before discarding all but the first result.
It should be possible to optimize this, both at the XPath layer and while marshalling results.
At the XPath layer, let's play with variations of (original-query)[1]
At the marshalling layer, let's discard the NodeSet and just return the single Ruby object.

Holy cow, I just discovered that libxml2 will automatically try to optimize any expression of the form (...)[1]:
#! /usr/bin/env ruby

require "bundler/inline"

gemfile do
  source "https://rubygems.org"
  gem "nokogiri", path: "."
  gem "benchmark-ips"
end

xml = "<root>" +
      (1..2000).map { |i| "<item>#{i}</item>" }.join +
      "</root>"
doc = Nokogiri::XML(xml)

Benchmark.ips do |x|
  x.report("optimized xpath") do
    result = doc.xpath("(//item)[1]")
    raise "unexpected result" unless result.size == 1
  end

  x.report("unoptimized xpath") do
    result = doc.xpath("//item[1]")
    raise "unexpected result" unless result.size == 1
  end

  x.compare!
end

reports
Warming up --------------------------------------
     optimized xpath     1.808k i/100ms
   unoptimized xpath     1.425k i/100ms
Calculating -------------------------------------
     optimized xpath     18.001k (± 1.6%) i/s -     90.400k in   5.023099s
   unoptimized xpath     13.702k (± 5.4%) i/s -     68.400k in   5.008044s

Comparison:
     optimized xpath:    18001.3 i/s
   unoptimized xpath:    13702.3 i/s - 1.31x  (± 0.00) slower

Seems like, other than having to deal with the crappy positional arguments for all the methods in Searchable, this should be an easy way to speed up at_css and at_xpath.

Hmm, actually this may not be as big of a win as I thought for some reason ...
xml = "<root>" +
      (1..2000).map { |i| "<item>#{i}</item>" }.join +
      "</root>"
doc = Nokogiri::XML(xml)

Benchmark.ips do |x|
  x.report("optimized xpath") do
    doc.xpath("(//item)[1]")
  end

  x.report("unoptimized xpath") do
    doc.at_xpath("//item")
  end

  x.compare!
end

reports
Warming up --------------------------------------
     optimized xpath     1.547k i/100ms
   unoptimized xpath     1.556k i/100ms
Calculating -------------------------------------
     optimized xpath     16.996k (± 1.9%) i/s -     85.085k in   5.008089s
   unoptimized xpath     16.038k (± 1.7%) i/s -     80.912k in   5.046467s

Comparison:
     optimized xpath:    16995.7 i/s
   unoptimized xpath:    16037.9 i/s - 1.06x  (± 0.00) slower

Weird.

OK, this has something to do with how expensive the node's context position is to calculate. If I replace //item with /root/item the benchmark shows optimization:
Warming up --------------------------------------
     optimized xpath     2.396k i/100ms
   unoptimized xpath     2.192k i/100ms
Calculating -------------------------------------
     optimized xpath     23.673k (± 2.0%) i/s -    119.800k in   5.062453s
   unoptimized xpath     21.616k (± 3.5%) i/s -    109.600k in   5.076759s

Comparison:
     optimized xpath:    23673.3 i/s
   unoptimized xpath:    21616.3 i/s - 1.10x  (± 0.00) slower

Shrug, I'm not sure it's worth making this part of at_css and at_xpath ... wrapping the query in (...)[1] doesn't seem like an automatic win.


Event Cards

[x] Collapsed card view
[x] Expandable card with full info
[x] register button
[x] display of events list

Event Cards ready for merge, works well with all the widgets


[24.11] bottles: 51.13 -> 51.15

Tests: Ran it multiple times and is already present in NisOS unstable
Package Information

Package name: Bottles
Latest released version: 51.15
Current version on the unstable channel: 51.15
Current version on the stable/release channel: 51.13

Changes: Minor changes so Latest version works in 24.11 release
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[X] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

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

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

bottles 51.17 is out and already in unstable, could you backport it instead?

Hi, I am sorry but i seriously don't know how to backport it,
I was facing a some issues with some of the dependencies in bottles and thought it might be issue with the version (Which i just went back to using flatpak),
But i don't know how to exactly backport it which is the reason i directly pushed it in 24.11 branch


Server Crash - MC-1.12.2 Forge 94 (Interaction with lootables mod) Debugging please leave open!

Crash Log
https://gist.github.com/P3rf3ctXZer0/d00c22035a9b27266dc37c66aefbe4b8
Full Log
https://gist.github.com/P3rf3ctXZer0/026e26c17d61391c96bdc29bf51fcc8f

https://github.com/MinecraftModDevelopmentMods/AdditionalLootTables/issues/9

That's a bug on their side. Wrong usage of the capability for players


Load.json & Tick.json

Fantastic project. Will be using this for a few things.
Is it possible to have a function be called upon /reload and every tick? Would be very useful.

Yes you can, like so:
include "Events"
include "Text"

[Event Events.tick]
void on_tick() {
    Text t;
    t << "tick";
    t.send_to_all();
}

[Event Events.load]
void on_load() {
    Text t;
    t << "load";
    t.send_to_all();
}


DEV-16148: Add support for object_filters: false to disable filtering and hide UI

set object_filters: false either in things/index.md front matter or objects.yaml to disable filter UI. When object_filters is not specified in either place, filters for all object properties will still render as before.

@mph changes in comments have been applied + tested


Error while building the documentation locally

I am trying to build the documentation locally by following the instructions given in
But I face the below:
UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 109702: character maps to <undefined>
while executing pydoc-markdown
If i try yarn start (without the previous step i.e, pydoc-markdown - I assume this is not the right order to execute though) I see the below error:

Please can someone help?

Are you using the instructions in https://microsoft.github.io/FLAML/docs/Contribute/#documentation?
What version of pydoc-markdown are you using?

I tried with pydoc-markdown==4.5.0 and saw the same error message.


Serverless prototype

The goal of this prototype is to implement a first version of serverless feature into amp. The proposal is to have a Faas (Function as a Service) built into AMP.
Story
As a user, I want to trigger a function, i.e. a container when I receive an HTTP POST on a given route.
Scenario
In order to achieve this story, we need 2 elements:

A container which will execute the function
Register this container as a function in amp

The function
In order to keep a maximum of flexibility on the technology used for functions, standard streams are used to pass function input and output parameters:

stdin : where the function gets its input parameter
stdout: where the function can produce a result
stderr: where the function logs

For instance, a function can be written like this:
package main

import (
	"fmt"
	"io/ioutil"
	"log"
	"os"
	"strings"
)

func main() {
	input, err := ioutil.ReadAll(os.Stdin)
	if err != nil {
		log.Fatal("Unable to read standard input", err)
	}
	log.Println("input", string(input))       // Logs on standard error
	fmt.Println(strings.Title(string(input))) // Writes response on standard output
}

This source code of the function is available here: amp-demo-function
Since functions are containers, the function developer needs to wrap his function inside a container. Let's consider the above snippet has been containerized in an image called amp-demo-function
Registering the function
In order to register a function against amp, the user would need to do the following command:
$ amp function create -i image-name function-name
<function-id>

Invoking a function via HTTP
In order to invoke a function, you can POST an HTTP request to function.local.axwaytest.net/<function-id>. The body will be transferred to the function on its standard input.
The call is blocking until the function sends a response.
$ cat Makefile | curl function.local.axwaytest.net/[function-id] --data-binary @-

Architecture

The idea is to have 2 new amp binaries:

amp-function-listener
amp-function-worker

amp-function-listener
This service role is to:

listen to HTTP events and send function request to NATS
listen to NATS for function responses

amp-function-worker
This service role is to:

listen to NATS events,
spawn container
pass the content of the message through standard input
get the response as standard output (if any)
post the response back to NATS

amplifier
The function API is served by amplifier and function information are stored in etcd.
We need to be able to add, list and remove functions dynamically.
amp CLI
amp CLI needs to be updated to support functions commands.
We need to be able to:

create a function
list functions
remove a function

Observations

Maybe we should not use NATS-streaming but NATS directly (without storage) since I'm not sure we will replay function calls.
We can have a better NATS usage if we know beforehand whether or not the function produces a return value (fire and forget)

This is well described @bquenin and makes sense to me and I think is a good place to start. I realize this is just in prototype phase and not sure what outcome we are expecting, but the topic of serverless computing immediately brings the following concerns to the forefront, in my mind:

How to handle response data (correlation id? callbacks? synchronous capability?)

Implementation of Claim Check pattern

Retries and Dead Letter Strategy

Should we create a cannonical message format?

Should we create a comprehensive catalog of all messages, their functions and rules?

Monitoring performance and backlog

I guess the question boils down to: how much onus do we place on AMP to solve these concerns versus bring your own?

@chrisccoy Thanks for your feedback Chris. I don't have all the answers and this prototype is definitely here to start discussions like this! Here is where I stand today:

Handling response data should be done via the natural symmetric of standard input, i.e. standard output. It may conflict with logs but we could standardize to use standard error for logs for instance.
Implementation of Claim Check pattern: I'm not sure I understand this pattern to its full extent, but all messages are stored in NATS making them available for reprocessing if necessary.
Retries and DLS: Again, message are stored in NATS so we can make sure the worker acknowledge messages only when it gets a response (if required). In case of a failure, another worker is able to take over.
Canonical message format: Yes, I think we should provide a structure for the input message and output messages. The structure of this message should be simple and probably mimic an HTTP request and an HTTP response.
Catalog of messages: I'm not sure since our message format should be pretty generic. The fine structure of the message is ultimately designed by the function developer.
Monitoring and performance: You're absolutely right. I'm not sure it's going to be covered by this prototype, but we definitely need to know the pressure on the NATS queue for instance in order to know if we need to scale the function workers.

I think function development in AMP should be code and forget, as in, AMP must be able to handle as much as possible in order to have a developer experience as smooth as possible.

@bquenin, nice work on #539. I have been researching the serverless architecture provided by ironio and it seems very popular to be using dind Docker In Docker
This might be something we want to research as well. It requires running the FaaS container in privileged mode, but seems well suited for this use case.

Hi @chrisccoy, I know about dind but I'm not sure we need to use it, we can use the actual docker daemon. What do you think are the pros/cons?

Honestly, difficult to say. For starters, I like the encapsulation aspect of dind; spun up containers don't hang around on the main daemon. Also, it is easy to cleanup, by simply removing the FaaS container(s).
However, I like the way you can attach to the socket and essentially "callback" to the main daemon, as the prototype does, but I think it warrants investigation, because it could be a bit messy.


HTTPS

HTTPS protocol included in instructions

Thanks @thaman1602!
I will merge now and add your name to the contributors' list of solids4foam at https://www.solids4foam.com/about/CONTRIBUTORS.html.


[CHALLENGE SUBMISSION]

Please provide the following information for your submission:
What is your name? (First, Last)
Chris Petit
Where is your GitHub Repo ?(Link to your challenge solution)
https://github.com/ChrisPetit/LovelyLadoos
What Challenge is this for? (Challenge number between 1 and 7)
2
(Optional) Anything else we should know? e.g., language used, location, blog post, tell us how you solved this solution, including any challenges you faced solving it.
 There is always the option to create and use your own models, but since this is serverless the most easy option is the best I Think.
Then I've Published a ASP.NET core MVC webapp to Azure...I'm sure there was an easier way, but I've challenged myself to solve al challenges using C#.
So basicly the logic for uploading a File or sending an Url to the API of CustomVision is in the FileUploadController.cs I know the name could be improved upon.
Through the index page we get variables, files and url, we check for nuls, create a httpRequestMessage, perform the request using a HttpClient and display the response through a razor page...
I've tried to keep the naming of the methods in the file pretty clear to make it understandable.
To build this WebApp open the Solution *.sln in Visual Studio 2019.
The Result of my WebApp is published in Azure WebApps and running serverless there:
https://lovelyladooswebapp20201203194115.azurewebsites.net/

From all of us at Developer Relations at Microsoft, thank you for participating in #SeasonsOfServerless and submitting a solution!

Feel free to share your solution on Twitter with the tag #SeasonsOfServerless.

Thanks for your awesome submission!
🎉 We've now added you and your solution to the Hall of Fame. 🎉


Upon enable makes Godot go through infinite bootloop

Exactly as title says. I enable the plugin and it causes my Godot to reset infinitely, and I have to shut down my computer to stop it.

Are you using Godot 4.2?
Also if this happens, you should be able to Alt+F4 right after launch.

Using 4.1.3 and the same problem happens here - on both the Steam and standalone versions.

The plugin requires Godot 4.2.
Until stable release (which will be very soon) you can get it here: https://godotengine.org/download/archive/

Yup, I checked, the plugin works normally on 4.2 RC1. I understand that it was built with 4.2 in mind and breaks with anything below, but I would also love for it to be usable in 4.1 if that's possible.
Regardless it's very nice and I appreciate your efforts in developing it - it's exactly what I needed and I was able to get it plugging with an existing player scenes and some level scenes quickly! Hope you have a nice day!

If you want it usable in 4.1, replace all Vector2i.MAX references to Vector2i(999999, 999999) (same with Vector3i and MIN if exists).

Confirmed that this works and allows the plugin to load in 4.1.3 normally. Thank you very much!

a60db99928513897c9bde93aef37d916138da8dd fixes infinite loop. 4.2 stable is planned to release tomorrow soon, so this is issue is no longer relevant.
Refer to the comment above if you still want to use Godot 4.1 for whatever reason.


Remote Substreams docs

[x] simplify _app.tsx
[x] simplify pages/[locale]/index.mdx (use better approach than proposed in https://github.com/graphprotocol/docs/issues/344)
[x] setup remote MDX for https://substreams.streamingfast.io/ https://github.com/streamingfast/substreams/tree/develop/docs

⚠ Sometimes we can get API rate limit from https://api.github.com/repos/streamingfast/substreams/git/trees/develop?recursive=1 (URL for getting all available docs files).
Let's test it for now as is and in the future we can fetch this data only once while building our app
error - Error: {
 "message": "API rate limit exceeded for <IP_ADDRESS>. (But here's the good news: Authenticated requests get a higher rate > limit. Check out the documentation for more details.)",
 "documentation_url": "https://docs.github.com/rest/overview/resources-in-the-rest-api#rate-limiting"
}

@B2o5T – Something else that I noticed: the substreams pages don't set the current page's title in the <title>. It's always just The Graph Docs, as opposed to e.g. Running Substreams - The Graph Docs.


MultiComboBox - Click item will select it, but also closes the dropdown at the same time

https://ui5.sap.com/1.108.11/#/entity/sap.m.MultiComboBox/sample/sap.m.sample.MultiComboBoxClearIcon
It would be better to keep the dropwon list open when clicking items and only closing the list when clicking outside of the control.
The current behavior is desired for a (single select) ComoBox, but not for a MultiComboBox.

Hello @piejanssens,
If you click on the Checkbox while selecting items, the dropdown stays open and is being closed on clicking outside of the control.
Otherwise, only if you click on the List Item while selecting items the dropdown is being closed.
Kind Regards,
Yana

I don't agree w ith 'invalid' / 'consulting'.
This is a bug IMO. Can you reconsider and discuss internally?

@piejanssens According to https://experience.sap.com/fiori-design-web/multi-combobox/#option-list

The option list (7) contains a list of selectable options (5). Clicking the label of an entry closes the option list and creates a token for the selected option. To enable multi-selection, every entry also has a checkbox (4). Clicking a checkbox creates a token. The option list remains open.

I guess you'd have to extend the control if the option list is supposed to stay open.


Bump Local Core version to 8

Bumps the version to 2.1.1
Bumps Local Core version to 8

Testing

Testing with Local 8.0.0 passed
Selecting a cloud backup from Dropbox worked:

…with no error in the console:

There are still a lot of linting errors from yarn lint. But those are outside the scope of this PR.

Ready for review if you have time!

CircleCI isn't activated for this repo.

Thanks for reviewing this, @bgturner!


Created a local reflection lock in order to work with spark 2.3.0.

Patch to make the connector work with Spark 2.3.0.
In Spark, this commit https://github.com/apache/spark/commit/425c4ada4c24e338b45d0e9987071f05c5766fa5 removes Scala 2.10 support. One of the things removed is the workarounds using ScalaReflectionLock, which causes a java.lang.NoClassDefFoundError when using the connector.
This pull request can potentially help resolve https://datastax-oss.atlassian.net/projects/SPARKC/issues/SPARKC-530.
While removing Scala 2.10 support might be the right way to go, as Spark seems to be going in this direction, it turned out to be a much more intrusive change.

Please post PR's on the corresponding Jira Ticket as we do not check Github Pull requests without corresponding Jiras.
Below is mostly copied from other PR for this workaround.
The main issue with this PR is that the fix is a workaround for the missing class. This will break compatibility with Spark 2.0-2.2. I think the correct thing to do here is to completely remove the SparkReflectionLock from the Spark Cassandra Connector as well as the Scala 2.10 compatibility code. This would bring our testing back up to green and we wouldn't have to worry about this issue in the future.
So in my eyes a Spark 2.3.0 branching event requires
Removal of SparkReflectionLock
Removal of Scala 2.10 support
All supported integration tests run green (Hopefully this is just free after removing Scala 2.10)

Yeah, that other pull request came after mine. As you can see it has my commit in there and the guy thought he wanted to improve it. I raised my concerns about modifying dependencies unrelated to what the PR is doing when he emailed me to inform me about it.
I completely agree with how the branching has to be approached. I've even stated it in my description.
There is a branch in my fork which does exactly that, but the change seems quite intrusive and I abandoned it for the time being. I actually needed some kind of confirmation by someone from DS to make sure I don't waste my time going that route.
It's a shame that you do not check Github - doesn't look like there is much noise in here anyway. If you had, that would have already been done.
I will see if I get some time to complete the work in my fork. If not, it will have to be someone else.

Created https://github.com/datastax/spark-cassandra-connector/pull/1175.


Crash on restore from background

Hi, I faced a problem, when I call Alert the Popup is appeared.
Then I minimize application, open about 10 different applications,
restore my application and got the crash.
Using Android 4.4.4 version
also reproduced on the syanogen Android 6.0.1
Here is the stacktrace
Caused by: android.runtime.JavaProxyThrowable: System.Collections.Generic.KeyNotFoundException: The given key was not present in the dictionary.
03-01 17:02:50.558 15291 15291 E AndroidRuntime:   at System.Collections.Generic.Dictionary2[TKey,TValue].get_Item (TKey key) [0x00021] in <2f4e091071e147db85c8b531e653aeba>:0  03-01 17:02:50.558 15291 15291 E AndroidRuntime:   at Acr.UserDialogs.Fragments.ConfigStore.Pop[T] (Android.OS.Bundle bundle) [0x0000d] in <74ea0cacc10942059c2f2e1dc6aab924>:0  03-01 17:02:50.558 15291 15291 E AndroidRuntime:   at Acr.UserDialogs.Fragments.AbstractAppCompatDialogFragment1[T].OnCreateDialog (Android.OS.Bundle bundle) [0x00013] in <74ea0cacc10942059c2f2e1dc6aab924>:0
03-01 17:02:50.558 15291 15291 E AndroidRuntime:   at Android.Support.V4.App.DialogFragment.n_OnCreateDialog_Landroid_os_Bundle_ (System.IntPtr jnienv, System.IntPtr native__this, System.IntPtr native_savedInstanceState) [0x00011] in <27c17fe440cf491ba8255bcefade6e02>:0
03-01 17:02:50.558 15291 15291 E AndroidRuntime:   at (wrapper dynamic-method) System.Object:1e0553ac-733e-4b5a-aa52-516b1c0d0daf (intptr,intptr,intptr)

Use the contribution guide (the one you deleted before entering your text) if you want help please.
Also, please include any code you are using.

I've fixed the description, please review the issue.

You should use the cancellation token within the async method OR dispose of the dialog (sync version) when the app goes to the background.  This will be a good workaround until I get around to fixing this

Thank you very much for your reply. I'll try the workaround you suggested.


Fetching data from grpc service

Product
Hot Chocolate
Is your feature request related to a problem?
I have a grpc service with well defined proto definitions. I would like to expose some of that data over graphql while using the same type definitions I have in proto.
The solution you'd like

Generate graphql schema from proto definitions
Fetch data from grpc service and transform results to the graphql schema.

This is on our internal backlog for fusion.

+1 for this

@michaelstaib is there any ticket or documentation I can refer to?


[BUG] MS connection when powershell module disabled

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions
[X] This issue is about a bug (if it is not, please use the correct template)

Describe your issue
connects to microsoft domain even if powershell module disabled
see command used below:
Steps to reproduce the issue
settings > powershell: disabled
quit app
reopen app
=> app attempts connection to go.microsoft.com by querying repos (Get-PSRepository)
WingetUI Log
N/A

Package Managers Logs
█▀▀▀▀▀▀▀▀▀ [28 March 2024 12:12:44] PowerShell ▀▀▀▀▀▀▀▀▀▀▀
█  Executable: C:\WINDOWS\system32\windowspowershell\v1.0\powershell.exe
█  Arguments:  -NoProfile -Command Get-PSRepository

Relevant information
expectation: app shouldn't connect to MS to query powershell modules repos if powershell is disabled in settings
Screenshots and videos
No response

Please share your entire logs

because it's too much to redact about the software i have installed (I kinda care about my online privacy). Sorry, i hope it helps. It should be easily reproductible though.

Then please redact it all, but I need to know which managers are running and which commands are they calling with which frequency, as well as the WingetUI logs, so I can see the base state of each manager and other useful info

At least share WingetUI Logs, there shouldn't be much to redact there, and I will be able to get advanced details about the package

I was able to reproduce the bahavior without any more logs aswell

Hello,
I am not trying to annoy anyone by asking for logs. I have tons of issues to debug and fix, as well as other features, and I obviously have to do other things in my life. The time I have available to build WingetUI is very limited, and I don't want to waste it tweaking my pc to recreate an issue I haven't succeeded to recreate, because no one wanted to upload their logs.
I understand your concerns with privacy, but the things shown on the logs, especially WingetUI logs, will not destroy your privacy. The shown API token, which is used for WingetUI Widgets, is generated each time WingetUI is restarted. And the next big issue are paths, from which you easily can remove your username.
I am sorry, but I will not invest more time on this issue until you share your logs.
I understand that your time is precious, but so  is mine. You are facing an issue you want to get fixed, and trust me, I am willing to fix it, but I will need your collaboration to do so.
If you choose to not share your logs, I will close this issue. As said, I did not manage to recreate the issue locally.

Thanks!


Added note about the new Gamestateintegration

Gamestateintegration has been disabled by default by valve, due to performance issues - This commit adds a note on how to enable it

I'am using Dota2Gsi in my development atm and all I did was added the config file.  I did not add the launchoption that you are describing. So is the launchoptions really required?


Unexclude runtime/ErrorHandling/*.java tests on JDK8

Problem has been fixed for JDK8 by JDK-8218613.
Issue: https://github.com/adoptium/aqa-tests/issues/121
Tests passed (aarch64_linux):
https://ci.adoptium.net/job/Grinder/7041/

Fixed merge conflict with https://github.com/adoptium/aqa-tests/pull/4478.

I'll reschedule tests

Tests passed (aarch64_linux):
https://ci.adoptium.net/job/Grinder/7052/
https://ci.adoptium.net/job/Grinder/7053/

Fixed another merge conflict. :)


Syntax error cancels auto-complete

Godot version: 3.2.stable
OS/device including version: 5.5.2-arch1-1
Issue description:
Auto-completion does not works when there are one or multiple errors in the script
Steps to reproduce:

Create a new script
Make a typo
Start typing something else and Ctrl + Space to trigger auto complete
Nothing shown
Fix typo
Repeat step 3
Auto complete works

Minimal reproduction project:
Any project, new script with the following content:
extends Node

expot (PackedScene) var typo

func _re

I'm fairly sure this has already been reported, but I can't find the issue in question.

Similar to https://github.com/godotengine/godot/issues/33191

This is a duplicate of #33191 indeed. Thanks @Zylann :slightly_smiling_face:


kpod: WIP info subcommand

Design: The output of the info subcommand ought to be directly
consumable in a format like JSON.
The structure being a map of sorts.
Each subsection of information being an individual cluster under the
top-level, like platform info, debug, storage, etc.
Even if there are errors under the top level key, the value will be a
map with the key of "error" and the value as the message of the
err.Error(). In this way, the command always returns usable output.
Ideally there will be a means for anything that can register info to do
so independently from it being in the single info.go, so this approach
is having a typed signature for the function that gives info, but i'm
sure it could be better.
Current iteration of this outputs the following as a limited user:
{
  "host": null,
  "store": {
    "error": "mkdir /var/run/containers: permission denied"
  }
}

Signed-off-by: Vincent Batts<EMAIL_ADDRESS>

design feedback please

@vbatts in other kpod PR we have a --json flag to have machine parsable output (I'll keep reading the first comment now)

but overall, json output with errors in maps looks good to me - we just need to agree on that being a flag or info always outputting json (I believe the latter).

@runcom i thought about that, like a --format//-f for json, yaml, etc

regardless, it ought to be deterministically parseable

I'm adding json output to kpod images right now and I am adding a -j flag.

@vbatts if you like, i could just do json for this one too?

is --json really preferable to --format json?

only bc I dont plan to do yaml or otherwise :)

but for this feature, i'm not sure what "otherwise" would be. if json is the default, then what does the --json flag do? ;-)

Should the default output be --human?

yaml is pretty close to what human could be.

Our goal with this is to follow the "docker CLI" standard since docker info outputs simple output, and have --json output in json format, for tools like atomic.

docker info have a --format json
On Thu, Jul 13, 2017 at 2:16 PM, Daniel J Walsh<EMAIL_ADDRESS>wrote:

Our goal with this is to follow the "docker CLI" standard since docker
info outputs simple output, and have --json output in json format, for
tools like atomic.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/cri-o/pull/652#issuecomment-315159712,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEF6S1yvcus07KgcR2baN5Wp023HtpRks5sNl7jgaJpZM4OXHpp
.

@vbatts It doesn't, they do have a -f but it is for specifying a go template. Their default output looks quite close to yaml.

oh! you're right, it's a go text template formating. well hrm.
On Thu, Jul 13, 2017 at 2:19 PM, Mrunal Patel<EMAIL_ADDRESS>wrote:

@vbatts https://github.com/vbatts It doesn't, they do have a -f but it
is for specifying a go template. Their default output looks quite close to
yaml.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/cri-o/pull/652#issuecomment-315160618,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEF6cvjwvo0A9ABDk2dXA2LSTAc9zfYks5sNl-3gaJpZM4OXHpp
.

so i'll see what a default of yaml looks like, with a --json
On Thu, Jul 13, 2017 at 2:23 PM, Vincent Batts<EMAIL_ADDRESS>wrote:

oh! you're right, it's a go text template formating. well hrm.
On Thu, Jul 13, 2017 at 2:19 PM, Mrunal Patel<EMAIL_ADDRESS>wrote:

@vbatts https://github.com/vbatts It doesn't, they do have a -f but it
is for specifying a go template. Their default output looks quite close to
yaml.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/cri-o/pull/652#issuecomment-315160618,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEF6cvjwvo0A9ABDk2dXA2LSTAc9zfYks5sNl-3gaJpZM4OXHpp
.

updated output thoughts.

ok, this is usable enough to get started.
Please review.

added kernel and uptime

LGTM

Missing:
Man pages  (kpod and kpod-info)
Update README.md
bash completions.

@rhatdan added docs.
All PTAL

Needs rebase now

rebased

not sure what failed?

i/o timed out trying to get image. I have restarted the job.

LGTM

@runcom PTAL

LGTM


⚠ Bofrost.be has degraded performance

In cb682b6, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 992 ms

Resolved: Bofrost.be performance has improved in 15a693f.


Proper Block Inscriptions HTML

See #2329

Getting: thread 'tokio-runtime-worker' panicked at 'index out of bounds: the len is 0 but the index is 36849', src/index/block_index.rs:177:25

Getting: thread 'tokio-runtime-worker' panicked at 'index out of bounds: the len is 0 but the index is 36849', src/index/block_index.rs:177:25

Improved error handling for when BlockIndex has not been initialized (happens on initial server startup, when index is initially catching up on new blocks). Message is now Failed to get inscriptions in block: Block index not yet initialized


[docs] add docs on function form of markdown arg to takeover remark pipeline

as requested to document https://github.com/contentlayerdev/contentlayer/issues/413

It would have been great if contentlayer exposes the option somehow as I would have to keep the pipeline in sync with upstream changes whenever/if anything changes.


Competing Language Configurations

Hi - I realize the issue tracker is not for questions, however, I've posted this question on SO, poured over Google search results, and asked the question in the Gitter chat room, all to no avail (with no responses), so I'm hoping to resolve this question with some insight from the developers / contributors, so I can finally move on. Your indulgence would be greatly appreciated (and then, of course, feel free to close the issue).
My question is around competing language configurations. I am building an extension, however the users of my extension are sure to have other extensions enabled which provide language configs for this particular language. So, how is this handled?
I am simply trying to add an auto-closing pair, however, when debugging and packaging the extension locally, I cannot seem to get it to work. I am certain I am using the correct language ID, and the "Contributions" section of the extension shows that I'm contributing the language:

My initial approach was to copy the language config from the other extension contributing this language, and just add the auto-closing pair, but I also tried just adding in only the auto-closing pairs section - neither seems to work.
My understanding from the docs, is that language configs can "enrich" VSCode's knowledge of a language it already knows about, so this suggests multiple config files should be OK. But what happens in the event that they both try to configure the same thing? Is that what's happening here? It's seems like its only taking auto-closing pairs from one config.
Apologies if I'm not being clear enough, please also see the SO question I linked above, perhaps that's clearer. If someone can offer some insight on this, that would be much appreciated.
Thanks.

@jrieken Any thoughts on this? As an update, I'm sure that my configuration file on its own is perfectly fine. When I disable the competing extension it works perfectly. So maybe the better question is: is there any way to augment an existing language configuration from a competing extension?

You have reverse-engineered it correctly :).
The behaviour of conflicting language configurations (multiple extensions contributing a language configuration for the same language) is not something handled especially, so the behaviour is emergent and not designed.
The current behaviour is that extensions are scanned and sorted alphabetically by their name (the property in package.json). Extensions are split into 3 buckets: built-in, user-installed and under-development. Inside each bucket they are sorted by their name and the buckets have the above order.
In this order, each extension is analyzed and its contributions processed. Ultimately, the language configuration is a map from language id to configuration which ends up overwritten by the later extensions.
If you really really want to win, you can define your configuration in code and use vscode.languages.setLanguageConfiguration to set it, which in principle should win (as it would execute later). But even that might suffer from a race condition, because when you first open a certain language, we then read the configuration file from disk asynchronously. So if your extension is active before the language is first encountered, it is possible that the configuration file might win.
I'm sorry that this conflicting case is not really thought through...


bonghab

Describe the problem and steps to reproduce it:
(Please include as many details as possible.)
What happened?
What did you expect to happen?
Anything else we should know?
(Please include a link to the page, screenshots and any relevant files.)

Describe the problem and steps to reproduce it:
(Please include as many details as possible.)
What happened?
What did you expect to happen?
Anything else we should know?
(Please include a link to the page, screenshots and any relevant files.)


Add a (very) short example test to the repo readme.

This PR adds an example test to the repo readme so that readers can get a sense of what swift-testing looks like without digging into the documentation.
Checklist:

[x] Code and documentation should follow the style of the Style Guide.
[x] If public symbols are renamed or modified, DocC references should be updated.

@swift-ci please test


Error loading account history

Having problems lately with bids forwarded  into the next round not getting up-vote or refund and log showing:
Sun Jun 24 2018 18:50:00 GMT-0700 (PDT) - Error Count: 1, Current node: https://api.steemit.com
Sun Jun 24 2018 18:50:00 GMT-0700 (PDT) - Error loading account history: Error: HTTP 504: Gateway Time-out
Sun Jun 24 2018 19:00:19 GMT-0700 (PDT) - Loaded STEEM Price from Bittrex: 1.306426878
Sun Jun 24 2018 19:00:19 GMT-0700 (PDT) - Loaded SBD Price from Bittrex: 1.0546405457999999
Sun Jun 24 2018 19:09:52 GMT-0700 (PDT) - Error Count: 1, Current node: https://api.steemit.com
Sun Jun 24 2018 19:09:52 GMT-0700 (PDT) - Error loading account history: Error: HTTP 504: Gateway Time-out
Sun Jun 24 2018 19:11:11 GMT-0700 (PDT) - Error Count: 2, Current node: https://api.steemit.com
Sun Jun 24 2018 19:11:11 GMT-0700 (PDT) - Error loading account history: Error: HTTP 504: Gateway Time-out
Sun Jun 24 2018 19:11:41 GMT-0700 (PDT) - Error Count: 3, Current node: https://api.steemit.com
Sun Jun 24 2018 19:11:41 GMT-0700 (PDT) - Error loading account history: Error: HTTP 504: Gateway Time-out
Sun Jun 24 2018 19:30:19 GMT-0700 (PDT) - Loaded STEEM Price from Bittrex: 1.2989972753
Sun Jun 24 2018 19:30:19 GMT-0700 (PDT) - Loaded SBD Price from Bittrex: 1.048283353

@bitcoiners the errors show that the node is throwing timeout errors, not sure this is related to the bids not getting votes or refunds. I recently pushed an update to switch from the steem js library to the dsteem library which seems to run much better.


[install-suse-cloud] log command line args

We need to move the logging of command line args before we parse and
shift them.

@MaximilianMeister that's a good point. I redid the patch a bit. What do you think?

@rsalevsky Sorry, I keep forgetting about that format :)

Backport at https://github.com/crowbar/crowbar/pull/2293


When can we expect support for guzzle 7?

I tried installing this on Laravel 8 and there is a version clash with guzzle 7. Check out the output below.
  Problem 1
    - Installation request for guzzlehttp/guzzle (locked at 7.0.1, required as ^7.0.1) -> satisfiable by guzzlehttp/guzzle[7.0.1].
    - bandwidth/iris v2.5.1 requires guzzlehttp/guzzle ~6.0 -> satisfiable by guzzlehttp/guzzle[6.5.x-dev].
    - bandwidth/iris v2.5.2 requires guzzlehttp/guzzle ~6.0 -> satisfiable by guzzlehttp/guzzle[6.5.x-dev].
    - bandwidth/iris v2.5.3 requires guzzlehttp/guzzle ~6.0 -> satisfiable by guzzlehttp/guzzle[6.5.x-dev].
    - Conclusion: don't install guzzlehttp/guzzle 6.5.x-dev
    - Installation request for bandwidth/iris ^2.5 -> satisfiable by bandwidth/iris[v2.5.1, v2.5.2, v2.5.3].

Hey @ryantxr, you should be good to use v3.0.0 of this package. Let me know if you have any issues


fix: avm/ptn/data/private-analytical-workspace - failing tests with random locations

Description

Fixes #3307
Pipeline Reference

Pipeline

Type of Change

[ ] Update to CI Environment or utilities (Non-module affecting changes)
[x] Azure Verified Module updates:

[x] Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] Breaking changes and I have bumped the MAJOR version in version.json.
[ ] Update to documentation

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] I have run Set-AVMModule locally to generate the supporting module files.
[x] My corresponding pipelines / checks run clean and green without any errors or warnings

Hey @jbinko,
the PR looks good from my side. There just seems to be a small issue with 2 post deployment tests that need fixing. Then we should be good to go. Thank you for all the effort 💪

Hey @jbinko, the PR looks good from my side. There just seems to be a small issue with 2 post deployment tests that need fixing. Then we should be good to go. Thank you for all the effort 💪

Thanks @AlexanderSehr - Yes, I was working on it meanwhile. :-) It was Azure backend complaining about some not fully removed resources from the previous run. Now it should be fine.

Great work. Thanks @jbinko. Off we go 🚀


STP - don't print entire infotons to agents.log

When infoton failed to be ingested/materialized, the entire infoton is printed to agents.log with proper failure reason. It loads the log file and makes it unreadable, It should not be printed in agents.log

Verified and closed


cmark-gfm: 0.29.0.gfm.0 -> 0.29.0.gfm.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/github/cmark-gfm/releases.
meta.description for cmark-gfm is: "GitHub's fork of cmark, a CommonMark parsing and rendering library and program in C"
meta.homepage for cmark-gfm is: "https://github.com/github/cmark-gfm"
meta.changelog for cmark-gfm is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

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

/nix/store/2aa0lg261x1lbnkdx6csx8svr65mrhq7-cmark-gfm-0.29.0.gfm.1/bin/cmark-gfm passed the binary check.

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

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

found 0.29.0.gfm.1 with grep in /nix/store/2aa0lg261x1lbnkdx6csx8svr65mrhq7-cmark-gfm-0.29.0.gfm.1

found 0.29.0.gfm.1 in filename of file in /nix/store/2aa0lg261x1lbnkdx6csx8svr65mrhq7-cmark-gfm-0.29.0.gfm.1

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

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

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

8 total rebuild path(s)

2 package rebuild(s)

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

First fifty rebuilds by attrpath
cmark-gfm
mindforger

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2aa0lg261x1lbnkdx6csx8svr65mrhq7-cmark-gfm-0.29.0.gfm.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2aa0lg261x1lbnkdx6csx8svr65mrhq7-cmark-gfm-0.29.0.gfm.1
ls -la /nix/store/2aa0lg261x1lbnkdx6csx8svr65mrhq7-cmark-gfm-0.29.0.gfm.1/bin

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

2 packages built:

cmark-gfm
mindforger

Maintainer pings
cc @cyplo for testing.

Result of nixpkgs-review pr 137978 at 898bc910 run on aarch64-linux 1

2 packages built successfully:

cmark-gfm
mindforger


API Returning out of time data

Api returns outputs but they aren't the ones from the correct time range

It was querying the wrong lead times from DB


User image is not visible in header icon

Describe the issue:
The User icon is not loaded in the header icon even though the user has an icon selected.

How to reproduce:

Create a User with general details
Before creating the user, select an image from the final step of the user wizard
Login with the new user

Expected behavior:
The selected image should appear in the header icon
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 6.0.0 RC2
OS: MacOS

Optional Fields
Related issues:

Suggested labels:

There is no way to select an image for an user.


Need a list of users that are ready for a call.

If there is a list of users that are ready for a call (ready function called) then the caller needs to see a list of phone numbers that he can make a call to.

Good idea.  To confirm, you are asking about an Address book or a Call Lobby ?
On Mon, Dec 17, 2018 at 4:31 AM Dainius Kreivys<EMAIL_ADDRESS>wrote:

If there is a list of users that are ready for a call (ready function
called) then the caller needs to see a list of phone numbers that he can
make a call to.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stephenlb/webrtc-sdk/issues/66, or mute the thread
https://github.com/notifications/unsubscribe-auth/AACwnrK0gYwEEyj68o1nWBxQuChCvppqks5u546_gaJpZM4ZWQ4y
.

--
Stephen Blum - CTO
https://www.linkedin.com/in/stephenlb/

Yes


Don't hot-loop on cert renewals when the ca expiration gets close

The NotBefore field in the certificate does not container the certifcate
issue date. It contains the issue date of the CA. This led to wrong
calculations of the certificate rotations. The closer we came to the CA
expiration date, the faster they were rotating.
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/cc @davidvossel

/cc @mhenriks

Adding more unit tests now.

/cherry-pick release-0.30

/hold
Want more testing.

/retest

/test all

/hold cancel

@stu-gott @mhenriks @davidvossel ready for review.

/lgtm

/retest

/cherrypick release-0.30

/cherrypick release-0.30

/cherrypick release-0.30


Add interpreter for DivideOp

closes #971
Add a binary op version of mapWithUpcastToDouble, add support for int and bool functions in mapWithUpcastToDouble functions.

Also we can add a simple testing strategy similar to https://github.com/openxla/stablehlo/pull/1079#issue-1574895499

Linking issue: Make sure that examples in the spec/ODS conform to spec_checklist.md


List "Azure Container Instances" under "Azure Services"

In the very last section of the page where it lists "Azure Services" it does not have a buttle for "Azure Container Instances". Probably want to add that :)

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

ID: e845d11e-5663-dfc5-93a5-e973c8ac4df9
Version Independent ID: 1b1e7269-30f7-19c5-2ee3-f89ab0add365
Content: Introduction to Docker - .NET Core
Content Source: docs/core/docker/intro-net-docker.md
Product: dotnet-core
Technology: dotnet-docker
GitHub Login: @Thraka
Microsoft Alias: adegeo

Thanks! I'll add that!


Serialization of double and float values not accurate enough

Hello,
while testing adding documents with attributes of various data types to Solr, I have found out that decimal values are rounded when serializing to XML. It seems like that SolrNet uses the ToStringFieldSerializer for such values and calls the toString() method to convert it to string. The toString() method cuts off some digits behind the comma and does some rounding.  However, by calling toString("R"), the full precision would be preserved.
In my example I have used double.MinValue which equals to -1.7976931348623157E+308. When calling toString() the result is 1.79769313486232E+308 which is lower then the minimum value and therefore -infinity is stored in Solr.
Is there any possibility to change the to string behaviour? E.g., by injecting another FieldSerializer to call toString("R") instead?
Additional information about my problem can be found on stackoverflow: https://stackoverflow.com/questions/50722770/solr-scientific-notation-of-double-cannot-be-parsed

To Solve this problem, I have implemented an own DoubleFieldSerializer (derived from AbstractFieldSerializer) which calls obj.ToString("R") in the Parse function instead. I did the same for Float values.
A drawback of my solution is that I need to override the DefaultFieldSerializer to be able to register my FieldSerializers. Would be great when this would be implemented as default behavior.

One additional point you need to take care of is that the toString() method formats the number according to the applications culture settings.

Hello @WSt89 let me check with Gidon
@gjunge what are your thoughts

Seems like a good addition. @WSt89  would you mind creating a Pull Request for this?


Changes in Typing Speed Test Game

PR Description 📜
I have changed background color and added hover effects to enhance user experience
Fixes #2801

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ ] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[ ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸
Earlier
https://github.com/kunjgit/GameZone/assets/127978268/f69acef0-3b3f-41ff-a2bb-0b0b71382b73
Now
https://github.com/kunjgit/GameZone/assets/127978268/7102070f-6980-4b88-8272-538cef4645fa

Thank you soo much for contributing to our repository 💗

Thank you  @akankshachanana1 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Adding button for jump to realtime bar

Does the lib have built-in feature to add button for jump to real-time bar like the button in the Binance app? Or is there a way to do it?

Desc
Photo on the Binance app

The real-time bar doesn't touch the yAxis

The real-time bar is over the yAxis

Special features, no development plan for the time being.

how to jump to specific time ? @liihuu


Find packages, using Manifest.in, and a small fix in setup.py

As discussed in issue #78, this PR adds:

package autodiscovery
data management through MANIFEST.in

Also, while looking at setup.py, I found that author_email was mentioned again in the bottom comments, but it had been already defined some lines above.

We ran into the packaging issue a while back on Fractal and it worked there. As for conda-build things, they install with pip install . and python setup.py install in 95+% of cases, which this should catch. We do some testing here thanks to @dgasmith clever emulation of the cookiecutter deployment and installing, so I'm fairly it will be safe, @loriab. If not, you can shame me for being wrong and we'll fix it!

Thinking more about your comment, what line in setup.py did conda-build start recommending?


can't install environment successfully use requirement.txt

such as cant install torch-sparse...

For installing pytorch_geometric and its related packages (e.g., torch-sparse), please refer to: https://pytorch-geometric.readthedocs.io/en/latest/notes/installation.html#

Or, you can directly copy the docker environment provided in: https://github.com/acbull/pyHGT/tree/master/Docker

Or, you can directly copy the docker environment provided in: https://github.com/acbull/pyHGT/tree/master/Docker

wonderful! i will try

thx！


Cloud-pak-deployer Multi-arch

The objective is to merge the modification made to support "ppc64le" arch in the main branch in order to keep track of the update.

In a first step, the Dockerfile will stay different (complicated to merge). Dockerfile will be the x86 one and Dockerfile.ppc64le has been created for Power.

cp-deployer script has been modified to detect the current architecture when used with build command, and point to the right Dockerfile and the right olm-utils image.

use ${ARCH} variable in Ansible and most of the shell in order to download the right binaries.

I've created 2 images and a manifest list for multi-arch purpose

  (x86) => quay.io/schabrolles/cloud-pak-deployer:240318-x86
  (ppc64le) => quay.io/schabrolles/cloud-pak-deployer:240318-ppc64le
  (manifest-multiarh) => quay.io/schabrolles/cloud-pak-deployer:240318 (and lastest)

So we can use a single image name (the manifest) in order to deploy it on power or x86
quay.io/schabrolles/cloud-pak-deployer:lastest
or quay.io/schabrolles/cloud-pak-deployer:240318
@fketelaars I'm testing this image to deploy a CP4D on Power and x86. If it works well, we can discuss a way to automate this multi-arch build to push into quay.io/cloud-pak-deployer

FYI:

Installation of CP4D with the multi-arch image (via manifest) works on power and x86.


ci: simplify code with unparam linter

Signed-off-by: Kuat Yessenov<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: kyessenov
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kyessenov: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
554a79974b9a231cd7eec956ae18523d083d1347
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-multicluster-e2e.sh
e006e4e651b001e8cd3ab038f0473769ec9e12c2
link
/test istio-pilot-multicluster-e2e

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

did you just manually fix all 66 files?

contributes to #9933

@rshriram can you lgtm? We want to stop further linter breakages, so it would be good to enable this check.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
e006e4e651b001e8cd3ab038f0473769ec9e12c2
link
/test istio-pilot-multicluster-e2e

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


Is the C API to embed wasmtime usable?

"Wasmtime exposes an API for embedding as a library through the wasmtime subcrate, which contains both a high-level and safe Rust API, as well as a C-compatible API compatible with the proposed WebAssembly C API."
Where can we find this "C-compatible API compatible with the proposed WebAssembly C API"?

CI has tests that run C examples from WebAssembly C API. See also custom Makefile

Where can we find this "C-compatible API compatible with the proposed WebAssembly C API"?

The releases have archives with "-c-api" suffix -- the built libraries with C API interface can be found there. Closing the issue as answered.


Game crashes with dxvk

As per title when I try to implement dxvk with select renderer my game crashes and this message pops up
I tried updating my graphic card drivers and directx 9, but it's all the same. I also ran dxdiag for more infos about my pc.

You'll want to report this to the SelectRenderer repo, I cannot help you unfortunately.


Update changelog for 0.53.0rc3

as titled

Will this need to be cherry-picked onto the release branch for 0.53?

Will this need to be cherry-picked onto the release branch for 0.53?

Think so.


#158423645 Update expanded ride offer

What does this PR do?

Adds passenger's view of an exapnded ride offer which can be requested

Description of Task to be completed?

Building (without functionality), the UI elements with HTML&CSS.

How the app should be tested
On a web browser:

Download the repo
Extract and navigate into the Ride-My-Way folder
in the UI folder, open expanded_ride_passenger.html on a browser

On a  terminal (Linux):

Install and set up git sudo apt install git

Clone the repository git clone https://github.com/artorious/Ride-My-Way.git

Run git checkout ft-passengers-expandedride-upd8-158423645

Run cd Ride-My-Way/UI/

Open expanded_ride_passenger.html on a browser

Any background context you want to provide?
The UI has no functionality.
Developed using HTML and CSS
What are the relevant pivotal tracker stories?
#158423645
Screenshots

Questions:

In your ride offer, I think you should combine date and time to be in the same field

What does the Detours field entail exactly?
Also, really look into improving the navigation bar, there is a lot going on here. However, these are adjustments that will come in later. Can be merged

@LWanjiru I will improve the UI description for detours to avoid ambiguity. Thank you.


Support HikariCP for RefreshScope

Is your feature request related to a problem? Please describe.
I saw that the Spring Cloud Commons doc mentions:

If you hava a DataSource bean that is a HikariDataSource, it can not be refreshed. It is the default value for spring.cloud.refresh.never-refreshable. Choose a different DataSource implementation if you need it to be refreshed.

Since HikariCP is the default CP solution for Spring Boot, can support be added to allow HikariCP datasource be refreshed?

No. The hikari implementation prevents it from being refreshable.


Migrate to Swift 4

What?
Migrate to Swift 4.

CC @pusher/mobile

          3 Warnings

:warning:
Any changes to library code should be reflected in the Changelog. Please consider adding a note there.
You can find it at CHANGELOG.md.

:warning:
PR is classed as Work in Progress

:warning:
The Cartfile or Cartfile.resolved was updated, but there were no changes in the podspec. Did you forget updating the podspec?

SwiftLint found issues
Warnings

File
Line
Reason

PCCurrentUser.swift
19
TODOs should be avoided (This should probably be [PCUse...).

PCCurrentUser.swift
61
TODOs should be avoided (When can percent encoding fail...).

PCCurrentUser.swift
364
TODOs should be avoided (Use the soon-to-be-created new...).

PCCurrentUser.swift
462
TODOs should be avoided (Do we need to fetch users in t...).

PCCurrentUser.swift
592
TODOs should be avoided (Where should all the makeActiv...).

PCCurrentUser.swift
598
TODOs should be avoided (Do I need to add a Last-Event-...).

PCCurrentUser.swift
602
TODOs should be avoided (What happens if you provide bo...).

PCCurrentUser.swift
629
TODOs should be avoided (This will probably be replaced...).

PCCurrentUser.swift
636
Function body should span 40 lines or less excluding comments and whitespace: currently spans 49 lines

PCCurrentUser.swift
188
MARK comment should be in valid format.

PCCurrentUser.swift
247
MARK comment should be in valid format.

PCCurrentUser.swift
5
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
39
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
158
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
175
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
209
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
264
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
324
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
730
Enum element name should be between 3 and 40 characters long: 'messageIdKeyMissingInMessageCreationResponse'

PCCurrentUser.swift
105
Collection literals should not have trailing commas.

PCCurrentUser.swift
550
Collection literals should not have trailing commas.

PCCurrentUser.swift
608
Collection literals should not have trailing commas.

PCCurrentUser.swift
197
Line should be 120 characters or less: currently 142 characters

PCCurrentUser.swift
209
Line should be 120 characters or less: currently 135 characters

PCCurrentUser.swift
213
Line should be 120 characters or less: currently 132 characters

PCCurrentUser.swift
516
Line should be 120 characters or less: currently 127 characters

PCCurrentUser.swift
580
Line should be 120 characters or less: currently 121 characters

PCCurrentUser.swift
621
Line should be 120 characters or less: currently 125 characters

PCCurrentUser.swift
629
Line should be 120 characters or less: currently 121 characters

PCCurrentUser.swift
669
Line should be 120 characters or less: currently 123 characters

PCCurrentUser.swift
746
File should contain 400 lines or less: currently contains 746

PCRoom.swift
17
TODOs should be avoided (Should each Room instead have ...).

PCRoom.swift
21
TODOs should be avoided (Is this going to be expensive ...).

PCRoom.swift
23
TODOs should be avoided (This will also not work well i...).

PCRoom.swift
66
Operators should be surrounded by a single whitespace when defining them.

PCRoom.swift
5
Variable name should be between 3 and 40 characters long: 'id'

PCRoom.swift
31
Variable name should be between 3 and 40 characters long: 'id'

PCSynchronizedArray.swift
26
TODOs should be avoided (Does this need a userInitiated...).

Errors

File
Line
Reason

PCCurrentUser.swift
4
Type body should span 350 lines or less excluding comments and whitespace: currently spans 412 lines

  Generated by :no_entry_sign: Danger

🚢


testing-feat-<IP_ADDRESS> fails to build with GHC-8.2 and older

testing-feat-<IP_ADDRESS> % cabal build -w ghc-8.2.2
Resolving dependencies...
Build profile: -w ghc-8.2.2 -O1
In order, the following will be built (use -v for more details):
 - testing-feat-<IP_ADDRESS> (lib) (first run)
Configuring library for testing-feat-<IP_ADDRESS>..
Preprocessing library for testing-feat-<IP_ADDRESS>..
Building library for testing-feat-<IP_ADDRESS>..
[1 of 7] Compiling Test.Feat.Class  ( Test/Feat/Class.hs, /codetmp/testing-feat-<IP_ADDRESS>/dist-newstyle/build/x86_64-linux/ghc-8.2.2/testing-feat-<IP_ADDRESS>/build/Test/Feat/Class.o )
[2 of 7] Compiling Test.Feat.Finite ( Test/Feat/Finite.hs, /codetmp/testing-feat-<IP_ADDRESS>/dist-newstyle/build/x86_64-linux/ghc-8.2.2/testing-feat-<IP_ADDRESS>/build/Test/Feat/Finite.o )

Test/Feat/Finite.hs:33:10: error:
    Not in scope: type constructor or class ‘Semigroup’
   |
33 | instance Semigroup (Finite a) where
   |          ^^^^^^^^^

I made a revision https://hackage.haskell.org/package/testing-feat-<IP_ADDRESS>/revisions/, it still allows GHC-8.4.

Also size-based-<IP_ADDRESS> is broken with GHC-7.10 and GHC-7.8, but https://github.com/size-based/size-based doesn't have issues enabled.
Build profile: -w ghc-7.10.3 -O0
In order, the following will be built (use -v for more details):
 - size-based-<IP_ADDRESS> (lib) (first run)
Configuring library for size-based-<IP_ADDRESS>..
Preprocessing library for size-based-<IP_ADDRESS>..
Building library for size-based-<IP_ADDRESS>..

Control/Enumerable/Count.hs:12:8:
    Could not find module ‘Data.Semigroup’

I made revision for it too: https://hackage.haskell.org/package/size-based


Remove console.info

Hi,

I noticed that the console.info also appears in production, I think it would be interesting to disable it. what do you think?
Thanks for this amazing repo btw 🙂

The TurboBoost Commands library has the concept of a logger. It would probably be a good idea to create something similar for the Streams library. 🤔