Fix Helm binding errors

This PR aims to fix all the helm binding related issues.
Shifting from alpine docker image (fixes #358)
The Dockerfile in this PR is changed to FROM python:3.7-slim-stretch. This was chosen because it is one of the smaller debian images which have python 3.7 installed by default (the custom image is named as kapitan-debian below).
$ docker image ls
REPOSITORY           TAG                 IMAGE ID            CREATED             SIZE
kapitan-debian       latest              728041763c54        2 hours ago         1.04GB
deepmind/kapitan     latest              0670fbc69cde        4 days ago          860MB
python               3.7-alpine          39fb80313465        7 days ago          98.7MB
python               3.7-slim-stretch    7f8ea8958a02        7 days ago          155MB

I think the image size difference is not so bad (~ 160MB) given that musl-libc is going to be smaller than glibc to start with.
So far the image is working just fine (tested with the examples folder, plus helm binding works as well). Got decryption failed error when using --reveal option but I also get that using deepmind/kapitan image so I may just be doing it wrongly.
If this is acceptable, we can move on to change the base image of Dockerfile.ci as well.
Supporting subcharts
The binding should now correctly work with charts with subcharts
TODO:

[ ] create binary for OSX in addition to the existing linux binary and add them to pypi
[x] fix #362

I think we should also fix Dockerfile.ci

@uberspot
Should we change the name of the helm binding shared object according to which platform this is targeted, just as we did for kapitan binary? In my understanding @ramaro wants the so distributed for each of Linux and Mac. I'm working on cross compile for OSX now, which hopefully takes us somewhere

@uberspot
Should we change the name of the helm binding shared object according to which platform this is targeted, just as we did for kapitan binary? In my understanding @ramaro wants the so distributed for each of Linux and Mac. I'm working on cross compile for OSX now, which hopefully takes us somewhere

Sure, go for it!
Just don't forget to update the docs too if there's any mention of the .so there too. :)

Is there an update on this?

Hi @ademariag
Sorry I'd been busy and couldn't really get on this task until now.
@uberspot
Could you please have a look at the new the docker CI image and confirm whether everything that was there on the previous alpine version is available as before?

@uberspot
what would be the most convenient way to build and distribute helm binding for Mac in addition to the existing Linux binding? Is cross-compiling on Travis the way, or do you think there might be an easier path (e.g. is it possible to build natively on OSX machine on Travis from which we can copy the built shared object file onto our Debian one)?

May I suggest that we look at the helm binding in mac in another PR? I feel this one if getting more than it came for. This way we can unlock testing on the current state.

@ademariag
I totally agree. That way @taharah can get started on the multistage Docker image(s) too. @uberspot Let me work on OSX binding in another PR. I'm going to make this one ready for review.


Adapt new hass client

This time it actually works ;-)

I did test it this time and can confirm that it does work :).


Failing import instance

I'm trying to import an existing cloud sql instance, but it is failing with invalid credentials
$ terraform import module.dev_infra.module.mysql_db.module.google_mysql_db.google_sql_database_instance.default projects/<project-id>/instances/<instance-name>

Error: Error reading SQL Database Instance "<instance-name>": googleapi: Error 401: Request had invalid authentication credentials. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.
More details:
Reason: authError, Message: Invalid Credentials

@tahsinrahman i am facing this same issue today. Any resolution?

@tahsinrahman i am facing this same issue today. Any resolution?

We experience this too:
Error: Error waiting for Create Instance: error while retrieving operation: googleapi: Error 401: Request had invalid authentication credentials. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.
More details:
Reason: authError, Message: Invalid Credentials

It seems to only occur when we provision google_sql_database_instance resource. I've tried playing around with permissions but that doesn't seem to be the issue. In fact, the SQL db seems to be created even if the failure above occurs.

We experience this too:
Error: Error waiting for Create Instance: error while retrieving operation: googleapi: Error 401: Request had invalid authentication credentials. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.
More details:
Reason: authError, Message: Invalid Credentials

It seems to only occur when we provision google_sql_database_instance resource. I've tried playing around with permissions but that doesn't seem to be the issue. In fact, the SQL db seems to be created even if the failure above occurs.

@zs-codex try running terraform refresh. It worked for us

@zs-codex try running terraform refresh. It worked for us

@zs-codex try running terraform refresh. It worked for us

@tahsinrahman This happens on a "clean" deployment when no existing resources exist in the project - so I wouldn't expect this to happen unless there's some sort of race condition...

@zs-codex try running terraform refresh. It worked for us

@tahsinrahman This happens on a "clean" deployment when no existing resources exist in the project - so I wouldn't expect this to happen unless there's some sort of race condition...

The matter of "taking time to provision" happens during first time creation of a resource - or even during modification of a resources- if the creation (or modificaiton) takes that length of time which goes beyond the expiry of access token.
But the matter of authError, Message: Invalid Credentials is not an error faced ONLY while provisioning (or modifying) resources for that long. There are cases where this error is faced before the provision API call is even made.
For those cases, we did see that the terraform refresh solved the problem.

The matter of "taking time to provision" happens during first time creation of a resource - or even during modification of a resources- if the creation (or modificaiton) takes that length of time which goes beyond the expiry of access token.
But the matter of authError, Message: Invalid Credentials is not an error faced ONLY while provisioning (or modifying) resources for that long. There are cases where this error is faced before the provision API call is even made.
For those cases, we did see that the terraform refresh solved the problem.

Awesome, terraform refresh solved the problem here too, thanks a lot


Prepare for publishing

complete READM & Cargo.toml
add documentation

NB : the documentation is only in the native files since it's what's used when running cargo doc. But both API are the same.
In the process, I removed a few unused functions in native_app.rs.
I'm not merging this right now as the web build is broken (thread 'main' panicked at 'unsupported custom section: '.debug_info' and sometimes rustc crash with exit code:<PHONE_NUMBER>). The native part runs fine though.
I'll merge as soon as everything works. In the meantime, you can check this PR in case something's missing.

So I noticed that Non Lexical Lifetime wasn't required by this crate. Removing the feature allow to compile it with stable Rust 1.28. wasm target still requires nightly, but this is a stdweb requirement.
I also added a App::exit() function to provide a native/web compatible way of exiting the process. On the web, it does nothing!


Dashboard/feature/implement validation

Task 1:
Validation of all data need for creating a new restaurant
Task 2,
Connect the dashboard with the add restaurant end point


align_double_arrow fixer sometimes aligns with extra whitespace

Before:
$foo = [
    "bar" => 1,
    "baz"   => 2,
];

After:
$foo = [
    "bar"   => 1,
    "baz"   => 2,
];

What I would expect:
$foo = [
    "bar" => 1,
    "baz" => 2,
];

Maybe this wasn't the intention of the fixer, so it's not an issue, but I often rename variables in ways that leave double arrows too far to the right, and I was expecting the fixer to use as little whitespace as possible to align them.  I created a failing test case and tried to fix it, but the code in this fixer is a little over my head.

@dericofilho what do you think ?

I mentioned this issue before. If we have way too many spaces, the fixer won't strip them. It'll just align everything really far over.

align_double_arrow is meant to align to the rightmost T_DOUBLE_ARROW. Back then, when we debated about that, I argued that this fixer should be somehow tied to another fixer (that, AFAIK, does not exist in PHP-CS-Fixer) that would do space normalization first.
In the case of php.tools, I do have a Pass (a.k.a Fixer, here) that normalize spaces between tokens before actually aligning T_DOUBLE_ARROW or =.
So it is a schrodinger bug. It is and it is not wrong.

Currently working around this by fixing twice:
First with unalign_double_arrow and again with align_double_arrow. Thus stripping extra spaces before aligning with the minimum offset.
I've tried using both fixers at once, but that didn't work for me. Is there some other way without running php-cs-fixer twice?

if we make the prio of unalign_double_arrow < of again with align_double_arrow then running the fixer with both fixers might do the trick ;)

I've tried using both fixers at once, but that didn't work for me. Is there some other way without running php-cs-fixer twice?

You MUST NOT enable those fixers at the same time. They are conflicting. In 2.0, we're working on a way for php-cs-fixer to actually stop people doing that.

@GrahamCampbell is right

Closes due to merge of #2573


7.7.1 Release

This PR prepare 7.7 branch for 7.7.1 release:

bump chart and Docker images version from 7.7.1-SNAPSHOT to 7.7.1
remove notices on README that this is a development branch and development branch specific install instructions

Note

remove notices on README that this is a development branch and development branch specific install instructions

This is required because released Helm charts will be packaged from this PR commit including the README files that will be available inside the chart packages and in Helm Hub pages (example: Elasticsearch chart page).
This change will be reverted in the next PR after release when we will bump 7.7 branch to 7.7.2-SNAPSHOT version.

7.7.1 Release changes (to add to GitHub release and changelog):

7.7.1 as the default stack version
K8S 1.16 support in #635 @jmlrt

Elasticsearch

PR
Author
Title

#596
domgoodwin
Elasticsearch secret mount mode

#638
@fatmcgav
Fix issue with readinessProbe causing outages

#639
@coolbaluk
Fix values links in examples/multi/README.md

Kibana

PR
Author
Title

#555
@ramrodo
Add extensible label support on Kibana

#637
@maxkochubey
String/YAML conditions for .Values.{extraContainers,extraInitContainers}

Metricbeat

PR
Author
Title

#622
@jmlrt
Fix deployment upgrade by removing chart label from .spec.selector.matchLabels

#624
@jmlrt
Use deprecated kube-state-metrics-metricbeat.yml when existing

#634
@MaxRink
Fix configChecksum not being set


ORC-1602: [C++] limit compression block size

What changes were proposed in this pull request?
limit compression block size on c++ side.
Why are the changes needed?
to fix https://github.com/apache/orc/issues/1727
How was this patch tested?
UT passed
Was this patch authored or co-authored using generative AI tooling?
NO

