Log which admin approves a user

The account email of the admin which approves a user should be stored in the ACL to provide some overview over who approves who.

This has been realized in the user_admin branch and just needs to be documented.


Improve Actions hygiene

👋 hello there! I'm a fellow Googler who works on projects that leverage GitHub Actions for CI/CD. Recently I noticed a large increase in our queue time, and I've tracked it down to the limit of 180 concurrent jobs for an organization. To help be better citizens, I'm proposing changes across a few repositories that will reduce GitHub Actions hours and consumption. I hope these changes are reasonable and I'm happy to talk through them in more detail.

Only run GitHub Actions for pushes and PRs against the main branch of the repository. If your team uses a forking model, this change will not affect you. If your team pushes branches to the repository directly, this changes actions to only run against the primary branches or if you open a Pull Request against a primary branch.

For long-running jobs (especially tests), I added the "Cancel previous" workflow. This is very helpful to prevent a large queue backlog when you are doing rapid development and pushing multiple commits. Without this, GitHub Actions' default behavior is to run all actions on all commits.

There are other changes you could make, depending on your project (but I'm not an expert):

If you have tests that should only run when a subset of code changes, consider gating your workflow to particular file paths. For example, we have some jobs that do Terraform linting, but they only run when Terraform files are changed.

Hopefully these changes are not too controversial and also hopefully you can see how this would reduce actions consumption to be good citizens to fellow Googlers. If you have any questions, feel free to respond here or ping me on chat. Thank you!

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

The commit should be named along the lines of .github/workflows: improve actions hygiene, or it won't pass CI.
Yes, we use forks and only single master branch.


Need to eventually add back these tests

Once this bug has been fixed:
https://github.com/ReactiveSocket/reactivesocket-java/issues/162
merge this branch so that the tests that are failing will pass.

You still have old commits showing up.  Can you look into why this keeps happening for you?
send me a fbpaste of first page of "git log master" and "git log pr_bug_fix"

I don't understand this branch.  I assume it #167 but without the tests at the end commented out?
If so, can we fix the tests first before integrating this into the build.

@yschimke sorry, I know why old commit are showing up, it's because I need to rebase from upstream. But in this case I didn't rebase because those commits that show up haven't been accepted yet, but thinking about it now, I probably should in order to avoid merge conflicts.
As for the purpose of the branch, it should just be the failing tests commented out. Let me rebase and do another request.

The commits looked very old. I did the same thing to @lehecka with overlapping PRs. So I share your pain. But can you check the earliest commits in this PR

@yschimke I don't think my local master is the problem, because I've tried that solution. It's the origin/master that has all the commits, and the only way to get rid of those is to rebase the fork on upstream/master or I guess delete the fork and refork. I just tried rebasing, and it seems like I got rid of the old commit.

Also, I don't think https://github.com/ReactiveSocket/reactivesocket-java/issues/162 has been marked as a bug yet. Would you be able to do that?


Fix findOne() method failure and improve error handling

Purpose

Fix findOne() method failure Fix https://github.com/wso2-ballerina/module-mongodb/issues/65
Improve error handling - Introduces ClientError and ServerError to indicate errors when initialising client and performing operations respectively (Fix https://github.com/wso2-ballerina/module-mongodb/issues/56 includes https://github.com/wso2-ballerina/module-mongodb/pull/61)
Improve test coverage - Fix https://github.com/wso2-ballerina/module-mongodb/issues/62
Fix upsert param in replace() being redundant - Fix https://github.com/wso2-ballerina/module-mongodb/issues/64

Goals
$subject
Approach

Return Json as an object from jBallerina implementation
MongoDB client errors are captured and mapped to ballerina erros

User stories
N/A
Release note
Improve error handling by introducing ballerina errors namely, ClientError and ServerError to indicate client initialisation exception and connection exception when performing operations.
Fix findOne() method failure
Documentation
N/A
Automation tests

Unit tests - Included
Integration tests - N/A

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment
MongoDB 4.2.0

We need to update the Module.md to reflect the latest version update.


set physical memory protection and remove redundant macros

Configuring PMP registers seems to be necessary for the example to work now.

I believe you.


document the need to re-generate webhook secret if it contains /

@nathanaelhoun can you please remove that license compliance check from repo. I don't think we need to run it for every commit.

@darkLord19 Sorry it was an error. I thought I had already removed it.


Update Install-the-Federation-Service-Proxy-Role-Service.md

Hi, the documentation is incorrect and we have a customer comment that points that out. Proposing changes to accurately reflect the installation steps for the Web Application Proxy role for AD FS

Fantastic! Thanks for fixing this. You'll see the changes live today (6/19).


Where the .mat file?

As described in the README of the Vehicle dataset, vehicle.mat can be downloaded at: https://drive.google.com/file/d/1x39M3wYLt0VKNS8RSiwohT-VfVoGeKlM/view.
Please let me know if you have problems accessing the dataset.


Update downloaders.py

fixed download progress info parser

You can share the URL to reproduce the error in the output try to fix. Is possible say

Which CLI Backend your using youtube-dl or yt-dlp ?
Full options to pass to the CLI Backend

Thanks for the patch. See  commit https://github.com/oleksis/youtube-dl-gui/commit/0c7475d482e0243a2588d966f9b0645e75435d91


eval incorrectly listed as unchanged from ROS 1

The launch migration guide incorrectly states "eval and dirname substitutions haven’t changed." Eval statements do often need some syntax changes while migrating. See this issue for an example of the confusion this causes.

Yes, that's true, there are some differences here (particularly in the escaping).  If you'd like to contribute a pull request to improve that, it would be much appreciated.

I was hoping someone knowledgeable could give a more comprehensive list of changes. I created a pull request which adds a note about escape characters.


Why using different hyperparameters in VOC split2?

Hi authors,
I found the hyperparameters for the experiments on VOC split2 are a little bit different from the ones used in split1. For example, the learning rate in split2 is 5x of the one in split1. And the learning rate schedule (the step to decrease learning rate) is also a bit different.
My question is that are the results in the paper generated by different hyperparameters for three splits? If so, do different hyperparameters offer better results than a unified set of hyperparameters?
Thanks for your reply in advance.

We found that for split 2 of PASCAL VOC, a lower learning rate improves the results, although not by much. This is probably due to the different set of categories, so different hyperparameters may help.
Hope that helps!


Empty grid with long header doesn't have horizontal scrollbar

jsfiddle: https://jsfiddle.net/rhwxue73/
Issue is based on https://github.com/tabalinas/jsgrid/issues/744

colspan Only specifies the number of columns that a cell element will overlap. It does not specify the intended width. Cell widths are determined by content, or css. Setting the width of the "Not Found"  element to the sum of all columns width should fix the problem.

I have the same problem, help me.
Scroll should be activated

Hi.
Kindly.
Is this problem still unresolved? or fixed that?
It have adversely effect on user experience.

After read some issue about this problem, I think if you want to resolve that as simple way, you can do like this https://jsfiddle.net/c6wtLmc0/1/
In other word, in my opinion  you can change insert button behavior very simply and apply all your validation in other place(for example validate in editing)
I know that doing by this way might not be good, but it works in my case.
I hope it can be help others.

It would be nice to have a fix for this. I have a table with lots of columns and filter boxes everywhere, and I observed the following:

scroll to the right to make a column visible that normally does not show
enter a filter condition that results in no data found
the refresh of the table will reset the scrolling to the left
now you can't see the filter condition anymore and you can't change it because there's no scrollbar.

A workaround is to click into any filter and then hit tab until the table scrolls to the right. However, if you have only one filter in the rightmost column you're kinda screwed (reload of the whole page is necessary).

Here's a modified _createNoDataRow that implements richraid21's idea:
        _createNoDataRow: function() {
            var amountOfFields = 0;
            var totalWidth = 0;
            this._eachField(function(field, index) {
                amountOfFields++;
                totalWidth += field.width;
            });

            return $("<tr>")
                .addClass(this.noDataRowClass)
                .append($("<td>")
                .addClass(this.cellClass)
                .attr("colspan", amountOfFields)
                .attr("width", totalWidth)
                .append(this.renderTemplate(this.noDataContent, this)));
        },

Works for me...

Hi,
I had the same problem and found a simple fix. Just paste the code directly before the jsgrid.
`

`
Here is the JSFiddle link: https://jsfiddle.net/kL8arzeo

Hi,
I had the same problem and found a simple fix. Just paste the code directly before the jsgrid-element.
<style>
     .jsgrid-grid-header,
     .jsgrid-grid-body {
         overflow: auto;
}
</style>

Here is the JSFiddle link: https://jsfiddle.net/kL8arzeo

As a quick and dirty workaround we show add a single empty row when there is no data.


add EPF dataset to Google Drive

Thanks to the open source model of TimeXer, I realised that the EPF dataset is not added to the Google Drive, could I think about adding it?
I've tried to go to the EPF dataset paper to find the link to download it, but the database address they gave is no longer valid.

Hi, we have provided EPF datasets in TimeXer's official repo.  Thanks for your valuable suggestion, we will take it into consideration.


Add discord link

Questions
Answers

Description?
Adding the field to add a discord link, request from @Petruzzo73 in this issue: https://github.com/PrestaShop/PrestaShop/issues/16356#issuecomment-1046090817

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes PrestaShop/Prestashop#12.

How to test?
#

I think we should add the discord icon, like I've read in the conversation of this PR (@PierreRambaud, @TristanLDD): https://github.com/PrestaShop/ps_socialfollow/pull/15
Added icon at module level, through a css file. The icon can be overridden with a css file and icon at theme level

I think you put the wrong issue in your table 🤔

I think you put the wrong issue in your table 🤔

Actually there isn't a real related issue 😊 but only the mentioned comment from @Petruzzo73

The module isn't that flexible, you could be able to add any social network you want if you provide the icon, SVG or whatever

Agree with you, a big change should be made, remove all that separated fields and create its own table where record each 'social' and associate link and icon/image

Have you checked if the Discord icon is available in front office?

As far as I know there no discord icon in the classic theme, how we can solve this?

@marsaldev
it has to be implemented on the theme level, too. You should make a pull request in the classic theme repository to add a .svg icon :-)
@NeOMakinG, what about the new theme?

@marsaldev
it has to be implemented on the theme level, too. You should make a pull request in the classic theme repository to add a .svg icon :-)
@NeOMakinG, what about the new theme?

