ida-free: remove icon due to hash mismatch

The "icon_free.png" that this derivative tries to pull for the shortcut icon keeps changing, causing different SHA values.
error: hash mismatch in fixed-output derivation '/nix/store/XXX-icon_free.png.drv':
         specified: sha256-widkv2VGh+eOauUK/6Sz/e2auCNFAsc8n9z0fdrSnW0=
            got:    sha256-oK++vjT4W+XsutWfR0hJeTxqoy4cdEqHBKEdZX2ylUI=

error: hash mismatch in fixed-output derivation '/nix/store/0pk9h8vvqxr4lcv6whnsgncdfdq6bjsp-icon_free.png.drv':
         specified: sha256-widkv2VGh+eOauUK/6Sz/e2auCNFAsc8n9z0fdrSnW0=
            got:    sha256-2PTGTOqYPyufIoosf3jmbJRrlpDJj+O1f593ZtN3spc=

I removed the fetching of the icon temporarily, but perhaps a better source can be used. Might be related to caching?
(apologies if I did not do this correctly, I did read through the contributing guidelines but might be lost a bit)

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Hi,
Thank you for your contribution.
About the contribution guidelines:

PRs should target master branch, so please re base against it and change the target branch of the PR. https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#branch-conventions
Commit messages should follow a structure https://github.com/NixOS/nixpkgs/blob/master/pkgs/README.md#commit-conventions , so in your case something like ida-free: remove icon should fit the guidelines (tho I would add into the commit body why it's removed so that people can understand without opening a PR)

We sometimes use the internet archive or related services to pull a snapshot of things that keep changing, that might be a good alternative.

This is caused by the web archive being down, and Hex-Rays updating the icon on their site due to the IDA 9 release. I'm against removing the icon, but would favor if we removed the web archive mirror as long as it's down (due to the attack on them) for the time being, until we can mirror the new icon again.

This is caused by the web archive being down, and Hex-Rays updating the icon on their site due to the IDA 9 release. I'm against removing the icon, but would favor if we removed the web archive mirror as long as it's down (due to the attack on them) for the time being, until we can mirror the new icon again.

Ah makes sense. Yes I don't like it either. I was contemplating using another website but I don't want to without making sure I'm not breaking any copyright / permission issues.

Also, please remove the merge commit and rebase c8b5e9352ef81a2f65c57961b4c503aa9eea2ddf onto master instead.

My bad. Shouldn't have tried to sneak around rebasing - would be good if github allowed rebasing on the website.
Will do so when back home!

We sometimes use the internet archive or related services to pull a snapshot of things that keep changing, that might be a good alternative.
I see actually that there's an archive service url inside already. What if only that one is kept?

I believe right now the first url retrieves the 404 page (which is causing the different SHA256), while the second one (archive) times out.

due to the attack on them) for the time being, until we can mirror the new icon again.

hm I just realised something. The installation packages are also sourcing from archive. I just tried using this patch and it does error out as expected:
curl: (28) Failed to connect to web.archive.org port 443 after 135261 ms: Could not connect to server
error: cannot download idafree84_linux.run from any mirror

Superseded by https://github.com/NixOS/nixpkgs/pull/349440


Translate your first component

Hello, I'd simply like to ensure my pull requests aren't overlooked. At least some kind of feedback would be nice ;)

Hi Dlurak,
your PR isnt overlooked, and thank you for sending it in. Ill try to take care of it as soon as I have enough time - earliest this weekend


from .nms.cpu_nms import cpu_nms, cpu_soft_nms ModuleNotFoundError: No module named 'utils.nms.cpu_nms'

from .nms.cpu_nms import cpu_nms, cpu_soft_nms ModuleNotFoundError: No module named 'utils.nms.cpu_nms'
I've run  make.sh, but still get this error

from .nms.cpu_nms import cpu_nms, cpu_soft_nms ModuleNotFoundError: No module named 'utils.nms.cpu_nms' I've run make.sh, but still get this error

Hi, this is a module taken from faster rcnn. You may search this error on google for a solution. It works fine for Ubuntu, but may cause problem under Windows.

