Set preference to _primary when searching control-center index

*Issue *
https://github.com/opensearch-project/index-management/issues/833
Description of changes:
This change is to set the query parameter preference to _primary when searching control-center index, because when indexing new documents in this index, we use RefreshPolicy.IMMEDIATE to force a refresh after the document is indexed which aims to make the new document visible immediately, but if segment replication is enabled in the control-center index, read-after-write is not guaranteed, so we need to search the primary shard only to make the document can be seen as soon as possible.
The search load on the control-center index is pretty low, it will not have some performance issue if we only search the primary shard.
CheckList:

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

No harm adding this. But may not be necessary IMO.
The reasoning is the time between LRON config indexed, and needed by sending notification would be much longer than the time for segRep to sync.


Integrate FlatImages better into desktop

Current desktop integration only covers adding .desktop file & icon to appear into menu.
This is a nice start.
But what I see that is currently lacking is:

support for opening links to redirect to your default web-browser & other similar scenarios (using xdg-mime)
option to set FlatImage application to be the default (also using xdg-mime)
support for MPRIS (media status notification)
etc.

I believe that equivalent to these above are portals in flatpak.
If those above can be achieved when FlatImage is in portable mode, that would be great!

Hi @fiftydinar , thanks for the suggestion! I'll give updates to these features here as they are integrated.

Update on this:

A portal implementation is now available
xdg-open will now redirect to the host, i.e., xdg-open www.google.com will use the default browser of the host
xdg-mime will now redirect to the host. More work is required on this I think.


find my phone, or tracker from api?

Have you looked into a way to find the phone, or tile tracker via the API?
I'd like to find my phone, similar to what you can do from their website.

The phone should be included in the API calls already developed for this library. Are you having trouble?

Closing due to inactivity.


jellyfin: 10.3.3 -> 10.3.4

Motivation for this change
Release notes: https://github.com/jellyfin/jellyfin/releases/tag/v10.3.4
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test jellyfin
@GrahamcOfBorg build jellyfin

@minijackson Thanks! :)


Using 'ssh for straight-vc-git-default-protocol raises an odd error

I don't believe this is actually an error in straight.el but I encountered it as a consequence so I'm wondering if you've already seen the answer. Feel free to simply close this without comment if the answer isn't obvious.
Per our earlier discussion (#441), I switched to using ssh as the default protocol for git. That allowed me to get packages from my non-github private repo.
In the course of exploring a broader re-write of my Emacs init file (what else are holidays for?), I had occasion to "start over" in a clean home directory. During the bootstrap process, straight.el comes to a github repo it wants to clone and falls over.
Cloning into '/Users/ndw/testhome/.emacs.d/straight/repos/git-timemachine'...
ssh_askpass: exec(/usr/X11R6/bin/ssh-askpass): No such file or directory
Host key verification failed.
fatal: Could not read from remote repository.

I'm running on MacOS and there's no /usr/X11R6/bin/ssh-askpass in sight. There is /usr/local/bin/pinentry-mac which I imagine would do the trick. So I go looking for where the X11R6 reference occurs. I expect to find a variable I can set to the correct pinentry program and all will be well.

It's not in straight.el
It's not under ~/.emacs.d
It's not AFAICT in any of the lisp files in /Applications/Emacs.app/Contents/Resources/lisp (and yes, I did unzip them before grepping 😄 )
And just FYI, I did start Emacs from the command line and the environment does seem to have the ssh-agent settings, but apparently it still wanted to ask.

I nudged the system to make sure it hadn't simply timed out, but no, ssh and friends were happy to go along with the agent values without asking me to refresh my password.
The workaround was simply to change the default protocol back to https, run the initial bootstrap, and then change it back to ssh. That'll be fine until the next time it tries to clone a new package, I guess, but hopefully I'll remember what to do.
Anyway, I'm just wondering if my ignorance is showing?

See #334. You should be able to work around the problem more easily by configuring ssh-agent and then running ssh-add in a terminal to get your passphrase cached. I've tried a couple of times to solve the issue, but I keep running into weirdness and race conditions in trying to handle process filters synchronously.

Alternatively, I believe you should be able to install an ssh-askpass program and it will prompt you graphically. The problem is that straight.el doesn't currently know how to attach a tty to the processes it starts, like Magit does.

I have ssh-agent running and point 4 in my initial report was an attempt to demonstrate that I've started Emacs with an environment that includes the relevant environment variables. I can see if the brew version of ssh-askpass will work graphically.

I have ssh-agent running and point 4 in my initial report was an attempt to demonstrate that I've started Emacs with an environment that includes the relevant environment variables. I can see if the brew version of ssh-askpass will work graphically.
(Nope. Maybe if I could change /usr/X11R6/bin/... somewhere...)

On Linux, I can install an ssh-askpass program and then export the SSH_ASKPASS variable to point at the full path of the executable, and then I get prompted graphically. That said, the usual way to work around this problem is by configuring ssh-agent correctly. I do not know what might be going wrong for it on your system. If you have the agent working from the command line, I assume you have AddKeysToAgent yes in your ~/.ssh/config already. Do the relevant variables show up from inside of Emacs using M-x getenv?
SSH_AGENT_PID=1354
SSH_AUTH_SOCK=/tmp/ssh-YfpfmKRyaOOj/agent.1352

Perhaps you have some package like exec-path-from-shell which is messing with your environment. Alternatively, sometimes upgrades and such can mess up the agent state, in which case restarting might help. You could also check to see what arguments ssh-agent is being started with. Maybe it's something you don't expect. I personally use ssh-agent -t 86400 in my shell init.

Ok. Thanks for the suggestions; it's clearly not a straight.el problem, but I appreciate the advice.

Well, it definitely is a straight.el problem in the sense that #334 needs to be solved, but yeah the environmental stuff I can't do much about here.

FYI: I found an easy workaround. If I add a new package, I start emacs in a shell without a window manager:
/Applications/Emacs.app/Contents/MacOS/Emacs -nw

And then it uses ssh-agent and ssh works fine. I don't know why -nw is different in this case, but it is.


Update ImGui.NET to 1.87.3

Summary of the PR
Updates all ImGui extensions to the latest revision of ImGui.NET
Further Comments
Also puts all of them in sync, as before one of the extensions was a different version to the other 2

Recommend revert, as it stands ImGui has (unintentionally) removed Mac support which is a bit of a regression.

There'll be a separate PR to rollback all the versions to one common known good version


fixes the broken fabric8:resource goal

lgtm [merge]

Argh, sorry. A classical one :-) sitting in the train to the first season's
match, will be back late and could still fix stuff. Just mail me.
Sorry and thanks.
FuseSource CI<EMAIL_ADDRESS>schrieb am Fr., 12. Aug. 2016,
16:38:

Merged #259 https://github.com/fabric8io/fabric8-maven-plugin/pull/259.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fabric8io/fabric8-maven-plugin/pull/259#event-754082240,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGDCJu03u8mEx6spY4_i3HLuqd5GvI1ks5qfIU4gaJpZM4JjJPa
.


Feature: GcsBucket.upload_from_dataframe

Overview

Implements upload_from_dataframe as proposed on #139
Enables the DataFrame persistence on GCS as .csv, .csv.gz, .parquet, .parquet.snappy, .parquet.gz
When the serialization_format is not specified, it defaults to .csv.gz (csv with 'gzip' compression)

Closes:
Upload pandas DataFrame to GoogleCloudStorageBucket #139
Example

import pandas as pd
from prefect_gcp.cloud_storage import GcsBucket

pandas_df = pd.read_csv(url="some_url")

gcs_bucket = GcsBucket.load("my-bucket")
gcs_bucket.upload_from_dataframe(
   df=pandas_df,
   to_path="/path/to/gcs/blob.parquet",
   serialization_format='parquet_snappy'
)

Screenshots

Checklist

[x] References any related issue by including "Closes #" or "Closes ".

If no issue exists and your change is not a small fix, please create an issue first.

[x] Includes tests or only affects documentation.
[x] Passes pre-commit checks.

Run pre-commit install && pre-commit run --all locally for formatting and linting.

[ ] Includes screenshots of documentation updates.

Run mkdocs serve view documentation locally.

[x] Summarizes PR's changes in CHANGELOG.md

Thanks! It's really close, just a couple more thoughts.

We should be all set now, I think. @ahuang11 .
I left some notes about PurePosixPath, but there are just observations.

Thanks so much @iobruno! This will be available in the next release!

Thank YOU!


Cannot unline device from google home

I have had few google assistant project setup on my raspberry pi3. I used to unlink devices and they dissapear from google home app. On the latest sdk I can't do that anymore. As I mess google assistant I have 4 additional devices in GH app.
Thank You.

Same here, got a unamed entry in my list after updating the SDK, which can't be removed.

You should be able to use the devicetool to delete a device of a given id.

I could use the tool and it would work but what I did is thst I have hanged
the google assistant project. Same account but different credential and my
old devices doesn't show up when I use the tool. Only the current one comes
up.
On Mar 6, 2018 18:40, "Nick"<EMAIL_ADDRESS>wrote:

You should be able to use the devicetool to delete a device of a given id.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlesamples/assistant-sdk-python/issues/197#issuecomment-370883449,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ah9Ze47CSrQ_ZqBXpWnW766dQTfNsfPWks5tbteigaJpZM4SeozF
.

Can you try using your previous credentials to delete them?

I would but I have deleted my secret json file in my project page and I
dont know how to restore it.
On Mar 7, 2018 00:11, "Nick"<EMAIL_ADDRESS>wrote:

Can you try using your previous credentials to delete them?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlesamples/assistant-sdk-python/issues/197#issuecomment-370975588,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ah9Ze-XaQHOyDV936mjTrtFP1YzmawpNks5tbyWtgaJpZM4SeozF
.

You can get all of your API keys in the Google Cloud console for your project.

Ok. I'll give it a go. Thank You
On Mar 8, 2018 1:49 AM, "Nick"<EMAIL_ADDRESS>wrote:

You can get all of your API keys in the Google Cloud console for your
project.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlesamples/assistant-sdk-python/issues/197#issuecomment-371350173,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ah9ZewEvmLJX388W0urgN3uGU6ZyfPeUks5tcI5DgaJpZM4SeozF
.


Update node's allocator package to support /31 and /127 networks and IP address exclusion

The PR contains two changes two the node's allocator package:

support for /31 and /127 subnets. Allocator should allocate all addresses for such networks (network + broadcast (ipv4))
support for IP address exclusions on the node level

coverage: 72.203% (+0.1%) from 72.083%
when pulling 7314b4f97248aa9c4f37b6bfe12537c2962aa7ce on ykulazhenkov:pr-update-allocator
into 18fc6a09eb2d71849ad506fa7d1715e6f7bb750d on Mellanox:main.


[FLINK-14699][type] Move ClosureCleaner to flink-core

