Installation fails for Python 3.9, 3.10, 3.11

Package fails to install under python 3.9, python 3.11, only seems to work under 3.6 3.7. Is this really necessary?

thank you for catching. could you provide the full error log?
fyi fracdiff is tested for 3.7.15, 3.8.15, 3.9.16.
https://github.com/fracdiff/fracdiff/actions

Same issue here, using Python 3.10.9 ~ Full install error log is below, it's quite long.
Any chance support could be added for other python versions?
fracdiff.log

thank you both. trying to resolve dependencies, where finding that statsmodel is the tricky part of it. appreciate pr

https://github.com/statsmodels/statsmodels/issues/8543

could you try fracdiff 0.9.1 ? @igorrivin @jpbreuer

Here's a log, looks like it was a successful install! Many thanks!
$ pip install git+https://github.com/fracdiff/fracdiff.git Collecting git+https://github.com/fracdiff/fracdiff.git   Cloning https://github.com/fracdiff/fracdiff.git to /private/var/folders/rh/xqqws2xs567cvlqrgzhr2kqw0000gn/T/pip-req-build-fr5as267   Running command git clone --filter=blob:none --quiet https://github.com/fracdiff/fracdiff.git /private/var/folders/rh/xqqws2xs567cvlqrgzhr2kqw0000gn/T/pip-req-build-fr5as267   Resolved https://github.com/fracdiff/fracdiff.git to commit 93ce110a4b8e7022d3507b83eb64bef4fa685179   Installing build dependencies ... done   Getting requirements to build wheel ... done   Preparing metadata (pyproject.toml) ... done Collecting statsmodels@ git+https://github.com/statsmodels/statsmodels.git   Cloning https://github.com/statsmodels/statsmodels.git to /private/var/folders/rh/xqqws2xs567cvlqrgzhr2kqw0000gn/T/pip-install-12pup802/statsmodels_3ee6d9630e2d40189068f04e6f1bc125   Running command git clone --filter=blob:none --quiet https://github.com/statsmodels/statsmodels.git /private/var/folders/rh/xqqws2xs567cvlqrgzhr2kqw0000gn/T/pip-install-12pup802/statsmodels_3ee6d9630e2d40189068f04e6f1bc125   Resolved https://github.com/statsmodels/statsmodels.git to commit 6d5f7461e7e25456fd38f032d3028945f5a54a82   Installing build dependencies ... done   Getting requirements to build wheel ... done   Installing backend dependencies ... done   Preparing metadata (pyproject.toml) ... done Requirement already satisfied: numpy<2.0.0,>=1.24.1 in /opt/homebrew/lib/python3.10/site-packages (from fracdiff==0.9.1) (1.24.1) Collecting scipy<2.0.0,>=1.10.0   Using cached scipy-1.10.0-cp310-cp310-macosx_12_0_arm64.whl (28.8 MB) Requirement already satisfied: packaging>=21.3 in /opt/homebrew/lib/python3.10/site-packages (from statsmodels@ git+https://github.com/statsmodels/statsmodels.git->fracdiff==0.9.1) (22.0) Requirement already satisfied: pandas>=1.0 in /opt/homebrew/lib/python3.10/site-packages (from statsmodels@ git+https://github.com/statsmodels/statsmodels.git->fracdiff==0.9.1) (1.5.2) Requirement already satisfied: patsy>=0.5.2 in /opt/homebrew/lib/python3.10/site-packages (from statsmodels@ git+https://github.com/statsmodels/statsmodels.git->fracdiff==0.9.1) (0.5.3) Requirement already satisfied: python-dateutil>=2.8.1 in /opt/homebrew/lib/python3.10/site-packages (from pandas>=1.0->statsmodels@ git+https://github.com/statsmodels/statsmodels.git->fracdiff==0.9.1) (2.8.2) Requirement already satisfied: pytz>=2020.1 in /opt/homebrew/lib/python3.10/site-packages (from pandas>=1.0->statsmodels@ git+https://github.com/statsmodels/statsmodels.git->fracdiff==0.9.1) (2022.7) Requirement already satisfied: six in /opt/homebrew/lib/python3.10/site-packages (from patsy>=0.5.2->statsmodels@ git+https://github.com/statsmodels/statsmodels.git->fracdiff==0.9.1) (1.16.0) Building wheels for collected packages: fracdiff, statsmodels   Building wheel for fracdiff (pyproject.toml) ... done   Created wheel for fracdiff: filename=fracdiff-0.9.1-py3-none-any.whl size=13724 sha256=74de61fc2138dc01dc79474cdd6db93a986f633d798ae0221a34b79ac32b81f2   Stored in directory: /private/var/folders/rh/xqqws2xs567cvlqrgzhr2kqw0000gn/T/pip-ephem-wheel-cache-e25jnguu/wheels/2b/c3/07/db8b0486e5ab2ca40a18921d9d87efe4135b9fa641397c6961   Building wheel for statsmodels (pyproject.toml) ... done   Created wheel for statsmodels: filename=statsmodels-0.14.0.dev719+g6d5f7461e-cp310-cp310-macosx_13_0_arm64.whl size=9410783 sha256=9ddd251b21162ce7f9a2196ec8a3a3c1d002751aac679cbb1213c5b237164728   Stored in directory: /private/var/folders/rh/xqqws2xs567cvlqrgzhr2kqw0000gn/T/pip-ephem-wheel-cache-e25jnguu/wheels/63/99/aa/bb07110a6e10e2d86fabb8c067f39ed97d662d486bb01ca22b Successfully built fracdiff statsmodels Installing collected packages: scipy, statsmodels, fracdiff   Attempting uninstall: scipy     Found existing installation: scipy 1.9.3     Uninstalling scipy-1.9.3:       Successfully uninstalled scipy-1.9.3   Attempting uninstall: statsmodels     Found existing installation: statsmodels 0.13.5     Uninstalling statsmodels-0.13.5:       Successfully uninstalled statsmodels-0.13.5 Successfully installed fracdiff-0.9.1 scipy-1.10.0 statsmodels-0.14.0.dev719+g6d5f7461e

Tentative solution:
pip install<EMAIL_ADDRESS>

The tentative solution worked for me up until now. I am using Google Colab and get the following error:
`---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xf . Check the section C-API incompatibility at the Troubleshooting ImportError section at https://numpy.org/devdocs/user/troubleshooting-importerror.html#c-api-incompatibility for indications on how to solve this problem .
RuntimeError                              Traceback (most recent call last)
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xf . Check the section C-API incompatibility at the Troubleshooting ImportError section at https://numpy.org/devdocs/user/troubleshooting-importerror.html#c-api-incompatibility for indications on how to solve this problem .
RuntimeError                              Traceback (most recent call last)
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xf . Check the section C-API incompatibility at the Troubleshooting ImportError section at https://numpy.org/devdocs/user/troubleshooting-importerror.html#c-api-incompatibility for indications on how to solve this problem .
RuntimeError                              Traceback (most recent call last)
RuntimeError: module compiled against API version 0x10 but this version of numpy is 0xf . Check the section C-API incompatibility at the Troubleshooting ImportError section at https://numpy.org/devdocs/user/troubleshooting-importerror.html#c-api-incompatibility for indications on how to solve this problem .
ImportError                               Traceback (most recent call last)
 in <cell line: 11>()
9 from tqdm import tqdm
10
---> 11 from fracdiff import fdiff
12 from scipy.stats import skew, kurtosis
13 from math import sqrt
10 frames
/usr/local/lib/python3.10/dist-packages/numpy/testing/_private/utils.py in
21 from numpy.core import(
22      intp, float32, empty, arange, array_repr, ndarray, isnat, array)
---> 23 import numpy.linalg.lapack_lite
24
25 from io import StringIO
ImportError: numpy.core.multiarray failed to import

NOTE: If your import is failing due to a missing package, you can
manually install dependencies using either !pip or !apt.
To view examples of installing some common dependencies, click the
"Open Examples" button below.
---------------------------------------------------------------------------`

Hi @L-Schiffer , are you using the tentatively fixed version in there? the following line in jupyter should install it in it:
!pip install<EMAIL_ADDRESS>


TNG-14: add attribute subpage for enum

The age work should go into a separate PR. This PR shouldn't just keep growing while in review.

@tjennison-google  Hi Tim. I clean out the age stuffs and pr separately only for the attribute page now.


Broken markdown highlighting after 1.92.0 upgrade

Does this issue occur when all extensions are disabled?: I'm reporting a conflict with an extension, so I left that one enabled.

VS Code Version: 1.92.0
OS Version: macOS Sonoma 14.4.1

After upgrading to 1.92.0 syntax highlighting for qmd files (Quarto extension) started showing the same issues that were described some days ago wrt the builtin markdown support. That issue was marked as duplicate (it's not clear to me that it's a duplicate) and then closed by a bot as not planned. I don't know if this is a regression in VSCode or an issue that should be addressed by the extension, but it's clearly something going on that's related to a change in latest release.

https://github.com/quarto-dev/quarto/issues/501
https://github.com/microsoft/vscode/issues/220026

Please Marking as caused by extension. Sounds like they may need to update their grammar based on changes to our markdown grammar
They can follow up here if there really a problem on the VS Code side

Ok @mjbvz thanks for the clarification.


Memory usage

In qdrant, how to find out which collection uses more memory, and how to improve this

I think you should put the issue qdrant repo instead of StarRocks ?


Time-Series Widget does not work if dataset doesn't have geometry

Hi @alonsogarciapablo  I see you were working on this - maybe related?
cc @zingbot @saleiva
Context
Not sure if this is a bug or something we're going to add in the future - what if a client wants to use widgets to filter non-geospatial data? If we want to support that, right now the Time Series widget isn't loading for datasets with null the_geom.
For example, what if we have a dataset without geometry and we want to use Formula widget to calculate the sum of a numeric column, but only for values filtered by Time-Series widget?
Process

Import a dataset that doesn't contain geometry & create a map from it in Builder.
From main map pane click Widgets tab, then Add
Choose Time-series & a select a date-type column. Click Continue. Leave the default widget settings.

https://team.carto.com/u/stephaniemongon/builder/3a19e660-6a27-11e6-aec1-0ee66e2c9693/embed
time_series_null_geom map (on 2016-08-24 at 19.25.30).zip
Expected result
Filterable time-series widget will appear at bottom of the map.
Current result

error: 'the tiler does not support non-torque layers just yet…'
also 400/Bad Request error:

We will not support that. We shouldn't let add widgets if there is no geom on the data, as nothing will work.


Feat/notebooks pipelines

Allow user to create a pipeline from a notebook.
Changes
Please list / describe the changes in the codebase for the reviewer(s).

