installing the package under python 3.12 fails

Current
When doing "python -m pip install ecoinvent_interface" in a python=3.12 environment, the installation fails with:
...
Collecting lxml
  Using cached lxml-4.9.2.tar.gz (3.7 MB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [7 lines of output]
      /tmp/pip-install-gp53p6d9/lxml_0e364b4116bd4d7c9728b6f67fdcb15a/setup.py:117: SyntaxWarning: invalid escape sequence '\.'
        is_interesting_header = re.compile('^(zconf|zlib|.*charset)\.h$').match
      /tmp/pip-install-gp53p6d9/lxml_0e364b4116bd4d7c9728b6f67fdcb15a/setup.py:67: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
        import pkg_resources
      Building lxml version 4.9.2.
      Building without Cython.
      Error: Please make sure the libxml2 and libxslt development packages are installed.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

Expected
works under python 3.12

ecoinvent_interface declares an abstract dependency on lxml (no version pinned), but also on pyecospold which has pinned lxml version to 4.9.2.
It seems to me that there are no wheels available for lxml 4.9.2 for linux x86_64 compatible with python 3.12, and hence, installing ecoinvent_interface will try to build lxml instead of grabbing just a binary dist.

A dependency of pyecospold, lxmlh has also pinned lxml to version 4.9.2.

if pull requests:

[ ] https://github.com/sami-m-g/pyecospold/pull/42
[ ] https://github.com/sami-m-g/lxmlh/pull/6

Are merged and accepted, we can test with py312 and update the dependency to pyecospold version

pyecospold 3.5.3 will have a fix for this once the PR is merged

fixed. pyecospold version 3.5.3 is python 3.12 ready now.


Automatically populating $HTTP_RAW_POST_DATA is deprecated

Hello,
When logging in, I receive a 500 error and notice about emergency shutdown as shown in this screenshot:  http://i.imgur.com/ChQFnzU.png
There is a corresponding entry in the nginx log file:
2016/02/11 13:37:49 [error] 15342#15342: *52 FastCGI sent in stderr: "PHP message: Fatal error: Automatically populating $HTTP_RAW_POST_DATA is deprecated and will be removed in a future version. To avoid this warning set 'always_populate_raw_post_data' to '-1' in php.ini and use the php://input stream instead" while reading response header from upstream, client: <IP_ADDRESS>, server: pathfinder.m0ne.com, request: "POST /api/map/updateData HTTP/2.0", upstream: "fastcgi://unix:/var/run/php5-fpm.sock:", host: "pathfinder.m0ne.com", referrer: "https://pathfinder.m0ne.com/map"

I did set always_populate_raw_post_data to -1 in /etc/php5/fpm/php.ini to no avail.  (I'm not sure what "the php://input stream" is... is that in instead of fastcgi://unix:/var/run/php5-fpm.sock in pathfinder.conf?)
Has anyone run into this before?
This seems to be relevant but the answer seems to be to set always_populate_raw_post_data = -1...  https://www.bram.us/2014/10/26/php-5-6-automatically-populating-http_raw_post_data-is-deprecated-and-will-be-removed-in-a-future-version/
PHP Info
# php -v
PHP 5.6.18-1+deb.sury.org~trusty+1 (cli)
Copyright (c) 1997-2016 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2016 Zend Technologies
with Zend OPcache v7.0.6-dev, Copyright (c) 1999-2016, by Zend Technologies
Thanks!

I've seen the error, and it worked after I set it to -1 and uncomment the line.

Interesting.  After a full system reboot it works...
I restarted nginx after making the php.ini change, ,maybe php5-fpm service needs to be restarted as well?
Anyway, thanks for the advice!

Yeah I had the same problem after switching to PHP7. Setting to -1 works for me too:
http://<IP_ADDRESS>/info.php


wialon

Hello the wiatrack pro application uses wialon protocol. Should I use wialon.utf8 to be able to receive messages ?
This is implemented in default.xml ?
<entry key='wialon.utf8'>enable</entry>

You can try with and without, but the value in your example is definitely incorrect. It should be either true or false. Not sure where you even got the enable value.

Thank you.


Podman and SELinux fixes to the Docker setup

The current Docker / Docker Compose setup does not work on Podman due to volume mount issues. It also does not work if SELinux is in enforcing mode, because volume mounts are not correctly labeled.
After applying these changes the setup will work with rootless Podman and SELinux. The changes have been tested on Rocky Linux 9.

I will set up a VM with Rocky Linux 9 and SELINUX=enforcing and dig into this.

I will set up a VM with Rocky Linux 9 and SELINUX=enforcing and dig into this.

Great! Let me know if you want me to test out something - I'm more than happy to do it. The more I can upstream the better :smile:.

I've tested deploying An Otter Wiki on Rocky9 with SELINUX=enforcing with both podman and docker it worked well. For podman the :z flag for the bind-mount was required, docker seems to set the required flags per default.
I covered this in the bran dnew FAQ and added a info box to the podman part of the Install Documentation.
Thank yor for bringing this to my attention!


Warning for shadowing format needs to be more like warning for emacs

Warning 274:

Implicitly opening namespace 'test.fstar.final.' shadows module 'testmain'
in file "/home/milnes/Tuff/FinalFlog/test/src/fstar/fst/TestMain.fst".
Rename
"/home/milnes/Tuff/FinalFlog/test/src/fstar/fst/TestMain.fst"
to avoid conflicts.

is not like Warning N: at
and the emacs regexps that match compilation errors don't like multiple lines.
I've tried and can't get this to match.
It's simplest just to make the error file single line in the system.
I have the warnings/errors matching and a very nice way to configure fstar-mode for
working with Forge and opam vs local (everest) install vs a system compile (with all
the right includes working). I'll doc it on the wiki soon.

And it's printing the wrong case too:

Implicitly opening namespace 'test.fstar.final.' shadows module 'testmain'


Adding hint and Constrained required in case.

The hint of special case in min. height of binary search tree.
Knowledge about Time complexity and space complexity is shared.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Hi, Please read our Guidelines for Contributing before opening a PR.
We are marking this as invalid and closing.
Thanks.


Add containerProps property for arbitrary props on the container

This PR adds a new property called containerProps to allow customization of props passed down to Autowhatever. It exposes the work done here: https://github.com/moroshko/react-autowhatever/pull/45
Hope the README update is enough!
Thanks

+1
We also needed the same.

+1
We also need this change ASAP.

Hi @FabienDeshayes are you still interested in this? I'd be happy to merge it if you've got a minute to rebase it and handle the quick comments above. If not no worries, but I'll close this PR in a few days.

Hi @pimterry thanks for the update.
I don't need it anymore but will look at the comments and try to rebase soon.

Can someone please work on this? I need this as well. Our company got the code audited for accessibility and it has come down to the fact that AutoWhatever defaults to role="combobox" but we need to override that.

Can someone please work on this? I need this as well. Our company got the code audited for accessibility and it has come down to the fact that AutoWhatever defaults to role="combobox" but we need to override that.

@pimterry I know this might sound that I am pushing way too hard, but could I please get a response on my previous comment

Can someone please work on this? I need this as well. Our company got the code audited for accessibility and it has come down to the fact that AutoWhatever defaults to role="combobox" but we need to override that.
@FabienDeshayes @pimterry Can I take over this? I'll probably just fork Fabien's repo and then add a new PR.
PS I am new to this so if you have a better way please let me know.

@pimterry I know this might sound that I am pushing way too hard, but could I please get a response on my previous comment

Can someone please work on this? I need this as well. Our company got the code audited for accessibility and it has come down to the fact that AutoWhatever defaults to role="combobox" but we need to override that.
@FabienDeshayes @pimterry Can I take over this? I'll probably just fork Fabien's repo and then add a new PR.
PS I am new to this so if you have a better way please let me know.

@aashishnagpal as you can see in the readme this project is looking for maintainers. There's a couple of us here and there who help out, but only intermittently when we can. If you have more time and you're interested in taking on maintenance for the project, that'd be very welcome!
That said, if you'd really like this fixed you are welcome to send a PR covering the comments above yourself, and I'll find some time to review it as soon as I can.

@aashishnagpal as you can see in the readme this project is looking for maintainers. There's a couple of us here and there who help out, but only intermittently when we can. If you have more time and you're interested in taking on maintenance for the project, that'd be very welcome!
That said, if you'd really like this fixed you are welcome to send a PR covering the comments above yourself, and I'll find some time to review it as soon as I can.

@pimterry I totally understand and I am sorry for being pushy. Thanks a lot for your response.
I just needed this change as my company is in process of closing out the A11Y audit this month and it's stuck because of this.
I'll put up the PR (along with the implemented comments) in a couple of hours. Thanks in advance for taking out the time to review and merge my PR.

@pimterry I totally understand and I am sorry for being pushy. Thanks a lot for your response.
I just needed this change as my company is in process of closing out the A11Y audit this month and it's stuck because of this.
I'll put up the PR (along with the implemented comments) in a couple of hours. Thanks in advance for taking out the time to review and merge my PR.

This is now finished up & fixed in #785, released as 10.1.0

This is now finished up & fixed in #785, released as 10.1.0


Carolyn branch

Doesn't look like my changes will merge and I can't see why not.  I added pic and deleted the zoom info on index but kept the excerpt with the end '
made changes to aging post, too.
Please help

File name out of alignment. Changed in branch, and opening a new request.


empty pom.xml in a published variant

Expected Behavior
I'd like to publish a variant from generated code with correct pom.xml in the artifact.
Current Behavior
it's published with empty pom.xml while the generated source code has correct pom.xml. Using this variant in another subproject works just fine as pom.xml is correct. Seems like this correct pom.xml isn't included in published artifact
Context (optional)
Hi, I'd like to publish a variant from generated code but it's published with empty pom.xml while the generated source code has correct pom.xml. Using this variant in another subproject works just fine as pom.xml is correct. Seems like this correct pom.xml isn't included in published artifact. So the question is how to include it in the correct way.
I've tried to ask the same question on Slack but got no response
Steps to Reproduce
There is a small project to reproduce the issue https://github.com/mikita-herasiutsin/spike-variant-publish/tree/fix/correct_configuration_for_publishing
Steps to reproduce:

run ./gradlew publishToMavenLocal
check what's inside client-app/build/javaClientV1/pom.xml and .m2\repository\org\spike\client-app-v1-client\1.0-SNAPSHOT\client-app-v1-client-1.0-SNAPSHOT.pom

Gradle version
8.0.2
Build scan URL (optional)
No response
Your Environment (optional)
No response

found a solution by manually creating pom.xml


Create stable-2.12 build / upload jobs

Depends-On: https://github.com/ansible/ansible-runner/pull/822
Signed-off-by: Paul Belanger<EMAIL_ADDRESS>

recheck

recheck


Add more examples?

Is it possible to add more examples so a Laravel newbie like myself could also use the library. For example what would be the best way to update relationships?
Category hasMany Product
Should I add the HasUpsertQueries trait to the Category or Product model to run the statement below?
category->products()->upsert([...])
I want to update or create products that belong to a specific category. How would I solve that?

Can you share the products migration?

@staudenmeir
Schema::create('products', function (Blueprint $table) {
    $table->id();
    $table->string('name');
    $table->string('description')->nullable();
    $table->string('image')->nullable();
    $table->integer('price');
    $table->timestamps();

    $table->foreignId('category_id')->nullable()->constrained()->onDelete('set null');
});

What does the data of new/changed products look like that you would provide to upsert() as the first argument?

Just bunch of JSON data, some of whom has been created or updated on the frontend.
{
  "id": 1,
  "name": "Product 1",
  "description": ...,
  "price": 79,
},
{
  "id": 2,
  "name": "Product 2",
  "description": ...,
  "price": 79,
},

What is id for new products? Is it missing?

It's missing. Shouldn't laravel create it by default?

The database creates it but using the package with an AUTO_INCREMENT primary key is not the most common use case and I want to test the solution before posting it.
What database are you using?

I use MySQL.

The UPSERT query requires every record to have the same attributes:

You need to add id => null to new records.
You need to add the category_id to new records. You can use null for existing ones because the column doesn't get updated (I assume).

$values = json_decode($json, true);

foreach ($values as $key => $value) {
    if (isset($value['id'])) {
        $values[$key]['category_id'] = null;
    } else {
        $values[$key]['id'] = null;

        $values[$key]['category_id'] = $category->id;
    }
}

Product::upsert($values, 'id', ['name', 'description', 'price']);

Here, $json would look like this: [{"id": 1, ...}, {"name": "...", ...}, ...]
Also be aware of this MySQL issue with gaps in auto-incremented IDs: https://stackoverflow.com/q/23516958/4848587

Thanks! So you can't use eloquent relationships but have to specify the category_id.

Yes, at least for now. I'll take a look if I can make $category->products()->upsert() work.


Percent-decoding entire URL components is not valid

These operations are not valid:
https://github.com/sindresorhus/normalize-url/blob/3c7235ebb06a6ec8b753d4d9d24d0b7adf2d425b/index.js#L178-L183
https://github.com/sindresorhus/normalize-url/blob/3c7235ebb06a6ec8b753d4d9d24d0b7adf2d425b/index.js#L243-L245
Since URLs are a textual format, certain characters have semantic meaning. Percent-encoding can be used to escape those characters. For example, if we want a single path component named "AC/DC", we'll have a problem, because "/" can mean a path separator:
/music/bands/AC/DC
             ^^^^^ - 2 components! ❌

So instead, we have to escape the use of "/" within name "AC/DC":
/music/bands/AC%2FDC
             ^^^^^^^ - 1 component ✅

If you percent-decode the entire path string, we irreversibly lose the information that "AC/DC" was supposed to be a single path component.
Instead, the correct way to do this is to split the component (still escaped) up in to its constituent parts, decode each component, escape any characters with semantic meaning, and join them up again. For the path, that means breaking it up in to path components and ensuring "/" and "" characters are escaped again in each component before you rebuild the path string. For the query, it means doing the same for each key and value (not each key-value pair - they need to be broken down in to their smallest subcomponents).

// @oswaldosalazar @ludofischer

Ah my bad - the path example is okay because decodeURI won't unescape some reserved characters, like "/", so it will keep the path component as "AC%2FDC". But for the query decoding, decodeURIComponent really does just unescape everything:
var url = new URL("http://example/?show=Tom%26Jerry&episode=3");

url.href;
// 'http://example/?show=Tom%26Jerry&episode=3'
//                          ^^^ - ❗
url.searchParams.get("show");
// 'Tom&Jerry'

url.search = decodeURIComponent(url.search);

url.href;
// 'http://example/?show=Tom&Jerry&episode=3'
//                          ^ - ❗
url.searchParams.get("show");
// 'Tom'

The problem is that URL.searchParams gives you the parameter already decoded, so we can't rely on that to preserve the original encoding. We can't use decodeURI instead of decodeURIComponent because decodeURI does not bring back the forward slash (it keeps encoded as %2F).


UI localization

Do you have any plans on localizing the UI after #3 is done?
I'd suggest using some online platform like crowdin.com (free) so the community could help easily (without them having to install Visual Studio and you handling pull requests for translation updates).
I'd also like to offer my help translating from English to Czech (cs-CZ) if needed.
Thanks for this great tool.

I can help with German

As Text Grab becomes more fully featured I'll be sure to add UI Localization. Thank you for offering to help with the translations!

I can help for Italian

I can help for Italian

Hello!
I am also interested in translating this app so more people could use it (I can help for the French translation).
What would be needed to make it happen?
I can have a stab at a PR if some development are needed.
Thank you!

hello @Indigo744 translation would be great. I would appreciate the help if you wanted to open a PR.


URGENT - labs need a reset code button

The "revert to saved code" button is great, but it has replaced the "reset code" button. Labs have seed code, if it gets messed up, the campers don't have any way to restore it. The seed code is often important to pass the tests.
I think this is really important to bring back before the launch.

The current behaviour is that everything clears the code in local storage and uses the code in the db (if it exists). That means there are a couple of ways to go.

Don't save lab code in the db
Add an extra button specifically for labs, so it has both "revert" and "reset" as options.

I very much prefer the first option, but it depends how long people are expected to spend on labs. i.e. how bad it would be if they lost their work.

Some labs can be pretty long, more than the current certification projects, most I would say are comparable to current certification projects. I would still prefer the reset button than the save button

is this an issue for the community members there are no labels as of now ?

I'm a little confused. I don't think there's a very clean way of adding both buttons, unless I learn more about how we are storing the project data to our database and retrieving it.  And definitely some of the labs are pretty much the exact same as our current certification projects.
https://github.com/freeCodeCamp/freeCodeCamp/blob/c407c4a5bfec76718024a95996f3287f567dee69/client/src/templates/Challenges/redux/index.js#L157-L180

I do have every attention on adding a seconf button though. Just need to experiment and find more info.

Even if the button type was changed to be labeled as revert , the save code button would store the code because it is a multifile challenge type.

We're a bit time constrained, so it might be easiest to create another challengeType so this isn't saved in the db (just locally).
We can look into complicating the UX next year.

Just a quick thought - it might be nice to have the reset button be a dropdown that allows you to reset specific files - reset -> html | css | js | all. Maybe too complicated for right now as well, but in my experience, I sometimes just want to reset some one or some of the specific files.

that would be nice! we can put it up as an issue to do later (or by contributors)

Just take it man major pointSent from my iPhoneOn Dec 7, 2024, at 12:26 PM, Ilenia @.***> wrote:
that would be nice! we can put it up as an issue to do later (or by contributors)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


Android 10 problems

Please fix problem with autofill at Android 10 https://prnt.sc/tdmjdi
save to google smart lock canceled16: Credentials API's save confirmation dialog has been disabled to avoid conflicts with the Android Autofill feature. This choice may be overridden via CredentialsOptions.
I enabled https://prnt.sc/tdmjnn
https://prnt.sc/tdmjx8
https://developers.google.com/identity/smartlock-passwords/android/store-credentials
Thanks for help

I had this option enabled on a previous version, but removed it since the documentation suggests you don't force it to keep the UI consistent with other apps.
This works on every project I've used it in, all the way until Android 10.
Are you wrapping the calls in try..catch like the example project is? You get this API error, but smart lock still works since it brings up the native UI.

@Gustash  It i only logbox), release works without crash

I assume you're doing a console.error. If you remove that, does it work?

So maybe fixed https://prnt.sc/tdmv6f

I assume you're doing a console.error. If you remove that, does it work?

Problem was with Google API

The error is disingenuous. It's not an actual error, it's more of a warning. It should function just fine if you ignore that

I'm closing this "error" can just be ignored. I'm not adding forceEnableSaveDialog to this library because the documentation recommends you don't use it to keep the flow consistent between apps.


Fix schemas containing both properties and additionalProperties

Fixes https://github.com/kube-rs/kube-rs/issues/844 with a second rewrite rule in StructuralSchemaRewriter.
(Note: This doesn't work for #[serde(flatten)] foo: serde_json::Value. But I think that's the wrong thing to flatten anyway.)

Codecov Report

Merging #845 (b89cc98) into master (0b5e803) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #845      +/-   ##
==========================================
+ Coverage   70.31%   70.39%   +0.07%
==========================================
  Files          59       59
  Lines        4184     4195      +11
==========================================
+ Hits         2942     2953      +11
  Misses       1242     1242

Impacted Files
Coverage Δ

kube-core/src/schema.rs
90.00% <100.00%> (+2.50%)
:arrow_up:

kube-derive/tests/crd_schema_test.rs
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Support :prepend and :append for the select helper

Fixes #147 for Bootstrap 4

Thanks for putting this together! Can you take a look into the test failure?

Looks good 👍

@donv, was there a reason why this was only added for select, and not collection_select or grouped_collection_select?

No, no reason.


Edabit Solution using Python

Want to Use Python for Edabit Code
Please Consider This Repo :- https://github.com/Py-Droid/Python-Thunder

Hi. I would love to contribute. Please assign it to me.


The map goes white ?

Hi and thanks for sharing this amazing project. I recreated the steps and I was successfully able to work with it at first. However, I rotated the camera and redeployed the app and the map goes white both on unity and the Hololens. Do you have any idea what the problem may be ? I am running this project with similar HolotToolkit version and the extension with a similar unity version.
Thank you

I am not sure what you mean. Can you give more of an explanation what you are doing, or possibly post a short video showing what's happening?

Hi. Thanks for the response. I believe there was a problem with the way I used your code from the blog. I followed the tutorial again with a new project and everything seems to be working fine. However, it does take some time for the map to load even on unity which I think is due to my slow internet connection. I have some problems about lowering the size of the map but it works very well.
Also, is it possible to use the satellite image to create this map in 3D mode. I am currently trying to use it in the 2D map but I wanted to see if you also tried to make this map as well.
Thank you for your help

That you can do. If you download Walk the World from the Store you can see for yourself this is perfectly possible with satellite pictures, and in 3D. You can get those from various places. You will need to get a Bing elevation API, and adapt the code to download satellite pictures from, for instance, Google Maps. Be aware this is technically violating their TOS ;)


