Fix broken headings in Markdown files

GitHub changed the way Markdown headings are parsed, so this change fixes it.
See bryant1410/readmesfix for more information.
Tackles bryant1410/readmesfix#1

@bryant1410 This seems to be a false positive.

Yup. Sorry, it's fixed now.

Worthwhile effort on the whole, though :)


Script for URL redirects

@majkinetor @ferventcoder @AdmiringWorm @gep13 @pascalberger
I have been seeing a lot of the websites that have software for download using a redirect. Although they don't need to be as complex as the now [Deprecated]  'Chocolatey FossHub Extension' package. This seems to be a needed item. The  @The-Running-Dev whipped up a useful script that is part of PR #664 that handles this redirect nicely. It can also be used in another or other package(s) that have redirects.
When this was mentioned on gitter @AdmiringWorm wanted to have it discussed.
This is the opened issue for that discussion.

The script is way simplistic. To get included into core it would have to support multuple redirections and proxy and headers.

@majkinetor Feel free to make it more complicated :-)

No I wont, but I have a great quote:

Everything Should Be Made as Simple as Possible, But Not Simpler

Closing due to no feedback.

I've added a few days ago a simple redirect script, it uses another way to read the redirected url than the function created by @The-Running-Dev.
Instead of disabling AllowAutoRedirect and reading the response status code, it allows it and  read the ResponseUri instead.
https://github.com/chocolatey/chocolatey-coreteampackages/blob/master/scripts/Get-RedirectedUrl.ps1
It's pretty simplistic right now, but for now it works. When it's needed it can be extended of course.

Wont do without proxy.

@majkinetor if it was part of the install script I would agree, but considering it's just a script for using with updating the package itself it doesn't really matter.
In which case, all of the calls (for every package) to iwr need to be changed as well to add support for proxies.

Same would be with the Get-RemoteFiles which uses the WebClient .NET class as well.

I thought it would be used within installer.
BTW, iwr handles proxy automatically from the system.

I thought it would be used within installer.

yeah no, only for the update script.

BTW, iwr handles proxy automatically from the system.

I see, didn't know that. I thought it didn't


Allowed for securityToken-free use of salesforce

I didn't test, but code LGTM.

I've created METAMODEL-1140 to reflect this, and will merge in the branch. Thank you for the contribution.


Resolution +/- sustain isn't quite right

Increasing and decreasing the resolution causes the notes to apparently sustain longer.  The sound should not change with lossless increase and decrease actions.

Math wasn't quite right for a stable sound over resolution changes: @c1ec6e2a6e2bc01d53228bb2ecf87a5b57a684fa


Can the height of the toolbar be customized

I hope the height of the toolbar or toolbarItem can be set. How can I do this

There is no way to change the height. This is a system limitation and is in place so people cannot create inconsistent settings UIs.


fix: Fix automatic sort of Composite items in fixed containers

Closes #5
How to test?
Does this PR introduce breaking changes?
No

Thanks for working on this @Genichiro00! I'll take a look at it this weekend (hopefully).

I'm also hitting this issue on a regular basis, it can be quite jarring on a long page – thanks for taking on the fix!

Good luck, good night


7c5tNmfdfhszNpSy8YBbSmsnKCnSyih9sCRnL95Spgdv

Token Address: 7c5tNmfdfhszNpSy8YBbSmsnKCnSyih9sCRnL95Spgdv
Token Name: LouDoge
Token Symbol: loudoge
Logo: https://github.com/loudoge/logo/blob/main/lousol1.png
Link to the official homepage of token: https://loudoge.com/
Coingecko ID if available): N/A
I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Please add logo and token


[Question]: why cant we use Nth(0)

Your question
This might have an obvious solution but its a bit of a pain i that i have to workaround.
Scenario:
i have a list of items that have buttons on the page
i want to select them by nth passing in an int from my specflow step i.e "1st" "2nd" etc where i can -1 on the int.
if i pick locator i get 2 or 3 separate locators that means i cant use just a single int to pick from the list
GetByRole(AriaRole.Button, new() { Name = "Remove", Exact = true }).First
GetByRole(AriaRole.Button, new() { Name = "Remove", Exact = true }).Nth(1)
GetByRole(AriaRole.Button, new() { Name = "Remove", Exact = true }).Last

Nth(1) being the second item on the list
i would want to use just Nth(0) for the first but i never finds the item instead i have to use a selector like this to use Nth(0)
 .Locator("button:has-text(\"Remove\") >> nth=0")

When you run with the above locator it resolve to First Anyway

OK, i tried it and it works this looks like its adjust a bug in the codegen that it doesn't highlight items with Nth(0).


Update to latest as-pect, fix compiler errors

Alright! Time to get working on this.
This currently breaks because of a compiler error. Inheriting from Uint8Array causes a compiler error, when accessing the bytes via the index signature:
ERROR TS2329: Index signature is missing in type '~lib/buffer/index/Buffer'.

   for (let i = 0; i < values.length; i++) result[i] = values[i];

Please advise 👍
Also, there seems to be an issue with Reflect.equals() which may potentially be a bug in my testing software, or is possibly related to the fact that type information is not passed from Buffer index signature. I'm confident it's the latter.

Currently, I borrowed the index signatures from Uint8Array to force the tests to pass and check out stdout.
Everything seems to work exactly as intended.

Thanks to 0.9.3/4 I can remove the index signatures. This whole pull request is basically to bring everything to work with the new ArrayBufferView and latest as-pect

Looks like the wat files weren't being output, so I used the same pattern I use in as-pect to write the files out without blocking tests.


simplify fetchPypi, remove need for overrides

We already have the name of the file that we need, so let's fetch it
directly instead of breaking it apart first, passing it to the Nixpkgs
fetchPypi, which would then put it together again.

There was a minor issue with how the URLs were constructed which you likely didn't notice because it was a fixed-output derivation.


Provide an option to by pass generating a settings.xml file

When using self-hosted runners we provision compute with a Maven settings.xml and currently this action will overwrite our correct settings.xml with one that's inoperable. It would be nice to signal to the action to not generate a settings.xml file.
Another option might be to allow a template .github/workflow/maven/settings.xml that can be provisioned on the runner. In the case of running with GitHub actions proper the template can use ${{ secrets.references }} and in self-host just normal envars which Maven will interpolate into the settings file. Either way it's secure.

I'd say that ideally, the default should be not to generate settings.xml.

If that is possible, sure. I assume there is an existing use case internally that conflicts with this.
I think the cases where it can safely generate a setting.xml are:

if there is a settings template in a known location, it can be rendered on to GHA node

if there are secrets with standard names to be used in a generated settings.xml, render a settings.xml to the GHA node

Hello everyone!
This request was implemented in v2-preview release via #136
You can test it right now:
- name: Setup Java
  uses: actions/setup-java@v2-preview
  with:
    distribution: 'adopt'
    java-version: '11'
    server-id: maven
    server-username: MAVEN_USERNAME
    server-password: MAVEN_CENTRAL_TOKEN
    gpg-private-key: ${{ secrets.MAVEN_GPG_PRIVATE_KEY }}
    gpg-passphrase: MAVEN_GPG_PASSPHRASE
    overwrite-settings: false

overwrite-settings: false does the magic.
We are going to release v2 stable by the end of next week but we would be very appreciate for any testing of v2-preview and confirmations that it works as expected.

I am still at loss as to why the action deals with maven at all. Why should installing java involve anything with Maven?

@maxim-lobanov could you please share again the reasoning behind the Maven specific functionality?

I don't have much historical context but @konradpabjan has shared some details in https://github.com/actions/setup-java/pull/132#issuecomment-796873588

The main reason why setup-java has Maven functionality in it is so that it's easier to publish to GitHub package registry. When the initial maven publishing feature was added there was a pretty lengthy debate (and I was very much against adding the functionality to setup-java) but we ultimately decided to do it so that publishing would "just work" without any extra configurations or any other actions. I agree that it somewhat distorts the single purpose intent that setup-java initially was meant to provide (and that most users think it provides) but we ultimately decided that the pros outweight the cons.
One solution that was floating around was to create a separate setup-maven or setup-gradle package but we don't have the bandwidth to support yet another action and the maintenance burden would also be very high so we just extended setup-java
Before we started work on the v2-preview, we had some discussions regarding what to do with the existing Maven publishing functionality and the general consensus was to just leave it as is. Don't prioritize any new features for it, don't break what we currently have, but keep the current functionality as is because we still want publishing to GitHub packages to "just work".

I will also add that support was expanded not just for Github package registry (you can publish to any Maven Central Repository). See this guide: https://docs.github.com/en/actions/guides/publishing-java-packages-with-gradle
setup-java does all the magic and the only thing the user has to do is gradle publish
      - name: Publish package
        run: gradle publish
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Without the Maven logic in setup-java there would be a lot more configuration needed for users. Easier publishing at the cost of Maven specific logic being added to setup-java was a tradeoff that we just decided to make.

Without the Maven logic in setup-java there would be a lot more configuration needed for users. Easier publishing at the cost of Maven specific logic being added to setup-java was a tradeoff that we just decided to make.

My concern with this thinking is that it opens the door to add a lot of other features.
More importantly, how is this impacted by a build that uses a matrix of multiple Java versions? Will settings.xml be updated every time for each version? What if I want to publish 2+ times, to different locations (e.g. private repo, public repo, staging repo)?

This concern is fair enough and we definitely need to be very selective for new features in future.
Unfortunately, we can't deprecate this functionality or switch it to be disabled by default without significant impact for customers. So we have added overwrite-settings: false that provides way to disable this functionality.


Exceptions silently fail to send if the stacktrace is too large

We are using raven-js in a React Native project and when an exception is thrown it isn't appearing in the Sentry UI. After much trial and error I've managed to track down the issue, it appears that if the 'exception.values.stacktrace.frames' array is too large then the exception will fail to send.
I think the '_trimPacket: function(data)' method needs to be expanded to trim enough excess data in order for the exception to be sent (reporting an exception takes priority over having all the required information so worst case it should trim the entire stacktrace).
This of course is all assuming that it's failing to send due to the size of the data for some reason.

