os: TestPipeEOF failures with "timed out waiting for read"

#!watchflakes
post <- pkg == "os" && test == "TestPipeEOF" && `timed out waiting for read`

2019-12-12T13:32:03-c39cd41/darwin-amd64-10_14:
--- FAIL: TestPipeEOF (1.08s)
    pipe_test.go:378: line 0
    pipe_test.go:378: line 1
    pipe_test.go:378: line 2
    pipe_test.go:386: timed out waiting for read
FAIL
FAIL	os	2.452s

See previously #35275, which had a very similar failure mode but with an additional file already closed message at the end.
This is the only failure I could find after CL 204800.
CC @ianlancetaylor

2021-02-19T00:05:03-0c63312/darwin-amd64-race
--- FAIL: TestFifoEOF (1.09s)
    fifo_test.go:99: line 0
    fifo_test.go:99: line 1
    fifo_test.go:99: line 2
    fifo_test.go:107: timed out waiting for read
FAIL
FAIL	os	12.691s

2020-03-14T07:03:15-d774d97/freebsd-arm-paulzhol
--- FAIL: TestFifoEOF (1.04s)
    fifo_test.go:100: line 0
    fifo_test.go:100: line 1
    fifo_test.go:108: timed out waiting for read
    fifo_test.go:97: read /tmp/workdir-host-freebsd-arm-paulzhol/tmp/TestFifoEOF145353261/fifo: file already closed
    fifo_test.go:86: error closing reader: close /tmp/workdir-host-freebsd-arm-paulzhol/tmp/TestFifoEOF145353261/fifo: file already closed
    fifo_test.go:66: error writing to fifo: write /tmp/workdir-host-freebsd-arm-paulzhol/tmp/TestFifoEOF145353261/fifo: broken pipe
FAIL
FAIL	os	3.573s

2020-08-17T14:50:17-7ee2622/solaris-amd64-oraclerel
2020-06-18T20:58:26-a07e281/solaris-amd64-oraclerel
2020-02-07T18:06:58-ff091b5/solaris-amd64-oraclerel

A more thorough history, including a failure in 2021:
greplogs --dashboard -md -l -e 'FAIL: TestPipeEOF .*\n(?:    .*\n)*    .*: timed out waiting for read'
2021-03-25T19:24:25-569c86d/darwin-amd64-10_14
2020-08-17T14:50:17-7ee2622/solaris-amd64-oraclerel
2020-06-18T20:58:26-a07e281/solaris-amd64-oraclerel
2020-02-07T18:06:58-ff091b5/solaris-amd64-oraclerel
2019-12-12T13:32:03-c39cd41/darwin-amd64-10_14
2019-11-04T10:13:30-1b2ff10/aix-ppc64
2019-11-02T21:51:09-61fa798/aix-ppc64
2019-11-01T00:04:51-4a09a9b/aix-ppc64
2019-10-31T21:47:51-15bff20/windows-amd64-2016
2019-10-31T16:02:25-d5caea7/aix-ppc64

This test has an arbitrary short timeout:
https://cs.opensource.google/go/go/+/master:src/os/pipe_test.go;l=400;drc=9839668b5619f45e293dd40339bf0ac614ea6bee
It also has several moderately long (10ms) sleeps, whose purpose isn't obvious:
https://cs.opensource.google/go/go/+/master:src/os/pipe_test.go;l=368;drc=9839668b5619f45e293dd40339bf0ac614ea6bee

The sleeps come straight from the test case in #24164.  I expect it could be written better.


rakudoc-l.rakutest fails on doc/Type/RakuAST/Doc.rakudoc

✗  xt/rakudoc-l.rakutest doc/Type/RakuAST/Doc.rakudoc
1..1
# Subtest: doc/Type/RakuAST/Doc.rakudoc
    ok 1 - /type/RakuAST/Doc/Block exists (primary)
    ok 2 - /type/RakuAST/Doc/Markup exists (primary)
    not ok 3 - /type/RakuAST/StatementList exists (primary)
    # Failed test '/type/RakuAST/StatementList exists (primary)'
    # at xt/rakudoc-l.rakutest line 63
    ok 4 - /type/RakuAST/Doc/Paragraph exists (primary)
    ok 5 - /type/RakuAST/Doc/Markup exists (primary)
    ok 6 - /type/RakuAST/Doc/Declarator exists (primary)
    ok 7 - /type/RakuAST/Doc/DeclaratorTarget exists (primary)
    not ok 8 - /type/RakuAST/VarDeclaration/Simple exists (primary)
    # Failed test '/type/RakuAST/VarDeclaration/Simple exists (primary)'
    # at xt/rakudoc-l.rakutest line 63
    1..8
    # You failed 2 tests of 8
not ok 1 - doc/Type/RakuAST/Doc.rakudoc
# Failed test 'doc/Type/RakuAST/Doc.rakudoc'
# at xt/rakudoc-l.rakutest line 89
# You failed 1 test of 1

Indicates that we have links to type documentation that hasn't been implemented yet.

@coke The missing documentation files are

/type/RakuAST/VarDeclaration/Simple
/type/RakuAST/StatementList

These have not been written.  And since the API isn't fixed yet, will not be written at this stage:
Three solutions:

ignore this error message
remove the links causing these messages
put in document stubs

Perhaps 3 is the best solution moving forward?

I'll todo these specific links, no changes to docs needed

Added some diagnostic output, here are current missing links with a count
# Listing by link
# [1] /type/X::Comp::AdHoc
# [1] /type/RakuAST::VarDeclaration::Simple
# [1] /type/RakuAST::StatementList
# [1] /type/X::Immutable
# [1] /type/Distribution::Resources
# [10] /type/X::IO::BinaryMode
# [2] /type/X::IO::Lock
# [1] /type/X::Numeric::DivideByZero
# [3] /type/X::Buf::AsStr
# [1] /type/X::IO::Flush
# [1] /type/CompUnit::DependencySpecification
# [1] /type/X::Numeric::CannotConvert
# [1] /type/X::Str::Match::x
# [3] /type/X::Cannot::Capture

Instead of having links that are whitelist to be OK to be 404s, will remove the links to the not-yet-complete RakuAST docs. (the whitelist should be on the test forcing it to be a link, not the one checking to see if target page exists)

Related to #4476, but these are findable inside the repo itself so we don't need to do a build on the new-raku site

[ ] [3] /type/X::Cannot::Capture
[ ] [1] /type/Distribution::Resources
[ ] [1] /type/X::Comp::AdHoc
[ ] [1] /type/X::Immutable
[ ] [1] /type/X::Str::Match::x
[ ] [1] /type/X::IO::Flush
[ ] [3] /type/X::Buf::AsStr
[ ] [1] /type/CompUnit::DependencySpecification
[ ] [2] /type/X::IO::Lock


fix: Race condition to save datasource and action in import flow

Description
We are seeing the flakiness in the import flow testcases, which is happening due to the race condition while saving the datasources and the actions. As we embed the datasource reference within the action object if because of the race condition action got saved prior to datasource imported application ends up with invalid action. References from CI failed jobs:

Failed Test
CI run

importApplicationFromValidJsonFileTest
https://github.com/appsmithorg/appsmith/actions/runs/3828799911/attempts/1

importApplication_withUnConfiguredDatasources_Success
https://github.com/appsmithorg/appsmith/actions/runs/3828799911/attempts/2

Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=1247d3c


Test command

Implements the command
ccic test [args]

to perform inference on a test data set and save it to disk.
Remarks:

Adds option to ccic.data.training_data.CCICDataset for inference to:

disable random rotations and crops
not return a random sample if the quality of the data does not match the criteria, but rather flag it

Aims to disable any randomness in the loading of the data for reproducibility
Skeleton for code not complete in https://github.com/adriaat/ccic/blob/26b17e14ee0ea2d273827dea34a3a1b474da157b/ccic/bin/test.py#L111-L132

The test test argument can be changed, but I couldn't come up with a better word.
This pull request is simply a draft/checkpoint which can be closed if a better approach is devised.

This work has been integrated through #27.


feat: integrate hiro ordinals api

Try out this version of the Hiro Wallet - download extension builds.

See title
To-do:

add hiro api tests

Might we want / need to adopt any of the sorting improvements made on this older / stale PR? https://github.com/hirosystems/wallet/pull/3322

Changes lgtm @alter-eggo 👍

FYI I recently created a TS library to consume the Ordinals API and provide types: https://github.com/hirosystems/ordinals-api/pull/58
I still need to push it to npm and create a new API release but you'll be able to use it soon

I also tested this build LGTM. Thanks for the comment on my question about blockstream @alter-eggo. That explains what I saw.


Improve param schema retrieval

Coverage decreased (-0.1%) to 79.174% when pulling ffd8d877adffd95fc1792efad07974981642a65a on rvegas:master into<PHONE_NUMBER>c3cc2e10632f4c6a0060d11c95d4cf on kusanagi:master.

Coverage decreased (-0.07%) to 79.256% when pulling 00104f3135b77f561d301841c4a34fba263fb400 on rvegas:master into<PHONE_NUMBER>c3cc2e10632f4c6a0060d11c95d4cf on kusanagi:master.

Coverage decreased (-0.04%) to 79.277% when pulling 5c5d743ddd3c54ecf0b19381ce9e41330081377e on rvegas:master into<PHONE_NUMBER>c3cc2e10632f4c6a0060d11c95d4cf on kusanagi:master.

I've added more tests.

Coverage increased (+0.2%) to 79.57% when pulling 1329610d471223967d5a1975be2a20ce417dd08f on rvegas:master into<PHONE_NUMBER>c3cc2e10632f4c6a0060d11c95d4cf on kusanagi:master.

I'm merging since I've got no reviews


Distorted Audio

I have implemented the library in my project and I am trying to play it on the client side but for some reason, it sounds like very distorted audio on the browser am not sure whether its the audio buffer am sending to the client that's not right or what the problem is.
Here are the steps to reproduce,

Add the dependency and implement the text-to-speech using the default config
Get the processed audio and send it as an unint8array to the client, and parse it using JS
Play the audio.

##What am getting##
An extremely distorted sound instead of the audio playing.
What I should get##
Audio playing with reasonable words.
Here's my code in python
    """Convert transcription to audio"""
    try:
        audio = generate(
            text=text,
            voice="Bella",
            model="eleven_multilingual_v2"
        )
        return audio
    except Exception as e:
        print(e)
        raise e

and here is my code in js
            // Convert String from base64 to ArrayBuffer
            const audioBuffer = base64ToArrayBuffer(data)
            console.log("audioBuffer", audioBuffer)
            // setAudioData(audioBuffer)
            const ac = new AudioContext()
            const assummedSampleRate = 44100
            const abuf = ac.createBuffer(1, audioBuffer.length, assummedSampleRate)
            const audioData = abuf.getChannelData(0)
            for (let i = 0; i < audioBuffer.length; i++) {
                // Convert the uint8 values to floating-point values in the range -1 to 1
                audioData[i] = (audioBuffer[i] - 128) / 128.0;
            }
            const source = ac.createBufferSource()
            source.buffer = abuf

            // Create a gain node to control volume
            const gainNode = ac.createGain();
            gainNode.gain.value = volume;

            source.connect(gainNode);
            gainNode.connect(ac.destination)

            source.start();
        }

