help: Document "User list style" display setting.

Documents the "User list style" (under Settings > Display settings)
Adds new section to /help/status-and-availability#statuses

Fixes #23492.
Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Automated tests.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Links.

I updated the PR and screenshots to address your feedback.

Hm, I actually just realized that what I suggested for the tip is already described in https://zulip.com/help/status-and-availability.
It's probably still useful to have a tip, but maybe we can match the wording better? E.g.:

You can always hover over a user's name in the right sidebar to view their status message if they have one set.

I'm also thinking we should adjust https://zulip.com/help/status-and-availability#view-a-status... Perhaps change the tip to something like this?

You can also click on a user's profile picture or name on a message they sent to view their status in their user card, or configure status text to always be shown in the right sidebar.

I updated the PR and screenshot in the description with the adjustments you suggested.

Works for me!

Merged, thanks @drrosa!


Deleting user can cause internal server error

Note: this user shared a widget. Error possibly related to this?

What do you mean with "shared widget"?

There is an option to share a widget: on the dashboard, in the top right of a widget, click the three vertically aligned dots, then a submenu appears with the "share" option.

I can't reproduce the problem. The docker image is updated, can you pull and test again?

I expect to have time for that next week.

I cannot reproduce with docker image 1.0.0-RC1, so closing this issue.


Update service-fabric-backuprestoreservice-quickstart-standaloneclust…

…er.md
Without providing SecretEncryptionCertX509StoreName argument cluster throws Invalid argument error. I put "My" as a default value, which is recommended according to: https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-cluster-fabric-settings .
For more info please contact<EMAIL_ADDRESS>

@Nhemisirmkow : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Note that publishing in this repository is suspended until Tuesday, March 2. Merging will resume at 7:00 AM PT Tuesday. Authors can still review and sign off on PRs during this time. Thanks.


Unsecure Default TLS Policy because of the AGIC controller using old network API version (< 2023-02-01) and deprecated Azure SDK

Looking at this line of code:
https://github.com/Azure/application-gateway-kubernetes-ingress/blob/8bc1e63c3714667bb59f8533fd39b8970e77bd2b/pkg/azure/client.go#L15
I understand the AGIC controller is using a deprecated Azure SDKs  and should upgraded following this guide:
https://github.com/Azure/azure-sdk-for-go/blob/main/documentation/MIGRATION_GUIDE.md
There is an impact on security. Because the deprecated Azure SDK consumes an ARM API version lower than 2023-02-01 in the Default TLS policy the minimum protocol version is set to TLS 1.0
This was originally reported by a customer here:
https://github.com/Azure/terraform-azurerm-aks/issues/532
How this is actionable ?
Please patch the code to use API versions 2023-02-01 or higher, then the minimum TLS protocol version is set to 1.2
Documentation reference:
https://learn.microsoft.com/en-us/azure/application-gateway/application-gateway-ssl-policy-overview#default-tls-policy
Cc: @chixcancode @JackStromberg

In addition to the default here, the workaround of applying the annotation explicitly on the Ingress resource doesn't result in the SSL policy being applied by AGIC:
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: my-ingress-all
  annotations:
    kubernetes.io/ingress.class: azure/application-gateway
    appgw.ingress.kubernetes.io/ssl-redirect: "true"
    appgw.ingress.kubernetes.io/ssl-policy: "AppGwSslPolicy20220101"

despite the annotation being visible in the portal:

@palma21 @JackStromberg could you please check if this GitHub issue is actionable to the AKS product team ?
My concern is that Application Gateways created by AGIC have a Default TLS policy with the minimum protocol version set to TLS 1.0
thank you

Acknowledging this ask, but don't have an ETA on when the behavior will change in AGC.
If using Application Gateway for Containers, we enforce TLS 1.2 or greater making it a non-issue in that solution.


Fix outline colors on subscribe links and volume div

Subscribe links outline color on focus was set to black, which was invisible against the black background. Updated to theme yellow.
Also, volume div was set to the webkit default blue while its sibling elements were set to the theme yellow. Updated to theme yellow.
P.S. this is my first pull request!

Hey Sergiy, I think I fixed the issue -- could you confirm? Thanks!

@416serg did I fix the issue you noted? I saw you added the await additionalinfo() label, wasn't sure if that meant you were waiting on me or not.

@ben-foster hey Ben, sorry for the confusion. However, I am still not sure as to why the package-lock.json file is being committed in the first place 🤔There seem to be some package differences there. Would you be able to just commit the _player.styl, _subscribe.styl and style.styl files so that it's an easy and clean merge without changing anything in package-lock.json? 😅 Thanks!

@ben-foster this is what I think happens in the background of git https://giphy.com/gifs/git-merge-cFkiFMDg3iFoI 😂
Looking forward to the new PR, closing this for now!


[BUG] CloudException is thrown with status code 200 when using SqlManagementClient.Databases.CreateOrUpdateAsync(...)

Description/To Reproduce

Get database properties from azure, keep it in memory as a backup.
Move the database out of standard elastic pool to premium standalone within same database server.
Try to copy the database to another server, but stop it because it took too long.
Revert database to original settings using SqlManagementClient.Databases.CreateOrUpdateAsync(...) and passing original database properties received from the 1st step.
Request throws CloudException with message The operation could not be completed because the read scale value specified is not supported for a 'Standard' database.. The response object in the CloudException returns status code 200.
Repeat steps 4 and 5 with retry logic.

Eventually, the code exhausts all the retries and fails, however in Azure everything seems to have worked fine and the database is in its original state.
Expected behavior
CloudException should not be thrown for status code 200.
Actual behavior (include Exception or Stack Trace)
CloudException is thrown with status code 200 and message  The operation could not be completed because the read scale value specified is not supported for a 'Standard' database.. Operation seems to be successful when checked on azure portal.
Environment:

Name and version of the Library package used: <PackageReference Include="Microsoft.Azure.Management.Sql" Version="1.19.0-preview" />
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): Windows Server 2016 DataCenter - Service Fabric 7.0.470.9590 - net472

@erich-wang: Would you be so kind as to help route this?

@jsquire Jesse, this issue doesn't seem to be related to SQL Managed Instance, but rather SQL DB.
Can you please review and update labels? Thanks!


Fix IllegalStateException in MultiFileMetadataSourceImpl

The exception occurs when getMetadataForNonGeographicalRegion is called with a country calling code which doesn't correspond to a non-geographic entity.
Also fix a typo in PhoneNumberUtil.

Looks good to me - please update release notes, then I'll merge.

Updated release notes.


Upgrade clang version used in CI to 14.0.5

What does this PR do?
This PR upgrades the clang and llvm version in the CI pipeline from 11.0.1 to 14.0.3.
The table below compares the sizes (in MB) of the binaries.

clang arm64
clang amd64
llc arm64
llc amd64

11.0.1
65
62
22
21

14.0.3
77
73
30
29

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

There are several places in the CI where the clang-11.0.1.$ARCH binaries are used. Those should be updated too.


Created Gameplay tab in ImGui

For https://github.com/nipkownix/re4_tweaks/pull/366#issuecomment-1364454566

UseSprintToggle and LimitMatildaBurst are technically more like controller tweaks. I suppose we could duplicate UseSprintToggle in the Keyboard and Controller tabs, and LimitMatildaBurst in the Mouse and Controller tabs?

Moved stuff to a new .cpp file and changes our .ini a bit. Should probably make sure everything still works correctly.. I've messed that up in the past ._.

UseSprintToggle and LimitMatildaBurst are technically more like controller tweaks. I suppose we could duplicate UseSprintToggle in the Keyboard and Controller tabs, and LimitMatildaBurst in the Mouse and Controller tabs?

Hmm, I think since the implementation for them is the same for KB/M and controllers, leaving them in the Gameplay tab should be fine?

Oh, it's a shame that moving settings between tabs resets people's existing config settings. Maybe that's why you were opposed to new tabs?

Any reason the ini reader shouldn't just check for the whole file?

Hmm, just to keep things organized, I guess?
It isn't that big a deal IMO. But should probably add a warning in the release changelog mentioning that some settings may have reverted back to default...

What about grouping DisableQTE and AutomaticMashingQTE within the same column, like in https://github.com/nipkownix/re4_tweaks/pull/366#issuecomment-1364454566? I can't tell if you didn't like that or if you just merged over it with your existing changes, heh.

Oh shoot, I think I may have accidentally overwritten that when changing the order of the settings. Sorry! I does look better.

Everything looks good to me. Thanks again, @pas-de-2.


Support colours

I am on channel with relaybot (bot which sends messages between channel in multiple networks) and to make it easier to separate people it colourizes relayed nicks. Communi Sailfish doesn't so those, but seeing them would be helpful.
Other thing which could replace colour support was parsing the lines of the relaybot so it's not needed, but that is probably more difficult and at least WeeChat script for it makes other bot commands fail, so the colours would be a lot better.

More important case would be OTR with BitlBee as it uses colour codes:

green, OTR with trusted party
red, OTR with untrusted key

Currently I don't know so easily if the query is encrypted or not.


nms_cuda RuntimeError

torchvision 0.3.0
pytorch 1.1
auto order_t = std::get<1>(scores.sort(0, /* descending=*/true));
auto dets_sorted = dets.index_select(0, order_t);
index_select break down.
RuntimeError: invalid argument 3: Index is supposed to be an empty tensor or a vector at /pytorch/aten/src/THC/generic/THCTensorIndex.cu:536
Thanks for your help!

when does this error happen? Can you share an example script that reproduces the problem?

@fmassa
The dim of the score tensor in python is (N, ), C++ API get a (N, 1) score tensor, and then the sort function got the order_t (N, 1), so, index_select function broke down. I just use it normally. And I change the code like this, it worked.
auto order_t = std::get<1>(scores.sort(0, /* descending=*/true));
order_t = order_t.squeeze(-1);
auto dets_sorted = dets.index_select(0, order_t);

How are you using the code? Is it entirely in the C++ API of torch? I don't yet understand, could you explain a bit more?

Just using python. I debug it in the cuda code, and I found that score's dim change from [N] to [N, 1]
box [N, 4], score[N]
keep = nms(boxes=bbox, scores=score, iou_threshold=self.nms_thresh)

Sorry, I make a big mistake. I checked my score tensor in RPN, it's [N, 1], before, I just checked score tensor in rcnn, sorry about that.
Thanks for your help. Maybe I could close this issue.

@lizhen2017 note that we also have faster r-cnn models directly available in torchvision now.


Holographic Camera should be child of Player not UNetAnchorManager

