Important: Docker is sunsetting Free Team organizations

I got the below email from Docker, I believe regarding the artichokeruby organization on Docker Hub:

Docker is sunsetting Free Team organizations
Free Team organizations are a legacy subscription tier that no longer exists. This tier included many of the same features, rates, and functionality as a paid Docker Team subscription.
After reviewing the list of accounts that are members of legacy Free Team organizations, we’ve identified yours as potentially being one of them.
If you own a legacy Free Team organization, access to paid features — including private repositories — will be suspended on April 14, 2023 (11:59 pm UTC). Upgrade your subscription before April 14, 2023 to continue accessing your organization.
If you don’t upgrade to a paid subscription, Docker will retain your organization data for 30 days, after which it will be subject to deletion. During that time, you will maintain access to any images in your public repositories, though rate limitations will apply. At any point during the 30-day period, you can restore access to your organization account if you upgrade to a paid subscription. Visit our FAQ for more information.

Details

 Screenshots of FAQ

 Screenshots of Pricing

The existing artichokeruby Docker Hub team has two users: me and a CI machine user.
I don't intend to buy a team subscription, which has a minimum of 5 seats.
@b-n Would you be able to take a look into what it would take to transition this repository to publish to the GitHub Container Registry? I'm not sure if this is relevant, but @artichoke is a paid GitHub Teams organization.

Yep, I'll have a look see!
Just a quick note from the beginning, this would immediately limit the people using these packages to having a github account: https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry#authenticating-to-the-container-registry (e.g. they need a PAT with read:packages at least).
I'll do a bit of research though, and see what migration options there are though!

Linking:

https://github.com/docker/hub-feedback/issues/2314
https://news.ycombinator.com/item?id=35154025
https://news.ycombinator.com/item?id=35166317

That's a fairly big oof.
Good news, publishing to github packages looks fairly straight forward. I'll prep some PRs in the coming day(s).

If, for some reason, the Docker-Sponsored Open-Source Program isn't working out, we are happy to help you out. We operate a Harbor-based container registry as a service that has many benefits over most of the other registries out there.
There are also features regarding containerized image distribution, that might be valuable too as well.

I forgot to mention, all of this is documented here: https://github.com/artichoke/project-infrastructure/blob/trunk/docs/docker-hub-accounts.md

The approach mentioned in this article is pretty cool. I think we could setup such a facade using an empty S3 bucket, CloudFront and CloudFront functions.

https://httptoolkit.com/blog/docker-image-registry-facade/
https://github.com/httptoolkit/docker-registry-facade

Oooo these are all nice things.
Curious, for the interim, do you want to apply for DSOS? It at least stops the immediate issue.
I think this ticket should still stay open though. The idea of "host your own registry" sounds fairly trivial if I'm looking at what they did in that link. My immediate thoughts are that the redirects in theory shouldn't be needed at all so long as the hashed object can be delivered directly from the /v2/library/nginx/blobs/sha256:$HASH which should be achievable by storing the objects directly like that in S3 :thinking:
What was your thought/reasoning on using the CloudFront functions? I can see a need for returning the correctly formatted HEAD result with the correct hash etc. I assume knowing which "tag => hash" could just be another object stored in s3 too, and using this cloudfront function for reformatting the response. Yeah?

FYI, Started working on a Cloudfront backed S3 Registry here: https://github.com/b-n/s3-docker-registry
If this is sustainable and if Artichoke would like it, then I'd be happy to bring this into Artichoke repos (terraform module?) instead of my private github.

