Move macwire away from adamw/softwaremill to its own organization

Packages and build organization should be renamed into org.macwire. The project should be moved to a macwire Github organization.
I guess that, as it will be a new artifact, it'll have version 1.0.0.

I've created https://github.com/mac-wire as macwire was already taken.

What's wrong with the current organization or account? It would also require getting a domain. I'm not sure it's worth confusing the users. There's a lot of popular projects on normal user accounts, no?

You don't really need a domain. You can use a github subdomain, very much like di-in-scala.
About the current organization (and user account for that matter), it's clearly bringing attention to softwaremill (and adamw ;)), which is fine as long as you're the only committer, or as long as the only committers are softwaremill-ers ;)
Most well-known open source projects aren't affiliated to a company because that would be unfair to the other companies/individuals that are also contributing (pants build is a good example). Plus you never know, tomorrow you might work in a different company or simply decide to rebrand your company (like typesafe).
Once macwire is on its own, we can add a credit section.

Well you need a domain if you want to deploy to maven central (as far as I know).
Anyway, I don't think that it was an issue so far, so I'm going to leave things as-is for now. If there are more users interested in such a move - this is a good place to voice your opinion :). I definitely don't want to scare any potential contributors away, but I hope that the account name doesn't matter that much really.
Moving to a separate organisation would make sense if there were several sub-projects (which I don't think is happening at least yet) or if this was a really huge "framework" on top of which you build applications. I still view macwire as a rather small helper library to make a programmer's life easier :)

I definitely don't want to scare any potential contributors away, but I hope that the account name doesn't matter that much really.

Sure, it's not the account name that bothers me the most, it's the organization. Here is what you see when you land on adamw/macwire: "Lightweight and Nonintrusive Scala Dependency Injection Library
http://www.softwaremill.com". So when you start contributing significantly, you wonder, do I work for softwaremill?

If there are more users interested in such a move

This is mostly about contributors, not users. As a user I don't really care if that's 'com.jujubes.macwire' or 'org.mydearcharity.macwire', as long as I can get the code with a permissive licence.

Well you need a domain if you want to deploy to maven central (as far as I know).

See http://central.sonatype.org/pages/choosing-your-coordinates.html
"Other examples are [...] github.com/yourusername -> com.github.yourusername"
Try a search: https://search.maven.org/#search|ga|1|com.github
"Displaying 1 to 20 of 6639" - a fairly common practice.

Moving to a separate organisation would make sense if there were several sub-projects

A good start would be a documentation website. Like I said in #60 I don't really like the state of affairs - hence my scarce contributions to the README.

I still view macwire as a rather small helper library

Size does not matter ;)

I guess you've already been there, but that's another good example: https://github.com/scaldi
A small scala DI library with its own organization.

Hmm why would you be working for softwaremill? It's the company I'm working at and I use that org for most open-source that I write. The same I don't have trouble contributing to com.google, I don't see why this would be a problem here :) I'm sure there's a lot of projects using the github domain, but this would be just confusing to users without a real benefit
As for docs, I think it's best if they live alongside the code, why would we want a separate repository :)

Hmm why would you be working for softwaremill?

Exactly, why? Softwaremill is the company you co-founded and you're obviously using open-source to promote it. Why don't you admit it? There's nothing wrong with that.
Like you I have no problem making small contributions to open-source projects that are advertising a lucrative company, but I feel differently when I'm rewriting the whole thing.

As for docs, I think it's best if they live alongside the code, why would we want a separate repository :)

You mean as a readme? I don't think that format is adequate, plus it's fairly easy to create a jenkyll documentation website with a decent navigation.

I definitely don't want to scare any potential contributors away

I've started this whole discussion because that's bothering me, so don't take my contributions for granted. I've actually done quite some work over the last few days that is on hold until this gets fixed.

this would be just confusing to users without a real benefit

I don't think that at this stage (how many projects are using macwire in production?) it'll confuse people that much. We're talking about renaming some imports and a dependency in your build configuration.

Exactly, why? Softwaremill is the company you co-founded and you're obviously using open-source to promote it. Why don't you admit it? There's nothing wrong with that.
Like you I have no problem making small contributions to open-source projects that are advertising a lucrative company, but I feel differently when I'm rewriting the whole thing.

Sure, it is promoting Softwaremill in some way, though I don't think it's really significant. And I don't think anybody views contributing to a library created by some company as working for that company. Also I think there's quite some code left from the original ;)

You mean as a readme? I don't think that format is adequate, plus it's fairly easy to create a jekyll documentation website with a decent navigation.

Not necessarily, it can be .md files in a docs subdirectory or anything else

I don't think that at this stage (how many projects are using macwire in production?) it'll confuse people that much. We're talking about renaming some imports and a dependency in your build configuration.

I don't have any numbers, but I think quite a few. As for now I'm not going to do the rename - sorry.

I've started this whole discussion because that's bothering me, so don't take my contributions for granted. I've actually done quite some work over the last few days that is on hold until this gets fixed.

Look, I'll be happy to merge more changes (as long as they make sense ;) ) but I'm also not going to do everything just to get them in. I really wouldn't like to see you go, and I'm not sure why this is so important suddenly, it's a simple open-source project, let's make sure it's as good as it can be and keep coding :)

Sure, it is promoting Softwaremill in some way, though I don't think it's really significant.

If that's not significant, why not doing it? I'd be happy to help you move the project, and create a good documentation.

I'm also not going to do everything just to get them in

Is that so much to ask? So you'd rather invest your time on coding and maintaining than simply transitioning the project to a more open and independent space where you can benefit, for free, from my contributions?
Take akka for example, it used to be se.scalablesolutions.akka. See https://github.com/akka/akka/commit/680ee7ddf105fa5c7057f30a899c4cad3a36a665

I'm not sure using Akka as an argument fits well into the discussion here.
Technically speaking akka is strictly published under the com.typesafe.akka groupId.
Packages inside the artifacts indeed akka.*, however this is for convenience / style.
Personally, I've been happily contributing to com.twitter.* or google or foursquare repositories with an "they started it, so it lives there" philosophy, and the situation here seems to be pretty similar.
Just my 2 cents, don't want to dive too deep into the discussion here – it's something the maintainer is ought to decide himself.

I'm not sure using Akka as an argument fits well into the discussion here.

The argument was that renaming the packages would be confusing to the users. I'm bringing Akka as an example of how a well-established project with a far larger user base was able to do such a change.

Personally, I've been happily contributing to com.twitter.* or com.google or com.foursquare or "someone's random repo" repositories

Again the question is about how much you contribute. I've also been contributing marginally to com.google, sun, ...

the situation here seems to be pretty similar.

When you're responsible for > 50% of the current code of the whole project it seems to me that you might be entitled to question how the project is used, and if some advertisement is derived from it, you should rightfully have a word to say.

it's something the maintainer is ought to decide himself.

Of course, I'm not forcing anyone... and nobody is forcing me to contribute.

If that's not significant, why not doing it? I'd be happy to help you move the project, and create a good documentation.

That was just a side note, not really important.

Is that so much to ask? So you'd rather invest your time on coding and maintaining than simply transitioning the project to a more open and independent space where you can benefit, for free, from my contributions?

I suspect that I would be maintaining the project anyway ;) I think the project is quite open, the code is there, all PRs are reviewed etc., so let's not create artificial problems.
Look, if you'd like to somehow underline that you are not from Softwaremill, we can create a "contributors" section with a link of your choosing or something like that.

I suspect that I would be maintaining the project anyway ;)

If you leave things as they are it's very likely. Otherwise I wouldn't mind maintaining it - as a matter of fact I'm already maintaining my own private version, and it's interesting to see that you're more or less (re)producing the same code as I did a week ago.

we can create a "contributors" section

Yep, that's what I suggested, once it's moved to its organization, only that it would be called "credit".
See https://pantsbuild.github.io/credits.html for inspiration ;)

If you leave things as they are it's very likely. Otherwise I wouldn't mind maintaining it - as a matter of fact I'm already maintaining my own private version,

You make it sound as if things were in a bad state. Sorry, but I really don't believe that the package name/organization can be such a problem. EOT on my side.

and it's interesting to see that you're more or less (re)producing the same code as I did a week ago.

which only validates the approach ;)

Yep, that's what I suggested, once it's moved to its organization, only that it would be called "credit".
See https://pantsbuild.github.io/credits.html for inspiration ;)

Well I guess that's a 5-minute job, go ahead :)

go ahead :)

Two things:

i'm on holiday for 2 weeks :)
if you don't agree, there won't be 2 versions of macwire on github, that would definitely create some confusion, plus that would sound like a fight, whereas I just want to work with you on a fair basis (at least fair in my opinion)

i'm on holiday for 2 weeks :)

Have a good time :)

if you don't agree, there won't be 2 versions of macwire on github, that would definitely create some confusion, plus that would sound like a fight, whereas I just want to work with you on a fair basis (at least fair in my opinion)

I'm not sure I understand this ;) Anyway, I guess I said everything that I wanted to here

I'm not sure I understand this ;)

It's simple, if you agree with the organization move, then the following (amongst other things) should be done:

move the project to github.com/macwire/macwire
have a link on github.com/adamw/macwire (at that point this repo will be empty), redirecting to macwire/macwire

When you said 'go ahead' I understood that you told me to fork your project on github - which I won't as I said previously.

No I meant adding the credits section.
I think I wrote a couple of time that I'm not going to change the organisation as of now.

I just created the macwire organization (that was blocked by a phantom user BTW). Now that one year has passed you might want to reconsider this?

@backuitist wow, looks like this is really important for you. Besides the enterprise-backed com.twitter|google|whatever, there are also lots of personal repos like non/spire, milessabin/shapeless etc, and both maintainers and contributors are fine with that. So, if no one raised this issue in the past year and the repo owner expressed his unwillingness pretty clearly, is there really a point in reiterating on this?
Usual disclaimer on 'my 2 cents' here.

I can reiterate my points for you: First, I don't feel comfortable with the advertisement done on the project (first and foremost link points to softwaremill), which IMO is not comparable with the other projects that you mentioned.
Second, it's a matter of involvement. It's funny that you consider the contributors as being fine with this although, to this date, if you remove softwaremill employees, you're basically left with me and some extremely marginal contributors ;)
The scala ecosystem is fragmented enough (look at json libraries) that it seemed wiser not to fork this project. But I guess I'll follow your advice and fork so I can release my work on better terms.

