Error installing feature node

Hi,
With a json file like this:
"features": {
		"ghcr.io/devcontainers/features/common-utils:2": {},
		"ghcr.io/devcontainers/features/node:1": {
			"version": "lts"
		},
		"ghcr.io/devcontainers/features/docker-outside-of-docker:1": {
			"version": "latest",
			"moby": false,
			"dockerDashComposeVersion": "v2"
		}
	},

I get the following error during start up:
#0 9.905 Now using node v18.14.0 (npm v9.3.1)
#0 9.979 Creating default alias: default -> lts/* (-> v18.14.0 *)
#0 10.03 => Node.js version lts/* has been successfully installed
#0 10.03 => Close and reopen your terminal to start using nvm or run the following to use it now:
#0 10.03
#0 10.03 export NVM_DIR="/usr/local/share/nvm"
#0 10.03 [ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm
#0 10.03 sh: 6: source: not found
#0 10.03 ERROR: Feature "Node.js (via nvm) and yarn" (ghcr.io/devcontainers/features/node) failed to install! Look at the documentation at https://github.com/devcontainers/features/tree/main/src/node for help troubleshooting this error.

Thanks.

Hi 👋
I tried to reproduce with this dev container 👇
{
    "image": "mcr.microsoft.com/devcontainers/base:ubuntu",
    "features": {
		"ghcr.io/devcontainers/features/common-utils:2": {},
		"ghcr.io/devcontainers/features/node:1": {
			"version": "lts"
		},
		"ghcr.io/devcontainers/features/docker-outside-of-docker:1": {
			"version": "latest",
			"moby": false,
			"dockerDashComposeVersion": "v2"
		}
	},
}

The node Feature installed without any errors for me.
@samstride  Can you paste your entire dev config? That might help investigate.
 For config options, see the README at:
// https://github.com/microsoft/vscode-dev-containers/tree/v0.245.2/containers/go
{
	"name": "dev",
	"build": {
		"dockerfile": "Dockerfile"
	},
	"runArgs": [
		"--cap-add=SYS_PTRACE",
		"--security-opt",
		"seccomp=unconfined"
	],
	// Configure tool-specific properties.
	"customizations": {
		// Configure properties specific to VS Code.
		"vscode": {
			// Set *default* container specific settings.json values on container create.
			"settings": {
				"terminal.integrated.defaultProfile.linux": "zsh",
			},
			// Add the IDs of extensions you want installed when the container is created.
			"extensions": [
				"golang.Go",
				"ms-azuretools.vscode-docker",
				"zxh404.vscode-proto3",
				"xaver.clang-format",
				"ms-kubernetes-tools.vscode-kubernetes-tools",
				"yzhang.markdown-all-in-one",
				"bradymholt.pgformatter",
				"emeraldwalk.runonsave",
				"redhat.vscode-xml",
				"redhat.vscode-yaml",
				"dbaeumer.vscode-eslint"
			]
		}
	},
	// Use 'forwardPorts' to make a list of ports inside the container available locally.
	// "forwardPorts": [],
	// Use 'postCreateCommand' to run commands after the container is created.
	"postCreateCommand": "devcontainer/init.sh",
	// Comment out to connect as root instead. More info: https://aka.ms/vscode-remote/containers/non-root.
	"remoteUser": "vscode",
	"features": {
		"ghcr.io/devcontainers/features/common-utils:2": {},
		"ghcr.io/devcontainers/features/node:1": {
			"version": "lts"
		},
		"ghcr.io/devcontainers/features/docker-outside-of-docker:1": {
			"version": "latest",
			"moby": false,
			"dockerDashComposeVersion": "v2"
		}
	},
	"containerEnv": {
		"TZ": "Pacific/Auckland"
	}
}

Extension version = v0.275.1
Thanks.

The issue is with your Dockerfile not correctly adding and setting the vscode user. As user is not set properly, the nvm install fails.
@samstride Can you update your devcontainer.json as follows? This solved the issue for me
// For format details, see https://aka.ms/devcontainer.json. For config options, see the README at:
// https://github.com/microsoft/vscode-dev-containers/tree/v0.245.2/containers/go
{
	"name": "dev",
	"image": "golang:1.20.0-bullseye",
	"runArgs": [
		"--cap-add=SYS_PTRACE",
		"--security-opt",
		"seccomp=unconfined"
	],
	// Configure tool-specific properties.
	"customizations": {
		// Configure properties specific to VS Code.
		"vscode": {
			// Set *default* container specific settings.json values on container create.
			"settings": {
				"terminal.integrated.defaultProfile.linux": "zsh"
			},
			// Add the IDs of extensions you want installed when the container is created.
			"extensions": [
				"golang.Go",
				"ms-azuretools.vscode-docker",
				"zxh404.vscode-proto3",
				"xaver.clang-format",
				"ms-kubernetes-tools.vscode-kubernetes-tools",
				"yzhang.markdown-all-in-one",
				"bradymholt.pgformatter",
				"emeraldwalk.runonsave",
				"redhat.vscode-xml",
				"redhat.vscode-yaml",
				"dbaeumer.vscode-eslint"
			]
		}
	},
	// Use 'forwardPorts' to make a list of ports inside the container available locally.
	// "forwardPorts": [],
	// Use 'postCreateCommand' to run commands after the container is created.
	"postCreateCommand": "devcontainer/init.sh", //Custom install script.
	// Comment out to connect as root instead. More info: https://aka.ms/vscode-remote/containers/non-root.
	"remoteUser": "vscode",
	"features": {
		"ghcr.io/devcontainers/features/common-utils:2": {
            "username": "vscode"
        },
		"ghcr.io/devcontainers/features/node:1": {
			"version": "lts"
		},
		"ghcr.io/devcontainers/features/docker-outside-of-docker:1": {
			"version": "latest",
			"moby": false,
			"dockerDashComposeVersion": "v2"
		}
	},
	"containerEnv": {
		"TZ": "Pacific/Auckland"
	}
}

ℹ Also, opened https://github.com/devcontainers/images/pull/412, we would release go:1.20 tomorrow, then you could simply use that image and avoid using common-utils Feature and remove some other properties (like remoteUser, few extensions) as well.
tip: The docker-outside-of-docker, adds the run-args, feel free to skip them!

@samruddhikhandale , thanks.

The issue is with your Dockerfile not correctly adding and setting the vscode user. As user is not set properly, the nvm install fails.

Can you elaborate on that. I was using this guide here: https://code.visualstudio.com/remote/advancedcontainers/add-nonroot-user#_creating-a-nonroot-user
Also, I forgot to paste the end of my Dockerfile, which is:
USER ${USERNAME}
# Install few other packages
.
.
>

I used the setting :
"ghcr.io/devcontainers/features/common-utils:2": {
  "username": "vscode"
}

and I still get the same error.
Btw, thanks for the tip :thumbsup:

tip: The docker-outside-of-docker, adds the run-args, feel free to skip them!

mcr.microsoft.com/devcontainers/go:1.20 image is live.
@samstride Can you update the image property and remove the common-utils Feature? That should fix your issue.

@samruddhikhandale , Thanks.
Still curious to know what is the correct way to set the user:

The issue is with your Dockerfile not correctly adding and setting the vscode user. As user is not set properly, the nvm install fails.

I was using this guide here: https://code.visualstudio.com/remote/advancedcontainers/add-nonroot-user#_creating-a-nonroot-user

I am not sure what's the exact issue was in there (I'll take a peek), but it's nice to use the common-utils Feature for that purpose.

Thanks, if we remove common-utils, it means that zsh and oh-my-zsh will not be setup, which we want in the dev container.
Also, the issue seems to pop up only if node feature is used. (nvm seems to be causing an issue). Without that feature, everything else seems to work.

Getting the same error after migrating from @microsoft/vscode-dev-containers.
Here's my complete devcontainer.json looks like.
Tried setting the username in both places mentioned above but still got the same error.

@kosalanuwan , I never got it to work.  Are you getting the exact same error? Does your devcontainer work if you remove node?
Since I use my own images (mostly because I want to maintain my own velocity), I installed node in the base image.
You could do something like this in your json to see if it will work:
// "image": "mcr.microsoft.com/devcontainers/base:dev-bullseye",
"build": {
		"dockerfile": "Dockerfile"
	},

Then create a Dockerfile in .devcontainer as follows:
FROM mcr.microsoft.com/devcontainers/base:dev-bullseye

# Install NodeJS LTS
RUN curl -fsSL https://deb.nodesource.com/setup_lts.x | bash - &&\
    apt-get install -y nodejs

@kosalanuwan Can you remove the common-utils Feature and retry? The base image is adding the vscode user with the common-utils Feature. I suspect that's the issue.

@kosalanuwan Can you remove the common-utils Feature and retry? The base image is adding the vscode user with the common-utils Feature. I suspect that's the issue.

@samruddhikhandale this worked for me! Appreciate the support 🥳

Still curious to know what is the correct way to set the user:

The issue is with your Dockerfile not correctly adding and setting the vscode user. As user is not set properly, the nvm install fails.

I was using this guide here: https://code.visualstudio.com/remote/advancedcontainers/add-nonroot-user#_creating-a-nonroot-user

I ran into this same issue and finally figured out what was happening. The guide for creating a nonroot user (linked to above) calls useradd but without a -s option to set the login shell. I believe it's a bug, but devcontainers/features/node is essentially expecting this to be bash because it has an explicit source in this line of install.sh. However if you don't add -s /bin/bash to the useradd then you get /bin/sh by default as the login shell for the user, which yields the sh: 6: source: not found error.
The simplest resolution is to add -s /bin/bash to your useradd command. Zooming out however I think this is a bug in devcontainers/features and I'm going to create an issue for it.

@kosalanuwan , I never got it to work. Are you getting the exact same error? Does your devcontainer work if you remove node?

@samstride I'd avoid managing Dockerfiles, so I use the dotnet/sdk:8.0-bookworm-slim image.
@samruddhikhandale It works fine with the common-utils and node features together. This is on an Apple M1 (2020) running Sonoma 14.1.1.


Pageview event in an SPA

Hi,
Is it possible to use your code in a single page application (NextJS) and send an event (e.g: ga('send', 'pageview') )  whenever a new page is loaded so we can track pages as well?
thanks

Maybe expose a function to the global could achieve this.

Done in https://github.com/SukkaW/cloudflare-workers-async-google-analytics/commit/4e6ac6956ff3a70973f56159a8b4e16a0c6a01f1 and launched in<EMAIL_ADDRESS>1.0.3 Release Notes.

