[RC2]  frequently closes by itself

Short description of the problem:
I have an <ion-menu> in app.html that, when opened, frequently closes by itself almost instantly (note: it doesn't happen all the time, but at the very least about ~50% of the time). This was previously working as expected before the RC2 update.
What behavior are you expecting?
I'm expecting my side menu to close only when I tap on one of the links in the side menu, or when clicking on the background overlay.
Steps to reproduce:

Create an <ion-menu> overlay in an app.html file
Have a button that toggles the ion menu open.
Watch it close by itself sometimes?

This is my app.html:
<ion-menu [content]="content" type="overlay" class="sidebar">
  <ion-content class="sidebar-content">
    <img class="sidebar-logo" src="assets/img/general/white-logo.svg">
    <ion-list class="sidebar-list">
      <button ion-item menuClose *ngFor="let link of links" (click)="goToPage(link.pageObj)" class="sidebar-item">
        {{ link.title }}
      </button>
      <button ion-item menuClose (click)="logout()" class="sidebar-item">
        Log Out
      </button>
    </ion-list>
  </ion-content>
</ion-menu>

<ion-nav [root]="rootPage" #content swipeBackEnabled="false"></ion-nav>

This is my dashboard.html page that contains the menu icon that opens the Menu:
<ion-header class="header">
  <ion-navbar hideBackButton="true">
    <ion-title>{{ pageTitle }}</ion-title>
    <button ion-button icon-only menuToggle right>
      <ion-icon name="menu"></ion-icon>
    </button>
  </ion-navbar>
</ion-header>

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
This seems to have happened with one of the code changes after RC1 was released, because this was working when I was on RC1.
Let me know if there's anything else I can share that may help with this issue! It might sound weird but I didn't make any coding changes for this to happen...
Which Ionic Version?
2.0.0-rc.2
Run ionic info from terminal/cmd prompt:
Cordova CLI: 6.1.1
Gulp version:  CLI version 3.9.1
Gulp local:
Ionic Framework Version: 2.0.0-rc.2
Ionic CLI Version: 2.1.0
Ionic App Lib Version: 2.1.0-beta.1
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v6.3.1
Xcode version: Xcode 7.3.1 Build version 7D1014

Hi guys,
An update. This only happens with an type="overlay" menu, I've tried reveal and push and they work as expected. So that helps narrow it down!

I can confirm that this happens every time in iOS no matter which type I use. The first time the menu is opened, it stays open. Subsequent attempts at opening automatically close the menu. Downgraded to rc1 and now it functions as expected.

@panduit-psh Interesting, I wish I could know why a different type works for me so that you wouldn't have to downgrade but I'm not sure :(

same issue here, thanks for reporting it so I didn't have to :+1:

@manucorporat #6405

This is still occurring in rc3.

@panduit-psh I might need to downgrade to RC1 since we are about to launch our iOS app. Is there a guide on how to do this?

Hello all, thanks for opening an issue with us! Would you mind creating a plunker using this template? Thanks!

@jgw96 Here you go. Let me know if I posted it incorrectly. Thanks for checking it out.

@panduit-psh Thanks for submitting the plunkr.  Not sure about @panduit-psh or @jourdan-jobox but for me, the issue does not occur while testing in the browser.  It only occurs when testing on a live device whether I build the app or use Ionic View or Test Flight.
Can you two confirm that is the case for you as well?

@tinnocente This only happens in iOS for me.

Can you guys try against nightly?
npm install ionic-angular@nightly --save

@manucorporat I can confirm that the issue is fixed for me in 2.0.0-rc.3-201611282011. Thanks for the help.

@manucorporat Having the same issue here. Using the nightly 2.0.0-rc.3-201611282011 resolved this for me as well.

@lincolnthree @panduit-psh glad to hear that! closing this issue...
Please feel free to reopen if you think this is still not fixed in nightly

type="push" is buggy but type="overlay" works very nicely.  Thanks I can launch now :)

@tinnocente can you describe how the type=push is buggy?

Out of curiosity. What was the issue / fix?

And thank you :)

@manucorporat When I first upgraded to nightly I had it set to type="push". After building it onto my test iPhone the menu was no longer auto-closing however after the menu opened it was like there was a timeout where I could not close the menu for several seconds by tapping outside the menu. If I waited a couple of seconds then I could tap outside the menu to close it. If I tapped my "Close" button the menu would close as expected.
I have not experienced any bad behavior from "Overlay" so at least I can launch the iOS app now 👍
Hope that helps!

Actually yes. I am seeing the same behavior. It takes several seconds before the menu will close via clicking outside. Menu toggle buttons work fine.


Put/Delete requests failing on oauth Filter

Hi,
I am using the 5.1.x version and whenver I send a PUT or DELETE request to a route behind the oauth fitler it fails with the "invalid_request" error. May I know how to solve this?
For GET and POST Method is working fine, Only having problem on PUT and DELETE.
Please Advise me to solve this problem.

Thanks for using this package!
For this kind of support questions, we'd like you to use the following resources:

Documentation
OAuth 2.0 Documentation
Laracasts
Laravel Slack
Stack Overflow

This will allow us to keep the issues for bug reporting only and make our work easier.


Build action failed