Space around zones value gets reset to 16

Microsoft PowerToys version
0.49.1
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Couldn't really identify when the value gets reset but I would say on update to new version, and sometimes on pc restart. It's really annoying as the default value unnecessarily wastes screen space by leaving empty space between the zones and/or edge of screen.
Has been happening for a lot of version now. Happens also with the latest one.
✔ Expected Behavior
value should remain as set no matter what happens with fancy zones (crashes, runs as admin, etc)
❌ Actual Behavior
value gets reset to 16
Other Software
No response

meh, duplicate of at least #13675


[Question]: How to use managed identity with DotNetCoreCLI@2 task

Task name
DotNetCoreCLI
Task version
No response
Environment type (Please select at least one enviroment where you face this issue)

[ ] Self-Hosted
[X] Microsoft Hosted
[ ] VMSS Pool
[X] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Ubuntu
Question
Hello, for AzureCLI task, it supports azureSubscription where we can pass in a service connection with managed identity. But for DotNetCoreCLI, is there a way to use managed identity/service connection?

Thanks

Hi @yanggaome, Unfortunately DotnetCoreCli doesnt support MI directly, but you can pass token from azurecli as shown here


feat(tabletoolbar): add output for custom cancel method

Closes IBM/carbon-components-angular#1611
This feature just adds an output to the ibm-table-toolbar so developers can execute something else at the moment they click on cancel on the toolbar when items are selected.
Changelog
New