Nice catch btw
The new theme should probably have the icon too, so it needs 2 more PRs, but it's a nightmare, why icons aren't on module level? Some access right issues?

Module should use icons on module level
If it's overridden in the theme, it will use the override

If we are able to do this, this is way more maintainable than the actual behavior

@NeOMakinG
The new theme should probably have the icon too, so it needs 2 more PRs, but it's a nightmare, why icons aren't on module level? Some access right issues?

Module should use icons on module level
If it's overridden in the theme, it will use the override

If we are able to do this, this is way more maintainable than the actual behavior

We can start a "transition" with this one, I can create a PR to add an icon only for Discord

wdyt @PrestaShop/prestashop-maintainers , maybe I miss something

Icons are related to the module, so it should be in the module first, then if the file is found in the current theme, it can be used.
But I see no reason to store icons only in the theme.

That's a lot of commits, please rebase your PR :)

@marsaldev Hi, thank you for your contribution.
Where did you find the icon ? What is the license of this image ? (@ttoine)
Thanks for your answer.

Taken on the official discord web site: https://discord.com/branding
I don't see any mention to any licence on the page 😕

Quote form https://discord.com/branding

We’re a playful and fun brand that doesn't take ourselves too seriously. That said, we're keen on keeping things quality. If you're gonna use our art for something please keep it tasteful and send it our way for approval.
Does it mean any usage of the logo, commercial or not, is to be approved by Discord ?

ping @PrestaShop/product-team ,
The issue related to this PR is closed and doesn't involve a discord link, but other social links.
What should be done ? Re-open the issue ? Create another one ? Something else ?
Thanks!

We can re-open.
@ttoine could you tell us about the icon license

My point of view:
Using an icon pack with a creative commons by-sa licence (or equivalent, or more permissive) would simplify the license problem: the IP work is done by the creator of the pack, and we don't have to check each time we add a new social network. We just have to update the pack from time to time or even better, to use it as a dependency at build. (And yes it would be great, then, to provide a way to add unlimited custom social network, of course in an other pr/issue, with icons from the pack or custom icons provided by the users --PM team can work on that)
Also in term of design, it's coherent, instead of having each icon being very different. And it solves the problem of future social networks all at once.
About the license of one specific logo:
It's a company/product/service logo, so it's a brand. In this case it's owned by Discord. If it's taken from their branding guidelines pack of artwork, then their copyright applies (like PrestaShop logo or any company/product/service logo, btw). We must add something about it somewhere, maybe in the readme.md, and if it was not done previously, it must be done too for other logos distributed with the module (same for the classic theme if any)
Something like the list of all the logos that are shipped, with a link to where people can find the original, and the copyright information

ping @ttoine Any help? 😊

at the bottom of the readme, you can add a section called "Third party logos" and add a list of icons and a link to the page where people can find more information. (typically for Discord the page where you downloaded the icon)

Hello @marsaldev ,
Thanks for the PR !
The Discord logo doesn't display on my shop, the logo is present in /assets/icon/discord.svg  :
https://user-images.githubusercontent.com/16019289/161512677-39c09cf2-2331-4e80-b2cd-a7657ed8fae0.mov
Could you check ? The rest looks good. The BO configuration for the logo is good too.

@florine2623

Hello @marsaldev ,
Thanks for the PR !
The Discord logo doesn't display on my shop, the logo is present in /assets/icon/discord.svg :
Screen.Recording.2022-04-04.at.11.10.43.mov
Could you check ? The rest looks good. The BO configuration for the logo is good too.

Weird 🤔, have you hooked the module in actionFrontControllerSetMedia?
Have you checked if the ps_socialfollow.css is loaded properly?

@marsaldev I think you forgot to create an upgrade file for the module to use this new hook in the new version

@marsaldev I think you forgot to create an upgrade file for the module to use this new hook in the new version

Yep, just done 😉

@marsaldev ,
Now I have the logo but the color is not correct and it seems a little too bug :
https://user-images.githubusercontent.com/16019289/161725026-0e325475-d5ab-44db-9e9e-87f2427b803b.mov
Could you check ? ^^
Thanks!

Hi @florine2623,
you are right, I've used the coloured one because I thought we cannot use others colours, but discord provides the white and black too.
So I've update the icon with the white one and resized a little bit the icon, now I think it shows properly 😊

Thank you @marsaldev ❤

Bravo @marsaldev

This is wrong! Only discord icon has styles provided by the module and all the rest is styled in the theme. It shouldn't be like that!. Plus the classic theme loads icons as background of a::before while css added by this PR forces discord as a background of a. It messes up custom themes:

All svg icons should be loaded by the template, not by css. Just like it was done in Hummingbird, which I've just noticed has discord and tiktok still missing:
https://github.com/PrestaShop/hummingbird/blob/34e15d0bd93cfeb2c69e407e3d89a91eec0559cd/modules/ps_socialfollow/ps_socialfollow.tpl#L9-L29

Sure thing. I just need to finish my current project first...


METRON-1189: Add alert escalation to the Alerts UI

Contributor Comments
This PR adds the ability to escalate alerts from the Alerts UI.  This is a follow on to https://github.com/apache/metron/pull/711.
This can be tested locally against full dev.  Follow the instructions in "Development Setup" section in "metron-interface/metron-alerts/README.md" to start the alerts UI.  Then in a separate session, ssh to vagrant and start the Kafka console consumer for the "escalation" topic.
Alerts can be escalated from 2 different views:  alerts list view and alerts detail view.  After logging in, a list of alerts will initially be displayed in a table.  Select multiple alerts with the checkboxes on the right side, then select the "Escalate" option from the "Actions" dropdown.  This will escalate these alerts by sending them to the "escalation" Kafka topic and update the "alert_status" field on these records in Elasticsearch.  Verify the alert_status column now shows "ESCALATE" for the selected alerts and that the alerts were produced to the Kafka console consumer.
Next drill into a single alert by clicking anywhere on a row that's not a link.  This will display a separate window with that alert's details.  At the top there will be several status options that can be selected (OPEN, DISMISS, RESOLVE, ESCALATE, etc).  Selecting any of these should update the alert_status in Elasticsearch and have that status reflected in the UI.  Selecting "ESCALATE" should produce a message to the Kafka console consumer for that alert.
This PR includes several improvements that were needed to complete this task.  These include:

Switching the "alert_status" update call to use the REST layer instead of going directly against Elasticsearch.  The REST endpoint does more than just update the index so this is absolutely necessary for the system to work correctly.
Slightly refactored angular services to match the REST layer.
Changed the backend used for e2e tests from mock services to a dedicated e2e index in Elasticsearch.  I found the mock services were insufficient for testing this feature and would require a lot of work to implement the various services needed (that won't scale well as we add more search features).  I switched to a dedicated e2e ES index instead and am now leveraging the REST service as it is deployed in full dev.  I'm happy to discuss this approach as needed but I feel it's an improvement and makes writing the e2e tests easier.  Scripts and javascript functions are provided to facilitate easy setup/teardown of this index.
The indices included in the alerts UI can now be specified in an environment setting so that different indices can be configured for different environments (for example "alerts_ui_e2e" index is used for e2e tests).  This may not end up being the final solution but they are currently hardcoded in a javascript file and I feel this is incremental progress towards something better.

This is not ready for primetime yet so please DO NOT MERGE.  The e2e tests are still failing intermittently in unrelated tests.  I was seeing this before these changes and the switch to using REST in full dev (instead of mock services) likely made it worse.  I am still working on making this more robust and also adding documentation for the e2e testing framework.  However, the functionality is there and I wanted to give people a chance to use it, review the changes and give some initial feedback.
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && build_utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

@merrimanr tested the UI works as explained in the comments

Should we use saveRefreshState() and restoreRefreshState() when the escalation rest API's are in progress? This will ensure that the page doesn't get refreshed and users would not lose track of their selected alerts.

I researched the saveRefreshState() and restoreRefreshState() addition and it uncovered a bug which then uncovered a problem with how our UI works.
The bug was that the selectedAlerts field (checkboxes in the list) was not getting cleared on any of the bulk actions.  As refreshes happened the checkboxes would disappear and be replaced by new results but that that field would constantly grow because they were never unchecked on the previous page. It was benign before this PR but is now causing a lot of duplicate REST requests.  We need to figure out the right way to clear these when a bulk action finishes but there is also another issue.
As I was testing for this I realized when I set the refresh rate to a high rate I had to click quickly or the page would change and I would lose my selections.  What @iraghumitra mentions above is a pause while the REST requests are being processed but I think it makes sense to pause even earlier, whenever you have a checkbox checked.  I can obviously get it to work if I pause it but I think that's a better UX.

@merrimanr I uncovered the issue when I was trying to integrate METRON-1189 to METRON-1123. I handled it in PR #768, the fix is to empty the selection list whenever we get new data. This would ensure that previous selections are not lingering around.  But as for selections getting removed due to higher refresh interval, I understand the issue the only option I see is pause refresh if any rows are selected on the UI. If you have a better solution in mind I would be glad to implement that too.

The latest commit merges in master and addresses your feedback @iraghumitra.  I solved the issue in my previous comment by pausing anytime a checkbox is checked and resuming when all checkboxes are unchecked or a bulk acton finishes (checkboxes for alerts included in a bulk action are unchecked after it finishes anyways).  Let me know what you think.

  Can we do that as a follow-on?

I wanted to suggest as a follow-on PR too, my bad missed it during the first round of review.

+1


Attachments - Cleanup pending attachments in background

Periodically ticketdesk needs to clean up any pending attachments from storage. These come from uploads for tickets that were either not created, or for uploads in ticket editor that were never saved to the ticket.
Cleaning up pending attachments should be safe if file created > 8 hours ago.

Merging this into new issue #53


Add async testing

This PR adds the ability to benchmark Aff blocks.  In order to deliver on this, I needed to refactor the internals of the library a bit.  The important change to the existing code is that instead of using the ST monad to keep a running state, I made each run return a runtime.  The array of all runtimes is passed in to a function results to calculate a Results record, which has a Show implementation that defines how it is printed on the console.  Let me know thoughts or if you'd like anything modified.

Sorry, but this is meant to be a minimal library. Depending on aff is not an option here. Thank you for the PR, and for working on this, but I suggest making this into a separate library which depends on both aff and minibench.

If you could write this in terms of Eff using callbacks, that would be different of course.

@paf31 it can't rely on minibench without significant refactor because the internals of minibench currently are not written in a way that can be easily reused.
FWIW the library size has not increased much other than relying on Aff.
If definitely not open to depending on aff, then I'll just fork it and publish it under a new name.  If I do so, how would you like me to credit you?

@paf31 sorry, just saw your comment--if you are open to me removing aff and keeping it as eff and callbacks, I'd be happy to do so--can you please re-open the PR and I'll make the change?

If definitely not open to depending on aff

Code size is not the only reason. The core libraries are self-contained and don't depend on things outside the purescript organization. So depending on aff is not an option unfortunately.
I'd definitely be fine with the callbacks option, and that way the aff wrapper could make a simple third-party library.

I don't know why but GitHub won't let me reopen the PR ...

haha, no worries, I'll put in a new PR, thanks for the comments

and totally agreed that the right approach is to not do aff and have a wrapper that does aff, that's how most other libs work anyways

If you decide to fork the library instead, just including a reference in your LICENSE file would be fine, thanks!

@paf31 before I put in the work to pull out aff, do you have general thoughts on the code structure, changes, or approach that you might want me to modify when I put in the follow up PR?

The general structure you have looks okay, but I do have one concern which is that callbacks (or aff) might add overhead to the running time which could affect the results. I'm not sure if this is worth worrying about, but one reason for using ST was that it gets optimized to local vars.

Hmm that's fair. I'll see what I can do about sticking with ST.
I don't think there's much I can do about callbacks, but since it's for testing async code anyways, I think that's probably ok


Deprecate legacy code

Summary
Closes #749
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

36 of 36 (100.0%)  changed or added relevant lines in 15 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 84.139%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
17782

Relevant Lines:
21134

💛  - Coveralls


Unhandled Runtime Error

Describe the bug
When launching pc run, I get this error "Unhandled Runtime Error - TypeError: Cannot read properties of undefined (reading '_bn')
To Reproduce
Steps to reproduce the behavior:
MacOS intel BigSur (also tried on Macbook M1 Latest MacOS)
python 3.11.1
node v14.17.6
pc init
pc run
Expected behavior
See screenshot
Screenshots
See screenshot
** Specifics (please complete the following information):**

Python Version:  3.11.1
Pynecone Version: v14.17.6
OS: 0.1.27
Browser (Optional): chrome

Additional context
Add any other context about the problem here.

Ok thanks for making an issue we will try and recreate on our end this doesn't seem to be a valid version of Pynecone v14.17.6 are you using the latest 0.1.27 release?

Yes sorry I am using 0.1.27. I pasted in the wrong value.

Ok thanks for making an issue we will try and recreate on our end this doesn't seem to be a valid version of Pynecone v14.17.6 are you using the latest 0.1.27 release?

@PlutoniaX  I'm not able to reproduce this error, did this happen after a new pynecone install or an upgrade from a previous version? Can you also share your code as well as node version?

new install. I only starting trying out pinecone yesterday. the code is just the boilerplate: https://github.com/PlutoniaX/pynecone-test after running pc init.
the webpage loads fine but just has that error message attached above which I can click away and it will hide at the bottom left corner.

Closing this as it concerns an old version.
Feel free to reopen if it still occurs on latest version.


[pegtl] Initial port

Hello everyone,
This is a port for PEGTL, from the original repository:
The Parsing Expression Grammar Template Library (PEGTL) is a zero-dependency C++11 header-only parser combinator library for creating parsers according to a Parsing Expression Grammar (PEG).
More info at https://github.com/taocpp/PEGTL

Thanks for the PR!


handle on* event handlers

Enabled handling of element.on* = (...)=>{ ... }  both for wrapping the callback and passing the right flowlet, and for detecting interactable elements.

Yes, will resolve and push again.


Log rails entry points in logstash

We need some way of knowing what entry points are being requested more often and also compare GET vs PUT for some of them for a given period of time.
I could gather some information from nginx events. The problem there is that the request param needs some processing to group requests by controller and action.
cc/ @zenitraM

After talking with Alejandro: there must be a gem or something to spit data we're interested in json format or anything suitable for logstash. I'll investigate it.

Doesn't look like a need anymore. Closing this for the moment.


Error of using !

I'm trying to set a parameter but an error occurs :
Target [Unisharp\Setting\SettingStorageContract] is not instantiable while building [\Unisharp\Setting\Setting]

Hi @AEK-BKF
does 83f9377 fix this issue?


[Improvement-14311][Permission]Remove duplicate checkProjectAndAuth

Purpose of the pull request

close: #14311

Brief change log

Remove duplicate checkProjectAndAuth when already use hasProjectAndWritePerm since the result of checkProjectAndAuth is never used

Verify this pull request
UT

Hi, @caishunfeng @SbloodyS could you please help review this


Update schemas for v1.2 to draft-07

Title pretty much says it all :)
This allows differentiating between the uri and uri-reference format. Draft-04 was ambiguous in that regard, causing false-positive validation failures with some validator implementations.

From our TDC call the sentiment is that this change could cause consumers to break.  We would rather not make this change.


How to use this without Cocoapods?

How can I used this plugin in a project without CocoaPods?
Steps to reproduce:

flutter create . --platforms ios
Add the plugin
cd ios
pod deintegrate
Open XCode and run the app

No, because you can't use "Flutter" without CocoaPods. It's a required component in Flutter's build process.

Oh, in that case I must have misunderstood the instructions for the Firebase plugins. They refer you to install them via Swift, but CocoaPods already installs all pods (including Firestore etc.) via flutter_install_all_ios_pods, so I thought I was supposed to deintegrate CocoaPods.


fixed the ability to create a group with an empty name #5050

Description

Related Issue

Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

:boom: Acceptance test localApiTests-apiGraph-ocis failed. Further test are cancelled...


Not listening 25 port anymore

Postal stuck and all outgoing messages were created as PENDING

Restarted Postal.

Postal is able to send messages now, but not listening port 25.

What can be the reason?

SMTP process is not running:
           ├─1338 [procodile] Postal (/opt/postal/app)
           ├─1344 [postal] web.1
           ├─1346 [postal] worker.1 (i
           ├─1348 [postal] cron.1
           └─1352 [postal] requeuer.1

smtp_server.log is empty

It was hosting provider issue. Anyway, it would be good to have some message in log in case smtp process can not listed on the port.

Most things are already logged but it can be in a few different places.
A way of debugging the problem is to run postal status and then try manually running the commands it shows there so you see the error output directly on screen.


There is bug in 1.24.0

VSCode Version:1.24.0
OS Version:win7/win10

I get a error what is "canot read property 'filtyer' of undefined" when i use the Tools For AI in VScode 1.24.0.

Can you add a screenshot of the error and also see if there is more information the Dev Tools Console?
('Help > Toggle Developer Tools')

I have the same error.
Error: Cannot read property 'filter' of undefined 	at PlatformManager.getAll (C:\Users\xinyi\.vscode\extensions\ms-toolsai.vscode-ai-0.1.8\out\src\platformManager.js:27:39) 	at ConfigurationManager.getAll (C:\Users\xinyi\.vscode\extensions\ms-toolsai.vscode-ai-0.1.8\out\src\configurationManager.js:51:48) 	at ConfigExplorerTreeDataProvider.getChildren (c:\Users\xinyi\.vscode\extensions\ms-toolsai.vscode-ai-0.1.8\out\src\treeView\configExplorer\configExplorerTreeDataProvider.js:47:96) 	at c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:485:478 	at c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:103:887 	at new n.Class.derive._oncancel (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:83:776) 	at Object.t.asWinJsPromise (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:103:850) 	at t.fetchChildrenNodes (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:485:430) 	at t.getChildren (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:483:292) 	at e.$getChildren (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:481:487) 	at e._doInvokeHandler (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:636:832) 	at e._invokeHandler (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:636:550) 	at e._receiveRequest (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:635:631) 	at e._receiveOneMessage (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:635:400) 	at c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:634:315 	at c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:637:395 	at c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:95:432 	at e.fire (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:96:764) 	at Socket.<anonymous> (c:\Program Files\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:154:338) 	at emitOne (events.js:96:13) 	at Socket.emit (events.js:191:7) 	at readableAddChunk (_stream_readable.js:178:18) 	at Socket.Readable.push (_stream_readable.js:136:10) 	at Pipe.onread (net.js:560:20)

This issue was moved to Microsoft/vscode-tools-for-ai#36

Thanks for reporting the issue.
We just published a hotfix to resolve this issue, thanks.


feat(utils): migrate crypto.js to TS

This PR migrates the crypto.js util to TS. It also merges the new crypto.ts file with the older crypto-ts.ts file and updates the corresponding imports.

@martinkaintas there's also a unit test regarding crypto at test/unit/crypto.js I think it would be good to migrate it too


lint shell scripts with shellcheck in CI

especially /quickstart/docker/image/ziti-cli-functions.sh

stale


Create protobuf protoc Grain generator

In Go, we have this Protoc.exe generator:
https://github.com/AsynkronIT/protoactor-go/blob/dev/protobuf/protoc-gen-typedgam/template.go
We need to have a similar for C#.
It should be possible to re-use the same Go generator and just switch out the template, no need to rebuild the actual generator in C#.

404 on the link

https://github.com/AsynkronIT/protoactor-go/blob/dev/protobuf/protoc-gen-protoactor/template.go

404 again

https://github.com/AsynkronIT/protoactor-go/blob/dev/protobuf/protoc-gen-csgrain/template.go
I'm working on translate the content to C# here. any day now


metric scrape from prometheus federation is different from the origin

Proposal
Use case. Why is this important?
metric scrape from prometheus federation is different from the origin
“Nice to have” is not a good use case. :)
Bug Report
What did you do?
I build a prometheus federation, one prometheus in the k8s cluster , and one out of k8s cluster.
I use the outside one to collect metrics from the inside one though federation. and I have  a metric called probe_success_detail
What did you expect to see?
I thought the query result would be the same from both side
What did you see instead? Under which circumstances?
but  the one inside k8s cluster returns 1 result, the one outside k8s cluster return 19 results, it seems federation uses some kind of cache causes this problem
the inside one's result:
probe_success_detail{body_match="false",cluster_id="3",duration="2.851163853",header_match="false",host="test.com",http_code="462",instance="http://test.com/",job="blackbox",name="d92d3b77-b182-49f7-961e-313520dceb1d",namespace="blackbox",path="/",schema="http"}
the outside one has the same label values  but with different durations

