Buffer leak in basic_parser::put

The version of basic_parser::put that takes a buffer sequence as an argument has a bug in the fallback code path (the one that is not optimized for single buffer sequences nor those whose size is smaller than max_stack_buffer).
The issue is that while buf_.get() has size buf_len_, it has only size number of valid octets. This causes some of the invalid octets (those with index >=size and <buf_len_) to "leak" from previous calls to put into the current one.
The fix is simple. I'm attempting to write a unit test, but that's a bit more challenging due to the fact that the buffers fed into the put function need to be crafted to avoid the optimized paths.
We noticed this bug in Boost 1.69, but it's still present in current 1.71 and the develop branch.

This is a case of me being too clever for my own good. The parser should simply require that the input is in a single contiguous buffer. In a future version of Beast I will likely just build the dynamic buffer into the parser itself and keep it flat to hide this implementation detail.

So for completeness, here is a code that reproduces the issue. It's quite ugly because it assumes a particular implementation of basic_parser::put(ConstBufferSequence const&,...) and thus I think it's best to just keep it here and not commit to unit tests(?).
    void
    testIssue1734()
    {
        // Ensure more than one buffer, this is to avoid an optimized path in
        // basic_parser::put(ConstBufferSequence const&,...) which avoids
        // buffer flattening.
        auto multibufs = [](auto buffers) {
            std::vector<net::const_buffer> bs;
            for (auto b : buffers) bs.push_back(b);
            while (std::distance(bs.begin(), bs.end()) < 2) {
                bs.push_back({});
            }
            return bs;
        };

        // XXX: This must be the same as basic_parser<...>::max_stack_buffer.
        // The latter is private however so we can't use it here.
        std::size_t constexpr max_stack_buffer = 8192;

        // Buffers must be bigger than max_stack_buffer to force flattening
        // in basic_parser::put(ConstBufferSequence const&,...)
        std::string first_chunk_data(2*max_stack_buffer + 1, 'x');

        std::string second_chunk_data_part1(max_stack_buffer + 2, 'x');
        std::string second_chunk_data_part2(max_stack_buffer + 1, 'x');

        multi_buffer b;
        parser_type<false> p;
        p.eager(true);
        error_code ec;
        std::size_t used;

        ostream(b) <<
            "HTTP/1.1 200 OK\r\n"
            "Server: test\r\n"
            "Transfer-Encoding: chunked\r\n"
            "\r\n";

        used = p.put(b.data(), ec);
        b.consume(used);

        BEAST_EXPECT(net::buffer_size(b.data()) == 0);
        BEAST_EXPECTS(!ec, ec.message());
        BEAST_EXPECT(!p.is_done());
        BEAST_EXPECT(p.is_header_done());

        ostream(b) <<
            std::hex <<
            first_chunk_data.size() << "\r\n" <<
            first_chunk_data << "\r\n";

        // First chunk
        used = p.put(multibufs(b.data()), ec);
        b.consume(used);

        BEAST_EXPECTS(ec == error::need_more, ec.message());
        BEAST_EXPECT(!p.is_done());

        ostream(b) <<
            std::hex <<
            (second_chunk_data_part1.size() +
             second_chunk_data_part2.size() ) << "\r\n" <<
            second_chunk_data_part1;

        // Second chunk, part 1
        used = p.put(multibufs(b.data()), ec);
        b.consume(used);

        BEAST_EXPECTS(!ec, ec.message());
        BEAST_EXPECT(!p.is_done());

        ostream(b) <<
            second_chunk_data_part2 << "\r\n"
            << "0\r\n\r\n";

        // Second chunk, part 2
        used = p.put(multibufs(b.data()), ec);
        b.consume(used);

        BEAST_EXPECTS(!ec, ec.message()); // <-- Error: bad chunk
        if(p.need_eof())
        {
            p.put_eof(ec);
            BEAST_EXPECTS(! ec, ec.message());
        }
        BEAST_EXPECT(p.is_done());
    }

Just add friend basic_parser_test;, this is one of the reasons I use a class for each test suite.

I'll incorporate it and make the adjustment, thanks!


How to implement evaluation for customdataset

Hello, I want to realize the evaluation of custom data set during training, but the program reports an error. I want to guide how to modify evaluation () function in custom_ dataset.py allows it to evaluate custom datasets. My dataset is labeled according to the kitti format, but I do not have calib, so you can understand that, except for xyzwhl class and angle, all other parameters are 0.
I copy the official evaltion() function which in custom_dataset.py to my custom_ dataset.py, and then run it directly. The following error occurs:
2022-11-23 14:57:08,407   INFO  Start evaluation custom_models/pointrcnn(default)
2022-11-23 14:57:08,408   INFO  Loading Custom dataset.
2022-11-23 14:57:08,409   INFO  Total samples for CUSTOM dataset: 74
2022-11-23 14:57:08,411   INFO  ==> Loading parameters from checkpoint /home/Downloads/OpenPCDet/OpenPCDet/output/custom_models/pointrcnn/default/ckpt/checkpoint_epoch_500.pth to GPU
2022-11-23 14:57:08,443   INFO  ==> Checkpoint trained from version: pcdet+0.6.0+f221374
2022-11-23 14:57:08,455   INFO  ==> Done (loaded 309/309)
2022-11-23 14:57:08,456   INFO  *************** EPOCH 500 EVALUATION *****************
eval: 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 37/37 [00:11<00:00,  3.25it/s, recall_0.3=(87, 88) / 90]
2022-11-23 14:57:19,855   INFO  *************** Performance of EPOCH 500 *****************
2022-11-23 14:57:19,855   INFO  Generate label finished(sec_per_example: 0.1540 second).
2022-11-23 14:57:19,855   INFO  recall_roi_0.3: 0.966667
2022-11-23 14:57:19,855   INFO  recall_rcnn_0.3: 0.977778
2022-11-23 14:57:19,855   INFO  recall_roi_0.5: 0.888889
2022-11-23 14:57:19,855   INFO  recall_rcnn_0.5: 0.944444
2022-11-23 14:57:19,855   INFO  recall_roi_0.7: 0.377778
2022-11-23 14:57:19,855   INFO  recall_rcnn_0.7: 0.544444
2022-11-23 14:57:19,855   INFO  Average predicted number of objects(37 samples): 1.054
Traceback (most recent call last):
File "train.py", line 227, in
main()
File "train.py", line 220, in main
dist_test=dist_train
File "/home/Downloads/OpenPCDet/OpenPCDet/tools/test.py", line 125, in repeat_eval_ckpt
result_dir=cur_result_dir
File "/home/Downloads/OpenPCDet/OpenPCDet/tools/eval_utils/eval_utils.py", line 128, in eval_one_epoch
output_path=final_output_dir
File "../pcdet/datasets/custom/custom_dataset.py", line 80, in evaluation
ap_result_str, ap_dict = kitti_eval(eval_det_annos, eval_gt_annos, self.map_class_to_kitti)
AttributeError: 'CustomDataset' object has no attribute 'map_class_to_kitti'
Although my dataset label is in kitti format, there is no calib file and image, so the corresponding values are all 0. How can I modify these functions to evaluate my dataset？

Please read the tutorial carefully.
https://github.com/open-mmlab/OpenPCDet/blob/master/docs/CUSTOM_DATASET_TUTORIAL.md

1.Of course I read this file，but it still reported this error when I added this in the yaml file.
2. So I changed the method and copied the evaluation () function in kitti, but a new error occurred :len(gt_annos) different from len(dt_annos),I debug and found that there are two more dictionaries in dt_annos than in gt_annos（score and alpha）,How can I correct this problem? Whether to add these two dictionaries in gt_annos or delete them in dt_annos？

I have now completed the create_infos、training and demo.py and can run successfully, but I cannot modify the evaluation to see the loss curve and evaluation index

Have you provide map_class_to_kitti in the config?

If you just want to record the loss, it is already recorded in the log and tensorboard. As for the mAP, we don't calculate the performance metric during training as it will cause a lot of time, but you can check the code in eval_utils.py if you want to add such feature.

Thank you very much, i have directly tried the complete modification method in repo. It can also work, but the following problems occurred during the evaluation. I hope you can help me。

When running train.py, the program will be stuck as soon as the evaluation part is reached。（But the first time I ran it, there were results，i will show you after. So I think this modification method which in repo is simple and feasible. What I learned in csdn before is complex.Thank you for your team work!!! This is exciting！！！）

Now i will show you the Screenshot of the only run result：

I have a question about this,because it only have the part result. I guess it may be because I have only one class,so i modify map_to_kitti to only {'ship':'Car'}. I want to ask you whether this is the reason, and whether the evaluation result of ship is correct after my modification. If I don't want the other two classes, how do I modify them?
I want to ask you whether this is the reason, and whether the evaluation result of ship is correct after my modification. If I don't want the other two classes, how do I modify them?
I think if these two problems can be solved, it should be a success. Thank you for your great team！！Looking forward to hearing from you.

