ImportError: dlopen: cannot load any more object with static TLS

Instructions
Hi, I had successfully installed svtk and used it correctly, but I'm not sure why it stopped working properly after some time. I wonder if I updated some Python modules. Can you help me with this?
When I tried to use it again, I encountered the following error message.
File "/xxx/software/miniconda3/lib/python3.9/site-packages/svtk/genomeslink.py", line 18, in
from scipy.sparse import csgraph
ImportError: cannot import name 'csgraph' from 'scipy.sparse' (unknown location)
Subsequently, I upgraded the scipy module, and it resulted in the following error.
Traceback (most recent call last):
File "/xxx/software/miniconda3/bin/svtk", line 45, in
import svtk.cli as cli
File "/xxx/software/miniconda3/lib/python3.9/site-packages/svtk/cli/init.py", line 12, in
from .adjudicate import main as adjudicate
File "/xxx/software/miniconda3/lib/python3.9/site-packages/svtk/cli/adjudicate.py", line 14, in
from svtk.adjudicate import adjudicate_SV
File "/xxx/software/miniconda3/lib/python3.9/site-packages/svtk/adjudicate/init.py", line 1, in
from .random_forest import RandomForest, rf_classify
File "/xxx/software/miniconda3/lib/python3.9/site-packages/svtk/adjudicate/random_forest.py", line 14, in
from sklearn.ensemble import RandomForestClassifier
File "/xxx/software/miniconda3/lib/python3.9/site-packages/sklearn/init.py", line 84, in
from .utils._show_versions import show_versions
File "/xxx/software/miniconda3/lib/python3.9/site-packages/sklearn/utils/_show_versions.py", line 13, in
from ._openmp_helpers import _openmp_parallelism_enabled
ImportError: dlopen: cannot load any more object with static TLS
I attempted to reinstall svtk again, and here's what I did, but I encountered a similar error. Could you help me resolve it?
git clone https://github.com/talkowski-lab/svtk.git
cd svtk/
python setup.py
Compiling svtk/utils/helpers.pyx because it changed.
[1/1] Cythonizing svtk/utils/helpers.pyx
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: setup.py --help [cmd1 cmd2 ...]
or: setup.py --help-commands
or: setup.py cmd --help
error: no commands supplied
find -name svtk
./scripts/svtk
./svtk
./scripts/svtk --help
Traceback (most recent call last):
File "/xxx/software/svtk/./scripts/svtk", line 42, in
import svtk.cli as cli
File "/xxx/software/miniconda3/lib/python3.9/site-packages/svtk/cli/init.py", line 12, in
from .adjudicate import main as adjudicate
File "/xxx/software/miniconda3/lib/python3.9/site-packages/svtk/cli/adjudicate.py", line 14, in
from svtk.adjudicate import adjudicate_SV
File "/xxx/software/miniconda3/lib/python3.9/site-packages/svtk/adjudicate/init.py", line 1, in
from .random_forest import RandomForest, rf_classify
File "/xxx/software/miniconda3/lib/python3.9/site-packages/svtk/adjudicate/random_forest.py", line 14, in
from sklearn.ensemble import RandomForestClassifier
File "/xxx/software/miniconda3/lib/python3.9/site-packages/sklearn/init.py", line 84, in
from .utils._show_versions import show_versions
File "/xxx/software/miniconda3/lib/python3.9/site-packages/sklearn/utils/_show_versions.py", line 13, in
from ._openmp_helpers import _openmp_parallelism_enabled
ImportError: dlopen: cannot load any more object with static TLS
I tried to replace the Python 3.9 under Miniconda with another Python 3.6 under Anaconda, but I was still unable to use svtk, and it gave me the following error. I think it might be because I didn't install the svtk.cli module. Can you help me with this?
Traceback (most recent call last):
File "/xxx/software/miniconda3/bin/svtk", line 45, in
import svtk.cli as cli
ModuleNotFoundError: No module named 'svtk.cli'
pip install svtk.cli
Collecting svtk.cli
Could not find a version that satisfies the requirement svtk.cli (from versions: )
No matching distribution found for svtk.cli
You are using pip version 10.0.1, however version 21.3.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
Bug Report
File "/xxx/software/miniconda3/lib/python3.9/site-packages/sklearn/utils/_show_versions.py", line 13, in
from ._openmp_helpers import _openmp_parallelism_enabled
ImportError: dlopen: cannot load any more object with static TLS
Traceback (most recent call last):
File "/xxx/software/miniconda3/bin/svtk", line 45, in
import svtk.cli as cli
ModuleNotFoundError: No module named 'svtk.cli'
Affected version(s)
$ python --version
Python 3.9.7

Hi @jingydz, the original repo https://github.com/talkowski-lab/svtk.git is no longer being maintained. You should use install the svtk module from gatk-sv instead and install:
git clone https://github.com/broadinstitute/gatk-sv.git
cd src/svtk/
pipe install -e .


[charts/redis-ha] Add readiness probe to chart

Signed-off-by: Michael Michalski<EMAIL_ADDRESS>What this PR does / why we need it:
Currently the chart supports liveness probes but not readiness probes.
The self hosted Kubernetes of our company dictates that these are set.
Which issue this PR fixes
Special notes for your reviewer:
Is there an interest in creating a dedicated readiness probe or is it good enough to reuse the /health/redis_liveness.sh
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
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

@DandyDeveloper would also be keen to see this merged, could you have a look?

On it first thing in my morning. Sorry its my birthday today!

Happy birthday! 🎂

I've requested a PR on your PR (https://github.com/michael-michalski/charts/pull/1 @michael-michalski ).
With recent changes to redis_liveness.sh, the POD (wrongly) will be marked as ready when a sync is in progress.
I've added an new redis_readiness.sh.

I also recommend setting readiness.successThreshold to higher values (e.g. 5), because when redis starts, first readiness checks wrongly reports ready state. But after some time (e.g. 1 minute), sentinel may trigger a resync and changes to an unavailable state.
This may cause total data loss in a rolling update process.


proper setting for searching with fuse?

New to docus and realize that there're enough info for how to configure the search functionality.
In the documentation, only one line mentioned useFuse options, but I cannot find any example config files/settings in the app.config.ts.
How to activate the search / deactivate it?
My website have the button, but nothing shows up if I type any keyword.

(fuse.js is manually added to the dependencies)

