Bogus error message

@hs-apotell this new error message is incorrect:
https://github.com/chipsalliance/Surelog/blob/b4ad3fc02ef02ad162f34ac7d18c397e5fd6e9d9/src/Design/FileContent.cpp#L689
The testcase black-parrot has indeed statements that cross files (Due to includes).
Please remove that check. It is not compliant with Verilog proprocessing.
The issue is discussed here, it is a sub-issue of that case:
https://github.com/antmicro/yosys-systemverilog/issues/1064#issuecomment-1255237701

Sorry, try to extract the example from black-parrot. Its your check. But before, try to debug the latest black-parrot from th original GitHub, not the snapshot in Surelog.
This is an important Surelog user complaining, please prioritize accordingly.

@hs-apotell, thanks.


Demonstrate source detection result evaluation

We should demonstrate how one could evaluate results of the source detection workflow. For example we could quantify quality vs. noise, quality vs. faint/strong sources, runtime, confusion matrix, ...
Perhaps Olga has some ideas what/how we should analyze source detection.

Example ideas for plots:

Plot accuracy vs. noise
Plot accuracy vs. source size
https://arxiv.org/pdf/2110.08618.pdf
https://web.archive.org/web/20190501212002id_/https://iopscience.iop.org/article/10.3847/1538-4357/aa8d1e/pdf

Need to test this with more sources.


How can I train my dataset based on the pretrained model?

Thank you for your good code. I have a question that, how can I train my own dataset based on the pretrained (coco) model? I want to fine-tunning using my data based on your model.
I don't know that way so if you answer my question, I will be happy from it.

@changhyuk27 did you find any solution?


Added support for symfony process 6.

I've updated the composer.json to permit symfony 6 components.
I used the Vagrant environment to run the tests using php8.0 with the symfony 6 components and everything passed fine.
If theres anything i've missed, let me know.

assuming the repo is dead, recommend anyone seeing this to use the spatie/image-optimizer replacement, it's pretty much a drop in replacement


fix(docs): update valibot snippet

Overview

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests / types / typos