Environment

System information:
insert output of uname -srm here

Prometheus version:
v2.13.1

Alertmanager version:
insert output of alertmanager --version here (if relevant to the issue)

Prometheus configuration file:

insert configuration here

Alertmanager configuration file:

insert configuration here (if relevant to the issue)

Logs:

insert Prometheus and Alertmanager logs relevant to the issue here

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Handle scudo rename and changes for rebranch

This patch set updates the old driver to handle changes to the scudo sanitizer in upstream LLVM.
The sanitizer is now standalone and shouldn't be used with other sanitizers. In this change, the library was renamed to scudo_standalone to reflect the change. As such, the driver maps the scudo sanitizer to the appropriate filename.
I've also included the updated logic in the driver to ensure that scudo is the only sanitizer passed to the driver, and to otherwise emit a diagnostic.
CC @Lukasa
rdar://114209408

@swift-ci please test Linux

Manually testing the error reporting with -sanitizer='address,scudo,thread' returns
<unknown>:0: error: argument '-sanitize=address' is not allowed with '-sanitize=thread'
<unknown>:0: error: argument '-sanitize=scudo' is not allowed with '-sanitize=address'
<unknown>:0: error: argument '-sanitize=scudo' is not allowed with '-sanitize=thread'

Linux failure
AttributeError: module 'lit.util' has no attribute 'abs_path_preserve_drive'

Should be fixed by https://github.com/apple/llvm-project/pull/7298

@swift-ci please smoke test Linux

Scudo is now failing silently. I'll have to take a closer look tomorrow on my linux box.

Issue was that not handles crashes differently than non-zero return codes. Passing --crash should fix it.

@swift-ci please smoke test Linux

@swift-ci please smoke test Linux

Driver flag testing wasn't accounting for the fact that scudo has to be run alone now. Fixed that test.

Okay, scudo tests are passing.


quassel: use wrapQtAppsHook for the client

Motivation for this change
#65399
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @phreedom @ttuegel

Since we are calling it with libsForQt5.callPackage, you don't have to manually use wrapQtAppsHook. The qt provided mkDerivation will do the trick (instead stdenv.mkDerivation).

@peterhoeg Does the core actually need wrapping...? I'll go find out, I guess.

I'm pretty sure you should wrap the core as well.

@ttuegel @worldofpeace I tested, quasselcore runs fine without wrapping. It does still use Qt, but it appears that all its Qt dependencies are compile-time -- there's apparently no run-time dependency on any plugins, and obviously not on any style engines or other GUI components.

It does still use Qt, but it appears that all its Qt dependencies are compile-time -- there's apparently no run-time dependency on any plugins, and obviously not on any style engines or other GUI components.

IIRC, it doesn't link libQt5Gui.so, so this sounds good.

Oh, dang, I didn't think of searching before writing a fix... #67746 solves the issue by using mkDerivation as given by callPackage, and removes the dangerous with stdenv;. Should we prefer using the right mkDerivation over re-implementing the patch logic?
Though, I guess this becomes an issue with quasselCore. Well, it does work fine when wrapped, so maybe it's a non-issue?

Though, I guess this becomes an issue with quasselCore. Well, it does work fine when wrapped, so maybe it's a non-issue?

Doesn't that mean quasselCore will end up pulling in extra runtime dependencies...?

Doesn't that mean quasselCore will end up pulling in extra runtime dependencies...?

I guess it's possible, then switching which mkDerivation to use on whether it needs Qt is probably the better option in that case. I just pushed that change. The core does start when using stdenv.mkDerivation (as expected). In my case everything else was a no-op compared to the state before the change.

@samueldr That seems a lot less clear as to what it is actually doing, though, because the difference between mkDerivation and stdenv.mkDerivation is not apparent to someone reading the derivation expression. Even assuming they're familiar with libsForQt5.mkDerivation, they still need to either check the call site or make the assumption that libsForQt5.callPackage was used there.
Actually, I guess that's more of an issue with the 'use a custom deriver' approach in general.

#67746 superseded this.

OK. I am feeling a little disappointed at the course of events here. I do not think that both PRs should have remained open; I believe that ideally either @samueldr should have closed theirs, or they or another maintainer should have asked me to close this one, in order to to ensure that further discussion and work continued in a single place rather than being confusingly fragmented.

OK. I am feeling a little disappointed at the course of events here. I do not think that both PRs should have remained open; I believe that ideally either @samueldr should have closed theirs, or they or another maintainer should have asked me to close this one, in order to to ensure that further discussion and work continued in a single place rather than being confusingly fragmented.

Indeed it did get a little messy.
From my perspective, as an integrator, neither of the changes purposed actually moved along with the changes that needed to happen for me to integrate them. So when I got the time today I finished https://github.com/NixOS/nixpkgs/commit/c52b5b8a5d3319e7e0321c57c9435b32de824cf2 (why it says co-authored). In the end the same goal was achieved, it's not broken, but I do understand the means to this weren't optimal. @Shados In the future I'll take extra measure to give direction to changes with competing PRs.


question: using pause/resume to control which partition gets consumed?