thanks, works great!


Only trigger UI type generation workflow if api.yml has changed

Only trigger the generate types workflow in hmpps-approved-premises-ui when a change to api.yml was merged

 Trigger UI type generation when API is updated


misinterpreted backslashes in registry_key resource?

In my recipe I simply do:
include_recipe 'sysinternals'
Also added "cookbook 'sysinternals', '~> 0.1.0'" to Berksfile and "depends 'sysinternals'" to metadata.rb
When running "kitchen converge" I get this error:
       Chef::Exceptions::Win32RegHiveMissing
       -------------------------------------
       Registry Hive HKEY_CURRENT_USERSoftwareSysinternals#{toolname} does not exist

       Resource Declaration:
       ---------------------
       # In C:\tmp\kitchen\cookbooks\sysinternals\recipes\default.rb

         1: #
         2: # Cookbook Name:: sysinternals

       Compiled Resource:
       ------------------
       # Declared in C:\tmp\kitchen\cookbooks\sysinternals\recipes\default.rb:38:in `block in from_file'

       registry_key("HKEY_CURRENT_USERSoftwareSysinternals#{toolname}") do
         action [:create]
         retries 0
         retry_delay 2
         default_guard_interpreter :default
         architecture :machine
         key "HKEY_CURRENT_USERSoftwareSysinternals\#{toolname}"
         unscrubbed_values [{:name=>"EulaAccepted", :type=>:dword, :data=>1}]
         values [{:name=>"EulaAccepted", :type=>:dword, :data=>1}]
         declared_type :registry_key
         cookbook_name :sysinternals
         recipe_name "default"
       end

   Running handlers:
   [2015-03-29T02:30:42-07:00] ERROR: Running exception handlers
   Running handlers complete
   [2015-03-29T02:30:42-07:00] ERROR: Exception handlers complete
   [2015-03-29T02:30:42-07:00] FATAL: Stacktrace dumped to c:/tmp/aleg/cache/chef-stacktrace.out
   Chef Client failed. 8 resources updated in 46.437495 seconds
   [2015-03-29T02:30:42-07:00] FATAL: Chef::Exceptions::Win32RegHiveMissing: registry_key[HKEY_CURRENT_USERSoftwareSysinternals#{toolname}] (sysinternals::default line 38) had an error: Chef::Exceptions::Win32RegHiveMissing: Registry Hive HKEY_CURRENT_USERSoftwareSysinternals#{toolname} does not exist

My workaround was to simply insert double-backslashes in the registry_key resource name.
Is this an actual issue or am I doing something wrong?
-Omer

Looks like you are right. Want to send a PR?

This is fixed in master now


conda env create -f environment.yaml fails in fresh conda install

I'm using windows10, with a fresh anaconda install.
conda env create -f environment.yaml fails, but the fix is simple: change "sklearn"  to "scikit-learn" in the yaml file.
c:\projects\open-musiclm>conda env create -f environment.yaml
Collecting package metadata (repodata.json): done
Solving environment: done
Downloading and Extracting Packages
Preparing transaction: done
Verifying transaction: done
Executing transaction: done
Installing pip dependencies: | Ran pip subprocess with arguments:
['C:\Users\Michael\anaconda3\envs\open-musiclm\python.exe', '-m', 'pip', 'install', '-U', '-r', 'c:\projects\open-musiclm\condaenv.388f_48u.requirements.txt', '--exists-action=b']
Pip subprocess output:
Looking in links: https://download.pytorch.org/whl/torch_stable.html
Collecting torch==2.0.0+cu117
Downloading https://download.pytorch.org/whl/cu117/torch-2.0.0%2Bcu117-cp310-cp310-win_amd64.whl (2343.6 MB)
---------------------------------------- 2.3/2.3 GB ? eta 0:00:00
Collecting torchvision==0.15.1+cu117
Downloading https://download.pytorch.org/whl/cu117/torchvision-0.15.1%2Bcu117-cp310-cp310-win_amd64.whl (4.9 MB)
---------------------------------------- 4.9/4.9 MB 7.3 MB/s eta 0:00:00
Collecting torchaudio==2.0.1+cu117
Downloading https://download.pytorch.org/whl/cu117/torchaudio-2.0.1%2Bcu117-cp310-cp310-win_amd64.whl (2.5 MB)
---------------------------------------- 2.5/2.5 MB 17.4 MB/s eta 0:00:00
Collecting einops>=0.4
Downloading einops-0.6.1-py3-none-any.whl (42 kB)
-------------------------------------- 42.2/42.2 kB 146.8 kB/s eta 0:00:00
Collecting vector-quantize-pytorch>=0.10.15
Downloading vector_quantize_pytorch-1.4.1-py3-none-any.whl (11 kB)
Collecting librosa==0.10.0
Downloading librosa-0.10.0-py3-none-any.whl (252 kB)
-------------------------------------- 252.9/252.9 kB 1.7 MB/s eta 0:00:00
Collecting torchlibrosa==0.1.0
Downloading torchlibrosa-0.1.0-py3-none-any.whl (11 kB)
Collecting ftfy
Using cached ftfy-6.1.1-py3-none-any.whl (53 kB)
Collecting tqdm
Using cached tqdm-4.65.0-py3-none-any.whl (77 kB)
Collecting transformers
Downloading transformers-4.29.1-py3-none-any.whl (7.1 MB)
---------------------------------------- 7.1/7.1 MB 16.1 MB/s eta 0:00:00
Collecting encodec==0.1.1
Downloading encodec-0.1.1.tar.gz (3.7 MB)
---------------------------------------- 3.7/3.7 MB 11.4 MB/s eta 0:00:00
Preparing metadata (setup.py): started
Preparing metadata (setup.py): finished with status 'done'
Collecting gdown
Using cached gdown-4.7.1-py3-none-any.whl (15 kB)
Collecting accelerate>=0.17.0
Downloading accelerate-0.19.0-py3-none-any.whl (219 kB)
-------------------------------------- 219.1/219.1 kB 3.3 MB/s eta 0:00:00
Collecting beartype
Downloading beartype-0.14.0-py3-none-any.whl (720 kB)
-------------------------------------- 720.2/720.2 kB 9.1 MB/s eta 0:00:00
Collecting joblib
Downloading joblib-1.2.0-py3-none-any.whl (297 kB)
-------------------------------------- 298.0/298.0 kB 3.7 MB/s eta 0:00:00
Collecting h5py
Downloading h5py-3.8.0-cp310-cp310-win_amd64.whl (2.6 MB)
---------------------------------------- 2.6/2.6 MB 16.8 MB/s eta 0:00:00
Collecting sklearn
Downloading sklearn-0.0.post5.tar.gz (3.7 kB)
Preparing metadata (setup.py): started
Preparing metadata (setup.py): finished with status 'error'
Pip subprocess error:
/   error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [18 lines of output]
The 'sklearn' PyPI package is deprecated, use 'scikit-learn'
rather than 'sklearn' for pip commands.
  Here is how to fix this error in the main use cases:
  - use 'pip install scikit-learn' rather than 'pip install sklearn'
  - replace 'sklearn' by 'scikit-learn' in your pip requirements files
    (requirements.txt, setup.py, setup.cfg, Pipfile, etc ...)
  - if the 'sklearn' package is used by one of your dependencies,
    it would be great if you take some time to track which package uses
    'sklearn' instead of 'scikit-learn' and report it to their issue tracker
  - as a last resort, set the environment variable
    SKLEARN_ALLOW_DEPRECATED_SKLEARN_PACKAGE_INSTALL=True to avoid this error

  More information is available at
  https://github.com/scikit-learn/sklearn-pypi-package

  If the previous advice does not cover your use case, feel free to report it at
  https://github.com/scikit-learn/sklearn-pypi-package/issues/new
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
failed
CondaEnvException: Pip failed

thanks for catching this! fixed


fix: Make MD040/fenced-code-language case sensitive.

Fix #627.

I added the parenthesis and a sentence in the documentation. I ran all build-*.
I also removed the trim for comparison with allowed languages so this code reports an error:
<!-- markdownlint-configure-file { "MD040": { "allowed_languages": ["js"] } } -->
``` js
// ^ Space!
let foo = "bar";
```

Only the first "word" of the info string is the language, right? That should be present so the code can be formatted properly. But the rest of the line is a mystery. It may be used to format better, but I'd worry about a setting that blocks if it may be needed for some cases?


DO NOT MERGE. Debug/055 failures 1

Debugging issues on mainline.

Reverting TBB changes had no effect 16fc05a
Reverting withlifting changes "fixed" it, but this might be down to test ordering c6e8505
Adding a change to make SVML tests run first in 742e4d6

On 30ea825 the SVML fail happens in the git diff part of the tests:
======================================================================
FAIL: test_float32_numpy8_fastmath_usecase (numba.tests.test_svml.TestSVMLGeneration)
----------------------------------------------------------------------
Parallel: 192. Serial: 82
Traceback (most recent call last):
  File "D:\a\1\s\numba\tests\test_svml.py", line 248, in test_runner
    self.fail("Process timed out.")
AssertionError: Process timed out.

/AzurePipelines run

/AzurePipelines run

Success on:
__Hardware Information__
Machine                                       : AMD64
CPU Name                                      : skylake-avx512
CPU Count                                     : 4
Number of accessible CPUs                     : ?
List of accessible CPUs cores                 : ?
CFS Restrictions (CPUs worth of runtime)      : None

/AzurePipelines run

Success again on:
__Hardware Information__
Machine                                       : AMD64
CPU Name                                      : skylake-avx512
CPU Count                                     : 4
Number of accessible CPUs                     : ?
List of accessible CPUs cores                 : ?
CFS Restrictions (CPUs worth of runtime)      : None

/AzurePipelines run

Success on:
__Hardware Information__
Machine                                       : AMD64
CPU Name                                      : skylake-avx512
CPU Count                                     : 2
Number of accessible CPUs                     : ?
List of accessible CPUs cores                 : ?
CFS Restrictions (CPUs worth of runtime)      : None

/AzurePipelines run

Closing for now.


Remove cabal database dependency

stack2nix: /nix/.cabal/packages/hackage.haskell.org/00-index.tar: openBinaryFile: does not exist (No such file or directory)

It seems this is a cabal2nix issue, reproduced with Travis build #36


async w/ python 3.8 compatibility sans warnings

this closes #10  by using async syntax that doesn't trigger warnings.
this pr also removes from travis the versions of python3 that are end of life'd (3.3, 3.4) adds the newer ones (3.7, 3.8) and increments the version (post0->post1) for release to pypi.

@RazerM can you look into this PR? I would like to have the update as well

With 3.9 this would be great. I also have another issue with
https://mystb.in/RepresentativeBestApplications.apache
Would love a fix!

Thanks @kapilt for this update. Really hoping @RazerM sees this and can review/merge/release.

This seems to still work correctly in a development version of Python 3.11, in which the warned-about asyncio.coroutine decorator is actually removed.

Any movement on this issue? I see that https://github.com/pypa/pypi-support/issues/954 resuled in @gear4s being added as a maintainer by @RazerM . I'd like to use this library and it would help me out if this PR could get merged and a new PyPi release happen.

Any movement on this issue? I see that https://github.com/pypa/pypi-support/issues/954 resuled in @gear4s being added as a maintainer by @RazerM . I'd like to use this library and it would help me out if this PR could get merged and a new PyPi release happen.

I will review and test this week, if all's good I will merge.

Any movement on this issue? I see that pypa/pypi-support#954 resuled in @gear4s being added as a maintainer by @RazerM . I'd like to use this library and it would help me out if this PR could get merged and a new PyPi release happen.

I will review and test this week, if all's good I will merge.

Awesome, looking forward to seeing this merged.

@gear4s have you had a chance to review and test this yet?

@gear4s have you had a chance to review and test this yet?

Not yet - I've dedicated some time Saturday to do this. Work is beating me right now unfortunately

@gear4s did you ever have a chance to try this?

This is officially removed in Python 3.11 so dependent packages are now breaking.

FWIW @gshuflin I patched my local version with this PR and it works fine. But merging would be preferred so users don't encounter the same error.

@gear4s would be great to have this merged and released now that projects that use ratelimiter are stuck from upgrading to Python 3.11

Hello folks, it's kinda self-promoting, but since this repo was not updated since 2018 consider using https://github.com/uburuntu/throttler for async cases -- it has pretty close syntax and is a bit more efficient.
Migration for context managers
from ratelimiter import RateLimiter

rate_limiter = RateLimiter(max_calls=10, period=1)
with rate_limiter:
    do_something()

⇾
from throttler import Throttler

rate_limiter = Throttler(rate_limit=10, period=1)
async with rate_limiter:
    await do_something()

https://github.com/uburuntu/throttler#simple-example
Migration for decorators
from ratelimiter import RateLimiter

@RateLimiter(max_calls=10, period=1)
def do_something():
    pass

⇾
from throttler import throttle

@throttle(rate_limit=10, period=1)
async def do_something():
    pass

https://github.com/uburuntu/throttler#throttler-and-throttlersimultaneous
Migration example
https://github.com/snakemake/snakemake/pull/1958


Japanese doesn't word wrap correctly in TextFields

Repro case:
https://gist.github.com/larsiusprime/5df42543a4b09e07274cce2ad9943b83
What it should look like (flash isn't loading the font but the wrapping is correct):

What it does look like (neko/html5 have same results):

As you can see, it seems to be an issue with Japanese specifically, as cyrillic (Russian in this case) text wraps correctly.
I tried with Korean text just now:

And that seems to be working correctly. I don't have any Chinese strings to test with.
The Korean test string is:
'시체\n시체들은 종종 죽은 지 몇 일 후에 구덩이 속에서 다시 일어납니다. 이런 괴물들은 ""다시 돌아온 자""라는 의미의 ""망령""이라 알려져 있습니다.'
And the font is "nanum gothic", bold, available here:
http://www.ffonts.net/NanumGothicBold.font
Everything else necessary to repro with for Japanese is included in the gist.

So from some conversations deduced it's because Japanese doesn't use spaces.
Update, just did a test to see what happens if I remove all the spaces from english text:
Flash:

OpenFL (neko/html5):

If you remove the english spaces, flash's behavior (but not openfl's) is to break on whatever character, even in the middle of a line.
So it seems like this approach would be "no worse" than flash, and would be appropriate regardless of locale.

Presumably the issue is somewhere in this giant function:
https://github.com/openfl/openfl/blob/develop/openfl/_internal/text/TextEngine.hx#L576

Every issue is in that function 😉
I'll look at it in a bit.

Ok problem: Part 1 also needs to do Part 2 when the next line break comes after a long, unbroken string.
Right now, everything assumes only spaces and crlfs can cause a line break, so it adds chunks of text at a time. Preeeeetty sure it needs to be done character by character since any of those chunks can be too big. I can't immediately think of a way to avoid that.

Getting there. Kinda looks like flash's text is slightly wider? Look closely at the n
HTML5:

Flash:

Someone else post, I don't want 5 in a row :(

Great work!

@MSGhero : here are the local changes I made that fix the "trailing premature broken character" edge case:
https://github.com/larsiusprime/openfl/commit/9ff46d6e15e8ab46b112e55cab641ffff1cd1a48

Edge cases 🙃🍷🙄😒😤😭

You also want these:
https://github.com/larsiusprime/openfl/commit/42c189d7d66c2cf74eb1287afb94c2d142083061
https://github.com/larsiusprime/openfl/commit/caaf65059b3c6e96aee4e26503b69fefac2cff4e
https://github.com/larsiusprime/openfl/commit/21685e0313fd796a0c3d5edaa50ed4e0b70af329

Here's a fun example, same font that you posted earlier:
var size = #if neko 12 #else 11 #end; // flash 11pt == neko 12pt for some reason???

var font = Assets.getFont("assets/mplus2cb.ttf").fontName;

var tf = new TextField();
tf.embedFonts = true;
tf.defaultTextFormat = new TextFormat(font, size, null, null, null, null, null, null, TextFormatAlign.LEFT);
tf.text = "こを使するこ123456789012345678901 23 4567890キストです。";
tf.x = 30;
tf.y = 30;
tf.width = 100;
tf.height = 200;
tf.border = true;
tf.wordWrap = true;
addChild(tf);

Neko top, flash bottom

On flash, the first break occurs after the first Japanese bit. On Neko, with my changes, the break occurs after some numbers. However, if you change the string to have English letters after the first Japanese bit (こを使するこabcde), it breaks mid-letters on Flash. Flash is purposefully breaking on numbers rather than doing it by lineWidth > width - 2.


Scrollbar with style added

Closes:  #103

Title : Change Style of Scrollbar
Name:Zala Bhavy
Idenitfy yourself: Gssoc'24

Describe the add-ons or changes you've made 📃
I have changed in file index.css
Added Stylish Scrollbar instead of default scrollbar.
I added scrollbar with different blue shades.
Type of change ☑

What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: ☑

[x] My code follows the Contributing Guidelines & Code of Conduct of this project.
[x] This PR does not contain plagiarized content.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] My changes generate no new warnings.

