Remove gh workflows because they're all on travis already

Description
Both of these checks were on travis already, they are duplicate and only time consuming and generating confusion.
Fixes # (issue)
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[x] Tests update
[x] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I run go fmt ./... to check that my code is properly formatted
[ ] I run go vet ./... to check that my code is free of common Go style mistakes

Codecov Report

Merging #569 (c4803f2) into main (aa84c93) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #569      +/-   ##
==========================================
- Coverage   29.28%   29.21%   -0.07%
==========================================
  Files          33       33
  Lines        3510     3508       -2
==========================================
- Hits         1028     1025       -3
- Misses       2339     2342       +3
+ Partials      143      141       -2

Impacted Files
Coverage Δ

pkg/services/files/extractor.go
50.00% <0.00%> (-5.27%)
:arrow_down:

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 aa84c93...c4803f2. Read the comment docs.


Day4_Umakshi.py

Here is the solution of Day 4 problem of Hacktoberfest 2K22 week.

https://forms.gle/BFPNMQf5jyEsLrsE7


Add valueFormatter, valueGetter callbacks to DataTableColumnDefinition

Modify and extend a column's definition with the following:

field property should be optional
add valueFormatter property: a callback function to transform the data (got from the row's model using field) to be displayed
add valueGetter property: a callback function to transform the row's model to a value to be displayed

Implemented in: https://github.com/i-Cell-Mobilsoft-Open-Source/icell-data-table/pull/17


[RayJob] Add testing for shutdownAfterJobFinishes

The option shutdownAfterJobFinishes in the RayJob spec is currently not tested in CI.  We should add unit tests and an end-to-end test for this.

Is shutdownAfterJobFinishes even working right now?

@dizhouwu It should be working, but it has only been tested manually so far.

@architkulkarni I don't see it working myself - I am using kuberay 0.5.2 with Ray 2.6.1. By the way, I don't see a documentation for the intended behavior - does Failed also count as Finished? What statuses are counted as "Finished"? Thanks

Unfortunately I only have a ray cluster on cloud k8s. Set to true for sure. I guess it must be issue on my side IF this is added to the regression test - so I can happily wait for that work is done. Thanks again!


RuntimeError: size mismatch, m1: [32 x 192], m2: [64 x 128] at /pytorch/aten/src/TH/generic/THTensorMath.c:2033

Hello, I am trying to use this with my custom dataset. I am using a dataloader (see here https://github.com/kevinzakka/recurrent-visual-attention/issues/18) though even when I cast my image input to Float32 and get rid of that error, I get a mismatch of tensors while training the network.
Traceback (most recent call last):
  File "main.py", line 49, in <module>
    main(config)
  File "main.py", line 40, in main
    trainer.train()
  File "/home/duygu/recurrent-visual-attention-master/trainer.py", line 168, in train
    train_loss, train_acc = self.train_one_epoch(epoch)
  File "/home/duygu/recurrent-visual-attention-master/trainer.py", line 252, in train_one_epoch
    h_t, l_t, b_t, p = self.model(x, l_t, h_t)
  File "/usr/local/lib/python3.5/dist-packages/torch/nn/modules/module.py", line 491, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/duygu/recurrent-visual-attention-master/model.py", line 101, in forward
    g_t = self.sensor(x, l_t_prev)
  File "/usr/local/lib/python3.5/dist-packages/torch/nn/modules/module.py", line 491, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/duygu/recurrent-visual-attention-master/modules.py", line 214, in forward
    phi_out = F.relu(self.fc1(phi))
  File "/usr/local/lib/python3.5/dist-packages/torch/nn/modules/module.py", line 491, in __call__
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/torch/nn/modules/linear.py", line 55, in forward
    return F.linear(input, self.weight, self.bias)
  File "/usr/local/lib/python3.5/dist-packages/torch/nn/functional.py", line 992, in linear
    return torch.addmm(bias, input, weight.t())
RuntimeError: size mismatch, m1: [32 x 192], m2: [64 x 128] at /pytorch/aten/src/TH/generic/THTensorMath.c:2033

I can not figure out what goes wrong. Is it about patches or weights? Any insights could be really helpful. Thanks.

So it turns out this might be a problem of pytorch, people did report problems with some image sizes. And an easy fix suggested (there seems to be a consensus) is to use adavptive pooling layer instead of average pooling. For example
resnet = torchvision.models.resnet18()
resnet.avgpool = nn.AdaptiveAvgPool2d(1)

is enough to overcome this problem. So I I did modify the nn parts that are imported and modified from torch models (in model.py) but it didn't work.

So at this point it doesn't support 3 color channel and variable width/height right?
Have you figured out how to do this?

@linzhiqiu I am not sure about the exact reason, the author implied it might be due to using rectangular size. I don't want to take crops though as that beats the purpose of using attention model in my dataset.  I haven't figured it out yet, let me know if you plan to use this repo with your custom dataset and how it goes.

Sure. In the meantime, are you aware of any repo that implemented Deep RAM (DRAM) instead of RAM? I think in the paper "Multiple Object Recognition with Visual Attention", they extended RAM to make it more suitable for natural image classification, for example the Google Street View multi-digit classification dataset. I think that might be a newer/more powerful model than RAM, but I don't think anyone has open-source their code yet. Anyway, I just thought that maybe DRAM can better work on our custom datasets, as mentioned in another issue that the performance is not good with dataset other than Cluttered MNist.

Hey did you figure out the reason why the error occur?
I ran into the same problem when resume my own training CNN for testing dataset, the training and testing datasets are of exactly the same dimension. So I'm confused why it report this error...
File "load_model_test.py", line 72, in
outputs = model1(inputs)
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/parallel/data_parallel.py", line 114, in forward
outputs = self.parallel_apply(replicas, inputs, kwargs)
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/parallel/data_parallel.py", line 124, in parallel_apply
return parallel_apply(replicas, inputs, kwargs, self.device_ids[:len(replicas)])
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/parallel/parallel_apply.py", line 65, in parallel_apply
raise output
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/parallel/parallel_apply.py", line 41, in _worker
output = module(*input, **kwargs)
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/usr/project/xtmp/superresoluter/approximator/model1/apxm.py", line 60, in forward
output = self.regressor(x)
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/modules/container.py", line 91, in forward
input = module(input)
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/modules/linear.py", line 55, in forward
return F.linear(input, self.weight, self.bias)
File "/home/home2/leichen/.local/lib/python3.5/site-packages/torch/nn/functional.py", line 992, in linear
return torch.addmm(bias, input, weight.t())
RuntimeError: size mismatch, m1: [7 x 1036320], m2: [1048576 x 256] at /pytorch/aten/src/THC/generic/THCTensorMathBlas.cu:249

@dearleiii @duygusar Hey guys, I have some free time in the coming week so I'll try and investigate this bug.

@dearleiii @duygusar Hey guys, I have some free time in the coming week so I'll try and investigate this bug.

Has you solved this problems?Can you give me some advice on how to fix it.Thx

Hi! I got the same problem, but found a solution. At line 63 of 'trainer.py', there is the definition of how many channels the input image should have. The problem is that this value is hardcoded and training with colored images give us this problem.
Hope I could help!

Hi, @annanurov! Maybe you are doing matrix multiplication, and for this operation, your matrices should be formatted as m1: [A x B], m2: [B x C]. Note that the number of rows from one matrix should be the same as the columns of another.

The error still persists for me even though I have dimensions such as  -
m1: [A x B], m2: [B x C]

when you give parameters, do --loc_hidden=192 and problem is solved. The reason is the code does not support multiple channels.

when you give parameters, do --loc_hidden=192 and problem is solved. The reason is the code does not support multiple channels.

thanks . i meet this problem, then modified full connect layer`s parameters

@dearleiii Were you able to solve your error?

Hi, I am facing the same issue as this :
RuntimeError: size mismatch, m1: [512 x 32], m2: [1024 x 10]
how to fix this?

Hello all, I am receiving this kind of error, while passing tensor to FC network
RuntimeError: size mismatch, m1: [200 x 10], m2: [11 x 64] at ..\aten\src\TH/generic/THTensorMath.cpp:961.
Need suggestions and help thank you in advance


android-studio error installing

I'm getting this error when installing android-studio.  I'm not sure what the underlying issue is, though, but I think for the time being, it's broken for me, which means it's quite likely broken for others:
C:\Users\klauer
λ  scoop cache rm *
C:\Users\klauer
λ  scoop uninstall android-studio; scoop install android-studio
Uninstalling 'android-studio' (<IP_ADDRESS>).
'android-studio' was uninstalled.
Installing 'android-studio' (<IP_ADDRESS>) [64bit]
android-studio-ide-173.4670197-windows.zip (854.2 MB) [========================================================================================================================================] 100%
Checking hash of android-studio-ide-173.4670197-windows.zip... ok.
Extracting... Unzip failed: Exception calling "ExtractToDirectory" with "2" argument(s): "Could not find a part of the path 'C:\Users\klauer\scoop\apps\android-studio\<IP_ADDRESS>\_tmp\android-studio\gradle\m2repository\org\jetbrains\kotlin\android\extensions\org.jetbrains.kotlin.android.extensions.gradle.plugin\1.2.30\org.jetbrains.kotlin.android.extensions.gradle.plugin-1.2.30-sources.jar.md5'."

This filename is 268 characters long:
C:\Users\klauer\scoop\apps\android-studio\<IP_ADDRESS>\_tmp\android-studio\gradle\m2repository\org\jetbrains\kotlin\android\extensions\org.jetbrains.kotlin.android.extensions.gradle.plugin\1.2.30\org.jetbrains.kotlin.android.extensions.gradle.plugin-1.2.30-sources.jar.md5
On many file systems, Windows has a 260 character limit on paths. See https://msdn.microsoft.com/en-us/library/windows/desktop/aa365247(v=vs.85).aspx

I have tried to install android-studio with scoop and no error during installation. I think the issue has been solved. I have a look at the newest zip installer file, and the path length has been optimized shorter.


PowerToys Run doesn't open

Microsoft PowerToys version
0.78.0
Installation method
Chocolatey
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
After update to 0.78.0 PowerToys Run doesn't open (tried different shortcuts). When I turn PT-Run off, it stays in the process explorer, after I turn it on again, a new process of PT-Run is created next to the original one.
✔ Expected Behavior
PT-Run is opening after ALT+SPACE.
❌ Actual Behavior
PT-Run doesn't open.
PowerToysReport_2024-02-02-09-20-18.zip
Other Software
Win11 22631.3085

Hi, looking at installationFolderStructure.txt, I see that there are two versions of the Everything plugin. <IP_ADDRESS> and 0.77.0
Would you mind:
1 - Uninstall PowerToys.
2 - See if the PowerToys install directory (default: C:\Program Files\PowerToys) is still there. If it is, delete it manually.
3 - Install PowerToys.
Please let us know if this works for you.
/needinfo

Hi,
I tried several uninstallation a installation from different source (Chocolatey, Winget, Github), remove Everthing plugin but nothing worked. Then I uninstalled PowerToys, removed %LocalAppData%\Microsoft\PowerToys (%ProgramFiles%\PowerToys was removed with uninstallation), installed from choco and suddenly it worked.
Thanks for your help!


ExtraSpacingCop has no way of opting-out of variable assignment

I personally agree with most of what's happening inside the ExtraSpacingCop, however I can't seem to opt-out of one of the cases.
The spec for it is https://github.com/bbatsov/rubocop/blob/master/spec/rubocop/cop/style/extra_spacing_spec.rb#L23
Basically, I'm looking for a way to have the rest of the checks but allow lining up variable assignments like so

Need to do a bit more investigation first


Deliberate conflict marker

Add an annotation (say, @Conflict), that can be added to an injector to mark it as conflicting with other injectors that target the same instruction. This tells MixinExtras to fail the injection in this case.
This has the following advantages:

Ability to explicitly conflict with other injectors in cases where this is not currently possible (e.g. with injection points that can only be used on non-conflicting injectors)
Clarifies the intention that the injector should be conflicting

This makes it easier for tools such as mcdev to know if a conflicting injector is intentional.

Possibility for future expansion of scope (e.g. conflicting under certain conditions)

Would it cause conflicts only with other injectors marked as Conflicting?

My idea was to conflict with any injector which targets the same instruction, regardless of what that other injector is or how it's annotated. But more complicated options could certainly be added to the annotation


More "Universal" JS Client

Think about creating a more universal NPM package that can be used with server side node apps, browsers, react/react-native apps.

1.0.30 has it already.


Make check error throw exception

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Allow all methods that require a ControllerName to accept the name with "Controller" so that nameof() can be used

The nameof() operator fills a void that has been missing for quite a while, but it doesn't work when a method requires a controller name, as nameof() will return the full class name (ending with "Controller").  It would be nice if all these methods accepted both the short class name (w/o "Controller") and the full class name.  While the short class name really isn't needed any longer with this operator, it needs to be kept around for existing code.

ACK, I created this in the wrong repo.  Sorry


[release-1.19] Warn muiltple istio namespace labels in analyze, include ambient

This is an automated cherry-pick of #46348

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio_release-1.19
195c32e35850c214b72040f0c7f6a58bed42913a
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Fixed Openweather, Adding Tomorrow.io

This pr will be continually updated, review it if you like.

This is not done. I still have to clean up the code and ensure functionality in a production environment.

It's been while since I looked at this workflow, and I already had a partial conversion from a couple years ago, so I just ended up finishing that up.

Just as a note, PRs are always welcome, and I do appreciate the work put in here. In this case it was just easier to pick up a branch I had been working on, especially after not having looked at this for a couple years 🙂

Yeah no worries! My branch had still been working for all this time, but I might switch back now, this merge was basically just out of necessity.

However, do let me know if you need contributions, I'd be happy to help with PR's


Multiplatform LRU cache

Introduce multi platform (platform agnostic)  LRU cache implementation based on linked hash map for storing items and double linked list for tracking recently used items.

Still in progress

Still in progress


Erased shapes should stay erased when moved

I am facing issue into eraser tool while using it with selection tool. when I draw something and erase some part and then select it and move to other position it move full object that I have created at first removed area also move with selected tool.
Eraser should remove drawing from view not paint with background colour
please help me thanks in advance

rashidabbas, it's a known issue, but I may not able to fix it currently. There are two drawing modes here, one called painting mode, the other called vector mode. In vector mode, you can move object, and object can be erased by object eraser. For painting mode, all stuff are traced as stroke, so you can use stroke eraser to remove part of them.
In your case, looks it's vector mode, and everything is an object. So I would suggest not using the eraser stroke.
It's hard to do a natural eraser function in vector mode.


CB-18858 Additional validation for fallback salt password rotation

Fallback implementation includes re-bootstrapping all nodes, so they have to be running. The lack of this validation caused start-stop autoscale enabled cluster failures.
See detailed description in the commit message.

Rebased to 2.63


YouTube API option does not work in IE Edge

When fed a youtube URL as the source, the video briefly starts, but then consistently errors out with the message "An error occurred. Please try again later." I'm able to reproduce this with my local environment as well as with the YouTube API example.
Tested on Windows 10 and IE Edge 20.10240.16384.0

I'm updating the player to favor the iframe API instead of Flash.
If you take the "video/youtube" entries out of mejs-namespace.js, does that work for you?

Removing "video/youtube" here doesn't seem to work.

Sorry, I meant from the Flash entry on https://github.com/johndyer/mediaelement/blob/master/build/mediaelement-and-player.js#L31

Cool, removing that works if I also remove the conditional here: https://github.com/johndyer/mediaelement/blob/master/build/mediaelement-and-player.js#L1502

@manoj382 Is this still an issue?

@manoj382 Closing issue per comments above. Thanks!


Google Auth

This PR is for review and discussion.
It might be useful to merge this before the actual functionality is finished, as to avoid merge conflicts with xcode project piling up.
The project seems to not be compatible with RN 0.40 right now but the fixes may be minor and we can do a PR for that, since this is one of the next things we need to implement.

I can merge this in after a rebase. Any idea what's missing for this to be ready to use?

Will have to reimplement with another component, this one is no longer supported and incompatible with latest React Native.


textarea tag closing problem not detected

i tried htmlhint to detect error or some missing tag in html but it doesn't catched missing  at end ot .
where i closed it as .

Can you show a demo code?

HTMLHint

<textarea rows="4" cols="20"  class="form-control" ng-model="user.address" required />

textarea is not in empty elements list: https://developer.mozilla.org/en-US/docs/Glossary/Empty_element


change fav icon to logo

https://github.com/ASKnetCommunity/LEAD/blob/main/assets/images/favicon.png

ggf. use this one
https://github.com/ASKnet-Open-Training/ASKnet-Track-II-2020-2021/blob/main/assets/images/logo.png


Use only artifacts built from main branch in package bundle

When generating the packages bundle, we only want to use artifacts built from main branch since we don't update and build packages from release branches. All artifacts built from main are tagged with latest so that makes it a useful filter.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve


LIBHYDRA-155. Use Grouper/LDAP to control access

https://issues.umd.edu/browse/LIBHYDRA-155

@mohideen Some notes:

the user list page still has a "New User" button, but the link is broken because the CasUsersController no longer has a "new" action.
there are a bunch of "test" and "prior" users listed the first time I start up the application, even after running db:reset and db:migrate. Are these left over from the tests?
I get the following error when I run the tests:

ERROR["test_disable_should_disable_an_enabled_download_url", Minitest::Result, 3.6289230003021657]
 test_disable_should_disable_an_enabled_download_url#Minitest::Result (3.63s)
ActionController::InvalidAuthenticityToken: Traceback (most recent call last):
/Users/peichman/.rbenv/versions/2.6.3/bin/bundle: undefined method `map' for nil:NilClass (NoMethodError)

I'm not sure if it is related or not, but I wanted to bring it to your attention.

@mohideen Please disregard my note about the tests, I was running them with an incorrect RAILS_ENV set in my shell.

@mohideen Likewise the test fixture leftovers, those are no longer showing up. I think it was the RAILS_ENV variable causing that. Sorry for the confusion.


LORM.xsl:67-88: Implement natural language processing...

The puzzle 112-f7b740c5 from #112 has to be resolved:
https://github.com/yegor256/jpeek/blob/7d6ae8c676b1b8e4e41f2a873e1c131d458349e3/src/main/resources/org/jpeek/metrics/LORM.xsl#L67-L88
The puzzle was created by Rok Povsic on 08-Feb-18.
Estimate: 30 minutes, role: IMP.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@yegor256/z please, pay attention to this issue

Job #160 is now in scope, role is DEV

@0pdd all puzzles are solved

@0pdd all puzzles are solved

@0pdd all puzzles are solved

The job #160 assigned to @paulodamaso/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@paulodamaso/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat refuse

@0crat refuse (here)

@paulodamaso The user @paulodamaso/z resigned from #160, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @paulodamaso/z

The job #160 assigned to @oridan/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@oridan/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat refuse

@0crat refuse (here)

@oridan The user @oridan/z resigned from #160, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @oridan/z

The job #160 assigned to @thomas-melville/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

The user @thomas-melville/z resigned from #160, please stop working. Reason for job resignation: It is older than 10 days, see §8

Resigned on delay, see §8: -30 point(s) just awarded to @thomas-melville/z

The job #160 assigned to @ilyakharlamov/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@0crat refuse

@0crat refuse (here)

@Serranya The user @serranya/z resigned from #160, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @serranya/z

The job #160 assigned to @rajanamistry/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

Resigned on delay, see §8: -30 point(s) just awarded to @rajanamistry/z

The user @rajanamistry/z resigned from #160, please stop working. Reason for job resignation: It is older than 10 days, see §8

The job #160 assigned to @dfrau32/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

The user @dfrau32/z resigned from #160, please stop working. Reason for job resignation: It is older than 10 days, see §8

Resigned on delay, see §8: -30 point(s) just awarded to @dfrau32/z

The job #160 assigned to @baybatu/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

Resigned on delay, see §8: -30 point(s) just awarded to @baybatu/z

The user @baybatu/z resigned from #160, please stop working. Reason for job resignation: It is older than 10 days, see §8

@0crat status

@0crat status (here)

@ebercamargo This is what I know about this job in C7JGJ00DP, as in §32:

The job #160 is in scope for 15mon
The role is DEV
The job is assigned to @ebercamargo/z for 13hr
There is no monetary reward attached, it's a free job
The job doesn't have any impediments
The budget is 30 minutes/points
These users are banned and won't be assigned:

@0pdd/z: This user reported the ticket
@paulodamaso/z: User was resigned from the ticket
@oridan/z: User was resigned from the ticket
@thomas-melville/z: User was resigned from the ticket
@ilyakharlamov/z: User was resigned from the ticket
@szaffarano/z: User was resigned from the ticket
@gauee/z: User was resigned from the ticket
@serranya/z: User was resigned from the ticket
@rajanamistry/z: User was resigned from the ticket
@dfrau32/z: User was resigned from the ticket
@baybatu/z: User was resigned from the ticket
@uurl/z: User was resigned from the ticket

Job footprint (restricted area)

@ebercamargo/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

Resigned on delay, see §8: -30 point(s) just awarded to @ebercamargo/z

The user @ebercamargo/z resigned from #160, please stop working. Reason for job resignation: It is older than 10 days, see §8

The job #160 assigned to @ideas-into-software/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

The job #160 assigned to @nermeen78/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@nermeen78/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat  wait to understand the task

@0crat wait to understand the task (here)

@Nermeen78 The impediment for #160 was registered successfully by @nermeen78/z

Hi.
On May 24th, few hours after I joined zerocracy, I applied to project
jPeek. Few days ago (after close to 4 weeks) I got assigned tasks... but
I see funding issue is still not resolved (subject of two JIRAs which I
created on May 24th, i.e.
https://github.com/zerocracy/zerocracy.github.io/issues/91 and
https://github.com/zerocracy/zerocracy.github.io/issues/92).
Until zerocracy resolves its funding issues I am not be able to devote
any of my time to any of zerocracy projects. Please stop assigning me
tasks / projects until then.
Regards,
--
Michal H. Siemaszko
Mobile: +48 723 039 978
Signal IM: +48668566023
Email<EMAIL_ADDRESS>Skype: mhs7thraylabs
WWW: http://ideas.into.software/
GitHub: https://github.com/ideas-into-software/
 LinkedIn: http://www.linkedin.com/in/mhsiemaszko/
On 6/25/19 7:44 PM, Nermeen Saleh wrote:

@0crat https://github.com/0crat wait to understand the task
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yegor256/jpeek/issues/160?email_source=notifications&email_token=ACE24QCEUCMFYLRBAK7QLPLP4JKRVA5CNFSM4EP2JSG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYRBGNQ#issuecomment-505549622,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACE24QCKYFKSPCOSOTXPD2TP4JKRVANCNFSM4EP2JSGQ.

Are you speaking to me or about me here; you must always start your message with my name if you want to address it to me, see §1

@0crat resign

@0crat resign (here)

@paulodamaso Job gh:yegor256/jpeek#160 is not assigned, can't get performer

The job #160 assigned to @aivinog1/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@0crat refuse

@0crat refuse (here)

@aivinog1 The user @aivinog1/z resigned from #160, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @aivinog1/z

The job #160 assigned to @robsonataide/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@robsonataide/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat refuse

@0crat refuse (here)

@paulodamaso The user @robsonataide/z resigned from #160, please stop working. Reason for job resignation: Order was cancelled

The job #160 assigned to @scrobot/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

The job #160 assigned to @fellahi-ali/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

0crat status

@fellahi-ali/z this job was assigned to you 9days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat wait for #332

@0crat wait for #332  (here)

@fellahi-ali The impediment for #160 was registered successfully by @fellahi-ali/z


deploy-rpc-playbooks.sh cannot be run independently of test-upgrade.sh

Do NOT include any confidential information, this issue will be created on a public repo
This section is completed by the issue reporter
On which version(s) do you encounter this issue?
Please provide the output to 'git describe --all' for  all deployments where you have confirmed it occurs.
r13.1.0rc1
What type of issue is this? Please tick the appropriate box.

[x] bug - existing functionality that does not work for you as documented
[ ] feature - new functionality that you would like to see added

What is the expected behavior?
The script runs and executes proper playbooks independent of other scripts
What is actually happening?
root@node84:/opt/rpc-openstack# bash scripts/deploy-rpc-playbooks.sh

set -o pipefail
scripts/deploy-rpc-playbooks.sh: line 6: BASE_DIR: unbound variable

What were you attempting to do when you encountered this error (give as much detail as possible)?
Finish a upgrade with a lab without maas that failed due to DEPLOY_MAAS being set to "YES" when it shouldnt have.
How can we reproduce this issue?
Attempt to run the script
How can we verify your issue is addressed?
The script runs and finishes all plays
If you are using or know of a workaround please describe the necessary steps?
I added the following lines to the top of the script under the setpipe
https://github.com/rcbops/rpc-openstack/blob/master/scripts/test-upgrade.sh#L21-L23
What is your evaluation on the importance of this issue, and why?
low, this script shouldnt really need to be run outside of the upgrade / deploy scripts which have the missing variables set

This section is completed during the issue resolution process. If you are reporting an issue, please do not modify the section below.
Which branches are impacted by this issue?
Remove branches that are not impacted.
For each pull request, update the branch line to reference it.

[ ] master
[ ] mitaka-13.0
[ ] liberty-12.2
[ ] kilo

Upgrade impact. Consider the impact on moving between major, minor and patch releases.

[ ] This change does not impact upgrades
[ ] This change addresses a potential upgrade impact
[ ] The upgrade impact is being tracked by another issue.

The issue tracking the upgrade impact is:
Testing impact

[ ] An issue/pull-request has been create or exists to track the testing gap, and is referenced below
[ ] This change includes additional testing
[ ] This change is tested by the existing setup

Additional testing links:
Release note

[ ] This change includes a release note.
[ ] This change does not require a release note.

Doc Impact
Does your change require documentation?

[ ] Yes, and I submitted an issue to Docs issues
[ ] Yes, and I contributed a documentation patch to docs-rpc
[ ] No documentation required for this change

Additional documentation links (please add links if you have ticked yes):

@jacobwagner this was a design choice and so won't be addressed at this time.


Transient for just 2 assoc! calls is slower

This is a very minor improvement but since I identified it, thought I might as well include it.

Nice. I'm planning to cut a release soon. I made a stupid mistake and forgot the password to the key that I used for Clojars, so I need to either make a new one or recover the password from memory before I release.


[ISSUE #405]modify the doc

#TBD

Codecov Report

Merging #424 (9fb3941) into 1.3.0 (c18adf7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             1.3.0    #424   +/-   ##
=======================================
  Coverage     4.77%   4.77%
  Complexity     141     141
=======================================
  Files          227     227
  Lines        10816   10816
  Branches       929     929
=======================================
  Hits           516     516
  Misses       10233   10233
  Partials        67      67

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 c18adf7...9fb3941. Read the comment docs.

@tydhot can you fix the conflicts?

@qqeasonchen done:)

nice


Deploying to Heroku or Openshift Assistance

Hi @Axiacore -- Just saw your knowledgebase repo and it looks lovely.
Is it pretty stable as of right now? Any info on how to deploy to Heroku or Openshift Online?
Thanks!

@brianjking It should be fairly easy to deploy, it is just a standard django project.
Please let us know how it goes for you :)

@camilonova I followed the instructions provided here:
Copy local_settings.py.template to local_settings.py and fill in your information. Have in mind that SITE_URL should be the url where you are hosting this project, and ALLOWED_DOMAINS should be the domains of the emails that are allowed to login and view private articles.

I then ran:

git add local_settings.py -f
git commit -m "settings"
git push heroku

Heroku build fails:
 $ python manage.py collectstatic --noinput
       Traceback (most recent call last):
         File "manage.py", line 22, in <module>
           execute_from_command_line(sys.argv)
         File "/app/.heroku/python/lib/python2.7/site-packages/django/core/management/__init__.py", line 367, in execute_from_command_line
           utility.execute()
         File "/app/.heroku/python/lib/python2.7/site-packages/django/core/management/__init__.py", line 359, in execute
           self.fetch_command(subcommand).run_from_argv(self.argv)
         File "/app/.heroku/python/lib/python2.7/site-packages/django/core/management/__init__.py", line 196, in fetch_command
           settings.INSTALLED_APPS
         File "/app/.heroku/python/lib/python2.7/site-packages/django/conf/__init__.py", line 53, in __getattr__
           self._setup(name)
         File "/app/.heroku/python/lib/python2.7/site-packages/django/conf/__init__.py", line 41, in _setup
           self._wrapped = Settings(settings_module)
         File "/app/.heroku/python/lib/python2.7/site-packages/django/conf/__init__.py", line 116, in __init__
           raise ImproperlyConfigured("The SECRET_KEY setting must not be empty.")
       django.core.exceptions.ImproperlyConfigured: The SECRET_KEY setting must not be empty.
 !     Error while running '$ python manage.py collectstatic --noinput'.
       See traceback above for details.
       You may need to update application code to resolve this error.
       Or, you can disable collectstatic for this application:
          $ heroku config:set DISABLE_COLLECTSTATIC=1
       https://devcenter.heroku.com/articles/django-assets
 !     Push rejected, failed to compile Python app.
 !     Push failed

Secondly, how do I go about getting the secret key https://github.com/Axiacore/knowledge-base/blob/master/app/local_settings.py.template
Thanks!

@camilonova Okay, I think I got the secret key worked out.
What about the Procfile that Heroku requires?

@brianjking If you create the Procfile send us a PR to add into the project, including any other information you might think is important when deploying to Heroku

@camilonova I'm working on it in my own fork/branch and will submit a PR if I ever get it working.
I've got it working for the most part, outside of the Django static file serving issue I do believe. See: https://guarded-ridge-41738.herokuapp.com/
Where are you deploying this without this trouble?

@brianjking We use a Digital Ocean Droplet

@camilonova If I try to go the docker-compose route as per your readme I receive this error:
docker-compose up                                                                                 11:14   08.09.16
Pulling npm (knowledgebase:latest)...
Pulling repository docker.io/library/knowledgebase
ERROR: Error: image library/knowledgebase:latest not found

Please do first docker-compose build

@camilonova Okay, I got it working finally this way. However, now I'm trying to get it to work with Gunicorn & NGINX. Using python manage.py runserver is definitely not suitable for running a production app.
I'm having trouble getting this to work inside of the docker-compose.yml, however, I guess I'll keep looking at it when I have time.

http://serverfault.com/questions/717568/risks-of-using-django-manage-py-runserver-for-production-in-a-small-scale-server

https://docs.djangoproject.com/en/1.8/ref/django-admin/#django-admin-option---nothreading


Integrated attack log

Optionally merge httpd and sshd logs into single coherent attack picture.

Also, allow integrated search.


background black level too black in switch-mins

Perhaps my monitors need adjusting -- my secondary monitor does have the unselected item (Webmin/Virtualmin) visible while on my primary monitor its just too dark and displays as a big black rectangle -- the words Webmin/VirtualMin are not visible when using the current 11,11,11 RGB setting for switch-mins.  I tested my proposed change using the custom CSS settings in Theme Config -- Theme Extensions.  I found this background setting first and tried 75,75,75 which worked, so did not explore any ways to adjust the 'greyed out' font text color or dimness -- its possible that route, if any adjustments are accepted, could be a better way or easier way to overcome my problem.  I bet almost anything there is a source code CSS file better suited for editing -- one that is used to build this bundle.min.css file.
in bundle.min.css I find this fragment:
.switch-mins{text-align:center;color:#c7d2e6;border-radius:0;background-color:rgba(11,11,11,.9)}
I propose replacing 11,11,11 with 75,75,75
See attached cell phone photos of my substandard monitor --- here it is with the updated CSS:

and below is the original CSS -- way too dark:

on my secondary monitor the original CSS, while still quite dark, is visible -- of course with my edit its much more visible.

and I should mention -- as I click around some more while exploring the Authentic Theme, I find additional menu/tab items that appear too dark as well, so its possible other CSS settings will need adjusting.

Hello! Thanks for the heads up!
Well, I agree that the contrast of the disabled switch in navigation menu can be improved. This is done on the latest commit.
Although, I can assure you that the colors/contrast/brightness levels on your monitor is way off.

And the CSS rule you should have used (which I've used on the latest commit) is this:
.switch-mins label
{
    color: #f3f3f3;
    opacity: .33;
}

thanks .... and of course the LABEL construct is much better ... of course I have to go off and read a lot about CSS to fully understand it all   :-)
any thoughts on the cleanest way to 'fix' the right-side-tabs flyout -- it appears to suffer as well from this issue of a darker part upon another darker part --- looking quickly I cannot see what CSS item controls those colors.

any thoughts on the cleanest way to 'fix' the right-side-tabs flyout -- it appears to suffer as well from this issue of a darker part upon another darker part

You're right. I have missed it. Fixed on the last commit.


SelectList: Add support for <optgroup>, replace items prop with subcomponents

A recent blogpost mentioned <optgroup>, which I'd never heard of before. This element allows for grouping/categorization within <select> items. This is a great fit for SelectList, as it could allow more users to use this simpler component rather than needing to use Dropdown. This PR adds support for <optgroup>…
…and also completely overhauls how users specify items in the list. The current items prop wouldn't scale well to support groups, leading to a tedious and confusing nested structure. Even without the introduction of groups, the array of objects isn't very readable, particularly if the user needs to reorder the items. Therefore, this PR creates SelectList.Option and SelectList.Group subcomponents. Users can compose these subcomponents as needed to create their list of items in a much more HTML-like format. And of course, an array of objects can be mapped to SelectList.Option when a more JS-esque approach is desired.
NON-GOALS OF THIS PR

Addressing the Flow types of SelectList's children to ensure that only the subcomponents are used. This is the same problem we have with Dropdown and SideNav, and really everywhere we use subcomponents. I'll work on a solution for all these issues in the future.
Using a more realistic example in the new Groups example. We can update this in the future. Anyone with a good idea for that example is welcome to fast-follow this PR with their fixes.

This looks good. I'm still not sure when to use this vs dropdown groups. I think this guidance might have to be updated?

The current items prop wouldn't scale well to support groups, leading to a tedious and confusing nested structure.

Also, just wondering, is this an example of something that is considered a breaking change because all the select lists in Pinboard have to be updated?
There do seems to be some werid HEAD tags in places causing some of the sandboxes to break:

This looks good. I'm still not sure when to use this vs dropdown groups. I think this guidance might have to be updated?

Yep, definitely.

Also, just wondering, is this an example of something that is considered a breaking change because all the select lists in Pinboard have to be updated?

Absolutely, hence the major version bump and the codemod.

There do seems to be some werid HEAD tags in places causing some of the sandboxes to break:

Those are from merge conflicts — but not anything an IDE would recognize as such because the code is "just a string". This is the biggest reason I'm excited about the new code editor! A functional UI is nice, but using real code for all these examples will catch so many errors like this.


Remove third party directory from package distribution

Packaging settings at pyproject.toml and setup.py has been change in version 0.2.0 to include public headers in package distribution. This results in occasional including third_party/eigen as well. This pull request fixes the issue.
# Before
ml_dtypes
ml_dtypes-0.2.0.dist-info
ml_dtypes.libs
third_party
#After
ml_dtypes
ml_dtypes-0.2.0.dist-info
ml_dtypes.libs

Wheel size decreased by 800 kb (from 1 Mb).

I believe that the third_party directory was deliberately included in the package build as part of https://github.com/jax-ml/ml_dtypes/commit/1bfd097e794413b0d465fa34f2eff0f3828ff521
I don't think there's any specific test coverage for what that PR accomplished – @cantonios can you weigh-in here? Is there a way for us to reduce the size of the wheel while keeping the headers that you added previously?

May be I didn't catch up the 1bfd097e794413b0d465fa34f2eff0f3828ff521 commit correctly. If you want distribute not only ml_dtypes/include but also third_party/eigen then I'd suggest to put eigen in ml_dtypes/eigen subtree (see PyTorch as an example). However, I think that it is not a good idea.

I think it's only the header files that are meant to be distributed.

We (TensorFlow) don't need the Eigen subtree, but we do need the ml_dtypes headers in the package.  This change should be fine for our use-case.

then I'd suggest to put eigen in ml_dtypes/eigen subtree (see PyTorch as an example). However, I don't think that eigen redistribution is a good idea.

If users use the ml_dtypes headers in this package, then technically they need the same Eigen distribution used to build the package (or risk build breakages, ODR issues).  So if we don't distribute the Eigen headers, then we may want to distribute a metadata file containing the exact commit.

If users use the ml_dtypes headers in this package, then technically they need the same Eigen distribution used to build the package (or risk build breakages, ODR issues). So if we don't distribute the Eigen headers, then we may want to distribute a metadata file containing the exact commit.

I don't think this is a problem, because the .so file being shipped as part of this package doesn't (definitely shouldn't) expose any Eigen symbols. So the header is a completely self-contained thing if the user builds with it.

I don't think this is a problem, because the .so file being shipped as part of this package doesn't (definitely shouldn't) expose any Eigen symbols. So the header is a completely self-contained thing if the user builds with it.

You're right about symbols - I verified the binary doesn't expose anything that might conflict.  There's still a potential issue for methods in float8.h that use Eigen methods: if any of those Eigen methods are renamed, you get a build break.  But thinking about this now, if you're going to try to link against a release version of this library, you could always go back to the corresponding branch and check the Eigen git submodule.
Conclusion: this PR is fine as is.


Draft a reference tree

Pull all sequences
Build a tree
Subsample tree to get representatives within each clade, maybe using augur filter, maybe using smot

Pick a reasonable outgroup
Pick whole genome vs gene alignment, the overhang will result in worse P scores

https://docs.nextstrain.org/projects/nextclade/en/stable/user/algorithm/07-quality-control.html#private-mutations-p

Tried the live build trees, dig into:
Error message: Error: When initializing Nextclade runner: When preprocessing reference tree node NODE_0000097: amino acid mutation M:D89E is outside of the peptide M (length 75). This is likely an inconsistency between reference tree, reference sequence, and gene map in the Nextclade dataset
Nextclade version 2.14.1 (commit: 85e00e8, branch: release)
Memory available: 3586 MBytes
User agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Browser details: {"browser":{"name":"Chrome","version":"<IP_ADDRESS>"},"os":{"name":"macOS","version":"10.15.7","versionName":"Catalina"},"platform":{"type":"desktop","vendor":"Apple"},"engine":{"name":"Blink"}}
Call stack:
Error: When initializing Nextclade runner: When preprocessing reference tree node NODE_0000097: amino acid mutation M:D89E is outside of the peptide M (length 75). This is likely an inconsistency between reference tree, reference sequence, and gene map in the Nextclade dataset
    at M (https://clades.nextstrain.org/_next/static/chunks/68.daad1778960b5734.js:1:15774)
    at https://clades.nextstrain.org/_next/static/wasm/cf835b4f9fbcf56c.wasm:wasm-function[1547]:0x1e3d5c
    at new a (https://clades.nextstrain.org/_next/static/chunks/68.daad1778960b5734.js:1:9424)
    at https://clades.nextstrain.org/_next/static/chunks/68.daad1778960b5734.js:1:24056
    at u (https://clades.nextstrain.org/_next/static/chunks/444.170c49d0571b2ba1.js:1:28086)
    at Generator._invoke (https://clades.nextstrain.org/_next/static/chunks/444.170c49d0571b2ba1.js:1:29376)
    at a.<computed> [as next] (https://clades.nextstrain.org/_next/static/chunks/444.170c49d0571b2ba1.js:1:28489)
    at p (https://clades.nextstrain.org/_next/static/chunks/68.daad1778960b5734.js:1:21841)
    at d (https://clades.nextstrain.org/_next/static/chunks/68.daad1778960b5734.js:1:22038)
    at https://clades.nextstrain.org/_next/static/chunks/68.daad1778960b5734.js:1:22097

Test superscript and subscript in order to reference more information 1
1 https://gist.github.com/bt5e/7507535

Double check

https://github.com/j23414/nextclade_dengue/blob/main/all/genemap.gff#L5

Against

https://github.com/nextstrain/dengue/blob/main/config/clades_genotypes.tsv


[CP] DevTools 2.31.1

Commit(s) to merge
TBD
Target
stable
Prepared changelist for beta/stable
TBD
Issue Description
Release a cherry-pick release of DevTools into the Dart / Flutter SDKs that updates the deep link validation tool
What is the fix
/
Why cherry-pick
TBD
Risk
low
Issue link(s)
TBD
Extra Info
No response

@jacob314 for review.  This is the cherry-pick that I mentioned earlier today.

@hangyujin - I'm assuming we can close this issue in favor of https://github.com/dart-lang/sdk/issues/54834?


Show example images of contrasts that are to be selected by a few functions

Functions such as sct_deepseg_sc, sct_register_to_template, etc. require an argument "-c" which corresponds to the image contrast. It might not be obvious for users how the image should look like, for a given contrast.
For this reason, it would be advisable to add examples of contrasts in the documentation and to refer to it in the script usages.
Example:
t1

t2

t2s

dwi

Something similar to the note in the Segmentation tutorial, then?

Substitutions might be good for this.

reST supports “substitutions” (ref), which are pieces of text and/or markup referred to in the text by |name|.

If you want to use some substitutions for all documents, put them into rst_prolog or rst_epilog

So, we could define, say, |contrast_note| and then use that wherever we want to include the note.

ah! it's already there! cool. Then in that case we just need to redirect to this in the functions' usage.  I was just talking with a user who had not read the doc online, but who had read the usage.

Good idea. There should definitely be a link between the basic usage and the in-depth tutorial.
Mind you, that's just for spinal cord segmentation. As you rightfully pointed out, the -c option is relevant for other SCT commands too (e.g. sct_register_to_template).
I'm working on the registration to template tutorial right now, so I'll definitely keep this in mind as I'm writing. I'll try to ensure that the docs are structured in a way that makes it easy for folks to arrive at related sections.


Registering missing institution_id entries for input4MIPs

Just linking across repos following https://github.com/PCMDI/input4MIPs_CVs/issues/8.
We need to register CNRM-CERFACS, IACETH, IAMC, ImperialCollege, MOHC and new institution CR (see https://www.climate-resource.com/)
This has a matching issue https://github.com/PCMDI/input4MIPs_CVs/issues/8

We  might require this only if "institution" were really a consortium, or if the "institution" was only a simple institution, it could be set to "NONE"

These can now be added via an issue template as of https://github.com/PCMDI/mip-cmor-tables/pull/49


Add istio installation number counter

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

/test all


Integrate pattern matching Patterns tightly into Javaslang in form of unapply()

Steps to make types directly pattern-matchable

[ ] Introduce Pattern.unapply() - Currently Pattern extends Function, i.e. it has apply(). This clashes with existing apply() methods of Value implementations. Therefor Pattern should not extend Function. Instead of apply() it should offer unapply() (which has to be unique for all patterns).
[ ] Let all Javaslang Values and Tuples extends some Pattern0..8 - Then the types can be passed to Match Cases. Pattern is a class. Java classes can have only one super-class. That should be no problem because our types are a tree of interfaces with final class leaves. However, we need to solve that the static of(...) methods need to return a specific impl type (like List.Cons, which is a Pattern1, and List.Nil, which is a Pattern0). The aliases API.* return also specific types. List() returns List.Nil (by calling List.empty()) and List(...) return List.Cons (by calling List.of(...)). What return aliases for interfaces like Seq() and Map()? Maybe they need to return impls, i.e. Vector and HashMap. This still needs to be investigated/solved.
[ ] Add deprecation-support to javaslang-match - The annotation processor needs to recognize deprecated types and deprecated methods. The generated source needs to reflect deprecated types and members.
[ ] Deprecate javaslang.$ (and all of its methods) - With the point above this will automatically deprecate the generated javaslang.Patterns type and all of its methods. (javaslang.$ will be removed in 3.0.0)

Driver for this issue was:

we have Patterns containing static Pattern factory methods Some(T), None, ...
now we have API containing static object factory methods Some(T), None

These clash are redundant.
Solution: We fuse Javaslang types with Patterns by adding unapply() methods to the types.
Goals:

deprecate javaslang.$ and javaslang.Patterns classes
let ADT implementations implement unapply(), which returns a Pattern (needed for pattern matching)

Impl details:

maybe the instance method unapply() will require us to make most of our collection classes interfaces (like HashMap, Vector, ...). Cons will have head, tail, Empty.unapply will return Pattern0. The existing classes cannot implement both.
The static API factory methods need to return sum types, e.g.

static <T, Z extends Option<T> & Pattern0<T>> Z Some(T t) { return Option.some(t); }

enhance javaslang-match annotation processor to recognize @Deprecated classes and methods and use these also in the generated code

Scala has tightly integrated ADTs, pattern matching etc. into the language. Sealed traits and case classes are used for these purposes:
sealed trait List[+A]
case object Nil extends List[Nothing]
case class Cons[+A](head: A, tail: List[A]) extends List[A]

Scala has apply/unapply methods that are needed to construct objects from parts and deconstruct objects into parts (informally):
def apply(head: A, tail: List[A]) = new List(head, tail)
def unapply(list: List[A]) = (list.head, list.tail)

In Java these features are missing on the language level. Javaslang implements the companion functionality (apply/unapply) in form of static helper methods:
interface List<T> {

    static List<T> empty() { return Nil.instance(); }
    static List<T> of(T...) { ... };

    T head();
    List<T> tail();

    static final class Cons<T> implements List<T> {
         ...
        Pattern2 unapply(); // TODO: add generic types here
    }

    static final class Nil<T> implements List<T> {
        ...
        Pattern0 unapply(); // TODO: add generic types here
    }
}

Please note that the static factories return the interface type, not the implementation types.

Facts:
Match Cases take Patterns.
Match(...).of(
    Case(pattern1, v1 -> ...),
    Case(pattern2, (v1, V2) -> ...)
)

where the Pattern hierarchy currently looks like this:
interface Pattern<T, R> extends java.util.Function<T, R> {}

// a pattern that matches a value and returns that value
abstract class Pattern0<T> extends Pattern<T, T> { ... }

// a pattern that matches a value and extracts one part
abstract class Pattern1<T, T1> extends Pattern<T, T1> { ... }

// a pattern that matches a value and extracts one part
abstract class Pattern2<T, T1, T2> extends Pattern<T, Tuple2<T1, T2>> { ... }

Some observations:

All Patterns are Functions that implement apply().
The patterns need to be (abstract) classes because having interfaces would lead to ambiguities: To the compiler all Case(Pattern*) variants would look the same because of type erasure.

More facts:

The compile time annotation processor generates our Patterns. These are upper-case methods like Some(T) and None().
The new API aliases overlap with the existing pattern names. For example we also have Some(T) and None().

Goals:

We simplify the above by letting our types implement Pattern directly. More specifically we need to extends one of the abstract classes Pattern1..8.
Here comes our interface-hierarchy to the rescue. Only the end-points in our type tree are (final) classes. That means we are able to extends one of Pattern1..8.
Because we extends Pattern (which is a Function and has apply()), there currently will be clashes. Seq, Set and Map also extends Function. Therefore we need to change Pattern in a non-backward compatible way - we will remove the Function interface from the hierarchy and rename apply() to unapply().

interface Pattern<T, R> {
    Option<R> unapply(T t);
}

Note: In 3.0.0 we should add a PartialFunction to the hierarchy that allows us to check, if it is possible to call unapply() - e.g. with isDefinedAt(T) as in Scala. This will save several Option instances, we could return R directly.
Changes:

as mentioned above, Pattern needs unapply() instead of apply()
the final classes of the type hierarchy need to extends one Pattern, e.g. Some/None, Success/Failue, List.Cons, List.Nil, etc.
The static factory methods of our types need to return the most specific type, e.g. Option.some(T) needs to return a Some<T> instead of a Option<T>, etc.
The aliases need also to to return the most specific type (see above)

Note: We did not return the most specific type to enable the project derive4j to use Javaslang in conjunction with the visitor pattern, as implemented in that project. However, the pattern matching will not work if we do not return a Pattern type, so we need to adjust that.

Obsolete. We can't make Vavr Values implement Pattern*


Add plugin scaffold

What was wrong ?

#3

How it was fixed ?

Added placeholder plugin as described here: https://docs.pretix.eu/en/latest/development/api/placeholder.html#placeholder-registration

To Do

[ ] Maybe replace with some static text (instead of the link) for the scaffold.

Hi @ligi , I am not sure which link you mentioned that should be replaced in #3

Hi @ligi , I am not sure which link you mentioned that should be replaced in #3

the main purpose of this plugin to replace a placeholder with a link generated by the alphawallet tool - this is the link I refer to in the issue

the main purpose of this plugin to replace a placeholder with a link generated by the alphawallet tool - this is the link I refer to in the issue


Fixed various method signatures in CoreGraphics.

These would previously result in linking errors.

Hi @triplef, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;


Add the ability to catch certain types of errors

Sorry @kwm4385 but I have one more error-related PR for you.
Some errors might be anticipated, so this adds the ability to catch those errors and not display them to the user.

I'm already doing this here https://github.com/HubSpot/Singularity/blob/3ad0867db1cc6ff83c33e4ba36c5c64cc6931b9d/SingularityUI/app/actions/api/base.es6#L83
with options.successResponseCodes but looking back it makes no sense to still return success when that's not the case. Maybe we can integrate the two so that your code uses options.successResponseCodes instead, then remove that from the success condition.

Yeah, that's what I was noticing. It actually looks like options.successResponseCodes is not even used anywhere, so we might be able to get rid of it. Alternatively, it could make sense to have both - in some cases certain response codes may be desired, wheras in others they're an expected error.

It actually is being used, despite the naming being a bit misleading. See FetchRequestRunHistory for example

Yeah, it probably makes more sense for the uses of those that I saw to catch those responses but not count it as success - since 404 isn't successful.
It might be more effort than it's worth to refactor all of those right now, but the request and deploy forms depend on getting the 400 back as an error. So maybe leave them both in for now.

It won't break anything I don't think for the existing stuff to get an error back instead of success when using successResponseCodes. I'd be happy just doing it your way, and I can fix any side-affects that may come up.

Yeah feel free to rename it.

Done

Away we go!


Installation fails with errors in notify_linux.go

Hey there, just tried installing this on Linux and got the following errors:
 △ ~ go install github.com/jangxx/SchubsiGo@latest
go: downloading github.com/jangxx/SchubsiGo v1.2.0
go: finding module for package github.com/GeertJohan/go.rice
go: finding module for package github.com/cratonica/trayhost
go: finding module for package github.com/shibukawa/configdir
go: finding module for package github.com/jangxx/go-poclient
go: finding module for package github.com/esiqveland/notify
go: finding module for package github.com/gorilla/mux
go: finding module for package github.com/skratchdot/open-golang/open
go: finding module for package github.com/godbus/dbus
go: downloading github.com/esiqveland/notify v0.11.0
go: downloading github.com/GeertJohan/go.rice v1.0.2
go: downloading github.com/jangxx/go-poclient v0.0.0-20200509114043-da642efce3a2
go: downloading github.com/cratonica/trayhost v0.0.0-20150112162955-98495206fd96
go: found github.com/GeertJohan/go.rice in github.com/GeertJohan/go.rice v1.0.2
go: found github.com/cratonica/trayhost in github.com/cratonica/trayhost v0.0.0-20150112162955-98495206fd96
go: found github.com/esiqveland/notify in github.com/esiqveland/notify v0.11.0
go: found github.com/godbus/dbus in github.com/godbus/dbus v4.1.0+incompatible
go: found github.com/gorilla/mux in github.com/gorilla/mux v1.8.0
go: found github.com/jangxx/go-poclient in github.com/jangxx/go-poclient v0.0.0-20200509114043-da642efce3a2
go: found github.com/shibukawa/configdir in github.com/shibukawa/configdir v0.0.0-20170330084843-e180dbdc8da0
go: found github.com/skratchdot/open-golang/open in github.com/skratchdot/open-golang v0.0.0-20200116055534-eef842397966
go: downloading github.com/godbus/dbus/v5 v5.0.3
# github.com/cratonica/trayhost
In file included from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/platform.h:10,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/trayhost.go:43:
<EMAIL_ADDRESS>In function ‘tray_icon_on_menu’:
.go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/linux/tray.c:62:5: warning: ‘gtk_menu_popup’ is deprecated: Use '(gtk_menu_popup_at_widget, gtk_menu_popup_at_pointer, gtk_menu_popup_at_rect)' instead [-Wdeprecated-declarations]
   62 |     gtk_menu_popup(GTK_MENU(menu), NULL, NULL, NULL, NULL, 0, gtk_get_current_event_time());
      |     ^~~~~~~~~~~~~~
In file included from /usr/include/gtk-3.0/gtk/gtklabel.h:34,
                 from /usr/include/gtk-3.0/gtk/gtkaccellabel.h:35,
                 from /usr/include/gtk-3.0/gtk/gtk.h:33,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/linux/tray.c:6,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/platform.h:10,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/trayhost.go:43:
/usr/include/gtk-3.0/gtk/gtkmenu.h:138:12: note: declared here
  138 | void       gtk_menu_popup                 (GtkMenu             *menu,
      |            ^~~~~~~~~~~~~~
In file included from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/platform.h:10,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/trayhost.go:43:
<EMAIL_ADDRESS>In function ‘create_tray_icon’:
.go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/linux/tray.c:73:5: warning: ‘gtk_status_icon_new_from_pixbuf’ is deprecated [-Wdeprecated-declarations]
   73 |     tray_icon = gtk_status_icon_new_from_pixbuf(pixbuf);
      |     ^~~~~~~~~
In file included from /usr/include/gtk-3.0/gtk/gtk.h:275,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/linux/tray.c:6,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/platform.h:10,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/trayhost.go:43:
/usr/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:86:23: note: declared here
   86 | GtkStatusIcon        *gtk_status_icon_new_from_pixbuf    (GdkPixbuf          *pixbuf);
      |                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/platform.h:10,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/trayhost.go:43:
.go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/linux/tray.c:76:5: warning: ‘gtk_status_icon_set_tooltip_text’ is deprecated [-Wdeprecated-declarations]
   76 |     gtk_status_icon_set_tooltip_text(tray_icon, menu_title);
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/gtk-3.0/gtk/gtk.h:275,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/linux/tray.c:6,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/platform.h:10,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/trayhost.go:43:
/usr/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:137:23: note: declared here
  137 | void                  gtk_status_icon_set_tooltip_text   (GtkStatusIcon      *status_icon,
      |                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/platform.h:10,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/trayhost.go:43:
.go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/linux/tray.c:77:5: warning: ‘gtk_status_icon_set_visible’ is deprecated [-Wdeprecated-declarations]
   77 |     gtk_status_icon_set_visible(tray_icon, TRUE);
      |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/gtk-3.0/gtk/gtk.h:275,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/linux/tray.c:6,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/platform/platform.h:10,
                 from .go/pkg/mod/github.com/cratonica/trayhost@v0.0.0-20150112162955-98495206fd96/trayhost.go:43:
/usr/include/gtk-3.0/gtk/deprecated/gtkstatusicon.h:151:23: note: declared here
  151 | void                  gtk_status_icon_set_visible        (GtkStatusIcon      *status_icon,
      |                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~
# github.com/jangxx/SchubsiGo
.go/pkg/mod/github.com/jangxx/!schubsi!go@v1.2.0/notify_linux.go:29:28: cannot use conn (type *"github.com/godbus/dbus".Conn) as type *"github.com/godbus/dbus/v5".Conn in argument to notify.New
.go/pkg/mod/github.com/jangxx/!schubsi!go@v1.2.0/notify_linux.go:35:21: notifier.ActionInvoked undefined (type notify.Notifier has no field or method ActionInvoked)
.go/pkg/mod/github.com/jangxx/!schubsi!go@v1.2.0/notify_linux.go:75:3: cannot use actions (type []string) as type []notify.Action in field value
.go/pkg/mod/github.com/jangxx/!schubsi!go@v1.2.0/notify_linux.go:76:3: cannot use map[string]"github.com/godbus/dbus".Variant{} (type map[string]"github.com/godbus/dbus".Variant) as type map[string]"github.com/godbus/dbus/v5".Variant in field value
.go/pkg/mod/github.com/jangxx/!schubsi!go@v1.2.0/notify_linux.go:77:3: cannot use int32(5000) (type int32) as type time.Duration in field value

Looks like something has changed in the current version of trayhost and the proper package versioning wasn't released yet. I will take a look as soon as I get around to it. In the meantime you can use the prebuilt binary, which was built before the breaking change was introduced.

I'm also experiencing the same issue
any update? it's the only single working pushover client I have found on linux
thank you so much!

Okay wow, I completely forgot about this issue.
Dusted off my Go skills again and replaced the broken package with a working one. Can you test it to see if the current version works for you? If it does I'm gonna tag it as the proper 1.3.0 release.

Since no one has responded for a full week now, I'm going to consider this issue as fixed in version 1.3.0.


Suggestion: tree-shakable icons

The final size of pack has a significant impact at application bundle size.
It would make a great impact if we can implement tree-shakable icons using svg-to-ts for example.
https://www.npmjs.com/package/svg-to-ts

This repo is not maintained anymore


Use codecov for coverage badge

Coveralls was having some issues and reporting 0% coverage.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@0e463e2). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #470   +/-   ##
==========================================
  Coverage           ?   84.83%
==========================================
  Files              ?       69
  Lines              ?     6720
  Branches           ?        0
==========================================
  Hits               ?     5701
  Misses             ?      757
  Partials           ?      262

Flag
Coverage Δ

unittests
84.83% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out 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 0e463e2...276c77d. Read the comment docs.


cycle times: sum up all recent deltas, remove a recent cycle time

this will tell us how much time we have made/lost in the past x matches

we also should be able to pick a recent cycle time to remove from calculations
Q6: took 3:40 to begin match; 0:20 faster
Q7: took 2:36 to begin match; 1:24 faster
Q8: took 9:10 to begin match; 5:11 slower
Q8: took 0:19 to begin match; 3:41 faster
in the above case, we would remove the second instance of q8 since we had to do a quick restart


Internet shortcut apps don't show up on being created because the file is being written to

This bug is related to the changes in this PR - #4960.
Steps to reproduce

Install a game using steam or epic games.

Expected behavior

The app should show up.

Actual behavior

The app does not show up sometimes because it is being written to line by line, by the app that is creating it. Adding a try catch block and retrying up to 5 times did not fix it.

Screenshots

We just released 0.20 and this should be resolved there.  Please try it out.


feat(DMP): bring back the workspace

resolves DSP-1570
The branch was checked-out from the develop branch and updated from the main branch

@subotic What's the timeline for this reactivation?


Make it easier to use a custom user model with user create/edit/account settings forms

Is your proposal related to a problem?

When a project creates a custom user model where first_name and last_name are optional, the Wagtail forms still display the fields as being required. In order to make the form fields optional, the developer must override the WAGTAIL_USER-EDIT_FORM and WAGTAIL_USER_CREATION_FORM with custom-defined models. They must also somehow override the Account Settings form for the user profile section (which I am still not clear how to do).
At the very least, the process is somewhat ambiguous, despite the available documentation:

https://docs.wagtail.io/en/stable/advanced_topics/customisation/custom_user_models.html
https://docs.wagtail.io/en/stable/advanced_topics/customisation/custom_account_settings.html

Describe the solution you'd like

Ideally, the forms would introspect the custom user model to determine whether the fields are required. Furthermore, it might be good to simplify the development process and just allow the user to specify a list of fields to render in the forms rather than creating three custom forms (create/edit/account settings).
Describe alternatives you've considered

The documentation could be improved around how to customize the user account settings form.
Additional context

We have a downstream issue where it became apparent how confusing it is to make changes to the user create and edit forms, as well as a follow-up to split the work:

https://github.com/brylie/wagtail-social-network/issues/18
https://github.com/brylie/wagtail-social-network/issues/27

I am also trying to customize the user account settings form to make the fields optional and finding myself at a loss as to how to proceed.

After digging in a bit more in our downstream issue, @Upasanadhameliya mentions:

https://github.com/wagtail/wagtail/blob/b349ba436eeb853a8c2b2ff749f173a1f7d26d59/wagtail/admin/views/account.py#L205
In the link above. they are providing the functionality of adding extra panels to the existing panels as written in the documentation. However they haven't provided the code to modify existing panels.

Yes, my user model has a single name, not first_name and last_name. When the account view was multiple forms, I used the construct_account_menu_items hook to modify the url for the change name view.
How do I replace/modify the NameEmailSettingsPanel now? The only solution I can see is to override the entire wagtail.admin.views.account.account view, which is I hope is temporary until someone adds the ability to change these panels.

If at least django.contrib.auth.base_user.AbstractBaseUser.REQUIRED_FIELDSis respected in making these fields optional, then I can use css to hide first_name and last_name, although it's not ideal.

There's already a settings.WAGTAIL_USER_CUSTOM_FIELDS, used in wagtail.users.forms.UserCreationForm and wagtail.users.forms.UserEditForm, so fixing wagtail.admin.forms.account.NameEmailForm is easy.

@helderco, could you give an example of using WAGTAIL_USER_CUSTOM_FIELDS to customize the UserCreationForm and UserEditForm? Resolving this issue might only need an update to the Wagtail docs.
cc: @gasman

No, not the docs. The current wagtail.admin.forms.account.NameEmailForm would have to be adapted to support settings.WAGTAIL_USER_CUSTOM_FIELDS and django.contrib.auth.base_user.AbstractBaseUser.REQUIRED_FIELDS.

I've just realised that I'm using my own overrides in settings.WAGTAIL_USER_EDIT_FORM and settings.WAGTAIL_USER_CREATION_FORM because the default form (wagtail.users.forms.UserEditForm) adds the fields settings.WAGTAIL_USER_CUSTOM_FIELDS, it doesn't replace the default first_name and last_namefields.

Same problem here.
Our userModel has an extra field for “suffix” because Dutch names can look like Jan van Schaffelaar.
So this field should show up in the NameEmailPanel.
The only possible way to achive this is to override the comlete view...

I'm having the same issue as above; having a hard time modifying (or even removing) the NameEmailSettingsPanel

Found hack, though it's not ideal. In the app ready signal:
from django.apps import AppConfig

class MyApp(AppConfig):
    def ready(self) -> None:
        from wagtail.admin.views.account import NameEmailSettingsPanel

        NameEmailSettingsPanel.is_active = lambda self: False
        return super().ready()

This cases this panel to be filtered out (see source).

This would be a good addition to the Wagtail Docs, under "Customizing the Settings Menu: Removing or Modifying User Settings Panel" or something similar :-)

@brylie PR submitted! #11670

Hi!
Just having trouble because our app does not include the last_name field in the User model. I imagine that following this documentation I should solve this, and I'll try, but it looked like so much of a hassle for something that's probably a common use case, that I ended up here when looking for simpler ways to solve it.
The current behavior that I'm trying to solve is that when a user edits its own profile, there's the "Last name" field (required) which obviously doesn't display it (we don't have it in the model) neither it saves it.
Same would happen when adding or editing other users ofc, but we've removed the access to Users and Groups altogether because we want them to manage that in the django's admin panel.
I hope this is information is useful, and thank you very much!

Thank you @perepicornell for raising it again. I think this issue is still valid. Our docs say that:

"The custom user model must at minimum inherit from AbstractBaseUser and PermissionsMixin."

Yet the following code:
https://github.com/wagtail/wagtail/blob/246f3c7eb542be2081556bf5334bc22256776bf4/wagtail/users/forms.py#L29-L30
incorrectly expects email, first_name, last_name, all of which come from AbstractUser (not AbstractBaseUser).
is_superuser and groups come from PermissionsMixin, which we already state as required, so they're OK.
I think we should change standard_fields so that it only adds those three fields when they are available on the user model.


inappropriate subclasses of 'abiotic stress trait'

The following are currently subclasses; for many, a different relation (e.g. causal) should be used.


Badges : Add goreportcard and coveralls community badges

Signed-off-by: Nicolas PLANEL<EMAIL_ADDRESS>

run tests

I updated them as I think would be good to gain visibility by adding Skydive in https://github.com/avelino/awesome-go
I had already submit a PR and in the process they ask a covertool report, so I propose to re-use coveralls or switch to another one. Let's discuss about, it's a quite small gap.

I will cleanup the last trace of coveralls in this patch

run tests


Error handling

Unknown command:

request:
{"command": "unknown123"}
response:

{"id":0,"error":"7 bad_cast_exception: Bad Cast\ninvalid name 'unknown123' in enum 'keychain_app::command_te'\n    {"key":"unknown123","enum":"keychain_app::command_te"}\n    exception.cpp:226 throw_bad_enum_cast"}
{"id":0,"error":"Bad Cast"}

The error name is "Bad Cast" which doesn't reflect the error type. Would me more clear if the error was called like "Unknown command".

If not valid json passed, the app crashes

request:
sdfsdfsd
response:

terminate called after throwing an instance of 'std::runtime_error'
what():  Parse error: common error while parsing command - unexpected symbols
[1]    25152 abort (core dumped)  ./keychain.ubuntu

if empty json passed:

request:
{}
response:
{"id":0,"error":"method is not implemented"}
Proposed format of errors:
https://github.com/arrayio/array-io-keychain/wiki/WebSocket-API#errors

fixed
{"command":"asdfsa"}
{"id":0,"error":{"code":2,"name":"rpc_command_parse_code","message":"Command cannot be parsed: invalid name 'asdfsa' in enum 'keychain_app::command_te' => cannot parse command","trace":[{"level":"error","file":"exception.cpp","line":239,"method":"throw_bad_enum_cast","timestamp":"2019-02-04T13:43:48"},{"level":"error","file":"keychain.cpp","line":85,"method":"operator()","timestamp":"2019-02-04T13:43:48"}]}}
Json objects are used as message separators. So, if keychain receive bytes outside of json object keychain does not know how interpret this bytes.
{} is equal {"command":0}. Null code is not implemented. What kind of error message would you prefer?


[timeline]vital issue, after use function"group together", when changing chara, timeline will be unable to play.

Many timeline author like to use function "group together" to make interpolables logically assessable. But in hf-patch's timeline, after using "group together" function before I replace the chara, the whole timeline will be messed up. Those interpolables will be blended into together and I can no longer delete replaced chara. The whole plugin will be broken. Actually in original version of timeline by joan6694 have no such a big problem, though his version have many bugs, but none of them are so vital.
In logs there wil be many loop error.
And most importantly, when using latest timeline to make a scene card, such a problem will sustain, even this card is loaded in old version of timeline. That means when such issue happens, the card saved in latest timeline plugin will never be able to replace charater.
NullReferenceException: Object reference not set to an instance of an object
at HSPE.AMModules.DynamicBonesEditor/TimelineCompatibility/<>c.b__1_14 (string,Studio.ObjectCtrlInfo,object) <0x00055>
at Timeline.Interpolable.get_name () <0x00059>
at Timeline.Timeline.ShouldShowInterpolable (Timeline.Interpolable,bool) <0x0008d>
at Timeline.Timeline/<>c__DisplayClass173_0.b__1 (Timeline.LeafNode1<Timeline.Interpolable>) <0x00031> at Timeline.Tree2<Timeline.Interpolable, Timeline.Timeline/InterpolableGroup>.Any (Timeline.GroupNode1<Timeline.Timeline/InterpolableGroup>,System.Func2<Timeline.LeafNode1<Timeline.Interpolable>, bool>) <0x00151> at Timeline.Timeline.UpdateInterpolablesViewTree (System.Collections.Generic.List1<Timeline.INode>,bool,int&,int&,single&,int) <0x009fc>
at (wrapper dynamic-method) Timeline.Timeline.DMD<Timeline.Timeline..UpdateInterpolablesView> (Timeline.Timeline) <0x0013e>
at Timeline.Timeline.RemoveInterpolables (System.Collections.Generic.IEnumerable`1<Timeline.Interpolable>) <0x002d5>
at (wrapper dynamic-method) Timeline.Timeline.DMD<Timeline.Timeline..Update> (Timeline.Timeline) <0x00605>
(Filename:  Line: -1)

For me, after "group together" I can replace characters without any problem. I'm also using the latest hf-patch's timeline.
Maybe you need to provide more specific information to reproduce the issue? or a scene file

https://mega.nz/file/wssHmCZL#DcONfEMKRvJmLCPDU3aOBj5PW67NokKEyPSh8Q1k6VI
this is a charged scene card by yuukis so please don't leak it elsewhere.

there are some other bugs which is not that vital. Such as after "group together", when I tried to ungroup them, I can't just "parent  to nothing" them. After "parent to nothing" them, there will be 2 same interpolables at same time, one in group, one outside the group. After save and load there will be only 1 left, I mean the upper one. That makes ungroup process very complicated.

Seems that the same issue can be reproduced even with Joan's original 1.1.0 version.

No, maybe it is me who didn't describe it clearly. the broken scenes, whatever what version of timeline the "who load the scene" have, the studio will be broken, old or new, it doesn't make a difference.
But if you are MAKING scene cards in hf's latest version of timeline, and you use the group together function, the scene card will be broken once saved. Thats different. It means the card is broken when make in latest version. In manlymarco's original version, when card is make in same way, this problem will not happen.
There must be something different
between original version and latest version in group together function. There are more and more scene maker are puzzled by this bug.

I know what you're talking about. But below is the process I tested.
I downloaded a timeline scene created nearby after the 1.1.0 release, used "Group together" with Joan's origianl 1.1.0 Timeline, saved it, and then loaded it.
The same issue happened.

Well, to be honest that I am using 1.1.0 to create scenes for long. Here is one of my scene.
Could you have a try on this scene? I can't make same issue happens to it actually.

Timeline 1.1.0(So, there is no Timeline button in the left corner) with the HF patch from 2 years ago.


separate scalar mode tests from full vector tests

As discussed at https://github.com/xtensor-stack/xsimd/issues/756 and at https://github.com/serge-sans-paille/pythran/issues/1999 ,  xsimd is designed to work in scalar mode on all architectures, even those without SIMD vector acceleration. pythran requires xsimd even on "unsupported" arches in order to access scalar mode.
Currently xsimd tests address full functionality and therefore fail on "unsupported" (SIMD-free) architectures.
The tests are defined in XSIMD_TESTS in test/CMakeLists.txt.  It would be useful if that list of tests could be split into scalar-mode and vector-mode tests (e.g. create two lists, XSIMD_SCALAR_TESTS and XSIMD_VECTOR_TESTS), with some kind of new cmake option to switch off XSIMD_VECTOR_TESTS when running on SIMD-free architectures.

I've run for i in *.cpp; do grep XSIMD_NO_SUPPORTED_ARCHITECTURE -q $i || echo $i ; done in test directory and no sensible file appear, which means that we currently disable all (vector) test on unsupported architecture.
I'll add a test that checks that covers all xsimd official functions and mke sure we provide the scalar version.


Update README_cn.md， FastAPI-PaddleSpeech-Audio-To-Text

PR types

PR changes

Describe

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

感谢贡献

辛苦先签署下 CLA 协议
辛苦加一下英文文档


Request for modification to alarm settings, separate from in-range settings.

I would like to request Glucose alarm levels that would be separate from the current Low and High settings.
My assumption is that the current Low and High would be used for one's target range...say, for example, 80-160mg/dL.
With the current setting options, if one has the Alarm Glucose Notification enabled, one would get an alarm for low glucose at 79mg/dL or for high glucose at 161mg/dL.
Given that one may frequently go out of range, but not so severely that it would require an alarm, my suggestion is to add another pair of settings for alarms, themselves, that could be set for the same or different levels, similar to the settings in the actual Libre app.
In this way, the user can set their target range and an alarm range wherein they would be able to A) be alerted if they've reached or exceeded a critical level and B) see when they were in range, out of range or in their perceived danger zones.

I agree with the above, this is a needed option.  + Additionally, please add the function of rising alarm (low+high) where the sound signal will be preceded by e.g. two or three cycles of vibration without sound signal and then the sound alarm is additionally activated.


feat(kbutton): component reskinning [KHCP-8970]

Summary
KButton component reskinning
Ticket: https://konghq.atlassian.net/browse/KHCP-8970
Figma: https://www.figma.com/file/Yze0SWXl5nKjR0rFdilljK/Components?type=design&node-id=0%3A1&mode=dev
Links:

Kongponents sandbox: https://deploy-preview-1730--kongponents-sandbox.netlify.app/#/button
Kongponents docs preview: https://deploy-preview-1730--kongponents.netlify.app/
khcp-ui adoption PR: https://github.com/Kong/khcp-ui/pull/6891
khcp-ui adoption PR preview: https://pr-6891--khcp-ui.cloud-preview.konghq.tech/

PR Checklist

[ ] Conventional Commits all commits follow the conventional commit standards outlined in the main README.
[ ] Tests coverage: test coverage was added for new features and bug fixes
[ ] Docs: includes a technically accurate README

This is looking great! There is one inconsistency that I'm noticing between what's in the browser and Figma. Also, this is nothing that you did wrong, it's just a difference in how Figma handles borders versus the browser.
In Figma, it doesn't add the border-width to the box model, so when we add borders to things it doesn't increase the width/height of a element.
Due to this, all the buttons in Figma are 4px smaller in height than what I'm seeing in the browser. The heights we chose for these buttons also line up with a lot of other components so it's important that we maintain them:

Large should be 40px height
Medium should be 32px height
Small should be 24px height

Looks like we could just reduce the padding on top/bottom to adjust this, or set a fixed height since button labels generally never wrap anyways... Thoughts? Open to chat over Zoom if needed.

Looks like we could just reduce the padding on top/bottom to adjust this, or set a fixed height

I'd prefer to adjust the padding so we don't have to deal with hard-coded heights (especially since they don't link to a token)

I'd prefer to adjust the padding so we don't have to deal with hard-coded heights (especially since they don't link to a token)

I'm open to whatever y'all think is best.

This is looking great! There is one inconsistency that I'm noticing between what's in the browser and Figma. Also, this is nothing that you did wrong, it's just a difference in how Figma handles borders versus the browser.
In Figma, it doesn't add the border-width to the box model, so when we add borders to things it doesn't increase the width/height of a element.
Due to this, all the buttons in Figma are 4px smaller in height than what I'm seeing in the browser. The heights we chose for these buttons also line up with a lot of other components so it's important that we maintain them:

Large should be 40px height
Medium should be 32px height
Small should be 24px height

Looks like we could just reduce the padding on top/bottom to adjust this, or set a fixed height since button labels generally never wrap anyways... Thoughts? Open to chat over Zoom if needed.

Just an update on this: Jason found a way in Figma to exclude borders outside of the box to match browser behaviour so he will do another iteration to make sure spacing/border width values are correct and then we revisit the code again to reflect those changes. After that look and feel in browser and in Figma should match sizing-wise (and, hopefully, everything else-wise).

@portikM Took another look at this... it's looking good, I did notice that the small buttons have the wrong border-radius sizing though. The small buttons should use border-radius-20 or 4px.

Another thing I noticed... and maybe @adamdehaven can weigh in on this:
I saw you were using the all property in your transitions for the buttons. I was under the impression this was really bad for performance because the browser is looking for every property to transition. I was always taught to be very specific about the properties that are transitioning (e.g. background-color, border-color) and to never use all. Should we be doing that here?

Another thing I noticed... and maybe @adamdehaven can weigh in on this:
I saw you were using the all property in your transitions for the buttons. I was under the impression this was really bad for performance because the browser is looking for every property to transition. I was always taught to be very specific about the properties that are transitioning (e.g. background-color, border-color) and to never use all. Should we be doing that here?

Targeting all is typically the norm (although as you mentioned, not ideal) - @portikM if it's easy enough, can you update to just transition the properties we are changing?

Addressed border-radius for small size and transition properties @adamdehaven @two24studios.