Dialogflow Fails to Parse "WebhookResponse"

It seems that when returning a WebhookResponse from a .NET Core API to Google DialogFlow for fulfilment that the response is mangled and incorrectly parsed in some way. Below is the simple controller I have servicing the fulfilment request:
// POST api/dialogflow
[HttpPost]
public IActionResult FulfillDialogFlowRequest([FromBody] WebhookRequest request)
{
    var response = new WebhookResponse
    {
        FulfillmentText = "Hello from " + request.QueryResult.Intent.DisplayName
    };
    return Ok(response);
}

This should return the a blank WebhookResponse with the fulfilment text: Hello from <intent name> apon receiving any WebhookRequest. However, DialogFlow receives the following fulfilment response:

Whereby, every character is seperated by a space. This does not occur when run locally with the sample webhook request and even copying the text removes the spaces displayed here.
The error message for this suggests a parsing error:
Webhook call failed. Error: Failed to parse webhook JSON response: Expect message object but got: \"笀∀昀甀氀昀椀氀氀洀攀渀琀吀攀砀琀∀㨀∀䠀攀氀氀漀 昀爀漀洀 吀攀猀琀嘀愀氀甀攀猀∀Ⰰ∀昀甀氀昀椀氀氀洀攀渀琀䴀攀猀猀愀最攀猀∀㨀嬀崀Ⰰ∀猀漀甀爀挀攀∀㨀∀∀Ⰰ∀瀀愀礀氀漀愀搀∀㨀渀甀氀氀Ⰰ∀漀甀琀瀀甀琀䌀漀渀琀攀砀琀猀∀㨀嬀崀Ⰰ∀昀漀氀氀漀眀甀瀀䔀瘀攀渀琀䤀渀瀀甀琀∀㨀渀甀氀氀紀\".
I've tried other methods to get this to work including returning the WebhookResponse object as a string or using Newtonsoft JSON to format the object as a JSON. I couldn't find a issue like this on here and couldn't find any samples that demonstrated this use case.
For added context, the API is being hosted on Azure and other controllers function properly when deployed, also below is my .csproj file to show versions of the various Nuget Packages etc.:
<Project Sdk="Microsoft.NET.Sdk.Web">
  <PropertyGroup>
    <TargetFramework>netcoreapp2.1</TargetFramework>
  </PropertyGroup>
  <ItemGroup>
    <Folder Include="wwwroot\"/>
  </ItemGroup>
  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.App"/>
    <PackageReference Include="Microsoft.AspNetCore.Razor.Design" Version="2.1.2" PrivateAssets="All"/>
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="2.1.4"/>
    <PackageReference Include="Newtonsoft.Json" Version="12.0.1"/>
    <PackageReference Include="Google.Cloud.Dialogflow.V2" Version="1.0.0-beta02"/>
  </ItemGroup>
</Project>

If any other information is required or other snippets from my code please don't hesitate to ask and I will reply as soon as I can!

Hi Alex, Even I'm facing the same issue, have you got any work around for this? Thanks


2x - [width, height] -> [width2-1, height2-1]?

Hey, thanks for the great implementation of RAISR!
I wanted to compare it to some other solutions and noticed that the output images are [width2-1, height2-1] instead of 2x (so test.py with a 100x100 image will result in 199x199). I need to read the paper to get a better understanding but is this an expected result?

Please refer the thread #15 . Thanks!

Whoops, apologies! :)


Code doesn't run, Pyruler gives the two red blinks denoting a code exception

I was so freaking stoked to find this, but yeah I can't get the code to run and I can't figure out what the problem is. Unless it involves a step other than add the included midi library then copy over the code and boot_out files over to circuitpy. Bummer.

 I installed CircuitPython v8.2.1 and it works fine, in case anyone else has the same lapse in critical thinking I did.
Thanks, this is going to be a great starting point for some things I want to play with, thanks a ton.


Fix: Updating Blog is not working

Fixes Issue
Closes #330
Changes proposed

-FIX: the server is using the "PUT" method instead of "PATCH" in update blog route.

-These changes fixes thunkAPI being undefined.
Screenshots
Note to reviewers
Changes made in CTF.jsx > line 11 NEEDS TO BE CHECKED  (it was a temporary fix because the frontend was not working)
Bugs Found (related to update blog):

[x]  Reloading update blog page bug (frontend issue): dashboard > blogs > + button > reload the page.
[x]  Tags updates to empty if left unchanged (frontend issue): when tags is left unchanged the front will send tags as an empty array which makes the server update it to empty.
[x]  Pressing cancel on update blog page raises an error

Code of Conduct

[x] By submitting this pull request, I confirm I've read and complied with the CoC 🖖

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.
[ ] Fix Bugs related to update blog

You can also join our Discord community.
Feel free to check out other cool repositories of the Thecyberworld.
Join the Thecyberworld GitHub Organisation by raising an issue (you will be sent an invitation).

fixed: Reloading update blog page bug (frontend issue): dashboard > blogs > + button > reload the page.

It seems like the front was trying to access variables inside searchedBlog when it was undefined (basically undefined.something raised an error).
searchedBlog will take {tags: []} instead of {} because of this at line 99 ( searchedBlog.tags.join(", ")).

It seems like when the tags is updated the blogData.tags takes the updated value, but when it is not updated then blogData.tags will be an empty array and [] || ["SomeOldTag"] would give [].

fix: pressing cancel on update blog page raises an error.
Make sure to check the CTF.jsx file the changes that i have made are in line 11, i only changed it because i got an import error.

Done fixing Bugs related to update blog page, Hope this helps.
Make sure to check the CTF.jsx file the changes that i have made are in line 11, i only changed it because I got an import error.

Nice, those are correct, i just merged those changes and forgot to fix the import in certificate
Thanks


docs: update directory, file name, wording

Signed-off-by: lilin90<EMAIL_ADDRESS>
What have you changed? (mandatory)

Update file directory
Update file name
Update "Note" to "Warning" for deployment files

Does this PR affect documentation (docs/docs-cn) update? (mandatory)
Relevant
Does this PR affect tidb-ansible update? (mandatory)
No.

LGTM


WMIC Nic NetEnabled property is not  available in Windows XP

Describe the bug
When calling networkInterfaces in Windows XP, the getWindowsNics method tries to get the NetEnabled property of the NIC class, but the NetEnabled property is not available in Windows XP. Later in parseLinesWindowsNics the NetEnabled property is used to filter the results of wmic nic. This results in error output to the console.
To Reproduce
Steps to reproduce the behavior:
const si = require( 'systeminformation');
console.log( si.version() );
return si.networkInterfaces()
  .then( data => {
    console.log( data );
  })

Current Output
5.6.12
Node - <ComputerName>
ERROR:
Code = 0x80041017
Description = Invalid query
.
.
.

Environment (please complete the following information):

systeminformation package version:
OS: WindowsXP
Hardware: VMWare VM

@summerswell-ie should be fixed. At least you should not get any error here ... can you test it on your side? Version 5.6.17 and Version 4.34.23 just released.

I've tested 5.6.17 in Windows XP using Node 4.9.1 and it is working as expected now.

@summerswell-ie thank you for your support!


Upgrading to v3 caused "... is not assignable to parameter of type 'QueryFunction..." error

Describe the bug
Using typescript v4.1.3 typescript ran fine, but when I upgraded react-query from v2 to v3.5.12 typescript threw an error if I did not inline my functions in useQuery. #291 exactly describes the issue I was having.
Not Working, But Should:
  async function fetchVehicleData(key: string, apiKey: string): Promise<{ routes: IFullRouteWithStops[]; vehicles: IVehicleData[]; }>{
      //...
      return {
        routes,
        vehicles
      }
    }
  }

  const {
    status: vehiclesStatus,
    data: vehiclesData,
    error: vehiclesError
  } = useQuery(['vehiclesData', clientSettings.apiKey], fetchVehiclesData, {
    refetchInterval: 3600000,
    refetchIntervalInBackground: true
  })

Which threw the following error in typescript on fetchVehiclesData:
Argument of type '(key: string, apiKey: string) => Promise<{ routes: IFullRouteWithStops[]; vehicles: IVehicleData[]; }>' is not assignable to parameter of type 'QueryFunction<{ routes: IFullRouteWithStops[]; vehicles: IVehicleData[]; }>'.

Working:
  async function fetchVehicleData(apiKey: string): Promise<{ routes: IFullRouteWithStops[]; vehicles: IVehicleData[]; }>{
      //...
      return {
        routes,
        vehicles
      }
    }
  }
  const {
    status: vehiclesStatus,
    data: vehiclesData,
    error: vehiclesError
  } = useQuery(['vehiclesData', clientSettings.apiKey], () => fetchVehiclesData(clientSettings.apiKey), {
    refetchInterval: 3600000,
    refetchIntervalInBackground: true
  })

To Reproduce
Unfortunately I could not reproduce this issue in codesandbox (I also tried downloading the sandbox locally and running it), so I'm not sure what about my environment didn't agree, I even tried the same ts config + all the same library versions - I know, that's not super helpful.
Additional context
Chatted directly with @tannerlinsley on Slack quite a bit about this and he recommended I post this issue and tag @boschni
Another thread concerning this issue, if that's at all helpful:
https://www.reddit.com/r/reactjs/comments/kqg27b/reactquery_typescript_question/

Hi @joaoreynolds! You'll need to use either inline functions or the query context: https://react-query.tanstack.com/guides/migrating-to-react-query-3#query-key-partspieces-are-no-longer-automatically-spread-to-the-query-function

Hi @joaoreynolds! You'll need to use either inline functions or the query context: https://react-query.tanstack.com/guides/migrating-to-react-query-3#query-key-partspieces-are-no-longer-automatically-spread-to-the-query-function

Since the linked reddit threads states that inline functions solved your problem, I’m gonna close this.

Since the linked reddit threads states that inline functions solved your problem, I’m gonna close this.


On the handling of exif orientation of labelImg and darknet

Hi,
If the orientation of Exif is 6, the image rotates 90 degrees clockwise, but labelImg seems to rotate counterclockwise 90 degrees.
Have the recent version of labelImg repaired issue # 198 .[https://github.com/tzutalin/labelImg/issues/198]?
Also, since darknet does not consider the orientation of Exif  (since darknet recognizes that the image is rotating 90 degrees clockwise), the coordinates of the bounding box we made when labelImg rotates the image 90 degrees counterclockwise does not become wrong coordinates for darknet Is it?
Thank you,
Kei
OS:
Windows7

