Why does JerryScript generate the all bytecode at once?

Hi there,
I have a question about compilation methodology of JerryScript.
It seems that JerryScript generates all bytecode for the whole source code at once (correct me if I'm wrong).
I think that it is inefficient for memory consumption because this approach holds the entire bytecode in memory but only part of source code is simultaneously executed during the runtime.
Is there any special reason for eager compilation?
And I have one more question.
Do you have any plan to implement the latest ECMA standard such as es10?
I ask you this because I also want to contribute to this project, so I'm searching for items now.

Hi @clover2123,
JerryScript uses compact byte code (CBC) format, which is a small representation of the source code. Usually the CBC form consumes much less memory than the source code (even with source code minification), which we discard after the compilation. We know this is not strictly ES compatible, since you cannot use toString() on functions, but on low-end systems reducing memory consumption is more important than keeping the source code.
Do you have specific features in your mind which you are interested? Currently the only major ES2015 feature which was missing is Proxies (but it is in progress), and some medium features such as function names. We have already did some simple ES2016+ features, such as allowing comma at the end of function call, and async function declaration parsing (but no implementation for await), but the rest is open to take.

@zherczeg Really thank you for the kind reply. I do not have any specific plan now, so I'll take a look at the code one by one first.
Close this issue.


🐛 Personal Profile: Leftover Images After Running Pytest

Description
Hi,
In the Personal-Profile app, I found out that after running:
pipenv run python pytest -v

the test pass, but create leftover images.
We need pytest to delete them automatically.
See the screenshot (commit<PHONE_NUMBER>e6e754ae1727eaaeffcfedaaf8af7c):

Also in PR #65 (commit dcef35d276e1cc84c5581030a102dc4762382816):

Hi, it seems that we have 2 problems in need of fixing here:

Deleting the files created from the tests, as can be seen, named "test_ resume" and "test_image".
Not creating multiple files from the data migrations (we have 4 users and whenever the migrations run they create more files each, although already exist).

I've tried many ways already: trying to delete them manually at the end of a test, using "get_or_create" instead of "create" in the data migrations, and also many different changes with no success.
Will continue to look further into it.

Walkthrough Of What I Have Done To View The Problem

Clean The Database
Steps I have made to setup a fresh start for the database:

I have run vagrant up.
I have the killed Django server, with:vagrant ssh
cd /vagrant
killall -9 python
exit

I have deleted the database (db.sqlite3 file).

Recreate The Problem

This has created the image files in the backend (that their name is appended with a generated ID, as expected), and inserted their path string to the database.

Let's it in the database though the shell:

As you can see, this works fine.
The profile_pic and resume return the path to the saved images we saw in the shell.

Now let's check this on admin view:

Create a superuser
Login with that user to http://localhost:8000/admin

It seems that the images in John Doe's profile are correctly linked to the images created in the database:

PROBLEM FOUND
Let's try to delete John Doe's profile, and see if the images get deleted from the database too.

PROBLEM
The images are still in the backend, though we have deleted the John Doe's personal_profile :

Conclusion
So this is why although pytest's tests delete the profile_profiles they create, the images it created still don't get deleted, and remain in the backend.
Suggestion (@ShirleyFichman tell me what are your thoughts about this)
Create our own implementation for PersonalProfile.delete that:

Deletes its created images in the backend:

Deletes the image that is in the path of: personal_profile.profile_pic
Deletes the image that is in the path of: personal_profile.resume

We can try this StackOverflow Answer for deleting Pillow images.

And deletes the personal_profile's record from the database (as the ORM does).

Suggested Implementation for personal_profile.delete
Override personal_profile.delete (thanks to this StackOverflow Answer),
by navigating to personal_profile/models.py,
and in the PersonalProfile class, add the following method:
class PersonalProfile(models.Model):

    def delete(self):
        file_paths_to_delete = [self.profile_pic, self.resume]

        # Filter list.
        file_paths_to_delete = [i for i in file_paths_to_delete if i is not None]
        if file_paths_to_delete:
            for file_path in file_paths_to_delete:
                os.remove(file_path, *, dir_fd=None)

        # Call Django's ORM delete this record.
        super(PersonalProfile, self).delete()

Suggested Implementation for personal_profile.delete
Override personal_profile.delete (thanks to this StackOverflow Answer), by navigating to personal_profile/models.py, and in the PersonalProfile class, add the following method:
import os
from app.settings import MEDIA_ROOT

class PersonalProfile(models.Model):

    def delete(self):

        # Get the media root path, and remove the first `/vagrant` folder.
        media_root_path = os.path.join(*(MEDIA_ROOT.split(os.path.sep)[2:]))

        file_paths_to_delete = [media_root_path + "/" + self.profile_pic.name,
                                media_root_path + "/" + self.resume.name]

        # Filter list.
        file_paths_to_delete = [i for i in file_paths_to_delete if
                                i is not None]

        # Delete files from the backend.
        if file_paths_to_delete:
            for file_path in file_paths_to_delete:
                print(f'deleting {file_path}')
                os.remove(file_path, dir_fd=None)

        # Call Django's ORM to delete this record.
        super(PersonalProfile, self).delete()

@taljacob2 Thank you so much for your kind help! Unfortunately, this still doesn't work, I'm trying to go in that direction of trying to specifically delete the files from the db.

I used https://stackoverflow.com/questions/16041232/django-delete-filefield to fix part of the bug- specifically, delete the files that were created for the tests!
The rest of the "doubled" files are created as part of the data migrations, I need to find a solution for that as well- but it is not connected to the tests.

https://stackoverflow.com/questions/44003620/how-do-i-run-tests-against-a-django-data-migration - I think this can help to solve this bug


"countUp" enhancement suggestion

A little enhancement suggestion - some users might need the script to count up from a past date. Add a parameter "countUp" (default: false) and in "refresh" function, after the seconds are calculated:
if (parameters.countUp && secondsLeft <= 0 )
secondsLeft *= -1;
and it'll work like a charm!

Hello @D-LUSiON !
Thanks a lot for your suggestion. I'm kind of busy right now so I don't have time to make this changes. But feel free to contribute by making a PR.
Thanks again :)

Greetings @D-LUSiON ! I just added the feature :) Took about one year to get time to add it, but it's here ;)


Create single active consumer documentation page