I was able to play the audio seems I was playing it using the wrong format


Report services are not using Blob storage SDK classes anymore.

Report services were using Storage SDK classes directly instead of our wrappers around them.  To centralize blob storage access code this change makes them use ICloudBlobClient.
Duplicated code that switches between the primary and alternate source for stats was removed from individual services and moved into DI container. A registration of ICloudBlobClient keyed as BindingKeys.FeatureFlaggedStatisticsKey resolves to a blob client that points to the proper storage account. Services are supposed to request Func<ICloudBlobClient> from the DI container that will provide proper client on each individual call.
A SimpleCloudBlobExtensions class was introduced to close the gap between CloudBlockBlobReference class provided by the storage SDK and CloudBlobWrapper in our code (specifically, our wrapper lacks equivalent of DownloadTextAsync).
Additionally, report services did a .ExistAsync() checks before accessing the blob contents, causing unnecessary network round trip. More helpers were added to SimpleCloudBlobExtensions to handle missing blobs gracefully on content retrieval instead, saving one HTTP request on each report content access.
Lastly, CloudDownloadCountService was made async to accommodate for async-only ISimpleCloudBlob.

looking now ...


Virtual Field creates real column in PostgreSQL

Link to reproduction
No response
Describe the Bug
When creating a virtual field "fullName" like in the example, when using PostgreSQL, a column in the database full_name varcharwill be created. It's not really a critical bug, but it bothered me when I saw it, and since it's still in beta, I hope it's okay to report it.
Also: The syntax for Virtual Fields seems a bit verbose. This probably belongs into improvements / feature requests rather than bug reports, but doing something like:
{
  name: 'fullName',
  type: 'virtual',
  return: ({ data }) => `${data.firstName} ${data.lastName}`,
}

would feel a bit easier, the access and hooks settings could then have sensible defaults. And it would probably be easier to not create a table column for this type of field?
To Reproduce
{
  name: 'fullName',
  type: 'text',
  admin: {
    hidden: true,
  },
  access: {
    create: () => false,
    update: () => false,
  },
  hooks: {
    beforeChange: [
      ({ siblingData }) => {
        delete siblingData.fullName;
     },
    ],
    afterRead: [
      ({ data }) => `${data.firstName} ${data.lastName}`,
    ],
  },
},

Payload Version
2.26.0
Adapters and Plugins
db-postgres

Hi @henk23
a PR is here https://github.com/payloadcms/payload/pull/7621 !

Thanks :) Did not see the PR or the Discussion. I guess this can be closed then.


can I switch enable/disable hoverTooltip?

I want to switch enable or disable hoverTooltip but don't know how can I do this.
Is there any ideas or hints?
Thanks for your helps!

you could remove the
<HoverTooltip ... />

that would not show it. I am not sure if I understood your question though


feat: add output verbosity control/levels

By design arx is verbose (and pretty!). One should be able to opt-out via a CLI option.

Wouldn't hurt to abstract the way everything get's printed out, i.e. some printer service.


Dependency resolution: Cyclic variable definition error

Using gradle with kotlin DSL, I've imported the library with
repositories {
  mavenCentral()
  maven { url = uri("http://neuroph.sourceforge.net/maven2") }
}
dependencies {
  implementation("org.neuroph", "neuroph-core", "2.96")
}

however, this yields to the error

cyclic variable definition: cycle = [parent.version, parent.version]

This is probably related to the Neuroph pom, that is included in neuroph-core as parent pom. There, the version is defined using ${parent.version}.
I am not sure if this is correctly defined and works for maven (and thus is a gradle bug), or if this occurs with maven as well. If anyone can confirm that this works using maven I'll file a gradle bug.
I could workaround for now by downgrading to 2.94

Using gradle with kotlin DSL, I've imported the library with
repositories {
  mavenCentral()
  maven { url = uri("http://neuroph.sourceforge.net/maven2") }
}
dependencies {
  implementation("org.neuroph", "neuroph-core", "2.96")
}

however, this yields to the error

cyclic variable definition: cycle = [parent.version, parent.version]

This is probably related to the Neuroph pom, that is included in neuroph-core as parent pom. There, the version is defined using ${parent.version}.
I am not sure if this is correctly defined and works for maven (and thus is a gradle bug), or if this occurs with maven as well. If anyone can confirm that this works using maven I'll file a gradle bug.
I could workaround for now by downgrading to 2.94

I had other unrelated trouble linking Gradle with the Maven repo. However I managed to find a Gradle import for Neuroph that might be of use to you @qw3ry
implementation 'org.beykery:neuroph:2.92'


CORS error

This issue is for a: (mark with an x)
- [ x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Clone, npm ci, follow setup instructions

Any log messages given by the failure

Access to fetch at 'https://developer.microsoft.com/graph' (redirected from 'https://graph.microsoft.com/') from origin 'http://localhost:3000' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

Expected/desired behavior

Get profile information

OS and Version?

Windows 10

I have tried using package.json proxy and manually setting up the proxy to no avail. What am I doing wrong?
https://create-react-app.dev/docs/proxying-api-requests-in-development/
https://www.npmjs.com/package/cors

Thanks! We'll be in touch soon.

@rexkenley I'm not able to reproduce this. Would you mind sharing your app configuration and a code snippet where you are calling the Graph API? Curious why your request is going to https://developer.microsoft.com/graph

@derisen , is this what you need?
authConfig.js

graph.js

@rexkenley thanks! so it looks to me that you are not calling the right Graph API endpoint. In authConfig.js line 64, graphMeEndpoint should have the value https://graph.microsoft.com/v1.0/me. More on the /me endpoint here

@derisen Thank you very much. That was it.


Can´t show the results from query using Tedious

I made a code using Tedious to connect a MS SQL Server DataBase. When execute the code, the browse just shows the header of my Web page, but not shows the query result, until I refresh the browse.
The error message that console sends me is: "requests can only be made in the LoggedIn state, not the Final state". I have moved my code a lot of times but the results is the same.
My code is:
Consulta.txt
Can someone help me???
Thanks.

Seems to me like you're creating a connection when application starts, but closing it with each request. So the second requests must end up with an error you get.


Establish needed relationships between users, recipes and ingredients

Relationships have been established and tables are being correctly created in the database


Possible to work only on code blocks?

@brendt awesome package 💯
I'm using Highlight inside CommonMark and I have a use-case where I only want the parser to run on code blocks (i.e. using triple-backticks: ```), but not on inline code (e.g. `code here`)

I figured it out.
I just created my own version of HighlightExtension which doesn't add the InlineCodeBlockRenderer
👍🏼
very nice


Fix missing dependency warning

Fix warning about missing peer dependency when use puppeteer ^2.0.0
Summary
I faced with warning from npm when updated puppeeter to 2.0.0 to fix headless: false mode running on latest MacOS

Test plan
Install puppeteer 2.0.0 and check warnings from npm

As I see it duplicates #289


Bump Dash to v3

Version 3 of Dash has been released, so let's update!

:+1:

Thank you for the pull request. This package is deprecated since boxen now works using homebrew cask. I recommend you to install dash homebrew cask. See boxen::personal::osx_apps example in https://github.com/boxen/our-boxen/tree/master/hiera/users#example


WIP: rancher-catalog-service v0.20.2

Needs to be merged at same time PRs for master are merged for:

https://github.com/rancher/cattle/pull/2800/
https://github.com/rancher/cattle/pull/2801/


Ensure support for intrinsics YAML shorthand

In Yaml, a user can write !GetAtt MyBucket.BucketName or !Ref MyBucket to refer to intrinsic functions.
Currently we don't have any tests or examples covering these. Need to make sure we support them everywhere.
Also need to check:

Does it work with Yaml Schema Validation?
Does it work with custom Intrinsic Functions? (Currently only CDK::GetProp)

Looks like there is some support for it already:
https://github.com/cdklabs/decdk/blob/615636fabc28b0635371d5a05b1ef0cb3806424c/src/parser/private/cfn-yaml.ts#L5

Yea it does. I tested with a sample template and it indeed passed test suite. But lacking support for parameter/condition/mapping limits what tests case we can have. So I'm thinking we can defer this task until more features were added.


Feature/Deletion of RepositoryAttributes

Summary by CodeRabbit

New Features

Introduced functionality for deleting repository attributes, including comprehensive validation and permission checks.
Enhanced attribute succession logic, ensuring accurate predecessor and successor relationships and content validation.

Bug Fixes

Refined error handling for attribute comparisons and specific cases related to successor attributes.

Refactor

Updated validation and error handling methods for improved clarity and specificity.
Removed outdated logic affecting attribute control flow.

Tests

Expanded test coverage to include new scenarios for attribute deletion and succession.

Documentation

Updated schema definitions and added detailed descriptions for new and existing properties.

@coderabbitai review

I mistakenly added the removal of the 'succeeds' field of own shared IdentityAttribute successors, if a RepositoryAttribute is deleted. Since the own shared IdentityAttributes aren't automatically deleted in that process, the 'succeeds' field of an own shared IdentityAttribute successor will still link to an existing Attribute. Only if the shared copy is deleted, the shared successor needs to be adjusted. This is taken care of in #70 .


[Feature request] /rtp Player

Please, add /rtp OtherPlayer, so admins may force users to teleport randomly into world.
Usefull for newcomers scheme commands, currently, i need to externalize this feature.
Thank you!

Currently HuskHomes' rtp command is fairly primitive, you may wish to look at HuskBungeeRTP, which supports this.


[StyleCop] Fix all the warnings on several libraries

Libraries:
•	Bot.Builder.AppInsights.Tests
•	Bot.Builder.Integration.AppInsights.Core
•	Bot.Builder.Dialogs.Tests
•	Bot.Builder.AI.LUIS
•	Bot.Builder
•	Bot.ApplicationInsights.core.Tests
Changes made:
•	Remove trailing whitespace
•	Remove multiple blank lines in a row
•	Add documentation for type parameter

Pull Request Test Coverage Report for Build 53108

0 of 0   changed or added relevant lines in 0 files are covered.
11 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 76.036%

Files with Coverage Reduction
New Missed Lines
%

/libraries/integration/Microsoft.Bot.Builder.Integration.ApplicationInsights.Core/ServiceCollectionExtensions.cs
4
86.15%

/libraries/Microsoft.Bot.Builder.AI.LUIS/LuisRecognizer.cs
7
76.71%

Totals

Change from base Build 53069:
0.01%

Covered Lines:
4347

Relevant Lines:
5717

💛  - Coveralls

No issues found in Microsoft.Bot.Builder.dll
No issues found in Microsoft.Bot.Builder.AI.Luis.dll
No issues found in Microsoft.Bot.Builder.AI.QnA.dll
No issues found in Microsoft.Bot.Builder.ApplicationInsights.dll
No issues found in Microsoft.Bot.Builder.Azure.dll
No issues found in Microsoft.Bot.Builder.Dialogs.dll
No issues found in Microsoft.Bot.Builder.TemplateManager.dll
No issues found in Microsoft.Bot.Configuration.dll
No issues found in Microsoft.Bot.Connector.dll
No issues found in Microsoft.Bot.Schema.dll

Hi @GasparAcevedoZainSouthworks, PR is approved but has conflicts, could you please fix those?
Thanks!

No issues found in Microsoft.Bot.Builder.dll
No issues found in Microsoft.Bot.Builder.AI.Luis.dll
No issues found in Microsoft.Bot.Builder.AI.QnA.dll
No issues found in Microsoft.Bot.Builder.ApplicationInsights.dll
No issues found in Microsoft.Bot.Builder.Azure.dll
No issues found in Microsoft.Bot.Builder.Dialogs.dll
No issues found in Microsoft.Bot.Builder.TemplateManager.dll
No issues found in Microsoft.Bot.Configuration.dll
No issues found in Microsoft.Bot.Connector.dll
No issues found in Microsoft.Bot.Schema.dll

Hi @GasparAcevedoZainSouthworks, PR is approved but has conflicts, could you please fix those?
Thanks!

Hi @munozemilio, conflicts solved!!!


Look into creating a "Remember Me" option

When selected a cookie is stored on the computer and saved personal information details are pre-populated on sub-sequent forms. Also, have an option for a "Welcome back {first-name}" message. That also has an option for "Not {first-name}" that will clear the cookie and reload the page.

This was done for PETA


Refactor to Model-Presenter-View architecture

Design to come, inspired by android implementation of The clean architecture

If anything, should update to MVVM, but that's unlikely to happen anytime soon.


v1.6.2 - Fixed opening a blank window when starting Tuneful

Fixed opening a blank window when starting Tuneful

/release


Grammatical Typo at line 45

Signed-off-by: Piyush Thange<EMAIL_ADDRESS>

Thank you @ghost!


Table Blocks

Create Table Block

create block using "zaa-table" -> http://luya.io/de/dokumentation/app-block-types#typen-feld-typen
Settings and Configs:

Should the first row act as Header? (thead)
Table width settings?


vagrant setup docs outdated

https://github.com/pump-io/pump.io/wiki/Vagrant-Installation does not work.
IanKelling on IRC suggested that it may be node / npm versions are too old in the image listed.
from screenlog.0

<EMAIL_ADDRESS>start /srv/pump.io
./bin/pump

module.js:340
throw err;
^
Error: Cannot find module 'bunyan'
at Function.Module._resolveFilename (module.js:338:15)
at Function.Module._load (module.js:280:25)
at Module.require (module.js:364:17)
at require (module.js:380:17)
at Object. (/srv/pump.io/bin/pump:29:14)
at Module._compile (module.js:456:26)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Function.Module.runMain (module.js:497:10)
npm ERR<EMAIL_ADDRESS>start: ./bin/pump
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is most likely a problem with the pump.io package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     ./bin/pump
npm ERR! You can get their info via:
npm ERR!     npm owner ls pump.io
npm ERR! There is likely additional logging output above.
npm ERR! System Linux 3.13.0-116-generic
npm ERR! command "/usr/bin/node" "/usr/bin/npm" "start"
npm ERR! cwd /srv/pump.io
npm ERR! node -v v0.10.37
npm ERR! npm -v 1.4.28
npm ERR! code ELIFECYCLE
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /srv/pump.io/npm-debug.log
npm ERR! not ok code 0

As I said on IRC the instructions in that wiki page aren't officially supported or recommended by us. It is unfortunate that they don't work, though, so I'll add a note at the top of the wiki page.
Official, in-tree Vagrant support is tracked in #1302.

I added a note to the wiki page stating that it's outdated/not supported. Closing as a duplicate of #1302.


How can I manually overwrite a dll?

I use a package with several dlls, one of which I need to manually replace with a custom-built version.
I have tried the following:

paket.references

The.Package
   exclude The.Package.Something.dll

MyApp.csproj

    <Reference Include="The.Package.Something">
      <HintPath>..\..\lib\The.Package.Something.dll/HintPath>
    </Reference>

When I run package install with the above, it not only excludes the package's dll, but it also removes the manual reference from the csproj.
Help!

Can you try to add a false inside your manual node? If that doesn't help, can you please create an minimal repro sample?


rustfmt did not remove the empty line between function call arguments

                                self.impl_or_trait_item(
-                                    self.tcx.impl_trait_ref(impl_did).unwrap().def_id,
-
-                                    item_name
-                                )
-                            }).unwrap();
-                        let note_span = self.tcx.map.span_if_local(item.def_id()).or_else(|| {
-                            self.tcx.map.span_if_local(impl_did)
-                        });
+                                self.impl_or_trait_item(self.tcx
+                                                            .impl_trait_ref(impl_did)
+                                                            .unwrap()
+                                                            .def_id,
+
+                                                        item_name)
+                            })
+                            .unwrap();
+                        let note_span = self.tcx
+                            .map
+                            .span_if_local(item.def_id())
+                            .or_else(|| self.tcx.map.span_if_local(impl_did));

