Add option to only lint changed files

This massively reduces runtime when you have a repository with half a thousand markdown files and/or cannot retroactively lint existing invalid markdown files.
Changed files can be determined as follows:
const fetched = await octokit.paginate(octokit.rest.pulls.listFiles, {
    owner: pull_request?.base.repo.owner.login,
    repo: pull_request?.base.repo.name,
    pull_number: pull_request?.number as number,
});

If you want to exclude files/paths from linting, that is already supported.
How long does a full linting pass take? Can you link to one of the runs?

As it turns out, it doesn't take too long (about 10 seconds). The problem is that due to the way the repository works, one can't just update all the files to use the new format, and so the linter should only care about the files that were changed, otherwise the CI would always fail with about 1.5k violations.

Saw this in the change notes of the action and realised there may be an issue with this approach.
As the version of this action (and indeed many other actions) change, new functionality is added and old functionality is improved.
This means that even files that were already linted, may now show new issues, i.e. newly found issues in existing files.
When you are scanning all files, such issues will show up every time a workflow runs.
But applying linting only on changed files means that those files never be linted again (unless subsequently altered of course) so will only be linted up to the version of this action which ran on them.
Which means that the lint status of even new files can gradually degrade over time.

This is intentional, as in our case we prefer degradation over time to forced updates.


WCIII reforged file format

I've been trying to parse files created by WCIII reforged and it's not compatible.  Can we collaborate on improving the parser?  I've never parsed binary formats before, but I can work through it.
Parsing the header fails really quickly with an offset error, which I assume means the byte counts are off compared to the original file format.
Thoughts?

Can you provide a sample file that we can use a common base? I did not take a look at the Reforged replay files yet, but will do now that you raised this issue and report back on you.

Here is a current one.
https://twitter.com/JonnyRoot_/status/1191783735475068928?s=20

Here is a new format:
https://www.dropbox.com/s/wl4q5hf72q3qubr/LastReplay.w3g?dl=0

https://github.com/PBug90/w3gjs/tree/feat-reforged-replays
This branch fixes the offset error. It looks like they changed the block size and compression algorithm in some way, so the compressed blocks can not be decompressed with the existing method.
I added two sample tests for reforged replays that can be run in isolation via jest:
https://github.com/PBug90/w3gjs/blob/feat-reforged-replays/test/replays.test.ts#L190
https://github.com/PBug90/w3gjs/blob/feat-reforged-replays/test/replays.test.ts#L194
I will further investigate. If you have an idea please comment aswell.

After futher investigation it looks like they added two more bytes between the blockDecompressedSize indicator and the actual compressed data. I update the branch with a working version that can parse the replay except its actions.

Awesome! I'll pull it today, I was super busy yesterday and didn't get a chance to look at the project yet.

Looks like they added a new action with id 0x7b which is always 18 bytes long (including the action identifier).
I updated the branch again to include this action and the parsing errors seem to have gone. I also reenabled the timeslot-event-emitting in the base parser so the parser output works as expected for reforged replays.
Still have to determine what the new action does though.

I've been trying to figure out what the action is, but I'm super slow at debugging these.  Is it the case for block type 34?

Can you elaborate on what you mean with your question?

I updated the investigation branch and added changed the action 0x7b parsing behaviour to create a buffer instead of just skipping it. This allows filtering for 0x7b actions in a .on('actionblock') event listener callback.
I updated one of the tests to console.log if the parser encounters a 0x7b block in the first reforged replay so that it logs 0x7b as a hex string on the console.

Awesome!  I'll pull that up today.  I was looking at a series of actions that have different "types" and there was a mix of 31 type id and 34 type id.  31 was handled, but 34 was not.

Heads up, the latest Reforged build added a completely new set of bytes to the GameMetaData
https://github.com/iggyfisk/w3gjs/commit/62d177a4a0c7725f8c30a128ef52993267980ba9#diff-bc9969e63009b789c7d3e540727ba6fb
Between the playerList and playerSlotRecords there's a bunch of mystery bytes, followed by ANOTHER player listing. This one includes the full battle.net tag (the names in playerList have a digit missing), plus some... stuff.
We're pretty sure the optional bytes (extraLength and extra in my quick fix version) are the players' portraits, with extraLength == 0 for nothing selected. It would make sense if selected skins are also in there, but we'll get back to you after some experiments.
Mad greetz from https://highper.ch R&D

Is the replay file provided in the diff one that should be correctly parsed with your adjustments?

I opened a PR but your changes do not seem to parse the file correctly.

That PR is missing the 0x7B action and the blockSize changes in the header, that you have in feat-reforged-replays. Which will break old replays. (Also Reforged is at 1.32 now)
Anyway we also figured out that in between gameName and encodedString there's the string hunter2 for private custom games, at least in Reforged. This broke private game parsing for us until https://github.com/iggyfisk/w3gjs/commit/7f4f495e2272d27b7cc8c207572ea10ef0ce5d40#diff-bc9969e63009b789c7d3e540727ba6fb
It's not unlikely that it's the same for private games in 1.31 but none of us have it installed anymore.
Wouldn't surprise me if they make more breaking changes before the "release" in a few weeks. Which is going to be a very broken "release" anyway so who knows how much will happen after that. You could hold off on trying to merge stuff until then, we'll keep testing every patch and reverse engineer changes as they happen.

You are right, I merged feat-reforged-replays and it parses the replay successfully now.
Waiting for release seems to be the reasonable way to go. Until then we should use feat-reforged-replays for all reforged-related changes made to the parser and implement decision making logic on how to handle vanilla and reforged replays in the future. I will merge feat-reforged-metadata into feat-reforged-replays and open another PR since that branch name seems to be more reasonable anyway.

Closing this issue as version 2.0 is close to release and version 1 can parse reforged replays properly at the moment. Feel free to open a new issue if you run into problems again.


Placeholder on vuestic-simple-select

Add placeholder, that store chosen value, when display value is empty.
When dropdown closes, value must return to input.

That's hard =.=.


refactor: use nanoid

This will prevent this happened
Error: crypto.getRandomValues() not supported. See https://github.com/uuidjs/uuid#getrandomvalues-not-supported
    at rng ($:/plugins/linonetwo/workflow-engine/startup/engine.js:16538:13)
    at v4 ($:/plugins/linonetwo/workflow-engine/startup/engine.js:16909:52)
    at _Execution.getUUID ($:/plugins/linonetwo/workflow-engine/startup/engine.js:17609:16)
    at new _Execution ($:/plugins/linonetwo/workflow-engine/startup/engine.js:17157:35)
    at Engine.<anonymous> ($:/plugins/linonetwo/workflow-engine/startup/engine.js:8456:29)

Also, nanoid is smaller and faster, and in our use case, it is as safe as uuid.
I also delete some unused imports.

Sad, this also requires crypto.

Use @lukeed/uuid , this works, and is faster https://github.com/lukeed/uuid?tab=readme-ov-file#benchmarks


Commented out commons/fonts.css styles

Description
There's a "not used" comment here for some font variables:
https://github.com/adobe/spectrum-css/blob/master/components/commons/fonts.css#L16
Expected behavior
Ideally this would at least get stripped from output. If the code shouldn't be outright deleted, a comment about when it might get used would be helpful.
Additional context
Noticed this when updating our script to pull this in for spectrum-web-components.

The comments here are not present in build output in dist/, so we'll go ahead and close this.


add github-readme-stats to made-in-india collection

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[x] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[x] Content (and my changes are in index.md)

Github Readme Stats by Anurag Hazra (@anuraghazra) is a very popular project not only because it has crossed over 20k stars but it is used in almost every alternate readme I visit. Definitely deserves to be in the list.
Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Thanks for contributing, @saurabhdaware!
@anuraghazra - Please confirm that you would want your project to be included in the Made in India collection :)

Thanks for contributing, @saurabhdaware!
@anuraghazra - Please confirm that you would want your project to be included in the Made in India collection :)

Yes! would love to see it added to the collection. 😃
Thanks @saurabhdaware for the PR.

README.md


how to fix ?

Code:
[50] $this->inventory->sendArmorContents($this);
[51]
[52] $this->server->getPluginManager()->callEvent($ev = new PlayerJoinEvent($this, TextFormat::YELLOW . $this->getName() . " joined the game"));
[53] if(strlen(trim($ev->getJoinMessage())) > 0){
[54] $this->server->broadcastMessage($ev->getJoinMessage());
[55] }
[56]
[57] $this->spawnToAll();
[58]
[59] if($this->server->getUpdater()->hasUpdate() and $this->hasPermission(Server::BROADCAST_CHANNEL_ADMINISTRATIVE)){
[60] $this->server->getUpdater()->showPlayerUpdate($this);
[61] }
[62] }
[63] }
[64] }
[65]
[66]
[67]
[68]
[69]

Please show the full crash dump
Or just post the base64 crash dump at the bottom.

@PEMapModder  I need how to fix this or tell me 1 what phar specter error I need it in two days to a need

ClearSky Crash Dump Fri Jan 29 06:12:05 AKST 2016
Error: Call to undefined method pocketmine\Server::getUpdater()
File: /Specter-master/src/specter/network/SpecterPlayer
Line: 59
Type: E_ERROR
THIS CRASH WAS CAUSED BY A PLUGIN
BAD PLUGIN: Specter v0.4
Code:
[50]             $this->inventory->sendArmorContents($this);
[51]
[52]             $this->server->getPluginManager()->callEvent($ev = new PlayerJoinEvent($this, TextFormat::YELLOW . $this->getName() . " joined the game"));
[53]             if(strlen(trim($ev->getJoinMessage())) > 0){
[54]                 $this->server->broadcastMessage($ev->getJoinMessage());
[55]             }
[56]
[57]             $this->spawnToAll();
[58]
[59]             if($this->server->getUpdater()->hasUpdate() and $this->hasPermission(Server::BROADCAST_CHANNEL_ADMINISTRATIVE)){
[60]                 $this->server->getUpdater()->showPlayerUpdate($this);
[61]             }
[62]         }
[63]     }
[64] }
[65]
[66]
[67]
[68]
[69]
Backtrace:
#0 /src/pocketmine/Server(0): pocketmine\Server->crashDump(boolean)
#1 /src/pocketmine/Player(1761): specter\network\SpecterPlayer->sendNextChunk(boolean)
#2 /src/pocketmine/Server(2516): pocketmine\Player->checkNetwork(boolean)
#3 /src/pocketmine/Server(2252): pocketmine\Server->tick(boolean)
#4 /src/pocketmine/Server(2130): pocketmine\Server->tickProcessor(boolean)
#5 /src/pocketmine/Server(1723): pocketmine\Server->start(boolean)
#6 /src/pocketmine/PocketMine(442): pocketmine\Server->__construct(boolean)
#7 (1): (boolean)
#8 (0): {main}(boolean)
PocketMine-MP version: 1.0dev #0 [Protocol 38; API 1.13.1]
Git commit:<PHONE_NUMBER>000000000000000000000000000000
uname -a: Windows NT WINDOWS2012-001 6.3 build 9200 (Windows Server 2012 R2 Standard Edition) i586
PHP Version: 5.6.6
Zend version: 2.6.0
OS : WINNT, win
Loaded plugins:
DevTools 1.10.0 by PocketMine Team for API(s) 1.3.1
essentialsTP 1.1.0 by  for API(s) 1.12.0
SignPortal 1.1.2 by 99leonchang for API(s) 1.3.1
Slapper 1.2.5fix0.13.1 by jojoe77777rBEdit for API(s) 1.12.0
SnowHalation 1.8-alpha by hm for API(s) 1.13.0, 2.0.0
SpecialBlock 1.3.4 by Primus for API(s) 1.0.0
WorldEditArt 1.0.0_beta_dev_#048 by PEMapModder for API(s) 1.0.0
WorldProtect 2.2.0 by aliuly for API(s) 1.10.0, 1.12.0
GrabBag 2.4.1 by aliuly for API(s) 1.10.0, 1.11.0, 1.12.0, 2.0.0
Specter 0.4 by Falkirks for API(s) 1.13.0