Change 1
Change 2

How/what to test
A few words about what needs to be tested, along with specific testing instructions if needed.
Screenshots / screencast
If relevant, please add some screenshots or a short video.

Yes, it’s better

On 24 Apr 2024, at 11:55, Cheikh Gueye Wane @.***> wrote:
@cheikhgwane commented on this pull request.
In hexa/pipelines/schema/mutations.py https://github.com/BLSQ/openhexa-app/pull/688#discussion_r1577620464:

   )

   notebook = input.get("notebook")

   if notebook:

       get_bucket_object(workspace.bucket_name, notebook)

       pipeline = create_pipeline_object(

           code=input["code"],

           name=input.get("name"),

           workspace=workspace,

           type=PipelineType.NOTEBOOK,

           notebook=notebook,

       )

       # automatically create a new version

       pipeline.upload_new_version(

           user=request.user,

           name=input.get("name"),

           zipfile=base64.b64decode("".encode("ascii")),

The zipFile field was required in the PipelineVersion model, but yeah it's better to modify the binaryField and allow it to be null. WDYT?
—
Reply to this email directly, view it on GitHub https://github.com/BLSQ/openhexa-app/pull/688#discussion_r1577620464, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMIO7JKBX4GWNN5U576VK3Y656PTAVCNFSM6AAAAABGOZUM3OVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMJZGQZDMNBVGA.
You are receiving this because your review was requested.

Better to keep it

On 25 Apr 2024, at 15:24, Cheikh Gueye Wane @.***> wrote:
@cheikhgwane commented on this pull request.
In hexa/pipelines/management/commands/pipelines_runner.py https://github.com/BLSQ/openhexa-app/pull/688#discussion_r1579465611:

@@ -236,7 +244,10 @@ def run_pipeline_kube(run: PipelineRun, image: str, env_vars: dict):
def run_pipeline_docker(run: PipelineRun, image: str, env_vars: dict):
from subprocess import PIPE, STDOUT, Popen

docker_cmd = f'docker run --privileged -e HEXA_ENVIRONMENT=CLOUD_PIPELINE -e HEXA_RUN_ID={env_vars["HEXA_RUN_ID"]} -e HEXA_SERVER_URL={env_vars["HEXA_SERVER_URL"]} -e HEXA_TOKEN={env_vars["HEXA_TOKEN"]} -e HEXA_WORKSPACE={env_vars["HEXA_WORKSPACE"]} --network openhexa --platform linux/amd64 --rm {image} pipeline cloudrun'

env_vars.pop("HEXA_PIPELINE_NAME", "")
I've seen that we really don't use it for the run_docker, but I can keep it (for future usage ?)

—
Reply to this email directly, view it on GitHub https://github.com/BLSQ/openhexa-app/pull/688#discussion_r1579465611, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMIO7PBAMUOQ3Y2ZFRI23DY7D72TAVCNFSM6AAAAABGOZUM3OVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMRSGQ4DONZVGI.
You are receiving this because your review was requested.

I think we are going to remove the code that allows the user to search for objects since it would required us to also implement this feature in s3.py. It's slow and might raise our costs.


Updates for Doom map

Changelog:

Brushwork improvements
Hidden surfaces now use caulk shaders
"no-struc-brushes" changed to detail-brushes
Added lava texture & shader (lava texture source: https://opengameart.org/content/yet-another-glowing-coal-pattern)
Added new textures

Merged!
Thx for the contribution, will update the TITLEPIC for add your name


Bug: stacking bug with grammerly playground demo

Lexical version: latest
Steps To Reproduce

Load the official playground from the website

Link to code example -> https://playground.lexical.dev/
The current behaviour
Lexical icons sit on top of Grammarly

The expected behaviour
Not to sit above or let users easily click the UI

Sounds like a grammarly bug? I guess we could have the action buttons hidden and they get shown on pressing some button

I can’t see Grammarly adjusting UI just for this product (although I could be wrong).
I wonder if there is some css positional tweaks which could be done if the UI.
Will take a look in the next few days.

Yea seems like a simple CSS fix - PR welcome. I wonder if Grammarly has configuration options to determine where that icon is positioned.

I looked more at this and I think those buttons used to be outside the editor element. Not sure when that changed.

Validated that this now works.


feat: trpc v11

#24
This at least demonstrates an example of some of the changes - they don't seem to be massive. With v11 not being stable yet however, it may not be worth merging this PR.
Doesn't seem to fix the examples, but I've not looked in to that yet, was just a pure port from the changelog.
. test:types:examples$ tsc -p examples --noEmit
│ examples/01_typescript/src/app.tsx(31,54): error TS7006: Parameter 'get' implicitly has an 'any' type.
│ examples/02_authtoken/src/app.tsx(37,4): error TS7006: Parameter 'get' implicitly has an 'any' type.
│ examples/02_authtoken/src/app.tsx(44,18): error TS7006: Parameter 'item' implicitly has an 'any' type.
└─ Failed in 863ms at /Users/lukeramsden/workspace/jotai-trpc
. test:spec$ vitest run
└─ Running...
 ELIFECYCLE  Command failed with exit code 2.
 ELIFECYCLE  Test failed. See above for more details.

I think they maintainers of trpc DO consider v11 to be stable see the comment in the beta release.
If this PR is ready to merge maybe it is worth releasing it under a beta flag also?
Thoughts @lukeramsden  @dai-shi

When it's ready I can release it. (Also, I'm looking for a maintainer.)


net/http: Expect: 100-continue changes behaviour if server closes request body

What version of Go are you using (go version)?

$ go version
go version devel go1.21-af8f94e3c5 Tue Jul 11 21:30:51 2023 +0000 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
N/A
What did you do?
https://go.dev/play/p/wwvMPANaBh4
What did you expect to see?
The same result regardless of whether the server closes the request body or not.
What did you see instead?
When the server closes the request body, the bytes of the request body are read inappropriately, probably because the server has sent a 100 Continue response.
Although http.Handler implementations should not close the body, it's not uncommon that to happen, for example here.
Replacing the Request.Body field (for example to replace it with a limited-size reader) has the same effect as closing it in this respect.

After a bit more investigation, it seems that the following is happening:

req.Body.Close ends up invoking http.body.Close via http.expectContinueReader.Close
http.body.Close hits the doEarlyClose branch.
this tries to read some of the request body, which will wait indefinitely because the client is waiting for 100 Continue before sending the request body
the client times out after 1s (http.Transport.ExpectContinueTimeout) and transfers the request body to the server.
the server reads the request body and sends the 200 response
the client sees the 200 response

The fact that the server is waiting for the request body when the client has explicitly said that it won't be sending it seems like it's probably a bug.

@bradfitz

per https://dev.golang.org/owners, we should probably ping @neild :)

One other thing to note: this behaviour does not occur when the body has an explicit content length that's more than 256KiB. In that case, the code gives up without trying to read the body which causes the client to see the response immediately: https://go.dev/play/p/ajHWF161sdD


docs: check updates for the Helm reference

Cilium's documentation has a reference for the Helm values supported in its Charts. The reference is auto-generated, and is supposed to be updated each time the Charts are modified. To help keep the reference up-to-date, Cilium's CI should warn when developers forgot to regenerate and commit the document.
Because the CI reported missing updates in the past, we thought that this was covered. But it turns out that the CI would only complain when it fails to update the Helm reference - typically when some words need to be added to the spelling list. If the update goes fine, there is no check in place to validate that the regenerated file is identical to the one currently in the repository. This has led to multiple PRs missing the update for the Helm reference in the past.
Address the issue by adding a check-helmvalues.sh script to validate that the current file is identical to the version in Git's HEAD. Run this script from the Makefile, as part of the check target.
We also create a update-helm-values target, which looks cleaner to add as a prerequisite for check instead of passing the name of a .rst file. We also introduce a FORCE phony target to explicitly mark that we want the file regenerated each time.

Travis error is related, it happens while we try to regenerate the Helm reference after building Cilium, as part of the postcheck target. Prior to this PR, we wouldn't update the Helm reference and check for differences. Now we do, and something goes wrong.
My understanding from the logs and error message is that the helm-docs Go binary that we try to launch from the Docker image doesn't work... on the arm64 architecture where the build is failing.
I'll update tomorrow, and probably disable the check if on arm64. I don't suppose it is worth building a second helm-docs image at the moment?

Travis build for arm64 went find and it has the message about the skipped action :+1:


Added 105 mm form length and made it default

Default form length was 210 mm (= A4 paper width). Adding a default of half that length (= A6 paper width) saved a bunch of paper for me.

Yeh, thank you!
I'm going to redesign the whole file, to make it 'right way' - i.e. via .drv compiled into .ppd with ppdc

Good idea! ppdc is deprecated, though

The xml file format of foomatic-db seems to be the modern way to do it.


Decrease Disk space usage of the SDK

Is your feature request related to a problem? Please describe.
I synced the Darksidewalletd seed phrase to chain tip (around a million blocks).
the cache db size is quite significant and could be discarded once the wallet has synced

The data db has 120 transactions and 1M blocks
Describe the solution you'd like
I'd like the SDK to perfore cleanups on the DB files regularly and also be vigilant of used spaced and minimizing it when possible.
Alternatives you've considered
DB Vacuum does not reduce the size of the DataDb at all.

fixed by fs_block_db implementation


Webpack plugin: support absolute entry paths

Adds support for Webpack config files that specify entries with an absolute path.

Thanks @rmunch!

:rocket: This pull request is included in v5.17.0-canary.0. See Release 5.17.0-canary.0 for release notes.
Using Knip in a commercial project? Please consider sponsoring me.


Create a handbook entry for requests for Miro and other paid services.

Right now there is no information on what to do if someone wants to subscribe to Miro or other paid services we use that are not provisioned for everyone.

The Miro entry has been added ... but regarding future requests, should this entry be at the top of the tech stack listing or on the Engineering handbook page?  For now I chose the latter... https://handbook.meltano.com/engineering/#adding-new-tools


Proper fix for catastrophic backtracking in function definition.

The curent fix for #287 is not a perfect fix. As https://github.com/jeff-hykin/cpp-textmate-grammar/issues/287#issuecomment-509021142 shows the issue can still appear. Additionally, this fix broke highlighting of function definitions when preceded by extern "C" (#319) or a macro.
This issue is to track any progress on preventing the function definition pattern from catostophically backtracking.

A less contrived example of how the fix is broken is when using fmtlib.
One might conceivably have the following snippet.
fmt::print("****** Warning {} Warning ******", msg);

Can we boil the example down and reproduce the catastrophic backtracking inside of a regex tool website? or do you think this is due to oniguruma?

From what I remember, the issue should be reproducible on regex testing sites.

Finally fixed this for good. It was the std_space pattern, making it greedy, reorganizing it a bit, and removing \b in favor of lookaheads/behinds overhauled performance


Fixed some unhandled exceptions

When trying to open a folder/run a file from a folder that doesn't exist
When trying to read songpack version from a nonexistent file

please use file.exists instead of directory.exists for files. also, OpenFolder(somefile.exe) is bad style...

and please add ex.Message to the error handling messagebox output.

Pinging this to make sure it's not forgotten.


Add border-width: 1px for horizontal rules

Fixes #979

Thanks @AlexVipond!

Was the intention to make
s 1px in height? Because a 1px border on all sides makes it 2px tall.

Should probably be border-bottom-width: 1px;.

Yep agree, will fix this for the next patch release. Sorry that it'll be another breaking-ish patch 😩

Was the intention to make
s 1px in height? Because a 1px border on all sides makes it 2px tall.

My intention was just to find something that looked like a good default—I was designing in the browser and not giving much thought to the actual pixel height of the element. Although if you put me on the spot, I probably would've have guessed they were 1px tall 😅
Personally I don't have a strong preference between a 1px or 2px tall default border. Maybe it's time for another @adamwathan twitter poll?


move firmware.bin out of the build/ dir

Right now, build/ contains all the build results. Makes a lot of sense, and convenient because "make clean" now simply deletes that whole directory.
But "firmware.bin" is normally kept around, so it gets checked into Github. With "make clean" you now end up with a state which is not 100% compatible with a "git commit".

Closing this, since development is moving away from C/C++ to Forth.


Merging Dev into Master

Codecov Report

Merging #10 into master will decrease coverage by 5.71%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master     #10     +/-   ##
========================================
- Coverage   10.75%   5.04%   -5.7%
========================================
  Files           4       1      -3
  Lines         512     377    -135
========================================
- Hits           55      19     -36
+ Misses        457     358     -99

Impacted Files
Coverage Δ

Travis/travis_uptimerobot_monitor_utility.sh
5.04% <100%> (-5.98%)
:arrow_down:

Travis/reset_expect_long.exp

Travis/reset_expect_short.exp

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 e5f82eb...1ac75a5. Read the comment docs.


Trying to use postman to make simple call failing https://api.figma.com/v1/me

When I try to make a figma call for the
get https://api.figma.com/v1/me

as described in the following documentation  I get a 400 bad request.
I am aware that I need a header item for the X-Figma-Token object.  I am using Postman to make this call, and it fails every time.
For reference I created my access token and the call does work successfully when I make a call using a curl inside studio code. Note this code is made from postman.  so I am pretty sure the actual curl command works.
curl --location 'https://api.figma.com/v1/me' --header 'X-Figma-Token: <myAccessTokenHere>'

Anybody been able to get this api to work when making the following call https://api.figma.com/v1/me from postman?

CC: @coreyward for maybe some help here.


object-conversion before running the blueprint definitions

We are using the Decorator-Pattern to extract formatting functionality from the models. It should somehow be possible to run the blueprints against a different object than the ones returned from the database:
XapianDb::DocumentBlueprint.setup :Person do |b|
  b.conversion { |person| PersonDecorator.new(person) }
end

the snippet above would be sufficient for our needs but probably it would be better if there was a general way to index fields, which aren't directly defined on the model-class itself.

Here are some suggestions:
a) use block attributes and implement your logic there
b) decorate your objects before you save them (might violate your layering architecture)
c) create a blueprint for your decorator instead of your model and (re)index your decorator whenever needed like so: XapianDb.reindex(decorator)