What is the purpose of the change
ClosureCleaner is currently under flink-java. However, it doesn't stick to flink-java and used in flink-streaming-java. IMHO flink-streaming-java should not base on flink-java( flink-batch-java in fact ). Thus, I propose to move ClosureCleaner to flink-core.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (don't know)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@aljoscha please give this pull request a look :-)

I think we better move the ClosureCleaner to the org.apache.flink.util package. What do you think?

@aljoscha the original purpose of this change is for module dependency issue. If we change the FQN, shall we potentially risk breaking b/w compatibility? IIRC ClosureCleaner once suffer from such issue.

FYI https://issues.apache.org/jira/browse/FLINK-13586

You're right. 😅 Good catch!


added in F to C conversion and option in config.json

I'll be honest, this is the first pull request i've ever done so hopefully this is correct. I've added in a few things

docker-compose.yml example (plan to update README eventually)
config.json has a TemperatureUnits option
main.go has a temp conversion function based on config.json input
adjusted prints depending on temp choice (degF or degC)

also open to critiques.

found some issues where i was converting the api data and also the influx resulting it bad writes to influx. going to resubmit


Create raspi-io-config CLI tool

The postinstall scripts have been getting more complicated as the years have gone by, and raspi-serial configuration is complicated enough that I just skipped it entirely.
Neither of these approaches work all that well anymore though, so instead I want to create an interactive CLI tool that configures it for you. I envision this as being pretty similar to npm init in that it'll ask a bunch of questions and then generate the resulting configs.
This should be especially useful for dealing with issues like serial, where there are multiple possible use cases.

Not gonna do this, since this project is in maintenance mode.

Not gonna do this, since this project is in maintenance mode.


content/join/individuals: Update page to serve as landing page for volunteer recruitment

This basically says how individuals can join, so update it with with
more concrete steps that connect to registration forms and
instructor coordination HackMD.
Review:

confirm the stuff that referrs to registration forms makes sense
confirm this doesn't make the overall text too long or hard to read.

Thank you!


Chore: This and that

About
A few preparations for another patch.

GH-21

Details
Most prominently:

Notebooks: Fix evaluating DEBUG environment variable
Notebooks: Make timeseries_data.ipynb idempotent on ALTER TABLE stmt

Note: This also requires content in the LMS to be updated where there are changes to the HTML files.

I've now made the corresponding update in the LMS so that the page there matches the HTML in the repo here.

Thank you!


writelto writes to B tape only; A tape rsync not permitted

gave these errors (among some others):
rsync: recv_generator: mkdir "/Volumes/17091A/1a_auto-parts_00000_pm0038726" failed: Operation not permitted (1)
  2017-10-03T21:49:11 -  [migratefiles]Error: Running: "rsync -rtvP --log-file=/Users/BLAS2/migrate.log --log-file-format=20171003-214911 '%f' %l '/Volumes/17091A/' /Users/BLAS2/Documents/AIP_TARGET// /Volumes/17091A/" gave an Error Code - 23

I reformatted the tapes and tried again. It worked, but I still don;t know what was causing the problem. The only difference between the tapes was that the A tape had some files that were not in subdirectories.
I will leave this open until it's clearer what happened.


Add bio.tools for unipept

Hi all,
This PR links the tools to its bio.tools entry: https://bio.tools/unipept
It will be useful to get EDAM ontology annotations
Bérénice

Working on it.

Thanks!!!


Can't run npm i -g @adonisjs/cli

Prerequisites
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade to kleur@3 or migrate to 'ansi-colors' if you prefer the old syntax. Visit https://github.com/lukeed/kleur/releases/tag/v3.0.0\ for migration path(s).npm WARN deprecated<EMAIL_ADDRESS>Critical bug fixed in v3.0.1, please upgrade to the latest version.
npm ERR! Unexpected end of JSON input while parsing near '...":">=0.0.4"},"devDepe'
npm ERR!      C:\Users\user\AppData\Roaming\npm-cache_logs\2020-01-04T08_50_18_721Z-debug.log
Delete the above section and the instructions in the sections below before submitting
Package version

Node.js and npm version
λ npm --version
6.13.4
λ node --version
v13.5.0
Sample Code (to reproduce the issue)

BONUS (a sample repo to reproduce the issue)

npm cache clean --force


Prepare for uplift of drop-bounds lint

cc https://github.com/taiki-e/pin-project/issues/272

bors r+
https://github.com/rust-lang/compiler-team/issues/347 is merged


Bump zeroize to a higher version

Hi there!
I'm using rsa with aes-gcm-siv and I've run into a cargo version conflict problem:
Execution failed (exit code 101).
/Users/mryao/.cargo/bin/cargo metadata --verbose --format-version 1 --all-features
stdout :     Updating crates.io index
error: failed to select a version for `zeroize`.
    ... required by package `num-bigint-dig v0.8.1`
    ... which satisfies dependency `num-bigint = "^0.8.1"` of package `rsa v0.7.0-pre`
    ... which satisfies dependency `rsa = "^0.7.0-pre"` of package `crypto-wasm-wxa v0.1.0 (/Users/mryao/IdeaProjects/crypto-wasm-wxa)`
versions that meet the requirements `^1.5` are: 1.5.7, 1.5.6, 1.5.5, 1.5.4, 1.5.3

all possible versions conflict with previously selected packages.

  previously selected package `zeroize v1.0.0`
    ... which satisfies dependency `zeroize = ">=1, <1.4"` of package `aes-gcm-siv v0.10.3`
    ... which satisfies dependency `aes-gcm-siv = "^0.10.3"` of package `crypto-wasm-wxa v0.1.0 (/Users/mryao/IdeaProjects/crypto-wasm-wxa)`

failed to select a version for `zeroize` which could resolve this conflict

It seems rsa is using an outdated version of zeroize, which is different from aes-gcm-siv.
So I'm wondering if we can bump zeroize to a higher 1.5.* version.
Looking forward to your reply!

The problem is num-bigint-dig (a dependency of rsa) requires zeroize 1.5, whereas aes-gcm-siv v0.10.3 requires <1.4. So it's aes-gcm-siv that's out-of-date.
There's a tracking issue here: https://github.com/RustCrypto/utils/issues/723
To get around this you can use aes-gcm-siv v0.11.0-pre.2. We'll have a final release out soon.


Plugin crashes app every time I try to export a page

Steps to reproduce:

Open sketch app
Install/enable "Print Export Plugin"
Navigate to page which you want to export.
In the menu bar, select "Plugins"-> "Print Export"
In the dialog box select select the following options:
— Sketch page per PDF Page
— Export current page)
— Show Artboard Shadow
— Show Artboard Name
— Show Prototyping Links
— Paper Size Standard, US
— Page Size, Tabloid
— Landscape
Click Export
Name new file with system dialog.

Expected Behavior
PDF file is created, & saved to location specified.
Actual Behavior
Sketch App Crashes
This behavior can be consistently reproduced.
On a related note, I suspect that the error has something to do with the size, content, or complexity of my sketch file.  I tried reproducing this error with a new sketch file with only a few empty artboards and the plugin worked as expected.
There is something about my specific file which is causing this problem.  Perhaps it's just too large?
Please prioritize this bug, I cannot use this plugin at all.

@andrewharwood-rev Does it work if you disable 'Show Prototyping Links', as it does for me?

@zedman ,
Yes the plugin does not crash if the "show prototyping links" box is left unchecked (see attachment).
This is really bad for me since I need the prototyping links to be displayed.

I experienced exactly the same as andrewharwood-rev. (Crash after click Export and no crash if uncheck "Show prototyping links")
I want to see the prototyping links in the pdf since it greatly facilitates the discussion in my team.
Many thanks ><"

I have been experiencing this issue.  I am on Version 68 (102490) with MacOS 10.15.6.  Hope that it gets fixed soon.  Or please share a walk around.


Implement task configuration avoidance for avro plugin

This PR supports https://github.com/davidmc24/gradle-avro-plugin/issues/97

@davidmc24 - not quite ready yet. This needs a bit of work to make sure the tests are passing. Some tests will need to be adjusted, but I think there is an issue with the code. I will look into it as soon as I find a bit of time to do so.

Sounds good. Let me know when it’s ready
On Mon, Feb 24, 2020 at 2:06 PM Denis Cabasson<EMAIL_ADDRESS>wrote:

@davidmc24 https://github.com/davidmc24 - not quite ready yet. This
needs a bit of work to make sure the tests are passing. Some tests will
need to be adjusted, but I think there is an issue with the code. I will
look into it as soon as I find a bit of time to do so.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/davidmc24/gradle-avro-plugin/pull/102?email_source=notifications&email_token=AADNKULEF2DPTSB5V3GJKCLREQLC3A5CNFSM4KTHBAQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMZEITI#issuecomment-590496845,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADNKUJ54BBOQIR5GGAZFP3REQLC3ANCNFSM4KTHBAQA
.

--
David M. Carr
<EMAIL_ADDRESS>

Went with a different implementation for #97.  Thanks for the contribution.  It was a helpful example of some of the techniques needed.


arm64 binary not working on rpi4 (aarch64) raspbian bookworm

Describe the bug
binary is not executable on this architecture
To Reproduce
i tried installing manually from releases (arm64) as well as with the install script
$ ./rtx/bin/rtx
bash: rtx/bin/rtx: cannot execute: required file not found

Expected behavior
rtx invokes successfully
rtx doctor output
not applicable

Additional context
$ uname -a
Linux 6.1.21-v8+ #1642 SMP PREEMPT Mon Apr  3 17:24:16 BST 2023 aarch64 GNU/Linux

$ lsb_release -a
Distributor ID: Raspbian
Description:    Raspbian GNU/Linux 12 (bookworm)
Release:        12
Codename:       bookworm

possibly relevant:
https://github.com/rust-lang/rustup/commit/745473d6a7c9810b03df36fbf9c764bd63bb9fb1
https://github.com/rust-lang/rustup/issues/3307
https://forums.raspberrypi.com/viewtopic.php?t=349291


Add -v flag to output version

This will output the version based off the git tag, e.g.:
» ./bin/kubeconform -v
development

Very good idea, thanks! Added a small comment :bow:

Thank you for your contribution! looks good to me :+1:  :rocket:


Improve Sidekiq worker resiliency

We were seeing the PostcodesCollectionWorker die after running for a few hours (possibly related to the nightly datasync).
Rather than have one long-running worker iterate through the entire 1.8 million postcodes in the database, we've now re-architected it to be short-lived, fetching the 3 oldest postcodes and triggering jobs to update those.
This also bumps our postcode updating rate from around 1 per second to around 3 per second, which cuts the total update time from around 21 days to around 7 days. We could in theory go even closer to the OS Places API 10-requests-per-second rate limit, but as Sidekiq is running across multiple instances and this figure is only approximate, it is safer set to the lower value of 3.
Finally, this PR splits the Sidekiq worker queues into two, so that we can allocate more priority to the updating of postcodes.
Trello: https://trello.com/c/jE4Qghdd/2902-fix-sidekiq-workers-locations-api
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