Been chatting with @b-n on discord. His CloudFront + S3 docker registry looks pretty nice. Together with docker buildx and some Python glue this looks pretty maintainable, cheap, and allows us to own our infra.
I've been purging images from Docker Hub to ensure I don't leave anything behind when they lock the image. As of 2 hours ago, the artichokeruby/artichoke repository on Docker Hub had 3502 tags.
I went to https://hub.docker.com/repository/docker/artichokeruby/artichoke/tags?page=1&ordering=last_updated and deleted all tags from 2020, 2021, and 2022. According to the UI all of these tags were "Inactive Image".
As of now, the repository on Docker Hub has about 150 tags.
I plan to push a PR which stops tagging images with the Git SHA of the artichoke/artichoke repo the Artichoke binaries are sourced from which should cut down on the spam.
TODO: update the README on Docker Hub to redirect to the new place we host the images.

Pressure is slightly off, but let's get this private registry thing working!
In theory we can do things like docker pull docker.artichoke.org/nightly:latest (e.g. just calling the image nightly or stable). Not sure if we want to, but we could!
I'm curious, would you want to move the ownership of my s3-docker-registry module to artichoke? I always trust packages/modules/etc more when they are owned by an organization rather than an individual. Doesn't bother me either way though.

Docker walked back these changes. Closing this as no longer required.


Fix HugeROM IC2 EXPSEL decode

Fixes https://github.com/rosco-m68k/hardware-projects/issues/43, see that issue for details.

Looks great, I had noticed a slowdown running memcheck past the 2MB mark, bus errors explains it :)
Thanks @0xTJ 👍


TypeScript: can't infer prop names and types when contains any type prop

Version
2.5.2
Reproduction link
https://github.com/wonderful-panda/vue-sandbox
Steps to reproduce
Compile this code by tsc
import Vue, { VNode } from "vue";

const NgComponent = Vue.extend({
    name: "NgComponent",
    props: {
        foo: { type: String },
        bar: {}    // any types are accepted
    },
    render(h): VNode {
        return h("div", this.foo);
    }
});

or clone from repro link and npm install && npm run build
What is expected?
Successfully compiled.
And this.foo is treated as string, this.bar is treated as any.
What is actually happening?
Failed to compile.
src/index.ts(10,30): error TS2339: Property 'foo' does not exist on type 'CombinedVueInstance<Vue, {}, {}, {}, Readonly<{}>>'.

Workaround:
import Vue, { VNode } from "vue";

Vue.extend({
    name: "Workaround",
    props: {
        foo: { type: String },
        bar: {} as (() => any)   // any types are accepted
    },
    render(h): VNode {
        return h("div", this.foo);
    }
});

Indeed it does not work if it has any typed props definition. I'm not sure about the reason, though.
@HerringtonDarkholme @DanielRosenwasser
I would appreciate if you can look into it.

I'm currently not by a computer but I'll guess a bit. Right now it's probably an assignability error for props in general, but we can do better there in future versions of TypeScript. I'm currently on a branch where this works better.
The thing is that even on this branch, the best inference I can get is {}.

With #6850 , you can do this.
import Vue, { VNode } from "vue";

Vue.extend({
    name: "Workaround",
    props: {
        foo: { type: String },
        bar: {} as Prop<any>
    },
    render(h): VNode {
        return h("div", this.foo);
    }
});

I think inferring any from empty prop type is too lenient. A lot of options will have empty field, which results in empty type parameter. If we make inference default to any, there will be too many errors hidden by unintended any.

Closing this one - please keep an eye on #6856 (to be merged for 2.6)


ioSender.resources.zh-CN.csv   Translation of

ioSender.resources.zh-CN.csv

Only translated one, I don't know if this is done correctly

Some items are not translated because I don't know what they mean


Migrate to TensorFlow r1.0 and Python 3

This pull request eliminates the warnings with the latest TensorFlow version. I have also refactored some functions for better readability. The code now requires Python 3, which lead to further simplifications.

Hi,
Thanks for that pull request! I had not been maintaining this repo. I'll go through it and merge. Did you evaluate the results for machine translation?

Thanks! In my refactoring I mostly focused on the prediction model. The training semantics has not changed.


