#87 youtube video quality assure video download at highest possible quality

Description
Updated the code so that it forces youtube-dl to download contents to the extent of available quality.
Fixes #87
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

[x] Test A
Followed Contributing to Udacimak guideline, tested the application on my local workspace and the application ran as intended and all functionality stayed the same.
Also tested for all the possible resolutions to download and ended with the successful attempt that the videos are now downloaded at the Top most quality available, then if the best quality not available it automatically tries for the second best and so on until the file is downloaded.
Top most quality -- in my case 720p

Point needed to be mentioned :

The chart below describes all the possible resolutions and quality which is available to download.
For some of the video it might happen that some resolutions may not be available and size mentioned varies video to video.

format code  extension  resolution note
249          webm       audio only tiny   51k , opus @ 50k (48000Hz), 233.69KiB
250          webm       audio only tiny   68k , opus @ 70k (48000Hz), 303.34KiB
140          m4a        audio only tiny  130k , m4a_dash container, mp4a.40.2@128k (44100Hz), 592.95KiB
251          webm       audio only tiny  133k , opus @160k (48000Hz), 586.06KiB
160          mp4        256x144    144p   63k , avc1.4d400c, 24fps, video only, 257.29KiB
278          webm       256x144    144p   84k , webm container, vp9, 24fps, video only, 373.73KiB
242          webm       426x240    240p  154k , vp9, 24fps, video only, 550.37KiB
133          mp4        426x240    240p  159k , avc1.4d4015, 24fps, video only, 640.21KiB
243          webm       640x360    360p  298k , vp9, 24fps, video only, 1.06MiB
134          mp4        640x360    360p  329k , avc1.4d401e, 24fps, video only, 1.27MiB
244          webm       854x480    480p  502k , vp9, 24fps, video only, 1.80MiB
135          mp4        854x480    480p  669k , avc1.4d401e, 24fps, video only, 2.55MiB
247          webm       1280x720   720p  885k , vp9, 24fps, video only, 3.36MiB
136          mp4        1280x720   720p 1368k , avc1.4d401f, 24fps, video only, 5.37MiB
248          webm       1920x1080  1080p 1524k , vp9, 24fps, video only, 5.56MiB
137          mp4        1920x1080  1080p 2724k , avc1.640028, 24fps, video only, 10.37MiB
18           mp4        640x360    360p  455k , avc1.42001E, mp4a.40.2@ 96k (44100Hz), 2.03MiB
22           mp4        1280x720   720p 1334k , avc1.64001F, mp4a.40.2@192k (44100Hz) (best)

As you can see the top most quality available is
👉  137   mp4   1920x1080  1080p 2724k , avc1.640028, 24fps, video only, 10.37MiB
Top most video quality, But its only video [audio is not included]
👉 140    m4a    audio only tiny  130k , m4a_dash container, mp4a.40.2@128k (44100Hz), 592.95KiB
Top most audio quality, But its only audio [video is not included]
But Udacimak can download only the files which have both audio and video included, as of now.
👉 22      mp4    1280x720   720p 1334k , avc1.64001F, mp4a.40.2@192k (44100Hz) (best)
To download the best of audio and video it needs to download each file separately and merge them into one to make them work for which youtube-dl requires some of the dependencies (requires ffmpeg or avconv installed) to do the same.

Test Results
Before Modification

View Sample Video (Before Modification)

After Modification

View Sample Video  (After Modification)

Configurations

OS: [Windows 10 Home]
Node version: (v12.16.1)
npm version: (6.13.4)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

@udacimak Thanks
Keep it up, it's an amazing project !! 👍
Hope for new feature additions. 💯

format code  extension  resolution note
249          webm       audio only tiny   51k , opus @ 50k (48000Hz), 233.69KiB
250          webm       audio only tiny   68k , opus @ 70k (48000Hz), 303.34KiB
140          m4a        audio only tiny  130k , m4a_dash container, mp4a.40.2@128k (44100Hz), 592.95KiB
251          webm       audio only tiny  133k , opus @160k (48000Hz), 586.06KiB
160          mp4        256x144    144p   63k , avc1.4d400c, 24fps, video only, 257.29KiB
278          webm       256x144    144p   84k , webm container, vp9, 24fps, video only, 373.73KiB
242          webm       426x240    240p  154k , vp9, 24fps, video only, 550.37KiB
133          mp4        426x240    240p  159k , avc1.4d4015, 24fps, video only, 640.21KiB
243          webm       640x360    360p  298k , vp9, 24fps, video only, 1.06MiB
134          mp4        640x360    360p  329k , avc1.4d401e, 24fps, video only, 1.27MiB
244          webm       854x480    480p  502k , vp9, 24fps, video only, 1.80MiB
135          mp4        854x480    480p  669k , avc1.4d401e, 24fps, video only, 2.55MiB
247          webm       1280x720   720p  885k , vp9, 24fps, video only, 3.36MiB
136          mp4        1280x720   720p 1368k , avc1.4d401f, 24fps, video only, 5.37MiB
248          webm       1920x1080  1080p 1524k , vp9, 24fps, video only, 5.56MiB
137          mp4        1920x1080  1080p 2724k , avc1.640028, 24fps, video only, 10.37MiB
18           mp4        640x360    360p  455k , avc1.42001E, mp4a.40.2@ 96k (44100Hz), 2.03MiB
22           mp4        1280x720   720p 1334k , avc1.64001F, mp4a.40.2@192k (44100Hz) (best)

👉  137   mp4   1920x1080  1080p 2724k , avc1.640028, 24fps, video only, 10.37MiB
Top most video quality, But its only video [audio is not included]
👉 140    m4a    audio only tiny  130k , m4a_dash container, mp4a.40.2@128k (44100Hz), 592.95KiB
Top most audio quality, But its only audio [video is not included]
* But Udacimak can download only the files which have both audio and video included, as of now.
👉 22      mp4    1280x720   720p 1334k , avc1.64001F, mp4a.40.2@192k (44100Hz) (best)

To download the best of audio and video it needs to download each file separately and merge them into one to make them work for which youtube-dl requires some of the dependencies (requires ffmpeg or avconv installed) to do the same.

@udacimak
Hey ,
As i mentioned above ☝  about the requirements for downloading the videos as its best possible quality. Adding the dependencies, I know its bit lengthy process, but i wanted to mention that i tried changing the format to '137+140' without adding the dependencies, i configured that Udacimak is unable to download two files for the same 'Youtube-Id' like audio and video files separately, so just wanted to ask you how can we patch it to download both the files no matter the dependencies is satisfied or not. I mean to say that if the user has the dependencies already installed globally it'll download '137+140' both the file and merge it but if the user doesn't satisfy the dependencies they will only be able to download at '22' quality. In this way the users can have two option for '22' format and '137+140' format who wishes to download the best will have to satisfy the dependencies themselves and proceed further.
Please try to go through the process i hope it'll be a big jump for Udacimak.


Fixed major problems caused by new Among Us features and UI updates.

Hi, I was able to fix the hotkey issue and mitigated the left and right movement drift.
I also fixed an issue when the player was not able to press buttons while moving.
Added support for new roles with the f button press.
Vent now works with the v button.
I think with the current changes it is playable for non public / voice chat based lobbies.
Could you please merge this and create a release for it? I have no prior experience with Swift, so I do not know how I could create a release.
Still open issues:

full screen does not work with joypad.
meeting is often recognised as ingame.
meeting return button is misplaced (this issue is irrelevant until the previous is resolved).
the close button does not have a fixed place.

This MR solves #26 .

Thank you @N0rbi @


Support multiple triggers on flops

Is your feature request related to a problem? Please describe.
Support multiple triggers for flops, like in SystemVerilog:
always_ff @(posedge clk or posedge reset)

Describe the solution you'd like
The FF and FlipFlop should have a syntax that accepts a list of signals instead of a single clk signal.
Describe alternatives you've considered
Using something like clk | reset is insufficient because there could be a rising edge of reset during a clock high period.

@mkorbel1 Is it possible to support negative edge trigger as well?
It seems like the rohd only support the positive edge single trigger with current version.
https://github.com/intel/rohd/blob/eb4fd660e9f4bb646c76f1bb7b900a445cc9ec79/lib/src/modules/conditional.dart#L222

@wswongat it wouldn't be too hard to add that, but I figured it wasn't really needed as a separate feature since you could always just use the posedge of an inverted signal, for example:
FF(clk, [...]); //posedge of clk
FF(~clk, [...]); // negedge of clk

Do you think that meets your needs or can you elaborate why you think a separate negedge trigger would be beneficial beyond this?

@mkorbel1 Your suggestion looks good. Looks like we need to find some way to support multiple triggers on flops

@wswongat #65 should address this issue and provide support for multiple triggers on flops


This converts the full_name input in the /account-settings/ section of
the settings panel to be autosaved whenever the input is blurred or the
save button is clicked.

Automated message from Dropbox CLA bot
@brockwhittaker, it looks like you've already signed the Dropbox CLA. Thanks!

I think before we can merge something like this, we need to reorganize that page a bit.  I think it'll be pretty confusing having a mix of auto-saved things and the rest.

Should this be merged? I think it is an improvement, and will make thinking about the rest of the page easier.
Also, once this is merged we can remove the "Save changes" button, and have that appear only for the password change.

We ended up making this area of the page nice another way.  Closing.


New files will be downloaded?

Since tumblr introduced the new naming conventions ThumblTree misses lot of files of new posts.
Older post are still working.
When TumblrThree will be fixed to handle new naming, it will be download now skipped files?
Last file remains opened by TumblrTree therefore I can't delete it unless I close TumblrTree.

Is TumblThree still maintained? I have unfortunately not seen updates for a long time,


Update cosigned admission controller documentation

Description
Update cosigned admission controller documentation to include ClusterImagePolicy. This should include discrete key and keyless examples.

Picked up by @elfotografo007

@sigstore/codeowners-sigstore-website We could close this issue. We have recently updated the documentation.


dot/Comma followed from a math block get a newline

As stated
$$.

will be formatted to
$$
.

and do it again will become
$$

.

thanks for the report!

I can't seem to reproduce this on master, have you only tried via typst-lsp?

Does typstlsp produce the issue? I remember I also try the release and reproduce the issue. I can check it again

If you tried the release then I'm surprised but things move so fast it's not impossible ;)
Anyway, I definitely fixed it now by reworking that part of the code and integrating it in my examples ;)
I'm closing this but feel free to test by yourself and confirm !

