Google Drive Upload - Last Chunk Fatal error: Uncaught exception 'Google_Service_Exception' with message 'Failed to parse Content-Range header.'

I am having problem with resumable upload. I am uploading a video to my drive. The last chunk size for my upload always produce "Fatal error: Uncaught exception 'Google_Service_Exception' with message 'Failed to parse Content-Range header."
Almost 99% of my upload is sent but every time the  upload is about to finish, the error always shows up. I have tried formats like .mp4 and .mkv file with different sizes; 800MB,1.5Gig,3Gig,4Gig but same thing happens.
I examine the Response Header and  noticed the server returns error code 400. The "[X-GUploader-UploadID]" header returns 2 element with the same value which I think is part of the resume URI.
Also, the "Range" and somewhat "md5" like content is missing for the last chunk upload.
Please enlighten me.
Here is the upload code
$resumeUri = "https://www.googleapis.com/upload/drive/v3/files?uploadType=resumable&upload_id=AEnB2Uqcn7Z4JGrdwP8PaivzBjLzlix5pzGZFLJaaRqaYD6vnWAufOTDw5Ptch8CymX3s_lgXH3aCWOdTKzeyFwstpjhZiDiO_c6HxTQA4EybFkQvv9tggw";

$file = new Google_Service_Drive_DriveFile();
$file->title = "Big File";
$file->setName($fname);
$file->setParents(array($parentFolder));

$chunkSizeBytes = 256*1024*20;

// Call the API with the media upload, defer so it doesn't immediately return.
$client->setDefer(true);
$request = $service->files->create($file);
// Create a media file upload to represent our upload process.
$media = new Google_Http_MediaFileUpload(
  $client,
  $request,
  $mtype,
  null,
  true,
  $chunkSizeBytes
);

$status = false;
$handle = fopen($filePath, "rb");

$chunckTotalSent = 0;
$fileSize = getFileSizePerl($filePath);
$media->setFileSize($fileSize);

$resChunkSize = $media->resume($resumeUri);
while (!$status && !feof($handle)) {
  if ($client->isAccessTokenExpiredInNSeconds(120)) {
      $credentialsPath = expandHomeDirectory(CREDENTIALS_PATH);
      print "/////////////////////////////////////////////////////////\n";
      print "[   REFRESH TOKEN USED TO FETCH ACCESS TOKEN  IN       ]\n";
      print "/////////////////////////////////////////////////////////\n";
      $rt = TOKEN_REFRESH;
      $client->fetchAccessTokenWithRefreshToken($rt);
      // $client->fetchAccessTokenWithRefreshToken($client->getRefreshToken());
      file_put_contents($credentialsPath, json_encode($client->getAccessToken()));
      $accessToken = $client->getAccessToken();
      $client->setAccessToken($accessToken);
  }//end else

  print "File name : ".$fname." - ".$mtype."\n";
  print "Before fread\n";
  $chunk = fread($handle, $chunkSizeBytes);
  print "Before next_chunk\n";
  $status = $media->nextChunk($chunk);
  print "After next_chunk\n";

  $chunckTotalSent += $chunkSizeBytes;
  $progressSize = $media->getProgress();
  $percentComplete = ((100/$fileSize)*$progressSize);
  $fileContent =
  " %Completed Upload : ".$percentComplete." \n".
  " File size : ".$fileSize." \n".
  " File Path : ".$filePath." \n".
  " MIME Type : ".$mtype." \n".
  " File in Bytes Sent : $chunckTotalSent \n".
  " Progress ".$progressSize."\n".
  " HTTPResultCode ".$media->getHttpResultCode()."\n".
  " File Size Sent ".$chunckTotalSent."\n".
  " Resume URI ".$media->getResumeUri()."\n".
  " ============================================================\n\n";
  //file_put_contents($assocFilename, json_encode($fileContent));
  print $fileContent;
 }

// The final value of $status will be the data from the API for the object
// that has been uploaded.
$result = false;
if($status != false) {
  $result = $status;
}

fclose($handle);
// Reset to the client to execute requests immediately in the future.
$client->setDefer(false);

PS.
$client->isAccessTokenExpiredInNSeconds($seconds) is the custom method I made. No other custom
methods aside from that.

In my case, I was upload a directory :face_with_head_bandage: , something like this:
$gdrive->resumableUpload('/home/myuser/Downloads');

@kocoten1992 and how did you fix it? I mean how did you resolve upload the document in the end?

it was thank to you @levieraf , my code is similar to your, and you said it work, so I take a step back and realize my mistake.
The error message was no helping though.

@kocoten1992 ohhh nice! well, remember to bring me a coffee xD!

Im having the same issue