Thanks for the info. To answer your initial question: Issue 198 is still unresolved.

I registered materials to explain this phenomenon in Github.
Please point out if there are any mistakes.
Exif-Eng
I think that older versions of labelImg could be learned correctly by darknet in order to be able to learn, even though the appearance is rotating differently from the smart screen.
Is this wrong?
I wrote a Python script that can rotate and convert from a set of images and annotation files and verify that annotations are correctly annotated to automatically generate 24 pairs of images and annotation files. (yolo-img-x28_windows.py)
Another script reads and saves an image with Exif information using OpenCV and PIL, so you can see what kind of image you can obtain. (exif-test.py)
Additional notes:
By using yolo-img-rotate-del_exif.py, you do not have to redo annotations.
img

How am i going to use your code in order to convert my yolo exif orientation infected .txt file that created in labelImg but shown wrong in CVAT and who knows inside Darknet training? As i understand under this method annotation orientation can be tweaked back?


[WIP] Moved --list to a command, as per #1253

This is a commit from @JacobHenner rescued, rebased to master and created as Pull Request :smile:

Unfortunately this is a breaking change, so we probably can't just outright remove the --list option itself.
I think I prefer the organization here, however, so perhaps we could mention that --list is deprecated and otherwise use this organization?

I'm taking a deeper look to the code and I will rethink the idea, I close the PR until I have a better proposal (Now have a lot of repeated code).


fix the release build

Hi there!
In the release build, Program.cs refers to an incorrectly cased mainForm. This pull request just fixes that small typo.

Thanks


Custom TS schema loaders are not working

Which packages are impacted by your issue?
@graphql-codegen/cli
Describe the bug
I am following the guide for creating a custom loader using TS.
It appears that the codegen fails to load the .ts loader file yet works ok with a .js one.
Your Example Website or App
https://codesandbox.io/p/sandbox/graphql-codegen-custom-schema-loader-issue-hkve7f
Steps to Reproduce the Bug or Issue

Run yarn generate
The command fails with an error

Change loader.ts to loader.js in codegen.ts

Run yarn generate
This time the command will work

