Compare microkernel benchmark IR against main for PRs

It would be useful to be able to detect whether or not the generated IR (Triton IR or TTGIR) changes for a given pull request (code change). For example, if the IR changes for a particular microkernel then we should have high suspicion for performance changes for that microkernel. Performance measurements are often noisy, whereas IR changes are essentially binary. This would also be useful for when we sync openai commits or make changes to the pass pipeline and want to know all the places that could be affected.
The proposal here is to implement a CI job that runs on every PR and compiles the Triton IR for the microkernel benchmark to TTIR/TTGIR using triton.compile. To keep things simple and keep runtime down, we will use the nightly wheels for our "golden reference". A suggested flow is:

Create script which runs the microkernel benchmark IR. Perhaps we can import this from the microkernel benchmarks directly?
Get latest Triton nightly wheels and run the microkernel benchmark IR script to generate TTIR/TTGIR for each microkernel. Note that a given microkernel benchmark may actually generate multiple TTIR/TTGIR. We can start with 1 though and improve as needed.
Run the same script as part of the build_and_test pipeline on the rolling driver.
Diff the IRs.

Note that many PRs will be expected to change the IR, so we do not want the job to fail if the IR differs (otherwise we will have a lot of red PRs that are actually fine, and merging might be blocked). But, it would be nice to have some way of notifying the user when the IR differs instead of having to remember to check the output of a given github actions run.

The motivation of this issue is good, we want to have a way to quickly identify if performance impact is variance or not.

Note that a given microkernel benchmark may actually generate multiple TTIR/TTGIR. We can start with 1 though and improve as needed.

What's your expected behavior when there are more than 1 TTIR/TTGIR?
In general, I worry that this approach may be too fragile, and end up reporting differences in most cases, and then developers start to ignore the report. MLIR changes all the time, if we use Triton nightly wheels output as baseline, there can be many PRs merged between that and the PR, or the PR could be based on a commit before the Triton nightly wheels.

Eventually I would like to be able to diff as much IR as possible, but to start I think grabbing one representative set of IRs from each microbenchmark would be ok. I am not sure how to best grab the IR without running the benchmark, figuring that out is part of this issue.

Just to clarify, do you mean diffing the final MLIR of one input shape?

Do you have another approach that would be less noisy that we could try?

If we reconsider the motivation, which is to have a way to quickly identify if performance impact is variance or not. Another way to resolve that could be CI automatically determine if the performance results are within standard deviation, and report only gains and degradations. Or CI automatically rerun benchmarks with potential performance impact.

The motivation is to be able to determine if a given PR is going to change the TTIR/TTGIR (and possibly even the MLIR). Detecting performance regressions, etc is just one benefit from knowing if a change is mutating the IR.
Yes, it seems like most microbenchmarks do parameter sweeps across input shapes but I imagine there are other possible parametrizations.

Offline, I mentioned some scripts from the LLVM infrastructure that can help to abstract over SSA value names.

For MLIR, there is this script: https://github.com/llvm/llvm-project/blob/main/mlir/utils/generate-test-checks.py
For LLVM IR, there is: https://github.com/llvm/llvm-project/blob/main/llvm/utils/update_test_checks.py

Even if we don't use the scripts themselves, we can maybe learn something from how they detect and abstract SSA value names.


Failures on [analyzer] Move 3 more HintCodes to be WarningCodes, UNUSED_LABEL, UNUSED_RESULT*...[co19] Roll co19 to 7b4f3e723211011878e185db6fa57fbafabd6bb7

There are new test failures on [analyzer] Move 3 more HintCodes to be WarningCodes, UNUSED_LABEL, UNUSED_RESULT*...[co19] Roll co19 to 7b4f3e723211011878e185db6fa57fbafabd6bb7.
The tests
web_2/internal/rti/required_named_parameters_test RuntimeError (expected Pass)

are failing on configurations
ddc-linux-chrome-unsound_2

According to https://dart-ci.firebaseapp.com/, this is a "new test" as of either https://github.com/dart-lang/sdk/commit/27cfabdac0d6c93c22eb01ee112a880b805d0742 or https://github.com/dart-lang/sdk/commit/5a207ea99afdbdb1cc2c77959c55aaba1edbbf26.  But that seems really unlikely, since the test has existed since 912005267d5862b25e8050b263e4870b757d83a6 and was last modified in b101a7d002fa6db1f436c96d3945aa62aec73ba9 (both in 2021).  So I'm not sure why this failure is only just showing up now.  But I'm going to approve the failure in order to land an unrelated CL.

The results appearing as "new" is probably related to my recent change to all of the ddc test and builder configurations. I suspect we didn't get a baseline result for that test copied to the database before we started running.
That said, I don't expect this test to be passing on the stable DDC builds until after we switch to the new runtime type system.


refactor: qq command to use post consumer

Codecov Report

Merging #80 (a1f483a) into master (5c47b37) will increase coverage by 0.70%.
The diff coverage is 29.41%.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   74.95%   75.66%   +0.70%
==========================================
  Files          24       25       +1
  Lines         599      600       +1
  Branches       94       90       -4
==========================================
+ Hits          449      454       +5
+ Misses        150      146       -4

Flag
Coverage Δ

unittests
75.66% <29.41%> (+0.70%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/command/qq/qq-command.ts
0.00% <0.00%> (ø)

src/services/qq-service.ts
0.00% <0.00%> (ø)

src/command/qq/consumer-handler.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


is there a paper with this?

Perhaps you are looking for original paper https://arxiv.org/pdf/1911.01547.pdf


#113 Upload / Download of categories does not work

In #113 the upload and download of categories does not work (Firefox and Chrome).

What does "does not work" mean? :wink:

Clicking on the download button: I don't get the file with the categories - nothing happens.
Clicking on the upload button: I can't upload a file with categories, because the browser to select the file isn't opened.

Interestingly enough, contrary to the original title of theis issue, the download and upload features apparently broke independently of one another.
According to git bisect the upload button broke in 7b603e3. Looking at this first now.


Prefer copied() iterators instead of cloned()

Copied is available since Rust 1.36 and Rayon 1.2.

Yeah -- it might have some codegen benefit in extend, but I wouldn't expect much. We don't need to release-note this IMO.


ValueError: bg_num_rois = 0 and fg_num_rois = 0, this should not happen!

I tried to run the code on Pascal VOC 2007 only, with resnet-50, and it worked.
Then I tried to run the code on Pascal VOC 2007+2012, with resnet-101, this bug appeared.
The environment I used is
CUDA 9.0
Python=3.7
PyTorch=0.4.1.post2
torchvision=0.2.1.post2
in colab
I tried several methods, such as
delete -1 in psacal_voc.py
    # Load object bounding boxes into a data frame.
    for ix, obj in enumerate(objs):
        bbox = obj.find('bndbox')
        # Make pixel indexes 0-based
        x1 = float(bbox.find('xmin').text) #- 1
        y1 = float(bbox.find('ymin').text) #- 1
        x2 = float(bbox.find('xmax').text) #- 1
        y2 = float(bbox.find('ymax').text) #- 1

and
delete -1 in imdb.py
for i in range(num_images):
  boxes = self.roidb[i]['boxes'].copy()
  oldx1 = boxes[:, 0].copy()
  oldx2 = boxes[:, 2].copy()
  boxes[:, 0] = widths[i] - oldx2# - 1
  boxes[:, 2] = widths[i] - oldx1# - 1

and
I changed the TRAIN.RPN_MIN_SIZE = 8 to 0
I've tried all the methods mentioned in https://github.com/jwyang/faster-rcnn.pytorch/issues/111  but it didn't work
Could you tell me how to fix the bug?

Called with args:
Namespace(TFA=False, batch_size=4, checkepoch=10, checkpoint=21985, checkpoint_interval=10000, checksession=1, class_agnostic=False, cuda=True, dataset='pascal_voc_0712', disp_interval=100, fix_encoder=False, log_dir='checkpoint', lr=0.001, lr_decay_gamma=0.1, lr_decay_step=4, max_epochs=21, meta_loss=True, meta_train=True, meta_type=1, net='metarcnn', num_workers=1, optimizer='sgd', phase=1, resume=False, save_dir='save_models/VOC_first', session=1, shots=1, start_epoch=1, use_tfboard=True)
Loaded dataset voc_2007_train_first_split for training
Set proposal method: gt
Appending horizontally-flipped training examples...
wrote gt roidb to /content/drive/MyDrive/few shot/data/cache/voc_2007_train_first_split_gt_roidb.pkl
done
Preparing training data...
done
Loaded dataset voc_2012_train_first_split for training
Set proposal method: gt
Appending horizontally-flipped training examples...
wrote gt roidb to /content/drive/MyDrive/few shot/data/cache/voc_2012_train_first_split_gt_roidb.pkl
done
Preparing training data...
done
before filtering, there are 12330 images...
after filtering, there are 12330 images...
before class filtering, there are 12330 images...
after class filtering, there are 12330 images...
12330 roidb entries
Loading pretrained weights from data/resnet101.pth
[session 1][epoch  1][iter    0] loss: 17.7643, lr: 1.00e-03
fg/bg=(38/474), time cost: 1.222608
rpn_cls: 0.8498, rpn_box: 0.4404, rcnn_cls: 15.8508, rcnn_box 0.4173, meta_loss 0.2060
Traceback (most recent call last):
File "train.py", line 477, in
rois_label, cls_prob, bbox_pred, meta_loss = fasterRCNN(im_data_list, im_info_list, gt_boxes_list, num_boxes_list)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 477, in call
result = self.forward(*input, **kwargs)
File "/content/drive/MyDrive/few shot/lib/model/faster_rcnn/faster_rcnn.py", line 84, in forward
roi_data = self.RCNN_proposal_target(rois, gt_boxes, num_boxes)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 477, in call
result = self.forward(*input, **kwargs)
File "/content/drive/MyDrive/few shot/lib/model/rpn/proposal_target_layer_cascade.py", line 47, in forward
rois_per_image, self._num_classes)
File "/content/drive/MyDrive/few shot/lib/model/rpn/proposal_target_layer_cascade.py", line 202, in _sample_rois_pytorch
raise ValueError("bg_num_rois = 0 and fg_num_rois = 0, this should not happen!")
ValueError: bg_num_rois = 0 and fg_num_rois = 0, this should not happen!