Got here from a different google product (storage, python to the json api) but the api is pretty much same. My error was resolved by not declaring the total size of the file being uploaded i.e Content-Range: bytes 0-148576/*


Use rendered rather than emitted CSS in AMP mode

Is your feature request related to a problem? Please describe.
AMP pages have a strict (and somewhat arbitrary) limit on the amount of CSS a page can include. They also provide a guarantee that doesn't exist in non-AMP mode, namely that if condition is false at SSR time...
<script>
  import Foo from '$lib/components/Foo.svelte';

  export let condition;
</script>

{#if condition}
  <Foo/>
{/if}

...we can safely omit the styles emitted by Foo.svelte.
It would be nice if we could take advantage of that and only include styles for components that were rendered, to increase the likelihood of staying beneath the 75kb ceiling.
Describe the solution you'd like

Configure the Svelte plugin to not emit CSS in AMP mode
Use rendered.css instead of inspecting the module graph

How important is this feature to you?
It depends on the outcome of the research we plan to conduct on how much traffic our app would lose if we told AMP to bugger off once and for all.

I'm not too bothered by it, I just had to Google about it today when I noticed amp in the code

@Rich-Harris I think its absolutely relevant to also think about adding an option for AMP to output AMP compliant HTML to a different folder, apart from the actual HTML. I've given an example in this issue #1308. Also, how can we handle CSS when we use preprocessor, I've described the problem that I faced when I'm using tailwindCSS in this issue #1310

Is there any reason to use AMP anymore? Google no longer prioritizes it in search results: https://www.lafoo.com/the-end-of-amp/. I would think you'd be better off not using it and that we'd be better off to drop support for it

I hope they ditch AMP too but they've recently invested in AMP stories and they are promoting that pretty strongly. I think Google will improve AMP implementation instead of just dropping it.


Issue about #29544 backport to v4 docs

Hi.
Recently, you have shipped #29544 and have backported it to v4.x documentations in project v4.4.0.
In file /site/_includes/docs-navbar.html, at line 18, the expression {{ .Site.Params.icons }}/ does not generate correct href for new icons hyperlink in navigation bar of v4.x docs.
It should be {{ site.icons }}/.
So. please consider it.
To ensure, get and build my forked repo.

Thanks, it's fixed in #29623


mgmt update acr api-version to 2022-12

Description
release request: https://github.com/Azure/sdk-release-request/issues/3781
manual code in b66792c75260c222deed68e99d5d9baa9ce01f2b to use uuid in manager, added revapi.skip=true
live tests pass
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

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


Add timestamps for video 33: Thomas Wiecki

Reference
Towards #11
Closes #23
Description
Add timestamps for video 33: Thomas Wiecki

Thanks for the PR @AnaRita93.  Confirming that I have:

[x] Added the timestamps to the video here: https://youtu.be/ZxR3mw-Znzc
[x] Marked this item as complete on issue #11
[x] Added the event hashtag at the top in the description


Add event filter to convert tooltips to richtext

Description

Type of change

[ ] Bug-fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References

Fixes #3096
How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

@sofroniewn I followed Pam's instructions and here is what came out. Feast your eyes 👀 😆

@tlambert03 added some tests and your suggestions. Could you take another look?

Perfect, I'm pretty sure the 20 minutes CI time we used for this extra commit are going to be quickly paid for by the 400ns we gain at each invocation of a tooltip.


mssql  intert data （mssql 插入数据）

Your Question
insert data to mssql , the SQL will becombined with 'OUTPUT Inserted.' , causes a conflict with the trigger.

The document you expected this should be explained

If the DML statement contains no With the OUTPUT clause of the INTO clause, the target table 'Table_Name' of the statement cannot have any enabled triggers
Expected answer

How to cancel 'OUTPUT Inserted.'

Won't include the OUTPUT Inserted. if you are using a struct has no primary key or default values fields,


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative/operations-wg-leads
/assign knative/operations-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/pkg f78a006...4a03844:
  > 4a03844 fix double close of channel (# 2575)
bumping knative.dev/caching 21a5534...9df7bb7:
  > 9df7bb7 upgrade to latest dependencies (# 670)
bumping knative.dev/networking 50ab590...e51df7c:
  > e51df7c upgrade to latest dependencies (# 703)
  > cb90216 upgrade to latest dependencies (# 702)
bumping knative.dev/eventing cb3901f...6eb4c78:
  > 6eb4c78 upgrade to latest dependencies (# 6487)
bumping knative.dev/serving dbd3fc3...8d0281e:
  > 8d0281e Update net-istio nightly (# 13222)
  > d60e5e7 upgrade to latest dependencies (# 13227)
  > d89869e Properly shutdown tracer when the work is done (# 13226)
  > b3a7557 Update net-contour nightly (# 13225)
  > 358d04e Update net-certmanager nightly (# 13224)
  > b48e0e5 Update net-kourier nightly (# 13223)

/test eventing-upgrade-tests_operator_main
/lgtm
/approve


Sprockets::ArgumentError: Asset logical path has no extension: adapters/application when I try to compile assets

I just added ember-rails to my Gemfile and generated the bootstrap with rails g ember:bootstrap -g --javascript-engine coffee. When I try to precompile, I have :
rake aborted!
Sprockets::ArgumentError: Asset logical path has no extension: adapters/application
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/asset.rb:36:in `initialize'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/bundled_asset.rb:14:in `initialize'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/base.rb:377:in `new'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/base.rb:377:in `build_asset'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/index.rb:94:in `block in build_asset'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/caching.rb:58:in `cache_asset'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/index.rb:93:in `build_asset'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/base.rb:287:in `find_asset'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/index.rb:61:in `find_asset'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/manifest.rb:211:in `block in find_asset'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/manifest.rb:257:in `benchmark'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/manifest.rb:210:in `find_asset'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/manifest.rb:119:in `block in compile'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/manifest.rb:118:in `each'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/sprockets/manifest.rb:118:in `compile'
/var/lib/gems/2.2.0/gems/sprockets-rails-2.3.3/lib/sprockets/rails/task.rb:70:in `block (3 levels) in define'
/var/lib/gems/2.2.0/gems/sprockets-2.12.4/lib/rake/sprocketstask.rb:146:in `with_logger'
/var/lib/gems/2.2.0/gems/sprockets-rails-2.3.3/lib/sprockets/rails/task.rb:69:in `block (2 levels) in define'
/var/lib/gems/2.2.0/gems/bugsnag-2.8.12/lib/bugsnag/rake.rb:12:in `execute_with_bugsnag'
Tasks: TOP => assets:precompile

This is a Rails <IP_ADDRESS> application.

Thanks for your report @GCorbel !
The generator should be fixed to generate xxx.js.coffee for Sprockets 2, not a  xxx.coffee.

The generator should be fixed to generate xxx.js.coffee for Sprockets 2, not a  xxx.coffee.

This is incorrect.
This is a ember-es6_template's bug that fixed by https://github.com/tricknotes/ember-es6_template/pull/7.
@GCorbel
I released a new version 0.4.4 including fixes for this issue :rocket:
Could you try it? If not fixed yet, feel free to reopen this issue :smile:


read the output file

Hi, how can i read the output file (.xyzd file)? it seem in binary format and i dont know how read it as a x  y  z coordinate.

Hi,
It depends on the language you are most comfortable with.
Please see this discussion: https://github.com/VasiliBaranov/packing-generation/issues/16
One option from this link is:

hexdump -v -e '4 "%f "' -e '"\n"' < packing.xyzd > centers_in_ascii.txt

Someone posted a python code here: https://github.com/VasiliBaranov/packing-generation/issues/30
Just reading the file as it is in python is:

import numpy as np
packing = np.fromfile('packing.xyzd').reshape(-1, 4)

Hope this helps!
Best Regards,
Vasili


Add part_cols to the shift function so the ordering happens per partition key value

Standard Spark functionality is missing but implemented in the inner _shift function. Adding part_cols will allow this to easily scale.

Had an offline discussion and found the request is to support groupby(...).shift().

I'm working on this.


Automatically check for Achievement diaries

Currently Ardy cloak and Explorers ring take one slot each for 2, 3 and 4. Automatically checking for what diary the player has completed will allow for 1 teleport option.
Also checking for Kandarin hard will allow for automatically selecting Seers Village teleport over Camelot teleport.

[x] Combined ardy cloak 2, 3 and 4 into one option
[ ] Combined explorers ring 2, 3 and 4 into one option
[ ] Combine Portal nexus Camelot with Catherby and add in Seers village, Rename it to Portal nexus and automatically chose what teleport to use in this order: Catherby > Seers > Camelot. If using any teleport option in the portal nexus has diary requirements we need to include logic for that.
[ ] When selecting Camelot teleport(With runes or tabs) Automatically choose the best option in this order:  Seers > Camelot, choosing Seers if the Kandarin hard diary has been completed.

May be complicated, but would be cool for the option to default to an 'backup' teleport method once all of your uses on the cape/ring have been used for the day.

It was fairly easy, but I had issues with something I'm not even sure what to describe. It was a sort of delay on fetching the information, had to be run in a different way than the rest of the code so referencing it caused issues. Looking back at it i probably could have fetched the information once and stored it in a separate variable, and reference it separate variable as a work around.
And yes, a backup option for Ardy cloak/Explorers ring would be nice.

https://static.runelite.net/api/runelite-api/net/runelite/api/Varbits.html
There is something called Varbits. I think the best way i can describe Varbits is that its player information stored on the account or something. Varbits is also the way i read the state of farming patches.

After giving it a little go I think remember why it was problematic last time.
I struggled with setupArdougneLocation(). Since it sets up all the locations at launch it can't check what Ardy diary you have done before you have logged in.
In Location.java we have this. Would it be possible to update the values of a location once to player has logged in? Same with item requirement?
public void setId(int id) {
            this.id = id;
        }

public Integer CheckArdyDiaryAndReturnItemID()
    {
        if (client.getVarbitValue(Varbits.DIARY_ARDOUGNE_EASY) == 1) {
            return (ItemID.ARDOUGNE_CLOAK_1);
        }
        if (client.getVarbitValue(Varbits.DIARY_ARDOUGNE_MEDIUM) == 1) {
            return (ItemID.ARDOUGNE_CLOAK_2);
        }
        if (client.getVarbitValue(Varbits.DIARY_ARDOUGNE_HARD) == 1) {
            return (ItemID.ARDOUGNE_CLOAK_3);
        }
        if (client.getVarbitValue(Varbits.DIARY_ARDOUGNE_ELITE) == 1) {
            return (ItemID.ARDOUGNE_CLOAK_4);
        }
        return null;
    }

I think something like this is all that's needed to return the ardy cloak ID based on diaries completed.
If this seems like a reasonable and possible way of doing it I would like to try implementing it myself as a revenge from last time >:)

Added logic for checking Ardy diary and returning cape ID based on wh…
CheckArdyDiaryAndReturnItemID() seems to return ID based on Achievement diary completed. Im not sure how to update locations to the values.

Edit: I just assumed you meant getting the information about the completion of diaries. If you were thinking about the logic of it all I guess that is some work as well.

Yeah, I was just talking about the 'backup' logic, but it was still a really interesting read. Thanks for sharing!

Yeah, I was just talking about the 'backup' logic,

Yesterday i realized it, but i figured it was just information that you potentially could need if you didn't know about it yet.
Currently setupLocation() is ran every time you press the herb run button. Should I clear the Location and Teleport before setupLocation() or will it just overwrite/ignore locations that already exist?

but i figured it was just information that you potentially could need if you didn't know about it yet.

Oh 100% it was an interesting read and useful info, thanks for sharing it.

Currently setupLocation() is ran every time you press the herb run button. Should I clear the Location and Teleport before setupLocation() or will it just overwrite/ignore locations that already exist?

This is part of the refactor I was trying to achieve. If I were approaching something like this from scratch, I would try to define the Teleport/Location bindings for each patch in some form of dataset, either in a static entity that can be loaded in at runtime (in code) or a flat file configuration like a json file that holds them all that again could just be loaded in at runtime. I build a timer app in Swift that did this and it made things really easy to define a new patch/'run', for an iOS app I sadly never released and is just sat in a repo somewhere complete and ready to ship 😂
Either way, it means that the static data is only loaded into memory and defined once centrally, the 'runs' should only then be calling in the ones it needs to build its own dynamic set of them when the button is pressed. This will become especially prevalent in #23 when the 'set' will have to be dynamically calculated at runtime, at the moment that will mean a re-definition and then at the run, where as we should aim to abstract out the definition side of it.
I still haven't quite got my head around the full end-to-end execution chain of all the classes/methods to understand properly what and where to refactor. But my gut feeling is we should be able to collapse back all the individual definitions centrally (which I did, but was met with a race condition similar to your one above) and simplify some of the entities that we have being passed around. Although doing that requires a full understanding of the above, which I don't yet have - which also may humble me on this whole idea as my idea may turn out to be complete poo with more of an understanding of how things are now 😄

I still haven't quite got my head around the full end-to-end execution chain of all the classes/methods to understand properly what and where to refactor.

Don't worry, I don't fully understand it myself yet xD While ChatGPT can write functioning code, it didn't do well with files once they got too large or multiple files, I had to ask for small code samples that isolated would do what I wanted, and then stitch several pieces together. It got a bit out of hand for someone with my experience, so it became hard for me to add in features or simplify/combining existing ones.
In #23 Would each "run button" be able to be stored and loaded as a JSON, and just pass the whole JSON according to what button clicked to a class that just loops through all the locations, and only moves to the next location once all patches in a location is completed? If the panel code reads all JSON files in a folder and generates an expandable button for a run like #23 suggests, and each Location is a "Card" we can add in to any run, couldn't we create 1 class to run it all no matter what type of run it is? Maybe predefine some code for each individual card to ensure weird teleports and highlights are dealt with properly, idk. (I'm no longer sure if this is the play, as it sounds like a lot of work. I guess this would be a rewrite of everything almost)
The more I wrote the worse it sounded, My fever is all over the place and I'm not sure how well described this is. But I might as well post it again just incase it has something going on.


Recommend default global gitignore file

[maintainer edit]
This issue is reserved for the MLH Fellowship program.
Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/get-started/getting-started-with-git/ignoring-files#configuring-ignored-files-for-all-repositories-on-your-computer
What part(s) of the article would you like to see updated?
The article recommends setting core.excludesfile and adding an ignore file to the user’s home directory.
This is unnecessary since this change in Git 1.7.12 (August 2012), which made Git default to reading the file $HOME/.config/git/ignore. Using this file is preferable as it avoids configuraiton and it’s the standard location for config, reducing bloat in the home directory.
Additional information
No response

@adamchainz Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

@adamchainz Thanks for opening this issue! I checked with the team, and this would be a great update for the docs. ✨
You or anyone else are welcome to submit a PR with this update 🔧

I’m too busy writing my own blog, relevant post.

@7ze You or anyone else is welcome to submit a PR with this fix. 💖

This issue is reserved for the MLH Fellowship program.

Xxxx

Xxx

Xxx

Xxx

not stale

Little query here. The default value of core.excludefiles is $XDG_CONFIG_HOME/git/ignore and if this file is empty of not set, then it uses $HOME/.config/git/ignore automatically.

However, in the article, it has been specified that the ignore files can be configured in Git globally.
Do we need to remove the configuration part as the default value is already set to $HOME/.config/git/ignore.?


Support to run web playground in remote env

I'm playing with it on remote host under https and there are different issues like:

ws protocol is hardcoded, although it should be wss if the page is served under https
source maps are not working in Chrome, since they expect file scheme. I can work around it by checking out locally the same version, but it is not very convenient.

Since the web playground works fine via HTTPS from https://vscode-web-test-playground.azurewebsites.net/?enter=true what else is there to do?


MINIFICPP-1806 - Use boyer_moore for extension verification

For a debug build the extension verification for libcore-minifi.so took ~1400 ms (which together with other dynamic libraries adds seconds to the startup time during development), with this change it takes ~100 ms.
Alternatives considered:

boyer_moore + custom span concat: ~150 ms
boyer_moore + ranges::concat: ~1700 ms
boyer_moore + mmap: ~100 ms (the mmap wrapper would add significant complexity)

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI results for build issues and submit an update to your PR as soon as possible.

I think the clang job should be investigated, the FileUtilTests failure might relate to this change:
double free or corruption (!prev)

99% tests passed, 1 tests failed out of 183

Total Test time (real) = 300.55 sec

The following tests FAILED:
	 27 - FileUtilsTests (Timeout)

I think the clang job should be investigated, the FileUtilTests failure might relate to this change:
double free or corruption (!prev)

99% tests passed, 1 tests failed out of 183

Total Test time (real) = 300.55 sec

The following tests FAILED:
	 27 - FileUtilsTests (Timeout)

yes it is currently being investigated, I will move it to draft until it is sorted out


refactor(mempool): simplify TransactionReference to contain only max L2 gas price

This is the only resource bound mempool logic uses, no need to overload this struct.

This change is


Extremely large error for RKF7(8)

For some reason, the error of RKF7(8) is extremely large, which causes it to adopt abnormally small time step.
E.g. Initial error in figure-8 orbit is ~ 50000, while the other three embedded RK method is ~0.5

SOLVED: Not an error. RKF7(8) should simply takes a higher tolerance.


Remove redundant rbac bootstrapping

In the "Bootstrapping the Kubernetes Control Plane" section there is a subsection dedicated to bootstrapping the RBAC defaults - namely the Apiserver to Kubelet access
https://github.com/kelseyhightower/kubernetes-the-hard-way/blob/master/docs/08-bootstrapping-kubernetes-controllers.md#rbac-for-kubelet-authorization
It is my understanding that RBAC Defaults provided by the authorizer plugin already includes those (albeit with a slightly different name)
The commit where this change was introduced is here (since v1.10.0-alpha.2):
https://github.com/kubernetes/kubernetes/commit/0682372bae44be6e54d5a716f2a795c1110fbbca
While providing an example of RBAC configuration might be a good idea, overriding an existing default configuration seems not. And while this might be e necessary setting for versions < 1.10 i think including it for more recent versions is confusing (had to do a little digging to understand if i really need to include this in my installation).

need to test again


Offer another course for pending conditions choices

Context
At the moment, the thing that we're asked to do most via support is changing an offered course for a pending conditions application.
This PR adds in this functionality to the support UI. It's currently behind a feature flag, but if we're happy with it i'll remove it pretty quickly.
The PR largely adds 3 new pages:

A search page which piggy backs from the functionality Toby previously built in the
SupportInterface::ApplicationForms::PickCourseForm.
A course option selection page where a support user can pick a course option to replace the current option with
A confirm selection page where a support user is provided guidance and asked to confirm they've read it.  They are also expected to input the Zendesk ticket where it was requested

Changes proposed in this pull request
Application form show page on the support UI

show Change offered course link for pending conditions application choices

Search page

Choose course option page

Confirmation page

Guidance to review
Probably best per commit, but i do break the controllers out later so bear that in mind.
Probs best to test it locally.
I've kept the validations pretty light touch. For example you can offer the same course etc. Happy to add stuff in, but as it's done via the support UI i'm not convinced it's needed.
Link to Trello card
https://trello.com/c/lFU7kZwN/3336-enable-course-to-be-changed-after-offer-acceptance
Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] This code does not rely on the addition/removal of Azure config environment variables in the same Pull Request
[x] New environment variables have been added to the Azure config

Just an open question - will/should there be any checks in place for what course can be offered in place of the existing one?

SupportInterface::PickCourseForm doesn't appear to be scoping by provider. Is that acceptable this cycle? Historically I don't think course codes have been unique, just unique per provider.

Yea this is an interesting one. Some courses do have the same code. a while ago i added the provider to the radio buttons to mitigate this issue as two courses have the same code and name.
If we want to make this error prone, we could do something like
@pick_course.course_options_for_provider(provider_id: @application_choice.provider.id)
then add a method which is similar to the course_options method, but scopes the courses to something like
Course.where(provider_id: provider.id).or(Course.where(accredited_provider_id: provider.id)

@davidgisbey That sounds like a good approach to me. Choosing a course from a different provider is something we actively don't want in this scenario, so having the form code match that business rule seems sensible.


Bejeweled

PR Description 📜
Fixes #2338

Mark the task you have completed ✅

[ x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @akankshachanana1 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @akankshachanana1 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


[hub-mirror] 请求执行任务

{
"hub-mirror": [
"ghcr.io/digininja/dvwa:latest"
]
}

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
Yogendra96, @tommy-ca, @revoltez, @alexgutjahr, @xiaoluoboding, @omotcha, @whunmr, @CCCCoder, @Selich, @kaksv, @DragonnZhang, @wafeishushu, @Faiznurullah, @apoorvlathey, @AdirAmsalem, @antirek, @tsujp, @kc97ble, @flumpus-dev, @drewstaylor


Static theme types + utilities

Overview
Implements static interfaces based on the current versions of our defaultTheme and classicTheme objects, which should alleviate some of the pain described in #1209
While we would like to export a utility for generating interfaces for any generic theme object, we aren't quite there yet. This should act as a holdover for the time being.
Additionally, I added some other types/utilities:

useTheme is now generically typed, which will default to DefaultTheme if not explicitly set.
Exported a mergeTheme function which is really just a wrapper around _.merge which should provide consumers an easy way to add/update the default theme as needed, since it does recursive merging unlike Object.assign or having to do multiple ... operations on the whole theme object.
Exported a getThemeContext function which returns a version of the ThemeContext which can be typed to a custom theme interface

Screenshots (if applicable)
mergeTheme util with proper implicit typing:

useTheme with default type set to DefaultTheme:

Explicit typing:

Factory function for returning a typed context:

Documentation

[x] Updated Typescript types and/or component PropTypes
[ ] Added / modified component docs
[ ] Added / modified Storybook stories

Hey, whats the state of this. This would be very nice to have :).

Hey, whats the state of this? This would be very nice to have :).

Hey @anuejn! I was hoping to get through this hard-coded version as a holdover, but we wanted to take the time to get the TypeScript types correct to more generically represent the shape of a theme (both the default theme and how it can be extended). I'm working on that in another branch but I don't have an ETA for when it'll be ready. Since evergreen is not natively written in TS, writing the definitions and making sure they are accurate (especially with such a complex object with dynamic props) is clunky and time consuming.
In the meantime, the hard-coded version of DefaultTheme can be copied into your local project for reference/overrides. You'll probably have to do some casting, but it was generated from running JSON.stringify(defaultTheme); so it should be reasonably accurate.

Okay, thanks for the the quick response. For now I will vendor the code from this branch into said project but I am thrilled to integrate the proper version when it is ready :)

@anuejn This is available in v6.7.0. Try it out and let me know if you run into any issues!

works for me :) (see https://github.com/audapolis/audapolis/pull/227 if you are interested)

Actually one thing would be quite nice for our use case: The ability to put arrays of strings into the colors section of the theme.
This is however nothing that is too important as we can work-around it by creating an object with integer keys for now (for an example use-case, see https://github.com/audapolis/audapolis/pull/227/files#diff-5dd9dde26bd45ecf6de8d7ee6a6a78ba430629ed9c57466fecf746b8d04a2f95R23).
However this already helps a lot and is a very nice solution to the typing problem :)

Ooh, interesting! We haven't had a use case for a set of color values without specifically named keys yet, but as long as it doesn't muddle with the typing of the string or nested object typing, I think we can probably widen that field's type 👍


buildNpmPackage: suggest to check if upstream lock files actually match

Description of changes
I have packaged two with with buildNpmPackage yet and both times I had the luck to package a project which lockfile was out of date.
Helps to fix the issue in #178737
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@lilyball friendly ping

Perhaps you meant to ping @lilyinstarlight?


Why is Newtonsoft.Json mentioned when discussing self-contained deployment with third-party dependencies?

Hello,
I'd like to know why Newtonsoft.Json is mentioned in the step 2, regarding self-contained deployments with third-party dependencies. Is that required in order to do a .NET Core self-contained app? When reading the phrase "with third-party dependencies", I think of Nuget packages that my .NET Core app depends on, is that what you meant here? If so, then Newtonsoft.Json is just an example and not a requirement. I think this should be explicitly written because it sounds confusing as is currently written.
Please clarify.

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

ID: 50511188-600e-18f6-6d92-c93b135d3db6
Version Independent ID: 559262e6-f27b-e581-255f-0967b006a922
Content: Deploy .NET Core apps with Visual Studio - .NET Core
Content Source: docs/core/deploying/deploy-with-vs.md
Product: dotnet-core
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

Yes, Newtonsoft is just an example and you're correct that we could be more explicit about that. Adding this to our backlog.


XLSX from-function reads only 42(!) columns

To be checked: there are some XLSX files, where Alasql XLSX-from function reads only 42 columns.

Fixed in version 0.0.40 (see test242).


Provisioner does not store shoot DNS configuration in the database

There was an extension of an input Gardener cluster configuration merged in PR 970  where DNS configuration was added to GraphQL GardenerConfig input struct to add support for DNS providers and custom domain names for provisioned shoots.
While working on fixing other problem I found that Provisioner does not preserve this DNS configuration used to create shoot cluster in its database.
For details please check the code that inserts cluster configuration into database:
https://github.com/kyma-project/control-plane/blob/main/components/provisioner/internal/provisioning/persistence/dbsession/write.go#L97
You can see that we do not store DNS config.
We should make sure DNS config is stored in the database.

This bug turned out to be rather low priority and it has zero impact on shoot upgrade functionality.
We have removed this issue from current sprint.  The branch with code can be found here

Workplan:

[ ] Rebase code with main branch
[ ] Uncomment commented code, make it build
[ ] Check if  unitttests work fine for DNS config in file components/provisioner/internal/provisioning/input_converter_test.go
[ ] Modify components/provisioner/internal/provisioning/graphql_converter.go add processing DNS config for output (like OIDC config)
[ ] Add unittest for for DNS config in file components/provisioner/internal/provisioning/graphql_converter.go
[ ] Add migration step in the database
[ ] Manual test if stored DNS config is retured by RuntimeStatus mutation


Type error with React 18 and TypeScript

Sample code
import ReactAvatar, { ConfigProvider } from "react-avatar";

const allowedColors: string[] = ["#ffffff", "#000000"];
export default function App() {
  return (
    <div className="App">
      <ConfigProvider colors={allowedColors}>
        <ReactAvatar name="Egor Koldasov" />
      </ConfigProvider>
    </div>
  );

ConfigProvider throws an error
Type '{ children: Element; colors: string[]; }' is not assignable to type 'IntrinsicAttributes & ConfigProvider'.
  Property 'children' does not exist on type 'IntrinsicAttributes & ConfigProvider'.

Sandbox example
https://codesandbox.io/s/heuristic-wing-jt95g8?file=/src/App.tsx
Example versions
"dependencies": {
    "react": "18.2.0",
    "react-avatar": "5.0.3",
    "react-dom": "18.2.0",
  },
  "devDependencies": {
    "@types/react": "18.0.25",
    "@types/react-dom": "18.0.9",
    "typescript": "4.4.2"
  },

I have some issue

@Egor-Koldasov did u manage to solve this issue ? any help will be appreciated !

I have same issue

Same issue here

For future readers : You can fix this lint error with a comment.
ReactDOM.createRoot(document.getElementById('root') as HTMLElement).render(
  <React.StrictMode>
    { /* @ts-expect-error : ConfigProvider has no children property in interface declaration */}
    <ConfigProvider colors={AVATAR_COLORS}>
      <App />
    </ConfigProvider>
  </React.StrictMode>
)

