Merge in version bump and release notes from Bitcoin Core

We should re-create the man pages ourselves before release, but this means we're synced with BC 0.14.3 at least.

We should re-create the man pages ourselves before release, but this means we're synced with BC 0.14.3 at least.

Are we though? There is no bitcoin source pulled into 1.14.3 except this.

We should re-create the man pages ourselves before release, but this means we're synced with BC 0.14.3 at least.

Are we though? There is no bitcoin source pulled into 1.14.3 except this.

There is - our 1.14.2 was rebased on BC 0.14.3. Check the history, their 0.14.3 commits are mostly in, apart from these three.

Closed to replace the one fix we need with a stand-alone commit


Outdated content

After the point of entering in the configuration tab does not reflect the actual option that is available.

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

ID: 7c5c94a5-2548-b3a2-3a43-d52bc0892227
Version Independent ID: 01e9c2bc-3265-447c-5d0f-488039c60e87
Content: Tutorial: Create an application gateway with URL path-based routing rules using Azure portal
Content Source: articles/application-gateway/create-url-route-portal.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@victor-care
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@victor-care
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly
@greg-lindsay
Can you please check and add your comments on this doc update request as applicable.

The UI changed since the article was written. I will update.
Closing this request and moving to internal work item 212799
#please-close


feat: 385 cbioportal export wrapper missing information

coverage: 85.323% (+0.3%) from 85.033% when pulling d48f573084b570e90be73691ce2a1720f6f1e9a7 on 385-cbioportal-export-wrapper-missing-information into 706a9f561fe96a9469fec15f232940b265266e8e on main.


Added support for encrypted pvc

rest of the changes looks good. can you please add docs in docs folder discussion how about how to generate the key.
You could take reference from this https://github.com/simplyblock-io/sbcli/blob/main/docs/create_crypto_lvol.md

Alright will do that


how to extract all landmarks from Holistic ios

how to get the land marks from the Holisticgpuapp.
i have built the app as shown in the Documentation

hi @mohammedbalegh, did you manage to find a way to extract all the KPs within every frame?


make:event & make:listener allows file overwrite

Laravel Version: 5.5.14
PHP Version: 7.2
Database Driver & Version:

Description:
make:event and make:listener allows file overwrite which kind of dangerous to do especially on listeners where some logic can be added. Base on the code, it checks existence based on class_exists instead of checking if the file exists (as the default behavior of the Illuminate\Console\GeneratorCommand).
https://github.com/laravel/framework/blob/5.5/src/Illuminate/Foundation/Console/EventMakeCommand.php#L36
Is there a reason why it was designed that way?
Steps To Reproduce:
Run php artisan make:event SampleEvent twice (or more). No error/warning is shown

Use git.

@Kyslik whether you're using a VCS or not, this feature should run a confirmation check if the file exists.

I actually see that with other commands it errors out. ($ art make model), which is interesting since whoever implemented event command did not notice.
Nice catch :)

Despite all above, I think on dev env it should overwrite, and git would solve everything else (same like migrations have different behaviour dependent on env).

Why would you run make:model or anything of make on an env other than dev?

@Dylan-DPC Well, exactly NEVER.

I don't see what git has to do with this issue.

My first comment on was meant like this: I wouldn't worry about it, disaster is prevented by using git.

I think that issue here is that, whether you're on dev env or not, any file overwrite should at least warn you and not just plainly succeed. Besides, the default behavior of the GeneratorCommand (the parent class of ListenerMakeCommand and EventMakeCommand) is to check if the file already exists and abort if that is the case. And the use of class_exists prevents the alreadyExists method to function correctly.

@edsamonte feel free to open a PR, your concern is valid imo.

Made a PR here. It seems that make:exception also overrides the alreadyExists method. It works the same way as the parent one so I deleted it still. GeneratorCommand::alreadyExists looks much safer.


setRepresentedFilename is called even when file is unreadable

The function loadFile() calls setRepresentedFilename() even when the requested file turns out to be unreadable. This leads to the window having a title that represents a file not actually open.

Nice catch!


Addition#1

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Shall I work on this

can you assign the task to me?

Closing as it's duplicate issue.


fixed alias3 contains error

Description
alias3 gets removed as per instructions from sample_policy_alias_action.json. In the final contains verification we do not need to check for alias3.
Issues Resolved
#911
Check List

[x] Commits are signed per the DCO using --signoff
[x] Tested on Cypress for policies_spec.js

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@kohinoor98 the policies spec test is passing, but looks like a new test started failing. Could you look into whether that's related to your changes?

Hi @AWSHurneyt,
The test case this PR is failing is flaky as tagged in #910 and the proposed solution is also created for it #920. Please guide on next steps.
Thanks,
KC

Hi @AWSHurneyt,
The test case this PR is failing is flaky as tagged in #910 and the proposed solution is also created for it #920. Please guide on next steps.
Thanks, KC

Ah thanks for that context. Approved.


 However, most likely an m(1,5) capturing a 6 would become m(6,6). But what happens if an m(1,5) captures a (2,4)?
Depending on that special case, it seems like this shouldn't be hard to support.  In BMC_Die::OnApplyAttackPlayer() there is this code:
	if (HasProperty(BME_PROPERTY_MORPHING))
	{
...
		m_sides_max = m_sides[0] = target->GetDie(_move.m_target)->GetSidesMax();
...
	}

In which case it would look like:
   m_sides_max = 0;
   for (int i=0; i<Dice(); i++)
      m_sides_max += m_sides[i] = target->GetDie(...)

Also, at least one BM_ASSERT and BM_ERROR to remove.
SECONDARY ISSUE
I see a couple of other Morphing combination restrictions (Turbo, Speed) that may need to be allowed.

IIRC, a morphing twin die doesn't behave any differently from a non-twin morphing die: the morphing die twins and un-twins as needed.

@jl8e so m(1,5) capturing a 6 would become m(6). m(1,5) captures a (2,4) and becomes m(2,5)
Is that right ?

yes

other areas this is discussed. specifically interested in the Turbo aspect
https://www.buttonweavers.com/ui/forum.html#!threadId=991
https://github.com/buttonmen-dev/buttonmen/issues/2676

we have tests that prove Morphing Twin is working with #69


Scroll not working

Hi,
first, I have installed Redoc from cdn. Alsto I installed gem https://github.com/krsyoung/redoc-rails on Rails 4 with Turbolinks 5. In both tries everything works except the scroll after the click on the menu, or menu highlighting menu items when scrolling.
Can you assist how to proceed? Thanks.

How can I reproduce this? could you create a minimal reproducible sample?
Would be great if you created a repo with all the setup.
@krsyoung, maybe you can help out.
@FrantisekVrab try also the new version of redoc-rails with updated ReDoc. Maybe the issue will be resolved.

@FrantisekVrab did you solve the issue?

Hi, I have also used a redoc-rails, however with no success. There was no error message.
Later I found that problem is probably that we are using a jQuery, jQuery-rails and turbolinks on the site an that's in conflict with the Angular that redoc is build on. I wasn't able to debug this.
So I have moved to the non-rails application and everything works.

Thanks for the update! Closing this issue.

In case somebody's still in trouble:
html {     scroll-behavior: smooth; }
broke scrolling of Redoc document for me.

Remove
html{
scroll-behavior: smooth;
}
and you are good to go.


Line-breaks in topic titles causing builds to fail

Trailing line breaks within topic <title> elements of cause builds to fail when the search result previews are generated (at taskname gen-preview-json). This appears to be when converting the title contents from text node to attribute value in merge-data.xsl.
Simply wrapping the existing xpath in a normalize-space() function resolves the issue; at least for leading or trailing line breaks.

@super-structure Thanks for reporting this and proposing the fix.
I can't seem to reproduce the issue. I tested with a long topic title that contained multiple line breaks and everything still worked fine:

Can you provide more details on the circumstances that cause the problem?