I wonder if using an exponential backoff strategy would be more resilient and future-proof, in case their API rate-limiting gets more strict.

The workers already use exponential backoff by default, except that we've set max_retries to 1, so no job will attempt to retry more than once.
This PR is about improving resiliency of the workers to our environment, i.e. datasyncs etc which cause workers to die.
I think what you're suggesting is a separate change to make our workers more resilient to Os Places API exceptions, which is fair, but probably something for a future PR. For now, if OS Places API limits change, we'll see lots of failed workers and we'll get a Slack notification, but nothing will fall over, Locations API would carry on working.

I know that's how Sidekiq works by default, was thinking of errors from OS Places API, but you're right that the app will not stop working and we'll get lots of alerts so should be easy to fix if it happens.


feat: add polar stat env extension

host cpu info
host memory info
host os info

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

/rebase-and-merge


[BUG] - Panick message when closing Termscp

Description
When exiting Termscp Windows version got the message:
thread 'main' panicked at 'Original console mode not set: TryFromIntError(())', C:\Users\chris.cargo\regist
ry\src\github.com-1ecc6299db9ec823\crossterm-0.20.0\src\event\sys\windows.rs:36:10
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Steps to reproduce
Use Termscp Windows version
Open Termscp in native command prompt or WSL
Exiting Termscp with ESC key
Got panick message
Expected behaviour
Not having panick message ;-)
Environment

OS: Windows 10 21H1
Architecture: 64bits
Rust version:
termscp version: 0.6.1
Protocol used:
Remote server version and name:

Log
Termscp.log attached with RUST_BACKTRACE=1 environment variable set
Additional information
termscp.log

Thanks again for reporting!
I've already solved this issue in 0.7.0


Sagemaker doesn't yet support tools

Related to  https://github.com/ucbepic/docetl/issues/160

awesome thx!


[DOC] Update spark-rapids-user-tools README

Report incorrect documentation
The changelog in spark-rapids-tools is out of date.
This page also appears to be replicated at https://pypi.org/project/spark-rapids-user-tools/
Location of incorrect documentation
https://github.com/NVIDIA/spark-rapids-tools/tree/dev/user_tools#changelog
Describe the problems or issues found in the documentation
The changelog refers to 22.10.1 and 22.10.2, even though we have had two releases since then.
Steps taken to verify documentation is incorrect
N/A
Suggested fix for documentation
Put the changelog in a separate file, and point users to the changelog from the README

Relevant to #7 ?

That section was removed from the README


3D features

Hi everyone!
I am about to start a feature extraction on a set of images obtained from cardiac MRI.
Since I will have to deal with 3D volumes (stacks of slices), I wanted to ask:
"are there some of the pyradiomics features that are suitable for these task or should I just apply the 2D feature extraction for each slice and then combine them in some other way?
Thanks a lot in advance.
Tommaso

Hello @tommydino93,
You can extract in 3D if you ensure that your voxels are isotropic (same dimension in-plane and out-of-plane). If this is not the case (the usual in MR), you can enable resampling). On the other hand you can also extract 2D. This does not produce a value per-slice, but just does not consider voxels on adjacent slices to be neighbours (only applies for texture, i.e. texture). 2D/3D therfore does not affect calculations in firstorder or shape statistics.

Ok thank you very much! To keep things simple, I think I will firstly extract 2D features.
Just some other few questions:

Can I run (and eventually HOW) pyradiomics programs in PyCharm?
Since I am new to GitHub, I also wanted to understand: "which version of the programs should I
use?"...meaning..."from which of the many folders I extract the correct (most updated) code that I can
later apply to my images?"

Sorry in advance if these are trivial questions but I can't really find a "help" or "guide" that explains how to use functions in GitHub (e.g: which are the inputs, which are the outputs, ecc)
Thank you again,
Tommaso

@tommydino93, yes you can run pyradiomics from PyCharm. Easiest to start with this is to take a look at the examples folder.
As to your second question, what exactly do you mean? Which version of PyRadiomics? In that case, I'd advise to clone the repository and run from the master branch. See here for making/cloning a git repository.
The git url for pyradiomics is https://github.com/Radiomics/pyradiomics.git.
TLDR: install git on your computer and run git clone https://github.com/Radiomics/pyradiomics.git from a bash shell

Thank you again! Before installing Git I wanted to give it a try with PyCharm. I followed the specific guide, created a specific repository. Everything seemed to work fine. I tried the "brain1" example but then I get this error:
**Error loading C extensions, switching to python calculation:
Traceback (most recent call last):
File "D:\Getting_Started_PyCharm\Pyradiomics\radiomics_init_.py", line 295, in
from radiomics import cmatrices as cMatrices
ImportError: cannot import name 'cmatrices'
Traceback (most recent call last):
File "D:/Getting_Started_PyCharm/Pyradiomics/Trial_Pyradiomics.py", line 5, in
from radiomics import firstorder, getTestCase, glcm, glrlm, glszm, imageoperations, shape
File "D:\Getting_Started_PyCharm\Pyradiomics\radiomics_init.py", line 307, in
getFeatureClasses()
File "D:\Getting_Started_PyCharm\Pyradiomics\radiomics_init.py", line 121, in getFeatureClasses
import('radiomics.' + mod)
File "D:\Getting_Started_PyCharm\Pyradiomics\radiomics\featureextractor.py", line 10, in
import pykwalify.core
ModuleNotFoundError: No module named 'pykwalify'
Process finished with exit code 1**
I have seen that this has also happened here. I followed that guide, I deleted the "radiomics" folder contained in the site-packages but I didn't understand quite well the second part. Where should I insert my new line of command?
Thank you very much in advance!
Tommaso

@tommydino93, issue #340 is not exactly the same as your error. In that case, it could not find the correct installation of PyRadiomics and subsequently, it could not load a module which was part of PyRadiomics. Your problem however, is that you are missing a 3rd-party package required by PyRadiomics to run.
There are several dependencies for PyRadiomics to function, and are detailed in requirements.txt, which is located in the repository root. Run the following command in a terminal to have python install all the requirements you need to run pyradiomics:
python -m pip install -r <path/to/requirements.txt>

@JoostJM Thank you, that did solve almost all the errors it gave me. Now I just get:
C:\Users\tommaso\PycharmProjects\Prova\venv\Scripts\python.exe D:/Getting_Started_PyCharm/Pyradiomics/Trial_Pyradiomics.py
Error loading C extensions, switching to python calculation:
Traceback (most recent call last):
File "D:\Getting_Started_PyCharm\Pyradiomics\radiomics_init_.py", line 295, in
from radiomics import _cmatrices as cMatrices
ImportError: cannot import name '_cmatrices'
[1 2]
Process finished with exit code 0
considering that I just tried a simple:
a = np.array([1,2])
print(a)
to see if it worked. Should I be worried about "Error loading C extensions" ?
Thanks

@JoostJM, I just tried following this PyCharm guide, I chose "develop" and a console opens saying:
**running develop
running egg_info
writing Solver.egg-info\PKG-INFO
writing dependency_links to Solver.egg-info\dependency_links.txt
writing top-level names to Solver.egg-info\top_level.txt
reading manifest file 'Solver.egg-info\SOURCES.txt'
writing manifest file 'Solver.egg-info\SOURCES.txt'
running build_ext
Creating c:\users\tommaso\pycharmprojects\prova\venv\lib\site-packages\Solver.egg-link (link to .)
Adding Solver 0.0.0 to easy-install.pth file
Installed d:\getting_started_pycharm
Processing dependencies for Solver==0.0.0
Finished processing dependencies for Solver==0.0.0
Process finished with exit code 0**
So it seemed ok. After this I re-tried running my "Trial_Pyradiomics" which is now just:
**from future import print_function
import numpy as np
import SimpleITK as sitk
import six
from radiomics import firstorder, getTestCase, glcm, glrlm, glszm, imageoperations, shape
imageName, maskName = getTestCase('brain1')**
and I keep on having the same error:
**Error loading C extensions, switching to python calculation:
Traceback (most recent call last):
File "D:\Getting_Started_PyCharm\Pyradiomics\radiomics_init_.py", line 295, in
from radiomics import _cmatrices as cMatrices
ImportError: cannot import name '_cmatrices'
Process finished with exit code 0**
I really don't understand what I'm doing wrong. Sorry if the question is trivial.
Thanks again!

@tommydino93, what is the content of the radiomics folder? is there a file called _cmatrices.pyd?
What is the output of the following:
import radiomics
print(radiomics.__file__)

on a side not, your last comment seemed to have installed something called 'solver', but not PyRadiomics. Do try to run the setup.py directly from the commandline (where you first cd into the root of the repository)

@JoostJM, inside pyradiomics/radiomics there is a sub-folder named src and the cmatrices file is inside that sub-folder but it has a .c extension and not a .pyd one.
Anyway:
import radiomics
print(radiomics.file)
gives me once again as output the error:
Error loading C extensions, switching to python calculation:
Traceback (most recent call last):
File "D:\Getting_Started_PyCharm\Pyradiomics\radiomics_init_.py", line 295, in
from radiomics import _cmatrices as cMatrices
ImportError: cannot import name '_cmatrices'
with the word radiomics in the code which is highlighted in red.
Moreover, I tried writing:
setup.py
in the PyCharm terminal but it tells me that the command is neither recognized as internal nor as external. Maybe I didn't understand precisely what you meant with "where you first cd into the root of the repository"?

@tommydino93 The contents of your folder mean that the extensions are not yet compiled and therefore not available when you try to start it.
To compile these you first need to run the setup script, which I usually do outside of PyCharm altogether.
If you execute the following steps, it should work:

start a command terminal (on windows: cmd.exe) in your repository. the easiest way to do this is to go to the repository in windows explorer and then click on the address bar. You should then be able to type an address, but instead, type "cmd" and press enter. This should start a terminal which has it's working directory set to the repository root.
Run the setup script by entering the following line in the terminal and press enter:
python setup.py develop
Check your script again in PyCharm

The reason setup.py did not work in the pycharm terminal is because you did not specify that it should be run with python (i.e. python setup.py). Moreover, for this script to work, you also need to specify the command (i.e. install, test, develop)

see this stack overflow thread for more info on what setup.py is.

Hi @JoostJM!
I arrived to the Pyradiomics directory through the cmd prompt, but then if I run python setup.py develop I get the error:
"python is neither recognized as an internal command nor an external command, nor an exe program, nor a batch file.
I tried typing just setup.py develop and the prompt gives me this error:
File "D:\Getting_Started_PyCharm\Pyradiomics\setup.py", line 7, in  import numpy
ModuleNotFoundError: No module named 'numpy'
which I believe it's strange because I checked in PyCharm and I have installed the numpy package. What am I doing wrong?
Thank you again!!!