Yes, it is indeed the environmental problem, I have solved it, your work is very good, thank you for your reply!
------------------ 原始邮件 ------------------
发件人:                                                                                "jhb86253817/PIPNet"                                                                                @.>;
发送时间: 2022年8月18日(星期四) 晚上11:39
@.>;
<EMAIL_ADDRESS>主题: Re: [jhb86253817/PIPNet] from .nms.cpu_nms import cpu_nms, cpu_soft_nms ModuleNotFoundError: No module named 'utils.nms.cpu_nms' (Issue #31)
from .nms.cpu_nms import cpu_nms, cpu_soft_nms ModuleNotFoundError: No module named 'utils.nms.cpu_nms' I've run make.sh, but still get this error
Hi, this is a module taken from faster rcnn. You may search this error on google for a solution. It works fine for Ubuntu, but may cause problem under Windows.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Fix for intern WeakValueDirectory leak

This fixes a leak in the WeakValueDirectory used in intern.py: the keys in that directory are Wrapper instances that themselves contain a weakref to the value which is used in eq/hash computations. This seems to cause an interference that prevents collection of dead values. I have written a new class, CanonStore, that fixes this issue.
It doesn't seem to change the test suite performance much if at all. I tried to find ways to reduce the overhead of intern besides that, but nothing really worked, so that's that.
Oh, I also added a --bw option for debugging with breakword more easily.

We still need to make sure jenkins runs on this.

@abergeron Jenkins and the rest of the tests succeeded.


Don't recreate LLMPipeline

Here's oneliner for windows cmd to reproduce by running multiple times cmd /c "for /l %i in (1, 1, 9999) do @echo starting %i && pytest tests\python_tests\test_generate_api.py::test_decoding --sw || exit %errorolevel%"


Visual issues on Image Resizer screen

ℹ Computer information

PowerToys version: 0.21 and 0.22
PowerToy Utility:
Running PowerToys as Admin: no
Windows build number: 19041.508

📷 Screenshots

Thanks! We are tracking this issue (and a potential solution) in #2813.


Update Sphinx config to use the normal ReadTheDocs theme

Title says it all - this should be changed.

Fixed by 79b3265580f37aa62d4fd07c393721bb1d6899e6


Dropped dependency with lm-sensors

lm-sensors is an old package and doesn't work anymore on my side. Replaced by the native way to get the CPU temperature on Linux.

Thanks for this PR , we'll need several weeks to look at it and review since we haven't been using zabbix for a while. We're definitely going to reinstall though soon.

Thank you for your PR, sadly it would remove the ability to see the minimal and maximal cpu temperature as far as I can see that.
I looked into other possibilities to find the CPU temperature and found hwmon.
However, I could not really find out whether hwmon is part of lm-sensors. If it is not, we could use
cat /sys/class/hwmon/hwmon*/temp*_input and cat /sys/class/hwmon/hwmon*/temp*_label

Why not just use both the original lm-sensors, and the CPUmaxtemp method listed in the PR ? Users can just choose which to plot.


swallow unfetch errors

unfetch errors may happen as a result of a "refetch" when reconnecting to a lost jet daemon.

Coverage increased (+0.003%) to 99.281% when pulling a6d0681214e9b59d4abfbfb3d366f8946013b2f1 on small-bugfix into 441dc6d1e86d0e136731ef4ca735276a2b7bef35 on master.


<EMAIL_ADDRESS>

Coverage remained the same at 63.711% when pulling 84a4f84795bbbb174e63bbe9a6f94f6ae17bf8c5 on<EMAIL_ADDRESS>into 37d9c6c06370677e5a561083876b21948a7373bd on master.

Coverage remained the same at 63.711% when pulling 84a4f84795bbbb174e63bbe9a6f94f6ae17bf8c5 on<EMAIL_ADDRESS>into 37d9c6c06370677e5a561083876b21948a7373bd on master.


jsdom does not support Node v6

See: https://github.com/jsdom/jsdom/commit/0cba358253fd5530af0685ac48c2535992464d06
Is it possible to pin jsdom to 12.0.0?

12.0.0 doesn't support Node v6 as far as I can tell?

They removed the package.json engine limitation after 12.0.0.
If 12.0.0 does not work, then the question is: can we ping jsdom to 11.12.0?

Versions of jsdom below v12 still work with Node.js v6, but are unsupported

Will check the performance difference of current (13.2.0) with 11.12.0 and if it's similar to what is now, think there's no problem pinning that version.

@kirrg001 Recapping a convo that happened offline. We decided to make HTML format support optional for node versions ^8, so we don't have to worry about the support of v6 which will be dropped in less than 2 months anyway.


Some environments require setting the 3rd argument to header()

FastCGI requires a Status: header from PHP:
http://www.fastcgi.com/docs/faq.html#httpstatus

Evidently providing the 3rd argument "handles" this for us:
http://core.trac.wordpress.org/browser/tags/3.0/wp-includes/functions.php#L1589

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 25 Jul 2010 at 4:01

Fixed in R420 (ConditionalGet.php and Minify.php particularly)

Original comment by<EMAIL_ADDRESS>on 25 Jul 2010 at 5:15

Changed state: Fixed


Use go-ethereum abigen cmd to generate Golang source code

See: https://pkg.go.dev/github.com/ethereum/go-ethereum/cmd/abigen

@mrekucci still needed?


Broken Link to the homepage

@techwithtwin are you still having issues viewing the homepage
The correct url is https://react-slideshow-image.netlify.app/
Can you please let me know where you found this url


Tab Completion Candidate Pager for when candidates don't fit on a single page

NB: This PR includes PR202 (https://github.com/chzyer/readline/pull/202) which is not yet merged. PR202 addressed various redraw issues with wide characters, edge of screen, multiple lines, masking etc. The main feature of the pager builds on these changes and redraw code so, I had to include them in this PR.
Main Features:

If there are too many candidates returned by the completer, completeMode and completeSelectMode did not work properly. Similar to the bash completion pager, list candidates and offer "--More--" on the end of each page. User can select " ", "y" or "Y" to keep listing or "q", "Q", "n", "N" to stop listing. When paging completes, we also exit out of completionMode.
Added aggregate completion when entering completeSelectMode where the candidates dwindle down sharing a larger common prefix. This makes typing a little faster than having to select. It's a more bash-like behaviour.

Other Fixes:

Fix various crashes where candidates are too wide for the width of the screen and causes division by zero.
Fix crash with wide (Asian characters) in completion.
Streamline redrawing as CompleteRefresh was called too often.
Fix crashes around ctrl-a and ctrl-e in select mode when candidates don't fit on a line
Fix prev/next candidates in select mode when candidates don't fit on a line
Fix crash when ctrl-k was pressed in select mode. This caused us to exit completeSelectMode which cleaned up all the data but left us in completeMode such that if CompleteRefresh was called directly, the data was not initialised.
Fix complete and select mode redraw issues when candidates did not fit on one line.
Fix cursor position issues after CompleteRefresh especially if the prompt and buffer also went over 1 line.
Fix redraw issue where exiting completion mode using certain key presses leaves candidates on the screen.

Fixes for Windows:

Use window size for visible height/width instead of buffer size
Adjust for Window's EOL behaviour.

Notes:

Added Height info to different structures as the decision to page or not required height information.
Added OnSizeChange(). Didn't know if I could get rid of the OnWidthChange()? Would be nice to remove the Width stuff and just have Size (width + height info).

I tested this patch on Linux and Window and it seems to work well.

Did some testing with fq master on macOS and did not notice anything strange

If there are too many candidates returned by the completer, completeMode and completeSelectMode did not work properly. Similar to the bash completion pager, list candidates and offer "--More--" on the end of each page. User can select " ", "y" or "Y" to keep listing or "q", "Q", "n", "N" to stop listing. When paging completes, we also exit out of completionMode.

Seems to work fine

Added aggregate completion when entering completeSelectMode where the candidates dwindle down sharing a larger common prefix. This makes typing a little faster than having to select. It's a more bash-like behaviour.

Not sure i understand how this works. Can you give an example somehow?

Thanks for having a look at this @wader .

Added aggregate completion when entering completeSelectMode where the candidates dwindle down sharing a larger common prefix. This makes typing a little faster than having to select. It's a more bash-like behaviour.

Not sure i understand how this works. Can you give an example somehow?

Sure. Let's say you are completing file names and you press tab, the completer returns:
afile
anotherfile
filename.1234
filename.1456
Old Behaviour:

We are in complete mode and 4 files/candidates above are displayed.
press f

Screen displays:   prompt> f
completer now returns filename.1234 and filename.1456 and they are displayed beneath.

press TAB

Screen displays: prompt> f
And enters selectmode where you can select between the 2 files.
if  there were more choices and you didn't want to select them, but rather just type to complete, you'd end up typing everything unless you exit complete mode and then tab into it again.

New Behaviour:

We are in complete mode and 4 files/candidates above are displayed.
press f

Screen displays:   prompt> f
completer now returns filename.1234 and filename.1456 and they are displayed beneath.

press TAB

Screen displays: prompt> filename.1  (the common prefix of both is completed) and bell is rung to indicate partial completion.
Then you either press tab again and/or keep typing to finish.

Sure. Let's say you are completing file names and you press tab, the completer returns:
...

Aha thanks for detailed explanation. Was a bit hard to find a good test case with fq but now i see and it works and feel natural i think, nice improvement 👍

Cool. Thanks for checking @wader . Hope you enjoy the patch :-)

@chzyer - let me know if you have any questions on this one either. Hope you find some time to review. I think this will also fix off some of the other open issues/prs.

Noticed a draw issue while testing. Seem to also happen with only #202 but does not happen with readline chzyer master.
Reproduction:
# run fq repl
$ go run fq.go -i
# press "r" then tab to see completions, now press enter to eval just "r"
# options are not cleared before retuning from readline, error message and prompt
# ends up mixed with completion options
null> r
error: expr: function not defined: r/0  remainder repeat    repl
null> e   rindex    rint      root      round     rpad      rtmp
rtrimstr

With readline master the options are cleared before returning.

Hi @wader - awesome. Thanks for the feedback. I have replicated it locally also. I will fix this either this weekend or Monday as time allows.

@tpodowd Great 👍 no stress

Hi @wader - I have added a commit that fixes this issue. Please have a look and let me know if it works for you. I was able to replicate the issue locally by adding some dummy completion candidates to my code and this fixed it so I believe it will work for you also.

@tpodowd Works great 👍

Thanks for the update @wader . Let me know if you notice anything else.  Enjoy.

Yeap will do!

@tpodowd Hey again, hope everything is good. I started to experiment with multiline input in fq but notice that one issue is that having \n in a readline string works a bit weird, once you start typing the prompt will start to feed way. Any idea if it would be possible to support?
Another issue is that history files at moment uses \n are separator.. but that probably "just" need some history API changes to workaround.

To clarify the use-case for me would be to support things like writing starting quote " and the past text that include "real" new lines (and there is code to understand that there is a unterminated string) and then write an ending ".

Hi @wader - sorry I've been traveling. Finally back. I don't actually use the multiline input in my use case. Not sure exactly what you mean by the prompt will start to feed away.  Was this an issue before the changes also, or only after this PR? Probably possible to fix things I would imagine. Just need a small example and some keys to type to help me replicate and understand. Is it possible to replicate the issue with any of the examples, perhaps example/readline-multiline?

Nice and hope the travels were good! No worries and maybe I should have been more clear this is probably more of a feature request than a bug, and I don't think your changes broke anything.
Think i might have a reproduction using the multiline example. The idea in fq is to have something similar to how shell readline can do multiline by opening a quote " or ' and press enter and do multiple lines and close it.
diff --git a/example/readline-multiline/readline-multiline.go b/example/readline-multiline/readline-multiline.go
index 2192cf6..090753f 100644
--- a/example/readline-multiline/readline-multiline.go
+++ b/example/readline-multiline/readline-multiline.go
@@ -32,7 +32,7 @@ func main() {
                        rl.SetPrompt(">>> ")
                        continue
                }
-               cmd := strings.Join(cmds, " ")
+               cmd := strings.Join(cmds, "\n")
                cmds = cmds[:0]
                rl.SetPrompt("> ")
                rl.SaveHistory(cmd)

And then run:
$ go run example/readline-multiline/readline-multiline.go
> line1
>>> line2
>>> line3;
line1
line2
line3;
# press arrow up and you get multiple lines as input
# press arrow left to try to go back into the multiline input
# the lines start to scroll up

Hi @wader - I see. I'm currently working on other projects not related to readline which will keep me busy for a while. Since this is unrelated to this PR, perhaps create an Issue for it instead. Maybe someone will get to work on it before I do. I wouldn't mind the ability to split input across multiple lines using " or ' similar to how bash readline works, but my project that uses readline is currently stable so have moved off it for a bit. If a bug pops up I'll have a look at that though. Feel free to reach out over mail if you want to run anything by me in the meantime.

No worries! and I agree, let's create a new issue for it. Will probably do it later today or so, should probably also include some things about that it will affect how history works.

Filed an issue https://github.com/chzyer/readline/issues/212

This PR has been rebased on top of PR202 again (PR202 was rebased on top of the latest master and one other commit was added to fix a race condition it introduced). So, as of now, d9af567 and 586d8ee are from PR202 and the 3 other commits are unique to this PR.
The first 2 commits e4b415f and 4fda9f0 are exactly the same as before. I added one new commit which fixes a race condition when the terminal is resized. Note that this race condition also exists in master.  The commit message explains it. I'll paste it below also:

Fix terminal resize race conditions
The runebuf, search and complete data structures all maintain their
own cache of the terminal width and height. When the terminal is
resized, a signal is sent to a go routine which calls a function
that sets the new width and height to each data structure instance.
However, this races with the main thread reading the sizes.
Instead of introducing more locks, it makes sense that the terminal
itself caches it's width and height and the other structures just
get it as necessary. This removes all the racing.
As part of this change, search, complete and runebuf constructor
changes to no longer require the initial sizes. Also each structure
needs a reference to the Terminal so they can get the width/height.
As the io.Writer parameter is actually the terminal anyway, the
simplest option was just to change the type from the io.Writer to
Terminal. I don't believe that anyone would be calling these
functions directly so the signature changes should be ok. I also
removed the no longer used OnWidthChange() and OnSizeChange()
functions from these three structures.

Hey, thanks for keeping this one rebased. Will probably do a new fq release soonish and plan on rebasing my changes on top on this one. Plan on rebasing it on #202?
Really wish we could end up with some readline fork with common changes somehow 🤔 a bit of a mess to keep track 😄

Hi @wader - good to hear from you. I added another patch to #202 a few minutes ago, so yes, I will also rebase this one again. Before I rebase, I'll probably do a little more testing. I'm hoping I didn't break anything on windows :-) so I want to test that first. I'll ping you also when its done. Would be great to get you to test it again also.

The same and hope all is good with you too. I'm keeping a close eye on this repo :) Great, will rebase and test once you ping me. And i should also get myself a windows VM somehow, i've previously used microsofts IE testing vm images but now i can't find them anymore.

BTW I had this setup for a while https://github.com/wader/fq/blob/master/doc/dev.md#setup-docker-desktop-with-golang-windows-container when iterating on shared linux/mac/windows code, but i think the windows cli behaved weirdly via docker so maybe not ideal for readline development.

Hi @wader - I rebased this branch on top of #202 again. I have tested it on linux and windows. Seems to work ok. I'll be doing more testing. Let me know if you find any issues.

Hey did some basic testing with fq on windows using the new "cmd.exe", seems to work fine 👍


Whole my project i was working on is gone..

Describe the bug
Whole my project i was working on is gone..  do you have a copy or what.. i cant find it bolt..
it have something.. but all time and effort i have done is gone... along with all the token...
i need you guys to look into it and find a back of the code and design.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-93g1qk
Steps to reproduce
Whole my project i was working on is gone..  do you have a copy or what.. i cant find it bolt..
it have something.. but all time and effort i have done is gone... along with all the token...
i need you guys to look into it and find a back of the code and design.
Expected behavior
Whole my project i was working on is gone..  do you have a copy or what.. i cant find it bolt..
it have something.. but all time and effort i have done is gone... along with all the token...
i need you guys to look into it and find a back of the code and design.
Screen Recording / Screenshot
Whole my project i was working on is gone..  do you have a copy or what.. i cant find it bolt..
it have something.. but all time and effort i have done is gone... along with all the token...
i need you guys to look into it and find a back of the code and design.
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 129
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 49375160,
  "usedJSHeapSize": 47818680,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = sheikhk
Chat ID = cfbca331548b

Additional context
Whole my project i was working on is gone..  do you have a copy or what.. i cant find it bolt..
it have something.. but all time and effort i have done is gone... along with all the token...
i need you guys to look into it and find a back of the code and design.

@SheikhCash info on how to open old chats can be found here along with our progress on implementing chat persistence in #39


Allow importing a livebook via file upload #645

Closes #645
I know that the issue mentioned Support live file upload by allowing a file to be dragged and dropped anywhere in the Import modal meaning that the whole modal should be a drop zone but I think that in the current form it is close enough :)
Here is a little demonstration.
https://user-images.githubusercontent.com/34857220/139603921-3fab3d6f-f372-4030-9881-236522975e3e.mov

