"gradle test"  in the parent module of a multi-module project does not run tests, while running its "test" task in the IDE does

While my IDE runs failed tests consistently, in the CLI nothing happens, it thinks all the tasks are up-to-date.
Which they cannot be, because some tests are still failing. Running the test task in the IDE for the top-level module verifies this.
Any help in troubleshooting this is welcome, I'm getting crazy over here.
Expected Behavior

Current Behavior
 $ gradle test
...
BUILD SUCCESSFUL in 1s
24 actionable tasks: 24 up-to-date

Context

I'm currently migrating a project to Gradle.
Some of the tests in two sub-modules of a multimodule project fail with

kotlin.jvm.KotlinReflectionNotSupportedError: Kotlin reflection implementation is not found at runtime.
Make sure you have kotlin-reflect.jar in the classpath

In the CLI, I try again ./gradlew test and nothing gets executed because it thinks everything is up to date.
In the CLI, I try again ./gradlew clean test -> ditto
In the CLI, I try again ./gradlew test --no-build-cache -> ditto
In the CLI, I try again rm -r .gradle; ./gradlew test -> ditto
In IDEA, running the test task always re-runs those tests and fails as expected
So running the test task in the IDE is working as I expect (some tests in the test suite fail), the CLI does not.

I thought I was doing something wrong. Now I think something is wrong with the assumptions Gradle makes in the CLI vs IDE.
It's like I'm dealing with two different programs.
Steps to Reproduce
So far only reproducible on a private project that I cannot share here.
I tried to create a minimal test project for reproduction, to no avail:
$ gradle init
Starting a Gradle Daemon, 1 incompatible Daemon could not be reused, use --status for details

Select type of project to generate:
  1: basic
  2: application
  3: library
  4: Gradle plugin
Enter selection (default: basic) [1..4] 2

Select implementation language:
  1: C++
  2: Groovy
  3: Java
  4: Kotlin
  5: Scala
  6: Swift
Enter selection (default: Java) [1..6] 4

Split functionality across multiple subprojects?:
  1: no - only one application project
  2: yes - application and library projects
Enter selection (default: no - only one application project) [1..2] 2

Select build script DSL:
  1: Groovy
  2: Kotlin
Enter selection (default: Kotlin) [1..2] 2

Project name (default: project): repro
Source package (default: repro):

> Task :init
Get more help with your project: https://docs.gradle.org/6.8.3/samples/sample_building_kotlin_applications_multi_project.html

I had to fix the generated project for it to build first:
$ vim buildSrc/src/main/kotlin/repro.kotlin-common-conventions.gradle.kts

     // Use JUnit Jupiter Engine for testing.
    testRuntimeOnly("org.junit.jupiter:junit-jupiter-engine:5.6.2") <-- added version, none was present, creating errors

Then add a test that fails with the same error.
$ vim list/src/test/kotlin/repro/list/LinkedListTest.kt

    @Test fun testThatFails() {
        String.javaClass.kotlin.isSealed
    }

With this generated test project the CLI  gradle(w) test command consistently fails as expected.
So there's something going on, I just don't know what, and I don't know how to figure out what's so special about my project, which I gradle init-ed from an existing maven project, and minimally modified to get things to compile, test, and publish. Not much magic.
Your Environment
CLI
 $ ./gradlew --version

------------------------------------------------------------
Gradle 6.8.3
------------------------------------------------------------

Build time:   2021-02-22 16:13:28 UTC
Revision:     9e26b4a9ebb910eaa1b8da8ff8575e514bc61c78

Kotlin:       1.4.20
Groovy:       2.5.12
Ant:          Apache Ant(TM) version 1.10.9 compiled on September 27 2020
JVM:          11.0.7 (Amazon.com Inc. 11.0.7+10-LTS)
OS:           Linux 4.15.0-140-generic amd64

IDEA
IntelliJ IDEA 2020.3.2 (Ultimate Edition)
Build #IU-203.7148.57, built on January 26, 2021
Licensed to C.O.L. CYPRIOPS LTD / Stephane Mikaty
Subscription is active until August 7, 2021.
Runtime version: <IP_ADDRESS>+11-b1145.77 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Linux 4.15.0-140-generic
GC: ParNew, ConcurrentMarkSweep
Memory: 1904M
Cores: 8
Non-Bundled Plugins: detekt, io.binx.cfnlint.plugin, AWSCloudFormation, aws.toolkit, org.jetbrains.kotlin, PlantUML integration, org.jetbrains.plugins.go
Current Desktop: XFCE

Build scan URL: on request

Are the projects properly referenced in the settings.gradle(.kts)?
If they are, please provide a reproducer as otherwise this will be hard to investigate.

Glad to hear I'm not dreaming, that this problem has been experienced
before, and still exists. So this issue will track it, and hopefully
someone from the Gradle team can weigh in.
I need advice on what to look for in the build scan, or contact details of
someone from the Gradle team who could look at the build scans (or know
what to do with them).
On Wed, Mar 31, 2021 at 11:20 PM Björn Kautler @.***>
wrote:

No, I'm just a Gradle Fellow, not an employee.
But what you describe sounds exactly like what I see from time to time at
some colleagues boxes with our main project which still is on a very old
Gradle version.
Hi hoped this is fixed in the meantime, I have no way to reproduce it and
was not able to investigate it yet, as it only happened at colleagues and I
had to bring them back to work fast. Would be great if you find what out
what causes this. In the past when I saw this behavior I let them delete
/.gradle and everything in ~/.gradle except for the
properties file and then it worked again like expected.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gradle/gradle/issues/16671#issuecomment-811435346,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJVHFGMCLRQIJGUASBYFBLTGN7ZJANCNFSM42BXIPSA
.

--

The line
ignoreFailures = true // Always try to run all tests for all modules

looks extremely suspect. It means you end up with BUILD SUCCESSFUL regardless of how many tests have failed.
This is turn, might give Gradle the green light to decide that the tests have run successfully, and that they no longer need to be re-run.
I contacted the author of the code that does the summarizing of test results (a feature still not in Gradle, even 7), to understand what the intent was, and to determine whether there is a better way.
https://gist.github.com/ethanmdavidson/a73147ce5bdcde4a87554c7303bae8f4#gistcomment-3722581

Closing it since my main suspect is now ignoreFailures = true, which is unrelated to Gradle.

Yes, ignoreFailures does exactly what you say and should explain the behavior.
It means as it says, that failures are ignored and thus the task will be seen as successful and thus not re-run.
IntelliJ takes special measures that if you run tests from IDE they are always rerun, even if all were successful before and nothing changed.
Judging from the comment after the ignoreFailures = true, I'd say the better way to achieve the wanted behavior is to use --continue which will still fail the build if there were failures, but will execute all tasks that do not depend on one of the failed tasks.

Thanks for following up and updating this issue.
Any test task that is successful, whatever the reason, can then be considered up-to-date and even have its output cached.
So ignoreFailures = true is indeed dangerous in that context.

I'd say the better way to achieve the wanted behavior is to use --continue

Agreed, came to the same conclusion.


Can't trigger fcitx

My system information
Linux 5.13.12-arch1-1 x86_64 GNU/Linux

Issue I met
I am a Chinese user. And I used fcitx as the Chinese input method. However, I can't trigger the input panel like other neovim GUI or the TUI when I am using the nvui.

Normal behavior in terminal

In nvui
It just outputs English nor creates the fcitx input panel.

My fcitx version:
fcitx version: <IP_ADDRESS>

This one is a little tough for me since I don't have much experience with Linux or input methods in general. I'll need to dig a little deeper into this.
Does Neovim-Qt work with fcitx?

This one is a little tough for me since I don't have much experience with Linux or input methods in general. I'll need to dig a little deeper into this.
Does Neovim-Qt work with fcitx?

Yes, it works.

nvim-qt

I've tried using a Chinese Pinyin input method on Ubuntu and can confirm that the IME display doesn't show.
I'll have to look into this.

Hello, can you check if #56 fixes this?

No, the fcitx panel is still not shown.

Also, here is my fcitx diagnosis information: https://cfp.vim-cn.com/cb30K. Maybe some input module missing.

My fcitx5 works fine in v0.1.2

My fcitx5 works fine in v0.1.2

How do you set your fcitx keymap shortcut? In my machine <Ctrl+Space> and <Ctrl+Shift> are all not working. The Ctrl+Space just act like a normal space input.

Also, here is my fcitx diagnosis information: https://cfp.vim-cn.com/cb30K. Maybe some input module missing.

The diagnosis information says "Cannot find fcitx input method module for Qt4". Maybe some Qt4 modules are missing? But  I can trigger the fcitx on nvim-qt, it is weird.

How do you set your fcitx keymap shortcut?

Ctrl+Space

May I ask how your fcitx environment variables are set?

May I ask how your fcitx environment variables are set?

GTK_IM_MODULE=fcitx5
INPUT_METHOD=fcitx5
QT_IM_MODULE=fcitx5
XMODIFIERS=@im=fcitx5

Same environment variable setting. So it's not related to it.

I think I found out the problem. For people who have trouble like me: you need to compile it at local, the binary in action is not compatible with fcitx.


Travel Budget Calculator Extension

Is your feature request related to a problem? Please describe.
The Travel Budget Calculator Extension is a Chrome extension that helps users estimate the total cost of their trip, including flights, accommodation, food, and activities.
Features

Calculate the travel budget based on:

Flights cost
Accommodation cost (per night)
Number of nights
Daily food cost
Number of days
Activities cost

Clear and intuitive UI
Responsive design for various screen sizes

Usage

Click on the extension icon in the Chrome toolbar to open the calculator.
Enter the cost of your flights.
Enter the cost of accommodation per night.
Enter the number of nights you'll stay.
Enter the daily cost of food.
Enter the number of days of your trip.
Enter the cost of activities.
Click the "Calculate" button to see your estimated total travel cost.
Click the "Reset" button to clear all inputs.

What problem is this feature trying to solve?
It is useful for travellers to estimate the total cost of their trip, including flights, accommodation, food, and activities.
How do we know when the feature is complete?
I will raise a PR for it.

I would like to work on this issue , please assign this to me.
@Sulagna-Dutta-Roy

@Antiquely3059 you should not take 2-3days for a single project.
If I don't received any update after 3days I will assign to another contributor thanks.


remove modal close on random clicks

it happened to me for a second time, i have somehow managed to close a modal with a random click inside a modal

this should not happen and modal should only close with the x-mark icon

it was an autoteleport modal, at the time when information popup with a hash come out

in this step

could you have a look @hassnian please