Furthermore, is it mandatory to have Git installed on my PC in order to use GitHub Packages in PyCharm? I am asking you just because I also tried running setup.py in PyCharm but on the event log I get these two errors:

Can't start Git: C:\Users\tommaso\Downloads\Git-2.16.2-64-bit.exe
The path to Git executable is probably not valid. Fix it.
Install packages failed: Installing packages: error occurred.

python is neither recognized as an internal command nor an external command, nor an exe program, nor a batch file.

This means python is not in your PATH variable. What kind of python interpreter did you install? Anaconda, Python 2.7, Python 3.x?
If you know how to update your PATH variable, add the folder containing python.exe and the scripts folder. In case of regularly installed python: C:\PythonX.X;C:\PythonX.X\Scripts. If not reinstall python and ensure you select "add python to PATH"

Furthermore, is it mandatory to have Git installed on my PC in order to use GitHub Packages in PyCharm?
For PyCharm I don't know, but PyRadiomics uses git to determine the version of PyRadiomics during installation, so for pyradiomics it is sort of required yes. It's good practice to have git installed anyway when you're working with GitHub repositories, because it allows you to have and work with local copies of the entire repository.

Thank you very much! I added the python PATH, I downloaded Git and integrated into PyCharm, and now the simple code I wrote there works without any errors. I finally managed to upload the "brain1" images and masks!
Now the real programming begins! I will surely ask you something about the programs too!! Please stick around @JoostJM! :) :)
Thanks again

You're welcome! Be sure to check out the examples and notebooks in the repository, the frequently asked questions in the documentation and questions from the community both here in the issues and on the google groups. Between them, you should be able to many answers for questions you might run into during your programming and use of pyradiomics.
Happy coding!


Update UNZIP156.Z80

Fiddlind around optimizations

Can you please update your modifications to omit the INC/DEC IY changes and add a comment to the storage definitions for bitbs and blfts to warn that a 16 bit load is used to access both.  You may wish to correct the typo on the pull request too.


Fix for #59

To be pickled

objects should not be local
-> moved locked_method to callable object LockedMethod
subclasses should not mess around with getattribute
-> Even though the method was found with getattribute, getattr
was called in pickle.load(). I assume this has something to do with the
order in which objects are deserialized. Overriding getattr of the
subclass with getattribute of the base class solves the issue.

I also expanded the pickle test in test_threading to check if locking still works after pickling.

Awesome, thanks!


Code for Sacramento as a Data Utility

Potential business model via the Administrative Data Research Network:

The Administrative Data Research Network is a UK-wide partnership between universities, government departments and agencies, national statistics authorities, the third sector, funders and researchers.

We help accredited researchers carry out social and economic research using linked, de-identified administrative data – information which is routinely collected by government organisations.

We make sure any information which directly identifies people (such as names and addresses) is removed from the data before any research is done.

An Approvals Panel examines every research proposal thoroughly. Once they give their approval, the data custodian has to agree to share its data before the research can go ahead.

We provide a secure environment for the researcher to work in. They can’t take any data out of this safe setting, and our staff will scrutinise their results before publication to make sure they are relevant to the project and don’t disclose any information about individuals.

I like it

Adding Data Driven Detroit as a similar data services-based model:

Data Driven Detroit (D3) provides accessible high-quality information and analysis that drives informed decision-making. Our vision is that essential and unbiased information is used by all.

Data Driven Detroit (D3) has experienced many transitions to get to get this point. D3 began as a unique data hub for foundations and other organizations in need of reliable data to help them make better decisions about the future of Detroit and the surrounding region. To maintain our relevance, in an ever changing landscape of data, we have remained as flexible, creative and innovative as possible; cultivating partners whose values have aligned with our own and who have been committed to creating excellent products that further our organizational mission.

In 2008, The Skillman Foundation and The Kresge Foundation awarded City Connect Detroit a $1.85 million grant to incubate Data Driven Detroit (originally named the Detroit-Area Community Information System).

Within its first year, D3 was selected by the Urban Institute to join the National Neighborhood Indicators Partnership (NNIP). NNIP is a select group of organizations that have built advanced and continuously updated data systems to track neighborhood conditions in their cities.

By December 2012, Data Driven Detroit became an affiliated program of the Michigan Nonprofit Association (MNA). This move strengthened D3′s operations and increased exposure to a statewide network of member nonprofits and philanthropic organizations. Our association with MNA been an important part of our evolution into our current L3C status.

In October of 2015, D3 transitioned to the structure of a Low-Profit Limited Liability Company (L3C). An L3C requires our partners and their projects to align with the mission of D3.

As we move forward, you can count on D3 to remain committed to the very things we always have: our core values, our trusted partners, and the accumulated wisdom gained by experiencing this growth together.


VScode electron-forge-vscode-nix error loading devtools extension

Please describe your issue:
Here is my launch.json in vs code
{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [
{
"name": "Debug Main Process",
"type": "node",
"request": "launch",
"cwd": "${workspaceRoot}",
"runtimeExecutable": "${workspaceRoot}/node_modules/.bin/electron-forge-vscode-nix",
"windows": {
"runtimeExecutable": "${workspaceRoot}/node_modules/.bin/electron.cmd"
},
"program": "${workspaceRoot}/src/index.js",
"protocol": "inspector",
"env": {
"ELECTRON_ENABLE_LOGGING": "1"
}
},
{
"name": "Debug Renderer Process",
"type": "chrome",
"request": "launch",
"runtimeExecutable": "${workspaceRoot}/node_modules/.bin/electron",
"windows": {
"runtimeExecutable": "${workspaceRoot}/node_modules/.bin/electron.cmd"
},
"runtimeArgs": [
"${workspaceRoot}/src/index.js",
"--remote-debugging-port=9222"
],
"webRoot": "${workspaceRoot}/src"
}
]
}
And here is the error i see when i try to debug. things work fine on the page but i see this error on the console
ENOENT: no such file or directory, open '/Users/dev/Library/Application Support/Electron/DevTools Extension
Detailed error
/Volumes/hd2/ss/node_modules/.bin/electron-forge-vscode-nix --inspect-brk=6967 src/index.js
Debugger listening on ws://<IP_ADDRESS>:6967/113e0992-e4e6-4294-b782-aac4e71bd9a2
For help, see: https://nodejs.org/en/docs/inspector
Debugger attached.
(electron) The 'lstatSyncNoException' function has been deprecated and marked for removal.
(electron) The 'lstatSyncNoException' function has been deprecated and marked for removal.
(electron) The 'lstatSyncNoException' function has been deprecated and marked for removal.
Failed to load browser extensions from directory: /Users/dev/Library/Application Support/Electron/DevTools Extensions
{ Error: ENOENT: no such file or directory, open '/Users/dev/Library/Application Support/Electron/DevTools Extensions'
at Object.fs.openSync (fs.js:577:3)
at Object.module.(anonymous function) [as openSync] (ELECTRON_ASAR.js:166:20)
at Object.fs.readFileSync (fs.js:483:33)
at Object.fs.readFileSync (ELECTRON_ASAR.js:563:29)
at App. (/Volumes/hd2/ss/node_modules/electron/dist/Electron.app/Contents/Resources/electron.asar/browser/chrome-extension.js:419:52)
at Object.onceWrapper (events.js:273:13)
at App.emit (events.js:187:15)
errno: -2,
syscall: 'open',
code: 'ENOENT',
path:
'/Users/dev/Library/Application Support/Electron/DevTools Extensions' }
[93364:1121/125514.535753:VERBOSE1:CONSOLE(7847)] "Main._createAppUI: 28.05712890625ms", source: chrome-devtools://devtools/bundled/shell.js (7847)
[93364:1121/125514.593779:VERBOSE1:CONSOLE(7847)] "Main._showAppUI: 56.808837890625ms", source: chrome-devtools://devtools/bundled/shell.js (7847)
[93364:1121/125514.637353:VERBOSE1:CONSOLE(7847)] "Main._initializeTarget: 16.013916015625ms", source: chrome-devtools://devtools/bundled/shell.js (7847)
[93364:1121/125514.661924:WARNING:CONSOLE(184)] "%cElectron Security Warning (Insecure Content-Security-Policy)", source: /Volumes/hd2/ss/node_modules/electron/dist/Electron.app/Contents/Resources/electron.asar/renderer/security-warnings.js (184)
[93364:1121/125514.786605:INFO:CONSOLE(184)] "%cElectron Security Warning (Insecure Content-Security-Policy)", source: /Volumes/hd2/ss/node_modules/electron/dist/Electron.app/Contents/Resources/electron.asar/renderer/security-warnings.js (184)
[93364:1121/125514.790147:VERBOSE1:CONSOLE(7847)] "Main._lateInitialization: 4.40185546875ms", source: chrome-devtools://devtools/bundled/shell.js (7847)
2018-11-21 12:55:14.887 Electron Helper[93367:1245202] Couldn't set selectedTextBackgroundColor from default ()
Debugger listening on ws://<IP_ADDRESS>:6967/113e0992-e4e6-4294-b782-aac4e71bd9a2
For help, see: https://nodejs.org/en/docs/inspector
What command line arguments are you passing?
Put the arguments here

What does your config.forge data in package.json look like?
Paste the config.forge JSON object here

"config": {
"forge": {
"make_targets": {
"win32": [
"squirrel"
],
"darwin": [
"zip"
],
"linux": [
"deb",
"rpm"
]
},
"electronPackagerConfig": {
"packageManager": "yarn"
},
"electronWinstallerConfig": {
"name": "ss"
},
"electronInstallerDebian": {},
"electronInstallerRedhat": {},
"github_repository": {
"owner": "",
"name": ""
},
"windowsStoreConfig": {
"packageName": "",
"name": "ss"
}
}
},
Please provide either a failing minimal testcase (with a link to the code) or detailed steps to
reproduce your problem. Using electron-forge init is a good starting point, if that is not the
source of your problem.

install clean app by npm -init -y
init by electron-forgre very clean, no template engine
create .vscode folder
add launch.json file

{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [
{
"name": "Debug Main Process",
"type": "node",
"request": "launch",
"cwd": "${workspaceRoot}",
"runtimeExecutable": "${workspaceRoot}/node_modules/.bin/electron-forge-vscode-nix",
"windows": {
"runtimeExecutable": "${workspaceRoot}/node_modules/.bin/electron.cmd"
},
"program": "${workspaceRoot}/src/index.js",
"protocol": "inspector",
"env": {
"ELECTRON_ENABLE_LOGGING": "1"
}
},
{
"name": "Debug Renderer Process",
"type": "chrome",
"request": "launch",
"runtimeExecutable": "${workspaceRoot}/node_modules/.bin/electron",
"windows": {
"runtimeExecutable": "${workspaceRoot}/node_modules/.bin/electron.cmd"
},
"runtimeArgs": [
"${workspaceRoot}/src/index.js",
"--remote-debugging-port=9222"
],
"webRoot": "${workspaceRoot}/src"
}
]
}