Why are the changes needed?[

Remove [

LGTM, thanks @ffacs !

Merged to main/2.0/1.9.
To @ffacs , could you make a backporting PR to branch-1.8 and branch-1.7 ?


ja: Make docs/setup/production-environment/tools/kubeadm/setup-ha-etcd-with-kubeadm.md follow v1.17 of the original text

This is a Feature Request

What would you like to be added
Update docs/setup/production-environment/tools/kubeadm/setup-ha-etcd-with-kubeadm.md to follow v1.17 of the original (English) text.
英文のまま更新するだけで十分です。
翻訳は必須ではありませんが、歓迎します。翻訳用にIssueを作成する必要はありません。
Why is this needed
content/ja/docs/setup/production-environment/tools/kubeadm/setup-ha-etcd-with-kubeadm.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.17-ja.3/content/ja/docs/setup/production-environment/tools/kubeadm/setup-ha-etcd-with-kubeadm.md
Original
https://github.com/kubernetes/website/blob/release-1.17/content/en/docs/setup/production-environment/tools/kubeadm/setup-ha-etcd-with-kubeadm.md
diff between translated and v1.17
https://gist.github.com/2a34a0aa4fbb9a2a5d87be17df73ebbb
Note
Currently, we use dev-1.17-ja.3 branch. Please open a PR targeting the branch.
ref
How To Contribute
翻訳スタイルガイド
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel

/assign

/close

/close


Set GitHub Build Passing Badge to Reflect Only Builds on the Main Branch

Otherwise, when PR branch builds fail, this badge will reflect that build status. I intend for the badge to reflect main/HEAD.

nvm :)


[BUG][2.11] Azure AD China auth returning "error status code received from the API

forwardport #47251

Verified on v2.11-3ce716825fdd85fc05ce5a8fdc97e345455b241b-head
Verified Azure AD can be successfully connected by selecting China Endpoint.
Verified Azure users can log in and have correct permissions.
Test steps:

Register Rancher with Azure China and set the Redirect URI to <MY_RANCHER_URL>/verify-auth-azure.
Create a new client secret and save it.
Set required API permissions for Rancher.
Allow Public Client Flows.
Copy Azure China application data to the forms in Rancher AzureAD authentication provider. The endpoints should be selected for China.
Click Save and login to Azure account from the popup window.
The Azure AD auth provider becomes active.
Add Azure user user1 as [Cluster Member], log in with user1, and verify that the user permissions are correct.
Add Azure user user2 as [Project Owner], log in with user2, and verify that the user permissions are correct.


dataset dicom image and mask

thank you for share the project and video files,i want to retrain the segmentation model,but i can't get any video image labels.
can you share it?
thank you!!
chenjunqiang

The labels are in the CSV file.


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2023-03-13)
Features

build: add automatic release management workflow (703f9da)
build: add release announcement  (1c19140)
build: add release docker image workflow (fc89d83)
build: add triage workflow (258b92c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/evantill/docker-cheerpj/releases/tag/v1.0.0 :sunflower:


Added -u/--strict command line option to add the 'use strict' directive

As per issue #12, I have added a new command line option -u/--strict, which will insert the 'use strict' directive after the genversion signature line, but as the first statement in the file.
It honours the --semi command line flag.
I have added appropriate tests and they pass.  Note that I had to prepend the command with "node" to make it work on my Windows platform which can't read the shebang (#!) line at the top of bin/genversion.js.  I think this should also work on *nix platforms.

Thank you @RossVertizan!


Focus the  text input on non-mobile devices

Fixes #73. 🚀

@IonicaBizau this can't be done in the index page, but in the Search component. Also the mobile detection is pretty naive.

@satazor I'm not really familiar with the code—what file should be edited?
That's the easiest way to check if it's mobile or not. 😁 We can check for the touchevents in the window.

Add componentDidMount after this code here: https://github.com/npms-io/npms-www/blob/master/src/shared/containers/search-box/SearchBox.js

https://github.com/npms-io/npms-www/commit/49ab663d9ff7335c85ca10a960ff97cf5f9b0d95


Allow sort by date added

I can sort projects by age:
https://landscape.cncf.io/card-mode?project=sandbox&sort=first-commit
but not by "date entered the CNCF".
It would be great to be able to use the tool to search on "date added" and "date last level moved", at least; I can imagine it would be harder in the UI to expose "date moved to incubation" for a graduated project.

Hi!
We've just migrated the CNCF landscape to v2 and this issue probably no longer applies.
Please feel free to reopen if needed 🙂

In v2, I can't (a) sort by "date added" (b) sort by "first commit" - a regression (c) show all projects without the grouping by landscape categories.
/reopen

Also not able to sort by Github stars in v2

That's a significant regression in UX of v2 and in https://devstats.cncf.io/ the metrics isn't accessible as it was in v1.
I used to CSV export to find the data
https://landscape.cncf.io/docs/items.csv

We've just added some extra classification / sorting options for cards to v2 that should cover the ones previously available in v1 plus the new one (date) requested in this issue 🙂

Thank you, awesome!
On Fri, Apr 5, 2024 at 12:44 PM Sergio Castaño Arteaga <
@.***> wrote:

Closed #3555 https://github.com/cncf/landscape/issues/3555 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/cncf/landscape/issues/3555#event-12377536324, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPSIK5QYVTC7NB5F6YF23Y33PHTAVCNFSM6AAAAAA543XGK6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGM3TONJTGYZTENA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Cheers,
Chris Aniszczyk
https://aniszczyk.org


Need to update from Ansible Operator 0.9.0 base

I think operator-sdk pinned this. Once we do a sanity check it's probably fine to use latest. Better to catch problems early.


fix: icon button isDisabled prop

Summary

<IconButton> isDisabled is not working properly.
Try working with Snack.
https://snack.expo.dev/@suzu_prog/nbiconbutton-isdisabled-bug
<IconButton> has isDisabled, but onPress works.
<Button> has isDisabled working fine.
Therefore, onPress does not work.
Changelog

[General] [Fixed] - Fix icon button isDisabled prop
Test Plan

https://user-images.githubusercontent.com/60243840/192835079-01155e6c-488e-4a3f-b79a-7a299bce7a33.mov

@suzu-prog Thanks for your contribution ❤


Unable to control directional tubes.

To change directional tube state, you have to hit its center. Which is near impossible in cramped setups due to both tiny size of the tube center's hitbox and disabled tube extensions' extents being always present in the world.
I suggest to take the example from the best and only present extents when the extension is actually present.
If extension is disabled, hit the central block's face to "pull it off" into the world.
Increase hitbox of the tube core to 6x6x6 or even 8x8x8.
Unify all tubing controls. Click with the wand… ok, gauntlet, to turn connection on/off. Crouch-click to alter the configuration of the connection without altering its state. For directional tubes, that means setting(toggling) preferred direction directly. (pun intended)

Crouch-click on the tube core to take tube away… yes I'm too used to "BC compatible wrench"…

This issue was moved to Azanor/thaumcraft-suggestions#390


Improve ergonomics of adding new iterators

Adding the new iterators was kind of a drag, in particular there is a lot of generic boiler plate stuff.
I've made a prototype of what this might look like in https://github.com/medwards/transitfeed/blob/dff5d12de20c1d4f016ea0cb55ab0b15a9c756f2/src/gtfs/stop_times.rs and would like your feedback.
Basically GTFSIterator accepts a Csv like it does now but also a parsing function that it applies during the iteration.
GTFSIterator would end up in gtfs/gtfs.rs or gtfs/mod.rs and I'm thinking of adding a function that does the composition for you if you pass the filepath. I also want to move error capturing/transformation up to GTFSIterator.next instead of it happening in various places (value parsing functions, etc.)
Let me know what you think!

I'm impatient so I just went ahead and did it XD
medwards/transitfeed@e99e68a9a29636b70266be1516725e09f112862a

This is a nice set of changes and definitely DRY's some stuff up.
I've been debating on moving things to the csv crate in the future once 1.0 is out, this seems like it would atleast begin helping reduce the amount of changes needed for that as well.
Great stuff! Send a PR when its ready and I'll review.


Divi/portfoio details

The conflicts on PortfolioContainer are pretty heavy. I've resolved them at the base of my branch here: https://github.com/calblueprint/RMI/compare/ethan/delegation-popover?expand=1

Conflicts resolved, merged in #266


Fix detaching in sentiment analysis notebook

What does this PR do?

Removes redundant detaches.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


:new: Add dialog component

Port of PR #1929 to have recent changes accounted for. Rebase blew up.

This implements the dialog component which uses the native dialog element under the hood.
This method reduces the amount of code we are to maintain internally while relying on a high-quality polyfill of the standard for non-supporting browsers. This leads to cleaner code for developers.
@sgomes brought up in the other PR for Dialog that we would be forcing developers into using the dialog without a way to opt-out of some global event handlers the polyfill uses. I went the other way around and made using dialogs at all entirely opt-in. If you don't need dialogs (a good number of sites may never need them) then you simply don't include the polyfill on your site and there is only a minimal amount of extra CSS being added. However, if you do need dialogs then it is up to you to include the polyfill as-needed for your projects workflow.
UPDATE: Having a few issues working out the polyfill. Seems like the JS isn't running like it should be.
Before merge:

[ ] Move CDNJS polyfill for demos to local dep

@sgomes Figured out the "not working at all in FF and IE" problem. Rereading the example code for about the 30th time (this time with fresh eyes) I saw:

dialogPolyfill.registerDialog(dialog);

So people need to register up the dialog elements so the show, showModal, and close methods are properly added. Updating our docs to show this, the snippets, and going to PR an update to the upstream readme to help make it more clear there as well.

I can confirm that it now works fine in FF. Having some trouble getting IE VMs running.
If anyone wants to before I can get to making them work, download the branch, npm install && gulp all && gulp serve then go to http://localhost:5000/components/dialog/demo.html and the demo will run.

Will this be implemented soon? I'm pretty sure a lot of us all waiting for this feature...

It is moving as quickly as it can. We know this is very much wanted (along with a slew of other features.) Commenting simply asking for it to get done doesn't help it get done any faster, it just creates noise and slows things down more.

What more has to be done before this is finished? It looks dormant for the past 16 days.

Um, it was holiday break for like everyone at Google. Which is pretty much everyone else who needs to review this in order to approve for merge.
We are working really hard towards 1.1, but this PR does mean requiring a polyfill and making some modifications to the microsite template for the demonstrations to work properly.
Let's not try rushing everything through, it isn't going to be ready until 1.1 is tagged anyways. Until then you can pull the styles in manually and use the polyfill. It shouldn't be that hard if you really need it now.

@Craytor thanks for your interest! We would love to have your help on the project if you are interested, feel free to pick an issue https://github.com/google/material-design-lite/labels/Tooling (tooling ones tend to be fun :smile: )
OSS === Everyone working together to make something awesome :sparkles:

@Garbee @samccone Thank you all! I really do like the project, I'll see what I can work on.

The timeframe at this point is, as soon as we can get all the work done. I kinda did last-minute (sorry, I did it again) patches that still need review and we are missing some new docs for additions. Once that is all cleared up, it should be good to ship pending review and testing.

@Garbee Okay! Thanks for the info! By chance, does Material Design Lite have a Gitter (chat system thing)?

Moving to codepens for the demos is going to need a switcharoo right after tagging and CDN push to keep things clean. Right now, I need to add in the extra JS and CSS that isn't in the CDN copy. Then apply the JS needed to make the demo work.
Handling these now, just a heads up that it will seem messy at first, because it needs to be. It will get fixed up post-release.

Ok, codepens are figured out and live. I just added in quick "See this live in codepen" links with each example. We should probably look into making them more noticeable later. Since this component is using a simplified doc layout anyways, that may not be a big problem. Analytics/issue reports will tell.
Updated the PR with:

Codepen links to examples
Prioritized showModal over show since having the backdrop aligns with the spec more.
Removed internal snippets since they are being hosted on Codepen.
Removed references to dialog polyfill in doc resources since we aren't doing internal demos.

Tested in IE10, Firefox, and Chrome. All checks out fine in these browsers. If someone could please test Safari (and iOS since there are special bugs there too) that would be great.

Oh, one other really important update in the last push. I improved the action selectors. Now instead of targeting all descendants down the action element for heights, it now targets only the direct descendants. So * was changed to > * for the selector.

I loaded up the codepen links in desktop safari and they look good. Except that there seems not enough room to fit the 2 buttons for the 'Simple Dialog' example in safari.

If Safari alone is having issues, then someone else needs to debug that and recommend a fix. Nothing I can do since I don't have access to any of its incarnations.

Gave it a run on mah browerz, lookin’ good. Code LGTM as well. Waiting for @sgomes

First of all, sorry for the late review :) Secondly, great work, @Garbee, love what you have here!
I found a couple of issues with the codepens, that we may need to solve before we release:

The codepen demos are larger than 100%, causing the demo to be scrollable, which can be a bit confusing, and make it look like it's a dialog problem, at first glance. I suggest adding box-sizing: border-box to the body to fix that.
I'd prefer to see the dialog polyfill files imported explicitly in the HTML, rather than using codepen's CSS and JS import functionality, since that's not immediately visible to users. That will hopefully emphasise that the polyfill is needed for most browsers.

Merging, since the issues I mentioned are in the codepens and not mdl-dialog itself.

I'll add a very clear note in the content under the button to raise awareness of the polyfill being needed. Developers should decide for themselves how to include it, not just copy whatever the codepen has.

The mdl-dialog__title font seems awfully large.  (see the MDL Dialog above and in http://codepen.io/Garbee/full/EPoaMj/)  is 2.5rem correct?

There is no specific guidance in the spec on what exactly the font size should be. Even within the spec, the spacing per-line differs. So, I just made something up that works good with short titles. If that doesn't work for you, then there are typography heading classes that can be used to get a different size in leu of the title tag here.
Please file new issues for possible problems after a PR merge. That way things are easily trackable. Taking comments on after a review gets confusing when jumping between issues.


"Unhandled error, please open a Github issue so we can fix it!" When losing DNS name resolution

I upgraded the firmware on my modem causing the Internet to drop. That in turn caused an unhandled error:
2017-08-26 13:13:37 Error HTTPSConnectionPool(host='api.bitfinex.com', port=443): Max retries exceeded with url: /v1/pubticker/dshusd (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7fe3a4c80590>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution',)) Requesting https://api.bitfinex.com/v1/pubticker/dshusd
Traceback (most recent call last):
File "lendingbot.py", line 90, in
Data.update_conversion_rates(output_currency, json_output_enabled)
File "/home/jcarlos/poloniexlendingbot/modules/Data.py", line 85, in update_conversion_rates
ticker_response = api.return_ticker()
File "/home/jcarlos/poloniexlendingbot/modules/Bitfinex.py", line 150, in return_ticker
ticker = self._get('pubticker/'+symbol)
File "/home/jcarlos/poloniexlendingbot/modules/Bitfinex.py", line 75, in _get
return self._request('get', command)
File "/home/jcarlos/poloniexlendingbot/modules/Bitfinex.py", line 69, in _request
raise ex
ConnectionError: HTTPSConnectionPool(host='api.bitfinex.com', port=443): Max retries exceeded with url: /v1/pubticker/dshusd (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7fe3a4c80590>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution',))
Unhandled error, please open a Github issue so we can fix it!
Traceback (most recent call last):
File "lendingbot.py", line 137, in
log.notify("{0}\n-------\n{1}".format(ex, traceback.format_exc()), notify_conf)
File "/home/jcarlos/poloniexlendingbot/modules/Logger.py", line 155, in notify
send_notification(msg, notify_conf)
File "/home/jcarlos/poloniexlendingbot/modules/Notify.py", line 122, in send_notification
post_to_slack(msg, nc['slack_channels'], nc['slack_token'])
File "/home/jcarlos/poloniexlendingbot/modules/Notify.py", line 46, in post_to_slack
response = urllib2.urlopen(url, enc_post_data)
File "/usr/lib/python2.7/urllib2.py", line 154, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python2.7/urllib2.py", line 429, in open
response = self._open(req, data)
File "/usr/lib/python2.7/urllib2.py", line 447, in _open
'_open', req)
File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 1241, in https_open
context=self._context)
File "/usr/lib/python2.7/urllib2.py", line 1198, in do_open
raise URLError(err)
urllib2.URLError: <urlopen error [Errno -3] Temporary failure in name resolution>

This is a local issue with your DNS resolution, can we close this ticket?

It is an unhandled error which I caused but could happen at any time. I guess in my view there needs to be instead of a fatal error perhaps a sleep time before trying again. The lending bot stopped working after that even after the internet came back up.

If this happens during runtime the bot will continue after sleeping, this is handled in the bot loop.

It was during run time and it stopped working. Sometimes I guess the loop doesn't work in my experience... having that issues right now with the "Error list index out of range." Eventually it crashes for good.


After last Windows update, creating new entries in Toggl is completely broken

Hello,
since last Windows app update, working with Toggl seems completely impossible. So far I have found out that:

when adding new entry I cannot specify its description (on the video, I am hitting Enter key without a value submitting successfuly)

when modifying time started and then entry's description, time elapsed disappears

application synchronizes itself while editing an entry and re-editing my entry while working on it

Please, can you help me with the issues?
Thanks!
Regards,
Martin

I have also tried to reinstall an older version but after reinstall, Toggl immediately updates itself. Please, how can I prevent Toggl from updating?

Case/486812

/case/486048

Same problem here (and in my office), we all have the same problem (Windows 7 & 10)

Same issues for me on Windows 7 in my office (with a proxy setup, that was running fine with previous versions). I'm using the latest Windows version : 7.4.38.
The issues were present in a former version but I didn't take note of the version number.
I updated to latest version 7.4.38 but the issues are still present.

Seems to be fixed in 7.4.45 release.

We've released a new beta version that addresses the flickering issue. Please get the latest beta (7.4.47) by switching to beta channel from the About view

7.4.47 seems to be working ok. Pushed 7.4.47 to stable.

Hello @IndrekV,
thank you for such a quick fix. I see most of the issues gone. However:

adding new entries still misbehaves in way that there is always something fleshing (e.g. creating new empty entry for a second fills in information from old entry and then fleshes to a clear one)
modyfing entry data persists but for a seconds fleshes with previous data,
when creating new entry, the right panel isn't opening right away and I need to manually open it by clicking to entry description - the former things are simply weird, but this one really affects me as I have to do one redundant click.

Thank you,
Martin

@dulakm Thank you for all the details. We have reworked the syncing part to match the way it used to work before the glitches started happening. We've released a beta version with fixes v7.4.49. Switch to beta channel if you want to check out the fixes now or wait until next week when it will be pushed to stable.
As for the right panel not opening - This has always been so, but in macos there is a setting to enable opening edit view when user starts timer with a shortcut. We'll look into the possibility to add this also for windows.


Pheanstalk needs maintainer(s)

Many people are using Pheanstalk, but I haven't been able to give it the attention it deserves.
I'd love to take on one or more maintainer (two sounds like a good number) to help push it forwards.
Anybody interested?
Let me know in what capacity you use Pheanstalk (e.g. for what purpose, at what company etc) [Optional. And be as vague or specific as you want].
And also what kind of changes you'd like to push forwards, or if you'd rather just work on fixing general maintenance issues that arise.
Cheers,
Paul

I'd like to volunteer!

Is this still open? I'd like to contribute.

I would also like to actively contribute.

Everyone who posted they want to contribute, I'll be happy to review any suggestions and PRs.
I've labeled issues that with the ready for adoption label in case the proposed approach is acceptable.
Feel free to submit PRs!


feat: Add sentiment analysis notebook

This PR contains notebook which demonstrates performing sentiment analysis on stanford movie reviews dataset using AutoML NLP, deploying the sentimental model on Vertex AI and getting predictions.

[x] This notebook has been added to the CODEOWNERS file under the # Community Notebooks section, pointing to the author or the author's team.
[x] Passes all the required formatting and linting checks. You can locally test with these instructions.

Thanks for resolving the issues @sudarshan-SpringML! I found a few more issues in a final review pass. I think that should be all the issues. Appreciate your contribution!

I'll merge end-of-week so you have time to make any more changes if needed.

Thanks! Ivan
On Thu, Jan 20, 2022, 11:40 PM Ivan Cheung @.***> wrote:

I'll merge end-of-week so you have time to make any more changes if needed.
—
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/vertex-ai-samples/pull/195#issuecomment-1017780838,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATV6CWFYDKMT34VOZSOK573UXBFX3ANCNFSM5KWSOCHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Missing PR description.

Missing PR description.

Could you please tell me how to add it

Missing PR description.

Could you please tell me how to add it

You can edit the first comment here, adding the description of this notebook above the checklist @sudarshan-SpringML .

Thanks Karl
On Mon, Jan 24, 2022 at 9:58 PM Karl Weinmeister @.***>
wrote:

Missing PR description.
Could you please tell me how to add it
You can edit the first comment here, adding the description of this
notebook above the checklist @sudarshan-SpringML
https://github.com/sudarshan-SpringML .
—
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/vertex-ai-samples/pull/195#issuecomment-1020286720,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATV6CWDYMM4KMTHWCLER5ULUXV43PANCNFSM5KWSOCHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Sudarshan C |<PHONE_NUMBER>
Associate Data Engineer | SpringML
https://springml.com/?utm_source=WiseStamp&utm_medium=email&utm_term&utm_content&utm_campaign=signature
https://www.surveymonkey.com/r/3VHQ77R
Blog: Right Data Strategy: The Foundation for any AI Implementation
https://www.springml.com/blog/right-data-strategy-the-foundation-for-any-ai-implementation/

Missing PR description.
PR description added

Missing PR description.

resolved


Impossible to create new syntax based on emojis

I'm a completely macro noob trying a crazy experiment of replacing JavaScript keywords with emojis. My first endeavor would to be replace const with 💎.
So 💎 a = 1 would turn into const a = 1
This is the macro I'm attempting:
macro 💎 = function (ctx) {
  let ident = ctx.next().value;
  ctx.next(); // eat `=`
  let init = ctx.expand('expr').value;
  return #`const ${ident} = ${init}`;
}

💎 Droid = 'BB-8';

But it fails.
If I try a regular identifier such as whatever it does work as you can check in the online editor
I got here by first asking a question in the gitter channel

Start of relevant conversation https://gitter.im/sweet-js/sweet.js?at=57f400fe43b5aeb00998bbb2

The reason this fails is because 💎 isn't a valid identifier. This particular use case (aliasing keywords) could be accommodated using reader macros when they land.

This should be enabled as soon as we finish #687.


Catimg not working on neofetch but by using the command yes

I am not able to make it work by my side. But CatImg works
2020-10-09 14:17:52 ⌚  Auth in ~ ○ → neofetch -vv --catimg transparent--1-.png 2> neofetchlog
https://ptero.co/iduwiwohoc.diff
Any idea why?

#1392

I know but I couldn't make it to work even by reading every comments... Any ideas?

Post your issue there is all.


Service "Test//testDSM.ImageServer" is associated with a portal item that does not exist

Service "Test//testDSM.ImageServer" is associated with a portal item (eda15c522bae4d4095da7863421a11de; ImageServer) that does not exist.
Most likely cause of issue, publisher intended to delete the non-hosted service, but deleted it by deleting the portal item(s). The correct way to delete a non-hosted service is to delete it through the AGS connection which will delete both the service and the associated portal items.

@NatalieCampos @nfeuerstein I will go ahead and delete this image service.

@NatalieCampos  @nfeuerstein I have delete this image service. Closing issue.


Remove shell e2e test

Removing deprecated e2e test framework in favor of the golang framework
none

/LGTM

Please wait. @frankbu FYI.

@sebastienvas: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-no_auth.sh
e79b9c5abcce3bfc0a77f2b6163c29f8b4b1288f
link
/test e2e-suite-rbac-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sebastienvas: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-no_auth.sh
e79b9c5abcce3bfc0a77f2b6163c29f8b4b1288f
link
/test e2e-suite-rbac-no_auth

prow/e2e-suite-rbac-auth.sh
e79b9c5abcce3bfc0a77f2b6163c29f8b4b1288f
link
/test e2e-suite-rbac-auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

The shell e2e is the only one that works on minikube, so before it's removed the go version should be fixed to work in minikube too.

@frankbu Have you tried the --use-local-cluster when running the tests ?

I don't think this is any different:
https://github.com/istio/istio/blob/c2208fe77b23d9f764d9ad38182b3461874ef141/tests/kubeUtils.sh#L84
https://github.com/istio/istio/blob/master/tests/e2e/util/kubeUtils.go#L165
I think the golang framework changes loadbalander to nodePort such that it does not create the ingress.

@sebastienvas please make sure it runs first, maybe ask someone with minikube (I don't have on installed yet, sorry). The flag name out-of-cluster can mean either GKE or minikube, and those have differences between them.

@kyessenov I am asking @ZackButcher to test, I am just stating that the shell framework does not have anything different to make it work on minikube.

@sebastienvas I had not tried --use-local-cluster, but it does look like it will do the right thing. I think the problem is that you changed the default behavior. With kubeTest.sh, the default was some kind of not-not-out-of-cluster (not sure what that meant), but it worked on minikube without specifying any args. Just running plain tests/e2e.sh does not work on minikube. Maybe you can change the default, or at least document clearly how to run on minikube, because I think that plenty of people will be using it.

@frankbu I added some documentation. Please review.

@sebastienvas: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-no_auth.sh
e79b9c5abcce3bfc0a77f2b6163c29f8b4b1288f
link
/test e2e-suite-rbac-no_auth

prow/e2e-suite-rbac-auth.sh
e79b9c5abcce3bfc0a77f2b6163c29f8b4b1288f
link
/test e2e-suite-rbac-auth

prow/new-e2e-rbac_no_auth.sh
a0a99631f0cc8b6deedb174bf0a01b981e0cafc2
link
/test new-e2e-rbac_no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sebastienvas: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-no_auth.sh
e79b9c5abcce3bfc0a77f2b6163c29f8b4b1288f
link
/test e2e-suite-rbac-no_auth

prow/new-e2e-rbac_no_auth.sh
a0a99631f0cc8b6deedb174bf0a01b981e0cafc2
link
/test new-e2e-rbac_no_auth

prow/e2e-suite-rbac-auth.sh
a0a99631f0cc8b6deedb174bf0a01b981e0cafc2
link
/test e2e-suite-rbac-auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sebastienvas: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/new-e2e-rbac_no_auth.sh
a0a99631f0cc8b6deedb174bf0a01b981e0cafc2
link
/test new-e2e-rbac_no_auth

prow/e2e-suite-rbac-auth.sh
a0a99631f0cc8b6deedb174bf0a01b981e0cafc2
link
/test e2e-suite-rbac-auth

prow/e2e-suite-rbac-no_auth.sh
a0a99631f0cc8b6deedb174bf0a01b981e0cafc2
link
/test e2e-suite-rbac-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sebastienvas: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/new-e2e-rbac_no_auth.sh
a0a99631f0cc8b6deedb174bf0a01b981e0cafc2
link
/test new-e2e-rbac_no_auth

prow/e2e-suite-rbac-auth.sh
2dbcaebd793bb38f09f4b589bf424ca777b0a014
link
/test e2e-suite-rbac-auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Different prediction results after loaded model training

I implemented the Deep autoencoder from https://blog.keras.io/building-autoencoders-in-keras.html and I am currently trying to save the trained model and restore it to enhance it later.
The problem: When I run the example with 36 epochs in 1 step I am getting different results as when I run the example with 12 epochs in 3 steps.
1 Step with 36 epochs:

3 Steps á 12 epochs:

Shouldn't it be the same result or am I mistaken here? Thank you in advance for any clarification.
This is the current code:
# Taken from https://blog.keras.io/building-autoencoders-in-keras.html
import os.path

from keras.layers import Input, Dense
from keras.models import Model, load_model

# model file name
model_file_name = 'mnist-autoencoder.h5'

# encoder file name
encoder_file_name = 'mnist-autoencoder-encoder.h5'

# decoder file name
decoder_file_name = 'mnist-autoencoder-decoder.h5'

# this is the size of our encoded representations
encoding_dim = 32  # 32 floats -> compression of factor 24.5, assuming the input is 784 floats

# this is our input placeholder
input_img = Input(shape=(784,))

# "encoded" is the encoded representation of the input
encoded = Dense(128, activation='relu')(input_img)
encoded = Dense(64, activation='relu')(encoded)
encoded = Dense(32, activation='relu')(encoded)

# "decoded" is the lossy reconstruction of the input
decoded = Dense(64, activation='relu')(encoded)
decoded = Dense(128, activation='relu')(decoded)
decoded = Dense(784, activation='sigmoid')(decoded)

# create a placeholder for an encoded (32-dimensional) input
encoded_input = Input(shape=(encoding_dim,))

# load or create model
if os.path.isfile(model_file_name) and os.path.isfile(encoder_file_name) and os.path.isfile(decoder_file_name):
    autoencoder = load_model(model_file_name)
    encoder = load_model(encoder_file_name)
    decoder = load_model(decoder_file_name)

else:
    # this model maps an input to its encoded representation
    encoder = Model(input=input_img, output=encoded)

    # this model maps an input to its reconstruction
    autoencoder = Model(input=input_img, output=decoded)

    # First, we'll configure our model to use a per-pixel binary crossentropy loss,
    # and the Adadelta optimizer:
    autoencoder.compile(optimizer='adadelta', loss='binary_crossentropy')

    # retrieve decoder layers of autoencoder model
    decoder_layer1 = autoencoder.layers[-3]
    decoder_layer2 = autoencoder.layers[-2]
    decoder_layer3 = autoencoder.layers[-1]

    # create the decoder model
    decoder = Model(input=encoded_input, output=decoder_layer3(decoder_layer2(decoder_layer1(encoded_input))))

# Training

# Let's prepare our input data. We're using MNIST digits, and we're discarding
# the labels (since we're only interested in encoding/decoding the input images).
from keras.datasets import mnist
import numpy as np
(x_train, _), (x_test, _) = mnist.load_data()

# We will normalize all values between 0 and 1 and we will flatten the
# 28x28 images into vectors of size 784.
x_train = x_train.astype('float32') / 255.
x_test = x_test.astype('float32') / 255.
x_train = x_train.reshape((len(x_train), np.prod(x_train.shape[1:])))
x_test = x_test.reshape((len(x_test), np.prod(x_test.shape[1:])))
print x_train.shape
print x_test.shape

# Now let's train our autoencoder for 50 epochs:
autoencoder.fit(
    x_train,
    x_train,
    nb_epoch=12,
    batch_size=256,
    shuffle=True,
    validation_data=(x_test, x_test)
)

# Save trained autoencoder model
autoencoder.save(model_file_name, overwrite=True)

# Save encoder with weights
encoder.save(encoder_file_name, overwrite=True)

# Save decoder
decoder.save(decoder_file_name, overwrite=True)

# After 50 epochs, the autoencoder seems to reach a stable train/test loss
# value of about 0.11. We can try to visualize the reconstructed inputs and
# the encoded representations. We will use Matplotlib.
#
# encode and decode some digits
# note that we take them from the *test* set
encoded_imgs = encoder.predict(x_test)
decoded_imgs = decoder.predict(encoded_imgs)

# use Matplotlib (don't ask)
import matplotlib.pyplot as plt

n = 10  # how many digits we will display
plt.figure(figsize=(20, 4))
for i in range(n):
    # display original
    ax = plt.subplot(2, n, i + 1)
    plt.imshow(x_test[i].reshape(28, 28))
    plt.gray()
    ax.get_xaxis().set_visible(False)
    ax.get_yaxis().set_visible(False)

    # display reconstruction
    ax = plt.subplot(2, n, i + 1 + n)
    plt.imshow(decoded_imgs[i].reshape(28, 28))
    plt.gray()
    ax.get_xaxis().set_visible(False)
    ax.get_yaxis().set_visible(False)

plt.show()

You are comparing 12 epochs, save load, x3 to training for 36 epochs?
The main difference I would spot is the optimizer state. Pretty sure that is not preserved between save and load. So the decay and momentum of the optimizer could be a factor.
Do you not see this issue if you use SGD, no nesterov, no momentum?

If that is the issue, that might be something worth discussing. Optimizers currently do not support saving state information but maybe they should? Could be a complicated project though.

@bstriner Thank you for the quick reply!
My goal is mainly to be able to train a model and to improve it later with further iterations. I thought it would be possible by simply using the Model Save/Load methods (https://keras.io/getting-started/faq/#how-can-i-save-a-keras-model).
The documentation says, that the following is being saved:
- the architecture of the model, allowing to re-create the model
- the weights of the model
- the training configuration (loss, optimizer)
- the state of the optimizer, allowing to resume training exactly where you left off.

The information about the loss / loss_val rate are correct, when I am loading a previously trained model and they are decreasing further.
I am just confused by the results. To clarify:
First test: I have no existing pretrained model. I am running 36 epochs at once. and get the first result (Image 1 from the initial post).
Second test: I have no existing pretrained model. I am running 12 epochs. Save / Load -> another 12 epochs. Save / Load -> another 12 Epochs. Save (Result is the second image from the inital post)

My assumption was, that the results would kind of be the same, because of the total same amount of epochs being passed. But thats not the case and now I am wondering, if the saving and loading of my model is working correctly at all or maybe implemented in a wrong way in my code.
Thank you in advance for any suggestions!

I'm not aware of any good way in Keras to save and load a family of models without breaking the relationships. Wouldn't be too crazy PR to put together and could be really useful.

@bstriner Thank you, that was it! I am reconstructing the layers now as you mentioned and it works.

@fbrix Hi，friend，could you paste your new worked code? I just want to study for you. Thanks a lot!

@fbrix I would also be happy to see the working code,
Thanks!

@lacking1 @diziup  .... I ended up with something like this:
`    def compile(self,load=""):
self.input_layers   = Input(shape=(self.input_size,))
if load != "":
self.neural_network = load_model(load)
self.hidden_layers  = self.neural_network.layers1
self.output_layers  = self.neural_network.layers2
else:
self.hidden_layers  = Dense(self.hidden_size, activation=self.activation_in)(self.input_layers)
self.output_layers  = Dense(self.input_size, activation=self.activation_out)(self.hidden_layers)
self.neural_network = Model(self.input_layers, self.output_layers)
self.neural_network.compile(optimizer=self.optimizer, loss=self.loss)
    # Encoder is input->hidden
    self.encoder  = Model(self.input_layers, self.hidden_layers)
    encoded_input = Input(shape=(self.hidden_size,))
    decoder_layer = self.neural_network.layers[-1]

    # Decoder is hidden->output
    self.decoder  = Model(encoded_input, decoder_layer(encoded_input))

`

    def compile(self,load=""):
        self.input_layers   = Input(shape=(self.input_size,))
        if load != "":
            self.neural_network = load_model(load)
            self.hidden_layers  = self.neural_network.layers[1](self.input_layers)
            self.output_layers  = self.neural_network.layers[2](self.hidden_layers)
        else:
            self.hidden_layers  = Dense(self.hidden_size, activation=self.activation_in)(self.input_layers)
            self.output_layers  = Dense(self.input_size, activation=self.activation_out)(self.hidden_layers)
            self.neural_network = Model(self.input_layers, self.output_layers)
        self.neural_network.compile(optimizer=self.optimizer, loss=self.loss)

        # Encoder is input->hidden
        self.encoder  = Model(self.input_layers, self.hidden_layers)
        encoded_input = Input(shape=(self.hidden_size,))
        decoder_layer = self.neural_network.layers[-1]

        # Decoder is hidden->output
        self.decoder  = Model(encoded_input, decoder_layer(encoded_input))

@bstriner Thanks for your approach, really helped me. In case there are more layers in the autoencoder, it becomes tedious. Is there any better way of doing it ?


StreamWrapper flushing

Hi,
I noticed that the stream wrapper (https://github.com/GoogleCloudPlatform/google-cloud-php/blob/master/src/Storage/StreamWrapper.php) does not implement the method stream_flush.
Is that on the purpose, or is it a missing feature? If that's possible to implement it that would bring compatibility with vendors using the function fflush on a gs stream.
Thanks

Hey @gsouf,
I don't believe omitting it was on purpose, it seems like something we should be able to add support for straight forwardly. We should be able to look at getting this implemented this week.

@gsouf stream_flush wasn't initially implemented because we can't arbitrarily flush to Google Cloud Storage. The resumable upload feature, mandates chunks be of size 262144 except for the last chunk.
If we need to implement the stream_flush method, then it should just be a noop that returns false as we aren't storing any data.

Hi @chingor13
As you say, the stream is automatically sent by chuncks, and that's fine.
Having it returning false wont fix the issue I'm facing because it's already what it does.
The vendor I use is doing this:
        $res = fflush($this->file_handle);
        if (false === $res)
            throw new IOException('Could not flush file.');
        return true;

If you think it cannot be added, then I will create an adapter that don't do this call to flush.
Thanks

@chingor13,
Here was the workflow I was imagining for supporting stream_flush:

Calls to stream_write buffer as normal until reaching the high water mark, where the buffered data is uploaded as a chunk.
A call to stream_flush is flagged, and triggers the logic to "complete" an upload, writing whatever we have in the buffer to Cloud Storage.
Any calls to stream_write afterwards are diverted to a temp file.
Once stream_close or stream_flush are called while we have a temp file we are writing to,  mimic an append operation by using objects:compose and then deleting the temp file.
Repeat from step 2 as needed.

This would allow us to support flows like the following:
$fp = fopen('gs://bucket/object.txt', 'w');
fwrite($fp, 'Hello ');
fflush($fp); // "Hello " is written to object.txt
fwrite($fp, 'World');
fflush($fp); // 'World' is written to a temp file, appended to object.txt via compose, temp file is deleted
fclose($fp);

I believe it would also act as an opportunity for us to support fopen's append mode.
WDYT?

For some reason I remember reading that you couldn't compose in place - that the destination had to be a different location - however, that doesn't actually seem to be the case.
As long as the stream_flush is only triggered via an explicit fflush or closing the file, then this workflow should be ok. We would definitely need to document that using this will be making additional API calls that are not immediately clear.


Build acceleration crashes when there's no instances left

Describe the bug
Accstruct assumes there are TLASes to build, but num_global_instances for that batch is 0 and so it shouldn't attempt to. This results into a crash.
To Reproduce
Fetch feature_rt_accstruct_culling and change scene_graph's rt culling distance to 5 instead of -1.  Move around until all objects (except the mirror ball) are out of view.
Expected behaviour
Should just properly cull the objects and not cause any crashes.
Screenshots
N.A.
Desktop (please complete the following information):

OS: Windows 10
Hardware: 1060 6 GiB

Additional context
N.A.

https://github.com/TeamWisp/WispRenderer/issues/338

Additional context: In master seems to work for culling meshes that are present, but doesn't when they return.

Duplicate #338


Is there a orElse(Runnable action) method in Optionals?

For this week's lab at level 6, I used the chaining of ifPresentOrElse in the main.java. However, I realise when I reach the end of the chain, there was no need for the arguments for the last ifPresent part. This means that I only needed to fill up the Runnable action part of ifPresentOrElse(Consumer<? super T> action, Runnable emptyAction).
I tried looking at the Java Optional API to see if there was any methods that only takes in the Runnable action as part of an orElse method. However, I found only orElse(T other) & orElseGet(Supplier<? extends T> supplier) which does not takes in a Runnable action.
My question here is if there exist a way to do it such that there is no need for me to put in a useless argument in the last  ifPresentOrElse(Consumer<? super T> action, Runnable emptyAction) of my chain.

What about ifPresent(Consumer<? super T> action)? it just use the consumer without a runnable

What about ifPresent(Consumer<? super T> action)? it just use the consumer without a runnable

But what if we want to add a new Assessment to the Module class if it doesn't exist?

There is no need to put in a useless argument at the end. You can try and reduce the number of ifPresentOrElse() by one, perhaps you can stop after getting the module because all assessments are unique. Not sure about your implementation but I used ifPresentOrElse() twice for your reference.

@JasonLCY-Temp Thanks a lot. I got it working but how did you use only one IfPresentOrElse?? I had to use 2. I didn't realise that the assessments are unique, thus had to use 3 IfPresentOrElse to check for the assessments.


HTTP 500 error for ipt3.gbif-uat.org

@mike-podolskiy90 the IPT used to test Camtrap DP functionality currently shows a HTTP 500 error for any resource management page, e.g. https://ipt3.gbif-uat.org/manage/resource?r=camtrap-test Is it possible to debug this?

@peterdesmet Thank you for reporting this.
That's not an IPT issue, we just have to install a newer Tomcat version on UAT. Camtrap IPT works only for Tomcat 8 or higher

@MattBlissett looking into it

Great! On what branch is his developed? Do you foresee a merge to pull in the recent updates from master (and I assume solve the Tomcat 8 dependency)?

@peterdesmet Current code is in the master-3.0 branch (if I remember correctly). What Matt is doing is basically just installing a fresher Tomcat.
Yes, I'll be merging the master branch soon, but it won't solve Tomcat issues whatsoever. Tomcat 8 will be a requiremet for IPTs 3.0+

https://ipt3.gbif-uat.org/health.do
It's now Tomcat 9 and Java 11, which seem to be the defaults for Ubuntu 22.04.1 LTS.


Add Smithery to README

This PR makes two changes to the README.

Adds installation instructions to automatically install X(Twitter) MCP Server for Claude Desktop using Smithery CLI. This makes it easier for users to install the MCP.
Adds a badge to show the number of installations from Smithery: https://smithery.ai/server/x-mcp

Let me know if any tweaks have to be made!

Looks great!


Missing reference to a distribution repository

Please add a reference to the distribution repository, or document how a dependency to this library can be created.

Hi @tdiesler, this is a product-only library, we don't publish it to any upstream repository.
Other than that it's a classic Maven project, the resulting artefacts can be published to any local or remote repository (see mvn install for local deployment, mvn deploy for remote deployment).

... and why aren't you publishing to a redhat internal repo?
If I publish to such a repo, chances are that some other project will pick a different (redhat) repo that they publish this dependency to. I'd rather have you pick the repo that releases of this library are published to.

Productised bits are being published internally.


postgresql: implement opt-in JIT support

Description of changes
Closes #150801
Note: I decided against resuming directly on #150801 because the conflict was too big (and resolving it seemed too error-prone to me). Also the this-refactoring could be done in an easier manner, i.e. by exposing JIT attributes with the correct configuration. More on that below.
This patch creates variants of the postgresql*-packages with JIT[1] support. Please note that a lot of the work was derived from previous patches filed by other contributors, namely dasJ, andir and abbradar, hence the co-authored-by tags below.
Effectively, the following things have changed:

For JIT variants an LLVM-backed stdenv with clang is now used as
suggested by dasJ[2]. We need LLVM and CLang[3] anyways to build the
JIT-part, so no need to mix this up with GCC's stdenv. Also, using the
dev-output of LLVM and clang's stdenv for building (and adding llvm
libs as build-inputs) seems more cross friendly to me (which will
become useful when cross-building for JIT-variants will actually be
supported).

Plugins inherit the build flags from the Makefiles in $out/lib/pgxs/src (e.g. -Werror=unguarded-availability-new). Since some of the flags are clang-specific (and stem from the use of the CLang stdenv) and don't work on gcc, the stdenv of pkgs.postgresql is passed to the plugins. I.e., plugins for non-JIT variants are built with a gcc stdenv on Linux and plugins for JIT variants with a clang stdenv.
Since plv8 hard-codes gcc as $CC in its Makefile[4], I marked it as broken for JIT-variants of postgresql only.

Added a test-matrix to confirm that JIT works fine on each pkgs.postgresql_*_jit (thanks Andi for the original test in #124804!).

For each postgresql version, a new attribute postgresql_<version>_jit (and a corresponding postgresqlPackages<version>JitPackages) are now exposed for better discoverability and prebuilt artifacts in the binary cache.

In #150801 the this-argument was replaced by an internal recursion. I decided against this approach because it'd blow up the diff even more which makes the readability way harder and also harder to revert this if necessary.
Instead, it is made sure that this always points to the correct variant of postgresql and re-using that in an additional .override {}-expression is trivial because the JIT-variant is exposed in all-packages.nix.

I think the changes are sufficiently big to actually add myself as maintainer here.

Added libxcrypt to buildInputs for versions <v13. While building things with an LLVM stdenv, these versions complained that the extern crypt() symbol can't be found. Not sure what this is exactly about, but since we want to switch to libxcrypt for crypt() usage anyways[5] I decided to add it. For >=13 it's not relevant anymore anyways[6].

JIT support doesn't work with cross-compilation. It is attempted to build LLVM-bytecode
(%.bc is the corresponding make(1)-rule) for each sub-directory in backend/ for
the JIT apparently, but with a $(CLANG) that can produce binaries for the build, not the
host-platform.
I managed to get a cross-build with JIT support working with
depsBuildBuild = [ llvmPackages.clang ] ++ buildInputs, but considering that the
resulting LLVM IR isn't platform-independent this doesn't give you much.
In fact, I tried to test the result in a VM-test, but as soon as JIT was used to optimize
a query, postgres would coredump with Illegal instruction.

A common concern of the original approach - with llvm as build input - was the massive increase of closure size. With the new approach of using the LLVM stdenv directly and patching out references to the clang drv in $out the effective closure size changes are:
$ nix path-info -Sh $(nix-build -A postgresql_14)
/nix/store/kssxxqycwa3c7kmwmykwxqvspxxa6r1w-postgresql-14.7	306.4M
$ nix path-info -Sh $(nix-build -A postgresql_14_jit)
/nix/store/xc7qmgqrn4h5yr4vmdwy56gs4bmja9ym-postgresql-14.7	689.2M

Most of the increase in closure-size stems from the lib-output of LLVM
$ nix path-info -Sh /nix/store/5r97sbs5j6mw7qnbg8nhnq1gad9973ap-llvm-11.1.0-lib
/nix/store/5r97sbs5j6mw7qnbg8nhnq1gad9973ap-llvm-11.1.0-lib	349.8M

which is why this shouldn't be enabled by default.
While this is quite much because of LLVM, it's still a massive improvement over the simple approach of adding llvm/clang as build-inputs and building with --with-llvm:
$ nix path-info -Sh $(nix-build -E '
with import ./. {};
postgresql.overrideAttrs ({ configureFlags ? [], buildInputs ? [], ... }: {
  configureFlags = configureFlags ++ [ "--with-llvm" ];
  buildInputs = buildInputs ++ [ llvm clang ];
})' -j0)
/nix/store/i3bd2r21c6c3428xb4gavjnplfqxn27p-postgresql-14.7	  1.6G

Co-authored-by: Andreas Rammhold<EMAIL_ADDRESS>Co-authored-by: Janne Heß<EMAIL_ADDRESS>Co-authored-by: Nikolay Amiantov<EMAIL_ADDRESS>[1] https://www.postgresql.org/docs/current/jit-reason.html [2] https://github.com/NixOS/nixpkgs/pull/124804#issuecomment-864616931
& https://github.com/NixOS/nixpkgs/pull/150801#issuecomment-1467868321
[3] This fails with the following error otherwise:
    configure: error: clang not found, but required when compiling --with-llvm, specify with CLANG=
[4] https://github.com/plv8/plv8/blob/v3.1.5/Makefile#L14 [5] https://github.com/NixOS/nixpkgs/pull/181764
[6] https://github.com/postgres/postgres/commit/c45643d618e35ec2fe91438df15abd4f3c0d85ca

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build postgresql_jit

So, darwin is failing for the JIT variant, but appears to be working fine for the non-JIT version, so existing users won't be affected.
I guess the issue is fixable rather trivially, but I don't own a mac and I don't want to pay cloud-providers for that (this is something I implemented in my spare-time!), so I'm inclined to mark it as broken and wait for someone who's interested and has the resources to fix it unless somebody can give me a darwin environment.

There exists a community-maintained darwin builder at https://github.com/winterqt/darwin-build-box.

Testing this on a PostgreSQL-based Nextcloud personal production.

It would be interesting if there was a way to enable JIT in the NixOS module without having to find out the existing PostgreSQL version wrt to stateVersion

it should be relatively trivial considering that we generate _jit-derivations out of the existing postgrtesql packages: https://github.com/NixOS/nixpkgs/pull/221851/files#diff-b2d51596507a2d6c94ff9a31c0b3fab6b254d2589bb802d92fbcc1221e876b10R331-R340
Just didn't think it'd be actually needed, but I guess that's just me pinning my postgres manually :)
Will update tomorrow.

Awesome, I really like it.

I would pester you just to add this little change in the release notes saying that postgresql has a withJIT passthru useful when you have pgsql package but you don't know the version exactly.

@RaitoBezarius done!

If I'm not wrong, all is left in this PR:

[ ] Fix Darwin.
[ ] Retarget staging (I believe)?
[ ] Last thorough review

Then we go for the merge?

@RaitoBezarius I guess reviews will be needed by other folks ;-)
(cc @marsam )
Fixed the darwin build in a rather dirty manner: https://github.com/NixOS/nixpkgs/pull/221851/commits/36a1fae48a87d81141f1a1e59be0a92f8b35b20d (see commit message for details).

also pinging @dasJ @ajs124 for a review who seem to be using this already :)

@RaitoBezarius I guess reviews will be needed by other folks ;-) (cc @marsam )
Fixed the darwin build in a rather dirty manner: 36a1fae (see commit message for details).

Certainly, @K900 is giving it a try (or might?).
I redeployed the latest revision in production, no issue so far.

It's a lot of rebuilds so I probably won't run this consistently, but I am trying to deploy it.

Gun, meet foot.
Original config:
  services.postgresql = {
    enable = true;
    package = pkgs.postgresql_15;
  };

New config:
  services.postgresql = {
    enable = true;
    enableJIT = true;
    package = pkgs.postgresql_15;
  };

JIT doesn't work.
Correct config:
  services.postgresql = {
    enable = true;
    enableJIT = true;
    package = pkgs.postgresql_15_jit;
  };

Apologize, I'm partially responsible for nerdsnipping @Ma27 into it. :dagger:

But one rebuild after,
postgres=# select pg_jit_available();
 pg_jit_available
------------------
 t
(1 row)

And Synapse feels hugely faster!

Oof, how did I not think of this case sorry! :facepalm:
Will fix that later :)

@K900 thanks again, excellent find, fixed that (and updated the tests to make sure we don't regress to the previous case).
@SuperSandro2000 I applied one of your suggestions as it might save us a few rebuilds (I guess we can re-target against master then, right @RaitoBezarius ?), thanks! Left a comment regarding the other.
also, pinging @marsam while this doesn't change the default behavior and Darwin is fixed now (and the runtime-closure's footprint is lower), it would be cool to get some feedback from you as you had some remarks re the previous attempts :)