Covers AMQP exclusive consume and the new single active consumer feature. Created in a standalone page, but the content should be included in a new page dedicated to consumers.
[#161090309]
References rabbitmq/rabbitmq-server#1743

Closed in favor of https://github.com/rabbitmq/rabbitmq-website/pull/620.


Batch Replacement of Text

The Batch >> button in the Replace dialog extends the dialog box to include a list for multiple Find/Replace combinations with options. Pressing the Batch Replace All button then replaces all of the multiple matched strings with the respective specified strings. This user interface allows much faster and more intuitive replace operations for frequent use of the replace feature.

Can you explain the issue?

Fixed in latest commit release not published yet


reducing generic mezzanine content length to match mezzanine content for splicing

The mezzanine content to be used for splicing has a shorter length than the 60s used for the generic mezzanine content.
We should consider harmonising the length of the mezzanine content, e.g. reducing the generic mezzanine content to the same length as the splicing content.

The splice_main content is 10s which may be enough for sequential track playback (incl. content options), encrypted content playback but perhaps not for random access, switching set playback.
30s may be enough for the rest where 10s is too short.

In addition to the content for splicing tests, the following 30s mezzanine content was created and approved in mezzanine release v3:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>The 30s mezzanine was used to generate the appropriate test vectors in the latest batch created (2022-10-17), as defined in the AVC test content sparse matrix


Azure Netapp Files Snapshot Policy ETA

Hi, can you provide an GA eta for ANF snapshot policy which is currently under preview? We want this for planning purposes with our customers. Thanks.
regards
Raj

Hi Raj,
Is this a duplicate question to https://github.com/MicrosoftDocs/azure-docs/issues/71042 or are you asking about the GA for a different service?
As I replied on that thread, this space is for questions about the documentation, not product questions. Community members and Microsoft employees are active on Microsoft Q&A and will be able to help you get any information we are able to share about GA timelines.

Hi Samara,
Yes it is a duplicate. I used the link at the bottom of this page Manage snapshots by using Azure NetApp Files | Microsoft Docshttps://docs.microsoft.com/en-us/azure/azure-netapp-files/azure-netapp-files-manage-snapshots#manage-snapshot-policies but when I hit the raise new issue button I don’t know why it is taking me to the docs area. I have now posted in the Q&A link you provided. Many thanks.
Regards
Raj
From: Samara Soucy<EMAIL_ADDRESS>Sent: 23 February 2021 01:09
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Munusamy, Rajkumar<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Azure Netapp Files Snapshot Policy ETA (#71043)
Hi Raj,
Is this a duplicate question to #71042https://clicktime.symantec.com/3AwGgYGN3BFj1akJCHem3Rn7Vc?u=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F71042 or are you asking about the GA for a different service?
As I replied on that thread, this space is for questions about the documentation, not product questions. Community members and Microsoft employees are active on Microsoft Q&Ahttps://clicktime.symantec.com/3CDLVfPnXYA9KDQZv9GMEBA7Vc?u=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fanswers%2Findex.html and will be able to help you get any information we are able to share about GA timelines.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://clicktime.symantec.com/3C8iEhCND1TyLB1ZxnYNBEg7Vc?u=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F71043%23issuecomment-783792629, or unsubscribehttps://clicktime.symantec.com/3V5gfAPF5RNZcAYjGQajMTc7Vc?u=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FARIYWPWIGEDTTKCSUOTJTOTTAL5Y5ANCNFSM4YAWGVJA.
DXC Technology Company -- This message is transmitted to you by or on behalf of DXC Technology Company or one of its affiliates. It is intended exclusively for the addressee. The substance of this message, along with any attachments, may contain proprietary, confidential or privileged information or information that is otherwise legally exempt from disclosure. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient of this message, you are not authorized to read, print, retain, copy or disseminate any part of this message. If you have received this message in error, please destroy and delete all copies and notify the sender by return e-mail. Regardless of content, this e-mail shall not operate to bind DXC Technology Company or any of its affiliates to any order or other contract unless pursuant to explicit written agreement or government initiative expressly permitting the use of e-mail for such purpose.

Thank you for your response.
I checked and it looks like you did get a response on Q&A. I'm going to go ahead an close this issue out here.


feat: support user-supplied  html

If theme user creates a "custom-head.html" partial, it will be included at the end of the  section.
Related discussion: https://github.com/hugo-sid/hugo-blog-awesome/discussions/152
PR Checklist

[x] I have verified that the code works as described/as intended.
[ ] This change adds a social icon which has a permissive license to use it.
[x] This change does not include any external library/resources.
[x] This change does not include any unrelated scripts (e.g. bash and python scripts).
[x] I have enabled maintainer edits for this PR.

Thanks for the feature @jannes. Could you please add the relevant documentation on README?

Sure, I will update the PR when I am back at my computer :)

Does that look good to you? Not sure whether an example is necessary?


Page::repack

repack: method to union many small pages to one big.

implemented. now it in 'compaction'.


Error after uploading a new item via the rich text editor

Describe the Bug
I am seeing an error with the rich text field where a new uploaded item is added to the content, the editor will error. In production I was able to save and refresh the page and continue editing, however locally it errors out and I am unable to save the current item I am working on.
Link to the code that reproduces this issue
https://github.com/madebyjonny/upload-error-payload-reproduction
Reproduction Steps
I have provided a video of the error I am seeing on the repo provided
steps to reproduce

Navigate to a collection with a rich text field
type /upload upload a new file
once added the error accures

Which area(s) are affected? (Select all that apply)
Not sure
Environment Info
next: 15.1.0
node: v20.11.1
payload: 3.7.0

Thanks for reporting! This will be in the next release


Remember width of down panel when it's displayed on right

Summary
Enable localStorage to remember position for both displaying down panel on bottom or right.
Misc
Change function/variables names to be more semantic.

Hey @willowcheng
Thank you so much for this PR! We're moving over to a mono-repo, since that makes a lot of sense for a project like this.
We would ❤ it if you could create a PR here: https://github.com/storybooks/storybook
That'd be awesome, thanks!


Replace hardcoded ANCHORE_CLI_URL

./scripts/deploy.sh requires two env variables to execute, however only one was being used
By removing the hardcoded anchore URL, deploy.sh now functions as described and deploy.sh can now be used for on-prem installations

looks good to me!


Create Unit test to assert mixin is mixed in #755

tests/unit/components/sl-checkbox-test.js, line 39 [r1] (raw file):
Change "name" to null. '"name" property is null by default'

Comments from the review on Reviewable.io

tests/unit/components/sl-checkbox-test.js, line 33 [r1] (raw file):
Just to be consistent change to '"label" property is false by default'

Comments from the review on Reviewable.io

tests/unit/components/sl-checkbox-test.js, line 33 [r1] (raw file):
Ignore above. Test needs to read '"label" property is null by default'

Comments from the review on Reviewable.io

tests/unit/components/sl-checkbox-test.js, line 27 [r1] (raw file):
Test needs to read '"disabled" property is false by default'

Comments from the review on Reviewable.io

tests/unit/components/sl-checkbox-test.js, line 21 [r1] (raw file):
For consistency, add "is false by default"

Comments from the review on Reviewable.io

Review status: 0 of 1 files reviewed at latest revision, 4 unresolved discussions, some commit checks pending.

tests/unit/components/sl-checkbox-test.js, line 21 [r1] (raw file):
Done.

tests/unit/components/sl-checkbox-test.js, line 27 [r1] (raw file):
Done.

tests/unit/components/sl-checkbox-test.js, line 33 [r1] (raw file):
Done.

tests/unit/components/sl-checkbox-test.js, line 39 [r1] (raw file):
Done.

Comments from the review on Reviewable.io

@azizpunjani ready to be reviewed again.

Review status: 0 of 1 files reviewed at latest revision, 4 unresolved discussions, some commit checks broke.

Comments from the review on Reviewable.io

Reviewed, looks good now.

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from the review on Reviewable.io


[JWPlatform] support iframes

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections

[x] Searched the bugtracker for similar pull requests

[x] Checked the code with flake8

[x] I am the original author of this code and I am willing to release it under Unlicense

[x] Improvement

Description of your pull request and other information
Support iframe embeds for JWPlatform.  Include test.
Code such as:
<div class="jwvideo" style="position:relative; padding-bottom:56.25%; overflow:hidden;">
  <iframe
    src="//content.jwplatform.com/players/O0c5JcKT-zffGtjRq.html"
    width="100%" height="100%" frameborder="0" scrolling="auto"
    allowfullscreen style="position:absolute;">
  </iframe>
</div>

Implementation as simple as going from <script to <(script|iframe).
Example failure without this change:
pb3:youtube-dl jhawk$ youtube-dl -v 'https://www.mediaite.com/tv/dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone/'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.mediaite.com/tv/dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.11.15
[debug] Python version 2.7.10 - Darwin-14.5.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg git-2017-02-28-7f62368, ffprobe git-2017-02-28-7f62368, rtmpdump 2.4
[debug] Proxy map: {}
[generic] dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone: Requesting header
WARNING: Falling back on generic information extractor.
[generic] dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone: Downloading webpage
[generic] dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone: Extracting information
ERROR: Unsupported URL: https://www.mediaite.com/tv/dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone/
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 2170, in _real_extract
    doc = compat_etree_fromstring(webpage.encode('utf-8'))
  File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 2539, in compat_etree_fromstring
    doc = _XML(text, parser=etree.XMLParser(target=_TreeBuilder(element_factory=_element_factory)))
  File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 2528, in _XML
    parser.feed(text)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1642, in feed
    self._raiseerror(v)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1506, in _raiseerror
    raise err
ParseError: undefined entity: line 61, column 0
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 784, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 437, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 3075, in _real_extract
    raise UnsupportedError(url)
UnsupportedError: Unsupported URL: https://www.mediaite.com/tv/dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone/

pb3:youtube-dl jhawk$

And now:
pb3:extractor jhawk$ PYTHONPATH=~/src/youtube-dl python -m youtube_dl -v 'https://www.mediaite.com/tv/dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone/'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.mediaite.com/tv/dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.11.06
[debug] Git HEAD: 5871ebac4
[debug] Python version 2.7.10 - Darwin-14.5.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg git-2017-02-28-7f62368, ffprobe git-2017-02-28-7f62368, rtmpdump 2.4
[debug] Proxy map: {}
[generic] dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone: Requesting header
WARNING: Falling back on generic information extractor.
[generic] dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone: Downloading webpage
[generic] dem-senator-claims-gary-cohn-faked-a-bad-connection-during-trump-call-to-get-him-off-the-phone: Extracting information
[JWPlatform] O0c5JcKT: Downloading JSON metadata
[JWPlatform] O0c5JcKT: Downloading m3u8 information
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'http://videos-f.jwpsrv.com/content/conversions/buXmWndW/videos/O0c5JcKT-24616869.mp4.m3u8?token=0_5a15d27c_0x19d1e473edcf53d04b0fd1fa7369e45eaa1c2d48'
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 31
[download] Destination: Dem Senator Claims Gary Cohn Faked a Bad Connection During Trump Call to Get Him Off the Phone-O0c5JcKT.mp4
[download]  13.1% of ~15.61MiB at 817.54KiB/s ETA 00:09^C
ERROR: Interrupted by user

Also, while looking at JW, fix up a bad test for http://suffolk.edu/sjc

Thanks for this!


Windows GUI: Use file folder and name as export default for single file

When only a single file is open in MediaInfo, use that file's name and folder as the default when exporting instead of MediaInfo's AppData directory.
Not sure if can consider issue #741 as resolved since this PR is only for when there is a single file open.

Made the change. Now it will retain the file extension and add .mediainfo before the exported file extension. So even if user has files with same name but one is .mp4 and another .mkv for example, there is also no risk of overwrite.

Now: my test file.mp4.mediainfo.txt

Oops, I missed that the extension is already there.
Please "mediainfo" --> "MediaInfo" (I am a uppercase letters addict).


Side

Simulating binomial distribution

I see numpy is installed and the random.binomial method is used correctly. Good Job!


Merge Sample Type UI creation and editing into one page

I opened an issue to track problems arising from this refactor.


Test perftest rig on self-hosted runner

For #1282, #1311 & #1313.

/run performance test

/run performance test

/run performance test

/run performance test


feat(intervals): add basic support for intervals

For now it just renders the intervals on the chart.
It uses pink stripes and alpha transparency rects to preserve the parent fill color.
In future will add tooltip rendering to intervals and proper self time calculation for parent node
to show active self time and  interval time

Sorry for the long absence. I finally freed myself and now I can devote my time to open source.)

Sorry for the long absence. I finally freed myself and now I can devote my time to open source.)

Thanks for the review, unfortunately I am away from laptop until 9th january, doing vacation, so won't be able to do any code.
Will only check stuff on the phone.
If you feel to push something on top of this feel free, otherwise I will try to implement your suggestions when I am back.
Its nice to see you back, hope everything is going fine for you.

sorry for not being able to come back to this
I believe it is obsolete now


How to save to mp3 ?

Currently it has no feature like that but you may use some conversion libraries.
but i really do welcome pull request regarding mp3 :)
Basically I have provided the way to integrate mp3 in this library. Developer can extend The WriteAction interface to provide the implementation of mp3 encoding.
There is Lame mp3 library which we can use to encode Raw audio bytes to mp3 bytes but i think it is not working with latest android versions. So i havent provided mp3. But if there is any solution to encode which works in all versions i would surely add it to OmRecorder :)

hope support mp3 once again!!!!!!

hope support mp3 once again!!!!!!


Systemd ulimits #56

Include system defaults properties in environment.j2 template
to create unit kafka-broker service.

@jvalderrama This is a great idea.  I would actually recommend increasing the limit to 500k.  I would also recommend applying this change to all components, not just the Brokers.  Connect for example, can use a lot of file handlers.
We have also just released 5.3.0 where cp-ansible is now fully supported.  Can you rebase your changes to the 5.3.0-post branch.

@JumaX I had increased the limit to 500k like you suggested, I am also applied this change to all componets, it is: connect-distributed, control-center, kafka-broker, kafka-rest, ksql, schema-registry and zookeeper.

@jvalderrama Thanks for your work on this.  We've actually merged in this request as part of the latest merged pr, found here: https://github.com/confluentinc/cp-ansible/pull/124.  Will close this out.


Support for Code Interpreter in OpenAI Assistants

https://platform.openai.com/docs/assistants/tools/code-interpreter


[vk] handle device creation error more gracefully