Please stop duplicating issues. If you want to use this plugin, use PocketMine. This plugin is highly hacky and can easily malfunction in unofficial versions of PocketMine.


Updated node stackName to the latest version.

The previous version doesn't exist (at least in US-EAST-1).

Thanks for catching this!


remove type coercion for physical ScalarFunction

Which issue does this PR close?

Closes #3731
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

The CI checks appear to be failing

The CI checks appear to be failing

Some errors about import in the test, I have fixed them


Lock skill tree until level 18

Skill points are only earned after the last tower is unlocked. It doesn't make sense for the skill tree to be accessible before the player earns the first skill point as it can be an overwhelming factor.

Resolved via commit 1c9f1a31f2b1f8758e9064d8d4c88894fdbeaab8.


New StreamProvider problem

Hello,
I'm using StreamProvider API from first day. I just notice new change in recent releases

Old API: I can download the file normally, then if I want to stream, just pick a file and it start play
New API: It's not possible. I must stop remove manager from current engine, then let StreamProvider create new manager.

Yes, this was a deliberate choice for the first release.
There's a little bit of complication around managing the lifespan of the stream and/or TorrentManager. I was able to avoid dealing with those corner cases by hiding the fact a TorrentManager is being used.
It's easy to restore the ability to create a stream from an active TorrentManager. The approach I'd probably take would be to retain the decision to only allow one stream at a time (for now), and then have the stream be disposed of automatically if/when the TorrentManager is stopped or if someone replaces the piece picker in the torrent manager. Also I'd have to change it so that disposing/closing the stream does not stop/unregister the torrent manager.
I reckon the API would be something like:
public async Task<StreamProvider> GetStreamProvider ()
{
    // 1) throw an exception if a StreamProvider was returned already and it hasn't been disposed
    // 2) Ensure the stream provider sets up the torrent manager without taking control of it's lifespan (i.e. it won#t call StartAsync/StopAsync etc)
    // 3) Ensure the streamprovider is disposed automatically when the torrentmanager is stopped by the user.

}

If you want to play around with the code and submit a PR


[Feature Request] Add support for biome tags in biomeFilters

I'm using Terralith alongside Ore Tweaker and realized that I was getting almost no vanilla oregen in my world.
While I did manage to resolve this by manually adding all the various Terralith biomes to the respective "OVERWORLD" whitelists of each file (and updating the "MOUNTAIN" and badlands ones to their specific values) it was a lengthy process and resulted in extremely large lists.
If biomes were instead filtered by tags, such as #minecraft:is_overworld this would not only allow for much shorter lists of modded biomes, but would also greatly increase "out of the box" support for other modded worldgen (provided it was tagged correctly).

For those who stumble across this:
Here is the list of a ton of biome ids that others can copy paste into their oretweaker whitelists.
biome list


access to modify topic 8

How can I modify the content in topic 8?
I would like to add to possible target treatments the following covering all the Linnaeus plant types:

Target treatments
•	https://tb.plazi.org/GgServer/srsStats/stats?outputFields=doc.uuid+doc.articleGbifId+bib.author+tax.name+matCit.verbatimMatCit+matCit.collectionCode+matCit.specimenCode+matCit.specimenHttpUri+matCit.accessionNumber&groupingFields=doc.uuid+doc.articleGbifId+bib.author+tax.name+matCit.verbatimMatCit+matCit.collectionCode+matCit.specimenCode+matCit.specimenHttpUri+matCit.accessionNumber&FP-bib.author=Jarvis%25&format=HTML or in JSON: https://tb.plazi.org/GgServer/srsStats/stats?outputFields=doc.uuid+doc.articleGbifId+bib.author+tax.name+matCit.verbatimMatCit+matCit.collectionCode+matCit.specimenCode+matCit.specimenHttpUri+matCit.accessionNumber&groupingFields=doc.uuid+doc.articleGbifId+bib.author+tax.name+matCit.verbatimMatCit+matCit.collectionCode+matCit.specimenCode+matCit.specimenHttpUri+matCit.accessionNumber&FP-bib.author=Jarvis%25&format=JSON

as well as

target treatments for collection = RMNH

HTML https://tb.plazi.org/GgServer/srsStats/stats?outputFields=doc.uuid+doc.articleUuid+tax.name+colls.code+matCit.verbatimMatCit+matCit.collectionCode+matCit.typeStatus&groupingFields=doc.uuid+doc.articleUuid+tax.name+colls.code+matCit.verbatimMatCit+matCit.collectionCode+matCit.typeStatus&FP-colls.code=RMNH&format=HTML
JSON https://tb.plazi.org/GgServer/srsStats/stats?outputFields=doc.uuid+doc.articleUuid+tax.name+colls.code+matCit.verbatimMatCit+matCit.collectionCode+matCit.typeStatus&groupingFields=doc.uuid+doc.articleUuid+tax.name+colls.code+matCit.verbatimMatCit+matCit.collectionCode+matCit.typeStatus&FP-colls.code=RMNH&format=JSON

as well

there are 480K material citations in GBIF [1]  of which 31.1.K have been detected to link to other records [2] . It’s all open on APIs though, so your folks could explore this. If they do, these are your records which cluster [3] and taking an ID from a record, you can see what it links to, such as this [4] showing it relates to the AMNH specimen. Someone could script up a report summarizing the relationships – it could even be a contribution to the hackathon topic next week. Tim Tobertson
[1] https://www.gbif.org/occurrence/search?publishing_org=7ce8aef0-9e92-11dc-8738-b8a03c50a862&advanced=1
[2] https://www.gbif.org/occurrence/search?publishing_org=7ce8aef0-9e92-11dc-8738-b8a03c50a862&advanced=1&is_in_cluster=true
[3] https://api.gbif.org/v1/occurrence/search?publishing_org=7ce8aef0-9e92-11dc-8738-b8a03c50a862&advanced=1&is_in_cluster=true
[4] https://api.gbif.org/v1/occurrence/3058638305/experimental/related

Thanks,
Tim

Can you put in a pull request for these changes Tim? If you already have, you can just close this issue.


Voxelization for 3D CNN

Hello, im trying to impelemnt a singlestage detector (perception and prediction) which is inspired from IntenNet and Fast and Furious.
These two approaches uses voxelization to create a 3D voxel grid from point cloud data and feed it to a 3D CNN.
My question is, can i get inspired by the voxelization process in voxelnet to do the same?
Thank you

Hello,
I don't know how the voxelization has to be done in these approaches, but I think i should be very similar.
Take a look at the process_pointcloud function in utils/preprocess.py.


Can't convert object to primitive value in IE11

Category

[x] Bug

Describe the bug
My project needs to be compatible with IE11 and Some packages I use depend on ES6 or ES7 , so I need to use polyfill.
Once I use @babel/polyfill  or @pnp/polyfill-ie11 in IE11 I get an error：Can't convert object to primitive value
Steps to reproduce

import "@pnp/polyfill-ie11"  or import "@babel/polyfill"
Run gulp serve
Open  SPO workbench Page in IE11 and add webpart into column
See error Can't convert object to primitive value

Expected behavior
After adding polyfill, webpart can run normally
Environment details (development & target environment)

Your Developer Environment: Windows 10
Target Environment: SharePoint Online
Framework: Node.js v10
Browser(s):  IE v11
Tooling: VS Code 、 SPFx v1.10.0

Additional context
Error message

Error breakpoint

Object Information

Normal operation after removing polyfill and ES6/ES7 dependent components

Can someone answer this question?

@rquanx - @pnp/polyfill-ie11 should work fine, we use it successfully in IE11.
Using @babel/polyfill in it entirety wont work due to the issues discussed in https://github.com/SharePoint/sp-dev-docs/issues/888
The workbench is also fairly flakey in IE11 to begin with. To avoid this we use a debug build deployed to a real SharePoint page rather than the workbench.

@rmillener  Thanks.

My users periodically have the same error 'Can't convert object to primitive value' in IE, sometimes it goes away by itself but comes back later. I use @pnp/polyfill-ie11 in 1.9.1 spfx webpart

I solved the problem by manually import each dependency package

do you mean each polyfill from core-js? or from @pnp/polyfill-ie11? Can you provide an example?

do you mean each polyfill from core-js? or from @pnp/polyfill-ie11? Could you provide an example?

yeah,
import "core-js/modules/es6.array.from";
import "core-js/modules/es6.array.fill";
import "core-js/modules/es6.array.iterator";
import "core-js/es6/promise";
import "core-js/es6/reflect";
import "es6-map/implement";
// import "core-js/es6/symbol";  // in my case ,symbol is unnecessarily
import "whatwg-fetch";

Is there any solution you could find? @aslanovsergey I have same problem as yours. Sometimes It works and mostly it does not.

Is there any solution you could find? @aslanovsergey I have same problem as yours. Sometimes It works and mostly it does not.

For IE, follow below steps
Install @pnp/polyfill-ie11

npm install @pnp/polyfill-ie11 --save

Import @pnp/polyfill-ie11

import "@pnp/polyfill-ie11";

