Python step implementations not found in VSCode devcontainer

Describe the bug
The gauge init project fails to run within a VSCode devcontainer.
To Reproduce
Steps (or project) to reproduce the behavior:

Ensure Docker is installed
Initialize a new gauge project
Open this project folder in VSCode
Ensure VSCode's Remote - Containers extension, ms-vscode-remote.remote-containers, is installed locally
Copy the below devcontainer.json and Dockerfile into a new folder at the project root named .devcontainer
Use the VSCode gauge extension to run the newly initialized project specification
It should run successfully
Open the command palette and select Remote-Containers: reopen in container
Wait for docker image to build, say yes to any requests to install recommended extensions inside the container
Re-run the gauge test using the vscode extension
Note that the step implementations are not found

devcontainer.json
{
	"name": "Python 3",
	"build": {
		"dockerfile": "Dockerfile",
		"context": "..",
		"args": {
			// Update 'VARIANT' to pick a Python version: 3, 3.6, 3.7, 3.8, 3.9
			"VARIANT": "3",
			// Options
			"INSTALL_NODE": "false",
			"NODE_VERSION": "lts/*"
		}
	},

	// Set *default* container specific settings.json values on container create.
	"settings": {
		"terminal.integrated.shell.linux": "/bin/bash",
		"python.pythonPath": "/usr/local/bin/python",
		"python.linting.enabled": true,
		"python.linting.pylintEnabled": true,
		"python.formatting.autopep8Path": "/usr/local/py-utils/bin/autopep8",
		"python.formatting.blackPath": "/usr/local/py-utils/bin/black",
		"python.formatting.yapfPath": "/usr/local/py-utils/bin/yapf",
		"python.linting.banditPath": "/usr/local/py-utils/bin/bandit",
		"python.linting.flake8Path": "/usr/local/py-utils/bin/flake8",
		"python.linting.mypyPath": "/usr/local/py-utils/bin/mypy",
		"python.linting.pycodestylePath": "/usr/local/py-utils/bin/pycodestyle",
		"python.linting.pydocstylePath": "/usr/local/py-utils/bin/pydocstyle",
		"python.linting.pylintPath": "/usr/local/py-utils/bin/pylint"
	},

	// Add the IDs of extensions you want installed when the container is created.
	"extensions": [
		"ms-python.python",
		"getgauge.gauge"
	],

	// 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": "pip3 install --user -r requirements.txt",

	// Comment out connect as root instead. More info: https://aka.ms/vscode-remote/containers/non-root.
	"remoteUser": "vscode"
}

Dockerfile
# See here for image contents: https://github.com/microsoft/vscode-dev-containers/tree/v0.148.1/containers/python-3/.devcontainer/base.Dockerfile

# [Choice] Python version: 3, 3.9, 3.8, 3.7, 3.6
ARG VARIANT="3"
FROM mcr.microsoft.com/vscode/devcontainers/python:0-${VARIANT}

# [Option] Install Node.js
ARG INSTALL_NODE="true"
ARG NODE_VERSION="lts/*"
RUN if [ "${INSTALL_NODE}" = "true" ]; then su vscode -c "source /usr/local/share/nvm/nvm.sh && nvm install ${NODE_VERSION} 2>&1"; fi

# [Optional] If your pip requirements rarely change, uncomment this section to add them to the image.
COPY requirements.txt /tmp/pip-tmp/
RUN pip3 --disable-pip-version-check --no-cache-dir install -r /tmp/pip-tmp/requirements.txt \
    && rm -rf /tmp/pip-tmp

# Add Gauge's GPG key
RUN sudo apt-key adv --keyserver hkp://pool.sks-keyservers.net --recv-keys 023EDB0B

# Add Gauge to the repository list
RUN echo deb https://dl.bintray.com/gauge/gauge-deb stable main | sudo tee -a /etc/apt/sources.list

# Install Gauge apt package
RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \
    && apt-get -y install --no-install-recommends gauge

# [Optional] Uncomment this line to install global node packages.
# RUN su vscode -c "source /usr/local/share/nvm/nvm.sh && npm install -g <your-package-here>" 2>&1

Logs
Localhost
Running tool: gauge run --simple-console --hide-suggestion
Python: 3.8.5
# Specification Heading
  ## Vowel counts in single word
  ## Vowel counts in multiple word

Successfully generated html-report to => /Users/znorris/src/git.scorevision.com/znorris/gauge_devcontainer/reports/html-report/index.html

Specifications:	1 executed	1 passed	0 failed	0 skipped
Scenarios:	2 executed	2 passed	0 failed	0 skipped

Total time taken: 69ms
Success: Tests passed.

Devcontainer
Running tool: gauge run --simple-console --hide-suggestion
Compatible version of plugin html-report not found. Installing plugin html-report...
........................
Successfully installed plugin 'html-report' version 4.0.12
Installing required plugins.
........................
Successfully installed plugin 'screenshot' version 0.0.1
Python: 3.9.1
[ValidationError] /workspaces/gauge_devcontainer/specs/example.spec:11 Step implementation not found => 'Vowels in English language are "aeiou".'
[ValidationError] /workspaces/gauge_devcontainer/specs/example.spec:17 Step implementation not found => 'The word "gauge" has "3" vowels.'
[ValidationError] /workspaces/gauge_devcontainer/specs/example.spec:26 Step implementation not found => 'Almost all words have vowels <table>'
Successfully generated html-report to => /workspaces/gauge_devcontainer/reports/html-report/index.html

Specifications:	0 executed	0 passed	0 failed	1 skipped
Scenarios:	0 executed	0 passed	0 failed	2 skipped

Total time taken: 66ms
Success: Tests passed.

Expected behavior
Gauge should find the python implementation even in a container.
Screenshots
If applicable, add screenshots to help explain your problem.
Versions:
Localhost
VSCode:
Version: 1.52.0
Commit: 940b5f4bb5fa47866a54529ed759d95d09ee80be
Date: 2020-12-10T22:46:53.673Z
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.1.0
VSCode extension: v0.0.17
% gauge -v
Gauge version: 1.1.6
Plugins
-------
html-report (4.0.12)
python (0.3.12)
screenshot (0.0.1)

Devcontainer
VSCode extension: v0.0.17
$ gauge -v
Gauge version: 1.1.6
Commit Hash: 3af456f

Plugins
-------
html-report (4.0.12)
python (0.3.12)
screenshot (0.0.1)

Additional context
Using devcontainers will allow me to quickly bring my entire team up to speed on Gauge without asking much of their local development environments.

@znorris Thanks for creating the detailed issue. I did some investigation and seems like the problem is happening cause of the Python version. The library used to parse the python implementation files is not working properly for the Python version (3.9.3).
We are looking into the issue, meanwhile, you can try a workaround by adding GETGAUGE_USE_0_3_3_PARSER = true in the env/default/python.properties file. This will make the python plugin to use an alternate library for parsing.

@BugDiver Thank you for taking the time to replicate my issue. Your work around was just what I needed! I can't wait to get this out to the rest of the team. 👍


Document behavior of self-healing annotation

