GetStreamingChatMessageContentsAsync not working

Hi there,
Thanks so much for the connector.
Please could you describe how to get message streaming working?
I have set up the project accordingly and when attempting to stream i receive an error:
result = chatCompletionService.GetStreamingChatMessageContentsAsync(
history,
executionSettings: _openAIPromptExecutionSettings,
kernel: kernel);
System.Text.Json.JsonException
HResult=0x80131500
Message='{' is invalid after a single JSON value. Expected end of data. Path: $ | LineNumber: 1 | BytePositionInLine: 0.
Source=System.Text.Json
StackTrace:
at System.Text.Json.ThrowHelper.ReThrowWithPath(ReadStack& state, JsonReaderException ex)
at System.Text.Json.Serialization.JsonConverter1.ReadCore(Utf8JsonReader& reader, JsonSerializerOptions options, ReadStack& state)    at System.Text.Json.JsonSerializer.ReadFromSpan[TValue](ReadOnlySpan1 utf8Json, JsonTypeInfo1 jsonTypeInfo, Nullable1 actualByteCount)
at System.Text.Json.JsonSerializer.ReadFromSpan[TValue](ReadOnlySpan1 json, JsonTypeInfo1 jsonTypeInfo)
at System.Text.Json.JsonSerializer.Deserialize[TValue](String json, JsonSerializerOptions options)
at Codeblaze.SemanticKernel.Connectors.Ollama.OllamaChatCompletionService.d__2.MoveNext()
at Codeblaze.SemanticKernel.Connectors.Ollama.OllamaChatCompletionService.d__2.System.Threading.Tasks.Sources.IValueTaskSource<System.Boolean>.GetResult(Int16 token)
at xxx.d__2.MoveNext() in xxx
at xxx.d__2.MoveNext() in xxx
at Program.<$>d__0.MoveNext() in xxxx
This exception was originally thrown at this call stack:
[External Code]
Inner Exception 1:
JsonReaderException: '{' is invalid after a single JSON value. Expected end of data. LineNumber: 1 | BytePositionInLine: 0.