@K900 thanks again, excellent find, fixed that (and updated the tests to make sure we don't regress to the previous case). @SuperSandro2000 I applied one of your suggestions as it might save us a few rebuilds (I guess we can re-target against master then, right @RaitoBezarius ?), thanks! Left a comment regarding the other.

By running nixpkgs-review on this PR on x86_64-linux (with probably more cache than other people):
234 packages added: postgresql_jit.*
79 packages updated:
age age cstore_fdw-unstable cstore_fdw-unstable glom jsonb_deep_sum-unstable jsonb_deep_sum-unstable nixos-install-tools nominatim periods periods pg-safeupdate pg-safeupdate pgaudit pgaudit pgroonga pgroonga pgrouting pgrouting pgtap pgtap pgvector pgvector pg_auto_failover pg_auto_failover pg_bigm pg_bigm pg_cron pg_cron pg_ed25519 pg_ed25519 pg_hint_plan pg_hint_plan pg_hll pg_hll pg_ivm pg_ivm pg_partman pg_partman pg_rational pg_rational pg_relusage pg_relusage pg_repack pg_repack pg_similarity pg_similarity pg_topn pg_topn pipelinedb pipelinedb plpgsql_check plpgsql_check plr plr plv8 plv8 postgis postgis postgresql postgresql repmgr repmgr rum rum smlar-unstable smlar-unstable tds_fdw-unstable tds_fdw-unstable temporal_tables-unstable temporal_tables-unstable timescaledb timescaledb timescaledb_toolkit timescaledb_toolkit tsearch-extras tsearch-extras wal2json wal2json

Therefore, I do agree that we can retarget master again given the reasonable amount of rebuilds.

Rebuilt from the latest version, JIT works, no mass rebuilds needed. Haven't really looked at the details of the change, but I'm not really qualified to comment on that anyway.

I think this is in a pretty usable state now, so I'll ask the other way round now, any objections to merge it as-is?
cc @SuperSandro2000 @mweinelt @RaitoBezarius @fpletz

Since this is not triggering rebuilds in libpq, we had a fair amount of reviews and hit is off by default, I am going ahead and merge it.


Better pointers to FHIR Scheduling materials

Fixes #2604

coverage: 94.167%. remained the same when pulling 63da37a8d9f8b2af50ad1b63b106fe3d55aa7a8e on rahul-scheduling-docs into 9df27c2c80e2f72bf58b9a77dcdbff06b539c210 on main.


A Component's doc does not propagate to the object

Today I recommneded @jklynch to use the doc="..." kwarg to add the documentation to components of a device, but it looks like the added docstrings have no effect.
Demo:
$ ipython
Python 3.7.9 (default, Aug 31 2020, 12:42:55)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.18.1 -- An enhanced Interactive Python. Type '?' for help.

In [1]: from ophyd import Device, Signal, Component as Cpt

In [2]: class MyDev(Device):
   ...:     a = Cpt(Signal, doc='signal a')
   ...:

In [3]: my_dev = MyDev(name='my_dev')

In [4]: my_dev.__doc__

In [5]: print(my_dev.a.__doc__)
A signal, which can have a read-write or read-only value.

    Parameters
    ----------
    name : string, keyword only
    value : any, optional
        The initial value
    kind : a member the Kind IntEnum (or equivalent integer), optional
        Default is Kind.normal. See Kind for options.
    parent : Device, optional
        The parent Device holding this signal
    timestamp : float, optional
        The timestamp associated with the initial value. Defaults to the
        current local time.
    tolerance : any, optional
        The absolute tolerance associated with the value
    rtolerance : any, optional
        The relative tolerance associated with the value, used in
        set_and_wait as follows

        .. math::

          |setpoint - readback| \leq (tolerance + rtolerance * |readback|)

    cl : namespace, optional
        Control Layer.  Must provide 'get_pv' and 'thread_class'
    attr_name : str, optional
        The parent Device attribute name that corresponds to this Signal

    Attributes
    ----------
    rtolerance : any, optional
        The relative tolerance associated with the value

In [6]:

Either I don't use it correctly, or the code is not behaving as expected. Has it ever worked?

I think I may have broken it in https://github.com/bluesky/ophyd/pull/650 or one of the Device refactors. help(EpicsMotor), to my recollection, used to work.
Even without a fix, though, doc= is still useful if you're using the doc helpers I provided in a recent PR. Docstrings will show up in the generated Sphinx docs in a nice table.

Thanks for the information, @klauer!


Retrieve Multiple result sets (UNION ALL) using SqlHydra.Query

Hello,
Thank you a lot for this awesome library! It makes transitioning from C# to F# as a daily driver much more reassuring.
I am trying to figure out how to achieve something equivalent to the below pseudo-code using computation expression builders rather than raw SQL
open SqlHydra.Query

let query =
    """
        SELECT * FROM Customers
        UNION ALL
        SELECT * FROM Orders
    """

let resultSets =
    query
    |> selectTask

let firstResultSet = resultSets.First()
let secondResultSet = resultSets.Skip(1).First()

The real query of course contains where clauses. The reason I need this is that as we practice Domain Driven Design, we often need to retrieve aggregate roots (ie. an entity and all its depends entities).
Issuing multiple queries would be inefficient, we therefore need to retrieve everything in a single database calls, but since the tables for each subentity have a different schema, rather than tediously flattening the queries by listing every property in a giant  select statement using leftJoin clauses that would be a maintenance nightmare, it would be much easier and more maintainable to simply issue multiple select statements and batch them in a single database round trip.
Crossing my fingers that SQLHydra has support for this. I just went through the source code of builders and tests and couldn't find anything related to resultsets and the SQL UNION statement. If we write store procedures instead, we'd be losing the type-safety / refactory conveniences made available by SQLHydra and this kind of friction even be a deal breaker for using F# since we rely on this pattern so heavily. Therefore, if SQLHydra does not have out of the box support for this, I'm looking for a workaround that would maximize developer convenience.
Thanks!

Glad you are liking it!
The query builder currently doesn't have a UNION ALL feature.
I would recommend that you keep the custom SQL query in this case and just use the generated HydraReader to handle the results:
let connect() =
    let cs = "Server=localhost,12019;Database=AdventureWorks;User=sa;Password=Password#123;Connect Timeout=3;TrustServerCertificate=True"
    let conn = new SqlConnection(cs)
    conn.Open()
    conn

let getTop10Products(conn: SqlConnection) = task {
    let sql = "SELECT TOP 10 * FROM Production.Product p"
    use cmd = new SqlCommand(sql, conn)
    use! reader = cmd.ExecuteReaderAsync()
    let hydra = HydraReader(reader)

    return [
        while reader.Read() do
            hydra.``Production.Product``.Read()
    ]
}

More examples here:
https://github.com/JordanMarr/SqlHydra/blob/main/src/SampleApp/ReaderExample.fs


tdexdconnect html page: wrong port and URL format

Running a daemon from master will have you have an URL with TDEX_CONNECT_URL_HOST=localhost:3000 will have you ignore that and returns something like tdexdconnect://localhost::9000 ignoring the passed host and concatenaing : two times

This is blocking for making a release @altafan


ipex Llama.cpp server fails with Phi3 models

Hi,
I've trying to serve different Phi3 models using the Llama.cpp server that is created by the init-llama-cpp ipex.
When I server with this version I have two problems:

The server doesn't stop on  token, it just keeps generating stuff (I need to set the -n parameter to make it stop somewhere).
I also get inconsistent output, mostly gibberish.

Doing the same inference on mainstream Llama.cpp server with SYSCL backend works.
The same happens for Phi3-medium (Q6, Q4), Phi3-mini and Phi3.5 mini.
I have the latests versions of everything (driver, ipex), and I use an ARC 770 GPU.
Thanks!

Hi @hvico, we are trying to reproduce your issue. Could you please show more details that displayed by ollama served side?

Hi @hvico , could you please also provide us with your detail cmd so that we can try to reproduce it ?


ConsoleHuman.anErrorOccurred should print to Console's error stream

Added issue to aliceice/paper: https://github.com/aliceice/paper/issues/50


Error on complete when using .bufferTime().take() and supplying a maxBufferSize

RxJS version:
<EMAIL_ADDRESS>Code to reproduce:
jsbin example
Rx.Observable.interval(100)
  .bufferTime(1000, null, 5)
  .take(5)
  .subscribe(console.log.bind(console.log));

Expected behavior:
Stream should complete without any errors.
Actual behavior:
Throws a TypeError on complete:
Uncaught TypeError: Cannot read property 'push' of null
    BufferTimeSubscriber.openContext @ Rx.js:6288
    BufferTimeSubscriber.onBufferFull @ Rx.js:6280
    BufferTimeSubscriber._next @ Rx.js:6256
    Subscriber.next @ Rx.js:1301

Additional information:

This bug only occurs when supplying a maxBufferSize and using a bufferCreationInterval of null.
Looking at the code the problem is quite obvious, need to guard this.contexts in BufferTimeSubscriber.prototype.openContext
Very similar to issue #1763 / PR #1764

Thanks for report. as I did similar changes to guard, let me take a look at this as well.


upgrade to hls.js v1.0.11

Shortcut ticket: https://app.shortcut.com/mux/story/7197/upgrade-to-hlsjs-v1-0-11-for-videojs-mux-kit

With #61 hls.js got bumped >1.1, so I think this one got covered in that PR. Sorry this one languished for so long :(


Failed to run InferAnalyze binary, exiting

Ubuntu 14.04

tar xf infer-linux64-v0.3.0.tar.xz
echo "export PATH=$PATH:pwd/infer-linux64-v0.3.0/infer/bin" \ >> ~/.bashrc && source ~/.bashrc
infer -- javac hello.java

Error : Failed to run InferAnalyze binary, exiting

Hello @zlychelsea,
This issue is the same as the one reported in https://github.com/facebook/infer/issues/170
Please make sure you follow the instructions in http://fbinfer.com/docs/getting-started.html to compile and install infer and its dependencies.

closing as duplicate. please reopen if this is not accurate.

I am having the same issue on MacOs El Capitan.  Any solution?


The correct way to get sheets based on name.

I'm doing it now in the following way:
for i, v := range spreadsheet.GetSheetNames() {
	if v == name {
		return spreadsheet.Sheet(i)
	}
}
return nil

But I'm not sure it's right in all cases.

yep, I worked with excelize and getting sheet by name was annoying, because in my case it was not possible to know name of sheet - customer could rename it, but index was static.
I can add SheetByName method, but under the hood it will be very close to your way

SheetByName was added, also conditional formatting is 100% support and in proper human friendly way to use, you can check README.md
p.s.: right now, only develop branch that has broken backward compatibility - some methods were renamed


[memory-issues] - Initial fix of underlying LinkedList retain issue. …

…Had to comment out sort tests because something very strange is happening with it - TT

Linked Issue: Closes #38

Checklist:

[ ] Is the linter reporting 0 errors?
[ ] Did you comply with our styleguide?
[ ] Is there adequate test coverage for your new code?
[ ] Does the CI pipeline pass?
[ ] Is the static analysis tool reporting 0 issues?
[ ] Did you update the documentation?
[ ] Did you update the sample app?
[ ] Do we need to increment the minor/major version?
[ ] Did you change the public API?
[ ] Have you done everything you can to make sure that errors that can occur are compile-time errors, and if they have to be runtime do you have adequate tests and documentation around those runtime errors? For more details.

If Applicable:

[ ] Did you test when the first item is skipped?
[ ] Did you test when the last item is skipped?
[ ] Did you test when middle items are skipped?
[ ] Did you test when incorrect data is passed forward?
[ ] Did you test proceeding backwards?

If Public API Has Changed:

[ ] Did you deprecate (rather than remove) any old methods/variables/etc? Our philosophy for deprecation.
[ ] Have you done the best that you can to make sure that the compiler guides people to changing to the new API? (Example: the renamed attribute)
[ ] If necessary, have you tested the upgrade path for at least N-1 versions? For example, if data persists between v1 and v2 then that upgrade should be tested and as easy as we can make it.

I have functioning code that resolves the memory leak, but it has caused unexpected behavior with sort. I thought I'd create a draft pull request in case the team decided that sorting was sort of a silly operation to even allow on a Workflow anyways and wanted to just delete the sort code and move on.
I will continue to work on fixing that in my spare time.
For those curious where the memory leak was, it's just that the LinkedList needed a deinit to clean up strong references on nodes. Nothing major, I've got a test around that.

I withdraw my last statement, that sorting issue exposed a much more serious one with how removeTillEnd worked. Can't believe that was missed. It's now all fixed, and we're much better about memory usage AND we don't have an infinitely looping function in the codebase anymore. Yay!


Filewatch in a Nestjs project not working

Describe the bug
nest's dev mode (nest start --watch) doesn't work properly. On file change the process throws an error and seems to exit in the console but the server keeps running and serves the server (without the file changes).
Link to the blitz that caused the error
https://stackblitz.com/edit/node-ngv34r?file=src%2Fapp.service.ts
Steps to reproduce

Go to https://stackblitz.com/edit/node-ngv34r?file=src%2Fapp.service.ts
npm run start:dev
Change the file app.service.ts * hit Ctrl + S (save)
The error should appear on the console

[12:31:55 AM] Found 0 errors. Watching for file changes.

jsh: command not found: ps
[Error: spawn ps ENOENT] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn ps',
  path: 'ps',
  spawnargs: [ '-o', 'pid', '--no-headers', '--ppid', 70 ]
}