@gernotkogler b.) and c.) are clearly the wrong path for my situation. This would make things worse than putting all the formatting logic directly onto the model. I guess a) works but when there are a lot of methods used from the decorators object-creating explodes...

I'm closing this now since it does not seem to be a problem anymore ;-)


Handle Missing Remotes (Git)

Description
If local repository does not contain any remote, git history tab gives error. So, this PR fix this problem.

Related Issues

N/A

Checklist

[x] I read and understood the contributing guide as well as the code of conduct
[x] The issues this PR addresses are related to each other
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] My changes are all related to the related issue above
[x] I documented my code

Screenshots

@allcontributors add @ibrahimcetin for bug


Broken link

Near the bottom of the page this link rewards the supplicant with a 404:
Integrate components into Razor Pages and MVC apps

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 13b7135b-b7a0-2fba-b846-530480aeb415
Version Independent ID: 69e5b8fb-04e0-817e-434f-72cf6ba12923
Content: Migrate from ASP.NET Core 3.0 to 3.1
Content Source: aspnetcore/migration/30-to-31.md
Product: aspnet-core
Technology: aspnetcore-migration
GitHub Login: @scottaddie
Microsoft Alias: scaddie

Thanks @kenleese ... The topic moved and this one was missed on the find-and-replace of links because these cross-links aren't normal XREF links ... they're relative links to the markdown files. I'll patch it up now.


Fix issue #43 - The basic demo doesn't work.

Remove extra semi-column at the start of mappings in basic demo.

Thanks!

Sorry I made a mistake in referencing the wrong issue.

Yeah I noticed that, don't worry about it :)


Add stop-the-world warning to this week's release notes.

The change to MaxOffset requires a stop-the-world upgrade.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e4e0c749697e3d52523ccadead5b94072c5410e1/

@swatikumar16, @jess-edwards, never mind. I just updated the post myself.


Client validation

Added validation of resource names to generated surface for java
Renamed method_base_name to entity_name
Updated generated javadocs with parameter rename

PTAL

LGTM


Issue running on Docker

Issue
Running video.Read() on Docker seems to run into the issue where the following loop ends after the first iteration:
for frameCount := 0; video.Read(); frameCount++  {

  // ... do some logic on frames

}

Not sure where I am going wrong, running the binary on macOS seems to give me no issues, but running on both alpine:latest and ubuntu:latest gives me this issue.
Things I tried

Using different base images
Copying video file directly into the container

What my docker image looks like
# Use the official Golang image as the base image
FROM golang:1.18 AS builder

# Set the working directory inside the container
WORKDIR /app

# Copy the Go module files
COPY go.mod go.sum ./

# Download and cache the Go module dependencies
RUN go mod download

# Copy the source code into the container
COPY . .

# Build the Go application
RUN CGO_ENABLED=0 GOOS=linux go build -a -installsuffix cgo -o vidstreamsplit .

# Use a minimal Alpine image as the base image for the final image
FROM ubuntu:latest

# Install FFmpeg and AWS CLI
RUN apt update && apt install -y ffmpeg

# Set the working directory inside the container
WORKDIR /app

# Copy the built binary from the previous stage
COPY --from=builder /app/vidstreamsplit .

# Make the binary executable
RUN chmod +x vidstreamsplit

# Copy the scripts directory into the container
COPY scripts ./scripts

