Enable contenthash in filename or chunk file name for web component async build

What problem does this feature solve?
It could help solving browser cache issue which sometimes doesn't give up cache.
https://webpack.js.org/guides/caching/
What does the proposed API look like?
in vue.config.js (or some other place to put configuration)
configureWebpack: {
    mode: 'production',
    plugins: [new CompressionPlugin(), new webpack.EnvironmentPlugin(['NODE_ENV', 'DEBUG', 'API_TOKEN'])],
    output: {
      filename: '[name].[contenthash].js',
      chunkFilename: '[name].[contenthash].js'
    }
  }

command: vue-cli-service build --target wc-async --name foo ./src/foo.vue
Expected:
The chunk filename generate by the build should contain chunkhash for example:

dist/foo.c0b14a2b59b6a2991383.min.js
dist/foo.1.c0b14a2b59b6a2991383.min.js

Link issue: https://github.com/vuejs/vue-cli/issues/5434

构建目标为库的都具有这个问题吧，有方案解决吗？

@yyx990803 any idea on this? It's been an issue for a while and all the sites I'm finding seen to link back to each other.
Sorry for the ping but I couldn't find the right team to ping. ❤

Hi! Curious if there has been any movement on this request? Any workaround?
Not having the ability to have hashes causes issues with releases and browsers caching JS files. This results in users having to manually clear their cache....

So, not sure if this is much of a viable solution. I hacked together a simple script that can be ran as an npm postbuild script after the async web component files are generated...
The script simply generates a hash/random id which is used to append to the file each .js file name. Then it searches the entry point script file for references chunked names and adds the has into the entry point file...
Obviously this will force a users browser to download a file anytime there is a new build. Not ideal...
const fs = require('fs');
const glob = require('glob');
const {nanoid} = require('nanoid');
const replace = require('replace-in-file');

const searchDirectory = './dist/';
const searchPrefix = 'foo-test' // wc-async name
const hash = nanoid();

glob(searchDirectory + searchPrefix + '?(.min).js', null, (err, files) => {
    if(err) {
        return console.error(err);
    } else  {
        replace({
            files,
            from: new RegExp(searchPrefix + '[.]', 'g'),
            to: searchPrefix + '.' + hash + '.'
        }, (err, results) => {
            if (err) {
              return console.error(err);
            } else {
                glob(searchDirectory + searchPrefix + '.[0-9].*js', null, (err, files) => {
                    if(err) {
                        return console.error(err);
                    } else {
                        files.forEach((file) => {
                            const newFileName = searchPrefix + '.' + hash + file.substring(searchDirectory.length + searchPrefix.length);

                            fs.rename(file, searchDirectory + newFileName, function(err) {
                                if (err) console.log('ERROR: ' + err);
                            });
                        });
                    }
                });
            }
        });
    }
});

@LinusBorg @OmgImAlexis @princesspoopan

A better workaround,  that does not mess with the file content as above, is to execute the following node script before the actual build
const fs = require("fs");
const glob = require("glob");
const replace = require("replace-in-file");
const resolveWcConfig =
<EMAIL_ADDRESS>
glob(resolveWcConfig, null, (err, files) => {
  if (err) {
    return console.error(err);
  } else {
    replace(
      {
        files,
        from: 'chunkFilename: `${libName}.[name]${minify ? `.min` : ``}.js`,',
        to: 'chunkFilename: `${libName}.[contenthash]${minify ? `.min` : ``}.js`,',
      },
      (err, results) => {
        if (err) {
          return console.error(err);
        }
      }
    );
  }
});


bau: Epdq is deprecated

Not only is epdq deprecated, these tests were never turned into canaries.

You'll need to remove the lines from webpack.config.js (and the commented-out bit of run-local/index.js) (oh and the epdq section of the README!).

I think we can remove the EPDQ section of the README now too?

I think we can remove the EPDQ section of the README now too?

Whoops, removed


Hardcoded "appInsights" prevents multiple JS clients

I have tried to create 2 telemetry clients for JS to send server data to one AI account and custom metrics to another (shared) AI account. I can't get it to work if I call it anything other than window.appInsights.
e.g. If I set window.appInsights to the page view one, that works but if I have another window.sharedAppInsights with a different key, it doesn't send anything (and doesn't error).
The culprit seems to be the hard-coded "appInsights" in Init.ts which is the only name that will cause initialisation to be carried out.
Ideally, the JS for app insights should not assume the name of a global variable set in user code. At very least, it should permit the name of the global to be specified in config when creating the client so that more than one client can be used.
I couldn't find another way to achieve this other than using one setup during page load and then overwriting window.appInsights in the method that sends custom events to use the second client.

similar to https://github.com/Microsoft/ApplicationInsights-JS/issues/576
We have multi-client support on our roadmap. Would something similar to our node.js SDK work for you? In node.js you can create multiple instances of new appInsights.TelemetryClient().

OK, I'll close as duplicate. Yes, it would be useful to be completely OO so that we can either create new TelemetryClient() or have a factory method that injects all config and does not call back onto global variables, so that it can be called multiple times.


支持自定义StarRocksGenericRowTransformer 清洗数据

What type of PR is this：

[ ] Feature

Which issues of this PR fixes ：
https://github.com/StarRocks/starrocks-connector-for-apache-flink/issues/323
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)
[ ] I have added documentation for my new feature or new function

 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.renzhimin7 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


How to suppress warning about Google Cloud SDK credentials?

I'm wondering how to suppress the warning:
WARNING: Your application has authenticated using end user credentials from Google Cloud SDK. We recommend that most server applications use service accounts instead. If your application continues to use end user credentials from Cloud SDK, you might receive a "quota exceeded" or "API not enabled" error. For more information about service accounts, see https://cloud.google.com/docs/authentication/.

Our application is a desktop application that uses the users default credentials to access files on the cloud and this warning is confusing to users.  Is there a way to suppress it?  I thought I'd found one but I accidentally asked in the php version of the auth library and not the java one.
Would it be possible to port the patch that they implemented to suppress the warning when the environment variable SUPPRESS_GCLOUD_CREDS_WARNING is set to true?
https://github.com/googleapis/google-auth-library-php/pull/199#issuecomment-411490419

There is currently not a way to suppress the warning. PHP has a unique situation where triggered warnings can make tests fail so we provided a mechanism to suppress the warning. We really advise using service accounts if it's a production application.
That said, we can take a look.

It's an application that users tend to run on their own laptops on their own files.  Getting them to set up a service account is a difficult undertaking.  We use service accounts in our large scale production setting with the same tools, but the personal account use is an important one.


⚠ Bofrost.lu has degraded performance

In 1eb8400, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 957 ms

Resolved: Bofrost.lu performance has improved in 66956b6 after 6 minutes.


As a designer I need default spacing and placement rules.

Need to document distance between components.
Guidelines where component should be placed.
In Figma: In Example project adda new File in Example Project named as Patterns (with multiple pages)
Should buttons be placed to left or right (guideline not an absolute rule)
Align to right in dialog boxes
Align to left in Pages
For Compact: Include spacing rules between components. This US does not cover making the individual  components compact.
Add documentation to StoreFront if needed.

@lucasveil
I started laying out a template for us so that we keep the page clean and clear. If you check "Bendik test" under 'Patterns' you will find some examples used in a different project. We can probably use some of that as inspiration.

You may have looked at it already, but can you approve or decline Spacings and Placement and order? @lucasveil


Allow aborted requested to be handled by custom policy

Allow customErrorHandler on aborted call
Allow retry on aborted call
Fix: Avoid duplicate call to _respond when the xhr was manually aborted (ie: when timeout is specified)

More informations, the double this._respond should also be fixed by this PR:
https://github.com/Microsoft/SimpleRestClients/blob/master/src/SimpleWebRequest.ts#L562
by
https://github.com/SeriousJul/SimpleRestClients/blob/master/src/SimpleWebRequest.ts#L207

The basic logic looks good, but you need to alter the DefaultErrorHandler to have the same behavior as before -- canceled = immediate reject, statuscode 0 = immediate reject.


Releasing and packaging the library

To enable easier reuse of the library, please consider releasing (tagging) and packaging it for major Coq versions in the Coq opam archive and elsewhere. Reuse potential would be even greater if #2 is taken into account also.

We plan to have an opam package available for 8.18 once the metacoq and the undecidability library opam packages are availble.

(And in that turn will also release packages for 8.16 and 8.17.)

Done: https://github.com/coq/opam/pull/2812 (v1.1+8.17 and v1.1.1+8.18)

Also done: https://github.com/coq/opam/pull/2825 (v1.0+8.17 and v1.0+8.18)

Thanks!


Getting unauthorized / not authenticated errors

from pfaw import Fortnite,Platform
fortnite = Fortnite(fortnite_token='my_game_token', launcher_token='my_launcher_token', password='password', email='email')

player = fortnite.player(username='Ninja')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/pfaw/core.py", line 28, in player
    response = self.session.get(constants.player.format(username))
  File "/usr/local/lib/python2.7/dist-packages/pfaw/core.py", line 78, in get
    response.raise_for_status()
  File "/usr/local/lib/python2.7/dist-packages/requests/models.py", line 935, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://persona-public-service-prod06.ol.epicgames.com/persona/api/public/account/lookup?q=Ninja

I converted the code from Python 3.X to Python 2.7 to work with a bot I've already wrote.
These calls were working the other day outside of the bot but now they don't. I've done a pip install fortnite -U to update and then a 3to2 after as well.

I just tested the player method and it's working for me. Maybe something is wrong with your tokens. Did you change your Epic Games password? Feel free to join the Discord for better assistance on this.

Worked it out over Discord.
Thanks again!


Investigate auth0 setup

Need to clear up the questions from #366:

[x] Does it allow us to get authenticate a user? Yes
[x] Does it let the user authorize actions on their behalf (OAuth)? Yes, with a custom proxy that

TBD:

[ ] What reusable client-side packages are there?
[ ] How do different auth providers differ from each other, and how does it impact the client-side API?
[ ] Can we used the popup-based auth currently used internally? Requires:

[ ] Custom html response
[ ] Custom nonce-cookie handling
[ ] Custom oauth state forwarding
[ ] Redirect-url generation for redirect response

Settled for passport for now.

@Rugvip is there an example repo that's using Auth0 provider?

@taras not that I'm aware of. Best bet would be to ask on Discord and perhaps see if any of the authors of the provider are around


Audit log does not list deleted roles when role list of a user is updated

doPostUpdateRoleListOfUser() & onPostUpdateUserListOfRole of UserMgtAuditLogger [1] does not log the deleted roles/users.
Hence, if we go and remove a role for a user from the mgt console, the audit log is written as below.