There is a closed issue (#11) that added an annotation to control the self-healing behavior and there is at least one example that includes the self-healing annotation.
However, there is no documentation anywhere else as to what exactly this is doing and how to apply it. I believe this annotation will address an issue on my project but I'm hesitant to use it without getting a better understanding of how it works.

@seansund This annotation is no longer in use. The default behavior is self-healing, meaning if you delete the service instance manually on bluemix, the operator will recreate it.
What is the issue you are experiencing?


updated to add new signature

Resolves #233

Thanks!

@all-contributors add @JoshuaTrujillo15 for docs


Peer to peer code review

Hello @RolandM99 Your project looks good, Code is very clean and organized. There is nothing that needs to be changed. Keep it up !
And Happy Coding!

Thank you @Tumworobere


Upgrade Grails from 2.3.11 to 2.4.5 with resources:1.2.1 to resources:1.2.14 -> Resource not found: /js/jsondoc/

Hi guys
Grails 2.3.11 with resources:1.2.1 worked fine.
After upgrading Grails from 2.3.11 to 2.4.5 with resources:1.2.1 to resources:1.2.14 we've got issues on production.
When testing local with the IDE it works fine.
When exporting war for production:
jetty log
2015-08-12 11:23:01,064 [qtp1407343478-12] ERROR resource.ResourceMeta  - Resource not found: /js/jsondoc/handlebars-v2.0.0.js
2015-08-12 11:23:01,064 [qtp1407343478-14] ERROR resource.ResourceMeta  - Resource not found: /js/jsondoc/bootstrap.min.js
2015-08-12 11:23:01,065 [qtp1407343478-18] ERROR resource.ResourceMeta  - Resource not found: /js/jsondoc/jlinq.js
2015-08-12 11:23:01,066 [qtp1407343478-20] ERROR resource.ResourceMeta  - Resource not found: /js/jsondoc/prettify.js
2015-08-12 11:23:01,067 [qtp1407343478-16] ERROR resource.ResourceMeta  - Resource not found: /js/jsondoc/bootstrap-button.js
2015-08-12 11:23:01,068 [qtp1407343478-21] ERROR resource.ResourceMeta  - Resource not found: /css/jsondoc/bootstrap.min.css
2015-08-12 11:23:01,111 [qtp1407343478-15] ERROR resource.ResourceMeta  - Resource not found: /css/jsondoc/bootstrap-responsive.min.css
2015-08-12 11:23:01,111 [qtp1407343478-20] ERROR resource.ResourceMeta  - Resource not found: /css/jsondoc/font-awesome.css

What I did about it:
I tried to play around with the resources plugin.
When I set the resources version to 1.2.8 I received at least the Jsondoc layout header.
I guess it is something wrong in rest-api-doc:0.6.1 with the usage of the resources plugin.
thanks for any help

Hello,
I have a similar problem but just upgrading resources:1.2.1 to resources:1.2.14, with grails 2.3.6. I have realise of this warning, then I get the same error:
WARN o.g.p.r.ResourceProcessor: Serving url file:/js/something.js isn't allowed.
Anyone knows what is happening?


Setup Swagger for API documentation

As we currently create a document listing all the various api_url calls in KnetMiner UI and API and the inputs they require plus the output returned, it would be good to use Javadocs or a Swagger.io-like framework to auto-document current and new API calls from KnetMiner common-datasource.
Current api calls (i.e., the various modes) include:

countHits
synonyms
genome (for user search; returns JSON response with 3 JSON objects, for geneTable, MapView and evidenceTable)
countLoci
countEvidences
network (KnetMaps json for UI users)
latestNetworkStats
genepage (API returns embedded KnetMaps rendered via genepage.jsp)

This is achieved most simply by adding JavaDocs to server-base/src/main/java/rres/knetminer/datasource/server/KnetminerServer.java and server-datasource-api/src/main/java/rres/knetminer/datasource/api/KnetminerDataSource.java

@AjitPS - Have assigned this to you as you have already started documenting this elsewhere.

Yes, I have drafted some docs at https://docs.google.com/document/d/1KyZaBwq0uLnK9NIArIytRrI1CN6xZ5hkG21Nro1KyCo/edit?usp=sharing @KeywanHP but would be better if we used JavaDocs or Spring REST docs to auto-generate this

I would consider Swagger. Maybe it requires Javadoc or Spring REST docs anyway, so a first step would still be having properly encoded documentation in code comments.

what we need @mdonepudi  is the Swagger framework setup for the API endpoints so our team here can document docs for each endpoint within it; and extend usage of the same frameworks for any new, future endpoints (and corresponding docs)

Hi @mdonepudi , we have the API docs for current KnetMiner endpoints (like genome, genepage, network, countHits, etc.) that I documented at https://docs.google.com/document/d/1KyZaBwq0uLnK9NIArIytRrI1CN6xZ5hkG21Nro1KyCo/edit#heading=h.qcrv35uwm5o6 . Would be good to have a basic Swagger setup for auto-documenting these so we can extend with new endpoints easily in the future.
All the request modes are at https://github.com/Rothamsted/knetminer/tree/master/common/server-datasource-api/src/main/java/rres/knetminer/datasource/api and point to methods in https://github.com/Rothamsted/knetminer/blob/master/common/server-datasource-ondexlocal/src/main/java/rres/knetminer/datasource/ondexlocal/OndexLocalDataSource.java . Hope this is useful.
fyi @KeywanHP

We can start with the endpoints documented in this shared Google Doc.

Due to the complexity Swagger configuration for dynamic request mapping, have defined API in Swagger JSON format and swagger-ui to visualise the definitions.

Installation
Un-comment the COPY line in Dockerfile before creating an image.
Configuration
Edit /swagger-ui/knetminer.json to edit:

hostname (e.g: knetminer-dev.mydomain.com/ws)
basepath (e.g: aratiny)
schemes (https/http)

What ever happened to this @marco-brandizi?

We have never done it, the mentioned PR isn't related, it appears here cause it used to happen that people used #nnn to reference PRs, but GH interprets this as issue refs.
Would be good to auto-publish API documentation, I know there are libraries able to generate that from Spring annotations, that they would work with our codebase is another story and I haven't much time to try.
I guess nowadays Swagger was superseded by openAPI.

As a bookmark, this should be the modern way to do it. As for the mentioned problem of KnetminerServer not listing all the API calls explicitly (ie, it dynamically takes the 'genome' fragment /ws/genome and tries to invoke the genome() method), maybe we now have explicit wrappers for all the possible calls, or if we haven't yet, we should introduce them and stop exposing the dynamic wrapper to Spring (ie. remove the respective annotations).


fatal error: 'boost/filesystem/path.hpp' file not found

When building non-QT version it fails.
All the Boost libraries are found:
checking for boostlib >= 1.20.0... yes
checking whether the Boost::System library is available... yes
checking for exit in -lboost_system... yes
checking whether the Boost::Filesystem library is available... yes
checking for exit in -lboost_filesystem... yes
checking whether the Boost::Program_Options library is available... yes
checking for exit in -lboost_program_options... yes
checking whether the Boost::Thread library is available... yes
checking for exit in -lboost_thread... yes
checking whether the Boost::Chrono library is available... yes
checking for exit in -lboost_chrono... yes

Yet for some reason, mooncoin doesn't find them
In file included from crypto/scrypt.cpp:31:
./util.h:27:10: fatal error: 'boost/filesystem/path.hpp' file not found
#include <boost/filesystem/path.hpp>

Litecoin builds without a problem on the same system.

Can I ask what your build environment consists of?
The following was completed days ago on a fresh Ubuntu 14.04LTS system, with Automake 2.69, GCC 4.8.4 etc.
git clone https://github.com/mooncoindev/mooncoin.git
cd mooncoin
./autogen.sh
./configure
..
checking whether to build Bitcoin Core GUI... no (Qt4)
checking for boostlib >= 1.20.0... yes
checking whether the Boost::System library is available... yes
checking for exit in -lboost_system... yes
checking whether the Boost::Filesystem library is available... yes
checking for exit in -lboost_filesystem... yes
checking whether the Boost::Program_Options library is available... yes
checking for exit in -lboost_program_options... yes
checking whether the Boost::Thread library is available... yes
checking for exit in -lboost_thread... yes
checking whether the Boost::Chrono library is available... yes
checking for exit in -lboost_chrono... yes
checking for working boost reduced exports... yes
checking whether the linker accepts -Wl,--exclude-libs,ALL... yes
checking whether the Boost::Unit_Test_Framework library is available... yes
checking for dynamic linked boost test... yes
checking for SSL... yes
checking for CRYPTO... yes
checking for RAND_egd in -lcrypto... yes
checking openssl/ec.h usability... yes
checking openssl/ec.h presence... yes
checking for openssl/ec.h... yes
checking for a supported OpenSSL version... yes
checking whether to build Mooncoind... yes
checking whether to build utils (Mooncoin-cli Mooncoin-tx)... yes
checking whether to build libraries... yes
checking if ccache should be used... no
checking if wallet should be enabled... yes
checking whether to build with support for UPnP... no
checking whether to build test_bitcoin... yes
checking whether to reduce exports... yes
checking that generated files are newer than configure... done
..
builds without issue.

I do remember an issue like this popping up early on whilst compiling the QT wallets for windows.
I'll have a look for you and see if theres an answer - but what you've posted indicates lib/include isnt being passed properly.

Was the environmental variable BOOST_PATH set?


Cannot grant "all" on tables and schemas

Grant "all" is a supported method of granting all access to a user or group for tables and schemas. But the current implementation of grant does not support that.
Can this is be a supported option?

Really curious about this one as well. Any updates or workarounds?


GH-33867: [Go][FlightSQL] Allow passing grpc call options to PreparedStatement methods

Rationale for this change
To allow propagating timeouts among other call options when using PreparedStatements with FlightSQL we need to update the function signatures to allow passing the call options to them.

Are there any user-facing changes?
The signatures for the methods on PreparedStatement objects have been updated to accept opts... grpc.CallOption as their last parameter. This shouldn't break any existing usage of the methods (since they are optional) but would cause a problem if consumers set up interfaces based on them.

Closes: #33867

I was considering it but wasn't sure. It won't break any usage of the methods (since call sites won't be modified because it's an optional amount of params) but if consumers built interfaces that depended on the prepared statement methods, then those might break. thoughts?
I thought the tests were covering it but it looks like i never actually put any call options into the test suite call options. I'm updating the tests now so these get covered and tested.

I think while we're doing major-version releases, let's be conservative and label it.


i wish for simpler way to run the model

i'm not well versed with python and where do i put the downloaded llama-2-7b-chat.Q4_0.gguf file?
i can make llama.cpp work real easy on my laptop but i cant seem to get this to work
i did git clone the neural speed, i did the pip install ... saved the file in run_model.py...
python run_model.py
from transformers import AutoTokenizer, TextStreamer
from intel_extension_for_transformers.transformers import AutoModelForCausalLM

# Specify the GGUF repo on the Hugginface
model_name = "TheBloke/Llama-2-7B-Chat-GGUF"
# Download the the specific gguf model file from the above repo
model_file = "llama-2-7b-chat.Q4_0.gguf"
# make sure you are granted to access this model on the Huggingface.
tokenizer_name = "meta-llama/Llama-2-7b-chat-hf"

prompt = "Once upon a time"
tokenizer = AutoTokenizer.from_pretrained(tokenizer_name, trust_remote_code=True)
inputs = tokenizer(prompt, return_tensors="pt").input_ids
streamer = TextStreamer(tokenizer)
model = AutoModelForCausalLM.from_pretrained(model_name, model_file = model_file)
outputs = model.generate(inputs, streamer=streamer, max_new_tokens=300)

(base) root@ubuntu:/usr/local/src/neural-speed# python run_model.py
Traceback (most recent call last):
  File "/usr/local/src/neural-speed/run_model.py", line 2, in <module>
    from intel_extension_for_transformers.transformers import AutoModelForCausalLM, WeightOnlyQuantConfig
ImportError: cannot import name 'WeightOnlyQuantConfig' from 'intel_extension_for_transformers.transformers' (/root/miniconda3/lib/python3.11/site-packages/intel_extension_for_transformers/transformers/__init__.py)
(base) root@ubuntu:/usr/local/src/neural-speed#

@kolinfluence sry about this.
I have checked your script. It's correct.
The reason may be an too old ITREX version. Have you installed the ITREX?
I can get the correct result using your script.

As you can see the ITREX version is 1.4.1.
Please reinstall ITREX, Neural Speed and re-run the script.
pip install intel-extension-for-transformers; pip install neural_speed

@kolinfluence

where do i put the downloaded llama-2-7b-chat.Q4_0.gguf file?

The script will download the file directly from HF and automatically place it into the local HF cache. The path is like

@kolinfluence OK. We also can inference offline.
Make sure you have the local file llama-2-7b-chat.Q4_0.gguf and model meta-llama/Llama-2-7b-chat-hf.
Please try this script. https://github.com/intel/neural-speed/blob/main/scripts/python_api_example_for_gguf.py
For example:
 python scripts/python_api_example_for_gguf.py --model_name llama --model_path /your_model_path/meta-llama/Llama-2-7b-chat-hf -m /your_gguf_file_path/llama-2-7b-chat.Q4_0.gguf

OSError: You are trying to access a gated repo.
Make sure to request access at https://huggingface.co/meta-llama/Llama-2-7b-chat-hf and pass a token having permission to this repo either by logging in with huggingface-cli login or by passing token=<your_token>.

This means you don't have the right to access the llama-2-7b-chat model on the HF. You have to apply for the access token first on the HF.

may i know what's the direction to take for this neural speed thing? are you guys going to improve or seeking to merge into llama.cpp or something?

The neural speed will not be merged into llama.cpp currently. Neural Speed aims to provide the efficient LLMs inference on Intel platforms. For example, Neural Speed provides highly optimized low-precision kernels on CPUs, which means it can get better perfommance vs llama.cpp. Please check this https://medium.com/@NeuralCompressor/llm-performance-of-intel-extension-for-transformers-f7d061556176.


feat: add missing Tx receipt info

Closes #1922
TODO:

[x]  Gas used
[ ] state_root

Remaining log fields :

[x] log_index
[x] transaction_log_index
[x] removed

Option 3 (update 2017.07.28: we are going with this one): For blocks where block.number >= METROPOLIS_FORK_BLKNUM, the intermediate state root parameter in the receipt should be set to a \x01 byte if the outermost code execution succeeded, or a zero byte if the outermost code execution failed.

ref EIP-98
I think status root has been replaced with status code  (ref EIP-658) which we already have in TransactionReceipt


Adds websockets streaming to Cartesia plugin

draft because haven't set up testing yet

Hey, I wanted to add tests but it seems like I don't have access to push changes to your branch.


Using the new .Csproj without .Net core

Apologies if this is the wrong repository.
Is it possible to use the new .Csproj format without targeting .Net core? I.e. targeting .NET 4.5.x or 4.6.x?

Yes, the SDK tooling works for .NET Framework without explicitly targeting .NET Core. However, there are caveats.  We've focused this release entirely about making just compile of Console Apps, Libraries and ASP.NET Core on NET Framework work. You cannot use the tooling build/use .NET Framework projects that contain Windows Forms, WPF, Workflow, ASP.NET Web Forms, etc.
Paging @onovotny, who does this a bit and can probably point to some guidance he's written.

Oh gotcha. Thanks for the quick response. That's too bad about the ASP.net web forms. Is supporting these other projects on the roadmap after the release of VS 2017, or are there other more urgent priorities?

We wanted to unify csproj tooling across all the .NET stacks this release, but it turned into a fairly large undertaking (who knew 15 years of tooling would be hard to replace!).  I don't have timelines when we'll do it, and what order we'll do it in, but I'm confident that we'll be able to move the majority of projects over to using the new project system (http://github.com/dotnet/roslyn-project-system), and then a significant of those over to the SDK-based format in future releases.

OK, thanks for the information that you're able to provide. I haven't heard of this SDK-based format. Do you have a link or keywords for where I can learn more? Google doesn't turn up anything.

Is it any plans to simplyfi vcxproj to in the future?

We doc'd some of it here: https://docs.microsoft.com/en-us/dotnet/articles/core/preview3/tools/csproj, you can see an example template here: https://github.com/dotnet/sdk/blob/master/src/Templates/ProjectTemplates/CSharp/.NETCore/CSharpConsoleApplication/ProjectTemplate.csproj. That "Sdk" attribute basically auto-imports a props at the top of the project and a targets at the bottom.

@magol To be honest, no plans as of yet. The C++ team are aware of the work we're doing, and once we've landed .NET Core scenarios lots of the new features we've added will just work for them, but I cannot promise anything.

@davkean Is it any list of the new features? For example,  what new features do you think will be usable in vcxproj?
Do the c++ team have any github repository or mail list?

An example of a couple of the new features that vcxproj might be able to benefit from, is auto-reload (ie project file changes on disk, we pick up the changes automatically), or project file editing while the project is open.  C++ team does not have a repo or a mailing list (that I'm aware of), Report a Problem/Suggestion from VS would be away to give feedback to them.

@davkean Great improvements.  But i can not expect any simplifications of the xml in the vcxproj file?

@magol I can only speak for things that I work on, I don't work on C++ - so can't talk about any format improvements that they may or may not.

You can build any project type with the SDK style builds. The main trick is to set the LanaguageTargets to the correct project specific targets for the type of project you want to build. I talk about how it all works here: https://oren.codes/2017/01/04/multi-targeting-the-world-a-single-project-to-rule-them-all/
That said, there are some additional things you'll need to add for certain project types that need additional actions - metadata for specific things like WPF pages, etc.

@davkean is there any roadmap or GitHub issue for tracking progress of the "new csproj format for all project types"?

https://github.com/dotnet/project-system/blob/master/docs/repo/roadmap.md - make note there is no dates associated with the "16.0" milestone that its associated with.

Anyone got this working? I'm able to compile it but I lost intellisense in VS 2017.
This is my csproj:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <LanguageTargets>$(MSBuildExtensionsPath)\$(VisualStudioVersion)\Bin\Microsoft.CSharp.targets</LanguageTargets>
    <OutputType>winexe</OutputType>
    <TargetFramework>net462</TargetFramework>
    <DebugType>Full</DebugType>
    <ApplicationIcon>res\ico\icon.ico</ApplicationIcon>
    <OutputTypeEx>winexe</OutputTypeEx>
    <StartupObject />
  </PropertyGroup>

  <PropertyGroup>
    <VisualStudioVersion Condition="'$(VisualStudioVersion)' == ''">15.0</VisualStudioVersion>
    <VSToolsPath Condition="'$(VSToolsPath)' == ''">$(MSBuildExtensionsPath32)\Microsoft\VisualStudio\v$(VisualStudioVersion)</VSToolsPath>
    <TransformOnBuild>True</TransformOnBuild>
    <TransformOutOfDateOnly>false</TransformOutOfDateOnly>
  </PropertyGroup>

  <Import Project="$(VSToolsPath)\TextTemplating\Microsoft.TextTemplating.targets" />

  <ItemGroup>
    <!-- App.xaml -->
    <ApplicationDefinition Include="App.xaml" SubType="Designer" Generator="MSBuild:UpdateDesignTimeXaml" />

    <!-- XAML elements -->
    <Page Include="**\*.xaml" SubType="Designer" Generator="MSBuild:UpdateDesignTimeXaml" Exclude="App.xaml" />
    <Compile Update="**\*.xaml.cs" SubType="Designer" DependentUpon="%(Filename)" />
    <Compile Include="$(IntermediateOutputPath)**\*.g.cs" Visible="false" />
    <None Include="$(ProjectDir)obj" Visible="false" />

    <!-- Resources -->
    <EmbeddedResource Update="Properties\Resources.resx" Generator="ResXFileCodeGenerator" LastGenOutput="Resources.Designer.cs" />
    <Compile Update="Properties\Resources.Designer.cs" AutoGen="True" DependentUpon="Resources.resx" DesignTime="True" />

    <!-- Settings -->
    <None Update="Properties\Settings.settings" Generator="SettingsSingleFileGenerator" LastGenOutput="Settings.Designer.cs" />
    <Compile Update="Properties\Settings.Designer.cs" AutoGen="True" DependentUpon="Settings.settings" />

    <None Update="App.config">
      <TransformOnBuild>true</TransformOnBuild>
    </None>
    <None Update="App.Debug.config">
      <IsTransformFile>True</IsTransformFile>
    </None>
    <None Update="App.Release.config">
      <IsTransformFile>True</IsTransformFile>
    </None>
  </ItemGroup>

  <ItemGroup>
    <Reference Include="PresentationCore" />
    <Reference Include="PresentationFramework" />
    <Reference Include="System.Xaml" />
    <Reference Include="WindowsBase" />
  </ItemGroup>

  <ItemGroup>
    <Compile Remove="Publish\**" />
    <EmbeddedResource Remove="Publish\**" />
    <None Remove="Publish\**" />
    <Page Remove="Publish\**" />
  </ItemGroup>

  <PropertyGroup>
    <AssemblySearchPaths>$(AssemblySearchPaths);{GAC}</AssemblySearchPaths>
  </PropertyGroup>

  <Import Project="$(MSBuildSDKExtrasTargets)" Condition="Exists('$(MSBuildSDKExtrasTargets)')" />
</Project>

@onovotny thanks for the great information dump on how to migrate existing WPF projects...
I was wondering if you could help in porting Windows Forms project to the new csproj.
I assume that except for <LanguageTargets> I also need to find the names of the generators for windows form classes... right?
In other words...
In "old style" csproj I have:
    <Compile Include="AmazingForm.cs">
      <SubType>Form</SubType>
    </Compile>

I'm assuming the .cs files that are form based need to have a special generator in order for them to compile....
What would that Generator be?

Anyone aware of how to include Telerik reports in the new project template?


[4.0][FIX] selected lang is no more properly forwarded to Odoo

fix typo into header introduced in fe895e2bc85e7a4e5164b70ef4a9c66762c079d7
ping @sebastienbeau

ping @sebastienbeau ping @sebastienbeau ping @sebastienbeau

@lmignon he's holidays... This fix is very straightforward: can't we just merge it?

We can merge-it but I don't know the release process


add new GetAccountInfoWithRpcContext method

that always returns RpcContext event if account is not found,
otherwise the existing GetAccountInfo just returns a notFound error if account is not found so we are not able to get the slot from the response
alternatively, we can decorate the existing ErrNotFound with RpcContext.. but that may break alot of existing clients
and explicitly not returning error seems to make the most sense
what are your thoughts?
thanks

ping?


[BUG] rancher deployed on arm machines cannot provision Harvester VM's

Describe the bug

Rancher deployed on ARM VM's cannot provisioned VM's on Harvester as the node driver packaged in rancher is amd64 version only.
The Harvester node driver packaging in rancher needs to be updated to include the arm specific image when running building the rancher arm artifacts.
To Reproduce
Steps to reproduce the behavior:

Deploy Rancher on a aarch64 VM
Import Harvester cluster into Rancher on aarch64 VM
Attempt to provision a downstream cluster

Irrespective of whether Harvester is running on aarch64 or amd64 architecture, the provisioning fails since the pre-packaged binary downloaded in the provisioning / deprovisioning pod is amd64 architecture.
Expected behavior

Provisioning should be successful
Support bundle

Added area/rancher as this will rely on Rancher.
cc @bk201 @rebeccazzzz

Hit similiar issue in last release testing. Seems relates to

rancher/rancher/issues/39735

changes are now available in v2.9-head and v2.9.3-rc3 onwards
To test:

install rancher on an arm architecture VM
import arm64 or amd64 harvester cluster to newly installed rancher
provisioning of downstream clusters on harvester should work as expected

Test FAIL before RKE2 provision, return to Implement.
Environment

harvester-v1.4.0-rc2

Profile: aarch64, Single node, 32C/64G/1.7T
ui-source: Auto

rancher-v2.9.3-alpha4 ^2

Profile: aarch64, Helm w/ v1.29.9+k3s1
cert-manager-v1.12.13 ^1

Prerequisite

 Create Image jammy-server-cloudimg-arm64.img

 Create Virtual Machine Network mgmt-vlan1

Test Suites

 :green_circle: Setup single node ARM harvester-v1.4.0-rc2

unamerancher@arm-147:~> uname -a
Linux arm-147 5.14.21-150500.55.80-default #1 SMP PREEMPT_DYNAMIC Fri Sep 13 11:29:56 UTC 2024 (12b6dd4) aarch64 aarch64 aarch64 GNU/Linux
rancher@arm-147:~>

Hosts

 :green_circle: Setup ARM rancher-v2.9.3-alpha4

unameroot@arm-133:~# uname -a
Linux arm-133 5.15.0-122-generic #132-Ubuntu SMP Thu Aug 29 13:45:17 UTC 2024 aarch64 aarch64 aarch64 GNU/Linux

Dashboard

 :red_circle: Import Harvester to Rancher

Hit harvester/harvester/issues/6760

 :white_circle: Provision RKE2 cluster

status

Debug Console

 :white_circle: Deploy Nginx and a LoadBalancer

seems like the issue was two instances of cert-manager deployed in different namespaces which was causing conflicts.
Post uninstall of cert-manager from default namespace and restarting rancher the harvester cluster which was flapping in its state became active and is now manageable via rancher.

Retest harvester/harvester/issues/6760 on amd64 looks fine, should be env. issue.
Will test this ticket again.

ddd
Environment

harvester-v1.4.0-rc2

Profile: aarch64, Single node, 32C/64G/1.7T
ui-source: Auto

rancher-v2.9.3-alpha5 ^2

Profile: aarch64, Helm w/ v1.30.5+k3s1
cert-manager-v1.5.1 ^1

Prerequisite

 Create Image jammy-server-cloudimg-arm64.img

 Create Virtual Machine Network mgmt-vlan1

Test Suites

 :green_circle: Setup single node ARM harvester-v1.4.0-rc2

unamerancher@arm-147:~> uname -a
Linux arm-147 5.14.21-150500.55.80-default #1 SMP PREEMPT_DYNAMIC Fri Sep 13 11:29:56 UTC 2024 (12b6dd4) aarch64 aarch64 aarch64 GNU/Linux
rancher@arm-147:~>

Hosts

 :green_circle: Setup ARM rancher-v2.9.3-alpha5

unameroot@arm-133:~# uname -a
Linux arm-133 5.15.0-122-generic #132-Ubuntu SMP Thu Aug 29 13:45:17 UTC 2024 aarch64 aarch64 aarch64 GNU/Linux

Dashboard

 :green_circle: Import Harvester to Rancher

 :red_circle: Provision RKE2 cluster

Create Cloud Credential

From Create Cluster

Directly via Cloud Credentials

Comparison with AMD64

 :white_circle: Deploy Nginx and a LoadBalancer

need to wait for PR https://github.com/rancher/rancher/pull/47548 to be merged before this change can be tested again

this should be fixed in v2.10


docs(Observable): fix typos

I'm not sure I made correct changes, but it definitely feels like something is missing in current docs here:

Each call to observable.subscribe triggers its own independent for that given Observer.

It seems like a word belongs after "independent", but I'm not sure what, so I just suggested "setup".
And for this one:

This shows how a subscribe calls are not at all shared among multiple Observers of the same Observable.

My removal of "a" is fixing a typo, but my removal of "at all" is just simplification. I see why you might want to keep it for emphasis.

Thanks for the fixes, these were originally just mistakes from my dyslexia

@staltz I thought you concurrently writing were documentation


Random packages corruption

I am experiencing random bower package corruption (not all files are present) . I clean the cache, reinstall, and than another package(s) get corrupt. I can see that corrupt packages are in the cache. I suspect glitches in github, but does bower have any checks that archive stream did not end prematurely? Maybe it makes more sense to download .zip files, at least they have crc32 checks in zip format itself?

There is a check here pretty much at the bottom.. it's not very neat though. We're generally experienceing problems with extracting zip files lately, not sure if that's related though!

Than I'm at loss. From "-V" logs I saw tar.gz downloads. I did not capture tcpdump because it is https.
Closing #2108 as a dup: symptoms sounds exactly like my case. If I find any new info, I'll report to #2101.


Wrong type in Issue9SQLiteInt32 example

Summary of the issue
Typo in Issue9SQLiteInt32 example. uint should be changed to ulong
Environment
NA
Example code/Steps to reproduce:
As in your example (lines 107-128):
// SQLite does not support ulong
            foreach (var property in builder.Model.GetEntityTypes()
                                                .SelectMany(t => t.GetProperties())
                                                .Where(p => p.ClrType == typeof(ulong)))
            {
                property.SetValueConverter(
                    new ValueConverter<ulong, long>(
                        convertToProviderExpression: ulongValue => (long)ulongValue,
                        convertFromProviderExpression: longValue => (uint)longValue //Change to ulong
                    ));
            }

            foreach (var property in builder.Model.GetEntityTypes()
                                                .SelectMany(t => t.GetProperties())
                                                .Where(p => p.ClrType == typeof(ulong?)))
            {
                property.SetValueConverter(
                    new ValueConverter<ulong?, long>(
                        convertToProviderExpression: ulongValue => (long)ulongValue.Value,
                        convertFromProviderExpression: longValue => (uint)longValue //Change to ulong
                    ));
            }

Thanks. Applied it via #https://github.com/VahidN/EFCoreSecondLevelCacheInterceptor/commit/1e187177fbd0b35188f3e3a2a19182f763ca765c

Thanks. Applied it via #https://github.com/VahidN/EFCoreSecondLevelCacheInterceptor/commit/1e187177fbd0b35188f3e3a2a19182f763ca765c


fix: Bump delta_kernel version for bug fixes

Description
delta_kernel bug fix: https://github.com/delta-incubator/delta-kernel-rs/pull/328
Related Issue(s)

Documentation

@lewiszlw the 0.3.1 release is not compatible with 0.3.0 since they inadvertently included a major upgrade of arrow-rs with it. The upgrade of the kernel dependency is tracked, and unfortunately blocked, with #2886

Ok... thanks for letting me know.


MethodNameCasing poorly implemented

Essentially the implementation of the MethodNameCasing recipe is inadequate.
All the documentation https://docs.openrewrite.org/recipes/staticanalysis/methodnamecasing
can tell me is Method names should comply with a naming convention.
Well what is that convention and how do I change it? There are no options for that.
A tag is listed: https://sonarsource.github.io/rspec/#/rspec/S100
Which includes the text For example, with the default provided regular expression ^([A-Z0-9_]*|[a-z0-9_]*)$, the method...
If this was in fact the default regular expression then why did it convert a method name get_CHAR_NullableOutputList to getCHARNullableOutputList?
A further issue is that it did not convert the method name get_LongInputList since there was already a getLongInputList method, and yet it did not cancel the recipe with an error/warning.
I'm logging this as a bug not a feature because it applied the recipe and broke the build.

Thanks for the feedback! I can understand the confusion; up to recently that recipe linked to an older source at Sonar that's not been maintained; they'd pointed that out and we've since updated the links.
The old link was a little more descriptive in what is and is not compliant, in particular using a Java example: https://rules.sonarsource.com/java/RSPEC-100/

Shared naming conventions allow teams to collaborate efficiently.
This rule raises an issue when a method name does not match a provided regular expression.
For example, with the default provided regular expression ^[a-z][a-zA-Z0-9]*$, the method:
public int DoSomething(){...} // Noncompliant

should be renamed to
public int doSomething(){...}

Exceptions
Overriding methods are excluded.
@Override
public int Do_Something(){...} // Compliant by exception

I guess it would help to update our recipe description now that the external source isn't helping understand the changes made. Perhaps working back from the tests we have to document the changes.
https://github.com/openrewrite/rewrite-static-analysis/blob/1ed9c44b1413a388a7d436e8a7ca5bcacc40fc94/src/test/java/org/openrewrite/staticanalysis/MethodNameCasingTest.java#L85-L108
@mike-solomon is this one you could take a look at to update the displayed name and description?
https://github.com/openrewrite/rewrite-static-analysis/blob/1ed9c44b1413a388a7d436e8a7ca5bcacc40fc94/src/main/java/org/openrewrite/staticanalysis/MethodNameCasing.java#L58-L64
Beyond that we can then see how what the recipe does might conflict with what you were expecting @delanym ; how did things break specifically? A before/after example would be best, just to be sure we fix the bug as well that prompted you to log it here.

Mike was kind enough to update the display name and description already in

#292

Would you feel we need to make further adjustments still? Perhaps an example with a before/after as seen in our tests would be best for anything not correctly handled right now. Making the examples runnable in that sense also helps rule out any outside influences, like Lombok leading to missing type information.


Render Occluded Lines as Dashes

In the example images, there is one example that shows several cubes whose occluded lines are drawn as dashes. How do you achieve this effect?

Unfortunately that never got exposed as an explicit API option. But you can see how it was done here:
https://github.com/fogleman/ln/blob/master/ln/path.go#L54


Azure problem

Repro steps:

Action: azureStorage.deployStaticWebsite
Error type: object
Error Message: {"key":"1","value":{"_creator":null,"_nextState":null,"_state":{"name":"error","done":null,"then":null},"_value":{"statusCode":400,"request":{"rawResponse":false,"queryString":{},"url":"https://management.azure.com/subscriptions/edaa3574-d80b-4170-9ad2-f9b619fb3f91/providers/Microsoft.Storage/storageAccounts?api-version=2018-03-01-preview","method":"GET","headers":{"Content-Type":"application/json; charset=utf-8","x-ms-client-request-id":"48ffdf75-f638-42cc-9181-8a7b29217ad7","accept-language":"en-US","user-agent":"Node/v8.9.3 (x64-Windows_NT-10.0.17134) ms-rest/2.3.6 ms-rest-azure/2.5.7 azure-arm-storage/5.1.1-preview Azure-SDK-For-Node"},"body":null},"response":{"body":"{"error":{"code":"DisallowedProvider","message":"The operation is not permitted for namespace 'Microsoft.Storage'. List of permitted provider namespaces is 'Microsoft.Authorization,Microsoft.Features,microsoft.insights,Microsoft.NotificationHubs,Microsoft.Resources,Microsoft.Sql,microsoft.support,microsoft.visualstudio,Microsoft.Web,SuccessBricks.ClearDB'."}}","headers":{"cache-control":"no-cache","pragma":"no-cache","content-type":"application/json; charset=utf-8","expires":"-1","x-ms-failure-cause":"gateway","x-ms-request-id":"b9f6d4c4-d776-4a44-9e87-0f0661eea6c5","x-ms-correlation-request-id":"b9f6d4c4-d776-4a44-9e87-0f0661eea6c5","x-ms-routing-request-id":"WESTEUROPE:20180829T110329Z:b9f6d4c4-d776-4a44-9e87-0f0661eea6c5","strict-transport-security":"max-age=31536000; includeSubDomains","x-content-type-options":"nosniff","date":"Wed, 29 Aug 2018 11:03:29 GMT","connection":"close","content-length":"361"},"statusCode":400},"code":"DisallowedProvider","body":{"code":"DisallowedProvider","message":"The operation is not permitted for namespace 'Microsoft.Storage'. List of permitted provider namespaces is 'Microsoft.Authorization,Microsoft.Features,microsoft.insights,Microsoft.NotificationHubs,Microsoft.Resources,Microsoft.Sql,microsoft.support,microsoft.visualstudio,Microsoft.Web,SuccessBricks.ClearDB'."}},"_listeners":null,"_isException":false,"_errorId":2}}
Version: 0.4.1
OS: win32

Do you have any trouble viewing the storage account in the Azure treeview?

The problem starts right after logging in to Azure (from Code).
I do not see the Azure treeview.
Version problems?

     Anders

Fra: Stephen Weatherford (MSFT<EMAIL_ADDRESS>Sendt: 29 August 2018 23:00
Til: Microsoft/vscode-azurestorage<EMAIL_ADDRESS>Cc: Anders Kristian Børjesson<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Emne: Re: [Microsoft/vscode-azurestorage] Azure problem (#225)
Do you have any trouble viewing the storage account in the Azure treeview?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vscode-azurestorage/issues/225#issuecomment-417104008, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADbAXR5-1urz9-en1pL6ZwuNY-YnebJlks5uVwC_gaJpZM4WRUKR.

@andersbor Sorry for the delayed response.  From the error message it looks like you don't have permissions for storage?

The operation is not permitted for namespace 'Microsoft.Storage'. List of permitted provider namespaces is 'Microsoft.Authorization,Microsoft.Features,microsoft.insights,Microsoft.NotificationHubs,Microsoft.Resources,Microsoft.Sql,microsoft.support,microsoft.visualstudio,Microsoft.Web,SuccessBricks.ClearDB'


Issue 44781: Update application.properties and LabKeyServer to use "encryptionKey"

Rationale
This PR replaces the usage of deprecated property master encryption key with encryption key in embedded tomcat.
Related Pull Requests

https://github.com/LabKey/server/pull/210

its fixed now.


[Github Actions] Make workflow configurable in pipeline config

Describe the context of this enhancement
Currently for Github Actions project, by default for backend it will fetch all workflow for the repo. However in some cases not all workflow are meaningful and never be cared to visualize (e.g. scanning, specific testing, spiking ,etc.). If we could exclude these unimportant workflows, we could speed up the data sync as well as reduce some potential issue may have due to them.
Describe the solution you'd like
Enable workflow configuration in the pipeline config.
Task 1: Specify workflows when create Github Actions pipeline

(FE) Enhance the pipeline config UI with a section below Personal Access Token
(FE) Once user clicks Verify button and verify successfully, a workflow API is also triggered to visualize the list of existing workflows
(FE) User is able to select the workflows that they are interested in
(BE) createPipeline(POST /project/{projectId}/pipeline) API accepts the additional configuration about workflow selections and store into DB

Task 2: Update workflows when configure Github Actions pipeline

(FE) Display workflow config once Pipeline config popup is triggered, and initially fetch the latest workflow list
(FE) User is able to update the workflow select and make an update
(FE) Once URL/Personal Access Token is updated, the "Verify" triggering will update the workflow list accordingly
(BE) getPipeline (GET /project/{projectId}/pipeline/{pipelineId}) API returns the additional configuration about workflow selections from DB
(BE) updatePipeline (PUT /project/{projectId}/pipeline/{pipelineId}) API accepts the additional configuration about workflow selections and store into DB

Task 3: Sync data with only configured workflows

(BE) Replace querying all runs to query only the configured workflows' run by the workflow run API
(BE) For compatibility, if workflows are not configured yet when the sync is triggered, keep to query all runs

Additional context
The config of workflow need to store both workflow name and workflow id.
UI position for the new config:

I have one workflow deploying to multiple environments, actually the one provided in input (in my case I have only one prod server).
Therefore what I'm missing is to select the environment I want to measure on.
I don't like to duplicate a workflow for a prod, because I want to ensure this workflow runs fine on lower environments.


CommonTermsQuery ignores disabling the coordination factor with Java API

https://github.com/elastic/elasticsearch/issues/11730

We can close this one because we always send disable_coord :boom:


Pass optional header params to email_alert_api.send_alert

This is required when called from email_alert_service
as this has picked a message up from rabbit MQ and
needs to manually pass the govuk request id in the
header.

@jennyd have fixed the rails 2.1 syntax issues


Spelling correction

Coverage remained the same at 100.0% when pulling d74ffc2305e45e04f3ff4dc369c68854f3fd28c2 on flyswatter:patch-1 into 9c0369667e6fdb7fd70b2b62a38eda9ecab91b57 on SilentCicero:master.


Pickling Error

I'm coming across the following error when trying to run a basic example (as specified in the documentation) to simulate reads.
Can't pickle <function __rule_5 at 0x7f466e8c4d08>: it's not the same object as snakemake.workflow.__rule_5
I installed using miniconda3 with python 3.5.6. on Linux. I noticed perhaps this is a python 3.5 issue with Snakemake, but not being familiar with Snakemake, this thread did not elucidate how to fix this issue other than downgrading my python version: .
I attempted to start a new conda environment with python 3.4 and rnftools, and instead run into a pysam/htslib issue:
ImportError in line 1 of /ccb/salz8-1/rsherman/alignmentSimulation/rnfToolsScripts/Snakefile:
libhts.so.1: cannot open shared object file: No such file or directory
  File "/ccb/salz8-1/rsherman/alignmentSimulation/rnfToolsScripts/Snakefile", line 1, in <module>
  File "/home/rsherman/bin/miniconda3/envs/rnftools_py34/lib/python3.4/site-packages/rnftools/__init__.py", line 8, in <module>
  File "/home/rsherman/bin/miniconda3/envs/rnftools_py34/lib/python3.4/site-packages/rnftools/mishmash/__init__.py", line 3, in <module>
  File "/home/rsherman/bin/miniconda3/envs/rnftools_py34/lib/python3.4/site-packages/rnftools/mishmash/Source.py", line 6, in <module>
  File "/home/rsherman/bin/miniconda3/envs/rnftools_py34/lib/python3.4/site-packages/pysam/__init__.py", line 5, in <module>

Any aid in getting this running with any version of python would be appreciated.

Using python 3.4 and then "conda update pysam" which downgraded samtools, bcftools, and htslib seems to have done the trick.
It does seem though that if python 3.5 is not supported this should be mentioned in the documentation, which specifies python 3.3+ as the requirement.

Hi Rachel,
First of all, thank you very much for using RNFtools and for writing the report.
It looks like the problem does not come directly from RNFtools or Python 3.4/3.5, but from some of the dependencies. I can really imagine how frustrating this can be.

Can't pickle <function __rule_5 at 0x7f466e8c4d08>: it's not the same object as snakemake.workflow.__rule_5

This looks like a bug in Snakemake; I remember similar issues in versions from ~2 years ago.

ImportError in line 1 of /ccb/salz8-1/rsherman/alignmentSimulation/rnfToolsScripts/Snakefile: libhts.so.1: cannot open shared object file: No such file or directory

This looks like a bug in HTSlib/PySam or their Conda packages.
Both Snakemake and PySam are RNFtools dependencies, and both of them had a few releases that were quite buggy. Could you please figure out their versions? I will then update the corresponding dependencies in Bioconda.
The best source of information would be the output of conda env export -n rnftools_py34. Could you please upload it here? I haven't managed to reproduce the error on any of my computers.
Btw. Please say hello from me to Florian :).

So, as I mentioned, using my environment with python 3.4, upgrading pysam changed the versions of samtools, bcftools, and htslib to earlier versions and now it works fine. I'm not sure what they were before, but I know all the versions were newer version numbers. So the output of conda env export -n rnftools_py34 (for those packages, which are the relevant ones) is now working with:
  - bcftools=1.4.1=0
  - htslib=1.4.1=0
  - pysam=0.11.1=py34_0
  - rnftools=<IP_ADDRESS>=py34_1
  - samtools=1.4.1=0

Where as for my other env with (python 3.5), which is not working, the versions of those are:
  - bcftools=1.9=h4da6232_0
  - htslib=1.7=0
  - pysam=0.14.1=py35hae42fb6_1
  - rnftools=<IP_ADDRESS>=py35_2
  - samtools=1.7=1

I just ran conda update pysam in the 3.5 env and that is NOT a fix in 3.5. Where as in 3.4 it downgraded the versions, here it upgrades, and I still get the same error running snakemake.:
  environment location: /home/rsherman/bin/miniconda3/envs/rnftools

  added / updated specs:
    - pysam

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    pysam-<IP_ADDRESS>             |   py35hdfb72b2_1         7.8 MB  bioconda
    samtools-1.9               |       h8ee4bcc_1         483 KB  bioconda
    htslib-1.9                 |       hc238db4_4         1.2 MB  bioconda
    ------------------------------------------------------------
                                           Total:         9.4 MB

The following packages will be UPDATED:

    htslib:   1.7-0                 bioconda --> 1.9-hc238db4_4          bioconda
    pysam:    0.14.1-py35hae42fb6_1 bioconda --> <IP_ADDRESS>-py35hdfb72b2_1 bioconda
    samtools: 1.7-1                 bioconda --> 1.9-h8ee4bcc_1          bioconda

Thank you very much. Could you please include also the versions of Snakemake?

With my 3.4 environment it's  snakemake=3.11.2=py34_1
With 3.5 it's  snakemake=3.6.1=py35_0


Can't do scope authentication

This is probably not an issue with surrealist, but I have tried for a long time, and hope you can solve my issue.
I have defined a database scope called user_scope with a signup query: CREATE user SET email = $email, pass = crypto::argon2::generate($pass) and signin query SELECT * FROM user WHERE email = $email AND crypto::argon2::compare(pass, $pass) with 12h session duration.
However when trying to do scope authentication in the connection details like this

It still says I am offline, and the only thing logged in the terminal is 2023-04-04T20:21:19.537330Z INFO surrealdb::net: <IP_ADDRESS>:65491 GET /rpc HTTP/1.1 101 "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko)" 136.791µs
What am I doing wrong?

Strange, I recreated your example and it is working successfully on my side (Running SurrealDB beta 9)
I'm not entirely sure what the problem could be. Are you sure a valid user account is present in the database before making the connection? Surrealist only authenticates and doesn't register


Concourse documentation updates

We are walking through docs/concourse/README.md today, and happy to contribute back minor fixes we see. But overall, seems awesome!

Just made public my Org membership in https://github.com/orgs/pivotal; closing and re-opening


Provide more detailed information when reading inconformant schema models

When reading a schema that does not conform to specification, goodtables currently just raises an InvalidSchemaError exception, but does not provide useful information on why it failed to validate (e.g. line numbers and position, reason for failure, etc.).
As an example:
$ goodtables schema examples/valid.csv --schema examples/schema_invalid_pk_array.json
...
  File "./local/lib/python2.7/site-packages/goodtables/processors/schema.py", line 91, in schema_model
    raise e
jsontableschema.exceptions.InvalidSchemaError

The error message should instead be something more useful like:
examples/schema_invalid_pk_array.json:14
 "primaryKey": ["id", "titel"]
                      ^
Invalid schema: reference to non-existing field name "titel".

Thanks, nice idea.

goodtables.next
$ python -m goodtables.cli table tmp/issue67.csv --schema tmp/issue67.json

DATASET
=======
{'error-count': 1, 'table-count': 0, 'time': 0.006, 'valid': False}
---------
[-,-] [jsontableschema-error] 'fields' is a required property

Fixed in https://github.com/frictionlessdata/goodtables-py/tree/next
next will be tested for several weeks now before we merge onto master.


entryForm recordType value is inconsistent

Your environment
OS : iOS Catalina 10.15.4
Node.js version : 14.14.4
suitecloud-cli version : 1.0.2
Terminal/CMD tool : iTerm2
Describe the bug
When importing a certain entryForm, there is inconsistency in the recordType value is has.
Sometimes MARKUPITEM is returned and sometimes DISCOUNTITEM.
custform_11_2239021_974.txt
custform_11_2239021_974 (1).txt

@omrilit This is not a problem related to the tool. It should be reported to netsuite custoemr support so they can forward it to relative team.

@christopherwxyz I didn't ask NetSuite customer support but handled it in my application


Initialize paver before the image is fully loaded

naturalWidth and naturalHeight are available as soon as the header of the image is loaded, which for panoramas could be available several seconds before the image is fully loaded.
To verify this I used a lightweight polling function and still kept a listener on the full load just in case that happens between each poll, which could save up to 50ms (!!!)

Thanks for the PR. If it is really better to poll the image header, I will probably refactor your code into the plugin and give you credit, as I'm not really a fan of having loose function declarations floating around the code (I prefer the var based function declaration).
I'll test it out a little and get back to you.

Here, this way you can just pull the PR and you can just leave the credit in the commits :)

Thanks for the PR. The code will be refactored into the v1.3 release :)


character becomes [space] in parameter to post

Hi,
I've got an endpoint that I post string parameters to.
My nswag-generated client uses Uri.EscapeUriString() on the parameter, like this:
if (email == null)
  throw new ArgumentNullException("email");
 else
   url_ += string.Format("email={0}&", Uri.EscapeUriString(email.ToString()));

Unfortunately Uri.EscapeUriString() leaves the + unescaped, so inside my controller it arrives as a space character.
I've patched the nswag-generated client to use with Uri.EscapeDataString() instead, and it does indeed then arrive in my controller as the expected string (with the '+' intact)
Is there a reason that the generated client is using Uri.EscapeUriString() instead of Uri.EscapeDataString()? I'm far from a URI expert, but this SO answer suggests Uri.EscapeDataString() is the one to use, and the answer below it is pretty scathing about Uri.EscapeUriString()

Fixed in https://github.com/NSwag/NSwag/commit/bd7055ea0d4c757fd22ddbebe9f6c09d452903db

Try v5.5

Epic - looks perfect, thanks.

Ok. Nice. Thanks for using the project.

Ok. Nice. Thanks for using the project.


Duplicate word in TSdoc

That file should not be hand-written, instead it should be generated via npm run fetch-idl DOM.
That said, the sentence is from https://dom.spec.whatwg.org/#interface-eventtarget:

Dispatches a synthetic event event to target and returns true if either event’s cancelable attribute value is false or its preventDefault() method was not invoked, and false otherwise.

Maybe we should do some markdown do support the styling.

Aw! Sorry, I knew that there was some automation but I didn't know what was copied and from where :)
Thanks for your explanation.
I'll prune this branch


glDrawElements: range out of bounds for buffer

Hey guys, I am only just getting to know Pixi.JS and I already love it.
But unfortunately I've been coming across a weird issue though, I've searched around for a proper solution but I couldn't really find one.

So I am getting this error here, and when it occurs one of my particle containers fails to render. But what is odd is that I can resolve this issue by re-ordering the children within my container. And the specific particle container that fails to render changes depending on the order of my children.
I'm using a few different particles containers within my game, and a sprite for the player. I came across this error because I wanted the player to be rendered on top of the particles, so I changed the order in which I add children to the container.
My desired child order (when the issue occurs):

The child order that doesn't cause any issues:

Notice, how the only change is that instead of adding the Player after the particle containers, I add the Player before.
Anyway, I thought this was pretty weird. Cheers for the help :)
Here's some more info:
macOS Sierra 10.12.6
Google Chrome Version 60.0.3112.90 (Official Build) (64-bit)
<EMAIL_ADDRESS>

