Expose metrics

To gain visibility into reconciliation latency, errors, usual resource metrics such as CPU, mem, fd usage etc. it would be good if the pulumi-operator would implement at minimum the controller-runtime default metrics, see here.

FWIW we do get the default metrics from the operator SDK and the endpoint is available at <IP_ADDRESS>:8383 on the pod. Probably need to update the spec with the appropriate scraping annotations. If there is a standard set we should add - happy to take a contribution on this front.

Adding on to this, I think having some of the following metrics for stack info would be nice (for the endpoint on <IP_ADDRESS>:8686).

Update duration
Number of stacks currently updating/reconciling
Number of stacks with last update failed
Number of stacks
Total number of updates (along with failures/success)
Number of resources created/modified/deleted

@viveklak Are there any updates or plans to implement this on the Pulumi side?

@viveklak Are there any updates or plans to implement this on the Pulumi side?

This is on our prioritized backlog. The team is currently working towards some time-sensitive releases but we intend to have support for this by end of October. Thanks for your patience.


Release 2.2.0

https://github.com/apache/camel-k/issues/4738

I've just discovered that the latest default runtime (3.6.0) we're planned to use is not working as expected in native mode, due to the fact it is based on a jdk-17 image. The fix is relatively easy and needs to be done in any case, but that means we need to postpone at least one week the release waiting for 3.6.1 to be out. We may even release keeping 3.2.0 as default since we do not have a strong dependency requirement, but probably it makes sense to wait for the fix.
@oscerd wdyt?

I guess it makes sense to wait.

I've done the change on the runtime. As soon as we publish the snapshot I'll test with https://github.com/apache/camel-k/actions/workflows/nightly-latest-runtime.yml and if all is fine I'll start a new release process for the runtime 3.6.1 before kicking off 2.2.

I may have found a way to solve the problem. Probably we won't need to wait for the runtime release and we can start the build today itself. Let's wait for the PR's check.

Vote started.


128k flash, but 'firmware1.bin' is 448004 bytes length

Hi!
Who can explain this?
743VIT has 2M, it is ok, no questions.
750VBT has 128k flash.
How is it possible to program it with 400k+ of code?
Or what am I missing?

750 and 743 have 2Mb flash, but in 750 have software block all bank over 0s. ST-LINK utility flashing all 2mb and you can use ful size of internal flash.

750 and 743 have 2Mb flash, but in 750 have software block all bank over 0s. ST-LINK utility flashing all 2mb and you can use ful size of internal flash.

Thank you for answer.
I thought memory is blocked because it does not work.
Can I use 2MB on any 750VBT or should I check it's memory blocks?

750 and 743 have 2Mb flash, but in 750 have software block all bank over 0s. ST-LINK utility flashing all 2mb and you can use ful size of internal flash.

Thank you for answer.
I thought memory is blocked because it does not work.
Can I use 2MB on any 750VBT or should I check it's memory blocks?

750 and 743 have 2Mb flash, but in 750 have software block all bank over 0s. ST-LINK utility flashing all 2mb and you can use ful size of internal flash.


No parse_ident?

Unless I'm missing something, there's no function to parse a &str as an Ident with validation, like the other parse_* family of functions.
My proc macro constructs Idents from user-supplied strings, so it would be nice to fail early instead of letting the non-ident bubble up all the way to the final TokenStream and fail with a non-descriptive LexError.

I added syn::parse_ident in 0.10.4.


GetBalance api is waiting forever

Describe the bug
GetBalance API is not responsive
To Reproduce

Firewall on (it means incoming traffic on 9651 is blocked). (this is not intentional, i missed to turnoff firewall)
Create user and wallet address
Add balance in faucet testnet
Hit the balance api. it shows balance as zero
wait for some time till you see following log in your node.
INFO [05-27|08:13:23] <chain 4ktRjsAKxgMr2aEzv9SWmrU7Xk5FniHUrVCX4P1TZSfTLZWFM> /snow/engine/avalanche/bootstrapper.go#234: bootstrapping finished fetching vertices. executing state transitions...
Hit GetBalance api. it waits forever.

Expected behavior
API shouldn't wait forever. should return zero.
Screenshots
If applicable, add screenshots to help explain your problem.
Operating System
Ubunut 1804
Additional context
Node console outout
INFO [05-27|08:07:27] <chain 4ktRjsAKxgMr2aEzv9SWmrU7Xk5FniHUrVCX4P1TZSfTLZWFM> /snow/engine/avalanche/bootstrapper.go#178: bootstrapping has fetched 5000 vertices

INFO [05-27|08:11:06] <chain 4ktRjsAKxgMr2aEzv9SWmrU7Xk5FniHUrVCX4P1TZSfTLZWFM> /snow/engine/avalanche/bootstrapper.go#178: bootstrapping has fetched 7500 vertices
INFO [05-27|08:13:23] <chain 4ktRjsAKxgMr2aEzv9SWmrU7Xk5FniHUrVCX4P1TZSfTLZWFM> /snow/engine/avalanche/bootstrapper.go#234: bootstrapping finished fetching vertices. executing state transitions...

INFO [05-27|08:13:23] <chain 4ktRjsAKxgMr2aEzv9SWmrU7Xk5FniHUrVCX4P1TZSfTLZWFM> /snow/engine/avalanche/bootstrapper.go#234: bootstrapping finished fetching vertices. executing state transitions...

<.... turned off firewall.....>
ERROR[05-27|08:17:38] <chain 4ktRjsAKxgMr2aEzv9SWmrU7Xk5FniHUrVCX4P1TZSfTLZWFM> /vms/avm/unique_tx.go#126: Failed to fund utxo %!s(uint64=1) due to packer has insufficient length for input
INFO [05-27|08:17:44] <chain 4ktRjsAKxgMr2aEzv9SWmrU7Xk5FniHUrVCX4P1TZSfTLZWFM> /snow/engine/avalanche/transitive.go#66: Bootstrapping finished with 2 vertices in the accepted frontier
INFO [05-27|08:17:46] <chain<PHONE_NUMBER>1111111111111111111111LpoYY> /vms/platformvm/vm.go#697: next scheduled event is at 2020-05-27 17:46:05 +0530 IST (9h28m18.727995s in the future)
INFO [05-27|08:17:46] <chain<PHONE_NUMBER>1111111111111111111111LpoYY> /vms/platformvm/vm.go#697: next scheduled event is at 2020-05-27 17:46:05 +0530 IST (9h28m18.7192507s in the future)

By submitting this issue I agree to the Terms and Conditions of the Developer Accelerator Program.

Having this problem too after sync.

I'm having this problem too on 0.5.5

Hi all, can you please see if you still have this issue when running with version 0.5.7?
Let me know. Thanks

(oops didn't mean to close this just yet)

sure. will test.

GetBalance api is still waits forever when X-chain is executing operations
See following logs when GetBalance api is hit
INFO [06-26|02:55:47] <X Chain> /snow/engine/avalanche/bootstrapper.go#330: executed 890000 operations
INFO [06-26|02:55:50] <X Chain> /snow/engine/avalanche/bootstrapper.go#330: executed 892500 operations
INFO [06-26|02:55:52] <X Chain> /snow/engine/avalanche/bootstrapper.go#330: executed 895000 operations
INFO [06-26|02:55:55] <X Chain> /snow/engine/avalanche/bootstrapper.go#330: executed 897500 operations
INFO [06-26|02:55:57] <X Chain> /snow/engine/avalanche/bootstrapper.go#330: executed 900000 operations

once X-chain bootstrapping is completed, GetBalance api is returning response.

Infinite waits are incorrect. i looked at the code and should use contexts here to set the timeouts in future if required ? thoughts ?

@adasari The expected behavior right now is that if bootstrapping is not completed, an API call may hang forever. As such, I'm going to close this issue since your call completed after finishing bootstrapping. However, you're totally right that we should use contexts and timeouts. I'll leave this for a separate feature request issue, though. Thanks for the suggestion :)


PVs where Phase=Available need reindexing

In case of cluster crash and status recovery, the persistentVolumeClaimBinder would not re-add available volumes to its internal index.  Users would not be able to claim those volumes.
@thockin

This is really bad.  I think we should just nix Pending phase and always consider every Available volume for indexing - map lookups are cheap.


gradle 4.2 - play 2.6.x - scala 2.11

I have a problem starting with play 2.6, but scala 2.11 just by running gradle playRun:

Failed to run Play
org.gradle.internal.UncheckedException: java.lang.NoSuchMethodException: play.core.server.DevServerStart$.mainDevHttpMode(play.core.BuildLink, play.core.BuildDocHandler, int, java.lang.String)

By looking into PlayRunAdapterV26X I see that it just extends PlayRunAdapterV24X which calls that mainDevHttpMode with reflections while using wrong arguments.
In change logs its written that there is a support to 2.6.x play versions, but I cannot just run the server. Maybe I am doing smthg wrong?

Could you please provide reproducible sample project?

Hi @bmuschko, I confirm having the same issue.
It looks like the updated plugin works just fine with Play < 2.6, but as soon as you set the platform to 2.6 the server cannot be started. A method signature was changed, and it is not reflected in PlayRunAdapterV26X as pointed out by @vkokurin
To reproduce, just create an empty project:
apply plugin: 'play'

model {
    components {
        play {
            platform play: '2.6.5'
        }
    }
}

Then, gradlew runPlay while fail and throw the exception mentioned above.

For reference - https://github.com/playframework/playframework/commit/7bf7571c6997761baff79a0e89b08f9f5525b08f#diff-58bf6474fbdd7970aeb1630b24c9f7e2

@nberthet yes I have exactly same configuration.

I encountered a similar issue - BTW, is there a bare bone sample project to illustration gradle 4.2 play 2.6.5(x) actually works?
-cl

Hi, @bmuschko.
This is a reproducer: https://github.com/marcospereira/play-java-starter-example
See more info at this specific commit:
https://github.com/marcospereira/play-java-starter-example/commit/d9cb6fab92a83bc12a73463e83f65ae9b415fc10
Best.

Also, see #3018 for a (possible) fix.

Is there any workaround available guys with current gradle version? (4.2.1)

I am very interested for it as well - it is kind of a showstopper for us ...
https://github.com/gradle/gradle/releases/tag/v4.2.0-RC1

...Play framework support is upgraded in Gradle 4.2: Play v2.6, built-in Twirl template types and user-defined Twirl formats are now supported....

@kknd22 well sbt is always there

yes, I am aware of it and thanks for pointing it out - however, our build/ci/cd pipeline are all in gradle and build scala are just part of whole thing - there are many other aspects (other non scala code) of the build It is impractical change to using mixing gradle/sbt because of this defect..

Is this fix scheduled to be released anytime soon? It has become a blocker for us

@lu4nm3, you can see the progress at #3018.

@vkokurin, @nberthet, @kknd22 and @lu4nm3,
Gradle 4.4 RC1 is out with a fix for this issue. Could you please try and report that it is working as expected?
Thank you!

Hi @marcospereira, sorry for the lack of feedback, I've been using the 4.4 nightly as soon as the update has been merged, on my end everything works fine
Thank you for the update and follow up !

@marcospereira Sorry for the late reply. Started using the RC yesterday and everything appears to be working well on my end. Thank you for all the hard work!


Need a fart sound in your soundmachine

It would be really comforting if you could include a wet fart sound in your sounds.   The kind that make you think you need to change your underwear.

Yes, I have every intention of adding this feature but I am waiting for some recording equipment to capture my very own farts in high definition audio, for that personal touch. :-)


