Imports not detected when client sends only textDocument/didOpen on newly created file (Unity project, neovim client)

Hi!
I'm trying to troubleshoot some issues the client I work on has with omnisharp-roslyn, and I was hoping I could ping you all for help.
The issue is, when opening a unity project, say one containing the following file:
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class NewBehaviourScript : MonoBehaviour
{
    // Start is called before the first frame update
    void Start()
    {
    }

    // Update is called once per frame
    void Update()
    {

    }
}

Everything is fine, our client detects the imports no problem.
However, the minute we open a new file (for which we send the requisite textDocument/didOpen notification) with the exact same content, in the exact same subdirectory, the server returns import error messages:
The type or namespace name 'UnityEngine' could not be found (are you missing a using directive or an assembly reference?) irective or an assembly reference?)

Other servers seem to be fine recognizing new files this way. Do we need to send an additional notification when creating a new file such as workspace/didChangeWatchedFiles?

Having the same issue in Helix.

However, the minute we open a new file

Is this a new file that has been saved and exists on disk?

Do we need to send an additional notification when creating a new file such as workspace/didChangeWatchedFiles?

If the file was added to the filesystem, then I expect that file watchers would need to be updated.

Is this a new file that has been saved and exists on disk?
It's a new file that has not been added to the project. If I'm understanding you correctly then you must regenerate the solution files after the addition of a new file for omnisharp to pick up the imports correctly?

I have the same issue... I got the same error as the second image by mjlbach.

Any progress on this one? This also happen after renaming of class. I have to regenerate those files from the button on External Tools > Regenerate project files.

