Remove version: inLuShan.Desktop version 0.1.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://inlushan.top/res/desktopApp/inlushanSetup_0_1_0.exe
Response status code does not indicate success: 521 ().
(Automated message - build 897)

Hi @Exorcism0666,
This manifest has the highest version number for this package. Is it available from another location? (This might be in error if the version is switching from semantic to string, or string to semantic.)
(Automated response - build 897.)


(Bug) Transaction confirmation doesnt open native share on mobile

Instead it copies to clipboard "[object] [object]"
on mobile devices button should say share also change text to share instead of copy

@LiavGut fyi

Link to the PR - https://github.com/GoodDollar/GoodDAPP/pull/1685

Text for transfer request link (specific amount)
"Test Quest, You've got a request from test test for 1 G$. To approve transfer open:" and the link is displayed
Name of the button was changed to "Share"
Checked on Dev env (V 0.19.5-0)
The device used:
LG V30+ // Android 8.0 // Google Chrome 81.0.4044.117
iPhone 8 Plus // iOS 13.1.3
Desktop// Windows 10 x64 // Google Chrome 81.0.4044.129
Video: https://www.screencast.com/t/hdTgTQmXvD


listing more than one tomograms

Dear team,
Could you help me with examples how to fill in the train_data_config.json file when you have more than two tomograms please?
When I listed more than two tomograms for even and odd each, in the right order, it seems to complain in where to put the data.npz file. I tried to provide the same number of output "path", but it gave another error. Could you help me giving me how would you write for instance if you have TS_01_evn& odd.mrc, TS_02_evn &odd.mrc, TS_03_evn& odd.mrc please?
"even": List of all even tomograms.
"odd": List of all odd tomograms. Note the order has to be the same as in "even".
"mask": If desired, a list of binary masks to limit where subvolumes are extracted, similar to IsoNet. Can be left out to skip masking.
"patch_shape": Size of the sub-volumes used for training. Should not be smaller than 64, 64, 64.
"num_slices": Number of sub-volumes extracted per tomograms.
"tilt_axis": Tilt-axis of the tomograms. We split the tomogram along this axis to extract train- and validation data separately.
"n_normalization_samples": Number of sub-volumes extracted per tomograms, which are used to compute mean and standard deviation for normalization.
"path": The training and validation data are saved here.

For future reference, you can find an example here https://github.com/juglab/cryoCARE_pip/issues/62
{
"even": [
"/home/tomograms/even/rec_Position_1_2_half2.mrc", "/home/tomograms/even/rec_Position_1_half2.mrc", "/home/tomograms/even/rec_Position_3_3_half2.mrc", "/home/tomograms/even/rec_Position_2_2_half2.mrc", "/home/tomograms/even/rec_Position_6_half2.mrc"
],
"odd": [
"/home/tomograms/odd/rec_Position_1_2_half1.mrc", "/home/tomograms/odd/rec_Position_1_half1.mrc", "/home/tomograms/odd/rec_Position_3_3_half1.mrc", "home/tomograms/odd/rec_Position_2_2_half1.mrc", "/home/tomograms/odd/rec_Position_6_half1.mrc"
],
"patch_shape": [
72,
72,
72
],
"num_slices": 1200,
"split": 0.9,
"tilt_axis": "Y",
"n_normalization_samples": 500,
"path": "/home/cryocareTest"
}


Blob storage link dead

The link in Challenge 2 - Task 1 - Step 4 is dead.
https://logsbenchmark00.blob.core.windows.net/logsbenchmark-onegb/2014/?sp=rl&st=2022-08-18T00:00:00Z&se=2030-01-01T00:00:00Z&spr=https&sv=2021-06-08&sr=c&sig=5pjOow5An3%2BTs5mZ%2FyosJBPtDvV7%2FXfDO8pLEeeylVc%3D

Found there's another link.


Credentials screen requires valid eWelink credentials, does not proceed with "any"

As per the subject.
I've tried entering both a fake phone number and a fake email, in both cases  leaving the password blank, but it always results in the same screen with the "user account XXXXX does not exist" message on top.
Is there a way to use this integration without a eWelink account?
This is with SonoffLAN-3.3.1 in HomeAssistant 2022.11.3