Same error here ;(

Hi - Same here. There seems to be an issue with the jsonResponse variable content in
https://github.com/BLaZeKiLL/Codeblaze.SemanticKernel/blob/main/dotnet/Codeblaze.SemanticKernel.Connectors.Ollama/ChatCompletion/OllamaChatCompletionService.cs#L67
When I try to run a streaming chat completion, the variable contains all response chunks:
{"model":"llama3:latest","created_at":"2024-06-14T08:19:40.607016404Z","message":{"role":"assistant","content":"The"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.618778759Z","message":{"role":"assistant","content":" sky"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.630561283Z","message":{"role":"assistant","content":" appears"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.642207649Z","message":{"role":"assistant","content":" blue"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.653852956Z","message":{"role":"assistant","content":" because"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.665474375Z","message":{"role":"assistant","content":" of"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.677132728Z","message":{"role":"assistant","content":" a"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.688783164Z","message":{"role":"assistant","content":" phenomenon"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.700451403Z","message":{"role":"assistant","content":" called"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.712002743Z","message":{"role":"assistant","content":" Ray"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.72352729Z","message":{"role":"assistant","content":"leigh"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.735108166Z","message":{"role":"assistant","content":" scattering"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.746673503Z","message":{"role":"assistant","content":","},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.758123132Z","message":{"role":"assistant","content":" where"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.769549156Z","message":{"role":"assistant","content":" shorter"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.780985197Z","message":{"role":"assistant","content":" blue"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.792404703Z","message":{"role":"assistant","content":" wavelengths"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.803854295Z","message":{"role":"assistant","content":" scatter"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.815322862Z","message":{"role":"assistant","content":" more"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.826758498Z","message":{"role":"assistant","content":" than"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.838206235Z","message":{"role":"assistant","content":" longer"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.849689916Z","message":{"role":"assistant","content":" ones"},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.861138488Z","message":{"role":"assistant","content":"."},"done":false} {"model":"llama3:latest","created_at":"2024-06-14T08:19:40.872654676Z","message":{"role":"assistant","content":""},"done_reason":"stop","done":true,"total_duration":497213323,"load_duration":2548022,"prompt_eval_count":27,"prompt_eval_duration":98577000,"eval_count":24,"eval_duration":265580000}
which can't be deserialized to a single OllamaChatResponseMessage instance.

I have an improved version based on @william-daconceicao #12  in #13 so that the (streamed) content of the response can start to be read as soon as the HTTP headers have been received.
See before/after here https://www.youtube.com/watch?v=toSgN-HUOIQ

Thanks for this, I have been working on getting the Ollama plugin merged in the source repository, so haven't tested this for a while but I have merged #13


Compatible releases are not compatible with Semantic Versions less than 1.0.0.

According to PEP440,

The "Major.Minor.Patch" (described in this PEP as "major.minor.micro") aspects of semantic versioning (clauses 1-9 in the 2.0.0-rc-1 specification) are fully compatible with the version scheme defined in this PEP

Also,

A compatible release clause consists of the compatible release operator ~= and a version identifier. It matches any candidate version that is expected to be compatible with the specified version.

According to Semantic Versioning clause 4,

Major version zero (0.y.z) is for initial development. Anything may change at any time. The public API should not be considered stable.

By this logic, I would not expect pkgdep ~= 0.1 to be satisfied, for example, by 0.2.0, since there is no guarantee that 0.2.0 will be backwards-compatible with 0.1.x; however, this seems to be the current behavior when using the compatible release operator in install_requires.
Strictly speaking, I'm not even sure pkgdep ~= 0.1.0 should imply 0.1.1, although I suspect that is divergent from de facto expectations.

but imho the semver spec is a bad design in that regard. In any case, there's little Setuptools can do about it. If the spec should be updated, that's with semver. If the pep should be clarified, that's with the pep process. If the implementation should change, that's with packaging, maybe with an update here following that release.
Good luck.

Oops, ya I suppose this is more of a pip issue... Moving to https://github.com/pypa/pip/issues/3982


Add jack_zhang in basic-branch-build-strategies

Description

Could you please grant @zhan9san permission to maintian the following repo?
https://github.com/jenkinsci/basic-branch-build-strategies-plugin
committers: @stephenc @fbelzunc @rsandell
It would be great if this pr can be merged.
https://github.com/jenkinsci/basic-branch-build-strategies-plugin/pull/12
Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[x] Make sure the file is created in permissions/ directory
[x] artifactId (pom.xml) is used for name (permissions YAML file).
[x] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[x] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[x] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @Wadeck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Hey,
the plugin specified is not labeled as up for adoption. Could you follow the adoption steps please and send a mail to the mailing list too?
Nevertheless, this proposal needs authentication by the current maintainers.

Thanks for your review.
I have sent an email following adoption steps.
A PR in this repo is required in the adoption steps, so I submitted it.


Word wrap?

Would it be possible to add an option somewhere to enable word wrap?

Might be able to work around with CSS (but would obviously prefer it just work).

Fixed in latest version that uses CM6.


M2 Mac Attempted Installation: LLVM ERROR: Option 'pbqp' already exists!

After building vLLM, Triton, and SGLang from source, upon attempting to run the following command.
python -m sglang.launch_server
I am met with the following error
LLVM ERROR: Option 'pbqp' already exists!
zsh: abort ...
Additionally, despite recieving the following message
Successfully built sglang
Installing collected packages: sglang
Successfully installed sglang-0.1.17
When accessing SGLang within VSCode:
sglang.srt.constrained cannot be found and
set_default_backend also cannot be found.
Note: I am using vLLM 0.3.3

ref https://github.com/sgl-project/sglang/issues/508#issuecomment-2254328607


Hello all of my docstrings turned green after latest update

Type: Bug
Hello all of my docstrings have turned green, all of my functions are no longer highlighted, and none of my variables are highlighted either! I am using python and here is a list of my extensions:
isort
jupyter
jupyter cell tags
jupyter keymap
Jupyter notebook renderers
jupyter slideshow
pylance
python
Thank you.
VS Code version: Code 1.79.0 (b380da4ef1ee00e224a15c1d4d9793e27c2b6302, 2023-06-07T14:26:35.552Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz (8 x 4200)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
31.96GB (26.48GB free)

Process Argv
--crash-reporter-id 4b8cf8a6-8f3e-47e5-ab7d-bf0946860ba0

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

isort
ms-
2022.8.0

python
ms-
2023.10.0

vscode-pylance
ms-
2023.6.10

jupyter
ms-
2023.5.1001582324

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263cf:30335440
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vscaat:30438848
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
282f8724:30602487
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
03d35959:30757346
pythonfmttext:30731395
pythoncmvfstrcf:30756944
fixhidewlkth:30730051
showsbindicator:30760979
pythongtdpathcf:30739705
dh2dc718:30763024
pythonnosmt12:30757131

/extPython

It looks like this is caused by the Python extension. Please file the issue to the Python extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!

Turns out this wasn't the extension and was vscode.
https://github.com/microsoft/vscode/pull/182162
Proposal to revert is https://github.com/microsoft/vscode/pull/184938


Enabling FILL button

I have a RapiDoc for an API documentation, clear button and try are working fine but i want when somebody needs to try he/she should not in put the vales in fields instead to fill default values by clicking fill button then try. How do i do that?.

In your schema for every field you can provide a default  and one or more example

default is what the server will substitute when the client do not provide any value.  RapiDoc being a client wont provide this value, its shows up in RapiDoc only for documentation purpose
example for the field is used to fill the data.

so if you want the FILL button to work, you must provide example for the corresponding fields in your schema
below is an example how you can provide examples in your OpenAPI spec
openapi: 3.0.0
info:
  version: 1.0.0
  title: Demonstrates schema examples
paths:
  /person:
    post:
      summary: Demonstrates schema examples
      parameters:
        - name: age
          in: query
          example: 24
          schema:
            type: integer
        - name: full-name
          in: query
          example: John
          schema:
            type: string
        - name: email
          in: query
          example<EMAIL_ADDRESS>          schema:
            type: string
      responses:
        '200':
          description: successful operation

Thanks alot.
On Thu, Feb 11, 2021 at 7:05 PM Mrinmoy<EMAIL_ADDRESS>wrote:

Closed #411 https://github.com/mrin9/RapiDoc/issues/411.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mrin9/RapiDoc/issues/411#event-4320657904, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOBVNJO42TRKY24I66BMPT3S6P54FANCNFSM4XNQ6C7A
.

one more thing,  I have my openAPI documentation, after logging in the user
gets the session Id which will be needed for all other requests. I need
this value to be put once by a user and all other requests needing it to
use it from authentication. How do I do that?
On Thu, Feb 11, 2021 at 7:33 PM Corrensa<EMAIL_ADDRESS>wrote:

Thanks alot.
On Thu, Feb 11, 2021 at 7:05 PM Mrinmoy<EMAIL_ADDRESS>wrote:

Closed #411 https://github.com/mrin9/RapiDoc/issues/411.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mrin9/RapiDoc/issues/411#event-4320657904, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOBVNJO42TRKY24I66BMPT3S6P54FANCNFSM4XNQ6C7A
.


Added hr_HR locale

Sorry for the wait, was really busy. Added more data and sorted alphabetically, as requested. Hope it's alright now.

Thanks!


Blocking websites in browser

I am  not sure about other browsers but in Chrome, for example I searched "Hosting" on Google and Google shows me the promoted websites on top with Google ad services, And I probably searched "Hosting" with the site Bluehost in mind or any other keyword just giving a example, so I click on the site that was advertised by Google, but I get the error that the site could not be reached, then you have to scroll then find the website you were looking for and continue, kind of annoying.
Please fix, Thank you.

It's basically blocking the ads on google. Ig it's more of a feature than bug. 💀


Symbolic parser

Write a symbolic wrapper to get metadata about the object's content
Requirements
struct Symbol {
   std::string name;
  std::size_t value;
};

[ ] Write a parse method
[ ] Write simplify method
[ ] Think about wrapping objects to symbols that are more ergonomic e.g. generative assignment to_symbol(object) -> symbol(name, value)

For now, settle with a dumb symbolic instance assignment
template <typename T, std::size_t Row, std::size_t Col> struct Symbol {

  Symbol() { m_counter++; }

  auto name() const noexcept -> std::string {
    return m_name + "_" + std::to_string(m_counter);
  }

private:
  static std::string m_name;
  static std::size_t m_counter;
};

template <typename T, std::size_t Rows, std::size_t Cols>
std::size_t Symbol<T, Rows, Cols>::m_counter = 0;

template <typename T, std::size_t Rows, std::size_t Cols>
std::string Symbol<T, Rows, Cols>::m_name = "x";

Feature is put on hold. Expression optimization turns out to be more complicated than it seems.

Checkout Embedded DSL efforts in Cpp Talk: https://www.youtube.com/watch?v=XH00wB_bbU4


New package: CUDNN v1.0.0

Registering package: CUDNN
Repository: https://github.com/JuliaGPU/CUDA.jl
Created by: @maleadt
Version: v1.0.0
Commit: 62473cac668a456d25f9a4a15dfa0ab1786bd33d
Reviewed by: @maleadt
Reference: https://github.com/JuliaGPU/CUDA.jl/commit/62473cac668a456d25f9a4a15dfa0ab1786bd33d#commitcomment-99040209
Description: CUDA programming in Julia.

Ah, commented on the wrong commit 🤦


d3.zoom() is not support ios7.1

v4.11.0 report gesture error in ios7.1
I changed to v3.5.17 to solve this error

I don’t have access to a iOS 7 simulator, and without more specific information such as the code that was being executed and the exact stack trace and error message, I’m not going to be able to investigate this issue. If you would be willing to use your developer tools on iOS 7 and provide more detail as to what is causing the error, that would be greatly appreciated and I would be happy to take a look provided more context. Thank you!


B991

This resolves bugs 991 & 992. In addition, it merges in some work that entered pysal's development branch by mistake.
Further, it clarifies & reinforces in the PR template that we are submitting against the master branch.

Coverage increased (+0.02%) to 83.79% when pulling c5289306ecdb4a0ae0e992fdb30a6aaa3dde77db on ljwolf:b991 into 2470792d542cf89a2f4105be97f8d6ed2292dd81 on pysal:master.

I got error this error
ImportError: cannot import name 'chisqprob'
when run below code. how to solve this. please help me.
#Import the necessary Python moduless
import pandas as pd
import geopandas as gpd
import numpy as np
from geopandas.tools import sjoin
import folium
from folium.plugins import MarkerCluster
from folium.element import IFrame
import shapely
from shapely.geometry import Point
import unicodedata
import pysal as ps


Add an --edge option to the new management command

Description
Let's make it possible to generate new projects (or applications) that use the development version of Marten through the use of an --edge / -e option. When this option is set, the generated project structure should include a shard.override.yml file (in addition to the shard.yml file) with the following content:
dependencies:
  marten:
    github: martenframework/marten
    branch: main

This will ensure that the project uses the development version of Marten (main branch) when running the shards install command (ref).

I would recommend --nightly, or --dev-preview as those seem less ambigous.


CardanoConfiguration and Config reconciliation

CardanoConfiguration and Config currently have different type signatures for the same values. This is somewhat confusing and we should probably decide on some kind of unification.
Related issue: https://github.com/input-output-hk/cardano-ledger/issues/429

This has already been discussed here
Conclusion:

So a pragmatic approach might be to keep the types for the config, but later we can opportunistically reuse types in some cases if we're confident that they're really 1:1 and are unlikely to change.

I just spoke with @ksaric re: The storing and validation of the AVVM UTxO and how to incorporate it in the cardano-shell config (CardanoConfiguration). See linked issue #429 above for current plan.


Cherry pick 2 fixes

[Explore] Fix Stop Query Button behavior (#4301)
[Bug] Resize should trigger chart re-render (#4322)

LGTM


Ignore unintended_html_in_doc_comment

MDN docs includes some variants of html tags that this rule disallows, like
, , and
. In order to make the CI pass, this is ignored for now. Fixes cases unrelated to the MDN docs.
cc @kallentu for some fun edge cases from the MDN.

Re the failure: we shouldn't run analyzer tests against 3.4 - just dev

nit: also: could we just add this ignore at the top of all generated files?

Re the failure: we shouldn't run analyzer tests against 3.4 - just dev

We should still make sure analysis passes with 3.4 so we don't release a package that breaks against an older Dart SDK version. We might need to ignore unrecognized_error_code to make this work however.

nit: also: could we just add this ignore at the top of all generated files?

Yeah, since it looks like we can't exclude files from a lint, I could try that.

Generated in the files themselves instead, which has the nice side-effect of it not affecting the 3.4 analysis.

I'm trying to validate @kallentu's fix for this, but am having trouble getting the false positives from unintended_html_in_doc_comment to fire here (using an sdk before the fix). I'm assuming I have a local config issue but mentioning here as I don't see the false positives.

Nevermind - I had to check out an older commit to get this to trigger; I do see these as fixed in head.

Cool, thanks Kallen! We'll still need the ignores for a little while until we move to a minimum SDK version that contains the fix.


SELECT command denied to user<EMAIL_ADDRESS>for table 'user_variables_by_thread'

error info :
execute error. SELECT variable_name FROM performance_schema.user_variables_by_thread WHERE variable_value IS NOT NULL
java.sql.SQLSyntaxErrorException: SELECT command denied to user<EMAIL_ADDRESS>for table 'user_variables_by_thread'
flywaydb version:

org.flywaydb
flyway-core
7.7.3

It seems like the user that flyway is using (or its IP Address) has not been granted permissions within your DB. Can you give further details on which DB you are using so we can give more specific assistance?

Closing until further activity

granted permissions to our db use following sql, it's okay, thanks. @DoodleBobBuffPants @madninjaskillz
GRANT SELECT ON performance_scheme.user_variables_by_thread TO 'username'@'%'

granted permissions to our db use following sql, it's okay, thanks. @DoodleBobBuffPants @madninjaskillz GRANT SELECT ON performance_scheme.user_variables_by_thread TO 'username'@'%'

fix the typo.
GRANT SELECT ON performance_schema.user_variables_by_thread TO 'username'@'%'


Clarify that reply chain fallback for threads is not required

Discussed in #matrix-spec.

Ugh no I did not intend to make this PR on my own fork.


Potential cost estimate

From Enterprise-Scale-APIM created by cykreng: cykreng/Enterprise-Scale-APIM#8

Closing due to age.


Publish new version

There has been some updates like adding multiple source which has not yet been released. Please when will this be released?

Android 1.0.2 has been published to bintray, if you need iOS/macOS framework, you can build it yourself.


Move svelte into peerDependencies

The latest Svelte update added a new element (<svelte:boundary>). However, upgrading an existing houdini + svelte project to this latest version gives errors when generating, as the version of the svelte compiler that houdini uses is outdated.
By moving the svelte dependency into peerDependencies, we use the svelte version that the project is using. The only requirement we now have is that it matches ^5.0.0.
(The updated wrangler version in e2e/react is because I was having compilation errors in better-sqlite3 while installing)
To help everyone out, please make sure your PR does the following:

[ ] Update the first line to point to the ticket that this PR fixes
[x] Add a message that clearly describes the fix
[ ] If applicable, add a test that would fail without this fix
[x] Make sure the unit and integration tests pass locally with pnpm run tests and cd integration && pnpm run tests
[x] Includes a changeset if your fix affects the user with pnpm changeset

I think it may be worth considering doing the same with vite

I think it may be worth considering doing the same with vite

Ah yeah good point. I might probably also move svelte-kit as well...


Just after adding the first list, the Words tab is empty

Hi, the words section is a record of items that you've studied and your progress on them. If you just added a list and haven't studied anything, it will still be empty. Look at the vocablist itself to see what words lie ahead in your studies.

I see. Thanks for the explanation!


Docker error:

Error response from daemon: Cannot start container 7285000a073bf7cc95bc3e9f48c3ecad260df5de0761ba7d39b12d5dee1d4bda: [8] System error: not a directory
I have other docker containers running just fine too.

Awkward, I can't read {INSERT LOCAL LOACTION HERE} lol


Fix Mind Reference

Two other notes:

The renaming can create confusion as there exists a test set just that I assume we don't have the labels - I clarified it here
MIND uses AUC & MRR & NDCG scores, not MAP, see https://msnews.github.io/

Which score should be the main one? Can you fix it as well?

Well I think we should use the same main score for all tasks in one task category, so sticking with MAP is fine I guess.
Albeit it would be nice to also compute AUC & NDCG scores for MIND. Opened a sep issue for that: https://github.com/embeddings-benchmark/mteb/issues/39


Remove ARAnalytics

Repo is no longer updated
https://github.com/orta/ARAnalytics

@stavares843 Thanks for the contribution!


security; widen use of scopes array to other securityScheme types

Split off from #1764 as discussed. In summary, I believe the outstanding issue is to find a non-contentious term possibly in preference to scopes or roles and to check this makes sense for all securityScheme types, including http's subtypes.

imho this change should include/reference some spec examples.

@darrelmiller Review naming of scopes vs roles

This PR allows the use of scopes for non OAuth2 security schemes.  It does not provide a way to describe the set of valid scopes for those scenarios.  We do believe that it would be valuable to have a place to define the allowed scopes but we need to spend some more time discussing how this can be incorporated into the SecurityScheme object.  We will do this in the rc1 timeframe.

we need to spend some more time discussing how this can be incorporated into the SecurityScheme object. We will do this in the rc1 timeframe.

WDYT to add some example to this PR as a comment?

As mentioned in #1106 and #1903, it would be really useful to support the ORing of scopes.  For example:
/pet:
post:
security:
- oauthScopes: [read, write] | [admin]
Will this be in the PR?  Thanks.

@glroman can you not do:
  /pet:
    post:
      security:
        - oauthScopes: [read, write]
        - oauthScopes: [admin]

Security Requirement Objects that contain multiple schemes require that all schemes MUST be satisfied for a request to be authorized. This enables support for scenarios where multiple query parameters or HTTP headers are required to convey security information.

This is ANDing. Here you can't have two references to oauthScopes because that would be a duplicate YAML/JSON key, but you could merge the scopes arrays.

When a list of Security Requirement Objects is defined on the OpenAPI Object or Operation Object, only one of the Security Requirement Objects in the list needs to be satisfied to authorize the request.

This is ORing. Here you can have multiple references to oauthScopes because they are in separate elements of the security array.

When I connect with a client that has "admin", this works:
/pet:  # Scenario 1
   post:
     security:
       - oauthScopes: [read, write]
       - oauthScopes: [admin]

but this doesn't:
/pet:  # Scenario 2
   post:
     security:
       - oauthScopes: [admin]
       - oauthScopes: [read, write]

I get:
{
  "detail": "Provided token doesn't have the required scope",
  "status": 403,
  "title": "Forbidden",
  "type": "about:blank"
}

Is this expected behavior?  My env:
python                  3.7.1
connexion               2.6.0
openapi-spec-validator  0.2.8

I think that may be an issue with connexion then.

@ioggstream  We have the opportunity to add examples in the examples not included in the spec.  I don't think we need to include an example in the specification because I do not think there is widespread usage of scopes in non-OAuth2 scenarios.
@MikeRalphson  I have come to terms with the use of the term "role".  I don't feel the need to change it here.
Defining scopes/roles at the security scheme level is not going to make it into 3.1.


Board not responding to taps

After multiple turns, the board stops putting indication circles around intersections where the user taps

This is actually a feature for the menus. Do you remember if there was a menu open? (trading menu, dev card menu, etc.)?

Fixed by making it more clear when you can and cannot select intersections.


python37Packages.google-cloud-bigquery: 2.7.0 -> 2.8.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.google-cloud-bigquery is: "Google BigQuery API client library"
meta.homepage for python37Packages.google-cloud-bigquery is: "https://github.com/googleapis/python-bigquery"
meta.changelog for python37Packages.google-cloud-bigquery is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.8.0 with grep in /nix/store/23n0xxs4x8rq8jf1r650mqns4c4prdiw-python3.7-google-cloud-bigquery-2.8.0

directory tree listing: https://gist.github.com/16ee96f176192588c8a215d2abb0882e

du listing: https://gist.github.com/bf15e4a42c6554b120738e919f4ab544

Rebuild report (if merged into master) (click to expand)

9 total rebuild path(s)

3 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
3 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python37Packages.google-cloud-bigquery
python38Packages.google-cloud-bigquery
python39Packages.google-cloud-bigquery

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/23n0xxs4x8rq8jf1r650mqns4c4prdiw-python3.7-google-cloud-bigquery-2.8.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.google-cloud-bigquery https://github.com/r-ryantm/nixpkgs/archive/c6fe0157f9bb1b027abc1a96e6259b44ef4a050c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/23n0xxs4x8rq8jf1r650mqns4c4prdiw-python3.7-google-cloud-bigquery-2.8.0
ls -la /nix/store/23n0xxs4x8rq8jf1r650mqns4c4prdiw-python3.7-google-cloud-bigquery-2.8.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python37Packages.google-cloud-bigquery
python38Packages.google-cloud-bigquery

Maintainer pings
cc @SuperSandro2000 for testing.

Result of nixpkgs-review pr 112995 at c6fe0157 run on aarch64-linux 1

1 package marked as broken and skipped:

python39Packages.google-cloud-bigquery

2 packages built:

python37Packages.google-cloud-bigquery
python38Packages.google-cloud-bigquery

1 suggestion:

warning: unused-argument
Unused argument: grpcio.
Near pkgs/development/python-modules/google-cloud-bigquery/default.nix:10:3:
   |
10 | , grpcio
   |   ^


Added some updates to the core spec from Michael

This adds some updates from @michaelpj .  It includes quite a lot of todo notes, so PLEASE DON'T MERGE IT.  If we turn off the notes it looks OK though.
This includes most of the changes from the PR at https://github.com/input-output-hk/plutus/pull/1064, where there's extensive discussion of some issues.

What's the status of this?

What's the status of this?

The PR in general?  It's waiting for Manuel to take a look.

Can we resolve any more of the todos first?

I'd rather wait for some new input from someone else.  There are two big things (erasure and saturated builtins) that require a lot of discussion, and a number of smaller things that an executive decision might resolve.

Prod - let's merge what we can from this rather than leaving it open indefinitely.

I assume we've taken everything we want from this.

Let me take another look at this.  I don't think everything was dealt with.  In particular, I think the following are outstanding:

I don't think we ever completeyl soretd out what was going on with error.  This version contains a remark saying that the semantics differ from what the CK machine does, but we can prove that that doesn't matter.  We should consult @jmchapman to see what he's actaully done about this.

I think the question of saturated/unsaturated builtins came up again, but then we forgot about it.  This may depend on @effectfully's dynamic builtin work.

There's an ambiguity in builtin function application: in )builtin b t_1 ... t_m v_1 ... v_n), how do you tell where the type arguments end and the value arguments begin?  @michaelpj wants to write (builtin b (seq t_1 ... t_m) v_1 ... v_n), but then when there are no type arguments (which is always the case at the moment) we have to write things like (builtin addInteger (seq) (con 1) (con 4), which is a bit strange.

This has huge conflicts, if we want to pick up the remaining pieces more we can write some more issues.

I've been meaning to take a look at this to check to  see if anything's unresolved.  I'll do that shortly and open some more issues if necessary.

I found open issues for most of the things that I was concerned about so I've given them all a prod as a reminder to think about them.


[INFRA] Close stale PRs

Closes #16819
Closes #13467
Closes #16083
Closes #17135
Closes #8785
Closes #16278
Closes #16997
Closes #17073
Closes #17220

Test build #75048 has started for PR 17386 at commit d88bc61.

@srowen, I have few you might want to consider. I picked up inclusively.
12059
12524
12888
16061

Test build #75048 has finished for PR 17386 at commit d88bc61.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75048/
Test PASSed.

Merged to master


shellIntegration-rc.zsh: escape values in "E" (executed command) and "P" (property KV) codes

This implements the string escaping scheme documented for the "E" (executed) and "P" (property KV) codes. This is "pure" zsh and relies on no external utilities.
Although currently the only "required" escapes are backslash, semicolon, \a, and perhaps newlines, this conservatively escapes all non-alphanumeric characters.
Backslashes are doubled, non-alphanumerics are hex-escaped.
Sibling to:

#165631
#165632
#165634
#165635

Relates to or resolves parts of:

#157546
#155639

I've opened #165665 re the lint failure on the diagram comment.
The existing "hygiene" script apparently only permits the characters ┌└├ (omitting soft elbows, down-left connectors of any kind, and horizontal/vertical lines) 😑
These could be redone in ASCII, but with loss of legibility.

By the way, sorry for stomping on that review with the push! I usually rebase my unreviewed branches in bulk just to keep them current and if I don't notice a review beforehand it'll clober. I'll look into updating my scripts to check for that in the last step, since reviewers also work in batches :) Done with that for now unless other change requests pop up.

No worries, that's best in my experience and what I do; don't force push after a review has occurred


One test fails when I run bin/test locally

Setup
Versions

Rust: rustc 1.31.0 (abe02cefd 2018-12-04)
Diesel: v1.4.2 and 0b6c59e36cef3faea0184404e43e3db4739d1193 (latest master at the time of writing)
Database: mysql  Ver 8.0.17 for osx10.13 on x86_64 (Homebrew) - client libraries, 8.0.17 - server.
Operating System docker host is macOS High Sierra 10.13.6 (17G65), not sure what OS mysql container runs on.

Feature Flags

diesel: mysql

Problem Description
When I run bin/test, all the tests pass except one.
What are you trying to accomplish?
I want to have all tests green on my machine in order to be able to play with diesel source code and possibly contribute.
What is the expected output?
All tests are green.
What is the actual output?
$ cd diesel && RUST_TEST_THREADS=1 cargo test --no-default-features --features mysql

...

---- src/query_dsl/mod.rs - query_dsl::QueryDsl::inner_join (line 398) stdout ----
thread 'src/query_dsl/mod.rs - query_dsl::QueryDsl::inner_join (line 398)' panicked at 'test executable failed:

thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `Ok([("Sean", "Sean\'s post"), ("Sean", "Sean is a jerk")])`,
 right: `Err(DatabaseError(__Unknown, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near \'|| ?)\' at line 1"))`', src/query_dsl/mod.rs:30:1
note: Run with `RUST_BACKTRACE=1` for a backtrace.

', librustdoc/test.rs:367:17
note: Run with `RUST_BACKTRACE=1` for a backtrace.

failures:
    src/query_dsl/mod.rs - query_dsl::QueryDsl::inner_join (line 398)

test result: FAILED. 116 passed; 1 failed; 16 ignored; 0 measured; 0 filtered out

I tried it on the latest master and on v1.4.2. Same error.
Are you seeing any additional errors?
Nope.
Steps to reproduce
I followed the contributing guidelines when setting up everything locally. Here is what I did, precisely:

Clone the repo and cd into it
cp .env.sample .env, also bump all ports by 1, i.e. 5432 -> 5433 and 3306 -> 3307.
Made the same changes in docker-compose.yml, so that ports from docker don't conflict with my local database servers.
brew install mysql
bin/test

Checklist

[x] I have already looked over the issue tracker for similar issues.
[x] This issue can be reproduced on Rust's stable channel.

It seems like this issue is specific to mysql 8.0.17. We are already aware of this but nobody had enough time to look into it yet. Maybe I will find some time in the next few days. In the mean time just use 8.0.16 which should work fine.
See #2108 for some details.

Yep, it worked. The following diff fixes the problem:
diff --git a/docker-compose.yml b/docker-compose.yml
index 80bd7b1a..9592e55a 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -1,7 +1,7 @@
 version: '3'
 services:
   mysql:
-    image: mysql
+    image: mysql:8.0.16
     container_name: diesel.mysql
     volumes:
       - "mysql-data:/var/lib/mysql/:delegated"

I also had to delete old docker volume with docker volume rm diesel_mysql-data because otherwise mysql wouldn't start.
Thank you for the prompt response and feel free to close this issue if it's a duplicate.

Looking at the error message and MySQL changelog, this seems relevant:
https://dev.mysql.com/doc/relnotes/mysql/8.0/en/news-8-0-17.html#mysqld-8-0-17-deprecation-removal

Use of || is deprecated unless the PIPES_AS_CONCAT SQL mode is enabled. In that case, || signifies the SQL-standard string concatenation operator).

Although we do set PIPES_AS_CONCAT there: https://github.com/diesel-rs/diesel/blob/0b6c59e36cef3faea0184404e43e3db4739d1193/diesel/src/mysql/connection/mod.rs#L134... So it should still work.

Thanks for your report, I'll take a look shortly.

I updated docker-compose to run the two versions of mysql side by side on different ports, and created a small isolated script to show the difference between them. Mysql client needs to be installed locally in order to run them.
# Note: the two commands are only different by port number.
$ echo "SET sql_mode=(SELECT CONCAT(@@sql_mode, ',PIPES_AS_CONCAT')); select 'ab' like 'a' || '%'" | mysql --port 3307 --host localhost --protocol=TCP --user root diesel_test
'ab' like 'a' || '%'
1

$ echo "SET sql_mode=(SELECT CONCAT(@@sql_mode, ',PIPES_AS_CONCAT')); select 'ab' like 'a' || '%'" | mysql --port 3308 --host localhost --protocol=TCP --user root diesel_test
ERROR 1064 (42000) at line 1: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '|| '%'' at line 1

My docker-compose:
version: '3'
services:
  mysql:
    image: mysql:8.0.17
    container_name: diesel.mysql
    volumes:
      - "mysql-data:/var/lib/mysql/:delegated"
      - "./docker/mysql/init:/docker-entrypoint-initdb.d"
    ports:
      - "3308:3306"
    environment:
      - MYSQL_ALLOW_EMPTY_PASSWORD=true

  mysql-old:
    image: mysql:8.0.16
    container_name: diesel.mysql.old
    volumes:
      - "mysql-data-old:/var/lib/mysql/:delegated"
      - "./docker/mysql/init:/docker-entrypoint-initdb.d"
    ports:
      - "3307:3306"
    environment:
      - MYSQL_ALLOW_EMPTY_PASSWORD=true

volumes:
  mysql-data:
    driver: local
  mysql-data-old:
    driver: local

So I've digged a bit further into this:
Enabling PIPES_AS_CONCAT seems to work fine on both versions because SELECT 'abc' || 'def'; fine for both.
The actual query that fails is this one:
SELECT `users`.`name`, `posts`.`title` FROM (`users` INNER JOIN `posts` ON `posts`.`title` LIKE `users`.`name` || '%');

Notably the concat operator is used in the on clause of an join.  I was also able to reproduce this using the concat operator as part of a normal where clause, so the new version seems to have changed the behaviour their. I would say this is a bug of mysql side and should be reported there. Anyone interested in doing that?
Interestingly this is fixable by just adding a set of brackets around the strings for the concat operator.  So the following query works fine:
SELECT `users`.`name`, `posts`.`title` FROM (`users` INNER JOIN `posts` ON `posts`.`title` LIKE (`users`.`name` || '%'));

I will think about what's the best way to work around that in short term on our end.

I'll report a bug. Will post a link there later.

Oh, good point, this query also works fine.
SET sql_mode=(SELECT CONCAT(@@sql_mode, ',PIPES_AS_CONCAT')); select 'ab' like ('a' || '%');

https://bugs.mysql.com/bug.php?id=96405

FWIW, I think I worked around the bug locally with slightly smaller changes than 56a0adf17827937325b48ac3f81c21fd72717248. The following patch fixes the test for 8.0.17, and 8.0.16 also works as before.  But it compiles!
diff --git a/diesel/src/expression_methods/text_expression_methods.rs b/diesel/src/expression_methods/text_expression_methods.rs
index 55395bb2..9e1e31e8 100644
--- a/diesel/src/expression_methods/text_expression_methods.rs
+++ b/diesel/src/expression_methods/text_expression_methods.rs
@@ -1,6 +1,7 @@
 use expression::operators::{Concat, Like, NotLike};
 use expression::{AsExpression, Expression};
 use sql_types::{Nullable, Text};
+use expression::grouped::Grouped;

 /// Methods present on text expressions
 pub trait TextExpressionMethods: Expression + Sized {
@@ -55,8 +56,8 @@ pub trait TextExpressionMethods: Expression + Sized {
     /// assert_eq!(Ok(expected_names), names);
     /// # }
     /// ```
-    fn concat<T: AsExpression<Self::SqlType>>(self, other: T) -> Concat<Self, T::Expression> {
-        Concat::new(self, other.as_expression())
+    fn concat<T: AsExpression<Self::SqlType>>(self, other: T) -> Grouped<Concat<Self, T::Expression>> {
+        Grouped(Concat::new(self, other.as_expression()))
     }

     /// Returns a SQL `LIKE` expression

Yes that also would work with the downside of changing the type signature of queries containing a concat operator. I would rather like to avoid changing that signature in a incompatible way, if there is another way, but probably other people from @diesel-rs/contributors have different opinion here.

I think 56a0adf17827937325b48ac3f81c21fd72717248 seems better that it just changes the way || parses without touching fn concat.

Looking at it - maybe it's a good idea to use parentheses more generously in general.
I just found another bug (in diesel this time). Consider this:
diff --git a/diesel/src/query_dsl/mod.rs b/diesel/src/query_dsl/mod.rs
index 584cb29d..db2c0f8d 100644
--- a/diesel/src/query_dsl/mod.rs
+++ b/diesel/src/query_dsl/mod.rs
@@ -415,6 +415,7 @@ pub trait QueryDsl: Sized {
     ///
     /// let data = users
     ///     .inner_join(posts.on(title.like(name.concat("%"))))
+    ///     .filter(title.eq("ABRA").eq(title.eq("CADABRA")))
     ///     .select((name, title))
     ///     .load(&connection);
     /// let expected_data = vec![

You can run the test with cd diesel && RUST_TEST_THREADS=1 cargo test --no-default-features --features postgres query_dsl::QueryDsl::inner_join.
This change should not break the test because title = "ABRA" is false for all rows in test data and title = "CADABRA" is also false. false = false should evaluate to true, so the generated WHERE shouldn't filter out any rows and the test should pass (even on 8.0.16).
It doesn't, however, because the generated condition evaluates to false, so no records are returned as a result. The reason is missing brackets.
-- this is generated
WHERE title = title = title = title

-- this should be generated instead
WHERE (title = title) = (title = title)

mysql> select (2=2)=(2=2), 2=2 = 2=2;
+-------------+-----------+
| (2=2)=(2=2) | 2=2 = 2=2 |
+-------------+-----------+
|           1 |         0 |
+-------------+-----------+
1 row in set (0.00 sec)

More than that, in postgres the title = title = title = title part yields a syntax error.
This example is a bit contrived, but I'm pretty sure there are other, more realistic cases for which missing parentheses mean broken SQL.


Remove local session warning

Summary
Details and comments
Fixes #1626

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.005%) to 83.662%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
6273

Relevant Lines:
7498

💛  - Coveralls


pitest seems not to be working

When ./gradlew pitest is executed some output is printed and looks like no mutation tests are running.
Output:
DEPRECATION WARNING. `testPlugin` is deprecated starting with GPP 1.7.4. It is also not used starting with PIT 1.6.7 (to be removed in 1.8.0).
> Task :clean

> Task :compileJava

> Task :processResources
> Task :classes

> Task :compileTestJava

> Task :processTestResources
> Task :testClasses

> Task :pitest
Option (* = required)                   Description
---------------------                   -----------
-?, -h                                  show help
--avoidCallsTo <comma separated list
  of packages to consider as
  untouchable logging calls>
--classPath <coma separated list of
  additional classpath elements>
--classPathFile <File: File with a
  list of additional classpath
  elements (one per line)>
--coverageThreshold <Integer: Line      (default: 0)
  coverage below which to throw an
  error>
--dependencyDistance <Integer: maximum  (default: -1)
  distance to look from test for
  covered classes>
--detectInlinedCode [Boolean: whether   (default: true)
  or not to try and detect code
  inlined from finally blocks]
--excludedClasses <comma separated
  list of globs for classes to exclude
  when mutating>
--excludedGroups <TestNG groups/JUnit
  categories to include>
--excludedMethods <comma separated
  list of filters to match against
  methods to exclude from mutation
  analysis>
--excludedTestClasses <comma separated
  list of globs of test classes to
  exclude>
--exportLineCoverage [Boolean: whether  (default: true)
  or not to dump per test line
  coverage data to disk]
--failWhenNoMutations [Boolean:         (default: true)
  whether to throw error if no
  mutations found]
--features <comma separated list of
  features to enable/disable.>
--fullMutationMatrix <Boolean: Whether  (default: false)
  to create a full mutation matrix>
--historyInputLocation <File: File to
  read history from for incremental
  analysis>
--historyOutputLocation <File: File to
  write history to for incremental
  analysis>
--includeLaunchClasspath [Boolean:      (default: true)
  whether or not to analyse launch
  classpath]
--includedGroups <TestNG groups/JUnit
  categories to include>
--includedTestMethods <Test methods
  that should be included for
  challenging the mutants>
--jvmArgs <comma separated list of
  child JVM args>
--jvmPath <path to java executable>
--maxMutationsPerClass <Integer: No     (default: 0)
  longer supported. Use CLASSLIMIT
  (limit[42]) feature instead>
--maxSurviving <Integer: Maximum        (default: -1)
  number of surviving mutants to allow
  without throwing an error>
--mutableCodePaths <Globs identifying
  classpath roots containing mutable
  code>
--mutationEngine <mutation engine to    (default: gregor)
  use>
--mutationThreshold <Integer: Mutation  (default: 0)
  score below which to throw an error>
--mutationUnitSize <Integer: Maximum    (default: 0)
  number of mutations to include
  within a single unit of analysis>
--mutators <comma separated list of
  mutation operators>
--outputFormats <comma separated list   (default: HTML)
  of listeners to receive mutation
  results>
--pluginConfiguration <KeyValuePair:
  custom plugin properties>
* --reportDir <directory to create
  report folder in>
--skipFailingTests [Boolean: whether    (default: false)
  to ignore failing tests when
  computing coverage]
* --sourceDirs <File: comma separated
  list of source directories>
* --targetClasses <comma separated
  list of filters to match against
  classes to test>
--targetTests <comma separated list of
  filters to match against tests to
  run>
--testPlugin <test plugin to use>       (default: junit)
--threads <Integer: number of threads   (default: 1)
  to use for testing>
--timeoutConst [Long: constant to       (default: 4000)
  apply to calculate maximum test
  duration]
--timeoutFactor [Float: factor to       (default: 1.25)
  apply to calculate maximum test
  duration]
--timestampedReports [Boolean: whether  (default: true)
  or not to generated timestamped
  directories]
--useClasspathJar [Boolean: support     (default: false)
  large classpaths by creating a
  classpath jar]
--verbose [Boolean: whether or not to   (default: true)
  generate verbose output]
>>>> verbosity is not a recognized option

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

See https://docs.gradle.org/7.6.1/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 3s
6 actionable tasks: 6 executed

Fixed.


CyberSecurity Tools

Adding information about different security tools

I would appreciated it if you assigned me 😊

Thank You for adding Cyber Security Resources.
Hope this will help others in their learnings. ❤


Make sure examples and readmes work in windows

Fixes #23.
Changes proposed in this PR:

Updated all predict-eth readmes and examples to be DRY and ensure compatibility with windows, mac and linux
in main2: replaced preinstalling wheel with the commands that prevent errors in mac and linux (step not needed in windows)
web3 dependency can cause conflict because brownie and pybundlr use incompatible versions. Leave like this because this will be fixed in pybundlr

@juanpablovs please lmk when you're ready for a review again. Thanks


Development scripts

closes #2927
Scripts
This adds the four scripts proposed in #2927.
The scripts all follow the basic pattern of iterating over the go modules in the repo and running the appropriate commands in each directory. Because of this, there is some duplicate code. I chose to keep it this way because deduping it added complexity, and I wanted to keep these scripts as simple as possible to understand.
The only dependencies for these scripts are git and a posix shell.

scripts/fmt is the simplest one. It runs go fmt ./... in each module directory.

scripts/generate runs go generate ./... and go mod tidy -compat '1.17' in each module directory. It also has a --check flag which causes it to run on a temporary git worktree and fail if any new diff is created.

scripts/lint runs golangci-lint run on each module then runs scripts/generate --check. It installs golangci-lint in bin/ if needed.

scripts/test runs go test in each module directory. If arguments are provided, those are used as arguments for go test, otherwise it uses the default arguments -race -covermode atomic ./...

scripts/lint and scripts/test skip the module in example/newreposecretwithlibsodium because it cannot compile unless libsodium is available.
CONTRIBUTING.md

Updates CONTRIBUTING.md to include the new scripts and generally updates the "Submitting a patch" section.

Workflow changes
linter.yml

Runs scripts/lint instead of using the golangci-lint action.
No longer uses a run matrix. Because scripts/lint iterates over modules, I was able to remove the run matrix. We now only run one lint job instead of four.
Stale generated files will now be reported on lint runs because script/lint validates go generate.

tests.yml

Runs scripts/test instead of running go test multiple times in separate steps.
No longer checks go generate now that it is checked in lint.
Only sets -coverprofile when codecov will be updated because -coverprofile prevents cached test results from being used.
Sets default shell to bash so that it doesn't need to be specified on each step.

Misc changes
Running scripts/test exposed an issue where the examples module won't compile on Windows because syscall.Stdin is typed differently on Windows. I resolved this by replacing syscall.Stdin with int(os.Stdin.Fd())

Would it bother/upset you if I asked you to rename all the bash scripts in the "scripts" directory to have a ".sh" suffix?
This is a convention I've been using for decades, and I'd love to continue that, but won't be heart-broken if you object.

I can go either way on script names. I noticed .gitignore has a *.sh line. I'll remove that altogether because I suspect it isn't necessary anymore. If that causes a problem I can restore it and add !/scripts/*.sh

I can go either way on script names. I noticed .gitignore has a *.sh line. I'll remove that altogether because I suspect it isn't necessary anymore. If that causes a problem I can restore it and add !/scripts/*.sh

Ah, yes, please leave the *.sh and add your workaround, as I have a slew of hacky bash scripts in the top-level that help me maintain this repo. Thank you!  :smile:

Thank you, @valbeat !
@WillAbides - do you want to change "scripts" back to your original "script" before I merge?

@gmlewis It's ready to go now


Mapfile support for explicitly untyped gvars

lgtm

Note you should probably always consider making your own dedicated feature branch when making PRs, it's surprising to see this branch is called debuginfo XD


Dependency error

It seems that an indirect dependency "Skinney/murmur3" is not downloadable.

Hi @myh98, thanks for flagging, I'm on it!!

@myh98 this Skinney/murmur3 package is not available in indirect dependencies anymore: https://github.com/peterszerzo/elm-arborist/blob/master/elm.json. Are you by chance looking at an older version that still uses elm-css?

Closing this as there are no issues with current versions. If there is more interest in getting an Elm 0.18-compatible version fixed, I can look into it at that point.


Flickering list of Traces in Chrome

Describe the Bug
In Chrome only : after a search, the list of traces flickers. It occurs only for maximised window, but very annoying.
Steps to Reproduce
Open Zipkin in a maximised Chrome window. Select a service which contains some traces, search... the list of traces starts flickering.

Can you confirm this is on latest version? We fixed a flickering bug before but I think there may still be some corner cases left and will look in.
Also would you mind sharing your screen resolution, which I guess influences the size of the maximized window?

version is 2.20.1
resolution is 1920*1080
Important : flickering happens only with a 150% size of text (display settings). With 125% and 100% it's fine.

This issue may be solved #3180 and #3076


Arc2d does not implement GizmoPrimitive2d

Bevy version
main
What you did
Try to draw a Arc2d using gizmos.primitive_2d().
What went wrong
Got an error:
Compiling bevy_contrib_raycast v0.1.0-dev (/Users/olle/Dev/bevy_contrib_raycast)
error[E0277]: the trait bound `bevy::prelude::Gizmos<'_, '_>: bevy::prelude::GizmoPrimitive2d<bevy::prelude::Arc2d>` is not satisfied
   --> examples/visual.rs:163:33
    |
163 |             gizmos.primitive_2d(&arc, position.0, rotation.0, RED);
    |                    ------------ ^^^^ the trait `bevy::prelude::GizmoPrimitive2d<bevy::prelude::Arc2d>` is not implemented for `bevy::prelude::Gizmos<'_, '_>`
    |                    |
    |                    required by a bound introduced by this call
    |
    = help: the following other types implement trait `bevy::prelude::GizmoPrimitive2d<P>`:
              <bevy::prelude::Gizmos<'w, 's, Config, Clear> as bevy::prelude::GizmoPrimitive2d<bevy::prelude::Dir2>>
              <bevy::prelude::Gizmos<'w, 's, Config, Clear> as bevy::prelude::GizmoPrimitive2d<bevy::prelude::Circle>>
              <bevy::prelude::Gizmos<'w, 's, Config, Clear> as bevy::prelude::GizmoPrimitive2d<bevy::prelude::Ellipse>>
              <bevy::prelude::Gizmos<'w, 's, Config, Clear> as bevy::prelude::GizmoPrimitive2d<Annulus>>
              <bevy::prelude::Gizmos<'w, 's, Config, Clear> as bevy::prelude::GizmoPrimitive2d<Rhombus>>
              <bevy::prelude::Gizmos<'w, 's, Config, Clear> as bevy::prelude::GizmoPrimitive2d<bevy::prelude::Capsule2d>>
              <bevy::prelude::Gizmos<'w, 's, Config, Clear> as bevy::prelude::GizmoPrimitive2d<bevy::prelude::Line2d>>
              <bevy::prelude::Gizmos<'w, 's, Config, Clear> as bevy::prelude::GizmoPrimitive2d<bevy::prelude::Plane2d>>
            and 8 others

For more information about this error, try `rustc --explain E0277`.

Additional information
`Arc2d was added recently, so we probably just forgot to implement this. IMO, this should happen before 0.14 though.

Yep, good to have. Not worth blocking the release over, but if it's done I'll review and merge it.

Oh while I'm at it, I'll fix it for CircularSegment and CircularSector too.


Setup some tooling for publishing

This PR sets up some tooling

Use @react-native-community/bob to build the package for publishing, this avoids extra setup needed for people using Jest
Set up husky with commitlint to enforce conventional commits, this is needed to be able to auto-generate changelog
Set up release-it for automated publishing (check README.md for pre-requisites)

but


Create and attach empty unmamaged disk

I cannot find info how to create an empty unmanaged disk on a storage account and attach it to a vm using Powershell.
Only how to create a manged one or attach an existing unmanaged.
There is an option with Azure CLI to attach a new empty unmanaged vhd. Why not with Powershell??

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

ID: 3799662b-8f0f-c4e6-26b5-b2ff24a9242a
Version Independent ID: 0c93223c-f28b-7ca7-0d10-c19e5fea9197
Content: About unmanaged (page blobs) and managed disks storage for Microsoft Azure Windows VMs
Content Source: articles/virtual-machines/windows/about-disks-and-vhds.md
Service: virtual-machines-windows
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the feedback! We are currently investigating and will update you shortly.

@michal-staniszewski
@roygara  I am not finding any specific documentation on it. Are you aware of any specific doc on creating new unmanaged disk on storage account and attaching it to VM, using Powershell?

@michal-staniszewski
Thanks for bringing this to our attention. I have assigned the issue to the content author to evaluate and update the document as appropriate.
We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion

@Karishma-Tiwari-MSFT I am not currently aware of any and haven't located one either. I have created a task and begun researching this though, for reference. We can keep this closed accordingly.
@michal-staniszewski thanks for pointing this out, we appreciate it! When I discover an answer I'll notify you via here.


[docs] Move more prop docs into IntelliSense

Includes a bunch of type fixes that might cause new TypeScript errors:

Rating:

Reject children. They have no effect.
Widen icon and emptyIcon to React.ReactNode (from React.ReactElement)
Add missing emptyLabelText

ExpansionPanel:

Reject undefined and null from children. This would cause a runtime error

FormControlLabelProps:

Reject children. They have no effect.

InputBase:

Add renderSuffix
Reject renderPrefix. This would cause a runtime warning. It was likely mistaken for renderSuffix
Reject children. They have no effect

Popper:

Reject children. They have no effect

ScopedCssBaseline:

Add classes prop

Slide:

Reject theme. It has no effect.

StepConnector:

Reject children. They have no effect.
Remove typings for private props

StepContent:

Remove typings for private props

StepIcon:

Reject children. They have no effect.
Remove typings for private props

StepLabel:

Remove typings for private props

Stepper:

Narrow connect to React.ReactElement (from React.ReactNode). Arbitrary nodes would cause a crash.

The following components need more work. I'll leave them be for now:

Hidden
Modal
RootRef
SwipeableDrawer

Hey @eps1lon where can I read on why some props (like StepLabel completed) are now considered private?
Those were documented in 4.8, and at first glance I can't see any implementation logic that has changed.

Those were documented in 4.8, and at first glance I can't see any implementation logic that has changed.

They were typed, but not documented: https://v4-0-2.material-ui.com/api/step-label/#props. Please open a new issue explaining your use case.

@tmikoss A related issue: #21326. Replacing the usage of the cloneElement API might cause issues in your case.


fix: autoplay on both android & iOS

Fixes issue #97
Description of changes:
The implementation of autoplay was broken, even if it was passed as false stream would still play. So the onLoad callback seemed ideal place to check if autoplay prop has been passed from the user. If it's true we call pause method and stop the stream. We had to provide custom implementation for autoplay because the underlying SDK itself doesn't yield this prop.
Usage:
  return <IVSPlayer
          autoplay={false}
          streamUrl={URL} />

Testing:

Before autoplay on android
https://user-images.githubusercontent.com/47336142/205951043-c86258ee-c525-4743-94d9-bbe96726d125.mp4

After autoplay on android
https://user-images.githubusercontent.com/47336142/205950978-4c7962c4-add4-46b8-ab35-632d52c71d5c.mp4

Before autoplay on iOS
https://user-images.githubusercontent.com/47336142/205950737-925bf521-229a-4a63-a300-a6f5d80d21fc.mp4

After autoplay on iOS
https://user-images.githubusercontent.com/47336142/205950852-0a3bfa47-3c64-4e63-b98c-5ec2dc345d56.mp4

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

With this change the behavior would now become:
With autoplay = true
play -> load is called, load event is raised -> play
With autoplay = false
play -> load is called, load event is raised -> pause
During the load is called, load event is raised step it is possible for the video to begin playing, then pause.

@david-pw Thanks for the review, it helped in addressing the changes more concretely 🌟


Import of ESM fails when declaration file present/not named as *.d.mts

Bug Report
The Typescript compiler reports following error when importing an ESM package (using "type": "module"):
"error TS1471: Module 'minimal-lib/dist/math.js' cannot be imported using this construct. The specifier only resolves to an ES module, which cannot be imported synchronously. Use dynamic import instead."
The problem seems to be how the .d.ts is interpreted (i.e. as CJS not ESM).
Removing the .d.ts or renaming to .d.mts resolves error.
Note: I briefly tried using .mjs as lib file extension, but that seemed to have problems too. (Even though declaration was then named .d.mts.)
🔎 Search Terms
.d.ts
🕗 Version & Regression Information
Using 4.7.4.
Haven't tried in other versions.
⏯ Playground Link
sorry, not provided.
💻 Code
See minimal reproduction code:
https://github.com/blayet/minimal-ts-bug
🙁 Actual behavior
Error compiling a lib that contains typescript-generated declaration file.
🙂 Expected behavior
Compilation by tsc of a lib generated by tsc should work without having to rename files.

Looks like duplicate of #50009

Not sure - that ticket talks about "Consider adding "type": "module" to this 'package.json' file."
That's already present in this case.

It's present in minimal-lib, but minimal-app doesn't have "type": "module" and therefore index.ts would be treated as CommonJS.
From common JS you can't require ESM file (which is pointed by error) as import would've been transpired to require if module was node16 (probably this is expectation for "moduleResolution: "node16"). If you add "type": "module" to minimal-app as error suggests - it's fixed.
With "module": "es2020" you have invalid CommonJS output as import isn't transpiled to require.
So not sure what is your expected runtime and use case for mixing "module": "es2020" and "moduleResolution": "node16", but I think there is mismatch between module, moduleResolution, and type in configs for minimal-app

Huh. Yeah looks like you're right. (I thought I had "type": "module" in the app too, to be honest.)
Looks like I was confused by error as described in #50009 - as you say.
Thanks :)
Regarding runtime and use case. Currently its just for internal usage. I want to build a lib for use in web apps. I'll use webpack to bundle the app. Not sure about others downstream although we can probably assume/mandate use of a bundler - and definitely for web apps.
I picked "module": "es2020" because (reading https://www.typescriptlang.org/tsconfig#module) I noticed that "ES2020 adds support for dynamic imports, and import.meta." I thought that might be relevant  - not now but at some future point.
As far as I understand it any ESM choice would have been fine.

It sounds like this is resolved?
We're working to put together some sample repos to hopefully clear this up.


Client Allocation Request for: IPFS Found

Client Allocation Questions
Core Information

Name: IPFS Found
Website / Social Media: https://filecoin.in/
DataCap Requested: 5TiB
Addresses to be Notarized: f1r2gyg6x7e4un527vmquzu6ysumtwr4iuqfrusry
Notary Requested: andrewxhill

Hi @IPFSUnion, as a first step could you first verify what region you are operating in? This may include where you company//organization is based, where your technology is based, and where your customers are based.

Hi @IPFSUnion, as a first step could you first verify what region you are operating in? This may include where you company//organization is based, where your technology is based, and where your customers are based.

thanks @andrewxhill,
IPFS Found  is developed by the IPFSUNION team, Based on Filecoin network, IPFS Found is designed as a storage and retrieval to help users store various types of files, including text, picture, audio, video, etc. And it supports storage of multiple data sizes. We aim to build more decentralized, stable and robust data storage platform.

About IPFSUNION:
Official website：https://ipfsunion.cn/
Self-developed Filecoin browser：https://filscout.io
Twitter: https://twitter.com/ipfsunion_cn
Github：https://github.com/IPFSUnion
Zhihu：https://zhuanlan.zhihu.com/p/173753718

IPFSUnion team is based on Shanghai, China. IPFS Found hope to get the datacap to encourage more miners to store the real data. For now we are particapanting the Slingshot Competition
IPFS found is developed based on the PowerGate to realize the data deal and retrieve.
For now, IPFS Found offerthe service of storing the data to the Filecoin network for free. Everyone can store the data to the Filecoin network by the IPFS Found. In the short term, we will continue to store the public data to the Filecoin network for free. In the long term, ‘IPFS Found’ will store more and more real data to the Filecoin network. Firstly. We will choose to store the public data.

thanks @andrewxhill,
IPFS Found  is developed by the IPFSUNION team, Based on Filecoin network, IPFS Found is designed as a storage and retrieval to help users store various types of files, including text, picture, audio, video, etc. And it supports storage of multiple data sizes. We aim to build more decentralized, stable and robust data storage platform.

About IPFSUNION:
Official website：https://ipfsunion.cn/
Self-developed Filecoin browser：https://filscout.io
Twitter: https://twitter.com/ipfsunion_cn
Github：https://github.com/IPFSUnion
Zhihu：https://zhuanlan.zhihu.com/p/173753718

IPFSUnion team is based on Shanghai, China. IPFS Found hope to get the datacap to encourage more miners to store the real data. For now we are particapanting the Slingshot Competition
IPFS found is developed based on the PowerGate to realize the data deal and retrieve.
For now, IPFS Found offerthe service of storing the data to the Filecoin network for free. Everyone can store the data to the Filecoin network by the IPFS Found. In the short term, we will continue to store the public data to the Filecoin network for free. In the long term, ‘IPFS Found’ will store more and more real data to the Filecoin network. Firstly. We will choose to store the public data.

Hey @IPFSUnion thanks for the info, it sounds like a great project. We're not focusing on the China region at this time, you should re-apply with a Notary that is focused on that area. You can find that info in the Location column of the verifiers table https://plus.fil.org/verifiers.


[glib] Build error "Host path separator (:) in path; this is unsupported."

Host Environment

OS: Pop!_OS (Ubuntu) 21.10
Compiler: cmake version 3.18.4 (I think this is what you need?)

To Reproduce
Steps to reproduce the behavior:
./vcpkg/vcpkg install glib
Failure logs
digitalkitsune@pop-os:~/github/Alicevision$ ./vcpkg/vcpkg install glib
Computing installation plan...
The following packages will be built and installed:
    glib[core]:x64-linux -> 2.70.1#2
Detecting compiler hash for triplet x64-linux...
Restored 0 packages from /home/digitalkitsune/.cache/vcpkg/archives in 4.776 us. Use --debug to see more details.
Starting package 1/1: glib:x64-linux
Building package glib[core]:x64-linux...
-- Using cached glib-2.70.1.tar.xz.
-- Cleaning sources at /home/digitalkitsune/github/Alicevision/vcpkg/buildtrees/glib/src/glib-2-3e22263b95.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /home/digitalkitsune/github/Alicevision/vcpkg/downloads/glib-2.70.1.tar.xz
-- Applying patch use-libiconv-on-windows.patch
-- Applying patch libintl.patch
-- Using source at /home/digitalkitsune/github/Alicevision/vcpkg/buildtrees/glib/src/glib-2-3e22263b95.clean
-- Getting CMake variables for x64-linux-dbg
-- Getting CMake variables for x64-linux-rel
-- Configuring x64-linux-dbg
CMake Error at scripts/cmake/vcpkg_host_path_list.cmake:43 (message):
  Host path separator (:) in path; this is unsupported.
Call Stack (most recent call first):
  scripts/cmake/vcpkg_host_path_list.cmake:72 (z_vcpkg_translate_to_host_path_list)
  scripts/cmake/vcpkg_configure_meson.cmake:488 (vcpkg_host_path_list)
  ports/glib/portfile.cmake:43 (vcpkg_configure_meson)
  scripts/ports.cmake:142 (include)

Error: Building package glib:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `git pull` and `./vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+glib
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[glib]+Build+error
including:
  package: glib[core]:x64-linux -> 2.70.1#2
    vcpkg-tool version: 2021-12-09-724cea8b33cbf06c645f5095fa29773697da9761
    vcpkg-scripts version: 7baf7bc9f 2022-01-26 (5 hours ago)

Additionally, attach any relevant sections from the log files above.

Additional context
I actually need opencv[eigen,ffmpeg,webp,contrib,nonFree,cuda] and openimageio[libraw,ffmpeg,freetype,opencv,gif,openjpeg,webp] (among other things) as a build dependency and this is just one of it's dependency's.

Try removing the : from your PATH ;)

Ok, but I'm on linux, what should I replace the separators with?
For refrence this is my path (newlines added for readability):
/home/digitalkitsune/github/meshroom/build/Linux-x86_64:
/usr/lib/cuda:/usr/lib/cuda/bin:
/home/digitalkitsune/github/emsdk:/home/digitalkitsune/github/emsdk/node/12.18.1_64bit/bin:/home/digitalkitsune/github/emsdk/upstream/emscripten:
/home/digitalkitsune/.deno/bin:
/home/digitalkitsune/.nvm/versions/node/v8.16.2/bin:
/home/digitalkitsune/.cargo/bin:
/home/digitalkitsune/.local/bin:
/home/digitalkitsune/bin:
/usr/local/sbin:/usr/local/bin:
/usr/sbin:/usr/bin:
/sbin:/bin:
/usr/games:/usr/local/games:
/snap/bin:
/home/digitalkitsune/.local/bin:
/home/digitalkitsune/.local/bin:
/home/digitalkitsune/.npm-packages/bin

I was thinking about posting a separate issue about this, as I've been following this for about a month, but there is finally activity again.
I was able to get around this by modifying vcpkg_host_path_list.cmake (line ~ 21-41) to:
       elseif("${VCPKG_HOST_PATH_SEPARATOR}" STREQUAL ":")
            set("${out_var}" "${lst}" PARENT_SCOPE)

            string(FIND "${lst}" [[\:]] index_of_host_path_separator)
        else()

This was just a quick hack to get things working, So far I haven't noticed anything else breaking, but I'm sure it's not correct.
I kept wondering why this only affects me at work and not at home, but this was broken in Nov. 2021, and I'm using an outdated version of vcpkg at home.
Also, this certainly isn't a port issue, it breaks a lot of packages. All of them that make direct/indirect calls to vcpkg_host_path_list, which includes all of the pkg_config, and meson ones (I think).
It's rather disconcerting more people haven't complained about this, apparently there aren't many Linux users, or they are using out-dated vcpkg.
Preventing the use of colons for paths certainly isn't an option, if you want to support Linux. It seems like there could just be a switch on platform to determine which separator to use, and ports need to use "VCPKG_HOST_PATH_SEPARATOR" instead of hard coding : or ;. But, I only dug deep enough to get it to work for my cases.
However, this is a major problem!!!

There are linux users and contributors, but they are not able to reproduce this issue.
The function you point to tranforms a CMake list of directories (/usr/local/bin;/usr/bin;/bin) into a host path list (/usr/local/bin:/usr/bin:/bin for Linux). On you computer, the input CMake list already contains the host path separator (e.g. /usr/local/bin;/usr/bin:/bin). This indicates that this list couldn't be processed correctly in earlier functions.
Perhaps you should share the raw output of echo "$PATH". Because what you shared above also contained the host path separator in some lines. You really need to check you PATH.

The error message from vcpkg_host_path_list is saying "you tried to add a new element to the path that itself contains :, which isn't possible on *nix".
https://github.com/microsoft/vcpkg/blob/5962cf5a3bfc2124e8de991a4e45a8edcb3b4c69/scripts/cmake/vcpkg_configure_meson.cmake#L489
I think the right fix is to change
vcpkg_host_path_list(APPEND ENV{PKG_CONFIG_PATH} "${pkgconfig_installed_dir}" "${pkgconfig_share_dir}" "$ENV{PKG_CONFIG_PATH}")

into:
vcpkg_host_path_list(APPEND ENV{PKG_CONFIG_PATH} "${pkgconfig_installed_dir}" "${pkgconfig_share_dir}")
if (NOT ENV{PKG_CONFIG_PATH} STREQUAL "")
    set(ENV{PKG_CONFIG_PATH} "$ENV{PKG_CONFIG_PATH}${VCPKG_HOST_PATH_SEPARATOR}ENV{PKG_CONFIG_PATH}")
endif()

or similar? It seems $ENV{PKG_CONFIG_PATH} likely already may have path separators in it.
/cc @strega-nil

I think the correct thing is to just turn vcpkg_host_path_list(APPEND ENV{PKG_CONFIG_PATH} "${pkgconfig_installed_dir}" "${pkgconfig_share_dir}" "$ENV{PKG_CONFIG_PATH}") into vcpkg_host_path_list(APPEND ENV{PKG_CONFIG_PATH} "${pkgconfig_installed_dir}" "${pkgconfig_share_dir}"); you're already appending to ENV{PKG_CONFIG_PATH}, you don't need to do it again.

Oh megaderp I didn't see that the variable was duplicated 😅


R-C3D for processing Charades dataset

Thanks a lot for providing the open-source code!
Could you provide the pre-processing file for R-C3D?
Highly appreciate your time and help!

It is already in preprocess folder.
On Dec 8, 2017, at 12:40, LiliMeng <notifications<EMAIL_ADDRESS>wrote:
Thanks a lot for providing the open-source code!
Could you provide the pre-processing file for R-C3D?
Highly appreciate your time and help!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/VisionLearningGroup/R-C3D/issues/3, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFOa_96zxY5BQUHXDgPQORFDuclRm-6yks5s-XSNgaJpZM4Q7a8k.

Thanks a lot!
It seems that only ActivityNet preprocess is in preprocess folder, not Charades.
Thanks again!

@LiliMeng Looks like the folders for both Charades and THUMOS has been left out from this git repo.

The codes for thumos'14 and charades are released.


Fix malformed bower.json

Bower throws an EMALFORMED when attempting to install recent materialize SHAs (i.e., ffb9783b388912fbe1245a1362f0b1ace03d46bd).

Thanks @mike-north


any progress on a linux version?

Since the README shows that version 2 was supposed to be release in 2020 and there hassn't even been a version 1 released yet, I am assuming that there hasn't been any further development on this? I was really looking forward to a linux version.

Sadly it seems like no progress on the main branch in ~3 Years. Wanted to roll out something like this for myself.


Support fallbacking to package.json

Some code in the wild does:
require("./package");

This will load the package.json. We have an existing fallback when a
404 occurs, to load name/index.js, so this just adds a second, in the
case where the address is package.js we'll try (after
package/index.js fails) to load it as package.json. Closes #105

This needs a release.


Add a flag to sfdx force:org:list --all --json to show the accessToken unencrypted for every connected environment

Is your feature request related to a problem? Please describe.
When I run sfdx force:org:display --json the accessToken is unecrypted. I can just get it and use on my requests with Postman.
But when I run sfdx force:org:list --all --json the accessTokens for every connected org is Encrypted.
What are you trying to do
I need to get all the access tokens for all my connected orgs with one command. Currently, I have to keep running sfdx force:org:display for every environment just to get the access token.
Describe the solution you'd like
Add a new flag to this command sfdx force:org:list --all -t --json so it will display the access token unencrypted. Or just don't encrypt the access token.
Describe alternatives you've considered
Maybe there is a way for me to unencrypt it. I saw a .key inside .sfdx. Can I use that? if yes, how would the process be?
Additional context
N/A

Hi @AllanOricil thanks for the input, we're open sourcing those commands here. We'll make the fix there

@AllanOricil we've fixed this in the new plugin-org, which fixes a number of other bugs and performance issues.
If you'd like to try it out early (before it gets bundled into the CLI), you can install it manually: https://github.com/salesforcecli/plugin-org#install

@AllanOricil we've fixed this in the new plugin-org, which fixes a number of other bugs and performance issues.
If you'd like to try it out early (before it gets bundled into the CLI), you can install it manually: https://github.com/salesforcecli/plugin-org#install

@mshanemc I will give it a try. Thanks for that!

@mshanemc it worked! Now I can see the Access Tokens "unencrypted". When will this be available when the user install sfdx? Is it already gonna be installed?

Is this plugin officially released as part of the sfdx?

Yes, in latest as of 4 days ago (updates go through a latest-rc, then promoted to latest the following week).


Add SUNY Polytechnic Institute

@MJWoodworth Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


move dependency cache to /var/cache/openrc

What is the motivation behind this?
This means that the dependency cache is unavailable before localmount is started if /var is mounted on separate partition.

The motivation is to comply better with the FHS guideline. Furthermore,
some systems would have /lib under read-only filesystem (e.g. embedded
systems, immutable systems). At the same time, system with external /var
seems to diminish in number (/var/log & /var/tmp are still popular).
I would also take this opportunity to ask: is the caching even necessary
(or desirable)?

The motivation is to comply better with the FHS guideline.

FHS assumes that those directories are available. The initsytems job is to make them available, an initsystem will always be special.

Furthermore, some systems would have /lib under read-only filesystem (e.g. embedded
systems, immutable systems).

If I understand the sources correctly, openrc will try copy the cache from $RC_LIBEXECDIR/cacheif it exists to /run/openrc/, and recreate it if its missing. If your /lib is read-only, you'd only need to disable the savecache (which is supposed to run during shutdown).

At the same time, system with external /var seems to diminish in number (/var/log & /var/tmp are still popular).

What do you base this on?

I would also take this opportunity to ask: is the caching even necessary (or desirable)?

I'd say yes. You can always disable it by disabling savecache during shutdown if you don't want it.

We do not attempt to save the cache if /lib or /lib/cache are read only, so the concern about read-only file systems is addressed.
Also, as stated, if we move everything to /var/cache, this would not work if /var is a separate file system from /.


Crashes app when container element has display set to none with error width and height must be > 0

Setting display: 'none' style on the container element crashes the app, for example:
<View style={{display: 'none'}}>
  <Svg width={100} height={100}>
    ...
  </Svg>
</View>

Shows a red screen with the error message: width and height must be > 0

I ran into the same problem, and I gave up using "display"

Bump. This is an annoying issue that shouldn't cause a crash.

+1

+1

+1

+1 here.
We "solved" this using conditional rendering instead of display:none .

This seems to work fine now.


flicker / issue rendering ?

./testgeometry : triangle disappears when moving / click with the mouse. it should rotate the triangle instead.
./testsprite2 flickers (not sure ?)
not sure why:  mouse coords ? maybe missing sensor/ joysticks locks ? )

Mouse coords became floats in SDL3, that should be the reason. Still a TODO:
https://github.com/libsdl-org/sdl2-compat/blob/main/src/sdl2_compat.c#L1357-L1358

ok thanks !
maybe the issue, i haven't investigated yet

maybe, does ./testgeometry compile ? if you run it ?


v2 - OpenApiModelFactory.GetStreamAsync should accept a cancellation token

https://github.com/microsoft/OpenAPI.NET/blob/cb5bb2797a4a4aa61c96d4fd453ac5257474848d/src/Microsoft.OpenApi/Reader/OpenApiModelFactory.cs#L300
To pass it to GetStreamAsync. (conditional compilation on NET5 or higher).
All callers to this method should pass the value, and accept one.
This is important as if the action is being cancelled on a backend service, this will forward the cancellation to the HTTP request being made, saving important COGS in the process.

The core library only targets netstandard 2.0


Build failure on Arch Linux

Hi,
i hit an error when install ruby. any idea? By the way, i'm running it archlinux and all build tools are installed.
$ rbenv install --version
ruby-build 20140204

$ uname -a
Linux box 3.12.9-2-ARCH #1 SMP PREEMPT Fri Jan 31 10:22:54 CET 2014 x86_64 GNU/Linux

$ rbenv install -v 2.1.0
................................
compiling iseq.c
compiling vm.c
In file included from vm.c:79:0:
vm_insnhelper.c: In function ‘vm_call_ivar’:
vm_insnhelper.c:1568:1: internal compiler error: Segmentation fault
}
^
Please submit a full bug report,
with preprocessed source if appropriate.
See https://bugs.archlinux.org/ for instructions.
Makefile:333: recipe for target 'vm.o' failed
make: *** [vm.o] Error 1
make: *** Waiting for unfinished jobs....

$ rbenv install -v 2.0.0-p353
................................
compiling dmyversion.c
compiling array.c
linking miniruby
generating encdb.h
./tool/mkconfig.rb:180: syntax error, unexpected ' ', expecting tSTRING_CONTENT or tSTRING_DBEG or tSTRING_DVAR or tSTRING_END
  arch_flag = ENV['ARCHFLAGS'] ...
  ^
./tool/mkconfig.rb:179: unterminated string meets end of file
uncommon.mk:528: recipe for target '.rbconfig.time' failed
make: *** [.rbconfig.time] Error 1
make: *** Waiting for unfinished jobs....
./tool/generic_erb.rb:3:in `require': /tmp/ruby-build.20140205212451.28425/ruby-2.0.0-p353/lib/optparse.rb:758: unterminated string meets end of file (SyntaxError)
/tmp/ruby-build.20140205212451.28425/ruby-2.0.0-p353/lib/optparse.rb:758: syntax error, unexpected end-of-input, expecting keyword_end
	from ./tool/generic_erb.rb:3:in `<main>'
uncommon.mk:886: recipe for target 'encdb.h' failed
make: *** [encdb.h] Error 1

I ran into this issue today on ubuntu 10.  The fix was to install/update systemtap-sdt-dev package.

I've got it working using clang instead of gcc $ ruby-install ruby 2.1.6 -- CC=clang


chore: add some typescript definitions

I've added some typescript definitions of the regexs that are exported.

To use these regex(s) one needs to add this to their tsconfig.json file:
"paths": {
    "ember-cli-app-version/*": [
       "node_modules/ember-cli-app-version"
     ],
}

This makes TS think you can import these from the root which is not correct, as they are located in addon/utils/regexp.js and not addon/index.js. Also, I would consider these private API and not import them in to your app.

I'm going to close this for now.
These regular expressions are only an implementation detail and should not be imported/used directly.

I'm going to close this for now. These regular expressions are only an implementation detail and should not be imported/used directly.

Understood. Thanks for the quick feedback.


Create a contact info block

Add links to social networks (Facebook or Instagram, etc.), phone number or email address, etc.

Done for bulka. Customization coming soon :D


Drafts deletion using delete hotkey triggers stream escape in the background

When you press delete to delete a selected draft in the draft overlay while narrowed in a stream, it deletes the draft, but also changes stream, causing the draft overlay to close by itself.

Merged, thanks @JPJPJPOPOP!


Migrate test cases into mocha test runner

Description: :coffee:
This PR's effort to track #1460, migrate test runner from jasmine to mocha (https://mochajs.org/) with chai for assertion suite, sinon for mocking.
PR contains amount of file changes, major changes are below.

use mocha as test runner
migrate global assertion patching in test-helper into mocha's custom interface by mocha's designed behavior of having separate global context per test suite
update test coverage, marble diagram test accordingly
now test coverage execution on CI is moved into optional phase completely, will make bit faster build turnaround time
fixes couple of tests similar to https://github.com/ReactiveX/rxjs/pull/1536

Related issue (if exists):
#1460
NOTE
: This PR doesn't migrate browser test. Current test environment using mocha with custom interfaces makes bit hard to easier migration of karma test configuration, haven't found simple solution yet.
/cc @staltz for visibility & comment.

Holy crap! Nice work.

Looks good overall, just left a note to spur discussion about where rxTestScheduler should live (don't take my comments and run with them, I just want to discuss it).
As far as I'm concerned, :shipit: ship it. I couldn't be happier to get rid of Jasmine.

As far as I'm concerned, :shipit: ship it. I couldn't be happier to get rid of Jasmine.

: Cool. Let me do this in this way then - check in this change first, then refactor custom interface per suggestion. It's bit hard to play with this PR since it contains too many files. :) Thanks for reviewing!

Merged with a1687c3, say goodbye to jasmine and have :coffee: .
@blesh , @staltz , @david-driscoll , @saneyuki , @trxcllnt ... this requires npm install once again to trigger. test. Just fyi, test execution time hasn't been changed and nearly identical.