Describe the bug
(https://github.com/emqx/nanomq/actions/runs/4328861530/jobs/7558953427)
OpenSSL dependency breaks CMake
Need to configute the OpenSSL in msuiqc for cross-compiling

Seems like an issue triggered by upstream mbedtls


python312Packages.pynslookup: init at 1.8.1

Module to do DNS lookups
https://github.com/wesinator/pynslookup
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 339626 run on x86_64-linux 1

4 packages built:

python311Packages.pynslookup
python311Packages.pynslookup.dist
python312Packages.pynslookup
python312Packages.pynslookup.dist


ODP sha1hash mismatch

hi all,
It is the response of get metadata
"file": {
    "hashes": {
      "sha1Hash": "53E8EA9AD3CD610F275D1BE816FB10D63EDE818A"
    },
    "mimeType": "application/msword"
  },

but after I download this file and compute file hash by open ssl
openssl sha1 file

the answer is 6df4ed032c4098a48aae761e5bc420b153bb871a
it is the file id
22408535811731DE!30076

Only parts of files would be wrong, is there any issue here?
thank you for help

I apologize for the severe delay, but are you still seeing inconsistent hashes for word documents? I believe there was an issue a long time ago, but it should have been resolved equally as long ago.

hi @ificator
Sorry for re-open this issue,
It is still a problem now,
The sha1 hash in file with extension docx  on ODP is different from the downloaded one.
Please check this case
Accept-Ranges: bytes
CTag: aYzo5RTNFMEI4NkZCRkFBMzY1ITY5NjU4LjMwNQ
Cache-Control: no-cache
Content-Disposition: attachment; filename*=UTF-8''Packing%20Index%281%29.docx
Content-Length: 0
Content-Length: 34181067
Content-Location: https://7quqea.dm2301.livefilestore.com/y3mn7q3JgH1TnuXfg9JepswvzKSULUhlVWoyLZkQo8g5k8lleplCbDOCTr-OP6aPXYCG8hOhrbcj8ddHD6X-EQiXZGGZuMqV8_PxIeJAlSjZGKeTl_LSeYTuDi-OoA1wSCUdDVn6XnfSZbYnWvWcjbq4i1QVGRucrOdcWmxlHxGwcxySfZ5AboM-Pc8XAZiyPw4
Content-Range: bytes 0-34181066/34181067
Content-Type: application/vnd.openxmlformats-officedocument.wordprocessingml.document
Date: Tue, 14 Mar 2017 08:21:39 GMT
Date: Tue, 14 Mar 2017 08:21:44 GMT
ETag: aOUUzRTBCODZGQkZBQTM2NSE2OTY1OC4yNQ
HTTP/1.1 206 Partial Content
HTTP/1.1 302 Found
Last-Modified: Sat, 27 Feb 2016 19:47:02 GMT
Location: https://7quqea.dm2301.livefilestore.com/y3m2Ks-voqphG2DKP2_-JRNK4QneNuqcdg1Jw42DxFp_EeajwBEBNsy_l_kQz2uTGVII2KxQTHOxhkOk7OQAaOcTMVuT7kriDtPzHd_XmhWgwD-D3qNt4cAVTzWIrHK82oiy9GWwvgnuGaKqdmEB9HSsloFEATn5uvvhnFZlgMt_DmXQR4_mKOUyYo6bALeQyl0/Packing%20Index%281%29.docx
P3P: CP="BUS CUR CONo FIN IVDo ONL OUR PHY SAMo TELo"
Server: Microsoft-IIS/8.5
Strict-Transport-Security: max-age=31536000; includeSubDomains
X-AsmVersion: UNKNOWN; <IP_ADDRESS>
X-Content-Type-Options: nosniff
X-MSEdge-Ref: Ref A: 5A5B66E9101D467BA47F34399CE57A99 Ref B: TPE30EDGE0120 Ref C: Tue Mar 14 01:21:39 2017 PST
X-MSEdge-Ref: Ref A: FACD9BAA728245828A70B363074B3D1F Ref B: TPE30EDGE0110 Ref C: Tue Mar 14 01:21:45 2017 PST
X-MSNSERVER: DM5SCH102211004
X-MSNSERVER: DM5SCH102211607
X-PreAuthInfo: rv;
X-SqlDataOrigin: S
X-StreamOrigin: G

file name: Packing Index(1).docx,
file id: 9E3E0B86FBFAA365!69658
mime_type: application/vnd.openxmlformats-officedocument.wordprocessingml.document,
sha1_hash: 90C30B3771E40FAA8FF18684838D25192725A874

After downloaded:
openssl sha1 sha1test
SHA1(sha1test)= e1f2cbb2f1f2742c72a7f7a3e3975d79631a2483

Thank you for help, and sorry for late reply
Best regard
Joe


Lexical GA

moved the beta editor (Lexical-based editor) to the default editor; all pages and posts will now use it
all mobiledoc posts will remain mobiledoc; if opened, they will be converted to Lexical on the fly and be opened within the new editor
we will likely migrate all posts to lexical at some point down the road


Build Linux Static binaries for InfluxDB 2.0

similar to the ones we have for 1.x, we will need a influxdb-2.0.0-static_linux_amd64.tar.gz binary

I just mounted the binaries we do provide for Linux into an empty container and they run just fine.
While they don't have static in their name, I believe these are static binaries.
Closing this for now. Please reopen if I've made a mistake, @russorat

reopening based on https://github.com/influxdata/influxdb/issues/18627

The build system already has support for this so it should just be updating the build command and installing musl. The release happens in a docker container so that needs to be updated.

Add the rust musl target when installing the cross compilation tools https://github.com/influxdata/influxdb/blob/3eac7ec1545cb39b83fd621913065f84d0258c87/.circleci/config.yml#L43 (rustup target add x86_64-unknown-linux-musl)
Switch the go build command to use the special one for static linking inside of goreleaser: https://github.com/influxdata/influxdb/blob/3eac7ec1545cb39b83fd621913065f84d0258c87/.goreleaser.yml#L17

The proper build command needs the following additions to these sections:
ldflags: -extldflags "-fno-PIC -static -Wl,-z,stack-size=8388608"
flags:
    - -tags 'osusergo netgo static_build'

We might also need -buildmode=pie but I'm not sure. We seem to use that for static building internally but I don't know if it's required.

Also the stack size may need to be not set for arm but I'm not sure. We use that to tell musl to restore the default stack size to 8MB instead of the default that musl has of 128k.


Expose Envoy's original_ip_detection custom_header extension

Describe the feature request
We have the problem that in our setup with Akamai as the edge server network, we cannot set a constant numTrustedProxies in the Istio Network Topology settings, because the number of hops that Akamai itself takes for a request is dynamic within the Akamai network. Sometimes, the XFF list contains 2, sometimes 3, sometimes 4, and sometimes up to 6 Akamai-internal IP adresses until it reaches the origin server.
I've seen that Envoy itself has an extension that can be used to read the external/original IP address from another header: https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/http/original_ip_detection/custom_header/v3/custom_header.proto#extension-envoy-http-original-ip-detection-custom-header
Alternatively, it would be great to setup Istio (Envoy) to simply always trust the leftmost XFF entry regardless of how long the XFF list actually is, since we absolutely trust the source IP (being Akamai) to give us a true XFF list (we actually use Akamai SiteShield so that we can whitelist (in an Istio AuthorizationPolicy) the source IP ranges).
Because currently, numTrustedProxies determines the entry from the right of the XFF list to trust. And setting this value to a value greater than the XFF list's length avoids parsing/trusting XFF altogether, and simply uses the IP packet's remote address (as is also documented).
Describe alternatives you've considered
I am not sure if it is possible to use the envoy custom_header extension via an EnvoyFilter, because I don't think it is an actual filter?
Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Dual Stack
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[X] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

I'd prefer if it was using a custom header than the left most XFF - the latter is not implemented in Envoy for a good reason. This should be easy to add as an alternative to XFF detection in https://istio.io/latest/docs/reference/config/istio.mesh.v1alpha1/#Topology API.


Fails to find a component in 2nd test case when using component testing

I'm using cypress 8.4.1. When running multiple tests using component testing (open-ct), the second test fails to find any components using either cy.getReact or cy.react. If I reverse the tests, the first test always passes. I have debugged this to (line 221 in reactHandler v2.3.11):
elements = window.resq.resq$$(component);

elements in this case is an empty array.
I have tried this using cy.waitForReact() after the mount in each test case, and without. The results are the same. I imagine this can be reproduced with a simple use of component testing. Since, for component testing, the root of the React tree is actually part of the component testing app, and is not changed/refreshed between test cases, this might have something to do with it, but I can't see how at this point. It looks like a new instance of resq is created after each waitForReact call.
To reproduce:
git clone https://github.com/francisu/test-material-ui
cd test-material-ui
git checkout componentTestSelector
yarn install
yarn cypress open-ct

(will open GUI - Click on the first test - autocomplete-spec.js)
====> The error will happen at that point

I had this same issue. After digging through the resq code, I discovered it internally keeps its own variable as to whether it had already loaded. I was able to workaround this issue by resetting that variable in my beforeEach:
beforeEach(() => {
  global.isReactLoaded = false;
});


Use SeaORM instead of SeaORM in comment

Using clippy:doc_markdown which is included in clippy:pedantic it warns if this (the actual comment) is used:
//! SeaORM Entity. Generated by sea-orm-codegen 0.9.3

instead of:
//! `SeaORM` Entity. Generated by sea-orm-codegen 0.9.3

What do you think to change it?

Hey @frederikhors, the clippy:doc_markdown lint is allowed by default. But just in case, yeah, I agree changing it to ticked SeaORM.
Would you like to submit a PR and update it?


Replaced word 'shit' with 'air'.

Let's keep it clean people.
Description
[Please describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussion that's relevant]
Check List

[ ] New functionality includes tests
[ ] All tests pass
[ ] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

Let's keep it clean people.

@rcsavage If you get a chance, do you mind adding a DCO sign off to your commit?  You can read about it here https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

Closing this out since the DCO hasn't been signed and we haven't heard back.


RN: Make hermes-inspector-msggen tests visible for jest matcher

Summary:
Changelog: [Internal]
Looks like all this time jest was not running these tests, because their file names are not matching the
https://www.internalfb.com/code/fbsource/[1e8a1b307ffd]/xplat/js/react-native-github/jest.config.js?lines=30

Renamed tests to kebab-case
Some of them were failing, now updated

Differential Revision: D45368748

@hoxyq does this mean all test files using other than kebab-case naming styles are also being ignored?

@hoxyq does this mean all (js) test files using other than kebab-case naming styles are/will be also ignored by jest?

Yes, every test suite with a file name, which doesn't match '/__tests__/.*-test(\\.fb)?\\.js$' will be ignored

@hoxyq does this mean all (js) test files using other than kebab-case naming styles are/will be also ignored by jest?

Yes, every test suite with a file path, which doesn't match '/__tests__/.*-test(\\.fb)?\\.js$' will be ignored

Sure. Tried grep to see if more tests affected by this, seems all other tests are using kebab-case 👍


load project from $cwd/.tmuxinator, then ~/.tmuxinator

Expanding the project search path, to first inspect any .tmuxinator folder beneath the current directory, will allow a tmuxinator project to be included with the source code of the project they relate to.

+1 to this, although the Gemfile changes really don't need to be in there, I would rebase them out personally..

@richoH, Agreed if Bundler was in a default group, but given the development group I thought it'd be useful to know if there are any bundler issues looming.


Building RPM for Centos 7

This is a repost of https://github.com/osuosl/dssp/pull/3 but targeting develop instead of master.

@mathuin where's the doc on what to do to build it? :)

Touché! :-)  I'll add that!

:+1:


Video Gravity for Configurable Resizing.

@eliotw1 everything here looks good to my eye. It looks like there are 3 features:

auto sizing the layer on view bounds change
a new optionality to auto run with an exposed start method
video gravity setting of the layer in the initial case
assuming we want all these I say ship it


opam-0install-cudf: Add a raw diagnostics API for conflicts post-processing

@talex5 do you know if there would be any unused variants in the Raw_diagnostics module definition? (akin to the cases with the assert false)

In the non-cudf API, we have:
  module Diagnostics : sig
    include module type of Zeroinstall_solver.Diagnostics(Solver.Output)
  end

  val diagnostics_rolemap : diagnostics -> Diagnostics.t

Is there a reason to do it differently for cudf?

OK. The asserts look safe to me, given the cudf stuff doesn't use machine groups, class conflicts or commands.

Moved to https://github.com/ocaml-opam/opam-0install-cudf/pull/1


Make heap memory for the Management API configurable

I set the JVM heap with -Xms128m and -Xmx128m - we can probably set both lower. Also setting Netty to use less memory would be good.
┆Issue is synchronized with this Jiraserver Task by Unito
┆Issue Number: K8SSAND-157
┆Priority: Medium

Might be better to make it so the min/max heap for Management API is configurable via an environment variable (with the default staying at the current 128m).


release 1.6.2

obdiag V1.6.2
新版本变更功能点

obdiag gather perf 时移除pstack的收集
cpu_high 场景收集时增加 OB 日志收集

注意
OBD 适配 obdiag V1.6.2 版本需要 V2.7.0 及以上。

Solution Description

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Teingi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add license information

Thanks Jay -- this is great stuff. Let's focus on this next week after completion of Sprint 1

Yup, just opening the issues before I forget.


Update Blog “biryani”

Automatically generated by Netlify CMS

👷 Deploy Preview for magenta-dragon-0c2e51 processing.

Name
Link

🔨 Latest commit
d5b6c112d68d446d8e761a1f1760d9b63b5cde8c

🔍 Latest deploy log
https://app.netlify.com/sites/magenta-dragon-0c2e51/deploys/65b08e0555d5720008641ef7


Pointer::last return None for root pointer

Hey again,
just hit this one: Pointer::last panics for empty/root pointers.
Added the root check as was already done for Pointer::front.
Cheers!

Awesome, thanks. Also, sorry about that.

v0.4.6 was published with this.

Awesome, thanks. Also, sorry about that.

No need to be sorry about anything! You created a nice library with jsonptr, and some things just surface when actually using it.
Thanks for being so responsive!


 You can do this with a lot of things, I suggest using saveTable (https://processing.org/reference/saveTable_.html) and just have every creature return a tableRow with all muscles, nodes and brain in it. This would be useful if anyone has a bug, they can simply share their creatures with you and you can also share the creatures you made in the video with your viewers.

darkracer has already implemented saving as json as per this PR https://github.com/carykh/evolutionSteer/pull/7


Portfolio setup : Header and Headline Section

Make page responsive for devices within the range of 375px and 768px
Use percentages in width of elements in divs instead of pixels
Set dimensions for logo

Hello @wayungi ,
I have made all the required changes as advised.


Increase Client Wait Time and Add MP4 to Audio Conversion

Summary
This PR includes two minor changes:

Increased Wait Time: Extends the client request timeout to accommodate longer processing times.
MP4 to Audio Conversion: Adds basic functionality to extract audio from MP4 files using fluent-ffmpeg. This was added to automate a quick conversion task but can be discarded later if not needed.

Code Snippet
response = operation.result(timeout=1800) # Allow up to 30 minutes

Notes

The audio conversion feature was included for convenience but isn’t a permanent addition unless deemed useful.
Ensure ffmpeg is installed locally for the conversion functionality.

Did you try just providing the mp4 to the transcribe.py script directly?  I feel like that should work without conversion.

Did you try just providing the mp4 to the transcribe.py script directly? I feel like that should work without conversion.

Thanks for the suggestion! I did try providing the mp4 directly to the transcribe.py script, and it worked perfectly. One of the files was recorded on a different platform (I believe Loom) and required a different audio output, which is why the script was created in the first place. That said, I definitely prefer uploading the mp4 file directly whenever possible—it’s much more straightforward!


brew error on M1 with macOS Monterey 12.3

I'm getting this.
→ brew install TomAnthony/brews/itermocil
==> Downloading https://pypi.python.org/packages/source/P/PyYAML/PyYAML-3.11.tar.gz
Already downloaded: /Users/Leeiio/Library/Caches/Homebrew/downloads/d98de4b2d4d986c9eacf67d572ad3182da5555a27ae39f38260d899047b9ef4b--PyYAML-3.11.tar.gz
==> Downloading https://github.com/TomAnthony/itermocil/archive/0.2.1.tar.gz
Already downloaded: /Users/Leeiio/Library/Caches/Homebrew/downloads/7660c48dfb6cb50ac68fa581c5de591c2db93fb698776e83084c669a45eb0299--itermocil-0.2.1.tar.gz
==> Installing itermocil from tomanthony/brews
==> python -c import setuptools... --no-user-cfg install --prefix=/opt/homebrew/Cellar/itermocil/0.2.1/libexec/vendor --install-scripts=/opt/homebrew/Cellar/itermocil/0.2.1/libexec/vendor/bin --single-version-externally-managed --r
Last 15 lines from /Users/Leeiio/Library/Logs/Homebrew/itermocil/01.python:
2022-03-16 04:59:53 +0000

python
-c
import setuptools, tokenize
__file__ = 'setup.py'
exec(compile(getattr(tokenize, 'open', open)(__file__).read()
  .replace('\r\n', '\n'), __file__, 'exec'))
--no-user-cfg
install
--prefix=/opt/homebrew/Cellar/itermocil/0.2.1/libexec/vendor
--install-scripts=/opt/homebrew/Cellar/itermocil/0.2.1/libexec/vendor/bin
--single-version-externally-managed
--record=installed.txt

If reporting this issue please do so at (not Homebrew/brew or Homebrew/core):
  https://github.com/tomanthony/homebrew-brews/issues

These open issues may also help:
I can't install itermocil.  https://github.com/TomAnthony/homebrew-brews/issues/4
itermocil install is failing https://github.com/TomAnthony/homebrew-brews/issues/2

Python support was dropped from MacOS 12.3. The script is pointing at /usr/bin/python which is now kaput!
I'll update if I find a decent solution.

Is there a temporary workaround?

I am using the following very hacky, very dirty workaround:

Install Python 2.7.18 through pyenv, given Homebrew no longer supports it:

> brew install pyenv
> pyenv install 2.7.18

Replace the Python2 bin location in itermocil binary:

> nano /usr/local/Cellar/itermocil/0.2.1/libexec/bin/itermocil

-#!/usr/bin/python
+#!/Users/YOUR_USERNAME/.pyenv/versions/2.7.18/bin/python2.7

Now the proper solution would be to migrate the codebase to Python3 and change the Python binary location, but I don't see that happening anytime soon unfortunately.

I have uninstall the itermocil and now I can not install it any more...

just in case. as this repo doesn't seem to be updated anymore, i've forked and updated for Python3: https://github.com/godbout/itermocil

Nice one @AdrienPoupa - this was my approach too.
But @godbout - legend. Your fork works a treat. Cheers.

@scottluxford enjoy 💫  had to get this fixed before working on my projects. life without iTermocil is so sparkless.

just in case. as this repo doesn't seem to be updated anymore, i've forked and updated for Python3: https://github.com/godbout/itermocil

Nice job! But when I install the itermocil and use itermocil test, I am getting this
Traceback (most recent call last):
  File "/Users/Leeiio/.pyenv/versions/3.7.12/bin/itermocil", line 11, in <module>
    load_entry_point('itermocil==0.2.3', 'console_scripts', 'itermocil')()
  File "/Users/Leeiio/.pyenv/versions/3.7.12/lib/python3.7/site-packages/itermocil.py", line 752, in main
    instance = Itermocil(filepath, here=args.here, cwd=cwd)
  File "/Users/Leeiio/.pyenv/versions/3.7.12/lib/python3.7/site-packages/itermocil.py", line 79, in __init__
    self.process_file()
  File "/Users/Leeiio/.pyenv/versions/3.7.12/lib/python3.7/site-packages/itermocil.py", line 568, in process_file
    self.arrange_panes(len(window['panes']), layout)
  File "/Users/Leeiio/.pyenv/versions/3.7.12/lib/python3.7/site-packages/itermocil.py", line 262, in arrange_panes
    for p in range(0, second_columns):
TypeError: 'float' object cannot be interpreted as an integer

Anyone know how to solve this? I have zero knowledge in 🐍

no idea sorry. zero Python here also. was not even aware that there were tests?
in my case i'm under Python 3.9. it's possible that the changes i've made are not enough for your config. they work for my owns with setting up vertical and horizontal panes though, and multiple tabs.

you have the stack in the error log, and the error shows an issue between float and integer conversion. that should help. maybe an issue with your config, maybe some more changes to be made. strings needed utf8 encoding/decoding. possible that there's more type restrictions now.

open itermocil.py file, change the code in line 255 second_columns = num_panes / 2 to second_columns = num_panes // 2, done.

nice :D do a PR!

open itermocil.py file, change the code in line 236 second_columns = num_panes / 2 to second_columns = num_panes // 2, done.

I would like to point out, that first of all

you have to install itermocil from this repo: https://github.com/TomAnthony/itermocil/issues/121#issuecomment-1069266639
Then change code in line 236
And only then install itermocil
python3 setup.py install --install-scripts=/usr/local/bin

ok so if no one does it i'll do it.
well i did it.

ok so if no one does it i'll do it.
well i did it.

huh i saw the PR made by @Leeiio only after i've made the changes :D sorry, didn't receive any notification. strange. anyways, still merged. i've made the changes necessary regarding this issue for the other layouts too.

@Leeiio i'll check. maybe if you're using my fork it's best to report bugs there.

windows:

name: test
root: ~
layout: tiled
panes:

commands:

ls

commands:

ls

commands:

ls
focus: true

commands:

ls

works here.

are you sure you're using the right iTermocil? if you make changes yourself in the itermocil.py file you need to run the file with python3 itermocil.py unless you run the setup each time.

@godbout Thanx, I have reinstalled the iTermocil, now everything is ok!

@godbout thanks so much for doing this, worked like a charm.  Do you have a buymeacoffee.com link or something?

@DougThwaites if you're into Vim i built Vim Mode for macOS. license is one coffee a month: https://github.com/godbout/kindaVim.theapp and https://kindavim.app
if you're not into Vim, then i have an old ko-fi account yeah but i might use the money to buy beers instead: http://ko-fi.com/A1553NO
else thoughts and prayers for GitHub to accept my sponsorship account. i've only been waiting for 2.5 years.
thank you!

Not really a Vim guy but looks cool.
Just chucked a few bob your way via Ko-fi, hope it'll get ya a couple of beers where you are.
Also that's crazy about the sponsorship account taking so long. Good luck.

@DougThwaites that one's on you:

(hopefully i'm better at building software than i am at taking pictures.)
thanks a lot. greatly appreciated!

and of course i'll update this thread with the next beer.

Found another solution.
https://github.com/mapbox/node-sqlite3/issues/1552#issuecomment-1075059490

@Lzmog Doesn't this change the default Python to 2.7.18 everywhere? Won't this break other apps?

- Doesn't this change the default Python to 2.7.18 everywhere?
As I understand, yes it changes the default Python into 2.7.18.
- Won't this break other apps?
As it is now, I didn't notice any breaks related to other apps, but it might effect some apps in future.

and of course i'll update this thread with the next beer.

as promised.

Amazing @godbout, thanks again for your work mate.

@DougThwaites hey no worries man. i did it selfishly for myself first. and it was just a couple of tweaks. best ration of work over love received! thanks again for the words and the 🍺

Additional point. With @godbout 's solution (thx!) I got this
error: The 'PyYAML' distribution was not found and is required by iTermocil
I tried to remove line 1 PyYAML==3.11 in file requirements.txt and line 21 PyYAML in file setup.py. Now it works like a magic

I tried to remove line 1 PyYAML==3.11 in file requirements.txt and line 21 PyYAML in file setup.py. Now it works like a magic

@henryalps This allowed installation to to go through, but executing the itermocil command still throws:
Traceback (most recent call last):
  File "/usr/local/bin/itermocil", line 33, in <module>
    sys.exit(load_entry_point('iTermocil==1.0.0', 'console_scripts', 'itermocil')())
  File "/Users/i349079/Downloads/itermocil/itermocil.py", line 733, in main
    instance = Itermocil(filepath, here=args.here, cwd=cwd)
  File "/Users/i349079/Downloads/itermocil/itermocil.py", line 53, in __init__
    self.parsed_config = yaml.load(f, Loader=yaml.Loader)
  File "/Users/i349079/Library/Python/3.8/lib/python/site-packages/yaml/__init__.py", line 79, in load
    loader = Loader(stream)
  File "/Users/i349079/Library/Python/3.8/lib/python/site-packages/yaml/loader.py", line 44, in __init__
    Reader.__init__(self, stream)
  File "/Users/i349079/Library/Python/3.8/lib/python/site-packages/yaml/reader.py", line 85, in __init__
    self.determine_encoding()
  File "/Users/i349079/Library/Python/3.8/lib/python/site-packages/yaml/reader.py", line 124, in determine_encoding
    self.update_raw()
  File "/Users/i349079/Library/Python/3.8/lib/python/site-packages/yaml/reader.py", line 178, in update_raw
    data = self.stream.read(size)
TimeoutError: [Errno 60] Operation timed out

The same error is thrown even if I try installing PyYAML and keeping it in the requirements.
Every other method failed for me. I've been so used to this tool on my old Mac for the past 3 years, that it will now become tough for me to use my Mac without it! XD

no idea if it's related to your issue but just in case if you're on an Apple silicon the install folder should be: /opt/homebrew/
hence if you're installing from my fork: python3 setup.py install --install-scripts=/opt/homebrew/bin
i'll update the README.

@godbout Thanks a lot for the suggestion. But this did not work too. Installation goes through fine, but the same runtime error still pops up. I tried uninstalling and re-installing PyYAML too, to the needed version (3.11), but no luck there as well.

@TomAnthony Would it be possible to give somebody from this thread access to this repository so that we can just fix this one instead of having to do all kinds of hacks/forks to get it working again? Itermocil is such a great piece of software that I used every day and can no longer. 💙

@godbout Thanks a lot for the suggestion. But this did not work too. Installation goes through fine, but the same runtime error still pops up. I tried uninstalling and re-installing PyYAML too, to the needed version (3.11), but no luck there as well.

just in case, i think PyAML is needed and you should put it back, if you're removed that line. not sure if it's a Python 3.8 error. i got a fresh install from last week on 12.3 and it worked fine. Python 3.9 (latest) installed from Homebrew.

@ruudk - I have added you. I haven't had time to maintain this for sometime now, and have reached out to a few people before to find a new maintainer without much luck. Happy if someone on this thread wants to take a wider maintainer role! :)

@TomAnthony Thanks for your reply! My Python skills are terrible.
@godbout Since you have been so helpful in this thread, would this be something for you to take on? I would be happy to offer you a few beers of coffee's 😊

@godbout I'd chuck another few beers in the pot along with @ruudk to get the original repo all ticky boo.

@DougThwaites i definitely can help with making the official repo up to date. i forked before instead because the repo didn’t seem maintained anymore. for future PRs tho i don’t think i’ll have the Python skills to judge. PHP JS Swift here and going full Mac apps dev so python is definitively not on the list. if it’s fine with you guys then i’m in.

@godbout That's great! We just need somebody to make it working again on Python 3.
@TomAnthony Could you invite @godbout ? Thanks!

@ruudk @godbout Done! :)

guys. i’m in the middle of applying for my French passport renewal after being stranded on 30 square kilometers for 2.5 FUCKING YEARS. they finally sent some consulate agent, coz it’s the French presidential elections. so i’m gonna finish this off first and only then i’ll be on iTermocil. thanks 😄

oh also @TomAnthony. not sure how you want to handle the repo now. are you completely out or you still want to have a say?
because what i usually do in my own projects is support only the two latest official language versions (usually the last one being the stable, the previous one being the security fixes and bugs updates).
for iTermocil what i would do is release a v1 for Python 3 (probably 3.9) that would break for Python 2.7 most probably. like if people want iTermocil for 2.7, i guess we would need to do a versioned release in Homebrew.
again, no knowledge of Python or the Python ecosystem here. i just know pip coz it sounds very funny in French.

Let's keep it simple. When people want to run this on Python V2, they're on their own and can use an old tag.
The project should just be bumped to Python V3 as that's the default Python in Brew now.

but iTermocil is important! can't we add this to homebrew-core? i've added stuff homebrew-cask but never to core or taps. not sure about their policies. thought brew taps where kinda obsolete now?

policies here: https://docs.brew.sh/Acceptable-Formulae

Let's first focus on fixing this, otherwise it will def never be accepted :)

ok i've made a PR with all the changes for Python3. what's needed now is to make a release. what i'm not sure about yet is how we want to install iTermocil. i wanna add it to homebrew-core and just do a brew install itermocil, but not sure yet whether it would be accepted or not.
if not we may need to update @TomAnthony's brews/taps. i have no knowledge of personal taps, but it seems that one is currently outdated and would need some rework: https://github.com/TomAnthony/homebrew-brews/issues

Let's first focus on fixing this, otherwise it will def never be accepted :)

i've updated to Python3. code works now. at least on my machine. not sure about M1. maybe best is that people install and try and report before we do a homebrew release.

@TomAnthony your brews' syntax need to be updated, currently they're not working. we can help here by:

updating your current tap. if it's ok with you, can you invite us as collaborators?
create a new tap, and then update the README with the new installation procedure. but then that kind of defeat the purpose of working on this current repo.

thanks.

@godbout Didn't had my notifications set up so therefore I missed the PR. Just enabled them.
I reviewed it just yet and it seems to be good.
@TomAnthony Please invite @godbout and @ruudk to the https://github.com/TomAnthony/homebrew-brews repo so that we can fix it there too.

ok. @TomAnthony any reason why this has to be installed through brew install TomAnthony/brews/itermocil and not directly in Homebrew core? brew install itermocil?

Just that it was a long time ago, and I tried to submit it to core a few times but couldn't get it accepted at the time.

@TomAnthony Please invite @godbout and @ruudk to the https://github.com/TomAnthony/homebrew-brews repo so that we can fix it there too.

Done!

oh also @TomAnthony. not sure how you want to handle the repo now. are you completely out or you still want to have a say?
because what i usually do in my own projects is support only the two latest official language versions (usually the last one being the stable, the previous one being the security fixes and bugs updates).

Whilst I would love to stay involved, realistically I am too time poor and I don't want to hold things back. I'm happy for you to make decisions. :) I might want to be involved more again in the future, but in that case I would expect to be more of an equal collaborator rather then benevolent dictator. :)
Thanks for your work on this!