Why flux's basePath is different from webMvc ,I request them over spring gateway ,the flux is always "/",but the mvc is service name!

this is flux server

this is mvc server

Assuming you figured out why it was doing that.


MenuItem keyboard focus style/focusVisibleClassName not working in Safari

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Steps:

In Safari 17.1.2 go to: https://mui.com/material-ui/react-menu/#basic-menu
Click on "Dashboard" button
Press down arrow on the keyboard

Current behavior 😯
No focus state is shown in menu.
Expected behavior 🤔
Focus state is shown in menu (Chrome, firefox).
Context 🔦
I want to show the focus state of the menu item in Safari.
Your environment 🌎

npx @mui/envinfo
System:
  OS: macOS 12.7
Binaries:
  Node: 20.2.0 - ~/.volta/tools/image/node/20.2.0/bin/node
  Yarn: 1.22.18 - ~/.volta/tools/image/yarn/1.22.18/bin/yarn
  npm: 9.6.6 - ~/.volta/tools/image/node/20.2.0/bin/npm
Browsers:
  Chrome: 120.0.6099.62
  Edge: 119.0.2151.97
  Safari: 17.1.2
npmPackages:
  @emotion/react: ^11.11.1 => 11.11.1
  @emotion/styled: ^11.11.0 => 11.11.0
  @mui/base:  5.0.0-beta.26
  @mui/core-downloads-tracker:  5.14.20
  @mui/icons-material: ^5.14.19 => 5.14.19
  @mui/material: ^5.14.20 => 5.14.20
  @mui/private-theming:  5.14.20
  @mui/styled-engine:  5.14.20
  @mui/system:  5.14.20
  @mui/types:  7.2.10
  @mui/utils:  5.14.20
  @types/react: ^18.2.42 => 18.2.42
  react: ^18.2.0 => 18.2.0
  react-dom: ^18.2.0 => 18.2.0
  typescript: ^5.3.2 => 5.3.2

Hey @gkiely, thanks for the report.
Safari (and MacOS) doesn't have Keyboard navigation activated by default, you can activate it from system settings. Have you tried that already?
Activating this option solves the issue on my side if you get to the dashboard button by keyboard navigation:
https://github.com/mui/material-ui/assets/16889233/2f94f80e-213f-4e5b-b82d-120a099af5e6
If you click the dashboard button, the issue persists, but it can be solved by activating the "Full Keyboard Access" option:
https://github.com/mui/material-ui/assets/16889233/38d22235-a548-40e1-8302-8b9cf561b3ac
I would say this is an issue on Safari rather than Material UI. Please let me know if it is solved on your side, and feel free to reopen if the issue continues.


#127 - Fix hasData return while data is false when casted as a boolean

Like @khylias figured out the hasData function from the ModalComponent works badly.
Unfortunately the function return false with some others values than 0  please see this fiddle
This PR fix the issue

Actually, I learned yesterday that null could be considered as a value.
By default, if you define a property without value in a class it is initialized to undefined.
IMO we should test only !== undefined, what do you think?

I only saw your last comment now. Yep, undefined is the essential check to perform. Let's do this! 👌

What about this issue ? https://github.com/biig-io/ngx-smart-modal/issues/129
And it seems to that we can close this issue 😄 #127


feat: add ability to change label on accessory bar

Feature Request
Ionic version:

[x] 4
Describe the Feature Request
Right now, when displaying the accessory bar Keyboard.setAccessoryBarVisible({isVisible: false}); you cannot change the "Done" label.
This is an accessory bar:

Thanks for the issue. This feature would be managed by the Keyboard Plugin, so I recommend you add this feature request on the Keyboard Plugin repo: https://github.com/ionic-team/cordova-plugin-ionic-keyboard.
I am going to close this as this is not something that would be managed by Ionic Framework. Thanks!


+= does not work on multi-dim arrays in some situtations.

