capture_exception and capture_message would be improved with shorthand syntax

We used the extra parameter to captureException and captureMessage in raven quite often, and what took one line now takes a minimum of three (plus one line for each attribute of the extra object), plus a level of indentation:
captureMessage('something', extra={'a': 1, 'b': 2})

# vs...
with push_scope() as scope:
    scope.set_extra('a', 1)
    scope.set_extra('b', 2)
    capture_message('something')

This was a conscious decision. Generally the idea is that extra data should be applied to entire codeblocks instead of single capture/logging statements. The old Raven SDK encouraged both but often there's no reason for the latter.
The one thing where it truly gets annoying is when you migrate from raven to the new SDK and don't want to refactor that part of your code. But I imagine the solution for this is writing your own helper functions for backwards compat. At least that's what worked well for us when porting sentry itself.

fair enough, I can write a wrapper method :)


ConsoleLogger should send LogLevel.Error to Error-Stream

LogError and LogCritical should send the logmessages to Console.Error-stream. Right now the output is just sent to Console.Out.
Minimal repro steps
namespace Foo
{
    class Program
    {
        static void Main(string[] args)
        {
            var loggerFactory = new LoggerFactory()
                .AddConsole();

            var logger = loggerFactory.CreateLogger<Program>();

            logger.LogInformation("Test");
            logger.LogError("Error");

            // Make sure the consumer on the write-side can drain the queue
            Thread.Sleep(500);
        }
    }
}

dotnet foo.dll > out.txt 2> err.txt
Expected result
err.txt should contain the logged error messaged. Critical messages, although not shown in the sample above, should also get sent to stderr.
Actual result
All logged messages are sent to the standard output stream.

Yea I don't know if I agree with this. I've never seen clear guidance on this. Can you provide some evidence that this is the norm?

I agree that there's no norm -- at least I'm not aware of one -- to where each LogLevel should send its output. But for error-levels this should be the error-stream, 'cause this is what error-stream is for.
Cf. Standard streams
The distinction between different (output) streams is useful -- especially on Linux -- for cronjobs, where one can send standard output to null, and error output to an email.
On execution and when standard-error is not redirected it is the same as standard-output. So nothing is won, nothing is lost. But we win the possibility to redirect the streams differently and more important: indepently.
Therefore I would make this the default, without any need to configure it explicitely.

I don't agree but I'm willing to be wrong here. Can you show me other platform console loggers that switch based on error vs not. It feels inconvenient that you would need to pipe both streams to capture logs to a file. On top of that, you lose the overall ordering because you split the logs between streams

It feels inconvenient that you would need to pipe both streams to capture logs to a file.

For me this feels natural ;-) because standard-out and standard-error are two different things. Anyway the error-stream can easily be redirected to standard-output with 2>&1 (cf. Redirecting Error Messages from Command Prompt: STDERR/STDOUT (same for Window and *nix)
IBM Knowledge Center - Logging (logging) is one example for sending errors to the error-stream. My home is .net, so I'm not really aware of other loggers (except my self-developed for .net full, based on TraceListener).
On *nix sending error-messages to the error-stream is kind of a best practice. I admit that on Windows this separation isn't done / used very often, although it would be "correct". Furthermore this is the reason why there exists the error-stream (on Windows and on *unix-shells).
I understand your points. So it might be good to make it configurable (as you suggested above), to address the desired behaviour for every user.
So sending errors to the error-stream would be an opt-in via config?

I have no doubt this should be configurable, but I'm just not convinced it should be the default.

I'm with you. Now I also think this shouldn't be the default.
When I find some time, I'll try to make the changes for the config-based solution. Should this be an update to #662 or a new PR? (sorry to ask, but I'm not so involved in the processes here, so I'll better ask)

@davidfowl Let's decide if we'd like this configurable, opt-in for 2.1.

@muratg

Let's decide if we'd like this configurable, opt-in for 2.1

Did you take any decision?

@gfoidl We haven't yet made the final decision, but it's unlikely to be done for 2.1 release with the current POR.

Hi there. Logging like this is needed for platforms liike cloud foundary. https://docs.cloudfoundry.org/devguide/deploy-apps/streaming-logs.html#types
I need to log to stderr console for error logs to be picked up as actual errors in CF. I'm having to roll my own console logger to do this atm because this console logger isn't allowing me to configure/override correctly to accomplish it

I would really like to see this option

We're using Istio on kubernetes which works seamlessly with loggers that write info/warning to stdout and errors to stderr. For now we're using NLog, but we'd prefer the ConsoleLogger if there was support for writing errors to stderr.


Support for the upcoming releases and release links apis

Looks like Gitlab 11.7 will promote Releases from a tag extension to a top level feature.
Two new apis are added for managing releases:

Releases, and
Release links

It would be great if go-gitlab would cover the new API's

Thanks for the PR @juranki!


Remove the usage of docker buildx

Docker Buildx is only required when creating multi-arch container images. Since we're not doing that here, it's better to not use it.
If we do use it, we have to set provenance = false in the docker/build-push-action step due to an issue with the image metadata when pushed to GitHub Container Registry. See https://github.com/docker/build-push-action/issues/778 for context. This issue was introduced in v3.3.0 of docker/build-push-action.

This PR is also bringing to light the recent release of SQLAlchemy v2's incompatibility with Flask-SQLAlchemy.

not quite as straightforward as updating Flask-SQLAlchemy .... Flask-SQLALchemy has been fixed last October. ... however that fix kinda came along with a requirement to update Flask to >= 2.2  (which we pin to < 2.1.0) ...
so ...either we pin sqlalchemy to < 3.0.0 or we update Flask to latest version ... either way should fix this dependency problem (I guess Flask 2.2 upgrade requires a bit of testing as well)

(and yes .. .this whole docker buildx,  DOCKER_BUILDKIT=1, etc... is a bit messy, and depends too much on the builders available to your environment (IMO)


Remove dynd from conda.recipe

dynd is currently unstable so do not test on it.

This was breaking the conda.recipe merging soon if no comment


A way of testing the build script

Currently there is no elegant way to test the build script when adding new enhancements/features/bugfixes. A virtual machine is proposed as a way to test the build script.
Any other suggestions are highly encouraged.

Maybe just creating a new user will do the job?

Since mac offers creating users from the terminal, it should be possible to automate tests for this. Should look into implementing them.


Add notebook about functions and the node sdk

Summary
This PR closes #743 by adding an article about function calling using the Node.js SDK. Given that the article is about JavaScript, and not Python, I wrote it as a .md file instead of as an .ipynb file, as I noticed other non-Python articles in the cookbook doing this as well.. Please let me know if you want me to use .ipynb instead.
In addition to the textual article, I have also recorded an interactive walk-through of the code. This will help learners who prefer watching videos over reading text.
Motivation
Almost all notebooks are based on Python. However, there are tons of JavaScript developers who use the OpenAI API as well. These people lack good tutorials, as exemplified by this post in the OpenAI community forum.
Self Review Checklist

[x] Is the writing easily skimmable?

Sections have informative titles.
Key takeaways are upfront.
Short paragraphs and topic sentences are used.

[x] Is the writing quality high?

Sentences are simple and unambiguous.
Demonstrative pronouns are avoided or clearly referenced.
No left-branching sentences.

[x] Is the content universally helpful?

Terminology is specific and avoids jargon.
Provides solutions to common problems.
Code examples are general and exportable.

[x] Is the content consistent?

Styling and formatting align with existing documentation (yes, except for it being written in markdown instead of as a Jupyter Notebook)
Consistent use of punctuation and case.

When contributing a new example, make sure to add a new entry for it in registry.yaml to render it on the cookbook website.
Note: For additional guidelines on writing good documentation, check out What Makes Documentation Good.

This is great! Thank you for the contribution. I'll review it early next week at the latest. For now one quick thing I just noticed is that the file is missing the .md extension.

Awesome! Just added the .md extension :)

Thanks for the approval @NancyAngels :)
Anything you’d like me to change @simonpfish or is it good to go?

Hello @perborgen! We have just created a new set of contribution guidelines for the cookbook, and updated our PR template. I'll review your notebook tomorrow, but in the meantime feel free to look at those resources if you're curious. I'll be referring to them as I review your PR.

That rubric was helpful! I went through the article again to improve spelling and clarity @simonpfish.
Btw, I recorded this interactive screencast to explain the code in detail. I've embedded the screencast into to the article via an iframe. However, whether the iframe will render or not depends on how your Astro project handles the markdown AFAIK.
I'd appreciate if you'd let me know if there's any issues with it, or if you have any comments regarding the screencast :)

Another minor edit to the title. Looks great! Merging now.


when you search for something in powertoys run,have a shortcut and have it open  right click options for the searched item

Description of the new feature / enhancement
first, I'm sorry that I'm not able to express it that good. when you search for something with powertoys run, a button/shortcut
to have an ui that is similar to what pops up when you right click,and allow us to rename it,unzip it,open with another app,
Scenario when this would be used?
when youre programming,you can search for the file ,you can perform this shortcut and open that file with your preferred IDE.
you can search for the item,cut/copy it, and then paste it to wherever you want without opening a second file explorer file
Supporting information
No response

Thanks for the suggestion @Akqli! You gave a good explanation, I understand the functionality you're describing. We'll log this under our enhancements milestone for now and revisit the idea following our general stability efforts.

@Akqli which items?  PT Run had multiple plugins and some of these may not make sense where others, they do.
When do you not have the preferred IDE as the file type?
These options add a lot of UX complexity which is why crisp use cases would be helpful.


Visdom Overlapping Images.

I am working on a neural network code and i am using Visdom server for my visualization. Everything was working well until last two days, my confusion matrix started replicating itself and covering my other graphs (test graph, test loss graph). My training accuracy graph is now totally messed up. I now have three confusion matrix instead of one.
Software used: Anaconda (using Pytorch)
OS: Windows 10
Browser: Chrome

I have uninstalled and reinstalled Anaconda

I uninstalled and reinstalled visdom.

i seem not to know what i am doing wrong.

I am using the code on a different machine and it is working well (as in all the graphs are showing correctly and in their place).

Hi @mightyabra - unfortunately I won't be able to debug what's going on here without knowing what VisdomPlotLogger and VisdomLogger are, as the bug is likely there. The behavior you're describing would be what happens if the logger started confusing the unique window ids being sent to the visdom package.

VisdomPlotLogger and VisdomLogger are torchnet modules for logging outputs to Visdom. Please take a look at this website https://buildmedia.readthedocs.org/media/pdf/tnt/latest/tnt.pdf.
Its a documentation about VisdomLogger and PlotLogger functions.

Sorry, I don't really know anything about this implementation, but from what I could see in a first pass it is doing something potentially strange to keep track of window ids. You may have more luck using it by specifying a win=<> argument to each logger that you specify a unique id such as 'train_loss_win'.