@renebigot that is not a "fix", that is a workaround.


Update Apple Device Scouting queues

Per customer request, we need to update the OS and XCode versions in the osx.amd64.appletv.scouting.open and osx.amd64.iphone.scouting.open queues.
Desired OS - macOS 14 (latest release)
Desired XCode - 15.4 (*.xip file upload to netcorenativeassets)

Release Note Category

[x] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description
Updates to AppleTV and iPhone scouting queues

fyi: @ivanpovazan

This has been completed. We will be performing a larger update to our Mac infrastructure after .NET 9 releases.


Training settings

Hello, I've read your paper recently. In this paper, you mentioned that the training epoch is set as 2, 5 and 2, but the source code is set 150. I want to know which is correct? Thank you!

Hello, sorry to bother again. In this paper, you show the results finetuned on the COCOstuff dataset. I want to know how to finetune the model on the COCOstuff dataset for ImageNet dataset. I don't know if I'm right, but I thought of the following steps:
(1) Train the model on the ImageNet dataset.
(2) Finetune the model from (1) on the COCOStuff dataset with a smaller learning rate.
Maybe I was wrong, but can you provide me some detailed information, such as finetune learning rate and some other parameters?


More docs!

Hi! Could you update the documentation to include examples that work and perhaps information about what the different flags do!