The link will be actually changed soon to point to our open-source page, which is in the works, though I don't think it really matters to that discussion. Is the link really such a problem? I find that hard to believe :)
As for involvement, if all major contributors are indeed SoftwareMill employees and you (haven't checked that), how is that an argument against using the com.softwaremill package?
It's open-source so you are free to fork of course, I'll be curious to check out what kind of improvements you have made, and maybe back-port some (retaining all attributions of course). And so there's no doubt: I'd be even more happy to get some PRs to this project.


refactor(server): Add set status code function

:heavy_check_mark: Checklist:

[x] Have only one commit (if not, squash them into one commit)
[x] Tests for the changes have been added (for bug fixes / features / refactoring)

Codecov Report

Merging #207 (2c22f50) into main (da87702) will increase coverage by 6.10%.
The diff coverage is 59.34%.

@@            Coverage Diff             @@
##             main     #207      +/-   ##
==========================================
+ Coverage   66.84%   72.94%   +6.10%
==========================================
  Files          10       12       +2
  Lines         368      451      +83
==========================================
+ Hits          246      329      +83
+ Misses        111       91      -20
- Partials       11       31      +20

Impacted Files
Coverage Δ

internal/apischema/find.go
21.21% <21.21%> (ø)

internal/openapi3/parse.go
47.05% <50.00%> (+12.44%)
:arrow_up:

internal/apischema/apischema.go
72.41% <72.41%> (ø)

internal/openapi3/openapi3.go
75.00% <75.00%> (ø)

internal/openapi3/schema.go
75.00% <75.00%> (ø)

internal/faker/faker.go
95.27% <100.00%> (ø)

internal/openapi3/example.go
100.00% <100.00%> (ø)

internal/openapi3/mediatype.go
100.00% <100.00%> (+14.28%)
:arrow_up:

internal/server/handler.go

internal/server/server.go

... and 1 more

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 2697298...2c22f50. Read the comment docs.


[QUESTION] Flat buttons with border

Question
I've added a push button to my application, with flat set to true. However, it still has borders on, contrary to the native version of the button.
Is this the expected behavior?
Dark Theme (On Light Theme it also has borders):

Windows 10 Native:

I believe they could be visually similar to the QToolButtons.

This is a bug, since well, it's supposed to be flat. It also contrasts with KDE's Breeze themes themselves, which is a stylistic inspiration for these stylesheets.


Error message regarding Ads API v14

Hi,
I've got access to the Basic Ads API so I have a token. But when I run the script, I get this error message which seems to implicate that the API needs to be updated:
Error: {"code":400,"message":"Request contains an invalid argument.","status":"INVALID_ARGUMENT","details":[{"@type":"type.googleapis.com/google.ads.googleads.v14.errors.GoogleAdsFailure","errors":[{"errorCode":{"requestError":"UNSUPPORTED_VERSION"},"message":"Version v14 is deprecated. Requests to this version will be blocked."}],"requestId":"qUu3ZSMS9jDpnmuJSBBbLA"}]}
Because of the bold text (v14 version) I think it is an error regarding the Ads API version but not sure if I need to do something regarding my Ads API or that it's regarding the Adios system?

Hi,
I got the same error today - I couldn't find a solution on my end.

Great news. I do not see the error message anymore. But I know have: SyntaxError: Unexpected token 'N', "Not Found" is not valid JSON
Lars Maat
Online Marketing Strateeg & Kwaliteitsmanager
Make an impact and inspire others to do the same!
Wij leveren impact - Wij inspireren - Wij zijn Maatjes
Bezoekadres
Telefoon privé
Telefoon algemeen
Mobiel
Website
Goudsesingel 190, 3011 KD Rotterdam
010 30 38 902
010 30 30 222
06 30 33 06 12
www.maatwerkonline.nl
Van: Andriy Kushnarov @.>
Verzonden: woensdag 3 juli 2024 11:47
Aan: google-marketing-solutions/adios @.>
CC: Lars Maat van Maatwerk Online @.>; Author @.>
Onderwerp: Re: [google-marketing-solutions/adios] Error message regarding Ads API v14 (Issue #47)
U ontvangt niet vaak e-mail van<EMAIL_ADDRESS>Meer informatie over waarom dit belangrijk ishttps://aka.ms/LearnAboutSenderIdentification
Hi everyone,
We're thrilled to announce the release of Adios v1.2! This new version now leverages the power of Google Ads API v17. We appreciate your patience during the development process.

Reply to this email directly, view it on GitHubhttps://github.com/google-marketing-solutions/adios/issues/47#issuecomment-2205594428, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJJYAG3ERTGZPNDNOZZ6KWDZKPCAXAVCNFSM6AAAAABJRQ662SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBVGU4TINBSHA.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Add release notes to the API docs

Context
This adds the release notes for the API.
Changes proposed in this pull request
This proposes some changes to how we do change noting.
While we're still in the pre-v1 phase, the release notes added to a separate "Alpha release notes" page. This is to signify that it's a different type of change - it doesn't break, since no one has been building on it.
I've also removed all references to sub-versions like "0.8.0". This is just confusing, because it implies we can "release" a version like that (we can't, since we deploy continously).
Guidance to review
Does it make sense?
Link to Trello card
https://trello.com/c/KENwAoIR/1357-merge-tech-docs-into-main-application

Does it make sense to include the spec.yml endpoint in the spec?

It's not JSON (I guess it could be?)
you'd only know about the endpoint if you had the spec in the first place

It's not JSON (I guess it could be?)
you'd only know about the endpoint if you had the spec in the first place

I think it's useful to expose the full YAML file. I considered adding it to the docs, but I thought it made more


Add real Time image Solve option using opencv2.

You can add a Real-time image solve option using cv2, if you do not have enough time I can do it for you
You can assign me and i can start working on it

Sure. 👍


What do the "deformed" and "prediction" mean?

HI!

In codes:
real_out_img_list.append(generated["prediction"])
real_warped_img_list.append(generated["deformed"])

In the demo, what do the second and the third part seperately mean? As i know, they are probably "deformed" and "prediction". Could you describe it in detail.
Appreciate for your patient answer

"deformed" means directly applying the optical flow (deformation flow) to the original image to show the warped result, while "prediction" represents the final output image from the decoder of our network.

Thanks a lot!

Thanks a lot!

"deformed" means directly applying the optical flow (deformation flow) to the original image to show the warped result, while "prediction" represents the final output image from the decoder of our network.


docs/man/docker.1.md: spelling mistake fix

Spelling mistake fix in the documentation

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:1HLtd/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

can you please squash your commits, you can do so by git rebase -it origin/master then squash the necessary commits.

Done :)
Sorry for the delay.

LGTM - @fredlf @jamtur01 @spf13 @moxiegirl


Support upstream requests per backend

Copy&paste from code in PR #14 (thanks to @zouyee). Code tested and currently running in production.
Fixes #59
Dashboard updates incorporating this change (once merged) will follow, @hadret is taking care of that :)

Thanks for your contribution!


Sort-comp rule doesn't enforce getDerivedStateFromProps placement

With "react/sort-comp": "warn" provided code shouldn't be considered valid:
static getDerivedStateFromProps(nextProps, prevState) {}

state = {}

Tested on version 7.8.2

What is your full eslint config?

@lynxtaa you should be specifying a version pragma in "settings" (see the read me), but I don't think that's it. Could you also share the entire component code in question?

Still the same issue with specified pragma.
Example component Test.jsx:
import React from 'react'

class Test extends React.Component {
	static getDerivedStateFromProps() {
		return null
	}

	constructor(props) {
		super(props)
		this.state = { test: 'test' }
	}

	render() {
		return this.state.test
	}
}

export default Test

React 16.3.2.

ok, what warning do you get with that component?

No warning. But i should get "react/sort-comp" warning according to this https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/sort-comp.md
getDerivedStateFromProps should be declared after constructor.

aha, this seems to be a bug in the sort-comp implementation then - since the lifecycle methods are typically instance methods, but getDerivedStateFromProps is a static method.
cc @joe-denea / #1771 which has tests for the "valid" case for this method, but not for the "invalid" case.

Sorry about that, I should definitely have tested that.

Fixed in #1795.


e2e: pr warning testTue, 25 Jul 2017 15:54:02 GMT

:warning: Validation status: warnings

File
Status
Preview URL
Details

E2E_DocsBranch_Dynamic/index.md
:warning:Warning
View
Details

E2E_DocsBranch_Dynamic/index.md

Line 2: [Warning] Invalid file link:(~/../nonexisted1.md).
Line 3: [Warning] Invalid file link:(~/../nonexisted2.md).
Line 4: [Warning] Invalid file link:(~/../nonexisted3.md).

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


settings unity project

Hi All,
I download this repo and in the example folder i find the unity project
is possible to have an info about unity configuration?
I try to open the project with unity 2021.3.8f1 but in the bar menu file there isn't a menu for RN export...
some guys spoke about a readme file but i dont' find that... but i find a older guide:
https://github.com/asmadsen/react-native-unity-view
is normal?
can you help me please?
thank you

@azesmway Can you explain to me if i need set some option in the editor?
-> i opened the example in Unity editor
-> switch android platform
-> add IL2CPP, and ARM64
AND?
thanks

Thank You for your fast help!

@azesmway the export work fine but in android studio the android build fail.
I have changed the ndk version: before -> 21.3.6528147 now -> 25.1.8937393
but i have 2 fails:
1 -
Execution failed for task ':unityLibrary:BuildIl2CppTask'.

Process 'command 'C:/Users/USER/Desktop/rn-unity/unity/builds/android/unityLibrary/src/main/Il2CppOutputProject/IL2CPP/build/deploy/netcoreapp3.1/il2cpp.exe'' finished with non-zero exit value 4

2 -

What went wrong:
Execution failed for task ':app:processDebugMainManifest'.

Manifest merger failed : Apps targeting Android 12 and higher are required to specify an explicit value for android:exported when the corresponding component has an intent filter defined. See https://developer.android.com/guide/topics/manifest/activity-element#exported for details.

build grandle:
ext {
buildToolsVersion = "31.0.0"
minSdkVersion = 21
compileSdkVersion = 31
targetSdkVersion = 31
ndkVersion = "25.1.8937393"
}
have any idea?