I'm in the same boat. Search used to work but somehow it doesn't anymore (I'm on 1.14.6) as I'm getting the "No results found. Try another query" message

I also faced in this issue.
It was resolved by pinning @nuxt/content version to 2.7.2.

Hello, could someone provide a minimal reproduction, so that, I'll take a look?

Hello, could someone provide a minimal reproduction, so that, I'll take a look?

Even in the clean starter template search doesn't work.
Created one (with pnpm), you can check it:
https://github.com/evermake/docus-tmp-repo

In fact, it does not work on the playground https://stackblitz.com/github/nuxt-themes/docus-docs-starter/tree/main/?file=content%2F1.guide%2F0.index.md

I would love to debug this an create a PR to solve this issue but I'm unable to start this project on WSL.
More than 20 seconds to start Vite and Nitro never start. 😅

The _empty field in the parsed content is returning undefined. Probably, something in nuxt content.
The easy fix is to check for it like below and the search will work properly:
     // _empty can return undefined, so we need to check for that
     (doc?._empty === undefined || doc?._empty === false)

I experience exactly the same issue.
I have the documentation built using docus and the search does not work.
I am using Nuxt Content 2.8.2, docus 1.12.3.
The code can be checked here https://github.com/Baroshem/nuxt-security/tree/chore/1.0.0-rc.1


Changing theme does nothing on Cygwin (followed usage instructions)

I've followed the usage instructions to install Antigen on Cygwin (Windows). My .zshrc is exactly like the one on the README and everything initializes and starts just fine the first time I launch my shell after installing Antigen. So far so good. This is my ZSH version:

zsh --version
zsh 5.3 (x86_64-unknown-cygwin)

But when I try to edit the .zshrc file to change a simple thing like the theme, say change antigen theme robbyrussell to antigen theme revised, the theme doesn't change. If I delete the ~/.antigen folder and restart my shell, the new theme will be loaded. If I try to change it again to something else, it won't load.
In other words, it only changes if I delete the ~/.antigen folder. What could possibly be the issue?

It's because Antigen caches its bundles and themes in $ANTIGEN_CACHE on the first start up.
You have to either:

antigen reset

ANTIGEN_CHECK_FILES=(/path/to/.zshrc)

Take a look at the configuration section.
Remember you can interactively switch between themes with antigen theme command.

That's it. Thank you :)


Adjust menu shadows

This commit makes menu and tooltip shadows slightly fainter and smaller.
It also introduces the option for shadows' top edges to be slightly inset relative to their parent window. This causes menus and tooltips to cast less of a shadow over the widget which opened them.
Closes https://github.com/emilk/egui/issues/3046.

Before/after comparison (animated):

bump


Feature request: Open ranger on a vertical split

Hi there,
I was wondering whether it is possible to set ranger to open in a vertical split (from the right) rather than the default of opening in a horizontal split from the top. If not, would it be possible to add to the extension?
Cheers!

Hi,
This should be possible. I have an unreleased branch where I added an additional panel which slides out from the left side of the screen, but I think being able to configure it to open at any edge would be more useful.
I'll have a look at this when I get the chance.


FInstancedStruct doesn't work

Hi,
Thanks for the awesome plugin.
The new FInstancedStruct from the StructUtils plugin does not work when converting an object containing Instanced Structs to JSON. I assume that's expected but as FInstancedStruct is supposed to support serialisation, it'd be nice if it could work with the Object to JSON node.

Working on support for instanced structs rn.
Do you have a discord? It would be easier for me to get in contact with you that way. My discord username is: epicgameguy

Working on support for instanced structs rn.
Do you have a discord? It would be easier for me to get in contact with you that way. My discord username is: epicgameguy

Glad to hear it, struct inheritance opens a lot of doors for JSON formatting too. I've sent you a friend request - username is the same as here.

Fixed by version 1.1


Update README.md

Coverage remained the same at 86.325% when pulling 99997a1528360bccabbbc1afc0b95d47f30ef73f on update-readme into ac54e4871a7934890aeb776e94e7f38a7ec9b501 on develop.


Output of P2H should not go to nearly zero when sector grows

I made a scenario based on II3050 - European.
With some sectors set to zero and no other flex apart from P2H.
I am confused by the behaviour of P2H when the chemicals sector grows.
Three examples:

Sector size of Chemicals 200%, 100% H2 heaters and P2H at maximum --> the behaviour of P2H is as I expect it to be. link
Sector size of Chemicals 210%, 100% H2 heaters and P2H at maximum --> the behaviour of P2H is not as I expect it to be. P2H boilers are hardly operated. link
Sector size of Chemicals 210%, 80% H2 heaters, 20% electric boilers and P2H at maximum --> the behaviour of P2H is not as I expect it to be. P2H boilers are hardly operated. link

Screenshots

2.

3.

Assigning @marliekeverweij please re-assign / relable when appropriate

Interesting issue, good catch.
There seems to be a tipping point between Size = 208% and Size = 209%.
@antw do you have time to look into this issue?


Simple call to api.

And add an ignore file with secret properties for api keys.

This pull request was replaced by another with the observations and improvements that have been noticed by the team so far.


Fix pre-commit and linter versions to avoid surprise

We have a lot of headache with linting this project. Although https://github.com/OpenDevin/OpenDevin/pull/1071 made a first attempt to unify the behavior of local linting and CI linting, it still has the following limitations:

The version of pre-commit run locally (pyproject.toml) and that in CI config (lint.yml) are not fixed, and can diverge.
The versions of linters (ruff and mypy) are not fixed, which means new releases of these libraries can bring us surprises.

To avoid surprising results on GitHub Actions, e.g. a new release of pre-commit starts to reject all PRs, fix it to the latest version, 3.7.0, in both pyproject.toml and lint.yml. This PR also fixes ruff and mypy versions in pyproject.toml since we very likely don't really need latest upgrades from linters.

@neubig Thanks for the quick review. Looks like I need you to sign-off once more since I pushed an one-liner after your approval.


Added the official Xamarin provided Mono packages

See http://www.mono-project.com/docs/getting-started/install/linux/#debian-ubuntu-and-derivatives. Required for https://github.com/OpenRA/OpenRA/pull/8933.

Since Travis runs on Precise LTS, wouldn't it also require the libtiff-compat repo?
http://www.mono-project.com/docs/getting-started/install/linux/#older-ubuntu-releases-ubuntu-1210-and-1204

I don't know. Gave up here.


HDDS-3850. Update the admin document to let user know how to show the status of all rules.

What changes were proposed in this pull request?
Update admin document, show the --verbose option can show the status of all rules.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3850
How was this patch tested?
No need.

@dineshchitlangia Please take a look at this document updated, with this update, user can know the --verbose option.

@dineshchitlangia Thank you for your suggestion. PTAL.

Thanks @maobaolong for the contribution.


DVC-6495: thread pool with event flushing

This adds event handling on top of #102.
To run the concurrent benchmark with events enabled:
go test -run=^$ -bench ^BenchmarkDVCClient_VariableConcurrent$ -benchEnableEvents ./

Current dependencies on/for this PR:

main

PR #103

PR #102

PR #109   👈

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #103

PR #102

PR #109   👈

This comment was auto-generated by Graphite.


Exception while using concurrent.futures.ProcessPoolExecutor and dask.multiprocessing, if tblib available

the root cause of this problem is this issue
Now tblib doesn't appear to be very actively maintained, so I thought I should raise the same issue here, since dask depends on it.
I put together a minimal docker file to show the problem:
Dockerfile.txt (you'll have to rename from 'Dockerfile.txt' to 'Dockerfile', github won't me let upload otherwise)
to build it: docker build -t map_bug .
to run it, without dask to show the expected output: docker run -it map_bug python3 test.py
output:
exception:  bar
 --- done ---

to run it, with task, to show the error: docker run -it map_bug python3 test.py dask
output:
exception:  bar
 --- done ---
Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "/usr/lib/python3.8/concurrent/futures/process.py", line 102, in _python_exit
    thread_wakeup.wakeup()
  File "/usr/lib/python3.8/concurrent/futures/process.py", line 90, in wakeup
    self._writer.send_bytes(b"")
  File "/usr/lib/python3.8/multiprocessing/connection.py", line 183, in send_bytes
    self._check_closed()
  File "/usr/lib/python3.8/multiprocessing/connection.py", line 136, in _check_closed
    raise OSError("handle is closed")
OSError: handle is closed

The problem (from dask point of view) is in dask.multiprocessing.py, line 102,
try:
    import tblib.pickling_support

    tblib.pickling_support.install()

    def _pack_traceback(tb):
        return tb

except ImportError:

    def _pack_traceback(tb):
        return "".join(traceback.format_tb(tb))

    def reraise(exc, tb):
        exc = remote_exception(exc, tb)
        raise exc

so if tblib is available, dask will import it, which then causes the problem with concurrent.futures.ProcessPoolExecutor.
The python code running inside the above docker is:
import sys
from concurrent.futures import ProcessPoolExecutor
if len(sys.argv) > 1 and sys.argv[1] == "dask":
    import dask.multiprocessing

def foo(x):
    if x > 2:
        raise RuntimeError("bar")

if __name__ == "__main__":
    with ProcessPoolExecutor(max_workers=2) as executor:

        try:
            list(
                executor.map(foo, range(50))
            )
        except Exception as ex:
            print("exception: ", ex)

    print(" --- done --- ")

Environment:

Dask version: 2.25.0
Python version: 3.8
Operating System: ubuntu 20.02
Install method (conda, pip, source): pip

(I uploaded the wrong Dockerfile earlier, now corrected)
It's not fixed, it reproduces on python 3.8.5 and 3.8.2. He was running on Fedora, so that could influence things. Will test further. I supplied him with a reproducing Dockerfile also.
Yeah our fix was to install 'dask[array]', which didn't include tblib. We could also have simply uninstalled it as you said. It's not really a problem with Dask per se, just thought you guys should be aware.

Just tried this today and am not seeing this issue. Going to go ahead and close. Thanks all 😄


Create a unified argparse for sema and its subcommands

Develop a main argparse in sema that can reuse the argparse setup of its subcommands.

Links to #2 , #3

@mpo-vliz , can this issue be closed sine there is a sema.commons.cli section?

No
commons.cli currently only had common support that different cli in sema can reuse sand benefit from
This issue is about creating an extra entry-level cli sema command that provides info and maybe access to the others.
Info meaning sema --help to list that check, get, harvest, ... subcommadns exist
Access mending it could actually be an active alias so sema get would be the equivalent of the current sema-get
So this is about discovery and convenience in all cli stuff in sema


Change "flow weighted" to "BOLD" in the modality drop down menu

Most users will not know what "flow weighted" is supposed to refer to.

This too has been mentioned before but opening an issue for book keeping.

Related to https://github.com/neurobagel/planning/issues/28


CC field is ignored

See this empty comment: https://gitlab.staging.haskell.org/ghc/ghc/issues/8809#note_77602
It is Ben adding himself to the CC list. This list is important as it means people who are not receiving all the emails receive updates to specific tickets.
CC events should correspond to the subscription API call I believe -
https://docs.gitlab.com/ee/api/issues.html#subscribe-to-an-issue
CC removal by the unsubscribe endpoint
https://docs.gitlab.com/ee/api/issues.html#unsubscribe-from-an-issue

One issue with this is that we need to block (or intercept) notifications during the import, otherwise gitlab will happily send them out as we continue importing status updates and such after adding a CC.

Yes, that is absolutely true. However, I believe this can be easily done in the GitLab configuration.

Seems to work, although I don't see any notes being inserted by gitlab. Not sure if that's supposed to happen, but I doubt anyone links to those comments anyway.

OK, so I verified that imported subscriptions reflect in the "notifications" slider when impersonating the subscriber. I take it that this means it works as intended, even if we're not seeing any additional UI cues.


Update the module to work with Bicep v0.4 assemblies

Is your feature request related to a problem? Please describe.
Bicep v0.4 will be released shortly. I have tested the module with new assemblies by running dotnet publish to build the DLLS, using the current state of the main branch to test with the latest changes. I get some issues using the assemblies since there are new arguments available for some of the methods.

Build-Bicep -Path C:\git\Azure\bicep\docs\examples\101\aad-domainservices\main.bicep
MethodInvocationException:
Line |
  16 |          $CompilationResults = $Compilation.GetAllDiagnosticsBySyntaxT …
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Exception calling "GetAllDiagnosticsBySyntaxTree" with "0" argument(s): "Could not load file or assembly 'Microsoft.Extensions.Configuration.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'. The system cannot find the file specified."
MethodInvocationException:
Line |
  53 |              $EmitResult = $Emitter.Emit($Stream)
     |              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Exception calling "Emit" with "1" argument(s): "Could not load file or assembly 'Microsoft.Extensions.Configuration.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'. The system cannot find the file specified."

ConvertTo-Bicep -Path C:\git\Azure\bicep\docs\examples\101\aad-domainservices\main.json
WARNING: Decompilation is a best-effort process, as there is no guaranteed mapping from ARM JSON to Bicep.
You may need to fix warnings and errors in the generated bicep file(s), or decompilation may fail entirely if an accurate conversion is not possible.
If you would like to report any issues or inaccurate conversions, please see https://github.com/Azure/bicep/issues.
MethodInvocationException:
Line |
  16 |          $CompilationResults = $Compilation.GetAllDiagnosticsBySyntaxT …
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Exception calling "GetAllDiagnosticsBySyntaxTree" with "0" argument(s): "Could not load file or assembly 'Microsoft.Extensions.Configuration.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'. The system cannot find the file specified."
MethodInvocationException:
Line |
  53 |              $EmitResult = $Emitter.Emit($Stream)
     |              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Exception calling "Emit" with "1" argument(s): "Could not load file or assembly 'Microsoft.Extensions.Configuration.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'. The system cannot find the file specified."

Describe the solution you'd like
The module needs to be update to work with the latest assemblies. Would be awesome if we could release an updated version of the module shortly after the Bicep 0.4 release.
Additional context
Add any other context or screenshots about the feature request here.

Closed with #167


formulas and matrices are shown in one long line in chrome - backslash problem

I have written some formulas and matrices in an .md file. They work OK on my desktop Editor. Look like this:

When I upload to the github, and open using chrome with MathJax extension, it shows all the formulas and matrices in a single line. Like this:

It looks like Backslashes are not properly interpreted. When I right-click on the formula on github page and click "Show Math as Tex Commands" I see that most of the Backslashes are lost. Like this:

How should I resolve the issue? I restarted the chrome and things but it didn't help. Also, I use Mac OSX.

I believe you have to double the backslashes to escape them: For each line break, instead of writing "\", write "\\" (four backslashes). See https://docs.mathjax.org/en/latest/tex.html#tex-and-latex-in-html-documents for more info


[Integration]: New domain request

Project name
NFT widget app
URL to be whitelisted
://.dcm-swiss.com/*
Code audit/review URL if available
No response
Project description
Integration of the casper NFTs into exsiting platform of nft distribution. Easy way to plugin nft into articles and running web3 campaigns.
Just an example https://imu-qa.dcm-swiss.com/share/7cf861dd-7d0a-4874-aa11-5eba2283b4cd
Project's phase
Testing
Who is the project owner?
Private person
Project owner's name
Sergii Bykov
What kind of Web3 functionality will you be offering in your product?
mint nft, transfer nft, contract calls etc
Contact details
<EMAIL_ADDRESS>Link to the code
Just an example of app https://imu-qa.dcm-swiss.com/share/7cf861dd-7d0a-4874-aa11-5eba2283b4cd
Is this a project by an enterprise partner of Casper Association or CasperLabs?
No
Grant/proposal number if this is a project funded by the DEVxDAO
No response

Waiting on Association approval. Tentative completion ETA - 21st Feb.

Hi @sergeybykov85 ,
The request is now complete. Refer Signer v1.4.25 release.

@devendran-m It works, thank you


make get_more_labels update the current_budget in the localinterface

In GitLab by @Schumann on May 1, 2020, 17:16

In GitLab by @Schumann on Jun 3, 2020, 15:24
moved to learn#16


Use to_bool on bool env vars

Coverage remained the same at 80.795% when pulling 49a4ad17b81ce4ea374a5ee828cd77ca53b9a0c7 on philipbjorge:patch-1 into 17a70bd51c123ca2be78f4f8ec7540044c0f1f60 on seatgeek:master.


Gencert supports generating cert signed by citadel.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: incfly
If they are not already assigned, you can assign the PR to them by writing /assign @incfly in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kyessenov FYI.

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

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
62be1ef60b45a10d5aa0a367d7f568b692784854
link
/test istio-presubmit

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: douglas-reid
If they are not already assigned, you can assign the PR to them by writing /assign @douglas-reid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: quanjielin
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: douglas-reid
If they are not already assigned, you can assign the PR to them by writing /assign @douglas-reid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: myidpt, quanjielin
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [myidpt]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/test istio-unit-tests

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
48f047c20f529944b36a6baae5fc59438ddadd95
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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


Box "Latest end time" on mobile phone too small

Latest versions of HA 2024.9.1 and EPEX Spot sensor.
Didn't use the setting for a longer time, I don't know since when the error accoured (perhaps HA 2024.9?)
Many Thx in advance.

May I ask again for a solution? Now in autumn /winter I need this nearly every day. Thx ☺

Hi, the issue still exists 😞

Hi, sorry for the delay - I'm too busy with other tasks.
This is a UI rendering problem which can't be solved in the EPEXSpotSensor integration (except I shorten the help text - but this is not what I want to do). Therefore, please reach out to the UI experts of Home Assistant.


[release-1.11] DestinationRule triggers lds for gateways

This is an automated cherry-pick of #34963

Regression due to CVE fix


Moderation Feature: View Mute Duration and Time Remaining

The title says it all. Infractions should list mute duration and time remaining (if applicable).
It would also be nice if users could check how long they have remaining on their mute, maybe through ClemBot's dashboard.

Covered by timeout addition in #539

tho adding that info to the infraction command could be cool


doc.output('bloburi') does not work after 2. sep 2017

I think there is a bug in your latest jspdf.js (2. sep 2017):
getBlob = function() {
return new Blob([getArrayBuffer()], {
type: "data:application/pdf;base64"
});
},
type should be "application/pdf".
This bug causes a blank page in a iframe and Save As dialog pops up.

Thanks. This apparently fixes Adobe Reader but shall change it back.

Hmm. Strange. Probably a security problem because now it is clear that it is base64 encoded data. probably before it was for the browser unreadable "garbage".

I'm using latest Chrome on Linux Mint.
Everything worked fine till I rebuild my jsPDF package with AutoPrint (latest jspdf.js, addImage, metrics & autoprint). Stopped working.
After half an hour scratching my head I had a look on the code and noticed that Type difference comparing to the previous version. Changed it back to "application/pdf" and now all works again.
There is another spot in the code using that "data:application/pdf;base64" too but I didn't change that:
output = SAFE(function(type, options) {         var datauri = ('' + type).substr(0, 6) === 'dataur' ?           'data:application/pdf;base64,' + btoa(buildDocument()) : 0;

I've reverted it for the moment, until we understand a little bit more about the issue


Don't Merge! PR real time test for webhook case

Docs Build status updates of commit 648669a:
:clock10: Pending: waiting for processors (21 builds ahead of you)
:warning: Docs Build is busy, currently there are 21 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 648669a:
:clock10: Full build: the average full build time is 6 min(s) 6 sec(s), this is based on the last 406 build(s) for this repository.

Docs Build status updates of commit 648669a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Grammer in the Tip section.

In the Tip section (green colored section titled Tip)
It starts "Check the out the steps in" I think there is an extra "the" and would make more sense if it was "Check out the steps in"

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

ID: 784a196f-3e0e-0c0a-0b2f-af2105a774d9
Version Independent ID: 08300f82-3b1d-b96d-bf10-e401ac6ef2c0
Content: Allow members to send as or send on behalf of a Group
Content Source: Office365-Admin/create-groups/allow-members-to-send-as-or-send-on-behalf-of-group.md
Service: o365-administration
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@AndyConlisk  Thank you for pointing this out. We appreciate you taking the time to help us improve the content.  The change will be made and once merged, this issue will be closed. Thanks again.

@AndreaBarr  please close


Implemented skin select.

Thanks for your contribution! I've also added --list-skins argument to view available skins, which can be used with --skin select argument. That also means the important notice about skins is no longer necessary.


Authentication with RID 500 admin

The default client.Authenticate(cred) works fine for all domain accounts, but if you step through the authenticate process in the Authenticate() Task, you will get KDC KDC_ERR_PREAUTH_FAILED: Pre-authentication information was invalid for RID 500 built-in domain\administrator account. All the other domain accounts work perfectly. Could you please help to check if there's anything wrong?
var client = new Kerberos.NET.Client.KerberosClient();
var kerbCred = new Kerberos.NET.Credentials.KerberosPasswordCredential(user, pass, domain);
var auo = AuthenticationOptions.RepPartCompatible |
    AuthenticationOptions.IncludePacRequest |
    AuthenticationOptions.RenewableOk |
    AuthenticationOptions.Canonicalize |
    AuthenticationOptions.Renewable |
    AuthenticationOptions.Forwardable |
    AuthenticationOptions.PreAuthenticate;
var asReqMessage = KrbAsReq.CreateAsReq(kerbCred, auo);
var asReq = asReqMessage.EncodeApplication();
var test = await transport.SendMessage<KrbAsRep>(kerbCred.Domain, asReq);

Thanks!

Are you saying all accounts work for Authenticate EXCEPT the DA 500? Or all accounts except DA 500 work when you call the transport directly?
This works for me:
var client = new KerberosClient();

var credential = new KerberosPasswordCredential("administrator", "P@ssw0rd!", "corp.identityintervention.com");

await client.Authenticate(credential);

My guess is the salt that's getting generated by KerberosPasswordCredential is wrong. The Client itself handles this logic by sending a request for pre-auth without any PA timestamp where the KDC returns an error message including the appropriate salts. Since you're using the transport directly this logic doesn't exist and the credential itself must guess at what salt to use. In this case it's probably guessing wrong. Not much can be done about it, you either guess right or you don't. You can supplement that by appending the salts to the credential with kerbCred.Salts though at that point I would recommend just using the client.
The built-in domain admin account is special in this regard. It gets its' salt information prior to the machine being converted into a domain, so the salt algorithm it uses doesn't match the standard implementation (chicken and egg problem -- the domain doesn't exist so you can't use it to provide data for the salt). The salt for this user is OriginalDCHostName|UserName. In my test domain that is WIN-RFTG0BD1S70Administrator .

Thanks for the answer, but from what I have observed, the DA 500 user has the same salt structure with other domain users. (The krb error message returns DOMAIN.LOCALAdministrator). Also if you issue the runas command to run as the DA 500 user, you will see the same salt that returned by the DC.

The password has probably been changed. That salt only applies to the first password created.
In either case it's the same basic issue. The key is attempting to use is the wrong salt. That's why the client always tries to get the salt first. This is not guaranteed to work without knowing for a fact what the salt is.

Should it work in such salt structure (DOMAINUser) for 500 admin account? I reckon all user should be able to authenticate successfully if KDC returns the same salt structure.

Assuming the actual salt it guesses matches then it should work. It just needs salt and password to match for the crypto to work.
If you can, take a peek at the internals of the KerberosKey.Salt property for the user. This is where it's attempting to guess the salt. The stack would look something like this:
Kerberos.NET.dll!Kerberos.NET.Crypto.AES.AesSalts.GenerateSalt(Kerberos.NET.Crypto.KerberosKey key)
Kerberos.NET.dll!Kerberos.NET.Crypto.AES.AESTransformer.String2Key(Kerberos.NET.Crypto.KerberosKey key)
Kerberos.NET.dll!Kerberos.NET.Crypto.KerberosKey.GetKey(Kerberos.NET.Crypto.KerberosCryptoTransformer transformer)
Kerberos.NET.dll!Kerberos.NET.Crypto.KerberosKey.GetOrDeriveKey.AnonymousMethod__0(string str)
System.Collections.Concurrent.dll!System.Collections.Concurrent.ConcurrentDictionary<string, System.ReadOnlyMemory<byte>>.GetOrAdd(string key, System.Func<string, System.ReadOnlyMemory<byte>> valueFactory)
Kerberos.NET.dll!Kerberos.NET.Crypto.KerberosKey.GetOrDeriveKey(Kerberos.NET.Crypto.KerberosCryptoTransformer transformer, string cacheKey, System.Func<System.ReadOnlyMemory<byte>, System.ReadOnlyMemory<byte>> dk)
Kerberos.NET.dll!Kerberos.NET.Crypto.AES.AESTransformer.GetOrDeriveKey(Kerberos.NET.Crypto.KerberosKey kerberosKey, Kerberos.NET.Crypto.KeyUsage usage)
Kerberos.NET.dll!Kerberos.NET.Crypto.AES.AESTransformer.Encrypt(System.ReadOnlyMemory<byte> data, Kerberos.NET.Crypto.KerberosKey kerberosKey, Kerberos.NET.Crypto.KeyUsage usage)
Kerberos.NET.dll!Kerberos.NET.Entities.KrbEncryptedData.Encrypt(System.ReadOnlyMemory<byte> data, Kerberos.NET.Crypto.KerberosKey key, Kerberos.NET.Crypto.KeyUsage usage)
Kerberos.NET.dll!Kerberos.NET.Credentials.KerberosCredential.TransformKdcReq(Kerberos.NET.Entities.KrbKdcReq req)
Kerberos.NET.dll!Kerberos.NET.Entities.KrbAsReq.CreateAsReq(Kerberos.NET.Credentials.KerberosCredential credential, Kerberos.NET.Client.AuthenticationOptions options)
ConsoleApp1.dll!ConsoleApp1.Program.Main(string[] args)

The only variable in this case is the salt itself.
The other thing worth checking I suppose is that it's not somehow attempting to use RC4. 500 admin is in protected users which blocks RC4 use.

Good point.  Thx a lot. Will check and get back to you soon

I still don't think this unexpected behavior is related to the salt... I checked the salt for RID 500 administrator is exactly what the KDC returns back to the client... (The salt I used for admin matches the what the KDC needs). Also for protect user groups, I was able to authenticate as users in this group. It has nothing to do with the encryption algorithm.

But it definitely works using the client authenticate method?
Let me see if I can repro the scenario better where salt matches.

From: dev2null<EMAIL_ADDRESS>Sent: Thursday, May 21, 2020 12:02:25 AM
To: dotnet/Kerberos.NET<EMAIL_ADDRESS>Cc: Steve Syfuhs<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [dotnet/Kerberos.NET] Authentication with RID 500 admin (#164)
I still don't think this unexpected behavior is related to the salt... I checked the salt for RID 500 administrator is exactly what the KDC returns back to the client... (The salt I used for admin matches the what the KDC needs). Also for protect user groups, I was able to authenticate as users in this group. It has nothing to do with the encryption algorithm.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/Kerberos.NET/issues/164#issuecomment-631921488, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAJHTYPQYLTXI2UVVBUICKLRSTGYDANCNFSM4NEUPGGA.

Do you have any updates? I still couldn't figure out what's wrong...
BTW, I want to ask what I should do with the license cuz I made a small tool based on your library and want to make it public, do I need to use the same license? and what else I should do? Thank you!

Sorry, just got back to it. The exact issue as I have been able to reproduce it is that it's still the salt. When you change the password the salt gets updated accordingly, so the two do match. However, the difference is that the 500 account uses a capital 'A' in the sAMAccountName which changes the salt, which causes the key to derive to something else. If you capitalize the username "Administrator" or<EMAIL_ADDRESS>the salt is generated correctly (for me). There are two solutions to make this work:

Make the user enter the username verbatim to what's recorded in AD
Request a pre-auth error and extract the salt from that

Option 2 is documented as the correct option as per protocol spec.
For licensing, you can redistribute the code however you want, you just need to put a notice somewhere that links to https://github.com/dotnet/Kerberos.NET/blob/develop/LICENSE saying the Kerberos.NET library code itself is copyright. Any derivative work that isn't part of the library is your copyright as you see fit. For instance this library has a NOTICES file that does just that.

Thank you very much! Problem solved!
And really appreciate the licensing guide.


Can not dev with docker

Hi,
When I try to build with docker, it fails when start installing pip requirements.
Here is my docker version:

$ docker version
Client:
Version:      1.12.3
API version:  1.24
Go version:   go1.6.3
Git commit:   6b644ec
Built:        Wed Oct 26 22:01:48 2016
OS/Arch:      linux/amd64

Server:
Version:      1.12.3
API version:  1.24
Go version:   go1.6.3
Git commit:   6b644ec
Built:        Wed Oct 26 22:01:48 2016
OS/Arch:      linux/amd64

Here is my docker-composer version:

$ docker-compose version
docker-compose version 1.9.0, build 2585387
docker-py version: 1.10.6
CPython version: 2.7.12
OpenSSL version: OpenSSL 1.0.2g  1 Mar 2016

Here is my docker info:

$ docker info
Containers: 4
Running: 0
Paused: 0
Stopped: 4
Images: 11
Server Version: 1.12.3
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 36
Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: bridge null host overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: apparmor seccomp
Kernel Version: 4.4.0-51-generic
Operating System: Ubuntu 16.04.1 LTS
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 7.688 GiB
Name: Latitude-E7450
ID: JJK2:EOEC:JXB2:SLFK:5UMN:H43J:Q7PY:3OST:4OUZ:4XSL:C5ZJ:4ZBX
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Insecure Registries:
<IP_ADDRESS>/8

And this is where it fails when I run,
docker-compose -f dev.yml build

---> Running in 0eac9a6fadb7
Collecting wheel==0.29.0 (from -r /requirements/base.txt (line 5))
Retrying (Retry(total=4, connect=None, read=None, redirect=None)) after connection broken by 'NewConnectionError('<pip._vendor.requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f7ed70320b8>: Failed to establish a new connection: [Errno -2] Name or service not known',)': /simple/wheel/
Retrying (Retry(total=3, connect=None, read=None, redirect=None)) after connection broken by 'NewConnectionError('<pip._vendor.requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f7ed7b28a90>: Failed to establish a new connection: [Errno -2] Name or service not known',)': /simple/wheel/
Retrying (Retry(total=2, connect=None, read=None, redirect=None)) after connection broken by 'NewConnectionError('<pip._vendor.requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f7ed7b28048>: Failed to establish a new connection: [Errno -2] Name or service not known',)': /simple/wheel/
Retrying (Retry(total=1, connect=None, read=None, redirect=None)) after connection broken by 'NewConnectionError('<pip._vendor.requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f7ed7b28c50>: Failed to establish a new connection: [Errno -2] Name or service not known',)': /simple/wheel/
Retrying (Retry(total=0, connect=None, read=None, redirect=None)) after connection broken by 'NewConnectionError('<pip._vendor.requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f7ed7b40898>: Failed to establish a new connection: [Errno -2] Name or service not known',)': /simple/wheel/
Could not find a version that satisfies the requirement wheel==0.29.0 (from -r /requirements/base.txt (line 5)) (from versions: )
No matching distribution found for wheel==0.29.0 (from -r /requirements/base.txt (line 5))
ERROR: Service 'django' failed to build: The command '/bin/sh -c pip install -r /requirements/local.txt' returned a non-zero code: 1

Does it mean, it cannot connect to outside world from inside the docker?
Hope I have described it all?

I don't know what causes it. But I restarted everything and it seems working now.
Really enjoying your book.

If it's working ok, should you close this issue?

@mrahma01 restarting docker just fixed this for me. thanks!


Requests > 2.16.1 breaks urllib3's Retry functionality

Hi,
We had code that used the vendorised urllib3 Retry functionality like so:
from requests.packages.urllib3.util import Retry

session = requests.Session()
retries = Retry(
    total=2, method_whitelist=['GET', 'POST', 'PATCH'],
    status_forcelist=[500, 502, 503, 504])
retryable_adapter = HTTPAdapter(max_retries=retries)
session.mount('http://', retryable_adapter)
session.mount('https://', retryable_adapter)

When we were running requests 2.13.0 this worked fine. However, today we upgraded to 2.16.5 and found that while the above code does not error, it now silently does nothing.
This  is because:
HTTPAdapter.__init__ calls self.max_retries = Retry.from_int(max_retries), passing my Retry instance into Retry.from_int.  Retry.from_int has this snippet of code:
if isinstance(retries, Retry):
    return retries

In earlier versions of requests this returned my Retry instance unchanged. However now the isinstance check fails because:
>>> type(retries)
<class 'requests.packages.urllib3.util.retry.Retry'>
>>> Retry
<class 'urllib3.util.retry.Retry'>

I note that in 2.16.1 the release notes mentioned this exact problem, and provided a workaround (that works!). However, the release notes for 2.16.2 say that the workaround is no longer necessary.
I'm happy with the workaround documented in the 2.16.1 release notes, but wanted to file this bug so you're aware that the advice in the 2.16.2 release notes is misleading, and leads to subtle bugs.
System Information
$ python -m requests.help

{
  "chardet": {
    "version": "3.0.3"
  },
  "cryptography": {
    "version": ""
  },
  "implementation": {
    "name": "CPython",
    "version": "3.4.3"
  },
  "platform": {
    "release": "4.10.0-21-generic",
    "system": "Linux"
  },
  "pyOpenSSL": {
    "openssl_version": "",
    "version": null
  },
  "requests": {
    "version": "2.16.5"
  },
  "system_ssl": {
    "version": "1000106f"
  },
  "urllib3": {
    "version": "1.21.1"
  },
  "using_pyopenssl": false
}

Thanks for this report! I think this is for @kennethreitz to deal with. 😁

Can you confirm for me that you don't have any older Requests *.pyc files in your environment? I can't reproduce this with the current master branch:
>>> import requests
>>> import urllib3
>>> x = urllib3.util.retry.Retry.from_int(5)
>>> isinstance(x, urllib3.util.retry.Retry)
True
>>> isinstance(x, requests.packages.urllib3.util.retry.Retry)
True
>>> y = requests.packages.urllib3.util.retry.Retry.from_int(5)
>>> isinstance(y, urllib3.util.Retry)
True
>>> type(x)
<class 'urllib3.util.retry.Retry'>
>>> type(y)
<class 'urllib3.util.retry.Retry'>
>>> requests.__version__
'2.16.5'

@thomir congratulations for being the first person to use $ python -m requests.help!

Can you try v2.17.1?

I've created a minimal script that reproduces the problem:
https://gist.github.com/thomir/f032c0fd56dbbad258b9ca99a2b15740
No doubt you have simpler methods than this, but I think it demonstrates the issue.
I'll try running 2.17.1 now.

@kennethreitz 2.17.1 does indeed work for me. Thanks!

\o/
Sorry for the issue!


Version 2.0.0

Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Whenever possible python packages should use pip. See https://conda-forge.org/docs/meta.html#use-pip

@conda-forge-admin, please rerender

@mimischi was this branch uploaded to this repo or from a fork?

I pushed it directly to this repository. Should one always do it from a fork for conda?

Any push to this repository triggers the CI builds including publishing a
repo. This is why the PR template makes you check you used a fork.
On Tue, Oct 30, 2018 at 9:32 PM Michael Gecht<EMAIL_ADDRESS>wrote:

I pushed it directly to this repository. Should one always do it from a
fork for conda?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/conda-forge/mdbenchmark-feedstock/pull/10#issuecomment-434458823,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEGnVt-4drON5sJ7aza9xO6pAa7lJbGdks5uqLdwgaJpZM4YBe3z
.

I see. Was wondering what that was about. Will make sure to do it
differently next time :)
Am Di., 30. Okt. 2018, 21:55 hat Max Linke<EMAIL_ADDRESS>geschrieben:

Any push to this repository triggers the CI builds including publishing a
repo. This is why the PR template makes you check you used a fork.
On Tue, Oct 30, 2018 at 9:32 PM Michael Gecht<EMAIL_ADDRESS>wrote:

I pushed it directly to this repository. Should one always do it from a
fork for conda?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/conda-forge/mdbenchmark-feedstock/pull/10#issuecomment-434458823
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AEGnVt-4drON5sJ7aza9xO6pAa7lJbGdks5uqLdwgaJpZM4YBe3z
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/conda-forge/mdbenchmark-feedstock/pull/10#issuecomment-434464684,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAQvEXdHTsKaPPObSC4Rbd0W_OTGDc7ks5uqLzdgaJpZM4YBe3z
.


Error response body is not parsed when API returns HTTP 400

As in my question at https://stackoverflow.com/questions/57272980/response-body-not-parsed-when-api-returns-http-400 the error response does not seem to get parsed when you get a 400

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

ID: a8601a9f-ca1e-49e0-bdf8-383431829997
Version Independent ID: 488d4db7-a2de-b47e-5c1c-a28a5f8e046f
Content: Xrm.WebApi (Client API reference) in model-driven apps - PowerApps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/reference/xrm-webapi.md
Service: powerapps
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

Any updates on this one? I would like to present the enduser the useable error message which you can see in the response.
Error function on XRM.WebApi. Returns an empty string as message.

@nicolausdewitt @EkoZuluDev Thanks for the feedback. We have fixed the issue and pushed it live. Closing the issue. If you need further help, please reopen the issue or create a new issue.


Next slide iframe is borked.

@faern I reproduce this. can you give me some more details?

@reimertz I don't know exactly how I can help. The next slide is simply white in presentation mode. I don't get anything in the console in the browser :(

Even if you change back and forth?
On Thu, 12 Jan 2017 at 10:21, Linus Färnstrand<EMAIL_ADDRESS>wrote:

@reimertz https://github.com/reimertz I don't know exactly how I can
help. The next slide is simply white in presentation mode. I don't get
anything in the console in the browser :(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reimertz/lagom/issues/20#issuecomment-272113685, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAmKh_FgZhhh0y5P1d4j8tMi6oCNBsrlks5rRfB8gaJpZM4Lg5XY
.

Yup. Did not see anything in my console.

Should note that I run Firefox 45.6

yep, confirmed borked in Firefox. thanks for the clarification.


zilencer: Fix server/register failing to check if server is deactivated.

A deactivated server shouldn't be updated like that - we want to return an error.

Merged, thanks @mateuszmandera!


@hci.h use of magic-number in bluetooth addr struct (Missing define in @bluetooth.h)

In bluetooth/hci.h the bluetooth address type is declared as
typedef struct {
	u8_t  val[6];
} bt_addr_t;

Instead of using magic number '6' this should be declared as a define in bluetooth.h:
#define BT_ADDR_LEN (6)

I'm not convinced it's worth it here. A Bluetooth address is defined to be six octets long, and that's what the struct definition demonstrates. There really isn't any "magic" about the number the way it's used here. Also, a define like that would not be used anywhere else (instead sizeof(bt_addr_t) should be used).

I think we can discuss the definition and use of Magic Numbers for a long time :)
However this might not be used anywhere else in HCI or Link Layer, but it would be a nice define to have available in the Application layer for different purposes (eg. copy or compare) of iterating over the BT address.
It makes for more readable code than using sizeof multiple places.

@TTJO there are already bt_addr_copy and bt_addr_cmp APIs, if you were looking for something like that. Same for the bt_addr_le type.

@TTJO I'd also like to add that one of the reasons these are typedefs instead of using them just as any other structs is specifically to highlight the fact that the internals is something the user should never (or extremely rarely) have to touch or care about.

@TTJO any feedback on my latest comments? I'd like to know whether I should close this issue or if there's still something unresolved. Thanks.

(added some maintainers as reviewers in case they want to voice their thoughts on this)

I would have to agree with @jhedberg if we do have a struct representing a bluetooth address the application should use sizeof that struct instead of a define, anyway sizeof will be evaluated at build time so in practice there is no real difference to a define except that sizeof would actually be more precise if the address ever change e.g. due to a bug.

I too agree with @jhedberg, we keep sizeof(bt_addr_t).

Closing, since there seems to be consensus that the current approach is ok


Move to Pandas dataframes as interface

Problem Statement

Currently, the Fondant component interface is based on a Dask Dataframe:
import dask.dataframe as dd

from fondant.component import TransformComponent

class MyComponent(TransformComponent):

    def transform(dataframe: dd.Dataframe, **kwargs): -> dd.Dataframe
        # User code goes here

Which means that the user needs to work with a Dask Dataframe, which raises two challenges:

Dask requires quite a big learning curve to use efficiently.
It's hard to let Fondant completely handle Dask optimization such as indexing and partitioning as this might interfere with user Dask code

Proposed Approach

Instead, we can base our interface on a Pandas dataframe:
import pandas as pd

from fondant.component import TransformComponent

class MyComponent(TransformComponent):

    def transform(dataframe: pd.Dataframe, **kwargs): -> pd.Dataframe
        # User code goes here

Which lets the user write code handling an in-memory Pandas dataframe, and moves all Dask specific optimization to Fondant.
Since a Dask dataframe is composed of many smaller Pandas dataframes (partitions) split along the index, this implementation should align nicely with the internal Dask design. It should also scale to distributed computing since the unit of distribution is a partition.
The Fondant component class can leverage map_partitions in the _process_dataset() method to achieve this.
Limitations

The map_partitions method assumes that the index and divisions are unchanged, and otherwise requires resetting the divisions on the result, which can be expensive if they need to be recalculated.
I propose to check if the returned index is identical to the original index. If it is, this assumption is valid. If it is not, we call clear_divisions() and defer division calculation to when it is needed.
If we clear the divisions, we should also check that additionalSubsets and additionalFields are set correctly. Since the index is changed, additionalSubsets should be false, and additionalFields should be false for any subsets part of the input subsets.

The user only gets access to one partition at a time, inhibiting any processing spanning across partitions (eg. global operations such as mean calculation). Enabling this kind of operation is a challenge for most data processing frameworks, not the focus of Fondant, and can be approximated by running on each partition instead.
By implementing this as a PandasTransformComponent subclassing the TransformComponent, users can still use the TransformComponent to work with Dask directly.

Implementation Steps/Tasks

[ ] Implement PandasTransformComponent
[ ] Update reusable and example components

Testing

We can test this with unit tests.
Documentation

We should document the PandasTransformComponent as the default way of building components with Fondant. Using the base TransformComponent directly should be explained as advanced usage.

Seems like we need to tackle #70 before implementing this feature

This isn't really a dependency as that issue is about the Fondant manifest index, while this is about the index of the pandas dataframe. It's trivial to check if 2 pandas Index objects are identical.

Looks good. I'll like the approach of just doing this while being very aware of the limitations which we can tackle later.
What about the LoadComponent ?

I propose to check if the returned index is identical to the original index. If it is, this assumption is valid. If it is not, we call clear_divisions() and defer division calculation to when it is needed.
If we clear the divisions, we should also check that additionalSubsets and additionalFields are set correctly. Since the index is changed, additionalSubsets should be false, and additionalFields should be false for any subsets part of the input subsets.

This actually won't work, as we need to decide if we call clear_divisions at graph building time, while the index is only available at computation time. We need to switch this order, and look at the additionalSubsets and additionalFields to decide if the divisions should be cleared.
This raises the question which default we want for additionalSubsets and additionalFields. Currently this is true, which leads to a broken component if the index is changed and these fields are not set correctly. If we would change the default to false, not setting these fields will lead to a less performant but working component.
Not sure which is better. Being more robust or failing when performance is not optimized.


Proposal: completely switch to py.test

this tools has been released recently: https://github.com/dropbox/unittest2pytest
if it does its job good, it shouldn't be such an effort to rewrite all tests as they are usable with pytest.
i'd be much in favor for it, alone for the heavily reduced char count of the tests.py. but certainly after an 1.0 release.

Interesting.

just two cents:

used it. it's realiable. it's not without straight forward effort anyway. but a good start.
pytest doesn't run on pyston yet.

Partially addressed by #259


LiftSystemDesigner: Yes this gets its own thread

Quick post of the issue, ill fill out this when i have time!  But General Lift Designer thread

LiftSystemDesigner is fixed and ready for multiplayer, I will add the commit shortly.


Could not resolve host

New installation.  Any host I try to proxy produces error: (6) Could not resolve host: www.someDomain.com

Link me to your proxy site

https://encroa.ch/proxy

Well, it magically began working :-)


Version/0.0.15

What is this PR for?

[x] Just updating content and/or documentation
[ ] This fixes issue #_____
[ ] Spot fix (no issue #)
[ ] Merging WIP feature #_____
[ ] Merging done feature #_____
[x] This is a merge from a version branch
[ ] Adding/Updating tests
[ ] This is a conflict resolution
[x] This is a branch cleanup
[ ] Other: ________________

I verify that...

[ ] I have logged my time in the commits
[ ] I have logged my time in this PR
[x] I have tagged all the relevant issues
[x] I am using VS Code for type checks and linting, or
[x] I have ran npm run test with no errors
[x] I have manually checked that this bug or feature is working

👋 @cblanquera
Thank you for raising your pull request. Please make sure you have followed our contributing guidelines. If you haven't done so in your commits, please reference the issue numbers as well as the time you spent on this in the comments below so we can track time. example fixed button #1001 3h


build(deps): bump github.com/cilium/ebpf from v0.9.1 to v0.10.0

Close: #8432

Files /home/runner/work/containerd/containerd/src/github.com/containerd/containerd/integration/client/go.mod and /tmp/tmp.NoduPFJetw/containerd/integration/client/go.mod differ
Files /home/runner/work/containerd/containerd/src/github.com/containerd/containerd/integration/client/go.sum and /tmp/tmp.NoduPFJetw/containerd/integration/client/go.sum differ

CI / Project Checks failed with error
  These files were modified:

   M go.sum

  Content of the diff:
  diff --git a/go.sum b/go.sum
  index 708bc793..9daff77c 100644

You need to run go mod tidy under both root dir and integration/client to fix it.

LGTM.
Integration tests randomly failed due to network issue.   May need maintainers to rerun the failed tests.   @samuelkarp
go<EMAIL_ADDRESS><EMAIL_ADDRESS>verifying module<EMAIL_ADDRESS>Get<EMAIL_ADDRESS>dial tcp <IP_ADDRESS>:443: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.


NWB: Handle LFP object (and other MultiContainerInterface objects)

NWB has several MultiContainerInterface types, which hold other types. For example the LFP object does not itself store data and time information, but is a Group that stores 1+ ElectricalSeries objects. These ElectricalSeries objects have arbitrary names.
e.g.
https://flatironinstitute.github.io/neurosift/#/nwb?url=https://dandiarchive.s3.amazonaws.com/blobs/c86/cdf/c86cdfba-e1af-45a7-8dfd-d243adc20ced

here:
https://github.com/flatironinstitute/neurosift/blob/0115f503ac6667c879f8d9b0b07b8136dba0bc51/gui/src/pages/NwbPage/NwbProcessingEcephysItemView/NwbLFPView.tsx#L18
you assume that there is one ElectricalSeries named "LFP", which will not be true in general. You instead need to fetch the names of all the ElectricalSeries within LFP and provide the option to view each of them.
Let's start with the LFP class, though this is a general pattern in NWB called a "MultiContainerInterface", where a neurodata type object can hold 0+ or 1+ objects of another neurodata type:

LFP holds ElectricalSeries
Fluoresence holds RoiResponseSeries
DFOverF holds RoiResponseSeries
BehavioralTimeSeries holds TimeSeries
EyeTracking holds SpatialSeries
Position holds SpatialSeries

@bendichter This has been fixed for the types you mentioned. They are hard-coded for now, but we'll want to handle this in a cleaner way in the future.
See: https://github.com/flatironinstitute/neurosift/blob/91e82cdc8bf927a9357de413653c1cef6c7556a3/gui/src/pages/NwbPage/NwbMainView/ProcessingGroupContentPanel.tsx#L63-L92
Closing, but we can open a similar issue in the future.

@magland do you really want to hard-code LFP here?
https://github.com/flatironinstitute/neurosift/blob/91e82cdc8bf927a9357de413653c1cef6c7556a3/gui/src/pages/NwbPage/NwbMainView/ProcessingGroupContentPanel.tsx#L77

Also, I don't see any difference for Fluorescence

That's because you misspelled it and I copy-pasted. :) Should be working now.
Thanks for catching my 'LFP' typo, that has now been fixed as well.


outlet template?

Hi there,
in case my homebridge-accessory is triggered by iOS HomeKit, I want to place a statement on the bashl. Therefor I used Fan_accessory.js and modified it for my needs. And I was successful, I am able to control the bash-command with siri. But one time, I said "turn off my devices" and Siri replied "There seems to be a problem with a fan" (The problem is ok, that was my fault). But I don't want Siri to call it "fan". I think thats why, because i copied the Fan_accessory.js. So I modified the file to change it to a Outlet. There is where the nightmare started: After that the HAP-NodeJS ist starting und parsing correctly the js-files, but in iOS I can't access the Node Bridge anymore. Undoing, re-Applying, rebooting and stuff wouldn't help. I was forced to deinstall and reinstall homebridge to get access to Node Bridge again.
The cause has to be in editing the Fan_accessory.js. I have only done to changes, marked for you two comments named LOOKHERE. It would be great if we could fix that! Thanks!
Cheers
Hellspawn0815
// HomeKit types required
var types = require("./types.js")
var exports = module.exports = {};
var exec = require('child_process').exec, child;
var execute = function(accessory,characteristic,value){ console.log("executed accessory: " + accessory + ", and characteristic: " + characteristic + ", with value: " +  value + "."); }

exports.accessory = {
  displayName: "Funksteckdose 3",
  username: "CC:22:3D:EE:5E:B3",
  pincode: "031-45-154",
  services: [{
    sType: types.ACCESSORY_INFORMATION_STYPE,
    characteristics: [{
      cType: types.NAME_CTYPE,
      onUpdate: null,
      perms: ["pr"],
      format: "string",
      initialValue: "Funksteckdose 3",
      supportEvents: false,
      supportBonjour: false,
      manfDescription: "Name of the accessory",
      designedMaxLength: 255
    },{
      cType: types.MANUFACTURER_CTYPE,
      onUpdate: null,
      perms: ["pr"],
      format: "string",
      initialValue: "Whatever",
      supportEvents: false,
      supportBonjour: false,
      manfDescription: "Manufacturer",
      designedMaxLength: 255
    },{
      cType: types.MODEL_CTYPE,
      onUpdate: null,
      perms: ["pr"],
      format: "string",
      initialValue: "Rev-1",
      supportEvents: false,
      supportBonjour: false,
      manfDescription: "Model",
      designedMaxLength: 255
    },{
      cType: types.SERIAL_NUMBER_CTYPE,
      onUpdate: null,
      perms: ["pr"],
      format: "string",
      initialValue: "A1S2NASF88EW",
      supportEvents: false,
      supportBonjour: false,
      manfDescription: "SN",
      designedMaxLength: 255
    },{
      cType: types.IDENTIFY_CTYPE,
      onUpdate: null,
      perms: ["pw"],
      format: "bool",
      initialValue: false,
      supportEvents: false,
      supportBonjour: false,
      manfDescription: "Identify Accessory",
      designedMaxLength: 1
    }]
  },{
    //LOOKHERE
    //sType: types.FAN_STYPE,
    sType: types.OUTLET_STYPE,
    characteristics: [{
      cType: types.NAME_CTYPE,
      onUpdate: null,
      perms: ["pr"],
      format: "string",
      initialValue: "Funksteckdose Drei",
      supportEvents: false,
      supportBonjour: false,
      manfDescription: "Name of service",
      designedMaxLength: 255
    },{
      cType: types.POWER_STATE_CTYPE,
      onUpdate: function(value)
      {
              console.log("Change:",value);
              if (value) {
                      child = exec('sudo /home/pi/433Utils/RPi_utils/codesend 6128412', function (error, stdout, stderr) {
                              console.log('On Success');
                      });
              } else {
                      child = exec('sudo /home/pi/433Utils/RPi_utils/codesend 6128403', function (error, stdout, stderr) {
                              console.log('On Success');
                      });
              }
      },
      //LOOKHERE
      //Change made because of https://github.com/KhaosT/HAP-NodeJS/issues/65
      //perms: ["pw","pr","ev"],
      perms: ["pr","ev"] ,
      format: "bool",
      initialValue: false,
      supportEvents: false,
      supportBonjour: false,
      manfDescription: "Change the power state of the fan",
      designedMaxLength: 1
    }]
  }]
}


[FLINK-27304][hive] Calcite's varbinary type should be converted to Hive's binary type.

What is the purpose of the change
To make Calcite's varbinary type convert to Hive's binary type instead of defualt void type which is not supported in Flink.
Brief change log

match varbinary type in HiveParserTypeConverter#convertPrimitiveType and convert it to Hive's binary type

Verifying this change
Additional query in HiveDialectQueryITCase#testAdditionalQueries.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? NA

@beyond1920 Could you please help review?


team section

footer of the team section and social link
please @codervivek5 assign me

Hi. Under GSSOC'24, I would like to work on this issue.
Please Assign me this task.

Yes definitely
On Mon, 13 May, 2024, 2:06 pm Somrima, @.***> wrote:

Hi. Under GSSOC'24, I would like to work on this issue.
Please Assign me this task.
—
Reply to this email directly, view it on GitHub
https://github.com/codervivek5/VigyBag/issues/1029#issuecomment-2106973990,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A45ZQXQINWILYK3KYCSEWR3ZCB3QPAVCNFSM6AAAAABHTVUMH6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWHE3TGOJZGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

kindly explain what change you want to make

I would like to work on this issue.
Please Assign me this task.

kindly explain what change you want to make

I want change the footer with media and all make it flex and giving alignment


feat(lane change): support new framework external request

Description
Use lane change module to generate external lane change.
Also support lane change new framework.
Related links
Tests performed
Please set following flag to FALSE, and confirm to pass all build.
https://github.com/autowarefoundation/autoware.universe/blob/ad5595389a629346476ee06391260ffea53159ef/planning/behavior_path_planner/CMakeLists.txt#L10
Notes for reviewers
At the moment, external lane change still couldn't be executed simultaneously.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@zulfaqar-azmi-t4 Do you have any result of Evaluator Test on old manager? If you have and there is no critical regression, it looks good to me.
On the other hand, unfortunatelly, since the new manager can't execute multiple candidate modules simlutaneously, maybe the ext LC module(s)  don't work as expected on new manager, and I'm working on fixing it now. So, I suppose that it is better to set config to not launch ext LC on the new manager until the issue is fixed.
    ext_request_lane_change_left:
      enable_module: false
      enable_simultaneous_execution: false
      priority: 6
      max_module_size: 1
    ext_request_lane_change_right:
      enable_module: false
      enable_simultaneous_execution: false
      priority: 6
      max_module_size: 1

@satoshi-ota
Yes, the PR has been tested on evaluator and no degradation found
TIER IV internal link

@zulfaqar-azmi-t4 Thanks :+1: I'll approve after the launch PR is merged.

@satoshi-ota
https://github.com/autowarefoundation/autoware_launch/pull/262
this is the modified parameter PR


test: Do not sleep in sessionAffinity tests

Instead, wait until the deleted pod is removed from the cilium endpoint lists.
This is still non-ideal, as the removal does not mean that cilium-agent has processed k8s Endpoint event which removes the endpoint from the affinity match map. However, while running the changed test in the loop I haven't witnessed the flake.

test-me-please

Coverage decreased (-0.004%) to 44.641% when pulling 5b7577c26b86b43621b24f903ffa859c7d454c53 on pr/brb/improve-session-affinity-test into 15f1a12a2992a7bb87ea96c5199dd035261df9f4 on master.

test-gke

test-with-kernel

test-me-please


Can I run rustfmt and clippy?

First of all, thank you for the very nice tool. This is exactly what I've thought of.
I wonder if I can run a rustfmt or clippy test with each rust code snippet. If it's not possible out of the box, is there any way I can access generated .rs files?

I'm only doing minimal maintenance on this project (since its original maintainer hasn't had time for it) and am not even currently using it myself. There is not currently a way to run rustfmt or clippy against it, but I'd be happy to review your PR if you want to add it (or answer your questions).


proposal: switch / link to unpkg

I think it is a good idea to switch to unpkg which uses the npm ecosystem and has a few advantages over cdnjs (where I also saw downtimes and other issues in the past).
https://metafizzy.co/blog/switching-out-cdnjs-for-unpkg/
And we should probably include the jsDelivr badge

[![jsDelivr](https://data.jsdelivr.com/v1/package/npm/foundation-sites/badge)](https://www.jsdelivr.com/package/npm/foundation-sites)

jsDelivr and unpkg are two different CDNs (which work similarly but jsDelivr is more focused on production usage and has larger network) so if you want to show the badge, you should also use jsDelivr links, otherwise there's no point in having the badge (since unpkg hits won't be included there).

Sure, I am aware of the difference.
But it seems there are no jsDelivr badges, even on shields.io.

Sorry but in that case I don't understand why you want to add a badge for one CDN and use the other.

It is better to have two than one which was sometimes unreliable in the past (cdnjs).
unpkg and delivrjs use the npm ecosystem and people can use both.
So we (can) use both.
jsDelivr provides statistics about the usage which unpkg does not.
The jsDelivr part was just another suggestion and not the main part of this issue.
We could also show the npm stats using npms / npm.

Your opinions @colin-marshall @ncoden @JeremyEnglert

Bump.

There is only the README and installation documentation to update, right ?
Also because it seems to be used in two different way (unpkg is more convenient for dev, and I guess that jsDelivr is better for an usage in production), why not both ?

Hm, not sure if we even need the badges.
But the documentation for the usage could be updated then?

https://unpkg.com/foundation-sites/dist/ (latest)
https://www.jsdelivr.com/package/npm/foundation-sites?path=dist (latest)

https://github.com/zurb/foundation-sites/commit/b3252acddd95eec496be8affc996c480d53ebb2a#diff-04c6e90faac2675aa89e2176d2eec7d8 so the badge is already in the readme in the master branch.


Missing requests dependency

Hi,
When trying the http transport I got the error: ImportError: No module named requests
Fixed it by pip installing "requests".

Hey @pelletiermaxime any chance of a PR to fix this? Thanks!

Not using this anymore, so no, sorry.


tap needs to be invoked differently on windows and linux

tap needs to be invoked differently on windows and linux, which means that npm test fails to be cross-platform.
Example: in jfhbrook/node-ecstatic I can write "test": "tap test/" and it works on windows but not linux and the other way. I can write "tap test/*.js" and it works on linux but not in windows.
I even think it is a regression. I'm not 100% sure but I believe tap test/*.js used to work in Windows back in v0.4

wow, didn't see that you pushed a new version. Just updated and it works fine in windows (even took care of the weird spawn error after the tests, I was seeing). Now I'm just waiting for confirmation on travis before closing this one.

#126 & #127 :-)

@rmg thanks! I spend an hour trying to test this on travis. I've stared the yellow dot to death, finally gave up an made an issue report and then I see #126! Perfect dramatic curve ;)

I think tap test/ should absolutely work on unix as well as Windows.  Are there tests in the test/ folder?  What happens?

tap test/*.js works. I just updated tap from 0.4 to 1.0 and moved all .js files, that was not tests, into a sub directory. As you can see on travis, it is working. I got another issue with a missing file, that works on windows but not linux. I don't think it has anything to do with tap, though.

and yes, tap test/ does also work on windows in v1.0.3.

hmm.. I just test on linux via cloud9 and tap test/ did not work, whereas tap test/*.js did.

@isaacs here is the failed test case: https://travis-ci.org/jfhbrook/node-ecstatic/jobs/62297377
Hmm hang on. This might be something else. I think that tap test/*.js runs the test independently, so I can see  the progress. While tap test/ runs everything before reporting back. So it might be failing tests that screw up tap.

Running tap test/ will recursively go through everything in the test/ folder running it with tap.
But in jfhbrook/node-ecstatic, you've got a TON of stuff in test/public/ that is not tests.  It's broken symlinks and text files and other junk. So, yeah, running tap on that stuff breaks.
I suppose it could be made to skip over things at are not either js or executables, patch welcome.

Even then, this will always fail:
$ tap test/public/d.js
C!!!
total ................................................. 0/2

  0 passing (121.359ms)
  1 pending
  1 failing

So, the requirement would be for tap test/ --no-recurse or something.

ahh ok. I only meant to run the test in the test/ directory. In v0.4, it seemingly worked. I just realized that it was giving false test numbers. Everything should be fixed in my latest commit, though.
The d.js broken symlink put me off for a while. Possibly a windows side-effect - not sure. But it was easily detectable and fixable once I got it on the cloud9 box.

@dotnetCarpenter Right, but how would tap know what is a test and what isn't?  It just arbitrarily crawls all the dirs it encounters.
In this case tap test/*.js is absolutely the right thing to be doing.  We just broke that on Windows, but it's fixed now.

well, tap could look for tap.test. I'm not sure if it's worth the trouble to implement. Ractive does something like that with esprima but also uses regex for that in the component loader.
But I really think it's a documentation issue not a missing feature.

Yeah, I'm not gonna have the tap runner read tests and try to figure out if they're loading or calling tap.  That's both too magical and a ton of work.
Besides, you might have something that outputs TAP format, but isn't actually loading reuquire('tap') to do it.  And you don't need to call tap.test() any more, you can just do var t = require('tap'); t.pass('ok') and it'll work.
Doc patch welcome.


Error loading postrender module @nexrender/action-copy no such file or directory

Dear Vlad/Inlife,
I am experiencing a bug with nexrender following the guide by you here:
https://medium.com/@inlife/guide-to-after-effects-automation-and-personalization-6039e9fa10d1
I made a JSON file very similar to the file in your guide (please find it below), and had a problem while running the following line in the cli (I am using After Effects CC 2015):

nexrender-cli-win64.exe --file document.json --binary="D:\ProgramFiles\Aobe\Adobe After Effects CC 2015\Support Files\aerender.exe" --workpath "D:\Fun"

The error is shown in the screenshot below:

In case the image is not clear, the error says:

job rendering failed
Error: Error loading postrender module @nexrender/action-copy: Error: ENOENT: no such file or directory, open 'D:\Fun\oZttHX9GXgHvU6WZ5Soz_\file\D:\Fun\myresult.mp4'
at module.exports.PromiseSerial.map.requireg.catch.err (C:\snapshot\nexrender\packages\nexrender-cli\node_modules@nexrender\core\src\tasks\actions.js:0:0)

The output I get is an encode.mp4 file (2 KB in size, basically empty, length=00:00:00), and a result.avi file (6MB in size, basically empty, length=00:00:00).
Most importantly:
To my surprise, I tried the file on aerender itself and it worked !
I used the following command:

aerender.exe -project "C:\Users\WaleedTemp\Downloads\Compressed\Adobe After Effects - Simple Logo Reveal\Adobe After Effects - Simple Logo Reveal\Simple Design Element Logo.aep"  -comp "render" -output "D:\Fun\tester.mp4"

It did not face the 'file not found problem' in the aerender log created by nexrender, and rendered an approx. 1GB, 6 seconds, .avi file. I then used FFmpeg to convert it to .mp4 and it gave me a nice 288 KB, 6 seconds, .mp4 file.
I have a few guesses regarding the source of error here:
1- I do not have QuickTime player installed, does that have to do with this issue? I know I can't render an mp4 file directly in after affects if I do not have QuickTime player installed. However, I can also see that you use FFmpeg with rendering an .avi file, so I guess you are using (avi+FFmpeg) instead of QuickTime, isn't it ?
2- That could have to do with how nexrender handles the assets in the .aep file itself, because the aerender log says:

"WARNING:After Effects warning: logged one error  error: Unable to Render: File Not Found C:\Users\jumbre23\Desktop\Simple Design Element Logo folder(Footage)\logo.png"

on the last line.
If you downloaded the .aep file from the link here, you will notice that the folder contains a (Footage) folder, that in turn contains a "logo.png" file, however, nexrender is not able to see this and somehow refers to the path that the original file creator used to input the "logo.png". To check this, I will try to substitute the assets and check if the problem was that nexrender could not deal with the default assets without substitution.
3- This error is very similar to the error in issue #132 , trying your suggested "--skip-cleanup" option, I get:

job rendering successfully finished

in the cli, however the video length is still 00:00:00 and it does not play.
Waiting to hear from you and thank you for your open source efforts !
Best Regards,
Waleed
json file

{
"template": {
"src": "file:///C:/Users/WaleedTemp/Downloads/Compressed/Adobe After Effects - Simple Logo Reveal/Adobe After Effects - Simple Logo Reveal/Simple Design Element Logo.aep",
"composition": "render"
},
"actions":{
"postrender": [
{
"module": "@nexrender/action-encode",
"preset": "mp4",
"output": "encoded.mp4"
},
{
"module": "@nexrender/action-copy",
"input": "encoded.mp4",
"output": "file///D:/Fun/myresult.mp4"
}
]
}
}

Hello @wmustafaAwad!
From what I see quickly looking over it, seems like there is an issue when the user provides custom work directory option AND also sets up an absolute path the "output" option of the action-copy. The error message pretty much reports that it can't find the path.
Please try removing --workpath "D:\Fun" argument and see if that helps with anything.

Hello @wmustafaAwad ,
there is missing colon in action-copy output value (it should be file:///D:/Fun/myresult.mp4
Also try set the output path without "file:///" so only "D:/Fun/myresult.mp4". It works for me and I also have --workpath set so I believe combination of those two is not the issue.

Hello @littlewall
Thank you for the help !
I got pretty excited (and..ehm.. well, a bit embarrassed) by the mistake. However, it did not work even after trying both approaches suggested by you, it gives me:
1- For the correction (adding colon):

job rendering failed
Error: Error loading postrender module @nexrender/action-copy: Error: ENOENT: no such file or directory, open 'D:\Fun\kO5az4443SU0o7ABuGcHX\file:\D:\Fun\myresult.mp4'
at module.exports.PromiseSerial.map.requireg.catch.err (C:\snapshot\nexrender\packages\nexrender-cli\node_modules@nexrender\core\src\tasks\actions.js:0:0)

2- For removing the whole 'file:///' thing:
It gives " job rendered successfully", but the output (now 'myresult.mp4') is a zero length 2KB mp4 file.
Will check out removing the workpath and return  back to you @inlife
Thank you for the help guys.

Ok, I looked at the issue once again. It seems like there are two problems, separate and unrelated.
And first, the main one is that rendering finishes with a video length of 0:00:00.
Considering there are no errors with rendering itself, are you sure that composition you are trying to render is set to be with a certain amount of frames?
Such warnings (like cant access footage file) should not be a causing critical render problem unless you add a flag that prevents rendering if footage item is missing.
However, regarding image substitution, you need to manually specify all needed footage assets in the job description (using job.assets as it showed in the readme), so they would be delivered to the machine/folder where rendering is taking place at.

so I guess you are using (avi+FFmpeg) instead of QuickTime, isn't it?

Yes, since AE pretty much disallows to render anything but avi/mov from the actual cli side, nexrender uses FFmpeg as one of the optional finishing steps.
That problem seems a bit weird, we need to understand what is causing the initial file to be wrongly rendered.

Hey @inlife  and @littlewall , it worked !
Thank you guys for the help.
What I did was:
I only added the paths of the assets in the JSON file.  Apparently, nexrender has a problem reading the path if it's not clearly stated in the JSON file, which is a very helpful tip for the future.
So, for your and any later inquirer's reference, I am putting the JSON file here:
{
    "template": {
        "src": "file:///D:/Fun/Adobe After Effects - Simple Logo Reveal/Simple Design Element Logo.aep",
        "composition": "render"
    },

    "assets": [
        {

            "src": "file:///D:/Fun/Adobe After Effects - Simple Logo Reveal/(Footage)/logo.png",
            "composition": "Your_Logo",
            "type": "image",
            "layerName": "logo.png"
        },
        {
            "composition": "your text",
            "type": "data",
            "layerName": "your text here",
            "property": "Source Text",
            "expression": "'Hello'"
        },
        {
            "composition": "shadow_comp",
            "type": "data",
            "layerIndex": 3,
            "property": "Source Text",
            "expression": "'Hello'"
        },
        {
            "src": "file:///D:/Fun/Adobe After Effects - Simple Logo Reveal/(Footage)/circle.mp3",
            "type": "audio",
            "layerIndex": 3
        }

    ],
    "actions":{
        "postrender": [
            {
                "module": "@nexrender/action-encode",
                "preset": "mp4",
                "output": "encoded.mp4"
            },
            {
                "module": "@nexrender/action-copy",
                "input": "encoded.mp4",
                "output": "D:/Fun/myresult.mp4"
            }
        ]
    }
}

very nice 👍

Hi All,
Till a few days ago the encoding process was working just fine. However now i keep getting this error. Any idea why this might be happening ?

Hello All,
@kushalkalambi I am facing same issue now.that is below
UnhandledPromiseRejectionWarning: Error: spawn EBUSY
how to resolve this issue?
Please help me.

@sitestrokes The issue people had before was with antivirus: https://github.com/inlife/nexrender/issues/285


section/code folding was not supportted

please provide such a function which support code block or chapter and section folding, that would helps a lot...

Folding in UDL requires open and close keywords. Seems it is not possible to fold headings # or ====/---- because the syntax does not have any closing keyword.
For code block, I considered it as important part for whole document. So there is no need to hide it. Wait to see other people who really need it?


Please invite me to the GitHub Community Organization

Name
Derek
Discord Username (if applicable)
No response
Additional Context
I hope to join this organization and contribute my projects

Welcome @SoftwareArchitect9817 🎉🥳


Type checking issue for union types: (json|error) cannot be assigned to (any|error)

Description:
import ballerina/io;

public function main() {
    io:StringReader reader = new("some words without quotes");
    io:println(reader.readJson());
}

Above gives a compile time error:
error: .::testMain.bal:21:16: incompatible types: expected '(any|error)', found '(json|error)'

seems the issue is with error:
    if(x is json) {
        io:println(x);
    } else {
        io:println(x);    // gives error "incompatible types: expected '(any|error)', found 'error'"
    }

This is because the error returned from readJson() is IOError which has a record for its detail while the builtin error has a map for its detail. This should work once the lang lib related changes are merged to master since the builtin error detail type was changed to a record type in that branch.

Works in the master


[Heap] Remove Heap's ascending and descending views

These two do very non-trivial processing, so they don't at all work like views in the Standard Library.
They also don't carry their weight -- they are simply lightweight wrappers around popMin and popMax, respectively.
Let's get rid of them before we etch them in stone as public API.
Checklist

[X] I've read the Contribution Guidelines
[X] My contributions are licensed under the Swift license.
[X] I've followed the coding style of the rest of the project.
[ ] I've added tests covering all new code paths my change adds to the project (if appropriate).
[ ] I've added benchmarks covering new functionality (if appropriate).
[X] I've verified that my change does not break any existing tests or introduce unexplained benchmark regressions.
[ ] I've updated the documentation if necessary.

Cc @AquaGeek


feat: enabledBottomInitialAnimation prop added

If true sheet will grows up from bottom to initial snapPoint.

can you add some video?

@osdnk sorry for disturbing, is this going to be released soon?

@russ666 @osdnk please give more information what about it?  Maybe that props helps me.


Added node 8 to CI

Implements #523.

Coverage remained the same at 91.453% when pulling 95613c9d09d24c26e4c657066d3ba9fe2603f41d on add-node-8-to-ci into 4a3b68025b4914c6559e05005d08999a5b615867 on master.

Thanks @nickytonline ! 👍


Provide test coverage for CDI Lite language model

CDI Lite comes with a new language model abstraction that needs to be tested.
As this API is a potential subject for extraction into a separate module, tests should be written in some standalone manner.

Resolved via https://github.com/eclipse-ee4j/cdi-tck/pull/310


Reduce boiler-plate for service creation to single class

simplify context creation
ClusterNode should be an interface, or be removed
base config path on service name

Investigate how contexts can be decoupled from logging.

added SimpleServiceContext that takes a CoreContext as a parameter and the service address.
added CoreHandler that extends verticle and is a simpler form of AbstractHandler.
SimpleServiceContext resolves the configuration filename from the service address.


Add register data fetching to decoder

Continuing the work on decoder, added crucial register data fetching.

No issues pointed out on Discord.


Music Recommendation System Project

Project Description 📄
Hello everyone! We are looking for contributors who can help us build a Music Recommendation System using Python. This is a beginner-friendly project and a great opportunity to learn and grow. The goal of this project is to create a system that can suggest songs to users based on their listening history.

Hi could you assign me this issue

Hey @barkhaaroraa go for it 🚀
If you're contributing as part of Hacktoberfest, please ensure that you make your pull request after October 1st.

alright

hi @Mrinank-Bhowmick can you assign me this , @barkhaaroraa can you colabrate?

Collaborate , assigning this issue to you also .
Just mention this issue in your pr with #

Hi, would I be able to contribute to this issue still?

Hi, would I be able to contribute to this issue still?

yes its open , go for it

Hi, would I able to contribute to it now?

Wow! . I will like to contribute

When can we start?> Wow! . I will like to contribute


Update LAB[PL-400]_Lab02_Model_Driven_App.md

Module: 00
Lab/Demo: 02
Exercice 1 - Task 1.5
Fixes # .
Changes proposed in this pull request:

Line 325 - Table name mistype. Here we are working on the "Permit Types" table and not the Inspection one.
Line 326 - The view name is "Active Permit Types" and not "Active Active Permit Types" as written.

Changes are correct


Upgrade Prometheus to v2.22.2

This should prevent the problem (mmap: invalid argument) outlined in this issue:
https://github.com/prometheus/prometheus/issues/8114
How to categorize this PR?

/area monitoring
/kind enhancement
/priority normal
What this PR does / why we need it:
Upgrade Prometheus to v2.22.2
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Upgrade Prometheus to v2.22.2. Sometimes Prometheus would have the error `mmap: invalid argument`. Prometheus v2.22.1+ provides a fix for this issue.

/cla


Changing upstream TCP packets on the fly

I am using OpenResty as a TCP (stream) proxy. I have been capable to read TCP packets using reqsock:peek and preread_by_lua_block directives.
However, would like to change certain bytes of the TCP packet as it flows though the proxy, so that the service behind gets a changed packet.
Is this possible to achieve with ngx_stream_lua_module and how?

Any updates on this? Is something like this possible?

Any updates on this?

Same here, couldn't find related APIs

+1

need to add body filter  feature


add support for index

Hey, I think it's useful to add index pragma.
type
  Model = object
    ...
    title {.index.}: string

Yup sounds useful 👌 should hopefully be able to implement it quickly

Thanks for adding this feature!
Please bump the version in .nimble file so I don't have to write:
nimble install ponairi@#head

Sorry about that, forgot to push the new tag
Done now :+1:


Fix dev image

Tests fail because of FAILED jupyterlab_vre/tests/test_handlers.py::HandlersAPITest::test_cells_handler - Exception: Error getting jobs for workflow run: {"message":"API rate limit exceeded for <IP_ADDRESS>. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)","documentation_url":"https://docs.github.com/rest/overview/resources-in-the-rest-api#rate-limiting"}
Restart later


fix: prefers-color-scheme only when no dark-mode setting by user

@all-contributors add @strdr4605 for ideas, code, infra, mentoring


Hubot Login Failure

It seems the Skype web portal has changed such that it no longer shows two fields by default. If you visit "https://web.skype.com" in incognito mode you see the same page that is rendered in login-failure.png:

Along with the following error:
 phantom stdout: TypeError: null is not an object (evaluating 'document.querySelector(selector).value = value')

phantom stdout:   undefined:5
  :8

[Tue Oct 18 2016 07:36:49 GMT-0600 (MDT)] ERROR Timeout in waiting for login success!
[Tue Oct 18 2016 07:36:50 GMT-0600 (MDT)] ERROR Screenshot saved at: /Users/kgorham/workspace/android/hubot/login-failure.png
[Tue Oct 18 2016 07:36:50 GMT-0600 (MDT)] ERROR Error: SkypeWeb adapter failure in initial login!

However, if you log in manually and then logout, you are presented with a page that has both fields. It seems that this adapter would need to be updated to handle the scenario where the password field is on a separate screen.
We run our builds in hubot daily and this is the first we've seen this happen. So I'm guessing it is new behavior on the Skype side.

Looking at the source, this "two screen" login is only expected when the username contains an @ (which mine does not). If I comment out the if self.username.indexOf('@') is -1 portion (lines 157-159), forcing it to run the else portion, I get beyond the first screen but now fail with ERROR Timeout in waiting for login success! and this screenshot:

After playing with this a bit, it seems that clicking the submit button no longer works. I'm not well versed in phantomJS or even JS but adding the following temporary hack fixes this:
open page_helper.coffee and comment out (around line 40):
document.querySelector("[type='submit']").click()
and replace with:
document.forms[0].submit()
This, of course is in addition to the hack mentioned above where you comment out the first part of the if clause in skypeweb.coffee:
          helper.wait '#username', ->
            # if false || self.username.indexOf('@') is -1
            #  helper.fillForm '#username': self.username, '#password': self.password
            # else  # login with a Windows Live account
            # Submit username to trigger redirect
            helper.fillForm '#username': self.username
            # Wait a redirect to Windows Live login page
            helper.wait 'input[type="submit"]', ->
              helper.fillForm 'input[type="password"]': self.password

I placed the above mentioned changes into pull request https://github.com/sdimkov/hubot-skype-web/pull/28