Thank you @Qizot, this looks great! One of the reasons I was thinking about making the whole import modal drag and droppable (or at least most of it) is so in the future people don't have to click the tab, they can just drag and drop.
Unless you think supporting this functionality like that would not be possible.

@josevalim I wonder about the discoverability of that though. I think for local usage people will default to navigating the file system and forking a file, and then in cloud they are likely to look for an explicit way of opening a file upload, in which case a clear option makes sense. wdyt?

Hey @Qizot, looks fantastic! Let's decide whether we want the tab or not first, but other than that just a few tiny comments :)

@jonatanklosko oh, to be clear, we will always have the tab. It is just the drag and drop always work too. Does that make more sense?

@josevalim I see! Technically we should be able to handle the upload on modal level and patch to the upload tab, but then the upload form would be in a different component and triggering the actual import may be tricky. That said, since we show the tab, I think it's totally fine to handle it only in that tab only.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


suggestion: round robin for input into resourceful furnace

might be good if resourceful furnace uses round robin to input items coming in to utilize the 3 items at a time cooking?
this, vs just filling first slot, then second, then third.

Just enchant the addon and it will insert it so fast that you wont need it

enchant addon?  What?
No clue what you're talking about there, but the issue isn't that it's moving items in fast enough, it's that if you don't have enough items coming in to make a complete stack, it's still not cooking more than one item at a time, but I guess that issue could go both ways so you're way is probably better.

Came here to ask this also. I am trying to automate the resourceful furnace but it only accepts items in the first(of the three) slot so it doesn't smelt three items at a time unless you input manually.


Add parameter for skipping backups only

Today on the EP project we have a need to skip backup tasks because the postgresql DB in use is not provided by the Heroku addon.
jumpup-heroku allow us to skip ALL dbs tasks but we need a fine grained control over what is going to be skipped as we still want to run migrations during a deploy

Hi @fgrehm, #70 which enable the SKIP_BACKUP option was merged and will be available in the next version.

:thumbsup:


Function autocorr.integrated_time() missing "window" keyword argument

It looks like when the autocorr.integrated_time() function was updated recently, the "window" keyword was removed. However, this function is still called with "window=50" as an argument in both ensemble.py and mh.py.
The keyword error was triggered for me using the example script quickstart.py.

This should be fixed in v2.2.1 that was just pushed. Sorry about that!


ADFS SAMLResponse

I'm trying to implement a SP using ADFS as my IDP and I successfully receive a SAML Response via request.form['SAMLResponse']. I can take this response and decode to a string with str(decode_saml_xml(request.form['SAMLResponse']).decode()), but I'm not sure what to do with it next. I'm trying to get to the Name ID via assertion data. What is the recommend way to complete this task? Are all necessary modules built in? The parser.py in sp seemed promising, but none of the functions work for me - I get list index out of range. I'm not sure if I'm going about it the wrong way, or passing the wrong data to the ResponseParser class. I thought the decoded XML string would be good, along with my SPs private key, but maybe that isn't the case. Any guidance you can provide would be fantastic.

(I know this is probably a bit late, but for future reference)
The SAMLResponse is already parsed as far as I know. On this line it’s passed to the auth_data.
You can access it through auth_data = sp.get_auth_data_in_session() with the nameid being in auth_data.nameid. Other attributes passed from the IdP are in auth_data.attributes.
Hope I correctly understood this question and that it helps other people in the future :)


Error/alert "Show Details" not working

https://github.com/BitShares/bitshares/issues/1333

https://github.com/BitShares/web_wallet/commit/e56a0b86388b4704cdc7f13189c7058c789d7319

Apparently still broken: https://github.com/BitShares/bitshares/issues/1364

this should be fixed by this commit https://github.com/BitShares/web_wallet/commit/e56a0b86388b4704cdc7f13189c7058c789d7319
but it didn't get into released version


JavaScript runtime error: Unable to get property 'shadowRoot' of undefined or null reference

When I press the Key - ESC than I always get the error obove. Is it possible, to do one more hook to validate shadowRoot? that would be perfect.
Thank you very much

2fc41e7045c0aa424f2da406a8eff40c9183874e fixes this.
Thanks for the report.

thank you for the fix 😎


After moving briefing book to subdir of my contents error while saving changes

I created a briefing book. By default it is saved to my contents on our argis portal. I moved the file to a subdir of my contents. It opened fine but when I tried to save changes I got an error. When I moved the file back to the my contents it works as expected.

@mhovens
This issue should be resolved in the up and coming release.
The issue can be tracked here: #175


[BUG] CDS-types version 8.0.3 different behaviour on standard events ?

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Hi,
In the newly version of the cds-types we have 7 overloads on the AFTER, in the previous version of the CDS 7 we had only 4 overloads.

LEFT IMAGE - CDS version 8 RIGHT IMAGE - CDS version 7
During testing a sample project of the standard events, I was not able to find a suitable applied overload for catching the events + the typing looks wrong if we take in consideration also using the CDS-Typer

LEFT IMAGE - CDS version 7 RIGHT IMAGE - CDS version 8
I was not able also to trigger the EACH on every Book analogue to the Array.prototype.forEach, practically it doesn't stop.
Expected Behavior

Typing should follow the Entity or should be corrected based on the CDS-Typer entity

References
CDS Version 8.0.3
Versions
This version all overloads are working as expected

@dxfrontier/cds-ts-dis
git+https://github.com/dxfrontier/cds-ts-dispatcher.git

@cap-js/asyncapi
1.0.1

@cap-js/cds-typer
0.23.0

@cap-js/openapi
1.0.4

@cap-js/sqlite
1.7.3

@sap/cds
7.9.3

@sap/cds-compiler
4.9.6

@sap/cds-dk (global)
8.0.2

@sap/cds-fiori
1.2.7

@sap/cds-foss
5.0.1

@sap/cds-mtxs
2.0.2

@sap/eslint-plugin-cds
3.0.4

Node.js
v22.4.1

This version somehow the overloads are acting awkard

cds8test

@cap-js/asyncapi
1.0.1

@cap-js/cds-typer
0.23.0

@cap-js/cds-types
0.6.0

@cap-js/openapi
1.0.4

@cap-js/sqlite
1.7.3

@sap/cds
8.0.3

@sap/cds-compiler
5.0.6

@sap/cds-dk (global)
8.0.2

@sap/cds-fiori
1.2.7

@sap/cds-foss
5.0.1

@sap/cds-mtxs
2.0.2

@sap/eslint-plugin-cds
3.0.4

Node.js
v22.4.1

Anything else? Logs?
No response

Change from data: Book to data: Books seems to be intentional: https://github.com/cap-js/cds-types/pull/45

Correct, this is option to have the typing specified inside of the callback
But somehow AFTER does not work like before, for EACH and also for normal READ.

Hi Daniel
Thanks for taking time to have a look over the issue.
import { Book, Books } from '#cds-models/CatalogService'
import cds from '@sap/cds'

class BooksService extends cds.ApplicationService {
  init() {
    this.after('READ', Books, (data, req) => {
      // data is of type Book and not Books
      debugger
    })

    // This is not triggered anymore like analogue forEach
    this.after('EACH', Books, (data, req) => {
      debugger
    })

    return super.init()
  }
}

module.exports = BooksService

I can create a very small repo example if you are not sure about code above.

I created PR #151 and a sample project with runtime tests. The PR should fix the current type inference apart from .drafts. If the handler registration should work for drafts the same way then the singular entity should get drafts: typeof Singular and the plural entity drafts: typeof Plural.

Hi @daogrady,
Is above PR all right, above issue are showstopper for TS projects as AFTER is a core event.

?

We would also be pretty happy to see progress on this topic because we also use the cds-ts-dispatcher package and this is blocking us from updating to CDS8. If we can support testing PR or some fixes just let us know, happy to help.
Here is the issue in cds-ts-dispatcher that documents the progress on "supporting" CDS 8: https://github.com/dxfrontier/cds-ts-dispatcher/issues/85
FYI @mvk-abs

Hi @stockbal @dragolea @Mischa1610 @hakimio ,
the PR looks good to me. Sorry for the delay everyone. and thanks for the contribution, Ludwig! Let's get this merged and released asap.
Best,
Daniel


notations lost their outer mrow

From @kohlhase on December 1, 2015 11:56
It seems that notations have lost their outer  element, and as a consequence, they are invalid.
in smglom/source/set.tex we get
      <omdoc:rendering>
        <m:mo cr="fun">{</m:mo>
        <m:mrow>
          <omdoc:render name="arg1" precedence="neginfty"/>
          <m:mo>∈</m:mo>
          <omdoc:render name="arg2" precedence="neginfty"/>
        </m:mrow>
        <m:mo cr="fun">|</m:mo>
        <omdoc:render name="arg3" precedence="neginfty"/>
        <m:mo cr="fun">}</m:mo>
      </omdoc:rendering>

instead of
    <omdoc:rendering>
      <m:mrow>
        <m:mo cr="fun">{</m:mo>
        <m:mrow>
          <omdoc:render name="arg1" precedence="neginfty"/>
          <m:mo>∈</m:mo>
          <omdoc:render name="arg2" precedence="neginfty"/>
        </m:mrow>
        <m:mo cr="fun">|</m:mo>
        <omdoc:render name="arg3" precedence="neginfty"/>
        <m:mo cr="fun">}</m:mo>
     </m:mrow>
  </omdoc:rendering>

as we used to.
I am not sure whether the mrow is necessary in the presentation process @m-iancu could you comment? If it is not, then I can tweak the schema else, we should reinstate it. @dginev, could you also comment?
Copied from original issue: KWARC/sTeX#199

Actually, @m-iancu said it will be quite simple for him to work without an outer mrow. So I will adapt the OMDoc schema I guess.

From @dginev on December 1, 2015 19:59
Doesn't ring any bells of the top of my head, but we should keep in mind Bruce could have introduced some underwater change of behaviour (or Hang could have). I can help debug if it gets hard to figure out if you get really stuck.
One thing I quickly checked is that LaTeXML always generates a wrapping <mrow> in expressions with at least one operator, so this could be an sTeX regression?

Do we still want to have mrow back if the schema has been adapted?

I guess we are OK with this for now.


[WIP] Unpack job security updates

Moves the unpacker job role to an external file

closing due to staleness


Warning | when PSP (Pod Security Policy) applied to Istio

Bug Description
Could you please help me, what would be the changes required in pod_security_policy-istio.yaml as per the latest Kubernetes versions (from v.21+)
When I applied pod_security_policy-istio.yaml for ISTIO pods it give me Warning

Version
Client Version: v1.23.4
Server Version: v1.22.12-gke.1200

Additional Information
No response

PSP is a k8s resource not istio, and it is warned by k8s.  You should upgrade the version accordingly