It seems to work now. Thank you!

We seems to have a different issue that the space before a math block is being removed.


DeepL is omitting parts of the text

First I thought that the translation via DeepL sometimes stops at line breaks. But it turned out to be more complicated.
Example:
Input:
trans -b -e deepl :de 'No heap allocation is involved in initializing this array of empty strings (C++ is not Java!). Local UnicodeString arrays like this are a very good fit for use with split(); after extracting the fields, any values that need to be kept in some more permanent way can be copied to their ultimate destination.'

You’ll notice that the first sentence is missing.
The same thing happens with Spanish as target language. However, the translation is complete with French as target language! Removing the -b switch doesn’t change anything in respect.
The DeepL webform delivers the complete translation for all languages.

Now back to the text.
If I add a sentence at the beginning of the text, then everything gets translated:
Input:
trans -b -e deepl :de 'Note that, in this example, “words” is a local, or stack array of actual UnicodeString objects. No heap allocation is involved in initializing this array of empty strings (C++ is not Java!). Local UnicodeString arrays like this are a very good fit for use with split(); after extracting the fields, any values that need to be kept in some more permanent way can be copied to their ultimate destination.'

Result:
Beachten Sie, dass in diesem Beispiel "words" ein lokales oder Stack-Array von tatsächlichen UnicodeString-Objekten ist, bei dessen Initialisierung keine Heap-Allokation erforderlich ist (C++ ist nicht Java!)

Or: If I cut the last sentence right after the “split();” then the translation is complete also:
Input:
trans -b -e deepl :de 'No heap allocation is involved in initializing this array of empty strings (C++ is not Java!). Local UnicodeString arrays like this are a very good fit for use with split().'

Result:
Bei der Initialisierung dieses Arrays von leeren Strings (C++ ist nicht Java!) ist keine Heap-Allokation erforderlich; solche lokalen UnicodeString-Arrays sind sehr gut für die Verwendung mit split() geeignet.

Any idea why this happens?
Note that the omissions…

do not happen with the web form
seem to be content and language dependent, but I was unable to find a pattern

I’m using Translate Shell <IP_ADDRESS>-git:865ac72. I’m on macOS 10.13.6, the user locale is set to en_US.UTF-8. Bash is 4.4.23, gawk is 4.2.1.

It turns out that sometimes the API requires texts to be split into sentences first. Should be fixed now.

OK, my sample text from above seems to work fine now.
However while throwing arbitrary text snippets at it I found another —probably related— issue:
Take a snippet from the issue text above:
trans -b -e deepl :de 'You’ll notice that the first sentence is missing.

The same thing happens with Spanish as target language. However, the translation is complete with French as target language! Removing the -b switch doesn’t change anything in respect.
The DeepL webform delivers the complete translation for all languages.'

Please note the two newlines after sentence 1 and the single newline before the last sentence. We could also say the text consists of three paragraphs.
Result:
Sie werden feststellen, dass der erste Satz fehlt. Die Übersetzung ist jedoch komplett mit Französisch als Zielsprache! Das Entfernen des -b-Schalters ändert nichts daran.

The last sentence (or paragraph) is missing.

In order to get the last sentence too, I have to remove the single newline before the last sentence (i.e., joining paragraph 2 and 3):
trans -b -e deepl :de 'You’ll notice that the first sentence is missing.

The same thing happens with Spanish as target language. However, the translation is complete with French as target language! Removing the -b switch doesn’t change anything in respect. The DeepL webform delivers the complete translation for all languages.'

Removing one or two of the newlines after sentence 1 doesn’t change anything. Adding a line feed (\n) at the end doesn’t help either.

Please let me know if I rather should open a new ticket for that.

The API doesn't seem to be very happy with requests containing newlines \n. Workaround pushed.

Only made a few tests, but seems great now with a03100a!
Besides that, now line breaks are correctly reflected in the translation. (Before, most of the times, all lines have been merged.)
Thanks for your great script and for all your work!
– Tom


Input: Local File

Description

Able to select a Request Body as a Local File
UI for selecting a file
Auto detect the Content-Type and override it

Done: https://download.proxyman.io/beta/GetAPI_1.0.9_Support_File_as_Body.dmg
Screenshots


⚠ ClamAV has degraded performance

In 5f9ac45, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in 277fcd3 after 5 minutes.


STP-3179: Crayport instances were replaced to My HPE Software Center

Description
Crayport instances were replaced with My HPE Software Center
Updated the content based on the JIRA ticket 3179
Checklist Before Merging

[ ] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[ ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[ ] My commits or Pull-Request Title contain my JIRA information, or I don't have a JIRA.

/backport release/1.0 main


corruption_test failed: db/corruption_test.cc:325: failed: 1 == 5

What steps will reproduce the problem?
1. make corruption_test
2. for((i=1;;i++)); do
       ./corruption_test
       if [[ $? -ne 0 ]]; then
           break;
       fi
   done
3.

What is the expected output? What do you see instead?

 On what operating system?
1.4, git commit "85584d497e7b354853b72f450683d59fcf6b9c5c"
OS: CentOS 5.4, kernel <IP_ADDRESS>, x64 virtual machine(VMWare)

Please provide any additional information below.

Original issue reported on code.google.com by tfengjun on 7 May 2012 at 6:21

[deleted comment]

I was not able to reproduce the bug in a real linux box, but it happened on 3
vm instances runing CentOS 5.4, with different kernels including: 2.6.18-238,
2.6.18-164, <IP_ADDRESS> and <IP_ADDRESS>.

Maybe it's a vm issue??

Original comment by tfengjun on 8 May 2012 at 2:07

You need to run the for loop for serveral minutes before you see the failed
result.
If you're lucky enough, you can see it happens very soon(it once failed in the
second run).

Original comment by tfengjun on 7 May 2012 at 6:41

der.pitti, I don't see the CorruptionTest.Recovery issue at all in the links
you gave, just an internal compiler error in the first one. Am I missing
something?

Original comment by<EMAIL_ADDRESS>on 5 Jun 2013 at 4:51

Apparently this is due to a race in test:

https://code.google.com/p/leveldb/issues/detail?id=182#c3

Original comment by<EMAIL_ADDRESS>on 28 Jun 2013 at 4:04

In Ubuntu this happens on real hardware without VMs, but only on ARM:
https://launchpad.net/ubuntu/+source/leveldb/1.10.0-1/+build/4616412
https://launchpad.net/ubuntu/+source/leveldb/1.10.0-1git1/+build/4640626

What does the "leveldb.num-files-at-level0" property mean, anyway? Is that test
maybe too strict, and some ">= 1" might be more appropriate?

Original comment by<EMAIL_ADDRESS>on 4 Jun 2013 at 3:32

Note, I re-tried the second build
(https://launchpad.net/ubuntu/+source/leveldb/1.10.0-1git1/+build/4640626) and
that worked the second time, so this confirms the race condition from the
initial description, and that it is not related to a VM.

Original comment by<EMAIL_ADDRESS>on 4 Jun 2013 at 3:52

Indeed, I'm sorry. It seems that only happened in the 1git1 build, and as I
retried the build until that "1 == 5" failure went away. It looked exactly like
the one in the description.

Original comment by<EMAIL_ADDRESS>on 6 Jun 2013 at 8:05

I have been able to reproduce this problem and I am working on a fix.  Thanks
for the reports.

Original comment by<EMAIL_ADDRESS>on 1 Jul 2013 at 9:31

Fixed in 1.13

Original comment by<EMAIL_ADDRESS>on 10 Dec 2013 at 7:51

Changed state: Fixed


feat: api support for geojson geometries

This extends the existing API endpoints to also support sending a Feature or a Geometry object in the request body. Should maintain 100% backwards compatibility with the added function that normalizes the request body into a Feature.

Moves the ApiLocation / ApiGeofence structs into more appropriate places (why are these even separate structs?)
Converts all pokestop related SQL to use geojson functions instead

It have had it in my mind to move to https://github.com/tidwall/geojson from the orb geojson that we use now. This is much more widely use and has built in fast quadtree/rtree and other high quality geojson utility functions.  Not sure whether it is helpful to you at this stage, but if you find it is easier to use capability here it will save rewriting your changes later.

This has been running in production without issues for several days now. I tried the other library but unmarshalling seems to be a bit more complex to distinguish between a geometry input and a feature input...


feat: add pay per result docs

Basic documentation on what pay per result actors are. Note this is only first version. I think the good approach here is to get out something basic as soon as possible and iterate later.
For better reading (images cannot be easily rendered locally)

Keep in mind this repo still has old content, I will move it on monday when I am back from vacation (pr is here already). So you should probably move this to the apify-docs-v2 repo.

Let me say it one more time, this should be targetting the apify-docs-v2 repo or wait before #517 gets merged.

@B4nan I don't need to deploy it until next week, so it should be fine. I looked at v2 repo as well, but I wasn't sure where it would be the best to put this content and how it exactly interacts with the old content in this repo. Let's discuss next week when you are back, it's not urgent. Thx

Closing in favor of #532


Create Login Screen

As a User of Andy X Portal, to secure that not everyone can access the portal, I would like to have a user and a password to access the protal.

Implemented


Add stargate whitelist for pool manager params

Closes: #XXX
What is the purpose of the change
Adds pool manager query params to stargate whitelist.
This is essential for contracts to be able to query pool creation fee.
Params from other modules are already whitelisted so adding this should not be problematic
Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[ ] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[ ] Code comments?
[ ] N/A

devbot help

Hi! I'm DevBot, a bot that helps with common tasks in the development process. Commands:

devbot fix: Fix basic errors in the PR. (e.g. imports, changelog conflicts, go mod conflicts)
devbot merge base: Merge the base branch into the PR branch.
devbot add changelog [misc/feat/bug/api/sec] [message]: Add a changelog entry to the PR. (e.g. devbot add changelog feat Added a new feature)

If message is blank, defaults to PR title.

devbot re-pr: Re-PR the PR, useful for external contributors where we have no edit perms.

devbot add changelog misc Add PoolManager Params query to Stargate Whitelist


Bulk api 2.x

Description
To allow users to control replication on multiple indices without having to list down every index, we will allow users to select indices based on wildcard patterns, this will allow users to quickly stop/pause/resume replication for multiple indices. This method will also allow users to use * as pattern to select all indices.
Issues Resolved
https://github.com/opensearch-project/cross-cluster-replication/issues/812
https://github.com/opensearch-project/cross-cluster-replication/issues/765
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Marking as closed


scripts: install-rkt.sh: fail install-pak on errors

When install-pak (called from install-rkt.sh) fails at some point
abort packaging.
This is based on a discussion in
https://github.com/coreos/rkt/pull/3127#discussion-diff-76763179 where
it was agreed upon that aborting on unexpected failures would be
desirable.

ok to test

LGTM


duplicate check for role in ConfigModelV7#TenantHolder#mapTenants

the check looks for kibana_user twice:
https://github.com/opensearch-project/security/blob/6b7a5869c84d453bb22be861a88b744512fa0d64/src/main/java/org/opensearch/security/securityconf/ConfigModelV7.java#L1130-L1131
this is obviously not needed. the question is what it should be in reality, if there should be something else or if the duplicate line can just be removed.
furthermore the case-sensitivity seems spotty as it explicitly checks the lower- & upper-case version of all_access but not of the others - is this as intended or should the check be done properly case-insensitive for all?

@Hailong-am / @davidlago: since #2183 has been merged i guess this is fixed? can you please double-check and close the issue (and mention the versions where it has been fixed)? thanks!

The fix is included in release 2.4


refactor(http-client-csharp): not use DefaultValue to pass literal value of constant parameters

resolve #4028

API change check
API changes are not detected in this pull request.


Trailing Comma - Snowflake

Curious what the thoughts are on supporting a trailing comma in SQL? Snowflake just announced support for this in their dialect.
Example:
SELECT emp_id,
       name,
       dept,
FROM employees;

This doesn't appear to be handled by tree-sitter-sql yet(as it was just announced) but curious if this is something tree-sitter-sql wants to support? Or would it be better to make a dialect-specific repo?

I'm up for adding it! You interested in writing the PR?

yeah, i'll give it a go. do you want it behind a flag or anything?

Adding it to the existing behaviour is good

Not sure if I am a fan of adding the trailing comma. This is a convenience addition by snowflake and - as of now - is only supported by snowflake. It saved me more than once to see an error in the highlighting, when having a trailing comma that my query engines (postgres/impala/athena) do not accept. Since this parser does not distinguish between dialects it would not through an error.
In this particular case it would not be wise to be permissive.


Apply sorting to the entire table

According to this message sorting and filtering in vscode extension is only applied to the experiments under commit.
I'm using the extension as a leaderboard and it would be pretty convenient to be able to sort/filter the entire table.

Filters were applied to commits in https://github.com/iterative/vscode-dvc/pull/4363, though sorting is currently only applied to experiments under commits.

I'm using the extension as a leaderboard and it would be pretty convenient to be able to sort/filter the entire table.

@Danila89, do custom plots work for your use case instead of table sorting? You can use them to compare a metric/param across all commits and experiments:
https://github.com/iterative/vscode-dvc/assets/43496356/ee65e4ac-292f-48f3-9328-87bca4679180

These plots are really helpful, but it is essential for me to use some filtering of the experiments before plotting them. I didn't find the way to filter experiments before plotting them in the vscode extension.
Plots are nice in addition to sorting, not instead of it)

