k8s: Update kubernetes deployment images

The kubernetes deployment images that are marked in the templates are
quite old (~2 years) and don't provide ARM images.  The image
publication has been moved to k8s.gcr.io/sig-storage in the interim
where ARM images are also available.
One of the updates also requires some additional role permissions for
the service user.
Additionally, this change updates the fsGroupPolicy for the CSIDriver as this
appears to function correctly and makes it more broadly useful.

Some of changes look duplicate with https://github.com/SynologyOpenSource/synology-csi/pull/9


fics LDMIA edge case

Thanks for the help on this one, fixed in https://github.com/mattrberry/crab/commit/3b71156bd3ca75daf84f946f0318f868307d9d70!


Project contributer section #28

for this implementation i make new page contributors.jsx and each card have button of contributors and it will redirect to the contributors page
Contributors page -> it will fetch the contributor data from the API "/api/projects/contributors/reponame" and display

@vasu-CE do above changes, look at current Projects.jsx component, don't remove current animations, you just need to add a button there.

#62


Abide by strict provenance

There's a chance that Rust will eventually adopt a "strict provenance" model for determining which operations on pointers are sound. Consistent with the guarantees we aim to make as part of #61, we should restrict ourselves to operations which are sound under this model. The sptr crate provides a polyfill that would allow us to make use of an API that is currently unstable in Rust itself.
Note that we are likely already compliant with strict provenance, as we test using strict provenance in our Miri tests.
This issue tracks the following tasks:

[ ] Ensure that zerocopy abides by strict provenance, for example by using the sptr crate*
[ ] Update our docs to describe our adherence with strict provenance

This is not quite as strong as a full forwards-compatible soundness guarantee, but it is still very strong

[ ] Work with Rust to promise that Rust will never adopt a model which is more strict than strict provenance

This will allow us to use our compliance with strict provenance as a proof of forwards-compatible soundness for #61

* Another option - since folks prefer crates without many dependencies - is to only use sptr as a dev-dependency which is exercised in CI. We could even define a polyfill shim inside of zerocopy which is just a thin wrapper around sptr so that we prevent ourselves from ever "slipping" from its usage (and to make conditional compilation less viral).

Strict provenance has made a lot of progress since this issue was opened, and may land soon: https://github.com/rust-lang/rust/pull/130350


Merge pull request #4 from aniket-2003-das/sanketb

Merge pull request #3 from aniket-2003-das/main

commit


Filter team members from /open-source

DEVX-1697
PR-Codex overview
This PR adds two new GitHub usernames to the list of contributors in the open-source.tsx file.
Detailed summary

Added Yash094 and gregfromstl to the list of contributors.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

📝 Changed routes:

/open-source

(view full size)

Commit fd4adc6e570e109355299b565c244622c3f8696d (https://thirdweb-p43y50g89.thirdweb-preview.com).


Tagging without multiple with new tags doesn't works when using angularjs 1.3.2

When using angularjs 1.3.2 tagging without multiple doesn't work for new items that are not added to the array.
I've also tried editing demo-tagging.html, by replacing the angularjs libraries with following:
<script src="http://ajax.googleapis.com/ajax/libs/angularjs/1.3.2/angular.js"></script>
<script src="http://ajax.googleapis.com/ajax/libs/angularjs/1.3.2/angular-sanitize.js"></script>

And it seems to confirm something changed in the 1.3.2 to break this feature.
I'm really hoping there's an easy fix for this :disappointed: as this feature was exactly what I needed.

And just trying to use angularjs 1.3.0 on the app I'm working on does run the tagging function, but doesn't seem to pass any arguments such as the new object.
$scope.tagTransform = function (newTag) {
    console.log("new tag", newTag, arguments);
    // ...
  };

+1

I've written a quick workaround you can find there.
Not sure it's good enough for a pull request but may do the job for the while.

+1 same issue here.

I am facing the same issue , tagging is not enabling with following code using angularjs 1.3.2

{{$select.selected}}

{{color}}

I actually see this same problem in 1.3.1 as well. Works fine in 1.3.0.

I'm facing the same problem.

same problem here
you can reproduce the problem in the official example
http://plnkr.co/edit/m1SQXUxftBLQtitng1f0?p=preview
by changing the angular version in line 21 from
<script src="http://ajax.googleapis.com/ajax/libs/angularjs/1.3.0/angular.js"></script>
to
<script src="http://ajax.googleapis.com/ajax/libs/angularjs/1.3.1/angular.js"></script>
now tagging of a single-select isn't working anymore
i also tried
<script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.4.0-rc.2/angular.min.js"></script>
and it's not working

Same problem  here. Any issue ?

I'm facing the same problem. Please help

This temporary fix worked for me:
https://github.com/angular-ui/ui-select/issues/983#issuecomment-109948338

Please make tagging work also without the workaround tagging-label="false"...

:+1:

I solved this with older version of ui-select. I am using version 0.11.0. With angular >1.3.2 it works as it's suppoused to and with angular <=1.3.2 you have to set tagging-label to "(new)" explicitly otherwise "undefined" will be shown.


2D Root Motion?

Godot version: 3.1
Issue description:
I know that Godot 3.1 has Root Motion feature for 3D. But how should I implement something similar in 2D? I would like to move character quite arbitrarily. Simple motion (like running) is controlled from script, but I want the character to be able to move by means of animation as well (for instance, parabolic dashes / rolls / whatever), though, with right physics processing. The character is controlled via KinematicBody2D.
What I have tried (and still trying):

Adjusted values of track keys responsible for position property. Then dynamically switch sync_to_physics option of KinematicBody2Don before animation start and off after it is finished. That results in a strange glitch with camera, but that is not the main problem. This approach does not support collisions (I managed to pass through walls).
Again adjusted values of keys, then dynamically calculated velocity necessary to move from key1 to key2, key2 to key3, etc. Turned off the position track so that not to affect position directly. Motion is performed with move_and_slide so physics is taken into the account. But the results are not stable: the result position differs quite significantly from what is expected (currently it is like there is one unexpected extra frame so the character sometimes moves further), though, the velocity is constant and delta is constant too (I call move_and_slide inside physics_process)

Anyway, both approaches do not seem like something completely right to me. Maybe, there's something else out there that influences motion and introduces some undesired variations. I think, I will check it out on a clean minimal project later.
The question is: how should I achieve root motion in 2D? Are the approaches above acceptable or just silly hacks? How do you implement both simple and complex motion in your games?
Examples of how I do it:
Approach 1
repro_root_motion_sync_to_physics.tar.gz
Approach 2
repro_root_motion.tar.gz
Hit Run and see position printed in the console. Press space to play animation, move with left/right arrows

The issue tracker is not a place for questions. You should use an appropriate channel for this(such as IRC, Discord, forums etc). Sorry but closed.

@Chaosus thanks a lot) I have asked the question in Q&A. Here's a link: https://godotengine.org/qa/42831/root-motion-in-2d