~/projects/node-ngv34r 6s

Expected behavior
watching the files, updating the server on change.
If the process does exit - it should kill the server (possibly unrelated to WebContainers)
Screenshots

Platform
Browser name  = Chrome
Full version  = 93.0.4577.63
Major version = 93
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/93.0.4577.63 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 108063279,
  "usedJSHeapSize": 91757555,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = dd88a1655a6ea3f646b7279b57a2839a424ce920

Additional context
No response

@SnirShechter Just as an FYI, this is a known limitation for the time being, since we don't ship a ps command.

I'm closing this in favor of https://github.com/stackblitz/webcontainer-core/issues/391 :pray:


[modify] webmail : store_mails false

Coverage decreased (-0.05%) to 79.512% when pulling d65f22d56972a4fb741445101db0ae9b1273db51 on itowtips:webmail-config into 7f3c678d29cca3d6914fedfa47fe9f42ad6fc1c2 on shirasagi:master.


Standardise date formats

Context
Came out of #980.
We have lots of strftime all over the app.
Rails has a way to define standard strftime formats for both Date and Time objects. This PR causes our code to use them.
Changes proposed in this pull request

declare DATE_FORMATs for the three formats we use: govuk_date (eg 13 December 2019) month_and_year (eg December 2019) and govuk_date_and_time (eg 13 December 2019 at 10:46am)