Screenshots 📷

@Avdhesh-Varshney Please check it and merge !!
Can you also assign me issue #106

@Avdhesh-Varshney Done please check it !


Add Jaeger tracing to distributor

As a first step, add tracing to distributor and lite's distributor part.
In some instances distributor suffers from high latency and this change will help highlight where the time is spent.
Tracing is disabled by default and can be enabled by setting the JAEGER_AGENT_HOST environment variable to point to a jaeger agent.

@jml I'll move to weaveworks/common once proved.

Part of weaveworks/service-conf#1512.


Update validation for EventGrid output binding and add live tests

This PR updates the validation when using the new Connection property.
We will now ensure that all settings are specified in either the Connection OR the TopicEndpointUri/TopicKeySetting properties, but NOT both. This means that in order to use Identity, you need to use the Connection property.
/cc @andrewjw1995
Event Grid test recordings haven't been migrated to assets repo yet - this will be done in a separate PR.

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

/azp run net - eventgrid - tests


Swap out author on topics

Would it be ok to swap me (guardrex) out for one of the regular authors on the repo in author metadata? It will help me with the ping load. I can do the PR if you let me know who the replacement author is.

Yes, it would be OK, @guardrex. You can assign the .NET Core topics to @mairaw and the .NET Framework topics to @rpetrusha.

Thanks @rpetrusha ... Will do.


Allow Multiple Syncs for Timer

Currently the timer won't work if you sync more than once

 Allow Multiple Syncs for Timer

This could potentially be an optimization bug in gcc.

Issue has been fixed


Follow-the-path task

The follow the path task including:

Example world
Practice worlds
We can reuse the navigation scoring plugin

How to test it?

Launch the simulation:

ros2 launch vrx_gz competition.launch.py world:=follow_path_task

Launch the joystick teleoperation node:

ros2 launch vrx_gz usv_joy_teleop.py

Teleoperate the robot through the course. The task should successfully finish after crossing the last gate. Crossing the gates in the wrong error should also trigger the task to finish (unsuccessfully).

I'm getting expected behavior from follow_path_task and practice_2022_follow_path0_task, which both use the same short_navigation_course_0. When I try to run practice_2022_follow_path1_task and practice_2022_follow_path2_task, I don't get any notification when I pass through gates and I'm unable to complete the task.

This looks like it's working but I think we should add more gates for runs 1 and 2. The reason is that we're reverting to a task more like the one we had in VRX 2019, which specified 2-10 gates, rather than the 2-5 for the gymkhana navigation channel. To set expectations correctly, I think it would be good to have 3, 6, and 9-10 gates for practice worlds 0, 1, and 2, respectively.
Also, in world 2 I notice the buoys in the second gate are very far apart. This may be fine if they are within 25 meters, but I thought it might be worth double checking.