cancel Output for ibm-table-toolbar

I will add test for this, the other spec file modified here was to fix the lint, you can check the commits 😄

It's weird to see that there's still diff in the spec but ah well

New, higher quality, spaces 😂


Fix typo in deferred device deletion

Signed-off-by: Shishir Mahajan<EMAIL_ADDRESS>

ping @rhvgoyal


Vet clinic database: add "join table" for visits

Activities
Created a table named vets
Established many-to-many relationship between the tables species and vets and created table specializations
Established a many-to-many relationship between the tables animals and vets and created a table visits
Inserted corresponding data in the created tables
Did queries that required joining the tables
Updated the REAME.md file

Hi @FrankMugagga,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

The Pull Request has a proper title and description. ✔
You have added a descriptive Readme file. Good Job. ✔
Project has met all the technical aspects you needed for this project. Keep rocking! ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


3.3.1 broken - "no permission"

(Required) What is your Quests version / build number (do not say "latest")?
3.3.1
(Required) Please describe the problem and how to replicate it. Any console errors?
after updating to 3.3.1 commands dont work anymore and every user get a "no permissions-message" in chat

Duplicate of #379


chrome's deleted , dashboard's editors can see ..

In the chrome wakatime deleted, why in the DashBoard or Editors can see chrome

You've uninstalled the Chrome extension but still see Chrome listed on your dashboard?

@alanhamlett yes...

You see Chrome activity for days after you uninstalled the extension? The past coding activity will still be on your dashboard, but no new activity should be generated after uninstalling the Chrome extension.


Trying to TSlint Javascript (.js) files

I have a straight up .Net Core Razor Pages application, that includes a .js file.
There isn't a tslint.conf file, there isn't any Typescript files. But for my .js file, I'm seeing this error (it's marked at the top of the file):
{
	"resource": "/c:/Users/John Tsombakos/Source/Repos/webapp/Web/wwwroot/js/app.js",
	"owner": "typescript",
	"code": "1",
	"severity": 8,
	"message": "Failed to load the TSLint library for the document c:/Users/John Tsombakos/Source/Repos/webapp/Web/wwwroot/js/app.js\nTo use TSLint for single file install tslint globally using 'npm install -g tslint'.\nTSLint has a peer dependency on `typescript`, make sure that `typescript` is installed as well.\nYou need to reopen VS Code after installing tslint.",
	"source": "tslint",
	"startLineNumber": 1,
	"startColumn": 1,
	"endLineNumber": 1,
	"endColumn": 2
}

The option
// Control whether tslint is enabled for JavaScript files or not.
"tslint.jsEnable": false,
is not enabled (false, as default).
No, I do not have tslint installed globally, as it's only used in Angular projects, where it is included in their package.json files.

This was closed with no resolution?


PA Onboarding - Add PA receiver to current PA organization

Problem statement
There is currently no receiver for PA at the state level. The receiver needs to be added the local, staging, and production environments.
Acceptance criteria

[ ] PA-ELR.ELR added to organizations.yml
[ ] PA-ELR.ELR added to staging settings
[ ] PA-ELR.ELR added to production settings

added receivers to staging and prod


Run fmt test when formatters change

Description
Have the tests run when their formatter changes.
Include the terraform package in the formatter dockerfile for use in the cloud-deploy-infra repo.
Release notes:
n/a
Checklist

[x] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[x] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)

Codecov Report

Merging #3400 (3907c1a) into main (1c9ab39) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #3400   +/-   ##
=========================================
  Coverage     80.51%   80.51%
  Complexity     2398     2398
=========================================
  Files           293      293
  Lines          9523     9523
  Branches        649      649
=========================================
  Hits           7667     7667
  Misses         1603     1603
  Partials        253      253

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update licensing-ps-examples.md

Seems functions and scripts are old and there is no attribute named isLicensed from MS Graph output anymore.

@barclayn
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


New package: CharFuncPricing v0.1.5

Registering package: CharFuncPricing
Repository: https://github.com/jherekhealy/CharFuncPricing.jl
Created by: @jherekhealy
Version: v0.1.5
Commit: 81ec9e340e0db8192222f1ced5dbaa2906362fe8
Git reference: master

[noblock]

[noblock]


[Feature]: Allow for histograms to be displayed in project view

Description
Currently histograms that are logged to WandB are only shown in the run view itself, and not when viewing this run on a project page. Since this makes it difficult to compare runs - one has to click on each run to see it's histograms - it would be nice if the histograms were also showing on the project page.
Suggested Solution
I understand that it is difficult/impossible to display histograms from multiple runs in one graph. Maybe we can have the histograms only displaying the most recently selected run, or allow for a dropdown menu on the histogram to choose which of the runs that are currently being displayed we want to see on the histogram (this could even be a global setting to allow for a more qualitative user experience).
Alternatives
No response
Additional Context
No response

Hi @ArneTroch99 thank you for this feature, I have now made this request to our engineering team. In the meantime, you may want to check the option of using Weave in the Projects page. If you are logging the data as wand.log({'data': data}) besides the histograms of this data, you can add a Weave panel in your project page and query these with an expression like: runs.summary["data"]["values"]
This will display your data from your runs as table but you can change the render option from the top right gear to Histograms and have these for your selected runs. Please let me know if this helps, and if have any further questions.

Hi @ArneTroch99 I am closing this ticket for now, as the engineering team will follow up with you once this feature is implemented. Please let me know though if you have any further questions regarding the Weave workaround.


Feature/meditor 731

To test, please run package/app's test via npm t. You can also run queries directly on the API, or visit the documentation to learn about the search endpoint (where you can also run queries). Since there are changes to the package.json file, you'll need to run npm i in package/app.

This looks awesome, thanks Ben


CAMEL-19912: camel-core-model - Add factory-bean parameter

Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

SplitterUoWIssueTest is a flaky test unrelated to this


IAPP Migration: Surveys

Describe the Task
Migration of IAPP "surveys" into InvasivesBC app
Acceptance Criteria

[x] Load data
[ ] Make UI

Notes
The Survey data is to be keyed to the Site data on the SiteID variable.
For every SiteID there will be at least one survey and up to 10 or more.
But in some cases a site may not have a survey as the surveys might have been deleted or merged at some point.
Do not display any site that does not have  survey.
The order of the survey data is to be maintained as in the survey list. They data is sorted by species with the surveys on each listed from newest to oldest.
The comment field in the survey has been truncated to 300 characters.
3 jurisdiction and jurisdiction percentages can be displayed within the site list as those are replicated on each survey per site.
Additional survey variables can be included in the survey display should space and layout warrant their inclusion.
Samples

SiteID, SurveyID, SurveyDate, EstArea, WeedsFound, MapCode, CommonName, Species, Genus, SurveyAgency, Jur1, Jur1pct, Jur2, Jur2pct, Jur3, Jur3pct, EmployerCode, Density, Distribution, SurveyType, PaperFileID, Comment

352565,782342,2020-11-06,0.0010,Y,BO,"Bohemian knotweed   ",FALL,BOH,FVIP,MOP ,100,    ,000,    ,000,200817,"2: Med (2-5 plants/m2)   ","5: a few patches or clumps              ",O: Operational  ,                    ," "

352564,782338,2020-07-09,0.0030,Y,YI,"Yellow iris <5m2    ",IRIS,PSE,CIPC,MOF ,100,    ,000,    ,000,200967,"1: Low (<= 1 plant/m2)   ","2: few sporadically                     ",C: Cursory      ,PC3                 ,"3 small plants"

Loading data took a bit longer, but it will now work for other kinds of attributes (treatments, biocontrolouputs, etc), as well as support longer running imports (no issues with keycloak sessions etc).
@dogcomplex stepped up and is going to work on the UI stuff this afternoon/tomorrow while I'm off to keep it moving forward and restyle the Site attributes as part of that.


[ja] Translate docs/concepts/security/cloud-native-security.md into Japanese

This is a Feature Request
What would you like to be added
Translate /content/en/docs/concepts/security/cloud-native-security.md into Japanese
Why is this needed
There is no Japanese translation for this page
Comments

File to translate:

https://github.com/kubernetes/website/blob/main/content/en/docs/concepts/security/cloud-native-security.md

Original:

https://kubernetes.io/docs/concepts/security/cloud-native-security/

日本語に翻訳する際は、以下のドキュメントを参考にしてください。

Japanese Localization document: https://kubernetes.io/ja/docs/contribute/localization/

/area localization
/language ja
/help

/triage accepted

/assign


Dynamic imports of routes (code splitting) requires vue.config.js tweak for IE11 support

Version
3.0.0-beta.16 - 3.0.0.-rc.3
Reproduction link
https://github.com/drewtownchi/VueRouterTranspile
Steps to reproduce

Create a new vue cli project
Change the about route import in router.js to const About = () =>  import ("./views/About.vue");
npm run serve
browse to home in IE.
Click on the about link and see the error in console [vue-router] Failed to resolve async component default: TypeError: [object Object] is not iterable
Create a vue.config.js
module.exports = {
transpileDependencies: [
"vue-router"
]
}
npm run serve again
Visit the home page and go to the about page again.  It works this time.

What is expected?
The page should display with out an error in the console and load the page
What is actually happening?
The page is not loading due to the dynamic import.

Documentation should be provided that dynamic imports or vue router code splitting needs to transpile the vue-router dependency or that an additional polyfill is necessary.  I'm not sure what the appropriate workaround is (if it is possible to get this to work without the transpile in vue.config.js)

Same issue here. Already tried with babel-polyfill.

I hit the same error recently while using a dynamic import to load a polyfill.
The polyfill IE11 needs is es6.array.iterator. By default, vue-cli includes es6.promise.
If you'd rather avoid transpileDependencies you can specify both in babel.config.js:
module.exports = {
  presets: [['@vue/app', {
    polyfills: [
      'es6.array.iterator',
      'es6.promise'
    ]
  }]]
}

Hm, weird. This was supposedly fixed in babel-preset-env back in February:
https://github.com/babel/babel/pull/7400

@LinusBorg That PR fixed detection of Promise usage in app code or dependencies listed in transpileDependencies. Prior to it, Babel was only providing core-js es6.promise (as vue/babel-preset-app currently does) so even adding vue-router to transpileDependencies would cause this error in IE11.
You can confirm this by following the same steps as @drewtownchi's instructions but instead of steps 6 and 7, adding this to main.js:
function fakePromise() {
  return Promise.all()
}

Relying on transpileDependencies is risky because even if you don't use some external dependency, you will get the same error using import() or require.ensure() to load an external script.

I took a look at the webpack docs and tried the Promise polyfills listed there and both work on their own, so it's specific to the modular core-js polyfills used by babel-preset-env (core-js es6.promise only provides Promise and it doesn't include Promise.all() by design)
Incidentally, when I tried to specify polyfills: ['es6.string.iterator', 'web.dom.iterable', 'es6.promise'] like the fix in babel/babel#7400, I got "Cannot read property 'android' of undefined" or "Cannot read property 'chrome' of undefined" in the CLI even though my browserslist covers it, and even when I force targets in babel.config.js:
module.exports = {
  presets: [['@vue/app', {
    polyfills: ['es6.string.iterator', 'web.dom.iterable', 'es6.promise'],
    "targets": { "chrome": "62" }
  }]]
}