Managed to get past the authentication screen using another browser: issue was on Firefox on Android, worked with Chrome on desktop Linux (entering "+9" on the username screen and leaving blank the pasword).
Closing this issue now.

Blank password for DIY mode


Losing audio while playing on ZonamaDev server

Speccy report - LIGHTSABER.pdf
No issues installing the server or using it, until I completely lose audio. I am using SPDIF for my connection to receiver.
I've attached a couple of files showing system specs and configuration information.

Just disable audio in the virtualbox settings.  You don't need audio to build the code or run server.

Sorry I wasn't clearer but I was losing audio on the host. But I've deleted all files and built another server. So far this time it isn't losing audio. Buy this is the third time I've deleted and redone everything so I'm not sure why it is choosing this time to function properly.
Thank you for responding so quickly.
Sent from my iPhone
On Sep 17, 2017, at 6:17 AM, Lord Kator <notifications<EMAIL_ADDRESS>wrote:
Closed #23https://github.com/Zonama/ZonamaDev/issues/23.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Zonama/ZonamaDev/issues/23#event-1252444804, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AeK7ISaPIQqY1yWUrHg7eGVBu9ua4eijks5sjP_SgaJpZM4PZlrj.

Ahh yes I've experienced this on osx with an older version of virtualbox.  I just upgraded to newer version and it went away without having to destroy the image or anything.


[Version] v1.1.0.

v1.1.0 - Baichuan models supported.
Models

Introduced Baichuan models support and added the convert tool for Baichuan models.

Performance Optimizations

Update xDNN to version 1.2.1 to improve performance of BF16 data type with AMX instruction on 4th generation Intel Xeon Scalable processors.
Improved performance of BF16 data type inference by adding matMul bf16bf16bf16 primitives and optimizing kernel selection strategy.
Improved performance of the model with unbalanced split allocation.

Functionality

Auto-detect oneCCL environment and enter single-rank model if oneCCL does not exist.
Add sample strategy for token search, support temperature, top k, and top P parameter.
Introduced grouped-query attention support for Llama2.
Remove mklml 3rd party dependency.
Add C++ exit function for multi-rank model.
Export normalization and position embedding C++ API, including alibi embedding and rotary embedding.
Introduced XFT_DEBUG_DIR environment value to specify the debug file directory.

BUG fix

Fix runtime issue of oneCCL shared memory model.
Fix path concat issue in convert tools.

@Duyi-Wang  Please create VERSION file to track release info.

@Duyi-Wang Please create VERSION file to track release info.

you mean a version file just containing a version number? or containing the changelog?

@Duyi-Wang Please create VERSION file to track release info.

you mean a version file just containing a version number? or containing the changelog?

Yes. includes version number and containing the changelog.

@Duyi-Wang Please create VERSION file to track release info.

you mean a version file just containing a version number? or containing the changelog?

Yes. includes version number and the changelog.

I will create a version file including the version number. Is changelog necessary? It is included on release page. The PyTorch project use a version file with num https://github.com/pytorch/pytorch/blob/main/version.txt

Need to rebase the base commit.

Need to rebase the base commit.

Wait for PR of convert tool

Add new features:

Introduced prefix sharing feature.
Introduce convert module to xfastertransformer python API.
Auto-detect oneCCL environment in compilation. If not detected, oneCCL will be built from source.


Update GPO to reflect right version

Summary
Update Version within GPO to the correct Desktop App version.
NONE

Hello @JtheBAB,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

/cherry-pick release-5.1

Cherry pick is scheduled.


Import command should also import Pluginfile

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
Doing an import of a Fastfile via import_from_git, the fastfile and actions are imported.
import_from_git(url: 'ssh://......git',  path: 'CommonFastfile')

But the Pluginfile is not imported, I think as the actions are imported it make sense to import the Pluginfile too.

Mh, I don't think that's gonna work, as all Ruby files are already loaded at that point by bundler. You'd have to run bundle update after importing the file, which is not something fastlane can do on its own as far as I know.

@KrauseFx what about fastlane install_plugins what does it do bundle install?
Alternative would import_from_git(url: 'ssh://......git',  path: 'Pluginfile') work (followed by bundle install)?


UI - results - Labels for concept classifying columns not depicted?