These plots are really helpful, but it is essential for me to use some filtering of the experiments before plotting them. I didn't find the way to filter experiments before plotting them in the vscode extension.

Filtering experiments in custom plots would be simple enough to implement! Would that enhancement work for your use case or would table sorting still be needed?

It is OK like a workaround (and a very useful feature by the way), but table sorting is also nice to have

Did some research for table sorting and tried a couple ideas.
1. Flattening then sorting the table
By flattening the table and turning off nested rows entirely, we could sort all rows from biggest to smallest and vice versa. While I don't think this would be too hard to implement on the backend, we'd need to rethink how our table would look while flattened (how do we tell what row belongs to what commit/branch). This would require some design work and adjusting our table components to handle the possibility.
2. Sorting commits along with experiments
Commits would be sorted along with the experiments based on their largest (smallest could work too) value among its baseline values and its subrows' values. While this does make sorting more complicated (and probably performant since we need to parse all the table values), it is doable. I was able to get a working demo made pretty quickly. The demo is using the commit's and its subrows' max value to sort:
https://github.com/iterative/vscode-dvc/assets/43496356/a13570ad-63f9-4d84-8ff1-9ec4efc6a205
3. Sorting commits and branches along with experiments
Branches would be sorted similarly to how commits are sorted in idea 2 aka max value among itself and subrow's values. I also think it's doable as well but would be a bit more complicated/performant than method 2 since now we're sorting through 3 row layers instead of 2.
What do we think? cc @shcheklein, @mattseddon

As discussed in planning we've decided on working on flattening then sorting the entire table.
Here's how the table looks flattened:
https://github.com/iterative/vscode-dvc/assets/43496356/60bb7342-76dd-473b-9d38-b894187ab935
Things to consider for this table:

[ ] We need a way to show what sub row belongs to what. Aka what experiment row belongs to what commit belongs to what branch. One option discussed was adding an extra column that gives git information (branch/commit sha)
[ ] There is no way to add or minus commits in branches. So far I'm not seeing a place for adding the +/- actions 🤔

Finished the first iteration of the sorted table design! Demo/Images:
https://github.com/iterative/vscode-dvc/assets/43496356/8863d2cf-66d8-468a-80a2-3947b7a976d7

I've added a column called "Branch/Commit" (Other options could be "Parents" or "Git Details") that gives branch/commit information and has a tooltip in the header that explains the currently missing functionality.
I do think we'll be able to add an action somewhere for increasing/decreasing commits but I figured we could do so in a follow-up since I wasn't sure where yet.
What do we think? cc @shcheklein

I do think we'll be able to add an action somewhere for increasing/decreasing commits but I figured we could do so in a follow-up since I wasn't sure where yet.

Good point about the need for the action! yes, can be a followup. I don't know how to do this also. Can be a context menu in a branch column? Can be +/- in the column itself? We can come up with something. Can be after all a way to quickly switch back to do that for now.

I've added a column called "Branch/Commit" (Other options could be "Parents" or "Git Details") that gives branch/commit information and has a tooltip in the header that explains the currently missing functionality.

my 2cs - let's split into Parent and Branch/Tags. Otherwise we are mixing semantics in the same column.

Updated the design by splitting the "Branch/Commit" column into "Branch/Tags" and "Parent" columns:
https://github.com/iterative/vscode-dvc/assets/43496356/ab08b26b-c2d9-4de4-8bb2-ebd80f903fb4

I kept the commit/branch icons to better differentiate them from regular cell columns, but here's an quick example of how they look without the icons:

cc @shcheklein

I still have concerns over the amount of space that we'll be taking up by adding two columns instead of one. We should also note that the "parent" or "commit" column will be empty when the record is a commit. I'd suggest changing the name to baseline even if we keep them separate.

I'd suggest changing the name to baseline even if we keep them separate.

Baseline is already used in Studio for delta / comparison mode. So, I think we should try to avoid it.
For commits we can show their actual parent commit?

We should also note that the "parent" or "commit" column will be empty when the record is a commit.

Updated the table to make commits rows' "Parent" column blank:

For commits we can show their actual parent commit?

Apologies, I'm not sure what you mean by "their actual parent commit". Commit rows don't have parent commits as far as I know?

@shcheklein is this what you're talking about:

The term Baseline is found all through the docs.
https://dvc.org/doc/user-guide/experiment-management
https://dvc.org/doc/user-guide/experiment-management/comparing-experiments
https://dvc.org/doc/studio/user-guide/experiments/run-experiments#manage-a-completed-experiment

Yes,Matt! That's the baseline that I meant.

So, I suggest not making it worse. I think parent is descriptive and applies to commits and exps, wdyt?
Overall, not a strong opinion.

As of release 1.1.0, the table now flattens when sorted, applying sorting to all rows (minus workspace):
https://github.com/iterative/vscode-dvc/assets/43496356/b1b8a945-b4e3-47de-bf58-a4331efe0af0
@Danila89, feel free to give us any feedback you have on the feature.

Works for me now, thank you very much!


Fix #649: Use CookieAuthStorageAdapter to allow cookie parsing/serialization for SupabaseClient for @supabase/ssr

What kind of change does this PR introduce?
Bug - fixes #649
Use CookieAuthStorageAdapter to allow cookie parsing/serialization for SupabaseClient when using @supabase/ssr
This incorporates #609 by @csenio to allow async getCookie methods for the StorageAdapter and get method for ServerCookieMethods in createServerClient in the
What is the current behavior?
Issue described in #649
What is the new behavior?
Creates a storage adapter ServerCookieAuthStorageAdapter in packages/ssr/src/createServerClient.ts by extending CookieAuthStorageAdapter which provides the cookie parsing/serialization expected by SupabaseClient's storage adapter.

Hi @prabhpreet this is intentional by design to not include the CookieAuthStorageAdapter. We will document how to use this library as we get ready for its official release. At the moment you will need to handle the cookie parsing yourself in your framework's code.

@silentworks May I ask the reasoning behind avoiding CookieAuthStorageAdapter?
Currently the set method for setting your own Cookies in ServerCookieMethods receives the entire stringified Session type for auth-token cookies, which can be a lot to store and cannot be serialised and stored as Cookies as-is.
Can using CookieAuthStorageAdapter be provided as another option to ServerCookieMethods? Reason being, auth-helpers/shared is parsing of auth-token cookies is separate from code-verifier and I didn't want to reinvent the wheel when using another framework.
Another option can be to provide a typed Session as a value for set and get methods.

We don't want to handle the overhead of how storage is handled inside this library itself since gotrue-js essentially already does that. We initially did this in the @supabase/auth-helpers because of cookie size limit, we wanted to reduce the size as much as possible when it comes to storing data into a cookie, however this required a number of workarounds which caused more issues than the ideal solution it was trying to provide. Once we have documented @supabase/ssr it will be more clear why we think this is a better approach.

Thanks, closing PR wrt above comment.


updates spring boot version

Description
updates spring boot to 2.1.5.RELEASE
Related Issue

Tasks

[x] I have signed the CLA.
[x] I have written tests and verified that they fail without my change.

PRB did in fact succeed, it was just reported wrong.


Feedback after coding day 1

