Fixed updating roles when custom user or port is specified.

In my previous PR #1223 I unintentionally reverted the fix from PR #1157, because fix there had no test and I missed it.
Here is my attempt to fix both issues #1071 and #1214, with tests for custom user/port.

Thanks @ayastreb for consolidating these fixes. Please try master, and make sure it fixes your issues, report back and I'll try and make time to prepare a release this weekend.

Hi @leehambley !
I only rely on the tests and I can't say if it works in real life setup :)
We will have to ask guys from #1157, #1214 and #1232

I can confirm it works in a real-life setup :+1:

Cheers, I'll check for making a new release soon.


Custom resource file for Fonts [Question]

Our resource file is currently at 138MB (167MB with the latest version of GUIX studio) and we've seen an out of memory issue in our customers CI when compiling. So I was looking into splitting the file up to make it faster to compile and avoid issues in the CI.
I've found the following text in the current documentation (similar to the documentation of the GUIX version we're actually using):

For Pixelmap and Font resources, you can specify a custom output filename for each pixelmap and font in theassociated resource editing dialogs. This feature allows you to put very large resources in distinct files, ratherthan putting all resources in one common output file. If you do not specify an overridden filename for a font orpixelmap resource, those resources are written into the common resource file.

I did indeed found a way to pull out the pixelmaps, however I wasn't able to find a way how to move the fonts to a custom file and the font specific documentation also doesn't mention this. Is this a removed feature? Or in the backlog? We'd really like to avoid writing a script to do the split up manually...Or am I just missing where/how this is done?

Hmmm... look like this was never implemented for fonts, I'm not sure why. I've added this task to the backlog. It may make it into the April update, but I can't promise that we have a pretty full task list for this update already.

This feature has been added in GUIX Studio version <IP_ADDRESS>.

@StefanHengeleinLumira please the link https://aka.ms/azrtos-guix-installer to install the latest GUIX Studio.  Let us know if the new version addresses your issue. Thanks!


Docs: installation-astro.mdoc: Add hint at rendering collections

I paused when I saw that the guide only renders one file. This PR links over to the astro docs which will lead one to create the rendering vor all posts.
I thought about adding a section with code right here, but the having this collection rendering makes most sense when one also has a layout file and that is too much to introduce right here, IMO. But users can figure this out in the great Astro guides easily.

Yep, agree this is useful but adding a code example would be too much, and deriving from the simplicity of this guide.
Thanks for this addition 👍


Refactor identity crate

This is an attempt to clean up the identity crate.
Open question: How can we make the top level API clearer. Right now there are three different authentication flows (and their associated types) all living in the top level namespace. It would be nice if it were more obvious to users how to use this crate.
To get an idea of how this crate now works, I'd suggest running  cargo doc and opening the associated generated documentation.

Open question: How can we make the top level API clearer. Right now there are three different authentication flows (and their associated types) all living in the top level namespace. It would be nice if it were more obvious to users how to use this crate.