As per the usage message, check_api doesn't take any flags. The example is so simple it should work as is.
check_api is designed to be used in a CI environment, and doesn't support passing flags to the underlying validators.

Closing due to inactivity - but please feel free to reopen the issue if necessary.


(persisted) - Add generateHash option to the persistedFetchExchange

Summary

This PR adds support for a custom hash function in the persisted query exchange. This is useful for non-browser environments like react-native, where you might want to use a different crypto API (like Expo's, for instance).
Usage:
import sha256 from "my-library";

const client = createClient({
  url: 'http://localhost:1234/graphql',
  exchanges: [
    dedupExchange,
    cacheExchange,
    persistedFetchExchange({ generateHash: sha256 }),
    fetchExchange
  ],
});

Set of changes

Adds an argument to the persistedFetchExchange factory. No breaking changes.

Thanks again! This has now been published as 1.1.0 https://github.com/FormidableLabs/urql/blob/main/exchanges/persisted-fetch/CHANGELOG.md#110

Cool! Thank you both ❤


Accessing objects with arrays

I find this one really funny as well:
var obj = { prop: 3 }
obj[['prop']] // -> 3

Of course the explanation is easy, the bracket operator converts everything you pass to a string, and converting an 1 element array to string is just returning the conversion of that element to string.

Multidimensional array
var map = {};
var x = 1;
var y = 2;
var z = 3;

map[[x, y, z]] = true;
map[[x + 10, y, z]] = true;

map["1,2,3"] // -> true
map["11,2,3"] // -> true

oh god
guys, can you please send corresponding PRs?

PR created:
#39


fix: allow to_additive when translation already exists

We allow using to_additive in the case that the translation already exists in the dictionary, but the additive declaration does not exist yet.
Useful if you have already manually inserted the name to the translation dictionary
See Zulip thread

bors r+


Integrate with Pika CI for PRs

Is your feature request related to a problem? Please describe.
The ability to test package changes without the need to publish them.
Describe the solution you'd like
Pika CI enables this for all PRs. https://www.pika.dev/cicd

wow

installed need to read more about it. thanks for the suggestion :)

going to close this issue, it's installed

Looks like it is working! Thanks for installing it!
Side Note: Looks like CodeSandbox is having an issue with the URL installs right now. Locally, I had no issues. I already contacted the owners about it. https://twitter.com/stramel89/status/1159161464277667841


issue #123 Status verb documentation

Why:

No documentation existed

This change addresses the need by:

Adding documentation in usage/status.rst

#123 Fix


Reduce transforms in LocalTrajectoryBuilder.

This speeds up AddRangeData because we avoid one copy and loop over all points.

@wally-the-cartographer merge

Merge requested by authorized user cschuet. Merge queue now has a length of 1.


MS Build error when removing $(MSBuildToolsPath)\Microsoft.CSharp.targets

Dear all,
forked the code to upgrade to Orleans 3.1.6.
Current issue is that the code is not compiling:
Clustering\CassandraClusteringTable.cs(130,56) "OrleansQueries" does not contains a definition of
"DeleteMembershipEntry".
I also looked in the whole repository. There is no occurence of "DeleteMembershipEntry".
Is that code somewhow generated?
Thank you and best regards
Kwemou

Sorry my error:
the code compile as I added

in the

Please ignore the issue.
solved

Error happens when removing old targets

Hi I did a fork and I put it able to work with Orleans 3.4, also I did small changes so, please take a look https://github.com/RuiEusebio/OrleansCassandraUtils

Hi I did a fork and I put it able to work with Orleans 3.4, also I did small changes so, please take a look https://github.com/RuiEusebio/OrleansCassandraUtils

@RuiEusebio I'd ask you to create a pull request with your changes, but renaming the storage table causes backwards incompatibility.

I can revert the table name to storage again if you want and I will open a pull request

@RuiEusebio that'd be wonderful :)


RCT_EXPORT_METHOD does not support multiple arities

RCT_EXPORT_METHOD does not support multiple arities. for example
RCT_EXPORT_METHOD(track:(NSString *)event)
{
  [Reporter track:event];
}

RCT_EXPORT_METHOD(track:(NSString *)event
                    properties:(NSDictionary *)properties)
{
  [Reporter track:event properties:properties];
}