@mpretty-homepass – what version of Raven.js are you using? We changed the React Native HTTP transport to POST requests in 2.2.0.

fwiw, it's still able to relatively easily overflow the max size for a POST (100k on getsentry.com) as well since we can't do compression on the client request.

@mattrobenolt – sure, but this issue was created before 2.2.0 was released. I'm pretty confident that is the cause.

@benvinegar hey, sorry for the massive delay in my response. I was using 2.1.1, have updated to the latest code (2.3.0) and it looks like the problem I was having is fixed (I don't need to trim the stack trace anymore).

Sweet!


add cloudflared arm64

Description

add cloudflared arm64

All Submissions:

[x] Have you ensured there aren't other open Pull Requests for the same update or change?
[x] Have you created a new branch for your changes and PR from that branch and not from your master branch?

build was fine for me:
https://github.com/gOOvER/own-pterodactyl-images/actions/runs/6313814121/job/17142726838

Is there a reason there is no $ before ARCHITECTURE and just {}

Is there a reason there is no $ before ARCHITECTURE and just {}

typo :) but fixed


Save button not working

Never mind, it saves in the same directory from where you imported the sprites.

?


Use ProvideTextStyle in the title of KaigiTopAppBar

Issue

close #372

Overview (Required)
-　Combine redundant style settings into one using ProvideTextStyle.
Links

Screenshot

Before
After

Looks great! Thanks!


mkl-paradiso support

Describe the bug
I'm currently hacking together a nix-expression for a legacy fortran tool. This tool links against mkl, and specifically mkl-paradiso. AFAIK neither mkl-paradiso nor anything fortran related is in the mkl package. What is the way to go here?
Notify maintainers

@bhipple
Metadata

system: "x86_64-linux"
host os: Linux 5.6.14, NixOS, 20.03.1950.48723f48ab9 (Markhor)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.4
channels(root): "nixos-20.03.1950.48723f48ab9, nixos-unstable-20.09pre226148.0f5ce2fac0c"
channels(wucke13): "nixos-unstable-20.09pre227577.135073a87b7"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
  - mkl
# a list of nixos modules affected by the problem
module:

I'm stupid.


Remove "Quantize Multiplier"

It is now possible to implement quantize multiplier to work on any noteskin, just add it in the mods menu of any theme.

https://github.com/hanubeki/noteskin-hanubeki/commit/17aa3ae29dc423ada24d96dd781e2dc35219444a


Update _config.yml

@armsp try merging this see if it work

nevermind i think you fixed it!

Oh yeah, had committed just a few seconds before i saw your PR


plagiarism script should support blacklisting files

For OO we provide project templates to students where they just have to fill in some of the classes. This causes a lot of false positives in the plagiarism detection script for files that students don't have to touch.
It would be nice to have a list of files to be ignored by the plagiarism script.

Take a look in the fp branch; we had the same problem in the FP course some years ago; there both plaggen.sh and dupes.sh contains a loop to filter out signatures of the templates given to students (that students are not supposed to alter drastically).
This functionality was removed when these scripts were ported over to the master branch, it should be easy to put back in.

Thanks! I'll have a look.


zfc.exe seems to generate serializers for the entire project

I'm prototyping the use of ZeroFormatter as a serializer and it seems like when I call the zfc.exe to generate the serializers for the IL2CPP... it actually generates serializers for ALL of the types in the project... not just the ones marked [ZeroFormattable].
The generated .cs file is almost 2MB in size. Seems a bit extensive.
Is there a way to limit the autogenerated .cs to create the serializers for only the types it needs?

It seems that it is limited to Enums only. It generates support for all the Enums in the whole project instead of just the ones referenced by [ZeroFormattable] types.

On the project I'm testing this on results in some errors like:
error CS0060: Inconsistent accessibility: base class .... is less accessible than class ..
error CS0122: 'PP.XMLReader.RecordScope.ScopeType' is inaccesible due to it's protection level
all of these errors are on the types that weren't registered to be serialized.
Is there some option/feature that I've missed?


Allow user defined post-provision-tasks to use tags while drupal vm does not

At the moment Drupal VM does not use ansible tags out of the box, but I wanted to use it for my own custom post-provision tasks. By default all the pre/post-provision tasks are excluded as the include statements are without tags.
This PR fixes the issue by delegating the decision of tags to the post-provision scripts. Rather than setting the tag to user-provision or something similar, I think they should be set it to always so that users can take full advantage of tags. http://docs.ansible.com/ansible/playbooks_tags.html#special-tags
DRUPALVM_ANSIBLE_ARGS='--tags=foobar' vagrant provision

Currently I'm only tagging the config includes and the post/pre-provision tasks. But maybe we should also tag Include OS-specific variables. and such?

Nevermind, that didn't work.

@oxyc - What's not working? Did you add a tag to a custom task, then set the arg in the vagrant command, and it didn't use just that tag?

