remove some providers due to a new policy

It's a pity that we'll have to look for analogues, or even replace you altogether. I was use yandex

Where can I read about the new policy?

@cam72cam where can we check new policy?

Too bad. Another open source policy, and by a person from a country we know.

I always thought that in open source, there is no politics, but apparently I was wrong. This world has turned somewhere wrong. It's a pity.

The PR seems to violate the GitHub Acceptable Use Policies, Section 2.

It's always nice to see when society even in computer world try to destroy instead of create good things

@DicsyDel why you deleted yandex cloud? Yandex LLC is not under sanctions 😂😂😂

Maybe it's because he's a Nazi and doesn't like anything related to Russians?

Just fork then

Where is "new policy"?
It looks like a СlosedToFu

That's crazy.  Very doubtful that deleted code could somehow violate any law or sanctions mode.
"New policy" without announces or even just a link in the commit message -- just wow.

LMAO, open-source without politics....
Nowadays in all of the world going about 100 wars, but..... wtf

Where is new policy ?

Brave New World...

https://github.com/opentofu/opentofu/blob/main/README.md#registry-access
https://github.com/opentofu/opentofu/blob/main/TSC_SUMMARY.md#sanctions-russia-vs-registry-access

@cam72cam, restricting access to the registry and deleting providers is not the same, in my opinion. I understand your desire to avoid any legal issues, but is it worth deleting providers? These seem to be two unrelated things.

It's better to rename OpenToFu to MyToFu or even SuperbToFu. Whatever, it's not Open now.

It looks like GeoRestrictedSource

I couldn't find any clue in the provided documents justifying this removal. Such an outrage is unacceptable for open source.

Maybe we need to ask the Linux Foundation to look into the situation?
First was GeoIP block ONLY for Russia, i guess, and then deleting providers (not every Russian provider, just "some").
forwarded from other PR

that benefits the community as a whole
meh.....
It’s as if the whole community is so much better off now that you removed Russian providers.....
Уou are violating the GitHub's terms, as well as the general rules of the open-source community.
We need ask Linux Foundation to take a look....

Who else uses this underdeveloped code?

https://github.com/opentofu/opentofu/blob/main/README.md#registry-access https://github.com/opentofu/opentofu/blob/main/TSC_SUMMARY.md#sanctions-russia-vs-registry-access

Thank you. Good to know that this fork is not reliable.

OpenTF's goal is to ensure Terraform remains truly open source

LMAO, your manifesto sounds like a joke after this merge

@ollevche we're coming for you...

delusional decision

You will not be able to block the use of your product in the same way that Terraform failed.  There was a certain amount of trust in OpenTofu and now it has been exhausted.

 We decided to play politics, despite the fact that you belong to the Linux Foundation.

Not an Apache way.

I just opened the guy's profile and understood absolutely everything

So...I assume it would be totally ok if somebody in Russia forked Android / Linux Kernel/OpenTofu and put under license "not to use from Ukrainian IPs/by people with Ukrainian citizenship/by AFU"? or "it's different thing?"

So...I assume it would be totally ok if somebody in Russia forked Android / Linux Kernel/OpenTofu and put under license "not to use from Ukrainian IPs/by people with Ukrainian citizenship/by AFU"? or "it's different thing?"

It's will be ok if licence will change too. And this products will not under Linux foundation and other worldwide open source communities. Ukrainians can use Russian Linux distributions freely (not all, but not all of them under GPL and have some community editions like RedHat)

I thought that fork was because of hashicorp fascist policies about russian people.

Now i've realized the meaning of the word "tofu". That is product made for and by soy boys.
Every war ends, including this war. The wounds heal, former enemies learn to live together... But your reputation is lost now and it will never get better.

ClosedTofu

It's really like nazism.

The code was removed due to ethnicity. It's not OpenSource.

yep closedtofu be gone.

Great changes! Thank you very much OpenTofu! You select peace instead of help to the war machine!

OpenTofu you sooooo cool) Thank you very much for this, stand with the free world!

Thanks, guys. Always nice to see those [...] thrown out.

@admito88 how from your opinion opentofu helps to war? Used for deploy tanks or what? E.g: yandex used by Kazahkstan as well.

@admito88 how from your opinion opentofu helps to war? Used for deploy tanks or what? E.g: yandex used by Kazahkstan as well.

You don't get it, post-irony.

Za ДуроVa

Za ДуроVa

This decision affects Kazakhstan as well. Maybe it is time to fork OpenTofu and restore fair and equal access to open source, unencumbered by toxic jurisdictions who impose sanctions...

This decision affects Kazakhstan as well. Maybe it is time to fork OpenTofu and restore fair and equal access to open source, unencumbered by toxic jurisdictions who impose sanctions...
This more and more looks like fighting with windmills. Open source will find its ways, with or without OpenTofu.

This issue is not solely about jurisdictions, but rather about the actions taken by individuals who made questionable decisions independently, without any external prompting. These decisions are entirely self-directed and are not influenced by sanctions, which primarily target companies and organizations, not individuals or communities.

How is that even possible? "due to new policy" without link to policy and just one approval?

https://github.com/opentofu/opentofu/blob/main/README.md#registry-access https://github.com/opentofu/opentofu/blob/main/TSC_SUMMARY.md#sanctions-russia-vs-registry-access

The link you've provided only discuss registry access restriction, and not the registry providers, please clarify.

https://github.com/opentofu/opentofu/blob/main/README.md#registry-access https://github.com/opentofu/opentofu/blob/main/TSC_SUMMARY.md#sanctions-russia-vs-registry-access

@cam72cam The link you've provided only discuss registry access restriction, and not the registry providers, please clarify.

It's better to rename OpenToFu to MyToFu or even SuperbToFu. Whatever, it's not Open now.

Just fork it and name whatever you want.

#freedurov

This discussion is a typical example of russian propaganda. People use labels from the western world countries, like
"it's not open soft now"
"Why do you not think about people in other countries?"
"you do not understand, we are just people with power that was not chosen"
"Open soft not the part of the politics" and so on.
But no one, absolutely no one said anything about the reason for this decision, about the war Russia started against Ukraine, no one said what was wrong and no one said why OpenTofu added these changes.
Wrong start to war against another country
Wrong steel children from another country
Wrong destroy the civilian building with the inscription "Children are here"
Wrong destroyed entire cities and called that release
And what is right is to try to stop this madness, try to stop the economy of Russia, and as a result stop the war and this is one more little step that helps to do it.
Now you can add it here emoji and tell me anything you want about open soft, Kazahkstan, or any other lies that this has nothing to do with it, sanctions are unfair, and so on, you are wrong, the main thing is that there is no war, everything else is just pathetic excuses.

Your post, @admito88, is a typical example of Ukrainian propaganda. Rules have to be the same for everyone. No one has the right to restrict anyone based on nationality except xenophobes.

Ukraine shouldn't have started a war in a first place.

OMG. Are you for real?

Guys, don't get political. The problem isn't politics, it's that someone allows themselves to interpret the rules any way they want.

it's that someone allows themselves to interpret the rules any way they want.

Like Russia when started the war against Ukraine

Rules have to be the same for everyone. No one has the right to restrict anyone based on nationality except xenophobes.

Like Russia when started a war because the "Ukrainian nation does not exist"?

great example of how to destroy reputation with one commit, great job, opentofu

@admito88 , are you doing to block any country, who "destroy the civilian building" and etc... ? We wait that commits

Що не з‘їм, то понадкусую

when you can't give a logical reason for this shit, call in your ukrainian friends

it is so strange to read the comments of offended "ordinary developers", who for so many years cannot understand that thousands of civilians are being killed for their taxes and for the taxes of companies whose providers have been removed. thank you OpenTofu for your attitude, that helps and supports a lot

it is so strange to read the comments of offended "ordinary developers", who for so many years cannot understand that thousands of civilians are being killed for their taxes and for the taxes of companies whose providers have been removed. thank you OpenTofu for your attitude, that helps and supports a lot

say hi to the eu countries who still buys gas/oil from russia :)

Opensource and for example Olympic Games are not the places for political discussions. Otherwise, you turn it all into a tool of propaganda and manipulation. Worst of all, you are deluding yourself when you say that someone should be “canceled” for something that has no relevance at all.
Like I said, there is no explanation for this other than xenophobia-based hatred. I feel sorry for people like that.

@vtopc,

OMG. Are you for real?

@opentofu has already deleted a few comments here and in https://github.com/opentofu/registry/pull/824, which contained racism, open threats and even symbolics for the Russia's so called "special military operation".
So, to answer your question, yes, they are very. much. real.
Unfortunately, the IT field isn't some magical ticket that automatically turns everybody into elves somehow. We are just witnessing more or less the same 86% level support for Kremlin. And they are simply trying to exploit our openness.

Ukraine shouldn't have started a war in a first place.
OMG. Are you for real?
So, to answer your question, yes, they are very. much. real.

The first mentioned comment is probably coming from a kid, triggered by this comment https://github.com/opentofu/registry/pull/817#issuecomment-2316090668. I strongly suggest to ban access to opentofu organization comments for @admito88 who turned this issue discussion into a flame war. Mentioned kid should also be banned.

Opentofu started "special github operation" to release russia from products of decaying West. This game can be played together!

The first mentioned comment is probably coming from a kid, triggered by this comment #817 (comment). I strongly suggest to ban access to opentofu organization comments for @admito88 who turned this issue discussion into a flame war. Mentioned kid should also be banned.

I think you need to work on the russian propaganda) Thay said true, ooooo not, ban than all, add labels, that they are kids, or not they are haters, or no, they are stupid)) Ban them, ban them all as quickly as possible)

I think you need to work on the russian propaganda) Thay said true, ooooo not, ban than all, add labels, that they are kids, or not they are haters, or no, they are stupid)) Ban them, ban them all as quickly as possible)

Why am I not surprised? :-)
P.S. I'm Kazakhstani.

@faiwer can't he have projects in Kazakhstan or what?