Helps with https://github.com/gfx-rs/wgpu/issues/1261, not sure if it fixes it though

bors r+


Landing page from scratch

Is your feature request related to a problem? Please describe.
The landing page is way too inconsistent due to different size of the UI of different projects.
Describe the solution you'd like
Rather than placing projects in a flex box it would look better if for desktop and tablet version we could add apps in a back and forth slider and for mobile version we can use flex box as it won't cause inconsistency in small screens.
Kindly assign me this issue ( also please let me know if bootstrap can be used )

Yes bootstrap & tailwind can be used, Only thing to consider is to make code base of landing page very simple so that any new contributor can also list their project on landing page by themselves. Put html,css,js codes in separate files if needed.
Are you contributing for gssoc22?

@Astrodevil  Yes I'm contributing for GSSoC'22


api.flutter.dev has a robots.txt blocking search engine indicies

https://api.flutter.dev/robots.txt is intentional?  I think that's part of why api searches for flutter are so wonky.
Came across by searching https://www.google.com/search?q=inherited+widget+flutter and it finding api.flutter.dev as the first hit, but it telling me that it has no description due to the robots.txt.
@gspencergoog
@Hixie

Whoa.  That is the intentional robots file for https://master-api.flutter.dev, since we didn't want that to become the definitive site for search results, but it definitely shouldn't be the case for api.flutter.dev. Perhaps something publishing to that site is copying the one off of the master-api site?

@CaseyHillers @christopherfujino Do you know how publishing the docs is copying from the master site, and if it's copying this file?

let me investigate

@godofredoc is this related to your recent changes to the packaging builder?

Yes, this is related to my optimizations of the documentation deployment process. Taking a look.

Fix: https://flutter-review.git.corp.google.com/c/recipes/+/35881

I suspect this is expected, but FYI this is still the case:
https://api.flutter.dev/robots.txt is still there.
I noticed this afternoon when searching for "ChangeNotifier flutter" and not getting any api.flutter.dev links:
https://www.google.com/search?q=changenotifier+flutter

@godofredoc that link isn't accessible outside corp, is there a public link for people who want to follow along?

Recipes CL used to be public, opened https://github.com/flutter/flutter/issues/115195 to investigate.
The CL landed a few hours ago. https://ci.chromium.org/p/flutter/builders/prod/Linux docs_publish/7560 is regenerating the docs and will be published soon.

Update: this is fixed.
robots.txt content
# All robots welcome!

I'll double check with @gspencergoog on Monday to ensure everything is working properly.

This seems to be corrected now.

https://www.google.com/search?q=inherited+widget+flutter now shows api.flutter.dev as the first result.  Thanks!


Not all packages with extensions are recognized

I found at least 9 packages in f8.list that register an extension, but are not listed in ec_sort.md or bad_errorcodes.md.
They are:
flake8-docstrings-catnado: https://github.com/tylertrussell/flake8-docstrings-catnado/blob/76a9e202a93c8dfb11994c95911487b3722d75c9/setup.py
flake8-cohesion: https://github.com/sasanjac/flake8-cohesion/blob/0927f1d005186006fe7839a02977f62464517e8e/pyproject.toml
flake8-errmsg: https://github.com/henryiii/flake8-errmsg/blob/400ea1ee53ef6eb0dc5a6179ffbad33cb52c23ea/pyproject.toml
flake8-noreturn: https://github.com/izikeros/flake8-noreturn/blob/f101ff38bcfcbac2d1792dd6bb50dd5961c53b0e/pyproject.toml
flake8-airflow: https://github.com/bradleybonitatibus/flake8-airflow/blob/cae970cb14395e8fb8e9124862264cdce4e20241/setup.cfg
flake8-assert-check: https://gist.github.com/GideonBear/96cedf735c98ea77e8ab3703f3ecde11
flake8-cgx: https://gist.github.com/GideonBear/1392f8326453ef8a27ffc9f114cc3258
Thanks for this repo!

How are you finding the entry points from the packages?

I found all of them:
flake8-qchecker: https://github.com/James-Ansley/flake8-qchecker/blob/7876205df89eb016fa17a4173d80ebc112d059bf/setup.cfg
flake8-obey-import-goat: https://github.com/Melevir/flake8-obey-import-goat/blob/f33370ec4367b227463d6f4a0469f623eeeede2d/setup.py
flake8-match: https://github.com/asottile/flake8-match/blob/0693036af1202318107381b8d1fd7952e1f92540/setup.cfg
flake8-num-positional-args: https://gist.github.com/GideonBear/7fc1d5894c9b8588c8269d2ad8eebb02

How are you finding the entry points from the packages?

Brute force. I install them no-deps one-by-one into the CI environment, inspect/extract any new entry points that show up, then uninstall before moving on to the next.
Impressive that they can install extensions in a way that's invisible to that approach.
Thanks for the report, I'll take a look!

Brute force. I install them no-deps one-by-one into the CI environment, inspect/extract any new entry points that show up, then uninstall before moving on to the next.

Ah, so you're relying on importlib.metadata? Maybe this is a bug there then...
I haven't actually checked if the extensions work, but I'm assuming they do (otherwise the plugins are completely useless).

ilmd.entry_points().get('flake8.extension') is working for me: EntryPoint(name='Q', value='flake8_qchecker:Plugin', group='flake8.extension')
There is a deprecation warning though: <stdin>:1: DeprecationWarning: SelectableGroups dict interface is deprecated. Use select.

Ok, thanks for your work digging into this. I'll take a look when I have a chance & try to get this working correctly.
Very likely this is due to me using some old interface here -- I really need to freshen the whole thing...it's still on Python 3.8!

flake8-unused-globals was recently added to f8.list, but not to the .md files. Seems like this is a pretty common problem...

I might need to completely reevaluate the approach and re-implement. E.g., there's something odd afoot around part of #23, too.

@bskinn I would be glad to help if needed.

Much appreciated, @GideonBear, thanks!
I implemented the thing in a more-complex-than-is-ideal way with regard to passing the historical data along from run-to-run of the CI, so it's going to be somewhat tricky to do the surgery needed to upgrade. I'm going to have to get deeply back into the code to remember exactly how it all lies together; at that point, I'll reach out if I can use a hand on the implementation.
For now, I think the best thing you can do is to document what the best approach is that you can find for correctly filtering packages that show up in the simple API into either "yes, report this" or "no, don't report this" categories. Also looping the needs of #23 into this would be terrific.

Deleted packages on the JSON API give the same response as non-existent packages, while the simple API responds with an empty list of links. I think it's better to use the JSON API than the simple API, as it was made to replace the simple API. An alternative is to use the RSS feed to only get the data we want. I don't use RSS often, so I don't know exactly how to set that up, but it might be even better:

For periodically checking for new packages or updates to existing packages, use our RSS feeds.

We could query it for new updates (which includes new packages (with at least one release)) and removals, filter them to contain flake(8), extract the entry points from updated packages, and replace the old entry points of that package.
I cannot find anywhere how to get older items on (PyPI) RSS though, so it might be impossible.

I cannot find anywhere how to get older items on (PyPI) RSS though, so it might be impossible.

Yeah, IIRC that was the problem I had with trying to use the RSS feed to supply the inputs for the bot here---I had no way to guarantee I would be able to query the RSS quickly enough to be sure it didn't miss anything.
I agree, the new JSON API likely is a better option, I'll look into doing it that way.

pypi-simple (an API wrapper for the PyPI API) might also be useful.


Expense - LHN preview for request report shows owner name next to request edit system message

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

Version Number: v1.4.34-0
Reproducible in staging?: yes
Reproducible in production?: no
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Precondition: User is an employee of Collect workspace on Old Dot.

Create a distance request in the workspace chat.
Go to distance request details page.
Go offline.
Click Description.
Edit the description and save it.
Note that the LHN preview shows the owner name.
Go online.

Expected Result:
In Step 6, LHN preview for the expense details report will not show the owner name (current user) next to the system message.
Actual Result:
In Step 6, LHN preview for the expense details report shows the owner name next to the system message.
The owner name disappears after returning online in Step 7.
The issue is reproducible without going online. In that case, the owner name appears and disappears quickly.
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/24c4be89-9e12-425d-becf-3297d3ec3800
View all open jobs on GitHub

We think that this bug might be related to  #wave6-collect-submitters
CC @dylanexpensify

This isn't a blocker, too niche.

Looks like https://github.com/Expensify/App/pull/34872 might be PR that caused this? @koko57 could you take a look and confirm or deny?

@koko57 sounds good, thank you! Is there already an issue and reviewer for that next PR or do you think I should mark this as external, get a C+ reviewer for it, and assign you to this issue? I'm fine either way.

@blimpich I can do this as a part of LHN follow-ups issue
cc @mountiny

@mountiny copying it over to the linked PR sounds perfect to me, thank you both!

Added, thanks!


OAuth2Client verifyIdToken response 403 forbidden using ipv6

System:
Ubuntu 20.04
Nodejs v18.17.1
google-auth-library v9.0.0

Problem:
Recent versions of nodejs prefer ipv6 DNS resolution, and the OAuth2Client is getting 403: Forbidden from https://www.googleapis.com/oauth2/v1/certs unless ipv6 is disabled on the machine globally, or forced to ipv4 in node. Disabling ipv6 for the entire nodejs process just to get OAuth2Client to work is not ideal.
Repro:
On Ubuntu 20.04 having ipv6 properly configured,
curl -v https://www.googleapis.com/oauth2/v1/certs
Response will be
Your client does not have permission to get URL /oauth2/v1/certs from this server.  That’s all we know.

Disable ipv6 for the next test
sudo nano /etc/sysctl.conf

net.ipv6.conf.all.disable_ipv6 = 1
net.ipv6.conf.default.disable_ipv6 = 1
net.ipv6.conf.lo.disable_ipv6 = 1

sudo sysctl -p

Do it again
curl -v https://www.googleapis.com/oauth2/v1/certs
That should work as expected.
Or, in windows open cmd:
curl -6 https://www.googleapis.com/oauth2/v1/certs
curl -4 https://www.googleapis.com/oauth2/v1/certs

-6 will fail, -4 won't.
OAuth2Client needs to make requests using ipv4 resolution, at least for this specific url.
Thanks

Thanks for the feedback; this is more of a platform concern rather than this library (as demonstrated via the curl commands). However, you may be able to use --dns-result-order=ipv4first in the command line to resolve this issue without changing OS-level configuration:

https://nodejs.org/api/cli.html#--dns-result-orderorder

Or, in windows open cmd:
curl -6 https://www.googleapis.com/oauth2/v1/certs
curl -4 https://www.googleapis.com/oauth2/v1/certs

-6 will fail, -4 won't.

Actually, curl -6 https://www.googleapis.com/oauth2/v1/certs works (I've verified this locally as well).

Or, in windows open cmd:
curl -6 https://www.googleapis.com/oauth2/v1/certs
curl -4 https://www.googleapis.com/oauth2/v1/certs

-6 will fail, -4 won't.

Actually, curl -6 https://www.googleapis.com/oauth2/v1/certs works (I've verified this locally as well). Perhaps there's another networking issue?

Perhaps, although it is strange because the server and my personal pc aren't on the same ip or region and have the same issue, and I've found dozens of questions/issues on stackoverflow that mention various google apis not being resolved via ipv6.
Thanks for the tip about --dns-result-order=ipv4first however :+1:


avator update error

Hello, z-song.
Thank you for your awesome packages. It's so helpful to me.
That screen shot is shown when I try edit avator image.
Could you let me know why? and how can I slove it?

I did not reproduce the problem, this work fine for me.

這個錯誤發生在同一個form裡有multiselect

Fixed in version 1.3.x-dev

@z-song Thank you your answer.
Don't you guess why?
Sorry for bothering you.


[Network] Renamed flag to enable/disable the Networking module

Description
This PR renames the flag to enable/disable the Liqo Networking module and avoid ambiguities.

/rebase


Stuck on Refreshing

I've tried running Inspect.exe on both the x64 and x86 and both times it will get stuck on Refreshing. It is able to open the correct Power BI file and will even set it as the active window but I'm having trouble making it click on the Refresh via win.Refresh.click_input().

I have a similar issue, I am using Version: 2.85.985.0 64-bit (September 2020) which is obviously more recent than the latest pbixrefresher.py - the UI structure seems to have completely changed.

It does now work, so long as Inspect.exe is running in the background

Hmm interesting that you got it to work.. I gave up on using Inspect.exe and simply changed the code to use pyautogui key presses to invoke PBI keytips in order to refresh/publish. It seems to work fine but maybe I should try upgrading to the September 2020 version and see if Inspect.exe will work.


Improve warnings for strings with different comments

Lists all files the string was found in
Admittedly, this list can be quite long, so maybe this needs to be reverted again
Lists all different comments attached to the string

Example output:
$ wp i18n make-pot foo-plugin
Plugin file detected.
Warning: The string "Hello World" has 2 different translator comments. (foo-plugin.php:7, foo-plugin.php:10)
translators: Translators 1!
Translators: Translators 2!
Success: POT file successfully generated!

Fixes #227

Related: #119


Change config file references in docs from config.yaml to .lakefs.yaml

Following a question in the help channel, it is best if we use the same terminology everywhere when referencing the configuration file.

if we guide the user to create .lakefs.yaml which is usually the default name we load from the home directory - we skip the use of --config flag later and specify the file and just specify that the file should be located under home.

Thought about it. Adding a note everywhere that the config file is being used from the home directory felt like too much. However I do want to keep that file naming as .lakefs.yaml so users would always know that we reference the same configuration. I think that an always-working-but-longer-example is better than a sometimes-working-shorter-example, WDYT?

So two things - the previous example of using the full path to a config.yaml is right and works.
Creating a dot file and reference it with a full path looks off. It is usually used to configures something to work on the specific folder or some default usually found in the user home folder.
The mix of the two looks strange.
One way is to show how we would like our end users to configure the system and add notes about the custom way it can specify the configuration.
Alternatively, show the custom way and add notes about how the default will work.

Closing since it's not as confusing as we initially though. If this reoccur we'll find the middle ground and improve the wording of referencing the configuration file.


Extra headers

Add extra_headers for cache proxy
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?

Hey, can you explain what this change is and why it's needed? Thanks :)

@alexrudall
For setting openai cache TTL

@alexrudall it is also needed to set Helicone-Auth header

@alexrudall Can we please merge this?

Hi: @alexrudall
I've noticed this PR is still open. Could you let me know if there are any changes required for it to be merged?
Thanks

Would love to get this merged! I actually made a PR myself to address this very issue. Didn't realize this PR existed until just now 😅
@deltaguita comparing our diffs, I noticed you're not using the extra headers when using Azure. Something you might want to add.

I can't remember my exact specifics but when I merged this PR into my branch it was necessary to check if extra_headers were present before merging with other headers like this: https://github.com/ScotterC/ruby-openai/commit/06024ec35e99a133dd60fc8799b551f961e316a9

Would love for this to be merged as well.

Thanks for the PRs @deltaguita & @marckohlbrugge and others chasing this! Released in v4.3


Work with async import

From @garryyao on March 21, 2017 7:4
The new build analyzer is cool while it's a pity it doesn't work by default with 'async' import link, I expect the analyzer is not taking this import into the shared bundle
<link rel="import" href="../iron-icon/iron-icon.html">
<link rel="import" href="../iron-iconset-svg/iron-iconset-svg.html">
<link rel="import" href="font-awesome-icons.html" async>

And my polymer.json looks like:
{
  "entrypoint": "index.html",
  "shell": "loader.html",
  "fragments": [
    "bower_components/px-polymer-font-awesome/font-awesome-icons.html",
  ],
  "source": [
    "elements/**/*",
    "images/**/*"
  ],
  "includeDependencies": [
    "bower_components/webcomponentsjs/webcomponents-lite.min.js",
    "bower_components/font-awesome/fonts/*"
  ]
}

Right now in order to workaround this, I have to imperatively import this per module to make it work:
<link rel="import" href="../iron-icon/iron-icon.html">
<link rel="import" href="../iron-iconset-svg/iron-iconset-svg.html">
<dom-module id="px-polymer-font-awesome">
  <script>
    Polymer({
      is: 'px-polymer-font-awesome',
      created: function() {
        this.importHref(this.resolveUrl('./font-awesome-icons.html'));
      }
    });
  </script>
</dom-module>

Copied from original issue: Polymer/polymer-build#164

Hopefully this concerns "deferred" attribute as well

@garryyao for clarification-- the behavior you would expect is that a defer'd or async'd import would not be bundled with their dependent file?

Hey, since HTMLImport is not going into the future Polymer so perhaps this's less critical now...


Hacktober Feature Request: Add Gallery of Speakers Page

Describe the feature

A page which has a grid of images of past speakers
Images could have speakers social links

Advantages

An easy place to see and find out about all our great past speakers

Acceptance Criteria
Functional and non-functional criteria that should be met by this work

[ ] Page with a gallery of speaker images
[ ] The page is responsive and the grid works on desktop, tablet, and mobile

I want to contribute to this issue

Sounds great @Divyanshwick! Consider yourself assigned!

Hey @pjgpetecodes, please can you add somewhere that the contributor can access the images?

Yup... I'll go and collect a bunch of photos and update the issue once I've sorted them!

I've now created a pull request containing the speaker photos... Once it's approved, I'll update this Issue.

@Divyanshwick - the photos are now available in the repo 😄


4.5.2 Release notes, version bump and readme broken link fixes

Docs updates for v4.5.2

the changelog should only include what is already merged. and then each pull request add its part


Mslq52 55

off-the-top-of-my-head version. while complete, I feel that proper way to verify would be with an actual setup.
This version also possesses updates to convention/terminology that we might want to standardize. hence the decision to throw it into review.

Looks good to me!

Made a couple of changes to the formatting to fix a few minor things I saw. Otherwise I think this is good to go if there isn't any other feedback and the checks seem to pass.


ERROR: get_path: Condition ' !is_inside_tree() ' is true - when returning from editor to project manager

When closing an open project to return to the project manager, the following messages are shown:
ERROR: get_path: Condition ' !is_inside_tree() ' is true. returned: NodePath()
   At: scene/main/node.cpp:1153.
ERROR: get_path: Condition ' !is_inside_tree() ' is true. returned: NodePath()
   At: scene/main/node.cpp:1153.
ERROR: get_path: Condition ' !is_inside_tree() ' is true. returned: NodePath()
   At: scene/main/node.cpp:1153.
ERROR: get_path: Condition ' !is_inside_tree() ' is true. returned: NodePath()
   At: scene/main/node.cpp:1153.

There are repeats several times, but it varies based on the project.

I can confirm this bug with current master.

I can confirm this even if not returning to Project Manager, but just closing the editor. (on 19a8f9b74d9ef7a4e2e9f5b1d1333995e2384943 )

This error comes from here: https://github.com/godotengine/godot/blob/8adc4ef65b76ee0e8a5fdf6c773d47716fa8511f/scene/main/node.cpp#L1153
So whatever calls Node::get_path() after the editor has been closed needs to be made aware that it's not a good idea :)


Warnings from KDE

Hello,
as I mentioned in my PR #3, there seems to be some UserWarnings raised when KDE is fit with a small number of samples. I put here a small chunk of code which reproduces the warning; that is using my code from #3, so using the ABCpy inference scheme. I have not tried with the other algorithms yet.
I realize there is not much you can do about this as it is due to KDE, but maybe it can be helpful to provide a more explicit warning message saying that the number of samples for KDE are small? Not sure, I realize also this is not super important.
import sbibm

task_name = "two_moons"

task = sbibm.get_task(task_name)  # See sbibm.get_available_tasks() for all tasks
prior = task.get_prior()
simulator = task.get_simulator()
observation = task.get_observation(num_observation=1)  # 10 per task

from sbibm.algorithms.abcpy.rejection_abc import (
    run as rej_abc,
)
num_simulations = 1000
num_samples = 10000
posterior_samples, _, _ = rej_abc(
    task=task,
    num_samples=num_samples,
    num_observation=1,
    num_simulations=num_simulations,
    num_top_samples=30,
    kde_bandwidth="cv",
)

Thanks for the issue @LoryPack.
#8 adds additional information to the logging message when using KDE. We give this additional general info instead of an actual UserWarning because it is not clear when exactly there too few samples, e.g., how it depends on the number of samples and the number of dimensions of the distribution.

Thanks for that


Ensure lock path is a unicode string

Kazoo expects paths to be either ascii-limited strings or unicode strings. Entity keys can contain non-ascii characters.

https://ocd.appscale.com:8080/job/Daily Build/3672/