All right, I just modified the navigation courses to have 3, 6and9` gates. I also added extra obstacles.

Courses 1 and 2 don't score past the third gate. I think because the scoring plugin initialization hasn't been updated to reflect the new models.

OK, I added those gates and it looks like it's working now.

OK, I added those gates and it looks like it's working now.

Thanks!


Decouple inline-styles from Dynamic Style Sheets

Using Global which extends CSSSheet for global styles is quite handsome, but inline-styles got no advantage of using Sheet anymore.

no longer depending on Sheet


add basic OpenSUSE workflow

Changes proposed in this pull request:

basic OpenSUSE build workflow

@andrewfer000 , can you have a look ?

It looks good but I would we need to verify that it works. I am new to openSUSE myself so I cannot be 100% sure. If this works for you then the best thing we can do is merge it and see. It's not like it can do much harm.
Also, I'd prefer to use ninja-make instead of regular make and I'm sure @davidebeatrici feels the same. It's also the only way I got wireguard to work so to be sure this script does not need changing maybe we should use ninja instead (see the bottom of #1200 for more detail)


Close tab keybind

Hello,
I've added a key to close tabs. It is useful together with recency sort to clean up excursions quickly.
I've chosen w for parity with Ctrl+w.
Cheers.

Thanks for this PR!
Right now, there's no visual feedback in the popup when you close a tab. The simplest way would probably be to just remove the active element from the DOM and TabList, and set the next one down to be active.  Another way would be to mark the tab as dead (though you'd need to look up the new ID once it's closed, as open tabs don't have unique IDs from recently closed tabs)
If you'd like to add that on this PR, then go ahead! Otherwise, I can do it :)
Also, make sure it builds correctly! In the keypress check, you accidentally put a [ instead of {. The command for building is npm run build:development or npm run build:production. I was going to point you to the build instructions, but I just realized that I listed the wrong command in there haha. I'll fix that tomorrow.

If you'd like to add that on this PR, then go ahead! Otherwise, I can do it :)

Please do, I've tried it reading your code and MDN but I didn't figure out how to do so. I'll try to do it for my own edification with the idea you put forth but I'd rather that not hold this back.

Also, make sure it builds correctly! In the keypress check, you accidentally put a [ instead of {. The command for building is npm run build:development or npm run build:production. I was going to point you to the build instructions, but I just realized that I listed the wrong command in there haha. I'll fix that tomorrow.

Your build instructions served well, I've been using this close tab button for a few weeks. Unfortunately I amateurishly fumbled interfacing with GitHub from my git setup and I ended up making the changes on the web editor out of late night frustration.
I hoped to be a bit more proactive but I didn't spare you any work, sorry about that 😄

Something that could be improved still is the behavior when closing the tab you're in. I think it would be preferable to keep the pop-up in focus, at the moment focus is lost as Firefox switches to the previous tab, although the popup remains open. Alternatively the popup can simply close.

There is no way for an out of focus window to regain focus, without the user specifically giving it (i.e. clicking or tabbing in). Closing or changing the selected tab will cause the popup to lose focus, and AFAIK there's nothing that can be done to prevent that. I think closing the popup after closing the selected tab is probably the best option.

Your build instructions served well, I've been using this close tab button for a few weeks. Unfortunately I amateurishly fumbled interfacing with GitHub from my git setup and I ended up making the changes on the web editor out of late night frustration. I haven't used git forges much. I hoped to be a bit more proactive but I didn't spare you any work, sorry about that 😄

Ah, it's alright. It was a pretty easy error to find and fix. It just made me a little worried that you didn't figure out how to build it haha. Glad to see I was wrong about that :)

Please do, I've tried it reading your code and MDN but I didn't figure out how to do so. I'll try to do it for my own edification with the idea you put forth but I'd rather that not hold this back.

Alright! I'll keep this PR open for now and add the relevant changes here.

Sounds good, I've been using it since  your commits without any bugs.
I appreciate it, thank you.
On November 7, 2023 4:10:05 PM UTC, Joe Jarvis @.***> wrote:

OK, I think this feature is basically done, so I'm going to merge this now.
--
Reply to this email directly or view it on GitHub:
https://github.com/Rayquaza01/quick-tab-switch/pull/23#issuecomment-1799049337
You are receiving this because you authored the thread.
Message ID: @.***>

@deadcombo Version 1.3.0 with the close tab keybind is now live on AMO.

@deadcombo Version 1.3.0 with the close tab keybind is now live on AMO.

I noticed, thank you! Happy holidays and thanks for making my browser better!


Add position for switch and checkbox

Add right position possibility for the label of switch and checkbox widgets
New parameter labelPosition="left". Default position still labelPosition="right" for these widgets

As discussed, apply same solution as switch for checkbox

It's done!

Thanks Ghiles. It's validated.
Can you please merge this feature both with main and integration branches ?


Proposal: Stepper Refactor

We have almost refactored everything we need to start implementing the behavior tree proposal. The biggest hurdle we have left (that I know of) is the current stepper.
Right now the stepper keeps track of the spiral it needs to walk, and moves towards the center of each part the spiral, and then calls work_on_cell for each cell. In order for it to be compatible with an event loop style system (and thus the behavior trees), is for it to take a single step in the correct direction and update some piece of memory with the current cell, and then have work_on_cell run after the stepper and process that cell.
Essentially what that would require is for the stepper to not walk all the way to its goal but to instead take a certain number of steps towards that goal and then return control to the rest of the program.
More concretely, I propose we split the current stepper into two parts:

Utility class walk_towards
Spiral Walker

The walk_towards class should keep track of where it was initialized, where it's end goal is and take steps in that direction every time a function like take_step is called. This same api would be able to be implemented with the current walk_towards, as well as a more complicated approach like the Polyline one in #697
The spiral walker would keep track of where it's center point is, and on every tick, do nothing if walk_towards hasn't finished, and if it has, call walk_towards to the next point. This would enable us to have different walkers. Such as one that walked to specified hotspots, or walked from LA to NY and back.
Any thoughts?

The walker is hardly extenable. I agree the refactor. We may need more input or start ?

just to be clear, Spiral Walker is what's actually controlling the walk patterns/logic, in this case? walk_towards is just a dummy "I'm here, I'm told to go there, I go there" class?

I'm not well-versed enough in python to know off the top of my head, but would it be a good idea/possible to implement a factory pattern to enable custom class creation based off an inheritance heirarchy that has the "walk_towards" functionality baked in? then someone could config their bot to load the default walker, or someone could create a new walker class and config the bot to load that?

@reddivision That is correct.
The biggest change of this proposal though isn't the splitting apart of those two things. The biggest change is that the stepper won't walk all the way to the end goal on each call. It will be called more frequently and be expected to take a few steps in the correct direction before returning control to the rest of the program.
Right now, the bot essentially says this:
while true: #right now this loop ticks every 10 minutes or so as a spiral is finished
  walk a spiral calling clear_inventory, evolve_pokemon, spin_forts, catch_pokemon on each step

and we will be turning it into this:
while true: # now this loop will tick every few seconds
  take a few steps in the intended direction along the spiral
  clear the inventory
  evolve pokemon
  spin forts
  catch pokemon

@reddivision The approach we will be taking to let people configure different types of walkers they want to use is outlined in the behavior tree proposal: #142

Can I suggest we use the tsp package for this https://pypi.python.org/pypi/tsp
From our starting location (wherever the bot starts) we should search the map around us (based on max steps) to find all the pokestops with the distance limit
then visit each one based on the solve() method of tsp

@MikeDX I think something like tsp is good, but out of scope for this refactor. This refactor would enable us to replace the spiral walker with something that uses tsp, and with this refactor would have no impact on any other parts.
The ideal outcome of this refactor is to have no behavior change, but set ourselves up for better behavior modifications after.

Sorry, I had assumed foolishly of course that the walk_towards was already independent and could read the 'next stop' from the tsp.solve() data.
I'll let you guys handle this refactor, whilst I go see if i can fix this blasted login expiry bug :)

Reading through the items in the other proposal, but, ultimately, I believe we're aiming for the same thing, so +1 from me.

One thing I would love to see is the ability to import a list of GPS coordinates to be used as waypoints and I see you kind of have that proposed in #142. I live in a city with a lot of lakes that have paths around them and the current walking system is terrible near water.

@MEGAMERICAN That's the plan. Instead of a SpiralWalker we can have a WaypointWalker or a DestinationWalker or a WanderWalker. The code for that will be able to be completely separate from everything else.


[v2] module vs modular builds

This is a (multiple allowed):

[x] bug

[ ] enhancement

[x] feature-discussion (RFC)

Framework7 Version: v2 branch from 04/10/17

Platform and Target: web

What you did
cloned v2 branch
npm run build-prod
Actual Behavior
3 js files were created in dist folder
framework7.js
framework7.module.js
framework7.modular.js
The framework7.js build file is the traditional bundling all together. All ok
The framework7.module.js build imports dom7 and template7 and bundles by default all components. ES6 code is transpiled.
The framework7.modular.js build imports dom7 and template7 and bundles only core components and exports the remaining separately. ES6 code is not transpiled.
In package.json sets module and jsnext:main fields to framework7.module.js
When using webpack or rollup framework7.module.js will be used but will not enable treeshaking (do not bundle not used code)
To enable treeshaking is necessary to import directly framework7.modular.js but this build does not transpiles ES6 code. The issue here is that the common setup of webpack / rollup does not transpiles code from node_modules requiring an extra configuration.
Expected Behavior
framework7.modular.js should keep the current ES import/export configuration but transpile the remaining code. For reference see: https://github.com/rollup/rollup/wiki/pkg.module
Also consider renaming framework7.modular.js to framework7.esm.js as other libraries are doing

Hi,
I am ok with renaming es packages with .esm prefix. But not about transpiling. I have tested a lot treeshaking with F7, and the best results we can got is having it in ES-next syntax. I was explained a bit here https://github.com/nolimits4web/Template7/pull/38 why it is happening. And with even more complex classes in F7 it almost doesn't work at all. So probably later when treeshaking algorithms of bundlers will be improved i will be fine with transpilation.

Ok. I think is necessary to document this. Is there something already?
If not, just tell me the appropriate place in docs and i can help with a PR

Yes, need to be documented, but there is no v2 docs yet, so there is no place to put it currently :)

Issue is closed because of outdated, irrelevant or not actual
If this issue is still actual and reproducible for latest version of Framework7, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Swining between Upgrade to 1.37 and Downgrad to 1.36

Issue Type: Bug
The issue is:
VSCode requests an update to version 1.37 and a few minutes later, it requests an update (it is actually a downgrade) to version 1.36
This is happening more the 5 times to me.
VS Code version: Code - Insiders 1.36.0-insider (0f3794b38477eea13fb47fbe15a42798e6129338, 2019-07-03T08:00:29.662Z)
OS version: Windows_NT x64 10.0.18922

I can confirm this is happening here as well. This same behavior has happened on multiple machines.

This was due to build problems we has. Should be solved now.


Add workshop notebook for TIG v2

Issue #, if available:
Description of changes:
Adding workshop notebook for TIG v2
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Do we need ALL those images in the images subfolder?
Before we output the image, can we show what the "input" image is? (basically plot the reference image)
Can we show the hex colors for color conditioning first before we show the output?
reference image and processed image for the second hex code example is very different
background removal looks good.

Overall comment: this still fits nicely in bedrock samples, but we should discuss and talk about merging the v1 and v2 notebook and talking about a marketing use case end to end for example

Nice! How about using the same modules you already have like this?

view reference image
generate cartoon image
place cartoon image on packet of dog food
Change color palette
Change the background
Bonus task: remove the background and provide just the packet for future work

Thank you for your patience and for all the changes!! Merging


Checkout-app is not fetching channels

It looks like https://github.com/saleor/react-storefront/commit/1134c729316f5d1e1c5519fce324ca2225178c59 broke appBridge integration and isAuthorized is never set to true.

Thanks for fixing this -- I will try it this weekend.  This is what I was referring to; I must have had something cached when it was "still working" again, but later that day I couldn't get it to work any longer and switched back to an older commit.


avoid infinite loop in EventBased splitting

see https://github.com/dmwm/CRABServer/issues/5599
this loop in EventBased.py will go into infinte loop if eventsPerJob is not strictly positive.
We will put a check in CRAB client, but it may be a good idea to protect the code here as well.
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/JobSplitting/EventBased.py#L148-L183

I think such protection should be placed way up in the code, such that a "sick" workflow never gets to the job splitting.
In central production services, we already have this validation in place:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WMSpec/StdSpecs/StdBase.py#L1201
Of course nothing refrain us from adding another check here... but if CRAB can have such validation during request creation, that's the cleaner option IMO.

I have put a protection in CRAB code (see the issue mentioned initially).
I surely agree that bad inputs should not be discovered so late in the processing. Whether an inner call should go into infinite loop or raise exception in case of problems is maybe a matter of tastel. In general sanitizing inputs is a good practice since there can be simply good honest (new) bugs upstream and the easier we make spotting them, the better for us. Of course there's a judgement call to make as to what is good protection and what's useless paranoia. This repository developers are surely more qualified than me to make that call.
And thinking about it, maybe we should put a second check in CRAB sort of mid-way.

Stefano, since we only call the job splitting algorithms when there is actual work to be splitted, I think it's Ok to add another check at that layer. PR 8290 was created and should go in in 1.1.8.pre5 tag


ClearLinux version should be specified for the docker image

The docker image has been changed to ClearLinux. However, later steps in the build use "swupd update", which try to update the image to the latest version. This is not only inefficient but also may introduce unnecessary instability. Since Clear Linux repository servers are slow, it may take twenty minutes just to download all packages for updating. Besides, Clear Linux itself sometimes contains unexpected changes that can have deprecations or bugs. I would suggest to specify a "known-to-work" version of the Clear Linux instead of using the latest one.

We have to use the latest Clear Linux, including all updates, when building a new image. Otherwise we build an image that may lack bug fixes. Manually tracking which Clear Linux release has no known bugs isn't feasible.
Yes, it may also have regressions. That's something that will have to be detected when testing that new image before promoting the image as the next image to be used in production.
Right now we don't have any of that, though, and just always build "the latest".

Same approach was discussed recently in connection to strategy selection for building published images.

PMEM-CSI releases use a fixed version of the Clear Linux base.

Thank you @pohly


Make declarative and imperative schema declaration result in the same type

Thanks for your work on this!
Currently, using the imperative DataFrameSchema API results in a different type of "schema" object as compared to if you create that same schema using the declarative class-based Schema Model API. This is a bit confusing once you have constructed your schema, because the two types have slightly different APIs. Also, you construct them slightly differently, using pa.Field for the declarative model and pa.Column for the imperative model.
Describe the solution you'd like
Ideally these two construction APIs would result in the same type of schema. Also, ideally you would construct them more consistently (eg using pa.Column in both).
It looks to me (though I'm not super familiar with it) that sqlalchemy has solved this problem already. See the Declarative and the Imperative style sections in this documentation. Perhaps we can use their implementation as inspiration?
Additional context
I realize this is very high-level and not very actionable at this point, but I just wanted to get this idea on your radar. The more pandera matures and more features are added, this will get harder to change. It already might be at the stage where this sort of change would be too disruptive.

hi @NickCrews, thanks for validating my own thoughts on this!
See this discussion for more context.

See the Declarative and the Imperative style sections in this documentation. Perhaps we can use their implementation as inspiration?

Looks like the link is dead.
At a high level I totally agree, although the direction I'm thinking about is going for a more abstract way of representing data containers and their components. This is because pandera is slated to support not just dataframes, but other objects, which for now I'll call "scientific and analytics data containers", i.e. data structures for data scientists/engineers/analysts, ML engineers (e.g. dataframes, xarray.Dataset, tensors, ndarrays, etc). that have some statistical capabilities associated with them.
Currently I feel the most appropriate abstraction is "Container" and "Fields", so for example,

For pandas, the "container" is a DataFrame, and the "fields" are Series (columns), Index, and Multiindex.
For xarray, the "container" is a Dataset, and the "fields" are DataArrays, coords, and attrs.
For sql, the "container" is a Table, and the fields are Column
etc.

Currently, using the imperative DataFrameSchema API results in a different type of "schema" object as compared to if you create that same schema using the declarative class-based Schema Model API

Just so I understand, why is having different types for each kind of API important? I get the point of a similar construction syntax being good for UX, but for this point:

This is a bit confusing once you have constructed your schema, because the two types have slightly different APIs

Are you referring to how you construct the schema, or how you modify that schema? Seems like it would be hard to get away from this, as the two APIs serve two different purposes/programming preferences... the DataFrameSchema is better for inline validation while SchemaModel is better for people who care about python type annotations.

Thanks @cosmicBboy! Fixed my link, and replied in the discussion that you linked.

Also, the abstraction of container and field makes sense.
I would add (spitballing totally from the armchair, feel free to totally ignore) that I might split up the concerns a bit:

A core schema, which deals with validation (both statistical and otherwise).
A generator add on which can take a schema and produce examples based off of it. I as a user don't care about about this functionality and I think it clutters up the rest of the validation API, and probably makes it harder to maintain.
Similarly, split the input/output conversion into a separate, higher level layer than the core schema.


[ISSUE-2851][improvement] improvement flink cluster status update

What changes were proposed in this pull request
Issue Number: close https://github.com/apache/incubator-streampark/issues/2851

Brief change log
Optimize the status update of the flink cluster, as detailed in:https://github.com/apache/incubator-streampark/issues/2851
Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (no)

cc @wolfboys @RocMarshal  PTAL.


added table of contents section

This MR adds a table of contents section with links to each section

Thanks


map the basic features of a meeting schedule

prefix and:

title
description
date and hour
place

prefix:  /schedule
title: $title
description: $desc
date: $date

dd/mm/yy
dd/mm
dd/mm/yyyy
weekday (closest one exclude today)
today

hour: $hour

hh:mm
hhmm


Undefined behaviour / program freeze while accessing first token

This is my code:
#define JSMN_STATIC
#include "jsmn/jsmn.h"

-- SNIP --

		debugPrintf("First 3 characters of RAM file: %c%c%c", ramFile[0], ramFile[1], ramFile[3]);
		debugPrintf("Last 3 characters of RAM file: %c%c%c", ramFile[ramIndex - 2], ramFile[ramIndex - 1], ramFile[ramIndex]);
		debugPrintf("Parsing keys.json...");

		jsmn_parser parser;
		jsmn_init(&parser);
		debugPrintf("Parser initialized!");
		int tn = jsmn_parse(&parser, ramFile, ramIndex, NULL, 0);
		//TODO: Error checking
		debugPrintf("Found %d JSON tokens", tn);
		jsmntok_t tokens[tn];
		jsmn_init(&parser);
		debugPrintf("Parser reinitialized!");
		if(jsmn_parse(&parser, ramFile, ramIndex, tokens, tn) != tn)
		{
			debugPrintf("Parsing error 1");
			//TODO
		}
		else
			debugPrintf("Parser reinitialized!");

		if(tokens[0].type != JSMN_ARRAY)
		{
			debugPrintf("Parsing error 2: %d", tokens[0].type);
			//TODO
		}
		else
			debugPrintf("JSON Array found!");

And this is the result of the debugging prints:
First 3 characters of RAM file: [{t
Last 3 characters of RAM file: "}]
Parsing keys.json...
Parser initialized!
Found 39832 JSON tokens
Parser reinitialized!

So it freezes at the line if(tokens[0].type != JSMN_ARRAY)  which should have been initialized by JSMN.
Please note that this is on an architecture which does not guarantee any memory to be zero allocated.
//EDIT: Was too fast. New code:
		debugPrintf("First 5 characters of RAM file: %c%c%c%c%c", ramFile[0], ramFile[1], ramFile[2], ramFile[3], ramFile[4]);
		debugPrintf("Last 5 characters of RAM file: %c%c%c%c%c", ramFile[ramIndex - 4], ramFile[ramIndex - 3], ramFile[ramIndex - 2], ramFile[ramIndex - 1], ramFile[ramIndex]);
		debugPrintf("Parsing keys.json...");

		jsmn_parser parser;
		jsmn_init(&parser);
		debugPrintf("Parser initialized!");
		int tn = jsmn_parse(&parser, ramFile, ramIndex, NULL, 0);
		//TODO: Error checking
		debugPrintf("Found %d JSON tokens", tn);
		jsmntok_t tokens[tn];
		jsmn_init(&parser);
		debugPrintf("Parser reinitialized!");
		if(jsmn_parse(&parser, ramFile, ramIndex, tokens, tn) != tn)
		{
			debugPrintf("Parsing error 1");
			//TODO
		}
		else
			debugPrintf("Tokens initialized!");

		debugPrintf("Examining tokens[0]:");
		debugPrintf("\ttype = %#010x", tokens[0].type);
		debugPrintf("\tstart = %#010x", tokens[0].start);
		debugPrintf("\tend = %#010x", tokens[0].end);
		debugPrintf("\tsize = %#010x", tokens[0].size);

Debugging output:
First 5 characters of RAM file: [{"ti
Last 5 characters of RAM file: 00"}]
Parsing keys.json...
Parser initialized!
Found 39832 JSON tokens
Spoofing tokens[0]...
Parser reinitialized!

So it's freezing at if(jsmn_parse(&parser, ramFile, ramIndex, tokens, tn) != tn).
Please note that the ramFile string is not terminated by a '\0' char. Could this cause the issue?
//EDIT²: Yes, fixed by adding \0 to the end. Not sure if you want to fix this (we pass the size of the JSON string to JSMN, so why doesn't it use that info?),

First thing, can you revert any changes and test with #define JSMN_STRICT ?
The non-strict mode, sadly the default for now, doesn't really work well and is planned to be removed. Have a feeling it might be causing the problems.

I tried as you suggested but the error is still the same.
If you need to know what data I'm trying to parse, it's this: http://cldr.xyz/json

I have tested your case with my eyes and valgrind and seems like the problem might be at some other point in your code as it doesn't seem to be in here. You should look around for possible memory corruption before this piece of code.

I will close this issue for now but feel free to reopen if you believe the problem is definitely with JSMN.

That JSON parsing is almost the first thing happening in the app. It only initializes a few things, reads a json config file, downloads the json file from the link above to ram and parses it. Also porting to another json library fixed the problem for me, so I'm really not sure what happened here.
Anyway, thanks for your support. I'll check JSMN out again at a later point in time (maybe I'm really overlooking some corruption but I searched for it the whole day before making this report... But I also need to inform myself how to use i.e. Valgrind on the platform I'm developing for).


Add AffinityRequired test to lifecycle

Adds a test to the lifecycle suite that checks for a label AffinityRequired: true to be supplied for workloads that require colocation of pods.

Tests pods, deployments, and statefulsets.
Adds "groupings" section(s) into the provider.go.
Adds AffinityRequired() and IsAffinityCompliant() funcs to each of the above structs.
Had to modify testPodNodeSelectorAndAffinityBestPractices to avoid pods that have AffinityRequired flags.  This func was already looping through NonGuaranteedPods.  Maybe there is a way to do cross-sections of different lists of pods we could implement later.
The IsAffinityCompliant tests are all relatively the same with some copy/pasting.

Hi @sebrandon1 , just to let you know, I've added this in the PR description: build-depends: 26152, in order to test this change with https://softwarefactory-project.io/r/c/dci-openshift-app-agent/+/26152, so that I can validate that, by including hugepages in our workloads, the test passes

Hi again @sebrandon1 , I've been doing some tests and I have a main question here: isn't this new test colliding with lifecycle-pod-high-availability workflow?
Because, in theory, in order to pass lifecycle-pod-high-availability test suite, apart from having more than 1 replica in the deployment/statefulset, it is also needed to have defined podAntiAffinity rules. However, in this new test suite, even though you set the AffinityRequired label and you have podAffinity rules defined, if a podAntiAffinity rule is found, then it doesn't pass the test.
Does it mean that both lifecycle-pod-high-availability and lifecycle-affinity-required cannot pass simultaneously?

@ramperher Thanks for the comment.  I forgot to change that test to look at the deployments/statefulsets that do not have the AffinityRequired flag only.  See this commit: https://github.com/test-network-function/cnf-certification-test/pull/523/commits/6f54cd25e170555650c5a6a5ac168272ee1e3ecc

@ramperher Thanks for the comment. I forgot to change that test to look at the deployments/statefulsets that do not have the AffinityRequired flag only. See this commit: 6f54cd2

Cool! Thanks for the update! I'll take a look tomorrow to test both cases then! Don't pay attention if there's any failure in today's jobs, they may fail because of the new affiliated-certification test, by the way


Install qiskit in ci env

Summary
This commit ensures that we're actually installing qiskit in the travis
environment. Previously we were just running tests out of the qiskit
repo without installing it. This made things like testing jupyter or
external commands more difficult because we had to properly setup the
path to make sure the qiskit code was discoverable. This fixes that by
installing qiskit (in develop mode) so that the code is linked to the
python site-packages and is discoverable/importable by other python
packages on the system, regardless of where they're run from. This is
just a stopgap patch until we switch to using tox for ci, which will do
this for us as well.
Details and comments

Labelling as on hold until after 0.7, as part of the CI discussion.


[Test-Proxy] Create a CI Analyze step for verifying UA header in recordings

Currently one of our preflight checklist items is to verify the User Agent header is correctly formatted.
We could add a step the Analyze pipeline to use the test recordings to verify the UA header is accurate.
Here's an example of an incorrect header
This step should probably not enforce the most up-to-date package version in the UA header, or it will force recordings to be re-recorded with every new version. Which ideally would be nice, is probably not practically realistic for all SDKs.

@annatisch what is in the example of an incorrect header that makes it incorrect?


Do we need minDelay & maxDelay?

What are their use-cases?
minDelay - for periodic syncs I think the minPeriod would be the min delay. Is there a case you'd want them to be different?
maxDelay - given https://github.com/slightlyoff/BackgroundSync/issues/35#issuecomment-86940721, I don't think we'd want to allow this.

These are the parameters that differentiate between, "sync runs whenever the hell the UA wants" and "sync runs when the user/developer wants". I think for a first (MVP) draft we'll stick with the former. But it really does hamper use cases.
One use of maxDelay is to allow a timer to start conservative but get more aggressive over time. E.g. I'll try waiting for charging on wifi for two hours, but after that point I'll settle for charging and cellular, and after that I'll take anything because the data is getting stale.
The purpose of minDelay is to set a start time. E.g., the user would like to sync around 5am every morning before getting on the train so set minDelay to a value that reaches 4am and minPeriod to 24 hours.

Note that min and max delay come from Android's [JobScheduler](https://developer.android.com/reference/android/app/job/JobInfo.Builder.html#JobInfo.Builder(int, android.content.ComponentName)).

Not in the spec.


Add config profile support

This adds the (optional) feature to be able to select different configuration profiles based off of a name (not a path). For instance:
myapp --profile x

would look for .myapp/x.yaml to load the app configuration from.
This also makes a breaking change where finders can return errors. This is also vital for ensuring that when -c FILE is specified that we will not use the default configuration (which seems to be a bug today).
This is a proof of concept and is not meant to be merged as is.

@kzantow and I discussed offline and think there is a better approach here


Notebook #2: Issue getting data from thredds server

thredds_server = "https://thredds.nci.org.au/thredds/dodsC/fs38/publications/"

#Temp at surf
ver = "CMIP6/CMIP/CSIRO/ACCESS-ESM1-5/historical/"
fpath = "r10i1p1f1/day/tas/gn/latest/tas_day_ACCESS-ESM1-5_historical_r10i1p1f1_gn_20000101-20141231.nc"
var = 'tas'
url = f"{thredds_server}{ver}{fpath}"

print(f'{url}.html')

ds = xr.open_dataset(url)

returns:
```OSError: [Errno -68] NetCDF: I/O failure: 'https://thredds.nci.org.au/thredds/dodsC/fs38/publications/CMIP6/CMIP/CSIRO/ACCESS-ESM1-5/historical/r10i1p1f1/day/tas/gn/latest/tas_day_ACCESS-ESM1-5_historical_r10i1p1f1_gn_20000101-20141231.nc'``
direct download:
Error {
    code = 400;
    message = "Unrecognized request";
};