hello,how did you solve this problem?


Support sorbet-typed v2

Convert version strings from their escaped form if the sorbet-typed version is 2.
This is necessary because <2.0 is an invalid directory name on NTFS file systems. Instead of <2.0' we can change that to lt;2.0` and it will work on all file systems.
Motivation
This resolves #1782 by supporting the use of escaped version identifiers in sorbet-typed.
You can see what this does with this commit in my project: https://github.com/connorshea/VideoGameList/commit/e884f7384fb19c926786ddcf25382480c7f4181f
You can reproduce it in your own by going into ~/.cache/sorbet/sorbet-typed/ and adding my fork of sorbet-typed as a remote (git remote add fork https://github.com/connorshea/sorbet-typed.git), then running SRB_SORBET_TYPED_REVISION='fork/sorbet-typed-v2-directory-names' bundle exec srb rbi sorbet-typed in your project to get the sorbet-typed repo with escaped directory names.
This is dependent on #2697. We should probably wait a few days to merge this after #2697 is merged.
Test plan
See included automated tests.

@connorshea do you mind if I close this? I think we'd prefer to go with the zip file approach instead of versioning sorbet-typed.

Yup, feel free.

Closing because we're closing #2697


BooleanField == true? Error

I'm trying to compare a BooleanField with the bool value true and the Visual Studio is giving me this error:
Error	CS0019	Operator '==' cannot be applied to operands of type 'BooleanField' and 'bool'.
The source:
var a = Serene1.Administration.Entities.UserRow.Fields;
if (a.RACS == true) // <-- Error right here..
{
request.Criteria = new Criteria(MyRow.Fields.UserID.Name) == (IdUser);
}
It must be some dumb thing that I am doing..

what is your class and method? maybe you need use Row.RACS == true
Fields is definition for sql query build and custom functionality. Values stored on Row class.

a.RACS is a field not a value. a.RACS[row].value is your value, where row is the entity you read

Read your RACS property from Row.RACS, Old.RACS etc.

Thanks  VictorTomaili, Estrusco and volkanceylan. Thanks to you I solve my problem.
The solution:
var a = Administration.Entities.UserRow.Fields;
var RACS = connection.Single<Administration.Entities.UserRow>(new Criteria(a.UserId) == IdUser).RACS;
if (RACS == true)
{
request.Criteria = new Criteria(MyRow.Fields.UserID.Name) == (IdUser);
}
Thanks again.


Consider alternative names to cmd!, cmd_unit!, and cmd_result!

This is totally subjective, so feel free to close this if you don't agree, but I kind of think the existing macro names aren't the best.
I think cmd_unit is actually extremely common, just judging from our own scripts, and it would be nice if it had a nicer, shorter name.
I like the names run!(…), output!(…), and result!(…). run is the default for when you don't want anything, and output and result tell you what they're going to return.

I think we would want run (for returning unit), run_output (for producing Output) and run_result. And we would want the same names for the macros and the new Input method interface from #139.


39: Allow roll overrides in stat rolls - [merged]

In GitLab by @Asacolips on Apr 11, 2021, 15:25
Merges 39-stat-rolls -> alpha
Closes #39

In GitLab by @Asacolips on Apr 11, 2021, 15:25
mentioned in commit 2e129fc3794b8fa799b59ce20118ddde92fc929a


Remove unused method

I've just found a private method that is not being used.

Looks like the code that used it was deleted ~10 years ago, so hopefully we don't need this anymore.  thanks!


feat(postgres): Add URL escaping to database connection string

Add url.QueryEscape() to properly handle special characters in PostgreSQL connection parameters. This prevents potential connection issues and improves security when credentials contain special characters.
Changes:

Escape username with url.QueryEscape()
Escape password with url.QueryEscape()
Escape database name with url.QueryEscape()

Example special characters handled:

@ in usernames (e.g<EMAIL_ADDRESS>Special chars in passwords (e.g., *, (, ), @)
Special chars in database names

This change ensures the DSN string is properly formatted regardless of the characters present in the connection parameters.
Related to PostgreSQL connection string format:
postgres://username:password@hostname:port/database

[!IMPORTANT]
Add URL escaping to PostgreSQL connection string in config.go to handle special characters in connection parameters.

Behavior:

Add url.QueryEscape() to escape special characters in PostgreSQL connection parameters in DSN() in config.go.
Handles special characters in username, password, host, and database name.

Security:

Prevents connection issues and improves security by ensuring DSN string is properly formatted.

This description was created by  for 4bf628d0df875dded90c0c7297329156470cd31e. It will automatically update as commits are pushed.

@danielchalef Is anyone around to review this at some point?

@alexbricepalo Thanks for your contribution

Published as part of v1.0.2


flet.Image animation controls, properties and link click event handlers

Some image formats are actually videos (animations). gif, webm, etc.
These would need optional controls (eg fullscreen, time-positioning), also methods to do these (jump_in_time, stop_playback), properties: repeat=False.
SVG can have links (like html) embedded, so that would need an on_tap_link event handler like markdown has.

https://www.w3.org/wiki/SVG_Links


Uncaught SyntaxError: Unexpected token ILLEGAL in $.extend(true, apos.data, { „mediaLibrary“: CONTENT })

Hi apostrophe-team,
we have some trouble with the script extending the apos.data object on the client side from https://github.com/punkave/apostrophe/blob/master/lib/push.js#L242.
The generated line in the browser extending the apos.data with the mediaLibrary information ($.extend(true, apos.data, { „mediaLibrary“: CONTENT })) throws an „Uncaught SyntaxError: Unexpected token ILLEGAL“ error in the browser. We can’t exactly specify what causes the error, as the CONTENT object is about 1.400.000 characters long (the object contains the blog page with all blogPost pages from the blog plugin).
CONTENT is similar to this:
{
  "mediaLibrary": {
    "aposPrefix": "",
    "doNotEdit": false,
    "offline": false,
    "demo": false,
    "loginButton": false,
    "siteTitle": "siteTitle",
    "shortName": "shortName",
    "hostName": "localhost:3000",
    "page": {
      "_id": "ID",
      "slug": "/",
      "path": "home",
      "title": "Home",
      "level": 0,
      "type": "blog",
      "published": true,
      "seoDescription": "",
      "tags": [],
      "orphan": false,
      "pagePermissions": ["edit-ID"],
      "andFromPagesIds": [],
      "andFromPagesRelationships": {},
      "sortTitle": "home",
      "highSearchText": "home",
      "highSearchWords": ["home"],
      "lowSearchText": "",
      "searchSummary": "",
      "blogWidget": {"slug": "/:blogWidget"...}
      "_edit": true,
      "_publish": true,
      "url": "/",
      "ancestors": [],
      "peers": [
        {"_id": ID}
      ],
      "children": [...],
      "tabs": [...]
    }
  }
}

If we add an empty object to the mediaLibrary property the error won’t occur and everything works fine (it seems that other CMS functionalities won’t be affected). Is it a normal thing that the mediaLibrary property has so many data? Can we leave it blank or will this lead to another negative impact? Is it possible, that the "mediaLibrary" object contains unescaped characters?
Thanks in advance for your response!

Hi Matthias,
I haven't seen this (the entire blog being pulled in by this). I checked it
in the standard sandbox when you first reported it. Do you have a test
project that reliably reproduces this that we could look at?
On Tue, Jan 5, 2016 at 5:13 AM, Matthias<EMAIL_ADDRESS>wrote:

Hi apostrophe-team,
we have some trouble with the script extending the apos.data object on the
client side from
https://github.com/punkave/apostrophe/blob/master/lib/push.js#L242.
The generated line in the browser extending the apos.data with the
mediaLibrary information ($.extend(true, apos.data, { „mediaLibrary“:
CONTENT })) throws an „Uncaught SyntaxError: Unexpected token ILLEGAL“
error in the browser. We can’t exactly specify what causes the error, as
the CONTENT object is about 1.400.000 characters long (the object contains
the blog page with all blogPost pages from the blog plugin).
CONTENT is similar to this:
{
"mediaLibrary": {
"aposPrefix": "",
"doNotEdit": false,
"offline": false,
"demo": false,
"loginButton": false,
"siteTitle": "siteTitle",
"shortName": "shortName",
"hostName": "localhost:3000",
"page": {
"_id": "ID",
"slug": "/",
"path": "home",
"title": "Home",
"level": 0,
"type": "blog",
"published": true,
"seoDescription": "",
"tags": [],
"orphan": false,
"pagePermissions": ["edit-ID"],
"andFromPagesIds": [],
"andFromPagesRelationships": {},
"sortTitle": "home",
"highSearchText": "home",
"highSearchWords": ["home"],
"lowSearchText": "",
"searchSummary": "",
"blogWidget": {"slug": "/:blogWidget"...}
"_edit": true,
"_publish": true,
"url": "/",
"ancestors": [],
"peers": [
{"_id": ID}
],
"children": [...],
"tabs": [...]
}
}
}
If we add an empty object to the mediaLibrary property the error won’t
occur and everything works fine (it seems that other CMS functionalities
won’t be affected). Is it a normal thing that the mediaLibrary property has
so many data? Can we leave it blank or will this lead to another negative
impact? Is it possible, that the "mediaLibrary" object contains unescaped
characters?
Thanks in advance for your response!
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/issues/397.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Hi @boutell,
thanks for your fast response! I think i have the problem now. After i read your answer i checked the error again and i saw that i get the position via firefox. It looks like we have some LINE SEPERATOR (<U+2028) characters in our database and JavaScript can't handle this, see http://stackoverflow.com/questions/2965293/javascript-parse-error-on-u2028-unicode-character
$.extend(true, {}, JSON.stringify({ "test": "first
 last"}))
Uncaught SyntaxError: Unexpected token ILLEGAL(…)

Between first and last is this character and if i delete this in the database it works for this, but fails on the next one.
Any idea to fix this?

Yes, this is a legitimate concern I've bumped into once. We'll be looking
into a good canonical fix. Looks like escaping it with \u2028 is correct,
but I want to make sure we cover all the cases and also don't make it very
slow. It's strange that I haven't been able to quickly find a valid
implementation of this in npm. PHP's json_encode does the right thing by
default (and it's possible for it to be the "right thing" for everyone
because with the escapes, it's still valid JSON as well as valid JS).
On Thu, Jan 7, 2016 at 9:23 AM, Matthias<EMAIL_ADDRESS>wrote:

Hi @boutell https://github.com/boutell,
thanks for your fast response! I think i have the problem now. After i
read your answer i checked the error again and i saw that i get the
position via firefox. It looks like we have some LINE SEPERATOR (<U+2028)
characters in our database and JavaScript can't handle this, see
http://stackoverflow.com/questions/2965293/javascript-parse-error-on-u2028-unicode-character
$.extend(true, {}, JSON.stringify({ "test": "first
 last"}))
Uncaught SyntaxError: Unexpected token ILLEGAL(…)
Between first and last is this character and if i delete this in the
database it works for this, but fails on the next one.
Any idea to fix this?
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/issues/397#issuecomment-169677830.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Thanks a lot!

It's strange that I haven't been able to quickly find a valid
implementation of this in npm.

Yes, i was also searching. Looking forward to your fix!

@jth- Do you know, when will Apostrophe 0.5 be released?

0.5 has been out for quite some time (: I think you mean 0.6 which is
pending soon as we're finally doing an in-house project with it. But either
way we don't have a clean fix for this bug yet.
On Wed, Feb 10, 2016 at 4:23 AM, Matthias<EMAIL_ADDRESS>wrote:

@jth- https://github.com/jth- Do you know, when will Apostrophe 0.5 be
released?
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/issues/397#issuecomment-182272672.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

I'm having a bash at this now, the fix looks like it adds not much overhead
given that we already have similar fixes for things that would end the
script tag.
On Wed, Feb 10, 2016 at 9:07 AM, Tom Boutell<EMAIL_ADDRESS>wrote:

0.5 has been out for quite some time (: I think you mean 0.6 which is
pending soon as we're finally doing an in-house project with it. But either
way we don't have a clean fix for this bug yet.
On Wed, Feb 10, 2016 at 4:23 AM, Matthias<EMAIL_ADDRESS>wrote:

@jth- https://github.com/jth- Do you know, when will Apostrophe 0.5 be
released?
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/issues/397#issuecomment-182272672
.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Fix is published. I added one more commit in master to make the replace properly global.

Thank you very much! We will update our project soon to your latest release.


ES6 exponentials (**) cause error:

Code:
/* jshint esversion: 6, node: true */
'use strict';

let a = 5;

console.log(a**2);

Error:
Five warnings
6	Expected an identifier and instead saw '*'.
6	Expected ')' and instead saw '2'.
6	Missing semicolon.
6	Expected an identifier and instead saw ')'.
6	Expected an assignment or function call and instead saw an expression.

Expected it to pass as this is a valid ES6 construct..
Thanks!

Just found #2602 which seems to discuss this issue from before its standardization...

The exponentiation operator was not standardized until version 7 of ECMAScript, which is why esversion: 6 does not support it. Support for ES7, enabled via esversion: 7, has been merged to the v2.10.0 branch and will be available in the next major release of JSHint. We do not currently have an expectation for the release date, so stay tuned.


Why does better-sqlite3 provide a transaction.deferred() method if DEFERRED already is the default transaction behavior?

Reference: https://sqlite.org/lang_transaction.html#deferred_immediate_and_exclusive_transactions

I guess symmetry and some people like to write explicit code without requiring future readers (including a future version of the author) from knowing what the default is.

👍 That makes sense. Thanks for the information.


Improve species generality: output report tables

With the updated framework for selecting species (see #197) to include in the estimates, the main report tables need to be updated. These include:

[x] make_strata_summary_table()
[x] make_johnson_summary_table()
[x] make_appendix_table()
[ ] make_goals_summary_table()

Closed by #205


Can't locate object method "authority" via package "Mojo::URL"

My smokers started to report the following failure:
Can't locate object method "authority" via package "Mojo::URL" at /home/cpansand/.cpan/build/2016072021/Mojolicious-Plugin-AssetPack-1.17-2/blib/lib/Mojolicious/Plugin/AssetPack/Store.pm line 162, <DATA> line 49.
t/sass-current-app.t ...............
Dubious, test returned 29 (wstat 7424, 0x1d00)
No subtests run

As Mojolicious 7.0 broke a number of modules I suspected another one caused by Mojolicious, but statistical analysis is not clear about it (i.e. theta is not -1.0 here):
****************************************************************
Regression 'mod:Mojolicious'
****************************************************************
Name                   Theta          StdErr     T-stat
[0='const']           1.0000          0.1949       5.13
[1='eq_6.65']         0.0000          0.3081       0.00
[2='eq_6.66']        -0.0370          0.2054      -0.18
[3='eq_7.0']         -0.6111          0.2105      -2.90

R^2= 0.425, N= 50, K= 4
****************************************************************

Relevant change: https://metacpan.org/source/SRI/Mojolicious-7.0/Changes#L5

Thank you! A new release is soon on CPAN. :


wine-staging 1.7.45 (new formula)

wine-staging, an unofficial place to collect wine patches and prepare them for sending to upstream, is popular among Linux users, since the wine upstream is often considered as being slow and strict to accept new patches.
This PR introduces an pre-patched git tag version to the Homebrew repository. Since it's based on the current devel version of wine, I have to put it here in homebrew-devel-only.

Uh-oh, maybe I should do some amend and force push to change the commit message...

Implemented Optional resources by adding if stuffs to the path.install stuffs.

It seems that the test bot is failing to checkout the commits.
39581fd9a03d50997d493 (refs/remotes/origin/pr/19/merge)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 342f07f0da6478948c539581fd9a03d50997d493
FATAL: Could not checkout null with start point 342f07f0da6478948c539581fd9a03d50997d493

What happened?

@BrewTestBot test this please

I believe that wine-staging: Patched wine with new, unoffical bug fixes and features. is only 70 chars long.

Oh crap…… It's giving me some W:157:  7: Useless assignment to variable - s. after removing what it referred to as C:166:  5: Redundant return detected..

is only 70 chars long.

As highlighted by brew audit --strict the character count includes the full name, rather than the formula name purely, so it would be homebrew/devel-only/wine-staging.

@DomT4 That can be assumed.

( Well actually I doubt if I should make it some --with-staging-patches in wine.. Since it's still somehow like a patch depot.

And for that not notable enough.. Nobody would like to follow a repo that rebases almost daily, applying the patches in another repo.

Nobody would like to follow a repo that rebases almost daily, applying the patches in another repo.

Note that you can star a repo without watching it. I starred 3.8k+ repos, I’m not watching all of them :D

@bfontaine but will such an automaically-patched repo, most people will simply star where the patches comw from -- https://github.com/wine-compholio/wine-staging in our case.

@BrewTestBot test this please.

@BrewTestBot test this please.

It's the notability warning:
==> audit problems
homebrew/devel-only/wine-staging:
 * GitHub repository not notable enough (<10 forks, <10 watchers and <20 stars)

@dunn And I explained that earlier…

Ouch, upstream is at 1.7.52 already…

And I explained that earlier…

Oh, I know.  I just meant there's no new test failure.

By the way, since this code shares a lot of wine package, I am not sure if I should add a with staging or whatever to wine's stuff directly, given that the git patchset can also be applied manually in the Formula..

@BrewTestBot test this please.

@BrewTestBot test this please.

Passing on this.


Fix Github actions warnings

Node.js 12 actions are deprecated:

Update actions/checkout to v3
Update actions/upload-artifact to v3

Codecov Report

Merging #2576 (19d225d) into main (228fe99) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 19d225d differs from pull request most recent head 1a85791. Consider uploading reports for the commit 1a85791 to get more accurate results

@@           Coverage Diff           @@
##             main    #2576   +/-   ##
=======================================
  Coverage   30.98%   30.98%
=======================================
  Files         254      254
  Lines       29623    29623
=======================================
  Hits         9179     9179
  Misses      20444    20444

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

The only warning remaining after this is the one about Ubuntu 18.04.

I think Noah's PR #2535 should clear that as well.


Deprecate Coding Challenge README's

Following up from #389, we can now remove the community contributions from the Coding Challenge README's and add a note to each one how to contribute with the new system.

I'm so sorry I have been inactive for the past few weeks, I'll get to this very shortly!

This transition is long done and issue could be closed.
@shiffman Could you confirm?

indeed!


Statement References section needs to be removed or re-written

Statement References
I'm unsure enough how to do this I propose we do not include statement reference constraints in the initial draft. It would probably be included as a special case in the statement template rules, above.

Should we therefore also remove the objectStatementRefTemplate and contextStatementRefTemplate properties?

That text is old, must have missed it in the excision


Refactor TypeScript type definition files

{ css }'s type completion has broken since some time or another.
I fix it and add a document😆  Please check my pull request!

Refactoring

Fix broken the type completion of css()
Add the new APIs
Annotate to the deprecated APIs
Use CSSProperties from @types/react (via. #196)

Add a document for use with the TypeScript

Sorry, my English is not good... 😢

Nice one one. Better that mine 👍 , but @threepointone still has to decide if he is willing to draw a dependency to React in index.d.ts.

I don't like that you have deleted the comments, as they actually is read by the typescript language service, and appears in the autocompletion as you can see in the screenshot you have provided.

Maybe we'll wait for a little feedback in #83 from @threepointone to see, if we can move the whole code base to TypeScript before we go further with this review. And I agree with @thetrompf that the comments shouldn't be deleted, they are very useful with the typescript language service. :)

I'm sorry and thank you for pointing out.
I will fix it and look again for similarly mistake.

@thetrompf @donaldpipowitch Could you review again?

@axross It's looking good, but you are missing the style() definition. Other than that, it looks good. It feels really nice with the autocompletion. 👏

You are still missing all the comments above the function definitions.

@thetrompf Thank you for review. I fixed it!

Is it possible to use lib.dom.d.ts (see here) instead of React's css property definitions? React's definitions has a lot of ambient declarations as compared to lib.dom more defined typings.

Certainly, I think glamor should not dependent to @types/react. But, between definition in lib.dom.d.ts and definition in @types/react have some different. (e.g. flexGrow lib.dom.d.ts / @types/react)
And, definition in lib.dom.d.ts includes fields for vendor prefix. I think there are not needed for glamor.

The lib.dom version does not support numeric values height, width, marginTop, etc. like glamor support. So I think reacts version is much closer to what glamor supports.

Maybe @basarat has an advice for us. He created CSS typings for https://github.com/typestyle/typestyle

I think it is same thing the glamor dependents type definition of react or other.
I want to decide either glamor (optional)dependents other package for type definition or not. (cc. @threepointone)

Sorry it took so long, but glamor is now written in TypeScript. I'll close this PR. Thanks for your work.
We'll discuss how to handle CSSProperties later™. I see three possible ways:

use CSSProperties from @types/react (but glamor itself doesn't need React, so I don't prefer this approach)
use a similar approach to TypeStyle (a lot of manual work, but maybe we can share efforts somehow)
try to push it into lib.dom.d.ts somehow (https://github.com/Microsoft/TypeScript/issues/10820)


Add how to install valgrind and clang-format on CONTRIBUTING.md #159

Added it to "Local Development" section in CONTRIBUTING.md;
Removed how to install valgrind/clang-format from main README.
Explained that make valgrind and make format must always be run before committing/making PRs.

@ArielMAJ @tauanesales Thank you so much!

@ArielMAJ @tauanesales Thank you!


Adjusting billing aggregation timeouts and memory.

Increased RAM and timeout for both seqr and hail aggregation functions.

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

Please upload report for BASE (main@43962b3). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #264   +/-   ##
=======================================
  Coverage        ?   90.93%
=======================================
  Files           ?        4
  Lines           ?      430
  Branches        ?        0
=======================================
  Hits            ?      391
  Misses          ?       39
  Partials        ?        0

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

60 mins as timeout, with RAM not sure, but we are getting close to what we need, only last week or so there was a lot of large batches. Atm it is running every 2hrs and processing last 4 hours of batches, we can change to process 2hrs of batches every hour, which would be less data if needed. For time being I will monitor alerts and adjust RAM/timeouts as needed.


Add zinit installation instructions

Zinit
zinit ice wait  # For faster prompt loading
zinit snippet 'https://raw.githubusercontent.com/CodesOfRishi/smartcd/main/smartcd.sh'

For copy paste:
### [Zinit](https://github.com/zdharma-continuum/zinit)

An improvement on this:
zinit ice wait depth"1"
zinit light 'CodesOfRishi/smartcd'

The previous snippet line should be removed in favour of these.
This will shallow clone the repository and allow the user to keep their own personal commits will be merged at the next pull.

Installation through shallow clone using Zinit, so as for the user to make personal commits & merge them with others, I should leave these parts for the users to decide and research about since I don't think it's very specific to SmartCd.


Partners can use tuple

Would love to see more information about how Partners can use the tuple in Partner Center and how this doesn't require people to open a box and gather the hash.  Happy to help write the blurb if necessary.  Thanks  --petl

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

ID: 3243b015-5b5f-7a62-0d9d-767d729dc475
Version Independent ID: ff8c0509-c075-8528-823f-c195ff4d2382
Content: Adding devices
Content Source: windows/deployment/windows-autopilot/add-devices.md
Product: w10
GitHub Login: @greg-lindsay
Microsoft Alias: greg-lindsay

@officedocsbot assign @e0i

@luriep I wanted to confirm if the level of details found in this article (which is referenced within the article you linked) is enough?
It provides a step by step guide and gives details about the tuple and HH.
If not, could you please write a few words on this topic?

@luriep Just to be more clear, you can find all the details you need if you go under "Partner Center" in the table that talks about "Tuple":

Hello @luriep ,
The issue is being closed since your question seems to be answered. Feel free to re-open this issue if you feel that it hasn't been answered or that there are further suggestions to improve the documentation itself.
Thanks.


Update what's new page with relevant issues for df.query() with in-pl…

[x] closes #10486
[N/A] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

…ace operator
The relevant issues added are :
BUG: query with invalid dtypes should fallback to python engine #10486
BUG: query modifies the frame when you compare with = #8664

this is actually editing an old whatsnew but ok.


Pre-declare all core dispatch table functions, and fix the internal ones

When assigning pointers to functions in an OSSL_DISPATCH table, we try
to ensure that those functions are properly defined or declared with
an extra declaration using the corresponding function typedefs that
are defined by include/openssl/core_dispatch.h.
For the core dispatch table, found in crypto/provider_core.c, it seems
we forgot this habit, and thus didn't ensure well enough that the
function pointers that are assigned in the table can actually be used
for those dispatch table indexes.
This change adds all the missing declarations, and compensates for
differences with functions that do the necessary casting, making those
explicit rather than implicit, thereby trying to assure that we know
what we're doing.
One function is not fixed in this change, because there's a controversy,
a clash between the signature of BIO_ctrl() and OSSL_FUNC_BIO_ctrl_fn.
They have different return types.

Regarding the BIO_ctrl vs OSSL_FUNC_BIO_ctrl_fn clash, the easy
(and most correct) fix would be this:
diff --git a/include/openssl/core_dispatch.h b/include/openssl/core_dispatch.h
index 6aeb9e61e1..0cf9f94dd6 100644
--- a/include/openssl/core_dispatch.h
+++ b/include/openssl/core_dispatch.h
@@ -173,8 +173,8 @@ OSSL_CORE_MAKE_FUNC(int, BIO_vprintf, (OSSL_CORE_BIO *bio, const char *format,
                                        va_list args))
 OSSL_CORE_MAKE_FUNC(int, BIO_vsnprintf,
                    (char *buf, size_t n, const char *fmt, va_list args))
-OSSL_CORE_MAKE_FUNC(int, BIO_ctrl, (OSSL_CORE_BIO *bio,
-                                    int cmd, long num, void *ptr))
+OSSL_CORE_MAKE_FUNC(long, BIO_ctrl, (OSSL_CORE_BIO *bio,
+                                     int cmd, long num, void *ptr))

 #define OSSL_FUNC_SELF_TEST_CB               100
 OSSL_CORE_MAKE_FUNC(void, self_test_cb, (OPENSSL_CORE_CTX *ctx, OSSL_CALLBACK **cb,

However, since this is a public facing interface, this change is
problematic.  Ideas welcome.

The BIO_ctrl() vs OSSL_FUNC_BIO_ctrl_fn clash is the only reason this is a draft.

Still looking for ideas re BIO_ctrl

Still looking for ideas re BIO_ctrl

I gave that some thought, and added a commit for it.  The solution may be controversial

Except I'd do s/BOGUS/INT/

That would suggest that it has proper usage, just happens to be "the int version".  I want it to be understood that it shouldn't be used...

I added a squash commit that makes for a leaner resolution of the BIO_ctrl() issue

@openssl/committers, second review desired

OTC: @levitte to create a new PR to make the alternatives visible

This is now self contained and doesn't touch BIO_ctrl at all.  #18583 and #18584 are the two solutions that do

@mattcaswell please re-approve

Or in fact, any @openssl/otc member could do the second approval

Even committer could approve.

This pull request is ready to merge

Merged
master:
9574842e90e29015daa2b071e965cec9aa885c17 Pre-declare all core dispatch table functions, and fix the internal ones
3.0:
372f4b80e35954d19ffb255692bdd6e4807e5f7a Pre-declare all core dispatch table functions, and fix the internal ones


ECONNREFUSED

When the database is not available, mysqldump throws an exception and stops the server.
Instead, it should call the callback function, giving the error as a parameter.

I'll take a look on that soon

Thank you very much.
There are cases when this may be a major problem.
Please, let me know if you correct the issue in a new NPM release.
Thank you again.


Fix export button styling in datagrid export component on loading.

Issue Reference
#9666

Checked and Found that the export icon issue has been fixed and working fine, please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/551ba81b-5d00-42a8-b737-dcd457bdc86c


Add snap ci check

[ ] Ensure SNAP_BRANCH is correct branch

Since Snap CI appears to be shut down, consider closing this issue


[Bug]: Webpack issue when using onPlayable callback

Version
1.1.6
Environment that reproduces the issue
Mac OS 14.1.1
Is it reproducible in the example application?
Yes
Reproduction steps

Create a tsx component in next js 14.
Run uploader.onplayable callback.
Error shows up.

Expected result
The expected outcome is for the callback to return the playable video in the callback. This error doesn't show up with the upload callback so I suspect it's specific to the onplayable callback.
Actual result
Webpack 5 error.

Additional context
No response
Relevant logs output
No response

Was an issue with webpack bundling server code in payloadcms


Polynomial division

Is there any support for laying out long polynomial division?

There is no support for long division in general.  @freemaths is there a particular package that you use for this in LaTeX?

@freemaths Sometimes one can hack a solution together from spare parts:

@kevinbarabash Is \longdiv something that belongs on my To Do list? We could use a method similar to KaTeX stretchy radicals.

@ronkok were you thinking of just rendering the \longdiv symbol or also laying out all of the steps too?

I'm thinking only of the symbol for now. Perhaps the steps can come later if there is much demand.

I imagine there's interest. But we should figure out which LaTeX package adds this functionality. I saw a few in a quick search. The most obvious one, longdiv.tex, does the whole derivation, which doesn't seem what we want (in particular it doesn't support polynomials as originally requested).

The Polynom package does polynominal division. It's rendering of the radical is poor. Here's how it renders \polylongdiv[stage=1]{X^3+X^2-1}{X-1}

Like package longdiv, polynom will render all the steps, at least if the polynomial is in one variable. And it does factorizations and remainders and other tasks, as well. So there's a lot to it. I don't know if we want to ship that much code for something that may not get much use. Maybe I should supply a longdivsymbol for KaTeX core and then parts from the rest of polynom or longdiv could be an extension.

The difficult part is the alignment rather than the symbol. The "hacky" solution I have come up with so far is to render white on white to hide things and keep aligned. Align is only interested in aligning equals signs as far as I can tell. (Note: not the best example as no polynomial carry but would work for that too).
('textcolor{white}{x-3}';4x^2+2)
(x-3'overline{'smash{)}4x^3-12x^2+2x-6})
('textcolor{white}{x-3}';'underline{4x^3-12x^2})
('textcolor{white}{{x-3}';4x^3-12x^2+}',2x-6)
('textcolor{white}{{x-3}';4x^3-12x^2+}','underline{2x-6})

Didn't know about \phantom and missed it on the help page. Same result but far less hacky than white on white:
(\phantom{x-3}{;4x^2+2})
(x-3\overline{\smash{)}4x^3-12x^2+2x-6})
(\phantom{x-3};\underline{4x^3-12x^2})
(\phantom{{x-3};4x^3-12x^2+},2x-6)
(\phantom{{x-3};4x^3-12x^2+},\underline{2x-6})

And with array{l}.

(\begin{array}{l}
\phantom{x-3;}{;4x^2+2}\
x-3\overline{\smash{)}4x^3-12x^2+2x-6}\
\phantom{x-3};\underline{4x^3-12x^2}\
\phantom{{x-3};4x^3-12x^2+},2x-6\
\phantom{{x-3};4x^3-12x^2+},\underline{2x-6}\
\end{array})


MVP Meshery action for SMI conformance validation

Signed-off-by: Rudraksh Pareek<EMAIL_ADDRESS>Description
This PR fixes life
Notes for Reviewers
Some hard-coding here and there restricts it to only work with minikube clusters and meshery in docker.
Its not a big restriction and there is a sample workflow included so this should be good for now: )
Will be handling all sorts of clusters in future.
Signed commits

[x] Yes, I signed my commits.

Signed-off-by: Rudraksh Pareek<EMAIL_ADDRESS>

Looks good to me, good work @DelusionalOptimist


Error when building cartographer:ros_log_sink.cc

In function cartographer_ros::ScopedRosLogSink::send(int, char const*, char const*, int, tm const*, char const*, unsigned long)': ros_log_sink.cc:(.text+0x144): undefined reference to google::LogSink::ToString[abi:cxx11](int, char const*, int, tm const*, char const*, unsigned long, int)'
I have never come across this problem in previous version. Any help would be greatly appreciated!

I solved this problem by upgradding the version of protobuf, thanks!

I solved this problem by upgradding the version of protobuf, thanks!

would you please share your version of protobuf? I met the same problem.THX!


OCI validation for unique employee email.

OCI validation for unique employee email.
Copy validation code from keypay-integration.
https://trello.com/c/JrRtBGVe/1201-p1-oci-validation-for-unique-employee-email

@thukim please help review my code

Reviewed 👍


Allow remote autofill toggling for all users

Task/Issue URL: https://app.asana.com/0/414235014887631/1204349262808609/f
Tech Design URL:
CC: @THISISDINOSAUR @amddg44
Description:
This PR makes the autofill subfeature toggleable for all users. The will remain hidden unless toggled to enabled on the privacy config.

Steps to test this PR:

Check autofill features are invisible when not internal
Check autofill features are visible when internal
Become non-internal (reinstall app), mock privacy config response enabling autofill subfeatures using Charles / Proxyman (or similar) and check that they are visible.

For making sure the privacy-config is re-fetched after you make a change to your mocked response:

Make sure you increment the eTag in the response header each time you change it
Tweak the AppConfigurationFetch.Constants.minimumConfigurationRefreshInterval as it’s set to 30 mins by default.
(You might need to relaunch the app to refresh the app’s UI state.)

Copy Testing:

[ ] Use of correct apostrophes in new copy, ie ’ rather than ’

Orientation Testing:

[ ] Portrait
[ ] Landscape

Device Testing:

[x] iPhone SE (1st Gen)
[ ] iPhone 8
[ ] iPhone X
[x] iPhone 14 Pro
[x] iPad

OS Testing:

[x] iOS 14
[ ] iOS 15
[x] iOS 16

Theme Testing:

[ ] Light theme
[ ] Dark theme

—
Internal references:
Software Engineering Expectations
Technical Design Template

I have no idea if it's related, but when trying to test email (right after logging in) I hit this assertion failure in BSK.APIRequest
private func assertUserAgentIsPresent() {
guard request.allHTTPHeaderFields?[HTTPHeaderField.userAgent] != nil else {
assertionFailure("A user agent must be included in the request's HTTP header fields.")
return
}
}

@THISISDINOSAUR Excuse the delay. Two things:

Make sure you increment the eTag in the response header each time you change it
Tweak the AppConfigurationFetch.Constants.minimumConfigurationRefreshInterval as it’s set to 30 mins by default.
(3.) You might need to relaunch the app to refresh the app’s UI state.

I did #1 and #3, re 2 I reset the refresh time using the debug panel which I figure should have the same impact, but I guess not? Either way I'll try #2

I tested a variety of things (e.g. sub features enabled but parent feature disabled, various mixes of sub features) and everything LGTM!


Add trait for compatibility with 3.59.2 and 3.59.3

Can be a quick fix for https://github.com/erickskrauch/php-cs-fixer-custom-fixers/issues/12
(I cannot run tests and code style locally on master branch)

@erickskrauch Can you release it as 1.2.5 ? Thank you :pray:

@alamirault, done ;)


kubernetes: fix startup by setting systemd as cgroup driver

Motivation for this change
https://github.com/NixOS/nixpkgs/pull/108960#issuecomment-770993006
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

I don't know if this is difficult, but could we have a test for those cases thinking ?

There is a test for kubernetes. I am sure it broke in the process.

We could add the kubernetes test to docker.pussthru.tests.

We could decide it based on the stateVersion in the module and set the system to use the old cgroup API if it's an old stateVersion.

Seems like the right fix because it also improves stability according to the docs, but does require instructions in the release notes, because of the following:

Changing the cgroup driver of a Node that has joined a cluster is strongly not recommended.
If the kubelet has created Pods using the semantics of one cgroup driver, changing the container runtime to another cgroup driver can cause errors when trying to re-create the Pod sandbox for such existing Pods. Restarting the kubelet may not solve such errors.

If you have automation that makes it feasible, replace the node with another using the updated configuration, or reinstall it using automation.

-- kubernetes.io/docs/setup/production-environment/container-runtimes/#cgroup-drivers

Not sure when I get back to this. When somebody else beat me to write the release notes I will cherry-pick from there.

Closing this for now for someone else to pick up.


Sentences with capital letter support

Closes #35

I'll close this in favor of #47, which opens up for mixed case keywords. Regardless, thanks for coming up with a solution for an issue from another user!


Infinite loop from useEffect()

Memory leak detected. Passing an empty array as the second argument to useEffect makes it only run on mount and unmount, thus stopping any infinite loops.

Can this be merged please? The console error pops up all the time.

Thanks @mqtik !

Thanks @mqtik !


Creating file share size greater than total pool capacity should not be allowed

Is this a BUG REPORT or FEATURE REQUEST?:
Creating file share size greater than total pool capacity should not be allowed

Uncomment only one, leave it on its own line:
/kind bug
/kind feature

What happened:
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Hotpot(release/branch) version:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

DB entry is creating before selecting pool so it can't be fixed, it is depend on #870


Ionic model select on ionic popover

I have an ionic popover with a form. In that form i want to use ionic model select for country selection.But when i open the ionic model select it apears behind the ionic popover back drop,Is there any solution?

Hi @anoopmm , sorry for the late answer.
Probably we could set an optional z-index option in order to override the one provided by ionic modal.
what do you think about?

What the update on this? Has I will like to use this in a popover and it shows the list straight away without you haven't to click on anything


Updated the plugin for 4.20.3

Only the plugin is updated, not the actual example project. As far as I can see the example project isn't really useful for anyone to download as the screenshots for how to use it are already made, but people can use the plugin in their 4.20.3 projects now.

Thanks!


Add support for workflow as code

Done: https://github.com/conductor-sdk/conductor-csharp/releases/tag/0.1.0


Update Widgets version to include security fix

This PR is a replacement for #208.

Since this is a small change but a necessary security fix, in the interest of keeping things simple, this PR won't bump the version.


General Home Page

Please do work for this task in a branch called issue-009.
Implement the general homepage UI, which will consist of three columns of information on Drivers, Riders, and FastRide.


Sendfile test crashes on FreeBSD due to casting 32-bit integer pointer to off_t which is 64-bit on AMD64

Hi,
The FreeBSD stack check reports that the sendfile test writes beyond its allocated stack. This simple patch fixes the issue for FreeBSD and possibly also other platforms aswell.
netperf.txt
Thank you!
--HPS // Mellanox // Nvidia

Ping


Publish results task optimization when merge test result is opted.

Publish results task optimization when merge test result is opted

Publishing in parallel to a single run.
Publishing in parallel with result file parsing.
Controlling the number of parallel tasks to MAX 10.

@navin22 do we still need this?


made the project compatible with netstandard2.0

Description
just 4 file was affected and the changes are very small.

Fix compatibility for netstandard20 without compromising the project or code.

https://github.com/DomCR/ACadSharp/pull/506


Support common filename for source and minified JS dependencies

It is currently possible to avoid including different JS(impl) files in index.html with:
Compile / fastOptJS / artifactPath := (Compile / fastOptJS / target).value / "frontend.js",
Compile / fullOptJS / artifactPath := (Compile / fullOptJS / target).value / "frontend.js",

The same approach cannot be applied to JS dependencies, since fullOptJS does not only depend on packageMinifiedJSDependencies but also to packageJSDependencies due to org.scalajs.sbtplugin.ScalaJSPluginInternal#scalaJSStageSettings:
key := key.dependsOn(packageJSDependencies, packageScalaJSLauncherInternal).value,

The latter overwrites the result of the former when their artifactPath is the same.
I've made some attempts with skip, but I'm not sure it's possible to only skip packaging in fullOptJS, e.g.:
skip in (Compile / fullOptJS / packageJSDependencies) := true

skips in fastOpt as well.
I'd expect that since fullOptJS only outputs minified JS files, it should only package minified dependencies, but obviously there are a few possible solutions (one of which is me using a separate SBT module for full packaging and skipping packaging JS dependencies there, but I'd rather avoid that :wink:)

I'm afraid the whole jsDependencies-related tasks are on life support. We do not change them anymore.
We encourage users to use scalajs-bundler instead.


Replace standard UIAlertController w/ FCAlertView

Source: https://github.com/k9101/FCAlertView

Solution: https://github.com/yoha/Notes/commit/ea7d123ac8be63a73d09d798341539ea972de508


ignition: Allow Ignition V1/V2, ignore template file extension

Allow V1 and V2 Ignition JSON
Allow V1 and V2 Ignition YAML
Stop requiring Ignition templates to use a file extension to specify the rendered format (#176)
Relates to #141

This change merely allows Ignition V1/V2 YAML/JSON to be used to begin migrating. Other Ignition 2.0.0 upgrade issues are tracked in #141.
cc @crawford


Add a creative Error 404 page (HTML & CSS required only)

This project uses Google material design light CSS.
Create a creative, stylsh Error 404 page.
The file for 404 page is here you need to edit only this file.
Your content goes in between line 16 to line 40

Can I work on this.

@samthom  sure you can 👍.


Different backup/prune schedule for development environments

Currently all production and development environments share the same backup and prune schedule.
It may be good to have a different schedule available for development environments

This is causing heavy load on clusters with many development environments combined with a short custom interval between backups.

ah, so digging in, there is a feature shortfall here. The remote-controller allows for the setting of a cluster-wide default schedule/retention, and the build-deploy scripts allow this to be overridden for production only on a per-project basis.
So we will need to add functionality in two places:

in the remote-controller to allow for the addition of additional production and development-specific overrides.
in the build-deploy script to check for the existence of any of these variables before determining the backup schedule for any given environment. Maybe we take the opportunity to factor out the backup schedule into a separate script, as we did for route processing.

Is the order of precedence:
Lagoon-defined prod specific > Cluster-defined prod/dev specific > cluster default?

I think k8up will solve part of the problem.

I think k8up will solve part of the problem, but not all of it.
If a customer has changed the default backup schedule at the cluster level to be, say, every 6 hours. And there are many projects that have more than 10 environments configured, you have the potential for a large number of unnecessary backup pods for environments that probably don't need backups every 6 hours, with or without k8up configured correctly.


[TW#25620] Heap Corruption

Environment

Development Kit:      [ESP32-Wrover-Kit| UBLOX NINA W102]
Kit version (for WroverKit): [v3]
Core (if using chip or module): [ESP32-Wrover-I| UBLOX NINA W102]
IDF version (git rev-parse --short HEAD to get the commit id.):
//b85e8e10b90ab7089ec7989ba513f7a2235f9ea4
//8d3d3fdd4ae1a94daf599545c26848495b2dfcf1
Development Env:		[Make]
Operating System:		[Windows|MacOS]
Power Supply:         [USB]

Problem Description
Long running code caused CORRUPT HEAP issues in malloc calls after a period of runtime in our main codebase. Attempted diagnostics using the heap_caps_check_integrity_all and  the heap tracing instructions.
If heap safety checks are turned to minimum, the code runs for a varying amount of time between 10-45 seconds before crashing out (in that time around 200-400 malloc's work).
When trying to debug the firmware using the above checks heap_caps_check_integrity_all would cause a crash whenever it was called, even if called at the beginning of app_main()
Then went and tested on the hello_world demo example, by enabling full heap checking and calling heap_caps_check_integrity_all at the beginning of the app_main code. This reproduced the same issue.
Tested on master (as of time of writing) and issue was reproducible. So also tested using the UBLOX NINA module and the issue is reproducible on that module as well. Also tested on a mac computer as well, and issue was reproducible.
Not certain if this is a config issue for the menuconfig or possible bug in the heap implementation?
Expected Behavior
Actual Behavior
Steps to repropduce

Compiled attached files
Flashed to different boards
Watch monitor for crash messages

Code to reproduce this issue
#include <stdio.h>
#include "esp_heap_caps.h"
#include "esp_heap_trace.h"
#include "esp_spi_flash.h"
#include "esp_system.h"
#include "freertos/FreeRTOS.h"
#include "freertos/task.h"
void app_main() {
  heap_caps_check_integrity_all(true);
  heap_caps_print_heap_info(MALLOC_CAP_DEFAULT);
  printf("Hello world!\n");

  /* Print chip information */
  esp_chip_info_t chip_info;
  esp_chip_info(&chip_info);
  printf("This is ESP32 chip with %d CPU cores, WiFi%s%s, ", chip_info.cores, (chip_info.features & CHIP_FEATURE_BT) ? "/BT" : "", (chip_info.features & CHIP_FEATURE_BLE) ?
I (90) cpu_start: Single core mode
I (91) heap_init: Initializing. RAM available for dynamic allocation:
D (94) heap_init: New heap initialised at 0x3ffae6e0
I (99) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
D (106) heap_init: New heap initialised at 0x3ffb37c8
I (110) heap_init: At 3FFB37C8 len 0002C838 (178 KiB): DRAM
I (116) heap_init: At 3FFE0440 len 00003BC0 (14 KiB): D/IRAM
I (123) heap_init: At 3FFE4350 len 00013CB0 (79 KiB): D/IRAM
I (129) heap_init: At 3FFFC000 len 00004000 (16 KiB): D/IRAM
D (138) heap_init: New heap initialised at 0x40089578
I (140) heap_init: At 40089578 len 00016A88 (90 KiB): IRAM
I (147) cpu_start: Pro cpu start user code
D (159) clk: RTC_SLOW_CLK calibration value: 3006144
D (168) intr_alloc: Connected src 46 to int 2 (cpu 0)
D (169) intr_alloc: Connected src 57 to int 3 (cpu 0)
D (170) stack_chk: Intialize random stack guard
D (174) intr_alloc: Connected src 24 to int 9 (cpu 0)
I (179) cpu_start: Starting scheduler on PRO CPU.
D (185) heap_init: New heap initialised at 0x3ffe0440
D (185) heap_init: New heap initialised at 0x3ffe4350
D (185) heap_init: New heap initialised at 0x3fffc000
D (195) intr_alloc: Connected src 16 to int 12 (cpu 0)
CORRUPT HEAP: Block 0x3fffc010 is outside heap (last valid block 0x00000000)
Heap summary for capabilities 0x00001000:
  At 0x3ffae6e0 len 6432 free 24 allocated 5976 min_free 24
    largest_free_block 24 alloc_blocks 24 free_blocks 1 total_blocks 25
  At 0x3ffb37c8 len 182328 free 171564 allocated 10540 min_free 171564
    largest_free_block 171564 alloc_blocks 11 free_blocks 1 total_blocks 12
  At 0x3ffe0440 len 15296 free 15248 allocated 0 min_free 15248
    largest_free_block 15248 alloc_blocks 0 free_blocks 1 total_blocks 1
  At 0x3ffe4350 len 81072 free 81024 allocated 0 min_free 81024
    largest_free_block 81024 alloc_blocks 0 free_blocks 1 total_blocks 1
assertion "next > (intptr_t)block" failed: file "/Users/fred/esp/esp-idf/components/heap/./multi_heap.c", line 116, function: get_next_block
abort() was called at PC 0x400d203f on core 0
0x400d203f: __assert_func at /Users/ivan/e/newlib_xtensa-2.2.0-bin/newlib_xtensa-2.2.0/xtensa-esp32-elf/newlib/libc/stdlib/../../../.././newlib/libc/stdlib/assert.c:63 (discriminator 8)

Backtrace: 0x40087480:0x3ffb4bc0 0x4008757f:0x3ffb4be0 0x400d203f:0x3ffb4c00 0x40086db6:0x3ffb4c30 0x400871b9:0x3ffb4c50 0x400d1e11:0x3ffb4c70 0x400d1f7e:0x3ffb4cb0 0x400d07ba:0x3ffb4ce0
0x40087480: invoke_abort at /Users/fred/esp/esp-idf/components/esp32/./panic.c:572

0x4008757f: abort at /Users/fred/esp/esp-idf/components/esp32/./panic.c:572

0x400d203f: __assert_func at /Users/ivan/e/newlib_xtensa-2.2.0-bin/newlib_xtensa-2.2.0/xtensa-esp32-elf/newlib/libc/stdlib/../../../.././newlib/libc/stdlib/assert.c:63 (discriminator 8)

0x40086db6: get_next_block at /Users/fred/esp/esp-idf/components/heap/./multi_heap.c:116
 (inlined by) multi_heap_get_info_impl at /Users/fred/esp/esp-idf/components/heap/./multi_heap.c:690

0x400871b9: multi_heap_get_info at /Users/fred/esp/esp-idf/components/heap/./multi_heap_poisoning.c:315

0x400d1e11: heap_caps_print_heap_info at /Users/fred/esp/esp-idf/components/heap/./heap_caps.c:389

0x400d1f7e: app_main at /Users/fred/Downloads/hello_world/main/./hello_world_main.c:21

0x400d07ba: main_task at /Users/fred/esp/esp-idf/components/esp32/./cpu_start.c:449

================= CORE DUMP START =================

Other items if possible

[X] sdkconfig file (attach the sdkconfig file from your project folder)
[X] elf file in the build folder (note this may contain all the code details and symbols of your project.)
[X] coredump (This provides stacks of tasks.)

Files in the zip
hello-world.zip
EDIT:
When running on the larger code base this is the issue that is currently showing up that started all of this:
CORRUPT HEAP: multi_heap.c:394 detected at 0x3ffbd9f0
abort() was called at PC 0x4008b940 on core 1
0x4008b940: multi_heap_assert at C:/msys32/home/ralim/esp/esp-idf/components/heap/multi_heap_platform.h:55
 (inlined by) multi_heap_malloc_impl at C:/msys32/home/ralim/esp/esp-idf/components/heap/multi_heap.c:394

Backtrace: 0x4008c064:0x3ffbd720 0x4008c157:0x3ffbd740 0x4008b940:0x3ffbd760 0x40084dbc:0x3ffbd780 0x40084dfd:0x3ffbd7a0 0x400852bd:0x3ffbd7c0
 0x400854ed:0x3ffbd800 0x400d5c2b:0x3ffbd820
0x4008c064: invoke_abort at C:/msys32/home/ralim/esp/esp-idf/components/esp32/panic.c:572

0x4008c157: abort at C:/msys32/home/ralim/esp/esp-idf/components/esp32/panic.c:572

0x4008b940: multi_heap_assert at C:/msys32/home/ralim/esp/esp-idf/components/heap/multi_heap_platform.h:55
 (inlined by) multi_heap_malloc_impl at C:/msys32/home/ralim/esp/esp-idf/components/heap/multi_heap.c:394

0x40084dbc: heap_caps_malloc at C:/msys32/home/ralim/esp/esp-idf/components/heap/heap_caps.c:116

0x40084dfd: heap_caps_malloc_default at C:/msys32/home/ralim/esp/esp-idf/components/heap/heap_caps.c:149

0x400852bd: trace_malloc at C:/msys32/home/ralim/esp/esp-idf/components/heap/heap_trace.c:324

0x400854ed: __wrap_malloc at C:/msys32/home/ralim/esp/esp-idf/components/heap/heap_trace.c:392

0x400d5c2b: devGPS::gpsThread() at C:/Users/Ralim/Documents/GitHub/propeller/aeropoints-firmware/esp32/aeropoint/main/ubloxGPS.cpp:266

@Ralim
For first part, there is issue in trax feature (enabled in your sdkconfig CONFIG_ESP32_TRAX) with respect to single core mode of operation, it reserves incorrect memory bank and hence results in heap corruption. Fix for that should appear soon in ESP-IDF.
For original issue of heap corruption in longer run, I am not sure but if you had kept this feature always enabled (and not just for debugging purpose), then it might fix your problem. I will notify once fix gets merged and maybe you can retest your use-case.

@Ralim
Please try with https://github.com/espressif/esp-idf/commit/902344c5163ff2e72b2f98996d170c5c09be718e

Hi @mahavirj ,
Looks to be working well at the moment.
I'll be back if it comes back up again.

CORRUPT HEAP: multi_heap.c:429 detected at 0x3ffd5cb8
abort() was called at PC 0x4008ebb3 on core 0
Any suggestion???


NotImplementedError on EntityFormField

I am using the latest version of Rasa and when I try to run my actions as an endpoint:
python -m rasa_core_sdk.endpoint --actions actions
I keep on getting an NotImplementedError
I realised that even if I have one line of code in the actions.py file:
from rasa_core_sdk.forms import EntityFormField
I still get this error!
Is that a bug?

Sorted!... I only installed the sdk today but it did not install the latest version, so then I thought of running:
pip install --upgrade rasa_core_sdk
and that fixed the issue


The JobAccountant is the most hard working component in the system but it needs a tune-up. In a high load MC agent I discovered backlogs as big as 2M jobs.
There must be some improvements we can do to this guy.

Getting back to this a few years later :) Indeed it does some heavy work and apparently it's performance is in the order of 10k jobs per hour (really?!? what a let down)
vocms0126 accumulated a backlog of 170k jobs, and we can see it took around 18h to go through them.
2017-09-07 11:11:44,292:140109560870656:INFO:JobAccountantPoller:Found 170089 completed jobs
2017-09-08 05:31:02,054:140109560870656:INFO:JobAccountantPoller:Found 82627 completed jobs

@ticoann since Erik G. is waiting for the new draining status update. Maybe this one would be a good candidate to get him involved?

The impact of very short jobs (<=1h), something like 15k jobs completed within an hour:
cmst1@vocms0253:/data/srv/wmagent/current $ grep 'completed jobs' install/wmagent/JobAccountant/ComponentLog | tail -n20
2018-08-06 12:14:14,414:139642164897536:INFO:JobAccountantPoller:Found 357543 completed jobs
2018-08-08 01:59:39,016:139642164897536:INFO:JobAccountantPoller:Found 569940 completed jobs
2018-08-10 09:53:06,364:139642164897536:INFO:JobAccountantPoller:Found 645218 completed jobs

Despite the vintage of this issue, @amaltaro confirms this is an issue that needs to be tackled before HL-LHC.


Unable to run tests without a host application

I am using fbsimctl via pxctest which I believe uses fbsimctl as the underlying mechanism for executing UI and none UI unit tests (the xctest files exist as plugins within the Project.app bundle.)
There appears currently to be no way to execute an xctest bundle directly, which is the resultant artifact of building a XCTest target that is not hosted within an application.
Is this functionality going to be added to fbsimctl or am I missing something?
Any advice or suggestions is greatly appreciated! Thanks a lot.

using fbsimctl via pxctest which I believe uses fbsimctl as the underlying mechanism

pxctest uses FBSimulatorControl, but not fbsimctl.
However if you want to see this feature in pxctest, you should probably open an issue there.

@plu thanks for the clarification, feel free to close this issue.

There is framework-level support for running "Logic Tests" (i.e. tests that run in the xctest command line rather than a hosted .app) in fbxctest. This is not currently part of FBSimulatorControl, however it probably should be hoisted to XCTestBootstrap at some point in the future.
You will find support for running Application Tests in FBSimulatorControl, just not logic tests.


fix: discord presence option was gone

So, a bit after 1.5.0 released, I noticed that something was missing while looking at presence activity code..
Couldn't point my finger on it until I wanted to get the activity but for some reason wasn't able to thanks to disabled Discord Presence, but later discovered I couldn't enable it back on since the setting was missing from the options menu..

oh oopsies,
Thank you! :D


Remove duplicate jobs from collection

When having less than 20 jobs in the database, the currently running job is duplicated in the returned list. This fixes that by removing duplicate jobs from the list before returning them.
Fixes #352

Nice!


Map child collection item?

I have this code:
(from hs in ctx.Codes
		from -->hsd in hs.Descriptions
		where hs.ParentId == query.Id
			&& -->hsd.LanguageId == query.LanguageId
		select hs)
new CodeForTreeDto
	{
		Id = hs.Id,
		Code = hs.Code,
		HsLevel = hs.HsLevel,
		ParentId = hs.ParentId,
		ParentCode = hs.Parent.Code,
		IsLastLevel = hs.IsLastLevel,
    --> Description = hsd.Description,
	})
.AsNoTracking().ToListAsync();

This will map the hs.Descriptions.Description property without emitting select N + 1.
But I have to do it manually.
How can I map it using Mapster.ProjectToType, so that I don't get a select N + 1?
If I'm using Include(hs => hs.Descriptions) in the IQueryable and
TypeAdapterConfig<Code, CodeForTreeDto>
		.NewConfig()
		.Map(dto => dto.Description,
		--> ent => ent.Descriptions.First().Description)
     .Config

I will get a select for each ent.Descriptions.First().

I haven't tried yet, but this should work
In Startup
TypeAdapterConfig<Code, CodeForTreeDto>.NewConfig()
    .Map(dest => dest.Description,
         src => src.Descriptions
             .Where(it => it.LanguageId == MapContext.Current.Parameters["languageId"])
             .Select(it => it.Description)
             .First());

In Controller
using (var scope = new MapContextScope())
{
    scope.Context.Parameters["languageId"] = query.LanguageId;
    var results = ctx.Codes.ProjectToType<CodeForTreeDto>();
}


config(ticdc): adjust config name

What problem does this PR solve?

Issue Number: ref #8393
What is changed and how it works?
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
No
Do you need to update user documentation, design documentation or monitoring documentation?
No
Release note
None

/run-all-tests

/run-all-tests

/merge

/run-verify


Added guards for test libraries

added guards for test libraries

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I believe the issue has already been addressed by https://github.com/aws-robotics/utils-common/pull/39. If so, please don't merge this in unnecessarily.


How to Migrate Xamarin.Android Projects whose min-sdk is 19?

Type of issue
Missing information
Description
So, I was trying to follow the tutorial to migrate my app to .NET Maui from a Xamarin.Android app
but to my surprise apparently Android KitKat isn't supported, and this API Level is being used by an important chunk of devices that are using my app,
so, I guess in the documentation this should be mentioned or providing an alternative solution for this case,
thank you!
and I'm trapped because I wanted to give support to API Level 34 but my Xamarin.Android project doesn't allow it, it only allows it to API Level 33
What would be the options for this case?
Page URL
https://learn.microsoft.com/en-us/dotnet/maui/migration/native-projects?view=net-maui-8.0
Content source URL
https://github.com/dotnet/docs-maui/blob/main/docs/migration/native-projects.md
Document Version Independent Id
56709a6f-c958-f4ce-1c74-65e20b384611
Article author
@davidbritch
Metadata

ID: 56709a6f-c958-f4ce-1c74-65e20b384611
Service: dotnet-mobile
Sub-service: dotnet-maui

The .NET for Android minimum version is Android 5.0 (API 21).
For targeting API 34 features, that shipped a while ago for .NET for Android (https://github.com/xamarin/xamarin-android/releases/tag/34.0.95) so file a bug report if you're having issues there.
Remaining with Xamarin.Android, while unsupported, you can continue to target API 19 and build with API 34 in order to be Play Store compliant. New features in API 34 may not be part of that SDK binding (hence you need to use .NET for Android instead). See the support policy at https://dotnet.microsoft.com/en-us/platform/support/policy/xamarin

Thank you, I will try to give support using Xamarin Android for the time being, but as for the document, I really think this should be mentioned, to avoid following a tutorial that will fail in cases like mine! that probably I'm not the first or last that will come across this issue!


NVIDIA GPU Driver Extension for Linux RHEL 7.8

Hello, I was wondering if the NVIDIA GPU Driver Extension for Linux works with RHEL 7.8?  If not, is there a time line when it will be available?  Thank you.

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

ID: 374c8361-9b31-2488-f04b-766e6305c91b
Version Independent ID: aebb9dd3-d1e6-4319-9ab1-0088bd8416e3
Content: NVIDIA GPU Driver Extension - Azure Linux VMs
Content Source: articles/virtual-machines/extensions/hpccompute-gpu-linux.md
Service: virtual-machines-linux
GitHub Login: @vermagit
Microsoft Alias: akjosh

Thanks for the question. We are currently investigating and will update you shortly.

@mbiver Looks like this issue was resolved with https://github.com/MicrosoftDocs/azure-docs/issues/55818
We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


Span tooltip not working in Chrome.

This fiddle works in Firefox and IE but not in Chrome. The span's tooltip is not shown.

Hello,
"Hello, please, thank you" are appreciated words.
The disabled button catches mouse events, there is nothing to do about it and it's not a tooltipster issue.
Thank you

@louisameline
Hello, I wanted to keep it short, sorry!
When setting pointer-events: none it works in Firefox and Chrome: https://jsfiddle.net/jdbLedwn/4/
Thank you!

No worries. Nice trick! Thanks for sharing.


Is it usable from browser?

I am using ajv validation from browser. By adding the following code in the HTML page.

Is there any way or possible to use ajv-errors in the HTML page. I had tried to use by the following default in the html page. But it is not working.
var ajv = new Ajv({allErrors: true}); // options can be passed, e.g. {allErrors: true}
var schemaForRegCheck ={
"type": "number",
"minimum": 10,
"additionalProperties": false,
"errorMessage": 'this is a custom error messages'
}
Thank you advance.

Yes, but the bundle is not supplied - it is best to bundle together with your application code.

Did you get it working in browser? I'm using it in following way, but I don't get any custom error messages!
import Ajv from<EMAIL_ADDRESS>import ajvErrors from<EMAIL_ADDRESS>
const ajv = new Ajv({
    allErrors: true,
    strict: false
});

ajvErrors(ajv);
const validator = ajv.compile(mySchema);
const valid = validator(json);

if (!valid) {
    console.log(validator.errors);
}


⚠ Slack has degraded performance

In 7c32d78, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 57 ms

Resolved: Slack performance has improved in da1e1e7.


ci: remove k8s 1.30 tests check

Describe what this PR does
k8s 1.30 tests were excluded for release-v3.11 but
mergify has status check condition for k8s 1.30 tests.
This commit removes these checks from mergify.
Related issues
Closes: #4625
Checklist:

[x] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
[x] Reviewed the developer guide on Submitting a Pull Request
[x] Pending release notes updated with breaking and/or notable changes for the next major release.
[x] Documentation has been updated, if necessary.
[x] Unit tests have been added, if necessary.
[x] Integration tests have been added, if necessary.

Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

/retest ci/centos/<job-name>: retest the <job-name> after unrelated
failure (please report the failure too!)

We also need to backport this to v3.11

@iPraveenParihar mergify rules are checked on only main/devel branch, if required we can backport it later.

@mergifyio queue


Add curl to install steps

Edits README installation section to specify downloading the setup.py script via wget or curl.

Thanks for contribution 🎉


UI unreadable sometimes

As shown in the image, with some covers the text on the bottom right is unreadable. Windows has this readability thing in the accent color menu. You cant pick some colors there, because they would cause unreadability.  I also don't know how you would calculate colors but well. Maybe somebody can help. I tried to look it up but I can't find anything because I have no idea how to google that issue lmao

this should not be a problem now with the new ambient background system


Remove import of Schema in migration stubs.

Default app/config.php contains a Schema alias to the facade.
So an import is unnecessary.

I think the purpose is to avoid relying on the "root aliases".

@vlakoff I don't know where the Schema is used in the framework elsewhere but then the alias is redundant if its not used by default?

@vlakoff If it's true, then you can delete my pr.

Yes, people sometimes remove those aliases.


Project naming

I'd love a way to name projects. Right now, the projects display using the local folder name. I have a over a dozen different projects in Brackets, and when I go to switch to a different project, it is difficult to find the one I need due to the way we name our project folders. Final web sites are put in a folder called 'www-'. So I see a long list of 'www-' in the Open Folder/Projects dropdown.
In Project Settings, I think there should be another field for the Project Name. Then, that name would appear in the project dropdown as well as at the top of the file sidebar.

+1 for this

+1 for this. I got a long list of the same named folders too ((

As a temporary solution, there is currently this:
https://github.com/petetnt/brackets-name-a-project
seems to work pretty well, though I would like if the name was stored in the brackets.json, but I guess we can't have everything :)

As there is already an extension that takes care of this- lowering priority. I have moved this issue to backlog for later consideration.

@mihawk90 The names should be saved to brackets.json... there was a short while between 0.0.1 and 0.0.3 when the preferences weren't correctly saved, but 0.0.3. should have fixed that. However if you still have that issue, file an issue to that repo and I'll investigate asap :+1:


Use storage stub for volatile storage in manifest

@shans Another question is about the failing integration test - this is the gifts test, and when mapping a person (in "Buy gifts for Claire"), the await newStore.cloneFrom(copiedStore); in Arc::instantiate fails with:
assert(false, shouldn't have rawData ${JSON.stringify(value.rawData)} here);
because the newStore is pouch (with referenceMode=true) and the copiedStore is volatile (with referenceMode=false).
I'm not sure what is the right approach here either.


[BULK] DocuTune - Rebranding

Global effort for the Microsoft Learn platform rebrand
The Microsoft team is applying changes to content for the Microsoft Learn platform rebrand. Changes include modifications to current Microsoft Learn and Microsoft Docs terminology as well as associated domain name and content URL path changes.
Please review this draft PR and provide any necessary comments as soon as possible. Because these fixes are tightly scoped, we have approval to automatically merge these changes to the default branch for publication.
This PR was generated via DocuTune. It includes only targeted fixes and minor formatting adjustments.
CorrelationId: d8c4c3ac-d49a-4c95-9ddc-9822ca297907
InitiativeId: docutune-rebranding-2022-09
PointOfContact: Alex Buck

#docutune-draft

Docs Build status updates of commit bd4b6dd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Community/accessibility-by-design.md
:white_check_mark:Succeeded
View

Community/glossary.md
:white_check_mark:Succeeded
View

Community/microsoft365-maturity-model--staff-and-training.md
:white_check_mark:Succeeded
View

Community/power-automate-vs-logic-apps.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

#docutune-ready


[ci:component:github.com/gardener/gardener-resource-manager:v0.21.0->v0.22.0]

*Release Notes:
A problem with long running ManagedResource reconciliations caused by unavailable `APIServices` was fixed.

The client QPS and burst settings do now also apply for the uncached client.

The `.spec.loadBalancerIP` value for `Service`s is now preserved.

Gardener-Resource-Manager now adds latest warning events to a ManagedResource's `.status.conditions` in case a Kubernetes `Service` cannot be deleted. This allows to get more context about the underlying problem e.g., when Cloud-Controller-Manager cannot delete the backing load balancer.

If a ManagedResource refers to a `Service` object of type `LoadBalancer`, the Gardener Resource Manager now regularly checks if the `Service` has an `Ingress` status and contributes the result of this check to the `ResourcesHealthy` condition.

Go dependencies have been updated to:
github.com/gardener/gardener v1.16.0
sigs.k8s.io/controller-runtime v0.7.1
k8s.io/* v0.19.6

Gardener-Resource-Manager now needs permission to `get`, `create`, `update` and `watch` `Lease` objects named `gardener-resource-manager` in order to perform leader election. For a reference, please have a look at the pre-delivered Helm chart in `charts/gardener-resource-manager`.

The `CheckDaemonSet` function does now lead to more accurate results.

The controller reconciliations are now limited to `1m`.

With this release for all objects managed by a `ManagedResource` an annotation `resources.gardener.cloud/origin` is set describing the `ManagedResource` that caused this object to be created. The format of the origin annotation is `[cluster id:]namespace/object-name`. For multi-cluster scenarios the GRM can be started with a `--cluster-id` options to enable the extended annotation format (see https://github.com/gardener/gardener-resource-manager/blob/master/docs/concepts/managed-resource.md for further details).

/assign
/hold