Ruby: fix credentials for longrunning and tests for clients with no major version

Codecov Report

Merging #1474 into master will increase coverage by 0.01%.
The diff coverage is 96.77%.

@@             Coverage Diff             @@
##             master   #1474      +/-   ##
===========================================
+ Coverage     86.98%     87%   +0.01%
- Complexity     4051    4063      +12
===========================================
  Files           389     391       +2
  Lines         16353   16380      +27
  Branches       1781    1787       +6
===========================================
+ Hits          14225   14251      +26
- Misses         1660    1661       +1
  Partials        468     468

Impacted Files
Coverage Δ
Complexity Δ

...codegen/viewmodel/testing/ClientTestClassView.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

...nsformer/ruby/RubyGapicSurfaceTestTransformer.java
100% <100%> (ø)
24 <0> (+1)
:arrow_up:

...codegen/viewmodel/testing/ClientInitParamView.java
100% <100%> (ø)
2 <2> (?)

...transformer/ruby/RubyImportSectionTransformer.java
98.55% <100%> (+0.06%)
22 <0> (+2)
:arrow_up:

...api/codegen/transformer/ruby/RubySurfaceNamer.java
96% <100%> (+0.13%)
58 <5> (+2)
:arrow_up:

.../transformer/ruby/RubyGapicSurfaceTransformer.java
100% <100%> (ø)
31 <0> (+2)
:arrow_up:

...ain/java/com/google/api/codegen/ruby/RubyUtil.java
83.33% <83.33%> (ø)
3 <3> (?)

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 9dc8b49...555500f. Read the comment docs.


feat(channel): Add new overwrite helper methods

About
Guys don't code when you're sick
Checklist

[x] The pre-commit code linter has been run over all edited files to ensure the code is linted.
[ ] I've ensured the change(s) work on 3.8.6 and higher.

I've made this pull request: (check all that apply)

[ ] For the documentation
[x] To add a new feature
[ ] As a general enhancement
[ ] As a refactor of the library/the library's code
[ ] To fix an existing bug
[ ] To resolve #ISSUENUMBER

This is:

[ ] A breaking change

Does it work with new changes?

Does it work with new changes?

on my machine yep


Dont trim whitespace from inputs

As of now, this doesn't matter because the agent does this anyways. We're adding an option to not do that from the agent and should not then enforce it here.
Agent PR - https://github.com/microsoft/azure-pipelines-agent/pull/3051
Ticket motivating this - https://developercommunity.visualstudio.com/content/problem/1000335/azure-pipelines-trims-leading-and-trailing-rn-from.html

CI failures look unrelated - they're related to the powershell lib


Suggestion regarding button width

Would be  great if you guys could add something like the following (widthSelector Prop) to allow some control over button width:

https://github.com/dancormier/react-native-swipeout/issues/42


[Question] Why is trace_references pushing objects to grey_stack?

Hi,
Thank you so much for sharing your great code and your experience in the blog.
I'm reading your GC code in unsafe https://github.com/ceronman/loxido/blob/unsafe/src/gc.rs and have a question.
Question
Is it intentional that trace_references is marking objects (= trace from roots) and pushing them to grey_stack?
Why I'm asking
In my understanding the GC works in 3 steps as below.

Mark roots and push them to grey_stack. (= Mark)
Iterate over grey_ stack and recursively mark used objects (= Trace).
Free unused objects (= Sweep).

I see grey_stack as a collection of root objects and it is created in 1 and used in 2.
But I see grey_stack is actually updated in 2 which makes me think why.
Thank you in advance,
Taro

One more note.
The original C implementation is doing similar. So I think this is intentional.
https://github.com/munificent/craftinginterpreters/blob/master/c/memory.c#L265


correctly name github workflow

Signed-off-by: Chuck Ha<EMAIL_ADDRESS>What this PR does / why we need it:
apparently the name of the file must be main.workflow.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Related to #67
NONE

/assign @ashish-amarnath

/lgtm


Powershell examples?

Make examples with Powershell

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

ID: 1ab4c199-c44d-b8c6-9bf5-5500a30a6992
Version Independent ID: ed692025-4ede-d7f1-be2f-6ee1f8ce16eb
Content: Obtaining Build Logs with MSBuild - Visual Studio
Content Source: docs/msbuild/obtaining-build-logs-with-msbuild.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

Example:
MSBuild with PowerShell

Add MSBuild.exe to Environment Variables:

This example consider that you installed Build Tools for Visual Studio 2019

Ctrl+R
type SystemPropertiesAdvanced and press Enter
click Environment Variables...
on System Variables panel select Path item and click Edit...
click New and insert the path to MSBuild.exe. With default install folder it
probably be in C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\MSBuild\Current\Bin.

After this add examples with Powershell in addition with cmd

descriptions
cmd and Powershell
msbuild MyProject.proj -t:go -v:diag

descriptions
Powershell