[AutoPR] cdn/resource-manager

Created to accumulate context: cdn/resource-manager

Closing PRs that have been open for more than 1 month.


Implement span ID deduper

Coverage remained the same at 100.0% when pulling bde46d8a557f250322ed53b69b6360c9558ba10a on adjusters into ee6c21b4d0b33503d438d25b2291f3c37c6d019b on master.

Coverage remained the same at 100.0% when pulling 28bb30b393a72d369e14d99d4e0d4f141f37f81b on adjusters into 5dc88f47a92b6649e0639ebc745fd8f0196678a9 on master.


File record crash

when recording from a file and then recording from another, a crash occurs

It may be related to a difference of image size


inlineResources not working (fetchResource() Error: No URL passed)

Before I begin, thank you for this awesome library.
inlineResources is not working:

This error was introduced in version 0.11.5 (issue #113, fix #116). Version 0.11.4 works fine. See screenshots below.
I suspect that fetchURL doesn't know to look in xlink:href instead of href.
Therefore element.href.baseVal is undefined and it outputs No URL passed (inline.ts, line 27)
JavaScript code used
I'm using the CDN version as recommended in #135. I've tested both versions.
async function takeSVGScreenshot(){
	console.log('[IK SVG] Importing dependencies...');
	const { elementToSVG, inlineResources } = await import('https://cdn.skypack.dev/dom-to-svg@0.11.5'); // or 0.11.4
	console.log('[IK SVG] Taking snapshot...');
	const svgDocument = elementToSVG(document.querySelector('#qtlw13'));
	console.log('[IK SVG] Inlining resources...');
	await inlineResources(svgDocument.documentElement); // <-- this is where the error occurs
	console.log('[IK SVG] Serializing...');
	const svgString = new XMLSerializer().serializeToString(svgDocument);
	console.log('[IK SVG] Sending data back to server...');
	callHTMLClient( '{"Sender":"TakeSVGScreenshot", "Data":"' + svgString.replace(/\"/g, '\\"') + '"}');
	console.log('[IK SVG] Done');
}

Version 0.11.4

Version 0.11.5

don't mind the impossible geometry :)

This is on Google Chrome btw.

This is on Google Chrome 95. It's not an issue on Firefox 94 yet.
xlink:href was deprechated in SVG 2 (MDN, spec from 2018)
Using element.getAttribute('xlink:href') instead of element.href.baseVal works.
Using element.getAttributeNS('http://www.w3.org/1999/xlink', 'href')also works. Tested both on Chrome and Firefox.
Assigning the url to both the xlink:href and href attribute could probably also fix it (messy though).
Sadly I'm currently unable to make and test a pull request :/

谢谢，这很有帮助。

This issue on the Inkscape issue tracker

Inkscape 1.3 has been released on July 23, 2023.


Python widget: use environment_detector

Currently we do this:
# TODO
        # FairnessDashboard._service.env.display(html)
        display(HTML(html))

which should be replaced with the environment_detector

This has been standardized between dashboard a while ago and now looks as follows:
self._service.env.display(html)


[SPARK-20173][SQL] Throw NullPointerException when HiveThriftServer2 is shutdown

What changes were proposed in this pull request?
Throw NullPointerException when HiveThriftServer2 is shutdown
How was this patch tested?
manual tests

Can one of the admins verify this patch?

@zuotingbing i think you included the wrong changes here?

@srowen yes, i will fix it right now. Thanks!  :)


Issue with bower package?

I've had problems whilst trying to work with this example http://codepen.io/SitePoint/pen/20353f8838bff9907a4ebf57d29010b4
When using the bower package this doesn't seem to compile correctly, the columns don't behave as they should. If I replace the bower package with the source from here then the example works fine.

Closing for now. If it is an issue you still find present in recent versions, please make a new issue!


SIV Doesnt work on Node, because of overwrite of slice Function in Node Buffer Impl

Hi, as the Node Buffer Implenetation .slice doesnt copy the array but just returns a new reference to same memory space the SIV decrypt breaks, when you give a Node Buffer to the decrypt func,  as the tag(last byte and the first with the counter) will be altered by the decryption logic, and the the byte compare fails. It can be solved by using the Uint8 Array Function directly in the aes logic on line 46 as Uint8Array.prototype.slice.call(tag). If you want i could open a PR for fix in the next Days

We only support Uint8Array-s because of reasons like that. If you're using Buffer, you're doing it wrong. Convert buffer to u8a first. Buffers are not cross-platform-compatible, there are no buffers in web.

In fact, by using Buffer, you may be having some security issues. For example, if you store private key in a buffer, it's likely globally accessible to all node.js functions through global buffer.


ng serve live reload takes a long time after initial load

Which @angular/* package(s) are the source of the bug?
compiler-cli, compiler
Is this a regression?
Yes
Description
I opened a new project with angular 17.  When I run 'ng serve', the initial browser load is normal and as I know from previous angular versions. But when I try to live reload after some minor change to my code, the "Reloading client(s)..." takes a long time to execute if successful at all. I had to exit the ng serve and rerun it for it to work again. Here is my bundle size:
Initial Chunk Files | Names         |  Raw Size
styles.css          | styles        | 342.01 kB |
polyfills.js        | polyfills     |  82.71 kB |
main.js             | main          |   8.73 kB |
                | Initial Total | 433.45 kB

Application bundle generation complete. [0.131 seconds]
Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
live reload takes a long time if works at all and rerun of ng serve is needed to reload the browser.

Please provide the environment you discovered this bug in (run ng version)
Angular CLI: 17.0.0
Node: 20.9.0
Package Manager: npm 10.1.0
OS: win32 x64

Angular: 17.0.2
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1700.0
@angular-devkit/build-angular   17.0.0
@angular-devkit/core            17.0.0
@angular-devkit/schematics      17.0.0
@angular/cdk                    17.0.0
@angular/cli                    17.0.0
@angular/google-maps            17.0.0
@angular/material               17.0.0
@schematics/angular             17.0.0
rxjs                            7.8.1
typescript                      5.2.2
zone.js                         0.14.2

Anything else?
No response

i think there has a been a few reports where ng serve is crashing and all you see on the CLI is "reloading clients"
have not been able to narrow down exactly why this is happening

Could this be a duplicate of #26334? Can you try again on 17.0.2 and let us know if it's still an issue for you?

Could this be a duplicate of #26334? Can you try again on 17.0.2 and let us know if it's still an issue for you?

Apologies for not seeing it's a duplicate.
I updated the version now, it does seem to resolve the issue!
It still says it reloads the clients even though I see it is already loaded, but the actual issue is resolved.

Glad to hear that fixes it.
Duplicate of #26334


With Chrome 61 some DDC tests started failing

Affected tests:
# In corelib_2:
double_parse_test/01: Skip
double_parse_test/02: Skip
double_parse_test/03: Skip
double_parse_test/04: Skip
double_parse_test/none: Skip

# in lib_strong:
html/client_rect_test: Skip
html/css_test/supportsPointConversions: Skip
html/css_test/functional: Skip

See https://dart-review.googlesource.com/c/sdk/+/3521

This is only failing in FF now. See: https://github.com/dart-lang/sdk/issues/49855


Fix for the Jira Issue : EMM-1008

I have notice many  console.log on js. Why do we need to push them to final code?

These method identified as send system related operations to usage which does need to be notify to the end user when the introduction of backend invoker .
But in the PR[1] the above mentioned console logs were translated to User interface appropriately.
[1]. #166


Update heketi vendor update to sha@300752517908f3456e90d1ee2a773e7d8ce1afe

Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

../../vendor/github.com/go-openapi/analysis/flatten.go:27: undefined: spec.ExpandOptions

../../vendor/github.com/go-openapi/analysis/flatten.go:28: undefined: spec.ExpandOptions

../../vendor/github.com/go-openapi/analysis/flatten.go:46: too many arguments in call to spec.ExpandSpec

	have (*spec.Swagger, <T>)

	want (*spec.Swagger)

../../vendor/github.com/go-openapi/analysis/flatten.go:155: undefined: spec.Debug

../../vendor/github.com/go-openapi/analysis/flatten.go:406: undefined: spec.Debug

../../vendor/github.com/go-openapi/analysis/flatten.go:443: undefined: spec.Debug

../../vendor/github.com/go-openapi/analysis/flatten.go:455: undefined: spec.Debug

../../vendor/github.com/go-openapi/analysis/flatten.go:539: undefined: spec.Debug

../../vendor/github.com/go-openapi/analysis/flatten.go:543: undefined: spec.ResolveRefWithBase

../../vendor/github.com/go-openapi/analysis/flatten.go:550: undefined: spec.Debug

../../vendor/github.com/go-openapi/analysis/flatten.go:550: too many errors

# github.com/kubernetes-incubator/external-storage/vendor/k8s.io/client-go/util/flowcontrol

../../vendor/k8s.io/client-go/util/flowcontrol/throttle.go:59: undefined: ratelimit.Clock

make[1]: *** [build] Error 2

make[1]: Leaving directory `/home/travis/gopath/src/github.com/kubernetes-incubator/external-storage/local-volume/provisioner'

make: *** [local-volume/provisioner] Error 2```

Closing this PR as this is taken care in other PR.


Support Pandas 2.0.1

There is at least one bug introduce in pandas 2.0.1
https://github.com/kmax12/gridstatus/issues/227#issuecomment-1560441715

Hi @kmax12 -- came across this repo recently. Any plans to tackle Pandas 2 support in the near future?

it's not currently scheduled to be worked on, but definitely still on our radar. we know there is at least one issue, but we haven't sized it up yet
first step here is to do the upgrade and then see how the test cases do
if there isn't much to fix, i suspect we can add support somewhat quickly. And we'd then drop Pandas <2 support.

I wonder if we should wait until Pandas 3 is out and skip over 2? The stable release is scheduled for April 1, 2024.

Completed in #345 and released in #349 thanks to @DavidRothblum!


App

[Enter feedback here]

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

ID: c56f2824-b04c-1e07-2de1-2f78f84834d1
Version Independent ID: b73656a9-ba3d-2411-dfca-031586aad016
Content: Choose your app platform
Content Source: hub/apps/desktop/choose-your-platform.md
Product: uwp
Technology: apps
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

It looks like there is no actionable feedback here, so I am closing the issue. Please open a new issue if you have feedback you would like to be addressed.


ASUS H87I-PLUS Can't flash modified BIOS; Security verficiation failed.

System

Motherboard: ASUS H87I-PLUS
BIOS Version: 2003 (2014-11-05)
GPU:
[x] CSM is turned off. Make sure to confirm this in the BIOS and not with GPU-Z or similar since it can be inaccurate
[ ] 4G decoding is enabled. Make sure to confirm this in the BIOS and not with GPU-Z or similar since it can be inaccurate
[x] UEFIPatch is applied (see Using UEFIPatch for more information). On some motherboards DSDT Patching is also needed
[x] I have read Common issues (and fixes)

Description
My ASUS motherboard doesn't support BIOS Flashback, so I attempted to flash via the BIOS interface (ASUS EZ Flash 2 Utility v02.00). When I try to flash the modified BIOS however, I get a Security verification failed. error.
I'm not sure how to bypass this security feature. I've tried another method using the ASUS DOS bupdater.exe, but I get the same issue, and I can't even do a BIOS backup. I have Secure Boot set to Other OS, there's no Disabled state from what I can tell.
Does this mean I'm SOL on getting Resizable BAR enabled on my motherboard?
Specifications: https://dlcdnets.asus.com/pub/ASUS/mb/LGA1150/H87I-PLUS/E8210_H87I-PLUS.pdf
4G decoding doesn't seem to be enabled or available on my motherboard. I tried to find the 4G Decod or Above 4G but only seem to find Memory Remapping.
This is the output of lspci -v of my system:
03:00.0 VGA compatible controller: Intel Corporation Device 56a6 (rev 05) (prog-if 00 [VGA controller])
	Subsystem: ASRock Incorporation Device 6007
	Flags: bus master, fast devsel, latency 0, IRQ 28
	Memory at f6000000 (64-bit, non-prefetchable) [size=16M]
	Memory at e0000000 (64-bit, prefetchable) [size=256M]
	Expansion ROM at f7000000 [disabled] [size=2M]
	Capabilities: [40] Vendor Specific Information: Len=0c <?>
	Capabilities: [70] Express Endpoint, MSI 00
	Capabilities: [ac] MSI: Enable+ Count=1/1 Maskable+ 64bit+
	Capabilities: [d0] Power Management version 3
	Capabilities: [100] Alternative Routing-ID Interpretation (ARI)
	Capabilities: [420] Physical Resizable BAR
	Capabilities: [400] Latency Tolerance Reporting
	Kernel driver in use: i915
	Kernel modules: i915

This is the output of dmesg | grep "root bus resource":
[    0.217132] pci_bus 0000:00: root bus resource [io  0x0000-0x0cf7 window]
[    0.217135] pci_bus 0000:00: root bus resource [io  0x0d00-0xffff window]
[    0.217137] pci_bus 0000:00: root bus resource [mem 0x000a0000-0x000dffff window]
[    0.217138] pci_bus 0000:00: root bus resource [mem 0xe0000000-0xfeafffff window]
[    0.217140] pci_bus 0000:00: root bus resource [bus 00-3e]

Some other guy told me, ASUS BIOS.rom has their own header that design for EZ Flash, need convert to .cap to remove header, maybe the the main factor cause UEFI tools ruin ASUS motherboard.

Some other guy told me, ASUS BIOS.rom has their own header that design for EZ Flash, need convert to .cap to remove header, maybe the the main factor cause UEFI tools ruin ASUS motherboard.

@Daishuku Ah so that means I'm probably SOL then?

There is no reason for you to flash BIOS if you use Linux, all you have to do is the Linux DSDT fix where you add a QWordMemory entry which the wiki explains


docs: remove array

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Simplify required provider services

Provider writers are required to implement/extend IDatabaseProviderServices. In stubbing out an empty provider, I found the following APIs cumbersome. Can we review/cleanup the following?

Required provider service EntityQueryModelVisitor.

It is abstract but has no abstract members.
It has 15 constructor arguments. Can we consolidate?
Providers must implement IEntityQueryModelVisitorFactory. This must also pass 15 constructor arguments along to EntityQueryModelVisitor. This means means each provider ends up with a class that looks like this: SqliteQueryModelVisitorFactory, even if they are not changing any base functionality. Can we get around required this kind of implementation?

These abstract classes implement required provider services but they have no abstract members. Are they intentionally abstract or is this left-over from API changes? Do providers really need to implement them if we have fully-functioning base implementations?

EntityQueryModelVisitor
ValueGeneratorCache
ModelSource
RelationalCompositeMethodCallTranslator
RelationalCompositeMemberTranslator

(nit) IDatabaseProviderServices is in the ".Storage" namespace. IMO seems like ".Infrastructure" would me more appropriate.

ValueGeneratorCache and ModelSource need to be implemented by providers so that each provider gets its own independent cache of these things.

Sending back to triage.


Fix #48

See #49 for more information.
Reopened because it was accidentally closed.
Closes #48
Blocked by https://github.com/crystal-lang/crystal/issues/12894

Tried again with crystal 1.10.1 and it still crashing the compiler 😭 .
I'll look for a workaround on this... or try to investigate the bug in the compiler itself.


Revert "[ffmpeg-vaapi] Update FFMPEG-VAAPI Encode AV1 cmd line"

Reverts intel/vaapi-fits#615

ffmpeg don't have option about -icq


Serve Third-Party JavaScript Locally

Originally reported by @dbrumley:

bootstrap and jquery should be served locally, unless at deployment their is a bandwidth issue. The reason is you cannot do development or deployment without internet access.

At the very least we should modify it such that the development environment serves them locally if not the production environment. The Jekyll configuration already does something similar for analytics.

Closed and ported to picoCTF-Web the actively maintained version.


Add convenience input/output functions

[ ] ChemKED class instance to ChemKED YAML
[ ] CSV to ChemKED datapoints record
[ ] ChemKED class to Pandas DataFrame
[ ] ChemKED class to NumPy record or structured array
[ ] NumPy record or structured array to ChemKed class
[ ] NumPy record or structured array to ChemKED datapoints record or ChemKED class instance?
[ ] Pandas DataFrame to ChemKED datapoints record or ChemKED class instance?
[ ] Excel file to ChemKED datapoints record or ChemKED class instance (via DataFrame?)?

For the CSV, NumPy, Pandas, and Excel input options, we could have the user supply the reference DOI as well to have it automatically create the reference object

Sure, we could probably develop that as a separate output function, then call it from any of these functions if the DOI is optionally passed


this exporter not working for ilo3 hpdl360g7 servers

this exporter not working for ilo3 hpdl360g7 servers

@Sudermoni This repository is not maintained anymore, can you try the community fork https://github.com/hpilo-exporter/hpilo-exporter/
besides that can you please provide more details what is not working, share errors messages etc.
also hp dl360g7 is pretty outdated already, not sure if this will be fixed soon. but MR are always welcome!


feat: add get_home_directory to repo_utils

Easier to use and read than get_env("HOME").... especially on Windows.

Type of change

New feature or functionality (change which adds functionality)

meh, this feels unnecessary to me but I won't stop you :)
https://github.com/bazel-contrib/rules_oci/pull/308/files felt fine to me

Feel wrong to get_env("HOME") on Windows although I don't particular care for Window 🤷 Main reason is probably readability downstream.

What changed your mind? This is what I suggested earlier to hide the special handling of HOME and maybe remove that handling from get_env in the future... can we add a TODO for that?

What changed your mind? This is what I suggested earlier to hide the special handling of HOME and maybe remove that handling from get_env in the future... can we add a TODO for that?

Slept on it and figured it reads like a bug if the rule set supports Windows and you're getting HOME


[sival] rv_core_ibex_nmi_irq_test_silicon_owner_sival_rom_ext failure

Description
From @pamaury: This test triggers an alert to test the NMI but it seems that once the alert has been triggered and the NMI received, the alert handle reports an idle status for the class instead of Phase0 so the test fails. This needs more investigation.

I think @nbdd0121 has investigated the test in more depth and might comment here?

@moidx is currently investigating this. If there are any comments from others please provide them

Configuring the alert_handler triggers nmi interrupt without any alert class association.
https://github.com/lowRISC/opentitan/blob/605da270ddcf70b2af5573e9524f9613bc89383d/sw/device/tests/rv_core_ibex_nmi_irq_test.c#L98-L101
The alert NMI is triggered even when there are no pending alerts. Verified this by removing the force alert.
https://github.com/lowRISC/opentitan/blob/605da270ddcf70b2af5573e9524f9613bc89383d/sw/device/tests/rv_core_ibex_nmi_irq_test.c#L203-L204
Disabling the ping timer removes the spurious NMI issue:
https://github.com/lowRISC/opentitan/blob/605da270ddcf70b2af5573e9524f9613bc89383d/sw/device/lib/testing/alert_handler_testutils.c#L139-L141
The following write in dif_alert_handler.c is what triggers the NMI from alert_handler:
  // Lock the configuration.
  if (locked == kDifToggleEnabled) {
    mmio_region_write32(alert_handler->base_addr,
                        ALERT_HANDLER_PING_TIMER_REGWEN_REG_OFFSET, 0);
  }

I tried switching the test to use a recoverable alert, different escalation sequences and a valid ping timeout timer configuration, but none of these changes helped to move the alert handler NMI.
I will submit these test updates separately, but I think at this point I am going to flag this for CDC analysis.
CC: @a-will @matutem @nbdd0121 who also took a look at this issue.

I am going to measure the delay between enabling the ping mechanism and the NMI to try to determine if this is an issue with the reverse ping mechanism supported by the receivers.
This was recommended by @msfschaffner.

Running the test with the alerts and ping configured, and without triggering the alert results in the first alert NMI triggering between 500-4000 microseconds.
This as measured with rv_timer using an equivalent 1us tick.

@andreaskurth, this test is reproducible without a ROM_EXT running. @a-will suggested we can try to get a DV test configuration ready to run in GLS in case this is something we want to try.
CC: @sha-ron @OTshimeon

By @moidx: We can run chip_sw_rv_core_ibex_nmi_irq with test ROM on the netlist. We need to update the testcase to not trigger a fake alert, just wait. Then we shouldn't see any NMIs and can wait for the timeout. We should run this GLS over the next weekend. @moidx will create a test case.

Created the test case in #23441 and added test point to the GLS test plan.

Discussed during triage meeting. Okay to move this to M5.

@moidx thanks for the heads up. IMHO this is OK for the testing we currently do, i.e. not fully invasive and cutting wires.

Moving this to M6 as P1. It should be tested early on the final netlist

https://github.com/lowRISC/opentitan/issues/24119 tracks the findings after running the sw_alert_handler_ping_ok test post synthesis.

@moidx We could add this test to the GLS test plan as a P2. It should now be passing now with the latest ECO fixes in.

Confirmed that chip_sw_rv_core_ibex_nmi_irq is on the GLS testplan. Suggest moving this to M7 (still as P1).

Closing this issue as the sw_alert_handler_ping_ok is now passing in GLS. We can create a new issue if rv_core_ibex_nmi_req_irq_test we are able to run the test in GLS and results in failure.


another mapDispatchToProps with bindActionCreators

Another version of mapDispatchToProps with bindActionCreators for a shorter form:
const mapDispatchToProps = (dispatch) => ({
  ...bindActionCreators(actions, dispatch),
});

I like this. :) I don't use bindActionCreators much as I like to sometimes change their names, but this would be useful for those who do. Thanks for the PR.


wow amazing, but seem to mess with my YAML header files, anyway to fix that?

Hi!
this is an amazing plugin! been doing it for months manually until i discovered this :D
one issue i see is that i use YAML headers (which i do ), it sees to mess up the file by adding the header on top of the YAML header (even thought there is a title below YAML), ie:

anyway this can be fixed since all my thousands of files have YAML headers :D
thx!
Z

This plugin only adds buttons to the page header, it never touches the editor or the files. Did you mean to write that bug report for another plugin?

oops yeah sorry about the noise :)


be able to delete folders and files

Delete folder is likely not going to implemented, but delete file is possible now.

Git doesn't have any concept of folders, deleting all files in a folder
will delete the folder.


Feature/webflux

What's this PR do/fix?
This PR provide a webflux support.
Are there unit tests? If not how should this be manually tested?
Curently existing unit tests are failed. I don't understand why. I need help on this part.
I have not already add unit test for webflux support.
Any background context you want to provide?
I have remove spring dependency to springfox-core. This can create some breaking change for thrid part libs.
What are the relevant issues?
see #1773

@deblockt Sorry its taken so long to get this PR, but Im getting ready to pull this in. Would it be possible to resolve the conflicts?

I can take a look as I want this to work.  It depends on spring 5; can that PR be merged in first.  There are several tests that fail so I may incrementally go through these changes.

FYI branch is https://github.com/ligasgr/springfox/tree/feature/webflux

@ligasgr that is awesome. You could create another PR that superscedes this one. FYI, One issue I had with this PR, that I would like to fix before pulling this in, is the duplication this introduces on the DocumentationBootstrapper. I think we should extract a chunk to spring-web and create a think shim for spring-web-mvc and spring-web-webflux, very similar to the thin shim we have for spring-data-rest

Thanks for your comments. I'll try to see how this could be achieved. Initial thoughts are:

Add some sort of factory for RelativePathProvider and have it injected intoDocumentationPluginsBootstrapper  that will inject it into DefaultConfiguration to avoid passing around ServletContext (it would only be autowired into that new factory). The code of DefaultConfiguration and DocumentationPluginsBootstrapper could then be moved back into common place (web) along with most of the tests that were moved to webmvc project (these that don't use mvc specific classes).
Merge SpringfoxWebMvcConfiguration and SpringfoxWebFluxConfiguration as they are exactly the same and move into web project.

How does that sound to you?

That does sound good. Essentially we want only one DocumentationPluginsBootstrapper and not one for the webflux request handlers and one for webmvc.


Use ProtoStream to encode sketches

What does this PR do?
This rewrites the sketch_series marhsaling to use ProtoStream (c.f. #9008).
In running the existing benchmarks, this is about 8% faster, but more importantly saves 50% on allocations / RAM.
name                          old time/op        new time/op        delta
MarshalSplitCompress1-16            1.97ms ± 0%        1.91ms ± 0%   -2.82%  (p=0.000 n=10+10)
MarshalSplitCompress10-16           18.1ms ± 0%        16.6ms ± 0%   -8.24%  (p=0.000 n=10+10)
MarshalSplitCompress100-16           176ms ± 1%         162ms ± 0%   -8.10%  (p=0.000 n=10+10)
MarshalSplitCompress1000-16          1.74s ± 0%         1.59s ± 0%   -8.37%  (p=0.000 n=9+9)
MarshalSplitCompress10000-16         17.4s ± 0%         15.9s ± 1%   -8.31%  (p=0.016 n=4+5)

name                          old payload-bytes  new payload-bytes  delta
MarshalSplitCompress1-16            5.15kB ± 0%        5.15kB ± 0%     ~     (all equal)
MarshalSplitCompress10-16           50.0kB ± 0%        50.0kB ± 0%     ~     (all equal)
MarshalSplitCompress100-16           498kB ± 0%         498kB ± 0%     ~     (all equal)
MarshalSplitCompress1000-16         4.98MB ± 0%        4.98MB ± 0%     ~     (all equal)
MarshalSplitCompress10000-16        49.8MB ± 0%        49.8MB ± 0%     ~     (all equal)

name                          old payloads       new payloads       delta
MarshalSplitCompress1-16              1.00 ± 0%          1.00 ± 0%     ~     (all equal)
MarshalSplitCompress10-16             1.00 ± 0%          1.00 ± 0%     ~     (all equal)
MarshalSplitCompress100-16            2.00 ± 0%          2.00 ± 0%     ~     (all equal)
MarshalSplitCompress1000-16           15.0 ± 0%          15.0 ± 0%     ~     (all equal)
MarshalSplitCompress10000-16           145 ± 0%           145 ± 0%     ~     (all equal)

name                          old alloc/op       new alloc/op       delta
MarshalSplitCompress1-16            1.41MB ± 0%        1.32MB ± 0%   -6.74%  (p=0.000 n=9+9)
MarshalSplitCompress10-16           6.93MB ± 0%        4.37MB ± 0%  -36.96%  (p=0.000 n=9+9)
MarshalSplitCompress100-16          52.8MB ± 0%        25.6MB ± 0%  -51.53%  (p=0.000 n=9+10)
MarshalSplitCompress1000-16          445MB ± 0%         172MB ± 0%  -61.48%  (p=0.000 n=10+8)
MarshalSplitCompress10000-16        4.37GB ± 0%        1.63GB ± 0%  -62.69%  (p=0.016 n=4+5)

name                          old allocs/op      new allocs/op      delta
MarshalSplitCompress1-16               857 ± 0%           469 ± 0%  -45.27%  (p=0.000 n=10+10)
MarshalSplitCompress10-16            8.21k ± 0%         4.15k ± 0%  -49.47%  (p=0.000 n=10+10)
MarshalSplitCompress100-16           81.7k ± 0%         40.9k ± 0%  -49.93%  (p=0.000 n=10+10)
MarshalSplitCompress1000-16           816k ± 0%          408k ± 0%  -49.97%  (p=0.000 n=10+8)
MarshalSplitCompress10000-16         8.16M ± 0%         4.08M ± 0%  -49.98%  (p=0.029 n=4+4)

Motivation
The existing code did a bit of this manual assembly of PB messages, but with
               // Magic number that occurs before the varint encoding

ProtoStream regularizes that a bit.
Describe how to test your changes

Configure an agent to produce large quantities of sketches (so that multiple payloads will be required)
Observe no crashes, see the sketches in the app

Checklist

[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] The appropriate team/.. label has been applied, if known.
[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.

This has been updated to use the ProtoStream functionality that just landed in Molecule.  This is a bit more efficient, so we see better CPU numbers than the earlier draft (5% improvement to 8.5% improvement).  The payloads are identical.  And the memory usage is substantially down.

Todo tasks
On Tue, Sep 14, 2021 at 3:50 PM Brian Floersch @.***>
wrote:

@.**** commented on this pull request.
In pkg/metrics/sketch_series.go
https://github.com/DataDog/datadog-agent/pull/9009#discussion_r708622326
:

const distributionSum = 6

const distributionV = 7
const distributionG = 8
const distributionDelta = 9
const distributionBuf = 10
*/
const dogsketchTs = 1
const dogsketchCnt = 2
const dogsketchMin = 3
const dogsketchMax = 4
const dogsketchAvg = 5
const dogsketchSum = 6
const dogsketchK = 7
const dogsketchN = 8

// generate a footer containing an empty Metadata field (TODO: this isn't

Is this TODO addressable now? or is this a future task
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DataDog/datadog-agent/pull/9009#pullrequestreview-754412016,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATTQVRFQ5WNRUPZSIZUY7Z3UB6YR7ANCNFSM5C56TWYQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I think encodeVarintAgentPayload can be removed now

Those are both good points, and I hope to revisit them when looking more closely at how the compression part of serialization is implemented.

In the benchmark environment, handling 160,000 distinct sketches, serialization took about 2.43s per 15s.  The vast majority of that was compression

Ignoring compression leaves about 250ms per 15s:

80ms of that is in the call to .Cols(), which would have to be done anyway.  So overall the actual PB serialization is trivial here.  A 5% improvement is likely to work out to 10ms per 15s only in this highly-loaded environment, so I doubt we'll see any noticeable change in CPU usage when this PR is deployed.


Update Calico to v2.4.1

network plugin: Update Calico to v2.4.1
One thing that needs to be mentioned is that this update includes a change in the default deny behavior for network policy, Calico has switched this behavior to match the move of Kubernetes NetworkPolicy to v1.  You can see the release notes for Calico at https://github.com/projectcalico/calico/releases/tag/v2.4.0 (see the changes under k8s-policy).
I am looking for guidance on how the kube-aws project would like to handle this behavior change or at least know that it is expected that users should understand the changes of the components they are using.
Just to point out how this new behavior works:

DefaultDeny is enabled for a pod if there is any policy that selects the pod

That means the new way to enable DefaultDeny is create a policy that selects all pods (in a namespace)
This also means that if DefaultDeny is not desired then all policy needs to be removed that targets a pod.

The old default deny annotation has no effect with the new Calico version

Codecov Report

Merging #832 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #832   +/-   ##
=======================================
  Coverage   34.38%   34.38%
=======================================
  Files          57       57
  Lines        3743     3743
=======================================
  Hits         1287     1287
  Misses       2311     2311
  Partials      145      145

Impacted Files
Coverage Δ

core/controlplane/config/config.go
59.97% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@redbaron @camilb @danielfm @c-knowles Are you using calico on your clusters? Anyways, FYI, there are some manual steps required to maintain your network policy behave the same as before while upgrading k8s to v1.7.
The new calico v2.4.1 introduced by this PR seems to correctly implement the semantics of networking/v1 api and therefore the change in k8s v1.7 becomes effective after merging this PR.

@tmjd Thank you very much for the contribution with the detailed description 👍

it is expected that users should understand the changes of the components they are using.

Yes, they should. I'm going to add a note about the changes in the release note for the next kube-aws release 👍

@mumoshu, thanks. Not using it here.

+1, I enabled it, but I'm not using network policies so far.

Same for me, enabled but with no policy configured yet.


[Bug] Unable to set Notepads as default text (.text) files editor - does not show as option

Describe the bug
Notepads does not show as an option for opening text files as the default editor at Settings > Apps > Default apps > Choose default applications by file type > .text.
To Reproduce

Install Notepads from the Microsoft Store.
Go to Settings > Apps > Default apps > Choose default applications by file type and scroll down until you find the .text file type.
Click the Default app box/icon. Notepads does not appear as an option.

Expected behavior
Notepads should be selectable as a default text editor.
Screenshots

Notepads does not show up as an option. As an example of the reverse, Notepads shows up as the default editor for the .t file type.
Desktop (please complete the following information):

OS: Windows 10 Pro 19041 10.0.19041
Version <IP_ADDRESS>

.text is not supported as of now. Only .txt is supported.


VA scores that fall on upper boundary are being set to NA

Describe the bug
If a user uploads data for a student and the prior attainment of that student falls on the upper limit of the prior attainment model the app will not calculate the VA of that student, but also does not flag the student as removed
To Reproduce
NA
Expected behaviour
The student is on the boundary, rather than above it, so the app should include the student and calculate their VA.

The pupil_pava_bands_flags section of the code that determines between which two pava bands a pupils prior attainment falls between has now been updated.
It is split into two parts to account for students who fall on the upper boundary of the maximum pava band and those who don't.


got an error when save query as a virtual dataset

A clear and concise description of what the bug is.
2022-04-08 08:00:45,691:ERROR:superset.views.base:Can't generate DDL for NullType(); did you forget to specify a type on this Column? Traceback (most recent call last):   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1516, in full_dispatch_request     rv = self.dispatch_request()   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1502, in dispatch_request     return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)   File "/usr/local/lib/python3.8/site-packages/flask_appbuilder/security/decorators.py", line 133, in wraps     return f(self, *args, **kwargs)   File "/app/superset/utils/log.py", line 245, in wrapper     value = f(*args, **kwargs)   File "/app/superset/views/core.py", line 2121, in sqllab_viz     db.session.commit()   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/scoping.py", line 163, in do     return getattr(self.registry(), name)(*args, **kwargs)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/session.py", line 1046, in commit     self.transaction.commit()   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/session.py", line 504, in commit     self._prepare_impl()   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/session.py", line 483, in _prepare_impl     self.session.flush()   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/session.py", line 2540, in flush     self._flush(objects)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/session.py", line 2682, in _flush     transaction.rollback(_capture_exception=True)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/util/langhelpers.py", line 68, in __exit__     compat.raise_(   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/util/compat.py", line 182, in raise_     raise exception   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/session.py", line 2642, in _flush     flush_context.execute()   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/unitofwork.py", line 422, in execute     rec.execute(self)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/unitofwork.py", line 586, in execute     persistence.save_obj(   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/persistence.py", line 248, in save_obj     _finalize_insert_update_commands(   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/persistence.py", line 1432, in _finalize_insert_update_commands     mapper.dispatch.after_insert(mapper, connection, state)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/event/attr.py", line 322, in __call__     fn(*args, **kw)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/orm/events.py", line 731, in wrap     fn(*arg, **kw)   File "/app/superset/connectors/sqla/models.py", line 2021, in after_insert     SqlaTable.write_shadow_dataset(target, database, session)   File "/app/superset/connectors/sqla/models.py", line 2375, in write_shadow_dataset     tables = load_or_create_tables(   File "/app/superset/connectors/sqla/utils.py", line 229, in load_or_create_tables     columns = [   File "/app/superset/connectors/sqla/utils.py", line 232, in <listcomp>     type=str(column["type"]),   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/type_api.py", line 623, in __str__     return str(self.compile())   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/type_api.py", line 606, in compile     return dialect.type_compiler.process(self)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 402, in process     return type_._compiler_dispatch(self, **kw)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/visitors.py", line 96, in _compiler_dispatch     return meth(self, **kw)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 3478, in visit_null     raise exc.CompileError( sqlalchemy.exc.CompileError: Can't generate DDL for NullType(); did you forget to specify a type on this Column?

@xiamicpp I have the same problem，Have you solved it yet！

@xiamicpp @mjcc007 thanks for reporting this. Can you specify which version of Superset (or SHA if you're running master) you're using?


Create an up-to-date diagram of sandbox components

We should create some nice diagrams to illustrate all of the Sandbox components and how they communicate with each other, under various scenarios (deployments / private transactions / public transactions / calls to view functions (public & private) / querying private data / querying public blockchain data / etc.)

The final diagram is available here.

I bought a miro subscription to get the diagrams out in a usable format. Here they are:


Fixed depreciation warnings

What does this PR do?
Fixed a depreciation error that I was getting in issue #193 and #202
Where should the reviewer start?
Just the first import sklearn
How should this PR be tested?
Run it and see if you get any depreciation errors still
Any background context you want to provide?
What are the relevant issues?
issue #193

Do the docs need to be updated?

Nah

Does this PR add new (Python) dependencies?

Nah

TL;DR I don't know how to code


Release 2024-02-12 - Removing PanningLink from default preset

Also to include reducing master volume for presets from 1.5 to 1.0, and expanding max musicians for default preset from 10 to 30

sounds good to me!


Using pyfolio independently

I ran into the issue below just trying to analyze returns. While I saw that there is another issue talking about working around the data issues, I wonder if you can use pyfolio as a stand alone package from zipline(and from data providers)?
If so, are there examples of using pyfolio to analyze returns/transactions/positions not from zipline?
import numpy as np
import pandas as pd
import matplotlib.pyplot as plt
import pyfolio as pf
names = ['foo','bar']
dates = pd.date_range(start='2017-01-01',end='2017-12-31', freq=pd.tseries.offsets.BDay())
n = len(dates)
rdf = pd.DataFrame(
    np.zeros((n, len(names))),
    index = dates,
    columns = names
)

np.random.seed(1)
rdf['foo'] = np.random.normal(loc = 0.1/n,scale=0.2/np.sqrt(n),size=n)
rdf['bar'] = np.random.normal(loc = 0.04/n,scale=0.05/np.sqrt(n),size=n)

pdf = 100*np.cumprod(1+rdf)
pdf.plot()
pdf.head()

pf.create_returns_tear_sheet(rdf['foo'])

Trying the code above throws the error below
RemoteDataError: Unable to read URL: https://finance.google.com/finance/historical?q=SPY&startdate=Jan+01%2C+1970&enddate=Apr+17%2C+2018&output=csv
Response Text:
b'Sorry...GoogleSorry...We're sorry...... but your computer or network may be sending automated queries. To protect our users, we can't process your request right now.See Google Help for more information.Google Home'

@JordanPlatts Have you had any success in getting around this issue?

To be honest, I haven't tried again.
I posted this because I was hoping someone would point me to an example of using pyfolio as a stand-alone package.

I think this is a dup of https://github.com/quantopian/pyfolio/issues/525, which we urgently need to fix.


RPC: endpoints for fetching checkpoints from Fullnode

@mlogan , @lxfind mentioned that you are still working on syncing the checkpoints on Fullnode. Do you have a rough ETA on the readiness?


Change the start_level of DAO as per Serokell Team's suggestion

Name
The start_level of the DAO should be around some level around which this contract becomes live. Currently, it's hardcoded to 100 for some reason, this doesn't affect the logic of the contract but is recommended to be set somewhere around the level DAO is live.
More Info - https://dorghq.slack.com/archives/C02KY15P5GE/p1669114227125199?thread_ts=1659726931.733339&cid=C02KY15P5GE
Description (what needs to be done) / Expected Behavior
Acceptance Criteria
Supporting information (Figma link, notes, etc)
When finished, please add Link to the PR or completed Figma / design asset, as a comment to the ticket.

Testing Scenario -

Create a new DAO.
It's cycle should start from 0 or 1, unlike previous DAOs which had 5-6 figure cycles.


stream mode always with \nObservation in Action Input

Action: DbTablesWithRelevanceScores
Action Input: query five records Observation
Observation: Table: aiops_metric_alarm, relevance score: 0.4535
I need to look at the schema of the relevant table.
Action: DbRelevantTablesSchema
Action Input: aiops_metric_alarm Observation
Observation: Tables not found in the database
Final Answer: The table 'aiops_metric_alarm' is not found in the database, so no records can be retrieved.
this is the table_names_list ['aiops_metric_alarm\nObservation']

@zhanpengjie We have recently updated the streaming endpoint, please pull the latest changes and let us know if there are issues with the format. Thank you


On dynamic slides must align slides with width if the number of slides is less then slidesperview ?

I display slides depends on the data from backend. If i have only 2 slides but if the slidesperview is 4 it will show 2 slides and space for another 2 slides but I want the 2 slides to take the whole width

You should use params rebuildOnUpdate and updated your params each time when you receive data from server.

Do you have an example? I can`t get it done


Pokeymanz Sheet - Minor bugfix

In the last update, we added a bonus value that could be set inside the sheet for the creature rolls, but ended mistakenly excluding the query for bonus value that was also useful for quick setting and variables. This quick update solves the problem.

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: Pokeymanz  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: Chronicle Delusionist (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: Pokeymanz (i.e. Dungeons & Dragons, FATE)

[X] I have followed the Character Sheets Standards when building this sheet.

[X] This sheet has been made on behalf of, or by, the game's publisher.
[X] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: Chronicle Delusionist
[X] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

In the last update, we added a bonus value that could be set inside the sheet for the creature rolls, but ended mistakenly excluding the query for bonus value that was also useful for quick setting and variables. This quick update solves the problem.

Character Sheet Info Roll20 Internal Use only.


Does Sisi support videos?

Unfortunately no, it only supports pictures and there is no plan for videos.