thanks @JackUrb  let me try that out and give you feedback.

Had the same thing happen to me. When a window id is not provided visdom creates one using, among other things, a function called get_rand_id. I think that function returns the same string if it is called twice in less than 1 millisecond. get_rand_id uses time.time() which might have a very, system dependent, bad resolution. Always specifying the 'win' params solves the issue.


Weird Boolean evaluation for Long attributes

Great Library!! Only the following scenarios is bothering me.
String expression = "((1565612699586 >= 5.5) &&<PHONE_NUMBER>586 <=<PHONE_NUMBER>894))";
Expression e = new Expression(expression);
e.eval();
I have a test around this which always returns 1 i.e. TRUE  which should be 0 i.e. FALSE,
The weird part is when I debug and evaluate the individual expression and then again create the same  expression object , it sometimes return 0 and sometimes 1
All other comparisons looks good like double, integer etc.
This issue is blocking us to use this library :(

I'm very sure that this is a duplicate of #114.
I know that there are always questions coming up regarding that, but there is no good way of addressing this problem on the API level of EvalEx. One could increase the default precision, but that would only push the problem to more niche cases. Setting the unlimited precision by default does also no help, as some expressions then will fail.
The best I can think of right now is to extend the README, somewhere around the "Usage Examples" with a clear note that one should set a precision suitable for their tasks.

Thank you for the quick reply!
currently, I am comparing long, double, and integer, so for long we have max precision is 19 .
Do you see any issue with this default setting for comparison with long, double  and integer?

With the current default setting you mean? Yes, definitely, it has a precision of 7 I believe. So what you want to do is create the expression and set a precision of 19, at least. I'd rather go with more, because if you have two large numbers and you do anything but comparing them, you might run into trouble there.
Unless you're doing something computive intensive here (and I can't think of an application were this would really matter), I'd suggest to go with the MathContext.DECIMAL128, as it will give you 34 digits to work with.

Thank you!!  Currently, we are gonna do just comparisons, so I will keep 19 :)

Great! If you don't have any further questions you can close this issue.

I've added a section to the README regarding the precision. Maybe that will help avoid confusion in the future.

Thank you !!


not an issue but suggestion

Could you please implement vapoursynth to this project.
https://github.com/sunny2109/SAFMN
cause it is fenomenal for low res input
Thank you very much in advance!

espeically with the SAFMN++, seems interessting.

If it's supported by chaiNNer then you can convert the model to onnx and run inference with vs-mlrt.

sadly model team23_safmnpp.pth is unsupported by chaiNNer.
But there is https://github.com/sunny2109/SAFMN/tree/main/scripts/to_onnx and SAFMN_onnx_pt.zip contains an onnx-file which supports 940x640 resolution, and:
[code]w = clip.width
h = clip.height
clip = core.resize.Bicubic(clip=clip, width=960, height=640, format=vs.RGBS, matrix_in_s="470bg", range_s="full")
from vsmlrt import Backend
clip = vsmlrt.inference([clip],network_path="C:/Users/Selur/Desktop/SAFMN_640_960_x2.onnx", backend=Backend.OV_CPU(num_streams=1))
clip = core.resize.Bicubic(clip=clip, width=w2, height=h2)[/code]
does seem to work.


Update pulumi-terraform-bridge to v3.49.1

Fixes https://github.com/pulumi/pulumi-aws/issues/2547
Figuring out the bridge and pulumi/pkg update and verifying the diffs before doing further surgery can be very beneficial. The V6 PR may need to be built on top of something like this. PTAL.

This looks good to me. Is there a reason we have a schema change with the missing property from just the bridge upgrade though?

That doesn't make sense to me, need to dig in.

Here is the definition:
https://github.com/pulumi/terraform-provider-aws/blob/a82c36c5556e2b8de91ba2a82a24bcd6074345be/internal/service/ecs/service.go#L383
Here is the override:
https://github.com/pulumi/pulumi-aws/blob/f854197a6ec1a3479edbbaba61b304ee84a26270/provider/resources.go#L1997
A quick derivation:
cd ~/code/pulumi-aws
jq '.resources["aws:ecs/service:Service"].properties.serviceConnectConfiguration' $(find . -name 'schema.json')
jq '.types["aws:ecs/ServiceServiceConnectConfiguration:ServiceServiceConnectConfiguration"].properties.services' $(find . -name 'schema.json')
jq '.types["aws:ecs/ServiceServiceConnectConfigurationService:ServiceServiceConnectConfigurationService"].properties.clientAliases' $(find . -name 'schema.json')

{
  "$ref": "#/types/aws:ecs/ServiceServiceConnectConfiguration:ServiceServiceConnectConfiguration",
  "description": "The ECS Service Connect configuration for this service to discover and connect to services, and be discovered by, and connected from, other services within a namespace. See below.\n"
}
{
  "type": "array",
  "items": {
    "$ref": "#/types/aws:ecs/ServiceServiceConnectConfigurationService:ServiceServiceConnectConfigurationService"
  },
  "description": "The list of Service Connect service objects. See below.\n"
}
{
  "type": "array",
  "items": {
    "$ref": "#/types/aws:ecs/ServiceServiceConnectConfigurationServiceClientAlias:ServiceServiceConnectConfigurationServiceClientAlias"
  },
  "description": "The list of client aliases for this Service Connect service. You use these to assign names that can be used by client applications. The maximum number of client aliases that you can have in this list is 1. See below.\n"
}

This field is now cientAliases instead of clientAlias. This seems to be correct - it's defined as MaxItems=1 in the schema but it's overridden in resources.go to ignore MaxItems=1. Ignoring it makes it a list, which also means the name is pluralized in Pulumi.

before the change this field was an array but called clientAlias
    "clientAlias": {
      "type": "array",
      "items": {
        "$ref": "#/types/aws:ecs/ServiceServiceConnectConfigurationServiceClientAlias:ServiceServiceConnectConfigurationServiceClientAlias"
      },
      "description": "The list of client aliases for this Service Connect service. You use these to assign names that can be used by client applications. The maximum number of client aliases that you can have in this list is 1. See below.\n"
    },

I think the change is https://github.com/pulumi/pulumi-terraform-bridge/releases/tag/v3.47.1 specifically https://github.com/pulumi/pulumi-terraform-bridge/pull/1117
@rquitales how does it sound? Should we keep this change as is, or should we restore the previous name by manually specifying the name to be "clientAlias"?

@t0yv0 Thanks for digging! Can we create an alias for that field name change as well to maintain forwards compatibility for existing users? Creating a pluralized name for that field makes sense to me.

How do you create an alias for a field? Something to set on SchemaInfo? Are there some examples? Thanks.

Update pulumi-terraform-bridge to v3.49.1


Question: What does the MS in MSPhysics stand for?

The name MSPhysics reminds me too much of the name "MSPaint," but I doubt MS is supposed to stand for "Microsoft" here. I'm curious though. I couldn't find any explanation anywhere. Is it supposed to be an abbreviation for "Master of Science?"

I actually don't know. It is just a great combination. When I first started writing it, I though naming it Math, Science, Physics and then My Sketchy Physics.


The magic short SQL syntax will also escape the table name

Fixes #79

Thank you for your contribution! I appreciate it 👍


Items on the ground are logged as loot when dying to the whisperer

Type
Incorrect behavior
Operating System
Windows
OS Version
Windows 11
Bug description
When dying to the whisperer the loot tracker will consider item on the ground at your respawn point as loot and add them as drop for a kill that didn't happened.
Screenshots or videos
No response
RuneLite version
Runelite version: <IP_ADDRESS>
Launcher version: 2.6.3
Logs
No response

this was fixed in 4f43819f59121465e221576f49ec9fa62759ff8a and 2c61107c9ccbe9a9fcd7dc258c61013c2c4b17e4


Get RuntimeError when pre-training model

Hi,
when pre-training the model use the command bash script/conala/train-pre.sh, I got the RuntimeError as the following:

Traceback (most recent call last):
File "exp.py", line 615, in
pretrain(args)
File "exp.py", line 500, in pretrain
ret_val = model.score(batch_examples, epsilon=1.0)
File "/home/vladimir/cg-rl/CG-RL-main/model/parser_pre.py", line 313, in score
query_vectors, new_batch = self.decode_reinforce(batch, src_encodings, dec_init_vec, epsilon=epsilon)
File "/home/vladimir/cg-rl/CG-RL-main/model/parser_pre.py", line 369, in decode_reinforce
batch = copy.deepcopy(batch)
File "/home/vladimir/.conda/envs/cg-rl/lib/python3.6/copy.py", line 180, in deepcopy
y = _reconstruct(x, memo, *rv)
File "/home/vladimir/.conda/envs/cg-rl/lib/python3.6/copy.py", line 280, in _reconstruct
state = deepcopy(state, memo)
File "/home/vladimir/.conda/envs/cg-rl/lib/python3.6/copy.py", line 150, in deepcopy
y = copier(x, memo)
File "/home/vladimir/.conda/envs/cg-rl/lib/python3.6/copy.py", line 240, in _deepcopy_dict
y[deepcopy(key, memo)] = deepcopy(value, memo)
File "/home/vladimir/.conda/envs/cg-rl/lib/python3.6/copy.py", line 161, in deepcopy
y = copier(memo)
File "/home/vladimir/.conda/envs/cg-rl/lib/python3.6/site-packages/torch/autograd/variable.py", line 91, in deepcopy
raise RuntimeError("Only Variables created explicitly by the user "
RuntimeError: Only Variables created explicitly by the user (graph leaves) support the deepcopy protocol at the moment

I think the code batch = copy.deepcopy(batch) should be modified to batch = batch.clone(), but I' m not sure whether gradient flow should be allowed.
Could you please fix this issue? @lighter47
Thank you.

I don't have this problem in pytorch 1.3.1, or you can try to use clone() instead of deepcopy(), there is no need for gradient flow.

I don't have this problem in pytorch 1.3.1. You can try to use clone() because there is no need for gradient flow. Thank you for your notification, I will check the version compatibility in the future.

I don't have this problem in pytorch 1.3.1. You can try to use clone() because there is no need for gradient flow. Thank you for your notification, I will check the version compatibility in the future.

Thank you for your response. My pytorch version is 0.3.1, which is same as the tranx. Could you please release your conda environment config file?
There are tow copy function, clone and detach, inpytorch. The gradient should be propagated to the original tensor and I should usecloneinstead ofdetach`. Is it right?

I don't have this problem in pytorch 1.3.1. You can try to use clone() because there is no need for gradient flow. Thank you for your notification, I will check the version compatibility in the future.

I found it didn't work because the batch is a Batch object instead of Tensor object. I think the only solution is using the same environment. Could you please release your conda environment config file? The configuration file config/env/tranx.yml mentioned in README file contains only the package name but not the version. Could you please release your configuration file with version? Thank you. @lighter47

I have released two configuration files with python2 and python3 version.

Thank you for your response!

And it should be removed to make the code more elegant
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2022年4月11日(星期一) 中午11:24
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [DeepLearnXMU/CG-RL] Get RuntimeError when pre-training model (Issue #3)
I have released two configuration files with python2 and python3 version.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Question: Changing peasant names

Hi all,
This isnt directly a patch question, I just want to know if its possible to change peasant names? I would assume it would beeasy to do so, but i cant find the file where they are specified, and cant find the string in the exe with cheat engine.
Names are currently things like "Jamie Brown" or "Roger Badbread", it seems to be generated from lists of first names and surnames. http://stronghold.heavengames.com/gameinfo/lists/people
If someone can point me to where I might be able to add my own names I'd appreciate it.
--GRhin

In my modification about Russia, the names of the peasants were not only replaced, but also changed to those ancient ones that we had in the 13th century. And the names too. So, what you say is no longer a news for me, like 10. You’re late to discover America))

@Russianstory I think you have misunderstood, I was asking how it can be done, if you are doing it in your mod maybe you could let me know how you did it?

In the Stronghold Crusader Modding Utilities, there is a Text Compiling Tool and all text of the game, including peasant names. One can change the text, compile it and include it in-game.

http://stronghold.heavengames.com/cgi-bin/forum/display.cgi?action=ct&f=4,3820,,10 : Second Link seems to work, but no liability ;)

Thanks @J-T-de, I'll take a look

No, my friend)) I just understood you correctly)). I do not remember exactly myself. Because I was another comrade. Everything can be enclosed in a file. cr.tex I was responsible for sounds, voiceovers and lyrics for voiceovers.

J-T-de's suggestion worked, I probably should have looked there first lol


Introduction to styled-components

Request a Screencast
I've seen a lot of the community talk about styled-components. Can you give an intro to that?

Article announcing v2 of styled-components
Library Home Page

Thanks! Looking forward to seeing what you cover on this!

Thanks for the suggestion. Yeah, I think a "styles" related episode is long due...


fix(costmodels): update confirmation dialog's cancel and warning icon

move the warning icon to the title
change the cancel button variation to "link"

Codecov Report

Merging #1593 into master will decrease coverage by 0.02%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #1593      +/-   ##
==========================================
- Coverage   78.71%   78.69%   -0.03%
==========================================
  Files         237      237
  Lines        3881     3882       +1
  Branches      712      713       +1
==========================================
  Hits         3055     3055
  Misses        727      727
- Partials       99      100       +1

Impacted Files
Coverage Δ

...costModels/costModelsDetails/components/dialog.tsx
94.44% <50.00%> (-5.56%)
:arrow_down:

Continue to review full report at Codecov.

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

There is a regression in coverage - I want to fix it before merging this.

Thank you guys @ddonahue007 & @dlabrecq


Trying different input for mnv2 model

Hi!
I'm hoping to test out running classification on some of my own images using the CFU-Playground framework with the mnv2 model. I wanted to ask exactly what kind of preprocessing was applied to the mnv2 input files provided.
From taking a look at them, it seems that they're 3 channel 160 x 160 pixel images with uint8 data (correct me if I'm wrong), but I'm unsure if any extra processing was applied to the image data. Any info would be helpful.
So far, I tried using something like this to get the model input data from a 160x160 jpeg, but I didn't get a good classification:
    img_jpeg = tf.io.read_file(filename)
    img = tf.image.decode_image(img_jpeg, dtype=tf.dtypes.uint8)

    img.numpy().tofile("out.dat")

Also, I was wondering if you have any info on interpreting the output of the model. Does it directly correspond to the mnv2 label?

Hi, @noorassan :
You could do some code trace to find out the input image format in framebuffer display code which I provide in "CSR_VIDEO_FRAMEBUFFER_BASE" defined section in
https://github.com/google/CFU-Playground/blob/8dddce38085becf9480121ee173ef7d90f81ef70/common/src/models/mnv2/mnv2.cc
BR, Akio

Thanks -- that was helpful. Could you give me some guidance on how to interpret the output? All of the trained mobilenet models that I've worked with return an output with length equal to the number of labels in the model with each value proportional to the likelihood of that label. It seems like the mobilenet model used in cfu playground just returns an output of length two. How should this output be interpreted?

Hi, @noorassan :
the output part is a relative value. for the most simple way is find out the 1st max value, the index will be the respect class. the 2nd max value's index is the 2nd class.
for the confidence ratio, you could using a approximation way for softmax or just a simple percentage ratio to evaluate the confidence value.
for me, i go the simple way.
BR, Akio

Sorry -- not sure if you misinterpreted my question or if I'm misinterpreting you, but I appreciate the response nonetheless. You said you can find the max value in the output and the index will be the respective class. However, the output of the mobilenetv2 model included in cfu playground only has 2 values while there are far more than 2 classes, correct? How should this output be interpreted?
For example, I ran an image of a persian cat through the model and received this output -- note that I didn't use CFU Playground for this, I just took the tflite file from the repo and ran the model on my local machine. How can I use this output to get a label?
[ 48 -48]

Again, thanks so much for the response -- hopefully I'm not missing something obvious...

Hi @noorassan !    Sorry for my delay helping out.   I just found a very useful site for viewing the raw data files: https://rawpixels.net/.   The parameters for the .dat files in the repository are width:160, height:160, offset:0, Predfined format:RGB24.   I didn't need to change anything else, but verify that you have: Pixel Format:RGBA, Ignore Alpha (yes checked), bpp1..4: 8,8,8,0.
I don't know where we got this particular trained model, but I'm pretty sure it is only trying to recognize person / no person.  @alanvgreen , do you remember where you got the MNV2 model?
If you get a different MNV2 model or train your own, you might need to adjust some of the C++ code to account for different image format and different number of classes in the output vector.

Hi, @noorassan :
the output interpret method is based on the network output. if the last layer is softmax, the interpretation will be what I am said. if the network is like yolo v1/v2/v3, you have to
know the structure to do interpretation.
due to there is no FPGA board in my hand, I could not get the output of MNV2.
Would you like to give the output of MNV2 ?
BR, Akio

Sorry -- not sure if you misinterpreted my question or if I'm misinterpreting you, but I appreciate the response nonetheless. You said you can find the max value in the output and the index will be the respective class. However, the output of the mobilenetv2 model included in cfu playground only has 2 values while there are far more than 2 classes, correct? How should this output be interpreted?
For example, I ran an image of a persian cat through the model and received this output -- note that I didn't use CFU Playground for this, I just took the tflite file from the repo and ran the model on my local machine. How can I use this output to get a label?
[ 48 -48]

Again, thanks so much for the response -- hopefully I'm not missing something obvious...

@akioolin  Sure, I included the output in the post that you quoted

@tcal-x Thanks for the explanation -- it makes sense that the model would be recognizing person/no-person based on the output

Hi @noorassan , Finally, how your code converting from .jpg to .dat file looks like? I have the same issue. Thank you.

Hi – sorry for the late response. I ended up using the following script. Hope it helps!
import sys

import tensorflow as tf

from tensorflow.python.ops.numpy_ops import np_config

if __name__ == "__main__":
    filename = sys.argv[1]
    np_config.enable_numpy_behavior()

    img_jpeg = tf.io.read_file(filename)
    img = tf.image.decode_image(img_jpeg, dtype=tf.dtypes.uint8)

    img.numpy().tofile("out.dat")


Fix INSERT_METHOD syntax

Index Issue: pingcap/tidb#11486
These are bad (incompatible) SQL cases:
ALTER TABLE t INSERT_METHOD = FIRST
ALTER TABLE t INSERT_METHOD FIRST

Claim this subtask based on the comments below this issue.

Let me fix it.


IsUnknown, IsNever and IsAny

Regarding IsAny I have a feeling that we need to prohibit it saying "We support only strict typescript projects"
Given IsUnknown and IsNever I would:

First of all, add them
Cover all existing types with test cases with unknown and never
Support all existing types with them (probably adjust the logic)

@krzkaczor wdyt?

Yes, InUnknown and IsNever are both useful. I didn't know that these types need special handling 😬

Regarding IsAny I have a feeling that we need to prohibit it saying "We support only strict typescript projects"

Can you elaborate on what exactly do you want to do with IsAny? I think we can expose it as sometimes it can be useful for ensuring that type inference works correctly but I don't think that we should tweak our types to prevent using any or something like this.

Yes, InUnknown and IsNever are both useful. I didn't know that these types need special handling 😬

Regarding IsAny I have a feeling that we need to prohibit it saying "We support only strict typescript projects"

Can you elaborate on what exactly do you want to do with IsAny? I think we can expose it as sometimes it can be useful for ensuring that type inference works correctly but I don't think that we should tweak our types to prevent using any or something like this.

At the moment, we allow any in our types but maybe it makes sense to prohibit at all, as we move towards strict project
However, I found several any in our types and maybe we still can allow some weakness regarding them
We just need to decide what is our way

I personally don't have a very strong opinion on this.
On one hand, I use any from time to time and I can clearly see that sometimes it's useful (but only in isolated scopes).
On the other hand, probably using 90% of types from ts-essentials with any doesn't make sense and no one does it (at least not on purpose).

On one hand, I use any from time to time and I can clearly see that sometimes it's useful (but only in isolated scopes).

It depends on who uses ts-essentials
For example, during my development I only use // @ts-expect-error comments (which I plan to fix later) as I usually make types working from the beginning
But that's only me, we need to know the opinion of other developers

On the other hand, probably using 90% of types from ts-essentials with any doesn't make sense and no one does it (at least not on purpose).

Yeah, exactly
If you want to use ts-essentials, probably you already know how to solve problems and won't have any that much
You're familiar with its evil "nature" and prefer either disable it with ts comments or make them too general from the beginning and then step by step make them more complicated
But again, that's only my approach, probably others will ask to have intermediate layer with any

So as I understand, first action points are to introduce 2 types: IsNever and IsUnknown
And then we will decide what to do with IsAny

So as I understand, first action point is to introduce 2 types: IsNever and IsUnknown
And then we will decide what to do with IsAny

Yup, makes sense to me.

Will plan to release IsAny soon – https://github.com/krzkaczor/ts-essentials/pull/264

Sorry, had problems with CI, release under https://github.com/krzkaczor/ts-essentials/pull/280


Art development general task

World Map
Units, Soldiers, Cavalry, Cannons
Cards
UI Design
Sounds
Music
AUDIO ROYALTY FREE


open file in new pane?

Hi, I have this command using your plugin with a bash script that does work to open a file:
xdg-open "obsidian://advanced-uri?vault=${VAULT2}&filename=${FILE4}"
Is there a way to force it to open the file in a new pane?  I'm usually viewing my files in obsidian but if they're not already linked in the current file calling them up with a rofi (bash) script to open them.  99% of the time I'd like to see the new file beside the current one, not replacing the current one.  Is this possible using your plugin (or any other way you can think of?)
I see a reference to your other plugin "obsidian-shortcuts-for-starred-files" that has that, but not sure if this functionality is also in the obsidian-advanced-uri plugin or not
thank you!

Dave

Thank you - I see it now (sorry, new to Obsidian)

No problem. Can I close this now?

Hmm I also can not see any setting in the advanced uri plugin settings for specifying a default openmode. Where might I find this? many thx

The only option in the settings is the "open file without write in new pane". So not all open mode  can be specified.


[] doesn't trigger onChange on clear

Environment

Tech
Version

@material-ui/pickers
3.0.0-beta.1

material-ui
4.0.0-beta.1

React
16.8.6

Browser
Chrome

Peer library
"@date-io/moment": "^1.1.0"

Steps to reproduce

Create a component

 <KeyboardDateTimePicker
            placeholder={`${today} 12:00`}
            value={convertedValue}
            onChange={handleChange}
            format="DD/MM/YYYY HH:mm"
            clearable
          />

Expected behavior
The onChange function triggers when I click the "CLEAR" button.
Actual behavior
The onChange function doesn't trigger when I click the "CLEAR" button. I get an error in the console.
Live example

FYI this should be fixed by #1036

@njetsy Thanks a lot, gonna check it out

Fixed in rev.0


Quieter Integration Tests

The integration tests produce far too much logging when they're running, so much that you cannot easily find errors.
This patch redirects the logging and adds some information to each test to print which test is running.
It requires that each integration runs SetTestForLogger. (This has been added to prepareTestEnv.)

Codecov Report

Merging #6513 into master will increase coverage by 0.11%.
The diff coverage is 92.79%.

@@            Coverage Diff             @@
##           master    #6513      +/-   ##
==========================================
+ Coverage   40.26%   40.38%   +0.11%
==========================================
  Files         403      404       +1
  Lines       54069    54077       +8
==========================================
+ Hits        21770    21837      +67
+ Misses      29282    29229      -53
+ Partials     3017     3011       -6

Impacted Files
Coverage Δ

modules/log/flags.go
100% <100%> (ø)

modules/log/file.go
77.62% <100%> (+2.09%)
:arrow_up:

modules/log/conn.go
96.55% <100%> (ø)
:arrow_up:

modules/log/smtp.go
89.36% <100%> (ø)
:arrow_up:

modules/log/colors.go
66.9% <37.5%> (+5.83%)
:arrow_up:

modules/log/console.go
75% <66.66%> (ø)
:arrow_up:

modules/log/writer.go
97.39% <97.39%> (ø)

routers/routes/routes.go
81.97% <0%> (+0.13%)
:arrow_up:

... and 6 more

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 d8802b1...3ed7e39. Read the comment docs.

On test failure the logs will be outputted Unfortunately they will be prefixed with a reference to the new wrapped logger, but I'm not sure there's a way to avoid this without rewriting the go testframework.


'await' not shown in suggestion list when type letter 'a' in a .js file or .ts file

type letter 'a', the keyword 'await' will not be shown in suggestion list. the problem appeard since this version 1.28.2.
the keyword 'await' is the first item in suggestion list at the version 1.27.2, and this is what i expected.

Duplicate of Microsoft/vscode#60133 / https://github.com/Microsoft/TypeScript/issues/27622


Remove C++ based HttpServer for testing in favor of Node.js based one

#647 introduced a Node.js based test server to get arround the limitations of the C++ based HttpServer specifically to get the macOS C++ unit tests to run.
https://github.com/maplibre/maplibre-gl-native/blob/ea234edf67bb3aec75f077e15c1c30c99756b926/test/include/mbgl/test/util.hpp#L61
Can we use the Node.js based server for other platforms as well? Then we can get rid of the HttpServer, since it is duplicated in two languages right now.

Trying to figure out how to get Qt tests running now I don't even see the server launched.
I'm a bit hesitant to introduce a node dependency for other platforms though, so we may still want to get the C++ based one running.

Not an option on iOS.


Re-authenticating Steam account crashes Playnite

Describe the bug
In 3.4, reauthenticating a private Steam account causes an unrecoverable error in Playnite. Does not occur on previous version.
To Reproduce
Replicated this twice.

Open the plugin settings and go to Libraries > Steam.
With 'is private' checked, click the Authenticate button.
Login.
After inputting the two-factor code from email, Playnite crashes.

Expected behavior
Not a crash!
Extensions log
Also contains playnite.log. Zipped for size.
extensions.log.zip

Can you try this version?
playnite-successstory-plugin_3_4.zip

That fixed the issue. I checked the version before and it said that it already was 3.4.

It's not working for me. No crash, but after inputting the two-factor code, I get "authenticating..." and then it goes back to Requires authentication.

Did you restart the application after installing the test version Lacro59 provided (even if you were "upgrading" from 3.4 to 3.4)? I was experiencing the exact behavior you were describing, and the fix worked immediately for me upon installing the offered version and rebooting Playnite.

Yes, I restarted. Then I restarted again after the first failure just to check.


fix: Performance improvements

use noodles async vcf reader & writer
cache some calls in hgvs-rs (see yet-to-be-created pull request in hgvs-rs)
add dependencies and do setup for DHAT, criterion, flamegraph, coz, pprof
add profiling and perf target profiles

I've also done a little bit of refactoring: the run_with_writer function now mostly has the DB setup and loops over the
VCF records, while the actual annotation is done in a different function etc. Makes for easier to read profiling results.

Currently, hvgs-rs and seqrepo-rs point to local project instances with some local changes.

Thanks! Could you fix the docker build? Somehow annotations seems to fail to build:
#27 0.097   can't find `annotation` bench at `benches/annotation.rs` or `benches/annotation/main.rs`. Please specify bench.path if you want to use a non-default path.

I've replaced the noodles-$project dependencies with the corresponding noodles group dependency, setting its features to the list of noodles-$projects that were used. That way, there's only one noodles version to bump and, more importantly, the projects are compatible.
This is especially relevant for vcf and bcf, as this ensures that the vcf::Header is the same across both, which allows us to create unified vcf/bcf readers/writers (i.e. mehari can read BCF now). We can move the relevant commits to a different branch, but for now it was more convenient (to me) to have them here.

Perhaps we can also take a look at https://github.com/apps/codspeed-hq (and https://github.com/CodSpeedHQ/codspeed-rust) to ensure performance doesn't degrade

[…] which allows us to create unified vcf/bcf readers/writers (i.e. mehari can read BCF now).

Once https://github.com/zaeleus/noodles/issues/266 is resolved, we can also write BCF (correctly). I don't think it will have much impact on performance, though, but it's nice to have, anyways.

Should we enable lto = "fat" and codegen-units = 1 (or some other small value) by default for the release profile? Currently, these are only set for the perf profile (such that the effect the settings have can be compared more easily)


Doc builds for naia-bevy-server are failing

See https://docs.rs/crate/naia-bevy-server/latest

Should be fixed now :+1: thanks for the heads up!


HeroCard buttons don't works

I'm trying to send simple yes/no buttons following the documentation.
Here's my code:
                    var activity = MessageFactory.Attachment(
                        new HeroCard(
                            title: $"Are you sure you want to delete order '{ this.State.Order.OrderNumber }'",
                            buttons: new CardAction[] {
                                new CardAction(title: "yes", text: "yes", value: "yes"),
                                new CardAction(title: "no", text: "yes", value: "no")
                            }
                        ).ToAttachment());

                    // Send the activity as a reply to the user.
                    context.Reply(activity);

I'm expecting messages when clicking on them; nothing happens.

@cleemullins , if this is not a bug in CardAction and you post the correct parameters to use, I will update the wiki page.

@JonathanFingold : Can you try it an see? 👍 The code is only 5 minutes to write and test.

@TheSquiffy you're not specifying the CardAction "type", should be "imBack", "postBack", or can be "messageBack" if using MS Teams

@Stevenic yes, you are right. Using imBack it's working, thank you. I think the wiki page should better clarify this ;-)

@JonathanFingold  - if you've a moment, could you update the Wiki, per this discusson? That change needs to be made across all the Wiki's...

@cleemullins Currently updating and verifying the C# sample. I'll port my changes over to the other wikis shortly.

While we're at it we should drop the text property from the samples. Only messageBack uses that field and messageBack is only supported on MS Teams.  The sample should use type, title, and value.

Fixed in the -dotnet repo, so closing here.

CardActions now all use type: ActionTypes.ImBack, title, and value.
Suggested actions now uses text to avoid the empty text bubble issue.

Will update the other repos presently.


cloud_to_cloud:node:... very flaky (TIMEOUT)

https://sponge.corp.google.com/target?id=80ee7ec6-5b64-4ed8-9704-e376ac09e159&target=github/grpc&searchFor=&show=ALL&sortBy=STATUS
For example:
cloud_to_cloud:node:csharpcoreclr_server:large_unary
cloud_to_cloud:node:python_server:large_unary
cloud_to_cloud:node:ruby_server:client_streaming

This dashboard shows how severe the issue is: https://performanceedit-dot-grpc-testing.appspot.com/explore?dashboard=5711255578869760: Node client is the most flaky one.

A bit more analysis shows that the node client is behind virtually all cloud_to_cloud interop test failures in the last days. Most failures are large_unary, followed by client_streaming.

grpc/ubuntu/master/grpc_interop_tocloud | 88 | null | cloud_to_cloud:node:c++_server:custom_metadata | cloud_to_cloud | node | c++_server | custom_metadata | TIMEOUT | 2017-11-03 22:05:43 UTC | 181.35 |
-- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | --
2 | grpc/ubuntu/master/grpc_interop_tocloud | 87 | null | cloud_to_cloud:node:c++_server:large_unary | cloud_to_cloud | node | c++_server | large_unary | TIMEOUT | 2017-11-03 17:48:06 UTC | 186.91 |
3 | grpc/ubuntu/master/grpc_interop_tocloud | 86 | null | cloud_to_cloud:node:csharpcoreclr_server:large_unary | cloud_to_cloud | node | csharpcoreclr_server | large_unary | TIMEOUT | 2017-11-03 16:28:36 UTC | 187.81 |
4 | grpc/ubuntu/master/grpc_interop_tocloud | 86 | null | cloud_to_cloud:node:c++_server:large_unary | cloud_to_cloud | node | c++_server | large_unary | TIMEOUT | 2017-11-03 16:28:43 UTC | 187.56 |
5 | grpc/ubuntu/master/grpc_interop_tocloud | 85 | null | cloud_to_cloud:node:c++_server:large_unary | cloud_to_cloud | node | c++_server | large_unary | TIMEOUT | 2017-11-03 15:48:23 UTC | 185.57 |
6 | grpc/ubuntu/master/grpc_interop_tocloud | 85 | null | cloud_to_cloud:node:python_server:large_unary | cloud_to_cloud | node | python_server | large_unary | TIMEOUT | 2017-11-03 15:47:48 UTC | 187.59 |
7 | grpc/ubuntu/master/grpc_interop_tocloud | 85 | null | cloud_to_cloud:node:csharp_server:large_unary | cloud_to_cloud | node | csharp_server | large_unary | TIMEOUT | 2017-11-03 15:47:31 UTC | 180.36 |
8 | grpc/ubuntu/master/grpc_interop_tocloud | 85 | null | cloud_to_cloud:node:python_server:client_streaming | cloud_to_cloud | node | python_server | client_streaming | TIMEOUT | 2017-11-03 15:48:30 UTC | 187.25 |
9 | grpc/ubuntu/master/grpc_interop_tocloud | 84 | null | cloud_to_cloud:node:c++_server:large_unary | cloud_to_cloud | node | c++_server | large_unary | TIMEOUT | 2017-11-03 14:46:00 UTC | 181.13 |
10 | grpc/ubuntu/master/grpc_interop_tocloud | 83 | null | cloud_to_cloud:node:ruby_server:large_unary | cloud_to_cloud | node | ruby_server | large_unary | TIMEOUT | 2017-11-03 09:45:30 UTC | 180.71 |
11 | grpc/ubuntu/master/grpc_interop_tocloud | 82 | null | cloud_to_cloud:node:python_server:large_unary | cloud_to_cloud | node | python_server | large_unary | TIMEOUT | 2017-11-03 07:46:03 UTC | 188.04 |
12 | grpc/ubuntu/master/grpc_interop_tocloud | 81 | null | cloud_to_cloud:node:python_server:client_streaming | cloud_to_cloud | node | python_server | client_streaming | TIMEOUT | 2017-11-03 02:23:04 UTC | 181.55 |
13 | grpc/ubuntu/master/grpc_interop_tocloud | 80 | null | cloud_to_cloud:node:c++_server:client_streaming | cloud_to_cloud | node | c++_server | client_streaming | TIMEOUT | 2017-11-02 15:07:50 UTC | 186.11 |
14 | grpc/ubuntu/master/grpc_interop_tocloud | 79 | null | cloud_to_cloud:node:c++_server:large_unary | cloud_to_cloud | node | c++_server | large_unary | TIMEOUT | 2017-11-02 13:31:24 UTC | 182.7 |
15 | grpc/ubuntu/master/grpc_interop_tocloud | 79 | null | cloud_to_cloud:node:ruby_server:large_unary | cloud_to_cloud | node | ruby_server | large_unary | TIMEOUT | 2017-11-02 13:31:08 UTC | 180.46 |
16 | grpc/ubuntu/master/grpc_interop_tocloud | 77 | null | cloud_to_cloud:node:python_server:client_streaming | cloud_to_cloud | node | python_server | client_streaming | TIMEOUT | 2017-11-02 03:11:00 UTC | 182.57 |
17 | grpc/ubuntu/master/grpc_interop_tocloud | 76 | null | cloud_to_cloud:node:c++_server:large_unary | cloud_to_cloud | node | c++_server | large_unary | TIMEOUT | 2017-11-01 20:54:49 UTC | 183.8 |
18 | grpc/ubuntu/master/grpc_interop_tocloud | 75 | null | cloud_to_cloud:node:python_server:large_unary | cloud_to_cloud | node | python_server | large_unary | TIMEOUT | 2017-11-01 15:54:38 UTC | 186.2 |
19 | grpc/ubuntu/master/grpc_interop_tocloud | 75 | null | cloud_to_cloud:node:ruby_server:client_streaming | cloud_to_cloud | node | ruby_server | client_streaming | TIMEOUT | 2017-11-01 15:55:50 UTC | 188.49 |
20 | grpc/ubuntu/master/grpc_interop_tocloud | 75 | null | cloud_to_cloud:node:csharpcoreclr_server:large_unary | cloud_to_cloud | node | csharpcoreclr_server | large_unary | TIMEOUT | 2017-11-01 15:55:08 UTC | 187.81 |
21 | grpc/ubuntu/master/grpc_interop_tocloud | 74 | null | cloud_to_cloud:node:c++_server:large_unary | cloud_to_cloud | node | c++_server | large_unary | TIMEOUT | 2017-11-01 13:34:11 UTC | 189.75 |
22 | grpc/ubuntu/master/grpc_interop_tocloud | 71 | null | cloud_to_cloud:node:c++_server:large_unary | cloud_to_cloud | node | c++_server | large_unary | TIMEOUT | 2017-10-31 22:31:47 UTC | 180.1 |
23 | grpc/ubuntu/master/grpc_interop_tocloud | 70 | null | cloud_to_cloud:node:csharpcoreclr_server:large_unary | cloud_to_cloud | node | csharpcoreclr_server | large_unary | TIMEOUT | 2017-10-31 16:35:16 UTC | 183.01 |
24 | grpc/ubuntu/master/grpc_interop_tocloud | 70 | null | cloud_to_cloud:node:csharp_server:client_streaming | cloud_to_cloud | node | csharp_server | client_streaming | TIMEOUT | 2017-10-31 16:35:11 UTC | 188.27 |
25 | grpc/ubuntu/master/grpc_interop_tocloud | 70 | null | cloud_to_cloud:node:python_server:large_unary | cloud_to_cloud | node | python_server | large_unary | TIMEOUT | 2017-10-31 16:35:00 UTC | 187.33 |
26 | grpc/ubuntu/master/grpc_interop_tocloud | 70 | null | cloud_to_cloud:node:csharpcoreclr_server:client_streaming | cloud_to_cloud | node | csharpcoreclr_server | client_streaming | TIMEOUT | 2017-10-31 16:35:21 UTC | 182.79 |
27 | grpc/ubuntu/master/grpc_interop_tocloud | 69 | null | cloud_to_cloud:node:csharp_server:large_unary | cloud_to_cloud | node | csharp_server | large_unary | TIMEOUT | 2017-10-31 13:34:46 UTC | 183.74 |
28 | grpc/ubuntu/master/grpc_interop_tocloud | 67 | null | cloud_to_cloud:node:ruby_server:client_streaming | cloud_to_cloud | node | ruby_server | client_streaming | TIMEOUT | 2017-10-30 22:35:24 UTC | 186.33 |

the query
select * from jenkins_test_results.interop_results where test_name like '%' and job_name like '%grpc_interop_tocloud%' and timestamp > '2017-10-25 00:00' and result != 'PASSED'  order by build_id desc limit 10000

Ping. This happens consistently.

Ping. Any update?

Another issue that started affecting node (looks like all node interop tests are failing).
Failure
/var/local/git/grpc-node/test/any_grpc.js:9
    throw new Error([
    ^

Error: Invalid value for global._client_implementation: undefined. If running from the command line, please --require a fixture first.
    at getImplementation (/var/local/git/grpc-node/test/any_grpc.js:9:11)
    at Object.<anonymous> (/var/local/git/grpc-node/test/any_grpc.js:23:20)
    at Module._compile (module.js:624:30)
    at Object.Module._extensions..js (module.js:635:10)
    at Module.load (module.js:545:32)
    at tryModuleLoad (module.js:508:12)
    at Function.Module._load (module.js:500:3)
    at Module.require (module.js:568:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/var/local/git/grpc-node/test/interop/interop_client.js:23:12)

That issue has been already filed as https://github.com/grpc/grpc-node/issues/89

@nicolasnoble and I talked offline about this yesterday. The plan is to have these be disabled while being investigated. @nicolasnoble can you confirm?

The fix for that latest issue is #13394.

This should be fixed by grpc/grpc-node#95.


GAP service hidden on Mac OS X Yosemite

First, I really appreciate what you are doing, keep going!!!!
I was able to confirm by using multiple peripherals products, that Linux shows GAP service characteristics for service discovery results ('uuid': '1800'), same for Generic Attributes ('uuid': '1801'), such items are just hidden in Mac OS X Yosemite.
Maybe related, reading GAP attributes is not possible (device name, appearance are mandatory, try with those please).
Related to reading GAP attributes, I can see that LightBlue reads the GAP device name attribute by issuing an ATT ready type request (start handle = 0x0001, end handle = 0xffff), as pointed out by core specification... so if there is no way Mac shows the service during discovery (so the actual handles can be read) this could be an alternative.
By the way... how do you get the com.apple.blued message id values and parameters (specially posting events to the xpc service)? I'm interested in providing patches (and bindings to Python)

@rene-aguirre yes, OS X 10.10 started behaving more like iOS's CoreBluetooth stack and does not report the GAP attributes.
The handles could be read, but they might not have the same handle number on all devices.
It would be great if you made a BLE library for Python! Checkout osx-ble-peripheral. If you uncomment the XPC_SPY define line  you can see all Central or Peripheral messages sent to blued for the CoreBluetooth API's you call.


fix: supply calldata as hexadecimal string array

Motivation and Resolution
The StarkNet OpenRPC API spec defines a FELT as a "...field element. Represented as up to 63 hex digits and leading 4 bits zeroed." The RPC write api defines three methods which take an array of FELT as calldata. The RpcProvider has three methods which incorrectly converts an array of BigNumbersih values into a decimal string array rather than an hexadecimal string array to pass as call data. Those methods are:

getEstimateFee()
deployContract()
invokeFunction()

This is the cause of the error described in issue #285

Adds new utility method bigNumberishArrayToHexadecimalStringArray() in utils/number.ts to return an hexadecimal string array from an array of BigNumberish values
Replaces the use of bigNumberishArrayToDecimalStringArray() in the three methods mentioned with bigNumberishArrayToHexadecimalStringArray()
Adds two tests for the RpcProvider test suite rpcProvider.tests.ts

Acknowledging the notice that invokeFunction() is deprecated, I have not added a test for it to the test suite.
Please feel free to make any changes to this PR, including the addition of a test for invokeFucntion(), if you deem it necessary.
Fixes #285
Usage related changes
Changes should be transparent to users.
Development related changes

Added 2 new tests to the RpcProvider test suite.

Checklist:

[X] Performed a self-review of the code
[X] Rebased to the last commit of the target branch (or merged it into my branch)
[X] Linked the issues which this PR resolves
[ ] Documented the changes
[ ] Updated the docs (www)
[X] Updated the tests
[X] All tests are passing

@dkillen we'll wait with this just for a bit, today/tomorrow once we restructure rpc tests and then merge

@dkillen hi! We merged the changes I mentioned in the comment before... So now, we have some conflicts... if you don't have time I can resolve them and the review comments I left... But I think it would have to be a separate PR, so wanted to see with you first, do you want to resolve it?

New PR: https://github.com/0xs34n/starknet.js/pull/315 so closing this one
Added you @dkillen as the co-author


Kontrol crashes upon reaching ECDSA.Recover

While debugging the WETH contract, I noticed that Kontrol would crash when attempting to prove the withdrawal function. Upon digging further into the reason for the failure, I've created a simplified, minimal version of the test that is failing to be proved, and placed it in ECDSA_Test branch. The crash log is as follows:
[Warn] Assertion failed
CallStack (from HasCallStack):
  assert, called at src/Kore/Builtin/Krypto.hs:308:5 in kore-<IP_ADDRESS>-AilWsOSAUb2IUrLwjjn1oR:Kore.Builtin.Krypto
ERROR 2024-02-07 11:03:50,562 kevm_pyk.utils - Proof crashed: src%ECDSARecover_Test.trigger_ECDSARecover(uint256):0
Runtime error
Traceback (most recent call last):
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/kore/rpc.py", line 808, in _request
    return self._client.request(method, **params)
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/kore/rpc.py", line 247, in request
    return self._default_client.request(method, **params)
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/kore/rpc.py", line 326, in request
    self._check(data)
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/kore/rpc.py", line 338, in _check
    raise JsonRpcError(**response['error'])
pyk.kore.rpc.JsonRpcError: Runtime error
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "/nix/store/nx08py50nggxs03bzv05cf34lb829drl-python3.10-kevm-pyk-1.0.418/lib/python3.10/site-packages/kevm_pyk/utils.py", line 117, in run_prover
    prover.advance_proof(
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/proof/reachability.py", line 773, in advance_proof
    self.advance_pending_node(
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/proof/reachability.py", line 1043, in advance_pending_node
    super().advance_pending_node(
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/proof/reachability.py", line 737, in advance_pending_node
    self.kcfg_explore.extend(
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/kcfg/explore.py", line 372, in extend
    extend_result = self.extend_cterm(
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/kcfg/explore.py", line 415, in extend_cterm
    cterm, next_cterms, depth, vacuous, next_node_logs = self.cterm_execute(
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/kcfg/explore.py", line 92, in cterm_execute
    response = self._kore_client.execute(
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/kore/rpc.py", line 856, in execute
    result = self._request('execute', **params)
  File "/nix/store/ap9c65dgpl2lfyfci44c5ddpvvbpwgff-python3.10-pyk-0.1.572/lib/python3.10/site-packages/pyk/kore/rpc.py", line 811, in _request
    raise KoreClientError(message=err.message, code=err.code, data=err.data) from err
pyk.kore.rpc.KoreClientError: Runtime error
[Info#proxy] Server shutting down
PROOF FAILED: src%ECDSARecover_Test.trigger_ECDSARecover(uint256):0

Previously to being brought here, this issue was also discussed with the Haskell backend team.
While I understand that Kontrol may fail to generate proof in such a scenario, I don't think that the tool should crash.
KEVM Claims generated by the minimal example: kevm-claims (1).txt

We ran some debugging with @anvacaru, and identified the potential root cause of the issue, which is the branching on symbolic CALLER_ID (i.e., msg.sender) address potentially being a precompiled address (which is not possible). In this case, #precompiled(X) is called on a symbolic value, while we're only computing it if X is in {0..9}.
The two possible ways to fix it are:

to label rules for isPrecompiled with [concrete] and [symbolic] tags so that it evaluates to false if X is symbolic (we should carefully consider the consequences of adding these and the necessary simplification tags):

https://runtimeverification.slack.com/archives/D05FXFMC2AZ/p1708674344850159
rule [isPrecompiledAccount.true]:  #isPrecompiledAccount(ACCTCODE, SCHED) => true  requires         ACCTCODE in #precompiledAccounts(SCHED) [concrete(ACCTCODE, SCHED)]
rule [isPrecompiledAccount.false]: #isPrecompiledAccount(ACCTCODE, SCHED) => false requires notBool ACCTCODE in #precompiledAccounts(SCHED) [concrete(ACCTCODE, SCHED)]
rule [isPrecompiledAccount.symbolic]: #isPrecompiledAccount(ACCT, _) => false                                                     [symbolic(ACCT), simplification]

add a constraint assuming that CALLER_ID is not a precompiled contract in Kontrol, similar to

_account_constraints.append(
  mlEqualsFalse(KEVM.is_precompiled_account(acct_id.args[0], cterm.cell('SCHEDULE_CELL')))
)

Further debugging and fixing of this issue is blocked by the fast that one of the Kontrol changes in the past 2-3 weeks introduced spurious branching to the test execution:
┣━━┓ constraint: { false #Equals ( false andBool #changesState ( CALL , ( chop ( ( 2300 *Int bool2Word ( VV0_wad_114b9705:Int ==Int 0 ) ) ) : ( CALLER_ID:Int : ( VV0_wad_114b9705:Int : ( 128 : ( 0 : ( 128 : ( 0 : ( 128 : ( VV0_wad_114b9705:Int : ( chop ( ( 2300 *Int bool2Word ( VV0_wad_114b9705:Int ==Int 0 ) ) ) : ( CALLER_ID:Int : ( VV0_wad_114b9705:Int : ( 271 : ( selector ( "testECDSARecover(uint256)" ) : .WordStack ) ) ) ) ) ) ) ) ) ) ) ) ) ) ) ) }
┃  │
┃  ├─ 4 (split)
┃  │   k: #next [ CALL ] ~> #execute ~> CONTINUATION:K
┃  │   pc: 1011
┃  │   callDepth: 0
┃  │   statusCode: STATUSCODE:StatusCode
┃  │   src: lib/forge-std/src/StdInvariant.sol:84:85
┃  ┃
┃  ┃ (branch)
┃  ┣━━┓ constraint: { false #Equals ( false andBool #changesState ( CALL , ( chop ( ( 2300 *Int bool2Word ( VV0_wad_114b9705:Int ==Int 0 ) ) ) : ( CALLER_ID:Int : ( VV0_wad_114b9705:Int : ( 128 : ( 0 : ( 128 : ( 0 : ( 128 : ( VV0_wad_114b9705:Int : ( chop ( ( 2300 *Int bool2Word ( VV0_wad_114b9705:Int ==Int 0 ) ) ) : ( CALLER_ID:Int : ( VV0_wad_114b9705:Int : ( 271 : ( selector ( "testECDSARecover(uint256)" ) : .WordStack ) ) ) ) ) ) ) ) ) ) ) ) ) ) ) ) }
┃  ┃  │
┃  ┃  ├─ 6 (split)
┃  ┃  │   k: #next [ CALL ] ~> #execute ~> CONTINUATION:K
┃  ┃  │   pc: 1011
┃  ┃  │   callDepth: 0
┃  ┃  │   statusCode: STATUSCODE:StatusCode
┃  ┃  │   src: lib/forge-std/src/StdInvariant.sol:84:85
┃  ┃  ┃
┃  ┃  ┃ (branch)
┃  ┃  ┣━━┓ constraint: { false #Equals ( false andBool #changesState ( CALL , ( chop ( ( 2300 *Int bool2Word ( VV0_wad_114b9705:Int ==Int 0 ) ) ) : ( CALLER_ID:Int : ( VV0_wad_114b9705:Int : ( 128 : ( 0 : ( 128 : ( 0 : ( 128 : ( VV0_wad_114b9705:Int : ( chop ( ( 2300 *Int bool2Word ( VV0_wad_114b9705:Int ==Int 0 ) ) ) : ( CALLER_ID:Int : ( VV0_wad_114b9705:Int : ( 271 : ( selector ( "testECDSARecover(uint256)" ) : .WordStack ) ) ) ) ) ) ) ) ) ) ) ) ) ) ) ) }
┃  ┃  ┃  │
┃  ┃  ┃  ├─ 8 (split)
┃  ┃  ┃  │   k: #next [ CALL ] ~> #execute ~> CONTINUATION:K
┃  ┃  ┃  │   pc: 1011
┃  ┃  ┃  │   callDepth: 0
┃  ┃  ┃  │   statusCode: STATUSCODE:StatusCode
┃  ┃  ┃  │   src: lib/forge-std/src/StdInvariant.sol:84:85
┃  ┃  ┃  ┃

I'm currently trying to isolate the version where this behavior was introduced.

I don't have any explanation for that yet, but it seems that this branching starts to occur in the mockCall PR in Kontrol: https://github.com/runtimeverification/kontrol/pull/318/files.

The solution to this problem, I believe, is to add constraints that say that the relevant contract address is not a precompiled account address. Something similar has been done in this PR.

Yes, it seems so, thank you @PetarMax! It's possible that this PR already fixed the issue  — the address in question was CALLER_ID, which was constrained. I'll give it a try today!

I just ran the test @ACassimiro provided
pragma solidity =0.8.13;

import "forge-std/Test.sol";

contract ECDSARecoverTest is Test {
    function testECDSARecover(uint256 wad) public {
        payable(msg.sender).transfer(wad);
    }
}

with the latest Kontrol version (0.1.187), and I don't see any of the above issues anymore: there's no spurious branching and no crashes on the #isPrecompiledAccount check (so it got fixed by https://github.com/runtimeverification/evm-semantics/pull/2317).
At the same time, it fails in a minute (with --fail-fast) with the KCFG I posted below. #checkBalanceUnderflow looks normal to me, considering how unconstrained the transfer in the test is, but I don't think #call_foundry 0 b"" is supposed to happen — I suppose it's an indicator that another constraint on CALLER_ID we should add is that it's not the Foundry cheatcode address.
┌─ 1 (root, init)
│   k: #execute ~> CONTINUATION:K
│   pc: 0
│   callDepth: 0
│   statusCode: STATUSCODE:StatusCode
│   src: test/ECDSATest.sol:4:9
│
│  (406 steps)
├─ 3
│   k: #next [ CALL ] ~> #execute ~> CONTINUATION:K
│   pc: 657
│   callDepth: 0
│   statusCode: STATUSCODE:StatusCode
│   src: lib/forge-std/src/StdInvariant.sol:76:79
┃
┃ (1 step)
┣━━┓
┃  │
┃  ├─ 4
┃  │   k: #addr [ CALL ] ~> #exec [ CALL ] ~> #pc [ CALL ] ~> #execute ~> CONTINUATION:K
┃  │   pc: 657
┃  │   callDepth: 0
┃  │   statusCode: STATUSCODE:StatusCode
┃  │   src: lib/forge-std/src/StdInvariant.sol:76:79
┃  │
┃  │  (5 steps)
┃  ├─ 6
┃  │   k: #checkCall<PHONE_NUMBER>85977040452943777879061427756277306518 VV0_wad_114b9705:Int ...
┃  │   pc: 657
┃  │   callDepth: 0
┃  │   statusCode: STATUSCODE:StatusCode
┃  │   src: lib/forge-std/src/StdInvariant.sol:76:79
┃  ┃
┃  ┃ (1 step)
┃  ┣━━┓
┃  ┃  │
┃  ┃  ├─ 7
┃  ┃  │   k: #call_foundry 0 b"" ~> #return_foundry 128 0 ~> #pc [ CALL ] ~> #execute ~> CONT ...
┃  ┃  │   pc: 657
┃  ┃  │   callDepth: 0
┃  ┃  │   statusCode: STATUSCODE:StatusCode
┃  ┃  │   src: lib/forge-std/src/StdInvariant.sol:76:79
┃  ┃  │
┃  ┃  │  (1 step)
┃  ┃  └─ 9 (stuck, leaf)
┃  ┃      k: #error_foundry 0 b"" ~> #return_foundry 128 0 ~> #pc [ CALL ] ~> #execute ~> CON ...
┃  ┃      pc: 657
┃  ┃      callDepth: 0
┃  ┃      statusCode: FOUNDRY_UNIMPLEMENTED
┃  ┃      src: lib/forge-std/src/StdInvariant.sol:76:79
┃  ┃
┃  ┗━━┓
┃     │
┃     ├─ 8 (split)
┃     │   k: #checkBalanceUnderflow<PHONE_NUMBER>85977040452943777879061427756277306518 VV0_wad_ ...
┃     │   pc: 657
┃     │   callDepth: 0
┃     │   statusCode: STATUSCODE:StatusCode
┃     │   src: lib/forge-std/src/StdInvariant.sol:76:79
┃     ┃
┃     ┃ (branch)
┃     ┣━━┓ constraint: { true #Equals 0 <Int VV0_wad_114b9705:Int }
┃     ┃  │
┃     ┃  └─ 10 (leaf, pending)
┃     ┃      k: #checkBalanceUnderflow<PHONE_NUMBER>85977040452943777879061427756277306518 VV0_wad_ ...
┃     ┃      pc: 657
┃     ┃      callDepth: 0
┃     ┃      statusCode: STATUSCODE:StatusCode
┃     ┃      src: lib/forge-std/src/StdInvariant.sol:76:79
┃     ┃
┃     ┗━━┓ constraint: { true #Equals VV0_wad_114b9705:Int <=Int 0 }
┃        │
┃        └─ 11 (leaf, pending)
┃            k: #checkBalanceUnderflow<PHONE_NUMBER>85977040452943777879061427756277306518 VV0_wad_ ...
┃            pc: 657
┃            callDepth: 0
┃            statusCode: STATUSCODE:StatusCode
┃            src: lib/forge-std/src/StdInvariant.sol:76:79
┃
┗━━┓
   │
   └─ 5 (vacuous, leaf)
       k: #next [ CALL ] ~> #execute ~> CONTINUATION:K
       pc: 657
       callDepth: 0
       statusCode: STATUSCODE:StatusCode
       src: lib/forge-std/src/StdInvariant.sol:76:79

┌─ 2 (root, leaf, target, terminal)
│   k: #halt ~> CONTINUATION:K
│   pc: PC_CELL_5d410f2a:Int
│   callDepth: CALLDEPTH_CELL_5d410f2a:Int
│   statusCode: STATUSCODE_FINAL:StatusCode

A call to Foundry cheatcode contract<PHONE_NUMBER>26547203313410069153905908525362434349) can be avoided using the notBuiltinAddress(address) function that is shipped with KontrolCheats contract:
contract ECDSARecoverTest is Test, KontrolCheats {
    function testECDSARecover(uint256 wad) public {
        vm.assume(notBuiltinAddress(msg.sender));
        payable(msg.sender).transfer(wad);
    }
}

In this case, though, the branching still occurs, but only negative branch gets executed (KCFG below). @ACassimiro @anvacaru do you think we can consider this issue closed?
┌─ 1 (root, init)
│   k: #execute ~> CONTINUATION:K
│   pc: 0
│   callDepth: 0
│   statusCode: STATUSCODE:StatusCode
│   src: test/ECDSATest.sol:5:11
│
│  (427 steps)
├─ 3 (split)
│   k: JUMPI 2230 bool2Word (<PHONE_NUMBER>26547203313410069153905908525362434349 ==Int CA ...
│   pc: 2195
│   callDepth: 0
│   statusCode: STATUSCODE:StatusCode
┃
┃ (branch)
┣━━┓ constraint: { true #Equals<PHONE_NUMBER>26547203313410069153905908525362434349 ==Int CALLER_ID:Int }
┃  │
┃  ├─ 4
┃  │   k: JUMPI 2230 bool2Word (<PHONE_NUMBER>26547203313410069153905908525362434349 ==Int CA ...
┃  │   pc: 2195
┃  │   callDepth: 0
┃  │   statusCode: STATUSCODE:StatusCode
┃  │
┃  │  (357 steps)
┃  └─ 6 (vacuous, leaf)
┃      k: #next [ CALL ] ~> #execute ~> CONTINUATION:K
┃      pc: 846
┃      callDepth: 0
┃      statusCode: STATUSCODE:StatusCode
┃      src: lib/forge-std/src/StdInvariant.sol:73:74
┃
┗━━┓ constraint: { true #Equals ( notBool<PHONE_NUMBER>26547203313410069153905908525362434349 ==Int CALLER_ID:Int ) }
   │
   ├─ 5
   │   k: JUMPI 2230 bool2Word (<PHONE_NUMBER>26547203313410069153905908525362434349 ==Int CA ...
   │   pc: 2195
   │   callDepth: 0
   │   statusCode: STATUSCODE:StatusCode
   │
   │  (400 steps)
   ├─ 7
   │   k: #next [ CALL ] ~> #execute ~> CONTINUATION:K
   │   pc: 846
   │   callDepth: 0
   │   statusCode: STATUSCODE:StatusCode
   │   src: lib/forge-std/src/StdInvariant.sol:73:74
   ...

The constraint on the address not being Foundry cheatcode's address will be added by @PetarMax in https://github.com/runtimeverification/kontrol/pull/355, which will close this issue. In the meantime, the vm.assume(...) workaround can be used to avoid reverting on a call to the cheatcode contract.

The above issue should get fixed by https://github.com/runtimeverification/kontrol/pull/427.

Thank you, @palinatolmach!


UNSAFE_componentWillReceiveProps warning appears when a day is clicked

Check that this is really a bug

[X] I confirm

Reproduction link
https://github.com/FrostAskar/Pawcients-Veterinary-Solutions.git
Bug description
I'm using this library for mi TFG project. The problem is that i am trying to put in a reactive variable the selected day on the calendary. For achieve this, i'm using the "onSelectSlot" event athat calls the handleSelectDay function, which will only be in charge of assigning the day received in the props as the value of the selectedDay variable.
In short, this is the way I use the calendar:
<Calendar
localizer={localizer}
events={events}
startAccessor="start"
endAccessor="end"
selectable={true}
onSelectEvent={handleEventSelect}
onSelectSlot={handleDateSelect}
onNavigate={ handleDateSelect }
min={minTime}
max={maxTime}
onView={handleView}
style={{ height: 600 }}
/>
And "handleDateSelect" is implemented as follows:
const handleDateSelect = date => {
setSelectedDate(date);
};
The problem is the following:
When I click on the "squares" of the calendar, which represent each day of the month, the function handleSelectDay is called correctly. However, when the specific number, that is, the button, is clicked, this function is not called and the selected day does not change. I have also tried using the onNavigate event, but the day is still not assigned properly. Also, when the day is selected using the button (and therefore the view changes to Day) the following error is displayed on the console:
"react-dom.development.js:86 Warning: Using UNSAFE_componentWillReceiveProps in strict mode is not recommended and may indicate bugs in your code. See https://reactjs.org/link/unsafe-component-lifecycles for details.

Move data fetching code or side effects to componentDidUpdate.
If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://reactjs.org/link/derived-state

Please update the following components: DayColumn"
I can't update the DayColumn component because it is internal to the library.
So I've done a lot of research on how to fix this error, trying to update the versions or trying different options, and I can only think that it could be a bug in the library.
Expected Behavior
I expect that the day changes when clicking on the button that switches you to the Day view.
Actual Behavior
When I click on the button with the number of the month, the onSelectSlot event is not emitted, therefore the day does not change and an error is displayed in the console.
react-big-calendar version
1.6.9
React version
18.2.0
Platform/Target and Browser Versions
Windows 10
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a react-big-calendar issue and not an implementation issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

The UNSAFE error is related to DayColumn and not related to not sending the event. There are other issues open for that. If you need clicking on the number to do something else, you can always pass a custom component.
Example from the doc:
https://github.com/jquense/react-big-calendar/blob/c7cd908936fefc989b86ed48d5c6248f87733af5/stories/helpers/customComponents.js#L5
Maybe what you're looking for is also onDrillDown?


Setup in readme return an error with eslint

Issues

[X] I have checked existing issues and there are no existing ones with the same request.

Feature description
Hi,
I'm new to neovim ecosystem, when following the readme to install none.ls in the Setup section I found a bug, when adding the following lines to my configuration:
        'nvimtools/none-ls.nvim',
        dependencies = {
            'nvim-lua/plenary.nvim',
        },
        config = function()
            local null_ls = require("null-ls")

            null_ls.setup({
                sources = {
                    null_ls.builtins.formatting.stylua,
                    null_ls.builtins.diagnostics.eslint,
                    null_ls.builtins.completion.spell,
                },
            })
        end

I have an error:
[null-ls] failed to load builtin eslint for method diagnostics; please check your config

Finding https://github.com/nvimtools/none-ls.nvim/discussions/81, I think that is because eslint has been moved to none-ls-extras.nvim because this is working:
        'nvimtools/none-ls.nvim',
        dependencies = {
            'nvim-lua/plenary.nvim',
            "nvimtools/none-ls-extras.nvim",
        },
        config = function()
            local null_ls = require("null-ls")

            null_ls.setup({
                sources = {
                    null_ls.builtins.formatting.stylua,
                    require("none-ls.diagnostics.eslint"),
                    null_ls.builtins.completion.spell,
                },
            })
        end

Can you please fix the readme if I'm right and add section about none-ls-extras.nvim? I can submit a PR if you want :slightly_smiling_face:
If I'm in the wrong can you explain me why? I can't find any answers.
Thanks in advance!
Help
Yes
Implementation help
No response

Thanks for the catch! This is now fixed in f44485f :)


on ubuntu 14, should we create an upstart-compatible service

Sorry for not answering the questions asked, but this is not a bug report.
If we created a /etc/init/haproxy.conf, we could for example add the respawn instruction to auto-restart haproxy if it crashes for some reason...
Would be a nice thing to have - alongside with the same thing for systemd and etc...

I'd be interested in someone picking this up with better haproxy experience than I.
If anyone could weigh in, on possible solutions have at it.

As ubuntu 14.04 is going EOL soon, unless someone wants to implement this the cookbook isn't going to support this going forward as we'd be stripping out the feature pretty soon.


Gene View - Numbers shown in legend are wrong

In the Gene View search results, the numbers of evidence per type shown in the legend are not correct; they seem to be higher than the actual number.
For example, search in Arabidopsis KM for "seed dormancy" matches 2 trait terms but the legend shows 14, see here:

yes, need to check the interactive GeneView legend code at https://github.com/Rothamsted/knetminer/blob/master/common/client-base/src/main/webapp/html/GeneView/summariseLegend.js that uses the full text from GeneView .tab to count each concept type.
Am not sure why this doesn't match the individual counts from the visible table.
Could be that we parsed the whole GeneView table .tab instead of just what's visible (i.e., we show all genes from gene list, if one was provided, or top 100 I think if a search returns 1000s of genes) but the .tab contains all the results not just what's visible, and the legend counts indicate the total count per concept-type & there may be more than 2 traits just not shown in the visible table.  @KeywanHP  @dicknetherlands

The Gene View Legend is even wrong when I search for ONE single gene:

The Evidence View Legend is correct though:

fixed in https://github.com/Rothamsted/knetminer/commit/58b1aab1340968f6954849b746680ccc91ffb1a9


Backbone of checks/trends system

It is still WIP, just playing around and trying to figure out how it could look like
Currently:

:fire: human like reasons rock!


Update PR guidelines in PULL_REQUEST_TEMPLATE

Cactoos project updated PR guidelines and I think it would be nice if we do that too. The new rules are:

You made changes related to only one bug (create separate PRs for separate problems, or leave puzzles)
You ran the build locally and it passed
Title begins with the issue's number, then a short title
Description begins with the issue's number, then enumerates the changes - summarized - in bulletpoints
Description does not contain GitHub keywords (https://help.github.com/articles/closing-issues-using-keywords/).

@0crat in

@0crat in (here)

@paulodamaso Job #312 is now in scope, role is DEV

Bug was reported, see §29: +15 point(s) just awarded to @fabriciofx/z

The job #312 assigned to @mohamednizar/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@fabriciofx , this was done, why the ticket still not closed ?

@0crat wait

@0crat wait (here)

@mohamednizar The impediment for #312 was registered successfully by @mohamednizar/z

Closed by #350

@paulodamaso/z please, pay attention to this issue

Closed by #350

@0crat continue

@fabriciofx/z please review this job completed by @mohamednizar/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@0crat continue  (here)

@mohamednizar Job #312 is not on hold, no impediment to remove

The job #312 is now out of scope

@0crat quality good

Quality review completed: +4 point(s) just awarded to @fabriciofx/z

Order was finished, quality is "good": +35 point(s) just awarded to @mohamednizar/z


fix: add publish config

adding publishConfig with access:public
the no-private param only is to exclude private packages from being processed.

Good :)


Changed CallActivity launchMode from singleTop -> singleTask

Purpose

On first app run before video permissions are set, during a call, tapping video to display the permission dialogue, then going to background and returning via Notification Tray would trigger CallActivity.onCreate(), since the previous launchMode was singleTop, which would do so, since the top task was the permission dialogue instead of CallActivity itself.
This fix calls CallActivity.onResume() properly, which prevents a crash

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

What to Check
Verify that the following are valid

Uninstall app, reinstall
Go into call before video permission are set
Tap Camera button to display Camera permissions dialogue
Minimize app
Re-open app via Notification Tray
Note that the app does not crash

Other Information

@Gemakk detected behaviour change:
on permission request pop up, deny video permissions -> application moves to intro screen
previously, application stayed on setup screen

on permission request pop up, deny video permissions -> application moves to intro screen
previously, application stayed on setup screen
tested, as per discussion above is not introduced by this change.


Remove table/column names from TupleValueExpression

I can't think of any reason that we need to store the table and column names in TupleValueExpression (other than for debugging). We should remove these attributes.

Removed it.


[SPARK-24252][SQL] Add v2 catalog plugin system

What changes were proposed in this pull request?
This adds a v2 API for adding new catalog plugins to Spark.

Catalog implementations extend CatalogPlugin and are loaded via reflection, similar to data sources
Catalogs loads and initializes catalogs using configuration from a SQLConf
CaseInsensitiveStringMap is used to pass configuration to CatalogPlugin via initialize

Catalogs are configured by adding config properties starting with spark.sql.catalog.(name). The name property must specify a class that implements CatalogPlugin. Other properties under the namespace (spark.sql.catalog.(name).(prop)) are passed to the provider during initialization along with the catalog name.
This replaces #21306, which will be implemented in two multiple parts: the catalog plugin system (this commit) and specific catalog APIs, like TableCatalog.
How was this patch tested?
Added test suites for CaseInsensitiveStringMap and for catalog loading.

@mccheah, @cloud-fan, could you review?
Matt, I've updated this to use CatalogPlugin from our discussion on #21306.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8336/
Test PASSed.

Test build #102846 has started for PR 23915 at commit c584e35.

Overall it looks reasonable; it's about what I would expect for a pluggable system that relies on classloading and reflective instantiation. We do this in other places too, for example in SchedulerBackend bootstrapping.

IIRC we want to follow Presto and support config files as well. Is it still in the roadmap?

Test build #102846 has finished for PR 23915 at commit c584e35.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class CaseInsensitiveStringMap implements Map<String, String>
public class Catalogs

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102846/
Test PASSed.

Could config files be implementation-specific? For example I'd want my Data Source implementation to be given a YML file but another data source implementation wants a properties file. Deserialization of the file doesn't make sense to be done on the Spark side, but in the internal implementation of the data source. Thus I'd imagine one of the properties passed to the data source implementation would be the path to the configuration to deserialize, and we leave it up to the implementation to decide how to handle that path in initialize.
So I don't think we need to make configuration files as an explicit part of the API here.

IIRC we want to follow Presto and support config files as well. Is it still in the roadmap?

I hadn't considered this before. Can we add it in a follow-up if it is something that we want to do?

@mccheah, @cloud-fan, I've updated the PR and replied to your comments. Please have a look. Thank you!

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8372/
Test PASSed.

Test build #102891 has started for PR 23915 at commit 17e6de1.

Test build #102891 has finished for PR 23915 at commit 17e6de1.

This patch fails Java style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102891/
Test FAILed.

Test build #102918 has started for PR 23915 at commit 6edb880.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8391/
Test PASSed.

Test build #102918 has finished for PR 23915 at commit 6edb880.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102918/
Test FAILed.

Retest this please.

Test build #103071 has started for PR 23915 at commit 6edb880.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8534/
Test PASSed.

Test build #103071 has finished for PR 23915 at commit 6edb880.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103071/
Test PASSed.

@cloud-fan, I've updated the docs as you requested. There should be no need to wait for tests because the change was to docs only.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8564/
Test PASSed.

Test build #103102 has started for PR 23915 at commit 7e50ca6.

Test build #103102 has finished for PR 23915 at commit 7e50ca6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103102/
Test FAILed.

The change itself LGTM(except one comment) but I need a little more information to justify the design:

why the initialize method takes the name parameter? Since one table catalog can be registered more than once withh different names, end users would expect the table catalog is not sensitive to the names. I think it's better if Spark can force the table catalog to be not sensitive to the names, i.e. do not give name when initializing table catalog.
why the table catalog needs to report the name? Spark has a map of string to table catalog, when Spark gets a table catalog, the name should already be known.

Interestingly the name doesn't exist in the original PR: https://github.com/apache/spark/pull/21306/files#diff-81c54123a7549b07a9d627353d9cbf95R49 . I'm wondering what has been hanged recently.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8580/
Test PASSed.

Test build #103118 has started for PR 23915 at commit 7e50ca6.

Test build #103118 has finished for PR 23915 at commit 7e50ca6.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103118/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8586/
Test PASSed.

Test build #103128 has started for PR 23915 at commit 7e50ca6.

Test build #103128 has finished for PR 23915 at commit 7e50ca6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103128/
Test PASSed.

Test build #103162 has started for PR 23915 at commit f118a3d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8615/
Test PASSed.

Test build #103165 has started for PR 23915 at commit 7c64a26.

Test build #103162 has finished for PR 23915 at commit f118a3d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103162/
Test PASSed.

Test build #103165 has finished for PR 23915 at commit 7c64a26.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class Catalogs
public class CaseInsensitiveStringMap implements Map<String, String>

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103165/
Test PASSed.

thanks, merging to master!

What's the difference between the existing org.apache.spark.sql.catalyst.util.CaseInsensitiveMap in sql/api and org.apache.spark.sql.util.CaseInsensitiveStringMap added here in sql/catalyst?
E.g. DataFrameReader and DataFrameWriter and then has to convert with val dsOptions = new CaseInsensitiveStringMap(finalOptions.asJava)

It's just a java friendly version of CaseInsensitiveMap.


Random tokens appearing on rename menu

This is a very specific bug that I have only gotten to occur on rare occasion, and thus I cannot recreate it consistently or identify any probable cause. The only constant I know here is that it is always with a program that has an 8 character long name.
The bug is that hitting rename on such a program displays a bunch of random tokens, that even wrap to the other side of the screen.
See here:

Can reproduce. I've got this on occasion myself -- I thought it was jus a glitch, not an actual bug. I guess the multiple recurrences should've clued me in -- hah!


Webpage not presenting correctly when published using GitHub Pages

I've been creating my site and everyting looks great locally (thanks for the template) but when I push to my main branch and the GitHub Action runs, my webpage appears completely broken (seems like missing css and images...). I'm sure I have just missed something simple but would appreicate the help. You can view the live site at Keydoo Software

Turns out the issue was between the chair and screen (me) - I had the base url wrong, apologies!


Discuss solution for object names in notifications

Notifications on a single line.  Will be taken care of with #1034


Use sscanf_s for msvc?

Derived from #26.
Use sscanf_s instead of disabling warning? This seems better for me.

I can do it, since I am a victim of MS compiler.
Originally i thought that you wouldn't want to write too much of platform specific code.