Without this, when the SpectatorView HoloLens comes online, the Holographic camera does not follow the remoted pose defeating the whole purpose of SpectatorView.
With this change, the correct calibration is also used...

@ei2kpi,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Ahhhhhh! Got it... makes complete sense... I was trying to run the sample without the color feed running and so it never actually hit those lines. (Also didn't help that VS had hidden that region of code :) )
It also makes sense that you want to update the Camera Pose only when you get a matching valid ColorFrame so you get a Hologram frame that is as close as possible to the Color Frame... make sense! Thanks!
I'll try it again after my BlackMagic card arrives and then report back / file a bug if I find any issues


Problem translating firebase UI in iOS swift project

I have:
authUI?.customStringsBundle = Bundle.main
And FirebaseAuthUI.strings in my swift project and I need to translate firebase auth UI for iOS so I can realese my project but only few strings in app are translated other not. Where is the problem? Off course I translate it all!

I resolve all, only cancel button item in top left I can't translate!

The cancel button isn't a string provided by FirebaseUI; it's just a system cancel button. The same should be true for all other cancel buttons in your app.

I know that but I must translate it, I must find the way for it. I already lose one day :)

Since this is a string in a system button, I'm going to close this as it's not a FirebaseUI issue.
Keep in mind system buttons will use whatever localization language defined in the device's settings. If you want to override this, you'll have to customize the entire button since you don't have control over the string loading.


Remove useless dependencies and refact your AssistantButton.

Thanks for your email and I'm glad to provide some suggestions.
It seems the project have some useless dep like langchain?
And the AssistantButton component are very unclean especially in:
<motion.div onClick={() => {...}}>

Thanks! I'll remove the unused dep and improve the component 🙏


Fix: Spelling mistake in packagemanifests deprecation message

Description of the change:
A spelling mistake "packagemanifes" has been fixed in two places
Motivation for the change:
The spelling was incorrect

@camilamacedo86 That should be done now, thanks!

Hi @jjkiely,
It gets lost and seems that is failing because is missing a rebase. Could you please rebase it with the master for we get it merged? Also, please feel free to ping me in the slack to merge when it be passing in all tests.

/restest


Memory usage between GEMM convolution and Winograd convlution

Hi,
Compute Library - 18.05
Can I please know the memory usage comparison between a CLGEMMConvolution layer and CLWinograd layer ?
I know how CLGEMMConv layer works using Im2col  but unaware of winograd layer and its internal buffer allocation . I wish to know the memory usage of winograd layer, if it really uses less memory than the case of Im2col in GEMMConvolution  ?
Awaiting for a response,
Thank you much

Is there any estimate of additional buffer usage in winograd layer ?

Hi,
Any info available ? Kindly share, if available
Thank you,

No we don't any data I'm afraid, you should be able to measure it by yourself though.

Hi @ved27,
I think you can find some information in my recent presentation at the embedded vision summit 2018 (https://www.embedded-vision.com/platinum-members/arm/embedded-vision-training/videos/pages/may-2018-embedded-vision-summit-iodice#comment-form). At minute 30, I evaluate the memory footprint for both GEMMConvolutionLayer and WinogradConvolutionLayer for the case 3x3 with 2x2 output tile. You should be able to extend the formula to other cases such as the 3x3 convolution with 4x4 output tile or the 5x5 convolution with 4x4 output tile.
i.e. 3x3 with 4x4 output tile
Input transform memory: 36*(W/4)(H/4)Cin
Output transform memory: 36(W/4)(H/4)*Cout
Hope this can help you.

Hello @ved27,
As per @gmiodice comment you can work out the space requirements of each convolution implementation.
Closing this issue. Reopen if needed.


Desynchronizated torque layers

With the last beta release, my map with two torque layers that were in sync now shows desynced.
Both datasets have a timestamp column that has the same range:
MIN(timestamp) = 2018-06-03T22:15:18Z
MAX(timestamp) = 2018-06-03T22:38:09Z
The first dataset is just points and the second one is a bunch of lines in a n lines:1 point relation, all n lines for a point has the same timestamp.
In the following GIF you can observe the issue (top image) and the expected result (bottom image) that is the behavior of the previous beta release.

Sorry for the videoconsole quality like GIF, but GitHub doesn't let me add images bigger than 10MB
Version
The issue showed in 0.5.0-beta.3
Last known working version was 0.5.0-beta.2
Example
https://skgsergio.github.io/cartovl/lightning-lines.without-hack.html
Code: https://github.com/skgsergio/skgsergio.github.io/blob/master/cartovl/lightning-lines.without-hack.html

I had a look the code and I discovered two fancy functions: {get,set}SimProgress():
> console.log(lightnings_layer._viz.filter.getSimProgress() + '\n' + lines_layer._viz.filter.a.getSimProgress());
0.4117499999999316
0.40599806518554815

And setting the progress to 0 makes both torques to be in sync again 😃
> lightnings_layer._viz.filter.setSimProgress(0); lines_layer._viz.filter.a.setSimProgress(0);

> console.log(lightnings_layer._viz.filter.getSimProgress() + '\n' + lines_layer._viz.filter.a.getSimProgress());
0.030016666666667365
0.030033333333333634

> console.log(lightnings_layer._viz.filter.getSimProgress() + '\n' + lines_layer._viz.filter.a.getSimProgress());
0.8711999999999919
0.8712166666666591

Interesting issue, I guess we should think on a standard way to sync animations.

At viz object level: viz0.syncAnimation([viz1, viz2])
At layer level: layer0.syncAnimation([layer1, layer2]);

maybe this should be the default behaviour
cc @rochoa @elenatorro @davidmanzanares @Jesus89

We could automatically sync all the animations by having a global list of animations that still haven't been loaded and a list of loaded, yet waiting list:

When an animation is created it is pushed to the TODO list
When an animation is loaded it is removed from the TODO and it is pushed to the WAITING. if the TODO list is empty, play all the animations of the WAITING list, and empty that list.

so from the user point of view, the animations will be always synced right?

Not always, if they manually use stop/pause or if they add a layer after other layers have been loaded, then the animations won't be synched.

ummm so what we do in case we need to load 2 layers asynchronously and then synchronize the animations?

viz1.anim.setProgress(viz2.anim.getProgress())

As @skgsergio reported and I reproduced this regression has been re-introduced.


Disable Violation Pragmas

feature enhancement idea:
When drafting an API, the API designer usually runs Zally multiple times.
He than checks Zallys violation output and decides how to handle each violation.
Sometimes API is changed as to avoid the violation, sometimes API is not changed on purpose -- often the case for SHOULD and MAY violations.
Improve API designer convenience via support of pragma directives inside API definitions that disable checking of specific rules for specific OpenAPI elements.
By this, API designers can avoid efforts for redundant checking of violations for multiple Zally runs during API evolution.
It is also a must-feature for integration of Zally in automated CD or operation monitoring tooling.

@tfrauenstein do I understand it correctly that you propose to modify swagger file itself? If so, can you give an example how these pragmas may look like? As the file with pragmas still needs to stay compatible with OpenAPI format.
As an alternative I can propose to add additional field to json payload (and corresponding cmd arg). This way we could avoid API file modification.

@mkulak Yes, I would prefer annotation inside OpenAPI definition to avoid multiple sources and correlation problems. As JSON does not support comments (IMO bad design choice, however) the only way I see is via specific fields. OpenAPI explicitly supports enrichments via vendor specific "x-" tags -- this would be the straight forward way to go for Zally pragma extensions.

Really nice idea, I think many linters are supporting that.
We can also go a bit further and let Zally user disable rules from command line tool configurations.

Hi guys, so I'm thinking of starting to implement this functionality. Let's discuss our visions of how this would look like.
We want to disable specific rules for specific json nodes, is that right? I checked the OpenAPI docs, basically every element contains the possibility to add vendor-specific tags (as @tfrauenstein correctly pointed out).
As I see it, we should agree on some specific syntax to disable any set of rules, that can be inserted on any level. Is this also the goal as you see it?
In my mind it should look kinda like this:
{
  "swagger" : "2.0",
  "x-zally": {
    "disable": [
      "SnakeCaseInPropNameRule",
      "LimitNumberOfSubresourcesRule"
    ]
  },
  "paths" : {
    "/products" : {
      "get" : {
        ...
        "x-zally": {
          "disable": [
            "PluralizeResourceNamesRule"
          ]
        },
      }
    }
  },
  ...
}

Implementing this seems to be a little tough though, because right now every rule executes it's own custom code getting the whole swagger doc as a parameter. Therefore checking these configs/exclusions will have to be done in every Rule separately.

@innokenty how about making it a bit easier? We have rule codes, e.g. M001. My proposal will be the following:
swagger: 2.0
x-zally-ignore:
  - M001
  - S001
...

and the same for inline definitions.
Maybe to keep it simple we can start with global level ignore definitions, it will already bring a lot of value and then later we can implement inline ignore statements.
What do you think?

Cool, codes are definitely better.
Yes, top level exclusions must be simple. I can implement that and then we'll see if we need more fine grained control.

@innokenty one more proposal: can we also add an X-Zally-Ignore header support? It might be useful later for CLI tool to locally disable some checks.

@netme Andrii suggests a parameter instead of a header, and I like it more actually. What do you think?
?ignore_rules[]=M001,S001&top=kek

hm.. I personally not a big fan of that idea, however we can use access log later for some statistics.
@maxim-tschumak , @maschleg what do you think about it?
If you will decide to go for URI params, please make them compliant: http://zalando.github.io/restful-api-guidelines/http/Http.html#should-explicitly-define-the-collection-format-of-query-parameters
Another option can be: CLI client injects ignore field in Swagger spec, then no changes will be required.

ough, I think injections are way worse and more complex, than params

Great ideas! my opinion:
Server implementation
I would go for top-level only x-zally-ignore field in the Swagger definition. This already has a great benefit for the users. We can still implement json-path based rule deactivation eventually.
Communication Clients <-> Server
Ignoring some rules has a direct effect on the query results. Proprietary X-Headers are deprecated. Only defined headers can be used (additionally they should be end-to-end) related Guideline.
I would prefer to have query parameter in place (csv or multi).
Alternatively, we can do filtering on the client-side.

Cool, then we have agreed :-)

Thanks for your initiative to move this feature forward.
I like the proposals so far, but still have some questions/hints:

We need a concept of Rule-Id (Proposed to be a code like "M001"); requirements here:

Rule-Id is mastered by the Guidelines and easy accessible here
--> e.g. Rule-Id should be added as rule headline suffix
Rule-Id must not change
--> to be ensured by API Guild
Rule-Id should be usable as link into Guidelines
--> guideline PR
Rule-ids / links should be used by Zally (e.g. for violation notification, disabling etc.)
--> Zally impl: find a (simple) way to have Zally Rule-Ids as a consistent copy of guideline Rule-Ids
Hint: Out of scope for the time being: versioning of rules and its ids; rules may evolve and become more or less restrictive; rule check disabling holds for all versions

Disabling of specific rules should be supported as local as possible.
For instance, rule "Should: Limit number of Sub-Resource Levels" only applies to resource paths and disabling should be for a specific path -- technically an OpenAPI "path item object" instance -- and not for the complete API. Hence disabling should only become effective, if it is done within the smallest context of the OpenAPI element relevant for the rule. Is this requirement agreed?

@tfrauenstein thanks for your input.
Rule-Id
I propose to switch to static human and machine readable strings which include the rule title without the type of the violation (e.g. secure-endpoints-with-oauth2.0). It has multiple advantages:

Static, no changeable names (identifiers of the rules)
Human-readable, SEO friendly links to the Guidelines
Links can be auto-generated
Type of the rule can be safely changed

This will be easy to implement after the Guidelines are migrated to ASCIIDoc.

Disabling of specific rules should be supported as local as possible
I'll create an issue for this feature request.

Unrelated: should I implement some parameter in the cli tool? If yes, then what did we agree on at the end?

I think #446 is more important now @innokenty.

Also much more complicated

@maxim-tschumak thx for response -- it would be great if you can take care of supporting the rule-id concept including consistency guidelines in course of its ASCII-doc-ization
Wrt. rule-ids standards I'm not convinced of non-numeric representation.

human-readable is IMO less important -- easy rule access (via links) based on id is important
ids as readable names tends to be too long -- at the end a (more or less short version of) the rule headline; ids short be short, easy to short-term memorize and easy to write in disabelling config and easy readable as appendix all rules headlines
yes, ids should not be generated from content that may change -- so just simple sequence number (w/o rule must/may/should info) is to be preferred

@tfrauenstein, @maxim-tschumak thanks for your input!
We have introduced Rule-IDs the time we introduced ignore-rules server configuration and we didn't thought on possible bigger scope for Rule-IDs.
Current implementation of rules is assuming that IDs are:

Sequential with prefix (we use C prefix for MAY because before it was called COULD and renaming it to M will conflict with MUST)
Unique: we have an integration test which validates the uniqueness of every newly introduced rules

Would be really great if we can have cross-links between rules and API guidelines. This topic is also related to #150 and #151 and hopefully it will also help to increase visibility for API guidelines and Zally projects.

@tfrauenstein @netme I'll take you points into account and come up with an ultimate Rule-Id concept for the Guidelines and Zally :)

@maxim-tschumak I like the word "ultimate" ! 👍
Please let me know if I can help.


src/bindings/python: Fix destructor for sfp_recv

In order to deallocate the memory for the "dataout" pointer it is required to
use "csp_free". Since "dataout" is not a "csp_packet_t" pointer the
"pycsp_free_csp_buffer" destructor fails leading to memory leak.
Signed-off-by: Iliya Iliev<EMAIL_ADDRESS>

Hi there.
It looks like a good catch.
This code was from @arrooney, I should ping.


message_edit_history: Dynamically load modal rather than statically.

Merged, after a small change to the documentation in the first commit to make more clear the caller's responsibility, since the previous language was a bit ambiguous.
Thanks @ganpa3!
One thing that might be a nice follow-up is changing the way we pass the auto_remove parameter to a format that lets us require callers to pass it by name, since open_modal("name", true) is hard to read.


Disposal units wont power on if a large amount of units are inserted prior.

Description
Disposal units wont power on if previously turned off if a large amount of units are inserted after turning it off. When pressed the button will flash for a second. Seems to break disposal units, the power button gets removed from the right-click options aswell
Reproduction

Put a couple dozen objects into an unpowered connected disposals unit (I personally used tiles.)
Try to power it back on, in both the UI and not with the UI.
Wont turn on.
cry.

Will test later if this perma-fucks a disposals unit, last i checked it did (unanchoring and reanchoring to try and fix.) uncertain.

https://github.com/space-wizards/space-station-14/assets/8107459/85c73c04-5272-4451-9d44-11e25a931c73
Can't reproduce personally.

Content.Client_RX3UWs7pO6.mp4
Can't reproduce personally.

strange, might be tiles only then? ill try recreating the circumstances it was in, i think i might have a replay file as well, just need to find it.

Tried various things
anchroing/unanchoring
shutting off power from the apc, and the UI
moving it around
having/trying to put items in, and not in it, when doing the above
Can't reproduce.
For this part of the issue in particular
"power button gets removed from the right-click options aswell"
The only way I can see that happening is if the server isn't responding.


How to use vue-highcharts with highcharts-pattern-fill?

Hi @weizhenye, thanks for the amazing work you've done.
I'm starting to learn Vue and I'm having a hard time to figure out how to use highcharts-pattern-fill (v2) with vue-highcharts.
In a common single page I just need to drop a <script scr=".../highchart-pattern-fill-v2.js"> in the head, but in Vue if I do the same I get an error saying Highcharts is undefined which is probably because I'm trying to run this file before loading Highcharts.
So, could you please provide any direction or example to how achieving vue-highcharts with highchart-pattern-fill?
Thanks a lot.

I was looking some old issues and I got a similar example that worked after little changes:
import Vue from 'vue'
import VueHighcharts from 'vue-highcharts'
import Highcharts from 'highcharts'
import PatternFill from 'highcharts-pattern-fill'
import App from './App'

PatternFill(Highcharts);

Vue.use(VueHighcharts, { Highcharts })

Thanks folks!

Still does not work

@PhouvanhKCSV https://jsfiddle.net/h85mx0bt/1/

@weizhenye Thank you.


[Track One] Tests for Service Fabric Processor error handling are unstable for CI and nightly runs

Summary
Exception tests for the ServiceFabricProcessor have recently begun to be non-deterministic during CI and nightly test runs, failing intermittently but frequently.
An example failure can be found in the nightly results fromSeptember 3.   The failure looks like:

Another example failure can be found in PPR #7467.   The failure looks like:

Scope of Work

Analyze the tests to understand the point of non-determinism and decide on an approach to stabilize for CI and nightly runs.

Refactor the tests as needed to ensure that the results are deterministic and consistently pass.

Out of Scope

Changes or refactoring in other areas.

Success Criteria

The suite of tests for the ServiceFabricProcessor no longer causes intermittent, but frequent, failures during nightly runs.

Any changes did not cause regressions elsewhere; CI and nightly test runs pass consistently and reliably with deterministic results.

Is there any guidance for diagnosing issues which only occur in the testbed? Is it possible to turn on tracing, for example? These tests all run reliably on my dev box, and the only info I currently have from the testbed is which assertion failed, so I'm reduced to guessing, and my best guess (increasing the startup timeout) obviously wasn't correct.

@jsquire, do you have any insights to James' questions?

I'll ask that @danieljurek and @weshaggard weigh in with thoughts for potentially enabling a more verbose logging or tracing within the pipelines.

For what its worth, I do see these intermittently fail when running locally with the command line runner against the netcoreapp2.1 framework, both under Linux (WSL) and Windows.   It'll often behave the first time but a few consecutive runs tends to bring out the intermittent failures.
Specifically, I'm issuing: dotnet test --framework netcoreapp2.1 --filter TestCategory!=Live
The main difference in the pipeline is that it is running both the track one and track two tests in parallel across the target platforms.  I'm speculating that the increased amount of resource contention is causing the timing to be more non-deterministic than you're seeing with a focused local run.   I've had to run down a set of these in the track two code as well, and believe that was the underlying cause for those as well.

I'll ask that @danieljurek Daniel Jurek FTE and @weshaggard Wes Haggard FTE weigh in with thoughts for potentially enabling a more verbose logging or tracing within the pipelines.

Yes I think it is a good idea to enable some more detailed logs, however I suggest they go to a file instead of the console and we attach the logs as artifacts to the pipeline. @chidozieononiwu is this something you can please file an issue on and add support for?

I'll ask that @danieljurek Daniel Jurek FTE and @weshaggard Wes Haggard FTE weigh in with thoughts for potentially enabling a more verbose logging or tracing within the pipelines.
Yes I think it is a good idea to enable some more detailed logs, however I suggest they go to a file instead of the console and we attach the logs as artifacts to the pipeline. @chidozieononiwu is this something you can please file an issue on and add support for?

New issue filed here

As far as Service Fabric Processor in particular, I expect to work on a new preview soon, and as part of that I will see if I can reimplement the tests in a more deterministic way.

Adding another test for investigation here:  Microsoft.Azure.EventHubs.Tests.ServiceFabricProcessor.CheckpointingTests.CheckpointBatchTest, which has begun misbehaving on .NET Core.    More details can be found in #7492
I'm going to mark it to be skipped, pending the future test work that James has mentioned.

We are evaluating the state of this issue, and will report back with a status update by 11/4/2020.

The Microsoft.Azure.EventHubs.ServiceFabricProcessor package is in preview and there are no plans for a stable release. This package will soon be deprecated and therefore we won't be taking any changes.


VectorRotate() function

Rotates vector around the pivot point by axis and angle. I took it from openFrameworks and modified for raylib. I find this function very helpful :) You're free to modify its name, parameters placement or anything you want.
Example:

    InitWindow(800, 450, "rotate vector test");

    Camera camera = {{ -5.0, 5.0, 5.0 }, { 0.0, 0.0, 0.0 }, { 0.0, 1.0, 0.0 }};

    Vector3 cubePosition = { 2.5, 0.0, 0.0 };
    Vector3 rotationOrig = { 2.5, 0.0, 2.5 };

    while (!WindowShouldClose())
    {
        // Rotate cube position by new center (blue sphere), X axis and custom degree
        VectorRotateAround(&cubePosition, rotationOrig, {1.0, 0.0, 0.0}, GetFrameTime()*100);
        camera.target = cubePosition;

        BeginDrawing();
            ClearBackground(WHITE);
            Begin3dMode(camera);

                DrawCube(cubePosition, 1, 1, 1, RED);
                DrawCubeWires(cubePosition, 1, 1, 1, MAROON);

                DrawSphere(rotationOrig, 0.2f, BLUE);

                DrawGrid(10.0, 1.0);

            End3dMode();
        EndDrawing();
    }
    CloseWindow();