run debug mode

go to Debug console in VSCODE
and see this error there
/Volumes/hd2/ss/node_modules/.bin/electron-forge-vscode-nix --inspect-brk=6967 src/index.js
Debugger listening on ws://<IP_ADDRESS>:6967/113e0992-e4e6-4294-b782-aac4e71bd9a2
For help, see: https://nodejs.org/en/docs/inspector
Debugger attached.
(electron) The 'lstatSyncNoException' function has been deprecated and marked for removal.
(electron) The 'lstatSyncNoException' function has been deprecated and marked for removal.
(electron) The 'lstatSyncNoException' function has been deprecated and marked for removal.
Failed to load browser extensions from directory: /Users/dev/Library/Application Support/Electron/DevTools Extensions
{ Error: ENOENT: no such file or directory, open '/Users/dev/Library/Application Support/Electron/DevTools Extensions'
at Object.fs.openSync (fs.js:577:3)
at Object.module.(anonymous function) [as openSync] (ELECTRON_ASAR.js:166:20)
at Object.fs.readFileSync (fs.js:483:33)
at Object.fs.readFileSync (ELECTRON_ASAR.js:563:29)
at App. (/Volumes/hd2/ss/node_modules/electron/dist/Electron.app/Contents/Resources/electron.asar/browser/chrome-extension.js:419:52)
at Object.onceWrapper (events.js:273:13)
at App.emit (events.js:187:15)
errno: -2,
syscall: 'open',
code: 'ENOENT',
path:
'/Users/dev/Library/Application Support/Electron/DevTools Extensions' }
[93364:1121/125514.535753:VERBOSE1:CONSOLE(7847)] "Main._createAppUI: 28.05712890625ms", source: chrome-devtools://devtools/bundled/shell.js (7847)
[93364:1121/125514.593779:VERBOSE1:CONSOLE(7847)] "Main._showAppUI: 56.808837890625ms", source: chrome-devtools://devtools/bundled/shell.js (7847)
[93364:1121/125514.637353:VERBOSE1:CONSOLE(7847)] "Main._initializeTarget: 16.013916015625ms", source: chrome-devtools://devtools/bundled/shell.js (7847)
[93364:1121/125514.661924:WARNING:CONSOLE(184)] "%cElectron Security Warning (Insecure Content-Security-Policy)", source: /Volumes/hd2/ss/node_modules/electron/dist/Electron.app/Contents/Resources/electron.asar/renderer/security-warnings.js (184)
[93364:1121/125514.786605:INFO:CONSOLE(184)] "%cElectron Security Warning (Insecure Content-Security-Policy)", source: /Volumes/hd2/ss/node_modules/electron/dist/Electron.app/Contents/Resources/electron.asar/renderer/security-warnings.js (184)
[93364:1121/125514.790147:VERBOSE1:CONSOLE(7847)] "Main._lateInitialization: 4.40185546875ms", source: chrome-devtools://devtools/bundled/shell.js (7847)
2018-11-21 12:55:14.887 Electron Helper[93367:1245202] Couldn't set selectedTextBackgroundColor from default ()
Debugger listening on ws://<IP_ADDRESS>:6967/113e0992-e4e6-4294-b782-aac4e71bd9a2
For help, see: https://nodejs.org/en/docs/inspector

@asifashraf That error is coming from Electron internally and is fixed on newer Electron versions


how do I turn it on on a mac?

i downloaded the file, extracted it, pasted the user file into tampermonkey, and put in the UUID into the config, but when I try double clicking on start.bat, it doesnt work

Use <IP_ADDRESS> in userscript and config

In what part of user script and config, exactly??
On Friday, April 1, 2016, philw1508<EMAIL_ADDRESS>wrote:

Use <IP_ADDRESS> in userscript and config
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/davidmann4/ogar-feeder-bot/issues/362#issuecomment-204580070


Enable ci for azure-maps geolocation, search, render and route packages

API change check
API changes are not detected in this pull request.


Use url-join to be more compatible for env config

Coverage remained the same at 34.391% when pulling 9c38ac009bd05537e6f0bdcdf7cabeafdacc35f1 on mintao/urljoin into a323f38c353d9bd09f6609768b4fe17f468f3d3f on master.


Publish 8.5-20231129110721+0000

Publish 8.5-20231129110721+0000

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.


IOU - "Request $xx" button appears at the top of the page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch the app and login
Click on green Fab button
Tap Request Money
Select amount and click Next

Expected Result:
"Request $xx"  button should appear at the bottom of the page
Actual Result:
"Request $xx"  button appears at the top of the page
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.1.16-1
Reproducible in staging?: Yes
Reproducible in production?: No
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

