Analsysis: Should we allow (in a standardised way) checkbox support for individual data model fields for each alternative

Analysis description
See the description below. In grooming, there was a debate if we think this should be something we facilitate being possible.
Scope of analysis

Define pros and cons for doing this change
If cons outweigh pros; define if we have an alternative solution for same need

Analysis
TBA
Conclusion
TBA
Original feature request
How it is now
As it is, the checkbox component stores all selected alternatives in one single field of the data model.
However, some data models expect the choices to be stored in separate fields for each of the alternatives and this isn't possible with the current component.
I know that there is a work-around where you can make a new field in the data model for storing all choices made in the question before separating this into the different fields already existing in the data model through coding in CalculationHandler, but doing this for every such question adds a lot of extra hours to the work-load (especially considering that checkboxes are pretty commonly used in forms). For example, I'm currently working on a form with around 24 different checkbox questions, which means a lot of extra work and a lot of extra calculations that will have to run when using the app.
How I'd like it to be
I'd like the option to enter more than one data model field for storing, one for each alternative.
A suggestion
Developers are already given the choice of entering the alternatives manually or through a code list.
Maybe a similar choice could be made between storage in one or multiple fields?
If the default is to store in one, maybe a change like this would break less of the already existing apps?

Additional context
There are likely a lot of forms in A2 that are currently using data models that expect the results of checkboxes to be stored in the fields of the individual alternatives, and thus their data ingestion system (mottakersystem) is expecting this as well.
I imagine the extra cost to either fix their model and system for A3 compliance or configure the A3-apps for the current data models would be a pretty big issue for potential costumers.

I suspect this is a (fancier) duplicate of #273. Also, #681 is very relevant here. Moving to app-frontend-react.


three: support WPA3 networks

Will test if it works tonight. My rsnflags: 1160
Reference: https://developer-old.gnome.org/NetworkManager/1.26/nm-dbus-types.html#NM80211ApSecurityFlags
Also supports NM_802_11_AP_SEC_KEY_MGMT_SAE, might want to add that.

Related: https://github.com/commaai/openpilot/issues/25731

False positive, my network isn't WPA3


Avoid checking if a directory exists before creating it

Typically it makes sense to check if the directory exists before creating it, however, this invokes 2 commands to achieve this.
MKDIR outputs an error if the directory already exists, so instead, we can just suppress the error and let it do its thing.
So this below, as an example:
if not exist "!rootdir!\Videos" (
    mkdir "!rootdir!\Videos"
)
if not exist "!rootdir!\Audio" (
    mkdir "!rootdir!\Audio"
)
if not exist "!rootdir!\Images" (
    mkdir "!rootdir!\Images"
)
if not exist "!rootdir!\Documents" (
    mkdir "!rootdir!\Documents"
)
if not exist "!rootdir!\Presenation" (
    mkdir "!rootdir!\Presentation"
)
if not exist "!rootdir!\Executables" (
    mkdir "!rootdir!\Executables"
)
if not exist "!rootdir!\Disc_Images" (
    mkdir "!rootdir!\Disc_Images"
)
if not exist "!rootdir!\Compressed_files" (
    mkdir "!rootdir!\Compressed_Files"
)

Can be changed to this:
2>nul mkdir "!rootdir!\Videos"
2>nul mkdir "!rootdir!\Audio"
2>nul mkdir "!rootdir!\Images"
2>nul mkdir "!rootdir!\Documents"
2>nul mkdir "!rootdir!\Presentation"
2>nul mkdir "!rootdir!\Executables"
2>nul mkdir "!rootdir!\Disc_Images"
2>nul mkdir "!rootdir!\Compressed_Files"
2>nul mkdir "!rootdir!\Images"

If the directory doesn't exist, it's created. Otherwise, the error message saying it does exist is suppressed

makes it look a lot nicer as well, thank you!


Style Update: added card style

Pull Request Type
What kind of change does this PR introduce?
[ ] Solved an issue/task
[ ] Bugfix
[ ] Feature
[x] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
What is the current behavior?
Design a Simple Card Using CSS
Issue Number: [Insert Issue Number (if applicable)]
1
What is the new behavior?

Other information

BEFORE:

AFTER:

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Not generating ouput with multiple entries

Do you want to request a feature or report a bug?

What is the current behavior?
I need to define two pages with two different entry points, as suggested here https://medium.com/a-beginners-guide-for-webpack-2/multi-page-applications-4ae2ebfabc37
If the current behavior is a bug, please provide the steps to reproduce.
I am using jhipster's webpack, so this is my configuration

this is the configuration that fails
common:
https://gist.github.com/superandrew/9222e7cb3fc7abe15c5935cad29f8a74
dev:
https://gist.github.com/superandrew/5f65ab0d1a5b4c406d8fb1f3a7accbd0

What is the expected behavior?
I want the plugin to generate in /build/www two files, index.html and lab.html. index.html should contain chunks polyfills, vendor and main, lab polyfills, vendor and lab
The output is
Child html-webpack-plugin for "build/www/index.html":
     1 asset
    [./node_modules/html-webpack-plugin/lib/loader.js!./src/main/webapp/index.html] ./node_modules/html-webpack-plugin/lib/loader.js!./src/main/webapp/index.html 880 bytes {0} [built]
            factory:361ms building:865ms = 1226ms
Child html-webpack-plugin for "build/www/lab.html":
     2 assets
    [./node_modules/html-webpack-plugin/lib/loader.js!./src/main/webapp/lab.html] ./node_modules/html-webpack-plugin/lib/loader.js!./src/main/webapp/lab.html 676 bytes {0} [built]
            factory:148ms building:1113ms = 1261ms
    [./src/main/webapp/lab/assets/images/ng-book-2-minibook.png] ./src/main/webapp/lab/assets/images/ng-book-2-minibook.png 90 bytes {0} [built]
           [] -> factory:3397ms building:622ms = 4019ms
webpack: Compiled with warnings.

but no html file is generated.
The only way to have the plugin generate a file is this:
 new HtmlWebpackPlugin({
                template: './src/main/webapp/index.html',
                chunksSortMode: 'dependency',
                inject: 'body'
            }),

However this generates only the last one if I put two plugins, so I get with and index.html with the content of lab.html or index.html depending on who's last in the plugin array.
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
I'm on Mac Os 13, the browser isn't involved, and node is v8.

Thanks for your report. It would be great if you reduce your issue to a small reproducible example. Best put this example into a github repository together with instructions how to get to the problem.

there's trick patches, maybe helpful for you.
https://github.com/soulteary/webpack-custom-plugin

btw, if you wanna use full dynamics path in your project, you may hack the html-template, css backaground url by yourself.

@superandrew problem still exists?


Drawable mesh needs to be triangulated on export

If the mesh contains n-gons, mesh.calc_tangents() will throw an error. mesh.calc_loop_triangles() does not fix this error, so it must be triangulated either through bmesh or the triangulate operator (if its possible to do so without affecting the original mesh).

A possible hack for this is to apply the triangulate modifier, use bpy.context.object.to_mesh(bpy.context.scene, True, 'PREVIEW') to generate a new mesh without applying the modifier to the original mesh, and do the calculations after that. Once that's finished, dump the triangulate modifier.
I've not tested this but I'll give it a go later today.

I ended up fixing it by just doing
tempmesh = bmesh.new()
tempmesh.from_mesh(mesh)
bmesh.ops.triangulate(tempmesh, faces=tempmesh.faces)
tempmesh.to_mesh(mesh)
tempmesh.free()


cannot decode EAN8, code93, UPC-E !!!???

i used the zxing c++ version to decode the following images, but it can work for the images about EAN8, UPC-E, but i cropped the barcode ROI for the image to decode and it doesn't work, what's the problem? i am a freshman in this and i hope i can get some help from you! Thank s a lot for your help! The images and their ROIs are listed in the following, and my code also:
EAN8 image:

and its ROI:

UPC-E image:

and its ROI:

#include "zxing_decode.h"

const char* my_code_type[] = {
	"NONE",
	"AZTEC",
	"CODABAR",
	"CODE_39",
	"CODE_93",
	"CODE_128",
	"DATA_MATRIX",
	"EAN_8",
	"EAN_13",
	"ITF"
	"MAXICODE",
	"PDF_417",
	"QR_CODE",
	"RSS_14",
	"RSS_EXPANDED",
	"UPC_A",
	"UPC_E",
	"UPC_EAN_EXTENSION"
};

bool decode_by_zxing(cv::Mat& matGray, std::string& code, std::string& code_type)
{
	zxing::Ref<zxing::LuminanceSource> source(new MatSource(matGray));
	int width = source->getWidth();
	int height = source->getHeight();
	fprintf(stderr, "image width: %d, height: %d\n", width, height);

	//zxing::Ref<zxing::Reader> reader;
	//reader.reset(new zxing::oned::EAN13Reader);
	//reader.reset(new zxing::oned::Code128Reader);
	zxing::oned::MultiFormatOneDReader reader(zxing::DecodeHints::DEFAULT_HINT);

	zxing::Ref<zxing::Binarizer> binarizer(new zxing::HybridBinarizer(source));
	//zxing::Ref<zxing::Binarizer> binarizer(new zxing::);
	zxing::Ref<zxing::BinaryBitmap> bitmap(new zxing::BinaryBitmap(binarizer));
	try
	{
		zxing::Ref<zxing::Result> result(reader.decode(bitmap, zxing::DecodeHints(zxing::DecodeHints::DEFAULT_HINT)));
		//zxing::Ref<zxing::Result> result(zxing::MultiFormatReader().decode(bitmap, zxing::DecodeHints(zxing::DecodeHints::EAN_13_HINT)));
		code = result->getText()->getText();
		code_type = my_code_type[result->getBarcodeFormat().value];
		/*fprintf(stdout, "recognization result: %s\n", code.c_str());
		std::cout << "barcode type is: " << my_code_type[result->getBarcodeFormat().value] << "\n\n";
		*/
		return true;
	}
	catch (zxing::Exception& e)
	{
		//std::cout << "Error: " << e.what() << "\n";
		return false;
	}

}

thanks for your good guys!


Update urlchecks.yml

We are going to be deprecating older versions of urlchecker (and there was a change in versioning to match the upstream package so the version only appears earlier) so I wanted to update here to make sure your workflows do not break!
The updated versions also run about 7x as fast, so that's an added bonus!

Thank you!!! One down, 180 to go, lol! Bonking in myself for doing this bad version change, but I'll make it right! :laughing:

Ouch, good luck and thank you for being so careful!


Set speedUpDuration to 15 days in VeJoeStaking deploy

Noticed its currently set to 15 seconds rather than 15 days

LGTM, once merged I will deploy today. Do you think we need to add a function to update this variable? setSpeedUpDuration(uint256 seconds)?

Supporting dynamic speedUpDuration would require some additional logic changes and from speaking w/ murloc it seems like we're fine keeping it as static value


Heavy physics during sprint start

I have a physics actor (a car) that is set to 1350kg. Player is set to 75kg.
If the player walks in to the car, nothing happens. If the player hits the car running full speed, it's mostly fine.
If the player hits the car during the velocity action from beginning to sprint, the car bounces like a beach ball.
I have disabled basically all of the 'force scaling based on velocity settings' already in an attempt to fix the issue.

https://github.com/Sixze/ALS-Refactored/assets/26638814/232a8d63-23d3-4227-894d-0db033b7dc60

Could you remove the code inside the UAlsCharacterMovementComponent::ApplyPendingPenetrationAdjustment() function and test this again?
By the way, ALS works on top of the character movement component and is not responsible for such interactions.

Could you remove the code inside the UAlsCharacterMovementComponent::ApplyPendingPenetrationAdjustment() function and test this again?
By the way, ALS works on top of the character movement component and isn't really responsible for such interactions.

That appears to have fixed the issue with the sprint start punting the vehicle.

Could you create a smallest reproducible example of this issue and send it to me? Something like " here's a cube with physics enabled, and it flies away when the player touches it".

Could you create a smallest reproducible example of this issue and send it to me? Something like "here's a cube with physics enabled, and it flies away when the player touches it".

Yeah for sure, will get it over to you this week.

Sorry for the delay, will work on this ASAP and get it over to you. It might be specific to Chaos vehicles as well so will test that.

Could you create a smallest reproducible example of this issue and send it to me? Something like "here's a cube with physics enabled, and it flies away when the player touches it".