Nah, forget about it - it's eulear rotation :-1: Should find quaternion one later...

Haha, digged into raymath more and found founctions which do exactly this but shorter:
void VectorRotate(Vector3 *v, Vector3 axis, float angle)
{
  Matrix a = MatrixFromAxisAngle(axis, angle*camLookSpeed);
  VectorTransform(v, a);
}

It's still eulear, though...

There are several versions of MatrixFromAxisAngle(), those functions need to be tested and probably still require some work... Including the use of quaternions!
Actually, it's in my TODO list... I should move that list to github issues! I need some 72 hours days... :P


[WIP] Update trajectory_highlighter

This updates trajectory_highlighter for the most recent changes in Makie.

Fix UndefVarError for color
get_series_idx now returns the adjusted index

This is not yet ready because changing the alpha for the histograms does not work due to https://github.com/JuliaPlots/Makie.jl/issues/617

Okay, let's wait a bit until the Makie-end has some progress. I will also add some updates into this PR, because in #7 I change the design of InteractiveChaos a bit to use Requires (since now it provides applications for pretty much all of JuliaDynamics, hooray!), so some headers need to be added here and there.

The next step is to wait for a solution to https://github.com/JuliaPlots/Makie.jl/issues/619

@SebastianM-C the linked issue was closed, do we wait for something else, or time for me to also review/help/commit/merge ?

Sorry for the delay. I'll implement a workaround a move on with this. I think that waiting for an API in Makie would take too long.

@SebastianM-C I've just run the code locally and it worked fine for me!
I've noticed that MakieLayout is about to be merged into Makie though. Do we have to update this code to use the Makie layout stuff instead of the old makie stuff?

In fact, is it possible to do bar plots in MakieLayout? Because if yes, only the 3D stuff use old code.

The most fragile part is computing the index correctly. I tried to hardcode as little as possible, but it could break again in the future.
The best solution would be to use an API supported by Makie (https://github.com/JuliaPlots/Makie.jl/issues/240#issuecomment-645615812). One of the current problems is that if you manage to click on an axis line (from the grid), you could potentially get some strange errors and undefined behavior.

is it possible to do bar plots in MakieLayout

You mean to layout the composed scene with MakieLayout instead of vbox?
I don't think it would be a good idea to hardcode MakieLayout-specific code now, as from what I understand, things like LAxis could disappear after the merge (I'm not 100% sure, better ask someone who knows better).
We could wait for the merge and then update the code with the new API (and maybe we won't have to make a distinction between the 2D and 3D cases).


The blob tags can be edited in one SAS attached secondary blob container with all permissions

Storage Explorer Version: 1.29.0-dev
Build Number: 20230221.6
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Ventura 13.2.1 (Apple M1 Pro)
Architecture: ia32/x64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Blob Containers.
Create one blob container -> Upload a blob.
Right click the blob container -> Click 'Open Secondary Location'.
SAS attach the secondary blob with all permissions.
Switch to the attached secondary blob container.
Right click the blob -> Click 'Edit Tags...'.
Check whether the blob tags are read-only.

Expected Experience
The blob tags are read-only.
Actual Experience
The blob tags can be edited.
Additional Context
In previous release 1.24.3, the 'Edit Tags...' action is disabled in one SAS attached secondary blob container.

Verified this issue on the build main 20230304.8. Fixed. So, we are going to close it.


Feature/weather anywhere

Hi there @genego-io ! Was this PR closed intentionally? Is there anything we can do to help?

@aconchillo Yes it was closed intentionally, but just for now! I am planning to further work on this together this before I contribute.


Fixed missing override from HistoricalReturnsAlphaModel

Description

Adds overrides to HistoricalReturnsAlphaModel.
Related Issue

Motivation and Context

Using HistoricalReturnsAlphaModel currently gives:
Runtime Error: Types deriving from 'AlphaModel' must implement the 'IEnumerable<Insight> Update(QCAlgorithmFramework, Slice) method.	 (Open Stacktrace)

Requires Documentation Change

How Has This Been Tested?

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] My branch follows the naming convention bug-<issue#>-<description or feature-<issue#>-<description>

@mchandschuh thanks, you're welcome!


Fixed incorrect answer

QuestionHelper is available: Symfony\Component\Console\Helper\QuestionHelper

Hello @inikolaievskyi,
thank for you contribution, but you forget that the Symfony (2) certification is based on 2.3 where the QuestionHelper wasn't available.
We may reword a little bit this question, what do you think ?
Mickaël

@mickaelandrieu that case there is no sense to change that question. I really forgot to check in symfony 2.3  but did it in 2.8

@inikolaievskyi yup, for now the Symfony 2 certification still follow the Symfony 2.3 program.
I think I will create a new deck of questions dedicated to Symfony 3 after the 3.4 release (LTS) to avoid any confusions.
Anyway, thank you for your contribution and good luck for your exam :)


SyntaxColoring: Highlight keywords immediately after periods as identifiers. rdar://33805178

@swift-ci please smoke test and merge


operator hive-operator (1.2.4164-b71778a)

Update Hive community operator channel alpha to 1.2.4164-b71778a
/hold

/unhold

/merge possible

/merge possible

/merge possible


OnTickerReceived receives only one message and stops

OnTickerReceived receives only one message and stops.
OnErrorReceived and OnWebSocketError are without any error.
Doesn't work with and without authentication.
Any hints?

hey @m1stegmann. what channels are you subscribing to and what product types are you using? I am able to receive all messages

Sorry, if i turn sandbox off it's working!


CAMEL-13701: Implementing PostgreSQL Replication Slot consumer Camel component.

See CAMEL-13701

Agree with @onderson about package naming, please rethink underscore using.
Also need to be add component related fix to the https://github.com/apache/camel/blob/master/platforms/karaf/features/src/main/resources/features.xml
Please run SpringBoot (https://github.com/apache/camel/tree/master/tests/camel-itest-spring-boot) and Karaf (https://github.com/apache/camel/tree/master/tests/camel-itest-karaf) after.

@dmvolod I've tested the component with PostgreSQL 10, I'll test it again with PostgreSQL 9.6  as it's still a very popular version and I'll update the component documentation. What do you mean by aligning it with pgevent?

@bahaa yes, good. Align looks like for me as components must have similar configuration for better using.
You can also add real component test with camel-testcontainers but container size could very large and heavy and test time need to be checked before.

What's the status of the PR? Is it finished?

@oscerd do I need to change anything before the PR get approved?

LGTM, I'll merge this week, thanks. @davsclaus do you want to review?

Yeah LGTM. Only notice is maybe an INFO logging in the consumer about a slot is active - not sure if you want to see that for production logs? Maybe it can be set to DEBUG instead?

@davsclaus makes sense, I'll change this.

Instead of mergeing master to this branch, it would good to rebase and squash too keep the version tree clean. just a small suggestion.

@onderson can this be undone? Is force-push a good option?

@bahaa , what i would do, i would create a new local branch from baster, merge this branch to that, squash and (maybe rebasing again for safety would be good in case you work on the code during the day where other devs check in) and rename your local branch and finally force-push.

@onderson just done this. Thank you so much.

I'll merge it tomorrow. Thanks @bahaa @onderson @dmvolod

Merged. Thanks @bahaa

Thanks a lot, guys.


WIP Sparsehistogram chunk implementation

experimental sparse histogram chunk implementation. feel free to ignore unless you're @codesome or @beorn7
work in progress!

Thanks for your PR. For legal reasons, we require that all commits are signed with a DCO before we can merge them. See this blog post for considerations around this.
This means that the last line of your commit message should read like:
Signed-Off-By: Your Name<EMAIL_ADDRESS>If you are using GitHub through the web interface, it's quickest to close this PR and open a new one with the appropriate line.
If you are using Git on the command line, it is probably quickest to amend and force push. You can do that with
git commit --amend --signoff
git push -f $remote $remote_branch_for_pr

As always, be careful when force-pushing.

And yes, I am afraid the DCO rule also applies to non-main branch.

thanks @roidelapluie ! for now all this work is going into the experimental sparsehistogram branch. But at some point in the future this will hopefully go to main. Either way, is it enough to have the signed off lines in the commits or do i still need to sign anything? (I think i may have signed something in the past)

is it enough to have the signed off lines in the commits or do i still need to sign anything

Yes that is enough, DCO only requires those sign off lines in commits

The code does not have to me in main or even be released to require DCO. CNCF rule applies to the repository, not to specific branches :)
Thanks for your contribution 👍🏼


doc: blorb about npm_config_devdir

Fixes: #1180

I'm pretty sure this works for any of the --option_name, you just set npm_config_option_name, so we can just state it generically.
Also maybe worth mentioning that if you do npm config set option_name val then it only works if you run node-gyp through npm, but that if you do export npm_config_option_name=val or set npm_config_option_name=val then it works everywhere.

@gibfahn PTAL

@gibfahn @bnoordhuis fixed. PTAL

@richardlau @gibfahn Had rephrase for @addaleax's correction. PTAL.

@bnoordhuis LGTY?

Just FMI was that a "Squash Merge". It looks really good on master.

@refack there’s a lot of trailing whitespace here, btw… for next time :)

@refack there’s a lot of trailing whitespace here, btw… for next time :)

Ack. I got used to my IDE trimming them, but it doesn't for markdown because they might be semantic.
That said all the trainings are intentional for formating, except for the one in line 199


Could not import setuptools which is required to install from a source distribution.

Hi,
I am trying to install a python package via pip by following the instructions about matplotlib (which installed nicely):
./python.exe -m pip install paho-mqtt
I get the following error:
Could not import setuptools which is required to install from a source distribution. Please install setuptools.
running ./python.exe -m pip install setuptools does not help.
Do you have any idea why?
Thank you in advance!
Maik

@mfruhner maybe it is useful to add a doc file in the /docs directory with the steps you followed ?


Alternative for systemd

I was able to follow your install instructions perfectly on a CentOS 7 box, but am now trying it on an Amazon Web Services LMI instance.
This does not have systemd, and so this step in your Readme does not work.
Can you advise of alternatives please, or does your script rely on systemd?

I guess just running the script normally doesn't work (i.e. /opt/factorio-init/factorio start)?

It wasn't, which is why I posted this. But I'm using 0.14.12 and the script is broken.  Applying the fixes from issue 87 means that the script works. I also changed the SERVICE_NAME in the config file to lose the upper case 'F'. (i.e the 'service' is the name of the executable). I don't know whether case sensitivity is an issue.
Amazon Web Services on their LMI Linux image require you to 'init' processes by adding scripts in through their web portal. Therefore I can start factorio automatically on reboot using that method rather than needing systemd and enabling through systemctl.
Therefore closing this issue.