**Expensify/Expensify Issue URL:**
**Issue reported by:** Applause
**Slack conversation:**
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Triggered auto assignment to @NikkiWines (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

It is missed in https://github.com/Expensify/App/pull/4070.
we just need to add flex1 to AnimationStep at https://github.com/Expensify/App/blob/c95c2ddbafc5c7e9904ca943cff783d913d53556/src/pages/iou/IOUModal.js#L397.
It fixes the issue. submitting a PR to resolve deploy blocker as there is a timezone gap. Feel free to close it if this is not appropriate.

@parasharrajat thanks for submitting a fix. Let me just check how compensation would work for fixing this deploy blocker.
Typically the engineer who introduced the bug should fix it in order to receive full compensation, but it looks like that contributor has been a bit difficult to contact recently.
@Expensify/contributor-management, do we have a set fee for fixing deploy blockers?

Code is now on staging. @kavimuru can you retest and close the issue if its been resolved.

@Expensify/contributor-management, do we have a set fee for fixing deploy blockers?

Yes, contributors get compensated $250 for fixing deploy blockers.  I'm going to auto-assign a CM via the External label then they can pay 7 days after there are no regressions

Triggered auto assignment to @dylanexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

On it!

Internal: https://www.upwork.com/ab/applicants/1463434565061685248/job-details
External: https://www.upwork.com/jobs/~010a9031d77677f005

Triggered auto assignment to @parasharrajat (Exported)

Current assignee @NikkiWines is eligible for the Exported assigner, not assigning anyone new.

Retest for https://github.com/Expensify/App/pull/6417 (Fix) was a pass! I think we can close this one out 🎉

Oh wops I think a payment is needed here. Will remove the deploy blocker label and reopen so the payment is tracked here

Greetings,
I've checked out your given link and understand what's the actual problem you're facing, basically, the widgets or the main axis alignment issue or maybe the whole content of the screen is in the same container so another container will be used and your problem will be resolved.. I've extensive experience in developing these applications and I am available to help you out in getting your desired result. I'd like to hop on a call and give you an exact strategy to how to get this handle for your business.
I Will provide you with the complete documentation and the source code for every customization of the application so that you can edit it whenever you want in the future. I have read the job description and find it a perfect match for me as I have 6 years of extensive hands-on experience in Application Development and knowledge about React native and flutter development as well. I have developed a lot of applications and stores using GroovePages/Drupal/React Native/WordPress/Woo-commerce/Shopify/Magneto/PHP/Larvel and would love to help you out as per your requirements.
Here are my current ongoing projects on which I’m working

https://functionalclinicalaesthetics.com/
https://feelthemerge.com/
You can pursue the owner to your own leisure when you have time.

I'd be happier to again cover more detail on call hopefully this is will valuable to you and here are some testimonials from my previous clients who've run e-commerce websites that I've done website dev, and E-commerce plugin integration, etc for
1- ninachristineperfumes.com
2- brickstowall.com
3- addisonkhansolicitors.com
4- arifanis.com
5- ewatchwholesale.com
6- nomadz.co
Looking forward to talking to you.
Thank You,
Best Regards,
Saad khan.

@dylanexpensify when you create jobs in Upwork, can you be sure to include the GH issue number in the title? It makes it much easier to find/reconcile them.

@NikkiWines Rajat was auto-assigned to this issue as a Contributor+ via the Exported label.  Should he have been assigned to this issue before since he submitted an acceptable proposal to fix the issue?
cc @Beamanator (we're working on C+ flows)

I think you're right @mallenexpensify he should have been assigned before adding External, so we would have had the automatic comment that said he was assigned to the job. At this point we should be able to remove & re-add him to get that comment, or just keep it as-is. But it looks like @NikkiWines basically gave him the job here

@mallenexpensify, yeah, he probably should've been assigned.
It was a little unclear to me if I needed to follow the typical triaging process given this was a deploy blocker and had some time urgency associated with it. In this case, Rajat had a fix up super quickly, so I didn't end up going through the standard process (sorry)

Issue not reproducible during KI retests. (First week)

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.1.16-10 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/6417

If no regressions arise, payment will be issued on 2021-12-06. :confetti_ball:

Ping for

@dylanexpensify

Issue not reproducible during KI retests. (Second Week)

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.1.17-7 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/6417

If no regressions arise, payment will be issued on 2021-12-13. :confetti_ball:

What's the appropriate payout date for this GH supposed to be? cc @NikkiWines for a buddy check perhaps?

Botify seems drunk 😅  I believe 2021-12-06 is the correct date, especially since QA has not been able to reproduce this issue for a couple of weeks now.

Ping for

@dylanexpensify

Perfect, thanks for clarifying! Paying now!

@parasharrajat sent offer! Once accepted I'll pay it out now!

Paid, contract ended, posting closed!


Exclude a Category and its jobs from the List, Sidebar, and Counts

Hey,
This is not actually an issue - but is a particular use case that I can't seem to implement and can't seem to figure out in reading the API docs. Here's the rundown:

We need to be able to exclude an entire Category (and all jobs in this category) from the job List, Sidebar and Counts (re: numbers beside the Locations in the sidebar)
The jobs that are in this Category should only be accessible in the Single Job Post view

Would someone be able to point me in the right direction?

Hello @timelytree,
You are able to adjust this within the files for the Search Service. Basically in there you are able to utilize the NOT operator within your queries.
Please let me know if you have any more questions.

Hey @charlesabarnes, thanks a million for the super quick reply. If it's not too much to ask, can you give me an example of how to use the NOT operator within queries?

An example would be here .
For a query this will evaluate as isOpen:1 AND isDeleted:0  What you can add to the end of that is NOT publishedCategory.id:{categoryIdHere}

Worked like a charm, thanks again!!


IPersistedGrantStore for the production mode

Can I ask you one questions, cause maybe I didn't clearly understand the documentation? We can configure PersistedGrantStore by implementation the IPersistedGrantStore interface. We can consider EF core and use suitable NugetPackage or we can implement own solution.
By default we keep reference and refresh tokens in memory. It works until the IS4 app pool won't be unloaded, becuase of the Idle timeout e.g. and after pool loaded once again, actually we have no in-memory data. It is according to logic.
Is it good idea if I use "Always on" option (disable Idle timeout) instead of the PersistedGrantStore implementation?
As a result my pool will never be unloaded and I always can use in memory data. We get some problems if app be crushed, but in general this way seems quite simple and working.
What is your recomendation about this, maybe I missed sothing important?

Well - if you only have short-lived data - this might be an option.
Keep in mind that this will not work once you try to scale out to more than 1 server.

Sure, we need to share data between servers in the scaling case.
I just starts with IS and at the first stage want to implement the simpiest production environment, investigate potentially proplems and after try other approaches, such as persistent storage.
Thank you for the answer!


Invalid Syntax error on multi-line values

Description:
When creating multi-line values, such as a dictionary, I am receiving an 'invalid syntax' error when trying to run Hydrogen within Atom. However, if I run the dictionary as a long single line, Hydrogen runs fine.
I have tested the multi-line code within my Jupyter notebook, and have run the .py file through Terminal with the multi-line code saved and both run just fine. Hydrogen is the only thing that I try to run that is having the issue.
Steps to Reproduce:

Create multi-line value (dictionary in this case).
Run hydrogen (shift+return)

Versions:
Which OS and which version of Hydrogen and Atom are you running?
Mac OS - 10.12.5
Atom - 1.18.0
Hydrogen - 1.18.0
Atom    : 1.18.0
Electron: 1.3.15
Chrome  : 52.0.2743.82
Node    : 6.5.0
You can get this information from copy and pasting the output of atom --version  from the command line.
Logs:
Please post any error logs and the output of the developer tools as described in our Debugging Guide.
File "", line 1
'population': [200.4, 143.5, 1252, 1357, 52.98]}
^
SyntaxError: invalid syntax
When using this dictionary:
worldDict = {'country': ['Brazil', 'Russia', 'India', 'China', 'South Africa'],
'capital': ['Brasilia', 'Moscow', 'New Delhi', 'Beijing', 'Pretoria'],
'area': [8.516, 17.10, 3.286, 9.597, 1.221],
'population': [200.4, 143.5, 1252, 1357, 52.98]}

I encounter the same issue with multi-line sections that I try to execute in one shot with Hydrogen.

I have figured out that when declaring a variable and assigning a value to it, you have to run the code from the top line (seems inconvenient, when I'm wanting to run the code upon completion of typing the values, but I get it).
However, I have not figured out a workflow to make if/elif/else statements run within Hydrogen.

However, I have not figured out a workflow to make if/elif/else statements run within Hydrogen.

I select all the text I want to run and hit cmd-enter (hydrogen: run command).

You can also use cells to run predefined code blocks.
I'm going to close this as a duplicate of #724. Please take a look at #724 for more information.


Add -v Flag

nvm --version for me returns:
0.35.3

nvm -v returns the output of an unrecognized command.
This is issue proposes adding -v as an alias for --version as it is for node, npm, and many other commands.

Sure, that seems reasonable.

Can I take this up ?

@chitranshi21 please; anything with the "pull request wanted" label, without a linked PR, is always up for grabs

I tried looking into the source code, but I have no idea where to make the change for this. 😅

Is there a quick guide on how to test it locally ? Install and uninstall from the cloned project?
The install/makefile are reaching out to the main repo.
Also does removing/uninstalling mean running  rm -rf $NVM_DIR ~/.npm ~/.bower?

@JimLynchCodes https://github.com/nvm-sh/nvm/blob/master/nvm.sh#L3535
@chitranshi21 ~/.npm and ~/.bower (which shouldn't even exist; bower's been dead/deprecated for years) never need to be touched. To uninstall nvm, you would rm -rf $NVM_DIR and then (optionally) manually remove the relevant lines from your profile files. To test it locally, you can cd $NVM_DIR and it's probably already a git repo, in which case you can just check out your branch.


[🔨common]: Add autocomplete for inputs fields

Expected result: when you fill out inputs fields, the browser automatically prepends values based on previously entered similar data. This feature will help to fill out the information faster.

Based on recent discussions, the implementation of this feature assumes that the uip Settings component will be wrapped in a form tag. And then inputs fields will be able to have autocomplete attribute


Update to Node 20

When running this action, I get this warning
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: yutailang0119/action-ktlint@v3.

@merlinpaypal I have released v4.0.0, which has been updated to Node 20.
Please give it a try.


Error occured during executing "${ command }" command

First of all thanks for writing this wrapper.
I did $ npm install react-native-google-place-picker --save and then a $ react-native link and ran into this issue:
$ react-native link
Preparing to link react-native-google-place-picker for iOS
Checking CocoaPods...
CocoaPods already installed
rnpm-install info Linking react-native-google-place-picker android dependency
rnpm-install info Android module react-native-google-place-picker has been successfully linked
rnpm-install info Linking react-native-google-place-picker ios dependency
rnpm-install info iOS module react-native-google-place-picker has been successfully linked
Checking Podfile in iOS project (/Users/samlau/Dev/alert_ios/ios/Podfile)
Adding Podfile to iOS project
Installing Pods
node_modules/react-native-google-place-picker/bin/cocoapods.sh: line 45: pod: command not found
/Users/samlau/Dev/alert_ios/node_modules/react-native/local-cli/rnpm/core/src/makeCommand.js:19
        throw new Error(`Error occured during executing "${ command }" command`);
        ^

Error: Error occured during executing "node_modules/react-native-google-place-picker/bin/cocoapods.sh" command
    at ChildProcess.prelink (makeCommand.js:19:15)
    at emitTwo (events.js:87:13)
    at ChildProcess.emit (events.js:172:7)
    at maybeClose (internal/child_process.js:821:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)

This is my package.json:
{
  "name": "alert_ios",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start"
  },
  "dependencies": {
    "axios": "^0.13.1",
    "lodash": "^4.15.0",
    "momentjs": "^1.1.15",
    "react": "15.2.1",
    "react-native": "^0.31.0",
    "react-native-google-place-picker": "^1.1.2",
    "react-native-navbar": "^1.5.0",
    "react-native-swipeout": "git+https://github.com/magrinj/react-native-swipeout",
    "react-redux": "^4.4.5",
    "redux": "^3.5.2",
    "redux-thunk": "^2.1.0"
  },
  "devDependencies": {
    "babel-eslint": "^6.1.2",
    "eslint": "^3.3.1"
  }
}

From your logs, pod: command not found  is the cause of failure.
To solve this, reinstall CocoaPods:
gem install cocoapods
Check this issue to get more info.


Examples of usage with non-coroutine based approaches

Right now the SDK is heavily based on Kotlin coroutines. This is how it works internally, and it also is exposed in our APIs.
However, not all users will be using coroutines in their apps. They may use alternative approaches (like Rx), or not use any such approach at all.
Should we provide examples of SDK usage from code written using Rx, or just "standard" Android code without coroutines?
Should we change/add APIs that do not impose requirements and/or additional dependencies on users in terms of our coroutines usage?
┆Issue is synchronized with this Jira Task by Unito

I agree that we should illustrate as many mechanisms for using the SDK as possible, however bear in mind that the Android documentation specifically states:

Coroutines is our recommended solution for asynchronous programming on Android.


Factory boy

refactor tests to use factory_boy

Coverage decreased (-0.006%) to 85.7% when pulling 9363ad21cf781734791af01606cad83e110f2119 on factory_boy into 54b99a8d363bed80975c41de479a841d36badb98 on master.


Viewing Pods and Nodes - Show the app in the terminal fails maybe due to trying port 80 when the server is running port 8080

I am working through https://kubernetes.io/docs/tutorials/kubernetes-basics/explore/explore-intro/ and the step "Show the app in the terminal" but that gets a "connection refused" it seems because curl http://localhost:8001/api/v1/namespaces/default/pods/$POD_NAME/proxy/ seems to try port 80 on the pod but the javascript server is listening on 8080. I have exec'd via bash and a local curl to 8080 works fine. Any ideas? Are the docs "wrong"? Do I need to reconfigure something for it to work?
later in the tutorial, https://kubernetes.io/docs/tutorials/kubernetes-basics/expose/expose-intro/, curl http://"$(minikube ip):$NODE_PORT" just times out.

pretty sure this issue is a duplicate of https://github.com/kubernetes/website/issues/41259, sorry if it is. I'll look more.

Looks like the proposed fix there is incorrect and on-hold. This comment, https://github.com/kubernetes/website/pull/40590#issuecomment-1503163124, seems to indicate that the /proxy/ ending does work, but in my case it does not apparently because of the port discrepancy?
$ curl http://localhost:8001/api/v1/namespaces/default/pods/$POD_NAME/proxy/
{
  "kind": "Status",
  "apiVersion": "v1",
  "metadata": {},
  "status": "Failure",
  "message": "error trying to reach service: dial tcp <IP_ADDRESS>:80: connect: connection refused",
  "reason": "ServiceUnavailable",
  "code": 503
}

I am checking https://kubernetes.io/docs/tasks/debug/debug-application/debug-service/ for the other issue I face with
curl http://"$(minikube ip):$NODE_PORT"

not working.

Page related to issue: https://kubernetes.io/docs/tutorials/kubernetes-basics/explore/explore-intro/
/language en
/kind bug

I also submitted zh-cn and pt-br fixes since the same pages existed there. I didn't see the same text/url anywhere else.
Still I am not sure why the service is not exposed but I suspect that is a different issue. I am using minikube as suggested by the tutorial.

pretty sure this issue is a duplicate of #41259, sorry if it is. I'll look more.

This indeed appears to be a duplicate of the mentioned issue #41259, marking this as a duplicate.
/triage duplicate

Sorry for not finding the duplicate. Thanks for finding.


Need to remove 2 regions from the azure-prefligh.sh

In the script, we have it looking at australiaeast and westus, as well as 12 other regions.  However, BigAnimal is unavailable in australiaeast and westus.  Therefore, shouldn't we remove those two regions from this preflight script?

This has been done in the latest PR (removed the westus)
The australiaeast is now a supported location.
Thank you very much for reporting.
Usage: biganimal-preflight-azure [options] <target-subscription> <region>

Arguments:
    <target-subscription>    Azure subscription for BigAnimal deployment
    <region>                 Azure region for BigAnimal deployment

Options:
    -h, --help               Print this help message
    -i, --instance-type      Azure VM instance for BigAnimal cluster, e.g., e2s_v3
    -a, --high-availability  Plan for BigAnimal cluster with high availability enabled
    -e, --endpoint           Network endpoint flavor for BigAnimal cluster
    -r, --activate-region    Include region activation, if no clusters exist in region
    --onboard                Check if the user and subscription are configured appropriately

Behavior defaults to --onboard if no other options are provided.

Examples:
    biganimal-preflight-azure --onboard 12412ab3d-1515-2217-96f5-0338184fcc04 eastus2
    biganimal-preflight-azure -i e2s_v3 --high-availability -e private 12412ab3d-1515-2217-96f5-0338184fcc04 eastus2

vailable regions are: australiaeast brazilsouth canadacentral centralus eastus eastus2 francecentral japaneast northeurope norwayeast southcentralus southeastasia uksouth westeurope westus2 westus3
Available instance types are: e2s_v3 e4s_v3 e8s_v3 e16s_v3 e20s_v3 e32s_v3 e48s_v3 e64s_v3
Available endpoint flavors: public private

check the above regions list in available regions


⚠ ClamAV has degraded performance

In 5846de5, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 66 ms

Resolved: ClamAV performance has improved in be14fb6.


FEATURE: Optimizer creates repairing policies in the TOSCA accepted by the Deployer

This PR makes the Optimizer module create repairing policies in the syntax given in Section 7.3 of this document https://docs.google.com/document/d/1Z9RdaWn54IomgI-jCiw-tdutsCdDoHfk67PNUASOFmY/edit# .
There is also a bit of refactoring of previous repeated code.
In order to have a finished way to create these policies, for the next PRs it is still required to specify which modules can scale. Following the last directives, I will create the documentation of the module description, usage and run (I do not add it now due to lack of time as somebody may want to see the code for discussion in the next meeting).

Thanks @perezp.
Could you add some test to check the policies generation?

I'm writing tests for the generated policies. Thanks @kiuby88

Current coverage is 32.14%

Branch #170 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.

Powered by Codecov. Updated on successful CI builds.

@kiuby88
I addressed your comments and I solved a bug which made travis fail.
There are now a lot of commits, do you want me to squash now or after you check the comments and changes?

+1

Thanks for the review @kiuby88 . Merging....


docs(jsdoc): JSX Renderer Middleware

This PR is to add JSDoc for JSX Renderer Middleware.
Note that the target of the PR is not main.
Related:

#1338

#2680

[x] bun denoify to generate files for Deno

[x] bun run format:fix && bun run lint:fix to format the code

@goisaki
The title is JSX Renderer, but the code is JWT Middleware. Can you check ti?

Aah! My mistake completely. I will correct it.


gotestfmt is moving to the GoTestTools org

gotestfmt is moving to the GoTestTools organization, please update your references. See the announcement/release here.

Thank you, sir.


Exception: Found syntax errors.

IDE: IntelliJ IDEA
flutter vesrion: 3.7.1
There is an error when I run the "pub get" button :
"D:\Program Files\Flutter\flutter\bin\flutter.bat" --no-color pub get
Flutter assets will be downloaded from https://storage.flutter-io.cn. Make sure you trust this source!
Generating synthetic localizations package failed with 1 error:
Exception: Found syntax errors.
is this flutter version problem?

I have solved this problem by a flutter issue 119593


Anyway to choose default camera, when user has different type of camera app?

What do you mean? The camera action is simply firing Camera intent
So if there's another camera app, you can choose it from the intent chooser right?

I'm also find issue like this, for some reason in some device brand when firing Camera just open the default camera even they have another camera installed in their device,, so the chooser is not show,,
any idea about this?

Unless you can comes up with a PR I'm closing this because dealing with custom intent will add more responsibility (and headache)


Google Colab

Let's get this repo working in Google Colab.

@akristoffersen @TheQuantumFractal @jake-austin who are working on this right now.

WIP Google Colab link. This will be cleaned up in the future.

colab_ssh works for port fowarding, it can be used to get --viz=viewer instead of tensorboard. Just tried and it works.

how would i implement it into the colab thats above in work in progress?

So I tried running this, but the tensorboard command just gives me a 403 error, any suggestions?

colab_ssh works for port fowarding, it can be used to get --viz=viewer instead of tensorboard. Just tried and it works.

Google banned the lab_ ssh, what should I do in such a situation

Oh I think the 403 error was because of browser settings, apparently cookie settings can cause that. I switched to Chrome and Tensorboard is working


Flutter Markdown: Allow aligning table cell content

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
When using flutter markdown, the text in the table cell is always aligned to the center of a table cell, horizontally and vertically. It would be my preference to be able to align the text to a specific side of the table cell.
Markdown(
              data: """
|   |   |
|---|---|
|  **09.00** | Test thing about this really long column that should wrap when doingstusfff jasijifjsai jgisajgisajigfaj jsaigfjsaigfjsai jgisajijisjgijasigfj  |
""",
              styleSheet: MarkdownStyleSheet(
                codeblockPadding: EdgeInsets.zero,
                tableBorder: TableBorder.all(color: Colors.black, width: 0),
                tableColumnWidth: IntrinsicColumnWidth(),
                tableCellsPadding: EdgeInsets.only(bottom: 10, left: 10),
                tableHeadAlign: TextAlign.start,
              ),
            ),

Proposal
Adding a field to the MarkdownStylesheet for aligning table cell elements
Markdown(
              data: """
|   |   |
|---|---|
|  **09.00** | Test thing about this really long column that should wrap when doingstusfff jasijifjsai jgisajgisajigfaj jsaigfjsaigfjsai jgisajijisjgijasigfj  |
""",
              styleSheet: MarkdownStyleSheet(
                codeblockPadding: EdgeInsets.zero,
                tableBorder: TableBorder.all(color: Colors.black, width: 0),
                tableColumnWidth: IntrinsicColumnWidth(),
                tableCellsPadding: EdgeInsets.only(bottom: 10, left: 10),
                tableHeadAlign: TextAlign.start,
               // Allow aligning content of table cells
                tableCellAlignment: Alignment.topCenter,

              ),
            ),

After taking a look at the code, it seems like it would be most straightforward to allow setting the defaultVerticalAlignment field. Right now it's hardcoded as "TableCellVerticalAlignment.middle"
https://github.com/flutter/packages/blob/f163786731b88201d487e57a902d4453a0bc726f/packages/flutter_markdown/lib/src/builder.dart#L421


Transparent compiler

Hi @0101,
I wrote some words for our session on 01/12.
Are you ok with us publishing this?
We would be happy to tweak anything you like.
Cheers,
Florian

@nojaf Sure, go for it :)