see the item_name param.
https://github.com/rust-lang/rust/pull/36736/files/1d1168ccea6ee1ede476cefb424c384ba7904bb5#diff-45ff4c3ed02cc75f5b8162af5d9c782bL98

I think this is indicative of a wider policy where we don't remove excess newlines in case they are signalling something. However, I think we should only do this between statements, not within an expression.


@prisma/client did not initialize yet

Have you experienced this bug with the latest version of the template?
Yes
Steps to Reproduce
run arc deploy --staging --prune
Expected Behavior
I am expecting this to work
Actual Behavior
After deploying the app to AWS.
The lambda gives this error:
"errorType": "Error",
    "errorMessage": "@prisma/client did not initialize yet. Please run \"prisma generate\" and try to import it again.\nIn case this error is unexpected for you, please report it in https://github.com/prisma/prisma/issues",
    "stack": [
        "Error: @prisma/client did not initialize yet. Please run \"prisma generate\" and try to import it again.",
        "In case this error is unexpected for you, please report it in https://github.com/prisma/prisma/issues",
        "    at new PrismaClient (/var/task/node_modules/.prisma/client/index.js:43:11)",
        "    at Object.<anonymous> (/var/app/database/db.server.ts:21:21)",
        "    at Module._compile (node:internal/modules/cjs/loader:1198:14)",
        "    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1252:10)",
        "    at Module.load (node:internal/modules/cjs/loader:1076:32)",
        "    at Function.Module._load (node:internal/modules/cjs/loader:911:12)",
        "    at Module.require (node:internal/modules/cjs/loader:1100:19)",
        "    at require (node:internal/modules/cjs/helpers:108:18)",
        "    at _tryRequireFile (file:///var/runtime/index.mjs:976:37)",
        "    at _tryRequire (file:///var/runtime/index.mjs:1026:25)"
    ]

server/node_modules/.prisma/client/index.js Looks like this file is not updated.

@zhex900 I'm going to close this ticket as this stack doesn't use Prisma


Compare ArraySeq (fka Spandex) with Vector using macrobenchmarks

This might give us new insights about the general performance of ArraySeq

Idea of macro benchmark: https://en.wikipedia.org/wiki/Quickhull

Do you mean that we should write a Quickhull algorithm and use its performance as a benchmark?

Yes, that’s the idea. What do you think?

I think it is a great idea. I have been working on an updated version of ArraySeq (f.k.a. Spandex) that is fully specialized. It can be thought of as a hybrid between ImmutableArray (which it behaves as for all non primitive types) and my previous work on Spandex (which it behaves as for all primitive types). I hope to have something ready for review in a couple of weeks (I’m currently working on making the views specialized as well). Hopefully the resulting collection will be performant for both primitive and non primitive types (with an edge towards the former).
Naturally, any additional or improved benchmarks (especially complex/composite ones) would be quite beneficial during the evaluation phase.
Greetings
Odd
Odd Möller
<EMAIL_ADDRESS>On Mar 2, 2018, 09:12 +0100, Julien Richard-Foy<EMAIL_ADDRESS>wrote:

Yes, that’s the idea. What do you think?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Assigning to backlog to reconsider for 2.14


look at the bug

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

It shows me "Open remote windows". How can I fix it

/gifPlease

/needsMoreInfo


Fix #774

Fixes the inverted drawing (which was also screwing up stereochemistry)
Makes the handling of super- and sub-scripts more visible (replaces integer constants with an enum)

At some point, I'll see if I can do a proper regression test of the SVG, but this looks good.


Replace "Electron" by app's name in menus and browser window title bar

This will break as name is also used to find the executable on OSX which is hardcoded to Electron in launchApp.jsx


Add component bundles

Right now, if we want to spawn a camera, we need to go through this ritual:
// Spawn the camera entity.
camera.entity = commands.create()
                    .add(Camera{.fovY = 60.0F, .zNear = 0.1F, .zFar = 100.0F})
                    .add(LocalToWorld{})
                    .add(Position{{5.0F, 5.0F, -10.0F}})
                    .add(Rotation{glm::quatLookAt(glm::vec3{0.0F, 0.0F, 1.0F}, glm::vec3{0.0F, 1.0F, 0.0F})});

This is a problem because specifying the rotation of the camera manually is should not be necessary, except in some exceptional cases.
Its also obvious that spawning a Camera component requires also spawning a LocalToWorld component.
Instead, we should implement something along the concept of the Bevy component bundles.
Basically, we could define the following bundle (maybe next to the Camera component):
struct CameraBundle
{
    float fovY;
    float zNear;
    float zFar;
    glm::vec3 position;
    glm::quat rotation;

    // Called by `Commands` on `.addBundle`
    void add(EntityBuilder& builder)
    {
        builder.add(Camera{.fovY = fovY, .zNear = zNear, .zFar = zFar})
                     .add(LocalToWorld{})
                     .add(Position{position})
                     .add(Rotation{rotation});
    }

   static CameraBundle lookAt(float fovY, float zNear, float zFar, glm::vec3 position, glm::vec3 center, glm::vec3 up) {
    // math stuff here
   }
};

And then use it like:
camera.entity = camera.create()
    .addBundle(CameraBundle::lookAt(60.0F, 0.1F, 100.0F, {0.0F, 0.0F, 0.0F}, {0.0F, 0.0F, 1.0F}, {0.0F, 1.0F, 0.0F}));