In my library/framework (https://github.com/BEagle1984/silverback/) I create a sort of blocking collection per each partition (a Channel actually) and I push the consumed messages into them. The messages from different partitions are therefore processed in parallel, while the blocking collection ensure that the backpressure is kept under control.
I wonder if I could/should use Pause and Resume on the consumer to further optimize this mechanism. The idea is to pause the partition as soon as a message is being consumed and sent through the processing pipeline, to resume it immediately after it the processing is completed. This would ensure that a message taking longer to consume wouldn't affect in any way the processing of the other partitions.
What do you think? Is this a good idea in your opinion @mhowlett?

unfortunately, librdkafka clears locally cached messages for partitions that are paused, making your idea inefficient. we ultimately hope to improve that behavior, but the work is not prioritized afaik. i'd recommend only using pause / resume for protection against more extreme cases. i.e. to protect against potentially very unbalanced queueing in your .net code (if you are seeing that in practice).

Got it, thank you for the quick and accurate reply.


New release?

We have a data set where the released version of Muon fails, but the main branch succeeds. Would it be possible to get a new release? It seems that 38 commits have been merged since the last release.

I just released v0.2.0.

Thanks!


[Freeze/Slow] Unable to save/format notebook

Marketplace review

It would be good if this was more stable, because every other week something goes wrong with Jupyter in VSCode. Suddenly you cannot save anymore, or the notebook stays in pending state for literal minutes, or you cannot run your formatter, or Copilot interacts weirdly and makes VSCode hang. All of these literally happened to me in the span of two weeks.

@andyscho - are you on vscode insiders, or stable?
It would be great to see a trace log from the window output when this happens - Developer: set log level command and choose trace.
Once the issue occurs, go to the output panel and select window from the drop down. You might need to copy the output into a file and attach it here since it will be a lot of logs.

This issue has been open for a while now, as we haven't had any new information provided. hence closing this issue.


How to Load and Enable mergerfs-tools?

Thanks for a very useful method for setting up a storage pool with mergerfs.
I've been able to make sense of all this for a Centos 7 setup. Not bad considering my limited skills with Linux. The mergerfs-tools are a logical next step but it's not clear to me how to install them, hence the title on this one.
Apologies if I missed something basic in the readme, wiki or the open & closed issues.

Use the makefile or just copy the file to your bin directory of choice.

Thanks for the assistance but your advice is based on assumed knowledge that I don't have. I'm after the dummies guide, please. I've searched the web and I haven't been able to find a step by step method that makes sense to me.
This piece is generally informative:
Make_(software)
but of limited use (see below).
This piece is a bit more focussed:
Re: How to install MAKEFILE Post 2
but doesn't seem to line up with the offerings available here, so again it doesn't deliver for me.
I've tried the following:

Open terminal to the user home directory as per default
$ git clone https://github.com/trapexit/mergerfs-tools
$ cd ~/mergerfs-tools
$ sudo make -f Makefile
.. gives ..
‘src/mergerfs.fsck’ -> ‘/usr/local/bin/mergerfs.fsck’ ‘src/mergerfs.mktrash’ -> ‘/usr/local/bin/mergerfs.mktrash’ ‘src/mergerfs.dedup’ -> ‘/usr/local/bin/mergerfs.dedup’ ‘src/mergerfs.ctl’ -> ‘/usr/local/bin/mergerfs.ctl’ ‘src/mergerfs.balance’ -> ‘/usr/local/bin/mergerfs.balance’
$ $ mergerfs.ctl -h
.. gives ..
/usr/bin/env: python3: No such file or directory

I'm trying to self help but I'm constrained by my lack of knowledge.
Thanks in advance.

Install Python 3


Write a JavaScript program to print reverse of a string by recursion

Description
Write a JavaScript program to print reverse of a string by recursion
Reverse of a string is the string with characters in reverse order. For example, reverse of "GeeksQuiz" is "ziuQskeeG".
Input  : abcd
Output : dcba

How to contribute

Save the solution in PrintReverseOfAStringByRecursion.js file
Add PrintReverseOfAStringByRecursion.js file in print-reverse-of-a-string-by-recursion folder

can i work on this?

!assign


Add a Dapp Frontend as the last step of Getting Started Guide

What problem does your feature solve?
Can't find where is adding a dapp frontend to increment contract
What would you like to see?
Move https://developers.stellar.org/docs/build/apps/dapp-frontend to step 5 of Getting Started guide

Hi! So I will explain the reasoning here. When we split the Build tab, we split it between Smart Contracts and Applications. We felt that building a dapp frontend fit better in the Applications section for a user looking for that section specifically.
We do have an Up Next link and pagination that directs users to the dapp frontend guide following the Getting Started section (see screenshot below) to make navigation easier.

Have there been many issues navigating this? I don't think either @ElliotFriend or I are super married to either flow but this made the most sense to us when splitting the content.

Anecdotally, at the hackathon I noticed more than one person trying to show me "some part" of the astro template guide they tried to copy and were having trouble with. Pretty much each time, they went to the "getting started" sidebar container, clicked 4-deploy-increment, scrolled to the bottom, and clicked that "build a dapp frontend" Next button.
That suggests to me people may associate the dapp frontend page more with the "getting started" container, and less with the "build applications" container. If that seems to be the general "vibe" of folks, I'd say it makes more sense to put it back into the "getting started" container.


Issue with fresh "cargo make"

Hi!
I've tried to compile the example code on Windows 11 ARM, but it fails with the following error message:
As user:
**********************************************************************
** Enterprise Windows Driver Kit (WDK) build environment
** Version ni_release_svc_prod1.22621.2428
**********************************************************************
** Visual Studio 2022 Developer Command Prompt v17.1.5
** Copyright (c) 2022 Microsoft Corporation
**********************************************************************
C:\Users\x\repos\Windows-rust-driver-samples>cargo make
[cargo-make] INFO - cargo make 0.37.10
[cargo-make] INFO - Execute Command: "rust-script" "target\\_cargo_make_temp\\persisted_scripts\\D4060E7434B3779E78A683E8BA00D06A5D08BE8C95BC432359E22F06CB30EF1C.rs"
[cargo-make] INFO - Calling cargo metadata to extract project info
[cargo-make] INFO - Cargo metadata done
[cargo-make] INFO - Build File: Makefile.toml
[cargo-make] INFO - Task: default
[cargo-make] INFO - Profile: development
[cargo-make] INFO - Running Task: legacy-migration
[cargo-make] INFO - Running Task: workspace
C:\Users\x\repos\Windows-rust-driver-samples
[cargo-make][1] INFO - Execute Command: "rust-script" "C:\\Users\\x\\repos\\Windows-rust-driver-samples\\target\\_cargo_make_temp\\persisted_scripts\\D4060E7434B3779E78A683E8BA00D06A5D08BE8C95BC432359E22F06CB30EF1C.rs"
[cargo-make][1] INFO - Calling cargo metadata to extract project info
[cargo-make][1] INFO - Cargo metadata done
[cargo-make][1] INFO - Project: echoapp
[cargo-make][1] INFO - Build File: Makefile.toml
[cargo-make][1] INFO - Task: default
[cargo-make][1] INFO - Profile: development
[cargo-make][1] INFO - Skipping Task: legacy-migration
[cargo-make][1] INFO - Running Task: build
[cargo-make][1] INFO - Execute Command: "cargo" "build"
    Finished dev [unoptimized + debuginfo] target(s) in 0.02s
echoapp does not contain a package.metadata.wdk section in its manifest. Skipping package-driver task.
[cargo-make][1] INFO - Skipping Task: package-driver-flow
[cargo-make][1] INFO - Running Task: default
[cargo-make][1] INFO - Build Done in 1.14 seconds.
[cargo-make][1] INFO - Execute Command: "rust-script" "C:\\Users\\x\\repos\\Windows-rust-driver-samples\\target\\_cargo_make_temp\\persisted_scripts\\D4060E7434B3779E78A683E8BA00D06A5D08BE8C95BC432359E22F06CB30EF1C.rs"
[cargo-make][1] INFO - Calling cargo metadata to extract project info
[cargo-make][1] INFO - Cargo metadata done
[cargo-make][1] INFO - Project: echo-2
[cargo-make][1] INFO - Build File: Makefile.toml
[cargo-make][1] INFO - Task: default
[cargo-make][1] INFO - Profile: development
[cargo-make][1] INFO - Skipping Task: legacy-migration
[cargo-make][1] INFO - Running Task: build
[cargo-make][1] INFO - Execute Command: "cargo" "build"
   Compiling wdk-sys v0.2.0
memory allocation of<PHONE_NUMBER>8 bytes failed132: wdk-sys(build)
[cargo-make][1] ERROR - Error while executing command, exit code: -1073740791
[cargo-make][1] WARN - Build Failed.
[cargo-make] ERROR - Error while running duckscript: Source: Unknown Line: 8 - Error while executing command, exit code: 1
[cargo-make] WARN - Build Failed.

I guessed that A required privilege is not held by the client. may mean that administrator rights are needed, to I re-ran it in an admin prompt.
As admin user:
**********************************************************************
** Enterprise Windows Driver Kit (WDK) build environment
** Version ni_release_svc_prod1.22621.2428
**********************************************************************
** Visual Studio 2022 Developer Command Prompt v17.1.5
** Copyright (c) 2022 Microsoft Corporation
**********************************************************************

C:\Users\x\repos\Windows-rust-driver-samples>cargo make
[cargo-make] INFO - cargo make 0.37.10
[cargo-make] INFO - Execute Command: "rust-script" "target\\_cargo_make_temp\\persisted_scripts\\D4060E7434B3779E78A683E8BA00D06A5D08BE8C95BC432359E22F06CB30EF1C.rs"
[cargo-make] INFO - Calling cargo metadata to extract project info
[cargo-make] INFO - Cargo metadata done
[cargo-make] INFO - Build File: Makefile.toml
[cargo-make] INFO - Task: default
[cargo-make] INFO - Profile: development
[cargo-make] INFO - Running Task: legacy-migration
[cargo-make] INFO - Running Task: workspace
C:\Users\x\repos\Windows-rust-driver-samples
[cargo-make][1] INFO - Execute Command: "rust-script" "C:\\Users\\x\\repos\\Windows-rust-driver-samples\\target\\_cargo_make_temp\\persisted_scripts\\D4060E7434B3779E78A683E8BA00D06A5D08BE8C95BC432359E22F06CB30EF1C.rs"
[cargo-make][1] INFO - Calling cargo metadata to extract project info
[cargo-make][1] INFO - Cargo metadata done
[cargo-make][1] INFO - Project: echoapp
[cargo-make][1] INFO - Build File: Makefile.toml
[cargo-make][1] INFO - Task: default
[cargo-make][1] INFO - Profile: development
[cargo-make][1] INFO - Skipping Task: legacy-migration
[cargo-make][1] INFO - Running Task: build
[cargo-make][1] INFO - Execute Command: "cargo" "build"
   Compiling proc-macro2 v1.0.78
   Compiling unicode-ident v1.0.12
   Compiling windows_aarch64_msvc v0.52.0
   Compiling once_cell v1.19.0
   Compiling windows-targets v0.52.0
   Compiling windows-sys v0.52.0
   Compiling quote v1.0.35
   Compiling syn v2.0.48
   Compiling uuid-macro-internal v1.7.0
   Compiling uuid v1.7.0
   Compiling echoapp v0.1.0 (C:\Users\x\repos\Windows-rust-driver-samples\general\echo\kmdf\exe)
    Finished dev [unoptimized + debuginfo] target(s) in 5.53s
echoapp does not contain a package.metadata.wdk section in its manifest. Skipping package-driver task.
[cargo-make][1] INFO - Skipping Task: package-driver-flow
[cargo-make][1] INFO - Running Task: default
[cargo-make][1] INFO - Build Done in 6.56 seconds.
[cargo-make][1] INFO - Execute Command: "rust-script" "C:\\Users\\x\\repos\\Windows-rust-driver-samples\\target\\_cargo_make_temp\\persisted_scripts\\D4060E7434B3779E78A683E8BA00D06A5D08BE8C95BC432359E22F06CB30EF1C.rs"
[cargo-make][1] INFO - Calling cargo metadata to extract project info
[cargo-make][1] INFO - Cargo metadata done
[cargo-make][1] INFO - Project: echo-2
[cargo-make][1] INFO - Build File: Makefile.toml
[cargo-make][1] INFO - Task: default
[cargo-make][1] INFO - Profile: development
[cargo-make][1] INFO - Skipping Task: legacy-migration
[cargo-make][1] INFO - Running Task: build
[cargo-make][1] INFO - Execute Command: "cargo" "build"
   Compiling proc-macro2 v1.0.78
   Compiling windows_aarch64_msvc v0.48.5
   Compiling windows_aarch64_msvc v0.52.0
   Compiling once_cell v1.19.0
   Compiling windows-targets v0.52.0
   Compiling windows-sys v0.52.0
   Compiling serde v1.0.196
   Compiling anstyle v1.0.6
...
   Compiling wdk-panic v0.2.0
   Compiling wdk v0.2.0
   Compiling echo-2 v0.1.0 (C:\Users\x\repos\Windows-rust-driver-samples\general\echo\kmdf\driver\DriverSync)
   Compiling wdk-sys v0.2.0
memory allocation of<PHONE_NUMBER>8 bytes failed132: wdk-sys(build)
[cargo-make][1] ERROR - Error while executing command, exit code: -1073740791
[cargo-make][1] WARN - Build Failed.
[cargo-make] ERROR - Error while running duckscript: Source: Unknown Line: 8 - Error while executing command, exit code: 1
[cargo-make] WARN - Build Failed.

Any ideas what went wrong?

To eliminate any possibility of the issue being in one of the cargo-make scripts, could you please just run cargo build and see if you still encounter this error.
   Compiling wdk-sys v0.2.0 memory allocation of<PHONE_NUMBER>8 bytes failed132: wdk-sys(build)
This seems to indicate that the build-script for wdk-sys failed to execute because it failed to allocate some block of memory. Are you running on a system with small amount of free RAM? although 26GB allocation seems like quite a bit...

Hi,
I ran jut cargo build with the following result:
C:\Users\x\repos\Windows-rust-driver-samples>cargo build
   Compiling proc-macro2 v1.0.78
   Compiling unicode-ident v1.0.12
   Compiling windows_aarch64_msvc v0.52.0
   Compiling windows_aarch64_msvc v0.48.5
   Compiling serde v1.0.196
...
   Compiling smallvec v1.13.1
   Compiling spin v0.5.2
   Compiling wdk-macros v0.2.0
   Compiling tracing-subscriber v0.3.18
   Compiling paste v1.0.14
   Compiling uuid-macro-internal v1.7.0
   Compiling uuid v1.7.0
   Compiling wdk v0.2.0
   Compiling echo-2 v0.1.0 (C:\Users\x\repos\Windows-rust-driver-samples\general\echo\kmdf\driver\DriverSync)
   Compiling wdk-sys v0.2.0
   Compiling wdk-panic v0.2.0
   Compiling echoapp v0.1.0 (C:\Users\x\repos\Windows-rust-driver-samples\general\echo\kmdf\exe)
error: failed to run custom build command for `wdk-sys v0.2.0`

Caused by:
  process didn't exit successfully: `C:\Users\x\repos\Windows-rust-driver-samples\target\debug\build\wdk-sys-4349d9c8a3407fd1\build-script-build` (exit code: 101)
  --- stdout
  cargo:rerun-if-env-changed=TARGET
  cargo:rerun-if-env-changed=BINDGEN_EXTRA_CLANG_ARGS_aarch64-pc-windows-msvc
  cargo:rerun-if-env-changed=BINDGEN_EXTRA_CLANG_ARGS_aarch64_pc_windows_msvc
  cargo:rerun-if-env-changed=BINDGEN_EXTRA_CLANG_ARGS
  cargo:rerun-if-changed=src/ntddk-input.h
  cargo:rerun-if-changed=src/wdf-input.h

  --- stderr
  C:\ewdk\Program Files\Windows Kits\10\Include\10.0.22621.0\shared\basetsd.h:318:33: warning: cast to smaller integer type 'unsigned long' from 'const void * __ptr32' [-Wvoid-pointer-to-int-cast]
  C:\ewdk\Program Files\Windows Kits\10\Include\10.0.22621.0\shared\basetsd.h:327:32: warning: cast to smaller integer type 'long' from 'const void * __ptr32' [-Wvoid-pointer-to-int-cast]
  C:\ewdk\Program Files\Windows Kits\10\Include\10.0.22621.0\shared\basetsd.h:336:12: warning: cast to 'void * __ptr32' from smaller integer type 'unsigned long' [-Wint-to-void-pointer-cast]
  C:\ewdk\Program Files\Windows Kits\10\Include\10.0.22621.0\km\crt\intrin.h:411:33: error: conflicting types for '__prefetch'
  C:\ewdk\Program Files\Windows Kits\10\Include\10.0.22621.0\km\crt\intrin.h:411:33: note: '__prefetch' is a builtin with type 'void (void *)'
  C:\ewdk\Program Files\Windows Kits\10\Include\wdf\kmdf\1.33\wdfdevice.h:1414:13: warning: enumeration value 'IdleCapsInvalid' not handled in switch [-Wswitch]
  clang diag: C:\ewdk\Program Files\Windows Kits\10\Include\10.0.22621.0\shared\basetsd.h:318:33: warning: cast to smaller integer type 'unsigned long' from 'const void * __ptr32' [-Wvoid-pointer-to-int-cast]
  clang diag: C:\ewdk\Program Files\Windows Kits\10\Include\10.0.22621.0\shared\basetsd.h:327:32: warning: cast to smaller integer type 'long' from 'const void * __ptr32' [-Wvoid-pointer-to-int-cast]
  clang diag: C:\ewdk\Program Files\Windows Kits\10\Include\10.0.22621.0\shared\basetsd.h:336:12: warning: cast to 'void * __ptr32' from smaller integer type 'unsigned long' [-Wint-to-void-pointer-cast]
  clang diag: C:\ewdk\Program Files\Windows Kits\10\Include\wdf\kmdf\1.33\wdfdevice.h:1414:13: warning: enumeration value 'IdleCapsInvalid' not handled in switch [-Wswitch]
  thread 'main' panicked at C:\Users\x\.cargo\registry\src\index.crates.io-6f17d22bba15001f\wdk-sys-0.2.0\build.rs:27:14:
  Bindings should succeed to generate: ClangDiagnostic("C:\\ewdk\\Program Files\\Windows Kits\\10\\Include\\10.0.22621.0\\km\\crt\\intrin.h:411:33: error: conflicting types for '__prefetch'\n")
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

A bit more information about the system I use:

Parallels VM on a Macbook Pro M2 (aarch64)
Windows 11 Pro Version 10.0.22631 Build 22631
4 Cores, 8 GB RAM

Is it possible that there is an out-of-memory due to RAM and VRAM exhausted?


Allow empty case without when

An empty case without when is currently a syntax error:
case "foo"
end # unexpected token: end (expecting when or else)

case "foo"
else # unexpected token: else (expecting when)
end

(the first syntax error even says expecting when or else, which would mean the second example should work. But it actually expects a when)
In regular code it certainly doesn't make much sense and is about as meaningful as if true. But it also doesn't do any harm to have a case without any when branch.
On the other hand, it comes in handy when the code for when branches is automatically generated (for example in a macro):
case foo
{% for case in cases %}
when {{ case.stringify }}
  # ...
{% end %}
end

If cases is empty, this macro will produce invalid code. It would require extra effort to handle this - and  it's ugly and hard to read. If more logic is involved, it might not be as simple as wrapping the entire case in {% unless cases.empty? %}.

example: https://github.com/crystal-lang/crystal/blob/master/src/json/serialization.cr#L170

Sounds good. If anyone wants to implement this, please go ahead.
I think case exp without anything inside it should be normalized to just exp. If it has an else (but no whens) then it should be normalized to exp; else_exp using an Expressions node.


Scrum 3678 Added facets for TET source fields

Added the new facets
Added the ability to "show more" to all TET facets
Removed extra calls to API to get initial facets
Added a new redux state variable to avoid sending search requests until all default facet values are set (only mod_in_corpus_or_needs_review per logged in user for now)

thanks @sweng66 !


Allow custom route names

Enhance the resource, has_one, and has_many keywords to take an options hash, which may include a custom route name to be used when drawing namespaces. (See also #9.)

This may be the solution to my problem, but is there no way to stop sinja from deciding where to put dashes in controller names (routes)? I have updated my test repo (which happily works now thanks to #12 being resolved), but it takes the TestKey and forces the route to be /test-key. What if I didn't want a dash, and instead simply wanted /testkey? Is this configurable? If not, maybe this is something I could help with / submit a PR for.

It draws the routes based on the symbol passed to the resource keyword. The symbol is dasherized and pluralized. Currently, there's no way to override this behavior, which is what this issue is intended to track. It's something I'm willing to prioritize if it helps an early adopter!
All that being said, your resource is named :keys, so the resource route should be /keys, not /test-keys. I think you may be looking at the links in the response payload. These are actually generated by JSONAPI::Serializers, and it is customizable! Just override the type method in your serializer class:
class TestKeySerializer
  include JSONAPI::Serializer

  attribute :name
  attribute :created_at

  def type
    'testkey'
  end
end

More information is available here.

Yes, that helped quite a bit for test scenario (I committed my update to reflect that), but unfortunately I think it needs to be done at a deeper level in sinja because it is failing in my application while including a related has_many resource. In my real use-case, I have GET /users/1?include=apikeys%2Cidentities (bringing along both apikeys and identities), but the response payload doesn't seem to rely on the underlying APIKey's type instance method when creating the has_many collection.
I am trying to build an app that relies on your library to implement the {json:api} spec, so I don't know if that makes me an early adopter, but I would certainly appreciate this customization being available. I can successfully POST to /apikeys and create a new key, but subsequently listing a user's keys produces unexpected results. Going from dashes to underscores in a consistent manner across libraries and possible languages is difficult, so being able to explicitly dictate the route would be helpful.

Nothing you've describe makes me think you need this feature. Do you have some time to chat with me on Gitter or somewhere else so I can help you work through this use case?


Bump to Fprime v3.4.0

This change:

Update ArduinoTime

For a future update:

Use ArduinoFw for FPRIME_PLATFORM
Use Generic for CMAKE_SYSTEM_NAME

@LeStarch I think this PR is ready for review. Would be nice to merge to devel (and main...?) ASAP since the current fprime-arduino is not compatible with fprime v3.4.0, and thus will break every new Arduino project using fprime v3.4.0.
Major Changes

New Arduino.ArduinoTime component since Svc.Time was replaced with an Interface.
Force RP2040 builds to use C++17 standard since the RP2040 package requires at least C++17

These changes were all inspired from the work/testing done throughout this issue.
After this is approved, I can bump these projects to the updated fprime-arduino and fprime v3.4.0:

fprime-arduino-deployment-cookiecutter
fprime-tutorial-arduino-blinker
fprime-baremetal-reference


minor misspelling fix

Hi @WangYuchenSJTU, thank you for pointing out that misspelling. I did leave a comment about the other grammar change that you made which isn't quite right. Please revert or update the change on line 292, and then I'll merge your fixes.

Thanks for making that update. #sign-off


TLSv1.3 0-RTT support

Enabling TLSv1.3 0-RTT support as mentioned in issue #4128.

Is there a way to check if ci is building with openssl 1.1.1?

@duke8253 It's not checked directly, but there're checks of OpenSSL v1.1.1 API on configure.
e.g. Jenkins Ubuntu
checking whether to enable TLSv1.3 ciphersuites configuration is supported... yes

For some jobs, downloading docker image might be another option.

Updated the docs and comments a little to reflect the conversation here.

[approve ci autest]

We will have two new seperated PRs that's related:

anti-replay support
alow configs in ssl_server_name.yaml

Changed to use MIOBuffer.

Given we are so close to branching ATS9 and the potential risks for folks planning to upgrade to ATS9 (see slack discussion in the trafficserver ASF channel on 5/29/19 for more context), an option could be to delay landing this until ATS9 is branched out and then add it to ATS9.1 or later with the discussed concerns addressed?

Changed the base to the new 0-rtt feature branch.

[approve ci]

[approve ci autest]

[approve ci autest]

@bryancall We're enforcing the return value to be 2 as described here (https://www.openssl.org/docs/man1.1.1/man3/SSL_CTX_set_tlsext_ticket_key_cb.html) so that every ticketed is only used once, this is the same anti-replay mechanism used in OpenSSL's internal cache.

[approve ci autest]

[approve ci autest]

[approve ci autest]

duke8253 h

Talking to @zwoop he suggested anding HEAD to the list of allowed methods.  We also talked about having a configuration option to enable and disable sending early data for URLs with parameters.

I've added the check for parameters and it appears to be working, will be adding a config option for that tomorrow.

[approve ci autest]

this feels like it should be a directive in sni.yaml

@zwoop we don't use sni.yaml for anything. Do you mean that it should be an optionally configurable there?

[approve ci]

This completely breaks TLS < v1.3. I think there's some places where you either have to make sure that
server_max_early_data = 0

for TLS < v1.3 (such that those code paths don't trigger, effectively forcing it turned off for TLS < v1.3). Or, consistently add the checks like
if (SSL_version(ssl) >= TLS1_3_VERSION ...)

before doing anything that's new and specific to TLS v1.3 / early data. The compile time checks is not enough, since that only checks for availability, you have to be careful not to use this new functionality for TLS versions < 1.3.

@masaori335

this feels like it should be a directive in sni.yaml

@zwoop we don't use sni.yaml for anything. Do you mean that it should be optionally configurable there?

Yes.

[approve ci]

This as been onDocs for two months already, anything else to do here before merging, @zwoop ?

Cherry-picked to v9.0.x branch.


Cross-seed based on data files

After #52 we'll have an abstraction over searchees. This means that we'll be able to easily create searchees out of entries in your data dirs.
Plan:

add a --data-dir command line option that can be used multiple times
concatenate those with the torrents found in the session dir
be careful to deduplicate these

This one will be slick. Nice one

I'm keeping an eye on this 👀

Just an FYI, this is how https://github.com/BC44/Cross-Seed-AutoDL operates

You do have torrent files inside your qbit config, just gotta hunt them down.

I found that folder. Luckily the docker container maps it to the outside so I got access too. Thanks. Hope that wasn't too off topic.

@byter09 No worries! I am gonna mark as off topic though.


Fix issue #102 Windows 7 issue with mincore.lib by using version.lib instead

Addressed issue #102 by removing mincore.lib and replacing with version.lib. I was able to successfully build and test on Windows 10. I don't have a Windows 7 machine or VM available at the moment. It would be good if someone could verify the change on Windows 7 as well.

Yup, I'm on it.

Is that really need to mention the lib files in project file?Visual Studio automatically get those.

@Biswa96 - At build time the linker uses the project settings to determine which libs to use. A missing lib will cause linking to fail, since the linker won't be able to locate functions that are implemented elsewhere.  In this particular case, removing version.lib results in these errors:
LNK2019	unresolved external symbol _GetFileVersionInfoW@16 ...
LNK2019	unresolved external symbol _VerQueryValueW@16 ...
That said, if you have a suggestion for automating this somehow in Visual Studio I'd like to hear more.

Oh, I forgot those Windows APIs. Sorry, for misleading...


register roles for each authorized address

each user must have a role in order to access the functions which are not public. the role is determined by the address.
Each address should get a list of roles it can use

Define modifiers for each role


Read OB & FC data ?

Hi guys .
i want to read data on OB1 AND FC2,3,4......
Can anyone help?
THANKS.

Have a look at the Directory / Block orientated functions. Please keep in mind that these functions are not available for the new 1200/1500 series. See the target compatibility in the table here .

thanks


Add columns to Advanced Storage view

group latency
allocation units

Should be done after there is storage type toggle and viewer/groups is used: #1156, #1160

Display Latency in format 50ms, use LatencyPutTabletLog


Rewrite README.md

Make it a bit more simple, leave only brief information for each section giving a link for more information.

Installation
Usage
Contributing
Website
Documentation
License

95b9f8b14382cebbf31c690986b4b60a6f5b00cb


Do not fail on setTarget(null)

In our React component we setTarget(null) on unmount of the component, however this causes an error in the background logic of ol-mapbox-style. This change protects the code against having no map target anymore.

thanks for the review @ahocevar


Update configuration.md

Fix typo (bindling)

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Hi - I'm not sure why the check above failed, gremlins? Thanks, Mike

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Prevent creating a docs PR if there's nothing to commit

Fixes bug in workflow where the workflow fails because it's trying to create a PR yet there are no new changes

@SilasKenneth kindly review.
Similar change to one you reviewed earlier on the v1 repo


Please consider adding to electron's showcase.

This looks good, but it seems you accidentally overwrote this project's README. Please undo that change and I'll ship it.

I've reverted the change. Thanks.


Create a proportion of examples without PV data, outside the UK

We currently only have PV data for the UK. We will at some point want to get PV data for elsewhere but, in the meantime, we'll need nowcasting_dataset to optionally output examples from outside the UK (to train the "image prediction" part of the model on the entire geospatial extent of the satellite imagery).
Maybe we should create two sets of batches on disk: one set which always has PV data (and is over the UK), and another set which is always from outside the UK (and doesn't have PV). Then the ML training script can mix-and-match examples on the fly to vary the ML training curriculum. To keep each batch ballanced, the ML training script will need to load at least two batches at once from disk (one with PV data, the other without) and create a single batch with a mixture of examples.

Not sure this is essential. For WP1 there are some GSP where there are very few / No PV systems i.e Scotland

@jacobbieker in order to train your models in SatFlow, do you think it's essential for the dataset to include training examples from outside the UK? (these examples wouldn't have any PV data yet...)

Its probably not essential, and for a model that will primarily be focused on the UK for now anyway, it probably doesn't matter as much!

Ill remove this from the NG project. Just to keep things really high priority in there


Please document release process

Please create a file RELEASING.md that describes the process to be followed when producing a release of this project.  This aids current and future project maintainers in following a consistent process.

As this project is now deprecated, CommerceHub will not be contributing further on this ticket.  Please feel free to fork the project and work on it further yourself under the terms of the project license if desired.


A compilable version of np.repeat.

A new keyword argument has been added to np.repeat, total_repeat_length, that can optionally be supplied to make np.repeat jit-able.

Hey,
Thanks for your comments!
So there are tests further down for multi-arrays, and I've added the compilation check to these when repeats is passed. Let me know if that covers your point.
In terms of the convenience function, in my understanding lax_numpy is exposed through the init of jax/numpy. To access the private method from the test, I'd either have to expose it through init or import lax_numpy directly. These seem a bit overkill, but happy to do so if you think it's worth it.

Since the "compute total repeats" operation is nontrivial and now duplicated in two places, I think at the very least it should be factored out into a helper function within the test file.

Looks great, thanks!


entry with nonlimber gg only, ~7.3 sec

nonlimber gg only.
Nonlimber-limber difference in gs and ss don't contribute to S/N much.
Current setting: nonlimber gg for \ell<200, limber gg for \ell>200; This can change.
FFT settings can also be changed for better accuracy or higher efficiency.
Current S/N difference is ~3, where gg part contribute to ~2.7. I'm not sure if it makes sense.
Auto-gg power work mostly well. Some very low-ell values are probably not behaving well in the benchmark version.
Some cross-gg power may need further check - however they shouldn't have contributed much to S/N. More over, those non-overlapping cross-bin gg power is zero in Limber, and tiny in nonlimber. I suspect they would be dominated by lens magnification in reality - which can use Limber.

Note that the use of python fftlog in this code is really suboptimal comparing to using the C version FFTW, especially for computing at many \ells with the same integrand. The C implementation would reuse the "FFTW plan", and reduce the total time by at least a factor of few.

Note that the use of python fftlog in this code is really suboptimal comparing to using the C version FFTW, especially for computing at many \ells with the same integrand. The C implementation would reuse the "FFTW plan", and reduce the total time by at least a factor of few.


💥Feat:UI improvement of Profile page

Related Issue
“None”
Fixes:  #4219
Description
The current UI of the Profile page is kinda boring. Here is the current Profile page Proposed overview.
Type of PR

[x] Feature enhancement

Screenshots / videos
https://github.com/user-attachments/assets/27d12e29-49a1-4680-b3c1-298f1fdf0a1f
Checklist:

[x] I have made this change from my own.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@abhi03ruchi please review it and merge it under gssoc and level 3 tag.

good work @akash70629 Keep contributing


BTE validation Error during mini-hackathon

This query generated an error during mini-hack on May 20, 2021.
Chunlei, please take a look.
Query:
qg = {
"nodes": {
"n0": {
"categories": [
"biolink:ChemicalSubstance"
],
"name": "Chemical Substance"
},
"n1": {
"name": "EGFR",
"ids": ["NCBIGene:1956"]
}
},
"edges": {
"e0": {
"subject": "n0",
"object": "n1",
"predicates": [
"biolink:decreases_abundance_of",
"biolink:decreases_activity_of",
"biolink:decreases_expression_of",
"biolink:decreases_synthesis_of",
"biolink:increases_degradation_of",
"biolink:disrupts",
"biolink:negatively_regulates_entity_to_entity"
]
}
}
}
Error message:
https://arax.ncats.io/?source=ARS&id=90080f07-808c-47db-a042-21a9ce6f4281

@newgene , I wanted follow up on this issue as we have the mini-hackathon this week and we will be revisiting the queries ran at the last round. Were you able to look into this error? Thanks.

@tursynay yes, we have identified the issue and are working on the fix.

@newgene , is there an issue that we can check on that shows the progress on the bug? Thank you.

sorry, should have updated here as well. We are tracking this issue here internally:
https://github.com/biothings/BioThings_Explorer_TRAPI/issues/171
and also this related one reported by Chris B:
https://github.com/biothings/BioThings_Explorer_TRAPI/issues/202
The fixes to these reported validation errors have already be deployed at https://api.bte.ncats.io/v1/query now.
We will close these issues (and this one too) after some tests.

The fixes have been deployed and look to be working. I think this issue can be closed.

That's great. Thank you @colleenXu and @newgene.


The instructions on how to deploy business network need correction

These instructions on how to deploy a business network need correction:
https://github.com/hyperledger/composer-sample-applications/blob/master/packages/digitalproperty-app/README.md
For hlfv1 "npm run deployNetwork" fails.
/root/.npm/_logs/2017-07-20T21_46_01_629Z-debug.log" indicates that "composer network deploy ***" command failed
Manual execution of
composer network deploy -p hlfv1 --archiveFile digitalPropertyNetwork.bna  --enrollId admin --enrollSecret adminpw
leads to
[InvalidArgumentException]
Command "network" is not defined.

That looks like composer-cli was not installed but something else that is called composer

Closing issue as appears to be resolved.


Publish code coverage task not able to load coverage data from cobertura.xml

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: Publish Code Coverage
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks/PublishCodeCoverageResultsV1
Environment

Server - Azure Pipelines or TFS on-premises? Azure Pipelines

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:
Account: Office
Team Project: OneNote

Agent - Hosted or Private: Private

If using private agent, provide the OS of the machine running the agent and the agent version: Linux

Issue Description
Not sure if this is a dupe of Issue 9865 or not...
Publish code coverage task is not reading the cobertura.xml file I have generated. I am generating code coverage for my Scala project using scoverage in a cobertura-compatible format.
The task uploads the file, but issues the warning:
##[warning]No coverage data found. Check the build errors/warnings for more details.
Here is the attached cobertura.xml file (with format renamed):
cobertura.xml.txt
From the xml itself, it looks like the xml is using the expected http://cobertura.sourceforge.net/xml/coverage-04.dtd schema.
Task logs
And here are the debug logs from the specific Publish Coverage task:
log_14_1978394.zip

Try setting a build variable called disable.coverage.autogenerate to true. They introduced some breaking changes to the task in a non-backwards compatible way.
More info here: #10094, #9704 & #9962.

Works now, thank you!


ISPN-6043 Log the stacktrace if a listener fails

Having only the exception message in the log is often not enough, i.e.
ERROR [org.infinispan.notifications.cachemanagerlistener.CacheManagerNotifierImpl] (sc-main) org.infinispan.commons.CacheListenerException: ISPN000280: Caught exception [java.lang.NullPointerException] while invoking method [public synchronized void org.radargun.service.InfinispanClustered.viewChanged(org.infinispan.notifications.cachemanagerlistener.event.ViewChangedEvent)] on listener instance: org.radargun.service.InfinispanClustered@78338b0a

Updated (feel free to change the log message).

Seems fine to me. @danberindei ?

Ok, integrated. Thanks Jakub!

@jmarkos JIRA reference seems wrong, though :)


igluctl: send data application/json content-type

Right now static push uses form-encoded data. We need to come up with deprecation plan and remove it entirely https://github.com/snowplow/iglu/issues/421
E.g,

igluctl 0.6.0 and Server =0.5.0 (R11): check Server version, use application/json
igluctl 0.6.0 and Server <0.5.0: check Server version, use form-encoded data, print deprecation message
igluctl 0.7.0 and Server <0.5.0: fail with meaningful message ("probably you're using outdated Server?")

Migrated to https://github.com/snowplow-incubator/igluctl/issues/15


What does Mat::clone do?

I wanted to ask what a clone on the Mat does. Does it:

Create a new Mat with a copy of the data
Create a reference to the original (like the default C++ behaviour)

If it is the second, what would the easiest way be to create a new mat with a copy of the original data - would you actually have to create a new mat and use the copy_to method?
Thanks

Well, in this case the docs are pretty self-explanantory: Mat::clone() and Mat::try_clone() so it's a first option from your list.

Ok, thanks! I just wanted to make sure.


Chart bug is back

I've bought NFT and this just happend, but I guess I can leave it as backlog once we will have new #1072

we have meanwhile new charts


Show places that need performance improvement

Also when rendering tons of signals are interacting with each other (filters, list of entities, entities, ...)
Also the indexeddb querying is horribly inefficient. I think it would be great if the list of ids would be replaced by a table per entity instead and then you can simply query that table. also filtering could be implemented at the database level

@mohe2015 so I get this right, you wanted to profile how much of a performance impact is caused by using REScalas updating

@mohe2015 so I get this right, you wanted to profile how much of a performance impact is caused by using REScalas updating?

Yes and also the popper popovers. If you removed both like in this PR and then did a benchmark using e.g. the chrome browser performance tools you could see how it looked way better.


socket shutdown vs close

the luasocket connector uses shutdown; https://github.com/xHasKx/luamqtt/blob/eacac48e31aa1a58974ef05d76687d8554a33207/mqtt/luasocket.lua#L22
the luasec based luasocket_ssl connector however uses close; https://github.com/xHasKx/luamqtt/blob/eacac48e31aa1a58974ef05d76687d8554a33207/mqtt/luasocket_ssl.lua#L45
is there a reason behind these asymmetrical calls?

When conn.sock is a luasec connection instance - it does not have the :shutdown() method AFAIK

https://github.com/brunoos/luasec/wiki/LuaSec-1.0.x#conn_close

ah, but the method is available. Sockets are just transparent wrappers around luasocket sockets, so it should have it.
Just isn't documented.

oops... you're right. looked at the close method, but indeed the shutdown is missing. My bad.

see https://github.com/brunoos/luasec/issues/181

Please note that "close" and "shutdown" can do similar things for plain socket, and completely different actions for SSL sockets. AFAIK, "shutdown" means "do SYN/FIN packets handshake to indicate connection closing", and "close" is just hung up socket on client side. But it's possible that for SSL socket the "close" method is doing the SSL+network handshake, leading to collision with "shutdown"

for now I just updated all to 'close', not perfect but will do I guess.


[Error Message] When session expired while updating SP give error message as null

Steps to reproduce:

Create a SP.
Update SP with some values and wait few minutes until session gonna expire.
Click "Update" button.

Expected Outcome:
It should be giving meaningful error.
Actual Outcome:
Giving following error [1].

original issue reported in [1]
[1] https://wso2.org/jira/browse/IDENTITY-7045

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


How to by pass TS2300: Duplicate identifier

Hello i have a wsdl having keys with maxOccursxs:sequence<xs:choice minOccurs="0" maxOccurs="unbounded">. I cant write my json body (for exemple { name:'john',name='paul'} beacuse the angular 6 doesn't allow it yet. Is there a workaround to write such json body ? Best Regards Cgristophe Maitrepierre

Hello @cmaitrepierre sorry for delay! It's not clear to me what you're trying to do. Sequences should be translated as arrays of objects.

Hello @lula i have followed your advise and use an array of objects [{name:'john'},{name:'paul'}] and it is working fine. This allows to continue my project related to Radio Policy Management with angular and Gemalto AOTA (Advanced Over The Air) Platform. Best Regards, Christophe

Hello @cmaitrepierre! Glad to hear your issue is solved!! 🎉


Subscription on netty http not working

I have a @SubscriptionMapping datafetcher which returns a flux. The data fetcher returns the flux to the client reactively when I run spring graphql on a websocket server and I'm happy with its performance. However, when I run spring graphql on netty webflux, the same datafetcher just returns:
{"data":{"upstreamPublisher":{"scanAvailable": true, "prefetch": -1}}}
and then terminates...
The only other reference to this that I can find is https://github.com/spring-projects/spring-boot/issues/30052
This made against the spring projects repo by that developer and was closed for not providing enough information.
Please let me know if you need more from me. Thank you!

Could you provide a sample application showing the issue? Ideally, an application created on https://start.spring.io, that we can git clone and run.

@bclozel You can clone from here: https://github.com/calebcodesgud/sgql-streaming-issue/tree/main
Forgive the lack of a virtual environment please, I've included my jdk and maven version in the read me.
Please try running the application from local then run each of the unit tests. I was able to reproduce the same behavior with this.

@calebcodesgud I think this was mostly an issue with the testing setup.
I've sent a PR to your sample repository, with various commits explaining the changes. See https://github.com/calebcodesgud/sgql-streaming-issue/pull/1
I'm closing this as the feature works as expected. Thanks!

Changing the test implementation here doesn't really solve the issue as I see the same problem when the application is deployed and called via webclient... should I be using a different client to execute the http subscription?

We're shipping a dedicated client for that case. In your original project you were using a mix of graphql-kickstart and hand-made HTTP requests.
You can check that the server side behaves properly by calling it from the GraphiQL UI.

Thanks @bclozel ! is adding http subscription support in the roadmap at all?

@calebcodesgud Not right now, see this comment for more context.

Same here, but I think we're trying to do get/post htpp request. It should be performed wss from what I understand.


error in modulated_deformable_col2im_cuda: no kernel image is available for execution on the device

When  I run DCN on EDVR, I meet the problem as follows:

same problem, any idea?

Same problem, have you solved this problem?
Run DCN of EDVR on TITAN RTX has no issues, but on Pascal P100, this error occurred.

How can u solve this problem?? It goes well on nvidia 4070Ti but has this problem on V100


Add DPulls GitHub Action from Marketplace

It's free for open source projects according to their pricing page.
https://github.com/marketplace/dpulls
https://www.dpulls.com/
Why though? The old dependency declaration bot that we are currently using in Cacti has broken down and haven't  been maintained in years so we need a new solution that does pretty much the same thing as the old robot, but with it also being maintained.

@petermetz added

@ryjones I was able to set it as a required check in the Cacti upstream repo, thank you!


Modulo operator is not supported

Using modulo operator % in the Scss code:
$my-variable: 5 % 2;

causes this error:
error Please check validity of the block starting from line #1  Fatal

Hi @VladimirPlokhov This is a parse issue with Gonzales-pe our AST. It's fixed in their latest release though which we will be moving up to in 1.6 which is just waiting on them to fix some windows installation issues.

@VladimirPlokhov So this is working for .scss format in sass-lint 1.6 but gonzales still throws a parse error in .sass format, we'll look to fix that soon. I'll create a new issue specifically for .sass format.
#636
https://github.com/DanPurdy/gonzales-pe/issues/6
https://github.com/tonyganch/gonzales-pe/issues/165


fix/multi-line-cohort

:tada: This PR is included in version 1.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Regrowth and Survive hitting mob causes Crash

Forge Version : 40.2.0
Minecraft Version: 1.18.2
WorldSeed: Any
Describe the bug
When hitting a mob with Regrowth and Survive installed game crashes.
To Reproduce
Steps to reproduce the behavior:

Create new world
Find a mob
Punch it
See error

Expected behavior
When hitting a mob the game won't crash.

crash-2023-05-24_23.33.02-client.txt

Okay, I've looked into this and it doesn't appear to be related to regrowth.  At least not directly in any way I can see.
Regrowth doesn't use attributes and isn't in the crash log.
All regrowth does it intercept the mob movement event, and check the block they are standing on and then possibly change the state of that block to a new block.
Going to close this issue for now.   Sorry for the delay, I had no power 7 days in May from a Derecho wind storm and then no power 7 days in July from a Hurricane.    Really beat me up there for a while.
Cheers,
Two Guys,
MacTso


[Concurrency] Drop the cloned code for Embedded around 'any Actor'

Now that any Actor is usable in Embedded Swift, we can drop all this ifdef'd code.

@swift-ci please test

@swift-ci please test