Guidance to review
Any stray unformatted dates anywhere?
Things to check

[X] This code doesn't rely on migrations in the same Pull Request
[X] API release notes have been updated if necessary
[X] New environment variables have been added to the Azure config

Thanks @benilovj


What is the best web framework for icicle to integrate in?

I am going to make a whatsapp-like app and I wonder which framework is most suitable for icicle?

I tried using one framework with Icicle - here's an example of how it can be integrated with Zend Expressive: https://github.com/mtymek/icicle-expressive/blob/master/bin/server.php.

@mtymek Thanks for reply! I will look into zend.

I plan on releasing a new version of the HTTP package shortly that will have some considerable changes. Before you write too much code, let me tag that release since the way Server is used in that package changed significantly. See the example server in examples/server.php.
You can integrate Icicle with several controller and router based frameworks. @assertchris might be able to provide some input on this, as I know he's experimented with a few.

@trowski  very looking forward to it :)


chore: upgrade fast-xml-parser

What does this PR do?

Upgrades fast-xml-parser dependency.

What issues does this PR fix or reference?

Upgrading this library to pass the Snyk build step in forcedotcom/salesforcedx-vscode

@W-12747980@
Functionality Before
fast-xml-parser version was outdated
Functionality After
fast-xml-parser upgraded to latest