This allows us to abstract away irrelevant components from the user and also concepts like quaternions.
One issue I can see is that it becomes less clear to know which components an entity will get, but IMO, thats a documentation problem.
It should be very clear which components are added through a bundle.

Closing this as it #724 would make this unnecessary.


macos version is not signed

)

I hope to solve this once I can digitally sign it, you can easily build for MacOS yourself if you are adventurous enough.

There is no financial way for me to support this, I am sorry.


Lite mode coins don't load in v0.6.5-2

After updating to v0.6.5-2:
No lite mode coin can be loaded.
Error message:  Unexpected token < in JSON at position 0
Removing the Verus-Desktop settings folder does not solve it.

Installation Verus-Desktop v 0.5.2 ín the folder of the previous version.
Leave the current wallet dat in VRSC.
Message:
Unexpected token < in JSON at position 0

Issue resolved in release v0.6.5-3


Code style

Please can someone explain why they would write 4 lines of code instead of one?
Example:
    if (keepAlive == null || !metadata.Connection.SupportsKeepAlive)
        {
            return false;
        }

This is 4 lines, an if clause, an open block, a return clause and a close block.
This should be a single readable statement i.e. an if and a single simple result.
    if (keepAlive == null || !metadata.Connection.SupportsKeepAlive) return false;

This makes it easier to read, easier to understand and is more concise.
The only argument I have heard is "well you might want to add additional statements to the result". My answer is that in that case ADD A BLOCK! Blocks should be for more than one statement i.e. a COMPOUND statement. When a diff is performed, it will be clear to the reader that this if statement was converted from the simple statement to a compound statement and we expect to see multiple actions.
This is not Javascript this is C#
"To have more than one statement execute after an if statement that evaluates to true, use braces, like we did with the body of the main function. Anything inside braces is called a compound statement, or a block. When using if statements, the code that depends on the if statement is called the "body" of the if statement."
https://www.cprogramming.com/tutorial/c/lesson2.html

The link is for C, C and C# are not the same, thus they followed different coding conventions.
C# layout conventions


Open a Folder with VSCode

What's the Problem?
We are no longer able to open a directory in VSCode, neither to other application (powershell, etc.)
Solution/Idea
Make it available.
Alternatives
There are no alternative to this.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
Version: <IP_ADDRESS>
OS Version: 10.0.22499.1010_
Windows Version
Windows 11 version dev
Comments
No response

@Wonder-NG can you please fill out the version number and add repro steps?

Thank you @yaichenbaum I did it :-)

@Wonder-NG can you upload the log files? My guess is that the full trust process crashed but I'll need the log files to confirm.


Ability to select namespaces by labels

Is your feature request related to a problem? Please describe.
We'd like to be able to define a ClusterFlow that defines an entire subset of namespaces which are labeled. This would allow teams to have a single ClusterFlow instead of potential hundreds of Flows.

As of 3.0 you should already be able to do this! Let us know if it doesn't work for you for some reason. See the clusterflow example here:
https://banzaicloud.com/docs/one-eye/logging-operator/log-routing/#match-statement

Hi @pepov, looks like you can list namespaces and and label selectors for pods. Not use label selectors for namespaces, or is that incorrect?

yes, I misinterpreted the issue statement, sorry

The problem with this approach that there are no namespace labels metadata in the log lines. So in the current architecture, it would be really hard to maintain information about the namespaces and making routing decisions based on namespace metadata. I understand the requirement and it is something I would be glad to see in the future but It would be extremely expensive to implement. In the meantime, you can use all namespaces and have dedicated labels for such purposes. (I know this is a best-effort workaround but I have no other better solution yet)

Thank you, we're looking at mutating them into the pod labels for now.

Bumping-up.
I'm also looking for this feature

The correct solution is to mutate the namespace labels to the pod labels. This is because it would require extra RBAC priveleges to query that information and I can't think of a proper way to do it correctly. By that I mean how do you figure out what were the labels on a namespace based on a kubernetes log line that was written in the past?
Let me know if someone has a good idea for a correct implementation.

I see that a pre-release of 4.7 has now appeared - is the plan still to get this into 4.7 when it releases properly?

Unfortunately it didn't made it to this release

FYI I've started to implement and test this:

https://github.com/kube-logging/fluent-plugin-label-router/pull/21
https://github.com/kube-logging/logging-operator/pull/1757

the new label router will be available with fluentd image v1.16-4.8 and the whole thing will be part of the 4.8 release scheduled Juli 2nd, but I'm also going to tag a dev image from master soon.


Mesh style artifacts on new scale 2x model

Today when I try new scale model, I found some images showing some 45 degree mesh on the surface where should be smooth. I upload two sets of sample to here. The mesh is on the her face and white sleeve.
Is this what you mentioned

"Now this model has problem in rare case(small pixel art). I will release the fixed versin in a few months."

in another issue? Thanks.
The new model preview server seems down. I didn't test them on your server.

Yes. That is known issue what i mentioned and tried to fix.
In that image, I think that cause is video encoding noise. (scale2x is trained on only noisefree images)
noise1_scale2x does not make artifacts like that.
And, I am planning to add noise0_scale2x model that is lower noise reduction filter than noise1_scale2x.

Thanks for your explanation. Noise-free training output with downscaled input, it dose make sense the network never see such slight video coding noises.
I doom in the 2x image and the mesh shows a pixel pattern of 4x4. Deconv seems to be more aggressive on this area than your old model.
For noise0_scale2x you mentioned, are you considering to train it with slight noisy groundtruth images in order to fix it as well as to low the noise reduction than noise1?

just with less jpeg compressed input images on clean output

this. current noise1 supports jpeg image input that jpeg compression quality is uniform(65, 85). and a user mentioned me 'we need more weak noise reduction filter'. I agree that because of jpeg compression quality, uniform(65, 85) is not very high quality. I will add noise0 that supports jpeg quality around uniform(85,95)? (i am still not considering it. I never add noise**-1** filter).

This happens on waifu2x caffe too (1.1.6) but this time it happens even greater than x2 scaling especially in UpRGB. With RGB noise then RGB scaling there's no mesh artifact. Maybe I should create an issue?

RGB denoise with TTA then UpRGB scale = mesh artifact

Could you link image? or send email to me.

http://www.pixiv.net/member_illust.php?mode=manga&illust_id=55542319
I used the first image. The mesh artifact happens above her right eye but it goes all the way up to her right forehead under the hair shadow. I used x5 scaling.

2x with noise0_scale2x:  no mesh artifact
4x with scale2x:  no mesh artifact (or mesh artifact a very little)
8x with scale2x:  mesh artifact
It seems that mesh artifact is amplified by repeatedly scaling with scale2x only model.
current scale2x model does not support repeatedly usage, actually. (It is not optimized for that)

2x with noise0_scale2x: no mesh artifact
4x with scale2x: no mesh artifact (or mesh artifact a very little)
8x with scale2x: mesh artifact
It seems that mesh artifact is amplified by repeatedly scaling with scale2x only model.
current scale2x model does not support repeatedly usage, actually. (It is not optimized for that)

Does that only apply to UpRGB scaling? Because I tried UpRGB denoise then RGB scale x8 I see no  mesh artifact (at least to me).

I used only UpRGB model.

Maybe here is more better for 8x.

noise0_scale2x (2x)
noise0_scale2x (4x)
noise0_scale2x (8x)

no mesh artifact and very similar result with above 8x scaling.

The mesh disappeared on the 1st method while the 2nd method it is not noticeable unless you really really look closely.
But if possible I really want to use use denoising only once but for now it seems unlikely :sob: .

4x and 8x are not fully supported because this is waifu2x :sob:

I updated 2x model https://github.com/nagadomi/waifu2x/commit/9563d843026ab356d11dfe11683124b9bb7ab2aa. (scale2.0x_model.json for waifu2x-caffe)
It seems that this issue is fixed.

Saw your new updated models. Do these new models without bias showing better performance in PSNR?

No, I thought that the bias in last layer is strong correlation between the mean color of dataset and superresolution task should be more important the input image. so I removed it. but it does not improves performance (PSNR of bias/nobias are very close).
Other way, I decreased -oracle_rate (it is parameter to proactively reduce loss of details). It seems that this change reduced the mesh artifact.
Sorry, I am not able to describe correctly this in my poor english :disappointed: . I am manually controlling the dataset distribution using -active_cropping_rate and -oracle_rate. active_cropping rejects the single color region of images (fanarts has lots of single color background region). -oracle_rate is similar to -active_cropping_rate but it is using training loss of patch in previous epoch. I you are wondering this, please read the code.

Thanks so much for your explanation. I see your point :key: . -oracle_rate seems to play a role of fine-tuning the Network on those hard case samples.
At first I thought you solved this artifacts by removing the bias on deconv layer and asked about the performance. Now I understand you used a better learning method to solve it.
So I guess, after killing this kind of artifact, does this new model with better -oracle_rate achieve a higher PSNR performance?

PSNR of latest scale2.0x_model  is higher than that of before one. I tried same change to train noise*_scale models but it did not improve PSNR and visual quality of the result image.

-oracle_rate, -active_cropping_rate

Those might be domain specific heuristic parameter. fanart/manga images are very sparse(lots of white background area and few lines ). When using uniform patch sampling, average of pixel-wise loss are the zero or very low, and unstable.

This issue is still not fully solved :cry:

Now I've updated upconv_7/art/scale2.0x model. Maybe this issue is fixed ..

Still need to wait for w2x-caffe to be updated though... :+1:

Maybe you can update waifu2x-caffe model with:

open bin/models/upconv_7_anime_style_art_rgb folder
download  scale2.0x_model.json from https://github.com/nagadomi/waifu2x/blob/master/models/upconv_7/art/scale2.0x_model.json and overwrite.
delete scale2.0x_model.json.caffemodel
restart waifu2x-caffe

I updated and there's no more mesh artifact on the pixiv image :relieved: Thanks.
but I noticed something
I tested 8x scaling and using denoising only once with TTA:

2x scaling 3 times
8x scaling in one go
Why is it 2x scaling 3 times output file is smaller than 8x scaling in one go?


[Feature] RPC Node Discovery

We want to have some sort of discovery mechanism for neon-js to be able to independently decide on the best RPC node to hook onto.
Currently, we rely on neon-wallet-db or neoscan to provide a best node URL. However, best merely means best in terms of block height. We also want to be able to function independently to determine the best node ourselves.
Specifics

Given only the seed nodes, discover new nodes and collate them.
Identify open RPC ports on nodes and determine if the network is the same (nodes can host different networks running on different ports)
Do a latency and blockheight test to detemine if the node is good enough.

I have figured someone have made the list of all available RPCs at https://raw.githubusercontent.com/CityOfZion/neo-mon/master/docs/assets/mainnet.json
What do you think of the implementation
getRpcsLatencyWithin(1000)
.then(urls=>urls.sort((a,b)=> a.result + b.result)[0])  // take the one that has the most block height
.then(url => {
  url // { jsonrpc: '2.0', id: 'https://seed1.switcheo.network:10331', result: 2140235 }
})