@hzxuzhonghu Can you please look in pod_security_policy-istio.yaml as it is written Based on https://github.com/istio/istio/issues/6806#issuecomment-406230278

please file a request to upgrade it in that repo.

@hzxuzhonghu is it same Repo https://github.com/istio/istio/issues/6806#issuecomment-406230278

I cannot understand  why https://github.com/GoogleCloudPlatform/fda-mystudies/blob/master/deployment/kubernetes/pod_security_policy-istio.yaml apply with warning has anything todo with istio

@hzxuzhonghu this  is our Repo  https://github.com/GoogleCloudPlatform/fda-mystudies/blob/master/deployment/kubernetes/pod_security_policy-istio.yaml and pod_security_policy-istio.yaml was written based on the https://github.com/istio/istio/issues/6806#issuecomment-406230278 (which is istio Repo) and we are facing Warn when we applied to  pod_security_policy-istio.yaml.

you can just upgrade the version for psp, so what would you want from istio?

i have changed the version and applied from policy/v1beta1 to policy/v1
error: unable to recognize "pod_security_policy-istio.yaml": no matches for kind "PodSecurityPolicy" in version "policy/v1"

https://kubernetes.io/docs/concepts/security/pod-security-policy/

@hzxuzhonghu we have that URL, and we are not expecting URL form you


allow update backupmariadbs finalizers

Creating backup CrinJob failed because the operator did not have the permission to update backupmariadbs finalizers
$ kubectl get backupmariadbs
NAME               COMPLETE   STATUS                   MARIADB   AGE
backup-scheduled   False      Error creating CronJob   mariadb   3s

Error logs in operator:
creating CronJob: cronjobs.batch is forbidden: cannot set blockOwnerDeletion if an ownerReference refers to a resource you can't set finalizers on

The problem occurred on OpenShift 4.11
Added permission to update backupmariadbs/finalizers

Good catch! Thanks for the PR.
I guess that Openshift requires a slightly different set of RBAC permissions maybe?:

https://sdk.operatorframework.io/docs/faqs/#after-deploying-my-operator-why-do-i-see-errors-like-is-forbidden-cannot-set-blockownerdeletion-if-an-ownerreference-refers-to-a-resource-you-cant-set-finalizers-on-

Our e2e tests run in vanilla Kubernetes, so we didn't catch this one. Also, we don't oficially support Openshift yet, but it is in our roadmap:

https://github.com/mmontes11/mariadb-operator/issues/10

Contributions are welcomed!


View company goals

view company goals this resolves issues #349
you can find this on goals.zuri.chat, components centered at bottom

I keep getting weird PR from you, what's the issue?

I keep getting weird PR from you, what's the issue?

I worked on something only to later realise  someone had worked on the same thing. Looking for something else to do.


Arabic translation of Full Stack JavaScript Projects

[ ] Get Set for Basejumps Incomplete
[ ] Build a Voting App Incomplete
[ ] Build a Nightlife Coordination App Incomplete
[ ] Chart the Stock Market Incomplete
[ ] Manage a Book Trading Club Incomplete
[ ] Build a Pinterest Clone

Closed in favor of https://github.com/FreeCodeCamp/TranslationExpansion/issues/32


How are requests with differing Host: header field values mapped to outbound connections?

This is a follow-up to PR #397, in particular https://github.com/runconduit/conduit/pull/397#discussion_r169507498.
In the case where we send the request to the SO_ORIGINAL_DST destination, because we didn't discover the destination using the Destinations service, we're not making anything w.r.t. whether requests with different Host request header field values are sent on the same connection; the original service is making the choice whether or not to do that. We should, at least, document this in the code with a comment.
In the case where HTTP/1.x requests with differing host values are inbound to the proxy from an external source:

What do we currently do?
What should we do? In particular, should we segregate such requests by Host header field value, and have (at least) one connection to the proxied service per Host field value?

In the case where HTTP/1.x requests with different host values are outbound from the proxy to an external source:

What do we currently do?
What should we do? Again, should we segregate such requests by Host header field value, and have (at least) one connection to the external service per Host header field value?