Just that it was a long time ago, and I tried to submit it to core a few times but couldn't get it accepted at the time.

you're definitely smarter than me. it took three days to understand the doc and it seems you made the right choice. doesn't go to core.

Done!

thank you!

Whilst I would love to stay involved, realistically I am too time poor and I don't want to hold things back. I'm happy for you to make decisions. :) I might want to be involved more again in the future, but in that case I would expect to be more of an equal collaborator rather then benevolent dictator. :)

tbh i'm like you, barely have time (other priorities). but on any project i work the first i do is... run iTermocil 😄 so we'll get it working at least. definitely no issue (probably would actually prefer) if you'd stay the benevolent dictator.

Thanks for your work on this!

no worries. thanks for the amazing tool you created!

Let's keep it simple. When people want to run this on Python V2, they're on their own and can use an old tag.
The project should just be bumped to Python V3 as that's the default Python in Brew now.

well, computers are not that simple 😅
i've updated the brew and iTermocil's code. now here's the thing. by default there's not Python interpreter on macOS 12.3 anymore. but once you install Xcode and the component tools, Python is installed. but it's v3.8. installing through Homebrew with brew install python you can 3.9. but 3.10 is actually the latest version (https://www.python.org/downloads/), and you can install it with brew install<EMAIL_ADDRESS>so. i have no idea if the changes i've made would work on the 3 versions. plus afaik (which means nothing), we need to point to an interpreter in iTermocil's code. currently it points to /usr/bin/python which is the one installed with Xcode/tools.
so bis. which one we go for? we'll have to edit the README i assume because afaik (again), iTermocil's users will have to install Python first. i'd say easiest would be just a brew install python, hence v3.9 (for now? maybe change in the future?). any thoughts?