Hey,
Discovered this project recently and I find it super interesting, I had a very superficial understanding of LSM engines having worked a little with Cassandra, but I never had the time and need to dig in and understand how it works in details. This tutorial sparked the motivation to do so and so far I find it really interesting! Thanks for taking your time to build such interesting learning resources.
I just finished the day 1 part for now but I feel that I have a bit of feedback to give:

I took me some time to understand that it's normal and expected that the BlockBuilder::add method is not responsible for sorting the entries of the block, but that instead the caller is responsible to insert entries in sorted order. After thinking quite a bit about it it finally made sense because blocks (and SSTables) are crafted from the in-memory tables and we therefore have all the data available when building a block, but it was not clear in the text and I spent some time wondering why we were implementing a bisect search in the block iterator while we didn't enforce any sorting in the block building.
Some things doesn't feel really "rusty". I'm thinking about the iterator interface you designed for BlockIterator, which looks really different from iterators we are used to have in rust. In particular, the idea of "invalidating" the iterator by using empty vecs as key and value when we consume all the iterator, looks really different from the pattern of the next method returning an Option<T>, with None marking end of iteration. This is not bad per se, this custom iterator having different requirements, but it's a bit surprising.
I think your unit tests could have more coverage, and also be split into smaller tests that do test a single property. For instance, when in the tutorial you specify a property like "Key length and value length are 2B, which means their maximum length is 65536.", I'm expecting to see a unit test that will cover this property and make sure it is enforced. I did some of this in my day 1 work in my fork if you wanna check it out (https://github.com/LeBoucEtMistere/mini-lsm/pull/1)
I think the tutorial steps could provide clearer entry points in code, i.e. "to solve this task, you will need to implement function in XXX".
I noticed you liberally use as conversions in your solution, they tend to be risky especially when you convert to a more restrictive type, e.g. usize to u16. In that case it's best to use try_into to handle the case in which the conversion fails. That's not a big deal but since that's a tutorial it's always nice to leave a note about this to teach people about the risks of as conversions :)
I never used the bytes crate before so I had to spend some time reading the doc and understanding how it works by looking at the solution code. I would have loved it if there was a note in the tutorial that mentioned it would be wise to leverage this crate to build the solution (because it is not mentioned anywhere except in the solution so I suspect many people might not know it's available to help), and maybe also giving a very quick example in the tutorial of how it can help playing with bytes.
In the part about the encoding of blocks, you give great schemas of the format, but I would also have loved seeing a concrete example to make it clearer, with actual example data.
I would have loved seeing some pointers to crates/algorithms in the extra tasks. I'm not very familiar with algos for compression/checksums, especially in the context of LSM engines, and it felt quite hard to explore this topic without a few pointers.

These are just a bunch of suggestions from my personal experience but I hope it helps you make your tutorial even better. Thanks for the great work, looking forward to working on day 2 tasks :D 🚀 !

Thanks a lot!

I took me some time to understand that it's normal and expected that the BlockBuilder::add method is not responsible for sorting the entries of the block, but that instead the caller is responsible to insert entries in sorted order. After thinking quite a bit about it it finally made sense because blocks (and SSTables) are crafted from the in-memory tables and we therefore have all the data available when building a block, but it was not clear in the text and I spent some time wondering why we were implementing a bisect search in the block iterator while we didn't enforce any sorting in the block building.

Strongly Agree

@LeBoucEtMistere

I took me some time to understand that it's normal and expected that the BlockBuilder::add method is not responsible for sorting the entries of the block, but that instead the caller is responsible to insert entries in sorted order. After thinking quite a bit about it it finally made sense because blocks (and SSTables) are crafted from the in-memory tables and we therefore have all the data available when building a block, but it was not clear in the text and I spent some time wondering why we were implementing a bisect search in the block iterator while we didn't enforce any sorting in the block building.

Can you explain it further? I still don't understand.. Thank you.

The SSTable is a durable storage primitive that is created by the DB once it needs to flush some in-memory data to disk. Therefore, the DB has total control on what data it puts in the SSTable and when it decides to create one. It does so when it has at least enough in memory-data to fully populate a single block (although this is just a required condition, I imagine it makes more sense to have more than one block before flushing). Since the DB is the one responsible to insert the records in the block and it knows them all before doing so, it can sort this data and insert it in the block while protecting this invariant that all the records in a block must be sorted.

Thanks for the feedbacks! I have rewritten everything up to what we had in 2023 and week/part 2 of this tutorial will be available soon! https://skyzh.github.io/mini-lsm/


Create 0278-first-bad-version.swift

File(s) Modified: 0278-first-bad-version.swift
Language(s) Used: swift
Submission URL: https://leetcode.com/problems/first-bad-version/submissions/1166677886/

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

@drxlx Thank you for your contributions 👍


Quantify donations on user profile

[ ] A "Donations from [username] have helped pay for N hours of freeCodeCamp server time" element on camper portfolios. Reddit uses this to help put the dollar value of donations in more tangible terms.

Should this be similar to the copy that we have in the donation page, since donating to help people learn is more relatable than paying for servers?
"Donation from [username have helped  provided 250 hours of learning to people around the world each month"
Additionally, the number of learning hours change based on the number of people using our platforms. What is a good way to reduce every dollar donated to maintain /learn, /news, and /forum into learning hours?
We could keep that number in .env, update it every quarter, and multiply it with the amount donated.
We could use the same multiplier for donate page message to keep it up to date.

I think we should have a call with Quincy, about this. Do you mind checking in with him and blocking some time. I am available at both your convenience.

I asked him about this during another call.
We could go with with the hours hours of learning and the multiplier that is based on hours of usage which translates roughly  about 50hr for 1 usd based on his last calculation.
let me know if you still would like to arrange an additional meeting.

let me know if you still would like to arrange an additional meeting.

If you have a clear path forward, go ahead with the pull-request when you are ready and we can take it from there.


Generate custom yaml without a Kubernetes cluster

Describe the solution you'd like
I'd like to generate custom manifest files without using a cluster.
Describe alternatives you've considered
To generate custom yaml with tridentctl install --generate-custom-yaml needs a Kubernetes cluster which can connect. If there is no cluster, the command will fail, but the Kubernetes cluster should not be necessary to do that.
Additional context
$ tridentctl version --client
+----------------+
| CLIENT VERSION |
+----------------+
| 20.01.0        |
+----------------+
$ tridentctl install --generate-custom-yaml
FATA Install pre-checks failed; could not initialize Kubernetes client; could not find the Kubernetes CLI; Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.2", GitCommit:"59603c6e503c87169aea6106f57b9f242f64df89", GitTreeState:"clean", BuildDate:"2020-01-18T23:30:10Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}
The connection to the server localhost:8080 was refused - did you specify the right host or port?

https://github.com/NetApp/trident/issues/247
にてHelm Chartがサポートとなりました。
本Issueの目的は達成されたかと思うので、Closeします。

tridentctl install --generate-custom-yaml requires access to the Kubernetes cluster to determine the Kubernetes version and the version of some Kubernetes CRDs. This information is used to generate the YAML to install Trident in that environment.


Add tests for distro-enabling options

#1435, #1436, #1438 are all options the current Microsoft builds don't use but are needed for distro maintainers.  We should add a Christmas-tree test that uses all these options.

Duplicate of #1476


NFS client provisioner stops working in OpenShift 4.8

Running OpenShift 4.8.0-fc.2 with Kubernetes 1.21.0 causes the NFS provisioner to stop working.
Using one of the later versions (k8s.gcr.io/sig-storage/nfs-subdir-external-provisioner:v4.0.2) seems to fix the issue.
However, I have not tested whether or not this version of the nfs provisioner works on any version below 4.8.

this is caused by https://github.com/kubernetes-sigs/nfs-subdir-external-provisioner/issues/25#issuecomment-742616668
and the suggestion is to move to the latest version of the nfs client provisioner which was adopted under an openshift sig
https://github.com/kubernetes-sigs/nfs-subdir-external-provisioner

Just-for-information, and might be worth checking/update the docs.:
https://github.com/kubernetes-retired/external-storage/tree/master/nfs-client is moved into https://github.com/kubernetes-sigs/nfs-subdir-external-provisioner

oh, we still use the quay.io/external_storage/nfs-client-provisioner:latest and that's quite old.
Update image to k8s.gcr.io/sig-storage/nfs-subdir-external-provisioner:v4.0.2 in devel tree for further testings.

Solved in devel, will me merged into master with  PR #183  - (Devel branch is at OCP 4.8.2)


Data Entered by Users needs Data Validation

As a human doing data entry I want to get feedback if I'm trying to enter or submit invalid data so that I submit data correctly the first time and it's harder for me to make mistakes.
Describe the solution you'd like and the specific acceptance criteria to be completed
We should help users avoid making mistakes by not submitting invalid data. This is needed on:
Design

[ ] Error messages/notifications/UI treatments for the different error states (if not included in the design system we use)

Engineering

[ ] Test Results- must pick a result before submitting
[ ] AOE questions- input fields
[ ] General Settings
[ ] Add new person UI
[ ] Update person details

Note: In a future release (post-MVP) we should add address normalization too.

Realized this is an Epic in our discussion, probably for next sprint
Also recognized needs another issue about what design standard we are using for inputs

Engineers! Talk to @hmyers-cms about this

Validation logic on results in Airtable needed to help flesh this out


📖 Docs: better document markup.toml and Goldmark

Background
Nuno's personal website has a markup.toml that begins like so:
# -- Markup --
# These settings are required for the theme to function.

https://blowfish.page/docs/configuration/#other-configuration-files mentions markup.toml, but doesn't really say what should be there.
https://github.com/nunocoracao/blowfish/discussions/881 is a discussion where HTML tables were enabled by including the below in markup.toml:
[goldmark]
[goldmark.renderer]
  unsafe = true

Request
It would be nice to flesh out the docs on markup.toml, including talking about what it should look like, and valuable entries to add.

Hello @jamesbraza thanks for the feedback. Goldmark is used by Hugo and these configurations affect all themes not just blowfish. Check more here - https://gohugo.io/getting-started/configuration-markup/
What else were you expecting the theme docs to contain?

Oh yeah sorry, I can clarify and add a PR to fix when I understand.
Does blowfish actually require a markup.toml natively?  And if yes, does it require a specific subset of the options listed on the Hugo page?  I ask because it seems to have been working without any markup.toml at all for me.
However, your personal website states that it does matter, so I am wondering if there is some like basic boilerplate markup.toml we should document