Add/Updated onInit in web part TS file
protected onInit(): Promise {
debugger;
sp.setup({
ie11: true,
spfxContext: this.context
});
return super.onInit();
}
Note: It won't work in workbench, you must generate package and check it on the SharePoint page

I'm going to close this one.  IE support has been deprecated.


shortcut for "go to parent"

The shortcut I really miss in logseq is to go to the parent block from the current one. Does it make sense to implement a command for that here in this plugin, or should I report an issue to logseq repo?

The shortcut I really miss in logseq is to go to the parent block from the current one. Does it make sense to implement a command for that here in this plugin, or should I report an issue to logseq repo?

Yeah, sounds like a Logseq's feature and not part of this plugin. You can create a feature request and describe your needs - best with some illustrations!

Thanks! Create a request here: https://discuss.logseq.com/t/jump-to-parent-block-shortcut-command/12706

I've also started looking into logseq API. Looks like  such methods as logseq.Editor.moveBlock could be of great help.
And actually, ctrl-a in logseq by default moves to parent, but requires an extra stroke.
On top of that, this plugin https://github.com/vipzhicheng/logseq-plugin-vim-shortcuts gives some other options for navigation I've been missing.


Update doc generation to incorperate annotation changes

Shall we update https://github.com/ballerinalang/docerina/blob/master/docs/Annotations.md as a part of this.


Server crashes when broadcasting to disconnected user.

Platform/Language: Windows (MSVC 2015, C++)
Using SSL?: No
Problem description:
The following code is pretty simple, it does the following: "Prints out any users that connects". When a user disconnects, it broadcasts to all connected users who is leaving. The problem happens when, for some reason, the server tries to send a message to a user already disconnected, it then tries to access a memory that was already erased.
Stack trace:
 	echo.exe!uS::NodeData::getSmallMemoryBlock(int index) Line 146	C++
 	echo.exe!uWS::WebSocket<1>::sendPrepared(uWS::WebSocket<1>::PreparedMessage * preparedMessage, void * callbackData) Line 97	C++
 	echo.exe!uWS::Group<1>::broadcast::__l2::<lambda>(uWS::WebSocket<1> ws) Line 138	C++
 	echo.exe!uWS::Group<1>::forEach<void <lambda>(uWS::WebSocket<1>) >(const uWS::Group<1>::broadcast::__l2::void <lambda>(uWS::WebSocket<1>) & cb) Line 73	C++
 	echo.exe!uWS::Group<1>::broadcast(const char * message, unsigned int length, uWS::OpCode opCode) Line 139	C++
 	echo.exe!main::__l2::<lambda>(uWS::WebSocket<1> ws, int code, char * m, unsigned int length) Line 29	C++
 	[External Code]
 	echo.exe!uWS::WebSocket<1>::onEnd(uS::Socket s) Line 170	C++
 	echo.exe!uWS::WebSocket<1>::sendPrepared(uWS::WebSocket<1>::PreparedMessage * preparedMessage, void * callbackData) Line 117	C++
 	echo.exe!uWS::Group<1>::broadcast::__l2::<lambda>(uWS::WebSocket<1> ws) Line 138	C++
 	echo.exe!uWS::Group<1>::forEach<void <lambda>(uWS::WebSocket<1>) >(const uWS::Group<1>::broadcast::__l2::void <lambda>(uWS::WebSocket<1>) & cb) Line 73	C++
 	echo.exe!uWS::Group<1>::broadcast(const char * message, unsigned int length, uWS::OpCode opCode) Line 139	C++
 	echo.exe!main::__l2::<lambda>(uWS::WebSocket<1> ws, int code, char * m, unsigned int length) Line 29	C++
 	[External Code]
>	echo.exe!uWS::WebSocket<1>::onEnd(uS::Socket s) Line 170	C++
 	echo.exe!uS::Socket::io_cb<uWS::WebSocket<1> >(uv_poll_s * p, int status, int events) Line 280	C++

How to reproduce:

Run the server with the code below
Connect a few clients to the server (say 10).
Disconnect them all at the same time.

Short code:
#include <uWS.h>
#include <iostream>

int main() {
  uWS::Hub h;

  h.onConnection([&h](uWS::WebSocket<uWS::SERVER> ws, uWS::UpgradeInfo ui) {
    std::cout << "Connected: " << (long)ws.getPollHandle() << std::endl;
  });

  h.onDisconnection([&h](uWS::WebSocket<uWS::SERVER> ws, int code, char* m, size_t length) {
    std::cout << "Disconnected: " << (long)ws.getPollHandle() << std::endl;

    std::string msg = "User disconnected: ";
    msg += std::to_string((size_t)ws.getPollHandle());

    h.uWS::Group<uWS::SERVER>::broadcast(msg.c_str(), msg.length() + 1, uWS::TEXT);

    // Print out who we will be broadcasting to.
    std::cout << "Broadcasting to: ";
    h.uWS::Group<uWS::SERVER>::forEach([](uWS::WebSocket<uWS::SERVER> ws) {
      std::cout << (long)ws.getPollHandle() << " ";
    });
    std::cout << std::endl;
  });

  h.listen(8000);
  h.run();
}

You are not allowed to use a disconnected socket in any way. "Disconnection" means that the socket is removed and invalid. It is also a bad idea to broadcast in the disconnection handler since that can lead to a stack overflow if you have many sockets disconnected. You need to broadcast in the next loop iteration (using uv_async_send or a timer).

Wasn't the client supposed to be erased from the 'connected clients list' on the disconnection handler? Meaning it shouldn't be sending a message to the one that disconnected.

Yeah in that case you are hitting a bug of a nested forEach. You could solve it by deferring the forEach to next loop iteration.

All right. I suppose that's what this todo is about.
I will defer it to the next loop then. Thank you for your input and for the quick response.

Problem is that if you disconnect 10 clients at the same time they will get their disconnection handler called one a time. If you then try and broadcast in the first handler then that broadcast will fail and immediately call the disconnection handler of next socket which will do the same which will trigger a broadcast inside a broadcast, etc. This leads to stack overflow (but in this case you are hitting a bug).

Yep. It's a bug, I didn't finish that forEach thing (broadcast calls forEach).

You could have a vector of pointers and simply append to that vector in disconnection handler and then call uv_async_send and have the handler merge those pointers in the vector into one single broadcast. That improves performance also.

Yeap, that's what I had actually. But I was having this problem, and I changed it to use broadcast(...) to try to nail the problem down, removing my own code's logic from the equation.

Yeah it's definitely a bug in µWS, but since broadcasting in the disconnection handler is a bad idea I haven't bothered fixing it :wink:

On that issue. Is there any reason why you implemented the linked-list of connected users manually instead of using the standard std::list implementation?

@lucasterra Could you try the same thing as described in #335. If you comment out these thee lines it should defer the disconnection handler by itself, which means you should be able to safely broadcast in the disconnection handler.

Trying right now...

@alexhultman yeap, it works. :)
Worked fine with 1000 clients. Want me to submit the PR?

We just need to wait and hear from #335 also


feat(linter): support more flexible config.globals values

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4990  👈
#4989
main

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

Join @DonIsaac and the rest of your teammates on  Graphite


Smarty/smartjs syntax would be awesome

Smarty/jsmart syntax would be awesome.
(Its just an idea)

Replacing Razor with another templating engine eg Smarty, is way to much work and Blazor is not the right project to adress this issue.

@peon501 Thanks for the suggestion! We don't have plans to support other templating engines in Blazor. But maybe this would a be a good community extension?
Since we don't have any plans here I'm going to go ahead and close this. I hope that's ok!


Change ANSIBLE_LIBRARY PATH processing

When the environment variable ANSIBLE_LIBRARY is used, molecule is
currently extending the path with the contents of the found variable,
however, this will cause problems as the system driven path will
conflict with a user defined path. To correct this issue the path
processing will now insert the user provided variable into the path.
Signed-off-by: Kevin Carter<EMAIL_ADDRESS>Please include details of what it is, how to use it, how it's been tested
PR Type

Bugfix Pull Request

This change will not pass the docs checks until https://github.com/ansible-community/molecule/pull/3138 merges.