I would prefer to rely on the network itself. Being a library means we should avoid as many hard coded stuff as possible.

Just a thought, do you think it is a good idea to have that implementation in place first in order to have better developer experience when they are using neon-js SDK.
I think it would be better than hard-code RPC endpoint.
In the future, we could totally swap the underline implementation with no dependency.

that would be nice. Though I think that the proper way is to provide the interface where you can have a Network class which holds the RPC endpoints and a separate method call to fill this Network class with RPC addresses. This way, the user can choose whether he wants to trust the COZ list or provide their own list (they might want to use their private nodes for example).
Are you looking to work on this? Else I will start working on building up the Network class.

Yeah, let me work on Network class.

For how a user could use Network, they can either use the list from COZ or they could give it to the class and we will help out to pick the best from for them.
const network = new rpc.Network({
  net: 'MainNet',
})
network
.getBestRPCWithLatency(2000)
.then(rpc => {
  /*
    {
      "jsonrpc": "2.0",
      "id": "https://pyrpc4.narrative.network:443",
      "result": 2201392
    }
  */
})

const personalNetwork = new rpc.Network({
  nets: [
    { url: 'pyrpc4.narrative.network', port: 443 },
    { url: 'seed1.neo.org', port: 20331 },
  ]
})
personalNetwork
.getBestRPCWithLatency(5000)
.then(rpc => {
  /*
    {
      "jsonrpc": "2.0",
      "id": "https://pyrpc4.narrative.network:443",
      "result": 2201392
    }
  */
})

We will not be including the list in the codebase so we need a api call to retrieve the list.

Yes, it would be a API call to get the list
For MainNet, https://raw.githubusercontent.com/CityOfZion/neo-mon/master/docs/assets/mainnet.json
For TestNet, https://raw.githubusercontent.com/CityOfZion/neo-mon/master/docs/assets/testnet.json'

We probably want a way to add new rpc addresses to an existing network. the MainNet Network object already exists so we might not want to override it with a new object.

What about providing more method to existing class of Network, that way, we could have backward compatibility

getBestRPCWithMaxLatency()

Class Network {
  ...
  /**
   * Get the best RPC base on block height and latency
   * @param {int} maxLatency Maximum Lantency among all the networks
   * @returns {string} Best RPC endpoint
   */
  getBestRPCAddress({ maxLatency = 5000 }){
    ...
  }
  ...
}

// How to use this?
const network = new rpc.Network({
  net: 'MainNet',
})
network
.getBestRPCAddress({maxLatency: 2000})
.then(rpcEndpoint => {
  console.log(rpcEndpoint) // https://pyrpc4.narrative.network:443
})

nets is not the appropriate name for a list of rpc addresses.

What about ...

const config = {
  net: 'CustomNet',
  extra: {
    addresses: [
      'https://pyrpc4.narrative.network:443',
      ...
    ]
  }
}

Do include the API call in this design.
Yes having a method to add rpc addresses would be ideal.
Your docstring declares an integer argument but your example uses an object argument. Please decide on one and justify.
im fine with addresses. My alternative is rpcs

@CHAOWEICHIU any updates?

Do include the API call in this design.

ok

Yes having a method to add rpc addresses would be ideal.

ok

our docstring declares an integer argument but your example uses an object argument. Please decide on one and justify.

I think it is good to just return url

im fine with addresses. My alternative is rpcs

I will be using rpcs, because it is much more related to what it does

What is rpc here?
const network = new rpc.Network({
net: 'MainNet',
})
network
.getBestRPCWithMaxLatency(2000)
.then(rpc => {
console.log(rpcEndppoint) // https://pyrpc4.narrative.network:443
})

This code needs refactoring, but you may find my work on neotools helpful. I've written a bunch of tools to pull all the stats of nodes. This needs to be rewritten to just pull all the data then rank based on that, but the basic functionality is there.
https://github.com/CityOfZion/neo-tools/blob/dbf9c55ca8598f9550df135cf6cd4a5f15d4c769/src/nodejs/neo-rpc/v2.9.0/client/module/getNodesBy.js


Add C++ namespace to Build Init C++ library template

Expected Behavior

The native library templates should offer a flag to configure the C++ namespace of a library (e.g. std or abseil).
Current Behavior

All headers are generated in, what JVM developer would call, default namespace. For libraries, it's usually bad practice given symbol collision can occur.
Context

Steps to Reproduce (for bugs)

Your Environment

Build scan URL:

Not to be confused with header namespace which is a header-language-base namespacing feature, applying to language such as C, Objective-C, Objective-C++. C++ namespace is no available in C but a similar pattern is used by prepending all methods with a specific string (e.g. foo_create more widely used or KnCreate used by Windows API).

The C++ namespace is almost piped through: https://github.com/gradle/gradle/blob/master/subprojects/build-init/src/main/java/org/gradle/buildinit/plugins/internal/CppProjectInitDescriptor.java#L124-L131

See https://github.com/gradle/gradle-native/issues/1022#issuecomment-495153554 for understanding the difference between header namespace vs C++ namespace.


Is it applicable for facial expression recognition tasks

Impressive work! May I ask if this work can be used for obtaining dynamic 3D facial videos (RGBD video) from monocular video input (RGB video), and using the output RGB and 3D information for facial expression recognition tasks?

We are not sure about your question. As you can see, we are more concerned with the rendering results and cannot guarantee the quality of depth and geometry.

Thank you very much for your reply! As we have a depth camera, we are also exploring the integration of depth information into 3D GS for more accurate results, such as precise facial 3D modeling for subsequent facial expression recognition. Do you know of any projects that are based on RGBD for 3D reconstruction? If possible, could you please recommend some to me?
Anyway，thank you for your excellent work and enthusiastic response！


ISPN-14767 CLI entry listing should respect line-breaks in values

https://issues.redhat.com/browse/ISPN-14767

Thanks @tristantarrant


Update documentation

Description
Updating repo documentation and adding maintainers
Issues Resolved
Closes #51
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Can we update CODEOWNERS as well?

Added.


Roll Flutter Engine from 8711c1fd2191 to 2942a132efde (1 revision)