Note in particular that in the outbound case we don't know whether the destination supports HTTP/1.1 or only HTTP/1.0; i.e. we don't know whether it might ignore the Host header. Also, even in the case where the destination claims to support HTTP/1.1, I'm not sure we can rely on this to mean that we can safely send it requests with differing Host header field values on the same connection. We should do what web browsers like Chrome do here.
All of the above is regarding plaintext (http://) connections. In the case of TLS-protected (https://) connections, I already know that at least empirical results from browsers require that the Host header field match the SNI (Server Name Indication) TLS extension. That is, HTTPS rules require us to avoid sending requests with different Host header field values on the same connection. See https://bugs.chromium.org/p/chromium/issues/detail?id=615413#c2.
Unless there's important reasons to do otherwise, I'd prefer to do the same for HTTP as is required for HTTPS; i.e. have distinct connections per Host request field value.

Similarly, we need to decide how to deal with the possibility that we might have requests claiming to be HTTP/1.0 and others claiming to be HTTP/1.1 that might be ending up on the same outbound (to the proxied service, or to an external destination). It seems like we should be segregating these by protocol version.

@seanmonstar @carllerche Do you have thoughts about this? Could you explain how things currently work and whether you think my suggestion above is good or bad and, if bad, why? Also, how much work would it be to change things, if we were to make these changes? Would we need to bump hyper, tokio, and/or tower?

It looks like @seanmonstar would know more about the impl details as it looks like the original PR came from him. It sounds like you want any connection pool to be keyed on the host header and not the socket addr?
As for the last question, I would highly doubt this would relate to Tokio and probably wouldn't touch Hyper either. It might relate to some code in Tower, but that is not on crates.io yet and we are tracking git, so if there are any required fixes, it should be easy to do.

It sounds like you want any connection pool to be keyed on the host header and not the socket addr?

Basically, for HTTP/1 + TLS, we need to ensure we're not mixing (outbound) traffic for different hosts on the same (outbound) connection, and I propose we do the same for non-TLS HTTP/1 traffic. That's partially because I want to minimize differences between non-TLS and TLS traffic, and partially because I"m concerned that not all destinations will correctly support mixing different Hosts on the same connection--especially since the destination might really be HTTP/1.0 even though we're sending HTTP/1.1 requests.

This seems fine to me.

Currently the HTTP/1 proxy part looks for the Host (or the authority of the URL if the request-target is in absolute-form), and uses that as the "key" for our Recognize implementation. After returning that key, the proxy asks the controller for a socket address for that Host, and once received sets up a hyper client that connects solely to that socket address.
The code that creates the hyper client does not check that the outgoing request host matches previous requests sent on the same connection.
To make sure the connections are keyed not only on the socket address but the host as well, we could probably adjust the Protocol part of our recognize key to enum Protocol { Http1(Host), Http2 }.

To make sure the connections are keyed not only on the socket address but the host as well, we could probably adjust the Protocol part of our recognize key to enum Protocol { Http1(Host), Http2 }.

What seems like an appropriate type to use to represent the Host in the Protocol key? Is it a Destination, the Host type in connect.rs, or its' own type in this instance?

The Host probably needs to be some Either<FullyQualifiedAuthority, Authority>. This would probably be needed in both inbound and outbound.

Okay, so I’m beginning to think that the most correct thing to do is just to reuse the Destination in outbound.rs for Host here.
I was going to have an enum that’s either FullyQualifiedAuthority or Authority, but this doesn’t handle the case where there’s no authority part in the request URI which is when we use SO_ORIG_DST to route on. GetOrigDst gives us back a SocketAddr rather than an Authority, which is what's in the External case of Destination.
Therefore, I think it's best to use Destination, and just factor out the logic for extracting Destinations from Requests from the impl of Recognize for Outbound. Does that seem reasonable to you all?

That seems fine, however there is a difference in how Destination is used currently, versus what is needed in this issue. Current Destination uses either FullyQualifiedAuthority, or SocketAddr. The second case doesn't mean the request didn't have a host header, it just means that the authority didn't look like a local service, and so we aren't looking up a new address, just using the original.
Even with a FullyQualifiedAuthority, we ask the controller for a socket address, and if it matches an existing connection we already have, I believe it is reused. So, specifically in HTTP/1, we'd need the key to be Destination plus the Host.

@seanmonstar Okay, that makes sense. Thanks for clearing that up for me, I think I understand what's necessary now.

Here's my understanding of how to determine a Host component for the Protocol key thus far:

if I have a HTTP/1.x request:

if the request URI has an authority:

if we can normalise that authority as a Fully Qualified Authority, the Host key is that FQA
else the Host key component is the authority

else if if the request has a Host: header:

if we can normalise the host header value as a Fully Qualified Authority, the Host key is that FQA
else the Host key component is the authority

else* ???

The question I've arrived at now is, what do we do if the request lacks both an authority in the URI and a Host: header? For routing, we use get_orig_dst in that case. Should the host key just be set to the socket addr in that case?

I wonder if to be on the safe side, an HTTP/1.0 request with absolutely no authority should just always have its own connection. It makes me sad to waste resources like that, but perhaps it's expected?

@seanmonstar that seems like a safe bet for now, at least.

The idea of normalizing to FullyQualifiedAuthorityis to help with TLS; we need to decide which form of the name that we're going to validate the TLS certificate for. It is basically the canonical name we want to use to refer to the host. Right now we FullyQualifiedAuthority::normalize() returns Some(FullyQualifiedAuthority) for DNS names that we should resolve using the Destination service, None for DNS names that we shouldn't try to resolve using the Destination services, and None for IP addresses. However, really it should return something like this:
pub enum NamedAddress {
    DnsName {
        name: FullyQualifiedAuthority,
        use_destination_service: bool,
    }
    // IP addresses.
    Ip(IpAddr),
}

Then we can have something like this:
pub struct NamedDestination {
    addr: ServiceAddress,
    port: Port,
}

Importantly, there's never a reason to have Either<Authority, FullyQualifiedAuthority> because FullyQualifiedAuthority is really just a type that indicates that normalize() has been called; i.e. that we're not dealing with unnormalized names.
Also note that we have to decide what to do in the case that FullyQualifiedAuthority::normalize() actually normalizes the name, e.g. changes name to name.namespace.svc.cluster.local. In that case, the Host header field of the outbound connection also needs to be changed from Host: name to Host: name.namespace.svc.cluster.local.

I also think we don't have to optimize the case where there is no Host header field in the request, at least for now. It's not clear yet if it would be safe to optimize, and almost every modern HTTP library is going to be formatting requests in HTTP/1.1 form or at least HTTP/1.0 form with a Host header field.

In fact, FullyQualifiedAuthority is already defined as FullyQualifiedAuthority(Authority), and Authority handles both DNS names and IP addresses, so we really seem to just need something like this:
pub enum NamedAddress {
    name: FullyQualifiedAuthority,
    use_destination_service: bool,
}

And then use FullyQualifiedAuthority::without_trailing_dot() wherever we need the value of the authority. We could change without_trailing_dot() return Authority too.

@briansmith Thanks for all of the additional information. Do you think all of the changes you're describing here are in scope for a branch to address ensuring that each host has its' own connection, or should some of these changes be done in additional PRs?

It would be good to first do the refactoring such that normalize() always returns a FullyQualifiedAuthority and an indication of whether the Destination service should be used. Then a PR for this issue could be built on top of it.
It would be bad to have a PR that uses Either<FullyQualifiedAuthority, Authority> or similar, since that's just perpetuating the confusion about what FullyQualifiedAuthority is for.

@briansmith okay, that sounds good to me. Should there be a separate issue to track the FullyQualifiedAuthority changes?

@briansmith okay, that sounds good to me. Should there be a separate issue to track the FullyQualifiedAuthority changes?

I don't care if there is a separate issue. A separate PR would be nice, though not strictly required.

@briansmith I've opened PR #476 to make the changes you described here.

A quick update: I think I've made the necessary changes here. Since this is hard to validate in a unit test, I'm talking to the release confidence team about end-to-end testing for this behaviour.

A quick update: I think I've made the necessary changes here. Since this is hard to validate in a unit test, I'm talking to the release confidence team about end-to-end testing for this behaviour.

I am pretty sure we can use (and/or extend) the proxy's own integration test framework to test this. I think the end-to-end tests should be reserved for Kubernetes- or environment- dependent behavior whenever possible.

If wanting to validate in a unit test, I would edit the proxy/tests/support/server.rs to allow the route functions to be able to access the Service, and since a Service is created to handle a single connection, a test can set some flag on the service, and error if it was already set.

I have these changes ready, but writing unit tests for them is currently blocked by a Hyper bug that's causing multiple connections to be opened and then immediately closed. I believe @seanmonstar is looking into that.
Should I go ahead and open a PR for this without the unit tests, and then add the tests in a subsequent PR?

Okay, so while trying to test the branch I wrote to add the Host header to Recognize keys, @seanmonstar and I determined that Hyper's connection pool already implements the behaviour we want --- even though we're using a custom connector, the pool in Hyper is still keying connections off of the Host header. This appears to be the behaviour we want, and I can open a PR to add a unit test to ensure that we continue to have this behaviour.
If we're still interested in merging the code I've written to modify our Recognize impls to also key off the host header, I believe that would ensure we still have this behaviour without relying on Hyper to provide it? This might be valuable if e.g. we were to replace Hyper's default connection pooling with our own connection management later on.

This appears to be the behaviour we want, and I can open a PR to add a unit test to ensure that we continue to have this behaviour.

Yes, please.

If we're still interested in merging the code I've written to modify our Recognize impls to also key off the host header, I believe that would ensure we still have this behaviour without relying on Hyper to provide it? This might be valuable if e.g. we were to replace Hyper's default connection pooling with our own connection management later on.

We don't need to merge the new code as long as we have a good test. If we change how connection pooling works in the future, then the test will catch any breakage, which is one of the main points of having a test. The even more important reason to have a test is to inform all of us how our product actually works. :)

Okay, opened PR #489 for the tests.

I suspect your code may still be needed to handle the case of there being no host header at all, since that just assumes the ORIG_DST as the host.

Ah, that's a good point, thanks @seanmonstar  for pointing that out.

@briansmith

Basically, for HTTP/1 + TLS, we need to ensure we're not mixing (outbound) traffic for different hosts on the same (outbound) connection, and I propose we do the same for non-TLS HTTP/1 traffic.

We're going to have to revisit all of this with logical routing, fwiw.

I"m concerned that not all destinations will correctly support mixing different Hosts on the same connection--especially since the destination might really be HTTP/1.0 even though we're sending HTTP/1.1 requests.

I'm very skeptical that this is a problem we should be concerned about, without a concrete use case.


Broken ingress-nginx after default all-in-one installation

After merging PR https://github.com/Netcracker/KubeMarine/pull/479 default kubemarine configuration is not enough in case of all-in-one scheme without balancer.
For example for cluster.yaml:
node_defaults:
  username: 'root'
  keyfile: '{{ env.NODE_KEY_FILE }}'
nodes:
  - name: node-all-in-one
    address: '<IP_ADDRESS>'
    internal_address: '<IP_ADDRESS>'
    roles:
      - control-plane
      - worker
cluster_name: test-local-k8s.com

Ingresses doesn't work for such configuration and all requests return Connection reset by peer, because of enabled proxy protocol, for example for dashboard:
root@node-all-in-one:~$ curl -v -k https://dashboard.test-local-k8s.com
*   Trying <IP_ADDRESS>:443...
* Connected to dashboard.test-local-k8s.com (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* TLSv1.0 (OUT), TLS header, Certificate Status (22):
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* OpenSSL SSL_connect: Connection reset by peer in connection to dashboard.test-local-k8s.com:443
* Closing connection 0
* TLSv1.0 (OUT), TLS header, Unknown (21):
* TLSv1.3 (OUT), TLS alert, decode error (562):
curl: (35) OpenSSL SSL_connect: Connection reset by peer in connection to dashboard.test-local-k8s.com:443

To solve that issue, it's needed to disable proxy protocol ic cluster.yaml, e.g.:
node_defaults:
  username: 'root'
  keyfile: '{{ env.NODE_KEY_FILE }}'
nodes:
  - name: node-all-in-one
    address: '<IP_ADDRESS>'
    internal_address: '<IP_ADDRESS>'
    roles:
      - control-plane
      - worker
cluster_name: test-local-k8s.com

plugins:
  nginx-ingress-controller:
    config_map:
      use-proxy-protocol: "false"

But in fact, it's not clear from documentation.
Additionally, we deploy such scheme in kubemarine default integration tests, but we don't check there, if ingresses  work. For future, we can add ingresses connectivity checks to check_paas or, (if it's not possible for some reason) as separate check in integration tests.

Closed in PR https://github.com/Netcracker/KubeMarine/pull/509


Add AccountsPendingAsync to RpcClient

I added RpcClient method for action accounts_pending
source: https://docs.nano.org/commands/rpc-protocol/#accounts_pending

I enabled the source parameter and changed AccountsPendingResponse so it uses ReceivableBlock instead, just like the Pending command. Let me know what you think of these changes.

Good idea 👍

When I was working on my project I ran into "Object reference not set to an instance of an object exception" in AccountsPendingAsync method.
Address.Value is null when there is no pending ReceivableBlock transactions, so I added a null check.
https://github.com/RestoreMonarchy/Nano.Net/commit/36bd089c9baca1f73302a5cf6aa7e01fa83cdac8
Do you want me to create another pull request?

Thanks for letting me know, that was an oversight on my end, I apologize. I'll fix it myself, don't worry.


ValueError while running LogAnomaly

Hi, I was trying to run loganomaly model, in the cmd line I simply change the model option to 'loganomaly'. It shows this error:
File "/home/LogADEmpirical/logadempirical/logdeep/dataset/vocab.py", line 46, in find_similar
    if sim > 0.90:
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

Could you please help with this?

Sorry for this problem. I must have made a mistake when commit the code. I plan to refactor the code and give a detail instruction to use it in the next week. By that time, this error will be fixed.
Thanks


fix(material-experimental/button): minor comment changes

Just a few minor places where comments need to be added in

Updated with more thorough jsdocs for the classes


How to save  HashMap<String, Object> ?

@mori-honest breaks relational databases. Cannot have more than one value per-column. Best thing you can do is convert it into JSONObject and use the JSONConverter.


v0.2.2 release

Coverage remained the same at 94.272% when pulling 8bb63c1883c6ccff08e5999ec2e21af7dc3aeb8e on v2.2 into c33e1a3ec4935fc91122cccf97dd8d237dbec0c2 on master.


Fix bug on finding reference resource while deleting

Codecov Report

Merging #542 (6d6e94a) into master (9972c85) will decrease coverage by 0.03%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #542      +/-   ##
==========================================
- Coverage   68.28%   68.25%   -0.04%
==========================================
  Files          96       96
  Lines        5774     5781       +7
  Branches     1002     1003       +1
==========================================
+ Hits         3943     3946       +3
- Misses       1831     1835       +4

Impacted Files
Coverage Δ

src/commands/delete.ts
39.80% <62.50%> (ø)

src/cli-command.ts
67.14% <100.00%> (+0.32%)
:arrow_up:

src/util/check-ref-resource.ts
95.00% <100.00%> (ø)

src/hub/task-page-view.ts
19.64% <0.00%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9972c85...6d6e94a. Read the comment docs.


Bump to vertx 4.x and generate rx-java3 bindings

Is this project still alive, would a PR with this change be merged?

I see that 2.0.0 was tagged depending on vertx 4.4.4 but it does not have the rx-java3 bindings anymore, can you please restore the bindings?


Bitcoin (Coinbase) donation link is broken

The link in the README.md to https://www.coinbase.com/groteworld gives a 404.

@groteworld is something up? Perhaps we should run everything though PayPal until I set up something new?


Syntax error in the POLAR_ALLOWED_HOSTS  var of the .env.template file.

There is a minor syntax error in the POLAR_ALLOWED_HOSTS  var of the .env.template file.
Current Behaviour
POLAR_ALLOWED_HOSTS=["localhost:3000", "<IP_ADDRESS>:3000"]
Ideal Behaviour
POLAR_ALLOWED_HOSTS='["localhost:3000", "<IP_ADDRESS>:3000"]'
Screenshot

Good catch! Feel free to open a PR about it 👍

Good catch! Feel free to open a PR about it 👍

Thanks! Opened the PR.


Issue on fetching data using  getDocuments - "Missing or insufficient permissions."

Overview of the Issue
Getting Error Code 7 - Missing Permissions for Get request.
Motivation for or Use Case
Same query using ListenerRegistration is working
Browsers and Operating System
iPhone - iOS 12,
FirebaseCore 6.0.4
Reproduce the Error
import Firebase

class FireStore {
  var db: Firestore!
}

class DebtsInfoFireStore: FireStore {

  class func shared() -> DebtsInfoFireStore {
    return sharedManager
  }

  private let storeName = "DebtsInfo"

  private static let sharedManager: DebtsInfoFireStore = {
    let manager = DebtsInfoFireStore()
    let settings = FirestoreSettings()
    Firestore.firestore().settings = settings
    manager.db = Firestore.firestore()
    return manager
  }()

  public func store(debts: Parameters) {
    let collection = Firestore.firestore().collection(storeName)
    guard let user = Auth.auth().currentUser else { return }
    var userData = debts
    userData["userID"] = user.uid
    collection.addDocument(data: userData) { (error) in
      print("Error \(String(describing: error))")
    }
  }

  public func fetchAllDebts(completion: @escaping (_ result: [DebtsInfo]?) -> ()) {
    db.collection(storeName).getDocuments { (snapshot, error) in
      guard let snapshot = snapshot else {
        print("Error fetching snapshot results: \(error!)")
        return
      }
      let models = snapshot.documents.map { (document) -> DebtsInfo? in
        if let model = DebtsInfo(dictionary: document.data()) {
          return model
        } else {
          print("Unable to initialize type \(DebtsInfo.self) with dictionary \(document.data())")
          return nil
        }
      }
      completion(models as? [DebtsInfo])
    }
  }

}

Database Rule
service cloud.firestore {
  match /databases/{database}/documents {

    match /DebtsInfo/{debtsInfo} {
      allow read: if request.auth != null
        						&& request.auth.uid == request.resource.data.userID;
      allow create: if request.auth != null;
      allow write: if request.auth != null
                     && request.auth.uid == request.resource.data.userID;
    }
 }
}

Sample Code - Similar to above
Code
Issue
Error Domain=FIRFirestoreErrorDomain Code=7 "Missing or insufficient permissions."
Fire Store Screenshot

Please correct me if I'm wrong--it looks like you're querying the entire collection and expecting security rules to exclude all the documents that cannot be read. This is explicitly not a feature of security rules; you should be using a filtered query instead. Something like
db.collection(storeName).filter(whereField: "userID", isEqualTo: user.userID)

@morganchen12 : Yes as per Security Rules Link you have provided it states - Security Rules are not filters. But in my security concern is simple only the users who write the data should be able to read it.
If I change the rule -
match /DebtsInfo/{debtsInfo} {
      allow read: if request.auth != null;
      allow create: if request.auth != null;
      allow write: if request.auth != null
                     && request.auth.uid == request.resource.data.userID;
}

The iOS fetching code works but as you have  suggested that would need to filter out other users data on client side.
For which the code is like
 db.collection(storeName).whereField("userID", isEqualTo: user.uid)
      .getDocuments { (snapshot, error) in

But my requirements is I just need to fetch the data written users and not other users so its is rather a case of security rules and not a filter.
Right now I am manually adding User ID field to each of the written data for filtering users data. Which again I think is not the proper way for securing user data.
So can you suggest me the simple Security Rules which allows read only data written by a users who have written the data.

So I think I solved the issue with change of security rules as follows -
match /DebtsInfo/{debtsInfo} {
      allow read: if request.auth != null
      	&& request.auth.uid == resource.data.userID;
      allow create: if request.auth != null;
      allow write: if request.auth != null
          && request.auth.uid == request.resource.data.userID;
}

The change I made is -
Previously it was
request.auth.uid == request.resource.data.userID;
Now it is
request.auth.uid == resource.data.userID;
So I think if is added as an example on documentation it would be helpful to others as well.
Also if there is already the resource on the difference about resource.data vs request.resource on read vs write it would be helpful.

That's documented here. I'll take a look at updating the docs to make it more clear. In the meantime, I'll close this issue since it seems your issue is resolved.
Thanks for filing!


fix: remove len calculation in iterable dataset

Fix.
Bug

In some cases, the self-implemented len property on the NativeGridDataset Class does not return the same number of samples that would be equivalent to that implied by the per_worker_init
In these cases this leads to things such as validation loop skipped, Hanging at end of a training epoch

To Reproduce
The example has the following setup:

multiple GPU
no limit on the training batches: dataloader.limit_batches.training: null
-> validation epoch is skipped.
deleting the len function solves this issue

Example of behaviour here: https://mlflow.copernicus-climate.eu/#/compare-runs?runs=["ab15907c4d5a45dd89a9c708eaea3dbf","dfb1bfa2d72e49769e28a5a138321006","8731ff0d85bf40cbbe7f313fa22af005"]&experiments=["81"]
Additional context
We don't have any functionality built on top of this function so deleting it is the easiest solution to remove the bug.
We are considering options to use a dataset class that allows us to calculate/obtain the length of the dataset.
Possible Reviewers:
@Rilwan-Adewoyin

We don't have any functionality built on top of this function so deleting it is the easiest solution to remove the bug.

If there is no functionality built on top, why does it cause a bug? :D Is it lightning auto calling this function?
Are you sure deleting it doesn't cause adverse effects elsewhere?
@JesperDramsch should we put this one in first release?

If there is no functionality built on top, why does it cause a bug? :D Is it lightning auto calling this function?

I do not fully understand why this is causing errors. I know for sure the number of batches is too high as when deleting the function the training epoch starts earlier. @Rilwan-Adewoyin do you have new insights on this one?

Are you sure deleting it doesn't cause adverse effects elsewhere?

This is a new feature (O(weeks)). The function was not implemented before and the code running smoothly then.
Some colleagues where thinking about building functionality on top of this in the future.

If there is no functionality built on top, why does it cause a bug? :D Is it lightning auto calling this function?

I do not fully understand why this is causing errors. I know for sure the number of batches is too high as when deleting the function the training epoch ends earlier. @Rilwan-Adewoyin do you have new insights on this one?

Are you sure deleting it doesn't cause adverse effects elsewhere?

This is a new feature (O(weeks)). The function was not implemented before and the code running smoothly then. Some colleagues where thinking about building functionality on top of this in the future.

So the reasoning for this was that per_worker_init and len function were not completely aligned although they used similar logic:

the per_worker_init function imposes that each worker for a model_comm_group must have the same number of batches
the len function previously returned the number of samples per model_comm_group
the updated len function now returns number of samples per model_comm_group adjusted for the constraint that the each worker on a model_comm_group returns the same number of elements

Additional context
I also don't think the logic in the per_worker_init is correct in that it enforces each dataloading worker in a model_comm_group to have the same number of samples, whereas it should be the number of samples in a model_comm_group should be distributed across all workers and workers allowed to have different number of samples.


bump electron-prebuilt dep

I also renamed deprecated method loadUrl to loadURL in test.js.
How do you think to deal with other deprecation electron introduce in release 35? Do you prefer to keep compatibily with previous releases or stay up-to-date with new electron modules naming?

up-to-date with new electron modules naming

:+1: Mind doing the name changes too?

Sure!

Thanks :)

my pleasure :smiley: . I'll do the same also on debug-menu and electron-localshortcut


Animation Sequencer does not see DOTween

After install I get errors in Editor:
Library\PackageCache\com.brunomikoski.animationsequencer@0.2.2\Scripts\Runtime\Core\DOTweenActions\AnchoredPositionMoveDOTweenActionBase.cs(32,89): error CS1061: 'RectTransform' does not contain a definition for 'DOAnchorPos' and no accessible extension method 'DOAnchorPos' accepting a first argument of type 'RectTransform' could be found (are you missing a using directive or an assembly reference?)
Library\PackageCache\com.brunomikoski.animationsequencer@0.2.2\Scripts\Runtime\Core\DOTweenActions\ColorGraphicDOTWeen.cs(33,68): error CS1929: 'Graphic' does not contain a definition for 'DOColor' and the best extension method overload 'ShortcutExtensions.DOColor(Camera, Color, float)' requires a receiver of type 'Camera'
Library\PackageCache\com.brunomikoski.animationsequencer@0.2.2\Scripts\Runtime\Core\DOTweenActions\FadeCanvasGroupDOTweenAction.cs(33,67): error CS1929: 'CanvasGroup' does not contain a definition for 'DOFade' and the best extension method overload 'ShortcutExtensions.DOFade(Material, float, float)' requires a receiver of type 'Material'
Library\PackageCache\com.brunomikoski.animationsequencer@0.2.2\Scripts\Runtime\Core\DOTweenActions\FadeGraphicDOTweenAction.cs(33,68): error CS1929: 'Graphic' does not contain a definition for 'DOFade' and the best extension method overload 'ShortcutExtensions.DOFade(Material, float, float)' requires a receiver of type 'Material'
Library\PackageCache\com.brunomikoski.animationsequencer@0.2.2\Scripts\Runtime\Core\DOTweenActions\FillImageDOTweenAction.cs(33,67): error CS1061: 'Image' does not contain a definition for 'DOFillAmount' and no accessible extension method 'DOFillAmount' accepting a first argument of type 'Image' could be found (are you missing a using directive or an assembly reference?)
I went to Packages>Animation Sequencer>Scripts>Runtime.
Checked BrunoMikoski.AnimationSequencer.asmdef
and there was no link to DOTween.Modules assembly

I created ASMDEFs from DOTween Utility Panel and got new error:
Library\PackageCache\com.brunomikoski.animationsequencer@0.2.2\Scripts\Editor\Core\AnimationSequencerControllerCustomEditor.cs(113,38): error CS0117: 'DOTweenEditorPreview' does not contain a definition for 'isPreviewing'

Update Dotween and DOtweenPro for last version, issue is gone :)