Someone has to dig inside ParticleContainer and investigate shaders/buffers problem :)

Hello,
I am new to Pixi as well - this sounds like a good opportunity to learn more about how everything works under the hood.
I would like to give this issue a shot if no one has any objections.  Cool?
Cheers,
Parmjit V.

@parmjitv Feel free!

Closing this issue due to inactivity, feel free to create a new issue if you are still having issues


CFP: Support basic-auth/oauth2/mTLS for north-south traffic (GatewayAPI/Ingress)

Cilium Feature Proposal
Describe the feature you'd like
I'd like to be able to expose my applications with various forms of authentication. For example, I want to expose hubble-relay outside of my cluster.
Various authentication mechanisms work better for various situations. Generally all 3 mentioned should be relatively standard APIs for configuration. One thing that would be really nice, is to be able to configure different authentication mechanisms based on different rules. For example, mTLS/client-certs if the request is coming from a certain IP range, and basic-auth/oauth2 if a specific user-agent is set.
(Optional) Describe your proposed solution
I would like these to be expose in GatewayAPI, but Ingress is fine too.

Planning to migrate from aws-cni + istio to cilium, we have external auth middleware on istios ingressgateway which redirects user to our keykloak instance, at the moment absence of this feature is one of the blockers for migration.

@artyom-p, could you point me to any documentation about that middleware? The work to design something to do this for Gateway API is still pretty early, so the more concrete examples we have, the better.
I agree these should be exposed in Gateway API, currently Policy Attachment is the way to do this, because noone is sure of the right way to describe the config.
tl;dr Upstream Gateway API design work needs to happen before we can implement this for Gateway API. I'd prefer to wait for that than to add a whole bunch of annotations to Ingress (annotations are sticky and unstructured, which is a bad combination for supportability.)