+= does not work on multi-dim arrays in some situtations.
Steps To Reproduce
compare
https://dev.deck.net/99b86b4650a88350374349224b817a92
to
https://dotnetfiddle.net/kHmyZb
using System;
using System.Collections.Generic;

public static class Program
{
    public static void Main()
    {
        var a = new int[3, 3];

        a[0, 0] = 1;

        for (var y = 0; y < (a.GetLength(1) - 1); y++)
            for (var x = 0; x < (a.GetLength(0) - 1); x++)
                a[x + 1, y + 1] += a[x, y];
        List<int> list = new List<int>();
        string s = "";
        foreach (var i in a)
            s += i;
        Console.WriteLine("Expected: " + 100010001);
        Console.WriteLine("Actual: " + s);
        Console.WriteLine("100010001"  == s ? "Passed" : "Failed");
    }
}

Hi @michaelcheers – Thanks for this defect report and excellent sample demonstrating how to reproduce. We will investigate and attempt to fix right away.

Merged upstream. The fix made it to Bridge 16.5.0!


Screen Reader support

Thanks for a very useful component! I am making use of this project to provide star ratings on a project and we're aiming for a great experience for screen reader users. Currently, when a screen reader hits the star ratings it just skips it.
My quick solution on my app was to do this:

    This clinic is rated {{ rating }} stars out of five.
    <star-rating v-model="rating"
                       :show-rating="false"
                       active-color="#d6612d"
                       :star-size="20"
      ></star-rating>

By adding that span, the screen reader gets a solid sensible experience, perhaps it would be a good idea to have something like it by default that is visually hidden but translates the component contents to something that verbally makes sense?

Thanks for this, I'll see if I can get this in in the the next release. :)


CSS Overflow Fix Needed on Language selector.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to homepage
Click on avatar in the top right of the UI.
Set language to "Bottom".
Go back to parent dropdown by clicking on the back arrow.
See error

Expected behavior
The icon/s should display and the extra text should be overflow: ellipsis or something like that or auto adjust the width.
Screenshots
https://cdn.discordapp.com/attachments/554612253639311360/841651129741606934/unknown.png
What device are you on?
Windows 10, Chrome 90
Additional context
Occurs on multiple languages.

Nah man, if you're using a meme language, bugs should happen. Especially with something like this. (just my opinion tho)

Occurs on multiple languages.

This occurs on multiple languages.

oh


Bug when trying to set params for a console command

I get this error that says yii\console\Exception: Unknown option: --param1 in C:\lamp\www\klusster\vendor\yiisoft\yii2\console\Controller.php on line 88. This happens when i supply --param1 as a parameter to the console command.
The error is from the code below, I suspect "(!is_int($name)" is suppose to be "(is_int($name)". Which will imply that numbers are not allowed when setting the paramerers.
/**
     * Runs an action with the specified action ID and parameters.
     * If the action ID is empty, the method will use [[defaultAction]].
     * @param string $id the ID of the action to be executed.
     * @param array $params the parameters (name-value pairs) to be passed to the action.
     * @return integer the status of the action execution. 0 means normal, other values mean abnormal.
     * @throws InvalidRouteException if the requested action ID cannot be resolved into an action successfully.
     * @throws Exception if there are unknown options or missing arguments
     * @see createAction
     */
    public function runAction($id, $params = [])
    {
        if (!empty($params)) {
            // populate options here so that they are available in beforeAction().
            $options = $this->options($id);
            foreach ($params as $name => $value) {

                if (in_array($name, $options, true)) {
                    $default = $this->$name;
                    $this->$name = is_array($default) ? preg_split('/\s*,\s*/', $value) : $value;
                    unset($params[$name]);
                } elseif (!is_int($name)) {
                    throw new Exception(Yii::t('yii', 'Unknown option: --{name}', ['name' => $name]));
                }
            }
        }
        return parent::runAction($id, $params);
    }

What is your controller action signature?

public function actionBar($param1, $param2)