Glad that you figured out :)


Update Composer dependencies (2021-05-28-00-08)

Visual regression test passed!
View the visual regression test report


Pickle only used symbols from main namespace.

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @silviulica
In particular, this means we can safely disable main session saving nearly everywhere.

LGTM and awesome! LEt's get this in and I will get the optional save_main_session right away out.

Thanks. Lucky for us, the "recursive" option was already part of dill (not that we wanted to use it everywhere of course).


chore(tools): run tests on Travis CI

What about run tests in Travis CI?

Looks great, thanks!
Do you have a Telegram or something? I want to ask for refactoring advice. 🙏
My telegram as my github account.


Use xml::EventWriter to serialize XML payload

Building XML by format! and naive string concatenations is potentially dangerous because Rusoto users have to escape special XML characters.
So I changed the code generator for rest-xml APIs to use EventWriter provided by xml-rs crate.
https://docs.rs/xml-rs/0.6.1/xml/writer/struct.EventWriter.html

That's fantastic, thanks for the PR.  I think we ended up with the naive string concatenations because EventWriter either wasn't ready or didn't even exist when the XML serializer code was written.  It might be the oldest part of this whole project (SQS was the first service we implemented), and the state of Rust XML support was close to nonexistent at the time.

Oh and an update to the CHANGELOG file would be 👍 .

This is next up on my to-review list. 👍

I'm still seeing the malformed message error from AWS when running the S3 integration tests.  Specifically the multipart upload test.  Anyone else seeing that as well?

The malformed XML error isn't related to this PR.
It should be fixed by these commits: https://github.com/rusoto/rusoto/issues/820#issuecomment-335093098 . Should I include these commints to this PR?