Fresh Akavache Install Rendered Xamarin iOS app Dead.

I've never used Akavache so on recommendation I installed it today to get it integrated.  I can no longer even debug the app.
The type initializer for 'Akavache.BlobCache' threw an exception. ---> System.IO.FileNotFoundException: Could not load file or assembly 'Splat, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies.
File name: 'Splat, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
When looking at the library references that have been added, it installed a referenced Splot v1.3.3, not <IP_ADDRESS> ...
I had to manually update Splat to 1.6.0 using NuGet.

duplicate

Closing as duplicate


Eliminate possible infinite loop in exec

If a user starts an exec session with a shell and leaves it idle long
enough, they will eventually hit the Kubelet's HTTP server's read/write
timeout of 5 minutes. At this time, the StartExec call to Docker exits,
but if the user requested a TTY, the exec'd process does not exit.
After StartExec finishes, we try to determine the exit code of the
exec'd process, but in this case, we'll never get it. This change exits
the loop after 5 tries if the process is still running.

LGTM - should be candidate for 1.0 due to leaking goroutines

And post 1.0, we should try to find something other than 'docker exec' if possible.

@k8s-bot test this please


ViaVersion 1.13-Blocks are strange

Glass is no longer a block
Restone can't connect with each other
Flags are invisible

#856


Breaking changes to config

https://github.com/open-contracting/kingfisher-views/issues/34

Do NOT merge till master til deploy changes done or you'll just break the live deploy!

This comes with https://github.com/open-contracting/kingfisher/pull/328/files
Problem; one of the reasons this was done was so we can remove the setup.py from process. But ...... look in .travis.yml - it needs to install process to make the tests work! But we can maybe merge this, then look at tests.

I figure the tests should git clone kingfisher-process and pip install its requirements – like when deploying the process app.

https://github.com/open-contracting/deploy/pull/37 is WIP for deploy repo but it hasn't been tested at all

I figure the tests should git clone kingfisher-process and pip install its requirements – like when deploying the process app.

I tried this in a separate branch but there are problems. Process is written with a different line length flake8 setting - longer than 119.  So I got the tests to work, but then flake8 failed. To avoid this pull request growing, I'll suggest we keep to this scope and then look into this later in a new issue.

I tried this in a separate branch but there are problems. Process is written with a different line length flake8 setting - longer than 119. So I got the tests to work, but then flake8 failed. To avoid this pull request growing, I'll suggest we keep to this scope and then look into this later in a new issue.

Sounds good. I think easiest might be to fix line lengths in KF Process, as I'm not sure why one project needs a longer line length.

I opened https://github.com/open-contracting/kingfisher-views/issues/41

Deploy commit merged and deployed. This branch Merged and deployed.


Roll engine d004bcd4d619..5153075e5fdf (1 commits)