@cloudnull Would you be so kind as to rebase your work on top of the main branch and then force push it? This should turn the last red test into a green one (In the log at https://readthedocs.org/projects/molecule/builds/13925451/, I still see the old extension being used). Thank you in advance!

@cloudnull Would you be so kind as to rebase your work on top of the main branch and then force push it? This should turn the last red test into a green one (In the log at https://readthedocs.org/projects/molecule/builds/13925451/, I still see the old extension being used). Thank you in advance!

Done


Change ModuleInfo._isVirtual to an enum

Start tracking peimage load layout by enum in accordance with https://github.com/dotnet/runtime/blob/bd6c81a944f609b88908864827b7797c6b127c5b/src/coreclr/src/vm/peimage.h#L287.
Note that this change is NOT intended to make this work correctly for all file layouts, but instead to change the public surface area of ClrMD to support future improvments here before RTM.
Contributes to https://github.com/microsoft/clrmd/issues/475.

@NextTurn Any thoughts on this?

isVirtual is not too bad. In most cases we only need a boolean to decide whether to use virtual address/size or file offset/size. I assume that the "mapped layout" and the "loaded layout" should always be equivalent.


Zulrah teleports

Ever noticed how zulrah drops zulrah teleports? I'll tell you a little story about them.
They're useless!!
Would be nice to be able to use them to speed up kills.
Cheers <3

If this PR is approved, should close this issue: https://github.com/oldschoolgg/oldschoolbot/pull/3140


Autocomplete replaces the value manually entered for the selected from autocomplete result list

Bug, feature request, or proposal:
Autocomplete when selecting a value from the autocomplete list
What is the expected behavior?
it should keep the text manually entered and have the option to ignore the selected to not modify what was typed within the input box
What is the current behavior?
It overrides what was in the input box by the selected value in the autocomplete result list
What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
What is the use-case or motivation for changing an existing behavior?
When searching people want to know what is the actual text they been using, and not to lose that value by the selection
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
"@angular/material": "^5.0.0-rc0",
Windows 8,
Chrome
Is there anything else we should know?
No, thank you

Could you give more details of the behavior you extected? Based on what I understood from your request, the autocomplete should not replace the typed text by what you've selected.
It's strange. Why would you select something if you don't want it do be the actual value in the input?

Thanks Juliano
Because you loose what you’ve written ? And perhaps you’d like to keep that
text for further searching ?
On Mon, 19 Feb 2018 at 13:19, Juliano<EMAIL_ADDRESS>wrote:

Could you give more details of the behavior you extected? Based on what I
understood from your request, the autocomplete should not replace the typed
text by what you've selected.
It's strange. Why would you select something if you don't want it do be
the actual value in the input?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angular/material2/issues/10017#issuecomment-366690815,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEW4EeQ1UQdtqnjeX7cqmLD_UCzdRZ6tks5tWXTTgaJpZM4SKb_x
.

Well... you can control what is shown in the input by the displayWith input. Then you can, for example save what was typed in another property to use in the displayWith.

I don't see the point of having an autocomplete if you want to maintain what the user has written. In that case why not use a regular input?

Perhaps it’s because there’s another use case where you don’t want to have
the autocomplete full behaviour just having the suggestions displayed and
use the results differently  will suffice
On Mon, 19 Feb 2018 at 19:51, Kristiyan Kostadinov<EMAIL_ADDRESS>wrote:

I don't see the point of having an autocomplete if you want to maintain
what the user has written. In that case why not use a regular input?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angular/material2/issues/10017#issuecomment-366788321,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEW4EXjRE221DaotGpHUJmKlxQgYjGASks5tWdDNgaJpZM4SKb_x
.

I see what you mean, but it goes against all of the assumptions made when building the autocomplete, not to mention the uncertainty about how accessibility would be handled.


Task/remove odf label

 Remove label odf = storage.ibm.com from objects and add new odf-fs label

Closing PR as according to RH such change is not supported.


"Could not sign the transaction" causes crash

Error report 8d08ebf shows a problem with signing a transaction. This should not trigger a crash but instead give the user clearer information on the nature of the failure.
[2015-11-03T17:20:59.364-06:00] DEBUG safe-fixed-bitcoin-network-2 org.multibit.hd.core.services.BitcoinNetworkService - Starting the send process
[2015-11-03T17:21:03.719-06:00] ERROR safe-fixed-bitcoin-network-2 org.multibit.hd.core.services.BitcoinNetworkService - Could not sign the transaction

org.bitcoinj.crypto.KeyCrypterException: Could not decrypt bytes
	at org.bitcoinj.crypto.KeyCrypterScrypt.decrypt(KeyCrypterScrypt.java:225) ~[multibit-hd.jar:na]
	at org.bitcoinj.crypto.DeterministicKey.findOrDeriveEncryptedPrivateKey(DeterministicKey.java:405) ~[multibit-hd.jar:na]
	at org.bitcoinj.crypto.DeterministicKey.decrypt(DeterministicKey.java:377) ~[multibit-hd.jar:na]
	at org.bitcoinj.crypto.DeterministicKey.decrypt(DeterministicKey.java:41) ~[multibit-hd.jar:na]
	at org.bitcoinj.core.ECKey.decrypt(ECKey.java:1091) ~[multibit-hd.jar:na]
	at org.bitcoinj.crypto.DeterministicKey.decrypt(DeterministicKey.java:388) ~[multibit-hd.jar:na]
	at org.bitcoinj.crypto.DeterministicKey.decrypt(DeterministicKey.java:41) ~[multibit-hd.jar:na]
	at org.bitcoinj.wallet.DecryptingKeyBag.maybeDecrypt(DecryptingKeyBag.java:50) ~[multibit-hd.jar:na]
	at org.bitcoinj.wallet.DecryptingKeyBag.findKeyFromPubHash(DecryptingKeyBag.java:67) ~[multibit-hd.jar:na]
	at org.bitcoinj.core.TransactionOutPoint.getConnectedRedeemData(TransactionOutPoint.java:182) ~[multibit-hd.jar:na]
	at org.bitcoinj.core.TransactionInput.getConnectedRedeemData(TransactionInput.java:316) ~[multibit-hd.jar:na]
	at org.bitcoinj.core.Wallet.signTransaction(Wallet.java:3810) ~[multibit-hd.jar:na]
	at org.multibit.hd.core.services.BitcoinNetworkService.signDirectly(BitcoinNetworkService.java:1035) [multibit-hd.jar:na]
	at org.multibit.hd.core.services.BitcoinNetworkService.performSend(BitcoinNetworkService.java:391) [multibit-hd.jar:na]
	at org.multibit.hd.core.services.BitcoinNetworkService.access$300(BitcoinNetworkService.java:61) [multibit-hd.jar:na]
	at org.multibit.hd.core.services.BitcoinNetworkService$2.run(BitcoinNetworkService.java:344) [multibit-hd.jar:na]
	at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source) [na:1.7.0_80]
	at java.util.concurrent.FutureTask.run(Unknown Source) [na:1.7.0_80]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source) [na:1.7.0_80]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) [na:1.7.0_80]
	at java.lang.Thread.run(Unknown Source) [na:1.7.0_80]
Caused by: org.spongycastle.crypto.InvalidCipherTextException: pad block corrupted
	at org.spongycastle.crypto.paddings.PKCS7Padding.padCount(PKCS7Padding.java:63) ~[multibit-hd.jar:na]
	at org.spongycastle.crypto.paddings.PaddedBufferedBlockCipher.doFinal(PaddedBufferedBlockCipher.java:287) ~[multibit-hd.jar:na]
	at org.bitcoinj.crypto.KeyCrypterScrypt.decrypt(KeyCrypterScrypt.java:221) ~[multibit-hd.jar:na]
	... 20 common frames omitted

I have tested MBHD with putting in a bad password on Send. It behavies as you would expect, with the send report screen indicating 'Could not decrypt bytes'.
I have pushed a small change so that only the error message is logged rather than the whole stack trace. This means the Error Reporter won't identify the stack trace in any future uploads. This will make the true fault more easy to spot in future.

Verified working.
Closing.


add minimal example for the "kubeadm kubeconfig user" command

The command only has Cobra (CLI) documentation and there are no pointers on what fields are relevant in the configuration passed with --config to it:
https://kubernetes.io/docs/reference/setup-tools/kubeadm/kubeadm-kubeconfig/
Add a minimal example + config for the command usage on this page:
https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/kubeadm-certs/
/sig cluster-lifecycle
/kind feature

/triage accepted

/language en


Improve related tag recommendation

Tasks

[ ] Some related tags might lead user to a dead end of content discovery. Need a new algorithm that includes factors not just related.
[ ] Update the backend query

fixed in
https://github.com/thematters/matters-server/blob/develop/src/connectors/tagService.ts#L1173
in dab1c8fc51ce
in #2783


Nuxt 3 support

Hi there!
I saw your post on Reddit and immediately wanted to try out Notivue in a Nuxt 3 application. I followed the guidelines in the Nuxt 3 docs and added a plugin file:
plugins/notivue.client.js
import { notivue } from 'notivue'

import 'notivue/notifications.css'
import 'notivue/animations.css'

export default defineNuxtPlugin((nuxtApp) => {
  nuxtApp.vueApp.use(notivue).mount('#app')
})