From @tomas-knap on December 13, 2016 19:51
When I run Odalic agains ADEQUATe KB, in case of disambiguation, UI properly shows labels (see below, URI contains UUID, but there is nice label). But in case of classificiation, label is not obtained and UUID is used. Why?

Solution: It should also take the label of the concept being suggested for classification.
@KataBoku did you encounter that as well?
Copied from original issue: odalic/odalic-ui#190

From @tomas-knap on December 14, 2016 16:56
So it seems that the problem is that the header column is not able to display labels contained within skos:prefLabel property (although such property may be defined in the config file for the KB - in the enums, but probably applies only to disambiguated entities?)
For disambiguated entities it works correctly.

@tomas-knap @KataBoku This looks more like a server issue, as the UI should just blindly display the label provided by the server. Assigning to @JanVana .

Also in https://grips.semantic-web.at/display/ADEQ/Number+of+new+babies+by+the+cities%2C+districts , the nutsCode (although it is too general class) also does not contain a label.

@josef-janousek Please re-test if still relevant.


Q: When a link is sent, could it be auto-opened in a desktop tab?

cc @xisberto in case you would have wanted to get notified of this and weren't.

There's currently no plans to further update this app.
The auto-open feature would depend on the extension constantly checking the Google Tasks service for new links, and this would require that:

Me, as the publisher, to request Google for larger limits on the API usage, or
Including on the code instructions for creating a Google API Console project and obtaining the app keys for users wanting this functionality.

Neither option is good for me. As the license allows it, others can fork the project, host it on its own Google Console keys and implement those features.

Ah, I see, that's a bummer that this would require polling. I was guessing/hoping that this was based on GCN (Google Cloud Messaging) or similar; relying on push events without polling. Understandable this way.


infinite loop when using placeholderdata and enabled false

Describe the bug
Hi,
During our migration from v3 to v4, we observed an infinite loop with a specific usage of placeholderdata and enabled false, if the placeholderdata is defined directly in the usage of useQuery we have it, if we defined in a constant the placeholderdata and use it in the useQuery the issue gone.
Here the project to reproduce the issue : https://github.com/CedricCouton/react-query-loop
Your minimal, reproducible example
https://github.com/CedricCouton/react-query-loop
Steps to reproduce
Use this repo : https://github.com/CedricCouton/react-query-loop
And use the useQuery as that :
 const response = useQuery(["users"], fetchUsers, { placeholderData: {}, enabled: false});
Expected behavior
Not having the infinite loop
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
All
react-query version
v4
TypeScript version
No response
Additional context
No response

not sure I understand what this code is supposed to be doing. There is a useEffect that runs after every render (no dependencies) that calls setState, which triggers a re-render. What's the purpose of that?

Yes it can seem weird, but React is capable to detect that the state1 is unchanged and stop to render the children of the Parent. Here just adding the row with the useQuery (with placeholderdata and enable false) generate a render of each children because the "response" value have a new reference each render, with the v3 of rq it was not the case.

I don't think response should be a different value each render, because placeholderData is structurally shared. Also, response is unused. Renders "coming from" react-query will only happen when data in the cache changes, which does't happen. So the render is top-down, and likely done by react. Maybe it cannot bail out of the useState update because of useSyncExternalStore in the query, idk. It's weird.

If we use a placeholder data defined outside the component, the issue gone.

If fixed this by always structurally sharing placeholderData. I'm unsure what causes the infinite loop, but this seems like a good fix regardless:
https://codesandbox.io/s/silent-sun-xwe8c0


[BUG] Citizen creation - Driving license categories are incorrectly assigned

What version of Node.js are you using?
does not matter
What version of npm are you using?
No response
What operating system are you using?
does not matter
Describe the Bug

When specifying the categories of driver's and flight licenses, then the flight categories apply to all.
In edit mode, the flight category cannot be deleted from the driver's license.
A citizen with a problem category is not removed

https://user-images.githubusercontent.com/4976837/140659545-f9affa45-1d3c-46f3-b961-11bbcb3dc7bd.mp4
Expected Behavior
--
To Reproduce
No response

I get nightmares about this one 😅, had this in dev too and seemed to be fixed. Will take a look!


MomentJS Note: Months are zero indexed, so January is month 0.

