Revert "Bump to fix localdev SSL error for ocp-artifacts"

Reverts openshift/aos-cd-jobs#3371
There is an issue with gen-payload. Reverting to figure out what's up at more leisure.

/lgtm


load baseSites only in smartedit mode

BaseSites data is only used in smartedit. So, it would not be used / loaded when we're in ordinary browsing mode, but only in smartedit mode

To code reviewer/QA:

open SPA, check "network" in browser, make sure there is nobasesites request sent; or check "redux", the baseSite details data is empty.

open SPA in smartedit, check "network" in browser, you should see one 'basesites' request sent; and open "redux", the baseSite details has data.

Tested on dev2.

On SPA, no more basesites request sent.
SPA in Smartedit sends 1 basesites request only.


Revert "[devicelab] remove globals and dependency tracking from technical debt"

Reverts flutter/flutter#80946
Didn't fix the tree

TBR @zanderso

lgtm


Formatting document on HTML does not work

Testing #57296
Repro steps

Open a new HTML file
Write HTML
Run "Format Document" command

Result

Error occurs in output panel

Expected result

Document to be formatted

Output log
[Error - 11:11:31 AM] Error while formatting range for untitled:Untitled-3: last_token is not defined
ReferenceError: last_token is not defined
    at m.beautify (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:7:93697)
    at e.exports (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:7:87722)
    at t.indent_size (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:7:105314)
    at Object.$e [as format] (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:7:105328)
    at Object.format (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:7:175102)
    at Object.t.format (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:7:178185)
    at Object.<anonymous> (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:7:122231)
    at /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:7:115360
    at Object.next (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:7:115465)
    at a (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:7:114211)

Works if I set the untitled file to be html -> verified

Can you expand upon what you mean be setting the file to be html -> verified?

@sathomps I think he just meant that this works after you've set the language mode for a new untitled file. I've also verified that this is now fixed in the latest insiders.


Move from Arrays to Maps for report

For quick access by known keys, it'd be nice to have the report as an object of objects indexed by name rather than an array of objects with the name inside.

closed as implemented in v0.2.x


More context required please

"Specifies the destination prefix of the route to add." does not tell me why & when I would want to use the "-DestinationPrefix" option. What effect does this have on the VPN connection?
"Specifies the metric of the route being added to the VPN connection" does not tell me why & when I would want to use the "-RouteMetric" option. What effect does this have on the VPN connection?

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

ID: 59b602a8-0420-adf8-bc2c-116a336fc37f
Version Independent ID: ac7c834d-81d1-30f9-951e-36bc3a30aaaa
Content: Add-VpnConnectionRoute (vpnclient)
Content Source: docset/windows/vpnclient/Add-VpnConnectionRoute.md
Product: w10
Technology: powershell-windows
GitHub Login: @andreabarr
Microsoft Alias: v-anbarr

@Function-10
The issue has been noted and the document will be revised in the light of your feedback.
Thank you.

Honestly, explaining routing table entries is way beyond the scope of this cmdlet documentation, IMO. Both of those parameters should be of obvious purpose to anybody who is adding entries to the routing table. If you don't know why or when you would modify the routing table, the help for the cmdlet to modify the routing table probably isn't the best place to start.

Honestly, explaining routing table entries is way beyond the scope of this cmdlet documentation,

The rule of definition is that the definition of any word X may not include the word X that is being defined. Quora
The nine word sentence for the parameter does not give an indication of the impact & change of behaviour caused when using it. This is the online help, the expanded version of what you get on the command line.

IMO. Both of those parameters should be of obvious purpose to anybody

The concept of common sense is a long-standing term, based on human experience and people's individual perceptions. Common sense isn't actually common, in either sense: it is different from person to person, and may not be employed even when many editors could agree on what it is in a particular situation.
Thus, when discussing issues of importance to projects on Wikipedia, don't consider your position, or the position that you agree with, or even a position that has consensus, to be "common sense", because it's nothing more than your perception. Your idea of common sense is likely to contradict someone else's idea of common sense
Wikipedia:Common sense is not common

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


2128/fix package selection

Fix for https://github.com/SAP/open-ux-tools/issues/2128
Two issues found

mandatory field not required
translation misconfigured

Wrong branch


Fix issue when parameters editor does not remove old value frame

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


Motion with gratings

I am going to close this one since we will not need motion with gratings in the localizer


feat(#minor); notional-finance; update supply side revenue per new methodology

Deployment Link: https://thegraph.com/hosted-service/subgraph/fortysevenlabs/notional-finance-ethereum

Todo

[ ] Fix build checks
[ ] Update methodology version

@dmelotik @ishraq8 fyi

Checked it out, only issue with the model is that you end up with negative revenue (not surprising, due to the secondary market nature). We will probably want to have supplysiderevenu>0. @dmelotik can give input too.

Ah, so I think before the first review I did we were updating financial snapshots only when the markets closed which was okay for revenue. But now we are updating financial snapshots more frequently (as we should), but the rev logic is still there. @fortysevenlabs can you separate the revenue logic from updateFinancials(). That should only run on market close. And it is okay to have a revenue that is stepped.
Once we finalize these changes, let's not forget to reflect it in the README

@fortysevenlabs can you separate the revenue logic from updateFinancials(). That should only run on market close. And it is okay to have a revenue that is stepped.

Unclear what you are suggesting here. Which revenue logic do you want to separate from updateFinancials()?

@dmelotik pushed the changes to use the prevSnapshot to calculate SSR.
One thing to look into is whether we are ok with the current schema for how we are managing interest rate entities and associating them with markets. We update interest rate entities in-place for a market. This should impact notional because its a fixed rate protocol but something to be aware of.

Yes, this is a new deployment. Let me review.
On Mon, Feb 27, 2023 at 6:48 PM, Ishraq Alim @.***>
wrote:

@.**** commented on this pull request.
In subgraphs/notional-finance/src/setters/financialMetrics.ts
https://github.com/messari/subgraphs/pull/1725#discussion_r1118724134:

const marketHourlySnapshot = getOrCreateMarketHourlySnapshot(event, marketId);
const marketDailySnapshot = getOrCreateMarketDailySnapshot(event, marketId);
const protocol = getOrCreateLendingProtocol();

const market = getOrCreateMarket(event, marketId);

const financialsDailySnapshots = getOrCreateFinancialsDailySnapshot(event);

// total/cumulative revenue calculation
const supplySideRevenueUSD = market.totalDepositBalanceUSD

is this the new deployment?
https://subgraphs.messari.io/subgraph?endpoint=fortysevenlabs/notional-finance-ethereum
For some reason seeing 0 for all revenue.
—
Reply to this email directly, view it on GitHub
https://github.com/messari/subgraphs/pull/1725#discussion_r1118724134,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA76K6BGVUOECWWEP4QZNGTWZSSUDANCNFSM6AAAAAAUYCEHUQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Best,
Tarun

@dmelotik @dhruv-chauhan thoughts on the latest changes? (haven't QAed it yet, lets discuss if this approach makes sense)

You can ignore the lint, or rebase and get less lint rules @fortysevenlabs

The logic looks great to me! I think this is much better.
The only thing I would flag is the _dailySnapshot helper. Unbounded arrays can blow up the storage size of a subgraph quickly
What if we just keep the previous snapshot id as a helper field in the market? And overwrite it each time?

What if we just keep the previous snapshot id as a helper field in the market? And overwrite it each time?

How do we find the previous snapshot id without the array?
I explored market.dailySnapshots to avoid the need for a helper but since it runs into null wasm, this wouldn't work. To deal with unbounded array, we could slice the _dailySnapshots array and only store the last X (may be 5) snapshot ids.

You only need the last snapshot ID right? I would make a field called Market._lastDailyId and then each time a new snapshot is made I would update that field.
Would that not work in this situation?

Yes, I considered that but how are you fetching the prevDailyId? We can't set it to the dailyId.

Updated to store and fetch prev day id without an array.

It has been a while since I have thought about Notional. But why aren't the market name's human readable? Even if the pool doesn't have a name we should derive a name. Market.Name is intended to be a human readable identifier.
Can be something like Notional [input token] Pool [pool number]

It has been a while since I have thought about Notional. But why aren't the market name's human readable? Even if the pool doesn't have a name we should derive a name. Market.Name is intended to be a human readable identifier.
Can be something like Notional [input token] Pool [pool number]

I went ahead with cToken-maturityDate. cToken because I already had a helper method for getting the cToken from the currencyId. maturityDate is ideal vs pool number. The pool names will look like:
cDAI-2021-12-29
cDAI-2022-03-29
cDAI-2022-09-25
cUSDC-2021-12-29
cUSDC-2022-03-29
cUSDC-2022-09-25

Pushed the following changes:

market a) total deposits b) input token balance c) revenue to 0 if negative d) tvl
market names (format now set to ETH-2021-12-29)
update normalizedLendingRate to be decimal instead of %
(new) fix for tvl calculation

@dmelotik for code review

Took a look, only thing that really stands out is TVL. basically 0 at protocol level even though there is TVL in pools. also it is off from DefiLlama. Is there a reasoning for this?

Took a look, only thing that really stands out is TVL. basically 0 at protocol level even though there is TVL in pools. also it is off from DefiLlama. Is there a reasoning for this?

It's because notional deployed new set of contracts recently. We will need to add support for them to start tracking notional again.


Possible logic bug in sentinel code

Looking at: https://github.com/andymccurdy/redis-py/blob/master/redis/client.py#L2163-L2183 it seems like there is a bug. If you have the following scenario:
1 Send command on socket 1
2 Try to read response and get error while doing so (socket disconnect say)
3 Reconnect to master with socket 2
4 Try to read response on socket 2 to message sent on socket 1
5 If the socket has a timeout, then parse_response will eventually timeout, leading to raised exception, but if timeout is None, I think this will block for ever.
Am I missing something? Should parse_response be bypassing the reconnect logic always?