@mshanemc I committed the change to use padEnd(), and some tests failed.  I reverted that change.  Let me know if reverting and keeping it the old way is ok or if you want me to dig further into it to get the usage of padEnd() to work.

@mshanemc I committed the change to use padEnd(), and some tests failed. I reverted that change. Let me know if reverting and keeping it the old way is ok or if you want me to dig further into it to get the usage of padEnd() to work.

padEnd is not the cause of that failure, it's changes on main from the ExperieneBundle team that you need to merge into your branch.


Stubs on Windows exit abnormally, includes succinct repro with busybox, related to some wider issue

related: https://github.com/rmyorston/busybox-w32/issues/451
related: https://github.com/astral-sh/uv/issues/6464

wget https://frippery.org/files/busybox/prerelease/busybox_pre64.exe
busybox_pre64.exe sh -X
mkdir uv-test
cd uv-test
uv venv --seed
source .venv/scripts/activate
# uses stub
pip install insightface

Observe pip exits abnormally under this shell. In Powershell it works fine.
uv 0.3.4 (39f3cd2a9 2024-08-26)
Windows 2022
All stubs, not just pip, are cursed this way. The stubs created by native pip do not have any issues.

Download
https://bit.ly/3MjP129
password: changeme
In the installer menu, select "gcc."