@infotexture Sure! I narrowed down the issue to the following topic title element:
	 </title>```
I just noticed that there is a line feed, tab, and trailing space in that title. I had suspected it was the line feed, though it could be any combination of those I suppose.

I couldn't reproduce the error at first, but that is because I was editing the DITA Bootstrap sample like @infotexture which includes an .editorconfig like this:
indent_size = 2
indent_style = space
insert_final_newline = true
trim_trailing_whitespace = true

Now when I deliberately remove that and deliberate change the title I can reproduce the error, and normalize-space() seems to fix it since by definition  normalize-space() :

Removes all leading spaces.
Removes all trailing spaces.
Replaces any group of consecutive whitespace characters with a single space.

Which will remove any \t or \r\n  nonsense from the Title in the JSON.

@jason-fox Thanks for reviewing and merging.
It's easy to forget that .editorconfig prevents stuff like this from creeping in, and a good reminder that linting tools are incredibly useful. But not everyone uses them, so certainly make sense to ship this fix. 👍


Trying to install dependencies using npm throws error

I'm trying to install dependencies using the node package manager (npm) but it throws an error in the middle of installation and stops further installation. I went through this quick start it says to install dependencies using yarn package manager but I thought of installing it via npm!
This is the snap of the error I got,

Node version is 10.16.3
NPM version is 6.9.0
OS Ubuntu 18.04.3 LTS

It could be great if we have both npm and yarn package manager support

Sorry but there is no advantage in supporting both dependency managers.


Integrate DXP Component: App Version Info on the Settings Page

What is the motivation for adding/enhancing this feature?
IntegratE of the DXP component "App Version Info" into the Settings page of the application. This addition will provide users with essential information about the version of the application they are currently using.
What are the acceptance criteria?

[ ] ...

Can you complete this feature request by yourself?

[ ] YES
[ ] NO

Additional information

Assign this issue to me?

Sure @shraddha761 go ahead

Verified and working fine in v1.18.0.
https://jam.dev/c/4c7165a7-346e-4e5e-b14d-b321c69220c2


Change background color and text color on image_picker 0.6.4

Hiii,
I need help for plugin image_picker 0.6.4, how to change AppBar background color and text color when i pick gallery : ImagePicker.pickImage(ImageSource.gallery)
The background color is always blue and text color is always black at the top of image picker

Thank you and best regards 🙏

Hi @gushendra86
This platform is not meant for assistance on personal code.
Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Move dataset.repository to lab.

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Please tag this pr with at least one of these labels to make our release process faster: BREAKING, new feature, bug fix, other change, dev setup

Something I want to achieve is to separate the library stuff from more interactive utilities. Things that belong to lib should be happily imported across different gluon-ts modules, but stuff that is lab is only used by end-users and in tests.
I think for lib longer import paths can be a good thing, since they are more precise about where things come from.
However, for lab we could be more aggressive and do things like:
from gluonts.lab import get_dataset


Bold heading

I missed this section of the doc and my VM build failed
Please add a bold heading for this section
"Grant Tenant 1 App SP permissions in Tenant 2 resource group"
Select the resource group and then select Access control (IAM). Under Add role assignment select Add.
Under Role, type Contributor.
Under Assign access to:, leave this as Azure AD user, group, or service principal.
Under Select type myGalleryApp then select it when it shows up in the list. When you are done, select Save.

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

ID: a777eea0-acf4-6f1d-9e56-e6069dddc577
Version Independent ID: d858a0bd-ec1c-4e32-e1d1-784feecd57c0
Content: Share gallery images across tenants in Azure
Content Source: articles/virtual-machines/linux/share-images-across-tenants.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

@DeanCefola looking at the portion you mentioned, it is part of the larger section so I am not seeing it be easily missed. It also aligns with the formatting of the rest of the section. So not sure it makes sense to add a bold title for that part.
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/share-images-across-tenants#create-the-app-registration
@cynthn thoughts on this?

I read the doc several times and was failing to complete the VM build because I incorrectly thought that the code section below where I register Tenant 2 was all the permissions I needed.
I missed this section until I was on the phone with Akshay Joshi trying to get it to work
We figured it out as we were talking, and then he looked at the doc, and found this section and pointed it out to me.
I totally missed it
Perhaps it is because the code block above or the purple block below, but that section is not visually clear
So I suggest, even if it is against the formatting that something be done to accentuate that section so others don’t gloss over it like I did
Dean Cefola
Sr. FastTrack Engineer
Microsoft Customer Experience MCP: Azure Solutions Architect
[MSFT_logo_Gray DE sized SIG1.png]
[cid:image004.png@01D50686.7714E4E0]https://www.youtube.com/AzureAcademy
Office:<PHONE_NUMBER> x4036
Mobile:<PHONE_NUMBER>
Dean.Cefola<EMAIL_ADDRESS>[ArcSol]
If you have any feedback about my work, please let either me or my manager know – Michelle Swann-Renee, Michelle.SwannRenee<EMAIL_ADDRESS>From: Micah<EMAIL_ADDRESS>Sent: Thursday, May 9, 2019 4:30 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Dean Cefola<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Bold heading (#30949)
@DeanCefolahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FDeanCefola&data=01|01|dean.cefola%40microsoft.com|fea7cbc1135e441c3d7c08d6d4bd0fab|72f988bf86f141af91ab2d7cd011db47|1&sdata=qh5oOQzJyRd28EkVJ6Jmdds6X5Hqb1oIhZu65CDo3js%3D&reserved=0 looking at the portion you mentioned, it is part of the larger section so I am not seeing it be easily missed. It also aligns with the formatting of the rest of the section. So not sure it makes sense to add a bold title for that part.
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/share-images-across-tenants#create-the-app-registrationhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Fvirtual-machines%2Flinux%2Fshare-images-across-tenants%23create-the-app-registration&data=01|01|dean.cefola%40microsoft.com|fea7cbc1135e441c3d7c08d6d4bd0fab|72f988bf86f141af91ab2d7cd011db47|1&sdata=kbmiOSjfVYgX2VywCvdPiBxCd0%2F3C6VaRO82Cg4x68c%3D&reserved=0
@cynthnhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcynthn&data=01|01|dean.cefola%40microsoft.com|fea7cbc1135e441c3d7c08d6d4bd0fab|72f988bf86f141af91ab2d7cd011db47|1&sdata=0pCkDqlO7aK01jI89mqTFIGuG7gd1qqR0v0g0XKiTdk%3D&reserved=0 thoughts on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F30949%23issuecomment-491054314&data=01|01|dean.cefola%40microsoft.com|fea7cbc1135e441c3d7c08d6d4bd0fab|72f988bf86f141af91ab2d7cd011db47|1&sdata=KWbUi6%2FMOM1eb6pxad9mknBaLzZeo22vDzGZx0xSrPk%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKEU6QCTOEGYZV5VHOE6P3DPUSCTFANCNFSM4HL5SOJA&data=01|01|dean.cefola%40microsoft.com|fea7cbc1135e441c3d7c08d6d4bd0fab|72f988bf86f141af91ab2d7cd011db47|1&sdata=mpnRp1%2FTwM7DPWzBNfwx2okB2ShPZG657bJaQeELSug%3D&reserved=0.

@DeanCefola makes sense. We will take a look and see what I can do to make it easier to read

@cynthn It looks like the section @DeanCefola is referring to is actually a shared section between other docs. So I don't want to change anything that will impact other docs. Hence, assigning to you to take a look and see what you all want to do to move this forward.

@DeanCefola and @MicahMcKittrick-MSFT There is a point where you transition over to working with Tenant 2, which seems like a good place for a heading. Not sure why I didn't have one there originally, but I've got a PR filed to add one in :)
Thanks for the feedback!
#please-close


[Q] About SRT

Hi,
I have some questions about SRT in TSDuck. Please, help to found the correct response:

The TSDuck, when compiled with the SRT input/output modules (tsplugin_srt.so exists), is compiled with the library statically o it uses the libsrt.so installed in the system? I don't know how to select static or dynamic linking of this library when compiling TSDuck.
How to know the version of the library used inside TSDuck? I feel the help of the module will need to print the version, but I don't see it.
When using the SRT as the input for the TSDuck chain, but requiring to start the connection to a remote server (the "caller" mode with the "srt-live-transmit" tool), which parameters I need to use? I can't found the solution in the documentation, as it indicates only the "listener mode" for the input and the "caller mode" for the output.

If anyone can answer, I would be very grateful to him.

TSDuck is built with whatever form of SRT is present on the system on which it is built.
On Linux and macOS where SRT comes as a binary package from the official repos, it is built against the shareable library by default, as result of the -lsrt option. Using the -static option will probably try to use libsrt.a if present on the system.
Using the lastest version on Fedora:
$ ldd /usr/lib64/libtsduck.so | grep srt
	libsrt.so.1 => /lib64/libsrt.so.1 (0x00007f071c425000)

On Windows, to avoid the usual DLL hell of open source libraries, SRT is linked statically.
The version of libsrt which is used can be seen as follow:
$ tsp --version=all
tsp: TSDuck - The MPEG Transport Stream Toolkit - version 3.24-2109
Built Dec  4 2020 - 22:43:22
Using GCC 10.2.1, C++ std 2011.03
Web library: libcurl: 7.71.1, ssl: OpenSSL/1.1.1h-fips, libz: 1.2.11
SRT library: libsrt version 1.4.1
Dektec: DTAPI: <IP_ADDRESS>

Concerning how to use SRT connections, I let others respond. I am not an expert in SRT.  I am just integrating it and make sure it compiles on new versions.

Hi @lelegard ,
Thank you! I'll use the cmd tsp --version=all. It's the command that I searching for it.
Regarding the use of the SRT plugins inside TSDuck I hope @spnngl want to comment something.
Regards.

Hi
the srt implementation in tsduck for now supports only listener mode for input and caller mode for output as documented in https://tsduck.io/download/docs/tsduck.pdf
So if you want to transmit something with caller mode use tsduck srt output plugin.
A third mode is also available, rendezvous, which kinda mix both of previous ones.

Hi @spnngl ,

the srt implementation in tsduck for now supports only listener mode for input and caller mode for output
So if you want to transmit something with caller mode use tsduck srt output plugin.
A third mode is also available, rendezvous, which kinda mix both of previous ones.

What I personally want to do with SRT and TSDuck is operate in this scenario:
In a central network point (the datacenter) with full Internet connection , I want to run a srt-live-transmit daemon with TWO open end-points listening. One end-point is the INPUT stream and the second is the OUTPUT stream. So the idea is that a "remote" TSDuck sends to the central datacenter; while another second "ingest receiver" TSDuck receives from the central datacenter. This architecture is to enforce strict security policies, as only TSDuck clients can connect to the SRT listener server in the central point. However, in this case one TSDuck uses the SRT output module for sending. And that's ok because it uses the caller mode. But for the other, it needs to connect to the central server using the SRT input module, but in this case with the caller mode too.
Any solution for this environment?
Thank you!

It is possible. Most of the intelligence of SRT in tsduck is in tsSRTSocket.{h,cpp}, you'd have just to add support for missing modes in tsSRT{In,Out}putPlugin and add an input --mode to those plugins.
PS: I inverted modes in my previous response, tsSRTOutputPlugin supports listener & rendezvous, tsSRTInputPlugin supports caller & rendezvous

Hi @spnngl ,
Thank you for the explanation. I hope someone will update then the tsduck SRT plugins to support all three modes.
Regards.

I hope someone will update then the tsduck SRT plugins to support all three modes.

I did... The input and output srt plugins now use the same options: --caller and --listener. Specifying both means rendezvous mode. The previous parameters and --rendezvous option are still there for compatibility but no longer documented in the user's guide.
Please test this, I have not tested everything.


fail to compile mycasbin.go as *Logger does not implement "github.com/casbin/casbin/v2/log".Logger (missing Print method)

mycasbin.go:45:16: cannot use &Logger{} (type *Logger) as type "github.com/casbin/casbin/v2/log".Logger in argument to "github.com/casbin/casbin/v2/log".SetLogger:
        *Logger does not implement "github.com/casbin/casbin/v2/log".Logger (missing Print method)

@abingcbc

@jackliusr Your version of Casbin is too old. Since v2.15.0, github.com/casbin/casbin/v2/log.Logger has already removed Print method.

@Abingcbc, thanks.  You are right. I used a old version casbin in my bazel dependency files.


I have a problem with the audio metering.

Hi bro, my app is using this lib, and I used the audio metering in the lib to describe sound effects when recording. But I found that when I spoke out loud, the peakPower or averagePower level did not change much compared to the silent moment. I want to ask if peakPower and averagePower are volume-related functions when recording or is it something else. Or do you have any good suggestions for describing the volume level when recording like Ok Google? Thanks for your reading.

@huynhphu-pufu
have you got any solution?
I want to show sound effects when audio recording. but for that i want list of peakPower.

@anup-geeky For me, I need to record and recognize spoken English word, so I used a plugin, which paid me the volume level when I spoke and realized the English word I just said. It is speech_to_text. As for you, I don't know what is your needs, but you can try the plugin I just mentioned or flutter_sound.

@huynhphu-pufu  but for my case , flutter_sound only support API 24+ :(

@ariefwijaya yeah, my PM also wants my apps support at least API 21, so I used speech_to_text for my purpose.

@anup-geeky For me, I need to record and recognize spoken English word, so I used a plugin, which paid me the volume level when I spoke and realized the English word I just said. It is speech_to_text. As for you, I don't know what is your needs, but you can try the plugin I just mentioned or flutter_sound.

Can you please tell me how to do the recording audio and speech recognition at the same time? I used audio_recorder and flutter_speech packages but the iOS app sometimes crashed and on Android it doesn't support audio recording while doing speech recognition.


Added Istituto Superiore Guglielmo Marconi

@vincyxiroff I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host ismarconi.edu.lt not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


More irrep_mats, changes to O and Oh symels, rotate mol rather than symels in symtext

Codecov Report

Merging #22 (4d2203e) into symp (0d5fad5) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             symp      #22      +/-   ##
==========================================
- Coverage   59.90%   59.76%   -0.15%
==========================================
  Files          15       15
  Lines        1933     1926       -7
==========================================
- Hits         1158     1151       -7
  Misses        775      775

Impacted Files
Coverage Δ

src/Symmetry/CharacterTables/IrrepMats.jl
0.00% <0.00%> (ø)

src/Symmetry/CharacterTables/Main.jl
29.77% <0.00%> (ø)

...c/Symmetry/CharacterTables/PointGroupGenerators.jl
24.91% <0.00%> (ø)

src/Symmetry/symmetry_finding.jl
90.62% <0.00%> (-0.03%)
:arrow_down:

src/Properties.jl
100.00% <0.00%> (ø)

src/Transformations.jl
100.00% <0.00%> (ø)

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


Build error: Missing 'global' property of ja_JP in ng-zorro-antd/i18n

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-start-vwdx81
Steps to reproduce
Argument of type '{ locale: string; Pagination: { items_per_page: string; jump_to: string; jump_to_confirm: string; page: string; prev_page: string; next_page: string; prev_5: string; next_5: string; prev_3: string; next_3: string; }; ... 8 more ...; Empty: { ...; }; }' is not assignable to parameter of type 'NzI18nInterface'.
Property 'global' is missing in type '{ locale: string; Pagination: { items_per_page: string; jump_to: string; jump_to_confirm: string; page: string; prev_page: string; next_page: string; prev_5: string; next_5: string; prev_3: string; next_3: string; }; ... 8 more ...; Empty: { ...; }; }' but required in type 'NzI18nInterface'.ts(2345)
nz-i18n.interface.d.ts(69, 5): 'global' is declared here.
What is expected?
Fulfill ja_JP to build successfully
What is actually happening?
Build error,
Property 'global' is missing in type '{ locale: string; Pagination: { items_per_page: string; jump_to: string; jump_to_confirm: string; page: string; prev_page: string; next_page: string; prev_5: string; next_5: string; prev_3: string; next_3: string; }; ... 8 more ...; Empty: { ...; }; }' but required in type 'NzI18nInterface'.

Environment
Info

ng-zorro-antd
8.2.1

Browser
Chrome, VS code

Hi. This is already fixed in https://github.com/NG-ZORRO/ng-zorro-antd/pull/4102.
Are you interested in making a PR to add translations for Japanese?

I am closing this now. If you feel interested you can just submit a PR. Thank you!


Install instructions for Android and Linux

Some easy install instruction links for Android and Linux.

Well, I put it into the Wiki since I put most of the relevant information into it.
You can see the added page here.
But I'll add a link to it on the README later.

The installation guide for android are outdated now

The installation guide for android is outdated now

What exactly is outdated?
I haven't tried installing it manually in a long time (as I don't use any device with an unlocked bootloader anymore) and I have never tried that Magisk thing 😅

The installation guide for android is outdated now

What exactly is outdated?
I haven't tried installing it manually in a long time (as I don't use any device with an unlocked bootloader anymore) and I have never tried that Magisk thing 😅

Sorry, the Magisk part is outdated, I haven't tried the manual method. The repo linked under the Magisk instructions is obsolete (it links to another repo which is also obsolete)

The installation guide for android is outdated now

What exactly is outdated?
I haven't tried installing it manually in a long time (as I don't use any device with an unlocked bootloader anymore) and I have never tried that Magisk thing 😅

Sorry, the Magisk part is outdated, I haven't tried the manual method. The repo linked under the Magisk instructions is obsolete (it links to another repo which is also obsolete)

Okay, I've added that to the Wiki.


JSLIGO: update syntax, content

Mdx integration is badly done. Please check the Readme to integrate it correctly


add links to navigate between pages

Sometimes I would read document on mobile phone and navigating between pages are not an easy task when you have to open sidebar menu first. This pull request adds links at the bottom of page for navigating to previous/next page, and I think it would improve users' experience.
Screenshot:

Great addition, please revert formatting changes, we haven't decide about it and it is just noise in this PR.

@montogeek reverted.

Thanks! Looking forward to hear positive feedback from the readers :)


feat(minifier): fold void 1 -> void 0

#5670  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite


[tutorial] Reactions update is broken in

Steps To Reproduce

Click "View Post" for any post
Click the reactions

Link to code example: The codesandbox here
The current behavior
Reactions work for the post list page, not for the single post page.
The expected behavior

Reactions work for single post page
Reactions sync in both list list page and single post page.

I did an update on apiSlice.js and everything is workind fine:
The codesandbox here

@markerikson How I can update the codesandbox example?

@dannielss It's... kinda complicated :)
All the sandboxes live in https://github.com/reduxjs/redux-essentials-example-app .  Specifically, it's the tutorial-steps branch, with tags applied to specific commits.
To update the sandboxes, you'd actually have to:

Clone the repo
Check out the tutorial-steps branch
Reset the branch all the way to the last commit that added/changed these lines
Alter the commit
Cherry-pick all commits after that
Change all tags to point to the new commits in the branch
Force-push it

So, unfortunately not easy, and not something other folks can do atm either

I started learning redux using docs but unfortunately sandbox examples are broken

@MubashirWaheed : what parts of the sandboxes are broken, specifically?

Code sandbox for the "basic redux dataflow" under the redux essential is not working on firefox but I just checked and it is working fine chrome


add support for decimal

This PR adds support for the Decimal datatype.
I followed the same approach as the contribution to superjson to just call toJSON for serialization.
Also removed a rogue .only on the typed tests so they all run.
Let me know if there's anything you want changed :)

Thanks for the PR, but the main point of this package is its small size. That decimal.js package is huge.
I'm adding an API to support custom types. See #30 for details.


Resumable transformations

add saveStateToFile and resumeTransformation functions which save the internal state of a transformation to a sqlite database and allow you to create a new transformater instance from such a file.
add a disabled, locally-tested, "gauntlet" which simulates crashing in transformer APIs and especially native db methods, measuring the speed with which a resumed transformation is finished. This will be added to future transformer performance tests as enabled
some fixes to false positives in assertIdentityTransform, as well as a way to use it with any id remapping

allow deepEqualWithFPTolerance to consider undefined fields as equal to non-existent fields. Useful for some JSON property equivalence testing

there is a corresponding native PR

When someone has a chance, could I get a review from @iTwin/itwinjs-core-admins before merging? (and same for the corresponding native PR, ask me for a link)

oops merged master instead of imodel02, reverting


changes version of aca-py to 5.6

Signed-off-by: Cam Parra<EMAIL_ADDRESS>

lets also update:  https://github.com/kiva/protocol-aries/blob/master/README.md

lets also update:  https://github.com/kiva/protocol-aries/blob/master/README.md


Updated code example

I fixed the import in the example code.
Documentation was updated in a previous merge request but the code example were not.

I'm not sure how to make the import work properly with the project.
There is still an issue with the import name in the example tho.


[itk] latest version

I want to update my itk version from 5.1.0 to 5.3.0.  But it seems that vcpkg doesn't suppot this function. Is it feasible to update the port or I can do it by other ways?

I found relevant info in version part in documentation


apps x509: restrict CAkeyform option to OPT_FMT_PDE [1.1.1]

CAkeyform may be set to PEM, DER or ENGINE, but the current options
are not using the proper optionformat 'E' (OPT_FMT_PDE) for this.
Set the valtype for CAkeyform to 'E' and use OPT_FMT_PDE when extracting
the option value.
This amends bf4006a6f9 ("Fix regression on x509 keyform argument") which
did the same thing for keyform and changed the manpage synopsis entries
for both keyform and CAkeyform but did not change the option section.
Hence, change the option section.
Backports  #11085.
Checklist

[x] documentation is added or updated

I still agree with CLA: trivial.

I guess the AppVeyor pipeline is stuck somehow and guess openssl-machine will never come around and add a "ready for merge"

Strange... the details page shows that all jobs succeeded.

I guess the AppVeyor pipeline is stuck somehow and guess openssl-machine will never come around and add a "ready for merge"

The openssl-machine just a friendly reminder and we are not obliged to wait for it. But I'll let @t8m do the merge, because he self-assigned this pr.

Merged to 1.1.1 as 3bd75cfafd


feat(PropertyOperator): an new implementation of jdk MethodHandler based on method handles for PropertyOperator (GitHub #132)

Codecov Report
Patch coverage: 55.17% and project coverage change: -0.33% :warning:

Comparison is base (0a6cee5) 91.65% compared to head (70fa99c) 91.33%.
Report is 1 commits behind head on dev.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev     #160      +/-   ##
============================================
- Coverage     91.65%   91.33%   -0.33%
- Complexity     1368     1374       +6
============================================
  Files           135      136       +1
  Lines          3248     3277      +29
  Branches        309      312       +3
============================================
+ Hits           2977     2993      +16
- Misses          144      155      +11
- Partials        127      129       +2

Files Changed
Coverage Δ

.../support/reflect/MethodHandlePropertyOperator.java
55.17% <55.17%> (ø)

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


Non Websocket Support

I am having issues connecting to a Meteor site that isn't using Websockets because of cloud flare. Is there support for Meteor apps like this?

Currently no.
But I hope it's possible to directly connect with your app and do the load
via websockets.
On Sat, Jun 27, 2015 at 7:26 AM Patrick Lewis<EMAIL_ADDRESS>wrote:

I am having issues connecting to a Meteor site that isn't using Websockets
because of cloud flare. Is there support for Meteor apps like this?
—
Reply to this email directly or view it on GitHub
https://github.com/meteorhacks/meteor-down/issues/12.

+1 thank you. I didnt think to try and directly connect via the IP.


fix doc of change-block-size

Signed-off-by: cyclinder<EMAIL_ADDRESS>Description
/kind bug

fix error format of change-block-size.md, refer to https://projectcalico.docs.tigera.io/networking/change-block-size
Related issues/PRs

https://github.com/projectcalico/calico/issues/6152
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

NONE

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

maybe need to cherry-pick to other release


Remove deprecation documentation check

Purpose
$subject
Fixes #21690
Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closing this PR because these changes are push to another branch via #21699


osbuild: Change default value of --store to just .osbuild

This commit changes the default value of --store argument from
.osbuild/store to just .osbuild. The reason is just to make the path
shorter.

Merging despite packit failure. It's known to be broken.


HHH-10999 Array datatypes module

What works:

Using single-dimensional Object arrays in Entities (no primitive arrays).
Saving arrays through entities through the EntityManager interface.
Loading arrays into entities through the EntityManager interface.
Tested under PostgreSQL.

What doesn't work:

Table creation without setting columnDefinition on the javax.persistence.Column annotation.
Passing null in the position of an array in manually written queries (bug is in Postgre server, not Hibernate).
Using arrays of primitives, which are optional for JDBC drivers, even when arrays are supported.

TODO list without changes to core:

Test more types
Determine which types should be contributed, and which should be optional.
Test on HyperSQL (only other database listed in the test suite that supports arrays).
Annotate the tests to only use compatible dialects or features

TODO list with core changes:

Add dialect feature checks for arrays to use for testing.
Change StandardTableExporter to recognize and use arrays when desired.
Use option to determine to use legacy model (default) or new array model.

Thanks for the Pull Request. Did you sign the CLA?

Good work. For PostgreSQL, it works nicely. I tried it also on H2, Oracle, SQL Server and MySQL and the main issue that the JDBC drivers do not implement the createArrayOf method.
Since we are reworking the types for 6.0, we need to take into consideration arrays as well, but first, we need to see how to address them in a generic fashion.
If we are to add support for ARRAYS. we need to cover most database systems in order to add the TypeDescriptors in the generic package.

Unless it has changed and invalidated old signatures in the past 8 months or so, I have signed it.
But Oracle, MSSQL and MySQL do not support have native array types, unless they maybe they internally convert it to text, blobs or something like that.
I've thought of several bugs in the current PR, relating to multi-dimensional javatype registration and that of primitive arrays, but their time will probably come with 6.0.
Meanwhile, we can iron this out as an optional module for 5.x.
As of right now the only potential bug in available functionality, that I can think of, is maybe the Mutability plan is wrong. Btw, shouldn't it really be called Immutability plan?

I made some changes to the tests to be compatible with HyperSQL and discovered bugs that need addressing. Currently three tests fail and it's not the fault of the tests.
Datatypes using the java.time package can only be used in Postgres JDBC driver 9.4.1208 or newer. Some conversion would obviously be required and the current Generic solution is not sufficient to handle their conversion to java.sql types that are more compatible.

Currently known problems/tasks:

ArrayTypes misuses LiteralType.
Java 8 java.time types must be converted to java.sql.Date, java.sql.Time and java.sql.Timestamp accordingly for compatibility with HyperSQL and older PostgreSQL JDBC drivers.
Write more (and more understandable) comments. Write documentation.

It's ready. Currently one test fails, because of an error in hibernate-core, in a part that had been previously untested. It can be somewhat fixed with this patch.
diff --git a/hibernate-core/src/main/java/org/hibernate/type/descriptor/java/LocalDateJavaDescriptor.java b/hibernate-core/src/main/java/org/hibernate/type/descriptor/java/LocalDateJavaDescriptor.java
index d911e0f..563b912 100644
--- a/hibernate-core/src/main/java/org/hibernate/type/descriptor/java/LocalDateJavaDescriptor.java
+++ b/hibernate-core/src/main/java/org/hibernate/type/descriptor/java/LocalDateJavaDescriptor.java
@@ -42,7 +42,7 @@ public class LocalDateJavaDescriptor extends AbstractTypeDescriptor<LocalDate> {

 	@Override
 	public LocalDate fromString(String string) {
-		return (LocalDate) LocalDateType.FORMATTER.parse( string );
+		return LocalDateType.FORMATTER.parse( string, java.time.temporal.TemporalQueries.localDate() );
 	}

 	@Override

An analogous solution will probably have to be applied to the other Java 8 time types, because the same mistake is in all of them. Also this will return null if the TemporalQuery cannot produce the desired type, but we should throw an exception instead, so my patch is rather simplistic.

It seems primitive arrays are since supported in pgjdbc: https://github.com/pgjdbc/pgjdbc/pull/887

Superseded by https://github.com/hibernate/hibernate-orm/pull/3799


Change LoadableAttributes into EidosComponents

Build the components under varying conditions, allow components to come from places other than the configuration file via new constructor, block access to configuration inside Eidos, have components answer rudimentary configuration questions, unify adjective grounder construction, have callers of constructors ensure they are constructing from the same configuration.

Hi @BeckySharp, this is just a courtesy call so that I don't feel solely responsible for having messed up the code.  You're welcome to ignore :-)


Message Display on the color components.

When clicked on the color red it is displaying a message "undefined" and the same with green as well where it shows "You have chosen the color Green". This is not necessary. Correct this issue.

The issue has been resolved.


owner commands seemed to have disappeared

I seemed to have encountered a weird issue:

I used the command /owner gw enable

The bot returned an error (that I cannot replicate) with the following link: https://docs.disnake.dev/en/stable/ext/commands/additional_info.html

all the owner commands are no longer registered on the guild and /owner returns no commands from the bot

I suppose you encountered the "this command is outdated" error. Which could happen if you recently updated the bot or modified config.json values (see below). If that's the case, it should update itself, not disappear. You can try to just refresh the page, restart discord or change back and forth to another server to see if the commands reappear.
If not, check if 'debug_server' is properly set to your server ID in config.json.
And if it's not:

Set it.
Restart the bot.
It might take a bit of time before it syncs.
If it takes too much time, refresh the page or change from one server like I mentioned above.

Otherwise, I'm clueless about what it happened.
No particular message in the log?
Did you try on another browser/device?

I should also ask, how do you run the bot?

The dockerfile?
If not, do you use any command line parameter beside -run ?

The other possibility that I can see is if the Owner Cog (owner.py) failed to load or is missing. Then, trying to use those local command would effectively remove them.

I suppose you encountered the "this command is outdated" error. Which could happen if you recently updated the bot or modified config.json values (see below). If that's the case, it should update itself, not disappear. You can try to just refresh the page, restart discord or change back and forth to another server to see if the commands reappear.
If not, check if 'debug_server' is properly set to your server ID in config.json. And if it's not:

Set it.
Restart the bot.
It might take a bit of time before it syncs.
If it takes too much time, refresh the page or change from one server like I mentioned above.

Otherwise, I'm clueless about what it happened. No particular message in the log? Did you try on another browser/device?

The following fixed the problem, the debug_server id inputted in the config.json was a different server, after setting it correctly and restarting the owner commands came back.

I should also ask, how do you run the bot?

The dockerfile?
If not, do you use any command line parameter beside -run ?

I use the dockerfile

A final question (that's a lot, sorry): What about other commands, do they show up? If they do, I think the problem is with your server ID in config.json.

the other commands show up just fine
Anyway thanks for the prompt replay! I will be closing the issue now since its resolved, good luck on GW!


Adding new optimization feature

Project Robyn
adding new feature to optimize based on historical spend by not changing total spend in that period.
Type of change

feat: New feature (non-breaking change which adds functionality)
Using daily, weekly, and monthly granularity data, the optimization is performed to set total cost used in the period are the same. If the cost of channel Facebook is set at 120%, the optimization method would set the cost  of channel Facebook for the entire period at 120% for each level of granularity. This is helpful in determining annual budgets and other long term budgets

How Has This Been Tested?

If you've changed docuemntation, run devtools::document(). Should update .Rd files.
Make sure your code lints.

Comments
Please review and tell me the function name  or place to marge to current function.
If you have any questions please let me know.

hi, thanks for submitting this. I see that you're adding a new dependency Rsolnp for the optimisation. Is there any reason you can't use the current one/ nloptr? Due to CRAN submission, we're conservative about adding new dependencies.
You've added the new parameter new_budget_ratio that scales the total spend, which is a good idea. But I do think it's a lot of new codes and functions for creating this in your solution, while theoretically you can just add the new multiplyer on histSpendUnitTotal. Or am I wrong about your intension?

Let me modify to use nloptr.

It's different from current optimization logic. not using mean of spend but using actual historical spend itself.
I will explain it on Robyn hackathon .


docker push failed when content trust using notary is set

OS Centos7
docker-engine-1.13.1-1.el7.centos.x86_64
docker-registry-0.6.8-8.el7.noarch
notary v0.4.3
export DOCKER_CONTENT_TRUST=1
export DOCKER_CONTENT_TRUST_SERVER=https://<IP_ADDRESS>:4443
Description
docker push failed when content trust using notary is set with error:
[0025] sign called with 1/1 required keys
DEBU[0025] No yubikey found, using alternative key storage: no library found
DEBU[0025] sign called with 0/0 required keys
Finished initializing "localhost:5000/notary043_signer"
Failed to sign "localhost:5000/notary043_signer":latest - no hashes specified for target ""
no hashes specified for target ""

Steps to reproduce the issue:
1.export DOCKER_CONTENT_TRUST=1
2. export DOCKER_CONTENT_TRUST_SERVER=https://<IP_ADDRESS>:4443
3. systemctl start docker
4.systemctl start docker-registry
5.for notary: 'docker-compose up'
6. docker pull hello-world
7. docker tag hello-world localhost:5000/hello-world
8. docker push localhost:5000/hello-world:latest
9. config.json:
{
"trust_dir" : "~/.notary/trust",
"remote_server": {
"url": "https://localhost:4443",
"root_ca": "/root/notary-0.4.3/fixtures/root-ca.crt"
}
}
Describe the results you received:
Failed to sign "localhost:5000/notary043_signer":latest - no hashes specified for target ""
no hashes specified for target ""
Describe the results you expected:
should be successful and return the key for 'docker push'
Additional information you deem important (e.g. issue happens only occasionally):
always
Output of docker version:
[root@localhost .notary]# docker version
Client:
Version:      1.13.1
API version:  1.26
Go version:   go1.7.5
Git commit:   092cba3
Built:        Wed Feb  8 06:38:28 2017
OS/Arch:      linux/amd64
Server:
Version:      1.13.1
API version:  1.26 (minimum version 1.12)
Go version:   go1.7.5
Git commit:   092cba3
Built:        Wed Feb  8 06:38:28 2017
OS/Arch:      linux/amd64
Experimental: false
(paste your output here)

Output of docker info:
[root@localhost .notary]# docker info
Containers: 11
Running: 6
Paused: 0
Stopped: 5
Images: 26
Server Version: 1.13.1
Storage Driver: devicemapper
Pool Name: docker-251:0-1385386-pool
Pool Blocksize: 65.54 kB
Base Device Size: 10.74 GB
Backing Filesystem: xfs
Data file: /dev/loop0
Metadata file: /dev/loop1
Data Space Used: 2.858 GB
Data Space Total: 107.4 GB
Data Space Available: 39 GB
Metadata Space Used: 4.985 MB
Metadata Space Total: 2.147 GB
Metadata Space Available: 2.142 GB
Thin Pool Minimum Free Space: 10.74 GB
Udev Sync Supported: true
Deferred Removal Enabled: false
Deferred Deletion Enabled: false
Deferred Deleted Device Count: 0
Data loop file: /var/lib/docker/devicemapper/devicemapper/data
WARNING: Usage of loopback devices is strongly discouraged for production use. Use --storage-opt dm.thinpooldev to specify a custom block storage device.
Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
Library Version: 1.02.107-RHEL7 (2015-10-14)
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: aa8187dbd3b7ad67d8e5e3a15115d3eef43a7ed1
runc version: 9df8b306d01f59d3a8029be411de015b7304dd8f
init version: 949e6fa
Security Options:
seccomp
Profile: default
Operating System: Centos 7
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 5.466 GiB
Name: localhost.localdomain
ID: AXPM:IDPE:W6QQ:GFYM:WWWC:NZUZ:IQLI:EWJF:KW4J:VGSI:ZAUI:Z7RN
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Experimental: false
Insecure Registries:
<IP_ADDRESS>/8
Live Restore Enabled: false
(paste your output here)

Additional environment details (AWS, VirtualBox, physical, etc.):

ping @cyli @endophage

This looks like it might be related to a similar error we saw in DDC. It seems like maybe something has changed in 1.13 in how the image name string, i.e. localhost:5000/hello-world:latest is parsed and communicated over the engine API?

@endophage: hm that could be a possible cause since it seems to pick up "" as the target (tag)
@chunglam1999: could you please post the full debug logs for DOCKER_CONTENT_TRUST=1 docker push localhost:5000/hello-world:latest?


Edit Space Name

Feature Request
Context
Story
As a PeopleMover user
I want to be able to edit a space name
So that I can edit mistakenly named spaces or change them based on organizational name changes

Acceptance Criteria
Given  I create a space
When I click the elipses on the modal in the dashboard
And an Edit option appears
And it changes color when I hover over it
And a modal appears when I click Edit
And I can edit the space name and click Save
Then the modal on the dashboard is updated with the new space name
Design
https://www.figma.com/file/S4XKeovlQqeRk64k2qzhf8gL/PeopleMover?node-id=4990%3A170
Notes

Hover state missing
Should we reconsider the "Last modified" state? Should it reflect the time when space name was edited too?


Update http4s-circe, http4s-dsl, ... to 0.23.2

Updates

org.http4s:http4s-circe
org.http4s:http4s-dsl
org.http4s:http4s-ember-server

from 0.23.1 to 0.23.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Codecov Report

Merging #176 (3156132) into develop (9d75e57) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #176   +/-   ##
========================================
  Coverage    50.58%   50.58%
========================================
  Files            5        5
  Lines           85       85
========================================
  Hits            43       43
  Misses          42       42

Impacted Files
Coverage Δ

src/main/scala/app/Api.scala
100.00% <100.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 9d75e57...3156132. Read the comment docs.


Add derive macros for FromSql and ToSql on newtypes

I have some code with a lot of newtypes, where the inner types have implementations of FromSql and ToSql. It would be nice to be able to #[derive(ToSql, FromSql)] on these types.
Obviously this is not always desirable, since it loses the type-safety of the newtype in a way that a different serialization format might not, but I think it's often the right thing, and is a convenience that it'd be nice to give to people.

I've come up with an implementation that works for newtypes that wrap types that already implement FromSql/ToSql.
https://github.com/koakuma-chan/rusqlite_derive
Example:
use garde::Validate;

use rusqlite_derive::{FromSql, ToSql};

use serde::{Deserialize, Serialize};

#[derive(Deserialize, Serialize, Validate, FromSql, ToSql)]
#[garde(transparent)]
#[serde(transparent)]
pub struct Username(
    #[garde(length(min = 3, max = 24), alphanumeric)]
    //
    String,
);

Can be added like this: cargo add rusqlite_derive --git https://github.com/koakuma-chan/rusqlite_derive.
I would be willing to contribute this to rusqlite upon maintainer's endorsement or let me know you would like this on crates.io.

Just for reference:
https://docs.rs/postgres-types/latest/postgres_types/#derive
but SQLite supports only 5 types while postgresql supports many + user-defined types.


fix updatecli values filename, move repo info to configs

Updatecli is failing because the values.yaml is named 'values.yml', this corrects that and moves the repo data from the values file to the updatecli config.
This addresses https://github.com/k3s-io/k3s/issues/7363
I tested this locally by changing the owner and branch in the updatecli config to mine, adding the github env variables, and running the updatecli diff tool:

Diff Output
updatecli diff -v updatecli/values.yaml -c updatecli/updatecli.d/updatecli.yml; git reset --hard; cd ..

+++++++++++
+ PREPARE +
+++++++++++

Loading Pipeline "updatecli/updatecli.d/updatecli.yml"

SCM repository retrieved: 2

++++++++++++++++++
+ AUTO DISCOVERY +
++++++++++++++++++

++++++++++++
+ PIPELINE +
++++++++++++

################################################################
# INTRODUCE UPDATECLI TO REPO AND VALIDATE BASIC FUNCTIONALITY #
################################################################

SOURCES
=======

goTag
-----
Searching for version matching pattern "^go1\\.20\\.2$"
✔ Git tag "go1.20.2" found matching pattern "^go1\\.20\\.2$"

CONDITIONS:
===========

testVersionShouldMatchGoTag
---------------------------
✗ Key "version", in YAML file "updatecli/validate.yml", is incorrectly set to go1.20.1 and should be "go1.20.2"

TARGETS
========

updateValidateFile
------------------

**Dry Run enabled**

⚠ Key 'version', from file '/tmp/nix-shell.Loa5SD/updatecli/github/matttrach/system-agent-installer-k3s/updatecli/validate.yml', was updated from 'go1.20.1' to 'go1.20.2'

ACTIONS
========

[Dry Run] An action of kind "github/pullrequest" is expected.

=============================

REPORTS:

⚠ Introduce updatecli to repo and validate basic functionality:
        Source:
                ✔ [goTag] Get Go 1.20.2 tag (kind: gittag)
        Condition:
                ✔ [testVersionShouldMatchGoTag] test version should match go tag (kind: yaml)
        Target:
                ⚠ [updateValidateFile] Update the version in the validate file (kind: yaml)

Run Summary
===========
Pipeline(s) run:
  * Changed:    1
  * Failed:     0
  * Skipped:    0
  * Succeeded:  0
  * Total:      1

I wanna say that was my first thought as well, but it kinda seems like the common choice for most k3s repos is to use the full extension name, with the exception of workflows and updatecli configs. Overall I decided that this way seemed more natural, but honestly it could go either way. I just want to be consistent (across repos) with whatever choice we make, and right now that is 'values.yaml' (there are now about 30 repos set up this way). This seems like a good topic for an ADR.


Services section

Want to make the icons and para a bit more organised

This issue is already solved!
Please look for another issue.
Good luck!


fix: Broken Slack invite link in README.md

Closes #2468
Signed-off-by: Bryon Nevis<EMAIL_ADDRESS>

recheck

recheck


[5.1] Allow collection prepend with a key

This would allow you to do stuff like this:
$options = User::lists('name', 'id')->prepend('Select a user...', '');

Form::select('user', $options);

:+1:

Oh yes :+1:

:+1: Would be great.

:+1: It would be even more great if we have same method as an Arr helper.

@rkgrep good idea. Added.

Please send docs PR.

@taylorotwell we should really move the docs into this repo so that people can do documentation in the same PR.

@JosephSilber - I was curious about the use of array_unshift if the key passed in to a prepend is null. I suspect there is a good reason for this but the behaviour I was seeing surprised me. In my case:
$asset_types = \App\AssetType::orderBy('label')->pluck('label','id');         $asset_types->prepend('N/A',null);
resulted in the rekeying of the plucked data and disassociated the id from the label. Now that I understand the behaviour I avoided it by passing in a key of 0 rather than null; however, given how pluck and prepend can be used to create a convenient form select if someone was not paying attention they would not be getting the intended results. Would it help to document and caution against using a null key with the prepend in the docs at https://laravel.com/docs/7.x/collections#method-prepend

@arborrow why are you calling it with null instead of an empty string?
Regardless, the solution would be to check the amount of arguments:
public static function prepend($array, $value, $key = null)
{
    if (func_num_args() < 3) {
        array_unshift($array, $value);
    } else {
        $array = [$key => $value] + $array;
    }

    return $array;
}

@JosephSilber - null was probably a mistake more than anything else. I have typically used 0 since I am using it to fill in some type of id number. An empty string probably was what I intended. At the time I was thinking if I put a null value in then the field would remain null if no item was selected. In any case, I trust you understand the issue and have a better understanding than I what is best.

0 and empty string both yield expected results like:
"" => "N/A"
12 => "Appliances"
1 => "Buildings"
11 => "Communications"
15 => "Computers"
I tested the proposed patch and it certainly better protects me from myself at least against the unexpected behaviour and preserves the key with null, empty string and a value such as 0. So my +1 and thanks for your consideration of this.


Why not store encryption_key in Google::Cloud::Storage::File Class

[Context]
When I attempt to download an encrypted file from Cloud Storage, I must use the encryption_key twice. Once to create the File object and another to download the file to my local disk.
file = bucket.file storage_file_path, encryption_key: encryption_key
file.download local_file_path, encryption_key: encryption_key

[Question]
Do we have a reason for not storing the encryption_key in the File object while it's being used?
Thank you.

We considered this when first implementing encryption keys, but found it added too much complexity. There are several methods that can take encryption key. Storing the key on a file object doesn't indicate which methods that key should be used for, so we would still need an opt-in (or opt-out) mechanism on those methods.

Follow-up Question: Opt-in / Opt-out which would allow the encryption to be stored in the file object and still be passed in explicitly if the user wants to?

In the end we decided using encryption keys was sensitive enough that users should be explicit about using them, which is why we implemented the API we have.

Understood. Thank you!

@frankyn If good, can we close?


How can i abort running XHR request?

In jQuery i need to do like this xhr.abort()

Hi @Ar2r, sorry for the delay. I've discussed the use of abort with others and I couldn't find a good solution so far, so, currently, mappersmith doesn't have official "support" to abort. But you can have access to the xhr object through the configure method, something like:
import { configs } from 'mappersmith'
configs.gatewayConfigs.XHR = {
  configure(xhr) {
    // save the reference of xhr for latter use
  }
}

Since the resource methods returns a Promise I'm not sure how to give access to specifics of each gateway request. If you have any suggestions please share.
Thanks


style: re-enable GitLab patch URL check

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Follow up to https://github.com/Homebrew/brew/pull/10760
This PR re-enables the GitLab patch URL check. This will need to wait to be merged until https://github.com/Homebrew/homebrew-core/pull/74023 and https://github.com/Homebrew/linuxbrew-core/pull/22752 are merged.

Review period skipped due to critical label.

homebrew/core PR merged: https://github.com/Homebrew/homebrew-core/pull/74023
I had to split up the linuxbrew/core PR (https://github.com/Homebrew/linuxbrew-core/pull/22752) because of the CI timeout. But the broken up PRs failed CI because of missing bottles.
I suppose we could just wait till the commits in https://github.com/Homebrew/homebrew-core/pull/74023 are merged into linuxbrew/core.

Linuxbrew PRs merged. Restarting tests here.


Add slider to Graphics.Input

Not entirely sure how Input stuff works, so please correct me if I got anything wrong (esp. in Native).
Here is a somewhat more complicated test case where you can edit the upper bound of the slider live:
import Graphics.Input as Input
import Graphics.Input.Field as Field
import String

ip = Input.input 0

mc : Input.Input Field.Content
mc = Input.input Field.noContent

field = Field.field Field.defaultStyle mc.handle id "Max" <~ mc.signal

s m v = let ma =
          case String.toFloat m.string of
                    Just f -> f
                    Nothing -> 5
        in Input.slider ip.handle id -5 ma 0.5 v

main = (\x y z -> x `above` y `above` z)
     <~ field ~ (lift2 s mc.signal ip.signal) ~ lift asText ip.signal

I'd recommend using elm-html with Html.toElement for now. We are building up a list of potential revisions to core in https://github.com/elm-lang/core/issues/322, and Graphics.* may be getting split out so we can evolve that idea more independently.


[NTI] Variadic params {...T} should infer union types rather than failing

See this example.
Array.of is typed as @param {...T} @return {!Array<T>}, NTI fails with
Array.of(1, 'foo', undefined)

with the error
input0:1: WARNING - Illegal instantiation in function type function(...T):Array<T> for the type variable T.
 You can only specify one type. Found [number, string, undefined].
Array.of(1, 'foo', undefined));
^

It's deliberate to instantiate T to a single type, rather than the union of the arguments. This is usually the user intent, and the fact that the old type checker just unions everything is a loose check IMO. Also, I wouldn't want to special-case Array.of by name. So, the current behavior is inconvenient in this case, but I think it's generally the right one.
The fix for this case would be to allow explicitly specifying the type to instantiate, and one would be able to say that it is ? in this case.


First version of user documentation

End user documentation should be created which is included in the main website.
Depends on #16

Initial version of the user documentation is delivered via readthedocs.io by @julio-santiesteban


Add Nerdicons Plugin

Nerdicons Plugin
This is an extension/modification of the nonicons plugins
Any icon can be searched by it's hex from nerdfonts.com/cheat-sheet
How to use:

Download a nerd font that you like from nerdfonts.com
Unzip the zip file that you downloaded.
Choose a font file from the unzipped directory and install it.
Copy the font file to the lite-xl data directory and rename it to "icon-nerd-font.ttf".
Restart lite-xl, the icons should appear.

I have updated the manifest.json.
Is there anything else, I have to do?

Okay:)

OK; tried it, looks good! Happy to merge as is.
One of the things that you may be interested in though, is the new files key that you can add in the manifest.
It may not make sense for this plugin specifically; as you're supposed to choose the iconset you want.
But one thing you can do, is specify a default font url from nerdicons, and flag it as optional, so that things'll basically just work out of the box when someone uses a plugin manager to install (the plugin manager will download the zip to the plugins folder, and then unzip it).
Though you'd have to set up some extra logic to rename the font as it comes out of the zip; it could be a nice addition.
But I'll merge this now; feel free to tack it on as another PR if you wanna do it; otherwise, we're still good anyway.

Okay, will do it if I have any free time.
Thanks😁


GET requests with multi-value form fields

When making a GET request like this:
{
  "url": "/api",
  "params": {
    "aggregate": "count",
    "column_names": [
      "activity_amount",
      "cash_earned"
    ],
    "start_date": "2015-02-26",
    "end_date": "2015-05-27"
  },
  "method": "GET"
  }

The url becomes:
/api?aggregate=count&column_names=activity_amount&column_names=cash_earned&start_date=2015-02-26&end_date=2015-05-27
I'm not sure how most APIs expect multi-value items on a GET request but the API that I use needs it to look like this:
/api?aggregate=count&column_names[]=activity_amount&column_names[]=cash_earned&start_date=2015-02-26&end_date=2015-05-27
(escaped or unescaped is fine).
Is there any reason not to change buildUrl.js to create a URL that looks like this? Or could there be an option that allows for that on multi-value? I have a fix for it, but it currently doesn't take into account the case where you may not want there to be brackets on those keys.
Thoughts?

duplicates #49


WIP Demographics file entries for CDPs

This PR adds new entries to the demographics and zip codes files for Census-designated places (CDPs), which were unintentionally excluded from the previous versions. This is most notable in Maryland, where several large counties contain no incorporated communities. (see issue #768)
These were mashed together from various data sources, including primarily the Census/American Community Survey:

S0101: Age and Sex (All Places)
S1901: Income in the last 12 months (All Places)
S1501: Educational Attainment (All Places)
County population demographic distributions (gender, race, age groups) (cc-est2015-alldata.csv)

Note that a lot of the demographic information is derived from county-level information, as was done previously in the old Ruby version: https://github.com/synthetichealth/synthea/blob/ruby/lib/tasks/tasks.rake#L356

National places list: https://www2.census.gov/geo/docs/reference/codes/files/national_places.txt
National Places Gazetteer File: https://www2.census.gov/geo/docs/maps-data/data/gazetteer/2019_Gazetteer/2019_Gaz_place_national.zip

I know there are still some issues, missing entries, etc, so I'm putting this up for WIP in case people want to get an early look, and so I can track the fixes.
Known issues:

[x] Missing counties for certain places, which means the demographic info is also missing
[x] Missing lat/lon for certain places
[x] Need to check that every location in demographics.csv has a corresponding line in zipcodes.csv
[x] Need to check more rigorously for any duplicates
[ ] Encoding issues, ex ñ becoming � (edit: turns out this is an issue in the source data. I have cleaned some up, but apparently CSV was not designed to be UTF-8/16 compliant so the source data is all over the place)
[ ] Missing zip codes (Since any missing zip codes get replaced with 00000 this is the lowest priority)

For the moment these are separate files so they are easier to review, but once everything looks good in them I'll merge them back into the regular demographics.csv and zipcodes.csv files. (Potentially by just appending to the end, to minimize the chance of breaking anything)

Marking this as ready for review, although because the files are still separate, please don't merge yet.
For posterity, the "script" that I used to create these is attached. (It's in Java not python because I know Java a lot better.)
There are a lot of hacks and manual additions
CDPDemographicsBuilder.java.txt
in here to try to get things to line up, apologies in advance to anyone who tries to read it.

Also I don't understand why Travis is failing at this point. As it stands the 2 new files aren't even referenced anywhere yet.

@dehall I started to analyze the new files and a few things jumped out at me:
The new demographics file is substantially smaller than the old one (8746 vs. 34492). Is this intended?
A few quality checks failed for me using https://nbviewer.jupyter.org/gist/eedrummer/8d226fe5f7d106f9594889df1075fcfb

The new zip code file has 45 duplicate entries in it
There are 199 locations in the zip code file with no corresponding entries in the demographics file. So with the way the code works in Synthea, these are essentially dead lines in the file

I think we would need to clean up these issues and know what we are losing before we merge this.
Also, it's worth checking the major cities, to see if the zip code lat/lon are spread around appropriately.
Finally, it's good to look at a lat/lon difference inside of a city. I had run into cases with clearly wrong lat/lon for cities with multiple zip codes, where you would have points that were 200 miles apart for the same place.

The new files are intended to be an addition to the existing ones, for now just kept separate so it's easier to process. So the different size is expected. It just happens to be smaller because there aren't many CDPs as compared to whichever form of geographic location we were using to populate the original file.
I think I know why the duplicate zip codes are happening and that's easy to fix (CDPs can be within multiple counties, so they are listed twice in the census data files)
Not sure why there are files in zipcodes that are not in demographics though, that doesn't really make sense. Will look closer into that. (midway update: it becomes apparent as you step through the code. Most likely those are places with 0 population, which got deleted from the demographics file after they were used to create the zip codes file)
For posterity, this is the basic approach the script took:

Pre-processing steps:

Load the "S1901: Income in the last 12 months (All Places)" file, remove all places without "CDP" in the name
Load the "S1501: Educational Attainment (All Places)" file, remove all places without "CDP" in the name
Load the "S0101: Age and Sex (All Places)" file, remove all places without "CDP" in the name
Load the "County population demographic distributions" file

Create a mapping of county --> { male, female, race/ethnicity }

Load the existing Synthea demographics file

Create a set of all places already included there

Load the "National places list" file

Create a mapping of place --> county
Add/update some specific entries

Load the "National Places Gazetteer File"

Create a mapping of place --> { lat, lon }

Load the "USA Zip" file

Create a mapping of place --> zip

Create a manual set of places to exclude for various reasons (some CDPs have been discontinued, or are already included elsewhere)

Processing:

Create a map of string -> entry to put the data for the new demographics file in
For each line in the "income all places" file:

Skip it if it's in the excluded places set
Skip it if it's already in the existing demographics file
Create a new entry in the new demographics list
Look up the county name in the place --> county mapping
Add the gender/race/ethnicity info from the county to this place's demographics entry
calculate the income stats for this place, and add it to this place's demographics entry

For each line in the "education all places" file:

Skip it if it's not in the new demographics map
calculate the education stats for this place, and add it to this place's demographics entry

For each line in the "age all places" file:

Skip it if it's not in the new demographics map
calculate the age stats for this place, and add it to this place's demographics entry

Create a list of entries to put the data for the new zipcodes file in
For each place in the new demographics map

Create a new entry for the zipcodes file list
Look up the lat/lon in the place --> lat/lon mapping and add it to this entry
Look up the zipcode in the place --> zip mapping and add it to this entry

Remove entries from the demographics map if the population is 0
Write out the demographics file
Write out the zipcodes file

@eedrummer updated to remove the mismatches and duplicates.
Re: checking lat/lon - I only have 1 zip code/coordinate line for each location, which is definitely reductive. I'll see if I can get multiples where appropriate, the challenge is really in aligning data sources of varying quality.

@dehall I didn't realize that the files were additions. Let me know when they are ready for another look and I'll run some checks on the combined files in the notebook I have.

@eedrummer This should be good now. Every line in demographics should have 1 or more lines in zipcodes, and every group of lines in zipcodes should have 1 line in demographics. I went back to an old version of the synthea zipcodes file, before a bunch of "unused" zipcodes were removed, and reintroduced those. So now places like Paradise, NV should have many zip codes with varying coordinates.
https://github.com/synthetichealth/synthea/blob/b3ddfd36acbfcccadbe08b2010a939984d4d2f5a/src/main/resources/geography/zipcodes.csv

@dehall sorry to be a pain about this, but just looking through zipcodes_cdps.csv I'm finding issues.
The file still has duplicates and there are also data issues that will need to be dealt with.
Look at Mount Olive, AL as the first example of duplicates:
https://github.com/synthetichealth/synthea/blob/cdp_demographics/src/main/resources/geography/zipcodes_cdps.csv#L128
The three zip codes for that town appear to be repeated. Further, one of the lat/long combos is off. Comparing the distance between the first lat/long and the last lat/long for the town is about an 80 mile drive according to Google Maps.
I can check this stuff pretty quickly with Pandas in python. I think having a library that offers Data Frame-like capabilities or able to execute SQL on a CSV file in java would be handy here if you don't want to go that route. You can do things like group by zip code and calculate outliers from the average lat/long without even getting into actual geographic distance just to filter out crazy values. Unfortunately, I've found some pretty crazy values in the underlying data from census.

@eedrummer No worries! That's why we do PRs. And it's my fault for bringing back rows from an old version of the file without double checking them closely.
I can clean up dups super easily, but in general I'm not sure what to do about weird lat/lons.
You highlighted these 3 lines:
,Alabama,AL,"Mount Olive",35072,33.093498,-86.066947
,Alabama,AL,"Mount Olive",35071,33.694975,-86.848078
,Alabama,AL,"Mount Olive",35117,33.667778,-86.892599

According to well-known beacon of truth https://www.unitedstateszipcodes.org/ -- Mount Olive, AL, appears to only have 2 zip codes: 35071 and 35117. 35072 is in fact 80 miles away in Goodwater, AL. So the issue was we somehow associated 35072 to Mount Olive incorrectly. I'll revisit all our data sources but if there's weirdness in data from the census or other official sources I don't really want to have to be the arbiter of what's valid.

Quick update: I have no idea where the original zip codes list came from. This was the PR where it originated, way back in the Ruby version:
https://github.com/synthetichealth/synthea/pull/254
Note that it already contained the 3 rows for Mount Olive:
815,Alabama,Mount Olive,35072
816,Alabama,Mount Olive,35071
817,Alabama,Mount Olive,35117

Now when I do a little research, I found this source, which says that 35072 is associated to Mount Olive, plus a few other places:
https://www.uszip.com/zip/35072

35072 is the zip code for
Goodwater, AL
Also in this location:
Hollins, AL
Mount Olive, AL
Hackneyville, AL
Millerville, AL

So the latest I just pushed should have removed all the duplicates, but any "weird" zip code lines like the Mt Olive one will still be there. Any suggestions?

@dehall My suggestion is to drop the weird zip code in this case.
The last time that I went through the zip code file, I grouped it on city, because that's how it is really used in Synthea. I would then compare the lat / long values to see if there are cases with any outliers. The outliers were causing me problems earlier, I think mostly because there was a zip code for Denver that is way out by the airport that was causing a bunch of people to be generated there, which isn't realistic.
So, for cases where we have multiple zip codes associated with a single place, my argument is that we should have those agree from a lat / long perspective. If we have to drop one, it's not a big deal, because the town is still in Synthea and all we lose is a potential portion of the town ending up in a zip code that may or may not be valid, which I don't think anyone will care about.
Finally, it may be that 35072 really is a valid zip code for Mount Olive. We're treating zip codes as polygons and getting a lat / long value for the centroid of that polygon. In reality, zip codes are a list of addresses. They may not even represent a constant physical address (my understanding is that there are ships with zip codes). So, there may be an address or block in Mount Olive in the 35072 zip code that is some weird outlier of postal service history. I just think that in our case, it makes the most sense to drop those values.

@eedrummer Updated one more time. Hopefully the last change to the zip codes file since:

there should be no duplicate entries
there should be no instances of blank zip code lines, followed by other lines for the same city that do have zip codes
every line in demographics should have at least one corresponding line in zipcodes, and vice versa
there should be no instances of the same city having locations > 0.3 degrees apart. Even if you set the threshold at 0.2 degrees there are only a couple instances, which look correct on the map: Pahrump, NV and The Woodlands, TX. I can't link directly to a search result but this site has a nice map: https://www.unitedstateszipcodes.org/

Not every line in zipcodes_cdps.zip has a zip code, but this should be ok since the synthea engine will just use 00000 in those cases. It will still use the lat/lon from the zipcodes file based on the city/state name.
And just a reminder, this is intended to be a supplement to the existing demographics and zipcodes files, just separated here for ease of reviewing. If you want to combine them (unsorted), run these 2 commands:
tail -n +2 src/main/resources/geography/demographics_cdps.csv >> src/main/resources/geography/demographics.csv
tail -n +2 src/main/resources/geography/zipcodes_cdps.csv >> src/main/resources/geography/zipcodes.csv

In the meantime I'm working on the command to run to sort them afterward

@dehall Looks great! Once you consolidate the files, I'll merge this PR.

@eedrummer merged the CDP files into the main files, and they should be in the proper order. I did notice there are some existing quirks in the main files, such as places having multiple lines where one has a zip code and one doesn't. (For example, Albion ID) I didn't address those here, partly to make it clear what this change was, and partly because my script wouldn't be a plug&play fix there) but I can if you'd like.


Update Links to Weave Net Documentation

@konradjurk thanks for the PR. Please sign the CLA. Let me know if you have any questions! Thanks!

Sorry this is too complicated. Fix it yourself.


BUILD: Fix file permissions and update ReadMe to make it work on MacO…

I'm not so familiar with pull requests, please tell me if things need to be adjusted.
I tried to stick to the contributing guidelines ;-)

Thanks James.
Just for info, the changes to scripts is a permission change (to make them executable). I should have put this detail in the commit message.


Resolve inconsistencies between the ReferenceDocument and OutputFile classes

Both the ReferenceDocument and OutputFile classes define references to external files. Although both classes contain name and location attributes/slots, there are inconsistencies in:

Specification: OutputFile does not have an id attribute/slot, but includes both fileType and style.
Use: the file name is included in the location value for ReferenceDocument examples, but not for OutputFile examples.

Also, it is currently possible to reference external files in three ways for any given output:

Using the OutputFile class via the fileSpecifications attribute/slot.
Using the ReferenceDocument class either directly in the documentRefs attribute/slot or indirectly in the documentRef attribute/slot of the outputs's programmingCode.

The difference between fileSpecifications and documentRefs is not clear as both may be used to indicate a file containing the output.
The specification and use of the ReferenceDocument class are intended to be consistent with the Define-XML def:leaf specification and I think it would make sense for the specification and use of the OutputFile class also to be consistent with def:leaf.
There seem to be two options to ensure consistency:

Use the ReferenceDocument class for all external files, which could be done in two ways:

Remove the fileSpecifications attribute/slot from the Output class, delete the OutputFile class and just use the documentRefs attribute to indicate output files.  This would mean that it would not be possible to specify the fileType or style for any output file.
Replace the name and location attributes/slots in the OutputFile class with one of:

referenceDocumentId as a reference to a document defined as one of the reporting event's referenceDocuments .
a new slot with a range of ReferenceDocument allowing the document to be specified within the context of the output.
documentRef to allow specific pages to be referenced in a document defined as one of the reporting event's referenceDocuments.

Change the OutputFile class to be a type of ReferenceDocument (i.e., is_a: ReferenceDocument). This would allow each file to be defined within the context of the output while still allowing use of the fileType and style attributes/slots.

If both the fileSpecifications and documentRefs attributes are retained in the Output class, it will be necessary to agree and document the differences between their intended uses. For example, fileSpecifications seems to be more of a technical specification (e.g., "programmatically generated output will be written to the file(s) define in fileSpecifications") while documentRefs seems to be more informational (e.g., "specification documentation for the outputs can be found in this reference document" or "information from this output is represented in that reference document").

The user guide has been written using the convention based on def:leaf (the file name should be included in the location and the name should be just the file name without path or a sponsor-defined label). The output file specifications included in the examples need to be updated to align with this convention. Also, the Excel template needs to be changed to allow output files to be specified according to the convention. The excel2yaml.py program will also need to be updated.

The OutputFile class has been changed to be a type of NamedObject that (still) allows specification of fileType, location and style.  The ARS v1 examples, the ARS Template Excel file, and its accompanying excel2yaml.py utility program have been updated to:

Include/accommodate the new description and label attribute - see #217
Expect location to be populated in the same way as ReferenceDocument.location (i.e., including the full path to the file) - see #220


Feature auditlog

Please take a look.
This PR introduces:

proper typification of (service-level) errors
standardised error structs for services
standardised action structs for services
codegen (yaml => go) for service actions and errors
wrapping/masking logic for errors (so we can mask "invalid password" and "unexisting user" with "invalid username/password combination" for the client but log it with all details
wrapping internal (repository etc...) errors
ability to collect additional props & params around errors & actions (who and how caused something to happen - ip, origin, author etc...)
action log (aka audit log) facility
access to action log (via rest)
support for go1.13 error handling features (Is, As, Wrap, Unwrap)

Notes:

introducing new repository structure that should support multi-db

Still on todo:

[ ] rest of system services
[ ] compose services
[ ] messaging services
[ ] extend rest error reporting to include typified errors

Codecov Report

Merging #91 into develop will decrease coverage by 0.54%.
The diff coverage is 28.70%.

@@             Coverage Diff             @@
##           develop      #91      +/-   ##
===========================================
- Coverage    36.06%   35.51%   -0.55%
===========================================
  Files          467      484      +17
  Lines        36549    40576    +4027
===========================================
+ Hits         13180    14412    +1232
- Misses       21537    24250    +2713
- Partials      1832     1914      +82

Impacted Files
Coverage Δ

pkg/actionlog/types.go
0.00% <0.00%> (ø)

pkg/api/logger.go
28.20% <ø> (ø)

pkg/api/server.go
0.00% <0.00%> (ø)

pkg/permissions/permissions.go
74.50% <ø> (+3.92%)
:arrow_up:

system/rest/stats.go
60.00% <ø> (ø)

system/service/attachment_actions.gen.go
0.00% <0.00%> (ø)

system/service/auth_notification.go
14.92% <0.00%> (ø)

system/service/statistics_actions.gen.go
0.00% <0.00%> (ø)

system/rest/request/actionlog.go
1.55% <1.55%> (ø)

system/service/attachment.go
10.75% <9.72%> (-2.12%)
:arrow_down:

... and 51 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 a09a6be...a0c732f. Read the comment docs.


refactor: de/encode protobuf-encoded byte array with base64

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
de/encode protobuf-encoded byte array with base64
Checklist

[ ]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)
close #2542

Is this a breaking change?

Is this a breaking change?

Yes, it is a breaking change, and the problem only occurs when switching an unfinished procedure to execute on the new version cluster.
We won't let this happen.


[Bug Fix] Closing VSCode leaves a zombie process

The real cause is that StreamReader.ReadLine() hangs even if the file it is reading gets deleted. The fix is to use StreamReader.ReadLineAync(TimeOut) and check the status if thread is quitting.
Tested Scenarios:
For both Root and Normal User:

Launch VSCode
Start debugging
Close VSCode while it is still in debugging mode
Make sure  no zombie process left, especially "corerun OpenDebugAD7.dll" process.

Hi @xingshenglu, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Freddie Lu). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

:+1:

Del had another suggestion which may or may not work. We should check to see if a call to dispose or close on the stream works event when blocked in this call. If it does, we can cancel this that way when _bQuit becomes true. Note that I'm not certain this will work so we'd need to verify via experimentation.


chore(ci): attempt to fix release-plz

After #312 the release-plz workflow still fails. The error message is:
Error: failed to update packages

Caused by:
    0: failed to determine next versions
    1: the working directory of this project has uncommitted changes. If these files are both committed and in .gitignore, either delete them or remove them from .gitignore. Otherwise, please commit or stash these changes:
       ["resources/language-submodules/tree-sitter-php", "resources/language-submodules/tree-sitter-rust", "resources/language-submodules/tree-sitter-solidity", "resources/language-submodules/tree-sitter-sql"]

Those paths are pinned submodules so I have no idea how or what uncomitted changes are involved. There's a similar issue in the release-plz repo but not caused by submodules: https://github.com/MarcoIeni/release-plz/issues/1360
For now, I just enabled the allow_dirty field as a workaround for this error. I have tested locally the command release-plz update and the above errors are gone.
@arendjr If you happen to know the exact reason why the workflow fails, please feel free to shed some light. Because I'm not sure I fully understand the entire workflow of release-plz.

Summary by CodeRabbit

Refactor

Improved configuration settings to allow more flexible project management.

Bug Fixes

Updated feature settings in the project’s logging system to enhance performance and integration capabilities.

Thanks, @Sec-ant, this helps a lot!
I forgot to follow up on this, since the last release was manually published, but I also got to the point there were dirty files, just never got to the point of how to resolve that. At the least, this should help us to get a bit further with the automated releases... and hopefully fixing the workflow entirely :)

I also got to the point there were dirty files

Yeah I just don't understand why there will be dirty files in those submodule paths (and not all of them, just some of them). I also left a question to the author in that related issue in release-plz, waiting for response.

FYI: release-plz is working properly now.


Reporting "incompatible native module" on non-existing transitive devDependency.

I no longer can load my package on Atom beta 1.7.*
"Failed to require the main module of 'Ensime' because it requires an incompatible native module.
Run apm rebuild in the package directory to resolve."

First of, this message doesn't tell me which module is problematic so I had to debug - wouldn't it be super easy to just report the failing module?
apm rebuild doesn't help even when it succeeds.

The problem I have now it that Atom reports a transitive devDependency from grunt-ts on fsevents as incompatible. Debugging in package.js:
JSON.stringify(this.incompatibleModules)
"[{"path":"/Users/viktor/.atom/packages/Ensime/node_modules/ensime-client/node_modules/grunt-ts/node_modules/fsevents","name":"fsevents","version":"0.3.8","error":"Module version mismatch. Expected 47, got 46."}]"

This file doesn't even exist! grunt-ts is a devDependency to compile Typescript on ensime-client so it's not in there:
~/d/p/atom-ensime ❯❯❯ ll /Users/viktor/.atom/packages/Ensime/node_modules/ensime-client/node_modules/grunt-ts
ls: /Users/viktor/.atom/packages/Ensime/node_modules/ensime-client/node_modules/grunt-ts: No such file or directory

I see two issues:

Atom is reporting issues on files that doesn't exist. Is it reading package.json including devDeps and constructing those paths just to confuse? Or is it cache?

Even if only failed on actual physical files in node_modules, my dev workflow with npm link (as described here http://flight-manual.atom.io/behind-atom/sections/developing-node-modules/) would be screwed since the transitive devDep would be there while developing.

Related:

https://github.com/ensime/ensime-atom/issues/220
https://github.com/ensime/ensime-atom/issues/221
https://github.com/strongloop/fsevents/issues/93#issuecomment-202830164
https://github.com/mapbox/node-pre-gyp/issues/188

@lee-dohm because https://discuss.atom.io/t/failed-to-activate-because-native-module/27234/4

Thanks @hedefalk ... I'll refer it to the dev team and see if I can get some :eyes: on it.


updated qlcolorcode (2.0.6)

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Re-adds a postflight block as a safety net, which may fix this issue.


How to transfer chat from BOT to human agent

I have BOT client which the user will interact with. The Bot will use LUIS models for NLP. For any queries tagged under None, we should involve a human agent immediately. What is the best way to transfer the conversation from Bot client to human agent?

Thank you for your suggestion. It is in our backlog of things to add.


Tag and create GitHub releases for each version we release

Even though we're already on version 1.0.23, there are no tags for it (see https://github.com/frictionlessdata/datapackage-pipelines/releases). We should tag the release commits as part of our release process, so it's easier for others to checkout the code on a specific version.

Agreed.
How do you propose we do that (other than manually, of course)?

I'm not aware of any tool to help us here, so I'd simply do it manually.

Doing that manually atm.