@jamesbraza as stated in the installation docs - https://blowfish.page/docs/installation/
That is one of the files that is required - yes. You can read more in the docs above about what those configs are driving. Removing config files shouldn't break the theme, just make something not look/feel the way they should.

So just to confirm, on initial getting started with blowfish, should markup.toml be added as an empty file?  Or should it contain some fields?  I can open a PR to add the details

Hello @jamesbraza the docs for Blowfish include that info. Please read them before opening more threads or PRs.


docker-machine env seems to be broken

Running the docker-machine env gives the following:
export DOCKER_TLS_VERIFY=1
export DOCKER_CERT_PATH="/Users/ryan/.docker/machine/machines/om-6"
export DOCKER_HOST=x.x.x.x

If this is copy pasted, bash works fine. However,  running $(docker-machine env) results in the following when running any docker command.
FATA[0000] Couldn't read ca cert "/Users/ryan/.docker/machine/machines/om-6"/ca.pem: open
"/Users/ryan/.docker/machine/machines/om-6"/ca.pem: no such file or directory

The quotes seem to be preserved literally.

This is my fault ;)
This isn't a bug, however it is a change in behaviour. You need to do eval "$(docker-machine env)" now. See #795 for the reasoning, and also #781 for the original bug that drove the PR (#783).

@hairyhenderson I figured there was good reason behind it. Thanks for the clarification! Feel free to close this (or keep it open if you expect others might need further clarification)

@rschmukler - I can't close it (I'm not an owner) - can you close it? This is covered in the changelog, and it's been explained a few times in IRC, so hopefully not too many more issues get logged about it :)

Oh, well then very sorry I didn't see it haha. Closing!


flux to lux in bayes-nn

It will be a great help if someone looks into this error, I do not know how it works!:
Code: https://github.com/shravanngoswamii/docs/blob/d3c0e0de7930bc862ec355edd614788f1079b040/tutorials/03-bayesian-neural-network/index.qmd#L209C1-L213C4

I do not know why the preview deployment is failing, its working fine when I open PR in my fork:
preview: https://shravanngoswamii.github.io/docs/pr-previews/9/tutorials/03-bayesian-neural-network/
pr: https://github.com/shravanngoswamii/docs/pull/9

@shravanngoswamii It is like permission issues. If so, you can always create new branches within this repo rather than from your fork.

Ohhk, let me look into this issue too as in future if someone else opens pr then this should be working fine!

All codes work well on my side.
Maybe we can reduce the trajectory length N = 5000 (Line 211) to N = 500 to reduce runtime.
Also, safe to modify the nn_predict as:
function nn_predict(x, θ, num)
    num = min(num, size(θ, 1))  # make sure num does not exceed the number of samples
    return mean([first(nn_forward(x, view(θ, i, :))) for i in 1:10:num])
end

Further, following code snapshot allows user to choose from the drop-down menu a sample and plots the corresponding decision boundary (using Plotly.jl):
using PlotlyJS

mcmc_samples = [θ[k,:,1] for k in 1:10:200] # thinning the chain

function generate_decision_boundary_plot(θ)
    Z = [first(nn_forward([x1, x2], θ, ps)[1]) for x1 in x1_range, x2 in x2_range]
    contour = PlotlyJS.contour(x=x1_range, y=x2_range, z=Z, colorscale="Viridis")
    return contour
end

initial_plot = generate_decision_boundary_plot(mcmc_samples[1]) # generate initial plot

# create dropdown menu for selecting different MCMC samples
dropdown_buttons = [attr(
    label = "Sample $i",
    method = "restyle",
    args = [[json([generate_decision_boundary_plot(mcmc_samples[i])])]]
) for i in 1:N]

layout = Layout(
    title="Decision boundary based on chosen sample",
    updatemenus=[attr(
        buttons=dropdown_buttons,
        direction="down"
    )]
)

fig = Plot([initial_plot], layout)
display(fig)

Not sure if this is what you wanted @yebai.

Code<EMAIL_ADDRESS>
@avik-pal do you know why we have this ReverseDiff related warning?

For the most part, that should be harmless; it is just telling you the layer got an Array{<:TrackedReal} instead of TrackedArray. The former tends to perform badly for neural networks, so we automatically convert that to a TrackedArray.

Let's keep things simple in this PR. I am mostly referring to improving the plotting and visualisation of results.

I will look into adding interactivity in plots! let's keep this feature for future!

@yebai This PR is good to go after this issue:
Code:
# Perform inference.
N = 500
ch = sample(bayes_nn(reduce(hcat, xs), ts), NUTS(; adtype=AutoReverseDiff()), N);

Warning
┌ Warning: Lux.apply(m::Lux.AbstractExplicitLayer, x::AbstractArray{<:ReverseDiff.TrackedReal}, ps, st) input was corrected to Lux.apply(m::Lux.AbstractExplicitLayer, x::ReverseDiff.TrackedArray}, ps, st).
│
│ 1. If this was not the desired behavior overload the dispatch on `m`.
│
│ 2. This might have performance implications. Check which layer was causing this problem using `Lux.Experimental.@debug_mode`.
└ @ LuxReverseDiffExt ~/.julia/packages/Lux/PsbZF/ext/LuxReverseDiffExt.jl:25
┌ Info: Found initial step size
└   ϵ = 0.8

Preview: https://turinglang.org/docs/pr-previews/462/tutorials/03-bayesian-neural-network/

It will be a great help if someone looks into this error, I do not know how it works!: Code: https://github.com/shravanngoswamii/docs/blob/d3c0e0de7930bc862ec355edd614788f1079b040/tutorials/03-bayesian-neural-network/index.qmd#L209C1-L213C4

Not sure if this helps things, but I found switching to Tracker.jl doesn't give any warnings. Results are fairly similar and Tracker.jl ends up being relatively fast (~85 sec to run 1,000 samples).


Commit history was lost

Why was this repo's history reset with a new "Initial commit" 3 days ago? There's nothing documenting why this was done.

That was our initial public release of this repo! I probably should have made the commit message be more descriptive :).


CreateGalleryItem parameter in New-Server2016VMImage and checking for trailing / in ArmEndpoin URL

Added CreateGalleryItem parameter in New-Server2016VMImage and call Add-VMImage based on CreateGalleryItem value
Checking for trailing / in ArmEndpoin URL in Remove-VMImage and Add-VMImage

@KirtiWandhare,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks!


Invalid Request when using next page token

Requesting the next page returns an invalid request, it looks like the param sensor=false is causing it as removing this from the url fixes the issue. Putting the param first also seems to fix the issue. Replacing & with & will also fix it.

This doesn't seem right at all... How quickly are you requesting the next page after the first page has been returned?
"There is a short delay between when a next_page_token is issued, and when it will become valid. Requesting the next page before it is available will return an INVALID_REQUEST response"

I had the same problem and tried around to find the time between requests. It didn't work for with a waiting time <= 100ms. 110ms works for me.


Deprecated Field Name in Documented Query

In the query documented as "To get a list of nodes sending any data (and the amount of data sent by each) the follow query can be used:" the results come back with a computer name:
deprecated field: see http://aka

@andrewatfornax Thank you for the question. In order to best address your question, could you provide us with the URL of the MS Doc that you were following, if any?

@andrewatfornax Any further update here?

https://docs.microsoft.com/en-us/azure/azure-monitor/logs/manage-cost-storage

@andrewatfornax Thank you for the response. I've routed this to the appropriate Team to investigate and update.

@andrewatfornax Great observation and thanks for reporting! I was also able to reproduce the same.

Assigning content owner @bwren for visibility, review and document enhancement consideration as appropriate.

I talked to the PM about this, and that row in the results can be ignored. He's going to update the query to exclude that row.
#please-close


Warnings : Should I be concerned?

I get these no matter how I build, it compiles fine, just concerned these warnings might mean something serious as I read your statement about the 0100003F on your main page, I get these? I have VS2017 and the same exact SDK version as your notes suggest. Happens if I build for x32 or x64 or anything else for that matter ...
Severity	Code	Description	Project	File	Line	Suppression State
Warning	C4244	'argument': conversion from 'double' to 'int', possible loss of data
JavaScript.Net	javascriptinterop.cpp	412
Warning	LNK4248	unresolved typeref token (0100004E) for 'v8.internal.Isolate'; image may not run	JavaScript.Net	JavascriptInterop.obj	1
Warning	LNK4248	unresolved typeref token (01000038) for 'v8.internal.Isolate'; image may not run	JavaScript.Net	JavascriptFunction.obj	1
Warning	LNK4248	unresolved typeref token (01000042) for 'v8.internal.Isolate'; image may not run	JavaScript.Net	JavascriptExternal.obj	1
Warning	LNK4248	unresolved typeref token (01000041) for 'v8.internal.Isolate'; image may not run	JavaScript.Net	JavascriptContext.obj	1

Not sure if this helps?
https://docs.microsoft.com/en-us/cpp/error-messages/tool-errors/linker-tools-warning-lnk4248?view=vs-2019

Regarding the "unresolved typeref token" warnings, the documentation you
provided says that it might cause a problem for CLR types.
v8.internal.Isolate is not a CLR type - it is native - so it should be
OK.  The other warning looks easily fixed, but also not likely to cause any
trouble - the values should be in range.
Regards,
Oliver
On Tue, Aug 6, 2019 at 12:25 AM Mike Griffin<EMAIL_ADDRESS>wrote:

Not sure if this helps?
https://docs.microsoft.com/en-us/cpp/error-messages/tool-errors/linker-tools-warning-lnk4248?view=vs-2019
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JavascriptNet/Javascript.Net/issues/81?email_source=notifications&email_token=AAVKA4LBCSQS7BSWT2QL5RLQDAZ53A5CNFSM4IJLWRM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3R7OZY#issuecomment-518256487,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVKA4IDZDWI7NGJWU6YYC3QDAZ53ANCNFSM4IJLWRMQ
.


Avoid ID clashes

Currently the the ID for a new paste is randomly generated in the caller of the database insert() function.  Then the insert() function tries to insert a new row into the database with that passed ID.  There can however already exists a paste in the database with the same ID leading to an insert failure, due to a constraint violation due to the PRIMARY KEY attribute.  Checking prior the the INSERT via a SELECT query would open the window for a race condition.
A failure to push a new paste is quite severe, since the user might have spent some some to format the input.
Generate the ID in a loop inside, until the INSERT succeeds.

An ID clash is indeed an extreme issue but perhaps we can solve it differently by constructing an INSERT query that generates a random primary key?