how exactly do you call the action?

See the docs: http://www.yiiframework.com/doc-2.0/guide-tutorial-console.html#arguments

When i use yii foo/bar --param1="test" --param2="testing" it fails but yii foo/bar --0="test" --1="testing" on the other hand works

the correct syntax would be ./yii foo/bar test testing.
The second syntax should fail though...

Okay, I get it now. Thank you!

Hi! I've some problem with console in yii2 advanced.
I tried to run console commands:
"php yii cron/test" or "./yii cron/test"
and after that I got the same error "Error: Unknown command cron/test".
I tried to search some information in the internet, but I not found anything.
I create my own console controller and test action by this link http://www.yiiframework.com/doc-2.0/guide-tutorial-console.html#create-command


[Beta] Solution doesn't pass for challenge "React: Use a Ternary Expression for Conditional Rendering"

Challenge Name

https://beta.freecodecamp.org/en/challenges/react/use-a-ternary-expression-for-conditional-rendering
Issue Description

Correct solution doesn't pass. Behavior of "Preview" page matches challenge description but these tests don't pass:

When a number of less than 18 is entered into the input element and the button is clicked, the button's inner text should read You Shall Not Pass. };
When a number greater than or equal to 18 is entered into the input element and the button is clicked, the button's inner text should read You May Enter. };
Once a number has been submitted, and the value of the input is once again changed, the button should return to reading Submit. };

Browser Information

Browser Name, Version: Chrome, Version 64.0.3282.140 (Official Build) (64-bit)
Operating System: Windows 10
Mobile, Desktop, or Tablet: Desktop

Your Code

const inputStyle = {
  width: 235,
  margin: 5
}

class CheckUserAge extends React.Component {
  constructor(props) {
    super(props);
    // change code below this line
    this.state = {
      input: '',
      userAge: ''
    };
    // change code above this line
    this.submit = this.submit.bind(this);
    this.handleChange = this.handleChange.bind(this);
  }
  handleChange(e) {
    this.setState({
      input: e.target.value,
      userAge: ''
    });
  }
  submit() {
    this.setState({
      userAge: this.state.input
    });
  }
  render() {
    const buttonOne = <button onClick={this.submit}>Submit</button>;
    const buttonTwo = <button>You May Enter</button>;
    const buttonThree = <button>You Shall Not Pass</button>;

    return (

Enter Your Age to Continue

        <input
          style={inputStyle}
          type="number"
          value={this.state.input}
          onChange={this.handleChange} />

        {

          /* change code here */
          this.state.userAge.length > 0 ? (this.state.input < 18 ? buttonThree : buttonTwo) : buttonOne
        }

    );
  }
};

Screenshot

this.state.input is a string, and you cannot compare it to the integer 18.
In JavaScript, strings are compared lexicographically (you can think of this as a fancier version of alphabetical ordering, the difference being that alphabetic ordering only covers the 26 characters a through z). Here are a few examples:
// true (we follow alphabetical order here)
console.log("a" < "b")

// true (since the first characters of both strings are equal, we look at the next one, and "a" < "b".)
console.log("aa" < "ab")

// true
console.log("1" < "3")

// false, because "2" comes after "1".
console.log("2" < "10")

I'll take a look at the tests and open a PR.

I've opened a pull request to fix these tests.

@jkao1 thanks for explaining and for fixing :+1:

@antonshevyrin anytime :smiley:

Hi guys, thanks for taking the time to report and fix, we will get this reviewed soon.
Happy contributing!

Hi, why this solution is not working.
It's working fine if I enter a value greater than 18 but fails when I enter a value lesser than 18.
{
      (this.state.userAge >= 18) ? buttonTwo : buttonThree && buttonOne
}