I actually was able to maintain a sustainable workflow by creating a second class (let's say Foo.cs) on a already "Synced File" (where the LSP features work normally) and using the Code Action Move type to Foo.cs.
That way it's possible to work on Foo.cs without problems.
But that's just a workaround, but maybe it can lead us somewhere.

A workaround I have found is to add the following line to Assembly-CSharp.csproj:
	 <Compile Include="**/*.cs" />
As mentioned above, each Unity code file needs to have <Compile..../> element in the .csproj file.
On neovim (using OmniSharp Roslyn LSP server), it seems like the .csproj file (specifically Assembly-CSharp.csproj) does not get automatically "refreshed" after it changes. The changes to .csproj file only seem to be applied after restarting neovim (and omnisharp).
This doesn't seem to be the case with vscode, however. When making changes to .csproj file (adding the compile element), vscode directly picks up the change and omnisharp server seems to be working as intended.
I believe "Regenerate Project Files" button on Unity ensures there's the <Compile.../> tag in the .csproj file for newly created/renamed files. There is a vscode extension (and a modified version to work with neovim) that automatically regenerates the project files when there's a new file. This is done in Unity though.
I doubt my workaround is going to be a good solution in the long run, mostly because Unity generates this file, which will override the line I've described above.

I'll test that approach later.
But what I've found is it seems like there're 2 different problems.
There're some repos on GitHub mimicking the behavior of Unity plugins for VSCode and JetBrains Rider, like this: https://github.com/neovim/neovim/issues/17808#issue-1175601347
I even tried to improve on it myself, but even using Unity's API to Regenerate Project Files automatically on file changed, the behavior described in this issue still occurs.

I've been looking into this in a similar issue for OmniSharp-vim. From what I've seen, I believe omnisharp-vscode has a file watcher watching the .csproj file, and when it is modified (by Unity, which adds a new <Compile ...> element for the new file) then vscode sends a /filesChanged request for the csproj to OmniSharp-roslyn, which then reloads the project.
I've tried to mimic this /filesChanged request from OmniSharp-vim and works - the new file is now part of the project, without requiring a full OmniSharp-roslyn restart.

That's great to hear! Is this change on the repo yet?

Which repo? OmniSharp-vim? Not yet. But I'm just adding it as a command, :OmniSharpRelosdProject, there aren't file watchers in OmniSharp-vim.
I was letting you know what my findings were in case you wanted to investigate making that /filesChanged request from neovim lsp

Oh... 1 issue may be that OmniSharp-vim doesn't run OmniSharp-roslyn on LSP mode. I'm not sure if omnisharp-vscode has switched to LSP mode either? So I don't know if the /filesChanged endpoint will have the same effect from neovim LSP. Hopefully?

is it possible to watch the files from nvim and send request when the document is changed and document created
for unity i notice the .csproj file is changing after adding, removing and renaming files. is it possible to watch this .csproj file ?
if it possible did the request to server reload all the index ?

Is there any update to this issue? Adding on to what has been previously said:
From my testing, although I'm not entirely certain, I've noticed that making changes (i.e. adding a new line of <Compile> tag) to the .csproj does not queue the project file for ProjectManager to process. Is this an expected behavior from file watchers?
Another combination I've tested is adding the <Compile> tag before adding the .cs file itself in a same session. This also didn't work, as .csproj did not get queued (according to my assumption).

Following the idea from @nickspoons above (with my little knowledge of LSPs) I tried to find if there's a way to make a similar request to this /filesChanged in to the Omnisharp LSP client.
Unfortunately, it seems that this endpoint doesn't exist for the LSP side.
These are the endpoints for Omnisharp, it's listed on the "v1", but I think these endpoints are only really valid for non LSP stuff, these are the handlers for the LSP part
The closest I could find was this: OmnisharpOnDidChangeWatchedFilesHandler
But doesn't seem that there's a way to request something from it, and I don't really know how to interact with it.
Sorry for the dumbness but I don't really know much about LSP, just leaving my little research.

@niscolas I just went through some painful issue-climbing(i.e. reading tons of old issues) and I finally found something related. Since you have done some assiduous effort I share this with you: https://github.com/neovim/neovim/pull/21293, which is a PR that will be a solution to the problem we care about.

@ChimpSpin have you managed to get it working? I see that some people above (https://github.com/OmniSharp/omnisharp-roslyn/issues/2250#issuecomment-1049056993) have vscode as their configured editor, and it seems it generates some files (seemingly config files) that may be involved in allowing the lsp to work correctly.
As a note, I don't even have basic functionality for a single file... so I would really thank you, if you could help me :)

@0rphee
I think that for being able to work without major issues with Unity (except the one discussed in this issue), you need:

A properly working Neovim + LSP setup
Have the Omnisharp LSP server installed and configured (I recommend Mason for that)
Have a secondary editor directly supported by Unity and with an official integration plugin (either Visual Studio or Jetbrains Rider, since Unity seems to be dropping support for VSCode - link)
That's what I've always done and had no problems besides this one

@nyngwang

@niscolas I finally found something related: neovim/neovim#21293, which is a PR that could be an important factor in finding/creating the solution to the problem.

That's super interesting!
That seems to indeed solve the issue here, have you tested the solution on that PR?

@niscolas thanks, though I forgot to mention that I don't use nvim! I use helix... anyway, maybe I will take a look at the links that you talk about. They may help, thanks!

@0rphee
I had to download https://github.com/OmniSharp/omnisharp-roslyn and add this to my languages.toml
[[language]]
name = "c-sharp"
language-server = { command = "<Your path to>/OmniSharp.exe", args = ["--languageserver" ] }

@ChimpSpin
I already have that in my languages.toml, but omnisharp doesnt seem to find UnityEngine as shown in the image from my first comment (https://github.com/OmniSharp/omnisharp-roslyn/issues/2250#issuecomment-1424992708). Did you have to change any settings specifically to get it working with Unity?
sorry for bothering :( its just that I am a bit lost with this...

@0rphee
Aside from that, I just had to generate the project files from within unity.
Edit > Preferences > External Tools > Regenerate project files

I don't think there was anything else to it.

@ChimpSpin thank you!

@niscolas FYI: The linked PR I mentioned 3 weeks ago had been merged into Neovim recently: https://github.com/neovim/neovim/pull/21293

Yay, that's great!
I'll be taking a look as soon as I can to check whether it solves the issue and if we will still need some kind of plugin in either side (Neovim / Unity)

@nyngwang How would one use this new feature to solve the new file issue?

@nyngwang How would one use this new feature to solve the new file issue?

Bump, I also wanna know. Kinda a newb in all this.

I believe that I just got it to work.

You should be using a build for Neovim that includes the changes in the PR above (link), I recommend just using bob to install Neovim nightly
Your LSP Capabilities table should include workspace.didChangeWatchedFiles.dynamicRegistration = true (Neovim news.txt), example:

local cmp_nvim_lsp = require("cmp_nvim_lsp")
local capabilities = cmp_nvim_lsp.default_capabilities()
capabilities = vim.tbl_deep_extend("force", capabilities, {
    workspace = {
        didChangeWatchedFiles = {
            dynamicRegistration = true,
        },
    },
})
// pass `capabilities` to your LSP server `setup()`

I only tested this by creating the file through Unity, so I think it automatically adds the file to the Assembly-CSharp.csproj.
If creating files directly from Neovim, there's the need of using a package in the Unity side, like this one: https://github.com/niscolas/com.unity.ide.neovim-fork
I didn't test it recently, but it should work, it also adds Neovim as an External Editor option in Unity's settings


Enable real browser test

Users
There are two ways if you want to test version >= 7.0.0 in real browser,

git clone then visit browser-test/index.html
Zero install using rawgit, link. Remember to change hash to latest commit.

Changes

[source] Change /JsRegexpLookbehind/ to new RegExp(JsRegexpLookbehind), prevent syntax error breaks user code.
[source] Use reverse-lookahead-reverse as fallback when not support JsRegexpLookbehind.
[source] Add URLSearchParameters.sort ponyfill as fallback.
[source] Add options preferJsRegexpLookbehind and preferURLSearchParamsSort to run fallback logic in modern runtime.
[types] Add above two options to .d.ts.
[source] Move normalizeUrl(options) options validation to most top.
[source] Treat custom protocol as .defaultProtocol, to prevent host as part of pathname.
[source] Change back custom protocol after urlObject.toString, to prevent pathname changed again.
[workflow] Add karma to enable automate headless browser tests.
[workflow] Add rollup to transform regexp named-capturing-groups and bundle tests for browser.
[tests] Change ava to mocha+chai, enable to test it in Node.js / Headless Chrome / Headless Firefox / Any real browsr.
[tests] Update prev invalid urls to match more than Node.js env.
[tests] Seperate some long tests in to small parts to easy identify problems.

Test Status
(32/32) Test passed with

Node.js 14
Chrome headless driver
Firefox headless driver
Chrome 92
Firefox 90
Firefox 68 ESR
Opera59
Yandex 7 (2017)

https://github.com/sindresorhus/normalize-url/issues/105

Safari support was fixed in https://github.com/sindresorhus/normalize-url/releases/tag/v7.0.1. I'm not interested in browser tests, browser-specific options, or the .sort polyfill. You also have way too many unrelated changes in a single pull request.

Hi, @sindresorhus

I'm not interested in browser tests
But how can you guarantee this library works in browser, without testing it in browser?

Eg: issue#147

browser-specific options
I know that you may not like that option, so I did try to remove it.
But test-runtime is modern, need to way to run all logic.
So I restore them.

You also have way too many unrelated changes in a single pull request
I'm kind of non professional newbie.
Should I combine all commits as one, and break them into small features.
Then one feature -> contribute -> one feature -> contribute?

FYI, there are something I found so far.

some rare url in browser, issue#147
Fixed in my latest fork
name capture group may break ok code like issues#105
https://github.com/sindresorhus/normalize-url/blob/24560330a31eda9253172c02339ec610f5917d4b/index.js#L8
options validate should ASAP
https://github.com/sindresorhus/normalize-url/blob/24560330a31eda9253172c02339ec610f5917d4b/index.js#L105

Have a nice day @sindresorhus 😊

But how can you guarantee this library works in browser, without testing it in browser?

I never made it for the browser.

options validate should ASAP

Not sure what you mean?

Old browser support is not a concern for this package. It's up to you to use Babel and polyfill if you need to use it in older browsers. I only merged the Safari fix as it applies to latest Safari.

In modern and old browser, custom protocol error like issue#147

#132

Not sure for that pull, but existing today-test.js failed for both modern and old browser issue#147

options validate should ASAP

Not sure what you mean?

 function normalizeUrl(urlString, options) {
 // ...code
	if (options.forceHttp && options.forceHttps) {
		throw new Error('The `forceHttp` and `forceHttps` options cannot be used together');
	}
// ...code
}

 function normalizeUrl(urlString, options) {
	if (options.forceHttp && options.forceHttps) {
		throw new Error('The `forceHttp` and `forceHttps` options cannot be used together');
	}
 // ...code
// ...code
}

But how can you guarantee this library works in browser, without testing it in browser?

I never made it for the browser. But it doesn't use anything specific to Node.js, so if it works in Node.js, it should work in modern browsers too.

But your "this child" seems to be also used in browser from recent issues?
Maybe consider using convert ava to mocha+chai+karma?
As it also provide automate browser test ability for headless browser drivers?

I'm not interested in browser testing.

But it doesn't use anything specific to Node.js, so if it works in Node.js, it should work in modern browsers too.

URL implement is different. When custom protocol, node behavior is whatwg-url,
but browser is not, A://B/C host in browser is empty.

I'm not interested in browser testing.

But I saw issues related to browser, may it give a try, decrease future potential issues?
As it can be automated?


⚠ Bofrost.be has degraded performance

In 7adf5c7, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1042 ms

Resolved: Bofrost.be performance has improved in b6e066f.


Page 35 - Ansible -m package can't tell which package manager to use

This could be an issue with my Ansible version (I'm using 2.6.0)
When i run the following command in chapter 3, page 35 (top of the page
ansible app -b -m package -a "name=git state=present"
I get the following.
<IP_ADDRESS> | FAILED! => {
"changed": false,
"msg": "Could not detect which package manager to use. Try gathering facts or setting the "use" option."
}
<IP_ADDRESS> | FAILED! => {
"changed": false,
"msg": "Could not detect which package manager to use. Try gathering facts or setting the "use" option."
}

Ansible 2.6 is far out of support, and I think most of the examples in the book would be broken under that version. Please install at least Ansible 2.9 (the oldest supported version) and see if you have the same problem with it.

Will do,
2.6 though put a nice line between server information like you have in the book.  2.9 Which I was using, crammed everything together.
If there is a simple switch to add a line break between server information sets, I'd love to know it for readability.


bump StartUpDG.jl version

Bump StartUpDG.jl version to 0.12, which reorders faces for Quad() elements. Previously, faces were ordered in a counterclockwise fashion. Faces are now ordered according to r = -1, r = 1, s = -1, s = 1 (this is consistent with the way Hex() element faces are ordered, and may make it easier to deal with a tensor product structure).

Thanks!


Fixes in data import

[x] Fix for contract date in data validation
[x] Update contracts attribute task execution upon Contract GoodsServices creation
[x] Add signal task to delete data import and its import batch
[x] other minor issues

Can this be merged?

This changes were deployed and working fine. Only exception was the Coverage issue.


fix some version of Node with single slash protocol in URL

I've tested with Azure's version of node and it fixed the issue.

Coverage decreased (-0.9%) to 99.115% when pulling 049bb95ec578c3d80d8cc72ca4fbd556f74808ce on chrischip:master into 4814647a9d21f14858da282387a4295c17f2851d on Rob--W:master.

Coverage decreased (-0.9%) to 99.115% when pulling 049bb95ec578c3d80d8cc72ca4fbd556f74808ce on chrischip:master into 4814647a9d21f14858da282387a4295c17f2851d on Rob--W:master.

Coverage decreased (-0.9%) to 99.115% when pulling 049bb95ec578c3d80d8cc72ca4fbd556f74808ce on chrischip:master into 4814647a9d21f14858da282387a4295c17f2851d on Rob--W:master.

Coverage decreased (-0.9%) to 99.115% when pulling 049bb95ec578c3d80d8cc72ca4fbd556f74808ce on chrischip:master into 4814647a9d21f14858da282387a4295c17f2851d on Rob--W:master.

Coverage decreased (-0.9%) to 99.115% when pulling 049bb95ec578c3d80d8cc72ca4fbd556f74808ce on chrischip:master into 4814647a9d21f14858da282387a4295c17f2851d on Rob--W:master.

This is not an issue with Node.js or this application. Your server (in front of Node.js) is mangling the URL.
The real fix is to prevent the server from normalizing URLs (e.g. replacing // with /), because there may be other URLs that are unexpectedly changed.
If you can't do that, a common solution is to omit the scheme. By default, http: is used. If you want to use https, use port 443. Examples:

https://cors-anywhere.herokuapp.com/example.com/ is the same as
https://cors-anywhere.herokuapp.com/http://example.com/
https://cors-anywhere.herokuapp.com/example.com:443/ is the same as
https://cors-anywhere.herokuapp.com/https://example.com:443/
(which is effectively the same as https://cors-anywhere.herokuapp.com/https://example.com/ ).

The problem is in the hosting environment like Azure where you have no control. Also the URL is from the users which you also have no control. I fully understand your point of view but in reality things are not perfect and the code should be flexible enough to adapt..... Thanks for your review anyway. Appreciate your good work.


Discrepancy in cytokine network expression data sources

@heimannch wrote
Here it goes a csv file with the cases of different values of gene expression across two different sources. I kept all ParticipantBarcodes that had at least one case of mismatch.
Each gene has two columns, and the source of the gene expression data is encoded as follows:
.x = gene expression data from shiny-iatlas repo object panimmune_data$im_expr_df
.y = gene expression data from file "GenExp_All_CKine_genes.tsv", from the cytokine-network-dev repo.
expr_mismatched.csv.gz

First test case, BTLA gene , ParticipantBarcode TCGA-ZU-A8S4 has values 6.5879 and 4.4871 respectively for x and y
SELECT *
FROM `pancancer-atlas.Filtered.EBpp_AdjustPANCAN_IlluminaHiSeq_RNASeqV2_genExp_filtered`
where Symbol="BTLA" and ParticipantBarcode="TCGA-ZU-A8S4" and SampleTypeLetterCode="TP"
LIMIT 1000

gives 4.4871

Might be related to doubled rows in im_expr_df (thought to have been addressed earlier)
> filter(im_expr_df,ParticipantBarcode=="TCGA-ZU-A8S4",Symbol=="BTLA")
# A tibble: 2 x 3
  ParticipantBarcode Symbol normalized_count
  <chr>              <chr>             <dbl>
1 TCGA-ZU-A8S4       BTLA               8.69
2 TCGA-ZU-A8S4       BTLA               4.49

The value 8.69 differs from 6.59 above, however.

(8.69+4.49)/2
[1] 6.59

On Fri, Nov 1, 2019 at 5:08 PM Vésteinn Þórsson<EMAIL_ADDRESS>wrote:

Might be related to doubled rows in im_expr_df (thought to have been
addressed earlier)

filter(im_expr_df,ParticipantBarcode=="TCGA-ZU-A8S4",Symbol=="BTLA")

A tibble: 2 x 3
ParticipantBarcode Symbol normalized_count

1 TCGA-ZU-A8S4       BTLA               8.69
2 TCGA-ZU-A8S4       BTLA               4.49
The value 8.69 differs from 6.59 above, however.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/CRI-iAtlas/shiny-iatlas/issues/155?email_source=notifications&email_token=AAEJFSJPSZXWO55XJZCT3LLQRTAJRA5CNFSM4JH6YLN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC4OQQI#issuecomment-548988993,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEJFSOI4PRBJU2TF6KVJJDQRTAJRANCNFSM4JH6YLNQ
.

A looks like this was addressed already
https://github.com/CRI-iAtlas/shiny-iatlas/issues/143
File did maybe not make it in.
@heimannch , would you be able to double check if the corrected im_expr_df solves this?

Yes, the gene expression data from issue #143 matches the gene expression data from file "GenExp_All_CKine_genes.tsv", from the cytokine-network-dev repo.
I will update the data for the cytokine network module - @andrewelamb do you want me to update this in the repo too?

I just took a closer look, and there are still doubled rows in the file attached in #143 - but at least one of the occurrences is equal to the one in the file "GenExp_All_CKine_genes.tsv".

Oh, drat! Thanks @heimannch . There should not be double rows at all. I will take a look.

I compiled the whole list of duplicated rows in the im_expr_df.feather.gz file from issue #143
The example query in the original issue is still with a problem:
> filter(expr_data2,ParticipantBarcode=="TCGA-BH-A1FC",Symbol=="IL13")
# A tibble: 2 x 3
  ParticipantBarcode Symbol normalized_count
  <chr>              <chr>             <dbl>
1 TCGA-BH-A1FC       IL13              0.588
2 TCGA-BH-A1FC       IL13              6.51

We also have some cases with three normalized counts, for example:
>filter(expr_data2, Symbol == "BTLA", ParticipantBarcode == "TCGA-06-0211")
# A tibble: 3 x 3
  ParticipantBarcode Symbol normalized_count
  <chr>              <chr>             <dbl>
1 TCGA-06-0211       BTLA               2.00
2 TCGA-06-0211       BTLA               3.63
3 TCGA-06-0211       BTLA               5.34

dupl_im_expr.csv.gz

Thanks, makes me think I uploaded the wrong file. Will look

@heimannch , can this be closed now?

Sure!


read-only/write-only properties

Read-only properties should only appear in asserts.
Write-only properties should only appear in request bodies

fixed by:
https://github.com/PatOConnor43/heave/releases/tag/v0.13.0


Using accumulators in BeamOperations

Context
Currently the development of Accumulator Framework is in progress. The idea is that Accumulator Framework is independent of the pipeline framework. This issue is about implementing usage of Accumulator Framework with Beam.
The interface of accumulators:
class CompoundAccumulator(Accumulator):
      def merge(cls,  accumulators: Iterable[Accumulator]) -> Accumulator
      def add_accumulator(self, accumulator:Accumulator) -> Accumulator # returns itself
      def compute_metrics()

Goals
To implement the following method of BeamOperations:
def reduce_accumulators_per_key(self, col):
where col contains elements:  (key, accumulator)
and the return is a collection of elements (key, accumulator), such that all elements per each key are merged.
Notes

Here (example 5) is an example of creating a custom CombinFn function (the idea is to use a Custom combine function)
In Beam CombinePerKey is slightly more general that we need, namely input and output elements might have different types, in our cases input and output are accumulators (which do some DP aggregations). In order to avoid confusions reduce is used instead of combine.

It was implemented on PR


ERROR: Service 'django' failed to build

Trying to install orochi on Ubuntu 20.04 VM. Get following error during docker-compose build / up.
Step 16/42 : WORKDIR /tmp/yara-python
 ---> Using cache
 ---> aa786afe3c8b
Step 17/42 : RUN python setup.py build
 ---> Running in 072ffdba7475
/usr/local/lib/python3.10/site-packages/setuptools/config/setupcfg.py:463: SetuptoolsDeprecationWarning: The license_file parameter is deprecated, use license_files instead.
  warnings.warn(msg, warning_class)
running build
running build_ext
building 'yara' extension
<--snip-->
<--snip-->

error: command '/usr/bin/gcc' failed with exit code 1
ERROR: Service 'django' failed to build: The command '/bin/sh -c python setup.py build' returned a non-zero code: 1

We're following current issue in yara-python https://github.com/VirusTotal/yara-python/issues/212
At the moment you should be able at least to run our images, are available since some month and are working also if don't have latest updated dependencies.
So with docker-compose pull and then docker-compose up at least you should be able to use and try the tool.

So with docker-compose pull and then docker-compose up at least you should be able to use and try the tool.

Thank you @garanews But I get same error with docker-compose pull  and then docker-compose up
ubuntu:~/orochi$ sudo docker-compose pull
WARNING: The http_proxy variable is not set. Defaulting to a blank string.
WARNING: The https_proxy variable is not set. Defaulting to a blank string.
WARNING: Some services (worker) use the 'deploy' key, which will be ignored. Compose does not support 'deploy' configuration - use `docker stack deploy` to deploy to a swarm.
 Defaulting to a blank string.
WARNING: The https_proxy variable is not set. Defaulting to a blank string.
WARNING: Some services (worker) use the 'deploy' key, which will be ignored. Compose does not support 'deploy' configuration - use `docker stack deploy` to deploy to a swarm.
Building django
Step 1/42 : FROM python:3.10-slim-buster as common-base
 ---> 8addbbd4348f

<--snip-->
<--snip-->

Step 17/42 : RUN python setup.py build
 ---> Running in df2316ccb809
/usr/local/lib/python3.10/site-packages/setuptools/config/setupcfg.py:463: SetuptoolsDeprecationWarning: The license_file parameter is deprecated, use license_files instead.
  warnings.warn(msg, warning_class)
running build
running build_ext
building 'yara' extension
creating build
creating build/temp.linux-x86_64-cpython-310
creating build/temp.linux-x86_64-cpython-310/yara
creating build/temp.linux-x86_64-cpython-310/yara/libyara
creating build/temp.linux-x86_64-cpython-310/yara/libyara/modules
creating build/temp.linux-x86_64-cpython-310/yara/libyara/modules/console
creating build/temp.linux-x86_64-cpython-310/yara/libyara/modules/demo
creating build/temp.linux-x86_64-cpython-310/yara/libyara/modules/elf
creating build/temp.linux-x86_64-cpython-310/yara/libyara/modules/hash
creating build/temp.linux-x86_64-cpython-310/yara/libyara/modules/math
creating build/temp.linux-x86_64-cpython-310/yara/libyara/modules/pe
creating build/temp.linux-x86_64-cpython-310/yara/libyara/modules/pe/authenticode-parser
creating build/temp.linux-x86_64-cpython-310/yara/libyara/modules/tests
creating build/temp.linux-x86_64-cpython-310/yara/libyara/modules/time
creating build/temp.linux-x86_64-cpython-310/yara/libyara/proc
creating build/temp.linux-x86_64-cpython-310/yara/libyara/tlshc
gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -fPIC -D_GNU_SOURCE=1 -DUSE_LINUX_PROC=1 -DHAVE_STDBOOL_H=1 -DHAVE_MEMMEM=1 -DHASH_MODULE=1 -DHAVE_LIBCRYPTO=1 -Iyara/libyara/include -Iyara/libyara/ -I. -I/usr/local/include/python3.10 -c yara-python.c -o build/temp.linux-x86_64-cpython-310/yara-python.o -std=c99
yara-python.c: In function ‘convert_object_to_python’:
yara-python.c:479:27: warning: comparison of integer expressions of different signedness: ‘int64_t’ {aka ‘long int’} and ‘long long unsigned int’ [-Wsign-compare]
       if (object->value.i != YR_UNDEFINED)

With docker-compose pull no need to build the images, they are downloaded from repo, like do:
docker pull ghcr.io/ldo-cert/orochi_django:latest
docker pull ghcr.io/ldo-cert/orochi_worker:latest
docker pull ghcr.io/ldo-cert/orochi_postgres:latest
...
...

Anyway if you want to build, at the moment need to use v4.2.x branch of yara-python , to do so edit those 2 lines
https://github.com/LDO-CERT/orochi/blob/master/compose/local/django/Dockerfile#L50
https://github.com/LDO-CERT/orochi/blob/master/compose/local/dask/Dockerfile#L35
replacing with:
RUN git clone --branch v4.2.x --recursive https://github.com/VirusTotal/yara-python

Hi @garanews Thank you for the help. I do not want to build it, but test existing features. As you can see, after docker-compse pull, I did docker-compose up as suggested, but it went to buid stage on it's own :(
Is there a way to manually suppressing  the build in the docker yaml script, instead using downloaded images ? (sorry for unfamiliarity)
Thanks for your time.

Just tried with everything clean (no previous images built or created, just deleted everything with docker system prune -a).

By the way, which versions of docker and docker-compose are you using? Maybe you need to update those.

At the moment we rebuilt images with inside the pinned version of yara-python
git clone --branch v4.2.x --recursive https://github.com/VirusTotal/yara-python
The images build successfully.


[Bitcoin] Add a warning/error when user tried to send too small amount

Users shouldn't be able to send 1 satoshi. This would almost certainly be a mistake, as fees are like 15-20k sats.

This is resolved with https://github.com/hirosystems/stacks-wallet-web/pull/3179


euphoria-beam: consider creating an executor based on apache beam

It'd be interesting to see an executor, which merely translates a euphoria flow into to the Apache Beam program. Euphoria programs could thus gain the potential to execute on engines supported by Apache Beam. Further, it could show the main differences between the two APIs and potentially uncover conceptual unsoundness in one or the other API.

Duplicate of #152


Remove kFromInitiator flags

Problem
They don't make sense as API (esp. in SendFlags), because the exchange context already knows that information.  Which is why they are unused.
Proposed Solution
Remove the flags.

Given this is an old issue, is this still present? If so, please re-open, thanks!

Still an issue.  Will just fix.


Setting a foreground blocks taps when scrolled

Issue details
When setting a foreground an empty view is created that acts as a placeholder and wrapper.
This has position: 'absolute' and the height according to the parallaxHeight property.
Because of this, the empty view is always in front of everything that renders as the content of the ScrollView.
          parallaxHeight={300}

          renderParallaxForeground={({ animatedValue, height }) => (
            <View style={{backgroundColor: 'red', width: '100%', height, opacity: 0.5}} />
          )}

slightly scrolled and selected with the React-Native inspector:

Everything below the box is not touchable, i.e. the "Alle Anzeigen" button

react-native 0.55.0
react-native-parallax-scroll 1.7.0

Affected platforms

Both

PR incoming

While the PR #24  does not fix it per se, it allows displaying headers that are supposed go with the scroll offset.
In theory the size of the view must be animated, however this blocks useNativeDriver as it cannot animate height outside of the Javascript engine.

@EyMaddis can I close this issue?

Well, technically it is not solved, but a workaround is feasible (at least for me).
I am fine with closing as long as you don‘t have an idea how to properly fix this.

Maybe this can be properly solved by applying the foreground translation to the placeholder view or container instead of only the foreground. Unfortunately, I do not have time at the moment to look into this.

Hi @EyMaddis, I just tried to reproduce your problem and it looks like everything works fine. The foreground wrappers have poinerEvents="box-none" prop, which means that touches won't work for this views, but will work for children. I think this problem can be solved in the client side app. What do you think about it?

I think this problem can be solved in the client side app

What do you mean? Client side meaning "by the user with renderPlaceholderView"?
The wrapper has box-none, but the empty view does not. Maybe the default placeholder view should also get this property.

Yep

What do you mean? Client side meaning "by the user with renderPlaceholderView"?

Will add this later. For now, I close this issue

Maybe the default placeholder view should also get this property.


Avoid index after query seq name & Deal with MPI parameter.

Hi,
Here are the following settings I used in order to get a .m8 file.
mmseqs2=/beegfs/home/TOOLS/mmseqs2/bin/mmseqs
$mmseqs2 createdb\
 /path/species.fa\
 /path/species_mmseqs2_db
$mmseqs2 search\
 /path/species_mmseqs2_db\
 /path1/path2/refseq/viral_queries_seq/mmseq_db\
 /path1/path2/species/result_mmseqs2\
 /path1/path2/species/tpm -a -s 7.5 -e 0.1
$mmseqs2 convertalis\
 /path/species_mmseqs2_db\
 /path1/path2/refseq/viral_queries_seq/mmseq_db\
 /path1/path2/species/result_mmseqs2\
 /path1/path2/species/result_mmseqs2_db.m8

and here is the head file.m8 I get:
scaffold_2855_1	YP_009173613.1	0.742	377	96	1	55020	53895	70	446	1.803E-180	566
scaffold_2855_1	YP_004091438.1	0.729	377	101	1	55020	53895	66	442	2.282E-173	546
scaffold_2855_1	YP_007676312.1	0.721	377	104	1	55020	53895	65	441	4.321E-170	536
scaffold_2855_0	YP_093986897.1	0.683	376	119	0	55020	53895	67	442	3.184E-167	528
scaffold_2855_2	YP_009894551.1	0.672	376	123	0	55020	53895	67	442	3.210E-164	519

Here is my question.
First, do you know how could I avoid to add the index _number after the query's name? such as scaffold_2855_1 or scaffold_2855_0 or scaffold_2855_2 etc.
Second question:
I did not really understand the MPI parameters in order to be faster. I have actually a SLURM server where I submit all my jobs (so actually it is one job with 20 threads per run), if I well understood I can run the program from different machines by using this parameter? How could I do with the settings I used above?
Thank you for you help.

For MPI you have to compile MMseq2 yourself with MPI development headers/libraries installed and cmake -DHAVE_MPI=1 .... The -MPI version suffix on any MMseqs2 module call will show up if you compiled it correctly. Then you have to define one environment variable RUNNER that contains your MPI runner call. E.g.:
export RUNNER="mpirun --pernode

Make sure to start only one MPI process per compute node (--pernode parameter in this case). The parallelization within a node still happens over OpenMP.
MMseqs2 workflows (search, cluster, ...) will automatically use the $RUNNER environment variable to parallelize modules that support it (e.g. prefilter, align, ...).
If you call MMseqs2 modules directly (e.g. again prefilter) you have to do a normal MPI call:
mpirun --pernode mmseqs prefilter ...

We don't really have a list of MPI enabled MMseqs2 modules, except for looking in the code for modules that call MMseqsMPI::init.

Ok thank you for your time, I was really helpful.
About the  index _number added at each queries' names, have you an idea to avoid a such adding? because I have to get the fasta sequences after this process and for that I need the exact scaffold's name, without an index added.

What exactly are you doing there? Is this a nucl-nucl or nucl-protein search? MMseqs2 will usually take characters until the first whitespace in the FASTA header as the identifier (if it does not recognize the format, it has a separate parsing logic for e.g. Uniprot format etc). I'd suggest you write a simple awk script that trims the suffix.

It is a blastx like run so a genome nucl query and a protein database. Yes that is what I thought to do but maybe there where another way directly in the parameters.

So the _N suffix comes already form your FASTA files?
If thats the case: We already have a feature request in #82 for being able to adapt the parsing logic for FASTA headers. However none had time to work on that yet. We will keep that in mind for the future.

No, as you can see for instance I got a result such as:
scaffold_770_2	979	NP_068312.1	0.439	41	22	1	30349	30469	469	508	5.595E-04	38
and when I do a grep 'scaffold_770_2' /beegfs/data/me/genome23.fa
I get nothing but on the other hand, scaffold_770  without the suffix does exist. It seems that it is during the mmseqs2 process that there was an addition of the _N suffix.

What version are you using? In the past we Splitted sequences but we stopped this in later versions.

It is this one MMseqs2 Version: 4e23d5f1d13a435c7b6c9406137ed68ce297e0fc

I think the feature was introduced after this commit. Could you rerun your example with the newest version?

It did work with the last version thanks a lot for your help.


python311Packages.triton build failure on x86_64-linux as of a4e8dd38

Issue description
Build of python311Packages.triton failed on x86_64-linux as of a4e8dd38d1bcd37deee4000b22fa1b92c0824471. This is currently breaking python3Packages.torchvision.
error: builder for '/nix/store/ca4dhkv72m0ijgxzhdqzkcmd902fl5ry-python3.11-triton-2.1.0.drv' failed with exit code 1;
       last 10 log lines:
       >     super().run_command(command)
       >   File "/nix/store/7fnhc3nd125lq9vzw6bghsvbnmcc8mc1-python3.11-setuptools-69.0.2/lib/python3.11/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
       >     cmd_obj.run()
       >   File "<string>", line 193, in run
       >   File "<string>", line 262, in build_extension
       >   File "/nix/store/asiphbpiy2gmidfm3xbwcikayhs66289-python3-3.11.7/lib/python3.11/subprocess.py", line 413, in check_call
       >     raise CalledProcessError(retcode, cmd)
       > subprocess.CalledProcessError: Command '['cmake', '--build', '.', '--config', 'TritonRelBuildWithAsserts', '-j8']' returned non-zero exit status 2.
       >
       > ERROR Backend subprocess exited when trying to invoke build_wheel
       For full logs, run 'nix log /nix/store/ca4dhkv72m0ijgxzhdqzkcmd902fl5ry-python3.11-triton-2.1.0.drv'.
error: 1 dependencies of derivation '/nix/store/z5ykkjccifj1j5fk34s5vprzg9hkxhjk-python3.11-torch-2.1.2.drv' failed to build
error: 1 dependencies of derivation '/nix/store/h9jsw9ff4ys79kl6cg19s8lkvhv8mnl0-python3.11-torchvision-0.16.2.drv' failed to build

full build log
python3Packages.torchvision maintainers cc: @ericsagnes
python311Packages.triton maintainers cc:
Other cc: n/a
Steps to reproduce

Checkout nixpkgs at commit a4e8dd38d1bcd37deee4000b22fa1b92c0824471
Run nix-build -A python311Packages.triton

Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 6.2.0-1018-azure, Ubuntu, 22.04.3 LTS (Jammy Jellyfish), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.16.1`
 - nixpkgs: `/nix/store/lx9nr76jv520f1nzlppj9fmrkgiinwax-source`

Contents of ~/.config/nixpkgs/config.nix:
{ allowUnfree = true; cudaSupport = true; cudaCapabilities = ["8.0"]; }

Misc.
This issue was automatically generated by nixpkgs-upkeep.

CI workflow that created this issue.
Internal tags:<PHONE_NUMBER>67875719<PHONE_NUMBER>297623484

fixed in master


Search - No tooltip when hovering over the search icon

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.50-0
Reproducible in staging?: Y
Reproducible in production?: N/A - new feature, doesn't exist in prod
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Go to staging.new.expensify.com
Open any report.
Hover over search icon on the top right.

Expected Result:
There will be tooltip when hovering over the search icon.
Actual Result:
There is no tooltip when hovering over the search icon.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/10cc26b1-4f1e-4f2d-a152-fff2b65a798a

View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Search - No tooltip when hovering over the search icon
What is the root cause of that problem?
We don't wrap this icon in a Tooltip component
What changes do you think we should make in order to solve the problem?

We should wrap this icon in a Tooltip component with text is search
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
What is the root cause of that problem?
The SearchButton component does not have a tooltip:
https://github.com/Expensify/App/blob/935067ed56dd45bd24edd3ce834edc2dd112bc7d/src/components/Search/SearchRouter/SearchButton.tsx#L26
What changes do you think we should make in order to solve the problem?

Wrap Tooltip around whose content can be "Find"
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

Not a blocker. Commented on the offending PR. Would allow PR author to fix it

Hey! I’m Jakub Szymczak from Software Mansion, an expert agency, and since I implemented Search 2.4 I’d like to work on this issue!

I assume we're going to try and have @rayane-djouah review this one too, as they were C+ on the PR that caused the regression as well?

Yes, I assgined @rayane-djouah to the issue too.

Great. TY!

PR out on staging

@rayane-djouah can you complete BZ here so we can prep for payment?

We only need to pay @rayane-djouah $250 here for C+ via Upwork.

All set. Thanks everyone!


Update custom pricing helm chart setup

With the addition of https://github.com/opencost/opencost-helm-chart/pull/109, the instructions for setting up custom pricing with the helm chart could use a refresh.

Good call, @mattray would you want to take a look?


Generate CLI for 2024-04-01 for StackHCI

Preconditions

[X] No need to upgrade Python SDK or the Python SDK is ready.

Related command
No response
Resource Provider
Microsoft.AzureStackHCI
Description of Feature or Work Requested
Generate CLI for 2024-04-01 for StackHCI and get it thoroughly tested
Minimum API Version Required
2024-04-01
Swagger PR link / SDK link
https://github.com/Azure/azure-rest-api-specs/tree/main/specification/azurestackhci/resource-manager/Microsoft.AzureStackHCI/StackHCI/stable/2024-04-01
Request Example
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/azurestackhci/resource-manager/Microsoft.AzureStackHCI/StackHCI/stable/2024-04-01/examples/CreateArcIdentity.json
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/azurestackhci/resource-manager/Microsoft.AzureStackHCI/StackHCI/stable/2024-04-01/examples/DeleteExtension.json
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/azurestackhci/resource-manager/Microsoft.AzureStackHCI/StackHCI/stable/2024-04-01/examples/PutArcSetting.json
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/azurestackhci/resource-manager/Microsoft.AzureStackHCI/StackHCI/stable/2024-04-01/examples/GetCluster.json
Target Date
15-07-2024
PM Contact
@thomaspf
Engineer Contact
@priyjain, @nikitasingh
Additional context
No response

Thank you for opening this issue, we will look into it.


To do: Add sample project to modify GPIO status with WebAPI

Done - hosted at https://github.com/jeremylindsayni/Bifrost/tree/master/Bifrost.Samples/GpioSwitcherWebApi.


[suggestion] v-model implementation?

Thanks for your great library!
I am really happy to use it.
This is not a problem but just a suggestion.
For reactive data binding, I wrote the following code:
<template>
  <star-rating
      :rating="rating"
      @rating-selected="value => {rating = value}">
  </star-rating>
</template>

<script>
  new Vue({
    data() {
      return { rating: 3 }
    }
  })
</script>

How about to set reactive v-model attribution?
The code should be like this, simply:
<template>
  <star-rating v-model="rating"></star-rating>
</template>

I think we can do it by adding value prop to star-rating component and emit input event with the argument this.selectedRating.
If you accept this feature, I will try to code.
Thanks!

Absolutely, I think that's a much cleaner solution. Feel free to submit a PR, but if you don't have the time I'll definitely add this.

Thanks for your quick response!
I will write some code during this weekend.


Add blurhash.cr

Link
https://github.com/Sija/blurhash.cr
Checklist

[x] - Shard is at least 30 days old.
[x] - Shard has CI implemented.
[x] - Shard has daily/weekly periodic builds (ideally with Crystal latest and nightly).

@Blacksmoke16 Why did you modify my description, without my knowledge nor consent? In addition quite wrongly I have to add, because the shard is 30 days old (https://github.com/Sija/blurhash.cr/commit/2db54e2072e81c2e852476e9972bc0a31c681294). I take it as a quite rude behaviour from your side.

Oh, then I wonder whats up with github.

Shows the initial commit was 17 hours ago.  Yet, when you click on the commit

It says it was on Feb 26?

@Blacksmoke16 I was talking about your behavior, not the GitHub, to which you seem to want to direct the attention atm.

@Sija I think there is some lack of testing. Imho specs are required for encode method.

@veelenga Thanks for reminding, I've amended those.


🧑💻: Create the Signup page[frontend + backend]

:red_circle: SignUp page :
:red_circle: Enhancement Aim :
:red_circle: Changes :
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name : Arunima Dutta
What is your participant role? (Mention the Open Source Program name. Eg. GSOC, GSSOC, SSOC, JWOC, etc.)
gssoc

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@arunimaChintu Sir assign me this issue i have worked on Signin/Signup page for many Repo's.


UsernameRequiredException in Guard.php: A username must be specified.

Laravel Version: 5.4.21
Adldap2-Laravel Version: 3.0.4
PHP Version: 5.6.24

Description:
I have installed via the quick setup documentation, when I attempt to login with a test user I receive an error: UsernameRequiredException in Guard.php line 110: A username must be specified.
Below is the full error text. I notice that in Resolver->authenticate a username value is present yet at Guard->attempt the value is null... I followed the install instructions and even did a complete  reinstall follow the quick start a second time in the event that I had over looked something.
in Guard.php line 110
at Guard->validateCredentials(null, '##########') in Guard.php line 34
at Guard->attempt(null, '##########') in Resolver.php line 67
at Resolver->authenticate(object(User), array('username' => 'Test3', 'password' => '##########')) in DatabaseUserProvider.php line 132
at DatabaseUserProvider->validateCredentials(object(User), array('username' => 'Test3', 'password' => '##########')) in SessionGuard.php line 380
at SessionGuard->hasValidCredentials(object(User), array('username' => 'Test3', 'password' => '##########')) in SessionGuard.php line 357
at SessionGuard->attempt(array('username' => 'Test3', 'password' => '##########'), false) in AuthenticatesUsers.php line 77
at LoginController->attemptLogin(object(Request)) in AuthenticatesUsers.php line 41
at LoginController->login(object(Request))
at call_user_func_array(array(object(LoginController), 'login'), array(object(Request))) in Controller.php line 55
at Controller->callAction('login', array(object(Request))) in ControllerDispatcher.php line 44
at ControllerDispatcher->dispatch(object(Route), object(LoginController), 'login') in Route.php line 204
at Route->runController() in Route.php line 160
at Route->run() in Router.php line 559
at Router->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 30
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in RedirectIfAuthenticated.php line 24
at RedirectIfAuthenticated->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in SubstituteBindings.php line 41
at SubstituteBindings->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in VerifyCsrfToken.php line 65
at VerifyCsrfToken->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in ShareErrorsFromSession.php line 49
at ShareErrorsFromSession->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in StartSession.php line 64
at StartSession->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in AddQueuedCookiesToResponse.php line 37
at AddQueuedCookiesToResponse->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in EncryptCookies.php line 59
at EncryptCookies->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 102
at Pipeline->then(object(Closure)) in Router.php line 561
at Router->runRouteWithinStack(object(Route), object(Request)) in Router.php line 520
at Router->dispatchToRoute(object(Request)) in Router.php line 498
at Router->dispatch(object(Request)) in Kernel.php line 174
at Kernel->Illuminate\Foundation\Http\{closure}(object(Request)) in Pipeline.php line 30
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in TransformsRequest.php line 30
at TransformsRequest->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in TransformsRequest.php line 30
at TransformsRequest->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in ValidatePostSize.php line 27
at ValidatePostSize->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in CheckForMaintenanceMode.php line 46
at CheckForMaintenanceMode->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 102
at Pipeline->then(object(Closure)) in Kernel.php line 149
at Kernel->sendRequestThroughRouter(object(Request)) in Kernel.php line 116
at Kernel->handle(object(Request)) in index.php line 54

Steps To Reproduce:
Try to login using username  from a site visitors prospective.

Hi @wv-it, it sounds like the attribute you're using to authenticate users by is empty.
Are you using OpenLDAP or Active Directory?
Could you also post your config/adldap_auth.php file?

Active Directory
Here is adldap_auth.php
<?php

return [

    'connection' => env('ADLDAP_CONNECTION', 'default'),
    'provider' => Adldap\Laravel\Auth\DatabaseUserProvider::class,
    'resolver' => Adldap\Laravel\Auth\Resolver::class,
    'importer' => Adldap\Laravel\Auth\Importer::class,
    'rules' => [
        Adldap\Laravel\Validation\Rules\DenyTrashed::class,
    ],

    'scopes' => [
        Adldap\Laravel\Scopes\UpnScope::class,
    ],

    'usernames' => [
        'ldap' => 'sAMAccountName', // Was userprincipalname
        'eloquent' => 'username', // Was email

    ],
    'login_fallback' => env('ADLDAP_LOGIN_FALLBACK', false),
    'password_sync' => env('ADLDAP_PASSWORD_SYNC', true),
    'windows_auth_attribute' => ['samaccountname' => 'AUTH_USER'],=
    'sync_attributes' => [
        'username' => 'sAMAccountName',
        'email' => 'userprincipalname',
        'name' => 'cn',
    ],
];

Can you lowercase sAMAccountName to samaccountname and see if this resolves your issue?

That resolved it, thanks. Don't know how I missed trying that in my initial trouble shooting. :open_mouth:

Perfect! Glad you've resolved it.
To be honest, I forgot that retrieving attributes we're case sensitive :smile:
I'll be sure to include this inside the documentation.
Thanks for following up quickly.

And thank you, for responding so quick.


[INGNORE THIS PR] Add log entries in AT-Log for action/trigger invocations in case of errors

removed

removed


Consider whether to suppress local authorities with low numbers of young people

Closing #306 as won't do - we can pick this up in the next phase if needed.


Interactive Filter Overlaps with Meta Data Details

When activating interactive filters (for example here the "show in %" filter) it overlaps with the Meta Data Details of the cube and is not toggleable in edit and also publish mode.

¨
https://int.visualize.admin.ch/de/v/UOtn0J4gn7kY
To Reproduce Steps to reproduce the behavior:

Go to add a segmentation and choose "Treibhausgas" for example
Click on the interactive filter options for %
See error.

Expected behavior A clear and concise description of what you expected to
happen.
They should not overlap
Tested on Edge web browser

Hi @tboeni
Thank you for reporting the issue. We already have this on our radar and are tackling it in this issue: https://github.com/visualize-admin/visualization-tool/issues/1818

@tboeni this is fixed on INT and ready for review


ORC-693: Update credential according to INFRA setup

What changes were proposed in this pull request?
This PR updates the credentials according to the direction of https://issues.apache.org/jira/browse/INFRA-21150 .
Why are the changes needed?
To publish snapshot.
How was this patch tested?
N/A (The snapshot publish job is triggered at commit only).

Could you review this, @pgaref ? This will work.


Expressionproperties change model through observable

Hello,
I see in the interface that expressionproperties take either a string, callback or an observable.
when using expressionproperties to change templateOptions i can succesfully provide an observable and it works.
But when i try to change the model through expressionproperties, i can't provide an observable.
Is this intended?
For example:
expressionProperties: {
        'model.text': of("test"),
        'templateOptions.label': of('labelObs'),
}

the 'templateOptions.label' does show the label correctly as labelObs.
The 'model.text' does not show 'test' as value inside the input..
i can only effectively change the model via expressionproperties by providing a callback like so;
'model.text': ( ) => 'test'
so is this the only options to change the model through expressionproperties?
Tnx!

This issue has been fixed and released as part of v5.5.12 release.
Please let us know, in case you are still encountering a similar issue/problem.
Thank you!

many thanks for the quick response! This will help us greatly in our project


Update tokio-tungstenite + rustls + rustls-pemfile

In test-helpers/Cargo.toml we depend on tokio-tungstenite from a shotover specific git dependency.
This is a maintenance hazard, so we need to swap this over to a crates.io dependency.
Thankfully, just recently, the changes we needed in our tokio-tungstenite fork have made their way to an upstream tokio-tungstenite release in version 0.23.0
So we should now upgrade tokio-tungstenite to 0.23 this can be done in both shotover/Cargo.toml and test-helpers/Cargo.toml.
However, because:

tokio-tungstenite exposes some rustls types in its public API
and, tokio-tungstenite 0.23 uses a newer version of rustls than was used in our old git fork

The test-helpers crate will fail to compile with rustls 0.21 and tokio-tungstenite 0.23.
To resolve this we must upgrade rustls to 0.23 and tokio-tungstenite to 0.23 at the same time.
However the rustls upgrade from 0.21 to 0.23 has a fair few breaking changes that we need to resolve.
We already performed this upgrade for shotover/Cargo.toml so you can refer to https://github.com/shotover/shotover-proxy/pull/1387 for a reference of how to make the required changes.
The changes for rustls are a bit complex but since there is an example PR to refer to I think this could be done by a newstarter.
Additionally rustls-pemfile requires the same upgrade since it depends on rustls in a similar way to tokio-tungstenite.
TL;DR
Upgrade tokio-tungstenite, rustls and rustls-pemfile for all crates in the repo to their latest versions.
https://github.com/shotover/shotover-proxy/pull/1387 will be a useful reference for the code changes required.

closed by https://github.com/shotover/shotover-proxy/pull/1712


Can't Create Area With Tools

Description
Areas don't generate when using the "Line" or "Area" tool:
(1) When using the "Area" tool:

No graphics are generated at all. When clicking on the map to start the edit, the starting node doesn't appear, and clicking again to place the next node just ends the edit with nothing produced. After the edit is 'done', the "Save" button highlights as if a feature was created.

(2) When using the "Line" tool:

The last segment of a polygon does not generate.
Ex: Only 3 sides of a square generate after double-clicking on the start node to complete the shape.

Version
2.0-alpha
What browser are you seeing the problem on? What version are you running?
Opera GX [LVL4 {core: 89.0.4447.98) (x86_64)}
Steps to reproduce
(1) Select the Area tool - click on the map - click on the map at different location ~ tool stops.
(2) Select Line tool - click on the map - continue to draw a square and double-click (on start node) to complete shape ~ generates incomplete shape.
The browser URL at the time you encountered the bug
https://mapwith.ai/rapid-v2-alpha#background=Bing&datasets=fbRoads,msBuildings&map=21.33/34.00626/-117.92483

Since this mode isn't implemented yet I just pushed up a change to disable the mode and call out that it isn't implemented yet in the tooltip:

Closing since we already have bug #519 for fixing this functionality.


problem loading a component that uses wasi?

I'm trying to get a working example of using a component with wasi from python.
I have a basic component created using cargo component:
cargo component new --lib mycomponent

I then build the component:
cargo component build

this builds to the default wasm32-wasip1 target, and I copy the .wasm file to the root of my python project.
The python code is:
import traceback
from wasmtime import Store, loader

from volt_ws_protocol_component import Root

def main():
    try:
        store = Store()
        component = Root(store)
        hello_response = component.hello_world(store)
        print(hello_response)
    except Exception as e:
        print(f"Failed to call hello: {e}")
        traceback.print_exc()

if __name__ == "__main__":
    main()

When I run this I get the following error:
❯ python main.py
thread '<unnamed>' panicked at src/bindgen.rs:2911:29:
not implemented: imported resources not yet supported
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Traceback (most recent call last):
  File "/Users/tobyealden/code/tdxvolt/tdxvolt-rs/python_harness/main.py", line 4, in <module>
    from mycomponent import Root
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 982, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 925, in _find_spec
  File "/Users/tobyealden/code/tdxvolt/tdxvolt-rs/python_harness/volt_env/lib/python3.9/site-packages/wasmtime/loader.py", line 153, in find_spec
    _component_bindings[origin] = self.load_component(origin, binary=is_binary)
  File "/Users/tobyealden/code/tdxvolt/tdxvolt-rs/python_harness/volt_env/lib/python3.9/site-packages/wasmtime/loader.py", line 131, in load_component
    return bindgen.generate("root", component)
  File "/Users/tobyealden/code/tdxvolt/tdxvolt-rs/python_harness/volt_env/lib/python3.9/site-packages/wasmtime/bindgen/__init__.py", line 151, in generate
    result = root.generate(store, name, component)
  File "/Users/tobyealden/code/tdxvolt/tdxvolt-rs/python_harness/volt_env/lib/python3.9/site-packages/wasmtime/bindgen/generated/__init__.py", line 301, in generate
    ret = self.lift_callee0(caller, ptr, len0, ptr1, len2)
  File "/Users/tobyealden/code/tdxvolt/tdxvolt-rs/python_harness/volt_env/lib/python3.9/site-packages/wasmtime/_func.py", line 103, in __call__
    raise WasmtimeError._from_ptr(error)
  File "/Users/tobyealden/.pyenv/versions/3.9.19/lib/python3.9/contextlib.py", line 126, in __exit__
    next(self.gen)
  File "/Users/tobyealden/code/tdxvolt/tdxvolt-rs/python_harness/volt_env/lib/python3.9/site-packages/wasmtime/_func.py", line 265, in enter_wasm
    raise trap_obj
wasmtime._trap.Trap: error while executing at wasm backtrace:
    0: 0x21b39e - bindgen.wasm!__rust_start_panic
    1: 0x21b238 - bindgen.wasm!rust_panic
    2: 0x21af5c - bindgen.wasm!std::panicking::rust_panic_with_hook::h5aa58467fd511e58
    3: 0x21a34c - bindgen.wasm!std::panicking::begin_panic_handler::{{closure}}::hddafbb74422ab5b5
    4: 0x21a27c - bindgen.wasm!std::sys_common::backtrace::__rust_end_short_backtrace::hee1cd7216ef8c172
    5: 0x21aa64 - bindgen.wasm!rust_begin_unwind
    6: 0x228092 - bindgen.wasm!core::panicking::panic_fmt::hdff7d27056516045
    7: 0x171c6 - bindgen.wasm!<bindgen::bindgen::FunctionBindgen as wit_bindgen_core::abi::Bindgen>::emit::h371becfed4ef60ed
    8: 0x29fec - bindgen.wasm!wit_bindgen_core::abi::Generator<B>::emit::h73aacf1c591a9bab
    9: 0x2b069 - bindgen.wasm!wit_bindgen_core::abi::Generator<B>::lift::hd694126ede7ccf9e
   10: 0x314cc - bindgen.wasm!wit_bindgen_core::abi::call::h9562672c6a7e0a75
   11: 0xd849 - bindgen.wasm!bindgen::bindgen::Instantiator::bindgen::ha0d69afed3096c45
   12: 0xb6ce - bindgen.wasm!bindgen::bindgen::WasmtimePy::instantiate::hcaee39c0c215ca15
   13: 0x63d5 - bindgen.wasm!bindgen::bindgen::WasmtimePy::generate::h0d55c547f641a6d3
   14: 0x3ebec - bindgen.wasm!<bindgen::bindings::PythonBindings as bindgen::bindings::Guest>::generate::hb5b6ad11d2bacd63
   15: 0x3ed9f - bindgen.wasm!generate

Caused by:
    wasm trap: wasm `unreachable` instruction executed

Presumably I have to initialise or link wasi in some way - what is the idiomatic way of doing this when loading components? I couldn't find any documentation or example code.
Note that this works as expected if I use cargo component build --target wasm32-unknown-unknown, but I need to be able to use wasi in my component (e.g. to generate a random number).

Ah the error you're running into is in the output:
not implemented: imported resources not yet supported

Resources haven't been implemented in wasmtime-py at this time, which means components using WASI can't be easily loaded at this time. The component support in wasmtime-py is still quite underbaked and not really ready for prime-time.

OK, I thought this might be the case - can you give a ballpark ETA of when this might be supported?
Thanks for getting back to me so quickly.

No ETA at this time, I don't believe anyone's queued up to work on it


[ISSUE #1019]🐛Fix message queue changed sendHeartbeatToAllBrokerWithLockV2 faild

Which Issue(s) This PR Fixes(Closes)

Fixes #1019
Brief Description

How Did You Test This Change?

🚀Thanks for your contribution🎉. CodeRabbit(AI) will review your code first🔥


[Feature]  Allow to pass an url as src to CldVideoPlayer

Feature Request
Sets up the CldVideoPlayer to allow passing in a valid Cloudinary URL as a src
Example:
src="https://res.cloudinary.com/next-cloudinary/video/upload/v1/videos/mountain-stars.webm"
Similar to CldImage, the URL must include a version number /v1234/

SEe how we're doing it in Next Cloudinary with the Cloudinary Util function: https://github.com/colbyfayock/next-cloudinary/blob/main/next-cloudinary/src/components/CldVideoPlayer/CldVideoPlayer.tsx#L47

This is already implemented


knot-dns: 3.4.0 -> 3.4.1

https://gitlab.nic.cz/knot/knot-dns/-/releases/v3.4.1
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

SystemCallFilter is breaking it 😡

For reference, the filtered syscall looked like this in log:
secondary # Module knotd without build-id.
secondary # Stack trace of thread 714:
secondary # #0  0x00007f5248dad23b fchown (libc.so.6 + 0xfe23b)
secondary # #1  0x00007f5249498772 knot_creds_update (libknot.so.15 + 0x17772)
secondary # #2  0x00007f52494988ff knot_creds_init (libknot.so.15 + 0x178ff)
secondary # #3  0x000000000042579e init_creds (knotd + 0x2579e)
secondary # #4  0x000000000042692b configure_sockets (knotd + 0x2692b)
secondary # #5  0x0000000000426e71 server_reconfigure (knotd + 0x26e71)
secondary # #6  0x000000000040e632 main (knotd + 0xe632)
secondary # #7  0x00007f5248cd927e __libc_start_call_main (libc.so.6 + 0x2a27e)
secondary # #8  0x00007f5248cd9339 __libc_start_main@@GLIBC_2.34 (libc.so.6 + 0x2a339)
secondary # #9  0x0000000000410465 _start (knotd + 0x10465)
secondary # ELF object binary architecture: AMD x86-64
secondary #
secondary # [    8.772490] systemd[1]<EMAIL_ADDRESS>Deactivated successfully.
secondary # [    8.776134] systemd[1]<EMAIL_ADDRESS>Consumed 773ms CPU time, 537.8M memory peak.
secondary # [    8.779541] systemd[1]: knot.service: Main process exited, code=dumped, status=31/SYS
secondary # [    8.780677] systemd[1]: knot.service: Failed with result 'core-dump'.
secondary # [    8.784113] systemd[1]: Failed to start Authoritative-only DNS server from .cz domain registry.


vm-RancherOS image is very old

An updated vm-rancheros image with working console and login password would be great!

Any change to get an updated vm-RancherOS 0.4.5?

Will RancherVM be more flexible to add a custom image in the future (https://github.com/rancher/vm/pull/54)? If not there should be an updated RancherOS image prepared.


if timeout.tasks is reached pipeline status.reason should be PipelineRunTimeout

Expected Behavior
if the tasks of a pipelineRun time out because of the timeout.tasks parameter, the reason for the PipelinRun satus should be PipelineRunTimeout.
Actual Behavior
status reason is Failed.
Steps to Reproduce the Problem

have pipeline that has a sleep for ever task
configure a pipelinrun and set timeout.tasks = "1m"
inspect pipelinerun status = false, reason=failed.

Additional Info

Kubernetes version:
Output of kubectl version:

Client Version: v1.25.3
Kustomize Version: v4.5.7
Server Version: v1.25.16-eks-8cb36c9

Tekton Pipeline version:
Output of tkn version or kubectl get pods -n tekton-pipelines -l app=tekton-pipelines-controller -o=jsonpath='{.items[0].metadata.labels.version}'

Client version: 0.31.2
Chains version: v0.16.0
Pipeline version: v0.47.5
Triggers version: v0.24.0
Dashboard version: v0.35.0

I do think it make sense (if any Task fails due to Timeout, to mark the PipelineRun has failed due to Timeout).
cc @tektoncd/core-maintainers

I do think it make sense (if any Task fails due to Timeout, to mark the PipelineRun has failed due to Timeout). cc @tektoncd/core-maintainers

Yes, this would be similar to what TaskRun has for timeout behaviour.
And just double checking if there is gonna be any concern with the change cc @AlanGreene 🤔

No concerns from the Dashboard's perspective

/assign


Project indexoutofbound exception in public test case 4

Summary
Put a summary of the question
Description
hi I tested my code against the cat 4.in in codcrunch and it shows indexoutofbound error. the code says the error appears at Serve class when updating the queue. the index of queue is 2 where there should be only two queues (one human and one selfcheckout) hence the max index should be only 1. i checked my initialiseQueue and it is initilaised correctly. I am now confused if there is any other areas that i can check for? any suggestions would be appreciated!
Screenshots (if any):

have you checked the lines specified in the stacktrace?

if you used any loops to loop through the queues, maybe you can check if your terminating conditions are correct, e.g. if you used something like  index <= queue.size() instead of index < queue.size()

I had this error for test case 11 too. what I realised was sometimes my +- 1 had some error, maybe review those parts if u have?

can check through all the loops you did in Serve (maybe each human queue and self check out queue). perhaps you can add print statements on the size of the whatever you're looping through and using that, see if there's any clash in logic with the number of loops you're doing in all the loops. hope this helps :)

yepp i think it's cuz you're trying to access something that is outside of whichever list you're looping through!

can add some print statements in your updateQ to check which part goes wrong, the problem I had is the index I use to get queue went out of bound

thank everyone! my error appears to be i did not return a self checkout server in one of the update servers method so the servers were turned into human servers eventually.


Modern test helpers

It would be great to add drag and reorder test helpers using the modern ember test helper patterns. The outcome is that apps using “classic” (globals) test helpers can continue as-is whilst modern apps can import helpers as follows:

import { module, test } from 'qunit';
import { visit, currentURL } from '@ember/test-helpers';
import { reorder } from 'ember-sortable/test-helpers';
import { setupApplicationTest } from 'ember-qunit';

module('Acceptance | hello', function(hooks) {
  setupApplicationTest(hooks);

  test('visiting /hello', async function(assert) {
    await visit('/hello');
    await reorder(
      'mouse',
      'li',
      '[data-test-id="3"]',
      '[data-test-id="2"]',
      '[data-test-id="1"]'
    );

    assert.equal(currentURL(), '/hello');
  });
});

Here's one for drag using the modern test helpers and vanilla JS. However, if we want to keep the registerAsyncHelper for backward compatibility, we might have to rename the modern test helper, else there might be a name collision.
import { triggerEvent, settled, find } from '@ember/test-helpers';
/**
 * Vanilla Javascript equivalent of JQuery's `.offset`.
 * Reference: https://github.com/nefe/You-Dont-Need-jQuery
 *
 * @method getOffset
 * @param {Element} el an element
 */
function getOffset(el) {
  const box = el.getBoundingClientRect();

  return {
    top: box.top + window.pageYOffset - document.documentElement.clientTop,
    left: box.left + window.pageXOffset - document.documentElement.clientLeft,
  };
}

/**
 * A re-implementation of the drag test helper that is compatible with the new qunit syntax without `registerAsyncHelper`.
 *
 * Drags elements by an offset specified in pixels.
 * Examples
 *    drag(
 *      'mouse',
 *      '.some-list li[data-item=uno]',
 *      function() {
 *        return { dy: 50, dx: 20 };
 *      }
 *    );
 * @method sortableDrag
 * @param {'mouse'|'touch'} [mode]
 *   event mode
 * @param {String} [itemSelector]
 *   selector for the element to drag
 * @param {Function} [offsetFn]
 *   function returning the offset by which to drag
 * @param {Object} [callbacks]
 *   callbacks that are fired at the different stages of the interaction
 * @return {Promise}
 */

export default async function sortableDrag(
  mode,
  itemSelector,
  offsetFn,
  callbacks = {}
) {
  let start;
  let move;
  let end;
  let which;

  if (mode === 'mouse') {
    start = 'mousedown';
    move = 'mousemove';
    end = 'mouseup';
    which = 1;
  } else if (mode === 'touch') {
    start = 'touchstart';
    move = 'touchmove';
    end = 'touchend';
  } else {
    throw new Error(`Unsupported mode: '${mode}'`);
  }

  await settled();

  const itemElement = find(itemSelector);
  const itemOffset = getOffset(itemElement);
  const offset = offsetFn();
  const rect = itemElement.getBoundingClientRect();
  // firefox gives some elements, like <svg>, a clientHeight of 0.
  // we can try to grab it off the parent instead to have a better
  // guess at what the scale is.
  // https://bugzilla.mozilla.org/show_bug.cgi?id=874811#c14
  // https://stackoverflow.com/a/13647345
  // https://stackoverflow.com/a/5042051
  const dx = offset.dx || 0;
  const dy = offset.dy || 0;
  const clientHeight =
    itemElement.clientHeight ||
    itemElement.offsetHeight ||
    itemElement.parentNode.offsetHeight;
  const scale = clientHeight / (rect.bottom - rect.top);
  const halfwayX = itemOffset.left + (dx * scale) / 2;
  const halfwayY = itemOffset.top + (dy * scale) / 2;
  const targetX = itemOffset.left + dx * scale;
  const targetY = itemOffset.top + dy * scale;

  await triggerEvent(itemElement, start, {
    clientX: itemOffset.left,
    clientY: itemOffset.top,
    which,
  });

  if (callbacks.dragstart) {
    await callbacks.dragstart();
  }

  await triggerEvent(itemElement, move, {
    clientX: itemOffset.left,
    clientY: itemOffset.top,
  });

  if (callbacks.dragmove) {
    await callbacks.dragmove();
  }

  await triggerEvent(itemElement, move, {
    clientX: halfwayX,
    clientY: halfwayY,
  });

  await triggerEvent(itemElement, move, {
    clientX: targetX,
    clientY: targetY,
  });

  await triggerEvent(itemElement, end, {
    clientX: targetX,
    clientY: targetY,
  });

  if (callbacks.dragend) {
    await callbacks.dragend();
  }
}

@ygongdev lovely stuff, thanks!
I think it’s OK for import { drag } from 'ember-sortable/test-helpers'; to clobber the global registerAsyncHelper version.
Could you wrap this up in a PR?

Yeah I can take a stab at it when I have some time.

I tried to convert drag and reorder to the new syntax, migrate existing tests to new test syntax, and remove dependency on jquery. However, it doesn't seem like the sorting is ever happening in the tests, thus all of them failed, so I'll need to keep into it more.

@ygongdev we’ll need to keep the tests for the classic-style helpers around too, to maintain backwards compatibility.

Modern test helper should also be moved to addon-test-support and out of the helpers dir since those are bundled with the addon and into the consuming app :-(

@ygongdev I managed to find the bug related to your comment above. When combined with this issue, or rather it's fix - it sorts as expected.
TL;DR: add to you Sass files
.sortable-item {
  transition: all .125s;

  &.is-dragging {
    transition-duration: 0s;
  }
}

Change merged in #271


Fix #450: Sign of energy infusion

Fixes #450: Sign of energy infusion has innate 100% bonus (as originally) and allows damage penalty

See #450 for comment with details


please add fast files dragging feature just like the stock one.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here. Ex: What device type/UI layout and app version?

Issue-Label Bot is automatically applying the label feature_request to this issue, with a confidence of 0.78. Please mark this comment with 👍 or 👎 to give our bot feedback!
Links: app homepage, dashboard and code for this bot.

@Bysic Please fill out the issue template so we can understand you feature request.


[BUG]: Missing Revolut pay confirmation method

What happened?
Missing Revolut pay confirmation method - confirmRevolutPayPayment
Environment
No response
Reproduction
No response

This is not missing, it is intentionally unavailable and unnecessary. You can confirm a revolut_pay using confirmPayment either with or without using Payment Element.


Convert delimited set values to ints from strings.

A recent change in the requirements of meta definitions means that all values must be ints and not strings. I'm just making this change to the Example Data (A).json example dataset.
Change example - before:
"q8": {
      "text": {
        "en-GB": "Which of the following do you regularly skip?"
      },
      "values": [
        {
          "text": {
            "en-GB": "Breakfast"
          },
          "value": "1"
        },
        {
          "text": {
            "en-GB": "Mid-morning snacking"
          },
          "value": "2"
        },
        {
          "text": {
            "en-GB": "Lunch"
          },
          "value": "3"
        },
        {
          "text": {
            "en-GB": "Mid-afternoon snacking"
          },
          "value": "4"
        },
        {
          "text": {
            "en-GB": "Dinner"
          },
          "value": "5"
        },
        {
          "text": {
            "en-GB": "None of them"
          },
          "value": "96"
        },
        {
          "text": {
            "en-GB": "Don't know (it varies a lot)"
          },
          "value": "98"
        }
      ],
      "name": "q8",
      "type": "delimited set"
}

After:
"q8": {
      "text": {
        "en-GB": "Which of the following do you regularly skip?"
      },
      "values": [
        {
          "text": {
            "en-GB": "Breakfast"
          },
          "value": 1
        },
        {
          "text": {
            "en-GB": "Mid-morning snacking"
          },
          "value": 2
        },
        {
          "text": {
            "en-GB": "Lunch"
          },
          "value": 3
        },
        {
          "text": {
            "en-GB": "Mid-afternoon snacking"
          },
          "value": 4
        },
        {
          "text": {
            "en-GB": "Dinner"
          },
          "value": 5
        },
        {
          "text": {
            "en-GB": "None of them"
          },
          "value": 96
        },
        {
          "text": {
            "en-GB": "Don't know (it varies a lot)"
          },
          "value": 98
        }
      ],
      "name": "q8",
      "type": "delimited set"
}

Should we retroactively edit scripts [that create new metadata] to this effect?

You'd only notice this if you were trying to convert to SAV using quantipy.core.dp.tools.io.write_spss(), so I wouldn't worry too much. In the case that you do have a file you converted from some other format a while ago, then you should just re-convert them to Quantipy and this will already be sorted out for you.

:+1:


Only consider removed_dates that have a valid note

For a service that looks like this:
{
  "valid_days": [
    1,
    2,
    3,
    4,
    5
  ],
  "typicality": "typical_service",
  "type": "weekday",
  "start_date": "2020-02-07",
  "service_date": "2020-02-18",
  "removed_dates_notes": {
    "2020-02-21": null,
    "2020-02-20": null,
    "2020-02-19": null,
    "2020-02-18": null,
    "2020-02-17": "Washington's Birthday"
  },
  "removed_dates": [
    "2020-02-17",
    "2020-02-18",
    "2020-02-19",
    "2020-02-20",
    "2020-02-21"
  ],
  "rating_start_date": "2019-12-22",
  "rating_end_date": "2020-03-14",
  "rating_description": "Winter",
  "name": "Weekday",
  "id": "BUS120-3-Wdy-02",
  "end_date": "2020-03-13",
  "description": "Weekday schedule",
  "default_service?": false,
  "added_dates_notes": {},
  "added_dates": []
}

it won't be considered current and valid for the removed_dates. Which works fine on 2020-02-17, where another service is available that's properly marked as a holiday service with the relevant added_date. But presents a problem for the other listed removed_dates, where other services have not been added, and typical service may exist (in this case the week is a school holiday).
A workaround here is to only consider removed dates with a non-null note, which will prevent the inappropriate "no service for today" message here:

@thecristen why are these dates in the removed_dates in the first place?

@thecristen ah, I see you said they're a school holiday. But is there no other service that applies to those days?

But is there no other service that applies to those days?

On the routes I looked at, no (just a typical saturday and typical sunday service). Why there isn't, say, a school holiday-relevant service.. is unclear.

@thecristen @phildarnowsky the 195 does have service this week, on the WinterWeekday service ID: https://api-v3.mbta.com/services/?filter[route]=195

@paulswartz @phildarnowsky  Yep. But this week's dates are also all listed under removed_dates. So removed_dates alone doesn't seem to indicate a presence/absence of service.

removed_dates means that particular service isn't active on that date, when it normally would have been given the start_date/end_date/valid_days.
In this case, the Weekday service is not active today because '2020-02-19is inremoved_dates, and the WinterWeekdayservice is active because it's a Wednesday between2020-02-18and2020-02-21` (inclusive).

Thanks @paulswartz -- I'll have to check what's going on on my end such that I'm not properly receiving and/or handling the WinterWeekday service.


Remove another unused import

Ideally, this would have been in #2942.

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 503d802 has been approved by alexcrichton

:hourglass: Testing commit 503d802 with merge fd90fd6...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: alexcrichton
Pushing fd90fd642d404d8c66505ca8db742c664ea352f2 to master...


Randomize amount stolen

Related to #911
Add some variety for amount stolen and weight stealing based on selected difficulty.
Learned when you make a std::map const, you must use std::map::at instead of std::map[] to pull values.

Ahh yes, std::map[] can have some somewhat surprising side effects when the key isn't found.


Allow overriding severity based on error code prefix match

See #195 for context.

@jnooree Thanks for the PR. Could you add tests for this scenario?
you can find an example here:
https://github.com/microsoft/vscode-flake8/blob/6f638010c80d4ee17a4ad515c3666f6d7cea7796/src/test/python_tests/test_linting.py#L198-L258

Do the tests added in 670dccb suffice?

Do the tests added in 670dccb suffice?

It looks good.


[WFLY-19537] Bind misc external testsuite servers to the server address to avoid test env networking issues

https://issues.redhat.com/browse/WFLY-19537
Various tests on the s390 CI job fail because the server can't connect to test processes listening on InetAddress.getLocalHost(). Avoid this by binding the test processes to the same address as the WF server.
It's not clear why they can't connect but it's safer to KISS and use the same address as the server.

LGTM, assuming CI passes ;)

/retest

Thanks @luck3y


Lib gen module added

Description

The changes made in this pull request should be briefly described.
I have added a LibGen Module.
It has a method getBooks() inside it which gives the books from the given book name

Resolves: #499
Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

[x] Have you adhered to the repository's defined coding convention rules?
[x] Have you updated the 'documentation.md' file with the method/function documentation?
[x] Have you sent a message along with the result or response?
[x] Have you used the try-catch technique?
[x] Has the method/class been added to the documentation (md file)?

Screenshots

Uploading a screenshot illustrating the approach you developed for validation is required.

Additional Notes/Comments

Any additional remarks or suggestions concerning this pull request are welcome.

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

Great work @Swapnil-Singh-99 !!
Do not forget to

Share your contribution on socials(twitter mainly), tag us and Clueless 🫂
Give this repository a star ✨
Follow Clueless Community on GitHub ✅


TOTP authentication

In a Web3 environment integrating TOTP authentication, we can envision a process involving three main actors: the Google Authenticator app on the user's device, the user's browser acting as the front end of the DApp, and the smart contract performing the verification. Here is the role of each participant and the detailed steps of the entire authentication process:
Role Definitions:

Google Authenticator: An application that runs on the user's device, capable of generating Time-based One-Time Passwords (TOTP). It requires a secret key (secretKey) to generate these passwords.

User's Browser: The platform running the front end of the DApp, responsible for receiving the TOTP input from the user, generating zero-knowledge proofs, and interacting with the smart contract.

Smart Contract: A contract deployed on the blockchain that verifies the zero-knowledge proof submitted by the user without needing to know the secretKey.

Authentication Process:
Initialization (One-time Setup):

Generate Secret Key:

The user generates a TOTP secretKey within the DApp.
This process is done entirely on the user’s device, and the secretKey never leaves the device.

Configure Google Authenticator:

The user receives a QR code through the DApp interface, which contains the secretKey.
The user scans this QR code with Google Authenticator, thus configuring the app to generate TOTP codes.

Authentication (Every Login or Sensitive Operation):

Generate TOTP:

When authentication is required, users open their Google Authenticator app and obtain the current TOTP.

Generate Zero-Knowledge Proof:

The user enters the TOTP in the DApp front end within the browser.
The DApp front end locally generates a zero-knowledge proof using the entered TOTP and the pre-set user secretKey information (such as its hash or a related commitment).
This proof can attest that the user has a valid TOTP without revealing the TOTP or secretKey itself.

Interact with Smart Contract:

The DApp front end submits the zero-knowledge proof to the smart contract.
The smart contract's internal validation logic checks the validity of the zero-knowledge proof.

Execute Operation:

If the smart contract validates that the zero-knowledge proof is valid, the user's requested operation is executed.
If the proof is invalid, the operation is rejected.

If the DApp front end locally generates the proof, the app front end needs to keep the secret key, and it seems not safe.
Maybe it should be generated in the Google Authenticator role?


Upgrade to py-evm v0.3.0-alpha.15

What was wrong?
Wanted access to some of the latest evm updates, like:

witness data in the Chain.import_block() result
eth-keys could be updated

How was it fixed?
Upgrade to py-evm v0.3.0-alpha.15, then:

Handle the new BlockImportResult returned by Chain.import_block()
py-evm's AtomicDB type signature was improved. A custom version of
AtomicDBWriteBatch had to be updated. It was shrunk down significantly,
depending on the functionality provided in py-evm's version.

~Also updated web3.py, eth-tester and eth-keys.~ This was too big, moved to #1725
To-Do

[x] Clean up commit history

[x] Add entry to the release notes

Cute Animal Picture

The wheel failure seems to be that it didn't >>> ping during the test. Maybe a network issue in the container, or got unlucky finding peers?


examples/widgets/mixed_viewport.dart crashes when you remove big nodes before small nodes

 Issue by Hixie
Tuesday Aug 25, 2015 at 21:33 GMT
Originally opened as https://github.com/flutter/engine/issues/820

To reproduce:

click add a few times until you have a big block followed by a small block
scroll the big block almost off screen, such that more of the block is off the top of the screen than the height of the small block
tap the big block

The small block is moved off the screen and then there's a crash.
CC @HansMuller

 Comment by Hixie
Tuesday Aug 25, 2015 at 21:39 GMT

I think you can also trigger this in examples/widgets/card_collection.dart by flinging a card out while scrolling.

 Comment by HansMuller
Tuesday Aug 25, 2015 at 21:52 GMT

I have looked into this (without success) more than once.  The failure is signalled by this assert:
'http://localhost:9888/packages/sky/widgets/framework.dart': Failed assertion: line 366 pos 14: 'oldNode.mounted' is not true.
The problem can be avoided by changing MixedViewport._doLayout():
    // Remove any old children.
    for (_Key oldChildKey in childrenByKey.keys) {
      if (!childrenByKey[oldChildKey].mounted)  // HACK
        continue;
      ...
     }

It's not clear how a !mounted Widget ends up in childrenByKey at this point.

 Comment by HansMuller
Monday Aug 31, 2015 at 18:25 GMT

I've checked this against the original test case and with the card_collection.dart (swipe away) demo. The bug appears to have been fixed. I haven't been able to pin down exactly which change gets the credit.

Is this still an issue?

I'm going to close this because I don't know how to reproduce it anymore.
I did get https://github.com/flutter/flutter/issues/5864 while trying to reproduce it...


Add complete support for new Events API payload format

As an addition to #570 , this pull request updates the Events API payload classes to have the following fields.

authorizations
is_ext_shared_channel

Also, I've tweaked the test utilities a bit.
Category (place an x in each of the [ ])

[ ] bolt (Bolt for Java)
[ ] bolt-{sub modules} (Bolt for Java - optional modules)
[ ] slack-api-client (Slack API Clients)
[ ] slack-api-model (Slack API Data Models)
[ ] slack-api-*-kotlin-extension (Kotlin Extensions for Slack API Clients)
[x] slack-app-backend (The primitive layer of Bolt for Java)

Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to the those rules.

Thanks for checking this, y'all!

I didn't run the integration tests, because I don't have my environment completely setup for all the live API tests.

I do understand it's not so easy. As the tests cover Grid use cases, admin installations, you need to set up workspaces, app permissions, several installations to run all the tests.


Gallery Webpage

I would like to add a Gallery webpage designed beautifully with the help of HTML, CSS, JS, and tools like Lightbox. Kindly assign me this issue; it will give a boost to my contributions to Hacktoberfest.

Go ahead @RaginaPhalangae

@shubham7668 Can you please add "Hacktoberfest accepted" label? It will be very kind of you and will certainly motivate me more to make a quality contribution.

@shubham7668 Thank you.

can you assign this task to me?

can you assign this task to me?

Go ahead @Deepanshu1008

please help me i am a newbie

please help me i am a newbie

What kind of help you needed

i have added the file using fork

please merge the pull request now

please merge the pull request now

Create a valid pull request
You have to add your files in projects folder else it will be mess
And deploy your project and give me link


noinclude-data-files added as argument

The way this is done has changed, no more is shell code needed as of last release at all, but only the help description. I am adding your text now, but I think we need to generate the actions.yml with a bit of Jinja2 out of nuitka --help at least somewhat.

This should work on main now, please try and close the PR if it works for you.


无法打开输入文件“..\lib\Release\DXCam.lib”	screenshot

When I want to run this example,I got this issue. I want to know if there is no this .lib file?

Did you compile this library first to get the binary .lib file? It should be in /lib in the output directory.

Firstly, I am very appreciated for your timely response! But, sorry, I couldn't find it in build/lib, and I also found a new issue this morning. It's decribed as internal compiler error. Detailedly, 错误	C1001	内部编译器错误。	DXCam	E:\c+test\DXCam-new\DXCam-CPP-main\src\util\io.cpp	54	. I am so sorry for I am not experienced in C++.

It seems that the DXCam library is not properly built since DXCam.lib does not output.
Could you provide more details on how you build the library? Or you can try these:

cmake.exe -DCMAKE_BUILD_TYPE=Release "-DCMAKE_MAKE_PROGRAM=ninja.exe" "-DCMAKE_C_COMPILER=cl.exe" "-DCMAKE_CXX_COMPILER=cl.exe" -DCMAKE_CXX_COMPILER_WORKS=1 -DCMAKE_C_COMPILER_WORKS=1 -G Ninja -B ./build/
cmake.exe --build .\build\ --target DXCam

About "Fatal Error C1001", I think it is an issue relative to MSVC. Someone said disabling compiler optimization may work (you can try -DCMAKE_BUILD_TYPE=Debug in the first command), but it is not a long-term solution. Anyway, you can try the above solution and see if it occurs again.

I am grateful to get your response again. I think I hava got the method to solve this problem. Thank you very munch!!!  Your are so excellent ！

You're welcome. Thanks for using my project :)


The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0

Environment
Describe your dev environment here, giving as many details as possible. If you have them, make sure to include:
Xcode Version: 12.0
Swift Version: 5.3
Installation Platform & Verison: Swift Package Manager 7.1.1
Goals
What do you want to achieve?
Remove warnings at compile time
Expected Results
What do you expect to happen?
I hope you can update your library
Actual Results
What actually happened? Can you provide a stack trace?
The iOS deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 14.0.99.

Are you sure this is not a CocoaPods issue? I've had a similar warning come up in my project and it was resolved with a modification to the the pods file.

@rickypowell
I don't use CocoaPods.
I am using Swift Package Manager.
Other libraries are responding to the issue through the latest updates.
IPHONEOS_DEPLOYMENT_TARGET = 9.0;
https://github.com/danielgindi/Charts/commit/e98681cc26aac6185ad73330a6e22b0e97a5b302
I think it will be solved by modifying the file below.

This warning is showing up because as of the current release, we still support iOS 8.0 as a valid target, by using an older version of Xcode.
We are committed to maintaining a sizeable amount of backwards compatibility, but understand this is pushing it a bit far.
I plan to do another assessment of minimum targets + swift versions before we do a 6.0.0 release (which already drops support for a number of Swift versions)

Resolved in https://github.com/ReSwift/ReSwift/releases/tag/6.0.0