Thats not related to Gateway API,  its a feature of Istio/Envoy itself, just a middleware. You can check example here.

Yeah, that filter would be pretty useful. It's pretty common to avoid double proxying with an auth proxy, and using the auth proxy to act more as an authentication/authorization API. This is often used with oauth2-proxy/authelia. I agree adding support for that would be very useful for a lot of users.

Ah, okay, that's using the ext_auth support, I agree that's very useful, thanks.

+1 for this feature! (:

Now that cilium supports running envoy in a separate DS, it would make sense to include more built-in extensions like ext_authz, etc.

I am looking for this as well.
Upstream GatewayAPI has an issue tracker for auth.  Its listed as backlog due to needing help on it.

Also a blocker for us to move to cilium ingress.

This is preventing us to fully move to Cilium as well

Is it possible to have the field under network policy's HTTP for jwt authz when the gateway API does not have the API? network policy can also be used between backend services, that's useful when the backend svc using jwt to do the authz

There was an excellent proposal in kubernetes-sigs/gateway-api#532 that would have allowed for authentication/authorization to be delegated to some external service like OPA. Ideally both Ingress, Gateway (global auth) and HTTPRoute could have something like this.

Here is an example of GatewayAPI auth using an API key as a reference as it seems the Kubernetes upstream is still listed as help needed.

Not sure if anyone looked at the Envoy Gateway (https://gateway.envoyproxy.io/latest/user/oidc/) but it has a concept called a security policy for oidc in the next release. It would be great if the security policy would be implemented in a similar manner.

+1 to this feature!

definite +1 for this!

+1

+1

+1

This is needed to switch to cilium service mesh.

For the mTLS side: https://gateway-api.sigs.k8s.io/geps/gep-91/

Auth for north-south Gateway API connections is still being discussed, it looks likely that we'll try and push something forward in Gateway API 1.3, which will start once Gateway 1.2 is out (around Kubecon NA 2024 in mid-November).
Just wanted to let everyone know that this issue is not forgotten, we definitely want to get to it, but we need to work with upstream on doing it the right way.

Oh, I should note that GEP-91 that @ormandj mentioned above is for specifying a client certificate, but it really only guarantees that the client certificate is signed by the specified CA, it can't handle any authorization based on the contents of the certificate or JWT or anything - that will require something else (which is what we're going to work on soon).

@youngnick good point - this has been a major sticking point for us with moving towards Gateway API for everything; how to not just authenticate clients but handle authorization based on their identity.
As a temporary solution, you can create and manage a CA per role and generate client certificates this way, but the management overhead isn't ideal. It'd be nice if there was something available like this, at least as a temporary stop-gap until the more correct/full-featured implementation was in place.

For the mTLS side: https://gateway-api.sigs.k8s.io/geps/gep-91/

I set this up, get no errors, and everything seems to be correct, but it does not work and no client cert is requested. This does not appear to be implemented. There is another issue for it: https://github.com/cilium/cilium/issues/34359

It would be great if one could add a header containing the client certificate DN (and possibly the CA chain that authenticated it). That way many applications can be extended to consume this (as long as that header can be trusted that is).

+1

The Gateway API Auth GEP is scheduled to be worked on as part of the v1.3 release, see kubernetes-sigs/gateway-api#1494 for the details.
Once this GEP has moved to Experimental, we will add support for it in Cilium. I can't guarantee that will happen in the Cilium 1.17 timeframe, but we will definitely add it once it's in upstream.


SSCS-10294 Support for adjournment feature flag

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/SSCS-10294
Change description
Added support for adjournment feature flag, so that old code is run without the flag and the new code
only runs with the flag, and the tests test both.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

Changes being made on another branch, so closing this PR


Add ability to regex postbacks

There was an open issue #87 which asked for ability to regex postbacks. Since I also really needed this functionality - I went ahead and did a quick implementation to get the discussion going.
The api is a complete copy of bot.hear(). This also takes in either a string | RegExp or an array of such, and makes a map to verify later.
I can go ahead and add tests for this, if this is something we'd want to move forwards with. @Charca what do you think?
//cc @mraaroncruz @PandawanFr as both of you originally requested this feature.

@Charca I know that I'm committing necromancy by reviving this issue, but what do you think of going ahead with this?
I've been realising that my branch is something like a year behind the master, and I'd like to update to a later version, but also would like to keep using the changes introduced here (I've been using them for more than a year, and no issues yet)

@Charca the Message Tag issue forced me to update my fork to work with master. Could we maybe use this opportunity to review the possibility of merging this PR?
Or is there something more that I should do before this is ready?
As I've mentioned above - I've been running this in my (tiny) production for ~2 years now without any issues.

Hey @FnTm, apologies for the huge delay. As I mentioned in other issues and PRs, I'm not actively working on this project and have not worked with Messenger bots in a while.
This PR looks good to me and I think we can merge it. Would it be possible to add a small section in the README file with some simple instructions on how to use this new method? Thank you.

yes, for sure! I'll add that ASAP, and ping you for a review.


#87 Add Support for specifying Http Headers in config

#87 developers can specify http headers in the config file like:

for elastic servers that require Authorization header.
the parameter is not mandatory.
this solves also issue #75

@mivano, are you going to pull this request?
this is a blocker for us. we write to elastic search only on production which means we must use Serilog app settings/configuration. and our elasticSearch server requires some authorization (like most elasticSearch  out there..).
since there is no way to specify authorization in config -> this is a blocker. and this pull request overcomes the problem.
it support all kind of authorizations: Bearer, basic,... whatever..
is there a chance you can respond?
tx

Sorry for the slow response. I hope I can dive into this soon. Do consider that you can add your own connection builder and read from the appSettings the right config values.

The 4.2 package is available. thanks for the PR!

https://github.com/serilog/serilog-sinks-elasticsearch/blob/dev/src/Serilog.Sinks.Elasticsearch/LoggerConfigurationElasticSearchExtensions.cs#L145
Currently it splits on =, it is possibly to have a value with an equal sign in it (base 64 encoded values), shouldn't we do:
var headerValue = headerValueStr.Split(new[] { '=' }, 2, StringSplitOptions.RemoveEmptyEntries);

@CumpsD, sure, we can change this to limit the number of returned tokens to 2 in order to support values that contains '='.
I will send a PR over the weekend.
tx

Thanks


Update to match recent algorithm changes

Update computation of quantile and cdf to match changes to the upstream repo in the last year. Also simplified some code (equivalent, but hopefully clearer) and did some general cleanups.

Fixes #14.


Use local identities to represent CIDR

This change is

Coverage increased (+0.08%) to 43.816% when pulling b876f35dbd743c3cef86b0dcbe9f4042d8ed2c05 on pr/tgraf/local-identity into 5104ff68d4296d66334747c51330109016d0f38d on master.

test-me-please

test-me-please

test-me-please

I reran the tests since the two fqdn fails might simply be small timeouts (3s) and the other test fail was an interrupted test.
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/8604/

The FQDN test failure seems legitimate. Will dig in.

Confirming that for FQDN policies are getting correct local identities but the ipcache plumbing is skipped for some reaosn

Found bug in ipcache event invocation. Fixed

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please


Does it stop to release as a MSIX package?

Does it stop to release as a MSIX package in the feature?

See readme notes. https://github.com/microsoft/PowerToys#msix--store-build-update


[swift2objc] Support the async annotation

This is implemented in pretty much exactly the same way as throws. Added a CanAsync interface, started parsing the annotation in parseFunctionInfo, and added the annotation to the generated code. The wrapper method gets an async annotation, and when invoking the wrapped method it must await the result.
Also like throws, there are edge cases where async APIs can't be marked with @objc. Support for these edge cases will be implemented later. See https://github.com/dart-lang/native/issues/1778
Fixes https://github.com/dart-lang/native/issues/1773

coverage: 88.759% (-0.01%) from 88.77%
when pulling dedbf1a9b1dc817ce5c82cd1ea9db70f3d4a991f on async
into 4b03d952059048149fd680ff77034887b53e7ed9 on main.


Comply with CIP-129

Context

Curently we are supporting supports CIP-105
As CIP-129 was ratified we need to support it to be 100% compatible.
Decision

We are going to support CIP-129 fully.
Acceptance Criteria

Development

QA

Can you clarify in this issue what work is needed to make Cardano-address CIP-129 compliant?


ULP I2C only available on GPIO0-3 (IDFGH-9021)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I was surprised to get this error on the ULP RISCV I2C init:
E (533) ulp_riscv_i2c: SDA pin can only be configured as GPIO#1 or GPIO#3
E (541) ulp_riscv_i2c: i2c_set_pin(91): RTC I2C GPIO config invalid
E (548) ulp_riscv_i2c: ulp_riscv_i2c_master_init(418): Failed to configure RTC I2C GPIOs
This is apparently some kind of hardware limitation, though you have to dig pretty deep into the TRM to find it.
Why is this?  The I2C can normally map to any pin, along with every other peripheral on the chip.  This seems like a very odd exception.
It's especially troublesome because GPIO0 is strapped as the boot pin, and GPIO3 is used for USB JTAG select.  So there's only 2 completely safe pins to use for the ULP I2C.
I guess I'll be writing a bit bang driver, but I'd at least like to understand why this extremely specific limitation exists for this extremely commonly used peripheral.  Are there any other weird pin connection limitations like this?

That's indeed a hardware limitation. For example, for ESP32-S3 it is mentioned both in the RTC I2C controller section and in the table 64 "RTC Functions of RTC IO MUX Pins". I'm not sure we can answer the "why" question in the context of the ESP-IDF project. Probably the chip designers have decided that it's not reasonable to spend more hardware resources for having a larger multiplexer or crossbar for the RTC IO controller.  I do agree with you that it ends up being pretty limiting this way.

@igrr Thanks for the response.
This, plus the sub-register limitations, unfortunately make it pretty useless in the general case.
I'm going to switch to a bit-bang driver for the ULP, as it is particularly well suited to bit-banging anyway.


Create absolute paths for jetson-image script.

This probably works ... but I didn't actually try running it after I overwrote the other one creating the branch.

Still not right, actually my fault, need to go crazier like
SCRIPTPATH="$( cd -- "$(dirname "$0")" >/dev/null 2>&1 ; pwd -P )"
On Thu, Apr 1, 2021 at 11:02 AM Jeff Bingham @.***>
wrote:

@.**** commented on this pull request.
In make_jetson_nano.sh
https://github.com/unhuman-io/freebot/pull/5#discussion_r605847653:

@@ -2,6 +2,9 @@

version=R32.5.0
release=5.0
+script_dir=pwd
Check it out now. Switched to using dirname and apparently back-ticks are
out-of-date, so $() is the way to go.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/unhuman-io/freebot/pull/5#discussion_r605847653, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEFINTLDSGUJFWOARG7QXTTGSYJTANCNFSM42BA5WNQ
.

Okay. Should follow what you asked for per: https://stackoverflow.com/questions/59895/how-can-i-get-the-source-directory-of-a-bash-script-from-within-the-script-itsel


training data

Hi,
What data was this trained on?

Never mind, found it.

Could you add the links to download the raw data please?

Hi @lukas1994 , thanks for opening the issue, I completely forgot to mention data used add the link! Could you wait until this weekend and I'll try to sort this out.
Ken

I've added the links to the raw data.


[102] publish 102. close #361

[102] publish 102. close #361

Thanks @jessesquires, checking now :)

Let's omit the description for now then.

Huh?

Ah - I was referring to the intro comments. Sorry for the confusion.


Making username and password as required fields for basic auth plugin

Fix for #1244

@sshniro I replied right now: https://github.com/apache/incubator-apisix/issues/1244#issuecomment-598574032
I think we can close this PR.

Thanks, that makes it clear.


Outgoing webhook: sender_realm_id is not set but it is used set_sender_avatar function

Using an outgoing webhook I got the following error:
Logger root, from module zerver.worker.queue_processors line 151:
Error generated by Anonymous user (not logged in) on ip-172-26-3-8 deployment

Traceback (most recent call last):
  File "/home/zulip/deployments/2020-07-20-06-57-59/zerver/worker/queue_processors.py", line 134, in consume_wrapper
    self.consume(data)
  File "/home/zulip/deployments/2020-07-20-06-57-59/zerver/worker/queue_processors.py", line 613, in consume
    request_data = service_handler.build_bot_request(dup_event)
  File "/home/zulip/deployments/2020-07-20-06-57-59/zerver/lib/outgoing_webhook.py", line 36, in build_bot_request
    keep_rendered_content=True)
  File "/home/zulip/deployments/2020-07-20-06-57-59/zerver/lib/message.py", line 204, in finalize_payload
    MessageDict.set_sender_avatar(obj, client_gravatar)
  File "/home/zulip/deployments/2020-07-20-06-57-59/zerver/lib/message.py", line 493, in set_sender_avatar
    sender_realm_id = obj['sender_realm_id']