now if i call Reporter.track('bla', { a: 1 }), thats fine. but `Reporter.track('bla') complains that the method expects 2 args and i gave only 1.
do i need to make an example app for that? its pretty easy to try out.
i guess the second call to RCT_EXPORT_METHOD replaces the first one or something?
is this on purpose, or just a bug?

React Native version: 0.41.0
Platform: iOS
Operating System: macOS

You can wrap the exposed native API in a JavaScript API that accepts multiple arities and provides default values when others aren't provided.
RCT_EXPORT_METHOD(track:(NSString *)event
                    properties:(NSDictionary *)properties)
{
  // if properties is given {
    [Reporter track:event properties:properties];
  } // else if it is not given {
    [Reporter track:event];
  }
}

in js:
export default function report(event, properties={}) {
  MyModule.report(event, properties);
}


Alertview doesn't appear

I saw that some people (like me) had in some project the alert view not displayed.
I discovered (in my case) that it was because the view is attached to a wrong window (don't ask my why but my project seems to have two, i suspect another library needs it).
Si if you replace in the CustomIOS7AlertView.m the line
    [[[[UIApplication sharedApplication] windows] firstObject] addSubview:self];

by this one:
    [[[[UIApplication sharedApplication] delegate] window] addSubview:self];

The problem disappear. The question is then: Why isn't it the default behavior? Why is it better to test the first object of the window array?

I'd same problem.and then I fixed it:
to use (- (id)initWithParentView: (UIView *)_parentView;)[You had to add this method in CustomIOSAlertView.h , it's aleady exists in CustomIOSAlertView.m].
The author set it deprecated.but in some condition,you had use this method.
There is my code:
var alertView = CustomIOSAlertView(parentView: self.view)
// ....
alertView.show()

And,I think it happeds when the rootController is setted by some unsafe code.
I used KGDrawerViewController,use
centerViewController = viewControllerForStoryboardId(storyboardId)
code.
In the end .I hope author could make "- (id)initWithParentView: (UIView *)_parentView attribute ((deprecated));" enable.
And sorry for my foor english.

I had the same problem and using YSDC's approach solved my problem.


[OTHER] Fix the footer alignment

What would you like to share or ask?

I wat to fix the alignment of 'Quicklinks' in the footer.
Please assign me this issue under GSSoC extnd. and Hacktoberfest.
Checklist

[X] I have checked the existing issues.
[X] I have read the Code Of Conduct
[X] The changes don't break the code
[X] Please attach the screenshots and video of the development work once it is done

Please assign this issue to me

I really like your application and want to work on your project ,please assign this issue to me under gssoc-ext 2024


[Feature-Request] Multi-camera stereo system for large baseline setup

Start with the why:
This feature would be useful as it would allow for running multiple cameras which would assist on giving better accuracy at further distances.
Move to the what:
A script that can be used to run multiple cameras for accurate object detection/tracking in 3D spaces at large distances (wide baseline between the cameras) and match the object being detected across the devices
Move to the how:
Possibly using the RGB camera from the Oak D devices and using them find the distance of objects, user FPSychotic on the discord suggested that a rolling color sensor wouldn't help but instead suggested using the stereo cameras on the devices.
The devices would also need to be calibrated together while looking at the same screen to get the extrinsics of the cameras in relation to each other

Yeah, that could be used @Erol444


Some screenshots because our server is sooo beautiful :)

For the website!

Nice :)
Just a few more and it will be all good :D

Yep, I want to show mobs, npc, player, and so on. but not tonight :P

Nice adds for banners / galery, thank you cyber ! :)

Yeah, screenshots are beautifull !

Add two new banners :

Now, more screenshots in the Galery : http://minetestforfun.com/galerie
and i translated in english this part of the website
!(new_galery)[http://i.imgur.com/F4sssHS.png]
(PS : add a "S" to BiomeS, for the biomes banner)

Everything is Done, i closed the issue
Thank you @Cyberpangolin !


Update virtual-network-multiple-ip-addresses-portal.md

Just tweaking format of "add a private IP address" title

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

Looks good; merging.

Hi @brianlehr - This PR was opened in the public repository. The contributor guide article Public vs. private repo guidance indicates that "content developers and core product team PMs should work in the private repositories so they can address quality feedback from the quality automation present in the private repos."
We'll merge this PR, but would you make future article updates in the private repo? Small fixes like this made in the private repo qualify for auto-merge, which is more efficient than a manual merge made in the public repo.
To edit an article in the private repo from within the documentation, after you select Edit on an article page,  add “-pr” to the repo name in the URL.

Also, if you're interested, SpineEdit is a Chrome extension that opens the edit option in the private repo, if there is one and if you have access to it.
Thanks!

Related links in the internal Microsoft Docs contributor guide:
Public vs. private repos | Move a commit to a different repo (public -> private) | GitHub for content reviews
Quick reference for contributing to the docs | Make changes to a Docs article | PR review quality criteria


Few random improvements related to getting a DiagnosticWithLocation

Closed because it conflicted with a recent PR, and I could now no longer find the code that this was supposed to simplify.


[Draft] Unicode Tears 🥲

All this to say:
I think the better solution here would be to split up the API for ranged encode enabled read. I don't exactly think that if we say read read(n) where n is number of bytes, yet we have different behavior if encode is enabled only if it is ranged.
In that case we cannot guarantee it is n bytes and it is rounded up to the nearest byte boundary.
But if that's okay, then I am unsure of how we want to calculate the buffer so we can avoid the unnecessary puny network calls 😄

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


Closes #170 | Create data loader for X-FACT

Please name your PR after the issue it closes. You can use the following line: "Closes #ISSUE-NUMBER" where you replace the ISSUE-NUMBER with the one corresponding to your dataset.
Checkbox

[ ] Confirm that this PR is linked to the dataset issue.
[ ] Create the dataloader script nusantara/nusa_datasets/my_dataset/my_dataset.py (please use only lowercase and underscore for dataset naming).
[ ] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _NUSANTARA_VERSION variables.
[ ] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[ ] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one NusantaraConfig for the source schema and one for a nusantara schema.
[ ] Confirm dataloader script works with datasets.load_dataset function.
[ ] Confirm that your dataloader script passes the test suite run with python -m tests.test_nusantara --path=nusantara/nusa_datasets/my_dataset/my_dataset.py.
[ ] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

i have accepted the change requests, I must've forgotten to delete said comments and files, thanks for pointing it out kak 👍 @christianwbsn


Optimise GCSFilesystem rename implementation

GCSFileSystem implements a rename() with a copy and delete operation.
However, GCS has an Objects: rewrite which looks like it would be a metadata operation only and therefore has the potential to be much quicker.
Once BEAM-5036 is fixed IOs that write files will make use of a rename operation.
Imported from Jira BEAM-5429. Original Jira may contain additional context.
Reported by: timrobertson100.

.close-issue not_planned
Function already there. Outdated.


Handle to a Process ID?

This is confusing.  Typically a Process ID is an integer (DWORD).  So a HANDLE to a Process ID is.... the actual Process ID?  Or the handle to the process (and not the PID?).  The discussion about how to get the PEPROCESS is probably trying to be helpful, telling about a way to get it if it hadn't just been passed in.  What would be more helpful is knowing what functions can be used with the PEPROCESS.

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

ID: da277b56-3966-a7d0-cd9d-5839a1bd1c62
Version Independent ID: 94b90c09-8827-243d-e58e-4f8ef7dc2405
Content: PCREATE_PROCESS_NOTIFY_ROUTINE_EX (ntddk.h) - Windows drivers
Content Source: wdk-ddi-src/content/ntddk/nc-ntddk-pcreate_process_notify_routine_ex.md
Product: windows-hardware
Technology: wdk-api-reference
GitHub Login: @barrygolden
Microsoft Alias: bagold

@poweradminllc, thanks for your feedback on this article.
You may have success posting a question about what functions can be used with the PEPROCESS in the following forums, where other driver developers and Microsoft engineers can be found.
Microsoft Q&A: Windows Hardware development
OSR Developer Community

Reply ABOVE THIS LINE to add a note to this request
Thank you.  I can find more about the PEPROCESS, but the documentation still seems like it is incorrect.  I can't think of any case where you get a HANDLE to an ID.  This must be some sort of typo.
Doug
Power Admin LLC
Email: @.***
Web: https://www.poweradmin.com/https://www.poweradmin.com/?ref=helpspot
Support: http://support.poweradmin.com/http://support.poweradmin.com/?ref=helpspot
Phone: +1<PHONE_NUMBER>
Fax: +1<PHONE_NUMBER>
If you love our products or service, please help us get the word out!
A simple Tweet, forum post or email to a buddy helps!
Thank you!
Barry Golden - @.*** - Jan 8 2024, 12:03 PM
@poweradminllchttps://github.com/poweradminllc, thanks for your feedback on this article.
You may have success posting a question about what functions can be used with the PEPROCESS in the following forums, where other driver developers and Microsoft engineers can be found.
Microsoft Q&A: Windows Hardware developmenthttps://docs.microsoft.com/answers/topics/windows-hardware.html
OSR Developer Communityhttps://community.osr.com/
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-driver-docs-ddi/issues/1498#issuecomment-1881576567, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABNFXYZ4DQNGMZOWEGAFB2DYNQYEBAVCNFSM6AAAAAA7J3A4DWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBRGU3TMNJWG4.
You are receiving this because you were mentioned.Message ID: @.***>


added karma to "bin"

Without installing Karma globally and only as a local devDependency, we can now write something like this into our package.json and call $ npm run karma:
{
  "scripts": {
    "karma": "karma start karma.conf.js"
  },
  "devDependencies": {
    "karma": "^0.12.0"
  }
}

@donaldpipowitch Thanks
You can install karma-cli locally it should fix your problem
https://github.com/karma-runner/karma/issues/1078


Unable to scroll to bottom with different table cell heights.

Thanks for the amazing library! It is really helpful and the documentation is great. Just one issue I am having:
I have a perfectly working table that can expand the way I want it to, but I have the requirement to have different heights on certain sections, this seems to cause the table's height to not change in some way (regarding the scrollable surface of the UIView its added to).
So when I expand sections they allow me to scroll further down, that still works, but I am unable to see all the cells; as if the UIView or table is unable to scroll all the way. The cells differentiating heights are no problem, that comes out perfectly.
- (CGFloat)treeView:(RATreeView*)treeView heightForRowForItem:(id)item {
    SomeObject* levelObject = item;

    if (levelObject.isChild == YES) {
        if ([levelObject.type isEqualToString:@"fatty"]) {
            return 140;
        }

        return 50;
    }

    return 60;
}

As you can see I set the height based on the type of the object, and whether or not it is supposed to be a section header (a child or parent). So this seems to break the RATreeView and so far unfortunately I cannot solve it. Maybe someone else has.

This was not an issue with the library. The way that the CollectionViewController and its inner ViewControllers' heights are stored was the issue I did not understand. in my UIViewController self.view.frame.size.height was returning the height of the entire CollectionViewController, which I overlooked in my attempts to solve the issue.
Apologies :p


AKS Cluster configuration with OMS

Hi, we are trying to implement regulated environment based on this guidelines and we are failing to get through our Pod.
We successfully deployed almost all resources (deploying on australiaeast for some reason we cant deploy agent pools) in azure and deployed cluster manifests (cluster-baseline-settings, ingress-nginx, kube-system). Then we changed from example in cluster manifests namespace creation yamls to our naming but all resources stayed the same. Same goes with workload yamls where, for now, we deployed single aspnetcore application hosted on kestrel and exposing http on port 8080.
Our application gateway backend health is showing 502.

 I am guessing from appgw probe.

The certificate used in ingress is full chain certificate and valid because we created VM inside network and made request on our ingress in browser and got valid certificate. On appgw we used root certificate from that chain as trusted root. Appgw is not accepting anything else.
If we port forward in this scenario from VM inside network to our pod service, we get application.
Also when we try to curl ingress from jumpbox we get this response.
curl -vvLk https://<IP_ADDRESS>/dashboard
* Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
* CAfile: /etc/ssl/certs/ca-certificates.crt
CApath: /etc/ssl/certs
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.3 (IN), TLS Unknown, Certificate Status (22):
* TLSv1.3 (IN), TLS handshake, Unknown (8):
* TLSv1.3 (IN), TLS Unknown, Certificate Status (22):
* TLSv1.3 (IN), TLS handshake, Certificate (11):
* TLSv1.3 (IN), TLS Unknown, Certificate Status (22):
* TLSv1.3 (IN), TLS handshake, CERT verify (15):
* TLSv1.3 (IN), TLS Unknown, Certificate Status (22):
* TLSv1.3 (IN), TLS handshake, Finished (20):
* TLSv1.3 (OUT), TLS change cipher, Client hello (1):
* TLSv1.3 (OUT), TLS Unknown, Certificate Status (22):
* TLSv1.3 (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / TLS_AES_256_GCM_SHA384
* ALPN, server accepted to use h2
* Server certificate:
* subject: CN=*.aks-ingress.*******.com
* start date: Nov 16 00:00:00 2021 GMT
* expire date: Feb 14 23:59:59 2022 GMT
* issuer: C=AT; O=ZeroSSL; CN=ZeroSSL RSA Domain Secure Site CA
* SSL certificate verify ok.
* Using HTTP2, server supports multi-use
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* TLSv1.3 (OUT), TLS Unknown, Unknown (23):
* TLSv1.3 (OUT), TLS Unknown, Unknown (23):
* TLSv1.3 (OUT), TLS Unknown, Unknown (23):
* Using Stream ID: 1 (easy handle 0x558574169600)
* TLSv1.3 (OUT), TLS Unknown, Unknown (23):
> GET /dashboard HTTP/2
> Host: <IP_ADDRESS>
> User-Agent: curl/7.58.0
> Accept: */*
>
* TLSv1.3 (IN), TLS Unknown, Certificate Status (22):
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* TLSv1.3 (IN), TLS Unknown, Certificate Status (22):
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* TLSv1.3 (IN), TLS Unknown, Unknown (23):
* Connection state changed (MAX_CONCURRENT_STREAMS updated)!
* TLSv1.3 (OUT), TLS Unknown, Unknown (23):
* TLSv1.3 (IN), TLS Unknown, Unknown (23):
* TLSv1.3 (IN), TLS Unknown, Unknown (23):
< HTTP/2 404
< date: Tue, 30 Nov 2021 19:42:00 GMT
< content-type: text/html
< content-length: 146
< strict-transport-security: max-age=15724800; includeSubDomains
<

<title>404 Not Found</title>

404 Not Found

nginx

* TLSv1.3 (IN), TLS Unknown, Unknown (23):
* Connection #0 to host <IP_ADDRESS> left intact

Here you can see how aks-ingress certificate is valid when making request from VM inside network.

We already implemented baseline and it is working properly with traefik as ingress controller. From my standpoint the only difference that would cause this is OMS?
Here is ingress script and I can post other but its same as in workload example only with client naming.
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: frontend
  namespace: ****s
  labels:
    app.kubernetes.io/name: ****s
    app.kubernetes.io/component: frontent
    pci-scope: in-scope
  annotations:
    kubernetes.io/ingress.allow-http: "false"
    kubernetes.io/ingress.class: nginx
    nginx.ingress.kubernetes.io/backend-protocol: HTTPS
    nginx.ingress.kubernetes.io/configuration-snippet: |
      proxy_ssl_name "frontend-sa.****s.cluster.local";
    nginx.ingress.kubernetes.io/proxy-ssl-secret: kube-system/osm-ca-bundle
    nginx.ingress.kubernetes.io/proxy-ssl-server-name: "on"
    nginx.ingress.kubernetes.io/proxy-ssl-verify: "on"
    nginx.ingress.kubernetes.io/use-regex: "true"
    nginx.ingress.kubernetes.io/force-ssl-redirect: "true"
spec:
  tls:
    - hosts:
        - ****s-00.aks-ingress.***up.com
  rules:
    - host: ****s-00.aks-ingress.***up.com
      http:
        paths:
          - path: /.*
            pathType: Prefix
            backend:
              service:
                name: frontend
                port:
                  number: 8080

Any idea what would be wrong with our setup?

The guide has incomplete setup regarding OSM.
If non-default service account is used then in mesh config ingress gateway should be set with SNI of that service account.
Also missing BackendIngress resource for web-frontent.


Engine - Collision of Status Change in Aysnchronous Task updates

Ran into some performance issues in the engine. Due to our now distributed nature of task execution, there was the possibility that a Handler could report back and ask for End task, before our Start task had finished processing the start. :disappointed:
Example
In about 46 milliseconds, the Handler would complete their task, which was faster than our system processing the task.
Commence Start -> [2023-02-14 04:55:12.973]
Commence End -> [2023-02-14 04:55:13.013]
Finish End -> [2023-02-14 04:55:13.024]
Finish Start -> [2023-02-14 04:55:13.094]

I've implemented a few TaskRun locks the same as WorkflowRun locks and waiting to see if that fixes the problem. Alternatively, I can remove the canCompleteTask check from Start and instead move it to the Queue. (looking into that too).

Essentially, If an external handler takes a queued task, sends back the start task (api) and then 120 milliseconds later ends the task (api) somehow the end task ends before the start task saves. This then cause the start to save over the top of the end.
I tried task level locks but that still didn’t work, end was faster than start.
Any thoughts on how I fix?

I could implement more detailed status / phase checks prior to any save, or
Remove the async on the queue and start methods and move it to only execute the individual function async such as the system tasks as the handler we now execute async itself so maybe the saving of the status and performing checks doesn’t need to be async any longer? or
Something else

I think the problem with (1) and probably all the options is I have no way of knowing which order the request is.
Because going from Ready to Running and Ready to Completed are both valid. I.e. you can skip the Running and fail the task.

Decision: move the start and end into non async as they are external influence i.e. API calls and need to update the status and phase asyncrhonously. The System Tasks will be a new async executeSystemTask method and the finish / executeNextSteps methods will move to async for the end. SImilar to queue workflow is synchronous vs start workflow being async.


Custom ramp value is not correctly reflected on the style panel

Context
Somehow the ramp that gets into the style panel when applying auto-style in the histogram widget is not in-sync with the values applied.
Current Result
When clicking on auto-syle a ramp is applied but within the style panel another ramp is shown in the dropdown.

Expected result
Style input should reflect the same ramp that is being applied.
.carto file
Don't forget to rename .carto file to .zip file to be able to upload to github
bicimad_locations map (on 2016-12-16 at 15.09.19).carto.zip

Fixed by https://github.com/CartoDB/cartodb/pull/10420/commits/a55490317cdcaa9f7bd55ebe853eb2b81610d23e and https://github.com/CartoDB/deep-insights.js/pull/502/commits/237d8b149d8d7c754d1ffa5c02dc2a5417260f08. Heading to ded08 for acceptance.


Vehicle connected binary sensor, bug fix & deprecation resolution

Hi!
This pull request adds a new entity, fixes a bug and resolves some deprecation warnings.

Adds a new binary sensor to indicate if a vehicle is connected to the charger (Using the same logic & endpoint as the app)
Fixes a bug where some entities would become 'unavailable' and the integration need to be reloaded (Resolves #16), this was due to the charging_time sensor being set to a state_class of 'total_increasing' while also defining 'last_reset' (this is unsupported in HA)
Resolves a number of deprecation warnings for recent versions of home assistant (2024.12.1)

Im running this branch on my HA instance (2024.12.1), unit tests should be passing too. Happy to answer any questions or make changes from review

The unit tests run locally for me;
pytest -qq --timeout=9 --durations=10 -n auto --cov custom_components.eo_mini -o console_output_style=count -p no:sugar --asyncio-mode=auto tests

Would appreciate any pointers on how to get these passing in the project github 'pull' workflow.

It looks like the python version in the pull workflow was outdated. Upgraded that + home assistant and now the tests all pass in the github pipeline


Page for cascading deletes has wrong preview content

This link: https://www.prisma.io/docs/guides/database-workflows/cascading-deletes/postgresql#9-validate-the-deletion-behavior-in-a-nodejs-script
Shows the following wrong preview when unfurled:

The page <title> is also wrong.
This is weird, as the mdx file on Github has the correct content and "Import" does not appear once in that page.

@janpio @nikolasburk I think it's because the name of the page isn't unique across  the entire site - there are multiple pages named 'postgres' in different sections. Suggest changing this to a website bug /cc @nilubava

Fixed by Daniel


MJ4 mathjax-tex: Prime (′) is low and long

Issue Summary
MJ4's mathjax-tex renders prime (′) as lower and longer than MJ2.
Steps to Reproduce:

Go to https://codepen.io/alexedgcomb/pen/rNqRJgy
Observed: Prime is low and long

Expected: Prime a bit higher and notably shorter, like in MJ2:
https://codepen.io/alexedgcomb/pen/XWxwRbw

Technical details:

MathJax Version: 4.0.0-alpha.1
Client OS: Mac OS 13.2.1
Browser: Firefox v113, Chrome v113, and Safari 16.3

I am using the following MathJax configuration:
MathJax = {output: {font: 'mathjax-tex'}}

and loading MathJax via
<script<EMAIL_ADDRESS>
Supporting information:
Perhaps MJ4's mathjax-tex is working as intended.

The MathML specification considers U+2032 to be what it terms a "pseudo-script" (see section 7.2.2 of the MathML3 specification).
The recommended encoding in that section for "f prime" is to use an <msup>>, as in
<msup><mi>f</mi><mo>′</mo></msup>

which will render as you are looking for in MathJax.
There have been several discussions about this (see #2791 and this user forum post).  The upshot is that, in order to handle both <msup><mi>f</mi><mo>′</mo></msup> and <mi>f</mi><mo>′</mo>, Mathjax needs two forms of the U+2032 character, one that is larger and not raised for use in the <msup> construct (where the shrinking and raising is done by the usual process of forming a superscript), and one that is shrunken and raised for use without <msup> (where the shrinking and raising must be part of the U+2032 glyph itself).
The TeX font on which the original MathJax TeX fonts (way back in v1, which have migrated to v2 and v3) has only one size and placement of this character: the larger, un-raised version (it is the character associated with the \prime macro in TeX).  When you enter f' in TeX, it is translated into the equivalent of f^\prime, so needs the larger un-raised version, as the shrinking and raising is done by the superscript construction.  The MathJax TeX font does not have a smaller, pre-raised prime character, because the TeX font didn't have one.
While v3 (and v4) does have the infrastructure to handle both <msup><mi>f</mi><mo>′</mo></msup> and <mi>f</mi><mo>′</mo> properly, it does need to have the necessary characters in the font in order to do that.  But unfortunately, the MathJax TeX font only has one of the two needed glyphs, so can't properly render the second form.  If I recall correctly, all the other v4 fonts do have the needed characters, and certainly the mathjax-modern one does, but mathjax-tex doesn't.  It is just a porting of the original MathJax TeX font into the v4 format, so has all the limitations of the v3 (and v2 and v1) font.
Note that your v2 example works for you because your test page uses the old HTML-CSS output jax, and that jax would try to detect the presence of locally installed STIX fonts.  If found, they would be used rather than the MathJax TeX fonts in order to avoid having download web fonts.  The STIX fonts do include the two forms, and so MathJax could handle both MathML constructs when the STIX fonts are used, even back in v2.  Since MacOS ships with STIX fonts installed, the HTML-CSS output will find and use them, and so you will get a different rendering than if you had forced the use of the MathJax TeX fonts.  Because the fonts in use are different in your v2 example from your v4 one, it is not really a comparable test.
In the end, this is expected behavior for the mathjax-tex font, which has far less glyph coverage than the other v4 fonts.

Thank you for explaining, @dpvc !


Restore --docker flag backed by embedded cri-dockerd

Proposed Changes
Restore --docker flag backed by embedded cri-dockerd
Types of Changes
enhancement
Verification
See linked issue
Testing
TBD
Linked Issues

https://github.com/k3s-io/k3s/issues/5741
https://github.com/k3s-io/k3s/issues/5910

User-Facing Change
The `--docker` flag has been restored to k3s, as a shortcut to enabling embedded cri-dockerd

Further Comments

Yeah, I need to add an ADR and tests. Kinda interesting that we never had even a basic cluster startup yes for docker support...


handlers.py: Exception caught sequence item 0: expected str instance, dict found

After setting up the CloudFormation hook and creating a stack with an S3 bucket with public access, the following was logged to CloudWatch for the hook:
Exception caught sequence item 0: expected str instance, dict found
Traceback (most recent call last):
  File "/var/task/cloudformation_cli_python_lib/hook.py", line 273, in __call__
    raise error
  File "/var/task/cloudformation_cli_python_lib/hook.py", line 262, in __call__
    caller_sess, request, invocation_point, callback, type_configuration
  File "/var/task/cloudformation_cli_python_lib/hook.py", line 100, in _invoke_handler
    return handler(session, request, callback_context, type_configuration)
  File "/var/task/styra_opa_hook/handlers.py", line 127, in pre_handler
    return opa_query(request, session, type_configuration, action)
  File "/var/task/styra_opa_hook/handlers.py", line 97, in opa_query
    message = " | ".join(body["violations"])
TypeError: sequence item 0: expected str instance, dict found

For reference, the OPA agent returned the following:
{"allow":false,"violations":[{"allowed":false,"message":"public access not blocked for bucket testing"}]}

Yeah, violations is currently assumed to be a set of strings only. While we could extend this to support more response formats, I'm not sure what the added value in this case would be? 🙂 That allowed is false is already implied by the presence of violations, so also adding that to each violation entry does not seems a little redundant.
One thing that might be useful though is if we had a case for adding other types of metadata to a violation, like severity or something like that. But whatever we'd go with, we'd need to document properly what works and what doesn't.

Regardless of that — we could definitely print something more informational than a stacktrace here. Just something like the below would be an improvement:
Unrecognized format in OPA response.
Expected: {"allow": <boolean>, "violations": <array<string>>}
Got: {"allow": <boolean>, "violations": <array<object>>}


Update docs for PR #36

Updated Godotcord.init for #36
Changed int createFlags to an enum

Better squash the pull request into one commit.

Better squash the pull request into one commit.

To Squash the commits:

git pull <your-origin> <your-branch> (eg: git pull origin patch-2)
git checkout <your-branch> (eg: git checkout patch-2)
git rebase -i <commit-hash> (eg: git rebase -i 96ebf5931acc20365bd199ca80a020918fc1591c)
replace pick with squash in all commits except the first one.
git push -f

And that's it.

To Squash the commits:

git pull <your-origin> <your-branch> (eg: git pull origin patch-2)
git checkout <your-branch> (eg: git checkout patch-2)
git rebase -i <commit-hash> (eg: git rebase -i 96ebf5931acc20365bd199ca80a020918fc1591c)
replace pick with squash in all commits except the first one.
git push -f

And that's it.

I can't run these commands because I'm not using the git client. I created this PR from the web.

I can't run these commands because I'm not using the git client. I created this PR from the web.

Ok np.

Ok np.


[OPPRO-358] Enable GlutenDateFunctionsSuite

What changes were proposed in this pull request?
Just enable  GlutenDateFunctionsSuite.
How was this patch tested?
Gluten-UT

cc @zzcclp, @rui-mo

LGTM

Thanks!


Update power-bi-visualization-waterfall-charts.md

@mihart
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Migrate Xamarin Android Project to AndroidX

This pull request migrates the Android application project to AndroidX.

Ah nice catch, I was switching between VS and Rider and didn't notice these being pushed up. I've updated the .gitignore.


Title shows theme name instead of an actual title.

See title.

Gonna fix that asap

Fixed the issue


Signer Panel button and error count

Action button for signer panel.

[x] style button according to figma.
[x] enable and disable button accordingly

Figma
https://www.figma.com/file/G7Z28scDlkNFvgDyR2nSnq/Playground-Library-|-Components-(External)?node-id=298%3A117977

completed


speech-synthesis: tts-stream hint text

Accessibility hint for speech-synthesis tts-stream.
Checklist

[x] npm test passes
[ ] tests and/or benchmarks are included

Landed as 510d5079184e8e784926f6b42bd6ccc15aafa138 and e220b63d945a0a0502218bfe1dcf3fa9c1b4b0f7 🎉


GA-28 Update test definitions

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] the keep-helm label has been added, if the helm release should be persisted after a successful build

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

@patelila @helenbirdhmcts this PR is downgrading tomcat version from tomcatVersion = '9.0.82'
to tomcatVersion = '9.0.75!!' which therefore is adding the following suppressions CVE-2023-41080, CVE-2023-44487, CVE-2023-42794, CVE-2023-42795, CVE-2023-45648
Since these were fixed in master by bumping tomcat to 9.0.82, just wondering why we need to downgrade it in this PR. Can I revert that back to 9.0.82 (and therefore ignore those added suppressions)?, would this PR need a new QA ?

@patelila @helenbirdhmcts this PR is downgrading tomcat version from tomcatVersion = '9.0.82' to tomcatVersion = '9.0.75!!' which therefore is adding the following suppressions CVE-2023-41080, CVE-2023-44487, CVE-2023-42794, CVE-2023-42795, CVE-2023-45648
Since these were fixed in master by bumping tomcat to 9.0.82, just wondering why we need to downgrade it in this PR. Can I revert that back to 9.0.82 (and therefore ignore those added suppressions)?, would this PR need a new QA ?

Please revert back as I am not sure why it was downgraded. probably a mistake when merging occurred.


N20: Learning Materials

@alexheretic @mrDIMAS @philipk - r? :)
Part of #531

:timer_clock: Merging in an hour or so if there're no objections.

No objections from me, looks good, thanks!


Getting bad response for Solr

Hello eevryone
I set up different NoSQL databases on my laptop such as Elasticsearch, Solr, OrientDB and MongoDB and wanted to measure the performance of these databases using YCSB.
The problem is Solr takes too much time for each workload as shown below
[OVERALL], RunTime(ms), 98412
[OVERALL], Throughput(ops/sec),<PHONE_NUMBER>5475348
[TOTAL_GCS_PS_Scavenge], Count, 16
[TOTAL_GC_TIME_PS_Scavenge], Time(ms), 47
[TOTAL_GC_TIME_%PS_Scavenge], Time(%), 0.04775840344673414
[TOTAL_GCS_PS_MarkSweep], Count, 0
[TOTAL_GC_TIME_PS_MarkSweep], Time(ms), 0
[TOTAL_GC_TIME%PS_MarkSweep], Time(%), 0.0
[TOTAL_GCs], Count, 16
[TOTAL_GC_TIME], Time(ms), 47
[TOTAL_GC_TIME%], Time(%), 0.04775840344673414
[READ], Operations, 4959
[READ], AverageLatency(us), 1867.9526114135915
[READ], MinLatency(us), 1288
[READ], MaxLatency(us), 19551
[READ], 95thPercentileLatency(us), 2217
[READ], 99thPercentileLatency(us), 4299
[READ], Return=OK, 4959
[CLEANUP], Operations, 1
[CLEANUP], AverageLatency(us), 1672.0
[CLEANUP], MinLatency(us), 1672
[CLEANUP], MaxLatency(us), 1672
[CLEANUP], 95thPercentileLatency(us), 1672
[CLEANUP], 99thPercentileLatency(us), 1672
[UPDATE], Operations, 5041
[UPDATE], AverageLatency(us), 17593.302916088076
[UPDATE], MinLatency(us), 9520
[UPDATE], MaxLatency(us), 183167
[UPDATE], 95thPercentileLatency(us), 30591
[UPDATE], 99thPercentileLatency(us), 40287
[UPDATE], Return=OK, 5041
Plus I don't know how to make a graph throughput vs latency?
is it possible that we can make changes (or define) to the throughput like 3k or 5k or 6k  per sec. if yes then how is it possible. I read that we can define the throughput using the target parameter but I am unsuccessful while applying  this tag
-p  target 300
I am very thankful to you if you can help me regarding these issues.
Best Wishes,
Adnan

is it possible that we can make changes (or define) to the throughput like 3k or 5k or 6k per sec. if yes then how is it possible. I read that we can define the throughput using the target parameter but I am unsuccessful while applying this tag
-p target 300

The target throughput rate is specified on the cli with -target 300

Plus I don't know how to make a graph throughput vs latency?

Use one of the exporters to get machine parsable metrics and thrn use the charting mechanism of your choice.
They're covered a bit on the docs page for core properties:
https://github.com/brianfrankcooper/YCSB/wiki/Core-Properties

Closing as stale, feel free to reopen if you have more details.


Combine global timeout with per operation timeout

I believe at the moment it's only possibly to either specify a per operation timeout or a global timeout. Would be good if it were possible to combine both timeouts, e.g.
HTTP.timeout(global: 60, connect: 5, write: 2, read: 10).get 'http://example.com'
This is needed to handle servers that send unending responses or that send out data very slowly.

Hm... But global timeout is doing a bit different thing, it's basically timeout for connect + write + read. IN other words it won't solve your problem if I understand correctly.

In that case perhaps a different term is needed, e.g.

elapsed time timeout
overall timeout
total time timeout
cumulative time timeout

Closing in favor of #773


make last refresh date more obvious when it's old

some users may not realize that clicking Test Now pulls data from a cache unless they explicitly click Update now. maybe the last refresh text could be colored red if it's quite old or something

it would also be nice to have the date be relative, like "4 days ago", or "1 year ago", instead of the timestamp we have now

it would also be nice to have the date be relative, like "4 days ago", or "1 year ago", instead of the timestamp we have now

this is done now


Mark avd setup as infra step for Linux android views

Example: https://ci.chromium.org/ui/p/flutter/builders/try/Linux android views/1740/overview
An infra step should be marked to surface as infra failure.

Could you generalize the wrapper to support the flag?

@keyonghan Yes, I can do that. Thanks for the response!

Resolved with change: https://flutter-review.googlesource.com/c/recipes/+/29502


⚠ Spryr has degraded performance

In 62c5e57, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3832 ms

Resolved: Spryr performance has improved in 1c6b8c0 after 10 minutes.


Deselecting 'Create Rigidbody' does not remove component

Deselecting 'Create Rigidbody' does not remove the Rigidbody component from the game object

I guess this is a bit of a misnomer .. "Create RigidBody" is more like .. "Automatically create rigidbody"
When you deselect it, you can still add or remove your own ..

I changed this so that there is an option that's called "Auto Rigidbody" which automatically creates or destroys the rigidbody and updates it's settings. If you turn it off you can do it manually


Error during code compilation using Arduino IDE

Hello, i tried to compile it for my ESP8266
In file included from C:\Users\Michal\Downloads\opentherm-gateway-simple-main\otgw_simple\otgw_simple.ino:10:0:
C:\Users\Michal\Downloads\opentherm-gateway-simple-main\otgw_simple\src\ESPAsyncWebServer\ESPAsyncWebServer.h:26:10: fatal error: functional: No such file or directory
 #include <functional>
          ^~~~~~~~~~~~
compilation terminated.

exit status 1

hi,
can i maybe see your code? or did you fix the problem already?

For me its compiling OK targeting ESP8266 (v. 3.1.2). @Mio3000 maybe try updating ESP core from boards manager?


Errors installing Swift-Runtime-Support

Hello. When I try to install xcbeautify the first step
brew cask install thii/swift-runtime/swift-runtime
My CircleCI build fails with the following error:
==> Purging files for version latest of Cask swift-runtime Error: Failure while executing; "/usr/bin/sudo -E -- /usr/bin/env LOGNAME=distiller USER=distiller USERNAME=distiller /usr/sbin/installer -pkg /usr/local/Caskroom/swift-runtime/latest/SwiftRuntimeForCommandLineTools.pkg -target /" exited with 1. Here's the output: installer: Package name is Swift 5 Runtime Support for Command Line Tools installer: Certificate used to sign package is not trusted. Use -allowUntrusted to override. ...
I guess it seems the certificate you use has expired or am I missing something here? Any help would be greatly appreciated!

Are you using macOS 10.14.3 or earlier? I think installing it shouldn't be needed on CircleCI as they have newer machines.

Using MacOS 10.13.6 so it is not working without the switft-runtime. Tried that one as well. Oddly it worked a few months back to install the swift-runtime when I did a POC.

Just tried downloading via curl and installing it but the package seems broken. We will try to update the MacOS version in our CI to fix that Issue. Thanks


Strings don't go back to being untranslated during reload when translation is removed

Removing all instances of a translation and then reloading doesn't remove the translation in-game without a scene change.
This might be a hangover issue from #8 . Everything else got fixed but this remains.

Not related to #8, nor is this a bug.
Why would you even want to restore strings you actually want to replace in the first place?
If you use EnableStringReload, that means you want to edit string translations and preview the changes live. That also means you already have the original string -- which you obtained by dumping assets or other means -- in the translation file. As a translator, your work process is most likely as follows

Translate the string and view the changes in-game
Fix the existing translation and preview the typesetting in-game
Move onto the next string

Given that you have the original string in the text files (and your task of replacing it with another string), there is no reason to ever want to restore the original string on the fly.
This is the reason why UTP nor TP implement such a feature.
All this rambling notwithstanding, I'll add the feature, since it requires only a small change.

Sorry about that, m8.
It's just, sometimes the game uses extremely similar looking strings to refer to the same thing but in different places, like this example:
シニヨン＆ロール <-For the hair in Maid Edit
シニョン＆ロール <-For the hair in the Shop
And I find untranslating one as an effective way to see which is which. I could normally just add a random letter or number at the end of the translation to see which is which but, and this might be a separate bug, i'm not sure, sometimes even different looking strings, when translated into something similar, become "linked" and any change on one affects the other too, like for example:
身長 <-on the little black stat card, in Maid Edit
全身 <-In the body category, in Maid Edit
If translating both to Height, any change on one after the fact affects the other, like adding a 1 to the end of one translated string will also add the 1 to the other line, even if the translation for the other doesn't have the 1.

The issue you mention is separate, but I shall address it here since it's not worth creating a new issue for.
That's because YAT (and UTP/TP) caches the translated strings as a translated-to-original dictionary -- the "original" being the latest instance of the untranslated string that was replaced.
When YAT attempts to retranslate the visible strings, it uses the dictionary to restore the translated text to its original form. Thus, if the same level contains multiple strings that were translated into the same string, the get restored to the same form. YAT (nor any other translation loader) cannot distinguish these kind of special cases, as it would require the game to provide some additional metadata for each single string.
TL;DR: YAT caches strings, but cannot be a perfect bijection because game. You'll have to live with it.

It's just, sometimes the game uses extremely similar looking strings to refer to the same thing but in different places, like this example:
シニヨン＆ロール <-For the hair in Maid Edit
シニョン＆ロール <-For the hair in the Shop

Then use level indices to distinguish between them. YAT (and pretty much any other plug-in) logs current level index into the console. Moreover, if the same string with the same translation is present in multiple levels, you should put them into a single file and tag it appropriately (e.g. translation.15-16-19.txt to enable the same translations for levels 15, 16 and 19).

Yeah, I had a feeling that it might not be a bug. Thanks for clearing that up, i'll be mindful of that onward.
I've been using level indices since we spoke on the Hongfire forums, no worries there. In hindsight I realize that was a bad example as they occur on .5 and .8, a more apt example would probably be:
特定のイベントを3種類見ている <-Daily Activity
特定のイベントを３種類見ている <-VIP Events
Both occur on level 24 and seem to only differ by the 3 in the string. It's not so much the level I have issues with, i'm just a bit anal and obsessive about making sure that everything is as proper as I can make it, so I end up triple checking myself just to be sure. Seeing one untranslate during a reload instead of having to worry about whether it's just the dictionary matching them really helps me out.
I've so far managed to completely translate the UI and items in the game, and i'm now starting to translate the dialogue. Your plugin and expertise are the main reason i've been able to figure out as much as I have, thanks again.


Ability to override getattr

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
class Foo(BaseModel):
    field: str = "default"

    def __getitem__(self, item):
        print("getitem")

    def __getattr__(self, item):
        print("getattr")

    def __setattr__(self, item, value):
        print("setattr")
        super().__setattr__(item, value)

f = Foo()
f["field"]   # prints "getitem"
f.field = 2  # prints "setattr"
f.field      # no output?

I could add RootModel (or some custom wrapper) but then I'd lose typing capability (accesing model.field where field would be proxied to model.root.field but wouldn't exist as explicit type).
What's the way to override __getattr__ in Pydantic V2? All the references I found suggest this worked in V1, but how about V2 then?
(Asking for https://github.com/pydantic/pydantic/discussions/9846, if anyone's curious.)
Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[X] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[X] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

So, __getattribute__ actually works. I'm super duper sure I tried also that before, but uh, I guess I did something wrong back then.
So, this ticket can be closed.

So, __getattribute__ actually works. I'm sure I tried also that before, but uh, I guess I did something wrong back then.


Phantom Read definition inconsistencies

Will improve the definition -> though I need to read a bit more (SELECT .. WHERE ... changing implies a tuple was changed -> repeatable read, but the bag of tuples returned has changed -> phantom read? This is a hard one)
I will overhaul transactions post-exams and get official answers on this.  Currently the definitions feel hazy at best.