TextDiffusers: question about Mario-LAION annotations

Hi,
I found some wrong annotations with misplaced bounding boxes in Mario-LAION after I downloaded the images in the url.txt.
Here are several examples:

Are these the examples with incorrect annotations, as mentioned in your paper, or do my downloaded images differ from the ones you used for the annotations? I want to check if my data processing is correct. And could you share more about the filtering? Are you using another ocr model to check if each bounding box contains the annotated text?
Thanks.

Thanks for your attention to our work! Could you send us the index (xxxxx_xxxxxxxxx) of samples that contain incorrect annotations?  I will check those samples then. The filtering rules are illustrated in Appendix E and no more ocr models are used.

Thanks for your response.
The examples above are randomly sampled; I forgot to keep the index :(
But could you check on these examples?
00000_000003058

00000_000004343

The images are downloaded with this command img2dataset --url_list=mario-laion-url.txt --output_folder=laion_ocr --thread_count=64 --image_size=512 following your README.

Thanks for your feedback. It is a mistake and the command should be:
img2dataset --url_list=url.txt --output_folder=laion_ocr --thread_count=64  --resize_mode=no

We will fix it in the readme file. Thanks!

I see. Thanks for your response. Additionally, could you kindly inform me when will Mario-10M be released?

It is hard to say and may take some time. Please stay tuned ;D

It is hard to say and may take some time. Please stay tuned ;D

hello, can you provide the training code for LayoutTransformer? I found that the text bbox is 4 points, i.e. [(x0, y0), (x1, y1), (x2, y2), (x3, y3)], however, the released layout transformer use [x, y, x, y]. Can you provide more detail for training LayoutTransformer?

Thanks for your feedback. It is a mistake and the command should be:
img2dataset --url_list=url.txt --output_folder=laion_ocr --thread_count=64  --resize_mode=no

We will fix it in the readme file. Thanks!

Hi,
So, are the provided OCR labels measured in original resolution of the image?

if i use this commond:img2dataset --url_list=url.txt --output_folder=laion_ocr --thread_count=64  --resize_mode=no

Hi! Thanks for your good job!  If i use this commond:"img2dataset --url_list=url.txt --output_folder=laion_ocr --thread_count=64  --resize_mode=no", where should  i resize the ori_image ? in the train.py ? (i find the size of the ori_image should be (512,512)). @JingyeChen

We notice that there exist a few samples with mismatched annotations caused by the resize operation. We will fix it within one week.

Hi! Thanks for your good job! If i use this commond:"img2dataset --url_list=url.txt --output_folder=laion_ocr --thread_count=64 --resize_mode=no", where should i resize the ori_image ? in the train.py ? (i find the size of the ori_image should be (512,512)). @JingyeChen

The problem is fixed. Please re-download the dataset using the link in README.md. After downloading, please resize every image to 512x512. Thanks!

@JingyeChen, Hi, are the image urls also updated? or it's just the meta data?

@JingyeChen, Hi, are the image urls also updated? or it's just the meta data?

Hi, I'm also curious about this issue. I noticed that some of the coordinates for OCR labels exceeds 512, and I'm suspecting that the labels are not measured on 512X512 resolution. Are meta files also updated? Thanks

The meta data, including detection and segmentation results, are updated. rec/det/seg are conducted with size 512x512. You can use np.clip(value, 0, 512) to clip the value.

@JingyeChen, Hi, are the image urls also updated? or it's just the meta data?

Hi, I'm also curious about this issue. I noticed that some of the coordinates for OCR labels exceeds 512, and I'm suspecting that the labels are not measured on 512X512 resolution. Are meta files also updated? Thanks

The meta data, including detection and segmentation results, are updated. rec/det/seg are conducted with size 512x512. You can use np.clip(value, 0, 512) to clip the value.

@JingyeChen, Hi, are the image urls also updated? or it's just the meta data?

Hi, I'm also curious about this issue. I noticed that some of the coordinates for OCR labels exceeds 512, and I'm suspecting that the labels are not measured on 512X512 resolution. Are meta files also updated? Thanks

Can I use the previously downloaded files with just clipping? or should i redownload them?

It is recommended to re-download it. Thanks!

Hi! Thanks for your good job! If i use this commond:"img2dataset --url_list=url.txt --output_folder=laion_ocr --thread_count=64 --resize_mode=no", where should i resize the ori_image ? in the train.py ? (i find the size of the ori_image should be (512,512)). @JingyeChen

The problem is fixed. Please re-download the dataset using the link in README.md. After downloading, please resize every image to 512x512. Thanks!

Hi! Thanks for your good job! If i use this commond:"img2dataset --url_list=url.txt --output_folder=laion_ocr --thread_count=64 --resize_mode=no", where should i resize the ori_image ? in the train.py ? (i find the size of the ori_image should be (512,512)). @JingyeChen

The problem is fixed. Please re-download the dataset using the link in README.md. After downloading, please resize every image to 512x512. Thanks!
I still have another question. After I re-download the metadata (the segmentation image inside remains unchanged), should I put the maion_laion images  into the corresponding directorys with their urls as the keys and resize them into 512*512 ?

Hi! I have something to confirm. After I redownload the metadata, should I put the maion_laion images into the corresponding directorys with their urls as the keys and resize them into 512*512 ? For the more , is the operation  of resize in the preprocess_train function in train.py? @JingyeChen

After I redownload the metadata, should I put the maion_laion images into the corresponding directorys with their urls as the keys and resize them into 512*512 ?

Yes, and you need to use resize each image to 512x512 during "putting the maion_laion images into the corresponding directorys" or perhaps add  image = image.resize((512,512)) in train.py

I get it！Thank you a lot, Senior Chen!

It is hard to say and may take some time. Please stay tuned ;D

hello, can you provide the training code for LayoutTransformer? I found that the text bbox is 4 points, i.e. [(x0, y0), (x1, y1), (x2, y2), (x3, y3)], however, the released layout transformer use [x, y, x, y]. Can you provide more detail for training LayoutTransformer?

Have u got the training code for Layouttransformer？

Thanks for your feedback. It is a mistake and the command should be:
img2dataset --url_list=url.txt --output_folder=laion_ocr --thread_count=64  --resize_mode=no

We will fix it in the readme file. Thanks!

Why can't we use --resize_mode=512 directly, while must like the operation you mentioned that "after downloading, you need to resize each image to 512x512. Please follow mario-laion-index-url.txt to move each image to the corresponding folders."
By the way, could you please offer a script to "resize each image to 512x512 and move each image to the corresponding folders."


Chaining llm chat with stdin

Would you be open to supporting passing in an optional initial message to chat requests? For eg.
I would expect this to work but seems to crash after the first response:
~ $ echo "hello world" | llm chat
Chatting with claude-3-5-sonnet-latest
Type 'exit' or 'quit' to exit
Type '!multi' to enter multiple lines, then '!end' to finish
> Hi! How can I help you today?
> Aborted!

This is what I'd actually like to do: files-to-prompt lol.txt --cxml | llm chat

There's a workaround for this at the moment:
echo "hello world" | llm
llm chat -c

The -c means "continue most recent conversation in a chat", which in this case will do the right thing.

Oh sorry no I misunderstood, this looks like a bug:
% echo "hello world" | llm chat
Chatting with gpt-4o-mini
Type 'exit' or 'quit' to exit
Type '!multi' to enter multiple lines, then '!end' to finish
> Hello! How can I assist you today?
> Aborted!

I think what's happening there may be that the equivalent of a Ctrl+D is being picked up?

Tried and failed to fix this like so:
diff --git a/llm/cli.py b/llm/cli.py
index ad7aeb4..e06d14c 100644
--- a/llm/cli.py
+++ b/llm/cli.py
@@ -511,14 +511,24 @@ def chat(
     if not should_stream:
         validated_options["stream"] = False

+    piped_input = get_piped_input()
+
     click.echo("Chatting with {}".format(model.model_id))
     click.echo("Type 'exit' or 'quit' to exit")
     click.echo("Type '!multi' to enter multiple lines, then '!end' to finish")
     in_multi = False
     accumulated = []
     end_token = "!end"
+    first = True
+    prompt = None
     while True:
-        prompt = click.prompt("", prompt_suffix="> " if not in_multi else "")
+        if first:
+            first = False
+            prompt = piped_input
+        else:
+            prompt = None
+        if not prompt:
+            prompt = click.prompt("", prompt_suffix="> " if not in_multi else "")
         if prompt.strip().startswith("!multi"):
             in_multi = True
             bits = prompt.strip().split()
@@ -550,6 +560,13 @@ def chat(
         print("")

+def get_piped_input():
+    """Read initial input pipe if it exists, otherwise return None"""
+    if not sys.stdin.isatty():
+        return sys.stdin.read().strip()
+    return None
+
+
 def load_conversation(conversation_id: Optional[str]) -> Optional[Conversation]:
     db = sqlite_utils.Database(logs_db_path())
     migrate(db)


Release 2.1.75 tag not signed

Hi, I package pex for the AUR on Arch Linux. Arch packaging guidelines state that if upstream provides signed sources, I need to verify using the GPG key. Additionally, I cannot remove GPG verification if upstream stops signing sources.
Release tag 2.1.75 isn't signed; please sign it, preferably without the GitHub web-signing key. I'd greatly appreciate it!

@lmartinez-mirror I had not hardened the release process; so this snuck through. At worst I'll get a fix out ~8am US Pacific time tomorrow.

Actually, the tag is signed:
$ git show v2.1.75 -q
tag v2.1.75
Tagger: Eric Arellano<EMAIL_ADDRESS>Date:   Thu Mar 31 11:07:43 2022 -0700

Release 2.1.75
-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEwJS9cBp16BL1lkgbdRi6HnNO8/oFAmJF7e8ACgkQdRi6HnNO
8/ptMQ//QySIuD9TUCsONCGkxtfBAdSqU4xwevVA4PgNhcLZBywHCxRuiOESuUsU
tmBap+Fees6HZ6wrRRWaRbBi6oBmYwP/iyA/cA9wIN63yG78c2HctgV2prlCBIu/
qcwZkVBn2fb5Bt4nzzbc3cB2DnF0kELaDFHVTMRgLxLfJtxbu4Z2ecJuIFZbNSb/
qzhqNDE7Tom69nyRNwzgo47Q03SvBF//jhw1JiFqtP2rfC2zGbIXrquGFXLbf09y
+atSIjBP/0I0KRaHVocRl6NMBkOWPsIxSllaASALW9SQxVdXHD31g0Uc0QjNdpXZ
aWVfwEh60RnEtCW9m+lpCtTTZ5c0oNOPKDWaSm9VMCI9mXsNVdQLsCfI5x4OK2LT
7n7oMapVqGAyuZt/l2NDFedM3Dq1428NYaVzfKZih/AbzmF/DV93nnzRTczeR/Zs
m0c5CVaMjuGIj91gDY7E5zCh2D9+edf6iUjV3p9vWRz4DFc+MrDpo7DKPMIIqx9r
9PEiGfMgXoOXupvK/xxS8zpLwU0wQAX8xWmuvPSnhpjyINGo5phB3/zukpzyWXtJ
BS3jFUlKrEfINlAMkm9/lilLS28udiQ5fSiyFGtmtRYX5RknaLCRmtNW/GYbL4y9
a5h7+JTjFY1zXGQM8/x56lVXtl1arCjSeeZgZEYQHbgEMLnzHHA=
=aHfW
-----END PGP SIGNATURE-----

commit ddc4ca107479ed9d12fe67defd01643f6c388b73 (tag: v2.1.75)
Author: Eric Arellano<EMAIL_ADDRESS>Date:   Thu Mar 31 11:02:37 2022 -0700

    Prepare 2.1.75 (#1699)

    Closes #1691.

@Eric-Arellano I think you just need to update your GitHub account settings to include your PGP key.
Looks like so:

After that, I think the tag will pick up an icon like this:

@lmartinez-mirror in the meantime, I've added a tag v2.1.75-public-signed: https://github.com/pantsbuild/pex/releases/tag/v2.1.75-public-signed
Perhaps you can modify your PKGBUILD for this release and use that tag?

hm idk what im doing wrong. i confirmed the key in github == what i use locally. i did not have on vigilant mode before but turned that on

@Eric-Arellano perhaps publish that key. It looks like you have only another published: https://pgp.mit.edu/pks/lookup?search=Eric+Arellano&op=index

@lmartinez-mirror you could also just skip 2.1.75 and move to 2.1.76 which was just released: https://github.com/pantsbuild/pex/releases/tag/v2.1.76

@Eric-Arellano I've handed this off to you to close out once you get your current in-use signing key published. All the other angles are now covered fwict.

Huzzah!

The issue is that my GitHub key only had one email address associated with it, and apparently I used a different email when I did the release. I changed my key to use the new email as well, re-uploaded to github, and now it works. Thank you John for the help, and thank you @lmartinez-mirror for the report!

@Eric-Arellano I don't think that will cut it. The GitHub icons are a guidepost - but GitHub is a walled garden. @lmartinez-mirror maintains an Arch Linux package that gets built as part of the Arch packaging system. It cares not a bit about GitHub or GitLab or even, necessarily git. It still wants to verify the tag though.
On a machine starting with no pgp setup:
jsirois@siroisdesign:~/pex$ git verify-tag v2.1.76
gpg: Signature made Mon Apr  4 17:18:39 2022 UTC
gpg:                using RSA key A1FE765B15233EAD18FA6ABB93E55CB567B5C626
gpg: Can't check signature: No public key
jsirois@siroisdesign:~/pex$ git verify-tag v2.1.75
gpg: Signature made Thu Mar 31 18:07:43 2022 UTC
gpg:                using RSA key C094BD701A75E812F596481B7518BA1E734EF3FA
gpg: Can't check signature: No public key

Ok, grab the keys:
jsirois@siroisdesign:~/pex$ gpg --keyserver hkps://keyserver.ubuntu.com --receive-keys 93E55CB567B5C626
gpg: /home/jsirois/.gnupg/trustdb.gpg: trustdb created
gpg: key 93E55CB567B5C626: public key "John Sirois (CODE SIGNING KEY<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
gpg:               imported: 1
jsirois@siroisdesign:~/pex$ gpg --keyserver hkps://keyserver.ubuntu.com --receive-keys C094BD701A75E812F596481B7518BA1E734EF3FA
gpg: keyserver receive failed: No data

That's bad and means @lmartinez-mirror still has a problem for v2.1.75.
IOW, I can now verify v2.1.76 but still can't verify v2.1.75:
jsirois@siroisdesign:~/pex$ git verify-tag v2.1.76
gpg: Signature made Mon Apr  4 17:18:39 2022 UTC
gpg:                using RSA key A1FE765B15233EAD18FA6ABB93E55CB567B5C626
gpg: Good signature from "John Sirois (CODE SIGNING KEY<EMAIL_ADDRESS>[unknown]
gpg:                 aka "John Sirois (https://people.apache.org/~jsirois/<EMAIL_ADDRESS>[unknown]
gpg:                 aka "John Sirois<EMAIL_ADDRESS>[unknown]
gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the owner.
Primary key fingerprint: A1FE 765B 1523 3EAD 18FA  6ABB 93E5 5CB5 67B5 C626
jsirois@siroisdesign:~/pex$ git verify-tag v2.1.75
gpg: Signature made Thu Mar 31 18:07:43 2022 UTC
gpg:                using RSA key C094BD701A75E812F596481B7518BA1E734EF3FA
gpg: Can't check signature: No public key

I think you really will have to update your published key / publish this new one.

I'll also assign to myself in tandem to get docs added to RELEASE.rst about the need for all this. Pants probably should be doing the same, but, as far as I know, is not an application packaed by any OS. Pex has been so since back in late 2011 IIRC starting with Debian / Ubuntu.

Thank you all for addressing this quickly! I'll go ahead and target release 2.1.76.

I've added myself to https://pgp.mit.edu/pks/lookup?search=eric-arellano&op=index.
pub  4096R/[734EF3FA](https://pgp.mit.edu/pks/lookup?op=get&search=0x7518BA1E734EF3FA) 2021-07-29 [Eric Arellano <ericarellano@me.com>](https://pgp.mit.edu/pks/lookup?op=vindex&search=0x7518BA1E734EF3FA)
                               Eric Arellano<EMAIL_ADDRESS>
❯ git verify-tag v2.1.75
gpg: Firmado el Thu Mar 31 11:07:43 2022 MST
gpg:                usando RSA clave C094BD701A75E812F596481B7518BA1E734EF3FA
gpg: Firma correcta de "Eric Arellano<EMAIL_ADDRESS>[absoluta]
gpg:                 alias "Eric Arellano<EMAIL_ADDRESS>[absoluta]

Thanks for the help! New world for me.

(reopening because we still need to fix our release docs so that this doesn't happen with future releases)


Recommend users a halo for U-Nets

This PR provides users a theoretical minimum halo for U-Nets, but in practice a smaller halo may also work well.
Changes
This PR attempts to fix #251

[x] Compute a halo based on model architecture
[x] Recommend users a halo for selected PlantSeg-compatible models (can be toggled off)
[x] Recommend users a halo based on images and architecture #307

Definition

Reference: Exact Tile-Based Segmentation Inference for Images Larger than GPU Memory

Last step is finished in #307


error using sudomy

/sudomy: line 358: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 359: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 360: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 361: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 362: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 363: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 364: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 365: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 366: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 367: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 368: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 369: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 370: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 371: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 372: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 373: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 374: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 375: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 376: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 377: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory

/ |  _ | ()() __ _  _
_ \ || / _  / __ \  ' \ || |
|/_,_,_/||__, |
|__/ v{1.1.9#dev} by @screetsec
Sud⍥my - Fast Subdmain Enumeration and Analyzer
http://github.com/screetsec/sudomy
[!] This tool is for educational purpose only.
Usage of sudomy for attacking targets without prior mutual consent is illegal
developers assume no liability and are not responsible for any misuse or damage cause by this program
[⍥] Perfoming Sud⍥my scans
[*] Load target domain: xxxxxxx.com
- starting scanning @ 2020-10-20 21:04:17
[+] Running & Checking source to be used
./sudomy: line 620: SHODAN: command not found
./sudomy: line 620: WEBARCHIVE: command not found
./sudomy: line 620: DNSDUMPSTER: command not found
./sudomy: line 620: CERTSPOTTER: command not found
./sudomy: line 620: CERTSH: command not found
./sudomy: line 620: BINARYEDGE: command not found
./sudomy: line 620: CENSYS: command not found
./sudomy: line 620: VIRUSTOTAL: command not found
./sudomy: line 620: THREATMINER: command not found
./sudomy: line 620: BUFFEROVER: command not found
./sudomy: line 620: FINDSUBDOMAIN: command not found
./sudomy: line 620: THREATCROWD: command not found
./sudomy: line 620: SECURITY_TRAILS: command not found
./sudomy: line 620: ENTRUST: command not found
./sudomy: line 620: ALIENVAULT: command not found
./sudomy: line 620: RAPIDDNS: command not found
./sudomy: line 620: URLSCAN: command not found
./sudomy: line 620: RIDDLER: command not found
./sudomy: line 620: COMMONCRAWL: command not found
./sudomy: line 620: HACKERTARGET: command not found
[+] Get & Count subdomain total From source
[+] Parsing & Sorting list Domain
./sudomy: line 339: ${OUT}/${DATE_LOG}/${DOMAIN}/${RESULT_SUBDOMAIN}: ambiguous redirect
sed: can't read /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
sed: can't read &subdomain/Sudomy/output/*.log: No such file or directory
./sudomy: line 919: /home/mrcyguru/Desktop/Find/script/Dns: No such file or directory
./sudomy: line 920: exec_httprobe: command not found
[+] Sud⍥my has been sucessfully completed

Did you run the script with sudo?

yes

Please, gives detail information like operating systems, bash version and what command do you use


Publishing Nuget package

Would you mind publishing an official nuget package? It is not difficult using appveyor.

I am aware of the existing packages but they do not contain an update 32 bit version.

Those unofficial packages pointlessly point to default openblas home page, especially the most downloaded one is old and buggy.
It is not that easy - MSVC build with appveyor is not the fast assembly one, current windows dll on sf.net is cross-built on Linux.
If you think you can do it right - go ahead.

so should this one be uploaded? https://travis-ci.org/xianyi/OpenBLAS/jobs/253894094

How that is a valid release?

It's not. this would be done with git tags.

Or ideally as part of release packaging.... Do you happen to know how to get pdb file, even partial, out of mingw build? Without that no big advantage over current binary zip.


When injecting a cart, the save function should automatically save to the loaded file

When you launch TIC by injecting it a cart via the command line, the save function (or ctrl+s) should automatically save to that cart, without needing to type its name.
If the injected cart isn't in the TIC folder, maybe display a message saying that we have to type the name explicitly, to avoid any unwanted overwrite.

I think it is not necessary to differentiate the files opened from the save folder or not (there is no safeguard in the save folder, why would other folders work differently?)
IMHO just save the tic file from wherever it was opened.

hmm, TIC was designed to work with local FS only, so it doesn't know anything about external files
as a variant, I can give to the loaded cart same name you provided in cmd and it will be saved to local FS without confirmation dialog

fixed with #175


Add support for mojo 0.6

I know it only came out yesterday but ;-)

Hi @clarkezone !
Do you see any issues with running it on Mojo 0.6.0?
It looks working fine on my side .
Also do you have any suggestions which new Mojo features you'd like to see implemented in llama2.🔥?

Hi @clarkezone !
Do you see any issues with running it on Mojo 0.6.0?
It looks working fine on my side .
Also do you have any suggestions which new Mojo features you'd like to see implemented in llama2.🔥?

Just tested if with 0.6 on AMD64 linux and it appears to work fine.  So probably safe to bump the readme.
I haven't looked at the llama2.🔥 code yet and still working my way through the 🔥 0.6 readme.  So not yet but will do for sure ;-). Nice job with this and getting into the keynote.  Now we just need a port of whisper-cpp ;-)


Create a Space Exploration Game Website

Pull Request Details:

Title: Create a Space Exploration Game Website
PR Link: #9
Issue: #8

Next Steps:
I'm ready for feedback and any necessary revisions. Please review at your earliest convenience, and feel free to provide comments or suggest improvements.
Thank you for your time and consideration!

Hi User,
Thank you for your pull request. I appreciate your contribution to our project.
I have reviewed your changes and everything looks good. Please be patient, and I'll get back to you as soon as possible.
If you have any questions or need further assistance, don't hesitate to ask.
Best regards,
Jayesh


Automated mapping to Darwin Core terms - bug report and suggestion

Today we received this email on Helpdesk.

Hi,
I am testing data mapping at https://ipt-test.gbif.no/manage/mapping.do.
This is just to let you know about two things which I have noticed: one error and a couple of suggestions.

Auto mapping
Most of the columns I’ve imported with names which match the core occurrence terms are automatically selected for mapping (eg materialSampleID). However, I have a column of data with the heading dcterms:type and it is not recognised as matching the term listed in the Record-level section. It works if I change the column name to type, without the prefix. Since it’s potentially a pretty generic term, and it could be handy to have them named this way in the original data, I thought it would be useful if the system was able to recognise and map columns named dcterms:type (or even dcterms.type). (There is such flexibility in recognition of the occurrenceID field: imported columns named occurrence.occurrenceID and firstTable.occurrenceID are both auto mapped ok).

Error when cancelling a Mapping
A couple of times I clicked Add with the wrong core/extension selected, or I did it just as a way to read the descriptions of extension types (eg trying to review the difference between MoF and eMoF). In each case I clicked the Cancel button on the Mapping page, but the system goes ahead and creates and saves a Mapping anyway, even if you just click on the Resource Title at the top to escape back to the Overview page. This isn’t so bad if you notice and delete it while still in the Mapping page, but if you just click the Resource Title it will remain as part of the resource and could be a source of error if published without noticing.

One way to allow access to the core/extension descriptions, without going into the Administration section (if you have access), could be to list the core/extension names as a radio button list, instead of a dropdown list. Each item could then have an info link beside it with a popup box displaying the description. I have attached a doc with images of how this could look, and code to reproduce the layout mockup. (Please excuse my unsolicited suggestions; I spent years working as a web developer and it’s kind of second nature to look for improvements 😊).
I hope that is of some use. Feel free to ignore the suggestions; I realise there are many other factors to take into account.
Best,
Gavin Alexander

EDIT: here is the document Gavin sent:
Alternative Darwin Core Mappings form in IPT.docx

This is fixed. Related issues https://github.com/gbif/ipt/issues/1739 and https://github.com/gbif/ipt/issues/1806


[ENG-4387] use osf metrics for node analytics

Ticket: ENG-4387
Feature flag: n/a

Purpose
keep the node analytics page running (without keen)

Summary of Changes

load data from osf-metrics "node analytics" api
remove keen js dependencies
add c3 dependency (for compatibility; used by keen-dataviz)

Screenshot(s)

Side Effects

QA Notes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

45 of 52 (86.54%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 72.396%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/analytics-page/addon/application/controller.ts
10
11
90.91%

lib/analytics-page/addon/application/route.ts
6
8
75.0%

lib/analytics-page/addon/components/analytics-charts/component.ts
15
19
78.95%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
5722

Relevant Lines:
7687

💛  - Coveralls


Too many to handle

Hey,
First: Great and helpful tool. Using this for one year and it was working most of the time.
Setting:
Debian
FTPd Server
Customer uploads pictures
watcher triggers a console application which makes a SQL Insert into our MySQL
a daemon handles this files and moves them where i want the files
Problem:
New customer upload sometime more than 200 Files at once
watcher seems not to be able to handle it
sometimes around 80 files don't get paste into MySQL
So i guess because of too many actions watcher can't queue the actions anymore
Suggestions? I will now check again what could be the problem.
Thanks!

Couldn't it be this issue with you as well?

Troubleshooting
If inotify is reporting that it can't create watches due to lack of disk space, but you have plenty of disk space and inodes available, then you may have to raise the max number of inotify watches.
To check the max number of inotify watches:
cat /proc/sys/fs/inotify/max_user_watches
To set the max number of inotify watches:
echo 20000 | sudo tee -a /proc/sys/fs/inotify/max_user_watches


dataclass_transform incorrectly infer __init__ kwargs

Environment data

Language Server version: Pylance language server 2022.9.10 (pyright aba41864) starting
Python version (& distribution if applicable, e.g. Anaconda): 3.8

Code Snippet
Reproduction instruction:
import typing_extensions

@typing_extensions.dataclass_transform(kw_only_default=True)
class A:
  x: int

class B(A):
  y: int

B(y=1)  # pylance expect B(x=, y=), but should instead be `B(y=)`

Explaination: In the above snippet, B is a dataclass-like object (because it inherit from a dataclass_transform class). However A itself should NOT be a dataclass-like object.
Semantically, dataclass_transform should be similar to:
import dataclasses

class A:
  x: int

  def __init_subclass__(cls):
    dataclasses.dataclass(cls)

class B(A):
  y: int

B(y=1)  # A is not a `dataclass`, so `B.__init__` is `(y=)` NOT `(x=, y=)`

Context: https://github.com/google/flax/issues/2416#issuecomment-1239658713

Our dataclass_transform support comes from Pyright, the type checker that Pylance is based on. This is a duplicate of https://github.com/microsoft/pyright/issues/3907, but we appreciate your thoughts on this issue.

I don't think this is a duplicate of https://github.com/microsoft/pyright/issues/3907. The other bug is about whether __init__ method synthesis is skipped based on the presence of an explicit __init__ within the parent class.
In this issue, the __init__ method synthesis is assumed, but there's a question of whether it should include fields from a class that is decorated with @dataclass_transform directly.
When we first authored PEP 681, we said that @dataclass_transform was intended to be used in two contexts:

To decorate a function that, itself, is designed to work as a class decorator and modifies the decorated class to give it dataclass-like properties. The stdlib dataclass is implemented as such a function.
To decorate a class that is intended to be used as a metaclass that implements dataclass-like functionality for any class that is created by that metaclass. This is how pydantic's ModelBase class works.

We later added a third case, which is used in the code sample above:

To decorate a base class that implements dataclass-like functionality for any class that derives from it.

This third case was added late in the review process, and I can see now that there were aspects that we didn't consider when spec'ing it or implementing it. We weren't clear in the spec about whether fields within the decorated base class would be treated as dataclass fields. Pyright's current implementation assumes that they are, which is why x and y are both assumed to be present in the synthesized __init__ method. @Conchylicultor, I can understand why you assumed that the fields within the base class should not be be included here. It's a reasonable assumption.
I think this is something we may need to go back and clarify in the PEP. Once clarified, we can make sure that all type checkers (including pyright) conform to the clarified spec.

@Conchylicultor, reading this again it's now clear to me that this is different from https://github.com/microsoft/pyright/issues/3907, and also that the expected behavior you described above is clearly correct -- attributes on the class decorated with dataclass_transform (or that class's base/ancestor classes) should not be considered to be fields.
I will clarify this in PEP 681 and we'll get the behavior fixed in Pyright/Pylance as well.

This issue has been fixed in prerelease version 2022.10.21, which we've just released. You can find the changelog here: CHANGELOG.md

Awesome! Thank you!


[ERROR] Device did not enter recovery mode successfully!

I have entered pwned dfu mode with signature checks removed but I still get this error:
[ERROR] Device did not enter recovery mode successfully! Make sure your device is in Pwned DFU mode with signature checks removed, then run this script again.
Exiting...

What device are you using, and what iOS version are you attempting to downgrade to?

I'm using an iPhone 5s (iPhone5,1) and attempting to downgrade to iOS 11.4.1
I have been using ipwndfu_public and running the command "python rmsigchks.py" to remove the signing checks.
I've double checked that the phone is pwndfu and I keep getting that same error.

Try pwning + patching signature checks with iPwnder32 perhaps? I've had issues with the signature check patches from Linus's ipwndfu fork in the past.

Sorry to ask, but I have a couple questions. First, how do I remove the signing checks with iPwnder32? Do I use the "--nibss" command or the "-f " command. Second, what is the difference between pwnedibss and pwndfu mode? Finally, are kdfu mode and pwned dfu mode the same thing? I'm a huge noob with this kind of stuff as I'm only 12 and only have a year and a half of jailbreaking and downgrading experience.

Nvm. I figured everything out. I restored my iPhone and everything works. Btw, I'm the kid from discord who asked about futurerestore.


Make BeakerContract error handling consistent with the kernel

In the kernel if a system call is successful, we simply return whatever the underlying opcode/call would have returned. If the system call fails, we indicate this by leaving a 0 on the stack after the delegatecall (via revert), and the return data is whatever is specified for error return values.
The way BeakerContract.sol works is that it always returns a tuple value. I think we should use the stack error value much as we do with the kernel. This will also simplify things a little I think.
We discussed and went with this plan for the kernel, it makes sense to use it for the stdlib as well.

Ok so from the description, here's a rough sketch:

Procedure X invokes cap to call Procedure Y and receives a (bool err, bytes output)
Kernel checks currentProc Id, and cap:

If the cap does not exist or invalid, the Kernel does a revert(errvalue)

Procedure X receives a zero on stack, and sets err = true, and output = errValue

Else the Kernel makes a call to Procedure Y:

If Procedure Y returns a 0 on stack, the Kernel does a return(procOutput)

Procedure X receives a 1 on stack, and sets err = false, and output = procOutput

If Procedure Y returns a 1 on stack, the Kernel does a return(procOutput)

Procedure X receives a 1 on stack, and sets err = false, and ouptut = procOutput

So in this case, the Kernel does not differentiate between a Procedure returning or reverting during execution. Now we could do this, but we would need a way to differentiate the two without relying on correct Procedure behavior.

The only difference with what is currently implemented is that the kernel will revert if a procedure reverts etc. That is, if Procedure Y puts a 0 on the stack, the kernel will also put a 0 on the stack. This is necessary for Procedure X to determine whether Procedure Y returned successfully.
If the kernel puts 0 on the stack Procedure X can further inspect the return data to determine what happened.

We could stick to that. However if the kernel reverts on a procedure reverting, the user won't be able to conclusively differentiate between a kernel revert and a procedure revert (since we can't rely on the procedure to give reliable information). Either the kernel prepends or appends additional data, or we do a return (stack 1).

I agree but pre-empted that question. A reverted syscall is always a revert from the kernel, and the data it returns is dictated by the kernel. I believe the current solution in the spec is what you've described in the first solution, but I'll clarify here. Let me know if it is the same. If Procedure X is calling Procedure Y and Procedure Y fails (be it a manual revert, out-of-gas, or illegal opcode) then:

The kernel will see the 0 on the stack due to the failure of Y.
The kernel will revert with the data 0x55 prepended to whatever data Procedure Y reverted/failed with.

Example 1:
Procedure Y fails due to out-of-gas. Out-of-gas errors return no data, so after the syscall Procedure X will find a 0 on the stack and returned data 0x55.
Example 2:
Procedure Y fails due to an element not existing in the list. The user has written the code such that it returns the error 0x45 and the name of the element in ASCII. In this case the name of the element is bob which is 0x626f62 in ASCII. So, upon reverting, Procedure Y will pass 0x45626f62 back to the kernel, and the kernel will revert and pass 0x5545626f62 back to Procedure X via a revert.
Procedure X will see the 0 on the stack, conclude that there was an error with the system call and the kernel has provided data. It will look at the first byte of the returndata (0x55) and understand that the kernel is saying that the procedure it was calling (Procedure Y) failed. Procedure X will then understand that the remaining bytes of the returndata (0x45626f62) is the failure data provided by Procedure Y.
Example 3:
The kernel runs out of gas during any part of the system call. In this case Procedure X will find 0 on the stack, but upon inspection of the data will see that there is no data. It can therefore conclude that the kernel failed due to out-of-gas or illegal opcode.
Example 4:
Something else. If the kernel hits some other error that I haven't covered or we haven't thought of yet, it can prepend something other than 0x55. For example, insufficient capability (pg. 9 of spec) will return 0x33.
This is currently specified on page 9 as below. Although admittedly there is no accompanying explainer.

Some less critical notes:

I should also note that currently the tests for account-call rely on this behaviour already. It simplifies many of the tests because it only checks for that first byte 0x55 indicating a certain procedure fails. When/if those procedures are updated with more features and complicated errors those tests won't care.
If we added a length parameter (i.e. 0x55len) then we could have full stack traces, but that would rely on user cooperation to ensure they reverted with the right data. Therefore I don't believe this should be specified by the kernel. However, full stack traces can be provided by a user if they wish, and this mechanism will support it quite easily.

Oh ok great! So just like in the spec the err value is uint8 and be set with 0 for no-error and everything else as an error code.
Let's stick to that then. Another thing we didn't touch upon is that in many cases Procedure Y might not revert at all, but simply have a custom error message in it's output. This is what is intended of course and will add another layer of error handling, but I digress.
I'd be keen to discuss how we could implement a stack tracing via a seperate issue. Maybe after #118.

We should probably add this information in the prod#call section.

Oh ok great! So just like in the spec the err value is uint8 and be set with 0 for no-error and everything else as an error code.

Slight correct. Procedure X looks at the stack. If the value is 1, the returndata is whatever was returned by Procedure Y, untouched by the kernel. Of course here we are talking about the syscall interface, not the stdlib. The functions in BeakerContract.sol will always return an error code as part of a tuple.

@JakeOShannessy: Does this have a deliverable we could track? Or can we close this?

I think we can close this. We've discussed and understood and the hard details are in the spec.


Are you open to PRs for this project?

I’m a software developer who was going to write some C# code generators for a fluent dto builder, and I saw your data-builder-generator which is very similar.
Instead of me writing my own to match my particular needs, would you be open to me opening some PRs to extend yours?

I’ve found a few small bugs (possible) where VS doesn’t like internal classes decorated with the GenerateDataBuilder attribute
I’d like to add an assembly-level attribute that can be used to specify a file to generate the data builder for (so instead of decorating a class, you could put these attributes somewhere else, like a test project, if you only use the builder pattern to test DTOs and don’t want to put the attribute on prod code). Maybe something like the following?

[GenerateDataBuilderFor(nameof(MyAssembly.MyClass), generatedNamespace: $"{nameof(MyAssembly.MyClass)}.Tests")]

I’d like to extend the collection-level builder so that one could add single items to ICollection<T>. So... with the following class:

[GenerateDataBuilder]
public class Family
{
    public List<string> FamilyMember { get; set; }
}

The builder would have the following method WithFamilyMember(string familyMember) that would add a single string to the list.

Yes if you'd like to extend the functionality and fix some issues, I'd be happy to work with you to get these out.
As I previously mentioned in https://github.com/dasMulli/data-builder-generator/issues/3, I wanted to hold off a bit waiting for good practices in source generators to emerge before refactoring / working on more features since it does what I need it to do.
But again, if you need any extensions, no need to wait.


New Prod error: assert(this.runtime.deltaManager.active, 0x25d /* "We should never connect as 'read'" */);

This query

union Office_Fluid_FluidRuntime_Error
| where Event_Time > ago(5d)
| where Data_loaderVersion contains "0.51"
| where Data_error contains "0x25d"
| summarize count() by Data_eventName, Data_error, Data_clientType

shows new error that was not present in earlier version of runtime - assert 0x25d
While number of hits is low, the usage is also very low. We can't easily estimate future impact, but given it's new assert, we should look at it very carefully and examine if this is latest regression. If it is, we should have a plan how to mitigate it pretty quickly.
@pleath , can you please take a first look? I've added this assert long time ago and I see no hits for months, so something somewhere changed. It basically says that summarizer is connected as "read" client, when it should never be able to do.
It's possible it's regression from my PR with good intentions :): https://github.com/microsoft/FluidFramework/pull/7753
I.e. connection now can be downsized to "read" without loosing it / reconnecting.
It was the case even before, but it was more implicit / did not show up in any state we track (so it showed up as nack on any op being sent). If that's the case, then we need to re-evaluate if we want to generate "disconnect" / "connect" transition (on same connection, but with different mode) to avoid surprises like that across stack
I believe it's not critical as client will recover just fine, but it does create noise.
@wes-carlson, @agarwal-navin - FYI.

BTW, feel free to dump item on me / Mark if you confirm that's indeed related to that PR and it's not critical. Assigning to you for initial help with triaging as there are a lot of things in flight RE GA  and help would be appreciated

Taking a look at this.

From my initial investigation, I think #7753 indeed caused this. Here's the sequence of events that happened:

The summarizer successfully connected in "write" mode and completed a summary.
The summarizer got a "leave" message and was downgraded to "read" mode. I believe we should not do this for summarizer.
The summarizer when trying to summarize the next time, checked for cancellation which resulted in the error 0x25d as it was downgraded to "read".
It's possible that I missed a step in between 2 and 3 where the summarizer should have never connected in "read" mode?

Here are the relevant logs:

Execute in [Web] [Desktop] [cluster('kusto.aria.microsoft.com').database('6a8929bcfc6d44e9b13fee392ada9cf0')]union Office_Fluid_FluidRuntime_*| where Event_Time > ago(5d)| where Data_loaderVersion contains "0.51"| where Data_docId == "bDW6iJQInwiOJjBqhgQnXe76ahjvnQJ2Dq5B1P475XM%3D"| where Data_clientType == "noninteractive/summarizer"| where Event_Sequence >= 142 and Event_Sequence <= 170| summarize count() by Event_Time, Event_Sequence, Data_eventName, Data_reason, Data_error, Data_clientType, Data_containerId, Data_connectionModeTable0

Event_Time
Event_Sequence
Data_eventName
Data_reason
Data_error
Data_clientType
Data_containerId
Data_connectionMode
count_

2021-11-19T20:46:31.695Z
162
fluid:telemetry:Summarizer:Running:UnexpectedSummarizeError

0x25d
noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:47:01.768Z
170
fluid:telemetry:Container:ContainerClose

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:46:31.692Z
161
fluid:telemetry:Container:connectedStateRejected

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:47:01.764Z
167
fluid:telemetry:Container:noWaitOnDisconnected

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:47:01.763Z
166
fluid:telemetry:Summarizer:WrapError

0x25d
noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:47:01.762Z
165
fluid:telemetry:Summarizer:StoppingSummarizer
parentShouldNotSummarize

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:47:01.766Z
169
fluid:telemetry:ContainerRuntimeDisposed

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:41:31.157Z
142
fluid:telemetry:Summarizer:Running:Summarize_end

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:46:31.691Z
160
fluid:telemetry:DeltaManager:ReadConnectionTransition

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:47:01.765Z
168
fluid:telemetry:Container:ConnectionStateChange_Disconnected
Container closed

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

Another information worth mentioning is that there was a LastSummaryProposedHandleMismatch error during summary upload which may have been responsible for the server sending a "leave" message for the summarizer in the first place:

Table0

Event_Time
Event_Sequence
Data_eventName
Data_reason
Data_error
Data_clientType
Data_containerId
Data_connectionMode
count_

2021-11-19T20:41:26.606Z
134
fluid:telemetry:Summarizer:Running:GarbageCollection_end

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:41:27.866Z
137
fluid:telemetry:OdspDriver:uploadSummary_end

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:41:27.867Z
138
fluid:telemetry:OdspDriver:uploadSummaryWithContext_end

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:41:27.932Z
140
fluid:telemetry:OpPerf:OpRoundtripTime

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:41:26.616Z
135
fluid:telemetry:OdspDriver:LastSummaryProposedHandleMismatch

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:41:27.868Z
139
fluid:telemetry:Summarizer:Running:GenerateSummary

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1

2021-11-19T20:41:27.932Z
141
fluid:telemetry:Summarizer:Running:SummaryOp

noninteractive/summarizer
4a0b8dc8-f2e7-47d3-b4db-fb7749aa552d

1


Better sudo detection/integration

I don't want sudo required commands to just hang when waiting for a password.
We need to find a way where we can detect if a user is nopasswd or not before even executing, or have the password prompt work as intended.

sudo -n true (-n is non interactive mode) works on Linux. Need to check macOS.

related to this, I had an issue when it would almost ask for the password in the background, it was very weird but I could only type password for a few characters and then it'd "lose stdin".
my fix was to run sudo for another command before loon so i'm "pre-authenticated" and then it just works fine.

Yeah, it doesn't work great if you don't have nopasswd setup.
I think what I need to do is run a preflight, check what stuff needs to be done, and then pre-emptively ask for sudo unless nopasswd is set.


Commited

Hey really appreciate your PR, but the cards are only meant for people from Sushiksha Organization. So will not be able to merge it. Sorry. thank you for the PR


Add daneads to maintainers

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

You can just include this commit in the PR in which you add youself to some meta.maintainers list.


Issue 169

This seems like the correct approach to me, but I'm assuming @bbockelm will have more comments.

Tests are currently failing because the test CI doesn't pull in the updates I've made to the dev container we use to run tests. Those updates are reflected in PR #450. I will re-run once that's merged, built, and released.


fix: [#9331] Substring function contains wrong tooltip information

Description
This PR fixes the tooltip description of the Substring function to reflect the current parameters.
Note: we tested the case reported in the issue:

a length from the startPos that surpasses the length of the string will generate an out of range exception, not be taken as the max length of the string.

but the exception was not thrown so we didn't update that part of the description.
Task Item
Fixes #9331
Screenshots
These images show the before and after.

Hello @OEvgeny ,
We reach out to you because we are having issues with Prs modifying extensions in BotFramework-Composer, and we saw that you created some successful PRs updating the same packages (# 9396).
Our PRs fail with the error: YN0028-The lockfile would have been modified by this install, which is explicitly forbidden.
And we tried different approaches to build the project and generate the yarn.berry.lock file but we couldn't make it work.
We would like to know the steps you follow to build the project after updating the code of an extension, and if there's any extra configuration for the checksums to be properly updated.
Thanks!

@erquirogasw the yarn build:dev command after some changes are made to shared packages usually works for me. If the sums didn't get updated, you could try removing node_modules from the extension package folder.

@erquirogasw please ensure you pulled the latest version from master before updating the sums.

This might be windows-related issue. Yarn has an issue with hash persistence across platforms unfortunately.

Coverage remained the same at 54.648% when pulling 20a82fe7114e05eaa009df9a286da87fa90a1e1e on southworks/update/substring-tooltip into a27c3e55332ee6b5a63619f375a5b2fd127923f4 on main.

Hi @BruceHaley, we have the E2E tests failing in this PR and in a few others. Also, there's a component governance error, but we don't have access to the pipeline to understand the problem and fix it. Could you help us with this?
Thanks!

@ceciliaavila e2e pipeline failure seem unrelated to the PR

Should be good to go after related package lock hash updates


Update Docs: Quick Start “start-with-a-template”

Automatically generated by Netlify CMS

The test works!


Update html5-and-css.json

Fix two of the tests in "Waypoint: Create an Ordered List"
See Issue #1015

For some reason I am being copied on all of your emails! Any chance some can remove me please
Sent from my iPhone

On Jul 27, 2015, at 2:51 PM, Timothy Cyrus<EMAIL_ADDRESS>wrote:
Fix two of the tests in "Waypoint: Create an Ordered List"
See Issue #1015
You can view, comment on, or merge this pull request online at:
https://github.com/FreeCodeCamp/freecodecamp/pull/1400
Commit Summary
Update html5-and-css.json
File Changes
M seed/challenges/html5-and-css.json (4)
Patch Links:
https://github.com/FreeCodeCamp/freecodecamp/pull/1400.patch
https://github.com/FreeCodeCamp/freecodecamp/pull/1400.diff
—
Reply to this email directly or view it on GitHub.

Hi, if you getting these emails, likely you clicked on the github "Watching" option for the repo.
Date: Mon, 27 Jul 2015 14:01:53 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: Re: [freecodecamp] Update html5-and-css.json (#1400)
For some reason I am being copied on all of your emails! Any chance some can remove me please
Sent from my iPhone

On Jul 27, 2015, at 2:51 PM, Timothy Cyrus<EMAIL_ADDRESS>wrote:

Fix two of the tests in "Waypoint: Create an Ordered List"

See Issue #1015

You can view, comment on, or merge this pull request online at:

https://github.com/FreeCodeCamp/freecodecamp/pull/1400

Commit Summary

Update html5-and-css.json

File Changes

M seed/challenges/html5-and-css.json (4)

Patch Links:

https://github.com/FreeCodeCamp/freecodecamp/pull/1400.patch

https://github.com/FreeCodeCamp/freecodecamp/pull/1400.diff

—

Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub.


Comments on constructor super call not preserved in ES5 unless it has additional empty line

Bug Report

🔎 Search Terms

comments
stripped
super
constructor

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________

⏯ Playground Link
Playground link with relevant code
💻 Code

class B {
    constructor(...args: any[]) {}
}

class A extends B {
    constructor(...args: any[]) {
        /** this comment is not preserved */
        super(...args)
    }
}

class C extends B {
    constructor(...args: any[]) {
        /** this comment is preserved */

        super(...args)
    }
}

🙁 Actual behavior
Comment on the super call was not preserved in the ES5 output
🙂 Expected behavior
Comment on the super call to be preserved

Comment preservation is not guaranteed in all cases, but this behavior does seem odd


:images main page should indicate whether execution environment use is enabled

On this page there is not the information about the EE enabled.
there is an information about the main EE used on the ansible-navigator config but if this EE is unabled we don't know.

config example:
ansible-navigator:
#   app: run
#   collection-doc-cache-path: /tmp/cache.db
#   cmdline: "--forks 15"
#   editor:
#     command: vim_from_setting
#     console: False
#   documentation:
#     plugin:
#       name: shell
#       type: become
  execution-environment:
    image: "quay.io/ansible/awx-ee:devel"
    enabled: False

  log:
    level: debug

We can add it to "primary"  eg ('primary/disabled") or primary/enabled

With :settings on it's way, this is no longer necessary and the proposed fix was confusing.


Issue: mi Smart plug failed auth

Mi two smart plugs always worked fine (Mi Cloud), starting from today homebridge console reports: 401 unauthorized, and the devices can’t be controlled

Maybe something changed with your account? A 401 in general means that you do not have access to the service/account.

Maybe something changed with your account? A 401 in general means that you do not have access to the service/account.

Nope, same credentials. App works regularly!

Nope, same credentials. App works regularly!

It does not matter if your app works or not, something might have changed with your account which now prevents access to some services (i guess maybe there was some kind of switch between years in your country regarding regarding your account?)

It does not matter if your app works or not, something might have changed with your account which now prevents access to some services (i guess maybe there was some kind of switch between years in your country regarding regarding your account?)

Hey, I have the same issue with all of my xiaomi devices.

Hey, I have the same issue with all of my xiaomi devices.

What country do you have your devices on and what country is your micloud account setup for? Also could you post a homerbidge debug log,deally with the the deepDebug parameter set to true , so at least i can have a look at what point it fails for you.
My suspicion is that either Xiaomi changed something or something with your account changed (might be specific for your country since for me all devices over micloud are working fine)

What country do you have your devices on and what country is your micloud account setup for? Also could you post a homerbidge debug log,deally with the the deepDebug parameter set to true , so at least i can have a look at what point it fails for you.
My suspicion is that either Xiaomi changed something or something with your account changed (might be specific for your country since for me all devices over micloud are working fine)

Hey, my xiaomi devices are in romania, but the only server that worked previously was DE, so it's set on Germany. How can i export my log file, or it's ok to post it here with copy paste?

Hey, my xiaomi devices are in romania, but the only server that worked previously was DE, so it's set on Germany. How can i export my log file, or it's ok to post it here with copy paste?

Interesting... I am not aware of any changes to micloud for the Germany region.
Please no copy/paste as it might get too big 😄  Try to attach it as a .txt file or post it on gist https://gist.github.com/ just make sure to remove any sensitive information.

Interesting... I am not aware of any changes to micloud for the Germany region.
Please no copy/paste as it might get too big 😄  Try to attach it as a .txt file or post it on gist https://gist.github.com/ just make sure to remove any sensitive information.

https://gist.github.com/iishrnika/1c57c643e7813955fc6e693c489a97e8

https://gist.github.com/iishrnika/1c57c643e7813955fc6e693c489a97e8

If i did it wrong please say.

If i did it wrong please say.

That does not help me much. Like i already mentioned above you need to run homebridge in debug mode and additionally enable the deepDebugLog  parameter for one of your devices which has this issue.

That does not help me much. Like i already mentioned above you need to run homebridge in debug mode and additionally enable the deepDebugLog  parameter for one of your devices which has this issue.

hey, i just updated it with the deepDebugLog on

hey, i just updated it with the deepDebugLog on

I don't know if it matters much, but I have the same issues and my account is also in Germany region.

I don't know if it matters much, but I have the same issues and my account is also in Germany region.

Well not sure what the issue here is, might be that something changed with the germany server (but why only Germany?) or it might be just a temporary issue due to a bug...
I think for now you need to observe that for a couple of days and see if it will start work again... other then that, you can try to move your devices to another server (i have all my devices on the CN server and they are working fine)

Well not sure what the issue here is, might be that something changed with the germany server (but why only Germany?) or it might be just a temporary issue due to a bug...
I think for now you need to observe that for a couple of days and see if it will start work again... other then that, you can try to move your devices to another server (i have all my devices on the CN server and they are working fine)

Thanks.

Thanks.

Okay, thanks for the help!

Okay, thanks for the help!

Have the same problem with mine...

@zormic also got your devices on the Germany server?

Yup!

Yes Germany server (one device requires it)

We had a worldwide server failure on December 31st. since then all servers are online, only the DE server is still causing problems, the devices are not listed on the server and you can only log in to a limited extent. Do you always have to rely on the server? Thought with the token it runs locally. However, like you, I am currently not getting the token. I think we have to wait or move to another server, which I don't want.

@Seger85 as i thought, there must have been something going on with the server...
Yes, for device which does not support the miot protocol natively (older devices, which was released before around 2020) all commands has to go over the micloud, which takes care of the translation of the older protocol to the miot protocol. On newer devices all commands are sent locally and a micloud connection is not needed.
Like i already mentioned the best would be first to wait a couple of days and see if xiaomi fixes the issue (maybe due to holidays in Germany nobody could have a look at the issue yet).
Another option would be to switch to another server since all other servers seems to work fine.

Hey @merdok,
you had just said that it is only for devices before about 2020 so how exactly do you find out and do you happen to know how it behaves with the device:
xiaomi air purifier 3c
The release year is not that long ago:
Late 2020/early 2021
Thank you for all and fingers crossed that the DE servers are back soon ;-)

@Seger85 air purifier 3c is a native miot device, you do not need micloud to control it, should also work with local commands.

@merdok
I don't know how it should work locally, the token must be entered in your plugin.
My token was reset somehow, so I have to get the current one.
My device was offline so I had to reset the wi-fi settings and it hasn't shown up in Homebridge since. It looks like my air purifier 3C got a new token.
Maybe you would be so kind and explain it to me here or by email.
Thank you very much,
Seger

@Seger85 this is not the correct place to discuss your token changes since it has nothing to do with the issue.
If your token changed, then you have to get a new one and enter it in the config.json

Hey buddy,
I'll try an e-mail, but I don't know if it'll get through. I don't know where else to ask for what I need ;).
My problem lies exactly in your description. I can only get a new token via this tool, which requests the server, right? Or do I have a possibility to get the new token locally? If so, then of course I have understood it and it would be my desired local solution.
Thank you very much,
Seger
Von meinem iPhone gesendet
Am 03.01.2022 um 07:53 schrieb Marcin @.***>:

@Seger85 this is not the correct place to discuss your token changes since it has nothing to do with the issue.
If your token changed, then you have to get a new one and enter it in the config.json
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

Please use the README for that, there is a dedicated page which gives a couple of options on how to get the token here: https://github.com/merdok/homebridge-miot/blob/main/obtain_token.md

Yeah, I have the same issue. The DE server is used and nothing works as a result - 401 authorization failure :(
And the funny thing here is that Xiaomi disabled LAN control for lots of devices (at least Yeelight devices), so now we have no choice but to use cloud (which doesn't work at this moment)

Ok got an update. It seems that Xiaomi changed the authentication on the DE server to a more secure one, but all other servers are still using the older one. I do not know why they did this only on the DE server.
Anyway, it seems that i will have to look at the MiCloud protocol code and update it with the authentication which is used on the DE server. I will be looking into that in the next days and hoping to get a fix as soon as possible!

It appears to be working since this morning!

I think they have some issues right now with their servers. I often see that my devices cannot sync even on china servers.

Hmm, seems like the Germany server is working again by using the old authentication. Can you guys confirm that the issue is fixed for you?

Can you guys confirm that the issue is fixed for you?

Yep, all seems to work just fine from this morning.

+1

+2
Sent from Yahoo Mail for iPhone
On Tuesday, January 4, 2022, 5:17 PM, Seger85 @.***> wrote:
+1
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.Message ID: @.***>

Ok, so it was indeed in the end just some kind of a bug with the DE server which caused the old authentication not to work anymore and they seem to fixed it. Good to hear 👍
I already did implement the new authentication thou (the same as the mi home app uses), and will be switching to that in a future update (no action will be needed from you), but at least i do not have to rush now and properly test everything!


Provide option to fail on timeout

Problem
We have noticed in one of our projects, that 100% of the mutants have been timed-out, no one was killed or survived.
The project was configured to fail when we don’t reach a mutations score of 100% and since a timeout is not counted as a survived test our mutation test in the CI pipeline of the project always passed.
When I’ve initially setup the project 10 months ago everything worked fine and I also had a detailed look at the HTML report. But since we reached 100% mutation score we never looked at the report or console output.
So we discovered this today by accident and we are very surprised that mutation testing basically stopped working since a couple of month.
Possible Solution
Provide an option that lets stryker fail whenever a timeout (or other kind of unexpected error) is encountered.

Hi @lo1tuma 👋 thanks for opening this issue.
Ai... 100% timeouts is... not good 😰
A timeout is not seen as "error" by Stryker, it is even counted as a killed mutant (that's why you're getting 100% score). The reason for this is that a timeout can be perfectly valid.
For example:
let i = 10;
while(i > 0) {
   i = i -1; // mutated to i + 1 by Stryker
}

The mutant i + 1 will result in a timeout in this case.
We're counting the timeout as a "killed" mutant because this is something test would have caught... because your test will probably timeout and fail in normal circumstances as well.
That said, I think we should maybe treat 100% timeouts as a special case and fail. Would that be OK for you?
Curious: did you find out what caused Stryker to end up in timeouts all of a sudden. That should not happen...

Hi @nicojs, thanks for your reply 🙂.
I haven’t thought about mutants that will cause an infinite loop, so I think your suggested solution would be better.

Curious: did you find out what caused Stryker to end up in timeouts all of a sudden. That should not happen...

My assumption is that we have those problems either since the update to v3 (update was done by renovate bot) or since a new big feature was implemented a couple of months ago which increased the total amount of mutants.
We are using the command runner with ava, which seems to be problematic since both tools try to utilize all available CPU cores. I’ve now limited the concurrency of ava and also added the fail-fast option. This already helped, and now I get zero timeouts again (although it is still quite slow, but that is a different problem).

a new big feature was implemented a couple of months ago which increased the total amount of mutants.

More mutants shouldn't equal more timeouts. The timeout time is calculated per test run, so more code means more time. This is the full formula that is used for the command test runner:
timeout = initialTestRunTime * timeoutFactor + timeoutMS
Side question: would you mind trying out the new beta? It might give you a performance improvement and you would help us out a lot with testing this specific AVA use case.
Please see https://stryker-mutator.io/blog/2020-07-13/announcing-stryker-4-beta-mutation-switching for more details.

so more code means more time

This might be the problem then.

timeout = initialTestRunTime * timeoutFactor + timeoutMS

Or this could be the problem as well. In case of ava and the command runner the initial test run is much faster, since ava can utilize all CPU cores and the command runner has not yet started to use multiple threads.. For the actual mutation tests both tools tried to utilize all CPU cores, which might slow down everything due to many context switches of the CPU cores.

Side question: would you mind trying out the new beta?

That sound interesting, I will give it a try.

Side question: would you mind trying out the new beta?

FWIW: I’ve just tested the beta with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Unfortunately the initial test run fails. It seems like one test file errors with error TS1160: Unterminated template literal.. All other tests (71) pass.


fix(deps): update time; fix build for rust 1.80

fix build for rust 1.80

Thanks. I'll update deps for 1.81 after this.


TyphoonDefinition.withClass + classOrProtocolForAutoInjection => Making both work together

Could you please tell me how would I resolve a component-by-type using TyphoonDefinition? Or am I missing something in the TyphoonFactoryDefinition?
func getClass<T>(props: NSMutableDictionary, type: T.Type) -> Any {

    Trial: 1-> When I try to init a class with TyphoonFactoryDefinition, am unable to resolve the class by type/key

    /*return TyphoonFactoryDefinition.withClass(SearchCategoryViewController.self, configuration: { (definition) in
        props.allKeys.forEach { (key) in
            if let key = key as? String, let value = props[key] {
                definition?.injectProperty(NSSelectorFromString(key), with: value)
            }
        }
        definition?.autoInjectionVisibility = TyphoonAutoInjectVisibility.byClass
        if let definition = definition as? TyphoonFactoryDefinition {
            definition.classOrProtocolForAutoInjection = SearchCategoryViewController.self
        }
    })*/

    Trial: 2 -> Here the definition gets registered and am able to resolve component by key,    BUT not by type because the property 'classOrProtocolForAutoInjection' is not available under TyphoonDefinition

    return TyphoonDefinition.withClass(SearchCategoryViewController.self, configuration: { (definition) in
        props.allKeys.forEach { (key) in
            if let key = key as? String, let value = props[key] {
                definition?.injectProperty(NSSelectorFromString(key), with: value)
            }
        }
        definition?.autoInjectionVisibility = TyphoonAutoInjectVisibility.byClass
        if let definition = definition as? TyphoonFactoryDefinition {
            /* control doesn't reach here cause it is not TyphoonFactoryDefinition */
            definition.classOrProtocolForAutoInjection = SearchCategoryViewController.self
        }
    })
}

And is there a way to access 'classOrProtocolForAutoInjection' in TyphoonDefinition?
Thank you :)

Hey @AdithyaReddy, this is not intended use of TyphoonDefinition, there is no way to use classOrProtocolForAutoInjection with TyphoonDefinition. Your code looks very custom, so I recommend you learn Typhoon sources and create own TyphoonDefinition subclass with isCandidateForInjectedProtocol and isCandidateForInjectedClass:includeSubclasses: methods overridden. See @interface TyphoonDefinition (/* Infrastructure */) category


fix(ci): better handle multi-arch images

:tada: This PR is included in version 1.0.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


added cmake build

added CMake build files

install headers in folder before include target
optionally build the example

Thank you! I only have on thing to discuss with you (see the review). Once cleared - I'll merge.

I think I'll soon merge this and I'll add some stuff I want fixed myself (for example, I want MetaStuff target to be defined for others to use)

ok!  glad it can be of use

@dand-oss take a look at this branch: https://github.com/eliasdaler/MetaStuff/tree/cmake
It's still a WIP, but I merged you branch into it and also did some small changes. For example, there's MetaStuff target now, so you can do:
target_link_libraries(${your-target} MetaStuff)


VideoLibrary stop working well

I think something is wrong, all my apps use VideoLibrary for Downloads videos stop working when I try download any video... This is the error:

Se proporcionó un URI de solicitud no válido. El URI de solicitud debe ser un URI absoluto o debe establecerse BaseAddress.
Translation:
An invalid request URI was provided. The request URI must be an absolute URI or BaseAddress must be set.

Any idea?

Same error for me!

Same thing on YoutubeExtractor, I think Youtube change something :/

Fixed it in my fork https://github.com/hig-ag/libvideo and added a pull request #61

Getting this error on the version I have from Nuget in Visual Studio 2015 with C#. Should I download your fork and compile it @hig-dev?

I'm getting the same error, glad to hear its not just me. Would be great to get pr #61 merged in and updated in NuGet for Visual Studio.

@TheLordAndrew You should if you don't want to wait for @jamesqo.

@hig-dev when do you think it'll be merged/jamesqo will write his own fix?

@TheLordAndrew I don't know. The latest commit was on 23 Jun 2016, so maybe never.

Yeah, that was my fear. Anyway, we'll see I suppose.

Fixed YouTube Decryption again

Fixed. Update 3.0.0


doc: Asynchronous Notifications: Public User API sys_notify_get_method() is not documented.

Describe the bug
Public User API sys_notify_get_method() function is not documented: https://docs.zephyrproject.org/latest/services/notify.html#c.sys_notify_get_method
Expected behavior
All Public User API functions should be documented.
Environment (please complete the following information):

Commit SHA 4a47803b0c06fa6081cfc1a1051433cc81ce0469

@butok Could you submit a PR for adding the missing Doxygen?

@butok Could you submit a PR for adding the missing Doxygen?

It should be done by an author of this function.

All Public User API functions should be documented.

This function is actually marked as internal. But since it's not done with the correct Doxygen construct, that makes it show up in the docs making people think it's public, while it's not


TST: dual_annealing set seed

Whilst developing code for a different minimiser this test started failing (without touching any dual_annealing code). I think it's due to the seed not being set on dual_annealing, different states of the numpy RNG could cause this. This PR sets a seed for the test that I'm experiencing the test failure.

I'm not certain, but I think the first time I saw this it was a different dual annealing test that failed. Maybe set the seed for the other dual annealing tests while you're at it?

Let's merge this as is to keep the ball rolling, and add seeds to other tests in a sequel. Thanks Andrew, Matt


support for multiple converters

cool! allow_list could maybe be turned into a command line argument in the future


Remove deprecated supports_streams() from spark_resource_adaptor.

Fixes #1814. (Possibly only partially.)
This change removes the override of spark_resource_adaptor::supports_stream(), in light of https://github.com/rapidsai/rmm/issues/1389, which deprecated the method.  This should get rid of the deprecation warnings in the spark-rapids-jni builds.
Similar to https://github.com/rapidsai/cudf/pull/14857, this change removes the deprecated override.

Build


videos

can we embed youtube or video links or playable audio in the slides?

The slides use reveal.js and are written in Markdown – so you can also include any HTML tags. Images are all embedded using <img>, so a YouTube embed code or HTML5 audio player, Soundcloud embed etc. should work, too. See here for a simple example of the markup.
If you want to adapt the course frame work for your own content, check out the Python starter repo: https://github.com/ines/course-starter-python


Fixed Stackoverflow for simpledoc

Description of the PR

included tests for validateSimpleDoc
fixed stackoverflow issue in validateSimpleDoc

PR Checklist

[x] All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
[x] All new changes are covered by tests
[ ] If GraphQL schema is changed, make generate has been run
[ ] If collectsub protobuf has been changed, make proto has been run
[ ] All CI checks are passing (tests and formatting)
[ ] All dependent PRs have already been merged

Thank you!


Improve Simulator CPU Usage up to 50%.

[X] Passes tests and documentation builds
[X] Passes just build
[X] Passes rustfmt
[X] No changes to public API.
[X] Closes #42

Description
This PR includes some minor changes to OutputSettings, Window, and SdlWindow which greatly improve the performance of the embedded graphics simulator.
Changes
Create TextureCreator and Texture at the initialization of SdlWindow, not on each frame.
@rfuest mentioned that the texture creator and texture had to be created on each frame because of a lifetime issue. I encountered this "self-referencing lifetime memory issue" too, but was able to overcome it by using the proc macro ouroboros on a new private struct SdlWindowTexture.
Limit FPS to 60Hz default.
One of the main factors that led to simulator programs usually taking ~100% of a CPU core was letting the program "run wild in the hot loop" by default. Now the program puts the thread to sleep if the draw took less than the desired fps. I set the default FPS limit as 60Hz, but this can be changed in OutputSettings. I imagine most programs written with the embedded-graphics framework in mind aren't FPS-dependent, but if they are I'm sure they can set the max fps to much higher in order to eliminate this effect.
Performance Improvements
I've found that with the input-handling example, the changes in this PR improve my CPU usage by ~50%!
Here are the results of ps -p <pid> -o %cpu,%mem,cmd on the master branch:
%CPU %MEM CMD
99.5  0.8 target/release/examples/input-handling

And on my fork:
%CPU %MEM CMD
49.7  0.8 target/release/examples/input-handling

No, LGTM. The only thing that is missing is a changelog entry for the new setting and the changed behavior of update.

I made a mistake naming this commit (CONTRIBUTING.md, not CHANGELOG.md), but it should address the issues @rfuest and @bugadani brought up.


Load Balancing Overview Does not Discuss "Azure Front Door + Azure Load Balancer"

On the Load-balancing options page, the docs do not discuss how / why you would use "Azure Front Door + Azure Load Balancer". However, the Decision tree for load balancing in Azure DOES show this combination as being a valid architecture option under certain circumstances.

It would be great to get the use case documented in the table:

Services
Article
Description

Azure Front Door + Application Gateway
Multitenant SaaS on Azure
Use a multitenant solution that includes a combination of Azure Front Door and Application Gateway. Azure Front Door helps load balance traffic across regions. Application Gateway routes and load-balances traffic internally in the application to the various services that satisfy client business needs.

Azure Front Door + AzureLoad Balancer

????

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 936c85d9-c953-2def-c399-77cbda32b24d
Version Independent ID: 9173c834-2c40-9169-b13b-6273759a2c8a
Content: Load-balancing options - Azure Architecture Center
Content Source: docs/guide/technology-choices/load-balancing-overview.yml
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @martinekuan
Microsoft Alias: pnp

@cancan101
Thank you for bringing this to our attention.
I've delegated this to content author @martinekuan, who will review it and offer their insightful opinions.

@Naveenommi-MSFT, in the future could you please not assign things to @martinekuan, as he is not a content developer nor in a position to directly add this content.

Hi @RobBagby, can you have a look at this?

Thank you for your feedback.  We have created internal ticket 245636 to track and investigate this issue.

#please-close


E2E - E2E Test workflow is flaky

For instance, this pull req required twice to pass the workflow. Some PRs require more and still fail to pass it.
https://github.com/hawtio/hawtio-next/actions/runs/5701830924

@mmuzikar Could you have a look?  Now it's very important to make the E2E stable so that we can receive and merge contributions without stopping them by the false alarms. Some checks in the E2E could be even degraded if it contributes to stabilisation of the check.

I have it on my radar. From my observations is almost always chrome issue. I will investigate further but for PRs we could temporarily not run on chrome.

@mmuzikar Right now, flakiness of the e2e tests with chrome is unmanageable. It's stopping smooth merging of pull reqs. Let me disable the chrome tests for a while. Once you can find a resolution to make it reliable, we can restore it again.

@tadayosi sure thing, I will continue to investigate sorry it's taking so long

CI has been stable for some time. Let's close it and we can revisit it with a new issue when we observe the flakiness again.


Fetch branches tool

This is the commit of the fetch branches tool that
will be in charge of checking the branch where a pull request
needs to be tested it.
Fixes #77
Signed-off-by: Gabriela Cervantes<EMAIL_ADDRESS>

@dlespiau  and @jodh-intel thanks for the feedback, I will work on the changes :)

@dlespiau and @jodh-intel changes were applied. Thanks

@jodh-intel thanks for the feedback, I did the changes. Regarding of using  https://github.com/mvdan/xurls, I did it and even implemented it however when I am doing the vendoring I see that it will upload around 11M as it is using or has many dependencies from google.golang.org, golang.org, and github.com. So, I decided not to use it :(

Hi @GabyCT - how about just using url.Parse()?

@jodh-intel , I tried using url.Parse however when we have the combination like branch_tests:https://github.com, it is complaining that the first path segment in URL cannot contain colon. So at the end, it did not help me to extract the URL.

Hi @GabyCT - sorry, what I meant was, split the branch_tests:https://github.com string apart as you are already doing, and once you've got what you think it the URL part of it, call:
_, err := url.Parse(urlPart)

Then check err. If it's nil, there's a good chance we really do have a URL. It's just a "safety-check" for verifying what we think is a URL actually probably is ;-)

@jodh-intel, ohhh got it :) thanks changes applied

lgtm

lgtm


Reproducer for Crop Box staying out of image.

Click on Select from album
Click Camera roll
Select the first waterfall picture
Click on Choose
Click on AlwaysOnePresetRatio

now when zooming in on the simulator with the option key pressed down, you can see at the end of the video sometimes the crop box just stays out of the image and does not bounce back. Here's a video:
https://user-images.githubusercontent.com/5759366/183083086-cd571a0e-8c3b-4872-ab19-c7f8ef583327.mp4
and a screenshot:

Hi @vanniktech
Thanks for reporting this issue. I also noticed it before but it seems only happened in simulators.
Does it also happen on your real devices?

Does it also happen on your real devices?

Indeed it does not. Thanks for responding so quickly!


Allow custom hosters

Hi,
This PR allow users to inject custom hosters.
Please can you merge it asap as we need it in our project ?
Thx

Thanks.


Add NullUnitConv class

Coverage increased (+0.008%) to 99.424% when pulling 507faae977b51f44a16f3fd493f1958ca82fb83d on T-Nicholls:nullUC into a38a5e8484ff305a1723a4eb3e6764328b2c790b on dls-controls:master.

Fixed the typos, this should be ready to merge now.


added presents

present button
picture backgrounds
viewed class
css
presents

@tichaelmurvey if something is in the past, and I've clicked on it, I think it should have a different color than when I've not clicked on it (e.g. coming to the website for the first time on Day 7).


Bump version v4.0.3

Signed-off-by: MyonKeminta<EMAIL_ADDRESS>
What problem does this PR solve?
Bumps version v4.0.3
Release note

No release note

/merge

/merge

/merge

/run-all-test

/run-all-tests

/run-test

/test


Organize CC reduction

Reducing cyclometric complexity in the Organize module

Codecov Report

Merging #349 (e789ea4) into master (edf0e19) will decrease coverage by 0.02%.
The diff coverage is 37.50%.

@@            Coverage Diff             @@
##           master     #349      +/-   ##
==========================================
- Coverage   71.80%   71.77%   -0.03%
==========================================
  Files         157      157
  Lines       28126    28119       -7
==========================================
- Hits        20195    20183      -12
- Misses       7931     7936       +5

Impacted Files
Coverage Δ

resqpy/organize/fault_interpretation.py
59.12% <37.50%> (-3.38%)
:arrow_down:

resqpy/organize/_utils.py
57.40% <0.00%> (-3.71%)
:arrow_down:

resqpy/organize/tectonic_boundary_feature.py
88.57% <0.00%> (-2.86%)
:arrow_down:

Continue to review full report at Codecov.

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


Added Dragon_World_Game(GSSOC '23) Issue: #244)

Added Dragon World Game under GSSOC '23, issue: #244
Screenshot:

Demo video:
https://github.com/ssitvit/Code-Canvas/assets/103595490/36c19312-7e0c-44d9-b283-5701fe85241a

@Dharmesh177 Please review the PR.

@Shreyaar12   @Dharmesh177 please review the PR and merge.

Add that in game

Please resove merge conflict

@cleveranu I have resolved the conflict, please review my PR

please add instructions how to play the game in the game


adding examples of  LICENCE / Dockerfile and kubernetes manifests

Hey man, thanks for the PR. I added an MIT license to a master branch. I don't think that Kubernetes files should be in the repo at all. Regarding Dockerfile, I also don't think that there is a case for it, because everybody has their deployment pipeline and most of the time people have their single web server serving multiple projects, so there is no need for a web server in a container.

Perfect.
Thx.


Endless Loop at end of deplyoment

Why is there an endless Loop in Run/MainLoop.ps1
Does not this slows the deployment?
On our test Server the systemsays "Ready for connections!" and sleeps till the end of days...

At this moment your service inside the container is ready to serve requests. The loop is endless, logically. In the standard images (not overrides by your own scripts) the code inside the loop just read and report related error messages (from Event Log).
Why slows the deployment? I don`t understand well. You can also kill the powershell session and work, the container is running in the background.
You can also use -d to start the container in detached mode but you won`t see the progress.
Please, read some docker basics if you are unfamiliar with Docker. It will help you to understand better.


Packaging Python vs C libs

Would this be a pseudo duplicate of #7?

Yes, I'll close #7. Some of the early topics were copied from surveys. Thanks for pointing it out. :)

Happened 2022-09-23 3:10 pm.


cannot read property of settings of undefined

hello there,
the best datepicker I've seen so far for angular 2 projects! thats why I really wanna use it ... butttt I have some integration errors..
I'm using the mgechev/angular2-seed project...
I installed the @vaadin/angular2-polymer and webcomponentsjs via npm...
loading them in the seed.config..
and added the bower packages...
as soon as I hit npm start.. everything compiles fine.. but I have an error in my console and i cannot figure out where it comes from
102 Error: TypeError: Cannot read property 'Settings' of undefined(…) "Report this error at https://github.com/mgechev/angular2-seed/issues"
I added a plunker with my component and the seed&config files...
https://plnkr.co/edit/qsjIAqZ146nrsaeqoIzs?p=catalogue
thats actually my first bower package.. so I'm not sure what exactly I'm missing out here
any ideas on this one?

Hello, the cause is that Polymer was not imported.
Just installing bower packages is not enough. You should also:

Import the Polymer elements you need, such as Vaadin Date Picker, by adding the HTML imports,

Make sure that the Angular 2 application code is imported after the HTML imports are loaded and the Polymer elements are registered.

Importing Polymer elements with HTML imports will also import their dependencies, including the Polymer library itself. This will resolve the error you have.
Please see the Step 4 of the Angular 2 with Polymer Elements Tutorial for the detailed instructions. These tutorial instructions are based on the Angular 2 QuickStart application code. We don’t have the instructions targeted for mgechev/angular2-seed-based projects. In our tutorial, we place HTML imports in the head section of the index.html file, and we postpone the System.import('app') call by wrapping it into an event listener for WebComponentsReady event. These implementation details might be different for your project base, though the requirements behind them are the same.
Apart from that, the error message you have is indeed not descriptive and unhelpful. This is an issue of the @vaadin/angular2-polymer package rather than of the Date Picker element. Filed there: vaadin/angular2-polymer#42


Support (ignore?) custom options

I'm using https://github.com/devongovett/protobuf-jsonschema which depends on this project to map protocol buffer IDLs to JSON schema. I also use https://github.com/GoogleCloudPlatform/protoc-gen-bq-schema/ to generate a BigQuery schema.
This last part requires providing a table name via a custom option named gen_bq_schema.table_name (see https://developers.google.com/protocol-buffers/docs/proto#options), for example
package foo;
import "bq_table_name.proto";

message Bar {
  option (gen_bq_schema.table_name) = "bar_table";

  message Nested {
    repeated int32 a = 1;
  }

  required int32 a = 1;
  optional Nested b = 2;
  repeated string c = 3;
}

At the moment, this fails:
/usr/local/lib/node_modules/protobuf-jsonschema/node_modules/protocol-buffers-schema/parse.js:90
        throw new Error('Unexpected token in message field: ' + tokens[0])
        ^

Error: Unexpected token in message field: en_bq_schema.table_name
    at onfield (/usr/local/lib/node_modules/protobuf-jsonschema/node_modules/protocol-buffers-schema/parse.js:90:15)
    at onmessagebody (/usr/local/lib/node_modules/protobuf-jsonschema/node_modules/protocol-buffers-schema/parse.js:161:26)
    at onmessage (/usr/local/lib/node_modules/protobuf-jsonschema/node_modules/protocol-buffers-schema/parse.js:210:14)
    at parse (/usr/local/lib/node_modules/protobuf-jsonschema/node_modules/protocol-buffers-schema/parse.js:583:30)
    at Compiler.open (/usr/local/lib/node_modules/protobuf-jsonschema/index.js:17:16)
    at new Compiler (/usr/local/lib/node_modules/protobuf-jsonschema/index.js:9:22)
    at module.exports (/usr/local/lib/node_modules/protobuf-jsonschema/index.js:192:18)
    at Command.<anonymous> (/usr/local/lib/node_modules/protobuf-jsonschema/cli.js:13:18)
    at Command.listener (/usr/local/lib/node_modules/protobuf-jsonschema/node_modules/commander/index.js:301:8)
    at emitOne (events.js:96:13)

As far as I can tell, this could safely be ignored for now, just like the reservedkeyword was yesterday

Ignoring is fine for now although it would be nice to actually expose option on the returned schema

Tested the new release, works as expected. Thanks for the quick work, appreciate it.


Rename frontend folder

After deleting folder with old frontend (#207), rename folder with current frontend from src.v2 on src.
src.v2 is located in Cooper/ClientApp/ folder.

#235


add support for Azure TDX preview

Whats the reason for 629e2bc?

The image definition name is not unique for each image. Uploading multiple images under the same definition will clash. The disk name is more unique, since it contains a name and version.


Delete metrics with policy label on policy delete

tetragon_policy_events_total metric has a policy label. When a TracingPolicy is deleted, the corresponding metrics are still exposed by Tetragon, leading to an overhead (usually not big, as policies are not churned frequently, but might be significant in some cases).
There was a similar problem with metrics that have a pod label. Now when a pod is deleted, Tetragon deletes its metrics from the registry (see https://github.com/cilium/tetragon/blob/main/pkg/metrics/metricwithpod.go).
We should add a similar handler for policy delete events. One small gotcha is that at the moment metrics don't distinguish between clusterwide and namespaced policies, so we should first add another label to distinguish between different policy kinds.

Hello @lambdanis
Just to get a proper understanding of the issue.
We are talking about the Policy_events_total metric. which does not have any Pod Delete Handler like this, registered in main. which makes sure that when a tetragon policy is deleted, corresponding metrics are also deleted.
So, to resolve this, following things need to be done:

[ ] Add a new label, that can tell if policy is cluster wide or namespaced. (what do you suggest the name of that label be ?)
[ ] Implement and Register handler, that will make sure tracing policy metrics are also deleted when the policy itself is deleted.

If I got it right, what is the deadline for resolution of this issue ? I would like to work on it.

Hi @prateek041

Add a new label, that can tell if policy is cluster wide or namespaced. (what do you suggest the name of that label be ?)

Yes, maybe "policy_kind" and "policy_namespace" (empty for clusterwide) would be good.

Implement and Register handler, that will make sure tracing policy metrics are also deleted when the policy itself is deleted.

Something similar. I think metrics deletion logic should be triggered from DeleteTracingPolicy in the sensors manager rather than directly from the k8s API watcher (WatchTracePolicy). That way stale metrics will be cleaned up also if policies are created via Tetragon API rather than via CRD.

Sure, thanks for the pointers, I have started working on it. Please assign it to me.


brew upgrade deck doesn't install 1.4.0

instead it says:
Warning: kong/deck/deck 1.3.0 already installed

just ran a brew install deck and now have 1.5.0. closing.


Node.js : error when using "now" module

i have installed js and now modules in my project .
"npm install js + npm install now + npm init"
the package.json generated  file seems correct
application  with "js" module works correctly.
but when using "now" module ("var nowjs = require('now');") their is an error :
"Error: Cannot find module 'now'.
module "now" is present in project\node-modules.
Do you have any idea for solving this
Thank you

From the 'now' docs (https://zeit.co/now#features), it doesn't appear that 'now' is a regular NPM package that you can require in that sort of way. It's a standalone CLI tool, not a package you can import.

Thank you


restorable: "pixels must not be stale when restoring" error is reported on Android

To fix this completely, we'd need to use Vulkan so that restoring is no longer needed 🤔

https://github.com/hajimehoshi/ebiten/blob/0f611b49ac19b9069b552c2a79c5c7e9ea13a12f/internal/graphicsdriver/opengl/context_mobile.go#L84-L89
These lines seems very suspicious: theoretically, chDone can receive even though there are still remaining GL tasks.
Then, how about calling GL functions directly instead of depending on golang.org/x/mobile/gl? I'll try later.
CC @wasedaigo

Hmm, worker usage cannot be removed due to gomobile-build. Hmm

Flush() is a sync function, which returns after actual flushing is finished. Then, the above code is not problematic. Really?

Looks like it was fixed.


Use options instead of default for theme controllers

When we added controller per theme in #134 we decided to let the user configure the alternate controllers in the route defaults. this means we need this cleanup code here: https://github.com/liip/LiipThemeBundle/blob/master/EventListener/ThemeControllerListener.php#L47-L53
Could we instead move that to the options part of the routing to avoid messing up the defaults? As there was no release of the 1.4 dev yet, we could fix this without keeping BC code in place.
@thormeier do you see any problem using options here?

fine for me

Fine for me as well, go ahead.

hm. the drawback with that is that we would need to go to the router, fetch the route collection and pick the "current" route to access the options. as done in https://github.com/thormeier/breadcrumb-bundle/blob/master/Provider/BreadcrumbProvider.php#L78
probably a bit too heavy on performance to do that in every request. lets keep with the way we have now.


Disallow selecting of tasks and teams

Changes

Disallow selecting teams and tasks through the UI (no change in color)
Add links to team description.
Add deadline + assignee + status of task to UI
Change font (not sure when it reverted back to using the initial one)

Codecov Report
Base: 57.53% // Head: 57.53% // No change to project coverage :thumbsup:

Coverage data is based on head (6ec1e75) compared to base (63bff0b).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #95   +/-   ##
=========================================
  Coverage     57.53%   57.53%
  Complexity      576      576
=========================================
  Files           128      128
  Lines          2230     2230
  Branches        266      266
=========================================
  Hits           1283     1283
  Misses          872      872
  Partials         75       75

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Unable to open app on Linux

Versions
EasyOpenX: Latest (source code)
Python: 3.10.1
Is the bug a crash?
No
Describe the bug
Added apps cannot be opened by program because os.startfile() does not work on Linux.
Steps to occur the bug
1- Run main.py on a Linux (in my case termux with python-tkinter and tigervnc packages)
2- Add some app and try to open
Screenshot

Fixed in EOX v1.1.


Cleanup

Found an error while trying to run the tests on Linux. Other than that,
only minor changes.

LGTM! I like the explicit continues. They make what's happening obvious.

Thanks, merged.