https://thredds.nci.org.au/thredds/dodsC/fs38/publications/CMIP6/CMIP/CSIRO/ACCESS-ESM1-5/historical/r10i1p1f1/day/tas/gn/latest/tas_day_ACCESS-ESM1-5_historical_r10i1p1f1_gn_20000101-20141231.nc.html
This looks fine to me and above code runs, perhaps it was down when you tried??

Yes, it works fine for me now.


No explanation of what change when something is EOL on the web page

So, while looking on https://www.gluster.org/release-schedule/ , I started to wonder what EOL mean, and there is no explanation on it anywehre.
For example, does that mean we will remove content from the download server, does that mean that all bugs are gonna be closed and/or considered, that documentation is gonna be cleaned up ?
As we have people coming on IRC with EOL version, a clear text to set expectation and to be able to give to people would greatly help on that front.

Adding@ShyamsundarR @vbellur -  I don't think we have a process currently written down for 'what happens to deprecated versions'. And we should!

Adding to TLC agenda for week of June 11th.

Resolved with new release schedule.


TableRelation = too much indention

Love this little extension, using it with a TableRelation with if's and where on different lines cause the following lines to indent.

Also CONST + FILTER is not changed to lower case.

I'm looking into it, thanks for notifying.

@epernst I'll put a new version up tomorrow, which solves both this issue and #3 .
Thanks again for helping me out :)

