Wrong compiled colors

Hey!
I just tried your library because it supports the latest versions of tailwind out of the box.
However, I've had some issue with colors being wrongly generated.
I have the following styled component:
const SelectableRowContainer = styled.div<SelectableRowProps>`
  ${props => (props.selected ? tw`border-yellow-500` : tw`border-gray-300`)};
  ${tw`border-solid border-2 w-full cursor-pointer flex`};
`

When compiling it with Babel, it generates the following:
var SelectableRowContainer = _styledComponents["default"].div.withConfig({
  displayName: "SelectableRow__SelectableRowContainer",
  componentId: "sc-1rdb9br-0"
})(["", ";", ";"], function (props) {
  return props.selected ? {
    "--border-opacity": "1",
    "borderColor": ["#ecc94b", "rgba(236, 201, 75, var(--border-opacity))"]
  } : {
    "--border-opacity": "1",
    "borderColor": ["#e2e8f0", "rgba(226, 232, 240, var(--border-opacity))"]
  };
}, {
  "borderStyle": "solid",
  "borderWidth": "2px",
  "width": "100%",
  "cursor": "pointer",
  "display": "flex"
});

The --border-opacity variable  shouldn't be there. This generates the css: border-color: #e2e8f0", "rgba(226, 232, 240, var(--border-opacity)) which is invalid. I had the same issue with a background-color.
I'm using @tailwindcssinjs/macro 0.3.1 and tailwindcss 1.4.4

I just tried to disable the borderOpacity corePlugin and it seems to generate the styles properly if that's any help!

Yes, you are correct tailwind v1.4.0 added opacity utilities that changed the tailwind colors.
Pre v1.4.0
.border-white {
  border-color: #fff;
}

After v1.4.0
.border-white {
  --border-opacity: 1;
  border-color: #fff;
  border-color: rgba(255, 255, 255, var(--border-opacity));
}

As you noticed tailwind has now 2 border-color rules the first rule is there as a fallback when the browser doesn't support the CSS variable in the second rule.
Interestingly Styled-components doesn't support this syntax
"borderColor": ["#fff", "rgba(255, 255, 255, var(--border-opacity))"]
But it works fine in Emotion because its supports fallbacks.
Here is a sandbox I used to test this.
https://codesandbox.io/s/test-2-0piru?file=/pages/index.tsx
Not yet sure what the best solution to fix this would be since this is specific to styled-components.

Thanks for the information ! I'll open an issue on the styled components repository then!
I'm closing this issue as this is not an issue related to tailwind.

@tailwindcssinjs/macro now has support for custom output syntax.
This makes it possible to use @tailwindcssinjs/macro with styled-components or any other CSS-in-JS library with different syntax.
Check out the tailwindcssinjs plugin documentation

Thanks! I appreciate that!


Upgrading terraform version in all-in-one image

Changed terraform version from v0.12.29 to v0.13.6.
Added steps for installing plugins terraform-provider-ibm v1.21.2 and terraform-provider-null v2.1.2.
Plugin path is as per https://www.terraform.io/docs/cli/config/config-file.html#implied-local-mirror-directories

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Rajalakshmi-Girish
To complete the pull request process, please assign mkumatag after the PR has been reviewed.
You can assign the PR to them by writing /assign @mkumatag in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @mkumatag

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mkumatag, Rajalakshmi-Girish
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mkumatag]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Fixvarcalc

@evolvedmicrobe, what affect does this have on our zscore filter?

It changes everything.

do not pull this.

Aye sir, I wasn't =)

Moved to CC2


First note not played (again)

Every MIDI file I give to MIDI.Player.loadFile will skip playing the first quarter note.
For example, using "Test 1" (commented out here), only three notes are played back.
The file clearly has four:

Reproduced here (master branch as of 2016-01-19).

For what it's worth, I can't reproduce this behaviour with jasmid.

I'm seeing this issue, too.

I don't understand the code yet, but it looks like startAudio() is counting the time of meta events. Here are the first four events in my file. The second, third, and fourth events are all caught by the if test that contains the continue statement.
data[0]:
[
    {
        event: {
            deltaTime: 0,
            microsecondsPerBeat: 277778,
            subtype: "setTempo",
            type: "meta"
        },
        ticksToEvent: 0,
        track: 0
    },
    0
]

data[1]:
[
    {
        event: {
            deltaTime: 0,
            subtype: "trackName",
            text: "Title of tune",
            type: "meta"
        },
        ticksToEvent: 0,
        track: 0
    },
    0
]

data[2]:
[
    {
        event: {
            deltaTime: 0,
            subtype: "endOfTrack",
            type: "meta"
        },
        ticksToEvent: 0,
        track: 0
    },
    0
]

data[3]:
[
    {
        event: {
            deltaTime: 0,
            channel: 0,
            deltaTime: 0,
            noteNumber: 64,
            subtype: "noteOn",
            type: "channel",
            velocity: 64
        },
        ticksToEvent: 0,
        track: 1
    },
    0
]

I still don't understand exactly what is going on in startAudio(), but changing this line appears to fix the problem for me:
if ((queuedTime += obj[1]) < currentTime) { // was "less than or equal", changed to "less than"
I'd be willing to create a pull request, but I'd like to know how I can tell if I broke something else.

@paulrosen thanks for looking into this!
Can confirm that it fixes the issue, at least partly.
For some reason I can't get it to play inlined base64 data at all with the above change, but loading midi data from an acutal file works.
Could you tell me if something like this works for you?
window.addEventListener("load", function () {
  // works:
  // var data = "test.mid";
  // doesn't work:
  var data = "data:audio/mid;base64,TVRoZAAAAAYAAQABAMBNVHJrAAAARwD/WAQEAhgIAP9RAwehIAD/AwlOZXcgVHJhY2sAwHMAkDxkMoA8MIEOkDxkMoA8MIEOkDxkMoA8MIEOkDxkgT+APDAB/y8A";
  MIDI.loadPlugin({
    soundfontUrl: "soundfonts/",
    instrument: "acoustic_grand_piano",
    onprogress: function(state, progress) {
      console.log(state, progress);
    },
    onsuccess: function() {
      MIDI.setVolume(0, 127);
      MIDI.Player.loadFile(data, MIDI.Player.start);
      console.log(MIDI.Player.playing);
    }
  });
});

That code is pretty much straight from the example above.  Tarball here to make it easier.
@mudcube does this sound familiar at all?

I seem to recall that you need to strip off the first part of your midi
string. (I actually construct my own structure so I don't use the
referenced code any more.) If you step through it you'll see where it is
expecting to find the header.
On 3/13/16 8:41 PM, Ruben Maher wrote:

@paulrosen https://github.com/paulrosen thanks for looking into this!
Can confirm that it fixes the issue, at least partly.
For some reason I can't get it to play inlined base64 data at all with
the above change, but loading midi data from an acutal file works.
Could you tell me if something like this works for you?
|window.addEventListener("load", function () { // works: // var data =
"test.mid"; // doesn't work: var data =
"data:audio/mid;base64,TVRoZAAAAAYAAQABAMBNVHJrAAAARwD/WAQEAhgIAP9RAwehIAD/AwlOZXcgVHJhY2sAwHMAkDxkMoA8MIEOkDxkMoA8MIEOkDxkMoA8MIEOkDxkgT+APDAB/y8A";
MIDI.loadPlugin({ soundfontUrl: "soundfonts/", instrument:
"acoustic_grand_piano", onprogress: function(state, progress) {
console.log(state, progress); }, onsuccess: function() {
MIDI.setVolume(0, 127); MIDI.Player.loadFile(data, MIDI.Player.start);
console.log(MIDI.Player.playing); } }); }); |
That code is pretty much straight from the example above. Tarball here
https://demo.rkm.id.au/midi-js-bug.tar.gz to make it easier.
@mudcube https://github.com/mudcube does this sound familiar at all?
—
Reply to this email directly or view it on GitHub
https://github.com/mudcube/MIDI.js/issues/165#issuecomment-196089559.

--
Paul Rosen
--- Life is a musical, every once in a while
the plot stops and you start singing and dancing ---
http://abcjs.net/
http://zuzushotfive.com/
http://itsfloorplay.com/

@paulrosen thanks for the hint!

I'm seeing this issue, too.
@paulrosen, thank you, that fixed this issue for me, too :)