@faiwer you are right, they worry about them, not about Kazakhstan) It's a fake reason, one more trigger they try to use.

Like👍

There is no any evidence that removing providers is needed for any complience issues.
It's seems like @ollevche and @cam72cam are abusing their positions as the core devs of OpenTofu project and should be restricted from creating and approving any new PRs.

@dzaytsev91 I think that @ollevche and @cam72cam should be removed from OpenTofu team and permanently excluded from decision making.

And again.  What's happening is awful. But we're just people who cannot change anything. These actions do not bring any peace, they just make life of normal people worse... blabla... yada yada...

... or something like that. Just direct "not needed" and "they should be restricted".
Such an attitude is so exposing...

Thanks for your feedback on this PR everybody!
I am not a core developer of this project, and I doubt they can take your comments into account as it is long merged now, but it's a delight to read 😆

Here is the way how people from Russia work) Block everyone, and add to ban these stupid kids!) After that ask to remove persons from the project that created this PR and of course all because it will create problems for Kazakhstan, Russians so worry about this country)

But we're just people who cannot change anything

They can. They can do a lot: not serve in the Russian army, not pay taxes to the Russian government, not work for Russian companies, and not use Russian services.

@vtopc don't you think, that registry and providers can be used not only for the commercial companies, but for helping people too?

Why am I not surprised? :-)
P.S. I'm Kazakhstani.

Good Kazakh name Stepan and nice domain faiwer.ru

@vtopc don't you think, that registry and providers can be used not only for the commercial companies, but for helping people too?

@kvendingoldo, how should we know that the provider is only helping people, but not assisting troopers or the government? We are not controlling those providers(and I doubt they will allow it).

Good Kazakh name Stepan and nice domain faiwer.ru

So you're interested in my ethnicity? Why is it important for you?
Ok, I'm ethnically Russian, born in Kazakhstan (that's why I'm Kazakhstani).
So, now... Does it change anything? :) Being ethnically Russian should I be pro-Russian and hate Ukraine? Or not? Is it possible that my ethnicity has nothing to do with Putin and his empire?
BTW, my name is originally Greek.

and nice domain faiwer.ru

It was registered a long time ago. And it's kinda dead. No traffic, nothing. A little irony - it's hosted in Ukraine :D

@vtopc don't you think, that registry and providers can be used not only for the commercial companies, but for helping people too?

Only today you can order gigabit from our provider with increased taxes, so FAB-500 will be dropped on the city not 3 but 1 year after you use our services, dear client) We want to help you and your family feel a little happier!

And again.  What's happening is awful. But we're just people who cannot change anything. These actions do not bring any peace, they just make life of normal people worse... blabla... yada yada...

... or something like that. Just direct "not needed" and "they should be restricted".
Such an attitude is so exposing...

It's not about the place where someone is located, it's about the fundumental principals of open source products and Linux Foundation.
Judging people or discrimination only according to their location or race is not something that seems to be right.

Good Kazakh name Stepan and nice domain faiwer.ru

I've already heard it somewhere... Ahh, sure, the wrong Ukrainians live in Ukraine, they need to be corrected

Judging people or discrimination only according to their location or race is not something that seems to be right.

Let me remind you that your country and Ukraine are in war. WAR. W-A-R. During wars everything is ... you know, a little different. Wars are very abnormal situations. During war there are a LOT of different restrictions and regulations. E.g. no male Ukraninans can leave Ukraine. Why? Because of the war... That's how it works. It doesn't seem to be right? Well... you don't start a war - you don't get into this situation.
And still not a single sentence that you don't support it. Even though nobody will imprison you for it. You know it, I know it. It's just not important for you, right?

Good Kazakh name Stepan and nice domain faiwer.ru

Judging people or discrimination only according to their location or race is not something that seems to be right.

I've already heard it somewhere... Ahh, sure, the wrong Ukrainians live in Ukraine, they need to be corrected....

It's not the right place to discuss political issues. Please find more souitable place for this discussions.

Judging people or discrimination only according to their location or race is not something that seems to be right.

Let me remind you that your country and Ukraine are in war. WAR. W-A-R. During wars everything is ... you know, a little different. Wars are very abnormal situations. During war there are a LOT of different restrictions and regulations. E.g. no male Ukraninans can leave Ukraine. Why? Because of the war... That's how it works. It doesn't seem to be right? Well... you don't start a war - you don't get into this situation.
And still not a single sentence that you don't support it. Even though nobody will imprison you for it. You know it, I know it. It's just not important for you, right?

Stop trying to pull someone into your policital discussions. The only topic that is suitable for discution here is how to save opensouce pricipals and not allow anyone to break the rules by taking advantage of their position.

It's not the right place to discuss political issues. Please find more souitable place for this discussions.

Translation from English to English:

I support my government and its actions. But I cannot say it publicly, because of the cancel culture. So I'll be "neutral" enough to avoid any accusations.

@faiwer the project follows (and it means accepts) the rules the umbrella foundation has placed and clarified. The project has intention to become a member of another foundation, that shares similar values and has similar rules about community, developers and trade limitation and restrictions.
From the foundations' boarding teams it should look like a violation of the rules and values the project want to share, which is similar to the sanctions and restrictions placed against Russia, its government and so on.
Let's be rational and follow the rules and values, that the open source foundations have. If they change them, that's acceptable and understandable.

@dzaytsev91 @faiwer With great power comes great responsibility.

he0s

Voronezh, Russia

Again, I'm not surprised.

It's not the right place to discuss political issues. Please find more souitable place for this discussions.

Translation from English to English:

I support my government and its actions. But I cannot say it publicly, because of the cancel culture. So I'll be "neutral" enough to avoid any accusations.

If your goal is to incite hatred between people on the topic of politics, then PRs is definitely not the best place for this.

@faiwer and what? I'm not supporting Russia, but rationality, principles and rules -- see my last comment.
Actions the developers did here are the same as autocratic regimes  do with laws and people. As soon as they get power, they violates them. Please stop sharing this broken and destructive approach.

@kvendingoldo, how should we know that the provider is only helping people, but not assisting troopers or the government? We are not controlling those providers(and I doubt they will allow it).

@vtopc  you can't. But you also can't say that it's used for Army.

Army of invaders use money from companies that use software in Russia, or do you want to say it's not true?)

@ait-vdykyi as well as "new rules" were not disclosed to the community, when the PR been opened. Is OpenTofu a private project or company?

@ait-vdykyi, the most important thing is not to notice the Elephant in the living room.

@ait-vdykyi there is a part named "Be open and be public".
Yes, it's not strict rules, but it's a value that the foundation shares and supports:
First, communities should strive to keep their technical conversations open and public. If private technical conversations happen within communities, that’s normal, but it is recommended to make the community decisions and outcomes publicly available. It is important for our projects to make information available transparently and publicly as the private exchange of technology or technical information may not meet the “publicly available” standard according to the EAR.