redirect request to real leader

Like etcd, if we send request to a follower, the follower will redirect the request to leader instead.
But the redirect may have two ways:

follower returns 301 and let client to redirect to leader
follower proxy the command to leader

We should check which etcd uses, and determine which we should use.
/cc @ngaut  @qiuyesuifeng @huachaohuang

I prefer option 2 in case of isolation of client and  pd leader.

Should we redirect both api and rpc requests, or just api requests?

I think just http api request is OK now.


Autoencoder models on huggingface have incorrect name

Hello,
The pretrained autoencoder for subject1 is called "epoch120.pth", which is what the code expects, however the pretrained models for subjects 2, 5, and 7 are called "last.pth", causing an error when the model is loaded in the Reconstructions script.

Thank you! I have changed the names for subj 2 5 7 to be epoch120 instead of last.pth on huggingface

Hey Reese,
Sorry to bother you. But I want to know have you tried to train that autoencoder through 'train_autoencoder.py'? It seems that it has missed one file named 'sd_image_var_autoenc.pth'.

Do you mind to help me out? Thanks!

Hello, I have only been using the pretrained models so I have not tested
the training script at all.
On Wed, Oct 18, 2023, 5:21 AM Tong Chen @.***> wrote:

Hey Reese,
Sorry to bother you. But I want to know have you tried to train that
autoencoder through 'train_autoencoder.py'? It seems that it has missed one
file named 'sd_image_var_autoenc.pth'.
[image: image]
https://user-images.githubusercontent.com/61048672/276210148-9fd3f3e5-5600-4e87-90eb-ed63bb938d1d.png
Do you mind to help me out? Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/MedARC-AI/fMRI-reconstruction-NSD/issues/29#issuecomment-1768141842,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASPBFOQRQ4AKABRVUGZ4EYDX76UTHAVCNFSM6AAAAAA52RQ6FGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRYGE2DCOBUGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Keep SPI branch up to date with main.

Merges in main to keep this branch up to date. Only change is resolving the merge conflict that arose from the change log. Resolved conflict from change-log and fixed a function signature (see commits for more details)

Since we haven't heard from @str4d, I'll merge these changes in.
As @QuantumEF mentioned, it's just to bring the branch up to data and resolve any merge conflicts.


Blog about Falco artifacts signing and signature verification

/area blog
What would you like to be added:
A blog about the signing of the Falco artifacts (plugins and rule set) and related transparent verification of the artifacts' signatures, as OCI artifacts.
The blog should explain:

the authenticity value added by signatures in the general supply chain security
which are the artifacts distributed as part of the Falco supply chain
how signing and verification support have been implemented for the Falco artifacts
how the end user can benefit from the security features that have been introduced.

Why is this needed:
The blog is needed and is important to share with the community that the Falco ecosystem continuously improve supply chain security.
Additional context:
For reference about the discussion and the decisions made on Falco artifacts signing and signature verification, please refer to https://github.com/falcosecurity/falcoctl/issues/174.
Individual works spanned across:

https://github.com/falcosecurity/plugins/issues/244
https://github.com/falcosecurity/rules/issues/65
https://github.com/falcosecurity/falcoctl/issues/174

/assign

As discussed with @LucaGuerra we'd like to work on this.

I will be very happy to work with Max on this. Thank you for tracking it :pray:

/assign


Add profile links for Mastodon and XMPP/Jabber

What
This PR adds profile links for Mastodon and XMPP/Jabber.
I'm not sure whether this is the proper way of doing this, I tried to mimic the existing code.
This is the server side of https://github.com/hummingbird-me/hummingbird-client/pull/808
Checklist
The changes I've made pass Rubocop and the tests I've written pass.
Rubocop complains about a lot of code that I did not touch.

[ ] All files pass Rubocop
[X] Any complex logic is commented
[X] Any new systems have thorough documentation
[ ] Any user-facing changes are behind a feature flag (or: explain why they can't be)

[ ] All the tests pass
[X] Tests have been added to cover the new code

See https://github.com/hummingbird-me/hummingbird-client/pull/808#issuecomment-565858730


Minor fine tune to reduce hue shift in red and blue region

More experimentation shows that the threshold for backoff doesn't seem to be needed. Also, we can approach a little closer on the last iteration. The last part seems to reduce hue shift a bit more.

That's fair, just didn't want to overstep.

Yup, I get it — better safe than sorry. Just mentioned it for future calibration :)


[ERC721] Write test suite

Write a comprehensive test suite for the erc721 implementation here: https://github.com/dojoengine/dojo/tree/main/crates/dojo-erc/src/erc721
Inspiration can be taken from the Open Zeppelin teams test suites

hii, can I take this?

i started to write some


Dismiss notification?

I can't figure out how to hide an LNNotification once it has been shown.

Hi,
To dismiss, either wait, tap on it or swipe up. If you tap, the notification's handler will be called if set.

Sorry I meant programmatically. Is there a way to dismiss a LNNotification object?