But this seems to double the last note .(

But this seems to double the last note .(

@paulrosen I get double notes with your suggested change. I think it's possible because startAudio loops over midi.data until it has queued 100 messages or reached the end of data, but there may be more events at the same time.
Next time startAudio is called it resumes at the same TIME, not the same place in midi.data, meaning some events may get repeated or skipped. If the time comparison is < then some events are repeated, if the time comparison is <= then some events are skipped.
I think it would be better if startAudio kept a track of where it was up to in the queue, but I guess that makes it more difficult to seek around. With the current algorithm it should not exit in the middle of events at the same time.

I had a problem with the suggested change where it would not really stop playing, seemingly bypassing the end of the data (probably some kind of doubling issue) and eventually crashing the browser, so I had to force it to stop by making the previously suggested line fix like this
    if ((queuedTime += obj[1]) < currentTime || currentTime >= midi.endTime) { // added endTime check

but that is possibly just my personal implementation that's behaving oddly, or something else that was interfering.

I was having problems where my MIDI files would not even play in the browser until I changed the code to what @vee-L proposed. Actually, even @paulrosen 's fix worked for me. Thanks @vee-L and @paulrosen for the fix. I'm not getting double notes playing my MIDI files.

I got a file where the first note isn't played, and if when apply the temporary fix, the song doesn't play - all notes are silent.
Here's the first measure of the file:
'data:audio/mid;base64,TVRoZAAAAAYAAQAHAMBNVHJrAAAAwwD/AytzY2h1dHpfd2VpaG5hY2h0c2hpc3RvcmllXzA1X2ludGVybWVkaXVtX2lpAP8BEkJ5IEhlaW5yaWNoIFNjaHV0egD/AiZUaGlzIHZlcnNpb246IENvcHlyaWdodCDvv70gMjAxMSwgSm9obgD/AhNBbGwgUmlnaHRzIFJlc2VydmVkAP8BIEdlbmVyYXRlZCBieSBOb3RlV29ydGh5IENvbXBvc2VyAP9RAwOFcQD/WQIAAAD/WAQGAhgIAP8vAE1UcmsAAABYAP8hAQAA/wMGVHJlYmxlAMBIAOAAQACwB38AsApAAJBIboEgkEgAIJBKboEgkEoAIJBIboEgkEgAIJBFboEgkEUAIJBGboEgkEYAIJBDboEgkEMAAP8vAE1UcmsAAABQAP8hAQAA/wMKU3RyaW5ncy1SMQDIMADoAEAAuAd/ALgKQACYPG4AmEVuhVCYPAAAmEUAMJg6bgCYPm4AmENuglCYOgAAmD4AAJhDAAD/LwBNVHJrAAAAUAD/IQEAAP8DClN0cmluZ3MtUjIAyjIA6gBAALoHfwC6CkAAmjxuAJpFboVQmjwAAJpFADCaOm4Amj5uAJpDboJQmjoAAJo+AACaQwAA/y8ATVRyawAAADgA/yEBAAD/AwpTdHJpbmdzLUwxAMsyAOsAQAC7B38AuwpAAJs1boVQmzUAMJs3boJQmzcAAP8vAE1UcmsAAAA4AP8hAQAA/wMKU3RyaW5ncy1MMgDMMADsAEAAvAd/ALwKQACcNW6FUJw1ADCcN26CUJw3AAD/LwBNVHJrAAAAIAD/IQEAAP8DC1RlbXBvIFRyYWNrALAHfwCwCkAA/y8A'

The temporary fix by @paulrosen causes timing problems later on. What is happening is that the first event with a time value of 0 is being processed, after which the currentTime variable is changed to the queuedTime - offset (which is 0.5 after this event). All following events that start at 0 will be caught in the continue loop because the currentTime == 0.5.
I'll see if I can think up a way to prevent it from skipping these notes. Maybe:
if (currentTime != 0 && obj[1] != 0 ) {    currentTime = queuedTime - offset; } // currentTime stays 0 if it was 0 and the time value of the event is 0

I had a somewhat lengthy fix to the timing issues but it is very particular to the application i was using this with. I can take a look at it again and see if I can make a PR with a more general version of it.

I'm getting this same issue and it's been open for almost a year. Are there plans for fixing this @mudcube ? This is a very cool project but I might have to use another library due to this issue. Please let me know what the plans are. Thanks!

I am looking for a fix for this too. @mudcube @vee-L


[Feature Request] Allow different SMTP host to send from

For me it appears 'localhost' doesn't actually work to send mail. Could we add a host (and port) option to this easily ?
https://github.com/mdomke/concourse-email-resource/blob/415c0bb1550f3377918037ba904af5f8e165f27a/opt/resource/out#L44

Done


Saving Cell Calls to the game file

Also, added Game.world:removeCall(name), Game.world:clearCalls() and adjusted the functionality of Game.world:replaceCall(name, index, scene)
BREAKING CHANGE!!!

I've done the changes you requested

@SylviBlossom The reason I kept it as registerCall instead of addCall is because we're also using registerAct while there is a removeAct.


QLC+5 and RPI5

Hi
Can QLC+ v5 be installed on Raspberry PI 5 on Debian OS ?.

In theory yes.
In practice I never tried it myself.
Just build from sources and find out.
Next time please use the forums to ask questions. Thanks


Website occasionally fails to load

This page isn't working
bitcoinops.org redirected you too many times.
Try clearing your cookies.
ERR_TOO_MANY_REDIRECTS

I just saw this problem again on Chrome/MacOS, so I do worry this is frequent enough and probably affects many/most/all users to warrant prioritizing it. What debugging information can I collect to help?
As for reproducibility on your end, my recommendation would be to excessively load the web page throughout the day like I do :)

If you're able to reproduce this, can you try loading the page with the 'Network' tab of Chrome's developer tools open, and see what redirects you're getting? I just get:

a 200 response when I navigate to bitcoinops.org
a 301 redirect to bitcoinops.org when I navigate to www.bitcoinops.org

(which is what I would expect).

I'm not sure how or where to access a log file (I did check "preserve log"). But here is a screenshot.

Here is perhaps a better view, as I cleared the log and refreshed the page, so this represents a single attempted page load.

I think I've fixed the DNS config that was causing this redirect loop.
@moneyball - please continue to monitor. I've only been able to repro this once (before updating the DNS config).

Seems to be fixed. Closing for now.
@moneyball - please reopen if you see this issue again.


Add instructions for building the Linux binary with Vagrant

I found the dependency notes in CONTRIBUTING a little light and I'd like to encourage others to get involved, so I put together some instructions for easily and repeatably getting a build environment set up for browserpass.
This is intentionally incomplete, it doesn't address installing chrome or doing a full build, I'm intentionally trying to just get this one thing done and hoping it's useful enough to others already.

would you mind adding the Vagrantfile as well?

I.. could do. I originally didn't as is a completely generic one written by vagrant init and I tend to shy away from committing anything machine-generated because the value is generally very low, and it just becomes a source of confusion later.
If you're still keen on having a Vagrantfile I'll throw it in - what would you like to do?

Well, I'll create my own Vagrantfile to test things. Thanks anyway.

Oh, maybe I'm misunderstanding you then. When I did this, I ran vagrant init minimal/xenial64 and it produced a Vagrantfile with everything necessary, no need to create one manually.
Is there a case where one would need to create one manually, or to edit the one generated? Wondering what I'm missing here. :)

You'd want a Vagrantfile which provisions the VM automatically as you described in the CONTRIBUTING.md.

I'll close this in favor of #106 🙂 Thanks for the contribution!


STORM-3135: Allow JCQueueTest to retry interrupting the consumer if t…

…he queue happens to be full
https://issues.apache.org/jira/browse/STORM-3135

The failure is unrelated, raised an issue at https://issues.apache.org/jira/browse/STORM-3137 to track.

@revans2 Good point. It probably doesn't hurt much in the real code, because the queue is only shut down when the worker or executor is shut down. It could be an issue for tests with LocalCluster though.
Took a different approach. As far as I can tell, there isn't a great reason that we need to interrupt the async loops through the JCQueue. We can just interrupt the threads directly, which is already happening during worker/executor shutdown.
I think the reason to interrupt via the queue is because the wait strategies now include use of LockSupport.parkNanos, so there's no guarantee that the executors will hit a Thread.sleep and throw an InterruptedException. We can fix this by checking manually for interrupts, either in the async loop, or in the wait strategies where we're calling parkNanos.

Sorry I don't think I submitted my nit.  We are calling isInterrupted in asyncLoop.  Is this going to leave the state of the thread as interrupted after we throw the exception.

Yes, the thread is still interrupted after the exception is thrown. I'll change it to use interrupted instead.

Fixed.


rename urlencode to snsUrlEncode

The custom urlencode() method creates a conflict with Lucee's built in urlencode() resulting in an AWS signature mismatch.

@gpickin can you please revise this.


Deployed Backend in Render and Updated README.md for Local Setup

Pull Request
Title
Deployed Backend in Render and Updated README.md for Local Setup
Description
This PR includes the deployment of the backend on Render, along with updates to the README.md file to include steps for setting up and running the Flask backend locally. This ensures clarity for contributors or users who want to run the project both locally and via the deployed version.
Related Issues
#93
Changes Made

Deployed the Flask backend to Render for remote access.

Updated the README.md file to include instructions for setup and running the backend locally.

[x] I have tested the changes locally

[x] Documentation has been updated (if necessary)

[x] Changes are backward-compatible

@manikumarreddyu  I am  one of the program manager. For readme only level1 is need to give and you have given  level1 for this  contributor in another pr. No level label for this. @MastanSayyad

hi @MastanSayyad ..we had already discussed about this thing..i hope you rememeber..three PRs are like this...please let me know any changes needed..

Yes I remeber @manikumarreddyu  but I told you to give all proper details and description to each PR like this I was reminding for this situation where we review contributors but I can see this is still the same, if the actual work is done and you are adding higher labels to these PR for behind the scene work then please add references, add PR link which is referred to add this level on such PRs


BitsAndBytes quantization RuntimeError: Lock objects should only be shared between processes through inheritance

'libs/FLash-VStream/flash_vstream/model/builder.py:98: in load_pretrained_model
    model = VStreamLlamaForCausalLM.from_pretrained(model_path, low_cpu_mem_usage=True, **kwargs)
../../.venv/lib/python3.10/site-packages/transformers/modeling_utils.py:2719: in from_pretrained
    modules_to_not_convert = get_keys_to_not_convert(model)
../../.venv/lib/python3.10/site-packages/transformers/utils/bitsandbytes.py:257: in get_keys_to_not_convert
    tied_model = deepcopy(model)  # this has 0 cost since it is done inside init_empty_weights context manager
/usr/lib/python3.10/copy.py:172: in deepcopy
    y = _reconstruct(x, memo, *rv)
/usr/lib/python3.10/copy.py:271: in _reconstruct
    state = deepcopy(state, memo)
/usr/lib/python3.10/copy.py:146: in deepcopy
    y = copier(x, memo)
/usr/lib/python3.10/copy.py:231: in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
/usr/lib/python3.10/copy.py:161: in deepcopy
    rv = reductor(4)
/usr/lib/python3.10/multiprocessing/synchronize.py:101: in __getstate__
    context.assert_spawning(self)
/usr/lib/python3.10/multiprocessing/context.py:373: in assert_spawning
    raise RuntimeError(
E   RuntimeError: Lock objects should only be shared between processes through inheritance

Sorry, we didn't meet this bug.


Disable window animation?

I want very fast responsiveness but window transition animations prevent quick manipulation.

%AppData%\VRCX\custom.css
.dialog-fade-enter-active,
.dialog-fade-leave-active,
.msgbox-fade-enter-active,
.msgbox-fade-leave-active,
.el-zoom-in-center-enter-active,
.el-zoom-in-center-leave-active,
.el-zoom-in-top-enter-active,
.el-zoom-in-top-leave-active,
.el-zoom-in-bottom-enter-active,
.el-zoom-in-bottom-leave-active,
.el-zoom-in-left-enter-active,
.el-zoom-in-left-leave-active,
.el-zoom-in-right-enter-active,
.el-zoom-in-right-leave-active {
  animation: unset !important;
  transition: none !important;
}


Update to the React 17

When I try to install it in my project, I get a dependency error and it only accepts React 16 and earlier.

duplicate of https://github.com/derrickpelletier/react-loading-overlay/issues/54


Error when processing JSONL with excluded null values using dataframe.read_json

Describe the issue:
In JSON format, a key with no value associated with it can be represented as "key": null, or by removing the entire key. Both forms seem to be commonly used.
When receiving data in the latter form, the current implementation of dask's dataframe.read_json seems to generate an error Order of columns does not match if the key in the first line is incomplete.
A minimal example is attached below.
Could you let me know a workaround to prevent the Order of columns does not match error, or fix this behavior?
This problem is caused by a missing column in the row, which can be handled by filling the corresponding column with null. However in some practical cases, such as with huge datasets, this is difficult or inefficient.
It would be possible to prevent the error if the order of the actual's columns could be enforced in some way, but I have not found a good way to do this.
Minimal Complete Verifiable Example:
import pandas as pd
import json
from dask import dataframe as dd

if __name__ == "__main__":
    data = [
        {"foo": 1, "baz": 3},
        {"foo": 1, "bar": 2, "baz": 3},
    ]
    jsonl_file_path = "example.jsonl"
    with open(jsonl_file_path, "w") as wf:
        for row in data:
            print(json.dumps(row), file=wf)

    dt = {"foo": "int64", "bar": "int64", "baz": "int64"}
    df = pd.DataFrame(columns=list(dt.keys())).astype(dt)
    ddf = dd.read_json(jsonl_file_path, meta=df)

    print(list(df.columns))  # ['foo', 'bar', 'baz']
    print(list(df.dtypes))  # [dtype('int64'), dtype('int64'), dtype('int64')]

    print(list(ddf.columns))  # ['foo', 'bar', 'baz']
    print(list(ddf.dtypes))  # [dtype('int64'), dtype('int64'), dtype('int64')]

    # this line raises
    # ValueError: The columns in the computed data do not match the columns in the provided metadata
    # Order of columns does not match
    ddf.compute()

    # according to the pydev debugger,
    # the actual dataframe column order is foo, baz, bar
    # check_matching_columns, utils.py:440
    # meta:
    #     Empty DataFrame
    #     Columns: [foo, bar, baz]
    #     Index: []
    # actual:
    #        foo  baz  bar
    #     0    1    3  NaN
    #     1    1    3  2.0

Anything else we need to know?:
Environment:

Dask version: 2024.6.1
Python version: 3.11.4
Operating System: MacOS
Install method (conda, pip, source): pip

Having a Dask DataFrame that has mismatching columns can cause all kinds of issues. I would definitely not recommend doing this.
I don't think that read-json has this built in, what you could do is use from_map with pd.read_json as the function and pass in the columns you need to do a reindex on the read_json result. Does that help?

Hi! I have the same issue @kzosabe describes. Did you finally find some work around to avoid keeping all null keys?

https://github.com/dask/dask/issues/11190#issuecomment-2180835903

Hi @phofl I haven't tried your proposal cause I cannot know a priori the columns so I cannot pass them (it's a generic class to validate the schema).
Though, good news! I found the solution, the answer here made the trick!


feat: Prevent resizing more than the current position quantity

While working on https://github.com/get10101/10101/issues/1625 I noticed that the way we do a resizing with changing the direction completely can't work the way we have it designed at the moment.
The issue is that we have a bid and an ask price. If we are changing the general direction of our position during resizing we are not using the correct price for the new position.
As a quick fix I've added a simple UI validation rule, that prevents the user from reducing a position further than the current quantity of the position.

Alternatively we could also automatically split up the order into two. 1. for closing the existing position and 2. for opening the new position. But the current design would not be able to show for what price the new position will be opened so I lean to preventing the user from reducing past the quantity of the existing position.
Simply close the position and open a new one. @luckysori @bonomat wdyt?

Alternatively we could also automatically split up the order into two. 1. for closing the existing position and 2. for opening the new position. But the current design would not be able to show for what price the new position will be opened so I lean to preventing the user from reducing past the quantity of the existing position.
Simply close the position and open a new one. @luckysori @bonomat wdyt?

I would be in favor of fixing it correctly if it is possible.

But the current design would not be able to show for what price the new position will be opened

Why not?

Why not?

Because at the moment we only show the bid or the ask price on the sell or buy tab.
Assume the following scenario: The user is 10 short and wants to go 20 long.
Our trade screen tab would need to show the ask price for the first 10 contracts and the bid price for the second 10 contracts. Our UI design is not able to show that properly at the moment.

Why not?

Because at the moment we only show the bid or the ask price on the sell or buy tab.
Assume the following scenario: The user is 10 short and wants to go 20 long.
Our trade screen tab would need to show the ask price for the first 10 contracts and the bid price for the second 10 contracts. Our UI design is not able to show that properly at the moment.

Not sure I follow, if the user is 10 short, and he wants to be 20 long afterwards, he needs to go 30 long. For 30 long he will need to pay the ask price. He only sees the ask price in the Long screen tab.

closed in favor of https://github.com/get10101/10101/pull/1627


Add anybar wrapper

@davydovanton Thanks for contributing. We use alphabetical case insensitive order. Please update, run tests locally and do git amend ;)

@veelenga done

Thank you!

thanks for merge :tada:


Problem importing large files

@joshrowley your chunking in atom-socket fixed this—i was just able to import at 30mb photo no problemo 💥


[BUG] ERROR: Simultaneous requests running!

Related issues:

https://github.com/ainblockchain/ain-blockchain/issues/948

Questions:

Can this case be caught by the code?
Can the job be cooled down for a moment?


Reorganize the docs, and link from the README to the docs (and vice versa)

The links from GitHub (README) to docs and docs to GitHub are really convenient to have.

Codecov Report

Merging #8 (52fc9b8) into master (3bcd5ed) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           98        98
=========================================
  Hits            98        98

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 3bcd5ed...52fc9b8. Read the comment docs.

Codecov Report

Merging #8 (52fc9b8) into master (3bcd5ed) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           98        98
=========================================
  Hits            98        98

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 3bcd5ed...52fc9b8. Read the comment docs.

Looks great, thanks.

Looks great, thanks.

I was hoping the relocatability fix would address;

ERROR: LoadError: InitError: BoundsError: attempt to access Tuple{ThreadingUtilities.ThreadTask, ThreadingUtilities.ThreadTask} at index [3]
Stacktrace:
[1] getindex(t::Tuple, i::Int64)
@ Base ./tuple.jl:29
[2] init()
@ ThreadingUtilities ~/work/ThreadingUtilities.jl/ThreadingUtilities.jl/src/ThreadingUtilities.jl:23

I was hoping the relocatability fix would address;

ERROR: LoadError: InitError: BoundsError: attempt to access Tuple{ThreadingUtilities.ThreadTask, ThreadingUtilities.ThreadTask} at index [3]
Stacktrace:
[1] getindex(t::Tuple, i::Int64)
@ Base ./tuple.jl:29
[2] init()
@ ThreadingUtilities ~/work/ThreadingUtilities.jl/ThreadingUtilities.jl/src/ThreadingUtilities.jl:23


---Feedback Admin

Can I suggest, if possible, that two tabs are added on the admin live page that lets you know how many candidates have applied in total and how many applications in total are in draft? It's just helpful to know during the time period that the exercise is open for application as our process is currently dependent on the total amount of applications we receive.
View in Slack


mTLS analyzer bails early if PeerAuthentication CRD exists

The beta security APIs have different semantics compared to the alpha ones.
Rather than update the analyzer, it is now instead marked as deprecated.

@selmanj: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-galley-k8s-tests_istio
348229260117127d5a4f0f76c6520fc9ac899f81
link
/test integ-galley-k8s-tests_istio

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

@selmanj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-galley-k8s-tests_istio
348229260117127d5a4f0f76c6520fc9ac899f81
link
/test integ-galley-k8s-tests_istio

integ-local-tests_istio
348229260117127d5a4f0f76c6520fc9ac899f81
link
/test integ-local-tests_istio

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

@selmanj: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-tests_istio
1c55669bd17e771bc7f19ce72c6ec6c414a707a0
link
/test integ-mixer-k8s-tests_istio

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

@selmanj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-tests_istio
1c55669bd17e771bc7f19ce72c6ec6c414a707a0
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
1c55669bd17e771bc7f19ce72c6ec6c414a707a0
link
/test integ-telemetry-k8s-tests_istio

integ-pilot-k8s-tests_istio
1c55669bd17e771bc7f19ce72c6ec6c414a707a0
link
/test integ-pilot-k8s-tests_istio

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

/retest

/retest

@selmanj: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test lint_istio

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

@selmanj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test lint_istio

gencheck_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test gencheck_istio

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

@selmanj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test lint_istio

gencheck_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test gencheck_istio

unit-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test unit-tests_istio

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

@selmanj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test lint_istio

gencheck_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test gencheck_istio

unit-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test unit-tests_istio

integ-local-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test integ-local-tests_istio

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

@selmanj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test lint_istio

gencheck_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test gencheck_istio

unit-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test unit-tests_istio

integ-local-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test integ-local-tests_istio

integ-conformance-k8s-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test integ-conformance-k8s-tests_istio

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

@selmanj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test lint_istio

gencheck_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test gencheck_istio

unit-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test unit-tests_istio

integ-local-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test integ-local-tests_istio

integ-conformance-k8s-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test integ-conformance-k8s-tests_istio

integ-galley-k8s-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test integ-galley-k8s-tests_istio

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

@selmanj: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-local-tests_istio
9c9ab0e50021ea3e503daf9fcb5ca8b48b99b0d8
link
/test integ-local-tests_istio

unit-tests_istio
477af3b18835f2cb4f1841b6b0b9cde31fd6f13e
link
/test unit-tests_istio

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

/retest

In response to a cherrypick label: #21487 failed to apply on top of branch "release-1.5":
.git/rebase-apply/patch:161: trailing whitespace.

.git/rebase-apply/patch:183: trailing whitespace.

.git/rebase-apply/patch:206: trailing whitespace.

.git/rebase-apply/patch:228: trailing whitespace.

warning: 4 lines add whitespace errors.
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	galley/pkg/config/analysis/analyzers/analyzers_test.go
M	pkg/config/schema/collections/collections.gen.go
M	pkg/config/schema/collections/staticinit.gen.go
M	pkg/config/schema/metadata.gen.go
M	pkg/config/schema/metadata.yaml
M	pkg/config/schema/snapshots/staticinit.gen.go
M	pkg/config/schema/staticinit.gen.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/config/schema/metadata.yaml
CONFLICT (content): Merge conflict in pkg/config/schema/metadata.yaml
Auto-merging pkg/config/schema/metadata.gen.go
CONFLICT (content): Merge conflict in pkg/config/schema/metadata.gen.go
Auto-merging galley/pkg/config/analysis/analyzers/analyzers_test.go
Patch failed at 0001 mTLS analyzer bails early if PeerAuthentication CRD exists


Avoid inlining functions with inline ASM blocks.

Also removes [__unsafeForceInlineEarly] from built-in methods that have inline spir-v
Fixes: #4924

@csyonghe: I think the change on line 5288 & 3959 should handle RWStructuredBuffer & RWByteAddressBuffer?


Add job titles for fr_FR locale

@fzaninotto Same things than https://github.com/fzaninotto/Faker/pull/888 but under CC license.

Thanks!

I just reverted the merge, see #900

:cry:


Run command with "Execute in a new dedicated console" option

Description
What steps will reproduce the problem?

I request either changing the behaviour of the Run command with the "Execute in a dedicated console" option or adding a new option to be able to create a new dedicated console for each run, due to the below reasons. If Spyder allows this like PyCharm does, it would be very helpful! Currently only solution I found is to manually create a new console and run on it, which is not practical and I sometimes forget create and waste the running time duration of the script.

For data analysis, algorithm development and debugging, I often need to interactively compare by running the same script with different parameters or changing parts of its codes. Therefore multiple consoles for different versions of the same script should stay open to be able to do various comparisions.

Since the Run command does not make a kernel reset, modules are reloaded without removing already loaded modules. As far as I know it is still not posible to remove loaded modules, though the variables can be cleared by an option. This can cause serious memory problem after running multiple times if a loaded module consumes considerable memory (happens with some of my scripts as they load data in memory)

Versions

Spyder version: 3.3.4
Python version: 3.6.6
Qt version: 5.9.6
PyQt5 version: 5.9.2
Operating System: Linux 4.15.0-50-generic

Dependencies
IPython >=4.0     :  6.5.0 (OK)
cython >=0.21     :  None (NOK)
jedi >=0.9.0      :  0.12.1 (OK)
matplotlib >=2.0.0:  2.2.2 (OK)
nbconvert >=4.0   :  5.3.1 (OK)
numpy >=1.7       :  1.15.0 (OK)
pandas >=0.13.1   :  0.23.3 (OK)
psutil >=0.3      :  5.4.6 (OK)
pycodestyle >=2.3 :  2.4.0 (OK)
pyflakes >=0.6.0  :  2.0.0 (OK)
pygments >=2.0    :  2.2.0 (OK)
pylint >=0.25     :  1.9.2 (OK)
qtconsole >=4.2.0 :  4.3.1 (OK)
rope >=0.9.4      :  0.10.7 (OK)
sphinx >=0.6.6    :  1.7.6 (OK)
sympy >=0.7.3     :  None (NOK)

I request either changing the behaviour of the Run command with the "Execute in a dedicated console" option or adding a new option to be able to create a new dedicated console for each run

Sorry but we're not interested in implementing this. However, you're welcome to send us a pull request for it.

For information: I'm working on this as it can be quite useful for some of my use cases as well.


rocminfo shows "no gpu" after installation

azureuser@gvme7b1132637:~$ /opt/rocm/bin/rocminfo
ROCk module is NOT loaded, possibly no GPU devices
azureuser@gvme7b1132637:~$ /opt/rocm/opencl/bin/clinfo
Number of platforms:				1
  Platform Profile:				FULL_PROFILE
  Platform Version:				OpenCL 2.0 AMD-APP (3305.0)
  Platform Name:				AMD Accelerated Parallel Processing
  Platform Vendor:				Advanced Micro Devices, Inc.
  Platform Extensions:				cl_khr_icd cl_amd_event_callback

  Platform Name:				AMD Accelerated Parallel Processing
Number of devices:				0

Strange thing is that opencl seems available.
The installation and tests were done on a NV4as_v4 VM with Instinct MI25 GPU.
ROCM version: 4.3 (Installed according to https://rocmdocs.amd.com/en/latest/Installation_Guide/Installation-Guide.html)
OS version: Ubuntu 20.04.2

Thanks @kindlychung for reaching out.
Can you please share the exact steps to reproduce the problem.
Thank you.

I have the same problem as well, I believe @kindlychung installed ROCm according to the installation guide. I'm getting exactly the same results even though I believe my GPU chip is of type 'polaris 12'. The steps I followed were exactly as mentioned in the guide and the commands /opt/rocm/bin/rocminfo gave the results as they got.
OS Version: Ubuntu 20.04
Kernel: 5.11.0-27-generic

Hi @Nishant-Pall
Polaris12 is NOT a supported card and so things will not work on them, which I can say that a different issue from the titled one.
Hi @kindlychung
Please help us with the exact steps you followed from start, so that we will understand better and tries to provide resolution.
Thank you.

I have the exact same problem, and I have the Radeon Pro 6600
uname -a Linux  5.11.0-27-generic #29~20.04.1-Ubuntu SMP Wed Aug 11 15:58:17 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
SYSTEM_NAME_or_ISSUEID.rocm_techsupport.log

Hi @mdberryh
ROCm does not support Navi10 cards and so things do not work on these cards as expected.
Please track supported hardware section @ https://github.com/RadeonOpenCompute/ROCm#hardware-and-software-support
Thank you.

@ROCmSupport  Thanks for your quick assistance! I was hoping to use AMD GPUs for neural networks, so I thought I'd give it a try. I am pretty confused with the differences between Rocm and AMD Pro drivers. Reading different frameworks both are mentioned and both use openCL. I'll see what I can do with the AMD Pro drivers for linux.

Hi @kindlychung
As mentioned before, can you please help me with the exact steps you followed from starting, so that we can debug the problem in a better way.
And also I recommend you to verify onc with the latest ROCm 4.5 and update.
Thank you.

Hi @kindlychung
Issue is not reproducible with ROCm 4.5. Request to verify with the same.
Thank you.

@ROCmSupport do you support Topaz XT [Radeon R7 M260/M265 / M340/M360 / M440/M445 / 530/535 / 620/625 Mobile]


Update delimit-volume-allocation.md

Calculate width does not work when using the script in Powershell ISE as it does not provide a (Get-Host).UI.RawUI.WindowSize.Width property.

Looks good - thanks for your contribution! Let's merge this!


Fix typo parrallelism -> parallelism

Verified that parallelism actually works in the YAML.

Ooops, thanks for sending this <3 <3 <3


Switch From GitHub Commit Statuses API to Checks API

Prow uses github commit statuses to interact with PRs (all the components that interact with these I don't know yet), should we switch from commit statuses to the new checks api? Checks seem much more featureful than commit statuses. Thoughts, comments, concerns?

+1
I talked about this with @petr-muller, and he would agree that Prow deserves some modernization.

I'm favor of this generally. Two roadbumps I recall from earlier discussions, they may not describe the current situation well though:

Checks API had some issues with PAT authentications, IIRC parts of it were only available to GH Apps? I hope most Prow installations use GH Apps now but it is possible that we'd need to deprecate PATs
IIRC check results were quite tricky to read / process from e.g. the Tide PoV, the check names / IDs were weird somehow. I remember this was causing issues when we tried to interface with GH actions better, but it's a while now

Checks API had some issues with PAT authentications, IIRC parts of it were only available to GH Apps?

There is conflicting information in the github documents about this, where it says you need to use a github app to use various checks apis, but then it says it supports "Fine-grained personal access tokens": https://docs.github.com/en/rest/checks/runs?apiVersion=2022-11-28#create-a-check-run

Fine-grained personal access tokens are quite new actually, introduced in late 2022, which means they are newer than apps. My memory is definitely from before that. So it is quite likely that only the "old" PATs are not able to interact with checks.

So maybe we should think about deprecating "classic" personal access tokens usage?

Yeah, I would not be against that, but we'd need a reasonable transition period.


No/Little Memory Reduction for HTC when training with FP16

I've tried fp16 on other models and it reduces memory usage a lot but has little impact on HTC model. Should I modify somewhere to support fp16 training on HTC?

Could you show the exact number of FP32 HTC and FP16 HTC.
I just test htc_r50_fpn_1x_coco.py with 1 GPU, in the first several iterations, the memory usage of
FP32 HTC: 5724 MB
FP16 HTC: 3803 MB (Just adding fp16 = dict(loss_scale=512.) at the end of the config)
It seems normal.

I rechecked the memory usage of htc on mmdet2.0 and it looks normal. Thx. The memory usage reduced from about 9000MB to 6500MB.


Formatting failure bug with if keyword likely introduced in #7060

Looks like I've found a bug which was likely introduced with the formatting for early return if statements (#7060). I have a case where  an assignment using if/then/else causes a formatting failure, and is being formatted with a hanging else in the debug output as though it were an early return:
setModels : Client, List Str -> Client
setModels = \client, models ->
    modelsOption = if List.isEmpty models
        then Option.none {}
        else Option.some models
    { client & models: modelsOption }

Causes a format failed bug, but gets formatted as the following in the debug output file:
setModels : Client, List Str -> Client
setModels = \client, models ->
    modelsOption = if
        List.isEmpty models
    then
        Option.none {}
        else

    Option.some models
    { client & models: modelsOption }

However, by moving the if onto a new line, formatting completes correctly. The following:
setModels : Client, List Str -> Client
setModels = \client, models ->
    modelsOption =
        if List.isEmpty models
            then Option.none {}
            else Option.some models
    { client & models: modelsOption }

Formats as:
setModels : Client, List Str -> Client
setModels = \client, models ->
    modelsOption =
        if
            List.isEmpty models
        then
            Option.none {}
            else

        Option.some models
    { client & models: modelsOption }

Link to zulip convo here


Don't delete unmodified fields in edit-profile submission

@keybase/react-hackers r? @chromakode (no rush, whenever)
When we load the Edit Profile view, its fields (bio, fullname, location) are populated by default from the userInfo object (result of identifying).  If we make changes, those changes live in the routeState object.
If you make a change to one field (e.g. location) and then submit your changes, the routeState object is used for all three fields, and so the ones you didn't change are reset to the empty string on the server, resulting in data loss of those fields.

@chrisnojima Thanks, I've switched to using your version now, it works.  PTAL?
But now I notice another problem, we're hitting CORE-3969 ("user card cached?").  @maxtaco says he'll probably promote that to this sprint.  Am guessing it won't need code changes on our end but I'll make them if it does.

@chrisnojima Thanks, removed the unnecessary onEditProfile render prop and flow type.

The commits 0d05a46afa42...60f1474109db introduce visual changes on linux. :mag_right: 2 changed

Changed: unlock_folders-Success (before) (after)

Changed: Tracker-Tracker_Error (before) (after)


Templated values are not evaluated in ApplicationSets before executing helm template

Hello there! First of all, I appreciate your work on this tool, it's easy to use and pretty straightforward. However, I encountered an issue.
Our ArgoCD ApplicationSets heavily relies on templating based on the labels of configured ArgoCD clusters. See this example:
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
  name: datadog
  namespace: argocd
spec:
  generators:
    - clusters: {}
  template:
    metadata:
      name: "datadog-{{name}}"
    spec:
      project: "datadog"
      sources:
        - repoURL: https://helm.datadoghq.com
          chart: datadog-operator
          targetRevision: 1.2.2
          helm:
            releaseName: "datadog"
            valueFiles:
              - $values/deployments/production/datadog/values/{{metadata.labels.datadog_instance}}.yaml
        - repoURL: git@github.com:redacted.git
          targetRevision: main
          ref: values
        - repoURL: git@github.com:redacted.git
          targetRevision: main
          path: deployments/production/datadog/manifests
          directory:
            recurse: true
            include: "{credentials,agents}/{{metadata.labels.datadog_instance}}.yaml"
      destination:
        server: "{{server}}"
        namespace: monitoring

This ApplicationSet installs a DataDog operator on every cluster registered in the ArgoCD with values set based on the label (DataDog offers multiple instances, e.g. US/EU/JP). This approach is working in the ArgoCD (by working I mean - it gets templated and applied correctly). However, when running Argocd-diff-preview I'm encountering this error:
❌ Failed to process application: datadog-in-cluster with error:
Failed to load target state: failed to generate manifest for source 1 of 3: rpc error: code = Unknown desc = `helm template . --name-template datadog --namespace monitoring --kube-version 1.30 --values <path to cached source>/deployments/production/datadog/values/{{metadata.labels.datadog_instance}}.yaml <api versions removed> --include-crds` failed exit status 1: Error: open <path to cached source>/deployments/production/datadog/values/{{metadata.labels.datadog_instance}}.yaml: no such file or directory

It seems to me that templating based on the metadata is not evaluated correctly.
I hope to dive deeper into the issue after my holidays.

Hi - Thank you for reporting this! 👏🏻
This is an interesting problem that I haven't encountered before.
My initial debugging questions:

What ArgoCD version are you running in your live cluster? argocd-diff-preview picks the newest version unless a specific version is specified.
How do you provide argocd-diff-preview with cluster credentials? (Just checking that the labels are actually on the secret resources before saving the credentials under /secrets)
Does your live ArgoCD instance use any special settings in the argocd-cm or argocd-cmd-params-cm ConfigMaps? argocd-diff-preview mainly use default ArgoCD settings, but I'll add support for more options/customization if needed.

Sorry for the late response, got sucked up into the whirl of projects straight away from the PTO 😅
AD.1:
{
    "Version": "v2.11.2+25f7504",
    "BuildDate": "2024-05-23T13:32:13Z",
    "GitCommit": "25f7504ecc198e7d7fdc055fdb83ae50eee5edd0",
    "GitTreeState": "clean",
    "GoVersion": "go1.21.9",
    "Compiler": "gc",
    "Platform": "linux/arm64",
    "KustomizeVersion": "v5.2.1 2023-10-19T20:13:51Z",
    "HelmVersion": "v3.14.4+g81c902a",
    "KubectlVersion": "v0.26.11",
    "JsonnetVersion": "v0.20.0"
}

Ad.2 That might be the cause I overlooked, we're using Argo to bootstrap itself - we have an Application resource that creates the Cluster Secrets. We've structured this in the repository like this:
clusters
├── development
│   ├── application.yaml
│   └── cluster-secrets
│       └── cluster-01.yaml
└── production
    ├── application.yaml
    └── cluster-secrets
        ├── cluster-01.yaml
        └── cluster-02.yaml

I think that to get Argocd-diff-preview work with that setup I have to apply these application resources/cluster secrets first, right?
Example Cluster Secret looks like this:
apiVersion: v1
kind: Secret
metadata:
  name: cluster-01
  namespace: argocd
  annotations:
    managed-by: argocd.argoproj.io
  labels:
    argocd.argoproj.io/secret-type: cluster
    cluster_environment: development
    datadog_instance: "eu"
type: Opaque
stringData:
  name: "cluster-01"
  server: "https://kubernetes.default.svc"
  config: |
    {
      "tlsClientConfig": {
        "insecure": false
      }
    }

Ad. 3 Nothing that would relate to the issue, mostly OIDC providers setup

Hi again :)

I think that to get Argocd-diff-preview work with that setup I have to apply these application resources/cluster secrets first, right?

Yes! You will need to provide argocd-diff-preview with the necessary credentials to access your external clusters.
There is a section in the README that describes how to place secrets in the /secrets folder: README#Private repositories and Helm charts
If your pipeline has access to your live Argo CD instance, you can do something like this::
jobs:
 build:
   ...
   steps:
     ...
   - name: Prepare secrets
     run: |
       mkdir secrets
       kubectl -n argocd get secret <the-cluster-secret>  -o yaml  > secrets/cluster-credentials.yaml

You can verify if argocd-diff-preview applies the secrests correctly by checking if the tool outputs 🤫 Applied 1 secrets
Let me know if this works as you expect or you still experience issues 🚀

It makes total sense, I'll try to set it up and I'll let you know of a result. Thanks!

Yup, the issue with not interpolating variables was caused by cluster secrets not being applied, thanks for you help ❤


Rule to ensure that ConfirmImpact=High has Confirm and -Force

Per PowerShell/PowerShell#3337, whenever ConfirmImpact=High is specified, we should suggest having a -Confirm and -Force parameter to ensure that users can skip the prompt in non-interactive scenarios.

If I understand this correctly, the rule needs to suggest the user to add the SupportsShouldProcess attribute if ConfirmImpact=High', since ForceandConfirm will be provided by SupportsShouldProcess. Is that right?


Web site for the OSSU CS Curriculum

I wounder if anyone else in the OSSU Community thinks that a web site for the curriculum would have any value for the community members? I mean a simple static site for reference purposes. The old firebaseapp site is out of date and requires time and commitment to maintain. But a simple GitHub Pages site could do the job as well.
Let me illustrate the idea on my own example. I personally prefer keeping track of my CS progress and a complete log of the related activities in a GitHub repository (see here). But in social networks and for sharing to a wider public (e.g. on CV), I prefer sharing a link to a personal web site rather than to the GitHub repository, which may seem a bit confusing for a non-tech public. My web site/page is based on the README.md file in the repository. It was easy to build using the GitHub Pages tool and one of the supported themes. It does not require any maintenance and updates as soon as I update the README.md file.
The OSSU CS Curriculum is currently mostly maintained in one README.md file and could similarly be mirrored to a GitHub Pages site for reference purposes. It would be automatically updated with each update of the README.md file with no maintenance efforts required! The GitHub Pages tool also allows a multi-page site linked to different files in the same project repository if needed.
Therefore, I call for an exchange of opinions from the OSSU community whether a simple web site for the CS Curriculum would be of any value for members? I personally find it a bit confusing that, when searching for "OSSU" in browser or when giving a reference to the OSSU Curriculum to a general public, the only option available is a GitHub repository. I also think that a mirror of the repository is enough, and no "login" or functionality of the firebase site is required, as students can easily track their progress in a public GitHub repository or in a public Trello board.

nope.. don't think so!
GitHub README page is so simple for public to read and i think it's somehow encouraging for someone who entering the field to know and know the git.

nope.. don't think so!
GitHub README page is so simple for public to read and i think it's somehow encouraging for someone who entering the field to know and know the git.

OK, accepted (not sure how reading a README.md file could teach git however :) ). Just wonder if you ever tried to share OSSU repository with someone not from CS.


Get rid of internal model listeners

As commented by @alonsogarciapablo  and discussed by @Jesus89 and the one and only @ivanmalagon we can get rid of the internal model listeners in internal dataview for properties that are being set by the wrapper object.

Let me know if im wrong.
we should remove the _listenToInternalModelSpecificEvents function and instead attaching a listener when the internalModel is changed we should trigger the xxxChanged event on the setter right?

We have a method called ._listenToInternalModelSpecificEvents in every implementation of a dataview. It's listening to the same options that we can set for everyone:

Histogram: we listen to

change:bins

Formula: we listen to

change:operation

Category: we listen to

change:categories
change:aggregation
change:aggregation_column

These listeners are only there to sync that internal model state with the wrapped one. But, we can only change those properties through the wrapped one, so it doesn't make any sense to listen for those specific events.
We did that being paranoid of something changing the internal models under the hood but thinking about that, it's not possible.
So we can get rid of listening to those specific changes.
Take into account that we should trigger xxxChanged events anyway. Now they're triggered as part of those listeners to be removed.

should we trigger the ' xxxChanged'  asynchronously?
Formula.prototype.setOperation = function (operation) {
  this._checkOperation(operation);
  this._operation = operation;
  if (this._internalModel) {
    this._internalModel.set('operation', operation, { silent: true });
    return this._internalModel._engine.reload()
      .then(function () {
        this.trigger('operationChanged', this._operation);
      }.bind(this));
  }
  return Promise.resolve();
};

or synchronous at the moment the attribute is changed?
Formula.prototype.setOperation = function (operation) {
  this._checkOperation(operation);
  this._operation = operation;
  this.trigger('operationChanged', this._operation);
  if (this._internalModel) {
    this._internalModel.set('operation', operation);
  }
  return this;
};

Synchronous. The attributes is already changed even if the operation fails, so if other model / object wants to know if the value has changed, it should be notified anyway.


allow factories to use methods defined in traits

I like to create a factory method for each model so that it's easy to create it with all its data and associations.
class Post extends Model {
   static async init ({ title, user }) {
      cost post = new Post()
       post.fill({ title })
       await post.owner().associate(user)
       await post.save()
   }
   ....
}

However, it is not possible to then call: Factory.model('App/Models/Post').init({ user }) since factories do not seem to expose the model's methods.
Would this be possible? Thanks.

Why would you use a factory in this case? Simply use the model and call the method.
use('App/Models/Post').init({ user })

I figured that that way I could make uses of the faker object, but I can also import that chancejs library myself otherwise

Okay, so what you want is a model instance pre-filled with faker data.
Then you should use .make method.
const postInstance = Factory.model('App/Models/Post').make()
postInstance.init({ user })

Ah, that makes sense. Thanks!


fix: plain javascript code added for once method

resolves #902
By submitting a PR to this repository, you agree to the terms within the Auth0 Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.
Description

Describe the purpose of this PR along with any background information and the impacts of the proposed change. For the benefit of the community, please do not assume prior context.
Provide details that support your chosen implementation, including: breaking changes, alternatives considered, changes to the API, etc.
If the UI is being changed, please provide screenshots.

References

Include any links supporting this change such as a:

GitHub Issue/PR number addressed or fixed
Auth0 Community post
StackOverflow post
Support forum thread
Related pull requests/issues from other repos

If there are no references, simply delete this section.

Testing

Describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.
Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.
Also include details of the environment this PR was developed in (language/platform/browser version).

[x] This change adds test coverage for new/changed/fixed functionality

Checklist

[ ] I have added documentation for new/changed functionality in this PR or in auth0.com/docs
[x] All active GitHub checks for tests, formatting, and security are passing
[x] The correct base branch is being used, if not the default branch

not 100% sure but i believe we do not need once at all... (?)
The event listener use once in itself, so it's not going to call error or done multiple times.
the listener gets removed once it have been called.
    callback = callback && once(callback);

    jws.createSign(...)
      .once('error', callback) // <--- `once` is used here
      .once('done', function (signature) { // <--- `once` is used here
        if(!options.allowInsecureKeySizes && /^(?:RS|PS)/.test(header.alg) && signature.length < 256) {
          // Return is used here so that the callback isn't called again.
          return callback(new Error(`secretOrPrivateKey has a minimum key size of 2048 bits for ${header.alg}`))
        }
        callback(null, signature);
      });

but that's ofc if not both error and done both get called somewhere else like: ee.emit('error', x); ee.emit('done', x)


fixed the log normalization for the logpdf.

Fixes #371 the log normalization for covariance values that are small and lead to numerical issues (->0.0) in higher dimensions (>50). also added two tests to check the numerical stability for diagonal and non-diagonal covs.

Thank you very much for the fix - and the tests.


Put badges below title

This is the way most repositories on GitHub are doing this, with an example being Numpy. Just makes for a more consistent experience.

Coverage remained the same at 83.091% when pulling 0473b6a12f033db41431b7cea2f3b187ed1fa3bc on malramsay64:patch-3 into e2934c30086792ac02fff325b3ab63b671f9f388 on mpurg:master.


Step 2

Advanced > Zoom for Developers no longer exists in the Zoom settings.
Step 2 part 3, this app is also not present unless it is created? which step states how to do this?

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

ID: 37280d15-75ea-7486-48d6-08e82fd4c1a4
Version Independent ID: 037a01c2-1908-6ca4-f868-a9cd488e85d5
Content: Tutorial: Configure Zoom for automatic user provisioning with Azure Active Directory
Content Source: articles/active-directory/saas-apps/zoom-provisioning-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @zchia
Microsoft Alias: Zhchia

@jackvdbuk Thanks for your comment. We will review your request and get back you shortly.

@zchia Could you check on this ?

#reassign:twimmers

Thanks @jackvdbuk for the feedback. I have made a doc change to address this issue and the change should be live tomorrow.
#please-close


fails to run on fresh install

><EMAIL_ADDRESS>start /Users/cdieringer/node/octagon.react
> patternplate start --open

⠹ Starting/Users/cdieringer/node/octagon.react/node_modules/patternplate/binary/patternplate.js:182
                throw error;
                ^

Error: Cannot find module 'patternplate-transform-browserify'
...

& patternplate-transform-react, patternplate-transform-react-mount, patternplate-transform-react-to-markup

Thanks for reporting! patternplate init currently does not perform npm install / yarn isntall. Can you try and do that before executing patternplate start ?

Released and published:
https://github.com/sinnerschrader/patternplate/releases/tag/v1.0.7
❯ npm show patternplate version
1.0.7


Updating the replication appliance requirement to use Windows 2016

Updating the replication appliance requirement to use Windows 2016 from 2022, which was updated by mistake.

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

#sign-off

#sign-off

Thanks, @pingvishal-msft. Merging now.
Can you open a PR in the private repo to update the author metadata field? It should be your GitHub alias (not your Microsoft one). That will clear up this prmerger signoff issue. Thanks!


Write some sort of test that ensures that all existing configs can be parsed

Essentially, if we make changes to functions so that historical configs will break, we want to be notified, and we want to ensure that all previously valid configs are still valid.
If they are not, we need to ensure that we have documentation for how to get them to a valid state, e.g. by having a lineage of valid configs.
Ideally, we also want to handle backwards compatability, but that might be a high ask. It could possibly happen at the schema level.
E.g.:

On each run, generate config file

If unchanged from latest on disk, do nothing
If changed, write a new config to disk

Try parsing the latest config to schema

Another:

Parse all historical configs to schema

This is related to #415.

And see #426.

[ ] Remove the whitelist control flow
[ ] Ensure that the test iterates over all functions that are represented as either: 1) cfgs in the static test cfg folder or 2) entries in the registry