# Make all the scripts in the scripts package executable
RUN chmod +x scripts/*.sh

Please let me know if you've run into this issue before. Happy to take a look and try to submit a change if you could point me in the write direction.

If the only issue is that the loop ends after the first iteration, then there most likely is an error in the init function, and the error is probably coming from the exec.Command initialization. My guess is that ffmpeg can't open the file even though you've downloaded it (might be an issue with docker) which would result in the behavior you've described. I'd recommend debugging and checking what the return value of init() is inside the Read() function. If the error is at all related to the ffmpeg process then you'll know that it can't open the video file you've given it.


Panels

... sigh

What I was really after was a titlebar aligned to the top of the callout, which is now in 0.2.0.


rc.sshd DSA host key

rc.sshd does no longer generate a dsa host key, but OpenSSH still defaults to 4 different HostKeys when not specified otherwise. rc.sshd does not specify HostKeys, hence sshd still looks for ssh_host_dsa_key.
To reproduce:
Factory reset (which deletes an existing ssh_host_dsa_key)
sshd -T | grep hostkey
=> 4 default host keys are listed and an error message is displayed, that ssh_host_dsa_key cannot be loaded.
Either put back generation of ssh_host_dsa_key or explicitely set the 3 supported HostKeys when generating sshd_config in rc.sshd.

@kevemueller Thanks, your right, we missed a spot there. I will include the supported hostkeys in the config generation.

Hi all,
@kevemueller I don't agree with this, as I really don't know what the issue here is. SSH works, even after reset, I see no errors for sshd -T on a 16.7.4 system. Can you please elaborate?
Setting host keys explicitly does not deprecate them, it disables them completely which is not what we want. :(
Cheers,
Franco

(stupid github closed this issue via a git revert)

I say we remove DSA support as in the not to distant future SSH will be dropping DSA support

it is removed, but the old DSA key keeps working for as long as it's not reset ;)

Oh okay

The problem is reproduceable. Install OpnSense from CD, upgrade via UI, reboot, perform a factory reset (reboots).
You will end up with no /usr/local/etc/ssh/ssh_host_dsa_key and sshd -T still looking for it.
The error line is not only produced by sshd -T, but also in /var/log/system.log at every new ssh connection.
I am agnostic about DSA, but if you want to remove it you should also remove it from /usr/local/etc/ssh/sshd_config. The patch above does  exactly that, by specifically enlisting valid HostKeys, so sshd does not revert to the default list, which still includes dsa host key.
I agree that sshd continues operating, but getting rid of
sshd[...]: error: Could not load host key: /usr/local/etc/ssh/ssh_host_dsa_key
in system.log is IMO more than a cosmetic change.

Alright, fine, I just needed to know where the error was :)
I've added a new commit that generates the keys before writing the config, so we can pick up the available files from the file system including the deprecated ones.
Cheers,
Franco

I found another bug with factory reset: the keys are not properly deleted and pulled back into the new system. I expect this to go into 16.7.x after one round in opnsense-devel, so 16.7.7.
The whole file has practically been rewritten... better safe than sorry.

Merging all changes in 16.7.8. Extra caution taken. :)


#161255248 Refactor code follow feature

What does this PR do?
It refactors code that implements the following endpoints

/api/profile/<username>/follow
/api/profile/<username>/followers
/api/profile/<username>/following

Description of Task to be completed?

Usernames provided are validated
Users with similar names e.g martin and martinkaburu can be differentiated
Code uses serializers to complete tasks

Related PivotalTracker stories
#161255248

Pull Request Test Coverage Report for Build 281

42 of 46 (91.3%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 92.118%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

authors/apps/profile/api/views.py
20
22
90.91%

authors/apps/profile/models.py
2
4
50.0%

Totals

Change from base Build 278:
0.3%

Covered Lines:
970

Relevant Lines:
1053

💛  - Coveralls


[Improve] Unified Naming Rules for Enumeration Classes (#3133)

#3133 All enumeration classes have been suffixed with Enum, and enumeration class members have been changed to uppercase letters separated by underscores.

hi, @xsymin Thanks for the contribution.
Could you help to resolve the conflicts & checkstyle & test-cases issue before reivew ?
Thanks a lot~


Handle escaped characters in StringLiterals

This change is

Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Option to persist React Hook Form DevTool in Production

It would be nice to have an option to display DevTool also in Production env.
For inspiration nice workaround was introduced by ReactQuery here: https://github.com/TanStack/query/discussions/3296 . Would it be possible something similar also here?

why would you want to ship dev tool to your prod code? it will incase your package size quite a lot.

+1

I can't get the devtools to work when used in module federation (remote app), and this might be the solution for just that.


Adding extensive automated tests

I’ve been thinking about how we could extensively test our library against a language to be able to affirm that the library officially supports it.
My first ideas were:

We should explicitly test all numbers from 0 to 100, as they are really common.
We should test some different combinations of digits and different lengths of numbers.
We should test edge cases or special cases for each language.

Doing the first it’s easy, but doing the second is not as easy. We can’t test all numbers, so we need to select a set of different numbers to check. After some time doing different, crazy, things, I got this list:
1234
23451
345612
4567123
56781234
678912345
7890123456
89091234567
909812345678
987123456789
98761234567890
876512345678909
7654123456789098
65431234567890987
543212345678909876
4321123456789098765
32101234567890987654
210012345678909876543
1000123456789098765432
1234567890987654321
12345678909876543210
123456789098765432100
1234567890987654321000

It’s created by appending the next digit to the end and then moving the first digit to the end (and then doing the same but with the digit before).
It’s not perfect, but I think that it’s diverse enough to be able to say that if the parser works for these cases, it will probably work for the most common existing combinations.
What do you think guys? @arnavkapoor @lopuhin @kishan3
I built some spiders to scrape different websites to get these numbers in words and added here the datasets: https://github.com/noviluni/numbers-data
Sources:

https://www.tools4noobs.com/online_tools/number_spell_words/
https://math.tools/calculator/numbers/words
https://clevert.com.br/t/en/numbers_to_words/

Unfortunately, they don't support Hindi, but I can search a website supporting Hindi and create a new spider if it's necessary.
If you like this idea, we can use these CSVs directly in the tests (don't worry about this, @arnavkapoor, I could show you how I would do it).
In case you have another idea of input numbers for the tests, I can generate for you a dataset for a lot of locales with the numbers you want.
Let me know what you think or if you have any other idea/approach. :slightly_smiling_face:
Originally posted by @noviluni in https://github.com/arnavkapoor/number-parser/issues/15#issuecomment-659295377

Creating a new issue to keep track of the point mentioned by @noviluni to add extensive testing.

Hi @arnavkapoor
I found that clevert.com.br does support Hindi. I will add the data when having a moment to the repo: https://github.com/noviluni/numbers-data
The only downside is that it has a maximum number supported, so the "permutations" file will be incomplete.

@arnavkapoor
Data is for hindi is now available:

hundreds
permutations

Thanks a lot @noviluni , I will now add support for hindi too in the latest PR #27. I have also mentioned the planned strategy for adding tests in this PR , please do review whenever possible.

Closing issue as #27 is merged.


typing: use autotyping to reduce implicit types

Summary

This PR runs Autotyping on all nextcord/ files. This uses LibCST's codemod to refactor code to add explicit -> None returns, __repr__ annotations, e.t.c.
This improves the pyright --veriftypes score by 0.8%, which does not sound like much but is a lot for it being automatic.
Checklist

[x] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.
[x] I have run task pyright and fixed the relevant issues.

[ ] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

libcst seems to have no wheels for 3.11, and pre-commit.ci does not allow customisation of the python version, not sure what to do.


ISPN-7999 NPE in ScatteredCrashInSequenceTest

https://issues.jboss.org/browse/ISPN-7999

Integrated, thanks Radim!


README.md: instructions for HTTParty's parsed_response

Thanks for submitting, however I am going to close this pull request for now.
I am working on building out a wiki section with examples for each of the specific HTTP libraries, and I will definitely pull you example then.

:+1:  Useful

@jtmkrueger thank you for reminding us about it.
I guess there was no time to build this wiki sections @davidbegin mentioned.
After reviewing it, I agree it's useful to have it in the readme. @Nowaker thank you for the reasoning 👍


[DGB-627]  Implement the ability to create data tables for a transaction entity

Implemented for m_savings_account_transactional
Description
Describe the changes made and why they were made.
Ignore if these details are present on the associated Apache Fineract JIRA ticket.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[ ] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[ ] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm with details of any API changes

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

Hi @y1zozulia  and @ysanko-tranzzo . Welcome to the Fineract upstream project. Your PR looks like it solves a real problem but I will like to ask you to create a JIRA ticket on the open source issue system here: https://issues.apache.org/jira/projects/FINERACT/issues/, this way we have a public description of the issue you are solving and can review this PR with more context. Also edit your PR to include the FINERACT-xxx issue name as oppose to your internal issue tracking number: DGB-627. After that, we will be able to review this PR and get it merged if everything looks Ok. Does this make sense to you all?

@ysanko-tranzzo I agree with @awasum that it would be nice to have a JIRA issue for this on the Apache JIRA, instead of DGB-627 which we're assuming is some internal issue tracking you use. Would you be willing to raise one, with a suitable description? Do you need any assistance to get started on the ASF JIRA?

@ysanko-tranzzo I agree with @awasum that it would be nice to have a JIRA issue for this on the Apache JIRA, instead of DGB-627 which we're assuming is some internal issue tracking you use. Would you be willing to raise one, with a suitable description? Do you need any assistance to get started on the ASF JIRA?

Hi @vorburger and @awasum. This PR need just only for our fork.

Hi @vorburger and @awasum. This PR need just only for our fork.


fn_args_layout is marked as stable, but not available on stable toolchain

I don't know whether this is an issue with rustfmt, rustup, or something else, but I am on the latest stable Rust x86_64-pc-windows-msvc toolchain and rustup component add rustfmt indicates that I am up to date. Despite fn_args_layout being marked as stable in the documentation, running cargo fmt with it set returns an "Unknown configuration option" error.
Is there any estimate for when it will be available?

It has been stable since rustfmt 1.3.0 [1]. I cannot be certain about when the version will be available via the stable channel, though it's likely that it would be available in the next release.
[1] https://github.com/rust-lang/rustfmt/blob/master/CHANGELOG.md#130-2019-06-09

The latest toolchain release was on July 3rd. I think it is odd that it is still not available. What makes you suggest that the next release will include it?


Missing mitochondrial transcripts

Describe the bug
RefSeq does not have transcripts for mitochondrial genes. Consequently, they are missing from CDOT.
To Reproduce
Steps to reproduce the behavior:

Go to https://reev.cubi.bihealth.org/internal/proxy/mehari/genes/txs?hgncId=HGNC:7461&genomeBuild=GENOME_BUILD_GRCH37&pageSize=1000

Expected behavior
We need chrMT transcripts in Mehari.
Additional context

https://github.com/varfish-org/varfish-server/issues/1419

Overall, we should either parse the RCRS entry from NucCore or use the ENSEMBL transcripts. Probably the latter is better as we otherwise won't have transcripts but would have to use the gene name for transcripts.

RNA genes from chrMT are not properly in CDOT https://github.com/SACGF/cdot/issues/72

The following chrMT transcripts have a CDS that is not a multiple of 3.
ENST00000361789.2
ENST00000361453.3
ENST00000361227.2
ENST00000361381.2
ENST00000361390.2
ENST00000362079.2

Poly-A is appended to the transcripts which we can emulate by padding the transcripts accordingly and adjusting the CDS on the fly.
cf. https://pubmed.ncbi.nlm.nih.gov/10076021/


Failed to query available provider packages

Good morning,
I tried this morining to init my terraform code with this provider and got the follwing error:
╷
│ Error: Failed to query available provider packages
│
│ Could not retrieve the list of available versions for provider hashicorp/azurecaf: provider registry registry.terraform.io does not have a provider named
│ registry.terraform.io/hashicorp/azurecaf
│
│ Did you intend to use aztfmod/azurecaf? If so, you must specify that source address in each module which requires that provider. To see which modules are currently depending on
│ hashicorp/azurecaf, run the following command:
│     terraform providers

I have entered the provider in the required_providers and created the provider configuration (empty). What am I doing wrong?
Best regards
skelso01

I found my fault. I forgot t define the provider in submodules.


comfui did not read the node

I installed this node on comfy ui, but comfui didn't read the node

@515753541 i have the exact same issue on windows. (on maocs the node worked out of the box).
initially, the node wouldn't load because it couldn't find refiners.fluxion.utils. could be that it cannot find the proper python module, then you have to hardcode the location in the python script.
when i tried fixing this by:

hardcoding the imports to use local files instead to both box_segmenter.py and grounding_dino.py.
creating my own utils.py with the required functions
implementing all the utility functions locally

the nodes would finally load after 1* being done, but then i hit another wall - the segmentation wasn't working correctly. then i tried fixing it with 2* and the masks were either completely empty or not properly segmenting the objects.

i tried several more fixes around the masking and coordinate scaling but nothing seemed to work quite right. might be worth looking at comfyui's native groundingdino implementation as a reference instead.
when reverting the imports to the original ones for grounding_dino.py the node again wouldn't load:

if you're still having issues or fixed it, i'd be curious to know if you've tried any different approaches that worked

@515753541 check out #475 - it's fixed now


[Fix] Fixed the issue of mismatched index between out_indices and layers.

Motivation
I found that there is a mismatched output between layers and out_indices config.
For example, the layer number ranges from 1 to 7, and out_indices are set to $[1, 7]$ by default.
However, the code here counted the layer index starting from 0 by default, which means that the output tensors are one layer earlier than expected tensors.
Modification
added index by one to consist with that in out_indices.
BC-breaking (Optional)
Use cases (Optional)
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
The documentation has been modified accordingly, like docstring or example tutorials.

Hi @tyl0622,
We'd like to express our appreciation for your valuable contributions to the mmdetection. Your efforts have significantly aided in enhancing the project's quality.
It is our pleasure to invite you to join our community thorugh Discord_Special Interest Group (SIG) channel. This is a great place to share your experiences, discuss ideas, and connect with other like-minded people. To become a part of the SIG channel, send a message to the moderator, OpenMMLab, briefly introduce yourself and mention your open-source contributions in the #introductions channel. Our team will gladly facilitate your entry. We eagerly await your presence. Please follow this link to join us: https://discord.gg/UjgXkPWNqA.
If you're on WeChat, we'd also love for you to join our community there. Just add our assistant using the WeChat ID: openmmlabwx. When sending the friend request, remember to include the remark "mmsig + Github ID".
Thanks again for your awesome contribution, and we're excited to have you as part of our community!


add support for arbitrary file descriptors on unix

This allows the use of arbitrary file descriptors on unix in response to issue #542. File Descriptors are supplied via a new variant of config::Bhaviour

Could you please explain why only unix is supported ?
And I am afraid that this PR doesn't work for telnet / ssh connection because isatty will return false (jline3 seems to use the system flag to skip isatty check). So I guess we need something more complicated than a file descriptor...

My main use case for this feature is working with pseudoterminals other than the one controlling the process.
Only Unix is supported because I don't have a Windows machine to work on and don't have any experience with Windows terminal handling. Not for any technical reason.
This does indeed not work well with things other than TTYs. I don't think you can reliably support anything that is not a TTY because you cannot change to raw mode without termios as far as I know. Do you think a flag to override isatty and just assume the user set everything up as needed would be a good idea? If so, Behavior::ArbitraryFileDescriptors seems like a good place to put it. Anyone using this should have to consciously make this choice, and the other behaviours likely don't need it.
As a side note: My current workaround for supporting ssh is to just hijack the pseudo terminal slave. If necessary, I send SIGSTOP to its controlling process to ensure it doesn't steal any of my input.


Error when converting from .flac to .wav

Hello,
I recently downloaded the Slakh dataset for use in the context of my thesis on musical source separation.
When trying to convert the data from .flac to .wav using the provided script (flac_converter.py in conversion/) that I call with
python conversion/flac_converter.py -i ../Data/slakh2100_flac/train/ -o ../Data/slakh2100_wav/train/ -c False

, at some point I keep getting the following error message repeatedly:
Input #0, flac, from '../../Data/slakh2100_flac/train/Track00061/stems/S00.flac':
  Metadata:
    ENCODER         : Lavf57.83.100
  Duration: 00:04:20.17, start: 0.000000, bitrate: 129 kb/s
    Stream #0:0: Audio: flac, 44100 Hz, mono, s16
Stream mapping:
  Stream #0:0 -> #0:0 (flac (native) -> pcm_s16le (native))
Error while opening decoder for input stream #0:0 : Resource temporarily unavailable

with different stems each time.
After this a few huge chunks of tracks are skipped, and the script goes on to convert a few more tracks before stopping completely. I also get the following somewhere near the end:
Input #0, flac, from '../../Data/slakh2100_flac/train/Track01142/stems/S06.flac':
  Metadata:
    ENCODER         : Lavf57.83.100
  Duration: 00:06:12.01, start: 0.000000, bitrate: 24 kb/s
    Stream #0:0: Audio: flac, 44100 Hz, mono, s16
Input #0, flac, from '../../Data/slakh2100_flac/train/Track01142/mix.flac':
  Metadata:
    ENCODER         : Lavf57.83.100
  Duration: 00:06:12.01, start: 0.000000, bitrate: 329 kb/s
    Stream #0:0: Audio: flac, 44100 Hz, mono, s16
Stream mapping:
  Stream #0:0 -> #0:0 (flac (native) -> pcm_s16le (native))
Error while opening decoder for input stream #0:0 : Resource temporarily unavailable
Stream mapping: time=00:00:56.63 bitrate= 703.6kbits/s speed=1.05x
  Stream #0:0 -> #0:0 (flac (native) -> pcm_s16le (native))
Error while opening decoder for input stream #0:0 : Resource temporarily unavailable
Traceback (most recent call last):
  File "flac_converter.py", line 207, in <module>
    args.end, args.num_threads, args.verbose)
  File "flac_converter.py", line 111, in _apply_ffmpeg
    pool.map(_apply_convert_dir, track_directories)
  File "/home/mthevoz/miniconda3/envs/MSS/lib/python3.7/multiprocessing/pool.py", line 268, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/home/mthevoz/miniconda3/envs/MSS/lib/python3.7/multiprocessing/pool.py", line 657, in get
    raise self._value
  File "/home/mthevoz/miniconda3/envs/MSS/lib/python3.7/multiprocessing/pool.py", line 121, in worker
    result = (True, func(*args, **kwds))
  File "/home/mthevoz/miniconda3/envs/MSS/lib/python3.7/multiprocessing/pool.py", line 44, in mapstar
    return list(map(*args))
  File "flac_converter.py", line 109, in _apply_convert_dir
    ffmpeg_func, verbose=verbose)
  File "flac_converter.py", line 90, in _convert_folder
    ffmpeg_func(src_path, out_stems_dir, verbose=verbose)
  File "flac_converter.py", line 45, in _flac_to_wav
    ffmpeg.input(input_path).output(output_path).run_async(overwrite_output=not verbose)
  File "/home/mthevoz/miniconda3/envs/MSS/lib/python3.7/site-packages/ffmpeg/_run.py", line 285, in run_async
    args, stdin=stdin_stream, stdout=stdout_stream, stderr=stderr_stream
  File "/home/mthevoz/miniconda3/envs/MSS/lib/python3.7/subprocess.py", line 775, in __init__
    restore_signals, start_new_session)
  File "/home/mthevoz/miniconda3/envs/MSS/lib/python3.7/subprocess.py", line 1453, in _execute_child
    restore_signals, start_new_session, preexec_fn)
BlockingIOError: [Errno 11] Resource temporarily unavailable

In the end my terminal becomes unresponsive and I end up with a bunch of tracks that seem to be properly converted but with big chunks missing in the middle and in the end. Any idea what's going on here?

Hmmm... I can't say that I've seen this issue before, but my guess is that there might be a bug with the threading. Can you try explicitly setting the [--num-threads NUM_THREADS] option to 1 or 2 form the command line and see if that still causes the issue?

I tried doing that and unfortunately it doesn't solve the issue, although changing the number of threads also changes which files get converted and which files do not, 2 threads giving the most successes as far as I can see. The process is still a bit random and there are slight differences between the end results of each run I attempt even with the same number of threads.
Note that I have the latest version of ffmpeg installed if that's relevant.

Hi, I wrote a piece of code that does the conversion with sox instead of ffmpeg and that worked fine for me. Hope it helps anyone that may run into the same issues:
import os
import argparse
import subprocess
from distutils.util import strtobool

def flac_to_wav(in_folder):
    extensions = ('.flac')
    for root, _, files in os.walk(in_folder):
        for file in files:
            if file.endswith(extensions):
                filename_wav = file.replace('.flac', '.wav')
                subprocess.run(['sox', root+'/'+file, '-r',
                                '44100', '-c', '2', '-b', '16', root+'/'+filename_wav])
                os.remove(root+'/'+file)

if __name__ == '__main__':
    parser = argparse.ArgumentParser()
    parser.add_argument('--input-dir', '-i', type=str, required=True,
                        help='Base path to input directory. (Required)')
    parser.add_argument('--verbose', '-v', type=lambda x:bool(strtobool(x)), default=False, required=False,
                        help='Whether to print messages while processing. (Optional)')

    args = parser.parse_args()
    flac_to_wav(args.input_dir)

Note that this version does not take an output directory but rather does the conversion "in place" by converting to .wav then deleting the original .flac file. I ran this on a copy of the original dataset so as to obtain the same result.
Cheers.

Hey, I'm glad you were able to find a solution. I think I might also have a solution, but I haven't tested it yet. I'll test your and my solutions next week and merge them in. Thanks for checking it out.

Based off of this:
https://stackoverflow.com/questions/33341303/ffmpeg-resource-temporarily-unavailable
I was able to fix this issue by adding 'threads=1' to every call of ffmpeg.input() and ffmpeg.output(). The parameterized thread count still works, as it splits apart the work across all of the tracks, however, my change forces ffmpeg to use only one thread at a time when dealing with the conversion of one file. I am not sure if this is the right way to fix this, especially if @ethman experiences no problems with the code as it stands. These observations seem to suggest that ffmpeg does not use one thread by default, and this can cause issues for some systems.


Add a linter to the ansible playbook as workflow

I am currently adding a new linter to the ansible:
https://github.com/peterzhuamazon/ansible-playbook/blob/opensearch-ansible-linter/.github/workflows/lint.yml
I see a lot of warnings and errors being exposed:
https://github.com/peterzhuamazon/ansible-playbook/runs/6335919476?check_suite_focus=true
I am wondering if this is worth adding as it probably will better serve our code.
@saravanan30erd @prudhvigodithi thoughts?
Thanks.

sure, it will be useful.


fixing class cast to support OPTIONAL_MATCH and path

Hi Wolfgang,
this is what I did to fix the ClassCastException. Togehter with the new ResultSettings flag I'm now able to use OPTIONAL_MATCH and path.
Best,
Marco

Your pull request has been merged, thanks for your contribution.
Best regards,
Wolfgang


Deploy demos to shinyapps.io

Deploys several demos to shinyapps.io for live access
Created "gmod" account under my email to do so. Let's you have five running apps on free account, so tries to consolidate the demos to that number

Includes app from @carolinebridge-oicr that is currently in the embedded_demos folder, but copied back to here. Can link to it from embedded_components page in docs though


Azure Health Bot Omission

Just wondering if there is any reason why the Azure Health Bot is not referenced in the section "Choose the right chatbot solution for your use case"?
https://docs.microsoft.com/en-us/azure/bot-service/bot-overview?view=azure-bot-service-4.0
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 49545c0d-aa2e-4336-731c-8f8dc9aeee12
Version Independent ID: a3de46d4-9ab3-4403-94c3-3c5704f73a2c
Content: Choose the right chatbot solution for your use case - Bot Service
Content Source: articles/bot-overview.md
Service: bot-service
GitHub Login: @JonathanFingold
Microsoft Alias: iawilt

Thanks for the question. For health care scenarios, use Health Bot. I'll work on updating the article.


Please show performance improvements in a table (before and after values) with the code change.
If you didn't do "make style" please run it and confirm no issues.
Run the fast tests and check for pass: tests/ci/fast_tests.sh and tests/ci/fast_tests_diffusers.sh
Run relevant slow tests and check for pass:  python -m pip install '.[tests]'; GAUDI2_CI=1 RUN_SLOW=1 python -m pytest tests/test_trl.py -s -v -k test_calculate_loss
You may want to add additional CI tests for this particular PR to test mixtral.

@libinta , this is a post 1.18 PR, thx.

Hello @lkk12014402 , Here are a few preliminary checks to complete for a review. Please show performance improvements in a table (before and after values) with the code change. If you didn't do "make style" please run it and confirm no issues. Run the fast tests and check for pass: tests/ci/fast_tests.sh and tests/ci/fast_tests_diffusers.sh Run relevant slow tests and check for pass: python -m pip install '.[tests]'; GAUDI2_CI=1 RUN_SLOW=1 python -m pytest tests/test_trl.py -s -v -k test_calculate_loss You may want to add additional CI tests for this particular PR to test mixtral.

enable pad_max when not packing data
not pad_max when not packing data

The README changes are covering both mistral and mixtral, so the title of this PR should also include mistral?

yes, I validated mistral sft and dpo training with trl on Gaudi

@lkk12014402 , As you are adding to the README, can you make sure you are getting best (highest) possible throughput for the batch size and other parameters noted in the README. Thanks!

@lkk12014402 , As you are adding to the README, can you make sure you are getting best (highest) possible throughput for the batch size and other parameters noted in the README. Thanks!

yes, I give the training commands to the README.

@lkk12014402 Please sync your branch with main and run make style again

yes, Thx~
have run make style now.


Add step definition for checking cookies:

@Then /^the browser should have an? "(?P[^"]*)" cookie$/

@msabramo could you resend PR against develop branch?


Adds Procfile to make it able to not run on Heroku

So now if you deploy it on Heroku it will be able to not run.
Same result if you don't deploy it at all.

should run nowhere

Tried it in Heroku-18 stack, works the same.


Make the Changelist public

Fixed in 1afb860aeae9695dfabbaf8cbf6c8ed9085f7fb2

No, good catch!

Please test again now @ntorsen

Please test again now @ntorsen

Working good 🥇


Edit inventory items

As a business owner, I want to edit inventory item(s), so I can easily change item details.

I've changed this issue and user story from tagging items to editing items as editing items allows for adding/editing tags of items as well.


Warn about unmatched records in ndjson-join

I'm using ndjson-join to make 1 to 1 joins between two files and I'd like to check that after the operation each record of the LHS stream is present in the final recordset.
Counting the final records by piping into wc -l and comparing to the length of the recordset might be misleading as we could have 0 matches for some records and 2 or more matches for some other.
For my use case It would be great to have a command line switch enabling warnings about unmatched records in  the LHS stream, for example --check-joins or something like that.

Release 0.3.1 adds support for left, right and outer joins (#23). This can be used to check for missing joins by filtering for null after joining.


Swarm process crashes on unsupported network driver

To reproduce the error, start a sward process and create a network by
swarmctl network create  --name n1 --driver testdriver

The error swarmd reports is
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0xa18219]

goroutine 130 [running]:
panic(0xc98760, 0xc4200100e0)
	/usr/local/go/src/runtime/panic.go:500 +0x1a1
github.com/docker/swarmkit/manager/controlapi.validateDriver(0xc420483c80, 0x0, 0x0, 0xd9e2a0, 0xd, 0x1, 0x1)
	/go/src/github.com/docker/swarmkit/manager/controlapi/common.go:97 +0x119
github.com/docker/swarmkit/manager/controlapi.validateNetworkSpec(0xc4200533c0, 0x0, 0x0, 0xc, 0xc4202f79d8)
	/go/src/github.com/docker/swarmkit/manager/controlapi/network.go:82 +0x9b
github.com/docker/swarmkit/manager/controlapi.(*Server).CreateNetwork(0xc4204dd620, 0x7f4d6f8251d0, 0xc420872810, 0xc4200261f0, 0x1, 0x1, 0x7f4d6f8251d0)
	/go/src/github.com/docker/swarmkit/manager/controlapi/network.go:99 +0x4f
github.com/docker/swarmkit/api.(*raftProxyControlServer).CreateNetwork(0xc42007a820, 0x7f4d6f8251d0, 0xc420872780, 0xc4200261f0, 0x0, 0x12, 0x12)

Ideally, the swarmctl should give the message like "unsupported network driver". Thanks.

I don't think network drivers are supported when using swarmd directly, but it's probably worth adding a check that makes validateDriver fail if pg is nil.

@aaronlehmann I am not sure if I understand your comment correctly, if I run swarmctl network create  --name n1 --driver overlay. The swarm process can create the network. Looks like the code to initialize the driver is [1].
[1] https://github.com/docker/swarmkit/blob/3794c9db58eaf2c9a0103eb7f2a6a7b4bf794c58/manager/allocator/networkallocator/drivers_network.go

My understanding is that overlay networking isn't supported when using swarmd directly. swarmd is more of a test tool, and doesn't support all features that swarm mode in Docker does.

I see.
But as you said, it is still worthwhile to check the pg==nil to ensure the swarmd does not exit with panic.

I think it makes sense to fail it and return messages like "swarmd doesn't support this network driver".

I tried loading an unknown driver with libnetwork, it gives the following messages:
WARN[0000] Unable to locate plugin: newdriver, retrying in 1s
WARN[0001] Unable to locate plugin: newdriver, retrying in 2s
WARN[0003] Unable to locate plugin: newdriver, retrying in 4s
WARN[0007] Unable to locate plugin: newdriver, retrying in 8s
2017/02/15 17:00:50 controller.NewNetwork: plugin not found

And the docker daemon engine has the same behavior. So there maybe something missing in swarmkit to initialize its plugins.


[TACHYON-1098] Stripping path from TachyonURI

https://tachyon.atlassian.net/browse/TACHYON-1098

Now AbstractLineageClient passes just the path part to
LineageMasterClient#createLineage, which, in turn, has been updated
accordingly.
InodeTree#createPath passes the correct path part of the TachyoURI to
PathUtils#getPathComponents

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/5657/
Test PASSed.

Thanks for fixing this bug and improving PathUtils#getPathComponents!

The new commit should address your comments. Thanks guys!

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/5661/
Test PASSed.

+1 LGTM. Thanks for the patch!

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/5671/
Test PASSed.

LGTM. Thanks for the fixes!


bug in handling of tokens vs list of tokens

in this code:
https://github.com/foundation-model-stack/foundation-model-stack/blob/main/fms/utils/tokenizers.py
sentencepiece vs char tokenizer convert_tokens_to_ids, when passed a list of tokens or a single token may behave differently for different implementations. we should make sure it works consistently in the same way.
This broke some testing code when switching between tokenizers.

Assigned to me.


Question: How to make text transcriptions?

Hi, I want to make transcriptions for audio datasets. I need an ability to make transcription either of single choosen segment or all utterance. Is it possible to perform in current UI? How to update the code if not?

Hello!
The primary use case for Pyrenote atm is class-based labeling for bioacoustic work, so we don't have transcriptions ready at the moment. That being said, Audino, the repo that Pyrenote is based on, was focused on human transcriptions. You may find that more useful for your specific case.


Can't get MediaQuery.of(context).viewInsets.bottom when in fullscreen and the keyboard is open.

When the keyboard opens, MediaQuery.of(context).viewInsets.bottom changes to reflect the new bottom (which is now at the top of the keyboard).
However, the Android app may be in fullscreen:
    <?xml version="1.0" encoding="utf-8"?>
    <resources>
        <style name="LaunchTheme" parent="@android:style/Theme.Black.NoTitleBar">
            <!-- Show a splash screen on the activity. Automatically removed when
                 Flutter draws its first frame -->
            <item name="android:windowBackground">@drawable/launch_background</item>
            <item name="android:windowFullscreen">true</item>
        </style>
        <color name="xxx">#FFFF00</color>
    </resources>

(See: https://medium.com/@diegoveloper/flutter-splash-screen-9f4e05542548)
When this is the case, viewInsets.bottom will always be 0.0, even when the keyboard is open, which is wrong and creates all sorts of problems.

Hi all, any update on this would be very appreciated!

Just encountered the same issue, any update on this?

Same problem on iOS, not even fullscreen

Also seeing this on IOS. My use case is a page within a PageView, plus I am doing custom scaling to the page widget, but the MediaQuery.of(context).viewInsets.bottom value is always zero whether keyboard is present or no.
I can post a gist if it would help to troubleshoot the issue.

Any news about this?

Also had this issue but after setting this scaffold property to false I am getting the insets again. Did you guys also set it to false? See: https://api.flutter.dev/flutter/material/Scaffold/resizeToAvoidBottomInset.html

I set resizeToAvoidBottomInset to false. Same problem.

I found that mediaquery.of (context). ViewInsets. Bottom always returns zero when used with bottomNavigationBar

I have this issue as well

I ran into this issue by adding to existing app, but my existing app not even set the fullscreen option.
It work well on iOS, but not in Android.
resizeToAvoidBottomInset is also useless in this situation.
Hope we got some news about this.

when I set FlutterActivity android:windowSoftInputMode="adjustResize" or other attributes，MediaQuery.of(context).viewInsets.bottom not return 0.0
I think viewInsets.bottom‘s value have a relationship with windowSoftInputMode. I just try this one.

if u are using sub widgets try pass scaffold buildcontext to this widget.
and try  MediaQuery.of(context).viewInsets.bottom  with scaffold context not child widgets one.

So I use a bottom navigation bar and I am trying to get the height of the keyboard.
When I call
mediaquery.of (context).viewInsets.Bottom

It always includes the keyboards height + the bottom navigation bar height. Is there anyway to just get the keyboard height?
I have tried to use
SizedBox(
              height: MediaQuery.of(contextField).viewInsets.bottom - widget.bottomNavigationBarKey.currentContext.size.height,
              child: Container(color:Colors.pink),
            )

But this fails because its trying to reference something that hasn't been built (bottomNavigationBarKey)  in the build method.

I think is worth noting that this issue also occurs when not using an app in fullscreen, but the demo app template instead. If one tries to get the bottom viewInset from a different context inside a Scaffold with a BottomNavigationBar, the viewInset is always zero.
Using a GlobalKey context to the Scaffold works correctly, but it's not reactive when the keyboard opens and closes.
Should the title of the issue be updated?
Minimal code sample

import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

final GlobalKey<ScaffoldState> scaffoldKey = GlobalKey();

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      key: scaffoldKey,
      appBar: AppBar(
        title: Text("View insets"),
      ),
      body: ScaffoldBody(),
      bottomNavigationBar: BottomNavigationBar(items: [
        BottomNavigationBarItem(
          icon: Icon(Icons.home),
          title: Text("Tab 1"),
        ),
        BottomNavigationBarItem(
          icon: Icon(Icons.settings),
          title: Text("Tab 2"),
        ),
      ]),
    );
  }
}

class ScaffoldBody extends StatelessWidget {
  const ScaffoldBody({
    Key key,
  }) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Center(
      child: Column(
        mainAxisAlignment: MainAxisAlignment.center,
        children: <Widget>[
          FlatButton(
            onPressed: () {
              final mq1 = MediaQuery.of(context);
              final mq2 = MediaQuery.of(scaffoldKey.currentContext);
              print("View insets (current context) = ${mq1.viewInsets}");
              print("View insets (scaffold context) = ${mq2.viewInsets}");
              print("");
            },
            child: Text("Print view insets"),
          ),
          TextField(
            decoration: InputDecoration(hintText: "Some textfield..."),
          ),
        ],
      ),
    );
  }
}

Any other news about it?

try MediaQuery.of(context).viewInsets.bottom with scaffold context not child widgets one.

Works for me

Hi @marcglasberg
Are you still facing this issue?
With the code bellow the print output is:
Restarted application in 1,336ms.
I/flutter ( 5326): Bottom is: 0.0

When the keyboard opens it prints:
I/flutter ( 5326): Bottom is:<PHONE_NUMBER>909091
Tested on Pixel 3a emulator with the fullscreen configuration in the Android side provided by you.
sample code
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

void main() => runApp(MaterialApp(home: InputPage(),));
class InputPage extends StatefulWidget {
  @override
  State<StatefulWidget> createState() {
    return InputPageState();
  }
}

class InputPageState extends State<InputPage> {

  void show() async {
    showGeneralDialog(
        context: context,
        barrierDismissible: true,
        barrierLabel:
        MaterialLocalizations.of(context).modalBarrierDismissLabel,
        barrierColor: Colors.black45,
        transitionDuration: const Duration(milliseconds: 200),
        pageBuilder: (BuildContext buildContext, Animation animation,
            Animation secondaryAnimation) {
          return AlertDialog(
              contentPadding: EdgeInsets.all(0),
              content: Container(
                  height: 200,
                  width: 200,
                  color: Colors.red,
                  child: TextField(
                    keyboardType: TextInputType.multiline,
                    minLines: 5, //Normal textInputField will be displayed
                    maxLines: 5, // when user presses enter it will adapt to it
                    textAlign: TextAlign.left,
                  )));
        });
  }

  @override
  Widget build(BuildContext context) {
    final bottom = MediaQuery.of(context).viewInsets.bottom;
    print('Bottom is: $bottom');

    return Scaffold(
      appBar: AppBar(
        title: Text("Test"),
      ),
      body: Center(
        child: TextField(),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 1.20.3, on Mac OS X 10.15.6 19G2021, locale en)
    • Flutter version 1.20.3 at /Users/pedromassango/dev/SDKs/flutter_stable
    • Framework revision 216dee60c0 (8 days ago), 2020-09-01 12:24:47 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.7)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.7, Build version 11E801a
    • CocoaPods version 1.9.3

[!] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] IntelliJ IDEA Community Edition (version 2020.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 48.1.4
    • Dart plugin version 202.7206

[✓] VS Code (version 1.48.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.13.2

[✓] Connected device (1 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86 • Android 11 (API 30) (emulator)

! Doctor found issues in 1 category.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

try MediaQuery.of(context).viewInsets.bottom with scaffold context not child widgets one.

Try to use scaffold's context instead of the local widget's. This situation is usually caused by incorrect context.

This is reproduceable in flutter beta 1.23.0-18.1.pre on an Android 10 emulator or device.  On an Android 11 device, it works as expected.
The sample code from @pedromassangocode 's comment https://github.com/flutter/flutter/issues/25050#issuecomment-689567401 is sufficient to reproduce this.
Steps to reproduce
Part 1: Control test

Create a new flutter project
Replace main.dart with the code from https://github.com/flutter/flutter/issues/25050#issuecomment-689567401
Run the flutter app on an Android 10 emulator or device; it should work as expected, showing a bottom: with some non-zero value.

Part 2: Introducing bug

Modify android/app/src/main/res/values/styles.xml, adding <item name="android:windowFullscreen">true</item> under <style name="NormalTheme" [...] to set the app as fullscreen
Rerun the flutter app on an Android 10 emulator or device.  Now notice that the viewInsets are always zero even when the keyboard is up.

@azack @pedromassangocode You just need to use the right context to work, The correct context is a full screen widget.
For example: scaffold Stack ...

@noteScript In my testing it appears to depend on the method in which the app is set to fullscreen, even when checking the correct context -  see the table below.
MediaQuery.of(SomeContext).viewInsets.bottom on API 29 & 30 Emulators
Using sample code above, with minor change to also examine Scaffold context

FullScreen Method
Works on API 29?
InputPageState / Scaffold
Works on API 30?
InputPageState / Scaffold

SystemChrome.setEnabledSystemUIOverlays([]);
Yes
393
Yes
297

<item name="android:windowFullscreen">true</item>
No
0
Yes
249

Both of the above
No
0
Yes
297

Not fullscreen
Yes
337
Yes
249

@azack
Sorry, I didn't encounter this situation. The version I used is:
Flutter channels:
  master
  dev
  beta
* stable

Flutter 1.22.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 84f3d28555 (- -) • 2020-10-15 16:26:19 -0700
Engine • revision b8752bbfff
Tools • Dart 2.10.2

Thanks for the additional information, @noteScript .  Maybe there's something else varying in our environments and with the original reporter.
I continue to see the bug with latest stable 1.22.4:

Method
Flutter Version
Works on API 29?
Bottom
Works on API 30?
Bottom

<item name="android:windowFullscreen">true</item>
1.22.4
No
0
Yes
249

flutter doctor
[√] Flutter (Channel stable, 1.22.4, on Microsoft Windows [Version 10.0.19042.630], locale en-US)
    • Flutter version 1.22.4 at development\flutter
    • Framework revision 1aafb3a8b9 (5 days ago), 2020-11-13 09:59:28 -0800
    • Engine revision 2c956a31c0
    • Dart version 2.10.4

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[!] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[!] VS Code (version 1.51.1)
    • VS Code at AppData\Local\Programs\Microsoft VS Code
    X Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[√] Connected device (1 available)
    • sdk gphone x86 (mobile) • emulator-5554 • android-x86 • Android 11 (API 30) (emulator)

! Doctor found issues in 2 categories.

Problem still exists in stable channel 2.0.5 Android and iOS
Instead of MediaQuery.of(context).viewInsets.bottom I am using WidgetsBinding.instance.window.viewInsets.bottom, but on some android 11 and 10 it remains as 0 when the keyboard is open.

I'm sorry to the maintainer for the bump, to this date as I write this, the bug is still exists in Android.
If you use GlobalKey<ScaffoldState> for the Scaffold's key, then you're probably having the same problem as I am.
The quick solution on how I fix this is simply by using MediaQuery.of(_scaffoldKey.currentContext!).viewInsets.bottom.

@andersonmendesdev
In the version of 'stable channel 2.0.5', this problem has been fixed, and mediaquery.of(context). Viewinsets. Bottom is no longer needed. However, in the version of 'stable channel 2.2.2', this problem occurs again.


Fix abstraction

Codecov Report

Merging #70 into master will decrease coverage by 0.06%.
The diff coverage is 70%.

@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
- Coverage   84.03%   83.97%   -0.07%
==========================================
  Files          20       20
  Lines        1510     1504       -6
  Branches      148      148
==========================================
- Hits         1269     1263       -6
  Misses        201      201
  Partials       40       40

Impacted Files
Coverage Δ

components/hatch.py
100% <ø> (ø)
:arrow_up:

autonomous/autonomous.py
64.28% <ø> (-0.19%)
:arrow_down:

components/climb.py
77.87% <ø> (+0.28%)
:arrow_up:

automations/climb.py
61.66% <0%> (ø)
:arrow_up:

components/cargo.py
92.53% <100%> (ø)
:arrow_up:

automations/cargo.py
88.09% <50%> (ø)
:arrow_up:

components/vision.py
96.07% <75%> (-1.04%)
:arrow_down:

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 67d7d93...2286556. Read the comment docs.


Update C# templates and generators to Cloud Adapter

This item depends on completion of #3307

Fixed on #3224


fix resolving alias with proxy alias

Issue https://github.com/zendframework/zend-servicemanager/issues/75 fix.

Duplicate of #71

Closed as duplicate of https://github.com/zendframework/zend-servicemanager/pull/71


Discrepancy between GN and Ensembl

ENSP00000328848 has 852 amino acids:
http://annotation.genomenexus.org/ensembl/transcript?proteinId=ENSP00000328848
But Ensembl has 995:
http://useast.ensembl.org/Homo_sapiens/Transcript/ProteinSummary?db=core;g=ENSG00000182872;r=X:47145279-47186813;t=ENST00000329236
It was found by Ritika in this patient (https://cbioportal.mskcc.org/case.do#/patient?studyId=mskimpact&sampleId=P-0027652-T01-IM6) for RBM10 mutation. The portal annotated as R775W but DMP annotated as R918W.

So grch37 has 852 AA:
http://grch37.ensembl.org/Homo_sapiens/Transcript/ProteinSummary?db=core;g=ENSG00000182872;r=X:47145279-47186813;t=ENST00000329236
and latest has 995:
http://useast.ensembl.org/Homo_sapiens/Transcript/ProteinSummary?db=core;g=ENSG00000182872;r=X:47145279-47186813;t=ENST00000329236

@ckandoth how do u think we should handle this ?

Apparently when using Refseq the change is R918W (merged=1 returns both ensembl transcript ids and refseq ids):
http://grch37.rest.ensembl.org/vep/human/hgvs/X:g.47045676C>T?content-type=application/json&merged=1
So in vcf2maf the canonical isoform's Refseq id has preference over the ensembl transcript id inidicated in the overrides file:
https://github.com/mskcc/vcf2maf/blob/master/data/isoform_overrides_at_mskcc#L330

Since we are using ensembl's vep it makes more sense to use ensembl transcript ids in genome nexus. Hopefully these kind of issues will be solved when showing all transcripts

Ensembl ENST isoforms for CRLF2, RBM10, and FAM58A in vcf2maf's isoform_overrides_at_mskcc are known to differ from DMP's preferred RefSeq NM_ isoforms. This is because there are no ENST isoforms (mappable to GRCh37) that are equivalent to those NM_ isoforms. Note that RBM10's ENST00000329236.8 (mappable to GRCh38, but not GRCh37) is equivalent to DMP's preferred NM_001204468.1. Via the GRCh37 API, this isoform is stuck at the older version ENST00000329236.7.
DMP's solution is to run vcf2maf/VEP with the merged Ensembl+RefSeq cache, and use a --custom-enst override file that uses NM_001204468.1 for RBM10. The Ensembl+RefSeq merged cache is safe to use for the ~400 IMPACT genes, but there are known bugs in that cache in many of the ~20000 genes across the exome. This is why we don't use that merged cache on the research side, where we handle a lot of exomes.

@ckandoth thanks for the explanation. I guess we have to live with it then...


There is no ICustomNode interface

Followed the guide, and Godot.Resource seems to work fine, I got the CRR tab and everything. But the interface does not seem to exists?
Anything looks out of the ordinary here?
screenshot of setup

My bad, I had the interface renamed to ICustomNodeType and forgot to update the README.md. My latest commits have renamed the interface back to ICustomNode. If you download this repo again you should be good to go.
After reinstalling the plugin, you may also have to import the namespace of the plugin by adding using MonoCustomResourceRegistry to the top of your C# class whenever you want to implement ICustomNode.

@TandersT if my new commits fixed the interface problem for you, make sure to close this issue.

I can access the interface now with the new commit. Thanks!


Allow overrides of environment variables without mkForce

Currently to override any plugin envvars set by musnix, you must use lib.mkForce.  This makes it possible to disuse lib.mkForce, and is backwards compatible.
If someone can think of a way to test this, I'd be happy to add a test too.

Much obliged!


Change README.MD

missing "," in "Use with Camunda properties" :
container: '#canvas',
// make camunda prefix known for import, editing and export
moddleExtensions: {

Please provide a PR to contribute such small improvements.


Fixed the About Us Section

Related Issues

Info about Issue or bug
About section was overlapping with Navbar

Fixes: #43
Proposed Changes

Changed some of the CSS Properties(Z-Index)

Great! @devenkapoor


Update Kong, reference to image pull secrets

What this PR does / why we need it:
Remove legacy reference to imagePullSecrets variable that was renamed to pullSecrets in the last merge and modified from string to list.
Special notes for your reviewer:
When I started the PR that was merged by @hbagdi, the ImagePullSecret reference was modified, and the kong deployment was modified accordingly, but not the kong controller deployment.
Thanks @shashiranjan84 and @hbagdi for your time !
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/ok-to-test
/assign
/lgtm


Adding Laravel5 support and some unit and integration tests

Drops Laravel4 and PHP5.X support.
Adds unit and integration testing.
Adds Laravel5 support.

The build is currently failing  due to versioning issues. Currently digging into that.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2b24114). Click here to learn what that means.
The diff coverage is 67.46%.

@@            Coverage Diff            @@
##             master      #10   +/-   ##
=========================================
  Coverage          ?   67.46%
  Complexity        ?       37
=========================================
  Files             ?        9
  Lines             ?      126
  Branches          ?        0
=========================================
  Hits              ?       85
  Misses            ?       41
  Partials          ?        0

Impacted Files
Coverage Δ
Complexity Δ

src/ServiceProvider.php
0% <0%> (ø)
4 <4> (?)

src/Curl.php
0% <0%> (ø)
1 <1> (?)

src/Facades/Facade.php
0% <0%> (ø)
1 <1> (?)

src/OAuthBase.php
100% <100%> (ø)
3 <3> (?)

src/UrlBuilder.php
100% <100%> (ø)
10 <10> (?)

src/NonceFactory.php
100% <100%> (ø)
1 <1> (?)

src/TimestampFactory.php
100% <100%> (ø)
1 <1> (?)

src/FatSecretApi.php
100% <100%> (ø)
4 <4> (?)

src/FatSecret.php
68% <68%> (ø)
12 <12> (?)

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 2b24114...22511dd. Read the comment docs.

I integrated codecov to monitor testing coverage) and StyleCI, to automatically find an fix linting and indentation issues. I spent some time struggling to ensure Laravel 5.0 to 5.5 compaibility, which also would require php 5.4 to 7.0 compatibility but ended up dropping it, I'm not sure this is a worthy battle took some effort and nobody should be runnning those php versions on production anyway.
The current laravel compatibility would now be all 5.6., 5.7. and 5.8.* Laravel versions, the build is passing now.

 If something does not work or if you need some help with the Laravel 6 version let me know.


[Backport] Rename data to X in predict_proba. (#6555)

New Scikit-Learn version uses keyword argument, and X is the predefined
keyword.

Use pip to install latest Python graphviz on Windows CI.

Codecov Report

Merging #6586 (23f341f) into release_1.3.0 (a78d0d4) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@                Coverage Diff                @@
##           release_1.3.0    #6586      +/-   ##
=================================================
+ Coverage          79.89%   79.98%   +0.08%
=================================================
  Files                 12       12
  Lines               3497     3497
=================================================
+ Hits                2794     2797       +3
+ Misses               703      700       -3

Impacted Files
Coverage Δ

python-package/xgboost/dask.py
80.97% <100.00%> (ø)

python-package/xgboost/sklearn.py
89.21% <100.00%> (ø)

python-package/xgboost/tracker.py
95.11% <0.00%> (+1.12%)
: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 a78d0d4...23f341f. Read the comment docs.

Codecov Report

Merging #6586 (23f341f) into release_1.3.0 (a78d0d4) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@                Coverage Diff                @@
##           release_1.3.0    #6586      +/-   ##
=================================================
+ Coverage          79.89%   79.98%   +0.08%
=================================================
  Files                 12       12
  Lines               3497     3497
=================================================
+ Hits                2794     2797       +3
+ Misses               703      700       -3

Impacted Files
Coverage Δ

python-package/xgboost/dask.py
80.97% <100.00%> (ø)

python-package/xgboost/sklearn.py
89.21% <100.00%> (ø)

python-package/xgboost/tracker.py
95.11% <0.00%> (+1.12%)
: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 a78d0d4...23f341f. Read the comment docs.


etcd with tls auto-healing workflow

Hi,
I am trying to design a self-healing etcd cluster in which in case monitoring finds that a node is dead or it is too over-loaded, or maintenance, or etc., a service kicks in which spawns new etcd member, adds it to the cluster and everything is normal again. This workflow seems to work fine so far.
Now, I want to take this to production & I am trying to use etcd with tls.
I am thinking of spinning up an instance with the certificates embedded in user-data file (for AWS EC2).
The major problem/confusion that I have is that for TLS to work, we need IP SANs, which is only available 'after' the instance spins up. I have no way of knowing this before-hand and I am stuck with this cyclic dependency of sorts.
Anyone has a hint about how to resolve this. Also, is there any way to work without IP SANs or with wildcards?
Thanks.

Auto scaling is a bad idea. The more etcd members you have, the worse write and l-read throughput you would expect. Auto healing might make sense, you could use dns instead of ip address.

@xiang90 alright, I can work with auto-healing! :)
Can you please elaborate on 'use of dns instead of IP address'?

@krish7919 I think you can use domain name in alternative subject name. it is easy to know the domain name beforehand. i am not a security expert.
/cc @gtank might have better answer for you.

The major problem/confusion that I have is that for TLS to work, we need IP SANs, which is only available 'after' the instance spins up.

Yes, this is the main problem we have with TLS support (it also affects Kubernetes). It can't be helped on cloud providers, but there are a couple of workarounds we've developed:

Use DNS SANs (instead of IP) that you can dynamically assign to particular hosts. This works, but means you'll pre-generate a bunch of key/cert pairs and store them around to distribute to new nodes. That's pretty common practice (though not great- you have to keep all those keys around on a single machine that could be compromised and mess with DNS).

Have the nodes check their own IP address on boot, generate a certificate signing request ("CSR") with the correct SANs, and submit it to a CA service. This ensures that the private keys are only ever on the host that will use them and scales better since it doesn't require pregeneration. The downside here is that you'll need an API-driven CA service - something like Let's Encrypt, or cfssl - to issue the certificates dynamically.

@krish7919 I am closing this issue, since I think @gtank provides two approaches. The 1st one is what i suggested, but in more details.


Improve Microsoft.WindowsSandbox sample configuration

Summary of the pull request
Adds README.md for Windows Sandbox sample configuration.
Adds a secondary Sandbox configuration.
References and relevant issues
Detailed description of the pull request / Additional comments
Validation steps performed
PR checklist

[ ] Closes #xxx
[ ] Tests added/passed
[x] Documentation updated


Datagrid: Opening datepicker input closes the filter flyout

While working on the Datagrid filter flyout, I noticed that when I l click on the Datepicker input, it closes the filter flyout. I believe it's because we are utilizing useClickOutside and the Datepicker input is being rendered in the <body> instead of the filter flyout.
Things I have tried:

using DatePicker's appendTo prop and this does not seem to work.

https://user-images.githubusercontent.com/12755042/196998516-1ddf6c31-1574-4f76-b045-0defaa5e4aec.mov

If we make filter flyout not close when you click out of it it will solve this issue.


Aissata Coulibaly assignment

@Couliais As far as I can see there is no additional content from your side?
Please submit your PR to the solutions-engineer branch of this repo.


aligning quick start

Is there anyway we could just link to an installation page or embed content? I'm imagining the install process eventually being able to specify versions and other things, and we'll have to update a bunch of pages each time.

Is there anyway we could just link to an installation page or embed content? I'm imagining the install process eventually being able to specify versions and other things, and we'll have to update a bunch of pages each time.

I like that idea, but not going to work on it over the weekend...
I think it's probably worth landing this in the mean time; current READMEs are insufficient to run the examples


Support base-4.18

Also would be nice to bump the Cabal bound to allow 3.10. And the unix bound will have to be bumped too.


Update types for World.getSystem() to return the type of System requested

Currently the types for World.getSystem() say it returns a generic System.
This updates the types to clarify that it returns the type of System requested.

Thanks @zeddic


Automated profile creation using Induction dataset.

Provide the excel sheet containing the new people and it generates profiles in the output directory in the python_scripts

👷 Deploy Preview for virtualrealitylab processing.

Name
Link

🔨 Latest commit
ad327ca1d7f40fc35013a3c4630b78848311c160

🔍 Latest deploy log
https://app.netlify.com/sites/virtualrealitylab/deploys/675867ba5513c70008d002ad


Numba decorators aren't explicit about nopython mode

Decorators like @jit and @njit are causing warnings like the one below, as a result of reliance on implicit / default argument behavior currently in numba but that is deprecated and will change in 2 versions (0.57 now, to be changed in 0.59). We need to update our syntax.
/home/vr/code/looptrace/looptrace/trace_analysis_functions.py:1010: NumbaDeprecationWarning: The 'nopython' keyword argument was not supplied to the 'numba.jit' decorator. The implicit default value for this argument is currently False, but it will be changed to True in Numba 0.59.0. See https://numba.readthedocs.io/en/stable/reference/deprecation.html#deprecation-of-object-mode-fall-back-behaviour-when-using-jit for details.

https://numba.readthedocs.io/en/stable/reference/deprecation.html


Add ulimit config option

The new sysv init configs need an option to raise the processes ulimit.

Closed by #20