Ah, I was confused as to where those changes were.
If possible, having the S3 fix in a new PR would be great.  If it's easier to put in this PR that'll work too. 👍

Sure. I've re-created a pachset against the master branch: #836 .
If it's gets merged, I'll rebase this PR to resolve conflicts.

Rebased.
S3 integration test is failing in my environment, but it seems to be caused by #837 .

Looks good for everything save the unrelated S3 issue.  Thank you!


Stresstest stoped to early on PI4

The sysbench parameter it to low for the fast Raspberry PI 4 "--cpu-max-prime=10000"
It stopes before the 60 seconds are run out.
Increased to it to 100000

What's the effect/impact on RPi3 devices..?


Renamed object doesnt update references

Hi,
when i try to rename object it doesn't update reference to that object name on others.

That's right.  It's insanely difficult to also rename references, while you actually only need it sporadically .. once in a lifecycle of a project.  It's easy to solve as well, so I didn't implement it.


$ref values must be RFC3986-compliant percent-encoded URIs

Hi,
when we create the swagger we get: $ref: '#/definitions/ResponseDto«CervedVerifyResponseDto»'
and when we validate it we get the following error:
Semantic error at paths./api/engine/cerved/verify.post.responses.200.schema.$ref
$ref values must be RFC3986-compliant percent-encoded URIs
It's related to the symbol «, how can we generate it correctly?

Sorry, I found the solution in the documentation: 6.8.3. Changing how Generic Types are Named


Refactor devices, add cover size stuff

A lot more information (codenames, storage, etc)
Cover sizes
Cleaner code, easier to extend in the future
Codenames now match the internal layout (class -> family + optional secondary)
Based on firmware 4.18.13737

Also see https://gist.github.com/geek1011/613b34c23f026f7c39c50ee32f5e167e and https://github.com/shermp/Kobo-UNCaGED/issues/16
cc @shermp, @NiLuJe, @davidfor

Have you thought about having a generic "fallback" device, name etc with safe defaults such that new devices can still use the library, even if they aren't supported yet?
Also, this looks awesome. I'm inclined to switch to this library in Kobo-UNCaGED to avoid duplication of code.

OK, I've tested the cover generation and some of the device detection as part of geek1011/kepubify#44, and everything seems to work well.

It's Qt4's QtHash, FWIW ;).
(c.f., https://github.com/NiLuJe/kfmon/blob/f19f8582e7b53d14b7a60cc5b8bc43bac1930b48/kfmon.c#L1096)

Ok, I was wondering how much time @shermp would have spent reversing this... 😃
I'll add a note about that for future reference.

Yeah, I lifted it straight from the Kobo driver in Calibre, which is where I originally saw it.

Unless anyone has more suggestions/objections, I'm planning to merge this within a day or two.

I've released this as v2.

@shermp, are you planning on updating KU yourself?

I was thinking of pulling in koboutils as a library and using it instead.
No need to duplicate code/effort :)

That was what I meant, but were you planning to implement it?

Yeah. Want to finalize the changes to UNCaGED first before updating KU.

Sounds good.

Speaking of UNCaGED, would you be willing to have a look at the open PR at some point? I left it for a while, seeing as you were taking a break from Kobo stuff.

Yep, will do today or tomorrow. I'll be taking another break afterwards until the next firmware release.

Thanks


Supporting Markdown tables

It would be useful to support Markdown tables and have them be converted into Typst tables.

is this issue still being worked on ?

Well, pulldown-cmark supports tables natively, so it’s just a question of converting Table into a Typst function call and each cell into a parameter. You should be able to handle the Table event in a new function, which looks for TableHead (converting it to table.header) and handles the cells appropriately. If there are any unexpected events from the parser, you can just panic, they shouldn’t be happening.


Present the UI font proposal to brand

Prepare proposal for meeting with brand Thursday March 14

Presented to and discussed with Nick. Inter approved (for mobile).


No camera permission requested on iOS

Expected Behaviour
I have added the provided usage description required by iOS inside the iOS platform tag of the config.xml file:
<edit-config target="NSCameraUsageDescription" file="*-Info.plist" mode="merge">     <string>To scan barcodes</string> </edit-config>
Actual Behaviour
Scanner works as intended on Android, but not on iOS. Scanner launches but the background is black. Launching the scanner function again then results in a popup stating that camera access is  prohibited and access must be granted in settings. The only permission available in settings is mobile data.
Additional code
This is my content policy of the index.html file if relevant
<meta http-equiv="Content-Security-Policy" content="default-src * 'self' 'unsafe-inline' 'unsafe-eval' data: gap: content:">

I updated this to
<gap:config-file platform="ios" parent="NSCameraUsageDescription" overwrite="true">         <string>To scan barcodes</string>     </gap:config-file>
and now the app asks for camera access permission when launching the scan function for the first time. Any ideas why this is?

@Jonkat323 I have the same problem, when you updated the config file and added:
<gap:config-file platform="ios" parent="NSCameraUsageDescription" overwrite="true"> To scan barcodes </gap:config-file>
did you solve the black screen problem?

@macirex Yes, this solved the black screen issue on iOS as the app now has access to the camera. It should now ask for permission the first time you try use the barcode scanner function.

@Jonkat323 Where did you put that line on the config.xml? I tried putting it on platform but the scanner app still opens before requesting permission and it still remains in black after not allowing it the permission.
This is where I put it:
    <platform name="ios">         <icon height="57" platform="ios" src="www/res/icon/ios/icon.png" width="57" />         <icon height="114" platform="ios"<EMAIL_ADDRESS>width="114" />         <icon height="40" platform="ios" src="www/res/icon/ios/icon-40.png" width="40" />         <icon height="80" platform="ios"<EMAIL_ADDRESS>width="80" />         <icon height="50" platform="ios" src="www/res/icon/ios/icon-50.png" width="50" />         <icon height="100" platform="ios"<EMAIL_ADDRESS>width="100" />         <icon height="60" platform="ios" src="www/res/icon/ios/icon-60.png" width="60" />         <icon height="120" platform="ios"<EMAIL_ADDRESS>width="120" />         <icon height="180" platform="ios"<EMAIL_ADDRESS>width="180" />         <icon height="72" platform="ios" src="www/res/icon/ios/icon-72.png" width="72" />         <icon height="144" platform="ios"<EMAIL_ADDRESS>width="144" />         <icon height="76" platform="ios" src="www/res/icon/ios/icon-76.png" width="76" />         <icon height="152" platform="ios"<EMAIL_ADDRESS>width="152" />         <icon height="29" platform="ios" src="www/res/icon/ios/icon-small.png" width="29" />         <icon height="58" platform="ios"<EMAIL_ADDRESS>width="58" />         <icon height="87" platform="ios"<EMAIL_ADDRESS>width="87" />         <splash<EMAIL_ADDRESS>/>         <resource-file src="www/GoogleService-Info.plist" />         <gap:config-file platform="ios" parent="NSCameraUsageDescription" overwrite="true"> <string>To scan barcodes</string> </gap:config-file>      </platform>
Any insights will be appreciated, thanks

No not inside the iOS platform. I put it after the list of plugins and before the platforms.

Okay, Thanks! I'll try again!

I couldn't make it work, I think the problem is that I'm using cordova and not phonegap and I don't know if this fix works on the config.xml of cordova

I am using Cordova too! So my plugin is
<plugin name="cordova-plugin-barcodescanner" version="0.7.4"/>
Then afterwards I have the following for Android:
<config-file target="AndroidManifest.xml" parent="/*" mode="merge">
    <uses-permission name="android.permission.CAMERA" />
    <uses-feature name="android.hardware.camera" />
    <uses-feature name="android.hardware.camera.autofocus" />
</config-file>

And then this for iOS:
<gap:config-file platform="ios" parent="NSCameraUsageDescription" overwrite="true">
    <string>To scan barcodes</string>
</gap:config-file>


I'm confused by time

help, see
The plan I started last night stopped this morning
I tried several times to no avail

At the beginning.  I can't directly 22:36-9:00
Help give pointers, thank you

The < and > means the previous / upcoming day.
2022-09-21
  09:00 - 10:00>   this translates to "9am the 21st of September" to "10am the 22nd of September"

This is why you get the "Entry in the future", because it ends on the 22nd of September, while it's still the 21st.

Looking at your example, if you want to add an entry for the 21st of September that started at 22:36 the day before, you could do:
2022-09-20
  <22:36 - 9:00

Finally, just to hammer down that you do not need either < nor >:
2022-09-21
  9:00 - 10:00   this is 1 hour total

和表示前一天</>即将到来的一天。
2022-09-21
  09:00 - 10:00>   this translates to "9am the 21st of September" to "10am the 22nd of September"

这就是为什么您会获得“未来条目”的原因，因为它在 9 月 22 日结束，而现在仍然是 21 日。
看看你的例子，如果你想为前一天 22:36 开始的 9 月 21 日添加一个条目，你可以这样做：
2022-09-20
  <22:36 - 9:00

最后，只是为了确定您不需要也不<需要>：
2022-09-21
  9:00 - 10:00   this is 1 hour total

thanks😁


Adds planner task reference add command. Closes #3283

Closes #3283
Bit of additional context.
I created the method customURIEncoder to encode some of the special characters needed to pass the url along. More info
Also within planner/commands.ts and docs/mkdocs.yml were a few commands that were not in alphabetical order. I rearranged them aswell.

Awesome! We'll review it shortly