Expected behavior
As a user, I expected that the custom schema loader written in TS will work if I follow the guide
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Linux]
NodeJS: 16.17.0
graphql version: 16.2.0
@graphql-codegen/* version(s): 2.6.2

Codegen Config File
import { CodegenConfig } from "@graphql-codegen/cli";

const config: CodegenConfig = {
  schema: {
    "schema.graphql": {
      // loader: "./loader.js", // JS - ✅ working
      loader: "./loader.ts", // TS - ❌ not working
    },
  },
  documents: "document.graphql",
  generates: {
    "types.ts": { plugins: ["typescript", "typescript-operations"] },
  },
};

export default config;

Additional context
Please, note that I am registering the ts-node for the codegen command.
graphql-codegen -r ts-node/register --config ./codegen.ts

Yes I'm seeing the same 3.2.1 works fine but upgrading to 3.2.2 gives an error of  Failed to load custom loader


Support query with and() filter condition

const docs = await getDocs(collectionRef, and(where("name", "==", "Bill"), where("age", "==", 42)))

Resolved by #55


mermaid: classDiagram breaks when using <>

When I create a mermaid diagram as follows (which works fine in mermaid's live editor), mermaid shows a "Syntax error in text" error on my hugo site.
```mermaid
classDiagram

class Foo {
  +a: int
}
<<interface>> Foo
```

The <<interface>> line is the culprit. I assume this is due to some HTML sanitization issue, where <interface> might be misinterpreted as a HTML tag.

Thanks for submitting. I fixed it pushed it. Check it out.

Works perfectly, thanks!


Add CI

We should add CI now that https://github.com/DiamondLightSource/python3-pip-skeleton-cli/releases/tag/3.1.0 is ready

As part of this we should do a v0.1 release to pypi


Add policies section and begin with a commit rights policy

This adds a new directory called "policies", to be a place to store current policies for the Astropy Projects.
Policies in this sense mean what we do now, so the idea is that this space is used for "living" policies.  Right now I'm thinking of this as a space to record the existing policies (many of which are currently un-written), but can also be used for development of new or currently not clear policies.
As a "test" of this, I've also written up what I see as the current commit-writes policy (which I am informally the holder of).  Note that this document indirectly depends on #93, since this references that document, but it's probably not critical to think of it as a blocker.

Looks good to me! I wonder if we want to say something in the README about how this interacts with APEs?


Added Log Levels according to API Response

Fixes #6 after acquiring log level from API Response

I am aware that the getLogLevel function becomes redundant, but that is addressed in PR #24  where the log levels are stored elsewhere and accessed.


Update WebSub module.md with basic auth support

Purpose

$Title

Issue https://github.com/ballerina-platform/ballerina-lang/issues/14971 created for the doc rev iew of this.


Use Regexp match to detect gem

It turns out coupling to Bundler::LockfileParser was a bad idea
and has caused odd behavior in practice. This change simplies
the approach and just looks through the lines of the raw
Gemfile.lock and looks for railties instead of using any
bundler components.
cc @wfleming

Minor :lipstick: but LGTM

Actually, looking into this more today, it seems like this bit of code isn't even necessary since the default configuration generated by codeclimate init provides really sensible defaults. I'll follow up with a clean-up pull.


multi-environment configuration

Privileged issue

[X] I'm @tiangolo or he asked me directly to create an issue here.

Issue Content
How can we implement multi-environment configuration in FastAPI, like in Spring Boot? Can we use YAML or property files to achieve multi-environment configuration?

I would like to know some official solutions.

Please create a discussion following the official template.

I'm @tiangolo or he asked me directly to create an issue here.

Also... You are not @tiangolo, neither he asked you to create this... 😅


Add GCC 4.1.2

As requested over email

Unfortunately a simple quick attempt using the existing scripts failed:
~/objdir ~
configure: warning: MAKEINFO=missing: invalid host type
creating cache ./config.cache
configure: error: can only configure for one host and one target at a time

Progress...of a sort :)
make[1]: *** [configure-fixincludes] Error 1
make[1]: *** Waiting for unfinished jobs....
Invalid configuration `CONFIG': machine `CONFIG' not recognized
configure: error: /bin/sh ../../gcc-4.1.2/libiberty/../config.sub CONFIG failed
Makefile:7404: recipe for target 'configure-zlib' failed
make[1]: *** [configure-zlib] Error 1
Makefile:5828: recipe for target 'configure-libiberty' failed
make[1]: *** [configure-libiberty] Error 1

fe26de27a1e4f44895aebeefe9674d1d18848c3c and 7620e6fbc001089192e7e6e695ef2ced878323e2 in mattgodbolt/compiler-explorer-image

And after some more work:
../../gcc-4.1.2/gcc/fortran/gfortran.h:1145:17: fatal error: gmp.h: No such file or directory

(despite the fact I am in theory building gmp and pointing GCC at it...certainly GCC's configure script finds gmp...)

Progress! Had to do a hack in a not-very-reproducible way... but:
> staging/bin/g++ --version
g++ (GCC) 4.1.2
Copyright (C) 2006 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Now to get this building on the CE servers.

Will push a site update shortly.

Ok, the built version has some hardcoded paths from its staging that don't correspond to its final location:
cc1plus: error: /root/staging/include/c++/4.1.2: Permission denied
cc1plus: error: /root/staging/include/c++/4.1.2/x86_64-linux-gnu: Permission denied
cc1plus: error: /root/staging/include/c++/4.1.2/backward: Permission denied
cc1plus: error: /root/staging/include: Permission denied
cc1plus: error: /root/staging/lib/gcc/x86_64-linux-gnu/4.1.2/include: Permission denied
cc1plus: error: /root/staging/x86_64-linux-gnu/include: Permission denied
Compiler returned: 1

(/root/staging is where it was built, and now it's deployed to a new location, these fail. I'll have to look and see how to relocate it, or build it in-place. Newer GCCs are relocatable, and don't have hardcoded paths.

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=35300 seems relevant; may not be the erros shown above: I think GCC is searching $prefix unconditionally. But it's also failing to find its own headers and libraries, so there's two problems here...

Ok; yes. The issue is it tries to unconditionally search the $prefix used during building. On CE nodes that directory is not even searchable (Permission denied); and that seems to stop GCC in its tracks. If it treated it as missing, al would be fine. sudo gcc works :slightly_smiling_face: -- but obviously that isn't an option here.
I'll build the compiler from a place that's world-readable at least so the compiler correctly gets ENOENT

GCC 4.1.2 is now available on the site


upgrade version to 0.1.3

@ZhijunZhao @aovechki

@yungez,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


[e2e] dump flare on test failures for host environments

What does this PR do?

Move helpers from e2e to utils/common to avoid circular dependencies
Add Diagnosable interface for environments
Implement Diagnosable on Host environment
Create one output dir per suite session

Motivation
This should help investigations for failing e2e tests, storing in artifacts a flare of the agent under test, including its configuration and its logs.
Describe how you validated your changes

Possible Drawbacks / Trade-offs
Additional Notes

Best reviewed commit per commit. I cleaned up some code that introduced output dir, moving it to base suite.

Split this PR in 3

add local_public_key_path to env store: https://github.com/DataDog/datadog-agent/pull/32539
create session output dir per test suite https://github.com/DataDog/datadog-agent/pull/32536
implement diagnosable for host environment https://github.com/DataDog/datadog-agent/pull/32537


feature: Start server on arbitrary free port

Feature Request
I want to be able to start a tonic server on an arbitrary free port and get back the port chosen.
Crates

tonic

Motivation

I want to start many servers in tests
I want to bundle the server in a cli that runs on a users machine and doesn't make them pick a port

Proposal
tonic::transport::server::Router::{serve, serve_*} should return Result<SocketAddr, ...> instead of Result<(), ...>. The SocketAddr will be the local_addr of the listener. If the user provides a normal port this will be identical to what they provided. If the user provides an addr with the port 0 to the builder this will contain the port chosen.
Alternatives
Check for a free port, and then provide that port to tonic. It's vulnerable to toctou issues and won't work on macos: https://github.com/rust-lang-nursery/rust-cookbook/issues/500

The recommended way to do that is to create the listener and bind to port 0. Then the OS assigned you a port which you can get back with .local_addr(). You can then wrap the listener in tokio_stream::wrappers::TcpListenerStream and pass it to tonic using serve_with_incoming.
There is an example here.

Thanks


0.10.9 updates

Some updates for the 0.10.9 release.
Mainly the renaming of model variables to remove "Now"

fixed in https://github.com/econ-ark/DemARK/pull/156


Webp 1.30 is available since 2022 - is it possible to see an update?

I have Adobe Photoshop Elements for years and I am very happy about your webp plugin, which perfectly works with it.
There is no alternative webp plugin available and Adobe will offer it for Photoshop from 2023 only. It is a shame.
At the end of last year (2022) a new webp library 1.30 was released and
I hope you continue your great project WebPShop!
Thank you!

Hello JamesPous,
Thanks for the question. Recent libwebp updates did not contain major core library improvements and should not impact WebPShop much, and as you said there is now (partial) native WebP support in Photoshop, so this is low priority for now. I may update the project when I have the time.


Master removing mysql dep

This PR removes the "Add Connection" dialog for Cloud SQL databases.

And fyi @jskeet

I also included the change to add the licenses for released components in this PR. This should make it the only one to track for the next RC.

This PR is not going to be used.


[d3d9] Minor changes to buffer locking

Now that we're uploading using a staging buffer,
we don't need to wait for non default buffers.
We should also respect READONLY for WRITEONLY buffers
(yes, it sounds dumb) because Nostale relies on that.

Changes were merged in #2111


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative/networking-wg-leads
/assign knative/networking-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/hack 7556452...9b76387:
  > 9b76387 output certificate as well (# 229)
  > d1317b0 errexit and pipefail should remain off until downstream is fixed (# 228)
  > df80419 :bug: Fix for test failures of knative/serving# 13292 (# 227)
bumping knative.dev/pkg 501fbd5...d1d5c84:
  > d1d5c84 Define a global for `reconciler.DefaultTimeout`. (# 2597)
  > df29e2a Avoid second lister fetch during finalizer reconciliation. (# 2596)
  > 5f66ecf Allow configuration of threads-per-controller (# 2567)
  > 717747b Register webhook GVKs for validating callbacks (# 2590)

/lgtm


Add ForceDrain description in 2012 Version

make examples formatting uniform in all versions.

Docs Build status updates of commit 99d1b45:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/failoverclusters/suspend-clusternode.md
:white_check_mark:Succeeded

docset/winserver2012-ps/failoverclusters/Suspend-ClusterNode.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/failoverclusters/Suspend-ClusterNode.md
:white_check_mark:Succeeded

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

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@dariomws306 @tseward
Peer review needed for this PR. Thanks.

Docs Build status updates of commit 05d25d8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/failoverclusters/suspend-clusternode.md
:white_check_mark:Succeeded

docset/winserver2012-ps/failoverclusters/Suspend-ClusterNode.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/failoverclusters/Suspend-ClusterNode.md
:white_check_mark:Succeeded

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

Hi @velkovb  - We understand that these PR criteria are new, and we appreciate your patience with the process.
Following are blocking issues that need to be resolved before we can merge:
Conflicting files
docset/windows/failoverclusters/suspend-clusternode.md

Reference
Criteria used for pull request reviews

Thanks

Docs Build status updates of commit 0a2f33c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/failoverclusters/Suspend-ClusterNode.md
:white_check_mark:Succeeded

docset/winserver2012-ps/failoverclusters/Suspend-ClusterNode.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/failoverclusters/Suspend-ClusterNode.md
:white_check_mark:Succeeded

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

Docs Build status updates of commit 0c58142:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/failoverclusters/Suspend-ClusterNode.md
:white_check_mark:Succeeded

docset/winserver2012-ps/failoverclusters/Suspend-ClusterNode.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/failoverclusters/Suspend-ClusterNode.md
:white_check_mark:Succeeded

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

Merge conflicts have been resolved.


GraphQL Apollo Server

Is your plugin request related to a problem? Please describe.
GraphQL batches multiple queries and entities as a data access protocol into a single HTTP request.
In the background, the GraphQL server will run multiple so-called resolvers that return data for a specific chunk of the graph to fulfill these GraphQL queries. Resolvers can run parallel and depend on each other while they usually fetch data from downstream services or from databases using data-loader concept which makes the data orchestration even more complex. Understanding this execution with timing and error is crucial to debug and optimize GraphQL servers.

Apollo Server is probably the most popular library to run GraphQL in production.
Commercial alternative for GraphQL Apollo Tracing
https://blog.apollographql.com/the-new-trace-view-in-apollo-engine-566b25bdfdb0
Is it applicable for Node or Browser or both
Node.js only.
Do you expect this plugin to be commonly used
Yes
Weekly Downloads:  don't know the exact number
Additional context

GraphQL Tracing format could may solve this in a more general way and provide compatibility for all GraphQL engines: https://github.com/apollographql/apollo-tracing

What's the status on this effort? I'm also looking at integrating OpenTelemetry into a GraphQL implementation using apollo-server.

As far as I'm aware, nobody is working on this right now

@jon-whit are you working on that ?

ok it looks like no1 is working on that, I'm taking over it

Hi @obecny just a suggestion, it would be great to support from GraphQL.js level, so it does not matter which GraphQL framework people are using. 😀

@Hongbo-Miao yes that's exactly what will be doing

@obecny how's this work coming along? Anything that I can help with? My team and I are looking to expedite this process a bit. We really would love to have this.

@jon-whit in progress (graphql first), FYI to be able to patch graphql I had to refactor plugin instrumentation too which was done, without this such plugin could not be done.


Remap

<CTRL-A> is widely used in vim (e.g. incrementing numbers). I think this is not a good default.

I'm open to changing this, what would be a better default?

Maybe <leader>ai ?
Or do not set a default at all. You have :AI anyways, so user can specify a binding on their own.

The documentation could provide a suggestion on the mapping, but still let the user decide.


#35 Added the Code and Junit Test in the right place

Sorry there was a silly mistake the last time. I have tried to do it right this time.

Hi,
Ok that's a start. Now look at the pseudo-code from the book and see how well your implementation maps to it. Ideally, follow the style of existing algorithms that have already been implemented that show the pseudo-code in the class comments and then structure the code so that its maps as best as possible to the pseduo-code. The algorithms itself for this issue shouldn't only work for a single example or have a main() associated with it. The related test cases should demonstrate how the general purpose algorithms should be used and its good to have the tests follow examples for the book. Also try to follow case and code layout similar to what is already implemented.

Ok, I am working on it sir. I am trying to make the code similar to the pseudo code in the book. Is the input style okay ? Is using a problem class like i have done here.. okay ?

Closing.


Create atom Babel Preset

By creating an atom babel preset that core and packages can use, we introduce the possibility for slowly dialing back the number of transforms required during transpilation as the version of node bundled in Electron progresses.
It also provides a vector for possible optimizations to be introduced across all atom packages using babel, by writing new transforms focused on performance.
Related to #10860.

You could consider https://github.com/ariporad/babel-preset-features , which detect transformers automatically.

I would also recommend that we make the version of the preset match the version of Atom, so it lets people target a minimum supported Atom version

There's all sorts of es6 and beyond stuff getting released in chrome lately, and I love using async/await, (the babelized version of which is not optimized well) so this gets more timely with each chromium release!

Are there babel presets for specific versions of chromium? That would be generally useful to the JS community, and won't need to be tied to atom. The atom preset, if we wanted to version it with the editor, could just import the numbered chromium preset.

I haven't found someone consistently creating chrome presets for the various chrome versions. Ideally, there would be one for each version such that we could update it as part of an electron upgrade @thomasjo @50Wliu .


Support smart select for JavaScript and TypeScript

Tracks adding smart selection support for JavaScript and TypeScript
The design of this feature and its implementation was tracked upstream: https://github.com/microsoft/TypeScript/issues/29071

First implemented with f635233740f2bb44d9e4663f4152f1980ce0bb2b
Fixed by 12cf3eb0f811724b49a741bcca7e0d4af3f914a3


Start work on async listen

Motivation
We want to translate all our APIs to async.
Changes

Add listen(_: String) -> PostgresNotificationSequence function

Result
Async listen is possible.

What's the state of this @fabianfett?

Hi @pepicrft, thanks for reaching out. I haven't touched this for quite a while and I'd be happy for someone else to drive this forward. Are you interested?
If that's the case I will write down my thoughts on this, to make getting started easier.

Hi @pepicrft, thanks for reaching out. I haven't touched this for quite a while and I'd be happy for someone else to drive this forward. Are you interested?

Very interested indeed. Would you brain-dumping everything in this issue?

Codecov Report

Merging #264 (af7e471) into main (061a083) will decrease coverage by 1.71%.
The diff coverage is 1.04%.

:exclamation: Current head af7e471 differs from pull request most recent head 2713191. Consider uploading reports for the commit 2713191 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #264      +/-   ##
==========================================
- Coverage   44.35%   42.64%   -1.71%
==========================================
  Files         118      121       +3
  Lines        8629     8952     +323
==========================================
- Hits         3827     3818       -9
- Misses       4802     5134     +332

Impacted Files
Coverage Δ

...es/PostgresNIO/Connection/PostgresConnection.swift
12.32% <0.00%> (-0.14%)
:arrow_down:

Sources/PostgresNIO/New/NotificationListener.swift
0.00% <0.00%> (ø)

Sources/PostgresNIO/New/PSQLTask.swift
33.33% <0.00%> (-2.67%)
:arrow_down:

...PostgresNIO/New/PostgresNotificationSequence.swift
0.00% <0.00%> (ø)

.../Connection State Machine/ListenStateMachine.swift
1.34% <1.34%> (ø)

...urces/PostgresNIO/New/PostgresChannelHandler.swift
47.67% <1.98%> (-13.32%)
:arrow_down:

Hi @pepicrft,
I sat down last night and tried to move this forward so that you can have an idea of where I want this to go. Please be well aware that the code is totally undertested and that likely only the happy path works. I'd really appreciate help here. However if you say this is already "too" done, and you are not interested in helping out anymore that is understandable. In that case you have a very bar bones implementation that you might be able to use.
But first let me discuss design goals:

We want to support listening to the same channel multiple times
We want to cancel listening on the connection (by telling the server UNLISTEN) when all listeners have cancelled
Users shall not be required to send LISTEN and UNLISTEN commands themselves to start the listening
If a connection dies all listeners shall be closed by throwing an error

Next let me explain, what types I've added and how all of this works together:

In PostgresConnection we now have an internalListenID atomic. We use this to generate unique ids for NotificationListeners. This is important to support correct cancellation.
The PostgresConnection now communicates with the PostgresChannelHandler using a HandlerTask. The HandlerTask is a superset of the existing PSQLTask and adds startListening and cancelListening
All state handling code that is required for Listening has moved from PostgresConnection into the PostgresChannelHandler
Inside the PostgresChannelHandler we use a ListenStateMachine to handle all state handling of listening to a given channel
The ListenStateMachine has a sub statemachine for each channel called ChannelState
The PostgresChannelHandler translates actions from the ListenStateMachine into tasks that need to be performed on the connection

makeStartListeningQuery
startListenCompleted
makeStopListeningQuery
forwardNotificationToListeners

The NotificationListener is a stateful reference type that represents the different states a single Listen command can be in. It is synchronized by using the connections EventLoop as synchronization primitive.

It supports the new API .streamInitialized, .streamListening
As well as the old existing interface .closure

We return an PostgresNotificationSequence which is a wrapper around an AsyncThrowingStream. We use the wrapper approach here to ensure that we can eventually back PostgresNotificationSequence with another AsyncSequence, without needing to change public API.

Feel free to use the commit in this PR and add further commits (by opening a new PR), that finalize the implementation and add tests. For an example on how to test a state machine, check out the ConnectionStateMachineTests in PostgresNIOTests.
How does all of this sound to you @pepicrft?
Small note: I'll be on vacation starting tomorrow until July 3. For this reason I might be slow to respond.

How does all of this sound to you @pepicrft?

Sounds perfect and thanks a lot for the thorough update on the state of things. I won't be able to dedicate myself full-time to it, but I hope to make some progress before you are back from vacation 😊.
I'll open a separate PR that we can sync on when you are back.

API break checker reports a false positive:
1 breaking change detected in PostgresNIO:
  💔 API breakage: func PostgresConnection.addListener(channel:handler:) is now with @preconcurrency


HTML

##User story
I am student
I want to be able to access study guide materials in an efficient way
that allows to find and study these materials easily.
##Acceptance criteria
It is done when I have created a study guide website
It is done when i visit the website in my browser
It is done when there includes four boxes containing titles HTML, CSS, Git, and Javascript

CSS
##User story
As a bootcamp student
I want to find these materials on a webpage
In a way that is easy to read and access
##Acceptance criteria
It is done when I have created a pre-work study guide website
It is done when I view the study guide
It is done when there contains a dark blue header and footer as well as four boxes with a shadow

Java Script
##User story
As a bootcamp student
I want to find these materials on a webpage
In a way that is easy to read and access
##Acceptance criteria
It is done when I have created a pre-work study guide website
It is done when I have viewed the website
It is done when I am able to see four topics (CSS,Javascript, Git, and HTML) and a suggestion on what topic to study first

README
##User Story
I am a bootcamp student
I want pre-work information to be structured on a webpage
In a way that is easily accessible  and to readable
##Acceptance Criteria
It is done when I have created a pre-work repository
It is done when I am able to find a README that provides information about the project and how to use it

Deployment
##User story
As a bootcamp student
I want pre-work information to be structured on a webpage
So that I am able to access and read these notes
##Acceptance criteria
It is done when I have created a pre-work study guide webpage
It is done when I visit the website URL
It is done when I am able to access the webpage on my browser


Test YOLOv3 weights trained with giou loss

News weights for YOLOv3 with another training method...  https://giou.stanford.edu/
Investigate if those weights : https://github.com/generalized-iou/g-darknet#pre-trained-models improves inference speed / accuracy
Some discussion with the darknet maintainer about this: https://github.com/AlexeyAB/darknet/issues/3249

The maintainer of darknet gently explained me that there was some improvements and how to use them with the darknet fork we are using: https://github.com/AlexeyAB/darknet/issues/3249#issuecomment-498842834

One interested thing is that they seem to have trained in on the VOC dataset, which would be nice as no official weight file has been publish for yolov3 for VOC

Unfortunately for the speed is only 5 FPS on TX2 with those weights for VOC .. So not enough..
And overall they seems to be a small improvement, will add their existence to the documentation but not worth spending more time on this

Example command for voc yolov3 with giou weights
./darknet detector demo cfg/voc.data cfg/yolov3-voc.yolov3-giou-40.cfg yolov3-voc_50000.weights "v4l2src ! video/x-raw, framerate=30/1, width=640, height=360 ! videoconvert ! appsink" -ext_output -dont_show


[dx12] Dota-related fixes and improvements

Fixes #2661
Fixes #2668
Fixes #2140
Fixes the sampler part of #2652
Fixes #1949 (I think? it enables dynamic storage buffers)
PR checklist:

[ ] make succeeds (on *nix)
[ ] make reftests succeeds
[ ] tested examples with the following backends:

I ran gfx-ocean just fine. Just stumbled upon a d3d12 validation error due to https://github.com/gfx-rs/gfx/issues/3401
bors r=msiglreith


RFE: better bug reporting and feature request page in the wiki

Improve upon the basic information on our bug report and feature request wiki page:
Wiki Page

We should make sure this page is linked from the project's README and wiki front page.

I just added some basic information.  However, additional guidance should be provided regarding issue labeling, issue subject lines, and descriptions.  We should also look into issue templates:

https://github.com/blog/2111-issue-and-pull-request-templates

The wiki page has been updated and an issue template added to the userspace repository.  I'm going to close this issue as we have a workable solution in place, any update requests can be handled via a new issue.


Bitbucket API SCM resolution

This change introduces a new SCM resolution mechanism for Bitbucket repos, to
hopefully avoid false-positives when reaching out to Bitbucket to get the repo's
metadata. Bitbucket is a weird code repository solution in that it supports both
git and hg (mercurial) for hosting repos. This makes checking out source
code from Bitbucket less straightforward than other places.
Previously the deducer tried to make educated guesses around which SCM was in
use based on a few things, largely hints from the package's import path. It also
had some fallbacks in place in case it got it wrong, but it did still cause some
issues in a few cases when it got it wrong.
This change introduces a minimal internal Bitbucket API client (bitbucketapi)
with the single purpose of reaching out to Bitbucket to get the SCM type for the
repository. This deprecates the need for the earlier heuristics, and should
result in a correct result every time. This client was written to be consumed as
a singleton (calling functions on the package) to avoid complicating the
bitbucketDeducer further.
The caller of this new API method now makes use of a context.WithTimeout
specifying a 60 second timeout for the calls to Bitbucket. This is a somewhat
arbitrary timeout value, but considering that we're only fetching the top-level
metadata about a Bitbucket repository it seems fairly safe.
The client has no caching of previously seen SCM results for a package, which
could result in more API calls than we'd like. One optimization would be to keep
a map of previously resolved packages, and to serve the result from there.
Tests are also implemented for the bitbucketapi package, hitting 97.0% code
coverage.
Signed-off-by: Tim Heckman<EMAIL_ADDRESS>

@theckman what do you think? should we merge? what do we need to do?

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


Let the OTA server know which version was announced via MQTT

As asked by @rschaten on #35

Thanks! :+1:

The new version string is<EMAIL_ADDRESS>
da828fe0: the device ID
sub2-relay: the firmware name
1.0.0: the firmware version
1.1.0: the OTA version announced via MQTT

This will be released with 1.3.0 when the stabilty issues are resolved.

Is there a particular reason you didn't opt for a single-character separator?
da828fe0=sub2-relay=1.0.0=1.1.0

That would make it a bit easier to split on reception, wouldn't it?

Well yes, it's way easier to split with a single char. I don't know why I did this. :disappointed_relieved:  Fixed 2481bdfa60b0c8f686c927cfefe47f240229590d

Released. :)


Unable to subscribe an Observable declared as lazy (Kotlin).

Have this as a class field:
private val logOutObservable by lazy {  RxView.clicks(root.logOutButton).throttleFirst(500, TimeUnit.MILLISECONDS) }
Trying to subscribe() then in OnCreateView(). Subscription does not work.

Can you provide a failing test case or executable sample? I can't reproduce.


Allow to run on php 8.1 / phpunit 9.5

Implements a minimal version of: https://github.com/daylerees/scientist/pull/32
We have a fork running on https://github.com/Respondens/scientist that works with php8.1 but nobody likes maintaining forks so it would be awesome if this could be merged into this repository.
If you really don't want to spend any work on this anymore please contact us to transfer the repository because we might as well share our work with the rest of the world.

I'd be up for a change of ownership or at least the addition of contributors. I know @daylerees hasn't been doing much PHP work at the moment.  I wanted to try out GitHub actions and added another pull request that runs PHPUnit against 8.0|8.1|8.2 without having to set up a PHP environment.

Hey folks, it's true I've had my hands full recently. Let me add you both as contributors if you're happy with that. Thank you. :)

Merged, added a 2.1.1 release and updated Packagist.


Replaced SECRET_KEY with environment variable

Added logic to check for the SECRET_KEY in environment variables.

Will apply changes to develop branch instead, closing PR.


Add ignore-rhs option for strict-boolean-expressions

PR checklist

[x] Addresses an existing issue: https://github.com/palantir/tslint/issues/4158
[x] New feature, bugfix, or enhancement

[x] Includes tests

[x] Documentation update

Overview of change:
If specified, the right-hand operand of && and || will not be checked
as a strict boolean expression.
CHANGELOG.md entry:
[new-rule-option] strict-boolean-expressions accepts ignore-rhs option to disable checking the right-hand side of the && and || operators as strictly boolean.

Thanks for your interest in palantir/tslint, @dobesv! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

Is there anything else I need to (or can) do for this PR to move forward?

I would love for #4158 to get solved, as it's blocking me from using strict-boolean-expresssions in my project.
So I'm wondering, what's up with this PR?
What would be needed to move this forward?

Has @palantirtech already updated this PR, or is it still waiting for the contributor license agreement to be signed?

It's just waiting for a team member review, I guess.  I haven't heard any feedback about it and I signed the CLA, so assuming I read it right it's just a matter of someone with write access reviewing and approving it.

Fixes https://github.com/palantir/tslint/issues/3279

@JoshuaKGoldberg can you review this one?

Will do, soon!
Note: I'm new to reviewing & maintaining TSLint, so I won't merge it in soon if it passes review. I'll just leave comments so that if a more experienced maintainer to double-check. Same with other PRs.

Any movement on this one?

@rhys-vdw unfortunately no 🙁. The Palantir folks are looking for someone to maintain TSLint full time, but until then there's no SLA on PR reviews.

@adidahiya I reversed the option as requested.  Let me know if there's anything else I can do.

How/when does this change get published?

@dobesv This is useful.  The ideal for us would be to still disallow RHS non-boolean items in places that require a boolean (e.g. ternary, if/else...) while allowing RHS non-boolean in any other locations (particularly React rendering - someBool && ).  Do you think this is a common enough case to address?

@lukewlms Are you trying to say that you want the rhs to be forced as a boolean in case like:
if(x && 1) return;  // <-- error: 1 is not a boolean

but still ignored in other expressions, like:
const whatever = { b: (Date.now() > 5000) && 1 }; // <-- OK because the surrounding expression is not a conditional

My change only ignores the rhs in the sense that it does not complain if the rhs or && or || is not a boolean.  However, the overall expression itself may still not be a boolean, and it's likely an existing rule requiring the condition of if() or the ternary operator would still report a problem independantly of this change.
Have you verified that your case isn't already covered by the rule as it is ?

@dobesv Yes, checked it - here's an example.  I'd like the first two to get flagged but the third to not be flagged - so in places where we absolutely must have a boolean, we're sure to get a strict one.
const func = (b: boolean, s: string) => {
  // Gets flagged with ignore-rhs on
  if (s) {
    // ...
  }

  // Does not get flagged with ignore-rhs on, but I'd like it to!
  if (b || s) {
    // ...
  }

  // This does not get flagged and that's good!
  return b || s;
}

I see.  That does seem like a bug or oversight.  I'd suggest filling a new
issue or a PR for it, since this one is closed.
On Apr 9, 2019 at 4:55 AM, <Luke Williams<EMAIL_ADDRESS>wrote:
@dobesv https://github.com/dobesv Yes, checked it - here's an example.
I'd like the first two to get flagged but the third to not be flagged - so
in places where we absolutely must have a boolean, we're sure to get a
strict one.
const func = (b: boolean, s: string) => {
// Gets flagged with ignore-rhs on
if (s) {
// ...
}
// Does not get flagged with ignore-rhs on, but I'd like it to!
if (b || s) {
// ...
}
// This does not get flagged and that's good!
return b || s;
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/palantir/tslint/pull/4159#issuecomment-481220290, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAUAmbWsGwz1BmeTjOcgpfzwGri8bvg3ks5vfH_KgaJpZM4WfvXz
.

I don't think we reached a consensus on whether boolean-coerced expressions in conditional statements should be treated differently from expressions in other positions in #3279. So I don't know if it's a "bug" per se. Please do file a new issue if you'd like to keep discussing your proposed rule behavior change.

Logged https://github.com/palantir/tslint/issues/4671 for addressing ignoring RHS only when a boolean value is not required.


bug: Pixel 4 XL registering platform as tablet device type

Bug Report
Ionic version:
[X] 4.11.10
Current behavior:
platform.platforms() returns Tablet for Pixel 4 XL device
Expected behavior:
platform.platforms() should not list Tablet
Steps to reproduce:
platform.platforms() returns wrong values
Related code:
import { Platform } from '@ionic/angular';
platform.platforms()

Thanks for the issue. I am going to close this as a duplicate of https://github.com/ionic-team/ionic/issues/19737.


post message on https://forum.sysinternals.com

maybe it can help people re-direct here

Forum moved here:
https://social.technet.microsoft.com/Forums/en-US/1ce18dbf-bf8f-40a4-9585-f25512c1f643/dbgview-and-debugview?forum=miscutils


Too many dns queries when one node is down

Bug Report
We had observed that in k8s a removed TiKV pod could cause very high DNS queries, which can be more than 1500 per second. It was caused by two reasons:

Current operator make 8 request to outer DNS server for every TiKV query, see pingcap/tidb-operator#3121,
TiKV doesn't backoff when creating new connections.

This can be solved by #6228, which will be split into several PRs. The first is #8475.

Other PRs including #8498, #8591 and 8663 are merged, this issue is resolved.


I6515 from master upgrade tonic version

Which issue does this PR close?
Issue #6515
Closes #6515.
The current master does not compile due to the error described in #6515.
Upgraded Cargo.toml to use the latest tonic version: "0.12.3"
This should not cause any user changes, but someone more familiar than me should review. There are no breaking API changes, but now it will have support for GRPC_STATUS so it can build.

I think this was likely caused by https://github.com/apache/arrow-rs/pull/6473#pullrequestreview-2334975052


Add Umple User Manual entry for Final States

Description
This PR addresses #1035 by adding an entry for "Final States" in the Umple User Manual. It also includes examples.

@TimLethbridge I think you should review and merge this.


Failed to resolve package "@shikijs/monaco"

Describe the bug
when i start to export the slides.md in pptx  ,it is happened
Minimal reproduction
Steps to reproduce the behavior:

Go to execute this command: slidev export slides.md --format pptx
See the error:

●■▲
Slidev  v0.49.29 (global)
theme       @slidev/theme-seriph
css engine  unocss
entry       /Users/san/tools/slidev/slides.md
▲ rendering...12:10:04 AM [vite] Internal server error: Failed to resolve package "@shikijs/monaco"
at resolveImportPath (file:///opt/anaconda3/envs/slidev/lib/node_modules/@slidev/cli/dist/chunk-HOVIRHCR.js:36:11)
at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
at async resolveImportUrl (file:///opt/anaconda3/envs/slidev/lib/node_modules/@slidev/cli/dist/chunk-HOVIRHCR.js:17)
✓ exported to ./slides-export.pptx
at async LoadPluginContext.load (file:///opt/anaconda3/envs/slidev/lib/node_modules/@slidev/cli/dist/chunk-YP37OZJY)
Environment

Slidev version: v0.49.29 (global)
Browser: chrome
OS: mac sonoma 14.6.1

Is slidev export pptx compatible with @ shikijs/monaco？

node:v22.7.0
npm : 10.8.2

Theoretically, this should not error. Have you tried deleting node_modules and re-install the package?


Log stdout + stderr to loki or flat files

We swallow stdout to generate stats. We should also log that so runs can be analyzed for debugging and optimizing.
Offer either or both of

Send log lines to loki (if easy / possible)
Write to a file in a directory where we only keep a few days worth of logs

Flat files is merged: https://github.com/cooperlees/ansible_shed/pull/10


contains? not supported on type: clojure.lang.Symbol

Hello,
I'm attempting to use clojurescript with figwheel-main, neovim, and conjure. I'm not well versed in Clojure or ClojureScript so I might be missing something but I keep getting the error in the title in my neovim messages when I attempt to connect conjure to the repl I'm running. I followed along with the cookbook to figure out how to get everything running https://oli.me.uk/clojure-socket-prepl-cookbook/.
My Clojure version is 1.10.1 and my clojurescript version is 1.10.597.
Below is the log file I'm seeing when I open up a cljs file.
Logs
20-04-08 19:49:44 arch-desktop INFO [conjure.main:39] - Logging initialised
20-04-08 19:49:44 arch-desktop INFO [conjure.main:41] - System versions
=== Conjure ===
v2.1.2-2-gd3356a4de4

=== OS ===
Linux arch-desktop 5.4.6-arch1-1 #1 SMP PREEMPT Sat, 21 Dec 2019 16:34:41 +0000 x86_64 GNU/Linux

=== Neovim ===
NVIM v0.4.3
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/bin/cc -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -O2 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -I/build/neovim/src/build/config -I/build/neovim/src/neovim-0.4.3/src -I/usr/include -I/build/neovim/src/build/src/nvim/auto -I/build/neovim/src/build/include
Compiled by builduser

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Run :checkhealth for more info

=== Java ===
openjdk version "1.8.0_232"
OpenJDK Runtime Environment (build 1.8.0_232-b09)
OpenJDK 64-Bit Server VM (build 25.232-b09, mixed mode)

20-04-08 19:49:44 arch-desktop INFO [conjure.rpc:195] - Starting RPC TCP server on port 39803
20-04-08 19:49:44 arch-desktop INFO [conjure.rpc:207] - Starting RPC loops
20-04-08 19:49:44 arch-desktop INFO [conjure.prepl:243] - Started prepl server on port 40361
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :notify, :method :up, :params [""], :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-set-var, :params ["conjure_ready" 1]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 2, :method :nvim-get-current-buf, :params nil}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result nil, :client :stdio}
20-04-08 19:49:44 arch-desktop INFO [conjure.main:46] - Everything's ready! Let's perform some magic.
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 2, :error nil, :result #msgpack.core.Ext{:type 0, :data #object["[B" 0x53da95bb "[B@53da95bb"]}, :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_buf_get_name" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x53da95bb "[B@53da95bb"]}]] ["nvim_buf_line_count" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x53da95bb "[B@53da95bb"]}]] ["nvim_buf_get_lines" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x53da95bb "[B@53da95bb"]} 0 25 false]] ["nvim_get_current_win" []] ["nvim_call_function" ["getcwd" (0)]])]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [["/home/colin/projects/aok-elo-graph-front-end/src/aok_elo_graph_front_end/core.cljs" 42 ["(ns ^:figwheel-hooks aok-elo-graph-front-end.core" "  (:require" "   [goog.dom :as gdom]" "   [reagent.core :as reagent :refer [atom]]))" "" "(println \"This text is printed from src/aok_elo_graph_front_end/core.cljs. Go ahead and edit it and see reloading in action.\")" "" "(defn multiply [a b] (* a b))" "" "" ";; define your app data so that it doesn't get over-written on reload" "(defonce app-state (atom {:text \"cats\"}))" "" "(defn get-app-element []" "  (gdom/getElement \"app\"))" "" "(defn hello-world []" "  [:div" "   [:h1 (:text @app-state)]" "   [:h3 \"Edit this in src/aok_elo_graph_front_end/core.cljs and watch it change!\"]])" "" "(defn mount [el]" "  (reagent/render-component [hello-world] el))" "" "(defn mount-app-element []"] #msgpack.core.Ext{:type 1, :data #object["[B" 0x1253eb17 "[B@1253eb17"]} "/home/colin/projects/aok-elo-graph-front-end"] nil], :client :stdio}
20-04-08 19:49:44 arch-desktop INFO [conjure.prepl:131] - Adding :pfig <IP_ADDRESS> 6776
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-get-var, :params ["conjure_log_blacklist"]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [], :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-execute-lua, :params ["return require('conjure').upsert_log(...)" ("/tmp/conjure.cljc" "small" false false true)]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :notify, :method :clear-virtual, :params [0], :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 2, :method :nvim-get-current-buf, :params nil}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result {"buf" 3, "win" 1001}, :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 2, :error nil, :result #msgpack.core.Ext{:type 0, :data #object["[B" 0x7dc6aec "[B@7dc6aec"]}, :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_buf_get_name" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x7dc6aec "[B@7dc6aec"]}]] ["nvim_buf_line_count" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x7dc6aec "[B@7dc6aec"]}]] ["nvim_buf_get_lines" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x7dc6aec "[B@7dc6aec"]} 0 25 false]] ["nvim_get_current_win" []] ["nvim_call_function" ["getcwd" (0)]])]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 2, :method :nvim-buf-line-count, :params [3]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [["/home/colin/projects/aok-elo-graph-front-end/src/aok_elo_graph_front_end/core.cljs" 42 ["(ns ^:figwheel-hooks aok-elo-graph-front-end.core" "  (:require" "   [goog.dom :as gdom]" "   [reagent.core :as reagent :refer [atom]]))" "" "(println \"This text is printed from src/aok_elo_graph_front_end/core.cljs. Go ahead and edit it and see reloading in action.\")" "" "(defn multiply [a b] (* a b))" "" "" ";; define your app data so that it doesn't get over-written on reload" "(defonce app-state (atom {:text \"cats\"}))" "" "(defn get-app-element []" "  (gdom/getElement \"app\"))" "" "(defn hello-world []" "  [:div" "   [:h1 (:text @app-state)]" "   [:h3 \"Edit this in src/aok_elo_graph_front_end/core.cljs and watch it change!\"]])" "" "(defn mount [el]" "  (reagent/render-component [hello-world] el))" "" "(defn mount-app-element []"] #msgpack.core.Ext{:type 1, :data #object["[B" 0xada2253 "[B@ada2253"]} "/home/colin/projects/aok-elo-graph-front-end"] nil], :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 2, :error nil, :result 1, :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_buf_set_lines" [3 0 1 false ["; conjure/out | Welcome to Conjure! (v2.1.2-2-gd3356a4de4)"]]] ["nvim_buf_set_lines" [3 -1 -1 false ("; conjure/up | Adding :pfig")]] ["nvim_win_set_cursor" [1001 [2 0]]])]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 2, :method :nvim-create-namespace, :params ["conjure_virtual_text"]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [[nil nil nil] nil], :client :stdio}
20-04-08 19:49:44 arch-desktop INFO [conjure.prepl:64] - Connecting through remote-prepl :pfig
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-win-get-cursor, :params [#msgpack.core.Ext{:type 1, :data #object["[B" 0x1253eb17 "[B@1253eb17"]}]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 2, :error nil, :result 2, :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [1 0], :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 2, :method :nvim-buf-clear-namespace, :params [#msgpack.core.Ext{:type 0, :data #object["[B" 0x7dc6aec "[B@7dc6aec"]} 2 0 -1]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 2, :error nil, :result nil, :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_buf_clear_namespace" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x53da95bb "[B@53da95bb"]} 2 0 -1]] ["nvim_buf_set_virtual_text" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x53da95bb "[B@53da95bb"]} 2 0 [["@> Assessing :pfig" "comment"]] {}]])]}
20-04-08 19:49:44 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [[nil 2] nil], :client :stdio}
20-04-08 19:49:44 arch-desktop TRACE [conjure.prepl:151] - Fetching current deps.
20-04-08 19:49:44 arch-desktop TRACE [conjure.prepl:68] - Read from remote-prepl :pfig - {:tag :out, :val "Open URL http://localhost:9500\n", :form nil}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :notify, :method :quick-doc, :params [0], :client :stdio}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-get-current-buf, :params nil}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result #msgpack.core.Ext{:type 0, :data #object["[B" 0x2b7a4451 "[B@2b7a4451"]}, :client :stdio}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_buf_get_name" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x2b7a4451 "[B@2b7a4451"]}]] ["nvim_buf_line_count" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x2b7a4451 "[B@2b7a4451"]}]] ["nvim_buf_get_lines" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x2b7a4451 "[B@2b7a4451"]} 0 25 false]] ["nvim_get_current_win" []] ["nvim_call_function" ["getcwd" (0)]])]}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [["/home/colin/projects/aok-elo-graph-front-end/src/aok_elo_graph_front_end/core.cljs" 42 ["(ns ^:figwheel-hooks aok-elo-graph-front-end.core" "  (:require" "   [goog.dom :as gdom]" "   [reagent.core :as reagent :refer [atom]]))" "" "(println \"This text is printed from src/aok_elo_graph_front_end/core.cljs. Go ahead and edit it and see reloading in action.\")" "" "(defn multiply [a b] (* a b))" "" "" ";; define your app data so that it doesn't get over-written on reload" "(defonce app-state (atom {:text \"cats\"}))" "" "(defn get-app-element []" "  (gdom/getElement \"app\"))" "" "(defn hello-world []" "  [:div" "   [:h1 (:text @app-state)]" "   [:h3 \"Edit this in src/aok_elo_graph_front_end/core.cljs and watch it change!\"]])" "" "(defn mount [el]" "  (reagent/render-component [hello-world] el))" "" "(defn mount-app-element []"] #msgpack.core.Ext{:type 1, :data #object["[B" 0x49b45589 "[B@49b45589"]} "/home/colin/projects/aok-elo-graph-front-end"] nil], :client :stdio}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_get_current_buf" []] ["nvim_get_current_win" []] ["nvim_eval" ["matchstr(getline('.'), '\\%'.col('.').'c.')"]] ["nvim_call_function" ["searchpairpos" ("(" "" ")" "bnzW" "!conjure#cursor_in_code()")]] ["nvim_call_function" ["searchpairpos" ("(" "" ")" "nzW" "!conjure#cursor_in_code()")]])]}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [[#msgpack.core.Ext{:type 0, :data #object["[B" 0x2d5bcd59 "[B@2d5bcd59"]} #msgpack.core.Ext{:type 1, :data #object["[B" 0x76c8aa1b "[B@76c8aa1b"]} "(" [0 0] [4 45]] nil], :client :stdio}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-win-get-cursor, :params [#msgpack.core.Ext{:type 1, :data #object["[B" 0x76c8aa1b "[B@76c8aa1b"]}]}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [1 0], :client :stdio}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_buf_get_lines" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x2d5bcd59 "[B@2d5bcd59"]} 0 4 false]])]}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [[["(ns ^:figwheel-hooks aok-elo-graph-front-end.core" "  (:require" "   [goog.dom :as gdom]" "   [reagent.core :as reagent :refer [atom]]))"]] nil], :client :stdio}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-buf-clear-namespace, :params [#msgpack.core.Ext{:type 0, :data #object["[B" 0x2b7a4451 "[B@2b7a4451"]} 2 0 -1]}
20-04-08 19:49:45 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result nil, :client :stdio}
20-04-08 19:49:45 arch-desktop TRACE [conjure.prepl:88] - Writing to tag: :pfig - (->>
 '(conjure-deps.javaclasspath.v0v3v0.clojure.java.classpath
   conjure-deps.toolsnamespace.v0v3v1.clojure.tools.namespace.dependency
   conjure-deps.toolsnamespace.v0v3v1.clojure.tools.namespace.track
   conjure-deps.toolsreader.v1v3v2.clojure.tools.reader.impl.utils
   conjure-deps.toolsreader.v1v3v2.clojure.tools.reader.reader-types
   conjure-deps.toolsreader.v1v3v2.clojure.tools.reader.impl.inspect
   conjure-deps.toolsreader.v1v3v2.clojure.tools.reader.impl.errors
   conjure-deps.toolsreader.v1v3v2.clojure.tools.reader.default-data-readers
   conjure-deps.toolsreader.v1v3v2.clojure.tools.reader.impl.commons
   conjure-deps.toolsreader.v1v3v2.clojure.tools.reader
   conjure-deps.toolsnamespace.v0v3v1.clojure.tools.namespace.parse
   conjure-deps.toolsnamespace.v0v3v1.clojure.tools.namespace.file
   conjure-deps.toolsnamespace.v0v3v1.clojure.tools.namespace.find
   conjure-deps.toolsnamespace.v0v3v1.clojure.tools.namespace.dir
   conjure-deps.toolsnamespace.v0v3v1.clojure.tools.namespace.reload
   conjure-deps.toolsnamespace.v0v3v1.clojure.tools.namespace.repl
   conjure-deps.compliment.v0v3v9.compliment.sources
   conjure-deps.compliment.v0v3v9.compliment.utils
   conjure-deps.compliment.v0v3v9.compliment.sources.ns-mappings
   conjure-deps.compliment.v0v3v9.compliment.sources.local-bindings
   conjure-deps.compliment.v0v3v9.compliment.sources.class-members
   conjure-deps.compliment.v0v3v9.compliment.sources.namespaces-and-classes
   conjure-deps.compliment.v0v3v9.compliment.sources.keywords
   conjure-deps.compliment.v0v3v9.compliment.context
   conjure-deps.compliment.v0v3v9.compliment.sources.special-forms
   conjure-deps.compliment.v0v3v9.compliment.sources.resources
   conjure-deps.compliment.v0v3v9.compliment.core)
 (filter find-ns)
 (set))
20-04-08 19:49:45 arch-desktop ERROR [conjure.rpc:?] - Error from thread 'RPC message handler': java.lang.IllegalArgumentException: contains? not supported on type: clojure.lang.Symbol
20-04-08 19:49:45 arch-desktop TRACE [conjure.prepl:68] - Read from remote-prepl :pfig - {:tag :ret, :val "#{}", :ns "cljs.user", :ms 10, :form "(->> '(conjure-deps.…"}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :notify, :method :clear-virtual, :params [0], :client :stdio}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-get-current-buf, :params nil}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result #msgpack.core.Ext{:type 0, :data #object["[B" 0xfe9205f "[B@fe9205f"]}, :client :stdio}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_buf_get_name" [#msgpack.core.Ext{:type 0, :data #object["[B" 0xfe9205f "[B@fe9205f"]}]] ["nvim_buf_line_count" [#msgpack.core.Ext{:type 0, :data #object["[B" 0xfe9205f "[B@fe9205f"]}]] ["nvim_buf_get_lines" [#msgpack.core.Ext{:type 0, :data #object["[B" 0xfe9205f "[B@fe9205f"]} 0 25 false]] ["nvim_get_current_win" []] ["nvim_call_function" ["getcwd" (0)]])]}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [["/tmp/conjure.cljc" 2 ["; conjure/out | Welcome to Conjure! (v2.1.2-2-gd3356a4de4)" "; conjure/up | Adding :pfig"] #msgpack.core.Ext{:type 1, :data #object["[B" 0x649a3436 "[B@649a3436"]} "/home/colin/projects/aok-elo-graph-front-end"] nil], :client :stdio}
20-04-08 19:50:02 arch-desktop WARN [conjure.util:76] - Failed to parse code clojure.lang.ExceptionInfo: EOF while reading. {:type :reader-exception, :ex-kind :eof}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-buf-clear-namespace, :params [#msgpack.core.Ext{:type 0, :data #object["[B" 0xfe9205f "[B@fe9205f"]} 2 0 -1]}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result nil, :client :stdio}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :notify, :method :quick-doc, :params [0], :client :stdio}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-get-current-buf, :params nil}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result #msgpack.core.Ext{:type 0, :data #object["[B" 0x17d81cf9 "[B@17d81cf9"]}, :client :stdio}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_buf_get_name" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x17d81cf9 "[B@17d81cf9"]}]] ["nvim_buf_line_count" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x17d81cf9 "[B@17d81cf9"]}]] ["nvim_buf_get_lines" [#msgpack.core.Ext{:type 0, :data #object["[B" 0x17d81cf9 "[B@17d81cf9"]} 0 25 false]] ["nvim_get_current_win" []] ["nvim_call_function" ["getcwd" (0)]])]}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [["/tmp/conjure.cljc" 2 ["; conjure/out | Welcome to Conjure! (v2.1.2-2-gd3356a4de4)" "; conjure/up | Adding :pfig"] #msgpack.core.Ext{:type 1, :data #object["[B" 0x24f3e46b "[B@24f3e46b"]} "/home/colin/projects/aok-elo-graph-front-end"] nil], :client :stdio}
20-04-08 19:50:02 arch-desktop WARN [conjure.util:76] - Failed to parse code clojure.lang.ExceptionInfo: EOF while reading. {:type :reader-exception, :ex-kind :eof}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_get_current_buf" []] ["nvim_get_current_win" []] ["nvim_eval" ["matchstr(getline('.'), '\\%'.col('.').'c.')"]] ["nvim_call_function" ["searchpairpos" ("(" "" ")" "bnzW" "!conjure#cursor_in_code()")]] ["nvim_call_function" ["searchpairpos" ("(" "" ")" "nzW" "!conjure#cursor_in_code()")]])]}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [[#msgpack.core.Ext{:type 0, :data #object["[B" 0x62ded372 "[B@62ded372"]} #msgpack.core.Ext{:type 1, :data #object["[B" 0x57d0709b "[B@57d0709b"]} ";" [0 0] [0 0]] nil], :client :stdio}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-win-get-cursor, :params [#msgpack.core.Ext{:type 1, :data #object["[B" 0x57d0709b "[B@57d0709b"]}]}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [2 0], :client :stdio}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [()]}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result [[] nil], :client :stdio}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-buf-clear-namespace, :params [#msgpack.core.Ext{:type 0, :data #object["[B" 0x17d81cf9 "[B@17d81cf9"]} 2 0 -1]}
20-04-08 19:50:02 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result nil, :client :stdio}
20-04-08 19:50:04 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :notify, :method :stop, :params [0], :client :stdio}
20-04-08 19:50:04 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-get-current-buf, :params nil}
20-04-08 19:50:04 arch-desktop TRACE [conjure.rpc:175] - Received RPC message: {:type :response, :id 1, :error nil, :result #msgpack.core.Ext{:type 0, :data #object["[B" 0xb8814c "[B@b8814c"]}, :client :stdio}
20-04-08 19:50:04 arch-desktop TRACE [conjure.rpc:163] - Sending RPC message: {:type :request, :client :stdio, :id 1, :method :nvim-call-atomic, :params [(["nvim_buf_get_name" [#msgpack.core.Ext{:type 0, :data #object["[B" 0xb8814c "[B@b8814c"]}]] ["nvim_buf_line_count" [#msgpack.core.Ext{:type 0, :data #object["[B" 0xb8814c "[B@b8814c"]}]] ["nvim_buf_get_lines" [#msgpack.core.Ext{:type 0, :data #object["[B" 0xb8814c "[B@b8814c"]} 0 25 false]] ["nvim_get_current_win" []] ["nvim_call_function" ["getcwd" (0)]])]}
20-04-08 19:50:04 arch-desktop INFO [conjure.main:21] - Shutting down
20-04-08 19:50:04 arch-desktop ERROR [conjure.rpc:?] - Error from thread 'RPC stdin handler': java.io.EOFException

I'm sorry about the issue! I'm not sure what's causing it right now and I'm not sure if there's an easy fix :thinking: it's probably incompatibilities with figwheel + prepl, it's not something I've used much at all.
Probably not what you want to hear, but I'm very deep into a rewrite right now that is based on nREPL, which will be MUCH easier to connect to ClojureScript and figwheel. It's the standard way people do their CLJ(S), prepl was an experiment for me. It works well for CLJ but not for CLJS, sadly.
I'll hopefully have early access to that on the develop branch in a few weeks. I think using the new rewrite would be a better fix than any I can provide for this error :cry: it's not particularly Conjure's fault, it'll be something in prepl or figwheel that will be very hard to fix.

You could skip on figwheel for a little bit while I work on the rewrite, just used the built in CLJS browser prepl. It's simpler and should work with Conjure. Propel should start a CLJS browser REPL for you: clj --main propel.main --env browser
Propel also has support for figwheel-main and the legacy lein figwheel, which may work fine for you! https://github.com/Olical/propel#main
If you could share how you're starting your prepl I may be able to spot an issue with it.

@ColinRyan - I got this message after trying the flappy-bird-demo-new from this page https://oli.me.uk/repling-into-projects-with-prepl-and-propel/ . I thought it was some mismatch in version because I had to update the demo project set up to update clojure version, but it turned out it was simpler than that. The .conjure.edn file I had didn't specify that the language is clojurescript. By adding a new .conjure.edn file in the project dir with the following content I got it to work:
{:conns {:local {:port #slurp-edn ".prepl-port"
                        :lang :cljs}}}

This is outlined in the link but I missed the first times around, might be the solution to your error too.

Thanks guys!.  @svenski, that did work I no longer get the error I mentioned. Getting a No sucessful connections error now. I'll see if I can figure this out.

These are things I've checked when I get that error:

If you're running starting with propel it needs a -w in order to write the file which conjure picks up.

Also another gotcha I found is to start neovim in the same directory where the propel file is written. So if your file is src/ttt/ttt.clsj start neovim in project base dir as opposed to src/ttt.

The develop branch is now usable and doesn't have this issue (among many others!) I highly recommend you check it out :smile: kind of fixed bugs by completely rewriting the project. May be closable since I'm deprecating the old JVM implementation.

Closing since it's a prepl specific ClojureScript thing and the new version is nREPL based. Shadow-CLJS support is now built in and working great, you even get completion!

Thanks @Olical I'll check it out


Failed to parse genesis.block

I try launch iroha daemon and got this error.
Before run command I changed genesis.block
New genesis.block is valid Json file, but it doesn't work.
Here is link to my genesis.block: https://gist.github.com/MikhailMe/3da894d91df7d7ba559ae6b26440fb5b

Amount value is represented as uint256:
https://github.com/hyperledger/iroha/blob/86a8c01bc2784371f8425770eed42e8916acd46d/schema/primitive.proto#L97
and uint256 itself is not built-in type, so you need to set it up as well:
https://github.com/hyperledger/iroha/blob/86a8c01bc2784371f8425770eed42e8916acd46d/schema/primitive.proto#L89-L94
Here's the fix for your genesis
Though, I think we should print more detailed error here

Thanks a lot, this is works
I am closing issue


Stop installing npm packages globally/asking for permissions

Switch to using the local version of an npm package instead of
overwriting system-wide state, and definitely do not do anything as root
as part of a repository-local build step

This change is

 You'll also want to update .travis.yml I think. In fact, you may be able to remove the vulcanize line entirely since this happens as a result of travis picking up package.json and calling it there.

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


All ansible steps for ssl and add vagrant back for testing

Closes #376.
TODO:

[x] Fix the chmod command by using the file command? (currently gives a warning)
The file and setfacl command combination seems to set the permissions to 640 but after adding the vagrant user to the group, root, server.pem is not readable. Must be an artifact from the setfacl command

-rw-------  1 root   root   1927 May  2 20:08 cert.pem
-rw-------  1 root   root   1647 May  2 20:08 chain.pem
-rw-------  1 root   root   3574 May  2 20:08 fullchain.pem
-rw-------  1 root   root   1704 May  2 20:08 privkey.pem
-rw-r-----+ 1 root   root   5278 May  2 20:08 server.pem

[x] Figure out how to use arangodb_root_password without a default
[ ] Determine if there is a way for us to test the server without regular tcp (since the ssl cert doesn't match localhost)
[x] Set up vagrant with equivalent settings to our prod server

This seems to look good, I haven't tested it locally though.

[ ]  Determine if there is a way for us to test the server without regular tcp (since the ssl cert doesn't match localhost)

Does this currently prevent testing locally?

This seems to look good, I haven't tested it locally though.

[ ]   Determine if there is a way for us to test the server without regular tcp (since the ssl cert doesn't match localhost)

Does this currently prevent testing locally?

Good question. I have a task on my list to upload a GPG-encrypted tarball containing the cert files so that people with the password could indeed test with SSL, and I think the current proposal for testing without SSL is to modify the playbook to omit the SSL steps.
I'd prefer to make this possible without modifying the file, but as Jack points out, it may not be worth much of our time to make that possible when the major users are us, and the major use case is with SSL.

Does this currently prevent testing locally?

It's possible to test it locally, insofar as you can spin up the server and verify it's running on ssl and using the cert key file. However, is not possible to test locally as in querying from the database, since any local connection will not be using the correct domain name (i.e. it will be localhost, not multinet.app).
Maybe we should wait to merge this until we reprovision the old server and test that there are no bugs the script?

Maybe we should wait to merge this until we reprovision the old server and test that there are no bugs the script?

That sounds like a good idea to me.


Load static files in Server Side Rendering

How can load statics files like a logo of example in create-react-app?
beacuse I have this problem when try to load it!
Warning: Prop src did not match. Server: "[object Object]" Client: "/static/media/logo.5d5d9eef.svg"

I think this issue is duplicate with https://github.com/antonybudianto/cra-universal/issues/12
In short, the starter doesn't support svg import at this moment. I think you can still refer the svg file as static, instead of importing them like import svg from '../icon.svg'


Adding Gmail account throws error

When trying to add gmail account, when logging into account I get "missing client_id".
I guess something is wrong?

same here with all google apps (calendar etc)

I'm running into this issue as well.

Same here, this is the error I am receiving:

Same here. The Windows version is
Windows 11 Pro 21H2 Build 22000.1042 licensed
for Station is

MacOS Monterey does not work as well with google accounts

Issue here is that the developer removed their API keys from the app :/
tor. 13. okt. 2022, 14:00 skrev rechungpa @.***>:

MacOS Monterey does not work as well with google accounts
—
Reply to this email directly, view it on GitHub
https://github.com/getstation/desktop-app/issues/134#issuecomment-1277497991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIPS6FIQDGOS3ZCGUO7B23WC72U7ANCNFSM6AAAAAAQTFCCTI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank you - and this will change for better - an update with the api keys?

Is there anybody still developing the station?

Switching to https://github.com/TheGoddessInari/hamsket
not as nice as Station... but at least, it works


🐛 Bug Report: Software templates - Create merge request for the existing gitlab repository

📜 Description
Hi Team Software templates, how to create merge request for the existing gitlab repository by modifying some files in existing gitlab repository.
Tried to create new repository its working, to modify the existing repository I am not getting any document how to achieve this
👍 Expected behavior
Hi Team Software templates, how to create merge request for the existing gitlab repository by modifying some files in existing gitlab repository.
Tried to create new repository its working, to modify the existing repository I am not getting any document how to achieve this
👎 Actual Behavior with Screenshots
should be able to create new update merge request for existing gitlab repository by modifying few fields
👟 Reproduction steps
Goto backstage left hand side menu
Click on +Create
Get the list of Available Templates
here how to modify the existing project gitlab repository
📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Yes! There's a gitlab module that you can install in the scaffolder which will allow you create merge requests.
https://github.com/backstage/backstage/tree/master/plugins/scaffolder-backend-module-gitlab
Once that's installed you can hit /create/actions on your backstage instance to see the list of the new actions and there should be some examples for merge requests.

Hi @benjdlambert
Please assist me in how to achieve these steps

After creation of cluster backstage is not keeping track of the cluster created, in case of update user has to enter details again
While creation of cluster in case of any errors in the Scalr/gitlab, user is not able to see what happening in the background - how to keep user in sync
In case for the same account already the cluster is created and user is trying to create cluster again how do we restrict
On the Backstage templates there will be maintenance activity, in case of any change in the terraform modules how to keep it in sync with backstage templates


Suggestion: How to host a code sprint event

There's a great guide to this at https://github.com/drud/sprint_guide/blob/master/brilliant-sprints.md ... seems like it would fit in nicely here with other advice for maintainers.
Assuming it's interesting to the maintainers, what's the right way to bring this content in? It's under the Creative Commons Attribution 4.0 International Public License, so technically we could just copypasta with an attribution link... Or perhaps we're considering a list of other helpful resources, and we could just link directly?

Thanks for the suggestions but we're not considering new guides at this time.


ASSWipe - LineBoundsBatch error, not sure of cause

Lua reported a runtime error:
    File "C:/Users\Fyurie\AppData\Roaming\Aegisub\automation\autoload\l0.ASSWipe.moon", line 289
<anonymous function at lines 131-327>
    File "C:/Users\Fyurie\AppData\Roaming\Aegisub\automation\include\a-mo\LineCollection.moon", line 182
runCallback
    File "C:/Users\Fyurie\AppData\Roaming\Aegisub\automation\autoload\l0.ASSWipe.moon", line 225
callback
    File "C:/Users\Fyurie\AppData\Roaming\Aegisub\automation\include\l0\ASSFoundation\LineContents.lua", line 136
callback
    File "C:/Users\Fyurie\AppData\Roaming\Aegisub\automation\autoload\l0.ASSWipe.moon", line 219
callback
    File "C:/Users\Fyurie\AppData\Roaming\Aegisub\automation\autoload\l0.ASSWipe.moon", line 182
removeInvisibleContoursOpt
attempt to call field 'LineBoundsBatch' (a nil value)

Lines processed to cause this error were:
Dialogue: 0,0:04:42.17,0:04:44.18,FyuSigns,,0,0,0,,{\bord1.5\blur0.6\shad1\an7\pos(0,0)\fscx100\fscy100\p1\3c&H2E2E2F&\c&H2E2E2F&\4c&H000000&\4a&HA0&}m 394 168 l 395 197 656 150 656 119
Dialogue: 1,0:04:42.17,0:04:44.18,FyuSigns,,0,0,0,,{\bord0\blur1\an7\pos(0,0)\fscx100\fscy100\p1\c&HB5B5B5&\4c&H000000&\4a&HA0&\shad1\4a&HFF&}m 394 168 l 395 197 656 150 656 119
Dialogue: 1,0:04:42.17,0:04:44.18,FyuSigns,,0,0,0,,{\c&H3C3C3C&\fs18\blur0.6\fnCorporate Rounded\b1\fax-0.18\pos(526.451,157.775)\frz10.37}T{\fax-0.18}o{\fax-0.18}k{\fax-0.18}y{\fax-0.18}o {\fax-0.18}C{\fax-0.18}i{\fax-0.18}t{\fax-0.18}y {\fax-0.18}D{\fax-0.18}i{\fax-0.18}s{\fax-0.18}a{\fax-0.18}s{\fax-0.18}t{\fax-0.18}e{\fax-0.18}r {\fax-0.18}P{\fax-0.18}r{\fax-0.18}e{\fax-0.18}v{\fax-0.18}e{\fax-0.18}n{\fax-0.18}t{\fax-0.18}i{\fax-0.18}o{\fax-0.18}n {\fax-0.18}S{\fax-0.18}t{\fax-0.18}o{\fax-0.18}r{\fax-0.18}a{\fax-0.18}g{\fax-0.18}e

This is caused by a recent ASSFoundation update overwriting the experimental version you were using before as a part of an experimental ASSWipe package i've sent you for testing quite some time ago.
I'm working on releases for all involved components, which need to be deployed at the same time to be compatible. In the meantime you could delete your local copy off ASSWipe and install the official version from DepCtrl. This will lose you some features of the experimental version (performance, extradata cleaning and quirks mode) but at least it works.

Yup, all good on my end for now. Looking forward to the update restoring my old functionality.

Fixed w/ ASSWipe v0.4.0 and ASSFoundation v0.4.0.


Remove unused code

Related to #587
I just removed what was reported to me as unused by code analyser. I wasn't sure if any of it should go the other way and be used rather than removed.
Let me know what you think.

Love it! How did you analyze it exactly?
I think we have dummy variables in more places. The out _ notation seems like it would help in a few places. I'll have a quick scan

Ok the existing dummy's are pass by ref instead of using out.  I tried moving it to outs and i think its better, but its a more significant change. So i'll make a separate PR

Thanks. I'm using vscode which uses omnisharp/roslyn to report this sort of stuff (and more).


[no ticket] Fix linting errors with CRLF vs LF on Windows machines

Ticket: n/a
Feature flag: n/a

Purpose
On Windows machines, Git installer's default is to checkout to CRLF line endings and commit LF. This was causing linting errors within the project. These changes just ensure linting matches to the machine it's running on.
Summary of Changes
Added check for linebreak style in .eslintrc.js and removed forced LF in .editorconfig.

Pull Request Test Coverage Report for Build 3447

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 75.47%

Totals

Change from base Build 3439:
0.0%

Covered Lines:
7787

Relevant Lines:
9062

💛  - Coveralls


[BUG] Geth errors with already known when two simultaneous sales statemachines attempt to approve tokens

The setup is: two nodes competing for two slots of a request. Both nodes attempt to fill both slots, which means that each node is running two simultaneous sales agent statemachines. These two statemachines attempt to call approve to approve deposit of TestTokens for collateral. The issue is that they both use the same nonce, as there hasn't been a transaction submitted with the first approve yet. This causes geth to return an error already known as the raw transactions are exactly the same. Upon return, the statemachine receives the error then moves to the SaleErrored state. Eventually, the sale is not fulfilled as neither node was able to fill both slots (the slots in error were removed from each node's slot queue upon being processed, and eventually errored).
Ethers populates the transaction with await signer.getTransactionCount(BlockTag.pending), however this isn't really sufficient as I mentioned, there hasn't been an initial transaction submitted yet. We can't really swallow the error, as we need to do two approvals. We also can't catch the error and increase the nonce by 1 because the concurrency of the slots is unknown/arbitrary and if there was a different transaction submitted at nonce+1, the second error would be different.

Fixed in https://github.com/codex-storage/nim-ethers/pull/54


Achievement detail page is a bit higgledy piggledy because different achievements show different sets of columns

Can we do something to make this nicer on wider screens? Moving TO achievement to the bottom at least?
Do we want to do more to our section behavior in general with the last 8 years of advances in CSS to avoid leaving big gaps/screen redraw?

Also the display of .detail on achievements detail page is wonky too much whitespace makes the table look disconnected also trophy emoji hugs a bit too tight


[Feature] pushdown or predicate (1): introduce PredicateTree

Why I'm doing:
Before:

All the predicates are stored in a vector,
and the relation among them are AND relations.

In order to support OR predicates,

we need store predicates in a tree,
and the relations among them are AND or OR relations.

This PR only introduce PredicateTree but doesn't use it.
PredicateTree only contains AND relations in this PR. When applying predicates, uses PredicateTree::get_immediate_column_predicate_map() to get vector<ColumnPredicate> and performs the previous logic.
What I'm doing:
Stores multiple ColumnPredicates in a tree. Internal nodes are AND or OR relations, and leaf nodes are ColumnPredicates.
The root node is guaranteed to be an AND relation. A specific example is as follows:
                    AND
               /   |   |      \
              /    |   |       \
            c3>3 c3<30 c4=1     OR
                            /        \
                          AND         AND
                         /    \      /   \
                       c1>1 c1<10   c2>2 c2<20

Provides methods and context for each node of the tree.
The PredicateTree owns all the nodes, and each node cannot be modified unless release_root() is called,
which also means that it has given up ownership of all the nodes.
All the methods are not thread-safe.
Relates to #43801.
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@Mergify backport branch-3.3