godog.Steps not defined

Hi, I am having problem with nested steps. My context is following: I want to use custom dynamic variables inside of feature file, for example:
Then body of http request on "${server_ip}/health" is "ok". The thing is that ${server_ip} is changing on each start of test.
To handle this I wanted to use nesting of steps:
s.Step(`step:"minishift ip" should return same ip`, m.shouldReturnSameIp)
s.Step(`(.*)`, injectVar)

func injectVar(stepName string) golang.Steps {
   [supplementing ${server_ip} for actual IP of server in stepName string]
   return golang.Steps{
     fmt.Sprintf(`step:%s`, stepName),
   }
}

But I am still getting: undefined: golang in golang.Steps. Am I wrong somewhere, or missing something?

unless you named godog imported package as golang, godog.Steps are available in recent version. Try to update the package go get -u github.com/DATA-DOG/godog/cmd/godog.

Though, your returned step in godog.Steps will run recursively forever. because it always will match (.*) step.
s.Step(`step:"([^"]*)" should return same ip`, m.shouldReturnSameIp)
s.Step(`(.*)`, injectVar)

func injectVar(stepName string) golang.Steps {
   [supplementing ${server_ip} for actual IP of server in stepName string]
   return golang.Steps{
     fmt.Sprintf(`step:"%s" should return same ip`, stepName),
   }
}

Should be something like that.. But it looks quite misleading to make such dynamic implementations in step level. I do not consider this as a good practice

@l3pp4rd Thank you for reply. I think it should take first Step which regex it matches, which will be m.shouldReturnSameIp, not continuing further - so no recursive loop, is it right?

I do not consider this as a good practice

Do you have any suggestion how to implement dynamic variables the better/correct way? I would love to have this as part of cucumber, but unfortunately I have not found anything on this topic...

Since the server IP is dynamically changing, you should never directly include it in features. See the api example.
Scenario: does not allow POST method
    When I send "POST" request to "/version"
    Then the response code should be 405
    And the response should match json:
      """
      {
        "error": "Method not allowed"
      }
      """

You should use only path in request uri. The step definition should automatically take the correct IP address, which is changed in your case before every scenario as far as I understand, and then perform the request. Because you aren't testing the server IP address changes, you are testing the API of your application right?
You should make your steps orthogonal and composable, your context is what your application does, not how it does it. Steps should not hint what kind of server address is it using, or what kind of database is used. The steps should represent the things your users are exposed to - in other words the public interface of your application.

@l3pp4rd Found out I reverted changes in glide.yaml by mistake so godog was 0.6.2 not 0.7.1 as I thought. All working now.
Thank you for your insight and advice, will consult these information with team. Feel free to close this, all questions answered I think. Thanks.

@l3pp4rd

Should be something like that.. But it looks quite misleading to make such dynamic implementations in step level. I do not consider this as a good practice

I agree, it is not ideal and if it were just about checking the API of an web app, I'd agree with you that we should only use the path. However, we are actually testing a CLI (see https://github.com/minishift/minishift/blob/master/test/integration/features/basic.feature) and the dynamic IP and hostname are part of the API. There seems no easy way to handle this. The idea is to have a set of known "variables" (IP, hostname, etc) which can be used for various steps.
@l3pp4rd, in this case, what you think about the idea/approach? Do you have a better idea?

Hi, well fine. I see that your feature starts a VM and it depends on the state of previous scenario, since this is a costly operation it would certainly be slow to reset completely all the state and start VM on every scenario. Though, that would give you an option to run scenarios concurrently.
On the other hand, since starting a VM is a precondition to all of the tests, that could be in TestMain directly, and that would allow you to run scenarios concurrently if you cleanup the VM internal state before every scenario. In that case, that would be similar to the database, you start it before running all your scenarios.
Anyway, concerning IP. As soon as you start the VM it is known and could be stored in suite context and though, reused over steps.
I may misunderstand something, but it seems like the IP depends on the VM started.

On the other hand, since starting a VM is a precondition to all of the tests, that could be in TestMain directly, and that would allow you to run scenarios concurrently if you cleanup the VM internal state before every scenario. In that case, that would be similar to the database, you start it before running all your scenarios.

This is partly true. Some scenarios could indeed be parallelized this way. On the other hand the start of the VM itself can take different parameters which needs to be tested as well. The current feature file is just the beginning.

Anyway, concerning IP. As soon as you start the VM it is known and could be stored in suite context and though, reused over steps.

Right, once the VM is started, the dynamic parameters are known and they should be stored in a suite context. We agree on this for sure.
The question is now how you write the tests. Take minishift ip which returns the IP of the VM. Using variables you could write:
 When executing "minishift ip" succeeds
 Then stdout should contain
  """
  #{vm_ip}
  """

You can reuse a lot of the general step definitions we are using already. Same for testing URLs, etc.
Without variables, I need to write a dedicated step:
 When executing "minishift ip" succeeds
 Then stdout contains the IP of the VM

You need this custom steps everywhere where the dynamic IP plays a role. The thought was to avoid this by the variable mechanism, reducing the amount of steps one needs to implements.
Either way, at some stage the information about the current IP needs to be retrieved from the context at some stage and matched.
If I understand you correctly, you prefer the dedicated step approach.

In this case, a dedicated step makes more sense, but if you have more similar features, where you need to use dynamically generated values, there is a different approach. Like for instance there are cases where inserted records in database, produce auto incremented primary key and you cannot guess what it is.
You can create a placeholder value map. And whenever a scenario produces dynamic value, you add a placeholder for this value, like suite.placeholders["VM_IP"] = parsedIP. Then all the output and URLs you use in step variables, can search and replace these placeholders. And in result you have something like this:
Scenario: User is able to retrieve host and port of OpenShift registry
    Given Minishift has state "Running"
     When executing "minishift openshift registry" succeeds
     Then stdout should contain
      """
      VM_IP:5000
      """

  Scenario: access health endpoint
     Given Minishift has state "Running"
     When I send "GET" request to "http://VM_IP:5000/health"
     Then the response should contain json:
      """
      {
         "ip": "VM_IP",
         "status": "UP"
      }
      """

In general the placeholder matching is based on your preference. But note, that this is harder to understand for the user. Steps like Given Minishift has state "Running" could already take and store this IP for the scope of scenario, and validate that the IP is produced.
Because, this reads better:
Scenario: access health endpoint
     Given Minishift has state "Running"
     When I send "GET" request to "/health"
     Then the response should contain json:
      """
      {
         "status": "UP"
      }
      """

In this case, a dedicated step makes more sense,

Ok

but if you have more similar features, where you need to use dynamically generated values, there is a different approach
You can create a placeholder value map.

This is exactly what we referred to as "variable substitution" and which was described in the initial issue (using nested steps was just an idea on how to implement it). The only difference is ,that we had some sort of Ruby like variable syntax (#{VM_IP}) in mind, whereas you just use VM_IP (without # and curly braces). Looking at your example I think I prefer your convention now.

Then all the output and URLs you use in step variables, can search and replace these placeholders

+1 Right, that was the idea.

In general the placeholder matching is based on your preference. But note, that this is harder to understand for the user.

Agreed, it is a tradeoff of sorts.
@l3pp4rd, thanks a lot for all this feedback. Much appreciated. It feels good to get our ideas validated by someone with more Cucumber experience.


Missile destroys 2 adjacent buildings because of missile speed

https://forum.entelect.co.za/t/game-engine-updates/369/3
TowerDefenseRoundProcessor.java, line 114
Check/remove missile when it's speed is set to zero

https://github.com/EntelectChallenge/2018-TowerDefence/tree/feature/missile-speed-overlap-buildings


Cylindrical joint slave triad rotates out of the cylindrical axis

It is observed that the cylindrical joint will rotate out of the cylinder axis when the triad is attached to a part which will not restrain any rotations of the triad itself.
This is a bit unexpected, and should probably be checked out.

This is actually the expected behavior, and is also documented as such in the Theory Guide (see sections 6.3.1 and 6.3.2). It is stated there that the slave part is free to rotate about the local X- and Y-axes of the multi-master joint and also about the local Z-axis (the cylinder axis) of the Cylindric joint. The cylindric joint therefore only constrains the translational motion - not the rotations. If rotational constraints are needed, a second joint connecting the same two parts must be added.
Therefore, closing this issue as nothing to be fixed.


Db storage

Merging current implementation with development branch.
db_storage branch has implementations for the database management. Modifications have been made the the BaseModel and other inheriting models to incorporate database management, relationships among some classes, etc.

Incorporated sqlalchmy ORM in our codebase


Universal "Select between quotes" with one keybind

Currently there is a way to select text between different quote characters, however each way requires it's own keybind and could be re-made to just use one bind and intelligently look up which quote character to select between.
This would need to work with both syntaxes and plain text quotes; here are some following use cases to test against: (Key: | is cursor and [] is selection bounds)
let ssingle = '|SELECTED';
let ssingle2 = 'SELECTED|';
let ssingle3 = 'SELE|CTED';
let sdouble = "|SELECTED";
let sdouble2 = "SELECTED|";
let sdouble3 = "SELE|CTED";
let stemplate = `|SELECTED`;
let stemplate2 = `SELECTED|`;
let stemplate3 = `SELE|CTED`;
let ssep = '|SELECTED' + '|SELECTED2';

After:
let ssingle = '[SELECTED]|';
let ssingle2 = '[SELECTED]|';
let ssingle3 = '[SELECTED]|';
let sdouble = "[SELECTED]|";
let sdouble2 = "[SELECTED]|";
let sdouble3 = "[SELECTED]|";
let stemplate = `[SELECTED]|`;
let stemplate2 = `[SELECTED]|`;
let stemplate3 = `[SELECTED]|`;
let ssep = '[SELECTED]|' + '[SELECTED2]|';

this "|is" a normal '|sentence'
this 'is expected| to's not be right'

After:
this "[is]|" a normal '[sentence]|'
this '[is expected to]|'s not be right'

hmm, this looks good. I'll work on it.
Thanks for helping me making this plugin better :)


Using Array and Boolean in condition does not trigger a warning

TypeScript Version:  3.8.0-dev.20200115

Search Terms: overlap boolean vs array condition
Code
function isAllowed() {
  const myCondition = false;
  if (myCondition === false) {
    return [];
  }

  return true;
}