@Randore  it won't work for values lesser than 18 as it this is an invalid expression (buttonThree  && buttonOne)
Instead, try the below solution :
{ (this.state.userAge == '' ? buttonOne: (this.state.input < 18 ? buttonThree : buttonTwo)  }


User unlocked screen: where to find?

Good morning.
I used ALEAPP on a full file system extraction. I was wondering where I can find the events related to the action of the user unlocking screen.
Could you indicate in which section o f the html report generated from ALEAPP I should find it?
Thank you in advanced and congratulations for this great work

Hello.
Any advice or information? I would like to inform you that is still relevant to me to get this information.
Kind regards

@igiuseppem Digital Wellbeing > Events is probably your best bet


how to display Glyphicons Halflings font family in PDF

Iam using jsPDF for generating HTML to PDF. Iam using bootstrap.css . however used style font-family: 'Glyphicons Halflings'; is not shown in generated PDF.  PFB Code snippet:

Duplicate of #115


Add a PATCH API

In the route routes/io, add a new API which can be used to test HTTP PATCH method.
This issue will be assigned only after you send across a working PR.

Hello, I would like to work on this issue. As part of GSSoC 2024 Contributor can you assign me the issue

WIll you asign this issue under GSSoc 24 I would like to contribute on this

assign me

@ashitaprasad I would like to work on this issue, Can you please assign me.

hey @animator can you assign this to me. I can do it


update to use heroku buildpacks

As of May (six months ago), Heroku directly supports multiple buildpacks.

cc: @greypants

:+1: Thanks!


bugfix for SetSelected (not it doesn't work with invisible, readonly,…

… disabled and not checkbox/option/radion)

No) Checked it, in this place it would treat invisible successfuly (but do
nothing about it)
Viele Grüße
Alexei Vinogradov
On 1. Februar 2018 at 21:55:22, Boris Osipov<EMAIL_ADDRESS>wrote:

@BorisOsipov commented on this pull request.
In src/main/java/com/codeborne/selenide/commands/SetSelected.java
https://github.com/codeborne/selenide/pull/684#discussion_r165484709:

@@ -13,9 +13,23 @@
public WebElement execute(SelenideElement proxy, WebElementSource locator, Object[] args) {
boolean selected = (Boolean) args[0];
WebElement element = locator.getWebElement();

if (element.isSelected() ^ selected) {

 if (element.getAttribute("readonly") != null)

   throw new InvalidStateException("Cannot change value of readonly element");

if (!element.isDisplayed()) {

 throw new InvalidStateException("Cannot change invisible element");

}

Btw why element.isDisplayed() check needs here? Doesn't Selenide check it
automaticaly whatever?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/codeborne/selenide/pull/684#discussion_r165484709,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGQ4YEZAtbZgMtN2lNE6y-k1GH3m36T4ks5tQiS6gaJpZM4RveL2
.

@asolntsev please review


Update teleportation.ipynb for running on a real device

Changed section on running on real device to highlight fact that it does not work, and returns a job error.

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


More exciting end-of-level results screen

Maybe this could be done with like a scrolling restaurant receipt which gets some stuff stamped onto it at the end.
     thank you
   for  choosing
   armadillo pos
      ,(((^^,

------ -----------
183001 .... +16.20
            (16.20)
183002 .... +22.05
           (38.25)
183003 ... +215.15
          (253.40)
183004 ..... +1.70
          (255.10)
...
183015 .... +33.65
          (779.15)
183016 ..... +1.15
          (780.30)
lst ...... -121.30
          (659.00)
------ -----------
            659.00

boop! (receipt appears, pause for 1 beat)
taka-taka-taka-taka-taka-taka-taka (receipt scrolls by for 3 beats)
ka-ching!!

(scrolling abruptly stops, receipt shakes)
(dollar amount pops up in big text)
(your current money is made visible, non-abbreviated)

(wait 1.5 beats)
dgggt!

(money counts up for 1-7 beats)

(wait 3 beats)
dk dk dk dk (grading criteria are stamped for 1 beat; wait 4 beats)

dk dk dk (stamp out the player's triangle grade; wait 2 beats)
dook! dook! (stamp out the player's star grade; wait 2 beats)
dk dk dk dk (stamp out the player's triangle grade; wait 4 beats)
dook. dook. dook. (stamp out the player's final grade, slower)

(wait 3 beats)
(buttons appear when player hits a key)

Player should be able to skip to the end with a key, too

This is a priority 2 because the player will see this all the time, and the screen is currently just a big block of xolonium text. It needs to be improved in some way, even if we decide this full animated results screen is overkill.

A lot of the suggestions here were implemented. The two key things which need to change are the panel UI (it should look like a receipt) and the grade stamps. But, the end-of-level results screen looks pretty good as is.
The most crucial thing is that when we implement the grade system, they need to be consistent, (we need to be able to display a 3-star grade), so this fix is essential for the grade system to work.

Maybe instead of explicitly rating the player on pickups/combos/boxes, it could just have something like "commendations" which appear if they do really well. The commendations could come in 3-4 categories so they'd just get a little "wow!" stamp showing they did well with pickups

The end-of-level screen also should communicate if you fail a boss stage.

The end-of-level results screen could rate you from 0-5 stars. Maybe it starts out by rating you from 1-3 stars, but once you earn a 3 star rating it unlocks the abillity to earn a 4 star rating. Then once you earn a 4 star rating, it unlocks the ability to earn a 5 star rating.

We originally had an idea for a receipt on the results screen, but I don't really like this anymore -- I think less is more.

I've designed a new 'Level Complete!' screen which looks like a torn-off receipt. It includes some fun food graphics, colorful stamps and an animated bar graph showing how you scored your points.


added some basic trig functions

For issue #180
sin, cos, tan, csc, sec, cot

Needs unit tests for checking proper functionality.
Needs unit tests for checking proper exception handling.
Should take parameters in radians, and not convert internally.
When in doubt (e.g. on parameters in radians or exception handling), check with Python's math module.
Should provide functions to convert degrees from/into radians, and probably something to wrap the argument back into +-pi as well.
How commonly used are the 1/sin, 1/cos, 1/tan things? First time I see them defined with own names (but this might be an intercontinental issue.)
Pull out the number of decimals places into a CONSTANT, or leave out altogether.
Should probably wrap the parameter into +-pi before proceeding.

Re cosecans et al.: Python's math doesn't seem to have them.
Feature requests:

Arcus functions
Hyperbolic functions
Area functions

This was needed for the SAS hack-a-thon.  Let's fix and merge...


_count=0 is not working

_count=0 returns the default number of resources.
Per FHIR spec, _count=0 should return summary (total) with no entries.

Looks fine to me? http://hapi.fhir.org/baseR4/Patient?_count=0

Closing this- Please feel free to reopen if you disagree


[office] Add links to release cycles

It is possible to use a changelogTemplate, but the template changed for Office 2021, and is not the same for Office 365 and all versions of Office for Mac. Moreover the fact that the service pack is stored in the release cycle name does not help. So it was more simple to specify each link.
This relates to #39, and I took the opportunity to normalize the page (#2124).

We should change to the pattern we use in mssqlserver releaseCycle without SPs and releaseLabel with SP. We can take care of that in a separate MR, merging this one.

Yep, done in #2599.


Hotfix email error

WHAT DOES THIS PR DO?

[x] Hotfix any emailing errors
[x] Modifications to the email message for the create inquiries

WHAT ISSUES ARE RELATED TO THIS PR?
Hotfix email error

Jira cards

PLAT-1167

HOW DO I TEST OUT THIS PR?
cd src/auth-service
npm run dev-mac

WHICH ENDPOINTS SHOULD BE READY FOR TESTING?:

[x] create inquiries

SCREENSHOTS?

Merging this PR in as I continue to fine tune the Create Inquiries Use Case. cc: @ja-odur @FaithDaka @Codebmk


#587 - Disable caching for packageRevisionResources

This change disables caching for packageRevisionResources in the porch controller.
Context -
The use case works for creating a PVS, but after someone creates a PVS, the controller logs are populated with errors saying the controllers cannot 'watch' the packageRevisionResources because there is no 'watch' function on the porch server side.
Example -
E0523 16:49:14.393038       1 reflector.go:140] pkg/mod/k8s.io/client-go@v0.26.9/tools/cache/reflector.go:169: Failed to watch *v1alpha1.PackageRevisionResources: the server does not allow this method on the requested resource (get packagerevisionresources.porch.kpt.dev)

This is a generated file based on kubebuilder directives in the controller go code. Please update those instead of this file.

This is a generated file based on kubebuilder directives in the controller go code. Please update those instead of this file.

Sure, I'll make the change.

/remove-approve

To my understanding the following is the issue:

controller-runtime runs caching clients by default, where the resources are list-and-watched by default. Because of this, when packageVariant controller runs the following line, the controller-runtime client starts watching the packageRevisionResources: client.Get
On the porch-server side, there is no implementation of watch for packageRevisionResources, only for packageRevisions. The porch-controllers gets back an error related to this from porch-server, and it gets printed out to the porch-controllers error log.

So the cache was never working, but instead of it failing back ungracefully to requesting the data every time, the cache is disabled by default.
An alternative solution could be that watch is implemented for packageRevisionResources. The metadata updates, including the resourceVersion is shared between the packageRevisions and packageRevisionResources. But because packageRevisionResources include all the data from the kpt package itself, building an in-memory cache of those might be quite expensive. And as I understand, there is no cache eviction policy, and packageRevisions are edited for a short time, then they become dormant as they get approved. All in all, I don't think it's a good idea for any client to cache packageRevisionResources.

To my understanding the following is the issue:
1. controller-runtime runs caching clients by default, where the resources are list-and-watched by default. Because of this, when packageVariant controller runs the following line, the controller-runtime client starts watching the packageRevisionResources: [client.Get](https://github.com/nordix/porch/blob/07d5d898731592b234bae7e9e3d124df46415eab/controllers/packagevariants/pkg/controllers/packagevariant/packagevariant_controller.go#L766)

2. On the porch-server side, there is no implementation of watch for packageRevisionResources, only for packageRevisions. The porch-controllers gets back an error related to this from porch-server, and it gets printed out to the porch-controllers error log.

So the cache was never working, but instead of it failing back ungracefully to requesting the data every time, the cache is disabled by default.
An alternative solution could be that watch is implemented for packageRevisionResources. The metadata updates, including the resourceVersion is shared between the packageRevisions and packageRevisionResources. But because packageRevisionResources include all the data from the kpt package itself, building an in-memory cache of those might be quite expensive. And as I understand, there is no cache eviction policy, and packageRevisions are edited for a short time, then they become dormant as they get approved. All in all, I don't think it's a good idea for any client to cache packageRevisionResources.

Nice explanation. Thanks @nagygergo

/approve

/approve

/approve
/lgtm


Add support for k8s v1.21

Adds support for k8s v1.21 to sync-core-components command
Bump relevant versions to sync with latest referenced in EKS docs.

Integration testing: https://github.com/gruntwork-io/terraform-aws-eks/pull/341

Thanks for review! I confirmed this is working on the regression test branch of terraform-aws-eks, so going to merge this in.


Add myself as a maintainer for the datatheorem app security plugin

I'm adding myself here to be able to maintain this plugin in the future.

Can you please add @magneto_marc as a reviewer to verify my identity here?

Sorry, @superlama I forgot how the deployment process works. you should add your artifactory repository name instead and then, in order to deploy you will have to save  your encrypted credentials to your maven configuration https://www.jenkins.io/doc/developer/publishing/releasing-manually/