Does it render some inconsistencies?
I have some problems while trying to save dates example.
Received Object in the same Method
Object { day: "12", month: "01", year: "2017", formatted: "12-01-2017", momentObj: Object }
When I try to save in that same Method
Object { uid_date: "killerengel_12-02-2017", name: "task1" date: "12-02-2017"}
What I use to save:
let chosenDate = moment(task.date).utc().format('DD-MM-YYYY'); //this gives me a month ahead
*task.date is the moment object itself.
Object { day: "30", month: "12", year: "2017", formatted: "30-12-2017", momentObj: Object }
Object { uid_date: "killerengel_Invalid date", name: "task2", date: "Invalid date" }

I used task.date.formatted and it worked fine, I feel stupid, sorry...


client is created although ssl certificate is expired

Hi,
I tried using this code to create a kubeclinet:
cert_store = OpenSSL::X509::Store.new
cert_store.add_cert(OpenSSL::X509::Certificate.new(ca_cert_data))
ssl_options = {
cert_store: cert_store,
verify_ssl: OpenSSL::SSL::VERIFY_PEER
}
client = Kubeclient::Client.new(
'https://localhost:8443/api', 'v1', ssl_options: ssl_options
)
with an expired ssl certificate and it worked. should it be like this?

Maybe :-) 🤔
Kubeclient cant generally tell whether a given cert will be valid and authorized for a given cluster.  Creating a client object does not yet make any requests.

I'll add that you can do client.discover to force a request (that would anyway be done before later requests, so not a waste).
However that accesses paths that are open without auth on many k8s versions, so not sure it'll reject the request when client cert is invalid.
@yifatmakias @masayag can we close this?
If you believe there is value in kubeclient checking timestamps, PR welcome.

I'll add that you can do client.discover to force a request (that would anyway be done before later requests, so not a waste). However that accesses paths that are open without auth on many k8s versions, so not sure it'll reject the request when client cert is invalid.
@yifatmakias @masayag can we close this? If you believe there is value in kubeclient checking timestamps, PR welcome.

Yes, you can.


Changed outbound transaction limit to 1,000,000

Changed outbound transaction limit to 1,000,000 and added two contextual sentences.

@ClearlyEleanor - can you please approve this request?  Thanks!


[OIDC] Adding a new publisher fails if the user needs to re-authenticate

Describe the bug
When trying to add a new OIDC publisher (clicking "Add"), I was prompted to re-authenticate. After re-authenticating successfully, I was returned to https://pypi.org/manage/project/disposable-email-domains/settings/publishing/, but the new publisher was not created.
Expected behavior
The new publisher was created.
To Reproduce
For testing in local dev, we'll need to reduce the amount of time that requires re-authentication here: https://github.com/pypi/warehouse/blob/8fec1ba0b60db3e3587d0c640305d464b8bc5440/warehouse/manage/init.py#L22

Taking a look now 🙂