Not possible at this time. Please state your requirements, as they are not clear. Once the notification is shown, it will be on screen for a few seconds and perish. Is it really needed to be able to hide it faster from code? I guess it is possible to develop, I just wonder what the use case is. Thanks

I'm very much in need of this feature too:
My app may display a few notifications in a root window, I might tap on one of them and go to another page (which will gracefully dismiss the notification too), but, if I tap a row in my table view and go to the same destination page (which is essentially what tap on notifications will result in too) the notification will wait out it's time and then gets dismissed.
I want to be able to dismiss the visible notification at any time. I can clear pending notifications, I know that, but if it was possible to dismiss the current notification too, that'd be awesome.
P.S.: LNNotificationBannerWindow already has the functionality: - (void)dismissNotificationViewWithCompletionBlock:(void(^)())completionBlock; and just needs a finishing touch to be able to link it to the current notification (a banner one of course) and we are good to go.

Cheers, now it's more clear. Will see what I can do.

@mohpor I am having a problem designing this workflow.
For you to know which notification is currently displayed, I could provide a currentlyPresentedNotification property, but managing such a property is difficult. For example, if notifications are presented as alerts, several notifications can overlap and when dismissing the top one, presents the previous one. Now imaging a mix of both banners and alerts.
You should change your design to not display notifications if the UI already presents the information of the notification. For example, in a mail application, you would not display a notification of new mails, if the user already has the inbox open and can select the new mails. Likewise, why display a notification if your user can already select a cell and open the view?
Notifications are meant to be set and forget.

@LeoNatan, I remember when I was reviewing the library to what we can do to mitigate this problem and I came to the same conclusion. but:
1- We can provide this possibility for notifications other than alerts, because alerts shouldn't be dismissed without user's action IMHO,
2- I already prevent the app from displaying notifications, but there are cases where we would move to a view which makes the notification obsolete. In those cases, it would very much help the UX if we could dismiss the current notification as well!

But even in Apple's case this is the case with their own implementation. If you receive a mail banner notification and while it is open, start the Mail app, the banner will also remain open until it closes.
I am not saying having this ability is not useful, just that it requires thinking.

OK, point taken. Lets move on for now and see what we can do in the future. Thanks for your time and cheers as always.

I will leave the the issue open. Perhaps in some time I will come back to it and think of something better. I am planning some major changes for this library eventually, and will look to incorporate this.

Sounds great, good luck. Let me know if I could help.


Typo fix request

Hello sir, that is yesn't acceptable pls be ✨ professional ✨ and change problem now thanks.

wow sir. this is a great change. thank you.


Let IronPython Compiler only create one exe file when target to exe or winexe

The issue is #1785
I add a new method CreateAssemblyGen in ClrModule to get AssemblyGen, so that I can add exe host code into ScriptCode assembly. Then I change the init code from
string currentDirectory = Environment.CurrentDirectory;
Environment.CurrentDirectory = new FileInfo(Assembly.GetEntryAssembly().Location).Directory.FullName;
string fullPath = Path.GetFullPath("bin/Program.dll");
Environment.CurrentDirectory = currentDirectory;
return PythonOps.InitializeModuleEx(Assembly.LoadFile(fullPath), "__main__", null, ignoreEnvVars: false, null);

to
return PythonOps.InitializeModuleEx(Assembly.GetExecutingAssembly(), "__main__", null, ignoreEnvVars: false, null);

since the ScriptCode assembly is itself.
But now it have some new problem. Since the AssemblyBuilder dose not have DefineIconResource method, I have no ideal to add win32 icon. And the Windows Defender said it is viruses when /standalone option is on...
PS. I use reflection to invoke SavableScriptCode.CompileForSave since SavableScriptCode is in DLR repository. It is better to add CreateAssemblyGen method for SavableScriptCode.

Looks fine with .NET Core App


react-native-window app crash when rendering svg as component and firing navigation.navigate

Description
desktop is crashing as soon as I fire navigation.navigate, it is happening if I am rendering svg as component, but works fine if I comment svg component.
Steps to reproduce
"dependencies": {
"@react-native-masked-view/masked-view": "^0.3.2",
"@react-navigation/native": "^7.0.13",
"@react-navigation/stack": "^7.0.18",
"react": "18.3.1",
"react-native": "0.76.3",
"react-native-gesture-handler": "^2.21.2",
"react-native-safe-area-context": "^4.14.0",
"react-native-screens": "^4.3.0",
"react-native-svg": "^15.10.1",
"react-native-windows": "^0.76.2"
},
"devDependencies": {
"@babel/core": "^7.25.2",
"@babel/preset-env": "^7.25.3",
"@babel/runtime": "^7.25.0",
"@react-native-community/cli": "15.0.1",
"@react-native-community/cli-platform-android": "15.0.1",
"@react-native-community/cli-platform-ios": "15.0.1",
"@react-native/babel-preset": "0.76.3",
"@react-native/eslint-config": "0.76.3",
"@react-native/metro-config": "0.76.3",
"@react-native/typescript-config": "0.76.3",
"@types/react": "^18.2.6",
"@types/react-test-renderer": "^18.0.0",
"babel-jest": "^29.6.3",
"eslint": "^8.19.0",
"jest": "^29.6.3",
"metro-config": "^0.81.0",
"prettier": "2.8.8",
"react-native-svg-transformer": "^1.5.0",
"react-test-renderer": "18.3.1",
"typescript": "5.0.4"
},
Snack or a link to a repository
currently not available
SVG version
15.10.1
React Native version
0.76.3
Platforms
macOS
JavaScript runtime
None
Workflow
None
Architecture
Fabric (New Architecture)
Build type
Debug app & dev bundle
Device
Real device
Device model
Win 10
Acknowledgements
Yes

FYI: the issue happened in windows desktop build.

this can be seen in visual studio, if we start debugging for the application
Exception thrown at 0x00007FFEB573FA4C in TnVForms.exe: Microsoft C++ exception: winrt::hresult_invalid_argument at memory location 0x0000007373AFD488.
Exception thrown at 0x00007FFEB573FA4C in TnVForms.exe: Microsoft C++ exception: [rethrow] at memory location 0x0000000000000000.
Exception thrown at 0x00007FFEB573FA4C in TnVForms.exe: Microsoft C++ exception: winrt::hresult_invalid_argument at memory location 0x0000007373AFDD08.
Exception thrown at 0x00007FFEB573FA4C in TnVForms.exe: Microsoft C++ exception: [rethrow] at memory location 0x0000000000000000.
A breakpoint instruction (__debugbreak() statement or a similar call) was executed in TnVForms.exe.
The program '[17304] TnVForms.exe' has exited with code 0 (0x0).

I think once an svg renders it is not unloading from memory on unmount, and that is creating memory leak and the crash, does no one come around with this issue? please help anyone ..

Hey! 👋
The issue doesn't seem to contain a minimal reproduction.
Could you provide a snack or a link to a GitHub repository under your username that reproduces the problem?

here is the repo link
https://github.com/akumar3408/svg-navigation-issue.git

anyone please?

CC @acoates-ms @marlenecota it seems to be a bug in the new implementation, is this something you could help with?

Hello @acoates-ms @marlenecota did you get any chance to check this issue?

Hey, anyone ???


fix(kvm): add encoding options to kvm page

Codecov Report

Merging #189 (18e4ba5) into master (7751332) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #189   +/-   ##
=======================================
  Coverage   50.36%   50.36%
=======================================
  Files          33       33
  Lines         818      818
  Branches       84       84
=======================================
  Hits          412      412
  Misses        375      375
  Partials       31       31

Impacted Files
Coverage Δ

src/app/devices/kvm/kvm.component.ts
67.61% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7751332...18e4ba5. Read the comment docs.


feat: add support for complex contracts with motsu

Currently default attribute cannot initialize complex contract (contract that contains child contract).
This pr should solve existing issue.
Follows up #51
PR Checklist

[-] Tests
[-] Documentation

In brief this pr eliminates DefaultStorageLayout attribute since StorageType has method StorageType::new(..). And it already implements construction in a "smart" recursive way

Those contracts that contains the other contracts inside.
Like this as I remember had issues with that macro
sol_storage! {     pub struct Erc721Consecutive {         Erc721 erc721;         Trace160 _sequential_ownership;         BitMap _sequentian_burn;         bool _initialized     } }

@ggonzalez94 So yeah. It was definitely my guess that DefaultStorageLayout macro will not work with complex types. That is why I hasn't checked it. And it works in reality. Since StorageType::new(..) already has recursive implementation.
If you didn't knew before how StorageType::new(..) is implemented with sol_storage macro. I think you would also expect that it won't work in this case.
If this pr doesn't add any new feature. I think I'll change title from feature to refactoring cc @alexfertel

Actually, do you mind fixing docs in this PR as well @qalisander. There are a few places where me have outdated comments like in the README and the lib.rs docs, etc.


Make sequence elements with no command hide the command bubble

People think it is broken when they see an empty bubble.

This has been resolved and integrated.


julia_17-bin: 1.7.0 -> 1.7.1

Note-to-self or anyone wanting to calculate the hash for this url, use:
nix-prefetch-url https://julialang-s3.julialang.org/bin/linux/x64/1.7/julia-1.7.1-linux-x86_64.tar.gz

without the --unpack flag.

Works for me. Thanks.

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

1 package built:

julia-bin (julia-stable-bin ,julia_17-bin)


Correct typo

The interactive parameter for use with docker container is "it", not "ti"

@way0utwest : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@way0utwest Thanks for catching this! #sign-off


feat: Docs search

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests

Description
see docs/readme.md
Use cases and why

One use case

Another use case

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

/draft

preview on https://yb-qwik.pages.dev/

Amazing job! However I am a bit concerned about the eagerness of executing all react to support this. I would look into QwikReact to not run jS until the user interacts with the searchbar

Amazing job! However I am a bit concerned about the eagerness of executing all react to support this. I would look into QwikReact to not run jS until the user interacts with the searchbar

and could I help with write whole widget in qwik at all ? maybe functionality & accessible feature not comparable to official one, but I will try my best.

progress update:  finished the MVP of search written in Qwik. Preview available in the same link https://yb-qwik.pages.dev/ . while it still can be improved for sophisticated UI, But I think it can be done later.
there is some issues currently I'm blocked by importing qwik component lib
when I import bundled component, showing following errors
Error: QWIK http://localhost:3000/src/docsearch_component_useclienteffect_zrgf0h9ksk4.js#_hW[2] does not export _hW

tried:

change vite.config.ts qwik plugin's option entryStrategy to hook works fine with app dev.ssr, but failed with app build
app directly import lib ts source files, works fine in dev.ssr & build, but failed with serve

can i see the repo of qwik-algolia search?

Found it! going to see if i can debug the issues with _hW make sure we are using the same version fo qwik, there are not cross compatibility yet

Thank you soo much! This was an amazing amount of work! Brilliant!