Factor out RawFileSystem.

(for #8995)
This separates the low-level filesystem operations from the high-level "fs utils".  The move to the VS Code FS API can then happen strictly inside RawFileSystem.  Also, this is part of what we reverted in #8970.
(Note that this branch is based on PR #9455.  I couldn't force CI to run against that PR so I'm hoping it's just a fluke, and can use this PR to run against CI.)

Codecov Report

Merging #9499 into master will decrease coverage by 0.08%.
The diff coverage is 37.85%.

@@            Coverage Diff             @@
##           master    #9499      +/-   ##
==========================================
- Coverage   60.94%   60.86%   -0.09%
==========================================
  Files         530      531       +1
  Lines       28628    28692      +64
  Branches     4338     4344       +6
==========================================
+ Hits        17448    17464      +16
- Misses      10229    10269      +40
- Partials      951      959       +8

Impacted Files
Coverage Δ

src/client/common/platform/types.ts
100% <ø> (ø)
:arrow_up:

src/client/common/platform/fs-paths.ts
71.92% <100%> (ø)
:arrow_up:

src/client/common/platform/fileSystem.ts
32.46% <32%> (+5.58%)
:arrow_up:

src/client/common/platform/fs-temp.ts
84.61% <84.61%> (ø)

src/client/common/utils/platform.ts
64.7% <0%> (-11.77%)
:arrow_down:

...lient/datascience/jupyter/kernels/kernelService.ts
54.92% <0%> (-5.8%)
:arrow_down:

src/client/interpreter/activation/service.ts
84.93% <0%> (-2.77%)
:arrow_down:

src/client/common/variables/environment.ts
89.61% <0%> (-2.6%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.36% <0%> (-2.28%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

... and 12 more

Continue to review full report at Codecov.

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


Trouble downloading the support package

In the Readme, you have this line...
Download the Python OS/X support package, and extract it. This will give you two frameworks.

    OpenSSL.framework
    Python.framework

When I download an extract it I see just one directory named "python" - is the link wrong? or am I missing something?
Other minor thing - the structure in the README differs from the cookiecutter output as a Resources directory is missing.

Okay, I got this working with a little fidling and ignoring the bit about frameworks - will send a pull request to clear up the docs.


Code link update

Hi,
Here is the GitHub repo for FuSAGNet (KDD22):
Thanks!

Thanks for the update, I have updated the repo. 😁

Update


Alert Facade doesn't work!

when I'm using alert()->message('Message', 'Optional Title');
it's work
but when I'm using Alert::message('Message', 'Optional Title');
it's not working
the session doesn't contain any sweet alert data When I'm using Facade
No error on Laravel
no error on console

same here.

@mosi3883 @robertotcestari
Are you importing the whole namespace for the facade?

I used use Alert;

same here :(

@mosi3883 Could you import the Facade using the whole namespace?
use UxWeb\SweetAlert\SweetAlert;

@mosi3883 Could you import the Facade using the whole namespace?
use UxWeb\SweetAlert\SweetAlert;
// or
use SweetAlert;

use UxWeb\SweetAlert\SweetAlert;
then using SweetAlert::message
doesnt work
use UxWeb\SweetAlert\SweetAlert;
then alert()->message('Message', 'Optional Title');
works
use SweetAlert;
then using SweetAlert::message
throws 'Class not found'
use Alert;
then Alert::message
doesn't work
use Alert;
then alert()->message('Message', 'Optional Title');
works

I decided to use main script and don't use packages so feel free to close this issue

@ijarinz
Facade and helper function usage are not related. If you want to use the facade just import it with:
use UxWeb\SweetAlert\SweetAlert;

// ...

SweetAlert::message(/*args*/);

// ...

If you want to use the helper function there's no need to import the Facade. Helper function is able to resolve the underlying service class.

im facing same problem, alert helper function works but facade doesnot work. im using laravel 8. Please any suggestion, why its behaving like this?

im facing same problem, alert helper function works but facade doesnot work. im using laravel 8. Please any suggestion, why its behaving like this?

use Auth;
Alert()->success('Message', 'Optional title');

@msaifull
It seems to be broken. I'm sorry, right I don't have the time to invest in this project, but you can send a pull request to fix it.
Thanks

@msaifull
The main problem is that time ago I added a __destruct() method in hope Laravel would call it and set the configuration on the session storage, but it seems Laravel is not calling this method.
Maybe the way to fix it is to call the flashConfig() method on each other method that mutates the configuration.


Support empty security in paths

I have a schema like this
  /api/account/change-email/:
    post:
      operationId: deprecated:change_email
      description: ''
      tags:
      - account
      requestBody:
        content:
          application/json:
            schema:
              $ref: '#/components/schemas/OldChangeEmailRequest'
          application/x-www-form-urlencoded:
            schema:
              $ref: '#/components/schemas/OldChangeEmailRequest'
          multipart/form-data:
            schema:
              $ref: '#/components/schemas/OldChangeEmailRequest'
        required: true
      security:
      - cookieAuth: []
security:
  - cookieAuth:
      type: apiKey
      in: cookie
      name: Session

When i try to validate my request i get error
>       raise InvalidSecurity()
E       openapi_core.validation.exceptions.InvalidSecurity: Security not valid for any requirement

I think because in my scheme is empty
      security:
      - cookieAuth: []

Could you help?

Hi @MissiaL
The following
security:
      - cookieAuth: []

means non-oauth security applied
If you want to disable security you must pass empty security list
security: []

Check Openapi3 documentation https://swagger.io/docs/specification/authentication/


Adds doc on how to make types printable

I spent a little while working my way through the source to find the comments about toString.  Overloading this function was the only way I was able to print a std::pair without reopening namespace std.  It's possible I've missed something, I'll be happy to amend it if you'd like to point me in the right direction.  I just created the document I was looking for before having to dive into the code.

Yeah that all looks perfect, thanks!


test wider use of pytree map

Usually the code around integrating equations and looping is array based, e.g
x += dt * f(x, t)

x needs to be array like and support arithmetic, which means you can't just drop a tuple in there and expect it to work, even though it's well defined in a "do what I mean sense".
concrete example
import jax, jax.numpy as np

def f(x, dx, dt=0.1):
    x += dx
    return x

x = np.ones(2)*0.2
dx = np.ones(2)*0.1
print(f(x, dx))  # typical so far

x = 0.2, np.ones(2)*0.2
dx = 0.5, np.ones(2)*0.1
print(f(x, dx)) # works but wrong (ﾉ☉ヮ⚆)ﾉ

# "what I mean" but w/ manual traversal (⌐■_■)
print( (f(x[0], dx[0]), f(x[1], dx[1])))

# "do what I mean" w/o manual traversal ʕっ•ᴥ•ʔっ
print(jax.tree_util.tree_map(f, x, dx))

outputs
[0.3 0.3]
(0.2, Array([0.2, 0.2], dtype=float32), 0.5, Array([0.1, 0.1], dtype=float32))
(0.7, Array([0.3, 0.3], dtype=float32))
(0.7, Array([0.3, 0.3], dtype=float32))

c98d0a2064 converts integrators to use tree_map so now models/equations with pytree states can be used.
I guess monitor support would be nice as well.


The cells are not scrolling correctly

The cells are not scrolling correctly.. Some of them not scrolled with others.. How can i fixed?
Check this images

@W7orld  HI,Have you solved this problem?Im also have this problem

@W7orld HI,Have you solved this problem?Im also have this problem

Hi, Not exactly but i used nestedscrollview and it's fine now it's works better
I putted the TableView inside Nestedscrollview

I have a similar issue. When I try using Nestedscrollview, the columns disappear. Did you come across similar issue @W7orld ?

@swatikch No.  Did you have other ViewObject inside the Nestedscrollview I mean with TableView dose he only inside Nestedscrollview?

@swatikch I also had this issue. I fixed it by making a separate xml file for the TableView and then inflating the TableView and adding it as a child of the NestedScrollView at runtime. Let me know if you have any problems.


Always bundle the theme CSS/JS

This removes the {% if READTHEDOCS %} conditionals around the theme JS/CSS. The goal here is to uncouple the code injected by readthedocs.org into this theme from the version of the CSS/JS on the theme itself.
This change will have a required change in the readthedocs.org repo which I will link when that PR is created. There is also an optional (but highly recommended) change coming in the readthedocs-sphinx-ext repo which I will also link to.

Here's the corresponding .org PR: https://github.com/rtfd/readthedocs.org/pull/3968
Without merging that PR, the theme JS will be included twice. This doesn't seem to break anything.
There's also a PR on the readthedocs-sphinx-ext repo: https://github.com/rtfd/readthedocs-sphinx-ext/pull/39
Without that PR, the theme CSS will be included twice. This doesn't break anything. One thing to note is that the CSS from media.readthedocs.org is included later in the DOM so it will take precedence over the theme bundled CSS. We should just merge the above PR to fix that.

Coupled with the other PRs, this fixes #610.

Will do. I will also update the changelog to mention that 0.3.0 has some issues.

Maybe we can remove the other 2 {% if READTHEDOCS %} sections, and have that be 1.0 :)

Maybe we can remove the other 2 {% if READTHEDOCS %} sections, and have that be 1.0 :)

Let's make that a goal for 1.0. However, I think targeting this for 0.4.0 would be good.


Update a number of Pirk's pom dependencies.

move Pirk to later versions of JMH, Hadoop, commons-math3, commons-net, json-simple, jacoco-maven-plugin, coveralls-maven-plugin, Surefire, maven-jar-plugin, and maven-release-plugin.

Note that Storm version 1.0.1 passes Pirk tests, but Storm version 1.0.2 fails with NoClassDefFoundError: org/apache/kafka/common/protocol/SecurityProtocol

+1 to merge

+1 will merge now


python.datascience.execSelectionInteractive exist in windows version but not at linux version

Environment data

VS Code version: 1.29.0
Extension version (available under the Extensions sidebar): 2019.1.0
OS and version: linux ubuntu 18.04
Python version (& distribution if applicable, e.g. Anaconda): python 3.5.2
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): virtualenv
Relevant/affected Python packages and their versions: all

Expected behavior
being able to run current selection in interactive window
Actual behavior
python.datascience.execSelectionInteractive exist in windows version but not at linux version
Steps to reproduce:

try to find run selection in interactive in keybinding with no success

You need to update your extension version. 2019.1.0 is before this command existed.


Energy & Heat capabilities

Switch IEnergyBlock and IHeatBlock as well as their Item counter parts to a more forge like implementation to reduce inheritance.
Wait until neoforge changes capability implementation!!!

nvm the current impl is fine


Support adding types for boot files (Axios, Vuex, etc)

As seen here, the AE should also check which boot files have been added from the Quasar CLI and add typings to those one too.

Example for i18n boot file
import messages from 'src/i18n';
import Vue, { VueConstructor } from 'vue';
import VueI18n from 'vue-i18n';

declare module 'vue/types/vue' {
  interface Vue {
    i18n: VueI18n;
  }
}

export default async ({ app, Vue }: { app: Vue; Vue: VueConstructor }) => {
  Vue.use(VueI18n);

  // Set i18n instance on app
  app.i18n = new VueI18n({
    locale: 'en-us',
    fallbackLocale: 'en-us',
    messages,
  });
};

After solving #31 if I run quasar dev I am getting
ERROR in src/boot/axios.ts(3,25):
TS7031: Binding element 'Vue' implicitly has an 'any' type.
ERROR in src/boot/i18n.ts(13,19):
TS7031: Binding element 'app' implicitly has an 'any' type.
ERROR in src/store/index.ts(21,5):
TS2322: Type 'string | undefined' is not assignable to type 'boolean | undefined'.
  Type 'string' is not assignable to type 'boolean | undefined'.
Version: typescript 3.5.3

... and it really hurts!  It's been half a day so far for me trying to get Typescript on Quasar (SPA + SASS + Axios + i18n) set up and I am constantly running into issues like ☝  with absolutely no example to follow.

Sorry you're having difficulties here. We're all volunteers working
together to try and move things forward, so please bear with us - we're
definitely NOT trying to hurt anybody!
We know that there needs to be better documentation, and that is coming
soon.

Also, until we gets the docs up to speed, we are pretty responsive to questions posted on the Quasar Discord Server typescript channel.

I'm solving it as follows:
export default async ({ Vue }: { Vue: any }) => {
  Vue.prototype.$axios = axios
}

Should be fixed by #48


[doc build] Use secrets

Companion pr to https://github.com/huggingface/doc-builder/pull/379
Please feel free to merge it yourself

Thanks a lot!


Update our team id for RoboCup 2022

It is 1, because we are number 1!!!
see here for team ids for this year:
https://github.com/RoboCup-Humanoid-TC/GameController/blob/master/resources/config/hl_kid/teams.cfg

LGTM :smile:
Just need to remember to change the robot_id for each robot before we begin play.


MemProcFS

Can you please add MemProcFS to your list of tools, for me it has been a gamechanger!
https://github.com/ufrisk/MemProcFS

Hello, @erdejong. That's a cool tool! I will include it in the 1.1 version. Stay tuned!
Thanks!

Closed 51392d0a4a2cd7c01a4e2ec9f8b8db2362e6b83d


[WIP] Adds Harbor Webhook Integration

The Harbor registry project has recently introduced webhooks to notify about pushed images, completed scans, etc.  This PR adds a webhook integration supporting the pushImage and scanningCompleted events. Support for the remaining events will be implemented as soon as Harbor allows for enabling/disabling individual event types.
Testing Plan:
The integration was tested using unit tests (see zerver/webhooks/harbor/tests.py) and manually using the Integrations Developer Panel.
GIFs or Screenshots:

@chgl awesome, thanks for working on this!  Looking at those screenshots, I'd love to include a link to the appropriate page in Harbor for more details on the results; is there a way to get such a link with their API?

@chgl awesome, thanks for working on this! Looking at those screenshots, I'd love to include a link to the appropriate page in Harbor for more details on the results; is there a way to get such a link with their API?

That would definitely be very convenient! Harbor currently does not provide a link back to an image as part of its event data. I've just created an issue for this, though: https://github.com/goharbor/harbor/issues/9487

I will definitely extend this once Harbor v1.11.0 hits, with support for selective webhooks and maybe even links back to the Harbor portal. Until then, the current implementation should be useful for some use cases and, if you're ok with it, @timabbott , can be merged.

Looks great, merged, squashed and merged thanks @chgl!
A follow-up PR once Harbor's webhooks can support nicer formatting would be great too :)


Improve rate limit explanation on “Add a repository collaborator”

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
Add a repository collaborator
What part(s) of the article would you like to see updated?
Rate limits on invitations of collaborators  to a repo
It seems unclear what kind of Invitation is meant here.
I tested sending 50 invitations, and it still does not limit me in any way.
Does it count for 50 individual invites?
Does it count for 50 accepted invites?
Or something else?
Would be nice to have that clarified.
Greetings Hecht.
Additional information
No response

@MikolaiHecht Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:


Add remove_proxies API for pallet-proxies #7557

Added a new method pub remove_all_proxy_delegates, moved the logic of remove_proxy there, and calling this remove_all_proxy_delegates from remove_proxies.
Fixes #7557

Hey, is anyone still working on this? Due to the inactivity this issue has been automatically marked as stale. It will be closed if no further activity occurs. Thank you for your contributions.

I will be working on it .

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2397934

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2397932

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2397943

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2410609

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable-int
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2471704

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2471708

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable-int
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2473446

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2473441

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable-int
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2475498

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2475502

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2879853

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2880414

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2880624

bot rebase

@juangirini I dismissed @ggwpez's review since his suggestions have been addressed.
You should be able to merge now.

bot merge


Please update React.findDOMNode to ReactDOM

React.findDOMNode is deprecated/replaced

Thanks for reporting! Can you link to the files that have these problem? The search string "findDOMNode" returns empty on this repository.

Closing as this is not an issue with our repository.


Support for self-hosted Windows runners

I tried this action on self-hosted runner and while Ubuntu worked, Windows did't:

##[debug]Evaluating condition for step: 'Install cargo-nextest'
##[debug]Evaluating: success()
##[debug]Evaluating success:
##[debug]=> true
##[debug]Result: true
##[debug]Starting: Install cargo-nextest
##[debug]Loading inputs
##[debug]Loading env
Run taiki-e/install-action@0163f6cf65d9b9bb0e5d1f0e3ae19280a06be4b0
##[debug]Evaluating: inputs.tool
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'tool'
##[debug]=> 'cargo-nextest'
##[debug]Result: 'cargo-nextest'
##[debug]Evaluating: inputs.checksum
##[debug]Evaluating Index:
##[debug]..Evaluating inputs:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'checksum'
##[debug]=> 'true'
##[debug]Result: 'true'
##[debug]Evaluating condition for step: 'run'
##[debug]Evaluating: success()
##[debug]Evaluating success:
##[debug]=> true
##[debug]Result: true
##[debug]Starting: run
##[debug]Loading inputs
##[debug]Loading env
Run bash --noprofile --norc "${GITHUB_ACTION_PATH:?}/main.sh"
##[debug]C:\cygwin64\bin\bash.EXE --noprofile --norc -e -o pipefail C:\actions-runner\_work\_temp\74c519f7-435e-4f1d-9e43-a0fae0148bd9.sh
info: install-action does not support cargo-nextest; fallback to cargo-binstall
info: installing cargo-binstall
info: downloading null
curl: (3) URL using bad/illegal format or missing URL
curl: (3) URL using bad/illegal format or missing URL
curl: (3) URL using bad/illegal format or missing URL
curl: (3) URL using bad/illegal format or missing URL
curl: (3) URL using bad/illegal format or missing URL
curl: (3) URL using bad/illegal format or missing URL
curl: (3) URL using bad/illegal format or missing URL
curl: (3) URL using bad/illegal format or missing URL
curl: (3) URL using bad/illegal format or missing URL
curl: (3) URL using bad/illegal format or missing URL
curl: (3) URL using bad/illegal format or missing URL
Error: Process completed with exit code 3.
##[debug]Finished: run
##[debug]Finishing: Install cargo-nextest

Would be convenient if it was supported in this environment as well.

Thanks for the report! There may be some issue with jq or path handling. Could you try aa0fb41b4db2493d2ef6e45c478fb01378bc2809?

host platform: x86_64_windows
jq version: jq-1.6

https://github.com/subspace/subspace/actions/runs/6188492029/job/16800658877#step:7:1

host platform: x86_64_windows
jq version: jq-1.6
info: install-action does not support cargo-nextest; fallback to cargo-binstall
info: installing<EMAIL_ADDRESS>info: downloading null

Hmm. jq at least succeeds in parsing the manifest the first and second time (since "1.3.0" is parsed), but the subsequent comparison to "null" (even though the actual value is "null") appears to be evaluated as false.
https://github.com/taiki-e/install-action/blob/c33cd1d5e1ef655caac77b00640bbbd1731caa2b/main.sh#L209
Given that the jq version is 1.6 and that a flag was added in 1.7 to fix the carriage return problem, I guess this is probably related to https://github.com/jqlang/jq/issues/1854.
Could you try c86cf7e603a60840f49eb6ec40eadb133072f91d?

I don't see anything helpful in the output: https://github.com/subspace/subspace/actions/runs/6188821326/job/16801617758#step:7:1
jq 1.7 if fairly new, I think there are plenty of environments where it is not yet present.

https://github.com/jqlang/jq/issues/1854 appears to indicate that the problem exists only with a particular type of jq binary, and the curl error (unable to write to hidden directory) appears to be specific to a particular build way. (Both work fine, at least when using the binaries included in the GitHub-provided runners.)
Could you try 77363e9d6d7d3414223a8f118f39650fb2d9c553? (added potential workaround for curl hidden directory issue)

Hm...
host platform: x86_64_windows
jq version: jq-1.7-dirty
info: install-action does not support cargo-nextest; fallback to cargo-binstall
info: installing<EMAIL_ADDRESS>info: downloading https://github.com/cargo-bins/cargo-binstall/releases/download/v1.3.0/cargo-binstall-x86_64-pc-windows-msvc.zip
info: verifying sha256 checksum for cargo-binstall-x86_64-pc-windows-msvc.zip
info: cargo-binstall installed at
+ cargo binstall -V
error: no such command: `binstall`

	Did you mean `install`?

	View all installed commands with `cargo --list`
Error: Process completed with exit code 101.

https://github.com/subspace/subspace/actions/runs/6189004596/job/16802168830#step:7:1

Didn't work either:
host platform: x86_64_windows
jq version: jq-1.7-dirty
info: install-action does not support cargo-nextest; fallback to cargo-binstall
error: no such command: `binstall`

	Did you mean `install`?

	View all installed commands with `cargo --list`
info: cargo-binstall already installed at /usr/local/bin/cargo-binstall.exe, but is not compatible version with install-action, upgrading
info: installing<EMAIL_ADDRESS>info: downloading https://github.com/cargo-bins/cargo-binstall/releases/download/v1.3.0/cargo-binstall-x86_64-pc-windows-msvc.zip
info: verifying sha256 checksum for cargo-binstall-x86_64-pc-windows-msvc.zip
Warning: cargo-binstall should be installed at /usr/local/bin; but cargo-binstall.exe not found in path
+ cargo binstall -V
error: no such command: `binstall`

	Did you mean `install`?

	View all installed commands with `cargo --list`

https://github.com/subspace/subspace/actions/runs/6189176189/job/16802753709#step:7:1
P.S. Every time I have an issue with CI it is never an exact science, but rather trial and error.

I found a bug related to determining the installation location that I can reproduce in other Windows environments.
I think that is the cause of the remaining problems.
Could you try cc5181c92a8beda8da32d4bd837b8b9c700ab348?

Something new this time:
host platform: x86_64_windows
jq version: jq-1.7-dirty
info: install-action does not support cargo-nextest; fallback to cargo-binstall
info: installing<EMAIL_ADDRESS>info: downloading https://github.com/cargo-bins/cargo-binstall/releases/download/v1.3.0/cargo-binstall-x86_64-pc-windows-msvc.zip
info: verifying sha256 checksum for cargo-binstall-x86_64-pc-windows-msvc.zip
info: cargo-binstall installed at /home/Administrator/.install-action/bin/cargo-binstall.exe
+ cargo binstall -V
error: could not execute process `C:\cygwin64\home\Administrator\.install-action\bin\cargo-binstall.exe binstall -V` (never executed)

Caused by:
  Access is denied. (os error 5)

https://github.com/subspace/subspace/actions/runs/6189633359/job/16804165414#step:7:1

I changed to prefer $RUNNER_TOOL_CACHE over $HOME in eb50d20172d8ae247b38475789825a64edef5ee0, so the permission issue should be fixed.

Nope, same error again: https://github.com/subspace/subspace/actions/runs/6190541151/job/16806980662#step:7:1

No, updated commit didn't work either with a different error:
info: host platform: x86_64_windows
info: cargo is installed at /cygdrive/c/Users/Administrator/.cargo/bin/cargo
info: jq version: jq-1.7-dirty
info: install-action does not support cargo-nextest; fallback to cargo-binstall
info: installing<EMAIL_ADDRESS>info: downloading https://github.com/cargo-bins/cargo-binstall/releases/download/v1.3.0/cargo-binstall-x86_64-pc-windows-msvc.zip
info: verifying sha256 checksum for cargo-binstall-x86_64-pc-windows-msvc.zip
Warning: cargo-binstall should be installed at /c/actions-runner/_work/_tool/install-action/bin/cargo-binstall.exe; but cargo-binstall.exe not found in path
+ cargo binstall -V
error: no such command: `binstall`

	Did you mean `install`?

	View all installed commands with `cargo --list`
Error: Process completed with exit code 101.

https://github.com/subspace/subspace/actions/runs/6190591939/job/16807141866#step:7:1

Hmm, sorry, that did not seem to be enough to fix the problem. (On second thought, if installing on HOME was really the problem, then the jq run would have failed as well)
The difference between the way cargo-binstall and jq are executed is whether it goes through cargo or not. I have created a patch (79a868a4c0889f8bc9ec358aec0009aa0a887db1) that calls cargo binstall without going through cargo, could you try that?

Still permission denied:
+ cargo-binstall binstall -V
C:\actions-runner\_work\_actions\taiki-e\install-action\79a868a4c0889f8bc9ec358aec0009aa0a887db1/main.sh: line 11: /home/Administrator/.install-action/bin/cargo-binstall: Permission denied
Error: Process completed with exit code 126.

https://github.com/subspace/subspace/actions/runs/6190724569/job/16807554656
Is there something like chmod +x on Windows? I didn't use Windows for anything work-wise for ~10 years now.

Is there something like chmod +x on Windows?

Good point. I found https://cygwin.com/cygwin-ug-net/using-filemodes.html

On FAT or FAT32 filesystems, files are always readable, and Cygwin uses the DOS read-only attribute to determine if they are writable. Files are considered to be executable if the filename ends with .bat, .com or .exe, or if its content starts with #!.

Whether or not the .exe is used at the time of the call is indeed another difference between the way cargo-binstall and jq were executed...
I updated my patch to use .exe (84cf87d08640570a4867937b58e6cab8d5b8fed9).

info: host platform: x86_64_windows
info: cargo is located at /cygdrive/c/Users/Administrator/.cargo/bin/cargo
info: jq version: jq-1.7-dirty
info: install-action does not support cargo-nextest; fallback to cargo-binstall
info: installing<EMAIL_ADDRESS>info: downloading https://github.com/cargo-bins/cargo-binstall/releases/download/v1.3.0/cargo-binstall-x86_64-pc-windows-msvc.zip
info: verifying sha256 checksum for cargo-binstall-x86_64-pc-windows-msvc.zip
Warning: cargo-binstall should be installed at /home/Administrator/.install-action/bin/cargo-binstall.exe; but cargo-binstall.exe not found in path
+ cargo-binstall.exe binstall -V
C:\actions-runner\_work\_actions\taiki-e\install-action\84cf87d08640570a4867937b58e6cab8d5b8fed9/main.sh: line 11: cargo-binstall.exe: command not found
Error: Process completed with exit code 127.

https://github.com/subspace/subspace/actions/runs/6190869896/job/16808019785#step:7:1

Oh, that's probably a bug about adding PATHs, and it should be fixed in 6b495ce909cc90247ed5baa261cf4bcf03f75eff.

Back to permission denied:
info: host platform: x86_64_windows
info: cargo is located at /cygdrive/c/Users/Administrator/.cargo/bin/cargo
info: jq version: jq-1.7-dirty
info: install-action does not support cargo-nextest; fallback to cargo-binstall
info: installing<EMAIL_ADDRESS>info: adding \home\Administrator\.install-action\bin to PATH
info: downloading https://github.com/cargo-bins/cargo-binstall/releases/download/v1.3.0/cargo-binstall-x86_64-pc-windows-msvc.zip
info: verifying sha256 checksum for cargo-binstall-x86_64-pc-windows-msvc.zip
info: cargo-binstall installed at /home/Administrator/.install-action/bin/cargo-binstall.exe
+ cargo-binstall.exe binstall -V
C:\actions-runner\_work\_actions\taiki-e\install-action\6b495ce909cc90247ed5baa261cf4bcf03f75eff/main.sh: line 11: /home/Administrator/.install-action/bin/cargo-binstall.exe: Permission denied

https://github.com/subspace/subspace/actions/runs/6191193015/job/16808987474#step:7:1

Hmm...

One of the other differences between jq and cargo-binstall is whether it is called with the full path or not, so changing that (582603535a55cc92add685115f1d18f98c8b020e) might work, but it probably would not resolve the underlying problem (except when there was a problem only with cargo-binstall).
Another possibility is that there is a problem with the unzipping process of cargo-binstall's zip file or with the binaries themselves. It might be helpful to see if cargo-llvm-cov, which uses tar.gz, or shfmt, which installs binaries directly, also has problems. (In this case, we may want to revert https://github.com/taiki-e/install-action/pull/183)

That said, it may be better here to try taiki-e/cache-cargo-install-action, which was created to support cases where install-action does not work.

/@nazar-pc Could that be the windows defender or other defending mechanism kicking in

Full path shouldn't be necessary. Just tried taiki-e/cache-cargo-install-action and it worked just fine, will go with it for now.

@nazar-pc Could that be the windows defender or other defending mechanism kicking in?

Anything is possible, asked DevOps person on our team to take a look there.

Maybe related: https://github.com/microsoft/winget-cli/issues/228


improve ss3 workflows

The no est and with est jobs could be combined, with an argument to tell which version to use.

There is also some overlap with the test for user-examples. Could make this a more general workflow to run Stock Synthesis models in any directory


Get STATE from the /scheduler/jobs Request

From a previous comment it looks like you can get the state of the job by calling /scheduler/graph/csv, and this only returns some data of the graph:
node,myjob1,fresh,running
node,myjob2,failure,idle
This would be a little tedious to parse, and would be an additional call.
I am a little surprised that the 'State' (i.e Running, Idle) is not being returned in the job JSON.  Can this please be added to the response, as this is a critical piece of info when consumed downstream, and should be a quick fix.  Unless I am missing something here.
Thanks!

+1

Interested to this

+1


elastic dump fails if indice having huge data

In order to help us troubleshoot issues with this project, you must provide the following details:

ElasticDump version 3.3.14
Elasticsearch version 2.x
NODE_JS_MAJOR_VERSION=8
-PYTHON_VERSION=3.5.5
-PYTHON_PIP_VERSION=10.0.1
-NODE_OPTIONS=--max_old_space_size=536870912
elasticdump

--limit=500
--input=https://estrialdata001A.mgmt.tv3cloud.net:9200/spark_app_log-2018.06.17
--output=$
--httpAuthFile=/tmp/tmpkj9uyvdg
--type=data

Error:
"events.js:183\n      throw er; // Unhandled 'error' event\n      ^\n
\nError: write ENOBUFS\n    at _errnoException (util.js:992:11)\n    at Socket._writeGeneric
\ (net.js:764:25)\n    at Socket._writev (net.js:778:8)\n    at doWrite (_stream_writable.js:395:12)\n
\    at clearBuffer (_stream_writable.js:501:5)\n    at onwrite (_stream_writable.js:449:7)\n
\    at WriteWrap.afterWrite [as oncomplete] (net.js:876:12)\n"

This is really a memory issue and not exactly an elastic-dump issue. There is so much that back-pressuring can do, but eventually if there is too much data it will fail. My advice, use searchBody to query subsets.

I am new to elastic . Please help me
My requirement is to dump the complete data from existing 2.x server.For indices with less volume of data, everything works fine, but when i tried to dump an indice of size 25 gb , it fails.
Please help what property i need to set (Are talking about ram? if yes, currently machine using 16gb ram).
NODE_OPTIONS=--max_old_space_size=536870912

Try writing to a file, but I remember dumping a large index (10GB) and it failed due to same memory issues. Back-pressure works for node, but when the output is piped. The post-processor might not have a large enough buffer to handle the output.
I am thinking of implementing a segment/split feature, to split after X MB/GB or N records.

We are dumping records into gzip file and we are splitting files  for every 1 million records.

@manuchitturi How are you doing the split? Currently running into the same issue, did you solve it?

Hi I am trying to dump a index of size about 10 GB using the following node.js code.
const exportData = () => {
    return new Promise((resolve, reject) => {
      const authOptions = `${USERNAME}:${PASSWORD}`;
      const authUrl = `http://${authOptions}@${HOST}/${INDEX}`;

      const dataDump = spawn(ELASTICDUMP_PATH, ['--input=' + authUrl, '--output=$']);
      const writeStream = fs.createWriteStream(DATA_FILE);

      dataDump.stdout.pipe(writeStream);

      dataDump.on('close', (code) => {
        if (code !== 0) {
          reject(`Data dump process exited with code ${code}`);
        } else {
          resolve();
        }
      });
      dataDump.on('error', (err) => reject(`Data dump error: ${err}`));
    });
  };

the file size has got above 17 GB, and it's sill running. I am not getting what's the problem here?

An Elasticsearch index is compressed data. 10 GB might be 25 GB uncompressed. There are various issues here.

Memory (machine or node limitation hence the max_old_space_size, which is only useful if you're reading faster than writing)
?Node string length limitation. If you're dumping too much string data at once.
FS limitation. Since the file has to be buffered and written to, sometimes appending might be a long operation, hence the file-size param to limit the overall output size of each file (splitting the files.)

i.e. It's not a one size fits all scenario


omegaconf.errors.ConfigAttributeError: Key 'input_feat_per_channel' is not in struct

🐛 Bug

I Have been training the wav2vec2 model. I have done pertaining and fine-tuning. Right now I want to run the inference for this which was earlier speech_recogntion, which has now been merged into speect_to_text.
I am trying to run the inference. But I am getting the error, that 'input_feat_per_channel' key is not in my model's configuration file.
To Reproduce
fairseq-interactive data_path \
--config-yam config_path+config.yaml \
--task speech_to_text \
--path path_to_checkpoint.pt \
--max-tokens 50000 --beam 4 --scoring wer

My code
fairseq-interactive /home/saurabh/python/fairseq/fairseq/examples/my_exp/small_dataset/train \
--config-yaml /home/saurabh/python/fairseq/fairseq/examples/wav2vec/config/pretraining/wav2vec2_base_librispeech.yaml \
--task speech_to_text \
--path /home/saurabh/python/fairseq/fairseq/examples/my_exp/saved_model/pre_train/checkpoint/checkpoint_best.pt \
--max-tokens 50000 --beam 5 --scoring were

Error which I am getting

2023-01-04 12:37:49 | INFO | fairseq_cli.interactive | loading model(s) from /home/saurabh/python/fairseq/fairseq/examples/vakyansh-wav2vec2-experimentation/checkpoints/finetuning/checkpoint_best.pt
Traceback (most recent call last):
  File "/home/saurabh/python/fairseq/bin/fairseq-interactive", line 8, in <module>
    sys.exit(cli_main())
  File "/home/saurabh/python/fairseq/fairseq/fairseq_cli/interactive.py", line 313, in cli_main
    distributed_utils.call_main(convert_namespace_to_omegaconf(args), main)
  File "/home/saurabh/python/fairseq/fairseq/fairseq/distributed/utils.py", line 369, in call_main
    main(cfg, **kwargs)
  File "/home/saurabh/python/fairseq/fairseq/fairseq_cli/interactive.py", line 145, in main
    models, _model_args = checkpoint_utils.load_model_ensemble(
  File "/home/saurabh/python/fairseq/fairseq/fairseq/checkpoint_utils.py", line 367, in load_model_ensemble
    ensemble, args, _task = load_model_ensemble_and_task(
  File "/home/saurabh/python/fairseq/fairseq/fairseq/checkpoint_utils.py", line 473, in load_model_ensemble_and_task
    model = task.build_model(cfg.model, from_checkpoint=True)
  File "/home/saurabh/python/fairseq/fairseq/fairseq/tasks/speech_to_text.py", line 126, in build_model
    args.input_feat_per_channel = self.data_cfg.input_feat_per_channel
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/dictconfig.py", line 285, in __setattr__
    raise e
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/dictconfig.py", line 282, in __setattr__
    self.__set_impl(key, value)
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/dictconfig.py", line 266, in __set_impl
    self._set_item_impl(key, value)
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/basecontainer.py", line 398, in _set_item_impl
    self._validate_set(key, value)
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/dictconfig.py", line 143, in _validate_set
    self._validate_set_merge_impl(key, value, is_assign=True)
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/dictconfig.py", line 176, in _validate_set_merge_impl
    target = self._get_node(key)
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/dictconfig.py", line 383, in _get_node
    self._validate_get(key)
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/dictconfig.py", line 135, in _validate_get
    self._format_and_raise(
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/base.py", line 95, in _format_and_raise
    format_and_raise(
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/_utils.py", line 694, in format_and_raise
    _raise(ex, cause)
  File "/home/saurabh/python/fairseq/lib/python3.10/site-packages/omegaconf/_utils.py", line 610, in _raise
    raise ex  # set end OC_CAUSE=1 for full backtrace
omegaconf.errors.ConfigAttributeError: Key 'input_feat_per_channel' is not in struct
	full_key: model.input_feat_per_channel
	reference_type=Any
	object_type=dict

Environment

fairseq Version (e.g., 1.0 or main): 0.12.1+d11b8fb
PyTorch Version (e.g., 1.0): 1.13.0
OS (e.g., Linux): Ubuntu 22.04
How you installed fairseq (pip, source): pip + source
Python version: 3.10.6
GPU models and configuration: NVIDIA GM108M [GeForce 940MX]

Additional context

Same problem has been bothered me. Please let me know if it is already solved!!!


renderDetailPanel get called twice for each row, when expand the detail view

material-react-table version
v1.3.16
react & react-dom versions
v18.2.0
Describe the bug and the steps to reproduce it

go to example https://stackblitz.com/edit/github-tcjwan-vm59w3?file=src%2FTS.tsx,
open the browser debugger with console tab
expand one node in the table
you can see renderDetailPanel for each row is called twice

Minimal, Reproducible Example - (Optional, but Recommended)
https://stackblitz.com/edit/github-tcjwan-vm59w3?file=src%2FTS.tsx
Screenshots or Videos (Optional)

Do you intend to try to help solve this bug with your own PR?
No, because I do not know how
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

What problem does this cause?

Just tested this out by removing React Strict Mode and using React 17 and it is fine. Only renders once.
https://stackblitz.com/edit/github-tcjwan-jnglrr?file=package.json,src%2Fmain.tsx
React 18 now renders components twice in strict mode (and probably future versions of react), but only once in when the app is compiled and running in production.
If you want to change how React renders components, you can get rid of Strict mode in your project.


Allow use as main formatter

Hi! I was wondering if it could be possible to integrate the Spotless formatting with the "standard" formatting system in the IDE, so we don't have to remember using a dedicated shortcut.
It should be relatively easy. Looking at the ktfmt plugin, the steps seem to be:

Registering a formattingService extension in plugin.xml:
<extensions defaultExtensionNs="com.intellij">
   <formattingService implementation="..."/>
   <!-- ... -->
</extensions>

Implement com.intellij.formatting.service.AsyncDocumentFormattingService

Adding a setting to enable replacing the default formatter with Spotless

I don't think I'll have time to implement this by myself any time soon, but maybe someone else can. At least I thought I should test the waters and see if y'all like the idea :)

Hi @rock3r!
ktfmt is a code formatter, where spotless is a tool gives you the ability to use different formatters with own configuration.
That's why, there is no way to call spotless apply function somehow apart gradle task.
Plugin already supports "On Save" action allows you to apply spotless automatically (https://github.com/lipiridi/spotless-applier/issues/16) - https://github.com/lipiridi/spotless-applier?tab=readme-ov-file#action-on-save
In addition, you can just override the shortcut to any other you want:

Thanks @lipiridi for the answer! I understand the technical constraints. As for the shortcut, that isn't something I had considered; however that is a global setting and not all projects I work on use spotless or ktfmt, unfortunately, so I can't replace the built-in formatter. I suppose I need to use the alternative shortcut in those projects, then.

Also wanted to say for the history - built-in formatters should support ability to format few files, when spotless gradle task supports either entire project or single file. But calling gradle task for each file in the package (for example) is much expensive and doesn't make sense.
Hope someday someone will write a useful connector to avoid gradle at all🙂


[Issue] Anchor output reserve - can't open channels

Background
Tried to open a channel, peer is online, I have enough funds in onchain, like 2.7M sats in 1 UTXO.
I always get an error that doesn't say too much:
reserved wallet balance invalidated
Your environment

LND v.13.3
Debian 10
Umbrel node v 0.4.4 (latest)
35 other channels open, working fine, routing etc, with  total liquidity of almost 30M sats

Steps to reproduce
I used RTL, Thunderhub, Umbrel dashboard, lncli, Zap mobile/desktop, Zeus to try to open a channel of 2 or 2.5M sats
Error in RTL: reserved wallet balance invalidated
Error in Thunderhub: failed to open channel
Umbrel: reserved wallet balance invalidated
Zap, Zeus: reserved wallet balance invalidated
lncli: rpc error: code = Unknown desc = reserved wallet balance invalidated
Whatever amount I put, is the same message, that doesn't say too much, what should I do or how to fix it.
I looked for more documentation and I found that is due to new feature release of anchor output reserve, but there is no place to explain how to deal with this "reserve", there's no specific amount that user can know to keep or use in "reserve", nothing.
I tried also disabling this feature editing the lnd.conf file with
[protocol]
protocol.no-anchors=true
restart lnd, tried again, same error
Expected behaviour

To have a damn explanatory message saying what to do or calculate or send to specific documentation.
Leave only the commit fees to calculate this reserve and deduct from the amount the necessary fee/reserve
make a way that user will see and label that specific reserve in a UTXO and be clear that is not to be touched/moved from that wallet. Also to not be reflected into "available balance".

Actual behaviour
Can't open channels and also I don't know or calculate how much I should have in "reserve".
Right now is just confusing, how much is in "reserve" and how much is available to spend. nobody knows that, none of wallet apps is showing that.
This will just create more issues and chaos with so many noobs running nodes.
Please advise how to deal with this. Make a specific documentation for users, it is very confusing.
Additional references
https://bitcoinops.org/en/topics/anchor-outputs/
https://github.com/lightningnetwork/lnd/issues/5599
https://github.com/lightningnetwork/lnd/issues/5648
I also posted some requests / fixes to these apps, to see if more people come up with a nice solution:
https://github.com/Ride-The-Lightning/RTL/issues/833
https://github.com/apotdevin/thunderhub/issues/352
https://github.com/getumbrel/umbrel/issues/1056
EDIT
additional test:
I sent 100k sats to a new UTXO to the node wallet.
Still doesn't let me to open a channel with the 2.7M sats, whatever amount I try to use.
This is getting ridiculous.
Another question: when I open a channel how do I know that is not taking from this "reserve" UTXO, how do I make coin control when I open a channel in TH, RTL, Zeus, Zap etc...

Ok, as I mentioned in edit part, I sent to another UTXO 100k (that is supposedly the higher limit to have as reserve).
Is still not letting me to open any channel of any amount.
How can I disable this anchor?
As I mention I added to lnd.conf file that option mentioned in the documentation
[protocol]
protocol.no-anchors=true
But seems that is ignoring it.
What other option I can put to skip the anchor output?
You said to have multiple UTXOs. But how will be decided, next time when I will open another channel? What is are used all of them?

You said to have multiple UTXOs. But how will be decided, next time when I will open another channel? What is are used all of them?

After the transaction has been crafted, so if it ends up combining that UTXO, then it'll run into the same condition on anything before master or 0.14.

Read my message again, it's a bug that's been fixed. You do in fact have coin selection if you use PSBT to fund the transaction or any one of the manual means.

What other option I can put to skip the anchor output?

That's the correct option, but once you've opened a single anchor channel, then system will ensure you have UTXO reserved since if you empty the wallet, fee bumping would require manual intervention.
Closing the issue as it's been resolved.

The fix of this issue deserve a new version. Lot's of our users have issues caused because of this.

@Darth-Coin are you able to repro w/ lnd 0.14?

@Darth-Coin are you able to repro w/ lnd 0.14?

Thanks for asking. Not yet. I am waiting for Umbrel devs to update the backend to v0.14.
I will come back to you after that.
What should be the behavior now?
Right now I have in onchain balance 3 UTXOs: 1x100k, 1x200k, 1x36k change.
In lnd.conf file I have protocol.no-anchors=true
Questions:

would be able to open a channel with all those UTXOs leaving empty the onchain?
would be able to deposit to a new UTXO 1M sats and open only with that UTXO 1 channel?
would be those 3 UTXO linked to all open channels? Can be that traceable?
would be possible to have some kind of coin control for all these anchor UTXOs? Like not showing "available" in total balance? Or at least a mention/sign "these funds are reserved", so all noob user will understand why is not possible to use them.

Hello, I'm getting Unable to open channel, reserved wallet balance invalidated: transaction would leave insufficient funds for fee bumping anchor channel closings (see debug log for details) when I try to open a new channel. Currently on 0.14.1-beta (latest). As @Darth-Coin mentioned, it would be very helpful to have some explanation or documentation regarding this message. Where can I disable anchor channels? What's the minimum UTXO balance that I need to have?

Where can I disable anchor channels?

Adding protocol.no-anchors=true in lnd.conf solved it for me. New channel is now opening.

Hello, I'm getting Unable to open channel, reserved wallet balance invalidated: transaction would leave insufficient funds for fee bumping anchor channel closings (see debug log for details) when I try to open a new channel

Anchor channels allow your node to increase the fee of a published commitment transaction. In order to do that, your node needs to maintain a certain amount of funds on-chain. If you ensure you have at least a single 100k satoshi output on-chain, then you'll be able to use everything as normal.

Adding protocol.no-anchors=true in lnd.conf solved it for me. New channel is now opening.

Adding that doesn't disable this check. The check is still executed as long as any anchor channels are open. Creating new channels will increase the amount of reserved funds up to 100k sats total. Note that anchor channels are safer than non-anchor, as they actually allow the commitment fee to be increased in a high fee rate environment. Otherwise, it's possible that you aren't able to publish your commitment transaction (to enter the mempool), meaning it can't be mined.

Thank you for your explanation. I understand it now.

@Darth-Coin are you able to repro w/ lnd 0.14?

Yes, today I could test it.
Umbrel with LND v0.14.1
Had 5 UTXOs with various amounts and 1 UTXO with 100k sats. Total was like 665000 sats
I tried first with all balance to open the channel, It came the message that I need more sats for anchor reserve
So I open a channel of 565000 sats.
I was expecting to get an error for insufficient funds, because of taking the fee from that 100k UTXO, but it didn't.
Channel is open and a small fee was taken from that last 100k UTXO.

Just wanted to add as Darth said above that I had exactly the same experience on v0.14.1, today.
using bos to batch open 3 channels, utilizing pretty much all of my utxos (pretty careless really)
Anyway, Just ran the same open with one channel at a significantly smaller size to cover the necessary buffer,
all channels opened fine.
Would be good to have a small addition to the error to know that you can rerun the open at a reduced size,
or some note explaining funds are probably not lost.

@Darth-Coin are you able to repro w/ lnd 0.14?

Thanks for asking. Not yet. I am waiting for Umbrel devs to update the backend to v0.14. I will come back to you after that.
What should be the behavior now? Right now I have in onchain balance 3 UTXOs: 1x100k, 1x200k, 1x36k change. In lnd.conf file I have protocol.no-anchors=true
Questions:

would be able to open a channel with all those UTXOs leaving empty the onchain?
would be able to deposit to a new UTXO 1M sats and open only with that UTXO 1 channel?
would be those 3 UTXO linked to all open channels? Can be that traceable?
would be possible to have some kind of coin control for all these anchor UTXOs? Like not showing "available" in total balance? Or at least a mention/sign "these funds are reserved", so all noob user will understand why is not possible to use them.

I wonder if all these genuine questions are solved with the latest version... Im experiencing the same when trying to open a channel with a single utxo in my wallet atm...


[iOS] Check if Xcode is installed in one-click deploy code.

Adds a Xcode install check to avoid unnecessary popups asking to install Xcode command line tools while enumerating running iOS simulator devices.
See https://github.com/godotengine/godot/issues/85114#issuecomment-1819981211

Thanks!


Bug: Replace text with icons

Replace the text with icons

Add Screenshots

The current behavior
Not professional

The expected behavior
better outlook of the website

Please assign this bug to me I am intereseted in resolving this bug.


EventHub Triggered Function runs very long, sometimes shows as "Never Finished"

Local testing of a webjob executable did not expose a problem where Function executions never finish.
Repro steps
Provide the steps required to reproduce the problem
WebJob Function code snippet:
public static void ProcessIncomingSignal(
            [EventHubTrigger(Program.EventHub)] string signal,
            [ServiceBus("downstream")] out BrokeredMessage outboundMessage)
        {
            outboundMessage = StaticObject.SynchronousMethod(Guid.NewGuid(), signal, "string label");
        }

Take string content from EventHubTriggered feed.

Apply business logic that includes storing a blob and creating a table row.

Output Servicebus brokered message onto queue for downstream processing.

Expected behavior
When run locally I get console logs like: Executed: 'Functions.****' (Succeeded) within a few seconds of function invocation.
Actual behavior
In the WebJob dashboard I see that function invocations be in "Running" state for upwards of 20 minutes before being moved to "Never Finished". It seems random, I've seen "Never Finished" after fewer than 5 minutes of run time.
Known workarounds
Please inform
Related information
Provide any related information

C#
Closed source MS internal project in OneBranch, contact me for source access.

Are you using Azure Functions or Web Jobs?  @MikeStall have you already been investigating this?

This looks like a SDK issue, not Script (see presence of C# attributes, and reference to "NeverFinished" status which isn't showing in Functions at the time of bug)
It looks like the host must be crashing in the middle of processing. This could be another case of the host getting starved out (CPU / network intensive) and then crashing (https://github.com/Azure/azure-webjobs-sdk/issues/899 )

@andunn Can you confirm that you're using WebJobs and not Azure Functions?

Yes, I’m using web jobs.
Happy Coding!
From: Donna Malayeri<EMAIL_ADDRESS>Sent: Monday, December 19, 2016 17:25
To: Azure/azure-webjobs-sdk-script<EMAIL_ADDRESS>Cc: Andrew Dunnells<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-webjobs-sdk-script] EventHub Triggered Function runs very long, sometimes shows as "Never Finished" (#872)
@andunnhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fandunn&data=02|01|andunn%40microsoft.com|0c2382ff079f42a2d4a908d4287701fb|72f988bf86f141af91ab2d7cd011db47|1|0|636177939039284163&sdata=fMT3CTc%2FUPayyR%2BF2WFdc9HspXZigzql1JolIlgrNb8%3D&reserved=0 Can you confirm that you're using WebJobs and not Azure Functions?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-webjobs-sdk-script%2Fissues%2F872%23issuecomment-268127354&data=02|01|andunn%40microsoft.com|0c2382ff079f42a2d4a908d4287701fb|72f988bf86f141af91ab2d7cd011db47|1|0|636177939039284163&sdata=gSPc03kEeXDccuOMJDZzIt0qfGhaQ8kmhjX%2BfwepUis%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOPezIC-huVXPc6bbtw-qv39Fl51S78Rks5rJy5mgaJpZM4Ko8hZ&data=02|01|andunn%40microsoft.com|0c2382ff079f42a2d4a908d4287701fb|72f988bf86f141af91ab2d7cd011db47|1|0|636177939039284163&sdata=uxyysJASAr7%2F%2FYw5mOJ7VicIQWnAOIV95VhyqTNdwX0%3D&reserved=0.

@andunn Do you mind recreating the issue here: https://github.com/Azure/azure-webjobs-sdk/issues? That way we can @ mention you and continue the conversation there.

Will do,
I’ll resurrect the code to see if the issue is still relevant first.
Happy Coding!
From: Donna Malayeri<EMAIL_ADDRESS>Sent: Tuesday, December 20, 2016 12:08
To: Azure/azure-webjobs-sdk-script<EMAIL_ADDRESS>Cc: Andrew Dunnells<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-webjobs-sdk-script] EventHub Triggered Function runs very long, sometimes shows as "Never Finished" (#872)
@andunnhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fandunn&data=02|01|andunn%40microsoft.com|f49637c8d9f6467badb908d42913d85f|72f988bf86f141af91ab2d7cd011db47|1|0|636178612583939184&sdata=hbxPSAkUrPMoX9TbTxVv4LUXkhPWKShCW3veki3Om2s%3D&reserved=0 Do you mind recreating the issue here: https://github.com/Azure/azure-webjobs-sdk/issueshttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-webjobs-sdk%2Fissues&data=02|01|andunn%40microsoft.com|f49637c8d9f6467badb908d42913d85f|72f988bf86f141af91ab2d7cd011db47|1|0|636178612583939184&sdata=xtx1bpXTn2N36Wz01jzHQHT%2BOb99dlCcdLP1s0ctRdY%3D&reserved=0? That way we can @ mention you and continue the conversation there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-webjobs-sdk-script%2Fissues%2F872%23issuecomment-268344927&data=02|01|andunn%40microsoft.com|f49637c8d9f6467badb908d42913d85f|72f988bf86f141af91ab2d7cd011db47|1|0|636178612583939184&sdata=B%2FKN89IUQj7ymWzFywvBngRCevlp4k7WQVgl%2BnG%2FJHs%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOPezFD75d2z3EM4G8fCsB7sTekKU4PUks5rKDWHgaJpZM4Ko8hZ&data=02|01|andunn%40microsoft.com|f49637c8d9f6467badb908d42913d85f|72f988bf86f141af91ab2d7cd011db47|1|0|636178612583949193&sdata=B%2B2fU2cRGhhA5zHj%2F6RnAqqB%2FTXTq9IcJGCGNK0mTuQ%3D&reserved=0.


Report unexpected exceptions

Add a config file setting that uploads unexpected exceptions to a paste service, and outputs the url in a predefined channel.

Yeah, I would have it for large errors only though. No need to do the processing on it for like 3-4 lines

"Large" errors? I feel like any unexpected exception would be worth investigating, since it's not getting caught by CommandFailed or other handlers.

Yeah, It should still go to the channel but what I meant was errors that are like 3-4 lines should just be sent as is, larger error messages should be uploaded to the paste service


Add new docker image for running kafka and zookeeper in single container

@ravisingal a high level comment: Did we check if there are any ways to run Kafka and ZK in the same JVM instead of two different processes in same Docker container? That would avoid some failure scenarios where one is running but not the other.

@ravisingal a high level comment: Did we check if there are any ways to run Kafka and ZK in the same JVM instead of two different processes in same Docker container? That would avoid some failure scenarios where one is running but not the other.

I've checked kafka quick start page (https://kafka.apache.org/quickstart). It also starts zookeeper as a separate process.


Linkify too aggressive

In my most recent PR #129 JupyterBook kept trying to render a file path as an html link. I think this is because we have linkify set up to automatically detect links, and it is too aggressive. I'm not sure if we want to disable linkify or perhaps find a way to tone it down.
https://github.com/ProjectPythia/pythia-foundations/blob/000661ff92f544e95efdaebe88c2c10b1f3c734b/_config.yml#L28

linkify-it does appear to include .py domains by default. Looks like myst/markdown-it-py don't provide for any config to linkify-it-py, so it doesn't look particularly straightforward/not-hacky for us to relax it ourselves?
I don't see us often typing "xxxx.py" without it being code, but also I don't think we're necessarily relying on linkify either. I guess we make sure people are using code thoroughly, and/or we disable linkify.
Tangentially related is that we can pretty easily add ignores to the linkcheck builder through sphinx options in _config.yml if we want, which I tried while checking this out and does work fine (doesn't address the root issue here though.)

BR, DC: we should not use linkify-it at all. Let's get rid of it. @kmpaul if you are ok with this can you make the change?

It might be nice to know what links fail without the checks failing.

It might be nice to know what links fail without the checks failing.

That would be nice! I do think that is separate from linkify and has more to do with link-checker though (perhaps a soft check with a warning instead of a failure).
Currently linkify automatically looks for links, which I'm not sure we need because all links should be intentionally by the author.
One work around we discussed in the IWG is that we should always style paths in codeblocks, so .py files aren't seen as potential links.
It seemed the consensus was to do both: have this consistent style - and remove any plugins that we don't find helpful.

+1 for removing plugins that we don't find helpful! Automation is great except when it gets in the way of doing what we want to do. I do not think we have identified any need for linkify.
@kmpaul I do think this is separate from checking links. linkify is turning things into link which were not meant to be links, and thus triggering false failures.

👍 Then I think we should remove linkify.


Channel close need reconnection

The channel may be closed due to some unexpected circumstances, and in this case, the handling of rebuilding the channel is also necessary.

Hi @icpd can you please share a little bit more of context on the issue you are facing?
As the library works right now, unless I've fucked up, is that if a channel is closed for some reason, then the consumer will get another channel from the connection. So I don't see the issue here. Can you please help me help you?

So basically, as you can see here: https://github.com/pmorelli92/bunnify/blob/main/bunnify/consumerLoop.go
The for loop of deliveries will exit when the channel is broken, then we check if the channel is closed, if not we close it not to leak it. Then the consumer.Consume will be called getting a new channel from the connection (unless the connection was closed voluntarily)

Sorry, I didn't look closely enough before. Indeed, the current handling method is fine.


Support prefer_literal option for intrinsic decorator

Fixes #6533
As titled, add support for prefer_literal option in intrinsic decorator.
PS: I am not so sure about the test I made, any suggestions would help.

class TestIntrinsicPreferLiteral(TestCase):
    def test_intrinsic(self):
        int64_100 = ir.Constant(ir.IntType(64), 100)
        int64_cafe = ir.Constant(ir.IntType(64), 0xcafe)

        def intrin(context, x):
            sig = signature(types.int64, x)

            if isinstance(x, types.IntegerLiteral):
                # With prefer_literal=False, this branch will not be reached.
                if x.literal_value == 1:
                    def codegen(context, builder, signature, args):
                        return int64_cafe
                    return sig, codegen
                else:
                    raise errors.TypingError('literal value')
            else:
                def codegen(context, builder, signature, args):
                    return builder.mul(args[0], int64_100)
                return sig, codegen

        prefer_lit =  intrinsic(prefer_literal=True)(intrin)
        non_lit = intrinsic(prefer_literal=False)(intrin)

        @njit
        def check_prefer_lit(x):
            return prefer_lit(1), prefer_lit(2), prefer_lit(x)

        a, b, c = check_prefer_lit(3)
        self.assertEqual(a, 0xcafe)
        self.assertEqual(b, 200)
        self.assertEqual(c, 300)

        @njit
        def check_non_lit(x):
            return non_lit(1), non_lit(2), non_lit(x)

        a, b, c = check_non_lit(3)
        self.assertEqual(a, 100)
        self.assertEqual(b, 200)
        self.assertEqual(c, 300)

(This is based on TestOverloadPreferLiteral testcase from test_extending.py)
I think this would more appropriate than the one I wrote previously.

class TestIntrinsicPreferLiteral(TestCase):
    def test_intrinsic(self):
        int64_100 = ir.Constant(ir.IntType(64), 100)
        int64_cafe = ir.Constant(ir.IntType(64), 0xcafe)

        def intrin(context, x):
            sig = signature(types.int64, x)

            if isinstance(x, types.IntegerLiteral):
                # With prefer_literal=False, this branch will not be reached.
                if x.literal_value == 1:
                    def codegen(context, builder, signature, args):
                        return int64_cafe
                    return sig, codegen
                else:
                    raise errors.TypingError('literal value')
            else:
                def codegen(context, builder, signature, args):
                    return builder.mul(args[0], int64_100)
                return sig, codegen

        prefer_lit =  intrinsic(prefer_literal=True)(intrin)
        non_lit = intrinsic(prefer_literal=False)(intrin)

        @njit
        def check_prefer_lit(x):
            return prefer_lit(1), prefer_lit(2), prefer_lit(x)

        a, b, c = check_prefer_lit(3)
        self.assertEqual(a, 0xcafe)
        self.assertEqual(b, 200)
        self.assertEqual(c, 300)

        @njit
        def check_non_lit(x):
            return non_lit(1), non_lit(2), non_lit(x)

        a, b, c = check_non_lit(3)
        self.assertEqual(a, 100)
        self.assertEqual(b, 200)
        self.assertEqual(c, 300)

(This is based on TestOverloadPreferLiteral testcase from test_extending.py)
I think this would more appropriate than the one I wrote previously.

Thanks for submitting this PR to the Numba issue tracker, it's been added to the queue for review. Please be advised however that we are currently in a PR freeze as we are preparing to ship a release candidate (RC). We will begin reviewing PRs again once the RC is out and we thank you in advance for your patience!
Also, having just skimmed the PR, perhaps try using context.resolve_value_type on a Numba type to get a LLVM type (and perhaps use numba.types.intp for your ints?). It also might be useful to call self.assert* from within the @overload to check that the types are literal and the amount of "trials" of resolving the overload is 1 in the case of prefer_literal=True.

Thanks for your suggestion @stuartarchibald ! Changing signature's return type to intp  seems to do the trick for 32 bit.

It also might be useful to call self.assert* from within the @overload to check that the types are literal and the amount of "trials" of resolving the overload is 1 in the case of prefer_literal=True

I am assuming you mean @intrinsic instead of @overload.
If I get you right, using self.assert* inside njit would result in compie error, wouldn't it?

We will begin reviewing PRs again once the RC is out and we thank you in advance for your patience!

No problem ! Take your time 👍


fiducials: 0.11.0-1 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository fiducials to 0.11.0-1:

upstream repository: https://github.com/UbiquityRobotics/fiducials
release repository: https://github.com/UbiquityRobotics-release/fiducials-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.8.0
previous version for package: 0.10.0-0

aruco_detect
* Compatibilty with Melodic

  remove opencv3 dependancy, opencv3 is no longer a package in melodic, so we use cv_bridge

  Fix build on OpenCV 3.2
* Import empy with full path to avoid pip conflict
* Use subprocess for cairosvg to avoid lack of python2 support
* call genMarker with correct arguments with no joblib
* Check ignoreID's in TF publishing
* Added topic that enables/disables Aruco detections
* Added rosdeps for cairo and joblib
* Add params for determining weighting of observations; prevent compiler warnings
* add cli argument for paper size to create markers
* Contributors: Jack Kilian, Jim Vaughan, Rohan Agrawal, Tim Übelhör, jack

fiducial_msgs

No changes

fiducial_slam
* Compatibilty with Melodic, remove opencv3 dependancy
  opencv3 is no longer a package in melodic, so we use cv_bridge
  to get the correct opencv transitively
* New fusion with planar-based error estimation
  * Estimate variance of fiducial obsevations based on how well they place the robot on the floor upright
  * use Kalman + David method of fusing estimates
  * split out Transform with Variance, do some cleanup
  * Simple unit tests for core fusion code
  * remove hokey outlier rejection
* Add support for overriding covariance diagonal
* Add fiducial transform array stamp (#156 <https://github.com/UbiquityRobotics/fiducials/issues/156>)
* Add publish_tf rosparam functionality and updated launch file accordingly (#154 <https://github.com/UbiquityRobotics/fiducials/issues/154>)
  * Added publish_tf rosparam to disable publishing pose tf
  * Change default publish_tf to True and fix 'if' formatting
* Contributors: Jack Kilian, Jim Vaughan, Rohan Agrawal

fiducials

No changes

Holding for sync


Navigation Controller won't come back up

I was wondering is there a way you can get the navigation controller to show again when segueing normally to another viewController because when I do so the navigation controller disappears.

was there ever a fix or way to do this?


Develop auto lemmatize function

I have been working with natural language processing and often needed to know which words were used in certain corpora. Many dictionaries are comprised of word stems, requiring the extraction of stems from sentences. For example, specific words can be key clues or carry important information, necessitating the extraction of sentences using these words, or processing sentence information with them.
In this context, I have developed a class called AutoLemmatizer in stem/wordnet.py that automatically performs tokenization and part-of-speech-based lemmatization, returning the lemmas of all words used in a sentence.

I also considered converting 'n't' to 'not,' but have not implemented it yet.

For big data, this process takes long time, and this task can be processed by multi-processing pipeline. if you interest, please let me know. I'm developing this.

I would be pleased if this contribution is considered useful and adopted.
>>> from nltk.stem import AutoLemmatizer
>>> auto_wnl = AutoLemmatizer()
>>> print(auto_wnl.auto_lemmatize('Proverbs are short sentences drawn from long experience.'))
['Proverbs', 'be', 'short', 'sentence', 'draw', 'from', 'long', 'experience', '.']
>>> print(auto_wnl.auto_lemmatize('proverbs are short sentences drawn from long experience.'))
['proverb', 'be', 'short', 'sentence', 'draw', 'from', 'long', 'experience', '.']

issue: #3256

@ekaf
Thank you for giving me the chance to contribute!
I have pushed the changes by incorporating your advice. Thank you again for advice.
As mentioned in the text, this task can take a long time to complete and can be processed using pipeline multiprocessing.
What do you think about applying multiprocessing or multithreading to the push?

Thanks again @Sion1225, your contributions are very welcome!
To accomodate ML, I guess that support for multiprocessing is actually needed, but it could eventually wait for a future PR.
What about implementing your function as a method of WordNetLemmatizer, instead of defining a new class? It could be advantageous to group all the methods that use morphy into one class, like in the following example. I moved the imports inside the function, and simplified the construction of the lemma_list:
    def lemmatize_text(self, text: str) -> List:
        """
        Tokenize input text, estimate the part-of-speech tag of each word,
        and return a list of lemmas.

        Returns each input word unchanged when it cannot be found in WordNet.

        :param text: The input text to lemmatize.
        :type text: str
        :return: A list with the estimated lemma of each `word` in the input text.

        >>> from nltk.stem import WordNetLemmatizer
        >>> wntl = WordNetLemmatizer().lemmatize_text
        >>> print(wntl('Proverbs are short sentences drawn from long experience.'))
        ['Proverbs', 'be', 'short', 'sentence', 'draw', 'from', 'long', 'experience', '.']
        >>> print(wntl('proverbs are short sentences drawn from long experience.'))
        ['proverb', 'be', 'short', 'sentence', 'draw', 'from', 'long', 'experience', '.']

        """
        from nltk.tag import pos_tag
        from nltk.tokenize import word_tokenize

        # Tokenize the sentence
        words = word_tokenize(text)

        # POS tagging
        pos_tags = pos_tag(words)

        # Lemmatize the words
        lemma_list = [
            self.lemmatize(pos_tag[0], self.pos_word_dict.get(pos_tag[1], "n"))
            for pos_tag in pos_tags
        ]

        return lemma_list

It is also possible to shorten the function even further. Some professors don't appreciate this concise style, but it tends to be popular in nltk. In my opinion, concise code is easier to understand, debug and maintain:
    def lemmatize_text(self, text: str) -> List:

        from nltk.tag import pos_tag
        from nltk.tokenize import word_tokenize

        return [
            # Lemmatize each POS-tagged word:
            self.lemmatize(word, self.pos_word_dict.get(tag, "n"))
            # Tokenize the input text and POS tag each word:
            for word, tag in pos_tag(word_tokenize(text))
        ]

I also contemplated whether implementing it as a method within WordNetLemmatizer or as an independent class would be easier to understand, so I welcome your opinion!

If your method does not use any information from wordnets, I believe it would be very missleading to implement it a method of the WordNetLemmatizer class. Unfortunately, the WordNetLemmatizer is not a subclass of a general Lemmatizer class, but making a Lemmatizer basic class would be great so WordNetLemmatizer and your class could be subclass of it. On the other hand, better find a more informative name instead of AutoLemmatizer.

@Sion1225, there is no reason to worry about your dictionary,
because it is equivalent to the following shorter helper function, which can be defined inside the lemmatize_text() function:
 def tag2pos(tag):
    return {"N": "n", "V": "v", "J": "a", "R": "r"}.get(tag[0], "n")

But there is a severe weakness in our approach, because it just returns the lemmas, while we also need the Pos if we want to use the output for something useful, like for ex. retrieving WordNet senses.
Hopefully, this could alleviate @arademaker's concerns above.
So I suggest we return (lemma, Pos) tuples, or evt. triples including the tag. Wouldn't that also be useful in your specific use case, where you wrote that you want to lookup the lemmas in a dictionary?
One use case I have in mind is lemmatizing the Brown Corpus, and comparing our output with the official Brown tags. Or what about the Gloss Corpus, @arademaker? I fear that our performance may not be very impressive, because of known issues with the lemmatize function, but it would be interesting to get some statistics.

@ekaf  That's a great method! It's something I hadn't thought of. It's definitely more concise. I'm learning a lot.
I think returning the current output (lemma reflecting POS) and POS together as a tuple is also a very good idea. I think it's more useful too. I'll incorporate it into the code.

@arademaker Our new function internally uses the lemmatize method of WordNetLemmatizer, so I think it would be okay to include it as a sub-method of WordNetLemmatizer. The current candidate name is TextLemmatizer, but if you have any better ideas, they are welcome!

I have started to measure the accuracy of this approach against the gold standard tags in the whole Brown Corpus, but I still need to tweak the tag2pos() function in order to accomodate some non-standard Brown Pos tags, like 'B' for the auxilary "be", and 'H' for "have". Nevertheless, it is already clear that we'll end up with more than 90% accuracy for  the Pos tagging.
So it is definitely worthwile to keep using the WordNetLemmatizer's lemmatize() function, even though I absolutely agree with @arademaker's objection against misusing the WordNet name.
@Sion1225's proposal could be extended to loop over any other lemmatize function, and there is also certainly room for developing new and better lemmatizers from scratch. But those goals are more ambitious, and better kept outside the scope of this PR.

@Sion1225, this PR should be converted to draft (using the link in the upper right corner), as long as it is "Still in progress", and not ready for merging.

I pushed code with changes. and moved examples from WordNetLemmatizer class to lemmatize because we have another method now.

@ekaf Yes, it definitely seems that the Universal Tagset is more useful. I would be happy if you could push the commit! Also, I forgot to change the output example in the method comments. If the output changes again, I think that part will need to be revised as well. Could you please take care of that too?

Ok, @Sion1225, I moved the universal_tag_to_wn_pos() out of the class, because it can also be useful elsewhere. Maybe it even should go into the corpora/readers/wordnet.py module instead?
Using Universal Tags, the lemmatize_text() function achieves 92.25% accuracy on the whole Brown Corpus, after discarding the sentences where the NLTK tokenization produces a different number of tokens than Brown's. These are not errors: the NLTK tokenizer tends to be more precise than Brown's, but those sentences are hard to compare when they don't align. This tokenization difference concerns 1.35% of the total number of tokens.

@ekaf I reviewed the code. Thank you for your modifications and push!
I don't have a deep understanding of the NLTK library's structure, but I think that as an independent function, it might fit better in a different module.
Is the tokenized Brown corpus provided by NLTK different from the result of tokenizing the Brown corpus using word_tokenize? It's an interesting analysis, and I'm curious about the remaining 7.75% difference.

Thank @ekaf for examples to understand. I realised this is because some words can have multiple parts of speech, such as both nouns and adjectives.
Regarding the usage example in the method's comments, I believe that it would be more intuitive to output the tuples directly rather than converting them to strings with the tuple2str function. This would make it easier to understand the function's behavior. What do you think?

Yes @Sion1225, the problem is that the most frequent words are often ambiguous. The wn._morphy() function returns a list with all matches, but among those,  lemmatize() picks the shortest word.
That's an arbitrary choice, which is not always the best. More linguistically ambitious applications consider all choices.
I don't mind if you prefer to present tuples in  the docstring, just feel free to make any adjustment you wish.

I modifed comments and I think now it's readied for pull request.
And, @ekaf I thought about universal_tag_to_wn_pos() it's suitable into wordnet.py imo.

Thanks @Sion1225! Yes, it seems ready now.


Why is datastore initialized differently than the other libraries?

See here for what I'm talking about. Seems that once gcloud is initialized, datastore cannot be initialized with different options. I think this is because we delegated it to the dataset object instead but that just is kinda messy. Would it be worth it to change this to be consistent with the other APIs?
Current way:
var gcloud = require('gcloud')(options);
var datastore = gcloud.datastore;
var dataset = datastore.dataset(newOptions);

Proposed way:
var gcloud = require('gcloud')(options);
var datastore = gcloud.datastore(newOptions);
var dataset = datastore.dataset();

ADDITIONAL CRAZY IDEA: What if we just eliminated the dataset altogether? Something like:
var gcloud = require('gcloud')(options);
var datastore = gcloud.datastore(newOptions);
var key = datastore.key('Company');
datastore.save(key, { data: { hello: 'world' } }, callback);

Anything preventing us from doing this?

I would really like to do the last one. @rakyll said to anticipate more things added to Datastore, which was why we couldn't do it.

I'm down for:
var gcloud = require('gcloud')(options);
var datastore = gcloud.datastore(newOptions);
var dataset = datastore.dataset('name of the dataset' /* optional */);
var key = datastore.key('Company');
datastore.save(key, { data: { hello: 'world' } }, callback);

The reason we can't kill data set is that we are in the process of adding the ability to have multiple datasets (we'll probably call them databases) per project, so datastore.save() is a misleading concept...
But I still think we need to allow you to configure stuff at the datastore level... ? right?

Oh, as far as I understand right now we have no concept of a name of a
dataset, right? In that case I like my first new proposal where it will
just take no options for the time being until this multi-dataset
functionality rolls around. This would be a breaking change.
On Tuesday, May 12, 2015, JJ Geewax<EMAIL_ADDRESS>wrote:

I'm down for:
var gcloud = require('gcloud')(options);var datastore = gcloud.datastore(newOptions);var dataset = datastore.dataset('name of the dataset' /* optional */);var key = datastore.key('Company');
datastore.save(key, { data: { hello: 'world' } }, callback);
The reason we can't kill data set is that we are in the process of adding
the ability to have multiple datasets (we'll probably call them databases)
per project, so datastore.save() is a misleading concept...
But I still think we need to allow you to configure stuff at the datastore
level... ? right?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-node/issues/593#issuecomment-101420687
.

datastore.dataset() should always still work, and default to the ID of the project. In the future, we need a way of passing in a name of a dataset (datastore.dataset('name')).
Don't think this should be a breaking change, right?

It would be breaking because gcloud.datastore will now be a function that
needs to be called with options. datastore.dataset will no longer accept
these options. And it doesn't make sense to have it in both places.
On Tuesday, May 12, 2015, JJ Geewax<EMAIL_ADDRESS>wrote:

datastore.dataset() should always still work, and default to the ID of
the project. In the future, we need a way of passing in a name of a dataset
(datastore.dataset('name')).
Don't think this should be a breaking change, right?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-node/issues/593#issuecomment-101423469
.

Gotcha - we couldn't make it work with and without the parens? (gcloud.datastore == gcloud.datastore()) ?

Maybe but if we're in the business of breaking things with the moving of
options from dataset to datastore then we should just break it. Also this
is much easier to do pre-1.0
On Tuesday, May 12, 2015, JJ Geewax<EMAIL_ADDRESS>wrote:

Gotcha - we couldn't make it work with and without the parens? (
gcloud.datastore == gcloud.datastore()) ?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-node/issues/593#issuecomment-101424894
.

Just want to make sure this is correct:
// uses dataset-id-1:
var gcloud = require('gcloud')({ projectId: 'dataset-id-1' });
gcloud.datastore.dataset();

// uses dataset-id-2:
var gcloud = require('gcloud')({ projectId: 'dataset-id-1' });
var datastore = gcloud.datastore({ projectId: 'dataset-id-2' });
datastore.dataset();

// uses dataset-id-3:
var gcloud = require('gcloud')({ projectId: 'dataset-id-1' });
var datastore = gcloud.datastore({ projectId: 'dataset-id-2' });
datastore.dataset('dataset-id-3'); // the projectId given above is useless.

@callmehiphop does this change how you would approach your PR (#845) at all?

I don't believe so, I think it would simplify the api object creation though.

This is going to be not-a-thing when datastore v1beta3 is released (see #897). The dataset concept is gone and there is only a "datastore" per project.


fix: cannot read property handlerType of undefined

fix: cannot read property handlerType of undefined
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[x] this video can play normaly

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #70 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #70   +/-   ##
=======================================
  Coverage   51.53%   51.53%
=======================================
  Files         123      123
  Lines        1721     1721
=======================================
  Hits          887      887
  Misses        834      834

Impacted Files
Coverage Δ

packages/griffith-mp4/src/mp4/mp4Box.js
88.88% <100%> (ø)
:arrow_up:

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 c8c91ff...cf431ff. Read the comment docs.


Sound clock for APU frame counter wrong bits selected

In the circuit diagram for timer module, the APU sound clock is derived from the DIV register bits 5 (4 MiHz clock) or 6 (8 MiHz clock), but these run at 256 Hz rather than the required 512 Hz. E.g. with an input clock of 4 MiHz (1 << 22) the 13th bit (DIV[5]) of the counter will divide this clock by 1 << (13 + 1), giving 1 << (22 - 14) = 1 << 8 = 256.
The correct bits for each speed mode should be DIV[4] (normal speed) and DIV[5] (double speed).
I think the error crept in because it's easy to miscount the frequency halving when 0-indexing.
https://gbdev.io/pandocs/Timer_Obscure_Behaviour.html#timer-global-circuit

Weird, I seem to remember DIV-APU events triggering at 256 Hz. Can we cite any source on that behaviour?

The section of the DIV-APU is correct AFAIK
https://gbdev.io/pandocs/Audio_details.html#div-apu

Since that section was written referencing DMG-CPU-Inside, I think it's more trustworthy. Alright then, thank you!

Can you assign this issue to me and explain on what needs to be done ? @maij

Hi! Thanks for participating ^^
There are two ways to go about this.

The simple way is to just edit the diagram so that the arrows coming from DIV's bits 5 and 6 come from bits 4 and 5 instead. (Preferably by moving everything they are connected to, as well.)
The more involved way would be to convert those diagrams into SVGs, and make the correction in the new diagram. It would take more time, but would improve the docs more!

Regardless of which option you prefer, I'll remain available if you need any help.

The more involved way would be to convert those diagrams into SVGs, and make the correction in the new diagram.

hello

No I'm not, feel free to go ahead.

Alright then, as per your request, I have assigned you to this issue!


HDDS-10628. Display if safemode exit was via force exit command.

What changes were proposed in this pull request?
Safe mode rules are not calculated/updated after a safemode force exit, If a user looks at the SCM web UI after a force exit , it would give an impression that SCM is in safemode by looking at the rules not being satisfied.
Displaying if safe mode was forcefully exited will be helpful in  this case.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10628
How was this patch tested?
Manually

cc @Tejaskriya

Thanks @sadanand48 for the patch, @ayushtkn, @myskov, @sumitagrawl, @Tejaskriya for the review.


Demo login not working

Unable to login to demo on heroku using the listed username and password.

what is username and pwd?

It should say in the banner:
username: demo
password: demo1234


v-chgri-commerce-ts-mig_2

Pull request guidance
Thank you for your contribution. The content team is monitoring the pull requests. Once your pull request is ready for review, please type "#sign-off" in a new comment or add a comment such as "PR is ready for review and merge."

This PR for Dynamics 365 Commerce troubleshooting article migration is a work in progress, please don't merge it until someone from Jo Sawyer's team signs off here. Thank you!

Hi @v-chgri, Thank you for your contribution. Once your pull request is ready for review, you can type "#sign-off" in a new comment or add a comment such as "PR is ready for review and merge." Then, we'll start to review the articles for you.

This PR for Dynamics 365 Commerce troubleshooting article migration is a work in progress, please don't merge it until someone from Jo Sawyer's team signs off here. Thank you!

Hi @v-chgri, thanks for this notice!

#sign-off
This PR is ready for review and merge. Article content is what is currently on the Commerce site. I've edited the toc.yml and welcome-commerce.yml files to reference the articles, but am unable to preview pages at this time. Please review the work done and let josaw and<EMAIL_ADDRESS>know of any issues/comments/concerns. Thanks!

#sign-off
This PR is ready for review and merge. Article content is what is currently on the Commerce site. I've edited the toc.yml and welcome-commerce.yml files to reference the articles, but am unable to preview pages at this time. (from Chris) Please review the work done and let josaw and<EMAIL_ADDRESS>know of any issues/comments/concerns. Thanks!

Thanks @v-chgri and @josaw1. We'll get started on this work and will let you know once this PR is ready for your final review.

Close this PR because the articles are reviewed and published via private repo.

https://github.com/MicrosoftDocs/SupportArticles-docs-pr/pull/4895


Why is the custom icon cropped to a circle?

This is the image I want to use as the icon. It has a transparent background but because it's all white Xcode shows it as having a grey background.

This is how it looks in the alert view:

As you can see, it is cropped. Only part of the "+" sign is shown. Can you make it not crop the image? There are a lot of empty space so just show the full image!

I have the same problem, anyone knows how to fix this issue?

i suggest you to comment this
"circleIconView?.layer.cornerRadius = circleIconView!.bounds.height / 2"

Which line is that? Can you tell me the line number?

Line number can differs, check this Function name below.
// showTitle(view, title, subTitle, duration, style)
open func showTitle(_ title: String, subTitle: String, duration: TimeInterval?, completeText: String?, style: SCLAlertViewStyle, colorStyle: UInt?=0x000000, colorTextButton: UInt?=0xFFFFFF, circleIconImage: UIImage? = nil, animationStyle: SCLAnimationStyle = .topToBottom) -> SCLAlertViewResponder  {
}

It was line 708 on mine as well 😄 . The solution worked. Thank you!

@Sweeper777 Hi , can you reopen the same? So that we can send fix.

https://github.com/vikmeup/SCLAlertView-Swift/pull/338


It seems since CUPS v2.3b3 the filters are no longer installed

Since CUPS v2.3b3 (it worked with v2.3b2) the filters are no longer installed.
At the end of building a RPM package I get

RPM build errors:
File not found: /home/abuild/rpmbuild/BUILDROOT/cups-2.3b3-227.1.i386/usr/lib/cups/filter/commandtops
File not found: /home/abuild/rpmbuild/BUILDROOT/cups-2.3b3-227.1.i386/usr/lib/cups/filter/gziptoany
File not found: /home/abuild/rpmbuild/BUILDROOT/cups-2.3b3-227.1.i386/usr/lib/cups/filter/pstops
File not found: /home/abuild/rpmbuild/BUILDROOT/cups-2.3b3-227.1.i386/usr/lib/cups/filter/rastertoepson
File not found: /home/abuild/rpmbuild/BUILDROOT/cups-2.3b3-227.1.i386/usr/lib/cups/filter/rastertohp
File not found: /home/abuild/rpmbuild/BUILDROOT/cups-2.3b3-227.1.i386/usr/lib/cups/filter/rastertolabel
File not found: /home/abuild/rpmbuild/BUILDROOT/cups-2.3b3-227.1.i386/usr/lib/cups/filter/rastertopwg

The filters are still compiled and linked,
e.g. for commandtops excerpts from my build log

Compiling commandtops.c...
cc  -Wall -Wno-format-y2k -Wunused -fPIC -g -fstack-protector -Wno-unused-result -Wsign-conversion -Wno-tautological-compare -Wno-format-truncation -D_GNU_SOURCE -I.. -D_CUPS_SOURCE -fomit-frame-pointer -fmessage-length=0 -grecord-gcc-switches -O2 -Wall -D_FORTIFY_SOURCE=2 -fstack-protector-strong -funwind-tables -fasynchronous-unwind-tables -fstack-clash-protection -fstack-protector -fPIE -fPIC -I/usr/include/libusb-1.0 -I/usr/include/dbus-1.0 -I/usr/lib/dbus-1.0/include -DDBUS_API_SUBJECT_TO_CHANGE -DDEBUG -D_REENTRANT -I/usr/include/p11-kit-1 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -D_THREAD_SAFE -D_REENTRANT   -c -o commandtops.o commandtops.c
...
Linking commandtops...
cc -L../cgi-bin -L../cups -L../filter -L../ppdc -L../scheduler -pie -Wl,-z,relro,-z,now -fPIE -pie -Wall -Wno-format-y2k -Wunused -fPIC -g -fstack-protector -Wno-unused-result -Wsign-conversion -Wno-tautological-compare -Wno-format-truncation -D_GNU_SOURCE -o commandtops commandtops.o -lcups -lgssapi_krb5 -lkrb5 -lk5crypto -lcom_err -lavahi-common -lavahi-client -lgnutls -lz -lpthread -lm -lcrypt   -lz

it seems the filters only get no longer installed.

FYI:
My current 2.3b3 build sources are here:
https://build.opensuse.org/package/show/home:jsmeix:branches:Printing/cups
and a current RPM build log is e.g. here
https://build.opensuse.org/package/live_build_log/home:jsmeix:branches:Printing/cups/openSUSE_Tumbleweed/x86_64

For me this is a fix:
https://build.opensuse.org/package/view_file/home:jsmeix:branches:Printing/cups/fix_filter_Makefile.patch?expand=1

Not sure why this didn't show up on my build farm, but will investigate and fix...

I submitted my home:jsmeix:branches:Printing/cups
to Printing/cups so that now the sources are in
https://build.opensuse.org/package/show/Printing/cups
any my fix is
https://build.opensuse.org/package/view_file/Printing/cups/fix_filter_Makefile.patch?expand=1

[master ab89234de] Fix installation of filters (Issue #5247)


error: use of moved value: request

Bug Report

Version
Tonic v2.0

Platform
Mac OS Catalin 10.15.4

Crates

Description
I have created the issue in stack overflow Click here for  Error description.  When I am creating the implementation of proto which is build using tonic build proto file when I use Request implementation in rust multiple time to get value from message field, which is created in proto the request.

Thanks for the response, it's working.


test utility emissions channel and update documentation

Follow steps from https://github.com/hyperledger-labs/blockchain-carbon-accounting/blob/main/utility-emissions-channel/README.md, get the utility emissions channel up and running.
Pls also suggest any improvements and updates to the documentation.

Can I take this up?

Yes please try it!  Let us know if there are any problems or if anything in
the documentation is not clear.
Si Chen
Open Source Strategies, Inc.
New Video: Creating Climate Solutions with Blockchain and Open Source
https://youtu.be/gtO3COq8crQ
On Wed, Jun 16, 2021 at 5:58 PM Chinmay Pandey @.***>
wrote:

Can I take this up?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hyperledger-labs/blockchain-carbon-accounting/issues/182#issuecomment-862832037,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANAS4JPOWT5F5VNHKHXM23TTFCCNANCNFSM46V6VGXQ
.

Hi @sichen1234 ,
I believe this can be closed. The pull request to resolve this has already been merged. Thanks :)

This is an "evergreen" issue for everybody to try out when they join the project, so please leave it open.

Hi everyone I want to work here can anyone please help me. @sichen1234

Sure.  What do you need help with?
Si Chen
Open Source Strategies, Inc.
New Video: Creating Climate Solutions with Blockchain and Open Source
https://youtu.be/gtO3COq8crQ
On Mon, Jul 19, 2021 at 12:46 AM Amisha Kumari @.***>
wrote:

Hi everyone I want to work here can anyone please help me. @sichen1234
https://github.com/sichen1234
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hyperledger-labs/blockchain-carbon-accounting/issues/182#issuecomment-882322650,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANAS4L7ZIHKRBEC74NXUQTTYPJ63ANCNFSM46V6VGXQ
.

Hello @sichen1234 I am interested to contribute for this Project.
I go through the Readme.md you mentioned above and start doing setup. At the point
I got one issue.
When I run the command sh start.sh to build and start the npm server. I got "tsc command is not found ". So I observerd tsc is typescript command used for the build. Then i install it manually by "npm i --save-dev typescript" and everythings works fine then.
So I think small update is required in package.json. We need to add "typescript": "^4.3.5" in  under devDependencies So that we didn't need to install it manually. Which I have done.
Can I submit the patch for it if it looks relevant to You??
Thank You :-)

Hello @Bhaarat-khatri , you are right about adding typescript as devDependencies into package.json. But there is no need for you to create a PR for that , since this will be fixed in #261 PR.

Oh that's cool @Zzocker :-). Thanks for let me know. I will try to work some other issue.