I'll keep looking, but from a quick scan the OIDC management endpoints aren't doing anything different than the other management endpoints in terms of reauth handling -- is it guaranteed that the other reauth-able POST endpoints will perform their original behavior after reauthentication?
(I'll also test that experimentally in a moment.)

is it guaranteed that the other reauth-able POST endpoints will perform their original behavior after reauthentication?

Should be, but maybe it's broken!

I think you're right -- I seem to recall this now, but can't find any related discussion.
Given that, I think it's OK to backburner this until we have a better general solution, rather than try and fix this for the OIDC beta.

@ewdurbin FYI, this affects some organization views as well. This won't affect any views that are currently available but once we allow users to configure billing this could happen for those views.

No, this wasn't resolved.


NPM_CONFIG_PRODUCTION problem

Hi thanks for the repo.
when I run "npm i" I get this error:
'NPM_CONFIG_PRODUCTION' is not recognized as an internal or external command,

Hi thanks for the repo.
when I run "npm i" I get this error:
'NPM_CONFIG_PRODUCTION' is not recognized as an internal or external command,

Hi! I had the same error when I launched the team "npm run build" on server. Did you manage to solve it?

@gakeko2018 @Marlin005
Hi Thanks for the reporting, it because you are in the windows environment, I update the code, so it works for you now, please pull from the original repo again.
git remote add upstream git@github.com:amazingandyyy/mern.git
git pull upstream master

if you

please feel free to reopen an issue if it still doesn't for you, and again, thanks for using this project :D

@gakeko2018 @Marlin005
Hi Thanks for the reporting, it because you are in the windows environment, I update the code, so it works for you now, please pull from the original repo again.
you can now do the following to pull the latest updates
git remote add upstream git@github.com:amazingandyyy/mern.git
git pull upstream master

Thanks for the quick contact! Yes, I ran in the windows enviroment  and there were also problems on the client with launch "npm run build". After installation cygwin it worked.
These configuration variables "NPM_CONFIG_PRODUCTION" are necessary for the platform, as I understand it.  So I installed Heroku CLI.

glad it works, let me know if anything doesn't work on windows!


Archboard Review Followup & Identity Consistency Review (TSG, Docs, Release Notes)

Tracking issue for Azure Identity SDK GA Release for October 2024

[ ] September Archboard Review Followups completed - https://github.com/Azure/azure-sdk/issues/7886 (@ahsonkhan)
[x] Troubleshooting Guide Updated for September Beta Release Features
[ ] Documentation Updated
[ ] Release Notes / Changelog updated

updated api view - https://apiview.dev/Assemblies/Review/fa0cfc224966452897799c82f45e5f8c?revisionId=9d5bd0193b004d77ae33b7281d29d327&diffOnly=False&doc=False&diffRevisionId=c1bb089b69b145a687128e910435591c


Problem when trying to $set object to nulled field

Suppose I have a record {nested: null}. If I try to update it to {nested: {sub: 1}}, I get an exception. The record is properly updated though.
The execution of this deepCopyObjectChanges raises the following exception: Exception in queued task: TypeError: Cannot set property 'sub' of null
Isolated repo here.
Click on clear,save: bug appears.
Click on drop,save: bug won't appear.

@ccortezia it is closed by your pull request?

Yes, we can close it now.


beginners level documentation

This is document explain how to launch your network by following step by step guide. It contains multiple ways to launch and test your network

Please merge latest changes from main
Also rename any references of Kylin node to Kylin Collator on the documentation

Updated the readme file to use kylin-collator

Can you merge the latest changes to resolve the submodule conflicts, ill merge later today


Handle unknown commands

Just to not fail silently. If you have more appropriate message please share :)

I think package.json should be also updated if file name has been changed.
https://github.com/AstarNetwork/swanky-cli/blob/09cfa063b3c35c582191ac6595d8dda0d7c28025/packages/cli/package.json#L86

You've already pointed this out, don't know how I could miss it.


ThrusterPlugin publish angular velocity

Per https://github.com/osrf/lrauv/pull/48#pullrequestreview-698970619 and discussions there, it is helpful for users of the ThrusterPlugin to have access to the angular velocity without having to get it from the ECM. This is best obtained via the JointVelocity component as is done in https://github.com/osrf/lrauv/pull/49
This update can be started in this repo, but should then be upstreamed to ign-gazebo.

See ignitionrobotics/ign-gazebo#1269


Storing several Slider(s) to variables lead to very bad reactiveness

Hello,
Following issue #299 I changed my Pluto.jl notebook to store sliders into 4 variables.
See https://gist.github.com/scls19fr/1486d2732db61d81d4b0021ca5e79481

am_modulation.jl without storing sliders into variable
am_modulation_v2.jl storing each slider into variables

v2 is much less reactive and nearly unusable
With v2 you always need to release left button each time you are changing value.
A fix about this will be great because the v2 code writing is much more convenient (less cells)
Kind regards

Can you record a video that demonstrates the problem?

Ah I see, you case is:
bond = @bind x Slider(1:10)

md"""
Bond: $bond

Value: $x
"""