@oxyc - Also note that tags should always be in an array (I'm 99% sure)—so, like tags: ['always'].

If I tag the include task with always, then all the tasks within that file inherit the tag. Thus I can't limit the tasks that actually run :/ I didn't know that's how tag inheritance worked http://docs.ansible.com/ansible/playbooks_tags.html#tag-inheritance
I was hoping I could somehow read the included file, and only execute the tasks within that file that had a particular tag. Seems that's not possible as --tags=foo,bar means the union of foo and bar, rather than the intersection.

@geerlingguy if you have any ideas on how to allow this, I'm very interested :)

Hmm...

That always behavior sounds more like a bug in Ansible.

I'll keep this open as a reminder to take a look at the ansible issue queue.

 We only need to tag the tasks that allow config.yml to be loaded. After that ansible does check the tags for each task within the included files.
Oh, and tags does not need to be a list. If you prefer a list, I can fix it.

Oh, and tags does not need to be a list. If you prefer a list, I can fix it.

Ah yes... I just always do a list in case I need to modify it. It just feels more correct to me... I'm one of those insane people that dictates PHP functions should only return one and only one type (e.g. empty array, or array... not null, array, FALSE, etc.). Can you change it?

There you go!


Backport fixes to version 1

Two important changes to back port:

#18
#16

Also autobahn in CI (#11)

Cherry picked #16 here: https://github.com/IBM-Swift/Kitura-WebSocket-NIO/pull/29

Created #30 and #31


Fix #11284 - Resolve tokens found in settings referenced by launch.json

This fixes the issue (https://github.com/Microsoft/vscode/issues/11284):

Launch.json can reference settings defined in settings.json via the syntax ${config.python.pythonPath}
if a setting in setting.json such as python.pythonPath="abc", then the value received in the debugger via launch.json is "abc"
However if the value for python.pythonPath="${workspaceRoot}/bin/python", then the value received by the debugger is "${workspaceRoot}/bin/python", when the value ${workspaceRoot} should be resolved.

Solution is to treat all values referenced by the syntax ${config...} as values that need to be resolved.

Coverage increased (+0.0007%) to 61.019% when pulling 228734ee179a33d76f8b9c0b77ff61af0ae63d1e on DonJayamanne:configResolve into 182764c361531fd27c78d4b9a4c50f92e49f7597 on Microsoft:master.

@dbaeumer @weinand not sure who owns this?

@bpasero we (debug) have already planned to consolidate the variable substitution across VS Code. So we can take care of this.
/cc @dbaeumer @isidorn

I'll take ownership

@weinand @isidorn I wonder if we should expand this feature to the general configuration service and so maybe we should talk next milestone when this becomes current.

@bpasero expanding it to everything and moving it to one central place would be awesome. Yeah let's talk about it in September

@DonJayamanne I have added some comments in your commit mostly regarding older changes to this file. The newest change makes sense though - especailly if you tested it.
Love that you have added tests!
Once you adress my comments we can merge this PR in and then @bpasero and me can look into centralizing this.

Merging this PR in as my concerns where unrelated to the code this PR contributes.


docs: Update README

agent-on-container
BOSH scenario

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

v1.0.0
GitHub release

Your semantic-release bot :package::rocket:


feat(service): support publishNotReadyAddresses  (#4500)

Backport
This will backport the following commits from k8s-30/main to k8s-29/main:

feat(service): support publishNotReadyAddresses  (#4500)

Questions ?
Please refer to the Backport tool documentation

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4511

Questions ?
Please refer to the Backport tool documentation


Prüfung der Machbarkeit Schnittstelle der Webuntis API

Task
Postman Requests for getting

classes and class teacher by room
pupils of class
the current lesson of class

Requirements

API-key -> ask sysadmin
Webuntis  API documentation
Postman

Contacted sysadmin on 16.12.2021, still no response.

Got some information from Prof. Stütz:
Example project
https://birklbauerjonas.github.io/webUntis-docs/
by a few former HTL students.
API Documentation
JSONRpc api
Here and/or here.
Additional info
Npm package for nodejs
Here.
Former project at HTL Here.


Initial UI design

A BW mockup of the homepage and side menu to help with finding a color scheme

Initial simple design to help figure out the colors. Will probably adjust the light pink to make the whole thing come together better.


Nightly CI in GitHub Actions

This PR merges some of the automated checks that are run on WBIA to run on new PRs and each push to a branch.  This will run a series of checks, including linting errors and Flake8 style-guides.

This PR is ready for review


Use a proper client-side search engine library to power the search functionality in our pages

For better search results, maybe a full-text search can be used?
The potential libraries are:

https://www.npmjs.com/package/itemsjs (with aggregation to support the filter)
https://www.npmjs.com/package/minisearch
https://fusejs.io/

I'm currently experimenting itemsjs for the search, some things to note:

The aggregation feature can be used for filter, but it's case sensitive and need exact term. Some of the data contain trailing spaces and same text with different variations (eg. Jakarta Barat, Jakarta barat). Need to decide whether the data will be cleaned on the source or on the application.

Sort can be easily set up (can be helpful if we add sort by feature)
There's no function like extractField in Mini Search, so if need it then we need to extend it ourselves. (for example when adding Verified column on the data to be indexed)

Need to decide whether the data will be cleaned on the source or on the application.

For the first phase, we can't rely on the cleaning up from the source sheets, because it is done manually. We should try cleaning it up on our side as much as possible.
However, in the future, we can try to implement a proper Data Validation mechanism in the Google Sheet. I need to talk to Mas Awe about it.

Sort can be easily set up (can be helpful if we add sort by feature)

Yes, please. Btw, based on #131, we shouldn't provide Terverifikasi filter and sort options for the users, but we should always prioritise the verified ones over the non-verified ones.

There's no function like extractField in Mini Search, so if need it then we need to extend it ourselves. (for example when adding Verified column on the data to be indexed)

Then let's focus on the itemsjs first. Moreover, please try to minimize its performance impact into our website by importing the library dynamically if possible. Thanks!
Btw, could you please submit a Draft Pull Request and link it to this issue to signal other contributors that you have started working on it? Thanks, Mbak @ekamuktia!

@zainfathoni Okay, sure! I'll work on this part then


Using an enum class as a template parameter will silently break reflection

Given this:
#include <iostream>
#include <glaze/glaze.hpp>

enum class BType {
    Type1,
    Type2,
};

template<BType T, typename DataType>
class B {
public:
    BType type { T };
    DataType data;

    B(DataType data_) : data(data_) {}

    struct glaze {
        static constexpr auto value = glz::object(&B::type, &B::data);
    };
};

int main() {
    B<ActualType::Type1, int> b(3);
    std::cout << glz::write_json(b) << "\n";

    return 0;
}

It will print out {"Type1, int>::type":0,"Type1, int>::data":3}. Basically it will only take the type until the first :: pair.
I don't know if this should be supported or not (it's a weird and very specific case), but it should be mentioned in the documentation at least.

Thanks for discovering this! Will look into it.

This was an issue with the member object pointer reflection logic. It has been fixed with #672


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1569

Questions ?
Please refer to the Backport tool documentation


RFC: Service discovery backend

Context
In our company we developed a really similar platform as backstage, and in our case we started by taking the services as the root of all connections. If we want to start considering backstage as an option, having a backend able to discover all our services it's a must.
Feature Suggestion
We have two ways of adding a service, by manual input, using some forms, or by detecting a .yml file in the root of the project, where all the info regarding the service is defined.
This feature request should be about the second part only as of right now. Not sure how manual input is seen in other environments different than ours, but technically it shouldn't be hard to do either.
Possible Implementation
To make it flexible and easy to integrate, I'd like to propose having a backend that could be configurable in two ways:

Regex/Glob for descriptor file
Schema of descriptor file

I'm suggesting a regex/glob for identifying the descriptor file because based on our experience, it's not easy to achieve a consensus about where things have to be stored. And I'm also thinking about monorepo support, where there could be a descriptor file inside every service in the monorepo if needed.
Regarding the schema, I'd like to use something like JSON Schema to define it. The question here is if there should be a minimum of fields required, which I'd say yes, to make integration with other backends/plugins easier.
Maybe just an ID? That should be enough to query more details about the service whenever needed, but in that case I'd probably leave it outside of the schema, like we have, and autogenerate it.
Not sure about this, let me know what do you think.
To finish, I'd like to post our current JSON Schema for identifying services:
    type: object
    properties:
        type:
            type: string
        name:
            type: string
        owner:
            type: string
        impact:
            type: string
            enum: ["profit", "customers", "employees"]
        status:
            type: string
            enum: ["beta", "production", "deprecated", "discontinued"]
        docs_url:
            type: string
        slack_channel:
            type: string
        sentry_project:
            type: string
        sonarqube_project:
            type: string
        pagerduty_url:
            type: string
        tags:
            type: array
            items:
                type: string
        environments:
            type: array
            items:
                type: object
                properties:
                    name:
                        type: string
                    dashboard_url:
                        type: string
                    service_urls:
                        type: array
                        items:
                            type: string
                    health_check_url:
                        type: string
    additionalProperties: false
    required:
        - type
        - name
        - owner

In this case, we would like to develop such backend, and make it available also as open source software. We would do it most probably in Go, probably not important, but I just want to know if there is some concern about this choice.

@aexvir Agree that we should do something in this space. We started discussing the model we want to use in this issue: #390.
One thing I'm curious about: How do you see this interacting with source repositories, eg Github/GHE or Gitlab? Because I think these as very related. We'd have some type of service with a REST or gRPC API that stores what entities exist, and some integration with version control that would track changes in master, pick up on those files as you describe, and push them to the service for storage? Like a Github app?

Hi there @mbruggmann
Yes, the relation between repository and service is essential, and it's also core of our implementation. What we have is a django app, with a periodic task that scans all the repositories of our internal GitLab and updates the info in its own PosgtreSQL database.
That can be enhanced by using system-level hooks, that would trigger the backend ad-hoc to register a new service (e.g. on project creation), and CI/Actions/jenkins/whatever jobs if we want to avoid periodic crawling.

@aexvir when you first implemented your Backstage-like internal system, how did you get all teams to agree on the metadata schema and ensure that all the teams implemented it across the company?
How do you ensure that the metadata is correct and fresh? Are there validation steps or systems which ensure people are periodically updating it.

@dtuite we just started up a discussion about the metadata format in a separate issue as well if you wanna have a look: https://github.com/spotify/backstage/issues/767

@dtuite I can speak for us at Spotify at least. Here, the format was essentially dictated by the infra team that owns the internal inventory service. It has very few required fields that are very understandable by humans, so running stale isn't often a thing. And then there's a random bag "put whatever you want in here" fact key as well, for auxiliary data.
The inventory system automatically keeps itself up to date with changes in git. If there are errors or conflicts, it keeps serving the old valid version and informs you that you have issues with the file.
Sometimes the facts random bag has been overused or kind of even abused. But it's not been a major issue.
Uptake of this service descriptor format has been stellar, partly because of our company focus on the golden paths, as we call them. When you create a new "thing", for example a backend service, you do it through Backstage. It scaffolds a bare bones thing for you according with all current best practices, and that includes this descriptor file, which is automatically registered for you. Another reason the uptake is great, is that it's the gateway into Backstage. You don't get all the goodies unless you register your thing in the inventory. We even have a little wizard that makes it trivial to generate just that descriptor, if you have an old and not-yet-inventorised thing.
Some attempts have been made to extend the core fields or rearrange things over the years, but they usually don't fare well - getting the bare essentials right is important.

@freben Appreciate the insight.

Uptake of this service descriptor format has been stellar, partly because of our company focus on the golden paths, as we call them.
Another reason the uptake is great, is that it's the gateway into Backstage.

Yeah we did something similar in Workday. Basically if you don't have your metadata in the catalog you can't get your stuff into production.
Backstage didn't exist at the very start of Spotify though (correct me if I'm wrong please!). So there must have been some moment where there were N teams building stuff but 0 had metadata in their service repos. The infra team invented the metadata schema, but then they had to convince people to use it. How did they then convince all N teams to add the metadata to their repos? There must have been some transition period where things which were already in production had metadata retroactively added to them.
In our case, we essentially put 10 people in a room for a few hours and they manually created metadata for (almost) every service in the system in order to bootstrap it to critical mass so it could start providing value. It kinda sucks but it worked.

@dtuite It wasn't really consulted with anyone, our team is the one centralizing this efforts and we just decided what was the minimum to cover. With time we have expanded it a couple times based on general interest, like when we added environments.
Usually this additions are optional, so it's not a big deal if the teams don't have them set up, but for cases like in the environments additions, we prepared a migration that would move every service's "environment related" fields under a new environment called "default" for example, to avoid friction.

@aexvir Thanks Alex. Sounds like what we did, used a highly motivated team to centralize the effort to get it done.

Time to close this issue now that we have decided on the format?
https://github.com/spotify/backstage/blob/master/docs/architecture-decisions/adr002-default-catalog-file-format.md
@freben

Agreed. @aexvir please have a look at the ADR if you have the time. It leaves a few degrees of freedom open:

The spec for Component is not set in stone yet. It just has a type field now. If we feel that there are more fields that we can place there, that are NOT a good fit for labels or annotations, we can discuss putting them as part of the core spec.
The labels are a possible place to put information directly related to the entity, like your impact and status. We will be looking eventually at a standard set of "well known" labels that both humans (in search filters etc) and plugins that consume entities can rely on, but for now it's left open.


Multiple duplicate Git Providers listed in Source Control on network folders

the issue is very similar to this one
https://github.com/Microsoft/vscode/issues/41636
with one exception, in still happens on windows network paths (vs code jan 2018 release x64 version).
$ git worktree list
//HYPPO/boblin/Work/noname b6ccd13 [master]
issue could be easily reproduced when project folder is located on windows network.

What Git do you have installed?

@joaomoreno git version 2.16.1.windows.1

If this helps, i had similar situation before january release, but back then it helped to restart vs code. now it does not.  Now it helps to map a drive letter to network share with subst, but this is a workaround, not a solution.

leave

I can't seem to reproduce this.

Can you try out our insider release?


Add new organization events to events.json

Appears GitHub has added two events for the revamped organizations.  This PR adds them to the events.json.

Thanks for updating this, @levithomason!

No problem.


Reboot Issues

Hello, I'm noticing some odd behavior with the "reboot" function and was hoping you guys would maybe look at it or provide some insight.
When the entity is rebooted and comes back online, either the entity is "unavailable" after (as declared by the console), or there are less descriptors available than what should be. Same thing happens when configurations are switched...the entity becomes totally unavailable.
Maybe you guys already know about this, but I attached some wiresharks, and the output of the console (for two different devices, Intercom, and J unit).
Thanks,
Katie
AVDECC_reboot_errors.zip

How fast does the device reboot?

Intercom usually reboots in 25 seconds or less, and J Unit is 30 seconds generally, but has taken as long as 45 seconds.

Which branch are you running: staging or master?

master

Another related observation is that in addition to having these "reboot issue", avdecccmdline does not recognize when devices are being taken off the network (unless they are being rebooted).

Okay good. I was concerned about a recent PR of mine on the staging branch that might be causing issues.

Off the network for how long? There are some timeouts that have to occur before a device is flagged as missing.

Even off the network for several minutes, I can even see the "DISCONNECTED" command in the console, yet none of the info updates.. (see attached. Jim's unit was taken offline).
endStation_disconnected.txt

What unit is 0x197cfffe0020c2? A list printout might be helpful, just so the Entity IDs in the log messages actually have meaning.

Even off the network for several minutes, I can even see the "DISCONNECTED" command in the console, yet none of the info updates..

@avd17  Hi Katie, can you clarify what you mean by "none of the info updates"?
I have not done much testing with the 'Reboot' API, but I can comment on what the expected behavior in avdecccmdline should be when devices go offline and back on again.
We do not clear out the device model in avdecc-lib when a device goes offline - we simply mark the end station as disconnected.
This is why you were still able see the device info in: > view all after the DISCONNECTED notification.


[resotocore][fix] Handle connection errors more gracefully

Description
ConnectionAborted Errors were not handled correctly during startup.
Increase the time to wait for a connection to the database.
Code of Conduct
By submitting this pull request, I agree to follow the code of conduct.

Codecov Report

Merging #844 (6c4288a) into main (9ffb7b2) will increase coverage by 0.00%.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##             main     #844   +/-   ##
=======================================
  Coverage   89.98%   89.98%
=======================================
  Files          69       69
  Lines       10002    10003    +1
=======================================
+ Hits         9000     9001    +1
  Misses       1002     1002

Flag
Coverage Δ

resotocore
89.98% <66.66%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

resotocore/resotocore/db/db_access.py
85.54% <33.33%> (ø)

resotocore/resotocore/__main__.py
88.88% <100.00%> (+0.07%)
: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 44a5a54...6c4288a. Read the comment docs.


git plugin - option to limit depth of historical scans (default all commits)

add a number argument that is going to limit the historical scans when using the git plugin

@jossef What does this "number" represents?
I think we should use a "range" of time like we do in Discord, or use a counter to limit how many commits we scan, or use a specific commit baseline.
Of course, as we are using the GitLog function from gitleaks, we first need to learn which limitations work with this function.

I'm working on this issue

@jossef What does this "number" represents?
I think we should use a "range" of time like we do in Discord, or use a counter to limit how many commits we scan, or use a specific commit baseline.
Of course, as we are using the GitLog function from gitleaks, we first need to learn which limitations work with this function.

number = how many commits to go back from HEAD


File whitelisting for npm publish

tl;dr let's standardize on https://docs.npmjs.com/files/package.json#files
background
The use of npm publishing is important for hpp-skel because header-only C++ libs are frequently used in node.js C++ addons. With npm publishing we want to ensure only the minimal files are included in the published tarball. The normal approach for this is to depend on .npmignore. .npmignore works the same as .gitingore: you blacklist things you don't want packaged (and by default if no .npmingore is used then .gitignore will be used in its place.
problem
But this solution does not work for a skel repo. Here we can't know what files will be created by authors using skel as a starting point. So, blacklisting is full of fail.
solution
So, we want to use whitelisting instead. The convention for header-only libs based on hpp-skel is that only the include_dirs.js and the include/ directory (in addition to the automatically included README.md and packag.json) will need to be packaged. Because there is no support for .npminclude (https://github.com/npm/npm/issues/13171) we can do this with the package.json files feature: https://docs.npmjs.com/files/package.json#files
/cc @mapsam @GretaCB @mikemorris

Note: to get this working we'll need to remove this empty .npmignore: https://github.com/mapbox/hpp-skel/blob/master/.npmignore

Hey @springmeyer picking this up again - should we include the Makefile as well?

We are no longer recommending publishing to npm. Rather we will only publish to mason. This issue has some history on the switch in mindset: https://github.com/mapbox/cpp/issues/20. Only using mason avoids having to publish to two places. This also means that, to install headers on windows, we'll need a small mason client written in javascript (https://github.com/mapbox/mason/issues/396). @mapsam - this would probably be a 1-2 day project to scope out - let's chat about this voice.

To be clear: so this issue can be closed. And we no longer need a package.json in this repo.

Sweet! I figured we were nearing that case. I'm very 👍 avoiding NPM entirely here.


Allowing parent component to multicast data to all children

Scenario
TodoList is a parent with many Todos child.
Todo can be highlighted only 1 at a time. Highlighting a Todo will make all the other Todos to be unhighlighted by toggling a class.
Current
Each Todo can talk to the parent TodoList .. but currently it's not simple to make a parent to talk to every Todos children at once without using  a proxy stream

I think you can make a hovers$ stream by flat-merging all of the item's hover sinks ithe same way as Collection.pluck flat-combines the sinks of given name. This stream can be passed as an additional source when you add the first item: collection.add({ hovers$ })

I ran into this but it was making a certain tab active and making the rest go back to normal.
I wonder if it's a common enough pattern that we should add an abstraction that each item in a collection can return a particular sink that gets wired to the sources of all the other collection items. Perhaps a Broadcast sink and source. That way when a Todo gets clicked it could Broadcast to all other items which would then be inactive.

Sounds good but how would u implement it right now? @Widdershin with the tab active etc

@FeliciousX at the moment you would need to implement a circular dependency using imitate, and the goal of this library in part is to remove the requirement for users to write circular dependencies in their code.
I can't think of any way to do it with the current API.
I've been tossing up a collection.update(...items, sources) that would allow you to send things into your child components, but that strikes me as not very reactive.
I might try implement the Broadcast style and see how that feels.

I've highlighted above one way to do that. The scheme is as follows:

define an addReducer which should pass an additional source
create a collection
create a collection stream by folding over a stream of reducers
do some kind of sink processing like Collection.Pluck does, maybe using merge instead of combine
store the resulting broadcast stream in a variable, which will be than passed by addReducer as a source

It might look that we have another a=b, b=a problem, but in fact we haven't as soon as addReducer is called asynchronously.

That is frickin ingenious.

Collection.merge now makes things a bit easier here.


Fix urgent crashes   A and E

Description
Fixes A and E from #1545 (I hope). It saves file which is being uploaded to a temp location, and uses Uri of that file as reference during upload process, instead of using Uri that content provider supported.
Tests performed (required)
Tested on API 23, beta debug. Tested actions: share photo from another application, share multiple photos from another application and contributing via contributions list.
Not: Finally a beautiful commit history!

Codecov Report

Merging #1749 into 2.8-release will decrease coverage by <.01%.
The diff coverage is 3.1%.

@@              Coverage Diff               @@
##           2.8-release   #1749      +/-   ##
==============================================
- Coverage         3.89%   3.89%   -0.01%
==============================================
  Files              151     154       +3
  Lines             7521    7657     +136
  Branches           705     723      +18
==============================================
+ Hits               293     298       +5
- Misses            7211    7341     +130
- Partials            17      18       +1

Impacted Files
Coverage Δ

...ain/java/fr/free/nrw/commons/upload/FileUtils.java
3.22% <ø> (ø)
:arrow_up:

...java/fr/free/nrw/commons/upload/ShareActivity.java
0% <0%> (ø)
:arrow_up:

...java/fr/free/nrw/commons/upload/UploadService.java
0% <0%> (ø)
:arrow_up:

...fr/free/nrw/commons/nearby/NearbyListFragment.java
0% <0%> (ø)
:arrow_up:

...main/java/fr/free/nrw/commons/utils/FileUtils.java
0% <0%> (ø)

...r/free/nrw/commons/contributions/Contribution.java
0% <0%> (ø)
:arrow_up:

...w/commons/contributions/ContributionsActivity.java
0% <0%> (ø)
:arrow_up:

...rw/commons/mwapi/ApacheHttpClientMediaWikiApi.java
5% <0%> (-0.02%)
:arrow_down:

...r/free/nrw/commons/utils/ExternalStorageUtils.java
0% <0%> (ø)

...n/java/fr/free/nrw/commons/MediaWikiImageView.java
0% <0%> (ø)
:arrow_up:

... and 10 more

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 25b4639...ec192c5. Read the comment docs.

Ready to be tested @misaochan :)


Carriage return ignored/forgotten

I am trying to run this code:
print("This text will be replaced\rWith this text")

This code outputs as:
This text will be replacedWith this text

The output should be:
With this text be replaced

I then ran this code:
import requests
code = """print('This text will be replaced\rWith this text')"""
resp = requests.post("https://emkc.org/api/v2/piston/execute", {"language": f"python", "version": "3.10.0", "files": [code]}).json()
output = "\n".join(line for line in resp['run']['output'].splitlines())
print(output)

Which gave me this output:
  File "/piston/jobs/8c28a39a-0386-4f93-86f5-1e5cc2ac494e/file0.code", line 1
    print("This text will be replaced
          ^
SyntaxError: unterminated string literal (detected at line 1)

This is an issue as I am trying to create a Python challenge for my Discord bot that contains the carriage return character.

The terminal emulator is responsible for the replacement behavior of carriage returns. This means that the expected behavior is to print out the whole string, including the carriage return.

You can see this with the following commands:
# Replacement happens
python -c 'print("This text will be replaced\rWith this text")'
# No replacement, file is 42 bytes
python -c 'print("This text will be replaced\rWith this text")' > file
du --bytes file

As for the second issue, the carriage return is interpreted before being sent to Piston so python sees a carriage return (functionally a new line) instead of \r.
You can see this by doing the following:
len("""test""")
len("""\r""")

If you want to keep the escape code intact you should escape the escape character, like so: string = """\\r"""


Support STDIN/STDOUT

If there was a flag like --stdout to output the changed text to STDOUT, that would be nice.
Being able to read from STDIN would make it perfect for autoformatters like neoformat.

I believe you're looking for --print-only?

My bad. Does this support reading STDIN as well (afk, so I can't check right now)

reorder-python-imports /dev/stdin --print-only should work:
$ echo -e 'import os\nimport pre_commit\nimport sys' | reorder-python-imports /dev/stdin --print-only
==> /dev/stdin <==
import os
import sys

import pre_commit

(note that the ==> /dev/stdin <== output goes to stderr)

closing: if --print-only doesn't satisfy your usecase please provide additional context and I'll reopen :)

Sorry, forgot about this. It needs to output specifically to STDOUT rather than STDERR since most editor formatter plugins only read STDOUT. Plus, it's not really an error as such, so it shouldn't be redirected there.

it does, you haven't tried it have you :P

You're right, I didn't. I misread the sentence "(note that the ==> /dev/stdin <== output goes to stderr)". I thought you meant all of it was redirected to STDERR rather than just that literal line. My bad.

On the other hand, I don't think Windows supports /dev/stdin. I don't use Windows, but I'd expect other people do.

If someone on windows wants the feature then we'll talk -- as it is it hasn't been an ask so far and if I'm not going to use it and you're not going to use it there's no point in complicating this tool

I'm trying to implement emacs support for this fine tool. This would be a little easier if --print-only always printed the result, whether there was a reordering or not. Is that possible?


Threads support

Additional context
https://support.discord.com/hc/de/articles/4403205878423-Threads
Blocked by serenity implementing thread support.

Bot can respond in threads.


Time.now from param in a resque job

Hi I don't know if this is a bug, or if it's something that I should know of and handle myself. I'm running 3.1 and I'm in config.time_zone = 'Copenhagen'
I have a job in resque that looks like this, when it's beeing queued:
Resque.enqueue(Lookups::Workers::CleaningJob, site, Time.now - 5.minutes)

Now, this stores a job like this in resque:
Crawling::Workers::AdvertCleaningJob 	["Google", "2011-12-08T22:30:52+01:00"]
Now the problem shows, when I later run the queued job. The job executes a query, that uses the timestamp (in a variable called last_cleaned_time =  "2011-12-08T22:30:52+01:00")
Site.where{(site == site_name) & (last_cleaned < last_cleaned_time)}

This executes a query looking something like this:
SELECT "sites".* FROM "sites" WHERE (("sites"."site" = 'Google' AND "sites"."last_cleaned" < '2011-12-08T22:30:52+01:00'))

And THAT is not what I want - the time has not been parsed correctly.
If I for testing do like this in the console:
site.update_attribute(:last_cleaned, "2011-12-08T22:30:52+01:00")

I can see from the resulting sql, that rails translates the time correctly
UPDATE "sites" SET "last_cleaned" = '2011-12-08 21:30:52.000000', "updated_at" = '2011-12-08 22:14:30.632682' WHERE "sites"."id" = 12

So in other words - I would have expected sqeel to handle this, just like rails, without doing a Time.parse() on the variable?

I'm afraid that isn't a Squeel thing. If you check into the Resque docs you'll see that job params are serialized to JSON. You'd probably do better to convert to a unix timestamp on the way in and out.

Thanks!


Dropdown Field with Editable New Values

Hey Dan,
Is there any way to create a dropdown list field with the option of the user type new values besides the possible values in the field's items?
Thanks! Your package is the main one in my app and I love it. Need to figure out how to do this.
Sincerely,
Daniel

Maybe method registerField can be help.
If this issue is resolved, please close it


Import fails, missing opencv-python requirements.txt

After installing only comfyui's and ComfyUI_Auto_Caption's requirements.txt, I have this error:
Launching ComfyUI from: /mnt/comfyui/app

Total VRAM 16073 MB, total RAM 32000 MB
pytorch version: 2.5.1+cu124
Set vram state to: NORMAL_VRAM
Device: cuda:0 NVIDIA GeForce RTX 4060 Ti : cudaMallocAsync
Using pytorch cross attention
[Prompt Server] web root: /mnt/comfyui/app/web
Traceback (most recent call last):
  File "/mnt/comfyui/app/nodes.py", line 2035, in load_custom_node
    module_spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/mnt/comfyui/app/custom_nodes/ComfyUI_Auto_Caption/__init__.py", line 2, in <module>
    from .auto_caption import Joy_Model_load, Auto_Caption
  File "/mnt/comfyui/app/custom_nodes/ComfyUI_Auto_Caption/auto_caption.py", line 13, in <module>
    from .lib.ximg import *
  File "/mnt/comfyui/app/custom_nodes/ComfyUI_Auto_Caption/lib/ximg.py", line 14, in <module>
    import cv2 as cv
ModuleNotFoundError: No module named 'cv2'

Cannot import /mnt/comfyui/app/custom_nodes/ComfyUI_Auto_Caption module for custom nodes: No module named 'cv2'

Import times for custom nodes:
   0.0 seconds: /mnt/comfyui/app/custom_nodes/websocket_image_save.py
   0.2 seconds (IMPORT FAILED): /mnt/comfyui/app/custom_nodes/ComfyUI_Auto_Caption

Starting server

To see the GUI go to: http://<IP_ADDRESS>:8080

Thanks
I can close this issue
I also closed my PR
Best regards


Ubuntu (maybe other) apt repo missing metadata for older versions

Being able to install specific versions of docker-engine is important for configuration management. Without this, I don't have control over which versions of docker-engine are installed on the various VMs, instances, and machines throughout my different environments.
The current apt repo has the binaries for older revs, but (at least for Ubuntu trusty) it's missing the metadata in order to install them. It's likely the metadata is missing for other distros, but I have not checked.
# cat /etc/apt/sources.list.d/apt_dockerproject_org_repo.list
deb https://apt.dockerproject.org/repo ubuntu-trusty main
# apt-get -qq update
# apt-cache show docker-engine | grep ^Version:
Version: 1.8.1-0~trusty

docker version: <trying to install 1.7.1>
docker info: <trying to install 1.7.1>
uname -a: Linux docker-foo 3.13.0-61-generic #100-Ubuntu SMP Wed Jul 29 11:21:34 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
Provide additional environment details (AWS, VirtualBox, physical, etc.): instance in AWS
List the steps to reproduce the issue:

Put deb https://apt.dockerproject.org/repo ubuntu-trusty main in /etc/apt/sources.list.d/apt_dockerproject_org_repo.list
apt-get -qq update
apt-get install docker-engine=1.7.1*

Describe the results you received:
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Version '1.7*' for 'docker-engine' was not found

Describe the results you expected:
Selected version '1.7.1-0~trusty' (ubuntu-trusty [amd64]) for 'docker-engine'
The following NEW packages will be installed:
  docker-engine

Provide additional info you think is important:
There are at least two other issues (#15581 and #9697) regarding specifying the version of docker-engine you with to install.

ping @jfrazelle

This is a dup

Dup of https://github.com/docker/docker/issues/13191

Sorry for the dup! Don't know why I couldn't find that issue earlier.
Thanks!


Gsearch opdated to use new gsearch v2 API instead of old geosearch API

The implementation now uses a parallel lookup to all selected resources for efficiency and a regex/string split parsing operation as loading Newtonsoft.JSON, GeoJSON.NET or required memory related references failed under ArcMap 10.8.1.
The test cases has been extended to improve result and operations stability also

Rejected due to bug in resource selection


Update authorfile1.md

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

Docs Build status updates of commit 60ba966:
:warning: Validation status: warnings

File
Status
Preview URL
Details

CSIDevDocsPublic/authorfile1.md
:warning:Warning

Details

CSIDevDocsPublic/authorfile1.md

Line 2, Column 8: [Warning: duplicate-titles - See documentation] Title 'Acrolinx report' is duplicated in: 'CSIDevDocsPublic/authorfile1.md(2,8), CSIDevDocsPublic/authorfile10.md(2,8), CSIDevDocsPublic/authorfile11.md(2,8), CSIDevDocsPublic/authorfile12.md(2,8), CSIDevDocsPublic/authorfile13.md(2,8),...'. Titles must be unique within a docset.
Line 12, Column 1: [Warning: duplicate-h1s - See documentation] H1 'Acrolinx report' is duplicated with other articles: 'CSIDevDocsPublic/authorfile1.md(12,1), CSIDevDocsPublic/authorfile10.md(12,1), CSIDevDocsPublic/authorfile11.md(12,1), CSIDevDocsPublic/authorfile12.md(12,1), CSIDevDocsPublic/authorfile13.md(12,1),...'. First level headings must be unique within a docset.
Line 3, Column 14: [Suggestion: duplicate-descriptions - See documentation] Attribute 'description' with value 'This article discusses issues and fixes planned for the Acrolinx report.' is duplicated in 'CSIDevDocsPublic/authorfile1.md(3,14)', 'CSIDevDocsPublic/authorfile10.md(3,14)', 'CSIDevDocsPublic/authorfile11.md(3,14)', 'CSIDevDocsPublic/authorfile12.md(3,14)', 'CSIDevDocsPublic/authorfile13.md(3,14)'....
Line 5, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'adundevster' is not a valid Microsoft alias.

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

#sign-off test

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @adunndevster

#assign-reviewer: james-yao-msft

Invalid command: '#assign-reviewer'. Only Microsoft employees can use this command.

#please-close

#please-open

Invalid command: '#please-open'. Only Microsoft employees can use this command.

#assign:meganbradley

Invalid command: '#assign'. Only Microsoft employees can use this command.

#unassign:adunndevster

Invalid command: '#unassign'. Only Microsoft employees can use this command.

#assign-reviewer:meganbradley

Invalid command: '#assign-reviewer'. Only Microsoft employees can use this command.


DRILL-6770: JsonTableGroupScan should use new MapRDB 6.1.0 APIs

MapRDB 6.1.0 has removed support for some APIs which breaks Drill code (Deprecation should have been used instead). This PR uses the new APIs instead. @amansinha100 can you please review the PR? Thanks!

The change should solve the issue. Looks like to avoid similar in future it would be good to get rid of using com.mapr.db.impl.TabletInfoImpl in Drill code, because it is marked as @Internal.
@gparai Is it possible to do it? Or maybe mark it with TODO comment somewhere in the code.

@vdiravka yes these changes will be overridden by the Secondary Index PR which is currently in review. For this PR, we just target to fix the failure since those other changes are bigger in scope.

+1  to get it merged to get past the current blocking issue.  There could be potential plan changes though (which are not tracked by functional tests) since the  filtered row count now will be higher than before (even though it will be more accurate).   So we should follow up with some more targeted tests.

@gparai, this PR causes the next functional tests failures:
Execution Failures:

/root/drillAutomation/drill-test-framework/framework/resources/Functional/hive/hive_storage/hive_maprdb_json/less_than.sql.explain

Plan Verification Failures:

/root/drillAutomation/drill-test-framework/framework/resources/Functional/hive/hive_storage/hive_maprdb_json/between.sql.explain

Random Failures:

/root/drillAutomation/drill-test-framework/framework/resources/External/mapr-drill-tests/Testcases/maprdb/limit0/plan/pushdown_q20.q
/root/drillAutomation/drill-test-framework/framework/resources/External/mapr-drill-tests/Testcases/maprdb/pushdown/query/pushdown_q8.q
/root/drillAutomation/drill-test-framework/framework/resources/External/mapr-drill-tests/Testcases/maprdb/limit0/data/pushdown_q18.q
/root/drillAutomation/drill-test-framework/framework/resources/External/mapr-drill-tests/Testcases/maprdb/limit0/plan/rowkey-range.q
/root/drillAutomation/drill-test-framework/framework/resources/External/mapr-drill-tests/Testcases/maprdb/limit0/plan/avg.q
/root/drillAutomation/drill-test-framework/framework/resources/Functional/schema_change_empty_batch/maprdb/binary_maprdb/emptyMaprDBUnion.sql
/root/drillAutomation/drill-test-framework/framework/resources/External/mapr-drill-tests/Testcases/maprdb/sanity/date_add.q
/root/drillAutomation/drill-test-framework/framework/resources/Functional/schema_change_empty_batch/maprdb/json/emptyMaprDBJsonLeftJoin.sql
/root/drillAutomation/drill-test-framework/framework/resources/Functional/schema_change_empty_batch/maprdb/binary_hbase/emptyMaprDBScan1.sql
/root/drillAutomation/drill-test-framework/framework/resources/Functional/hive/hive_storage/hive_maprdb_json/project.sql
/root/drillAutomation/drill-test-framework/framework/resources/Functional/hive/hive_storage/hive_maprdb_json/like.sql.explain
/root/drillAutomation/drill-test-framework/framework/resources/Functional/hive/hive_storage/hive_maprdb_json/filter_id.sql.explain
/root/drillAutomation/drill-test-framework/framework/resources/Functional/hive/hive_storage/hive_maprdb_json/filter_id.sql

and advanced tests failures:
Execution Failures:

/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/sanity/maprdb/json/query23_max.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/smoke/maprdb/json/join03-merge.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/smoke/maprdb/json/group-by01.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/smoke/maprdb/json/group-by02.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/smoke/maprdb/json/failed04.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/original/maprdb/json/query10.sql

Data Verification Failures:

/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/sanity/maprdb/json/query30_distinct.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/original/maprdb/json/query13.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/original/maprdb/json/query18.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/original/maprdb/json/query4.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/original/maprdb/json/query8.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/smoke/maprdb/json/failed02.sql

Timeouts:

/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/original/maprdb/json/query1.sql
/root/drillAutomation/drill-test-framework/framework/resources/Advanced/tpch/tpch_sf1/sql_functions/strings/maprdb/json/query10.sql

Could you please take a look?


Organizations: create_organization fails to add master account and default policy to mock organization

the mock organization does not contain the master account or the default policy.  After running
create_organization, the list_accounts and list_policies methods each return an empty list.

patch forthcoming


Enable conflict marker decorators even if merge editor is enabled.

That is exactly the problem, with this fix we basically have to remove the conflict markers, as the code lens would show up in the merge editor.
This is a hot fix to address the most pressing issues that we broke the existing workflow.


Will it run in background?

Will it run in background? (i.e) will the tracking persist even when the app is swipe closed from app switcher?

Here's some guidance on how to do background execution in Android.


New package: PackageTest v0.1.0

Registering package: PackageTest
Repository: https://github.com/yogaz33/PackageTest
Created by: @yogaz33
Version: v0.1.0
Commit: c1a3fe14fea6f3b5e56dfb7a6fa1bb4ac8097cb2
Reviewed by: @yogaz33
Reference: https://github.com/yogaz33/PackageTest/issues/1

Welcome! It looks like you are testing out how to make a package. If that is the case, then I don't think this PR needs to be merged, right?


Help me with item achievements.

Hello, i'm new here and i've got some problems. Long ago, i unlocked all the achievements by this program in Team Fortress 2. In their number, there was some that will give you special items on unlock. So, recently, i deleted all achievement items and erased all the achievements in the game(there is a button that deletes them), and tried to unlock them again. But, when i unlocked them by SAM, commited my changes, and look for the game, it gave me nothing. I tried everything - guides, reset, restart. But it does nothing. I'm asking for help, if someone can give me some advices.

No, those items are one time only, once lost you can't get them back


Fix failing tests due to url-encoded file paths

Of course, this only fails on internal CI otherwise life is too simple to debug.

:broken_heart: Tests Failed

Expand to view the summary

Build stats

Build Cause: [Pull request #1218 opened]

Start Time: 2020-06-05T14:42:07.390+0000

Duration: 17 min 16 sec

Test stats :test_tube:

Test
Results

Failed
1

Passed
953

Skipped
1

Total
955

Test errors
Expand to view the tests failures

Name: Tests / Smoke Tests 02 / testQueueSendReceiveOnNonTracedThread[BrokerFacade=co.elastic.apm.agent.jms.ActiveMqFacade@abf688e] – co.elastic.apm.agent.jms.JmsInstrumentationIT

Age: 1
Duration: 0.184
Error Details:
Expecting actual not to be null

Steps errors
Expand to view the steps failures

Name: Shell Script

Description: ./scripts/jenkins/smoketests-02.sh

Duration: 5 min 18 sec

Start Time: 2020-06-05T14:53:44.886+0000

log

Log output
Expand to view the last 100 lines of log output

[2020-06-05T14:59:17.620Z] [INFO] Codecov: Getting branch ref...
[2020-06-05T14:59:17.663Z] [INFO] Codecov: Getting branch ref...
[2020-06-05T14:59:17.668Z] Masking supported pattern matches of $GITHUB_TOKEN
[2020-06-05T14:59:17.714Z] Masking supported pattern matches of $GITHUB_TOKEN
[2020-06-05T14:59:17.958Z] [INFO] Codecov: Sending data...
[2020-06-05T14:59:18.072Z] [INFO] Codecov: Sending data...
[2020-06-05T14:59:18.446Z] + curl -sSLo codecov.sh https://codecov.io/bash
[2020-06-05T14:59:18.446Z] + bash codecov.sh
[2020-06-05T14:59:18.446Z]
[2020-06-05T14:59:18.446Z]   _____          _
[2020-06-05T14:59:18.446Z]  / ____|        | |
[2020-06-05T14:59:18.446Z] | |     ___   __| | ___  ___ _____   __
[2020-06-05T14:59:18.446Z] | |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
[2020-06-05T14:59:18.446Z] | |___| (_) | (_| |  __/ (_| (_) \ V /
[2020-06-05T14:59:18.446Z]  \_____\___/ \__,_|\___|\___\___/ \_/
[2020-06-05T14:59:18.446Z]                               Bash-20200602-f809a24
[2020-06-05T14:59:18.446Z]
[2020-06-05T14:59:18.446Z]
[2020-06-05T14:59:18.446Z] ==> Jenkins CI detected.
[2020-06-05T14:59:18.446Z]     project root: .
[2020-06-05T14:59:18.446Z] --> token set from env
[2020-06-05T14:59:18.446Z]     Yaml not found, that's ok! Learn more at http://docs.codecov.io/docs/codecov-yaml
[2020-06-05T14:59:18.627Z] + curl -sSLo codecov.sh https://codecov.io/bash
[2020-06-05T14:59:18.627Z] + bash codecov.sh
[2020-06-05T14:59:18.627Z]
[2020-06-05T14:59:18.627Z]   _____          _
[2020-06-05T14:59:18.627Z]  / ____|        | |
[2020-06-05T14:59:18.627Z] | |     ___   __| | ___  ___ _____   __
[2020-06-05T14:59:18.627Z] | |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
[2020-06-05T14:59:18.627Z] | |___| (_) | (_| |  __/ (_| (_) \ V /
[2020-06-05T14:59:18.627Z]  \_____\___/ \__,_|\___|\___\___/ \_/
[2020-06-05T14:59:18.628Z]                               Bash-20200602-f809a24
[2020-06-05T14:59:18.628Z]
[2020-06-05T14:59:18.628Z]
[2020-06-05T14:59:18.628Z] ==> Jenkins CI detected.
[2020-06-05T14:59:18.628Z]     project root: .
[2020-06-05T14:59:18.628Z] --> token set from env
[2020-06-05T14:59:18.628Z]     Yaml not found, that's ok! Learn more at http://docs.codecov.io/docs/codecov-yaml
[2020-06-05T14:59:18.708Z] ==> Running gcov in . (disable via -X gcov)
[2020-06-05T14:59:18.708Z] ==> Python coveragepy not found
[2020-06-05T14:59:18.708Z] ==> Searching for coverage reports in:
[2020-06-05T14:59:18.708Z]     + .
[2020-06-05T14:59:18.889Z] ==> Running gcov in . (disable via -X gcov)
[2020-06-05T14:59:18.890Z] ==> Python coveragepy not found
[2020-06-05T14:59:18.890Z] ==> Searching for coverage reports in:
[2020-06-05T14:59:18.890Z]     + .
[2020-06-05T14:59:19.283Z]     -> Found 24 reports
[2020-06-05T14:59:19.283Z] ==> Detecting git/mercurial file structure
[2020-06-05T14:59:19.283Z] ==> Reading reports
[2020-06-05T14:59:19.283Z]     + ./apm-agent-core/target/site/jacoco/jacoco.xml bytes=1045020
[2020-06-05T14:59:19.283Z]     + ./apm-agent-plugins/apm-redis-plugin/apm-lettuce-plugin/target/site/jacoco/jacoco.xml bytes=24314
[2020-06-05T14:59:19.283Z]     + ./apm-agent-plugins/apm-redis-plugin/apm-jedis-plugin/target/site/jacoco/jacoco.xml bytes=7365
[2020-06-05T14:59:19.283Z]     + ./apm-agent-plugins/apm-redis-plugin/apm-redis-common/target/site/jacoco/jacoco.xml bytes=3102
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-redis-plugin/apm-redisson-plugin/target/site/jacoco/jacoco.xml bytes=4672
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-jaxrs-plugin/target/site/jacoco/jacoco.xml bytes=18877
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-grpc/apm-grpc-plugin/target/site/jacoco/jacoco.xml bytes=47457
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-es-restclient-plugin/apm-es-restclient-plugin-6_4/target/site/jacoco/jacoco.xml bytes=9248
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-es-restclient-plugin/apm-es-restclient-plugin-common/target/site/jacoco/jacoco.xml bytes=13247
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-okhttp-plugin/target/site/jacoco/jacoco.xml bytes=36388
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-java-concurrent-plugin/target/site/jacoco/jacoco.xml bytes=9173
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-jmx-plugin/target/site/jacoco/jacoco.xml bytes=41105
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-mongoclient-plugin/target/site/jacoco/jacoco.xml bytes=12681
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-profiling-plugin/target/site/jacoco/jacoco.xml bytes=333100
[2020-06-05T14:59:19.284Z]     + ./apm-agent-plugins/apm-api-plugin/target/site/jacoco/jacoco.xml bytes=83169
[2020-06-05T14:59:19.285Z]     + ./apm-agent-plugins/apm-error-logging-plugin/target/site/jacoco/jacoco.xml bytes=10841
[2020-06-05T14:59:19.285Z]     + ./apm-agent-plugins/apm-httpclient-core/target/site/jacoco/jacoco.xml bytes=5071
[2020-06-05T14:59:19.285Z]     + ./apm-agent-plugins/apm-jaxws-plugin/target/site/jacoco/jacoco.xml bytes=4575
[2020-06-05T14:59:19.285Z]     + ./apm-agent-plugins/apm-hibernate-search-plugin/apm-hibernate-search-plugin-5_x/target/site/jacoco/jacoco.xml bytes=6020
[2020-06-05T14:59:19.285Z]     + ./apm-agent-plugins/apm-hibernate-search-plugin/apm-hibernate-search-plugin-6_x/target/site/jacoco/jacoco.xml bytes=5917
[2020-06-05T14:59:19.461Z] --> No coverage report found.
[2020-06-05T14:59:19.461Z]     Please visit http://docs.codecov.io/docs/supported-languages
[2020-06-05T14:59:19.548Z]     + ./apm-agent-plugins/apm-jdbc-plugin/target/site/jacoco/jacoco.xml bytes=92239
[2020-06-05T14:59:19.548Z]     + ./apm-agent-plugins/apm-apache-httpclient-plugin/target/site/jacoco/jacoco.xml bytes=41392
[2020-06-05T14:59:19.548Z]     + ./apm-agent-plugins/apm-asynchttpclient-plugin/target/site/jacoco/jacoco.xml bytes=17673
[2020-06-05T14:59:19.548Z]     + ./apm-agent-plugins/apm-jms-plugin/target/site/jacoco/jacoco.xml bytes=44717
[2020-06-05T14:59:19.548Z] ==> Appending adjustments
[2020-06-05T14:59:19.548Z]     https://docs.codecov.io/docs/fixing-reports
[2020-06-05T14:59:19.810Z]     -> No adjustments found
[2020-06-05T14:59:19.811Z] ==> Gzipping contents
[2020-06-05T14:59:19.847Z] Failed in branch Smoke Tests 01
[2020-06-05T14:59:20.072Z] ==> Uploading reports
[2020-06-05T14:59:20.073Z]     url: https://codecov.io
[2020-06-05T14:59:20.073Z]     query: branch=elastic%2Ffix-snapshot-ci&commit=0ef30999b9f905292a638ab3fd6258f80223d915&build=1&build_url=https%3A%2F%2Fapm-ci.elastic.co%2Fjob%2Fapm-agent-java%2Fjob%2Fapm-agent-java-mbp%2Fjob%2FPR-1218%2F1%2F&name=&tag=&slug=elastic%2Fapm-agent-java&service=jenkins&flags=&pr=1218&job=
[2020-06-05T14:59:20.073Z]     -> Pinging Codecov
[2020-06-05T14:59:20.073Z] https://codecov.io/upload/v4?package=bash-20200602-f809a24&token=secret&branch=elastic%2Ffix-snapshot-ci&commit=0ef30999b9f905292a638ab3fd6258f80223d915&build=1&build_url=https%3A%2F%2Fapm-ci.elastic.co%2Fjob%2Fapm-agent-java%2Fjob%2Fapm-agent-java-mbp%2Fjob%2FPR-1218%2F1%2F&name=&tag=&slug=elastic%2Fapm-agent-java&service=jenkins&flags=&pr=1218&job=
[2020-06-05T14:59:20.335Z]     -> Uploading
[2020-06-05T14:59:20.909Z]     -> View reports at https://codecov.io/github/elastic/apm-agent-java/commit/0ef30999b9f905292a638ab3fd6258f80223d915
[2020-06-05T14:59:21.204Z] Failed in branch Unit Tests
[2020-06-05T14:59:21.505Z] Stage "Integration Tests" skipped due to earlier failure(s)
[2020-06-05T14:59:21.604Z] Stage "AfterRelease" skipped due to earlier failure(s)
[2020-06-05T14:59:21.650Z] Stage "AfterRelease" skipped due to earlier failure(s)
[2020-06-05T14:59:21.752Z] Stage "AfterRelease" skipped due to earlier failure(s)
[2020-06-05T14:59:22.287Z] Running on Jenkins in /var/lib/jenkins/workspace/_java_apm-agent-java-mbp_PR-1218
[2020-06-05T14:59:22.398Z] [INFO] getVaultSecret: Getting secrets
[2020-06-05T14:59:22.474Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2020-06-05T14:59:23.298Z] + chmod 755 generate-build-data.sh
[2020-06-05T14:59:23.298Z] + ./generate-build-data.sh https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1218/ https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1218/runs/1 FAILURE 1035637
[2020-06-05T14:59:23.849Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1218/runs/1/steps/?limit=10000 -o steps-info.json
[2020-06-05T14:59:24.400Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1218/runs/1/tests/?status=FAILED -o tests-errors.json
[2020-06-05T14:59:24.400Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-java/apm-agent-java-mbp/PR-1218/runs/1/log/ -o pipeline-log.txt

Codecov Report

Merging #1218 into master will increase coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #1218      +/-   ##
============================================
+ Coverage     59.45%   59.48%   +0.03%
- Complexity       87     2448    +2361
============================================
  Files           339      339
  Lines         15967    15969       +2
  Branches       2228     2228
============================================
+ Hits           9493     9499       +6
+ Misses         5824     5823       -1
+ Partials        650      647       -3

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/co/elastic/apm/agent/util/IOUtils.java
57.00% <0.00%> (-1.09%)
16.00 <0.00> (+16.00)
:arrow_down:

...ic/apm/agent/profiler/collections/LongHashSet.java
18.65% <0.00%> (+0.79%)
0.00% <0.00%> (ø%)

...o/elastic/apm/agent/profiler/SamplingProfiler.java
72.84% <0.00%> (+1.01%)
0.00% <0.00%> (ø%)

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 595a1b6...0ef3099. Read the comment docs.

I've managed to reproduce it locally simply by having a space in the path of the project folder (because it's what happens when you have a space in your jenkins job name).
With that I've found another place where it was failing in a similar way, and we can now re-enable some tests that were skipped on CI due to this :tada:


Followup on Generic Typing of Layers and type of layer.mode

🧰 Task
Currently, #5869 #5852 #5871 have to type layer.mode as str, because fo some complexity.

Layer base need to refer the a "self.mode",
Each subclass has self.mode  be a different type.
The base class need to handle common logic.
The runtime value of the Mode type is held in _modeclass

But all those does not help with type checking.
[I believe] we can solve that with generics, but this seem a bit hard to do that in the above mentioned PRs, so opening an issue to track this.

I think we're at least in part running into https://github.com/python/mypy/issues/3004, and one improvement would be to have the mode property return a Mode instead of str (which I think you may have suggested in one of my PRs)


Add output templating support to input branch

Closes #188
Relates to #180
Summary
Enable templating of output values into the branch input. Include an example of how to use this in the recipes for the /pr Action variant.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5f9baa7) 98.57% compared to head (1eb38e5) 98.57%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #193   +/-   ##
=======================================
  Coverage   98.57%   98.57%
=======================================
  Files           4        4
  Lines         140      140
=======================================
  Hits          138      138
  Misses          2        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


can't get the exact answer of the question exact in faq.json

I have installed bert as a service and action working well and launching the rasa x
but the answer not from a bert model i feel it's random and when I let the model to return any answer of bert i get illogical output

I have tried to run the encode_standard_question() function and it's working well now
Thanks


Make pkgs output *nix friendly

Constraining the output of a package to one line makes it easy to use
tools like grep to search or AWK to get the latest version of a specific
package.
Remove comma separator because space is default separator for most cli
tools that work with columnar data, like AWK.
Example:
rebar3 pkgs | grep hackney | awk '{print $NF}'

Same result without changing output:
rebar3 pkgs | grep -A1 hackney | awk -F, '{print $NF}' | tail -n1

(technically, rebar3 pkgs | grep -A1 hackney | awk '{print $NF}' | tail -n1 works since you want the last field and it is not followed by a comma, so the change in separator is not even needed. tail -n1 drops the name, but even with that it is not too bad)

I don't know? I'm more concerned about readability than the rest, and it sounds that if we need unix tools to read the output of the command, we should probably add a filter to the command such that we could do rebar3 pkgs hackney and have that return a list of versions instead, removing the need for tool usage for such basic reading?

Readability is subjective. I don't know which one is easier to read.
Imagine a hypothetically common workflow :)

User wants to add a particular library (hackney) to the project
User wants to know if that package is available and if so which version of it.
rebar3 pkgs | grep hackney tells me the package is available but not which version of it
Curious. What other information does it have?
rebar3 pkgs Prints a few pages of info. Damn!
rebar3 pkgs | less <space> <space> <space>
Finds the versions manually and adds it config.

This is what happened to me when I was searching for a package (HdrHistogram_erl) and you can guess how I ended up with this patch :)

About rebar3 pkgs hackney. The command is semantically equivalent to grep. Let's write programs to do one thing and do it well :D

A few other outputs for comparison.
$ gem list --local
bigdecimal (1.2.0)
CFPropertyList (2.2.8)
erubis (2.7.0)

$ stack list-dependencies
HTTP 4000.2.20
QuickCheck 2.8.1
array <IP_ADDRESS>

I don't have anything else installed on this machine now. The current output looks like that of stack (haskell package manager)

About rebar3 pkgs hackney. The command is semantically equivalent to grep. Let's write programs to do one thing and do it well :D

grep is not available out of the box on Windows as far as I know, to name one platform. That there are tools to work around inadequacy of other tools is not an excuse to make inadequate tools. I'd rather implement rebar3 pkgs <name> and serve everyone than change the output so people on non-windows platform can get the same result by toying with external tools.

Yea, I've always just done grep -A1 for this :)

@jaseemabid for those examples of npm and stack I think you want rebar3 tree to compare against.

Superceded by #1126


Signature notation

This PR adds a signature notation to the created signatures.

Codecov Report

Merging #43 into master will increase coverage by 0.06%.
The diff coverage is 59.52%.

@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
+ Coverage   59.37%   59.44%   +0.06%
==========================================
  Files          14       41      +27
  Lines         832     4537    +3705
==========================================
+ Hits          494     2697    +2203
- Misses        252     1392    +1140
- Partials       86      448     +362

Impacted Files
Coverage Δ

...endor/golang.org/x/crypto/openpgp/packet/reader.go
0% <0%> (ø)

....org/x/crypto/openpgp/packet/one_pass_signature.go
0% <0%> (ø)

...ndor/golang.org/x/crypto/openpgp/packet/literal.go
0% <0%> (ø)

mail/multipart.go
66.66% <100%> (-0.38%)
:arrow_down:

...r/golang.org/x/crypto/openpgp/packet/compressed.go
15.55% <15.55%> (ø)

...olang.org/x/crypto/openpgp/packet/public_key_v3.go
36.42% <36.42%> (ø)

...olang.org/x/crypto/openpgp/packet/userattribute.go
37.5% <37.5%> (ø)

...golang.org/x/crypto/openpgp/packet/signature_v3.go
42.25% <42.25%> (ø)

...endor/golang.org/x/crypto/openpgp/packet/config.go
45.83% <45.83%> (ø)

...x/crypto/openpgp/packet/symmetrically_encrypted.go
47.48% <47.48%> (ø)

... and 46 more

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 d2d11e4...2144864. Read the comment docs.


Objects are not valid as a React Child

Hey,
I'm trying to support object rather than single value but there is a problem with the array.
https://pastebin.com/P3pR4ds4
error image

fixed by replacing this :
{item}
by this :
{item.label}


connection from authority_view

In principle there should be an easy way to establish a connection (performing name resolution if required) from an authority. However, the URL library should not depend on Asio. Perhaps this could be an example. Or we could devise some trick to make it work or facilitate the operation.

out of scope


Refactor API for use as Python package

Currently the demo code for writing MDF is all at top level in this repo. Ideally it should be made into an installable python package for use elsewhere, e.g. in the code @davidt0x has created for mapping to/from ONNX: https://github.com/ModECI/MDFTests/tree/main/ONNX/onnx-mdf
In the longer term the Python API could have its own repo, but for simplicity as the language is in development, the core spec/examples/api can all stay here.
The simplest name of the package would be "mdf". However... there is a python package already called MDF: https://pypi.org/project/mdf/ with a scope not a million miles from this...
So... call it "modeci-mdf"?

In progress here: https://github.com/ModECI/MDF/tree/refactor

In progress here: https://github.com/ModECI/MDF/tree/refactor

Package now installable


Create back-up script and daily cron job

Here's a sample. Needs to be improved.
#!/bin/bash

# Switch to home directory
cd

# Delete old backups
rm -rv backup/*

# Make new backup directory
bu_date=$(date +%F)
mkdir -p backup/$bu_date

# Backup default region and r directory to backup directory
$gtm_dist/mupip backup DEFAULT backup/$bu_date
tar --exclude '*.o' -czf backup/$bu_date/r.tgz ./r/

# tar into single file and remove directory
tar czvf backup/${bu_date}.tgz backup/${bu_date}
rm -rv backup/$bu_date

Better script
#!/bin/bash

# Switch to home directory
cd /home/vehu/

# Source variables
. /home/vehu/etc/env

# Delete old backups
echo "Deleting old backups..."
rm -rf backup/*

# Get date for file name
bu_date=$(date +%F)

# Make global backup directory
mkdir -p backup/g

# Backup default region
echo "Creating backup of globals..."
$gtm_dist/mupip backup -BKUPDBJNL=OFF DEFAULT backup/g

# Tar and gzip files
echo "Tarring and gzipping routines and globals..."
tar --exclude '*.o' -cf backup/$bu_date.tar ./r/
tar -rf backup/$bu_date.tar -C ./backup/ ./g
if hash pigz 2>/dev/null; then  # Try pigz to speed up gzip, which is the slowest operation here
    echo "Using pigz to gzip..."
    pigz backup/$bu_date.tar
else
    gzip backup/$bu_date.tar
fi

# Delete dat file directory
echo "Deleting gzipped globals backup..."
rm -r backup/g

# Inform the user of the backup directory and size
echo -n "Backed up to /home/vehu/backup/${bu_date}.tar.gz "
echo "Size: " $(stat --printf="%s" /home/vehu/backup/${bu_date}.tar.gz | numfmt --to=si)

# Print file path for scripts
echo /home/vehu/backup/${bu_date}.tar.gz

that better script looks good here. the command line arg -BKUPDBJNL=OFF to mupip backup I'm not 100% familiar with off the top of my head, but the general pattern looks good

only other thing is maybe don't delete the old backup before the new one is taken or at least allow it to have n number on file

Better version
#!/bin/bash

# Exit on error
set -e

# Switch to home directory
cd /home/vehu/

# Source variables
. /home/vehu/etc/env

# Get date for file name
bu_date=$(date +%F)

# Remove today's backup if it exists
rm -f backup/$bu_date.tar.gz

# Make global backup directory, rm old contents just in case
mkdir -p backup/g
rm -f backup/g/*

# Backup default region and turn off journaling
echo "Creating backup of globals..."
$gtm_dist/mupip backup -BKUPDBJNL=OFF DEFAULT backup/g

# Tar and gzip files
echo "Tarring and gzipping routines and globals..."
tar --exclude '*.o' -cf backup/$bu_date.tar ./r/
tar -rf backup/$bu_date.tar -C ./backup/ ./g
if hash pigz 2>/dev/null; then  # Try pigz to speed up gzip, which is the slowest operation here
    echo "Using pigz to gzip..."
    pigz backup/$bu_date.tar
else
    gzip backup/$bu_date.tar
fi

# Delete dat file directory
echo "Deleting gzipped globals backup..."
rm -r backup/g

# Delete old backups
echo "Deleting old backups..."
mkdir backup/tmp
mv backup/$bu_date.tar.gz backup/tmp
rm -f backup/*.tar.gz
mv backup/tmp/$bu_date.tar.gz  backup/
rmdir backup/tmp

# Inform the user of the backup directory and size
echo -n "Backed up to /home/vehu/backup/${bu_date}.tar.gz "
echo "Size: " $(stat --printf="%s" /home/vehu/backup/${bu_date}.tar.gz | numfmt --to=si)

# Print file path for scripts
readlink -f /home/vehu/backup/${bu_date}.tar.gz


cmd/covdata: short read on string table when merging coverage counters

What version of Go are you using (go version)?

$ go version
go version go1.20 darwin/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/yihuang/Library/Caches/go-build"
GOENV="/Users/yihuang/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/yihuang/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/yihuang/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/nix/store/0m3v5yn7s5ipvzp7p462pvjknfcn8c6s-go-1.20/share/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/nix/store/0m3v5yn7s5ipvzp7p462pvjknfcn8c6s-go-1.20/share/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.20"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/yihuang/src/cronos/go.mod"
GOWORK=""
CGO_CFLAGS="-I/nix/store/qg6n4gdcccfd93lnih1yh8a894ya6akr-rocksdb-7.9.2/include"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-L/nix/store/qg6n4gdcccfd93lnih1yh8a894ya6akr-rocksdb-7.9.2/lib -lrocksdb"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/3_/dt3cb8952_12y960nh8fhqx40000gn/T/go-build3007670481=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Use -cover to collect coverage report in integration tests.
What did you expect to see?
go tool covdata should be able to process generated file succesfully.
What did you see instead?
$ go tool covdata merge -i integration_tests/coverage -o profile.txt
error: error: reading counter data file integration_tests/coverage/covcounters.627546bfc793cab3f06764545dba1bd1.91113.1675846217322297000: error: short read on string table

The tarball of generated coverage directory are attached, it can reproduced stably, both in CI(linux) and laptop(macOS).
coverage.tar.gz

(attn @thanm; CC @golang/compiler)

Thanks, I will take a look.

Thank you for sending this report, it is an interesting bug in the coverage profile reader, specifically the portion that reads counter data files.  It appears that the bug is only tickled when you have a small counter data file (large ones will be mmap'd) that has a string table larged than the bufio.Reader 4k buffer size (which might explain why I didn't hit it previously in my testing).
Sending a CL with a fix shortly.  I will also ask the release team if the fix can be back-ported to Go 1.20.1

@gopherbot please backport to 1.20

I have started the process to get this approved for backport to 1.20, however I think it will have to wait until the March minor release (too late for the Feb cycle), just FYI.


Add WTFPL license

@nikitavoloboev

There's so many of them 😄

There's so many of them

Totally!


Pancake Sort

Is your feature request related to a problem? Please describe.
It's a good solution to solve the mathematical problems. It is an important sorting technique using flip operation
analogous to flipping pancakes.
Do you want to work on it
Yes, I want to work on it in C++

@plazzy99 @rudrakshi99 @nakul-19 @raksha009 @rish-singhal @Rohan27s @yasharth291 please assign this issue to me

Please assign this to me in Python

#1059