KeyError: 'sender_realm_id'

Deployed code:
- ZULIP_VERSION: 2.1.x
- version: 2.1.3

Request info: none

It looks like the sender_realm_id is required although the webhook format that is described here:
https://zulipchat.com/api/outgoing-webhooks does not reference such a field.
As a quick and dirty work around for my production server, in the _finalize_payload method of message.py, I added a try / except around the set_sender_avatar and I checked dictionary values existed first before deleting them (see lemoineat:fix_outgoing_missing_sender_realm_id). But there is probably more to do.

@showell any idea how this might be possible?
@miredo can you confirm what Zulip server version you're running?

@miredo can you confirm what Zulip server version you're running?

The problem was initially seen on Zulip 2.1.3. But since there has not been any change:

on message.py in the methods _finalize_payload and set_sender_avatar
in outgoing_webhook.py on the call to _finalize_payload
no documention update for the webhook
I would suppose the problem is still here unless the documentation on api/outgoing-webhooks is deprecated.

I may try though to remove the patch on my production server (latest stable version) and check if the problem occurs again.

On the production server (latest Zulip release), I reverted my patch and I could not reproduce the problem. So the problem might have already been fixed (Commit queue_processors: Avoid unchecked casts from Anders Kaseorg on queue_processors.py on June 23st 2020 ?)