INFO {AUDIT_LOG}-  Initiator<EMAIL_ADDRESS>| Action : Update Roles of User | Target : admin | Data : { Roles : [] } | Result : Success

[1] https://github.com/wso2/carbon-identity-framework/blob/master/components/user-mgt/org.wso2.carbon.user.mgt/src/main/java/org/wso2/carbon/user/mgt/listeners/UserMgtAuditLogger.java#L156

Still, this will not write logs related to internal roles due to https://github.com/wso2/product-is/issues/2249


Dev mobile work about section

My Port Folio: Mobile Version- Work and about section

Thanks cynthiainga for the honest review. Highly appreciated.

Thanks KayLemba for the optional review.


Games Size Calculator: Error due to path length

Games Size Calculator shows this error every time it is run. Most likely because the path is longer than the default Windows path limit - which can be disabled in Windows 10 and newer.

This used to be Games Size Calculator functionality, but darklinkpower has since folded it into Playnite 10. Games Size Calculator now only fetches install sizes online from Steam and GOG for games that aren't installed.


Python GAPIC: Logging v2

This pull request was generated by artman. Please review it thoroughly before merging.

Close this PR, which is a testing.


5375 bugfix embedded entity edit

Description
Enabling the possibility to add new nested entities that are within embedded entities.
closes #5375
https://github.com/inmanta/web-console/assets/44098050/d8f2015e-be7e-4737-9a5c-c661f8f07e76

Processing this pull request

Merged into branches master, iso7 in 3538a1c9f4109b4db9a1bcc14139254cc46dc82a


Missing instructions on how to find the Service Principal Id

The "Create an Azure Resource Manager service connection with an existing service principal" section mentions the "Service Principal Id" as a required field when manually setting up a service connection.
No instructions on how to obtain this value are available in this document, or anywhere else on the docs site for that matter.
**Please provide clear instructions on how to obtain the "Service Principal Id" for an existing Service Principal. **

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

ID: 26477658-d1ad-a4c9-0d9e-59aeead8a71e
Version Independent ID: 45dfbd24-b9f4-fdf2-48a2-5648220204fe
Content: Connect to Microsoft Azure - Azure Pipelines
Content Source: docs/pipelines/library/connect-to-azure.md
Product: devops
Technology: devops-cicd-library
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

@urig -- Uri, thank you for your feedback. Here are a couple of options where you might consider looking:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

@RoopeshNair -- Roopesh, please look into this issue.

Thank you @WilliamAntonRohm .
I was able to find my Service Principal Id and Service Principal Key using both the Azure Portal and Azure PowerShell like so:
a. Ran *PowerShell* (v5.1) "as Administrator".

b. `Install-Module -Name AzureAD`

c. `Connect-AzureAD -TenantId <tenant-id-from-the-app-registration-overview>`

b. `Get-AzureADServicePrincipalKeyCredential -ObjectId <object-id-from-the-managed-application-overview>`

The essence of what I'm requesting in this issue is to add detailed explanations of:

What Service Principals are.
What their relations are to the terms App Registration, Managed Application and Enterprise Application.
How to locate a Service Principal Id.
What a Service Principal Key is and how to locate it.

Thank you,
urig


[Event Request] codeunit 7153 "Item Analysis Management" - OnBeforeDimCodeToOption

We would like to add a new event OnBeforeDimCodeToOption in the following procedure:
local procedure DimCodeToOption(DimCode: Text[30]; ItemAnalysisView: Record "Item Analysis View"): Integer
var
    Location: Record Location;
    Item: Record Item;
    OptionValue: Integer;
begin
    //A200F0010.Event.ns
    OnBeforeDimCodeToOption(DimCode, ItemAnalysisView, OptionValue);
    if OptionValue > 0 then
        exit(OptionValue);
    //A200F0010.Event.ne
    case DimCode of
        Item.TableCaption:
            exit(0);
        Text000:
            exit(1);
        Location.TableCaption:
            exit(2);
        ItemAnalysisView."Dimension 1 Code":
            exit(3);
        ItemAnalysisView."Dimension 2 Code":
            exit(4);
        ItemAnalysisView."Dimension 3 Code":
            exit(5);
        else
            exit(-1);
    end;

Event Signature
[IntegrationEvent(false, false)]
local procedure OnBeforeDimCodeToOption(DimCode: Text[30]; ItemAnalysisView: Record "Item Analysis View"; var OptionValue: Integer)
begin
    //A200F0010.Event
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Fix: parse Service options from 'npv.service.options'

Hi,
with this proto:
service Echo {
    option (meta).description = "Echo webservice";
    option (meta).basePath = "/echo";

    rpc ....
}

Service options was not parsed correctly because options are stored in npv.service.options. (i don't know why)
This PR parse npv.service.options if defined.
Thanks

Hello,
Thanks for the PR. Can you add a test please. Either add a couple of options to the existing proto and modify the expected one; or add a simple sample proto file with option and add a test for it similar to the existing ones.
Thanks.

Sorry for missing tests, should be ok :)


[API Feature Request] Windows and/or Linux support

It would be really awesome to get this app available in Windows and/or Linux.

Closing this here since it's not directly related to the API and extensions. (Windows/Linux support might be coming in the future but there's no timeline yet.)

Even if the code isn't open source you could at least open a project for the core app so there could be a public issue tracker...


Area3D sometimes fails to detect ConcavePolygonShape3D

I have a character body 3d attached with a area 3d, and there is a static body 3d, with concave collision shape.
The area 3d sometimes cant detect it collide with the static body, as shown below:
Cant detect:

Can detect:

And I found that, if I remove below code in my character body 3d, it can always detect the static body.
	if not is_on_floor():
		velocity.y -= gravity * delta

I also found if I change to convex collision shape, it can always detect the static body too.
Thanks
Sample Project (without the addon, as it is too big):
concave.zip

For whatever reason it seems every triangle collision gets discard as being backfacing, which is strange.
This does mean that you can work around this by enabling the "Backface Collision" property on the ConcavePolygonShape3D, like so:

The CharacterBody3D and gravity was inconsequential. The important part seemed to be the Y-position of the Area3D, so I reduced the repro further: BoxSensorVersusMesh.zip

I've reported this upstream, as I'm pretty sure this is a bug in Jolt itself: jrouwe/JoltPhysics#1138

Thanks for your prompt actions!
Hope it will be fixed soon.
And in the mean time will try to enable the "Backface Collision" property.
Thanks!

It turned out that this wasn't really a bug in Jolt after all, but rather a mismatch of expectations.
This should be fixed by #899, which essentially forces Area3D to treat every shape as if "Backface Collision" was enabled.

0.13.0-stable has just been released, which includes the fix for this.
Note that the release in the Godot Asset Library is still pending approval as of typing this, so you might need to download it from the link above.


Implement #159 -- Semantic validation of min_transfer_time field from transfers.txt

Summary:
This PR provides support to semantically validate the value of field min_transfer_time.
Expected behavior:
At this step of valdiation it is assumed that non null values of min_transfer_time are positive integers. To reach feature parity with the existing Python validator, we implement a rule checking that said field is within a certain range.
This PR makes the range use configurable, so that any user can specify the min and max values acceptable for min_transfer_time.
If these threshold are not specified default values will be used:

min range: 0 sec
max range : 86400 sec (24 h)

Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Run the unit tests with gradle test to make sure you didn't break anything
[x] Format the title like "Fix #<issue_number> - " (for example - "Fix #1111 - Check for null value before using field")
[x] Linked all relevant issues
~[ ] Include screenshot(s) showing how this pull request works and fixes the issue(s)~

Reply to https://github.com/MobilityData/gtfs-validator/pull/297#pullrequestreview-449954519:
As discussed, the schema will be updated to avoid confusion regarding range validation. Issue and PR to be opened. Will update this comment once they exist.

Per discussion, semantic validation of numeric fields will not be implemented yet.


Dataset

Hi，I have read your article, the method proposed is very good, but there are a lot of problems when running the code, I hope you can help me, I have got access to the ADNI database, but I don't know how to download  the required data set, can you share the data set you use with me?

Ok,Thank you for your repley.I will use the pytorch version.If I have a problem that I don’t understand, I hope you can help me with it.

Sure.

Hi! I'm back again. In the paper you mentioned random forests to synthesize images. I have also seen this paper using random forest composite images. Do you have its source code? If so, can you share it with me at<EMAIL_ADDRESS>Thank you!

you should ask the first author of that paper for the code. I cannot share other guys' code without their permission.

Hi, I have applied for and obtained access to ADNI data set. I don't know how to get the two datasets used in the paper to evaluate the model. Could you please send me a link，thanks！

Or could you share the training data set? This is my email<EMAIL_ADDRESS>

Hi @lyc-lang the dataset used in the paper is private containing medical information and as such we cannot share it publicly.


GEODE-6593: fix test elapsed time computation

The test was using System.currentTimeMillis to verify that
the product has blocked for as long as expected.
But the product is using System.nanoTime.
This can cause race conditions because these two different
clocks can be updated at different times.
The test now also uses System.nanoTime.
Also removed some dead test code.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

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

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

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

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Thanks for the feedback. I have added time unit suffixes and 100ms of wiggle room

Using TimeUnit constants avoids confusion in math too. https://docs.oracle.com/javase/8/docs/api/java/util/concurrent/TimeUnit.html


No ability to prevent server.env from getting copied from src/main/liberty/config

Say I have src/main/liberty/config/server.env in my project and want to use this value in dev-time usages BUT then eliminate it in another test or production deployment (maybe it has secret or inappropriate values).
I don't see a way to use profiles or any properties to configure such a build to NOT copy server.env.    I think this is because the server.env path is defaulted in code when there is a null value for the serverEnvFile parm.
If instead, what if the plugin defaulted the plugin mojo serverEnvFile parm to the  src/main/liberty/config/server.env value, and then simply did nothing if this parm was unset or set to empty/null/whatever?
That all said, there's an easy workaround in newer releases, which is to use:
-Dliberty.env.dummyVar=dummyVal
though it looks like a hack.
I tried this on the v3.0.1 plugin, BTW.
Could this be done without breaking compat?

We used to have defaults for the file parameters (serverEnvFile, bootstrapPropertiesFile, etc). That was changed to not have defaults, and instead have a default for configDir (which is src/main/liberty/config). If the server.env file is located in that folder, it will always get copied unless serverEnvFile is specifically set. This is intended behavior. But even when we had defaults for the file parameters, it still would have copied that file over.
I think your best option is to move the file out of that folder (you can explicitly set serverEnvFile to point to wherever it is located, and then unset it when you no longer want to use it). You could use profiles to determine whether or not serverEnvFile is set.

