openTextDocument(uri: Uri) duplicates document on save

Issue Type: Bug

VS Code Version: 1.58.2 (user setup)
OS: Windows_NT x64 10.0.19043
No extensions installed

Steps:

Set up command and handler function to use. The openTextDocument is given an untitled scheme with a system location:

// extension.ts
function creation() {
    const newFile = vscode.Uri.parse('untitled:' + 'c:\\test.txt');

    vscode.workspace.openTextDocument(newFile).then((document) => {
        vscode.window.showTextDocument(document).then((editor) => {
            if(!editor){
                console.error("Could not show.");
                return;
            }

            editor.insertSnippet(new vscode.SnippetString("Hello"));
        });
    });
}
export function activate(context: vscode.ExtensionContext) {
  const createFileDisposable = vscode.commands.registerCommand('composition-companion.createFile', creation);
  context.subscriptions.push(createFileDisposable);
}

Run command
Hit Save

The file is duplicated and the duplicated file is the one that is saved at the provided location while the one that was originally created remains unsaved.

I rollbacked to the vscode version 1.57.1 and did not encounter this issue. I expect a single document to pop up and it should be the one to save at the location provided.
VS Code version: Code 1.58.2 (c3f126316369cd610563c75b1b1725e0679adfb3, 2021-07-14T22:10:15.214Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10610U CPU @ 1.80GHz (8 x 2304)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
47.76GB (33.42GB free)

Process Argv
--crash-reporter-id 18ff36bc-0ca0-4df0-89f9-5ad152a9aa65

Screen Reader
no

VM
0%

Extensions: none
A/B Experiments
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
vspor879:30202332
vspor708:30202333
vspor363:30204092
vspre833:30321513
vshan820:30294714
vscus158:30321503
vscod805:30301674
bridge0708:30335490

I have the same issue.

This seems like bad creation of a URI, it works for me with: vscode.Uri.file('c:\\Users\\bpasero\\Documents\\ticino-playground\\test.txt').with({ scheme: 'untitled' });
I will update the JSDoc to indicate this.

The path I am using is coming from vscode.workspace.workspaceFolders[0].uri, ant it looks like this: /c:/Users/MyName/code/myproject.
Could that be an issue? I do use the proper .with call for setting the scheme.

One of the things I see is that vscode.Uri.joinPath does not maintain fsPath value.

I rather think that the duplication only happens when editor.formatOnSave is enabled . @bpasero , could you please try it with a formatter?

I had that enabled when trying:

But is there a formatter for .txt? In any case, I do not see the change in file in your recording. Could you try doing it in a way that changes the file's contents as the result of formatting?

I think that would be the responsibility of the author of the issue to give exact steps for me to reproduce?

I am sorry it's so hard...
The issue seems to happen at the conflation of custom extensions that create files (a relatively rare case) and possibly language-specific formatters (not quite common either).
I am not sure what formatters you have available in your environment. Maybe the JavaScript one? I am not very good at JavaScript, but if you create a .js file rather than .txt,  then paste a poorly-formatted JS code in there, it may trigger the formatting. If you happen to have clang-format, I can give more exact instructions.


Add files via upload

@doansondc sis.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


libgdiplus

somewhere in the recent website transition information about apt-get install -y libgdiplus was lost

I don't know if this has been fixed yet, but this should be added to Eto when it's added to the docs. Also probably a note about adding System.Drawing.Common to the .csproj

Using .NET 6.0, macOS 12.1, ScottPlot.Avalonia/4.1.29 I get the following error:
Unhandled exception. System.TypeInitializationException: The type initializer for 'Gdip' threw an exception.
 ---> System.DllNotFoundException: Unable to load shared library 'libgdiplus' or one of its dependencies.

After running brew install mono-libgdiplus as indicated here I get the plot to display but it shows some weird Chinese characters instead of numbers on each axis.

I think installing System.Common.Drawing using dotnet add package System.Common.Drawing will help.

System.Drawing.Common is only supported on Windows.

System.Drawing.Common is not supported on non-Windows platforms

@royalstream this is only true with .NET 6 - the System.Drawing.Common package is supported on Linux and MacOS on .NET Framework 4.6.1-4.8 and .NET Core through .NET 5. A temporary solution could be to target your project for .NET 5 👍
#1036 attempts to solve this for .NET 6 and newer by migrating to Microsoft.Maui.Graphics but that library is not mature yet...

System.Drawing.Common is not supported on non-Windows platforms

this is only true with .NET 6

... Actually, this information should be added to the quickstarts too

I already downgraded to .NET 5.0 and I get the same behavior.
I don't think it's related to missing DLLs because the program compiles and it displays the graphic correctly (under both .NET 6.0 and 5.0).
The problem is only related to the fonts and I strongly suspect it's related to Unicode.
For example, if I add a label to the XAxis:
avaPlot1.Plot.XAxis.Label(label: "ABCDE", fontName: "Arial", size: 24, color: Color.Red, bold: true);
The label displays something like **E instead of ABCDE (where * stands for some weird Chinese character)
Changing the font makes no difference.
If I make the label exactly one character like "A" or "1" it works too.
If I change the label to "123456789" it shows like **9

Hi @royalstream,
This issue sounds very similar to https://github.com/ScottPlot/ScottPlot/issues/1220
Can you open an issue over on https://github.com/ScottPlot/ScottPlot/issues (this repo is just for the website) and include a screenshot and the sample code used to make it? I think there's a sample console program there which can reproduce the issue in just 3 lines https://github.com/ScottPlot/ScottPlot/issues/1220#issuecomment-904233580 - I'd be interested to see the output of that program on your system. Posting the content of your csproj file would be helpful too!

Problem solved. And it works under both .NET 5.0 and .NET 6.0.
I was running into issue 1079.
The following snippet works great as long as I don't add a reference to ScottPlot.
Bitmap bmp = new Bitmap(200, 100);
using var gfx = Graphics.FromImage(bmp);
gfx.Clear(Color.Navy);
Font fnt = new Font("Arial", 18);
gfx.DrawString("test123", fnt, Brushes.Yellow, 10, 10);
bmp.Save("test.bmp");

As you all suspected, System.Drawing.Common was the culprit.
This is the version it uses by default (no reference to ScottPlot):
System.Drawing.Common, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51
/.nuget/packages/system.drawing.common/4.5.0/ref/netstandard2.0/System.Drawing.Common.dll

And this is the version ScottPlot brings to the party (causing the problem):
System.Drawing.Common, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51
/.nuget/packages/system.drawing.common/4.6.1/ref/netcoreapp3.0/System.Drawing.Common.dll

I cannot add a reference to the latest System.Drawing.Common (6.0.0) because I get a PlatformNotSupportedException.
So I tried using version 5.0.2 like they mentioned in the comments and voilá! that did it.
Thank you all.

Awesome, I'm glad you got this to work! Thanks for following-up here and in #1079 with your solution 🚀
System.Drawing can be touchy and I will be happy to leave it behind in a future version of this library 😅


Support bedGraph output in only-depth

Support bedGraph output in only-depth:
This is basically perbase -z output without header and chromosomes sorted by lexographically.
This would allow to create bigwig files relatively easily. (naive sorting takes quite a while instead of reading the reads by chromosome in lexographical order).
perbase only-depth -z ${bam} | tail -n +2 | LC_COLLATE=C sort -k 1,1 -k 2,2n -k 3,3n > ${bedgraph}

bedGraphToBigWig ${bedgraph} ${chrom_sizes} ${bigwig}

bedGraph format:
http://genome.ucsc.edu/goldenPath/help/bedgraph.html

I'll looking into making this more natively supported. There is also the --bed-format option which, with -z should make the output more BED like. The output order is currently dictated by the BAM header, if the input BAM is sorted lexographically then so will the output of perbase (not ideal, I'm just noting it).
Options to make this work would be:

Add a "lexographical sort" flag that sorts the BAM header before processing, this could be fine...but I could see it still resulting in slightly different sort orders than what is needed downstream.
Add support for outputing in the same order as an input BED file or sequence dictionary. This probably provides the most flexibility / explicitness
Add an optional input that is just a list of chromosome names that represent the order things should be output. meh


calico-ipam replaced to kube-ipam  ?

Question : Whether it's when I try to use this kube-ipam  plugin  for fixed my pod IP ,  my cluster  k8s cluster CNI plugin which config and deploy form  offical calico-etcd.yaml  ,version : v3.19.x .
I try to modify /etc/cni/net.d/10-calico.conflist  which created by initContainer : install-cni  like :
{
  "name": "k8s-pod-network",
  "cniVersion": "0.3.1",
  "plugins": [
    {
      "type": "calico",
      "log_level": "info",
...
        "ipam": {
                "name": "kube-subnet",
                "type": "kube-ipam",
                "etcdConfig": {
                        "etcdURL": "https://<IP_ADDRESS>:2379,https://<IP_ADDRESS>:2379,https://<IP_ADDRESS>:2379",
			"kubeConfig": "/etc/kubernetes/pki/kubectl.kubeconfig"
                        "etcdCertFile": "/etc/kubernetes/ssl/etcd.pem",
                        "etcdKeyFile": "/etc/kubernetes/ssl/etcd-key.pem",
                        "etcdTrustedCAFileFile": "/etc/kubernetes/ssl/ca.pem"
                },
                "subnet": "<IP_ADDRESS>/16",
                "rangeStart": "<IP_ADDRESS>",
                "rangeEnd": "<IP_ADDRESS>",
                "gateway": "<IP_ADDRESS>",
                "routes": [{
                        "dst": "<IP_ADDRESS>/0"
                }],
                "resolvConf": "/etc/resolv.conf"
        },
      "policy": {
          "type": "k8s"
      },
      "kubernetes": {
          "kubeconfig": "/etc/cni/net.d/calico-kubeconfig"
      }
    },
...
    {
      "type": "bandwidth",
      "capabilities": {"bandwidth": true}
    }
  ]
}

So if I do this replacement  is work or not ？   Thanks a lot

kube-ipam supports fixing the IP address of pod in calico CNI network. Please see  https://github.com/cloudnativer/kube-ipam/blob/main/docs/kubeipam-with-calico.md for details.
In addition, here is an example 10-calico.conflist:

#
# cat /etc/cni/net.d/10-calico.conflist
{
  "name": "k8s-pod-network",
  "cniVersion": "0.3.1",
  "plugins": [
    {
      "type": "calico",
      "log_level": "info",
      "log_file_path": "/var/log/calico/cni/cni.log",
      "datastore_type": "kubernetes",
      "nodename": "<IP_ADDRESS>",
      "mtu": 0,
      "ipam": {
                "name": "kube-subnet",
                "type": "kube-ipam",
                "kubeConfig": "/etc/kubernetes/ssl/kube.kubeconfig",
                "etcdConfig": {
                        "etcdURL": "https://<IP_ADDRESS>:2379",
                        "etcdCertFile": "/etc/kubernetes/ssl/kubernetes.pem",
                        "etcdKeyFile": "/etc/kubernetes/ssl/kubernetes-key.pem",
                        "etcdTrustedCAFileFile": "/etc/kubernetes/ssl/k8s-root-ca.pem"
                },
                "subnet": "<IP_ADDRESS>/16",
                "rangeStart": "<IP_ADDRESS>",
                "rangeEnd": "<IP_ADDRESS>",
                "gateway": "<IP_ADDRESS>",
                "routes": [{
                        "dst": "<IP_ADDRESS>/0"
                }],
                "resolvConf": "/etc/resolv.conf"
      },
      "policy": {
          "type": "k8s"
      },
      "kubernetes": {
          "kubeconfig": "/etc/cni/net.d/calico-kubeconfig"
      }
    },
    {
      "type": "portmap",
      "snat": true,
      "capabilities": {"portMappings": true}
    },
    {
      "type": "bandwidth",
      "capabilities": {"bandwidth": true}
    }
  ]
}

#
#

So if I do this replacement is work or not ？ Thanks a lot

yes, you can see https://github.com/cloudnativer/kube-ipam/blob/main/docs/kubeipam-with-calico.md for details.
There is an example calico.yaml in the yaml directory. We use this calico.yaml to install calico CNI. Please note the settings of kubeConfig, etcdConfig, subnet and other parameters. subnet and CALICO_IPV4POOL_CIDR parameters should be set to the same value.


Fix vets cache issue

Fix issue #220 by restoring "vets" cache.

Already fixed in 80269539e2 I think. Thanks, though.


Delimiter option not working

Hi,
I try to change the default delimiter with -d ';', the delimiter in CSV file is still ","

Same here.

I've added PR #54 that should fix it.

any news on this?