Fixed context domain export and updated year

Updated the copyright year in all C# code
Fixes context domain export for issue #200

General note on copyright headers: if you haven't edited or changed a file during the year, you generally wouldn't need to update the copyright year (like a book or other work, source copyright generally applies to when the work was authored or modified)


Calculator for checking whether two lines are perpendicular to each other or not

I would like to add a calculator for finding whether two lines in slope form are perpendicular to each other or not under coordinate geometry calculator section.
Please assign this issue to me.

Go ahead
Deadline:23 April 2021


Update model stats represent the currently visualised data

The model stats display currently represents the whole of the current data field. It would be quite helpful if the stats were updated to reflect the area of the data currently being viewed.

No longer relevant to latest implementation of FOREST.


Logger

Updates logger to not use interfaces and instead just a single static call that handles different types. Currently blocked by a phoenix 6 bug that makes refreshAll not function correctly for certain signals.

Unblocked by Phoenix 25.0.0-beta-2. Should probably look into Epilogue instead.


Remove duplicate dates declaration in getRangeDates function

Which package has an issue?
@yamada-ui/react
Package Version
1.4.5
Description
Unintended behavior is occurring because the dates array is being defined twice in the getRangeDates function. Specifically, it is causing the function to always return an empty array.
Link to Reproduction
No response
Steps to Reproduce
No response
Browser
None
Operating System