I realize this is for pubsub, but wouldn't this be a race condition at best. When the connection is re-established, the subscriptions seem to be renewed by on_connect. However if the response was sent after disconnect and before re-subscribe, shouldn't it be lost, assuming that the publisher will not send any more responses until it gets some sort of ack from the subscriber?

Line https://github.com/andymccurdy/redis-py/blob/master/redis/client.py#L2181 mostly (entirely?) prevents against your hypothetical. It ensures there's data in the recv buffer of the socket. With that, read_response should never fail.
It's certainly possible there's some edge case that I'm not considering.

That would only help if you use the non default options for non blocking and timeout, correct?
I think in the case that I am seeing (celery), the calling code is actually invoking parse_response separately. One potential issue seems to be that on_connect was not registered as a callback somehow. It looks like in that case the connection was registered just to listen, but the caller does the connection logic itself:
https://github.com/celery/kombu/blob/master/kombu/transport/redis.py#L944-L951
So this is likely more an issue with kombu than redis-py. Even so, should there be some sort of logic in _execute like:
if self.on_connect not in connection._connect_callbacks:
        connection.register_connect_callback(self.on_connect)
connection.connect()

From the Python docs: https://docs.python.org/2/library/select.html#select.select "A time-out value of zero specifies a poll and never blocks." So the default values will check whether there's data available to be read and immediately respond.
The callback registration happens in execute_command (https://github.com/andymccurdy/redis-py/blob/master/redis/client.py#L2152-L2159) when executing the first pubsub command. This is when the connection is first used and seems like a logical place to setup initialization behavior.
It appears that Kombu is doing something different and using the Connection object directly.

I may be missing something here, but isn't default for block True, which means can_read will not be invoked? parse_response would need to be invoked with block=False in order for it to check for data and immediately respond. (ie non default option)
In any case, it definitely looks like the main issue here is kombu using the interface poorly.

Oh, you're right about the block. Sorry. can_read won't get called with the default options.


[Feature Request] templating for the plugin

It seems that the plugin lacks a way to change the output, like this other plugin does.
This projects works better than the other one, but it is very difficult to change the output: I had to resort to changing the core, which I know is not good in the long run.

Most wanted feature for this plugin!

@jimblue this may help you:
I added a new variable alt, and managed to change the way the plugins render the results. It was not as easy as with a template because the author is using a special library to handle the DOM, but it did work.
This is the place were I added the new variable:
                $caption = $image->getAttribute('title');
                $caption2 = $image->getAttribute('alt');

And here is the place were I set the new code adding elements to the code:
                    $figcaption_content = new Element('p',$caption2);
                    $figcaption_title = new Element('strong',$caption);
                    $figcaption_content_break = new Element('br');
                    $figcaption_content->appendChild($figcaption_content_break);
                    $figcaption_content->appendChild($figcaption_title);

                    $figcaption = new Element('figcaption','', ['class' => $figcaption_class]);
                    $figcaption->appendChild($figcaption_content);

This is the resulting image, in this case the caption appears on rollover

This is the hacked function — I do not advice changing the core of the plugin, make sure that you rename the plugin, otherwise your changes will be wiped on — I hope it works out for you
   protected function processFigures($content)
    {
        $document = new Document($content);

        $scope = trim($this->grav['config']->get('plugins.image-captions.scope'));
        $figure_class = $this->grav['config']->get('plugins.image-captions.figure_class');
        $figcaption_class = $this->grav['config']->get('plugins.image-captions.figcaption_class');

        if (count($images = $document->find($scope)) > 0) {
            foreach ($images as $image) {
                $caption = $image->getAttribute('title');
                $caption2 = $image->getAttribute('alt');
                if ($caption) {
                    $figure_classes = [$figure_class];

                    // If there are any `caption-*` classes on the imageadd them to the figure
                    foreach (explode(' ', $image->getAttribute('class')) as $class) {
                        if (preg_match('/^(caption-|figure-).*/', $class)) {
                            $figure_classes[] = $class;
                        }
                    }
                    // $figcaption = new Element('figcaption',$caption, ['class' => $figcaption_class]);

                    $figcaption_content = new Element('p',$caption2);
                    $figcaption_title = new Element('strong',$caption);
                    $figcaption_content_break = new Element('br');
                    $figcaption_content->appendChild($figcaption_content_break);
                    $figcaption_content->appendChild($figcaption_title);

                    $figcaption = new Element('figcaption','', ['class' => $figcaption_class]);
                    $figcaption->appendChild($figcaption_content);

                    $items = [$image, $figcaption];
                    $figure = new Element('figure', '', ['class' => implode(' ', $figure_classes)]);
                    $figure->appendChild($items);
                    $image->replace($figure);
                }
            }
            return $document->html();
        }

        return $content;
    }
}

I think this would best be done with a twig template that could be overridden.. I'll add it as an enhancement for when i get some time (unless someone else wants to take a whack at it and submit a PR)?


[Sage 10] Move built assets to storage/theme/assets

I think it is easier to have 1 directory (storage) for generated files instead of 2 (storage and dist).
As far as semantics go, public — Laravel’s default (Mix’s too) for built files — doesn’t make sense for Sage since it isn’t the only public directory in Sage. dist is a bit better, however, I think it’s still a bit misleading since it’s not the only theme folder which is distributed. storage makes sense, since it a place for the theme to store files it creates.
Obviously, storage/theme/assets is rather long to write out, but that could be remedied by using a mix.config.publicPath helper within webpack.mix.js.