Spent some time trying to repro this without the use of the specific vehicle plugin in question, which is based on the legacy vehicle system (not the new chaos one) and doesn't seem like any issues outside of the plugin, so it's likely that the plugin has done something additional that is causing these issues.
(unless there's been changes to ALS since I last updated my main project - I was trying to repro using the latest version)
I'd consider it closed for now unless I'm able to repro without using another plugin.
Thanks for all you do!

If you are sure it is about the UCharacterMovementComponent::ApplyPendingPenetrationAdjustment() function, I can make it optional and disabled by default.

If you are sure it is about the UCharacterMovementComponent::ApplyPendingPenetrationAdjustment() function, I can make it optional and disabled by default.

Commenting out the code in that function did completely fix the issue in the video, just not positive what else that code might be handling otherwise.
I currently completely disabled player physics collisions with the vehicles for the time being.

I figured it out. The character mesh needs to be set to full collision rather than query only, then it will punt stuff during sprint start like the video I sent. Working on getting you a full repro. Also testing removing the collision from my project on the mesh to see if it breaks anything.

I added the bAllowImprovedPenetrationAdjustment setting to UAlsCharacterMovementComponent, so you can disable it to fix this issue.


Will there be any future updates for this?

title explainsqq

Future update for what??

like add the newest june updates to this if possible


add argument quoting to shell commands

related to issue #51

This change has a meaningful compatibility problem: build-args: ["-j32 -l32"] will be treated as a single complex argument, being wrapped and turned into cmake whatev '-j32 -l32' what ever. The solution is to change every project.yaml in existence, separating the build-args list into two elements. I've made that project.yaml change to the obi templates (cpp gspeak greenhouse growroom seabed).

Good catch on the backwards compat issue. We probably have a handful of similar issues in various project.yaml files.
How does this change interact with the debuggers map?  We specify those as strings which get prepended to the launch-args with a space:
debuggers:
  gdb: "gdb -ex run --args"
  lldb: "lldb --"
  strace: "sudo strace"
  apitrace: "apitrace trace"

OK, I've convinced myself re: debuggers that there shouldn't be a problem. You're still welcome to find a problem I missed of course.

The solution is to change every project.yaml in existence, separating the build-args list into two elements.

The resolution of #51 does not warrant changing every project.yaml in existence. Could we get a fix that's backwards compatible with the existing project.yaml files?

I could change it to not escape build arguments. I ran into compatibility problems escaping cmake-args, for example, so you'll see that they're not passed through the shlexquote function.
The way I see it, build-args: ["-j8 -l8"] is wrong and should be changed to build-args: ["-j8", "-l8']. I'm happy to make those fixes to project.yamls.

re: backwards compatibility, we could introduce a new key for build-args/whatever that does quoting properly, and obi could give some helpful spew in the case that the old key is detected in a project.yaml file. then, in a future major version of obi, we'd rip the band-aid, and accept either the old or new key but require quoting for both.


specify FluxCD addon install namespace using systemDefinitionNamespace

Description of your changes
specify FluxCD addon install namespace using systemDefinitionNamespace
Fixes #
I have:

[x] Read and followed KubeVela's contribution process.
[x] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[x] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested
helm install vele-core -n myns --create-namespace --set enableFluxcdAddon=true
Special notes for your reviewer
Perhaps adding extra value to specify FluxCD's namepsace instead?

closing as the changes have already been included, and fluxcd is now an addon in catalog repo.


SSL_CTX_set_keylog_callback not invoke after SSL_renegotiate_abbreviated and or SSL_key_update

SSL_CTX_set_keylog_callback function function is not call / invoke after SSL_key_update and or SSL_renegotiate_abbreviated!
Is this a normal behavior or a bug?
I'm running under openssl-devel-1.1.1k-6.el8_5.x86_64
Thank you,
Jean-Marc

Worth noting that a TLS1.3 key update, caused by SSL_key_update, calculates new traffic secrets based on the old ones, so logging these events isn't necessary to decrypt the subsequent data transmitted.
For TLS1.0-1.2 the SSL_renegotiate_abbreviated function uses session resumption, so being privvy to the keylog data from the original session should enable you to decrypt the traffic, if I understand it correctly.

May be I wrongly exposed the situation.
Wireshark does not decrypt anymore the flow after a key renegociation.  Wireshark use what is produced from the callback function.

De : Hugo Landau @.>
Envoyé : lundi 4 avril 2022 13:09
À : openssl/openssl @.>
Cc : jlamond @.>; Author @.>
Objet : Re: [openssl/openssl] SSL_CTX_set_keylog_callback not invoke after SSL_renegotiate_abbreviated and or SSL_key_update (Issue #18001)
Worth noting that a TLS1.3 key update, caused by SSL_key_update, calculates new traffic secrets based on the old ones, so logging these events isn't necessary to decrypt the subsequent data transmitted.
For TLS1.0-1.2 the SSL_renegotiate_abbreviated function uses session resumption, so being privvy to the keylog data from the original session should enable you to decrypt the traffic, if I understand it correctly.
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fopenssl%2Fopenssl%2Fissues%2F18001%23issuecomment-1087420322&data=04|01||fca880c1f9814e22879308da162b8d6e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637846673564458372|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=3riyXIP9NGQyrRkypbg9cXNpWKEJF4MmRKUxklgXreE%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAYP24NZ6G4LIQX2KWXQ6HW3VDLEVTANCNFSM5SFI5AXQ&data=04|01||fca880c1f9814e22879308da162b8d6e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637846673564458372|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=Gh6%2FpUjZv2KvUZ8wD5VjbSZU%2BJRMF%2BtFlTjtzAHzk2o%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

My guess is that this is probably a limitation of Wireshark. Though it's possible we could log under these circumstances anyway to help it, even if not strictly necessary.

I don't know how much work this would require but I have to admit that this is very cool to look a decrypted application data frame with Wireshark.
Keep me posted and best regards,
Jean-Marc

De : Hugo Landau @.>
Envoyé : lundi 4 avril 2022 14:35
À : openssl/openssl @.>
Cc : jlamond @.>; Author @.>
Objet : Re: [openssl/openssl] SSL_CTX_set_keylog_callback not invoke after SSL_renegotiate_abbreviated and or SSL_key_update (Issue #18001)
My guess is that this is probably a limitation of Wireshark. Though it's possible we could log under these circumstances anyway to help it, even if not strictly necessary.
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fopenssl%2Fopenssl%2Fissues%2F18001%23issuecomment-1087499400&data=04|01||1b200c4eb5004280331908da16379c1e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637846725332612576|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=B%2BAAKwSZWVFDMPx0IJzjh7dj7lNhsvWSfFeB0PljMwk%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAYP24NZ622MOZCJSLOA2SYLVDLOZHANCNFSM5SFI5AXQ&data=04|01||1b200c4eb5004280331908da16379c1e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637846725332612576|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=tTkDWJN6xSuy%2FoPBNRbno4On97RygC%2BhPPG9vDDSlbQ%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

The version 1.1.1 is not supported anymore. Closing. If this is still an issue with currently supported releases, please open a new issue.


Missing JSON opening brackets

Same test code as in https://github.com/noamgat/lm-format-enforcer/issues/80  with LMFE version 0.9.6, now occasionally generates JSONs with missing opening bracket, e.g.
"airports":["A", "B"],"cost_of_flight":1250}

Upon running your notebook I encountered the same issue in #80 again and found a problem with whitespaces.
https://github.com/noamgat/lm-format-enforcer/commit/71736596792c8c7d1fe35a65af562135100aef95
After that I successfully finish the loop.
Can you check if you still reproduce the error on v0.9.8?

I just retested it on v0.9.8 and was still able to reproduce the missing opening bracket problem

Very strange, I am unable to reproduce. The only error I get after hundreds of generations is when the generation ends due to num tokens and the object is not closed properly. Can you try to modify the notebook (set a certain seed etc) to consistently get this problem?

I can reproduce the error consistently by fixing the seed in the completion call as follows:
response = lm("", logits_processor=logits_processors, max_tokens=None, seed= 316)

My version of llama-cpp-python is 0.2.62.

I tried reproducing using your version and seed, and could not reproduce... The only times I got parse errors was when the output finished due to "length" reason (meaning max-tokens cut generation midway) and the JSON object wasn't finished

OK, I was able to reproduce the bug, thanks!
It comes down to the following problem:
self.decoder([6377])
'{"'
self.decoder([1, 6377])
'"'

It seems like the token 6377 decodes to {" on its own, but the sequence 1,6377 decodes to ". I am not sure why this happens, does anyone know?

OK, this is a bug with llama.cpp
from _interals.py:
def detokenize(self, tokens: List[int]) -> bytes:
        assert self.model is not None
        output = b""
        size = 32
        buffer = (ctypes.c_char * size)()
        for token in tokens:
            n = llama_cpp.llama_token_to_piece(
                self.model, llama_cpp.llama_token(token), buffer, size
            )
            assert n <= size
            output += bytes(buffer[:n])
        # NOTE: Llama1 models automatically added a space at the start of the prompt
        # this line removes a leading space if the first token is a beginning of sentence token
        return (
            output[1:] if len(tokens) > 0 and tokens[0] == self.token_bos() else output
        )

However, llama tokenizer doesn't always add a leading space, as is with 6377 token. This causes llamacpp to remove the { from the beginning of the output, generating your bug. Will open a PR for llama-cpp-python.

Please vote / comment on the issue so that it gets visibility and is merged soon.

Excellent, thanks for digging into that!

Merged in llama-cpp-python


GO evidence code with incorrect reference

Could you change any GO annotation with
ISM which has GO_REF:0000024 to GO_REF:0000050
in
legacy_go_annotations_from_contigs.gaf.tsv

Ansy ISS with PANTHER:***** to  ISM/GO_REF:0000050

Could you change any GO annotation with
ISM which has GO_REF:0000024 to GO_REF:0000050

I've done that with:
perl -pne 's/GO_REF:0000024\tISM/GO_REF:0000050\tISM/' legacy_go_annotations_from_contigs.gaf.tsv > legacy_go_annotations_from_contigs.gaf.tsv.new
mv legacy_go_annotations_from_contigs.gaf.tsv.new legacy_go_annotations_from_contigs.gaf.tsv

Ansy ISS with PANTHER:***** to ISM/GO_REF:0000050

There were only two of those, which I changed in emacs.


Cannot start jquery insert plugin

I installed all the dependencies using bower and the last release of the plugin, and get it this error
Uncaught TypeError: Cannot set property 'updatePlaceholder' of undefinedCore   core.js:73

This should be fixed as soon as #196 will be merged.
But before that, you can lock bower deps of medium-editor to 4.11.1. And you'll be able to use the medium-editor-insert-plugin.

Hello,
Hi there,
I tried locking as you said and actually set the insert plugin version down to 1.7 to resolve dependency issues. My  bower file now looks like this
{
"name": "dsl-generated dependencies",
"dependencies": {
"medium-editor": "4.11.1",
"medium-editor-insert-plugin": "1.7.0"
}
}
yet I still get
"Uncaught TypeError: Cannot set property 'updatePlaceholder' of undefined".

Im using a div to create the editor and not a textarea like so

Could you tell me the exact versions which should work together so i can isolate this bug?

Did you try to jump to 2.0.1 instead ?
I'm using medium-editor-insert-plugin @ 2.0.1 with medium-editor @ 5.0.0 without any problems. If you made some custom extension for medium-editor, you can easily update them following the Upgrade file.
After few tests, it seems that you need to lock medium-editor @ 4.10.1. Placeholder has been converted to an extension on the 4.10.2 (a bit too early I think)

Hi @j0k3r
Thanks for a very quick reply.
Ok i locked medium editor to 4.10.1 and insert plugin to 2.01.
But i still get the same error
Uncaught TypeError: Cannot set property 'updatePlaceholder' of undefined

Chrome inspector tells me its failing on the last line of the Core function
 function Core (el, options) {
        var editor;

        this.el = el;
        this.$el = $(el);
        this.templates = window.MediumInsert.Templates;

        if (options) {
            // Fix #142
            // Avoid deep copying editor object, because since v2.3.0 it contains circular references which causes jQuery.extend to break
            // Instead copy editor object to this.options manually
            editor = options.editor;
            options.editor = null;
        }
        this.options = $.extend(true, {}, defaults, options);
        this.options.editor = editor;

        this._defaults = defaults;
        this._name = pluginName;

        // Extend editor's functions
        if (this.options && this.options.editor) {
            this.options.editor._serialize = this.options.editor.serialize;
            this.options.editor._destroy = this.options.editor.destroy;
            this.options.editor._setup = this.options.editor.setup;
            this.options.editor._hideInsertButtons = this.hideButtons;

            this.options.editor.serialize = this.editorSerialize;
            this.options.editor.destroy = this.editorDestroy;
            this.options.editor.setup = this.editorSetup;

            this.options.editor.getExtensionByName('placeholder').updatePlaceholder = this.editorUpdatePlaceholder;  // <<---- FAILS HERE
        }
    }

Here's how i'm requiring the various dependencies (its a rails project but they are all getting loaded)
//= require jquery
//= require jquery_ujs
//= require bootstrap-sprockets
//= require cocoon
//= require typed
//= require medium-editor/dist/js/medium-editor
//= require handlebars/handlebars.runtime
//= require jquery-sortable/source/js/jquery-sortable
//= require blueimp-file-upload/js/vendor/jquery.ui.widget.js
//= require blueimp-file-upload/js/jquery.iframe-transport.js
//= require blueimp-file-upload/js/jquery.fileupload.js
//= require medium-editor-insert-plugin/dist/js/medium-editor-insert-plugin
//= require_tree .

here is my bower file
{
  "name": "dsl-generated dependencies",
  "dependencies": {
    "medium-editor": "4.10.1",
    "medium-editor-insert-plugin": "2.0.1"
  }
}

Actually ive created a standalone repo which reproduces this bug. You can see it here if it helps. Load up the index.html page in the browser with chorme inspect and you can see the error in the console.
https://github.com/robodisco/medium-editor-insert-plugin-bug

You misunderstood what I said :smile:
If you choose to use medium-editor-insert-plugin 2.0.1 you must use medium-editor > 4.10.1. And if you choose this option, I recommend you to use medium-editor 5.2.0.
Your bower.json should be like that:
{
  "dependencies": {
    "medium-editor-insert-plugin": "^2.0",
    "medium-editor": "^5.0.0"
  },
  "resolutions": {
    "medium-editor": "^5.0.0"
  }
}

The resolutions is here to force bower to use the latest of medium-editor because medium-editor-insert-plugin lock < 5.0.0

Haha sorry - yes I wasn't 100% sure what you meant in your earlier comment. Thanks for clarification. Good news its working.
Thank you for you help! :-)


[Bug]: Mutable stringset race condition

What happened?
#32650 introduced a race condition when stringset is getting rate (runner about to report metrics) and adding more values. In particular, on Dataflow legacy runer it may throw runtime Exceptions when reading here:
https://github.com/apache/beam/blob/4f37e53a355261dc0f697cdf7d551e43815e4e90/runners/google-cloud-dataflow-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/MetricsToCounterUpdateConverter.java#L105
This prominently happens on Beam 2.60.0 streaming pipeline using FileIO read from an unbounded input of file metadata, and files come in with different directories. In this way stringset add does not happen at once, and likely trigger the race
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Infrastructure
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

Let me know if this does not block 2.61.0.


Hole for Captive Nuts

Create a new hole type - captiveNut that looks like:

Need to handle square nuts too

Duplicate of Issue #39


Addition of Hamza Nawab as contributors

Hey @blazenn2
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

SignIn / SignUp in ClueLess Website .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.

@nikhil25803
This is my profile url: https://www.clueless.tech/profile/admin/64981254


--rm-cache-dir/--no-cache-dir - What exactly are they used for

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

I have written a GUI for youtube-dl and for the last couple of hours I have been experincing some '403: Forbidden` errors when downloading items from yt playlists.
After g searching for a while the most common suggestions I came accross was to run '--rm-cache-dir'.
I searched what exactly they are for but didn't quite understand the very few explanations I found.
When I run  '--rm-cache-dir' what does this mean for the end-user. what if I just run ' '--no-cache-dir' for every youtube-dl call?
A side-note question: I would very much to share my GUI program( not open source yet, to be soon I hope), where can I do that?

Search the youtube_dl source tree for *.py that contain the string .cache to see which extractors use the downloader cache, and hence what will be

removed if you use --rm-cache-dir, or
never cached if you use --no-cache-dir.

I see that in youtube.py it stores youtube-sigfuncs. What exactly are they and what are they for.
If I choose not to cache them, it will affect the speed of the download of youtube videos?

some youtube videos require a signature to be downloaded. This signature is dervied from a function in a /player URL. youtube normally caches these function definitions. If you chose not to use the cache, this /player/ page has to be re-downloaded each time. That is 1 additional request per video. There is no other impact.
The situation is similar for other extractors that use the cache

@pukkandan  will the user notice the any difference?

@hbtalha

will the user notice the any difference?

Yes;

performance
network usage


Add NTV-e

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x ] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

@BagCollector Read guidelines for next PRhttps://developer.trustwallet.com/add_new_token_image#image-requirements. For now added here https://github.com/TrustWallet/tokens/commit/0a3199eed2ab013439218a246d8234936c5f603b


PHP 8.2 Support

Looks like PHP 8.2 is producing this error:
Deprecated: mb_convert_encoding(): Handling HTML entities via mbstring is deprecated; use htmlspecialchars, htmlentities, or mb_encode_numericentity/mb_decode_numericentity instead.

I tried to fix it as the error suggests in a couple of ways, but the tests are breaking.  After my attempted fixes, the tests seem to expect HTML back, but are returning plain text, which is the whole point of package, so not sure what is going on.
An update for PHP 8.2 would be greatly appreciated.  Let me know of anything I can do to help.

This is fixed in https://github.com/soundasleep/html2text/pull/101


make imports from separate files instead of "react-bootstrap" entrypoint

As suggested by https://react-bootstrap.netlify.app/docs/getting-started/introduction#importing-components to keep apps lighter

Hi there! Support for this was added already: https://github.com/justinmahar/vscode-react-bootstrap-snippets#individual-import-snippets
All snippets for individual imports begin with rb:import:only followed by the component name.
For example, rb:import:only:button, rb:import:only:grid, etc.
Closing this for now, feel free to reopen if you meant something different!


Add a return code after the checks

Added a simple return code after all the checks, 0 if there is nothing to update, 1 otherwise.
I didn't put it behind a feature flag because I think the impact is minimal, but I can look into adding it if necessary.

Thanks for the contributing! Can you make it behind a feature flag? This library is used in a GitHub Action (see here). Return code 1 if there's no update would "break" it, I think.
Check out the flags in __main__.py. Looks like we would need to modify args.py and config.py

I'm not sure about the flag name; I'm open to suggestions if you have a better one 😅

maybe it should fail by default in "check mode", and not fail when using the -u option ?

@all-contributors please add @atchisson for code


Pusher Error 1006

Hi, I am using the Google Cloud Ubuntu 14.0.1. I have setup the peatio site and can create account and logon. I am using my own pusher account, but i kept getting error: "Pusher : Error : {"type":"WebSocketError","error":{"type":"PusherError","data":{"code":1006,"message":"Connection interrupted (0)"}}} application-89b9c8d2652301fa3c8f31f8e97d0d1f.js:156:1880".
Any idea why and steps to troubleshoot? Thank you.

you can use a a standalone server ruby implementation of the Pusher protocol like slanger

Thanks, i will try slanger now.

@chuakwee are you using

@chuakwee are you using the account

@chuakwee are you using your own pusher account? Did you specify pusher cluster?

Hi @dkkoval, i first timer to Rails and not sure where to change the code for adding pusher cluster, what i did is i have created the Pusher app defaulted to the first cluster.

@chuakwee I can help you. I am having a vast experience in Peatio, Crypto Currency exchange and trading platforms with FIAT currency based on Bitcoin, litecoin, Ethereum, XRP etc, Core Wallets and Daemons. I am having expertise in ICO ie Initial Coin Offering full system development, White Paper creation with full architecture and high level documentation, smart contract writing and many more skills.
My Skype : sushilraghav and email is<EMAIL_ADDRESS>

@chuakwee you can check our fork, we're upgrading lots of components there. You can also check our demo (contact me if you have any questions).

@chuakwee - You need to use Ubuntu 16.04 and use my repo:
http://github.com/scatterp/peatio
Its a one command install.

@dkkoval ，thanks for your work!  I find you added only one coin: ripple, do you will add other coins?eth, zcash, etc.

Hi @dkkoval and @scatterp , I will try both forks. Thanks and will close this issue.

@taylorshuang yes, soon we will work on adding other most popular coins.


Remove vendoring specific logic now that go 1.9 handles it for us

Previously, we used govendor +local to execute commands on the local keytransparency repo without diving into the vendor folder.  This is now the default behavior of the go tools as of go 1.9.

Codecov Report

Merging #828 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #828   +/-   ##
=======================================
  Coverage   45.56%   45.56%
=======================================
  Files          34       34
  Lines        2739     2739
=======================================
  Hits         1248     1248
  Misses       1302     1302
  Partials      189      189

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 ff7f3be...d8e61ba. Read the comment docs.


unable to DL from Bitchute.com

$ youtube-dl -v -F https://www.bitchute.com/video/cTaQFilUsGW6/
[debug] System config: ['--prefer-free-formats']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-F', 'https://www.bitchute.com/video/cTaQFilUsGW6/']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.10.8 (CPython) - Linux-6.0.8-100.fc35.x86_64-x86_64-with-glibc2.34
[debug] exe versions: ffmpeg 4.4.3, ffprobe N-51556-ge278500
[debug] Proxy map: {}
[BitChute] cTaQFilUsGW6: Downloading webpage
[BitChute] cTaQFilUsGW6: Checking video URL
[BitChute] cTaQFilUsGW6: video URL is invalid, skipping: HTTP Error 404: Not Found
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/sdb3/home/jd/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
return func(self, *args, **kwargs)
File "/sdb3/home/jd/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
ie_result = ie.extract(url)
File "/sdb3/home/jd/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
ie_result = self._real_extract(url)
File "/sdb3/home/jd/bin/youtube-dl/youtube_dl/extractor/bitchute.py", line 66, in _real_extract
self._sort_formats(formats)
File "/sdb3/home/jd/bin/youtube-dl/youtube_dl/extractor/common.py", line 1374, in _sort_formats
raise ExtractorError('No video formats found')
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Checklist

[ ] I'm reporting a broken site support
[ ] I've verified that I'm running youtube-dl version 2021.12.17
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar issues including closed ones

Verbose log

PASTE VERBOSE LOG HERE

Description

WRITE DESCRIPTION HERE

Test PR #31019 (comment there), or, for now, use --force-generic.


Allow moderators to add questions

Features implemented:

Categorized Question Bank:
I’ve created a Question Bank page where questions are organized into specific categories, making it easy for users to browse and practice. Users can also filter questions by category to focus on a specific topic.

Circuit Templates for Questions:
Each question in the bank includes a circuit boilerplate, which contains input/output probes and a pre-configured testbench. This gives users a ready-made setup to start solving digital logic problems.

Markdown Support for Questions:
I’ve added Markdown support for creating and displaying questions, allowing for rich text formatting, including diagrams and explanations, to enhance the learning experience.

Question Bank Moderation Role:
I’ve introduced a new question_bank_moderator role for selected CircuitVerse users, empowering them to add and edit digital logic questions. Only admins can assign this role, ensuring that content management is controlled.

Auto-Verification of Submitted Answers:
I’m using CircuitVerse’s testbench feature to automatically verify users’ circuit submissions against predefined inputs and outputs. Users progress will be stored so that he/she can continue the next time where he/she left off when he/she visits that question. In practice mode, I’ve disabled user access to the testbench, so it’s only used for verification.
https://github.com/user-attachments/assets/3ea8d0ef-8e38-473a-90f7-785ccaaffaec

Personalized Progress Dashboard:
I’ve implemented a personalized progress dashboard as part of the user profile, where users can track their submission history and progress. Users also have the option to make their dashboard public or private based on their preference.

@jaydip1235 tests are failing

@jaydip1235 tests are failing

Hi @tachyons , as per discussion with mentors @tanmoysrt and @VaibhavUpreti , i will add the test case later of all the use cases that i implemented, which is also added as a task in the project board for 2nd last week. But if its a priority for now , I can pull that task for the top. Please do let me know

https://github.com/CircuitVerse/Blog/pull/227#issuecomment-2364993855
cc @jaydip1235


persist=True not honored in st7735 device

Type of Raspberry Pi
RPi 3 B+ running Arch Linux Arm
Linux Kernel version
Linux alarmpi 4.19.25-2-ARCH #1 SMP PREEMPT Tue Feb 26 01:20:52 UTC 2019 armv7l GNU/Linux
Expected behavior
I have written a test program to learn how to use your libraries, test.py:
import time
import RPi.GPIO as GPIO
from luma.core.interface.serial import spi
from luma.core.render import canvas
from luma.lcd.device import st7735
from luma.lcd.aux import backlight
from PIL import ImageFont

serial = spi(port=0, device=0, gpio_DC=25, gpio_RST=27)
device = st7735(serial, width=128, height=128, h_offset=1, v_offset=2, bgr=True, persist=True)

gpio_LIGHT = 24
light = backlight(gpio=GPIO, gpio_LIGHT=gpio_LIGHT, active_low=False)

font = ImageFont.truetype("Roboto-Bold.ttf", 14)

light.enable(True)

# Box and text rendered in portrait mode
with canvas(device) as draw:
    draw.rectangle(device.bounding_box, outline="white", fill="black")
    draw.text((10, 50), "Hello World", fill="yellow", font=font)
    draw.text((10, 30), "Hello World", fill="white", font=font)

input()

light.enable(False)
time.sleep(2)

I expected that the graphics will remain on screen and backlight will turn off and stay off after python script concludes.
Actual behavior
I used persist=True in the device setup expecting that the image would stay on the screen, but it did not, the screen becomes blank white. I also tried adding a switch at the end to turn the backlight off expecting it to stay off, but again, backlight is ON and screen is white. Finally, I added a 2 second delay at the very end to be able to see if the backlight control was working. It was, but the screen turns white with the backlight ON after the script ends.
I think that the cleanup of the gpio pins, turning them to tri-state inputs, causes the active high backlight to activate due to a 3.3V pullup in the circuit on the HAT, and the RESET to appear to go low, which would reset the chip.
Am I using the persist option correctly?
Anything else I should try?
Thanks
-Ted

I really wanted to leave the backlight off on my display when not using it, and possibly someone else reading this does also, so here's a workaround using the wiringpi command gpio. Since the gpio cleanup in the luma.lcd devices only cleans up those pins that were accessed, using a system call with gpio doesn't get registered with the luma routines and stays how it is setup after the script ends. The new version of test.py sets up the backlight pin (24 in this case) as an output and turns it on before printing its graphics, and then turns the backlight off (leaving the gpio pin as an output) after the script ends.
test.py
import subprocess
import time
import RPi.GPIO as GPIO
from luma.core.interface.serial import spi
from luma.core.render import canvas
from luma.lcd.device import st7735
from luma.lcd.aux import backlight
from PIL import ImageFont

serial = spi(port=0, device=0, gpio_DC=25, gpio_RST=27)
device = st7735(serial, width=128, height=128, h_offset=1, v_offset=2, bgr=True, persist=True)

gpio_LIGHT = '24'
# light = backlight(gpio=GPIO, gpio_LIGHT=gpio_LIGHT, active_low=False)

font = ImageFont.truetype("Roboto-Bold.ttf", 14)

# light.enable(True)
subprocess.run(["gpio", "-g", "mode", gpio_LIGHT, "output"])
subprocess.run(["gpio", "-g", "write", gpio_LIGHT, "1"])

# Box and text rendered in portrait mode
with canvas(device) as draw:
    draw.rectangle(device.bounding_box, outline="white", fill="black")
    draw.text((10, 50), "Hello World", fill="yellow", font=font)
    draw.text((10, 30), "Hello World", fill="white", font=font)

input()

# light.enable(False)
# time.sleep(2)
subprocess.run(["gpio", "-g", "write", "24", "0"])

This doesn't work around the graphics being erased.

@Rippert - what is the output of: pip list | grep luma ?
One thing that has never been clear to me is why there is a desire to not get the controlling program to manage its clean-up on exit. By not cleaning-up properly, this leaves the display in an inconsistent state for the next invocation of any program that may want to use the device; admittedly, the luma libraries will always try and reset the device and re-supply the correct init sequence, so this is somewhat a moot point.
Based on the number of issues this has generated in the past, there is obviously a sentiment for this type of behaviour though. Clearly one way of achieving this (whilst keeping the clean-up on exit), is to keep the python program running with a long sleep in a loop. I suppose then there is an overhead in managing the program to run in the background.
Would be interested to canvas people's opinions in why the persist=True behaviour is desirable, and why the alternative is not.

$ pip list | grep luma
luma.core       1.8.3
luma.emulator   1.1.0
luma.examples   0.0.0   /home/alarm/src/luma.examples
luma.lcd        1.1.0
luma.led-matrix 1.1.1
luma.oled       3.1.0

In terms of why I want to use the persist=True option: For this specific case, after the cleanup the screen is left with the backlight on and blank. That's a useless and wasteful state. I either want the screen to be off, including the backlight, or to be on with something useful displayed on it. That is really the screen manufacturers fault, since the default really shouldn't be that the backlight is on, but I can see why they want their customers to see something happen when you boot up, so they don't just return perfectly good screens.
I know I could just leave the python script running, but I looked it up and there is a persist option, so I thought it should work.
There's nothing wrong with having your scripts automatically clean up, especially since you provide the persist option. But there are cases where users do want persist=True so I think it's important that it work. Now maybe my system is setup incorrectly, so I'm not trying to criticize, just to get it working.
In more general terms, I can think of using the screen as a sort of "whiteboard" where I print out some useful message about it's state every so often, and instead of having to write some interface code in a python script that I keep running, I might like to just have some pre-written scripts that say one thing or another and call them as I need to.
So I think your persist=False default is correct, but it's useful to be able to override that.

@Rippert what about pip list --outdated?

$ pip list --outdated
Package   Version Latest Type
--------- ------- ------ -----
luma.lcd  1.1.0   1.1.1  wheel
luma.oled 3.1.0   3.1.1  wheel
pip       18.1    19.0.3 whee

I went ahead and updated everything in the list. It didn't make a difference, which I thought would be the case looking at the commits between 1.1.0 and 1.1.1.
Thanks for pointing out the pip command. I'm just getting started in python, so it's good to know.


slash escaping

I've had difficulties verifying the signature of jws signed with this lib due to a difference in slash escaping with namshi/jose. They took care of the issue in this PR, but I have no idea which behavior is correct and which is not... is it possible to clarify it?

Thank you for reporting me that issue.
As I understand, you create JWT with that library and try to verify it with namshi/jose.
This library implements tests based on the RFC7520 and all these tests pass. I am not sure namshi/jose implement those tests.
Plus there is a known bug using ESxxx algorithms.
In any case, the bug may come from this library.
Can you indicate which algorithm you are using?
If you have some examples you can share, do not hesitate to send them to me.

in fact it's just the opposite: I'm generating JWS with this library and try to read them with namsh/jose, using RS512. I'll try to make a php showing the problem tomorrow.

I made some checks and I am certain the error comes from namshi/jose.
A claim iat is added in the payload when it is loaded with SimpleJWS. Because this claim is added, the signature becomes invalid for the new payload and causes the error.
There is no error if:

that iat claim is already set in the payload.
you use JWS instead of SimpleJWS

Ok thanks a lot! As I'm quite sure I would make many mistakes if I do it myself, would you mind reporting the issue on the namshi/jose tracker?

No I will not open an issue. This topic has already been discussed and I gave my POV (see https://github.com/namshi/jose/issues/19#issuecomment-72953334).
That is one of the reasons why I developed that library.

It was already reported here, closing


add touchAction property when using PointEvent

ref #1118, #1107, #1109, #1100

add touchAction property when using PointEvent
add utils.getTouchAction
update build files

@RByers thanks. I use your code snippet.

Looks great, sorry for the trouble this caused you and thanks for landing a fix quickly!


Updated mark down inline example with an alert example.

Updated code to include a mark down example for users of quickstart.  Addresses issue #203

Deploy preview
https://deploy-preview-206--quickstarts.netlify.app/


Revert "MAX-32464 Add log to check cost not report issue on zinc related instance"

Reverts ServiceMax-Engineering/langchain#2
The cost not calculate issue already fixed by https://github.com/ServiceMax-Engineering/help-doc-ai/pull/184, just revert to remove the log.
@shinxi, please help review.

https://github.com/ServiceMax-Engineering/langchain/releases/tag/v0.1.3 @benjaminliugang


freebayes v1.2.0 claims to be version 1.0.0

due to version_release.txt not updated when git is not availble in the  version is determined from this file
so make autoversion fails to set the correct version
my I suggest version_release.txt be updated when generating a release tag using
make setversion VERSION=X.Y.Z

I second that. It would be great to update the version number to its correct value.


Get Collection Name in RESULT

my environment running ArangoDB

[X ] 3.3

this is an AQL-related issue:
[x] Foxx
Is it possible to get the name of a collection in the RESULT?
I am using a workaround, but it costs a lot of performance.
This is my workaround:
COLLECT category = LEFT(v._id,FIND_FIRST(v._id, "/", true)),
edgeType = LEFT(e._id,FIND_FIRST(e._id, "/", true))
INTO groupCategory
Is there nothing more efficient than getting the collection name with LEFT?
After three hours of searching in the turorial and just trying I hope to get a solution from you guys...
Thanks a lot,
Tobi

The simplest way is probably to use PARSE_IDENTIFIER.

Thanks, I tried. It works, but after that the grouping does not work anymore. So I stay with the LEFT. Thank you anyways a lot!

Why shouldn't the grouping work anymore by using PARSE_IDENTIFIER?
COLLECT
  category = PARSE_IDENTIFIER(v._id).collection,
  edgeType = PARSE_IDENTIFIER(e._id).collection
INTO groupCategory

should work as well.

Oh, thanks a lot!!
That was a helpful question - I tried it again with PARSE_IDENTIFIER(v._id).collection
This one delivers exactly the same result as my version with LEFT(v._id,FIND_FIRST(v._id, "/", true))
I had tried with PARSE_IDENTIFIER(v._id) without the .collection, which also delivers results, but in a different structure.
Don't know, but if you paste things like this into the arangoDB manual, it might be helpful for many others, too, as this are the things that take a lot of time trying, especially if one is new to AQL. But thanks anyways, works perfectly!!

Please leave that ticket open, so we can add it to the documentation, and close it once this happened.


How to get node pointer to dom element?

I want to use canvas fill_rect() on a canvas element in yew. How can I do that?
I need to find a way to get the element's dom node pointer, right? But how?
e.g. html! { <canvas></canvas> }
How to get the pointer to that canvas element, in the yew component's update method, so I can call fill_rect() on it?

It seems you can use VRef, but I need to investigate this. I want to check yew with Ace editor and I think I'll know the answer after that experiment.

You should be able to do this using VRef and raw stdweb:
let canvas = web::document().createElement("canvas").unwrap().try_into().unwrap();
let render_ctx: web::CanvasRenderingContext2d = canvas.get_context().unwrap() // to actually control the canvas
let component = VNode::VRef(canvas.into());

@Kwarrtz Any news on this? This seems really interesting, but stdweb seems pretty intimidating in comparison with Yew...

The short story is: it wasn't as simple as I thought (surprise!).
It's been a while so I don't recall the details, but the gist of it is that I couldn't find a good way to maintain a link between the canvas element (the thing you return from your view function) and the service (the thing you make calls to to actually draw on the canvas). I played with it for a while but eventually decided I didn't understand the framework well enough to make a really viable solution and that the problem was better left to someone else.
If you're interested in picking up where I left off (or if someone like @DenisKolodin wanted to help me work through the kinks) I'd be willing to go back and try to work out the details of what I was trying to do and why I couldn't get it to work again.

I believe https://github.com/yewstack/yew/issues/665 would solve this issue, re-open otherwise :)


Protobuf version

Bug report
I have cloned the graph-node repo and when I try to run cargo build I get these errors
Using protoc version 3.6.1
thread 'main' panicked at 'Failed to compile Substreams proto(s): Custom { kind: Other, error: "protoc failed: Unknown flag: --experimental_allow_proto3_optional\n" }', graph/build.rs:20:10

if I just remove the experimental flags I get
thread 'main' panicked at 'Failed to compile Substreams entity proto(s): Custom { kind: Other, error: "protoc failed: codec.proto:44:12: Explicit 'optional' labels are disallowed in the Proto3 syntax. To define 'optional' fields in Proto3, simply remove the 'optional' label, as fields are 'optional' by default.\ncodec.prot

Where I see it uses proto2 syntax, so my question is which version of proto should I use to make it work?
Relevant log output
No response
IPFS hash
No response
Subgraph name or link to explorer
No response
Some information to help us out

[X] Tick this box if this bug is caused by a regression found in the latest release.
[X] Tick this box if this bug is specific to the hosted service.
[X] I have searched the issue tracker to make sure this issue is not a duplicate.

OS information
None

I am getting this error when trying to build:
  thread 'main' panicked at 'Could not find `protoc` installation and this build crate cannot proceed without
      this knowledge. If `protoc` is installed and this crate had trouble finding
      it, you can set the `PROTOC` environment variable with the specific path to your
      installed `protoc` binary.You could try running `brew install protobuf` or downloading it from https://github.com/protocolbuffers/protobuf/releases

I solved it by re-installing protobuf via brew https://grpc.io/docs/protoc-installation/ and then linking again:
brew link --overwrite protobuf


Icon Request: Cast/Sling

We could have variations on this one:
"Arm in Cast"

"Hand in Cast"

"Arm in Sling"

I've added these to NEW ICONS and started working on them. They'll get finished soon.


Publish to crates.io?

Hey y'all,
Love the work you're doing here. I'm looking to setup Powershell support for a project of mine. However, in order to publish packages to crates.io, I need all my dependencies to also be on crates.io.
Not to mention that it would make it easier for all other Rust-based projects to consume your work.
It looks like this repository is already setup with Cargo, so it should trivial to get it published. Is there anything I can do to make that happen?
Thanks, Elijah

Could we have the same for NPM?

I'd like to second this request -- I also have a project (zizmor) that could make use of a crates.io distribution of this tree-sitter grammar.

For those interested: I forked this repo and got the Rust crate working locally, at least for my purposes: https://github.com/woodruffw-forks/tree-sitter-powershell/pull/1
I would greatly prefer to upstream my work here, but worst comes to worst I'll publish tree-sitter-powershell from my own fork, which I'd be happy to hand over to the Airbus CERT folks if/when they want it.

Do it.

Do it.

OK. I'll likely publish the crate tomorrow, and I'll ping back here when I do.

Done ! https://crates.io/search?q=tree-sitter-powershell

Thanks @citronneur! Is there any chance you could re-generate the parser + Rust bindings to use the latest stable tree-sitter? The current crate uses ~0.22.6, which cargo won't allow in resolutions with any other tree-sitter language that uses 0.23 or newer.
If it would help, I can send a PR doing that regeneration for you -- it requires modifying the repo's metadata slightly, since newer versions of tree-sitter have their own JSON config file.

For context, this is the conflict that happens when two tree-sitter language crates depend on conflicting native library versions of tree-sitter:
    Updating crates.io index
error: failed to select a version for `tree-sitter`.
    ... required by package `tree-sitter-powershell v0.1.0`
    ... which satisfies dependency `tree-sitter-powershell = "^0.1.0"` of package `zizmor v0.6.0 (/Users/william/personal/devel/zizmor)`
versions that meet the requirements `~0.22.6` are: 0.22.6

the package `tree-sitter` links to the native library `tree-sitter`, but it conflicts with a previous package which links to `tree-sitter` as well:
package `tree-sitter v0.23.2`
    ... which satisfies dependency `tree-sitter = "^0.23.2"` of package `zizmor v0.6.0 (/Users/william/personal/devel/zizmor)`
Only one package in the dependency graph may specify the same links value. This helps ensure that only one copy of a native library is linked in the final binary. Try to adjust your dependencies so that only one package uses the `links = "tree-sitter"` value. For more information, see https://doc.rust-lang.org/cargo/reference/resolver.html#links.

failed to select a version for `tree-sitter` which could resolve this conflict


LLVM Strings (UTF-8-interpreted byte buffers)

Adding Stringss to the LLVM backend.
All strings are UTF-8 encoded; stdout-specific encodings are not respected.

3eef4a190e303c73146542057803088aab94383f will be squashed, 2fd4853cca79f4bab5771a6c680aabbf191d1114 still needs to be merged in.

Since we decided not to grow strings and need a reference count, the buffer representation will have to be changed drastically.

Were feature/llvm-doubles stable, the only non-accounted-for commit would be https://github.com/effekt-lang/effekt/commit/2fd4853cca79f4bab5771a6c680aabbf191d1114.

My recent force push should only have introduced master changes; all commits after the note were incorporated.

I will wait for this merge to complete before starting a new branch were 8475e7b27a2abf490aaf0cd15410a1762bc51140 will be reverted.


feat: 🎸 heatmap enhancements

Codecov Report

Merging #347 (101d580) into develop (28d5026) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #347   +/-   ##
========================================
  Coverage    86.89%   86.89%
========================================
  Files          550      550
  Lines         7081     7081
  Branches       713      713
========================================
  Hits          6153     6153
  Misses         842      842
  Partials        86       86

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 28d5026...101d580. Read the comment docs.


Aave strategy implementation

Modeled after tom's curve implementation, and should be landed afterwards.

I did not see any unit tests. Did you miss committing the file that has them?


Acala: Build a dApp with Acala EVM+ (Polkadot Wallet IT)

Acala: Build a dApp with Acala EVM+
Use at least one of the following features:
• Wormhole bridge
• Acala Predeploy contracts
• Polkadot wallet integration
Prizes: 1st: $2000, 2nd: $800, 3rd: $300

Picked-Up

• Polkadot wallet integration


No options under AnyShortcut in the plugins tab on Mac

Hi,
I am having an issue where no options show up under the AnyShortcut plugin on Mac. I have attached a screenshot of my issue.
Thank you for your help!

Hi, sorry for the late reply. It seems I don't get notifications for new issues on Github for some reason..
I don't have a Mac to test on myself, so I have not been able to test the bug. Can you please try with the latest release and see if you get any error messages?
If that is the case:

Copy the error messages and add them here.

Can you try replacing AnyShortcut.py with this version, stop the add-in and then run it again. You should get a prompt that (hopefully) shows two file paths. Please include that prompt here.

It should look something like this:

The add-in tries to add the following path to the "Deploy" path. Can you see if it exists?
/Neutron/UI/Commands/Resources/Camera/LookAt

In my case, it is:
C:/Users/Thomas/AppData/Local/Autodesk/webdeploy/production/857e9b22c3aee8f76d8c95036c0e01aeea4aa86a /Neutron/UI/Commands/Resources/Camera/LookAt

When I run the latest version of the script I get a new error message:

I also ran the new script you gave me and got this output:

I checked to see if the file path you provided was present on my system. It was not.
Thanks for your help!

Thanks for checking. It is as I suspected: The icons are not in the same locations on Mac.
I have updated AnyShortcut.py once again. Can you please check the message you get when running this file?
Would you mind searching for a folder called CheckBoxChecked in /Users/brianrosen/Library/Application/Support/Autodesk/webdeploy/production ? It should have PNG files in it, named something like 16x16.png

here is the output from the new script:

I do not have any PNG files in the folder you specified.
Thanks!

Thanks. The application directories seem to be much more spread out on Mac.
I think I have fixed everything, so please try AnyShort.py from the "next" branch :) .

AnyShortcut.py in "next" branch works!

Great! Thanks for helping me track it down. I have created release v1.0.1.


#27: Handle absence of the folder

#27

When there is no folder:

[ ] Attempt to save link must result in an alert
[ ] User should be aware that absence of links is caused by absence of the folder

When there is invalid path specified:

[ ] User should be informed that the path is wrong
[ ] User should be suggested to select the path again

If we take an existing folder and try to open it, the app crashes.
Meta and previews for existing links are not generated, only for new ones.
ACRA doesn't suggest the report dialog.


Implement sort on typed list

Sort implementation is borrowed from reflected list.
supports key and reverse argument.

Note:
sorted() signature returns a list. We can't modify it to return typed list until we allow the disable_reflected_list switch.

Testing the impact of zip refleak fix

The failures were caused by requirement on scipy.

That should be it.


139 | Word Break | C++

Title -
What will change -
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[ ] Improving Code
[ ] Improving Documentation
[ ] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[ ] Python
[x] C++
[ ] Java
[ ] SQL

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

I want to work on this issue

@ricky1812 done with the problem. Can you assign this to me

@ricky1812 done with the pr. Can u check?

@ricky1812 cool down for a while. We'll let others contribute too.

[x] Assigning to @vrakr19

@SSKale1 ok


Python 3.10 support

Change our code to support python3.10 again.
Import all typing stuff from typing_extension which backports typing features and in current python versions, reexports from typing.
Replace tuple unpacking in typehints by Unpack. This look a bit less nice but might be an ok trade-off.
We can revert this commit if we want to drop python3.10 support again

Should be merged after #460

@ckolbPTB
did you have a docker file for python3.10? I think we need stupid if case in the install script:

I am getting hit by a chicken and egg problem:
During build of the docker image, mrpro cannot be installed with python 3.10 as we currently install from github main (and there the min version is python 3.11)
This results in missing requirements later on
This results in the tests not passing.
So I have to split this into two PRs.
One PR will enable 3.10 compatibility
Second PR will enable tests for 3.10
...or we would have to fix the docker logic, but this would also be a second PR as I want to keep this one only python3.10 related to be able to just revert it later on
Any other suggestions? @ckolbPTB @schuenke @JoHa0811 ?

This should be merged as two commits to make undo of the python3.10 changes easier in the future.

Any idea why https://github.com/PTB-MR/mrpro/actions/runs/11451086815 is still running after ~6h ?

Seems to me a lot of work for something which is outdated and we will have to revert soon again. Would it not make more sense to try to overcome the problem of python 3.10 in colab? We are not the only ones complaining about this. Are there no options to upgrade python or run one of our containers in colab (maybe something like this: https://github.com/drengskapur/docker-in-colab)?

A few questions out of curiosity:

Any idea why https://github.com/PTB-MR/mrpro/actions/runs/11451086815 is still running after ~6h ?

I have to manually kill them or wait till they time out. There is a bug in pip that installing packages from '/' does not work and freezes. In docklerfiles, it is common to COPY to '/', run and remove afterwards. Took me a few tries to realize this is the issue.

Any idea why for the build of py310 and py311 the "Cached" value is ~7%, but for py312 ~28%? see here

No idea. One possible difference is how far previous runs for this task got.

In general, splitting the docker build is a good idea imo. However, it seems like it's not resulting in a faster build yet, no? Would this change with the next run?

It should. Here the pyproject.toml changed, which it will not do most of the time..

Any idea why for the build of py310 and py311 the "Cached" value is ~7%, but for py312 ~28%? see here

No idea. One possible difference is how far previous runs for this task got.

In general, splitting the docker build is a good idea imo. However, it seems like it's not resulting in a faster build yet, no? Would this change with the next run?

It should. Here the pyproject.toml changed, which it will not do most of the time..

Lets try adding a scope for each docker image (py10,py11,py12):
https://docs.docker.com/build/cache/backends/gha/#scope

Scope is a key used to identify the cache object. By default, it is set to buildkit. If you build multiple images, each build will >overwrite the cache of the previous, leaving only the final cache.

To preserve the cache for multiple builds, you can specify this scope attribute with a specific name. In the following example, the >cache is set to the image name, to ensure each image gets its own cache:

GitHub's cache access restrictions, still apply. Only the cache for the current branch, the base branch and the default branch is >accessible by a workflow.

This now also changed when dependencies are updated to newest versions.
We do no longer update on each push to main all dependencies.
Our normal PRs should not fail because a dependency is updated, this was a bit annoying in the past.
Instead, dependencies are updated if either the pyproject.toml is changed, or the VERSION is changed.
Long term, it would be great to get a dependabot-like action checking for requirements going.


[FEATURE] create Officer Record view

The current FIGMA design is available for reference, but it has not been finalized.
Requirements
Take the API data for an Officer Record and display it in a modal frame. Don't spend too much time on the presentation, because it's probably going to change.
Here are some fields that should be visible:

Name
Badge No.
Rank/Status
Department
Demographic info
Work history - list of departments
Incident reports - incident table with links to Incident Record view - (#135)

Looking to start on this, but not sure where to start.

[] Add functionality to make it possible to get officer from search
[] Link to Fill Incident Record when it's functional
[] Polish Final Design


Back button doesn't work

When you go to the homescreen of itrecyclingweek and press the back button, the isitrecyclingweek screen reloads instead of going to the previous screen, in my case the phone homescreen.

Thanks for using the app! Sorry it misbehaved on you.
Was this the flow to reproduce the bug?:

Open app for the first time
Set location
Arrive at home screen of the recycling app
Press "back" triangle
Expected: Phone home screen Actual: back on recycling app home screen

I was unable to reproduce the problem using those steps.  Do you know if there were other steps in triggering the bug that I missed?

I attempted to set the location but my location wasn't found (probably
because I'm not in the city of Appleton). So it stays at the location
screen and when I press back, it just reloads the location screen.

Ah! I see.  I neglected to sufficiently test the address-not-found scenario.
Thanks for pointing that out.  I'll fix that with the next version.


Update Switch Docs

Added a note concerning the switch using the button DOM element.

I heartily concur with the need for more documentation.  Perhaps when I'm on the other side of what I'm trying to do I'll write some, but if someone beats me to it in the mean time...

I heartily concur with the need for more documentation.  Perhaps when I'm on the other side of what I'm trying to do I'll write some, but if someone beats me to it in the mean time...


Canonical path to TLS key/pem

# The certificate/private key pair to use for TLS, which is enabled if both are set.
tls_key_path = "~/.cert/loopystudios-key.pem"
tls_cert_path = "~/.cert/loopystudios-cert.pem"

Attempting the settings yields:
2024-09-12T04:24:58.639322Z ERROR error getting canonical path to tls_key_path file "/Users/simbleau/git/nova_client/~/.cert/loopystudios-key.pem"

I assume that ~ is the problem. IIRC, that is normally expanded by the shell, not by the OS's FS layer.

Yes, ~ is the problem.
Maybe we want to resolve that specially with the 'homedir' crate, if it is present.

Might be convenient. But where does it end? :rofl: Having ~ might make one ask why we don't support ${HOME} too. Do we support this for all paths or just for some?
I think it would be a valuable addition. Could you come up with a PR?


SetOf(type) and other collection checkers

What about SetOf(type), like ArrayOf but for ES6 sets?
Maybe also MapOf(keyType, valueType)?
This also got me thinking about extensibility... for example, if I am using an a custom/library-provided collection type like Immutable.OrderedSet, is there some way I could define my own ImmutableOrderedSetOf(type) (which I could then compose with other built-ins like Optional or ArrayOf(), etc)?
Maybe extensibility could be provided with a 'custom' type checker, which lets you supply a callback to examine the value in a custom way, like this:
import {Custom} from 'decorate-this';

const ImmutableOrderedSetOf = Custom((value, type) => {
  return value instanceof Immutable.OrderedSet && value.every(item => {
    return item instanceof type;
  });
});

@param(ImmutableOrderedSetOf(String))
function (strings) {
  // ...
}

(I don't actually know if this would work, just an idea.)

I do eventually want to support custom types.  For now, I will probably just add SetOf, MapOf. I'd also like a nice way to support immutables..

I think this will get us by.
const ImmutableMap = CollectionShape(Immutable.Map, map => map.toObject());
const ImmutableListOf = CollectionOf(Immutable.List, map => map.toArray());

class T {
    @param(ImmutableMap({key: String, val: String}))
    doThing() {
    }
}

CollectionShape takes two arguments, a constructor/class and a transformation function. It returns a function which takes a type.

The transformation function must return a plain Object

CollectionOf takes two arguments, a constructor/class and a transformation function. It returns a function which takes a type.

The transformation function must return either an Array or an Object.
The returned value will be iterated over with a for...in loop
Each member of the value will be validated against the type

looks great


Empty Message Case

When I backed up all of my messages with retrieve_msgs.py, one of the msg variables in the
for loop on line 173 was:
msg = {
    u'user_id': u'',
    u'attachments': [],
    u'source_guid': u'',
    u'text': None,
    u'created_at': -62135596800,
    u'sender_id': u'',
    u'sender_type': u'',
    u'system': False,
    u'favorited_by': [],
    u'avatar_url': None,
    u'group_id': u'4745311',
    u'id': u'137584906799629655',
    u'name': u''}

Because of the negative created_at field, the conversion to a datetime failed at line 179 in retrieve_msgs.py.
created_at = datetime.datetime.fromtimestamp(msg['created_at']).strftime('%Y-%m-%d %H:%M:%S')

Not sure if this is a problem with GroupMe's API or an unhandled message case or something else.
Anyways, thanks for sharing!

I submitted the patch. Thanks for pointing it out!


Windows Export should warn if custom resources are specified but rcedit isn't

Godot version:
3.1 stable
OS/device including version:
Windows 10 x64
Issue description:
When specifying application values in the Desktop target, no indication is given inside the editor itself that these resource values can't be assigned to the exported executable without setting the location to rcedit in the editor settings.  Of course, there is documentation explaining this, but the editor itself doesn't stop you from inputting these values nor does it give any indication that these values won't be applied if rcedit isn't there.

I found out there's already an issue about this (https://github.com/godotengine/godot/issues/18532), so I'll close this one as a duplicate. Thanks for reporting it :slightly_smiling_face:


Tracking

Is there any way we could add a feature or create a tracker for google quiz submissions? I know sometimes we can submit them incorrectly or forget whether or not we had submitted it or just viewed it. Google quizzes allow for a toggle switch to be added at the end where you can indicate you want your answers emailed to you based on the earlier email input question. This could just be a great help in checking that we submit all of the quizzes properly/don't accidentally forget one! Or printing the results to an excel and allowing us to view only an auto-updating sheet with the quiz number and name included in the submissions or something. Either way I think this could be a great addition!
Thanks :)

This is a great idea! I can look into these and add them to future quizzes

@DaniKerich I have added the feature that allows you get a copy of your responses emailed to you!
Great suggestion!


write buffer before running :Neomake

To be able to ghcmod/hlint with single ,hc/,hl command without having to :w to write the buffer first.

Hm, I'm worried that this might cause surprises. If someone believes that they can check the syntax of an unsaved buffer and it causes the file to be changed it might be worse than just failing to check. Can you remind me what the existing behavior is? Does it provide a clear message saying to save the file first?

Is there a way to check without saving first?
Maybe I stupidly missed something, when I want to run ,hc/,hl vim will give me warning that buffer is not saved and do nothing.
This change is the only solution I know to be able to run ghcmod/hlint within vim.

I think you do have to save the file before ghc-mod will work. I usually save manually every so often and then use ghc-mod on the saved file. I guess it's kind of annoying but I got used to it. Doing an auto file save during linting seems unexpected, like I would be surprised if an IDE did that. Thoughts?

I do understand your point. Will close and this PR then.


Architecture as Code: Onsite Accelerator - London 2024-03-12

Signup
As mentioned in the Working Group call on 2024-02-27 @ojeb2 has kindly offered to host a second onsite accelerator. Please comment with your name and company on this issue if you would like an opportunity to attend.
When Tuesday 12th March 2024, 15:00 - 18:00
Where City of London (address will be confirmed to attendees)

Miguel Gasca/Morgan Stanley

Pratik Thanki / Trayport

Alex Armstrong / Morgan Stanley

Aidan McPhelim / Morgan Stanley

Matthew Gardner/Morgan Stanley

Matthew Bain / Morgan Stanley

Yan Tordoff / Morgan Stanley

James (Jim) Gough / Morgan Stanley
-- James only if you need ID 😄

Aldwin Saugere / Morgan Stanley

Event is taking place tomorrow as planned. All folks on this thread have been registered and should have received details on the location.

Minutes
On Tuesday 12th January we were very kindly hosted by @ojeb2 at the London Stock Exchange Group in Paternoster Square.

For the first part of the meet-up we recapped the progress that has been made in the last couple of months, where we seem to be on a great upwards trajectory of contribution and development.
@rocketstack-matt first gave an overview of the migration of CALM to be a JSON Meta Schema and associated vocabulary and the rationale for doing do, namely it enabled the development of new schemas as patterns. @jpgough-ms delved deeper into how this enables us to develop architecture patterns that are far more opinionated rather than being mere base schemas to be extended and we were able to live demo the cool code completion features this gives us in JSON aware IDEs such as IntelliJ and VS Code.
Next up we had two cool demos showing the fledgling visualisation capabilities. First @aidanm3341 showed us his native rendering of CALM via a VS Code extension.

Then @Budlee was able to demonstrate the new translator service mapping CALM to the well used C4 model.

Next on the agenda was a discussion around Composable Interfaces - in other words how could we combine architecture specifications into more complex interacting schemas. The initial discussion was focused around the existing api-pattern and how it is in effect an architecture description of systems that in themselves should have complex CALM representations.

There was a long and detailed discussion, far too long to capture here, but you can see the full detail of what was captured for consideration on this issue.
Following a brief hiatus to grab coffee, we entered into a discussion on the need to start looking at the control requirements aspect of architecture. The control requirements are the aspects of policies and governing bodies for architectures that are to be built / deployed to ensure they fulfil necessary regulatory, audit or firm standards requirements.
@rocketstack-matt was able to give a little background on a conversation we had with another partner organisation the week prior where we were fortunate to be given a behind the curtains peek at some work that company has already done on capturing the control requirements and policy aspects of architectures as code. Whilst we can't reveal any more in the minutes at this point in time we're hopeful that partner org will be able to come to a future meet-up soon to reveal more and perhaps even some contributions to bootstrap this aspect of the architecture model.
As we neared the end of the event, we moved to @jpgough-ms to walk through another proposal for a new feature, this time that of a new CLI capability which would allow for the rapid bootstrapping of architecture instantiations and could also act as the foundations for a broader array of CLI functions.
The final sections of the meet-up led to a call for continued commitment and finally a brainstorming session where we captured ideas for new features and folks signed up to write up the issues for them ahead of the next working group - you know who you are!

As we wrapped, we were super fortunate to be meeting on the same day as LSEG were also hosting the London Enterprise Technology Meet Up where we were able to decamp for a drink and debrief before some fascinating insight into the Digital Operational Resiliency Act (DORA), but we'll leave that for another post!
Thanks again to @ojeb2 and LSEG!

Great write up of the event

added https://github.com/finos-labs/architecture-as-code/issues/84 to capture the Data Catalog / Lineage integration issue from the action board.


METRON-522: Need to mandate Client installation on Metron Host

Added metainfo.xml dependency on HBASE_CLIENT.  Modified service_advisor to force colocation of metron_parsers with ZOOKEEPER_CLIENT and HDFS_CLIENT, along with metron_enrichment colocating with HBASE_CLIENT.
Given that the enrichment master has to be colocated with parsers master in the current setup, I didn't duplicate the constraints there for brevity, although if anybody has a preference towards explicitly adding them in code (beyond the comments mentioning it) I could be pretty easily persuaded.
Tested by spinning a cluster as described in https://www.evernote.com/shard/s530/sh/c5551fbd-0ac1-4861-89ce-9c5e37065c52/b13e05f39eaac1a6
After setup, the cluster had to be modified to ensure each message could be hit in the appropriate location (e.g. ensure a given node has Kafka Broker + HDFS client, but not ZK client, etc.)

+1, looks good!


Fix example return_after_run

First crash
Example return_after_run is currently crashing when closing the first window:
     Running `target/debug/examples/return_after_run`
Running first App.
Running another App.
thread 'main' panicked at 'Could not set global default tracing subscriber.: SetGlobalDefaultError { _no_construct: () }', crates/bevy_log/src/lib.rs:83:22
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

The issue is that as the default plugins setup a global tracing subscriber, doing it a second time fails. https://docs.rs/tracing/0.1.22/tracing/dispatcher/fn.set_global_default.html for more information
My fix is to ignore errors, as it just means that there is already a subscriber configured.
As an added bonus, the user can now set up their own subscriber however they want.
Second crash
The example was still crashing:
     Running `target/debug/examples/return_after_run`
Running first App.
Running another App.
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', crates/bevy_winit/src/lib.rs:343:66
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

After some investigation, it's because the event loop for the second app receives an event Focused(false) from the window of the first app. It can be reproduced when using only winit, and looking at how it works it may be platform dependant (I'm on macOS) and events are just translated from the OS, so not something fixable.
My fix is to ignore Focused(false) events for unknown window, as they are pretty much meaningless anyway. I kept the crash for a Focused(true) event on an unknown window.
It runs!
Now it works :)
     Running `target/debug/examples/return_after_run`
Running first App.
Running another App.
Done.

I had the same errors while doing integration tests (#1057). I've had not submitted a PR for the subscriber thing yet because I used the workaround of not including the LogPlugin in my test. Still, I checked a bit the documentation of the tracing library to see if there was something that we could to fix the set_global_subscriber problem.
I don't think silently ignoring the subscriber errors is a good idea long term. We could still merge this but open an issue to make it better later. I think a have three options.

Add an option in LogConfig to disable setting the subscriber. It would definitely work and is pretty simple but it feel weird since you the resources can invalidate the LogPlugin completely. Then, could we just prevent adding the LogPlugin then? Well, that is annoying for the user since it cannot use DefaultPlugins. Finally, the resource need to be added before the LogPlugin since the subscriber is set on the plugin build step and not in a startup system.
Put the LogPlugin behind a feature flag. You can then disable the logging if you application encounter the bug. The problem is that you probably still want logging in tests or in your application. Maybe that would mean that you would set the default subscriber yourself before building and running your apps.
Have a tighter integration between BevyLog and BevyTask (if I remember correctly). The idea is to set the created subscriber as a default for each thread used by bevy. BevyLog can also set it for the main thread. Then, when thread a created in BevyTask, you need to get the dispatcher on the current thread and set it on each new thread that is created. This is my favorite solution. I found that tokio was doing something like this but they remove all logging last year. It is the only solution that would allow different bevy app to have different logging config/setup. However, I'm not sure about the performance. Is it better? Is it worse? Also, logging wouldn't work on user created thread.

I hope that some of that make sense.
Next, let talk about the window id issues. I see that you fixed it only for the Focused event. My question would be: what does it mean to receive an event for another window? Should we do anything? I don't think so. That why my PR (#1072) sanitize event for all window event. The downside is a slight degradation of performance on events that would get the window object. I also work on the assumption that all event could be received for the wrong id. I may be wrong about that. If we are convinced that only the Focused event could be received with the wrong id, then you solution has merit. (Maybe I would recommend add my settings to let the user choose between panic and ignoring).

You're right about logs, I changed the PR:

improved the error message in case of error telling what to do
example now disable the LogPlugin for the second run, and shows that log is still working

That looks good to me for the subscriber thing. I didn't know about add_group_with.
As for the window id problem, I'll let other comment on it since I'm partial to my own fix.

This looks good to me. @refnil  I prefer "one way that works by default" instead of given people too many choices. I'm going to merge this because 0.4 is dropping soon. But I do the approach in your PR to factor out the winit window fetch. It cuts down on duplicate boilerplate and ensures we ignore events the same way everywhere. If you can rebase your changes on master (after I merge this pr) and then remove the configuration option, I'd love to merge those changes (although i won't block the 0.4 release on it).


[stable/prometheus-cloudwatch-exporter] add extra annotations and fix serviceAccount

Signed-off-by: ephur<EMAIL_ADDRESS>What this PR does / why we need it:
This PR addresses issues with the service account being improperly configured.
This deployment also offers the ability to add additional annotations to the deployment. These may be needed for a variety of different reasons. For my use case it allows for handling by some mutating webhooks.
Which issue this PR fixes

fixes #17733

Special notes for your reviewer:
@gianrubio
@torstenwalter
Checklist

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test

looks good. Could you update the changelog?

@ephur
Would you mind to resolve conflicting files please? Thank you.
stable/prometheus-cloudwatch-exporter/CHANGELOG.md
stable/prometheus-cloudwatch-exporter/Chart.yaml


Docs for using output with ObjectDetector

🐛 Bug
I'am trying to use FiftyOneDetectionLabelsOutput output for Object detection but i got the following error
model = model_type.model(backbone=backbone, num_classes=num_classes, **kwargs)
TypeError: model() got an unexpected keyword argument 'output'
How can i correctly setup the output?
To Reproduce
from flash.image.detection.output import FiftyOneDetectionLabelsOutput
from flash.image import ObjectDetector
out= FiftyOneDetectionLabelsOutput(threshold=0.7)
objDetc=ObjectDetector(num_classes=81,backbone="medium",head="yolov5",output=out)
Expected behavior
Class init correctly as described in documentation
https://lightning-flash.readthedocs.io/en/latest/api/generated/flash.image.detection.model.ObjectDetector.html#flash.image.detection.model.ObjectDetector
There is a parameter output in the description. Maybe is an old value
Environment

OS (e.g., Linux): Linux
Python version: 3.8
PyTorch/Lightning/Flash Version (e.g., 1.10/1.5/0.7): 1.10 / 1.5.8 / 0.7.4
GPU models and configuration: cuda 11.3

Ah, thanks @eric-tc, looks like it just needs to be removed from the docstring here: https://github.com/PyTorchLightning/lightning-flash/blob/61e1a2d3b72f8fbbffe6ace14fb5b5bb35c5f131/flash/image/detection/model.py#L36
@eric-tc would you be interested in making a PR to contribute the fix?

Hi @ethanwharris,
yes I am interested. How I need to do the PR?  I need to follow this guidelines https://github.com/PyTorchLightning/lightning-flash/blob/master/.github/CONTRIBUTING.md Thanks


UI: Fix Table Headers in All Modules

[ ] Implement a solution to wrap all of the table headers across all data tables in all of the ECMPS modules so they look like the table headers in the screen shot below

Private Zenhub Image

Note to Tester: Please check tables across the entire site and make sure they look okay.


Added fakes for GDTTransport.

Discussed issue with @mikehaney24 .
We need a fake for GDTTransport in our tests and hence, sending a PR for the same.

Closing this PR. Created a new PR with the renaming fixes.


pwsh.exe shim from .NET tool install is not signed

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
dotnet tool install --global PowerShell
Expected behavior
`%USERPROFILE%\.dotnet\tools\pwsh.exe` should be a signed file

This is particularly needed for the .NET SDK container images which [install PowerShell as a global tool](https://github.com/dotnet/dotnet-docker/blob/6ba62d6825b9f45b3d665ab6cc930808a40d63c6/src/sdk/8.0/nanoserver-ltsc2022/amd64/Dockerfile#L39-L47).

Actual behavior
`%USERPROFILE%\.dotnet\tools\pwsh.exe` is unsigned

Error details
No response
Environment data
PowerShell 7.3.8

Visuals
No response

There is documentation for how this shim can be predelivered rather than generated at install time. This would allow PowerShell to deliver a signed version of it.
https://github.com/dotnet/sdk/blob/main/documentation/general/signing-global-tool-packages.md

cc @adityapatwardhan


feat: 🎸 added option to handle prev/next button in the browser

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

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

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
closes https://github.com/ngneat/bind-query-params/issues/13

Please add test

@NetanelBasal done

@shaharkazaz please check this

:)

@akaNightmare didn't forget that, I'm on vacation 😀
I'll review once I get back


EQCSS

What css-element-queries is better than https://github.com/eqcss/eqcss (by CSS specs)?

pragmatic vs over-engineered.


fix: incorrect import of KeyboardKeyPressHandler.h

Hi!
First of all:
Thanks for all your hard work on this library.
It is a crucial component for us to make our apps properly accessible.
We used version 0.2.1 for a little while, but after upgrading to 0.4.2 I ran into the following issue:

Seems like the imports of the KeyboardKeyPressHandler.h were incorrect.
Applyling the changes as shown in this PR solved the issue for us!

No rush! We've applied these changes locally using patch-package.
So we're able to build and continue our work.
I understand that you need to properly test these changes against the latest RN version etc.

If you update your example podfile to use use_frameworks! :linkage => :static you can this this error to happen.
This is a requirement of all react native apps using react native firebase. See docs here: https://rnfirebase.io/#altering-cocoapods-to-use-frameworks
You'll also need to make sure the GameController.framework is included in your Link Binary With Libraries list under Build Phases, otherwise you get an undefined symbol compiler reference to the GCKeyboard.h import in the RCA11yModule.mm file

Hello @gp3gp3gp3,
Thank you for suggestions, I will check it in some days. It makes sense, and I probably will add some changes.

Hello @boazpoolman, @gp3gp3gp3 ,
I reproduce the issue, and I would like to say that you are incredible) I will add your cahnges, and update some logic to be sure that the issue is fixed. Thank you a lot.
This lib has been created after one project, and since then, I didn't use it in the production. I am trying to improve and add new functionality based on different ideas and issues from RN repo, and your help is really appreciated.

Thank you a lot)


is there any API made by u for getting awesome

Do you mean that a web service with the ability to provide awesome list in JSON format?
If yes, no for now.
But you can run this module to generate awesome list in JSON format, like
$ npm run awesome # To get the latest awesome list.
$ npm run build # To get whole links under every awesome.

Thanks.

yes , web service....