Cheryl, thx for explaining more.
I was trying to not have to invent a special pattern for the base case, where you do want server.env copied, but only add the extra config when you don't.  Maybe a better solution then for me would be to add a blank empty.env

Closing since no change is planned.


BUG Silverstripe SEO - namespace change

Update the Vulcan namespace to QuinnInteractive in the following extensions in base-site-config.yml

Vulcan\Seo\Extensions\PageHealthExtension
Vulcan\Seo\Extensions\PageSeoExtension

Update requirements to "quinninteractive/silverstripe-seo": "^1.0.8"

This will need to wait for another tagged release as 1.0.8 template path has not yet been updated with the new namespace

closed via #16


Homework 6

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Complete the last homework
What did you learn from the changes you have made?
How Ethics, Big Data, Data Science and the use of information responsibly impacts everyone around us. Also, how to use data fairly, without biases.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
N/A
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
N/A
How were these changes tested?
N/A
A reference to a related issue in your repository (if applicable)
N/A
Checklist

[X] I can confirm that my changes are working as intended

uoft-dsi-c3-homework_6-20240524
uoft-dsi-c3-sql_homework_6-20240524


added eleventy-plugin-reader-bar to plugin list

Added eleventy-plugin-reader-bar to plugin list.

Thank you!


feat: return event id of captured event

We get an event id from sentry after getting captured
This PR makes it available to use by returning it from the parent function

Looks fine, but is there any way to ensure it's working? I am not sure all the return types are, and I can see you are doing some pointer dereferencing in some places.


Drop win-32 platform CI from appveyor matrix?