Thanks for digging into this.
The error about Cannot read property 'android' of undefined seems to be related to the web.dom.iterable polyfill (removing it from your babel config makes the build run fine.  Is that one also required to make Promises work in IE?
If not, I would like to skip this error in the context of this issue, and make sure that we add es6.array.iterator (you used es6.string.iterator, but I assume that was a typo?) to the default polyfills that get included.

Yep, es6.array.iterator is the one IE11 needs.
The string iterator one wasn't a typo, but I initially tried to fix the error by applying the same polyfills from that Babel PR in my babel.config.js (['es6.object.to-string', 'es6.string.iterator', 'web.dom.iterable', 'es6.promise']) but then I always got the error Cannot read property 'android' of undefined.
What confused me is that if you force Babel to apply the same polyfills by having Polyfill.all() in your code, you don't get a build error.
I presume es6.array.iterator is a subset of web.dom.iterable, and there may be some usecase (outside of webpack's dynamic modules) where web.dom.iterable may be needed, but I don't know for sure. es6.array.iterator is sufficient to fix this issue with IE11.


Example Log migration to FastDDS [6584]

Example code of migration guide:
https://gitlab.intranet.eprosima.com/eProsima/fastrtps/wikis/deprecation
This unneeded and small PR should be done eventually. Doing so as the migration guide example is very useful for learning purposes.
It is divided into 4 commits, in order:

Apply coding style.
Move and copy source and header files.
Fix CMakeLists.txt files of broken unit tests.
Modify header guards, include files and namespaces, adding aliases in old API.

Build status:

Linux
Mac
Windows
Performance

@LuisGP Windows reports 3 new warnings

Build status:

Linux
Mac
Windows
Performance

@LuisGP  there is still one warning

Build status:

Linux
Mac
Windows
Performance

Merging, as Windows failed due to SimpleComunicationSecureBestEffort and performance test failures can't be related.


Cherry-pick Pick "[Featrue](default value) Support bitmap_empty default value (#40364)"

Proposed changes
Pick #40364

run buildall


Update Castle.Windsor and clean-up container structure

With the latest release of Castle.Windsor we no longer need to replace anything and just update and clean-up for MORYX 8.

Summary:

I removed the inheritance tree caused by the global/local split
I removed the registrator and everything associated and put it directly in the container wrapper
All overloads of registration in container and registrator were replaced by extensions
LoadComponents and installers were replaced by extensions
All Resolve overloads were replaced by extensions
Dependency overrides in the registration were replaced with sub resolves in the MoryxFacility

This means that our wrapper of Castle.Windsor has gotten much smaller and all MORYX specific code and behavior is now located in a MoryxFacility as intended by Castle architecture.

Since there was a change in the Resources module I guess there might be some changes in this MR that should be reflected in the Migration guide for MORYX 8. Can you confirm that @Toxantron and if yes add the respective parts? 😬


Kaggle benchmark

Issue #, if available:
ag.utils.generate_csv(inds,csv_path)
Description of changes:
Added training process for five kaggle datasets with some tricks (label smooth, mixup, distillation).
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@aptsunny Please fix the comments before you click 'resolve conversation'.


Update the algorithm for number of nodes to support many users multiple orgs

Currently the algorithm only deploys the highest request for any number of users.
This will not work for many users.
One idea is to make it additive of the highest outstanding request of each user

This was included in v3.2.10


Click event not work

Hi,
I already fixed the problem by set directly on list item the JS code onclick="myfunciton()":
< ul class="list">
< li onclick="myfunciton()>
< li onclick="myfunciton()>
< li onclick="myfunciton()>
< /ul>
Anyway i report you my issue: after I search some element and list is updated, the items not run nomore the click event of codes:
$("list li").click(function () { ... }); same for $("list li").on('click',(function () { ... });
Work only first items, strange situation. I know is not a problem of your plugin but this is problem:
JQuery events click not work on dyamically generated HTML elements...happen me many times, never found a solution...
Thanks

A better solution is to use something like this:
$(document).on('click', '.list-item-class', function(e) {
  console.log('omgomg');
});


Reload UI menu when Settings.prototype changes

Depends on

[x] https://github.com/ManageIQ/manageiq-ui-classic/pull/8886

@agrare Please review
Part of https://github.com/ManageIQ/manageiq-ui-classic/issues/8715

Backported to petrosian in commit 27c38f4270d1dee5d7ef047813f4fc5c184221de.
commit 27c38f4270d1dee5d7ef047813f4fc5c184221de
Author: Adam Grare<EMAIL_ADDRESS>Date:   Fri Aug 4 15:50:41 2023 -0400

    Merge pull request #22646 from Fryguy/reload_menu_when_prototype_changes

    Reload UI menu when Settings.prototype changes

    (cherry picked from commit f39a4986ebab964f406eb9f2ff9836b7d390ab57)


the cost of leader change

Now if a PD node becomes leader, it will load all stores and regions metas first from etcd, if we have many regions, we can't bear leader changing frequently.
Maybe we should think another way to handle cluster initialization.
/cc @qiuyesuifeng @huachaohuang @overvenus

Hi @huachaohuang
We can add some results here, for loading about 300000 regions or even more.

Here are some results we benched on an 8 cores SSD machine so far:
For 300,000 regions, it costs 1.3s.
For 3,000,000 regions, it costs 13s.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Superseded by https://github.com/cdklabs/awscdk-asset-awscli/pull/19


Fix Actions version in Workflows and Go mod tidy on Example App

Issue #, if available:
N/A
Description of changes:
Fixing broken example app and updateing versions of actions used in Github workflows
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@vastin Can we get a version cut after merging this?


Improve response time of app metrics endpoint

After enabling the app metrics endpoint in a production environment we noticed that the response time was between 3 and 5 seconds for few hundred circuits.
After some investigation, I was able to reduce the response time significantly by adding some select_related tables to the main query.
In my test enviroments, it improved the response time by 100x

LGTM, thanks @dgarros , makes sense to optimize for the DEFAULT_LABELS, fetching the 4 attributes that it provides by default. Maybe, to make this evident, I would add a note in the code to "link" the tables selected to it, so in the future we change one, we update the other.

Good point I added a comment


Fix task delay logging/monitoring

...since it was broken by https://github.com/snarfed/bridgy-fed/issues/1354#issuecomment-2389510907.

Getting back to this. What I'm looking for here is, for a given incoming activity or post, the earliest timestamp that we should ideally have seen it. I think that's:

ActivityPub: receive task created time? could be Object.created but we don't create that until inside the receive task
Web: same?
ATProto: time in the firehose #commit

Done! I think.


Fixes #2478 by describing the reporter command line flags

(and setting the right version)

Nice! Minor comments.

LGTM!


Package request: Py-ART

Hey @jjhelmus : how hard would it be it package Py-ART?
Just got my Pi today and think this could be a cool "Py-ART in a can" tool

Not hard to build at all.  Recipe added in b55ed129776f9f26b9f39fcfb98a317af49116fe.  Packages will be in the rpi channel shortly.


Cannot enable a disabled policy when edit the policy

Fixed with PR #1891

@ywk253100
Please verify now?

Verified.


refactor: add spacing to buttons for comment edit

This is a preview of editing a comment. There must be spacing between Kaydet and Iptal as well as between comment text area.

Hi,
I'm not sure if we don't really need a space between the buttons and the text area. The reason is that when I looked at it in three different browsers, there was a space between the buttons and the textarea.
Safari:

Brave:

Google Chorme:

Hi, I'm not sure if we don't really need a space between the buttons and the text area. The reason is that when I looked at it in three different browsers, there was a space between the buttons and the textarea.
Safari:
Brave:
Google Chorme:

Thank you so much for this detailed analysis. My screenshot was from Firefox browser. Unfortunately this doesn't work on Firefox :(

thank you for the info i will work on it


add additional class for hover guest images

I don't know if this happened because it's GSuite or if the class name has changed.
Before:

After:

Thanks for this fix. Indeed, it's hard to maintain such a userstyle with Google's frequent changes.


c++ speech sample question.

hello. i'm beginner.
i'm trying to build this sample. but i can't build and run this sample.

i'm install gRPC, and protobuf, and i try build

Generate googleapis gRPC source code.

but filed to make, and I try use bezel build success.
but when i try 8. 9. i'm can't build

cd cpp-docs-samples/speech/api
make run_tests

error is log.severity.pb.h is not found.

i'm usinb grpc 1.30.0

my question is..
now, this sample is can build? and run?
thank you.

I am not sure if this helps.  It seems the instructions are extremely outdated, I will try to get somebody interested in fixing them.  Meanwhile: the makefiles in speech/api depend on successfully running make in step 7, can you tell us how those instructions failed? Also please include the following information:

This example has only been tested on Linux, please tell us what version of Linux are you using.
What version of protobuf did you use?
Where is protobuf (and most importantly protoc) installed?
Is protoc in your $PATH?  The googleapis/googleapis/Makefile requires changes if this is not the case.

now, this sample is can build? and run?

It does, but with relatively old versions of gRPC, protobuf, and googleapis. You are trying to use it with much newer versions.

I am not sure if this helps. It seems the instructions are extremely outdated, I will try to get somebody interested in fixing them. Meanwhile: the makefiles in speech/api depend on successfully running make in step 7, can you tell us how those instructions failed? Also please include the following information:

This example has only been tested on Linux, please tell us what version of Linux are you using.
What version of protobuf did you use?
Where is protobuf (and most importantly protoc) installed?
Is protoc in your $PATH?  The googleapis/googleapis/Makefile requires changes if this is not the case.

now, this sample is can build? and run?

It does, but with relatively old versions of gRPC, protobuf, and googleapis. You are trying to use it with much newer versions.

thanks, coryan.
i'm failed to build thils sample use grpc 1.30.0
so i retry grpc 1.26.0

i'm use linux ubuntu 16.04.
now i'm using grpc 1.26.0, protoc 3.8.0,
protoc is in /usr/local/bin
yes

grpc 1.26.0 i'm success build googleapis and this sample.
i have another question.
i'm try to run_tests but not response anything.
i open transcribe.cc and

auto creds = grpc::GoogleDefaultCredentials();

seems this line not working.
i already set export GOOGLE_APPLICATION_CREDENTIALS=/path/to/your/credentials-key.json
please help. thank you.

grpc 1.26.0 i'm success build googleapis and this sample.

Yay! 🎉

i have another question.
i'm try to run_tests but not response anything.
i open transcribe.cc and

auto creds = grpc::GoogleDefaultCredentials();

seems this line not working.

Okay, this is probably more of an authentication problem.  You probably tried the ideas below, but in case you have not.   I would try to follow the speech-to-text quickstart first:
https://cloud.google.com/speech-to-text/docs/quickstart-gcloud
Once that works then probably your problems with grpc::GoogleDefaultCredentials() would go away.  You may also want to learn about authentication about GCP:
https://cloud.google.com/docs/authentication

hi, coryan.
i retry 1.26.x (not 1.26.0 release, 1.26.0 has bug)
https://github.com/grpc/grpc/tree/v1.26.x
i finally success run this sample.
thank you!


Updates for Blob Store integration

Hey @0kashi , from a skim-read this looks nice to me! Thanks for the update.
I've not been told of anybody else using AROWorkshop.io for ongoing events that this change would break, so I feel it's safe to merge immediately.


Add way to avoid adding --browse to "hub pull-request" for bump-formula-pr

[x] Confirmed this is a problem with Homebrew/brew and not one or more formulae? If it's a formulae-specific problem please file this issue at https://github.com/Homebrew/homebrew-core/issues/new
[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[x] Ran brew config and brew doctor and included their output with your issue?

When running brew bump-formula-pr, it always passes the --browse argument to hub pull-request (code ref: https://github.com/Homebrew/brew/blob/master/Library/Homebrew/dev-cmd/bump-formula-pr.rb#L322). This doesn't work when running on a headless server such as a CI system, as these systems intentionally don't have a browser configured.
To github.com:Daniel15/homebrew-core.git
 * [new branch]      yarn-0.23.0 -> yarn-0.23.0
Branch yarn-0.23.0 set up to track remote branch yarn-0.23.0 from Daniel15.
Please set $BROWSER to a web launcher
Error: Failure while executing: hub pull-request --browse -m yarn\ 0.23.0

Created\ with\ `brew\ bump-formula-pr`.

I encountered this on Linuxbrew but the code is exactly the same in Homebrew.
I'd suggest adding some configuration options, or a --non-interactive command-line option which would avoid anything that does not work in this environment.
Also see https://github.com/yarnpkg/yarn/issues/2841
brew config output:
HOMEBREW_VERSION: >1.1.0 (no git repository)
ORIGIN: https://github.com/Linuxbrew/brew
HEAD: 59c9ddbc1d9b9dfdeb0514768e0efd0112c63b25
Last commit: 6 hours ago
Core tap ORIGIN: https://github.com/Linuxbrew/homebrew-core
Core tap HEAD: 304e81be80435d9c169e552c858518e62fbcba8b
Core tap last commit: 5 minutes ago
HOMEBREW_PREFIX: /home/daniel/.linuxbrew
HOMEBREW_REPOSITORY: /home/daniel/.linuxbrew
HOMEBREW_CELLAR: /home/daniel/.linuxbrew/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://linuxbrew.bintray.com
CPU: single-core 64-bit haswell
Homebrew Ruby: 2.3.3 => /usr/bin/ruby2.3
Clang: N/A
Git: 2.11.0 => /usr/bin/git
Perl: /usr/bin/perl
Python: /usr/bin/python => /usr/bin/python2.7
Ruby: /usr/bin/ruby => /usr/bin/ruby2.3
Java: N/A
Kernel: Linux 4.7.0-1-amd64 x86_64 GNU/Linux
OS: Debian GNU/Linux 9.0 (stretch)
Codename: stretch
OS glibc: 2.24
OS gcc: 6.3.0
Linuxbrew glibc: N/A
Linuxbrew gcc: N/A
Linuxbrew xorg: N/A

I'd suggest adding some configuration options, or a --non-interactive command-line option which would avoid anything that does not work in this environment.

Sounds like a good idea. Could you try and open a pull request? This document should help and we're happy to walk you through anything else.
Thanks!

I don't have time to work on a PR for this at the moment, but for now I just hacked around it by setting BROWSER=/bin/true.

Glad you found a solution, thanks for letting us know.

I've opened https://github.com/Homebrew/brew/pull/3398 to address this.


Update New-MoveRequest.md

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/move-and-migration/New-MoveRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: heathca sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/move-and-migration/New-MoveRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


remove unnecessary template variable in grafana

Current, two dashboards defined template variables task_role and task. These two variables are useless for the queries. For example, query container_NetIn{job="$job", task=$task} and container_NetIn{job="$job"} are essentially same since task variable select all available task label values as we did.
So, I removed useless variables, this will not affect queries since no semantic changes made.

what's the policy of hotfix? Cherry-pick all fixes to the latest release branch? Maybe we should also pick #1718 .

@xudifsd , yes. we can cherry-pick the two PRs to 0.8.y. please contact the Release Manager for a new release tag/announcement.


MudTable - Multi level grouping not in sync with items selected

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Component
Component name
MudTable
What happened?
A MudTable with multi level grouping and multi selection does not mark nested groupings as selected unless another row-expansion is performed.
Exact same issue as https://github.com/MudBlazor/MudBlazor/issues/5759, but for multi level groupings.
Expected behavior
Intermediate groupings should be able to render their selection state checkbox regardless of whether the innermost group is expanded or not.
Reproduction link
https://try.mudblazor.com/snippet/QuwyOBcmoQHHnOLx
Reproduction steps

Select a top level grouping
Expand top level grouping
Observe that nested groupings do not show as being selected
Expand nested grouping
Observe that all groupings now show the correct selection state

Relevant log output
No response
Version (bug)
7.3.0
Version (working)
No response
What browsers are you seeing the problem on?
Firefox, Chrome
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Can be fixed by updating checkbox state when already doing special treatment of grouped in OnInitializedAsync
diff --git a/src/MudBlazor/Components/Table/MudTableGroupRow.razor.cs b/src/MudBlazor/Components/Table/MudTableGroupRow.razor.cs
index 5c9742303..75f8f4a4c 100644
--- a/src/MudBlazor/Components/Table/MudTableGroupRow.razor.cs
+++ b/src/MudBlazor/Components/Table/MudTableGroupRow.razor.cs
@@ -195,6 +195,7 @@ namespace MudBlazor
             {
                 _innerGroupItems = Table?.GetItemsOfGroup(GroupDefinition.InnerGroup, Items);
             }
+            Context?.UpdateRowCheckBoxes();
         }

         /// <summary>


Duplicated banner in the wrong position

@Koded - I slid an extra alpha banner in here. Bag pardon

@Koded - please can you merge this one in last?


Omit option to add/subtract

I have no idea if this is actually a good idea. In fact, it's probably terrible. But I thought I'd suggest it in case anyone thinks it's a good one.
There are numerous requests to support work days in Moment, but it's a technically challenging problem. The constant-time solution itself is interesting in and of itself, but more importantly, work days and week ends aren't the same for every country.
One way to handle this would be a way to omit values when adding or subtracting. This you could simply specify what values you want to include or not include. It could even work for time scales other than days.
I kno, I kno. Bad idea. But I just had to put it out there. :speak_no_evil:

Well, I'm perfectly fine somebody implementing a plugin, that can be fed working days and then handles addWorkdays and subtractWorkdays. This is fairly trivial to implement, but I don't think it has place in core for now.
So you can write one and share it on github ;-)

I made moment-business to do this (-ish). It only works for western work weeks, and only 1/2 of the methods are general enough to be abstracted to be as general as what this issue is describing.


Clean up xrefs

update PMCID to PMC
Add redundant PMC in beginning of PMC identifiers
Replace PSI-MI:MI: with MI:
Fix spaces in PMIDs
Fix typo in a single EDAM xref

Hi @cthoyt
I had a chance to look at the changes, they all look good. I should be able to merge them later this week after discussing with Michelle.
Thanks for helping us cleanup.
Suvvi


Python: What are Bitwise operators?

Description :
What are Bitwise operators?
Note :

Changes should be made inside the Python/ directory & Python branch.
Issue only for GWOC'21 contributors.
Issue will be assigned on a first come first serve basis, 1 Issue  ==  1 PR.
In the PR, keep this issue's title as your PR's title.

Must Follow : Contributing Guidelines & Code of Conduct before start Contributing.

please assign me (DOCUMENTATION)

plz assign me

I would like to work on this issue via documentation. Thank you!!

Hi I'd like to contribute through documentation on this topic
Harmandeep - python - batch 4

I would like to work on this issue through documentation.
Vibhu - Python Batch 5

Please assign me this issue . I am ready to contribute in any of the three ways.

I would like to work on this issue via documentation.

@adityabisoi
Sir, I would like to contribute to this issue.
Name: Arushi Rautela
Type: Video
Batch: 6

I would like to contribute to this issue via documentation.
Thanks!
Yash Kataria
Python batch-3
@adityabisoi

Please assign me this issue . I am ready to contribute in any of the three ways.

Audio.

Please assign me this issue . I am ready to contribute in any of the three ways.

Audio.

Thank you

Thank you!!
On Sat, 18 Sept 2021 at 20:13, Aditya Bisoi @.***>
wrote:

Assigned #548
https://github.com/girlscript/winter-of-contributing/issues/548 to
@prettylittleminds https://github.com/prettylittleminds.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/issues/548#event-5323080683,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASTXFAARIE5KVWLQGX7YB4LUCSQPXANCNFSM5EIZBU4A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@ankurg132 Is the mentor of this issue. Please reach out to him in case of queries.

@ankurg132 Is the mentor of this issue. Please reach out to him in case of queries.

Ok sir

/assign

Hey @kritikarag, all the contributors are currently assigned for this issue already.

#2438 @ankurg132  Sir, Is it fine for merging?

Yup, added my review in the pull request


KVStore: LargeTxn spill: Store kv pairs of large txns in dedicated CFs

What problem does this PR solve?
Issue Number: ref #8736
Problem Summary:
We will store kv pairs of large txns seperately in a dedicated cf, which is called LargeTxnDefaultCf.
The struct is a double level map start_ts -> pk -> (TiKVKey, TiKVValue). The inner map is RegionCFDataBase<LargeDefaultCFDataTrait>, in which LargeDefaultCFDataTrait is something very similar to RegionDefaultCFDataTrait.
We have to introduce a RegionPersistExtension::LargeTxnDefaultCfMeta in order to parse LargeDefaultCFDataTrait in RegionData, if it actually exists.
We won't actually write anything into persisted region data, until the feature GAs.
Meanwhile, we have also

dbgKVStore that can mitigate the "friend" problem.
A two level iterator for LargeTxnDefaultCf.
A RegionSerdeOpts to enhance original Region::CURRENT_VERSION.

What is changed and how it works?
Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

Interrupted: /tiflash/gtests_dbms AsyncTasksTest.AsyncTasksNormal

/run-unit-test

/hold

/hold


Add support for image loading with CocoaPods 0.36 and frameworks

Hi there,
I am using your library in my hobby app for some time now and I am very happy with it. Unfortunately I found an issue with using it in CocoaPods 0.36. The image loading mechanism fails. Details are here:
http://blog.cocoapods.org/CocoaPods-0.36
This pull request fixes that issue.
Thanks for your work ;-)

I cannot reproduce a crash or failing pod install/update commands.
Please provide more details / an example project showcasing your particular problem.
General information:
CSNotificationView generates the CSNotificationView bundle through CocoaPods.
# CSNotificationView.podspec
s.resource_bundle = { 'CSNotificationView' => ['CSNotificationView/Resources/*.png'] }

I am using the new CocoaPods 0.36 together with swift that is using frameworks internally. The crash does not happen during pod install/update but at app runtime. In my case I was calling [CSNotificationView dismissWithStyle:message:duration:animated] which called [CSNotificationView imageForStyle:]. And the crash happens inside that method. [NSBundle mainBundle] returns nil because the resource loading mechanism did change with the latest CocoaPod version. This is described at http://blog.cocoapods.org/CocoaPods-0.36 (scroll down to Caveats).
For reference, this is my podfile:
use_frameworks!
pod 'CSNotificationView'

Thanks for your contribution.

Update has been pushed. Thanks again!


Linux build error

Tried building the master branch on Ubuntu 22.04 but get the following error:
$ ./gradlew jpackage --info
Initialized native services in: /home/jameson/.gradle/nativeInitialized jansi services in: /home/jameson/.gradle/native
Received JVM installation metadata from<EMAIL_ADDRESS><EMAIL_ADDRESS>JAVA_VERSION=1.8.0_292, JAVA_VENDOR=AdoptOpenJDK, RUNTIME_NAME=OpenJDK Runtime Environment, RUNTIME_VERSION=1.8.0_292-b10, VM_NAME=OpenJDK 64-Bit Server VM, VM_VERSION=25.292-b10, VM_VENDOR=AdoptOpenJDK, OS_ARCH=amd64}
The client will now receive all logging from the daemon (pid: 47791). The daemon log file: /home/jameson/.gradle/daemon/8.5/daemon-47791.out.log
Starting 2nd build in daemon [uptime: 2 mins 51.57 secs, performance: 97%, GC rate: 0.50/s, heap usage: 3% of 341.5 MiB, non-heap usage: 17% of 384 MiB]
Using 4 worker leases.
Now considering [/home/jameson/code/sparrow, /home/jameson/code/sparrow/buildSrc] as hierarchies to watch
Now considering [/home/jameson/code/sparrow/buildSrc, /home/jameson/code/sparrow] as hierarchies to watch
Watching the file system is configured to be enabled if available
File system watching is active
Starting Build
Settings evaluated using settings file '/home/jameson/code/sparrow/settings.gradle'.
Projects loaded. Root project using build file '/home/jameson/code/sparrow/build.gradle'.
Included projects: [root project 'sparrow', project ':drongo']

> Configure project :buildSrc
Evaluating project ':buildSrc' using build file '/home/jameson/code/sparrow/buildSrc/build.gradle'.
Resolve mutations for :buildSrc:compileJava (Thread[Execution worker,5,main]) started.
:buildSrc:compileJava (Thread[Execution worker,5,main]) started.

> Task :buildSrc:compileJava FAILED
Caching disabled for task ':buildSrc:compileJava' because:
  Build cache is disabled
Task ':buildSrc:compileJava' is not up-to-date because:
  Task has failed previously.
The input changes require a full rebuild for incremental task ':buildSrc:compileJava'.
Full recompilation is required because no incremental change information is available. This is usually caused by clean builds or changing compiler arguments.
Compiling with toolchain<EMAIL_ADDRESS>Compiling with JDK Java compiler API.
/home/jameson/code/sparrow/buildSrc/src/main/java/org/gradle/sample/transform/javamodules/ExtraModuleInfoTransform.java:148: error: cannot find symbol
                outputStream.write(inputStream.readAllBytes());
                                              ^
  symbol:   method readAllBytes()
  location: variable inputStream of type java.util.jar.JarInputStream
1 error

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':buildSrc:compileJava'.
> Compilation failed; see the compiler error output for details.

* Try:
> Run with --scan to get full insights.

BUILD FAILED in 963ms
1 actionable task: 1 executed

Pretty sure I have the right dependencies...
$ java -version
openjdk version "21.0.2" 2024-01-16
OpenJDK Runtime Environment (build 21.0.2+13-Ubuntu-122.04.1)
OpenJDK 64-Bit Server VM (build 21.0.2+13-Ubuntu-122.04.1, mixed mode, sharing)

Looks like this was an issue of my env var pointing to a really old java version even though my new java version was on the default path
<EMAIL_ADDRESS>for some reason; updating it to JAVA_HOME="/usr/lib/jvm/java-21-openjdk-amd64/" helped get past the first error.
Now the build fails with:
$ ./gradlew jpackage

FAILURE: Build failed with an exception.

* What went wrong:
Could not determine the dependencies of task ':compileJava'.
> Could not resolve all task dependencies for configuration ':compileClasspath'.
   > Could not resolve project :drongo.
     Required by:
         project :
      > No matching configuration of project :drongo was found. The consumer was configured to find a library for use during compile-time, compatible with any Java version, packaged as a jar, preferably optimized for standard JVMs, and its dependencies declared externally, as well as attribute 'javaModule' with value 'true', attribute 'org.gradle.native.operatingSystem' with value 'linux', attribute 'org.gradle.native.architecture' with value 'x86-64' but:
          - None of the consumable configurations have attributes.

Wow I can't catch a break
$ git clone --recursive https://github.com/sparrowwallet/sparrow.git

Cloning into 'sparrow'...
remote: Enumerating objects: 24581, done.
remote: Counting objects: 100% (4242/4242), done.
remote: Compressing objects: 100% (1808/1808), done.
error: RPC failed; curl 92 HTTP/2 stream 0 was not closed cleanly: CANCEL (err 8)
error: 5154 bytes of body are still expected
fetch-pack: unexpected disconnect while reading sideband packet
fatal: early EOF
fatal: fetch-pack: invalid index-pack output

Finally got it working git ssh, which was a pain because my yubikey kept timing out to authorize the ssh requests.


Fix ops.stop_gradient for KerasTensor

Fixes #20124.

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 57.54%. Comparing base (6fd4873) to head (380b973).

Files
Patch %
Lines

keras/src/ops/core.py
0.00%
1 Missing and 1 partial :warning:

:exclamation:  There is a different number of reports uploaded between BASE (6fd4873) and HEAD (380b973). Click for more details.
HEAD has 4 uploads less than BASE

Flag
BASE (6fd4873)
HEAD (380b973)

keras
3
1

keras-tensorflow
1
0

keras-jax
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #20125       +/-   ##
===========================================
- Coverage   73.70%   57.54%   -16.16%
===========================================
  Files         501      501
  Lines       46988    46990        +2
  Branches     8663     8664        +1
===========================================
- Hits        34632    27041     -7591
- Misses      10659    18275     +7616
+ Partials     1697     1674       -23

Flag
Coverage Δ

keras
57.54% <0.00%> (-16.09%)
:arrow_down:

keras-jax
?

keras-numpy
57.54% <0.00%> (-0.01%)
:arrow_down:

keras-tensorflow
?

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

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


Migrate pair_page.dart to hooks

@DreadBoy Tested your PR. It throws the following error when pairing a watch. https://pastebin.com/iBV4sb1k

I forgot useEffect is sync in Flutter, it's async in React. :/ I fixed it and added test for this case.

@DreadBoy Currently the skip button does not show up for me even on a fresh install of this PR. Is the skip button showing up for you?

It's shown only if you navigate to PairPage from MyWatchesTab, if you navigate from FirstRunPage, you can't skip this setup.

My understanding of the design is the reverse being true.

@matejdro what are your thoughts on "skip"

I think @piggehperson should decide here

Skip button is supposed to show on initial setup, but not when opened from Devices page. You might skip the initial pairing for debugging purposes or just to check out the app before switching, but when navigating from the Devices page, you can just press back and go back to the main page.

@DreadBoy Making progress! Currently I'm running into the current error when pairing a watch with your PR - https://pastebin.com/QgF0X2sw

Hmm, that's strange. :/ I don't have any Pebble device so I can test only with unit tests and this exception doesn't throw in tests. Fix is simple, I can move call to pairedStorage to next frame but I'm not sure how to reproduce it in tests.

I can't reproduce that even if I mock pairing on PairPage screen like this :(
final _scan = useProvider(scanProvider);
final _pair = useProvider(pairProvider);
useEffect(() {
  Future.delayed(Duration(seconds: 2))
      .then((_) {
        _scan.state = ScanState(
          false,
          [PebbleScanDevice("Test", 1, 'v1', '0000000', 0, true, true)],
        );
      })
      .then((_) => Future.delayed(Duration(seconds: 2)))
      .then((_) {
        _pair.state = 1;
      });
  return null;
}, []);

@matejdro Any ideas on how to replicate pairing in a unit test properly?

Rebased my branch back onto master so it now has the permission fixes and such

Rebased my branch back onto master so it now has the permission fixes and such

@DreadBoy are you still working on this PR?

@DreadBoy are you still working on this PR?

No, I think I'm done unless I missed something.
On Fri, 15 Jan 2021, 18:51 Alice<EMAIL_ADDRESS>wrote:

@DreadBoy https://github.com/DreadBoy are you still working on this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AliceGrey/mobile-app/pull/1#issuecomment-761087731,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ7PFLGRMUKLVQCKSKVPKLS2B6BHANCNFSM4U5WG74Q
.

No, I think I'm done unless I missed something.
On Fri, 15 Jan 2021, 18:51 Alice<EMAIL_ADDRESS>wrote:

@DreadBoy https://github.com/DreadBoy are you still working on this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AliceGrey/mobile-app/pull/1#issuecomment-761087731,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ7PFLGRMUKLVQCKSKVPKLS2B6BHANCNFSM4U5WG74Q
.

No, I think I'm done unless I missed something.
…
On Fri, 15 Jan 2021, 18:51 Alice, @.***> wrote: @DreadBoy https://github.com/DreadBoy are you still working on this PR? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJ7PFLGRMUKLVQCKSKVPKLS2B6BHANCNFSM4U5WG74Q .

@m

No, I think I'm done unless I missed something.
…
On Fri, 15 Jan 2021, 18:51 Alice, @.***> wrote: @DreadBoy https://github.com/DreadBoy are you still working on this PR? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJ7PFLGRMUKLVQCKSKVPKLS2B6BHANCNFSM4U5WG74Q .

Did you make the latest changes @matejdro suggested? Not super clear on mobile sorry.

No, I think I'm done unless I missed something.
…
On Fri, 15 Jan 2021, 18:51 Alice, @.***> wrote: @DreadBoy https://github.com/DreadBoy are you still working on this PR? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJ7PFLGRMUKLVQCKSKVPKLS2B6BHANCNFSM4U5WG74Q .

@m

No, I think I'm done unless I missed something.
…
On Fri, 15 Jan 2021, 18:51 Alice, @.***> wrote: @DreadBoy https://github.com/DreadBoy are you still working on this PR? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJ7PFLGRMUKLVQCKSKVPKLS2B6BHANCNFSM4U5WG74Q .

Did you make the latest changes @matejdro suggested? Not super clear on mobile sorry.

@DreadBoy It looks like you didn't complete the changes @matejdro suggested for pair_provider.dart

@DreadBoy It looks like you didn't complete the changes @matejdro suggested for pair_provider.dart

Yeah I would appreciate if you could fix last two of my comments, they are pretty important.

Yeah I would appreciate if you could fix last two of my comments, they are pretty important.

LGTM!

LGTM!


Revert "Use emojis iff emojione contains as defaultEmojis"

Reverts mazun/ASlack-Stream#73
We don't wanna show the emojis aren't in the Slack emoji list.

Thanks. I had not known the background of using a list manually written.


fix: transliteration of non-combined accent marks

T́ma has to be transliterated to Т or Ть.
Previously, the algorithm did not recognize the acute as a part of the word (instead, it treated it as T and ma)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.518%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1489

Relevant Lines:
1572

💛  - Coveralls


merge tests that are mostly the same

From @hterkelsen on May 5, 2016 23:58
super_in_async[1-6]_test.dart and static_field_override[1-3]_test.dart can be merged into just 2 tests
Copied from original issue: dart-lang/dev_compiler#554

This is not really a DDC issue. While I'd love to see our tests get some love (lots of tiny test files imposes a large cost on our build bots and dev cycle), I think we'll need a team wide agreement to make progress on this.


Migration Guide 0.6

Summary
Adds the migration guide going from 0.5 to 0.6
Details and comments
Closes #1095

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 85.964%

Files with Coverage Reduction
New Missed Lines
%

qiskit_nature/drivers/second_quantization/psi4d/psi4driver.py
2
84.43%

qiskit_nature/second_q/drivers/psi4d/psi4driver.py
2
83.05%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
18576

Relevant Lines:
21609

💛  - Coveralls

I have done the following updates:

ensure all links work as expected
improved the opflow-related comments
included properly working examples in the Factory migration guide


v5 Beta - Exception when combine Enumerable with Async/Await

Hi!
After some experiments with IAsyncEnumerable I found this issue and decided to investigate it more. Surprisingly same exception is thrown with simple IEnumerable and async inside foreach.
Minimal repro:
using System.Linq;
using System.Threading.Tasks;
using Xunit;

namespace LiteDB.Tests.QueryTest
{
    public class AwaitWithEnumerable_Tests : Person_Tests
    {
        [Fact]
        public async Task Query_Enumerable_With_Await()
        {
            var items = collection
                .FindAll()
                .Take(10);
            foreach (var item in items)
            {
                await Task.Delay(10);
            }
        }
    }
}

Exception details
  Message:
    System.AggregateException : One or more errors occurred. (The read lock is being released without being held.) (ENSURE: share counter must be > 0 in Release())
    ---- System.Threading.SynchronizationLockException : The read lock is being released without being held.

LiteDB TargetFramework: both netstandard1.3 and netstandard2.0
App TargetFramework: both UWP and NetCore3.0
Note: after I have added new test from minimal repro, it started to fail only when I run it single. When I run all tests - it doesn't fail for some reason.

Workaround - materialize IEnumerable before async/await usage.
But it doesn't help in cases, when I don't want to read too much from collection.


Make commands for viewing hints

eugene hints should list all the names - codes of the hints we have
eugene hint <name> should give detailed information about the rule/pattern, see ex https://github.com/ankane/strong_migrations?tab=readme-ov-file#adding-a-unique-constraint

This is going to require restructuring hints.rs a little bit, namely we'll need to expose the structure of the hints.

Feels like this closes #30 and #9 on a "good enough for now" level.


Fix build break and update dependencies

Waiting for the CI.

:tada:

Investigating the restore failure. Seems to relate new DNX. I may replace the dnu restore with dotnet restore. Stay tuned.

Nice, is it ready to go you think?
I'm currently building locally and will test with atom for a bit.

PackagesRestoreTool seems to crash the server now.
Also the scripts need their dnx rc2 version updated (I can do that later)

I think it's good enough to unblock the build. But yeah, DNX is going to be removed very soon.

/cc @david-driscoll shall I merge the change now?

Yeah, lets merge for now, still worried it's going to break in 2 days again.

You don't have to way for 2 days. It's already broken now :P. The 16420 build is missing. I upgrade them to 16425. I'll wait for the CI pass.

CI passed. Let's go.


Guards Scala Debug against fetching data outside of presentation comp…

…iler. Scala Model gets data eagerly.

done, thanks
2017-04-13 12:48 GMT+02:00 Simon Schäfer<EMAIL_ADDRESS>
@sschaef commented on this pull request.
In org.scala-ide.sdt.core/src/org/scalaide/core/internal/
jdt/model/ScalaElements.scala
https://github.com/scala-ide/scala-ide/pull/1155#discussion_r111360997:

 if (pkgSym.isEmptyPackage)

   super.getFullyQualifiedName

 else {

   val pkg = pkgSym.javaClassName

   pkg + "." + getTypeQualifiedName('$', /*showParameters =*/ false)

 }

declaringType.flatMap { typ =>

 pc.asyncExec {

   val pkgSym = typ.typeSymbol.enclosingPackage

   if (pkgSym.isEmptyPackage)

     super.getFullyQualifiedName

   else {

     val pkg = pkgSym.javaClassName

     pkg + "." + getTypeQualifiedName('$', /*showParameters =*/ false)

   }

 }.get.left.toOption

If you do a import org.scalaide.core.compiler.IScalaPresentationCompiler.
Implicits._ you can do .getOption(), which has the advantage that a
timeout will be used.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/scala-ide/scala-ide/pull/1155#pullrequestreview-32600224,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALEzS6n-PfQouj2Ei_T8a626INkJK3Mwks5rvf2TgaJpZM4M7XhX
.


Angular-cli local demo inside library project

I would like to setup a demo with angular-cli in a demo directory (located in the root of the library) in order to build it and push it to GitHub Pages.
I ran ng new demo --style=scss from the root of the library project, but it installs its own node_modules.
When I go into the demo directory, I run ng serve just to test if the angular-cli demo runs well (without doing anything else for the moment) but it seems there are conflicts between the two node_modules folders (the one in the root of the library and the other in the demo folder.
Do you have any idea how can I make it work ?
Do I need to put all the library inside the demo made with angular-cli ?
I saw some issues related to this kind of need, but it seems not related with my issue : https://github.com/angular/angular-cli/issues/1465
Hope that is possible 😉

I just tried to create a Demo app with Angular CLI inside the library, and I can run it without problems.
Try to uninstall/install your Angular CLI.
Then you have to exclude in tsconfig file the Demo app: https://github.com/robisim74/angular-library-starter/issues/14

Thank you, it works as is! 😄 But now how can I link my library to it ? I tried to do something in app.module.ts like this:
import {BrowserModule} from '@angular/platform-browser';
import {NgModule} from '@angular/core';
import {NgxSmartModalModule, NgxSmartModalService} from '../../../';

import {AppComponent} from './app.component';

@NgModule({
  declarations: [
    AppComponent
  ],
  imports: [
    BrowserModule,
    NgxSmartModalModule
  ],
  providers: [NgxSmartModalService],
  bootstrap: [AppComponent]
})
export class AppModule {
}

But I run into the following error:
vendor.bundle.js:32040 Uncaught Error: Unexpected value 'NgxSmartModalModule' imported by the module 'AppModule'. Please add a @NgModule annotation.
    at syntaxError (vendor.bundle.js:17730)
    at vendor.bundle.js:31426
    at Array.forEach (<anonymous>)
    at<EMAIL_ADDRESS>(vendor.bundle.js:31409)
    at<EMAIL_ADDRESS>(vendor.bundle.js:42839)
    at<EMAIL_ADDRESS>(vendor.bundle.js:42812)
    at<EMAIL_ADDRESS>(vendor.bundle.js:42741)
    at<EMAIL_ADDRESS>(vendor.bundle.js:48427)
    at<EMAIL_ADDRESS>(vendor.bundle.js:48413)
    at Object.../../../../../src/main.ts (main.bundle.js:33682)

I found a little workaround: add the following to the package.json of the demo app:
...
"dependencies": {
    ...
    "ngx-smart-modal": "git+https://github.com/maximelafarie/ngx-smart-modal-refacto.git",
    ...
  },
...

It doesn't allow me to dev in local on the library but the demo's only here for "demo" purposes (GitHub Pages). So it works, but it could be nice to link the demo directly to the local library.

?

import {NgxSmartModalModule, NgxSmartModalService} from '../../../';

Maybe:
import { NgxSmartModalModule, NgxSmartModalService } from '../../../src/ngx-smart-modal';

Always this error:
Uncaught Error: Unexpected value 'NgxSmartModalModule' imported by the module 'AppModule'. Please add a @NgModule annotation.
    at syntaxError (vendor.bundle.js:24500)
    at vendor.bundle.js:38196
    at Array.forEach (<anonymous>)
    at<EMAIL_ADDRESS>(vendor.bundle.js:38179)
    at<EMAIL_ADDRESS>(vendor.bundle.js:49609)
    at<EMAIL_ADDRESS>(vendor.bundle.js:49582)
    at<EMAIL_ADDRESS>(vendor.bundle.js:49511)
    at<EMAIL_ADDRESS>(vendor.bundle.js:55197)
    at<EMAIL_ADDRESS>(vendor.bundle.js:55183)
    at Object.../../../../../src/main.ts (main.bundle.js:33685)

@maximelafarie That problem is not related to this starter. You are trying to import the modules of the library from a nested app, and they import two different version of Angular from different node_modules. I think you can't do it. Your demo should have the library installed.

Thank you @robisim74 ! 👌


Replace WMI-powered filesystem compression

closes HaloSPV3/HXE#253
Utilizes https://github.com/dotnet/pinvoke
TODO

[x] Test via CLI switch
[ ] hook into Installer.cs
[ ] handle Access/Security exceptions
[ ] handle "File/Directory in use" access exceptions

https://twitter.com/VoidsShadow/status/1513087907145269249
Directory handles can now be acquired properly.
Next step: handle "resource is in use by another process" exceptions

Suggested edit:
diff --git a/src/HXE.csproj b/src/HXE.csproj
index c66e9f5..4194a86 100644
--- a/src/HXE.csproj
+++ b/src/HXE.csproj
@@ -118,7 +118,7 @@
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
       <PrivateAssets>all</PrivateAssets>
     </PackageReference>
-    <PackageReference Include="Microsoft.Windows.CsWin32" Version="0.1.635-beta">
+    <PackageReference Include="Microsoft.Windows.CsWin32" Version="0.2.252-beta">
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
       <PrivateAssets>all</PrivateAssets>
     </PackageReference>


Table Display Issue

Hi. Great effort on the template, very clean. I've got this issue with tables. As shown in the image above, tables are not getting styled as they should. The markup is just a replica of what is in the template table file. What am I missing?
Thanks.

I would like to try and help on this one!
I think the table is missing the id and js to set the table info;
<table id="example1"
And corresponding js:
$(function () {     $('#example1').DataTable();     $('#example2').DataTable({         "paging": true,         "lengthChange": false,         "searching": false,         "ordering": true,         "info": true,         "autoWidth": false     }); });

Yep looks like JS is not right

I finally got to resolve it. The table looked messed up because of some tags missing in the HTML. It's funny because it displays well in the template and messed up in my app. The missing tags are <thead> and <tbody>. After adding the tags, the table displayed as it should.


[AMD] WMMA dot operand conversion to Linear Layout

This PR implements conversion of WMMA dot operand layout to linear layout and adds related tests.

Hi @binarman , let's have a chat about this on Monday :)

@lezcano
I plan to continue discussion with you next week. After we come to an agreement regarding order, I will rework all AMD converters in one style.

Yeah let's follow up on that later.


ref_fine_mels and coarse

Are they same？

During training, they are not the same (ref-coarse-mel is just the target speech, while ref-fine-mel is another random speech from the speaker);  during inference, they are the same one.
In my opinion, this strategy can make sure that the content information will not leakage through the fine-grained encoder during training, and solve the train-inference mismatch problem.
You can refer the original paper for more details: https://arxiv.org/abs/2005.08484

Thanks you.

During training, they are not the same (ref-coarse-mel is just the target speech, while ref-fine-mel is another random speech from the speaker); during inference, they are the same one.
In my opinion, this strategy can make sure that the content information will not leakage through the fine-grained encoder during training, and solve the train-inference mismatch problem.
You can refer the original paper for more details: https://arxiv.org/abs/2005.08484

Can we use the random slice mel as fine mel just like your CDFSE?


difference between istioctl manifest generate and istioctl install in mutatingwebhook

Bug Description
If a tagged mutatingwebook (e.g. istio-revision-tag-default) exists istioctl install/upgrade and istio manifest generate different manifests.
install/upgrade first applies the same manifests as generate, but then changes the selectors to this:
  namespaceSelector:
    matchLabels:
      istio.io/deactivated: never-match
  objectSelector:
    matchLabels:
      istio.io/deactivated: never-match

While istio manifest generate generates the normal selectors
  namespaceSelector:
    matchExpressions:
    - key: istio.io/rev
      operator: In
      values:
      - default
    - key: istio-injection
      operator: DoesNotExist
  objectSelector:
    matchExpressions:
    - key: sidecar.istio.io/inject
      operator: NotIn
      values:
      - "false"

This only happens if a tagged mutating hook already exists, the code that does it is here: https://github.com/istio/istio/blob/master/istioctl/pkg/tag/util.go#L154
Imo the generated manifests should be the same as applied in install/upgrade but it also creates problems when you deploy istio with install/upgrade and then apply generated manifests on top to e.g. label and properly prune stuff. This then fails due to a difference in manifests:
 Error: failed to install manifests: errors occurred during operation: creating default tag would conflict:
Jan 13 13:25:06 Error [IST0139] (MutatingWebhookConfiguration istio-sidecar-injector) Webhook overlaps with others: [istio-revision-tag-default/namespace.sidecar-injector.istio.io]. This may cause injection to occur twice.

Version
client version: 1.16.1
control plane version: 1.16.1
data plane version: 1.16.1 (19 proxies)

Additional Information
No response

hey @juliantaylor, could you provide some details about what settings you installed and upgraded your Istio?
I am trying to reproduce this problem.

This is because manifest generate may not have the in-cluster environment, and therefore, the installed webhook cannot be processed after the installation.

Maybe we can improve --cluster-specific to detect webhook and revise the related webhook config.


Fix query tracking across multiple command buffers on the same queue.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

LGTM :+1:

cherry-picked and pushed to master


Log correlation

What does this PR do?
Adds log correlation by patching console.log in node. Adds DD_LOGS_INJECTION environment variable support.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@dde7973). Click here to learn what that means.
The diff coverage is 94.11%.

@@            Coverage Diff            @@
##             master      #41   +/-   ##
=========================================
  Coverage          ?   91.77%
=========================================
  Files             ?       24
  Lines             ?      681
  Branches          ?      108
=========================================
  Hits              ?      625
  Misses            ?       35
  Partials          ?       21

Impacted Files
Coverage Δ

src/trace/patch-http.ts
91.66% <ø> (ø)

src/index.ts
88.73% <100%> (ø)

src/trace/tracer-wrapper.ts
93.1% <100%> (ø)

src/utils/log.ts
69.56% <100%> (ø)

src/trace/patch-console.ts
100% <100%> (ø)

src/trace/listener.ts
92% <100%> (ø)

src/trace/trace-context-service.ts
75.86% <42.85%> (ø)

Continue to review full report at Codecov.

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