I'll have a look later. There are a lot of formulas that depend on Python 3 and then use a homebrew helper to get that specific path.

ok done. i can personally install through the usual instructions from the README (added the brew install python step tho). installs fine, then using it works well too. iTermocil itself should work with any Python3 version, but the setup is currently kinda hardcoded to 3.9. not sure how to handle that yet. will research if it's an issue for someone.
you guys please try and report! thank you.

The problem has been fixed, please run brew update && brew install tomanthony/brews/itermocil to retry.
If you encounter issues, please create a new issue.

@godbout @DougTGP https://Ko-fi.com/home/coffeeshop?txid=f8f4359a-cbed-4a09-acb6-18facd883573&mode=public&img=ogiboughtsomeone as promised, cheers 🍻

hey @ruudk thanks but you did some great work too. so back to you. where's your own ko-fi page? :D


pageIndex is not reset on Fully Controlled Component

Describe the bug
If user go to page index 1 or more and then user filter the data, page index isn't reset to 0 but it's still in the last page index user has visited
Codesandbox!
https://codesandbox.io/s/manualfiltersdoesntresetpageindex-bs4ud

Fixed in latest commits to master and slated for release. Thanks!


Remove protective copying operations from the spark BQSR

Turns out these may not be needed in spark (unlike in dataflow).
Let's get expert confirmation that this is the case, however,
before merging.

@tomwhite and/or @laserson Could we please get your expert opinions as to whether it's safe to remove the protective copying operations from the spark BQSR like this?


About  container PRECESS NAME'S issues

HI'
we  use Centos  for  machine, and  ubuntu14.10  for container  images.

[root@CWEB~]# docker version
Client version: 1.2.0
Client API version: 1.14
Go version (client): go1.3.3
Git commit (client): 2a2f26c/1.2.0
OS/Arch (client): linux/amd64
Server version: 1.2.0
Server API version: 1.14
Go version (server): go1.3.3
Git commit (server): 2a2f26c/1.2.0
 now, we found container's  precess  USER NAME  may  used  by  machine system USER_NAME so that  PRECESS USER  not  right.

The container  /etc/passwd follow  info:
more /etc/passwd
dockerweb:x:1000:1000::/home/dockerweb:/bin/bash
the machine  system /etc/passwd  follow info:
more /etc/passwd
inter:x:1000:1000::/home/inter:/bin/bash
we  start  container  , the  PRECESS NAME  should  be  “dockerweb” ,But  in fact  it's “inter”.
<EMAIL_ADDRESS>thanks.

closing please let us know the exact steps to reproduce


Change Winget Releaser job to ubuntu-latest

Winget Releaser now supports non-Windows runners, and ubuntu-latest is generally faster.

Nice, thank you.


Multiple Problems (sync, freeze, updates stopped)

Mac OSX 10.10.3 (Yosemite)
Multibit 5.18
I have most of the same problems reported here,
-takes forever to synch
-freezes, have to force quit
-another program has changed this wallet
I don't have the tech know how to do the things you've suggested to others, I'm 65 years old. Can't you do something to help in an update to Multibit? All I want is my Bitcoins back.
Thanks,
Luca

You might want to try the steps for transfering your wallet to blockchain.,info:
https://multibit.org/en/help/v0.5/help_movingAWallet.html
blockchain.info has been up and down for maintenance this weekend so you might want to wait for that to be resolved.

Hi Jim,I do not have the tech know how to move my wallet to blockchain, like I mentioned earlier I'm 65 yrs old and very new to bitcoin.I have another solution, tell me how to send you my private key you cash out my wallet and take a donation. I'll give you a new wallet address and you won't hear from me again. I only have about $30 in my wallet but I don't want to lose it.ThanksLuca
 On Sunday, June 7, 2015 2:25 PM, Jim Burton<EMAIL_ADDRESS>wrote:

You might want to try the steps for transfering your wallet to blockchain.,info:
https://multibit.org/en/help/v0.5/help_movingAWallet.htmlblockchain.info has been up and down for maintenance this weekend so you might want to wait for that to be resolved.—
Reply to this email directly or view it on GitHub.

Hi Lucapao,
If you get your private key exported you are almost there.
We don't offer personal support as you can imagine it gets problematic with the number of users MultiBit has.
Another option would be to wait for the refresh of MultBit we are planning which should tidy up the 'updates stopped' problem.

Closing due to no activity.


Add link color override rule for PF styles in production builds

Description
Regular anchor  links have a PatternFly/Tailwind style conflict.
PatternFly's default style is to make links blue with color: var(--pf-global--link--Color);, while Tailwind reverts links to black with color: inherit;. In development links are the correct blue color, but they become black in prod due to our Tailwind style sheet being loaded first in production builds.
This ensures that the PF rule takes precedence with .pf-c-page__main-section a (0-1-1) over the Tailwind a (0-0-1) on all pages that are within a PatternFly . Once Tailwind is removed from the project this rule can be deleted.
Alternatively we could convert all links to use a PatternFly <Button variant='link' isSmall isInline>, but that seems a bit more heavyweight.
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
Visit the Dashboard in a production environment and ensure that the links are blue and not black.

Current dependencies on/for this PR:

master

PR #2241   👈

This comment was auto-generated by Graphite.


Xia Portal Won't Open

Maybe I'm unclear on what I'm supposed to do exactly but the instructions seem fairly straight forward. I tried throwing a void star into a hole of soft bedrock and then I tried throwing the void star and then placing soft bedrock around it, neither seems to have worked. I tried completely enclosing the star in soft bedrock and waited a little while but still nothing happened. Am I doing something wrong or is this some kind of bug?

It should be a 3x2x3 of soft bedrock where the top layer should be empty in the center, that's where you throw the star

2 high huh? Just tried that too, still nothing, here's a picture, you can see the star just spinning around down there. I know it's still entirely possible I'm just misunderstanding.

I'll check it out incase something broke

Oh I should probably have mentioned this is 1.11, and I just remembered you're not supporting that anymore so never mind. Thanks for replying so quick though, my mistake. ^.^

Ah yea, possible it's broken in that version, I can't remember. However this is what it should look like and does work in the current versions just fine.


Memory Leak in VmModule VmModule::FromFlatbufferBlob using  Python Bindings

What happened?
While tuning 10,000+ kernels using the IREE python bindings, after several hours (+/- days) of running, the process will OOM:
[Sun May 21 23:59:29 2023] oom-kill:constraint=CONSTRAINT_NONE,nodemask=(null),cpuset=/,mems_allowed=0,global_oom,task_memcg=/user.slice/user-1007.slice/session-6.scope,task=python3,pid=445577,uid=1007
[Sun May 21 23:59:29 2023] Out of memory: Killed process 445577 (python3) total-vm:91281012kB, anon-rss:85251192kB, file-rss:74836kB, shmem-rss:8192kB, UID:1007 pgtables:168244kB oom_score_adj:0