Yeah, we definitely fixed some bugs with that attribute in that code path (before the 3.0 release), so if you were on an older version, it's likely this is already fixed.  I'll close this, please comment and we can reopen if you see this after upgrading to 3.x.


mark_delete leads to exception at query time

I have an index containing 8 million items. Retrieval works fine for 1 million query with k=60. However, after I deleted 18k items using mark_deleted(). There were 8 queries giving exceptions of "Cannot return the results in a contigious 2D array. Probably ef or M is too small".
I think by marking an item as deleted it leads to unexpected bugs. Could you also explain how exactly a deletion happens? Will it guarantee to give back K results despite half of the index is marked deleted?

Hi @siminchengithub
Deletion happens by marking the elements as deleted and skipping them during formation of the final output (though, the elements are still traversed during the search). It should work even if almost all of all elements.
E.g. such an example works despite deletion of almost all elements:
import hnswlib
import numpy as np
import pickle

dim = 16
num_elements = 1000000

# Generating sample data
data = np.float32(np.random.random((num_elements, dim)))
data_labels = np.arange(num_elements)

# Declaring index
p = hnswlib.Index(space = 'l2', dim = dim) # possible options are l2, cosine or ip

# Initializing index - the maximum number of elements should be known beforehand
p.init_index(max_elements = num_elements, ef_construction = 100, M = 16)