There exists a sqlite built-in functionrandom(and randomblob), but this would not solve this issue, cause the function might still generate a result already existent in the database.
Also we need the ID for the URL path generation, so this approach would require a secondary SQL query to get the ID in all cases (also non-clashing).


Multi GPU support

How to utilize multiple GPUS, so that I can get higher FPS.
Also, any way to decrease the inference time per image, even with single GPU.

Multi-GPU inference is not implemented. It would be quite challenging to get a higher FPS that way.
If you use our inference code, it should be fairly optimized. You could try using all 16-bit inference rather than mixed precision.


BeerEx: A Faster Implementation of Beer

When drinking BeerEx, it'll go into your blood about 15-times as fast as before. And even getting rid of it is faster: Studies have shown that BeerEx dissolves about four times as fast as normal Beer!

Thank you for your contribution, nice work!
I'm currently a bit confused about the IEnumerable DeserializeBeerImpl.
It would also be nice, if you add some comments to all your code like in Beer.cs.


(1/4) Trigger v0.17.6 Bundle Staging Pipeline

Issue #, if available:
Description of changes:
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm

/retest

/retest

/retest


Update Azure.Analytics.Synapse.Artifacts

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API changes have been detected in Azure.Analytics.Synapse.Artifacts. You can review API changes here

API changes have been detected in Azure.Analytics.Synapse.Artifacts. You can review API changes here

@annelo-msft could you please help review? Thanks in advance.

Hi @annelo-msft , thanks for your review. Using object type is by design. In Synapse Pipeline, there is a feature called "dynamic content". If the property need to support "dynamic content", it's type would be string (or Expression with resultType string), so we use object type.
Here is an example for previous property: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/synapse/data-plane/Microsoft.Synapse/stable/2020-12-01/entityTypes/Pipeline.json#L191-L194

Hi @wonner,

If the property need to support "dynamic content", it's type would be string (or Expression with resultType string), so we use object type.

If the value of this API is either string or “lazy” string, why is it not typed as string? Also, what does the string represent? Is it JSON?
Also, is this true for properties like Uri and Encrypted Credential?


Disable react/jsx-indent

It is incompatible with prettier, and unnecessary now that we have it.

:tada: This issue has been resolved in version 4.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Led S.O.S and Blink Elimination

This is not a malfunction, but a proposal for a modification. If this request is in the wrong place, I apologize. I would like to propose eliminating the flashing of the LED when using the flashlight, leaving only the on/off function. Would that be possible? Do you consider it useful?

I think the code part is this one, in flashlight.h
#ifndef APP_FLASHLIGHT_H
#define APP_FLASHLIGHT_H
#ifdef ENABLE_FLASHLIGHT
#include <stdint.h>
enum FlashlightMode_t {
FLASHLIGHT_OFF = 0,
FLASHLIGHT_ON,
FLASHLIGHT_BLINK,
FLASHLIGHT_SOS
};
extern enum FlashlightMode_t gFlashLightState;
extern volatile uint16_t     gFlashLightBlinkCounter;
void FlashlightTimeSlice(void);
void ACTION_FlashLight(void);
#endif
#endif

Hello,
Of course, it's possible. But the question is why ?
The FlashLight action lets you choose between several states: on (fixed), flashing, SOS or off.
Each of these states has its own purpose.
Armel.

First of all, thank you for your reply. The reason for my suggestion is that I find it tedious, once the LED is activated, to have to press and hold the button multiple times just to turn it off.  How often do I use the flashlight on the radio: a few times. How many times in my entire life have I used the blinking and even worse the SOS? Never :-) However, I admit that if I am the only one who feels this need, it is likely that the issue is of little importance ;-) May I ask if it is possible to download an uncompiled version of your firmware?

Okay, thank you. I understand.
Well, if I have several similar requests about the LED, I'll do it. No problem ;)

Thank you!
:-)


Consider downloading in parallel

https://github.com/PowerShell/PowerShellGet/blob/98aadd21b741c69ad782c6438663b18a309ff9e3/src/InstallPSResource.cs#L686
You went to all the trouble to implement async, why not download in parallel, wait for the tasks to complete, and then move on, rather than download one package at a time?

Thanks @JustinGrote we are hoping to achieve this, going to mark as a duplicate of the issue we are tracking for this #69


Unable to copy from Logs

I am unable to copy text from the logs for a pod.  I'm not sure if this is a bug or a feature.  But it is often nice to copy the information for put in either DevOps tasks or bugs.

That would definitely be a bug!  Thanks for letting us know, we'll take a look.

Hi @werebear73, could you give us more infos to replicate it? What do you do exactly? I tried to get logs for a pod and copy/paste part of them and it worked fine. What am i doing differently from you?

I using the same process, but it just pastes old clipboard contents.
I'm running Windows 10 and VSCode Version 1.63.2

@itowlson can you try to replicate it? It works fine for me

I did.  Here is a video.
https://user-images.githubusercontent.com/3290150/149814159-b548405e-7aee-4f91-bcef-b9d3d6f8d3a2.mp4

You're going to love this @lstocchi.
If I copy from the logs window using right-click > Copy, I see the behaviour @werebear73 describes - the old clipboard contents remain.
If I copy from the logs window using Ctrl+C, I see the behaviour you describe - the selected logs are now on the clipboard.
Riddle me that eh.
I am on plain Windows in this case (Docker is running under WSL but the extension environment is all Windows).
@werebear73 Could you try using Ctrl+C and see if that works for you too?

Same behaviour in WSL by the way

what the heck @itowlson ... I updated everything i had on my windows machine and now i have your exact behavior, lol.  I'll investigate why this happens. Sorry for the trouble @werebear73

Thank you

Update: still haven't a working hack for this but there is an opened issue in vscode https://github.com/microsoft/vscode/issues/141080 . Hopefully they will fix it soon.


Fix userlist bug

:+1:

@fridays I notice you're not in userlist.js, do you want to add yourself in this PR as well?


[Machine Learning] Is a Knowledge Graph Just Another Means of Knowledge Visualization?

Proposed title of article
Is a Knowledge Graph Just Another Means of Knowledge Visualization?
Proposed article introduction
Businesses have to face the pace of constantly evolving data needs. Knowledge graphs can help companies move away from traditional databases and use the power of natural language processing, machine learning, and semantics to better leverage data.
knowledge graphs are the foundation of modern data and analytics, with capabilities to enhance and improve user collaboration, machine learning models, and explainable AI. Although graph technologies are not new to data and analytics, there has been a shift in the way they are used. A knowledge graph brings together machine learning and graph technologies to give AI the context it needs.
Key takeaways

What is a knowledge graph?
Knowledge graph and visualizations
Knowledge graphs as a solution to machine learning problems
Where are used today

Article quality
This article is in-depth and covers additional topics.
References
N/A

Please add more depth in your Article Quality section so we can review your proposed article. What additional topics do you cover that no other article does?

@louisefindlay23 This article will show the ready why the knowledge graph is more than just visualizations. An argument that is not available in other blogs. It will also explain how knowledge graph is assisting to solve machine learning issues.

Approved but make sure to add enough unique value and content. See this for an article on a similar topic to yours.


Change user's email address and password

Users need a way to set their password and email address with a profile page (profile page to be created).
Admins need a way to set these on behalf of the user from the user management page at /users-list .