[ ] macOS
[ ] Windows
[ ] Linux
[ ] iPadOS
[ ] iOS
[ ] Android

Do you know how to fix the issue?
Yes
Are you willing to participate in fixing this issue and create a pull request with the fix?
Yes
Additional Information
No response

I'll try it.


Create CODE_OF_CONDUCT

@cb1kenobi Take a look at this please. Proposed community code-of-conduct based on MariaDB's code of conduct. Wording has been cleaned up and formatted in markdown for optimal display here on Github.


[$250] Refactor SMS Delivery Failure sign in flow

We need to refactor the existing flow blacklisting phone numbers (see steps in screenshots). This is triggered when the user tries to sign in with a phone number that failed to receive Twilio SMS. This is a follow up to this issue.

When calling BeginSignIn, if response[smsDeliveryFailureStatus] is not null, then it should display the following message:
"We've been unable to deliver SMS messages to [PHONE-NUMBER], so we've suspended it for 24 hours. Please try validating your number:"

There should also be a "Validate" button that calls ResetSMSDeliveryFailure.
If response[hasSMSDeliveryFailure]is set totrue, we should display Validation failed because it hasn't been 24 hours since your last attempt. ${response.message} and a "Got it" button that sends the user back to the beginning of the sign in flow.
If response[hasSMSDeliveryFailure]is set tofalse, we should display Your number has been validated! Click below to send a new magic sign-in code. and a "Send" button that calls BeginSignIn as usual.
Once this is complete and merged I'll do the internal QA with the BE.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021867038175379475893
Upwork Job ID:<PHONE_NUMBER>379475893
Last Price Increase: 2024-12-12