# Element insertion (can be called several times):
p.add_items(data, data_labels)

# Controlling the recall by setting ef:
p.set_ef(100) # ef should always be > k

# Query dataset, k - number of closest elements (returns 2 numpy arrays)
labels, distances = p.knn_query(data[0], k = 989000)
print("closest before deletion (_, label, distance):")
for idx, l in enumerate(labels[0]):
    if idx<5:
        print(idx, l, distances[0][idx])
    p.mark_deleted(l)

labels, distances = p.knn_query(data[0:2], k = 10000)
print("closest after deletion  (_, label, distance):")
for idx, l in enumerate(labels[0]):
    if idx<5:
        print(idx, l, distances[0][idx])

I wonder if your data has duplicates? Can you please share the data?

Hi Yury, thanks for the sample code. Will try to grab the data, it's quite big (10+gb). Could you tell me the implication of having duplicates? duplicates as in items having different index id but same vector, or having the same index id regardless of the vector?

Hi Yury, thanks for the sample code. Will try to grab the data, it's quite big (10+gb). Could you tell me the implication of having duplicates? duplicates as in items having different index id but same vector, or having the same index id regardless of the vector?

also a small correction. The process was indexing of 8million -> addition of 30k -> deletion of 18k -> querying

duplicates as in items having different index id but same vector, or having the same index id regardless of the vector?
Duplicates are the same vector with different ids. They might make the graph go crazy sometime (graph might lose connectivity), though not sure if it can affect deleted elements.