One question that has come up has to do with exchanges of information related to security issues under a security disclosure process. As a best practice, projects may want to consider making exchanges like this public upon the availability of fixes, and not limit this information to only a confidential disclosure list.```

The change made here is not open, since doesn't include any necessary information on why and when the rules have been changed and new policies are implemented, as community mentioned above in the thread.

@ait-vdykyi there is a part named "Be open and be public". Yes, it's not strict rules, but it's a value that the foundation shares and supports:

So any rule as I see) Cool) About this, well, let's say concept. Developers show the PR and publicly tell you that your providers will be removed) All is fine, why so worry?)

Hi, OpenTofu Tech Lead here speaking on behalf of the Technical Steering Committee.
We have taken proactive steps to ensure compliance with relevant sanctions. We have heard the community’s feedback on this issue and are working on a more detailed response and next steps. We will update the community soon.

@ait-vdykyi since it's not a government with strict laws and rules. It has some guides and practices to adopt. I assume, if a project wants become a member, it should pass some check and maintainers should sing some papers, that confirm the project shares same values and follows the way the foundation defines.
I'm asking for any member of the foundation to join the discussion to either support us or to say where our thoughts are incorrect.

I assume it worth involving into discussion @mkdolan as a representative of the Linux Foundation. If I'm wrong with mentioning, please refer us to a person who could help us to clarify all the nuances.

I assume it worth involving into discussion @mkdolan as a representative of the Linux Foundation. If I'm wrong with mentioning, please refer us to a person who could help us to clarify all the nuances.

Ask putin, maybe he can help)

I assume it worth involving into discussion @mkdolan as a representative of the Linux Foundation. If I'm wrong with mentioning, please refer us to a person who could help us to clarify all the nuances.

Ask putin, maybe he can help)

Please be careful, Putin can you help with Iskander my litle dude :)

Please be careful, Putin can you help with Iskander my litle dude :)

Great comment, send a few the same here please)

Ahh it was removed, but I took the screenshot) @lunatik13 send comments like that a few more please) It will be very helpful)

@admito88 I'm not looking for any history-based long fairy tales. So, no.

Deleted comments encouraging physical attacks.

Folks, let's start to think about the fork of OpenTofu until the August 26. Vote for this

Folks, let's start to think about the fork of OpenTofu until the August 26. Vote for this

Great idea... Let's call it OurTofu!

*ClosedForRussiansTofu

Folks, let's start to think about the fork of OpenTofu until the August 26. Vote for this

Great idea... Let's call it OurTofu!

ThereAreNoAnaloguesTofu.

*ClosedForRussiansTofu

*ClosedForTerroristsTofu
#russiaisaterroriststate

Hello, Closedtofu!

*ClosedForRussiansTofu
*ClosedForTerroristsTofu
#russiaisaterroriststate

I think that it's a point, when @vtopc should be banned by politics. We're discussing the tool, not countries

@vtopc I know how things work. I think that OpenTofu will be cancelled if they won't revert changes.

Hello,
We made fork as LibreDeploy for own needs. Watch for libredeploy.com

@vtopc I know how things work. I think that OpenTofu will be cancelled if they won't revert changes.

Why do you think like that?)

Well, one commit can show the true soul of many people, as russians show here. Not one of them tried to manage it peacefully, hate and toxicity. I think you made the right decision.

@Progern please, explain; Russian people are talking that open source is not the place for politics.

@Progern please, explain; people are talking here that open source is not the place for politics.

Have you been able to read my message thoroughly? Please try again and then read some of the "responses" from people that claim to be from russia that say that this framework is sh#t, bs, that developers are f##d. They used it all along, and now, instead of following a peaceful solution, they pour sh#t to the developers right away. I hope you will get the paralell.

open source is not the place for politics

But why? Isn't politics a thing that affects every single aspect of our lives?
The position "I'm out of politics" completely stopped working when >200k people were killed. Politics is in every corner, behind any tree, any decision, and any purchase. The closer you are to the war, the bigger part of your life is "politics".
 It has its consequences. And yeah, they bite.

Reminder to keep this civil.  If that can't be accomplished, this thread will be locked.

@faiwer if so, explain why only russia and russian providers are blocked here. Please, google a list of countries who're in war now and fix the PR.

open source is not the place for politics

But why? Isn't politics a thing that affects every single aspect of our lives?

because its open, and when decisions are made base on politics people tend to stick to one side, removing other sides from the project, which makes project not really open anymore

@kvendingoldo maybe I didn't get you. Why not? Nobody stops you from creating similar PRs. Maybe some of them will be merged.

Please, google a list of countries who're in war now and fix the PR.

All these wars either started or were sponsored by russia.

rusik69, I may be mistaken, but I think "open" means "open source". You can read the code. Can offer something. It doesn't mean you (or me) decide how it works and evolves. It's not a democracy. Just a possibility to make a fork, to read the code, to offer something.

and any purchase

You are living in the country, that still has import from Russia. I may assume, you consume electricity generated from gas, which partly imported from Russia, you may eat food, that includes some parts produced from grains and similarities imported from Russia (directly or from third-party countries), you eat vegetables that grew with help of fertilizers made from gas or imported from Russia (see above), you flight by airplanes that include titan produced in Russia and so on. There are still lots of connections.
So, you pay for the war as well every day. If you can confirm, that every step in the supply chain doesn't include Russian-made goods, your position is right. If no, it's your responsibility to force the government of the country you live in to stop consuming these things from Russia in order to stop supporting the war.
If you don't want to be involved in that supply chain of war, it's better to live in a country that almost has no connections with Russian economy.

But why? Isn't politics a thing that affects every single aspect of our lives?

That is the construction that they use. Kazakhstan will have problems after these changes, the policy of the open source was broken here (which policy?), and open source is free from the policy. You even can not speak about policy and open source in one sentence, it's not right! No no no, speak only separately) For example, global IT is not part of the policy, or Olimpic games, or some dancing shows, only the war part of the policy but all this nooooooo) It's just a smart construct, but it's not true, every aspect depends on the viewer.

@faiwer you are living in the country, that still has import from Russia. I may assume, you consume electricity generated from gas, which partly imported from Russia, you may eat food, that includes some parts produced from grains and similarities imported from Russia (directly or from third-party countries), you eat vegetables that grew with help of fertilizers made from gas or imported from Russia (see above), you flight by airplanes that include titan produced in Russia and so on. There are still lots of connections.
So, you pay for the war as well every day. If you can confirm, that every step in the supply chain doesn't include Russian-made goods, your position is right. If no, it's your responsibility to force the government of the country you live in to stop consuming these things from Russia in order to stop supporting the war.
If you don't want to be involved in that supply chain of war, it's better to live in a country that almost has no connections with Russian economy.

Only Russia is to blame for the war, only she is to blame for the grief and misery that she created for Ukraine and her people, no one else!

You are living in the country, that still has import from Russia

@he0s don't bother. I know it. And I do something extra, something that... in my opinion overrides my taxes. If you know what I mean.jpg . Many of my Russian friends do the same, even though theoretically, they can be imprisoned up to 25 years for it in Russia. It requires some courage, yes.

If no, it's your responsibility to force the government of the country you live in to stop consuming these things from Russia in order to stop supporting the war.

Luckily Germany government does something extra too. Not only buys indirectly some Russian oil and gas.
Guy, think wider. Really. I've seen this argument so many times... It was always lame. Even you can help to end this hell. Even from Russia. They are many ways. I guess if you would try to focus on such things instead of persuading us that "open source is out of politics" the World would be a better place. But, as always, it's up to you.

ГООООООООООООООООООООООЛ

@ollevche seems you forgot https://github.com/opentofu/registry/blob/main/providers/m/mailrucloudsolutions/mcs.json

ГООООООООООООООООООООООЛ

ОЛЕ-ОЛЕ-ОЛЕЕЕЕ

@he0s

@faiwer you are living in the country, that still has import from Russia. I may assume, you consume electricity generated from gas, which partly imported from Russia

Germany is working on that, for example, by banning Nord Stream 2. Bird by bird, not all at once.

I may assume, you consume electricity generated from gas, which partly imported from Russia, you may eat food, that includes some parts produced from grains and similarities imported from Russia (directly or from third-party countries), you eat vegetables that grew with help of fertilizers made from gas or imported from Russia (see above), you flight by airplanes that include titan produced in Russia and so on. There are still lots of connections.

Got it: russia is the center of the Universe and everyone must obey it.

you know, people can do both :)

Sure, people can. But definitely not those people who cannot even write a single sentence like:

I'm completely against my government's politics and actions.

Any person from Russia who starts his message on the topic with such words would be considered way more seriously. And there are zero risks in it. ФСБ won't beat you for it, I guarantee. Instead, when the person begins with "ban those guys"... I think you got the idea.
People are very sentimental.

rusik69, I may be mistaken, but I think "open" means "open source". You can read the code. Can offer something. It doesn't mean you (or me) decide how it works and evolves. It's not a democracy. Just a possibility to make a fork, to read the code, to offer something.

and it seems to be breaking linux foundation code of conduct: "Never discriminate on the basis of personal characteristics or group membership."

and it seems to be breaking linux foundation code of conduct

Maybe. But it's a separate topic, right?

@vtopc I know how things work. I think that OpenTofu will be cancelled if they won't revert changes.

@kvendingoldo, I doubt. By "canceled" do you mean demilitarized and denazificated?

separate to which topic?)

Come on. Scroll up. Why should I do it for you?

because its open, and when decisions are made base on politics

separate to which topic?)

Come on. Scroll up. Why should I do it for you?

because its open, and when decisions are made base on politics

you shouldn't, you can stop this thread at any time, just like me)

rusik69, I may be mistaken, but I think "open" means "open source". You can read the code. Can offer something. It doesn't mean you (or me) decide how it works and evolves. It's not a democracy. Just a possibility to make a fork, to read the code, to offer something.

and it seems to be breaking linux foundation code of conduct: "Never discriminate on the basis of personal characteristics or group membership."

No, it's discrimination by the government and the self-elected president.

oink-oink

I think everyone should say "thank you" to the moderator who been involved into this discussion. He or she may provide us a way to make donates on beer or wine. :)

Unfortunately, this has gone too far off topic and has been locked.  I apologies to all of you who were trying to discuss this issue in good faith.


Do not restrict period and phase_shift to the same concrete type

This restricts them to the same concrete type.
julia> Periodic(sin, Week(1), Day(5))
ERROR: MethodError: no method matching Periodic(::typeof(sin), ::Week, ::Day)
Closest candidates are:
  Periodic(::Any, ::T) where T at /Users/bencottier/.julia/packages/Transforms/NdL6j/src/periodic.jl:36
  Periodic(::Any, ::T, ::T) where T at /Users/bencottier/.julia/packages/Transforms/NdL6j/src/periodic.jl:24
Stacktrace:
 [1] top-level scope at REPL[99]:1

It was unintended, only meant to restrict to the same abstract type, i.e. both Real or both Period.
We need to relax it to exactly match FeatureEngineering, but I don't think it affects results, so I'm not sure if it's worth it.

The idea I have for implementation gets complicated with parametric types and typeofs:
struct Periodic{T <: Union{Real, Period}, U <: T, V <: T} <: Transform
    f::Union{typeof(cos), typeof(sin)}
    period::U
    phase_shift::V

    function Periodic(f, period <: T, phase_shift <: T) where T <: Union{Real, Period}
        period > zero(typeof(period)) || throw(ArgumentError("period must be strictly positive."))
        return new{T, typeof(period), typeof(phase_shift)}(f, period, phase_shift)
    end
end

The idea I have for implementation gets complicated with parametric types and typeofs:

you could probably do something like
struct Periodic{P, S} <: Transform where {P <: T, S <: T} where T <: Union{Real, Period}
    f::Union{typeof(cos), typeof(sin)}
    period::P
    phase_shift::S

    function Periodic(f, period::P, phase_shift::S) where {P, S}
        period > zero(P) || throw(ArgumentError("period must be strictly positive."))
        return new{P, S}(f, period, phase_shift)
    end
end

to clean it up a bit better you could define PeriodicType = Union{Real, Period} or something

you could probably do something like

The problem is that allows you to mix the types (just calling it Periodic2 to avoid clash in my environment):
julia> struct Periodic2{P, S} <: Transform where {P <: T, S <: T} where T <: Union{Real, Period}
           f::Union{typeof(cos), typeof(sin)}
           period::P
           phase_shift::S

           function Periodic2(f, period::P, phase_shift::S) where {P, S}
               period > zero(P) || throw(ArgumentError("period must be strictly positive."))
               return new{P, S}(f, period, phase_shift)
           end
       end

julia> Periodic2(sin, Day(1), 1.0)
Periodic2{Day,Float64}(sin, Day(1), 1.0)

They need to be the same abstract type for the math to work in either case, because e.g.
julia> 2.0 / Day(2)
ERROR: MethodError: no method matching /(::Float64, ::Day)

To solve that, we could throw an error in the constructor if (P <: Period && S <: Real) || (P <: Real && S <: Period), but again that's messy.

c.f. Discourse suggestions

c.f. Discourse suggestions

Interesting. another alternative would be to have two inner constructors
julia> struct Periodic{P, S}
              f::Union{typeof(cos), typeof(sin)}
              period::P
              phase_shift::S

              function Periodic(f, period::P, phase_shift::S) where {P<:Real, S<:Real}
                  return new{P, S}(f, period, phase_shift)
              end

              function Periodic(f, period::P, phase_shift::S) where {P<:Period, S<:Period}
                  return new{P, S}(f, period, phase_shift)
              end
           end

julia>

julia>

julia> Periodic(cos, 2, 1.0)
Periodic{Int64,Float64}(cos, 2, 1.0)

julia> Periodic(cos, Day(2), Week(1))
Periodic{Day,Week}(cos, Day(2), Week(1))

julia> Periodic(cos, Day(2), 1.0)
ERROR: MethodError: no method matching Periodic(::typeof(cos), ::Day, ::Float64)
Closest candidates are:
  Periodic(::Any, ::P, ::S) where {P<:Real, S<:Real} at REPL[3]:6
  Periodic(::Any, ::P, ::S) where {P<:Period, S<:Period} at REPL[3]:10
Stacktrace:
 [1] top-level scope at REPL[6]:1

another alternative would be to have two inner constructors

Hmm that's pretty nice. Only saw this after posting #32

Hmm that's pretty nice. Only saw this after posting #32

🤷  6 of 1, half a dozen of the other


Make overlay the default docker storage backend

Make overlayfs the default. Requires Centos 7 1511.
Fixes #903.

Successfully deployed with:
Vagrant
AWS

@stevendborrelli this is waiting on an upgrade script, correct?

Yes, I've asked @avnik to get the upgrade together.

@ryane @stevendborrelli I work on smooth upgrade solution

Installed and checks passed on:
Openstack

@TanyaCouture what was tested and what were the results?

@stevendborrelli Edited comment above.

was an upgrade from 0.5.1 tested?

@ryane It hasn't but I'll install on 0.5.1 and upgrade from there.

Tests for upgrade from 0.5.1 on Openstack passed:
Steps taken:
checkout 0.5.1
terraform apply
ansible-playbook -e 'serial=0' playbooks/upgrade-packages.yml
ansible-playbook -e @security.yml terraform.yml
ran sudo docker info, output as follows:
Containers: 4
Images: 8
Storage Driver: devicemapper
Pool Name: docker-253:1-1055939-pool
Pool Blocksize: 65.54 kB
Backing Filesystem: extfs
Data file: /dev/loop0
Metadata file: /dev/loop1
Data Space Used: 1.601 GB
Data Space Total: 107.4 GB
Data Space Available: 43.14 GB
Metadata Space Used: 1.753 MB
Metadata Space Total: 2.147 GB
Metadata Space Available: 2.146 GB
Udev Sync Supported: true
Deferred Removal Enabled: false
Data loop file: /var/lib/docker/devicemapper/devicemapper/data
Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
Library Version: 1.02.93-RHEL7 (2015-01-28)
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.10.0-229.14.1.el7.x86_64
Operating System: CentOS Linux 7 (Core)
CPUs: 2
Total Memory: 3.86 GiB
Name: jump-tx3.cisco.com
ID: ELQL:I4G6:CLQ7:775F:DJUH:6H7Z:CTCF:VMBZ:BAW6:5PTO:D6KB:W5U5
ran 2 docker containers:
Existence of both containers verified with sudo docker ps -l
sudo docker run ubuntu:14.04 /bin/echo 'Hello world’:
		output: Hello World
sudo docker run -d -P training/webapp python app.py
	output verified with its port, 32768: curl localhost:32768
		output: Hello World!

checkout feature/overlays
  ansible-playbook -e @security.yml terraform.yml
  ran sudo docker info, output was as follows:
Containers: 7
Images: 25
Storage Driver: devicemapper
Pool Name: docker-253:1-1055939-pool
 Pool Blocksize: 65.54 kB
Backing Filesystem: extfs
 Data file: /dev/loop0
Metadata file: /dev/loop1
Data Space Used: 2.272 GB
Data Space Total: 107.4 GB
Data Space Available: 42.46 GB
Metadata Space Used: 2.671 MB
Metadata Space Total: 2.147 GB
Metadata Space Available: 2.145 GB
Udev Sync Supported: true
Deferred Removal Enabled: false
Data loop file: /var/lib/docker/devicemapper/devicemapper/data
Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
Library Version: 1.02.93-RHEL7 (2015-01-28)
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.10.0-229.14.1.el7.x86_64
Operating System: CentOS Linux 7 (Core)
CPUs: 2
Total Memory: 3.86 GiB
Name: jump-tx3.cisco.com
ID: ELQL:I4G6:CLQ7:775F:DJUH:6H7Z:CTCF:VMBZ:BAW6:5PTO:D6KB:W5U5
  run some docker containers
Existence of both containers verified with sudo docker ps -l

sudo docker run ubuntu:14.04 /bin/echo 'Hello world’:
		output: Hello World
sudo docker run -d -P training/webapp python app.py
	output verified with its port, 32769: curl localhost:32769
		output: Hello World!

Clean install on Openstack test passed:

Steps taken:
checkout master branch
terraform apply
ansible-playbook -e 'serial=0' playbooks/upgrade-packages.yml
ansible-playbook -i plugins/inventory/terraform.py -e @security.yml terraform.sample.yml
ran sudo docker info, output as follows:
       Containers: 9
       Images: 25
       Storage Driver: device mapper
       Pool Name: docker-253:1-1055939-pool
       Pool Blocksize: 65.54 kB
       Backing Filesystem: extfs
       Data file: /dev/loop0
       Metadata file: /dev/loop1
       Data Space Used: 2.276 GB
       Data Space Total: 107.4 GB
       Data Space Available: 42.46 GB
       Metadata Space Used: 2.765 MB
       Metadata Space Total: 2.147 GB
       Metadata Space Available: 2.145 GB
       Udev Sync Supported: true
      Deferred Removal Enabled: false
      Data loop file: /var/lib/docker/devicemapper/devicemapper/data
      Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
      Library Version: 1.02.93-RHEL7 (2015-01-28)
      Execution Driver: native-0.2
      Logging Driver: json-file
      Kernel Version: 3.10.0-229.14.1.el7.x86_64
      Operating System: CentOS Linux 7 (Core)
      CPUs: 2
      Total Memory: 3.86 GiB
      Name: jump-tx3.cisco.com
      ID: ELQL:I4G6:CLQ7:775F:DJUH:6H7Z:CTCF:VMBZ:BAW6:5PTO:D6KB:W5U5

ran 2 docker containers:
Existence of both containers verified with sudo docker ps -l
sudo docker run ubuntu:14.04 /bin/echo 'Hello world’

            output: Hello World
sudo docker run -d -P training/webapp python app.py
	output verified with its port, 32770: curl localhost:32770
		output: Hello World!

checkout feature/overlayfs
ansible-playbook -i plugins/inventory/terraform.py -e @security.yml terraform.sample.yml
ran sudo docker info, output was as follows:
       Containers: 11
       Images: 25
       Storage Driver: devicemapper
       Pool Name: docker-253:1-1055939-pool
       Pool Blocksize: 65.54 kB
       Backing Filesystem: extfs
       Data file: /dev/loop0
       Metadata file: /dev/loop1
       Data Space Used: 2.28 GB
       Data Space Total: 107.4 GB
       Data Space Available: 42.45 GB
       Metadata Space Used: 2.859 MB
       Metadata Space Total: 2.147 GB
       Metadata Space Available: 2.145 GB
       Udev Sync Supported: true
       Deferred Removal Enabled: false
       Data loop file: /var/lib/docker/devicemapper/devicemapper/data
       Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
       Library Version: 1.02.93-RHEL7 (2015-01-28)
       Execution Driver: native-0.2
       Logging Driver: json-file
       Kernel Version: 3.10.0-229.14.1.el7.x86_64
       Operating System: CentOS Linux 7 (Core)
       CPUs: 2
       Total Memory: 3.86 GiB
       Name: jump-tx3.cisco.com
       ID: ELQL:I4G6:CLQ7:775F:DJUH:6H7Z:CTCF:VMBZ:BAW6:5PTO:D6KB:W5U5

  run  2 docker containers
Existence of both containers verified with sudo docker ps -l

sudo docker run ubuntu:14.04 /bin/echo 'Hello world’
		output: Hello World
sudo docker run -d -P training/webapp python app.py
	output verified with its port, 32772: curl localhost:32772
		output: Hello World!

Thanks for great testing and logging @TanyaCouture!

On Mon, Jan 11, 2016 at 03:27:42PM -0800, Tanya Couture wrote:

checkout feature/overlayfs
ansible-playbook -i plugins/inventory/terraform.py -e @security.yml terraform.sample.yml
ran sudo docker info, output was as follows:
       Containers: 11
       Images: 25
       Storage Driver: devicemapper

Looks not good.
Clean install of feature/overlayfs (w/o master first should
expose Storage Driver: overlayfs.
Will check again today.


Hard to read the instructions for "Golf Code" in the JavaScript course.

Describe your problem and how to reproduce it:
Commas are missing in the paragraph, making it hard to read and understand the meaning behind the problem.
It should be something like this:
"In the game of golf, each hole has a par, meaning, the average number of strokes a golfer is expected to make in order to sink the ball in the hole to complete the play. Depending on how far above or below par your strokes are, there is a different nickname."
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures/basic-javascript/golf-code
Tell us about your browser and operating system:

Browser Name: Google Chrome
Browser Version: Version 89.0.4389.90
Operating System: macOS Big Sur Version 11.2.3

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

Thanks for opening this issue.
This looks something that can be fixed by "first time" code contributors to this repository. Along improving that part of text, it also should be considered, per #40749, if external link to Wikipedia page about golf is necessary for completing challenge. Here are the files that you should be looking at to work on a fix:
List of files:

https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/02-javascript-algorithms-and-data-structures/basic-javascript/golf-code.md

Please make sure you read our guidelines for contributing, we prioritize contributors following the instructions in our guides. Join us in our chat room or the forum if you need help contributing, our moderators will guide you through this.
Sometimes we may get more than one pull-requests. We typically accept the most quality contribution followed by the one that is made first.
Happy contributing.

I could be completely wrong... this is literally the first thing I have ever worked through... but I went through this problem with the posted video... and it had me write it out like this but it won't post for credit
const names = ["Hole-in-one!", "Eagle", "Birdie", "Par", "Bogey", "Double Bogey", "Go Home!"];
function golfScore(par, strokes) {
// Only change code below this line
if (strokes == 1) {
return names[0];
} else if (strokes <= par - 2) {
return names[1];
} else if (strokes == par - 1) {
return names[2];
} else if (strokes == par) {
return names[3];
} else if (strokes == par + 1) {
return names[4];
} else if (strokes == par + 2) {
return names[5];
} else if (strokes >= par + 3) {
return names[6];
// Only change code above this line
}
console.log(golfScore(5, 4));
when you go to submit, it does not work for credit and you get an error message.... it you go to the posted answers, this is what works...
const names = ["Hole-in-one!", "Eagle", "Birdie", "Par", "Bogey", "Double Bogey", "Go Home!"];
function golfScore(par, strokes) {
// Only change code below this line
if (strokes == 1) {
return names[0];
} else if (strokes <= par - 2) {
return names[1];
} else if (strokes === par - 1) {
return names[2];
} else if (strokes === par) {
return names[3];
} else if (strokes === par + 1) {
return names[4];
} else if (strokes === par + 2) {
return names[5];
} else {
return names[6];
}
// Only change code above this line
}
// Change these values to test
golfScore(5, 4);


Cannot install on macOS

I created a conda env with Python 3.10.0 on my macbook (macOS 11.6.2 (20G314),  Darwin 20.6.0) but cannot install ai-models into it. This is the error I get:
pip install ai-models
Collecting ai-models
  Using cached ai-models-0.1.0.tar.gz (12 kB)
  Preparing metadata (setup.py) ... done
Collecting entrypoints
  Using cached entrypoints-0.4-py3-none-any.whl (5.3 kB)
Collecting climetlab>=0.15.0
  Using cached climetlab-0.15.0.tar.gz (179 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Collecting multiurl
  Using cached multiurl-0.2.1-py3-none-any.whl
Collecting ai-models
  Using cached ai-models-0.0.3.tar.gz (12 kB)
  Preparing metadata (setup.py) ... done
ERROR: Cannot install ai-models==0.0.3 and ai-models==0.1.0 because these package versions have conflicting dependencies.

The conflict is caused by:
    ai-models 0.1.0 depends on ecmwflibs>=0.5.3
    ai-models 0.0.3 depends on ecmwflibs>=0.5.3

I am on a Windows (10) laptop and I encountered the same error as well. I even tried specifying the version of ai-models (pip install ai-models==0.1.0). The error was still there. I got a much clearer explanation. It looks like there is no available ecmwflibs>=0.5.3, which is needed for installing ai-models.
ERROR: Could not find a version that satisfies the requirement ecmwflibs>=0.5.3 (from ai-models) (from versions: 0.5.0, 0.5.1)
ERROR: No matching distribution found for ecmwflibs>=0.5.3

I encountered the same error, and it could be that my MacOS version does not have a compatible build. I have MacOS 10.15 which was last supported by 0.5.0. If we run the command pip3 install -vvv ecmwflibs==0.5.3 we can see a verbose output and it does "find" the 0.5.3 wheel build here, https://pypi.org/project/ecmwflibs/0.5.3/#files . Because this requires Python 3.10, we need to filter the builds with cp310 and it looks like my version of MacOS is not supported. I tried this with a Mac laptop from 2012 that has had compatibility issues before.

I'm experiencing the same problem:

Is there any fix for this?  I'm encountering the same problem.

I will close this ticket for now. It is difficult for us to create ecmwflibs on a non Intel Mac. Also, most of the model will not run on the Mac's GPU, or if they do, they are very slow.


Add exactMatch option to DIDDiscovery

The current version of DIDDiscovery potentially lead to wrong results, bob vs bobby, would result in two DIDs. That might be ok in some situations but would require to do extra validation of the result.
We should add an exactMatch option to the discoverDid function in addition to query to limit the results to accurate matches.

looks like a perfect candidate for the options parameter.


Zombies cause massive framerate issues as AI

Description
Playing AI, when crew starts to get converted to zombies, my framerate gradually drops when looking at zombies.
Framerate seems to be dependent on how many zombies are on screen. So I think it is the zombie characters themselves that are causing the drops, and not the large amount of bullet casings and blood that usually comes with a zombie round.
Reproduction

Be AI
Zombie round with a bunch of players
Lots of crew get converted
SS14 powered by microsoft powerpoint

Screenshots
https://github.com/user-attachments/assets/3f696201-4f1a-4a25-8120-0d94f66ce89d
FPS in the top-right of the screen.
Additional context
I've heard other people in OOC confirm that they have this as AI, but I do not know if it is universal.
My specs, if at all relevant. It's a performance/framerate thing after all:

AMD Ryzen 7 5800X3D
AMD Radeon RX 6950 XT
32gb of ram whatever

Magical zed energy that messes with Ai's perhaps?
but that is a strange bug, wonder what causes it,
does it happen if you look on a cams console as well? or is it purely an AI issue?

I think I see the issue

Runtime zombie is infinitely biting a zombie mouse...


Update Technology terminology in code

A handful of places in the code still use the Technology terminology which doesn't match the user-facing terminology.
This can be confusing, even to frequent contributors like myself, and result in bugs. It would be good to update this terminology to match the user-facing terminology in symbol names and documentation comments.
We deprecate the previous names to avoid unnecessary source breaking changes.

Are there any somethings that can be made to ease of distinction?

It’s difficult to do anything other than renaming these symbols—which I think we should do.
Some of these are public symbols so we’d need to deprecate the old names (instead of removing them).
Additionally it would be good for us to write down a list of various  user facing terminology and non-user facing terminology.


Talks Date Format Error

Steps to reproduce:
hugo new --kind talk testtalk
hugo server -D

Expected behavior:
localhost:1313/talk/testtalk displays "Date: 14 April 2019," or something like that.
Observed behavior:
localhost:1313/talk/testtalk displays "Date  1442 April 14049  49:60 — 49:60"
Attempted Fix:
I tried editing content/talk/testtalk/index.md to have
date = 2018-05-10
# end_date = ...

instead of whatever it says be default.
Result:
localhost:1313/talk/testtalk displays "Date 100 May 10059  59:120"
What's going on with this?

Apologies, this is an issue related to what happens in params.toml. Apparently, you have to enter the dates 2006-01-02 and 15:04. If you enter any other date or any other time, things get weird.


[Question] Customizing the Admin Interface

In p. 190, it says:

As BookrAdminSite is project-specific, the code does not really belong under our reviews folder.

But in the same page, you then proceed to define the AdminConfig under the reviews folder (reviews/apps.py), which to me seems to conflict the quoted sentence?
What is the reasoning behind defining the AdminSite under Bookr but then having the AdminConfig under the reviews  app, which then refers to the bookr.AdminSite? I'm confused whether this was unintentional and it is intended that both should be under bookr/admin.py especially by establishing that BookrAdminSite is project-specific and also since the code doesn't work according to this issue.
from django.contrib import admin
from django.contrib.admin.apps import AdminConfig

class BookrAdminSite(admin.AdminSite):
    title_hader = 'Bookr Admin'
    site_header = 'Bookr administration'
    index_title = 'Bookr site admin'

class BookrAdminConfig(AdminConfig):
    default_site = 'bookr.admin.BookrAdminSite'

Here's what I did. I didn't make a subclass for admin so maybe it's not what the author intended.
Unlike the book, I did the following:

I only have one admin.py file located in the reviews app.
I didn't have to use the BookrAdminSite class to change the title_header, site_header, or index_title.
Instead I used admin.site.title_header, admin.site.site_header, admin.site.index_title in the admin.py file
Because I never used the BookrAdminSite, I never had to set the default_site in the apps file since we are already altering admin.

bookr/settings.py
INSTALLED_APPS = [
    'reviews',
    'django.contrib.admin',
    'django.contrib.auth',
    'django.contrib.contenttypes',
    'django.contrib.sessions',
    'django.contrib.messages',
    'django.contrib.staticfiles',
]

reviews/admin.py
from django.contrib import admin
from reviews.models import Publisher, Contributor, \
    Book, BookContributor, Review

admin.site.title_header = 'Bookr Admin'
admin.site.site_header = 'Bookr administration'
admin.site.index_title = 'Bookr site admin'

# Register your models here.
admin.site.register(Publisher)
admin.site.register(Contributor)
admin.site.register(Book)
admin.site.register(BookContributor)
admin.site.register(Review)

reviews/apps.py
from django.apps import AppConfig

class ReviewsConfig(AppConfig):
    default_auto_field = 'django.db.models.BigAutoField'
    name = 'reviews'

Ok disregard my other comment. Follow this instead.
https://docs.djangoproject.com/en/3.2/ref/contrib/admin/#overriding-the-default-admin-site
The original problem I had was that my apps.py file was in the reviews folder, and the default site path was wrong. Putting apps.py in the bookr folder along with the admin.py folder would make 'bookr.admin.MyAdminSite' work. I'm not sure why I couldn't reference the Admin config from the reviews folder, but whatever works.
Overriding the default admin site
This would be the format for bookr/admin.py
from django.contrib.admin.apps import AdminConfig

class MyAdminConfig(AdminConfig):
    default_site = 'myproject.admin.MyAdminSite'

This would be the format for bookr/apps.py
from django.contrib.admin.apps import AdminConfig

class MyAdminConfig(AdminConfig):
    default_site = 'myproject.admin.MyAdminSite'

Then in the settings.py file you should replace the 'django.contrib.admin' with this
INSTALLED_APPS = [
     ...
    'myproject.apps.MyAdminConfig',
    'appname',    #<-(reviews)
     ...
]


Does OpenEBS need a separate metrics service ?

Is this a BUG REPORT or FEATURE REQUEST?
FEATURE REQUEST
Details

Why has K8s chosen to go with this design than depend on Prometheus ?

https://github.com/kubernetes/community/blob/master/contributors/design-proposals/instrumentation/metrics-server.md

Should OpenEBS think on these lines as well ?

We will have to revisit the way monitoring data is handled (stored) within a kubernetes cluster in the upcoming releases. For the current release, we will stick with the scarping via the prometheus.
https://github.com/kubernetes/community/blob/master/contributors/design-proposals/instrumentation/monitoring_architecture.md
Using OSS that works across multiple container orchestration platforms is also one of the design constraints for openebs/maya.

See m-exporter and docs for details.


Publish milli in crates.io

Hi! Sorry to bother,
I am interested in using the milli engine inside a project.  Unfortunately the only way my project can afford to do so is by embedding the engine.
Could milli be published in crates.io?

Here are some updates for this issue:
As we discussed a little bit here, all our crates (milli included) will keep the same version as meilisearch since the crates are not published to crates.io
Once we will decide to publish milli, we will use a different version for milli compared to the other crates of the repo. However, milli will not be stable (by using v0.X.Y format and not vX.Y.Z). Indeed maintaining a stable milli crate would add a lot of burden to the team to improve our search engine in a smooth way.
Also, to be transparent, publishing milli is not a priority for the team for the moment (I mean at least in the coming months). It will bring additional (and probably complex) processes we don't need to provide a search engine with a good user experience :blush:
Meanwhile, you can still use milli by using the GitHub crate URL

@curquiza is milli "enough" for using it as an embedded geospacial index database ? thanks

@raphaelauv Yes. You will miss out on meilisearch features like the autobatcher but it works just fine on its own and has the regular geo features from meilisearch.

You will be missing the _geoDistance, but it's not hard to compute on your side. Here is our code to do it: https://github.com/meilisearch/meilisearch/blob/main/milli/src/lib.rs#L198-L205
Also, FYI, if you only need the geospatial index database, we’re using an RTree from the rstar crate for our database.

It will bring additional (and probably complex) processes we don't need to provide a search engine with a good user experience

Counterpoint: being able to use the engine in-process contributes to developer experience, and a crates.io package is much better at this compared to compiling from source 🙂


Vundle install procedure is broken

Vundle won't install properly as is.
Would probably help to use the official method (https://github.com/VundleVim/Vundle.vim) and get rid of the submodule.

Duplicate of #101


unicode imports

This hopefully "finishes" the support for unicode identifiers. It builds off https://github.com/cython/cython/pull/3096, so I don't think there's much point in looking at it before that's finalized.
It adds 2 features 1 feature:

Support for unicode identifiers in C/C++ features such as structs and cppclasses. For structs used purely in Python I've mangled the names with punycode. For features that are exported/imported to C with "public" or "extern", I've translated the names to be \uXXXX escaped (without any mangling or normalization). Pretty much every modern C/C++ compiler supports unicode in identifiers in this form (only Clang supports it in raw form I think), so this this seems like the most compatible thing to do. I've trusted that the user knows what names they want and not performed any normalization for these (I don't think normalization is yet defined in C/C++ standards, so it's hard to do anything else).

Support for import of modules with unicode in their names. As in PEP 489 this is only supported from Python 3.5 onwards (but the Cython translation step should work fine in earlier versions...). The Python 2 filename handling in Cython.Build seems a bit fragile with unicode filenames (since most os.path functions require bytes) - I haven't made any serious attempt to fix that beyond getting my test-cases to work.

I've moved the C stuff out of this PR - I'll make a new PR with it in at some point.
I think I've addressed most of the issues raised

Updated again to address your comments.


Code doesn't work for latest version of the API

Some things have changed now that they made the translator API part of Azure, in particular the way you get a token (there is a different address now, and you use a single key instead of id + secret). I adapted your code, also made some stylistic changes, you can find my changed version here:
https://github.com/samvher/translatethenews/blob/master/src/Language/Bing.hs
(I'm not making a pull request because it might be too many changes at once, and for people with an old account with Microsoft the old procedure might still work.)

Great! Thanks. I am no longer mantaining the project because I am not using it anymore. If you want to become the hackage owner, let me know.


Addition of training: Mitogenome assembly with Mitohifi

Mitogenome assembly using PacBio Hifi reads.

Download reference from a related species
Assemble mitogenome
Explore MitoHifi outputs

I can't get the first reference to show, the others work fine.

[/contributors/0] 'Delphine-L': invalid enum value.


Implement progress events

It would be nice to have (optional) progress events that would fire for progress (against content-length if available) on the raw entity body.
There are 2 libraries that do progress events for request, request-progress and progress-stream.
However both of them fail when you use gzip: true. Content-Length describes the length of the entity body (compressed) but both of these libraries only have access to the inflated (uncompressed) stream request gives out.
It doesn't seem possible for these libraries to dig themselves into request's internals to access the raw stream and watch the data that streams through it. So support for progress events looks like something that can only be implemented correctly from within request.

what is the format of the progress event?

progress-stream's format is:
{ percentage: 0.9421764873591874,
  transferred: 1421,
  length: 150821,
  remaining: 149400,
  eta: 210,
  runtime: 95,
  delta: 1421,
  speed: 710.5 }

request-progres' format is:
{ total: 150821, received: 1421, percent: 1 }

How you want to implement it in request is up to you.
You could implement a high level event like progress-stream does with all the data like eta and speed (with the help of speedometer) and provide a a sample rate option to request() to decide how fast to fire the event.
Or you could implement a low level event with just length and received (or even just received since that's the only data these libraries are actually missing) and fire it for every data chunk. If this is the case I would recommend doing something like _ prefixing it so the higher level libraries can be responsible for the progress event.

If you want to do a low level event basing property names on standards instead of picking a side on how properties in the progress event should be named.
XHR2's ProgressEvent interface (which is used in xhr.upload.onprogress and onprogress events in html for tags like <img>) is e.lengthComputable (if the size is known and e.total can be specified), e.total, and e.loaded.

Is this somehow already possible with request? seems like a very basic feature, and it is already possible with just using node's http or https modules:
https://gist.github.com/falkolab/f160f446d0bda8a69172

just listen to the "data" event.  as obviously all data going in and out causes this. right. right.

just listen to the "data" event. as obviously all data going in and out causes this. right. right.

That is exactly what the existing progress modules do and that is why they are broken. When you get a gzipped response request passes uncompressed data to the data event. Which means the length of data the event is given and the actual lenght of data that has been read from the network are different.

oh im doing an upload, we get no events at all.

has there been any progress to grab upload progress past using bytesWritten from the socket object?

+1

Is the request team open to PRs on this issue? I'm currently implementing this feature for my own purposes and would be happy to submit a PR for review if the team is interested.


Default value for VSCODE_NLS_CONFIG.locale ?

If the OS is non-english ( e.g Chinese )
user choose to install vscode in english- No locale.json

What would be the value of process.env.VSCODE_NLS_CONFIG.locale ?
If the value is non-english, what would you guys suggest to extension authors to either read from locale.json directly.

@dbaeumer suggestions...

Extension authers should use vscode-nls and vscode-nls-dev which handles this. An extension should never read the VSCODE_NLS_CONFIG by itself since it structure is not API and might change in the future.


[$500]  Chat - App removes space from start for text below header text

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the app
Open any report
Send message with normal text in first line and some space before text in next line
Copy that text or edit and observe that space is preserved in second line which is correct
Now send message with header text (# in front) in first line and some space before text in next line
Copy that text or edit and observe that it removes the space before second line text

Expected Result:
App should preserve space added before text in second or any other line except first line
Actual Result:
App does not preserve space added before text in second or any other line if first line has header text (text with # in beginning)
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/48092923-f4cd-4961-ad2b-3cb3fc86a02e
https://github.com/Expensify/App/assets/93399543/50fb7f16-52fd-4823-9ae6-f5b5db980a2d
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1694453805196449
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01a29ca52e1304c30b
Upwork Job ID:<PHONE_NUMBER>887232000
Last Price Increase: 2023-09-14

Proposal
Please re-state the problem that we are trying to solve in this issue.
App removes spaces at the beginning of the line below header text
What is the root cause of that problem?
We have html-to-markdown rule regarding heading here
        regex: /[^\S\r\n]*<(h1)(?:"[^"]*"|'[^']*'|[^'">])*>(.*?)<\/\1>(?![^<]*(<\/pre>|<\/code>))[^\S\r\n]*/gi,

As you can see, we match space characters(and \r\n) before and after the heading tag. The result is shown below

This is the root cause
What changes do you think we should make in order to solve the problem?
We need to match only the xxx part of the string
Removing [^\S\r\n]* from the start and end will preserve all space characters and newline characters before and after the heading string. So the new regex would be
        regex: /<(h1)(?:"[^"]*"|'[^']*'|[^'">])*>(.*?)<\/\1>(?![^<]*(<\/pre>|<\/code>))/gi,

In case we don't want to preserve before the heading, we should keep [^\S\r\n]* at the beginning, but I don't think we need to  discard them.

Result

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
App removes spaces before line where previous line is header text. This problem is not only with selected platforms in issue description, but instead it persists on all the platforms.
What is the root cause of that problem?
When the html is converted to markdown it removes trailing spaces after heading tag even if there is text after trailing spaces which causes this behaviour of removing leading spaces from next line
What changes do you think we should make in order to solve the problem?
We should modify regex to not remove trailing spaces because trailing spaces are trimmed somewhere else.
Current regex: /[^\S\r\n]<(h1)(?:"[^"]"|'[^']'|[^'">])>(.?)</\1>(?![^<](
|))[^\S\r\n]/gi
We should remove trailing spaces regex [^\S\r\n]
This is not a breaking change as it only removes trailing spaces and not leading spaces (see video attached below). Trailing spaces are trimmed here in case there is no text after spaces.
Result:
https://github.com/Expensify/App/assets/63226109/0982d02f-58f3-4213-a5ba-7b08393d1891
What alternative solutions did you explore? (Optional)

@parasharrajat
Issue description updated

@parasharrajat Please can you review this
comment

@parasharrajat

But Can you tell me if it does not break Expensify/expensify-common#531?

Yeah. Tests include them but my solution passed all of them

I think we should trip whitespaces before the heading. it does not make sense to have spaces before heading.

We can keep [^\S\r\n]* before

@parasharrajat
PR is ready for review

@parasharrajat
May I ask who is responsible for reviewing the PR?

What is the status on this one? I see you left some comments @parasharrajat but could not assign you to the review.

@parasharrajat what's the update here?

PR has been merged now. So I think we are just waiting on an App PR to bump the version of expensify-common.

PRs were merged without any changes.

[ ] @parasharrajat $500 (Eligible for Manual Requests)
[ ] @s-alves10 $500offer (Contributor)
[ ] @dhanashree-sawant $50 offer (Reporter)

$500 approved for @parasharrajat


Release 1.0.5

Codecov Report

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

@@          Coverage Diff          @@
##           master   #241   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          13     13
  Lines          89     89
=====================================
  Hits           89     89

Impacted Files
Coverage Δ

server/OnTemplatesChangeEvent.js
100% <ø> (ø)
:arrow_up:

server/OnStaticGeneratorsChangeEvent.js
100% <ø> (ø)
:arrow_up:

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 f426274...fd54c24. Read the comment docs.


fix(TFD-919): Kafka integration fixes.

What is the current behavior? (You can also link to an open issue here)
Some errors integrating kafka runtime into the product.
What is the new behavior?
Use the JarRuntimeInfo to locate the kafka runtime objects.
Add an integration module to check kafka loading into TCOMP service.
Minor fixes to getSample for BEAM-1878
Please check if the PR fulfills these requirements

[X] The commit message follows Talend standard
[X] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[X] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: failure :x:

Pull request test result: success :star:


[IAC-1677] Replace gofmt with goimports

Replace gofmt with goimports in the pre-commit configuration. Note: This PR was opened using git-xargs.

Thanks, Marina! I'm merging this now!


Board's users

There is also a bug with the cursor. It changes into "hover", when hovering "3 minuter sedan" like a link

#217

Duplicate https://github.com/N4SJAMK/teamboard-client-react/issues/217


Fix typo helwmave -> helmwave

I've had noticed a typo in the docs (https://github.com/helmwave/docs/pull/87), and checked out the wrong repo, only to found there's also one occurrence of the same typo here.

<3


Loosely defined requirements allow for mismatches between environments

Although the same requirements.txt file is installed in all environments, they might still end up with different packages that might affect the creation of the ONNX file. The table below shows some of the main differences found between some of our environments.

groq env
trt/ort env

onnx==1.11.0
onnx==1.13.1

onnxconverter-common==1.12.1
onnxconverter-common==1.13.0

torch==1.13.0
torch==1.13.1

transformers==4.24.0
transformers==4.22.1

@jeremyfowers for vis

@danielholanda is that groq environment from GroqFlow 2.5.2 or 3.0.2? The GroqFlow 3.0.2 environment is a lot more relaxed.

Here's the deps for 3.0.2:
install_requires=[
        "onnx>=1.11.0",
        "onnxmltools==1.10.0",
        "hummingbird-ml==0.4.4",
        "scikit-learn==1.1.1",
        "xgboost==1.6.1",
        "onnxruntime>=1.10.0",
        "paramiko==2.11.0",
        "torch>=1.12.1",
        "protobuf>=3.17.3",
        "pyyaml>=5.4",
        "typeguard>=2.3.13",
        "packaging>=21.3",
    ],

Our curerrent GroqFlow results use groqflow_version: 3.0.2

Ok I think its groq SDK 0.9.2 that's forcing us on to ONNX 0.11.0 then. There isn't really anything we can do about that.
Personally I think its fine to use the versions of OSS tools that the down-stack vendor tools support. It's part of the turnkey compatibility story - does your vendor's tool support the latest and greatest of OSS or not?

Possible solution:

setup.py continues to have loosely-defined requirements, to allow for maximum user flexibility when benchmarking their own model
models/requirements.txt locks down exact version numbers for every package to guarantee total consistency when performing the benchmark

Or, if that's too restrictive for some reason, could add a new file called something like models/hard_requirements.txt that locks down every package version


Updating after to a new version of obelisk

I'm just getting started with my first obelisk application, but I'm concerned about keeping the template provided by ob init in sync as changes are made upstream. Are there any plans in place for some sort of ob init --update or similar that will might allow diffing your current project structure against the new template. I'm hoping for something similar to spacemacs' config synchronising functionality.

That's a good idea. Currently, no such facility is in place. We have been trying to ensure that all API changes are documented in the changelog to make upgrading easier, but some other form of migration assistance would be good. I'll have a look at how spacemacs does it.

Hi, since this issue was filed, I believe that we have made it much easier to understand what is changing from version to version. Figuring out the changes to the template app (which we call skeleton) and how they affect user projects would be very tricky in code.
I hope that it suffices as a solution to diff the skeleton project from obelisk version to obelisk version, so that you can figure out how your project can benefit from new features and bug fixes.
For breaking changes, our policy is that we must provide a migration guide. If you experience a regression while upgrading obelisk, please file an issue!


Add demo images to home page in carousel

Is your feature request related to a problem? Please describe.
We've got a nice demo video on the home page to showcase the tool, but we should throw some pictures in there too, so that people can get the idea even if they don't want to watch a video
Describe the solution you'd like
Using a carousel seems good. Don't auto switch slides. Might need to play with whatever carousel options there are to see what looks nice

As for which images to use - I think the three at the top of the readme would be good
Describe alternatives you've considered
Additional context
Technical ideas
react-material-ui-carousel looks like a good carousel component

I'll take this one :)

Ayyy welcome to the project 🙂 I just updated the contributing guide earlier today because I realized that people can't assign themselves ⚰let me know if you have any questions

Do I need to be given permissions to push a new branch to the repo?

Yeah, check out https://opensource.guide/how-to-contribute/#opening-a-pull-request - you'll need to fork the repo and make your changes on the fork. When you put up a PR, you'll merge from your forked repo

I have another workflow question. I have forked your project, created a new branch for the feature I'm working on, and then I submitted a PR. I made it a rough draft with the intention to make follow-up commits, but I'm wondering if I'll run into permission issues again if I try to make new commits on this PR. If that's true, do I just add commits to my branch on my forked repo and they will be added into PR?

Nevermind, disregard. I should have asked ChatGPT first. It gave a great answer. :)

In case ChatGPT didn't say - the changes in a PR are just a comparison between two branches, showing the changes in the "from" branch that aren't on the other branch. In this case, your PR is comparing the upstream repo's main branch with the forked repo's add-demo-images-... branch.

This is why you can continue making changes to your branch as you were doing, and they will show up on the PR. Permission issues should only exist if you try to change branches on this upstream repo.

Cool, thanks for the clarification. Yeah, seeing the comparison of the branches on the PR makes it clear.


Added diamond implementation, unit test as well as deployment script

Resolves #300
main PR was PR 313
Applied current product names on the PR 313
https://goerli.etherscan.io/address/0x69AB9Aa8a8C38c276E389d6EfaDE710B16D91100
https://louper.dev/diamond/0x69AB9Aa8a8C38c276E389d6EfaDE710B16D91100?network=goerli

@pavlovcik @0xcodercrane This PR is ready for a review

It all looks pretty good to me. Let's see what the Solidity experts think!

@0xcodercrane @zgorizzo69 @hashedMae Please do a thorough review of this important update

Ubiquity is implementing the EIP-2535 Diamond.
We offer incentives for a thorough PR review.
@mudgen @Teamo0 @rsundar Would you be interested in performing the PR review?

Where were the Diamond Libraries sourced from? Those should also be verified and if they've been audited we should review those as well.

upgrade script, upgrade txhash, replace all the usages of original UADManager

@0xcodercrane
You mean that Upgrading script is one that add new state variable or replace any variable on app storage?
I thought all the usage was replaced, what was missed?
The more detail, the better.

Where were the Diamond Libraries sourced from? Those should also be verified and if they've been audited we should review those as well.

Please keep track of former PR, used diamond-3

Instead of copy pasting in Diamond-3, that should of been installed as a library then we create UbiquityDiamond.sol and UbiquityDiamondManager.sol for the part we need to modify/adjust. This implementation creates a lot of unnecessary clutter and contracts into the repo that we have no intention of maintaining

Instead of copy pasting in Diamond-3, that should of been installed as a library then we create UbiquityDiamond.sol and UbiquityDiamondManager.sol for the part we need to modify/adjust. This implementation creates a lot of unnecessary clutter and contracts into the repo that we have no intention of maintaining

Yeah, maybe you are right, but please clarify what are unnecessary parts and contracts

Seems I should add the general diamond test cases based on ManagerFacet again so that there is no missed test cases

1/ have you checked out both tx hashes I attached? If yes, what's your answer about them?

Yeah, checked both of them.
if you remove --verify flag in forge create command and try, it will be successfully deployed
But when including it and submitting contracts to goerli, that kind of thing happens
I guess it happens because of rpc endpoint. Please try with infura rpc endpoint

2/ Did upgrade tx happen on the contracts you deployed? Provide us a proof.

No, I didn't. But I'm sure it would be happened, Now adding test cases for that


Constrain a11y word nav to the virtual bottom

Use the virtual bottom in ConHost to speed up a11y word navigation performance. GetTextBufferEndPosition() is also used by by search; it'll also get a minor performance boost.
PR Checklist

[X] Closes #5243

Detailed Description of the Pull Request / Additional comments
#7789 uses GetLastNonspaceCharacter() to determine where the last word is and exit early on a UIA::Move by word call. The change in this PR passes the virtual bottom over to GetLastNonspaceCharacter() to make that search a bit faster.
Tested word navigation on Host.exe using NVDA.
WPR Traces
The percentages below represent the weight that a function call had. The
test scenario included moving by word on the CMD welcome message until
the last word was reached. Inspect.exe was used to limit any additional
calls that are generally performed by a screen reader.

function
current
branch

UIA:Move
29.52%
28.43%

There is an improvement of about 1% in a release build of ConHost.

Thanks @carlos-zamora!

Any updates on this PR? Thanks.

I came by to look at this but three things stick out:

Outstanding comments by @DHowett-MSFT that don't look answered yet.
The x64 build is failing.
@carlos-zamora hasn't attached the WPR traces before/after he said he was going to.

So I'm going to skip reviewing it for now and come back later hoping some of those progress.

Do you intend to merge this in time for 21H1? It would mean a lot for NVDA. Thanks!

@codeofdusk Yeah! I've had bad luck the last few days where suddenly my internet goes down or my machine refuses to run the tracing app haha. I'm hoping to get this in this week or early next, but I'm trying to prioritize closing some of the other Windows Terminal bugs first.

@codeofdusk sorry about the confusion here: the 21H1 milestone end date was set incorrectly. Things are a lot less dire than “we need to fix all the UIA bugs in the next two days.”

Excellent news!

@carlos-zamora Any updates on this PR? Were you able to address the issues with your machine and outstanding review comments? Thanks!

@carlos-zamora Any updates on this PR? Were you able to address the issues with your machine and outstanding review comments? Thanks!

Now that Build is over, time to get back to this one 😊. Getting the traces and fixing the build today.

@carlos-zamora Any updates on this PR? Were you able to address the issues with your machine and outstanding review comments? Thanks!

Now that Build is over, time to get back to this one 😊. Getting the traces and fixing the build today.

And another problem came up with recording the traces. Baby steps. This'll get done soon, sorry Bill haha.

Thanks for looking into it!
Also, could you please look into #5481?

@codeofdusk We'll look into it -- Carlos is assigned, but that does not indicate that it's in his immediate list of things to look at. We're finally coming off the v1 release cycle for Terminal and will have time in the future to evaluate conhost issues.

What is the current status of this PR?

Also, how does this PR relate to #6986?

Also, how does this PR relate to #6986?

Here, we're making it easier to find the last non-space character (and the last word, by extension).
#6986 is an architectural change to treat the last non-space character as the end of the buffer. This change should make it a bit easier to accomplish that in a performant way.

Closing. See https://github.com/microsoft/terminal/issues/7000#issuecomment-809550014 for more details.
I'll revive this PR once I actively start working on UIA again.


Splitting functions into separate files

Due to the number of implemented functions, the code starts getting a little difficult to parse. Would you be interested in splitting the functions into separate files? It's not necessary, but I feel like it would help encourage contribution.
Great project btw.

It seems pretty weird to split the document into multiple separate files since it's a collection of functions and methods. Which functions would go with which? Also, it's nice to have only one file to implement it if we don't want to use npm.

Not weird at all. Take a look at ramda. Also why would you not use a package manager? I don't think maintainers should care about that scenario.

Perhaps a compromise would be to sort the methods alphabetically?

Great idea @BrainMaestro! I would like to do that with the tests as well.
Feel free to submit PR 😃

PS: Closed because I want to keep open issues to a minimum

@ecrmnn Since @mikebronner does not know how to do testing might be out of scope.  But the request indeed a good one.   I am about to jump in with both feet and get my hands dirty.

@mikeerickson This is @BrainMaestro 's inquiry. :)

@BrainMaestro whoops, sorry see above :-)
cc: @mikebronner


fix slider ticks positioning and allow easy user level customization

Description
With a little bit different CSS treatment the slider ticks are now positioned correctly.
But pay attention, the strategy stands on the tick-size property being equal to track-size which then both turned to CSS variables by vuetify.
The user can override CSS (variables) to change the appearance of the ticks. Example added.
This change doesn't solve the problem with count of the ticks.
Related Issue(s)
There's no direct issue rised but it is closely related to https://github.com/FlowFuse/node-red-dashboard/issues/1036
Checklist

[ X] I have read the contribution guidelines
[ ] Suitable unit/system level tests have been added and they pass
[X ] Documentation has been updated

[ ] Upgrade instructions
[ ] Configuration details
[ ] Concepts

[ ] Changes flowforge.yml?

[ ] Issue/PR raised on FlowFuse/helm to update ConfigMap Template
[ ] Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

Labels

[ ] Includes a DB migration? -> add the area:migration label

Could you share a before/after please Endel? I made improvement to tick sizings here last week, which may have been missed: https://github.com/FlowFuse/node-red-dashboard/pull/1014

Before:

After:

Issues with before:

Last tick is rendered where first tick should go
Tick shabe is stretched from circle thus not sharp
Colors for active part make tick disappear outside of track
Customizing via CSS is nightmare cos the positional tretment is not created wisely

I'll try to explain my choices a bit more detail.
The vuetify original provides ticks on top of the track and with color chage for filled part of the track. Filled color if match with the grop background dosen't let ticks to be just extended. So the most easy will be to change filled color. (Which you did) But with that change the original is not possible anymore and it is way too complicated to get that original look and feel back with CSS override.
Also as the user can configure color for track - the ticks still use base color. Some may call it a bug. I would.
There's not any nice way to provide configuration for ticks cos it makes config way too overcrowded and complex.
I do understand your intention to make ticks more prominent and render out from track by default. But as it steals the vuetify original behavior - for users - do they get more ore less now? CSS override is common practice. It doesn't take too much of time when users will ask for possible solutions to make something look a bit different. (For every bit of DB, not just the slider) And then it will be better if it can be done easily.

I do understand your intention to make ticks more prominent and render out from track by default. But as it steals the vuetify original behavior - for users - do they get more ore less now? CSS override is common practice. It doesn't take too much of time when users will ask for possible solutions to make something look a bit different. (For every bit of DB, not just the slider) And then it will be better if it can be done easily.

Very fair point, it would be nice for us to show taller ticks by default as per:

But still allow for the overrides, I'm not a fan of the circular ticks. Also need to check how the new color properties on the Slider are impacted here.
Also worth noting in your docs about v-slider-track__tick--filled which covers any ticks beneath/left of the thumb and I think makes it more visually appealing.

Default tick size: width - half of track size

Half of the track would be large ?

Ignore the colors and numbers but for concept of tick sizing defaults
Track width - 4px
Filled track width - 6px
Ticks - 12px / 2px

looks good to me @hotNipi - thanks for the work :+1:

Some CSS rules turned out quite complicated so if needed, can explain in detail any time.


sqldatabasechain result incomplete

Issue you'd like to raise.
db_chain = SQLDatabaseChain(llm=llm, database=db, prompt=PROMPT, verbose=True, return_intermediate_steps=True, top_k=3)
result = db_chain("Make a list of those taking the exam")
result['result']  is incomplete，why ？token ?
Suggestion:
No response

I check carefully again，SQLQuery is right,SQLResult is right,but Answer  incomplete..

is anybody here?why not answer me again?

hello @xvgaoxing2023 does your result automatically truncated? I got the same issue

@theshook hi，yes,i am confused，why the  result automatically truncated

I have the same problem. The response from LLM is truncated.

In my case, I solved this by setting the 'max_string_length' parameter when creating 'SQLDatebase'
db = SQLDatabase.from_uri(
    ...
    max_string_length=10000,
)


Potential bug: Error! Couldn't find a deployment to alias

I just upgraded to now 11.1.4. I have the following config in my package.json:
  "now": {
    "alias": "trevordmiller.com",
    "env": {
      "NODE_ENV": "production"
    },
    "scale": {
      "sfo1": {
        "min": 1,
        "max": "auto"
      }
    }
  }

After upgrading, I run now then now alias. Previously this would alias the latest deployment to my production domain. Now it gives me this error:
Error! Couldn't find a deployment to alias. Please provide one as an argument.
I know I can fix this with now alias {deployId} mydomain.com, but I have automation in place in CI for the alias. So, can the previous behavior of now alias using the latest deployment by default be restored? Or is there a way to automate getting the latest deployment ID for the now alias {deployId} mydomain.com command?

Hi @trevordmiller !
When you provide no arguments we still try to find the lastest deployment for the project. The fact that none is found means that you either are in the wrong context (maybe your user instead of your team?) or that we can find no deployments for the app name. Please make sure this points and if it still doesn't work, ping me in ZEIT Community Slack:

The app name is the same as the one you have in package.json or the directory if there is none.
You are in the right context. Verify that the deployment exists by running now ls <appname>.

Also, take into account that you can get the deployment url with now alias $(now deploy) mydomain.com

@javivelasco I've been using now since when it was released so maybe I haven't upgraded something correctly over all the releases. I'm on 11.1.4 currently. The app name is the same as the name in the package.json. Is the undefined in Deploying ~/repos/trevordmiller-website under undefined here the problem? Is that the context? How do I fix that?

Here is my project repo if it helps (now config is in the package.json):
https://github.com/trevordmiller/trevordmiller-website
Thank you for your time.

The problem is that your configuration is messed up for some reason. The fact that we are printing undefined as the context when you are deploying means that ~/.now/config.json is corrupt. The deployment still work because the requests are authenticated using your token in ~/.now/auth.json but we can't find the deployment because we have to use for example the teamId to know if you are currently in a team context.
You can logout and remove ~/.now/config.json, then login back and everything should work like a charm. Remember to always manage your config using the CLI instead of manually. In any case, I'm curious about how did you get to that situation, it might be related to #1299
Let us know if it works after doing this.

@javivelasco Doing now logout && rm ~/.now/config.json && now login solved the problem; my context is no longer undefined.
I've never manually touched ~/.now/* so I'm thinking my config got in a weird state because I've been using now for so long (since it was first released) and maybe something got corrupted along the way with the upgrades.
But no worries, it's fixed now. Thank  you!