Yes my goal was to have the authentication flows working (the certificate auth during client credential flow is still missing) and then clean them up. We also need to replace the naive_redirect_server with something more reliable.
I think that keeping the auth flow names in sync with the official docs makes it easier to understand what the functions are supposed to do (they are not casual APIs, one needs to know what is doing or they probably won't work). Maybe we can move each flow in its module? Also a scope module would be nice (right now we are just accepting plain strings).


Missing Documents for Ionic Hosting

I'm submitting a ...  (check one with "x")
[ ] bug report
[ ] feature request

URL:

Current behavior:

Expected behavior:

Steps to reproduce:

Other information:

Issue moved to USER.


Cooler example for drenv

Running a while loop in busybox pod is not very exciting. It would be more interesting if
we do something like:

Enable ingress addon in minikube
Deploy nginx on both cluster
Configure a test html page on both cluster with the cluster name
Deploy service exposing nginx to the world via the ingress
When the drenv start example.yaml finish, open a browser showing the deployed we page.

From a standpoint of workloads, testing should include workload patterns and operator generated workloads (say kafka operator), which are more critical form a functionality point of view.
Just stating it here, so that if possible the example from a demo perspective can be cooler and possibly also test a more complex workload than a bare pod.

I will work on this


feat: add skip registry option to init

What issue type does this pull request address? (keep at least one, remove the others)
/kind feature
Does this pull request has user-facing changes? (e.g. config changes, new/modified commands, new/modified flags)
Yes
Does this pull request add new dependencies?
No

Force pushed for squashing and rebasing the PR.


Use with UITableViewController or UICollectionViewController?

Is there any suggestion for using this with a UITableViewController or UICollectionViewController?

I just tried an experiment where I made StatefulViewController an extension of UIViewController instead of a subclass meaning that it provides its functionality to any UIViewController (whether you want or not). It seems to work fine with these caveats:

viewWillAppear can't be overridden by an extension so instead, view controller implementations need to call the replacement method statefulViewWillAppear().

As an extension, all UIViewControllers now inherit the computed properties defined by StatefulViewController (currentState, lastState, loadingView, etc.).

In order to store properties (which aren't supported by extensions), I use objc_setAssociatedObject.

I'm not a big expert on Swift yet, so I can't say if this is a viable solution. It seems a bit kludgy. The other thought is that StatefulViewController could be a protocol.

This sounds very promising indeed. Do you mind sharing your code or even open a WIP pull request?

See PR https://github.com/aschuch/StatefulViewController/pull/8

Hi @markkrenek, based on the changes of protocol extensions in Swift 2.0, I decided to turn StatefulViewController into a protocol instead of directly providing a UIViewController extension.
This gives us the flexibility of an extension, but not every UIViewController inherits properties that may never be used.
Would love to get your feedback on my PR #10.
:heart:


 I am bit lost in understanding your code. You open the GameDetails with the navigator from GamePanel. In the route you place game and date. In the GameDetails you get actions from the props. Where do they come from?
Thank you

@andtur
This is a good question. To put it simply, I made use of Navigator.
You can see in app > containers > Game.js line 22, every component in Game page is supported(rendered) by Navigator, and the same props is passed in by Navigator.
You may wonder where {...this.props} come from, you can check the App.js in containers folder, line 41, it comes from Redux store.


Fix native interactive window export to preserve outputs and omit sys info cells

For https://github.com/microsoft/vscode-jupyter/issues/6779
@DonJayamanne FYI, we cannot simply use the notebook serializer to convert NotebookDocument to bytes/string like I initially thought, because we do all kinds of stuff to manipulate the notebook cells before serializing and writing it to disk e.g. inserting change directory, and we would need to rewrite all of that functionality for NotebookCell/NotebookDocument (I started down this path and had touched 10 files after an hour).
@rebornix had a proposal that we should rewrite our export to deal with nbformat directly rather than ICells, which I agree with. In the meantime this PR is a localized fix for the original problem (export dropping notebook cell outputs) to avoid introducing regressions for native interactive window, but when we create an npm package with shared code for converting NotebookData to nbformat and delete ICell https://github.com/microsoft/vscode-jupyter/issues/6488, this will go away

sal that we should rewrite our export to deal with nbformat directly rather than ICells,

Agreed, we'd first need to get rid of the noise and the consider building a better solution.

Codecov Report

Merging #6784 (320f687) into main (8dfc17c) will decrease coverage by 0%.
 Consider uploading reports for the commit 289f482 to get more accurate results

@@          Coverage Diff          @@
##            main   #6784   +/-   ##
=====================================
- Coverage     68%     68%   -1%
=====================================
  Files        411     411
  Lines      28626   28628    +2
  Branches    4280    4280
=====================================
- Hits       19681   19668   -13
- Misses      7245    7259   +14
- Partials    1700    1701    +1

Impacted Files
Coverage Δ

src/client/datascience/export/exportDialog.ts
35% <0%> (ø)

src/client/datascience/jupyter/kernels/kernel.ts
64% <0%> (-3%)
:arrow_down:

src/client/datascience/cellFactory.ts
65% <50%> (-2%)
:arrow_down:

...datascience/interactive-common/notebookProvider.ts
81% <0%> (-6%)
:arrow_down:

.../datascience/jupyter/kernels/cellExecutionQueue.ts
89% <0%> (-5%)
:arrow_down:

src/client/common/application/applicationShell.ts
18% <0%> (-5%)
:arrow_down:

src/client/datascience/raw-kernel/rawSocket.ts
84% <0%> (-2%)
:arrow_down:

...t/datascience/notebook/vscodeNotebookController.ts
79% <0%> (-1%)
:arrow_down:

src/client/datascience/baseJupyterSession.ts
66% <0%> (-1%)
:arrow_down:

... and 5 more


zib-VisualAcuity

Ticket ZIB-1441 for missing SNOMED code
Ticket ZIB-1442 for reference to AnatomicalLocation that should be AnatomicalLocation::Laterality (or AnatomicalLocation::Location should be fixed to 'eye')

Reviewed and made some tweaks to the profile right away. One open conversation which is kind of handled for now..

Lokale validatie is in orde


Warp flow reference models

Hi @yjxiong ,
Could you please provide the reference models for warp flow? I noticed that the scripts/get_reference_models.sh only download rgb and flow models.
Thank you so much in advance.

Do you come across the error when you install dense flow @dougsoares

Sorry, we don't provide that. Warp flow is costly to use and provides only marginal accuracy improvements.


Remove 3.6 classifier in pyproject.toml

Proposed changes:

Fixes https://github.com/RasaHQ/rasa/issues/10138

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

actually @TyDunn we don't need to do anything for Python 3.9, it is handled automatically by poetry (see my comment in the code)


Corrected syntax issues with bash file

Fixes #107, tested using PyScaffold (3.0a2) and on Travis
Removing extra CRLF before elif as is content within the elif

Coverage remained the same at 95.638% when pulling ae160afd3e0561417d7a8ce45c2d38451a94cd08 on willu47:bug/travis into 082b2e15ad9b07ef79ef0ec925590f238425884c on blue-yonder:master.

@willu47 Thanks! I just released v2.5.8rc3. Hopefully, this one makes to a final release. Also I cherrypicked your PR onto v3.x but will release it later.


DOC: Incorrect link to ARPACK

Issue with current documentation:
The link http://www.caam.rice.edu/software/ARPACK/ which is present in the documentation for scipy.sparse.linalg.eigs and scipy.sparse.linalg.eigsh, and at https://scipy.github.io/devdocs/tutorial/arpack.html leads to a 404 error on the Rice University website

Indeed, thanks. Would you be interested in fixing it?

Hi, Can you please explain which link we have to provide instead of " http://www.caam.rice.edu/software/ARPACK/"?

Hello, @j-bowhay ... I researched, and also thanks to @ThijsDrost for the statement. Ya, that's true ARPACK is now moved to Github : https://github.com/opencollab/arpack-ng, but in the documentation needs to update, So, I would like to work on this and want to change some paths like ...

Hi I would like to work on this issue. @Saksham093 are the links provided resembles the correct link to be updated.

Hey @Papaganti-sravya ... You can work, I think their is not active on this issue.
Do some research and fix it.

Hi, @Saksham093 Glad for the oppurtunity provided  to work on this issue, Do I have any person to talk on this issue for further assistance, share my thoughts and get this fixed.

Hi @Saksham093  I have done some resech and got these links, please check and let me know if any of these links can be useful.
http://rsusu1.rnd.runnet.ru/libraries/ARPACK/node6.html
https://people.sc.fsu.edu/~jburkardt/f_src/arpack/arpack.html

Hi @Saksham093 If you feel like these links
http://rsusu1.rnd.runnet.ru/libraries/ARPACK/node6.html
https://people.sc.fsu.edu/~jburkardt/f_src/arpack/arpack.html
works as a replacement of https://cmor.rice.edu/software/ARPACK/  for reference let me know I will update on the main page.
Thank you
Sravya

@Papaganti-sravya as far as I know scipy uses (the still maintained) arpack-ng, which is located at: https://github.com/opencollab/arpack-ng

Hi @ThijsDrost this is my first contribution can you guide me and let me know if i just have to update https://github.com/opencollab/arpack-ng on the referece part of https://scipy.github.io/devdocs/tutorial/arpack.html
Thank you

@Papaganti-sravya I neither know how to do it

Hi @Saksham093 Can you provide me some ideas on how to solve this issue, that would be much appreciated.
Thank you
Sravya

Hi @melissawm Thank you so much for the information It is really helpful. The link for the reference (reference.) is not working do you have any reference link to update in all threee documents.

I believe the reference is now https://github.com/opencollab/arpack-ng
However, it does seem odd that going through the Rice website, there are still references to this page (maybe there is some internal error?) I have reached out via email to the Rice University folks to see if they can clarify this. In the meantime, adding the link to the arpack-ng github repository is a nice addition. Thanks!

Here's the official response:

Dear Melissa,
Unfortunately, we are no longer the official link to ARPACK since Professor Sorensen retired several years ago.
As far as know, due to stalled upstream development, ARPAСK has been forked into ARPACK-NG as a form of a collaborative effort of the various groups that rely on ARPACK.
https://github.com/opencollab/arpack-ng
Regards,
CMOR Staff

Is this issue still active or has it been resolved?

Resolved by #18173


Accordion doesn't re-collapse unless allow_multiple=True

Describe the bug
If you use the Accordion component without setting allow_multiple=True, even if you only have one item, you can open it but it won't collapse again when you click it. You can see this for yourself in the documentation: https://pynecone.io/docs/library/disclosure/accordion
To Reproduce
Steps to reproduce the behavior:

Create an Accordion
Don't include allow_multiple=True
Click to open the Accordion; it opens
Click to close it; it doesn't close
Set allow_multiple=True
Repeat the process and it will work

Expected behavior
Accordion should both open and close without needing to specify allow_multiple=True.
Screenshots
Just go to https://pynecone.io/docs/library/disclosure/accordion.
Specifics:

Python Version: 3.11.4
Pynecone Version: 0.1.33
OS: MacOS Ventura 13.4
Browser (Optional): Chrome 114.0.5735.106

Did you use allow_toggle if you are omitting allow_multiple ?
If not, then this is normal behaviour.

Ah, it would indeed seem that this was the issue and I should have read the documentation more closely.
Unless my read of the code history is wrong, it looks like the default for allow_toggle used to be True and was very recently switched.
I had repurposed some code and when it wasn't working assumed there was a bug. Sorry for the fire drill.

Yeah we had to put a fix recently because using allow_multiple was conflicting with allow_toggle and triggering lot of warning in the console on frontend.


update: enabled cose to verify time in Tag1 Datetime format

This PR enables COSE to verify time in Tag1 Datetime format.
Some details that worth to be mentioned:
Sign (Encode/MarshalCBOR):
in go-cose, time.Time will be encoded as numerical (in integer) representation of seconds since January 1, 1970 UTC.
For more details: https://github.com/fxamacker/cbor/blob/7704fa5efaf3ef4ac35aff38f50f6ff567793072/encode.go#L75
https://github.com/veraison/go-cose/blob/f72b6bc06b90d1e9c07ab11332df92c7838e3027/cbor.go#L30
Verify (Decode/UnmarshalCBOR):

CBOR times (tag 0 and 1) decode to time.Time. (This applies to COSE signatures provided by PRSS/ESRP.)
decode CBOR uint/int to Go int64.  (This applies to signatures signed by Notation Sign. Although, user passes time.Time into signRequest in Notation Sign, after MarshalCBOR (during Sign) and UnmarshalCBOR (during Verify), this input time.Time is actually converted to an int64 by go-cose.)
For more details: https://github.com/fxamacker/cbor/blob/7704fa5efaf3ef4ac35aff38f50f6ff567793072/decode.go#L52
https://github.com/veraison/go-cose/blob/f72b6bc06b90d1e9c07ab11332df92c7838e3027/cbor.go#L43

Signed-off-by: Patrick Zheng<EMAIL_ADDRESS>

Codecov Report

Merging #95 (7cd80be) into main (cd56ef6) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #95      +/-   ##
==========================================
+ Coverage   78.76%   78.85%   +0.09%
==========================================
  Files          28       28
  Lines        2067     2076       +9
==========================================
+ Hits         1628     1637       +9
  Misses        336      336
  Partials      103      103

Impacted Files
Coverage Δ

signature/internal/base/envelope.go
97.72% <ø> (ø)

signature/cose/envelope.go
96.31% <100.00%> (+0.08%)
:arrow_up:

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


Docs: add reference to canal

Flannel doc needs reference to "canal" for SEO.

Approved by Alex
Merge to nightly
Merge docs to Enterprise

Deploy preview for calico ready!
Built with commit 2f1244bcbcfd2ddea0f2a260ebb80bcafb5abad0
https://deploy-preview-3444--calico.netlify.app


Fix elastic search plugin not work.

Miss ComponentsDefine and pom, I miss that because of merging. Sorry.

Coverage decreased (-0.004%) to 14.193% when pulling 65749751ec3b71bec925a9f793c9e5388e7e63dc on es-fix into<PHONE_NUMBER>97de66f3eabf952b24b13184f6e915 on master.


Add USER_EMAIL_FIELD as a workaround for Django < 1.11

For now we will need to provide USER_EMAIL_FIELD (at least until Django 1.8 goes out of life).
Based on #206

Done.


Avoid array temporaries

Fortran creates temporary arrays when passing sub-ranges to subroutines.

We could also get rid of these warnings by explicitly creating a temporary variable ourselves.

What do you think is best practice/most efficient? I would be OK with either solution. Guess this is the next one to merge after the period BCs PR.

Avoiding temporary arrays is definitely more efficient as we do not copy memory. The indices -1 and nz+1 and we can either set them to zero or (maybe better) use linear extrapolation.

My suggested change: https://github.com/EPIC-model/epic/pull/504/commits/5759bceae46483e0074891346e54e4fa4acc5bb6

Let's use the extrapolation, which I agree should be safer.


WIP: Consume rte e2e

This PR contains the necessary changes in order to consume RTE tests and run them:

Split suite tests - We would like to reuse the basic_install tests in order to set up and test the environment for the other tests.
We'll consume the other tests from https://github.com/k8stopologyawareschedwg/resource-topology-exporter and run them on the environment that prepared in advanced by basic_install suite.

Update GH action - Run the additional RTE e2e tests.
Since we have more than a single test suite we can't use the binary version.
We're using 'go test' to run the suites directly instead.

Bump RTE dependencies to v0.2.7

WIP:
We need the operator to deploy RTE image v0.2.7 which we don't have yet.

/retest

  Running [/home/runner/golangci-lint-1.41.1-linux-amd64/golangci-lint run --out-format=github-actions --timeout=15m0s --verbose] in [] ...
  Error: SA4023: this comparison is always true (staticcheck)
  Error: SA4023(related information): the lhs of the comparison is the 1st return value of this function call (staticcheck)```

Getting this error in the ```CI GO / lint``` lane, but it's not clear from the log where the issue is.

  Running [/home/runner/golangci-lint-1.41.1-linux-amd64/golangci-lint run --out-format=github-actions --timeout=15m0s --verbose] in [] ...
  Error: SA4023: this comparison is always true (staticcheck)
  Error: SA4023(related information): the lhs of the comparison is the 1st return value of this function call (staticcheck)

Getting this error in the CI GO / lint lane, but it's not clear from the log where the issue is.

@fromanirh Any idea why/where it's happening? the error doesn't provide the line of the faulty code

Getting this error in the CI GO / lint lane, but it's not clear from the log where the issue is.

@fromanirh Any idea why/where it's happening? the error doesn't provide the line of the faulty code

Indeed, we should improve the lane to have better output. Meantime it's just simpler to run the tool in your local checkout.

Ah, nice, the lint errors are displayed inline: https://github.com/openshift-kni/numaresources-operator/pull/11/commits/a3fe88171680f5a0de9cbc57426960d0df61f4e0

Ah, nice, the lint errors are displayed inline: a3fe881

What do I see here? it's just the last commit I added

Ah, nice, the lint errors are displayed inline: a3fe881

What do I see here? it's just the last commit I added

No sure I see this because I have admin rights on the repo, though.

/lgtm


Remove rootPageId parameter from TableFile constructor

Closing pull request since additional unrelated work has been added since it was created. The intended content of this pull request will be made via another temporary branch off of the original commit.


Getting Server cannot append header after HTTP headers have been sent

Hi,
I am getting Server cannot append header after HTTP headers have been sent when I try to sync the roles with unicorn. Any ideas?
Regards
Danny

Duplicate of #155. This is fixed in the current prerelease version already.
I am assuming that you're using Sitecore 8.1U3 or 8.2. You can work around this by setting the 'X-Frame-Options' header yourself prior to Sitecore trying to set it on request end.


Transposed Numbers

While running tests on a project, I've noticed that a test is failing intermittently when entering credit card numbers into an input field that uses cc-number.  I've ran the test and have seen that while the card number is being entered, the pointer position is set somewhere in the middle of the string leaving the final number to be all wrong.  This was reproduced once manually.
We are using version 3.1.6 of this project.  I've tried upgrading to version 3.1.8 and am still seeing this issue.
Browser: Chrome 69.0.3497.81
One of the failing numbers being entered:<PHONE_NUMBER>139424
Number being set in the input field:<PHONE_NUMBER>199010

Hi, I'd welcome a PR. I'm unlikely to have time to look into this but will leave it open for a bit in case you'd like to contribute.

Just a quick update: We (@awesomeousPrime and I) think we might have a solution. We'll let you know.


More typings

All errors done. Please merge this. It has no new feature. If you like to add more typings you can do this in another pull request.

@normanjaeckel I added the option check_untyped_defs to check code outside of utils. This option can be removed when the entire project has type annotations. I added one #TODO (without the space) with a question. What should we do in this case?

All done and rebased. I merge after travis is ok


Switching role

When taxi asked my if I want to assign an activity to a role:

I mostly say no, because there are no easy way to change my mind... But this is sad, because I have to answer again and again the same questions...
(In the screenshot above, I use my BizDev role, because I working on recruiting a new bizdev. This will be the case for the next weeks/months, but it will no more be the case in 3 months...)
Can we imagine some alternative? What I can think about are:

One more option, "Yes remember it for the next XX days", so that I can safely save it for 1 week or 1 months
One more option, "Yes, but ask me to confirm anyway", so that on the next activity usage I can just type enter to confirm it

WDYT?

One suggestion from Tom:

Have a new option yes but ask from [c]onfirmation. This would allow to set a role, use it in 90% of the case, and if during the confirmation, you decide to pick an other role, then you go again on the normal process

If I understand you use case correctly, you'd like to avoid having to enter the role to use on a given activity, while being able to change it at some point.
I think the role selector shouldn't be more difficult to use as it is now (it would also not be that easy to implement this new "ask for confirmation" option). I think an easier (and more elegant) way to solve your problem would be to propose a default role for the "Select a role" question, which would be the role you used in your last timesheet on this activity.
That way, you would answer "Never" when getting asked for always using this role, and you'd just have to hit "enter" on the role selection on subsequent pushes on this activity. And when you want to change it, you'll just select a different role instead of hitting "enter".
This would require having a new Zebra endpoint, which would take an activity id, and return either the last role id used for this activity, or the last X timesheets created with this activity (and taxi-zebra would then take the most recent one and extract the role id from it).
What do you think?

That's how it work right now on Zebra (it remember the previously entered role) so yes your proposal makes sens.
What do you exactly need?
 /GET /activity/used      => [{id: 1234, name: 'Activity 1', last-usage: '2020-09-29', role-used: 167 }, {}]

Something like that?

Exactly, this would be perfect.

@sephii Here we are: https://zebra.liip.ch/api/#!/Activities/get_latestActivityRoles I make it very simple. It's just a map activityId => roleId. Tell me if you need more...


EditorUserBuildSettings.GetWSADotNetNative is abandoned in 2019

Describe the bug
In BuildDeployWindow.cs, EditorUserBuildSettings.GetWSADotNetNative occurs error, because of .Net scripting backend is abandoned in Unity2019.
Information(please complete the following information):

Version: [2<IP_ADDRESS>]

Hi @tattichan, thanks for opening this issue.
Is this occurring in Unity 2019 beta?


some modded enchants plainly dont work on shields

mods like spellbound add enchants that are compatible with shields. for example hearty. it adds hearts but you cant get it on your shields.

I'll look into it when I get the chance. There is a known issue with Fabric Shield Lib that causes a lot of problems with enchantments that shouldn't be able to be put on shields being put on them & enchantments that are only for shields not working properly.

https://www.curseforge.com/minecraft/mc-mods/spellbound-enchantments
I'm adding this link here so I can look into this more.

Fabric Shield Lib has been updated to fix this.


adding Coproducts and Generic to some constraints

I went ahead with adding both generic and Coproduct instances to some of the various constraints. Comments about this PR:

I did not add Coproduct instances to Key or Value constraints. I've not used Shapeless Unions before and didn't know how much value it might add. Can always be another PR.
For the life of me, I can't get Const[Foo]#ƛ to work with anything involving UnaryTCConstraint and Generic.
I moved things that were no longer just "HList" constraints to a "constraints" file. Ditto for the unit tests.

My sincere apologies for not having responded to this before. This breaks binary compatibility, so it will have to wait for 2.4.0, but otherwise it LGTM. Many thanks.

No problem Miles. Would there be any value to adding in anything for shapeless Union types here?

What did you have in mind for unions?

Same corollary as that from hist to record types as we have from coproduct to union.

Ahh ... yes, of course. Yup, adding instances for unions would make sense too.

I'll look to rebase and/or open up a new PR given the activity these past few days.

I'll look to rebase and/or open up a new PR given the activity these past few days.

Many thanks 👍 Working to clean up the 2.4.0 milestone so we can release.
This can definitely make it in.

Closing this and being replaced with another PR.


Fix CODEOWNERS file (opensearch-project#1193)

opensearch-security pull request intake form
Please provide as much details as possible to get feedback/acceptance on your PR quickly

Category: (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
Maintenance

Github Issue # or road-map entry, if available:

Description of changes:

Why these changes are required?

What is the old behavior before changes and new behavior after changes? (Please add any example/logs/screen-shot if available)

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)

Is it backport from main branch? (If yes, please add backport PR # and commits #)

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (opendistro-1.12@7e48953). Click here to learn what that means.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##             opendistro-1.12    #1280   +/-   ##
==================================================
  Coverage                   ?   64.54%
  Complexity                 ?     3201
==================================================
  Files                      ?      244
  Lines                      ?    17147
  Branches                   ?     3039
==================================================
  Hits                       ?    11067
  Misses                     ?     4540
  Partials                   ?     1540

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 7e48953...504bdbc. Read the comment docs.


Some Image URLs or local Images not supported in MultimodalConversableAgent

What happened?
I followed Engaging with Multimodal Models: GPT-4V in AutoGen to interpret and analyze the image. somehow the image mentioned in example works but in my case when I try with different image like this https://i.imgur.com/ElLuHVN.png then it  gives the reponse as below.
Warning! Unable to load image from https://i.imgur.com/ElLuHVN.png, because cannot identify image file <_io.BytesIO object at 0x000001AD3EBD9440>
Is there any specific image format or prompt structure to analyze the image? ?
Additionaly, I just wanted to let you know that the model gpt-4-vision-preview used in the example is deprecated from OpenAI
Thanks!
What did you expect to happen?
I expected that MultimodalConversableAgent will interpret and analyze the image url I provided but it does not work that way.
How can we reproduce it (as minimally and precisely as possible)?
import json
import os
import random
import time
from typing import Any, Callable, Dict, List, Optional, Tuple, Type, Union
import dotenv
import base64

# import matplotlib.pyplot as plt
# import numpy as np
import requests
from PIL import Image
from termcolor import colored

import autogen
from autogen import Agent, AssistantAgent, ConversableAgent, UserProxyAgent
from autogen.agentchat.contrib.capabilities.vision_capability import VisionCapability
from autogen.agentchat.contrib.img_utils import get_pil_image, pil_to_data_uri
from autogen.agentchat.contrib.multimodal_conversable_agent import (
    MultimodalConversableAgent,
)
from autogen.code_utils import content_str

dotenv.load_dotenv()
# OPENAI_API_KEY = os.environ.get("OPENAI_API_KEY")

config_list_4v = autogen.config_list_from_json(
    env_or_file="OAI_CONFIG_LIST",
    filter_dict={
        "model": ["gpt-4o-mini"],
        "api_key": os.environ.get("OPENAI_API_KEY"),
    },
)

# gpt4_llm_config = {"config_list": config_list_gpt4, "cache_seed": 42}

config_list_gpt4o_mini = autogen.config_list_from_json(
    env_or_file="OAI_CONFIG_LIST",
    filter_dict={
        "model": [
            "gpt-4o-mini",
        ],
        "api_key": os.environ.get("OPENAI_API_KEY"),
    },
)

image_agent = MultimodalConversableAgent(
    name="image-explainer",
    max_consecutive_auto_reply=10,
    llm_config={"config_list": config_list_4v, "temperature": 0.5, "max_tokens": 300},
)

user_proxy = autogen.UserProxyAgent(
    name="User_proxy",
    system_message="A human admin.",
    human_input_mode="NEVER",  # Try between ALWAYS or NEVER
    max_consecutive_auto_reply=0,
    default_auto_reply="nothing found",
    code_execution_config={
        "use_docker": False
    },  # Please set use_docker=True if docker is available to run the generated code. Using docker is safer than running the generated code directly.
    llm_config={
        "config_list": config_list_gpt4o_mini,
        "temperature": 0.5,
        "max_tokens": 300,
    },
)

# Ask the question with an image
user_proxy.initiate_chat(
    image_agent,
    message=f"""Describe this image: <img https://i.imgur.com/ElLuHVN.png>""",
)

AutoGen version
0.2.37
Which package was this bug in
AgentChat
Model used
gpt-4o-mini
Python version
3.10.0
Operating system
Windows 10
Any additional info you think would be helpful for fixing this bug
No response

It's probably due to the image file downloaded from https://i.imgur.com/ElLuHVN.png cannot be decoded properly. @BeibinLi can you comment on this?

Yes,imgur has some special mechanism to protect their images. Can you check with another URL link from Google or BIng? You can use gpt-4o instead of gpt-4-vision for the task.

Yes,imgur has some special mechanism to protect their images. Can you check with another URL link from Google or BIng? You can use gpt-4o instead of gpt-4-vision for the task.

Thank you. Yes other image urls worked.
For those who are looking for openAI compatibe alternative to imgur, check out postimages

@haard7  BTW, I do not recommend gpt-4o-mini for vision task, as its tokenization for images are much larger than GPT-4o. You should compare the cost and decide which one to use.

Thanks for the quick reply all.
Shall we close this as completed @haard7 ?


ImageFileReader extracts wrong subregion of the data if the ExtractIndex larger than 1024

Hi,
I was following the tutorial https://simpleitk.readthedocs.io/en/master/link_AdvancedImageReading_docs.html to extract a large file by extracting subregions given my memory constraints. However, I notice that if I do something like 👍 file_reader.SetExtractIndex(current_index) with any dimension of "current_index" larger than 1024, the file_reader gives me wrong subregion of the data.
Could you fix it?
Best,
Di

Hello @dichen9412,
The description of the issue is too broad. It isn't clear what "wrong subregion" means and what exactly is causing the problem. Please provide a minimal working example illustrating the issue. Below is some code showing that the extract region works as expected with an index larger than 1024. Feel free to modify this code to illustrate the issue.
import SimpleITK as sitk

file_name = 'large_image.mha'

image = sitk.Image([5000,5000,50], sitk.sitkUInt8)
image.SetOrigin([0.6,0.7,0.8])
image.SetSpacing([0.5,0.5,0.5])
sitk.WriteImage(image, file_name)

# Read image information without reading the bulk data.
file_reader = sitk.ImageFileReader()
file_reader.SetFileName(file_name)
file_reader.ReadImageInformation()
print(f'image origin: {file_reader.GetOrigin()}\nimage size: {file_reader.GetSize()}\nimage spacing: {file_reader.GetSpacing()}')

current_index = [1025, 1025, 0]
extract_size = [100, 100, 50]
print(f'index physical position: {image.TransformIndexToPhysicalPoint(current_index)}')
file_reader.SetExtractIndex(current_index)
file_reader.SetExtractSize(extract_size)
subimage = file_reader.Execute()
print(f'subimage origin: {subimage.GetOrigin()}\nsubimage size: {subimage.GetSize()}\nsubimage spacing: {subimage.GetSpacing()}')

Hi Zivy,
Thanks for replying to my issue!
I can't give you the exact data I use but I think I can illustrate the problem better:
In your code, you extracted a region without any error prompts.
However, your entire image is probably just 0s. So you don't see the problem.
Can you fill your image such that image[i, :, :] are filled with number i.
Then you can check if your code extracts the right region of the image.
With your code and the image I described above, the region should have numbers ranging from 1025 to 1125.
However, if the index is larger than 1024, it extracts from the beginning, i.e., [0, 0, 0], resulting in a sub-image with numbers ranging from 0 to 100.
Hope my suggestion reproduces the bug on your side.
Best,
Di

Hello @dichen9412,
Based on your suggestion, below is a minimal working example illustrating that reading the subregion works as expected:
import SimpleITK as sitk
import numpy as np

file_name = 'large_image.mha'

image = sitk.GetImageFromArray(np.arange(5000*5000*50).reshape((50,5000,5000)))
sitk.WriteImage(image, file_name)

current_index = [1025, 1025, 0]
extract_size = [100, 100, 50]
file_reader = sitk.ImageFileReader()
file_reader.SetFileName(file_name)
file_reader.SetExtractIndex(current_index)
file_reader.SetExtractSize(extract_size)
subimage = file_reader.Execute()

print(sitk.GetArrayViewFromImage(subimage))

Hi zivy,
Thanks for following up on this issue. I just tried to reproduce this with my minimal code on a MacBook pro. However, I don't see the issue there. Then I go back to my server and the issue exists again. My server is an AWS P3.x8large instance with
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.7 LTS
Release:	16.04
Codename:	xenial
.
The Simpleitk version I use on both my MacBook Pro and the ubuntu is 1.2.4. Interestingly, I can reproduce the bug on ubuntu but not on my MacBook pro.
The input file is a .nii.gz file with a dtype=float64. I also tried on the same size but dtype=int8. The issue still exists.
(I can send you the small segmentation mask file via email if you want.)
Let me know if you need any further information.
Here is my code: (on Mac the diff is 0 while on ubuntu it was  > 0).
"""
Script to reformat 2020 data and explore
"""
import os
import zipfile
import shutil
#import nibabel as nib
import numpy as np
from skimage import measure
import time
import SimpleITK as sitk
import tracemalloc
import gc
def reverse_axes(image):
return np.transpose(image, tuple(reversed(range(image.ndim))))
def read_image(imagefile, verbose=True):
file_reader = sitk.ImageFileReader()
file_reader.SetFileName(imagefile)
file_reader.ReadImageInformation()
#image = sitk.ReadImage(imagefile)

raw_direction = np.asarray(file_reader.GetDirection()).reshape(3, 3)
direction = np.zeros((3, 3))
for i in range(3):
    idx = np.argmax(abs(raw_direction[:, i]))
    if raw_direction[idx][i] > 0:
        direction[idx][i] = 1
    else:
        direction[idx][i] = -1
header = {
    'size': file_reader.GetSize(),
    'spacing': file_reader.GetSpacing(),
    'origin': file_reader.GetOrigin(),
    'direction': direction,
    'raw_direction': file_reader.GetDirection(),
}
if (verbose):
    #print("dtype", data.dtype)
    print("raw_direction", raw_direction)
    print("direction", direction)
    print("shape", header['size'])
    print("raw_spacing", header['spacing'])

image_size = file_reader.GetSize()
tot_vol = image_size[0] * image_size[1] * image_size[2]

extract_size = list(file_reader.GetSize())
dim = np.argmax(extract_size)

parts = min(int(tot_vol / (512 * 512 * 900)) + 1, extract_size[dim])
extract_size[dim] = (extract_size[dim] - 1) // parts + 1
current_index = [0, 0, 0]

print("dim", dim, "parts", parts)
data = np.zeros(image_size, dtype="float64")

for i in range(parts):
    current_index[dim] = i * extract_size[dim]
    if (i == parts -1):
        extract_size[dim] = image_size[dim] - current_index[dim]

    print("current_index", current_index, "extract_size", extract_size)
    file_reader.SetFileName(imagefile)
    file_reader.SetExtractIndex(current_index)
    file_reader.SetExtractSize(extract_size)
    sub_image1 = file_reader.Execute()
    sub_data = sitk.GetArrayFromImage(sub_image1)
    #del sub_image1
    #sub_data = np.clip(sub_data, -1000, 10000, out=sub_data).astype("float16", copy=False)
    sub_data = reverse_axes(sub_data)
    l = np.asarray(current_index, dtype="int")
    r = l + np.asarray(extract_size, dtype="int")
    print("sub_data-%d"%i, sub_data.shape, "l", l, "r", r)
    data[l[0]:r[0], l[1]:r[1], l[2]:r[2]] = sub_data

    import matplotlib.pyplot as plt
    n_col = 3
    n_row = 1
    fig, axes = plt.subplots(n_row, n_col, figsize = (n_col * 5, n_row * 5))
    axes[0].imshow(data[data.shape[0]//2].astype("float32"))
    axes[1].imshow(data[:,data.shape[1]//2].astype("float32"))
    axes[2].imshow(data[:, :, data.shape[2]//2].astype("float32"))

    plt.show()
del sub_image1

gc.collect()
return data, header

imagefile = "/DRAI/Spine/Data/New/verse602_CT-iso_seg.nii.gz"
image1 = sitk.ReadImage(imagefile)
data1 = sitk.GetArrayFromImage(image1)
data1 = reverse_axes(data1)
print("data1", data1.shape, image1.GetSize())
data2, header = read_image(imagefile)
print("data2", data2.shape)
print("diff", np.mean(abs(data1-data2)))

Hello @dichen9412,
Not sure what is the issue then. More likely it is on your side than in SimpleITK.
Having said that, you appear to be working in a very old environment, ubuntu 16.04 which reached its end of life last year and SimpleITK 1.2.4 which was released 2.5 years ago, Nov. 2019.
As a first step I would recommend moving to a 2.x version of SimpleITK, note that it will not bebackwards compatible with the 1.x versions (see migration guide). If the problem still exists there then we can further look into it.

Hi zivy,
Just for you to know，I tried to upgrade to the latest simpleitk version 2.x.x but the problem is there.
Best，
Di

Hi zivy,
Here is a short example to reproduce the bug:
(I actually tried this with another machine ubuntu18.04 + simpleitk <IP_ADDRESS> and it still gives wrong results. (The behavior on ubuntu 16.04 and 18.04 are a bit different but are both wrong.))
import SimpleITK as sitk
import numpy as np
file_name = 'large_image.nii.gz'
data = np.zeros((512, 512, 2023), dtype="float64")
for i in range(2023):
data[:, :, i] = int(i // 1000)
image = sitk.GetImageFromArray(data.transpose(2, 1, 0))
print("image size", image.GetSize())
sitk.WriteImage(image, file_name)
current_index = [0, 0, 1200]
extract_size = [100, 100, 50]
file_reader = sitk.ImageFileReader()
file_reader.SetFileName(file_name)
file_reader.SetExtractIndex(current_index)
file_reader.SetExtractSize(extract_size)
subimage = file_reader.Execute()
print(np.mean(sitk.GetArrayViewFromImage(subimage)))
print(np.mean(data[0:100, 0:100, 1200:1250]))

Hello @dichen9412,
The code is very strange, some operations do not make sense. For example data[:, :, i] = int(i // 1000). This fills the image slices 0-999 with zero, slices 1000-1999 with one and slices 2000-2022 with two. I suspect this is not what you want.
Also, do not use mean to compare two images, subtract them and look at the min and max. Make sure that the indexing is correct. In SimpleITK index order is [x,y,z], and in numpy it is [z,y,x].
Finally, please format code using the GitHub syntax.

Hi zivy,
I give the strange value to the scan just to differentiate different slices. So the slices 0-999 are just 0s and 1000-1999 with 1s and 2000-2022 with 2s.
The subregion starting with slice 1200 should be all 1s. So the average should be 1. However, the average of the subregion doesn't give the right value. On ubuntu 16.04 it was 0. On ubuntu 18.04 it was a very large number.
I'm pretty sure the indexing is correct since I have transposed the data to form the image.
Best,
Di

Thank you for the question. Hopefully the problem has been resolved since it was asked.


SIP status: disabled (Apple Internal)

I disabled sip in recovery, then its status turned to disabled (Apple Internal)  which is weird that I think normally it won't display Apple internal.
And the script that I am using to enable HIDPI still tells me SIP need to be disabled
Configuration

HaC Mini version : 2.6
OSX version : 10.15.3

edit efi/oc/config.plist
'NVRAM'->'Add'->'7C436110-AB2A-4BBB-A880-FE41995C9F82'
add 'csr-active-config' = E7030000(type:Data) to disable sip
	<key>NVRAM</key>
	<dict>
		<key>Add</key>
		<dict>
			<key>7C436110-AB2A-4BBB-A880-FE41995C9F82</key>
			......
			<dict>
				<key>csr-active-config</key>
				<data>5wMAAA==</data>
			</dict>
		</dict>
	</dict>

That’s not a good way because you can’t use recovery mode.

kriscris@KrisCriss-Mac-mini ~ % nvram -p
boot-args	alcid=11 -disablegfxfirmware shikigva=32 shiki-id=Mac-BE088AF8C5EB4FA2
BootCampHD	%02%01%0c%00%d0A%03%0a%00%00%00%00%01%01%06%00%00%17%03%12%0a%00%03%00%00%00%00%00%7f%ff%04%00
bluetoothInternalControllerInfo	%90%82%ac%05%00%00@%14%d8%96%95%f3%a1Z
prev-lang:kbd	en:252
SystemAudioVolumeDB	%00
platform-uuid	%06%f5%cb%14%83%8e[M%b91T%b2%03%08%00%01
bluetoothActiveControllerInfo	%90%82%ac%05%01%00%00%00@%14%d8%96%95%f3%a1Z
fmm-computer-name	KrisCris%e2%80%99s Mac mini
efi-boot-device-data<EMAIL_ADDRESS>SystemAudioVolume	W
efi-boot-device	IOMatchIOProviderClassIOMediaIOPropertyMatchUUID004BF2E6-FF6F-44C7-8DFB-BA3D9A22E9AFBLLastBSDNamedisk2s2IOEFIDevicePathTypeMediaFilePathPath\1B6F04CD-7708-4967-A763-9EE3407F7CD7\System\Library\CoreServices\boot.efi%00
LocationServicesEnabled	%01
There it is, not sure what's going on that they looks so messy.

Hmm I don’t see csr status. Try sudo nvram csr-active-config

nvram: Error getting variable - 'csr-active-config': (iokit/common) data was not found
weird...

I did reinstalled HacMini for several times switching back and forth the HW ACC options.

Try ioreg -p IODeviceTree -l | grep csr-active-config

Try ioreg -p IODeviceTree -l | grep csr-active-config

It returned nothing.

I reversed csrutil and it seems like this is the default behaviour (even on real macs). You need to specify --no-internal to have it show up as disabled. But I don't think that makes any difference.

The issue is that https://github.com/xzhih/one-key-hidpi this tool need sip to be disabled, but after I disabled SIP in this way, it still asks me to disable SIP.

Then you should use the --no-internal flag

Then you should use the --no-internal flag

Thanks, I’ll try tomorrow.


Fee policies for legal entity

Add EntityFeePolicyStore to handle token fee policies
Add FeeCalculatorV2Facet to handle fee policies per user and token
Add LibFeePolicy with storage
Update LibFeeCalculator.distributeRewards to check and use fee policies
Add scrpts/fee-policy-management.js to handle fee policies management
Update README.dm with fee policy scripts
Signed-off-by: Trayan Manolov<EMAIL_ADDRESS>

The base branch is changed to the epic branch with name 681_fees_per_user_contracts (which was the initial idea). We may have to make changes in RouterFacet after the tasks in hashport-validator.

We should update the deploy router script as well so new deploys include the new facet.

When you execute npx hardhat coverage, there are failing tests


homebrew formula for ltfs

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
A hope for brew install ltfs or brew install some-tap/ltfs to work.
Describe alternatives you've considered
Following the installation instructions of the readme.
Additional context
It seems like the installation instructions put a lot of consideration to homebrew environments but I don't see a formula for ltfs itself. I've started a draft of that work at https://github.com/amiaopensource/homebrew-amiaos/pull/220 but am stuck on an installation issue. I wanted to share that here in case any others are interested in contributing towards having an ltfs brew formula.

Close this issue because we don't have any comment about 1 month.
Please reopen this if you cannot resolve the problem yet.


/whois  - Bot crash and never respond.

Required Information

PHP version: 7.0.13
PHP Telegram Bot version: 0.39.0
Using MySQL database: yes
Update Method: Webhook
Self-signed certificate: no
RAW update (if available):

Expected behaviour
Bot should reply to commands.
Actual behaviour
Bot stucks and is not responding to any messages/commands after an unknown issue. Sometimes I can solve this cleaning DB or removing/uploading again files.
Steps to reproduce
Execute /whois <string>, and randomly all commands stop working. Bot don't reply anymore to any command after this.
Extra details
Dumps:
_error.log = empty
_access.log:
{"update_id":990XXXXX, "edited_message":{"message_id":5XXX,"from":{"id":32XXXXXX"first_name":"nesttle","username":"nesttle"},"chat":{"id":32XXXXXX,"first_name":"nesttle","username":"nesttle","type":"private"},"date":148646XXX,"edit_date":1486468XXX,"text":"/whois testing","entities":[{"type":"bot_command","offset":0,"length":6}]}}
{"update_id":990XXXXX, "edited_message":{"message_id":5XXX,"from":{"id":32XXXXXX"first_name":"nesttle","username":"nesttle"},"chat":{"id":32XXXXXX,"first_name":"nesttle","username":"nesttle","type":"private"},"date":148646XXX,"edit_date":1486468XXX,"text":"/whois testing","entities":[{"type":"bot_command","offset":0,"length":6}]}}
{"update_id":990XXXXX, "edited_message":{"message_id":5XXX,"from":{"id":32XXXXXX"first_name":"nesttle","username":"nesttle"},"chat":{"id":32XXXXXX,"first_name":"nesttle","username":"nesttle","type":"private"},"date":148646XXX,"edit_date":1486468XXX,"text":"/whois testing","entities":[{"type":"bot_command","offset":0,"length":6}]}}
{"update_id":990XXXXX, "edited_message":{"message_id":5XXX,"from":{"id":32XXXXXX"first_name":"nesttle","username":"nesttle"},"chat":{"id":32XXXXXX,"first_name":"nesttle","username":"nesttle","type":"private"},"date":148646XXX,"edit_date":1486468XXX,"text":"/whois testing","entities":[{"type":"bot_command","offset":0,"length":6}]}}
{"update_id":990XXXXX, "edited_message":{"message_id":5XXX,"from":{"id":32XXXXXX"first_name":"nesttle","username":"nesttle"},"chat":{"id":32XXXXXX,"first_name":"nesttle","username":"nesttle","type":"private"},"date":148646XXX,"edit_date":1486468XXX,"text":"/whois testing","entities":[{"type":"bot_command","offset":0,"length":6}]}}

_update.log
{"update_id":990XXXXX, "edited_message":{"message_id":5XXX,"from":{"id":32XXXXXX,"first_name":"nesttle","username":"nesttle"},"chat":{"id":32XXXXXX,"first_name":"nesttle","username":"nesttle","type":"private"},"date":14864XXXXX,"edit_date":14864XXXXX,"text":"/whois testing","entities":[{"type":"bot_command","offset":0,"length":6}]}}

Command which made my bot crash:
[12:57] nesttle: /whois AB1234c
[12:57] MYBOT: Chat not found!

After that, I executed correctly 2 commands, but looks like bot stucks on command above (I can see again and again same input on dumps: _update.log and _access.log)

Also, on db table.edited_message I can see same request again and again (200+ same entries):
http://i.imgur.com/s1fsHZG.png :: Database

@jacklul Any ideas, I'm a bit lost here at the moment...

I honestly have no idea what is causing this...

@nesttle Any extra info you could give us?
Can you call the /debug command on your bot and post the output here please?
Maybe that will help with something.

@noplanman tried /debug but not responding to any commands

I have the same behaviour.
When bot responds with 500 error, Telegram tries to resend message, afaik. Which causing 500 error and so on. When user comments out custom commands, those messages got at least Request::emptyResponce, not 500.

+350 requests inserted into db, bot still not working :(
http://i.imgur.com/erE7enW.png Screenshot

What if there is some kind of issue with PHP7?
Any chance you could try running bot with PHP 5.6?

It was the same, on 5.6 for me.
You could reproduce it with making any code in execute() that will generate
fatal error. You'll see "pending_update_count" going up on
https://api.telegram.org/botXXXXXXXXX:XXXXXXXXXXXXXXXXXXXXXXXXX/getWebhookInfo
If it's over 100 you'll unlike get an answer from the bot.
С приветом, Чванов Даниэль http://chvanov.ru/?ml=1.
On Wed, Feb 8, 2017 at 3:08 AM, jacklul<EMAIL_ADDRESS>wrote:

What if there is some kind of issue with PHP7?
Any chance you could try running bot with PHP 5.6?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/akalongman/php-telegram-bot/issues/406#issuecomment-278124634,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMhOxprZSUkvjSRgQdYy9NlCCbWDpl33ks5raM9PgaJpZM4L5v2r
.

If you can, try my cgi workaround in https://github.com/akalongman/php-telegram-bot/issues/406#issuecomment-278124634

Thanks for your replies guys.
Atm, I made it working again with the next "workaround":

Make a new hook_new.php without any MySQL/customs commands. Hook.php:

<?php
// Load composer
require __DIR__ . '/vendor/autoload.php';

$API_KEY = '3XXXXXXXXX';
$BOT_NAME = 'MYBOT';

try {
    // Create Telegram API object
    $telegram = new Longman\TelegramBot\Telegram($API_KEY, $BOT_NAME);
    // Handle telegram webhook request
    $telegram->handle();
    file_put_contents('_qnctalog.log','OK'.PHP_EOL, FILE_APPEND | LOCK_EX);
} catch (Longman\TelegramBot\Exception\TelegramException $e) {

    file_put_contents('_qnctalog.log', $e, FILE_APPEND | LOCK_EX);
}

Make a new set_test.php with the following code:

<?php
require __DIR__ . '/vendor/autoload.php';
$API_KEY = '3XXXXXXXXXXXXXXXX';
$BOT_NAME = 'MYBOT';
$hook_url = 'https://www.yourdomain.com/path/hook_test.php';
try {
    // Create Telegram API object
    $telegram = new Longman\TelegramBot\Telegram($API_KEY, $BOT_NAME);

    // Set webhook
    $result = $telegram->setWebhook($hook_url);
    if ($result->isOk()) {
        echo $result->getDescription();
    }
} catch (Longman\TelegramBot\Exception\TelegramException $e) {
    echo $e;
}

Also I truncated the following DB tables: edited_messages, messages
Enter url https://www.yourdomain.com/path/set_test.php
Talk with your bot, now commands are working again for me, don't know why but lol.
Visit your old https://www.yourdomain.com/path/set.php
Working

Ok, this seems related to an issue I had as well at some point, where updates weren't being saved to the DB any more. After truncating it worked for me again too, so some SQL query is messing stuff up...

I think this issue is related when you edited a sent message. This time bot stucks when:

/whois @AB1234c
Edited message to /whois AB1234c
Bot broken

@nesttle Can you reproduce that on a clean install?
It works for me without any issues 😕
Have you tried enabling the error logging? If so, please set up a clean install, enable the error log and try to break it again.
Really not sure what I'm looking for here...

Will try today on a clean install and update this

@jacklu any plans to implement any solution like your CGI fix on next release? (to avoid bulk requests when bot comes online after a shutdown). Anyway I'll try your solution, thanks

Depending on webserver configuration errors and exceptions can modify headers with 500 error, to get rid of this you might have to dig around the webserver software you're using, you might also try overwriting the header before throwing exceptions (like header('HTTP/1.1 200 OK'); at the top of the hook script - no idea if that will work!).

Ok!  Will try to disable 500 Error response with this: http://stackoverflow.com/a/2687747
error_reporting=E_ALL display_errors=On and display_startup_errors=On

Found that somewhere on stackoverflow:
  /***
   * Error handling
   ***/
  /**
   * Error handler, passes flow over the exception logger with new ErrorException.
   *
   * @param $num
   * @param $str
   * @param $file
   * @param $line
   * @param null $context
   */
  function log_error( $num, $str, $file, $line, $context = null )
  {
    log_exception( new ErrorException( $str, 0, $num, $file, $line ) );
  }

  /**
   * Uncaught exception handler.
   * @param \ErrorException $e
   */
  function log_exception( $e )
  {

    $message = "Type: " . get_class( $e ) . "; Message: {$e->getMessage()}; File: {$e->getFile()}; Line: {$e->getLine()}; Time: ".date("H:i:s / d.m.Y")."; Trace: {$e->getTraceAsString()}";
    file_put_contents( __DIR__ .'/error.log', $message . PHP_EOL, FILE_APPEND );

    exit();
  }

  /**
   * Checks for a fatal error, work around for set_error_handler not working on fatal errors.
   */
  function check_for_fatal()
  {
    $error = error_get_last();
    if ( $error["type"] == E_ERROR )
      log_error( $error["type"], $error["message"], $error["file"], $error["line"] );
  }

  register_shutdown_function( "check_for_fatal" );
  set_error_handler( "log_error" );
  set_exception_handler( "log_exception" );
  ini_set( "display_errors", "off" );
  error_reporting( E_ALL );

Maybe it could be added to hook.php to return code 200 and emptyResponce (or not empty) via log_exception()?

@nesttle Getting a 500 error code has a reason, so it's all about finding the issue, not just hacking a fix.
Is your bot behind a reverse proxy?
Could you give us more info about your setup to help find the culprit?


Added support for multi service dependancy watchdog.

What this PR does / why we need it:
The dependency watchdog now checks for service dependency hierarchy (eg. etcd -> apiserver, apiserver -> other controlplane components). The watch duration has been raised to 5m from 2m as it was observed that pods had entered into CrashloopBackoff after 2m post with Dependency watchdog.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

The dependency-watchdog controller does now check for service dependency hierarchy (e.g., etcd -> kube-apiserver, kube-apiserver -> other control plane components). This will lead to even faster restarts of the control plane components after CrashLoopBackoffs.

Enabled the dependency-watchdog to have hierarchy of service-pod dependency.

Thank you. One question, I assume this hierarchy is hard-coded at the moment? Can we maybe make it flexible via labels or annotations that can be defined? This way extension controllers could also use it.


Change requestDisk rounding to base 10

Just to keep aligned with the current schedd rounding (though that will be removed). And because it looks more clear.

@amaltaro, Alan patch looks good to me. Can I merge? You don't have to create the wmagent branch separately. I will create a branch with this.

I think we can wait to see whether more patches will be needed and then backport them altogether. It's likely better to branch off of patch1 too (you see, that's why I didn't want to create the wmagent branch ;))


[add] FAQ to README for #update_all

Thanks!

Sure thing!  Thanks for your work on the gem.  I'll be taking a deeper dive over the coming weeks now that we have the initial implementation in place.


Broken mouseover with Geom.bar on js.svg backend?

Any chart I generate with Geom.bar and the default backend has an issue where when I mouseover the plot, the grid lines get thicker as intended, but when I un-mouseover they don't shrink back to normal size.  Every time I do this they get thicker and eventually they just become a solid grey background.  Verified in both IE and Chrome, but I noticed that this doesn't happen in the bar plot shown on the manual page -- maybe it's already been fixed and I wasn't able to find the commit, or maybe these were generated with a previous version of Gadfly?  I've only seen it in bar, but i haven't exhaustively checked other geometries.

I just checked in chrome, firefox, and safari on osx and it works. If convenient, could use master on Compose/Gadfly (Pkg.checkout("Compose"); Pkg.checkout("Gadfly")) to generate a plot and see if it still happens? I'll try it out on windows when I have a chance.

I just made a clean install at home with the current release and it's working as intended.  I'll take another look at work on Monday, and report back if I see anything, but perhaps it was fixed very recently or maybe a windows 8 verses 7 issue.

Ok, it turns out this appears to be an issue with Guide.xticks().  Here's a minimal example to reproduce:
using DataFrames, Gadfly
data = DataFrame()
data[:turtle] = ["Suzy", "Suzy", "Bob", "Batman", "Batman", "Batman", "Bob"]
data[:time] = [6.2 , 6.7 , 2.1, 3.2, 3.1, 0.9, 2.4]
data[:photo] = ["111.jpg","123.jpg","145.jpg","167.jpg","189.jpg","190.jpg","195.jpg"]

#Works
nWafersPlot= plot(data, x="turtle", y="time", Geom.bar)

#Does not work
nWafersPlot= plot(data, x="turtle", y="time", Geom.bar, Guide.xticks(label=false) )

Something about hiding xticks is messing up the mouseover code.  It's also nothing specific to bar, it also happens on line for example (I had a bunch of plots and the only one I was hiding xticks on was the bar one).

Should be fixed now. Thanks for the test case.

Great, thanks!


Write object documentation strings.

The main documentation is automatically generated from the base object documentation strings. A documentation style guide has been written. anchor.py is the prototype/example to follow.
Documentarians: please edit this list or add a comment to this issue stating which object(s) you are working on or plan to work on. That way we won't duplicate our efforts. Check the object off when the first draft of the documentation is complete.

[x] Font - @typesupply
[ ] Info
[X] Groups - @benkiel
[X] Kerning - @benkiel
[X] Features - @benkiel
[ ] Lib
[x] Layer - @typesupply
[ ] Glyph - @letterror
[ ] Contour
[ ] Segment
[ ] bPoint
[x] Point - @typesupply
[ ] Component
[X] Anchor - @typesupply
[ ] Image - @typesupply
[X] Guideline - @benkiel
[x] Normalizers - @typesupply

cc @LettError @benkiel @frankrolf

I'll start with Glyph

I did features (yes, with a really ugly git push/merge/is this thing on? — but it seems to not have done any damage) as a test. Look good?

Perfect!

Style question: fontParts or FontParts?

FontParts unless we're showing a code sample: "To use FontParts you do this: import fontParts."

@benkiel Are you working on groups now? That, kerning and lib are going to be a bit different since they all subclass BaseDict. They are very barebones so they don't have __getitem__ and all the other dict methods. To document these we'll need to either:

Write generic docstrings in BaseDict that refer to "key" and "value" abstractly. Write top level documentation in the subclasses that defines what key and value are with lots of examples.
Add dummy methods that call super to the group, kerning and lib classes as shells for object specific documentation.

I'm leaning towards 2 even though it's a bit more cumbersome in the code. In the documentation it will be much easier for scripters to understand.

I am, slowly. I think that you're right, we should do number 2, so that it's easier for scripters.

FontPants

hi. I've been going through the docs and completing missing general descriptions for some objects (glyph, image, bPoint, component, now working on segment). I'm following the example of font, which is already complete.
I've noticed that the general descriptions are somewhat redundant, given that most of the information appears again in the Reference section below it (pulled from the docstrings in the code).
so, I feel the need to clarify what exactly is the purpose of having general descriptions:

they are grouped into sections, while the reference entries are not
they are short, while the info in the reference section is sometimes longer, contains code examples etc.
anything else?

are the general descriptions needed at all? if the main reason for them to exist is having the information grouped into sections, we could simply add headers and change the order of the generated docs (reference), without duplicating information.

so, I feel the need to clarify what exactly is the purpose of having general descriptions:

they are grouped into sections, while the reference entries are not
they are short, while the info in the reference section is sometimes longer, contains code examples etc.
anything else?

I was trying to make a quick reference at the top. I always find it annoying to have to scan through really verbose documentation when all I'm doing is looking to see what is generally available, refresh my memory on an attribute name and stuff like that. That's what I'm using documentation for 99% of the time.

are the general descriptions needed at all? if the main reason for them to exist is having the information grouped into sections, we could simply add headers and change the order of the generated docs (reference), without duplicating information.

I don't like that we have to write the shorter description and organize that stuff manually. If there is a better way in Sphinx to do this, I'm all for it. I just don't want scripters to have to wade through a whole bunch of text when all they are trying to do is remember if it's glyph.appendComponent or glyph.insertComponent for the millionth time. Totally talking about a friend here. Not me. I always remember that it is, hang on, appendComponent.

thanks for the feedback, Tal.
I went ahead and implemented my proposal to remove general descriptions, and group the generated docs under the existing sections. it's in a separate branch for now. this avoids duplicating content for every attribute/method in every object. (!!)
what is missing, I think, is some sort of TOC to collect headers and their methods/attributes at the top of each page. neiter .toctree:: nor .contents:: does the job. but it should be possible to write a custom Sphinx directive to accomplish this (hint hint @typemytype).
with the proposed structure, your friend can find things by using the section headers, which now appear in the sidebar navigation. hope that works for him! :)

I guess this is what you are looking for: https://gist.github.com/typemytype/57e433984ee83c5becfb0192eb364bf2
add it to conf.py and use fe:
.. autosummarymethodlist::

	BaseGlyph

The order of methods is in this example alphabetically, not much to improve, I guess. Looking up and sorting the methods by line numbers is no option as subclassing will disorder it again.

he Frederik, thanks for this.
I've added it to conf.py. but now I'm getting this error when running make html:
File "/Library/Python/2.7/site-packages/sphinx/writers/html.py", line 750, in unknown_visit
    raise NotImplementedError('Unknown node: ' + node.__class__.__name__)
NotImplementedError: Unknown node: autosummary_table

I've upgraded to the latest version of Sphinx (v1.5.2), same problem.
does it work on your machine?

I see :)
you'll need to activate the auto summary in the extensions in your conf.py file
extensions = [
    'sphinx.ext.autodoc',
    'sphinx.ext.todo',
    'sphinx.ext.coverage',
    'sphinx.ext.ifconfig',
    'sphinx.ext.viewcode',
    'sphinx.ext.autosummary',  # <-------- this one
]

now it works – thanks!

Closing this and opening individual tickets to make it easier to break up the work


libvirt: 4.5.0 -> 4.7.0

Motivation for this change
version bump

Success on x86_64-linux (full log)
Attempted: libvirt
Partial log (click to expand)
patching script interpreter paths in /nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0
/nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0/var/lib/rc.d/init.d/libvirt-guests: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0/var/lib/rc.d/init.d/virtlockd: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0/var/lib/rc.d/init.d/libvirtd: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0/var/lib/rc.d/init.d/virtlogd: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0...
moving /nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0/sbin/* to /nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0/bin
/nix/store/94hcvb11qwp68kp67ksrqxg6jmhq5npr-libvirt-4.7.0

Success on aarch64-linux (full log)
Attempted: libvirt
Partial log (click to expand)
patching script interpreter paths in /nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0
/nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0/var/lib/rc.d/init.d/libvirt-guests: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0/var/lib/rc.d/init.d/virtlockd: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0/var/lib/rc.d/init.d/libvirtd: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0/var/lib/rc.d/init.d/virtlogd: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0...
moving /nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0/sbin/* to /nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0/bin
/nix/store/1x7vl4bkd5n7sa5xarxdqz361rqymz8y-libvirt-4.7.0

Thanks. libvirt-4.7.0 works for me.

I have deployed 4.7.0 to production a few days ago.
No problems so far, but there are only LXC slaves, so I had no problems with 4.6.0 too :)

let's check darwin build
@GrahamcOfBorg build libvirt

Success on x86_64-linux (full log)
Attempted: libvirt
Partial log (click to expand)
patching script interpreter paths in /nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0
/nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0/var/lib/rc.d/init.d/libvirt-guests: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0/var/lib/rc.d/init.d/virtlogd: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0/var/lib/rc.d/init.d/libvirtd: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0/var/lib/rc.d/init.d/virtlockd: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0...
moving /nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0/sbin/* to /nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0/bin
/nix/store/jnkdzrvh4i1nkwfpaa0h5s2ybnjy7881-libvirt-4.7.0

Success on x86_64-darwin (full log)
Attempted: libvirt
Partial log (click to expand)
Removing /nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/share/gtk-doc
rmdir: failed to remove '/nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/share': Directory not empty
gzipping man pages under /nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/share/man/
strip is /nix/store/4w56qihlrddav67p7d1vy5qkyayaqw11-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/lib  /nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/libexec  /nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/bin  /nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/sbin
patching script interpreter paths in /nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0
/nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/z2pc6bbx9d1mwwahnds31a7nlvjbi9zr-bash-4.4-p23/bin/sh"
/nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/z2pc6bbx9d1mwwahnds31a7nlvjbi9zr-bash-4.4-p23/bin/sh"
moving /nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/sbin/* to /nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0/bin
/nix/store/4p9qiam525lxz8qf37y7mgfz9nmqw57f-libvirt-4.7.0

Success on aarch64-linux (full log)
Attempted: libvirt
Partial log (click to expand)
patching script interpreter paths in /nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0
/nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0/bin/virt-xml-validate: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0/var/lib/rc.d/init.d/libvirt-guests: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0/var/lib/rc.d/init.d/virtlockd: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0/var/lib/rc.d/init.d/libvirtd: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0/var/lib/rc.d/init.d/virtlogd: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0/libexec/libvirt-guests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0...
moving /nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0/sbin/* to /nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0/bin
/nix/store/c8z0ws7l40z745hj2g0czxn8xbplhk8q-libvirt-4.7.0

ran nix-review successfully


[php] Incorrect color for switch default label when using alt-syntax

The keyword default is white like a regular label instead of purple if it is on a new line, has leading whitespace, and is followed by a colon.
switch ($a):
  default:
    break;
endswitch;

That has been fixed in the latest insiders version.

Odd, I had just updated...

Okay, it isn't fixed here, but in the grammar.
https://github.com/atom/language-php/commit/8b4ab3e8acaf50da14d6fac4d797a69f0918f760
but the vscode team didn't updated the grammar, yet, or the parser does not understood the grammar correctly.

Ah, well judging by the comment in that commit, it seems the problem is upstream:
# but NOT switch(expression):
Note the trailing colon...

Then you need to file this issue there.

Oh right. Didn't really want to go through their wall-of-text issue template while at work yesterday.
I'll leave this open until the change gets pulled into vscode.

Fixed by f31c369.


Support min_snr_in_db == max_snr_in_db in AddBackgroundNoise

torch.distributions.Uniform raises a ValueError when low == high:

ValueError: Uniform is not defined when low>= high

AddBackgroundNoise only raises when min_snr_in_db > max_snr_in_db, and not when min_snr_in_db == max_snr_in_db

There is therefore an inconsistency between the two behaviors.
I think supporting min == max is the way to go.

Thanks for filing this. Yeah, low == high isn't properly supported yet in this transform. I guess it could be patched by using torch.full, like here: https://github.com/asteroid-team/torch-audiomentations/blob/cf33679368125b0ff6db6680a7cf0b289386849f/torch_audiomentations/augmentations/shift.py#L119
As a workaround until it is fixed, one can do something like this, which for all practical intents and purposes gives the same result:
AddBackgroundNoise(
    bg_path, min_snr_in_db=12.0, max_snr_in_db=12.0000001
)

If you'd like to contribute, that would be a welcome PR 😄

@FrenchKrab I think I noticed this max_snr = min_snr + epsilon hack in your code.
Would you like to contribute this change?

I opened a PR that adds the suggested torch.full fix

Thanks


Router

This PR avoids:

heavy operation with router.resolve

router.resolve performs RegExp.exec() when mapping the whole routes array to find a matched result.

duplicated route records with redirects and maps holding data, components, now a single map holds all pages info.

This pr reduce outputsize and improves performance at the same time.
Breaking changes:

routeMeta was renamed to meta
Getting page meta with router.resolve is deprecated, people should migrate to resolve in @vuepress/client to get page meta. (Calling router.resolve to get a correct page path is still supported)


[9.4.x] ISPN-10678 Cluster Expiration should only expire primary owned entries

Change reaper to expire entries only from segments it owns

https://issues.jboss.org/browse/ISPN-10678
https://issues.jboss.org/browse/ISPN-10668
https://issues.jboss.org/browse/ISPN-10696
https://issues.jboss.org/browse/ISPN-9191

@wburns I think this PR needs to be updated. the 10.0 PR has more code changed :)

Yes, I was planning on updating this this week, sorry I missed it earlier.

Should be updated now.

Cherry picked commit to fix ExceptionEvictionTest random failure.

integrated! thanks @wburns !


Error with peer dependencies/ Typing error on API

Im trying to get this template up and running with a headless Shopify implementation (already set up my .env.local); cloning the project and attempting to install with
npm install
I get errors that some of the packages have not been updated to use Nextjs-10 etc...
I tried using
npm install --force
and
npm install --legacy-peer-deps
but the same error still occurs at
./framework/bigcommerce/api/utils/fetch-store-api.ts and
./framework/bigcommerce/api/utils/fetch-graphql-api.ts
specifically on this line the function

const fetchGraphqlApi: GraphQLFetcher = async (
  query: string,
  { variables, preview } = {},
  fetchOptions
) => {
  // log.warn(query)
  const config = getConfig()
  // Type error on the 2nd argument options object to fetch()
  const res = await fetch(config.commerceUrl + (preview ? '/preview' : ''), {
    ...fetchOptions,
    method: 'POST',
    headers: {
      Authorization: `Bearer ${config.apiToken}`,
      ...fetchOptions?.headers,
      'Content-Type': 'application/json',
    },
    body: JSON.stringify({
      query,
      variables,
    }),
  })

  const json = await res.json()
  if (json.errors) {
    throw new FetcherError({
      errors: json.errors ?? [{ message: 'Failed to fetch Bigcommerce API' }],
      status: res.status,
    })
  }

  return { data: json.data, res }
}

(parameter) fetchOptions: RequestInit | undefined
Argument of type '{ method: string; headers: { 'Content-Type': string; Authorization: string; } | { 'Content-Type': string; Authorization: string; } | { 'Content-Type': string; length: number; toString(): string; toLocaleString(): string; ... 31 more ...; Authorization: string; } | { ...; }; ... 7 more ...; timeout?: number | undefin...' is not assignable to parameter of type 'FetchOptions'.
  Type '{ method: string; headers: { 'Content-Type': string; Authorization: string; } | { 'Content-Type': string; Authorization: string; } | { 'Content-Type': string; length: number; toString(): string; toLocaleString(): string; ... 31 more ...; Authorization: string; } | { ...; }; ... 7 more ...; timeout?: number | undefin...' is not assignable to type '{ agent?: Agent | undefined; retry?: RetryOptions | undefined; }'.
    Types of property 'agent' are incompatible.
      Type 'Agent | ((parsedUrl: URL) => Agent) | undefined' is not assignable to type 'Agent | undefined'.ts(2345)

Here's the entire output TS error
I'm just starting with typescript but how to resolve this?

Seemed to work when I used Yarn instead of NPM, will close.

This is still happening to me even with Yarn


Unable to pull helm charts through proxy

I am getting this error caused by our corporate proxy:
[1] KNV1068: failed to run kustomize build in /repo/source/d523134f8a76659526b7ab6b0ea3cc3908a00100/cert-manager, stdout: : Error: accumulating resources: accumulation err='accumulating resources from 'base': '/repo/source/d523134f8a76659526b7ab6b0ea3cc3908a00100/cert-manager/base' must resolve to a file': recursed accumulation of path '/repo/source/d523134f8a76659526b7ab6b0ea3cc3908a00100/cert-manager/base': Error: looks like "https://charts.jetstack.io" is not a valid chart repository or cannot be reached: Get "https://charts.jetstack.io/index.yaml": dial tcp <IP_ADDRESS>:443: connect: connection timed out : unable to run: 'helm pull --untar --untardir /repo/source/d523134f8a76659526b7ab6b0ea3cc3908a00100/cert-manager/base/charts --repo https://charts.jetstack.io cert-manager --version v1.5.3' with env=[HELM_CONFIG_HOME=/tmp/kustomize-helm-924204340/helm HELM_CACHE_HOME=/tmp/kustomize-helm-924204340/helm/.cache HELM_DATA_HOME=/tmp/kustomize-helm-924204340/helm/.data] (is 'helm' installed?)   For more information, see https://g.co/cloud/acm-errors#knv1068
How do you configure the corresponding namespace reconciler to use a proxy? Editing the ns-reconciler (or reconciler-manager) configmap to add an HTTPS_PROXY environment variable doesn't seem to be possible (automatically overridden).
Thanks.
Laurent Philippart

Just confirmed that with the proper environment helm pull works fine in the hydration-controller:
export HELM_CONFIG_HOME=/tmp/kustomize-helm-723481838/helm
export HELM_CACHE_HOME=/tmp/kustomize-helm-723481838/helm/.cache
export HELM_DATA_HOME=/tmp/kustomize-helm-723481838/helm/.data
export HTTPS_PROXY=<our proxy>
helm pull --untar --untardir /repo/source/358e7d8ecf8a79d1f70b1a561f744f96a2894474/cert-manager/base/charts --repo https://charts.jetstack.io cert-manager --version v1.5.3
ls -al /repo/source/358e7d8ecf8a79d1f70b1a561f744f96a2894474/cert-manager/base/charts
total 16
drwxr-sr-x 4 65533 65533 4096 Aug 31 14:23 .
drwxr-sr-x 3 65533 65533 4096 Aug 31 14:23 ..
drwxr-sr-x 3 65533 65533 4096 Aug 31 14:23 cert-manager
drwxr-sr-x 2 65533 65533 4096 Aug 31 14:23 cert-manager-v1.5.3.tgz

@philippart is the issue that there is no setting for the HTTPS proxy in the ConfigSync API?

yes and alternatively I don't see how I would be able to customize the configmap.


uplot-vue: after updating data attribute, chart is not "rerendering"

hello,
i am using:

uplot: 1.6.11
uplot-vue: 1.0.2

i was wondering why after updating the data attribute the chart is not re-rendered.
after a long forensic search i think i found the error.
in the code lines here in the current commit https://github.com/skalinichev/uplot-wrappers/blob/2632d20cc45499ba89867a593569dac9cc304a60/vue/uplot-vue.js#L38 the parameter order of the watcher is correct, but i noticed that in the release (https://github.com/skalinichev/uplot-wrappers/archive/refs/tags/v1.0.2.zip) itself the order is wrong, so it always updates with the old data.
in uplot-vue.tsx:44
data(prevData: uPlot.AlignedData, data: uPlot.AlignedData) {

and also in uplot-vue.js:241 that is shipped via npm install
data: function data(prevData, _data) {

reference to the vue watch api:
https://vuejs.org/v2/api/#vm-watch
maybe u can build a new release 🙂

Yeah, people started asking a lot about the new release recently. Will try to do it tomorrow (there is still a small regression that needs to be fixed before that). Sorry for the inconvenience (

Should be fixed now


Mainnet bridge seems to be in trouble

tftchainstellarbridge:v2.5.3
It keeps restarting with

fixed on v2.5.5


I366 unittest

Solves issue #366

Uses unittest (Python standard library) instead of nose (which is no longer actively developed) for tests
Removes nose dependency
Adds pytest dependency for easy test runs
notes testing command in the README>

@npdoty , could I ask you for a review on this?

One side-effect of interest: if you happen to have a file lying around with a filename foo_test.py (because you were just testing something out) pytest will try to run it as part of your testing. I don't think we have anything checked in like that, but if you have it in your untracked files (as I did), you may notice it.


Error running CNView

I have an input coverage matrix of 176 samples where across a small region; each row in the bedfile covers only 1bp (unlike your example, but the region is pretty small so I'm hoping this won't cause issues).  There are no missing bp intervals in the file for the region.
Running the following:
Rscript CNView.R chrX<PHONE_NUMBER>96 417_S9  chrX-308758-314496_matrix.txt chrX-308758-314496.pdf

Where the matrix looks like:
chrom	start	end	100_S2	105_S5 ...
chrX	308605	308606	0	0
chrX	308606	308607	0	0
chrX	308607	308608	1	1
chrX	308608	308609	1	1
chrX	308609	308610	1	1
chrX	308610	308611	1	1
...

I get the following error
Loading required package: optparse
+-------------------+
| CNView Visualizer |
|     (c) 2017      |
+-------------------+
Sample ID file '100_S2' not found, assuming single sample ID provided
Loading required package: RMySQL
Loading required package: plyr
Loading required package: MASS
Loading required package: metap
Loading required package: Rsamtools
Filtering & loading coverage matrix... Complete
Error in .subset(x, j) : invalid subscript type 'list'
Calls: CNView -> [ -> [.data.frame
Execution halted

I saw an old issue posted here: https://github.com/RCollins13/CNView/issues/1 which had the same error but since it was fixed it's likely not to be the same problem.
Thanks for your attention!

Hi Lindsay,
Happy to help.
At first glance, the input coverage matrix looks like a potential source of the problem.
CNView expects the first column to be named “Chr”, not “chrom”. I am pretty sure this is what’s causing your problem. Refer to the example coverage matrix provided in the CNView documentation if you’re curious.
If that doesn’t work, here are a few more ideas that may also represent problems:

I’ve never tried 1bp intervals before, and CNView wasn’t necessarily designed with this scale of resolution in mind. If you’re interested in per-base coverage plots for small CNVs, I’d recommend something simpler like running a samtools mpileup over your region of interest and plotting that.

In my recollection, R doesn’t like column names that start with numbers. I thought we patched a fix for this into an earlier release, but you could try appending some string-based prefix, like “sample_”, before your sample IDs in the header column of your coverage matrix.

How many bins (rows) in total are in the matrix? CNView run at default parameters needs a large window of bins surrounding your region of interest (this is how the coverage normalization is performed).

Those coverage values look unusually low. What data type is this, and how are you collecting coverage? I don’t know what to expect if CNView were fed a matrix of 0’s and 1’s (or extremely sparse counts).

Hope this helps,
Ryan

On Dec 11, 2018, at 2:30 PM, Lindsay Liang<EMAIL_ADDRESS>wrote:
I have an input coverage matrix of 176 samples where across a small region; each row in the bedfile covers only 1bp (unlike your example, but the region is pretty small so I'm hoping this won't cause issues). There are no missing bp intervals in the file for the region.
Running the following:
Rscript CNView.R chrX<PHONE_NUMBER>96 417_S9  chrX-308758-314496_matrix.txt chrX-308758-314496.pdf
Where the matrix looks like:
chrom	start	end	100_S2	105_S5
chrX	308605	308606	0	0
chrX	308606	308607	0	0
chrX	308607	308608	1	1
chrX	308608	308609	1	1
chrX	308609	308610	1	1
chrX	308610	308611	1	1
...
I get the following error
Loading required package: optparse
+-------------------+
| CNView Visualizer |
|     (c) 2017      |
+-------------------+
Sample ID file '100_S2' not found, assuming single sample ID provided
Loading required package: RMySQL
Loading required package: plyr
Loading required package: MASS
Loading required package: metap
Loading required package: Rsamtools
Filtering & loading coverage matrix... Complete
Error in .subset(x, j) : invalid subscript type 'list'
Calls: CNView -> [ -> [.data.frame
Execution halted
I saw an old issue posted here: #1 https://github.com/RCollins13/CNView/issues/1 which had the same error but since it was fixed it's likely not to be the same problem.
Thanks for your attention!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/RCollins13/CNView/issues/10, or mute the thread https://github.com/notifications/unsubscribe-auth/AHjBxJ6sEFx33F1z9PNzoMgjtSlyN9a3ks5u4AfBgaJpZM4ZOFYh.

Hi Ryan,
Thanks for the response! I made the simple changes in the matrix by changing 'chrom' to 'chr' while also removing 'chr' from the rest of the rows in that column; that seems to have fixed the errors that caused the program to halt.  There was also an issue of R appending an X to the beginning of the sampleID variable, but I just added a line to remove that first character.
Thanks again!
-Lindsay


chore: release main

:robot: I have created a release beep boop
ui5-cc-excelupload: 0.13.0
0.13.0 (2023-04-13)
Features

add event uploadButtonPress (#143) (23b918c)
new option to upload in standalone mode (#145) (ead496f)

ui5-cc-excelupload-button: 0.2.0
0.2.0 (2023-04-13)
Features

add event uploadButtonPress (#143) (23b918c)
new option to upload in standalone mode (#145) (ead496f)

Dependencies

The following workspace dependencies were updated

dependencies

ui5-cc-excelupload bumped from 0.12.3 to 0.13.0

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/marianfoo/ui5-cc-excelUpload/releases/tag/ui5-cc-excelupload-v0.13.0 :sunflower:


[linux] Implement GetCurrentThreadStackBounds

A stub API- GetCurrentThreadStackBounds- is defined in CommonPal currently. We need to implement it to replace instances in the code where we rely on the Win32 specific-NtTeb to get the current thread's stack base and limit. One possible implementation for this API would be to use the NtTeb approach on Windows and pthread_attr_getstack on platforms with pthread support. It's unclear if we can do this without using pthread_attr_get_np though.

It seems the code already does what is described here (NTCurrentTeb on Windows, GetCurrentThreadStackLimits with pthread_attr_get_np on Linux). Is anything missing?

There is not much to do here. Closing


Cannot access share that uses a user with name and password guest

Describe the bug
Connecting to a shared folder that is shared with authentication and has a user name guestand password guest results in an empty folder with the hint to sign in through CIFS provider.
The credentials are of course set up correctly.
Changing the authentication user name and password in the share solves the problem and the folder can be accessed.
Linux and Windows 10 can access the share without problem.
Tested with a share made from Linux and Windows 10.
To Reproduce

Create a share with authentication and user name guest and password guest
Enter credentials and host in CIFS provider
Try to o open a shared folder of that host
See an empty folder with hint to sign in to this share

Expected behavior
The share is accessible.
Screenshots
CIFS Documents Provider

App Versin: 2.3.0
Connection Settings

Storage Type: SMB2,3(JCIFS-NG)
Authantication: User&Password
Safe Data Taransfer: no
Other: ...

App Settings

Show notification when opening files:
Use as local storage:
Other: ...

App installed device (please complete the following information):

OS: LineageOS 21 android 14
Device:
Cliant App:

Server (please complete the following information):

OS: arch Linux or Windows 10
Device: PC x86_64
Server Software: Samba or windows share

Additional context
Can someone please try to reproduce this?

This is because in cifs-documents-provider, once the username is left blank or set to 'guest', the guest credentials are used directly without trial and fallback processing.
A further reason is that cifs is a file sharing mechanism on Windows, and ‘guest’ is the default guest username on Windows. When this user is enabled, you can log in with any password.
It is theoretically possible to use 'guest' as a normal username, but there seems to be no strong enough reason to be compatible with this situation, especially since the current situation itself is caused by the behavior of compatiting guest users.
By the way, in the default behavior of the jcifs-ng library, once the username is set to guest, it will automatically replace it with GUEST.


remove obsolete test utils

Some files in test/utils/ were made obsolete by using orbit-db-test-utils
this PR:

removes unused utils files
removes unused IPFS imports test/offline-mode.js and test/v0-open-load.js

@tabcat Looks like you may need to rebase this again - there are changes in the concurrent browser tests that don't look like they've been picked up

@tabcat sorry :P wanna rebase again?

LGTM!


Add Loki Path config to change deprecated API usage

#172

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

9 of 9 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.009%) to 94.398%

Totals

Change from base Build<PHONE_NUMBER>:
0.009%

Covered Lines:
3404

Relevant Lines:
3606

💛  - Coveralls


llmq: Clean old DKG contributions up

DNM until after https://github.com/dashpay/dash/pull/4752 is merged

@UdjinM6 is this milestone correct?


No mobile formed

Please make a mobile from it please

This is a mod of a gamecube game - you need an emulator to run it. Gamecube emulators exist for mobile.
TLDR yes, a mobile version already exists.


Zend\Ldap\Dn::isChildOf - should it be case insensitive?

This is a "reopen" of zendframework/zendframework#6299
I just ran into the same problem. The comparison in isChildOf is case sensitive in regard to the attribute values (not the attribute names which have been changed to lower case before). This is incorrect if the attribute is, for instance, of type DirectoryString using matching rule caseIgnoreMatch for equality match.
For example, we use base DN "ou=people,o=ldap". A user DN might be "uid=test,ou=People,o=ldap" (note the capital "P").
isChildOf won't match in this case, even though the user DN is of a child of the base DN. This is incorrect as the ou attribute must be checked case insensitive.
As a quick workaround I modified the line
if ($cdn[$i + $startIndex] != $pdn[$i]) {

in Dn.php to this test:
$result = array_udiff_uassoc($cdn[$i + $startIndex], $cdn[$i + $startIndex], 'strcasecmp', 'strcasecmp');
if (! empty($result) ) {

This compares attribute names and values case insensitive. Of course, this is not complete as whitespace should be removed as well and some attributes in the DN might actually be case sensitive. Thus I guess ideally, the test should be done by an ldap query instead...

Thanks for raising the issue! I'll have an in depth look at it later.

The comparison should use the schema-information for the attribute to decide whether to check case sensitive or insensitive. As that might include a severe performance-issue I'll need to check how to implement that in a way that satisfies all needs…

This repository has been closed and moved to laminas/laminas-ldap; a new issue has been opened at https://github.com/laminas/laminas-ldap/issues/4.