kernel: mailbox: Add some testcases for mailboxs

Add some testcases to tesing different condition of mailbox
API. Check API robust in different input parameters is handled.
such as invoke API to get data with NULL input.
Signed-off-by: Jian Kang<EMAIL_ADDRESS>

@andyross , can it be merged now?

@andyross , can it be merged now?

@KangJianX  please rebase it and push again, thanks.

@KangJianX  please rebase it and push again, thanks.

@enjiamai updated

@enjiamai updated

I think the PR can be merged now.

@KangJianX Could you rebase it?

@nashif @andyross Could this PR merge now?


[allegro5] Add Minimp3 support

If this PR updates an existing port, please uncomment and fill out this checklist:

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

@microsoft-github-policy-service agree
Nov 3, 2024 12:45:10 AM microsoft-github-policy-service[bot] @.***>:

@HisuianZoroark69[https://github.com/HisuianZoroark69] please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@.*** agree [company="{your company}"]
*
Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@.*** agree
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@.** agree company="Microsoft"
*Contributor License Agreement

Contribution License Agreement
This Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
and conveys certain license rights to Microsoft Corporation and its affiliates (“Microsoft”) for Your
contributions to Microsoft open source projects. This Agreement is effective as of the latest signature
date below.

Definitions.
“Code” means the computer software code, whether in human-readable or machine-executable form,
that is delivered by You to Microsoft under this Agreement.
“Project” means any of the projects owned or managed by Microsoft and offered under a license
approved by the Open Source Initiative (www.opensource.org[http://www.opensource.org]).
“Submit” is the act of uploading, submitting, transmitting, or distributing code or other content to any
Project, including but not limited to communication on electronic mailing lists, source code control
systems, and issue tracking systems that are managed by, or on behalf of, the Project for the purpose of
discussing and improving that Project, but excluding communication that is conspicuously marked or
otherwise designated in writing by You as “Not a Submission.”
“Submission” means the Code and any other copyrightable material Submitted by You, including any
associated comments and documentation.
Your Submission. You must agree to the terms of this Agreement before making a Submission to any
Project. This Agreement covers any and all Submissions that You, now or in the future (except as
described in Section 4 below), Submit to any Project.
Originality of Work. You represent that each of Your Submissions is entirely Your original work.
Should You wish to Submit materials that are not Your original work, You may Submit them separately
to the Project if You (a) retain all copyright and license information that was in the materials as You
received them, (b) in the description accompanying Your Submission, include the phrase “Submission
containing materials of a third party:” followed by the names of the third party and any licenses or other
restrictions of which You are aware, and (c) follow any other instructions in the Project’s written
guidelines concerning Submissions.
Your Employer. References to “employer” in this Agreement include Your employer or anyone else
for whom You are acting in making Your Submission, e.g. as a contractor, vendor, or agent. If Your
Submission is made in the course of Your work for an employer or Your employer has intellectual
property rights in Your Submission by contract or applicable law, You must secure permission from Your
employer to make the Submission before signing this Agreement. In that case, the term “You” in this
Agreement will refer to You and the employer collectively. If You change employers in the future and
desire to Submit additional Submissions for the new employer, then You agree to sign a new Agreement
and secure permission from the new employer before Submitting those Submissions.
Licenses.

Copyright License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license in the
Submission to reproduce, prepare derivative works of, publicly display, publicly perform, and distribute
the Submission and such derivative works, and to sublicense any or all of the foregoing rights to third
parties.
Patent License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license under
Your patent claims that are necessarily infringed by the Submission or the combination of the
Submission with the Project to which it was Submitted to make, have made, use, offer to sell, sell and
import or otherwise dispose of the Submission alone or with the Project.
Other Rights Reserved. Each party reserves all rights not expressly granted in this Agreement.
No additional licenses or rights whatsoever (including, without limitation, any implied licenses) are
granted by implication, exhaustion, estoppel or otherwise.

Representations and Warranties. You represent that You are legally entitled to grant the above
licenses. You represent that each of Your Submissions is entirely Your original work (except as You may
have disclosed under Section 3). You represent that You have secured permission from Your employer to
make the Submission in cases where Your Submission is made in the course of Your work for Your
employer or Your employer has intellectual property rights in Your Submission by contract or applicable
law. If You are signing this Agreement on behalf of Your employer, You represent and warrant that You
have the necessary authority to bind the listed employer to the obligations contained in this Agreement.
You are not expected to provide support for Your Submission, unless You choose to do so. UNLESS
REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING, AND EXCEPT FOR THE WARRANTIES
EXPRESSLY STATED IN SECTIONS 3, 4, AND 6, THE SUBMISSION PROVIDED UNDER THIS AGREEMENT IS
PROVIDED WITHOUT WARRANTY OF ANY KIND, INCLUDING, BUT NOT LIMITED TO, ANY WARRANTY OF
NONINFRINGEMENT, MERCHANTABILITY, OR FITNESS FOR A PARTICULAR PURPOSE.
Notice to Microsoft. You agree to notify Microsoft in writing of any facts or circumstances of which
You later become aware that would make Your representations in this Agreement inaccurate in any
respect.
Information about Submissions. You agree that contributions to Projects and information about
contributions may be maintained indefinitely and disclosed publicly, including Your name and other
information that You submit with Your Submission.
Governing Law/Jurisdiction. This Agreement is governed by the laws of the State of Washington, and
the parties consent to exclusive jurisdiction and venue in the federal courts sitting in King County,
Washington, unless no federal subject matter jurisdiction exists, in which case the parties consent to
exclusive jurisdiction and venue in the Superior Court of King County, Washington. The parties waive all
defenses of lack of personal jurisdiction and forum non-conveniens.
Entire Agreement/Assignment. This Agreement is the entire agreement between the parties, and
supersedes any and all prior agreements, understandings or communications, written or oral, between
the parties relating to the subject matter hereof. This Agreement may be assigned by Microsoft.

—
Reply to this email directly, view it on GitHub[https://github.com/microsoft/vcpkg/pull/41918#issuecomment-2453044468], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AFLB6GWJCVGYT34LNBJIA7LZ6T6RNAVCNFSM6AAAAABRB35N76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJTGA2DINBWHA].
You are receiving this because you were mentioned.
[Tracking image][https://github.com/notifications/beacon/AFLB6GVYYNKCFWUHMAGPVQ3Z6T6RNA5CNFSM6AAAAABRB35N76WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUSGZ6PI.gif]

Another failed review :facepalm:


iOS build failed

Hello,
my build fails with the following output:
Xcode's output:
↳
/Applications/flutter/.pub-cache/hosted/pub.dartlang.org/tflite-1.1.2/ios/Classes/TflitePlugin.mm:21:9: fatal error: 'metal_delegate.h' file not found
#import "metal_delegate.h"
^~~~~~~~~~~~~~~~~~
1 error generated.
/Applications/flutter/.pub-cache/hosted/pub.dartlang.org/tflite-1.1.2/ios/Classes/TflitePlugin.mm:21:9: fatal error: 'metal_delegate.h' file not found
#import "metal_delegate.h"
^~~~~~~~~~~~~~~~~~
1 error generated.
note: Using new build system
note: Planning
note: Build preparation complete
note: Building targets in parallel
note: Removed stale file '/Users/alain/Library/Developer/Xcode/DerivedData/Runner-aluwvsnnrmgyolbvyvhthhxkqxbu/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/FBSDKCoreKit_Basics.build/InputFileList-5A3770FC020BB2992A8A91D6A694C4BB-FBSDKCoreKit_Basics-xcframeworks-input-files-d62456de18fc9278816caf1edc8ec4be-resolved.xcfilelist'
note: Removed stale file '/Users/alain/Library/Developer/Xcode/DerivedData/Runner-aluwvsnnrmgyolbvyvhthhxkqxbu/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/FBSDKCoreKit_Basics.build/OutputFileList-5A3770FC020BB2992A8A91D6A694C4BB-FBSDKCoreKit_Basics-xcframeworks-output-files-af501755e41daaa5055957e344b6aa79-resolved.xcfilelist'

note: Removed stale file '/Users/alain/Library/Developer/Xcode/DerivedData/Runner-aluwvsnnrmgyolbvyvhthhxkqxbu/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/FBSDKCoreKit_Basics.build/Script-5A3770FC020BB2992A8A91D6A694C4BB.sh'

note: Removed stale file '/Users/alain/Library/Developer/Xcode/DerivedData/Runner-aluwvsnnrmgyolbvyvhthhxkqxbu/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/PromisesObjC.build/Script-9B55CA93E42083CEA14CCF83D3DB055B.sh'

/Users/alain/Arduino/Bird Feeder/App/flutter/happy_birds/ios/Pods/Pods.xcodeproj: warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 15.0.99. (in target 'leveldb-library' from project 'Pods')

Could not build the application for the simulator.
Error launching application on iPhone 11 Pro.
Could you please help ?

I have the same problem on apple m1


resources: manage headers properly

Summary
Previously we had an R"( string in OITraceCode.cpp which allowed us to include the file as a string. Instead, keep OITraceCode.cpp a fully formed C++ file and utilise the build system to turn it into a string. This will be used for more header files that are needed both as valid headers and as strings for JIT compilation in the Typed TreeBuilder work.
Test plan

make test
CI

Codecov Report

Merging #139 (29628b3) into main (9731a30) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #139   +/-   ##
=======================================
  Coverage   37.14%   37.14%
=======================================
  Files           1        1
  Lines          35       35
  Branches       14       14
=======================================
  Hits           13       13
  Misses         13       13
  Partials        9        9

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

CodeCov is back 🎉


agglomerateByRank: agglomerated z-transformed assay lead to meaningless values

Hi,
I found a feature from agglomerateByRank that might be misleading.
agglomerateByRank sums values. It does it for every assay. For example, counts or relabundance do not cause any problems.
However, some other transformations like Z-transform lead to meaningless values when it is agglomerated. If user does not check this or if user is not aware of this, this can lead to errors. (In some transformations it might be hard to tell...)
> tse <- transformFeatures(tse, method = "z", pseudocount = 1)
> head(apply(assay(tse, "z"), 1, mean))
       549322        522457           951        244423        586076        246140
-1.918412e-17 -1.705533e-17 -4.270214e-16 -1.704699e-17 -6.831933e-17  1.366658e-16
> head(apply(assay(tse, "z"), 1, sd))
549322 522457    951<PHONE_NUMBER>76 246140
     1      1      1      1      1      1
> tse1 <- agglomerateByRank(tse, "Genus")
> head(apply(assay(tse1, "z"), 1, mean))
   Class:Thermoprotei      Genus:Sulfolobus           Class:Sd-NA           Order:NRP-J        Family:SAGMA-X Family:Cenarchaeaceae
        -1.274355e-17         -4.270214e-16          1.025697e-16         -5.124773e-16         -3.927856e-16         -3.608225e-16
> head(apply(assay(tse1, "z"), 1, sd))
   Class:Thermoprotei      Genus:Sulfolobus           Class:Sd-NA           Order:NRP-J        Family:SAGMA-X Family:Cenarchaeaceae
             1.963880              1.000000              2.835166              3.640426              2.964775              8.371031

Should there be an argument that specifies, which assays are added to new agglomerated TreeSE?
By default, abund_values = c("counts")
-Tuomas

Agreed. Only "counts" by default. Rest has to be explicitly specified by the user, including relabundance for the agglomerated data. This can help to avoid mistakes and confusion.

I don't think this is worth doing. You will never catch all mistakes a user can make with this function. I wouldn't try to do it because it will make a complex function even more complex. In addition, the assay name doesn't mean anything and since we don'Tt force assay names for certain types of data, the user must make sure to analyze the data correctly.
However, is there a test which can tell, if summing is applicable? Like test for negative values or binary values? A warning might do the trick with additional clarification in the examples

Also the rule is always to agglomerate before transformation. I mean this is common sense, isn't it?

Perhaps indeed a test and warning for suspicious cases would be a better way of doing this.
I have learned to be careful in trusting common sense, it is good if can help to avoid common accidental mistakes but yes this has to be balanced with other design considerations.

That might be better way to approach this


[FEATURE] Add localization for maps

Is your feature request related to a problem?
At present, the map only supports English. In order for users in different languages, the map will be provided in multiple languages.
What solution would you like?
Display content on the Maps like countries names, streets etc in preferred language. The default language will be consistent with the OpenSearch dashboards language.

Can you help me add permission to the repo? It seems that I have no permission to edit tags. @junqiu-lei @VijayanB


[KYUUBI #1439] Add KYUUBI_CTL_JAVA_OPTS to configure java opts of kyuubi-ctl.

Why are the changes needed?

Add KYUUBI_CTL_JAVA_OPTS to configure java opts of kyuubi-ctl. For details: #1439.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[X] Run test locally before make a pull request

Check all kyuubi configs *** FAILED ***

3000 java.lang.AssertionError: assertion failed: /home/runner/work/incubator-kyuubi/incubator-kyuubi/docs/deployment/settings.md out of date, please update doc with KYUUBI_UPDATE=1 build/mvn clean install -Pspark-provided -DwildcardSuites=org.apache.kyuubi.config.AllKyuubiConfiguration

MySQLAuthPacketSuite:
- encode MySQLHandshakePacket *** FAILED ***
  UnpooledByteBufAllocator$InstrumentedUnpooledUnsafeHeapByteBuf(ridx: 0, widx: 103, cap: 256) did not equal UnpooledHeapByteBuf(ridx: 0, widx: 103, cap: 103/103) (MySQLCodecHelper.scala:43)
- decode MySQLHandshakeResponse41Packet
- encode MySQLAuthSwitchRequestPacket
- decode MySQLAuthSwitchResponsePacket

@yaooqinn ,it doesn’t seem to be caused by this PR, can you recheck it?

MySQLAuthPacketSuite:
- encode MySQLHandshakePacket *** FAILED ***
  UnpooledByteBufAllocator$InstrumentedUnpooledUnsafeHeapByteBuf(ridx: 0, widx: 103, cap: 256) did not equal UnpooledHeapByteBuf(ridx: 0, widx: 103, cap: 103/103) (MySQLCodecHelper.scala:43)
- decode MySQLHandshakeResponse41Packet
- encode MySQLAuthSwitchRequestPacket
- decode MySQLAuthSwitchResponsePacket

@yaooqinn ,it doesn’t seem to be caused by this PR, can you recheck it?

Fixed in master by #1442, please rebase master

Codecov Report

Merging #1440 (bbbf1f2) into master (c85d640) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1440   +/-   ##
=========================================
  Coverage     58.88%   58.88%
  Complexity      155      155
=========================================
  Files           232      232
  Lines         11931    11931
  Branches       1460     1460
=========================================
  Hits           7026     7026
  Misses         4313     4313
  Partials        592      592

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 c85d640...bbbf1f2. Read the comment docs.

thanks merged to master and 1.4


sanity check whether the input data contains duplicated entities

I am willing to work on this. Maybe you can explain a bit more.

hey @piyush0609 , glad that you volunteered, just added some description, feel free to reach out if it's still unclear;-)

So what do we want to do here @jbao, from your comment earlier what I understood is that we have to make some changes in time_resolved metrics.

basically we need to distinguish the two metric types:

if aggregated, we need to add a checkpoint to ensure all entities are unique
if time-resolved, we need to ensure the data is unique based on the combination of entity and time

I am not sure that I can do it, but I will definitely try and I will be needing your help.


Create ArduSub custom configurations for different sets of thrusters

Custom thruster configurations for different sets of available thrusters.
The bluerov has in total 6 thrusters. When one or more thruster fails the frame configuration model that ArduSub uses should be replaced by one that represents the runtime configuration.
More info:
https://www.ardusub.com/quick-start/vehicle-frame.html
http://www.ardusub.com/developers/developers.html#making-a-custom-configuration
https://discuss.bluerobotics.com/t/sitl-with-qgc-and-gazebo-with-custom-frame/9323
https://discuss.bluerobotics.com/t/ardusub-sitl-custom-frame/7403
https://discuss.bluerobotics.com/t/custom-frame-configuration/4428
https://discuss.bluerobotics.com/t/build-custom-frame/9709

Posted a question in bluerobotics discourse: https://discuss.bluerobotics.com/t/is-it-possible-to-adapt-at-runtime-the-motors-contribution-to-the-6dof/13079

This will require some modifications to ArduSub code.

https://github.com/ArduPilot/ardupilot/issues/9175

https://ardupilot.org/dev/docs/debugging-with-gdb-on-linux.html

Idea for workaround: change speed when thruster fails


Don't pass empty lists of certificates to the oVirt SDK

The oVirt SDK accepts lists of trusted CA certificates. But when an
empty list, or a list containing just 'nil' elements is passed, the
result is that no certificate is trusted, not even the system default
trusted certificate. To avoid that this patch changes the oVirt provider
so that it passes 'nil' instead of empty lists.

@miq-bot assign @oourfali
@borod108 @pkliczewski please review.

@durandom please review.

@oourfali the changes in this PR looks trivial.  Please give your 👍 so we can merge.

Looks good to me.

@miq-bot assign @agrare
@agrare pls merge

Thanks @agrare.


Make the CI abort actions on older commits when a new commit gets pushed

Right now the CI seems to continue running actions on older commits when I push newer commits to a PR branch. This results in a lot of spam in my notifications and also probably clogs up the CI job action queue.
It might be a good idea to halt all previous actions once a new commit gets added, so that the actions will only be run on the latest commit in a PR.

Fixed in #3244


Mockup up request page for (Admin: ICS Dept Head)

Please do work for this task in a branch called issue-020.

form to reserve


2 Max Poolings in a row

Hi,
thank you for the great tutorial. BUT: when running this with a single convLayer, you use Pooling twice. The first time it pools within the convLayer and the second time right after:
if use_two_conv:
conv1 = conv_layer(x_image, 1, 32, "conv1")
conv_out = conv_layer(conv1, 32, 64, "conv2")
else:
conv1 = conv_layer(x_image, 1, 64, "conv")
conv_out = tf.nn.max_pool(conv1, ksize=[1, 2, 2, 1], strides=[1, 2, 2, 1], padding="SAME")
Is this just to keep the dimension of the flattened tensor the same?

Thanks for the report. Fixed it.

With that change there is a problem of dimensionality between logits_size[400,10] and labels_size[100,10]. I'm not sure how to fix it.

@Marfly851 sorry about that. I've fixed it.


Instructions for setting up a new server

Currently there's no simple way for a developer to get started with this project. No mention of what packages need to be installed or what commands should be run to compile things. Not even a helper .env file or mention of the websocket configuration...
:sob: :mag_right: :page_facing_up:

Fixed as of be19068e02a4e4f4e7b4e19a8a6c189284fa81bf


[WebOS] No episodes will load

Describe the bug
When loading an episode, it only shows the black empty screen with a default UI overlay.
To Reproduce
Steps to reproduce the behavior:

Select an episode to watch
See error/nothing

Expected behavior
The video loads normally.
Additional context
I tried several episodes from several series.
Started happening today.
Using LG Smart TV.

Hi @SirH3nry, this same error was reported to me today, crunchyroll added a little more security to the access of the videos that's why the error occurs. I'm going to try to solve it.
regards

hi @SirH3nry, i make a temporary fix, it works, maybe there is an error. Any problem, report it and I will fix it. thanks


Fix additional property name for TermsQuery

This seems to be a copy/paste mistake made by me when I implemented the behavior_meta tag.
@swallez @Anaethelion @ezimuel @picandocodigo @miguelgrinberg @l-trotta @pquentin @JoshMock   Please check, if this is a breaking change in any of your clients. Most likely only the static language clients should be affected - if at all.

I'm not using the key and key and value names, just their types, so for the Python DSL client this is fine 👍

AdditionalProperty behaviors are currently ignored by JS codegen. :+1:

It's not implemented yet in the Java generator, so no problem.


KAFKA-14274, #7: introduce FetchRequestManager to integrate fetch into new consumer threading refactor

This change introduces the FetchRequestManager that will be responsible for:

Formatting fetch requests to the background thread
Configuring the callback on fetch responses for the background thread

The response handler will collect the fetch responses from the broker and create CompletedFetch instances as is done in Fetcher. However, unlike the Fetcher, the foreground logic will decompress/deserialize/and convert the data in CompletedFetch before returning to the user.

Re-draft-ing this PR in favor of PR #19 first.

Nice improvement removing the dependency on the networkClient when not needed in the unit tests. I think we can remove several calls to networkClientDelegate.poll that are not needed anymore, like the ones I pointed out above.


Revert accidental changes to UITest projects

#779 included accidental changes to the UITest project files, presumably from migrating the solution locally in Visual Studio. These should not be checked in to the repo.

/azp run


Reimplement as RktMachine.

Combined with Corectld.app into single menubar app.
Added mDNS to Core OS VM.
Documentation.
Made Core OS VM root directory persistent.
Installed acbuild on CoreOS VM.
Removed libev dependency.
Removed Docker.
Removed most shell scripts.
Removed notifications.
Added example containers.
DMG packaging.
New About dialog.

Quick note since this is basically a full rewrite: I'd reconsider keeping this repository marked as fork of the original. Not only do works get deprioritized in the general search, but it also disables really important features like repo-local code search. GitHub forks are only meant to be short-lived branches that are to be merged back into the main repo. For longer-running or permanent forks it's better to create a completely new repo and leave a note about the origin somewhere.


Feature/from cache than refresh

Add options for get image first from cache and then refresh image (download from source). This is useful when you want to display a changeable image behind the same url at the same app session and use cache if no internet.

Thanks for the PR. But I don't understand what is the difference between this value and the .forceRefresh option?

If  .forceRefresh set, Kingfisher will ignore the cache. But .fromCacheThenRefresh options get image from cache and then try to download

3 бер. 2019 р. о 3:50 пп Wei Wang<EMAIL_ADDRESS>написав(ла):
Thanks for the PR. But I don't understand what is the difference between this value and the .forceRefresh option?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/onevcat/Kingfisher/pull/1128#issuecomment-469024806, or mute the thread https://github.com/notifications/unsubscribe-auth/AIbUL06kO8xASe1Q3sgNb3huRxPsRSVYks5vS9MqgaJpZM4ba_R3.

Oh, I see it.
I guess this addition makes the behavior not stable anymore. For these reasons:

The completion handler will be called twice, which is not expected actually. One invocation should correspond one and only one completion callback. The additional option logic here makes confusing on this convention and introduces a different pattern. It might be very hard to track and maintain later. There might be also some end users logic replies on the fact that the completion handler would be called once, it also breaks these kinds of size effect code.
As the image changed behind the same URL, end users may always want the latest image. I cannot see a good example of the use case for this option. A flickering of the image from a previous version to a newer version may also be thought as an implementation bug.
I guess the same behavior can be implemented easily by nesting two setting image methods, like this:imageView.kf.setImage(with: url) { result in
    imageView.kf.setImage(with: url, placeholder: result.value?.image, options: [.forceRefresh])
}

This might be clearer to describe what is going on.
What do you think about it?

Yes, you are right. Invocation twicely completion handler is will hard complicate logic and will provoke various bugs in the future. I use your example for implement cacheThenRefresh logic in my project. Thanks)


feat(perception): add data_path argument to launch file

Description

Belongs to Standardize a strategy for packages with downloaded artifacts
As artifact downloading in ansible was merged to Autoware.universe. It is possible to use artifacts downloaded there.
This PR introduces new launch argument data_path in the form:
  <arg name="data_path" default="$(env HOME)/autoware_data" description="packages data and artifacts directory path"/>
It should be downstreamed to all packages with downloaded artifacts, mostly perception.
Tests performed

Logging_simulator and planning_simulator launch fine with this parameter.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@lexavtanke I've enabled automerge for this PR, once CI passes it'll be merged.


wrapper_target recursively called number of times the requests hit before a url with dynamic profiling set

File "/project-virt-env/lib/python3.4/site-packages/silk/profiling/profiler.py", line 159, in wrapped_target
File "/project-virt-env/lib/python3.4/site-packages/silk/profiling/profiler.py", line 159, in wrapped_target
File "/project-virt-env/lib/python3.4/site-packages/silk/profiling/profiler.py", line 159, in wrapped_target
File "/project-virt-env/lib/python3.4/site-packages/silk/profiling/profiler.py", line 159, in wrapped_target

multiple profiles of wrapper_target created instead of just the actual function.

This might be co-related to #2

I'm running into the same issue, did you figure out how to fix this?


WASM / Rust version

Since edge runtimes have WASM support, it'd be good if there was a planetscale driver in WASM instead of Js. Edge resources are limited so the more savings the better :)

I've made rust version that runs on wasm (i've only tested it on cloudflare workers).
https://github.com/filipton/planetscale-driver

I'm going to close this out given that @filipton has made this available. Thanks for your contribution!


How to get a value of false (as opposed to "false")

I can get (encode-json) to emit a value of true quite easily: just use t. But I cannot figure out how to get a value of false.
I'm not talking about the strings "true" and "false". I'm talking about the javascript keywords true and false.
I tried (cl-json:json-bool nil). This gives me ["false"]. I also tried (cl-json:json-bool :false). It gives me ["true"].
What is the trick to getting false into my JSON?

I notice that the explicit encoder can do it, although you then must use explicit data structures all through:
* (in-package :cl-json)
#<PACKAGE "JSON">
* (with-explicit-encoder (encode-json-to-string `(:alist . ((:yes . ,(json-bool t)) (:no . ,(json-bool nil))))))
"{\"yes\":true,\"no\":false}"

Take a look at (DESCRIBE 'ENCODE-JSON) under the method that accepts a list, for information on the explicit encoder.

Got this solution incorporated into my code. It works! Thanks.


new issue

this is when im generating the audio ?
segment_text: hello i am ai voice
--Segment 1/1: est. 2.00s (1 of 1 iterations)
hello i am ai voice
Traceback (most recent call last):
File "C:\Users\tomas_000\AppData\Roaming\Python\Python310\site-packages\gradio\routes.py", line 422, in run_predict
output = await app.get_blocks().process_api(
File "C:\Users\tomas_000\AppData\Roaming\Python\Python310\site-packages\gradio\blocks.py", line 1323, in process_api
result = await self.call_function(
File "C:\Users\tomas_000\AppData\Roaming\Python\Python310\site-packages\gradio\blocks.py", line 1051, in call_function
prediction = await anyio.to_thread.run_sync(
File "C:\Users\tomas_000\AppData\Roaming\Python\Python310\site-packages\anyio\to_thread.py", line 33, in run_sync
return await get_asynclib().run_sync_in_worker_thread(
File "C:\Users\tomas_000\AppData\Roaming\Python\Python310\site-packages\anyio_backends_asyncio.py", line 877, in run_sync_in_worker_thread
return await future
File "C:\Users\tomas_000\AppData\Roaming\Python\Python310\site-packages\anyio_backends_asyncio.py", line 807, in run
result = context.run(func, *args)
File "C:\Users\tomas_000\AppData\Roaming\Python\Python310\site-packages\gradio\helpers.py", line 588, in tracked_fn
response = fn(*args)
File "C:\Users\tomas_000\Desktop\bark-main\bark_webui.py", line 432, in generate_audio_long_gradio
full_generation_segments, audio_arr_segments, final_filename_will_be = api.generate_audio_long_from_gradio(**kwargs)
File "C:\Users\tomas_000\Desktop\bark-main\bark_infinity\api.py", line 829, in generate_audio_long_from_gradio
full_generation_segments, audio_arr_segments, final_filename_will_be = generate_audio_long(**kwargs)
File "C:\Users\tomas_000\Desktop\bark-main\bark_infinity\api.py", line 1123, in generate_audio_long
write_one_segment(audio_arr = np.concatenate(audio_arr_segments), full_generation = full_generation_segments[0], **kwargs)
File "C:\Users\tomas_000\Desktop\bark-main\bark_infinity\api.py", line 360, in write_one_segment
write_seg_wav(filepath, audio_arr, **kwargs)
File "C:\Users\tomas_000\Desktop\bark-main\bark_infinity\api.py", line 409, in write_seg_wav
write_audiofile(filepath, audio_arr, **kwargs)
File "C:\Users\tomas_000\Desktop\bark-main\bark_infinity\api.py", line 436, in write_audiofile
audio = AudioSegment.from_wav(temp_wav)
File "C:\Users\tomas_000\AppData\Roaming\Python\Python310\site-packages\pydub\audio_segment.py", line 808, in from_wav
return cls.from_file(file, 'wav', parameters=parameters)
File "C:\Users\tomas_000\AppData\Roaming\Python\Python310\site-packages\pydub\audio_segment.py", line 728, in from_file
info = mediainfo_json(orig_file, read_ahead_limit=read_ahead_limit)
File "C:\Users\tomas_000\AppData\Roaming\Python\Python310\site-packages\pydub\utils.py", line 274, in mediainfo_json
res = Popen(command, stdin=stdin_parameter, stdout=PIPE, stderr=PIPE)
File "C:\Program Files\Python310\lib\subprocess.py", line 971, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "C:\Program Files\Python310\lib\subprocess.py", line 1440, in _execute_child
hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] The system cannot find the file specified

no this is when im using the regular tts ?

i choose the speaker and i just typed up text i am a bot just testing this and im trying to just generate audio from my text and it comes up with this error also is my bark infinity folder supposed to be 32 mb in size ?

You can also try install regular windows ffmpeg. Try to install it so if you type:
ffmpeg -version

it works where you run bark.

I think I have a fix that works in most systems, more reliably.
pip install ffmpeg-downloader
ffdl install --add-path

That install ffmpeg via python library designed for it, and seems to do it in a way where torch and python can more reliably find and use it.
After you run those commands you have to exit the miniconda prompt (or the terminal if you aren't using conda or mamba at all) and restart.
Then type
ffmpeg -version
ffprobe -version

to make sure they work.
I'll to a quick update today and maybe a bigger install update...

Can I have the link of discord group? I'm kinda overwhelmed with all of this button. 😅

It's the Bark discord: https://github.com/suno-ai/bark link on the top of the readme.


Change lscpu to uname -m

fix for https://github.com/kontena/pharos-cluster/issues/501

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Drone is :x: because master is broken pending #504, otherwise LGTM


Allow overriding or customizing the template files

[x ] I'd be willing to implement this feature

Description
Allowing the generator files to be modified automatically to include or omit features.
Motivation
There are any number of customizations we'd like to make to the project generators, such as creating a SAM template.yaml, adding specific sections or configuration to pyproject.toml, adding or modifying tasks in project.json, and so on. We could create our own generator that calls the @nxlv generator, but that seems wasteful and somewhat difficult if all we need to do is, for example, change the pytest configuration.
Suggested Implementation
Passing in a simple --template-files-dir that overrides the location of the template files. Or somewhat more complicated, only overrides the files that are present in the additional directory. This would introduce a minimal maintenance burden.
Alternate Implementations
There may be nx conventions for generators of which I am unaware, but some other approaches are:

Add a features parameter to the generators that is used to toggle the presence of sections of the files or modify behavior, similar to addDevDependencies but generalized. Additional features would require code additions though.
Ensure that each of the functions used to perform actions in the current generator are exported and composable by another generator. An example would be the project configuration object, which is both created and directly used in the project generator function. An exported function that returns a default configuration suitable for modification would be useful to call from another generator.

Hey @scottanderson42, I agree, currently, the project generator is not customizable enough, let me research a little bit, look at how Nx does for their own generators, and discuss with you more about it.
I agree that could be not productive if you write your generator trying to remove or modify the current generator, another suggestion might be a light version of the project generator that only generates a minimal number of files, in this way, it is easy to extend the @nxlv generator to your uses cases without any workaround to change what the base generator created.
Sounds good?

Sounds good to me!

Hey @scottanderson42 sorry for the delay, I finally had the time to raise the PR that implements a new project generator, more flexible, I also used your suggestion to have the --templateDir CLI option to point for a custom template directory.
PR: https://github.com/lucasvieirasilva/nx-plugins/pull/123
It also fixes #117 because I'm not installing the tomli anymore.


Actions are now Rules

[Enter feedback here]
This whole page should be updated to reflect the updated Game Manager experience, most notably on this page is that Actions are not called Rules and are in a completely different section of the game manager, and the tabs for game manager are now totally different than what is shown in the screenshots on this page.

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

ID: f4b91dbe-6769-b52e-823f-ae08a75e55f5
Version Independent ID: 17ed3030-63fb-e789-ef8b-1fdca12be986
Content: Writing custom CloudScript (Legacy) - PlayFab
Content Source: playfab-docs/features/automation/cloudscript/writing-custom-cloudscript.md
Service: playfab
GitHub Login: @williacj
Microsoft Alias: cjwill

Updated docs to reflect current Game Manager UI.


Ability to enable/disable the renderers on an object

Show and hide an object (e.g. controllers) in a scene.

Reopening this issue as the current solution doesn't work with Renderers


[CADL APIView] Add cadl-apiview emitter

Adds APIView for the Cadl language.
A sample of the generated APIView:
https://apiview.dev/Assemblies/Review/3d72026d2d6e4b588f330a6143acc050
Outstanding issues:

[ ] add test pipieline (@praveenkuttappan #4182)
[ ] add some basic tests

/checkenforcer override

@tjprescott I have added a commit to this PR to create a pipeline for CADL emitter.

/azp run tools - apiview-cadl-emitter - ci

Adding @benbp to this PR to review CI.yml I have added for cadl emitter pipeline.

Minor comment but looks good.

@benbp Is it okay If I make npm install step change in next PR when I make additional changes in this pipeline to release this package to npm instead of devops feed?

@benbp Is it okay If I make npm install step change in next PR when I make additional changes in this pipeline to release this package to npm instead of devops feed?

Works for me.


Fetching pull requests failed: Cannot read properties of undefined (reading 'pullRequestPage')

Saw this after checking out xtermjs.org master:

Log:
[Info] Looking for git repository
[Info] Found 0 repositories during activation
[Info] Git repository found, initializing review manager and pr tree view.
[Info] Registering git provider
[Info] Review> Validate state in progress
[Info] Review> Validating state...
[Info] Review> Validate state in progress
[Info] Review> Validating state...
[Info] Found GitHub remote
[Info] Review> no matching pull request metadata found for current branch undefined
[Info] Review> no matching pull request metadata found on GitHub for current branch undefined
[Info] Found GitHub remote
[Info] Review> no matching pull request metadata found for current branch tyriar/update_docs
[Info] Review> no matching pull request metadata found on GitHub for current branch tyriar/update_docs
[Info] Review> Queuing additional validate state
[Info] Repo state for file:///Users/daimms/dev/Tyriar/xtermjs.org changed.
[Info] Repo file:///Users/daimms/dev/Tyriar/xtermjs.org has already been setup.
[Info] Review> Validating state...
[Info] Found GitHub remote
[Info] Review> no matching pull request metadata found for current branch master
[Info] Review> no matching pull request metadata found on GitHub for current branch master
[Info] Review> Queuing additional validate state
[Info] Review> Queuing additional validate state
[Info] Review> Validating state...
[Info] Review> Validating state...
[Info] Found GitHub remote
[Info] Found GitHub remote
[Info] Found GitHub remote
[Info] Found GitHub remote
[Info] Found GitHub remote
[Info] Found GitHub remote
[Info] Review> no matching pull request metadata found for current branch undefined
[Info] Review> no matching pull request metadata found on GitHub for current branch undefined
[Info] Found GitHub remote
[Info] Found GitHub remote
[Info] Review> no matching pull request metadata found for current branch master
[Info] Review> no matching pull request metadata found on GitHub for current branch master

What are some verif steps for this?

I wasn't able to repro the issue. Removing from the milestone as this isn't verifiable.


Draft: Post rendering function

First try in having a post rendering function as discussed at https://github.com/emilk/egui/discussions/1578#discussioncomment-2692266
I'm not sure if the full PaintCallbackInfo would be needed. Either way I don't know how to fill half its fields.
Frame is needed to get access to the gl handle.
Should I wrap the screen size and the gl handle into an own Struct? Or is it ok to provide a readonly reference to Frame?

Looks good!
It would be great with an example app (that also serves as a test) which paints a GUI and then can take a screenshot of it using this feature!


Tolerate overshooting during density tests

This will allow us to proceed w/ testing catastrophic HA recovery scenarios (i.e. controller manager dies) without failing.   Otherwise, density can fail pretty easily in a large cluster where there might be pods transitioning to running, because it will overschedule (temporarily).

Allows us to survive pending complete fix to #10030

I would change
fmt.Errorf("Only %d pods ...
and remove "Only" since you are now relying on this for overshoot as well.

Can you do this only when HA=true? See  https://github.com/GoogleCloudPlatform/kubernetes/issues/10030#issuecomment-122041920 for details on overshoot scenario. We still want to fail on overshoot.

@bprashanth Is there a way we can inspect controller-manager API server or ETCD to determine if controller-manager was restarted ?  That way we dont need to specify "ha" as an external metadata, and we can tolerate overshoots in those cases .

Yeah, you can send an event at startup time and list events when you observe an overshoot (anywhere in this callstack https://github.com/GoogleCloudPlatform/kubernetes/blob/master/cmd/kube-controller-manager/app/controllermanager.go#L144).
While that's useful, a controller manager restart is enough reason to fail any test that didn't explicitly kill it, overshoot or not. Since this is density and you are manually killing it to trigger HA (right?), I'd rather not ignore overshoots by default (if you don't want to add a --ignore flag I'll take another addition to https://github.com/GoogleCloudPlatform/kubernetes/blob/master/test/e2e/density.go#L168 for now).
Basically I don't want our e2e runs papering over controller manager restarts and/or replica overshoots.

@jayunit100 @bprashanth - perhaps we should re-purpose and automate a failover case instead and copy density as part of it?  I'm guessing the HA set of tests will grow...
Thoughts?

Yup - thats a good way to to id .  ROOT CAUSE = util.go deciding wether or not a test fails or succeeds - the coupling is causing a problem.   So lets let RunRC return some info to the calling test, and then using @timothysc 's idea,  HA density tests can have more lax failure criteria for now.

@jayunit100 ideally we can trip a failover on the test.

how? thats a great idea.

Fyi if you want something that can restart daemons easily you might just wait for https://github.com/GoogleCloudPlatform/kubernetes/pull/11339

summarizing:

bprashant: we need the test to fail on overshoots
bprashant: but we want the test to allow overshoots if doing HA, because that is a known consequence of HA right now.
jayunit100: util.go is failing tests , embedding logic on when to fail, needs decoupling for pure reasons, and also benefits this particular issue.
timothysc: we should just have a go test which knows its going to overshoot and parameterizes runRC accordingly
bprashant: there is a PR in flight #11339 which can trip failover, which means we can consolidate failover test into one go method.  This allows us to have a smart ha density test which trips itself.  Even until then, decoupled RC can simply have a ginkgo filter so we can run HA density tests .

all in agreement?

SGTM. I'm happy as long as the common case (our hourly jenkins runs) don't gloss over bugs :)
The proposed solution of adding an HA ginkgo flag and tripping e2e tests (doesn't have to be just density right) via daemon restart triggered by some restart-y framework, makes sense as well.

+1.

I'm happy as long as the common case (our hourly jenkins runs) don't gloss over bugs :)
Never happens ;-)

great ! ill update this test accordingly to have an HA Variant (makes tim happy), and parameterize RunRC (makes me happy).  And the net result will be a density test which is strict (makes bprashant happy).  SO MUCH HAPPINESS.

rob brought up the fact that Error isn't ACTUALLY causing test fail, so... maybe this is a no-op and we should close - and just interpret HA test results with a grain of salt knowing that they will overshoot.?

CLOSED ! This really is a no-op.  We need Density to continue failing when overshoots happen, and the simplest way to deal with this is by interpretting the test results proactively when doing ha tests.


Unique, periodically refreshed hash

Adds a unique hash that is refreshed every second, so certains browsers don't detect multiple separators with the same name as duplicates and remove them.
The typeof check is there because crypto.randomUUID() has only been standard/implemented  for a couple years (and I wanted to be extra cautious), and as a placeholder in case a polyfill is ever to be used; so it will work exactly the same as before in case someone is using a really outdated browser.
Should address #12

This is a great idea and the changes look good. Let me do some tests and then I think this can be merged.


quassel: fix sources

Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Fixes changes after #14080 where the updated hash was missing.
Put the source in a common file so there is only one place to update.

I should have known after https://github.com/NixOS/nixpkgs/pull/14080#issuecomment-198957553 that something was up. /cc @ttuegel

That patch is no longer necessary for Qt 5.5, I assume?

Correct. In the previous PR I said it was necessary but it was building
against the wrong source. I'm running kde5.quasselClient from this PR
locally and it works fine.
On Tue, 22 Mar 2016 13:24 Thomas Tuegel<EMAIL_ADDRESS>wrote:

That patch is no longer necessary for Qt 5.5, I assume?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/14118#issuecomment-199812570


freeze Keras model (h5) to Tensorflow (pb)

Added script to freeze the Keras model file (h5) to TensorFlow frozen model format (pb). Also added inference script for the TensorFlow model file.

Hi @Manthan274 , thanks.


Special characters not working

Special Characters/Emojis/Symbols not working in app (via ^⌘Space) 💔💦.

Most likely related to #8


Anyone else having issues with wrong colors?

I'm using a generic D1 mini along with a D1 Mini Matrix Breakout v1.4.0 and P3 64x32 LED matrix I purchase off of ebay. Everything seems to work OK but the colors are all wrong. Does anyone know where or how to make changes so colors come out correct?

SetColorOrder could help

greetings
Apparently you miss some clables pin location, check the RGBR1G1B1 cables
again... one is in miss-order
Best regards
El vie., 23 oct. 2020 a las 11:04, Dominic Buchstaller (<
<EMAIL_ADDRESS>escribió:

SetColorOrder could help
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/2dom/PxMatrix/issues/248#issuecomment-715432504, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIGUXZZ3X3PKCWPPIMJ6M3SMGSPTANCNFSM4S4VM3NA
.

--
David Elias Flores Escalante
TeleTracking SAC

greetings Apparently you miss some clables pin location, check the RGBR1G1B1 cables again... one is in miss-order Best regards El vie., 23 oct. 2020 a las 11:04, Dominic Buchstaller (<<EMAIL_ADDRESS>escribió:
…
SetColorOrder could help — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#248 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIGUXZZ3X3PKCWPPIMJ6M3SMGSPTANCNFSM4S4VM3NA .
-- David Elias Flores Escalante TeleTracking SAC

I was afraid of pin connectors in wrong order but it is a breakout board so it wasn't my wiring. I was thinking there must be a way to change in software but don't know how.


PATH is updated only based on the last flag that sets it

I'm having some issues when creating a Singularity definition file. It seems that the PATH is updated only based on the last flag that sets it. For example, creating a definition file with both --miniconda and --afni results in the path not being correctly updated to point to miniconda if --afni comes after --miniconda, or vice-versa (only miniconda is set if the afni flag precedes it).
  docker run --rm kaczmarj/neurodocker:master generate singularity \
    --base=neurodebian:stretch-non-free \
    --pkg-manager=apt \
    --install vim datalad ffmpeg gcc g++ swig xvfb \
    --user=neuro \
    --miniconda \
      conda_install="python=3.6 jupyter jupyterlab jupyter_contrib_nbextensions
                     matplotlib scikit-learn seaborn scikit-image ipython keras" \
      pip_install="tensorflow==1.5.0 joblib pymvpa2 nilearn nipype[all] datalad[full] duecredit pybids xvfbwrapper" \
      create_env="neuro_py36" \
      activate=true \
    --run "mkdir /idata /ihome /dartfs-hpc /data /dartfs" \
    --env "KERAS_BACKEND=tensorflow" \
    --run-bash "source activate neuro_py36 && export KERAS_BACKEND=tensorflow && python -c \"from keras.applications.vgg16 import VGG16\"" \
    --add-to-entrypoint "export KERAS_BACKEND=tensorflow" \
    --afni version=latest \
    --workdir /home/neuro

%environment
export LANG="en_US.UTF-8"
export LC_ALL="en_US.UTF-8"
export ND_ENTRYPOINT="/neurodocker/startup.sh"
export CONDA_DIR="/opt/miniconda-latest"
export PATH="/opt/afni-latest:$PATH"
export KERAS_BACKEND="tensorflow"
export AFNI_PLUGINPATH="/opt/afni-latest"

hi @mvdoc - thank you, this is definitely a bug. i have a PR for the fix and will update shortly. thanks for bringing this to my attention.

fyi the %environment section in the fix looks like this:
%environment
export LANG="en_US.UTF-8"
export LC_ALL="en_US.UTF-8"
export ND_ENTRYPOINT="/neurodocker/startup.sh"
export CONDA_DIR="/opt/miniconda-latest"
export PATH="/opt/miniconda-latest/bin:$PATH"
export KERAS_BACKEND="tensorflow"
export PATH="/opt/afni-latest:$PATH"
export AFNI_PLUGINPATH="/opt/afni-latest"

awesome, thanks for the super quick fix :)

@mvdoc - fix has been merged. the PATH vars should be correct now. i updated the master docker image as well, so you can use your command as you had it.


[Migration] Rename "Contributor" to "Data Collector"

This change should be restricted to references at the code level and should not be applied to the collection and document names in Firestore.

Added a subtasks of #1114.


Octopus Testing -Testing VCP supports node VM migration to another VC

Test steps:

Create storageclass and pvc to provision volume dynamically.
Migrated VM to different VC.
Created Pod with volume provisioned at step 1.
Executed command kubectl describe pod.

Expected result:

Volume got provisioned successfully.
Volume got attached successfully.

Manually Tested this test. Got this issue https://github.com/vmware/kubernetes/issues/390 while testing.

This will be covered by #365 and #366.


Add my team listing

Nice.

burrito doesnt actually exist, so this is invalid...?

it depends on the screen resolution/window size for some reason

burrito doesnt actually exist, so this is invalid...?

....
uh what happened, i got a reply from a person with no name and no pfp, with just "...."
like i said burrito doesnt exist so idk why this was submitted

a lot of people are cut off and it depends on the size of the website, on my ultrawide i see myself fine


infinite loop in MonarchLexer's getloadstatus

VSCode Version: n/a -- Using Monaco
OS Version: MacOS Mojave

Steps to Reproduce:

Create a language syntax that references itself as an embedded mode.
Call colorizeElement using that language for syntax highlighting.

Does this issue occur when all extensions are disabled?: Yes
Code in question is:
https://github.com/microsoft/vscode/blob/master/src/vs/editor/standalone/common/monarch/monarchLexer.ts#L423
Proposed fix is:
if (tokenizationSupport instanceof MonarchTokenizer) {
  const nestedModeStatus = tokenizationSupport.getLoadStatus();
  ...

become:
if (tokenizationSupport == this) continue;
if (tokenizationSupport instanceof MonarchTokenizer) {
  const nestedModeStatus = tokenizationSupport.getLoadStatus();
  ...

I've just stumbled upon this too.
In case it's useful I've noticed that the call to colorize/colorizeElement works at first if I'm just viewing the rendered Markdown, which is the language that references itself as an embedded mode for me, but then if I fire up a monaco editor instance with that same Markdown content and switch back to rendering and syntax highlighting it than the bug occurs. It doesn't happen if I start editing some Markdown which doesn't embed a markdown code block inside itself and then switch back to the Markdown I was rendering initially.

I can confirm that e4b4c1751d37f8b22b5a3eab20021e3c781fa33e resolved the issue 👍


Use getSubclassTableSpan instead of getTableSpan, fixes #521

The trivial fix for #521. Surprisingly the reproducer #522 failed to address this issue. I am still working on a true reproducer.

Description

Related Issue

Motivation and Context

The integration tests timed out in two of the environments, have we seen this before?

Travis CI sometimes has some problems. Simply restarting the jobs should do it.

I looked for the restart button but I couldn't find it. Perhaps its limited to maintainers?


[Cody completion]: Improve trailing completion based on suffix string

Fixes https://github.com/sourcegraph/cody/issues/264
Prior to this PR, we cut off some trailing parts of completion based on the number of connected characters sequence in the begining of the suffix part (see this PR for more details https://github.com/sourcegraph/cody/pull/108). In this PR, we do simular same logic, but we try to compare strings a bit deeper based on a minimum number of edits operation over strings. If there is more than 30% of differences, we won't cut off the trailing completion. Otherwise, we cut off the completion string. This should handle cases like
// Todo: cancel network request
const abortController = new AbortController() // ← completion
const networkAbortController = new AbortController()

To be clear, even this comparing isn't ideal because there will be some corner cases when we will or won't cut off parts that same/different compared to the first suffix string. But it should cover more cases than the current logic.
Test plan

Testing is the most tricky part here, but we need to find places where Cody generates completion similar to the suffix (next string after insertion happens). You can find a few examples of this behaviour in the issue thread https://github.com/sourcegraph/cody/issues/264

Ok, it looks like everything is ready to merge, I hope this will work as we think it works =)


Variable naming in to_bdd()

@mvcisback
Renamer only accepts integers in the line below.
input_refs_to_var = {ref: renamer(i) for i, ref in enumerate(circ.inputs)}
Ideally, calling the line below or a similar variant would yield a bdd with variable names that are identical to the names in aiger circuit.
to_bdd(aagcircuit, renamer = lambda x: str(x))
Otherwise, we need to get the desired outcome above we need to rename with the dd interface after calling to_bdd.

@ericskim
renamer now has the following signature:
def renamer(idx: int, ref: str) -> str:
    pass

So to get the names in the aiger circuit, you can use:
to_bdd(aagcircuit, renamer = lambda x: str(x[0]))
I still pass in the index in case you want to use the index in your renaming.


removed method badconn.badConnError.Unwrap

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report

Merging #347 (c03082c) into master (5eedcb8) will decrease coverage by 24.45%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #347       +/-   ##
===========================================
- Coverage   59.80%   35.34%   -24.46%
===========================================
  Files         205      113       -92
  Lines       22153    15859     -6294
===========================================
- Hits        13248     5605     -7643
- Misses       8099    10032     +1933
+ Partials      806      222      -584

Flag
Coverage Δ

1.16.x
35.34% <ø> (-24.03%)
:arrow_down:

1.18.x
?

1.19.x
?

connection
?

database_sql
?

discovery
?

e2e
?

integration
?

macOS-latest
35.34% <ø> (-0.39%)
:arrow_down:

ratelimiter
?

scripting
?

table
?

topic
?

ubuntu-latest
?

unit
35.34% <ø> (-0.49%)
:arrow_down:

windows-latest
?

Flags with carried forward coverage won't be shown. Click here to find out more.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[bytecode verifier] Fix stack management in Reference Safety

Motivation

Fixed stack balancing issues in the new borrow checker implementation
@wrwg This will fix your bug

Test Plan

Ran em
new test
verified problematic code was fixed

@dariorussi r=darioncassel

@bors-libra r=dariorussi

:pushpin: Commit f139202 has been approved by dariorussi

:hourglass: Testing commit f13920232dbf5af3a82e8863752aaa42a692edff with merge 91eceb8fdd5638f7852482bf8040135044798984...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: dariorussi
Pushing 91eceb8fdd5638f7852482bf8040135044798984 to master...

Thanks for the fast fix!


[REQUEST] Support for the new Command-r7b

Problem
Hello, can we, please, get support for the 128k context Command-r7b, I would like to use it both as fast tool selection based on user prompt, before calling the big Command-r brother, and also as a draft model accelerator for Command-r or Command-r plus.
Solution
An exl2 supporting the full 128k that can be also used as draft model.
Alternatives
No response
Explanation
Faster Command-r generation and almost instant tool selection.
Examples
No response
Additional context
No response
Acknowledgements

[X] I have looked for similar requests before submitting this one.
[X] I understand that the developers have lives and my issue will be answered when possible.
[X] I understand the developers of this program are human, and I will make my requests politely.

I've added support in the dev branch, or at least an attempt at it. It appears the architecture is identical to Cohere with the exception of SWA on all but every 4th layer.
It seems to be coherent at least up to the model's native 8k context limit, but although the readme mentions a 128k context length, I don't see any hint of that in the model's config, or in the HF implementation. Is it supposed to use YaRN?

Hi, thank you for your work on this. Can it be only a config default value issue as it was for previousCommand-r?  https://huggingface.co/CohereForAI/c4ai-command-r-v01/discussions/12

It might be. You can always try just giving it a longer max_seq_len to override the default. I tested it a bit as just an instruct model, using the old Cohere template, and it does eventually turn incoherent, but I'm not sure it's supposed to be used this way. I.e. it might be fine producing tool calls from a 100k prompt.

I don't know how to build the windows exllama from the dev branch to be able to test.. I am using it via tabbyapi..

There should be a new release shortly.

0.2.7 is released now if you missed it. Feel free to open another issue if something is still broken.

Thank you! I will test it and let you know if any issues!


Odd behavior with Python project, empty package name

release-please(-action) results in some odd behavior with this revision of this project:
https://github.com/scop/pytekukko/tree/f0e87b16737d1cbbcf25dc7e527223ed87921261
Per the job log at https://github.com/scop/pytekukko/runs/4436620942?check_suite_focus=true#step:3:18, we're unsuccessfully trying to look for __init__.py and src//__init__.py, meaning projectName must have been empty.
But if it was empty, IIUC we should not have entered the code block that adds those files to the mix in the first place:
https://github.com/googleapis/release-please/blob/d8bb7caddfa14aabd3bfa19008c10ed911638a66/src/strategies/python.ts#L83
...i.e. !projectName would have evaluated to true. But it didn't, for some reason the branch with "No project/component found." log message was not taken, but rather the else branch was.
I understand I should have probably given a package-name in the actions config or added a [project] section in pyproject.toml so that projectName would have been initialized to something useful. But that's not the point of this report, rather I'm wondering why an empty projectName resulted in the above behavior. What made !projectName for an empty projectName evaluate to false, what am I missing?

@scop you are looking at main, but release-please-action@v2 doesn't use that. The relevant version in the action is v12.6.0, which didn't check for a non-null projectName eg https://github.com/googleapis/release-please/blob/9850c20a7d0dce0d6ca0b14c9cc45d41752f36fc/src/releasers/python.ts#L111
Those warnings were bugging me as well. They have got resolved in v13.0.0
If you wanted to test the latest action, please check https://github.com/google-github-actions/release-please-action/pull/398/files and specify the SHA of that

Ah! Makes sense now. I can live with this, need to add package-name anyway until the tooling around (not release-please but other stuff) is more ready for specifying everything in just pyproject.toml. Thanks!


Fix test cases

Closes #199.

cc @Mainak431 could you adjust mean_filter tests to match these. I may have gave some bad advice on running all tests in graph/eager for example the None shape dimension would only occur in graph mode so you can decorate that test with compat.v1 instead of disable eager which affects the subsequent tests. The other fixes should be described in #199

Thanks @WindQAQ


Nvui-* commands and related helps did not load

I tried to use nvui in working, but nvui at my work computer did not include all Nvui-* commands and related helps. But nvui and neovim (without plugins) is working correctly.

But I did not get this problem on my personal computer.
The version of nvui is 0.2.1, download from GitHub release.

The latest version that builds from GitHub Action is working correctly.


snippetsPreventQuickSuggestions may just be a "Bug"

I previously asked the Git Hub and Stack Overflow about the circumstances under which snippetsPreventQuickSuggestions need to be set to true, but I didn’t get any examples, so I have reason to think that the so-called "other side" people simply don't exist.
https://github.com/microsoft/vscode/issues/102580
https://stackoverflow.com/questions/62922172/vscode-why-is-the-default-setting-snippetspreventquicksuggestions-changed-to-tru
I know this is verbose, but if you can't give a scenario where you need to set snippetsPreventQuickSuggestions to true, then snippetsPreventQuickSuggestions can only be considered an unreasonable bug.

/duplicate of https://github.com/microsoft/vscode/issues/102580


Feat: add support for initializing vecs client with custom schema

What kind of change does this PR introduce?
Feature: add support for connecting to a user-specified database schema  (instead of hard-coding vecs).
This was accomplished by adding the schema argument to the Client class and replacing all hard-coded references to schema vecs.
Also added schema parameter to create_client method. For backwards compatibility, it still defaults to vecs.
What is the current behavior?
Currently, the client defaults to schema vecs. Users may want to place vecs collections under a schema according to their own naming conventions.
Addresses #62
Please link any relevant issues here.
What is the new behavior?
Users can now instantiate a Client instance with a custom schema. For example:
DB_CONNECTION = "postgresql://<user>:<password>@<host>:<port>/<db_name>"

# create vector store client
vx = vecs.create_client(DB_CONNECTION, schema="my_schema")

Additional context
Add any other context or screenshots.
Test coverage attached

@olirice  thanks for the feedback! sure, I'm definitely up to implement that change.

hey @olirice, made some updates. let me know your thoughts, thanks!

looks great. if we can address those last few concerns this will merge

I'm looking forward to this PR being merged!

hey, thanks @olirice for the review! will finish this up this week


Fix and improve timezone cache concurrency

Instead of doing homegrown caching using CSV files and having to handle tricky concurrency issues use built in python module SQLite (simple SQL-database) for the ticker/timezone-cache.
Hopefully this should lead to less issues and better performance.
I did some performance tests and on my machine (*) adding 1000 tickers and reading them back took 1s. And 2:nd run when they did not need to be added to cache it took 100ms.
Also did some test with running concurrent python scripts reading/writing the same db and it seems to work well.

Windows 10, Python 3.8 AMD Ryzen 5 3600 and SSD)

def main():
    res_list = []
    for i in range(1000):
        k = f"key_{i}"
        v = f"val_{i}"
        res = tz_db.get(k)
        if not res:
            tz_db.set(k, v)
        res_list.append(tz_db.get(k))
    return res_list

if __name__ == '__main__':
    start = time.time()
    main()
    end = time.time()
    print(f"Took: {end-start}")

First run and no existing db or empty:
Took: 1.1271133422851562

Second run when all keys was found:
Took: 0.015013933181762695

Thanks for this! But can you make some changes for me please:

as sqlite is thread-safe then cache_mutex can be removed
when creating DB first time, if CSV already exists copy data over. And then delete the CSV.
add sqlite to requirements.txt

Thanks for the quick feedback!

as sqlite is thread-safe then cache_mutex can be removed

I had to set the "check_same_thread" to False to be able to have different threads read/write. According to the documentation you then have to take care of the serialization yourself:

check_same_thread (bool) – If True (default), only the creating thread may use the connection. If False, the connection may be shared across multiple threads; if so, write operations should be serialized by the user to avoid data corruption

when creating DB first time, if CSV already exists copy data over. And then delete the CSV

Sure thing, will fix that.
One other thing, should it changed to lazy creation of the database until it is needed or is it ok to have it on module import as it is now?
Thank you for working on yfinance, been using it alot lately.

Lazy. Probably some users never use price data.

Have updated the PR with lazy init and migration for old tz cache.
Refactored the cache code a bit and also introduced type hints for _KVStore class. Not sure what your stance are on type hinting if it is ok or not to use in the codebase. Not a big fan myself but it makes sense in some cases.

Also let me know if you think the cache code should be broken out to a separate module instead of utils.

weak opinion on type hinting - don't use myself but looks sensible.
I think cache can stay in utils now. But if any future significant additions then probably will move.

So looks ready for a merge. Let me know if anything else on your mind, otherwise I'll merge it in.

Ok, thanks
Feel free to merge.


Increase default HTTP client max pool size

During some perf-testing for KafkaSource, we've observed
that the Vert.x's default for maxPoolSize is too low
for parallel event dispatch.
This is just changing the default values to improve the out
of the box experience and the common use case, people can
still change it.
Signed-off-by: Pierangelo Di Pilato<EMAIL_ADDRESS>

/cc @aavarghese @aslom

/cherry-pick release-1.6

/retest-requied

/retest-required

@pierDipi @aslom Similar to max.poll.interval.ms , this value gets overridden for atleast Source V2 (with scheduler) implementation here . Inform that via a comment? Todo: need to change that size to #ofpartitions instead of vreps

@pierDipi @aslom Similar to max.poll.interval.ms , this value gets overridden for atleast Source V2 (with scheduler) implementation here . Inform that via a comment? Todo: need to change that size to #ofpartitions instead of vreps

This makes can we capture it in a different issue?

@pierDipi @aslom Similar to max.poll.interval.ms , this value gets overridden for atleast Source V2 (with scheduler) implementation here . Inform that via a comment? Todo: need to change that size to #ofpartitions instead of vreps

This makes can we capture it in a different issue?

Sure, one of us will remember now. :)

/retest-required

/retest

/test upgrade-tests_eventing-kafka-broker_main


Http Status codes on repost should be 409 and not 500

Hi,
What a great tool! and what a list of issues... :) I will add one to the queue, since I am not sure if you accept pull request or agree on the solution.
When I post elements without id the browser accepts them and assigns an id. Just as expected.
When I post elements having an 'id' the browser accepts them and stores them under the id. Just as expected.
However, when I repost an element with an id all ready in the collection I get HTTP 500 Error: Insert failed, duplicate id. Not what I expected.
I will suggest replacing the error code with HTTP 409 Conflict: Insert failed, duplicate id.
The main argument is that 5xx is server errors that the client can not fix. But re-posting is a client behaviour problem, and can be easily fixed by sending with another id, or leave it without.
Thanks for a great piece of software!

Hi, I would like to contribute for this issue. Could you assign it to me?


Color dot dissapears when adding a custom label

STR

Add legend.

Add a custom label.

The color disappears from the legend dot.

Could you take a look @nobuti please 💋 ?


Flexx doesn't work on Windows 10

Starting apps doesn't work for me on Windows 10.
However, using the same code, it works on ubuntu (wsl2).
This is the code:
from flexx import flx

class Example(flx.Widget):
    def init(self):
        flx.Label(text='hello world')

if __name__ == "__main__":
    app = flx.App(Example, title='Test')
    app.launch()
    flx.run()

Using Python 3.8 the app.launch() line raises a NotImplementError, using Python 3.7 I get the following error:

Is there a fix?

Just a note: Your code works perfectly with Python 3.6 and Windows 10.
I do not have 3.8 to test. Maybe someone else can try.

@Telcrome what version of webruntime are you using? You could try pip install -U webruntime to get the latest.

Update code around the first line to:
import sys import asyncio from flexx import flx if sys.platform == 'win32':     asyncio.set_event_loop_policy(asyncio.WindowsSelectorEventLoopPolicy())
The problem is discussed in #619

This solved the NotImplementedError and the event loop starts and the application works as expected.
In the documentation the way to run as a desktop app is described as app.launch('app').
This (and using app.launch()) yields the error message from my initial post along with the warning:
Using the Firefox web runtime and Qt (PySide/PyQt4/PyQt5) together may cause problems.
Which is weird to someone experimenting with flexx,
because I never specified any of those frameworks, nor do I use firefox to run the app.
Still, uninstalling Firefox solves both issues for me

@Telcrome what version of webruntime are you using? You could try pip install -U webruntime to get the latest.

If someone runs into the same issues: I did not test this, because uninstalling firefox and the workaround of @LewisNeal worked for me


Switch to a smaller slugifier.

Currently, we use the slug module for creating URL slugs from titles. The problem is that slug bundles an entire Unicode lookup table, and that gets bundled into our distributable. it currently accounts for 30.5% of our bundle size. This should probably be changed to something a little lighter weight.
Related to #328.

@biilmann suggested using urlize from lib/urlHelper:
https://github.com/netlify/netlify-cms/blob/04ae5051ceafc0c600eace5e88671b2766f81fe6/src/lib/urlHelper.js#L15-L20

The slugify package is a port of slug without the Unicode lookup table.

If urlize can indeed do the trick, we should use it.

Possible slugifier tests:
urlize.spec.js
https://github.com/netlify/netlify-cms/issues/501#issue-245930865
https://github.com/netlify/netlify-cms/pull/414#issue-227276693

So here is what WordPress does slugification:

Accented letters get the accents removed: remove_accents().

All other unsafe characters (Japanese, Chinese) get URL-encoded. Newer browsers will convert them on the fly, so to the user they still appear as normal characters.
The other option here is just to remove the unsafe characters.

Newer browsers will convert them on the fly, so to the user they still appear as normal characters.

Not sure when this conversion would occur - would the resulting filename have these characters encoded?

Not sure when this conversion would occur - would the resulting filename have these characters encoded?
@erquhart Yes, in almost every case they would have to.

Just testing -- both DatoCMS and Forestry strip unsafe chars from the slug.

If it's par for the course and simpler, let's do it.

So we have three options:

Stay with the current slug package, just tell it not to worry about converting emojis (this would remove the hash table).
Switch to the slugify package, it is pretty much the same expect more maintained and without hash table.
Use the urlize function that Matt ported, use the remove-accents package on top.

I also think we should throw an error if the slug is going to be blank.

@biilmann @erquhart I'm trying to understand why the urlize function seems to leave some reserved characters in the URL. The list of RFC3986 unreserved characters is \w-._~, but urlize uses \w./\_#+-. Obviously those need to be kept for the entire URLs, but not for slugification. Is there something that I am missing here?


Compilation error with Trunk

Using version 0.7.0 compiling with cargo works fine. However, when attempting to build as WASM with trunk, I'm getting errors with visibility in Tokio.
error[E0603]: module 'runtime' is private    --> /Users/logankonopatzke/.cargo/registry/src/index.crates.io-6f17d22bba15001f/walkers-0.7.0/src/tokio.rs:16:34     | 16  |             let runtime = tokio::runtime::Builder::new_current_thread()     |                                  ^^^^^^^ private module     | note: the module 'runtime' is defined here    --> /Users/logankonopatzke/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tokio-1.29.1/src/lib.rs:540:5     | 540 |     pub(crate) mod runtime;
This occurs when doing trunk serve or trunk build on MacOS.

WASM is not supported at the moment, see https://github.com/podusowski/walkers/issues/34

Gotcha, thank you. I'll go ahead and close this.


Unable to load some GLB files

Description
When I load a particular GLB file, I get the error:
System.AccessViolationException: 'Attempted to read or write protected memory. This is often an indication that other memory is corrupt.'
This doesn't seem to happen for all GLB files. I've tried two files. One works but the other doesn't. These GLB files are examples provided by KhronosGroup:

OK: https://github.com/KhronosGroup/glTF-Sample-Models/blob/master/2.0/AnimatedMorphCube/glTF-Binary/AnimatedMorphCube.glb
NOT OK: https://github.com/KhronosGroup/glTF-Sample-Models/blob/master/2.0/Fox/glTF-Binary/Fox.glb

The model AnimatedMorphCube.glb loads fine. But I can an error for the fox model when calling
Model.FromFile("Fox.glb");
I've loaded both GLB files into this viewer (https://sandbox.babylonjs.com/), and both seem to load fine without error.
Platform / Environment
I am running the flatscreen simulator on a Windows machine (Windows 10). This is C# code. I began the project with the StereoKitUniversal template.
I wasn't able to attach the solution to this bug report (file too big). But here is a repo to repro the bug - https://github.com/jackdaus/SKDebug. (Maybe there is a better way to share the code in the future? just lmk)
Logs or exception details
[SK diagnostic] Initializing StereoKit v0.3.6 Win32 x64...
 [XR_ERROR_FORM_FACTOR_UNAVAILABLE]
[SK info] MixedReality display mode failed, falling back to Flatscreen
[SK diagnostic] Starting flatscreen mode
[SK diagnostic] Created swapchain: 1280x720 color:rgba32_linear depth:depth32
[SK diagnostic] Initializing Audio
[SK diagnostic] ISAC audio backend not available, falling back to miniaudio! It's likely the device doesn't have Windows Sonic enabled, which can be found under Settings->Sound->Device Properties->Spatial Sound.
[SK info] Using audio backend: WASAPI
[SK diagnostic] Initializing Defaults
[SK diagnostic] Initializing World
[SK diagnostic] Initializing Sprites
[SK diagnostic] Initializing Lines
[SK diagnostic] Initializing UI
[SK diagnostic] Initializing Renderer
[SK diagnostic] Initializing Physics
[SK diagnostic] Initializing Input
[SK info] Initialization successful
[SK warning] Animated weights unsupported
Fatal error. System.AccessViolationException: Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
Repeat 2 times:
--------------------------------
   at StereoKit.NativeAPI.model_create_file(Byte[], IntPtr)
--------------------------------
   at StereoKit.Model.FromFile(System.String, StereoKit.Shader)
   at StereoKitApp.App.Init()
   at SKLoader.Main(System.String[])

Okay, it looks like the fox model has no index buffer! The GLTF spec apparently says that's okay, so I added some code to generate indices to match a purely vertex based mesh. Works great! This is currently on /develop and will be in the next preview build. I'm not in a rush for the next preview build, so let me know how critical this feature is and when you need it, and I can set that up to build sooner :)

SK's GLTF loader is relatively robust by now, but there are still edge-cases in it. I need to spend some time and set up testing to ensure it's completely compliant, maybe support some additional extensions, but right now I'm handling cases as they come! So if you encounter more of these in the future, just link the GLTF and I can load it up and make it work :)

Okay, v0.3.7-preview.1 is up now and contains fixes for this! I'll close this issue now, but please feel free to re-open it if the problem persists :)


Get http://localhost:8080: read tcp [::1]:53942->[::1]:8080: read: connection reset by peer

Describe the bug
I am running the test described in this repo
To Reproduce
Steps to reproduce the behavior:

go get -u github.com/myzhan/boomer
go run server.go
locust -f dummy.py --master --master-bind-host=<IP_ADDRESS> --master-bind-port=5557
go build -o a.out urlbash.go
./a.out --request-increase-rate 10 --max-rps 1000
--master-host=<IP_ADDRESS> --master-port=5557 --rpc=zeromq
http://localhost:8089
Set a 1000 users with a hatch rate of 10 (not sure how this interacts with the new flags)

Expected behavior
10 requests per second up to 1000 RPS
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: macOS Sierra
Go Version: go version go1.10 darwin/amd64

Seems ok with the --no-web flag. Not sure whether to use zeromq or tcp. RPS on the server and Locust are about 80 RPS out.

Thanks for the clarification. I'll think I'll run with --no-web and amend the calculations on the go server.


CSS not refreshed with ember server when children less are modified

My Ember2 project is configured to use pods.
So I have this structure:
├── ember-pod/
│    ├── app/
│        ├── adapters/
│        ├── models/
│        ├── pods/
│        │   ├── components/
│        │   │   ├── list-item/
│        │   │   │    ├── style.less
│        │   ├── route1/
│        ├── styles/
│            ├── app.less
In app.less I have:
@import app/pods/components/list-item/style
Then I run: ember server
When I modify the app.less file, the change is detected, the browser is refreshed, the CSS includes my change.
When I modify the style.less file, the change is detected, the browser is refreshed, the CSS does not includes my change. (I have to stop and restart ember server)
Is there something to do in the configuration?

The current version of PODs does not support having styles in the PODs structure. All styles must be in the styles folder, and only app.* is consumed i.e. import sub files into app.less

Is it an issue with this package or with Ember?

With ember-cli, not so much an issue, just not implemented yet. Check out https://github.com/ebryn/ember-component-css, it might help in the interim.


Expected BEGIN_OBJECT but was BEGIN_ARRAY at line 1 column 2 - When posting new Statement

Guys,
I am trying to use jxapi with OpenLrs- which is an open source LRS written in Java. So when I try to Post a single statement (or multiple statements for that matter), it returns Json response as Jsonarray. For eg:
For a post statement below:
{
  "actor": {
    "name": "Sally Glider",
    "mbox"<EMAIL_ADDRESS>  },
  "verb": {
    "id": "http://adlnet.gov/expapi/verbs/experienced",
    "display": { "en-US": "experienced" }
  },
  "object": {
    "id": "http://example.com/activities/solo-hang-gliding",
    "definition": {
      "name": { "en-US": "Solo Hang Gliding" }
    }
  }
}

OpenLrs' response is:
[
  "c250f7a5-859b-4080-a8cb-0972fa76b25d"
]

Now this throws an error:
Expected BEGIN_OBJECT but was BEGIN_ARRAY at line 1 column 2
in the class-StatementClient in the function:
public String postStatement(Statement statement)
at line:
JsonArray jsonResult = gson.fromJson(result, JsonArray.class);
I even tried to convert the response to Jsonarray at the source- Openlrs before returning, but it still throws the same error.
Could someone tell me what's the expected response? In the specification, there is no hint given except that it says the POST should return an array of id, which as I see it, the Openlrs does follow suit? Could anyone please help me? Thanks

The jxapi postStatement method will return a single ID, and it's a string.
String publishedId = statementClient.postStatement(statement);

The postStatements method will return an ID for each statement in an ArrayList.
ArrayList<String> publishedIds = statementClient.postStatements(statements);

Hi,
Thanks for the reply. I still have a doubt on this. I agree that:
String publishedId = statementClient.postStatement(statement);
returns a single ID which is a string. But, in the code StatementClient.postStatement(statement), you still parse an array of IDs and return the first ID:
JsonArray jsonResult = gson.fromJson(result, JsonArray.class);
return jsonResult.get(0).getAsString();

So basically both statementClient.postStatement(statement) and statementClient.postStatements(statements) expect results with same JSON format from LRS. My question is what is this format should be, because:

[
  "c250f7a5-859b-4080-a8cb-0972fa76b25d"
]

for statementClient.postStatement(statement), throws an exception. For example, could you tell me what should be a return JSON object from the LRS for a statement:
{
  "actor": {
    "name": "Sally Glider",
    "mbox"<EMAIL_ADDRESS>  },
  "verb": {
    "id": "http://adlnet.gov/expapi/verbs/experienced",
    "display": { "en-US": "experienced" }
  },
  "object": {
    "id": "http://example.com/activities/solo-hang-gliding",
    "definition": {
      "name": { "en-US": "Solo Hang Gliding" }
    }
  }
}

Based on that I will change the LRS side response code. Note that the statement is successfully stored in the LRS and the Statement ID is returned. The problem is only in the parsing of the JSON response at line:
JsonArray jsonResult = gson.fromJson(result, JsonArray.class);
Thanks

Whenever one or more statements are POSTed, the LRS must return an array of IDs, like you described. So if a single statement is posted, the LRS should return ["c250f7a5-859b-4080-a8cb-0972fa76b25d"]. jXAPI then just parses that result and returns the single ID as a string for usability. Have you tried POSTing to a different LRS to see if the error still occurs?

Hi,
I tried to POSTed a statement to

https://lrs.adlnet.gov/xAPI/

LRS successfully stored the statement and returned the ID. And the jxApi still threw an JsonSyntaxException as mentioned above.
So guess its a bug?
You can find the following statement at http://adlnet.github.io/xapi-statement-viewer/:
{
  "verb": {
    "id": "http://adlnet.gov/expapi/verbs/launched",
    "display": {
      "en-US": "launched"
    }
  },
  "version": "1.0.0",
  "timestamp": "2017-06-05T12:53:27.836827+00:00",
  "object": {
    "id": "https://www.youtube.com/watch?v=tlBbt5niQto",
    "objectType": "Activity"
  },
  "actor": {
    "mbox"<EMAIL_ADDRESS>    "objectType": "Agent"
  },
  "stored": "2017-06-05T12:53:27.836827+00:00",
  "authority": {
    "mbox"<EMAIL_ADDRESS>    "name": "xapi-tools",
    "objectType": "Agent"
  },
  "id": "a14600cc-4e39-483b-b040-0aa0ae94f04f"
}

This test posts and returns an ID with no exceptions thrown.
StatementClient client = new StatementClient(lrs_uri, username, password);

Statement st = new Statement(new Agent("test", mbox),
				new Verb("http://adlnet.gov/expapi/verbs/launched"),
				new Activity("https://www.youtube.com/watch?v=tlBbt5niQto"));

String publishedId = client.postStatement(st);

It returns d03a76ac-c3e6-4e3d-8354-ce7bc6898489
The statement stored:
{
  "verb": {
    "id": "http://adlnet.gov/expapi/verbs/launched"
  },
  "version": "1.0.0",
  "timestamp": "2017-06-05T13:18:12.401968+00:00",
  "object": {
    "id": "https://www.youtube.com/watch?v=tlBbt5niQto",
    "objectType": "Activity"
  },
  "actor": {
    "mbox"<EMAIL_ADDRESS>    "name": "test",
    "objectType": "Agent"
  },
  "stored": "2017-06-05T13:18:12.401968+00:00",
  "authority": {
    "mbox"<EMAIL_ADDRESS>    "objectType": "Agent"
  },
  "id": "d03a76ac-c3e6-4e3d-8354-ce7bc6898489"
}

Is there a public link to your LRS for me to test against?

Originally I had my own local server from compiled from:

https://github.com/Apereo-Learning-Analytics-Initiative/OpenLRS

But I also tested with the ADL's LRS. The following code also throws an exception:
Statement statement = new Statement(new Agent("test", "mailto:jxapi@example.com"), new                Verb("http://adlnet.gov/expapi/verbs/launched"),
				    new Activity("https://www.youtube.com/watch?v=tlBbt5niQto"));

StatementClient client=new StatementClient("https://lrs.adlnet.gov/xapi/", "xapi-tools",
					"xapi-tools");

			// send the statement, if successful store the uuid for future
			// references
			try {
				String publishedId = client.postStatement(statement);

		} catch (JsonSyntaxException e) {
			// ignore due to a bug
			System.out.println("Exception while reading the Id!!");

		}

I got the same exception when sending a xAPI statement to a Learning Locker instance. Anyway, the statement is properly saved in the store.


Updates graphql & peerDependencies in package.json for graphql

Addresses #8

this isn't quite right – we should actually use the literal null support and drop < 0.8.0 support entirely

or at least test for it explicitly, if covered


updated to latest node version (lts)

Updated to latest node 14 (coming today lts)
``
The once experimental feature, Diagnostic Reporting, is now stable!
V8 has been upgraded to V8 8.1
Node Streams have been updated
Removal of Experimental Modules Warning

https://auth0.com/blog/nodejs-v-14-whats-new/

need to bump the node version once it lands later today, but it should be all ready to go

gonna just sit on this for a bit more, want to confirm it does really work


Implemented Ad Progress detector

Changes

Implemented Ad Progress detector.

Workflow tests

@VerizonAdPlatforms/video-partner-sdk-developers: Please review.

PR is updated and RFR.


Add proxy upgrade contract example w HTS

Problem
Hedera is able to support upgradeable contracts using the proxy method.
However, no example exists in this repo yet
Solution
Add

a set of contracts example under contracts/proxy-upgrade that showcase how an upgradeable contract may be utilized w HTS logic e.g. very simple DEX ref.
unit tests that show valid contract execution
[Stretch] manual upgrade flow in a test that verifies eventual updated logic

Note there may be some dev tool limitations due to CREATE 1 logic by Hedera vs Ethereum but this should either have documentation or a work around provided.
Alternatives
No response

Blocked by https://github.com/hashgraph/hedera-json-rpc-relay/issues/703


Cannot read some jpeg files

This image is from COCO 2017 valid, but jpegio cannot read this file. There are several files in COCO that jpegio cannot read.
jpegio.read("000000095843.jpg")
gives
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xa8 in position 31:  invalid start byte
Although, PIL can read this file.

Could you tell me about your working environment such as OS, python version, etc?

@dwgoon It's Windows 10, python 3.6. Now I've tried in python 3.7 and got the exact same error.
Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "C:\Users\Frank\Anaconda3\envs\temp\lib\site-packages\jpegio-0.2.1-py3.7-win-amd64.egg\jpegio\io.py", line 8, in read     obj.read(fpath)   File "jpegio\decompressedjpeg.pyx", line 34, in jpegio.decompressedjpeg.DecompressedJpeg.read   File "jpegio\decompressedjpeg.pyx", line 47, in jpegio.decompressedjpeg.DecompressedJpeg.read   File "jpegio\decompressedjpeg.pyx", line 104, in jpegio.decompressedjpeg.DecompressedJpeg._read_markers UnicodeDecodeError: 'utf-8' codec can't decode byte 0xa8 in position 31: invalid start byte

I have changed the type of DecompressedJpeg.markers. From now on, users should decode the bytes to str using an appropriate encoding scheme.

@dwgoon Thanks for fixing. It works.

That sounds great :)


Introducing Microstate.from

Microstate.create allows to build an instance of a microstate from a Type and a value. Microstates will use this Type to build a data structure with all of its transitions and state. This works great when the data structure is known ahead of time and you use it enough to warrant declaring the structure of your data before calling Microstate.create.
What if you have a POJO that you need to transition but you don't have its type?
Before this PR, you were out of luck. This PR introducing Microstate.from which takes a value and builds a microstate that you can call transitions on. Microstate.from doesn't currently have any way to match custom types from your application but it can detect primitive values and provide transitions for that type.
For example, you can do
import { from } from 'microstate';

from(42).increment();
from(true).toggle();
from('hello world').concat('!!!');

You can also pass deeply nested POJOs.
from({ a: { b: { c: 42 } } }).a.b.c.increment().valueOf();
// => { a: { b: { c: 42 } } }

You can do,
from([[[false]]])[0][0][0].toggle().valueOf();
// => [[[true]]]

Coverage increased (+0.3%) to 89.694% when pulling 234d50e17b148dab79e91d52f77c2b6ee7590ba0 on from into d7e45d2f68587f347bd11a690f78c3dc4629158a on master.

Coverage increased (+0.3%) to 89.694% when pulling 234d50e17b148dab79e91d52f77c2b6ee7590ba0 on from into d7e45d2f68587f347bd11a690f78c3dc4629158a on master.

Coverage increased (+0.3%) to 89.694% when pulling 234d50e17b148dab79e91d52f77c2b6ee7590ba0 on from into d7e45d2f68587f347bd11a690f78c3dc4629158a on master.

@cowboyd any objections to merging this branch?

I still need to have a look at it.

Introduced by #113


Consider convenience method to stamp Date header in response

Date can be cached thread-locally and formatted only when formatted value is expected to change (seconds-level granularity).
e.g. Encoder.add_date() or Encoder.add_auto_date()

It might be a good idea to add a method, it's not there because it needs another dependency and it's oneliner. But I'm not sure about caching.
Is it really justified by benchmarks or it's just an idea?

Just compared (code here: https://github.com/nayato/hyper-test/tree/a618125e2ee34900742cdec54d92c28ef021ea34) and in such a basic setup difference is ~18% (154K req/sec without caching Date vs 182K with).

Hm, that's a pretty large difference. Can you also try httpdate insead of time crate? (httpdate has a hardcoded date format, so might be faster). We might also just need to add inline(always) to format_header (because it's in the different crate it might not get inlined).
BTW, how tk-http compares to hyper? (I see example in your repo)

Hyper - 223K, same functionality.

tried non-cached httpdate and re-ran benchmark on a different VM.
hyper: 212K
tk-http, cached date using time crate: 201K
tk-http using httpdate (no caching added): 186K

194K with httpdate, no caching when with this change https://github.com/nayato/hyper-test/commit/5590d994f9ff4bc7bad008f6770fbd7b79c15599

Okay with this kind of difference there is much less sense for caching. In more real workload it should not be noticeable at all.
Still it's unfortunate that tk-http is slower than hyper :) How do you benchmark? It would be nice that I could replicate the test and see if something might need to be fixed in tk-http. Also, typical benchmarks like ab tend to send  two request headers, while normal browser send ~10, and that may be important since the two libraries handle headers in a very different way.

I benchmarked with wrk, without pipelining, on two Ubuntu 16.04 VMs in Azure - 4 core server (D3v2), 16 core client (F16, but may go lower here).
I agree on real life perf diff not being big. On dependency mgmt, considering how ubiquitous Date header in HTTP is, I sure wouldn't mind having OOTB support for stamping it. Also, considering how small and focused httpdate is... 😉

We're also going to improve the performance of httpdate itself: https://github.com/pyfisch/httpdate/pull/2


Implement an API to download the artifacts (logs and chaos results) from the litmus

Implement an API to download the artifacts (logs and chaos results) from the litmus.
This will be of great help when the litmus chaos is being used from CLI or some python script to automate the Chaos Testing.

Hi @spshuklaji , this is a good feature request. Would you like to join the Litmus contributors call, where you can pitch this idea to the team. We will decide on the same.


image: Support building packages from list with -b

This should be pretty straight forward, similar to how testport works.
Something like this should do it.  Note that you would need to add -p -z and -j flags for PTNAME, SETNAME and JAILNAME.
: ${BUILD_PARALLEL_JOBS:=${PARALLEL_JOBS}}
: ${PREPARE_PARALLEL_JOBS:=$(echo "scale=0; ${PARALLEL_JOBS} * 1.25 / 1" | bc)}
PARALLEL_JOBS=${PREPARE_PARALLEL_JOBS}

MASTERNAME=${JAILNAME}-${PTNAME}${SETNAME:+-${SETNAME}}
_mastermnt MASTERMNT
export MASTERNAME
export MASTERMNT
export POUDRIERE_BUILD_TYPE=bulk

jail_start ${JAILNAME} ${PTNAME} ${SETNAME}

LISTPORTS=$(cat ${PACKAGELIST})
prepare_ports
markfs prepkg ${MASTERMNT}
POUDRIERE_BUILD_TYPE=bulk parallel_build ${JAILNAME} ${PTNAME} ${SETNAME}
[ ${BUILD_REPO} -eq 1 -a ${nbbuilt} -gt 0 ] && build_repo
commit_packages

@evadot @bapt

was_a_bulk run in common.sh needs to be updated to look for image.sh as well.

… with -b

https://github.com/freebsd/poudriere/blob/1257b1d492ca1db88f7f61d51e1bfa03a2cd9e0a/src/man/poudriere-bulk.8#L145
https://github.com/freebsd/poudriere/blob/1257b1d492ca1db88f7f61d51e1bfa03a2cd9e0a/src/man/poudriere-testport.8#L72