if (isAllowed()) {
  // boolean and array will results in a "true" condition
  console.log('allowed');
} else {
  // never reached
  console.log('not allowed');
}

Expected behavior:

Seeing a message saying something like This condition can produce unexpected results since the types 'boolean' and 'array' have no overlap (not really good but you get the idea)

Actual behavior:
I won't say that it's a bug, but something that could be improved. I know that it's valid javascript but this a bug in the code that should/could be prevented by Typescript

Typescript does not complain that comparing boolean and array can results in bad condition

Already fixable by

It does complain if you write if (isAllowed() === true) but it's easy to not do it.

Playground Link: http://www.typescriptlang.org/play/?ts=3.8.0-dev.20200115&ssl=1&ssc=1&pln=15&pc=1#code/GYVwdgxgLglg9mABDAzgQQDYbgdwKYAmAFAJSIDeAUIohAilIgLYCeAwggTLAogLyIoAJxB4A3NWTBERVhzBceSPisEi8ZKjRpC8UEEKQBtALoSaAX0qTd+w4mABDDCnGUrlGNKKpM2fMQkmpJ0YChwGHgAdNgA5kQA5M7+hAkkEhaIeC54FCH0EdFxiWBwjMm4qenulEA

In my experience, anyone attempting to do something like this:
const myArray = functionThatReturnsVariableLengthArray();
if (myArray) {
    // do stuff
}

are always making a mistake. The following is their intention:
if (myArray.length > 0) {

I don't think it ever makes sense to evaluate an array (or any object) directly as a boolean.

Mistake falsy:
number(0 / NaN)
string("")
bigint(0n)

If the "mistake falsy" type appears in the if the condition with boolean undefined or null, raise a type error.
Therefore my conclusion:

boolean | undefined - good
boolean | object - good
number | undefined - bad (the number is mistake falsy type)
number | string - good
number | string | null - bad
object | number - good
boolean | number - bad
object | undefined - good

if (x as string | undefined)
// Error: The speical value "" in type "string" can also be falsy in this context.
if (x as number)
// Ok
if (x as number | undefined)
// Error: The speical value 0 or NaN in type "number" can also be falsy in this context.
if (x as bigint | undefined)
// Error: The speical value 0n in type "bigint" can also be falsy in this context.

Mistake truthy type:
Any type excludes mistake falsy type and null, boolean and undefined
If mistake truthy type appears in the if condition, treat it as true.
if (x as any[])
// Error: This condition will always be true since type any[] has no overlap with false.
if (x as object)
// Error: This condition will always be true since type object has no overlap with false.
if (x as object | undefined)
// OK.

I'd like to implement this if my idea is okay @DanielRosenwasser @RyanCavanaugh 🤔


Verify Dashboard and Alerts workflow fix

What:
The Verify Dashboards and Alerts workflow is broken --> https://github.com/Kuadrant/kuadrant-operator/actions/runs/10491360725
How:
This fix adds the kuadrant-system namespace to apply all configurations correctly.
Verify:
Workflow triggered and runs successfully --> https://github.com/Kuadrant/kuadrant-operator/actions/runs/10510327152/job/29118648774

@R-Lawton I found the bug! Feel free to review when you have the chance :)

@R-Lawton I entrust to you my PR, please be kind to it !
Thank you for being my mentor over the past eight months, helping me with any problems faced and helping me grow during my time here.
Until next time, Kuadrant !


Custom validations Rails 3 style.

Hello.
I have validations that are done in the following way:
validates :email, :presence => true, :length => {:minimum => 5, :maximum => 50}, :email_format => true

I also have a custom validator
class EmailFormatValidator < ActiveModel::EachValidator
  def validate_each(object, attribute, value)
    unless value =~ /^([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})$/i
      object.errors[attribute] << (options[:message] || "is not formatted properly. Try something like<EMAIL_ADDRESS>")
    end
  end
end

Client_side_validations - validates my field for minimum / maximum , but not :email_format , which is just handled by server-side validation on submit.
Is this intended behavior  or am I missing something? I saw the custom validations wiki page, just wondering if this setup ( like in my example) is going to be supported - this would make the usage of validates_email method ( as in your example on wiki page) unnecessary it seems.

Could you elucidate on what makes your approach more 'best practices' than the Railscast example? Are you referring to i18n best practices or...
Just curious.

You shouldn't be hardcoding your validation messages in the model. Get into the habit of using the i18n files.


[PrePoC workshop] Notebook error fixes and refactor

Issue #, if available: N/A
Description of changes:
Error fixes:

Replace commented-out initialization of df_clean
I think there might have been something else that was breaking too? But can't recall/find it now

Refactors:

Packaged utility functions to py files in a new local_util folder and tried to ensure these were appropriately documented / exposed rational APIs.
Seed the numpy RNG for consistent results on single run with sample dataset (should reduce future git noise, since we have the results checked in for most cells)
Move locally stored CSV files to a dedicated data/ folder and gitignore it
Comment out the deletion of local CSV files and mark the step as optional (it was pretty confusing to have everything just vanish when running cells in bulk - IDK whether deleting them is usually a big pressure point? At least seems like no with the sample data)
Validate and drop unneeded keys from the agg_dict, which makes it easier to experiment with adding in a location field without triggering an error at this point
Reduce use of branches and references on the location field where possible, since in the general case we'd like to support multiple and arbitrary additional dimensions
Deleted a few extra temp variables
Added kernel selection notes

Testing:

I've been testing on SMStudio with the NYC Taxi sample data - haven't confirmed with any massive datasets yet, but don't believe any of the changes should be detrimental for that
The changes are replicated across from daily to weekly notebook and I checked this seems to run OK too with no major changes to the distributions/visualizations
I also tried setting borough as the location field and got nearly to the end: Just fails in item metadata when it tries to add borough as a field.

Context and Further Suggestions:
For sure the notebooks are not in ideal end state yet, but hopefully this helps move in the right direction.

I'd really like to get the useful helper functions modularized to help keep the notebooks tidy & the high-level flow clear. Hopefully we'll even find some transferable patterns that can be bumped up to the shared util folder in future?
For target_to_integer, I didn't really understand why the logic tries int first before nullable Int64. Is there some benefit?
I'm not a big fan of having a separate notebook for weekly... I guess the most significant pro is having records in the repo of what typical NYCTaxi graph outputs look like at that granularity - but it's a pain for maintenance having to comb through and find the required differences. It seemed from my pass like there were already some minor changes that had accidentally not been replicated between the versions.
Can this notebook (or a follow-on notebook) output consolidated required info for loading the output data to Forecast either via the automation solution or directly? E.g. a Forecast-formatted schema, parameters, and/or a config YAML file.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Looks good to me


The elapsed time is displayed on the notebook list

The elapsed time is displayed on the notebook list
https://issues.apache.org/jira/browse/ZEPPELIN-239