I try your example:
https://drive.google.com/file/d/1W1wiwH7HVMKnWDi6uJN8RqQRmKh7KgTw/view?usp=sharing
https://github.com/azesmway/react-native-unity/issues/34
But not compile

@azesmway ok now work your source example and the message from unity work but not working the message from RN to unity after 6s.
The last problem is:
from Main screen to Unity Woork fine
but when i come back from Unity screen to Main the app make a Crash.
Any Suggestion?
Thanks

@rbrogi I fixed this issue.
unity/builds/android/unityLibrary/build.gradle

android {
    compileSdkVersion 33
    buildToolsVersion '30.0.3'
    ndkVersion '21.3.6528147' // << add ndk version!

    compileOptions {
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVer


Add attribute to prefix all machine run_list

Need a way to add to machine run lists either through a prefix or postfix in the env json.
@dcbitman would like to be able to add recipes to the setup recipes machine runlists.

Enhancement implemented with https://github.com/opscode-cookbooks/delivery-cluster/pull/109


Allow Threat_Note to run on Postgres

Right now the assumption is people will run Threat_Note on Sqlite. Sqlite has some great characteristics:

Portable
Easy to setup
Scales more than people think

But Postgres is pretty great as well. It's fast, scales really well, supports lots of datatypes, and (and I think most interesting of all) allows applications to run on Heroku. The foundation was already set by @alxhrck when things were cut over from direct sqlite queries to SqlAlchemy.
Here are some useful links:

Heroku: Getting Started with Python on Heroku
Flask and PostgreSQL on Heroku
How to Setup Postgresql (Python-flask) App on Heroku and Local Machine

The key is this should not break the current method of local deploying with Sqlite. This remains useful for people running small instances and development.
Thoughts?

Certainly sounds reasonable to me; I'm glad it keeps it friendly for those
wishing to run it locally. Speaking of Heroku, what are your plans for SSL?
On Feb 13, 2016 10:01 AM, "Scott J. Roberts"<EMAIL_ADDRESS>wrote:

Right now the assumption is people will run Threat_Note on Sqlite. Sqlite
has some great characteristics:

Portable
Easy to setup
Scales more than people think

But Postgres is pretty great as well. It's fast, scales really well,
supports lots of datatypes, and (and I think most interesting of all)
allows applications to run on Heroku. The foundation was already set by
@alxhrck https://github.com/alxhrck when things were cut over from
direct sqlite queries to SqlAlchemy.
Here are some useful links:

Heroku: Getting Started with Python on Heroku
https://devcenter.heroku.com/articles/getting-started-with-python#introduction
Flask and PostgreSQL on Heroku
http://blog.y3xz.com/blog/2012/08/16/flask-and-postgresql-on-heroku
How to Setup Postgresql (Python-flask) App on Heroku and Local
Machine
http://beatofthegeek.com/2013/04/how-to-setup-postgresql-python-flask.html

The key is this should not break the current method of local deploying
with Sqlite. This remains useful for people running small instances and
development.
Thoughts?
—
Reply to this email directly or view it on GitHub
https://github.com/defpoint/threat_note/issues/136.


ReShade Toggler affecting gammma

Gamma seems to be affected when UI is off screen or off screen, some have been able to trace it back to REST, specifically the UI 1 and 2  in the drop down.

i made a pull request, check my version and let me know if this helps

hey! my pull request is nullified. the issue stems from using the version of REST listed in the README, which is not the version that should be used anymore. 4lex4nder is i believe updating the README now, but you should now be using the LATEST version of REST instead of v1.3.21. just go re-grab REST latest, and then replace your config again and all should be good :)

Yep! Just tried it and it worked!

Updated from 1.3.20 to 1.3.22. Replaced both the .addon files from REST and the .ini file from REST-FFXIV, and currently have this issue... but much worse.

Current REST version in ReShade showing <IP_ADDRESS>3. Turning off UI 2 seems to solve the issue as stated above.


Added missing FastAPI type

Description
(Trivial) Added missing FastAPI type

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

Comparison is base (6840c5f) 95.81% compared to head (a5fbe72) 95.82%.
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #358   +/-   ##
=======================================
  Coverage   95.81%   95.82%
=======================================
  Files          32       32
  Lines        1052     1053    +1
=======================================
+ Hits         1008     1009    +1
  Misses         44       44

Files
Coverage Δ

ols/src/ui/gradio_ui.py
95.83% <100.00%> (+0.08%)
:arrow_up:

/lgtm

/approve


Convert createKnitService to accept a config callback

As discussed we update the createKnitService to accept a config callback. The API is now similar to ConnectRouter.

I may be missing some context but the API that connect-router uses looks (roughly) like:
const router = createConnectRouter().rpc(...).service(...);
// Do something with router

This provides us a chainable api so we could theoretically do something like:
const routerA = createConnectRouter().rpc(...);
const routerB = routerA.service(...);
// Do something with routerA and routerB separately

This change to use service over addService feels like it doesn't provide the same functionality. If we're looking to match the API of connect-router, would it make more sense to make it chainable instead of a callback?

@paul-sachs So the current API in practice will look like:
createKnitService({
    configure({service}) {
        service(a);
        service(b);
    }
});

Do you think chaining is useful here? We don't expose the Gateway (yet), do you think we should? I am not sure users will create multiple gateways similar to multiple routers.

@srikrsna-buf I guess I'm thinking as a wholistic API, if both connect-router and knit have a similar API mechanism, it's just one less thing to learn. I guess I would prefer that if we're saying we want the APIs to be similar to connect-router that it actually makes sense for that to be the case.


RuntimeError: [../third_party/gloo/gloo/transport/tcp/pair.cc:534] Connection closed by peer

Issue: Training Process Halts with Errors on Applio
Environment Specifications:

Operating System:  Debian GNU/Linux 12 (bookworm) x86_64  6.1.0-9-amd64
Python Version: 3.9.18
GPU: Dual NVIDIA GeForce RTX 4090s
Python Environment: Conda virtual environment specifically for Applio

Steps to Reproduce:

Installed Applio using the official installation script.
Went into the Applio's site-packages directory and manually updated PyTorch from version 2.0.0 to 2.1.0. This was necessary to support CUDA version 11.8, which is required by my graphics cards.
Launched Applio and uploaded my dataset to the specified folder.
Navigated to the training tab, performed preprocessing, and then feature extraction.
Started the training process.

Issue Encountered:
During training, I encountered a series of errors, causing the training process to halt. The detailed error log is as follows:
Process Process-2:
Traceback (most recent call last):
  File "/home/hkyouma/ai/voicegen/Applio-RVC-Fork/lib/infer/modules/train/train.py", line 280, in run
    utils.latest_checkpoint_path(hps.model_dir, "D_*.pth"), net_d, optim_d
  File "/home/hkyouma/ai/voicegen/Applio-RVC-Fork/lib/infer/infer_libs/train/utils.py", line 212, in latest_checkpoint_path
    x = f_list[-1]
IndexError: list index out of range

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/hkyouma/miniconda3/envs/voicegen/lib/python3.9/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/home/hkyouma/miniconda3/envs/voicegen/lib/python3.9/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/home/hkyouma/ai/voicegen/Applio-RVC-Fork/lib/infer/modules/train/train.py", line 317, in run
    logger.info(
UnboundLocalError: local variable 'logger' referenced before assignment
INFO:modeltraining:Loaded pretrained assets/pretrained_v2/f0G48k.pth
INFO:modeltraining:<All keys matched successfully>
INFO:modeltraining:Loaded pretrained assets/pretrained_v2/f0D48k.pth
INFO:modeltraining:<All keys matched successfully>
/home/hkyouma/ai/voicegen/Applio-RVC-Fork/.venv/lib/python3.9/site-packages/torch/functional.py:650: UserWarning: stft with return_complex=False is deprecated. In a future pytorch release, stft will return complex tensors for all inputs, and return_complex=False will raise an error.
Note: you can still call torch.view_as_real on the complex output to recover the old return format. (Triggered internally at ../aten/src/ATen/native/SpectralOps.cpp:863.)
  return _VF.stft(input, n_fft, hop_length, win_length, window,  # type: ignore[attr-defined]
/home/hkyouma/ai/voicegen/Applio-RVC-Fork/.venv/lib/python3.9/site-packages/torch/functional.py:650: UserWarning: stft with return_complex=False is deprecated. In a future pytorch release, stft will return complex tensors for all inputs, and return_complex=False will raise an error.
Note: you can still call torch.view_as_real on the complex output to recover the old return format. (Triggered internally at ../aten/src/ATen/native/SpectralOps.cpp:863.)
  return _VF.stft(input, n_fft, hop_length, win_length, window,  # type: ignore[attr-defined]
/home/hkyouma/ai/voicegen/Applio-RVC-Fork/.venv/lib/python3.9/site-packages/torch/functional.py:650: UserWarning: stft with return_complex=False is deprecated. In a future pytorch release, stft will return complex tensors for all inputs, and return_complex=False will raise an error.
Note: you can still call torch.view_as_real on the complex output to recover the old return format. (Triggered internally at ../aten/src/ATen/native/SpectralOps.cpp:863.)
  return _VF.stft(input, n_fft, hop_length, win_length, window,  # type: ignore[attr-defined]
/home/hkyouma/ai/voicegen/Applio-RVC-Fork/.venv/lib/python3.9/site-packages/torch/functional.py:650: UserWarning: stft with return_complex=False is deprecated. In a future pytorch release, stft will return complex tensors for all inputs, and return_complex=False will raise an error.
Note: you can still call torch.view_as_real on the complex output to recover the old return format. (Triggered internally at ../aten/src/ATen/native/SpectralOps.cpp:863.)
  return _VF.stft(input, n_fft, hop_length, win_length, window,  # type: ignore[attr-defined]
/home/hkyouma/ai/voicegen/Applio-RVC-Fork/.venv/lib/python3.9/site-packages/torch/functional.py:650: UserWarning: stft with return_complex=False is deprecated. In a future pytorch release, stft will return complex tensors for all inputs, and return_complex=False will raise an error.
Note: you can still call torch.view_as_real on the complex output to recover the old return format. (Triggered internally at ../aten/src/ATen/native/SpectralOps.cpp:863.)
  return _VF.stft(input, n_fft, hop_length, win_length, window,  # type: ignore[attr-defined]
Process Process-1:
Traceback (most recent call last):
  File "/home/hkyouma/miniconda3/envs/voicegen/lib/python3.9/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/home/hkyouma/miniconda3/envs/voicegen/lib/python3.9/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/home/hkyouma/ai/voicegen/Applio-RVC-Fork/lib/infer/modules/train/train.py", line 359, in run
    train_and_evaluate(
  File "/home/hkyouma/ai/voicegen/Applio-RVC-Fork/lib/infer/modules/train/train.py", line 572, in train_and_evaluate
    scaler.scale(loss_disc).backward()
  File "/home/hkyouma/ai/voicegen/Applio-RVC-Fork/.venv/lib/python3.9/site-packages/torch/_tensor.py", line 492, in backward
    torch.autograd.backward(
  File "/home/hkyouma/ai/voicegen/Applio-RVC-Fork/.venv/lib/python3.9/site-packages/torch/autograd/__init__.py", line 251, in backward
    Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: [../third_party/gloo/gloo/transport/tcp/pair.cc:534] Connection closed by peer [<IP_ADDRESS>]:53406

Observations and Concerns:

The IndexError in train.py suggests an issue with the checkpoint loading process.
An UnboundLocalError followed, hinting at a problem in the exception handling mechanism.
The application is using deprecated features of PyTorch (stft with return_complex=False), raising concerns about compatibility with newer versions of PyTorch.
A RuntimeError related to Gloo indicates a potential issue in the network communication setup.

I suspect these issues might be related to the PyTorch version upgrade I performed. However, this upgrade was essential for compatibility with my hardware setup.
Any assistance or guidance on resolving these errors would be greatly appreciated.

I'd appreciate any input on the issue whatsoever.

Perhaps an important addition. This error does not appear during inference. In fact, inference goes just fine.

don't use 3.9 on linux. use 3.11 it says it in the readme

I'm getting this issue as well, it ONLY happens when I have both GPUs selected, otherwise the issue disappears


Added CORS headers

Added CORS headers to allow frontend to access server.
Jira Ticket: https://seng499-b01-company2.atlassian.net/browse/C2-62

I just noticed that this middleware method is commented out in main.go. Is there a reason for this? @vibhubehl

@jberryman566 no that commented-out line can be removed. We are already calling the middleware. I forgot to remove it.


Azure SDK for python behind a corporate proxy

Type of issue
Code doesn't work
Description
Hello,
We are using a proxy and we intercept login.microsoftonline.com to verify which tenants users are going. However here when we intercept this request user get's error that ssl certificate is invalid. The proxy certificate is added to trusted root certificate so it works correctly for browser but not from Visual Studio. We tried whitelisting for ssl certificate bypass login.microsoftonline.com/tenantID but it fails, it only works when bypassing the whole login.microsoftonline.com which is not a good solution as we lose the tenants visibility in proxy.
Can you share how to use this code in VS behind a proxy ?
Cheers,
Arthur
Page URL
https://learn.microsoft.com/en-gb/python/api/azure-identity/azure.identity.clientsecretcredential?view=azure-python
Content source URL
https://github.com/MicrosoftDocs/azure-docs-sdk-python/blob/main/docs-ref-autogen/azure-identity/azure.identity.ClientSecretCredential.yml
Document Version Independent Id
03ee17be-6659-76b6-bcc1-44b2cb64b094
Article author
@lmazuel
Metadata

ID: 71fdf00e-23cc-b602-03c7-072d908d1f44

Thanks for reaching out.
Could you help give more details about:
"it works correctly for browser but not from Visual Studio"?

Hello,
Apology for delayed response.
Normally in company we test connection in browser and in application. Here the user in VS is trying to obtain secret password using your azure sdk for python script. However there can be a situation that our policy blocks the connection due to specifing rule in place. This is why if possible we test also in browser and verify the logs. In browser we don't use azure sdk we just go to vault and check if connection is proper and we can verify the connectivity to the website.
However when using azure sdk for python user issues a connect request to login.microsoftonline.com which we ssl intercept on proxy and here it fails. This is probably request for azure environment. We use login.microsoftonline.com for proxy authentication and all O365 apps and it works correctly but here there is something maybe in the method of the request that it fails. In logs we only see the connection through VS is there, it's allowed but size in bytes is different.
The problem is we don't know why your azure sdk fails with login.microsoftonline.com through proxy. Is there a method to bypass the proxy, point to proxy (so we are sure that whole traffic always hits proxy), is there a token generated through your request that proxy strips, is your code validating if there is men in middle attack (proxy is a main in the middle device) and blocks the sending of some token etc.
In short.... how we can we approach your script when user is behind corporate proxy.
Thank you and happy holidays.

Happy new year!
Thanks for sharing the information.
You can specify the proxy information when creating the credential object.
e.g.:
cred = ClientSecretCredential(proxies={'http': 'http://<IP_ADDRESS>:3148'})
please let us know if it does not work.

Hello,
We tried this but it did not work.
Our proxy lurks on the network level, you might say it's a bit of a
transparent type of proxy, it steals the traffic the user is generating on
port 80 + 443 and pushes it to the proxy.
We can see the traffic in wireshark(tool for looking at packets)  and we
can that proxy got the traffic but the part for login.microsoftonline.com
fails. It's only when we stop decrypting the traffic for this domain for
your app when it starts to work.
Normally this can happen when there is some token hidden tricks or some
specific form of authentication and both of them don't like men in the
middle to look at what they are doing or when some secure features are
enable and when men in the middle is noticed they trigger and break the
connection.
Cheers.
Artur
wt., 2 sty 2024 o 18:32 Xiang Yan @.***> napisał(a):

Happy new year!
Thanks for sharing the information.
You can specify the proxy information when creating the credential object.
e.g.:
cred = ClientSecretCredential(proxies={'http': 'http://<IP_ADDRESS>:3148'})
please let us know if it does not work.
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-python/issues/33607#issuecomment-1874330393,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMEHNFUSBTETHVIOSOGLWJ3YMRACZAVCNFSM6AAAAABAZNLBQCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZUGMZTAMZZGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Pozdrawiam,
Artur

Adding @rayluo for more insights.

I don't really have a specific suggestion here.

On the one hand, Azure SDK's underlying auth library - MSAL - supports a proxies parameter. You can probably try it by this one-liner:
python -c "import msal; print(msal.ConfidentialClientApplication('your_client_id', client_credential='your_secret').acquire_token_for_client(['your_scope']))"

You can try this to see whether it works in your proxy environment.

On the other hand, that parameter in MSAL only works when using the default http client which is based on requests. But, AFAIK, Azure SDK uses its own http client, so, the test above may not mean anything in terms of the Azure SDK's behavior.

@ArturSyl
Could you help to try the code Ray provided:
python -c "import msal; print(msal.ConfidentialClientApplication('your_client_id', client_credential='your_secret').acquire_token_for_client(['your_scope']))"
and let us know if it works?

Hey Xiang,
I will send the code to the user, will update the setup tomorrow if he has
time or next week and I will get back to you guys. Thank you nevertheless
for swift replies on the topic.
Cheers,
Artur
czw., 4 sty 2024 o 17:10 Xiang Yan @.***> napisał(a):

@ArturSyl https://github.com/ArturSyl
Could you help to try the code Ray provided:
python -c "import msal;
print(msal.ConfidentialClientApplication('your_client_id',
client_credential='your_secret').acquire_token_for_client(['your_scope']))"
and let us know if it works?
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-python/issues/33607#issuecomment-1877361601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMEHNFWC3UFMVXX7FYJLZU3YM3H7DAVCNFSM6AAAAABAZNLBQCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZXGM3DCNRQGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Pozdrawiam,
Artur

Hey Guys,
I think we can close it for now. We found a workaround, not great but
working. We disabled ssl interception for login.microsoftonline for the
devs working with azure sdk.
czw., 18 sty 2024 o 22:34 github-actions[bot] @.***>
napisał(a):

Hi @ArturSyl https://github.com/ArturSyl, we're sending this friendly
reminder because we haven't heard back from you in 7 days. We need more
information about this issue to help address it. Please be sure to give us
your input. If we don't hear back from you within 14 days of this
comment the issue will be automatically closed. Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-python/issues/33607#issuecomment-1899237540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMEHNFXU4PV2TNC6ZT52N3TYPGIOHAVCNFSM6AAAAABAZNLBQCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJZGIZTONJUGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Pozdrawiam,
Artur


Can't refund subscription

Hi. Thanks for making such a great product, but I haven't used it in a while as there is no need for it, I forgot to cancel my subscription and was charged this morning. Is there any way to get those funds back?

Hey @diassique,
Thanks for using AskTheCode and sorry for the long reply!
Sure, I can process the refund for you. Please reach out to me via email at<EMAIL_ADDRESS>with the details about your account, and I will process it for you.


ci(gha): fix missing tag in kubectl binary

Add fetch-depth to fetch all history for tags.

How can we test it before approving ?

Umm... you could try on your own fork. You can take this change, and create a branch, and create a github release with a new tag.

How can we test it before approving ?

Proof: https://github.com/tiagolobocastro/mayastor-extensions/actions/runs/5273295855

nit: maybe move this new 4th step right next to the first step, because they are of the same context.

Indeed, done!

bors merge


Body of custom route request is empty

Here is my code for setting up the route and the request itself. However, on the receiving end, it reads the userID argument, but says the body of the post is empty.
wp.rcpUp = wp.registerRoute( 'rcp/v1/members', 'update/(?P<id>\\d+)' );
rcpUpdateData = { 				status: "active", 				subscription: 5, 				recurring: 1, 				expiration: expDate, 			}
wp.rcpUp().id(user.userID).update(rcpUpdateData)
EDIT: Is there a way I could view the request without sending it to see all the pieces of the original request, so I can determine why the body is empty?

bump

@theturtleroom Thanks for the bump. Regarding your example, a few questions to double-check my understanding of what you are asking --

Having update in the REST route is a little unusual, so just to check first off: you expect a PUT to rcp/v1/members/update/<id> will succeed? Does it work correctly with curl if you pass a json payload?
I presume this record already exists, via creation on the WP side or from a prior .create()? If the latter, does .create work fine?

This record does exists. The routes were designed by the makers of the plugin, and yes, update is in the route. It does succeed; I receive the positive response from the WP server. However, when I view the body from the WP end after it received the request, it says the body is empty.
I cannot get a .create() to work, either. I get the same positive response, but the body appears to be empty.
I am very new with doing HTTP requests...how would I craft a request through curl with a json payload?

So, thanks....that was it The API was using get_body_params() instead of get_json_params(). Now I can get the full body to do what it is supposed to!.

OK! @theturtleroom Does that mean you're sorted, now? Apologies not to provide a more specific curl example, but hope that it did end up helping you get to the answer!

Yes! It is working now!

Glad to hear it, thanks for following up!


[Eligibility Questionnaire] Add filtering functionality

[ ] Decide how to store state and implement
[ ] Determine whether we have the necessary fields in the backend to filter on the question answers
[ ] Get filtering working

Finishing this is also blocked on getting information on how we should compute eligibility. I can start working on certain aspects of it.

Can we downscope this ticket to what we can finish this sprint and open another ticket for any blocked work?


Type definitions files are not shipped with v1

Hi @tannerlinsley,
First of all, thank you so much for your awesome job! Version 1 is a huge improvement, but I am now having a little issue: my TS project is complaining about missing type definitions files for react-query. Was this intentional?
Cheers.

Type definitions are available on DefinitelyTyped.

Type definitions are available on DefinitelyTyped.

Yes, but they are not updated for version 1.

If that’s the case it’s probably easiest to create an issue on DefinitelyTyped or make a PR yourself if a lack of typings are a blocker for you.

I am asking if the choice of removing type definitions from the lib was intentional. If this is the case, then I will open an issue on DefinitelyTyped ;)

Type definitions were removed because they were outdated - see https://github.com/tannerlinsley/react-query/commit/d06353a6146facc608eb73a4ef62ffb2944b6cb5

Ok, so we have to manually add @types/react-query as a project dependency because it was intentionally removed from the lib's dependencies, but we cannot do it unless @types/react-query gets updated for react-query v1.

I am closing this, see this issue on DefinitelyTyped.

Yes it was intentional. They were severely out of sync with v1, so it made very little sense to keep them as a dependency. If and when the types can catch up, I’ll add it back.
On Feb 27, 2020, 1:50 AM -0700, Dennis Morello<EMAIL_ADDRESS>wrote:

I am asking if the choice of removing type definitions from the lib was intentional. If this is the case, then I will open an issue on DefinitelyTyped ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

FYI - https://github.com/tannerlinsley/react-query/discussions/362


chore: add oauth2 proxy provider backend module

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

I'll probably have time after wellness weak & BackstageCon to pick this up further. Seems generating the api reports is error'ing on packages that weren't changed.


Implement actionsBlacklist like in RN debugger

For some time, people are still likely to need to use RN debugger together with flipper.
This PR implements the actionsBlacklist middleware debugging option (named the same as RN debugger https://github.com/jhen0409/react-native-debugger/blob/2ea070367a39d065be737a060fddf479326db054/app/worker/reduxAPI.js#L231) to keep things consistent for developers.
As implemented, any substring in the blacklist that is matched by the action.type will not be sent to the flipper plugin.

Hi @andrewzey , thanks for this. Can you add the config with some explainations in the README?

@jk-gan Of course, sorry about the omission! I've updated the README, please have a look.


Add a CHANGELOG to this repo

Add a changelog that follows this format

add the date this repo was created as the first entry

I can do this for you if you like? 😄


Move SortExec partition check to constructor

Note: migrated from original JIRA: https://issues.apache.org/jira/browse/ARROW-11625
SortExec has the following error check at execution time and this could be moved into the try_new constructor so the error check happens at planning time instead.
{code:java}
if 1 != self.input.output_partitioning().partition_count() {
return Err(DataFusionError::Internal(
"SortExec requires a single input partition".to_owned(),
));
} {code}

Comment from Hendrik Makait(hendrik.makait) @ 2021-02-14T17:46:12.067+0000:
I'd love to check that out tomorrow.
Comment from Hendrik Makait(hendrik.makait) @ 2021-02-15T20:05:17.782+0000:
Moving this check into the constructor leads to failing tests. As far as I can see, this is because the planner does an optimization step that inserts a merge for children that contain multiple partitions.
{code:java}
match plan.required_child_distribution() {
    Distribution::UnspecifiedDistribution => plan.with_new_children(children),
    Distribution::SinglePartition => plan.with_new_children(
        children
            .iter()
            .map(|child| {
                if child.output_partitioning().partition_count() == 1 {
                    child.clone()
                } else {
                    Arc::new(MergeExec::new(child.clone()))
                }
            })
            .collect(),
    ),
}
{code}
 What's the reason for moving this check into planning time? How should I proceed?
Comment from Andy Grove(andygrove) @ 2021-02-16T14:44:11.773+0000:
I see. Ok, maybe we can't do this at planning time then.


Fix diagnostics message for conditional conformance with marker protocol

Current:
protocol P { }
struct S<T> {}
extension S: P where T: Sendable {}
// Conditional conformance to non-marker protocol 'P' cannot depend on conformance of 'T' to non-marker protocol 'Sendable'

Second non-marker is typo of marker.
It should be:
Conditional conformance to non-marker protocol 'P' cannot depend on conformance of 'T' to marker protocol 'Sendable'

I fixed diagnostics message template.

@xedin I updated existing test case for this diagnostics.

@swift-ci please test


Interrupting training crashes Python kernel when training on GPU for the first time with MirroredStrategy

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
TensorFlow installed from (source or binary): binary (pip install tensorflow)
TensorFlow version (use command below): 2.4.3
Python version: 3.7
GPU model and memory: Nvidia Tesla V100

Describe the problem.
When using multi-GPU training in a Jupyter Notebook:
strategy = tf.distribute.MirroredStrategy()
model_ = ...
with strategy.scope():
    model = model_.compile(loss='categorical_crossentropy', optimizer='adam', metrics=["accuracy"])

Then launching training:
model.fit(...)

Describe the current behavior.

when initial training runs to the end, subsequent trainings can be interrupted fine
when initial training itself is interrupted, the Jupyter kernel dies

Detailed step-by-step procedure:

compile the model with mirrored strategy
launch training with model.fit()
let the training go to its own end -> otherwise kernel dies if interrupted here
after initial training completed, re-compile the model with mirrored strategy
launch another training
interrupt this training -> works fine, kernel doesn't die

If training was interrupted at step 3. then the kernel dies systematically.

Describe the expected behavior.

when initial training is interrupted, the Jupyter kernel should not die

Source code / logs.
No error message — the Jupyter iPython kernel simply dies with no message.
Current workaround.
The current workaround I found is to run the first training with just 1 epoch, let it complete, then I can recompile the model and re-train at will with no crashes. Less than ideal, but it works.

@JivanRoquet Could you please try with the latest stable TF v2.7.0 and let us know if it helps ? Thank you!

@sushreebarsa I did the following:
pip install tensorflow --upgrade
pip install tensorflow-gpu --upgrade

Now I have TensorFlow v2.7.0 but when running model.fit() I have the following error repeatedly:
UnknownError: 5 root error(s) found.
  (0) UNKNOWN:  Failed to get convolution algorithm. This is probably because cuDNN failed to initialize, so try looking to see if a warning log message was printed above.
	 [[node model_transformer/conv_block_1/conv1d_3/Conv1D
 (defined at /home/ubuntu/anaconda3/envs/tensorflow2_latest_p37/lib/python3.7/site-packages/keras/layers/convolutional.py:238)
]]

I didn't change anything apart from upgrading TensorFlow. What do I need to do in this case?

@JivanRoquet Quick question to find whether the root-cause is keras or distribution strategy. I guess distribution strategy.
Are you using Keras or TF model?
Did you try without Mirrored strategy? Do you notice this issues without distribution strategy.
If you think this is more related to distribution strategy, please open this issue in TF repository https://github.com/tensorflow/tensorflow/issues as this keras repo is mainly for keras related issues. Thanks!

I guess this would be the distribution strategy then because effectively, the problem doesn't appear when not using the mirrored strategy. I'm using a Keras model however (function API), so perhaps the problem might be the result of a combination of the two.

@JivanRoquet Thanks for the quick response. This is more related to distribution strategy. Can you please open the issue in TF repository.  Thanks!

Sure thing. Closing here for now.


v1.9: Update Go to 1.15.14

test-backport-1.9

test-1.13-4.9

k8s-1.12-kernel-netnext failed on #13853 https://jenkins.cilium.io/job/Cilium-PR-K8s-1.12-net-next/1771/
k8s-1.17-kernel-5.4 failed on #16050 https://jenkins.cilium.io/job/Cilium-PR-K8s-1.17-kernel-5.4/69/

test-runtime


Build gnuplot from sources

The Dockerfile for opentsdb includes the gnuplot package from the
Alpine distribution which is built without terminal png support. This
affects gnuplot to be effectively used in the opentsdb web UI.
The current commit adds the required runtime and build dependencies to
build gnuplot from sources which should fix
PeterGrace/opentsdb-docker#24

very good ! it works

Thanks. It works for me.

Hi !
It works for me!
but It has error :(
I access http://localhost:4242 and POST data http://localhost:4242api/put.
I get response code 204.

http://opentsdb.net/docs/build/html/api_http/put.html
By default, the put endpoint will respond with a 204 HTTP status code and no content if all data points were stored successfully.

However It doesn't show gnuplot graph.

When will this be merged ? This commit worked for me ...


stream__val value for hipStreamCreate* gives invalid values

The arguments for these callbacks include .stream and .stream__val. Based on /opt/rocm-5.2.0/include/hip/hcc_detail/hip_prof_str.h .stream__val should be the value when dereferencing .stream in the EXIT phase. Though I get different values between *.stream and .stream__val for all hipStreamCreate* functions.
An example:
the callback sees the following values in the EXIT phase:

.stream = 0x7ffc6046c4a0
*.stream = 0x2379830
stream__val = 0x2375cb0

The caller of hipStreamCreate sees this value: 0x2379830
Can someone sched some light on the difference. Thanks.

@bertwesarg Apologies for the lack of response.  Can you please check you still see this issue with the latest ROCm 6.2? If not, please close the ticket. Thanks!

@all As roctracer will be replaced with rocprofiler SDK, I don't see a reason to continue here, do you?

@bertwesarg , feel free to close this issue.


⚠ Bofrost.be has degraded performance

In ce886f4, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2056 ms

Resolved: Bofrost.be performance has improved in ba0029b.


PAT creation default to no access

This also reverses the list so that All Access is last.

Coverage remained the same at 72.533% when pulling 16cba87b1bbdad914e30726685f6e0a6c2ce2734 on eatonphil:pat-creation-default-no-access into 83504c48049c21a7aa9ab91db643acc557fbb270 on linode:develop.


Fix typing errors and add additional explanation about the fork/join notation in PBFT activity diagram

Fix typing errors and add additional explanation about the fork/join notation in PBFT activity diagram

PROJECT-IMPLEMENTATION-DETAILS.md에 PBFT consensus를 위한 proposed architecture를 추가하였습니다. 구현에 참고하세요.


[ondemandkorea] PermissionError: [Errno 13] Permission denied

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.05.09. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.05.09

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v http://www.ondemandkorea.com/im-not-a-robot-e3-1.html
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'http://www.ondemandkorea.com/im-not-a-robot-e3-1.html']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.05.09
[debug] Python version 3.6.5 (CPython) - Linux-4.16.8-1-ARCH-x86_64-with-arch
[debug] exe versions: ffmpeg 4.0, ffprobe 4.0
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (CA) as X-Forwarded-For.
[OnDemandKorea] im-not-a-robot-e3-1: Downloading webpage
[OnDemandKorea] im-not-a-robot-e3-1: Downloading m3u8 information
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'https://aws3.ondemandkorea.com/v1/201712/drama/im-not-a-robot/im-not-a-robot-e3-1/p720_b2208000_v2071000_a129000_chunk.m3u8'
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 189
[download] Destination: 로봇이 아니야  - 03회, I'm Not a Robot  - E03-im-not-a-robot-e3-1.mp4
ERROR: unable to download video data: [Errno 13] Permission denied: "로봇이 아니야  - 03회, I'm Not a Robot  - E03-im-not-a-robot-e3-1.mp4.ytdl"
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/youtube_dl/YoutubeDL.py", line 1908, in process_info
    success = dl(filename, info_dict)
  File "/usr/lib/python3.6/site-packages/youtube_dl/YoutubeDL.py", line 1847, in dl
    return fd.download(name, info)
  File "/usr/lib/python3.6/site-packages/youtube_dl/downloader/common.py", line 365, in download
    return self.real_download(filename, info_dict)
  File "/usr/lib/python3.6/site-packages/youtube_dl/downloader/hls.py", line 105, in real_download
    self._prepare_and_start_frag_download(ctx)
  File "/usr/lib/python3.6/site-packages/youtube_dl/downloader/fragment.py", line 69, in _prepare_and_start_frag_download
    self._prepare_frag_download(ctx)
  File "/usr/lib/python3.6/site-packages/youtube_dl/downloader/fragment.py", line 179, in _prepare_frag_download
    self._write_ytdl_file(ctx)
  File "/usr/lib/python3.6/site-packages/youtube_dl/downloader/fragment.py", line 87, in _write_ytdl_file
    frag_index_stream, _ = sanitize_open(self.ytdl_filename(ctx['filename']), 'w')
  File "/usr/lib/python3.6/site-packages/youtube_dl/utils.py", line 457, in sanitize_open
    stream = open(encodeFilename(filename), open_mode)
PermissionError: [Errno 13] Permission denied: "로봇이 아니야  - 03회, I'm Not a Robot  - E03-im-not-a-robot-e3-1.mp4.ytdl"
...
<end of log>

Description of your issue, suggested solution and other information
When attempting to play any videos from OnDemandKorea.com, youtube-dl returns an error as shown above. I am using macOS 10.13.4.

As clearly stated in the error message you must have write access to the directory the video is being downloaded to.


Uncaught RangeError: Offset is outside the bounds of the DataView

Hey, I'm seeing this error since adding a createdBy property to my state. If I remove that property it goes away. Any ideas what that's about?
Uncaught RangeError: Offset is outside the bounds of the DataView
    at DataView.getUint32 (<anonymous>)
    at Reader.readUInt32 (:3001/node_modules/.vite/bin-serde.js?v=c1087c99:271:31)
    at :3001/@fs/home/kaspar/projects/calm-go/code/api/types.ts:374:22
    at Array.map (<anonymous>)
    at decodeStateUpdate (:3001/@fs/home/kaspar/projects/calm-go/code/api/types.ts:373:50)
    at WebSocket.HathoraConnection.socket.onmessage (:3001/@fs/home/kaspar/projects/calm-go/code/client/.hathora/client.ts:63:63)

I ran your add-created-by branch locally and I noticed that if I join as a second player there are two websocket connections to hathora established at the same time:

I think this is ultimately causing issues (it's interfering with the hathora's client logic which assumes the very first message it receives is a snapshot, and the rest are updates). I would check your connection logic and make sure you only make a single connection for a given stateId per browser tab

Ok yes, thanks, making sure to just have one connection seems to have fixed it.


Add watch or live reload mode.

exprexo could easily benefit from a watch mode where the server will either reload (the easiest for now) or the routes cache is refreshed.
Reasoning
When in development mode it's quite common to update any .json or .js route and the desired outcome is most likely to be able to see that change when calling the route again. As exprexo is using require under the hood those changes are not reflected unless exprexo is shutdown and started again.
Proposal
Add either nodemon or a similar, maybe not as "heavy", solution to implement a watch mode feature.
Maybe under a new set of flags: -w and --watch (check UNIX style for this) http://www.catb.org/~esr/writings/taoup/html/ch10s05.html
Meanwhile
There is a short term solution for anyone needing this. exprexo can be run with nodemon watching for the routes folder or any other that the user might need 👇
npx nodemon --exec exprexo <options>
or double npx 😅 (not really fast if both are not installed)
npx nodemon --exec 'npx exprexo'
Additional notes
At the moment of writing nodemon is a dependency, probably because this feature was already thought to be implemented with nodemon. If nodemon is not the final module to use it should be moved to devDepencencies.

Just added http://www.catb.org/~esr/writings/taoup/html/ch10s05.html as reference.


Route handle failed to plan route from road sholder

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
The route handler failed to plan route from the road shoulder as in the situation below. The cause is the failure of the planPathLaneletsBetweenCheckpoints function. It fails to get start lanelets from getCurrentLanelets since it does not support the road shoulder.

Expected behavior
Route should be calculated when the ego vehicle is on the road shoulder.
Actual behavior
Route is not calculated when the ego vehicle is on the road shoulder.
Steps to reproduce

run psim
set ego pose on the road shoulder
set goal pose on any lanelets.

Versions
No response
Possible causes
No response
Additional context
No response

This does not occur in the main branch.


Gpl 426 many materials

Changes proposed in this pull request:

Allow a container to have many materials

I suspect this needs some careful integration with the tubes being where they are now


move custom class into L.esri.Cluster.featureLayer

L.esri.Cluster.clusteredFeatureLayer becomes L.esri.Cluster.featureLayer
migrate from babelify to rollup for building source

rename built output from esri-leaflet-clustered-feature-layer.js to esri-leaflet-cluster.js

stop hardcoding version number in raw source
bump internal dependencies
make sure npm start triggers local dev setup
removed some old commented code
removed version number from bower.json

as long as we're trimming the fat, i vote we rename the repo 'esri-leaflet-cluster' too.

if i don't hear from any @Esri/leaflet-contributors (w/ questions, concerns etc.) by tomorrow, i'll assume its safe to merge this myself.


"Your version of Windows is out of date" error message

Prerequisites

[X] Verify that this is not a Windows issue;
[X] Refer to the System requirements;
[X] I do not use a homebrew Windows image;
[X] I did not tweak Windows before that could cause system instability;
[X] Attach a created log. You may find it in the Logs folder.

Steps To Reproduce

Start SophiApp 1.0.97

Expected behavior
SophiApp menus should be functional
Actual behaviour

I get the attached error "Your version of Windows is out of date."  Obviously, I have a much higher version (26120) so am not clear on how to resolve.   Thanks
Windows Version
Windows 11 Pro, version 24H2, OS Build 26120.461
SophiApp version
1.0.97
SophiApp release type
SophiApp
Logs
Windows 10 Pro 24H2 build 26120.461
Computer name: OCTOPUS
User: brian
User domain: OCTOPUS
User culture: English (United States)
User region: United States
App version: <IP_ADDRESS>
App is release: True
App folder: "C:\opt\SophiApp"
App localization: EN
App theme: DARK
08.05.2024 16:03:26 Debug mode is: False
08.05.2024 16:03:26 Active view is: Loading
08.05.2024 16:03:26 Advanced settings is visible: False
08.05.2024 16:03:26 The "UWP for all users" switch state is: UNCHECKED
08.05.2024 16:03:26 The OS conditions check started
08.05.2024 16:03:26 OsVersionCondition run result: True
08.05.2024 16:03:26 The next condition to be run: OsBuildVersionCondition
08.05.2024 16:03:26 OsBuildVersionCondition run result: False
08.05.2024 16:03:26 The next condition to be run: OsFilesCorruptedCondition
08.05.2024 16:03:26 Active view is: OsBuildVersion
08.05.2024 16:03:27 It took 0 second(s) to check the OS conditions

Hi. The SophiApp 1.x development has been stopped in order to create 2.0 version. This bug will be fixed automatically with a 2.0 release.
https://github.com/Sophia-Community/SophiApp?tab=readme-ov-file#sophiapp-20

Dmitry, Thanks for your prompt response. That's fine.

I'd definitely use Sophia Script once the app will be released and tested. If everything is fine, we will complete the first public build somewhere in September. :)

Yes, will be doing that. Thanks


Translate 3 fortran files

Translates

[x] mo_reorder_kernels.jl
[x] mo_fluxes_broadband_kernels.jl
[x] mo_optical_props_kernels.jl

Looks good to me!


how to apply addImageBlobHook in Vue.js??

Summary
Hello. I'm trying to apply addImageBlobHook, a function that fires when an image is uploaded.
When uploading an image to the editor, I am trying to output the editor by receiving the url of the photo as a return value as a return value as a callback function without saving base64-encoded data.
I searched for a way to remove the default function using removeHook and set it again, but I have no idea how to apply it. Any help would be appreciated..
Screenshots
<editor     :options="editorOptions"     minHeight="400px"     initialEditType="wysiwyg"     ref="toastuiEditor"   />
`import<EMAIL_ADDRESS>import 'tui-color-picker/dist/tui-color-picker.css';
import<EMAIL_ADDRESS>import colorSyntax from '@toast-ui/editor-plugin-color-syntax';
import { Editor } from '@toast-ui/vue-editor';`
It was imported as above, and the data is written as follows.

When declaring the callback function, when I tried to call a method using this, an error appears in the console window as well.
Is it not using this syntax.. or how to apply the code below in Vue.js..

Version
"@toast-ui/editor": "^3.1.2",
"@toast-ui/vue-editor": "^3.1.2",

@rkddbs1031
It works normally when done as below. Can you check to pass props options?
If it doesn't work, please upload your code in code sandbox to resolve the issue.
new Vue({
  data() {
    return {
      initialValue: content,
      options: {
        hooks: {
          addImageBlobHook(blob, callback) {
            // do something
            callback('image url', 'alt text');
          },
        },
      },
    };
  },
  template: '<editor height="400px" :options="options" />',
}).$mount('#editor');

here is my solution:

instead of using the @toast-ui/vue-editor, i use the @toast-ui/editor, because it provide the consturtor function

HTML:

Script:

then you could create an initEditor  funtion and use it in the mount life cycle

holpfully can help you!


sqlite3  API open

I'm a plugin developer,At first I wanted to find globalState store location and found it use sqlte3 when I looked at the source code.
i want to use sqlit3 store some information,However, due to different platforms, it can not be very compatible, so I want to use sqlite.note provided by vscode, because it has achieved platform compatibility

we hope you can open sqlite3 api in the feature,thx.

sqlite3 is an internal detail to store our state and we provide an API to manage the extension state. So in our opinion it does not makes sense to expose such an API as it is not inline with our API.


Easier shape construction

Construction syntax for partially static shapes is IMO really cumbersome - the axis placeholder syntax is cool and useful certainly for some situations, in others (most ones in our use cases) however it would be easier if we could have the 'old school less cool eigen' syntax ;D +i'd expect this syntax to be faster to compile ;D
i.e. so that one can simply pass values to the constructor using plain C syntax - and the library will simply assert that the values passed for fixed/static axis match the static values...
A companion version that takes only as many arguments as there are dynamic dimensions (and assigns them in order) would also be useful...

i.e. so that one can simply pass values to the constructor using plain C syntax - and the library will simply assert that the values passed for fixed/static axis match the static values...

I am not a fan of that.

I thought we agreed already it need to change? I don't like the axis ctor after all.

Don't remember that discussion...
I like the axis syntax - as nerd porn for rubbing it in back to the Python ML engineers 😁
...just that in 'real life' it requires serious/verbose gimnastics (which also certainly aren't CT cheap)...

The accept the exact # of dynamic size should be trivial.

👍

What is the eigen way ?

the thing you aren't a fan of 😁

axis is just a funky object that do some stati_assert in a ... so not costly.
I'll made amendment for the shape{int...} constructor to take static_size in consideration it will already helps

axis is just a funky object that do some stati_assert in a ... so not costly.

The cost is not in the axis object itself but in what it requires 'around' itself (e.g. case in point is exactly the of_shape implementation)

OK, what if we do it the other way around?
shape<extent()[4]()> x(3, _, 7);

Still has the issue of trying to construct the type in a generic context :/

OK, what if we do it the other way around?
shape<extent()[4]()> x(3, _, 7);

this is cool too :)
i'd like the version where you could also simply write 3, 4 ,7...but i can first try with this...

I have to convince myself the full blown own is easier.
Maybe we can make it so if you pass a value, it is checked against the static ones and _ is a joker of some sort
Thus it'll fit both use cases

nice 👍

https://github.com/jfalcou/kiwaku/pull/36

Merged


Google Play Android Developer API return 404

Scenario:

Purchase sku1 (token1)
At this point you are still able to  get the purchase info through the api with token1

Purchase sku1 again (token2)
Now you are only able to check the second purchase with token2. But when you try to check the first one you get "The purchase token was not found." 404 error.

It looks like when the user purchases the same sku multiple times, all purchase tokens except the last one is invalidated/destroyed/removed.
If I purchase different product sku2 ( token3) then token3 and token 2 are both valid.

This repository is for the library, not service issues. Please make a question on Stack Overflow with the tag google-play.


ST3: Windows, Linux - not working

Autocomplete - just silence.
Goto definition - error: "/path/to/tmpxyz": no such file or directory

On Windows this happens when the path to racer isn't correctly set. Visit the package settings and point them to the racer executable directly. Example:
{
    // The full path to the racer binary. If racer is already
    // in your system path, then this default will be fine.
    "racer": "C:/Users/jeremiah/.cargo/bin/racer.exe",

    // A list of search paths. This should generally just
    // be the path to the rust compiler src/ directory.
    "search_paths": [
        "C:/Users/jeremiah/src/rust/src"
    ]
}

Nope, racer was spawned, but without any visible reactions at ST view.

Just to pitch in, things are working fine for me on Linux with the following config file:
{
    // The full path to the racer binary. If racer is already
    // in your system path, then this default will be fine.
    "racer": "/home/alec/.cargo/bin/racer",

    // A list of search paths. This should generally just
    // be the path to the rust compiler src/ directory.
    "search_paths": [
        "/usr/src/rust/src"
    ],

    // Additional situations to trigger auto complete
    "auto_complete_triggers": [
        {"selector": "source.rust", "characters": "."},
        {"selector": "source.rust", "characters": ":"},
        {"selector": "source.rust", "characters": "("}
    ]
}

Yep, problem issued on Windows only.

It works well for me.My syetem is win7 b4bit.And I set nothing in the config file.Just put the CARGO_HOME and the RUST_SRC_PATH in system enviroments.


Export

Export to CSV now works, including text in DataTables search.

Overriding traditional approval process


Nav Document Outline

Noticed in the nav examples that <h6> is being used for a dropdown header which gives a nav document outline of

Thoughts?

Hi @chrisbo I couldn't find it here: http://getbootstrap.com/docs/4.0/examples/navbars/
Can you send me the URL where you found it please?

There's an <h6> in our dropdowns.md file @andresgalante. I'm unsure what the screenshot is from, but it seems to be unrelated to our code :).

@andresgalante Just bumped into an example in the documentation:
[https://getbootstrap.com/docs/4.0/components/dropdowns/#menu-headers]
An <h6> shouldn't be used for styling.

BTW this was caused by the workaround we had to use so that unrelated ToC entries weren't added. This issue is now fixed and we can safely use whatever header we want to. It's just that we don't have a list so it's hard to find these.

As per my knowledge normally we do not use h6 for styling headline or even nav dropdown. But it depends on the designer.

What should we do about callouts though?
/CC @patrickhlauke

Ok I made a branch. While there are some good cases this can be fixed, there are others, namely some examples, that this will cause an extra class.
Also, the current ToC plugin will include some of the new headers. This will be fixed when we switch to Hugo though.
I'm not sure how to proceed.

as mentioned before, having heading level gaps is not ideal...but nobody really dies from that. if we can't rewrite heading levels on the fly for things that get included (callouts) to have the appropriate level, then having those at <h6> is the lesser evil (it would at least guarantee they're always subordinate to whatever heading comes hierarchically above it/the section they're in, assuming those only go to <h5> at the most)

I'm actually more concerned about the examples. See the above diff.

this is where the semantics of HTML aren't quite sufficiently expressive...you'd really want to be able to somehow demarcate those example bits of code as something that doesn't really pertain to the overall page structure...
but, as that's not possible, i'd say those are fine as per the diff. Once these don't show up in the TOCs either, it's probably as good as they can be made...

@patrickhlauke do we close this one?

yup, i'd say close


[PRM-1482] Updated Penalties Connector with new parameter

Also could you resolve the conflicts, please?


many Lags after connecting my RS-System to a Colosal-Chest

Issue description:
Lags after connecting my RS-system to a   ##colosal-chest
What happens:
s.a.
What you expected to happen:
Steps to reproduce:

I connect a colosal-chest to my system with an external storage bus.

...
Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: 12.18.2234
Refined Storage: 1.2.22

Does this issue occur on a server? [yes/no]
no
If a (crash)log is relevant for this issue, link it here:
stall-2017-03-06_18.13.02-2.txt
crash-2017-03-06_17.54.48-client.txt

https://refinedstorage.raoulvdberge.com/wiki/profiling

i hope this helps.
Uploading report.zip…

damn -_- wrong button sorry

And you're saying the lag starts just after connecting the Collosal Chest? There is NO lag before that?

Yes i mean that. The more stuff is stored in the chest, the slower is my RS-system. btw. the chest is only 1% full.
Now I can't even more craft a couple Stacks of compressed cobblestone, without getting minecraft to freeze.

Do you have Common Capabilities installed?

Yes, not by hand but it's in the modpack.

It looks like it isn't detecting Collosal Chests.

@Curse187 Don't connect the External Storage to the "Collosal Chest Interface", but to the "Collosal Chest Core".


Adds instructions for JIRA and PD cli setup

Adds some basic instructions for how to set up JIRA and PD Cli to at least give a starting point.

/lgtm
/approve


Incompatible with GUI Scale Hack

About GUI Scale Hack: https://www.curseforge.com/minecraft/mc-mods/gui-scale-hack
Screenshot: https://imgur.com/a/c3aYemr
Screenshot showing another scale-tweaked GUI (for comparing): https://imgur.com/a/OlD3zPS

Don't intend to add support.


Feature Request: alert user if there is opened tab with unlocked wallet.

I always have several browser windows with multiple tabs opened, and I often forget to close myetherwallet tab,
Alert or signal would be very good in this case.

Closing for inactivity.


Fix regexp of yum repolist to match /

Ref #499
I've also added specs for yumrepo to the integration tests.
https://github.com/serverspec/serverspec-integration-test/commit/37b28e7b5b2edb06731bae748cbc55e1bfdbe5a5

Released as v2.44.5.


Notification check height changes

Wandering traders can spawn outside the 48 block default radius on the Y level.
Currently, the plugin checks on a spherical radius centered on the wandering trader's position, however that can lead to issues.
This PR consists on adding a new configuration option "sphereRadius" which changes the mode on how the player check is done.
The function getNearbyPlayers calls getNearbyEntities with a single double as the radius.

Consider setting up your git author details: https://www.git-scm.com/book/en/v2/Customizing-Git-Git-Configuration
Build tools is weird and will set itself as author when running if you don't have anything configured.

Also, we need to check if the server is Paper before using that method and if not print a warning that this feature does not work on CraftBukkit/Spigot due to missing API. This is better UX than a no such method error.

Consider setting up your git author details: https://www.git-scm.com/book/en/v2/Customizing-Git-Git-Configuration
Build tools is weird and will set itself as author when running if you don't have anything configured.

Made use of Craftbukkit/Spigot's getNearbyEntities instead.
Also changed the author from BuildTools

I was wondering if it would be more efficient to use √((x2 – x1)² + (z2 – z1)²), completely ignoring the Y coordinates instead of using minecraft's own code in this scenario.


Offical project exists here. Brightcove is deprecated.

From their site: https://diamond.readthedocs.io/en/latest/#repos
Historically Diamond was a brightcove project and hosted at BrightcoveOS. However none of the active developers are brightcove employees and so the development has moved to python-diamond. We request that any new pull requests and issues be cut against python-diamond. We will keep BrightcoveOS updated and still honor issues/tickets cut on that repo.
Wasn't sure if the docs (exists in this PR) or their Github repo (https://github.com/python-diamond/diamond) was more appropriate. Happy to update the PR if you'd like it to point to the repo instead.

This is perfect!  Thanks, @esacteksab


compatibility with sklearn BaseEstimator

Question: how do we ensure compatibility with sklearn BaseEstimator and similar classes?
This might not be out of the box, if we check inheritance from BaseObject as opposed to availability of certain methods.
We could register a list of similar classes, but it would be better if this is out of the box compatible, e.g., get_params nesting and so on.

@fkiraly Whats the context for this? Are you thinking in terms of an eventual check interface (e.g. check_is_object) or something else?
I'm in favor of duck typing that check but including a param to require a "strict" check (is also a sub class of BaseObject).
We might consider grouping the checks used in something like check_is_object by interface and having a parameter that controls what to check for (e.g. just get_params/set_params, tags, testing, etc). That would provide the most downstream comparability (sklearn baseobject could for instance could pass) and provide users the means to test for just the interface required in their use case.
In terms of the interface implementation: I believe the get_params/set_params interface uses duck typing, so it should work with like objects. One option is to ensure that remains and make sure the other interfaces (testing and tags) do the same.

Based on our call we also want to have tests that test the ability of estimators inheriting from BaseObject to pass scikit-learn's check_is_estimator.
To do this we need to look into sklearn's check_is_estimator to see the scope and format of sklearn's checks. If their checks are duck typed, a nice outcome is that estimators that inherit from BaseObject are able to pass sklearn's check_is_estimator directly. If the checks in sklearn's check_is_estimator are instead checking for direct inheritance from the sklearn BaseEstimator (in addition to checking the interface) we may need have tests that validate that you can construct an estimator object that inherits from sklearn's BaseEstimator and baseobject's BaseObject to get the BaseObject interface and pass sklearn's check_is_estimator.

An alternative would be to check the main interoperation points: get_params and set_params.
That would require building composites either way round and see whether the nested access and write still works.

From https://github.com/sktime/BaseObject/issues/17, another interoperation point is pretty printing.

We should have a list of interoperation points we need to test, actually.


Add support for documentation comments

Hi, this PR is to add support for documentation comments on the data classes.
I've added a single test for this and am unsure where to look to add more tests to make sure it is robust.
Since this is my first contribution to moor, please let me know if there's anything you'd like changed.
Thanks for making moor, it's a really useful package. :)

I had only skimmed the contribution guidelines before, I'll reopen this MR against the develop branch instead. Sorry for the noise.


[Type mismatch] Tween interpolates by the int type of the from argument, not the float type of the property.

Tested versions
4.2.1
System information
windows10
Issue description

extends Node2D

func _ready() -> void:
	var t := create_tween()
	t.set_parallel()
	t.tween_property($Icon1,'rotation', PI, 1).from(0) # row1 left
	t.tween_property($Icon2,'rotation', PI, 1).from(0.0) # row1 right
	t.tween_property($Icon3,'rotation', PI, 1).as_relative().from(0) # row2 left
	t.tween_property($Icon4,'rotation', PI, 1).as_relative().from(0.0) # row2 right

tween interpolates by the type of the from argument, not the float type of the property.
Does not work if as_relative is used.
Steps to reproduce
Run the MRP project attached below.
Minimal reproduction project (MRP)
tween-int.zip

Related to https://github.com/godotengine/godot/pull/73178
Reading the code, it seems godot does use the from value to determine final value. cc @KoBeWi

I'm getting a different result in 4.3 dev2:
https://github.com/godotengine/godot/assets/2223172/eb9f084f-ae44-4997-886d-e6522ae41186
Seems like this was fixed by some of the animation changes.

Can you confirm the issue is fixed for you in the newest version?

The as_relative().from(0) one was fixed, but from(0) wasn't.

Fixed by https://github.com/godotengine/godot/pull/86046

but from(0) wasn't.

Not sure what you are referring to, but if you mix Int and float, the float will take precedence to make subsequent transitions consistent. This is an intended behavior.
If you have any other problems, please check the latest master or 4.3.dev2 or later versions and report them if you still have problems.

See how the t.tween_property($Icon1,'rotation', PI, 1).from(0) # row1 left behaves (top-left in the examples above).
Calling .from(0) makes the tweener tween with integers even though both the property and the final value passed are floats. So it's not true that float always take precendence in case of mixing float/int.
To me it seems like unintended behavior / bug.

Specifically when calling from(an_int) both initial_val and final_val end up being ints:
https://github.com/godotengine/godot/blob/0bcc0e92b3f0ac57d4c4650722f347593a258572/scene/animation/tween.cpp#L506-L512
https://github.com/godotengine/godot/blob/0bcc0e92b3f0ac57d4c4650722f347593a258572/scene/animation/tween.cpp#L73-L74
Hence delta_val etc. is int:
https://github.com/godotengine/godot/blob/0bcc0e92b3f0ac57d4c4650722f347593a258572/scene/animation/tween.cpp#L564


hunspellDicts: add chromium variants (.bdic)

chromium requires a custom format for hunspell dictionaries which they provide as blobs in an upstream repository.
building from source (using convert_dict from the chromium monorepo and applying it to already packaged dictionaries) would not yield the same results (chromium packages adjustments to the dictionaries themselves) and would increase the maintainance cost.
these files could be linked into ~/.config/chromium/Dictionaries to make spell checking work in ungoogled-chromium, my plan would be to adopt this into the chromium home-manager module.
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

@Janik-Haag

hm perhaps these should be in a different attribute set?
some packages seem to just include all dictionaries in their build process
https://github.com/NixOS/nixpkgs/blob/639ad2f7510a154bc11184dc8b07b10e120c2240/pkgs/applications/editors/rstudio/default.nix#L153

@ofborg build hunspellDictsChromium


Fix a problem in Settings>Transitions>IonMobility where Skyline would hang if user tried to "Edit current" when there is no current IMSDB…

…, or throw a null pointer exception when user tried to add an IMSDB via "Edit List > Add" instead of just using "Add"

Is there any concept of a generic "IonMobilityWindowWidthCalculator" that you could fall back to using if you can't find a "IonMobilityFilteringUserControl"?

Unfortunately not - that's a document setting we need for "Use Results".

Is there any concept of a generic "IonMobilityWindowWidthCalculator" that you could fall back to using if you can't find a "IonMobilityFilteringUserControl"?

Unfortunately not - that's a document setting we need for "Use Results".

(But it might not be saved to the current document yet, which is why we grab it from the control)


#67 Upgrade menu item to be able to select pages

closes #67

I will merge this and try to bring this into my branch ;-)
Good work 👍


dns: fix headless service handling

Before, we treated any service without a VIP as headless. This is not
correct. Consider
apiVersion: networking.istio.io/v1
kind: ServiceEntry
metadata:
  name: external
  hosts:
  - httpbin.org
  location: MESH_EXTERNAL
  ports:
  - number: 80
    name: http
  resolution: DNS

We should NOT treat this as headless.
The implementation is a total hack, but a better fix requires WDS
changes so for now a short approach is best
Fixes https://github.com/istio/ztunnel/issues/909
Fixes https://github.com/istio/ztunnel/issues/554

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
952c855ebd801bd812924fe5503fdf648cfca27c
link
true
/test test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #1058


feat: add serbian (latin and cyrillic) translations

Description
Add Serbian (Latin and Cyrillic) translations.

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[ ] Chore (non-breaking change which does not add functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Change to the templates directory (does not affect core functionality)
[ ] Change to the examples directory (does not affect core functionality)
[ ] This change requires a documentation update

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

@nikolasp, thank you so much for the translations!


Feature: Support different movement actions when keyboard appears depending on fits view or not.

 They have different inputViews with different height.
And it would be nice to have ability to specify different behavior for different cases.
In my case I want view to be centered if it fits on screen and be attached to top otherwise.
It's like "take as much place as you can and truncate top (bottom) otherwise".

Just set keyboardAppearence to do nothing are set malualy presentedFSFrame

From where to set it?
ViewController is not aware how it is presented.

Supported in new https://github.com/m1entus/MZFormSheetPresentationController , you can add your custom view controller movement style.


⚠ ClamAV has degraded performance

In 8229abc, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 569 ms

Resolved: ClamAV performance has improved in 8958096.


[hotfix][test] Fix TypeHintITCase if adaptive scheduler enabled

What is the purpose of the change
Fix TypeHintITCase if adaptive scheduler enabled
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

Thanks for the review.


vector tile work

vector tile work - moving to separate files so easier to merge into react-leaflet

don't delete this branch just yet


Issues: introduce issue column to execution list

https://staging.dashboard.kernelci.org:9000/tree/59cbd4eea48fdbc68fc17a29ad71188fea74b28b?tableFilter={"buildsTable"%3A"all"%2C"bootsTable"%3A"all"%2C"testsTable"%3A"failed"}&origin=redhat&currentTreeDetailsTab=treeDetails.tests&diffFilter={"archs"%3A{}}&treeInfo={"gitBranch"%3A"master"%2C"gitUrl"%3A"https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Fvirt%2Fkvm%2Fkvm.git"%2C"treeName"%3A"kvm"%2C"commitName"%3A"for-linus-1-g59cbd4eea48f"%2C"headCommitHash"%3A"59cbd4eea48fdbc68fc17a29ad71188fea74b28b"}

It's to have something like an asterisk or icon to show which test is generating an issue

Moved to P1 because we barely have issues to see this werking