Has not been fixed. In fact it seems it has gotten worse.
Try to run it with this code:
`trigger OnValidate();
begin
if true = false then
Status := 1;
if true and
(Status = 2)
then begin
Status := 3
end else
if true and
(Status = 2)
else
if true and
(Status = 2) then
Status := 3
end;
`


Revert Delivery Date Feature

It reverts to the previous commit made by the repo owner. This undoes the feature- 'delivery-date' added in the previous commit.

Reverted to previous app.py and orders.html code.


Add FAROUT token

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
J5gLhk6mmQ4PSoir1Ufh8JY2ytEHA93YupzYiTFVCgcL
Token Name:
Far-Out
Token Symbol :
FAROUT
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
assets/mainnet/J5gLhk6mmQ4PSoir1Ufh8JY2ytEHA93YupzYiTFVCgcL/logo.png
Link to the official homepage of token:
http://far-out.biz
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):
NA

Have I missed something ?

@rmshea please review. Waiting for this pull request to open market on serum and start operations. Thank you !

nothign happening creating a new one !


fix(TDI-39742):tNetsuiteInput cannot retrieve Custom fields of

VendorBill Record
What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-39742
What is the new behavior?
Please check if the PR fulfills these requirements

[ ] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:
Sonar results


Chat bubble doubling when swiping right to go back to the main view.

Please check this video, It's all here. https://www.youtube.com/watch?v=nY-Mgw5FtxU&feature=youtu.be

@jheazzzar Can you post the code you wrote how you add message data? What I can think of is the logic of adding message data sits at either viewWillAppear: or viewDidAppear:. That's how you can this strange behavior. Let us know. 😄

I referred to this tutorial by David East from firebase.
https://www.raywenderlich.com/122148/firebase-tutorial-real-time-chat


Calibre interference

Hello.
I recently installed Calibre in my system and noticed that whenever I try to view reanimate example animations with stack I get a new window with a Calibre EPUB viewer, and a pop-up window asking me to import an HTML or DOCX file as an EPUB, instead of the usual browser tab -and the preview is nowhere to be seen.
Here is an image of what happens when I try to view an example animation with Calibre installed:

Uninstalling Calibre fixes the issue, but I'd like to be able to keep that program and view animation previews in-browser. I'm somewhat new to Haskell, but if anyone considers this a "good first issue" and can give me any pointers, I'd greatly appreciate it, since I'd like to try helping out.
Cheers.

It looks like Callibre has been set up as your default app for HTML documents. If you're using Ubuntu, try going into Settings -> Applications and then tell Callibre to not handle HTML files.