MSBuild -v:q `
  -fl -flp:LogFile=./artifacts/build.log`;Verbosity=q`;Encoding=UTF-8`; `
  Sandbox.Accounts.sln `
  -p:Configuration=Release `
  -m


Support for BOSH deployment manifests in configgin

[ ] Change configgin so it accepts input from a BOSH deployment manifest
[ ] Change fissile so it generates pods that mount a deployment manifest secret
[ ] Add --deployment-manifest-template to fissile build helm

Full spec for this feature lives here.

I wonder what a thought experiment around "what would happen if instead of configgin taking a BOSH deployment manifest we would try and remove configgin from the equation?" would yield.

@bkrannich we just need "something" to render erb templates.
Right now configgin wraps the bosh-template gem and adds a bit of code to be kube aware and environment aware.
In the end, configgin will be a very thin wrapper for the gem, just so you can call it from the CLI, and it won't exist in each deployment container. It will be used by the operator directly.
Going through the thought experiment:

remove configgin
what do we use for rendering ERB templates?
we need something ruby - no way around that
we should use the bosh-template gem so we get results consistent with the bosh agent
how do we call the bosh-template gem? our process is go
we need to wrap it (with a CLI or some API)
I think CLI is simpler and would work well as a Kube job

@viovanov - that's for cases where there's actually ERB as part of the manifest, I guess. What about cases where there isn't (or are those too rare to consider)?


Flow tab shows up twice after merge

Current Behavior / steps to reproduce

Make sure there is only one flow
Open Node-RED in two different browser tabs
Change the flow and deploy it
Open the second browser tab
Click on Review changes in the update notification

Click on Merge

Now the Flow tab is displayed twice and the tab currently selected is empty:

When clicking on the second tab, the flow is displayed correctly. Now when clicking on the fist tab, the flow is displayed correctly as well.
Reloading the page solves the issue.
When multiple flows exist, similar issues occur on merge.
Expected behaviour
After the click on Merge the Flow tab should only be visible once.

Example flow
The issue occurs with all flows.
Environment

Node-RED v2.2.0-beta1 and commit 211a5eb2bbb64f3c00c30427808ed0f56db9ec97
Node.js v16.13.1
npm v8.1.2
Host operating system: Ubuntu Focal with Kernel 5.4.0
Operating system running the browser: Windows 11 Version 21H2 Build 22000.434
Browsers: Microsoft Edge v97.0.1072.62 and Mozilla Firefox v96.0.1

Hi @hfedcba I can confirm the same issue on ubuntu 20.04 with Node-RED v2.2.0-beta1, nodejs 16

and when running on windows...


IndexOutOfRangeException for Empty Message

Somehow someone sent a message with length 0.
From TwitchLib\Models\Client\ChatMessage.cs (netcore):

To fix this, prepend Message.Length > 0 && to the condition of the if statement on line 178 (netcore).

I have seen these zero length messages too
Will patch tonight

Recommended fix applied, feel free to reopen if the issue isn't solved.


If cloudformation stack IN_REVIEW, then a change request needs to be CREATE

Please check if the PR fulfills these requirements

[x] The commit message describes your change
[ ] Tests for the changes have been added if possible (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)
[x] Changes are mentioned in the changelog (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
bug fix

What is the current behavior? (You can also link to an open issue here)
If a stack is IN_REVIEW, it exists, so the current logic attempts to do an UPDATE, but that fails with a stack does not exist failure from cloudformation

What is the new behavior (if this is a feature change)?
This forces the CreateChangeSet request to use a change set type of CREATE if the stack is in review.

Does this PR introduce a breaking change? (What changes might users need to make in their setup due to this PR?)
no

Other information:

Thanks for the fix. The tests seem to find NullPointers. Can you please check this?


[BUG]: theme toggle button not properly working 🙅🏻❌

[x] Would you like to take up this task?
[x] Are you from GSSoC'23?

🐛
Bug report

Describe the bug
The Bug is about the svg elements present in the theme toggle failing to change its color, So the svg icons not visible after toggling the theme

Location of the bug
Nav bar

Expected behavior
The  svg icons not to be affected during the theme toggle

Screenshots
https://github.com/sourabhsikarwar/Scene-Movie-Platform/assets/92252895/5bef0ee9-ae26-4d9e-9422-cdc45f2b2202

Hey there @sourabhsikarwar 👋🏻
Please make sure to  assign this issue to me to make the work done, Thank you!

Please make the default theme dark @sourabhsikarwar


Creation of a project structure

@GoroBob
I've created a project structure.
It would be nice to reserve 1 to 2 hours of meeting to tell you about this structure and how to use it.


Formatter: breaks copyright formatting

After running "mvn formatter:format" on DataVec, copyrights notices look as follows:

Yes, expected behavior. The issue is in the shape of the copyright comment (see § 5.1.1 here:
http://www.oracle.com/technetwork/java/codeconventions-141999.html )
The fix is here :
https://github.com/deeplearning4j/DataVec/commit/1d5c7db0ce2bbcb917625246e90fca4433e73f7c


Text input field character limit

The input box currently has a char limit of 240. Why? I realize that this might be related to potential difficulties when running the vector search, though. would swapping to ST instead of OpenAI potentially help, if the limit is related to the search?

Oh, lol, just noticed #62 ... nevermind :D


Add VNET rules in table azure_mysql_server

Reference:
<EMAIL_ADDRESS>

@ParthaI When we are creating any replica server list call is not working , it is just getting stuck.
Please find the below logs which may help.
request: StatusCode=500 -- Original Error: context canceled
2023-01-05 12:13:06.536 UTC [ERROR] steampipe-plugin-azure.plugin: [ERROR] 167292: table 'azure_mysql_server' failed to get column data, connectionCallId azure-167292: table 'azure_mysql_server' column 'server_configurations' requires hydrate data from listMySQLServersConfigurations, which failed with error mysql.ConfigurationsClient#ListByServer: Failure sending request: StatusCode=500 -- Original Error: context canceled.
2023-01-05 12:13:06.536 UTC [WARN]  steampipe-plugin-azure.plugin: [WARN]  167292: QueryData StreamError table 'azure_mysql_server' column 'server_configurations' requires hydrate data from listMySQLServersConfigurations, which failed with error mysql.ConfigurationsClient#ListByServer: Failure sending request: StatusCode=500 -- Original Error: context canceled.


please help , CUDA mining not working

I tried to do the following , but it is not working
Building on Windows
download or clone this repository
download and install Visual Studio 12 2013 and CMake
run getstuff.bat in cpp-ethereum/extdep
open a command prompt and navigate to cpp-ethereum directory
mkdir build
cd build
cmake -DBUNDLE=cudaminer -G "Visual Studio 12 2013 Win64" ..
if you don't want/need CUDA support, use "miner" instead of "cudaminer". This will only compile OpenCL support
to speed up compilation a bit, you can add -DCOMPUTE=xx , where x is your CUDA GPU Compute version * 10. i.e -DCOMPUTE=52 for a GTX970.
you may disable stratum support by adding -DETH_STRATUM=0
When CMake completes without errors, opn ethereum.sln created in the build directory in Visual Studio
Set "ethminer" as startup project by right-clicking on it in the project pane
Build. Run
the error I am getting is when I type this
cmake -DBUNDLE=cudaminer -G "Visual Studio 12 2013 Win64" ..
Microsoft Windows [Version 6.1.7601]
Copyright (c) 2009 Microsoft Corporation.  All rights reserved.
C:\Users\Khalid> cd\
C:>cd cpp-ethereum-master
C:\cpp-ethereum-master>cd build
C:\cpp-ethereum-master\build>cmake -DBUNDLE=cudaminer -G "Visual Studio 12 2013
Win64"
'cmake' is not recognized as an internal or external command,
operable program or batch file.
C:\cpp-ethereum-master\build>cmake -DBUNDLE=cudaminer -G "Visual Studio 12 2013
Win64"
'cmake' is not recognized as an internal or external command,
operable program or batch file.
C:\cpp-ethereum-master\build>

Need to install cmake first,
https://cmake.org/
This is not a bug.

After that how do i start mining ?
Regards,
Khalid Al Ghamdi
0097333454988
0097332100046

On Aug 8, 2016, at 10:16 PM, mailming<EMAIL_ADDRESS>wrote:
Need to install cmake first,
https://cmake.org/
This is not a bug.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

You only need to build if you're experimenting with the code.  If you just want to mine, pull a pre-build image from:
https://github.com/Genoil/cpp-ethereum/tree/110/releases


'Find Unused Step Definitions' command doesn't handle 'StepDefinition' attributes well

Used Visual Studio
Visual Studio 2022
Are the latest Visual Studio updates installed?
Yes
Content of reqnroll.json (if present)
{
"bindingCulture": {
"name": "en-GB"
},
"language": {
"feature": "en-GB"
}
}
Issue Description
Step definitions declared with the 'StepDefinition' attribute are showing multiple times inside the new 'Find Unused Step Definitions' menu option. These appear to show Given/Then/When usages, and implies that the Step Definition is unused - which is not always the case.
Is it possible to change this behavior to show which methods decorated with 'StepDefinition' are unused?
Steps to Reproduce

Have a Reqnroll project
Create a new 'Step Definition' file, adding the [Binding] attribute
Create a new step definition, using the 'StepDefinition' attribute. For example: [StepDefinition("I can do the thing")]

Use this StepDefinition in a feature file. For example: Given I can do the thing

View the 'Find Unused Step Definitions' context on the Step Definition file.
You will see:

[Then("I can do the thing")]
[When("I can do the thing")]

Link to a project repository that reproduces the issue
No response

That makes sense.
Reqnroll internally generates three step definition binding from the [StepDefinition] attribute and this is what the VS extension receives. At that point it is not possible to tell if they were coming from the same attribute or separate ones, but if they belong to the exact same method and have the exact same expression we could assume that these were created using [StepDefinition].
The improvement could be done by modifying the UnusedStepDefinitionsFinder.FindUnused method (plus add tests for it to FindUnusedStepDefinitionsCommandTests).
I think the implementation should:

add an additional loop before the return (here)
go through the ones that are considered unused (stepDefUsageCounts.Where(x => x.Value == 0))
for each of these, check if there is a used one (stepDefUsageCounts.Where(x => x.Value > 0)) that has the same Expression and the same Implementation (ref equals).
If there is one, it should not be counted as unused

Do you want to try making a PR for this?

@gasparnagy Sure I'm happy to contribute, just a couple of questions:

Is it possible to debug the Visual Studio extension? I can't seem to debug/run from the Reqnroll.VisualStudio.Package project without installing a 'preview' package into Visual Studio?
I don't seem to have rights to create a new branch in this repository, is that something that could be sorted?

Thanks!

@gasparnagy Thanks for sorting that, I can push new branches/etc to the repository now.
From the guide it says:

You can also set the Reqnroll.VisualStudio.Package project as startup project. It is configured in a way that if you use the Start Debugging command (F5) it will automatically start the Visual Studio Experimental Instance with the debugger attached. So you can debug the full life-cycle of the extension, right from the loading. (Similarly, the "Start Without Debugging" (Ctrl+F5) is also working that simply starts Visual Studio Experimental Instance and you don't have to find it in the Start menu.)

For me, when loading the extension project, I don't see the option to debug, and get the following popup:

Normally I'd think it was something with my Visual Studio install, but I'm able to debug/launch other extensions fine.

That is strange. But I think you can just set the debug config to debug the devenv.exe process (from the vs install location) with the params /rootSuffix Exp. But I will check it once I am back at my desk.

I have checked and indeed the package project cannot be started by default. I might have created a launch profile earlier. Anyway: I have documented the steps to configure the launch profile: https://github.com/reqnroll/Reqnroll.VisualStudio/blob/main/CONTRIBUTION.md#debug-the-visual-studio-package
I hope it will work for you as well.

@gasparnagy That worked great, thanks for writing that - hopefully it helps other contributors in the future as well 😃

@gasparnagy unrelated to this specific issue, I've noticed that the 'Find Unused Step Definitions' doesn't handle it very well if step definitions are included in a separate project.
I've tried to access the context menu option from this separate project and it throws the error below. Is this intended behavior or something that also needs to be fixed? I'm happy to look into this while I'm in this area.

@UL-ChrisGlew Thx. We were going back and forth with @clrudolphi about how the projects should be handled. So you should definitely check the discussion we had from here down. We also created a scenario to cover the cases, but it can happen that we still miss a combination.

IIRC we decided to not look at external assemblies because of the risks of false positives.
An external step definition assembly may be used by many different projects/solutions. A step that is not used by one project may be used by another. To mark it as 'unused' would be misleading.
We could perhaps fine-tune that with some heuristics. If the external step definition project is incorporated via Nuget for example it would be excluded but if the project containing the steps is a project within the current solution then perhaps we include it.

Ah okay, it's not as simple as a problem to solve as I first thought - it's not a dealbreaker at the moment for us but would be useful in the future.

I close this issue now as we will probably not be able to improve this in the close future. If anything comes up, feel free to reopen.


yahtzee_sol: Test against a distribution

Make Python match the other examples and instructions
Closes: #158

There is a copy-paste problem in the Python code. Two lines repeated.


MNT: Re-rendered with conda-build 3.18.7, conda-smithy 3.4.1, and con…

…da-forge-pinning 2019.08.12

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

We can use this to perform the R 3.5.1 version of #105

For whatever reason this didn't remove the CONDA_BUILD_SYSROOT stuff like #105 did.

@conda-forge-admin please rerender

Ah, my conda-build is old it seems.

@conda-forge/r-base Anyone else want to have a look at this? After merging this #105 can then be merged.


Extend the "automatically apply traffic lights when applicable" policy to allow manual configuration of signaling on each junction type.

Describe your idea
In my mind the policy would change to "automatically configure signalling based on junction type" and for each of the following include a drop down menu to choose the default type of signalling (Options I've thought of also listed below.)

Local to Local

Local to Collector

Collector to Collector

Collector to Arterial

Arterial to Arterial

The (sane) options I can think of:
I Have no idea what half of these signal layouts should actually be named, so names will probably have to change.
Available for all Junction Types

None

Stop Lights

All Way Stop

All Way Yield

Only Available for Local/Collector & Collector/Arterial Junctions

Minor Yield to Major

Minor Stop For Major

We've pondered this sort of thing in the past and ran in to some difficulties becuse the segments attached to a junction are not always representative of the roads further back from the junction.
For example, if there's a 2+2 road at a junction, is that a collector, or did the user just replace some segments of a 1+1 road with 2+2 road so they could add dedicated turning lanes?

segments attached to a junction are not always representative of the roads further back from the junction.
Hmm. Fair. If someone needed to upgrade that section to add turning lanes, one could argue that they probably are having traffic issues there, and using signalling for that larger segment may not be a bad idea? That or they are probably already adjusting signalling manually and would see the change.

Past that though, I would say to an extent, that is the exception rather than the rule. Its also far less work to configure signalling when you add a dedicated turn lane than having to configure every new local to collector connection by hand.

One of the workarounds we've used for some of the tools (like lane connectors and lane arrows) is to have multiple configurations available - each use of the shortcut cycles through whatever different options there are for the place where it's being applied.  However, that still needs user to decide which set of customisations is best suited so it's not ideal for automatic application.

I'd say its kind of the intended effect; Its just something that happens. It would be based on the junction type; the road that connects to the other roads to form the junction, NOT the roads leading up to the junction. If its made clear that that's how its mean to function, and that if someone wants to do something other than that, they have to do it manually, i don't see an issue.

(continued explanation that might make more sense.)
If you have a junction like this as a 4 way stop,

you aren't going to leave it as a 4 way stop if you add dedicated turn lanes.

If i'm understanding road hierarchy correctly, by adding those turn lanes, (even if its still only a 2 lane road before the junction) you have still made that the priority movement, and would assign it as such, by either removing the stop sign from that road and leaving the others, or adding an stop light to the junction.

What happens if the road has bus lanes, or asymmetric lanes, etc?

Good question, I don't have an answer off the top of my head for that one.

4-way yield and stop junctions in TM:PE don't work the way they are supposed to, so I don't recommend using them. Even 3 cars is enough for them to deadlock, and the deadlock won't resolve until you give one of the cars priority.

4-way yield and stop junctions in TM:PE don't work the way they are supposed to, so I don't recommend using them. Even 3 cars is enough for them to deadlock, and the deadlock won't resolve until you give one of the cars priority.

hmmm... I guess it would have to give one road priority then. Issue is what factors should be considered when selecting the priority road?

I guess to explain my more specific original reason that made me think of this to begin with is that with the policy as it is currently, any time a junction is added to a collector or arterial, it adds stop lights. Problem is that that means any time a local road connects to a collector, it adds stoplights. For me this ends up causing traffic issues, because every time someone wants to turn onto the collector from any local roads, it stops traffic on the priority movement; the collector. Basically all i really want is the option to have yield/stop signs placed on local -> collector connections to prioritized that collector movement, instead of traffic lights. the two example photos being  and  (Of course, this would also apply to any other local connections within the junction, so if the local road were to continue 'north' that side would also have the yield / stop sign. I can't just disable the policy, because then arterial -> collector and collector --> collector connections then don't get lights and then they end up with traffic issues.

Regarding deadlock at junctions where all roads have same priority, that's something @krzychu124 is investigating as part of major improvements to traffic handling at junctions - see vids in this long thread (deadlock issue is shown in one of them): https://github.com/CitiesSkylinesMods/TMPE/issues/1229#issuecomment-995324724


fix: go sum conflicts due to dependabot

Signed-off-by: Kush Trivedi<EMAIL_ADDRESS>Signed commits

[x] Yes, I signed my commits.

/cc @leecalcote

Is this needed? I’m confused as to why this needs fixed all the time. Should this just wait until the next feature/bugfix PR?

Is this needed? I’m confused as to why this needs fixed all the time. Should this just wait until the next feature/bugfix PR?

Dependabot PR already has an issue opened for this, and this is why I made the frequency of updates from one week to one month so that we can merge all PRs and then make a successive PR to fix all conflicts.

Is this needed? I’m confused as to why this needs to be fixed all the time. Should this just wait until the next feature/bugfix PR?

Got it let's handle this in another bugfix PR!


Add alternate use keybinding

This would allow for more complicated interactions like choosing the direction gearboxes face when placed, and allowing both placing and using buckets.
https://github.com/BTW-Community/BTW-source/issues/46

Moving the next version because of comprehensive controls gui changes necessary. All current space is occupied so more control options added would require a rewrite of the gui organization.

Completed by 1.3.0


Fix READMEs and bug in example startup script

The latest version of Consul uses -http-addr and not rpc-addr. Update the example script to use it.
Fix some minor issues in the READMEs.

Simple fix. Merging now. Feedback welcome!


[Azure iot SDK node js]event hub receive data error "message container is being closed"

[Issue describe]
Customer send data to iot hub.
And we have event hub sample, used the iot hub edndpoint to monitor the d2c data, but during about two hour, we will get exception.
[Information]
In device:
Debugger listening on [::]:5858
Connect IoT Hub
[IoT Hub Device] Client (45-sas) connected

{"type":"Alert","message":"IoT Hub device connected","device_id":"45-sas"}
{"sensor_id":234,"time":"2017-07-17T06:10:20.638Z"}
{"sensor_id":234,"time":"2017-07-17T06:11:20.663Z"}
…
<- issue occur in event hub node js command prompt , but client still send the message ->
…
{"sensor_id":234,"time":"2017-07-17T09:13:21.415Z"}
{"sensor_id":234,"time":"2017-07-17T09:14:21.419Z"}

In event hub monitor node js:
Debugger listening on [::]:5894
Created partition receiver: 1
Created partition receiver: 0
{"sensor_id":234,"time":"2017-07-17T06:18:20.725Z"}
{"sensor_id":234,"time":"2017-07-17T06:19:20.728Z"}
…
{"sensor_id":234,"time":"2017-07-17T07:57:21.100Z"}
{"sensor_id":234,"time":"2017-07-17T07:58:21.106Z"}

About 3 hours,
Error from event hub: The message container is being closed (42864). TrackingId:8ed64a36-f063-43c8-97a5-382fe36360cf_B2, SystemTracker:NoSystemTracker, Timestamp:7/17/2017 7:58:23 AM
The code is below:
'use strict';

var EventHubClient = require('azure-event-hubs').Client;

//var connectionString = '[connectionString]';
var connectionString = '[connecting string]';
var eventHubPath = '[path]';

var printError = function (err) {
  console.log('Error from event hub:',err.message);
};

var printMessage = function (message) {
  //console.log('Message received: ');
  console.log(JSON.stringify(message.body));
  //console.log('');
};

var client = EventHubClient.fromConnectionString(connectionString, eventHubPath);
client.open()
    .then(client.getPartitionIds.bind(client))
    .then(function (partitionIds) {
        return partitionIds.map(function (partitionId) {
            return client.createReceiver('$Default', partitionId, { 'startAfterTime' : Date.now()}).then(function(receiver) {
                console.log('Created partition receiver: ' + partitionId)
                receiver.on('errorReceived', printError);
                receiver.on('message', printMessage);
            });
        });
    })
    .catch(printError);

You are simply using the simple_eventhubs_client.js?
The connection string you are using.  You got it from where?  If you could supply the connection string in reply WITHOUT POSTING THE KEY, I repeat, WITHOUT POSTING THE KEY, that could be helpful.

Hi Anthonyvercolano,
Thanks for your help, may I know what is  "WITHOUT POSTING THE KEY", it's mean don't paste the connection string in the article ? so you had help us to replace the information :
var connectionString = '[connecting string]';
var eventHubPath = '[path]';
or you have other meaning, thanks.

Typically when you create a device you have (eg for an IoT Hub) HostName=test.private.azure-devices-int.net;DeviceId=mycooldevice;SharedAccessKey=
Actually all I really need is the HostName portion of the connection string.  From that I can look up logs and get a better Idea of what happened.  So feel comfortable just positing something like: test.private.azure-devices-int.net

It appears that the client needs to handle this and retry the connection. See this: https://stackoverflow.com/questions/42563043/azure-event-hub-servicebusexception-causing-skipped-messages.

@kuomingwu - We have a new preview version "0.1.0" of the event hubs node.js client. This client was re-written from scratch. Please try the new version and let us know if you still see issues. You can find examples over here.

Thanks Amar, I will try it.
Regards,
-Kuomingwu
Cheers,
Kuoming Wu
2018-04-25 2:46 GMT+08:00 Amar Zavery<EMAIL_ADDRESS>
Closed #29 https://github.com/Azure/azure-event-hubs-node/issues/29.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-event-hubs-node/issues/29#event-1592363485,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFyiqFT3v4fc3F1sYzZmn-Tu0IGrz5fRks5tr3MNgaJpZM4OZ-Vz
.


Update wallet dependencies

It has been a while since all of the SDK and hardware wallet dependencies have been updated. We need to go through and check for updates and update to the latest versions for the following dependencies:
@ledgerhq/hw-app-eth
@ledgerhq/hw-transport-u2f
@portis/web3
@unilogin/provider
@walletconnect/web3-provider
authereum
fortmatic
squarelink
trezor-connect
walletlink

Closed via #407


Edit EFO traits for 'age of onset' studies in bulk

A number of new EFO terms have been created to better represent studies on the age of onset of a disease. These terms take the form:

age of onset of [disease X]

They will replace previous annotations where two separate terms were used:

age at onset | [disease X]

As multiple studies need to be reannotated, and many were created before the new deposition software, these edits will need to be performed manually, in bulk. The attached spreadsheet contains the PMID and STUDY ACCESSION for each affected study, and the current MAPPED_TRAIT and MAPPED_TRAIT_URI. These should be replaced with the new terms under NEW_MAPPED_TRAIT and NEW_MAPPED_TRAIT_URI.
Note that a handful of studies also contain a third EFO term that is not affected by these merges, and therefore should not be removed or changed. For those examples, the additional term is included in both the MAPPED_TRAIT and NEW_MAPPED_TRAIT columns.

https://app.zenhub.com/files/2995118/e1dbe68b-05c7-4bfe-855e-7b3f35cd85f1/download

@Elliot the file is in funny format unable to format in excel with delimiters , can you share over email or slack the file

@Elliot the file is in funny format unable to format in excel with delimiters , can you share over email or slack the file

@sajo-ebi just sent an updated copy of the file to your email

Hi @eks-ebi , for GCST90027235 in the Mapped Trait is 'migraine without aura, susceptibility to, 4' , there is no EFO Trait  like this in DB , but I can find two entries for 'migraine without aura' in DB , can you confirm what is the exact EFO, also we might have to delete the duplicate EFO from DB for 'migraine without aura'

ah ok, I see - the two entries for 'migraine without aura' in the database are http://purl.obolibrary.org/obo/MONDO_0100431 (which has the correct name) and http://purl.obolibrary.org/obo/MONDO_0011847 (which is officially called 'migraine without aura, susceptibility to, 4' in EFO). It might be that the commas caused some problems when this was added to the database.
For the study in my spreadsheet (GCST90027235), I believe it was originally annotated with http://purl.obolibrary.org/obo/MONDO_0011847, which is why I used that term. However, I actually made a mistake with the name of the "age of onset" term that should replace it. The replacement term name should be just "age of onset of migraine without aura" - the EFO URI is correct though (http://purl.obolibrary.org/obo/OBA_2001028).

@eks-ebi the studies have been updated in Curation DB kindly validate

Confirmed! I checked about half of the studies and those all looked good

@eks-ebi to confirm after DR

@eks-ebi please close the ticket if you are happy

all look good in production, can close ticket


Improve ECS and Fargate tagging consistency

ECS classic tasks now have a cluster_name
ECS classic tasks' task_name tag is copied to task_family
for consistency
Add task_arn tag to both collectors

Sister PR in integrations-core to come.

Codecov Report

Merging #2675 into master will decrease coverage by 0.01%.
The diff coverage is 29.41%.

@@            Coverage Diff            @@
##           master   #2675      +/-   ##
=========================================
- Coverage   56.62%   56.6%   -0.02%
=========================================
  Files         379     379
  Lines       24052   24069      +17
=========================================
+ Hits        13619   13625       +6
- Misses       9514    9525      +11
  Partials      919     919

Impacted Files
Coverage Δ

pkg/tagger/collectors/ecs_main.go
5.4% <0%> (-0.48%)
:arrow_down:

pkg/util/ecs/ecs.go
40.17% <0%> (-3.35%)
:arrow_down:

pkg/tagger/collectors/ecs_extract.go
91.3% <100%> (+1.83%)
:arrow_up:

pkg/tagger/collectors/ecs_fargate_extract.go
83.33% <100%> (+0.4%)
:arrow_up:

pkg/forwarder/worker.go
93.75% <0%> (+1.25%)
:arrow_up:


feat: Add datatype enum.

Inspired by upstream changes: https://github.com/openapi-generators/openapi-python-client/pull/478
Adds a DataType enum. This:
Improves validation by rejecting invalid keywords which don't match the spec at parse-time.
Improves developer experience, allows IDE autocompletion to show allowed values.

Hey @kuimono any comments on this and #13?

Adding the DataType enum could be a breaking change for users who are only using it as string.
Therefore this PR maybe pending for next major release.

I would argue that it will only be a breaking change if using it to parse schemas that are invalid from the OpenAPI spec 😉, but I guess you are technically right. Any idea when you'll release the next major? I was looking to get these changes in to allow this project to be a non-vendored dependency in the aforementioned downstream project 🤔.

@mike-oakley -> youre welcome to submit any and all improvements to https://github.com/starlite-api/pydantic-openapi-schema, we'll gladly release this PR (except for code duplication, which we should strive to reduce when possible).


Add Parallel GZIP Optimization.

Will likely change this to use https://engineering.linkedin.com/blog/2019/02/migz-for-compression-and-decompression

is this an actual issue? even the biggest data the game has is not above 8mb files usually, looking at those benchmarks from linkedin and the perf improvements on GB large files, this would give micro seconds improvements at best. depending on the initial overhead it may not be an improvement at all, do you have any data to support that this helps?

@phit none currently, it seems it could help during large pregens and retrogen operations but I need to test it properly

Closing this until i can do tests


Docs.MS Release Updates for azure-communication-phonenumbers

Update docs metadata and targeting for release of azure-communication-phonenumbers

Docs Build status updates of commit 96fe4a2:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


WebText Dataset format

Which is the meaning of length, ended in the dataset lines:
{"id": 1, "ended": true, "length": 66, "text": "LeSean McCoy going through warmups with first team offense. To my eye, does not look close to 100 percent when cutting and exploding.\n\nABOUT COOKIES\n\nTo help make this website better, to improve and personalize your experience and for advertising purposes, are you happy to accept cookies and other technologies?"}

also I can see that there are newlines followed by indexes like in
{"id": 0, "ended": true, "length": 138, "text": "These girlfriends deserves a special mention for going that extra mile, hopefully doesn't set too many guys off on the path towards outrageous demands.\n\n1. She knows the severity of man-flu\n\n2. All fun and games is all good\n\n3. A voucher that says 'I love you'\n\n4. When arguments don't drag on forever.\n\n5. Providing everything he needs.\n\n6. Very understanding\n\n7. As awesome a gesture as this is, we are worried about this man's cooking skills.\n\n8. Nice cake\n\n8. Fair bargaining\n\n9. Excellent gift choice\n\n10. Very thoughtful"}

so \n\n3...\n\n8. What does this mean? Is it just a questionnaire style scraped document?
I can see that the detector does not use those info anyways: https://github.com/openai/gpt-2-output-dataset/blob/master/detector/dataset.py#L17
Thank you.

length is the length in BPE tokens (see gpt-2 paper for information on tokenization scheme).  ended is whether the sample contained (and is truncated at) an endoftext token


Import Civic Authentation to Passport.js?

when will you import Import Civic Authentation to Passport.js?
https://www.civic.com/

Here you go :) https://www.npmjs.com/package/passport-civic

passport-civic is now indexed and searchable on passportjs.org.


Update the icon color in onMounted

Fixes

Fixes #5227 by @obulat
Description

This PR replaces the computed values for icon names with ref that are updated in onMounted (which runs after the hydration is done). The license icons and the theme icons are affected, but I can revert the change for the theme switcher if #5236 is the preferred approach, @dhruvkb .
Testing Instructions

Select the "system" mode and refresh a single result page. Make sure your system is set to dark mode by default (or use the browser devtools). Refresh the page and see that there are no client-server mismatch warnings: neither for the theme switcher (sun/moon icons) nor for the license icons on every image result:

Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (ov just catalog/generate-docs for catalog
PRs) or the media properties generator (ov just catalog/generate-docs media-props
for the catalog or ov just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I'll let you both coordinate on these PRs.


Think about removinig the template code out of this module into  its own "mett - more/moerills enhanced textured templates"

This module has outgrown its original idea and the whole template stuff is propably worth its own module now.. So I consider moving that out of mess

Can I suggest "MATTE - Moerill's Added Template Texture Enhancements"?  ;)

This would be awesome, please.  At the moment, it's the only thing I use MESS for, and would love to have it by itself.


Enable APIScan for VS-MEF

Enable APIScan for VS-MEF as required for compliance

To change one file, it appears you're proposing to merge 6 commits. Is this something you plan to squash?
Have you validated that this actually works with our devdiv CI pipeline (by pushing to this repo via a topic branch and manually queuing a build)?

Here's the validation build of these changes:
https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=4698147&view=logs&j=2d2b3007-3c5c-5840-9bb0-2b1ea49925f3&t=15480a3f-208f-51cb-1c92-300604e6d0ba


WIP migrating to Azure Storage SDK 10.0

@xperiandri firstly thank you for creating this PR and apologies (I know we've spoken already about this) for the delay in getting to it.
At first glance, is there a reason why we need to move to three separate type providers? What are the benefits from both the user's point of view as well as development experience?

@xperiandri I've picked up where I left off with the TP - basically the latest versions of F# have build issues relating to the use of the FSharp Compiler Tools dependency (which is coupled to F#4.1) so I'm removing that dependency, whilst also upgrading to FAKE 5. Once they're both out of the way (hopefully today) let's work together to get this ready to go in - I'm willing to split out into three assemblies as long as it's clear what the benefits are.

Sounds good
Was I wrong changing getting current executing assembly to getting current assembly?

@xperiandri where? I'm just getting the current couple of branches merged into master and then will start looking at this PR in more detial.

OK
In TP initialization

By the way, I would prefer to move to a model where you can create a provider with a connection string.
Like ExcelProvider does.

@xperiandri you can already do that?

I mean dynamically. Now you either pass it into TP in code, or when calling a method

So move away from method-based overrides to a single override at the "top" level? That's something I'd love to do.

Yes. Cool!

Regarding get executing assembly vs current assembly using a type - both should work; is there any difference?

I guessed that it will fix an issue with that you need to have Tables.json in all project referencing project with type provider

I've rebased my changed on top of yours


Bump ggez to version 0.5.0-rc.2

Issue #408 and #484 are both caused by the bug in ggez's screen resizing on mac os. Version 0.5.0-rc.2 of ggez disabled support for hdpi screens, so there is no automatic resize at app launch. Thanks to that mouse click offset issue reported in #408 no longer exists.
closes #408
other changes in this pr:

added .editorconfig
added .DS_Store entry to .gitignore. .DS_Store is a system file created automatically on macos

@ozkriff thanks for the review. I understand your concerns. I remote unrelated changes from this pr and open a new pr only with .gitignore amendment

Thanks, merged. :)
I'll think over your DS_Store and editorconfig proposals.


Implement Binary Ninja-based get_cfg

This should be pretty straightforward since all it needs are basic blocks, a list of known code symbol names, and whether or not those symbols are exported.

https://github.com/trailofbits/remill/pull/36


[WIP] Simplify devfile adapter [step 2]

What type of PR is this:
/kind code-refactoring
What does this PR do / why we need it:
Which issue(s) this PR fixes:
Fixes #
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

Replaced by #6771


Host and deploy Blazor: Index topic

Title: Host and deploy Blazor
UID: client-side/blazor/host-and-deploy/index
URL: https://docs.microsoft.com/aspnet/core/client-side/blazor/host-and-deploy

Standalone deployments (notes on Azure CDN, file server, GitHub Pages)
Redirects for correct routing
Hosting in ASP.NET Core
Publishing
IL trimming

Fixed by https://github.com/aspnet/Blazor.Docs/pull/15


Include linkify MyST extension

To convert URLs to links automatically, add the 'linkify' option.

in the docs, etc. tasks in noxfile.py, replace myst-parser with myst-parser[linkify].
add linkify to netlify dependencies.

See comment in #506. This isn't worth it, especially, because I only have one raw URL.


Contact form property

We need to notify the users somehow. Possible contact forms would be:

Email
Text message
Facebook (?)

We opened #6 for this


Facebook conversions don't pass correct currency

I have been testing using analytics.js and have noticed that conversions with currency are not being tracked properly.
I know what the fix is but can't get npm test to work? do i need a particular version of node or npm?

@albybarber thanks for flagging this! the package.json should specify the required toolchain in devDeps. I know this is tangential from addressing your specific issue, but we want to make sure these repos are self-contained and testable without any obstacles, so would you mind sharing the error you're seeing and the version of node you're running?
If you can explain the specific issue you're seeing wrt currency we're happy to help with the fix, as well! Thanks!

Thanks for the rapid response @sperand-io
I have created a pull request for the fix (ignore the first one something happen to the tabbing)
I am happy to switch versions of node or npm.
I am currently on node - v4.2.2 and npm - 3.3.12
I get this error after an npm install
npm WARN ENOENT ENOENT: no such file or directory, open '/Users/alby/gitCode/analytics.js-integration-facebook-conversion-tracking/node_modules/combine-source-map/package.json'
npm WARN EPEERINVALID<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

When I run npm test

node: bad option: --harmony-generators
make: *** [test-phantomjs] Error 9
npm ERR! Test failed.  See above for more details.

@albybarber thanks! damn yea, looks like something funky going on with duo-test for node >4. I think the CLI tool is passing that flag automatically — perhaps @yields @stevenmatheison @dominicbarnes or @ndhoule could weigh in on that :grin:
Could you try running on 0.12 :cry:
We'll get this addressed ASAP!

@sperand-io That seems to get it working!! Thanks.
I still get sauce lab errors but i think they are less important, see

This integration has been deprecated in favor of Facebook Pixel

@hankim813 if that's the case, let's boneyard this repo?

@f2prateek yeah not sure what the state of deprecation regarding this repo is. @sperand-io did all the migration so I'm sure he can confirm


[PackageModel] Add sdk options to platform description

This adds a way to express generic "sdk options" in string version of
the platforms API. The idea is that a package might be able to express
some platform-specific options for the underlying build system in
a generic way. Currently, this is just for experimentation purposes and
we might want some other (better, more typesafe) way of allowing
extending the package description language.
rdar://problem/59788403

@swift-ci smoke test

@swift-ci smoke test


Auto-bell

If someone uses the !bell command, then perform !does bell to show who's ringing the bell.

Implemented in v1.4.2


Bias distortion per-block rather than with mean

AWCY
Previously the distortion was computed for a given block, then biased with the mean importance of this block.

Coverage increased (+0.04%) to 71.408% when pulling 9c0e06e561ad82854cb6dd527cda9c449bd9a781 on YaLTeR:per-block-distortion into 61819f77083892e565e0d20cf9bd23fb26245e9e on xiph:master.


Suppoort vector drawable

Changed some png icons to vector icons.

I'm going to send small pull request to this branch. Because diff becomes bigger and it is hard to check.


Mistakenly complaining about fps info not found

Line 43 of offset.py makes the software complain and print the ffmpeg output if the fps is given in a format different than 00.00. For example, my videos' fps are given as 12 and 24 fps (without decimals) and thus the program complains and the output is unecessarily large.
A solution would be to change the regular expression to '(\d+(.\d+)?)\sfps' instead.
However, if I do so, another problem arises. It happens that my videos have different fps. I don't mind about that because I'm only using the program to calculate the offset (option -coo). However, as verification of whether videos have the same fps is done in offset.py (line 216) which is run when option -coo is enabled, the program fails.
I imagine this should be verified within the run_sync function (line 57, app.py) instead, but I didn't want to try anything because I'm not familiar with this. Anyway, I couldn't find anywhere in the code where the fps are actually used.

@diegodlh,  If you'd like to add the capability to check for the different fps format (I'm not sure what's the standard there), please fork and make the corresponding PR. However, it should work for the old format as well, so the fps token should be optional, i.e. (:?fps)?.
The program checks for the fps being the same since the algorithm actually assumes that they are. I'm not sure the output would look at all correct if the fps were different. It might be an easy fix, though. Again, fork & PR welcome, as long as it doesn't break anything.

I'm going to close for now, please reopen if this is still needed.


One granularity

This PR builds on the work of #3766.
Druid currently has the following classes to manage granularity

Granularity - enum. Used for segments and general granularity
QueryGranularity
BaseQueryGranularity
PeriodGranularity
DurationGranularity
AllGranularity
NoneGranularity

This PR aims to unify the concept and share the period based Query Granularity with Segment Granularity.

Granularity - Base class. Not an enum
QueryGranularity
BaseQueryGranularity
PeriodGranularity - Used for segments and querying
DurationGranularity - Only for querying.
AllGranularity - Only for querying
NoneGranularity - Only for querying

This unifications implies that

segmentGranularity field used for ingestion can also support THIRTY_MINUTE and QUARTER.
queryGranularity can be a String or a Period or a Duration. Whereas segmentGranularity can only be a String or a Period. We can expand segmentGranularity to also be represented with duration if we need to.
queryGranularity when represented as string was defaulting to use DurationGranularity. But that has now been defaulted to use PeriodGranularity instead. No impact with this change.

These changes are in production running on 2bfcc8a.
cc: @fjy @leventov @himanshug

@leventov i see that you've self-assigned this. Can you pls review? I can make the changes together all at once then

generally LGTM.

Thanks @nishantmonu51

I want to review this PR before the end of this week. Please don't merge before.

Note, we are running this in production.

@leventov mostly addressed the stuff you wanted. let me know

@praveev there's a few merge conflicts
@leventov can we please finish this off?

@leventov can you please take a look again? otherwise I'd have to keep resolving conflicts.

@praveev I will look on Monday

@praveev @leventov can we please finish this?

@leventov can you do another pass? this is marked for 0.10.0. can we please try to get this merged by then

@leventov anything else? if not, can you pls merge.

thanks everyone

io.druid.granularity.QueryGranularityTest is very easy to fail after this PR, could you fix it? @praveev
it seems 60 seconds are not enough to forName 1000 times because Granularity is more complex than QueryGranularity.

@kaijianding double checking - is this an intermittent failure? Does it pass locally for you?
i'm going to open a separate issue for this.


Cool 2646

It's the consumer that needs a max priority.. apparently if you leave this out it will not look at queue priorities at all.
So I'm telling the consumer to pick up all messages with Prios up to 5. In the future, we could make a new cloud run worker with min prio set to 6 and max to 10 (or something) and then we would know that certain instances cannot pick up certain messages.
Does that make sense?


[dv] More flexible riscv_core_setting.sv generation

Currently the DV flow patches the riscv_core_setting.sv file if certain configs are used to remove CSRs those configs do not have and adjust PMP settings.
We need something more flexible that can look directly at the Ibex config and generate the file based upon settings in there.
In particular we need this to enable https://github.com/lowRISC/ibex/pull/1776. With better CSR coverage we're seeing failures in different configs due to different CSR setups.
I would suggest we an existing python templating library (e.g. Jinja2). We'll need a small python script that can read in the ibex config and pass it through to jinja2 to produce the final riscv_core_setting.sv.
@marnovandermaas @ctopal @hcallahan-lowrisc any thoughts?
(Estimate based on us following my idea above)

estimate 4

We could maybe extend the current patch setup to do more things using more patch files but it quickly becomes brittle. So I think this is something that's worth doing for V2 (may end up more time juggling patch files later on to hit V2 as we discover other things the templating setup becomes useful for).
Another option would just be to cease running CI against anything but the opentitan config, but I do not like that idea.

Yes, I added the patch mechanism when I enabled the PMP tests, but it was definitely meant as a short-term solution as long as that was the only check that needed to happen. I've never used this particular templating library, link for reference:
https://jinja.palletsprojects.com/en/3.1.x/
But it looks like a good option so that we only have to specify the differences between the different configurations (which was the original goal of the patch system as well).

@marnovandermaas and I talked about templating as an option when discussing PMP stuff a few weeks back. I think it makes the most sense here. Maybe we should use Mako as that is already used within the Ibex project (it's already in the python_requirements.txt), though I wouldn't be against using Jinja2 either.
A related comment I have is that it does feel a bit confusing with options to riscv-dv being passed through both the core_settings file, and on the cmd line. Plus the other yaml files. Eg. you set supported_isa[$] in core_setting.sv and also pass --isa  to riscvdv/run.py. Though that's a discussion for another time I think.

@hcallahan-lowrisc good idea about Mako, I'll paste the reference here for convenience: https://www.makotemplates.org/

Maybe we should use Mako as that is already used within the Ibex project (it's already in the python_requirements.txt)

Ah yes we should. For some reason I thought we were using jinja2 within OpenTitan but we're not it's Mako everywhere.

A related comment I have is that it does feel a bit confusing with options to riscv-dv being passed through both the core_settings file, and on the cmd line. Plus the other yaml files. Eg. you set supported_isa[$] in core_setting.sv and also pass --isa to riscvdv/run.py. Though that's a discussion for another time I think.

Definitely, on both points. A complete refactor to make configuration more sane is needed but not something we want to look at now.


Overriding single-product-reviews.php

Hi,
I'm trying to rearrange the way WC reviews are being displayed, but it seams that not all WC templates files can be overriden when i use your library.
If i add a single-product-reviews.twig file in my views/woocommerce/ directory, it's not taken into account. It's working fine for single-product.twig and friends.
Is it normal ?

Comments templates seams to be a specific thing in WC. As stated in class-wc-template-loader.php, comments template is checked to be in one of those dirs :
public static function comments_template_loader( $template ) {
	[...]
	$check_dirs = array(
		trailingslashit( get_stylesheet_directory() ) . WC()->template_path(),
		trailingslashit( get_template_directory() ) . WC()->template_path(),
		trailingslashit( get_stylesheet_directory() ),
		trailingslashit( get_template_directory() ),
		trailingslashit( WC()->plugin_path() ) . 'templates/',
	);

So i ended with overriding single-product-reviews.php at the root of my theme.


Revisit the code that populates the search results table

It works well sometimes, but other times it is comically slow, and it's hard to detect a pattern. I suggest we start by refactoring the worker to make it update a row at a time rather than a cell at a time and see how far that gets us.
https://github.com/bluesky/bluesky-widgets/blob/c2edf3cb3d5d5c66545e2130712390edc70485c7/bluesky_widgets/qt/_search_results.py#L23-L32

Part of the explanation for the slowness of cell-based updates is probably https://github.com/bluesky/tiled/issues/645. Row-based updates will be faster, but still subject to really inefficient paging until that is fixed.
Once the issue is fixed, row-based updates will be ok, but still subject to latency on a per-row basis. I have code that does efficient chunk-based updates, which work well, though they will also get faster once https://github.com/bluesky/tiled/issues/645 is fixed.


feat: verkle integration with disk db

This is the first pr for the integration of verkle tries in reth, and finally, code modifications for stateless-reth using verkle, I am doing this project as part of my Ethereum protocol fellowship with EF-stateless team, the project will first aim to integrate verkle in reth, then it will aim to pass all the execution-spec-tests designed for Kaustinen devnet-7(verkle's devnet), latest test fixtures for this devnet were released earlier and are available here: verkle-devnet-7-test-fixtures, I have started implementing these tests, beginning with EIP-4762, EIP-6800, remaining EIPs like EIP-7709 will be implemented subsequently, tough these tests can't be added directly to reth since this is blocked by #10595 , since reth does not have yet tool like geth's evm-t8n for running execution-spec-tests (I may be missing something here), hence I am using currently hive for these tests
aim of this pr is to integrate rust-verkle into reth's storage, tough future iterations will have a native verkle-trie, using Alloy-types inherently
would like reviews, and further discussions on it @rkrasiuk @onbjerg @mattsse @gakonst
@mattsse ig a separate branch for verkle might be needed, will push my changes there, also EF-testing team requires a separate branch for final devnet tests?

@1010adigupta we talked about expectations and verkle in DMs. Due to its complexity, this is not something that we have resources to review rn.

Relaying here -
I'm reopening this because I think this is valuable. However please note verkle is low prio for us right now, so reviews might not be super thorough. I also don't think we want to merge this into main, at least for now, but I'll set up a verkle branch we can merge stuff into - it's ok if this diverges from main, so don't bother rebasing it that much. This will be helpful to see what parts of the node are affected by verkle and can perhaps inform us on future direction no matter if Ethereum ends up doing verkle or some other form of state commitment.

Relaying here -
I'm reopening this because I think this is valuable. However please note verkle is low prio for us right now, so reviews might not be super thorough. I also don't think we want to merge this into main, at least for now, but I'll set up a verkle branch we can merge stuff into - it's ok if this diverges from main, so don't bother rebasing it that much. This will be helpful to see what parts of the node are affected by verkle and can perhaps inform us on future direction no matter if Ethereum ends up doing verkle or some other form of state commitment.

I have changed the base to verkle, please review and merge, I understand the reviews may not be that thorough, but I will try to provide as much info as possible in my prs, for easy reference

Had to follow this method to get rust-verkle, integrated in reth's disk, so that I could start with execution-spec tests for transition, gas-costs etc(eips basically)
I am parallelly designing the trie-from scratch so that we use reth's existing crates, types and procedure as much as possible

Just echoing Oliver's comments to ensure expectations are set!

agreed, adding fixes and more functions

hey @1010adigupta, want to ensure I did not drop the ball here - is there something you need from me in terms of feedback at this stage?

hey @1010adigupta, want to ensure I did not drop the ball here - is there something you need from me in terms of feedback at this stage?

Hey @onbjerg , devnet-7 of verkle tries is going to be released in a week, the major change it contains is of eip-4762, which has gas cost changes to evm and some changes to interpreter, so I am mostly doing my modifications on revm currently to pass 4762 execution spec tests, the current model is working for passing these tests, one these tests have passed I will address the changes suggested in this pr, some tweaks to LMDBX config will be needed though, once 4762 tests are completed, I will push my commits here

gotcha, feel free to ping me whenever, just making sure you're not waiting on me:)


Investigate on k3d integration to support provisioning temporal management clusters

Investigate on k3d integration to support provisioning temporal management clusters. It will be beneficial to support k3d for management cluster creation as it is wrapper for K3s which is a fully conformant production-ready lightweight Kubernetes distribution.
This will hopefully allow us to improve time to provision our temporal management cluster

/help


comments PR #1021

refactoring json2csv functions to make them more usable: hierarchical fields instead of dictionaries
adding header to json2csv functions
handler api should return no values

Btw, I don't really have a problem with handle having a return value, but it must be explicitly documented, particularly because the name provides no intrinsic documentation

@jnothman Yes maybe I understood it a bit literally, anyhow I hope the api is more clear now. Btw thanks for your review and all your comments.

Many thanks @lrnzcig and @jnothman!


Update docs to use KnownAuthorities

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

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

General Guidelines and Best Practices

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

Testing Guidelines

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

What's the motivation behind this? We added AzureAuthorityHosts to be consistent with other SDKs and would have removed KnownAuthorities had it not been in 1.0.


Flutter: Can't find cupertinoicons.tff in flutter generated pubspec.yaml file.

Steps to Reproduce
Specs: I'm using a fresh install of Flutter, Android Studio and Java on Windows 10
Try 1:

Install Dart Extension with VS Code
Create new Flutter project
At DOS prompt flutter run
App as shown in logs

Try 2:

flutter create new project at DOS prompt
flutter run
App as shown in logs

Error is this:
Error: unable to locate asset entry in pubspec.yaml: "assets/CupertinoIcons.ttf".
I can fix the error if I comment this line out of the pubspec.yaml file.
Running flutter packages get within the project folder does not fix this issue.
UPDATE:
I have solved this by manually downloading the CupertinoIcons.tff file and copying into: C:\Users\kilroyjones\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\cupertino_icons-0.1.1
It was odd that this folder was there, but that the assets folder was not created. Anyways, cheers!
Logs
$ flutter run --debug
Launching lib/main.dart on MI 5 in debug mode...
Initializing gradle...                                0.8s
Resolving dependencies...                             1.1s
Running 'gradlew assembleDebug'...
Error: unable to locate asset entry in pubspec.yaml: "assets/CupertinoIcons.ttf".
Error building assets for F:\Flutter\my_app\build\app\intermediates\flutter\debug/app.flx

FAILURE: Build failed with an exception.

* Where:
Script 'E:\Apps\flutter\packages\flutter_tools\gradle\flutter.gradle' line: 421

* What went wrong:
Execution failed for task ':app:flutterBuildDebug'.
> Process 'command 'E:\Apps\flutter\bin\flutter.bat'' finished with non-zero exit value 1

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

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

BUILD FAILED in 8s
Gradle build failed: 1
Run your application with `flutter run` and attach all the log output.

Flutter Doctor
$ flutter doctor -v
[√] Flutter (Channel beta, v0.1.5, on Microsoft Windows [Version 10.0.16299.192], locale en-US)
    • Flutter version 0.1.5 at E:\Apps\flutter
    • Framework revision 3ea4d06340 (8 days ago), 2018-02-22 11:12:39 -0800
    • Engine revision ead227f118
    • Dart version 2.0.0-dev.28.0.flutter-0b4f01f759

[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at C:\Users\kilroyjones\AppData\Local\Android\Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • ANDROID_HOME = C:\Users\kilroyjones\AppData\Local\Android\Sdk
    • Java binary at: C:\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)

[√] Android Studio (version 3.0)
    • Android Studio at C:\Android Studio
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)

[√] VS Code (version 1.20.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Dart Code extension version 2.9.2

[√] Connected devices (1 available)
    • MI 5 • 59630a95 • android-arm64 • Android 7.0 (API 24)

• No issues found!

Ummm, this is pretty strange. When you run flutter packages get in your project, C:\Users\kilroyjones\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\cupertino_icons-0.1.1 is empty?

It's not empty, it's just lacking the assets folder.

I assume this issue is stale.
Please add a comment if you disagree and I'll reopen.


Docs: "auto" is not a valid tooltip placement option

Currently Bootstrap only allow 4 values for placement option: top, bottom, left, right. "auto left" as explained in the document is an incorrect value and will cause error "Tether Error: You must provide an attachment".
http://v4-alpha.getbootstrap.com/components/tooltips/

Why is auto not a valid option? It would be very helpful to have it. I the meantime, the online bootstrap 4 docs are still wrong.

Placement auto has been removed from the tooltip docs, but it is still mentioned in the popover docs.
http://v4-alpha.getbootstrap.com/components/popovers/#options

It's been a year since this issue. Was issue "auto" not suported? Or was it that the documentation is wrong? I feel many of us want he "auto" placement to be a valid flag, instead of fixing the documentation.


[WIP] Chatops integration

Proposal for chatops integration base on #17
As mentioned in the tracking issue:
[ ]  start with defining a Dockerfile for st2chatops
[ ]  st2chatops configuration could be controlled by passing ENV variables to override defaults
[ ]  creating a K8s resource for st2chatops as well as corresponding Helm values
find out more about the why here

Thank you @armab for the suggestions. I ended up doing that after tested the other approaches. as suggested, This is a base for discussions/suggestions.
I took this repository as a stepping stone for the new integration, There aren't much of division form it; However, there might be unnecessary values passed around which is left for cleaning up.
Corresponding  st2-dockerfile PR   In my approach I am using st2chatops package. TBD if there's a better source.
I probably need a few more days to bring it to a clean state, feedback, contributions very welcome as I won't be able to commit time next week.

Hey Sure,
This branch is outdated as I worked on it locally. Will push changes later today.
Thanks

On Mar 19, 2019, at 8:12 PM, Eugen C<EMAIL_ADDRESS>wrote:
@armab requested changes on this pull request.
Can you please sync-up your branch with the upstream master, so we'll see only the diff from this specific change?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/StackStorm/stackstorm-ha/pull/55#pullrequestreview-216365713, or mute the thread https://github.com/notifications/unsubscribe-auth/AE_WySOXcphnXbPpGdU3hKtUqL2_3-Vhks5vYTaCgaJpZM4b2w-r.

Looks like the fork/branch still needs to be synced with upstream properly as Github diff shows unrelated to your PR changes https://github.com/StackStorm/stackstorm-ha/pull/55/files

Since I'm try

Looks like the fork/branch still needs to be synced with upstream properly as Github diff shows unrelated to this PR changes https://github.com/StackStorm/stackstorm-ha/pull/55/files

Might it happen because I have local branch set to something other than master ?
I tried...
git pull upstream

git remote -v shows
origin	https://github.com/rapittdev/stackstorm-ha.git (fetch)
origin	https://github.com/rapittdev/stackstorm-ha.git (push)
upstream	https://github.com/StackStorm/stackstorm-ha.git (fetch)
upstream	https://github.com/StackStorm/stackstorm-ha.git (push)

@armab I think we're better off closing this PR when all the details are cleared and submit a new one with proper implementation. If you agree I'll get started with that. This was just an early setup to better get a grasp of what we're dealing with.

@armab @warrenvw I tried syncing with upstream and after some-time wrestling, I think this is best i got. I am not sure if that's updated since I see merging blocks. But I have an idea. If you agree you can open a new branch for chops so the community can directly contribute to. I think that'll save us time.
I also was able to deploy to  k8 while pulling the image of the st2-docker file. I added some cleanups on mu code which I will cherry peak into the new branch.

@rapittdev As I see, the github diff still shows old content that was already contributed to upstream master branch. We can't open an all-writeable branch for community per Github limitations.
But if you want to open a new PR instead of this one, - that would be fine too.

Sorry for excessive notifications. I tried to sync this fork with the master as much as possible. The current approach is not final but since I received a message from the community members who wanted to contribute, having our fork synced with the master can help them get started easier.
I was still trying to mount env variable in an easy way for the st2chatops docker to use. you're welcome to contribute or use this for inspiration and submit your own PR.

@mosn Sounds good, I’ll start looking into converging this PR into prod state soon.
But after looking at the changes made, - it’s a great work there! Much more cleaner and stable than before 👍

Thanks for the info. I still cannot decide on environment variables  if it’s better to use command arguments or try mounting volumes.  I cannot get the second one to work and I also cannot find a clean way with arguments.
Sent from my iPhone

On 11. Apr 2019, at 11:52, Eugen C<EMAIL_ADDRESS>wrote:
@mosn Sounds good, I’ll start looking into converging this PR into prod state soon.
But after looking at the changes made, - it’s a great work there! Much more cleaner and stable than before 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@armab if you agree, since this was a playground for chatops integration, I close this PR and you can start from your own branch, I think that will also save time. I continue working on this branch working on Ingress with auto-ssl.

@mosn Branches are free and it's easy to create one.
If you think you want to close this PR I'm fine with that, but I would prefer to work based on your code, current PR and keeping this discussion history inplace for future context and preserve that contribution credit for you who started chatops work.

Sounds good to me 👍 let’s keep it here.

On Apr 11, 2019, at 6:15 PM, Eugen C<EMAIL_ADDRESS>wrote:
@mosn https://github.com/mosn Branches are free and it's easy to create one.
If you think you want to close this PR I'm fine with that, but I would prefer to work based on your code, current PR and keeping this discussion history inplace for future context and preserve that contribution credit for you who started chatops work.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/StackStorm/stackstorm-ha/pull/55#issuecomment-482181253, or mute the thread https://github.com/notifications/unsubscribe-auth/AE_WyWzIYhGmGoGiBAuTqcmvIW-5G6WUks5vf1-dgaJpZM4b2w-r.

👍👍👍 Thanks @armab  @warrenvw This chart was already ahead, now it's about to get wings!  Thank you for helping us finalize this.  We're planning to launch our product with Stackstorm-Ha!

@mosn Thank you for starting this K8s Chatops work, making changes and bringing this feature to a working state! Appreciate your contribution :+1:
Just added a little bit of polishing here and there and tried to find possible corner cases. One of them is reported in https://github.com/StackStorm/st2chatops/issues/124
@warrenvw I'll wait for your additional review in this PR and @mosn if you can double-check :eyes: and verify this PR in action too, - that would help before we merge it early next week.

@armab I'll be able to start the integration at 15:00 CET. I will provide updates as I progress through the implementation.

Cluster is ready based on following best practice
https://github.com/bitnami/kube-prod-runtime/blob/master/docs/quickstart-gke.md
Going to start with Stackstorm and chatops now

it works great too!

@armab I had to move to EWC 3.0 because of this. So far the improved workflow was able to do 5 hours job in 30min. Very helpful with a syntax error. As for chatops I'll finish converting all my mistral chains and then look at that, hopefully, I can work out detailed feedback later in this week.


Allow GA cache to scan in batch

So we not lose success ones when something bad happens. This is less useful in normal cases. But in perf test, we have to scan thursands of repos by this code.

LGTM.


email content only displayed half way

Since update to Android 5.02 on the Sony Experia Z2, the lower part of the email is blank. Not always, but mostly.
Will try to attach a Screenshot

Duplicate of issue #598


Disconnection after sending message to the hub

Description
Disconnection occurs after sending any message to Small Technic Hub when using Chrome-based browser under Linux.

Environment

SteamOs, Chrome, v120.0.6099.224
SteamOs, Edge, v120.0.2210.144
Ubuntu 22.04, Chrome, v119.0.6045.199

Workaround added, use 'connectHub(bluetooth, { useLinuxWorkaround: true })'