Issue OwnerCurrent Issue Owner: @hungvu193

@nyomanjyotisa 's proposal looks good to me!
🎀 👀 🎀

ResetSMSDeliveryFailure has not changed, it's the same structure as before.
BeginSignIn's response currently includes:
"smsDeliveryFailureStatus": {
    "hasSMSDeliveryFailure": true / false
},

This will change to:
"smsDeliveryFailureStatus": null / not null

Draft PR is created.


Feat/makefile and tests

This pull request introduces several key changes, including the addition of a Makefile for build automation, updates to the go.mod dependencies, and the addition of various unit tests to improve code coverage and reliability. Below is a summary of the most important changes:
Build Automation:

Added a Makefile to streamline the build, test, and run processes for the project. This includes targets for building, cleaning, formatting, running tests, and installing dependencies.

Dependency Management:

Updated the go.mod file to include several new indirect dependencies such as github.com/davecgh/go-spew, github.com/pmezard/go-difflib, github.com/stretchr/testify, and gopkg.in/yaml.v3.

Unit Tests:

Added unit tests for key mappings in helpers/keymaps_test.go to verify the initialization and functionality of common and suggestion input key bindings.
Added unit tests for the HubModel in hub/hub_test.go to test initialization, update functions, search functionality, and repository addition.
Added unit tests for the InstallModel in releases/install_test.go to verify initialization, key handling, and message handling for the installation process.
Added unit tests for the AddModel in repositories/add_test.go to verify initialization, key handling, and input clearing functionality.