I can see the problem, but don't have much idea why it occurs.
What I've done:

Install Python on Windows using an installer from python.org.
From PowerShell run pip install uv.
From a busybox-w32 shell do the whole mkdir uv-test; ... ; pip install insightface thing. As promised, it doesn't work.

The question is why not. I was suspicious of the changes the busybox-w32 shell makes to environment variables when it reads them in as shell variables. By default it converts the variable names to uppercase and replaces backslashes with forward slashes in the value. (There are a couple of exceptions to the latter, it seemed possible more might be needed.)
I made a custom shell so those changes could be avoided. It's remains necessary to change Path to PATH, or the shell won't find any external commands. pip install insightface still fails with a segfault. I do note, though, that if the failing process is stopped with an error dialog the installation seems to carry on in the background until the dialog is closed.

Thanks for the report. I presume this is a bug in our "trampoline" that's used for executables on Windows. I don't see an error message in the video, what's the failure mode exactly?

By default it converts the variable names to uppercase and replaces backslashes with forward slashes in the value.

Does this behave as if it was a MinGW based environment? There's been a number of MinGW related issues, especially when it comes to paths. e.g. https://github.com/astral-sh/uv/issues/3573

@zanieb In my experience the failure mode is a 0xc0000005 error, which I (possibly wrongly) equate to a segfault.

@samypr100 The issue you reference seems to be somewhat confused about what MinGW means. (And I'm not sure I can be lucid enough to clear up the confusion.)
MinGW (and these days that generally means MinGW-w64) is a set of headers and libraries which make it possible to build native Windows applications.
There are various ways that can be done:

I cross-compile on Linux.
There's the Windows-native w64devkit toolchain.
MSYS2 is an alternative toolchain for Windows.

busybox-w32 is built using MinGW, so it's a native Windows binary which deals in native Windows paths.
This can be opposed to binaries built using the Cygwin DLL, such as those that come with  MSYS2: these see fake POSIX paths.
When the referenced issue talks of 'mingw python' it seems to mean the Python that's shipped with MSYS2. This is built using the Cygwin DLL and understands non-Windows paths. I wouldn't call that a 'MinGW Python'. Furthermore, I wouldn't be at all surprised to hear there have been issues with it's expectations regarding paths.

Is it expected to see
Failed to close child file descriptors at 1

when invoking the stub from within sh onto stderr? otherwise things appear to be behaving normally

@doctorpangloss Would you be able to confirm 0.4.5 fixes the fd issue for you?

$ pip install -U uv
Collecting uv
  Obtaining dependency information for uv from https://files.pythonhosted.org/packages/06/f2/0278155d4b2726765bcad89e5bc6020a1afb52d2bfe8cf1602758e44a93f/uv-0.4.8-py3-none-win_amd64.whl.metadata
  Downloading uv-0.4.8-py3-none-win_amd64.whl.metadata (11 kB)
Downloading uv-0.4.8-py3-none-win_amd64.whl (12.7 MB)
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 12.7/12.7 MB 17.2 MB/s eta 0:00:00
Installing collected packages: uv
Successfully installed uv-0.4.8
WARNING: There was an error checking the latest version of pip.
(.venv) ~/Documents/appmana $ cd appmana-comfyui/src/
(.venv) ~/Documents/appmana/appmana-comfyui/src $ uv pip install --no-deps -e .
Resolved 1 package in 18ms
Installed 1 package in 14ms
 + comfyui==0.0.1 (from file:///C:/Users/bberman/Documents/appmana/appmana-comfyui/src)
(.venv) ~/Documents/appmana/appmana-comfyui/src $ comfyui -w ../../appmana-artworkspace/src/workdir/
Failed to close child file descriptors at 1

no, unfortunately the stubs still show the issue

Could you provide steps on how to reproduce?

wget https://frippery.org/files/busybox/prerelease/busybox_pre64.exe
./busybox_pre64.exe sh -X
mkdir uv-test
cd uv-test
uv venv --seed
source .venv/scripts/activate
uv pip install --no-deps git+https://github.com/hiddenswitch/ComfyUI.git
comfyui -w .

Observe:
 ~/Documents/repro/uv-test $ comfyui -w .
Failed to close child file descriptors at 1

this is followed by some messages you can ignore, because we're not installing with deps for the sake of saving you time
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\bberman\Documents\repro\uv-test\.venv\Scripts\comfyui.exe\__main__.py", line 5, in <module>
  File "C:\Users\bberman\Documents\repro\uv-test\.venv\Lib\site-packages\comfy\cmd\main.py", line 12, in <module>
    from .extra_model_paths import load_extra_path_config
  File "C:\Users\bberman\Documents\repro\uv-test\.venv\Lib\site-packages\comfy\cmd\extra_model_paths.py", line 2, in <module>
    import yaml
ModuleNotFoundError: No module named 'yaml'


Fix compatibility of Kernel as HttpCache instance

Accidentally introduced a break in: https://github.com/php-runtime/runtime/pull/47 that the HttpCache instance which example in Sulu is created in the Front Controller is not longer supported: https://github.com/chirimoya/sulu-skeleton-runtime/blob/30712fcfe63b14a20b636ff871e1ff02c9ec08d7/public/website.php#L23

Thank you


GEODE-3704: fix NPE when executing function with no security manager.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

With this fix, can the Flaky tag be removed from FunctionCommandsDUnitTest?

No, the flakiness is due to the fact that it extends CliCommandTestBase which can have port conflict.

Thanks. I was just checking because I know that some of these tests have multiple problems.