------------------ 原始邮件 ------------------
发件人:                                                                                "open-mmlab/OpenPCDet"                                                                                @.>;
发送时间: 2022年11月26日(星期六) 下午3:06
@.>;
<EMAIL_ADDRESS>主题: Re: [open-mmlab/OpenPCDet] How to implement evaluation for customdataset (Issue #1197)
If you just want to record the loss, it is already recorded in the log and tensorboard. As for the mAP, we don't calculate the performance metric during training as it will cause a lot of time, but you can check the code in eval_utils.py if you want to add such feature.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Finally, I have another question. My training has been completed, but when I want to view Loss in tensorboard, the following error appears. How can I solve it?
And  this is the train.py's log，It's stuck in the evaluation. I don't know why, but when an updated pth is not generated, it can be evaluated。
Finally, I have another question. My training has been completed, but when I want to view Loss in tensorboard, the following error appears. How can I solve it?

------------------ 原始邮件 ------------------
发件人:                                                                                "open-mmlab/OpenPCDet"                                                                                @.>;
发送时间: 2022年11月26日(星期六) 下午3:06
@.>;
<EMAIL_ADDRESS>主题: Re: [open-mmlab/OpenPCDet] How to implement evaluation for customdataset (Issue #1197)
If you just want to record the loss, it is already recorded in the log and tensorboard. As for the mAP, we don't calculate the performance metric during training as it will cause a lot of time, but you can check the code in eval_utils.py if you want to add such feature.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Feature: Switch git account and log out

What feature or improvement do you think would benefit Files?
At the moment you can't log out easily of git. You need to open Windows credential manager and then delete the token.
I think we can improve the UX of git adding a git section in the settings, where the user can manage the account they want to use
Requirements

Add a Git section in settings
Add a button to Log out
Add a button to Sign in
Add a button to Switch account

Files Version
2.5.21
Windows Version
Windows 11Pro 10.0.22621.1992
Comments
No response

Add a Git section in settings

Should this have a separate page?

Add a button to Switch account

Can you be signed into multiple accounts at the same time?

Add a Git section in settings

Should this have a separate page?

We should use either Advanced page  or a new one

Add a button to Switch account

Can you be signed into multiple accounts at the same time?

I guess we can do it. We might have a popup when there are two or more accounts

If you do add a Settings Page - I have made a Toggle Icon for it

@ferrariofilippo is "Switch account" the same as logging out?

@ferrariofilippo is "Switch account" the same as logging out?

Actually not but it may be confusing. I think it's better to simply log out.


Processing file build/app/outputs/flutter-apk/app-release.apk
target file name: karmm.apk
2023/10/08 17:30:22 Unable to retrieve files: googleapi: Error 404: File not found: ., notFound**

Same error on me when using overwrite option.
Unable to retrieve files: googleapi: Error 400: Invalid Value, invalid

Same here:
Error: creating/updating file failed with error: googleapi: Error 404: File not found: ***., notFound
with<EMAIL_ADDRESS>Did you have any workaround?

I was in the same situation.
I gave write permission to write the email address of the service account for the folder I was uploading to.
As a result, it uploaded successfully. If you haven't tried this, I suggest you do.


Release 0.16

master now contains a refactoring which separates Celluloid::Cell from Celluloid::Actor
It'd be great to release this soon, lest it langer on master without exercise from Celluloid users. I think this refactoring is important for Celluloid's future and maintainability.
There are some lingering issues we need to get resolved though. I don't know them offhand @halorgium, perhaps you could fill us in? :wink:

This is long since done/past. Remainder is handled in #536 it seems.


Bugfix: fix config check

The docker container was failing to start.

LGTM! This should have been part of https://github.com/Couchers-org/couchers/pull/270 but I did inadequate testing there.


is this project still alive???

too many PR and issues without answer. is this project alive?

Somewhat.  I'm trying to help keep the project going with existing features... not so sure it should take on all these new features.  And I don't have time to test them all.
@rebeccahughes is pretty busy and can't maintain the project.
Ideally, we'd have a good way to test the features and make sure they aren't just bloat, but that's not something easy to determine.


Highways: Hard shoulder / emergency lane

In mod options, there's a setting to make Heavy trucks prefer outer lanes on highways.
However, many highways will have an outside lane that's used as an emergency 'hard shoulder' (eg. where broken down cars try and get to so they're not blocking traffic while they wait for service van).
Would it be possible to add an additional option (Policies tab in mod settings), something like:
--[ ] Outermost highway lanes are emergency lanes
The effect would be to restrict all but emergency traffic (police/ambulance/fire/disaster) from using the outermost lane via 'virtual vehicle restrictions' (in much the same way that the Ban private cars and trucks on bus lanes  option does.
Doing it this way would allow users to optionally allow vehicles back on to outermost lane for some highways, eg. those that don't have an emergency lane.
One problem might be what happens if user doesn't have TM:PE installed but uses a highway asset that has hard shoulder? I was thinking if the hard shoulder was a bus lane in the road asset, that users without TM:PE would get an "almost hard shoulder" (busses, taxis, etc, would still use it, but it would be mostly clear of traffic).
What  do you think?

I am not sure if I understand what you want to do :confused: . Do you want to somehow unlock these currently disabled outer lanes and restrict usage to all but emergency vehicles? I am thinking about 3 lane highway and its white continuous lines on both sides. Could you add some images with roads with that 'hard shoulder'?

First, to clarify what I mean by emergency hard shoulder: https://en.wikipedia.org/wiki/Shoulder_(road)
Example from England:

Note: 'Central reservation' lane is often omitted (it's usually found near junctions); but the outer 'hard shoulder' is almost always present on UK highways.
In England they are sometimes given a red texture (especially on older highways):

In Ireland they usually have a solid yellow line:

The wikipedia article above shows other examples.
Next comment will explain how option would work...

The outer lane is converted to emergency lane by adding virtual vehicle restrictions
It's same approach as the Ban private cars and trucks on bus lanes option; only in this case it's detecting outermost vehicle lane and restricting so that only emergency vehicles can use it.

Asset authors, who want to produce assets which take advantage of the new option, but also want to make it 'kind of work' for players without TM:PE, could optionally designate their outer lane as a bus lane (like normal roads can have bus lanes, only this would be a bus lane on a highway).
While not perfect, players without TMPE would see far less traffic on the outer lane due to it being flagged as a bus lane.
However, players with TMPE will get the full 'emergency lane experience' via the new option (the new option doesn't care whether the outer lane is a bus lane or normal road lane).

Note: Without the option, players could just manually vehicle restrict their existing outside lane to emergency vehicles, but it would be tedious. Also it would add lots of data to save games which can already be problematic due to https://github.com/VictorPhilipp/Cities-Skylines-Traffic-Manager-President-Edition/issues/160
With the dedicated option, players get a nice way to quickly give their highways a hard shoulder. There are some other benefits, for example if there is single lane ramp joining a highway, it would need lane connecting to both outside lanes (the hard shoulder and the 'normal traffic' outside lane).
Hope that clarifies everything.

Yup, now I understand everything, thank you for your explanation :wink:

Closing this as it's now tracked by #515


VINVL code on the VizWiz datasets

Have you ever run the VINVL code on the VizWiz datasets? I extracted the img feats and generate the object tags with the same model weights. But the evaluation results remain same with the training process continues. Is is due to the object tags?
Have you ever met this issue on other datasets?

Hi Runze,
Could I ask how did you generate the trainval_ans2label.pkl file for the VizWiz dataset?
Thank you!

Hi Runze,
Could I ask if your object tags are generated from VinVL or Mask-RCNN?
I run the VinVL image features extraction and the output is predictions.tsv. But according to @xjli in https://github.com/microsoft/Oscar/issues/13#issuecomment-645809973_ , they are using Mask R-CNN for fine-tuning VQA. See:
There are two kinds of corpus in the work, pre-training corpus and downstream task finetuning corpus, both are using tag sets. For finetuning VQA, we observed that the best tag set is from COCO, not VG tag set (80 categories), because Faster R-CNN (pretrained on VG corpus), its tag prediction precision is not good enough, though VG tag set has more categories (1600). Here we use a high-precision Mask R-CNN (trained on COCO tag set) to generate the tags. For pre-training corpus, we use VG tag set from Faster R-CNN. For downstream task finetuning corpus, you can use any off-the-shelf object detector to generate the tags, it is a trade-off on high precision or more categories. And one more example is NoCaps finetuning corpus, we use OpenImage tag set, which is better on NoCaps task.
Also, I still have no idea how to generate the img_frcnn_feats.pt and trainval_label2ans.pkl.
Any help is appreciated. Thanks!


Support for fine grained permissions via UMA

Any plans to support central management of fine grained permissions via User-Managed Access (UMA) Profile of OAuth 2.0?
https://docs.kantarainitiative.org/uma/draft-uma-core.html
https://kantarainitiative.org/confluence/display/uma/Home

@thomasdarimont We don't have any plans yet. We are exploring different ways to achieve fine grained authorization. It may most likely be a separate project than UAA.


ENH: Update Montage remote module

Update Montage remote module to its latest commit.
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)

The Circle CI failure has more to do with sending the report to CDash I think, since its build and tests are passing.
The rest of builds are passing. Merging.


JS client may not give an error if sending an HTTP request fails

See: https://github.com/aspnet/SignalR/issues/957#issuecomment-333652247
In ReactNative the client was not able to connect to the server at all and the Error message was empty because both status and statusCode were undefined. It appears however that XmlHttpRequest contains a useful error message which we should include in the Error.

We'll reopen if someone still observes.

It was just observed: https://github.com/aspnet/SignalR/issues/957#issuecomment-371947683

I'm a little confused. In the linked example, the error message is in the response property. I don't think we should be expecting to read an error out of that field, particularly if statusCode isn't defined. This seems like a bug in the implementation of XHR.


fix(IAM Authenticator): canonicalize iam url & improve iam error reporting

This commit includes two changes:

The IAM Authenticator is now more flexible with respect to the
configured IAM url specified by the user.  The path part ("/identity/token")
is now optional.
For errors returned by the IAM token server, if the response received
from the token server does not contain an error message, then a default
error message containing the unexpected status code will be returned rather
than an empty string.

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


Remove docker images on make clean_all

#21 nominally added this feature but when I ran it I found the following error message:
rm -rf /home/ellen/dev/headway/data/*.mbtiles
rm -rf /home/ellen/dev/headway/data/*.nominatim.sql
rm -rf /home/ellen/dev/headway/data/*.osm.pbf
rm -rf /home/ellen/dev/headway/data/*.photon
rm -rf /home/ellen/dev/headway/data/bbox.txt
rm -rf /home/ellen/dev/headway/data/sources
rm -rf /home/ellen/dev/headway/data/nominatim_flatnode
rm -rf /home/ellen/dev/headway/data/nominatim_pg
docker images | grep ^headway_build_ | tr -s ' ' | cut -d' ' -f3 | xargs docker rmi
Error response from daemon: conflict: unable to delete f7354fa160e3 (must be forced) - image is referenced in multiple repositories
Error response from daemon: conflict: unable to delete 32a0db2e6aa4 (must be forced) - image is referenced in multiple repositories
Error response from daemon: conflict: unable to delete 7687b7da0d06 (must be forced) - image is referenced in multiple repositories
Error response from daemon: conflict: unable to delete 7f5243e4dfa0 (must be forced) - image is referenced in multiple repositories
make: *** [Makefile:150: clean_all] Error 123

This is likely because of the tagging renaming - that is, when rebuilding with different versions you ended up with the same image with different names. You could verify with e.g.  docker images | grep f7354fa160e3.
So I guess two options: Force-delete or just untag. I'm going with the latter.


fix: changed the download paths

Fixed the s3 image download location path.

Please delete options that are not relevant.

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

How Has This Been Tested?
Locally
Checklist:

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

/ok-to-test sha=a2130c73acdf266f79f2c70750ffd37bf0b7e779


Re-brand JobsReborn to be an alternaitve to AutoRank

Please can you adapt JobsReborn to be an alternative to AutoRank, the structure is there to have a fully automated ranking system based on Stats, Add more stats like blocks broken/placed and distance travelled and instead of money have we run commands to upgrade ranks, its so close to being an alternative to autorank, but with a better UI in-game.

AutoRank is compatible with JobsReborn, and these are what you want to do not in this plugin, but in the AutoRank plugin.


[REQUEST] detect fonts automatically

在提出功能请求之前,考虑在https://github.com/textualize/rich/discussions中发布反馈。
I have browsed all the relevant issues about "font". so many developers got an unexpected output,  caused by it.
你如何提高富贵?
I think rich can add a feature that automatically detects terminal fonts. using ansi char when proportional font, otherwise not.
它能为你解决什么问题?
in that case, rich can work better in Asia environment

I'm afraid there isn't a way to detect the font or any possible font fallbacks in the terminal.

https://learn.microsoft.com/en-us/windows/console/getcurrentconsolefontex

Interesting, but this page even warns that "This API is not recommended and does not have a virtual terminal equivalent". Presumably this would only work in the classic conhost and not the new Windows Terminal?

We do not recommend using this content in new products, but we will continue to support existing usages for the indefinite future.

Let me clarify: even if your feature request could be implemented, it would only work on Windows and specifically in the old conhost, not in the new default Windows Terminal?

yes, it would only work on Windows. but it also be used in the new terminal.

using ansi char when proportional font, otherwise not.

I'm not clear on what output you would expect with a proportional font. Very little of Rich would work without a monospace font.
Does this related to using Chinese characters in the terminal? Is it common for Chinese users to use proportional fonts?


construct complex return values in epilogue

This is needed to support more complex output types (eg for HF transformers).

still some corner cases to fix.


configure Travis to configure Git with name/email of user

Travis auto-deployment requires Git to be configured with the name and email address of the user who will commit the deployed files to the gh-pages branch, and while the name can be arbitrary, the email address has to be the one associated with the user whose token is being used to authorize the deployment.
But the configure task doesn't configure Travis to configure Git with this info. It should do so.
For example, in my .travis.yml file, I manually configure Travis to configure Git via a section like the following:
before_script:
  - 'git config --global user.name "Travis-CI"'
  - 'git config --global user.email<EMAIL_ADDRESS>
The configure task should determine the user's email address and use it to configure .travis.yml similarly.

Fixed by https://github.com/mozilla/oghliner/commit/2f03490cf348b5722eb98fe0a717650d22da0aa3.


about sources

I cannot help but think that source data installation is needlessly complex.
Ideally there should be a default path list (/usr/share/wikiman, ~/.local/share/wikiman) that could be overriden using a config and the download procedure should be a subcommand of wikiman that would try to write in the aforementioned path list sequentially. (So a sudo would allow me to write globally).

Hi,
The source data installation was constructed like this because the only original data source was Arch Wiki which was installable by the system's package manager as a shared database.
The other ones were just appended afterwards, and having the download code separate from the utility means it always knows which snapshots are the latest. For instance, due to having to make a very hacky snapshotting tool for the Genoo Wiki, it is updated less often than the rest. Updating data sources would require either the change of hardcoded links in the utility, or a way to look up links to the latest snapshots.
The idea you are proposing is sound and I like it; it only needs to fix the aforementioned problem. Would you like to write it yourself as a Pull request? I am currently busy with other things, so implementing it might take a while.

Sorry I don't have context into the code and the benefit would not be worth the extra boot-up time :|
Thanks for the great software though!


TTF_GlyphMetrics - returned "advance" value appears to be wrong

The "advance" value returned by the function TTF_GlyphMetrics appears to be wrong.
The font I am loading is: /usr/share/fonts/truetype/liberation2/LiberationMono-Regular.ttf
The OS is Debian 11.
The widest character in this font set is the "_" character, which supposedly has a width of "9" pixels. It has an "advance" value of "7" pixels. (Which is universal to all characters in the font, I believe.)
Weirdly, if I take a screenshot of a printed "_" character, and open this with GIMP and zoom in to count the pixels, the width is either 8 or 7 depending if you count the "hinted" pixel. (The rendering type is "blended" so the rightmost pixel of the underscore character is only partially colored, it is partially transparent.)
Thinking about it, the "advance" value might be correct, but the "width" calculated from the "minx" and "maxx" values might be incorrect? I'm not sure about this, it is a bit confusing.
The below screenshot isn't great but should illustrate the point. The solid part of the line is 7 pixels long. There is one "hinted" pixel at the right end.

Actually, if you zoom in further, you can see there is also a hinted font pixel on the left. The color is very close to that of the background color.

So the query is, why is the maxx - minx value producing "9" when the advance value is only "7"? Does this have something to do with using the "blended" rendering mode?

Can you try with latest SDL_ttf head version ?
You can have  maxx - minx value producing "9" and advance value  "7". This isn't incoherent.
(Usually you always have blank around the glyph.)
see there for the conventions:
https://www.freetype.org/freetype2/docs/glyphs/glyphs-3.html
I don't think one should use this API, but just the text render functions.

From https://freetype.org/freetype2/docs/glyphs/glyphs-3.html
For example, the advance width of ‘VA’ isn't the sum of the advances of ‘V’ and ‘A’ taken separately.


Edit the BillList to StatisticsList and clean up the storage component for statistics

There was a confusion and mix up between billList and statisticsList. This is also related to the statistics storage component

Now under the model Component for Statistics, there are these following classes:
****For future command arguments
Day.java
Month.java
Year.java
****For Bill Command
Bill.java (For specific bills of each orders. These Bills are not saved in the storage)
****For Statistics Storage
DailyRevenue,java (This will be stored in the storage. Each of these are specific to its date)
DailyRevenueList.java (A list of DailyRevenues)
Statistics.java (Wraps all statistics related data)
ReadOnlyStatistics (Unmodified view of statistics)
***Exceptions
BillNotFoundException.java
DailyRevenueNotFoundException.java


Upgrade to Scintilla 4.3.3

@blairmcg Is there an option to autoclose a predefined set of characters? It is something that should be done client side? I got used to mainstream editors and when I press " ' [ I'm always waiting for the autoclose char. Also got used to CTRL+D to duplicate lines which is correctly defined in Scintilla keybindings but in workspaces it's used for "Display it". Maybe a better default would be ctrl+alt+D

As you suspect that would be a something done at the level of the Scintilla host application, not by Scintilla itself. Scintilla provides the toolkit, but generally not high-level editing behaviours.  As an example see ScintillaView>>braceHighlight. The Scintilla control provides both a way to search for matching brackets, and a way to highlight them, but it doesn't tie it altogether into a brace matching feature, even as a default. There is a deliberate architectural layering that the Scite example editor helps to delineate.
It would be entirely possible to do that in the Smalltalk integration layer - probably harder to decide exactly how you want it to work, than to actually do it.
As for the Ctrl+D clash, yes that is unfortunate. The Dolphin workspace shortcuts were decided long before Scintilla was integrated. I believe Ctrl+D to duplicate lines was probably following the Visual Studio convention, which didn't exist when we started on Dolphin either. I don't remember whether the early VC++ editions we used had that key binding, but if so we either ignored it or didn't notice. It does seem rather late to change such longstanding choice now. What is really needed is better customisation capability so that you could both change it to suit your preference, and also have that persisted in a convenient way that doesn't get lost and which gets picked up in any new image. At present most of the workspace key bindings are not even easily configurable because they are stored in the menu definitions in the view resources.

Yes it's a bit tricky to autoclose but we can adhere to Visual Studio Code behavior. I didn't notice it strange.
For the Ctrl + D my suggestion was changing the duplicate command to Ctrl + Alt +D instead of changing #displayIt


Issue-1294: Implementing ArtifactGenerator in NpmPackageGenerator

Fixes #1294
Implements the ArtifactGenerator interface in the class NpmPackageGenerator.
Re-uses the class logic from MavenArtifactGenerator.

@ogryniuk ,
Thanks for having a look at this, but I think you're very confused.
NPM is a build/packaging/dependency management tool like Maven. Strongbox supports the NPM layout format. NPM does not use jar files and I have no idea how you arrived at this conclusion.
The goal of the task, is to do the necessary refactoring, so that the NpmPackageGenerator implements the ArtifactGenerator interface.


@conda-forge-admin, please rerender

[ ] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/goosefem-feedstock#44.


Some VBlank timing roms are failing

Only remaining issue here is ppu_vbl_timing / 10-even_odd_timing.nes where

Not sure why yet

Ok, so that test is specifically testing what happens when BG is enabled exactly as the dot that would be skipped is happening.
On cpu cycle 1548589 STY PPUMASK occurs with Y = 8 (enabling BG, previously rendering entirely disabled). Mesen claims that's dot 329 of pre-render. Because STY PPUMASK writes on the 4th cycle this implies that the write is on dot 339.
We're then skipping dot 340 because BG is now on, but Mesen isn't so we're off by a dot going forwards. Not clear to me why what I'm doing is wrong though

https://github.com/SourMesen/Mesen/blob/dbe7e0dec8801a9fb8da8b710bc8c10c1e69c381/Core/PPU.cpp#L1349 explains the issue but it utterly defeats me where this is documented! I can't see anything on nesdev wiki or forum so who knows how Sour found out!


Capture user name with Azure SSO

What does this PR do?
When using Azure Tomcat service, the user name is not properly captured by Java agent.
This PR adds a fallback heuristic to capture the provided user name based on official MS code samples and documentation.
Checklist

[ ] validate implementation manually with Azure/Tomcat deployment
[x] This is an enhancement of existing features, or a new feature in existing plugins

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that prove my fix is effective or that my feature works n/a, will require manual testing.
[ ] Added an API method or config option? Document in which version this will be introduced n/a
[ ] I have made corresponding changes to the documentation n/a no doc change required.

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-09-05T10:31:58.952+0000

Duration: 37 min 8 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
2505

Skipped
31

Total
2536

:green_heart: Flaky test report
Tests succeeded.
:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark tests.

run jdk compatibility tests : Run the JDK Compatibility tests.

run integration tests : Run the Agent Integration tests.

run end-to-end tests : Run the APM-ITs.

run windows tests : Build & tests on windows.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Can it be made into windows application so that don't have to open web pages every time.

Hope:
Can it be made into windows softWare so that don't have to open web pages every time.
Supplement：
I don't know if the code for this repository is the source code? If so, because it is the Dart language, can it be directly made into a Windows application?

More convenient and powerful dart entity and model generator, I build. Please visit (I will increase more features, upon community request)-
https://json-to-dart.siamahnaf.com/


Add a test for DeprecationReportBody

Does this make sense? Stil learning how to write tests for the collector.
Inspired from the CSPViolationReportBody test.

Closing this. Hard to write a cross-engine test for this right now.


Update swagger docs

As there already is a swagger generator integrated into the project it could be helpful to deploy a static version of the swagger docs next to the new documentation of #282.
Plan

[ ] update swagger definitions
[ ] add swagger to docs (https://github.com/rohit-gohri/redocusaurus)
[ ] create TS api client (https://www.npmjs.com/package/swagger-typescript)
[ ] create woodpecker-go api client

@6543 @jdoubleu I updated the description & title of this issue and added the idea to use swagger to generate client libs.


[website] update Table group documentation and links to table guides

No dev review needed.
I'm updating documentation for DataTable component so it can meet the requirements for:

correct english
correct structure
simplicity
etc.

Added all these changes in PR https://github.com/semrush/intergalactic/pull/877


SCRIPT: Ability to rearrange order of stories #490

I've added logic in item store file for reaaranging the order of stories as per mentioned in the ticket.

Hello @mrjamieeast


Change recommended on-premises version number

Category

[x] Content fix
[ ] New article
[ ] Example checked item (delete this line)

What's in this Pull Request?
Changed the recommended version number for on-premises projects from 1.10.1 that does not exist to 1.10.0 which is also referenced elsewhere in the article.

Docs Build status updates of commit 1c3670e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/spfx/sharepoint-2019-support.md
:warning:Warning
View
Details

docs/spfx/sharepoint-2019-support.md

Line 2, Column 1: [Warning: ms-prod-missing - See documentation] Missing attribute: 'ms.prod'. If you specify 'ms.technology', you must also specify 'ms.prod'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[Update Request]: KDiff3 still at version 1.9.6

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
KDE.KDiff3
Package Version
1.10.4
Please describe the changes you would like to see
STEPS TO REPRODUCE

Run "winget install -e --id KDE.KDiff3" in PowerShell

OBSERVED RESULT
Version 1.9.6 is installed
EXPECTED RESULT
Version 1.10.4 is installed
ADDITIONAL INFORMATION
Question:
Is KDiff3 inside winget store still maintained?
Do we (layman users) need to file such request for every new version of this application?
If it's not maintained, may I suggest you to delete everything about KDiff3 in winget store.  In this way, people would be forced to download the latest version from your web site.  This could eliminate a lot of potential problems.
However, if it's still maintained, please fix the issue.  Thanks in advance.

This package might not be able to be updated because the installer seems to always throw this prompt -

I'm a simple end-user.  I know nothing about the making of package.
FYI, a few hours ago, I posted a bug report to KDE about this issue:
https://bugs.kde.org/show_bug.cgi?id=470221
That bug was immediately closed by them, saying that they "don't have any part in the maintenance or access to the store".
Back to your comment about the "throwing the prompt", in my perspective as an end-user, I would say:
Could the "mechanism" be programmed to detect pre-1.8 versions?  If pre-1.8 version is installed, is it possible to just display a message, through PowerShell of course, to tell the user that KDiff3 has to be uninstalled (manually) first?
If nobody is going to take care / maintain this package, please delete everything in the package.

That bug was immediately closed by them, saying that they "don't have any part in the maintenance or access to the store".

Correct. The winget source is maintained by the community. While publishers can submit their own updates, most packages are submitted by the community

Back to your comment about the "throwing the prompt", in my perspective as an end-user, I would say:
Could the "mechanism" be programmed to detect pre-1.8 versions? If pre-1.8 version is installed, is it possible to just display a message, through PowerShell of course, to tell the user that KDiff3 has to be uninstalled (manually) first?

Not exactly. The prompt is thrown even on a fresh install of windows. Since the prompt is part of the KDE installer, that is most likey either a bug or an intentional design choice by KDE. It is a requirement that all packages in the winget source must be able to be installed without user interaction, which makes this a blocking issue unless 1) there is a way to suppress the message box, 2) KDE removes or fixes the message box, or 3) winget implements filtering by installation mode.
As to your idea around winget detecting pre-1.8 versions, there is a feature request open for that type of behavior (see https://github.com/microsoft/winget-cli/issues/3007). However, since the prompt is shown even when KDiff3 has never been installed, there would be no difference even if we could detect it.

If nobody is going to take care / maintain this package, please delete everything in the package.

Some packages are quite popular and are maintained by users regularly. Others are not as commonly used and a request like this is needed to inform a contributor that a newer version is available. Thats one of the downsides of being a community-driven repository. However, thats not to say "nobody" is maintaining it; rather - we are trying to maintain it but the popup presents a blocking issue.

I'm a simple end-user. I know nothing about the making of package.

The best way I can explain it is that there are package publishers (Like KDE or Google or Zoom) and there are winget contributors, who are just end users like you or me.
A publisher actually creates the software. They create the installer, the uninstaller, the actual application that runs. Anything that has to do with the behavior of the application is the responsibility of the publisher.
A winget contributor submits a data file called a manifest to this repository, the winget source. The data file only tells winget how to download the installer the publisher created, if any special arguments are needed for an unattended install, and some information needed to detect the software is installed.
All winget does is download the installer from the publishers website and run it. For an uninstall, it just calls the program's default uninstaller.

@Trenly
Thanks for your explanations on making the package.  Really very very appreciate it.  But I'm afraid you have wasted your time on me.  You see, I have my job and family.  I don't quite have spare time.  Writing bug reports like this is already a large effort from me.  The information you gave me, I'm not going to use it anywhere, unfortunately.  I'm not going to make manifest file, sure of that.
I see that the setup-making application for KDiff3 is NSIS.  I think maybe you could look into the NSIS files to understand how to get around it.
For the time being, I'm going to forget about winget, download the setup and just install it in the old way.
Good luck to you, guys. 👍

I'm a simple end-user. I know nothing about the making of package.
FYI, a few hours ago, I posted a bug report to KDE about this issue: https://bugs.kde.org/show_bug.cgi?id=470221
That bug was immediately closed by them, saying that they "don't have any part in the maintenance or access to the store".

This is unfortunately a very common way for open-source projects to handle bug reports - closing with extreme prejudice. In my personal account and on my own time, this kind of passive aggression is a major reason why I haven't contributed to most open-source projects, despite even some YouTube videos begging viewers to do so. The requirement that a person spend months or years learning both the repo history and the behaviors of the major contributors is too high of a barrier to entry for the majority of users to surmount, just to make one suggestion to one codebase. One of my goals in this role is to at least provide closure and background info, especially to newer users, and help to offset the trend for excessively succinct closures.

The user interaction has been amend to honor the silent switch. That was an oversight on my part.  Unfortunately I rely craft for the windows build and thus part of the install script is beyond my control. This prevents me from detecting if the older version was present as craft generate script currently runs the uninstaller for that version before I have a chance to do anything.
The patched install should be available under  kdiff3-1.10.4-slientfixed-windows-x86_64.exe in the next 24 to 48 hrs. 1.10.5 will include the change.


AES containers sending DNS/port queries to bitcoin mining addresses

Describe the bug
We've noticed an extremely high volume of requests to Bitcoin-related endpoint addresses originating from our AES containers. Detection via both Falco and AWS GuardDuty confirms the same. Is this expected behavior? Our deployment is via the official Datawire Ambassador Helm Chart. If the pods are deleted no more requests. What's going on?
To Reproduce
Deploy AES into K8s using your preferred method and container origination source as the quay.io repo.
Expected behavior
No queries to Bitcoin addresses
Versions (please complete the following information):

Ambassador: AES 1.2.0 (and 1.1.1)
Kubernetes environment: EKS
Version [e.g. 1.14.9]

That's emphatically not expected behavior. Do you see the requests originating with your Ambassador pods, or are you seeing them transiting your Ambassador pods somehow?

(In either case, I would encourage you to join our Slack as well; that's likely a simpler way to communicate.)

They are DNS queries transmitting from the ambassador edge stack pods. The behavior is quite reproducible if we destroy the pods and recreate them, on/off for the detection. Could be a false/positive, but I wanted to raise the observation in case there’s a more insidious underlying root cause. The process originating the requests is:
diagd /usr/bin/diagd /ambassador/snapshots /ambassador/bootstrap-ads.json /ambassador/envoy/envoy.json --notices /ambassador/notices.json --kick kill -HUP 1
From: Flynn<EMAIL_ADDRESS>Date: Tuesday, March 3, 2020 at 12:52 PM
To: datawire/ambassador<EMAIL_ADDRESS>Cc: Jason Shrack<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [datawire/ambassador] AES containers sending DNS/port queries to bitcoin mining addresses (#2369)
That's emphatically not expected behavior. Do you see the requests originating with your Ambassador pods, or are you seeing them transiting your Ambassador pods somehow?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/datawire/ambassador/issues/2369?email_source=notifications&email_token=ABPV2TD3V27SS3I5ZOPBLV3RFVUWJA5CNFSM4LAQ2EZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENVDEOA#issuecomment-594162232, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABPV2TFHUL5QDS2T2VZPSWTRFVUWJANCNFSM4LAQ2EZA.


extract-zip relies on outdated debug package with security issue

It looks like extract-zip requires<EMAIL_ADDRESS>which is vulnerable to a regular expression denial of service exploit. This exploit is fixed in<EMAIL_ADDRESS>or<EMAIL_ADDRESS>

@malept @maxogden Can we have a release cut with this fix?

That's a major version bump. I'll need some time to look at the potentially breaking changes.

@malept It seems that no breaking changes impact extract-zip:
https://github.com/visionmedia/debug/blob/master/CHANGELOG.md#300--2017-08-08

Here is the advisory and recommendation from versioneye:
https://www.versioneye.com/nodejs/debug/2.2.0

@maxogden : can you please merge this and cut a release?
If the 3.0 is really a concern, you can just bump to 2.6.9 or higher to be on the safe side, while clearing out the security violation at the same time.

For the people waiting on this to be merged:
If you're using yarn, you can override the dependency chain with resolutions in package.json.
If the top level dependency causing problems is html-pdf, for example, you would use the following to force extract-zip to use a newer version of debug:
 "resolutions": {
   "html-pdf/phantomjs-prebuilt/extract-zip/debug": "2.6.9"
 }

So maybe this is a little stupid but another way to handle this is if you are using npm@5 with package-lock.json you can just change the dependency to<EMAIL_ADDRESS>in this file to get the updated version. So for me this is in dependencies.extract-zip.requires and dependencies.extract-zip.dependencies:
  "requires": {
    ...
    "debug": "https://registry.npmjs.org/debug/-/debug-2.6.9.tgz"
    ...
  }

  "dependencies": {
    "debug": {
      "version": "https://registry.npmjs.org/debug/-/debug-2.6.9.tgz",
      "integrity": "sha1-XRKFFd8TT/Mn6QpMk/Tgd6U2NB8="
    }
    ...
  }

When you install again it will complain because of mismatching integrity but when you install yet again it works fine and the dependency is updated in the node_modules folder.
What this does not change though is nsp still reporting the vulnerability because this is based on the version numbers in package.json, but as effectively the vulnerability is now patched I can safely add an exception to .nsprc for https://nodesecurity.io/advisories/534.
This is kind of a hack but fixes the vulnerability, so am I missing something important?

@malept Getting messages about this thread but don't wanna break anything like the last time I tried to merge a security fix in this repo :D Any chance you can take another look?

Closed by https://github.com/maxogden/extract-zip/commit/94b877a54dfaac8ada949cf385afb41fa0102099


UniFi - Temporary workaround to get device tracker to mark clients as away

Breaking change

Proposed change

Type of change

[ ] Dependency upgrade
[x] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

# Example configuration.yaml

Additional information

This PR fixes or closes issue: fixes #32170
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist
[x] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

[ ] No score or internal
[ ] 🥈 Silver
[ ] 🥇 Gold
[x] 🏆 Platinum

106.2 not getting updates. All my devices are not home.

106.2 not getting updates. All my devices are not home.

Same here with 0.106.2, devices still not be updated

Yeah same here

Same here as well

Please open an issue or comment in an existing open issue if there's a problem.
Don't comment with plus 1 or same issue etc. That's just spam. Please only comment if you have additional info that can be of help to resolve the issue.


[SUPPORT] how to use flink offline with occ

Tips before filing an issue

Have you gone through our FAQs? y

Join the mailing list to engage in conversations and get faster support at<EMAIL_ADDRESS>

If you have triaged this as a bug, then file an issue directly.

Describe the problem you faced
A clear and concise description of the problem.
To Reproduce
Steps to reproduce the behavior:

Expected behavior
A clear and concise description of what you expected to happen.
Environment Description

Hudi version : 0.12.1

Spark version : 2.4.5

Hive version : 2.1.1

Hadoop version : 3.0.0

Storage (HDFS/S3/GCS..) : HDFS

Running on Docker? (yes/no) : no

Additional context
Add any other context about the problem here.
i use flink sql bulid hudi sink and use code has been corrected (https://github.com/apache/hudi/pull/7320)
It seems normal
CREATE TABLE IF NOT EXISTS customer_hudi(
`C_CUSTKEY` int,
`C_NAME` varchar(25),
`C_ADDRESS` varchar(40),
`C_NATIONKEY` varchar(10),
`C_PHONE` char(15),
`C_ACCTBAL` decimal(15, 2),
`C_MKTSEGMENT` char(10),
`C_COMMENT` varchar(117),
`ts` bigint,
PRIMARY KEY (C_CUSTKEY) NOT ENFORCED
) WITH (
 'connector' = 'hudi',
 'path' = 'hdfs://ip:8020/hudi/customer',
 'table.type' = 'MERGE_ON_READ',
 'hive_sync.metastore.uris' = 'thrift://ip:9083',
 'index.bootstrap.enabled' = 'true',
 'hoodie.write.lock.provider' = 'org.apache.hudi.client.transaction.lock.ZookeeperBasedLockProvider',
 'compaction.delta_seconds' = '120',
 'compaction.async.enabled' = 'true',
 'hive_sync.table' = 'customer',
 'hoodie.datasource.write.recordkey.field' = 'C_CUSTKEY',
 'hoodie.write.lock.zookeeper.port' = '2181',
 'hoodie.write.lock.zookeeper.base_path' = '/hudi',
 'archive.max_commits' = '6',
 'metadata.enabled' = 'true',
 'hoodie.write.lock.zookeeper.lock_key' = 'customer',
 'hive_sync.enable' = 'true',
 'clean.async.enabled' = 'true',
 'hive_sync.db' = 'hudi',
 'hive_sync.skip_ro_suffix' = 'true',
 'hoodie.write.concurrency.mode' = 'optimistic_concurrency_control',
 'compaction.delta_commits' = '1',
 'compaction.trigger.strategy' = 'num_or_time',
 'archive.min_commits' = '4',
 'clean.retain_commits' = '2',
 'hoodie.write.lock.zookeeper.url' = 'ip',
 'hive_sync.support_timestamp' = 'true',
 'read.streaming.check-interval' = '20',
 'hoodie.cleaner.policy.failed.writes' = 'LAZY',
 'metadata.compaction.delta_commits' = '1',
 'hive_sync.mode' = 'hms',
 'hoodie.datasource.write.operation' = 'bulk_insert'
)

if i use "compaction.async.enabled=false" and use flink offline to compact
the hudi flink offline support it now?
how to build flink run?
./bin/flink run-application -t yarn-application -p 2 -Dyarn.application.name="customer compaction" -Dyarn.provided.lib.dirs="hdfs://ip:8020/flink/app_new/" -c org.apache.hudi.sink.compact.HoodieFlinkCompactor lib/hudi-flink1.14-bundle-0.12.1.jar --path hdfs://ip:8020/hudi/customer --min-compaction-interval-seconds 60 --service --compaction-tasks 2

thanks
Stacktrace
Add the stacktrace of the error.

i want to use flink offline and add occ config
hoodie.write.concurrency.mode=optimistic_concurrency_control
hoodie.cleaner.policy.failed.writes=LAZY
hoodie.write.lock.provider=org.apache.hudi.client.transaction.lock.ZookeeperBasedLockProvider
hoodie.write.lock.zookeeper.url
hoodie.write.lock.zookeeper.port
hoodie.write.lock.zookeeper.lock_key
hoodie.write.lock.zookeeper.base_path

@danny0405 @yuzhaojing could any of you help the user on the Flink related issue?

There is no need OCC here for offline compaction, all you need to do is the start the HoodieFlinkCompactor app.

There is no need OCC here for offline compaction, all you need to do is the start the HoodieFlinkCompactor app.

@danny0405 thanks, the flink sql set 'metadata.enabled' = 'true', but in flink offline compaction i can't find any metadata properties in org.apache.hudi.sink.compact.FlinkCompactionConfig
if offline compaction not update metadata table , is the Index unavailable ？
i try the spark offline compaction, it seems to be working fine, and find the zk lock node is created, and hdfs hudi table path .hoodie/metadata/files/ is update
/opt/spark-2.4.5-bin-without-hadoop/bin/spark-submit \
--master yarn \
--deploy-mode client \
--class org.apache.hudi.utilities.HoodieCompactor hudi-utilities-bundle_2.11-0.12.1.jar \
-sm 2g \
--mode execute \
--base-path hdfs://ip:8020/hudi/customer \
--table-name customer \
--hoodie-conf hoodie.embed.timeline.server=false \
--hoodie-conf hoodie.write.concurrency.mode=optimistic_concurrency_control \
--hoodie-conf hoodie.cleaner.policy.failed.writes=LAZY \
--hoodie-conf hoodie.write.lock.provider=org.apache.hudi.client.transaction.lock.ZookeeperBasedLockProvider \
--hoodie-conf hoodie.write.lock.zookeeper.url=ip \
--hoodie-conf hoodie.write.lock.zookeeper.port=port \
--hoodie-conf hoodie.write.lock.zookeeper.lock_key=customer \
--hoodie-conf hoodie.write.lock.zookeeper.base_path=/hudi \
--hoodie-conf hoodie.metadata.index.bloom.filter.enable=true \
--hoodie-conf hoodie.metadata.index.column.stats.enable=false \
--hoodie-conf hoodie.enable.data.skipping=false

Yes, metadata table is not adapted for offline compaction yet, I would suggest disabling the metadata table now in production.

closing this as suggestion was provided.


adding some content design guidelines to style guide

these are taken from @SaraDPH's work to define content design guidelines
super open to any and all feedback

Whoa, @cnunciato did you merge this on purpose? There was suggested changes on it.

Oh yes, I did, sorry -- I misread and thought you'd applied them.


community: Add Baichuan Text Embedding Model and Baichuan Inc introduction

Description: Adding Baichuan Text Embedding Model and Baichuan Inc introduction.

Baichuan Text Embedding ranks #1 in C-MTEB leaderboard: https://huggingface.co/spaces/mteb/leaderboard

can we remove the changes to libs/langchain? only libs/community needs to be updated for new code, the imports in libs/langchain are only kept around for backwards compatibility and will be removed in the next minor version release

Sure thing. Done.

Why I can't find BaichuanTextEmbeddings in langchain v0.1.4

Why I can't find BaichuanTextEmbeddings in langchain v0.1.4

This PR was merged 2 days ago. You might need to wait for the next langchain release :)


Replace all CIs for GitHub Actions

Description of the desired feature
After last community call, we decided to move all our automated CIs to GitHub Actions.
Actions work faster than Travis and Azure with much easier configuration, specially for deployment and key managements.
The visualization of the processes is also better because it's integrated into GitHub, including also errors raising on the workflow configuration file, which makes much easier to debug the workflows.
Currently Pooch has already made the change, so we need to do the same on:

[x] harmonica
[x] verde
[ ] rockhound
[x] boule
[ ] contributing

Feel free to open PRs on each repository and ping this issue on it.

Remember to stop saving the docs HTML as build artifacts. See https://github.com/fatiando/boule/pull/58

We opted not to port the contributing CI and rockhound will need a major refactoring anyway so closing this as it's basically done.


characters do not appear... but are in the document

I tried to write a couple Chinese characters into a test document, between Hello and World.
Space is left for them, but they do not appear. Selecting and copying the whole line of text from the PDF document using SumatraPDF, and pasting into a text editor shows the characters are there... but invisible.
If I can figure out how to attach files, I'll attach  the sample program and PDF output...

I also have this problem.

Hi v-python,
I also have this problem. Did you work out how to display the Chinese characters with SimSun font?
Thanks.

No, I haven't.
From following along the project activity for a while, it appears likely that the related "fontkit" project might help with this, but...
There is no browser-loadable package provided for "pdfkit with fontkit"; I haven't yet invested the time to figure out either coffeescript or nodejs deeply enough to build one; Issue #473 tells of someone that figured out how, and built one, it may be useful to you, I have it saved in my email for when/if I get a chance to look at pdfkit with fontkit again.  Fontkit still seems to be beta (last I looked), which probably explains the lack of a pre-built package.
node-webkit (with Chromium) didn't work well with the debugger last I tried (see my December 27, 2014 comments). I have no idea if this has been improved since.
I had another project take precedence over this one in which I was trying to use pdfkit + Chinese. The other project took my whole life for a year, although it is now complete. I have started the process of trying to catch up with the rest of my life; I probably won't get back to this until about September this year, or later.
I hope these comments are helpful, and that you can make some progress, and that the projects have made and continue to make some progress, in the meantime. pdfkit with fontkit sounds like it can become a very powerful PDF creation solution, but at the end of 2014, was not ready for production.

v-python,
Thank you so much!
Follow your instructions I got https://www.npmjs.com/package/fzcs-pdfkit-fontkit, which works well for Chinese fonts.
Thanks.

fontkit was released in v0.8.0.


Line breaks within quotes should not start new record

CSV Lint is wrongly interpreting line breaks within quotes as starting a new record. So

A,"B
C",D

is two records of two fields, when it should be one of four.
This is as per RFC 4180 section 2.6

I sense that this probably is better applied to https://github.com/theodi/csvlint.rb

This issue was moved to theodi/csvlint.rb#204


Container analysis update

Updated container analysis samples:

each sample is in its own file, and includes imports
added new samples:

pollDiscoveryOccurrenceFinished
findVulnerabilityOccurrencesForImage
findHighSeverityVulnerabilitiesForImage

updated comment style to be in line with new canonical format
samples now client creation as part of sample, rather than accepting it as a parameter
updated region tags to be compliant with sample tracker

@kurtisvg Since this is in the new format, I'd like you to weigh in. I'm not happy seeing lines like \\ String projectId = "my-project-id"; I'd prefer to see // projectId = "my-project-id" (No String or ;) as it's a comment, not a line that should be commented out. Ideally we'll have standard language we use to explain we are showing "Example values:" (which I consider a huge win)

The idea is to make it as easy as possible to get users up and running. If they copy/paste the snippet, they can either remove the args and uncomment the snippet, or if they are calling it from another file they can copy the comments and use them before calling the method.
It's not a perfect solution for a variety of reasons, but since Java doesn't do default values it was the best compromise we could find. The only information we pass in as an argument should be whats absolutely required for testing (so probably values related to project info) to minimize this as much as possible.


Question or feature: adding arguments to executable

Hi,
any possibility to add arguments to the executable, e.g. appending them to entrypoint?
Is that breaking some principle, in case it is implemented?

The triggering need is to be able to run <my_go_exec> <args...> at runtime, with custom <args...>. In detail, <my_go_exec> needs a config file: it runs with <my_go_exec> -c <config_file_with_path> , and <config_file_with_path> can change. I currently edit the docker image entrypoint and add that there. Not best likely. Might be using env variables is a better option.
And, no, this is not about setting args to go build.

the case is a bit different: the image yes tested on (docker), but then loaded to kubernetes clusters, requiring different <config>. My choice is either handle it with differnet executables, which I do not like because testing should be done on exactly same exec, or change entrypoint in the docker image (which is the loaded into kubenetes). Note: the same image is pushed to the docker registry for prod deployment.
Again I'm not a pro here, can I add -c <config> to the kubernetes yaml deploy. Or?

Something like this fits my purpose, but not sure it is a good merge || there are better solutions: https://github.com/ko-build/ko/pull/1023

Again I'm not a pro here, can I add -c <config> to the kubernetes yaml deploy. Or?

You can!
containers:
- name: app
  image: ko://github.com/my/app/cmd/server
  args: ['extra', 'args', '--and', '--flags']

This will start the container and run the entrypoint, with the given args, e.g., /ko-app/server extra args --and --flags, just as if you'd done docker run <image> extra args --and --flags.
Give it a shot and let me know if it works. I'm not hugely opposed to adding entrypoint args, but I'd rather not if it turns out you don't need to yet.

args: ['extra', 'args', '--and', '--flags']
...oh. You made my day. I can happily now use mainstream ko with your approach. Thanks!


Combining with elements in other namespaces

How does this syntax work with elements in other namespaces? For example, I have a Behavior that hooks an event that is not defined in API version < 5. The behavior object itself is defined in a namespace. e.g. icore:EventTriggerBehavior. Is it possible to combine these?

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

ID: e4adf9e3-46b9-732a-7659-07d34a9ff469
Version Independent ID: 0dd55994-fed9-6d25-38c8-85e1ee2322ff
Content: Conditional XAML - UWP app developer
Content Source: windows-apps-src/debug-test-perf/conditional-xaml.md
Service: unspecified
GitHub Login: @jwmsft
Microsoft Alias: jimwalk

Here is how I managed to do it with my UserControl:
The control was in namespace: Holla.Controls
the conditional XAML namespace is: xmlns:contractLevel5NotPresent="using:Holla.Controls?IsApiContractNotPresent(Windows.Foundation.UniversalApiContract, 5)"
This will make Visual Studio complain about the format but it works as expected.
Hope this helps


CellEditingStopped in fullRow Edit Mode

I am editing a row in ag-grid with editType=fullrow. This works fine for the cell editors that are packaged with ag-grid but I am having a problem with the AutoComplete editor.
When I click on a selection then the cellEditingStopped and rowEditingStopped functions are called. The result is that my row is submitted before I am ready and the other rows have been completed,
I'm quiet new to Angular and Ag-Grid so I am not sure if this is a bug or my misunderstanding of how it should work.
Thanks

Hi there !
I'm afraid that 'fullrow' editing, is not, and will probably not be available for ag-grid-autocomplete-editor.
Because, this cellEditor is an 'PopupEditor' (like agPopupSelectCellEditor is) and that, it is not recommended by ag-Grid itself to make this kind of editor work with 'fullrow' editing (see ag-grid-docuumentation about that here).
TLDR:
Full row editing is not compatible with popup editors.

However, I know that SelectCellEditor work with fullRowEdit probably due to the fact that it use the native browser select element which handle a lot by itself. So we may probably be able to make autocompleteEditor work as well with fullRowEdit.
But it will probably be very tricky and require some change in both autocompleter subpackage (and the way it handle lot of things with global DOM attributes) and ag-grid-autocomplete-editor.ts class itself.
I will probably don't work on it anytime soon, but if someone make it work with this edit mode without too much dirty hacks, I will consider merging it.

Thanks for the fast response. Its a great component so thanks for all your work on this.
In my case all of my grids are in fullRowEdit. Is there a way (even if its a bit hacky) that I can prevent it calling cellEditingStopped? I tried to intercept the onSelect but to no avail.

You will probably face some other side effects even if you are able to bypass this one (the first I can think of is if you have multiples AutocompleteCellEditor on a same row)
But on your question precisely, I would dig onto the way destroy function work:
...
    destroy(): void {
        this.focusOut();
        if (this.stopEditing) {
            this.stopEditing();
        }
    }

Maybe removing the call to stopEditing will did the trick for you.


EditorAttributes not working with class inherited from NetworkBehaviour

I'm using Netcode for Gameobjects and I think NetworkBehaviour class uses other PropertyDrawer than the default MonoBehaviour.
Not sure if this can be considered as a potentional fix in the future because Netcode is changing constantly but it would be great to hear any suggestion to solve this issue.
This image shows one example but basically EditorAttributes are unusable in any networked class.

Yeah I know about this. I'm guessing netcode extends the editor class and does some ImGUI stuff there or it has a custom property drawer thats using ImGUI. I will have to research to see if its possible to detect and draw ImGUI properties with UI toolkit but there is no guarantee. The only other solution is to manually find and modify that drawer to use UI toolkit but for that you will have to clone the package in your project and handle any merges with new updates which can be a pain. Or just wait until unity decides to use UI toolkit consistently for their editors.

Thanks for your reply, I'll try somethin custom.

Ok after further research it seems that the NetworkBehaviour editor class uses ImGUI to draw all properties in the inspector and if you use a property drawer that makes use of UI toolkit, it will show the No GUI Implemented text because you cannot draw UI Toolkit stuff with ImGUI. So the only solution to this is to migrate the NetworkBehaviour editor to UI toolkit or create an ImGUI version for each attribute in the package.
I recommend just submitting a ticket to unity that UI toolkit property drawers don't draw when using them in a NetworkBehaviour and hope they handle it themselves.


Compile commands only generates entry for nothing.c

I'm trying to generate a a compile commands json file in a mono repo. Unfortunately, it is only giving me a compile_commands.json with nothing.c entry and not my files.

Node Version: 12.18.0 (npm 6.14.4)
Platform: Linux ubuntu 18 kernel 4.15.0
Compiler: gcc 7.5.0
Module: custom

Incomplete compile_commands.json
It's a mono repo with multiple packages, one requires a native add on, it compiles fine with:
node-gyp clean
node-gyp configure
node-gyp build

It also compiles fine with rebuild command. However when I try to generate a compile_commands.json it only includes one entry for nothing.c and not for the files specified in binding.gyp:
node-gyp clean
node-gyp configure -- -f gyp.generator.compile_commands_json.py
node-gyp build

This goes ok but the compile_commands.json file only has one entry for nothing.c.
The repository structure is more or less like:
root
|- node_modules
|- packages
   |- package1
      |- src
   |- package2
      |- src
      |- c
   |- packageN

I'm almost sure that the repository structure has something to do, because i tried in another repository and it seems to be working fine. Just not sure how to work around it. Of if someone else can confirm a similar behavior.

It seems that my problem is because my source code files have the cpp extension instead of cc. compile_commands_json.py only considers c and cc. I created a pull request adding cpp and cxx.


QUARKS-95: Disable MqttOpenTest.testConnectRetrySub from running on Travis

Failed in WebSocketClientTest.testReconnectBytes(), another test with intermittent failures on Travis CI.


Implement @max_inner_dims and @nobarrier

Implementation of @max_inner_dims and @nobarrier modifier attributes relying solely on the CLang.

Do not merge yet. This branch has a bug when @max_inner_dims is first.


implement value to relatively insert a sequence

implement value to relatively insert a sequence.
e.g. with % values.

for example in the middle of the sequence: 50%
or leading: 0%
or trailing: 100%


My initial sprite via https api broken

Setup no longer provides first/last initials.  Replaces last initial with 2 instead of real last initial.  appears it is reading the URI encoded value %20 rather than the initial
You can see issue in your own examples page where John Doe === J2

Direct input to api in browser does not recognize last initial

I'm on it.

Should be fixed now. Can you confirm this? (empty cache if necessary)

Yep - all good again.  Thanks for the great lib and fast response


Colab links always show up by default

Google Colab buttons will show up by default if launch buttons are set to true. This is because colab has a default URL that it uses so it's "on" by default.
I think instead of this, we could just set colab to a true/false value instead of asking for a URL, since to my knowledge there is no other Colab URL available.
thanks @colliand for reminding me about this!

I think I already have a fix for this.

Also, one for repo buttons so that they are not hard-wired to github, but can get what they need from _config.yml.
Could you point me to the tests for these?

I think this is where you'll want to look: https://github.com/executablebooks/sphinx-book-theme/blob/master/tests/test_build.py#L211

Google Colab buttons will show up by default if launch buttons are set to true. This is because colab has a default URL that it uses so it's "on" by default.

Are you sure that this is true? I am looking through the changes that I have made and I have not done anything specific to 'colab', and I am not getting a colab link under the launch button.
Relevant _config.yml section
#######################################################################################
--
  | # Launch button settings
  | launch_buttons:
  | notebook_interface        : jupyterlab
  | binderhub_url             : [An internal URL here]

Omitted from the launch button settings.  This is documented for _config.yml:
colab_url: "https://colab.research.google.com" but it is not required and I don't think any code forces it.
I think the existing code already handles this appropriately. If colab_url is not in the launch_buttons section, the generated HTML won't have a colab launcher.
That is not to say that it might be good to have a test for the inclusion of colab_url, but the existing test that I found does not have it.
Perhaps this issue can be resolved.

@choldgraf
I got confused after trying the existing tests. From the tests, it appears that the Google Colab button will show up by default. I think that is an artifact of how the testing is done. I tried to isolate this by trying different "command line" defines in the tests, but attempting to define values for the URL did not work.
In my fork, I had simply removed the corresponding line in the HTML template since our GitHub instance is Git Hub Enterprise and it is behind a firewall, so we can't reach Google Colab at all. So True/False, like Thebe, would work. If you switch to that, then creating a "replacement" for Colab would not have a mechanism for specifying the URL. This seems to be the way Thebe works. I am not really familiar with how that is different from mybinder.
I like the way the binder URL works because we can send our proprietary stuff to our instance of binder. I think it would be nice if the "Colab" and "Thebe" options behaved the same.
I created a "binderized" experiment to explore the Colab button behavior. In each variant, the Colab button was not generated which I think is the correct behavior.
tl;dr

I think that Colab links do not show up by default from jupyter-book.
I have not yet figured out how to modify the tests that exist to demonstrate that.

I found the config option to jupyter-book to see what conf.py gets generated.
It seems like testing might be better if tests ran from _toc.yml or _config.yml files
Is there some conflict between conf.py generated from jupyter-book and the one being used in sphinx-book-theme?

hmmm - so I can't tell if this is a bug or not anymore haha :-)
In general, nothing should behave too differently in the theme in Sphinx directly vs. being used with jupyter-book...unless a different default value were used or something but I don't believe that is the case.
I think the config that we'd want to test is just to add any of the binderhub/jupyterhub launch button configs, and not colab, and confirm that the colab HTML doesn't make it into the page.

@choldgraf I don't think that the bug exists. You can launch this "binderized" experiment from a badge corresponding to each branch. The generated books all have the binder rocket where there is a notebook and none have Colab. (If I have done something wrong in the experiment, please just create an issue and I will investigate further.)
I agree that the tests should behave the same. I think there is something a little off about how it is being tested. I want to understand better the "architecture" - there are a few moving parts in here, jupyter-book, sphinx-book-theme, ...
I want to understand the relationships so that I can easily add (fix if necessary) some tests. I also want to create a PR that will allow people to generate books behind firewalls, using for example GHE. I have the changes working, but I want to be able to create a PR with proper testing so that it will be easy to approve the merge.
Would it be better if I opened up a discussion about how the tests behave?

sure thing - I think we should treat it as an exercise to improve the developer docs a bit 👍

this one was closed by https://github.com/executablebooks/jupyter-book/pull/1026


Improve performance of coverage tool

As it turns out, the overhead on complex programs of the prior mechanism for measuring coverage was too substantial for me to be able to efficiently compute the coverage of the EVM semantics, which I am trying to do so I can get that data to IOHK.
In order to solve this, I rename #appendToFile to #logToFile and give it a hook IO.log which maintains a mapping from paths to string buffers and appends to the files only after rewriting ends.
Furthermore, it was discovered that the rule instrumentation used previously made it so that tail recursive K functions did not compile down to tailcalls in OCAML, which led to stack overflows occurring. Here we introduce a minor change to the OCAML backend in the form of a new attribute indicating a function that always returns .K, which is compiled down to a let statement instead of a list append statement, preserving the tailcall.
Finally, in the course of testing this PR I was required to test the behavior of a program on the latest version of OCAML. The bug was still present in that version, so I worked around it, which is why I am not updating the ocaml versions at this time, but in the interests of saving time if we do end up upgrading at some point in the future, I am checking in the changes to OPAM switches and patches required for OCAML 4.06.1 compatibility with K.

Can one of the admins verify this patch?

I haven't tried flushing more often but also it doesn't seem to be an issue...
The patch is definitely submitted upstream, see https://github.com/ocaml/ocaml/pull/1786
As for the java backend, it will work if I add the hook, do you want me to do that? it doesn't really strike me as necessary...


Add support for AWS and/or GCP TTS API

Many people already have access to AWS or GCP API keys for TTS.
Please add support to one or both

Considering I'm personally not really using STT / TTS this is probably a WONTFIX unless someone has interest (PRs are always welcome)!


More about geo-replication in the Asia-Pacific geography

To protect data during hardware or service failures, Azure Storage geo-replicates customer data between two regions in the same geography. And customer data in Brazil is replicated to the South Central US region for disaster recovery purposes since there is only one region in Brazil.
Now, I found the following issue that explains EA region has been discontinued.

[https://github.com/MicrosoftDocs/azure-devops-docs/issues/13138]
.... We no longer support creating organizations in East Asia,

It means that there is only one region (SEA) in Asia Pacific geography now. But it is not clearly stated where the customer data in the SEA region will be replicated to.
As well as Brazil, the destination for replication of customer data for the SEA region should be clearly indicated in the documentation.
Also, as of 2023/03/01, there are a few organizations that still have EA as the region notation in the Organization Settings.
Since the EA region has been discontinued, I assume that these region designations will be updated to SEA in future DevOps rollout, but this may be confusing to users.
I think it is necessary to announce from the official documentation that the data center itself has already moved to SEA and that the region designations in organization settings will be updated to SEA from time to time.

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

ID: 52eb9f91-b952-afb6-6d17-d7092f46440d
Version Independent ID: bb9113a5-8b8b-b20b-9c6f-09a57f53ba7c
Content: Data protection overview - Azure DevOps Services
Content Source: docs/organizations/security/data-protection.md
Service: azure-devops
Sub-service: azure-devops-security
GitHub Login: @chcomley
Microsoft Alias: chcomley

@tomoshi-ms -- Tomomi, thank you for your considered feedback.
@chcomley -- Chrystal, please look into this issue.  This article may be related:

Data locations for Azure DevOps

Thanks for reaching out, @tomoshi-ms. I'm looking into this.

Hi @tomoshi-ms, the primary region for our Asia geo is the SEA region (Singapore) and the secondary is SE region (HK). The secondary is on standby mode and available in case of regional disaster in SEA region and therefore there are no IP addresses available.
For more info on Azure region pairs, please use this page: https://learn.microsoft.com/en-us/azure/reliability/cross-region-replication-azure#azure-cross-region-replication-pairings-for-all-geographies
I'll be closing this issue, but do let me know if you feel the article needs an update please. You can @mention me. Thank you!

Hi @tomoshi-ms, the primary region for our Asia geo is the SEA region (Singapore) and the secondary is SE region (HK). The secondary is on standby mode and available in case of regional disaster in SEA region and therefore there are no IP addresses available.
For more info on Azure region pairs, please use this page: https://learn.microsoft.com/en-us/azure/reliability/cross-region-replication-azure#azure-cross-region-replication-pairings-for-all-geographies

Hello @chcomley , thank you very much for your detailed answer.  Your answers helped me to understand the following:

In the Asia Pacific geography, the primary region is SEA (Singapore) and the secondary region is SE (Hong Kong).
However, the secondary region is in standby mode and is only enabled during regional disasters in the SEA region. Therefore, during normal operation, the IP address of the secondary region is disabled.

Data locations for Azure DevOps page also has a proviso to refer to Data residency in Azure page for more information, but for most users this is unfriendly information.

Data locations for Azure DevOps
..... For more information, see Data residency in Azure.

By reading Data locations for Azure DevOps page and Data residency in Azure page, it is difficult for users to understand the above two points by themselves.
In fact, I have identified several organizations around me that still have East Asia as their region. They still believe that their data resides in the SE region. Thus, when they are told that the primary region is SEA and the secondary region SE is only valid during regional disasters, they will be confused, "But my organization still exists in the SE region, right?".
For this reason, I think the following statement should be added to either Data locations for Azure DevOps page or Data protection overview page.
Would you consider adding it to the Learn site?

Currently, all of the organization's customer data in the Asia Pacific geography is deployed in the SEA region data center.
Some organizations in the Asia-Pacific geography still show "EA" as region designation, but these will be changed to "SEA" from time to time.
In the Asia Pacific geography, the primary region is SEA (Singapore) and the secondary region is SE (Hong Kong).
The secondary region is in standby mode and is only enabled during regional disasters in the SEA region. Therefore, during normal times, the IP addresses of the secondary regions are disabled.

Hello @chcomley , do you have any updates?  Our important customers are very eager for these informations to be released by Microsoft.

Currently, all of the organization's customer data in the Asia Pacific geography is deployed in the SEA region data center.
Some organizations in the Asia-Pacific geography still show "EA" as region designation, but these will be changed to "SEA" from time to time.
In the Asia Pacific geography, the primary region is SEA (Singapore) and the secondary region is SE (Hong Kong).
The secondary region is in standby mode and is only enabled during regional disasters in the SEA region. Therefore, during normal times, the IP addresses of the secondary regions are disabled.

Hi @tomoshi-ms, I've contacted the PM again.  Feel free to reach out to my Microsoft email address and I can connect you, otherwise I will update you as soon as possible.

Hi @tomoshi-ms, here's some information:
For these two bullets:
•	Currently, all of the organization's customer data in the Asia Pacific geography is deployed in the SEA region data center.
•	Some organizations in the Asia-Pacific geography still show "EA" as region designation, but these will be changed to "SEA" from time to time.
We are aware that there is a bug that incorrectly displays data location in EA, when the data is actually stored in SEA. We have an existing WI to address this bug and the estimated fix should be live no later than 4/21.
For the remaining 2 bullets:
•	In the Asia Pacific geography, the primary region is SEA (Singapore) and the secondary region is SE (Hong Kong).
•	The secondary region is in standby mode and is only enabled during regional disasters in the SEA region. Therefore, during normal times, the IP addresses of the secondary regions are disabled.
These suggestions are already captured in our existing documentation (data protection) and Data residency in Azure.
Hope this is helpful.


Crash on changing dark mode status after visiting Popular tab when internet is off

Steps to reproduce:

Turn device internet off (wifi and data both)
Close the app from recent apps (if applicable)
Start the app
Navigate to the Popular tab and then to a different tab
Change Dark Mode status (if currently light, set dark; if currently dark, set light)

Expected outcome:
App changes and does not crash.
Actual outcome:
App changes briefly and then crashes.
Notes:
This happens only when internet is off.  If internet is on for app launch, even if internet is turned off afterword, no crash is experienced.  However, after re-launching the app, the new dark mode status works just fine.
Crash log:
java.lang.IllegalArgumentException:
  at com.google.android.material.snackbar.Snackbar.makeInternal (Snackbar.java:198)
  at com.google.android.material.snackbar.Snackbar.make (Snackbar.java:157)
  at com.gbros.tabslite.TopTabsFragment$subscribeUi$1$1.run (TopTabsFragment.kt:66)
  at android.os.Handler.handleCallback (Handler.java:938)
  at android.os.Handler.dispatchMessage (Handler.java:99)
  at android.os.Looper.loop (Looper.java:233)
  at android.app.ActivityThread.main (ActivityThread.java:8010)
  at java.lang.reflect.Method.invoke (Native Method)
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:631)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:978)

I think this has something to do with trying to load popular songs.  This doesn't happen when on the popular tab.  Maybe changing dark mode status triggers a refresh of the popular tabs, and the view it chooses for the snackbar causes issues.


[feat][workflow] add instructions on "How to Contribute Connector"

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
Issue
Currently, we do not have instructions on how to contribute a connector.
Devs might forget to add a connector to the download page after they finish the development (e.g., https://apache-pulsar.slack.com/archives/C5Z4T36F7/p1676570833081629).
Even though users can check and download all connectors on the archive page, we still want to provide complete info to users in quickest time.
Solution
We need to document the connector contribution workflow. It might contain the following steps:

Code: How to submit the code?
Doc: How to write a doc for it?
Download: How to add it to the download page?

Alternatives
No response
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

This should be a good supplement for the contribution guide. Compared with Python DevGuide, it provides a page talking about how to add a new module to stdlib https://devguide.python.org/developer-workflow/stdlib/. We can write a similar page for Pulsar connectors contribution.


{"message":"Invalid handler specified"}

hi, I added the repository via hacs, but when adding the integration I get the error:
{"message":"Invalid handler specified"}
Ha version: 2023.7.3

Hello, which version of the component are you trying to install?

I'm using the version: v0.9.0

According to the release notes, version 0.9.0 requires Home Assistant 2003.8 or higher to work.
The best option is to update Home Assistant to the latest version.
If for some reason the Home Assistant update is not included in your plans, you can install version <IP_ADDRESS>, which is compatible with Home Assistant 2023.6 and higher

thanks for the feedback, and congrats on the integration. It worked!


Add Intel Open Image Denoise

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you including a repo_url? You need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project open source?
[x] Has this project been used successfully in the production of a film, television program, animated content or other projects within the scope of the ASWF. Was it designed specifically for the entertainment industry?
[x] Is the project currently and actively maintained (e.g., accepts pull requests, replies to issues, has been tested against / builds properly with the compilers for this year's VFX platform)?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you added the SVG for your logo to hosted_logos and referenced it in the entry in landscape.yml?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

@theanimator - Reminder on logos for this and Embree :-)

Bump on this @theanimator ;-)

Sorry John - I have you on my todo list but I need to get on my vpn’d machine to get them for ya. Will be on it later today.-LauraSent from my iPhoneOn Aug 11, 2023, at 7:00 AM, John Mertic @.***> wrote:
Bump on this @theanimator ;-)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

No worries - I appreciate it, Laura!
On Fri, Aug 11, 2023 at 12:52 PM Laura Wieme @.***>
wrote:

Sorry John - I have you on my todo list but I need to get on my vpn’d
machine to get them for ya. Will be on it later today.-LauraSent from my
iPhoneOn Aug 11, 2023, at 7:00 AM, John Mertic @.***> wrote:
Bump on this @theanimator ;-)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are
receiving this because you were mentioned.Message ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/AcademySoftwareFoundation/aswf-landscape/pull/757#issuecomment-1675088666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACIOILQJRDVVNBPHCAZH2DXUZPMFANCNFSM6AAAAAA3GE3DYI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Two list updates.

The supplied pull request contains an update for the Audiveris project. Moreover, it adds a new scoring software named Dorico which competes with Finale and Sibelius.

Any signs of life?

Yes, alive and merged 😁. Thanks!


Grab connection before finalizing statement for SQLite

When dropping statement, inner_statement is finalized first,
and then is used to get the raw connection, which may cause a crash.
https://www.sqlite.org/c3ref/finalize.html

It seems like we just need to grab the connection before calling finalize

@sgrif Are there other problems? Please review this commit once more.

The ci needs to pass before any further review is meaningful.

Already fixed by 1ba00c9eda7a3dbd6ec8b229c91c2f2965a34c54


Optimisation levels for benchmarking

This PR adds two flags to the benchmarking pipeline:

--opt-level-ir: This flag is used to run -On passes on the generated LLVM IR module.

--opt-level-codegen: This flag is used for setting optimisation level for machine code generation inside the JIT target machine.

Example:
$ ./nmodl file.mod \
     llvm --ir --vector-width 1 \
     benchmark --run --instance-size 10000000 --repeat 20 --opt-level-ir 2 --opt-level-codegen 2

Note: --opt flag in LLVM visitor has not been removed as we may want to experiment with some specific passes, etc.
Also, some benchmarking and JIT driver functions have been refactored from class members and static utilities. More documentation comments were added.
fixes #616

@georgemitenkov : looks good to me but I have two questions:

--opt-level-ir 2 flag should not be in the benchmark section but in llvm section. This way, I can use it to compare optimisations at IR level?  (wouldn't mind if you move --opt-level-codegen 2 also in codegen if you prefer to keep them together)
when we dump .ll file to output directory using -o <dirname>, we should dump IR after optimisation pass.


Hightlights in code blocks

Hey!
Thanks for this plugin. This is not really an issue, but rather a question. I didn't find this in the docs:
How can you use the data elements like shown here? https://revealjs.com/code/#line-numbers-%26-highlights
This kind of works
#### Hello from Cover Page
---

- test dfkldsfjklsd
- sdfjlsdfkljlk

		Apples
		$1
		7

		Oranges
		$2
		18

but introduced additional tags like tbody.

I missed this feature in the last few weeks, too
Im no really sure what a good markdown syntax will be for highlighting line in code.
I'll do some research to see if there are already solutions in other markdown-based tools that I implement for advanced slides as well


Update to latest mattermost version....

Hi, please update to latest version(security fixes), see
https://docs.mattermost.com/administration/changelog.html

I opened #275 to upgrade to 4.10.1, but this version is not released yet. I will upgrade the repository when Mattermost 4.10.1 will be officially released.


skelet app [question]

hi there;
I've read the documentation and it looks nice, but now i want to try all the things and here the documentation doesn't really helps me as i don't know where to start.
I have working apigility project with few doctrine entities that are working as expected.
Now, how to use all the goodies from this module, especially filtering?
I expected it to work after including "zfcampus/zf-apigility-doctrine": "dev-master", into my composer.json and I imagined it to work after querying my collection as the following: /equipment?orderBy[id]=DESC
but it doesn't affect the result :(
anyone can help me with required steps to get it all together?
thank you

Hi Tom,
Great work with RollnAPi and great to see it's basis now being worked on here.
I'm new to ZF2 and Apigility and am trying to get this Module working with Apigility but the documentation is non-existant to say the least.
Are you able to point us in the direction of a Skeleton App for this module as the RollnAPi seems to have terminated.
Thanks in advance,
Steve

Roll'n API is now two modules:
https://github.com/TomHAnderson/apigility-oauth2-doctrine-skeleton
https://github.com/TomHAnderson/zf-oauth2-client-test