Description
I was trying to implement modular forms like on docs but when I installed valibot ^0.32.0 found there is no exported type Input (I think it's outdated).
When I looked in valibot docs they use InferOutput instead.
Use cases and why

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

const LoginSchema = object({
  email: string([
    minLength(1, 'Please enter your email.'),
    email('The email address is badly formatted.'),
  ]),
  password: string([
    minLength(1, 'Please enter your password.'),
    minLength(8, 'Your password must have 8 characters or more.'),
  ]),
});

I'm not a Valibot expert but this snippet from docs gives this error:
Argument of type '(MinLengthAction<LengthInput, 1, "Please enter your email."> | EmailAction<string, "The email address is badly formatted.">)[]' is not assignable to parameter of type 'ErrorMessage<StringIssue> | undefined'.ts(2345)
When I wrote it that way 👇. The error removed but I'm not sure if it's the same thing.
const LoginSchema = object({
  email: pipe(
    string(),
    minLength(1, 'Please enter your email'),
    email('he email address is badly formatted'),
  ),
  password: pipe(
    string(),
    minLength(1, 'Please enter your password.'),
    minLength(8, 'Your password must have 8 characters or more.'),
  ),
})

const LoginSchema = object({
  email: string([
    minLength(1, 'Please enter your email.'),
    email('The email address is badly formatted.'),
  ]),
  password: string([
    minLength(1, 'Please enter your password.'),
    minLength(8, 'Your password must have 8 characters or more.'),
  ]),
});

I'm not a Valibot expert but this snippet from docs gives this error:
Argument of type '(MinLengthAction<LengthInput, 1, "Please enter your email."> | EmailAction<string, "The email address is badly formatted.">)[]' is not assignable to parameter of type 'ErrorMessage<StringIssue> | undefined'.ts(2345)
When I wrote it that way 👇. The error removed but I'm not sure if it's the same thing.
const LoginSchema = object({
  email: pipe(
    string(),
    minLength(1, 'Please enter your email'),
    email('he email address is badly formatted'),
  ),
  password: pipe(
    string(),
    minLength(1, 'Please enter your password.'),
    minLength(8, 'Your password must have 8 characters or more.'),
  ),
})

@fabian-hiller you are the master here :smile: can you have a look pls?

Thanks @zougari47! Can you also change the imports to import * as v from 'valibot';? Here is more context:

https://valibot.dev/blog/valibot-v0.31.0-is-finally-available/
https://valibot.dev/guides/migrate-to-v0.31.0/

Thanks @zougari47! Can you also change the imports to import * as v from 'valibot';? Here is more context:
* https://valibot.dev/blog/valibot-v0.31.0-is-finally-available/

* https://valibot.dev/guides/migrate-to-v0.31.0/

Done

Thanks @zougari47! Can you also change the imports to import * as v from 'valibot';? Here is more context:
* https://valibot.dev/blog/valibot-v0.31.0-is-finally-available/

* https://valibot.dev/guides/migrate-to-v0.31.0/

Done ✅. Thank you for the review.

Thanks @zougari47! Can you also change the imports to import * as v from 'valibot';? Here is more context:
* https://valibot.dev/blog/valibot-v0.31.0-is-finally-available/

* https://valibot.dev/guides/migrate-to-v0.31.0/

Done ✅. Thank you for the recomandation.

Can you also update this file? https://github.com/QwikDev/qwik/blob/main/packages/docs/src/routes/demo/integration/modular-forms/index.tsx

One last thing. Can you change v.minLength(1, 'Please enter your email.') to v.nonEmpty('Please enter your email.')?

One last thing. Can you change v.minLength(1, 'Please enter your email.') to v.nonEmpty('Please enter your email.')?

Done ✅. Thank you for the review.


[0.2] Backports

Backport the following:

https://github.com/rust-lang/libc/pull/389

Can't backport https://github.com/rust-lang/libc/pull/3905 due to MSRV


[Enhancement]: Pagination for Contributor section

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
The contributors section is too long and can be overwhelming.
Describe the solution you'd like

Add pagination at the bottom of the contributor section
Use 20-30 contributors per page
Add 'Total contributors so far! : (number)' subheading to indicate contributor count, below 'Our Valuable Contributors' heading

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @ShreeluSantosh !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Pull request: #3964

Hey @ShreeluSantosh ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Laravel Updater

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v7.28.2 branch.
Before merging, you need to:

Checkout the shift-ci-v7.28.2 branch
Review all pull request comments for additional changes
Run composer update
Test your application (no tests?)

:information_source: You are running an older version of Laravel. The latest version is 8.3.0. Run the Laravel 8.x Shift to upgrade to the latest version of Laravel.

:warning: You are running version 2.4.0 of laravel/ui. The latest version is 3.0.0. You should review this dependency to see if you can upgrade to the latest version.


Memory leaks using JFXMasonryPane

Hi ! I'm facing an issue and I don't know if it's a bug or if i don't use the Masonry component correctly.
1. Issue
I have a use case where i need to add and remove childrens in the masonry pane. For example, displayed nodes can be filtered using a text input.

So, when the user submit a text query:

I remove all child in masonry pane using getChildrens.clear()
I  clear previous layouting using masonryPane.clearLayout()
I add nodes corresponding to the user text query getChildrens.add(...)

Maybe the component is not initially designed to do that but after each use of my filtering feature, i noticed that the memory footprint of my app increase. So i suspected a memory leak somewhere. I run an analysis on my app memory dump and the tool seems to indicate that the JFXMasonryPane component keep holding all nodes references throughout the animationMap attribute.

I started my investigation reading the JFXMasonryPane.java file and i can't see any point where the content of this map is flushed / cleared... So for me, the map keeps references of removed child nodes.
2. Reproduce
I've created a small example to reproduce the issue. You can add nodes to the masonry pane, clears them, add new ones and see the animationMap size growing.
public class Main extends Application{

  @Override
  public void start(Stage stage) throws Exception {

    VBox root = new VBox();
    JFXMasonryPane masonryPane = new JFXMasonryPane();
    JFXButton addButton = new JFXButton("+ Add a node in masonry pane");
    addButton.setOnAction(e -> {
      masonryPane.getChildren().add(new Label("Hi there !"));
      displayMasonryPaneStats(masonryPane);

    });

    JFXButton clearButton = new JFXButton("- Clear all nodes");
    clearButton.setOnAction(e -> {
      masonryPane.getChildren().clear();
      masonryPane.clearLayout();
      displayMasonryPaneStats(masonryPane);

    });

    root.getChildren().add(addButton);
    root.getChildren().add(clearButton);
    root.getChildren().add(masonryPane);
    Scene scene = new Scene(root, 400, 400);
    stage.setScene(scene);
    stage.centerOnScreen();
    stage.show();
  }

  public void displayMasonryPaneStats(JFXMasonryPane masonryPane) {

    try {
      //Prints masonry pane childrens list size
      System.out.println("MansoryPane - Childrens count:" + masonryPane.getChildren().size());
      //Prints masonry pane animation map size using reflection (can be monitored using debugger)
      Field f = masonryPane.getClass().getDeclaredField("animationMap");
      f.setAccessible(true);
      HashMap<Region, Transition> animationMap = (HashMap<Region, Transition>) f.get(masonryPane);
      System.out.println("MansoryPane - AnimationMap size:" + animationMap.size());

    } catch (Exception e) {
      e.printStackTrace();
    }
  }
}

Output example
MansoryPane - Childrens count:1
MansoryPane - AnimationMap size:0
MansoryPane - Childrens count:2
MansoryPane - AnimationMap size:1
MansoryPane - Childrens count:3
MansoryPane - AnimationMap size:2
MansoryPane - Childrens count:4
MansoryPane - AnimationMap size:3
MansoryPane - Childrens count:0
MansoryPane - AnimationMap size:4
MansoryPane - Childrens count:1
MansoryPane - AnimationMap size:4
MansoryPane - Childrens count:2
MansoryPane - AnimationMap size:5

3. Possible solution
For me, a possible fix is to remove childs from animationMap when they are removed from children collection. This can be done on the existing childrenListener using:
for(Node removedNode : change.getRemoved()) {
    animationMap.remove(removedNode);
}

Since, i don't really know how the MasonryPane animation works, i don't know if it's really safe to remove animationMap entries in the childrenListener.
Maybe there is other alternatives to fix this issue, so i will wait for feedbacks before submitting any PR.
Thank you.
DropSnorz.

Hello @DropSnorz,
Thanks for pointing out the issue 👍 , yes you are right animationMap is not being cleared when removing masonry pane children. I believe your solution is a valid solution, go ahead and create PR for the issue.
Thanks again 👍
Regards,


Vision Face detection uses non-standard words

When I do face detection, the API surfaces uses certain terms for emotions, and we seem to be using arbitrary synonyms for those, which doesn't seem quite right...
I suspect it's because the sentence we were using for emotion to complete is "Is the face in this picture ______." When we should probably have used the sentence "Does the face in this picture show ____" for the emotions.

Us
API
What to do

Blurry
Blurred
Not sure

Dark
Underexposed
Not sure

Hat
Headwear
Change to headwear (eg, what about hijabs?)

Happy
Joy
Change to joy

Sad
Sorrow
Change to sorrow

Mad
Anger
Change to anger

Surprised
Surprise
Change to surprise

/cc @omaray (since this is beta blocking!)

PR sent: https://github.com/GoogleCloudPlatform/google-cloud-node/pull/1858


[Docs] Reorganize and improve documenation of include_dirs and include_path

This PR:

Fixes wrong documentation of include_path which does not control search path of C/C++ headers - fixes #1480
Improves documentation of Extention parameters used in example to have it self contained and reader does not need to visit setuptools documentation for details.
Adds Compilation using setuptools chapter to Cython for NumPy users and include numpy related documentation from Source files and compilation.

#1480 is long term issue which seems to be old pain point for users. I would close it as soon as possible to close this chapter. I hope that there is no opposition to the documentation changes/improvements. I think it makes sense to move numpy related compilation documentation to numpy-related chapter instead of general source compilation one.

OK just last change mentioning that include_path affects also search of .pxd files. When no comments pop out, I will merge the PR.

Apart from one suggestion for a link, it looks like a better place for this stuff to be documented to me.

Thank you @da-woods for your review!


Library relocation

Description
Like in the issue https://github.com/kotools/types/issues/63, we would like to relocate this library to the org.kotools group.
Dependencies
This issue is blocked by the following ones:

[x] #39
[x] #41

Checklist

[ ] Update the group in the Gradle build script.
[ ] Update the README file.
[ ] Update the Unreleased section in changelog.

The Kotools Assert project will be archived.
We recommend to use the testing library provided by Kotlin instead.


Uses of postMessage cause failures when testing with jsdom

In this library, postMessage calls pass a single argument rather than two (see https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage)
When this code is executed in a test context, jsdom will throw an exception as it expects both the message and the targetOrigin (see https://github.com/jsdom/jsdom/blob/master/lib/jsdom/living/post-message.js#L11_)

Fixed in #27


Indescriptive error on port overlap assertion

There is an assertion set to prevent two servers from using the same port. However, a message is not set, producing a confusing error when the assertion is triggered:
error: attribute 'message' missing

       at /nix/store/369wr4laqdqzh5brq1g7nfc1pbfh23sn-source/nixos/modules/system/activation/top-level.nix:122:30:

          121|
          122|   failedAssertions = map (x: x.message) (filter (x: !x.assertion) config.assertions);
             |                              ^
          123|
(use '--show-trace' to show detailed location information)

Huh, I thought I would have set a message there, but apparently not!


Feature: contentFor block helper

Love the project. By far the easiest to get going so far for this node.js/express newbie.
The express-hbs project has a really neat feature I'd love to see added. I'm not well enough versed yet to create this and file a pull request. The contentFor helper is basically equiv to content placeholders in asp.net and very useful. Basically allows to adding content to a parent template/layout. Here's the usage from express-hbs
layout.handlebars
{{{block "pageScripts"}}}

index.handlebars
{{#contentFor "pageScripts"}}
  CONTENT HERE
{{/contentFor}}

Is this something we can get added, or would it be possible to get a helper function we can add as middleware?

It doesn't work for me. It just returns a string inside the HTML and nothing happens
contact.hbs
{{#contentFor "pageScripts"}}
  <script src="js/pace.min.js"></script>
  <script src="js/wow.min.js"></script>
{{/contentFor}}

main.hbs
{{block "pageScripts"}}

Great!

This is working really well. Cheers!!

To properly run js, instead of displaying it,
I needed to change the notation from
{{block "pageScripts"}}  to {{{block "pageScripts"}}}


DFP-34X-2C2 - SN no state 05

Hello, I am not able to place the RN according to the image in the HGU.

I'm using FW V1.7.1-220916

By telnet is also not possible.
Thanks.

first, SN must be 4 ASCII + 8 HEX
so,
your 16 hex long SN must convert the first 8 HEX into ASCII
please re-read the flash get set readme


[Question]  Why does my netty is unresponsible after two blocked request?

Expected behavior
I expect I/O Threads to keep consuming requests.
Actual behavior
After 2 blocked requests. It starts to not accepting requests. Even If I am using 16 Threads as a child worker Thread
Steps to reproduce
1-) Send open the websocket request
2-) On the Server side block the in the coming event before handshake for 20 Secs (In My real case, It is a blocked JDBC call before the handshake. I need to validate the incoming request with querying DB before opening the socket)
3-) Do It above Steps two times
4-) For requests after 3, the server starts to not answering until the first blocking request done.
Minimal yet complete reproducer code (or URL to code)
@Override
    protected void channelRead0(ChannelHandlerContext channelHandlerContext, HttpRequest httpRequest) {
    Thread.sleep(20000); // Represents my blocked logic before handshake
    // Handsake or not with result
    Do stuff...
    }

EventLoopGroup bossGroup = new NioEventLoopGroup(1);
EventLoopGroup workerGroup = new NioEventLoopGroup();

ChannelPipeline pipeline = socketChannel.pipeline();
        pipeline.addLast(new HttpObjectAggregator(65536));
        pipeline.addLast(HTTP_CODEC.value(), new HttpServerCodec());
        .
        .
        .

Netty version
4.1.59.Final
JVM version (e.g. java -version)
1.8 don't be mad at me :P
I am passing blocked business logic to the custom thread pool. But, In the handshake case, I can not do that. and After two blocked requests my netty server is unresponsible until the first one ends. Is there any good way to handle the blocked handshake logic without block child threads.

You should never block I/O threads else you'll end up with weird problems. Better use ExecutorService for costly operations.

Yes. I am doing that. I am running my blocking logics from a separate thread except for the Handshake phase.
For handshake pahse: Is there any way to handle the handshake phase from another thread with asynchronously?

Well, if you wanna control WebSocket handshake then you should extend the implementation class and do it.

If you can explain in little more detail about use case and all, I can help.

Sure, Below I shared the sample code. When I do the handshake like this (async without blocking I/O Thread). It does not work. Is there an option to not block the I/O thread and doing blocked business logic before the handshake for example JDBC call?.
'''
@Override
protected void channelRead0(ChannelHandlerContext channelHandlerContext, HttpRequest httpRequest) {
Thread handshakeWorker = new Thread(() -> {
// For example my blocked logic works here before handshake
try {
Thread.sleep(1000);
} catch (InterruptedException e) {
e.printStackTrace();
}
// And after It is okay I do handshake or not with result of the blocked process
WebSocketDecoderConfig webSocketDecoderConfig = getDecoderConfig();
WebSocketServerHandshaker handShaker =
new WebSocketServerHandshaker13(getRemoteURL(httpRequest), null, webSocketDecoderConfig);
handShaker.handshake(channelHandlerContext.channel(), httpRequest);
});
handshakeWorker.start();
}

'''

so what I want to do

I do not want to block IO Thread
I want to do some blocking logics(JDBC call) before handshake
with the result of the blocking call I wan to do handshake or not

Well, the business logic looks good. As far I can understand, when you block and wait for database operation to complete, the client is getting times out.
Like this:
12:00:00 - Client fired WebSocket handshake request
12:00:01 - Your app makes call to database.
12:00:05 - Client didn't get any response, times out.
12:00:07 - Your database call is finished
12:00:08 - You try to complete handshake but client is already gone.

Yes. but even if I switch the JDBC call with Thread.sleep(500). The above code does not work.

But when I do the process without creating a new thread. It works. Even If, I sleep It for 10 secs or more than 10. but The I/O Threads are pausing and stop consuming

Aaaaaaaaaaaaa, I gotcha. You're using 'SimpleChannelInboundHandler'. It'll release Http Request as soon as it finishes processing channelRead0. (Even before your logic finishes in 500ms).
Use ChannelInboundHandler.

Wow. but how do we do the handshake without FullHttpRequest? Is there any documentation/resource about that?

Thanks for the help @hyperxpro It was about to retain/release the message as you said. Very thanks for the answer to my question


"Ulma" is listed in the drop-down menu

Description:
"Ulma" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Ulma" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Ulma, SUCEAVA".


Preview Triggers a patch call on engagements

https://met-web-test.apps.gold.devops.gov.bc.ca/eao/engagements/173/view
clicking preview of this engagement brings up Error Updating engagement ..

Verified.


Sfx gapless looping

A small (and slightly hackish) fix to the small gaps that happened when looping some songs. Note that this does not solve the start/end gaps inherent to MP3 compression. As a side effect it adds the option to play a piece of music N times via play().
Edit: Nevermind, while this fixes gaps when looping, it adds problems with pausing the music.

Bees are neutural, Wasps are.
Date: Sun, 21 Feb 2016 20:13:30 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: Re: [FlashPunk] Sfx gapless looping (#119)
Closed #119.
—
Reply to this email directly or view it on GitHub.


Can be change DraggableScrollableSheet initialChildSize

I want to adjust the height that comes at the beginning. The user can increase it if he wants. For example, if the maximum is 0.9, I want the length to come at the beginning as 0.4.

Or we should be able to set all the values eğer biz bir değer vermezsek varsayılan olarak standart olan kullanılsın
      initialChildSize: maxChildSize,
      maxChildSize: maxChildSize,
      minChildSize: _sheetProvider.minExtent,

I'll dig into this as soon as I can this week. Thanks for the ticket!

@cnkygmr This turned out to be a bit bigger than expected. I have a path forward and I will have a PR up within the next couple days.

It has been a bit, but there is a PR open to fix this


windows_certificate create action does not appear to be idempotent

Converging on a Windows 2012 RTM virtual box the windows_certificate resource always runs irrespective of whether the certificate is installed in the certificate store.
Example usage of the windows_certificate resource
  windows_certificate "C:/certs/my_cert_file" do
    pfx_password 'my_password'
  end

Having looked at the code I think the powershell resource used in the create method needs changing to specify the guard interpreter
  powershell_script new_resource.name do
    guard_interpreter :powershell_script
    code code_script
    not_if guard_script
  end

Alternatively if a guard_interpreter is not specified the resource can be made idempotent by returning an exit code of 0 or 1 from the script by changing it to
def cert_exists_script(hash)
  <<-EOH
$hash = #{hash}
If ((Test-Path<EMAIL_ADDRESS>-eq $true) {exit 0} else {exit 1}
EOH
end

Both solutions above tested with Chef 11.18.6 and 12.4.1.

The default guard_interpreter for powershell_script is powershell. I sanity checked this running the certificate recipe in the minimal test cookbook included with this repo and the certificate create was idempotrent.
Could you provide debug output of a non idempotent run?

Hi Matt, thank you very much for taking the time and trouble to look into this issue, it is very much appreciated, even just for the sanity check.
My understanding is that the default guard interpreter for a powershell_script resource was powershell which is why I'm perplexed.
It was our managed Chef server that alerted us to the fact that the resource does not appear to be idempotent as it appears in the run report after every run

At first I thought that it might be an issue with the updated_by_last_action not being set correctly but it definitely appears to be an issue with the guard.
Here is the debug output from running the resource.
chef_node_converge.txt
Running the guard code directly in PowerShell returns True

What version of chef client is your node running? 11.18.6? I'll see if I can reproduce there today.

The converge this morning was with Chef client 12.4.1

Oh OK. Good to know.

Matt, sorry I was incorrect, the windows_certificate does not appear to be idempotent in Chef client 11.18.6 but does appear to work perfectly well in Chef client 12.4.1, I assume because of the wrapper code that the Chef client adds to PowerShell script (as seen in the debug output).
Please accept my apologies, I'll understand if you want to close the issue due to us using an outdated Chef client.

Hey no problem at all. We'd certainly like this to be backward compatible to v11 so thanks a bunch for the update and I will test there.

Tested the v1.38.4 release of the windows cookbook. windows_certificate is idempotent in Chef 11. Thank you.

Ah great to hear!


use envoy.reloadable_features.remove_legacy_json deprecation

Your change #14467 (Replace rapidjson with nlohmann/json) introduced a runtime guarded feature. It has been 6 months since the new code has been exercised by default, so it's time to remove the old code path. This issue tracks source code cleanup so we don't forget.

@dio do you know if anyone has been running this with the flag flipped to true?

the flag is false by default

@alyssawilk oops sorry too fast to comment. Will ask around.

@wbpcode could you help to test this, since it seems you have a sensible environment to test this given: https://github.com/envoyproxy/envoy/issues/4705#issuecomment-599934902. Thanks!

Talked to @ramaraochavali from Salesforce offline, it seems he can help to test this in their environment. Thanks, @ramaraochavali!

@dio we deployed with that run time flag and basic flows (Service -> Service) communication are working

@dio Sorry for the delayed reply. I can do a quick test in my local env with config of the production environment.

It works well for me. Although in my scenario, I just use it to convert a string into a Json object and process the sub-objects in it.


Create rave

Ravensbourne University London
Ravensbourne University London

@dave-oneill lib/domains/rave: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@dave-oneill The filename provided does not conform to this repository rules located at the bottom of this page. It's impossible to guess what is the domain you wish to add.
I have to close the request without merging. Please check the rules and create a new request if necessary.
Thank you for your understanding!


Update script.js

fixed the js bug caused by this line in debugging while making it responsive

@SauravMukherjee44 plz review


[Fix rubocop#10813] Fix recursive deletion to suppression in Lint/NonAtomicFileOperation

Fixes: rubocop#10813
This PR is suppress offense in case of
recursive deletion in Lint/NonAtomicFileOperation.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thank you so much. I updated this PR.


Incorrect application of future grant * in snowflake

Terraform CLI and Provider Versions
Terraform v1.6.1
Snowflake-Labs 0.82
Terraform Configuration
terraform {

  required_version = ">= 1.6.1"

  required_providers {
    snowflake = {
      source  = "Snowflake-Labs/snowflake"
      version = "0.82.0"
    }
  }
}

provider "snowflake" {
  account       = local.snowflake_account
  authenticator = "JWT"
}

provider "snowflake" {
  alias         = "account_admin"
  role          = "ACCOUNTADMIN"
  account       = local.snowflake_account
  authenticator = "JWT"
}

provider "snowflake" {
  alias         = "security_admin"
  role          = "SECURITYADMIN"
  account       = local.snowflake_account
  authenticator = "JWT"
}

provider "snowflake" {
  alias         = "sys_admin"
  role          = "SYSADMIN"
  account       = local.snowflake_account
  authenticator = "JWT"
}

resource "snowflake_grant_privileges_to_role" "g14" {
  privileges = ["SELECT"]
  role_name  = "TEST"
  on_schema_object {
    future {
      object_type_plural = "TABLES"
      in_schema          = "*"
    }
  }
}

Expected Behavior
This should grant select on future tables in all schemas in the specified database.
Actual Behavior
It incorrectly grants select on future tables in all schemas. It grants select on future tables in schema DB.TABLE when it should be DB.SCHEMA.TABLE
Whenever a new table is added to a schema we need to regrant permissions. This occurs for all database objects including: views, materialized views, external tables, file formats etc.
Steps to Reproduce

terraform apply

How much impact is this issue causing?
 Thanks for the reply! I have not tried that but I believe that would grant these privileges at the database level correct? We can't grant at the database level because we commonly grant at the schema level which nullifies the db grants. From the snowflake docs: "When future grants are defined on the same object type for a database and a schema in the same database, the schema-level grants take precedence over the database level grants, and the database level grants are ignored. This behavior applies to privileges on future objects granted to one role or different roles."

@coleheflin-gt
So I pointed you to where the problem is because the star notation shouldn't be allowed there and the error should appear (error from Snowflake indicating syntax error). With your version of the provider, you should get an error like this.

If that didn't happen that's strange...
Going back to the problem according to the grant privileges docs, we only care about those two branches
| { schemaObjectPrivileges   | ALL [ PRIVILEGES ] } ON { <object_type> <object_name> | ALL <object_type_plural> IN { DATABASE <db_name> | SCHEMA <schema_name> } }
| { schemaObjectPrivileges   | ALL [ PRIVILEGES ] } ON FUTURE <object_type_plural> IN { DATABASE <db_name> | SCHEMA <schema_name> }

because tables are schemaObjects. As you can see the functionalities are limited when it comes to your use case. What you can do with the Terraform is, e.g. create a data source that would fetch all of the schemas you are interested in and then use HCL for_each meta-argument. Then you can perform grant <privileges> on future tables in schema <schema_name>. That would look something like this
resource "snowflake_grant_privileges_to_role" "g14" {
  privileges = ["SELECT"]
  role_name  = "TEST"
  for_each = data.snowflake_schemas.selected_schemas
   on_schema_object {
    future {
      object_type_plural = "TABLES"
      in_schema              = each.name
    }
  }
}

If you care about current tables you can also add the same config that would grant privileges to all tables in schemas, e.g.
resource "snowflake_grant_privileges_to_role" "g14" {
  privileges = ["SELECT"]
  role_name  = "TEST"
  for_each = data.snowflake_schemas.selected_schemas
   on_schema_object {
    all {
      object_type_plural = "TABLES"
      in_schema              = each.name
    }
  }
}

Ah I see, thanks for letting us know. We originally were using the data object and iterating over the schemas to grant all/future privileges as you described. We moved to the * grant because the aforementioned strategy significantly increases our terraform plans/runtimes but it sounds like that is the only option so we will move back to that.

@coleheflin-gt
Hey again. Sorry, I was confused because granting with * is something I didn't encounter before. Actually, I think this may be possible to grant x on_future with the current provider. The following setup works for me:
resource "snowflake_grant_privileges_to_role" "g14" {
  privileges = ["INSERT"]
  role_name  = "TEST_ROLE_123"
  on_schema_object {
    future {
      object_type_plural = "TABLES"
      in_schema          = "\"TEST_DB\".\"*\""
    }
  }
}

I created a table after applying this resource and indeed it had insert granted. Again, sorry for the confusion. I hope this helps.

Thanks for the response @sfc-gh-jcieslak.
The set-up you've provided above mirrors what we were doing and that was not working properly for us. In your testing environment, did you have any schema level grants? I'm guessing the reason this worked for you is it is being treated as a database level grant and without any schema level grants it works. If you did not, can you try adding a schema level grant and then attempting this permission to check if it applies correctly? Here's a screenshot from the snowflake docs regarding the order of precedence of grants that I'm thinking is affecting this.

Yeah that makes sense. Would it be best to create a ticket internally with snowflake to address this?

Yeah, an internal Snowflake ticket would be the best option. If @sfc-gh-asawicki and I could be cc'd that would be great. We would like to know the result of it.

sounds good, I've created a ticket but I do not have your emails so I passed along your names and requested they cc you.

Great, Thank You


Update CakeML AM with latest CVM and appraisal functionality from Coq.

Have cakeml AM use latest Copland Virtual Machine and generalized appraisal functions from here.
Best option is to extract them via Coq --> CakeML synthesis (see @barclata for latest status on this).  Alternatively, or as a first pass, do manually.
This issue depends on #32 being implemented first.

This has been completed many different times


Added dependabot.yml file

Closes issue #389
Addition of dependabot.yml file to keep dependencies up to date.
@codervivek5 Please review it.

kindly share your change so that I can review and merge your PR

@codervivek5 I have added this file to keep dependencies up to date.

Please review it.

Thank you for your contribution :)


Flame name conflict

This project looks really cool, but it's a bit unfortunate that the name conflicts with another really big and very active open source project with a logo that looks identical: https://github.com/flame-engine/flame
Are there any options for clarifying the name, like "Phoenix Flame" or something?

Name overlaps are very common. Given those projects are in different languages and have different targets (web/server vs game), I believe there isn't much to worry about. Thanks!

I checked your GitHub and it seems you are the intersection of both communities in the venn diagram. :D I understand the request but I think generally speaking there should not be much confusion.


Go through ARexx scripts in distribution

The ARexx scripts in DistrSkeleton/rexx were taken from the distribution of NiKom 1.60 as it was assumed that this would be the latest "clean" versions of the scripts suited for a new BBS setup.
It seems though as if NiKom 1.60 was shipped with whatever Kärkis BBS was running which can be either customized scripts or older versions since Kärkis BBS started with a pretty early version of NiKom.
Go through the scripts and compare with both earlier distributions and/or NiKom BBS.

@joacimmelin and @jayminer81 you may want to update the files in NiKom:Rexx since you both recently have set up your systems...

Will do!  Thanks for heads-up!


Roll engine 72986c39ea36..711d8431da47 (17 commits)

https://github.com/flutter/engine/compare/72986c39ea36...711d8431da47
git log 72986c39ea3669ef26d4d7d7cd0c44f513fcd117..711d8431da47256262bca95757d68ad2d2dfcd42 --no-merges --oneline
711d8431d Roll src/third_party/skia 76e626d9bb55..c33e6dcc700b (4 commits) (flutter/engine#8547)
82e6d6899 Eliminate unused write to local (flutter/engine#8541)
f53e477a6 Correct nullability for FlutterStandardReader (flutter/engine#8537)
c00364a6c [font_collection] Add missing semicolon (flutter/engine#8546)
ae4df6fd1 Revert "Change Rect internal representation from Float32List to Float64List (#8524)" (flutter/engine#8545)
ffdddb07e Roll src/third_party/skia 6d60534e95d8..76e626d9bb55 (1 commits) (flutter/engine#8544)
5ac728eb6 Roll src/third_party/skia 4d657d5e894a..6d60534e95d8 (3 commits) (flutter/engine#8543)
ee462ff68 Roll src/third_party/skia 99d792276740..4d657d5e894a (1 commits) (flutter/engine#8542)
18816c50f Roll src/third_party/skia 42280f8961fa..99d792276740 (8 commits) (flutter/engine#8539)
892591da8 Android Embedding PR28: Report app is active to Flutter in FlutterFragment.onResume() instead of onPostResume() forwarded from Activity. (flutter/engine#8536)
330c6c168 Use code cache dir for engine cache on API >= 21 (#14704). (flutter/engine#8534)
c10ae2be2 Add an option to build the GLFW shell on macOS (flutter/engine#8531)
29aa5fca0 Roll src/third_party/skia b5c685991faa..42280f8961fa (32 commits) (flutter/engine#8535)
0a2869e5c Added support for authentication codes for the VM service (flutter/engine#8527)
36d2135ad Redo a fix for cull rect calculation on TransformLayers with a perspective transform (flutter/engine#8528)
0b36d3e2f Change Rect internal representation from Float32List to Float64List (flutter/engine#8524)
b1ae0ccb7 Roll src/third_party/skia 10bf7020aa15..b5c685991faa (3 commits) (flutter/engine#8526)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5813563670134784

Commit queue failed; closing this roll.


"Visual Studio Code has crashed" when editing an ES6 file

VSCode crashes every time I open an ES6 file, after a few seconds. This started happening after the 0.10.3 update.
Here's the output:
$ ./code --verbose

### VSCode main.js ###
c:\Program Files (x86)\Microsoft VS Code\resources\app { pathArguments: [],
  programStart: 0,
  workers: -1,
  enablePerformance: false,
  verboseLogging: true,
  debugPluginHostPort: undefined,
  debugBrkPluginHost: undefined,
  logPluginHostCommunication: false,
  firstrun: false,
  openNewWindow: false,
  openInSameWindow: false,
  gotoLineMode: false,
  pluginHomePath: undefined,
  pluginDevelopmentPath: undefined,
  pluginTestsPath: undefined }
Lifecycle#unload() 1
IPC#vscode-workbenchLoaded

<--- Last few GCs --->

   11777 ms: Scavenge 715.1 (742.6) -> 715.1 (742.6) MB, 0.2 / 0 ms (+ 5.0 ms in 1 steps since last GC) [allocation failure] [incremental marking delaying mark-sweep].
   11865 ms: Mark-sweep 715.1 (742.6) -> 714.3 (738.6) MB, 88.2 / 1 ms (+ 6.0 ms in 2 steps since start of marking, biggest step 5.0 ms) [last resort gc].
   11919 ms: Mark-sweep 714.3 (738.6) -> 714.4 (737.6) MB, 54.1 / 1 ms [last resort gc].

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 12320E39 <String[7]: file://>
    1: slowToString [buffer.js:344] [pc=0F1B78F8] (this=0F9F3DA5 <an Uint8Array with map 34609681>,encoding=123236C9 <String[4]: utf8>,start=0,end=3694146)
    2: toString [buffer.js:~371] [pc=24AAD709] (this=0F9F3DA5 <an Uint8Array with map 34609681>)
    3: arguments adaptor frame: 3->0
    4: write [string_decoder.js:129] [pc=24A3D6FD] (this=0F9FD0C5 <an InternalDecoder with map 2C6981DD>...

[VS Code]: render process crashed!
Lifecycle#window-all-closed
Lifecycle#before-quit
App#dispose: deleting running instance handle

@jacek-dargiel Can give me estimate of your workspace size?

This particular project has ~30 JS files. There's also quite a few JS files in web/node_modules, but I'm excluding them with a jsconfig.json file:
{
	"compilerOptions": {
		"target": "ES6",
		"module": "amd"
	},
	"exclude": [
		"node_modules"
	]
}

yeah, we also won't go into those folders for perf reasons... can reproduce take a look at the process tree. one of the child process should go crazy (CPU, mem) before the crash happens

How can I do this on Windows? What info do you need?

the best tool is this: https://technet.microsoft.com/en-us/sysinternals/processexplorer.aspx

Yes, I can confirm that one of the code processes did consume 800+ MB of memory just before crashing.

Thanks. What is the name of the process? even better can you attach a screen shot of process explorer before it happens?

Here you go. This is moments before the crash.

And here are the details of the of the process:
The command line is: "C:\Program Files (x86)\Microsoft VS Code\code.exe" --type=renderer --no-sandbox --lang=en-US --app-user-model-id=Microsoft.VisualStudioCode --node-integration=true --enable-pinch --device-scale-factor=1 --enable-delegated-renderer --num-raster-threads=4 --gpu-rasterization-msaa-sample-count=8 --content-image-texture-target=3553 --video-image-texture-target=3553 --channel="4212.1.1281448818\2102388047" /prefetch:673131151

It seems like a worker in going crazy and is able to bring down the system. Soon, we will move the JavaScript language service out of process which will make this issue go away. Also, there was a problem with exclude-rules not being honoured. That got fixed with #703.

Looks like a good chance for endgame leak testing if it still happens.

@bpasero I don't see how that will be useful? Everything happens now in a separate process.

@jrieken yeah looks like this should move to salsa if it still happens

dupe of #4001


WIP: GLM class now inherits from BaseEstimator and RegressorMixin

It should Close #72. BaseEstimator produces automatic __repr__ method and prints human readeable rep of object

Coverage increased (+6.4%) to 92.946% when pulling aeab0d661d9e50a0717972cef4f197a898430bb3 on daniel-acuna:master into 5c99bbe9d220a981c7ab43c8f19d5d9612b674e1 on pavanramkumar:master.

@daniel-acuna what is sample_weight=None in score function?
ideally, score() would accept y, yhat, ynull and method as parameters, where method='deviance' or method='pseudo_R2'
and accordingly returns a list of scores. that way we don't need to pass X which can be large, and it wouldn't need to call predict()

@pavanramkumar  Unfortunately (or fortunately!), to conform to sklearn RegressorMixin, we need to override the score function which follows the conventions listed here. You can still call pseudo_R2 of course. If this is really inappropriate, we could raise a warning when calling score

@daniel-acuna If you define a new score() function as you have done, doesn't it override the inherited function?
To conform as much as possible, we can pass X and y and call predict(). We would need another parameter called method. We can also raise appropriate warnings (or document really clearly) that sample_weights is never used.

I know that this won't pass flake8 test because code base from sklearn doesn't that I copied that doesn't conform to it! any chance we can add an exception to the flake8 command line

What about running flake8 only for the main script?
flake8 --count pyglmnet/pyglmnet.py

@daniel-acuna it's possible to add exceptions to certain files files / folders. Something like this: https://github.com/mne-tools/mne-python/blob/master/setup.cfg#L31.
The diff looks pretty big. Is this smaller by any chance: https://github.com/mne-tools/mne-python/blob/master/mne/decoding/base.py#L15-175 ? It also conforms to pep8 I think ...

before we merge, we should also smooth out the license issues.

we can have an externals/ folder where we put code copied from other libraries and also add the corresponding license files in there.

The lines you suggest only add the BaseEstimator functionality. CV needs a
bunch of functionality to replicate models during every iteration. I will
try to put my code on a diet and see how it goes
On Thu, May 12, 2016 at 6:41 PM Mainak Jas<EMAIL_ADDRESS>wrote:

@daniel-acuna https://github.com/daniel-acuna it's possible to add
exceptions to certain files files / folders. Something like this:
https://github.com/mne-tools/mne-python/blob/master/setup.cfg#L31.
The diff looks pretty big. Is this smaller by any chance:
https://github.com/mne-tools/mne-python/blob/master/mne/decoding/base.py#L15-175
? It also conforms to pep8 I think ...
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/pavanramkumar/pyglmnet/pull/78#issuecomment-218916239

Should I put function outside then? I was looking for modularity but it
looks ugly i guess
On Thu, May 12, 2016 at 7:01 PM Daniel A<EMAIL_ADDRESS>wrote:

The lines you suggest only add the BaseEstimator functionality. CV needs a
bunch of functionality to replicate models during every iteration. I will
try to put my code on a diet and see how it goes
On Thu, May 12, 2016 at 6:41 PM Mainak Jas<EMAIL_ADDRESS>wrote:

@daniel-acuna https://github.com/daniel-acuna it's possible to add
exceptions to certain files files / folders. Something like this:
https://github.com/mne-tools/mne-python/blob/master/setup.cfg#L31.
The diff looks pretty big. Is this smaller by any chance:
https://github.com/mne-tools/mne-python/blob/master/mne/decoding/base.py#L15-175
? It also conforms to pep8 I think ...
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/pavanramkumar/pyglmnet/pull/78#issuecomment-218916239

if you look at this, you might be able to figure out what's needed. Probably just a transformermixin and estimatormixin? I am sure they use it in an example with cross_val_score

I will try to put my code on a diet and see how it goes

Yes!!!!

Coverage increased (+2.4%) to 88.976% when pulling f8b764f236ffeed11bcd91787dc7aab205e63810 on daniel-acuna:master into 5c99bbe9d220a981c7ab43c8f19d5d9612b674e1 on pavanramkumar:master.

OK, no need for BaseEstimator, pyglmnet is really slim now!

@daniel-acuna Do I understand correctly that the user is left to use cross_val_score from sklearn to do their own cross-validation? Or do we provide a wrapper for users that calls cross_val_score?
This decision is important and will affect API design for cvpyglmnet which should have tools to estimate optimal lambda

@pavanramkumar yes, the user have to call cross_val_score themselves. For cvpyglmnet, I think sklearn provides a nice template to follow in ElasticNetCV

ok, great. is this PR still WIP or can it be merged?

Coverage increased (+2.4%) to 88.976% when pulling 287b605e2e9105a6af3aa63002da17ae19f7d5bb on daniel-acuna:master into 5c99bbe9d220a981c7ab43c8f19d5d9612b674e1 on pavanramkumar:master.

@pavanramkumar I think it is ready but I am not sure if you have a proccess in place to accept PR. At the very least, @jasmainak should give the OK. I could merge it myself too after that!

@daniel-acuna i think @jasmainak has a talk today, we can wait for him to have a look, no rush.
do you want to rename this PR before merging?
right now it does a few different things -- maybe choose an appropriate name to avoid confusion later:

verbose fixes
added cross_val_score example to test_pyglmnet.py
added get_params method

Coverage increased (+3.8%) to 90.323% when pulling 431216f43b3307141e88c3d526b0555e0e456869 on daniel-acuna:master into 5c99bbe9d220a981c7ab43c8f19d5d9612b674e1 on pavanramkumar:master.

@pavanramkumar I removed the verbose fixes as they were not needed anymore. I think its ready to merge.

coverage is above 90% that's awesome!


write_to_annotation error when trying to write changes to database

I get the below error:
Writing changes to database:
Traceback (most recent call last):
File "/home/navidrome/Downloads/itunestoND.py", line 157, in
write_to_annotation(artists, 'artist')
File "/home/navidrome/Downloads/itunestoND.py", line 61, in write_to_annotation
cur.executemany('INSERT INTO annotation VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)', annotation_entries)
sqlite3.OperationalError: table annotation has 8 columns but 9 values were supplied

It looks like the Navidrome annotation table has been changed and no longer requires the ann_id field.
In itunestoND.py the ann_id field is used in these two lines:
annotation_entries.append((generate_annotation_id(), userID, item_id, entry_type, play_count, play_date, rating, 0, None))
cur.executemany('INSERT INTO annotation VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)', annotation_entries)
As a quick test I modified these lines to remove the ann_id field but the script crashes at a different point and I don't have enough understanding of the script to fix it.
This is the Navidrome change that removed the ID field in v0.53.0:
https://github.com/navidrome/navidrome/commit/47378c68828861751b9d1a05d3fc9b29ce8dd9f0
This is the new structure of the annotations table from v0.53.0:
    user_id    varchar(255) default ''    not null,
    item_id    varchar(255) default ''    not null,
    item_type  varchar(255) default ''    not null,
    play_count integer      default 0,
    play_date  datetime,
    rating     integer      default 0,
    starred    bool         default FALSE not null,
    starred_at datetime,
    unique (user_id, item_id, item_type)

I installed version 0.52.5 to see if I still get the error. I don't, so @c0f, I think you were right.
I haven't tried to fix the file and see if it works on 0.53.x version.


break mesh tally loop on coincidence

This does not fix a bug. But, this improves some accidental inefficiency. Previously, we always trace the meshes all the way to the problem boundary, even beyond where the particle stops. This is due to the mesh tally scoring function that does not anticipate the new geometry coincidence treatment.

Closing this because it got mixed up with other things. Separate PRs will be made.


[6.0] Fix diagnostics in Objective-C documentation comments appearing on the wrong line

Explanation: This fixes an issue where diagnostics for in-source Objective-C comments displays on the wrong line. DocC previously added a workaround to offset the source ranges in Clang SGFs but now that Clang has updated its SGF source ranges in to align with Swift, that DocC workaround causes diagnostics to appear on the wrong line. This removes that workaround in DocC.
Scope: Diagnostics for Objective-C source comments appear on the wrong line.
Issue: rdar://129668876
Risk: Low
Testing: Existing, updated tests verify that diagnostics have the expected source ranges. Manually tested that both Swift and Objective-C diagnostics appear on the expected lines.
Reviewer: @anferbui
Original PR: #949

@swift-ci please test


[bug] CoinbaseWallet used with the QR login did not work

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
2.1.3
wagmi Version
2.11.2
Current Behavior
dApp works normally but on Coinbase wallet users are having issues
Everytime I try to do an action I got the same error

[!CAUTION]
Uncaught (in promise) TransactionExecutionError
at getTransactionError (chunk-MFUBTO2W.js?v=e3af3d8e:1317:10)
at sendTransaction (chunk-MFUBTO2W.js?v=e3af3d8e:1401:11)
at async writeContract2 (chunk-MAVCKE2U.js?v=e3af3d8e:376:16)Caused by: UnknownRpcError
at withRetry.delay.count.count (chunk-MFUBTO2W.js?v=e3af3d8e:1823:19)
at async attemptRetry (chunk-MFUBTO2W.js?v=e3af3d8e:1756:22)Caused by: ReferenceError: Buffer is not defined
at ensureBuffer (dist-O27QDEZM.js?v=e3af3d8e:578:7)
at WalletLinkSigner._prepareTransactionParams (dist-O27QDEZM.js?v=e3af3d8e:3610:56)
at WalletLinkSigner._eth_sendTransaction (dist-O27QDEZM.js?v=e3af3d8e:3762:25)
at WalletLinkSigner._handleAsynchronousMethods (dist-O27QDEZM.js?v=e3af3d8e:3568:25)
at dist-O27QDEZM.js?v=e3af3d8e:3531:16
at new Promise ()
at WalletLinkSigner._sendRequestAsync (dist-O27QDEZM.js?v=e3af3d8e:3518:16)
at WalletLinkSigner._request (dist-O27QDEZM.js?v=e3af3d8e:3493:35)
at WalletLinkSigner.request (dist-O27QDEZM.js?v=e3af3d8e:3464:23)
at Object.sign (dist-O27QDEZM.js?v=e3af3d8e:4252:40)

Expected Behavior
No response
Steps To Reproduce
No response
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

@mentasuave01 Can you provide a minimal reproducible example ?

i'm also facing same issue

Hi seems like it was cause Buffer wasn't available in your JS env. Were you using Node.js when this error happened? If not, can you try switch to Node and see if this issue persists? Or you can also install Buffer to your package.json separately

buffer is already present in my package.json separately but still facing same issue

@Rohitbaghel11 Can you provide a minimal reproducible example ?

@Rohitbaghel11 I can't seem to reproduce this. Are you able to share a small reproducible example ?


[iOS] Add folly_compiler_flags to Core subspec

Summary
This fixes the following iOS build error when using React via pods;

project/Pods/Folly/folly/portability/Config.h:20:10: 'folly/folly-config.h' file not found

https://github.com/react-native-community/react-native-releases/issues/79#issuecomment-463628317
https://github.com/react-native-community/react-native-releases/issues/79#issuecomment-463675118
https://github.com/react-native-community/react-native-releases/issues/79#issuecomment-464117823
Changelog

[iOS] [Fixed] - Fixed a "'folly/folly-config.h' file not found" build error when using React via CocoaPods
Test Plan
Before:

After:

(test_ios on master was failing prior to this PR, https://circleci.com/gh/facebook/react-native/73018)


Material: Fix disposing resources when disposing of a material

Follow up to #15964

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15967/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15967/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15967/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15967/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15967/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15967/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15967/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15967/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15967/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15967/merge/testResults/webgl2playwright/index.html


Fields on nested structs

With this struct:
type Config struct {
	Kafka struct {
		CA   string `default:"/etc/kafka-ca/tls/ca.crt" file_json:"Kafka.CA" file_yml:"ca"`
		Cert string `default:"/etc/kafka/tls/user.crt" file_yml:"kafka_cert"`
		Key  string `default:"/etc/kafka/tls/key.crt"`
	} `file_yml:"kafka"`
}

and this configuration file:
kafka:
  ca: "/tmp/foo"

kafka_cert: "/tmp/crt.pem"

I'd have expected Kafka.CA to be set (and Kafka.Cert not to be set really), but the reverse is true.

Sorta related to https://github.com/BoRuDar/configuration/issues/31

Hi @abh.
Could you provide the sample where configuration.New is defined please?
The order of providers defines which one will be called first


update dependencies

I had error coming from winit:
thread 'main' panicked at 'attempted to leave type `platform::platform::x11::util::input::PointerState<'_>` uninitialized, which is invalid', library/core/src/panicking.rs:123:5

So I updated to glutin 0.28.0 and updated other dependencies. I have problems compiling glutin 0.29 because of wayland dependencies, and glutin 0.30 has changed to be independent of winit which I don't think has been updated yet to work with the newest glutin.

Hi, given this PR didn’t get merged, I’ve taken it and updated to the latest glutin and winit in #70.


upgrade versions

Let's upgrade them

@rultor release, tag is 0.20.2

@rultor release, tag is 0.20.2

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.20.2

@yegor256 Done! FYI, the full log is here (took me 7min)


in dnn.cpp

There is a problem with the calculation process, and the final accuracy is only 0.08.

Consider code correctness or data type.

I fixed a problem in the initialization section. Now the accuracy raise to 0.1, but still not enough. The goal is greater than 0.9.

solved.
The reason is I misunderstood the algorithm. The output should be the index of output[], not the value.


✨ Show last_committed stats in databases/show

#484  👈
main

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

Join @RTLS and the rest of your teammates on  Graphite

This method of measuring replication lag is imperfect and is not an improvement to the UX in my opinion.
pg_stat_replication gives us flush_lag for active slots. When there is constant activity, flush_lag is a good measure of how long transactions take from committing on the primary to acknowledgment by Sequin.
When there is no activity on the primary for tables in the publication slot, the flush_lag grows linearly with time. Eventually, Postgres returns null for the value which we can use as an indicator that we are caught up. The transtion to null seems to occur after 13 minutes.
We can also measure the delay in bytes, which is supposedly more accurate / less prone to errors due to inactivity, but on my system it also grows linearly with time
-- Monitor lag in bytes
SELECT
    slot_name,
    pg_size_pretty(pg_current_wal_lsn() - restart_lsn) as lag_bytes,
FROM pg_replication_slots
WHERE slot_type = 'logical';

We can opt to only show replication lag if we are currently ingesting messages from upstream. But this does not solve whether we have accurate stats; just whether we show inaccurate stats to our users.
The most robust way to measure replication lag is to intentionally set a row in the primary database with a timestamp then measure when we receive that row in the replication slot. Is it worth doing this work?


Any option for gzip compression?

Hi Cortesi,
This is an awesome project!  Totally prefer to use this then SimpleHTTPServer.  However, I saw on a review site, which led me here, that they had a screenshot where it showed gzip compression.  Is there a switch or config file change that I am missing or does that not exist in DevD?  If not, then are there plans to include this feature in a future release?  I also searched for plugins but couldn't find anything in a quick Google search.  If you could provide an option for gzip, it would be greatly appreciated!
Thanks!

Hi there - glad you've found devd useful. When livereload is enabled, we request upstream resources with the "identity" encoding so we can easily inject our reload script without needing to decode and re-encode. When livereload is off, we just reflect whatever the client requests and whatever the server sends back.
I'm guessing you need a different behaviour, but it's not quite clear to me what. Could you describe your use case a bit more, please?

Hi there - I'll close this ticket for now. Please do feel free to re-open and describe what the behaviour is that you expect.


Philippines Source URL update

Hi,
The Source URL for the Philippines is out of date.
UACS Philippines can now be found at https://uacs.gov.ph/resources/latest-uacs/106/organization
Thanks! :)

Thanks for this, @IsabelBirds !
Does the codelist also now need to be updated?

The code format etc is unchanged, but yes there are a few new organisations to add.


Add link to connected request from query

Can already go from request to query, but there's no link back.

So, I started going down this route, but it made me wonder, is a separate page for a specific query necessary?
What if the list of queries that happened during a request just expanded to show the query specifics instead of taking you to a different page?

You still might want to see a list of queries, and then be able to drill down in to them. The queries index page actually helped me a few times with our current app. We could get a little more fancy and stuff, but I think keeping it super simple with a click through is probably the easiest right now.


MethodSeparationFixer - Fix spacing around comments

RFC, at see: https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/1421 after https://github.com/symfony/symfony/pull/15890
Targeting master because the fixer does not exists in the 1.x branches.

@GrahamCampbell you have rised an issue, please, could you check this PR?

what is the state of this PR?

Wait for feedback. I don't like to tag people, but maybe it would be useful to get the reporters in here as well (Stof should be back next week I think)

@GrahamCampbell I remember that you play with the fixer on Symfony repo, could you discuss with @SpacePossum and then both of you could get this PR done smooth and shine

Rebased, anyone who would like to take a look?

rebased an squashed

:+1:

One more :+1: and good to merge!

Closing as it does not seem to be possible to find a consensuses about this one.
Kindly note @keradus that you might want to move the fixer to contrib (i.e. out of symfony level) as @ symfony they don't agree about the fixer behaviour.

Reopen, will have a deeper look...

rebased
ping @keradus
I would really like to get this one over with after 4 months of rebasing

I found one last bug and good to go: https://github.com/SpacePossum/PHP-CS-Fixer/pull/8

Thanks for checking @keradus and the bug report + test :+1:
reworked.
There are still a few hits on symfony 2.3;
	modified:   src/Symfony/Bridge/Twig/Tests/Node/TransNodeTest.php
	modified:   src/Symfony/Component/Console/Tests/Descriptor/AbstractDescriptorTest.php
	modified:   src/Symfony/Component/CssSelector/Tests/Node/AbstractNodeTest.php
	modified:   src/Symfony/Component/CssSelector/Tests/Parser/Handler/AbstractHandlerTest.php
	modified:   src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services10.php
	modified:   src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services12.php
	modified:   src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services9_compiled.php
	modified:   src/Symfony/Component/Form/Tests/Extension/HttpFoundation/HttpFoundationRequestHandlerTest.php
	modified:   src/Symfony/Component/HttpFoundation/Response.php
	modified:   src/Symfony/Component/HttpKernel/Tests/Fragment/RoutableFragmentRendererTest.php
	modified:   src/Symfony/Component/Security/Tests/Core/Encoder/EncoderFactoryTest.php
	modified:   src/Symfony/Component/Validator/Tests/Mapping/Loader/AnnotationLoaderTest.php

I'll see if I PR those or maybe move the fixer to contrib level.

for me it shows only 9 files, and if they are not autogenerated code they should be merged

ready to merge

Thank you @SpacePossum.

thanks all for reviewing and testing :)

That was long, but we finally did it.
Great work @SpacePossum

That was long, but we finally did it.

yeah, great to see this in, first fixer I wrote is accepted :D

Really? It was your first fixer? no way!


Cloze hints do not work in Remnote as of 1.13.22

Hello,
I've updated to the latest Remnote version and it seems the plug in no longer works to insert cloze hints. Please reference the video in this thread to see my attempts to press "CMD+SHIFT+:" to insert the hint. I've ensured that the keyboard shortcut is configured properly in the settings as well.
I've tried with both desktop and web app versions. Both of them do not work...
https://github.com/bjsi/cloze-hints/assets/52617485/c788ae20-af36-488a-979d-6eabefe624ba

Correction,
it seems like if the text is highlighted a certain color as seen in the video, the cloze hint does not work. Could you please help on fixing that part?


Implement test on the process error handling of the pipeline

Implement test to check a failed file being moved to the fail respective folder and to check a corresponding log being created there as well

I claim it. This is my first time contributing to open source. Can you help me please?

Nice, @renilbabu03 !
First, let's tackle https://github.com/tiagostutz/demo-warehouse-software/issues/9 and after this is done, we can move forward on this one. 👍🏻


AbstractClassWithoutAbstractMethod turn off issues

Content managers of the course ask to turn off AbstractClassWithoutAbstractMethod issue because it is normal for them to use abstract class without abstract methods (like in step https://hyperskill.org/learn/step/5768).

Fixed in 6c8d506.


Error quickly flashes before each run if no tsconfig found

I'm using hopa for js, not ts, so I don't have a tsconfig file. On each save, I see the following error for like half a second before it starts running my file:

Same issue here, also for js files.
The error is briefly displayed before the run.
Then the error is flashed 1 second after and stays below my script output.

Hey, can you paste the content of your file here. Also node version and OS will be helpful.

node v12.16.3
Linux Mint 19.3 Tricia
The error disappears if you create a tsconfig, which isn't needed if you're just using hopa for JS.

Interesting, tsconfig.js in the folder where you run hopa?

If I have a tsconfig in the folder, the error doesn't appear. If I don't have one, I see the error.

Awesome. That's enough. I'll debug.

@AliRain @qchenevier it should be fixed now. Can you please try the latest version npm i hopa@latest.
fix

It works for me. You rock !

@AliRain I hope it works for you too. If not please comment below. Closing for now.


Module #1: "Empty Project"

The wording for creating a new project has changed in RStudio. Instead of "Empty Project" it now says "New Project".

Fixed.


on_key_down listener on a TextInput widget

I would like to trigger an event when the user changes a value in a TextInput widget.
So I do : my_text_input.set_on_key_down_listener(...)
But now, when I try to change the value in the TextInput, my change is only taken into account half the time. See this example to watch the behaviour.

@batzkass Should be fixed now in development branch. Please confirm this. Furthermore I would like to inform you a new event "onkeyup" implemented. "onkeyup" generates when user releases a key. In comparison "onkeydown" event returns always one char behind.

Many thanks, your fix is working in my setup.


Components or Incidents not Reverting back to Fixed or Operational

At the moment the incidents are being created and the components are being set to either "Partial Outage" or "Major Outage", as planned, but they don't revert back to "Fixed" or "Operational". Is there something that I am overlooking?
Here is a screenshot of cachet-monitor running. The creating incident works but not the resolved downtime...

Monitor Code:
"monitors": [
{
"name": "Phone Directory",
"url":  "Site",
"threshold": 80,
"component_id": 2,
"interval": 5,
"timeout": 5,
"expected_status_code": 200,
"strict_tls": true
}
Any Ideas?

Could be cachet's api being more up to date than the code here. The last work on the project was done in 2016 June/July.
I have plans to update this project and add more features in the near future.
Code that updates the component: https://github.com/CastawayLabs/cachet-monitor/blob/master/incident.go#L40

Thank You for your time on this great project.

@Mikagami can you check with the latest release?

@matejkramny Hi, i tried with the latest release. I got this error, but i checked twice my json.

time="2017-02-13T11:58:44-03:00" level=warning msg="Resolving incident" fields.time="13/02/2017 11:58:44 BRT" monitor="Portal Institucional" time="13/02/2017 11:58:44 BRT"
time="2017-02-13T11:58:44-03:00" level=info msg="Error sending incident: Cannot parse incident body: unexpected end of JSON input, " fields.time="13/02/2017 11:58:44 BRT" monitor="Portal Institucional" time="13/02/2017 11:58:44 BRT"

{   "api": {     "url": "https://xxx.xxx.xxx.xx/api/v1",     "token": "ODhIjEQZMJKk0H2yPS6r",     "insecure": true   },   "date_format": "02/01/2006 15:04:05 MST",   "monitors": [     {       "name": "Portal Institucional",       "target": "https://ufrn.br",       "strict": true,       "method": "GET",       "component_id": 1,       "metric_id": 1,       "template": {         "investigating": {           "subject": "{{ .Monitor.Name }} - {{ .SystemName }}",           "message": "{{ .Monitor.Name }} check **failed** (server time: {{ .now }})\n\n{{ .FailReason }}"         },         "fixed": {           "subject": "{{ .Monitor.Name }} - {{ .SystemName }}",           "message": " ***  F I X E D  ***"         }       },       "interval": 10,       "timeout": 1,       "threshold": 80,       "headers": {         "Authorization": "Basic <hash>"       },       "expected_status_code": 200     }   ] }

Cachet version?

Btw you don't need the templates as it uses default messages if it's empty

i'm building cachet from master branch @matejkramny

I was checking whether the incident would update after the issue has been resolved and I get the following error when it tries to update the incident.

Ok let me investigate & come back to you both

I'm getting the same result:
time="2017-03-21T15:27:29+11:00" level=warning msg="Resolving incident" fields.time="21/03/2017 15:27:29 AEDT" monitor="Document Template PDF Rendering" time="21/03/2017 15:27:29 AEDT"
time="2017-03-21T15:27:29+11:00" level=info msg="Error sending incident: Cannot parse incident body: unexpected end of JSON input, " fields.time="21/03/2017 15:27:29 AEDT" monitor="Document Template PDF Rendering" time="21/03/2017 15:27:29 AEDT"

This is the version of cachet I'm using:
{
  "meta": {
    "on_latest": true,
    "latest": {
      "tag_name": "v2.3.10",
      "prelease": false,
      "draft": false
    }
  },
  "data": "2.4.0-dev"
}

This is the relevant error from the Cachet laravel log
[2017-03-21 05:37:29] production.ERROR: exception 'ErrorException' with message 'Missing argument 10 for CachetHQ\Cachet\Bus\Commands\Component\UpdateComponentCommand::__construct(), called in /var/www/Cachet/app/Bus/Handlers/Commands/Incident/UpdateIncidentCommandHandler.php on line 101 and defined' in /var/www/Cachet/app/Bus/Commands/Component/UpdateComponentCommand.php:121

Hi!
Any solution? I'm with the same error and i'm using the latest version - 2.3.9
WARN[0010] creating incident. Monitor is down: Expected HTTP response status: 404, got: 200  fields.time="16/06/2017 15:29:19 -03" monitor="Blog Talita" time="16/06/2017 15:29:19 -03" INFO[0010] Error sending incident: Cannot parse incident body: unexpected end of JSON input,   fields.time="16/06/2017 15:29:19 -03" monitor="Blog Talita" time="16/06/2017 15:29:19 -03"

hi guys, I am facing same issue as @HenriqueAndre . Do we have solution for this? Can somebody help ?

With the current2.4.0-dev version of Cachet is this no longer an issue :)

With the current2.4.0-dev version of Cachet is this no longer an issue

I wonder if I am doing something wrong in that case. The status is never changed from partial/major outage to something else. I do not see any attempts either, all I see coming by in the logs is WARN[0030] <name> is now saturated

I'm having the same issue. can someone help me? I tried to test using a very simple monitor but still getting the same
INFO[0010] monitor down 100.00%/80.00%                   monitor=google time="28/09/2020 01:14:27 GMT"
WARN[0010] creating incident. Monitor is down: Expected HTTP response status: 400, got: 200  fields.time="28/09/2020 01:14:27 GMT" monitor=google time="28/09/2020 01:14:27 GMT"
INFO[0010] Error sending incident: Cannot parse incident body: unexpected end of JSON input,   fields.time="28/09/2020 01:14:27 GMT" monitor=google time="28/09/2020 01:14:27 GMT"
DEBU[0010] Sending lag metric ID:4 RTT 65ms
cat sample.yaml

api: ~
date_format: "02/01/2006 15:04:05 MST"
insecure: true
monitors:
component_id: 1
expected_status_code: 400
interval: 1
method: GET
metric_id: 4
name: google
strict: true
target: "https://google.com"
template:
  fixed:
    subject: "I HAVE BEEN FIXED"
  investigating:
    message: |-
        {{ .Monitor.Name }} check **failed** (server time: {{ .now }})

        {{ .FailReason }}
    subject: "{{ .Monitor.Name }} - {{ .SystemName }}"
threshold: 80
timeout: 1


Timeline summary contains CPU, GPU and Memory usage

As of flutter.dev/go/engine-cpu-profiling, we collect the CPU and
Memory usage. With work being done to collect GPU usage on iOS as well.
This adds them to the timeline summary.
Fixes: https://github.com/flutter/flutter/issues/58803
Tests
I've added ProfilingSummarizer tests to timeline_summary.dart. I've also amended the tests in timeline_summary.dart to account for cpu and memory usage.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

@liyuqian I've addressed the pending comments. PTAL.


Removed unnecessary error from _area_inout

The error was previously removed from _body_inout
for the same reason. Fixes #28022.

Thanks!

Cherry-picked for 3.1.1.


[Feature][seatunnel-connectors] Support Flink ClickHouse Connector

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Support Flink ClickHouse Connector
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

How is this feature going? I am interested in it, and I'm willing to submit a PR.

How is this feature going? I am interested in it, and I'm willing to submit a PR.

The implementation will refer to the ideas in #1382, you can wait for PR to enter before starting


receive firehose events through params

@bonzofenix Sorry for the delay on getting this merged.  Looks like a good extension.


Modify README

Add coverage badge
Add license

Coverage remained the same at 100.0% when pulling 1ff6f7123439bd77e6397c309070a46fa000d690 on modify-readme into 417429ad60b93c311e6511d1c3c41e8bd464da79 on develop.

LGTM


How to generate report file with Japanese SpecFlow keyword (Given, when, then, ...)

I try to generate a Japanese report but no idea how to do that
What I haved tried
I am using .NET 8 with SpecFlow
The feature file is Japanese (followed this keyword https://github.com/cucumber/gherkin/blob/main/gherkin-languages.json)
var sparkReporter = new ExtentSparkReporter(...)
_extentReports = new ExtentReports();
_extentReports.GherkinDialect = "ja";
_extentReports.AttachReporter(sparkReporter);
...
_feature = _extentReports.CreateTest(
new GherkinKeyword(Feature.Name.MapToLocaleKeyword()), // change "Feature" to "フィーチャ"
FeatureContext.FeatureInfo.Title,
string.Empty);
...
_extentReports.Flush();
But the report file still have SpecFlow keyword as English
Can you help me about that. Thanks

@duy123a  Hi,
I checked it in the mater branch, but it is displayed normally in Japanese.
master branch
5.0.3-beta
  _path = DateTime.Now.Millisecond + FileName;

  _extent = new ExtentReports();
  _spark = new ExtentSparkReporter(_path);

  _spark.Config.Theme = AventStack.ExtentReports.Reporter.Config.Theme.Dark;
  _extent.GherkinDialect = "ja";

  _extent.AttachReporter( _spark);

_extent.CreateTest(new GherkinKeyword("フィーチャ"), // change "Feature" to "フィーチャ"
"フィーチャ  ",
 string.Empty);
_extent.Flush();


Update README.md: typo fix

One character typo fix.
What is the purpose of your pull request?

[X] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Deleted a single comma that shouldn't have been present.

The comma is meant to be there: "... a different, or even the reporter's own, application." That is, "... a different (or even the reporter's own) application."


Update README.md

Adding a version of g-1 written in R

This is very cool! Can you change the PR to point to #develop? Then I will approve and merge.

This is very cool! Can you change the PR to point to #develop? Then I will approve and merge.

Thank you for the nice comment and the merge!


It is Blocked Out When I Open Media

Step to reproduce

Start the Clock Bar, it is should be available at the most right of the small touch bar.
Play a media (e.g. Spotify) that will trigger the media icon on the touch bar to be appeared at the most right.

Expected Behaviour
Both clock and media icon appearing side to side
Actual Behaviour
Media icon is appearing buth clock bar is not appearing

These apps overwrite the icon on the right. That is normal behavior on opening the media apps. I am not sure how to make the clock bar permanently fixed. Making it permanent is not what the apple would allow. Thanks


routes_it.rs:35-37: Enable integration test case for home...

The puzzle 79-c1f43fa7 from #79 has to be resolved:
https://github.com/h1alexbel/fakehub/blob/6932d8bc8d0c334bc7dba4106a820a77e305f622/server/tests/routes_it.rs#L35-L37
The puzzle was created by rultor on 18-Jul-24.
Estimate: 35 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd thanks for detailed and disciplined report.

I see you're talking to me, but I can't reply since I'm not a chat bot.

Let's try to use Test Axum Server: https://docs.rs/axum-test/latest/axum_test/


docs(changelog): fix markdown syntax problem

What kind of change does this PR introduce?
docs change. Its a small changelog entry syntax tweak.

Did you add tests for your changes?
N/A
If relevant, did you update the documentation?
N/A
Summary
Fixed a syntax issue in the changelog

Does this PR introduce a breaking change?
no

Other information

Thanks! We pushed this on main.


Implement Deactivate Functionality in the Error Suppression Table

As an ECMPS user, I want to be able to deactivate data in the error suppression table so that it will no longer appear in active searches
Given a user has navigated to the error suppression table
When  the user clicks on the deactivate check mark for a line item
Then the warning pop-out will appear
Given a deactivate warning pop-out appears
When the user confirms in the pop-out that they want to deactivate the data
Then the pop-out closes and the deactivate check box will be marked for the selected line item
Given a user has successfully deactivated a line item in the error suppression table
When the user views that line item
Then the deactivate check box will be disabled (greyed out)  and that set of data will no longer appear in the active search
https://projects.invisionapp.com/freehand/document/1dwq6maiy?zoomToItems=Y2xidmJqaWZrMDBhajNiNnh1bWdycXRtcw%3D%3D

Note: The Invision board was updated such that there is a status column showing active vs. inactive status, a checkbox to select any line item, and a button above the table that deactivates any selected line items. After an active item is deactivated, it's status changes to inactive in the status column, and nothing is greyed out, since the select checkbox still needs to be available for cloning if necessary.


New Tool for Per-Sequence Depths

Hi @sstadick  and thanks a lot for developing perbase!
I'd love to use perbase to estimate the final coverages of multisteps short/long/hybrid bacterial whole-genome assemblies. For these use cases, a distinct tool for per-sequence summarizing information would be very helpful.
For example if we could get the following information for each sequence within an assembly:

seq-id
min  coverage
average coverage
max coverage
normalized cov

By "normalized cov" I mean the average coverage of a seq normalized to the average coverage of the longest sequence, like in Unicycler. By this we get important information, as for example that a sequence "B" has 5x the coverage of sequence "A" (often the longest sequence is the chromosome or at least a larger part of it).
Again thanks and best regards!

Hi! I'll keep this open in the event that I come back into perbase to do more development, but I don't currently have enough bandwidth to be doing a lot of work here, PRs are always welcome though!


Upgrade openssl to 0.10. or replace it with rusttls

I have an issue when building a project that use ftp 'Unable to detect OpenSSL version
from here you can see that we need to upgrade openssl, I saw that we had the same issue before so why we don't move to rusttls which is newer than OpenSSL, and doesn't provide any support for TLS 1.1 and older, or any known-insecure algorithms. As a consequence, it's a lot simpler than OpenSSL, and is built purely in Rust, so doesn't require any magic to cross-compile.

I am also facing this issue on Manjaro. Here's the backtrace :
`
[lucas@lucas-vm test-ftp]$ export RUST_BACKTRACE=1
[lucas@lucas-vm test-ftp]$ cargo build
Compiling num-traits v0.2.6
Compiling winapi-build v0.1.1
Compiling semver-parser v0.7.0
Compiling num-integer v0.1.39
Compiling libc v0.2.51
Compiling winapi v0.2.8
Compiling cc v1.0.35
Compiling num-iter v0.1.37
Compiling pkg-config v0.3.14
Compiling foreign-types-shared v0.1.1
Compiling openssl v0.9.24
Compiling bitflags v0.9.1
Compiling lazy_static v1.3.0
Compiling regex-syntax v0.3.9
Compiling utf8-ranges v0.1.3
Compiling lazy_static v0.1.16
Compiling kernel32-sys v0.2.2
Compiling semver v0.9.0
Compiling foreign-types v0.3.2
Compiling rustc_version v0.2.3
Compiling thread-id v2.0.0
Compiling memchr v0.1.11
Compiling time v0.1.42
Compiling thread_local v0.2.7
Compiling aho-corasick v0.5.3
Compiling openssl-sys v0.9.43
Compiling regex v0.1.80
Compiling num v0.1.42
Compiling chrono v0.2.25
error: failed to run custom build command for openssl v0.9.24
process didn't exit successfully: /home/lucas/IdeaProjects/test-ftp/target/debug/build/openssl-7ffbea3cdfc8134c/build-script-build (exit code: 101)
--- stderr
thread 'main' panicked at 'Unable to detect OpenSSL version', /home/lucas/.cargo/registry/src/github.com-1ecc6299db9ec823/openssl-0.9.24/build.rs:16:14
stack backtrace:
0: std::sys::unix::backtrace::tracing::imp::unwind_backtrace
at src/libstd/sys/unix/backtrace/tracing/gcc_s.rs:39
1: std::sys_common::backtrace::_print
at src/libstd/sys_common/backtrace.rs:70
2: std::panicking::default_hook::{{closure}}
at src/libstd/sys_common/backtrace.rs:58
at src/libstd/panicking.rs:200
3: std::panicking::default_hook
at src/libstd/panicking.rs:215
4: std::panicking::rust_panic_with_hook
at src/libstd/panicking.rs:478
5: std::panicking::begin_panic
at /rustc/91856ed52c58aa5ba66a015354d1cc69e9779bdf/src/libstd/panicking.rs:412
6: build_script_build::main
at ./build.rs:16
7: std::rt::lang_start::{{closure}}
at /rustc/91856ed52c58aa5ba66a015354d1cc69e9779bdf/src/libstd/rt.rs:64
8: std::panicking::try::do_call
at src/libstd/rt.rs:49
at src/libstd/panicking.rs:297
9: __rust_maybe_catch_panic
at src/libpanic_unwind/lib.rs:87
10: std::rt::lang_start_internal
at src/libstd/panicking.rs:276
at src/libstd/panic.rs:388
at src/libstd/rt.rs:48
11: std::rt::lang_start
at /rustc/91856ed52c58aa5ba66a015354d1cc69e9779bdf/src/libstd/rt.rs:64
12: main
13: __libc_start_main
14: _start
warning: build failed, waiting for other jobs to finish...
error: build failed
`

UPDATE: I just came back to this and realized someone updated the cargo.toml file to upgrade the openssl dependency (PR #86). Although it has not been merged yet...


Running more apps in the same workspace?

Not an issue - question - feature addition idea
When using this I think the whole thing has an amazing OS feel - can it run even more standalone apps in the same workspace?

I don't think this will be a feature of this project. However, I would be open to making Winamp2-js available in such a way that someone who was making a Window's desktop clone (like http://www.windows93.net/ or something) could include Winamp2-js.
That said, it would be a bit of a process to figure out how the API etc should work.
Thanks for your interest!


Future Goals Center

https://github.com/leela-zero/leela-zero/issues/1758

Many have done by KataGo and KaTrain, so LZ#1758 needs to be filtered.

lightvector/KataGo#676

A/B testing: sanderland/katrain#544

dynamic draw: https://github.com/kaorahi/lizgoban/issues/100


Fix gcc compiler warnings

Fix various warnings related to unused typedefs, wrong deallocators,
using * for booleans, etc.
Also, the change isnan to std::isnan is required to make the code compile with ROOT master.

@amarini @nsmith- , can this be marked safe to test? Thanks!


Promote books from /stories

Problem to solve
Books are currently hived away under 'What We Do', without any link to other narrative content within Stories.
Books sit more naturally with Stories because:

Audience of people who like to read interesting things

Hypothesis

Promoting /books from /stories will make them more findable
Linking to books from a higher-traffic section will result in more traffic to individual books pages.

Success measures

We would expect to see a higher CTR to /books from /stories than /what-we-do
Users can find it in navigation testing

TODO

Build book promos per @GarethOrmerod design
Create list of 3 promos to display on Stories

Questions (raised in #4664)

Should the 'Book' label always appear (even on /books)?
Should we display author info in the promo?
Where do they appear within /stories?
Which should appear in /stories (latest 3)?

Should the 'Book' label always appear (even on /books)?
For now, yes. There's an open ticket about optimizing this across multiple types. #3072

Should we display author info in the promo?
Not now

Where do they appear within /stories?
Between 6-part serial and 'You may have missed'

Which should appear in /stories (latest 3)?
Latest 3 that have been published in the real world (not recently published in Prismic).
It'll be:

Spectacle of Illusion
Making of You
Shapeshifters

@davidpmccormick can this issue be closed now?


[feature] to remove duplicates

I think its generally expected feature of this tool, it can be interactively like "select file to delete: 1, 2, 3" with some --interactive option or just auto delete first match (keep only last one) with --remove.

@wa1kb0y excellent suggestion, thank you! The works for a TUI is in the pipeline. Some basic setup code for the TUI is already in the codebase. The --interactive options seems like a much simpler solution as you proposed. I will experiment with this solution a bit before getting deep into the TUI.

@wa1kb0y The latest version 0.0.7 includes the --interactive mode. Let me know if it looks good to you.  https://github.com/sreedevk/deduplicator/pull/16

@sreedevk It works perfect, thank you for great tool!


Be nitpicky about it

Document my magic number
Do a count once instead of filter.length
Try lazy val within method

I wonder if 5 or 6 is too low a value for the single char, especially given that most punctuation is a single char and has already been counted.  The period at the end of the sentence counts for one and pairs of quotes and parentheses count as two.

nice thought, this is one of the reasons I love working with you.  I didn't
even think about it.
Maybe instead we should check for only _.isLetter and .length == 1 (if you
get my drift)
On Wed, Jun 9, 2021 at 3:38 PM Keith Alcock @.***>
wrote:

External Email
I wonder if 5 or 6 is too low a value for the single char, especially
given that most punctuation is a single char and has already been counted.
The period at the end of the sentence counts for one and pairs of quotes
and parentheses count as two.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I would be (a little) more comfortable with that, I guess.  The previous sentence already has 6.

Quite a few tests are failing.  The first one I checked had filtered out this sentence:
Crisis (IPC Phase 3) is widespread and
Emergency (IPC Phase 4) outcomes exist in parts of Unity,
Western Bahr el Ghazal,

This was even after the change to the single letter condition.  The numbers probably need to be higher or the kinds of symbols different.

OK, i'll look tomorrow, thanks for point this all out!
On Wed, Jun 9, 2021 at 10:45 PM Keith Alcock @.***>
wrote:

External Email
Quite a few tests are failing. The first one I checked had filtered out
this sentence:
Crisis (IPC Phase 3) is widespread and
Emergency (IPC Phase 4) outcomes exist in parts of Unity,
Western Bahr el Ghazal,
This was even after the change to the single letter condition. The numbers
probably need to be higher or the kinds of symbols different.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I could output the counts for each measurement and then look at their distribution for at least the tests so that they can be made to pass.  It might be a good idea to check a real corpus eventually.

Here it is for numSingleAlpha:

count
instances

0
10384

1
666

2
92

3
8

4
1

5
0

6
0

7
0

8
0

9
0

10
0

11
0

12
0

13
0

14
0

15
2

16
0

The 33 and 15 are for the one sentence we're trying to avoid.

We know all the one-letter English words, don't we?  I and a.  Perhaps they should not be counted.

Maybe not count valid numbers and punctuation.  For purposes of ruling out the single sentence, numSingleAlpha < 10 is sufficient.  We're getting too many false fubars otherwise IMO.

sounds fine to me
On Fri, Jun 11, 2021 at 11:07 AM Keith Alcock @.***>
wrote:

External Email
Maybe not count valid numbers and punctuation. For purposes of ruling out
the single sentence, numSingleAlpha < 10 is sufficient. We're getting too
many false fubars otherwise IMO.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

This code isn't as elegant as that of @BeckySharp, but it does pass the tests.

Do you think this is OK, @BeckySharp?  The numSingleAlpha from the table above looks like a more reliable indicator of a problem that causes the program to hang.  Because there are some valid 1-letter words in English that I wanted to take into account, the code changed location a little.

Good, valid point.  I flipped a coin.  Luckily it's only about three lines of code.  I imagined a table detector, graph detector, PDF problem detector, ways to combine them, plugins, and language specializations... that would eventually be easier if it was a class.  For today, it's close enough for me.


Clara DSL throws NPE for .. macro symbol

clara.rules.dsl mistakenly treats the .. macro symbol as a Java/host constructor call, which causes a NPE to throw.

Adding a special case to avoid the .. macro symbol in the constructor call conditional

Merged. Thanks, Mike!


No CSS when bundled with aurelia-cli (requirejs)

I'm submitting a bug report

Library Version:
1.0.0-rc.1.0.3

Please tell us about your environment:

Operating System:
Linux (Ubuntu 16.04)

Node Version:
7.10

NPM Version:
4.2

Browser:
all

Language:
all

Current behavior:
Adding dependencies as shown in the docs here:
{
  dependencies: [
  //...
    {
      "name": "aurelia-dialog",
      "path": "../node_modules/aurelia-dialog/dist/amd",
      "main": "aurelia-dialog"
    }
  //...
  ]
}

When opening the modal, ai-dialog custom elements are not decorated.
Expected/desired behavior:

What is the expected behavior?
ai-dialog custom elements should be decorated.

Temporary workaround
I added another section to dependencies:

{
  dependencies: [
  //...
          {
            "name": "aurelia-dialog-resources",
            "path": "../node_modules/aurelia-dialog",
            "main": "karma.conf",
            "resources": ["styles/*.css"]
          }
  //...
  ]
}

Then in my modal template, add <require from="aurelia-dialog-resources/styles/output.css"></require>

Never mind, I was using old elements (ai-dialog).
When I switched my code to using ux-dialog elements, everything works without having to require styles/output.css


Bug: Extension not respecting currencies setting

I have show additional currencies disabled, but when I enter the Tracklayers' City or do anything there, the Hinterland Prosperity display appears in the sidebar.

Yes, the new indicator has some issues in regard to its activation. I will soon release a bugfix release, stay tuned.

sigh I really need to up my testing game... Will fix it soon.
Each subsetting seems like a good idea, but the current layout of the "Preferences" screen seems ill-suited for that. Maybe after a rework?


Allow connecting to Postgres database via SSL

Postgresql servers can be configured to force a client to connect via ssl. Kong should allow for ssl connections to postgres in its config, as is available for cassandra. This is especially important given all the secrets that are stored in the database in plain text (e.g. jwt secrets).
For reference: https://www.postgresql.org/docs/9.1/static/ssl-tcp.html

This is the blocker for Kong with Postgres on Heroku, because Heroku Postgres requires SSL: https://github.com/heroku/heroku-buildpack-kong/issues/7
I'm selfishly in support of forking pgmoon to get this fix @thibaultCha

Alright, we just created a temporary fork at Mashape/pgmoon and we will include Postgres SSL connections in the upcoming 0.9 release (this month).

#1425 implements those! Testing appreciated!

Has been implemented.

Thanks! I've been on vacation but I will take a look this week.


As per meeting notes of Jan 17, reword the 25% requirement.

As per meeting notes of Jan 17, reword the 25% requirement.

May I suggest a minor rewording:
"an active contributor to the project in general and the code base in particular"

I think this is reasonable

just confirming that  I'm good with Jeff's latest edits


Add base64 examples

A base64 digit is the ideal candidate for safe_integer<6, overflow_tag> where overfloat_tag is trap in Debug and ub in Release.

Not much interested in doing this now...


Fix corner case when /v2/apps returns apps across all spaces.

There's a corner case given a developer has access to multiple spaces
where there are apps with the same name (despite having different
routes). Prior to this patch, it would DoesAppExist would return
false because it would see more than one app thus causing the script to
deploy as a first time deploy which would really cause downtime if it's
pushing to an existing app in the space. This patch will search for the
apps and correlate between app and it's space and count them.
cc @adelevie

Thanks for finding this edge case and figuring out how to fix it! We implemented a more succinct solution based on the pull request you submitted. Please let us know if it does not solve your problem.