https://github.com/flutter/engine/compare/8711c1fd2191...2942a132efde
2023-10-07<EMAIL_ADDRESS>Roll Dart SDK from 7ba556c43d35 to a88ed8cafd27 (1 revision) (flutter/engine#46647)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Updated C# session description and linked to setup information

@gaelcolas I know you have a million things going now but if you or @SQLDBAWithABeard or someone could merge this PR that'd be great.

Done 👍


v-skeleton-loader flashing animation cpu load

Recently the infocard placeholders during loading have been changed into vue skeleton loader objects. This involves a box with a grey outline of the infobox, with a white flashing effect looping over it. I noticed that it takes longer to load and the page becomes unresponsive, compared to the previous implementation.
I looked at the html/css code and it turns out that this is not a simple animated gif/png file. This explains why my animation blocker extension wasn't working. It's not even an image. Instead, it is a collection of stacked divs with some complex css styling to make the blocks appear fancy and curved like that. The flashing effect was achieved with an infinitely looping animation transform, using a sliding opaque rectangle overlay (animation: loading 1.5s infinite; transform: translateX(-100%);).
 Just their documentation page causes 30% cpu load (Chrome pages are single-threaded). The animation still plays even when scrolled off-screen. When holodex is loading 24 of these, it drives the core to 100% and I assume the animation drops frames to compensate.
If it is not essential to the user experience, I suggest removing the flashing animation. I myself felt like the previous loading placeholders were fine when I was using the site. The queries usually take a second at most to complete, so it's not like I need something to distract me while I'm waiting.

Fixed in #199


Release for API version 2024-11-04?

Which service(blob, file, queue, table) does this issue concern?
All.
Which version of the Azurite was used?
3.31.0
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
DockerHub
What's the Node.js version?
None.
What problem was encountered?
The latest versions of the Azure Blob Storage SDKs have bumped their API version to 2024-11-04.
For instance in Python the package release azure-storage-blob==12.23.0.
These latest releases are currently don't work together with the latest Azure release, erroring with:
azure.core.exceptions.HttpResponseError: The API version 2024-11-04 is not supported by Azurite. Please upgrade Azurite to latest version and retry. If you are using Azurite in Visual Studio, please check you have installed latest Visual Studio patch. Azurite command line parameter "--skipApiVersionCheck" or Visual Studio Code configuration "Skip Api Version Check" can skip this error.

Are there plans to release a new Azurite version that supports the new API version?
Steps to reproduce the issue?
Use latest SDK version with latest Azure version.
Have you found a mitigation/solution?
No, perhaps manual API version control on client side, but that's tedious and in general we would like to use the latest API version against productive services.

Sorry I have missed that there is a release, my bad!


added user module

Module for show login user, user default avatar(.face) or custom image, system work time or system start date

Nice!
Can you add a man page and add this to github wiki?

Nice! Can you add a man page and add this to github wiki?

I don't know English well and I made my own wiki to check before publishing
https://github.com/Dordovel/Waybar/wiki/Module:-User


Update grammar to follow Textualize/textual#3566

Changes merged in https://github.com/Textualize/textual/pull/3566 need to be reflected in the grammar (and then in the extension).

Add rules overlay and constrain with values none, screen, x, y, both, inflect.
Add value initial.

Fixed in 1245f06.


CGBA-50: Add metrics for the balance request responses

Adds a metric for measuring the XML recieved on the BalanceRequestResponseController.
Open thoughts:

For the moment, I have the metrics log to a histogram. I don't know if there is anything that is a better option.
I get the size of the XML from the content length header. Would we rather get it from the size of the body?
Key name is not set in stone.

It also looks like scalafmt and scalafix have altered a couple of other files, mostly imports. I may remove those changes to reduce the diff.

I'm going to request review on the understanding that I got the intention of the ticket right in the first place. This will require adding the metric to the Grafana dashboard with the following call to the appropriate dashboard:
    .withRow(
      maxMeanGraphBuilder("transit-movements-guarantee-balance", "balance-request-response-size")
    )

but I'll add that PR in the new year assuming that I've done the right thing here.

Updated with the changes from the suggestions (thanks!) - it seems to have also made a couple of changes to imports but I'll leave them in, they're harmless.


Add bits::u128 and bits::i128

The num::u128 and i128 strategies are present, but the bits::u128 and i128 strategies are missing. Having this would complete the set of strategies for bits for all primitive integer types.

I would like to work on this. Could you please assign it to me?

@tltsutltsu thank you for volunteering! you have been assigned

@tltsutltsu if you'd still like to work on this, let me know but I'm going to remove the assignment for now


Do not publish jar for android modules

Fixes #869, #852.

Codecov Report

Merging #870 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #870   +/-   ##
=========================================
  Coverage     97.98%   97.98%
  Complexity      811      811
=========================================
  Files            98       98
  Lines          2783     2783
  Branches        308      308
=========================================
  Hits           2727     2727
  Misses           27       27
  Partials         29       29

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ccc9760...645f89a. Read the comment docs.


webclient crash on importing huge libraries

When importing huge libraries (10k+ files, 400+albums, 1000+artists), the webclient may freeze and crash the browser (FF40+). This happens because every library update requests a DOM update in the client, and the rendering is not finished by the time the next song is scanned and added to the library.
I suggest updates from the filesystem to be clustered and pushed only every x (milli) seconds (configurable?), and the client to only update it's tree after it has finished rendering the new tree. I also suggest to use a smart DOM update library that doesn't try to update a part of the DOM that has not changed, like React or DomChanger.

This may be somewhat related to #164. The DOM update scripts, however, currently dirty all the dom elements in the library except for the container element, and also re-builds the expanded elements every update.
This is what I found:
The function renderLibrary() at line 1922 of app.js does indeed preserve most direct child DOM elements of the libraryArtistsDom, but only the first elements are kept. Every DOM update the contents of those child's childs are reset, classes are changed and possibly changed back, and expanded elements are deleted and possibly re-created. (respectively lines 1960, 1958 and 2165 by call in 1981, and lines 1963 ... 1965 and 2018 ... 2119 of app.js at master)


Stats page

Hello,
i'm having problems with the stats page
i put the rtmp_stat lines  in nginx config file and i moved the stat.xsl to the root of the nginx html module
However i get 403 error.  i have 755 for the stat file.
Do i need to input something else or have additional files?
any suggestions?
Cheers

move it to the directory of nginx


Get a Proposal by its uniqueIdDraft

Expected behavior
We sometimes start with a Snapshot Draft type as we do not want to immediately "sponsor" it. This means we need to always use the Draft's ID (the original id) to reference the proposal going forward where we have stored it in immutable systems like Moloch DAO v3.
In the case of Snapshot Proposal type we need to be able to query the GET .../proposal/... API by the uniqueIdDraft field. This means, after we have created a Snapshot Proposal we can still get the correct information using the original ID.
Actual behavior
Today, we need to GET all proposals and filter() them on the client.
Steps to reproduce the behavior
n/a

@jtrein
Get proposal by id endpoint was updated to filter proposals by their draft ids if the filter flag is present in the query parameters.
Get Proposal by Draft Id
Request
GET {{baseUrl}}/api/:space/proposal/:id?isUniqueDraftId=true HTTP/1.1

Response
{
  "0x1433749efa5a3c2ef89aa0962159fb88cbd834681386e230d07f4a443c931682": {
    "address": "0xEd7B3f2902f2E1B17B027bD0c125B674d293bDA0",
    "msg": {
      "version": "0.2.0",
      "timestamp": "1611091325",
      "token": "0x8f56682a50becb1df2fb8136954f2062871bc7fc",
      "type": "proposal",
      "payload": {
        "body": "ASd",
        "choices": ["Yes", "No"],
        "end":<PHONE_NUMBER>,
        "metadata": {
          "uuid": "0xEd7B3f2902f2E1B17B027bD0c125B674d293bDA0",
          "private": 0,
          "type": "governance",
          "subType": "governance"
        },
        "name": "ASd",
        "start":<PHONE_NUMBER>,
        "snapshot": 1
      }
    },
    "sig": "0xf2469a902f630f4d29e439baf9f94dd405901c8e6e0d702c8b47676ea7e35f271c858babcc168a8dd0e1689d1e66f7dfc1a876c4b62c7f1c9042afbd2dffff491c",
    "authorIpfsHash": "0x1433749efa5a3c2ef89aa0962159fb88cbd834681386e230d07f4a443c931682",
    "relayerIpfsHash": "QmU5Zd4zYP812RCKNM8esELNPuqASGfDxRiQDFQRCvGbvY",
    "actionId": "0x4539Bac77398aF6d582842F174464b29cf3887ce"
  }
}

Closing.

@jtrein
Get proposal by id endpoint was updated to filter proposals by their draft ids if the filter flag is present in the query parameters.
Get Proposal by Draft Id
Request
GET {{baseUrl}}/api/:space/proposal/:id?isUniqueDraftId=true HTTP/1.1

Response
{
  "0x1433749efa5a3c2ef89aa0962159fb88cbd834681386e230d07f4a443c931682": {
    "address": "0xEd7B3f2902f2E1B17B027bD0c125B674d293bDA0",
    "msg": {
      "version": "0.2.0",
      "timestamp": "1611091325",
      "token": "0x8f56682a50becb1df2fb8136954f2062871bc7fc",
      "type": "proposal",
      "payload": {
        "body": "ASd",
        "choices": ["Yes", "No"],
        "end":<PHONE_NUMBER>,
        "metadata": {
          "uuid": "0xEd7B3f2902f2E1B17B027bD0c125B674d293bDA0",
          "private": 0,
          "type": "governance",
          "subType": "governance"
        },
        "name": "ASd",
        "start":<PHONE_NUMBER>,
        "snapshot": 1
      }
    },
    "sig": "0xf2469a902f630f4d29e439baf9f94dd405901c8e6e0d702c8b47676ea7e35f271c858babcc168a8dd0e1689d1e66f7dfc1a876c4b62c7f1c9042afbd2dffff491c",
    "authorIpfsHash": "0x1433749efa5a3c2ef89aa0962159fb88cbd834681386e230d07f4a443c931682",
    "relayerIpfsHash": "QmU5Zd4zYP812RCKNM8esELNPuqASGfDxRiQDFQRCvGbvY",
    "actionId": "0x4539Bac77398aF6d582842F174464b29cf3887ce"
  }
}

Closing.


pull request for e2e test at 12/10/2016 05:26:02

:white_check_mark: Validation status: passed
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.


feat: Adding optional fields to Create Event Layout

Is it a ?

[ ] Bug Report

[x] Feature Request

[ ] Chore

Feature Description
Modify the Create Event layout to request all the optional fields from the user as well.
Would you like to work on the issue?
Yes

What about if the global event is selected "free" and the ticket created is "paid" or vice versa.

That's for the API to handle

@iamareebjamal There's an issue with Timezone. The server accepts only this format - Country/City,  others give HTTP 500 error. So the format returned by Timezone.getAvailableIds() is rejected by server. That's why to populate timezone spinner I will have to make a new resource file with all the timezones in proper format or we'll have to stick with text based input.


Bugfix: containers failing to start in cgroups v2 container

Type of change

Bugfix

Description
The script was failing when run in a container due to the unbound variable when we came to use the cpu_period in a calculation.
[roby@roby kroxylicious]$ docker run quay.io/kroxylicious/kroxylicious-developer:0.5.0-SNAPSHOT
Unable to find image 'quay.io/kroxylicious/kroxylicious-developer:0.5.0-SNAPSHOT' locally
0.5.0-SNAPSHOT: Pulling from kroxylicious/kroxylicious-developer
2f5211d9dccf: Already exists
95ac063522bb: Pull complete
bb08e15b96d0: Pull complete
3711a7982fe2: Pull complete
Digest: sha256:d15249bdca5b533b5b515bb9083366699864b9986a123abf52925b9642ae2d85
Status: Downloaded newer image for quay.io/kroxylicious/kroxylicious-developer:0.5.0-SNAPSHOT
/opt/kroxylicious/bin/run-java.sh: line 171: cpu_period: unbound variable

Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Make sure all tests pass
[ ] Review performance test results. Ensure that any degradations to performance numbers are understood and justified.
[ ] Make sure all Sonarcloud warnings are addressed or are justifiably ignored.
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] For user facing changes, update CHANGELOG.md (remember to include changes affecting the API of the test artefacts too).

as you said it's cgroups v2 only so only happens in the presence of a /sys/fs/cgroup/cpu.max file. Maybe some environmental difference across linux/OSX minikube, or something I've done to mine to set cpu limits?


Create CODEOWNERS

This PR adds CODEOWNERS that specifies who will be required to review.
@neubig and @pfliu-nlp are specified as global owners.
@odashi is specified as a code maintainer.

@neubig Almost yes, but they should be added only when they can take full responsibility of the portion.

Right, makes sense.


add Darwin support

Added OS X support, plus some clean up.

Sorry for the delayed response. Been busy building my startup company! I feel confident enough to merge this into develop, and will prepare a new release to NPM from it.

Yay, nice! Good luck with the startup. :smile:
I'll keep an eye out for the release, for the time being I'll use my own GitHub branch as a dependency of the package I'm developing.


Crashes Android 5.0.2 application on Cordova 4.3.0 with Ionic

Only by adding the plugin it makes my app crash on Android.
I'm using Cordova 4.3.0 with the Ionic Framework and when I run
cordova run android --device

the app just show an alert that it has crashed.

Can you share your code with me perhaps?

I only did a fresh install of ionic and then added the plugin and platform, after that I ran the app on the device and crashed. Didn't add code at all, yet.

Luis Rogelio Hernández López
GitHub: @Kurtz1993
Sent using CloudMagic
On Mon, Mar 23, 2015 at 10:28 am, Eddy Verbruggen<EMAIL_ADDRESS>wrote:Can you share your code with me perhaps?
—Reply to this email directly or view it on GitHub.

Any update on this? A user with 5.0.2 just reported us crashes after upgrading to 5.0.2

Any word on this yet? Still an ongoing issue?

Perhaps it's the same issue as #365 which I fixed a few days ago?


Added the volume action class

Fixes #52
Now SUSI can change the volume according to user commands

Sorry Had some errors while rebasing


Math docs are down?

Description
Going to
https://mc-stan.org/math/
Should take us to the math docs but it looks like the link doesn't exist anymore?
Current Version:
v4.0.0

I see it too... is it a dns thing?

Maybe the website didnt build, though I guess then doxygen would fail as well.

Back online. Closing.


Cant seem to make it work with serenity

Hello. Sorry to bug you but I'm trying to adapt this to a serenity that I have in visual studio and can't make it work. WOuld you be able to possibly help ?

I'm not sure what you are referring to. This is targeted for Mendix applications.


shared_ptr object creation is not working between swift/c++ interoperability.