tracemalloc shows persistent growth in memory from subprocess beyond invocation of IREE tools, leaking from the use of compile_str.
/usr/lib/python3.10/subprocess.py:2035: size=8404 KiB, count=11, average=764 KiB
...
  File "/usr/local/google/home/kooljblack/Code/iree-samples/tuning/profiler.py", line 52
    flatbuffer_blob = ireec.compile_str(
  File "/usr/local/google/home/kooljblack/Code/iree-build/compiler/bindings/python/iree/compiler/tools/core.py", line 278
    result = invoke_immediate(cl, immediate_input=input_bytes)
  File "/usr/local/google/home/kooljblack/Code/iree-build/compiler/bindings/python/iree/compiler/tools/binaries.py", line 192
    process = subprocess.run(command_line, capture_output=True, **run_args)
  File "/usr/lib/python3.10/subprocess.py", line 505
    stdout, stderr = process.communicate(input, timeout=timeout)
  File "/usr/lib/python3.10/subprocess.py", line 1154
    stdout, stderr = self._communicate(input, endtime, timeout)
  File "/usr/lib/python3.10/subprocess.py", line 2035
    stdout = b''.join(stdout)

Cause
The runtime bindings rely on the creation of a VmModule via pybind11 to expose functions and invocation. VmModule VmModule::FromFlatbufferBlob() adds a reference to any passed flatbuffer object: https://github.com/openxla/iree/blob/161daa7f21ea06de46582e24595a224bf24ce611/runtime/bindings/python/vm.cc#L170
In my case, this buffer built on stdout from iree-compile, but any passed in flatbuffer will leak a reference. Deleting python objects will not release the reference. There is no equivalent function in the bindings to release the reference.

Are you 100% positive about that? the ctl_fn immediately above that is what should be decreasing the reference count.

Its possible I'm using the API incorrectly. Here is a simple case of creating a new buffer and going through the VM to show what happens:
    flatbuffer_blob_copy = bytearray()
    print(f"ref new buffer: {sys.getrefcount(flatbuffer_blob_copy)}")
    for b in flatbuffer_blob:
        flatbuffer_blob_copy.append(b)
    print(f"ref count after fill: {sys.getrefcount(flatbuffer_blob_copy)}")
    # Create a module
    config = ireert.Config(driver_name=driver)
    vm_module = ireert.VmModule.from_flatbuffer(
        config.vm_instance, flatbuffer_blob_copy)
    print(f"ref count after vm: {sys.getrefcount(flatbuffer_blob_copy)}")
    del config
    del vm_module
    print(f"ref count after del: {sys.getrefcount(flatbuffer_blob_copy)}")

and this is the printout:
ref new buffer: 2
ref count after fill: 2
ref count after vm: 4
ref count after del: 3

I can't explain why the count starts at 2 instead of1, but if I create a context then module then delete both, its now 3, not 2. Can't show the ref count if I also del flatbuffer_blob_copy
Also, through process of elimination, I'm certain the cause of the leak is passing flatbuffers to the VM module. emitting this code but keeping everything else including compile results in 0 leaks.

Put a printf in the ctl_fn.

For ASan, see https://github.com/openxla/iree/issues/9067

For ASan, see #9067

Looks like running ASAN with the CUDA driver doesn't work. Once I enabled buidling with shared libraries and used the fix from Stellas comment, I ran into this error on VM creation:
Traceback (most recent call last):
  File "/usr/local/google/home/kooljblack/Code/iree-build/runtime/bindings/python/iree/runtime/system_setup.py", line 84, in get_first_device
    return get_device(device_uri, cache=cache)
  File "/usr/local/google/home/kooljblack/Code/iree-build/runtime/bindings/python/iree/runtime/system_setup.py", line 50, in get_device
    device = driver.create_device_by_uri(device_uri)
RuntimeError: Error creating device: iree/runtime/src/iree/hal/drivers/cuda/cuda_driver.c:123: INTERNAL; CUDA driver error 'CUDA_ERROR_OUT_OF_MEMORY' (2): out of memory; creating device 'cuda'

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

Traceback (most recent call last):
  File "/usr/local/google/home/kooljblack/Code/iree-samples/tuning/tuner.py", line 130, in <module>
    main(parse_arguments())
  File "/usr/local/google/home/kooljblack/Code/iree-samples/tuning/tuner.py", line 122, in main
    run_program(
  File "/usr/local/google/home/kooljblack/Code/iree-samples/tuning/tuner.py", line 53, in run_program
    run_profile(
  File "/usr/local/google/home/kooljblack/Code/iree-samples/tuning/profiler.py", line 270, in run_profile
    benchmark_results, benchmark_err = run_benchmark_module(compilation_result.flatbuffer_blob, entry_function="forward",
  File "/usr/local/google/home/kooljblack/Code/iree-samples/tuning/profiler.py", line 130, in run_benchmark_module
    config = ireert.Config(driver_name=driver)
  File "/usr/local/google/home/kooljblack/Code/iree-build/runtime/bindings/python/iree/runtime/system_api.py", line 71, in __init__
    self.device = get_first_device(
  File "/usr/local/google/home/kooljblack/Code/iree-build/runtime/bindings/python/iree/runtime/system_setup.py", line 95, in get_first_device
    raise RuntimeError("Could not create device. "
RuntimeError: Could not create device. Exception for last tried follows.

To reiterate above, I'm confident the leak is in the VM module since pythons memory analysis tracemalloc shows exactly whats leaking and gc illustrates where its happening. I'll try ben's print suggestion.

Try with CPU - there's nothing CUDA specific here.

So, building with ASAN did not reveal any memory leaks. I tried this without modifying the code, but then also removing the ctl_fn() and then removing  flatbuffer_blob.inc_ref();, still no leaks. I gather that because the leak happens at the python interpreter level, its not something ASAN would track per se?
Anyway, as expected removing  flatbuffer_blob.inc_ref() indeed fixes the leak. The new ref counts from the code in my earlier comment:
ref new buffer: 2
ref count after fill: 2
ref count after vm: 3
ref count after del: 2

Count goes up one when bound to the module then resets back to 2 when deleted. Clearly this is expected behavior.
But without the manual reference increase, what actually ups the count by 1? Turns out assigning the stashed_flatbuffer_blob:https://github.com/openxla/iree/blob/main/runtime/bindings/python/vm.cc/#LL184C1-L184C1
pybind is somehow smart enough to automatically handle references on C++ assignment and deletion. There isn't much documentation on how this works however.
If I remove this, no ref count change in python:
ref new buffer: 2
ref count after fill: 2
ref count after vm: 2
ref count after del: 2

@benvanik I checked the ctl_fn, it is getting called. From what I understand, this is not part of pybinds but instead the python's C Api: https://docs.python.org/3/c-api/refcounting.html. From what I gather, the free is happening on the raw PTR to the block data within the bytebuffer object, not the bytbuffer itself. The macro silently does nothing in this case and removing ctl_fn has no change in reference counts.
Anyway, solution here seems simple. Just remove the superfluous flatbuffer_blob.inc_ref();

That's not superfluous - that's what keeps the memory live for the duration the module hangs on to it.

as it is right now, assigning the flatbuffer object and manually increasing the ref count results in refcount +=2. But deleting the VmModule only results in refcount -=1:
ref count after fill: 2
ref count after vm: 4 # Double ref count from a single VM Call

To rephrase my point: Its not clear why we need to double increment this object's reference count when its only held once in the VM? Assigning it to the VmModule will keep it alive for the duration of the VmModule. Also, it won't die during the call to FromFlatbufferBlob since the python object also has a live reference in the callees stack frame.

What line 170 (the increment) and line 167 (the decrement) are doing is conceptually correct and must not be removed. If you pass data to iree_vm_bytecode_module_create you must retain that data and then release it in the allocation callback (ctl_fn). Any other retain/release is orthogonal and must not be relied upon for correctness. Printf'ing the reference count at certain points in time and saying "we could probably get away without one of them in some cases" is an extremely yolo approach to working with GC'ed/reference counted systems ;)
IOW: retaining the backing flatbuffer memory for the lifetime of the iree_vm_module_t (which may outlive the C++ VmModule!) is critical, that's what the code is supposed to be doing, but maybe it's not and that's what needs to be figured out.
Reading the code I suspect the bug is that it's casting the pointer to be freed to a PyObject* before decrementing the reference - which obviously isn't valid as the pointer to be freed is the flatbuffer data and not a python object. I'm going to guess that instead of passing NULL for self on L171 you'll want to pass flatbuffer_blob_object.ptr() and then do Py_XDECREF(static_cast<PyObject*>(self)) in the ctl_fn and completely ignore the inout_ptr. This just looks like a latent 3 year old bug.

Found the reference for how reference counting on PyObjects is handled automatically: https://pybind11.readthedocs.io/en/stable/reference.html#classobject
Reference calls out:

Manually increase the reference count of the Python object. Usually, it is preferable to use the object class which derives from handle and calls this function automatically. Returns a reference to itself.

This follows what I've found above from experiment. @benvanik  my point is that the reference counting to keep the flatbuffer alive is already happening on assignment and deallocation through pybind. ctl_fn only runs when the object is deallocated, also when pybind releases its reference.
It not clear why we'd do this separate manually on top of that, which we're currently not clearing properly. rather than try to balance, I'd like to remove the extra reference. Am I missing something?

aaah, looks like we commented the same time. I see your point about separate lifetimes. I'll try fixing the ctr_fn.


Dotnet-monitor hangs for apps spawned in the background

When creating apps in the background and using the hosting process (e.g. dotnet run &), dotnet-monitor will hang when making rest calls. So far, this has been reproduced on WSL2 with Ubuntu 18.04 and 20.04.

Example callstack of hang:
System.Net.Sockets.dll!System.Net.Sockets.SocketPal.Receive(System.Net.Sockets.SafeSocketHandle socket, System.Net.Sockets.SocketFlags flags, System.Span<byte> buffer, byte[] socketAddress, ref int socketAddressLen = 0, out System.Net.Sockets.SocketFlags receivedFlags =<PHONE_NUMBER>, out Interop.Error errno = SUCCESS)
System.Net.Sockets.dll!System.Net.Sockets.SocketPal.TryCompleteReceiveFrom(System.Net.Sockets.SafeSocketHandle socket, System.Span<byte> buffer, System.Collections.Generic.IList<System.ArraySegment<byte>> buffers, System.Net.Sockets.SocketFlags flags, byte[] socketAddress, ref int socketAddressLen, out int bytesReceived = 0, out System.Net.Sockets.SocketFlags receivedFlags =<PHONE_NUMBER>, out System.Net.Sockets.SocketError errorCode = Success)
System.Net.Sockets.dll!System.Net.Sockets.SocketAsyncContext.ReceiveFrom(System.Memory<byte> buffer, ref System.Net.Sockets.SocketFlags flags = None, byte[] socketAddress = null, ref int socketAddressLen = 0, int timeout = -1, out int bytesReceived = 0)
System.Net.Sockets.dll!System.Net.Sockets.SocketPal.Receive(System.Net.Sockets.SafeSocketHandle handle, byte[] buffer, int offset, int count, System.Net.Sockets.SocketFlags socketFlags, out int bytesTransferred)
System.Net.Sockets.dll!System.Net.Sockets.Socket.Receive(byte[] buffer = {byte[14]}, int offset = 0, int size, System.Net.Sockets.SocketFlags socketFlags, out System.Net.Sockets.SocketError errorCode =<PHONE_NUMBER>)
System.Net.Sockets.dll!System.Net.Sockets.NetworkStream.Read(byte[] buffer, int offset, int size)
System.Private.CoreLib.dll!System.IO.BinaryReader.ReadBytes(int count = 14)
Microsoft.Diagnostics.NETCore.Client.dll!Microsoft.Diagnostics.NETCore.Client.IpcHeader.TryParse(System.IO.BinaryReader reader = {System.IO.BinaryReader})
Microsoft.Diagnostics.NETCore.Client.dll!Microsoft.Diagnostics.NETCore.Client.IpcMessage.Parse(System.IO.Stream stream = {Microsoft.Diagnostics.NETCore.Client.ExposedSocketNetworkStream})
Microsoft.Diagnostics.NETCore.Client.dll!Microsoft.Diagnostics.NETCore.Client.IpcClient.Read(System.IO.Stream stream = {Microsoft.Diagnostics.NETCore.Client.ExposedSocketNetworkStream})
Microsoft.Diagnostics.NETCore.Client.dll!Microsoft.Diagnostics.NETCore.Client.IpcClient.SendMessage(Microsoft.Diagnostics.NETCore.Client.IpcEndpoint endpoint = {Microsoft.Diagnostics.NETCore.Client.PidIpcEndpoint}, Microsoft.Diagnostics.NETCore.Client.IpcMessage message = {Microsoft.Diagnostics.NETCore.Client.IpcMessage})
Microsoft.Diagnostics.NETCore.Client.dll!Microsoft.Diagnostics.NETCore.Client.DiagnosticsClient.GetProcessInfo()
Microsoft.Diagnostics.Monitoring.dll!Microsoft.Diagnostics.Monitoring.EndpointInfo.FromProcessId(int processId = 39)
Microsoft.Diagnostics.Monitoring.dll!Microsoft.Diagnostics.Monitoring.ClientEndpointInfoSource.GetEndpointInfoAsync.AnonymousMethod__2()
System.Private.CoreLib.dll!System.Threading.Tasks.Task<Microsoft.Diagnostics.Monitoring.EndpointInfo>.InnerInvoke()
System.Private.CoreLib.dll!System.Threading.Tasks.Task..cctor.AnonymousMethod__274_0(object obj)
System.Private.CoreLib.dll!System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(System.Threading.Thread threadPoolThread = {System.Threading.Thread}, System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, object state)
System.Private.CoreLib.dll!System.Threading.Tasks.Task.ExecuteWithThreadLocal(ref System.Threading.Tasks.Task currentTaskSlot = Id = 2, Status = Running, Method = "Microsoft.Diagnostics.Monitoring.EndpointInfo <GetEndpointInfoAsync>b__2()", Result = "{Not yet computed}", System.Threading.Thread threadPoolThread)
System.Private.CoreLib.dll!System.Threading.Tasks.Task.ExecuteEntryUnsafe(System.Threading.Thread threadPoolThread)
System.Private.CoreLib.dll!System.Threading.Tasks.Task.ExecuteFromThreadPool(System.Threading.Thread threadPoolThread)
System.Private.CoreLib.dll!System.Threading.ThreadPoolWorkQueue.Dispatch()
System.Private.CoreLib.dll!System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()

This instance of dotnet-monitor is attempting to invoke the ProcessInfo command on process 39.
Process 39 is the dotnet run & process.
ps -aux:
jander      39  0.3  0.6 4053812 81360 pts/0   Tl   16:26   0:02 dotnet run
jander      64  0.6  0.0      0     0 pts/0    Z    16:26   0:04 [dotnet] <defunct>

This hang would be mitigated if cancellation on the DiangosticsClient was implemented: https://github.com/dotnet/diagnostics/issues/2151

Killing the target process (process 39) makes dotnet-monitor return an HTTP status of 500 on the blocked requests.
All HTTP requests are blocked by this when running dotnet-monitor in 'connect' mode because all of the requests enumerate the diagnosable processes, which invokes the ProcessInfo command on each process.

The "dotnet run" process (process 39) is still running however the child process that it spawns to run the built application is zombied (the Z in the STAT column above means it exited but the parent process hasn't observed the exit, so the process block is't cleaned up yet).

If I run it with nohup (e.g. nohup dotnet run &), then everything works correctly: the child process runs (and if a web app, responds to http requests) and dotnet-monitor is able to enumerate all of the processes.


Feat: extend test coverage for halo2-base

Does test-case allow you to do e.g., 100 tests of the same function with random inputs?

Replaced by https://github.com/axiom-crypto/halo2-lib/pull/35


Design decision - what to show for all occurrences of a value in our texts?

@arsimr16 , @NinaCengic , @zeljkasalopek , @azugic - guys, there is a design decision with which I need your help. We will have a URL showing all occurrences for a value - for example, all x mentions of the place y in our corpus. The occurrences are first shown as a number, let's say 7. The number is a link, and when we follow the link, we see the list of segments where the place is mentioned (a, b, c... all seven of them). The question is: what else to show, beside the CITE URN for places a, b, c... and the CTS URN for the segment? What would be useful? Once we have the CITE URN for the segment, we have access to everything, to all notes linked to the URN - what would you like, what would you need to see? The context, the periods, the word forms, the morphology, the lemmata, the standard place names...? Ideas are welcome!

I think lemmata and context would be useful.
~Alex
On Wed, Dec 7, 2016 at 5:16 PM, Neven Jovanović<EMAIL_ADDRESS>wrote:

@arsimr16 https://github.com/arsimr16 , @NinaCengic
https://github.com/NinaCengic , @zeljkasalopek
https://github.com/zeljkasalopek , @azugic https://github.com/azugic

guys, there is a design decision with which I need your help. We will
have a URL showing all occurrences for a value - for example, all x
mentions of the place y in our corpus. The occurrences are first shown as a
number, let's say 7. The number is a link, and when we follow the link, we
see the list of segments where the place is mentioned (a, b, c... all seven
of them). The question is: what else to show, beside the CITE URN for
places a, b, c... and the CTS URN for the segment? What would be useful?
Once we have the CITE URN for the segment, we have access to everything,
to all notes linked to the URN - what would you like, what would you need
to see? The context, the periods, the word forms, the morphology, the
lemmata, the standard place names...? Ideas are welcome!

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nevenjovanovic/croala-pelagios/issues/49, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFTY4es6Vng02nq3OGMgFnDZrCSY3bhEks5rFtvMgaJpZM4LGyGg
.

A draft is here: croala.ffzg.unizg.hr/basex/cp-loci-cite/urn:cite:croala:loci.locid26446 (for the CITE URN rn:cite:croala:loci.locid26446), accessible (from one direction) through a list such as croala.ffzg.unizg.hr/basex/cp-loci-id/urn:cts:croala:bunic02.croala1761880.croala-lat2w.
I will add the lemmata and context now.

In the light of the fact that we are doing this as a project under umbrella
of linking
places, I think that this time we could maybe choose to show the standard
place names.
Otherwise, lemmata are a standard, of course.
2016-12-07 18:33 GMT+01:00 Neven Jovanović<EMAIL_ADDRESS>
A draft is here: croala.ffzg.unizg.hr/basex/cp-loci-cite/urn:cite:croala:
loci.locid26446 (for the CITE URN rn:cite:croala:loci.locid26446),
accessible (from one direction) through a list such as
croala.ffzg.unizg.hr/basex/cp-loci-id/urn:cts:croala:
bunic02.croala1761880.croala-lat2w.
I will add the lemmata and context now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nevenjovanovic/croala-pelagios/issues/49#issuecomment-265514750,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALfJXmRkUI76v3jBRX07VvUeete2Rq5Rks5rFu3igaJpZM4LGyGg
.

@NinaCengic and @arsimr16 - I have followed your suggestions. I would like to somehow squeeze the periods in too, but the space is cramped already; check out now the URL listed above. I think I will use the CITE URN ana as the hyperlink behind the CTS URN (though this is a bit inconsistent).

Now it looks rather crowded, but first, it's not bad per se.
If we look how to 'improve' it, or modify it, I would prefer to drop the
context. It stands out a bit as
a different kind of information here.
I would try and see how it worked with just cite urn, cts urn, lemma, place
name, time period and a link to the context, morphology, and the word forms
(a sort of first-hand, basic information first, and then links to more
detailed explanations).
2016-12-07 20:17 GMT+01:00 Neven Jovanović<EMAIL_ADDRESS>
@NinaCengic https://github.com/NinaCengic and @arsimr16
https://github.com/arsimr16 - I have followed your suggestions. I would
like to somehow squeeze the periods in too, but the space is cramped
already; check out now the URL listed above. I think I will use the CITE
URN ana as the hyperlink behind the CTS URN (though this is a bit
inconsistent).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nevenjovanovic/croala-pelagios/issues/49#issuecomment-265544105,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALfJXsXvagfDvfG4v3vH3FdkMdLRrJC_ks5rFwZJgaJpZM4LGyGg
.

@NinaCengic - the period field is added. It is indeed a bit crowded now, but to me the context is quite useful, both for control and as an argument for our comments.

Great!
If the context is useful as an argument for our comments, that we say, no,
it's not crowded,
we just go along the nije-beg-cicija road.
But, if we want to have it like that, is it possible to 'hide' cts and cite
somewhere, behind a link of a sort,
like CTS URN and CITE URN?
As the same type of phenomena class.
I was now considering the column 'form', if we could do something about it.
It seems a bit redundant if we present it together with the context...
2016-12-07 21:23 GMT+01:00 Neven Jovanović<EMAIL_ADDRESS>
@NinaCengic https://github.com/NinaCengic - the period field is added.
It is indeed a bit crowded now, but to me the context is quite useful, both
for control and as an argument for our comments.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nevenjovanovic/croala-pelagios/issues/49#issuecomment-265562683,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALfJXjQVeczKay42RaDJitUdB3BY0v5Gks5rFxW9gaJpZM4LGyGg
.

I wouldn't add any more fields. I would leave both "form" and "lemma" as they sometimes differ (one or more words). I think the "context" is useful, and "period" should definitely stay as it is one of the things we are trying to draw attention to.
I agree with Nina, it would be great if the cts urn and/ or the annotation creator columns could be more narrow. I don't have much experience with this, but is it possible the cts urn to be for example in  three rows instead of two? As for annotation creator, if we don't have to do it one by one, maybe we could put just "orcid" (or initials) - an expression that links to the specific orcid page...
The page to which the cts urn leads, with all the annotations, looks great!
Unfortunately, I think I will not be able to look at this more carefully again tomorrow before the meeting. See you then!

Considering our Locus occurit in CroaLa page this morning, I think that we
could remove period.
I am looking forward to seeing you in the afternoon!
2016-12-07 23:47 GMT+01:00 zeljkasalopek<EMAIL_ADDRESS>
I wouldn't add any more fields. I would leave both "form" and "lemma" as
they sometimes differ (one or more words). I think the "context" is useful,
and "period" should definitely stay as it is one of the things we are
trying to draw attention to.
I agree with Nina, it would be great if the cts urn and/ or the annotation
creator columns could be more narrow. I don't have much experience with
this, but is it possible the cts urn to be for example in three rows
instead of two? As for annotation creator, if we don't have to do it one by
one, maybe we could put just "orcid" (or initials) - an expression that
links to the specific orcid page...
The page to which the cts urn leads, with all the annotations, looks great!
Unfortunately, I think I will not be able to look at this more carefully
again tomorrow before the meeting. See you then!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nevenjovanovic/croala-pelagios/issues/49#issuecomment-265598393,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALfJXtP0Yj18LG9aB6otuGbEpeoytsu2ks5rFzeUgaJpZM4LGyGg
.


feat: Add chunking mechanism to push_data, cleanup TODOs

Adds chunking to push_data, add an integration test for it, cleanup TODOs (issues have been created instead)

Just one test is failing @jirimoravcik

Just one test is failing @jirimoravcik

Yeah, I know, looking into it


feat: Add support for Darwin ARM64 static

Because @ruudk mentioned the Darwin ARM64 compiled version (https://github.com/jolicode/castor/pull/353#issuecomment-2027095462) I had a quick look and saw almost everything was here to make it possible so continued to have a look.
This used to fail because of an issue in static-php-cli (https://github.com/crazywhalecc/static-php-cli/pull/391) so be sure to use the latest nightly.
Also, this will probably fail for now in the CI because of the XZ issue (but I managed to make it work locally by commenting out XZ download in SPC) but I prefer to open the PR so that I don't forget about this work :)

@lyrixx done.
but beware it looks like the CI is not actively building the static binaries outside of the release so this WILL break releasing (see https://github.com/tucksaun/castor/actions/runs/8510887723/job/23309432207)

@lyrixx thanks for finishing it 👍


expect(locator) not supported?

I m trying to use the (locator.1)or_(locator) like in the example here
https://playwright.dev/python/docs/api/class-locator#locator-or i.e. with Locator Assertions https://playwright.dev/python/docs/api/class-locatorassertions
but name 'expect' is not defined ??

Have you imported it? From the link you shared:
from playwright.async_api import Page, expect

Have you imported it? From the link you shared:
from playwright.async_api import Page, expect

f, duh, thanks
intellisense in vscode wasnt seeing it - It was seeing Expect


Functional Debt: PDF for Orders and Applications

The following PDFs for Features and applications need to be completed for Release on 23rd Sep:
Order for Hearings:

Schedule any hearing
Assign new hearing date after rescheduling
Approval/Rejection of Reschedule Request
Order for Approval/Rejection of Check out (Emergency Reschedule) request
Order for Acceptance/Rejection of Voluntary Document Submission
Order for Mandatory Submissions and Responses
Order for Bail
Order of Issue for Summons
Orders for Issue of Notices
Reschedule Request
Check out request (Emergency Reschedule Request)
Extension of Submission Deadline (Judge ordered doc submisison)
Production of Documents
Generic Application
Generic order

Templates here:
https://docs.google.com/document/d/1OnvQsq0lITOn4KN8FMMcAWLRHfH4bA_LTbTHHmUScSk/edit

@ashishbeehyv  and I have performed 2/3 rounds of dev testing and we can push this feature into QA testing @Ramu-kandimalla
But before that, @Taherabharmal  can we please freeze the order/application templates to avoid any descripencies. We're having Reschedule Hearing Request, Production of Documents Application and rest of the application types are pointed toward generic application. Please let us know if anything is missing.
@rajeshcherukumalli  Please let us know about these descripancies, since the final templates might be a little different than one's we've developed.

Why are these pointing towards generic templates?

Initially we were asked to create these three templates on priority and use generic application templates for all the remaining
it was in the sheet shared by @subhashini-egov
https://docs.google.com/spreadsheets/d/1dFyIJS9l0wR6BoORelZd8BCYWOIW1kisdPbL4O3Uo9Q/edit?gid=1784780428#gid=1784780428
c.c. @Ramu-kandimalla

Hi @Bhuvanyu  Please verify and confirm

HI @Ramu-kandimalla,
All Orders PDF mentioned in the ticket are working fine in DEV,
Will close the ticket after testing in UAT.


Missing Property Generation For Type object

When generating the Http Client for the below W model, Properties X and Y are generated while Z is not. The code snippet below is generated from Swashbuckle's swagger.json. Is there a correct way to handle this?
      "W": {
        "type": "object",
        "properties": {
          "X": {
            "type": "string",
            "nullable": true
          },
          "Y": {
            "type": "string",
            "format": "uuid"
          },
          "Z": {
            "nullable": true
          }
    }

Hi @JohnDebono
Thanks for using kiota and for reaching out.
In your example, is nullable: true the only information about Z? if so this is by design, this property doesn't have enough type information to be projected to the generated model.
However, if something comes back from the service, it'll be in the additional properties of the model.
Let us know if you have further questions.

Thanks @baywet worked with the Additional Data


Cluster label fails to propagate when changing autoscaling

When making a change to autoscaling configuration such as changing {min,max}_node_count, it seems that the annotations are not updated in the machineDeployment section of the cluster (and since we also manually update the machine deployments after because propagation doesn't happen yet, it doen't go into the actual machine deployment).
It also leaves the node group in a UPDATE_IN_PROGRESS state.

*_node_count can be updated by either cluster_update or ng_update actions. cluster_update action has not be implemented in the current mcapi driver. So i assume this error can be reproducible by ng_update action.

When we create a cluster autoscaling-enabled, max_node_count and min_node_count labels are not set in the nodegroup correctly. It is just considered as labels in ng.
ubuntu@mcapi-large:~$ o coe nodegroup show scale default-worker -f json
{
  "uuid": "c61acc4b-41f6-4ea9-834d-e0e04914a96b",
  "name": "default-worker",
  "cluster_id": "4e1910e1-35d1-4d99-913d-2ee5765156b2",
  "project_id": "2ffc6c32d8794e3289a34ca890c089f9",
  "docker_volume_size": null,
  "labels": {
    "auto_scaling_enabled": "true",
    "max_node_count": "2"
  },
  "labels_overridden": {},
  "labels_skipped": {},
  "labels_added": {},
  "flavor_id": "m1.medium",
  "image_id": "ef107f29-8f26-474e-8f5f-80d269c7d2cd",
  "node_addresses": [],
  "node_count": 1,
  "role": "worker",
  "max_node_count": null,
  "min_node_count": 0,
  "is_default": true,
  "stack_id": "kube-carvi",
  "status": "CREATE_COMPLETE",
  "status_reason": null
}

Seems this is an upstream issue? I can see that create_cluster handler just sets min_node_count for ng.
https://opendev.org/openstack/magnum/src/commit/b578bd8a78d890ad32aeb6066cc22ece165f54e2/magnum/conductor/handlers/cluster_conductor.py#L61-L63
https://opendev.org/openstack/magnum/src/commit/b578bd8a78d890ad32aeb6066cc22ece165f54e2/magnum/conductor/utils.py#L170-L196

btw, i was able to reproduce this issue. The reason is;
when the ng is updated, both the cluster status and ng status are updated as UPDATE_IN_PROGRESS. Then driver.update_nodegroup() is executed.
At the same time, the periodic task for sync cluster status will trigger the driver.update_cluster_status() because the cluster status is in IN_PROGRESS.
Before ng update handler is finished, the driver.update_cluster_status() (which runs update_nodegroup_status inside it) is finished. i.e. the cluster status and ng status are set as UPDATE_COMPLETED before the ng udpate handler sets the ng status as IN_PROGRESS.
Once the cluster status is UPDATE_COMPLETED, this cluster is skipped from the periodic task for sync cluster. i.e. driver.update_cluster_status() will never be executed even if the ng status is UPDATE_IN_PROGRESS.


feat: add new "Get Card" CTA button to profile

Description
This PR replaces the old Get Card CTA button with a new one

What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[x] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
closes #1829

Mobile & Desktop Screenshots/Recordings
Mobile

Desktop

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

I find it hard to decipher what in the icon. Can we see the white as a comparison?

Added tooltip to the CTA

Testing out the white variant of the CTA and i feel the gradient version is bit better visually

cc: @bdougie @isabensusan

pushed a commit so you can test @bdougie


test_implicit_cast crashes gcc 4.7.3

Hi although this is not a bug in the code itself, gcc 4.7.3 crashes with the building test_implicit_cast:
[100%] Building CXX object test/CMakeFiles/test_implicit_cast.dir/test_implicit_cast.cpp.o
In file included from /home/nhoeft/projects/luabind/luabind/detail/policy.hpp:29:0,
                 from /home/nhoeft/projects/luabind/luabind/detail/convert_to_lua.hpp:28,
                 from /home/nhoeft/projects/luabind/luabind/detail/object.hpp:40,
                 from /home/nhoeft/projects/luabind/luabind/make_function.hpp:9,
                 from /home/nhoeft/projects/luabind/luabind/function.hpp:8,
                 from /home/nhoeft/projects/luabind/luabind/class.hpp:81,
                 from /home/nhoeft/projects/luabind/luabind/luabind.hpp:28,
                 from /home/nhoeft/projects/luabind/test/test_implicit_cast.cpp:24:
/home/nhoeft/projects/luabind/luabind/detail/decorate_type.hpp: In instantiation of ‘static int luabind::detail::match_struct<StackIndexList, SignatureList, End, Index>::match(lua_State*, TupleType&) [with TupleType = std::tuple<luabind::default_converter<LBENUM_t, void> >; StackIndexList = meta::index_list<1u>; SignatureList = meta::type_list<LBENUM_t, LBENUM_t>; unsigned int End = 2u; unsigned int Index = 1u; lua_State = lua_State]’:
/home/nhoeft/projects/luabind/luabind/detail/call.hpp:489:6:   required from ‘static int luabind::detail::invoke_struct<PolicyList, Signature, F>::invoke(lua_State*, const luabind::detail::function_object&, luabind::detail::invoke_context&, F&) [with PolicyList = meta::type_list<>; Signature = meta::type_list<LBENUM_t, LBENUM_t>; F = LBENUM_t (*)(LBENUM_t); lua_State = lua_State]’
/home/nhoeft/projects/luabind/luabind/detail/call.hpp:532:76:   required from ‘int luabind::detail::invoke(lua_State*, const luabind::detail::function_object&, luabind::detail::invoke_context&, F&) [with PolicyList = meta::type_list<>; Signature = meta::type_list<LBENUM_t, LBENUM_t>; F = LBENUM_t (*)(LBENUM_t); lua_State = lua_State]’
/home/nhoeft/projects/luabind/luabind/make_function.hpp:58:6:   required from ‘static bool luabind::detail::function_object_impl<F, Signature, InjectorList>::invoke_defer(lua_State*, luabind::detail::function_object_impl<F, Signature, InjectorList>*, luabind::detail::invoke_context&, F&, int&) [with F = LBENUM_t (*)(LBENUM_t); Signature = meta::type_list<LBENUM_t, LBENUM_t>; InjectorList = meta::type_list<>; lua_State = lua_State; luabind::detail::function_object_impl<F, Signature, InjectorList> = luabind::detail::function_object_impl<LBENUM_t (*)(LBENUM_t), meta::type_list<LBENUM_t, LBENUM_t>, meta::type_list<> >]’
/home/nhoeft/projects/luabind/luabind/make_function.hpp:79:72:   required from ‘static int luabind::detail::function_object_impl<F, Signature, InjectorList>::entry_point(lua_State*) [with F = LBENUM_t (*)(LBENUM_t); Signature = meta::type_list<LBENUM_t, LBENUM_t>; InjectorList = meta::type_list<>; lua_State = lua_State]’
/home/nhoeft/projects/luabind/luabind/make_function.hpp:33:41:   required from ‘luabind::detail::function_object_impl<F, Signature, InjectorList>::function_object_impl(F) [with F = LBENUM_t (*)(LBENUM_t); Signature = meta::type_list<LBENUM_t, LBENUM_t>; InjectorList = meta::type_list<>]’
/home/nhoeft/projects/luabind/luabind/make_function.hpp:107:161:   required from ‘luabind::adl::object luabind::make_function(lua_State*, F, meta::type_list<Types2 ...>, meta::type_list<TailTypes ...>) [with F = LBENUM_t (*)(LBENUM_t); SignatureElements = {LBENUM_t, LBENUM_t}; PolicyInjectors = {}; lua_State = lua_State]’
/home/nhoeft/projects/luabind/luabind/make_function.hpp:113:119:   required from ‘luabind::adl::object luabind::make_function(lua_State*, F, meta::type_list<Types2 ...>) [with F = LBENUM_t (*)(LBENUM_t); PolicyInjectors = {}; lua_State = lua_State]’
/home/nhoeft/projects/luabind/luabind/function.hpp:27:61:   required from ‘void luabind::detail::function_registration<F, PolicyInjectors>::register_(lua_State*) const [with F = LBENUM_t (*)(LBENUM_t); PolicyInjectors = meta::type_list<>; lua_State = lua_State]’
/home/nhoeft/projects/luabind/test/test_implicit_cast.cpp:139:1:   required from here
/home/nhoeft/projects/luabind/luabind/detail/decorate_type.hpp:76:81: internal compiler error: Segmentation fault

The rest builds fine. Any idea?

Closing this as it's getting outdated. Hopefully you were in a position to bump the compiler version.


Unable to add multiple predefined tasks

With chaingang I am unable to add multiple predefined tasks, cause when no name is given the default task name is generated by task.toString() which is always equal but the tasks are initialized with different data.
I thought when no name is given the naming feature is ignored, but this is not the case. Is there a way to disable this feature? Cause I simply want a queue with worker support.

Example: https://gist.github.com/1446757
I think to predefine tasks is a common use case and add them to the same queue.

This is why you set names.  Set the name to a value that identifies that data.  If something tries to queue up the same job, it'll wait on the existing one.
There's no way to disable the feature.  Tested patches are welcome.


This was causing issues with profile publisher

This should fix the profile publisher issues

@halloffame what problem is this trying to fix?

I need to talk to @rift137 before merging this one.

We are not sure exactly why that was causing a problem, but it is not really needed and removing it fixes the problem.
Just for reference, this was causing profile publisher's tests to fail.


Cache inference

I spoke with @aviatesk and I think this is fine, but also sadly this did not resolve the issue”we spend like 50 mins in absint” problems


Strip option with template

I just try to set strip option to true and does not work with template
    <!-- build:template:dist
        some code
    /build -->

And when run another target 'some code' stay at document.

 I don't see why this issue is closed as its very much an issue. You currently cannot have a template that is removed when a different target is used. I've tried combining build:remove and build:template with strip set to both true and false and no combination works.

I agree with @nathggns and @luiscoms
Well I think the strip-option acts confusing since the comments work diffenrently when you use a template:

Remove: <!-- build:remove:dist --> (comment closed)
Template: <!-- build:template:dist (comment not closed)
I also ran into the trap assuming "strip: true" would strip the whole block as the comment is not closed. It seems this behaviour is intended. So maybe you can create another option "stripTemplate" that strips the template-block as well.

First of all, template comments are not closed because lodash template syntax is not valid html and will break your html from rendering and that is not what you want. Is not an issue nor a bug having the template block not closed.
I will take a further look into the fact that template blocks are not being stripped when the current env doesn't match the block and strip is true.

in fact, we can't just strip template comments. because of the same reason they are not closed in the first place.
if the processed file for the matching target didn't meant to process the template and we strip the comments (imagine we could, given the fact that the comment opened) the template will break your html.
so what I suggest, is you could use other blocks to achieve the same thing.
if there are no further questions may I close the issue? :)
any ideas from your side guys?
thx

After I thought about the problem and your argumentation for a while, I agree with you. I played around a bit and yeah, we just can't strip the whole template comments. This would not work the way processhtml (and the strip-option) is meant to be.
But this leads me to rethink my deployment-process. Actually, I can't have a block that is hidden/commented in my development-environment (where processhtml won't process any file). e.g. tracking-code that I want to be added when my HTML is being deployed to the live-environment (although I know I can exclude my pageviews otherwise - this is only an example).
So I might search for another solution. You may close this issue.

Just one last comment, you could include your tracking-code in live-environment as follows:
template.html
<html>
  <head>
    <!-- build:include:prod tracking-code.html -->
    <!-- /build -->
  </head>
</html>

with following grunt options:
      prod: {
        options: {
          strip: true
        },
        files: [{
          'dist/index.html': ['template.html']
        }]
      },
      dev: {
        options: {
          strip: true
        },
        files: [{
          'dist/index.html': ['template.html']
        }]
      },

this will include you tracking-code in production, and remove the include block in dev to keep it clean

Thanks a lot!


Added missing colon

Hello,
I added the missing colon
Whitespace was removed accidentally (my VSCode settings)

Thank you for your contribution !


Add ARIA attributes

https://www.w3.org/TR/wai-aria-practices-1.1/#intro

[x] add aria-attributes
[ ] add keyboard navigation

Nit: aria-orientation: undefined is not a valid ARIA attribute so it gets flagged on tests like Lighthouse. See my fork for fix to this and #98: https://github.com/danbovey/react-contextmenu/commit/d6a927bb36ba7afce8ea1f0032d175b98ce2bc6c

@danbovey can you open a PR with ARIA fix?


'Setting the timezone' support

Currently there seems no way to set the time zone, may need the support of the tzdata library.
https://wiki.alpinelinux.org/wiki/Setting_the_timezone
https://bugs.alpinelinux.org/issues/5543
https://forums.docker.com/t/synchronize-timezone-from-host-to-container/39116

host machine timezone: America/Los_Angeles
docker container timezone needs is:Asia/Shanghai
usage: -v /custom/timezone:/etc/timezone:ro -v /custom/localtime:/etc/localtime:ro -e TZ=Asia/Shanghai
for this image does not work, but centos, ubuntu is working

The original alpine mirror is also working.
Dockerfile
FROM alpine:3.6

docker build -t my/test:1 .
docker run --rm --name test -v /custome/localtime:/etc/localtime:ro -it my/test:1
'/custome/localtime' file you can find in linux '/usr/share/zoneinfo/Asia/Shanghai' directory.

This is what I did to get timezone in image using this as base image -
Add apk add --update tzdata && \ here
If TZ value is set from ENV, then add that to /etc/timezone like this
While running the container, you can pass-in TZ environment variable -e TZ=Asia/Shanghai.
This is working for me.

@manikmagar That second link is broken. Got one that works?

@jazzepi here is the new link -   https://github.com/javastreets/mule-docker-image/blob/master/Dockerfile#L28
Looks like I renamed my repo later 🙂

On Oct 17, 2018 at 5:33 AM,  <Michael Pinnegar (mailto:notifications@github.com)>  wrote:
@manikmagar (https://github.com/manikmagar)  That second link is broken. Got one that works?
—
You are receiving this because you were mentioned.
Reply to this email directly,   view it on GitHub (https://github.com/anapsix/docker-alpine-java/issues/58#issuecomment-430610022), or   mute the thread (https://github.com/notifications/unsubscribe-auth/AA1i5kfdaHRFhkp3MAFBj1gJff1OwtNSks5ulyO2gaJpZM4Rxs5Q).


Warning in console when picking optional color not previously set.

Low priority issue I presume, but when a new color is added, and only the base color is set, if you select an optional color input(such as hover or active) and then click in the color picker, a react warning appears in the console:

Tidy up uncontrolled input required (?)

Ah yes, I noticed this too, I think this is because the value of the input is originally blank. I think considering it's a warning it's ok for now, but definitely something we want to fix. Feel free to take a stab if you want @prichter!

Comments in EditColorModal.tsx indicate this needs backend fix?

Waiting for colorpicker to be reinstated before revisiting this...

Could not replicate this issue with new color pickers in the modal (probably because a default variant is added), so closing.


Synching colorranges on the same axis?

I've just noticed that it is a bit problematic to use a numeric color attributes in separate plots in the same axis.
For example, the following:
using CairoMakie

f = Figure()
ax = Axis(f[1, 1])
x1, y1, z1 = 1:10, 1:10, rand(10)
x2, y2, z2 = 11:20, 1:10, rand(10) .+ 100
scatter!(ax, x1, y1, color = z1)
scatter!(ax, x2, y2, color = z2)
f

generates this figure

whereas IMO it should look something like this:

That is to say, I imagine that the "colorrange" transform should be a property of the axis, pretty much like the x and y limits.
I'm not sure what is the best fix is in practice, maybe there could be some linkcolors! analogous to linkaxes!?

That sure is confusing, although I wouldn't want to restrict one axis to one colormap. People want to do weird things with multiple scales sometimes. linkcolors would then have to work on the plot level, splicing the same observable in as the colorrange maybe?

That sure is confusing, although I wouldn't want to restrict one axis to one colormap. People want to do weird things with multiple scales sometimes.

Good point. I personally think that that's a bit of a questionable practice, but it should be up to the user to decide.

linkcolors would then have to work on the plot level, splicing the same observable in as the colorrange maybe?

Yes, I imagine linkcolors would take either a collection of plots (say linkcolors(plots::AbstractPlot...)) or a collection of axes (say linkcolors(axs::Axis...)), in which case, it would consider all plots with a colorrange within those axes. Then it would just sync all colorranges to be the "shared range".
I'm not sure if it should actually replace the observables to be the same observable, or sync those that exist already. I'm not completely up to speed with the latest versions of Observables so I'm not sure how easy it is to sync a bunch of observables without triggering a large amount of callbacks all the time.


w_dir_main ignored in python csearch

hello,
I am trying to change the work directory of csearch  changing the w_dir_main argument. But it seems that this variable is overwritten in load_variables
When I run this:

from aqme.csearch import csearch
import os

thispath = os.getcwd()
sdf_path = thispath+'/quinine_folder'
wrk_path = thispath+'/quinine_wrk'

smiles = 'COC1=CC2=C(C=CN=C2C=C1)[C@H]([C@@H]3C[C@@H]4CCN3C[C@@H]4C=C)O'
name='quinine'
program='rdkit'
csearch(
        w_dir_main=wrk_path,
        destination=sdf_path,
        smi=smiles,
        name=name,
        program=program)

I expect that the .dat and .csv files are written in wrk_path, but the files are dumped in os.getcwd()
I think the w_dir_main is overwritten in here
I will send a pull request, feel free to reject if it is not useful.

Hi juanvi, I just made pull request.
Quick question:
What would be the best way to suppress the creation of the DAT file or creating in a predetermined folder?

I'd say that we could include a verbose option to skip the creation of any DAT/CSV file.
Since the pull request is failing CircleCI, can you redirect the request to jv_branch instead of master?

Hi @alvarovm ,
I tried to edit your PR, but the pytests in CircleCI encountered issues when modifying w_dir_main. Instead, I added a verbose option that should do the trick. If you include verbose=False in any of your AQME runs, it will prevent the creation of DAT or CSV files.
To install this new version, please download the code from the jv_branch branch and execute the command "pip install ." in your terminal. This version is safe to use as it's passing all the pytests.
Thanks for the feedback!


[UBS-User. Bonus notification] The notification about recieved bonuses is not displayed the payment sum received from the order form

Environment: all.
Reproducible: always.
**Build found:**21.06.22
Preconditions

Create a user and make an order that is paid via Fondy (e.g. 'Безпечні відходи' in quantity of 4 packages (each package is per 250UAH), total amount - 1000UAH).
User is Logged in to GreenCity as an Administrator: https://ita-social-projects.github.io/GreenCityClient/#/ubs

Steps to reproduce

Open 'Замовлення' tab and find order mentioned in preconditions
Open the order found in step 1 and change order`s status to the 'Привезе сам'
Update the page
Change order`s status to the 'Виконано'
Make the amount of bags less, so that the paid sum is higher than it became as the result
Transfer the rest money to bonuses
Click on the 'Зберегти' button
Log in to GreenCity as a User (https://ita-social-projects.github.io/GreenCityClient/#/ubs Use credentials as for Client mentioned in preconditions)
Open 'Повідомлення' tab and find order mentioned in preconditions
Open the order found in step 9

Actual result

The payment sum is not  received from the order form

Expected result

The payment sum is received from the order form

User story and test case links
E.g.: "User story #2796
Test case"