Then I followed step 2 and 3 from your documentation. When I open the app in the browser I get this error:
Cannot read properties of undefined (reading 'push')
Some additional info about the Nuxt app:
------------------------------
- Operating System: `Darwin`
- Node Version:     `v18.15.0`
- Nuxt Version:     `3.2.2`
- Nitro Version:    `2.2.3`
- Package Manager<EMAIL_ADDRESS>- Builder:          `vite`
- User Config:      `app`, `modules`, `supabase`, `pinia`
- Runtime Modules:  `<EMAIL_ADDRESS>`<EMAIL_ADDRESS>`<EMAIL_ADDRESS>- Build Modules:    `-`
------------------------------

Hi Mogli, I suspect that you're not calling usePush in the setup function of your component as displayed in the Quick Start section.
<script setup>
import { usePush } from 'notivue'

const push = usePush()
</script>

Here, a working example on Nuxt 3 using StackBlitz.
Please, let me know if everything's sorted out.


Moving docker conf mustache to k8s configmap

Purpose
Moving docker-conf to a k8s configmap.
Updating docker image for the apim-operator
Goals
DevOp will have the control to change the docker conf template if necessary
Approach
Removing existing docker-conf template/mustache and moving the template to a k8s configmap
User stories

Summary of user stories addressed by this change>

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Training

Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable

Certification

Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to<EMAIL_ADDRESS>and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.

Marketing

Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

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

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Migrations (if applicable)

Describe migration steps and platforms on which migration has been tested

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

Learning

Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.

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


Running security bench: Cannot reach trust server at this time

I am running the security bench per the documentation for Ubuntu
docker run -it --net host --pid host --userns host --cap-add audit_control \
    -e DOCKER_CONTENT_TRUST=$DOCKER_CONTENT_TRUST \
    -v /etc:/etc:ro \
    -v /lib/systemd/system:/lib/systemd/system:ro \
    -v /usr/bin/containerd:/usr/bin/containerd:ro \
    -v /usr/bin/runc:/usr/bin/runc:ro \
    -v /usr/lib/systemd:/usr/lib/systemd:ro \
    -v /var/lib:/var/lib:ro \
    -v /var/run/docker.sock:/var/run/docker.sock:ro \
    --label docker_bench_security \
    docker/docker-bench-security

I get this output:
docker: unable to reach trust server at this time: 500.
See 'docker run --help'.

Is there something I am missing? Do I need to setup the DTR per here
I'd truly appreciate any help or suggestions or guidance.

Hi @rirlswift, are you running the officail Docker image, or have you built your own?
If you're using the official image, could you please try running the script on the host or build your own image?

Hey  konstruktoid,
Thanks for helping me get this to work. I am now running the script on the host via
git clone https://github.com/docker/docker-bench-security.git
cd docker-bench-security
sudo sh docker-bench-security.sh

Looks great, even on CE.

ref https://github.com/docker/docker-bench-security/issues/405


libow: init at 3.2p1

Motivation for this change
libow 1-wire file system library (owfs) package
includes libraries and headers only
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
[x] other Linux distributions

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

libow 1-wire file system library (owfs) package
includes libraries and headers only

@GrahamcOfBorg build libow

Please split your changes into two commits:

Adding yourself as maintainer
Adding the package

Done

Now the commit where you add the package does not confirm to the CONTRIBUTING anymore.
Sorry for bothering you with this, but as you seem to be new I try to show you how to do it right so that people do not have to tell you later :smile:

Now the commit where you add the package does not confirm to the CONTRIBUTING anymore.
Sorry for bothering you with this, but as you seem to be new I try to show you how to do it right so that people do not have to tell you later 😄

Done )

This pull request has been mentioned on Nix community. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/12

Good day,
I see but 3.2p1 is the only production release used by other distributions (the package was actually ported from Arch, while Ubuntu is still on 3.1.x). If you want I can change source to github right now, however I suggest to keep 3.2p1 version until the new one get a positive feedback (at least for x86-64 and ARM). Unfortunately I'm not a software author (but heavy user :) and can't give any warranties about new versions.
So, let's change source to github right now or when versions never than 3.2p1 go in production? What do you prefer?

@divi255 Since you're way more of an expert than me on this piece of software, I defer to your opinion. Let's go with the github source using fetchFromGitHub but keep the version. :)

@divi255 Since you're way more of an expert than me on this piece of software, I defer to your opinion. Let's go with the github source using fetchFromGitHub but keep the version. :)

Changed to use github and fetchFromGitHub . Seems to work fine )

@GrahamcOfBorg build libow

@GrahamcOfBorg build libow

Thank you!


Move tests to BATS 1.0

We're currently on sstephenson/bats 0.4 package: https://github.com/ahmetb/kubectx/blob/db8b70661279c7bebec7dda1cf077226d1fefd06/.travis.yml#L2
BATS 1.0 is now maintained at https://github.com/bats-core/bats-core and has its own distribution. Let's move to that.

@ahmetb If anyone should take up the subject. Do you want the CI to stay with Travis or do you want to move it to Github Action as well?

GitHub Actions would be better but I am in the process of rewriting it in Go so I don’t advise working on it right now.


Fix formatting for @at-root and @-ms-viewport

CSSfmt turns this:
@at-root {
  @-ms-viewport {
    width: device-width;
  }
}

...into this:
@at-root  {
  @-ms-viewport  {
    width: device-width;
  }
}

Extra extra spaces have been added before curly braces. These changes make sure that those atrules have only 1 space.

Nice work!!


Metal3: helm dependency update . failes

When trying to update Metal3's charts dependencies I got:
Error: dependency "baremetal-operator" has an invalid version/constraint format: improper constraint:
The reason is that in charts/metal3-deploy/0.1.0/Chart.yaml for each dependency the version key is missing but it is actually required.
I figured out that this was the reason I encountered the issue #10 (and opened #11) but I think we can close them and leave this one since by fixing the dependencies' version field properly also the aliases metal3-<subchart name> work.

For the moment, I think all of the issues you've encountered are fixed within my fork linked here: https://github.com/dbw7/charts
Test out the charts in the /charts directory but do note that the charts in /packages and /assets are outdated at the moment because of an issue with the chart building scripts that we will discuss with the team.
Give it a try and let me know if you run into any issues 😄

Hello, this issue has been fixed in #15


Configurable Questionnaire config type

Describe the feature request.
A clear and concise description of the feature or what problem is being solved
There is a requirement that results in the need of configurable Questionnaire config type using rules.
Additional context

Needed for Engage, but it should be a common use case.
Use case: After user initially submits a task related Questionnaire, the user wants to open the Questionnaire again to edit it's answer. This is a change of Questionnaire config type from DEFAULT to EDIT.

Acceptance criteria

[x] The Questionnaire config type accept a rule from the rules engine

Area path
Checkout and run the QuestionnaireConfigTest.
Implementation plan (For Engineers)

[x] Change the Questionnaire config type property from ENUM to STRING
[x] Interpolate the Questionnaire config type
[x] Adapt the functions that are affected by the change of Questionnaire config type property change from ENUM to STRING
[x] Adapt the test to accept rules

@f-odhiambo LOE less than a day


Description
I will be creating a documention website using docusaurus where everything about the project will be mentions including installation, use case, local setup, architecture of the project, technologies used etc. All the contributors and collaborators will be mentioned there in.
Suggested Solution
I think docusaurus will be a good tool to create website.
Alternatives
If anyone has, any other idea, feel free to suggest
Additional Context
No response

Would like to do this

@Bhanu-code

@Bhanu-code Sir i would like to work on this as i prior experience regarding this


Improve this to make best use of the information available. We should be able to make use of the --service argument, and --from when it's a URL.

merged in #49


monocot inflorescence development

This is a sourceforge item to work out terms for monocot
inflorescence development. It follows on from
[ 877837 ] inflorescence development (+more)
https://sourceforge.net/tracker/index.php?
func=detail&aid=877837&group_id=36855&atid=440764
Reported by: jenclark
Original Ticket: "geneontology/ontology-requests/1780":https://sourceforge.net/p/geneontology/ontology-requests/1780

thanks www.movied.org


Infinite loop if you don't read cells

I tried a small variation on the example in the README to try and get a sense for how fast this is. If you don't iterate over the columns, it goes into an infinite loop.
import { readCSV } from "https://deno.land/x/csv/mod.ts";

const f = await Deno.open(Deno.args[0]);

let n = 0;
for await (const row of readCSV(f)) {
  n++;
}
console.log('Read', n, 'lines');

f.close();

$ deno run --allow-read csvtest.ts test.csv
Compile file:///Users/danvk/github/deno-play/csvtest.ts
^C

My CSV file looks like this:
A,B,C
1,2,3
4,5,6

Hello @danvk and thank you for your feedback!
I've fixed it by this commit.
Now line will be read till end even you skipped that part.

@danvk FYI currently deno-csv should be very slow because it based on ES generators and not optimised yet, but I'm going to optimise it soon.

Benchmarked against https://csv.js.org/parse/:
deno run --allow-env --allow-read --allow-hrtime deno-csv.ts 500000-Records.csv
Read 500001 lines for<PHONE_NUMBER>94 seconds
node npm-csv-parse.js 500000-Records.csv
Read 500001 lines for 8.543203381 seconds

@danvk I've released some optimisations https://github.com/vslinko/deno-csv#benchmarks

Thanks @vslinko, that is definitely an improvement!
The generally slow speed of async iteration in Node has always been a pet peeve of mine, see this post from a few years back: Async iterators: These Promises Are Killing My Performance!.
The good news is that things have gotten considerably faster from node 8 → 10 → 12. I have a repo with some benchmarks and sample numbers for a 32MB CSV file here.
On my machine, Python's CSV reader reads that file in ~0.93s:
$ python3.7 csv_reader.py
Read 549996 lines, 4949964 cells in 0.936364 s

deno-csv takes ~5s, so it's slower but within an order of magnitude:
$ deno run --allow-read --allow-hrtime csvtest.ts ~/github/async-iteration/stop_times.txt
Read 549997 lines in 4825.432208 ms

The off-by-one error is concerning — I think it yields a blank line at the end of the file.


Compilation errors with non-latest stable cargo/rustc

ref: https://github.com/Stebalien/tempfile/pull/118#issuecomment-645553590
Looks something like this:
error[E0599]: no associated item named `MAX` found for type `isize` in the current scope
   --> C:\Users\appveyor\.cargo\registry\src\github.com-1ecc6299db9ec823\fastrand-1.1.0\src\lib.rs:177:46
    |
177 |               if low == $t::MIN && high == $t::MAX {
    |                                                ^^^ associated item not found in `isize`
...
282 | / integer!(
283 | |     isize,
284 | |     gen_u32,
285 | |     gen_mod_u32,
286 | |     "Generates a random `isize` in the given range."
287 | | );
    | |__- in this macro invocation

That's with rustc/cargo 1.32.0 in the CI failure.
I'm also using fastrand in https://github.com/joshuarli/epub-optimizer. It's way past terminal scrollback now but I believe I remember I was on 1.43.0 and got the same failure. Works on 1.44.0.
If there's interest I can verify the failure on 1.43.0, or bisect MSRV if I remembered incorrectly.

This should be fixed in just published v1.2.2
Let me know if the problem persists!

It still fails on 1.32.0, but that's kind of old. I didn't yet test 1.43.0, but do you know the new MSRV? I wasn't able to find anything quickly online.

If you do find out the new MSRV it would be nice to include that in the changelog and readme.

Published v1.2.3 and checked that it works with Rust 1.32.0

Nice, just confirmed in https://github.com/Stebalien/tempfile/pull/118. Thanks @stjepang! I really like fastrand btw. Hoping rust ecosystem moves towards it from rand, I'm betting majority rand users usecases would easily be satisfied with fastrand.


Add check for drop table calls

What has been done
I've add a check for the drop table command and provided a warning about dependant code

Oops, this was mean to be against our repo - might PR this a little later!

Hey Michael,
Just two comments on this PR if you do end up PR'ing to us eventually.

You might want to include in your warning that this is an irreversible migration and should be in an up method as opposed to a change method.

Minor nitpick, but I noticed a few typos in the comments
seperate --> separate
dependant --> dependent

Hey @mattgmarcus, thank you for taking the time to look through this anyway.
Great points, I'll definitely fold them into our own forked version for now (and don't worry about nitpicking, details are important).
The only reason why I wouldn't necessarily PR this change is because I added instructions that are specific to our deploy environment (blue/green). Would this type of change be something you'd ultimately like to include?
TBH I want to get the full gambit of troublesome migrations catered for, loosely based off: https://www.braintreepayments.com/blog/safe-operations-for-high-volume-postgresql/.


The template misses some replacements

After scaffolding, the project won't build.
This is due to the template missing 3 replacements from:
pallet-template

to
pallet-{{pallet_name}}

I can build the project after this fix.

Fixed in d0b5dae75e1229844761fcd695ec644ea1b43fca


[QUIC]add QUIC connection close detail to transport close details

Commit Message: add QUIC connection close detail to transport close details
Additional Description: n/a
Risk Level: low
Testing: existing tests
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

/retest

/assign @alyssawilk


[Codegen] Add pass to decompose pack unpack ops at dispatch boundaries

This PR adds a wrapper pass around DecomposePackUnPackOps, which adds a control function for decomposing only packs and unpacks whose reshapes can be folded with dispatch tensor loads/stores.
This PR also removes the public pass constructor with a control function, opting to use wrapper passes in place of constructing passes with arbitrary control functions. This is better for creating simple bug repros, since the control function is part of the pass.

Currently depends on https://github.com/iree-org/iree/pull/18843 for some StaticValueUtils.


Formatter migration path not explained

Summary
Migration path for the new formatter API is not explained anywhere. Due to parallel_tests, I'm getting a large number of:
WARNING: The formatter ... is using the deprecated formatter API which will be removed in v4.0 of Cucumber.

But apart from https://github.com/cucumber/cucumber-ruby/blob/c26c2d102bb72193fe299b2d25edc4e82fdbba54/features/docs/extending_cucumber/custom_formatter.feature#L44-L85 I can't find any good summary of how the new version works / differs.
The existing formatters still use the deprecated API and the wiki shows only the old format at https://github.com/cucumber/cucumber/wiki/Custom-Formatters
Expected Behavior
Link to some explanation of the new API and possibly a migration path.
Current Behavior
Lots of repeated warnings in every build with no explanation of how to fix them
Possible Solution
Wiki/web page explaining the migration.
Your Environment

Version used: 3.1.0
Operating System and version: MacOS 10.13
Link to your project:

Still an issue for me.

+1

@viraptor @grosser I created this minimal example to try to reproduce the deprecation warnings you're seeing. However, if I run my features with parallel_cucumber features --runtime-log, it doesn't seem to trigger the formatter to be used. What do I need to do differently to trigger one of the formatters to be used? I'm trying to create a PR to make the formatters compatible with the Cucumber events API but I need to be able to repro first.

Duplicate of: https://github.com/cucumber/docs.cucumber.io/issues/233 (?)

@mlvandijk Yes. I just created a PR to fix this problem in the parallel_tests gem, and now I'm looking at the docs to try and improve our documentation of the new API.

thx!
On Thu, Oct 25, 2018 at 5:40 PM Dana Scheider<EMAIL_ADDRESS>wrote:

@mlvandijk https://github.com/mlvandijk Yes. I just created a PR to fix
this problem in the parallel_tests
https://github.com/grosser/parallel_tests gem, and now I'm looking at
the docs to try and improve our documentation of the new API.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cucumber/cucumber-ruby/issues/1294#issuecomment-433249939,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAsZ8FOzYds_c4l_2NYwMGvQP0GUtD8ks5uoln0gaJpZM4T6xqM
.

That's awesome @danascheider Let me know if you need help with the docs!

Thanks @mlvandijk, I might. I haven't worked a whole lot with custom formatters, luckily the one that needed fixing in this case was pretty straightforward.
Also @viraptor @griffincox Update parallel_tests to version 2.26.0, which just dropped today, and you should see the problem is fixed. I just merged a PR at my job updating this and the deprecation warnings went away. Let us know if the problem persists for you.

Information regarding Ruby formatters can be found here: https://docs.cucumber.io/cucumber/reporting/


api.search() results don't match query

I am using tweepy to find tweets matching a query. After setting up an api instance, I do the following:
tweepy = api.search("tweepy")

for i in tweepy:
    print("tweepy" in i.text)

> True
> True
> False
> False

Any idea what is happening? Why don't all the results match the query?
Thank you for the reply!

Text can sometimes be "extended text" so if the string "Tweepy" exists in the tweet at all, it shows up in the returned variable.
Some texts that are over (I think) 100 characters need to be called as "extended_text". If you print(i.text) the "false" values should appear with "..." at the end of them.

There are multiple possible reasons for that statement to be printing False.
The most common is likely due to the standard search API not being case-sensitive. Therefore, any Tweet result with "Tweepy" in it (or with any other variation in case) will print False with that statement.
Another possible reason is that without using extended mode, only the first 140 characters of the Tweet's text will be returned, and the rest will be truncated. The standard search API will still return results matching the query in the parts of the Tweet's text that are truncated. This means your statement will print False in the cases that the query is only in the Tweet's text past 140 characters.
This will also be the case for a truncated Retweet.
Additionally, the query itself might not even be in the Tweet's text. The standard search API will also return results with the query in the user's name, the user's screen name, a URL entity part of the Tweet, etc. This applies to Retweets as well.


Update FSI docs to cover dotnet fsi and package management

A quick pass through the FSI reference docs to update it for .Net Core
I updated references to fsi.exe in favor of dotnet fsi, and added a section about package management.
I'm hoping to link to this page/section on package management directly from nuget.org (see https://github.com/NuGet/NuGetGallery/pull/8167) so I want the section on package management to be relevant.

All great suggestions, and all applied. Thanks for the review pass!

Thank you, friend. May the memes be with you


Location

Location in new app does not work in report card.
it is also strange that we see no google maps image @ when and where - button.
maybe it is due to that problem.

If we could not parse the location the map is not shown

what is wrong with the location.
I have the feeling we didn't have to deal with this issue before the new lay-out or am i wrong ?
the thing is, I need to give a solution to the customers. what is the solution ?

No its always been like this.
Google just could not parse the address. However i think it looks oke...

Same issue as #138


[improve][transform] flink engine support flink-sql、udf components

Purpose of this pull request

Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

+1

We have spent a long time in the past six months removing the dependencies between Flink SQL and Spark SQL, and implementing SeaTunnel's own SQL Transform, which is designed to run on both Flink/Spark/Zeta. I don't think adding Flink SQL back is a good way.

I will close it first, if you have any advice, please start a discussion in<EMAIL_ADDRESS>


Navbar is not sticky - causing problem in navigation [GSSoC'23]

Describe the bug
The navbar of the website is not sticky, This will cause problem in navigation for the user.
To Reproduce
Steps to reproduce the behavior:

Go to any page in the website
Scroll down
See error

Expected behavior
Navbar must be sticky
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Please assign me this issue under GSSoC'23.

It was made sticky ig, in recent update

Ok. Should I close the issue then?


[keymgr] Clarify the sideload_sel error function

clarifies some issues found in #15862, #15858
This better describes what the check does and makes it slightly more useful.

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

alright @weicaiyang , this is IMO more useful.
To test this, instead of randomly forcing the sideload_sel like before, you should force the valid from the actual key slot.
When you do this, if the intended function is not actually for it to be valid, an error will trigger.

alright @weicaiyang , this is IMO more useful. To test this, instead of randomly forcing the sideload_sel like before, you should force the valid from the actual key slot.
When you do this, if the intended function is not actually for it to be valid, an error will trigger.

sounds good, Thanks Tim
I think we still need to do the force during a sideload operation, right?

alright @weicaiyang , this is IMO more useful. To test this, instead of randomly forcing the sideload_sel like before, you should force the valid from the actual key slot.
When you do this, if the intended function is not actually for it to be valid, an error will trigger.

sounds good, Thanks Tim I think we still need to do the force during a sideload operation, right?

right, but this is intended for the attack..where at the end of the operation, someone tries to load the key into a slot that it was not intended for (for example kmac -> otbn).


Implemented changes from torrid per https://github.com/lucidrains/dee…

https://github.com/lucidrains/deep-daze/issues/9#issuecomment-764007289
Again, haven't yet tested so probably best not to merge. Will try to hop on colab in a bit and see what the output is like.

"Daft Punk performing to the residents of a trailer park"
LGTM

"Daft Punk performing to the residents of a trailer park"
LGTM

@afiaka87 i'm not sure if this is the right way to go about it, since in the case we do need uniformity in one of the channels, this would just normalize it to noise

@afiaka87 i'm not sure if this is the right way to go about it, since in the case we do need uniformity in one of the channels, this would just normalize it to noise

Agreed.

Agreed.


Add Constantize Feature

Good work guys!
ActiveSupport has a constantize method that turns an input into a CONSTANT_CASE.
If you're interested in the feature, I can look if I find time to contribute it.

 It rather tries to find and return a declared constant with the name specified in the string. Since that's not readily doable in JS, I didn't port constantize in the first place.
Nevertheless I do think your idea would be a great addition. So please go for it!

There it is.
I've named the function toConstantCase to avoid confusion with rails's constantize. I've found this useful when generating names for redux saga actions.


requirements.txt file for dependencies

made a requirements.txt file and updated readme

Thanx. Have u pushed the requirements file? I see only changes in readme

so sorry i think i made a mistake, i will create a new pr

Thanx. Have u pushed the requirements file? I see only changes in readme

please check https://github.com/tyiannak/pyAudioAnalysis/pull/248


Confusing Documentation around named parameters

The problem
I was reading the docs at https://docs.perl6.org/language/objects#Object_construction and I found the signature of BUILD confusing. I also noticed that https://docs.perl6.org/language/functions#Arguments does not really talk about the named parameter syntax much.
Suggestions
A link to https://docs.perl6.org/language/functions#Arguments could be good from the object construction phase. It would also be good to have some documentation on how named parameters work and how they differ from pair constructors. It's not clear if they are a special syntax or a special case of something else.

You mean in the way it can define alias for attributes? I guess it wouldn't hurt to link that. Thanks for the report.


Add Pin/Unpin Option To Panels

I suggest that panels be closed by clicking outside their bounds and instead use the pin / unpin option to keep panels open or closed.
I think adding this feature will help both improve the user experience and speed up the use of this tool.

very good Idea, to keep panels open when user click on Pin Icon.


How to use html_rtsp_player in angular app 6

I want use library html_rtsp_play in angular app 6 . Help me

need integrate it, write me to email<EMAIL_ADDRESS>

I can't show you all my code but here is the most important piece. By the way I use the videojs plugin.
Before the component declaration
declare const videojs: any;
declare const streamedianPlayer: any;

/* tslint:disable:no-var-keyword */
var videojsPlayerIndex: number = 0;

(<any> videojs).getTech('Html5').registerSourceHandler({
	canHandleSource: (source: Source) => {
		if (source.type.indexOf('rtsp') !== -1) {
			return 'propably';
		}
		return '';
	},
	handleSource: (source: Source, tech: any) => {
		let socket = 'ws://youreproxyserver.com/ws-stream-proxy'; // todo define from somewhere
		let el = tech.el();
		let wsPlayer = streamedianPlayer(el, socket, source.src);
		return wsPlayer;
	},
}, 0);

In the component (component template is empty as I create the video element here, not sure it's the best things to do).
	protected initializePlayer(): Observable<any> {
		let subject: Subject<any> = new Subject();

		let options: PlayerOptions = {};
		options.controls = false;
		options.autoplay = false;
		options.preload = 'auto';
		// options.fluid = true;
		options.techOrder = ['html5'];
		options.children = ['mediaLoader', 'posterImage', 'textTrackDisplay', 'textTrackSettings'];
		options.plugins = {};

		const video = document.createElement('video');
		video.setAttribute('id', this.videojsPlayerId);
		video.setAttribute('class', 'videojs-player');
		this.elementRef.nativeElement.appendChild(video);

		this.videojsPlayer = videojs(video, options);

		this.videojsPlayer.ready(() => {
			this.zone.run(() => {
				this.log.debug('Videojs is ready');
				subject.next(this.videojsPlayer);
			});
		});

		this.log.trace('Creating videojs');
		return subject;
	}

thank you for help
On Thu, May 17, 2018 at 8:15 PM, Gaétan Collaud<EMAIL_ADDRESS>wrote:

I can't show you all my code but here is the most important piece. By the
way I use the videojs plugin.
Before the component declaration
declare const videojs: any;declare const streamedianPlayer: any;
/* tslint:disable:no-var-keyword */var videojsPlayerIndex: number = 0;
( videojs).getTech('Html5').registerSourceHandler({
canHandleSource: (source: Source) => {
if (source.type.indexOf('rtsp') !== -1) {
return 'propably';
}
return '';
},
handleSource: (source: Source, tech: any) => {
let socket = 'ws://youreproxyserver.com/ws-stream-proxy'; // todo define from somewhere		let el = tech.el();
let wsPlayer = streamedianPlayer(el, socket, source.src);
return wsPlayer;
},
}, 0);
In the component (component template is empty as I create the video
element here, not sure it's the best things to do).
protected initializePlayer(): Observable {
let subject: Subject = new Subject();
  let options: PlayerOptions = {};
  options.controls = false;
  options.autoplay = false;
  options.preload = 'auto';
  // options.fluid = true;		options.techOrder = ['html5'];
  options.children = ['mediaLoader', 'posterImage', 'textTrackDisplay', 'textTrackSettings'];
  options.plugins = {};

  const video = document.createElement('video');
  video.setAttribute('id', this.videojsPlayerId);
  video.setAttribute('class', 'videojs-player');
  this.elementRef.nativeElement.appendChild(video);

  this.videojsPlayer = videojs(video, options);

  this.videojsPlayer.ready(() => {
  	this.zone.run(() => {
  		this.log.debug('Videojs is ready');
  		subject.next(this.videojsPlayer);
  	});
  });

  this.log.trace('Creating videojs');
  return subject;

}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Streamedian/html5_rtsp_player/issues/86#issuecomment-389862120,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ac6OSt4-7F-POae8GRz59ivfJ00toqM9ks5tzXfzgaJpZM4T9ZZs
.


Appears to build to static URLs making it unmovable to another URL or service

The "yarn build" generates static HREFs (or incorrect HREFs) in the index.html restricting the JS App from being moved to another part of a URL tree.
Requires manually editing of the index.html to allow dynamic HREFs

Looks like @rufusu-sony removed "homepage": "." from the package.json in the last PR, which I suspect was inadvertent.

Should be fixed by b8772428e1164b949c13f3cb0163f77f8c52ea8d. Thanks for the report, Rich.


[$250] Bank account - "Hmm... it's not here" page appears after selecting Incorporation state

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.5-5
Reproducible in staging?: Y
Reproducible in production?: Unable to check
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4703397
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Pre-requisite: the user must have created a Workspace and have enabled Workflows.

Go to Settings > Workspaces > Workspace > Workflows.
Tap on "Connect bank account".
Select "Connect manually".
Go trough the add bank account process using the testing credentials.
Once you reach the Incorporation state page on the Company info section, tap on the field and try to select a state.

Expected Result:
Once the user taps on an option, the selected option should be displayed on the Incorporation state field and the user should be able to continue with the add bank account process.
Actual Result:
Once the user taps on an option, "Hmm... it's not here" page is displayed and the user is not able to continue with the add bank account process.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/1c005cef-1a7b-4284-b567-13fec433ab9c

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018f7892dcd73eb461
Upwork Job ID:<PHONE_NUMBER>946419022
Last Price Increase: 2024-07-09

Issue OwnerCurrent Issue Owner: @s77rt

@puneetlath I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

Doesn't look related to backend changes. Removing the label.

Hey @izarutskaya, what is the reason that we cannot test this in prod?

Reproducible in production?: Unable to check

@dominictb these options make sense to me. I like solution 2 for now as we don't need  a more complex and scalable solution yet.

Hi @s77rt can you please review the proposal above, ideally using solution 2?
We need the solution to resolve two bugs, so the changes will need to pass the test cases in both of these PRs:

https://github.com/Expensify/App/issues/45060
https://github.com/Expensify/App/pull/44981

2 makes sense to me as well but happy to hear what @s77rt says.
Alternately - do we want to get @nkdengineer / @hoangzinh on this since it seems like a regression from https://github.com/Expensify/App/pull/44981 ?

@dominictb Your RCA is correct. However regarding the solution I would prefer to revert https://github.com/Expensify/App/issues/44957 being a workaround, the backTo param shouldn't be manipulated. Then we need to find another solution for https://github.com/Expensify/App/pull/44981. The RCA for that issue is not totally clear.

If we look at the param in the URL path when selecting state, we'll see that &country=AQ (old country code) is there

Why do we even have that param in the backTo param if the original page didn't have it?
https://github.com/Expensify/App/assets/16493223/ea6d900c-5ebc-4f35-8b20-a533baa810d1
cc @nkdengineer

cc @mateuuszzzzz seems like the issue is not completely fixed

@s77rt This is last of 2 blockers, I think the issue that @nkdengineer PR fixed was less severe, user could still add the correct country and state afterwards I assume.
So maybe we can demote that one and revert the CPed change to fix this issue. To unblock a deploy

@mountiny I agree. Let's revert the PR

Ok I have raised the revert PR here https://github.com/Expensify/App/pull/45121 and going to demote the other issue

The PR will be QAed in staging


[heimdal] WIP new port

this is a work in progress, im working on this + cyrus-sasl + librdkafka for sasl support in the last.

[x] Changes comply with the maintainer guide.
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[ ] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx.
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[ ] The generated "usage text" is accurate. See adding-usage for context.
[ ] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

@microsoft-github-policy-service agree

Before reviewing any further: Why doesn't this port use vcpkg_configure_make etc.?

@dg0yt hey, I originally attempted to use vcpkg_configure_make and vcpkg_install_make but those did not work on my Mac due to configure.ac:21: error: possibly undefined macro: AC_PROG_LIBTOOL which was a bit weird as lib tool is installed (though on Mac its installed as glibtool not libtool).

Hm, it can't be the only package using that macro. And it is only used with AUTOCONFIG. So it is not a reason to avoid vcpkg_configure_make. Which we might really want to see used here because duplicating what it does is not maintainable.

I'll give it a shot, from a raw tarball configure and make build worked totally fine, it was only within vcpkg that the error occurred which was a bit strange.

Just don't use AUTOCONFIG if unsure. Use DETERMINE_BUILD_TRIPLET instead.

@dg0yt yep that worked, testing other platforms now

might not be needed due to https://github.com/microsoft/vcpkg/pull/38402


Add functionality to use check-in app for QR code scanning with eventyay

This PR introduces a comprehensive integration of the updated Eventyay-Tickets, enhancing support for both the Badge Plugin and Exhibitor Plugin.
Added/Modified Functionalities:

Login Page Role Selection

Updated the login page to allow users to select their roles, enhancing the flow for different user types (e.g., exhibitors, checkin-staff).

Device Registration with QR Scanning

Implemented QR-based device registration to streamline device setup for event check-ins and lead scanning.

Event Selection Page

Added a page for users to select specific Eventyay events that is categorised.

Exhibitor Authentication Page

Added an authentication page for exhibitors to log in and access lead scanning functionalities.

Lead Scanning Functionalities

Enabled exhibitors to scan leads, add tags, and attach notes to each lead.
Added an option for exhibitors to export scanned leads data in CSV format.

Check-in Functionalities

Added the capability to scan tickets and generate badges.
Users can choose to either print badges or download them.

This integration will provide a seamless experience for attendees, exhibitors, and event organizers, supporting more efficient event management and lead tracking.
Summary by Sourcery
Integrate updated Eventyay-Tickets with enhanced support for Badge and Exhibitor Plugins, introducing role selection, QR-based device registration, event selection, lead scanning, and check-in functionalities.
New Features:

Introduce role selection on the login page to enhance user flow for different user types such as exhibitors and check-in staff.
Implement QR-based device registration to streamline device setup for event check-ins and lead scanning.
Add an event selection page for users to choose specific Eventyay events, categorized for better navigation.
Enable exhibitors to scan leads, add tags, attach notes, and export scanned leads data in CSV format.
Add check-in functionalities to scan tickets and generate badges, with options to print or download them.

The Previous commit resolves #11 explaining both the exhibitor and checkin workflow


Blender 4.1 - QuickSnap 1.4.5 last version - EdgeMid Points Issue

Hi=) When I switch to mode 2 (edge midpoints) an error appears. On the screen there is an artifact square from the addon. and the quick menu remains on top.

Hi! Thank you for reporting the issue, could you try out the version here and tell me it fixes the issue?
https://github.com/JulienHeijmans/quicksnap/tree/4-0a-Fix (Download Link)

No=( same issue


Binary artifacts

Hi there!
We're looking at your SDK and trying to integrate into our app.
The problem is that we don't use CocoaPods, but static frameworks only.
Technically we can install the SDK manually via powerful cp -R dependency manager, but it looks like not a very good option.
The question is:
Are there any ways to get static framework instead of copying sources (manually or via CocoaPods)?

Hi @AlexDenisov - I think making a static option available is a great idea. Do you want to provide a PR with a build process for the static option, and then we'll use that to build each release, and host on Bintray?

@alexanderdean sounds great, I'll investigate and we'll see how can I help here!
Thanks for the fast response :+1:

No worries at all, sounds good!

Closing as duplicate of #171


Improve travis ci configuration

Use containerized (non-sudo) containers
Use container-provided node.js and io.js builds
Simplify

I don't fully recall the reasons for the current travis config, perhaps for io.js testing before Travis provided official support for it.
Nonetheless, this isn't working atm, although I'm not sure why.
Theoretically install and script could be removed too, as they match the language: node_js defaults, right?

In theory! But it turns out it has to use the objective_c runtime because it needs to run on a Mac to link the fsevents frameworks.

Oh right, that's what it was. :facepalm:
An older iteration used the multi-os config, but we really only need mac here.


Nvidia profile not working

Steps to reproduce:
Install podman, podman-compose and nvidia-container-toolkit
Run:
$ sudo podman-compose  --profile nvidia up -d
Error: default OCI runtime "nvidia" not found: invalid argument
Error: no container with name or ID "ollama_server_nvidia" found: no such container

podman requires CDI (container device interface) to communicate with nvidia cards. ATM void has a outdated version of nvidia container toolkit, already tried updating it, max version I could do was: 1.13.5


Add tag and template dataset card when pushing to Hub

It's great to see a library focusing on using local LLMs for synthetic data generation!
When doing the push_to_hub, it could be nice to add a template dataset card or some tags to the dataset so it's easier to find datasets created using promptwright. To add tags, something like:
from huggingface_hub import DatasetCard

def update_card_with_tags(repo_id: str):
    card = DatasetCard.load(repo_id)
    # Initialize tags if not a list
    if not isinstance(card.data.tags, list):
        card.data.tags = []
    # Add tags if they don't already exist
    tags_to_add = ["promptwright", "synthetic"]
    for tag in tags_to_add:
        if tag not in card.data.tags:
            card.data.tags.append(tag)
    card.push_to_hub(repo_id)

would already help with discoverability. Example repo with these tags: https://huggingface.co/datasets/davanstrien/promptwright-test. You could also add a more expansive dataset card using a template in the future. There is a nice example from distilabel of this kind of template: https://github.com/argilla-io/distilabel/blob/main/src/distilabel/utils/card/distilabel_template.md

Great idea, thanks @davanstrien  , will take a look!

Great idea, thanks @davanstrien , will take a look!

Awesome! Happy to review if useful :)

hey @davanstrien , I was thinking this would be a nice addition for you to add if you're up for it? if time is scarce, happy to pick it up. you seem to have figured out what needs doing anyhow, so makes sense for you to ship this.

we now have tags support in : https://github.com/StacklokLabs/promptwright/blob/main/promptwright/hf_hub.py


[flash_ctrl] Fix buffer enable / disable timing

fixes #16979
In the previou design, the buffer enable was allowed to change whenever the read pipeline was idle.  This however does not account for the scenario where on the same cycle there is a new flash read request.  In the buffer enable were changed from 1->0, it would allow a new request to match a buffer entry, while disallowing that buffer from responding in subsequent cycles.
Update the design such that when buffer enable is changed the request is back pressured during that same cycle.

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

still running some local sims to confirm.


Error when install dapr on a specific namespace then uninstall and install into another namespace

In what area(s)?

/area runtime
What version of Dapr?

1.3.0
Expected Behavior
Reinstallation of dapr should work
Actual Behavior
Deploying the Dapr control plane to your cluster...
❌  rendered manifests contain a resource that already exists. Unable to continue with install: ClusterRole "dapr-operator-admin" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-namespace" must equal "cc-service": current value is "cc-infra"
./k8s-1-deploy-infra.sh: line 44: SERVICE: command not found
❌  No status returned. Is Dapr initialized in your cluster?
Steps to Reproduce the Problem
(GKE cluster, kubernetes 1.20)
dapr init --wait -k -n cc-infra
dapr uninstall -k
dapr init --wait -k -n cc-service

(There is one thing in my deployment, I missed  installing the Jaege operator, but the config in my app was with "tracing")


Catch promise exceptions in processQueue.

Promise error handling is complex, this is a good example of such.
processQueue relies on .then() to continue process the queue
promises to resolve PromseWell::onComplete. If one of those
queue'd promises errors, the queue stalls and node-platform
based apps can hang on the server, leaving connections open
and users with a blank page forever (or until they hit 'stop').
👓 @phil303 @uzi

:fish: We should revisit what to do about the catch later... add a comment to that effect and we're good!


WIP: Noko

Merge isn't necessary until specs are completed.
It's just to keep you up to date.
This is the start of the solution to #33.
Transition is definitely worth it (front of V3 is fully nokified, that shows the benefits).

@paolobrasolin this does look good -- could we rebase and merge this first? We can split Noko work into separate PRs.

@ronaldtse Cleaned up, nokified V3 blocks (with green specs) and rebased. Ready to go!

@paolobrasolin feel free to merge it yourself 👍


Leaving IBM

User story
As an IBM.com user, I need to be notified when I'm leaving IBM.com domain so that I can understand the new context.
Notes

v18 documentation

This is a duplicate of issue (#3324).  Closing.  The work will be managed under the previously mentioned issue.


[ALLUXIO-2288] Fix javadoc in BlockWorkerClient.java

https://alluxio.atlassian.net/browse/ALLUXIO-2288
Update "Id" to "ID" in the javadoc

Can one of the admins verify this patch?

ok to test

Jenkins, add to whitelist

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11692/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11682/Build result: ABORTED[...truncated 2910 lines...]	at org.codehaus.plexus.classworlds.launcher.Launcher.launchStandard(Launcher.java:330)	at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:238)	at jenkins.maven3.agent.Maven31Main.launch(Maven31Main.java:181)	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)	at java.lang.reflect.Method.invoke(Method.java:606)	at hudson.maven.Maven3Builder.call(Maven3Builder.java:136)	at hudson.maven.Maven3Builder.call(Maven3Builder.java:71)	at hudson.remoting.UserRequest.perform(UserRequest.java:120)	at hudson.remoting.UserRequest.perform(UserRequest.java:48)	at hudson.remoting.Request$2.run(Request.java:326)	at hudson.remoting.InterceptingExecutorService$1.call(InterceptingExecutorService.java:68)	at java.util.concurrent.FutureTask.run(FutureTask.java:262)	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)	at java.lang.Thread.run(Thread.java:745)[INFO] [INFO] --- maven-jar-plugin:2.4:jar (default-jar) @ alluxio-tests ---[WARNING] JAR will be empty - no content was marked for inclusion![INFO] Building jar: /home/jenkins/workspace/Alluxio-Pull-Request-Builder/tests/target/alluxio-tests-1.4.0-SNAPSHOT.jar[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: java.io.IOException: Backing channel is disconnected.[INFO] [INFO] >>> maven-source-plugin:2.3:jar (attach-sources) @ alluxio-tests >>>
Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11695/
Test PASSed.

LGTM


Can't use RCTAppSetupDefaultRootView.appProperties to modify properties when Fabric is enabled

Description
can't use RCTAppSetupDefaultRootView.appProperties to modify properties when fabric enable
Version
0.68.0
Output of npx react-native info
System:
OS: macOS 12.6
CPU: (8) arm64 Apple M1
Memory: 124.52 MB / 8.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 18.9.0 - /opt/homebrew/bin/node
Yarn: 1.22.19 - /opt/homebrew/bin/yarn
npm: 8.19.1 - /opt/homebrew/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 16.0, macOS 12.3, tvOS 16.0, watchOS 9.0
Android SDK: Not Found
IDEs:
Android Studio: Not Found
Xcode: 14.0/14A309 - /usr/bin/xcodebuild
Languages:
Java: Not Found
npmPackages:
@react-native-community/cli: Not Found
react: Not Found
react-native: Not Found
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
nothing
Snack, code example, screenshot, or link to a repository
nothing

it still not work at 0.70.2


SCSt-GH-916: Configure Producer Error Channel

Requires: https://github.com/spring-cloud/spring-cloud-stream/pull/1039
Publish errors (returned and nack'd messages) to the error channel.
Resolves spring-cloud/spring-cloud-stream#916

merged via<PHONE_NUMBER>7a9b03326b3277ed24113178ebdb8e


Add warning box option before closing spyder

I would like ask that an option be put into spyder to enable a warning box before closing the application. This would not be enabled by default, but would add it as an option. Currently the individual console tabs have a warning before closing. I would like this functionality adding to the overall application. I have had several instances of closing spyder by accident while halfway through a 12-hour run of a script. I would like to be able to turn on a safeguard to prevent this from happening in the future.

That option is already available. You can find it by going to the menu
Tools > Preferences > General > Advanced Settings

and activating the option called
Prompt when exiting

Now found under Tools > Preferences > Application > Advanced settings

@Femkemilene, that's because you're using an old Spyder version. Please update.


DEFAULT.QC_Accessibility_Eservices - Defect # 495

Description:
Examine the outline view to ensure that:
Headings and Page Title are present,
The headings are nested correctly, and
The headings provide an effective overview of the contents of the page.
Expected:
Heading structure is correct.
Remember to close the outline view window.
Actual:
Headings incorrectly nested with two H1 headings.

Duplicate of #82


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 10:15:29 on 2023.04.15
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 6, errors out: 0
Original problem ID: 17636
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=17636

Problem has been resolved in 6m 0s at 10:21:29 on 2023.04.15
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 17636


Clean up config types

The big change is that config.Accumulated.Networks and .Directory are going away, and instead we will have a list of BVN names and a map from DN/BVN name to address. This particular change is necessary for DN to BVN mirroring and anchoring.

Eliminate RPC
Eliminate API.RESTListenAddress
Rename API.{JSON,}ListenAddress
Move website-related config into its own type
Move network type, id into its own type
Replace Accumulated.Networks and .Directory with Network.BvnNames and .Addresses
Add helpers in config for changing the port number

And update everything touched by these changes.
No functional changes. To verify, run the node like normal and test it.

Looks good to me.


Parallelize download for manifest v2 (fixes #236)

Needs some discussion before review.

Codecov Report

Merging #276 into master will increase coverage by 0.58%.
The diff coverage is 71.83%.

@@            Coverage Diff            @@
##           master    #276      +/-   ##
=========================================
+ Coverage   82.92%   83.5%   +0.58%
=========================================
  Files          37      37
  Lines        1698    1704       +6
=========================================
+ Hits         1408    1423      +15
+ Misses        290     281       -9

Impacted Files
Coverage Δ

hca/dss/__init__.py
87.44% <71.83%> (+0.8%)
:arrow_up:

hca/cli.py
67.82% <0%> (+6.95%)
:arrow_up:

Continue to review full report at Codecov.

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

@hannes-ucsc I looked into minimizing the diff. It doesn't seem like there is much to do actually to minimize (the diff looks a lot smaller on PyCharm). I had to do some refactoring in order to make test surfaces that made the most sense. The bulk of the "functional" changes are just around https://github.com/HumanCellAtlas/dcp-cli/pull/276/files#diff-f725c4fc41ac1c4f26ed32b556830bc7R297
I'm curious if you think this test is even worthwhile.

@jessebrennan It has a bigger diff because you need to pull from master.  It doesn't know about your last PR so it thinks those changes are from this PR.

@hannes-ucsc build is passing now. I'm not sure why status isn't updating to reflect.

For some reason, Travis is not updating the status check but the push build did actually succeed:
https://travis-ci.org/HumanCellAtlas/dcp-cli/builds/507935561?utm_source=github_status&utm_medium=notification


Don't have compatibility to keyboard

The calculator does not has the compatibility using the keyboard.
Example: Pressing some number do not reflect them on the screen
This functionality can help to improve fast calcs and accessibility

This pr improve the project structure and add this feature


hi

hello sorry i have same question
https://stackoverflow.com/questions/71950670/contest-platform-protection-from-bad-codes?noredirect=1#comment127141029_71950670

add

https://stackoverflow.com/a/72994165/8269581 is what I would also recommend here.