Description
Creating a shared_ptr object in the swift file is not working. It is stated in the documentation that shared_ptr functionality is supported but it is not working.
C++ header file: Sample.h
#ifndef Sample_h
#define Sample_h
#include
#include
inline std::shared_ptr makeSharedData(int data) {
return std::make_shared (data);
}
class sample {
public:
sample(std::shared_ptr data);
int read_data();
void write_data(int data);
void ref_func(int& data);
private:
std::shared_ptr m_data;
};
#endif /* Sample_h */
Respective Sample.cpp file :
#include "Sample.h"
sample::sample(std::shared_ptr data) {
m_data = data;
}
int sample::read_data() {
return *m_data;
}
void sample::write_data(int data) {
*m_data = data;
}
void sample::ref_func(int& data) {
data = 100;
}
Swift File Contents : TestFile.swift
import Foundation
func connectCPP() -> Int32 {
//1. making use of the inline function in header file and calling that to get the shared_ptr object which in turns used to make the
// class object. This is throwing below error :
/*   std::__1::__shared_ptr_emplace<int, std::__1::allocator>::~__shared_ptr_emplace(), referenced from:
vtable for std::__1::__shared_ptr_emplace<int, std::__1::allocator> in TestFile.o
std::__1::__shared_ptr_emplace<int, std::__1::allocator>::~__shared_ptr_emplace(), referenced from:
vtable for std::__1::__shared_ptr_emplace<int, std::__1::allocator> in TestFile.o
clang: error: linker command failed with exit code 1 (use -v to see invocation)
*/
var obj = sample(makeSharedData(30))
//2. If I am trying to use make_shared directly in swift file, It is throwing error of symbol not found for make_shared and
//    shared_ptr.
var data = std::make_shared<int>(10)
var obj1 = sample(data)
obj.write_data(12)
return obj.read_data()

}
If shared_ptr support is present in current version of interoperability, then it should be working fine without any error.

inline makes problem here. If you separate your function definition and declaration without using inline, your code compiles.
The expression std::make_share<int> is not a thing on Swift-side.

// sample.h
#include <memory>

std::shared_ptr<int> makeSharedData(int data);

class sample {
public:
    sample(std::shared_ptr<int> data);
    int read_data();
    void write_data(int data);
    void ref_func(int& data);
private:
    std::shared_ptr<int> m_data;
};

// sample.cpp
std::shared_ptr<int> makeSharedData(int data) {
    return std::make_shared<int>(data);
}

sample::sample(std::shared_ptr<int> data) {
    m_data = data;
}

int sample::read_data() {
    return *m_data;
}

void sample::write_data(int data) {
    *m_data = data;
}

void sample::ref_func(int& data) {
    data = 100;
}

// Test.swift
func connectCPP() -> Int32 {
    var obj = sample(makeSharedData(30))
    obj.write_data(12)
    return obj.read_data()
}

Thanks.. It is working fine for me now.
Just a small question in addition to this:
Can we have this similar approach for unique_ptr as well?

I'm chiming in into this issue, because there is something that I might not completely understand about the new swift<->c++ interop: although sharing std::shared_ptr (like in this example) or std::vector works, I find it still quite underwhelming and not really "interop" material, since all you can do is pass around shared_ptrs or vectors by value, that are necessarily owned or created by their C++ counterpart. Despite what the swift doc is saying (and that one is really good for once, so much that it might be overselling the feature at that point...).
So quick questions:

For a shared_ptr, is there a way to access the underlying object from swift ? I'm making sure the pointee is a swift-compatible object, it's API is available from swift, but I can't access the object from the shared_ptr itself, quite useless...
For a vector: maybe it used to work, but as of right now (swift 5.9, Xcode 15 beta 4), conversion between a simple swift::Array and a std::vector<uint8_t> doesn't work, from both languages, the vector isn't usable as a RandomAccessCollection from Swift (or anything else for that matter, nothing to use it efficiently), so is the Array on the C++ side (only a subset of the read-only API is actually exposed to C++)
I've been reproducing the examples from the swift main doc, or from https://github.com/apple/swift-cmake-examples, and most of the stuff that's presented in either the WWDC video or the docs isn't actually working, and the CMake examples are not even compiling...not to mention trying to modify them to experiment with vector/shared_ptr or multi-arch configurations etc.
I was thrilled to give a try to that brand new promising feature, but after a few days trying things and selling the promise to a few colleagues, I might say it's a bit underwhelming so far.
I hope I misunderstood something and I'm doing it all wrong !

Sorry to hear that @moretromain , let me try to respond to specific points:

I'm chiming in into this issue, because there is something that I might not completely understand about the new swift<->c++ interop: although sharing std::shared_ptr (like in this example) or std::vector works, I find it still quite underwhelming and not really "interop" material, since all you can do is pass around shared_ptrs or vectors by value, that are necessarily owned or created by their C++ counterpart. Despite what the swift doc is saying (and that one is really good for once, so much that it might be overselling the feature at that point...). So quick questions:

For a shared_ptr, is there a way to access the underlying object from swift ? I'm making sure the pointee is a swift-compatible object, its API is available from swift, but I can't access the object from the shared_ptr itself, only the pointer's API (empty, get refcount etc.), quite useless...

You should be able to access fields and call the specific methods on the pointee, for instance:
ptr.pointee.callMethod()

The ergonomics for shared_ptr are not great at the moment, and we're going to be working on improving them. We will need some Swift language additions as well, specifically easier ways to borrow values: https://forums.swift.org/t/desired-swift-language-features-that-can-advance-the-state-of-c-interoperability-support/66144/9. We are planning to work on it though!

For a vector: maybe it used to work, but as of right now (swift 5.9, Xcode 15 beta 4), conversion between a simple swift::Array and a std::vector<uint8_t> doesn't work, from both languages, the vector isn't usable as a RandomAccessCollection from Swift (or anything else for that matter, nothing to use it efficiently), so is the Array on the C++ side (only a subset of the read-only API is actually exposed to C++)

It sounds like you're hitting https://github.com/apple/swift/issues/67410 . We're investigating it right now. You should be able to work around it by manually conforming a specific vector type you need to CxxRandomAccessCollection.
Once it's conformed you should be able to convert it to array by just going Array(vector), but that conformance is required first unfortunately.

I've been reproducing the examples from the swift main doc, or from https://github.com/apple/swift-cmake-examples, and most of the stuff that's presented in either the WWDC video or the docs isn't actually working, and the CMake examples are not even compiling...not to mention trying to modify them to experiment with vector/shared_ptr or multi-arch configurations etc.

Could you file an issue that describes the CMake examples build errors you're seeing to https://github.com/apple/swift-cmake-examples and CC @etcwilde on it.

I was thrilled to give a try to that brand new promising feature, but after a few days trying things and selling the promise to a few colleagues, I might say it's a bit underwhelming so far.
I hope I misunderstood something and I'm doing it all wrong ! (I'm surprised there's no way to locally use a std::vector from swift to easily transform/use it into something else using some '.withUnsafeXXX' mechanism, I'm not a swift expert, still learning, but that would be the way to interact between types will keeping the unsafe zone quite localised that I would expect..)

@moretromain, yes, please do let me know what you're seeing, what Swift version you have, and what CMake version. Thanks!

You should be able to access fields and call the specific methods on the pointee, for instance:
ptr.pointee.callMethod()

Actually it is not as simple as you say. It is necessary for the class of the underlying managed object to have the SWIFT_IMMORTAL_REFERENCE attribute; otherwise, it will be freed after being accessed via .pointee, which in practice is a serious problem when you need to use a class from an external library to which you can't assign that attribute.
Minimal example
CppObject.hpp
#include <memory>

struct CppObject {
	using Shared = std::shared_ptr<CppObject>;
	char *string;

	CppObject(int size) noexcept;
	~CppObject();
	static Shared make_shared(int size);
	void print() noexcept;
};

CppObject.cpp
#include "CppObject.hpp"
#include <cstring>
#include <iostream>

CppObject::CppObject(int size) noexcept {
	(string = new char[size + 1])[size] = '\0';
	std::memset(string, 'x', size);
}

CppObject::~CppObject() {
	delete[] string;
}

CppObject::Shared CppObject::make_shared(int size) {
	return std::make_shared<CppObject>(size);
}

void CppObject::print() noexcept {
	std::cout << string;
}

Swift
let test: CppObject.Shared = CppObject.make_shared(256)
test.pointee.print()

Result

Undesirable fix
#include <swift/bridging>
/* ... */

struct CppObject {
	/* ... */
} SWIFT_IMMORTAL_REFERENCE;