Buckets() for categories

Support color: ramp(buckets($type, 'typeA', 'typeB', typeC'), [red, green, blue, gray])

Since it was related I fixed top() too:
Before:

After:

New feature:

Tested with
width: 3
color: ramp(buckets($category, "Moda y calzado", "Bares y restaurantes", "Salud"), Prism)

Not deployed yet

nice!


Rewards related changes

Should be reviwed and merged first https://github.com/Taraxa-project/taraxa-evm/pull/121. Some needed changes was in the same commit with docs, so docs related changes also included here

prnet cleaned-up successfully


Update dependencies

Codecov Report
Patch and project coverage have no change.

Comparison is base (9df8bbe) 100.00% compared to head (3e4dbc2) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #179   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines          882       882
  Branches       193       193
=========================================
  Hits           882       882

Impacted Files
Coverage Δ

src/coola/allclose.py
100.00% <ø> (ø)

src/coola/equality.py
100.00% <ø> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Revert "BUG-104696 – fix for passing tag key with only space value"

Reverts hortonworks/cloudbreak#3553

@gregito this one should go first to the rc-2.7


Fix toolchain warning during build in Gradle 8.x

Pull Request Details

Description
Fixes

Using a toolchain installed via auto-provisioning, but having no toolchain repositories configured. This behavior is deprecated. Consider defining toolchain download repositories, otherwise the build might fail in clean environments; see https://docs.gradle.org/8.1.1/userguide/toolchains.html#sub:download_repositories

Related Issue
None, but the issue is internal and trivial, does not affect users, happy to open if needed.
Motivation and Context
Cleaner build during contribution, so it's clear that the build works as expected.
How Has This Been Tested
gradlew build -x test
Types of changes

[x] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have included my change in the CHANGELOG.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thanks!

@hsz there's something strange, there are extra commits in this PR that were not on master, not sure what's up I branched from master.

No worries about them, all's good.


[DRAFT] Override equals and hashcode for DiskInfo

What changes were proposed in this pull request?
Override equals and hashcode for DiskInfo
Why are the changes needed?
What are the items that need reviewer attention?
Do we need to remove some unnecessary fields from hashcode() and equals() ?
Related issues.
None yet
Related pull requests.
How was this patch tested?
/cc @AngersZhuuuu
/assign @main-reviewer

Why need to override this?

Why need to override this?

Currently we doesn't use DiskInfo as HashMap's key or compare it for equivalence,  but other similar class such as DeviceInfo, WorkerInfo has override them. Shall we only override them when code logic needs?


don't use intake v0.6.5, and get the CI passing

Intake 0.6.5 seems to have a bug in it causing errors in databroker.
So the CI run against intake master is expected to fail.
See issue https://github.com/intake/intake/issues/643
Pymongo 4 dropped Collection.insert() so we have to use Collection.insert_one() or Collection.insert_many()

We should either drop the intake master branch tests or sort out how to provide that missing attribute.

We should either drop the intake master branch tests or sort out how to provide that missing attribute.

+1 for dropping the tests against intake master.


fix: 🐛 Update compatible version to 5.2.x

Description
Update compatible version to 5.2.x
Breaking Changes
NA
JIRA Link
NA
Checklist

[x] Updated the Readme.md (if required) ?

:tada: This PR is included in version 19.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 19.2.0-alpha.2 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


E2E Migration

This is rather task than an issue atm: Verify, the VM migration works.
Opening it just as a reminder to not skip it.
On my cluster with 2 compute nodes, the migration object is created but the VM is not actually migrated.
The VM Status is repeatedly switching between Migration and Running (as the underlying migration pod is started/terminated).
My dev env might not be properly set to support migration.
If so, the user needs to be informed about what's missing.
Kubevirt v0.11.0
Upshift environment

Except other issues: consider VMI object's Consition: DisksNotLiveMigratable, LiveMigratable, etc.

/reopen
/remove-lifecycle rotten
On the wish-list as we have spoken with @rhrazdil


Update sample with latest changes to converter

Final nail in #161
Closes #161

👍  would love to see this merged~ #171 to be rebased on this


Add all the contributors in table format at the bottom

Can you elaborate more?
I would like to work on this.

Can you elaborate more?
I would like to work on this.

Add something that automatically adds contributor's names in the main readme file of datascience mashup at the bottom @sanjay270899

Add something that automatically adds contributor's names in the main readme file of datascience mashup at the bottom @sanjay270899

I would Like to work on this.

I would Like to work on this.


[Upgrade] Lucene 9.0.0 release

This PR upgrades the core codebase from Lucene 8.9 to lucene-9.0.0-snapshot-803d131fd08. It includes all necessary refactoring of features and API changes when upgrading to a new major Lucene release.
closes #1067

TODO:

remove all deprecated features from Legacy version 6.x (will open a meta issue to track)

> Task :modules:mapper-extras:compileJava FAILED
/var/CITOOL/workflow/OpenSearch_CI/PR_Checks/Gradle_Check/search/modules/mapper-extras/src/main/java/org/opensearch/index/mapper/SearchAsYouTypeFieldMapper.java:403: error: PrefixFieldType is not abstract and does not override abstract method valueFetcher(QueryShardContext,SearchLookup,String) in MappedFieldType
    static final class PrefixFieldType extends StringFieldType {
                 ^
/var/CITOOL/workflow/OpenSearch_CI/PR_Checks/Gradle_Check/search/modules/mapper-extras/src/main/java/org/opensearch/index/mapper/SearchAsYouTypeFieldMapper.java:441: error: method does not override or implement a method from a supertype
        @Override
        ^
/var/CITOOL/workflow/OpenSearch_CI/PR_Checks/Gradle_Check/search/modules/mapper-extras/src/main/java/org/opensearch/index/mapper/SearchAsYouTypeFieldMapper.java:445: error: incompatible types: MapperService cannot be converted to QueryShardContext
            return SourceValueFetcher.toString(name(), mapperService, format);
                                                       ^
/var/CITOOL/workflow/OpenSearch_CI/PR_Checks/Gradle_Check/search/modules/mapper-extras/src/main/java/org/opensearch/index/mapper/SearchAsYouTypeFieldMapper.java:534: error: ShingleFieldType is not abstract and does not override abstract method valueFetcher(QueryShardContext,SearchLookup,String) in MappedFieldType
    static class ShingleFieldType extends StringFieldType {
           ^
/var/CITOOL/workflow/OpenSearch_CI/PR_Checks/Gradle_Check/search/modules/mapper-extras/src/main/java/org/opensearch/index/mapper/SearchAsYouTypeFieldMapper.java:547: error: method does not override or implement a method from a supertype
        @Override
        ^
/var/CITOOL/workflow/OpenSearch_CI/PR_Checks/Gradle_Check/search/modules/mapper-extras/src/main/java/org/opensearch/index/mapper/SearchAsYouTypeFieldMapper.java:551: error: incompatible types: MapperService cannot be converted to QueryShardContext
            return SourceValueFetcher.toString(name(), mapperService, format);
                                                       ^
Note: Some messages have been simplified; recompile with -Xdiags:verbose to get full output
6 errors

Odd this check has been going for almost four hours! If it doesn't resolve soon we're going to need to re-trigger.

Reproducible test timeout. Documented at #2063. Muting for now:
  2> REPRODUCE WITH: ./gradlew ':server:test' --tests "org.opensearch.cluster.routing.allocation.IndexShardHotSpotTests.testClusterScaleInWithSkew" -Dtests.seed=20DC11D9C132C489 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=es-HN -Dtests.timezone=Europe/Volgograd -Druntime.java=17

org.opensearch.cluster.routing.allocation.IndexShardHotSpotTests > classMethod FAILED
    java.lang.Exception: Suite timeout exceeded (>= 1200000 msec).
        at __randomizedtesting.SeedInfo.seed([20DC11D9C132C489]:0)

Another test timeout. Not reproducible locally:
REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.cluster.routing.DelayedAllocationIT.testDelayedAllocationTimesOut" -Dtests.seed=5875B3A40C200121 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=es-SV -Dtests.timezone=Pacific/Galapagos -Druntime.java=17

org.opensearch.cluster.routing.DelayedAllocationIT > testDelayedAllocationTimesOut FAILED
    java.lang.AssertionError: timed out waiting for green state
        at org.junit.Assert.fail(Assert.java:89)
        at org.opensearch.test.OpenSearchIntegTestCase.ensureColor(OpenSearchIntegTestCase.java:985)
        at org.opensearch.test.OpenSearchIntegTestCase.ensureGreen(OpenSearchIntegTestCase.java:924)
        at org.opensearch.test.OpenSearchIntegTestCase.ensureGreen(OpenSearchIntegTestCase.java:913)
        at org.opensearch.cluster.routing.DelayedAllocationIT.testDelayedAllocationTimesOut(DelayedAllocationIT.java:116)

start gradle check

Massive one, @nknize ! Great job

start gradle check

@nknize Sent this one https://github.com/nknize/OpenSearch/pull/1, all HighlighterSearchIT pass and hopefully with JDK-11 build fixed.

@nknize more test case fixes https://github.com/nknize/OpenSearch/pull/2

@nknize sorry, DCO check failed for some reasons :( could you please amend the last commit? thank you


Automatic Tagging

added generic tag recognition logic to the project
added new option in settings screen to on/off this new feature ("Automatic Tags")
added specific tag recognition for trello (using trello card labels) [ignores unnamed labels]
added specific tag recognition for asana (using asana task tags)
fixed bug: after fetching projects when constructing html to return there was a condition deleting projects from clients with only one project.
previous conflicting condition was: (wsHtml[key][ckey].match(//option/g) || []).length > 1)
changed to: (wsHtml[key][ckey].match(//option/g) || []).length > 0)

Hi,
Thank you for contributing. I'm terribly sorry for the lack of feedback.
I think we should implement this in a couple more tools before we can release it. Namely Asana, Jira and Todoist.
We can maybe help out with those but the PR needs a rebase first.

Hey @dbarletta,
I realize it has been a while since you submitted this PR (2015 :fearful:). We will close this PR for now as we decide whether to implement this feature or not. If we do, we'll make sure to give you some well deserved credits!
I opened an issue here related to your PR here: #1186


Full resolution Color image

Thank you for these very nice demos indeed.
I noticed you are visualizing the low-resolution Preview of the color stream.
Is there a way to get the HD or even full 4K resolution from the camera? If so, where do we need to modify it? (At the moment when the Preview is turned off, we get no images at all)

Hi ! yes, usually color image preview is low-res (same input size of NN).
If you want to retrieve full res image, I'd suggest to take a look to the streams example and just modify the pipeline to retrieve full image instead of preview
https://github.com/luxonis/depthai-unity/blob/main/src/device/Streams.cpp
Just change the link from the camera node to output node
That said, I will try to work on the example itself so you can select the resolution from unity interface

Hi! Thank you for this repo.
Could you describe where it needs to be changed, the size of the image.("Just change the link from the camera node to output node") How to build the files? Because when I try, I always get an error.

Hi @balogistvan
Camera node has preview and normal output
You can find in line 56 of Streams.cpp
colorCam->preview.link(xlinkOut->input);
To build the plugin dll you need to follow the steps here:
https://github.com/luxonis/depthai-unity/blob/main/docs/Build_unity_lib.md
What errors are you getting?
Let me know if this info it's not enough, I will try to make the changes in a branch of the repo

Thanks for the answer. I'll try to build it and post the result here.

Hello. Now I am getting this error. Could this be because there is no depthai camera connected to the computer?
C:\Users\balog\depthai-unity\depthai-core\include\depthai/device/Device.hpp(67): error C2061: syntax error: identifier 'Device' [C:\Users\balog\depthai-unity\build\depthai-core\depthai-core.vcxproj] C:\Users\balog\depthai-unity\depthai-core\include\depthai/device/Device.hpp(67): error C2238: unexpected token(s) prece ding ';' [C:\Users\balog\depthai-unity\build\depthai-core\depthai-core.vcxproj] C:\Users\balog\depthai-unity\depthai-core\include\depthai/device/Device.hpp(97): fatal error C1903: unable to recover f rom previous error(s); stopping compilation [C:\Users\balog\depthai-unity\build\depthai-core\depthai-core.vcxproj] Generating Code...
And in line 56 of Streams.cpp colorCam->preview.link(xlinkOut->input);
The correct code is:
if (config->previewSizeWidth > 0 && config->previewSizeHeight > 0) { xlinkOut = pipeline.createdai::node::XLinkOut(); xlinkOut->setStreamName("preview"); colorCam->setPreviewSize(config->previewSizeWidth, config->previewSizeHeight); colorCam->color.link(xlinkOut->input); }
or:   colorCam->normal.link(xlinkOut->input); } ?
Is there anything else that needs to be changed? Thank You.

Seems error compilation and it does NOT depends on depthai camera connected or not.
I see some typo here:  xlinkOut = pipeline.createdai::node::XLinkOut
To choose another output from the colorcamera node you can check here: https://docs.luxonis.com/projects/api/en/latest/components/nodes/color_camera/?highlight=colorcamera
So in case to retrieve full res image you can use ->isp or ->still or even ->raw
Will try to setup a branch in next days

Dear Gerard. Can you upload a verson with full resolution images output? Thank You.

Thank You.


chore: mark AREnableActivity as ready for release

This PR resolves []
Description
Echo PR: artsy/echo#336
PR Checklist

[ ] I tested my changes on iOS / Android.
[ ] I added screenshots or videos to illustrate my changes.
[ ] I added Tests and Stories for my changes.
[ ] I added an app state migration.
[x] I hid my changes behind a feature flag.
[ ] I have prefixed changes that need to be tested during a release QA with [NEEDS EXTERNAL QA] on the changelog.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

mark AREnableActivity as ready for release - dimatretyak

Need help with something? Have a look at our docs, or get in touch with us.

The tests will be fixed after the first echo deploy


fzf doesn't accept input

Category

[x] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Today I just upgrade the fzf to 2d61691 and run fzf on current directory, but it doesn't accept any input..
I just bisect it, found that this commit is the culprit: 6ccc12c332f97f6be9487d8289c46964e2d41bd3,
Any help? Thanks.

Please provide more details about your environment: OS distro, terminal emulator, fzf --version, etc. Do you have the same issue when you do fzf --height 30%?
6ccc12c332f97f6be9487d8289c46964e2d41bd3 was made in January, and no one has reported this issue so far. Hmm.

@junegunn
> fzf --version
0.16.7
> uname -a
Linux totorow 4.4.6-gentoo #1 SMP Wed Apr 26 15:08:41 CST 2017 x86_64 Intel(R) Core(TM) i5-4590 CPU @ 3.30GHz GenuineIntel GNU/Linux

And I use urxvt, its version is: rxvt-unicode (urxvt) v9.21 - released: 2014-12-31

Do you have the same issue when you do fzf --height 30%?

Yes, it doesn't work as expected, it just hang:
> fzf --height 30%

And ctrl-c doesn't work, I have to kill it in another terminal.
Do you need any other information?

it just hang

So you see nothing? Same issue with echo foo | fzf --height 30%?

Does the following command report the position of your cursor?

echo -e "\x1b[6n"

And can you test it with a different terminal emulator? (I'm not saying that urxvt is not supported. I can't reproduce the problem with urxvt installed on my mac.)

So you see nothing? Same issue with echo foo | fzf --height 30%?
Yes.

Does the following command report the position of your cursor?

/home/tw/golib/src/github.com/junegunn/fzf [tw@totorow]
> echo -e "\x1b[6n"
"
/home/tw/golib/src/github.com/junegunn/fzf [tw@totorow]
> ;1R;2R

And can you test it with a different terminal emulator?

I test it in xterm, same issue, weird...

That's strange. Can you check what's going on under the hood using strace?
FZF_DEFAULT_COMMAND='seq 100000000' strace -o /tmp/o -f fzf

trace.txt

Trace shows that fzf successfully started child process (seq) and it worked as expected (read(5, ...) and close(5)). fzf is also reading your keyboard input (read(3, ...)). write(2, ...) indicates that it's rendering the finder on STDERR, which is also expected. I don't see why it's not rendered on your screen.
You seem to be using fzf binary built from source. I'm not sure if it'll make any difference, but can you test again with the prebuilt binary below?
https://github.com/junegunn/fzf-bin/releases/download/0.16.7/fzf-0.16.7-linux_amd64.tgz

Ah, the prebuilt binary works !
I just go get -u github.com/junegunn/fzf/src/fzf, is there any difference?

I have no idea. I used go version go1.8.1 to build the binaries.

I have just bisect go version locally, this commit caused the issue.

Thanks, I'll take note.


ARROW-950: [Website] Add Google Analytics tag to site

I created this GA property. I will be happy to give access to any PMC who wishes to look at the analytics data. It would be better to be collecting the site visitor data than not.

+1

I'm interested in this. Please give me access. My Google Analytics account is<EMAIL_ADDRESS>

Done

Thanks! Confirmed.

@wesm please also add me


CI: Keep the mysql DB running

This PR modifies the deploy script keep the deployed mysql-DB in our CI test environment running. It achieves that by moving the mysql kubernetes config in a separate file. Resources are now deleted by file instead of by namespaces.
Tested by running the deploy script from my machine.
resolves #746

Codecov Report

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

@@          Coverage Diff           @@
##           master    #747   +/-   ##
======================================
  Coverage    48.7%   48.7%
======================================
  Files          30      30
  Lines        2505    2505
======================================
  Hits         1220    1220
  Misses       1088    1088
  Partials      197     197

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 bb9607e...c4f8603. Read the comment docs.


[refactor]: move expression len out of public API into core

Description
let's not pollute public API with internal stuff
Linked issue

Closes #{issue_number}

Benefits

Checklist

[ ] I've read CONTRIBUTING.md
[ ] I've used the standard signed-off commit format (or will squash just before merging)
[ ] All applicable CI checks pass (or I promised to make them pass later)
[ ] (optional) I've written unit tests for the code changes
[ ] I replied to all comments after code review, marking all implemented changes with thumbs up

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 93 (18.28%)  changed or added relevant lines in 2 files are covered.
6252 unchanged lines in 108 files lost coverage.
Overall coverage decreased (-0.8%) to 58.645%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

data_model/src/evaluate.rs
3
6
50.0%

core/src/tx.rs
14
87
16.09%

Files with Coverage Reduction
New Missed Lines
%

cli/src/main.rs
1
0.0%

cli/src/samples.rs
1
0.0%

cli/src/torii/pagination.rs
1
98.9%

config/base/src/runtime_upgrades.rs
1
51.72%

config/src/torii.rs
1
96.67%

core/src/smartcontracts/isi/block.rs
1
87.5%

crypto/src/merkle.rs
1
96.23%

data_model/src/domain.rs
1
48.25%

logger/src/lib.rs
1
95.15%

config/src/lib.rs
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.8%

Covered Lines:
21698

Relevant Lines:
36999

💛  - Coveralls

@mversic we use len to check limit in max_instruction_number TransactionLimits. How user can evaluate it's transaction limits if len is moved into core?

@mversic we use len to check limit in max_instruction_number TransactionLimits. How user can evaluate it's transaction limits if len is moved into core?

they already cannot do it because the code for instruction_size is in the core since #3949, I just moved the remainder. Until I see a request for such a feature in the client, I wouldn't like to keep it in the public API. Public API can always be expanded, but not reduced. Also, in the new ISA expressions will be evaluated in Executor
That being said, max_instruction_number sounds like a misnomer since it tracks expressions


[anomalydetector] Fixed typos

Description
closes https://github.com/Azure/azure-sdk-for-python/issues/22641
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

@KdHyeon0661 Thanks for your contribution!


Need Travis Configured?

Are there tests for this? Should we have travis configured on this project?

We will before too long.  No tests yet, because I'm in early development still.

Travis is now enabled for the repo.


Fix the issue of "Initializers are not allowed in ambient contexts" for 3.0.x version of TS

@bterlson - Please review


dxhd spawning child processes

Take a look at this, every time i use a keybind, i get one of these children processes. They don't seem to remove themselves.
Using the latest git version.

Oh, so dxhd does spawn Zombie processes. That's bad. Thanks for reporting! I will fix this.

  However, I'll figure this out, I don't want users to have lots of zombie processes.

Fix: Run processes as goroutines, and pass them top-level context, to stop the process on context.Done()

That's fixed by the latest release.


compare-dependencies does not fail if remotePom is invalid

versions:compare-dependencies:2.16.2 shows no error if remotePom parameter value is non existing artifact.
[INFO] The following differences were found:
[INFO]   none
[INFO] The following property differences were found:
[INFO]   none
I think it should fail in case the remote pom cannot be resolved.

please provide a simple project to reproduce


Improve detokenization performance

Two main changes:

if we are using a fast tokenizer, we do not enter the slow _convert_tokens_to_string_with_added_encoders loop as the fast tokenizers do not use it in base transformers
Use cached properties for added_tokens_encoder and all_special_tokens.

Those 2 changes improved detokenization speed for 4096 tokens from 13ms to 2ms.

cc @WoosukKwon @zhuohan123


Fixing Issue 70. WIP

Tried my hand at solving issue 70.  Couple of things I wanted to ask:

When I hover over submodules for the packages (For example the clusters submodule under the samplers package), the descriptions don't show up despite adding them to their respective __init__.py files.
Would it be possible to get a brief description of some of the packages?

Thank you for taking care of this! Addressing your questions:

I have no problem showing the tooltips for submodules, like clusters. Have you created and sourced the venv at packages/clusview? This is from a file at the root level of the package (packages/clusview) on my local machine.

Which packages/files are you struggling with? If I end up giving you descriptions for the packages I would almost be solving the issue in the process 😉.


Failed to load resource: net::ERR_FAILED

After removing the NewTab extension I'm seeing the errors below on console for every new chrome tab opened.
newtab:7 SW registered
redirect.html Failed to load resource: net::ERR_FAILED

Sorry I missed this issue. Seems like you needed to restart the browser. This includes any status bar icons. Once Chrome is fully released from memory, it will no longer hold references to removed extensions.


digitalocean_sd_configs Unable to authenticate you

What did you do?
I have got this error in DigitalOcean discovery
ts=2022-06-10T08:55:02.897Z caller=refresh.go:79 level=error component="discovery manager scrape" discovery=digitalocean msg="Unable to refresh target groups" err="error while listing droplets page 0: GET https://api.digitalocean.com/v2/droplets: 401 (request \"8c4d699e-65ea-43a4-a17c-15c235953b92\") Unable to authenticate you"

But I can get the correct response using this command, so we can expect the key is correct.
curl -X GET -H "Content-Type: application/json" -H "Authorization: Bearer cce5eb9ba587f....." "https://api.digitalocean.com/v2/account"
In my config, both variants of authentications aren't work
What did you expect to see?
No response
What did you see instead? Under which circumstances?
ts=2022-06-10T08:55:02.897Z caller=refresh.go:79 level=error component="discovery manager scrape" discovery=digitalocean msg="Unable to refresh target groups" err="error while listing droplets page 0: GET https://api.digitalocean.com/v2/droplets: 401 (request \"8c4d699e-65ea-43a4-a17c-15c235953b92\") Unable to authenticate you"

System information
Linux 5.4.0-107-generic x86_64
Prometheus version
prometheus, version 2.36.1 (branch: HEAD, revision: 3c0a100dec2950f23e91f7efbf8867750960807d)
  build user:       root@e59c79158868
  build date:       20220609-15:25:21
  go version:       go1.18.3
  platform:         linux/amd64

Prometheus configuration file
global:
  scrape_interval: 60s
  evaluation_interval: 60s
  external_labels:
    monitor: digitalsilk
rule_files:
  - /opt/prometheus/etc/rules/redis_exporter_recording_rules.yaml

scrape_configs:

  - job_name: donodeexporter
    scrape_interval: 60s
    scrape_timeout: 15s
    digitalocean_sd_configs:
      - bearer_token: cce5eb9ba587f.....
#      - authorization:
#          credentials: cce5eb9ba587f.....
      - port: 9100
      - refresh_interval: 120s

Alertmanager version
No response
Alertmanager configuration file
No response
Logs
ts=2022-06-10T08:55:02.897Z caller=refresh.go:79 level=error component="discovery manager scrape" discovery=digitalocean msg="Unable to refresh target groups" err="error while listing droplets page 0: GET https://api.digitalocean.com/v2/droplets: 401 (request \"8c4d699e-65ea-43a4-a17c-15c235953b92\") Unable to authenticate you"

Your yaml config is incorrect
 - job_name: donodeexporter
  scrape_interval: 60s
  scrape_timeout: 15s
  digitalocean_sd_configs:
    - authorization:
        credentials: cce5eb9ba587f.....
      port: 9100
      refresh_interval: 120s

Thank you.
Now, I have got the new error
ts=2022-06-10T09:49:12.612Z caller=scrape.go:1313 level=debug component="scrape manager" scrape_pool=donodeexporter target=http://<IP_ADDRESS>:80/metrics msg="Scrape failed" err="Get \"http://<IP_ADDRESS>:80/metrics\": context canceled"

Mmh it's strange, did you add the port?
Le ven. 10 juin 2022, 11:50, Rostyslav @.***> a écrit :

Thank you.
Now, I have got the new error
ts=2022-06-10T09:49:12.612Z caller=scrape.go:1313 level=debug component="scrape manager" scrape_pool=donodeexporter target=http://<IP_ADDRESS>:80/metrics msg="Scrape failed" err="Get "http://<IP_ADDRESS>:80/metrics\": context canceled"
—
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/issues/10856#issuecomment-1152183226,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACHHJQFJDEXLS4TCN7Z62DVOMFWLANCNFSM5YM7TE5Q
.
You are receiving this because you commented.Message ID:
@.***>

Yes, I have added port
scrape_configs:

  - job_name: donodeexporter
    scrape_interval: 60s
    scrape_timeout: 15s
    digitalocean_sd_configs:
      - authorization:
          credentials: dop_v1_58f1e93....
        port: 9100
        refresh_interval: 120s

But the target still has 80 port, instead 9100
ts=2022-06-10T09:53:41.278Z caller=scrape.go:1313 level=debug component="scrape manager" scrape_pool=donodeexporter target=http://<IP_ADDRESS>:80/metrics msg="Scrape failed" err="Get \"https://<IP_ADDRESS>/metrics\": x509: cannot validate certificate for <IP_ADDRESS> because it doesn't contain any IP SANs"

I have added relabel rules to narrow the number of instances for debugging. But suddenly I started to receive data. :)
  - job_name: donodeexporter
    digitalocean_sd_configs:
      - bearer_token: 'dop_v1_58f...'
        port: 9100
    relabel_configs:
      - source_labels: [__meta_digitalocean_tags]
        regex: .*,monitoring_yes,.*
        action: keep


Add gamma correction feature from the SDK

This commit modifies gamma_correction, capture_2d, and capture,hdr_complete_settings samples to include gamma correction feature from the SDK. It also modifies the CaptureTutorial to keep code snippets valid due to the above mentioned code modifications.

#99  is merged. I've rebased. Need another review/approval @robertkt1 and CI to pass.

#99  is merged. I've rebased. Need another review/approval @robertkt1 and CI to pass.

Looks like the CI has a legitimate complaint. You can fix that, and I can approve.

Looks like the CI has a legitimate complaint. You can fix that, and I can approve.


How to use fullBuild.xlam

Maybe due to my ignorance I can't manage to use any object in FullBuild.xlam when I add it as an addin. Is there somewhere a procedure I can follow to do it? Is it even possible?
Thank you

Post-build you need to:

Ensure all objects are exposed
Reference the xlam in your vba project

Realistically, I wouldn't do it this way, though some people prefer it. I'm still awaiting a means to build this automatically on github which will at least make only step 2 required.


DP schema enumeration order lost

When mapping vocabularies from DP schemas the ordering is lost.
E.g. ranks in NameUsage ColDP

Thanks.
Should we preserve the order from the source vocabulary? Or sort alphabetically?

I suppose we should just preserve the initial order

please preserve the original order

deployed


Store App context with crash to send on next app start

Problem Statement
Currently the app start time can be after when a crash happened as the app context is not saved see screenshots, where the start time matches the time the event was received (CEST = Z+2h).

Solution Brainstorm
Having this information stored with the rest of the scope and minidump will keep information consistent. Somehow related to: https://github.com/getsentry/sentry-electron/issues/679

Is this a native crash in the main process?

Main crash, here is a simple repro:
electron.zip
just add DSN in main.js, npm install and run.

Closed via #683

@timfish, thanks for the fix, I want to verify it to see if it still repros, but I tried npm install git@github.com:getsentry/sentry-electron.git#85f82bc --save with no luck. could you tell me how to do it? since sentry might not release 4.9.0 too soon.
(I am this issue original reporter.

The SDK cannot be installed via git because it uses TypeScript and needs to be built first. I'll try and get it released on npm today!

When this issue closes, v4.9.0 will have been published to npm.


Task - Geographic Area Lookup: Total number display

Feature or enhancement
Would like to have a TOTAL number of results displayed from ADM1 level down, i.e., if there were 10 results for a given geographic area. Theoretically, there would be one result per country.
@bpescador, @cgrinter, @rich-keller
Location
Task - Geographic Area Lookup
Screenshot, napkin sketch of interface, or conceptual description
No response
Your role
No response

See coments on the nature of the build of the table at #3252.


Revert "Only report edge plugin health if sync is enabled."

This reverts commit 7a6f2ee154d6905272435fcb0a71d54b14ddc19c.
It was a mistake to omit the "sync" health status when the edge plugin isn't enabled. The plugin can be enabled and disabled dynamically through discovery calls and it's better to always include its status in the health service.
I originally made the change because it looked like the kubernetes startup probe was getting back NOT_SERVING responses, but I tested it again with the change in this PR and the overall pod health is reported as healthy even if the "sync" service's status is NOT_SERVING.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 28.405%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/app/topaz.go
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>6:
0.006%

Covered Lines:
1320

Relevant Lines:
4647

💛  - Coveralls


Job Board

Job posting example YAML
---
title: [string]
company: [string]
description: [string]
remote: [true|false]
location: [string]
apply: [string]
date: [datetime]
status: [open|closed]
---
[Full job listing goes here with markdown support]


Clean up _non_metered functions

In auth there are a few functions (e.g. to_xdr and to_xdr_non_metered) that have largely the same logic. Consider consolidating them with the _non_metered version calliing the other one with budget.with_free_budget(|_| {...}).

Resolved by #1123


UFO sources?

Cool project!
I wonder: thoughts on UFO sources, instead of only TTF?

That's understandable. When/if you finish it, it's no problem for me to include w/the MFEK distribution. I'd call it MFEKtruetype but of course prominently display the source, that's just for internal consistency with the other tools. My only real requirements are that it be ① really useful; ② not reliant on one specific font editor or another; ③ Apache-2.0 compatible; and ④ cool.

(And if you never get to it, I may end up doing it or hiring someone to do it. :-) )

Oh, I'll get to it. It came up a while ago in the Typedrawers forum, and it's been on the to-do list for a while.
I wouldn't call it cool, though.

Really? Why not? It's really cool!

That makes total sense to me!

A progress report. (I'm not putting this in the repository until it's complete.)
Just as you can begin a project by opening a .ttf font, so you can now begin one by opening a UFO. After that, ygt stores its source in the UFO data directory instead of in a separate .yaml file. (The data directory was new in UFO 3, so the UFO needs to be at least that version to work properly.)
There are two next steps. One (the easy one) is to export compiled TrueType instructions to the UFO (instead of generating a new TTF). I say it's easy because UFO 3 prescribes places to put all instruction-related data, and fontTools provides methods for storing it.
The second step is "save as," necessary for moving back and forth between free-standing ygt source and ygt source stashed inside the UFO. This is more complicated because, while reading a UFO and afterwards generating a .ttf is easy, I can't find a tool for going from .ttf to .ufo without calling an external program, which I don't particularly want to do.
So I am thinking of saying that I'm not in the business of generating UFOs. If you have a free-standing .yaml file and want to "save as" a UFO, you'll need to select an existing UFO (version 3 or higher) to save in.
I'm still hoping to find a suitable tool for saving a UFO, but this is the state of things right now.

This feature is complete but subject to revision and debugging.
The behavior of ygt is determined by the extension of the source file.
If you begin a project by opening a .ttf file, the source is saved as a .yaml file and "Export" produces a hinted .ttf font.
if you begin a project by opening a .ufo, the source is saved in the data directory of the UFO and "Export" writes hints and associated data to the appropriate places in the UFO.
"Save As" lets you move back and forth between these two modes. I'm not sure I'm completely satisfied with the behavior of "Save As" yet: I might have to add a separate menu item File-->Save to UFO, since the behavior of "Save As" is different depending on whether you're writing a new .yaml file or updating an existing UFO.
An important point:
Ygt is happy to hint a font with CFF outlines, even though CFF fonts can't be hinted with TrueType hints. If you do this, you should hide off-curve points display and record points as coordinates (View-->Point coordinates), since the point indices of the generated font will be different from those displayed by ygt, and the off-curve points will be different as well. Further, before you export TrueType hints to a UFO, you should make sure that it contains quadratic, not cubic outlines: otherwise the point indices will be catastrophically wrong.
You can change a UFO's cff to quadratic outlines by running cu2qu on the UFO.
Finally I should say that I have no idea whether any program can make use of hints embedded in a UFO3, though it would be easy to do so.

The UFO-handling version of ygt is in the repository now.

Hi Peter, when working with UFO 3 sources, I guess it’d be useful to somehow be able to specify which layer to take the outline data from. That way, one could store cubic outlines in the foreground layer, but have a quadratic outline elsewhere in a background layer, ready to be hinted.

@verbosus: Good idea. I'll look into it.

So far I'm not finding a lot of facilities in the standard UFO tools for dealing with layers: ufoLib2 knows about layers, but it seems there's not much in fontTools.ufoLib or ufo2ft (which converts a UFO to a fontTools TTFont.
What are you using to manage layers in your UFOs?
BTW, you can base your hints on the on-curve points in a CFF font. Just set up Ygt to use coordinates instead of indices as point identifiers and turn off display of off-curve points.
I haven't done a lot of experimenting with UFOs yet, and I've still got a good bit to learn.

With fonttools.ufoLib you can get a list of all layer names and use that to get a GlyphSet with a specific layerName.
defcon is also aware of layers, as is ufoLib2 as you mentioned. The latter two are requirements of ufo2ft. While the layer support in ufo2ft is not entirely comprehensive, I’ve been landing a few patches that aim to improve that, which is then reflected in fontmake. For instance, it is now possible to build static instances from a UFO source specifying a custom layer—though I’m still working on a patch for the variable font path.


Noteskin Error

Describe your bug here.
i've made a Bar Noteskin and the game ran correctly, but it crashed moments later, what can i do

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
just edited the "list.txt" on the noteskins assets

NEVER EDIT THE LIST.TXT FOR NOTESKINS!


Formalize collaboration process

Describe the issue
Differentiate between community of practice vs ability to pathfind
Own the establishment of a solid MVP foundation before opening up to community of practice
Definition of done
Get air cover to pull in people onto our team rather than shipping work out to other teams

Notes from discussion with Todd.  Requires refinement in order to estimate


Update image in CatalogSource

Noticed at other places in task repos we are using a different image.
So updating the image in CatalogSource to be consistent with other task repositories

@vdemeester
@divyansh42 mentioned that he will use 4.15 now because if he change to 4.16 he has to change task config and ci config
so when he add changes to multiple OCP version support he will make 4.15 as optional and 4.16 as default


Trying to use strict BEM but scss-lint complains

.block-name__element-name_modifier-name doesnt work ..i am bit confused
.block-name__element-name -> works
.block-name_modifier-name -> works
.block-name__element-name_modifier-name -> doesnt
am i missing something ? thanks for your time

Have you tried .block-name__element-name--modifier-name? That's how I understand the syntax to work.

doesnt work either but according to https://en.bem.info the syntax i tried is correct .

.block-name__element-name_modifier-name

I am not familiar enough with the BEM methodology to weigh in here. Would love a pull request fixing this issue if there is indeed a problem. Thanks!

@sds sorry i don't know how do fix it via a pull request, i just stumbled across it.

There are two syntax for the bem convention, one of which is the @abea reply.
So, the lint should support both.

Keen to bump this issue.
Currently this is breaking for me when using the strict_BEM convention. Writing BEM but the linter is returning warnings. Seems to be when using the double hyphen for modifier.
See examples below:
.box__heading works
.box--large warning
.box__heading--large warning

@ryanlloydfrench Judging by your example, I believe you should switch to hyphenated_BEM (which is that other syntax @widoz is referring to). According to "strict" BEM, modifiers use a single underscore.

Ok, so I am also coming across the issue when using the strict_BEM convention. The following fails the linting:
.nav__link_active {
  color: #fff;
}

The above is a valid BEM strict syntax, but scss-lint raises a warning.
The tests also seem to support the above as being invalid, but it is valid.


[nrf52811] add support for nRF52811 platform

This PR adds support for a new platform - nRF52811.
Additionally a small re-factorization of third party Nordic Semiconductor components have been made. It makes cleaner separation and also fixes autotools problem with make distclean. Funny fact, when two platforms include the same files (e.g. in third_party) in Makefile.am, make distclean removes *.Po files from them, when cleaning first platform and then it fails since it tries to remove the same *.Po files for the second platform.

@jwhui i have updated this PR to use new Travis test approach from #3562. So now it's ready to be reviewed and merged.

@LuDuda, thanks!


arrayOfAllBlocks / mapOfAllBlocks - a user function that combines blocks with the same type

Say I have
block1 "hello" {
     message = "hi"
}

block1 "goodbye" {
    message = "peace"
}

we could have something that automatically converts these to an array without having to specify them
arrayOfAllBlocks or mapOfAllBlocks is an example of the function we could make - they do not need to be called it exactly
gen testfile {
    data = {
        messagesArray = arrayOfAllBlocks(block1)
        messagesMap = mapOfAllBlocks(block1)
    }
}

which would generate:
messagesArray: [{ message: "hi" }, { message: "peace" }]
messagesMap: { hello: { message: "hi" }, goodbye: { message: "peace" } }

may need more thought: I assumed that arrayOfAllBlocks ignores the labels of the block, maybe we should consider them or make them optional?

ended up looking like this:

block1 "hello" {
     message = "hi"
}

block1 "goodbye" {
    message = "peace"
}

gen testfile {
    data = {
        messagesArray = alloflist("block1")
        messagesMap = allof("block1")
    }
}


SB T2 Fixing Live test

We are accepting session and doing other things after that, So even before we receoveMessage, the session lock expire. Fixing this.


puppet: Use pip to install wal-e dependencies.

@gnprice what do you think of using pip for this case (version pinning ftw)? The blocker here is testing. I'd be grateful if you could check this PR is working as expected via zulipstaging.com

Thanks @rht for taking a look at this!
As you see, a change like this doesn't involve much code; the bulk of the work is in getting it right, mostly by testing it out in production-like environments (for each of our supported OS releases), and poking around in the environment when one attempt doesn't work. So I think it makes the most sense to leave this for someone to tackle by doing that. Closing this PR.


[Cards] Centered Cards Not Functioning

This simple HTML,

fdsfsdf

fdsfsdf

fdsfsdf

should center the cards and then stack them. However, on IE(10) and Safari on iOS and possibly other browsers as well, they are left-aligned all the time, even when stacked.
https://github.com/Semantic-Org/Semantic-UI-Meteor/issues/26

Hi, centered column is functioning as expected. That will just center a columns content in its row.
The adjusted html

fdsfsdf

fdsfsdf

fdsfsdf

However is not functioning correctly


question/bug: cappa.Arg( ) value_name format

It's working, but I found the problem, when I set value_name='<OPENAPI JSONFILE / URL, PROJECT>',
it gets parsed as <OPENAPI-JSONFILE-/-URL,-PROJECT>, instead of the raw string.

mmm. Yea so I noticed the potential for confusion if you set value_name to something like 'foo bar baz`, which would render ambiguously and look like three separate args.
I suppose I can remove that dash casing and leave it up to the user to choose their value intelligently


[wpantund] Add ff05::c multicast address to network configuration

wpanctl insertprop IPv6:MulticastAddresses ff05::c

This will also need an updated wpantund version since the current one does not have implemented IPv6:MulticastAddresses key.

Resolved by #16


Delivering an event to multiple Process Managers

This PR extracts common functionality of having functions for generating multiple IDs into abstract base called EventDispatchingRepository.
ProcessManagerRepository and ProjectionRepository inhering from this class.

Current coverage is 83.11% (diff: 95.52%)

Merging #256 into master will increase coverage by <.01%

@@             master       #256   diff @@
==========================================
  Files           165        168     +3
  Lines          6098       6105     +7
  Methods           0          0
  Messages          0          0
  Branches        615        613     -2
==========================================
+ Hits           5068       5074     +6
- Misses          918        919     +1
  Partials        112        112

Powered by Codecov. Last update 4f1bd38...397f8bb

@armiol PTAL

@armiol I've reverted the name back to IdSetFunctions. Please also note that in the last commit I'm hiding interfaces and classes for working with event fields.
The only class that remains visible is GetTargetIdFromCommand. It is visible because it's used by several packages. The class is annotated @Internal.


Double check the role exists for everything

This was an issue in the Ruby implementation -- it never cared enough to check for permissions, nor if the channel even existed.
The current solution will be done:

Upon addition to a server, the default log names will be used.
Log will be off by default.
#mod-log will be optional.
The server owner will be DMed regarding results, or sent information about configuration of the log.

Upon failure to access log:

Owner sent DM
Log type disabled.

This is being taken care of in 3 parts:

What I'm calling FirstJoinAnalyzer, which'll determine logs, and state findings to the owner
The config command, which'll act similar as before but without any role configuration (unless it's decided they're needed again)
The actual event listeners for ban, unban and etc. I'm researching looking into audit logs to see if I can determine a kick from that.

pretty much finished


Add Prompt primitive

Adds a prompt primitive to trigger a prompt NAK or prompt KeepAlive PDU from the mission specific User process.
I tried to be maximally flexible about when prompts could be generated in the Send, but it just ended up being the highest priority PDU for the Send transaction. At first I was not totally sold on the idea, but it mirrors the Recv's handling of prompt so maybe it is okay. I also figure, if the mission is trying to send a Prompt it should be handle ASAP.
thoughts @xpromache ?
closes #34

Codecov Report
Patch coverage: 90.47% and project coverage change: -0.17 :warning:

Comparison is base (1fc1277) 95.80% compared to head (9aa567a) 95.64%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #41      +/-   ##
==========================================
- Coverage   95.80%   95.64%   -0.17%
==========================================
  Files          17       17
  Lines        7581     7666      +85
==========================================
+ Hits         7263     7332      +69
- Misses        318      334      +16

Impacted Files
Coverage Δ

cfdp-core/src/daemon.rs
72.10% <0.00%> (-1.42%)
:arrow_down:

cfdp-core/src/transaction/send.rs
97.48% <97.43%> (-0.01%)
:arrow_down:

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@xpromache I'd like to get this one in for a misson I'm working on, but if you have a better idea of how to implement it I'd love to hear it.

I looked through the standard and your implementation makes sense.
I agree that if the user triggers a Prompt PDU it has to be the highest priority; if there is a file being sent, you don't want to wait for the EOF, it doesn't make any sense.


Enable depedency caching for travis

Would be interested to know why maven dependencies haven't been cached on Travis. Thank you.

Thanks for merging my pull request.
Any reasons why bundler caching wasn't enabled before, please? It was available since December 2014.
Thank you.

Thanks for merging my pull request.
Any reasons why dependency caching wasn't enabled before, please? It was available since December 2014.
Thank you.


Matlab

The matlab code so far.
Known issues:

The Gui lacks error handling and could use more comments. The delete and reload buttons (both added on a whim) also do nothing so far. Open for suggestions as to the contents of the "infobox" in the top-right

There's currently no way to clear the cache, except for manually removing files.

This looks good! I will go ahead and merge, I left a couple of comments to review.


Improve description of language/feature diagram

@luckyLukac

@luckyLukac , what is the status of this issue?


Bug: bubble line not working in latest version

Self Checks

[ X] I'm using the latest lualine.
[ X] I didn't find the issue in exsisting issues or prs.

Expected / Actual behaviour
Good morning,
I'm having some issues while theming lualine. The highlights are not applied correctly.

As you see there some blue hightlight that shoudn't be there on the side. I coudn't figure if it's something wrong with my config, the theme or lualine but skimming throught the theme repository and my config I coudn't find anything that could cause this issue so I'm reporting here where you might be able to help more.
Minimal config to reproduce the issue
Here is my configuration (theme from projekt0n/github-nvim-theme:
colorscheme github_dimmed

lua << EOF
local github_colors = require('github-theme.colors').setup({ theme_style = "dimmed" })
local github_util = require('github-theme.util')
local github_theme = require('lualine.themes.github_dimmed')

local tint_lualine_group = function(color)
local group = {
    a = {bg = color, fg = github_colors.bg},
    b = {bg = github_util.darken(color, 0.2), fg = github_util.lighten(color, 0.2)},
    c = { bg = github_colors.bg, fg = github_util.lighten(color, 0.4, github_colors.fg)}
    }
return group
end

github_theme.normal     = tint_lualine_group(github_colors.red)
github_theme.insert     = tint_lualine_group(github_colors.green)
github_theme.command    = tint_lualine_group(github_colors.bright_magenta)
github_theme.visual     = tint_lualine_group(github_colors.blue)
github_theme.replace    = tint_lualine_group(github_colors.yellow)
github_theme.inactive.a = github_theme.normal.b
github_theme.inactive.c = {bg = github_colors.bg}

require'lualine'.setup {
    options = {
        icons_enabled = true,
        theme = github_theme,
        section_separators = { left = '', right = ''},
        component_separators = { left = '|', right = '|'},
        disabled_filetypes = {},
        always_divide_middle = true,
        },
    sections = {
        lualine_a = {{ 'mode', separator = { left = '', right = ''}}},
        lualine_b = {'filename', 'diff',
        {'diagnostics', sources={'nvim_diagnostic', 'coc'}}},
        lualine_c = {},
        lualine_x = {},
        lualine_y = {'encoding', 'fileformat', 'filetype'},
        lualine_z = {{ 'location', separator = { left = '', right = ''}}},
        },
    inactive_sections = {
        lualine_a = {{ 'filename', separator = { left = '', right = ''}}},
        lualine_b = {},
        lualine_c = {},
        lualine_x = {},
        lualine_y = {},
        lualine_z = {{ 'location', separator = { left = '', right = ''}}},
        },
    tabline = {},
    extensions = {coc_extension}
    }
EOF

try adding
hi! link StatusLine Normal

somewhere between after setting colorscheme and before calling lualine setup.

Indeed it works. I should probably dig deeper and fix the theme then but I don't have time for now.

Indeed it works. I should probably dig deeper and fix the theme then but I don't have time for now.

Hi! I have similar issue. Have you solved it?


IMPRO-1331: Recursive filter modified to use symmetric padding

This PR modifies the pad_cube_with_halo function to accept any np.pad method. It then uses this functionality in the RecursiveFilter plugin to implement the padding using the symmetric method.

Unit tests are updated to reflect the change in values.
Additional tests added to cover a few more pad options (non-exhaustive).
KGO updated and plotted in associated notebook.

Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

 Here is an overview of what got changed by this pull request:

Clones added
============
- improver_tests/utilities/test_pad_spatial.py  8

Clones removed
==============
+ improver_tests/nbhood/recursive_filter/test_RecursiveFilter.py  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Clones added
============
- improver_tests/utilities/test_pad_spatial.py  8

Clones removed
==============
+ improver_tests/nbhood/recursive_filter/test_RecursiveFilter.py  -2

See the complete overview on Codacy


Recursively binding Multipart form data to complex nested request DTOs

I have this request:
public record Request
{
    public IList<MediaWithDescription> Medias { get; set; }

    public class MediaWithDescription
    {
        public IFormFile Media { get; set; }
        public string? Description { get; set; }
    }
}

Normally, I would add [FromForm] to get this correctly in an ASP.Net Core MVC project. It works fine.
For example:
    [HttpPost("test-upload/")]
    [TranslateResultToActionResult]
    [AllowAnonymous]
    [Consumes("multipart/form-data")]
    public async Task<Result> Test([FromForm] Request request)
    {
        return Result.Success();
    }

However, when using FastEndpoints, it doesn't work—Medias is always null

public class UploadMediaPet : Endpoint<Request, Results<NoContent, ProblemHttpResult>>
{
    public override void Configure()
    {
        Post("/api/pet/{PetId}/medias/");
        AllowFileUploads();
    }

    public override Task<Results<NoContent, ProblemHttpResult>> ExecuteAsync([FromForm] Request req, CancellationToken ct)
    {
        return base.ExecuteAsync(req, ct);
    }
}

What should I do in this case? Should I handle it manually?

binding form data to complex nested dtos is currently not supported.
as of now, only the root level can be automatically bound.
going to need the help of the community to implement this one as it's a bit of a complicated task and i don't have the capacity right now to look at it.
basically need to refactor this method to be able to recursively bind deeply nested dtos.

for the time being, you'll have to resort to a regular minimal api endpoint as they seem to support this scenario from .net 8 onwards.

I'll try to implement this when I get the chance.
Thanks for the clarification.

for anyone looking in to this, the multipart form binding code should be able to correctly handle a complex request dto object graph such as the following:
class Book
{
    public string Title { get; set; }
    public IFormFile CoverImage { get; set; }
    public IFormFileCollection SourceFiles { get; set; }
    public Author MainAuthor { get; set; }
    public List<Author> CoAuthors { get; set; }
    public Address PublisherAddress { get; set; }
    public List<Address> RetailerAddresses { get; set; }
}

class Author
{
    public string Name { get; set; }
    public IFormFile ProfileImage { get; set; }
    public IFormFileCollection DocumentFiles { get; set; }
    public Address MainAddress { get; set; }
    public List<Address> OtherAddresses { get; set; }
}

class Address
{
    public string Street { get; set; }
    public IFormFile MainImage { get; set; }
    public IFormFileCollection AlternativeImages { get; set; }
}

added an integration test to work against:
https://github.com/FastEndpoints/FastEndpoints/blob/49e85726c0ff6730bfc6e0b0f2ebbb22c20e5fc7/Tests/IntegrationTests/FastEndpoints/BindingTests/BindingTests.cs#L674-L726

can you give v<IP_ADDRESS>-beta a try?
i've done a crude implementation.
there may be edge cases that i haven't thought of and there's definitely room for improvement.
since my capacity is limited, i'd like the help of the community to optimize that class.

have a look at the integration test and the dto structure to get an idea of how to use.
basically, just like the [FastEndpoints.FromBody] attribute usage, use the [FastEndpoints.FromForm] attribute to trigger binding complex data from forms.

I am testing the v<IP_ADDRESS>-beta and still having issues.
public record Request
{
    [FromForm]
    public List<MediaWithDescription> Medias { get; set; }

    public class MediaWithDescription
    {
        public IFormFile Media { get; set; }
        public string? Description { get; set; }
    }
}

Pretty much the DTO you provided.

![image](https://github.com/user-attachments/assets/ca1a9260-ae1a-4ffa-b627-f307ce280c47)

System.Reflection.TargetParameterCountException: Parameter count mismatch.
at System.Reflection.RuntimeMethodInfo.InvokePropertySetter(Object obj, BindingFlags invokeAttr, Binder binder, Object parameter, CultureInfo culture)
at System.Reflection.RuntimePropertyInfo.SetValue(Object obj, Object value, Object[] index)
at FastEndpoints.ComplexFormBinder.BindPropertiesRecursively(Object obj, IFormCollection form, String prefix)
at FastEndpoints.ComplexFormBinder.Bind(PropCache fromFormProp, Object requestDto, IFormCollection forms)
at FastEndpoints.RequestBinder1.<BindFormValues>g__Execute|27_0(<>c__DisplayClass27_0&)    at FastEndpoints.RequestBinder1.BindFormValues(TRequest req, HttpRequest httpRequest, List1 failures, Boolean dontAutoBindForm)    at FastEndpoints.RequestBinder1.BindAsync(BinderContext ctx, CancellationToken cancellation)

it has to be wrapped in a class like this:
public class Request
{
    [FromForm]
    public FormBody Form { get; set; }

    public class FormBody
    {
        public List<MediaWithDescription> Medias { get; set; }

        public class MediaWithDescription
        {
            public IFormFile Media { get; set; }
            public string? Description { get; set; }
        }
    }
}

v<IP_ADDRESS>-beta will correctly warn you if used incorrectly.

It worked. Awesome

next version of FE ships with a source generator for reflection based methods which should help with this.


shall we add Google's Maven Repository ?

repository is here : https://dl.google.com/dl/android/maven2/index.html

Awesome, can you please make a pull request?


⚠ Avatar API Endpoint has degraded performance

In 379e298, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 4676 ms

Resolved: Avatar API Endpoint performance has improved in 0c68577 after 13 minutes.

Resolved: Avatar API Endpoint performance has improved in 60b7689 after 17 minutes.

Resolved: Avatar API Endpoint performance has improved in 41fdfe5 after 38 minutes.


Mongodb adapter:  E11000 duplicate key error collection: refresh_tokens index: id_1 dup key: { id: null }

Hey :)
Everything was working fine yesterday, but now when I try to login to an existing account, I get this error:
 E11000 duplicate key error collection: tmqsk.refresh_tokens index: id_1 dup key: { id: null }

Singing up a new user works, siging in with the wrong password gives the correct error.
But signing in with the right password does not work.
The error arises when the adapter runs setRefreshToken
setRefreshToken: async (refreshToken, userId) => {
            try {
                const refreshTokenDoc = new RefreshToken({
                    refresh_token: refreshToken,
                    user_id: userId,
                });
                await refreshTokenDoc.save();
            }
            catch (e) {
                console.error(e);
                throw new Error("DATABASE_UPDATE_FAILED");
            }
        },

The error is saying there is a duplicate key error, and the key it is trying to set is null, meaning either the refreshToken value is null, or userId is.
I'm not quite sure what to do, I've not changed how I log in

This SO answer says that one can get a duplicate key error on null if the collection has a unique index on that field id, and a doc in that collection doesn't have that field.
This is true. The docs in my refresh_tokens collection do not have a id field on them

Ahh ok, so I checked the indexes on the collection, and mongoose had auto created these indexes:

Now, I'm not sure why it created that id_1 index on the id field, because that isn't even a field in collection?
But I believe it is because, when connecting to mongoose, the autoIndex property is on by default.
So, turning this off should prevent this from happening in future:
const adapter = (url) => {
    const clientPromise = mongoose.connect(url, { autoIndex: false });
    ...

Should be fixed with #32
adapter-mongodb is now deprecated and changed to adapter-mongoose. Now requires the user to set the model/schema by themselves.
Doc: https://lucia-sveltekit.vercel.app/adapters/mongoose


Show photo clubs per photographer

With the app, a photographer is linked to at least 1 photo club.
On the Who’s Who screen, these photo clubs should be shown for each Photographer.
This could be in sentences like “Vice-chairman and member of Fotogroep Waalre” as already used in the Portfolios screen.
With a clickable link to the images related to the club-photographer combination.

First implementation


Enforce DASA requirements; define a DASA user

Alright, looking at where we are using _getValueFromOAuth('email') some more, I think we should create a new function, something like get_admin_email() which will first check for GA_ADMIN_EMAIL env variable / config setting. GA_ADMIN_EMAIL should be set to the G Suite admin user (not the dasa email address). If GA_ADMIN_EMAIL is not set and dasa is enabled, we should error out with a message. If dasa is not enabled, we go to _getValueFromOAuth.


Apply fails to replace a node group (for adding a launch template) with "NodeGroup already exists with name internal-t2_medium and cluster name development-pod01"

I have issues
I'm submitting a...

[x] bug report
[ ] feature request
[ ] support request - read the FAQ first!
[x] kudos, thank you, warm fuzzy

What is the current behavior?
Node group replacement fails with "Error: error creating EKS Node Group (clustername:nodegroupname): ResourceInUseException: NodeGroup already exists with name nodegroupname and cluster name clustername"
Terraform plan indicates that the node groups need to be replaced:
Resource actions are indicated with the following symbols:
+ create
+/- create replacement and then destroy

...

  # module.eks.module.eks.module.node_groups.aws_eks_node_group.workers["1"] must be replaced
+/- resource "aws_eks_node_group" "workers" {
...

  # module.eks.module.eks.module.node_groups.random_pet.node_groups["1"] must be replaced
+/- resource "random_pet" "node_groups" {

Application of the plan fails
module.eks.module.eks.module.node_groups.random_pet.node_groups["1"]: Creating...
module.eks.module.eks.module.node_groups.random_pet.node_groups["1"]: Creation complete after 0s [id=immortal-parrot]
module.eks.module.eks.module.node_groups.aws_eks_node_group.workers["1"]: Creating...

Error: error creating EKS Node Group (clustername:nodegroupname): ResourceInUseException: NodeGroup already exists with name nodegroupname and cluster name clustername
{
  RespMetadata: {
    StatusCode: 409,
    RequestID: "abc"
  },
  ClusterName: "clustername",
  Message_: "NodeGroup already exists with name nodegroupname and cluster name clustername",
  NodegroupName: "nodegroupname"
}

If this is a bug, how to reproduce? Please include a code sample if relevant.
I added a launch template to the definition of an existing node group by adding this:
      launch_template_id      = aws_launch_template.default.id
      launch_template_version = aws_launch_template.default.default_version

What's the expected behavior?
The node group is replaced and the new node group has a launch template.
Are you able to fix this problem and submit a PR? Link here if you have already.
Environment details

Affected module version:
terraform-aws-modules/eks/aws: 13.2.1
kubernetes/1.13.3
local/2.0.0
null/3.0.0
template/2.2.0
random/3.0.1
aws/3.24.1

OS: amazon linux 2

Terraform version: 0.14.2

Any other relevant info

This might be related to #1152

This might be related to #1152


[codespaces] Add libreadline-dev for building ruby

rbenv fails to install rubies without libreadline-dev.

Looks good to me!
Could you add this change to https://github.com/microsoft/vscode-dev-containers/blob/master/script-library/ruby-debian.sh - this is the "centralized" source for this script. The file you edited is just a copy of this file (and your change to this mirror would eventually get overwritten by CI)

@joshspicer thanks! It took me another try to find the right ruby-debian.sh. Do I need to worry about those copies being overwritten, as in revert those changes? Or should it be alright to let CI handle it?

This looks good now, thanks!
No need to revert, the scripts in container's directory will get overwritten regardless of its content.


Error on non existing map[float64]float64 json key

Hey @goccy, thanks for the nice library 👍🏻
I have an issue with parsing the map[float64]float64 fields.
Given the following example (simplified):
package main

import (
	"github.com/goccy/go-json"
)

type Foo struct {
	Bar map[float64]float64 `json:"bar,omitempty"`
}

func main() {
	data := []byte(`{"name":"test"}`)

	nc := &Foo{}

	err := json.Unmarshal(data, nc)
	if err != nil {
		panic(err)
	}
}

error: json: cannot unmarshal object into Go struct field Foo.Bar of type float64.
In this case, there is no such key bar in the incoming JSON. I expect, that the structure will have just nil Bar field w/o errors, instead, in the case of map[float64]float64 I get an error.
I guess, that happens because there is no case for the *decoder.floatDecoder in the internal/decoder/compile.go compileMapKey function.

Thank you for reporting ! I'll fix this problem with #321


Limit the number of task outputs persisted

We made an adjustment to save task outputs to the persistent state / state registry along with the output of operations. This was to solve an immediate need - have the ability to reference the contract address of the last origination.
Although I'd treat this as an experimental feature that could be dropped at some point, until the state API has been fully realized this is a good feature to offer in the midterm.
However, all task outputs are persisted, meaning that the persistent state can grow large and quickly. What would be best is to add a configurable limit per task in terms of how many outputs are persisted.
I think the last 5 outputs is a reasonable default.
In the config file, the end-developer can adjust the default:
"state": {
  "tasks": {
     "limit": 5,
     "originate": 10
  }
}

The above configuration would ensure that the last 5 outputs of any task are persisted, with the originate increased to the last 10 outputs.

Currently seeing that plugins will overwrite other plugins information in the environment-state.json file (default configuration will give you a development-state.json file). Steps to reproduce:

Initialize a taqueria project
Install the ligo, taquito and flextesa plugins
Start a sandbox and copy a ligo contract into the project contracts folder
Compile the contract a few times
Open up the development-state.json file and see the compilation records multiple compile executions
Open up the config.json and add storage for the ligo contract you have
Deploy the contracts to the local sandbox
Open up the development-state.json file and see that the compile tasks have been erased


Distribution error

Environment
Please provide the following information about your environment:

Development OS: Mac, macOS Sierra - version 10.12.6
React Native : 0.57.7
Viro react: 2.13.0
Android Studio 2.3.3
Testbed Device(s): Nokia 8.1

Description
Ran command "./gradlew assembleGvrRelease" but received the below error messages:
Starting a Gradle Daemon (subsequent builds will be faster)

Task :app:mergeGvrReleaseResources FAILED
[string/app_name] /Users/keneseu/Desktop/projects/virosample/android/app/src/main/res/values/strings.xml        [string/app_name] /Users/keneseu/Desktop/projects/virosample/android/app/build/generated/res/resValues/gvr/release/values/generated.xml: Error: Duplicate resources

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:mergeGvrReleaseResources'.

[string/app_name] /Users/keneseu/Desktop/projects/virosample/android/app/src/main/res/values/strings.xml      [string/app_name] /Users/keneseu/Desktop/projects/virosample/android/app/build/generated/res/resValues/gvr/release/values/generated.xml: Error: Duplicate resources

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 16s
10 actionable tasks: 1 executed, 9 up-to-date
kindly note the content for strings.xml within "resource" tag is :
<string name="app_name">virosample</string>

While the content for generated xml is:

<!-- Automatically generated file. DO NOT MODIFY -->

<!-- Values from product flavor: gvr -->
<string name="app_name" translatable="false">ViroSample-gvr</string>

Not sure if it is due to adding product flavors to build.gradle:
  flavorDimensions "version"
    productFlavors {
        ar {
            resValue 'string', 'app_name', 'ViroSample-ar'
            buildConfigField 'String', 'VR_PLATFORM', '"GVR"'
        }
        gvr {
            resValue 'string', 'app_name', 'ViroSample-gvr'
            buildConfigField 'String', 'VR_PLATFORM', '"GVR"'
        }
       ovr {
            resValue 'string', 'app_name', 'ViroSample-ovr'
            applicationIdSuffix '.ovr'
            buildConfigField 'String', 'VR_PLATFORM', '"OVR_MOBILE"'
        }
    }

Would appreciate your help. Thank you!

remove the virosample and rebuild. The error went off with that


complete to 2-4

using cpp_(:з」∠)_

Compared with soup teacher, I still have a lot more to work on.

you are the best  big big ! believe yourself. you will achieve your dream.


Implement real FFT

This isn't ready to merge yet, but it's in a previewable state.
I used the RealFFT crate as a starting point for the new structs, and I was able to make a few optimizations to bring down the overhead by about a third.
Todo:

[ ] Find a paper or something for odd-sized real FFTs
[ ] Implement an AVX version of c2r and r2c
[ ] Clean up unit tests
[ ] Write some integration tests
[ ] Benchmark and write some c2r and r2c butterflies, so that using r2c is always faster than or equal to a normal FFT
[ ] Decide what to do with the planner. Should this stuff go in the current planner? Should I make a new planner that wraps the current one? If we made a new planner, it would make it harder to integrate AVX. But I'm worried about a slippery slope where we bloat the planner struct until it's this giant monolith.

cc: #28

cc: #28

Very nice! For the first point, I have tried and failed. The only thing I have found is the implementation in fftw, but it's very hard to read. And they don't provide any references (that I have been able to find at least).
When you look under the hood of most other implementations, they turn out to cheat and just use a C2C transform of the same length.
For the planner, I also think adding more and more things in the current planner is risky. But the stuff added in this PR are reasonable, just reusing the existing things and wraps them up as a R2C or C2R. But that simplicity disappears if there are special R2C butterflies.. Probably needs some more thinking.

Very nice! For the first point, I have tried and failed. The only thing I have found is the implementation in fftw, but it's very hard to read. And they don't provide any references (that I have been able to find at least).
When you look under the hood of most other implementations, they turn out to cheat and just use a C2C transform of the same length.
For the planner, I also think adding more and more things in the current planner is risky. But the stuff added in this PR are reasonable, just reusing the existing things and wraps them up as a R2C or C2R. But that simplicity disappears if there are special R2C butterflies.. Probably needs some more thinking.

This branch has conflicts that must be resolved

Given what I've learned about this since opening the PR (IE that implementing odd R2C essentially requires fully rewriting the FFT stack from the ground up), I don't think I'll ever be doing it this way, so I'm closing this.
If I ever fully implement r2c etc FFTs, it'll be in another crate,


WIP: Use "buffer" switch type for non-configurable switches (e.g. ppips).

This is part 1 of #1019

@litghost: I am trying to get this to work.
I have locally fixed VTR to accepct Cinternal on buffer switches, but it fails now with the following error:
Type: Routing
File: /home/alessandro/projects/symbiflow/vtr-symbiflow/vpr/src/route/check_rr_graph.cpp
Line: 546
Message: Non-configurable BUFFER type switch must have only one driver.  Actual fan-in was 2 (expected 1).
  Possible cause is complex block output pins connecting to:
    RR node: 12223 type: CHANX location: (7,8) <-> (9,8) track: 826 len: 2 longline: 0 seg_type: OUTPINFEED dir: BI_DIR capacity: 1

This because, apparently, buffer type switches must be unidirectional for VPR.
I am unsure whether this is something to be solved in this PR or in VPR.

@litghost: I am trying to get this to work.
I have locally fixed VTR to accepct Cinternal on buffer switches, but it fails now with the following error:
Type: Routing
File: /home/alessandro/projects/symbiflow/vtr-symbiflow/vpr/src/route/check_rr_graph.cpp
Line: 546
Message: Non-configurable BUFFER type switch must have only one driver.  Actual fan-in was 2 (expected 1).
  Possible cause is complex block output pins connecting to:
    RR node: 12223 type: CHANX location: (7,8) <-> (9,8) track: 826 len: 2 longline: 0 seg_type: OUTPINFEED dir: BI_DIR capacity: 1

This because, apparently, buffer type switches must be unidirectional for VPR.
I am unsure whether this is something to be solved in this PR or in VPR.

Ah, I think I remember this now.  So this is a bug (I believe) in check rr graph.  Basically, the fan in edge in this case is part of a short.
So the structure looks like:
CHAN -> buffer -> CHAN <- short -> CHAN

Check route has no valid solution to this configuration.  A short requires bidirectional edges, and buffer requires fan in = 1.  Double check that I'm remembering this correctly, and file a bug upstream.

@acomodi In the short term, just disable check rr route when generating the lookahead.  We already disable it during P&R to save time.

@litghost All clear, thanks.


Print position of style-invalid error

I see what error happened but not where it happened. (wasting a lot of time figuring that out)
flatpak run --command=flatpak-builder-lint org.flatpak.Builder --exceptions repo repo
 in dir /srv/buildbot/worker/build-x86_64/build (timeout 1200 secs)
 watching logfiles {}
 argv: b'flatpak run --command=flatpak-builder-lint org.flatpak.Builder --exceptions repo repo'
 using PTY: False
{
    "errors": [
        "appstream-failed-validation"
    ],
    "warnings": [
        "appstream-screenshot-missing-caption"
    ],
    "appstream": [
        "style-invalid         : Content before <ul> is too short [13], at least 20 characters required",
        "style-invalid         : Content before <ul> is too short [19], at least 20 characters required"
    ]
}
program finished with exit code 1
elapsedTime=52.389721

appstream-glib does not mention the line number. That is just a formatted output of what you'll get with https://docs.flathub.org/docs/for-app-authors/appdata-guidelines/#use-flathubs-appstream-util
appstreamcli however does, so when Flathub eventually switches, it can be shown.


As a manager/reviewer, clear and easy way to view review allocations and completions (and what's not done/covered)

linking to #320 are the same


Traefik adds huge overhead time to requests

I am using Traefik as the ingress controller for my Kubernetes setup. I decided to run some performance test for my application but I faced a huge difference when I sent the requests through the Traefik.
The test consists of sending 10K request in parallel and the application returned the compiled result and based on the logs of my application it needs around 5 milliseconds to process one request. The results for the performance test are as below:

Native application:
Execution time in milliseconds: 61062
Application on Kubernetes (without going through Traefik and just using its IP):
Execution time in milliseconds: 62337
Application on Kubernetes and using Traefik:
Execution time in milliseconds: 159499

My question is why this huge difference exists and is there a way to reduce it (except adding more replicas).
I am using these yaml files for setting up Traefik:
https://raw.githubusercontent.com/containous/traefik/v1.7/examples/k8s/traefik-rbac.yaml
https://raw.githubusercontent.com/containous/traefik/v1.7/examples/k8s/traefik-ds.yaml

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Traefik!
We dedicate the issue tracker to bug reports and feature requests only. My advanced AI tells me this one is neither of them.
I encourage your to seek community support: join our Slack workspace and reach out to us on the #support channel.
Another option is to use Stack Overflow and tag your question traefik.
You can of course double check Traefik's documentation :sweat_smile:

Just as an update: I tried Ambassador as my ingress in kubernetes and its result was much better than Traefik and very close to using native and IP of the container (63394 milliseconds).


attrs class object shared between processes

I would like to share a class object (or even a plain dict) between processes.
I came across a project called datafiles on PyPi, which uses python 3.7 dataclasses and transparently stores/updates the file and registered objects if anything changes.
https://pypi.org/project/datafiles/
Is there anything similar for attrs, where I can transparently sync shared attrs object between processes?

Given how similar attrs and dataclases are internally, I would assume it shouldn't be too much work to make datafile work with both?

I suspect this feature would be out of scope for Attrs proper… I suggest you look into adding support for Attrs to datafiles or something similar, as @hynek suggests, and then let us know so we can add the result to the wiki.

closing since this does not belong in attrs proper as stated above


[Feature] Small YOLO

Prerequisites
Please answer the following question for yourself before submitting an issue.

[x] I checked to make sure that this feature has not been requested already.

1. The entire URL of the file you are using
https://github.com/tensorflow/models/tree/master/official/vision/beta
2. Describe the feature you request
A clear and concise description of what you want to happen.
Small version of decoder used in tiny yolov4.
3. Additional context
Add any other context about the feature request here.
4. Are you willing to contribute it? (Yes or No)
Yes.

Bumping to not urgent due to #76, which reduces model size drastically.

Closing this, alternate issue #76 solves the underlying problem. Feel free to open if size becomes a problem again.


Adds kivy namespaces.

Still need to update docs to give simpler examples.

In your first example, what if i open 2 popup, there would be 2 widget for knspace.keyboard, or only the last one would work? Or does it crash?

If you create two widgets with the same name, but you have only one namespace, then the last one will be the one stored in the namespace. So when you do knspace.my_widget, it'll point to the last. There shouldn't be any crashes. If you need to have multiple widgets with the same name you should fork, as in the second example.

Docs has been updated to be easy to understand and go from simple to more advanced. This should clarify things. Please read that to understand what the pr is doing.
Also super comprehensive tests have been added and bugs found have been fixed.

I'm open to changing the name to kname, knsname, or kns_name or whatever is appropriate (I think one of the first two probably).

+1 for knsname.

Renamed name to knsname.

I like the api and the docs are well organized and clear. fatty +1

+1 for merging

\o/

@matham nice work !


Remove unused refs

I prefer keeping them here, I think there is a reason why I added them. The legacy .NET Framework have weird behavior concerning bindings.
I plan to move this project in .NET Core 3.0 when I can, so I will be able to cleanup the csproj.


Automated Package Upgrade

:tada: This PR is included in version 26.0.138 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Citation

Sorry if I missed it, but how would you prefer nflows to be cited?
A bibtex entry for example would be useful. :)

Good suggestion -- thanks! I've now added info on citing nflows to README.md.

Thanks! Note that you're missing an end-brace in the current version, under title, though. :)

Thanks! Note that you're missing an end-brace in the current version, under title, though. :)

Oops, good catch — fixed! :-) Thanks!


feat: installed oauth

close: #40
Abstract the Authenticator and add InstalledFlowAuthenticator.

@lquerel
The cargo test in CI is failing, but after setting the necessary environment variables locally and running the test, the cargo test succeeds.
Looking at the CI error message (↓) , it appears that there is a problem with the service account key. Could you please take a look at the service account key?
thread 'dataset::test::test' panicked at 'expecting a valid key: InvalidServiceAccountKey(Custom { kind: InvalidData, error: "Bad service account key: EOF while parsing a value at line 2 column 0" })', src/lib.rs:70:14

Thanks for this generalization of the authenticator and the support of InstalledFlowAuthenticator.
Give me a few days to approve these changes and see what is wrong with the account key service.


add minecraft api endpoints

Hey,
im using this personally and made some additions.
would be nice if you check them when you got time.

Removed the yarn.lock file and added it to .gitignore