Admins now have a way to set the display name, password, email address, and phone number on the user management page 👍
The users need a profile page (...and preferences page, but that's for another GitHub issue).

A profile page has been created for the user to view his/her details - we now need the ability for the user to change the profile fields.
A change password page has been created.

This has been completed in 5469fc5d329bd247fcec393d75191016e06449dd


node-sqlite3-wasm.d.ts missing

It would be nice to have d.ts file (maybe adapted sqlite3.d.ts) to omit warnings in VSC.
Or maybe @types/node-sqlite3-wasm ?

Hi, I added type definitions in PR #19. However, I'm not sure whether the types are properly recognized by TypeScript. Maybe some kind of export is required. Any feedback is welcome.

There is a new, IMHO correct PR. Sorry for the old one.

Done in #19


Adding GPU support for LLVM backend (+ some refactoring)

This is a draft PR that aims to add GPU support for LLVM backend, focusing on CUDA targets and producing NVVM IR. What has been done:

[x] New Target class has been added. The idea behind it is that previously vector width, precision of floating-point types, math libraries were passed separately. If we add GPU support, then we also would want to have GPU math library, vendor name, etc. Passing all these parameters to LLVM visitor does not seem like a nice solution, so I decided to create a  class  that encapsulates user-specified "ideal" target  - CPU/GPU.

For example:
#  default CPU
bin/nmodl mod/test.mod -o out llvm --ir

#  SIMD CPU
bin/nmodl mod/test.mod -o out llvm --ir cpu --name skylake --vector-width 2

# SIMD CPU with VLA support (for future SVE and not used atm)
bin/nmodl mod/test.mod -o out llvm --ir cpu --name aarch64 --vector-width 2 --vla

# GPU
bin/nmodl mod/test.mod -o out llvm --ir gpu --name cuda

This is also useful because API is nicer (target->is_simd_cpu() instead of vector_width > 1), classes have less fields, now there  is a better encapsulation of classes/members. Moreover, in the future we can add LLVM API for target triple, library information, etc. to query via Target class.

[x] GPU AST transformation and a new AST node
Now there is a GPUId node that represents a calculation for the thread id. Together with AST transformation, NMODL AST for GPU code generation looks like:

VOID nrn_state_hh(INSTANCE_STRUCT *mech){
    GPU_ID id
    INTEGER node_id
    DOUBLE v
    IF (id<mech->node_count) {
        node_id = mech->node_index[id]
        v = mech->voltage[node_id]
        mech->m[id] = (mech->minf[id]-mech->m[id])/mech->mtau[id]
    }
}

This currently gives a runtime error during code generation because visitor for  GPUId is not added yet.
TODOs:

[ ] Add visitor for GPUId.
[ ] Add tests for AST transformation and NVMM generation.

Also, the command line has changed now, see full spec below:
llvm
  LLVM code generation option
  Options:
    --ir REQUIRED                         Generate LLVM IR (false)
    --no-debug                            Disable debug information (false)
    --opt-level-ir INT:{0,1,2,3}          LLVM IR optimisation level (O0)
    --precision INT:{32,64}               Precision of floating-point types (64)
    --fast-math-flags TEXT:{afn,arcp,contract,ninf,nnan,nsz,reassoc,fast} ...
                                          Fast math flags for floating-point optimizations (none)

cpu
  LLVM CPU option
  Options:
    --name TEXT                           Specific CPU target to use
    --math-library TEXT:{Accelerate,libmvec,libsystem_m,MASSV,SLEEF,SVML,none}
                                          Math library for SIMD code generation (none)
    --vector-width INT                    Explicit vectorization width for IR generation (1)
    --vla BOOLEAN                         Generate IR for VLA target (false)

gpu
  LLVM GPU option
  Options:
    --name TEXT                           Specific CPU target to use
    --math-library TEXT:{libdevice}       Math library for SIMT code generation (none)

benchmark
  LLVM benchmark option
  Options:
    --run                                 Run LLVM benchmark (false)
    --opt-level-codegen INT:{0,1,2,3}     Machine code optimisation level (O0)
    --shared-libs TEXT:FILE ...           Shared libraries to link in JIT driver
    --mechanism-data-size INT             Size of mechanism data instance (10000)
    --num-experiments INT                 Number of experiments for benchmarking (100)
``


POST support in xhr, etc

Not sure if this is philosophical, technical, or time-based choice, but the implementation of xhr doesn't currently support POST requests.  An optional data argument could be passed, and the method can be GET if data is null and POST otherwise.  If it is a matter of time, I would be willing to fork and write it, but I would prefer to know beforehand that you wouldn't reject the change on philosophical or technical grounds.

I'm okay with supporting POST, but I'm not sure about more optional arguments; the mime type is already optional, and there's a point where you have too many optional arguments and it's really confusing as to how to interpret them. (Case in point: node-mongodb-native.)
One way to avoid this might be method chaining, like the rest of D3, giving the arguments names:
d3.xhr(url).on("success", callback); // get
d3.xhr(url).method("post").on("success", callback); // post
d3.xhr(url).data(object).method("post").on("success", callback); // post with data

I guess if we only cared about GET and POST methods, we could make this even shorter:
d3.xhr.get(url).on("success", callback); // get
d3.xhr.post(url).on("success", callback); // post
d3.xhr.post(url).data(object).on("success", callback); // post with data

(Not sure if that's a good idea, though, since we might want HEAD, PUT and DELETE in the future… and delete is a JavaScript keyword.)

I started playing with it a bit and I found myself starting with xhr_post, which feels a little distasteful.
Although your second example does feel niftier, I agree that the keyword collision makes it less desirable.  Just now I played with some combination of your two examples, but I couldn't come up with anything that didn't just seem silly.  I'd say the first example is the winner.
I'm new to github (and to git, really).  I'm assuming that if I were to do this, I'd fork and then issue a pull request?
Also, does this need to be a breaking change? It sort of seems like it will have to be due to the way xhr works currently.


Access quantized weights

I have been trying to access the quantized weights, but the quant layer has no attribute 'weight'. Looking at the code, I think that the quant layer only quantizes the input (i.e. the result of previous layers) in the forward pass, and not the weights of layers which I want quantized. Is there any workaround for this?

You can refer to quantize.py.
if args.param_bits < 32:
    state_dict = model_raw.state_dict()
    state_dict_quant = OrderedDict()
    sf_dict = OrderedDict()
    for k, v in state_dict.items():
        if 'running' in k:
            if args.bn_bits >=32:
                print("Ignoring {}".format(k))
                state_dict_quant[k] = v
                continue
            else:
                bits = args.bn_bits
        else:
            bits = args.param_bits

        if args.quant_method == 'linear':
            sf = bits - 1. - quant.compute_integral_part(v, overflow_rate=args.overflow_rate)
            v_quant  = quant.linear_quantize(v, sf, bits=bits)
        elif args.quant_method == 'log':
            v_quant = quant.log_minmax_quantize(v, bits=bits)
        elif args.quant_method == 'minmax':
            v_quant = quant.min_max_quantize(v, bits=bits)
        else:
            v_quant = quant.tanh_quantize(v, bits=bits)
        state_dict_quant[k] = v_quant
        print(k, bits)
    model_raw.load_state_dict(state_dict_quant)

Thank you. It worked, but I have some confusion regarding 'sf'. My param_bits is 8 but the sf is often greater than 8. Can you please explain this?

Hello again. I am checking my quantized network after specifying sf=6 in LinearQuant. the problem is that when I am quantizing forward iteration and evaluating the accuracy for vgg16 network, it comes out to be 68%. But when I am done quantizing forward iteration and evaluate the model again, the accuracy drops to 2%. Your help would be appreciated.

sf decide the least precision, bits decide the range. So if sf=2 and bits=8, it means the least precision is 0.25, and range is [-127, 128] * 0.25


Upgrade attempt to LVGL 8.3

Would like to upgrade the LVGL component to 8.3 and report here what challenges it might bring.
Expected Results
Same examples done till now but with an upgraded library.
ESP32 Chip version
Actually using epdiy v7 hence ESP32S3.
ESP-IDF version  Using version 5.1 (5.3 still has issues to resolve in epdiy v7)

Done and merged. Only pending to review & finish the https://github.com/martinberlin/lvgl_epaper_drivers/pull/6 lvgl_epaper_drivers PR


Remove Azure Functions 4.0 preview related text

I think we could also clean up the https://aka.ms/functions-dotnet6earlypreview-wiki link or point it somewhere else.

@kendaleiv : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@kendaleiv : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


maven.sumup.com is returning 403

Since yesterdy we've been getting this error when trying to build our Android app:
      > Failed to list versions for androidx.appcompat:appcompat.
         > Unable to load Maven meta-data from https://maven.sumup.com/releases/androidx/appcompat/appcompat/maven-metadata.xml.
            > Could not GET 'https://maven.sumup.com/releases/androidx/appcompat/appcompat/maven-metadata.xml'. Received status code 403 from server: Forbidden

Me too....

Could not resolve all files for configuration ':app:devDebugCompileClasspath'.
Could not resolve com.sumup:merchant-sdk:5.0.0.
Required by:
project :app
Could not resolve com.sumup:merchant-sdk:5.0.0.
Could not get resource 'https://maven.sumup.com/repository/releases-local/com/sumup/merchant-sdk/5.0.0/merchant-sdk-5.0.0.pom'.
Could not GET 'https://maven.sumup.com/repository/releases-local/com/sumup/merchant-sdk/5.0.0/merchant-sdk-5.0.0.pom'. Received status code 403 from server: Forbidden

Since yesterdy we've been getting this error when trying to build our Android app:
      > Failed to list versions for androidx.appcompat:appcompat.
         > Unable to load Maven meta-data from https://maven.sumup.com/releases/androidx/appcompat/appcompat/maven-metadata.xml.
            > Could not GET 'https://maven.sumup.com/releases/androidx/appcompat/appcompat/maven-metadata.xml'. Received status code 403 from server: Forbidden

Same here:
> Could not GET 'https://maven.sumup.com/releases/com/google/firebase/firebase-sessions/maven-metadata.xml'. Received status code 403 from server: Forbidden

Same here


Add Python prototype

This is a configuration prototype in Python.
AFAIK, other prototypes currently parse the configuration file into a certain, language-specific model, then use if/elses to query this model, depending on what is found in the model with this querying actual objects are created.
This approach has a few disadvantages:

It requires manual if/else querying.
It doesn't guarantee a complete implementation of all the possible scenarios that any configuration file may have.

The previous point 2 requires a bit more explanation. For example, a configuration implementation that manually does if/else querying of the model may miss a possible scenario that is included in the configuration file but is not implemented. For example, a tracer provider may have batch and simple span processors, but maybe the implementation only queries the configuration model for batch, forgetting about simple. For someone implementing the configuration model in this way, it is easy to make this kind of mistake because it is not obvious from any configuration file or the schema which are all the possible configuration scenarios.
This prototype implements configuration in a different way.
It first resolves the schema, by replacing all the external $defs in the schema with their actual definition. This produces a complete tree representation of the schema.
Then it recursively traverses the schema tree to find all the possible paths. With this information, it writes a Python file with one function per schema path. The arguments that these functions receive are taken from the schema. All these functions are associated with their schema path in a Python dictionary.
Now, I implemented the content of these functions (so far I have only implemented the content of the functions that are related to Resource and TracerProvider creation). It is important to mention here that this means that this prototype can also have an incomplete implementation (as mentioned in point 2 above), but in this case it is clear for whoever is implementing this what is missing from the implementation: the functions whose content has not been implemented yet.
Finally, this prototype takes the configuration file, parses it into a Python dictionary (which is another tree representation). It recursively traverses this tree and when it finds a node it this tree it calls the function associated with this node path. For example, to create a resource:
It first finds the resource node in the configuration tree. It traverses through all the attributes and finds that first it needs to create the object for attributes, so it recursively keeps traversing the configuration tree until it finds the resource/attributes node. Since all the attributes for resource/attributes are terminal (strings, ints, floats, booleans), they can be used right away to create the attributes object. This object is recursively returned to the resource node which can now complete the creation of the Resource object which is finally returned.
To try this prototype, install nox, go to the opentelemetry-configuration/prototypes/python directory and run:
nox -- -s -vvvv -k test_create --pdb
This will run this test case which uses a "kitchen sink" configuration file (without samplers, more about that later). In that test case a Resource object is created first, followed by a TracerProvider object. The execution will stop in every function that is called to create those objects. Use c to move forward. Type the name of any function attribute to check its value.
The current schema has a recursive definition for the Samplers. This doesn't work with this implementation because it is not possible to define the paths for a sampler if the sampler can include infinite other samplers. I think that we don't need a recursive definition of the samplers (I don't think anyone needs to define a sampler that includes an arbitrarily long sequence of nested samplers). I think to solve this situation we can separate the definition of samplers into samplers that can contain one sampler and terminal samplers which are contained by other samplers.

Closing since this repository is not meant to house implementations of file configuration.


Building against master with mono

Hi,
Is it meant to be possible to build against godot master with mono?
The error seems to be in the glue code, I have no clue on what's wrong,... so a PR is welcome of course, but I don't know enough about mono/C# to fix it myself.


Random crashes only on dust2 comp

Tried debugging but it only shows me "Exception thrown at 0x6CE536B4 (cheat.dll) in csgo.exe: 0xC0000005: Access violation reading location 0x000000F4." this only happens when ive injected and in a comp game on dust2

0xC0000005 means bad pointer
0x000000F4 looks like an offset added to nullptr
attach a debugger and tell us at what line it crashes.

Like @xAkiraMiura said, you need to debug it, so compile it in Debug x86 mode and put the following files in the same folder with csgo.exe and inject from that location as well:
Debug\Osiris.dll
Debug\Osiris.ilk (if exists)
Debug\Osiris.pdb
Debug\vc142.idb (if exists)
Debug\vc142.pdb (if exists)
It should show you the line like in the picture.

ok i have tried with the debug .dll and i cant even open the menu without it throwing me loads of imgui errors in imgui.cpp and i havent even touched that file.

IM_ASSERT(g.CurrentWindowStack.Size > 1 && "Calling End() too many times!"); in imgui.cpp line 5805

0x000000F4 indicates you're trying to read team netvar of a nullptr entity. This error is not possible in the unmodified version of the source code as we're using team netvar only in one place which has null pointer check. Call stack in debugger window will reveal you were you forgot to check entity pointer.


First iteration: refactor PublishingAPI event handling

Trello card
We still have a few iterations in front of us, but this is the
first step forward.
We have made it explicit the different flows when processing
events so the code is easier to follow. We still have classes
that do too much, but we plan to simplify them in incoming PRs
For the time being we have split the code in:

messages/*
handlers/*
Consumer

So we have two types of messages, two types of handlers, and
a consumer that starts the flow.

Rebased and force pushed after addressing comments


feat(query): Add name to class

Codecov Report
Patch coverage: 25.00% and project coverage change: -0.02% :warning:

Comparison is base (97c5cd3) 89.42% compared to head (343bf89) 89.41%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #720      +/-   ##
==========================================
- Coverage   89.42%   89.41%   -0.02%
==========================================
  Files         207      207
  Lines       42883    42891       +8
==========================================
+ Hits        38347    38349       +2
- Misses       4536     4542       +6

Files Changed
Coverage Δ

crates/oxc_query/src/properties.rs
54.54% <25.00%> (-0.51%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


fix copy/paste error

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (42dcea4) 91.19% compared to head (7a3aacc) 86.10%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2446      +/-   ##
==========================================
- Coverage   91.19%   86.10%   -5.09%
==========================================
  Files         260       31     -229
  Lines       15753     1713   -14040
==========================================
- Hits        14366     1475   -12891
+ Misses       1387      238    -1149

Flag
Coverage Δ

jobs_cache_maintenance
?

jobs_mongodb_migration
?

libs_libapi
87.10% <ø> (?)

libs_libcommon
?

services_admin
?

services_api
85.99% <ø> (ø)

services_rows
82.95% <ø> (ø)

services_search
?

services_sse-api
?

services_worker
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Change "System" to "Project"

Requested by @brettfiedler today.

To do this right we should update code references, server API, and database to use project terminology.

"System" has been renamed to "Project" throughout, including the db columns. I did some testing for creating/deleting/saving/loading projects and they everything seemed to work OK. The old projects are still available for use. Closing.

This changed was eagerly merged into main because we only have one db. It would be so nice to have a dev db for this kind of thing.


Add WebRTC sample app using OpenWebRTC

Add a sample app with WebRTC capabilities using OpenWebRTC: https://github.com/EricssonResearch/openwebrtc
Would it be a new app or added functionality to the existing one?

Addressed by https://github.com/matrix-org/matrix-ios-sdk/pull/9

OpenWebRTC can be added to a matrix app thanks to https://github.com/matrix-org/matrix-ios-openwebrtc-wrapper
The pod is MatrixOpenWebRTCWrapper

Very nice! Great work.


mnist tests

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes # (issue).
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @williamFalcon! Thanks for opening this PR.

In the file tests/models/test_base_templates.py:

Line 12:46: E231 missing whitespace after ':'

Do see the Hitchhiker's guide to code style


[PHPStanRules] Clean up PreventDuplicateClassMethodRule: no need $reportedClassWithMethodDuplicate

Thank you :+1:

Thank you :+1:


No way to substitute JsonSerializationException by custom exception in CSharp client

Last 4 hours I've been struggling with the question of how to substitute JsonSerializationException by custom exception. Please correct me if I'm wrong but it seems that UpdateJsonSerializerSettings() doesn't help since there is no way to handle serialization in JsonSerializerSettings.Error handler and rethrow a new one from that context at the same time. I hope there is a workaround w/o implementing/changing the current template for C# client. All my attempts to put the exception handler failed - I just don't see a way to substitute hardcoded
var content_ = new System.Net.Http.StringContent(Newtonsoft.Json.JsonConvert.SerializeObject(entry, _requestSettings.Value));

Newtonsoft.Json.JsonSerializationException is not desired since I'm working on public API. What do you do to handle/translate such exceptions?

This is the type that is generated by Newtonsoft.json. If you catch such type in your client, it means that the server returns invalid data according to the spec, and maybe you should fix the spec.
Otherwise, I'm pretty sure there is no option to change that, and you would need to override the template to catch/retrhow what you want.

@jeremyVignelles thanks for the reply, but this exception also can be thrown if some request parameter has null or doesn't have some property that must be there or should not be nullable. I'd not say it's an edge case since parameters come from the caller side and out of my control. Yes, I'm inclining towards changing the template and creating a PR if this is really impossible with the current implementation.

if some request parameter has null or doesn't have some property that must be there or should not be nullable

That's called "Breaking the contract".  Or maybe I missed your point.

Yes, I'm inclining towards changing the template and creating a PR if this is really impossible with the current implementation.

You can override the template yourself in your project : https://github.com/RicoSuter/NSwag/wiki/Templates

That might be cleaner, but that's a breaking change for code that catches specific Exception type

As a proposal I'd like to mention is to introduce a way of adding an exception handler to the class template by allowing to specify a catch block between existing try and finally blocks.

if some request parameter has null or doesn't have some property that must be there or should not be nullable

That's called "Breaking the contract".  Or maybe I missed your point.

Yes, I'm inclining towards changing the template and creating a PR if this is really impossible with the current implementation.

You can override the template yourself in your project : https://github.com/RicoSuter/NSwag/wiki/Templates

Having the following contract doesn't leave a room for manoeuvre when I don't control a component which consumes my generated .NET client:
public class FooDTO {
  [JsonProperty(Required = Required.Always)]
  public string Bar { get; set; }
}

public void MyPostMethod(FooDTO object){...} //method my .NET client provides

For API consumer to catch  Newtonsoft.JsonJsonSerializationException means to be tightly couples with that JSON library which sometimes is not desired.
Thank you for pointing me out to the template documentation, however I don't see a way to override non-empty templates such as class template, so it seems for me useless to achieve what I'd like. I guess I might be missing something though.

The Generated JsonProperty is a tight coupling anyway, but I agree that catching a JSON Exception is probably not a good idea.
Why is your property generated as Required.Always? Do you have control over the server code?

The Generated JsonProperty is a tight coupling anyway, but I agree that catching a JSON Exception is probably not a good idea.

If I had access to generated client implementation I'd have throw ArgumentException in this case.

Why is your property generated as Required.Always? Do you have control over the server code?

That's because .NET client is generated from OpenAPI 3.0 specs where that field is required.


SAS Job Fails due to possible error in airflow

[2023-10-15T20:02:01.265+0100] {logging_mixin.py:120} INFO - Waiting for Job to run to completion...
[2023-10-15T20:17:50.735+0100] {logging_mixin.py:120} INFO - HTTP Call failed with error "('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))" from /jobExecution/jobs/e6335a5e-dd62-4498-a14d-bea035299caf/state. Will set jobstatus=unknown and continue checking...
[2023-10-15T20:17:33.840+0100] {base_job.py:232} ERROR - LocalTaskJob heartbeat got an exception
Traceback (most recent call last):
And then a lot of trace info, please see the log for the details
[2023-10-16T02:16:44.084+0100] {taskinstance.py:1401} INFO - Marking task as SUCCESS. dag_id=GDW_PostETL, task_id=Check_Scheduled_SourceSystems, execution_date=20231015T010500, start_date=20231016T010504, end_date=20231016T011644
[2023-10-16T02:16:45.115+0100] {standard_task_runner.py:100} ERROR - Failed to execute job 595174 for task Check_Scheduled_SourceSystems ((psycopg2.errors.DeadlockDetected) deadlock detected
DETAIL:  Process 1879381 waits for ShareLock on transaction 155630385; blocked by process 1845775.
Process 1845775 waits for ShareLock on transaction 155630384; blocked by process 1879381.
Looking at the two DAG-logs attached I see the following reference:
"Background on this error at: https://sqlalche.me/e/14/e3q8":

OperationalError
Exception raised for errors that are related to the database’s operation and not necessarily under the control of the programmer, e.g. an unexpected disconnect occurs, the data source name is not found, a transaction could not be processed, a memory allocation error occurred during processing, etc.
This error is a DBAPI Error and originates from the database driver (DBAPI), not SQLAlchemy itself.
The OperationalError is the most common (but not the only) error class used by drivers in the context of the database connection being dropped, or not being able to connect to the database. For tips on how to deal with this, see the section Dealing with Disconnects.
This indicates an error in the underlying metadatabase that Airflow uses (typically MySQL or Postgress). See https://towardsdatascience.com/sigterm-signal-fix-airflow-486ab704b126 for how to tweek the configuration of Airflow (requires Medium membership to view).

@twon007 , could you please provide more context to explain how this is related to custom steps in SAS Studio? Issues related to the SAS Airflow Provider would need to be reported as a GitHub Issue on https://github.com/sassoftware/sas-airflow-provider.
From the info provided, am I correct in guessing that you have an Apache Airflow DAG that uses a SAS Airflow Operator (which of course executes SAS code on a SAS server). The SAS-provided Airflow operator tries to retrieve a status by calling the /jobExecution/jobs//state endpoint as shown in the partial dump of the log you included. It seems that while the operator performs that call, something goes wrong.
All of this indicates to me that this is an issue with either Apache Airflow and/or SAS Airflow Provider. If you agree with my guess and assessment, then once you have created a GitHub Issue in that repo, please post the URL for that issue in the comments of this thread ("to close the loop" 😃 ). I will then close this specific issue.

@twon007, do you have an update on this?

Closing. This does not seem to be SAS Studio Custom Step related and instructions have been provided to explain where to report SAS Airflow Provider related questions.