Fix compilation error introduced with fc5ef6de6 (#1184)

Last-minute rushed changes, never good... :-(
Thanks for the fix!


[build] produce multiple "flavors" of runtime packs

Right now our runtime packs are all identical for each .NET "runtime", but we want to be able to produce different packs, such as:
Microsoft.Android.Runtime.Mono.35.android-arm
Microsoft.Android.Runtime.Mono.35.android-arm64
Microsoft.Android.Runtime.Mono.35.android-x86
Microsoft.Android.Runtime.Mono.35.android-x64
Microsoft.Android.Runtime.NativeAOT.35.android-arm
Microsoft.Android.Runtime.NativeAOT.35.android-arm64
Microsoft.Android.Runtime.NativeAOT.35.android-x86
Microsoft.Android.Runtime.NativeAOT.35.android-x64

Currently, the contents of all these packs are identical, but this is an initial step to build multiple packs.
The plan is for the workload to only install the "Mono" packs, and the "NativeAOT" packs can be restored via NuGet.

I think the main question here is if we are ok with the new naming scheme:

Microsoft.Android.Runtime.Mono.35.[RID]
Microsoft.Android.Runtime.NativeAOT.35.[RID]


Missing binary_sensor for compressor1 & compressor2

Ні,
as in #129…
I am only shown the pulses and operating hours of my two compressors.
A compressor is detected but is displayed as Temperaturen
I am missing the binary sensor for the compressors if and which of the two is running.
id: ID_WEB_VD1out
id: ID_WEB_VD2out

For ID_WEB_VD1out see binary_sensor.luxtronik_*_compressor.
For ID_WEB_VD2out: I will add this in the next prerelease.

See 2023.10.11-Beta

Thanks for the great work. It Workshop great.
I still have one wish. It is about writing…

ID_Einst_Effizienzpumpe_Nominal_akt
ID_Einst_Effizienzpumpe_Minimal_akt
ID_Visi_SysEin_EffizienzpumpeNom
ID_Visi_SysEin_EffizienzpumpeMin

Should I create a new Issue?

Should I create a new Issue?

Yes. Please provide the following sensor informations for the parameters:

English Key
English Name
Unit
Min+Max Value
Step
Perhaps a factor (How to interpret the value "1000"?)
device_class
icon

parameters
867  ID_Einst_Effizienzpumpe_Nominal_akt: 1000
868  ID_Einst_Effizienzpumpe_Minimal_akt: 1000
visiblities
239  ID_Visi_SysEin_EffizienzpumpeNom: 1
240  ID_Visi_SysEin_EffizienzpumpeMin: 1


CI fail

https://xly.bce.baidu.com/baiduapollo/demo-project/newipipe/detail/4097057/job/9362514

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


Data processing

[ ] Remove "End of Document" and any text that comes after it.
[ ] Remove "\nblock-time published-time 3.09am GMT" time stamps.
[ ] Remove "\nblock-time updated-timeUpdated at 3.16am GMT" time stamps.
[ ] Remove "Photograph: Anadolu Agency/Anadolu/Getty Images".
[ ] Remove "\narrtl???? | ????? #???????  : ???? ???????? ?????????? ????? ????? ?????? ???? ????? ??????? ???? ???? ??????? ?????????\n— ??????? - ????". I'm guessing these are HTML embeddings that can't convert to .txt?
 I'm guessing these are embedded tweets?
[ ] Remove "Sign up here for our daily newsletter, First Edition" or references to promotions.
[ ] Take accent marks into consideration (ex: Médecins Sans Frontières, António Guterres, etc.).
[ ] Convert spelling to either American or British English.

Also, for NYT articles, we might need to remove the "Briefings", since they usually combine different topics. They're called day of the week + "Briefing", i.e. "Monday Briefing:", etc.


Rotating private/public keys causes login page to Internal Server Error

If you rotate the private/public keys of a Renarde application then when you visit directly the login page it causes an Internal Server Error by RenardeSecurity#makeRedirectToLogin.
if (request.uri().equals(config.getLoginPage())) {
            // this would cause a redirect loop, not sure how to handle, but not by redirecting
            log.errorf("Redirect loop at %s, giving up on clearing bad JWT cookie", config.getLoginPage());
            return Response.serverError().build();
}

Do you have a way to reproduce this?


Bug: useTheme() reports incorrect type since v1.52.7

Current Behavior
Since v1.52.7, the return type of useTheme() returns a different shape for theme variables that doesn't reflect the actual shape.
useThemeResult.{themeKey}.val is now reported by the types as a nested variable:

But the changes reflected by these types don't seem to be real?
The expression useThemeResult.{themeKey}.val still resolves to the raw value as it did before in previous versions.
Expected Behavior
This was the previous shape of the types:

Tamagui Version
1.52.7 - 1.53.1

Reproduction
This happens when creating themes using colors from tokens:

import { createTheme, createTokens } from '@tamagui/core'

const tokens = createTokens({ /* ... */ })

const lightTheme = createTheme({
  color: tokens.color.foo123,
  // ...
})

System Info
No response

fixed v1.53.2

@natew here's a reproduction in 1.53.2:
https://github.com/nderscore/tamagui-usetheme-val

The fix in v1.54.1 seems to be working 🙏 Thanks Nate!


[IMPROVEMENT] Support FIPS encryption mode for Longhorn volume

Question
I have a requirement to run RHEL 7 in FIPS.  I am trying to encrypt volumes using Longhorn v1.2.4.  Volume mounts fail with the following error:
MountVolume.MountDevice failed for volume "pvc-025b6b62-10fc-473f-a57f-1659015e8c34" : rpc error: code = Internal desc = failed to encrypt device /dev/longhorn/pvc-025b6b62-10fc-473f-a57f-1659015e8c34 with LUKS: failed to run cryptsetup args: [-q luksFormat --type luks2 --hash sha256 /dev/longhorn/pvc-025b6b62-10fc-473f-a57f-1659015e8c34 -d /dev/stdin] output: error: exit status 1
When I try to run the command manually I get the following error:
argon2i key derivation function is not allowed in FIPS mode
Is there a way to not use argon for keys?

cc @mantissahz this is probably related to the cypher change capability in 1.4.0.

REF: this can be done together with https://github.com/longhorn/longhorn/issues/3353

Duplicated as https://github.com/longhorn/longhorn/issues/4829, but leave it for testing purposes.
@mantissahz Even though this was reported by users, could you be able to provide the test steps to let QA verify this issue? or @coreylarew could you provide the steps? Thanks.

Hi @longhorn/qa,
Here are steps to enable FIPS mode on RHEL7/8
For RHEL7:

Install packages
# yum -y install prelink dracut-fips
Disable prelink:
# cat /etc/sysconfig/prelink | grep PRELINKING
PRELINKING=no
# prelink -u -a
Recreate the initramfs file:
$ dracut -f
Edit /etc/default/grub to add fips=1

GRUB_CMDLINE_LINUX="fips=1 console=ttyS0,115200n8 console=tty0 net.ifnames=0 rd.blacklist=nouveau nvme_core.io_timeout=4294967295 crashkernel=auto"

Regenerate grub.cfg:
# grub2-mkconfig -o /boot/grub2/grub.cfg
reboot

For RHEL8:
# fips-mode-setup --enable
# reboot
Verify FIPS is enable
# sysctl crypto.fips_enabled
crypto.fips_enabled = 1

or
# cat /proc/sys/crypto/fips_enabled
1

or
# fips-mode-setup --check
FIPS mode is enabled.

Ref: https://www.dogtagpki.org/wiki/Configuring_FIPS_on_RHEL
Then install the cryptsetup package
# yum install -y cryptsetup
Check default PBKDF:
# cryptsetup --help|grep "Default PBKDF"
RHEL7 Result:
Default PBKDF for LUKS1: pbkdf2, iteration time: 2000 (ms)
Default PBKDF for LUKS2: argon2i

argon2i is not supported for FIPS mode and it caused our volume encryption steps failed.
RHEL8 Result:
Default PBKDF for LUKS1: pbkdf2, iteration time: 2000 (ms)
Default PBKDF for LUKS2: pbkdf2

After these steps, you could setup a general Kubernetes cluster with longhorn system on nodes you enable the FIPS mode for.

Verified passed on master-head (longhorn-manager 15badde) by following the steps  to setup FIPS enabled environments, and then executing the test steps to verify the encrypted volume works or not.
Tested on both FIPS enabled RHEL 7.9 and RHEL 8.6, the workload using the encrypted volume can read/write the volume without problems.

can someone please provide an example for FIPS mode.  I am still having a problem getting the volumes to mount

cc @mantissahz @yangchiu

@mantissahz I notice @coreylarew use
CRYPTO_KEY_CIPHER: "aes-xts-plain64"
instead of
CRYPTO_KEY_CIPHER: "aes-cbc-essiv:sha256"
provided in the example.
Will this affect the result?

@yangchiu
I believe that aes-xts-plain64 could be used in the FIPS environment because I just create a volume with this type in RHEL7 cluster that FIPS mode is enabled
@coreylarew
Could you describe more about you environment? about LH version, and how did you create the volume and mount the volume?
Any abnormal log in longhorn-manager and maybe what does kubectl describe po [your workload] show?
And it would be great if you could provide the support bundle.


CVE - generic push gateway (t-shirt)

So hyuge and undefined, it gets TWO XXL placeholders for now

@rob-h-m
This is specific to Columbus and should move to SmartColumbusOS

Based on conversation with Jarred 9/4, the plan will be to leave this placeholder as is and play a spike to determine how the system will handle large amounts of data and what might need to be enhanced. We'll revisit the t-shirt size after that spike is played.

Original t-shirt size = 55 points
Reducing to 42 points due to the addition of the following cards:
https://app.zenhub.com/workspace/o/smartcitiesdata/smartcitiesdata/issues/944 (5pts)
https://app.zenhub.com/workspace/o/smartcitiesdata/smartcitiesdata/issues/941 (3pts)
https://app.zenhub.com/workspace/o/smartcitiesdata/smartcitiesdata/issues/940 (3pts)
https://app.zenhub.com/workspace/o/smartcolumbusos/scosopedia/issues/259 (2pts)

Reducing t-shirt size to 10 points.
Possible remaining work:

Limiting number of messages that are sent to the OS (If Kapsch can do this, 1-2 point card. If we have to do this in the push gateway, 3-5 point card)
Creating separate datasets for each message type - 2pts

Reducing placeholder to 8pts due to the addition of the following card:

https://app.zenhub.com/workspaces/smartcolumbusos-5d08f55f08ccbe75911ce796/issues/smartcolumbusos/scosopedia/269 (2pts)

Closing, as this was a placeholder card for grant forecasting purposes.


📃: Lunar Cycle - Moon Phase Visibility

🔴 Title : Lunar Cycle - Moon Phase Visibility
🔴 Tech stack : html css vanilla-js
🔴 Objective : This application can displays current date's moon phase, moon-phase for the next 7 days and the date for next new moon.
📸 Screenshots

✅ Details to Include When Taking the Issue:
Name : Ananya Gupta
Participant Role GSSOC, Hacktoberfest

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

@ananyag309 See the guidelines. Contributors are free to make as many changes as they like to the website, but the number of points they can earn from new projects is limited to 150.

thanks for informing this

can you please assign this issue to @Aditijainnn on my behalf

can you please assign this issue to @Aditijainnn on my behalf

thank you @ananyag309
assign this to me


Group messages

So this is example:
id | conversation_id | text | user_id
1 | 1 | Test0 | 1
2 | 1 | Test1 | 1
3 | 1 | Test2 | 1
4 | 1 | Test3 | 2
5 | 1 | Test4 | 2
6 | 1 | Test5 | 1
7 | 1 | Test6 | 1
8 | 1 | Test7 | 1
How to group messages from id 1-3, 4-5, 6-8, so when user sends few messages in row it's a group until another user sends message.

markdown table:

id
conversation_id
text
user_id

1
1
Test0
1

2
1
Test1
1

3
1
Test2
1

4
1
Test3
2

5
1
Test4
2

6
1
Test5
1

7
1
Test6
1

8
1
Test7
1

I would do it in the blade view - save the last user-id and compare it to the current one. If they match don't start a new message block, just a new paragraph.

Can you provide code for that?

@php($lastUserId = null)
@foreach($messages as $message)
    @if($lastUserId != $message->user_id)

    @endif

{{ $message->Text }}

    @php($lastUserId = $message->user_id)
    @if($lastUserId != $message->user_id)

    @endif
@endforeach

It's untested but in my head it works 😅🙄


[WIP] Update lerna config & related cleanup/enhancements

Description
A large number of open source projects use lerna; some of these larger projects include jest, babel, typescript-eslint, webpack/webpack-cli.
Since our adoption of lerna we've had problems with the tool (namely around lerna bootstrap) but we also haven't invested in improving our usage of the tool after 2018.
This PR will:

Move away from custom solutions for lerna-solved problems hand-rolled (e.g. updating version numbers)
Avoid future bugs by updating lerna config (e.g. using --force-local in lerna bootstrap)
Use the same version of lerna locally and on CI machines
Document the ways this repository uses lerna to lower friction when contributing to this project
Prepare for additional monorepo tooling such as yarn (see https://github.com/microsoft/botbuilder-js/issues/1383)

Commits (oldest to most recent)
"lerna focused changes" https://github.com/microsoft/botbuilder-js/commit/3b268b36073185fd9af30e171ecce90ddf111013

add "private": true to test packages

(example from jest)

add lerna "version" config

move away from hand-rolled scripts for updating package versions (WIP: preview versioning is not supported)

add explicit lerna "npmClient" in prep for yarn

Related to https://github.com/microsoft/botbuilder-js/issues/1383 and work in josh/yarn

Pull Request Test Coverage Report for Build 155018

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.07%) to 82.993%

Files with Coverage Reduction
New Missed Lines
%

libraries/adaptive-expressions/src/builtinFunctions/getNextViableTime.ts
1
88.89%

libraries/adaptive-expressions/src/builtinFunctions/getPreviousViableTime.ts
1
87.3%

Totals

Change from base Build 154972:
0.07%

Covered Lines:
15004

Relevant Lines:
17187

💛  - Coveralls


Quick CSS fix to make tables look better in posts

@Floppy this should be quick for you to tackle. Can you please check https://alpha.openaddressesuk.org/blog/2015/02/18/crowdsourcing-challenge-update and make it so that:

cell content is valigned to the top
some border is visible and
if necessary, the table is made generally more readable perhaps playing with cellpadding and cellspacing?

Ta.

I think this can be done in the markdown... will try it.

I believe Peter tried that already, my guess is that the CSS is missing settings for tables.

I've done it, fix is in https://github.com/OpenAddressesUK/theodolite/commit/5811f740ba280b1726a1a2f6eb29b50d92e6c81c


Add library API to C++

It would be nice to add the library to be used in C++ and with port on vcpkg or any other package manager.

Hi @mhco0 , sorry, not sure what you mean. This is a Java library, how do you expect to use this? I believe you can use JNI to do this (see https://docs.oracle.com/javase/7/docs/technotes/guides/jni/spec/invocation.html), but it's a bit beyond the scope of our project at the moment.

Hi @bodiam, hope you doing well. Sorry, I should have express me better, but you're right, JNI already fulfill my needs.
Thanks!


Implement real-time webhook forwarding

subscription webhookCreated to "listen" for new webhook creation events.
Use Apollo Client for React to handle subscriptions

#6 was not linked to this issue, but completed it.

@BuzzBumble Thank you! Closing.


add logo (#501)

Stack:

#28
#27
#26
#25
#24
#23 ⬅
#22
#21
#20
#19
#18
#17
#16

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.


Logstash 5.1.1 - error

Please post all product and debugging questions on our forum. Your questions will reach our wider community members there, and if we confirm that there is a bug, then we can open a new issue here.
For all general issues, please provide the following details for fast resolution:

Version: ELK 5.1.1 with couchdb_changes
Operating System: LinuxMint 18 x64
Config File (if you have sensitive info, please remove it): any config file

input {
  couchdb_changes {
      db => "mydbname"
      host => "<IP_ADDRESS>"
      port => 5984
      initial_sequence => 0 #this is only required for the an initial indexing
  }
}
filter {
  mutate {
    add_field => { "action" => "%{[@metadata][action]}" }
  }
  if [action] == 'delete' {
    elasticsearch {
      hosts => ["<IP_ADDRESS>"]
      query => "_id:%{[@metadata][_id]}"
      fields => ["type", "type"]
      sort => ""
    }
  } else {
    mutate {
      add_field => { "type" => "%{[doc][type]}" }
    }
  }
}
output {
  elasticsearch {
    action => "%{[@metadata][action]}"
    document_id => "%{[@metadata][_id]}"
    host => "<IP_ADDRESS>"
    index => "myindexname"
    protocol => "http"
    port => 9200
  }
  #stdout { codec => rubydebug } #enable this option for debugging purpose
}

Sample Data: any sample data
Steps to Reproduce: start CouchDB, Elasticsearch and Logstash and then look in the log file of logstash.

Please see: https://discuss.elastic.co/t/couchdb-changes-plugin-and-logstash-5-0-1/66600
update line 180 in couchdb_changes.rb to
@sequence = event.get("[@metadata][seq]")

Thank you very much!

this is fixed by #37, that was merged but not released yet

@ph @untergeek @jsvd
Could you please tell if you plan to release soon?
This is preventing us to upgrade from logstash 2.x to 5.x

@dragos-constantin-stoica @YannRobert I've release 3.1.1 on rubygems ,


ECDSA signing

How do I generate the Recovery ID? Ethereum transactions uses v,r,s values how do I generate the value v ?
is there any tests written on this could you please advice and share the line of code ?

Very simple code search for the word recover will find these results:
https://github.com/trezor/trezor-crypto/blob/0c622d62e1f1e052c2292d39093222ce358ca7b0/ecdsa.c#L1028
https://github.com/trezor/trezor-crypto/blob/0c622d62e1f1e052c2292d39093222ce358ca7b0/tests/test_check.c#L3269


Answer to exercise 5.13 - b: ix under case 1 is not initialized

ix under case "1" is not initialized, UB will happen.

Agree, could you please fixed it and send me a pull request?


Use errgroup instead of counting goroutines

What this PR does / why we need it:
Use errgroup instead of counting goroutines for syncing multiple goroutines.
This is much safer for future modification.

/lgtm pending E2E thanks @cpuguy83!


XS✔ ◾ Changes

❌ Add a description.

@bilalbokharee Image is available for testing. docker pull stakater/workshop-exercise:SNAPSHOT-PR-177-85f4990d


Enum values not generated

After migration from Swashbuckle 5.4.1 to DotSwashbuckle 3.0.6 the enums list of available values is no longer generated in OpenAPI v2.
        "parameters": [
          {
            "in": "query",
            "name": "period",
            "description": "period",
            "type": "string",
            "default": "Current",
>>>
            "enum": [
              "Current",
              "Previous"
            ]
<<<
          }

This seems to be a bug indeed, looks like enums are not serialized as enum ref in parameters. I will check

fixed in 3.0.7

Works like a charm. Thank you.


docs: translate to zh-CN

translate docs to Chinese.
Welcome to calibration and review.

Oh, nice! There is a lot of traffic from China in the docs, so I think this would be nice to be integrated in the repository but it will have to be reviewed by some of the core team so it might take some time

Oh, nice! There is a lot of traffic from China in the docs, so I think this would be nice to be integrated in the repository but it will have to be reviewed by some of the core team so it might take some time

Oh, nice! There is a lot of traffic from China in the docs, so I think this would be nice to be integrated in the repository but it will have to be reviewed by some of the core team so it might take some time

Got it.

Oh, nice! There is a lot of traffic from China in the docs, so I think this would be nice to be integrated in the repository but it will have to be reviewed by some of the core team so it might take some time

Got it.

Hi @posva
This PR is readly for review.

Thanks a lot! I will relay the message to get some reviews!

@antfu thanks for your review.I will check the things you mentioned above in each file.

Let's merge this to make it available for all Chinese users and they will be able to propose changes as they see.
I have seen a few usability issues when it comes to language selection but I will have to fix those in Vitepress
Thanks a lot @edison1105 @antfu @meteorlxy


fix typo

Cheers! Incidentally, thanks to your PR I realised the Random test was wrong, I'll fix it ;)


Modern Greek data issues

There are 2 major issues with the Greek data.
They tend to produce µ (micro sign) instead of μ (Greek m letter) and despite choosing Modern Greek (ell), some characters have accents that belong to polytonic Greek.

https://github.com/tesseract-ocr/langdata_lstm/tree/main/ell contains training text and a word list with the same issues, so the model was trained to produce such results.

Right, so how can this be fixed? For example I can see in https://github.com/tesseract-ocr/langdata_lstm/blob/main/ell/desired_characters and https://github.com/tesseract-ocr/langdata_lstm/blob/main/ell/ell.unicharset the existence of polytonic characters which should not be there.

In a first step you could send a pull request for langdata_lstm which fixes the files there. But finally new trainings are required, maybe based on the existing models for Greek.

OK, I may need some guidance please. I created a fork. So do I simply have to remove non-valid characters from above mentioned files?
I also see
tessedit_load_sublangs grc
https://github.com/chopinesque/langdata_lstm_modern_greek/blob/main/ell/ell.config#L2
I am not sure whether this line should be there going forward.

So do I simply have to remove non-valid characters from above mentioned files?

Remove or replace, what fits better.

Thanks. If I replace, I need to know about the structure, for example,
ὶ 3 0,255,0,255,0,0,0,0,0,0 Greek 124 0 124 ὶ	# ὶ [1f76 ]a
How is the 124 0 124 derived?

You can keep the unicharset file unmodified. A replacement will be created when a new training is run.

tessedit_load_sublangs grc

That line tells Tesseract to always use grc in addition to ell. Therefore wrong glyphs can also come from grc as long as that configuration is there.

You can keep the unicharset file unmodified. A replacement will be created when a new training is run.

Not sure then which files I should change. I don't think I have the knowledge to do any training (I also use Windows).

tessedit_load_sublangs grc

That line tells Tesseract to always use grc in addition to ell. Therefore wrong glyphs can also come from grc as long as that configuration is there.

So this line should be removed.


Update Link to RDF Ancillary documents page

Right now the page "Ancillary documents related to the Darwin Core RDF Guide" resides in Google Code at https://code.google.com/p/tdwg-rdf/wiki/DwCAncillary. The Darwin Core RDF Guide has links to this location. When the RDF Task Group Google Code site gets migrated to GitHub, update the links in the Darwin Core RDF Guide to the new location.

I think that I have been assuming that at least part of the content in this document would reside somewhere in the DwC site.  For example, the document about "Sources of IRIs recommended for use as controlled values" was intended to allow us to avoid enshrining those IRIs in the standard itself and allow for more nimble updating.  So for at least that case, we would want people to find those IRIs easily when learning about how to use DwC.  I could also see how the other documents would be relevant to DwC implementers.  I don't know that they would necessarily think of looking in the RDF TG web material to find it.

Same thing with examples.  Keep them out of the standard so that they can be changed, but close by where people could easily find them.

There is no reference in the RDF Guide to
https://code.google.com/p/tdwg-rdf/wiki/RecommendedIRISources
nor is there one to
https://code.google.com/p/tdwg-rdf/wiki/DwcRdfOccurrences
These can easily go in the Darwin Core wiki if you like. Just make issues
to that effect and assign them to @tucotuco. I am only worried at the
moment about the ones that are referenced in the RDF Guide, namely
http://code.google.com/p/tdwg-rdf/wiki/Beginners
and
https://code.google.com/p/tdwg-rdf/wiki/DwCAncillary
I will publish the RDF Guide with the Google Code references in it for now.
On Tue, Jun 2, 2015 at 12:40 PM, Steve Baskauf<EMAIL_ADDRESS>wrote:

Same thing with examples. Keep them out of the standard so that they can
be changed, but close by where people could easily find them.
—
Reply to this email directly or view it on GitHub
https://github.com/tdwg/dwc/issues/94#issuecomment-107994433.

I think the intention was to a single landing page for ancillary documents with a stable URL so that the RDF Guide document would not have to be edited if the various ancillary documents got moved around.  If you can decide where it makes sense for that landing page to be on the Darwin Core GitHub site, you can put a placeholder there and hyperlink the RDF Guide references to that page.  The various individual reference documents could even be linked back from the landing page to the Google Code pages until someone got around to creating them on GitHub.

Link to Beginner's Guide is
https://github.com/tdwg/rdf/blob/master/Beginners.md
The issue of a landing page for ancillary documents still remains.  Perhaps it could be created in the part of the DwC site residing under www in @peterdesmet's map of the site. The link to that page (outside of the standards documents) could be made in the RDF Guide and then as the ancillary documents are developed, the landing page could be updated appropriately.

@baskaufs can you check if this issue is still relevant?

@peterdesmet the links point to pages on GitHub now, so I think we are fine.


Hide cursor on leave

Hides the hw cursor plane when it leaves the output.
Ref #7412
needs testing

The hiding part works for moving screens, however when the mouse moves over any window decorations it still appears for a split second then disappears again.
I think it maybe the call from CInputManager::processMouseRequest to CPointerManager::setCursorBuffer as that calls updateCursorBackend -> attemptHardwareCursor which sets the buffer bypassing the screen checks.

should be fixed, try now

So close, only one thing happening now. when the cursor transitions from one monitor to another and immediately encounters a window decoration or something like waybar the previous screens cursor remains until the current screens cursor leave the decoration/input area.

@drammelt That's two cursors for the price of one.

I'd say that's minor and overall better than main so I'll merge this


cli: fitconv add CSV to FIT File converter

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

Project coverage is 100.00%. Comparing base (21db25f) to head (e723400).

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #168   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           42        42
  Lines         3471      3471
=========================================
  Hits          3471      3471

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


Move grid layout specific code to its own HorizontalGridDividerItemDe…

…coration
Extend HorizontalDividerItemDecoration as it basically is the same except for calculating how many children to draw and not to redraw for children that already haven been drawn.

Hi @yqritc , took a bit longer to get to it then i thought, seems like some code has changed since my last pull request, somehow can't figure out how to merge this properly right now, feel free to close this request  otherwise I will try it later as I have no time right now.

@tobibo
thanks for PR, and sorry for making you put on effort for this.
I told u separating item decoration for grid may be better, but I found implementing in same item decoration is fine. So, I merged ur last PR and fixed some code.
Current master branch should work well for both linear and grid item decoration.

@yqritc that fine :)


nix-buffer--nix-build: Symbol's function definition is void: make-process

I think this is because I am on emacs 24. I will report back when I have upgraded.

Yes now I am on emacs 25, I no longer get the error.


feat: refactor and test Event types

Note this is currently a WIP
The goal of this PR is the following

Simplify the Event types (currently Event, EventTypeCallback) into a simpler structure. Currently there is a fair amount of abstraction that isn't really necessary
Introduce additional unit tests to validate serialization / deserialization. The goal is ensure that inbound JSON payloads deserialize correctly as compared to the JSON in the Slack API documentation.

For point two above, I've introduced the assert-json-diff crate which allows you to more easily find the differences between serde_json Value objects. The goal is to use this to ensure that we have all the expected keys in the JSON objects.

FYI @Gompei - my motivation currently with this draft PR is to demonstrate some of the suggestions that I made in #23. My secondary goal is to add some more unit tests (as I've described in a few other PRs and issues). If you're supportive of this direction LMK - I'd like to continue through the Events API stuff presented here.


More visualisers

I have wishes for additional visualisers;
C64 font:
The c64 uses fonts - tiles of 8x8 pixels. They can be hires (8x8 pixels) and multicolour (4x8 pixels, where two bits define one out of 4 colours - just like the sprites already in place).
It would be good to be able to select a range. A full set is $0800 bytes, but not all projects use a full set, so a start and end would give the full flexibility to capture all permutations.
More on visualisers:

The sprite visualiser works fine, but also here there is a need for being able to select a range of sprites, rather than selecting them individually. (I guess this would call for a setting that defined the range with or without a padding byte between sprites).
We have had a discussion on structs before and I know you moved it to the TO DO list. In general a soft introduction could be to have a visualiser optionally tied to a struct definition, where data would be formatted as per the struct. Applying a visualiser to a location in memory and defining it as a sprite or a font, I would always have the data structured as per the standard formatting for sprites and fonts. This introduction would not push for a need to introduce a struct editor GUI nor the complexity or variable lengths and optional elements just yet, as we discussed before.
I would also like you to consider an option to hide the data which is already displayed in the form of a graphical component. In practice, showing a font graphically, there is no added value for me to scroll through the data as well. The visualiser already shows it, just not also in the rather redundant data form. (I am aware this makes the struct suggestion above less relevant ;-)

Fonts are pretty straightforward.  The Apple II version takes width, height, and item count:

The glyphs are arranged in a grid, and drawn white-on-black with transparent pixels between items.  For C64 we could drop the item width/height and add a "multicolor" checkbox.  Is color palette selection important, or would black/dk-grey/lt-grey/white work?  (Bearing in mind this is a disassembler, not a graphic converter.)  Is the background color of a font transparent, and if so, should it be for the generated visualization (and how do we draw the borders between glyphs if so?)
Multi-image sprite atlases work similarly.  The Apple II version has a ton of parameters because bitmap rendering is all done in software, and everybody does it a bit differently:

Note this is different from creating multiple individual sprites all at once: they all end up tiled on a single image, and the same color palette and parameters are applied to each (so they're all double-wide, or none of them are).  This also raises the question of how to handle transparency and the borders between elements.  I seem to recall that C64 sprites must be 64-byte aligned, so there should always be a padding byte between sprites?
The most important thing I would need to do the work would be examples of everything, in the form of a project file and binary.  Otherwise I won't know if I'm drawing the font upside-down and backwards, or mapping the colors wrong.  (I used a C64 sprite editor called SpritePad to generate some ugly assets to get the basic sprite visualizer working.)
As far as hiding goes, I have a todo-list item for hiding stuff, with e.g. "have a mode where only the first line of long string and dense-hex items are shown. Can be toggled with a keyboard shortcut".  I hadn't thought of tying it to the presence of a visualizer.  It's a little awkward because visualizers don't have a range... you can put them anywhere, and they can pull data from anywhere.  For example, the classic arcade game Centipede splits the bitmap bit planes across two different ROM chips, so half the data for each tile or sprite is at offset +$800.  You could get most of what you want by having an app setting that allowed you to hide bulk data if a visualization is present at the same start offset, but that doesn't work well if the bulk data is broken into multiple lines for easier viewing.  Hmm.

Invitation sent ...

Sample visualizer sent ...

Humble thanks - it's top notch great! Looking forward to see this available in the release version...

Released in https://github.com/fadden/6502bench/releases/tag/v1.8.3-dev3


How to compile

I have a mac, how to compile ?

I'll update the instructions this weekend and add cross platform Travis and Appveyor CI tests.  For now you can follow the README on this project
https://github.com/forexample/hunter-simple
Once you download and install polly, you can run a build command like this:
polly.py --toolchain libcxx --verbose --install

This should work on Mac OS as is in the current project state.  I will make boost serialization optional to avoid the additional build time (cereal works well and is lightweight).

The main project README has been updated with build instructions.  I've also added cross platform Travis and Appveyor build support which can be used as a reference if needed + a high level regression problem unit test.  Closing...

thanks


chore: bdd-nh-import-1586942826 to 0.0.1

chore: Promote bdd-nh-import-1586942826 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


wire events to operator

Wire events from library-go to operator.
/cc @deads2k
/cc @sttts

/retest

/lgtm


gt hangs when changing the superclass of a class

Instance of ZnRequestBeaconSignal class did not understand #handlesAnnouncement:
ZnRequestBeaconSignal class(Object)>>doesNotUnderstand: #handlesAnnouncement:
AnnouncementSubscription>>handlesAnnouncement:
[ :each |
(each handlesAnnouncement: anAnnouncement)
	ifTrue: [ s nextPut: each ] ] in [ :s |
subscriptions
	do: [ :each |
		(each handlesAnnouncement: anAnnouncement)
			ifTrue: [ s nextPut: each ] ] ] in SubscriptionRegistry>>subscriptionsHandling: in Block: [ :each | ...
IdentitySet(Set)>>do:
[ :s |
subscriptions
	do: [ :each |
		(each handlesAnnouncement: anAnnouncement)
			ifTrue: [ s nextPut: each ] ] ] in SubscriptionRegistry>>subscriptionsHandling: in Block: [ :s | ...
Array class(SequenceableCollection class)>>new:streamContents:
Array class(SequenceableCollection class)>>streamContents:
SubscriptionRegistry>>subscriptionsHandling:
[ interestedSubscriptions := self
	subscriptionsHandling: anAnnouncement ] in SubscriptionRegistry>>deliver: in Block: [ interestedSubscriptions := self...
[ aBlock value ] in SubscriptionRegistry>>protected: in Block: [ aBlock value ]
[ caught := true.
self wait.
blockValue := mutuallyExcludedBlock value ] in Semaphore>>critical: in Block: [ caught := true....
BlockClosure>>ensure:
Semaphore>>critical:
SubscriptionRegistry>>protected:
SubscriptionRegistry>>deliver:
Announcer>>announce:
Beacon>>announce:
Beacon class>>announce:
GlutinEventSignal(Announcement)>>emit
GlutinEventSignal(BeaconSignal)>>emit
GlutinEventsFetcher>>processGlutinEvent:
[ :aGlutinEvent |
| aControlFlow |
GlutinEventClass := aGlutinEvent event_type.
aControlFlow := GlutinControlFlow wait.
self processGlutinEvent: aGlutinEvent.
aGlutinEvent event_type = GlutinEventType RedrawEventsCleared
	ifTrue:
		[ "We should only let other processes run within redraw events cleared"
		Processor activeProcess priority: userInterruptPriority.
		"Force the process to adopt its new lower priority.
					The #yield primitive won't do this for us"
		[ prioritySempahore signal ] fork.
		prioritySempahore wait.
		Processor nextReadyProcess priority > idlePriority
			ifTrue: [ "There's a process with priority between: 12 and: 50 wanting to run"
				idleProcess ifNotNil: #terminate.
				idleSemaphore consumeAllSignals.
				idleProcess := idleProcessBlock forkAt: idlePriority.
				(idleSemaphore waitTimeoutMSecs: 50)
					ifTrue: [ aControlFlow := GlutinControlFlow poll ]
				"deadline expired, let's fetch the next one *now*" ]
			ifFalse:
				[ "Only the idleProcess wants the CPU, wait" aControlFlow := GlutinControlFlow wait ] ].
wantsToShutdown
	ifTrue: [ aControlFlow := GlutinControlFlow exit ].

"This callback is about to terminate. The interrupt handler should ignore
			this process if it is not finished when the interrupt is run."
GlutinIgnoreDuringInterruptStatus value: true.

"Ensure that the current callback process terminates before the next one runs"
Processor activeProcess priority: lowIOPriority.
aControlFlow ] in [ eventsLoop
	run: [ :aGlutinEvent |
		| aControlFlow |
		GlutinEventClass := aGlutinEvent event_type.
		aControlFlow := GlutinControlFlow wait.
		self processGlutinEvent: aGlutinEvent.
		aGlutinEvent event_type = GlutinEventType RedrawEventsCleared
			ifTrue:
				[ "We should only let other processes run within redraw events cleared"
				Processor activeProcess priority: userInterruptPriority.
				"Force the process to adopt its new lower priority.
					The #yield primitive won't do this for us"
				[ prioritySempahore signal ] fork.
				prioritySempahore wait.
				Processor nextReadyProcess priority > idlePriority
					ifTrue: [ "There's a process with priority between: 12 and: 50 wanting to run"
						idleProcess ifNotNil: #terminate.
						idleSemaphore consumeAllSignals.
						idleProcess := idleProcessBlock forkAt: idlePriority.
						(idleSemaphore waitTimeoutMSecs: 50)
							ifTrue: [ aControlFlow := GlutinControlFlow poll ]
						"deadline expired, let's fetch the next one *now*" ]
					ifFalse:
						[ "Only the idleProcess wants the CPU, wait" aControlFlow := GlutinControlFlow wait ] ].
		wantsToShutdown
			ifTrue: [ aControlFlow := GlutinControlFlow exit ].

		"This callback is about to terminate. The interrupt handler should ignore
			this process if it is not finished when the interrupt is run."
		GlutinIgnoreDuringInterruptStatus value: true.

		"Ensure that the current callback process terminates before the next one runs"
		Processor activeProcess priority: lowIOPriority.
		aControlFlow ] ] in GlutinEventsFetcher>>basicPharoEventLoop in Block: [ :aGlutinEvent | ...
BlockClosure>>cull:
BlockClosure>>cull:cull:
[ :anEventAddress |
| aControlFlow |
aControlFlow := aBlock
	cull: (GlutinBaseEvent fromHandle: anEventAddress)
	cull: self.
(aControlFlow isKindOf: GlutinControlFlow)
	ifFalse: [ aControlFlow := GlutinControlFlow poll ].
aControlFlow ] in GlutinEventLoop>>run: in Block: [ :anEventAddress | ...
[ block valueWithArguments: args ] in FFICallback>>valueWithArguments: in Block: [ block valueWithArguments: args ]
BlockClosure>>on:do:
[ Processor terminateActive ] in [ :ex |
| onDoCtx handler bottom thisCtx |
onDoCtx := thisContext.
thisCtx := onDoCtx home.

"find the context on stack for which this method's is sender"
[ onDoCtx sender == thisCtx ]
	whileFalse: [ onDoCtx := onDoCtx sender.
		onDoCtx
			ifNil: [ "Can't find our home context. seems like we're already forked
				and handling another exception in new thread. In this case, just pass it through handler." ^ handlerAction cull: ex ] ].
bottom := [ Processor terminateActive ] asContext.
onDoCtx privSender: bottom.
handler := [ handlerAction cull: ex ] asContext.
handler privSender: thisContext sender.
(Process forContext: handler priority: Processor activePriority)
	resume.

"cut the stack of current process"
thisContext privSender: thisCtx.
answerAction cull: exception ] in BlockClosure>>on:fork:return: in Block: [ Processor terminateActive ]

Oops, my bad. The issue was that the new superclass was not a Signal class and the already started SignalLogger would freeze the entire UI.


Fix #260 Enable to use respond utility in app.view listeners (only when response_urls exists)

This pull request resolves #260 by enabling respond utility in view_submission listeners. Check the issue for learning the goal.
Category (place an x in each of the [ ])

[x] slack_bolt.App and/or its core components
[x] slack_bolt.async_app.AsyncApp and/or its core components
[ ] Adapters in slack_bolt.adapter
[ ] Document pages under /docs
[ ] Others

Requirements (place an x in each [ ])
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run ./scripts/install_all_and_run_tests.sh after making the changes.

Thanks for the review!

Can you share any example on how to implement this in python, even one response url will work


No linebreaks added on crlf

Hello,
I'm experimenting with the WikiPdfExport and I'm running into an issue.
I generate a releasenote using Richard Fennells 'Generate Release Notes'.
This generates an .md with at the beginning the following:

When I export this to a Pdf using the WikiPDFExport tool (on a windows machine), the headers are not on new lines.
When I run the tool in debug, I see that the headers are wrapped in a
 without
.

Only when I add 2 spaces in front of the crlf, a
 is added.
So after Release-40 there is no break, after  (GMT+02:00) there is one.
This results in this pdf:

Is this by design or is this a bug in the html engine?
How (except for adding 2 spaces) can I fix this or workaround this?

Okay, so it is by design. Thanks for the link...


refactor(fiatconnect): separate out fetch providers

Description
Refactors the way that providers for fiatconnect are fetched so that they are fetched a single time when a user enters the CICO flow. This makes fetching quotes cleaner and later will make fetching fiatAccounts cleaner.
Tested
unit tests
How others should test
Does not need to be tested by QA

hey @MuckT Since NA is mentioned in how other should test section we are skipping this PR
Cc: @jh2oman
thanks.!


Made some good changes in file"README.md"

Hi
I have made some changes in file

Please mark as invalid and spam, this is an obvious attempt to get a free t-shirt by 'contributing'(hacktoberfest)
This will also ban the user for trying to abuse the system by doing marginal effort


Map rerender

We have state in Home component so whenever a value changed, it re-renders. For example on toggleDrawer, it close opened cluster, it's because we re-render and loses its state.

@hamitaksln can you resolve conflicts?

@eraygundogmus Sure

@hamitaksln can you open a pr for readme.md? we need turkish docs to at least refer to state management repository

@eraygundogmus Sure I will do that.


e2e: add client tests

This PR adds basic e2e testing for the api client to ensure no regressions in logic. More robust tests will be added once #85 merges.
Signed-off-by: Sam Batschelet<EMAIL_ADDRESS>

tungstenite failure now resolved https://github.com/snapview/tungstenite-rs/issues/321


Pull data from www.borischen.co

EDIT: found that borischen hosts all the data on txt files available via aws
https://s3-us-west-1.amazonaws.com/fftiers/out/text_WR-PPR.txt
Each one has a scoring suffix, so will need to add a scoring option to the UI. Also, add logic to avoid spamming calls too often.
Scope

Add scoring option dropdown/radio
Fetch data when scoring option changed
store last time data was fetched
when FUSE initializes, check if it's been more than a day since last fetch, if yes, then auto fetch new data
add button to allow user to force fetch
show loading indicators when fetching

Old
Accessing the Google sheet data or scraping the home page are potential methods.
There's a page intended for drafting, but might be kept updated each week
http://www.borischen.co/p/draft-sheets.html
It links out to a google doc and a CSV download (though standard scoring only).
Alternative approach is to just scrape the html of the site, though not sure if it could be done on just one page or if we'd need to make multiple requests for each position.
Would also need to add options to the Borischen tab asking what scoring the league uses (standard, .5 PPR, PPR)

Looks like this will be blocked due to CORS after all
Here's the initial implementation I put together
const fetchDataFromBorisChenWebsite = async (scoring) => {
    const scoreSuffix = {
        "PPR": "-PPR",
        "HALF": "-HALF",
        "STANDARD": ""
    }

    const positionsToGet = [
        { key: 'QB', val: `QB` },
        { key: 'RB', val: `RB${scoreSuffix[scoring]}` },
        { key: 'WR', val: `WR${scoreSuffix[scoring]}` },
        { key: 'TE', val: `TE${scoreSuffix[scoring]}` },
        { key: 'FLEX', val: `FLX${scoreSuffix[scoring]}` },
        { key: 'K', val: `K` },
        { key: 'DST', val: `DST` },
    ]

    let rawTierData = {};

    const promises = positionsToGet.map(
        position => fetch(makeUrlString(position.val)).then(p => rawTierData[position.key] = p.text())
    );

    await Promise.all(promises);

    return rawTierData;

    function makeUrlString(position) {
        return `https://s3-us-west-1.amazonaws.com/fftiers/out/text_${position}.txt`;
    }
}

And just added a button to trigger that fetch and update the position specific inputs
const positions = ['QB', 'RB', 'WR', 'TE', 'FLEX', 'DST', 'K'];

const fetchDataBtn = makeButton('Fetch', async () => {
    const rawData = await fetchDataFromBorisChenWebsite('PPR')

    for (const position of positions) {
        let positionInput = document.getElementById(`${selectors.settingPanel.borisChen}_${position}`);
        positionInput.value = rawData[position];
    }
});
tab.appendChild(fetchDataBtn);

The cors headers get automatically applied by the browser to network request

The response headers don't include anything stating cross site requests are ok

And the result is CORS errors

I think the only options that remain (outside of hosting a reverse proxy) is to convert FUSE to

a greasemonkey/tampermonkey/Violentmonkey userscript
chrome extension

I think both approaches have a way to bypass CORS on requests. Violentmonkey type of thing might be the best option since it's cross browser. A chrome extension will only work for one browser. Either approach will require a bigger refactor and add some overhead friction to installation.


Improve development install with symlinks

This should remove the requirement to reinstall after every change.  It installs the geonotebook extension as a symlink with development installs.  The javascript can be automatically rebuilt by running npm run watch in the js subdirectory.

Current coverage is 38.35% (diff: 100%)

Merging #58 into master will not change coverage

@@             master        #58   diff @@
==========================================
  Files             8          8
  Lines            73         73
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits             28         28
  Misses           45         45
  Partials          0          0

Powered by Codecov. Last update 140dbd8...f5a7450

This works perfectly. It also allows for changing javascript without restarting the notebook server.
Thanks!


Allow entering an email address even when the mention character is @

When the mention character is an @ I can not enter an email address such as<EMAIL_ADDRESS>because the @ in the email is treated like the start of another mention.
With the isolateCharacter option enabled, it is possible to ignore the @ in the email, but still not possible to treat<EMAIL_ADDRESS>as a single mention. Only everything preceding the ignored @ is considered a mention, so only @foo is part of the mention.
The reason behind this behavior is because the mention char index is always the last @ found.
I suggest adding a new option to customize that parsing behavior. The option's value would be a function that returns the index of the mention character and its default value would be the current behavior of returning the last index found. This is the approach I took in a fork to work around this problem.
Would love to merge the changes upstream. If this sounds like a good idea then I can open a PR. Or should there be a different way to support this scenario?

did you ever find a solution to this?

Sorry for late response @son-le but if you're interested in fixing this, I can give you commit access to this repository.

@MadSpindel Sure, I can add a fix for this.

Created PR #331 for this, but I don't have a codebase handy anymore that can verify this works as expected in a real project. Anybody here want to try this out and see if works for them?


compiler bug: visitor didn't yield any result

The compiler reported a bug when trying to compile the attached project.
wicket-test.zip
compiler.txt
Reported by @ageery at https://github.com/ceylon/ceylon-compiler/issues/2434

The problem is the call to add() in this code:
class MyPanel(String id) extends Panel(id) {
	add(Label("myLabel"));
}

which causes an exception in the code generator. I don't understand why that code is allowed by the typchecker though: We using an inherited declaration in an initializer, which is normally prohibited. @gavinking can you explain?

Doesn't seem right to me.
Sent from my iPhone

On 03 May 2016, at 12:47, Tom Bentley<EMAIL_ADDRESS>wrote:
The problem is the call to add() in this code:
class MyPanel(String id) extends Panel(id) {
add(Label("myLabel"));
}
which causes an exception in the code generator. I don't understand why that code is allowed by the typchecker though: We using an inherited declaration in an initializer, which is normally prohibited. @gavinking can you explain?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

I tried the following two variations and both produced the same error:

Putting the add call in an explicit constructor; and
Putting the add call in the onInitialize method (which is another way of adding components to a panel)

class MyPanel extends Panel {
shared new(String id) extends Panel(id) {
	add(Label("myLabel"));
}

}
class MyPanel(String id) extends Panel(id) {
shared actual void onInitialize() {
	add(Label("myLabel"));
}

}

Some small remarks:
class Panel extends MarkupContainer
class MarkupContainer extends Component
MarkupContainer declares a method: add(Component... childs)
Component declares a method: add(Behavior... behaviors)
So we have two overloaded variable arity methods and unqualified call from Panel
to MarkupContainer.add.
It seems it is the root of the problem, and all three ingredients are important.
I have created the following test in order to reproduce this error: https://github.com/MichaelNedzelsky/ceylon/commit/52b2b9ce9b510762a2e7cd0a9bb801896963c181
What is going on:

Unqualified call leads to invocation of ExpressionTransformer.addThisOrObjectQualifierIfRequired;

inside addThisOrObjectQualifierIfRequired (at line 4983) we try to determine, is given declaration is inherited, and we fail,
because

instance of TypeDeclaration for MarkupContainer has two members with name "add" and signature "Component?[]" ---
real one and additional abstract member, which has been added in AbstractModelLoader (at line 2774)
as extra "abstraction" method for overloaded methods and

TypeDeclaration.isInheritedFromSupertype has the following code in the class Criteria:
type.getDirectMember(member.getName(), signature, false /* for variadic */);
and it selects wrong ("abstraction") method.
I think the following commit will fix this error: https://github.com/MichaelNedzelsky/ceylon/commit/a5cabb8c347a89ef852c58da5e0b6339f6d9c59d
(I have checked for original problem (wicket-test) and run all ConcurrentTests).

@MichaelNedzelsky that's all just perfect. I missed the add(Behavior... behaviors), shame on me.
Anyway, thank you! Could you send a PR (or if it's easier, just give your permission to merge those patches, per https://github.com/ceylon/ceylon/blob/master/README.md).

@tombentley
I have created PR: https://github.com/ceylon/ceylon/pull/6254

@ageery
BTW, it is possible to work around this problem using qualified call:
   this.add(Label("myLabel"));

Thank you!  That does work, but why does it work?  Is there an add function
in scope that was confusing the compiler?
Andrew
On Sat, May 7, 2016 at 3:13 AM, Michael Nedzelsky<EMAIL_ADDRESS>wrote:

@ageery https://github.com/ageery
BTW, it is possible to work around this problem using qualified call:
this.add(Label("myLabel"));
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/ceylon/ceylon/issues/6244#issuecomment-217612752

@ageery
There are two 'add' functions in a scope: MarkupContainer.add(Component... childs) and
Component.add(Behavior... behaviors).
Qualified call does work, because the corresponding implementation is correct --- there was an error for unqualified call, more exact, in case for combination of unqualified call and variable arity overloaded methods.

Got it -- thanks for explaining the bug.
On Wed, May 11, 2016 at 6:17 AM, Michael Nedzelsky<EMAIL_ADDRESS>
wrote:

@ageery https://github.com/ageery
There are two 'add' functions in a scope: MarkupContainer.add(Component...
childs) and
Component.add(Behavior... behaviors).
Qualified call does work, because the corresponding implementation is
correct --- there was an error for unqualified call, more exact, in case
for combination of unqualified call and variable arity overloaded methods.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/ceylon/ceylon/issues/6244#issuecomment-218419045


ensure mutated strings are mutable

Hello, thank you for the incredibly useful library!
I was experimenting with freezing strings in my application by default with RUBYOPT=--enable=frozen-string-literal, and rack-timeout has a couple places I ran into that are preventing it.
I opted for a minimal changeset as I wasn't sure if there would be openness to freezing strings within the library itself with # frozen_string_literal: true (but I'd be happy to include that here too if you'd like).
Error messages I received in testing:
can't modify frozen String: "source=rack-timeout" (FrozenError)
can't modify frozen String: "Request " (FrozenError)

Hi, wanted to check in to see if there was something I could do to move this forward 🙂

Following up to check if I can help with getting this merged or reviewed. Thanks again for rack-timeout!

Appreciating that this PR exists :) Especially given MRI 3.4 will output deprecation warnings about changes to frozen strings!

I’m happy to do what’s needed to get this reviewed/merged 🙂

is there a syntax that can be used that is backwards-compatible with older versions of Ruby? I'm not sure when the +"mutable string" syntax was added, but I assume it's relatively recently (this is the first time I've seen it).

@wuputah String#+@ was added in Ruby 2.3 though I think String.new works as well.

@mitchellhenke beat me to it - I was just hunting through Ruby docs to confirm when it was introduced (and yes, it's in 2.3 but not 2.2.

Ah, ok, quite some time ago. So happens I just dropped support for 2.2 in CI because it's so old Github deleted it.

Of course, and I am happy to add frozen_string_literal in a separate pull request as well if that's something you'd be interested in!


[ISSUE #3475] Fix sofa, motan and dubbo don't work issue caused by Curator version incompatible in bootstrap

// Describe your PR here; eg. Fixes #issueNo
For #3475 , related issue and PR links: #3461  #3470.
This PR has done following things,

replace CuratorCache with TreeCache, which class is both exists in ver 4.0.1 and 5.2.1.
test and make sure all works well for bootstrap zk data sync in my local machine.

Welcome to test and let me know if any comment. Thank you!
Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3476 (077146d) into master (246f25c) will decrease coverage by 0.44%.
The diff coverage is 70.58%.

@@             Coverage Diff              @@
##             master    #3476      +/-   ##
============================================
- Coverage     62.95%   62.51%   -0.45%
+ Complexity     5529     5491      -38
============================================
  Files           834      834
  Lines         23092    23101       +9
  Branches       2091     2089       -2
============================================
- Hits          14538    14441      -97
- Misses         7220     7318      +98
- Partials       1334     1342       +8

Impacted Files
Coverage Δ

...he/shenyu/sync/data/zookeeper/ZookeeperClient.java
50.00% <46.66%> (-2.06%)
:arrow_down:

.../sync/data/zookeeper/ZookeeperSyncDataService.java
96.00% <89.47%> (-1.53%)
:arrow_down:

...plugin/logging/rocketmq/LoggingRocketMQPlugin.java
22.72% <0.00%> (-72.73%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
75.00% <0.00%> (-25.00%)
:arrow_down:

...ogging/rocketmq/body/LoggingServerHttpRequest.java
0.00% <0.00%> (-20.00%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
47.91% <0.00%> (-18.75%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...ng/rocketmq/rocketmq/RocketMQLogCollectClient.java
68.00% <0.00%> (-8.00%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

... and 30 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 246f25c...077146d. Read the comment docs.


Add a very useful picture how to use the data provided

I think people could benefit from seeing this picture:

From here: https://spotthestation.nasa.gov/images/astro_horizon.png

Yes, I've seen this picture and it is very informative. Where do you suggest I use it? In the readme?

README sound good.

added


Implement directory content equality.

Coverage decreased (-0.2%) to 89.325% when pulling a1ee2d2ab8b6e50dfe6408a69f39924a2355d840 on garydgregory:PathUtils_directory_equals into e46146c67026956d3f48d44be5c4df0fb78d5a42 on apache:master.


[STYLE] Scrape button it not fully responsive.

Button goes outside the parent div border on some screen size.
##Visuals ->
1-

2-

##Fix -
Need to modify some CSS.

Assign it to me!
@singodiyashubham87

@singodiyashubham87 assign this to me now!

@professorabhay Want to work on this issue?

Sure!

@professorabhay Go Ahead!

@professorabhay Are you still working on the issue?


register with Julia registration bot

Hi @Datseris, I forked your spike synchrony package and then registered it as a Julia package. I hope you don't mind. The problem is I have another package that uses it as a dependency and its hard to get a julia build to work universally if it contains multiple installs of the Pkg.add(url="...") kind.
If you want to register it, I could de-register it.

Hi,
Have you actually worked on the package? I.e., added more features to your fork?
For me the best case scenario would be to actually move the package to some kind of org that cares about these topics.

Yes it would be awesome if there was a Neuroscience Julia org.
No, I haven't worked on the module, I did write interfacing code (so it can interface with Julia spiking neural network simulator spike train formats (as in I wrangled different spike train formats into formats that can work with this spike synchrony.
It's okay I can un-register it, I just wanted to see if it would make my package build better (I think it did).
If there was a neuroscience Julia org I would definitely join it. Also in an earlier time, I would have started one myself, but I am looking for work atm the moment now though.

No, I haven't worked on the module

But then it is confusing to me that you'd register a fork instead of asking the original to be registered... I mean, it does not matter that much for me, as I'm happy to transfer the package anyways, but this move doesn't look like good practice me, but looks rather weird.
There is clearly a Julia Neuroscience community:
https://juliapackages.com/c/neuroscience
https://julianeuro.github.io/
the community seems to be coordinated by Alexander Morley: https://github.com/alexmorley
@alexmorley , are you okay with transfering this package to the Neuro org, and registering it from there?

https://github.com/JuliaNeuroscience

I'm confused now... why is there both https://github.com/JuliaNeuroscience and https://github.com/JuliaNeuro  ...? Maybe these two should be merged @Tokazama and @alexmorley ? JuliaNeuroscience has more packages so I'm inclined to move the package there. @Tokazama can you please invite me to the organization and make a team called "SpikeSynchrony" which has access-level rights to this repo, and put me there (and also feel free to put there @russelljjarvis or anyone else that wants to use this package in any way). As soon as you invite me I'll transfer the package.

Hi @Datseris, wow thanks for looking into all of this. I thought you meant a Github organization, not a Julia organization, yes I had noticed those Julia Neuroscience organizations. Perhaps they are all the same on some level, but I can't find GitHub Julia organization web pages.
I think some proportion of the packages on both neuro-organizations, will not build properly with the latest Julia LTS release 1.6.2, that is kind of why I think a Github Julia neuroscience organization is more useful, because when you are reading over github packages you can see build status badges, and the date of the most recent commit etc.
It seems like you can't easily un-register a package from the general registry, but you can probably update it with a newer release https://discourse.julialang.org/t/remove-package-from-general-registry/71668/2. Let me know if I am wrong.
"@alexmorley , are you okay with transferring this package to the Neuro org, and registering it from there?" -- Would this overwrite my registration (if so good!). If not let me know if there are additional steps I can do to facilitate correcting my spurious package registration.

No, what you said is correct: you cannot unregister a package, now what is done is done. What you can do is do a manual Pull Request to the general registry that changes the github url, git SHA (should be the same actually), and other metadata to the GitHub instance you'd like. This is e.g., something typically done once packages are transferrred to orgs. I've done it many times and so far noone raised any red flags. Just wait until after I move this to JuliaNeuroscience and then you can do the general registry PR.

"But then it is confusing to me that you'd register a fork instead of asking the original to be registered... I mean, it does not matter that much for me, as I'm happy to transfer the package anyways, but this move doesn't look like good practice, but looks rather weird."  -- Yes, I know, I agree I can see that now. The way I was thinking about  it was, A. I incorrectly assumed the process would be reversible. B: I was struggling with management. C: I was not sure how responsive you would be.
Anyway. I agree its bad etiquette and it looks bad, but given the situation, I would like to make errands.

No, what you said is correct: you cannot unregister a package, now what is done is done. What you can do is do a manual Pull Request to the general registry that changes the github url, git SHA (should be the same actually), and other metadata to the GitHub instance you'd like. This is e.g., something typically done once packages are transferrred to orgs. I've done it many times and so far noone raised any red flags. Just wait until after I move this to JuliaNeuroscience and then you can do the general registry PR.

AOK this is what I will do then! Thanks and also sorry.

https://github.com/JuliaNeuroscience. I would love to have someone else participate in basic PR reviews over there. Kind of hard to know if I'm doing anything helpful for others while talking to the void

I think what you are doing is useful! I wonder if this organization is also a pure Github Organization too?

I'm confused now... why is there both https://github.com/JuliaNeuroscience and https://github.com/JuliaNeuro ...? Maybe these two should be merged @Tokazama and @alexmorley ? JuliaNeuroscience has more packages so I'm inclined to move the package there. @Tokazama can you please invite me to the organization and make a team called "SpikeSynchrony" which has access-level rights to this repo, and put me there (and also feel free to put there @russelljjarvis or anyone else that wants to use this package in any way). As soon as you invite me I'll transfer the package.

Is one more neuroimaging scale related, and the other a mixture of mesoscale modelling and neuroimaging? I agree a merge would probably be good. Or a merge with a shedding of broken and currently un buildable/un usable packages.

I wonder if this organization is also a pure Github Organization too?

Both orgs we've linked are GitHub organizations. The JuliaNeuro one has a website as one of the repositories of the org. Maybe you want to clarify what you mean with "organization" in the above question? It feels like there is some confusion on the matter.

Or a merge with a shedding of broken and currently un buildable/un usable packages.

Personally I see no reason to remove anything that hasn't been updated. Maybe at some point someone wants to update them. It has happened in the past with many orgs.

I wonder if this organization is also a pure Github Organization too?

Both orgs we've linked are GitHub organizations. The JuliaNeuro one has a website as one of the repositories of the org. Maybe you want to clarify what you mean with "organization" in the above question? It feels like there is some confusion on the matter.

Or a merge with a shedding of broken and currently un buildable/un usable packages.

Personally I see no reason to remove anything that hasn't been updated. Maybe at some point someone wants to update them. It has happened in the past with many orgs.

You are probably right and there are build status badges there too.
I see now there is a gitlevel org that is also a julia org https://github.com/JuliaNeuroscience
Actually, I am discovering a lot of new stuff. The situation is evolving fast, or I have not been searching effectively, or both.

Still sounds like there is some confusion, from the statement "A GitHub level org that is also a julia-org". So, to clarify:

Every organization we've linked so far is a GitHub organization. It is nothing else than that. A collection of packages under the same banner, that is accessible by a specified group of developers.
None of these organizations are "officially affiliated with the Julia language".
The reason each of these organizations has the name "Julia" in their name is because they are composed almost entirely out of Julia packages

Furthermore:

is the organization directly hosted and reflected inside GitHub or not?

Everything in the Julia General Registry is on GitHub. Doesn't matter its context.

AOK Thanks for clarifying!

BTW, can you please link the original PR of the package registration here?

Yes. Just a second...

I've already made the transfer.

This is great, sorry for consuming your time on this issue. I am glad there is a good resolution!

Just link here the PR that corrects the links in the General Registry when you have it open and we can leave this behind us without much worry :)

AOK, just seeing this now. I will look for the PR that corrects the links in the general Registry. Bare with me...

I will look for the PR

This doesn't happen automatically, you have to do it...

Oh, I see. I will read about how to trigger it.

Found it.

What you can do is do a manual Pull Request to the general registry that changes the github url, git SHA (should be the same actually), and other metadata to the GitHub instance you'd like. This is e.g., something typically done once packages are transferrred to orgs. I've done it many times and so far noone raised any red flags. Just wait until after I move this to JuliaNeuroscience and then you can do the general registry PR.

Oh so, just unpacking this. I think I should fork the general registry:
https://github.com/JuliaRegistries/General
and do a PR, which is similar to the one the registration bot made on my behalf, I just replace the link to my fork with https://github.com/JuliaNeuroscience/SpikeSynchrony.jl

yes, you just replace the repo here: https://github.com/JuliaRegistries/General/blob/master/S/SpikeSynchrony/Package.toml#L3

Thanks for that!
I hope this will fix it:
https://github.com/JuliaRegistries/General/pull/51935
Also, I noticed that I did do a minor change to the code that I forgot about. I just changed a single for loop to have the @inbounds macro, to see if it was faster. If its a bad change it should be easy to fix.

as long as you run the tests and it didn't break anything then its fine.

https://github.com/JuliaRegistries/General/pull/51935/files#diff-9f173466d557da5313913c5e18c976c96bfa000a5c73a13b0d06fb692ff4c2cd

Now forked from says this:

Perhaps this means efforts have succeeded?

@russelljjarvis no, it's not fixed. Please try to read carefully and precisely what I'm saying.
I've transfered my repo to the Neuroscience organization. This changes nothing regarding commits.
Your original fork, from which you tagged a release from, has had additional commits that are different from the original repo that you forked from (mine). Even though you said you didn't do any changes, any kind of commit matters. Look here:

You see that it says that your master branch and our master branch are completely out of sync? That's because you were doing different kind of commits in the different repos, instead of properly doing a pull request as this the way things should be done.
Now, if you read carefully what user giordano said at the general registry PR, you can't change the link of the registered package to the link of the Neuroscience org package, because their git histories are out of sync.
There are two ways to fix this:

Do whatever is necessary to make the tagged commit of your fork be the current last commit of Neuroscience/SpikeSynchrony.jl
Ask the people that run the General Registry to delete your registration, so we can tag a new release from the correct repo.

To be safe I might try 2 at this point. I will ask giordano, in the second conversation to delete it.

 I don't know if issuing will have the intended effect.
git push origin v0.1.0```

2. Ask the people that run the General Registry to delete your registration, so we can tag a new release from the correct repo.

The git hash is already registered in the general registry. It is useless to attempt any change in your fork, because its history is saved in the Registry already. The only way forwards is to make the necessary change in this original repo. I guess only I can do this as I need to force push current master branch.

Ok I will wait for you to do that.

okay this is resolved now, i've also tagged version 0.1.1 which is current master branch to ensure things work.

Phew! Thanks and also sorry. Rest assured I won't intervene in anything SpikeSynchrony.jl unless you ask me to henceforth.

just do pull requests for any feature you want, it's almost certain they will be merged instantly.

If you want a set of guidelines to follow for collaborating you might be interested in https://github.com/SciML/ColPracf

Reading it now. I think the 'f' on the end of ColPracf is a typo but I believe this is the correct document:
https://github.com/SciML/ColPrac#readme

😆 sorry, I'm clearly doing too many things at once.


[GUI] Groups: cards are hidden

When the window is resized, the cards are hidden.

The same behavior applies to all pages.


All plugins: Username+Password via Environment Variables

Describe the solution you'd like
Extend any plugins that ask for username and password by trying to read them from environment variables first.
Additional context
No response

Let's have a look at how mydumper/myloader handles this.

Have a look at https://stackoverflow.com/a/10551190.


add -ca-bundle flag and support for custom CAs to vSphere, Openstack

What this PR does / why we need it:
This makes it possible to completely override the set of CA certificates used to communicate with the vSphere/Openstack APIs. If -ca-bundle is set, the certificates from the machine-controller(-webhook) pods are not used anymore.
The first commit shows the naive approach, where the CA bundle file is given through the stack until it arrives at the cloud provider instances. This was mainly because at the time, I thought the SetRootCAs() function was the only way to inject the CAs in the vSphere provider.
However the second commit retracts that approach and favors setting the CA as a global variable. I hate global state, but

this approach uses much less ugly code
this approach loads the CA bundle once and then re-uses the x509.CertPool
the CA bundle will not change during the runtime anyway

A new e2e test has been added that is doing a "negative test": We override the CA bundle with a useless set of random certificates and then expect Machine validation from the webhook to fail. This demonstrates that the CA bundle is indeed loaded, parsed and used by the cloud provider (openstack in this case).
Optional Release Note:
Add support for custom CAs for Openstack, vSphere

/retest

/hold
still want to find a way to add some rudimentary tests

/hold cancel
e2e test has been added and seems to work pretty well :)

/hold

/cc @kron4eg if you are ok with the changes remove the /hold ;)

/hold cancel

Thanks guys, we got this :-) :tada:


Some problems about wrap a sliver widget

When I wrap a sliver widget in the ZoomableWidget,I found that I can't scroll it.So I go to the source code, and add a Listener() widget contains the GestureDetector.But it's too hard to handle the clash between sliver widget and onScaleUpdate .

Oh, if you are using some kind of scrollable widgets in ZoomableWidget, the events of these widgets will be ignored.

Oh, if you are using some kind of scrollable widgets in ZoomableWidget, the events of these widgets will be ignored.

Yes.Is there any way to handle this?

No, i have no ideas with this. I built the ZoomableList with GestureDetector and Column, you will need to implement your own one for your needs.

Ref: https://github.com/flutter/flutter/issues/15011


Preview dbt Cloud feature: Slim CI

Description & motivation

Document job deferral in dbt Cloud as a preview feature
Update best practice on slim CI

@drewbanin Could you take a look at the language here? I don't think we have any other "preview" features in dbt Cloud that are available on all plans.
Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[x] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

Thanks for the review @clrcrl! I added one more tiny note about both jobs needing to run dbt >= v0.18.0, given feedback from beta tester extraordinaire @aescay.
I'm going to merge this in so that I can announce it in dbt Slack. We can always adjust the language later.

Meeep - apologies for the delay. Glad you merged this one without me!


'sbt package'  fails due to schema proto files

sbt package
Compiling schema /databricks/learning-spark-master/src/main/protobuf/places.proto
java.lang.RuntimeException: error occured while compiling protobuf files: Cannot run program "protoc": error=2, No such file or directory
Caused by: java.io.IOException: Cannot run program "protoc": error=2, No such file or directory
at java.lang.ProcessBuilder.start(ProcessBuilder.java:1048)
Caused by: java.io.IOException: error=2, No such file or directory
at java.lang.UNIXProcess.forkAndExec(Native Method)
at java.lang.UNIXProcess.(UNIXProcess.java:248)
[error] (protobuf:protobufGenerate) error occured while compiling protobuf files: Cannot run program "protoc": error=2, No such file or directory

after some googling: Found that I needed to install 'protobuf', before sbt package
On mac I just did 'brew install protobuf'
and then sbt package worked.


Multiple glTF/glb model components are not imported as a whole entity

Describe the bug
Actions:

Decompile an entire folder with multiple models (E.g.: Dota 2 pak01_dir.vpk -> models -> heroes -> drow folder )
In Blender import all glTF/glb files (see picture1)

Observed behaviour:

Base model is not aligned with the rest of components
Animations act only on base model and not over all components

Expected behaviour:

Base model should be aligned with the rest of components
Animations should act on all components (see picture2_original_model_from_valve)
Child components(armatures) should have a relation with base_armature (i guess)

What Valve game does this happen in
Dota 2
Reference file in a Source 2 game
pak01_dir.vpk -> models -> heroes -> drow folder
Expected behavior

Base model should be aligned with the rest of components
Animations should act on all components (see picture2_original_model_from_valve)
Child components(armatures) should have a relation with base_armature (i guess)

What version of VRF are you using? On what platform?
<IP_ADDRESS>2

Duplicate of #251
They're not supposed to be automatically combined, as otherwise you could accidentally combine parts that were meant for other models, especially when exporting large folders such as models/ or models/heroes/.
There should probably be a separate dialog for that, as you might easily want separate parts from different folders, example being econ wearables - e.g. something from models/items/ and the rest from models/heroes/.

Then can be changed to feature task. When you are exporting 2 or more models to have the option to merge it in a single one. Just an idea.

I'm assuming they are bonemerged in engine. If so then just look up how to bonemerge in blender?

I found how to merge all components by changing Objecct -> Modifier -> Armature -> Relations -> Parent to base object(E.g.: see attached image). The easiest way is to create a python script for multiple models to do this.