This is not supported. It's not possible to place the bond (the result of @bind x Slider(1:10)) and the bound value (x) in the same cell.
This is because moving the slider changes x, which causes cells that reference x to re-run. Re-running the md""" cell will show the new value of x, but also re-render the bond, so a new slider is created. The new slider does not have mouse focus so you can't continue sliding.
A workaround for displaying a bond and a bound value in the same cell is to use one of Pluto's built-in combined displays. The main system is in PlutoUI.ExperimentalLayout (hbox, grid, etc). You can also use a DataFrame and put the bond and bound variable in different cells.

I posted for comparison a Marimo notebook with exactly same features
https://gist.github.com/scls19fr/1486d2732db61d81d4b0021ca5e79481#file-am_modulation-py
but that's life ... if I want to use Pluto.jl... I know now that I need to have more cells.

Late to the party (just saw this issue in the release notes), but the show_value = true argument
might be of interest here, when the slider should be in the same cell as its value.

Of course the centering is not right, but that would be another issue.

And like I mentioned, you can use ExperimentalLayout to but bonds and values together. Here is a silly example:
https://github.com/JuliaPluto/PlutoUI.jl/assets/6933510/e2d07618-a4d7-4e9e-bcfd-7cbde0e47d24
# ╔═╡ b4c48706-021f-11ef-1413-059aae52cfc4
using PlutoUI

# ╔═╡ d3d667e4-4e98-44d0-b1f0-cde287afe969
bond = @bind x Slider(1:100);

# ╔═╡ aed4e404-2ca3-4db2-9ac5-d9fd65dbdd98
PlutoUI.ExperimentalLayout.grid([
	bond Text("x² = $(x*x)")
	x    bond
])


Release

Codecov Report

Merging #13 (63e70ad) into master (0beb47b) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##             master      #13   +/-   ##
=========================================
  Coverage     95.95%   95.95%
  Complexity       80       80
=========================================
  Files             6        6
  Lines           321      321
  Branches         49       49
=========================================
  Hits            308      308
  Misses            1        1
  Partials         12       12

Flag
Coverage Δ

kotlin
95.95% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add Llama 3.2 models

Fixes #71
Description of changes:
This PR adds LLama 3.2 models and fixes Llama 3.1 tool_use.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for the contribution!


Create release with binary for different platforms

Why is this not happening?

wonder why this issue was closed...


Better toString()

Resurect @YannRobert 's PR https://github.com/jmxtrans/embedded-jmxtrans/pull/108 that was lost with the force git push.

@YannRobert the force git push closed https://github.com/jmxtrans/embedded-jmxtrans/pull/108
Can you check this? it should be the same

https://github.com/jmxtrans/embedded-jmxtrans/pull/108/files diff looks identical to me.

Replaced by https://github.com/jmxtrans/embedded-jmxtrans/pull/117


Improvement idea: Move "data-import" page to be the landing page "index"

It's one extra click for users to go from the landing page to the data import, which isn't a necessary addition. I think we should move all /data-import starting path to be /, and have the data import page be the Sprout landing page.

Things will get rearranged anyway, so close this for now.


Add https://0xc3.win

Adds https://0xc3.win/
Weighs 151 KB

Fixed a few issues and it's down to 90.6 KB now.
https://yellowlab.tools/result/guffy1wyax

Added separately, welcome to the club!


Enable HTTP Response Logging only for Error responses

Is it possible to log HTTP responses only for status codes other than 2xx?
I would like the HTTP responses to be filtered for status codes 2xx and only log in case of 4xx/5xx etc.
Is there a way, we can configure this in the spring boot application.yaml file under logbook properties?

See https://github.com/zalando/logbook/issues/227

Closing as duplicate.


Custom domain does not reflect css

Goal

[ ] css is reflected on custom domain (https://chatvrm.nisshi.dev/)

Overview
Next.js site deployed on Github Pages does not reflect css with custom domain

ref
https://blog.takunology.jp/entry/2022/07/10/003611


[wip] raft: add flow control for progress

Each progress has a inflighs sliding window. When the progress
is in replicate state, inflights will control the sending speed
of the leader.
The leader can have at most maxInflight number of inflight
messages for each replicate progress. Receving a appResp will
move forward the sliding window. Heartbeat will clear the sliding
window.

@yichengq @bdarnell
I am still working on tests and cleaning up the code. But the overall idea is there, let me know what do you think.

@bdarnell @yichengq
How about this:
clear at most one slot from the window when receiving a heartbeat message from the follower.
We can improve when we have the real time. It is fine with me even to implement the full congestion control if necessary. But I do not want to over-engineering this at this moment.

@xiang90 +1
It becomes more cautious and controls the speed of the flow better.

@yichengq @bdarnell PTAL. Tests are coming.

@yichengq @bdarnell All done. Please take a look. Thanks!

lgtm

@yichengq I am going to wait for @bdarnell's opinion a while since it is kind of an important change that might affect others.

LGTM


Add ID to sign out button

Add id to sign out button for integration tests.

Codecov Report

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

@@          Coverage Diff           @@
##           master     #34   +/-   ##
======================================
  Coverage    99.7%   99.7%
======================================
  Files          20      20
  Lines         334     334
======================================
  Hits          333     333
  Misses          1       1

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 4b26ced...1320cfd. Read the comment docs.


Setting requeue=false

Hi, I'm consuming my rabbitmq message with this (very useful!) client.
I'm going to implement a  Dead Letter Exchanges mechanism as described here. In particular i want to know if is possible to configure the client with the second scenario described:

The message is rejected (basic.reject or basic.nack) with requeue=false,

How can i implements this features?
Thanks in advice
Matteo

Hi @mbiagetti ,
Nice to hear you're using my client and thanks for reporting this issue.
Two things need to be changed to get this working:

change the declaration of the queue so that it support x-dead-letter-exchange and x-dead-letter-routing-key, relevant code can be found in https://github.com/ricbra/rabbitmq-cli-consumer/blob/master/consumer/consumer.go#L118
refactor the consumer itself which is now hardcoded to Ack on success and Nack with requeu on failure.

So I'l have to think of some new configuration settings and implement this feature.

Hi @ricbra !
Thanks for your response.
At the moment I'm investigate howto integrate RabbitMq in our LAMP/Symfony2 application.
We actually integrate RabbitMqBundle for send message.

The Dead Letter Configuration is made on the producer side, like this:
    fleet_report:
        connection:       default
        exchange_options: {name: 'fleet_report', type: direct}
        queue_options:    {name: 'fleet_report', arguments: {'x-message-ttl': ['I', 5000],'x-dead-letter-exchange': ['S', 'error-exchange']}}

So, in reference of the Dead Letter Exchanges mechanism the second and third scenario are already satisfied.
So, I think only the last thing of your list can do that the client, on an error, can deliver the message on the error-exchange.
Tell me if I'm wrong. Any other help is appreciated.
I'm not a go language developer but let me know how and if I can help you!

You can may be use the return code of the program to do different things:

0 = ACK
1 = NACK
2 = NACK + REQUEUE
99 = NACK + Stop consuming and exit

I'm adding the last one because I have a use case where I want to stop consuming (and restart the client with other parameters) in some case.

I also added a config setting for default behaviour on failure (onFailure) so closing this issue for now.


Automate language parsing and translations

Describe Your Changes

Adding script to automate language translations from the Excel file to the required JSONs
Added a make command to create the execution

Non-Obvious Technical Information
Checklist Before Requesting a Review

[x] The code runs successfully.


I thought I specified the correct format, but an error'AcmNotPossible calling acmFormatSuggest' occurs.

I have 4bit compressed adpcm raw data and
I want to decode this into pcm and save it in a wav file.
I have installed the ms-adpcm and ima-adpcm codecs.
As stated in'Enumerate ACM Drivers', the codecs are listed.
ADPCM or PCM related codec will be attached.
codec.txt
I coded it like this:
(mainAudioData is RawData)
var originalFormat = WaveFormat.CreateCustomFormat(WaveFormatEncoding.DviAdpcm, 11025, 1, 5588, 256, 4);
var mainStream = new RawSourceWaveStream(mainAudioData, 0, mainAudioData.Length, originalFormat);
var pcmstream = WaveFormatConversionStream.CreatePcmStream(mainStream); <- error is here.
WaveFileWriter.CreateWaveFile("D:\\output.wav", pcmstream);

originalFormat is obviously on this codec list.(dviadpcm, Samplerate:11025hz, channels:1, AverageBytesPerSecond: 5588, Block Align:256, bits:4)
However, I get the error'AcmNotPossible calling acmFormatSuggest'.
Why is that?

ACM codecs are very fussy about you specifying the exact from and to codecs. You can use the NAudioDemo application to examine byte for byte the valid combinations of input and output WaveFormat structures.


Istio sidecar upstream pod reconnection issue

Describe the bug
I'm running istio 1.0.6 (no mTLS) on our EKS cluster and am experiencing connection issues between two pods that I can't seem to figure out.
I have a gRPC client (abc-client) and server (abc-server) running both written in go. The cluster has automatic sidecar injection enabled, so both have it running.
Everything is working fine if I just start the two services, but whenever the server restarts, the client is not able to reconnect to it. From the restart onwards I continuously get the following error message on the client:
{"level":"error","error":"rpc error: code = Unavailable desc = upstream connect error or disconnect/reset before headers","time":"2019-03-19T12:07:52Z","message":"abc-server service unhealthy"}
Without the istio sidecar the reconnection is working immediately. When I look at the sidecar logs of the abc-client I see a lot of the following messages for the health checks the client is trying to do:
2019-03-19T12:09:41.527Z] "POST /grpc.health.v1.Health/CheckHTTP/2" 200 UF 5 0 1002 - "-" "grpc-go/1.15.0" "6a874d67-9c43-4b67-ad24-7337a26cac8d" "abc-server.abc-namespace:8086" "<IP_ADDRESS>:8086" outbound|8086||abc-server.abc-namespace.svc.cluster.local - <IP_ADDRESS>:8086 <IP_ADDRESS>:44994
My takeaway from these logs is that the proxy is experiencing problems connecting to the upstream server that is not there anymore, hence the UF flag (upstream failure) but is still returning a 200 status code to the client. To me it seems that this behaviour doesn't trigger a reconnection.
I have not any VirtualServices or DestinationRules set up, it's just a simple istio cluster with the sidecars enabled.
What am I missing? Thanks in advance for your help.
Expected behavior
The client should reconnect to the server when it restarts.
Steps to reproduce the bug
Have a gRPC client and server running in kubernetes. Restart the server and see if the client can still access the server.
Version
Istio
Version: 1.0.6
GitRevision: 98598f88f6ee9c1e6b3f03b652d8e0e3cd114fa2
User: root@464fc845-2bf8-11e9-b805-0a580a2c0506
Hub: docker.io/istio
GolangVersion: go1.10.4
BuildStatus: Clean
Kubernetes
Client Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.2", GitCommit:"bb9ffb1654d4a729bb4cec18ff088eacc153c239", GitTreeState:"clean", BuildDate:"2018-08-07T23:17:28Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10+", GitVersion:"v1.10.13-eks-g484b8", GitCommit:"484b857e3134d55ac6373fea2f51798fefa0533f", GitTreeState:"clean", BuildDate:"2019-03-08T05:32:36Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}
Installation
Via Helm
Environment
Dev machine: macOS
Cloud: AWS EKS
Cluster state

The problem is connecting to headless services. My service.yaml looked like this:
apiVersion: v1
kind: Service
metadata:
  name: grpc-test-server
spec:
  type: ClusterIP
  clusterIP: None
  selector:
    app: grpc-test-server
    role: grpc-test-server
  ports:
    - name: grpc
      port: 9999
      targetPort: grpc

As per the issues https://github.com/istio/istio/issues/10659 and https://github.com/istio/istio/issues/7495 istio doesn't work yet with headless services.
So, if you don't need your service to be headless, you can just remove clusterIP: None from the yaml and everything should work as expected.
I still don't know though how to make statefulsets work which require headless services.

The problem is connecting to headless services. My service.yaml looked like this:
apiVersion: v1
kind: Service
metadata:
  name: grpc-test-server
spec:
  type: ClusterIP
  clusterIP: None
  selector:
    app: grpc-test-server
    role: grpc-test-server
  ports:
    - name: grpc
      port: 9999
      targetPort: grpc

As per the issues #10659 and #7495 istio doesn't work yet with headless services.
So, if you don't need your service to be headless, you can just remove clusterIP: None from the yaml and everything should work as expected.
I still don't know though how to make statefulsets work which require headless services.

Battling with this issue aswell,
there's a suggested solution in #7495 - but the solution just doesn't work, there's no TLS communication even with a per-pod ServiceEntry configuration.
It looks like we need to abandon istio until this is fixed or there's a workaround