These changes collectively enhance the project's build process, dependency management, and test coverage.

@pidanou  Let me know if you like the style , adding some CI would be great.

Thank you for the thoughtful feedback! After considering both options, I agree that Option 1 is the better approach. Having each model manage its own focus state keeps things encapsulated and makes the code easier to maintain and scale.
Let me know if there's anything you'd like me to adjust.

@pidanou Let me know if you like the style , adding some CI would be great.

Hey! Thank you so much for the PR. Now that I look at your tests, I am wondering if we should or not focus textinputs (or any model) on init.
How I see it:

Option 1 : All the models should have an input focused at init. All the state is managed on that model and we never focus from a parent. We just need to make sure there is no way a keypress is sent by accident to the child as something is always focused; and that after pressing esc or "leaving" the model, we manage the focus accordingly.
Option 2: We don't focus on init and we focus a child model  input (or any other model) only on parent keypress. This means, there will be no way that a keypress is sent to the child model by accident. But we manage the state in 2 different places.

Option 1 looks prettier as we never care about the child models.
Tell me what you think! If you also think option 1 is the best, I think there are other places where I didn't focus on init.
Apart from that question, the PR is great and I'll be happy to merge it.

Thank you for the thoughtful feedback! After considering both options, I agree that Option 1 is the better approach. Having each model manage its own focus state keeps things encapsulated and makes the code easier to maintain and scale.
Let me know if there's anything you'd like me to adjust.