The issue can be interpreted as a valid feature proposal (perhaps needs to be reworded accordingly so it doesn't come off as a pure question).
According to my little research, not many game engines have root motion for 2D, but we know Godot is unique so it makes sense to implement it!

Alright

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Use joernio fork of php-parser for build robustness and char bug fix

With this PR, php2cpg will use the joernio fork of PHP-parser at https://github.com/joernio/PHP-Parser instead of the standard php-parser release installed with composer. This comes with 2 benefits:

the new version doesn't rely on PHP composer's autoloaders, so we can fetch the source directly from the release and avoid the step of installing the parser via composer. This has been quite flaky in the past, so the change should improve general setup stability
the new version includes a fix for missing files found in wordpress, and any other codebase that uses single byte characters with values greater than 127 (in the case of wordpress, \xFF)

This PR also adds a php test to distroTests.

@johannescoetzee this change removed joern-cli/frontends/php2cpg/bin/installdeps.sh which is stil referenced in https://github.com/joernio/joern/blob/b916a61/.github/workflows/upgrade-deps.yml#L23-L24 and therefor reddifies the 'upgrade-dependencies' runner: https://github.com/joernio/joern/actions/runs/4760723865/jobs/8461301739
I guess we can just drop that call?


Clarify lambda interface to map.upsert

The current example is unclear as to which value is the old value
and which is the new value when setting up the lambda. By switching
from addition to subtraction in the example, this becomes clear.
This documentation could be improved further but this is a decent
start towards resolving that possible confusion without having
to look at the source code.
Close #1337

Good call.  Thank you!


Add null check for GDExtension deinitialization

Fixes #87937

Please squash your commits into one, see here

Thanks!

Cherry-picked for 4.2.2.


Remove the share parameter since the accessMode should be determined by the PVC

Just leaving some explanation here, why the share flag can be removed.
If I remember right the share flag is to force RWX mode, which isn't needed it's better to have the PVC determine the access mode instead. This way non RWX migratable volumes are just regular RWO volumes.

Since RWX is a requirement to have access to a volume from different nodes at the same time.
Migratable just determines whether we use the NFS path for "real" rwx or the "hacky" RWO path for 2 different engines that cannot be used concurrently.

The absence of the field is the same as having it set to false.
But I agree it's better if it's not even shown


dbeaver/pro#2712 Added a shortcut for advance copy with last settings

Added a shortcut Ctrl+Shift+Alt+C for an advance copy with the last settings. If the last settings are undefined (e.g. at the first time) then it should work as a usual advance copy. Added hint on the form.

https://github.com/user-attachments/assets/67519303-1ced-47b3-b881-de79a7ae583c

verified


How to insert a value into a specific cell of an existing excel file?

I know kind of a noob question but I can't figure out how to put a value into specific cell of an existing excel file.
For example I have an excel file with the headers "First Name" "Surname" and "Phone Number" and now I want to insert a value into cell B2. How can I do that?
This is a code snippet from what I already (unsuccessfully) tried:
Import-Module ImportExcel
$directory = get-location
$excelFile = Open-ExcelPackage "$directory\Test123.xlsx"
Set-ExcelColumn -ExcelPackage $excelFile -Worksheetname "Tabelle1" -StartRow 2 -Value "Smith!"
Close-ExcelPackage -ExcelPackage $excelFile -Show
And this is the exmaple excel file:
Test123.xlsx
Thanks for your help! :D

-PassThru returns the "Excel Object" same as the Open-ExcelPackage.
$data = ConvertFrom-Csv @"
Region,State,Units,Price
West,Texas,927,923.71
North,Tennessee,466,770.67
East,Florida,520,458.68
East,Maine,828,661.24
West,Virginia,465,053.58
North,Missouri,436,235.67
South,Kansas,214,992.47
North,North Dakota,789,640.72
South,Delaware,712,508.55
"@

Remove-Item ./test.xlsx -ErrorAction SilentlyContinue
$xlpkg = $data | Export-Excel ./test.xlsx -PassThru

Set-ExcelRange -Worksheet $xlpkg.Sheet1 -Range "D13" -Value "Hello World" -AutoSize -BackgroundColor Red

Close-ExcelPackage $xlpkg -Show

Closing this issue now, found a work around with CSV files.


Emitting Hyper_L

Xmodmap has support for the historic Hyper key (archwiki source):
$ xmodmap -pm

xmodmap:  up to 4 keys per modifier, (keycodes in parentheses):

shift       Shift_L (0x32),  Shift_R (0x3e)
lock        Caps_Lock (0x42)
control     Control_L (0x25),  Control_R (0x69)
mod1        Alt_L (0x40),  Meta_L (0xcd)
mod2        Num_Lock (0x94)
mod3
mod4        Super_R (0x86),  Super_L (0xce),  Hyper_L (0xcf)
mod5        ISO_Level3_Shift (0x5c),  ISO_Level3_Shift (0x6c),  Mode_switch (0x85),  Mode_switch (0xcb)

Which seems to have keycode 0xcf. Sxhkd has support for catching Hyper as one of the modifiers.
However neither keycode nor symbol can be found in linux/input-event-codes.h.
More generally, maybe aliasing a keycode directly could serve as a workaround for similar cases.

Yeah, this is another example of that difference between linux kernel key events and X11 key events. X11 takes the linux-kernel events and maps them onto its own version of key-events, but this is completely out of scope for kmonad since we try to stick as close to kernel as possible (this way it generalizes, for example, to Wayland, or even to Windows (which, admittedly, is a huge headache, but it works :-)).
I'm going to close this issue, since there is nothing I can do about it in kmonad, but it is a good idea to add a bit of documentation about this in the README at some point.

Actually, here's a bit of documentation about how to map linux key events to X11 key events to get a hyper-key to register:
https://wiki.archlinux.org/index.php/Xmodmap#Turn_CapsLock_into_Control,_and_LeftControl_into_Hyper

@david-janssen An example or some docs on how to get a key to double as hyper and a standardd key with kmonad would be great. I can't figure it out.

Hey Divan,
So, the issue is that HyperL is not a linux-key at all, it is a special key that gets supported by X11. KMonad doesn't stray at all into X11 territory, this way we can function independently. Therefore, to make HyperL work you will need to do 2 things:
Map a key in KMonad to emit some linux keycode, K
Instruct X11 to tread K like HyperL
The easiest way to do that, I guess, would be to figure out exactly what key you are using right now to trigger HyperL. Let's say you current OS treats right control as HyperL, then simply mapping some key to emit right control should do the trick.
An additional source of confusion is that the keycodes used by xmodmap don't actually line up with the linux event-codes. So if you look at this subset of a .xmodmap configuration I copy-pasted from the above wiki link:
clear      lock
clear   control
keycode      37 = Hyper_L
keycode      66 = Control_L

Then you can see that they map 37 ('Control_L') to 'Hyper_L', and 66 ('CapsLock') to 'Control_L'.
So basically, to repeat: figure out which intermediate button you want to use, then make sure your kmonad config emits that keycode in the way you want it to (i.e. which layer, tap-hold combo etc.), and also make sure that X11 knows that that key should be 'HyperL'.
Sorry, it's hard to explain this stuff in text without being confusing. Hope I managed to clarify things? I think the idea of writing some documentation explaining this is a good idea, so I am reopening this issue.

@david-janssen Brilliant. Thanks for the detailed quick reply. Let me have a look at it and will revert.

Yeah, this is another example of that difference between linux kernel key events and X11 key events. X11 takes the linux-kernel events and maps them onto its own version of key-events, but this is completely out of scope for kmonad since we try to stick as close to kernel as possible (this way it generalizes, for example, to Wayland, or even to Windows (which, admittedly, is a huge headache, but it works :-)).

Hmm, it seems that the super key is not listed in input-event-codes.h either (am I missing it?), yet you mention that super can be configured with kmonad. How do you map the super key?

If you use emacs, the manual says that you can enter C-x @ h to add the Hyper flag to the next character (i.e. your mimicking H- hyper key.
So, this setup worked for me:
(defalias hyper #(C-x S-2 h) ;; S-2 produces the '@'

and just assign @hyper to any button you fancy.


getting unrecognized selector sent to instance

var document = sketch.selectedDocument; does not seem to work anymore.

It seems the API has changed.


GET calls to 'http://<IP_ADDRESS>/metadata/instance/compute' appear after using this library

We have started seeing HTTP GET calls to http://<IP_ADDRESS>/metadata/instance/compute in our Seq logs, after we introduced this library for Seq tracing. Why is it so and can it be prevented?
On a side note, all these logs are logged as Error, with no other information available. Screenshot below:

Using:
    .Instrument.WithDefaultInstrumentation(false)

will turn off HttpClient instrumentation entirely, while leaving everything else intact. As Liam suggested, filtering these particular ones out is likely a better strategy.
The null status code generally means that the request was canceled or an exception thrown - my guess is that the exception is being thrown is caught and suppressed elsewhere in the stack.

Using:
    .Instrument.WithDefaultInstrumentation(false)

will turn off HttpClient instrumentation entirely, while leaving everything else intact. As Liam suggested, filtering these particular ones out is likely a better strategy.
The null status code generally means that the request was canceled or an exception thrown - my guess is that the exception is being thrown is caught and suppressed elsewhere in the stack.

Can you point me to the library code where this particular HTTP GET call happens. I tried digging the source code but couldn't find any reference to the mentioned IP address or the relative URL.

SerilogTracing isn't making the HTTP call, it's just recording that the HTTP call is being made. I think that request has something to do with Azure infrastructure, perhaps? It'll be one of the components in your app performing the HTTP request. SerilogTracing is just informing you about it.

Thank you @nblumhardt for the pointer.
Those HTTP GET calls in my case are due to Azure Application Insights enabled. Disabling it makes the calls disappear as well.

@nabeel-servcorp  Were you able to figure out how to get QuickPulse heartbeats or v2/track calls ignored when using Application Insights? When in "developer mode" it literally just spams non-stop in-a-loop tracking of v2/track... have you hit that?

@nabeel-servcorp Were you able to figure out how to get QuickPulse heartbeats or v2/track calls ignored when using Application Insights? When in "developer mode" it literally just spams non-stop in-a-loop tracking of v2/track... have you hit that?

Yes. Two ways to do that.

Add a Seq filter to hide them
Use code something like the snippet shown below:

var aiOptions = new ApplicationInsightsServiceOptions();
// Disables QuickPulse (Live Metrics stream).
aiOptions.EnableQuickPulseMetricStream = false;
services.AddApplicationInsightsTelemetry(aiOptions);

@nabeel-servcorp  - Do you have NewRelic instrumentation / agents installed on your hosts? We are having exactly the same issue and trying to work out what is causing it.

@nabeel-servcorp - Do you have NewRelic instrumentation / agents installed on your hosts? We are having exactly the same issue and trying to work out what is causing it.

No.

@nabeel-servcorp  Did you end up working out what is causing it or did you just disable HttpClient instrumentation?
Although from what i can see, its being logged via the AspNetCode request instrumentation.


Use SVG vector icons instead of PNG.

Before:

After:

NB: SVG icon used here is a modified version of the MicroPython logo already included in the project. This is due to IDEA adding drop shadow behind icon in the menu, which in turn blurs into solid blocks of gray between different "twists". I've pushed them apart a little bit to offset the effect and it now looks cleaner. :)

@lensvol Thanks for updating the icons!


Add new docs

[Current coverage][cc-pull] is 97.17%

Merging [#154][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #154   diff @@
==========================================
  Files            13         13
  Lines           531        531
  Methods           0          0
  Messages          0          0
  Branches         54         54
==========================================
  Hits            516        516
  Misses            8          8
  Partials          7          7

Powered by Codecov. Last updated by [ec1ddc0...0aaf0e2][cc-compare]
[cc-base-branch]: https://codecov.io/gh/lxc/pylxd/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/lxc/pylxd/compare/ec1ddc0baf7f3c4feddf830786dae60fd9f30b88...0aaf0e27d3df3b33907669e3636755f50b7452e3
[cc-pull]: https://codecov.io/gh/lxc/pylxd/pull/154?src=pr

awesome


Improve error messages for registering prod services

What type of PR is this?

What does this PR do / why is it needed ?
Improve error messages for registering prod services

Which issue(s) this PR fixes:

Fixes #
Other notes for reviewers:
Does this PR introduce a user-facing change?

Reviewed / Approved by @MauricioUyaguari  and @aziemchawdhary-gs


net/http: Transport crash in tryPutIdleConn

Test flake in trybot run of https://go-review.googlesource.com/c/38351/
https://storage.googleapis.com/go-build-log/7db6e167/linux-amd64-race_2fc40adc.log
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x428943]

goroutine 136 [running]:
net/http.(*Transport).tryPutIdleConn(0xc42032c5a0, 0xc42033fb00, 0x0, 0x0)
	/tmp/workdir/go/src/net/http/transport.go:688 +0x29a
net/http.(*persistConn).readLoop.func2(0x0, 0xc420386201)
	/tmp/workdir/go/src/net/http/transport.go:1490 +0x6f
net/http.(*persistConn).readLoop(0xc42033fb00)
	/tmp/workdir/go/src/net/http/transport.go:1587 +0x1549
created by net/http.(*Transport).dialConn
	/tmp/workdir/go/src/net/http/transport.go:1168 +0xbf5
FAIL	net/http	0.207s

Maybe CL 38351 is to blame? But in case it's not, filing this.

Likely caused by the CL - the runtime channel code it touches has raceenabled in there, so it could be that I simply broke -race. I'll look into this tomorrow.

Yeah, I can reproduce the crash via go test -race net/http. Closing as it's a regression in the CL, not a flake already present in tip.


Updated interop tests with support for compression.

Incluiding UNCOMPRESSABLE payload type, which relies on a 512KB file with data from /dev/urandom, rnd.dat, also included in this PR.
This PR includes #2586 (to be merged separately).

Sorry for the delay. The code looks fine but the change breaks the spec in https://github.com/grpc/grpc/blob/master/doc/interop-test-descriptions.md
I would suggest adding new test cases to the spec instead of altering the existing test case since there are existing implementations in other languages also.

Done. PTAL. I'll update the documentation if everything looks fine.

PTAL after offline discussion about unnecessary changes to server.

LGTM. Can we add a comment at the last line you removed saying if the response_type is not set, it defaults to COMPRESSABLE? Note in places we set it explicitly but not in DoLargeUnary().

Done.

The sanity tests are failing, maybe try regenerating the project files?

Merged with master, regenerated (Grpc.mak was outdated) and rebuilt on Jenkins... let's see

Merged.

Ping :)

It seems the sanity is failing. Could you take a look?
On Mon, Aug 17, 2015 at 2:01 PM, David G. Quintas<EMAIL_ADDRESS>wrote:

Ping :)
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/2603#issuecomment-131960014.

Yeah, I re-merged and re-run the tests. It's about to finish. So far so green. PTAL.

Jenkins is finally green-ish


Replace Array.slice.call to while loop for IE8 support.

Replace Array.slice.call to while loop for IE8 support.
Because IE8 not recognize Array.slice.call then instafeed totally not works in IE8.

Hey @dorajistyle! I wasn't able to merge your PR directly, but v1.4 fixes this bug.
Thank you for you patience!


axes dropping sign

Examine the attached image. Plot mesh plot, add transform operator with scale by 3 in y and translate by 2 in y. Note the resulting missing negation sign on the y axis '5'
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 2737
Status: Rejected
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: axes dropping sign
Assigned to: -
Category: -
Target version: -
Author: Mark Miller
Start: 01/14/2017
Due date:
% Done: 0%
Estimated time:
Created: 01/14/2017 09:09 pm
Updated: 01/17/2017 06:48 pm
Likelihood: 3 - Occasional
Severity: 2 - Minor Irritation
Found in version: 2.12.0
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
Examine the attached image. Plot mesh plot, add transform operator with scale by 3 in y and translate by 2 in y. Note the resulting missing negation sign on the y axis '5'
Comments:
Not a bug.


Upgrade our vcpkg usage

Summary

This PR updates our commit for vcpkg to its 2021.05.12 release: https://github.com/microsoft/vcpkg/tree/2021.05.12
It updates our local package ports to the newer "manifest format", which uses vcpkg.json files instead of CONTROL files. See https://vcpkg.readthedocs.io/en/latest/maintainers/manifest-files/
The newer vcpkg uses<EMAIL_ADDRESS>Luckily there was already a PR in conduit for this (https://github.com/LLNL/conduit/pull/727).  I converted this to a patch for vcpkg and then did some debugging.

I patched conduit's setup_deps to account for the vcpkg installation and it now passes our Windows TPL tests:
https://github.com/LLNL/axom/actions/runs/972579755


freeimage: support cross-compilation

Motivation for this change
Support cross-compilation for freeimage. Build for pkgsCross.aarch64-multiplatform works.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Can you please test execution of cross-compiled imv with this change on aarch64-linux?

Then you can mark "Tested execution of all binary files (usually in ./result/bin/)".

imv does not cross-compile, since some dependencies don't cross-compile properly yet (to name a few):

netsurf-libnsgif
llvm12
librsvg

But things are slowly coming together, giving us the ability to check how well these binaries work. For now, getting everything to compile is first priority for me.


Feat/411 5 get transaction details and get utxos

Requires firstly that #431 be merged.
Description:
Implements chronik blockchain client methods: getTransactionDetails and getUtxos
Test plan:

To test getUtxos, you can test the getBalance endpoint, since it uses the getUtxos method. Open, for example: http://localhost:3000/api/address/balance/ecash:qp0kfg0uk64k4pa7tw226djx4v30attzzslrec0ypu
To test getTransactionDetails, you need any transaction id (you can get it on the block explorer) and then open, for example: http://localhost:3000/api/transaction/bd56e6aaa8aa71e5239ec89fef8ddb7b91e9cf5a321102232967046f94ee0be9?networkSlug=ecash

Probably going to need to dig into the client to see what we need here.

Closing this one as it will be addressed after the stream of new smaller PRs originated from #431. Maybe after those, this one will also be broken into smaller ones.


Error downloading YouTube video with expiration feature

Checklist

[x] I'm reporting a broken site support issue
[X] I've verified that I'm running youtube-dl version 2021.12.17
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar bug reports including closed ones
[X] I've read bugs section in FAQ

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-4vC', '--retries', 'infinite', '--fragment-retries', 'infinite', '--user-agent', 'Mozilla/5.0 (Windows NT 10.0; rv:102.0) Gecko/20100101 Firefox/102.0', '--recode-video', 'mp4', '--audio-format', 'mp3', '--audio-quality', '0', '--embed-thumbnail', '-o', '%(title)s %(release_date)s', 'https://www.youtube.com/watch?v=eL5tVGEuPK8']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.9.2 (CPython) - Linux-6.1.0-1parrot1-amd64-x86_64-with-glibc2.31
[debug] exe versions: ffmpeg 4.3.6-0, ffprobe 4.3.6-0
[debug] Proxy map: {}
[debug] Public IP address: <IP_ADDRESS>
[youtube] eL5tVGEuPK8: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/dist-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/lib/python3.9/dist-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python3.9/dist-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python3.9/dist-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/usr/local/lib/python3.9/dist-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

It seems to be an error in the Python code regarding the uploader id extraction. I don't know Python, or I'd have fixed it myself and posted the solution here, so I don't know what might be relevant. The only thing that comes to mind is that this video has an expiration period feature enabled. I don't know how long this feature has been around on YouTube, though. The way I see it, adding an option for disabling the uploader id extraction might be a workaround. I don't know how to be of more help, I'm sorry for that.

#30839 !


Attempt to decompress GZip if it is suggested by headers

Rewind and retry standard read when gzip reading fails

it should be already supported by lib curl, try: downloader.accept_encoding = "gzip, deflate"


Backend - User-Profile-Logic - Self-Sprint-1

@askinss
"The task involves testing the profile creation and update functionality on the platform. The tests include verifying that users can successfully create and update their profile with a profile picture, about section, social media links, location, website, and join date. It also includes testing the correct storage and display of the profile picture, social links, location, website, and join date in the database. Additionally, the about section should be limited to a certain number of characters. The goal of the testing is to ensure that all profile elements are accurately stored and displayed, and that the user experience is smooth and error-free


Add Extract Component Function and Tests

For issue #332 Problem with ExtractComponent

Codecov Report

Merging #338 (6cf4f0b) into master (a0e7070) will increase coverage by 0.15%.
The diff coverage is 89.65%.

@@            Coverage Diff             @@
##           master     #338      +/-   ##
==========================================
+ Coverage   52.18%   52.33%   +0.15%
==========================================
  Files         125      125
  Lines       18151    18180      +29
==========================================
+ Hits         9472     9515      +43
+ Misses       8679     8665      -14

Impacted Files
Coverage Δ

python/mspasspy/algorithms/basic.py
95.08% <89.65%> (-4.92%)
:arrow_down:

python/mspasspy/db/database.py
79.56% <0.00%> (-0.06%)
:arrow_down:

python/mspasspy/util/logging_helper.py
57.77% <0.00%> (+11.11%)
:arrow_up:

cxx/src/lib/algorithms/seismogram_helpers.cc
42.45% <0.00%> (+12.26%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Bosch heat pump CS7001iAW 9 ORE: wrong energy consumption values

Hello,
Describe the bug
Home Assistant shows wrong energy consumption values. It looks like the value is constantly counting up in the same speed but that can clearly not be the case.
Screenshots

Version

HA version? 2022.7.2
HA Bosch component version? 0.18.1
Python Bosch HTTP client version? don't know

** Debug SCAN **
** IMPORTANT **
I don't know what uuid to put into the parameter. Where can i find this?
Thanks

Hi el,
i do have the same setup and hardware.
I start with the HA integration but can't connect till now.
What have you typed in for serial device / access token / password?
I tried several but nothing helped.
After connection I could have a look on the energy consumpion :P

After using it a week now, on my side it also doen't work.
"energy consumption" still shows 0 kWh while the water is heated every two days
@elbielefeld22 do you have a solution / work around?

Enable recording sensor if you have those. Disable energy consumption, many people don't have this sensor.
UUID is uuid of your thermostat. It can be found as "Hardware" in Device info.

Hello,
I enabled the energy consumption sensor but still have wrong values:

As you can see, the head pump was only running at the following time:

Here's the scan file
bosch_scan.json.txt


ngrxFormControlState set id attribute and make label break

Sorry for my previous inactivity about issue #84
I found that label for any input field doesn't work if that input has ngrxFormControlState.
By "doesn't work", I mean when clicking the label, the input field is not focused.
I kind of find the reason, the ngrxFormControlState will programmatically override id attribute of input, so the for attribute of label won't match the user-set id of the input.

Indeed, I will probably fix this behaviour by checking if the user already set an ID. In the meantime you can always just use the ID assigned by ngrx-forms in the label's for attribute, i.e. use something like this:
<label [for]="myGroup.controls.myControl.id">My Control</label>
<input type="test" [ngrxFormControlState]="myGroup.controls.myControl" />

Is it possible to not touch the native id but creating some private name like ngrx-forms-id?

Setting the id is meant purely as a convenience feature so that users don't have to come up with their own unique ID if ngrx-forms can already provide one, so renaming the property wouldn't work.
However, I have just published version 2.3.6 which contains a fix that prevents setting the id if it is already set otherwise. This should fix your issue. If not, please let me know here and I'll re-open this issue.


stream from stdin, instead of download.

Problem

Would be nice to have a stream option like @ritiek 's V2 had, was very useful not having to download first, for example: streaming Spotify via Termux when away from home (commuting to work etc).
Solution

V2 used commands like:
spotdl -s [SpotifyURL] -f - | mpv -

Is this still available? or has the syntax changed? if not please add this feature! thank you!

Would adding this feature really help the majority of spotDL users? - Be worth the time that would need to be put in?
What is preventing you from downloading the files?

Would adding this feature really help the majority of spotDL users? - Be worth the time that would need to be put in?
What is preventing you from downloading the files?

Also note that
spotDL's original author, ritiek attempts to maintain a v2 fork. See it at ritiek/spotify-downloader. I think its currently dysfunctional. It has no support from the spotDL team.

Also note that
spotDL's original author, ritiek attempts to maintain a v2 fork. See it at ritiek/spotify-downloader. I think its currently dysfunctional. It has no support from the spotDL team.

Yes, I do truly believe that this feature would benefit many other users! why wouldnt it? after all, Spotify is a streaming service itself, allowing you to access your music anywhere in the world, without downloading them all first, the same ability in spotdl would most likely benefit alot more people than just I.
Thank you for you consideration!

Yes, I do truly believe that this feature would benefit many other users! why wouldnt it? after all, Spotify is a streaming service itself, allowing you to access your music anywhere in the world, without downloading them all first, the same ability in spotdl would most likely benefit alot more people than just I.
Thank you for you consideration!

We had a brief discussion about this in the spotDL development team. Spotify is a streaming service, and spotDL is for quote

Download your Spotify playlists and songs along with album art and metadata (from YouTube if a match is found).

You can attempt to stream songs without downloading it yourself by using spotDL as a library, by making a PR with the changes or attempting to use ritiek's v2 fork (see above)

We had a brief discussion about this in the spotDL development team. Spotify is a streaming service, and spotDL is for quote

Download your Spotify playlists and songs along with album art and metadata (from YouTube if a match is found).

You can attempt to stream songs without downloading it yourself by using spotDL as a library, by making a PR with the changes or attempting to use ritiek's v2 fork (see above)

Hey @5c0tt-b0t. We aim to be the best at what we do and give the users the best we possibly can. The reason we decided to not implement this is because Spotify is an undeniably better streaming option and anything we called provide would only be second best and not even necessarily accurate.
If you would still like to implement this, you can in your personal fork, we'll link to it in the Readme.
cc:@Silverarmor

Hey @5c0tt-b0t. We aim to be the best at what we do and give the users the best we possibly can. The reason we decided to not implement this is because Spotify is an undeniably better streaming option and anything we called provide would only be second best and not even necessarily accurate.
If you would still like to implement this, you can in your personal fork, we'll link to it in the Readme.
cc:@Silverarmor


feat: Added Hot Water Accessory

Hot water control through the Tado platform is provided through the optional Tado Extension Kit (https://www.tado.com/gb/products/extension-kit). This is a device which is directly connected to the boiler instead of a Tado Smart Thermostat and is therefore powered through the mains. Along with enabling hot water control, the Extension Kit allows users to connect the Tado platform with wireless thermostats.
Hot water control is a simple ON/OFF relay switch and the Tado Extension Kit cannot control water temperature due to the hot water thermostat being a "dumb" device physically attached to the water tank which in turn is linked directly to the boiler.
The Tado API accepts a PUT onto the overlay endpoint to switch on hot water (with the Extension Kit expected to be at zone/0) and a DELETE to turn off.

@storeyleon Thanks for the PR. Can you explain how the hotwater device from Tado works? Does it report temperature, does it have different modes?

Added and enhanced with #15.


Build swift-corelibs-foundation as a package on top of swift-foundation

This contains the changes required to build swift-corelibs-foundation as a package on top of swift-foundation. It includes some @_spi interfaces for SCL-F, so it has access to internals which we generally don't want exposed to other projects.

@swift-ci test


No such file or directory: 'output/Conv_bn_moving_mean.dat'

I have this problem while using load_caffe_weights.py.
I have checked ssdlite_mobilenet_v2_coco_2018_05_09/frozen_inference_graph.pb and there is no tensor with 'Conv' and 'moving_mean' in the name. Does any one have similar problem?

It seems that tensorflow model has merge batchnorm  of Conv with convolution of Conv. FeatureExtractor/MobilenetV2/Conv/BatchNorm/batchnorm/mul_1 and FeatureExtractor/MobilenetV2/Conv/BatchNorm/batchnorm/add_1 are now Conv2D op and addtion op. Does this mean the load_caffe_weights.py needs write? @chuanqi305

How to solve the problem?

M


Rename all code to use gvf2_ prefix

and then copy meta-data into src folder and commit

prefix has been added to lightning bundles


[Zeppelin-2572] multiple paragraphs actions

What is this PR for?
This PR adds the ability to perform actions on several paragraphs simultaneously. He adds the checkbox to each paragraph. When you select at least one paragraph in the action bar, buttons appear that perform actions on the selected paragraphs.
What type of PR is it?
Feature
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2572
How should this be tested?
1. Select several paragraphs (In the actinBar appear buttons with actions for these paragraphs)
2. Perform several actions on paragraphs (by pressing the buttons)
3. Click "Reset selection" button
Screenshot

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

Ready to review

Ready to review

Ready to review

Ready to review

Just noticed this.. I think it's a great feature!
In Jupyter we can also select multiple paragraphs.
Although it is made more seamless.. I believe you could Ctrl+click anywhere on a cell to select it.
So no separate UI element was necessary (like a checkbox that you have).
Selected cells are just slightly highlighted then.
Two pros for this approach:

easier to select multiple cells as you could click anywhere on a cell to add it to a selection;
no additional UI elements
easier to find what cells were selected.

Will reopen soon.


Linter rule NoUnreferenced set true then *PSSR is marked with warning

See issue #170.
The same needs to be done for *PSSR.
Thanks.

@m-tyler Any chance you can provide a code sample of this? Thanks!

ISSUE_175.RPGLE.txt
FYI, this was with me being at version 0.16.0.  I tried updating to the latest version but I was having some issues so I downgraded.

Thanks, I will get a test and fix in for this ASAP.

Ready to be released.

This just went out!


Move Scheduler related spi util classes into schedulers package

As #1604 mentioned, this PR is trying to move some spi-classes into schedulers package.
There are five src and two test classes included totally:
c.t.h.spi.utils.Runtime
c.t.h.spi.utils.SchedulerConfig
c.t.h.spi.utils.Shutdown
c.t.h.spi.utils.LauncherUtils
c.t.h.spi.utils.SchedulerUtils

c.t.h.spi.utils.SchedulerUtils.LauncherUtilsTest
c.t.h.spi.utils.SchedulerUtils.SchedulerUtilsTest

The key changes are following:

move classes into c.t.h.schedulers.utils
modify c.t.h.schedulers.BUILD and modify c.t.h.scheduler-core.BUILD
modify related class invoke info

I've no idea about how to refactor SchedulerConfig class. So this PR only move it into schedulers package without any change. Let me know if there has anything I could do and any comment is welcomed.
cc @billonahill

Updated

move classes into schedulers-core
refactor SchedulerConfig class
add test_classes

@billonahill Please take a look again when you have a chance.

👍
Thanks for the cleanup!


Missing git tag 1.3.9 has pre-release versions out of sync

The git tag 1.3.9 is missing for that version. This is causing the pre-release builds for master to be lower than the release 1.3.9. Since I'm mostly looking to have PyYAML > 4.2b1 (https://nvd.nist.gov/vuln/detail/CVE-2017-18342) which has already been fixed on master, I would love to get a 1.3.10 (or 1.4.0?) release anyways.

Yeah my team and I recently faced a similar issue. Locking moto to version 1.3.8 fixed our PyYaml warnings. Strange 1.3.9 has this issue.

@spulec said that 1.3.9 was not an intended release https://github.com/spulec/moto/issues/640#issuecomment-507486727

Sorry about that. I'm still trying to get the release script working properly.
I've tagged 1.3.9 and released a full version of 1.3.10 as well.


Refactoring pyluna-core

Refactor src to pyluna-core
Move unused code to .archive (be visited regularly)
clean up README

Codecov Report

Merging #237 (0254619) into master (6c7f31a) will decrease coverage by 1.63%.
The diff coverage is 46.34%.

:exclamation: Current head 0254619 differs from pull request most recent head cf0bc2a. Consider uploading reports for the commit cf0bc2a to get more accurate results

@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
- Coverage   53.94%   52.31%   -1.64%
==========================================
  Files          77       79       +2
  Lines        5950     5794     -156
==========================================
- Hits         3210     3031     -179
- Misses       2740     2763      +23

Impacted Files
Coverage Δ

pyluna-core/__init__.py
0.00% <ø> (ø)

.../luna/api/radiologyPreprocessingLibrary/service.py
100.00% <ø> (ø)

pyluna-core/luna/clinical/proxy_table/generate.py
97.67% <ø> (ø)

...pathology_annotations/get_pathology_annotations.py
66.66% <ø> (ø)

pyluna-core/setup.py
0.00% <0.00%> (ø)

...luna-core/tests/external_process_patient_script.py
0.00% <ø> (ø)

...ts/luna/arc_test_data_ingestion_template_schema.py
0.00% <ø> (ø)

.../tests/luna/arc_test_dicom_to_delta_integration.py
0.00% <0.00%> (ø)

pyluna-core/tests/luna/arc_test_dicom_to_graph.py
0.00% <0.00%> (ø)

...yluna-core/tests/luna/arc_test_neo4j_connection.py
0.00% <0.00%> (ø)

... and 45 more

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 6c7f31a...cf0bc2a. Read the comment docs.


Make the filter box a standard UX pattern

Here's my proposal.

Dropdown uses secondary button style.
Go uses primary button style
No boxes selected, user activates Select/Deselect all: All boxes selected
User then deselects some boxes: Select/Deselect all box becomes unchecked
User then checks 'Select/Deselect All' again: All boxes checked
User will see what happens as soon as they activate the checkbox:

If no boxes selected, then all boxes are selected.
If some boxes selected, then all boxes are selected.
If all boxes are selected, then all boxes become deselected.

For the case where user manually checks/unchecks all the boxes, 'Select/Deselct all' control does not have to respond to the manual selection.

This description consolidates discussion from below.

I think it would violate the principal of least surprise if the person unselects the "select all" and all the everything get unselected. I think this is confusing because I would think that not having "select all" would be "select some" rather than "select none". Can you show me a place where this pattern is used elsewhere?

On the github issues page, there is a select all/none option. https://github.com/LD4P/sinopia_editor/issues

If there aren't that many options, then I'm ok w/not having the select all/none option, but if there are potentially more than 10, it would be nice to give users that select all/none option.

The github widget is not as simple as that. It has three states.  If you select all and then unselect some issues it switches to this state

:

@NancyL the top checkbox is Select/Deselct all, are you asking about a button somewhere else?
@jcoyne is enough design provided now?

works but see #3200


InvalidOperationException - TaskCompletionSource.SetException

Getting an exception of type System.InvalidOperationException: An attempt was made to transition a task to a final state when it had already completed.
This happened when iOS device was offline.
I'm using library version 2.1.7
Stack trace from Hockey App -

System.Threading.Tasks.TaskCompletionSource.SetException(Exception exception)
FFImageLoading.TaskParameterExtensions.<>c__DisplayClass11_0.b__0(Exception ex)
FFImageLoading.Work.ImageLoaderTaskBase.<>c__DisplayClass51_2.b__3()
CoreFoundation.DispatchQueue.static_dispatcher_to_managed(IntPtr context)
CoreFoundation.DispatchQueue.static_dispatcher_to_managed(intptr)(wrapper native-to-managed)
UIKit.UIApplication.UIApplicationMain(int, string[], intptr, intptr)(wrapper managed-to-native)
UIKit.UIApplication.Main(string[] args, string principalClassName, string delegateClassName)
iOS.Application.Main(string[] args)

Hi @keshavkaul  2.1.7 is quite old release. It's already fixed now.

@daniel-luberda Thanks, which version has the fix?

I don't remember but I would suggest using the newest one, it has a better performance.


Added queue repeat functionality

Added a new Repeat option to enable queue looping.
!repeat command toggles it on and off
!queue now shows the status of the repeat option

Is this a current feature? I'm trying to figure out how to loop a youtube playlist without autoplayist (can't be random)

@cordlessvii No. This is not currently implemented.

@mattbsg thank you for the quick response. Is there anything i can do to have a feature like this? The auto playlist works but it randomizes the list. Is there a way I can loop a playlist or set the autoplaylist to not random? Thanks again

@cordlessvii We can only ship features that are actually made obviously, so feel free to send a a PR with updates, but if you don’t know how, there is a feature in the review branch to play the autoplaylist in order. So you use the review branch or wait until the next version release depending on how much of importance this is to you (make sure to reconfigure the bot from scratch if you use review so you get the updated options.ini file).

@MattBSG I downloaded the review branch and reconfigured the options.ini as you mentioned. I saw the option for AutoplaylistRandom and marked it as no, but it still plays random. Any ideas or something I'm doing wrong? I'm hosting this on heroku so I had to copy over the runtime.txt and Procfile from the master branch.
Also, I'm able to run the review branch bot locally on my computer (still plays autoplaylist random though) and when I try to host it on heroku I get the error message below, even though I did install it through Git.
2018-02-10T15:08:22.529417+00:00 app[worker.1]: [CRITICAL] launcher: Bot wasn't installed using Git. Reinstall using http://bit.ly/musicbotdocs.
2018-02-10T15:08:22.529595+00:00 app[worker.1]: Press enter to continue . . .Traceback (most recent call last):
2018-02-10T15:08:22.529601+00:00 app[worker.1]:   File "run.py", line 421, in
2018-02-10T15:08:22.529917+00:00 app[worker.1]:     main()
2018-02-10T15:08:22.529923+00:00 app[worker.1]:   File "run.py", line 332, in main
2018-02-10T15:08:22.530212+00:00 app[worker.1]:     sanity_checks()
2018-02-10T15:08:22.530214+00:00 app[worker.1]:   File "run.py", line 198, in sanity_checks
2018-02-10T15:08:22.530328+00:00 app[worker.1]:     req_ensure_env()
2018-02-10T15:08:22.530334+00:00 app[worker.1]:   File "run.py", line 282, in req_ensure_env
2018-02-10T15:08:22.530511+00:00 app[worker.1]:     bugger_off()
2018-02-10T15:08:22.530517+00:00 app[worker.1]:   File "run.py", line 182, in bugger_off
2018-02-10T15:08:22.530694+00:00 app[worker.1]:     input(msg)
2018-02-10T15:08:22.530729+00:00 app[worker.1]: EOFError: EOF when reading a line

@cordlessvii Heroku won’t work with the bot well, that message about the environment check is also because you need to download with git not from the web view. If you need more support hop on into our discord server and give me a ping today and I’ll pop in to help. https://discord.gg/bots

@MattBSG thanks man, I'm in the server now. What's your name on there or you can message me @cordlessvii#3722

Closing due to inactivity.


Change rpc_server::rpcs::common::ErrorData to also include HighestBlockHeight(u64)

Change the get_lowest_contiguous_block_height_from_storage to get_highest_contiguous_range_from_storage and update as necessary in storage to provide the lowest contiguous block height and the highest block height in a single responde

The error response will now look like this:
{
	"jsonrpc": "2.0",
	"id": "1",
	"error": {
		"code": -32001,
		"message": "block at height 200 not stored on this node",
		"data": {
			"highest_contiguous_block_height_range": [3, 93]
		}
	}
}


Remove version

It's not necessary

Optional if the package repository can infer the version from somewhere, such as the VCS tag name in the VCS repository. In that case it is also recommended to omit it.

https://getcomposer.org/doc/04-schema.md#version

Thanks.
For information, I will create a new PHP 8 compatible release


country data JSON lack data...

Need to start adding data into the country JSON files so it will appear on each country screen. Need to keep it as organized and formatted as possible

fixed in 771df40481e832820884c169b01d09d67a1d2d6b Deleted files


feat: Nitro speed up for 1st inference time after model loaded

Problem
The first request to nitro web server is slow, which make me frustrated. When it's ready, pls make sure me as a user can get quick result.
Success Criteria

1st request from user should be quick
You should do mock request after loading model for the first time
The /health should yield 500 indicating the model warm up process is not done yet, else 200 if done. The case for process exit has been handled

Additional context
None atm

I second this feature. From the app side, if a model is "started", we expect the models to respond lightening fast.


Can't create buckets associated to a device resource when device id contains dashes (-)

Describe the bug
When creating a bucket and configuring the data source to a device resource, if the device id contains dashes (-) a 400 bad request response is given as the schema fails to validate.
Expected behavior
Bucket should be created
Additional context
https://community.thinger.io/t/cant-create-buckets/4817/3

This bug is also present for Alarm creation - I received a 400 error while attempting to create alarms and spent 30 minutes trying to debug it until I found this issue.

Fixed on the upcoming 5.4 version.


Only records the bottom half of screen when set to a 9:16 ratio

When recording to a 9:16 ratio, it bugs out the top half of the recording and only records the bottom half of the screen. I was using the mod blockbuster while I was recording with minema.

Here is the result
https://user-images.githubusercontent.com/103904665/170915698-7a6aba40-51fc-457c-b2cf-6f6a8ffe774f.mp4

I also tried to record this without using the button in blockbuster and it recorded the same way
https://user-images.githubusercontent.com/103904665/171032791-2cdb9ede-1cf0-4683-a80d-b80dee307e0b.mp4


Use r4i4 ncep_sp in gmao_transf

Use this r4i4 version of ncesp_sp in gmao_transf. This is needed so that the GEOSgcm.x will link to this version allowing the JCAP functionality to work again in mkiau gridcomp as that needs the r4 version. Note this is non-zero diff if using stochastic phyiscs (SPPT: 1 and SKEP: 1). However, I would be suspicious of any answer from that as if you turn on that option with debugging it fails in the stochastic physics with an out of bounds error.

I added the non-0-diff label, though...I guess it's hard to label "Allows code to work" as being 0-diff or non-0-diff...so I'll add 0-diff as well. Hmm.


Improved build performance for repeated builds

Description
Some Gradle tasks were configured to prevent Gradle from using up-to-date checks (and build caching, if enabled). This meant that some tasks were being executed even if the output was not different, wasting build time.
This PR proposes changes to the build scripts to make the builds more efficient for repeated builds on the same machine. This should save a few seconds, with more improvements if build caching were to be enabled.
Some key highlights:

tasks that share output locations (eg. tasks that downloaded files directly into build/classes) would prevent Gradle from identifying the owning tasks, which disables caching/up-to-date checks. These were split into separate locations and left to the processResources/jar tasks to collect. For some modules, the file was placed directly into the source, and the version property was injected to ensure repeatability in the build  - feedback is welcome on this change.

tasks that do not declare inputs/outputs or have no outputs also disable the checks. These tasks were updated to include the right inputs/outputs. In the case of the verifyJava task, a new output file was created to let Gradle know when the task was completed successfully previously. If inputs do not change, Gradle will automatically skip these tasks, as the output would have remained the same.

absolute paths were changed to relative paths (or normalized to only consider relative paths). If remote build caches are enabled, the build cache will be used for these tasks across any connected build machine.

The improvements were discovered through analyzing the build with Develocity. This Build Scan is of a second build run after the improvements were implemented.
The timeline view shows a breakdown of where time was spent in the build.  In this case, as it was performed on a machine that had already run the build task, you can see here that no tasks were executed as they were all up to date.

@runningcode @ribafish does the PR description make sense?
If there are no objections (I think I addressed all the comments), I'd like to propose the PR to the original repo

Merged upstream! Closing this PR


adding articles to ezt for blogs migration

This is needed for https://github.com/apache/infrastructure-website/pull/70 to work and not break builds. I tested locally and if a site does not have ../content/articles the build continues to work.


Fix pickling for BoundRecArrays

Fixes #27.
BoundRecArrays were losing their attributes when pickled. This can be fixed by overriding their __reduce__ and __setstate__ methods. I've basically just adopted a generalized version of this stackoverflow answer.
I had some trouble with the python 3.6 travis build, but fixed it by setting numpy>=1.14.0 in the requirements. With a lower version of numpy, un-pickling a BoundRecArray throws the error:
ValueError: Invalid data-type size.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #28   +/-   ##
=======================================
  Coverage   66.02%   66.02%
=======================================
  Files           8        8
  Lines         571      571
=======================================
  Hits          377      377
  Misses        194      194

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 d9727ca...21fc2e6. Read the comment docs.

Oh damn, why did I forget about this... Thank you very much!


reduce number of ss3 compile runs in github actions

The github test runs for every push to the stock_synthesis code repo has grown to >10 and takes nearly an hour.  Would it be possible to reduce this to 3 (compile for windows, compile for unix, run using existing parameters).  Then invoke the remaining actions in a manually triggered request?

Feel free to reopen if more changes are desired, Rick!


AuthorizationCallback is wrong format

@austeja-bentley @calebmshafer
The AuthorizationCallback should probably be renamed to AccessTokenCallback, and return a Promise that contains the token in the same format as IModelApp.getAccessToken(). The rest of the iTwin API has changed over to this string-only format for token handling, instead of authorization.
Right now, in order to make this work with IModelApp.getAccessToken(), a user has to strip the "Bearer " prefix off of the accessToken value, then build an Authorization object out of the resulting string and "Bearer".

@tcobbs-bentley
The name AuthorizationCallback was chosen deliberately as iModels API can and will eventually support more ways to authenticate that with a Bearer access token. This client will also have to work with iTwin Stack so the structured callback result allows for better extensibility. The @itwin/imodels-client-management and @itwin/imodels-client-authoring packages are of course a part of the platform but they are intended to be used in more places than applications built using iTwin.js libraries so their interfaces were designed having that in mind.
What applications can do is build an adapter to their dependencies. The @itwin/imodels-access-frontend and @itwin/imodels-access-backend act as exactly that - adapters that bridge the iModels API client interface and the iTwin.js core libraries and they do indeed have that bit of code that parses the access token into scheme and token. I think it is pretty common to have dependencies that have an interface that does not match your needs exactly but as long as conforming to them does not cause too much trouble I feel like it's fine.
Does your application use the IModelApp class and also iModels API client directly (not via the @itwin/imodels-access-frontend and @itwin/imodels-access-backend packages)?

@austeja-bentley My app is actually a mobile sample for users to look at to see how to use iTwin on a mobile device (using mobile-specific packages to make that work). And yes, it uses IModelApp. Given that @itwin/imodels-client-management will likely be used a lot by people who are also using IModelApp, it seems to me that it should make it easier to make use of the token string returned by IModelApp.getAccessToken(). Right now, in order to use it, it has to strip off the "Bearer " prefix that is part of that string.
@calebmshafer What are your thoughts on this?

I tend to agree with @tcobbs-bentley, as I just hit the same thing he did when updating the itwinjs-core repo to use the new clients.
The motivation behind moving towards a single string in the iTwin.js core code is to be agnostic to the type of auth that people would like to use and support. Most common workflows (OAuth 2.0, Basic and API key) can all be handled via a single string to interpret them, helping to reduce complication on the consumer of the API. It would be up to that consumer to use the appropriate implementation of the AuthorizationClient interface (for IModelApp/IModelHost) that supports the workflow needed, as @tcobbs-bentley mentions that interface only has a single method that returns only a string.
Now, I understand that the imodels client libraries has different requirements since it has to actually construct the requests, either by adding it to the auth header or as a query param in the case of API key. However, in all cases I think it makes sense for the @itwin/imodels-client-management to make it as simple as it is in the Frontend/Backend Hub Access clients are for the main cases we want to support.
Does that sound like it would work? @austeja-bentley @Animagne @robertas-kerpys

@tcobbs-bentley @calebmshafer
This PR was merged to help applications that rely on IModelHost/IModelApp to communicate with the plain IModelsClient. With the conversion functions exposed calls to the plain client would look something like this:
const accessToken = await IModelHost.getAccessToken();
const briefcase: Briefcase = await backendIModelsAccess.IModelsClient.briefcases.acquire({
  authorization: AccessTokenAdapter.toAuthorizationCallback(accessToken),
  iModelId: "some id"
});

With this solution client applications do not have to implement the conversion functions themselves and IModelsClient interfaces remain extensible.
Please let me know if there are any concerns regarding this.


formatter removes significant comma, leading to non-compilable code

Running crystal tool format on
record Foo,
  bar : String, #
 do
  def baz
  end
end

gets formatted to illegal code because the comma gets removed
record Foo,
  bar : String #
 do
  def baz
  end
end

Now it gets rejected by the compiler:
$ crystal run example.cr
In example.cr:3:2

 3 | do
     ^
Error: expecting token 'EOF', not 'do'

$ crystal -v
Crystal 0.35.1 (2020-06-21)

LLVM: 10.0.0
Default target: x86_64-pc-linux-gnu

I think this is more an issue with having the do keyword on the next line.
https://play.crystal-lang.org/#/r/9qab
1.times
do
  puts "hi"
end

syntax error in eval:2
Error: expecting token 'EOF', not 'do'

Ofc the solution/more idiomatic way would be to just not do that.  But it would be nice if the formatter  could correct that for you.

Just verified that the problems is reproducible on master, too:
Crystal 1.0.0-dev [135e9fbc3] (2020-09-18)

LLVM: 10.0.1
Default target: x86_64-pc-linux-gnu

@asterite Thank you, just tried with the PR #9762 . That solves it. :+1:


Multipath routing integration

Multipath routing implementation for Floodlight. It implements getRoutes() method from IRoutingService by computing all paths with DFS algorithm. Another method (getMultipath()) has been added in case of using multipath routing from Forwarding module. Two additional parameters in configuration file has been added: "useMultipathRouting" and "numberOfMultipaths".  Unit tests have been added too.

@osinstom thanks! do all the unit tests pass on your local repo? If not, could you take a look at the tests in error? Looks like a few are in error. Fortunately, errors are typically (from my experience) related to unit test structure and are easier to resolve than failures.

@osinstom figured it out. Try and give the numberOfMultipaths parameter a default value (e.g. 3) when you define the global variable. Then, in init(), do something like this:
String numMps = context.getConfigParams(this).get("numberOfMultipaths");         if (numMps != null) {             try {                 numberOfMultipaths = Integer.parseInt(numMps.trim());             } catch (NumberFormatException e) {                 log.error("Could not parse numberOfMultipaths {}. Using default {}",                         numMps, numberOfMultipaths);             }         }

@osinstom any updates here? I have a PR outstanding on your fork. If that can be accepted and merged the checks will restart here

@osinstom I've incorporated your work into a different pull request https://github.com/floodlight/floodlight/pull/683, since we need this in master quickly. I'll close this for now, but feel free to open a new PR if you have any features you'd like to add. Thanks!


The design might be built slightly wrong

Matching should allow people to affect the whole amount matched, by any base amount.  So if someone wants to match a match they should be able to contribute any amount they want, and allow that to effect the total amount of the match.  Maybe.  Unless its a hardline app that only allows match base amounts set by the creator.  otherwise a new match would need to be made.  Maybe in that case tags and comments would be good.

Maybe after clicking "match that" it allows you to select between matching the amount already matched, or the base amount - or a third option - custom amount.


Can't resolve localhost in selenium on linux

Hi,
I have very simple test in sprinng boot 2.2.5 with testcontainers 1.13.0 and selenium 3.141.59.
It looks like this:
@RunWith(SpringRunner.class)
@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
@AutoConfigureTestDatabase(replace = AutoConfigureTestDatabase.Replace.NONE)
@ActiveProfiles("integration-test")
public class LoginPageIT {

@LocalServerPort
private int port;

@Rule
public BrowserWebDriverContainer chrome = new BrowserWebDriverContainer()
        .withRecordingMode(BrowserWebDriverContainer.VncRecordingMode.RECORD_ALL, new File("./target/"))
        .withCapabilities(new ChromeOptions());

@Test
public void shouldShowLoginPageAndSuccessfullyLoginToAdminAccountUsingAdminCredentials() {
    RemoteWebDriver driver = chrome.getWebDriver();
    String url = "http://host.docker.internal:" + port + "/";

    driver.get(url);

    WebElement elementById = driver.findElementById("login-form");
    assertNotNull(elementById);
}

}
On windows 10 it works as expected but on linux (ubuntu 18.0.4) it doesn't work. When I look at video for failed test I see that it's trying to reach this url "data:,". Is that a known issue?

host.docker.internal is not cross platform. Consider using https://www.testcontainers.org/features/networking/#exposing-host-ports-to-the-container
You will find an example here:
https://github.com/testcontainers/testcontainers-java/blob/3f9216ebda2087b3311de979a96918e3ccaefc2d/examples/selenium-container/src/test/java/SeleniumContainerTest.java#L31


Ability to pull rancher images from a self-hosted registry

Our hosts are having connection problems with Docker Hub because of the special Internet limitations in China. To improve the availability of the service, we are planning to clone rancher images from Docker Hub so hosts can pull them from our own registry. If we could set a default registry of racncher images, this can be solved.

You can read about how to launch Rancher server without internet here:
http://docs.rancher.com/rancher/installing-rancher/installing-server/no-internet-access/
Please note that right now we don't have the ability to use the catalog without internet access.

You can add/config your own registries in Rancher already;
Read more here: http://docs.rancher.com/rancher/configuration/registries/

@wstam88  @deniseschannon  can you help with my private registry secured trusted ssl cert (docker login https://myregistry.domain.com works fine on the host.
I added my registry in web gui together with necessary auth.
When I try to use image from my registry in agent logs I get:
2016-02-21 20:13:35,799 ERROR agent [139842942469456] [event.py:112] Error in request : ccdafca3-1cec-425f-b98f-144d97b722f2
Traceback (most recent call last):
  File "/var/lib/cattle/pyagent/cattle/agent/event.py", line 95, in _worker_main
    resp = agent.execute(req)
  File "/var/lib/cattle/pyagent/cattle/agent/__init__.py", line 15, in execute
    return self._router.route(req)
  File "/var/lib/cattle/pyagent/cattle/plugins/core/event_router.py", line 13, in route
    resp = handler.execute(req)
  File "/var/lib/cattle/pyagent/cattle/agent/handler.py", line 34, in execute
    return method(req=req, **req.data.__dict__)
  File "/var/lib/cattle/pyagent/cattle/storage/__init__.py", line 27, in image_activate
    action=lambda: self._do_image_activate(image, storage_pool,
  File "/var/lib/cattle/pyagent/cattle/agent/handler.py", line 72, in _do
    action()
  File "/var/lib/cattle/pyagent/cattle/storage/__init__.py", line 28, in <lambda>
    progress)
  File "/var/lib/cattle/pyagent/cattle/plugins/docker/storage.py", line 160, in _do_image_activate
    raise e
ImageValidationError: Image [myregistry.domain.com/app/web:release-1-t5] failed to pull: Authentication is required.


dart2js: specialize method bodies to arguments if known

consider this example program: (dead code added to the Foo constructor to make sure it isn't inlined)
void main() {
  new Foo(false);
}

class Foo {
  Foo(x) {
    if (x) {
      print('a');
    } else {
      print('b');
    }
    int i = 1;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
    i = i;
  }
}

The compiler should know that the only caller of the Foo constructor passes 'false' as the argument and specialize the body to eliminate the true branch of the if-statement.

This is related to #24355 which together with this issue is causing dart2js not to remove unused code

Closed with 28a803d3435c6565227ad31cb5db6cbe2173ca25


feat/author-flag

Add author flag for matching on an author, useful for larger repositories where multiple people are working on separate things to filter out work you have not done.

Rather than pulling this from a command line arg, you could pull this from
git --get user.name
Or
git --get user.email

I was considering doing something like that, but then I didn't want to restrict it to only being the current user.
Having it as a simple strings.Contains feels like it is a bit more flexible, if for whatever reason you want to also search for someone else. You then don't need to do git config user.name <new_person> each time, granted that might be pretty rare.
I could be wrong here of course, but I felt like this was a nice approach to it :blush:
Definitely a good suggestion though :rocket:


implement the get session logs endpoint for backend

Fixes #3033
This PR implements the get session logs endpoint for the backend.
Signed-off-by: Chanwit Kaewkasi<EMAIL_ADDRESS>

Tested with curl

Payload example:
  --data-raw '{"sessionId": "run-master-8615cb75-dirty", "token":"", "clusterName": "Default", "namespace":"flux-system"}'

Token must be "" to start retrieving from the first log entry.
Then you keep the nextToken value and pass it to the next call to retrieve the rest.
namespace is the namespace of the dev-bucket source from which we obtain the bucket information.


Reject/Decline Post

UX Could be improved by implementing two new features:

Auto reject submission based off new parameter defined during thread/channel initiation, "Declined Vote Threshold"
Manual button to reject/decline submission by specific roles. Mostly used for when posts are being submitted that wildly miss the mark for PoW.

The decline functionality should unpin the post, delete the bot's message

Auto reject submission

You mean to add a timeout and after its expiration the message would be unpinned?

[ ] Auto reject submission based off new parameter defined during thread/channel initiation, "Declined Vote Threshold"
[ ] Manual button to reject/decline submission by specific roles. Mostly used for when posts are being submitted that wildly miss the mark for PoW.


[TC-PCC-2.3]Control Mode Attribute not supported in yaml framework.

Problem
[TC-PCC-2.3] Primary functionality with server as DUT, step3a requires us to write attribute ControlMode [Pump Configuration and Control] which does not seem to be supported currently.
![image](https://user-images.githubusercontent.com/12559578/138837641-32f9d46d-17f5-481c-9930-cbd449001743.png
PCC_2_3.txt
)

Doesn't this just mean the ControlMode isn't implemented?

Based on https://github.com/project-chip/connectedhomeip/blob/master/src/app/tests/suites/certification/Test_TC_PCC_2_3.yaml#L40 it seems to work now.


feat: first iteration on multipleChoiceList

This PR is not the complete feature, mostly done so that @pixelastic can start hacking on the right code (some refactor was done)

Thanks @vvo


Unecessary complex code

Try to simplify this code, there is no need to cache the oldHash value
https://github.com/TimoVerkroost/minor-web-app-from-scratch/blob/9bc8c36b8c68ceb1fdd9ab81e69a127dfe7c013c/opdracht-5-spa/assets/js/app.js#L14

Thanks I think i fixed it. The code is much shorter.


👀 iPhone X support

Before:

After:

@peduarte it looks like the increased width makes it display at a different breakpoint. An unintended side-effect of changing the viewport setting 🤔

@lukemartin thanks man. What do you think? Looking at the screen shot, the latter looks a bit weird - it has lost balance and looks disproportional. Probably related to your comment above.

@peduarte I think they both look weird :s

Need to change the bounds of a medium width site here: https://github.com/fathomlondon/fathomlondon.github.io/blob/develop/_sass/_variables.scss#L2
My suggestion is maybe also add a min-height to media queries to capture devices in this orientation

Made @ibrennan's suggested change to the breakpoints which improves the balance a lot:

👍


/me action shows up in active room rather than room it was posted in

if a UserA is in Room1 and Room2 and UserB types a /me message in one Room1 and UserA is focused in Room2 the /me action will show up in Room2 and not Room1 where it should be

Fixed in 1bc428a


Configurator should automatically modify kf job manifest with volume mounts and env vars

the configurator need to provide volume information and environment variables to the kubeflow jobs, this can be done through modifying the generated manifest.

/priority p1

/assign


Vector editing with WFS-T

Capturing Brent Fraser's fixes for WFS-T editing here so it doesn't get lost:
https://lists.osgeo.org/pipermail/geomoose-users/2016-March/005945.html

Add controls to the Census Cities layer in mapbook.xml:

<layer title="Census Cities"  activate="true" src="census_cities" fade="false" unfade="false" metadata="false" legend="false" draw-point="false" draw-line="false" draw-polygon="true" edit-attributes="true" edit-shape="true" remove-feature="true" remove-all-features="true" buffer="true"></layer>

Add  "auto:true" to the save strategy in GeoMoose's WFS.js (line 36):

this.save_strategy = new OpenLayers.Strategy.Save(*{auto:true}*);

this means that when you unselect the feature after editing the changes will be automatically saved

Also in WFS.js, change line 44 to be:

projection:*this.srsName*, // new
OpenLayers.Projection(CONFIGURATION.projection),

In the mapbook add srs="EPSG:4269" to the mapsource:

<map-source name="census_cities" type="wfs" active="true" *srs="EPSG:4269"*>

this causes OpenLayers to re-project the feature's geometry before passing it to TinyOWS (I don't think TinyOWS does re-projection).

In the TinyOWS config.xml file, set:

*check_schema="0*"

in the TinyOWS config.xml file.  There are some schema problems I need to track down, but this works for now.

When you create the table in PostGIS using ogr2ogr, make sure it creates it with geometry type of POLYGON by including "-nlt POLYGON", otherwise the type is MULTIPOLYGON and PostGIS complains when TinyOWS tries to insert a POLYGON.

Regarding schema problems, I faced that too with the MS4W demo; to solve that I had to copy the 'schema' folder from the tinyows source into /ms4w/apps/tinyows/, so I bet that you would also have to copy that same folder from /ms4w/apps/tinyows/schema/ to inside your GeoMOOSE application directory, or, as you said, set check_schema to 0.

Fix successfully tested


Keep log output path on v1+

Description
** READ BEFORE CONTINUING: If your issue is not specific to AWS, please cut a ticket in github.com/kubernetes-sigs/karpenter.
What problem are you trying to solve?
Reference: https://github.com/aws/karpenter-provider-aws/issues/5288#issuecomment-1855229201

logConfig is currently deprecated and will be removed in v1.
We run Karpenter on Fargate.
To forward logs on Fargate, you can create an logging configMap to use AWS' Fluent Bit log forwarder
Fluent Bit does not support Loki yet (https://github.com/aws/containers-roadmap/issues/1242)
We run Promtail as a sidecar instead to forward logs to Loki
Promtail reads logs from the filesystem.
If logConfig is removed, we can no longer configure log output to the filesystem

How important is this feature to you?
Logging is important for us to see how Karpenter is working and the decisions it has made when debugging.

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Promtail reads logs from the filesystem

Can you use the var/log/pods path on the host to grab logs from the filesystem? Stdout and stderr logs should go over to this filepath AFAIK.

Looking at this deeper, I'm realizing that using Fargate would mean that you can't grab logs from the hostpath here, which is why you are indicating that you can't just use the standard stdout/stderr write-streams. I need to look a bit deeper into how Fargate expects that you scrape the logs.
This seems crazy to me that every component would have to support a separate volume output path for this to work. @calvinbui How do you handle this with other components that don't support output paths?

How do you handle this with other components that don't support output paths?

We use Fargate to avoid having to spin up a node at the beginning in our cluster just to run Karpenter. All our nodes are created and managed by Karpenter afterwards.

We use Fargate to avoid having to spin up a node at the beginning in our cluster just to run Karpenter. All our nodes are created and managed by Karpenter afterwards

Right, but I was asking more generally about how other components surface logs when it comes to fargate. I know that most kubernetes controller components don't surface output paths through their environment variable/CLI configuration so (for these components), if there's no way to access the hostPath on the node, I'd be interested to see how other components do log forwarding on Fargate.

We use Fargate to avoid having to spin up a node at the beginning in our cluster just to run Karpenter. All our nodes are created and managed by Karpenter afterwards

Right, but I was asking more generally about how other components surface logs when it comes to fargate. I know that most kubernetes controller components don't surface output paths through their environment variable/CLI configuration so (for these components), if there's no way to access the hostPath on the node, I'd be interested to see how other components do log forwarding on Fargate. The callout here is that I'd be surprised if this problem is unique to Karpenter.

For non-fargate hosts/pods, they have promtail, grafana agent or filebeat running as a daemonset to forward logs to their respective servers.

Got it. I'll pull-in some Fargate people into this conversation. I'm hoping there's a workaround to this problem outside of having to configure custom outputPaths in the logger.

We are also looking at this currently.  Karpenter is spun out on Fargate and then our EKS cluster nodes are handled from there.  We already have a logging stack using fluentbit and elastic running on our kubernetes clusters so would prefer to make use of those rather than breakout in to cloudwatch.  I was hoping to make use of the logconfig options to output to a file we can then scrape via a sidecar and push in to elastic running in cluster.

I was hoping to make use of the logconfig options to output to a file we can then scrape via a sidecar and push in to elastic running in cluster.

+ 1 native fluent-bit in fargate does not support loki, so I have to write to file and deploy flb as sidecar container.

Looking at this, are you able to share some more of your configuration around Loki? For example, are errors and output sent to the same filepath?

Missed the call for configs but we use the zap-logger-config to set the logs to different files and the point fluent-bit at those files to tail in to loki
apiVersion: v1 data:   loglevel.controller: info   loglevel.webhook: error   zap-logger-config: |     {       "level": "info",       "development": false,       "disableStacktrace": true,       "disableCaller": true,       "sampling": {         "initial": 100,         "thereafter": 100       },       "outputPaths": ["stdout", "/tmp/output.log"],       "errorOutputPaths": ["stderr", "/tmp/error.log"],

Hi @jonathan-innis @timbgn @rschalo
iam running karpenter controller in bottlerocket os , I am trying to forward the karpenter controller logs to graylog elastic search using filebeat.
if i use volume mount with host path getting below error.
Error: failed to generate container "4580c2878f037c731ae6df75dfd2fea0ca917ee252d913379d05546553fdea53" spec: failed to apply OCI options: failed to mkdir "/.bottlerocket/rootfs/var/log/pods/": mkdir /.bottlerocket: read-only file system
if i use "shareProcessNamespace: true" , iam getting  Permission denied
/proc/7/root/proc/34/task/40/ns': Permission denied
Please share suggestion implement fileabeat to send karpenter controller logs to elastic search.

We also depend on logOutput and errorLogOutput to send logs to a file that is picked up by a sidecar to ship the logs.


enable download of Molecular function substrates

final items from
#576
actually  this has never been asked for (physical interactions has). So let's not do this at least at present.
more like people would want to "send the MF targets to the query builder.
You can do this already for physical interactors. There is a link
View interactions for cdc2 at BioGRID ...
Query using interacting genes ...
perhaps we could add a link
Query using XXX substrates genes ...
So for cdc2 this would put all of te cdc2 targets into the query builder
https://www.pombase.org/gene/SPBC11B10.09
cdc25,clp1,
crb2, sds23
bit61, cdc15, cdc23, crf1, csi1, dbl7, dfp1, dnt1, drc1, fkh2, ies4, iss10, lem2, mdb1, mis4, msh6, mvp1, orc1, rad2, sec16, sgo2, sld3, snt1, ulp2 during mitotic G2 phase
  | has substrate abp1, adn2, alp14, ani1, ase1, ask1, atf1, bir1, bit61, bpb1, bqt4, brf1, bud6, bun107, byr4, ccq1, cdc15, cdc23, cdc25, clp1, cmk2, cnt5, cph1, crf1, csi1, cut3, cyk3, dbl7, dbp10, dfp1, dis1, dnt1,drc1, ebs1, elo2, epe1, fkh2, gef1, hal4, hip1, hmo1, ies2, ies4, inp53, iss10, kin1, klp9, kms2, lem2, lnp1, mac1, mcm4, mdb1, mde4, mgs1, mis4, mkh1, mmb1, mmi1, mod5, mpd2, msh6, mtl2, mto1, mug176, mvp1,nrm1, nsk1, nup124, nup146, nup40, nup60, nur1, orc1, orc2, pan3, pch1, pdr16, pic1, plo1, pob1, pof1, pom1, pom2, ppc89, ppk30, pre6, prp3, psy2, puf6, pxl1, rad2, rad4, rad52, rga2, rga3, rga4, rgf3, rif1, rod1,sad1, sce3, scw1, sec16, sep1, sfr1, sgo2, shd1, sid2, sld3, slf1, snt1, sog2, spo12, spo15, srp40, sup35, swi10, tam10, tea2, tfg1, tmf1, ulp2, utp502, vps5, vts1, wis1, wis4, wpl1, SPAC24B11.07c, SPAC24H6.08,SPAC9G1.07, SPBC13E7.07, SPBC17A3.05c, SPBC17D1.05, SPBC428.10, SPBC9B6.03, SPBC9B6.11c, SPBP8B7.26, SPBPB7E8.02

This is related to the 'feature' that the Nurse lab wanted (it would be useful to get lists of substrates, or targets).
I will put this on the discuss list to chat about.

perhaps we could add a link
Query using XXX substrates genes ...

That would probably be quite quick.
But it sounds quite all or nothing.  Do we need any download options so you could get, for example, just the targets of one MF term?

I don't think that is necessary, but mainly because I don;t know of a gene with targets  with 2 different molecular functions.
Obviously cdc2 has
cyclin-dependent protein kinase activity
cyclin-dependent protein serine/threonine kinase activity
protein kinase activity
protein serine/threonine kinase activity
protein serine/threonine/tyrosine kinase activity
but these are all essentially describing the same activity.

I've added a "Query using substrate genes ..." link on my desktop version:
https://desktop.kmr.nz/gene/SPBC11B10.09
It will appear in the dev site tomorrow but is currently not enabled on the main site.
I'm not sure about the "Query using substrate genes ..." link text.  I wonder if something like "Send substrate genes to the advanced search" would be better?  That would be closer to what we have in the motif search: "Send gene list to advanced search".

I haven't implemented that yet but I don't think it would take long.

This is great! We definitely want to be able to filter by phase too.
I was also wondering ( you have probably told me, but why is the phase filter only available under the full view since the phases themselves are visible in the summary view?

Maybe
Send 'substrate' gene list to advanced search
to be clear?

https://github.com/pombase/website/issues/1887
is the same feature, but is more extensive (to be able to do 'target_of' lists too)

The download substrates link is now in the dev site: http://dev.pombase.kmr.nz/gene/SPBC336.12c

We definitely want to be able to filter by phase too.

I'll work on that after the paper is submitted.  It will probably only take half a day.

We definitely want to be able to filter by phase too.

Should there be an option to download the targets that don't have an annotated phase?

I think for now, all targets or specific phase targets might be enough
It makes it less complicated and it is easy to figure out the "no phase annotated" in the query builder once you have the lists.
We can work on adding the phases.

The query server side is complete now.  Here's an example:
https://pombase.org/results/from/id/e592ee17-f995-452d-8277-3c32ac413f0c
... but there's no interface to get that result yet.  I'll do that next.
I'd say this is two thirds done now.  I think another 2-3 hours.

I've mostly got this working,  I'll demonstrate on the next call.

I've fixed a few bugs and changed things so that you can get all the substrates if you don't select a "During" filter.  I decided that it's working enough that it be can released it so I've updated the main site.
The wording of the link text isn't great at the moment.  Any suggestions?  Currently it's:

Send all substrate genes to Advanced Search ...

or it includes the During term name if the list is filtered:

Send substrate genes during mitotic M phase to Advanced Search ...

Looks good. One thing. When I did all cdc2 substrates
These appeared to be missing?

Systematic ID
Gene name
Product description

SPBC342.05
crb2
BRCT and tudor domain DNA checkpoint protein Crb2

SPBC83.18c
fic1
contractile ring-membrane anchor Fic1

SPBC582.03
cdc13
G2/M B-type cyclin Cdc13

SPCC18B5.03
wee1
M phase inhibitor protein kinase Wee1

SPBC725.16
res1
MBF transcription factor complex subunit Res1

SPCC1322.12c
bub1
mitotic spindle checkpoint kinase Bub1

SPBC646.13
sds23
serine/threonine protein phosphatase PP2A inhibitor Sds23/Moc1

SPAC144.13c
srw1
substrate-specific mitotic G1 APC coactivator Srw1/Ste9

Systematic ID	Gene name	Product descriptionSorted by: Product description
SPBC342.05 crb2	BRCT and tudor domain DNA checkpoint protein Crb2
SPBC83.18c fic1	contractile ring-membrane anchor Fic1
SPBC582.03 cdc13	G2/M B-type cyclin Cdc13
SPCC18B5.03 wee1	M phase inhibitor protein kinase Wee1
SPBC725.16 res1	MBF transcription factor complex subunit Res1
SPCC1322.12c bub1	mitotic spindle checkpoint kinase Bub1
SPBC646.13 sds23	serine/threonine protein phosphatase PP2A inhibitor Sds23/Moc1
SPAC144.13c srw1	substrate-specific mitotic G1 APC coactivator Srw1/Ste9

These appeared to be missing?

Fixed!

I think you can release this, it will be useful

I found a small issue.
If I search on all substrates for cdc2 and then send to "advanced search"
then send
| Results | Previous | Query history
-- | -- | -- | --
  | 143 |   | Substrates of cdc2 during mitotic M phase
to the ad vanced search, the M-phase substrates  query overwrites the total substrates (I wanted to see the difference)

Thanks.  I've found and fixed that bug.  I'm re-releasing the main site now.

on www advanced search
Substrates of cdc2 during mitotic M phase
still overwrites
All substrates of cdc2
(and vie versa)

The site hasn't re-released yet.  oliver1 ran out disk space so the re-release failed.  I'm cleaning up the disk at the moment.

The query overwriting issue is now fixed on the main site.

I think this is done.  We can open new issues is we see any bugs.

I will announce today