Duplicates are the same vector with different ids. They might make the graph go crazy sometime (graph might lose connectivity), though not sure if it can affect deleted elements.

ah check. This type of duplication can happen rather often though, especially in an advertisement use case where different items can have the same description. How would you suggest to handle duplicates in this case?

@siminchengithub The deduplication is supposed to be happen outside, though it could have been implemented inside the library (the downside is some slowdown during the construction and additional structures).
If a user has duplicates, they can deduplicate, merge the labels of the duplicates (as the elements are exactly the same) and use the library.


Fix issue #241  where -printcoinage causes silent crashes in linux

@mitchellcash does this effect mingw compiling in windows at all (assuming not I want to go ahead and merge it as it's fairly minor)?

This compiles fine on mingw. Let's get it merged


declare '#app' {} not working for useNuxtApp()

Vue - Official extension or vue-tsc version
2.0.26
VSCode version
1.91.0
Vue version
3.3.13
TypeScript version
4.9.5
System Info
System:
    OS: Windows 11 10.0.22631
    CPU: (8) x64 Intel(R) Core(TM) i5-8300H CPU @ 2.30GHz
    Memory: 2.57 GB / 15.88 GB
  Binaries:
    Node: 18.20.2 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 10.5.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Chromium (126.0.2592.87)
    Internet Explorer: 11.0.22621.3527

Steps to reproduce
'#app' module with NuxtApp not working but '#app/nuxt' module with NuxtApp working.
It works:

But it doen't work:

Or it:

What is expected?
Working '#app' module
What is actually happening?
Not working '#app' module
Link to minimal reproduction
https://stackblitz.com/edit/github-fxc8uz?file=index.d.ts
Any additional comments?
No response

Experiencing the same issue though not using nuxt and just global declare for some internal package in my monorepo.
Last working version for my was vue-tsc 2.0.22. 24 starts failing for me

@minht11 for me it doesn't work even for<EMAIL_ADDRESS>and Vue extension with this version. Generally, since the beginning of Nuxt 3 I have had problems with typing in useNuxtApp, but now I finally managed to find some specifics.

Thank you for reporting this issue. I think this issue can't be fixed in this repo, because it may happen in any TypeScript Nuxt projects. Please report the issue to Nuxt. Thanks!


Adding tails server and tunnel agents to docker compose file

Adding tails server and tunnel agents to docker compose file :)
Tested and ran fine under
docker-compose build
docker-compose run tests
docker-compose down
Signed-off-by: Alexandra N. Walker ☀

Yay! I think I did this successfully @dbluhm


JS tests flaky in master

See https://circleci.com/gh/weaveworks/scope/5669

Fixed by simon in 072bea9


Restriction using throw statement.

Hi,
This code in BeanShell works.
throw new Exception();

This code should, but doesn't, because the BeanShell
throw statement has to be used in tandem with an
Exception, rather than a Throwable object.
throw new Throwable();

Reported by: amc1
Original Ticket: "beanshell/bugs/30":https://sourceforge.net/p/beanshell/bugs/30

Closed: duplicate #546

Resolved.
BeanShell 3.0.0-SNAPSHOT.463
bsh % throw new Exception();
// Error: Target Exception: TargetError : at Line: 1 : in file: <unknown file> : throw new Exception ( ) ;
Caused by: java.lang.Exception
bsh % throw new Throwable();
// Error: Target Exception: TargetError : at Line: 2 : in file: <unknown file> : throw new Throwable ( ) ;
Caused by: java.lang.Throwable
bsh %

try {
    throw new Throwable();
} catch (e) {
    print(e);
}

java.lang.Throwable
--> void


feat: add new option "return_segments" to allow accessing word probabilities and other meta information

Currently RealtimeSTT doesn't expose the meta information about segments and their words, which are otherwise available when passing word_timestamps=True to the faster-whisper model. This information is very nice to have when building an application on top of RealtimeSTT, since it allow displaying word detection certainties, or to align words temporally.
This introduces a new option return_segments that enables word_timestamps=True on both the realtime and main model, and causes the related functions to take a tuple (text, segments) instead of just text. This change is opt-in and fully backward compatible.

I will address the conflicts as soon as #71 is addressed, otherwise I can't test my changes.


tranquility-server  realtime(not creating merged beam)

2018-08-22 16:53:00,950 [ClusteredBeam-ZkFuturePool-e24324d4-4cf8-4472-998e-b99cdfd423bb] INFO  c.m.tranquility.beam.ClusteredBeam - Global latestCloseTime[2018-08-22T08:00:00.000Z] for identifier[druid:overlord/test] has moved past timestamp[2018-08-22T08:00:00.000Z], not creating merged beam
2018-08-22 16:53:00,962 [ClusteredBeam-ZkFuturePool-e24324d4-4cf8-4472-998e-b99cdfd423bb] INFO  c.m.tranquility.beam.ClusteredBeam - Turns out we decided not to actually make beams for identifier[druid:overlord/test] timestamp[2018-08-22T08:00:00.000Z]. Returning None.
task of segment2018-08-22T08:00:00.000Z in overload has killed, how can i retry it or new other?

yeah I have the same issue


GPYX

GPYX
https://etherscan.io/token/0xbc7f402f3bc1c6d56c8041f551b47a0ad7714d1b
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[ ] Filename in the lowercase format
[ ] Image format: png
[ ] Image size: 256 by 256 px

Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwallet.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp


Fix incorrect results in COUNT(*) queries with LIMIT

Which issue does this PR close?
Closes #8048.
Rationale for this change
While testing #8038, I ran into some incorrect results cases in COUNT(*) queries
from a LIMITed relation, related to exact statistics.
The issue is due to use of the fetch value plus the skip value in
stats for GlobalLimitExec as the output stats num_rows, plus use of Exact
statistics for the output in cases where the input has Inexact statistics.
What changes are included in this PR?
Fix incorrect results in COUNT(*) queries with LIMIT
This commit reworks the cases in GlobalLimitExec::statistics to cap output
stats num_rows at the fetch value instead of the fetch+skip value.
Also, the following cases are modified:

Output stats are copied from input stats when # of input rows is less than fetch rows, and skip is 0
if (# of input rows - skip) <= fetch, output num_rows = input num_rows - skip
if input stats are Inexact or Absent, output stats are Inexact
if (# of input rows - skip) > usize::MAX and fetch value is None, output stats are Inexact

Are these changes tested?

[x] unit tests for GlobalLimitExec statistics, both Exact and Inexact.
[x] sqllogictests for GlobalLimitExec statistics

Are there any user-facing changes?
No

@berkaysynnada Does this change look OK to you?


Ss 190 fetch studentsand exams by query api

PR tested. Working as expected

@navin9584 / @apurvabisht97  , sonar scan report for BE,FE plz

FE sonar scan report


openblas: build broken on aarch64 (master & release 19.03)

Issue description
It looks like that https://github.com/NixOS/nixpkgs/pull/56216 broke the build on aarch64:
gfortran: error: unrecognized command line option '-m64'

Steps to reproduce
For example see https://hydra.nixos.org/build/89399945 for a build log.

I stumbled onto this when trying to build postgis on master.


Appstudio update test-component-pac-doii

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-doii-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
26 seconds
build-container

✅ Succeeded
9 seconds
deprecated-base-image-check

✅ Succeeded
9 seconds
inspect-image

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
42 seconds
clamav-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
14 seconds
label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-doii-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
30 seconds
build-container

✅ Succeeded
16 seconds
deprecated-base-image-check

✅ Succeeded
17 seconds
inspect-image

✅ Succeeded
10 seconds
clair-scan

✅ Succeeded
43 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
14 seconds
label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary


Fluent DSL TableAdmin client

Can you please check the latest commit. It should be fixed there.
On Wed, Jun 20, 2018 at 11:45 AM, Igor Bernstein<EMAIL_ADDRESS>wrote:

@igorbernstein2 commented on this pull request.
In google-cloud-clients/google-cloud-bigtable/src/main/java/
com/google/cloud/bigtable/admin/v2/models/TableAdminResponses.java
https://github.com/GoogleCloudPlatform/google-cloud-java/pull/3395#discussion_r196833809
:

}

@Override
public String toString() {

 return MoreObjects.toStringHelper(this)

     .add("tableName", tableName)

     .add("timestampGranularity", timestampGranularity)

     .add("clusterStates", getClusterStates())

     .add("columnFamiles", getColumnFamiles())

     .toString();

}
}

/** Wrapper for {@link ClusterState} protocol buffer object */
public static final class ClusterState {
private String id;

please fix
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-java/pull/3395#discussion_r196833809,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfisGFYsBh1tGALGoWPmlXom-mR70kGwks5t-m4EgaJpZM4UuKE1
.

@pongad thanks for looking.
LGTM after typo fix


Fix yaml parsing issue

1e-4 is interpreted as an str, the fix is to pass a float
cc @lvwerra

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


moved target test versioning test before problem specific test

I just did accumulate, added test versioning and moved the version test to be the first function in the test file. Wanted to be sure its correct before starting the rest. See #465

Based on consensus in #465, not going to move the test version test to the top so this is no good.


Round elapsed time

Reporting the elapsed time down to the nanosecond is a bit overkill, round to the closest second looks nicer.
Before:

Chef run succeeded on node webserver (9.062432019 seconds). 3 resources updated

After:

Chef run succeeded on node webserver. 3 resources updated in 9 seconds.

Matches better with the official cli output:

Chef Client finished, 3/23 resources updated in 9 seconds

Hi @martinb3. Any chance you could release a minor release that includes this change? :) Thanks!