I will merge you PR for now as I believe changing the message flow is out of the scope of this PR. I will open an issue about changing the state management. Thanks a lot!


Add full CRD support back into KCP controlplane

Add full CRD support back into the KCP control plane.
In the current context of what Kubernetes is, the most straightforward way to do it is to
bring back the aggregation apiServer (even if it will not be used to delegate to external
aggregated servers) since it contains part of the logic that initializes and leverages the
CRD apiExtensions server.

cc @smarterclayton


out of memory using cuda

Found suitable CUDA device [GeForce GTX 760] with<PHONE_NUMBER> bytes of GPU memory
i  16:28:08|stratum  Connecting to stratum server daggerhashimoto.eu.nicehash.com:3353
i  16:28:08|stratum  Connected!
i  16:28:08|stratum  Starting farm
i  16:28:12|stratum  Extranonce set to 0024cb
i  16:28:12|stratum  Subscribed to stratum server
i  16:28:12|stratum  Authorized worker 1CJimUU4g74s825ZYbitCxwP15z6Mxp6hh.WORKER-NAME
i  16:28:12|stratum  Difficulty set to 0.05
i  16:28:12|stratum  New job 0000003afc593b28 target 0013ffec header 36e6e708
i  16:28:12|cudaminer0  set work; seed: #9336adfd, target: #00000013ffec
i  16:28:12|cudaminer0  Initialising miner...
Using device: GeForce GTX 760 (Compute 3.0)
CUDA error in func 'ethash_cuda_miner::init' at line 243 : out of memory.
is there a way to fix this?