It looks like it works as expected. Note that test.pointee returns a copy of the value stored in the shared_ptr: test (on stack). (The pointee property in Swift is different from the deference operator T& operator*() in that it doesn't return a reference to the pointee but a copy of the value.)  The value gets destroyed at the end of main. The code desugars to:
let test: CppObject.Shared = CppObject.make_shared(256)
let temp: CppObject = test.pointee // test.pointee is copied into temp
temp.print()
// temp is destroyed here and the destructor of temp is called
// test.pointee is generally valid and usable here. However, in this case the "string" stored in CppObject is actually shared by all instances and therefore it gets destroyed when `temp` gets destroyed. If CppObject behaves like value type and deep copies the string, `test.pointee` would still be valid here.

This is not different from what happens in this case:
let x = CppObject(256)
x.print()
// destructor of 'x' is called here.

@redcode It looks like it works as expected. Note that test.pointee returns a copy of the value stored in the shared_ptr: test (on stack). (The pointee property in Swift is different from the deference operator T& operator*() in that it doesn't return a reference to the pointee but a copy of the value.)

The problem is that the object, in my example, is not trivially destructible. The copy of pointee deletes CppObject::string when destroyed, which is deleted again when test is destroyed at the end of the scope. Obviously, this could be fixed by using a copy constructor that would take care of copying CppObject::string, but that would be another undesirable fix, wouldn't be?

Obviously, this could be fixed by using a copy constructor that would take care of copying CppObject::string, but that would be another undesirable fix, wouldn't be?
yes, while my explanation was to reason about the behavior within the current semantics/model, there is certainly a lot of room to improve support for C++ smart pointers in Swift e.g. as highlighted here: https://forums.swift.org/t/desired-swift-language-features-that-can-advance-the-state-of-c-interoperability-support/66144/9

Also, since CppObject is actually a type with shared storage, I think it is quite reasonable to import it as a Swift class with SWIFT_IMMORTAL_REFERENCE annotation, as mentioned. API Notes can come in handy for adding these annotations to a type defined in an external library.

API Notes can come in handy for adding these annotations to a type defined in an external library.

This is very interesting, I was not aware of this new Clang feature. It seems to be just what I need to solve a number of problems.
Thanks :)

I found two very interesting "issues" with shared_ptr:

As y'all have noticed, .pointee unfortunately copies the instance T instead of borrowing it. I'm wondering - instead of marking the pointer T as SWIFT_IMMORTAL_REFERENCE, is there maybe a way to get pointee as a borrowing T in Swift to avoid that copy? 🤔
Passing a shared_ptr to a C-style Swift function (like a "callback") causes a heap corruption because it seems to be destroyed twice. Filed a bug report here: https://github.com/swiftlang/swift/issues/78292


Use Whack Whack as target for console applications

It would be awesome if I could use this as a target for console apps. Current default VS behavior for debugging console apps is to launch a native windows console, execute the project, then close the console on program exit. It order to prevent closing the terminal and see the output of the program, you usually have to do something like set a breakpoint, use Console.ReadLine(), or run without debugging. Even with that, the launching of the console breaks you out of the IDE, hides the code you were just looking at with the launched console, and generally is disruptive to flow.
The desired functionality would be that when I either "Start Debugging" or "Start without Debugging" on a console project, the "Terminal Window" panel would gain focus and provide console interaction with the console app.

I do not currently believe it is possible to hook into the Debug routines in this way. I agree that this would be a great feature to have but I don't think its possible with the APIs that VS provides today.


Allow specifying global security schema for all routes

Is your feature request related to a problem? Please describe.
Currently, specifying a security schema happens on a per-route basis with the @security annotation. Swagger allows specifying one security schema globally for all routes, e.g. https://swagger.io/docs/specification/authentication/bearer-authentication/
Describe the solution you'd like
Allow specifying a global list of security schemas in the configuration object, or marking security schemas defined in security as global, e.g.
{
    security: {
        BearerAuth:  {
            type: 'http',
            scheme: 'bearer',
            global: true,
        }
    },
}

🤔 I see your point. Right now, it is only configurable per route. Adding a global one might be a good idea but not all the API uses global, so we should keep both options. We will think about it.
We want to add a better security definition for Oauth2 so we will consider this in that development.


Is the heigh argument in Long constructor required?

The type definitions in @types/long define a type constructor like this:
new( low: number, high?: number, unsigned?: boolean ): Long;

The optional height allows users to write new Long(123). This is a problem as it makes it way too easy for callers to assume this is the correct number to Long conversion because it works for a very long time until it breaks.
The README marks heigh as required. The implementation works fine if unset because undefined | 0 is 0.
Is it safe to assume heigh is required and this can be considered a bug in the types?

Thanks for the detailed answer and README fix!


Add bash completion with available commands

I think it would be a nice idea to someway add bash completion to program names based on the index.json contents.

adding
complete -W "$(tldr 2>/dev/null --list)" tldr

will do the right thing.


Add Search Page

The users can be searchable by their names, description, and links. We can even add a Company & Location section, which can be used for search purposes.
Do you think something like that would be useful?

I didn't get where exactly on the website would this be implemented, being honest. Could you please be more specific?

@esau-morais I imagined a different page on either links.dev/search or search.links.dev (with this option we can initialize a new repository and work with next or nuxt js)

Before that, as a first step, we can make links.dev fully seo compatible, so search engines can index all users' pages. Do we have such support now?

We have 85 pages indexed @haliliceylan. Regarding the search page, I will close the issue for now and I'd like to observe the user growth a bit.


loading spinner for grid

i'm kind of new to react, wandering if the react grid has a loading spinner or i need to code that my self? i noticed two leads of (loading mask and loading-bar) but couldn't find an example on how to use them.

By default the grid comes with a very simple loader plugin. It must be enabled for it to be visible:
const plugins = {
    LOADER: {
        enabled: true
    }
}

Once you have it enabled, you can apply any custom CSS to change the look and feel. I've added an example on the demo site so you can get an idea.
http://react-redux-grid.herokuapp.com/CustomLoader
It's important to note that the component will handle when the loading mask should be visible/not visible. You can also set the loading state yourself by callling Actions.LoaderActions.setLoadingState.

thanks, will try that. Would appreciate if you update the react-redux-grid-demo repository

Done.

awesome, thank you soo much


Compile Fails: Some components (espressif/fmt) in the "managed_components" directory were modified (PACMAN-553)

Answers checklist.

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

IDF version.
v5.0-rc1
Operating System used.
Linux
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
None
What is the expected behavior?
Compile and build without issues like it does on Windows.
What is the actual behavior?
Works on Windows.
Fails on Linux/Mac.
Steps to reproduce.
# Clone recursive
git clone --recursive https://github.com/sukesh-ak/ESP32-TUX
cd ESP32-TUX

# Select the device through menuconfig
idf.py menuconfig

# Build and error shows up
idf.py build

Processing 2 dependencies:
CMake Error at /home/ubuntudev/esp/esp-idf/tools/cmake/build.cmake:519 (message):
  ERROR:

          Some components (espressif/fmt) in the "managed_components" directory were modified         on the disk since the last run of the CMake. Content of this directory is managed automatically.

          If you want to keep the changes, you can move the directory with the component to the "components"         directory of your project.

          I.E. for "espressif__fmt" run:
          mv /home/ubuntudev/github/ESP32-TUX/managed_components/espressif__fmt /home/ubuntudev/github/ESP32-TUX/components/espressif__fmt

          Or, if you want to discard the changes remove the ".component_hash" file from the component's directory.

          I.E. for "espressif__fmt" run:
          rm /home/ubuntudev/github/ESP32-TUX/managed_components/espressif__fmt/.component_hash

Call Stack (most recent call first):
  /home/ubuntudev/esp/esp-idf/tools/cmake/project.cmake:440 (idf_build_process)
  CMakeLists.txt:28 (project)

[2/2] idf (5.0.0)-- Configuring incomplete, errors occurred!

Though the error says, some changes were done but no changes were done.
Also it works fine on Windows and looking at the error details, sounds like a known issue.
Build or installation Logs.
CMake Error at /home/ubuntudev/esp/esp-idf/tools/cmake/build.cmake:519 (message):
  ERROR:

          Some components (espressif/fmt) in the "managed_components" directory were modified         on the disk since the last run of the CMake. Content of this directory is managed automatically.

          If you want to keep the changes, you can move the directory with the component to the "components"         directory of your project.

          I.E. for "espressif__fmt" run:
          mv /home/ubuntudev/github/ESP32-TUX/managed_components/espressif__fmt /home/ubuntudev/github/ESP32-TUX/components/espressif__fmt

          Or, if you want to discard the changes remove the ".component_hash" file from the component's directory.

          I.E. for "espressif__fmt" run:
          rm /home/ubuntudev/github/ESP32-TUX/managed_components/espressif__fmt/.component_hash

Call Stack (most recent call first):
  /home/ubuntudev/esp/esp-idf/tools/cmake/project.cmake:440 (idf_build_process)
  CMakeLists.txt:28 (project)

ninja: error: rebuilding 'build.ninja': subcommand failed
cat: and: No such file or directory
[0/1] Re-running CMake...
-- Component directory /home/ubuntudev/github/ESP32-TUX/components/ConfigBase does not contain a CMakeLists.txt file. No component will be added
-- Building ESP-IDF components for target esp32
Processing 2 dependencies:
[1/2] espressif/fmt (9.1.0)
[2/2] idf (5.0.0)
-- Configuring incomplete, errors occurred!
See also "/home/ubuntudev/github/ESP32-TUX/build/CMakeFiles/CMakeOutput.log".
FAILED: build.ninja
/home/ubuntudev/.espressif/tools/cmake/3.24.0/bin/cmake --regenerate-during-build -S/home/ubuntudev/github/ESP32-TUX -B/home/ubuntudev/github/ESP32-TUX/buildCMake Error at /home/ubuntudev/esp/esp-idf/tools/cmake/build.cmake:519 (message):
  ERROR:

          Some components (espressif/fmt) in the "managed_components" directory were modified         on the disk since the last run of the CMake. Content of this directory is managed automatically.

          If you want to keep the changes, you can move the directory with the component to the "components"         directory of your project.

          I.E. for "espressif__fmt" run:
          mv /home/ubuntudev/github/ESP32-TUX/managed_components/espressif__fmt /home/ubuntudev/github/ESP32-TUX/components/espressif__fmt

          Or, if you want to discard the changes remove the ".component_hash" file from the component's directory.

          I.E. for "espressif__fmt" run:
          rm /home/ubuntudev/github/ESP32-TUX/managed_components/espressif__fmt/.component_hash

Call Stack (most recent call first):
  /home/ubuntudev/esp/esp-idf/tools/cmake/project.cmake:440 (idf_build_process)
  CMakeLists.txt:28 (project)

ninja: error: rebuilding 'build.ninja': subcommand failed
cat: and: No such file or directory
[0/1] Re-running CMake...
-- Component directory /home/ubuntudev/github/ESP32-TUX/components/ConfigBase does not contain a CMakeLists.txt file. No component will be added
-- Building ESP-IDF components for target esp32
Processing 2 dependencies:
[1/2] espressif/fmt (9.1.0)
[2/2] idf (5.0.0)
-- Configuring incomplete, errors occurred!
See also "/home/ubuntudev/github/ESP32-TUX/build/CMakeFiles/CMakeOutput.log".
FAILED: build.ninja
/home/ubuntudev/.espressif/tools/cmake/3.24.0/bin/cmake --regenerate-during-build -S/home/ubuntudev/github/ESP32-TUX -B/home/ubuntudev/github/ESP32-TUX/build

More Information.
Here is the working version with demo on windows.
https://asciinema.org/a/549415
After idf.py menuconfig, if I manually delete the folder rm -rf managed_components it compiles again.
managed_components folder is already in .gitignore so I am guessing its created during idf.py menuconfig stage.

Hello @sukesh-ak,
I noticed that your issue seems to be similar to the one discussed in issue !15. We are currently working on a solution.
As a temporary workaround for ESP-IDF 5.0, you may try downgrading idf-component-manager to version 1.1.4. To do this, please run the following command within your ESP-IDF environment (after running export.ps1):
pip install idf-component-manager==1.1.4

I hope this helps.

Thanks. I looked into issues on IDF repo so didn’t see this.
It compiles after deleting that folder rm -rf managed_components.
I am using Windows so for now its ok. Some of the people who are using my project reported this issue so reported it here.

@sukesh-ak
A new bugfix release with the fix has been made available on PyPI. You can update to v1.2.2 either by running ./install.sh from the ESP-IDF directory or by running pip install -U idf-component-manager within the ESP-IDF environment (after sourcing export.sh).
If you need any help, please don't hesitate to reopen this issue.

@sukesh-ak A new bugfix release with the fix has been made available on PyPI. You can update to v1.2.2 either by running ./install.sh from the ESP-IDF directory or by running pip install -U idf-component-manager within the ESP-IDF environment (after sourcing export.sh).
If you need any help, please don't hesitate to reopen this issue.

Thank you for the update. Will test once I get a chance.


Update mod_tracing.rb feature capitalization

The feature is: IIS-HttpTracing per: https://docs.microsoft.com/en-us/iis/install/installing-iis-7/understanding-setup-in-iis
Thus this fails on windows versions that have case sensitive feature installation (windows 7 for example).
Description
This change allows http tracing to be installed on windows 7.
Issues Resolved
[List any existing issues this PR resolves]
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

SIGNOFF - Obvious fix.

Happy to fix the DCO on the commit message if y'all would like.  However, given how minor this is, I'm hopeful that someone wouldn't mind just merging this as is if approved :blush:

As this is just a simple fix we can merge it. After investigation it does look like this is correct. One thing to note that this cookbook may work on Windows 7 but it's not a supported Platform as per the README.
Thanks so much for taking the time to submit a PR.