@pnp/cli-for-microsoft-365-maintainers  could I ask someone for a double check 🙏`

Hi,
I am going to review it this week-end.
Cheers

@plamber, openTypesEncoder has now been moved to the formatting utils.

Hi,
thank you all for the great work. Manually merged.
Cheers


Add pagination to People index

I'd be interested in addressing this issue, is there an accepted gem/pattern for pagination today you'd like implemented?

Hello @yez! 👋🏾  Appreciate your interest.
We haven't added pagination to any of our pages/endpoints yet, so no accepted gem or pattern is in place.
That said, i've been meaning to give pagy a go.
Open to thoughts on this?

Pagy looks neat, I'll try it out, thanks!


cannot docker:remove

Is there a way to pass -f when docker:remove runs?

mvn -Ddocker.image=registry.jufis.net:443/jufis-dev/docker-rest-server:latest -Ddocker.removeAll clean docker:remove

[ERROR] DOCKER> Conflict, cannot delete 5a45a0e0111c because the container 3feabe04b961 is using it, use -f to force (Conflict: 409)
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 2.309 s
[INFO] Finished at: 2015-05-22T02:32:15+03:00
[INFO] Final Memory: 17M/297M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.jolokia:docker-maven-plugin:0.11.4:remove (remove) on project docker-rest-server: Unable to remove image [registry.jufis.net:443/jufis-dev/docker-rest-server:latest] -> [Help 1]

Cheers!

By default 'force' is always applied, but there is a bug which prevent this ;). This will be fixed in 0.11.5.


Support for multiple grouping columns for calculations like increaseColumns

Increase Columns only supports one column as the grouping option.
I have metrics from the ClickHouse Otel Exporter.
https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/clickhouseexporter
I'd like to be able to run increase calculations that support multiple grouping columns, because the metrics typically have multiple attributes that require distinct groupings for correct increase calculations.
Your example statement uses runningDifference, which Calculates the difference between two consecutive row values in the data block.
SELECT
    t,
    groupArray((increaseColumns, max_0_Increase)) AS groupArr
FROM
(
    SELECT
        t,
        Protocol,
        if (runningDifference(max_0) < 0 OR neighbor(increaseColumns,-1,increaseColumns) != increaseColumns, 0, runningDifference(max_0)) AS max_0_Increase
    FROM
    (
        SELECT
            (intDiv(toUInt32(EventTime), 60) * 60) * 1000 AS t,
            Protocol AS increaseColumns,
            max(Requests) AS max_0
        FROM requests
        WHERE ((EventDate >= toDate(1535711819)) AND (EventDate <= toDate(1535714715)))
        AND ((EventTime >= toDateTime(1535711819)) AND (EventTime <= toDateTime(1535714715)))
        AND (Protocol IN ('udp', 'tcp'))
        GROUP BY
            t,
            Protocol
        ORDER BY
            t,
            Protocol
    )
)
GROUP BY t
ORDER BY t

How can I group by additional columns from the data.
If my table was:
CREATE TABLE
(
   EventTime,
   Requests,
   Protocol,
   IsSecureRequest
)

How could I get distinct increases for Protocol, IsSecureRequest
The existing query won't distinguish between the second grouping IsSecureRequest.
Ideally I would have multiple columns that need to be grouped into Increases.
The current approach won't support this.
I tried expanding IncreaseColumns:
(Protocol, IsSecureRequest) AS increaseColumns
Sense the "the difference between two consecutive rows" doesn't understand each grouping is its own logical set of rows the end result merges the two instead of groups by them.

try to use
$increaseColumns(concat(Protocol,if(isSecureRequest,"_secure","") AS increaseColumns)
and report is it success for you
we will have one data block for runningDifference cause use sub queries


Running on VS Code

Can you teach me how to setup my code on VS Code?
What I did was open up the Terminal. Then I copied from the C# Command Line and pasted
"
dotnet new console -lang Q# --output Bell
cd Bell
"
into the Terminal. But then I received an error saying
"
Creating this template will make changes to existing files:
Overwrite   Program.qs
Overwrite   Bell.csproj
Rerun the command and pass --force to accept and create.
"
this in the terminal.

@mohangude I believe you executed dotnet new console -lang Q# --output Bell twice
So, nothing wrong with your commands, and you are acutely ready to go with your Bell project


feat: allow direct opening of variant details via results table (#1057)

This will need varfish-server-worker=0.10.0 in bioconda channel.

OK, putting this back to 0.9.0 for now, we can use the built master worker.


plot spatial error with multiple slides

I followed the latest tutorial (https://cell2location.readthedocs.io/en/latest/notebooks/cell2location_tutorial.html) and was able to reproduce the results successfully. When I used my own data with multiple slides, I followed the tutorial (https://cell2location.readthedocs.io/en/latest/notebooks/cell2location_short_demo.html#1.-Loading-Visium-data) to load the visium data, then proceeded with the latest tutorial. An error occurred as follows:
fig = mod.plot_spatial_QC_across_batches()

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/cell2location/models/_cell2location_model.py", line 421, in plot_spatial_QC_across_batches
    scanpy.pl.spatial(
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/scanpy/plotting/_tools/scatterplots.py", line 1002, in spatial
    axs = embedding(
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/scanpy/plotting/_tools/scatterplots.py", line 391, in embedding
    cax = scatter(
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/scanpy/plotting/_utils.py", line 1107, in circles
    x = x * scale_factor
TypeError: can't multiply sequence by non-int of type 'float'

The same error occurred when I tried to plot cell abundance in spatial coordinates:
from cell2location.utils import select_slide
slide = select_slide(adata_vis, 'A1')

with mpl.rc_context({'axes.facecolor':  'black',
                     'figure.figsize': [4.5, 5]}):
    sc.pl.spatial(slide, cmap='magma',
                  color=['CD14+Mo', 'CD16+Mo', 'MoMP', 'LYVE1+IGF1+MP', 'LYVE1+TIMD4+MP',
                  'LYVE1+MP_cycling', 'DC', 'Neut'],
                  ncols=4, size=1.3,
                  img_key='hires',
                  # limit color scale at 99.2% quantile of cell abundance
                  vmin=0, vmax='p99.2'
                 )

Traceback (most recent call last):
  File "<stdin>", line 3, in <module>
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/scanpy/plotting/_tools/scatterplots.py", line 1002, in spatial
    axs = embedding(
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/scanpy/plotting/_tools/scatterplots.py", line 391, in embedding
    cax = scatter(
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/scanpy/plotting/_utils.py", line 1107, in circles
    x = x * scale_factor
TypeError: can't multiply sequence by non-int of type 'float'

Hi @jil100
as you note with your second example, this is an issue with scanpy spatial plotting (related to scale_factor) rather than with cell2location. Can you plot the expression of genes? If you get the same error with gene plotting I would suggest creating an issue with scanpy package.

@vitkl
Thank you for the quick response. It did seem to be a scanpy spatial plotting issue, and I was able to plot cell abundance in spatial coordinates after I followed the suggestion given in https://github.com/scverse/scanpy/issues/2499 (although I still got the same error for   fig = mod.plot_spatial_QC_across_batches()).  However, when I tried to save the adata_vis as follows:
adata_file = f"{run_name}/sp.h5ad"
adata_vis.write(adata_file)

I got the following errors:
Traceback (most recent call last):
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/utils.py", line 246, in func_wrapper
    return func(*args, **kwargs)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/specs/registry.py", line 311, in write_elem
    return write_func(store, k, elem, dataset_kwargs=dataset_kwargs)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/specs/registry.py", line 52, in wrapper
    result = func(g, k, *args, **kwargs)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/specs/methods.py", line 359, in write_vlen_string_array
    f.create_dataset(k, data=elem.astype(str_dtype), dtype=str_dtype, **dataset_kwargs)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/h5py/_hl/group.py", line 183, in create_dataset
    dsid = dataset.make_new_dset(group, shape, dtype, data, name, **kwds)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/h5py/_hl/dataset.py", line 166, in make_new_dset
    dset_id.write(h5s.ALL, h5s.ALL, data)
  File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
  File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
  File "h5py/h5d.pyx", line 283, in h5py.h5d.DatasetID.write
  File "h5py/_proxy.pyx", line 145, in h5py._proxy.dset_rw
  File "h5py/_conv.pyx", line 444, in h5py._conv.str2vlen
  File "h5py/_conv.pyx", line 95, in h5py._conv.generic_converter
  File "h5py/_conv.pyx", line 249, in h5py._conv.conv_str2vlen
TypeError: Can't implicitly convert non-string objects to strings

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_core/anndata.py", line 1959, in write_h5ad
    _write_h5ad(
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/h5ad.py", line 92, in write_h5ad
    write_elem(f, "obs", adata.obs, dataset_kwargs=dataset_kwargs)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/specs/registry.py", line 353, in write_elem
    Writer(_REGISTRY).write_elem(store, k, elem, dataset_kwargs=dataset_kwargs)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/utils.py", line 248, in func_wrapper
    re_raise_error(e, elem, key)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/utils.py", line 246, in func_wrapper
    return func(*args, **kwargs)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/specs/registry.py", line 311, in write_elem
    return write_func(store, k, elem, dataset_kwargs=dataset_kwargs)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/specs/registry.py", line 52, in wrapper
    result = func(g, k, *args, **kwargs)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/specs/methods.py", line 579, in write_dataframe
    _writer.write_elem(
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/utils.py", line 248, in func_wrapper
    re_raise_error(e, elem, key)
  File "/home/il55/cell2loc_env/lib/python3.9/site-packages/anndata/_io/utils.py", line 229, in re_raise_error
    raise type(e)(
TypeError: Can't implicitly convert non-string objects to strings

Above error raised while writing key 'in_tissue' of <class 'h5py._hl.group.Group'> to /

@vitkl Thank you for your quick response. It was indeed an issue with scanpy spatial plotting.


import { NgSpinKitModule } .... can find it

For some reason I don't have this module

ERROR in NgSpinKitModule is not an NgModule


PyTorch 1.8.1 - CUDA 11.1 - Windows 10 - unsatisfied link

I am running into a Problem on a MS Windows 10 OS.
With  djl.version=0.11.0-SNAPSHOT and
      <dependency>
          <groupId>ai.djl.pytorch</groupId>
          <artifactId>pytorch-native-auto</artifactId>
          <version>1.8.1-SNAPSHOT</version>
      </dependency>

on CUDA 11.1 I get a java.lang.UnsatisfiedLinkError. I see links to the following libraries that are not found

on CUDA 10.2 (on a Turing GPU) everything works fine.

@goswamig Please check the CUDA dependencies missing issue.

Looks like we are missing packages around here. Do you find any other packages that are missing? @enpasos

We tried to fix it with : #850 and #852. Please try again once the nightly build is passed: https://github.com/awslabs/djl/actions/runs/743340536

Thanks.
Nightly build says past.
I still get the unsatisfied link exception on

In dependency analyser I see

and in my filesystem

@enpasos you can clean up cache, like delete .djl.ai and try again.

@lanking520 deleted .djl.ai and .m2 ... tried again ... same picture as reported above

@enpasos can you try with auto package? This change is not on Maven snapshot yet

@lanking520 What do I have to do for "auto package"?

Is it more than

@enpasos Nope, let me clean up the cloud front cache and help you out。 Now please try again

@lanking520 Thank you. Now the two missing dependencies are there. Java still complains but my dependency analyser says ok. I will investigate ... will take some time.

@enpasos try to set a break point here: https://github.com/awslabs/djl/blob/master/pytorch/pytorch-engine/src/main/java/ai/djl/pytorch/jni/LibUtils.java#L120 and debug line by line. Let me know which line you hit the unsatisfied link issue

@lanking520
I debugged LibUtils ... it was not reaching the code you added ... but after adding the libraries to the "load later set" it worked and the "unsatisfied link" problem diappeared

@enpasos nice catch! I will apply this change accordingly

I checked out the lates code with #854, compiled it, tested -> problem solved