Anyone know how to solve this issue ?

Only a GPU with > 2 GB RAM will work these days.

It work, but i solve a problem once in a day lol !
is it normal ?
even with a low hashrate, i thought that i could solved problem every 20-30min ...
why this is not working ?
Thanks for your help !

I had the same "out of memory" problem, using a GTX 830M 2002MiB, in Ubuntu 16.04... after a reboot, I run "nvidia-smi" and 98MiB of device memory was occupied by Xorg and compiz, then I got ethminer running with 1890MiB (total 1988MiB of 2002MiB, only 14MiB left!). As the DAG grows, it won't be possible anymore, but I enjoy while I can :)


Add ember-intl support

This project uses semver, please check the scope of this pr:

[ ] #patch# - backwards-compatible bug fix
[x] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Added support to automatically add ember-intl dependencies to unit tests that specify service:intl in their dependencies
Updated lint rules and fixed new warnings

Approved

@sandersky self-merging


Icon replacement

Hello, I would like to ask how to change these icon symbols? I can't find the entry point for changing them.

Hi @Eventidus
If you go to the home page you will see that the names of the icons are passed by parameter:
https://github.com/onwidget/astrowind/blob/main/src/pages/index.astro#L69
Those icon names follow a pattern <icon pack name>:<icon name>, for example tabler:brand-tailwind
You can find more information about how to use the icon library and how to find the icons in these two links:

https://www.astroicon.dev/
https://iconify.design/


test-callback-order.c     may be wrong ?

``
/* idle_cb should run before timer_cb /
static void idle_cb(uv_idle_t handle) {
ASSERT(idle_cb_called == 0);
ASSERT(timer_cb_called == 0);
uv_idle_stop(handle);
idle_cb_called++;
}
static void timer_cb(uv_timer_t* handle) {
ASSERT(idle_cb_called == 1);
ASSERT(timer_cb_called == 0);
uv_timer_stop(handle);
timer_cb_called++;
}
// ? 应该是timer_cb 先执行吧
static void next_tick(uv_idle_t* handle) {
uv_loop_t* loop = handle->loop;
uv_idle_stop(handle);
uv_idle_init(loop, &idle_handle);
uv_idle_start(&idle_handle, idle_cb);
uv_timer_init(loop, &timer_handle);
uv_timer_start(&timer_handle, timer_cb, 0, 0);
}
``

idle_cb should run before timer_cb ?  or,  timer_cb first ?

I've opened #3619 to remove the test. The test is wrong, that's why it's disabled - timer_cb comes first.


Fix dynamic compilation potentially including files from wrong namespace

Ran into this a few times recently with taiko/mania classes having the same names.
Has been tested against the cases that were failing for me.

TestSceneSliderBar, change+recompile BasicSliderBar<T>, now results in:
[runtime] 2020-04-24 06:22:44 [important]: Recompiling TestSceneSliderBar...
[runtime] 2020-04-24 06:22:44 [error]: Error with dynamic compilation!
[runtime] 2020-04-24 06:22:44 [error]: System.InvalidOperationException: /Users/smoogipoo/Repos/osu-framework/osu.Framework/Testing/TestScene.cs(110,61): error CS0012: The type 'IPrincipal' is defined in an assembly that is not referenced. You must add a reference to assembly 'System.Security.Principal, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
[runtime] 2020-04-24 06:22:44 [error]: Error with dynamic compilation!
[runtime] 2020-04-24 06:22:44 [error]: System.InvalidOperationException: /Users/smoogipoo/Repos/osu-framework/osu.Framework/Testing/TestScene.cs(110,97): error CS0012: The type 'IPrincipal' is defined in an assembly that is not referenced. You must add a reference to assembly 'System.Security.Principal, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
[runtime] 2020-04-24 06:22:44 [error]: Error with dynamic compilation!
[runtime] 2020-04-24 06:22:44 [error]: System.InvalidOperationException: /Users/smoogipoo/Repos/osu-framework/osu.Framework/Testing/TestScene.cs(129,36): error CS1503: Argument 1: cannot convert from 'osu.Framework.Testing.TestScene [/Users/smoogipoo/Repos/osu-framework/osu.Framework/Testing/TestScene.cs(26)]' to 'osu.Framework.Testing.TestScene [/Users/smoogipoo/Repos/osu-framework/osu.Framework.Tests/bin/Debug/netcoreapp3.1/osu.Framework.dll]'

Hmm, something specific to the generics?

Possibly but Animation<T> seemed to work...


auth/authz review

Hey @eco-minio , also tagging @aead as I think you're both security-minded.
This is a fake-ish PR for some of the work I did. minio-authentication-authorization-review.rst is a copy of an existing file, because I can't find a trick to make git think a file is new otherwise.
The goal here is to document all of the various things you can set in a policy document, along with having some examples of usage. Ideally a user does not have to visit AWS docs to be able to create custom policies for use with MinIO.
LMK if you have any questions.

@aead @eco-minio if this otherwise looks good I'll merge it up. Thanks again!

@eco-minio implemented your changes with some additional refinements. LMK what you think.


Add ltypes attribute to return Series

A series makes for a better output when viewing in a Jupyter notebook and matches pandas's dtype behavior

So the WoodworkTableAccessor only has the logical_types property that's the dictionary. What's missing is ltypes which would be the DataFrame (if we decided to match DataTables' exact behavior)

@tamargrey gotcha, edited the title to make that clear. Thanks!