https://github.com/flutter/engine/compare/d004bcd4d619...5153075e5fdf
git log d004bcd4d619fc3574761d63d7cf7b7291332c79..5153075e5fdfdbcdb2c7e9fae3743ca015c68773 --no-merges --oneline
5153075e5 Roll src/third_party/skia f3881b278e69..6bbeb4ab7222 (1 commits) (flutter/engine#9438)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (amirha@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Views in ForEach recomputed upon element addition

Description
TCA 1.9.2 (also tested on 1.7.3 and main).
When elements are added to an IdentifiedArray that drives ForEach, every View inside the loop gets recomputed. This behavior is present in both iOS 17+ and earlier versions (with WithPerceptionTracking). I have also tried the old method - using WithViewStore, ForEachStore, and State not being @ObservableState, but it doesn't seem to make any difference.
In vanilla SwiftUI, this does not happen in iOS 17+ with the @Observable macro. Only views corresponding to added elements get recomputed. However, it does happen in vanilla SwiftUI on iOS <17 with the ObservableObject protocol and @Published.
Please refer to the videos below::
Vanilla SwiftUI with @Observable macro:
https://github.com/pointfreeco/swift-composable-architecture/assets/118171669/18370627-708f-4991-8a54-aefd3a846a60
TCA:
https://github.com/pointfreeco/swift-composable-architecture/assets/118171669/4e7bf5b9-aa85-4614-b1dd-9403adcfce1c
Sidenote:
I am using TCA in a large application with lists containing dozens of heavy views, and adding elements to the driving array while scrolling causes hangs because each view gets recomputed.
Checklist

[X] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[X] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
Views inside ForEach don't get recomputed, as in vanilla SwiftUI.
Ideally on iOS <17 as well.
Actual behavior
Every view that is placed in a ForEach gets recomputed when elements are added to the corresponding array.
Steps to reproduce
Here is a sample project where this issue reproduces.
https://github.com/bohdany-cricut/TCA_Perception

Open Project
Uncomment either TCA or SwiftUI version of the view in the TCA_PerceptionApp.swift
Run the App
Observe View updates

The Composable Architecture version information
1.9.2
Destination operating system
iOS 17.2, iOS 15.0
Xcode version information
Version 15.3 (15E204a)
Swift Compiler version information
swift-driver version: <IP_ADDRESS> Apple Swift version 5.10 (swiftlang-<IP_ADDRESS> clang-15<IP_ADDRESS>)

Hi @bohdany-cricut, thanks for bringing this up! It is true this happens in TCA today, and we have plans to address it, but it's going to take time to get there.
Luckily this only affects IdentifiedArray (since it is not observable itself) and further only affects the rows visible on the screen. So hopefully it isn't that big of a deal.
I'm going to convert this to a discussion since it can't be addressed right away, but do know that we are working towards a world where this will be possible. Please feel free to continue the conversation over there if you want.


Looks like it's working but no sound

Looking in my audio settings i can see that the volume is going up and down but no sound coming out from my speakers
Boardname
meep
Logs
Generate logs with [debug-logs.tar.gz](https://github.com/WeirdTreeThing/chromebook-linux-audio/files/14194045/debug-logs.tar.gz) and attach the output here.

debug-logs.tar.gz

[   11.497037] input: HDA Intel PCH HDMI/DP,pcm=3 as /devices/pci0000:00/0000:00:0e.0/sound/card0/input7
[   11.497122] input: HDA Intel PCH HDMI/DP,pcm=7 as /devices/pci0000:00/0000:00:0e.0/sound/card0/input8
[   11.497192] input: HDA Intel PCH HDMI/DP,pcm=8 as /devices/pci0000:00/0000:00:0e.0/sound/card0/input9

Your kernel doesn't have sof drivers?

So i'll have to install that then

No, they should be there. Maybe it isn't selecting the correct driver though. Make a file /etc/modprobe.d/sof.conf with this line options snd-intel-dspcfg dsp_driver=3.

I have no clue why it didn't use the correct driver by default.


Tile 5165 marked unsure -- petal 7 had poor positioning, should be omitted entirely

Hi @akremin , any chance you could reprocess tile 5165, but omit Petal 7 entirely?  It was processed and the majority of sources were marked as bad due to poor positioning, but I think we'd prefer to just ignore the whole petal.

20221001/145206, tileid 5165.  P7 needs to be marked bad and then 5165 reprocessed---thanks!

@akremin , pinging on this outstanding low dec tile, thanks!

p7 on 145206 has been marked as bad and the exposure+tile reprocessed. I've rerun the daily tables and regenerated nightqa.

Looks good, QA is now good, closing.


Add food carts

About the PR
Adds hot and cold food carts for kitchen staff to stretch their legs and sell their goods around the station. Can be mapped in or bought from cargo.
Media
https://github.com/space-wizards/space-station-14/assets/107660393/ffe2b188-de88-49fa-8028-9cb6e9948d3a

[x] I have added screenshots/videos to this PR showcasing its changes in-game, or this PR does not require an in-game showcase

Changelog
:cl: Velcroboy and Rosy!

add: Added food carts. Hot dogs here! Get yer hot dogs here!

i really really like these

Requests resolved.

these sounds dont appear to be licensed under cc0

these sounds dont appear to be licensed under cc0

Can you explain that for me? I know absolutely nothing about licensing.

Initially I tried to put them under "CC BY 4.0" but that caused the test to fail.

the website they are from says they are licensed under cc by 4.0 which means you are free to share and adapt as long as there is attribution and no restrictions.

the website they are from says they are licensed under cc by 4.0 which means you are free to share and adapt as long as there is attribution and no restrictions.

Did I not attribute correctly?

use CC-BY-4.0

K, I think that worked

i wish they opened up like ss13 and was anchored with a little bench people can take food from


[Feature Request]

Can you add  migration features same as  orginal tachiyomi. In original you don't have to search more it can directly search pinned source but in sy we have to select sources or  it will be automatically selected.
Check on this.
Thanks for reading.

When you get to the menu where it searches for the manga you can click the overflow icon beside the manga and search manually


Issue with CarlaUE4.exe requiring specific DirectX 11 feature level

Hello Carla developers,
I am trying to run Carla on a Windows Server 2019 instance on AWS EC2 g4dn.xlarge, which comes with an NVIDIA Tesla T4 GPU. However, when I try to run CarlaUE4.exe, I encounter an error message stating "DX11 feature level 10.0 is required to run the engine".
My system supports DirectX 12, and when I run dxdiag, it shows DirectX 12. Also, I have updated my NVIDIA drivers to the latest version.
I have tried the solutions suggested in a previous issue (#3904), but they did not resolve the problem. Specifically, I downloaded and installed an older DirectX 11 installer and tried Carla version 0.9.4, but I still get the same error message.
Additionally, I am using NICE DCV for remote desktop connection, which should properly expose the GPU.
If you have any advice on this issue, please let me know. Any suggestions are welcome.
Thank you in advance.

@sanadaryo, I have been facing the same issue,
Please let me know if you have figured out the solution.

me too


Array dimensions are transposed in MATLAB and in HDF5 file

Similar to this issue in snirf_homer3 https://github.com/fNIRS/snirf_homer3/issues/4
easyh5's array dimensions are transposed inside matlab and on disk (in hdf5 datasets).
This can be reproduced by running this in matlab
data2hdf=reshape(1:(2*4*6),[2,4,6]);
size(data2hdf)
saveh5(data2hdf,'test.h5')

then in a terminal, start python (after sudo apt-get install python-h5py python-numpy), and run
import h5py
import numpy as np

dat=h5py.File('test.h5','r')
d1=np.array(dat.get('/data2hdf'));
d1.shape
    (6, 4, 2)

or run h5dump test.h5
fangq@mars:~/space/solar/Gitroot/Project/github/easyh5$ h5dump test.h5
HDF5 "test.h5" {
GROUP "/" {
   DATASET "data2hdf" {
      DATATYPE  H5T_IEEE_F64LE
      DATASPACE  SIMPLE { ( 6, 4, 2 ) / ( 6, 4, 2 ) }
      DATA {
      (0,0,0): 1, 2,
      (0,1,0): 3, 4,
      (0,2,0): 5, 6,
      ...
      (5,3,0): 47, 48
      }
   }
}
}

as we can see, the python dataset dimensions are transposed compared to matlab's array, as expected. This is known because of the row-major/column major differences, see
https://www.mathworks.com/matlabcentral/answers/308303-why-does-matlab-transpose-hdf5-data
as the SNIRF specification defines arrays in HDF5 dataset constructs, so, this is considered a non-compliance, and should be fixed.
Note that this is not an issue when JSNIRF is used because JSONLab/JSNIRFY toolboxes transpose arrays before saving to JData _ArrayData_ construct (also row-major) and convert back when decoding.

Issue is now fixed with the above patch. Both loadh5 and saveh5 now have a new option Transpose and by default it is set to 1. Therefore, when saving, matlab data is converted to c/hdf5 array, and then when loading, c/hdf5 arrays are converted back to matlab. Arrays loaded in python now show consistent dimensions and values. see
data2hdf=reshape(1:(2*4*6),[2,4,6]);
size(data2hdf)
data2hdf(1,3,2)
data2hdf(19)
saveh5(data2hdf,'test.h5')
newdata=loadh5('test.h5');
size(newdata.data2hdf)
newdata.data2hdf(1,3,2)
newdata.data2hdf(19)

in Python, both the dimensions and indexing are consistent (python indices starts from 1)
>>> dat=h5py.File('test.h5','r')
>>> d1=np.array(dat.get('/data2hdf'));
>>> d1.shape
(2, 4, 6)
>>> d1[0,2,1]
13.0
>>> np.ravel(d1,order='F')[18]
19.0


The InferenceService "sklearn-iris" is invalid: metadata.managedFields.fieldsType: Invalid value: "": must be FieldsV1

/kind bug
What steps did you take and what happened:
I am trying to test KFServing Installation, setting up for the first time from Test KFServing Installation .
Not really sure if its a bug?
Commands shown below
kubectl create namespace kfserving-test
kubectl apply -f docs/samples/sklearn/sklearn.yaml -n kfserving-test

Error message:
The InferenceService "sklearn-iris" is invalid: metadata.managedFields.fieldsType: Invalid value: "": must be `FieldsV1`

What did you expect to happen:
$ inferenceservice.serving.kubeflow.org/sklearn-iris created

Anything else you would like to add:

I have got istio installed with manifest from Installing Istio without sidecar injection and Knative components using Operators
Not sure if its an issue with any of the versions i am using?

$ kubectl get pods --namespace knative-serving
NAME                                READY   STATUS     RESTARTS   AGE
activator-6d9f95b7f8-k5gtd          2/2     Running    0          24m
autoscaler-597fd8d69d-b6htb         2/2     Running    0          24m
autoscaler-hpa-5878b57d96-fvrm7     2/2     Running    1          24m
controller-7479cc984d-m7dxh         2/2     Running    1          24m
istio-webhook-64b48869b8-nmckd      2/2     Running    1          24m
networking-istio-7bb58f6d55-2hlgw   1/1     Running    0          24m
storage-version-migration-bcq59     1/2     NotReady   2          24m
webhook-bf465f954-2nbq9             2/2     Running    1          24m

Environment:

Istio Version:

$ istioctl version
client version: 1.6.4
control plane version: 1.6.4
data plane version: 1.6.4 (10 proxies)

Knative Version:

v0.15.0

KFServing Version:

$ kubectl get KnativeServing knative-serving -n knative-serving
NAME              VERSION   READY   REASON
knative-serving   0.15.0    True

Kubeflow version:
NA

Kfdef:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm]
NA

Minikube version:

$ minikube  version
minikube version: v1.11.0
commit: 57e2f55f47effe9ce396cea42a1e0eb4f611ebbd

Kubernetes version: (use kubectl version):

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.5", GitCommit:"e6503f8d8f769ace2f338794c914a96fc335df0f", GitTreeState:"clean", BuildDate:"2020-06-26T03:47:41Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"windows/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.3", GitCommit:"2e7996e3e2712684bc73f0dec0200d64eec7fe40", GitTreeState:"clean", BuildDate:"2020-05-20T12:43:34Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
``
- OS (e.g. from `/etc/os-release`):
```sh
Edition          Windows 10 Enterprise
Version             1903
Installed on       24/01/2020
OS build            18362.778

@kmr0877 you are on kubernetes 1.18 and looks like this is the same issue as https://github.com/GoogleCloudPlatform/flink-on-k8s-operator/issues/210, if you downgrade to 1.17 that should work. We will need to upgrade k8s client dependency for KFServing controller.

@yuzisun Seems like on downgrade to 1.17 works fine for me. Thanks for your response.
Worth upgrading k8s client, probably better to keep the issue open until its upgraded else we might end up seeing duplication as you mentioned above.
Feel free to close the issue if you have a different opinion.

released in 0.4.1


Does this system ensure no freak trades happen ?

We have support for both SL-L and SL-M orders.
SL-L orders do not guarantee execution and are hence monitored in a way that if the order stays Open for more than 30seconds after getting triggered, the algorithm would modify the order type from Limit to Market
While, SL-L orders are live, the modification of order type from Limit to Market is still under testing and will be rolled out in the coming week.

Thanks for the info.


Fixing one Typo in otrmodule.py:80 and case in connection.py:5

Not merging this as-is since "import queue" does not work in Python 2, which this module targets.

Okay, thank you


update the chunking type situation

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

Comparison is base (581bcf1) 68.48% compared to head (b69ba79) 68.48%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage   68.48%   68.48%
=======================================
  Files          17       17
  Lines         882      882
=======================================
  Hits          604      604
  Misses        278      278

Flag
Coverage Δ

unittests
68.48% <100.00%> (ø)

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

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


Dataset balancing

🚀 Feature
The ability to have a Flash DataModule sample its data in manner that counters any imbalance in the number of labels present for each class.
Motivation
When training a classifier on a dataset where all classes are not represented equally in numbers, the resulting model is likely to develop a bias for the classes that are more represented and become less capable at identified under represented classes. Currently when using a Flash DataModule subclass for loading data, not only does Flash not provide any solutions to this problem, the current implementation of the DataModule actually makes it harder to address the issue than plain pytorch lightning.
Pitch
I suggest a two-step approach to solve this.
Step 1: expose the option to pass a torch.utils.data.Sampler to the pytorch DataLoader being used in the DataModule class. This paves the way for allowing custom sampling strategies in general.
Step 2: add an option for the DataModule to count the number of labels for each class in a dataset, create a set of weights for the sampling probability for each sample depending on the label it contains, and pass a torch.utils.data.WeightedRandomSampler with the given weights to the training set dataloader.
Alternatives
Step 1 described above will on its own at the very least not hinder people from implementing their own sampling logic will still using the convenience of the Flash DataModule.
Additional context
It should be noted that my above suggestion in step 2 for the logic to be used is fairly trivial in cases with one class label per sample. However, in cases with multiple labels per sample, it quickly becomes less simple. I don't think that this should be an argument for not supplying the option for cases when it can be useful, though.
Note also that I have not completely covered the logic needed here. For instance, if we want to retain the concept of epochs, it can no longer mean the time when the entire dataset has been seen once, since allowing each sample to be seen exactly once will mean that the model ends up being trained with the same skewed distribution as is present in the dataset.

Hi @lillekemiker thanks for the feature request! Step 1 from your proposal definitely sounds like something we can do. In general it would be nice to have a way of customising the data loader creation in the datamodule. Adding sampler arguments might work but would need those arguments to be added to each from_* method, which could get a bit messy. I guess we could have them as properties instead so you could do:
datamodule.train_sampler = ...

We could also create a ClassificationDataModule that has a balance method or similar to auto generate a weighted sampler. Do you have any other ideas about how it could be done? Would you be interested in having a go at this yourself?
Thanks :smiley:


Simple append example fails

const uuid = require('node-uuid');
const EventStore = require('ges-client');

const connection = EventStore();

connection.on('connect', ()=> {
    const streamId = 'intro-stream';
    const appendData = {
        expectedVersion:EventStore.expectedVersion.noStream,
        events: [
            {
                eventType: 'Foo',
                eventId: uuid.v4(),
                data: {
                    name: `Hello World!`,
                    number: Math.random()
                },
                metadata: {
                    addedDate: new Date().toISOString()
                }
            }
        ]
    };

    connection.appendToStream(streamId, appendData, (err, results)=> {
        if (err) return console.error(`Ooops`, err);

        connection.readStreamEventsForward(streamId, {start: 0, count: 1}, (err, results)=> {
            if (err) return console.error(`Oppps read:`, err);

            console.log(results.Events);
        });
    });
});

This gives me an error of
node_modules\node-uuid\uuid.js:67
    s.toLowerCase().replace(/[0-9a-f]{2}/g, function(oct) {
     ^

TypeError: Cannot read property 'toLowerCase' of undefined

Looking through the code it appears you are using EventId, though the README.md doesn't discuss the internals of what events look like.  Captializing EventType, EventId, Data and Metadata however gives me
protobufjs\node_modules\bytebuffer\dist\ByteBufferNB.js:378
                throw TypeError("Illegal buffer");
                ^

TypeError: Illegal buffer

What would be the correct version of the initial test code?

Hi,
Sorry that the documentation is lacking. For now, this would be the way to fix your code would be to create the event in one of the two following ways:
const uuid = require('node-uuid');
const EventStore = require('ges-client');

const data = new Buffer(JSON.stringify({
                    name: `Hello World!`,
                    number: Math.random()
                });
const metadata = new Buffer(JSON.stringify({
                    addedDate: new Date().toISOString()
                });

const event1 = EventStore.createEventData(uuid.v4(), 'Foo', true, data, metadata);

or
const event1 = {
                EventType: 'Foo',
                EventId: uuid.v4(),
                Data: new Buffer(JSON.stringify({
                    name: `Hello World!`,
                    number: Math.random()
                })),
                Metadata: new Buffer(JSON.stringify({
                    addedDate: new Date().toISOString()
                }),
               // You likely want to add IsJson here
               IsJson: true
            }

There is something more user-friendly coming in the next release, which will allow javascript objects to be passed into the createEventData function (see https://github.com/bmavity/ges-client/blob/dev/eventData.js ), but the buffers are ultimately necessary for the EventStore.

Thanks for the heads up, dealing with the buffers directly is fine, and long as I know that's what you need to do.  Probably should keep open until the README.md matches this.

So the first way worked fine, the second however gives this error
ges-client\node_modules\protobufjs\dist\ProtoBuf.js:2640
                        throw(e);
                        ^

Error: Illegal value for Message.Field .EventStore.Client.Messages.WriteEvents.events: .EventStore.Client.Messages.NewEvent (Error: Missing at least one required field for Message .EventStore.Client.Messages.NewEvent: .EventStore.Client.Messages.NewEvent.eventType)
    at Error (native)

I fixed the typo in the above. As far as ES6 goes, I will eventually move that way, but I need to do a bit more reading on the proper way for module to handle this when it is still targeting v0.10.x.


Disable TLS 1.0/1.1

Is it at all possible to disable TLS 1.0 and 1.1.
PCI scans have started to fail due to TLS 1.0 being supported by this package, as such, it would be great to only support TLS 1.2+ (with maybe an option to configure the minimum supported version should 1.2 become deprecated also).

If all else fails, you can fork the repo :D

If all else fails, you can fork the repo :D


Proposal to consolidate csproj files to a single .NET SDK style project

With VS 2017 (and the dotnet CLI) comes the ability to have a single project target multiple frameworks and have a far simpler project file.
I'm preparing a change to this repo to migrate to it. Would you entertain a PR?

This was already done for Json.NET here. Maybe this was posted in the wrong repo?

Ah! Thanks. I was looking at the 9.0.1 release's source code.


set ses email content type to utf-8 instead of string

I was testing SES notification and hit this error...
Traceback (most recent call last):
  File "/opt/app/lemur/lemur/notifications/messaging.py", line 97, in send_notification
    notification.plugin.send(event_type, data, targets, notification.options)
  File "/opt/app/lemur/lemur/plugins/lemur_email/plugin.py", line 115, in send
    send_via_ses(subject, body, targets)
  File "/opt/app/lemur/lemur/plugins/lemur_email/plugin.py", line 71, in send_via_ses
    'Charset': 'string'
  File "/usr/local/lib/python3.5/site-packages/botocore/client.py", line 310, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/usr/local/lib/python3.5/site-packages/botocore/client.py", line 599, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (InvalidParameterValue) when calling the SendEmail operation: Unsupported charset 'string'.
Finished notifying subscribers about expiring certificates! Sent: 0 Failed: 2

In the email plugin, it looks like Charset is set to the value 'string', whereas SES needs an actual charset.
So I think the safest might be to use UTF-8 for now. I set it to this and my notifications then worked.

Coverage remained the same at 60.493% when pulling 07151b17692d30a91997adfd7374494afebdb4de on pr8kerl:ses-fix into 98907e66e9230b2019d328198b287eda7c7fc046 on Netflix:master.

I agree utf-8 is probably safer here. Thanks!


Support for private blobs for primaryscripturi and secondaryscripturi

[Enter feedback here]
Does this feature for deployment scripts in templates only work with public blobs for primaryscripturi and secondaryscripturi? I am looking to use efmigrate to run my db migration as part of my deployment and using custom scripts seemed to be the solution but it is unable to download files from private blobs and does not have any auth mechanism to use the way we do with sql import. Is their a way it can work?

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

ID: b5c4c8c5-7757-a969-5800-17f2f2312649
Version Independent ID: 9ced8dd5-43ca-6d1f-c054-842299db8327
Content: Use deployment scripts in templates - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/deployment-script-template.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@abhinavdawra Thanks for your comment! We will review and provide an update as appropriate.

@abhinavdawra - You can use SAS token.  See https://docs.microsoft.com/en-us/azure/azure-resource-manager/templates/deployment-script-template?tabs=CLI#use-external-scripts.
#please-close


Error: failed to download module

Description
Unable to download these modules.
Error: Failed to download module

Could not download module "eks" (eks.tf:74) source code from
"git::https://github.com/terraform-aws-modules/terraform-aws-eks?ref=v19.16.0":
error downloading
'https://github.com/terraform-aws-modules/terraform-aws-eks?ref=v19.16.0':
git must be available and on the PATH.

Error: Failed to download module

Could not download module "external_dns_irsa_role" (eks.tf:218) source code
from
"git::https://github.com/terraform-aws-modules/terraform-aws-iam?ref=v5.29.1":
error downloading
'https://github.com/terraform-aws-modules/terraform-aws-iam?ref=v5.29.1': git
must be available and on the PATH.

[ x] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Module version [Required]:

Terraform version:
v1.5.6

Provider version(s):

Git version:
2.420

Reproduction Code [Required]

Steps to reproduce the behavior:

Expected behavior

Actual behavior

Terminal Output Screenshot(s)

Additional context
I am running on an Apple M1 Max Chip. OS is Ventura 13.5.1

I'm not seeing any issues with pulling down module definitions with terraform init


Can compile invalid Elm with self-reference in closure. Allows undefined value and can cause runtime error.

The following program defines badboy of type string, which uses itself in it's definition. I'm not sure if this should recurse (or not compile at all), but what happens is you see 'undefined' on the screen once compiled.
This allowed me to introduce a crash in another program where I was trying to access a property on a record set up a similar way to badboy below.
Windows 10 / Chrome / Elm 0.18.0
module Main exposing (..)

import Html

main : Program Never () ()
main =
    Html.program
        { init =
            ( ()
            , Cmd.none
            )
        , view =
            \_ ->
                let
                    badboy =
                        Just "hello"
                            |> Maybe.map (\_ -> badboy)
                            |> Maybe.withDefault ""
                in
                    Html.div [] [ Html.text badboy ]
        , update = \_ mdl -> ( mdl, Cmd.none )
        , subscriptions = \_ -> Sub.none
        }

Thanks for the report!
In 0.19, this particular example will refuse to compile with the familiar "cyclic value" error.
The general idea is that cyclic values will no longer be allowed, except if there is at least one function (rather than just lambda) involved in the cyclic chain. This should still allow expressing whatever you need when you do need cyclic stuff, but hopefully prevent a bunch of silly-yet-easy-to-make mistakes.
Hope this helps!


Refactor SubmitTransaction to fix multiple send issue

Fixes #15403
Sending more than one transaction had an unexpected behavior, sending empty transaction bodies after the first request.
This PR refactors the way we build the transaction request so that there is less state to keep track of and make sure we reset the remaining state and pipeline policies before starting to build a new Transaction Request

@joheredi I notice we now have a circular import between TableClient.ts and TableTransaction.ts -- doesn't this cause some trouble for bundlers?


Sync merge.

Does the Poort8.NoodleBar.postman_collection.json in the Wiki need an update that we know?

Yes, the docs will be up next.


Delete apps/volsync directory

I don't use this image anymore, if you are using this please migrate to butnami instead
https://hub.docker.com/r/bitnami/volsync

The bitnami image sadly doesn't work with the official volsync helm chart out of the box.

the manager binary is not in /manager anymore
the --*-container-image flags are not "supported"
additionally the usual Registered Movers: [rclone restic rsync rsync-tls syncthing] logged list is empty
and it logged quite a few errors that the "original" didn't^^

I wasn't able to find the source code of the container, so no idea what they did :sweat_smile:


Fixed standalone example

Fixed docs to be more hello-world friendly for standalone
Makes the red dot visible (at zoom 3 or 1 it was not visible) #2935

For #2935

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

Coverage remained the same at 61.813% when pulling 376c38e520d7c385d21f94e5e1424d5fe7a4f5db on DominusVilicus:patch-1 into 2fc418d6505f300ad74d818f3b445fea99541cdf on uber:master.


Create page for the Windows/macOS specific aspects of "Platform Specific Code"

This page exist in the main documentation:
https://reactnative.dev/docs/platform-specific-code
I don't think we have a clear place in the documentation that covers what strings to use for Platform.select, for example. We should at least highlight that, if not any other information/tips for people to organize their platform code.

Correct string for Windows should be "windows". Not sure what the macOS one is. Should be able to look at what Platform.OS reports.


မရits set true/false in the appropriate methods(reserveToken\mintItemForWord) in the contract.

မရits set true/false in the appropriate methods(reserveToken\mintItemForWord) in the contract.
But how is this related -  ipfs metadata and contract?
Originally posted by @keepRunning in https://github.com/GraphemeNFT/rarible-scaffold/issues/93#issuecomment-911242936

မရ


Visualizer doesn't reflect when a node has been promoted

After promoting a node
docker@node1:~$ docker node promote node2
Node node2 promoted to a manager in the swarm.

The visualizer still shows it is a worker:

After refreshing the visualizer, it shows the correct role:

Fixed


[SPARK-12103][Streaming][Kafka][Doc] document that K means Key and V …

…means Value

Test build #47147 has started for PR 10132 at commit b98317f.

Test build #47147 has finished for PR 10132 at commit b98317f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47147/
Test PASSed.

LGTM

LGTM.

Merged to master/1.6


Option to not insert blank lines between headings.

Does this feature exist in Emacs orgmode core?
Yes
Orgmode link
http://doc.endlessparentheses.com/Var/org-blank-before-new-entry.html
Feature value
No response
Additional context
No response

@kristijanhusak is it possible to change the behaviour of orgmode-org_meta_return
your plugin currently insert a line under the header, I want it to be like doom emacs
doom emacs

nvim

@rahulsalvi @mortezadadgar Support for this is added. You can see it in docs here
TL;DR, add this to your orgmode config:
  org_blank_before_new_entry = {
    heading = false,
    plain_list_item = false,
  }

If I hit <Leader><CR> the new TODO headline is not inserted after the very of the current TODO as it can be seen in the following screencast

Is this the normal behavior?

@steigerth yeah, that's normal. To add at the end, do <leader>oit.


autorandr: 1.6 -> 1.7

Motivation for this change
Update autorandr: 1.6 -> 1.7
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x ] Fits CONTRIBUTING.md.

Success on aarch64-linux (full log)
Attempted: autorandr
Partial log (click to expand)
    udevadm control --reload-rules

post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/wr5chpl569i7ylxqw5jsiwjj2q2n11c4-autorandr-1.7
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wr5chpl569i7ylxqw5jsiwjj2q2n11c4-autorandr-1.7/lib  /nix/store/wr5chpl569i7ylxqw5jsiwjj2q2n11c4-autorandr-1.7/bin
patching script interpreter paths in /nix/store/wr5chpl569i7ylxqw5jsiwjj2q2n11c4-autorandr-1.7
/nix/store/wr5chpl569i7ylxqw5jsiwjj2q2n11c4-autorandr-1.7/bin/autorandr: interpreter directive changed from "/usr/bin/env python" to "/nix/store/965nqm136r3b11s4qv2kxflzm0czpjrk-python3-3.6.6/bin/python"
checking for references to /build in /nix/store/wr5chpl569i7ylxqw5jsiwjj2q2n11c4-autorandr-1.7...
/nix/store/wr5chpl569i7ylxqw5jsiwjj2q2n11c4-autorandr-1.7

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: autorandr
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: autorandr
Partial log (click to expand)
    udevadm control --reload-rules

post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/qd6fq3wnwakz9f5icgx00rmd6f5jwfnj-autorandr-1.7
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/qd6fq3wnwakz9f5icgx00rmd6f5jwfnj-autorandr-1.7/lib  /nix/store/qd6fq3wnwakz9f5icgx00rmd6f5jwfnj-autorandr-1.7/bin
patching script interpreter paths in /nix/store/qd6fq3wnwakz9f5icgx00rmd6f5jwfnj-autorandr-1.7
/nix/store/qd6fq3wnwakz9f5icgx00rmd6f5jwfnj-autorandr-1.7/bin/autorandr: interpreter directive changed from "/usr/bin/env python" to "/nix/store/s3nl02c26303mfjvaaxnc19f8ks9hc8c-python3-3.6.6/bin/python"
checking for references to /build in /nix/store/qd6fq3wnwakz9f5icgx00rmd6f5jwfnj-autorandr-1.7...
/nix/store/qd6fq3wnwakz9f5icgx00rmd6f5jwfnj-autorandr-1.7


fix(notify): 对所有通知默认加入系统管理员接收人

What this PR does / why we need it:

[ ] Smoke testing completed

Does this PR need to be backport to the previous release branch?:
If no, just write "NONE".
If don't know, write "UNKNOWN", and let the reviewer decide.
If yes, write the release branches name in the below format and submit the related cherry-pick PR:

release/3.10

/cc @ioito  @swordqiu  @zexi

/lgtm


Sonar Collector only showing ncloc values

Hi,
I have Sonar 6.3, latest Hygieia code, but it only appears to be pulling ncloc values, (that's all i see in Mongo db code_quality)
 - MONGO_PORT=tcp://hygieia-mongodb:27017
  - SPRING_DATA_MONGODB_DATABASE=dashboard
  - SPRING_DATA_MONGODB_PASSWORD=dbpass
  - SONAR_CRON=0 * * * * *
  - SONAR_URL=http://uk-sonar.almuk.xxxxxxxxxx.corp
  - SONAR_METRICS=ncloc,line_coverage,violations,critical_violations,major_violations,blocker_violations,violations_density,sqale_index,test_success_density,test_failures,test_errors,tests
  - SONAR_VERSION=6.3
  - SONAR_USERNAME=xxxxx
  - SONAR_PASSWORD=xxxxx

If i go to the API in sonar, for the project in question, i get the following values.
https://uk-sonar.almuk.xxxxxxxxxx.corp/api/project_analyses/search?project=com.xxxxxxxx:dualadmin-api
{"paging":{"pageIndex":1,"pageSize":100,"total":4},"analyses":[{"key":"AVzLPVPwLmg51I2DfLPj","date":"2017-06-21T15:18:22+0000","events":[{"key":"AVzLPV_rLmg51I2DfLPu","category":"VERSION","name":"0.0.10-SNAPSHOT"}]},{"key":"AVzLMZpcLmg51I2DfLOs","date":"2017-06-21T15:07:10+0000","events":[{"key":"AVzLMaQxLmg51I2DfLPH","category":"QUALITY_GATE","name":"Green (was Red)","description":""}]},{"key":"AVzLKXsr8cqmUoHeyKgS","date":"2017-06-21T14:58:14+0000","events":[]},{"key":"AVzLJPK_8cqmUoHeyKdD","date":"2017-06-21T14:53:16+0000","events":[{"key":"AVzLJPzz8cqmUoHeyKf2","category":"QUALITY_GATE","name":"Red","description":"Reliability Rating > 3"}]}]}
Any ideas?
@nireeshT

to add, if i insert the test data (https://github.com/capitalone/Hygieia/blob/5bfb135d8a8cfe73dead7d1004454b19da711007/UI/src/test-data/ca_detail.json) then, it all displays as expected, so its definitely something to do with the collector not getting values back.

something else to add, if i change the sonar.metrics, to have say.. "quality_gate_details" first, instead of "ncloc", only "quality_gate_details" data is written into the code quality table..
It appears to be collecting data for only the first metric, and no more..

Note that the properties need to be in this format:
# Sonar server(s) (required) - Can provide multiple
sonar.servers[0]=http://sonar.company.com
# Sonar version, match array index to the server. If not set, will default to version prior to 6.
sonar.versions[0]=6.31
# Sonar Metrics - Required.
#Sonar versions lesser than 5.4
sonar.metrics[0]=ncloc,line_coverage,violations,critical_violations,major_violations,blocker_violations,violations_density,sqale_index,test_success_density,test_failures,test_errors,tests
#for Sonar 5.4 and above
sonar.metrics[0]=ncloc,violations,new_vulnerabilities,critical_violations,major_violations,blocker_violations,tests,test_success_density,test_errors,test_failures,coverage,line_coverage,sqale_index,alert_status,quality_gate_details

arrrrrrrrrrrrrg..
I'm using dockers, the issue is that the shell script to create the .properties is wrong (sonar-codequality-properties-builder.sh)!
Currently its,
#Sonar Metrics
sonar.metrics=${SONAR_METRICS:-ncloc,line_coverage,violations,critical_violations,major_violations,blocker_violations,sqale_index,test_success_density,test_failures,test_errors,tests}
Instead of
#Sonar Metrics
sonar.metrics**[0]**=${SONAR_METRICS:-ncloc,line_coverage,violations,critical_violations,major_violations,blocker_violations,sqale_index,test_success_density,test_failures,test_errors,tests}
Changed, and its working, thanks for pointing me in the right direction @AuditHawks


Duplicate property error message should include the name of the model.

With a spec like:
model Foo { ...Bar, ...Baz }
model Bar { qux: string }
model Baz { qux: string }

It'd be helpful if the error gave the name of the model:
- error duplicate-property: Model already has a property named qux
+ error duplicate-property: Model Foo already has a property named qux

Playground Link

I would expect qux from Baz to override qux from Bar as Baz comes last. Maybe I'm just biased by how Javascript works

potentially refactor the messages for this error


OADP 486 - Update velero service account permissions

/approve

If we are removing velero-privileged here maybe remove the SCC from reconciler in go too?

I didn't even realize we were still creating that SCC. I don't want to introduce too much churn into 1.0.3.... I will keep that for now
open a separate issue to remove the SCC bits

/retest


Some questions about the eval metrics of this paper.

After reading your paper and code, some questions confused me. I would be very appreciate if someone can explain my question.
First, it was argued swin-T is used in this paper, but actually, in your model definition code, it seems to be swin-L.
Second, your evaluation code calculates top-5 in function _average_top_k_result(eval.py)，using all output of modules, such as select, drop, FPN layers, combiner, original output, and cat those output to get the final metrics, is this reasonable?
Thirdly, the highest-5 acc   in your code are not equal to any layer, printed as this picture. So, what is the meaning of highest 1-5? And how to get them?

Well, now I know swin-t == swin-transformer. But the last two question still confuse me a lot now.

And in my environment, my result is as this picture:

So it confuses me that highest results is different with layer 1-4 and the combiner layer.

Hello, I have read the code of this PIM program and there is a part that confuses me. The author seems reluctant to answer questions. I wonder if you could help me with it? In the paper, the author used four types of errors for classification training, but it seems that only the selected points were used for prediction through graph convolution in the end. I'm not sure if my understanding is correct. If it is, why use four types of errors for training instead of just using the error generated by the prediction of graph convolution?


Update readme with minimum coverage reporters

@pmcelhaney, I just noticed that with https://github.com/markis/jest-ratchet/commit/853ba2fe024ec9bd75fbe61c295d308e8520e338, the documentation was updated to include more coverageReporters than are necessary. Was this intentional, or accidental copy-paste?

Coverage remained the same at 95.192% when pulling 9ef2af40f214c24bf8c34f4253900246203c3c6b on readme into e472c7e95974cc30d09cc415498a433ce39828a4 on master.

Coverage remained the same at 95.192% when pulling 9ef2af40f214c24bf8c34f4253900246203c3c6b on readme into e472c7e95974cc30d09cc415498a433ce39828a4 on master.

That was intentional. If someone doesn't have coverageReporters configured and they add jest-ratchet only, they'll lose the default coverage reporters.
If they already have reporters configured, I'm assuming they know what they're doing and won't be confused by the extra entries.

Ah, yeah that's a good point.
There should be an easier way to visually explain to the reader what is necessary and what's not. I know I explain it in the documentation, but since about 99% of developers just copy and paste code without reading the documentation.


Link to the supplementary material doesn't work

The XGBoost paper says a detailed description of the weighted quantile sketch is given in the link below. I tried to open it, but got Not-Found Error.
http://homes.cs.washington.edu/˜tqchen/pdf/xgboost-supp.pdf

There is a reference section in the readme.  Use that link.


ENV field in CHANGE_HISTORY table

Hello,
When I use snowpipe in my installation script, I use a variable --vars "{"ENV":"DEV"}"  to install into a specific environment (using the "USE DATABASE DB_{{ ENV }}_WH" command).
My DBs are named like DB_DEV_WH, DB_UAT_WH, DB_PPD_WH, DB_PRD_WH.
Would it be possible to have a 'environment' field added in the CHANGE_HISTORY table ?

Thanks for the suggestion @qqcvd. I thought about this, adding an environment column to CHANGE_HISTORY table, but after playing around with it a bit decided that it would be more flexible to add the -c option and let the user decided exactly how to deal with environments. Part of the challenge is that if we add native environment support then there needs to be a way to incorporate that into the change scripts themselves. To be fair when I thought through this before we didn't have variable support which made it more difficult. But that said I still think it's more flexible this way.
What's your primary concern? Is it to have only a single CHANGE_HISTORY table per Snowflake account for all environments? The way it works currently you'd have to maintain a separate CHANGE_HISTORY table per environment and leverage the -c option to give each a different name (they can all be in the same DB or separate ones).

to be short, before I encounter you tool, I had already done a logging system to log the installations of my SQL packages. So there is a PACKAGE_DELIVERY_TRACKING table in each DB (an environment = only one DB)  with the following fields :
PACKAGE_ID
INSTALL_DATE
ENVIRONMENT (DEV / INT / PPD / PRD)
TYPE_ACTION (INSTALL / ROLLBACK)
USERNAME
DESCRIPTION
On top of this, in the SF account, there is an admin DB which contains a View accross all the tables, in order to present the log like this (thanks to an union and a sql pivot) :
PACK_ID      INT                    PPD                    PRD
S26_US1000   2020-06-29 16:47:09    [NULL]                 [NULL]
S26_US123    2020-07-06 14:23:42    2020-07-09 16:27:08    [NULL]
S26_US575    2020-10-09 15:49:15    2020-10-12 11:53:54    2020-10-14 11:54:33

My aim is to have a map of the packages installed with the dates.
I agree that I can do the same thing on top of CHANGE_HISTORY table, with the -c option and the variable support.


/status API endpoint overrides configured endpoint

Given a configuration
{
    "provider": "ConceptApi",
    "api": "http://example.org/",
    "schemes": "http://example.com/voc"
}

The schemes endpoint will be ignored but overridden by what is returned by /status endpoint.
I think this and #20 could be solved by filling this._api only after initialization with values from /status in _setup instead of _prepare.

I see your point. There are several different issues here though:

I agree that existing values in the configuration should definitely not be overridden by either assumed endpoints or the result of /status.
Then there's the question if any endpoints should even be assumed. My thought was that if we assume missing values, it would be easier to set up for example DANTE without having to explicitly naming every endpoint. On the other hand, assuming the endpoints also assumes that those endpoints even exist (which is the issue behind #20).
I would think that if there is a response on /status, we don't need to assume any another values. But /status itself has to be assumed if only api is given.

What do you think?

A solved my use case for this issue by putting jskos-server instance behind a proxy so I could change endpoint base URIs. We could document the current behavior:

all endpoints are automatically set when api is given. Explicitly disable an endpoint by setting an it to undef (not tested, should solve #20)
when a status endpoint is set (explicitly or implicitly via api) it's return value will override all endpoints.

That's a good solution. Would you be opposed to using null instead of undefined for explicitly disabling an endpoint? It's much easier to check: status.type === null vs status.hasOwnProperty("type") && status.type === undefined.

Nevermind null!

So, it is now implemented this way:

Explicitly set endpoints always have priority and will not get overridden (even if they are null).

This allows us to disable endpoints if necessary for some reason.

Endpoints that are then still undefined will be set with the result of the status endpoint.
Endpoints that are then still undefined will be implicitly set if the api key is given.

Also, if the request to /status returns explicitly with a 404 error (i.e. it's not a network issue or some other server issue), we'll define that status endpoint as null.
Together with https://github.com/gbv/jskos-server/issues/119, this should now work as expected.

We could change one more thing @nichtich, if you want: If the request to /status was successful, even endpoints that are not explicitly set to null will not be assumed to be there, considering that the server should know best which endpoints exist. What do you think?

This makes a lot of sense

I forgot to implement this because the issue was closed, oops. Will ship a 1.0.1 tomorrow.

[ ] If the request to /status was successful, even endpoints that are not explicitly set to null will not be assumed to be there, considering that the server should know best which endpoints exist.
[ ] Explain this in the documentation!


Check metrics (errors / panics / restarts / etc.) in CI

Optional for neondatabase/company_projects#187.
Problem description / Motivation
When there's degradations that ultimately still allow e2e tests to succeed, our feedback cycle is quite slow — we only find out after cutting a release and deploying to staging (often many days after merging).
There's some clear signals we can check in CI to help with this — e.g. number of failed requests, number of panics (should be zero!), no component restarted, etc.
Feature idea(s) / DoD
This should be part of the e2e-tests workflow, mostly invisible unless it flags something.
Implementation ideas
We can probably make a small script to do this, or maybe write the shell code directly in the github action.

Also can check metrics in unit tests


Update

Hello Wan
Unfortunately I am getting an error.
File "tutsplus-dl.py", line 82
print report
^
SyntaxError: invalid syntax
Using Python 3.4.1 on Windows, got pip, requests and beautifulsoup4 installed and running correctly.
Any help is much appreciated.

Stumbled upon the same error. Hope you have time to fix it :) Nice work though!

@marcianosr Hi noted. I will update the script to support the new site and Python 2 & 3, when I got time.

would also be interested!

I third that!

This script by @mhanson01 is working but has lacks some features like
file sizes and download progress indicator.
Wish @sfdye and @mhanson01 could merge and make this all work beautifully

@monolux @simonschaufi @d3banjan
I'm interested in creating a Chrome Extension that will work with the current layout (I think a Chrome Extension would be best as you get js execution and download progress--not to mention pause & resume & simultaneous downloads--for free). However, my annual membership has lapsed and I can't justify spending the funds to renew. If anyone could lend me their credentials for the purpose of building this, then I'd happily build it! :)

I don't have a membership either.

Well a downloader would only be of any use to someone with download access (i.e. with an active annual membership). So if anyone else here's membership is still active, or if any new folks come along looking for this, I'll be right here.

I tried the following command but failed  python tutsplus-dl.py -u *** -p **   https://code.tutsplus.com/courses/----


TypeError: undefined is not a function when uploading Xcode+Slather reports

We are encountering this issue using the Codecov CircleCI orb 3.2.2, but the problem may be due to a bug in the uploader. We were previously uploading coverage reports successfully with the bash uploader.
Environment is a macOS executor with Xcode 13. We use Slather to generate cobertura xml reports from Xcode's coverage format. We use the extra arguments that Slather shows in their examples. We've tried omitting some of these args and the uploader seems to fail to find the coverage file (which is a strange message, because it appears even though we specify the file location).
Full extra arguments:
-v -Z -R ~/workspace -f ~/workspace/coverage.xml -X xcode -X gcov -X coveragepy
[2021-11-12T20:55:09.760Z] ['verbose'] Start of uploader:<PHONE_NUMBER>760...
[2021-11-12T20:55:09.766Z] ['info']
     _____          _
    / ____|        | |
   | |     ___   __| | ___  ___ _____   __
   | |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
   | |___| (_) | (_| |  __/ (_| (_) \ V /
    \_____\___/ \__,_|\___|\___\___/ \_/

  Codecov report uploader 0.1.12
[2021-11-12T20:55:09.766Z] ['info'] => Project root located at: ~/workspace
[2021-11-12T20:55:09.767Z] ['info'] ->  Token found by arguments
[2021-11-12T20:55:09.768Z] ['error'] There was an error running the uploader: undefined is not a function
[2021-11-12T20:55:09.768Z] ['verbose'] The error stack is: TypeError: undefined is not a function
    at main (/snapshot/repo/dist/src/index.js)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at process.runNextTicks [as _tickCallback] (node:internal/process/task_queues:65:3)
    at Function.runMain (pkg/prelude/bootstrap.js:1848:13)
    at node:internal/main/run_main_module:17:47
[2021-11-12T20:55:09.768Z] ['verbose'] End of uploader: 8 milliseconds

Exited with code exit status 255

Hi @ahampe
Can you set verbose on the orb and reshare the output?
Additionally, -X xcode -X gcov -X coveragepy are not valid values to the new uploader, and are not needed as it doesn't run them by default. What errors do you get with them removed?

Interesting, I thought this failed earlier when I removed the -X arguments, but the issue may have had something to do with using relative rather than absolute paths. It now finds the coverage file when using absolute:
[2021-11-17T17:48:03.763Z] ['verbose'] Searching for files in /Users/distiller/workspace
[2021-11-17T17:48:04.026Z] ['info'] Searching for coverage files...
[2021-11-17T17:48:04.226Z] ['info'] => Found 1 possible coverage files:
  /Users/distiller/workspace/reports/coverage.xml

I still wonder why codecov even needs to search for files when it's specified already (either with -f directly or with file: in our yaml), but it found the right file and is successfully uploading coverage now, so I'll close this.

Interesting, I thought this failed earlier when I removed the -X arguments, but the issue may have had something to do with using relative rather than absolute paths. It now finds the coverage file when using absolute:
[2021-11-17T17:48:03.763Z] ['verbose'] Searching for files in /Users/distiller/workspace
[2021-11-17T17:48:04.026Z] ['info'] Searching for coverage files...
[2021-11-17T17:48:04.226Z] ['info'] => Found 1 possible coverage files:
  /Users/distiller/workspace/reports/coverage.xml

I still wonder why codecov even needs to search for files when it's specified already (either with -f directly or with file: in our yaml), but it found the right file and is successfully uploading coverage now, so I'll close this.

It absolutely does not, thanks for the feedback.
Can you provide a bit more on where you are defining file: in your YAML, I'm blanking on the context.
The uploader doesn't read the codecov.yml, we can add that ability if it would be useful 🤔

Can you provide a bit more on where you are defining file: in your YAML, I'm blanking on the context.

Sorry, it was unclear which yaml I was referring to, but I meant our CircleCI config yaml. We're not even using the file: param anymore though, and are just adding -f /Users/distiller/workspace/reports/coverage.xml in extra args


Fixed scrolling issue in editor

Now the user would be able to scroll the output field of the editor if the input data is large.
Closes #658

Great Work @Rishab-j ✨✨

Thank You @anushbhatia


Shouldn't have to tell my gender if nobody cares

Steps to reproduce
Went to "Look for a Ride"
Clicked on a carpool
Requested a ride
Result: I was asked for my gender
Expected: Since the carpool didn't restrict based on gender, I shouldn't have been asked.

rethinking this one....


Add data migration to include parent content ID on email signups

This is intended to be used by the email-alert-frontend to send on to the
email-alert-api, allowing policy subscriptions to be created by content
ID.
Trello: https://trello.com/c/ZD65QUBN/326-email-alerts-based-on-links-hash

Closing this temporarily while I rethink the order of these policy-related changes - I suspect policy publisher should be updated first so that email-signups carry the links data by default.

policy-publisher has been updated as required: https://github.com/alphagov/policy-publisher/pull/111


[SR-15599] Unexpected thread sanitizer issues with actors

Previous ID
SR-15599

Radar
None

Original Reporter
Cameron (JIRA User)

Type
Bug

Environment
macOS Version 12.1 (21C52)
Xcode Version 13.2 (13C90)

Additional Detail from JIRA

Votes
1

Component/s
swift

Labels
Bug

Assignee
None

Priority
Medium

md5: e537527687d15564192ad661b44dddad

Issue Description:
I am seeing what I believe are unexpected thread sanitizer issues with the following actor code.
Here's the actor:
public actor Example<Input: Hashable, Output> {
    private var inProgress = [Input: Task<Output, Error>]()

    private let transform: (Input) async throws -> Output

    public init(_ transform: @escaping (Input) async throws -> Output) {
        self.transform = transform
    }

    public func transform(_ x: Input) async throws -> Output {
        if let task = inProgress[x] {
            return try await task.value
        } else {
            let task = Task {
                try await transform(x) // the races are reported on this line
            }
            inProgress[x] = task
            defer {
                inProgress[x] = nil
            }
            return try await task.value
        }
    }
}

Here's the test code that demonstrates the issues:
import XCTest
import MyLibrary

final class MyLibraryTests: XCTestCase {
    func testExample() async throws {
        let example = Example { (x: Int) -> String in
            String(x)
        }
        let value = try await example.transform(1)
        XCTAssertEqual(value, "1")
    }
}

Here's my testing output:
Test Suite 'All tests' started at 2021-12-15 10:26:19.904
Test Suite 'MyLibraryTests.xctest' started at 2021-12-15 10:26:19.907
Test Suite 'MyLibraryTests' started at 2021-12-15 10:26:19.908
Test Case '-[MyLibraryTests.MyLibraryTests testExample]' started.
==================
WARNING: ThreadSanitizer: data race (pid=83691)
  Read of size 8 at 0x7b6000020458 by thread T3:
    #0 (1) suspend resume partial function for closure #1 in Example.transform(_:) <null>:2 (MyLibraryTests:x86_64+0x9f1e)
    #1 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Previous write of size 8 at 0x7b6000020458 by thread T1:
    #0 closure #1 in Example.transform(_:) <null>:2 (MyLibraryTests:x86_64+0x9dc3)
    #1 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Location is heap block of size 1016 at 0x7b6000020400 allocated by thread T1:
    #0 malloc <null>:3 (libclang_rt.tsan_osx_dynamic.dylib:x86_64h+0x51d9a)
    #1 swift::StackAllocator<1000ul>::getSlabForAllocation(unsigned long) <null>:2 (libswift_Concurrency.dylib:x86_64+0x32769)
    #2 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Thread T3 (tid=1111954, running) is a GCD worker thread

  Thread T1 (tid=1111953, running) is a GCD worker thread

SUMMARY: ThreadSanitizer: data race (MyLibraryTests:x86_64+0x9f1e) in (1) suspend resume partial function for closure #1 in Example.transform(_:)+0x4e
==================
ThreadSanitizer report breakpoint hit. Use 'thread info -s' to get extended information about the report.
==================
WARNING: ThreadSanitizer: data race (pid=83691)
  Write of size 8 at 0x7b6000020438 by thread T3:
    #0 (1) suspend resume partial function for closure #1 in Example.transform(_:) <null>:2 (MyLibraryTests:x86_64+0x9f3c)
    #1 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Previous write of size 8 at 0x7b6000020438 by thread T1:
    #0 closure #1 in Example.transform(_:) <null>:2 (MyLibraryTests:x86_64+0x9e14)
    #1 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Location is heap block of size 1016 at 0x7b6000020400 allocated by thread T1:
    #0 malloc <null>:3 (libclang_rt.tsan_osx_dynamic.dylib:x86_64h+0x51d9a)
    #1 swift::StackAllocator<1000ul>::getSlabForAllocation(unsigned long) <null>:2 (libswift_Concurrency.dylib:x86_64+0x32769)
    #2 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Thread T3 (tid=1111954, running) is a GCD worker thread

  Thread T1 (tid=1111953, running) is a GCD worker thread

SUMMARY: ThreadSanitizer: data race (MyLibraryTests:x86_64+0x9f3c) in (1) suspend resume partial function for closure #1 in Example.transform(_:)+0x6c
==================
ThreadSanitizer report breakpoint hit. Use 'thread info -s' to get extended information about the report.
==================
WARNING: ThreadSanitizer: data race (pid=83691)
  Read of size 8 at 0x7b6000020460 by thread T3:
    #0 (1) suspend resume partial function for closure #1 in Example.transform(_:) <null>:2 (MyLibraryTests:x86_64+0xa03b)
    #1 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Previous write of size 8 at 0x7b6000020460 by thread T1:
    #0 closure #1 in Example.transform(_:) <null>:2 (MyLibraryTests:x86_64+0x9dae)
    #1 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Location is heap block of size 1016 at 0x7b6000020400 allocated by thread T1:
    #0 malloc <null>:3 (libclang_rt.tsan_osx_dynamic.dylib:x86_64h+0x51d9a)
    #1 swift::StackAllocator<1000ul>::getSlabForAllocation(unsigned long) <null>:2 (libswift_Concurrency.dylib:x86_64+0x32769)
    #2 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Thread T3 (tid=1111954, running) is a GCD worker thread

  Thread T1 (tid=1111953, running) is a GCD worker thread

SUMMARY: ThreadSanitizer: data race (MyLibraryTests:x86_64+0xa03b) in (1) suspend resume partial function for closure #1 in Example.transform(_:)+0x16b
==================
ThreadSanitizer report breakpoint hit. Use 'thread info -s' to get extended information about the report.
==================
WARNING: ThreadSanitizer: data race (pid=83691)
  Read of size 8 at 0x7b6000020450 by thread T3:
    #0 (1) suspend resume partial function for closure #1 in Example.transform(_:) <null>:2 (MyLibraryTests:x86_64+0xa050)
    #1 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Previous write of size 8 at 0x7b6000020450 by thread T1:
    #0 closure #1 in Example.transform(_:) <null>:2 (MyLibraryTests:x86_64+0x9dd8)
    #1 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Location is heap block of size 1016 at 0x7b6000020400 allocated by thread T1:
    #0 malloc <null>:3 (libclang_rt.tsan_osx_dynamic.dylib:x86_64h+0x51d9a)
    #1 swift::StackAllocator<1000ul>::getSlabForAllocation(unsigned long) <null>:2 (libswift_Concurrency.dylib:x86_64+0x32769)
    #2 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Thread T3 (tid=1111954, running) is a GCD worker thread

  Thread T1 (tid=1111953, running) is a GCD worker thread

SUMMARY: ThreadSanitizer: data race (MyLibraryTests:x86_64+0xa050) in (1) suspend resume partial function for closure #1 in Example.transform(_:)+0x180
==================
ThreadSanitizer report breakpoint hit. Use 'thread info -s' to get extended information about the report.
==================
WARNING: ThreadSanitizer: data race (pid=83691)
  Read of size 8 at 0x7b6000020468 by thread T3:
    #0 (2) await resume partial function for closure #1 in Example.transform(_:) <null>:2 (MyLibraryTests:x86_64+0xa180)
    #1 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Previous write of size 8 at 0x7b6000020468 by thread T1:
    #0 closure #1 in Example.transform(_:) <null>:2 (MyLibraryTests:x86_64+0x9e73)
    #1 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Location is heap block of size 1016 at 0x7b6000020400 allocated by thread T1:
    #0 malloc <null>:3 (libclang_rt.tsan_osx_dynamic.dylib:x86_64h+0x51d9a)
    #1 swift::StackAllocator<1000ul>::getSlabForAllocation(unsigned long) <null>:2 (libswift_Concurrency.dylib:x86_64+0x32769)
    #2 swift::runJobInEstablishedExecutorContext(swift::Job*) <null>:2 (libswift_Concurrency.dylib:x86_64+0x2d913)

  Thread T3 (tid=1111954, running) is a GCD worker thread

  Thread T1 (tid=1111953, running) is a GCD worker thread

SUMMARY: ThreadSanitizer: data race (MyLibraryTests:x86_64+0xa180) in (2) await resume partial function for closure #1 in Example.transform(_:)+0xe0
==================
ThreadSanitizer report breakpoint hit. Use 'thread info -s' to get extended information about the report.
Test Case '-[MyLibraryTests.MyLibraryTests testExample]' passed (4.684 seconds).
Test Suite 'MyLibraryTests' passed at 2021-12-15 10:26:24.594.
     Executed 1 test, with 0 failures (0 unexpected) in 4.684 (4.686) seconds
Test Suite 'MyLibraryTests.xctest' passed at 2021-12-15 10:26:24.595.
     Executed 1 test, with 0 failures (0 unexpected) in 4.684 (4.688) seconds
Test Suite 'All tests' passed at 2021-12-15 10:26:24.596.
     Executed 1 test, with 0 failures (0 unexpected) in 4.684 (4.692) seconds
ThreadSanitizer: reported 5 warnings
Program ended with exit code: 66

The generics should require Sendable from Input & Output. But both Int and String are Sendable so this does look like an issue in TSan.

cc @ktoso


fix error in totals computation with sparse data

Hello,
I found that when there are sparse data for some titles (i.e. only data for a few months are present in the input data), the totals for each month are not computed correctly. The first present value would be summed into the first month total even if the value was actually for a much later month.
I created a simple fix and also added a test for this case.
Cheers
Beda

Coverage increased (+0.03%) to 97.824% when pulling ed9b81ede3796b99ce5e6b3672c13860f704a71d on beda42:master into 08e0f42e6724e5b79efcf830ff31c941666ce7f8 on pitthsls:master.


Global network providers and abstract escrow.

Brilliant work, I like it a lot 👍

Thank you for your input, sorry it's a bit from two issues and some cleanup.


Updated 242-Valid-Anagram.java

Added solution which is more inline with video explanation. It's not ideal performance wise but understandable in one go.

duplicate


Slack community link is not working

Description
The slack community link is not working anywhere be it on any repository or any link on meshery website.
Expected Behavior
The link should work
Screenshots
here is a screenshot

Enviroment:

OS: Mac Linux Windows
Browser: Chrome Safari Firefox
Version:
Device: Desktop Mobile

Contributor Guides and Handbook
The meshery.io website uses Jekyll and GitHub Pages. Site content is found under the master branch.

📚 See the Contributing to Meshery.io Website section of the readme.md and other contributing instructions, too.
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Assign me this issue

"Assign this issue to me" is what I think that you mean. With a side of "please" would be a nice addition as well, @maushish.

Yeah my bad for being impolite let me rephrase my words -Could you please assign this issue to me @leecalcote


Fixed broken link in the pod file for Mongo::DataTypes.

HI, please review the above changes.
Many Thanks.
Best Regards,
Mohammad S Anwar

Thank you!  Merged to master.


Added Severity level to tracing

Don't know how to change and run the automated test. Tested manually.

@webbes,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@webbes, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Thanks for the contribution!


Add a way to limit max file size in FileDiagnosticsTelemetryModule

We are considering to leverage FileDiagnosticsTelemetryModule in an Azure App Service as a way to log to a file system in emergency cases when App Insights cannot be reached by any reason. But the FileDiagnosticsTelemetryModule doesn't provide a way to limit a max file size and allowing it to grow infinitely doesn't look safe. If this feature is really missing - can you consider it?

Yes this is a known limitation and will be considered.
@TimothyMothra is working on changes to this area.

I wonder if you have updates on the availability.

This is part of the newly released self-diagnostics capability
https://github.com/microsoft/ApplicationInsights-dotnet/tree/develop/troubleshooting/ETW#self-diagnostics


Add 'Elasticsearch mapping' page to docs

Description

This document explains the need for changes in #684.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

This documentation seems like a PR description (describing old implementation and proposed changes) and not like documentation. Would it be better to document things as they are in the code with their justification without going into the chronology of what was done before (there's Git history for that) and what should be changed later (this is the scope of GitHub issues)?

Other than the content changes I mentioned above, I don't have strong opinions on where the document lives. Still better to have code-related docs with the code it documents but yeah, there's always hyperlinks to bridge that gap.

Agreed, this is great documentation Olga! It would be useful to have this somewhere, but the language of it does make it seem like it's ephemeral (which might be more appropriate for a Make WP post or handbook page).
I don't have particularly strong feelings though. My only structural comment is - if we want this to show up in the docs pages, we'll need to add an index.md file to the explanations folder and add the explanations to the top level table of contents:
https://github.com/WordPress/openverse-api/blob/6cc0551c639081f895d974b30cc465713d0ce360/api/docs/index.md?plain=1#L10-L12

I'm always grateful when there are these kinds of "history of XYZ topic" documentation next to the code it describes. I don't care where it lives necessarily as long as it's easy to find from the code itself. A README.md in the same directory could be really helpful.
As for @dhruvkb's point about it being a PR description rather than a documentation, I think a documentation helps with visibility in a way that having to dig through git commit comments doesn't. It's fairly annoying to dig through git history of a file and try to reconstruct the narrative of something like this.

I think a documentation helps with visibility in a way that having to dig through git commit comments doesn't. It's fairly annoying to dig through git history of a file and try to reconstruct the narrative of something like this.

+1 for @sarayourfriend's point here. I've definitely encountered many issues where knowledge of the historical decisions is very helpful, and having to go sleuthing through old issues/PRs isn't desirable (especially when they span repos). With the links to PRs that Olga has added here I think it's contextualized well, so it doesn't feel more ephemeral to me than docs in general 😄
I like the current location but would be just as fine with placing it elsewhere -- but wherever we put it, I think we should probably link to it from the actual es_mapping code.

@obulat Is this depending on #684 to be ready? Needs rephrasing or something else? I'm trying to figure out what should we do here to move forward.

#684 was closed in favor of #850. @obulat are these docs still relevant?

@krysal @stacimc I think we can close this PR if @obulat agrees with these points:

It has gotten a bit out of sync with the code reality.
The description seems more suited for a post in P2 or Make.
I'll add documentation about the new mappings in PR #850 that'll live with the code and get bundled into the dev docs.

Thank you, @dhruvkb!


Opt-in to Marshal dump mocking

Monkey-patch Marshal.dump to enable dumping of mocked or stubbed objects. By default this will not work since RSpec mocks works by adding singleton methods that cannot be serialized. This patch removes these singleton methods before serialization.
Potentially but unlikely to be removed post upgrade.

@miq-bot add_label rspec3, test


Github to GitBook sync

I am unable to see new pages I have pushed to github on GitBook.
I’ve looked into a couple of possible reasons why, tried adding .gitbook.yaml and updating SUMMARY.md file per their troubleshooting tips, but haven't solved the issue yet.
Wondering if @frjo, @maxpearl, or @danblah have had any experience with this, or have any insight about how the sync can be achieved.

For reference, I'm accessing GitBook link to Hypha-Meta here.  I think that's still the correct address, but am happy to be correct if I'm wrong.

@emlini @maxpearl Can one of you give me access to the Hypha GitBook space?

GitBook documentation says,
"Webhooks notify GitBook when your GitHub repository changes.
If your GitHub changes are not available on GitBook, the main source of the problem is the webhook. You can check the state of your webhook in your repository's settings."

@emlini @maxpearl Can one of you give me access to the Hypha GitBook space?

I don't have the right level of access to add other users (I'm a Writer, not an Admin or Owner).  @maxpearl or maybe @danblah will have to do it.

I suspect the level of access is also why I can't make any changes regarding the github-GitBook integration (here's the doc page for that)

I don't have permissions for the group in gitbook. @danblah is going to have to give permissions, or update the webhook.

just bumped y'all up to admin and sent an invite for fredrik
On Wed, Jul 7, 2021 at 2:04 PM Max Pearl @.***> wrote:

I don't have permissions for the group in gitbook. @danblah
https://github.com/danblah is going to have to give permissions, or
update the webhook.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/HyphaApp/hypha-docs/issues/12#issuecomment-875815107,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEXOMDFGHTBMVAKQ57MRE3TWSJJVANCNFSM47XJUXRA
.

--
Dan Blah
pgp 0x36377134 | Signal/Wire/WhatsApp +1.202.873.5460

This should work again now.
I also updated the domain to https://docs.hypha.app/. And renamed the space to https://app.gitbook.com/@hypha/s/docs/

It works!  Takes a minute (and a couple refreshes) for the changes to appear on docs.hypha.app, but they're there!


Remove time.sleep in Speech system tests.

The Speech system tests contain defensive code to address a backend issue with the new v1 endpoint (it sometimes returns v1beta1 LRO URLs).
This is supposed to be fixed in the backend soon; remove the defensive code once it is.

This code is gone.


Inconsistent gene number in geneAbund.txt

I ran hisat2 v2.1.0 + stringtie v1.3.4d and found inconsistent gene number in geneAbund.txt by wc -l
58383 sample1.geneAbund.txt
58383 sample2.geneAbund.txt
58382 sample3.geneAbund.txt
58382 sample4.geneAbund.txt
58382 sample5.geneAbund.txt
58382 sample6.geneAbund.txt

And, duplicated genes existed in all six samples checked by grep -n ENSG00000275395.5. e.g., in sample2, sample5 and sample6:
36702:ENSG00000275395.5	FCGBP	chr19	-	39863323	39906323	0.000000	0.000000	0.000000
55608:ENSG00000275395.5	FCGBP	chr19	-	39913690	39934626	0.0	0.0	0.0

35772:ENSG00000275395.5	FCGBP	chr19	-	39863323	39906323	0.009627	0.014371	0.023534
55475:ENSG00000275395.5	FCGBP	chr19	-	39913690	39934626	0.0	0.0	0.0

35254:ENSG00000275395.5	FCGBP	chr19	-	39863323	39906323	0.000000	0.000000	0.000000
55262:ENSG00000275395.5	FCGBP	chr19	-	39913690	39934626	0.0	0.0	0.0

I used GENCODE v27, but the gene had only one entry.
chr19	HAVANA	gene	39863323	39934626	.	-	.	gene_id "ENSG00000275395.5"; gene_type "protein_coding"; gene_name "FCGBP"; level 2; tag "reference_genome_error"; tag "fragmented_locus"; havana_gene "OTTHUMG00000182580.5";

Also, It seemed that the output was unsort. However, a sorted output will be helpful,  e.g. for making matrix.

I believe I am encountering the same issue with this gene: ENSG00000240761. I believe that the issue is caused by the existence of multiple transcripts which do not overlap, causing Stringtie to assemble them as separate features in some samples (but not all).


markdown link to file highlight fails for paths with spaces

VSCode Version: 1.50.1-1
OS Version: Manjaro 20.1.1 Mikah

Steps to Reproduce:

open a new markdown file;
write something such as  [text](<path with spaces>) *;
notice how that is highlight differently compared to other links in the form [text](path_without_spaces).

(*angle brackets should allow for links with spaces)
Similar issue: https://github.com/Microsoft/vscode/issues/24763#issue-221783835
Except there the syntax was not complying with the CommonMark spec, while this should.
Again, links work in the PDF visualization, this is a text highlight issue, not a rendering one.

Does this issue occur when all extensions are disabled?: Yes

Duplicate of #96867 (see that for why it was won't fixed)

Or actually, this is a slightly different case. Moving to correct repo


⚠ Bofrost.be has degraded performance

In 1310558, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1472 ms

Resolved: Bofrost.be performance has improved in d0155c0 after 9 minutes.


Tried using wormhole to convert SRM tokens- arrived to Phantom wallet as "Unknown Tokens"

Hello! I would be so grateful if someone here could help me with an issue. I used Wormhole Bridge to convert some Serum (SRM) ERC-20 tokens in my MetaMask wallet into SPL tokens in my Phantom wallet. The Wormhole bridge conversion of the token from ERC to SPL transaction was successful, but when the coins arrived to my Phantom wallet, they showed up as an Unknown Token. The quantity of tokens was correct, but Phantom does not recognize the tokens as SRM.
Phantom provides the option to "edit the token metadata".. I wonder if doing this might help resolve the issue, and allow Phantom to recognize the token as SRM?
The Mint Address currently listed in the metadata is: 2jXy799YnEcRXneFo2GEAB6SDRsAa767HpWmktRr1DaP
The "Name" and "Symbol" fields are currently blank.
Here the tokens/ transaction is shown on Solana block explorer: https://explorer.solana.com/address/9MRwLJVfs3gtu3bvygMDUoFBmSNqC49vy3RdiTcT2T7b
https://explorer.solana.com/tx/2m566KL6x7PQZF16cFBC4JRfRUhbu2KMk5TWiJjrgzdLp7pbVeQnmQkuPTh1gPjuaCifiBbYd8V8sBCSWVEYQw6j

Any help would be greatly appreciated!

This seems like a Phantom UI issue, which doesn't recognize Wormhole-wrapped SRM (likely because it's not in https://github.com/solana-labs/token-list).
Does https://app.saber.so recognize your wSRM?

According to saber: https://explorer.solana.com/address/2jXy799YnEcRXneFo2GEAB6SDRsAa767HpWmktRr1DaP?cluster=mainnet-beta this is wSRM as leo mentioned.

Thank you very much for the quick responses! Yes you are right, it appears that Saber recognizes it as wSRM. Correct me if I'm wrong, but it seems like I can use saber now to swap my wSRM for SRM?