Storage isn't just used for generated files, to be clear. It can be used to simply store stuff that doesn't belong elsewhere. Since Acorn Application can only have a single base path, the default base path is your theme (in our case, Sage) folder. So all the normal Laravel stuff involving storage will use sage/storage/*.
This has conflicts now since I merged that other PR, but I'd rather wait for input from others before merging anyway. My preference is to keep the dist folder, but I'm not super tied to it.

I'm kinda leaning towards dist.
https://app.test/app/themes/sage/storage/theme/assets/styles/app.css just feels a little intense to me.

Let’s stick with dist.


configuring eslint/noConfig handler ?

Description
according to https://github.com/neovim/nvim-lspconfig/commit/fe65c927d53da7f7964e7ac113a9ab4eafdf96f4#diff-abae925898033611bb8a6fe94b196d364c64999f6e6a12bb553486d88c62158dR124
we should be able to add a handler in the config for eslint lsp
{
  handlers = {
    ["eslint/noConfig"] = function()
      vim.notify("handled")
    end,
  }
}

@entropitor this doesn't seem to work, am i doing something wrong?

Looks about right to me. You can always set LSP to debug/trace level to see if the request actually comes in. And check with LspInfo if the server is actually bound to your buffer. Maybe you are not actually registering the server?


Remove types.Incomparable

Currently, types.Compare could return four results: less, greater, equal, and "incomparable". The latter is used in two cases: when comparing NaNs, and when comparing documents. Both those cases should no longer exist after #1151 and #1521. Let's remove types.Incomparable.

To remove incomparable, we need to a way to find if two values are same type or not. The user case is for $gt, $gte, $lt or $lte. For example if we compare {"foo" {$lt: 5}} on value {"foo": "bar"}, it should be false because string and number cannot be compared.
At the moment, it is handled by returning types.Incomparable to indicate 5 and "bar" cannot be compared.
How about exposing Comparable(a, b any) bool from types package which allows us to check if two values can be compared, then we perform comparison. Suggestion welcome.

if we compare {"foo": {$lt: 5}} on value {"foo": "bar"}

We never compare (using types.Compare) those two values. In this example, types.Compare is called with (5, "bar").

string and number cannot be compared

Any two values can be compared. In this case, 5 < "bar" due to types order: https://www.mongodb.com/docs/manual/reference/bson-type-comparison-order/

So this one is about filtering and not about comparison. If I understand correctly, we must filter so we never call (5, "bar") for this specific example.

So this one is about filtering and not about comparison

This issue is about removing types.Incomparable. That value is returned only by types.Compare. This issue is mostly about the comparison.

If I understand correctly, we must filter so we never call (5, "bar") for this specific example.

No. Let's say we run find command with filter argument being equal to {"foo": {$lt: 5}}. We fetch documents and call common.FilterDocument(doc, filter). That function should return true if doc matches filter, and false otherwise. It eventually calls types.Compare(doc.Get("foo"), 5) and compares the result with types.Less:
if types.Compare(doc.Get("foo"), 5) == types.Less {
  return true
}

doc.Get("foo") might return a "bar" string, so we do call types.Compare("bar", 5).


Login bug

Sorry, it seems that all my changes were attached to this PR.
I have found problem in UserREST.java:
https://github.com/Athou/commafeed/pull/963/commits/f4c13efcf0460222ef6f4edbc5e77597d3edaa10


macvim/python loads the wrong Python libraries

This ticket is probably related to Homebrew/homebrew issues #26426,  #17908, #20075, #20392.
I tried the following with vim 7.4 (brew version) and macvim release 83.
With vim
:!which python
/usr/local/bin/python
:py import itertools
:py import os; print os.__file__
/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/os.pyc

With macvim
:!which python
/usr/local/bin/python
:py import itertools
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: No module named itertools
:py import os; print os.__file__
/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/os.pyc

Workarounds
brew unlink python but that is not what I would call a fix.
Also, when I launch macvim from a virtualenv, things work ok!

You can set pythondll like #121 and you may need to set PYTHONPATH environment variable as well.

Setting pythondll is of no help.  Besides, I keep changing python/python3, brew link, brew unlink, virtualenv, etc. Hardcoding some PYTHONPATH for macvim is just a way to make me swear each time I use python3 :laughing: )
However, your comment gave me ideas to investigate more, so I had a look at the content of sys.path on a shell Python, inside vim and inside macvim.
There are 8 entries starting with /usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7 that vim finds and that macvim does not.
What stays mysterious to me is how vim and macvim do not behave similarly regarding Python?
What does vim that macvim forgets to do when loading all it needs for Python?

Vim loads(dlopen)  pythondll dylib or system Python2.7 dylib. That's all. You should ask at Vim mailing list or Vim github project issue If you have further question.

Could we leave the issue open so that to encourage the community (including me, esp. me!) to work for a fix?
I would definitely share my work to be included if I find a proper and clean solution, and closing the issue may give the wrong signal around...

you should share it on Vim community.

Why? There is no problem with vim and nobody complains there about Python issues.
The issue is on macvim which is not behaving how it should.
What's wrong with leaving it the issue open and accepting solutions when they come, as long as they are decent?

Besides, asking around on vim community is not inconsistent with finding a solution to this issue yet to be solved then closed.

The issue is definitely part of Vim (--enable-python=dynamic) and you can compile Vim if_python with static link not dynamic link. Also the MacVim binary distribution can't support any Python environment.

Thank you for the extra information.
Let's see where this will lead us (not today I am afraid...)

Sorry, I close this issue. The reason is that this project issues concentrate MacVim issue not Vim issue. You can still post solutions here for sharing it for MacVim community.

MacVim is linked against the system Python.  Your Vim is linked against a brewed Python.  I don't remember what it was, but at least at one point in time there was a reason why MacVim needs to go to the system Python.  It doesn't matter what which tells you, when you're using the :py command; the :py command will use what Vim/MacVim has been built against, not what's first in your $PATH.
In fairness, @splhack, this might well be a MacVim issue; a user reports that Vim works much better than MacVim regarding Python, and I can see no reason given in this thread yet why this is "just a Vim issue".
But it also might not be a MacVim issue at all; we don't know yet.  @xoolive you didn't say what flags you brewed Vim with.  Also, you should try brewing MacVim with --HEAD --with-python and match as closely as you can the flags you brewed Vim with.  Otherwise, it's not a fair comparison.

Thank you @chdiza. I just brew everything standard :laughing:
For MacVim I would need XCode, so that's why I take the precompiled binaries (note: released two days ago) :)
I understand what you say about :py which points at the system python, so it gives consistency in the series of outputs I pasted. But then, it wouldn't explain why import itertools fail here, does it?

I'll change the default path of Python2.7 dylib, it might change the behavior. or not. If not, you need Xcode in order to build MacVim.

But then, it wouldn't explain why import itertools fail here, does it?

Maybe not, but there are lots of moving parts here besides MacVim.  As you said yourself: "I keep changing python/python3, brew link, brew unlink, virtualenv, etc.".  I don't see how you can do this and expect MacVim to behave consistently.
The thing to do is just install Xcode and brew MacVim from --HEAD and see what happens.

FWIW, I don't usually build with python support.  But I did it just to test, and I get no error upon importing itertools.  Also, MacVim found and built against a non-system python that was in my $PATH.

MacVim snapshot 83 binary works fine Homebrew python with pythondll and PYTHONPATH.
:let $PYTHONPATH="/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/"
:set pythondll="/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/libpython2.7.dylib"
:py import os; print os.__file__
/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/os.pyc

Forgot to mention, my non-system python isn't Homebrewed, and I didn't set either PYTHONPATH or &pythondll.  Don't know whether those datapoints help with anything.

I am really confused now.
If I take the binary (version 83) from the release page, what is supposed to be in my sys.path? (with no $PYTHONPATH set)
I mean, from what I understand, :py calls the Python that was used to build MacVim, which would be the system Python. But then things only get properly loaded when the sys.path contains the library path for the Python that comes first in the $PATH and some libraries fail to load if they come from the system path.
What did I get wrong here?

If I take the binary (version 83) from the release page, what is supposed to be in my sys.path? (with no $PYTHONPATH set)

If you use Vanilla OS X(10.8, 10.9, 10.10, and 10.11, no additional applications, no additiona frameworks, no special environment variables), Vim loads the dylib from /System/Library/Frameworks/Python.framework/Versions/2.7 and sys.path includes /System/Library/Frameworks/Python.framework/Versions/2.7.
:py can call Python 2.7 dylib, the default path is in the system path (the above path). You can set pythondll in Vim command line or .vimrc to specify the Python 2.7 dylib path. It could be your own Python 2.7 build, Homebrew, MacPorts, etc. But you may need to set $PYTHONPATH environment variable (I don't know the reason. Also I have totally no idea $PATH environment variable affects it or not)

I'll change the default path of Python2.7 dylib to the absolute path.

Updated https://github.com/macvim-dev/macvim/releases/tag/snapshot-84
https://s3.amazonaws.com/archive.travis-ci.org/jobs/92834400/log.txt
-DDYNAMIC_PYTHON_DLL=\"/System/Library/Frameworks/Python.framework/Versions/2.7/Python\"

I live it here: https://github.com/macvim-dev/macvim/issues/121

If I change python3dll and create symlink to /Libraries/Framework/.. etc, it able to load correct Python 3.5.
So the problem I see is MacVim (vim?) doesn't initialise correctly Python when it is placed in place with other base than all Python Home specified in configure.
I haven't tested with removing Python2.7 from system (in any way like chmod/mv/rm), but we should start to look in difference of Python2 and Python 3 initialisation. And we should change both if initialisation depends only on predefined variables but not in python3dll and pythondll.

@splhack setting PYTHONPATH environment is not work in any way

As I said, it is Vim issue not MacVim issue. Please go to Vim issues or Vim mailing list.

snapshot-85 https://github.com/macvim-dev/macvim/releases/tag/snapshot-85 included $PYTHONHOME fix experimentally. It may be changed in the future, and, still, it is Vim issue, please go to Vim issues or Vim mailing list if you want to discuss about it.

Hi,
I still have same issue with MacVim running on Mojave, (/Applications/MacVim.app/Contents/bin/vim)
Build it without python3 fix the problem.
brew reinstall  -s --with-python@2 --without-python macvim

Details for macvim that come from brew bottle:
Press ENTER or type command to continueVim: Caught deadly signal ABRT
Vim: Finished.
fish: 'vim' terminated by signal SIGABRT (Abort)

$ brew list | grep python
python
python-markdown
python@2

The issue is not preset when i'm using build-in vim (/usr/bin/vim):
:version
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Aug 17 2018 15:22:29)
Included patches: 1-503, 505-680, 682-1283
Compiled by<EMAIL_ADDRESS>


.Deploy-Arm-Azure.ps1 issue with postgres version

the prompt to specify a postgres version keeps failing me. I've tried 11.4, 9.6, 9, 11 and all fail. I'm passing it using single quotes''. Please advise, with this failure the WorkspaceDeployment and the SolutionDeployment both succeed so I think it's just the PostGres that isn't created.
Below is error message (RESOURCE ID removed)
{ "id": "/subscriptions/9aa7299f-fdd4-47b8-acaf-825d2bdc8bba/resourceGroups/AKSThirdAttempt/providers/Microsoft.Resources/deployments/deployment/operations/3F5E3780F0842002", "operationId": "3F5E3780F0842002", "properties": { "provisioningOperation": "Create", "provisioningState": "Failed", "timestamp": "2019-07-26T17:08:48.2614439Z", "duration": "PT1M1.0999929S", "trackingId": "0cbfe728-4ada-4d3c-80c8-ccd10fb1e5cc", "statusCode": "Conflict", "statusMessage": { "status": "Failed", "error": { "code": "ResourceDeploymentFailure", "message": "The resource operation completed with terminal provisioning state 'Failed'.", "details": [ { "code": "InvalidVersion", "message": "Version ''11'' is not supported." } ] } }, "targetResource": { "id": "/subscriptions/9aa7299f-fdd4-47b8-acaf-825d2bdc8bba/resourceGroups/AKSThirdAttempt/providers/Microsoft.DBforPostgreSQL/servers/ttpggneoho5zai6dw", "resourceType": "Microsoft.DBforPostgreSQL/servers", "resourceName": "ttpggneoho5zai6dw" } }}

https://docs.microsoft.com/en-us/azure/templates/Microsoft.DBforPostgreSQL/2017-12-01/servers this shows the supported versions as 9.5, 9.6, 10, 10.0, 10.2, 11

Update: I worked through the issue. turns out the prompt didn't need any type of quotes around it. Since the version type is a number I'd consider revising the wording of the prompt as "Please provide string value for 'pgversion' (? for help):" led me to believe my input needed to be '11' and not 11. Just a thought!


[Feature] OpenTelemetry instrumentation

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
Currently, the project exposes metrics in a vendor-specific format (Prometheus) and logs via slf4j.
Instrumenting the project with OpenTelemetry would allow

to get a better understanding of the overall performance and behavior of the application (metrics, spans, distributed traces)
in a vendor-agnostic way, allowing users to choose their preferred OpenTelemetry backend

Solution

Ingesting existing metrics that are currently exposed via Prometheus and JMX into OpenTelemetry
Ingest logs into OpenTelemetry (resource 1, resource 2)
Instrument the application to gain additional insights (distributed tracing)

Additional resources: OpenTelemetry Observability Primer
Anything else?
This feature has low priority.
Willingness to contribute

[X] I'm willing to submit a PR!

Hi @michaelkoepf , how do you plan to integrate OpenTelemetry? Is it like flink-metrics-otel in Flink metrics? If yes, you can refer the code of flink-metrics-otel, because the metric system of Fluss is very similar to Flink.


Scylla CDC Connector Tests

Currently CDC connector is severely lacking in tests department. This results in unnecessarily long processes of manual testing, even when introducing the smallest of changes. And even after such testing it is hard to tell if it was extensive enough.
What kind of tests would be useful:
Unit tests – Currently there are 0 unit tests. First step should be covering at least the critical parts. Would greatly help with quickly checking nothing important is breaking with new changes.
Integration tests – Here debezium-connector-cassandra may be a good reference. We could similarly set up Scylla using Testcontainers and java-driver. After that thoroughly check if the CDC tables are translated into Kafka messages correctly, without yet sending them.
E2E – The most expensive type, but would save a lot of pain from setting up every component manually. It would be good to at least have 1 setup with recent Scylla version and one type of Kafka cluster.
Other nice-to-haves:
Stress tests – Mainly to see if anything breaks only under load. We could also think later about tracking performance.
“Nemesis” type of tests – Scenarios where we intentionally throw a wrench in-between important operations. For example: does the connector correctly resume if we crash it after processing pre-image event but before processing the insert it is related to? Will the offset be correct and pre-image reread upon restart?
Other debezium connectors that may be used as a references:
https://github.com/debezium/debezium-connector-spanner
https://github.com/debezium/debezium-connector-jdbc (This is a sink connector)
https://github.com/debezium/debezium-connector-cassandra
https://github.com/debezium/debezium-examples (Not a connector repo, but has some end-to-end examples)

For the E2E and stress tests we can/should use SCT infrastructure and intergrate it with our current tests.
We can either open a new task here or in "Qa-tasks" repo and assign it to @fruch, so we can plan it for next sprints.


Console

console error report with wrong line number and console.table is undefined

Thank you for reporting the issue.


Console Error (Mentioned in Issue #81 Comment)

This was the error I've mentioned before:

It seems to related to how the Load CSV and Create New Table buttons doesn't work properly at times.

Closed for the same reason #81 was.

Now this error appears while the Load CSV button doesn't work (instead):

Do you have any way of reproducing this error? After I fixed it I spent about 30 min yesterday refreshing the page and using the button in our 3 target browsers and didn't encounter it once.

Not exactly sure, but it seems I have to do something after loading a csv file. I'd normally just play around with colors and buttons, then I reload the page and gets the error at times.

One again I might have fixed this. Let me know if you see it pop up again.

Okay.

I think I'm getting the same/a similar error again in Firefox:

I haven't gotten this issue


Insertion without a model

How to use bun as a query builder to insert a row into the database?
When I try this:
_, err = r.db.NewInsert().
	Table("custom_path").
	Value("port_a_id", portAId).
	Value("port_b_id", portBId).
	Value("checkpoints", strings.Join(checkpoints, ",")).
	Value("path", "st_geomfromgeojson(?)", pathJson).
	On("conflict (port_a_id, port_b_id, checkpoints) do update").
	Set("path = excluded.path").
	Exec(r.ctx)

I get an error "bun: Model(nil)".

Looks like a bug.

Hi, I'd be very interested by this feature aswell. Do you have a plan to implement this?
Thanks

Still not fixed? Well


Resource cloudflare_access_mutual_tls_certificate doesn't work with api_token authentication

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version

Terraform version : 0.15.3
Cloudflare provider version : 2.21.0

Affected resource(s)

cloudflare_access_mutual_tls_certificate

Terraform configuration files
provider "cloudflare" {
api_token = var.api_token
account_id = var.account_id
}

resource "cloudflare_access_mutual_tls_certificate" "ca_certificate" {
zone_id = "zone_id"
name = "Root CA certificate"
certificate = "certificate content"
associated_hostnames = ["some.hostname.com"]
}

Debug output
No response
Panic output
No response
Expected output
Resource created successfully
Actual output
│ Error: error creating Access Mutual TLS Certificate for zone "zone_id": HTTP status 405: POST method not allowed for the api_token authentication scheme (10000)

Steps to reproduce

Use the code I shared above
Run Terraform plan then Terraform apply

Additional factoids
The api_token used for the creation has the permission Access: Mutual TLS Certificates:Edit along with other permissions.
The workaround is to use the legacy Email + API Key for authentication, but we want to use the API token.
References
No response

I'm afraid this isn't something we can fix in the Terraform provider as this is something we need the service team to adjust. The provider only wraps the public API with lifecycle improvements, we don't manage any of this internally. Have you raised a Cloudflare Support ticket for this that I can forward to the right place?

@jacobbednarz Yes I have raised a Cloudflare Support ticket for this, and they recommended creating an issue in this repo.

What is the ticket ID?

2189409

The Access folks have a ticket in their backlog to address this one. In
the meantime, you'll need to use API key + email for authentication.


Support for the has_one association in perform_search_on_associated_collection

Why does not exist has_one association in the:
  # rails_admin.rb

  list do
    .....
    field :has_one_associated_field, :string do
      searchable [{:has_one_associated_field => :name}, {:has_one_associated_field => :description}]
    end
    .....
  end

  #rails_admin/lib/rails_admin/adapters/mongoid.rb

  ......

  def perform_search_on_associated_collection(field_name, conditions)
    target_association = associations.find{|a| a[:name] == field_name }
    return [] unless target_association
    model = target_association[:model_proc].call
    case target_association[:type]
    when :belongs_to, :has_and_belongs_to_many
      [{ target_association[:foreign_key].to_s => { '$in' => model.where('$or' => conditions).all.map{|r| r.send(target_association[:primary_key_proc].call)} }}]
    when :has_many
      [{ target_association[:primary_key_proc].call.to_s => { '$in' => model.where('$or' => conditions).all.map{|r| r.send(target_association[:foreign_key])} }}]
    end
  end

  ......

Currently I have a problem with a filter based on has_one association.
Thanks.

I have some problem with
has_one, through relation

Rails Admin is worst

Can't handle has_one through association. No proper document on Custom Autocomplete Select Box with selected value etc...... Sucked my all time in digging this.

rails_admin has supported has_one through since this was merged in 2016: https://github.com/sferik/rails_admin/commit/b92a4d1a30b706d08df2aee4d0a59dad698a0552
To configure:
class Model
  rails_admin do
    list do
      field :other_model do
        eager_load true
        searchable [:id, :some_other_field]
      end
    end
  end
end

I am no longer aware of it, I will close it.


Omni for Mattermost

Add a new theme to the organization
If you have already created a theme, make sure you check the following requirements:

[x] The theme follows the template;
[x] The theme follows the Omni Color Palette;
[x] The branch name is main (we will keep this name pattern from now on);
[x] The theme has correct instructions on how to install it;
[x] The repository name is lowercase and contains just the application name. E.g. visual-studio-code.

Application: Mattermost
Application URL: https://mattermost.com
GitHub URL: https://github.com/gprando/mattermost

Thanks for creating another great theme, @gprando! It looks awesome 🚀
Since you are already a member of the organization, you can transfer your repository!

@jpedroschmitz transfer completed, added links and opened #PR39

Thanks, @gprando!
Make sure to read the maintaining a theme section.
It's a small guide with some tips on how to maintain an Omni theme!


Debugging guide references DEBUG_FD environment variable which was removed from node debug module

The debugging page of the Guide mentions several environment variables which can be used to control the debug module.  However, DEBUG_FD was removed mid-2017 (https://github.com/debug-js/debug/commit/ff432e76e98f4224917af75a2d2dd1057edff3ac).  It might be better just to refer to the debug documentation, and mentioned that there are some environment variables, without listing them.  That would avoid this problem and avoid getting out of sync with any future changes.

Express stil has DEBUG_FD, though, which is why we document it. Is it not working for you?

Hi Doug.  That wasn't obvious to me from the documentation -- I assumed the express docs were just excerpting the debug docs (linked shortly later) for convenience.  I don't know much about express and was diving in to diagnose a problem.  Maybe it would have worked, but I just followed the link shortly to the "debug" docs, saw no mention of DEBUG_FD there, and got confused.
Maybe the fix here is to document that some/all of these environment variables are interpreted directly by express, regardless of how "debug" interprets them, or something like that?
(I ended up solving my problem a different way because it turned out that, in my context, both stderr and stdout were captured together with explicit app-level logging, but to a "secret" extra log file, by a different part of our overall setup.)

There is only a small mention of the debug module on the debugging page, so I guess we can just remove that so our docs stand alone so there is no confusion -- it documents how to debug in Express.


Address feedback in #101467

This PR addresses feedback in https://github.com/microsoft/vscode/issues/101467#issuecomment-684724245
Couple of notes:

enum now only used to provide autocompletion suggestions and renamed accordingly.
Apparently errorMessage will be shown only from first item in anyOf
type: 'number' should be first otherwise number validation Value is above the maximum of 1000. won't be shown for numbers, however it can be moved lower to have same error message for 1001 and "10001"

/cc @alexdima @jrieken

Thank you!

@alexdima I've remembered why I haven't used clampedInt - font-weight: 10000 will be totally ignored by browser therefore my initial check was min <= val <= max to match browser behavior. Demo
I think it's better to match browser behavior and revert code back to  min <= val <= max probably add explicit isNaN check if needed.

I agree that this is the behaviour of CSS, but these are editor settings, the behaviour of out-of-range values doesn't need to conform to the CSS specification. I think it is OK if we are consistent with our other settings, so we clamp small/large values. IIRC we do the same for fontSize and other editor settings.


⚠ Bofrost.fr has degraded performance

In ef952cf, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1168 ms

Resolved: Bofrost.fr performance has improved in e918a62 after 7 minutes.


Can't export opened database to file.

Is it possible to export the opened database to a file.
I plan to upload it to a remote server.

Probably - it works just like any other database. So fetch your data and save to a file.


[PRODUCT]: New NOHRSC Grids (Freezing Degree Days and Thawing Degree Days)

Product Source (URL Preferred)
https://www.nohrsc.noaa.gov/  (Interactive Snow Information, Physical Element MENU: Climate Data)
https://noaadata.apps.nsidc.org/NOAA/G02158/
NOHRSC POCs and Contact Info Available in the initial email request to the CWMS-GriddedData-Support DLL on January 16, 2024.
Product Description
thawing degree day can
Also the accumulated freezing degree day to predict frost formation in the ground.
Also might be helpful to predict Ice Bite on Mississippi (causing sudden drop of approx. 30% in flow rate in the Mississippi River ) and possibility ice movement in rivers.

Accumulated Freezing Degree Days:  Help predict the frost out conditions
Accumulated Thawing Degree Days: Help predict frost formation in the ground, Ice Bite, and ice movement.

Product Format

[ ] Geotiff
[ ] Grib2
[ ] NetCDF
[X] Other (Please provide description below)

Format Description
https://nsidc.org/data/g02158/versions/1

The freezing and thawing degree day grids are not located at the Snow Data Assimilation System (SNODAS) Data Products at NSIDC, Version 1 repo. Sent a message asking where they may be obtained.


Automated Package Upgrade

:tada: This PR is included in version 7.0.213 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Filtering

Resolver operators may want to provide filtering as an optional service, or may be forced to do it. We need text covering this topic.

We're not really using these issues to track stuff any more.


Totals

I see I could specify itemType to get the total, but how do I set the itemtype ?

itemtype is a interface that your model has been implemented. You can create your own interface and define type and then you can find items that implements your interface.


Add CSFML Support

I would like to see support for the CSFML graphics library located here(https://github.com/SFML/CSFML).

@fudgenuggets12, where is CONTROL and portfile.cmake file for CSFML?

You accidentally created a PR from one of our internal branches. Please create a new PR from one of your forks instead.


tlog - nbd : multiple connection or tlog client for each vdisk

We currently only use one tlog client per vdisk.
Need to check if it is a good idea to make it multiple clients or tcp connections in single client.
the server also need to be adapted to to handle unordered messages

Why would we need multiple clients per vdisk? Can't we use a single client for sending blocks in parallel?

Can't we use a single client for sending blocks in parallel?

No, if we only have one TCP connection.

I updated the issue's title and description, our main point is using multiple connections.

Ah yeah, in that case, this does indeed sound like a good proposition 👍

my tests using bigger image

without tlog : 5.8 s

tlog doesn't store the messages -> no flush:

current code : 11 s
increase nbd transactionChCapacity to 100 (high watermark of tlog flusher) : 10 s
use 2 tlog clients : 9 s
use 2 tlog clients, set transactionChCapacity to 100: 9 s

tlog store the messages

current code : 18 s
use 2 tlog clients, set transactionChCapacity to 100: 18 s -> yes, no improvement

another interesting test data
with flush, but no capnp encoding. appended all block's data into one big []byte

current code : 12 seconds
2 tlog clients : 11 seconds

but please note that when i said 2 tlog clients, the tlog server hasn't been modified to handle the needed changes (i.e. : handle unordered data). This handling itself can slow down the server a bit.

in latest develop:

with single client : 12.7 s
with 2 clients : 11.1 s

i'm still hesitant to use 2 clients now because:

it added complexity to server (handling unordered data), it will slow the server down by some factor
not sure about complexity in the client

i'm just thinking to use two clients:

one to build the capnp
one to send the data to server

lets see how it goes.

i'm still hesitant to use 2 multiple clients now because:

but i guess it is something that unavoidable.

i'm just thinking to  add another worker to build the capnp object

it only gave small (0.1 s from 12.4 s) improvement.
Instead of using mutiple clients, i think it is better for the client to create multiple (configurable) number of tcp connections. It will be simpler to use.
But before adding multiple tcp connections, we need to add unordered message support to the server.
The C++ version support this behavior by simply using C++ std::map, which is ordered by it's key.
For Go, i can see two alternatives:

use slice like now, but we sorted the slice.
use 3rd party data structure : https://github.com/emirpasic/gods, https://github.com/Workiva/go-datastructures, etc....

depend on #143

@GlenDC
trying to answer your question  at
https://github.com/zero-os/0-Disk/pull/209#issuecomment-305071125

No, it is single client with multiple TCP connections

Oh... Well if you can make it work, I'm all happy with that.
How will that work with the handshake and stuff?
Can you describe some flows, as I really don't se how that will work.

There won't be any problem with handshaking, we do these:

each connection do it's own handshaking
one of the connection failed to handshake, client.New returns error

The first thing we need to do is to encapsulate the socket, reader, and writer into single connection struct.
Otherwise the reconnect logic will become a mess.
Marked this as low priority,

Feel free to move it to milestone alpha-5, if you feel like it isn't required to start using tlog.


Require using make_template() if providing a template to ChunksToZarr?

Currently we support passing an xarray.Dataset full of chunked dask.array objects as template into ChunksToZarr.
This is convenient in simple cases, but makes it easy to write pipelines that are super slow to setup, if you pass in a chunked Dataset with many small chunks (e.g., the default output of xarray.open_zarr()).
The breaking change here would be to require that the template argument was created via make_template(), by checking that each dask.array argument in the supplied Dataset only consists of a single chunk. We would also make zarr_chunks required when supplying a template, because it makes no sense to copy chunks from a template if using make_template.

As an alternative, we could instead perhaps use make_template() internally inside ChunksToZarr.

As an alternative, we could instead perhaps use make_template() internally inside ChunksToZarr.

I implemented this in https://github.com/google/xarray-beam/pull/62


Round 7 - Food Inventory - Food List

On the list of foods, left justify the food list dividers (alphabet letters A, B, C...) so that they line up directly above (or closer to) the alphabetized food names (list). This will reduce visual scan on a dense display.

Fixed.


Update core changelog with attribution for a curl endless loop bug fix.

Here's the fix:
https://github.com/Azure/azure-sdk-for-cpp/pull/5589
cc @CurtizJ

Note that we need to approve the original PR before we submit this PR.

Note that we need to approve the original PR before we submit this PR.

It's approved. Shall I go ahead and merge it, @LarryOsterman?


Add default announcement banner

Adds a default announcement banner to be created by the initializer

@Maffooch This on causes a bug. If the init container starts more than once, multiple announcements are created, which results in errors, for example when you edit a user.
Is suggest to revert this MR.


[SPARK-46297][PYTHON][INFRA] Exclude generated files from the code coverage report

What changes were proposed in this pull request?
This PR proposes to exclude generated files from the code coverage report, pyspark/sql/connect/proto/*.
Why are the changes needed?
For correct test coverage report, and make it easier to read.
https://app.codecov.io/gh/apache/spark/commit/1a651753f4e760643d719add3b16acd311454c76/tree/python/pyspark/sql/connect/proto
Does this PR introduce any user-facing change?
No, dev-only.
How was this patch tested?
Manually tested.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master.


tag/release a version of this repo for project-monai/monai v0.5.1

Is your feature request related to a problem? Please describe.
v0.5.0 is released https://github.com/Project-MONAI/MONAI/tree/0.5.1
would be great to select and tag a corresponding tutorial release

I think we could check out a new release branch, replace all the pip install commands with pip install monai==0.5.1 and then tag a milestone. or should we go for a gitflow approach (https://github.com/Project-MONAI/MONAI/issues/2041) for this repo as well? cc @Nic-Ma  @ericspod @rijobro

Gitflow is maybe a bit too complicated for this repo, we don't anticipate making a lot of changes and it's not a dependent of other things. We should set the install command to be for the most recent released version of MONAI and whenever a new release goes out we need to check all the notebooks and increment their versions.
The alternative is to set existing notebooks as monai>=0.5.1 and leave those alone, new notebooks needing new features would have later versions as their minimum. This starts fragmenting what version notebooks rely on but if people have to use a specific older version of MONAI there will be notebooks they can run with it to test it works at the very least.


Goals for 9/27

@ebridge2

[ ] add covariance and mean prediction using old method (consider covariance everywhere a condition is present, and weight time points by the amount of signal present). Deliverable: same simulation results as last week, but this time with the new "initializer" method.

[ ] make markdown file with fMRI acquisition settings.

[ ] find peeps to use my tutorials for functional connectome estimation (probably group members). Show that they can reproduce my mnr results starting with nothing but the repo and a link to download fMRI scan data from (ie, BNU1).

[ ] review merging eric-dev-master with greg.

[ ] send simulation results to kara and susan. Hopefully they will be excited.

@ewalke31

[ ] implement standard EM in R
[ ] if possible, try to see where EM KF algorithm is going wrong
[ ] show in R markdown w/ examples

@ronnyb29

[ ] Do first chapter of probability primer (https://www.youtube.com/playlist?list=PL17567A1A3F5DB5E4) and take notes
[ ] Help Eric with tutorials on functional connectome estimation and take notes if time permits

@tagarwa2

[ ] Implement Kalman Filter in R
[ ] Learn how to use R Markdown

@ebridge2 @ewalke31 👌 ; @ronnyb29 add yours?

@02agarwalt
run on single subject of your choice. Provide me with plots, from R, of the downsampled timeseries file (y axis is the intensity, or the value, x axis is the time, and the grouping is by roi label), as well as a single correlation matrix from this subject.
@ewalke31
Run fngs pipeline on subset of BNU_2 dataset. The data is found at /mnt/ssd2/data_fmri/BNU_2. Run on all subjects found here.


Add base64 image encoding to Image object

HTML supports embedding data uris with base64 encoded images into the src attribute of an img tag. Something like src=data:image/png;base64,encodeddata...
This would be a great addition given Python provides easy encoding functions (though different between 2.7 and 3)
More info here: http://www.bigfastblog.com/embed-base64-encoded-images-inline-in-html

Fixed with 1741e7a970902f64d156736441cd50301cedaf78


test--5-main

Please pull this in!

 Quality gate: Failed
26 Issues

 24 Code issues
 2 Anti-patterns

Details

 Quality gate: Failed
26 Issues

 24 Code issues
 2 Anti-patterns

Details


Accuracy with NN drops, compared to GLM

Hi,
first, thank you for this library. I do appreciate a lot the work.
I made experiments to compute the laplace approximation and then evaluate the resulting bayesian model.
I did observe that the accuracy of the bayesian model is similar to the original when using "glm" as predictions type. But it significantly drops when using "nn" as prediction type.
I did experience this situation with different networks. But a simple example was a fully connected layer applied on the IRIS dataset.:
FeedForward(
  (flatten): Flatten(start_dim=1, end_dim=-1)
  (linear_relu_stack): Sequential(
    (0): Linear(in_features=4, out_features=16, bias=True)
    (1): ReLU()
    (2): Dropout(p=0.2, inplace=False)
    (3): Linear(in_features=16, out_features=8, bias=True)
    (4): ReLU()
    (5): Dropout(p=0.2, inplace=False)
    (6): Linear(in_features=8, out_features=3, bias=True)
  )

With "glm", I obtain an accuracy of 1 on validation dataset, but with "nn" it drops to 0.25.

Hi, this is somewhat expected behavior for the NN predictive, see https://proceedings.mlr.press/v130/immer21a.html. However, you can control this by increasing the prior precision of the Laplace approximation using the prior_precision argument. Setting it high enough can always make the NN predictive work as well but we recommend using the default glm for the reasons explained in the linked paper.

Thank you for the fast reply.
Glad to see it does not come from my implementation.
I preferred to use the NN predictive as I want to make 2 additional analysis:

analyze the output (mean and variance) of each submodule by adding a forward hook
analyze the output when the model is pruned
These approaches will however not work with GLM predictive, because the original model does not seem to be called. Do you have recommendations for me to do the above tasks with GLM predictive ?

I think if you modify weights, e.g., set them to zero during pruning, it should still be reflected in the GLM predictive as well. However, getting layer-wise predictive uncertainties is more tricky and requires computing the Jacobians per layer manually, I think.

About the pruning, it seems a bit more complex. I see changes in the performance after pruning, so I assume the pruning is effective. However, when I undo the pruning, the performance is still bad. Can it be that the backend still stores information from previous runs ?
About the jacobians, it seems to be the most intuitive approach, however very arduous, as I would have to make it a new forward pass per layer. My intuition tells me that if I average the jacobians \nabla of shape (batch, parameters, outputs) over the last dimension (outputs), I could already obtain information from it

1- It did come from my implementation. I used additional dropout layers to use Monte Carlo Dropout, however some of them were either still in training in eval or applied multiple times. Which led to bad performance.
2- I could not find an intuitive way to deal with it, so I sticked to Monte Carlo Dropout to estimate the uncertainty values for this approach.


Fixes m365 spo page set

Closes #5938

Thank you @MathijsVerbeeck, we'll review it anytime soon.

Merged manually, thanks! 👏


C# module unloaded after injected

My dll just unloaded after it's successfully injected. This is my run() code

public void Run(RemoteHooking.IContext InContext, String inChannelName)
{
try
{
MessageBox.Show("success");
Winsock_Recv = LocalHook.Create(LocalHook.GetProcAddress("ws2_32.dll", "recv"), new recvw(myrecv), this);
Winsock_Recv.ThreadACL.SetExclusiveACL(new Int32[] { 0 });
Winsock_Send = LocalHook.Create(LocalHook.GetProcAddress("ws2_32.dll", "send"), new sendw(mysend), this);
Winsock_Send.ThreadACL.SetExclusiveACL(new Int32[] { 0 });
}
catch
{
MessageBox.Show("fail");
}
}

Messagebox (success) shows up and then dll unloaded. Do i need to create endless loop?

Endless loop (with sleep), WaitEventHandle or similar.


Add RSpec/PendingWithoutReason cop

Motivation
I think it is a bad practice to use pending or skip without giving any reason. So, how about discouraging such practices by default?
About naming
Since the convention of calling "pending or skip" as "pending" seems to already exist as RSpec/Pending, I decided to name it RSpec/PendingWithoutReason here as well for consistency. Incidentally, other conventions of calling "pending or skip" as "skip" also seem to exist in RSpec::Cop::RSpec::SkipOrPending#skipped_in_metadata?.
Also, RuboCop::Cop::RSpec::Pending says "pending or skip", while this RSpec::Cop::RSpec::SkipOrPending says "skip or pending". I decided to use "pending or skip" in this case in preference to RSpec/Pending.

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[x] Added tests.
[x] Updated documentation.
[x] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

If you have created a new cop:

[x] Added the new cop to config/default.yml.
[x] The cop is configured as Enabled: pending in config/default.yml.
[x] The cop is configured as Enabled: true in .rubocop.yml.
[x] The cop documents examples of good and bad code.
[x] The tests assert both that bad code is reported and that good code is not reported.
[x] Set VersionAdded: "<<next>>" in default/config.yml.

Regardless of whether this Cop is added or not, the other three commits are changes that should be put in independently of each other, and this Cop is dependent on those changes. That is why I am adding 4 commits this time. If you have a policy that each should be separated into a Pull Request, please let me know so I can try to separate them.

Thank you, @r7kamura !


Possibility to serialize Color after deprecation of Color.value

Use case
The getter Color.value is now deprecated:
@Deprecated('Use component accessors like .r or .g.')
  int get value {
    if (_a != null && _r != null && _g != null && _b != null) {
      return _floatToInt8(_a) << 24 |
          _floatToInt8(_r) << 16 |
          _floatToInt8(_g) << 8 |
          _floatToInt8(_b) << 0;
    } else {
      return _value;
    }
  }

This came in handy for serialization when storing color in e.g. database or shared preferences, since there is a default constructor that takes the value for initialization:
const Color(int value)
      : _value = value & 0xFFFFFFFF,
        colorSpace = ColorSpace.sRGB,
        _a = null,
        _r = null,
        _g = null,
        _b = null;

Proposal
There should be a way to get the color as a primitive data type that allows colors to get serialized and de-serialized.

This is the flip-side of https://github.com/flutter/flutter/issues/157128, which seems reasonable.


Rewrite BSplineGenerator::generateBSplines().

The current implementation has an O(2^k) complexity. Rewrite it to remove the exponential scaling.
One possible solution is to implement the recursion not on the level of each individual BSpline, but on the set of all BSplines. I.e. generate all splines of zeroth order, use the recursion relation to generate all splines of first order, then use these to generate all second order splines and so on.

Done with commit 570e11ce1b0d07c059f68d548511b0672fb53348.


Fix build for OSX

Issue #699: build broken on macos
Description of changes:

Add ifdef for OSX to get_gmt_offset.
Remove includes from OSX CommonCrypto.
Remove dangling get_nanoseconds_since_epoch.
Add some EOF newlines.

Notes:
@colmmacc The CommonCrypto includes are from the initial commit. Was there a design decision that I'm messing up by just going Openssl for everything?
@JonathanHenson Is that __USE_BSD still needed? That is, was it just supposed to be targeting OSX?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

When s2n was first released, the CommonCrypto implementations were just faster ... that's the reason why. I think they inlined to SHAX instructions, but the OpenSSL ones were still C.

Got it, sounds like ripping it out is probably OK for now. I think the alternative is figuring out a good ifdef strategy for the code paths that need x509 or the hash functions, which is definitely doable, but outside the scope of just getting things working again.

Looks good to me, but there's a merge conflict. Can you either merge or rebase?

I tried building this on my mac (10.11.6), and got errors:
s2n_config.c:38:11: error: implicit declaration of function 'clock_gettime' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
GUARD(clock_gettime(S2N_CLOCK_HW, &current_time));
^
s2n_config.c:38:25: error: use of undeclared identifier 'CLOCK_MONOTONIC'
GUARD(clock_gettime(S2N_CLOCK_HW, &current_time));
^
s2n_config.c:29:22: note: expanded from macro 'S2N_CLOCK_HW'
#define S2N_CLOCK_HW CLOCK_MONOTONIC
^
s2n_config.c:50:25: error: use of undeclared identifier 'CLOCK_REALTIME'
GUARD(clock_gettime(S2N_CLOCK_SYS, &current_time));
^
s2n_config.c:32:23: note: expanded from macro 'S2N_CLOCK_SYS'
#define S2N_CLOCK_SYS CLOCK_REALTIME
^
3 errors generated.

@danielsn This is the OSX < 10.12 issue I mentioned earlier:

Also it looks like Apple only actually added clock_gettime in 10.12/Xcode8, so earlier versions will still break with this. I figured that can be shimmed-in later if someone still needs to build for old stuff.

The build breaks for all OSX without this patch, and breaks for OSX < 10.12 with this patch.


Module Inclusion Cycle with addsTo

Maybe I'm misunderstanding how the graph works, but I'm getting cycle errors at compile-time when using includes and addsTo like so:
@Module(
  includes = {
    ChildModule.class
  })
public final class ParentModule {
  // ...
}

@Module(
  addsTo = ParentModule.class)
public final class ChildModule {
  // ...
}

It compiles and runs fine if I get rid of the addsTo attribute and add complete = false, library = true to both modules, but that's gross because it practically disables all the compile-time validation.
User error or bug?

Still a massive help after all these years, thanks @mattprecious and @cgruber


Fix CI failures

Attempting to see why CI fails... hard.

Do we know what about setuptool 66 broke things?

I just succeeded in recreating this on my own computer with docker.
I'm still investigating, but if you want to merge this into the main branch I will open a new PR with a more permanent fix.


Fix all type errors except for ripple imports

This fixes packages at the version needed to successfully import them into a local project. Does not fix the issue with relative ripple imports.
#936

Codecov Report

:exclamation: No coverage uploaded for pull request base (rc0.14.0@bdb3f43). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             rc0.14.0     #951   +/-   ##
===========================================
  Coverage            ?   94.12%
===========================================
  Files               ?       86
  Lines               ?     3644
  Branches            ?      576
===========================================
  Hits                ?     3430
  Misses              ?       90
  Partials            ?      124

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 bdb3f43...eede28c. Read the comment docs.

Important note: this does not modernize all type dependencies within the repo, but does reduce existing errors to 0 and automatically checks to ensure future changes do not break anything :-)

That's not good. I'll play around with it locally then :-(

@moog16 https://travis-ci.com/material-components/material-components-web-react/jobs/210813262 shows a build that works, except for the (now detected) ripple errors.

https://github.com/material-components/material-components-web-react/pull/952 is a better fix. Will wait for commit.

@arichiv #952 is merged

@moog16 this is ready for merge then. The next PR will uncomment the travis-ci test and fix ripple imports. This fixes all other package version errors :-)

Manually tested...everything looks good :)


Android: Story titles sometimes narrower than screen

See screenshot: https://getsatisfaction.com/newsblur/topics/feed-list-view-compressed

Revisiting this, the root cause is that Android impl of a TextView with both the ellipsize and maxlines attributes is just incredibly buggy.  (see https://google.com/search?q=site%3Acode.google.com+textview+ellipsize+maxlines for just a sample of the mess)
It looks like later platform versions have made some attempts at improving things, and I certainly have never seen this in action on contemporary OS versions.  If anyone can reliably repro this on a 5.x device, we can reopen and consider whether to roll our own widget or change what we display.


added genshin api explainaiton for most endpoints(will add more next few days)

will add hsr next few days

Could you make this part of the actual docs so that it can also be on GitHub pages

sure, il move it

Could you maybe make some of these collapsible with

Can we consider this PR abandoned?


Please increase the dosage and balance display to facilitate viewing of usage

Is your feature request related to a problem? Please describe:
Please increase the dosage and balance display to facilitate viewing of usage

we've recently shipped this in the settings page on sidebar


"downloads the package from WMIS"

I am troubleshooting this process.  When you refer to the information in my subject, could you include more detail on that - where exactly is WMIS - URL, ports?
event logs of all our devices have messages in the logs saying the devices can't access so I'm assuming we need a Proxy/firewall config change to allow it, but can't find that required information.  error message being - "A connection to the Windows Metadata and Internet Services (WMIS) could not be established."

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

ID: 9d5f648a-f925-7bca-7bad-b97fded54baf
Version Independent ID: e6a51140-c16d-8096-7888-2536b23a3ebc
Content: Installing Device Metadata Packages from WMIS - Windows drivers
Content Source: windows-driver-docs-pr/install/installing-device-metadata-packages-from-wmis.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

@PaulKlerkx. As per my knowledge, WMIS online sever contains OEM certified device drivers for particular drivers

Thanks for the feedback and apologies for the delay. It's possible that group policy might be set to prevent the WMIS acquisition. I'm afraid I don't have any expertise with it though, so if this is still an issue, I'd recommend posting in the following forums, where other driver developers and Microsoft engineers can be found. The community there is quite responsive.

Windows Hardware WDK and Driver Development
OSR's ntdev List

Again, thanks for reaching out. Best of luck with this issue.


Add some building logs for diskann

Signed-off-by: cqy123456<EMAIL_ADDRESS>#253

/lgtm

/lgtm

/lgtm


Correct ts-rank calculation when considering tags in search

This is with regard to the discussion at, https://github.com/opencollective/opencollective-api/pull/7284
It seems that with the aforementioned PR we are calculating the ts-rank incorrectly by changing the were clause thus always resulting in an incorrect sort. The purpose of that change is to support tags such as jhipster-tag where there is punctuation; which is an obvious use case. With the previous version it seems that if the user types jhipster-tag no results appear as array_to_tsvector seems to do some normalization. If they search for jhipstertag results appear but that isn't intuitive for the user.
However we need to make changes to the TS_VECTOR rather than changing the WHERE clause as mismatching these two results in an incorrect sort. Thus I am changing the code to what is suggested here; https://github.com/opencollective/opencollective-api/pull/7284#issuecomment-1074876710

Let's update the indexes too. You can copy-paste migrations/20211123102842-add-search-collectives-index.js and just replace the index content.

Thanks. I've updated the migration as well. 👍🏽

@SudharakaP : Sorry for taking some time one this one, I was stuck with the defining of the function through Sequelize. Anyways, now I've seem to figured it out. Take a look and let me know if you see any issues. The issue here was that we had an immutable function array_to_string within the index creation which is not valid. So I created a wrapper function array_to_string_immutable so that we don't have this issue.
Reference: https://dba.stackexchange.com/questions/299039/optimize-query-matching-first-n-items-of-an-array


Implement a parser for RFC 3339 dates.

Google Cloud Storage returns RFC 3339 dates in the JSON messages.
I could not find a parser for them, the nlophmann::json library
does not have own, curl has a parser that accepts too many other
formats. Protobuf has a parser, but that would introduce a strange
dependency, and (I believe, by code inspection), the parser does
not handle leap seconds correctly.

PTAL.  Also, we run Windows builds here:
https://ci.appveyor.com/project/coryan/google-cloud-cpp

@mbrukman PTAL.


Add codecov after travis success

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@62d5218). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage        ?   83.33%
=======================================
  Files           ?        3
  Lines           ?       30
  Branches        ?        0
=======================================
  Hits            ?       25
  Misses          ?        5
  Partials        ?        0

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 62d5218...625da2f. Read the comment docs.


Make code production ready

Hi @moyix,
Great work on FauxPilot! I'm an engineer at GitLab and am experimenting with code completions tools, I wanted to give FauxPilot a try and noticed a few things that I thought could be improved for my needs and figured I might as well give back 😄
Changes / updates:

Use FastAPI instead of Flask for higher performance
Added Uvicorn to serve a production server instead of a development server
Upgraded to Python3.10
Pinned versions of pip packages
Added Swagger / OpenAPI spec
Made a few more variables settable via environment variables
Separated the API from CodeGen proxy, added input model based on OpenAI spec
Cleaned up code a little bit
Improved the setup script so it can re-use existing converted models

Let me know what you think, if you have any questions etc.

Wow, this is awesome, and I'm so glad to hear FauxPilot has been useful to your work! :) Let me read through these changes but I'm really excited to see this work on making FauxPilot more usable for production deployments!

Sorry, the approval it was an error, I was having a look at the code

Looks great, thanks for the contribution!

Thanks! I'm curious about future plans you have for FauxPilot.


Require image for ARM64 architecture

Hi Team,
I am trying to use “cron-resource” over ARM64v8 but it seems it does not have an arm64 supported tag in docker hub.
I have successfully built the image using the command “docker build -t <image_name> .” and used it successfully on arm64 platform.
Do you have any plans for releasing an ARM64 image?
May I know how the amd64 image is getting published on Dockerhub?
It will be very helpful if the image is released with ARM64 support. To start with can I get suggestions on which CI you are using to push the image to Dockerhub.

We use Concourse with the docker-image resource to build the image. We don't currently have plans to release an ARM64 image, and the team that maintains this image does not do very much active work on it so your best bet may be to fork the image. But if you want to pair on this I would be willing to help you get it done. Out of curiosity, what are you trying to do?

Here is the job config we are using to build the image:
  - name: update-cron-resource-image
    build_logs_to_retain: 25
    serial: true
    plan:
    - get: cron-resource_image
      tags: [gcp]
      params:
        save: true
    - put: archived_cron-resource_image
      tags: [gcp]
      params:
        load: cron-resource_image
    - get: cron-resource-dockerfile
      tags: [gcp]
      trigger: true
    - put: cron-resource_image
      tags: [gcp]
      params:
        build: cron-resource-dockerfile
        dockerfile: cron-resource-dockerfile/Dockerfile


Is recall calculated correctly?

I wanted to pull out the tp, fp,, tn, fn from this evaluator and calculate the recall and precision values myself. During that I've stumbled into the compute_precision_recall(tp, fp, n_positives) function.
def compute_precision_recall(tp, fp, n_positives):
    """ Compute Preision/Recall.

    Arguments:
        tp (np.array): true positives array.
        fp (np.array): false positives.
        n_positives (int): num positives.

    Returns:
        precision (np.array)
        recall (np.array)
    """
    tp = np.cumsum(tp)
    fp = np.cumsum(fp)
    recall = tp / max(float(n_positives), 1)
    precision = tp / np.maximum(tp + fp, np.finfo(np.float64).eps)
    return precision, recall

shouldn't recall look like:
recall = tp / tp + fn ?
or is it that 'num positives' stands for the tp and fn?

Nevermind, it's correct. Closing the issue. Maybe I will make a PR for extracting tp, fp and n_positives as it seems pretty usefull.


DBscan implementation is non-exclusive

Hi,
I tried to use your DBSCAN implementation on lat/lon pairs for geo-clustering and the results frequently came back with location pairs in more than one cluster. Pretty sure that the DBSCAN algo results in exclusive clusters.
Reimplemented the code using python scikit and I got the expected result...
HTH,
-FT

This is possibile. I implemented most simple DBSCAN and test it with very simple dataset (you can see tests in tests\ClusteringDBSCANTest). If you share some code and data perhaps I could write it better. You can send it here or directly to me.

Hi @ftrotter, could you send me (or write here) some examples so i can update DBSCAN?

I will try to once I start working on the analysis again...
But it will likely take some time...
-FT
On Tue, Apr 4, 2017 at 2:21 PM, Arkadiusz Kondas<EMAIL_ADDRESS>wrote:

Hi @ftrotter https://github.com/ftrotter, could you send me (or write
here) some examples so i can update DBSCAN?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/php-ai/php-ml/issues/48#issuecomment-291604988, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAFEvcOIItRW0OIcu1gYJNf0F8Q-hpL_ks5rsphLgaJpZM4L-yJe
.

--
Fred Trotter
Blog: http://radar.oreilly.com/fredt
Twitter: @fredtrotter
CareSet<EMAIL_ADDRESS><EMAIL_ADDRESS>DocGraph<EMAIL_ADDRESS>P.S. I do Open Source healthcare software. Because I frequently give
code/data away for free, I get lots and lots of email, expecting more free
stuff. So much so that I now have trouble keeping up with email. If you are
or plan on paying me, and you are having trouble getting ahold of me,
please contact Ashish Patel. If not, please expect that I might take
between 30 and 90 days to get back to you. I will also probably not read
emails where I am merely cced. Its not personal. I just cannot spend 8
hours everyday emailing. I still love you all. Really.  -FT
P.P.S if you are trying to get in touch with me during a conference, you
will have better luck with my twitter handle: @fredtrotter

Hi @itcraftsmanpl,
Is it correct that duplicates values appear in a result?
For example:
$samples = [[1, 1], [9, 9], [1, 2], [9, 8], [7, 7], [8, 7]];
$d = new DBSCAN($epsilon = 3, $minSamples = 2);
$r = $d->cluster($samples);
print_r($r);

--- output ---

Array
(
    [0] => Array
        (
            [0] => Array
                (
                    [0] => 1
                    [1] => 1
                )

            [1] => Array
                (
                    [0] => 1
                    [1] => 2
                )

        )

    [1] => Array
        (
            [0] => Array
                (
                    [0] => 9
                    [1] => 9
                )

            [1] => Array
                (
                    [0] => 9
                    [1] => 8
                )

            [2] => Array
                (
                    [0] => 7
                    [1] => 7
                )

            [3] => Array
                (
                    [0] => 8
                    [1] => 7
                )

            [4] => Array
                (
                    [0] => 9
                    [1] => 9
                )

            [5] => Array
                (
                    [0] => 9
                    [1] => 8
                )

            [6] => Array
                (
                    [0] => 7
                    [1] => 7
                )

            [7] => Array
                (
                    [0] => 8
                    [1] => 7
                )

            [8] => Array
                (
                    [0] => 9
                    [1] => 9
                )

            [9] => Array
                (
                    [0] => 9
                    [1] => 8
                )

            [10] => Array
                (
                    [0] => 7
                    [1] => 7
                )

            [11] => Array
                (
                    [0] => 8
                    [1] => 7
                )

            [12] => Array
                (
                    [0] => 9
                    [1] => 9
                )

            [13] => Array
                (
                    [0] => 9
                    [1] => 8
                )

            [14] => Array
                (
                    [0] => 7
                    [1] => 7
                )

            [15] => Array
                (
                    [0] => 8
                    [1] => 7
                )
        )
)

It seems like something wrong with preserving keys...

@ftrotter I'm considering using this DBSCAN implementation for identifying clusters in latitude/longitude pairs (as I believe you had tried).
Does the rewrite in #185 now work properly for your use case in feeding lat/long samples directly into $dbscan->cluster() ?


[release-1.19] Bump grpc-go and x/net

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

Test name
Commit
Details
Required
Rerun command

gencheck_istio_release-1.19
652f3a7be917198f2ccf81064b646d2ff3baec34
link
true
/test gencheck

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.

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

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio_release-1.19
ee96374fd58ca8ebf300b2338d8fa74afc777b64
link
true
/test integ-ambient

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


Support for tidal colocation with HPA and node pool management

What would you like to be added?
This issue propose the addition of a new feature that enables time-shared node reuse on Kubernetes. This feature aims to enhance resource utilization and efficiency by allowing multiple types of workloads, such as online services and batch jobs, to share a node in a time-sliced manner. The feature consists of two key capabilities: HPA and node pool management.

HPA Enhancement: Extend the existing HPA functionality to support scaling workloads based on a schedule, i.e. CronHPA. This enhancement will enable workloads to reduce their resource footprint when demand is low, freeing up resources for other workloads.

Node Pool Management: Introduce a node pool management mechanism that dynamically reallocates nodes between different types of workloads. When a workload is scaled down, the vacant nodes will be identified and assigned to another workload that is experiencing higher demand. This will facilitate the efficient utilization of nodes and prevent resource wastage.

Why is this needed?
The need for tidal colocation arises from the desire to optimize resource utilization and enhance cost efficiency within Kubernetes clusters. Currently, workloads often run on dedicated nodes, leading to suboptimal resource usage and potential underutilization during off-peak hours. By implementing time-shared node reuse with HPA and node pool management, several benefits can be realized:

Resource Efficiency: Many workloads experience varying levels of demand throughout the day. By allowing workloads to scale down during low-traffic periods and releasing their nodes to other workloads, we can ensure that resources are used more effectively.
Workload Isolation: With the proposed mechanism, different types of workloads run on different nodes, avoiding interference and resource contention between workloads.
Dynamic Scaling: The time-shared node reuse feature will enable dynamic scaling, allowing clusters to adapt more efficiently to changing workloads without manual intervention.

/assign


[Feature request] Define dimensions of autoLaunch window

I'd like to control the size of the Chrome window that automatically launches when running vite build --watch
I only need a small window to test my extension, but it currently launches as a large window - meaning I have to resize it every time I run the command
Is there already a workaround for this?

Chrome supports the --window-size argument when launching from the CLI.
web-ext, the tool the plugin uses to open a browser and install the extension, supports configuring the CLI arguments passed to the  browser start command via the --args flag.
To configure this within the plugin:
export default defineConfig({
  plugins: [
    webExtension({
      webExtConfig: {
        args: ["--window-size=400x300"],
      },
    }),
  ],
});

This is untested, web-ext's JS docs aren't great, so I'm assuming --args on the CLI maps to a string array called args in the JS config

I see that the webExtConfig option is not documented in the README, I'll add a section for it.

https://github.com/aklinker1/vite-plugin-web-extension/commit/947b5d987f7c6d722eaf86e783cb73889fb5009e

Thanks very much, the below snippet worked in my case (needed to change x to ,)
webExtension({
  manifest: ...,
  webExtConfig: {
    args: ["--window-size=400,300"],
  },
})


v2.0.5 should not be a minor

Our build broke today because of version 2.0.5
One of the changes in this version is the removal of a full directory. This is clearly not minor change as it's not backward compatible.
I'm not proposing to change the version now, but in the future it would be nice to consider backward compatibility and thus stick to semver.
Thanks in advance

@lipsumar, I'm sorry about that. I got into some troubles authenticating my npm account before, and then I ignore dist folder and for some reasons, it got messed up.
Is anything good now?

Well I’m now stuck on v2.0.0
How about a v2.0.6 that adds back the “dist” directory?

@lipsumar done. v2.0.6 is out.


Edit codeowners

remove Franck and add Nick to solidity contract codeowners

@sparrowDom can you add @rafaelugolini as a reviewer of the /dapp folder.
We can add Rafa to the /dapp-oeth folder in a later PR as the OETH app hasn't been merged to master yet.

LGTM :)


LESS: @grayLight is undefined?

I'm getting an odd error, but the compile still actually goes through. Can't figure out why it's not defined, because it is defined in variables.less.
Anyone else run into this?

If you're saving from a file other than bootstrap.less, you'll always get undefined errors in the Mac app. Try saving from the main file and see if you have the same problem.

Assuming this is resolved now.


Azure CLI Example missing content

Hi
The Azure CLi example is missing some content in the following location:
https://docs.microsoft.com/en-us/cli/azure/role/definition?view=azure-cli-latest#examples
When run through CloudShell, the example fails as it is missing '' under actions and should look like the below:
"Actions": [
"Microsoft.Compute//read",
"Microsoft.Compute/virtualMachines/start/action",
"Microsoft.Compute/virtualMachines/restart/action",
"Microsoft.Network//read",
"Microsoft.Storage//read",
"Microsoft.Authorization//read",
"Microsoft.Resources/subscriptions/resourceGroups/read",
"Microsoft.Resources/subscriptions/resourceGroups/resources/read",
"Microsoft.Insights/alertRules/",
"Microsoft.Support/*" \
Thanks
Chad

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

ID: 40e649cf-40e3-1b9e-141a-c62d244eda61
Version Independent ID: 78ea89c1-7c56-e33b-d10a-dae05642aa34
Content: az role definition
Content Source: src/command_modules/azure-cli-role/azure/cli/command_modules/role/_help.py
Service: azure-resource-manager
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

The contents are displayed correctly on Chrome and can be executed in Cloud Shell without any error. Could you share a screenshot of what is shown on your browser?

add to S172


Getting all calendars of organisation users

Hi,
I'm trying to figure out if as an organisation admin I can get all my users calendar and meeting.
Is it possible? Could not find any documentation for it.
Thanks!

Hi @shilomagen, you would want to use a service account with domain-wide delegation. There is a bit about that in the Calendar API documentation: https://developers.google.com/calendar/concepts/domain#accessing_domain_calendars_as_an_app

Thank you @busunkim96 !


Simplify the definition of deprecation prefixes/suffixes

Signed-off-by: Kevin Petit<EMAIL_ADDRESS>

 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.

@jrprice Thanks for the review.


Feature: using "paths" config in boatsrc to define absolute paths

Re-created from https://github.com/liffery-com/boats/pull/51
Please ensure your pull request:

[x] Include additions to the docs, found in the docs/readme file, as required
[x] The new code you have written is unit tested
[x] Existing code you have modified is also unit tested or existing tests updated
[x] You have respected the current syntax style of this project
[x] The body of the text for this PR includes enough text for a reviewer to understand the context of the change

@TomFreeman  looks great! Thanks for the massive commitment :+1:

@TomFreeman  as this is a pretty big change I would like to just play it safe and run this from develop on a few projects for a few days:
{
  "name": "ms_someapi_d",
  "version": "1.0.0",
  "description": "",
  "scripts": {
    "build:json": "boats -i ./src/index.yml -o ./build/${npm_package_name}.json",
    "build:yaml": "boats -i ./src/index.yml -o ./build/${npm_package_name}.yml",
    "build": "npm run build:json && npm run build:yaml"
  },
  "private": true,
  "dependencies": {},
  "devDependencies": {
    "boats": "liffery-com/boats#develop"
  }
}

FYI:  @p-mcgowan and @zewish

Release to master PR:  https://github.com/liffery-com/boats/pull/55


import of data extremely slow

Note: for support questions, please join our Discord server

I'm submitting a ...
[x ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Action taken (what you did)
Importing data which these characteristics:
Market: bitfinex:USD/BTC

From:
2017-05-09 10:45:00
To:
2017-11-09 11:45:00

Expected result (what you hoped would happen)
To import the data.

Actual result (unexpected outcome)
I waited 2 hours and only got data for 3 hours up untill now. How can I speed things up?

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)

Is it happening with other exchanges?

This is a limitation of Bitfinex's API. They rate limit queries to about 1 query every 2.5 seconds and only return a max of 500 trades per query. Bitfinex also has immense volume compared to other exchanges when it comes to BTC and ETH. So yes, it is very normal that it takes several hours to import that amount of data you are requesting, and unfortunately there is not much we can do about it.