Post Logout user is not forced to enter credentials and tokens issued

Hi Team,
I am facing an issue with logout and looking for your support. I have implemented the code similar to the sample implementation provided at [(https://github.com/jeroenheijmans/sample-angular-oauth2-oidc-with-auth-guards)] and the server used here is okta.
In order to logout I have used this.oauthService.logOut(); and post this call, I can see that the localStorage is cleared and the user gets redirected to the post_logout_url configured. However, when i hit the login button again, this time the login page of the okta server is not shown and user directed gets the tokens and the isAuthenticated flag is true.
I tried to empty the cache manually and then tried, this time i get the login page.
I want to mandate the login page of okta post logout. Below is the config i am using,
export const authConfig: AuthConfig = {
  issuer: '<okta>',
  clientId: 'clientid', // The "Auth Code + PKCE" client
  responseType: 'code',
  redirectUri: window.location.origin,
  silentRefreshRedirectUri: window.location.origin + '/silent-refresh.html',
  scope: 'openid profile email api', // Ask offline_access to support refresh token refreshes
  useSilentRefresh: true, // Needed for Code Flow to suggest using iframe-based refreshes
  silentRefreshTimeout: 5000, // For faster testing
  timeoutFactor: 0.25, // For faster testing
  sessionChecksEnabled: true,
  showDebugInformation: true, // Also requires enabling "Verbose" level in devtools
  clearHashAfterLogin: false, // https://github.com/manfredsteyer/angular-oauth2-oidc/issues/457#issuecomment-431807040,
  nonceStateSeparator : 'semicolon', // Real semicolon gets mangled by IdentityServer's URI encoding
  postLogoutRedirectUri : window.location.origin+'/logout'
};

Also the code flow is,

App component had runInitialLoginSequence() and i have checked if the isAuthenticated=false then call login(). With this on application launch we redirect user to the login page of okta.
On logout, created an new component LogOutComponent, however this component is never shown to the end user, because after loading the Logout component, the appComponent then checks if isAuthenticated=false, and then executes the login(). However, here we want the end user to navigate to okta login page always.

Please let me know if i am missing something?

Sounds like the cookie at Okta is not being cleared on the first logout redirects. Something to check.
Apart from that, I'm afraid it's gonna be super specific to your situation, will need to debug it yourself and let us know what you find, so others might find your solution too.
(Only way for us to further help would be with a full but minimal repro. I'm tagging it for that, but even with it I'm not sure if any community member here will step up and help get your issue fixed...)

Thank you for the reply. I figured out the issue, the logout api was getting canceled which should internally clear the cache in order to force the user for login.
When i debugged the code of oauth lib i found that the logout() needs logoutUrl from authConfig. When i mentioned there the logout url of okta then it worked.
So does it mean we need to know the url of the IDP server then only logout will work?

If your IDP supports the end_session_endpoint in its OIDC Discovery Document then I don't think you need to provide a URL. You can see in this library code that it will use that entry from the discovery document, unless you provide your own logoutUrl.
So yeah, if your end_session_endpoint is not 'discoverable', then you need to provide it yourself.
Either way, glad to hear you found the solution! Thx for sharing :)

Hi @jeroenheijmans , @sonalithakare
I am facing the same issue. I dont have the logout url, i am using a service which invalid the current token and by using route.navigate i am going to logout page.
While in logout function i am calling this.oauthservice.logout() function but did not get any result.
Can you please help me in this

Same thing here, storage is cleared whut when trying to log in again, it was happening without being redirected to login page.
So after reading @sonalithakare and @jeroenheijmans comments, i found that my idp server have ping_end_session_endpoint, so i set the value as the logoutUrl, and now things have changed but not yet the desired result. Now after logging out, i am redirected to a page that says you have been logged out ... But didn't find a way to get redirected back to my app, even if i set the postLogoutRedirectUri.


Prometheus endsAt timeout is too short

Proposal
Improve the calculation used to generate the endsAt parameter sent to AlertManager?
We have encountered AlertManager closing and re-opening alerts on multiple occasions even though Prometheus itself does not close the alert.
Currently the endsAt parameter sent to AlertManager defaults to a short time frame which results in AlertManager closing and then re-opening alerts if Prometheus is unable to update the alert within the time frame.
The endsAt parameter is set to 3x the greater of the evaluation_interval or resend-delay values. This results in a default maximum value 3 minutes in the future. In practice, this is often less due to the delay between evaluation and the message being sent to AlertManager.
This is fine under normal running operations. However under normal running operations, Prometheus will send [resolved] messages to AlertManager and the endsAt field adds little value.
Under other operational processes, such as restarting Prometheus or reloading a large configuration, these processes may start sometime after the last message was sent to AlertManager and not complete in time for Prometheus to complete it's evaluation cycle and send the next expected message to AlertManager.
Additionally, the 'rules.alert.for-grace-period' was specifically introduced to allow persisting of alert state, so the endsAt field should account for this too.
If this proposal is agreed, it should be a fairly easy improvement.

This results in a default maximum value 3 minutes in the future. In practice, this is often less due to the delay between evaluation and the message being sent to AlertManager.

That sounds like plenty of time to me. Can you share the timings you're seeing?

Additionally, the 'rules.alert.for-grace-period' was specifically introduced to allow persisting of alert state, so the endsAt field should account for this too.

That's a separate system.

Thanks for your response. I have the logs from the AlertManager perspective.
Both Prometheus and AlertManager time is in sync, the resend-delay on Prometheus is set to 1m.
When the Prometheus configuration is reloaded, it skips one message to AlertManager and this is enough for AlertManager to mark the Alert as resolved. Prometheus is actually only 1s too slow in notifying AlertManager of the continued firing Alert.
The Alertmanager logs at debug level (ts and aggrGroup removed to reduce noise):
Feb 25 19:55:56 host alertmanager[16012]: level=debug  caller=dispatch.go:444 component=dispatcher msg=Flushing alerts=[LinkDown[48042ac][active]]
Feb 25 19:57:29 host alertmanager[16012]: level=debug caller=dispatch.go:201 component=dispatcher msg="Received alert" alert=LinkDown[48042ac][active]
Feb 25 19:58:56 host alertmanager[16012]: level=debug caller=dispatch.go:201 component=dispatcher msg="Received alert" alert=LinkDown[48042ac][active]
Feb 25 19:58:56 host alertmanager[16012]: level=debug caller=dispatch.go:444 component=dispatcher  msg=Flushing alerts=[LinkDown[48042ac][active]]
Feb 25 19:58:56 host alertmanager[16012]: level=debug caller=dispatch.go:444 component=dispatcher  msg=Flushing alerts=[LinkDown[48042ac][active]]
Feb 25 20:01:56 host alertmanager[16012]: level=debug caller=dispatch.go:444 component=dispatcher  msg=Flushing alerts=[LinkDown[48042ac][resolved]]
Feb 25 20:01:56 host alertmanager[16012]: level=debug caller=dispatch.go:444 component=dispatcher  msg=Flushing alerts=[LinkDown[48042ac][resolved]]
Feb 25 20:01:57 host alertmanager[16012]: level=debug caller=dispatch.go:201 component=dispatcher msg="Received alert" alert=LinkDown[48042ac][active]
Feb 25 20:03:26 host alertmanager[16012]: level=debug caller=dispatch.go:201 component=dispatcher msg="Received alert" alert=LinkDown[48042ac][active]
Feb 25 20:04:56 host alertmanager[16012]: level=debug caller=dispatch.go:201 component=dispatcher msg="Received alert" alert=LinkDown[48042ac][active]
Feb 25 20:04:56 host alertmanager[16012]: level=debug caller=dispatch.go:444 component=dispatcher  msg=Flushing alerts=[LinkDown[48042ac][active]]
Feb 25 20:04:56 host alertmanager[16012]: level=debug caller=dispatch.go:444 component=dispatcher  msg=Flushing alerts=[LinkDown[48042ac][active]]

AlertManager incorrectly marks the alert as resolved because the endsAt period has passed. The Alert itself remains firing on Prometheus the entire time.
When I query AlertManager using the /api/v1/alerts endpoint, the endsAt value is rarely more than 2 minutes in the future even though the resend-delay is 1m. I assume this may be due to internal delays between the evaluation cycle and re-sending alerts.
The reload event is our scenario, but it's possible to foresee others such as load spikes or connectivity issues which occur in production environments and it's exactly times like that where you don't need incorrect information.
Regarding the rules.alert.for-grace-period, the reason I mention that is that it is broken by the endsAt value being set using a lower time period.

Prometheus is firing an alert with a for value of 5 minutes. It sends firing messages with an endsAt 3 minutes in the future.
Prometheus is restarted for some reason. The grace-period is 10minutes and all alerts with a for value of less than 10 minutes are not automatically recalculated. They must wait until the 5 minute for time period expires.
Any alerts with a for value of greater than 10 minutes are recalculated on the next evaluation and put straight back into the firing state. AlertManager maintains those Alerts in the firing state.
After 3 minutes, the endsAt on AlertManager passes and resolved notifications are sent.
After 5 minutes Prometheus puts the Alert back into firing state and more notifications are sent.

If the endsAt value was greater than the time+rules.alert.for-grace-period then all firing alerts on AlertManager would correctly remain in the firing state. This seems more desirable than the current behavior and would also mitigate the specific issue we are having.

What's your evaluation interval, and how long is that group taking to evaluate?

This seems more desirable than the current behavior and would also mitigate the specific issue we are having.

It's not unexpected that alerts will flap on a restart, the persistence stuff is more for alerts with long fors.

Our global evaluation_interval is 15s, some rule groups use interval 30s.
The max prometheus_rule_group_last_duration_seconds for all rule groups is consistently under 1s.
We currently use 7 Prometheus instances, each monitoring thousands of targets, many with hundreds of metrics.  The inventory is dynamic and bespoke so there is no off the shelf service discovery. Instead the file_sd_config are regenerated frequently and each Prometheus instance may reload the config numerous times per day. At any time, there will be many alerts firing at the Prometheus level, although these are grouped, silenced and routed differently up the stack.
If there's a possible race condition or edge case, chances are it will show up sooner or later on our environment.
Basically, the file configuration and distributed nature of Prometheus is perfect for our use case. Flapping alerts are not so good because they will bother somebody who is not even aware Prometheus has just restarted or had a brief glitch.

Looking more closely at your logs, Prometheus skipped two messages. The settings are only meant to be resilient to one. #5126 should help with that, which will be in 2.8.

We also noticed the exact same behavior (and messages in data), and we also have quite dynamic prometheus instances, which reload a lot with approx the same config
We have around 100s of prometheus worldwide, and the one which seems the most affected are the one with a high number of opened alerts (we have one with 50+ opened alerts at all times, which reproduce this behavior in a "stable" way)
But we are already in 2.10 on theses instances :(
Would it make sense to make the delay multiplier configurable somehow ?

I don't think it should be configurable, but if the current hardcoded value isn't enough we should understand why and then bump it accordingly.

:+1:
I will check if increasing the fixed value fixes anything in our testable instance then try to understand why this happens
On a side note, we may want to avoid being failed due to a single second late on the final notification by putting some additional tolerance (3-5 seconds ?) when setting the EndsAt
WDYT ?

There's already meant to be tolerance for that.

I've gone through the math, and as designed this is resilient to one failed send/evaluation running over time. It seems some of the above issues were due to issues fixed by #5126, but allowing for two things going wrong wouldn't be the worst idea while still being reasonably responsive to a Prometheus going away.


Fix broken link to entities.md

This link 404s on the live version of the site because of a typo.

Thanks for cleaning up after me! 😅 Sorry I left this PR dormant for a while.


using duphold on a multisample trio VCF file

Hi,
I have a multisample trio VCF file generated using Oxford Nanopore long read data. Is there any way where duphold can be used on a multisample VCF file?
Regards,
Prasun

Hi, you can use smoove duphold command for exactly this. It will parallelize the samples on a single machine and generate a single VCF for output.

Thanks @brentp! My VCF file is already joint genotyped using sniffles2. It seems that smoove will the samples (please correct me if I am wrong). My aim is to just annotate my SVs using duphold annotations.


[Compiler Validator] Compiler doesn't validate equality of entities between submodules

Description:
If we copy the same entity to the sub-module. the compiler doesn't give an error for duplicated table names, instead updates the SQL scripts with duplicating table create scripts

Shall we avoid creating an entity with the same name inside sub-modules of the same project?
This kind of a limitation, but if there are entities with the same name, there is an issue with clients.

However, if the user the has overridden the table name, table name should be taken into consideration not record name.
Improvements,

Improve error message to indicate where the other duplicated entry is
The line indicated in the error message must be specific to either tableName field/ record name. No need to indicate the entire entity.

Closing this as the tableName field in annotations is removed in V2


[release-4.16]Revert "build-patched-kao-kcmo-images.sh: get image name from brew container image metadata

This reverts commit a2b6ece823dfade602a128585e477fbdf6ac56ab.
Revert "kao-kcmo-images.sh: Don't hardcode the image name"
This reverts commit d4ca7af70589d951ec19df5363da52b1757ff307.
Revert "kao-kcmo-images.sh: Add reference for image_from_brew variable"
This reverts commit 8c64f2d9e24713c615a92f40376d39f4a0ab23cf.
We need to revert since the metadata file is not available for quite some time even after the brew build successful and CI is failing because of that with following error.
++ curl -L https://download.eng.bos.redhat.com/brewroot/packages/crc-cluster-kube-apiserver-operator-container/v4.15.0/202406052127.p0.geae6926.assembly.stream.el9/metadata.json
++ jq -r '.build.extra.image.index.pull[0]'
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100   328  100   328    0     0   2000      0 --:--:-- --:--:-- --:--:--  2012
parse error: Invalid numeric literal at line 1, column 10

This isn't merged yet, but I believe 4.16 bundles were successfully generated anyway? Do we need this change?

This isn't merged yet, but I believe 4.16 bundles were successfully generated anyway? Do we need this change?

patched images are generated using current 4.16 branch but we needed to perform same build 2 times with 15-20 mins duration to have it available. So yes we still need this change.


For a 0.18.25

#668 #670 #674

Codecov Report

Merging #676 (8b75a34) into master (efe0fd8) will decrease coverage by 1.42%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #676      +/-   ##
==========================================
- Coverage   85.30%   83.87%   -1.43%
==========================================
  Files          39       39
  Lines        3417     3356      -61
==========================================
- Hits         2915     2815     -100
- Misses        502      541      +39

Impacted Files
Coverage Δ

src/data/data.jl
93.63% <100.00%> (-0.17%)
:arrow_down:

src/utilities.jl
81.66% <100.00%> (ø)

src/measures/meta_utilities.jl
25.00% <0.00%> (-71.67%)
:arrow_down:

src/measures/loss_functions_interface.jl
73.07% <0.00%> (-18.82%)
:arrow_down:

src/composition/abstract_types.jl
40.00% <0.00%> (-17.15%)
:arrow_down:

src/measures/measure_search.jl
50.00% <0.00%> (-6.25%)
:arrow_down:

src/measures/continuous.jl
91.30% <0.00%> (-4.53%)
:arrow_down:

src/measures/probabilistic.jl
61.33% <0.00%> (-1.83%)
:arrow_down:

src/hyperparam/one_dimensional_range_methods.jl
84.34% <0.00%> (-0.14%)
:arrow_down:

src/measures/finite.jl
98.10% <0.00%> (-0.03%)
:arrow_down:

... and 2 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 efe0fd8...8b75a34. Read the comment docs.


Upgrade Operator SDK to version 0.1.1

Followed this migration doc https://github.com/operator-framework/operator-sdk/blob/master/doc/migration/v0.1.0-migration-guide.md

code looks ok, will give it a try


Fix nu section in key-bindings.md - set-env -> let-env

I don't think set-env exists 🤔
❯ : set-env ATUIN_NOBIND = true
Error: nu::shell::external_command

  × External command failed
   ╭─[entry #6:1:1]
 1 │ set-env ATUIN_NOBIND = true
   · ───┬───
   ·    ╰── did you mean 'let-env'?
   ╰────
  help: No such file or directory (os error 2)

Can you triple check what version of nu you are running? Other has nu made yet another breaking change 😭?

Can you triple check what version of nu you are running? Other has nu made yet another breaking change 😭?

I'm running 0.81.0 which is the latest version. The extra weird thing is that I can't find a reference to set-env anywhere in the changelogs.

Yeah I guess it must have been a typo from day one!

anyways, thanks for fixing @aopb
Seeing as this is your first time contributing, if you would like a holographic contributors-only Atuin sticker, then please fill out this form!
We do also have a Discord if you'd like to ask any questions, or just fancy hanging out!


[OSB] Tradable items - dt2 rings/vials of blood

The desert treasure 2 rings need to be made tradable
the 4x dt2 fully completed rings need to be made tradable items,
Venator ring
Ultor ring
Bellator ring
Magus ring
The graphical appearence of these rings also needs to be updated, as currently they're using the placeholder graphics, below is an example of the venator ring.
Venator ring Current osb/bso graphic = placeholder graphic
https://oldschool.runescape.wiki/w/Ventor_ring_(beta)
It should be this
https://oldschool.runescape.wiki/w/Venator_ring
--
Vials of blood from tob need to be made tradable, especially now considering that scythe is now a global boost.

The vestiges aren't tradeable in game, just the finished rings.

O yeah you;re right., I thought I remember selling my vestiges, I must have made them into rings, ty will edit post.

Duplicate of #3378, #5284, #5282


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Human Resource Flow Josie Notes

Mrs Habgood Lesson 4
07/03/23

Thank you for providing notes, fixing up and merging.


docs: Change lets to const when creating store

Noticed that code examples use:
let store = createStore(...)
Since store is never reassigned, we should use const.

let's update a few more... Thanks!

Sweet. Thanks @timdorr!


AS A MUSLIM I DON'T CARE ABOUT SANTA, SO PUT IT BACK

VSCode Version:
OS Version:

Steps to Reproduce:

Does this issue occur when all extensions are disabled?: Yes/No

Trying to see if Microsoft listens to the other religion

@microsoft please delete this

Why @J0s3f ?

I will change the title if you want

Why @J0s3f ?

I don't like muslims!

The muslims did nothing wrong, @J0s3f .

Why @J0s3f ?

I don't like muslims!

@Isuru-Nanayakkara Great idea

The muslims did nothing wrong, @J0s3f .

But I stand with Israel!

If we want to win against this jew troll, we should stand against their religion to get Santa back !!

 WE ARE GETTING SANTA

Why @J0s3f ?

I don't like muslims!

I don't like muslims!

What about getting a blowjob from santa ?

doing my job here

Thanks Ma'am

doing my job here

we have to continue

This is the real meme

imagine getting offended over a goddamn hat lmfao

@fofofofofofofofoXD agreed

https://github.com/VSCodium/vscodium

Hi. I am neither Jew or Muslim.


If classes.dex is loaded the metadata is not shown

fixed here https://github.com/google/android-classyshark/commit/872f0977cd40823cb45416c1f96c50736066d4d1 added identity check


Issue 1767 - Speed up permanova

Refactor permanova. Now about 100x faster.
Remove the large intermediate buffers and implement the time consuming part in cython.
Issue #1767
Please complete the following checklist:

[x] I have read the guidelines in CONTRIBUTING.md.

[x] I have documented all public-facing changes in CHANGELOG.md.

[ ] This pull request includes code, documentation, or other content derived from external source(s). If this is the case, ensure the external source's license is compatible with scikit-bio's license. Include the license in the licenses directory and add a comment in the code giving proper attribution. Ensure any other requirements set forth by the license and/or author are satisfied. It is your responsibility to disclose code, documentation, or other content derived from external source(s). If you have questions about whether something can be included in the project or how to give proper attribution, include those questions in your pull request and a reviewer will assist you.

[x] This pull request does not include code, documentation, or other content derived from external source(s).

Note: REVIEWING.md may also be helpful to see some of the things code reviewers will be verifying when reviewing your pull request.

@wasade

Note: None of the validation failures is due to code changes. The Linux 3.6 error is due to a flaky test, the others are failing for master, too.


MFXVideoENCODE_Query parameter change will cause GPU hang later

hi
firstly, the behavior of MFXVideoENCODE_Query does not match the function name. It makes sense that query does not change anything. But looks that MFXVideoENCODE_Query does change something within msdk.
as discussed in https://github.com/Intel-Media-SDK/MediaSDK/issues/1817, multi frame encode and roi encode conflict, and cause GPU hang. But it turns out that GPU will hang later once MFXVideoENCODE_Query contains both mfe and roi in mfxVideoParam.ExtParam, see code at https://github.com/guoyejun/ffmpeg/commit/534712d3376c9557734b877f1c46a84743aaffbf, this patch just adds roi structure into the parameter (no need to enable roi at encode time, in other words, mfxEncodeCtrl.NumExtParam is still zero) and reproduces a gpu hang.
I think that a query parameter change should not cause a following GPU hang.

@DenWolf  any plan when this issue can be fixed? thanks.

Hi @guoyejun ,
About the first part of your question:

But looks that MFXVideoENCODE_Query does change something within msdk.

Regarding Media SDK documentation function MFXVideoENCODE_Query can change some parameters if they are incompatible with other parameters, you can find it here:
https://github.com/Intel-Media-SDK/MediaSDK/blob/master/doc/mediasdk-man.md#MFXVideoENCODE_Query
About observed GPU hang - could you please provide command line for reproducing the issue (it will be great if it's reproducible by Media SDK samples) ? And what platform do you use?
This information will be very helpful for analyzing the issue first and reproduce/debug it further.
Also, I have a question about the priority - what's the impact of this issue and how urgent it is?
I asked because we have a public holidays soon (Jan 1-8), also we need to set correct priority of this task.
Of course I will try to look at it (by last working days in this year) regarding your information, but if the issue will be hard to analyze/resolve fast and it will take deep development investigation - I think we can start full investigation not early than in 2 weeks (after holidays).
Every information from you will be very helpful :)
Best regards,
Denis

thanks @DenWolf for your quick reply.

Hi @guoyejun ,
About the first part of your question:

But looks that MFXVideoENCODE_Query does change something within msdk.

Regarding Media SDK documentation function MFXVideoENCODE_Query can change some parameters if they are incompatible with other parameters, you can find it here:
https://github.com/Intel-Media-SDK/MediaSDK/blob/master/doc/mediasdk-man.md#MFXVideoENCODE_Query

I see.  I meant that the function name 'query' has no meaning to change something. It would be more logical if the function name looks like query_and_fix or something else. I understand it's hard to change public API. Just mention it in case that you plan to do a change.

About observed GPU hang - could you please provide command line for reproducing the issue (it will be great if it's reproducible by Media SDK samples) ? And what platform do you use?

I'm using skylake + ubuntu 16.04.
I just quickly looked at the msdk samples, but did not find where to set mfxVideoParam.ExtParam in pipeline_encode.cpp.
The key changes to reproduce the issue is to add both mfe and roi into mfxVideoParam.ExtParam. The key code looks like:
mfxExtEncoderROI extroi;
extroi.Header.BufferId = MFX_EXTBUFF_ENCODER_ROI;
extroi.Header.BufferSz = sizeof(extroi);
extroi.ROIMode = MFX_ROI_MODE_QP_DELTA;
extroi.NumROI = 256;
// due to the requirement of msdk, we must set non_empty rect for query
for (int i = 0; i < sizeof(extroi.ROI)/sizeof(extroi.ROI[0]); ++i) {
extroi.ROI[i].Right  = 16;
extroi.ROI[i].Bottom = 16;
}
            mfxExtMultiFrameParam   extmfp;
            extmfp.Header.BufferId     = MFX_EXTBUFF_MULTI_FRAME_PARAM;
            extmfp.Header.BufferSz     = sizeof(extmfp);
            extmfp.MFMode = MFX_MF_AUTO;

Hope it helps. I also give the detail steps to reproduce the issue with ffmpeg below.

This information will be very helpful for analyzing the issue first and reproduce/debug it further.
Also, I have a question about the priority - what's the impact of this issue and how urgent it is?

I'm enabling ROI encoding in ffmpeg for qsv path. Without the fix, i just did a workaround in ffmpeg, but it is not accepted by the community. So the ROI encoding could not be enabled for qsv path in ffmpeg if it is not fixed.
To reproduce the issue with ffmpeg:
git clone https://github.com/guoyejun/ffmpeg.git
cd ffmpeg
git checkout -b qissue qsvqueryissue
mkdir build
cd build
../configure --enable-libmfx
make
./ffmpeg -s 352x288 -i input352x288.yuv -c:v h264_qsv -y qsv.h264
we'll see a gpu hang.
if we just revert https://github.com/guoyejun/ffmpeg/commit/0f510765852fc40a838d1ad462522cd79fcd0899, no gpu hang.

hi, could you reproduce this issue? thanks.

thanks, and do we have an estimate when this will be solved? thanks.

any estimate when this will be solved? thanks

Hi @guoyejun ,
Sorry for the late response - our team was extremely busy last month, @pgribov and me had several tasks in parallel, which slowed down investigation of the current issue.
But we have an interesting update about the issue and found that it is not on Media SDK side - it's on driver side (https://github.com/intel/media-driver).
By preparing simple reproducer we've have found that issue with observed hang exists only on BRC mode (not reproduced on CQP) with MFE+ROI enabled and - the most important - it's related to SingleTaskPhase mode (https://github.com/intel/media-driver/blob/master/media_driver/agnostic/common/codec/hal/codechal_encoder_base.h#L1676) - when the singleTaskPhase mode is enabled (be default), issue is observed; when we disabled singleTaskPhase - issue is gone.
After looking to the source code of the driver and collecting dumps we suspect incorrect programming issue (looks like for the kernels - ENC stage for BRC case) which related to combination of SingleTaskPhase+MFE+ROI+BRC(ex: CBR)

@DenWolf , is it a bug or "per-design" limitation? If it's a "per-design" limitation, can we disable RIO (in case of MFE)  via corresponding libva caps?

@dmitryermilov - we need more time for the further investigation and understanding the root cause of this issue - is it a bug or it's expected.
For now it's trending as a bug for me, but I'm totally agree with you that if we will find that this is expected limitation, this case should be definitely disabled in CAPS

thanks.
From MSDK user's perspective, it is weird that query parameter could cause gpu hang.
Is it possible that MSDK first do a tiny change to solve this issue?  For example, just ignore ROI internally when both ROI and MFE are enabled in MFXVideoENCODE_Query.


calico node can not ping another node's tunl0

Expected Behavior
i have a kubernetes cluster,i have the calico to manage the pod's ip,calico manage four nodes.
my problem is i can not ping the tunl0 of slave1 from master1,but i can ping the tunl0 of master2 from master1.
how to solve the problem?

What cloud provider are you running on?
Are your slave and master instances in the same Security Groups and do they allow IPIP between the hosts?

i have my kubernetes cluster in my own openstack。
my master and slave instances are in the same security groups.but i do not know if they allow ipip between the hosts,how to judge?
i find master1 can not ping tunl0 of slave1，but slave1 can ping tunl0 of master1.
when i ping tunl0 of master1 from slave1,then i can ping tunl0 of slave1 from master1,i do not know why .i want to know the reason.

Are all your calico-node pods running successfully?
If yes, could you install calicoctl and run calicoctl node status on one of your slaves and one master and include the results here.
https://docs.projectcalico.org/v3.1/usage/calicoctl/install#installing-calicoctl-as-a-binary-on-a-single-host Note: It will not work to use the containerized versions for this specific command and it is not necessary to configure calicoctl.
With your last comment about the ping seeming to fix things I think you should take a look at, https://github.com/projectcalico/calico/issues/1357.

thanks all.
i have found the reason about the problem.
the network flow table of master1 was lost,so when slave1 ping master1,master1 have cache the flow table,then master1 can ping slave1.but when cache was invalid,master1 can not ping slave1.
so i will find the reason of network flow table lost.

so i will find the reason of network flow table lost.

@shigy123456 I encountered the exactly the same problem as you had, may I ask how you solve this ?

Facing the same issue, could you confirm the changes?

In which version is this problem resolved

i have my kubernetes cluster in my own openstack。 my master and slave instances are in the same security groups.but i do not know if they allow ipip between the hosts,how to judge? i find master1 can not ping tunl0 of slave1，but slave1 can ping tunl0 of master1. when i ping tunl0 of master1 from slave1,then i can ping tunl0 of slave1 from master1,i do not know why .i want to know the reason.

我也是这个问题,请问怎么解决的啊?


Move multi-var detection out of NL server

With the intent of getting rid of the NL server, strip down its functionality to compute query embeddings and look it up in its index.  This will be followed by a real "Vector DB" support.
This is a ZERO functionality change PR.
TODO:  Due to this move there is one multi-var embeddings test that needs to be rewritten in the explore_test framework.  Since there is a concurrent PR (https://github.com/datacommonsorg/website/pull/4092) modifying those files, I'll make that test change in a follow up PR.

Nice! Only thing is I see some changes moving towards renaming sv -> vars but also not renaming it everywhere, wonder why the change and should we try to be consistent with var/sv?

Sorry for not explaining the rationale.  I was thinking of renaming things such that "vars" can mean => StatVar or Topic or Prop, whereas "sv" just means StatVar.
But wanted to limit the scope of the changes.  So, in this PR, renaming just the:

The high-level variable detection entrypoint "detect_vars"
The Lower level embeddings search endpoint "search_vars"

Wdyt?

Nice! Only thing is I see some changes moving towards renaming sv -> vars but also not renaming it everywhere, wonder why the change and should we try to be consistent with var/sv?

Sorry for not explaining the rationale. I was thinking of renaming things such that "vars" can mean => StatVar or Topic or Prop, whereas "sv" just means StatVar.
But wanted to limit the scope of the changes. So, in this PR, renaming just the:

The high-level variable detection entrypoint "detect_vars"
The Lower level embeddings search endpoint "search_vars"

Wdyt?

I'll submit this, to follow up on test changes, we can discuss further name changes next week.


[FLINK-6580] Sync default heap sizes from code with config file

Flink didn't start on YARN anymore without explicit configuration of JM and TM heap because the default heap sizes in the yaml file were set too low.
This PR increases the default heap sizes according to the TaskManagerOptions and JobManagerOptions classes.

Do we know the underlying reason that is causing YARN to fail now? None of the config values seem to have changed in a long time.

nvm, found the reference to FLINK-5904 in the JIRA.

@zentol do you agree to merge this now. Its the last thing I would like to get into RC1.

+1 to merge after you've changed it back to 1024.


Bug 1829735: fix(operator): map top-level csv phase to component conditions

Description of the change:
Map the top-level status of a CSV to a single status condition of an Operator component reference. A corollary is that a CSV component reference will only have the latest phase recorded as a condition.
Motivation for the change:
The CSV status condition type doesn't match component status conditions, resulting in a bad projection:
- lastTransitionTime: "2020-04-30T07:05:13Z"
  lastUpdateTime: "2020-04-30T07:05:13Z"
  message: all requirements found, attempting install
  reason: AllRequirementsMet
  status: ""
  type: ""

Additionally, CSV conditions are effectively a log of past phases rather than a summary of the current status - mapping 1:1 produces noisy Operator resources.

/bugzilla refresh

/retest

/retest

/retest

/retest

/retest

/hold

https://github.com/operator-framework/operator-registry/pull/437 has merged and should rectify the test timeouts.
/retest

/hold cancel
/retest

/retest


Support returning complex structs from public functions

What is wrong?
Complex structs (structs where the type of at least one field is not a base type) can't yet be returned from public functions.
How can it be fixed
We need to implement ABI encoding for these types of structs.

will be resolved in #676


GetAvailableSlotsForEmployee_EmployeeIdNoFound_ArgumentException doesn't test the lack of a valid employee ID

From SlotsServiceTests.cs:
[Fact]
public async Task GetAvailableSlotsForEmployee_ServiceIdNoFound_ArgumentException()
{
    // Arrange

    // Act
    var exception = await Assert.ThrowsAsync<ArgumentException>(
        () => _sut.GetAvailableSlotsForEmployee("SomeServiceId", "SomeEmpId"));

    // Assert
    Assert.IsType<ArgumentException>(exception);
}

[Fact]
public async Task GetAvailableSlotsForEmployee_EmployeeIdNoFound_ArgumentException()
{
    // Arrange

    // Act
    var exception = await Assert.ThrowsAsync<ArgumentException>(
        () => _sut.GetAvailableSlotsForEmployee("SomeServiceId", "SomeEmpId"));

    // Assert
    Assert.IsType<ArgumentException>(exception);
}

From SlotsService.cs's GetAvailableSlotsForEmployee method:
var service = await _serviceRepository.GetItemAsync(serviceId);
if (service is null)
{
    throw new ArgumentException("Record not found", nameof(serviceId));
}

var employee = await _employeeRepository.GetItemAsync(employeeId);
if (employee is null)
{
   throw new ArgumentException("Record not found", nameof(employeeId));
}

Both of the above methods call the same method, but are supposed to test the validity of different parameter values.
However, since neither the service ID nor the employee ID is valid, the block of code that gets the employee is never executed.

The state of the code would be for page 261 in the book.


[Task] getKeyPairDetails feature in account manager uses JS SDK

Task
Contributors are able to complete this task and earn thenewboston coins.  Check out the labels to learn how much you can earn for contributing by completing this task.  Please make sure to be honest if you wish to contribute by saying you can't finish this and we can just un-assign you with no harm done!  There is no point in delaying tasks from being completed for miscommunication!

Overview
Public key
Public key hex
Signing key
Signing key hex
getKeyPairFromSigningKeyHex
Behavior
A description of how the contributor's work should behave.
/* code examples are awesome */

Please ask for this task to be assigned to you and earn and its sweet reward 😉
Remember to include your account number in your PR description for us to pay you 💰

https://github.com/thenewboston-developers/Account-Manager/pull/609


[New chain]: Hemi Mainnet

Fixes #771
Singleton factory is deployed as part of predeploy and hence I am using values from OP chain artifact file.

Hmm, I'm actually not sure what the transaction and gas fields should be in this case... One issue I have with the current PR is that the signed transaction is not signed for the correct chain ID.

Sorry for the back-and-forth. We discussed internally and proposed the following for networks with pre-installed Safe Singleton Factory contracts:
https://github.com/safe-global/safe-singleton-factory?tab=readme-ov-file#op-stack

Verified deployment code with:
curl -s -X POST https://8deb9c0b.rpc.hemi.network/rpc -H 'content-type: application/json' --data '{"jsonrpc":"2.0","id":0,"method":"eth_getCode","params":["0x914d7Fec6aaC8cd542e72Bca78B30650d45643d7","latest"]}' | jq -r .result

And that it is in fact a pre-installed contract (as the signer account has a 0 nonce):
curl -s -X POST https://8deb9c0b.rpc.hemi.network/rpc -H 'content-type: application/json' --data '{"jsonrpc":"2.0","id":0,"method":"eth_getTransactionCount","params":["0xE1CB04A0fA36DdD16a06ea828007E35e1a3cBC37","latest"]}' | jq -r .result

Thanks @rokso!


Dynamic Spawn timers

Something I made my GINA triggers do is a logic check on a timer if it is a named. Eg.
Normal Spawn trigger, 22m in Mistmoore:
 (?:you\s+have\sslain\s+(.+?)!|(.+?)\shas\s+been\sslain\s+by)
Cancel early Trigger:
have\s+slain\s+An advisor|(An advisor\s+has\s+been\s+slain)
An Advisor Trigger, 2 hours:
have\s+slain\s+An advisor|(An advisor\s+has\s+been\s+slain)
The behaviour is it cancels the generic timer for 22 mins and creates the named timer for 2 hours.
Lots of zones have these sort of named timers that can be put in, from BBM (Dwarf guards), Oasis zone timer of 6:40 mins, but A Spectre is (in Oasis only) is 16:30, right through to ToV etc. I'm happy to help collect exceptions as I am sure the community will for their favorite camp.
I have also been considering how I might be able to use an EXP message if there is no auto timer (because you are too far away to get slain message), or named timer. But it looks like the Slain message is before the EXP message.
So can't use the same format as above eg:

EXP trigger to give a basic timer -> Cancel early if "slain" message can give a timer with a mob name
Slain message to give a mob name in the timer -> Cancel early if "named" trigger.
Name timer trigger.

But maybe EQTool can do something GINA can't!?

Can you restate what your request is? I dont understand.

I am constantly adding timers for all mobs in game. If you find a spawn timer is different than the zone spawn timers, it can be added simply
https://github.com/smasherprog/EqTool/blob/e87ebc09bd929861a7f76af4be036ca6e981089b/EQToolShared/Map/ZoneParser.cs#L1068
Do a PR or let me know of incorrect spawn timers

So does the tool allow for different timers in zones? Looking at the code it seems like an Advisor which is a 2 hour spawn (120 minutes) is showing as a 22 minute spawn?
Or where is the list of Notable NPCs and their specific timers held?
            ZoneInfoMap.Add("mistmoore", new ZoneInfo             {                 Name = "mistmoore",                 ShowAllMapLevels = false,                 ZoneLevelHeight = 10,                 NotableNPCs = new List<string>()                 {                     "an advisor","an avenging caitiff","Black Dire","Butler Syncall","a cloaked dhampyre","a deathly usher","Enynti","Garton Viswin","a glyphed ghoul","an imp familiar","Lasna Cheroon","Maid Issis","Mayong Mistmoore","Mynthi Davissi","Princess Cherista","Ssynthi","Xicotl",                 },                 RespawnTime = new TimeSpan(0, 22, 00)             });

I have just done some killing in Oasis for example, A Dune Tarantula had a 16:30 Spawn timer, so did a sand Giants which should be 6:40 spawn timer.
A Spectre did have 16:30 which is correct for the Oasis.
Tunare was just on the standard Zone timer also.

Fixed following timers:
an advisor -> 4 hours
oasis zone -> 6:40
Tolapumj -> 2:45
Sebilite protector -> 2:45
Code will be available in 5 minutes to update

let me know of any other timers that are not correct
I want everything to just work with as little configuration as possible


Create a style guide

This application does not have a style guide whatsoever. It would be great to be able to propose colors, logos, icons, etc. in order to help the developers who have limited design knowledge to build something pretty.
An example of style guide I like is https://frontify.com/styleguide.

Some style guides I found pretty cool:

Gitbook
Mapbox

codeforamerica

Even more examples: styleguides.io

Not sure if this is a solved problem but are you familiar with fabricator?
https://github.com/fbrctr/fabricator

@kcwoodfield I did not know about this project, But it looks exactly what we need! However, it has been archived, and it looks like it is abandoned :( Is there any active alternative?

@rgreinho you're probably right.  I wonder if that is a recent thing.  Anyhow, lets not use this then - it may create more headaches than we want.

@kcwoodfield This page seems to have very interesting tools to create style guides: https://github.com/davidhund/styleguide-generators

I really like this one: http://astrum.nodividestudio.com/pattern-library/#group-branding-component-icons

@rgreinho I really like that one.
If using react this is nice also: https://github.com/storybooks/storybook

@kcwoodfield It's definitely a nice tool, but it is to help people developing UI components, not creating a living styleguide.

@UX-Chris Since we merged #34 , can we close this issue? Or do you still need to work more on the styleguide?

No, a lot of the style has changed so I'll update the style guide as get get things finished.


Temper when full demo runthrough is executed on PRs

Currently: each push to PR going into main
Suggestions:

when label is added
only on this repo id (no forks)
only delete project if previous steps succeed?

Add a comment on which label needs to be added to a PR to run tests
https://github.com/marketplace/actions/comment-pull-request


When key does not exists i get Index was outside the bounds of the array

For some reason, in production, when the key does not exists it is throwing an Index was outside the bounds of the array, this happens when we are enumerating though a large collection...

Hmmm, what code is precipitating this? Sounds like a server error the constructor isn't handling appropriately.

@slorello89
        public async Task<IEnumerable<GroupMemberModel>> GetAsync(Expression<Func<GroupMemberModel, bool>> predicate)
        {
            return (await _groupMembers.Where(predicate).ToListAsync()).Select(x => x);
        }

I'm pretty sure this is the same as #334 - so it will be fixed by #367.


How to configure keepalived?

WHen using the same httpclient still creates multiple TCP connection.
The retryOnConnectionFailure is configured and it's true.

Thanks for your question. This issue tracker is only for bug reports with test cases and feature requests. Please ask usage questions on Stack Overflow.
https://stackoverflow.com/questions/tagged/okhttp

font{
    line-height: 1.6;
}
ul,ol{
    padding-left: 20px;
    list-style-position: inside;
}

Hi Jesse,    The issues cause by I don't close Response, please see:    try (Response response = httpClient.newCall(request).execute();)
        But how to close response when using async request?Thanks,Jim.X

On 5/13/2020 19:22，Jesse<EMAIL_ADDRESS>wrote：

Closed #6049.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.


people: Verify utcToZonedTime returns a Date before formatting it.

current_date being invalid here could result in user card not being displayed to users. So, it is important to verify that the date is valid so that we don't run into any errors.
This fixes (unverified) an issue where user card was not being displayed to some users on Zulip Desktop app because current_date was not valid.
discussion: https://chat.zulip.org/#narrow/stream/16-desktop/topic/set.20status.20after.20upgrade.20to.206.2E0

I pushed a revision that adjusts the comments and removed the isNaN check; haven't had a chance to investigate the CI failure.

Should be fixed now.

Looks good, tagged to merge once CI passes.

Backported to 6.x as 40d61be293244045d32784e7f654b5a4f9b4552e followed by fda73976bde70cd30a2aacabb637451b62f1cd69 for #23659.


Prevent MetroTabItem closing

I just got a small question cuz I don't know how to fix it.
I'm using the MetroTabItem controls because they come with a close button "out the box". The content of the tabs is text that can be edited by the user. I want to include a save prompt, when closing the tab without saving the text. But I don't know how to actually "cancel" the closing process. The CanExecute method for the CloseTabCommand is being called when clicking on the close button, but even if I return false, the tab is being closed. I don't know if this is a bug or intended behaviour but regardless, it is quite troublesome.
Anything I can do to prevent the tab from closing?

Hi @MaxDeBy
For a notmal TabControl this is marked as todo in the source, so I think this is indeed a bug:
https://github.com/MahApps/MahApps.Metro/blob/29a0ac40a9b99e192aeff63a6def6580c0203076/src/MahApps.Metro/Actions/CloseTabItemAction.cs#L55
For MetroTabControl there is a demo available which shows how to prevent the closing event. The demo lives here:
https://github.com/MahApps/MahApps.Metro/blob/29a0ac40a9b99e192aeff63a6def6580c0203076/src/MahApps.Metro.Samples/MahApps.Metro.Demo/ExampleViews/TabControlExamples.xaml#L290-L334
Happy coding
Tim

Hey @timunie
Thanks for pointing me towards the example. I didn't bother to try and check the TabControl (or MetroTabControl in this case) itself. It just didn't seem logical to me to look at the parent for an event that's about a specific instance of a child.
Regardless, that did the trick. Thanks a lot.

@MaxDeBy
can you reopen the issue as I think there is a bug anyway?
Moreover here is very nice lib for handling tabs: https://dragablz.net
Happy coding
Tim

Woops. Didn't see your message. I'll reopen it then.


User can choose February 31 in ion-datetime

Bug Report
Ionic version:

[x] 4.x
Current behavior:

In the ion-datetime user can choose dates that do not exist, for example, February 31. Only after selecting February, closing ion-datetime and clicking again on the ion-datetime, the number of days is updated to 28.
Expected behavior:

Dynamically update the number of days in ion-datetime.
Steps to reproduce:

Create ion-datetime. Select February. In days will be 29-31. Exit ion-datetime. Select ion-datetime again. Only now the number of days will update to 28.
Related code:
-->
<ion-datetime display-format="DD/MM/YYYY">

Ionic info:

Ionic:

   ionic (Ionic CLI)             : 4.10.3
   Ionic Framework               : @ionic/angular 4.0.1
   @angular-devkit/build-angular : 0.13.1
   @angular-devkit/schematics    : 7.3.1
   @angular/cli                  : 7.3.1
   @ionic/angular-toolkit        : 1.4.0

Capacitor:

   capacitor (Capacitor CLI) : 1.0.0-beta.17
   @capacitor/core           : 1.0.0-beta.17

Hi there,
Thanks for the issue! I’ve added this to issue #16630 to keep track of. We’re going to close out the individual issues and address all of the Datetime issues at once. We’re not forgetting about your issue, just trying to keep them all together. 🙂
Thanks for using Ionic!


JdbcTokenStore uses query that contains reserved word in oracle

Basic information

Axon Framework version: 4.6.0
JDK version:  8
Complete executable reproducer if available (e.g. GitHub Repo):

Steps to reproduce
Use Oracle as a token store
Actual behaviour
Since size is a reserved word in oracle (docs), this query will lead to trouble.

Closing this issue, since pull request #2413 resolved it.


Summarize problem specification

Does the hackmd description resolve this?

Does the hackmd description resolve this?

Yes it does, I was just thinking of somehow making it also available in the repository.


No Producer configuration available in AsyncApiConfiguration.java

when tried adding producer data for AsyncApiConfiguration.java for AMQP example I'm getting issue for binding

kindly help with the configuration for producer in AMQP(RMQ)

You should use:
import com.asyncapi.v2.binding.amqp.AMQPOperationBinding;

...
 .binding(ImmutableMap.of("amqp", new AMQPOperationBinding()))
...

Thank you this worked for me.


Captions and images to work out

Ocean Climate

DO Panel

Final tab (Trends - Near Bottom DO)

[x] Resize image; make it bigger

[x] Also for Halibut Map (first tab)

Ocean Acidification

[x] Three tabs; middle tab is proper size (e.g., Make all three same size as Benthic pCO2)

Oceanographic Indexes

[x] Drop LUSI and TUMI (first tab)
- Keep it as a tab under Physical Processes

Deep Sea Floor

[x] Tab: Trends in Abundance (change tab header!!!)
- Currently just an image pointing to DSCS database

Pelagic
- Cetaceans

[ ] “Grey whales” needs to be corrected (Gray)

[ ] Move “Trends - Gray whale strandings” to the far right
- Trends - Gray whales

[ ] Edit caption; figure references North Ocean and South Ocean; directly references the Makah U&A
- North ocean: Cape Flattery to Cape Alava
- South Ocean: Cape Alava to Sea Lion Rock
- “North ocean (blah) and south ocean (blah); see Maps- Gray Whales”
- Larvae and Zooplankton
- Maps - Larvae and zooplankton stations

[ ] Caption is OK, but “legend” is missing (what do the “+” mean?)
- Noting location of surveys

[ ] Needs a citation!! Look in Condition Report, maybe follow up with Jenn
- Maps - Phytoplankton
- What’s going on w/ the arrows? What’s the “2” referencing? Caption is not clear. Look at Peterson et al., 2017 for help

Seabirds

At Sea surveys

Maps - Marbled Murrelet

[ ] Caption says “density information”; just a map showing where murrelet densities are calculated.

[ ] Move Trends - Marbled Murrelet next to “Maps - Marbled Murrelet” (and drop the s!)

Beaches

Shellfish closures
- Trends - Kalaloch Domoic Acid Levels

[ ] Add (WDFW) to the tab title! E.g.
- “Trends - Mocrocks Domoic Acid Levels (WDFW)”

[ ] Group (CCIEA) tabs w/ (WDFW) tabs (keep it alphabetical)
- Trends - Biotoxin Closure

[ ] Add “WDOH” to tab title

Human Connections

Harvest

Trends - Dungeness crab catch, revenue

[ ] Resize, make image bigger

Trends - Halibut biomass

[ ] Resize, make image bigger

Trends - Revenue, salmon troll

[ ] Resize, make image bigger

Trends - White catching

[ ] Resize, make image bigger

Heritage

Map - Sunken vessels

[ ] Resize, make image bigger

Military activity

Maps - Military activity zoning

[ ] Resize, make image bigger

Non-consumptive recreation

Map - Recreational activities

[ ] Resize, make image bigger

Ocean Sound

Maps - Sound monitoring and zoning areas

Legend says PAM a lot

[ ] Add in Passive Acoustic Monitoring into caption

[ ] Take a crack at expanding the caption

Spills and discharges

Map - Petroleum movement

[ ] Resize, make image bigger

Trends - petroleum movement

[ ] Resize, make image bigger

[ ] Move next to Map - Petroleum movement

Vessels

Trends - Cruise passengers

[ ] Resize, make image bigger

Map - Vessel Density

[ ] Resize, make image bigger

Wildlife disturbance

Map - Entangled whales

[ ] Resize, make image bigger

Oopsie, must have accidently closed this issue. It's very much still open as of 6/28!


Use istio operator for installing istio

Description

Change reconciler logic to installing Istio Operator and applying Istio CR, then waiting for Istio to get ready

Related

[ ] https://github.com/kyma-project/kyma/pull/17963/files

/retest

/retest

/retest


Add MinerArgs for CLI

Describe the feature
add a clap container type for --miner.* args, see --debug.* for reference:
https://github.com/paradigmxyz/reth/blob/cf18edb0fe45abc36325dec905e8efec7800773c/bin/reth/src/args/debug_args.rs#L6-L9
see also geth --help
Additional context
No response

I will pick this.

awesome,
for first PR a simple struct def is sufficient


Update entrypoint.sh

Fix xargs delimiter for directories with spaces.
source: https://stackoverflow.com/a/16758699/2891492
Situation
./this folder has spaces/and_a_dinky_file_without.md
Result
ENOENT: no such file or directory, lstat './this'

I don't have merge privs, could you smash that merge button?

I don't have merge privs, could you smash that merge button?

Sorry, done! I'll publish a new release this week!

@jonathanchancey Or now! Released as v1.3, thanks again!


Issue for Multistore Factfinder feedexport when catalog_product_flat table is used

FACT-Finder SDK version (see package.xml):
Magento edition and version: Magento Enterprise 1.1
PHP version: 7.2
Expected behavior: Correct deeplink should be exported per store in a multi-store environment when flat product tables are used
Actual behavior: Incorrect url generated for all but the first store due to use of the same flat table for all subsequent stores
Steps to reproduce: Export Factfinder using --exportAll as an argument.

Once we try to export the feed in multistore using --exportAll, the getSearchableProducts method in src/app/code/community/FACTFinder/Core/Model/Export/Type/Product.php will always take the first store flat table.

Which extension version do you use?
I setup a magento standard shop with sample data. Installed latest version. The standard shop has one website, one store group and three stores/store views. I added also one more test website with one further store group and another store.

I switched "Use Flat Catalog Product" and "Use Flat Catalog Category" to yes. Reindexing.
All stores do have own subdomains. This will be respected executing the shell command with --exportAll switch.
Are there further steps you did to configure your multi store setup?

I'm not able to reproduce this issue wihout further information. I'll close it.


v1.9 backports 2020-10-31

#13821 -- cilium: minor lb follow-ups (@borkmann)

Once this PR is merged, you can update the PR labels via:
$ for pr in 13821; do contrib/backporting/set-labels.py \$pr done 1.9; done

test-backport-1.9

Green GKE run, testing again to see if it's reliably passing: https://jenkins.cilium.io/job/Cilium-PR-K8s-GKE/3011/

test-gke

netnext provisioning failure: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.12-net-next/1033/execution/node/120/log/

retest-net-next

Second green GKE run: https://jenkins.cilium.io/job/Cilium-PR-K8s-GKE/3012/

test-gke

Third green GKE run in a row: https://jenkins.cilium.io/job/Cilium-PR-K8s-GKE/3013/


Export a snapshot of a running warnet simulation

The longer nodes run, the more rich their states become, such as wallet histories and AddrMan entries. Ideally, a user should be able to stop a simulation and pick back up later, or run a simulation for x days to "seed" certain behavior and then use this as the starting point. For this we would need to export the data directory for each node.
One easy way to do this could be create new contains from existing containers. This would capture everything and save them as images that can be used to restart the warnet from that point later

I think this can be accomplished without too much fuss. Strategy should be to iterate through all containers and use commit to freeze them into new images. The original docker-compose.yml file would need to be modified to load the newly created images instead but I think all the other settings in that file should still work. The consumer will need to load all the images into docker daemon and then compose-up the modified file. There is one other option we should include requested by @carlaKC which is to detach the warnet from real-world time. I'm pretty sure this can be done by disabling the systemd-timesyncd service in all containers. The goal would be to generate some bitcoin and LN activity, then stop time and freeze the warnet at a specific point such that the entire situation can be restarted from the snapshot in the future without the containers having actually lapsed any time.

Would it not just use the host time in docker?

Not sure, but the request is to stop time when the snapshot is created and continue from that exact second no matter when the snapshot is reloaded and continued.

Seems to be some other crazy ways about... https://croz.net/news/using-datefudge-to-fake-docker-date-time-for-testing/

Not sure, but the request is to stop time when the snapshot is created and continue from that exact second no matter when the snapshot is reloaded and continued.

This seems like a reasonable request. Seems like it could be best to test your way first. :)

Done in #555


Helm template typescript function

Addresses https://github.com/GoogleContainerTools/kpt/issues/586

/fixes https://github.com/GoogleContainerTools/kpt/issues/586


morphia MappingException: Error setting value from converter (IdentityConverter)

Due to issues observed with bulk-write, I am trying to upgrade Morphia jar of my project.
From Morphia 0.99 to 1.1.1. After update I am having problem while reading Embedded class objects, which implemented Iterable.
Following is the error I am seeing while reading, persisted record:

java.lang.RuntimeException: org.mongodb.morphia.mapping.MappingException: Error setting value from converter (IdentityConverter)

I am using Mongo 2.7.3, I can not do an upgrade at the moment.

Can you post a test case and we can take a look?
And by Mongo 2.7.3 you mean of the server?  or the driver?

Sorry for the late reply. It's Mongo 2.7.3 server.
I have solved my issue by keeping Morphia 0.99. Closing this issue.


Edit Moneropedia entry 'Coinbase' to clarify it has nothing to do with the exchange

Many confuse it with the exchange.

I was going to start working on this but it looks like there is already a coinbase entry in Moneropedia. Or were you looking for one with more detail than this?
https://www.getmonero.org/resources/moneropedia/coinbase.html

You are right. I think would be good to specify that it's something difference from the exchange. I'm gonna edit the title.
Better to not work on user guides and moneropedia for now. We will change the way they are localized soon (#1270)

I came here because I thought it was talking about coinbase the exchange, and I was going to make an issue saying somebody screwed up the definition.


CommandCallable Command Alias Issue

Description
In CommandCallable, the getHelp method and other methods do not recognize all CommandService registered command aliases.
Proof Explanation
In my example below, I show how my plugin recognizes both the /tp and /teleport commands.  They use the same class, and are simply aliases of each other.  I then do /help for each of the aliases, but only /teleport shows the help, and /tp shows as unrecognized.  There is also a picture showing the line registering the command. Note: I didn't quite catch the first command I ran, but it was /tp.
Registering the Command

Running the Test Process Explained Above

@zml2008 or @JBYoshi
Need you to look into this

@Zidane I'll look into this.

I can't reproduce this. @mrgoodrich Does this still happen with the latest Sponge?
Commands:
/echo Hello World -> You ran the command. Arguments: "Hello World"
/doecho Hello World -> You ran the command. Arguments: "Hello World"
/help echo -> long description
/help doecho -> long description
Code: (Yes I know I'm not using CommandSpec. I wanted to get a working command quickly, so I just had my IDE generate the methods for me.)
package jbyoshi.sponge.test;

import com.google.common.base.Optional;
import com.google.common.collect.ImmutableList;
import org.spongepowered.api.event.Listener;
import org.spongepowered.api.event.game.state.GameInitializationEvent;
import org.spongepowered.api.plugin.Plugin;
import org.spongepowered.api.text.Text;
import org.spongepowered.api.text.Texts;
import org.spongepowered.api.util.command.CommandCallable;
import org.spongepowered.api.util.command.CommandResult;
import org.spongepowered.api.util.command.CommandSource;

import java.util.List;

@Plugin(id = "jbyoshi.sponge.test", name="JBYoshi Sponge Test")
public final class SpongeTestPlugin implements CommandCallable {

    @Listener
    public void init(GameInitializationEvent e) {
        e.getGame().getCommandDispatcher().register(this, this, "echo", "doecho");
    }

    @Override
    public CommandResult process(CommandSource source, String arguments) {
        source.sendMessage(Texts.of("You ran the command. Arguments: \"" + arguments + "\""));
        return CommandResult.success();
    }

    @Override
    public List<String> getSuggestions(CommandSource source, String arguments) {
        return ImmutableList.of();
    }

    @Override
    public boolean testPermission(CommandSource source) {
        return true;
    }

    @Override
    public Optional<? extends Text> getShortDescription(CommandSource source) {
        return Optional.of(Texts.of("short description"));
    }

    @Override
    public Optional<? extends Text> getHelp(CommandSource source) {
        return Optional.of(Texts.of("long description"));
    }

    @Override
    public Text getUsage(CommandSource source) {
        return Texts.of("[args]");
    }
}

@JBYoshi
May be worth ensuring it isn't something wrong with CommandSpec.

Also test with overwriting a Vanilla command unless echo is one lol.

@Zidane Yep, it's a problem with overriding vanilla commands. I changed the init() line to:
e.getGame().getCommandDispatcher().register(this, this, "tp", "echo", "give");

Typing /help tp or /help give fails, but typing /help jbyoshi.sponge.test:tp or /help jbyoshi.sponge.test:give works. It happens with both /tp and /give, so it's regardless of the primary alias. I'll keep looking into it.

Actually, it's a problem anytime there are multiple commands registered with the same alias. I'm about to make a PR for it.


[PE-D][Tester B] UI for event cannot scroll unless it exceeds the size of frame at fullscreen

As title have mentioned, the Ui for Event is only scrollable when it has exceed the size of the frame at fullscreen. For reference, my monitor is 2k resolution at 125% scale. This will cause some inconvenience when the user are restricted from using fullscreen.

Labels: severity.VeryLow type.FeatureFlaw
original: TehOPanas/ped#8

invalid issue

Duplicate of #90


QA: Can it be used with React Native?

This looks very promising I was wondering if it can be used with react-native and possibly expo.
I saw the html elements but they can be easily switched to the react native elements, is there anything that would stop this from working in react native and expo?
Thanks!

Hey, thanks for your interest in this library!
I think it should work in React Native, since there are no HTML elements used in this library, I think the HTML elements you are referring to are in the src/App.jsx file - these file is for the demo of the library, and is not included in the build, meaning if you install this package using npm or yarn you should not have this file, and therefore there shouldn't be any HTML elements, and it should work in React Native.
Let me know if you need further guidance


adventure.img.jpg

Hi, @Saiharshith007! Amazing image - is there somewhere that you would like us to include it?


Regular ncurses without wide support is getting linked into outcurses-demo

Due to a bug in CMake (see https://gitlab.kitware.com/cmake/cmake/issues/19883), we link in the unnecessary -lform, which drags in -lncurses. As a result, wide-character support breaks down :(.

I've switched to using pkg-config for ncursesw for now :/.

To get things working all the way, I still needed to add a setlocale() to outcurses-demo.

Submitted a merge request: https://gitlab.kitware.com/cmake/cmake/merge_requests/3947

Huzzah, my patch made it into CMake 3.16 after all! https://gitlab.kitware.com/cmake/cmake/issues/19883


Stories, functional testing, Cucumber Scenarios

Hey, so I was wondering what's mocha's philosophy on functional/acceptance, high level testing? I understand how to use Mocha for unit testing with describe, it, but how do you use it for higher-level stuff?
Something in the realm of Cucumber.
good add-on syntax for Mocha?
it could be simply like Feature, Scenario, Given, When, Then = describe()
You could write things like, and it'll put everything in a describe()
Feature 'Test',
  "In order to test",
  "as a developer",
  "I use mocha", ->

    Scenario 'BDD', ->
       Given 'should', ->
       When 'true', ->
           it('test'), ->
                 true.should.be.true
       Then 'assertTrue', ->

Edit:
messing around with it now

much like cucumber (at least when I was using ruby) it should be a separate project that just compiles down to whichever test framework

i think there is a js cucumber implementation, maybe we just need bindings for it or something


Add command to move to a set position

The robot will need to move to set positions.  So we need a command to move the arm, wrist, extension to a position.  We can then define the positions in constants.

Added 5 set positions and command to move using xbox buttons


Incorrect prediction from Shufflenet

Hi I tried to run the tutorial using shufflenet however i am getting incorrect predictions on given images I am trying to run attached script after installing shufflenet checkpoint
shufflenet_code.txt
Got following results
results shape:  (1, 1, 1000)
Prediction:  862
Confidence:  0.9999902
Raw top 5 results: [array([862.0, 0.9999902248382568], dtype=object), array([789.0, 3.7160550618864363e-06], dtype=object), array([879.0, 3.410564204386901e-06], dtype=object), array([852.0, 2.5731721962074516e-06], dtype=object), array([462.0, 3.151346371355057e-13], dtype=object)]
Top 5 classes in order: [862, 789, 879, 852, 462]
Batch Shape:  (7, 3, 224, 224)
NCHW image (ready to be used as input):  (7, 3, 224, 224)
Squeezed Predictions Shape, with batch size 7: (7, 1000)
Results for: 'images/cowboy-hat.jpg'
Prediction:  429
Confidence:  0.8792148
Results for: 'images/cell-tower.jpg'
Prediction:  904
Confidence:  1.0
Results for: 'images/Ducreux.jpg'
Prediction:  852
Confidence:  0.94463617
Results for: 'images/pretzel.jpg'
Prediction:  402
Confidence:  1.0
Results for: 'images/orangutan.jpg'
Prediction:  852
Confidence:  1.0
Results for: 'images/aircraft-carrier.jpg'
Prediction:  852
Confidence:  0.9999987
Results for: 'images/cat.jpg'
Prediction:  852
Confidence:  0.6261599
This script however works fine for squeezenet.

Problem is resolved as pre-processing was not configured with required normalize, mean and std.


Provide an option to use classes instead of hardcoded styles

Do you mean the ability to completely disabling of the default styles?

i want to be able, to assign custom classes like
i) selected
ii) selected-start
iii) selected-end
iv) selected-range

I see. Css classnames are already available (.rdr-DateRange, rdr-Calender etc.). But there are no way to disable default styling for now. Having an option like onlyClasses: true can be nice for extended custom styling support. Thanks.


[Event Modification] Codeunit 5055 CustVendBank-Update - OnAfterUpdateBankAccount

Hi, would it be possible to modify the event OnAfterUpdateBankAccount in Codeunit 5055 CustVendBank-Update to insert ContBusRel parameter, like this :
            "No. Series" := NoSeries;
            "Our Contact Code" := OurContactCode;
            "Last Date Modified" := Today;
            OnAfterUpdateBankAccount(BankAcc, Cont, ContBusRel);
            Modify;
        end;
    end;

```

[IntegrationEvent(false, false)]
local procedure OnAfterUpdateBankAccount(var BankAccount: Record "Bank Account"; Contact: Record Contact; var ContactBusinessRelation: Record "Contact Business Relation")
begin
end;
Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


feat(@telicent-oss/ies-service): [TELFE-840] created

Ticket: https://telicent.atlassian.net/browse/TELFE-840
Goal
Stub new package for IES-focused service
Work Completed

Created stub components
Created test with that uses docker
Created a rough README.md

Testing Method
I ran:

npx nx affected:build
yarn test

Out-of-scope

publishing
Any implementation

🕵 Review Guidance

General guidance

Generally, approve a PR if it makes the system better, even if it's not perfect. — Google: The Standard of Code Review
The aim of both PR AUTHOR and PR REVIEWER is to get the code merged
Aim for consensus, defined as everyone can live with the outcome

For PR REVIEWER:

Read the ticket & description
Review the code
Request any changes that are essential.
For non-essential comments:

Use prefixes, e.g. "nit: change to Pascal case"

nit: small, non-essential change
obs: just an observation, doesn't affect the PR
idea: a suggestion to think about
q: questions

Use modifiers, e.g. "obs[pr-owner-resolve]: Jim is also editing this file"

pr-author-resolve PR author can resolve after reading
pr-author-delete (rare) delete after reading to avoid confusion

try to add at least a helpful comment per ~500 lines; less if the PR is already busy

For PR AUTHOR:

Aim for enough detail in PR description for things to go smoothly
After requested changes, re-request a review (so the PR shows up in reviews-requested:@me)

@MonkeyChap

Code style/tech It'd be cool to pair-program with FE team as time allows
RE: The interpretation of IES4: If we produce some easy-to-digest (and approve and modify) artefact for DATA team

could be as simple as a PR, or a series of commits. But could be decoupled utility library/class and unit test suite (current thinking)

@afizzycola  @Helena-telicent FYI see point 2

@MonkeyChap @davec504
Also I'd really love to pull apart all existing  services into files-per-class before we start adding more code


Question about token ttl expiration.

The ttl token, starts to count after the last request and if one request is done the token ttl is renewed? or ttl counts form the last login ?
Thanks
Regards

Hi,
The token is only generated after the form login, there is no concept of "refreshing" or "renewing" in JWT. The TTL is part of the signature so you cannot update it without invalidating the token.
Once the token has expired you must generate a new one, either by asking for the user credentials or programatically.
Regards.

@slashfan Thanks.
Which service can I use to generate new one ?
Thanks

Hi, the only way I can think of would be to bypass the expiration checking, but I wouldn't recommand it.

ok, then I'm thinking to generate new token every x time, and return allways the token in every request.
There is some method to remove the old token ?
Thanks

It could work, you can change the ttl too, or configure your client application to ask for a new token periodically.
For the token invalidation, look at this cookbool entry and the IP flag examples, you should be able to customize the token validation by using the Events::JWT_CREATED and Events::JWT_DECODED events.
For example, you could configure an application wide key or hash, add it to the token payload and change it when you want to invalidate every token in the application.
If you just want to invalidate a specific user, store a user key directly in the user entity and change it when neeeded.

Hi, I'm having trouble with the token validity.
I'm using the default token_ttl value, but users  have to re-connect too often.
Can nelmio_cors "max_age" parameter interfer with the ttl_value and result in such an issue ?
Any idea ?
Thx

The CORS will not help you in this case. When a token expires you have to issue a new one.
Using the gesdinet/JWTRefreshTokenBundle, your client will be able to renew a token with a refresh token.

Thank you for reactivity.
You mean that I need to install whole of another bundle just in order to make token last longer ?
I must be missing something :(

No that bundle does not extend the lifetime of the token.
It adds a flow to issue new tokens with user interaction.
When the bundle is installed an configured, your client will receive 2 tokens:

the token you already receive and that you use for your API calls,
a refresh token to issue new tokens.

At the moment, your client acts as below:

Your client calls the API,
The call is rejected because the token expired,
Your client redirect the user to the login page and get a new access token,
Your client calls the API and is granted

With the refresh token enabled, the step 3 is Your client calls the refresh token endpoint and get a new access token.

@stphane What about first setting the token_ttl with a greater value than the default one?

Thank you for details and advices.
By "reconnect to often" I meant like every 5 minutes.The default value of 84600 should cover an entire day right!?. I now suspect the appGyver developer to not join the token along some requests from inside the mobile application which cause itself to prompt its login form.
I will let you know once the developer has inspected his code.


Log extra_optjons of data import

Add a serialized extra_options attr in the data_imports model and set a common_data: true if the import comes from a common_data

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

@Kartones please CR

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)

:+1:


Start of AVX2 functions

First batch of AVX2 functions, with tests. I've tried to emulate all the style conventions, let me know if I have missed some.

updated

Great, thank you!


Podcasts: add 245 recap transcription

closing in favor of https://github.com/bitcoinops/bitcoinops.github.io/pull/1083


Add vue tests

I think it was a great idea to add some tests
Related:

vuejs unit-testing

If you want you can write some tests :-)
Or at least a setup to test would be nice.


fix: Setting View UI

@Shikhar12233456 you have to merge all the commits into single one. Just google git rebase and follow the steps.

closing for inactivity


NSNotificationCenter Connection invalid

Trying to identify what could potentially be causing the below message when using SSH to run a notification. The notifier works fine when its run locally, VNC session, and Self-Service(JAMF MDM Store).
Have tried everything I can think of, but not sure whats going on. Is it possible this is permissions based? Tried changing it to a few different things but the result is the same. Any help appreciated!
bash-3.2# /Applications/Utilities/yo.app/Contents/MacOS/yo -t "MyCompany" -s "Header" -n "Message ..."
2018-12-05 12:53:06.442 yo[65652:994752] NSNotificationCenter connection invalid
2018-12-05 12:53:06.443 yo[65652:994751] Posting notification.
com.mycompany.yo
2018-12-05 12:53:06.444 yo[65652:994752] Connection to notification center invalid. ServerConnectionFailure: 1 invalidated: 0

Looks like you are calling yo as root. This should be called directly as the end user who is logged into the GUI or via yo_scheduler as root.

Ahh, thanks @sphen13 ... that did it. Appreciate your help!

@stickman112 please share your code
I did the following and still failing
sudo -u $currentUser /Applications/Yo.app/.....

here is a sample that i have
consoleUser=$(/usr/bin/python -c 'from SystemConfiguration import SCDynamicStoreCopyConsoleUser; import sys; username = (SCDynamicStoreCopyConsoleUser(None, None, None) or [None])[0]; username = [username,""][username in [u"loginwindow", None, u""]]; sys.stdout.write(username + "\n");')
consoleUserUID=$(/usr/bin/id -u "${consoleUser}")
launchctl asuser "$consoleUserUID" "$notificationAgent" -z Tink -t "$notificationTitle" -n "Updates have been due for $oldestUpdateDays days, Please Update Now" -b "Update Now" -o "Later" -a munki://updates > /dev/null 2>&1

Failed to get user context: 1: Operation not permitte
when using
currentUser=$(stat -f%Su /dev/console)
launchctl asuser "$currentUser"

please see code above - the launchctl command require a uid not a name - you are supplying a name. This is not a yo issue.

Ok thanks will fix that thanks


Ensure nodes can re-sync if anything goes wrong in AcceptBlock

As observed on testnet, when an error happens during AcceptBlock, the node seemingly stops accepting messages and just hangs. It should be able to re-synchronize with the network in this case.

As was previously proposed by @goshawk-3, it seems that syncs are stalled by the dupemap, which disallows the failed block from being downloaded from a peer. In a recent DevNet run, it was observed that a node stalled at block b3881eca39f2e1cabc77d246f637e26e29c05f1552dab187cd8e7c5c31d47342, due to a problem related to #973. After enabling trace logging, it was found that the previous hash, and hashes of subsequent blocks are all being rejected, which clearly explains the problems with re-syncing.
Merging #972 should therefore close this issue.


Hash Table: Add Integer to Roman in Python

https://leetcode.com/problems/integer-to-roman/

I will take this up.


Update Failsafe Error data

Details in https://app.clubhouse.io/rollbar/story/14288

Hi there, I’m closing out all issues opened before 2018 that haven’t had any activity on them since the start of this year. If this is still an issue for you, please comment here and we can reopen this. Thanks!


tf.reset_default_graph() doesn't work on kubernetes?

In my understanding reset_default_graph() will reset the whole graph. However, it seems that this will only reset the value of variables when running parameter servers on kubernetes.
First, I tried to get variable values before and after reset_default_graph(). The value is reset (value is 0) after running reset_default_graph().
Then I tried to run a totally different algorithm using the same parameter server, it reported the error below:
Assign requires shapes of both tensors to match. lhs shape= [5,5,32,64] rhs shape= [50000]
Only chief run the reset_default_graph()  cmd. The code is:
is_chief = (FLAGS.worker_index == 0)
if is_chief: tf.reset_default_graph()

The whole error message is:
Traceback (most recent call last):
  File "word2vector.py", line 160, in <module>
    with sv.prepare_or_wait_for_session(FLAGS.worker_grpc_url, config=sess_config) as sess:
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/supervisor.py", line 685, in prepare_or_wait_for_session
    config=config, init_feed_dict=self._init_feed_dict)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/session_manager.py", line 163, in prepare_session
    sess.run(init_op, feed_dict=init_feed_dict)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 340, in run
    run_metadata_ptr)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 564, in _run
    feed_dict_string, options, run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 637, in _do_run
    target_list, options, run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 659, in _do_call
    e.code)

tensorflow.python.framework.errors.InvalidArgumentError: Assign requires shapes of both tensors to match. lhs shape= [5,5,32,64] rhs shape= [50000]

         [[Node: Variable_3/Assign = Assign[T=DT_FLOAT, _class=["loc:@Variable_3"], use_locking=true, validate_shape=true, _device=
"/job:ps/replica:0/task:3/cpu:0"](Variable_3, zeros_S1)]]
Caused by op u'Variable_3/Assign', defined at:
  File "word2vector.py", line 127, in <module>
    nce_biases = tf.Variable(tf.zeros([vocabulary_size]))
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variables.py", line 209, in __init__
    dtype=dtype)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variables.py", line 308, in _init_from_args
    validate_shape=validate_shape).op
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_state_ops.py", line 40, in assign
    use_locking=use_locking, name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/op_def_library.py", line 655, in apply_op
    op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2154, in create_op
    original_op=self._default_original_op, op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1154, in __init__
    self._traceback = _extract_stack()

Variables sizes do not get reset. Variables buffers are allocated in a
container, which is a property of a session. There's no way to destroy a
variable in a session, so to reset size you need to create new session, or
create new variable (use different name)
On Tue, Jun 28, 2016 at 11:50 AM, 郑泽宇<EMAIL_ADDRESS>wrote:

In my understanding reset_default_graph() will reset the whole graph.
However, it seems that this will only reset the value of variables when
running parameter servers on kubernetes.
First, I tried to get variable values before and after
reset_default_graph(). The value is reset (value is 0) after running
reset_default_graph().
Then I tried to run a totally different algorithm using the same parameter
server, it reported the error below:
Assign requires shapes of both tensors to match. lhs shape= [5,5,32,64]
rhs shape= [50000]
Only chief run the reset_default_graph() cmd. The code is:
is_chief = (FLAGS.worker_index == 0)
if is_chief: tf.reset_default_graph()
The whole error message is:
Traceback (most recent call last):
File "word2vector.py", line 160, in
with sv.prepare_or_wait_for_session(FLAGS.worker_grpc_url, config=sess_config) as sess:
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/supervisor.py", line 685, in prepare_or_wait_for_session
config=config, init_feed_dict=self._init_feed_dict)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/session_manager.py", line 163, in prepare_session
sess.run(init_op, feed_dict=init_feed_dict)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 340, in run
run_metadata_ptr)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 564, in _run
feed_dict_string, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 637, in _do_run
target_list, options, run_metadata)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 659, in _do_call
e.code)
tensorflow.python.framework.errors.InvalidArgumentError: Assign requires shapes of both tensors to match. lhs shape= [5,5,32,64] rhs shape= [50000]
     [[Node: Variable_3/Assign = Assign[T=DT_FLOAT, _class=["loc:@Variable_3"], use_locking=true, validate_shape=true, _device=

"/job:ps/replica:0/task:3/cpu:0"](Variable_3, zeros_S1)]]
Caused by op u'Variable_3/Assign', defined at:
File "word2vector.py", line 127, in
nce_biases = tf.Variable(tf.zeros([vocabulary_size]))
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variables.py", line 209, in init
dtype=dtype)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variables.py", line 308, in _init_from_args
validate_shape=validate_shape).op
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_state_ops.py", line 40, in assign
use_locking=use_locking, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/op_def_library.py", line 655, in apply_op
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2154, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1154, in init
self._traceback = _extract_stack()
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/3073, or mute the thread
https://github.com/notifications/unsubscribe/AABaHKDyt4ZMvgAkoMYlbJv0G39xETpTks5qQOBygaJpZM4I_4JN
.

how could I create a new session without restarting the server?

Test testSameVariablesClear
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/training/server_lib_test.py#L74
looks
relevant
IE, tf.Session.reset(target)
On Tue, Jun 28, 2016 at 5:13 PM, 郑泽宇<EMAIL_ADDRESS>wrote:

how could I create a new session without restarting the server?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/3073#issuecomment-229061600,
or mute the thread
https://github.com/notifications/unsubscribe/AABaHII6C1_5DEMLg0WYxAROaS9o_MWeks5qQSwhgaJpZM4I_4JN
.

It seems that I can't find tf.Session.reset in both github api doc and tensorflow.org api doc
When I tried to run the test code, it also reports error:
AttributeError: type object 'Session' has no attribute 'reset'

I tried to run locally with tensorflow 0.9.0 and python 2.7; I also tried with docker image tensorflow/tensorflow:r0.9;  both of them reported the same error message.

btw, if I use different name scopes for different runs, will the server stores all the variables from different runs and uses lots of disk?

From GIT blame, it seems that test was added 5 days ago. TensorFlow version
0.9.0 is several weeks old so you need to build from source.
You can select which variables are saved by specifying it in Saver
constructor
On Wed, Jun 29, 2016 at 4:42 AM, 郑泽宇<EMAIL_ADDRESS>wrote:

btw, if I use different name scopes for different runs, will the server
stores all the variables from different runs and uses lots of disk?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/3073#issuecomment-229233828,
or mute the thread
https://github.com/notifications/unsubscribe/AABaHJmYPjsBLu0mPsbsTnVNNp1VF5yBks5qQc2AgaJpZM4I_4JN
.

Thanks. At least it worked using different name scope.


Bug: cannot import name 'raise_fastapi_validation_error'

Describe the bug
Error running faststream CLI.  Error raised on all commands
How to reproduce
Include source code:
faststream --version

And/Or steps to reproduce the behavior:

run any faststream cli command

Expected behavior
Should report faststream version
Observed behavior
Exception raised
Screenshots
root@8761fe349f39:/var/app# faststream --version
Traceback (most recent call last):
  File "/usr/local/bin/faststream", line 5, in <module>
    from faststream.__main__ import cli
  File "/usr/local/lib/python3.11/site-packages/faststream/__main__.py", line 1, in <module>
    from faststream.cli.main import cli
  File "/usr/local/lib/python3.11/site-packages/faststream/cli/main.py", line 10, in <module>
    from faststream.cli.docs.app import docs_app
  File "/usr/local/lib/python3.11/site-packages/faststream/cli/docs/app.py", line 10, in <module>
    from faststream.asyncapi.generate import get_app_schema
  File "/usr/local/lib/python3.11/site-packages/faststream/asyncapi/generate.py", line 17, in <module>
    from faststream.broker.fastapi.router import StreamRouter
  File "/usr/local/lib/python3.11/site-packages/faststream/broker/fastapi/__init__.py", line 1, in <module>
    from faststream.broker.fastapi.route import StreamMessage, StreamRoute
  File "/usr/local/lib/python3.11/site-packages/faststream/broker/fastapi/route.py", line 29, in <module>
    from faststream._compat import raise_fastapi_validation_error
ImportError: cannot import name 'raise_fastapi_validation_error' from 'faststream._compat' (/usr/local/lib/python3.11/site-packages/faststream/_compat.py)

Environment
Include the output of the faststream -v command to display your current project and system environment.
faststream 0.2.5
Same error in both python 3.11 docker container and in WSL with python 3.11.5
Additional context
Provide any other relevant context or information about the problem here.

@kreneskyp can you please add reprodusable dockerfile?

@kreneskyp I just tried the same in the fresh virtual environment and here is my output:
$ faststream --version
Running FastStream 0.2.5 with CPython 3.11.5 on Linux
$ python --version
Python 3.11.5
$ pip list
Package           Version
----------------- -------
annotated-types   0.6.0
anyio             4.0.0
click             8.1.7
fast-depends      2.2.1
faststream        0.2.5
idna              3.4
pip               23.2.1
pydantic          2.4.2
pydantic_core     2.10.1
setuptools        65.5.0
sniffio           1.3.0
typer             0.9.0
typing_extensions 4.8.0
uvloop            0.18.0
watchfiles        0.21.0

[notice] A new release of pip is available: 23.2.1 -> 23.3
[notice] To update, run: pip install --upgrade pip
(.env-3.11) davor@tr3970x-1:/work/davor/projects/airt/faststream$

Can you please share the result of pip list?

Tracked it down to an incompatibility with pydantic 1.10.  I need finish updating to v2 on my end.

@kreneskyp unfortynatelly, this is not PydanticV1 compatibility bug. Sometimes this function returns True even if library is not installed and this occured in your case. But I can't reproduce this behavior regularly and have no idea why python lie to me...

Hmm. In any case I worked through my remaining Pydantic v2 issues and was able to successfully run faststream.  Possibly related to another lib that needed an update?
root@1f1e0a5187a3:/var/app# faststream --version
Running FastStream 0.2.5 with CPython 3.11.6 on Linux

I also updated FastAPI with Pydantic 2.x.  It's working now so no need to investigate further.  Thanks!


Support scanspec deserialization in plans with pydantic

With apischema we could have Spec[Movable] and custom converters to deserialize a scanspec while simultaneously looking up devices. This has proved hard with pydantic and has required several workarounds. See plans.py in https://github.com/DiamondLightSource/blueapi/pull/90/files

Some background:
The bluesky worker takes the parameters of a plan and internally creates a pydantic model for future validation e.g.
def my_plan(a: int, b: str = "b") -> Plan
    ...

# Internally becomes something like

class MyPlanModel(BaseModel):
    a: int
    b: str = "b"

That way, when the plan parameters are send in JSON form, they can be parsed and validated by pydantic and we don't have to worry. However, what happens when your plan doesn't take a simple dictionary, list or primitive? What happens if it takes, say, a detector as an argument?
def my_plan(a: int, b: Readable) -> Plan:
    ...

We don't want to pass the whole Ophyd object as JSON because it has state. We need to pass a reference to it that we know the worker understands. At the moment we pass its name according to the worker's BlueskyContext, so we want some intermediary process to perform that lookup when needed. Before pydantic we used apischema, which had an ideal feature for this called conversions.
I have written a similar feature of blueapi to work with pydantic called type validators:
source
tests
so that our dynamically generated pydantic model can look roughly like this:
def my_plan(a: int, b: Readable) -> Plan:
   ...

# Becomes

class MyPlanModel(BaseModel):
   a: int
   b: Readable

   @validator("b")
   def valdiate_b(self, val: str) -> Readable:
       return ctx.find_device(val)

Which is all well and good, but what about the Readable type being placed at various type levels? For example:
def my_weird_plan(
    a: Readable,
    b: List[Readable],
    c: Dict[str, Readable],
    d: List[List[Readable]],
    e: List[Dict[str, Set[Readable]]]) -> Plan:
    ...

The code also copes with all of the above, as seen in the tests. The only thing left to do is make it work with scanspecs.
The following already works
def my_plan(spec: Spec[str]) -> Plan:
    ...

In the plan, you can access a scanspec object where the axes are strings, which is helpful, but ideally you want them to be motors, so we need
def my_plan(spec: Spec[Movable]) -> Plan:
    ...

This is the last remaining case.


[WFLY-11455] Validate requirement for modules previously exported by javax.ejb.api on org.jboss.narayana.txframework

Althoug it is a deprecated module, we can clean up unused dependencies beside of those that were being exported by javax.ejb.api. I have not found direct uses from org.jboss.narayana.txframework:txframework on any of the removed dependencies.
Jira issue: https://issues.jboss.org/browse/WFLY-11455
cc: @ochaloup

perfect, nice.


Update attribute type Ltree error

Error syntax in update attribute type Ltree with alter directly table.. Only being able to update via sql query

Please tell DBeaver version, your database and steps to reproduce.

bug fixed in version 21.0.1


adds animatedContainer extension

Description
The PR adds basic AnimatedContainer extension on Widgets. It also adds mixin for Alignment, duration, curves which can be used for other widgets too. Remove the redundant code of Alignment from VelocityBox and uses Alignment mixin.
Syntax
      Container()
            .animatedBox
            .width(width)
            .height(height)
            .color(color).easeIn.milliSecs(milliSec: 500)
            .make()
            .centered()

Image

super

Superb @AyushBherwani1998