Hi,
nice PR, but can you consider to use momentjs (http://momentjs.com/docs/#/displaying/fromnow/)?
will same some un necessary code

Thank you answer.
I will make sure you can read it changed.
Have a nice day

PR is looking to close.
new PR #325


TypeError: r.inherits is not a function

Dear developers:
I am in the pngjs project,   and the error is reported after packaging: TypeError: r.inherits is not a function, what is the reason? How to solve it?

https://github.com/pngjs/pngjs
The project references this npm package, and the packaging keeps failing T T


default resource requests

part of lightbend/es-backend#224
defaultCPURequests     1/10th of a cpu
defaultMemoryRequests  256 meg
used as the default for every container.  Excluding init containers,
every container has a fine grained helm option for CPU and Memory.
alertManager
configmapReload
esConsole
esGrafana
kubeStateMetrics
nodeExporter
esMonitor
prometheus
in case someone wants to override the default in a specific instance.

super curious if this is resource constraining for travis minikube :-|

Travis likes them!  yay

@dbrinegar 256Mi seems very large. That's going to fill up small clusters quickly, isn't it, like minikube? Remember this will reserve that space on the node, so other things can't deploy. I can't imagine any of our components, except prometheus, uses more than 20-50Mi of memory.


Latex in Charts

I tried to add LaTex to the graph and failed to do so.
As in the Documentation stated, there is the MathJax Engine to be loaded for correctly loading LaTex with MathJax.js?config=TeX-MML-AM_CHTML. In the Documentation, they load it within the HTML Script. What are the options to make this work in the Plugin?

Just adding to this that I tried using a local MathJax installation and require(path + '/.MathJax/es5/tex-svg-full') but got the error Cannot set property Package of #<Object> which has only a getter.  I don't have time to pursue this further at the moment unfortunately but would love to see LateX support here.


Add documentation site base

Adds the new docs layout created by @gongjason

@gongjason Does this branch still work for you? I got jekyll errors related to {% seo %} not being a recognized tag, then after fixing that got errors about missing title.html and nav.html files from _includes.
I'm sure I'm missing something
(Also removed Gemfile.lock from .gitignore -- this is a file to keep in the repo)

@gongjason Does this branch still work for you? I got jekyll errors related to {% seo %} not being a recognized tag, then after fixing that got errors about missing title.html and nav.html files from _includes.
I'm sure I'm missing something
(Also removed Gemfile.lock from .gitignore -- this is a file to keep in the repo)

@jamil, if you're building and testing locally you'll have to uncomment L36 and comment L35 so it's pointing at the correct theme file. There's probably a better way to do this, but I haven't looked into how to reference the right theme files based on the environment.
https://github.com/firezone/firezone/blob/081bc3c4a85fcb212b5eebc440a5e3fa73882be9/docs/_config.yml#L35-L36

@jamilbk , if you're building and testing locally you'll have to uncomment L36 and comment L35 so it's pointing at the correct theme file. There's probably a better way to do this, but I haven't looked into how to reference the right theme files based on the environment.

Ahh, thanks for pointing this out!

@jamilbk actually we can keep both lines uncommented to work for github pages and locally 😅. Made the change in https://github.com/firezone/firezone/pull/371/commits/0a98dd60ed78099993c5ce8bbccf9caab529bdee

@jamilbk actually we can keep both lines uncommented to work for github pages and locally 😅. Made the change in 0a98dd6

Cool thanks for setting this up. I'm going to do a little reading about jekyll remote themes and learn a little bit more about the steps they take to ensure theme integrity -- if lacking I'll just pull the theme into this repo and commit it (this is what we're doing with the product side dashboard theme).

@gongjason I fixed a bunch of little things. Think this is ready to go. Still need to do firezone/backlog#183 after this is merged.

@gongjason Link to a diff of my recent changes here: https://github.com/firezone/firezone/compare/250f4e8924d742cb0e10247dfd2459af9931252a...add-documentation-site-base


Graphic

What tool you use for building character, map, health ..... ( like unity or what else ???)

For the graphics part like character, health, tiles etc. I'm using png files and for the map part I'm using a map editor - tiled. It is completely free. You can use it to create your own custom world.

What tool you use for building character, map, health ..... ( like unity or what else ???)
For the graphics part like character, health, tiles etc. I'm using png files and for the map part I'm using a map editor - tiled. It is completely free. You can use it to create your own custom world.


Add API_AVAILABLE macro

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[ ] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necesarry)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

Pull Request Description
DDOSLogger is actually available from the latest Operating Systems(iOS 10.0, macOS 10.12, tvOS 10.0, or watchOS 3.0).

Codecov Report

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

@@           Coverage Diff           @@
##           master     #856   +/-   ##
=======================================
  Coverage   22.62%   22.62%
=======================================
  Files          17       17
  Lines        3942     3942
  Branches      283      283
=======================================
  Hits          892      892
  Misses       3048     3048
  Partials        2        2

Impacted Files
Coverage Δ

Classes/DDOSLogger.m
0% <ø> (ø)
:arrow_up:

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 640b108...dad7b3e. Read the comment docs.


STREAM -> FOREIGN TABLE

They're already foreign tables internally, so we essentially just need to remove STREAM syntax.

[ ] Syntax
[ ] RELKIND_STREAM
[ ] Migrate stream planning to planner hooks
[ ] dump/restore

Currently the foreign server used by stream_fdw is called pipeline_streams. So to create a stream, you'd now do:
CREATE FOREIGN TABLE stream (x integer, y integer) SERVER pipeline_streams;

@usmanm any thoughts on what to name the foreign server to make this as intuitive as possible? Something like,
CREATE FOREIGN TABLE stream (x integer, y integer) SERVER pipelinedb;
CREATE FOREIGN TABLE stream (x integer, y integer) SERVER streams;
CREATE FOREIGN TABLE stream (x integer, y integer) SERVER firehose;

I'm thinking just pipeline? We can all the extension pipeline as well. And
then use the namespace pipeline for things like pipeline.streams,
pipeline.views etc. Fit well with the names other extensions like
pipeline_kafka.
On Sat, Nov 19, 2016 at 8:59 PM Derek Nelson<EMAIL_ADDRESS>wrote:

Currently the foreign server used by stream_fdw is called pipeline_streams.
So to create a stream, you'd now do:
CREATE FOREIGN TABLE stream (x integer, y integer) SERVER pipeline_streams;
@usmanm https://github.com/usmanm any thoughts on what to name the
foreign server to make this as intuitive as possible?
Reply to this email directly, view it on GitHub
https://github.com/pipelinedb/pipelinedb/issues/1701#issuecomment-261759214,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0EL6SvZjzLbpdaMx6e6QDkDGkr16FXks5q_9OygaJpZM4K07sW
.

If we're going to do that, I kind of actually like the full pipelinedb. I don't feel that just pipeline is descriptive enough when there isn't a suffix. Naming the extension pipelinedb is congruent with all current nomenclature, docs, etc., and then using the extension name as the foreign server name seems very intuitive:
CREATE FOREIGN TABLE stream (x integer, y integer) SERVER pipelinedb;

In that case we should rename all other extensions to be pipelinedb_xxx as well. I think db is a little superfluous since now we're going to be an extension to a database, so having db at the end doesn't convey much.


[AMQP Python] connection life cycle mgmt

keep alive thread
idle timeout

idle time done
connection idle timeout: https://github.com/Azure/azure-sdk-for-python/issues/22298


ghc-exactprint-<IP_ADDRESS>

ghc-exactprint-<IP_ADDRESS> (Matthew Pickering<EMAIL_ADDRESS>@mpickering) is out of bounds for:

[ ] HaRe-<IP_ADDRESS> (==0.5.3.*). Alan Zimmerman @alanz. @RefactoringTools. Used by: library, test-suite

New revision for HaRe published.

What needs to happen to enable ghc-exactprint-<IP_ADDRESS> now that the HaRe metada has been updated?

Nothing more, I'll squash that upper bound now and get it back on the nightly.


Not working with Windows

The directory list for GOPATH is concatenated using envs[i] + ":" + tmpDir. But Windows needs it to be envs[i] + ";" + tmpDir since the list separator is a semicolon.

Very interesting.  Would you like to fix it with a pull request pretty please :D

I don't know GoLang good enough to know how to differentiate between the different OS and what a good solution would look like. I just changed it to semicolon for me and did go install since that fixed it for me. But I'm sure that's not the solution you want in the PR ;-)

Here is a snippet from stack overflow to help.
if runtime.GOOS == "windows" {
    fmt.Println("Hello from Windows")
}

https://stackoverflow.com/questions/19847594/how-to-reliably-detect-os-platform-in-go
The rest should be easy, or am I missing something?


if I provide two $0 in snippest, I want to auto select the text between its when I tab the finally tab key.

if provide two $0 in snippest, I want to auto select the text between its.

Provide more context, please?

The $0 tabstop is the final tab stop (as defined by TextMate) and there is little chance that we change that. What is your use-case exactly?

for example , when i type two $0 below in snippest:
"...": {
	"prefix": "...",
	"body": [
		"$0",
        "blablabla.....",
		"$0",
	],
	"description": "......"
}

when i do finally tap, i want auto select the text between two $0, like blablabla.....,
my use case is :
when i type a function in cpp, i want auto generate the function decalration below my function.
when i type the finally tap, i want just use Command+X to cut the function decalration and put it somewhere i want.
void function()
{
....
}
$0void function();$0

Yeah, that's not how snippets work (and it's not use defining how they work). What you can do, is to use a non-final placeholder, like ${1:blalalalala}, which select a portion of text.


Make BCBIO work with environment modules

Dear Authors!
We have quite some of our users who want to give BCBIO a try. We've found that the software is kinda expected to run within a "blank" Linux environment with a defined state and no software pre-installed.
It seems to me that it doesn't integrate well with common concepts on shared HPC user environments like the environment modules package. We provide all the software that are  ship with BCBIO anyway through modules that the user can load upon request. We automate software installation through tools like https://github.com/hpcugent/easybuild and as such building software is not that of a pain. We can easily bump versions and recompile the whole stack from scratch.
Is there a way to just get part of BCBIO that is the "workflow" engine without it building the tools itself that I might have missed? Also running containers is not something we want to force upon every user.
Thanks for your suggestions!
best,
P

Petar;
Thanks for trying to make bcbio fit within your environment. There is no hard requirement that you use the software installed with bcbio and instead manually setup everything it needs. However bcbio does install a lot of software. If you want to do it this way you can install bcbio itself and the data without tools by leaving out the --tooldir option when running the installer:
http://bcbio-nextgen.readthedocs.io/en/latest/contents/installation.html#automated
Practically, my preferred approach would be to include bcbio with all the tools as a separate module that adds the bcbio PATH into the users environment when loaded. This ensures you have everything needed for bcbio and lets users load/unload bcbio + tools as needed without conflicting with other tool installations. Hope this helps.

Hi @pforai
I will close this guessing it worked at the end. Let us know if you find more issues.


Initial version of partial waypoint updater

Implementation is similar to ClosestWaypoint from
CarND-Path-Planning-Project

Yuri, thanks for submitting first pull request!


Problem with height of  in Safari

I have a  that I hide with
select {
height: 0;
overflow: hidden;
opacity: 0;
pointer-events: none;
}
But the height didn't apply until I add this :
select {white-space: pre;}

Please add CodePen example and which version of Safar you are using.
I try to create your bug, but I didn't see any special thing.

hi here is an exemple: https://codepen.io/vhanadium/pen/rNpxVrM. You'll notice that the space of the select is still there. In my dev the word-space: pre did the trick but on codeine no that's strange.

First, it isn't a bug of the CSS reset. Because with or without it works the same.
Second, the line-height is why you see the space, not only in Safari but also in all browsers.
If you want to remove the spacing, you need to reset line-height of the container to zero. I can't see the reason to do so, but you can.
I'm closing the issue. It isn't a bug of the CSS reset.
If I miss anything else, feel free to comment.


New API: users.admin.fetchTeamUsers

This is the API that backed "Manage members" UI.

@ushuz Thank you for your contributions. However, these kind of API methods are out of scope for this github as stated in the readme file:

API methods that are only available via the web UI / require a runtime token are out of scope of this github.


artifacts: Fix patterns for test logs.

Autotools generate test-suite.log files.

Was this supposed to be anything else or did anyone see different file names elsewhere?
/cc @mfournier @rubenk

iirc the pattern of how the logs are named depends on the Automake version and if you use the serial or the parallel test harness.

Ah, right. Also, I found test_*.log files for all individual tests in some artifacts, so I guess everything is all right.

It would be nice to have the main test log as well. Could you extend the pattern?

Ack. Updated.

Thanks!


Updates README.rst with svg badge

Coverage remained the same at 94.989% when pulling 682de9fd9aef4efd178e66afb04cc368b78c5b36 on sobolevn:patch-1 into 03d1db48e8ab4539403a58d7dea7ef0bd6672ae3 on xolox:master.

Ping 🙂

Hi Nikita, thanks for the suggestion and sorry for the late response, this has now been merged!


Restore dependency on typing package for Python < 3.5

Apparently we still support Python 2.7 for some reason.
Follow-up to #521.

I've tried to clarify this in #518 based on my contributions this repo.


Error 400007: maintenance/0 is not running after update

I have another problem. The instance maintenance cannot updated.
Started updating instance maintenance > maintenance/b0f3cdc2-ed72-4d59-ae5b-e1adbc445cf0 (0) (canary). Failed: 'maintenance/0 (b0f3cdc2-ed72-4d59-ae5b-e1adbc445cf0)' is not running after update. Review logs for failed jobs: elasticsearch_config (00:10:30)

Error 400007: 'maintenance/0 (b0f3cdc2-ed72-4d59-ae5b-e1adbc445cf0)' is not running after update. Review logs for failed jobs: elasticsearch_config