That's exactly it. I fixed it by setting Firefox as my default app for HTML documents.
For further reference, I did this in Arch by installing perl-file-mimeinfo, downloading an html file (named, for example, myhtmlfile.html), running the command mimeopen -d myhtmlfile.html in the download directory, and then choosing Firefox on prompt.
Thank you for the quick response! This issue has been resolved.


help: Document "Show unread count for" display preference.

Adds a new page in the "Reading messages" section to document this new feature (#24237).

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability

Communicate decisions, questions, and potential concerns.

[x] Automated tests.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Links.

@alya This PR is ready for review.
@zulipbot add "help center review"

Let's stick with the convention of not listing options in the instructions. We can use the same formula as https://zulip.com/help/change-the-time-format.

(I'll also take a revision pass on the intro paragraph when I get a chance.)

Could you please include a screenshot for the left sidebar?

Let's drop "for streams" from the name and URL of this page. If we ever make DM unread counters configurable, we'll very likely want to document that on the same page in any case.

Sounds good! I updated the PR and screenshot.

We can just explain that it only applies to streams in the body of the page.

I'm not sure where to add an explanation saying that this feature/setting only applies to streams. Maybe there's no need to explain this since the text is already focused on streams. What do you think?

Yeah, I think it's clear enough as-is.
I left a tiny comment above -- please go ahead and mark the PR for integration review when you've made the tweak. Thanks!

@zulipbot remove "help center review"
@zulipbot add integration review

@zulipbot add "integration review"

Merged, thanks @drrosa!


Add config to error object.

Would you be open to providing the config object along with the error?
Makes things consistent with other rejection cases.
Fixes https://github.com/mzabriskie/axios/issues/315, https://github.com/mzabriskie/axios/issues/242

Coverage remained the same at 91.304% when pulling 72829cf94c0a3bcfca7ad29c2e091e8e1ec2ac72 on azizhk:patch-1 into c311466a2c1ad9afa1bced687db44d8aa3bd4ffd on mzabriskie:master.

I have no idea why "eval npm install " is failing in the travis build.

This has been resolved with #345.


I can't play music D=

Whenever I click the play button it won't play at all

Firstly, Commenting here, On the forums and On CurseForge within 13 hours, isn't the best to do, and as well as this, since you have provided me with little to no information, I must request logs, since, it might actually be some other issue which could be detailed in the forge log files.

Due to lack of response, I am closing this as invalid.


Preflight Checklist

[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.
[X] I am not looking for support or already pursued the available support channels without success.

Version
1.0.0
Installation Type
Official Docker Compose
Service Name
Deploy(DongTai-deploy)
Describe the details of the bug and the steps to reproduce it

Additional Information
No response
Logs
No response

fixed in v1.1.0


Minimal work to support the new NPM protocol for package search.

This is a minimal change to the interaction with the NPM registry, instead of using a local cache this queries the NPM registry directly.
Only removed the minimum amount of code to clean the UI up.
If the network is unavailable the UI will keep spinning on the search screen, but not crash VS. The user is still able to close the UI manually.
Note: the refresh button is removed.

Looks like tests are failing the build. I know the tests aren't much use, but hopefully is a simple fix.

Hello,
I'm experiencing the same problem.  Is the a fix or work around?

@josephdubeau This is fixed in 1.3 for Visual Studio 2015, and Update 3 for Visual Studio 2017. Otherwise you can use npm from the command line to add packages to the package.config and/or install packages.

Thank you so much, I will try the update first.

From: Paul van Brenk<EMAIL_ADDRESS>Sent: Monday, July 24, 2017 10:18 AM
To: Microsoft/nodejstools
Cc: josephdubeau; Mention
Subject: Re: [Microsoft/nodejstools] Minimal work to support the new NPM protocol for package search. (#1558)
@josephdubeauhttps://github.com/josephdubeau This is fixed in 1.3 for Visual Studio 2015, and Update 3 for Visual Studio 2017. Otherwise you can use npm from the command line to add packages to the package.config and/or install packages.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/nodejstools/pull/1558#issuecomment-317492418, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFIr5YGgk_7bwYXBKQizeKmSnB9kaLWDks5sRNH7gaJpZM4NRYG2.

I able to do a command line update but not in VS 2017.  I have version 15.2 (26430.16)
Where can I download updates?

From: Paul van Brenk<EMAIL_ADDRESS>Sent: Monday, July 24, 2017 10:18 AM
To: Microsoft/nodejstools
Cc: josephdubeau; Mention
Subject: Re: [Microsoft/nodejstools] Minimal work to support the new NPM protocol for package search. (#1558)
@josephdubeauhttps://github.com/josephdubeau This is fixed in 1.3 for Visual Studio 2015, and Update 3 for Visual Studio 2017. Otherwise you can use npm from the command line to add packages to the package.config and/or install packages.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/nodejstools/pull/1558#issuecomment-317492418, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFIr5YGgk_7bwYXBKQizeKmSnB9kaLWDks5sRNH7gaJpZM4NRYG2.

Previews for Visual Studio 2017 are available here. [0]
[0] https://www.visualstudio.com/vs/preview/

Hi, I have visual studio 2017 enterprise and, as said by @josephdubeau it still happens in latest update (26430.16) (or its a regression IDK)
Is there something I can do to fix this? or should I rely to commandline npm?

@Dredok that version number points indicates you have update 2, this is fixed in Update 3 which is in preview 8 currently.

@paulvanbrenk  oh I see! I thought it was a public update ... so I guess I will have to wait. Thanks

@Dredok to be clear it is public, just as a preview.


Trouble posting data from Android app

First off thank you very much for you efforts into developing a high precision FOSS WPS suite!
In an effort to compare VoIP (Asterisk PBX) QOS statistics with positional information, I installed the FIND3 Android client on a subsequent wireless VoIP peer device (Samsung Galaxy S6) and the FIND3 Docker server image on the main server PC.
So, my aim is to automatically record location of the peer in the event of VoIP communication.
However, despite following the documentation, I am unfortunately not able to gather the required data to achieve my goal. My initial problems seem to mirror the following Github threads: #138 and #159.
When plainly connecting to the server, walking around the campus and afterwards checking the localhost dashboard, a message appears, notifying me of the following:

You need to add learning data first: group 'targaryan' does not exist

This seems indifferent to the network setup because I tested on two independent networks, three servers (with or without UFW enabled) and the cloud.internalpositioning.com. server.
When I take a look in the local /FIND_DATA/data/ folder, no new .sqlite3.db or .ia file has been created.
In an attempt to mitigate this problem, I thought of manually adding the family name to the server via the terminal. I did this because the python script in /main/testing/ succeeds in creating a family (and posting data).
The following command adds a group (/family):
user@server$ curl -H "Content-Type: application/json" -X POST -d '{"username": "user", "group": "targaryan", "wifi-fingerprint": [{"rssi": -94, "mac": "80:37:73:ba:f7:d8"}], "location": "bogus", "timestamp":<PHONE_NUMBER>831, "password": "fugazi"}' http://<IP_ADDRESS>:8005/learn
Then I deleted the bogus data with:
user@server$ http DELETE http://<IP_ADDRESS>:8005/api/v1/location/targaryan/bogus
The dashboard no longer reports the unknown family and a new .sqlite3.db has been created in  /FIND_DATA/data/. Finally, I tried to add learning data via the app with the following input:

Family: targaryan
Device: user
Server: http://<IP_ADDRESS>:8005
Location: Living room
Submit GPS: False
Mode: Learning

The above mentioned location was varied per room in 5 minutes intervals. After walking to a new room, I hit "Scan" and waited 5 minutes.
The following info appears in /FIND_DATA/data/logs/main.stdout:

...
2019-05-02 13:31:51 [INFO] [PID-14] server.go func1:1360 <IP_ADDRESS>:39495 GET /ws?family=targaryan&device=user 171.886µs
2019-05-02 13:31:51 [DEBUG] [PID-14] db.go getRows:1055 getting columns
2019-05-02 13:31:51 [DEBUG] [PID-14] db.go getRows:1062 getting sensorstringsizer
2019-05-02 13:31:51 [DEBUG] [PID-14] db.go getRows:1079 getting locations
2019-05-02 13:31:51 [DEBUG] [PID-14] db.go getRows:1085 got locations
2019-05-02 13:31:56 [DEBUG] [PID-14] server.go handlerData:927 [targaryan] problem parsing: problem validating data: sensor data cannot be empty
2019-05-02 13:31:56 [INFO] [PID-14] server.go func1:1360 <IP_ADDRESS>:39496 POST /data 260.998µs
...

Afterwards no new data appears to be added to the dashboard and the message on the non affability of learning data is still present. This might be related to the validation debug message.
Could it be the app is faulty or am I (most probably) missing something crucial?
I hope you find the time to investigate and respond.
FIND_DATA_Z.zip

Hey, @EdgarSnoeck I think the issue is on the app.
I get the same errors. So far I can tell is that the receiving data are empty and therefore there will be not used on the backend system. Since empty data will be ignored your family will not be created.
On your mobile device: Did you try to give your app full access and open Bluetooth, GPS as well as W-LAN at the same time (open access privileges as well)?
This helped me to send some data. The problem I have now as you can imagine is as long as I close Bluetooth connection this won't work again.
But I'm still investigating.

Hi Kubilay,
Thanks for the input. I verified my settings and the app has access to all requested system functions. Tomorrow I will test the app with Bluetooth and GPS e

Hi Kubilay, thanks for the input!
I verified all the settings and the app has access to all requested system functions. Tomorrow I will test the app with the different wireless tech. enabled and post the results.

The "group" is never created for me either seemingly making the app completely non-functional. (only tried using the public server)

New Android OS prevents the app to send the data.
You may need to enable Developer Mode for your phone. Wait a while.
Or try esp-client on an esp32 board.
Good luck.


Unbind + re-bind doesn't work.

o = $(".text")[1]
m1 = Mousetrap(o)
m1.bind("y", function() { console.log("m1 called"); }); // Press y, "m1 called" is printed
m1.unbind("y") // Press y, nothing gets printed
m1.bind("y", function() { console.log("m1 called"); }); // Press y, nothing gets printed

Whoops, entirely my fault. Looks like I was removing and inserting the relevant DOM element back on every change inside it, and unbinding Mousetrap in the process.


Update encryption-scope-manage.md

Upload a blob with an encryption scope section the portal screenshot under the step 5 needs to be changed as current screenshot has "setting access tier is not supported with Customer-key encryption" which is incorrect.

Docs Build status updates of commit 1774d18:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/storage/blobs/encryption-scope-manage.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 7585e5d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/storage/blobs/encryption-scope-manage.md
:warning:Warning

Details

articles/storage/blobs/encryption-scope-manage.md

Line 195, Column 3: [Warning: duplicate-alt-text - See documentation] Alt text 'image' is duplicated. Within an article, alt text must be unique.
Line 295, Column 7: [Warning: duplicate-alt-text - See documentation] Alt text 'image' is duplicated. Within an article, alt text must be unique.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@tamram

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
If you want to add these images, the image files will need to be uploaded to the repo (and the image tags in the doc will need detailed alt-text).

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@tamramhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftamram&data=05|01|ramya.harinarthini%40microsoft.com|d9a0c36877c340a0e28508da9d752199|72f988bf86f141af91ab2d7cd011db47|1|0|637995423701365142|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ow4dkN7vpklMdQH6NSv30nulzrLnl1nR%2F3arTP4Njv4%3D&reserved=0
Here is the screenshot which I wanted to include in the document.
@.***
Regards,
Ramya Harinarthini.
From: Jak Koke @.>
Sent: Friday, September 23, 2022 8:36 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Ramya Harinarthini @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update encryption-scope-manage.md (PR #98885)
@tamramhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftamram&data=05|01|ramya.harinarthini%40microsoft.com|d9a0c36877c340a0e28508da9d752199|72f988bf86f141af91ab2d7cd011db47|1|0|637995423701365142|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ow4dkN7vpklMdQH6NSv30nulzrLnl1nR%2F3arTP4Njv4%3D&reserved=0

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
If you want to add these images, the image files will need to be uploaded to the repo (and the image tags in the doc will need detailed alt-text).

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-teamhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Forgs%2FMicrosoftDocs%2Fteams%2Fpublic-repo-pr-review-team&data=05|01|ramya.harinarthini%40microsoft.com|d9a0c36877c340a0e28508da9d752199|72f988bf86f141af91ab2d7cd011db47|1|0|637995423701365142|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=SiXz54y0ApPVRkILNOWeBDo9PLl%2FEn3guGG630yQjVY%3D&reserved=0

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F98885%23issuecomment-1256335220&data=05|01|ramya.harinarthini%40microsoft.com|d9a0c36877c340a0e28508da9d752199|72f988bf86f141af91ab2d7cd011db47|1|0|637995423701365142|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=FBeH51j4BKwtMn5ZDo%2Bh9yJtAMB1qwtSXnpGCSiD0eE%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAT35O7V2NZNG4PIPTLR6AA3V7XBNVANCNFSM6AAAAAAQTZVI3I&data=05|01|ramya.harinarthini%40microsoft.com|d9a0c36877c340a0e28508da9d752199|72f988bf86f141af91ab2d7cd011db47|1|0|637995423701365142|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=NXV7scC2eJJM7q19KTiiD3dg%2F6BjOmaP3teax30oJfI%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@Ramyaharinarthini The PM for encryption scopes says "Currently, only hot tier is supported when encryption scope is in effect." That applies to both types of scopes. Regardless, I updated all of the screen images to make them current. I also changed some of the text and added a NOTE about the limitation of changing the access tier if an encryption scope is used. The updated docs shoudl be published late this afternoon.

#please-close

@Ramyaharinarthini Actually,

@Ramyaharinarthini The PM for encryption scopes says "Currently, only hot tier is supported when encryption scope is in effect." That applies to both types of scopes. Regardless, I updated all of the screen images to make them current. I also changed some of the text and added a NOTE about the limitation of changing the access tier if an encryption scope is used. The updated docs shoudl be published late this afternoon.

Actually, the design seems to be that you cannot change the access tier rom the default for a blob during upload when an encryption scope is in effect.


[Intel MKL] add missing attr

add missing attr for QuantizedDepthwiseConv2D

@penpornk Thanks for your comments about the API, I have update the api file based on your command.


simple_event seems to be unusable - Compilation error with wil::simple_event, const vs non-cost handler.

Attempting to use wil::simple_event with C++/winrt generated code yields compilation errors.
The C++/winrt generated code is similar to:
*token = detach_fromwinrt::event_token(this->shim().MyEvent(reinterpret_cast<winrt::Windows::Foundation::EventHandler const>(&handler)));
This is supposed to call this function in event_base:
winrt::event_token operator()(T&& handler)
{
return m_handler.add(std::forward(handler));
}
But it can't cast the const handler from the C++/winrt generated code to the non-const in wil::details::event_base.
Note the original concept code from Raymond Chen's blog on this originally used const and can be used with C++/winrt.

I hit this as well. I think it's simply a typo.
sklar's initial implementation uses a const&, not a &&.

Fixed by #352


auth - security - passwords are only base64 encoded, not hashed

when adding a new user via API, the password gets stored in LDAP in plain text (base64 encoded)

This is an ui issue. The ansible generated passwords SSHA hashed

found a link explaining that hashing behaviour should be configurable in openldap server:
source: https://stackoverflow.com/questions/11731547/how-do-you-turn-on-password-hashing-ssha-in-openldap#11748013
When you tried to store userPassword attribute in add/modify LDAP operations, userPassword value is stored as plain text. But you can override this behavior using ppolicy_hash_cleartext option in ppolicy overlay module in OpenLDAP. Once you enable it, when client sends a plain text password, it is stored as SSHA by default. You can find more details on enabling hash password in OpenLADP from here


Update Apache POI to 3.10.1

We are currently using 3.8 and there have been three releases since then, but 3.11 was just released today, so a safer strategy is probably to update to 3.10.1.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/7180817-update-apache-poi-to-3-10-1?utm_campaign=plugin&utm_content=tracker%2F32795&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F32795&utm_medium=issues&utm_source=github).

The latest version is 3.12
https://poi.apache.org/download.html#POI-3.12
Should we update to this version?

done. PR #1029 created for this.


Add support for white pixels

Got myself a warm white & cold white led strip with WS2811 controller, which, after playing with the demo samples you added recently, was able to figure out that 1 pixel (1 IC) was lighting on warm white on Red, cold white on Blue, and Green was well... ignored.
So I went on to modify the pixelType type and added a new one:
  pixelType_t WHITE_WxC={15,0,23,0}; // GBR, R=warm white, G=nothing, B=cold white

and added a new Color constructor as such:
      Color WarmColdWhite(uint8_t warm, uint8_t cold)
      {
        this->red = warm;
        this->blue = cold;
        return(*this);
      }

with its respective static public method in the Pixel class:
   static Color WarmColdWhite(uint8_t warm, uint8_t cold){return(Color().WarmColdWhite(warm,cold));}  // an alternative method for returning an RGB Color

then modified your flashColor() method sample with a flashWhite()...
void flashWhite(bool warm, bool cold) {
  for(int i=0;i<MAX_BRIGHTNESS;i++){
    testPixel.set(Pixel::WarmColdWhite(i*warm,i*cold),NPIXELS);
    delay(4);
  }

  for(int i=MAX_BRIGHTNESS;i>=0;i--){
    testPixel.set(Pixel::WarmColdWhite(i*warm,i*cold),NPIXELS);
    delay(4);
  }
}

and initialized the PixelStrand sample with pixel=new Pixel(pin,PixelType::WHITE_WxC);.
So far so good, all is working fine for now.
Now, where I am having issues is with a change to the Holiday Lights project which, using Eve, allows you to set the color of the RGB leg strip using the px->H.getValue(), S and V received from the app. The app has two tabs to define the color. One tab is called "Color", the second tab is called "White". From there, it looks like you can define start and end value for a gradian, from Warm White to Cold White. Defining Characteristic::ColorTemperature colorTemp; within the PixelStrand class, I was hoping to see a change to px->colorTemp.getNewVal<uint32_t>() within the ManualControl : SpecialEffect class, to no avail.
Setting LogLevel to 2, we can also see that even though the White tab is selected, Eve seems to send back HSV values, not any type of color temperature, or gradiant-style values..
Any suggestion to turn HSV values to this WarmWhite / ColdWhite pair of values?

.. I spoke too soon, ColorTemperature does seem to get updated now. I will do some cleanup and may end up submitting a draft PR if all works well.

Can you send me a link to the specific strip you have -- I'm curious as to how they present the underlying specs.  Since it seems like every possible combination is possible, rather than add a new Pixel type to HomeSpan, it may make sense to simply add some documentation so that these new use-cases can be extended directly from the users sketch.

Sorry, I saw your comment before I pushed a draft PR. You will notice the new method to set white colors, which makes the final code cleaner. Also, the new Pixel type will (I think) ensure that the Green is never set. Without this, I seemed to have strange lighting.
I can push a PR to Holiday Lights, or add an example if you'd like to see how it all worked out for me.

+1 I'm using a strip that uses red for the 3000K and green for the 5000K
https://www.amazon.com/dp/B0BNDRF325?ref=ppx_yo2ov_dt_b_product_details&th=1

@JordanMalan, are you currently using HomeSpan 1.9.0, or the "dev" branch?
If you are using the "dev" branch, which pixelType_t type are you currently using to get warm for red, and cold for green?
Are you able to pull from my fork to cherry-pick my commit? I can try adding more white-only pixelType_t values so you can use it the way I did.

@frankonski I'm using 1.8.0 since 1.9.0 seems to have some issue when adding to HomeKit. I'm using the example Pixel sketch.
The only real reason I wish this was better supported in homespan is so the color picker in the home app is correct.
I'll try to pull your code and test, I actually don't mind having many pixel types for different pin arrangements
You're using WHITE_WxC where I might use WHITE_WCx. But I wonder if there's a better way to do this in general, like if someone made a hypothetical red / white / blue strip for example it would be nice to have morph led which takes any user defined colors 🤷

If you have pulled my fork for testing, try adding this pixel type (and using it) to src/extra/Pixel.h:
  pixelType_t WHITE_WCx={15,32,0,0}; // GBR, R=warm white, G=cold white, B=nothing

Let me know if it works.

@JordanMalan, did you get a chance to try my fork fork testing? I solved a last issue in my use of a cold/warm white LED strip, and it is now working like a charm using either the Home app or Eve app, which show a “temperature” light selector (no rainbow color selector).
I use a modified version of the Holiday Lights project, so I don’t know if @HomeSpan would prefer a PR in an new example in HomeSpan, or in Holiday Lights itself so it supports the new Pixel constructor…

Given the new release I think we can close this one.