Conda-forge has dropped support for 32-bit windows builds (https://github.com/conda-forge/cftime-feedstock/issues/2#issuecomment-385485144). Do we want to continue testing against this environment? The point becomes moot after #1876 gets wrapped up in ~7 months.
xref: https://github.com/pydata/xarray/pull/1252

there's something wrong with the bot, this is not stale but closed

Yup, I just reported. It does act up occasionally!
Though it is nice to not have to go through the 550 issue backlog, and mostly rely on it to notify & clean the dead ones...


Add missing interrupt definitions so that generated pac's can be used in an RTIC application

The intention is to regenerate stm32-metapac crate,  to include interrupt definitions, similar to those in peripheral access crates that are based on svd2rust, so that the embassy-stm32 HAL can be used in an RTIC application.
With current functionality, it is possible to do this in an RTIC app - thanks to @korken89 and @MathiasKoch for the pointers :
pub mod pac {
    pub const NVIC_PRIO_BITS: u8 = 2;
    pub use cortex_m_rt::interrupt;
    pub use embassy_stm32::pac::Interrupt as interrupt;
    pub use embassy_stm32::pac::*;
}

#[app(device = crate::pac, peripherals = false)]

The intention would be to reduce it to:
#[app(device = embassy_stm32, peripherals = false)]

WIP plan of attack:
[x] Expose interrupt attribute macro, Interrupt enum. (This was the 'easy' part, and I am using it to validate that all the [patch.crates-io] paths work in my tests.). Using this PR to update the stm32-metapac, this reduces the required code from above to:
  pub mod pac {
      pub const NVIC_PRIO_BITS: u8 = 2;
      // pub use cortex_m_rt::interrupt;
      // pub use embassy_stm32::pac::Interrupt as interrupt;
      pub use embassy_stm32::pac::*;
  }

  #[app(device = crate::pac, peripherals = false)]

[x] Expose NVIC_PRIO_BITS in chiptool
[x] Expose NVIC_PRIO_BITS in stm32-metapac Unmerged PR
[x] Test with new generated stm32-metapac. Can successfully build and run an RTIC v2 app, using the following:
#[app(device = embassy_stm32::pac, peripherals = false, dispatchers = [FDCAN1_IT0, FDCAN2_IT0])]

TBD : @Dirbaio , and @korken89, the device = <crate> still requires appending the ::pac qualifier. Is it necessary to modify the embassy_stm32 crate to eliminate this, or is this an acceptable user experience?

@Dirbaio This turned out to be easier than I thought ... mostly thanks to your tips. If you are OK with this, then I will update the stm32-data PR when this is merged.

the device = <crate> still requires appending the ::pac qualifier. Is it necessary to modify the embassy_stm32 crate to eliminate this, or is this an acceptable user experience?

This is intended, the device attr needs the PAC, not the HAL. You can do device = "embassy_stm32::pac", or device = "stm32_metapac". The former is better though because you automatically get the same PAC version.


Fix Intel test runner

Pin runner to ubuntu-18.04

Codecov Report

Merging #16 (334d6c8) into master (fee5d7a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   71.89%   71.89%
=======================================
  Files          10       10
  Lines        1327     1327
=======================================
  Hits          954      954
  Misses        373      373

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 fee5d7a...334d6c8. Read the comment docs.


Look into terminal email client

want to fool around with a terminal email client
https://aerc-mail.org/
Other clients from article
https://www.tecmint.com/best-commandline-email-clients-for-linux/#:~:text=In this review, we shall exclusively dive into looking at#:~:text=In this review, we shall exclusively dive into looking at

would be cool, will never do this for security reasons. Might host an SMTP server one day but that will probably be handled by a vm on my home lab.


Update compose matrix to include Compose versions

Proposed changes
Changes the docker compose matrix to also include the version of Docker compose where support for given file format was introduced. To avoid having to crawl true release notes every time you wonder about this for updating your minimum requirements and so on when bumping the file format version.

Now the only missing piece of info is what changed between the given smaller releases (3.0 => 3.1 => 3.2 => 3.3), this seems complex as some fetaturs seems ot be added afterwards to given versions.
E.g from recent 1.14.0:
## Compose file version 2.2:
- Added the following parameters in service definitions: cpu_count,
cpu_percent, cpus

@shin- PTAL

See previous efforts in this direction: https://github.com/docker/docker.github.io/pull/3580
The concern to me is that people will see it as a tacit endorsement of (sometimes dramatically) older versions of the software which we no longer support.

Thanks @shin- for finding the previous thread, and clarifying that here. Let's keep this out of the docs then, as we decided before. It's sending the wrong message both per the complexities of small releases that are skated over per the comments above, and for the larger issue re: unsupported older versions.

Ok, but re-phrasing what I said in the description: As software vendor, when I bump Compose file version and need to update minimum requirements, not having this is painful

But maybe @shin- is saying what I was saying in the side note above, given certain features of a compose file version is added (fixed, improved, forgotten) later, I can see why adding this can be largely misleading.
It's basically close to impossible unless you test your compose file defintion across all "supported" (by the given software vendor) docker compose versions.
So I'm fine with this being closed if that is the reason.
Question is if the list of Docker versions have the same flaw, aka are there changes/fixes/improvements in a versioned file format behavior in later versions?

@shin- and/or @dnephin  can you respond to @andrerom 's last question just above ^^ ?

I'm not sure I understand the question.
If there are bugs in a version of docker-compose or docker/cli, the fix could apply to all version of the Compose file supported by that tool, yes.

@dnephin I was referring to these kind of changes to docker compose format support as found in docker compose v1.4.0:
Compose file version 2.2

- Added the following parameters in service definitions: cpu_count,
cpu_percent, cpus

But I don't think I have seen similar changes in later releases in docker itself, so I guess that answers my question. (as in: no the same issue does not apply to docker versions listed here in the matrix)


Allow the "ban" button to be customizable

Describe your issue
As of yesterday, Twitch has made it so banning a user will strip them of their followage, because of this the ban button Chatterino has in the usercard is now a very risky button.
My immediate thought is we should just move this button to be a customizable button, it would allow for an extra button slot, and would probably be a good slot for the 2w button, since we removed the 2w button being default in a prior change.
[2021-06-17 21:24:40] #39daph 39daph: wait
[2021-06-17 21:24:44] #39daph 39daph: chatterino took away my 2w timeout button
[2021-06-17 21:24:46] #39daph 39daph: donkAngry

Screenshots
Example:

If the button is set to ban (probably still default) the box will be grayed out, choosing one of the other options [s/m/h/d/w] will allow for customization again.
OS and Chatterino Version
N/A

Update

That being said, this change implementation is probably not the worst idea either way.


T060101c: Getting to “Hello” - typo

In Section 6.1. there is a typo:
" This will be handy for workng on all ...."
"workng" should be "working"

Fixed.  Thanks for the catch, @band.


OpenPose editor  txt2img and img2img  error message

After adjusting the human skeleton in the OpenPose editor, clicking on "txt2img" to transfer the image to ControlNet and selecting "none" for preprocessing, the image cannot be generated and an error is reported in the console. However, adding the skeleton image manually from ControlNet allows for successful generation of images. It seems that there is a problem with the transfer of images from OpenPose to ControlNet, and no images can be generated thereafter. Disabling the ControlNet function does not solve the problem, and the only solution is to refresh the page with F5 to restore normal functionality. I can confirm that the ControlNet function is working properly and that both manual drawing and importing of skeletons can be used to complete the drawing. The problem only occurs when using the OpenPose editor's txt2img and img2img functions.
=============================================
Console error message：
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "E:\AI\novelai-webui\novelai-webui-aki-v2\py310\lib\site-packages\gradio\routes.py", line 337, in run_predict
output = await app.get_blocks().process_api(
File "E:\AI\novelai-webui\novelai-webui-aki-v2\py310\lib\site-packages\gradio\blocks.py", line 1013, in process_api
inputs = self.preprocess_data(fn_index, inputs, state)
File "E:\AI\novelai-webui\novelai-webui-aki-v2\py310\lib\site-packages\gradio\blocks.py", line 911, in preprocess_data
processed_input.append(block.preprocess(inputs[i]))
File "E:\AI\novelai-webui\novelai-webui-aki-v2\py310\lib\site-packages\gradio\components.py", line 1830, in preprocess
ff.run()
File "E:\AI\novelai-webui\novelai-webui-aki-v2\py310\lib\site-packages\ffmpy.py", line 98, in run
raise FFExecutableNotFoundError(
ffmpy.FFExecutableNotFoundError: Executable 'ffmpeg' not found
==========================================

I have the same problem

same problem
Traceback (most recent call last):
File "D:\程\stable diffusion\stable-diffusion-webui\venv\lib\site-packages\gradio\routes.py", line 337, in run_predict
output = await app.get_blocks().process_api(
File "D:\程\stable diffusion\stable-diffusion-webui\venv\lib\site-packages\gradio\blocks.py", line 1013, in process_api
inputs = self.preprocess_data(fn_index, inputs, state)
File "D:\程\stable diffusion\stable-diffusion-webui\venv\lib\site-packages\gradio\blocks.py", line 911, in preprocess_data
processed_input.append(block.preprocess(inputs[i]))
File "D:\程\stable diffusion\stable-diffusion-webui\venv\lib\site-packages\gradio\components.py", line 1830, in preprocess
ff.run()
File "D:\程\stable diffusion\stable-diffusion-webui\venv\lib\site-packages\ffmpy.py", line 98, in run
raise FFExecutableNotFoundError(
ffmpy.FFExecutableNotFoundError: Executable 'ffmpeg' not found

I have this problem too.

same problem

Same thing here.

same problem too

Me too

There's a pull request to fix this, hopefully it'll be merged soon.


Move errata tool template information out of code

This removes errata tool values from the source code and into an
ocp-build-data like datastore. In addition to moving everything to
a single source of truth, this will enable non-OpenShift teams to
use the errata tool functions of elliott in the future.

Depends on https://github.com/openshift/ocp-build-data/pull/25


Birthdays

Thank you for adding the birthdays. Looks great, except the month and day are missing the initial 0 (i.e., July 8, 2000 is shown as 2000-7-8 and not 2000-07-08). This would help for extracting date parts for queries - strings trim that looks for specific locations.
As a second unrelated issue that is easy is adding a space after the comma when pushing multiple responses  (i.e., emails and phone numbers). This would hopefully make them clickable in the future as links to directly send an email or call a number).

I was able to make it work using an if and slice js program to correct. All works now.


puzzled with the module naming mqtt-cs & mqtt-ds

would you please explain the difference bewteen mqtt-cs and mqtt-ds ? what cs and ds is  stand for?

The CS module is positioned as the protocol access layer, and the DS module is positioned at the service processing layer. The two are decoupled and designed to be independently deployed in the future.


[WIP] Bugfix: More consistent weight vector for ranking objectives.

Overview
This is a proposed bugfix based on PR https://github.com/dmlc/xgboost/pull/3379 originally submitted by @ngoyal2707, which intends to change how the instance weight vector is defined.
For a training dataset of K groups and N total training examples/instances, @ngoyal2707's original PR requires a weight vector of length K. That is, each group has its own weight, and the same weight is shared by all instances in the same group. While this scheme of specifying group weights works without issues in isolation, it however does conflicts with the computation of certain ranking metrics (e.g. auc and aucpr) in https://github.com/dmlc/xgboost/blob/master/src/metric/rank_metric.cc, which expects the weights to be populated for every training example.
Example
To demonstrate the bug, we can use the following example Python code:
import numpy as np
import xgboost as xgb

# some mock training data with 20 training examples and 10 features, divided into 4 groups
# each group has 5 training examples.
X = np.random.rand(20, 10)
y = np.array([1.0, 0.0, 0.0, 0.0, 0.0, 1.0, 0.0, 0.0, 0.0, 0.0, 1.0, 0.0, 0.0, 0.0, 0.0, 1.0, 0.0, 0.0, 0.0, 0.0])
weights = np.array([1.0, 2.0, 3.0, 4.0])

# construct DMatrix
group = np.array([5, 5, 5, 5], dtype=np.uint)
dmatrix = xgb.DMatrix(X, label=y, weight=weights)
dmatrix.set_group(group)

train_params = {
    "learning_rate": 0.1,
    "min_child_weight": 1,
    "tree_method": "exact",
    "objective": "rank:ndcg",
    "nthread": -1
}

xgb.train(train_params, dmatrix, 10, evals=[(dmatrix, 'train')])

The above code will run without issues. But if we add a ranking metric that also uses weights to the eval_metric parameter, like:
train_params = {
    "learning_rate": 0.1,
    "min_child_weight": 1,
    "tree_method": "exact",
    "objective": "rank:ndcg",
    "nthread": -1,
    "eval_metric": "auc"
}

xgb.train(train_params, dmatrix, 10, evals=[(dmatrix, 'train')])

The xgb.train function call will fail with the following exception (most likely due to the underlying C++ code accessing memory that is out-of-bound):

XGBoostError: b'[12:31:37] src/metric/rank_metric.cc:137: Check failed: !auc_error AUC: the dataset only contains pos or neg samples\n\nStack trace returned 7 entries:\n[bt] (0) 0   libxgboost.dylib                    0x0000001a12bff100 dmlc::StackTrace(unsigned long) + 464\n[bt] (1) 1   libxgboost.dylib                    0x0000001a12bfede4 dmlc::LogMessageFatal::~LogMessageFatal() + 52\n[bt] (2) 2   libxgboost.dylib                    0x0000001a12c6fea6 xgboost::metric::EvalAuc::Eval(std::__1::vector<float, std::__1::allocator > const&, xgboost::MetaInfo const&, bool) const + 1846\n[bt] (3) 3   libxgboost.dylib                    0x0000001a12bfbe1b xgboost::LearnerImpl::EvalOneIter(int, std::__1::vector<xgboost::DMatrix*, std::__1::allocatorxgboost::DMatrix* > const&, std::__1::vector<std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator >, std::__1::allocator<std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > > > const&) + 1275\n[bt] (4) 4   libxgboost.dylib                    0x0000001a12c1773d XGBoosterEvalOneIter + 717\n[bt] (5) 5   libffi.6.dylib                      0x000000010cdfb884 ffi_call_unix64 + 76\n[bt] (6) 6   ???                                 0x00007ffee4554950 0x0 +<PHONE_NUMBER>15312\n\n'

Proposed fix
To resolve the issue demonstrated above, this PR proposes to change the calculation of ranking objective in rank_obj.cc such that the utilization of weights is aligned with the implementation in rank_metric.cc.
Note that the alternative is to change how the weights are accessed in rank_metric.cc, such that when training a ranking model, the calculation of AUC and AUC-PR metrics assumes that the weights are provided on a per-group basis. However, using a per-group weighting scheme does reduce the overall flexibility of specifying weights.
ToDo
Update the documentation in https://xgboost.readthedocs.io/en/latest/python/python_api.html#xgboost.DMatrix.set_weight so that it reflects the changes.

@CodingCat @ngoyal2707 @hcho3

@xydrolase Any updates? How can I help?

@trivialfis I noticed lack of tests in ranking. And when I tried some weighted data, I am getting AUCPR > 1.0 :( Will need to look deeper...

It turns out that AUCPR is quite broken for learning-to-rank task (rank:pairwise, rank:ndcg etc). This is bad...

@trivialfis @RAMitchell I'm inclined to merge this pull request, and submit another pull request to fix the AUCPR metric. What do you think?

Don't merge this yet; #4436 should be merged first.


Hide Warning

Is there any way to hide or disable the warning that reads
Jan 25, 2021 8:56:09 PM com.google.cloud.translate.TranslateOptions <init> WARNING: Ignoring Application Default Credentials GOOGLE_APPLICATION_CREDENTIALS: using explicit setting for API key instead.

I don't believe so. Using Application Default Credentials is considered a Google Cloud best practice.
See the following:
https://cloud.google.com/docs/authentication
https://cloud.google.com/docs/authentication/production

I don't believe so. Using Application Default Credentials is considered a Google Cloud best practice.
See the following:
https://cloud.google.com/docs/authentication
https://cloud.google.com/docs/authentication/production

Additional thought: With Translation Advanced (v3), you can only use service accounts:
https://cloud.google.com/translate/docs/migrate-to-v3#service_accounts

Additional thought: With Translation Advanced (v3), you can only use service accounts:
https://cloud.google.com/translate/docs/migrate-to-v3#service_accounts


Support Multi-Architecture Docker Builds and Docker Image Push from Github

@chiefwigms I'll work with you on getting the docker hub stuff setup... or if you could get a DOCKER_USERNAME and DOCKER_PASSWORD configured in the Github repository environment variables that would do it.
Fixes https://github.com/chiefwigms/picobrew_pico/issues/221 and makes the device supported a bit clearer for new and existing users.

added env vars - should work 🤷♂


Migrate Postwoman to Nuxt.js (full Vue and SCSS support)

I have migrated Postwoman to Nuxt.js configured with static site mode.
In this PR, I've also updated the README to reflect these changes.
This PR solves https://github.com/liyasthomas/postwoman/issues/31.
I did not use https://github.com/chrisvfritz/vue-enterprise-boilerplate, as @diego81b proposed, because it would - as he said - need to be appropriately resized. Nuxt is, at least in my opinion, better in this respect as it is far more modular and allows us to just pick and choose modules we want.
Key changes are the styles.css file: => /assets/css/styles.scss (you'll also see fonts.scss that I created specifically for adding, importing and declaring font faces.)
All of the PWA code has been moved into a standalone file => /assets/js/pwa.js (which is used in /layouts/default.vue.)
The error page is now in /layouts/error.vue
The main application page is in /pages/index.vue - except for the header and footer, which is in /layouts/default.vue so it'll be added to every page.
Finally, the main application code has been moved from script.js to /pages/index.vue (I didn't really want to mess with the application JS itself all in one go.)

Quick footnote: Nuxt does have support for tailwind CSS but your CSS code is tidy and when it comes to design you really seem to know what you're doing! 😃
So I did not include the Tailwind module because I personally don't think it's necessary, however if you decide you would prefer to have the module, it's easy enough to add.

Hey @NBTX,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 6bc4c680-c61d-11e9-8aae-f1e5ed08699b

Hey @NBTX,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 6fbd29d0-c61d-11e9-8aae-f1e5ed08699b

Hey @NBTX,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: ffb4d370-c61e-11e9-8aae-f1e5ed08699b

Hey @NBTX,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 02dbdd00-c61f-11e9-8aae-f1e5ed08699b

Also, if you want to test this, you can just clone my fork and follow the instructions in the README.
Link for convenience: https://github.com/NBTX/postwoman-vue/

I hope this probably solves #37 #32 #31

Can confirm this fixes #37.

But this is something we need bad....

Hey @NBTX,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 38fc0220-c66f-11e9-be9f-2924d72427a9

Hey @NBTX,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 3d8dbf40-c66f-11e9-be9f-2924d72427a9

@NBTX lets do it. But i need your help on fixing #33 since @AndrewBastin said this will probably break his PR for "Code highlight on response body". Please check out his commit: https://github.com/AndrewBastin/postwoman/commit/73b5c6a54eda51bdf63bee85d3bb2416ed2f6a33 and consider an alternative for it.

@NBTX lets do it. But i need your help on fixing #33 since @AndrewBastin said this will probably break his PR for "Code highlight on response body". Please check out his commit: AndrewBastin@73b5c6a and consider an alternative for it.

Hey @liyasthomas, no probs, I will rework the commit for the new system, as I said, the current system is really hacky.... Not a big fan of it, it kinda comes really close to XSS exploitability than I like...

@NBTX hosting onto GitHub pages is broken/missing
Refer here: https://nuxtjs.org/faq/github-pages/ and fix hosting as ASAP

How about server build? I use travis ci.

I updated .travis.yml and build pushed to gh-pages but GitHub pages but deployment keeps on loading spinner. I'm not with my PC right now to check what went wrong. Please do check it and inform me.

Would it be worth considering reverting this PR and aae579c65073e4bf5a7ad5225f384af27cb2840b until Nuxt deploys have been fully figured out? Considering the app is down at the moment and all.

@larouxn yeah, I think so too... we should really have been working on this in a separate branch as this aint stable yet...
So, I can still work on improvements on the new system, while peeps can still use the app and these issues can be resolved

I think this revert PR (https://github.com/liyasthomas/postwoman/pull/47) should do the trick. 🤞

Confirmed. We're back up. 🚀

Reverted

@liyasthomas @NBTX can we get this merged into a separate branch titled something like 'migrate-nuxt' until we can figure out how to work out this stuff ?
Helps out a ton for working on features...
A small NOTE on Readme for new contributors would also be nice to let them know that we are migrating to Nuxt

@AndrewBastin Will have seperate branches until everything get fixed.

Another stuff to think about is the usage of the pwa plugin shipped with nuxt instead of the actual custom flow.
IMHO relying on localstorage is a little bit risky the same for installation (mobile browser already manage installarion prompt without any manual trick).
Nuxt behind the scene use workbox (if i remember well) that works well for many scenarios (updated version, caching resources, ecc...)
My 2 cents 😉

I've used workbox on most of my projects. Workbox integrates seamlessly with PWA, service workers and all other PWA technologies. If there's a PWA plugin exclusively for out of the box support, I would like to see it implemented.

https://github.com/liyasthomas/postwoman/tree/migrate-nuxt Let this PR relive on this branch. @NBTX @larouxn @AndrewBastin

@diego81b Yes Nuxt uses workbox internally.
@liyasthomas My bad, I didn't check it with gh-pages myself either. I'll work on fixing that and then should I open another PR?

Of course you should consider making another pulls. All of us would love to see this feature implemented 🔥✌

@liyasthomas I've fixed the Travis configuration. I think the problem may have been the missing router base path configuration. (It's automatically generated during npm run build now.)
This time I've tested it - and all appears good 😄
https://nbtx.github.io/postwoman-vue/

Refer to PR https://github.com/liyasthomas/postwoman/pull/49 for the changes.
I've also included instructions on how to set up Travis and GitHub pages in .travis.yml.

Thanks for this PR, this made it a lot easier for me to understand the flow of the app :)


board: neucleo_f429zi: fix wrong comment

The clock frequency is 168MHz on Nucleo board instead of 180MHz.
Signed-off-by: Jun Li<EMAIL_ADDRESS>

Codecov Report

Merging #14808 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #14808   +/-   ##
=======================================
  Coverage   52.51%   52.51%
=======================================
  Files         309      309
  Lines       45048    45048
  Branches    10419    10419
=======================================
  Hits        23656    23656
  Misses      16584    16584
  Partials     4808     4808

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 cb44b7e...32cb24e. Read the comment docs.

Just a typo in the commit tittle: "nu-e-cleo"

Done. Thank you!


Adding another option for startPlug or endPlug in my personal fork of leader-line

Hi, I have a site that uses leader-line. Can you help me understand how leader-line works? Can you send me the files and line numbers that I need to change in order to render a customized svg endPlug option?
Here is what I tried:
// Select my custom svg for the startPlug/endPlug: <g id="leader-line-plus"> ... </g>
// I made this myself. It is a circle with a plus sign inside.
// It has an ID that matches `leader-line` start and end plug svgs
plus = document.getElementById("leader-line-plus")

// I select the element that contains the leader-line start and end plug svgs
lldefs = document.getElementById("leader-line-defs")
lldef_tag = lldefs.firstElementChild.nextElementSibling

// Add my custom svg to the defs
lldef_tag.appendChild(plus)

But the problem is when I call
line = new LeaderLine(div2, div1, {endPlug: "plus"})
// Leader line does not recognize "plus"
// It switches to the default `endPlug` and doesn't use my custom end plug
// HELP: line.endPlug == "arrow1" but I want line.endPlug == "plus"

What can I do to fix this? Where does this happen in the code?

Hi @djpasseyjr, thank you for the comment.
I don't know whether you want to customize the library or change current SVG.
If you want to customize the library for the option, you have to edit symbols.html to recognize your new symbol (*Plug options) and compile the library with customized files.
Or, if you want to change current SVG only, you can edit the symbol that you will choose in svg#leader-line-defs > defs (e.g. #leader-line-arrow1).

I'm glad if I could help you. :smile:
Could you close this issue when it is solved?

It worked! Thanks for the help.
If I used:
<script src="path/to/leader-line.min.js"></script>

Then I could create my own svg with id="leader-line-arrow1" in my document and it was preferred to the arrow1 svg defined in the .min.js file. If I run into problems having multiple elements with the same ID I can always change the one in the min.js file.

:smile:


Only insert cache item into the database when there is no item already i...

I made a change to the cache db backend to only do an insert when it's needed.
When there is already an result and its not expired the insert is going to fail so it could be skipped.
Or am I missing something?

There are some failing tests with this change:
======================================================================
FAIL: test_add (cache.tests.DBCacheTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/tim/code/django/tests/cache/tests.py", line 248, in test_add
    self.assertFalse(result)
AssertionError: True is not false

======================================================================
FAIL: test_forever_timeout (cache.tests.DBCacheTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/tim/code/django/tests/cache/tests.py", line 500, in test_forever_timeout
    self.assertEqual(added, False)
AssertionError: True != False

You are right sorry.
This should fix it!

buildbot, add to whitelist.

I think the code could be written this way to avoid a race condition where the key is deleted after the "SELECT cache_key, expires..." query and before the "INSERT" query.


feature(spanner): low-cost instances

Support the provisioning of Spanner instances using processing_units
as an alternative to node_count. 1 node == 1,000 processing units.
Note: This commit is for a preview release. DO NOT MERGE AS IS.

This change is

@devbww is there anything else you need or are waiting for on this PR?

@devbww is there anything else you need or are waiting for on this PR?

No, nothing to do for now.  I can close.
The next step will be when the proto is released and we update our googleapis version.  Then this (or a new variant) will come along that we'll be able to merge for the actual release.


noaa_tides missing required dependency numpy

Home Assistant release with the issue:
.77.2

Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
Hassbian updated to .77.2

Component/platform:

noaa_tides
Description of problem:
I have just installed noaa tides, and I am receiving errors indicating that numpy doesn't exist.
It does, and I have tried uninstalling and re-installing pandas and numpy, to no avail.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

sensor:
  - platform: noaa_tides
    station_id: 8659665

Traceback (if applicable):
Error while setting up platform noaa_tides

Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=hass.loop)
  File "/usr/lib/python3.5/asyncio/tasks.py", line 400, in wait_for
    return fut.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/sensor/noaa_tides.py", line 57, in setup_platform
    noaa_sensor.update()
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/sensor/noaa_tides.py", line 118, in update
    from py_noaa import coops  # pylint: disable=import-error
  File "/srv/homeassistant/lib/python3.5/site-packages/py_noaa/__init__.py", line 1, in <module>
    import py_noaa.coops
  File "/srv/homeassistant/lib/python3.5/site-packages/py_noaa/coops.py", line 4, in <module>
    import pandas as pd
  File "/srv/homeassistant/lib/python3.5/site-packages/pandas/__init__.py", line 19, in <module>
    "Missing required dependencies {0}".format(missing_dependencies))
ImportError: Missing required dependencies ['numpy']

Additional information:
I am wondering if this is related to issue #16295 .

Try pip3 install py_noaa (in your venv if needed).

I did the install referenced, and I believe it is already installed according to the log for installation below:
(homeassistant) homeassistant@hassbian:/home/pi $ pip3 install py_noaa Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple Requirement already satisfied: py_noaa in /srv/homeassistant/lib/python3.5/site-packages (0.3.0) Requirement already satisfied: pandas in /srv/homeassistant/lib/python3.5/site-packages (from py_noaa) (0.23.4) Requirement already satisfied: requests in /srv/homeassistant/lib/python3.5/site-packages (from py_noaa) (2.19.1) Requirement already satisfied: numpy in /srv/homeassistant/lib/python3.5/site-packages (from py_noaa) (1.15.1) Requirement already satisfied: python-dateutil>=2.5.0 in /srv/homeassistant/lib/python3.5/site-packages (from pandas->py_noaa) (2.7.3) Requirement already satisfied: pytz>=2011k in /srv/homeassistant/lib/python3.5/site-packages (from pandas->py_noaa) (2018.5) Requirement already satisfied: certifi>=2017.4.17 in /srv/homeassistant/lib/python3.5/site-packages (from requests->py_noaa) (2018.8.24) Requirement already satisfied: idna<2.8,>=2.5 in /srv/homeassistant/lib/python3.5/site-packages (from requests->py_noaa) (2.7) Requirement already satisfied: urllib3<1.24,>=1.21.1 in /srv/homeassistant/lib/python3.5/site-packages (from requests->py_noaa) (1.23) Requirement already satisfied: chardet<3.1.0,>=3.0.2 in /srv/homeassistant/lib/python3.5/site-packages (from requests->py_noaa) (3.0.4) Requirement already satisfied: six>=1.5 in /srv/homeassistant/lib/python3.5/site-packages (from python-dateutil>=2.5.0->pandas->py_noaa) (1.11.0) (homeassistant) homeassistant@hassbian:/home/pi $

Same problem for me.

I think I have found a fix for this.. After a bit of searching , I found that there was a /usr/lib/python2.7/random.pyc and that it needed to be removed.
Once removed, the tide information began showing up in a bubble.
Regards

Apologies... I don't use Hassio, though if you ask some of the people that use hassio on here they may be able to tell you.
I use hassbian which does turn on ssh and that is how i did the surgery referenced above.

I am having the exact same issue as the OP.

I had to tweak the code a bit to show the real error:
Traceback (most recent call last):
File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform
SLOW_SETUP_MAX_WAIT, loop=hass.loop)
File "/usr/lib/python3.5/asyncio/tasks.py", line 400, in wait_for
return fut.result()
File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
raise self._exception
File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
result = self.fn(*self.args, **self.kwargs)
File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/sensor/noaa_tides.py", line 57, in setup_platform
noaa_sensor.update()
File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/sensor/noaa_tides.py", line 118, in update
from py_noaa import coops  # pylint: disable=import-error
File "/srv/homeassistant/lib/python3.5/site-packages/py_noaa/init.py", line 1, in
import py_noaa.coops
File "/srv/homeassistant/lib/python3.5/site-packages/py_noaa/coops.py", line 4, in
import pandas as pd
File "/srv/homeassistant/lib/python3.5/site-packages/pandas/init.py", line 11, in
import numpy
File "/srv/homeassistant/lib/python3.5/site-packages/numpy/init.py", line 142, in
from . import add_newdocs
File "/srv/homeassistant/lib/python3.5/site-packages/numpy/add_newdocs.py", line 13, in
from numpy.lib import add_newdoc
File "/srv/homeassistant/lib/python3.5/site-packages/numpy/lib/init.py", line 8, in
from .type_check import *
File "/srv/homeassistant/lib/python3.5/site-packages/numpy/lib/type_check.py", line 11, in
import numpy.core.numeric as _nx
File "/srv/homeassistant/lib/python3.5/site-packages/numpy/core/init.py", line 26, in
raise ImportError(msg)
ImportError:
Importing the multiarray numpy extension module failed.  Most
likely you are trying to import a failed build of numpy.
If you're working with a numpy git repo, try git clean -xdf (removes all
files not under version control).  Otherwise reinstall numpy.
Original error was: libf77blas.so.3: cannot open shared object file: No such file or directory
And the fix is:
sudo apt-get install libatlas-base-dev

OK.  Please close issue.

Yes, please close this. I created due to there being a different numpy. I removed numpy from standard python install and pip installed numpy into the virtual env.
Thanks


Apply code style

Right now code style is kind of hit and miss and inconsistent. Doesn't look like there were a ton of tools for this in C# besides having Visual Studio do it for you (surprisingly enough, given I figured it was easier to parse/lex/sema than C++), but the Roslyn .net core project apparently recently open source a tool that they're using to apply and enforce a code style, and it's probably worth looking into: https://github.com/dotnet/codeformatter

I don't have any experience with this tool in particular, but based on my general experience, I would caution against an automated tool to reformat code. These tools tend to be quite fragile, even when they're based on a robust code-AST API like roslyn.

Well, we're using clang-format on all the C++ code with good results - there again, built on a compiler infrastructure.


push_notifications: Send timestamp with only second granularity.

This reverts b8581e2895e242208beb3ac9fe0c2cf1de233ac3.  The mobile client on Android parses this field using:
                timeMs = data.require("time").parseLong("time") * 1000

This throws an error if value is not long (i.e. an integer), resulting in dropped notifications on Android from servers which had deployed b8581e2895e242208beb3ac9fe0c2cf1de233ac3.
Switch back to sending an integer, but keep the behaviour from fd6091ad17b5 where we send the timestamp in the payload of both Android and Apple push notifications.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

OK, merged, thanks @alexmv!

We ended up not backporting this before 8.0 was released, so removing that label.


Use test preconditions in :kotlin-dsl* tests

Instead of JUnit assume.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

There's merge conflict between your PR branch and base branch. Please fix it and try again.

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.


[BEAM-459] Add BigInteger to TypeDescriptors

Added BigInteger to TypeDescriptors class.

LGTM, thanks Jesse!


Add Telegram to Communities section of READMEs

Fixes #72

The Telegram group link is https://t.me/PepecoinGroup


Classes_devel

THe code is functioning as expected, tuning for the hardware still required but all compoments behave as expected. the code is been rewritten in a object oriented way to improve readability and maintainability

Sorry I selected the wrong upstream repo


[Feature] Improve Readability by Using Margin Shorthand Property

Problem
The current style.css code uses separate properties for margin-left margin-right , margin-top and margin-bottom when it could use the margin shorthand property to achieve the same effect. Using the shorthand property can make the code more concise and easier to read.
Description
For example, the following code:
#downloadbuttonholder {
height: 100px;
margin-left: auto;
margin-right: auto;
margin-top: 10px;
margin-bottom: 10px;
}

Could be rewritten using the margin shorthand property as:
#downloadbuttonholder {
height: 100px;
margin: 10px auto;
}

Implementation

[x] I would be interested in implementing this feature.

@harshhh11 straight-forward, but good point. Assigning.


Remove faraday-retry warning

This warning is printed every time dependabot is required.
To fix it, I think we can actually add the gem and make sure it's used by Octokit internally, instead of rolling our own implementation.

Yeah, in my opinion, this warning is fundamentally wrong because there's no way out of it if you don't want to use retries. And all warnings should be fixable.
Luckily we already use retries so I think it does make sense for us to use the extracted gem.


Optimize pass-docker the "dev" deployment

Currently, pass-docker (or pass-app if project is renamed) launches all services. For our dev.eclipse-pass.org site, we might not
necessarily want all this.  In this ticket we will isolate all the desired services that must be run and turn off any optional ones that are not needed (e.g. emails).
This will most likely result in the current DEV configuration being relabelled as full, so that we don't lose that for local development / testing purposes separate from our dev which is for dev.eclipse-pass.org.
Note that these names (dev and full) are working titles and might change during implementation.

@markpatton re-name ticket "Investigate DSpace deployment in pass-docker or manual testing"


How to re-scale Y Axis when (un)zooming?

Chart below looks flat because Y Axis is compressed.  Desired:  Y-Axis scaled to actual values +- 10% or something close.
Have seen this:  https://apexcharts.com/docs/zooming-in-category-x-axis/ and this:  https://apexcharts.com/docs/methods/#resetSeries but it does not have desired effect.

Hi,
Have you checked autoScaleYaxis

Thank you.


[DOCUMENTATION] Initial revisions for docgen

This PR should not yet be merged, it is still a WIP. We are making changes to improve docgen

Docstrings must have blank lines between the description and the parameter markup
Types, when possible, should be linked with class markup
Docstrings should no longer have lines that extend beyond 100 chars

Remaining concerns:

How to handle multiple types for a parameter or return value (e.g. twin operations, message operations)
Consistency of detail. Some docs are extremely in depth re: explaining IoTHub functionality, some assume IoTHub literacy. We should be consistent.
Validity of docs. Some of them need revision for accuracy of content.
Classmethod documentation is not yet being generated correctly wrt the cls arg

/azp run Azure.azure-iot-sdk-python-horton-e2e

    """Send a response to a method request via the Azure IoT Hub or Azure IoT Edge Hub.

Does this need to be on a separate line.

Refers to: azure-iot-device/azure/iot/device/iothub/sync_clients.py:178 in e5e5f1a. [](commit_id = e5e5f1a2575980e49d4525dfb869550435a797aa, deletion_comment = False)

    """Send a response to a method request via the Azure IoT Hub or Azure IoT Edge Hub.

Yes. Python docstrings are supposed to start with a single line, simple explanation, and then have more detailed explanation further down (if necessary).
We not the best about ALWAYS following this style, but I try my best.

In reply to: 535270265 [](ancestors = 535270265)

Refers to: azure-iot-device/azure/iot/device/iothub/sync_clients.py:178 in e5e5f1a. [](commit_id = e5e5f1a2575980e49d4525dfb869550435a797aa, deletion_comment = False)


fix(getImportedLibName): should handle "lib" and 'lib' correctly

resolve https://github.com/Brooooooklyn/ts-import-plugin/issues/11

Coverage remained the same at 100.0% when pulling 093d1a945bd329ac5385d3a4a1438b02ec9f4b7e on issue/get-imported-lib-name into a9f4e139e63f7404264c3e7e5e494a5ca8e6b9f2 on master.


Example code for SubredditRules is incorrect

Describe the bug
The example code for the subreddit rules loop produces a method error.
To Reproduce

Code as per example:
https://praw.readthedocs.io/en/latest/code_overview/other/subredditrules.html#praw.models.reddit.rules.SubredditRules

Tried in Jupyter notebook

import praw
CLIENT_ID = ''
CLIENT_SECRET = ''
USER_AGENT = ''
r = praw.Reddit(client_id=CLIENT_ID, client_secret=CLIENT_SECRET, user_agent=USER_AGENT)
for rule in r.subreddit("AskReddit").rules:
    print(rule)

TypeError: 'method' object is not iterable

Expected behavior

List of subreddit rules to be visible in some form

Code/Logs
My code that I am using, that works
for key, value in r.subreddit('askreddiit').rules().items():

Produces another dictionary, for which I've done this:
for key, value in r.subreddit('askreddiit').rules().items():

    if key == 'rules':

        for rule_num, rule in enumerate(value):
            print('\n','-'*100)
            print(f'Rule : {rule_num+1}')

            for key2, value2 in rule.items():
                print('\n' + str(key2) + ': ' + str(value2).replace('\n', '\n\t'))

System Info

OS:  Windows 10
Python:  3.8 64 bit
PRAW Version: 7.0.0

@epicmindwarp you're looking at the latest development code, which is not yet pushed out to master. Try https://praw.readthedocs.io/en/v7.0.0/, which is the documentation for the latest version on PyPI.


Is Marimo really reproducible?

Describe the bug
In the project description and docs it's claimed several times that Marimo is reproducible and has "no hidden state".
However, I noticed, that python objects can be changed by repeated and automatic execution, so they could contain content that might not be part of the notebook anymore.
Is this behavior intended or could this be fixed? Did I oversee or misinterpret anything?
Nontheless, great work on this nice project!
Environment
{
  "marimo": "0.2.2",
  "OS": "Linux",
  "OS Version": "<IP_ADDRESS>-microsoft-standard-WSL2",
  "Processor": "x86_64",
  "Python Version": "3.10.6",
  "Binaries": {
    "Chrome": "--",
    "Node": "v20.11.0"
  },
  "Requirements": {
    "black": "24.1.1",
    "click": "8.1.7",
    "jedi": "0.19.1",
    "pymdown-extensions": "10.7",
    "starlette": "0.37.0",
    "tomlkit": "0.12.3",
    "typing_extensions": "4.8.0",
    "uvicorn": "0.27.0.post1"
  }
}

Code to reproduce
Example code for repeated cell execution changing the output. First execute all cells and then repeatedly execute cell 2 and see the output of cell 3 change.
import marimo

__generated_with = "0.2.2"
app = marimo.App()

@app.cell
def __():
    # create some object
    mylist = []
    return mylist,

@app.cell
def __():
    # have some value that you change and then rerun this cell
    value = 6
    return value,

@app.cell
def __(mylist, value):
    # modify the object in a cell using that value
    value
    mylist.append(1)
    print(mylist)
    return

Example code for deleted content affecting cell output. First execute all cells and then delete cell 2.
import marimo

__generated_with = "0.2.2"
app = marimo.App()

@app.cell
def __():
    # create some object
    mylist = []
    return mylist,

@app.cell
def __():
    value = "Is this reproducible?"
    return value,

@app.cell
def __(mylist, value):
    # modify the object in a cell using that value
    try:
        mylist.append(value)
    except:
        pass
    print(mylist)

However, I noticed, that python objects can be changed by repeated and automatic execution, so they could contain content that might not be part of the notebook anymore.

That's correct -- this can happen.
Part of the contract the user must adhere to get reproducibility for free in marimo is to not mutate objects. This is covered in some of our guides/best practices documentation.
Tracking mutations is a huge rabbit hole, and has correctness cliffs as well as performance cliffs that are very difficult to reason about. It would require instrumenting Python code at runtime (performance cliff), and is impossible to get 100% correct (correctness cliffs). For these reasons marimo only uses static analysis in constructing its graph.
marimo's execution order is reproducible in the same sense as Pluto.jl; Pluto.jl also has package reproducibility, which we currently do not have but may in the future. Hope that helps clarify things.

Converting this to a discussion, in case you'd like to chat more.


Allow the user to specify the project sub-directory and namespace

Fix for issue #2750. (Replaces PR #2822).
Also updated -dnx.sln file to include SQLite projects (and generated their .xproj files).

Ping...

So questions on behavior: what should happen for when the user specifies:

Relative, up levels ../../output
Absolute C:\output\
Non-existent folder ./folder_that_does_exists_yet

Absolute - you'll get an error. Non-existent - it'll create the folder for you.
For up-levels I haven't decided yet - and of course it can be more complicated still e.g. "looks\like\subdir............\but\really\is\not". At the moment it will create the directory where you specify even if it is outside the project (provided of course you have permission to do so). I've been thinking of turning the full path into some kind of canonical path and then checking that that canonical path still starts with the project path i.e. you are absolutely, even including all possible ".." parts, still a sub-directory. And throwing an error if not. But was planning on a TODO for that.

Absolute - is this deliberate? Or just an implementation error? We should still allow absolute paths.
Non-existent -- good :+1:
Up-levels -- we can collapse them using Path.Combine and Path.GetFullPath

The approach to absolute is deliberate. I want to only allow the user to specify a path which is within the project. The project path is already passed in automatically by either Program.cs or Executor.cs - so all I need is the relative path.
Up-levels - yes - sounds good.

IMHO, it seems silly like overkill to let users specify the namespace. We should always just gen as RootNamespace.Directory. If the user follows some non-standard namespace, they can change it after generation.

I agree that we should update the namespace to reflect the output directory. I will add that (and that makes it even more important that we only allow relative paths).
The purpose of being able to override the namespace is for if you are generating lots of files. I guess it's still true that you can fix that with one big search-and-replace but it's also easy for us to provide the ability. I guess I won't fight hard to keep the feature if there are more people who don't like it.

One of our guiding principles in the EF7 commands was to have fewer command-line parameters and smarter conventions.
/cc @rowanmiller in case he has an opinion.

@bricelam - true. But this is not mandatory - if the user omits it we do as you say. And it seems (to me) to set us up well for when we integrate with VS and have a dialog that is asking these questions. But as I say, if there are more people who don't want this feature then I can easily remove.

Agree that we don't need this knob, it's a widely accepted coding guideline that namespaces match directory structure. If you really want to change it then you can just add the templates and modify it there.

@bricelam @rowanmiller OK. Latest commit removes that functionality.

@natemcmaster have put in the code to use GetFullPath() to check that relativeOutputDir must be a relative sub-directory of the project dir.

IMO absolute paths should still be allowed, but the code looks fine. :shipit:

Merged 5b19fbbff82987ba9e1aafe051ff8c4fd02bf8cf


Multi_threads Or Using multi-cores of pyarrow

Hello everyone，I'm trying to find the multi-cores way to process the data,but when I use pyarrow.set_cpu_count or pyarrow.set_io_thread_count to set a bigger int but the cost time basically no change. Is it caused by Python's GIL? How can I working out a parallel-working way ? Thx for replies!!!!!!

I'm afraid you will need to explain in more detail what you are trying to do and what results you are expecting to get.  Some operations in Arrow-C++ (which pyarrow uses) will take advantage of multiple threads.  These operations are controlled by the functions you mentioned.  Mostly today this is in file parsing and decoding.  On the other hand, some functions are implemented completely in python (and thus subject to the GIL) and others are implemented in C++ and release the GIL but do not use multiple threads.

I'm afraid you will need to explain in more detail what you are trying to do and what results you are expecting to get. Some operations in Arrow-C++ (which pyarrow uses) will take advantage of multiple threads. These operations are controlled by the functions you mentioned. Mostly today this is in file parsing and decoding. On the other hand, some functions are implemented completely in python (and thus subject to the GIL) and others are implemented in C++ and release the GIL but do not use multiple threads.

Sorry master，I didn't clarify what I was trying to do.Here is the my problem: I'm trying to use gandiva to speed up the filter operations.But I dont know how to use multi-cpus to speed up gandiva. So I wanna find a way to acheive this goal. Looking for


Invalid syntax error of Vim plugin

Operating system:
Ubuntu 16.04
Python version:
Python 3.6.5
Black version:
Vim Plug installs master from Github
Does also happen on master:
Yes.
After installation of the Vim plugin (using Plug under Vim 8.1.20) I get the following error every time I launch Vim:
Error detected while processing /home/ubuntu/.vim/plugged/black/plugin/black.vim:
line  114:
  File "<string>", line 17
    return venv_path / 'lib' / f'python{pyver[0]}.{pyver[1]}' / 'site-packages'
                                                            ^
SyntaxError: invalid syntax

Any idea how to fix this?

Your Vim is compiled with Python 2. Use Vim compiled with Python 3 (or both).


refactor(FMP-357): rename CrossChainDepositLayout -> DepositFlow

Current dependencies on/for this PR:

main

PR #49

PR #50   👈

This comment was auto-generated by Graphite.


Azure CCM/CSI updates

Which issue(s) this PR fixes:

xRef #2835
What type of PR is this?
/kind chore
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

* Update Azure CCM
* Update AzureDrive CSI
* Update AzureFile CSI

Documentation:

NONE

/retest

/retest


Consider move the repo somewhere else

Synr is a tvm-related project so far, which helps TVMScript works well. It does a great job to make AST parser stable across different python versions. However, it is not easy to update, since:

It is a dependent repo. There are so many steps if we need to add new features or fix bugs: make a PR; then wait for merge and new release; update TVM install script; add TVM feature and ut.
The repo under OctoML will block some of the contributors (due to some company policy) and we must wait for the guys in OctoML to release new releases, which makes it harder to fix bugs.

There are two possible proposals here:

Move it to tlc-pack: tlc-pack is the convention where we maintain tvm-related dependency. More people can manage it there.
Merging into tvm is the best way to make things maintainable. So, we don't need to install a specific version of synr for dev on a specific branch

cc @tqchen @jroesch @junrushao1994

Either options work for me.
Personally I hit lots of pain switching synr version in-between branches, and frontend development (e.g. @shingjan’s recent PR) needs to update both sides and keep doing minor releases, which is less convenient. Moreover, contributors to this repo should be honored in tvm too, so merging into TVM might be a good option IMO

Will vote for integrating synr into TVM as moving it into tlcpack would still leave the process of checking synr changes into TVM cumbersome.

Hi @Hzfengsy, there are definitely some challenges with maintaining synr as a separate repo, but I think they are outweighed by the benefits. To address your points specifically:

Yes, there are a couple more steps involved in updating synr, but the benefit we get is that synr does not have to go through the TVM CI (which takes a long time). I think we've also done a good job of merging PRs faster than what you would see in the TVM repo.
Have we had any contributors have this issue? The benefit here is that it is easier to run CI for the synr repo.

Regarding counter arguments to moving to specific repos:

There doesn't seem to be any benefit to moving to tic-pack besides maybe consistency. And then we'll have to setup CI and we won't be able to use the (freely provided) OctoML CI.
There are two options here: 1. we integrate synr in tree 2. we make synr a submodule. For 1. we now have to go through TVM CI for all changes, which takes a long time. It also prevents other projects from using synr, which was one of the original goals of the project. For 2. now users need to add another path to their PYTHOPATH and need to ensure they always keep the submodule up to date. This would be better handled by fixing dependency handling in TVM.

For your issue @junrushao1994, the only solution would be to put synr in-tree for TVM, which then hits the issues with slow CI and the inability for other projects to use synr.
The main reason why we started synr as a separate project was so that other projects could use synr for their ast parsing needs.

@tkonolige Thanks for the clarification. My points:

It is a good thing that as a separate repo, synr could have its own CI. But almost every change we bring to synr will end up on TVM, meaning the change we bring to synr/TVM will need to go through TVM CI anyway. Not to mention the fact that there is cost to run CI on its own and updating the version of synr in TVM.
Maintenance cost. Right now every time we update the version of synr, one TVM developer will need to fetch the latest TVM and manually fetch the latest synr from pip to make it work as synr is referenced roughly at three different places in the repo. This process could be confusing for first timers. Same issue will exist if we make synr a submodule, likely running git submodule update all the time.
We couldn't spot any change runs fine on synr's CI but not on TVM's CI now. There is no way to do that until we update the version of synr. And at that time we still need to decide if we want to roll back or roll out a new version.

I believe that right now synr is an active project we will be working on extensively. As a major dependency of TVM we could integrate it into the TVM tree so we can enjoy the benefit while making TVM the best DL compiler in the field. And if the development of synr is stabilizing and many other projects find this piece of work useful then we can separate this project again.


TEST-#5802: update some actions in CI

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #5802
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

@mvashishtha It's a pretty simple change, can you merge it?

Thanks @mvashishtha!


[Task] Add field appKey to ShenyuRequestLog

Description
Log statistics need to be analyzed from the application dimension，but in ShenyuRequestLog have not application info
Task List
1 Add field appKey to ShenyuRequestLog
2 Add appKey info in LoggingConsle plugin ( Print Request Info:)

it may not be necessary for shenyu request log..., Shenyu has reserved the implementation of ShenyuRequestLog,Parameters are passed in as generics


Can't install the addon

My Home Assistant versione: 0.98.1
This is my error log:
19-09-29 10:09:31 INFO (SyncWorker_8) [hassio.docker.interface] Pull image dwelch2101/zigbee2mqtt-armhf tag 1.6.0.
19-09-29 10:09:33 ERROR (SyncWorker_8) [hassio.docker.interface] Can't install dwelch2101/zigbee2mqtt-armhf:1.6.0 -> 500 Server Error: Internal Server Error ("readlink /mnt/data/docker/overlay2/l: invalid argument").

Are you using HassOS or some generic docker installation? Looks like remote server (which in this case would be docker hub) replied with 500.
Maybe do a retry and see if it works

I have a similar issue:
ERROR (SyncWorker_14) [hassio.docker.interface] Can't install dwelch2101/zigbee2mqtt-armhf:1.6.0 -> 500 Server Error: Internal Server Error ("Get https://registry-1.docker.io/v2/dwelch2101/zigbee2mqtt-armhf/manifests/1.6.0: Get https://auth.docker.io/token?scope=repository%3Adwelch2101%2Fzigbee2mqtt-armhf%3Apull&service=registry.docker.io: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)").

same here. Can not upgrade from 1.5* to 1.6.
19-10-27 23:58:21 INFO (SyncWorker_3) [hassio.docker.interface] Update image dwelch2101/zigbee2mqtt-armhf:1.6.0 to dwelch2101/zigbee2mqtt-armhf:1.6.0 19-10-27 23:58:21 INFO (SyncWorker_3) [hassio.docker.interface] Pull image dwelch2101/zigbee2mqtt-armhf tag 1.6.0. 19-10-27 23:58:21 ERROR (MainThread) [aiohttp.server] Error handling request Traceback (most recent call last):   File "/usr/local/lib/python3.7/site-packages/docker/api/client.py", line 261, in _raise_for_status     response.raise_for_status()   File "/usr/local/lib/python3.7/site-packages/requests/models.py", line 940, in raise_for_status     raise HTTPError(http_error_msg, response=self) requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: http+docker://localhost/v1.39/images/sha256:6a0cca52aeaeaf381a15838b41df29ec0cf9c87b786668542be211474d7e6d5b/json During handling of the above exception, another exception occurred: Traceback (most recent call last):   File "/usr/local/lib/python3.7/site-packages/aiohttp/web_protocol.py", line 418, in start     resp = await task   File "/usr/local/lib/python3.7/site-packages/aiohttp/web_app.py", line 458, in _handle     resp = await handler(request)   File "/usr/local/lib/python3.7/site-packages/aiohttp/web_middlewares.py", line 119, in impl     return await handler(request)   File "/usr/src/hassio/hassio/api/security.py", line 147, in token_validation     return await handler(request)   File "/usr/src/hassio/hassio/api/utils.py", line 39, in wrap_api     answer = await method(api, *args, **kwargs)   File "/usr/src/hassio/hassio/addons/__init__.py", line 205, in update     await addon.instance.cleanup()   File "/usr/src/hassio/hassio/utils/__init__.py", line 29, in wrap_api     return await method(api, *args, **kwargs)   File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run     result = self.fn(*self.args, **self.kwargs)   File "/usr/src/hassio/hassio/docker/interface.py", line 320, in _cleanup     for image in self.sys_docker.images.list(name=self.image):   File "/usr/local/lib/python3.7/site-packages/docker/models/images.py", line 364, in list     return [self.get(r["Id"]) for r in resp]   File "/usr/local/lib/python3.7/site-packages/docker/models/images.py", line 364, in <listcomp>     return [self.get(r["Id"]) for r in resp]   File "/usr/local/lib/python3.7/site-packages/docker/models/images.py", line 316, in get     return self.prepare_model(self.client.api.inspect_image(name))   File "/usr/local/lib/python3.7/site-packages/docker/utils/decorators.py", line 19, in wrapped     return f(self, resource_id, *args, **kwargs)   File "/usr/local/lib/python3.7/site-packages/docker/api/image.py", line 246, in inspect_image     self._get(self._url("/images/{0}/json", image)), True   File "/usr/local/lib/python3.7/site-packages/docker/api/client.py", line 267, in _result     self._raise_for_status(response)   File "/usr/local/lib/python3.7/site-packages/docker/api/client.py", line 263, in _raise_for_status     raise create_api_error_from_http_exception(e)   File "/usr/local/lib/python3.7/site-packages/docker/errors.py", line 31, in create_api_error_from_http_exception     raise cls(e, response=response, explanation=explanation) docker.errors.APIError: 500 Server Error: Internal Server Error ("readlink /mnt/data/docker/overlay2/l: invalid argument") 19-10-27 23:58:41 INFO (SyncWorker_10) [hassio.docker.addon] Start Docker add-on dwelch2101/zigbee2mqtt-armhf with version <IP_ADDRESS>
HassOS 2.12
hass.io supervisor: 192

You could try uninstall / reinstall of the addon, as most of the settings are persisted in /share/zigbee2mqtt

I had same error earlier, in homeassistant forum they recommended to check if there disk is full,
in console run command:
df -h
if the disk is not full, then you might have  some corrupted files, to fix them run command:
hassio su repair
that fixed my error.

Yeah, these errors are usually because of disk space or some other issue with the docker host.

As pointed out by @Reisga hassio su repair solved the issue

I cant confirm "hassio su repair" works


Crash on Android - vulcan - stack trace

Tested versions

From Godot 4.1 android export

Report on phone model :

[samsung a05s (Galaxy A05s)]   [ vivo V2342 (Y100) ] [ Redmi tapas (Redmi Note 12) ]  [ TECNO TECNO-KG5k (TECNO SPARK 8C) ]

On Android Version

[Android 14 (SDK 34)]  [Android 13 (SDK 33)]  [Android 11 (SDK 30)]
System information
Godot 4.1 (stable)
Issue description
Multiple crashes , impacting about 10% of users on Google Play.
I only have the crash report from Google Play Console.
Never had this issue locally.
I can't bring any other information. Sorry.
Steps to reproduce

pid: 0, tid: 19534 >>> ca.pixels777.bacb <<<
backtrace:
Unknown
#01  pc 0x00000000026d6054  /data/app/~~2wEnTMcbG5v47NHBf-yvXg==/ca.pixels777.bacb-nf_zFBDiNzs7m9gYx13oOA==/split_config.arm64_v8a.apk!libgodot_android.so
#02  pc 0x0000000000b7c944  /data/app/~~2wEnTMcbG5v47NHBf-yvXg==/ca.pixels777.bacb-nf_zFBDiNzs7m9gYx13oOA==/split_config.arm64_v8a.apk!libgodot_android.so
#03  pc 0x0000000000b55480  /data/app/~~2wEnTMcbG5v47NHBf-yvXg==/ca.pixels777.bacb-nf_zFBDiNzs7m9gYx13oOA==/split_config.arm64_v8a.apk!libgodot_android.so (Java_org_godotengine_godot_GodotLib_step+252)
#04  pc 0x000000000001ea3c  /data/app/~~2wEnTMcbG5v47NHBf-yvXg==/ca.pixels777.bacb-nf_zFBDiNzs7m9gYx13oOA==/oat/arm64/base.odex (art_jni_trampoline+108)
#05  pc 0x000000000004f674  /data/app/~~2wEnTMcbG5v47NHBf-yvXg==/ca.pixels777.bacb-nf_zFBDiNzs7m9gYx13oOA==/oat/arm64/base.odex (org.godotengine.godot.vulkan.VkThread.run+900)
#06  pc 0x000000000033b3a4  /apex/com.android.art/lib64/libart.so (art_quick_invoke_stub+612)
#07  pc 0x000000000023a5d0  /apex/com.android.art/lib64/libart.so (art::ArtMethod::Invoke(art::Thread*, unsigned int*, unsigned int, art::JValue*, char const*)+144)
#08  pc 0x0000000000539b2c  /apex/com.android.art/lib64/libart.so (art::Thread::CreateCallback(void*)+1600)
#09  pc 0x00000000005394dc  /apex/com.android.art/lib64/libart.so (art::Thread::CreateCallbackWithUffdGc(void*)+8)
#10  pc 0x00000000000fd004  /apex/com.android.runtime/lib64/bionic/libc.so (__pthread_start(void*)+208)
#11  pc 0x0000000000096928  /apex/com.android.runtime/lib64/bionic/libc.so (__start_thread+68)
Minimal reproduction project (MRP)
sorry, no hints on how to reproduce.

Devices that only have Vulkan 1.0 support in their drivers generally have broken Vulkan support. I recommend using the Compatibility rendering method instead.

Closing due to lack of response, if you provide the requested information this can be reopened


fluid.bufnmf blocks bang till it finishes processing

In fluid.bufnmf-help.pd, when i click on the bang to start [fluid.bufnmf], the bang doesn't highlight until fluid.bufnmf finishes processing. I'm not sure how the other pd objects do it, but usually the bang highlights and the object carries on processing.

Hello! It is not an issue per se - all Pd object are on a single thread and the default threading is to stay in that thread... nmf is a huge process so it stalls the rest. But don't despair, check the top right of the help file, in [note_on_multithreading] and enjoy the blocking mode 0!


Update day5.ipynb

4c can be unclear to some, 4 cents is better to use instead

@Ishan-Karpe thanks so much - good catch!

@ed-donner does the PIL Libary exist? I can't download for some reason. it's not in requirements.txt, so it wasn't installed initially. Any alternatives to PIL?

Hey @Ishan-Karpe oh that's odd, it should be installed as a prerequisite of other things in requirements.txt (you don't need to list all packages in requirements.txt, pip automatically installs prerequisities).
The package is called pillow..
pip install pillow
https://pypi.org/project/pillow/


Breaking changes ? In general or possibly windows binary build.

Hello mxnet suddenly stopped working between builds
Converges for binary build :
20151209 and 20151228
No learning happens for binary build:
20160106 and 20160113
I took the MNIST example as a reference but other example also showed no learning.
With mnist I tried 'lenet' and 'mlp'. Both did not work. Tried examples with softmax and regression.
All had the same phenomenom.
I checked the dll libraries that were loaded and I did not see anything suspicous.
See the attached image. I cleaned the python dir and reinstalled.
All to no avail.
Does anyone know about a possible breaking change between the 20151228 and the 20160106 release ?
Update 1:
The MNIST example works for the CPU. So it has to do with the GPU code.

Excuse me.. There might be something more to it.
The MNIST example also does not converge for lenet and mlp..
Closing for now

I just hit the same problem with MxNet.cpp. I thought it was the cpp wrapper that is causing the problem. Let me try to figure this out.

Thanks for the quick reply:
I ruled out a few things:

It works for CPU
I isolated/left out many layer/symbol types :
Convolution (problem is also when I don't use them)
Fully Connected  (problem is also when I don't use them)
BatchNorm (problem is also when I don't use them)
Softmax (problem is also when I don't use them)
The newest release did work on a completely clean other windows system.
However.. looking at the loaded libraries it seems exactly the same

@juliandewit We are now trying to diagnose the problem in #1228 . Let's move the discussion there. I am closing this issue for now.