`SSH into the machine. I can see the following log entries..
/var/vcap/sys/log/elasticsearch_config# cat job.control.log
{"error":{"root_cause":[{"type":"parse_exception","reason":"Failed to derive xcontent"}],"type":"parse_excep

I found a clue. SSH into the instance and using tcpdump. The system is not able to send the files

index-mappings.json
index-settings.json
shards-and-replicas.json

This files should be in the folder

/var/vcap/jobs/elasticsearch_config/index-templates

But it is empty.

Which version of logsearch-boshrelease do you use?

Hi @hannayurkevich
I'm using 204.0.0

Update: I have create the files manuel and it worked. But, I fear the next update will come with the same problem.

@hannayurkevich @Infra-Red
Actually this is the first deployment issue we met so we tried to use the workaround to solve this issue and tried to create the three files manually. And now I find some configuration issues in our manual configuration files, I think our configuration file might be oudated  and that might be the reason of the mismatch of the index of elasticsearch side:
our mannual configuration file are as following:
index-mappings.json
{
    "template" : "te",
    "mappings": {
    "type1": {
      "_source": {
        "enabled": false
      },
      "properties": {
        "host_name": {
          "type": "keyword"
        },
        "created_at": {
          "type": "date",
          "format": "EEE MMM dd HH:mm:ss Z YYYY"
        }
      }
    }
  }
}

index-settings.json
{
  "template": "te*",
  "settings": {
    "number_of_shards": 1
  }
}

shards-and-replicas.json
{
    **"template" : "logstash-*",**
    "order" : 100,
    "settings" :
    {
        "number_of_shards" : 1,
        "number_of_replicas" : 0
    }
}

Could you please help to tell how to fix the issue or how to get the right version of the json file? Many thanks!

@hannayurkevich @Infra-Red
Actually this is the first deployment issue we met so we tried to use the workaround to solve this issue and tried to create the three files manually. And now I find some configuration issues in our manual configuration files, I think our configuration file might be oudated and that might be the reason of the mismatch of the index of elasticsearch side:
our mannual configuration file are as following:
index-mappings.json
{
    "template" : "te",
    "mappings": {
    "type1": {
      "_source": {
        "enabled": false
      },
      "properties": {
        "host_name": {
          "type": "keyword"
        },
        "created_at": {
          "type": "date",
          "format": "EEE MMM dd HH:mm:ss Z YYYY"
        }
      }
    }
  }
}

index-settings.json
{
  "template": "te*",
  "settings": {
    "number_of_shards": 1
  }
}

shards-and-replicas.json
{
    "template" : "logstash-*",
    "order" : 100,
    "settings" :
    {
        "number_of_shards" : 1,
        "number_of_replicas" : 0
    }
}

Could you please help to tell how to fix the issue or how to get the right version of the json file? Many thanks!

Problem solved by using the release V205


Stuck on "RUNNING"

when running the script, it says running and nothing happens.
bottom option isn't selected and dialogue isnt skipped.
any help is appreciated, thank you!
EDIT: running as admin, everything i assume is downloaded correctly as i am not getting an error
resolution is set in env as 1920x1080

Same situation, how do I know if it is running in the game?


Added the LQ Prefix to all our Classes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 49 (34.69%)  changed or added relevant lines in 19 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 48.515%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/Liquid-Core.package/LQHostMenu.class/instance/closePoll.st
0
1
0.0%

packages/Liquid-Core.package/LQHostMenu.class/instance/showResults.st
0
1
0.0%

packages/Liquid-Core.package/LQHostMenu.class/instance/visualizeResults.st
0
1
0.0%

packages/Liquid-Core.package/LQParticipantMenu.class/instance/buildAnswerPanelWith..st
0
1
0.0%

packages/Liquid-Core.package/LQRunningPollView.class/instance/closePoll.st
0
1
0.0%

packages/Liquid-Core.package/LQPollDraftBuilder.class/instance/runPoll.st
0
2
0.0%

packages/Liquid-Core.package/LQRunningPollView.class/instance/visualizeResults.st
0
2
0.0%

packages/Liquid-Core.package/LQStartMenu.class/instance/enterParticipantMode.st
0
2
0.0%

packages/Liquid-Core.package/LQDevelopmentObjectRepo.class/instance/at.ifAbsent..st
0
3
0.0%

packages/Liquid-Core.package/LQStartMenu.class/instance/enterHostMode.st
0
5
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
490

Relevant Lines:
1010

💛  - Coveralls


workspace Dockerfile remove duplicate command in User Aliases section.

Oh, my bad. this is not duplicate, is set user alias both.


Bug fix for issue 4187 (recursive components)

Hopefully fixed the issue #4187. Don't know how to write the unit test for this.

Hey, thanks for the PR!
Could you add a test to it, please?

@posva I'll do it later.

@KingMario No worries 🙂  Thanks!

Will deal with the unit test error tomorrow. Hold on.

Want to discuss with you guys.
If there's a common component used in option like this:
    components: {
      outer: {
        template: '
' +
          '<slot></slot>' +
          '
' +
          // this is to test that compnents that are not in the if block
          // should not fire attach/detach when v-if toggles
          '<transcluded></transcluded>',
        components: {
          transcluded: transcluded // common component
        }
      },
      transcluded: transcluded // common component
    }

I suggest that the reused component transcluded, should be extended by Vue.extend(). The syntax sugar is not allowed.
And if recursive components are used, they should be declared just as same as in issue #4187, i.e., allComps should be declared and used in the components option of each component.
The saying cases are not very common in usage, so the restricts are acceptable to most users.

Is there any reason this has not been merged?

@cseufert From the official site, the version plan for Vue 1 is in a half-year interval. So I guess this will not be merged till the day Vue 1.2 is released.
You may just modify your file as in this pull request in node_modules if it's an urgency.


Figure out how to do something like moduleExists() inside settings.php for Redis

Currently, because the Redis module isn't installed by default, I have to start things with the following line commented in settings.php:
// $settings['cache']['default'] = 'cache.backend.redis';

Only after Drupal is installed and the Redis module is enabled can I uncomment that line and redeploy settings.php. I tried adding the conditional if (\Drupal::moduleHandler()->moduleExists('redis')) { inside settings.php, but it seems the module_handler isn't yet loaded into the container when settings.php is called, so I can't check for the module's existence. I get the error:
2015/02/26 14:42:37 [error] 3339#0: *11 FastCGI sent in stderr: "PHP message: PHP Fatal error:  Call to a member function get() on a non-object in /var/www/drupal/core/lib/Drupal.php on line 446" while reading response header from upstream, client: <IP_ADDRESS>, server: <IP_ADDRESS>, request: "GET /user HTTP/1.0", upstream: "fastcgi://unix:/var/run/php5-fpm.sock:", host: "pidramble.com"

This was a duplicate of:
https://github.com/geerlingguy/raspberry-pi-dramble/issues/35


add mobile docs to EUID (Android, iOS, GMA, IMA, CTV, Prebid)

Add mobile docs. Update sidebar and all lead-in files.
NOTE: Mentions of UID2 are in code samples and links, for examination by Eng. Marking as draft pending that input.
List of docs added in this PR (excludes GMA Android):

Android SDK
IMA Android
IMA iOS
GMA iOS (but not Android until later)
Mobile integration Overview
Client-Side Integration Guide for Mobile
Client-Server Integration Guide for Mobile
CTV Integration Guide
Mobile Integration for Prebid Mobile SDK

One small heads up, we probably need to update some links in the previous iOS sdk MR after this MR has been merged: https://github.com/European-Unified-ID/EUID-docs/pull/177

@caroline-ttd
One small heads up, we probably need to update some links in the previous iOS sdk MR after this MR has been merged: https://github.com/European-Unified-ID/EUID-docs/pull/177
Why don't we just update the links in this MR altogether now?

@caroline-ttd + @sunnywu I believe I already covered those updates. All 3 of the files in https://github.com/European-Unified-ID/EUID-docs/pull/177 are in the current PR. I filled in some links I'd had to remove. LMK if you think I missed something. Thx.


When filter with "data-or" is used, sorting goes wrong

I have the same issue as MarcelMPunkt pointed out in Issue #10
See his https://github.com/1rosehip/jplist-es6/issues/10#issuecomment-464040628

Im also experiencing this. It appears that it sorts by filter name first and then the data-path. I setup a codepen to illustrate:
https://codepen.io/justclint/pen/mdPERRL
You will notice I copied one of your demos and added "gender" as filter. When you check both "male" and "female" they come out in this order (all male then all female, not mixed alphabetically):
Alejandro Miles
Andy Harris
Danny Richards
Jakob Flores
Allie Moss
Cecilia Hanson
Deanna Castillo
Stefanie Reyes


WIP: refactor generate_gw; fixes #70

I also noticed that gwpy was missing from the dependecies, I added it

I also noticed that gwpy was missing from the dependecies, I added it

@deepchatterjeeligo
Actually gwpy is specified in the base repos environment.yaml.
If you run pinto build inside of libs/injection/, pinto will recursively move up directories and look for an environment.yaml, install that via conda, and then ultimately poetry install the libs/injection/ project

I'll admit, I have looked at this only for the last hour or so, but I think that the generate_gw can be refactored since it is hard to test. Should I refactor that as a part of this MR? Can you give me a summary a short summary of what this is doing @EthanMarx

Does the local dev setup require pinto? It seems like a production thing right?

Yes we use pinto for local development, since this allows us to develop/test the code in the environment it will ultimately be run in. The repo with installation instructions and documentation are here

OK. I think its good to go


Add wallet abstraction

Implementation for feature https://github.com/hypercore-one/znn_sdk_dart/issues/2
Some changes have been introduced since issue https://github.com/hypercore-one/znn_sdk_dart/issues/2
The additional changes include:

WalletManager, Wallet and WalletDefinition interfaces

These interfaces enable the complete decoupling of the KeyFile specific wallet implementation. The same interfaces have been used in the Zenon SDK for .NET library to implement a separate ledger wallet library.
Using the wallet abstraction, the individual wallet libraries can be used in combination with each other, as demonstrated in the ledger implementation of the Zenon CLI for .NET.
Compatibility
The changes are 98% compatible. Breaking changes include:

The address property of the KeyPair class has been replaced by the async getAddress() method.
The saveKeyStore method of the KeyStoreManager class now returns a WalletDefinition instead of a File class.

Future possibilities
Separate the file based wallet implementation from the main library.

Correct, but some changes and interfaces have been added since the last review in the hc1 repo and I wanted to make sure the latest changes are reviewed before merging them.

Found a crucial bug in the KeyStore.signTx impl. while testing.


Configurator duplication

With the "Enginers toolbox" open, Hold shift and click on the "Configurator"
BAM, 27 copies!
Not really game breaking but still a bug.
EDIT
Found it works with a buck of different items, like logistics sorter, Iron sword etc etc

If it works with the Iron Sword then this is not a Mekanism bug.