[BUG] AzureServiceTokenProvider.GetAccessTokenAsync no longer works with Visual Studio 16.3

Describe the bug
I work on a project that uses Azure Managed Identities. AzureServiceTokenProvider.GetAccessTokenAsync() is used to get a token, and to debug locally a token provided by Visual Studio is used. After upgrading to Visual Studio 16.3 this no longer works. I have verified this to be the case on two different computers after upgrading to Visual Studio 16.3.
I don't know if this is a problem with Visual Studio or the NuGet package but the exception is thrown by the NuGet package so I'm reporting it here.
Exception or Stack Trace

Parameters: Connection String: RunAs=Developer; DeveloperTool=VisualStudio, Resource: https://management.azure.com, Authority: . Exception Message: Tried to get token using Visual Studio. Access token could not be acquired.
Exception for Visual Studio token provider Microsoft.Asal.TokenService.exe : Token response is not in the expected format. Exception Message: There was an error deserializing the object of type Microsoft.Azure.Services.AppAuthentication.TokenResponse. Encountered unexpected character 'D'.

   at Microsoft.Azure.Services.AppAuthentication.AzureServiceTokenProvider.d__14.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at Microsoft.Azure.Services.AppAuthentication.AzureServiceTokenProvider.d__18.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1.ConfiguredTaskAwaiter.GetResult()
   at Microsoft.Azure.Services.AppAuthentication.AzureServiceTokenProvider.d__16.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at UserQuery.d__0.MoveNext()
To Reproduce
Use Visual Studio 16.3 and configure an Azure account in Tools > Options > Azure Service Authentication > Account Selection.
Create a new instance of AzureServiceTokenProvider with connection string "RunAs=Developer; DeveloperTool=VisualStudio" and then call GetAccessTokenAsync("https://management.azure.com") to get a token.
Code Snippet
var connectionString = "RunAs=Developer; DeveloperTool=VisualStudio";
var azureServiceTokenProvider = new AzureServiceTokenProvider(connectionString);
var token = await azureServiceTokenProvider.GetAccessTokenAsync("https://management.azure.com");

Expected behavior
GetAccessTokenAsync should return a token that can be used to authenticate with Azure based on the configured account in Visual Studio and not throw an exception.
Screenshots

Setup (please complete the following information):

OS: Windows 10 Pro 1903 18362.356
IDE: Visual Studio 16.3.0
Version of the Library used: Microsoft.Azure.Services.AppAuthentication 1.3.1

Additional context
None.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[X] Repro Steps Added
[X] Setup information Added

Hi @Liversage I've tried reproducing just now using a Functions app built with Visual Studio 2019 16.3.1. I was also using Microsoft.Azure.Services.AppAuthentication 1.3.1. I wasn't successful at recreating the issue. Given that the component that is raising the exception is ASAL, I have asked the visual studio team if there were any known issues or there are any logs that might help us dig deeper. I assume you're using 16.3.1?

Hi @loarabia, thanks for looking into this issue.
I was using 16.3.0 when I reported the issue. That was the version that was offered to me when I used the Visual Studio Installer to upgrade Visual Studio (I was using 16.2 before the upgrade). I reported the issue shortly after upgrading my second computer because I became certain that this problem was not isolated to a single computer.
However, since 16.3.1 is the latest version I have upgraded to this version on one of my computers to further investigate:

Test
Result

Snippet in LINQPad (AnyCPU edition using full framework)
Exception is thrown

My application (.NET Core App 2.2)
Exception is thrown

Switching configured account in Visual Studio
Exception is thrown

Having a single account configured in Visual Studio (I normally have two)
Exception is thrown

Snippet in .NET Core App 3.0 console app
Exception is not thrown

Snippet in .NET Core App 2.2 console app
Exception is not thrown

Snippet in full framework console app
Exception is not thrown

Summing up:

I'm unable to reproduce the issue except in my own project and in LINQPad.
I can resolve the issue I'm having with my own project and LINQPad by downgrading from Visual Studio 16.3 to 16.2.
I have the issue on two different computers.

So I'm pretty sure the issue is real but since I'm having problems reproducing it myself I can see that it's hard to investigate. Would you be able to test my snippet in LINQPad?

I had the same error. I opened up VS2017 and re-entered my credentials there. After that, it works.

Thanks to open source I was able to troubleshoot this issue further.
The Visual Studio access token is acquired by executing Microsoft.Asal.TokenService.exe. This is the command line on my computer:
"c:\program files (x86)\microsoft visual studio\2019\enterprise\common7\ide\extensions\qglq5p5b.btx\TokenService\Microsoft.Asal.TokenService.exe" --resource https://management.azure.com --serviceHubConfig "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\Common7\servicehub.config.json"

Executing this manually returns JSON containing the Azure token as expected. However, in some cases, when using the NuGet package, the Microsoft.Asal.TokenService.exe process writes the following to standard output:

Devhub Core CLR service module host. Copyright (c) Microsoft Corp
  Host id: desktopClr.x86$IdentityHost
  Controller pipe: net.pipe://C7EB239D4C5E775A2743A1938D70EA14
Running host until the hub controller sends 'exit' command...
{"access_token":"...",."expires_on":...}

The token is still in the output but for some reason there are four lines of text before. The first character of this text is D which explains the exception message:

There was an error deserializing the object of type Microsoft.Azure.Services.AppAuthentication.TokenResponse. Encountered unexpected character 'D'.

I don't know why the output of Microsoft.Asal.TokenService.exe sometimes contains extra text but it explains my issue.

@Liversage awesome that actually makes this much easier to dive into. Let me see if I can get someone on the ASAL side of things to weigh in next week.

@Liversage, I have been able to reproduce the error in linqPad using the snippet, I was also not able to reproduce the issue using .net core applications either launching the process directly or using the app.authentication nuget package.
However, I did find that older versions of the token service for example one from vs2017 work fine even in linqpad.
What I think is happening is that linqPad is doing some sort of redirection of the standard out handles in a way which is causing the tokenservice.exe and one of the child exe process it launches to now share the standard out steam.  This is then read by the app.authentication library and it fails because it sees output in the standard out stream.
You mentioned that your application also fails, does that application to any redirection of standard our or standard error?

We use Service Fabric which uses output redirection.

To follow up on the comments about console redirection and Service Fabric:
My system consists of multiple .NET Core App 2.2 console apps. When deployed they do execute in Service Fabric. However, this issue is about local testing and in that case my apps are configured to just execute as normal console apps without any redirection of standard out or error. I don't believe this execution path will ever call Service Fabric code but the DLLs are still present in the bin folder so this might be what triggers the problem. (I'm actually having a bit of a problem running my system in the local test cluster because it executes using a Windows service account which does not have access to my Azure credentials.)
It's my understanding that different versions of Visual Studio will each supply their own version of Microsoft.Asal.TokenService.exe linked to the Azure account selection option in that specific version of Visual Studio. The availability of a token service is recorded in %LOCALAPPDATA%\.IdentityService\AzureServiceAuth\tokenprovider.json where each version has a priority. The NuGet package will enumerate and try each version until it gets a token. My belief is that when using Visual Studio 2017 to set the Azure account it becomes the one with the highest priority which fixes the problem. Unfortunately, I don't think it's possible to install 16.2 together with 16.3. Otherwise using 16.2 for the Azure account selection would allow me to use 16.3 for daily work. (I'm not going to install 2017 to fix this issue.)

We have been able to reproduce the issue in both LINQPad and on the commandline directly by redirecting std out to a file and seeing the extra text be printed to std out.  We believe this issue is caused by how the identity modules are inheriting the handles during process launch.  An internal bug has been created to address this issue.   I do not know of any work arounds at the moment, other than the one you suggested about installing a different version of visual studio.

I just installed Visual Studio 2019 16.4.0 and as far as I can tell this problem has been fixed.

For me, the problem appeared after upgrading to Visual Studio 2019 16.4.0 from the previous version. I've found that this was caused by invalid TokenProvider path, so I was forced to change content of "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\Common7\servicehub.config.json" file:
{   "TokenProviders": [     {       "Path": "c:\\program files (x86)\\microsoft visual studio\\2019\\enterprise\\common7\\ide\\extensions\\1rzmhu0l.2xx\\TokenService\\Microsoft.Asal.TokenService.exe",       "Arguments": [         "--serviceHubConfig",         "\"C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Enterprise\\Common7\\servicehub.config.json\""       ],       "Preference": 0     },     {       "Path": "c:\\program files (x86)\\microsoft visual studio\\2019\\enterprise\\common7\\ide\\extensions\\qglq5p5b.btx\\TokenService\\Microsoft.Asal.TokenService.exe",       "Arguments": [         "--serviceHubConfig",         "\"C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Enterprise\\Common7\\servicehub.config.json\""       ],       "Preference": 1     },     {       "Path": "c:\\program files (x86)\\microsoft visual studio\\2019\\enterprise\\common7\\ide\\extensions\\4rin0bzc.yf5\\TokenService\\Microsoft.Asal.TokenService.exe",       "Arguments": [         "--serviceHubConfig",         "\"C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Enterprise\\Common7\\servicehub.config.json\""       ],       "Preference": 2     }   ] }
to (note a new extension folder - "ys2aolp3.2rs"):
{   "TokenProviders": [     {       "Path": "c:\\program files (x86)\\microsoft visual studio\\2019\\enterprise\\common7\\ide\\extensions\\ys2aolp3.2rs\\TokenService\\Microsoft.Asal.TokenService.exe",       "Arguments": [         "--serviceHubConfig",         "\"C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Enterprise\\Common7\\servicehub.config.json\""       ],       "Preference": 0     }   ] }
and then everything started to work again!

@vlardn The "good" (version 16.4) extension is in folder ys2aolp3.2rs. The "bad" (version 16.3) was in folder qglq5p5b.btx. Apparently, your service hub configuration was incorrect after the upgrade and your information is useful in case others have the same problem (and really is a new issue in itself). However, I still believe that Visual Studio 16.4 fixes the token provider problem that was introduced with Visual Studio 16.3.
I think I will leave it to the maintainers of this repository to close this issue, though.

I'm now seeing this with both Visual Studio 16.4.5 and az cli.
I've tried using both Microsoft.Azure.Services.AppAuthentication v1.0.3 (transitive from Microsoft.Extensions.Configuration.AzureKeyVault) and v1.2.1 explicitly.
Error message using both Visual Studio Services and az cli with same issue:
parameters: Connection String: [No connection string specified], Resource: https://vault.azure.net, Authority: https://login.windows.net/{tenantId}. Exception Message: Tried the following 4 methods to get an access token, but none of them worked.
Parameters: Connection String: [No connection string specified], Resource: https://vault.azure.net, Authority: https://login.windows.net/{tenantId}. Exception Message: Tried to get token using Managed Service Identity. Unable to connect to the Managed Service Identity (MSI) endpoint. Please check that you are running on an Azure resource that has MSI setup.
Parameters: Connection String: [No connection string specified], Resource: https://vault.azure.net, Authority: https://login.windows.net/{tenantId}. Exception Message: Tried to get token using Visual Studio. Access token could not be acquired.
Exception for Visual Studio token provider Microsoft.Asal.TokenService.exe : Token response is not in the expected format. Exception Message: There was an error deserializing the object of type Microsoft.Azure.Services.AppAuthentication.TokenResponse. Encountered unexpected character 'â'.
Parameters: Connection String: [No connection string specified], Resource: https://vault.azure.net, Authority: https://login.windows.net/{tenantId}. Exception Message: Tried to get token using Azure CLI. Access token could not be acquired. Token response is not in the expected format. Exception Message: There was an error deserializing the object of type Microsoft.Azure.Services.AppAuthentication.TokenResponse. Encountered unexpected character 'à'.
Parameters: Connection String: [No connection string specified], Resource: https://vault.azure.net, Authority: https://login.windows.net/{tenantId}. Exception Message: Tried to get token using Active Directory Integrated Authentication. Access token could not be acquired. password_required_for_managed_user: Password is required for managed user

I'm also getting this when I set my Visual Studio Services authentication to a different account:

Again the same issue when upgrading from VS 2019 16.5.5 to 16.6.0 :(
And again I need to manually fix the new extension folder name in tokenprovider.json.
Microsoft guys! When you will fix this annoying issue occurring every second VS update??

@AlexGhiondea:  Would you be so kind as to help route to the right individual?

What is likely happening is that the tokenprovider.json file is not being correctly updated after an upgrade.  The file is updated when you go to Tools->options->Azure Services Authentication and click OK. Or load a project which has a dependency on the App.Authentication nuget package.  This is supposed to run a target which causes the json file to be uploaded.
This was done because we only wanted to update the file if the feature was being used.  However, it seems that this target is not always running which leaves the file out of date and as for the tools options route that would normally be done once and it not obvious that you need to go back there and click it again (and you should not really have to either).
Because of this we are changing how we write this file such that it is always updated without having to go to the dialog and click ok or have a certain dependency in the project.

@crmann1 Is there a bug / issue we can link to for the new behavior in updating tokenprovider.json?

@crmann1 Has this been fixed in recent VS updates?

@jsquire, this is not a mgmt plane related issue. I am removing the incorrect label. Thanks.

@jsquire, this is not a mgmt plane related issue. I am removing the incorrect label. Thanks.


Competition Edit, phase dates duplicates between phases

If you try to change the start and end dates on multiple phases without reloading the page, it will duplicate the dates to all phases you try to edit.

Fixed in PR #506.


TestPlan: PlotViewer on the web

Testing the PlotViewer on the web.
Refs: https://github.com/microsoft/vscode-jupyter/issues/10405

[ ] web @tanhakabir

Complexity: 3
Authors: @sadasant
Create Issue
Create Issue

Requirements

Install prerelease jupyter extension into http://insiders.vscode.dev
Have a python environment that you can run jupyter from. (https://jupyter.org/install)
Be familiar with jupyter notebooks
(Good to have) Familiarity with https://matplotlib.org

Set up

Go to a command prompt where you've installed jupyter into a python environment
Install matplotlib.

For Conda: https://anaconda.org/conda-forge/matplotlib
For pip: https://matplotlib.org/stable/users/installing/index.html

Switch to a folder you're using for testing
Run jupyter with the following command jupyter notebook --no-browser --NotebookApp.allow_origin=*

Test steps

Open insiders.vscode.dev and create a new .ipynb file.
Connet to the jupyter server:

Run the 'Jupyter: Specify jupyter server for connections' command.
Pick 'Existing'.
Paste the URL from the jupyter notebook server you started in step 3 of the setup.

In one of the Notebook cells, create a plot. You can use this code:

import matplotlib.pyplot as plt
plt.plot([1,2], [1,2])

Run the cell in any way. For example: by clicking the ▶ button at the left of the cell.
It might ask which kernel to use. Pick the remote kernel.
Hover over the plot. You should see three buttons at the top-right of the plot.
The first button from left to right should allow you to copy the plot, and paste it elsewhere.
The second button should expand the plot in a separate tool (this is the Plot Viewer).
The third issue should allow the plot to be saved.

https://user-images.githubusercontent.com/417016/180882962-4dce66bd-0521-4a3c-83b4-17ed32b91a2b.mov
Known issues (may o may not happen to you)

When saving the plot in the web: If you use the button that appears when you hover the plot, the file will be saved as .ipynb, however it will contain a python file (with the source code of the plot). In the desktop version however, you can download the .png version of the plot. #10921
The plot viewer itself (the one that is accessed via the second hovered button) has a save button that fails to save the file in the web, citing that local folders are not supported #10922

The basics seem to work well!


Maximum length of Command parameters in Windows 8 scheduler

I have a console application that takes input command line arguments. I have to schedule this task on Windows 8 using Win32.TaskScheduler or directly from Windows 8 Task Scheduler . The problem is whenever my command line arguments length exceeds 450 characters, i get a warning like this :

Task registered task "TaskName" , but not all specified triggers will start the task. User Action: Ensure all the task triggers are valid as configured. Additional Data: Error Value:<PHONE_NUMBER>.

And the task does not run eventually at the run time. Is there any limit on command line arguments' length?
But the same thing works on windows 7 scheduler.
Originally posted: 2014-02-17T02:15:03

There is nothing in the Microsoft documentation on command line argument length. There is certainly nothing in the wrapper library code that would limit the length. I would suggest posting this in one of the Microsoft Forums to see if someone at MS can answer it.
Originally posted: 2014-04-17T17:31:41


[PORT]fix the issue while multiline content starts with "#"

Original issue: https://github.com/microsoft/botbuilder-dotnet/issues/4485
Originally, multi-line supports such format:
# template
- ``` abc ```

OR
# template
- ```
 asdasdas
```

OR
# template
- ```asdasdas
```

OR
# template
- ```
asdasdas```

But the below is not supported:
# template
- ```asdasdas
## this is not a template
```

The grammar would trigger the "#" in multi-line as a new template name mark.
This PR make this format work.

Pull Request Test Coverage Report for Build 161592

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

Files with Coverage Reduction
New Missed Lines
%

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

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

Totals

Change from base Build 160401:
0.01%

Covered Lines:
15159

Relevant Lines:
17391

💛  - Coveralls


🐛  Scanned value on QR code returns null on Android, works in iOS

What's happening?
First of all I admire the work and support of the creator as well as the community.
I am working on the process of digitizing the social security number in the institution where I currently work. Initially I tested on iOS and everything worked perfectly.
When testing on Android I realized that the scanned QR always returns null.
Initially I thought it was an implementation or configuration error of mine. But testing with online QR code decoders I realized that several work and others do not.
The format of the scanned QR is as follows:
||Número de Seguridad Social:<PHONE_NUMBER>3 |Nombre: NAME OF THE PERSON |CURP: XXXXXXX8408343||
Package.json settings :
"react-native-vision-camera": "^3.6.4"  "react-native": "^0.71.6",
BuildScript ext
buildToolsVersion = "33.0.0"  minSdkVersion = 26  compileSdkVersion = 33  targetSdkVersion = 33
BuildScript dependencies
 classpath("com.android.tools.build:gradle:7.3.1")
AndroidManifiest:
  <meta-data       android:name="com.google.mlkit.vision.DEPENDENCIES"       android:value="barcode" />
Reproduceable Code
import * as React from 'react';
import { Alert, Image, StyleSheet, View } from 'react-native';
import { useIsFocused } from '@react-navigation/native';
import { NoCameraPermission } from 'src/components/pages';
import { GoBack } from 'src/components/atoms';
import { useBoolean, useCustomNavigation } from 'src/hooks';
import { Camera, useCameraDevice, useCodeScanner } from 'react-native-vision-camera';
import customStyles from 'src/styles';

const QRScanner: React.FC = (): JSX.Element => {
  const { value, toggle } = useBoolean();
  const device = useCameraDevice('back', {
    physicalDevices: ['wide-angle-camera']
  });
  const isFocused = useIsFocused();

  React.useLayoutEffect(() => {
    (async () => {
      const status = await Camera.requestCameraPermission();
      toggle(status === 'granted');
    })();
  }, [value]);

  const codeScanner = useCodeScanner({
    codeTypes: ['qr'],
    onCodeScanned: codes => {
      Alert.alert(codes.length > 0 ? "Has data" : "No data", `Type ${codes[0].type} and value ${codes[0].value}`);
    },
  });

  return device != null && value ? (
    <View style={[customStyles.safeAreaContainer, { paddingTop: insets.top }]}>
      <View style={styles.navigationContainer}>
        <GoBack route="Home" screen="User" />
      </View>
      <View style={[styles.backgroundContainer, customStyles.center]}>
        <Image
          source={require('src/images/qrScan.png')}
          style={styles.backgroundImage}
        />
      </View>
      <View>
        <Camera
          style={styles.camera}
          device={device}
          isActive={isFocused}
          codeScanner={codeScanner}
        />
      </View>
    </View>
  ) : (
    <NoCameraPermission />
  );

};

export default QRScanner;

const styles = StyleSheet.create({
 ...
});

Relevant log output
onCodeScanned value scanned return null only on Android. Works perfectly on iOS

Camera Device
Any camera device

Device
Samsung S21 with Android 12
VisionCamera Version
3.6.4
Can you reproduce this issue in the VisionCamera Example app?
Yes, I can reproduce the same issue in the Example app here
Additional information

[ ] I am using Expo
[ ] I have enabled Frame Processors (react-native-worklets-core)
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

Hello I'm experiencing the same
Error : unknown/unknown: [unknown/unknown] failed to create thin barcode scanner.
Device : S21+

Hello I'm experiencing the same Error : unknown/unknown: [unknown/unknown] failed to create thin barcode scanner. Device : S21+

i solved the problem by updating google play services

Hello I'm experiencing the same Error : unknown/unknown: [unknown/unknown] failed to create thin barcode scanner. Device : S21+

i resolved the issue by updating Google Play Services

Had the same problem. Turns out if the code is not in UTF-8 you'll get null as response using Android: https://developers.google.com/android/reference/com/google/mlkit/vision/barcode/common/Barcode#public-string-getrawvalue. Funny thing is my code is in UTF-8 (with special characters) and its still not working 😓
The displayValue on the other hand is working (at least in the scenarios I tested/care-about).
You can apply a patch to CameraView+Events.kt to make it work. Just replace
code.putString("value", barcode.rawValue)

with
code.putString("value", barcode.rawValue ?: barcode.displayValue ?: null)

or any other Shenanigans you want to to do with it. But that might produce unforeseen consequences...

@GolexPrime Hey man, you did it! I applied the patch in CameraView+Events.tk and it worked. I think this is either an MLKIT bug or there is something I may be overlooking.

Thank you very much.
It worked well too! However, only a small portion of the Japanese was garbled, so it looks like it needs to be fixed.
I would be very grateful if you could just tell me what needs to be fixed.

However, only a small portion of the Japanese was garbled, so it looks like it needs to be fixed. I would be very grateful if you could just tell me what needs to be fixed.

Thats the direct output provided by MLKIT - it doesn't seem to like anything fancy in its scanned content 😓

I was able to successfully avoid the garbled characters using the code below.
import java.nio.charset.Charset

val string = rawqrcode?.let { String(it, Charset.forName("Shift_JIS")) }

Had the same problem. Turns out if the code is not in UTF-8 you'll get null as response using Android: https://developers.google.com/android/reference/com/google/mlkit/vision/barcode/common/Barcode#public-string-getrawvalue. Funny thing is my code is in UTF-8 (with special characters) and its still not working 😓
The displayValue on the other hand is working (at least in the scenarios I tested/care-about).
You can apply a patch to CameraView+Events.kt to make it work. Just replace
code.putString("value", barcode.rawValue)

with
code.putString("value", barcode.rawValue ?: barcode.displayValue ?: null)

or any other Shenanigans you want to to do with it. But that might produce unforeseen consequences...

Thank you

Worked for me as well! Thank you.
@mrousavy, would you consider making this fallback to displayValue the default behavior of the library on Android or returning both raw and display values?
I had trouble scanning one particular QR code and thought I would need to use another barcode reader instead of the ML Kit that is used by default. There is a community plugin for the frame processor using the Dynamsoft reader. I tried their demo, and I could scan the QR code. ML Kit can obviously scan it too, but there was no way for me to know until I found this issue, and that took me a while.
Thanks.


Suggestion: TypeScript support instead?

I see that using ECMAScript's specification is exceedingly difficult. Would it sound more reasonable to use Rapidus for a strictly-typed version of TypeScript? Much like how strictly-typed TypeScript code can be compiled into a WASM module (except the compilation part).

@undefinedbuddy if you want compiled into a WASM module, why not use Rust ? :D

The point of the suggestion wasn't to imply that I needed to compile TypeScript to WASM. Rather, I think it would be more meaningful if rapidus works with TypeScript (strictly checked preferred) than ECMAScript's standards.
Having a standard library based off dynamic libraries in Rust with a single build script to trigger cross-compilation of the core and built-in libraries would be sweet as well.

TypeScript support is a great idea. I'll do it if I feel like it someday.

Great. Looking forward to seeing that implemented!


🆙 Update XCM coefficients for DEV env

This PR was generated automatically 🤖

Tested by:

Moonriver KSM -> Basilisk
Kusama KSM -> Basilisk

Moonriver XRT -> Basilisk
Robonomix XRT -> Basilisk

Statemine USDT -> Basilisk

Moonriver KSM -> Kusama
Mangata KSM -> Kusama


Installing via Package Manager causes compile error from several targets in the library.

Expected behavior
Library is correctly imported into iOS project
Actual behavior
Packet manager prompts what targets to include. Instructions / readme doesen't say which to include. Selecting all wil cause several targets in the Library to fail.
Steps to reproduce

Install via Packet Manager in Xcode 13
Select all targets  to include when prompted
Try to compile your iOS app

If possible, minimal yet complete reproducer code (or URL to code)

RosSwift version/commit hash
Nevest release
Swift & OS version (output of swift --version && uname -a)
Swift 5 + Xcode 13
Notes
I also don't understand why listener, publisher, msgbuilder and msgs is puclic target to include.
Is something wrong in the library configuration?

It should be sufficient to only include the RosSwift library from the Package

Removing the targets in question made it work allthough it's not nescessary to include other than RosSwift as stated above.


Submit color and says not right

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

How do I send a screenshot?

Please reach out to the https://forum.freeCodeCamp.com


init launcher start shortcut with default if it does not exsist in th…

Summary of the Pull Request
What is this about:
If the PT Run settings do not exist at the start of PT most probably PT Run will not open on Alt+Space. It is a fix for it.
What is include in the PR:

Check if the settings contain the shortcut info and if not initialize m_hotkey with default one.
Additional logs

How does someone test / validate:

Delete 'PowerToys Run\settings.json'
Start PT run
Check that PT run starts on Alt+Space

Quality Checklist

[X] Linked issue: #7847 #9018
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[X] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

One open question is how to eliminate duplication of default shortcut('Alt+Space') in both projects 'Microsoft.Launcher' and 'PowerLauncher'.

One open question is how to eliminate duplication of default shortcut('Alt+Space') in both projects 'Microsoft.Launcher' and 'PowerLauncher'.


Add stdio-based CLI

Create some recipes and allow STDIO based piping for content through our CLI

Fixed in #23


Our recommended way to write testbenches is racy

We currently nudge people towards writing testbenches where the testbench modifies signals (inputs, registers) on the same edge as the edge-triggered synchronous logic (i.e. posedge for default posedge domains), but this is wrong:

testbenches written like this are susceptible to races, especially if yield Settle() is used;
VCD files produced by testbenches written like this are hard to read because changes to inputs (and, rarely, registers) made by the bench, as well as any combinatorial logic triggered by those changes, appear at the same instant of time in the waveform viewer as the changes caused by the posedge.

Migen did not suffer from the problem (1) because it had a far simpler simulator that always ordered synchronous logic after the testbench logic and had no way to advance the simulation by zero time. (However, this caused severe issues for testbench logic by making it impossible to observe combinatorial feedback; e.g. it is not possible to implement a 1-cycle FIFOInterface.read() in Migen).
Migen did suffer from the problem (2).

See also #228.

What I think we should do is to recommend using the inactive edge of a clock domain for testbench logic. This is actually already the recommended way to use CXXRTL, for the same reason. This implies eliminating or reworking Tick as it currently exists.

There is a partial solution to issue (1) in a branch. We discussed issue (2) on IRC and it is possible that nothing needs to be done there because if the DUT was embedded in a normal module, the waveform dump would have the same problem as with the recommended way of writing testbenches.

This issue is resolved by the combination of #1232 (which allows visualizing delta cycles in Python simulations) and #1213 (which is a meta-issue that thoroughly improves our simulation semantics).


Limit number of simultaneous drags

What could be improved
Specifically looking at DragCallbacks and the ability to have it so that only 1 (or n) draggable component(s) can be dragged at one time. Think of a chess board and only being allowed to move 1 piece rather than multiple at the same time.
Why should this be improved
This is currently possible to implement by manually keeping track of the current component being dragged and then returning from void onDragUpdate(DragUpdateEvent event). However, I imagine this is something that is implemented quite a lot and so would be nice for flame to handle it internally, making everything simpler.
Any risks?
None that I can forsee. Defaults should be provided to keep the behaviour as it is now and have the feature as "opt-in".
More information
My original question on SO

I think we should create a new dispatcher and new mixin for this, because single drags could be combined with scale events too, which isn't possible in a nice way with the multi-drag dispatcher.


How to generate single class based feature

I have similar network and I want to generate single class based feature as shown in readme.md
But in document using deepdream you are only visualizing layer not one class?
So how we extract one class based feature?
So please guide us for our research ....🙏

Hello @robonetphy
You can specify layer and the class index for which you want to generate deepdream visualization.
is_success = deepdream_visualization(sess_graph_path = tf.get_default_graph(), value_feed_dict = {t_input : im},                                       layer=layer, classes = [1, 2, 3, 4, 5],                                      path_logdir=os.path.join("Log","Inception5"),                                       path_outdir=os.path.join("Output","Inception5"))
Above example will generate 5 deepdream visualizations each for different classes(1, 2, 3, 4, 5). And 'layer' should be name of the last layer/last operation.
Example Notebook: https://github.com/InFoCusp/tf_cnnvis/blob/master/examples/tf_cnnvis_Example3.ipynb

Thank you very much.this will help a lot to achieve our goal.


[Feature] Add analytics tracking to FaucetButton component

#3977  👈
main

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

Join @jnsdls and the rest of your teammates on  Graphite


wireguard 脚本在CentOS8 下编译和repository两种方法均失败

[Sat Dec  7 09:45:07 CST 2019] systemctl start wg-quick@wg0
Job for<EMAIL_ADDRESS>failed because the control process exited with error code.
See "systemctl status<EMAIL_ADDRESS>and "journalctl -xe" for details.
[Sat Dec  7 09:45:07 CST 2019] Execution command (systemctl start wg-quick@wg0) failed, please check it and try again.

[root@bogon ~]# journalctl -xe
Dec 07 09:44:29 bogon systemd[1]: Starting man-db-cache-update.service...
-- Subject: Unit man-db-cache-update.service has begun start-up
-- Defined-By: systemd
-- Support: https://access.redhat.com/support
--
-- Unit man-db-cache-update.service has begun starting up.
Dec 07 09:44:32 bogon systemd[1]: Started man-db-cache-update.service.
-- Subject: Unit man-db-cache-update.service has finished start-up
-- Defined-By: systemd
-- Support: https://access.redhat.com/support
--
-- Unit man-db-cache-update.service has finished starting up.
--
-- The start-up result is RESULT.
Dec 07 09:45:07 bogon systemd[1]: Reloading.
Dec 07 09:45:07 bogon systemd[1]: Starting WireGuard via wg-quick(8) for wg0...
-- Subject: Unit<EMAIL_ADDRESS>has begun start-up
-- Defined-By: systemd
-- Support: https://access.redhat.com/support
--
-- Unit<EMAIL_ADDRESS>has begun starting up.
Dec 07 09:45:07 bogon wg-quick[25551]: [#] ip link add wg0 type wireguard
Dec 07 09:45:07 bogon wg-quick[25551]: RTNETLINK answers: Operation not supported
Dec 07 09:45:07 bogon wg-quick[25551]: Unable to access interface: Protocol not supported
Dec 07 09:45:07 bogon wg-quick[25551]: [#] ip link delete dev wg0
Dec 07 09:45:07 bogon wg-quick[25551]: Cannot find device "wg0"
Dec 07 09:45:07 bogon systemd[1]<EMAIL_ADDRESS>Main process exited, code=exited, status=1/FAILURE
Dec 07 09:45:07 bogon systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.
Dec 07 09:45:07 bogon systemd[1]: Failed to start WireGuard via wg-quick(8) for wg0.
-- Subject: Unit<EMAIL_ADDRESS>has failed
-- Defined-By: systemd
-- Support: https://access.redhat.com/support
--
-- Unit<EMAIL_ADDRESS>has failed.
--

我用官网 的方法安装失败，后来就想用大佬的脚本测试，一看果然也是挂的。CentOS8自带内核感觉跑不动。

你需要将自带内核升级到最新版，然后重启一下，然后再用脚本来安装。


Module 'IzzyParser' Not Found After Migrating from SPM to CocoaPods

Description:
I recently migrated my project from using Swift Package Manager (SPM) to CocoaPods in order to integrate GoogleMLKit, which required CocoaPods. However, after migrating and installing IzzyParser via CocoaPods, I am encountering the error module 'IzzyParser' not found, even though IzzyParser appears to be correctly installed in the Pods directory.
Environment:

Xcode Version: 15.2
IzzyParser Version: 1.0.0
Platform: iOS 12.0

Steps to Reproduce:

Initially integrated IzzyParser using Swift Package Manager.
Migrated the project to use CocoaPods for dependency management.
Added IzzyParser to the Podfile and ran pod install.
Cleaned the build folder and deleted derived data.
Attempted to build the project.

Podfile Configuration:
platform` :ios, '12.0'

target 'IzoAuth' do
  use_frameworks!
  # Pods for IzoAuth
end

target 'IzoCore' do
  use_frameworks!
  # Pods for IzoCore
end

target 'IzoElektro' do
  use_frameworks!
  pod 'GoogleMLKit/BarcodeScanning'
  pod 'Firebase/Messaging'
  pod 'DGCharts'
  pod 'Kingfisher', '~> 7.0'
  pod 'iOSDFULibrary'
  pod 'iOSMcuManagerLibrary'
  pod 'GoogleToolboxForMac'
end

target 'IzoNetworking' do
  use_frameworks!
  pod 'IzzyParser'
end

target 'Lumber' do
  use_frameworks!
  # Pods for Lumber
end

post_install do |installer|
  installer.pods_project.targets.each do |target|
    target.build_configurations.each do |config|
      config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '12.0'
    end
  end
end

Troubleshooting Steps Taken:

Verified that IzzyParser is listed under the Pods project in Xcode.
Ensured that use_frameworks! is enabled for all targets in the Podfile.
Updated Runpath Search Paths to include $(inherited), @executable_path/Frameworks, and @loader_path/Frameworks.
Cleaned the build folder and deleted derived data.
Reinstalled pods using pod deintegrate and pod install.

Current Issue:
Despite following these steps, the error persists, indicating that Xcode is unable to locate the IzzyParser module.
Additional Information:
Screenshots and relevant code snippets are attached for further context.

Request:
I would appreciate any guidance or suggestions to resolve this issue. If additional configuration or steps are required to correctly integrate IzzyParser via CocoaPods, please provide detailed instructions.
Thank you for your assistance.

@KristjanIzo
Hi!
Thank you for using IzzyParser in your project. From the configuration steps you have provided I can see you have done everything correctly, yet I can also see the problem. Unfortunately IzzyParser is only available in iOS 14 or higher as it is stated in the Podfile of the example project provided in the repo.
platform :ios, '14.0'
If you can please upgrade your project to minimum iOS 14 and IzzyParser will work as expected.
Thank you!


Migrate tests to Node.js test runner

Replace jest with the Node.js test runner (and tsmatchers for assertions if needed).

Fixed via 58a1d8b2dd75e8fb972e9075ed2c0058dfa16588


fix redoing visual mode operations with .

Fixes #737.
Differs from VIM in that if the visual mode selection was characterwise over multiple lines, and the last line of the selection was longer than the corresponding line where we are redoing the operation, VIM would remove the newline and this PR doesn't. See #737 for more information on this.
Any comments will be welcome.

Very very impressive.  I've banged on it, can't find anything wrong.  As promised, it's in vim-mode-next 0.54.12.
Vim's charwise multi-line behavior feels like a bug.  I definitely vote for your current behavior.

I've been trying this in vim-mode-next (currently 0.54.15) and repeating commands in line wise visual mode doesn't seem to work correctly. For example, take this file:
foo
bar

Now select both lines with Vj and indent with >
  foo
  bar

Now when I try to repeat the command with ., only the first line gets indented again.
    foo
  bar

Wow.  I'm using 0.54.15 too and I don't see that.  Both lines indent every time I hit ..
Silly question, you sure vim-mode didn't re-enable itself somehow?

I tried it again with a clean install on a different computer and it still behaves as I described.

I've just reproduced it - the problem shows up as described on a file with only two lines. Try this in a new file:
ifoo<cr>bar<esc>kVj>.
Strangely enough, it works with v instead of V, and it also works with an extra <cr> after bar and an extra k before V. I'll have to look into it. But I hope a fix for this can be another PR if this one can otherwise be merged - it seems to work well enough in situations where I've tried it. If this PR is merged, an issue for the strange behaviour should be opened.

Nice!!  Yes, the missing newline on the last line must be the culprit.  I'm seeing it too.
I've noticed that files that don't end in newlines trip up some other Vim commands too.

:sparkles:


Receive signal

User story

As a software developer
I want / need the signals sent from generator be received at the front end
So that I will be able to use it for visualization.

Acceptance criteria

[ ] Successfully receiving data sent from generator.

Definition of done (DoD)
Not available yet
DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

Started working with @PhlppKrmr

Goal: Use UDP for data transfer between back- & frontend (frontend is a plain browser application)
Issue: There is no way of creating a plain UDP socket directly within the browser. It does not support browser required security concerns.
Solutions:

Switch to websockets over TCP
Switch to WebRTC protocol
Create a frontend server (like Node.JS) which handles the back-&frontend communication


heart rate and RR interval

A few levels under abnormal cardiovascular system physiology, there are terms for increased and decreased heart rate (MP:0002626 and MP:0005333) and separate terms for prolonged and shortened RR interval (MP:0010506 and MP:0010507). The term definitions state that RR interval is the inverse of the heart rate, so these two sets of terms describe the same underlying phenotype. However, as far as I can tell, the equivalence is not recorded formally in the ontology. Would it be possible to join these terms?
One possibility would be to turn two of these terms obsolete (thus reduce the overall number of terms) and then adjust the other two terms and their parents accordingly.

I have made shortened and prolonged RR interval children of increased and decreased heart rate, respectively. I don't think we want to merge the terms since you can measure heart rate without using an ECG but I agree that these should be related.


Update LCA.py

I think gumbel_softmax is not performing up to the mark on some datasets, I re evaluated it

Please give me some details about performance on gumbel_softmax

Please give me some details about performance on gumbel_softmax

not performing up to the mark on Sony total dark, SICE

How much have the PSNR and SSIM metrics dropped, respectively?

How much have the PSNR and SSIM metrics dropped, respectively?

something less then 22

What about the LOL related datasets?


⚠ Bofrost.ch has degraded performance

In 24c6b38, Bofrost.ch (https://www.bofrost.ch) experienced degraded performance:

HTTP code: 200
Response time: 2087 ms

Resolved: Bofrost.ch performance has improved in fb7f0a0.


Ability to differentiate null and empty values in CSV table

From getgauge google groups forum.

As discussed in the google groups thread, csv specification doesn't have any concept of null value. Hence Gauge returns an empty string.
For more detailed discussion, please refer the thread mentioned above.


refactor(note): move fifths distance functions to class methods

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 18 (72.22%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+21.2%) to 62.153%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/src/tonality/tonality.dart
0
5
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
21.2%

Covered Lines:
179

Relevant Lines:
288

💛  - Coveralls


fmt(cstyle): cstyle fixes in src/mock_errored_replica.c

Fixes openebs/openebs#2021

@payes I tried compiling it, but I am on mac and the steps given in README.md is not working in mac.
Is there some separate docs for macOS


Website needs to be opened - mentionbb.com

Friends, I am not willing to leave such a beautiful project in the middle :) How about a small update every day? At the very least, I think it is enough to make it clear that the project is being taken care of.
At least we are not updating, at least let's open the forum :)

Thank you.
Actually, the project was not abandoned. I am making small updates but I haven't been able to do much with it due to my work these days.
I will activate the website as soon as possible.

If you want, let's do it like this, when you open the site, let's test it in free time for review and development, let's take note of the deficiencies and errors, let's go step by step and update it.
As a tester, I have a lot of review time in my free time, but how much time do you have to update 😁

If something is broken or needs to be updated, it needs to be updated immediately, so that time can be created. I'll post it here when I open the site.
Until then, you can pull the project to local. The project actually had a Doctrine repo, but it's older than mine. So I need to open that too. :D

It seems that you did not discard the project but abandoned it 😁😁😁
Please can I ask you to take over the project as soon as possible so that we can continue this work more seriously and with greater dedication 😁😁

No no, the project has not been abandoned. I try to make updates occasionally.
I wrote Doctrine above, it was to be Docker. This is proof that I don't have time. 😁

If you are going to use it with Docker, I upload repo.
https://github.com/mentionbb/mentionbb-dockerized

Eğer Docker ile kullanacaksanız, repo yükler.
https://github.com/mentionbb/mentionbb-dockerized

In fact, it should be a very simple installler for the end user, not for us, we are aware of this, but I think this is one of the most important ones, isn't it?
Because unzip it from the zip, enter the sql information, create admin and log in to the system, I think the first step is to create such a simplified installation package :)
I will examine it a little personally, but as I said, this should be one of the most important ones, I think we can say that it should be done first.

Eğer Docker ile kullanacaksanız, repo yükler.
https://github.com/mentionbb/mentionbb-dockerized

In fact, it should be a very simple installler for the end user, not for us, we are aware of this, but I think this is one of the most important ones, isn't it?
Because unzip it from the zip, enter the sql information, create admin and log in to the system, I think the first step is to create such a simplified installation package :)
I will examine it a little personally, but as I said, this should be one of the most important ones, I think we can say that it should be done first.

This is true. In fact, the project has an ongoing Install interface. However, sometimes the core files in the project can change and therefore the Install to be done at this stage can cause me more trouble.

Yes, you may be right about this, but if we look at things from the point of view of the end user, not from our point of view, it offers a very negative pre impression to the user.

Actually, everything is ready for this. Table layouts etc. just need to be directed to the install wizard correctly. I will update this issue as I find time.
Theoretically, it should be ready by the end of the year.

For the forum site's theme, it would be great if the design prioritizes user convenience, while also ensuring that moderators and administrators can easily manage tasks within the admin panel. These are just my suggestions; I don’t have much expertise in PHP, Docker, or similar technologies. Ensuring compatibility with existing sites is crucial for the project's continuity. Having an install page that automatically sets up the data would also be beneficial.

For the forum site's theme, it would be great if the design prioritizes user convenience, while also ensuring that moderators and administrators can easily manage tasks within the admin panel. These are just my suggestions; I don’t have much expertise in PHP, Docker, or similar technologies. Ensuring compatibility with existing sites is crucial for the project's continuity. Having an install page that automatically sets up the data would also be beneficial.

I would like to state that I have a more modern approach in terms of design. I have moved away from the classic, vBulletin, XenForo sketch and have been inspired by more modern projects such as Discourse and Flarum.
Actually, I need to focus on Install but I don't have much time, but that doesn't mean it will never be done. There are a lot of negative comments about this and they are right. I will increase my work on the Install interface.

I will set up the site according to the instructions and take a look at it. Is there additional language support?

I will set up the site according to the instructions and take a look at it. Is there additional language support?

Currently Turkish and English support only.

Thank you, I will provide my comments on this topic from a user perspective after a detailed review over time.

Frankly, the feedback is as follows. The database config file is very complex and cannot be set up properly; I haven’t been able to find where I am making a mistake, but I am continuing to try to install it. If there is an explanation video, etc., it would be very helpful. (This is regarding cPanel installation)

Dear friends, please let's not use the topic in a brutal way, I opened this topic, everyone should go to their own topic :))))
Just kidding, now let me give additional information for the part that the friend said, I did a rough installation yesterday, but it is necessary to edit everything manually, so I tried a little bit and got bored and left it :)
To be honest, it seems that there should be an installation package for a while, at least for those at the basic level.
I got stuck on the Nginx configuration part, I didn't dwell on it too much to be honest :)

Dear friends, please let's not use the topic in a brutal way, I opened this topic, everyone should go to their own topic :))))
Just kidding, now let me give additional information for the part that the friend said, I did a rough installation yesterday, but it is necessary to edit everything manually, so I tried a little bit and got bored and left it :) To be honest, it seems that there should be an installation package for a while, at least for those at the basic level.
I got stuck on the Nginx configuration part, I didn't dwell on it too much to be honest :)

Actually, the nginx configuration file is related to the server, not the software. But I still gave an example of this.
For example, the Install interface of high-level software such as vBulletin and XenForo does not say anything about the Nginx installation. In fact, it cannot be intervened because, as I said, it is related to the server.

Dear friends, please let's not use the topic in a brutal way, I opened this topic, everyone should go to their own topic :))))
Just kidding, now let me give additional information for the part that the friend said, I did a rough installation yesterday, but it is necessary to edit everything manually, so I tried a little bit and got bored and left it :) To be honest, it seems that there should be an installation package for a while, at least for those at the basic level.
I got stuck on the Nginx configuration part, I didn't dwell on it too much to be honest :)

Actually, the nginx configuration file is related to the server, not the software. But I still gave an example of this.
For example, the Install interface of high-level software such as vBulletin and XenForo does not say anything about the Nginx installation. In fact, it cannot be intervened because, as I said, it is related to the server.
Additionally, MentionBB also runs smoothly on Apache. Nginx is not a requirement.

No no no, it's not a problem, it's all about me, you know nginx is like a nice lady who asks for special configurations so I didn't dwell on it too much.
But this installation part that the other friend mentioned is really vital, I'll even give you a little suggestion for the installer in the installation part if you want to take a note;
We can show a small warning so that the installer automatically recognizes the server server and redirects accordingly,
For example: Server type: Nginx, see special configuration file for installation (nginx-install.txt)
Server type: Aphache, You do not need to take any action, you can continue with the installation.
Server type: Litespeed, No action required.
You can also use these suggestions, of course this is my own opinion :)

Dear friends, please let's not use the topic in a brutal way, I opened this topic, everyone should go to their own topic :))))
Just kidding, now let me give additional information for the part that the friend said, I did a rough installation yesterday, but it is necessary to edit everything manually, so I tried a little bit and got bored and left it :) To be honest, it seems that there should be an installation package for a while, at least for those at the basic level.
I got stuck on the Nginx configuration part, I didn't dwell on it too much to be honest :)

Actually, the nginx configuration file is related to the server, not the software. But I still gave an example of this.
For example, the Install interface of high-level software such as vBulletin and XenForo does not say anything about the Nginx installation. In fact, it cannot be intervened because, as I said, it is related to the server.
Additionally, MentionBB also runs smoothly on Apache. Nginx is not a requirement.

No no no, it's not a problem, it's all about me, you know nginx is like a nice lady who asks for special configurations so I didn't dwell on it too much.
But this installation part that the other friend mentioned is really vital, I'll even give you a little suggestion for the installer in the installation part if you want to take a note; We can show a small warning so that the installer automatically recognizes the server server and redirects accordingly, For example: Server type: Nginx, see special configuration file for installation (nginx-install.txt)
Server type: Aphache, You do not need to take any action, you can continue with the installation. Server type: Litespeed, No action required.
You can also use these suggestions, of course this is my own opinion :)

Ah yes, that makes a lot of sense actually. I'll add it to my notes. Actually, Apache doesn't need any settings, Nginx is a bit of a delicate tool.
Actually, I have a lot of plans for the install; ai language translation, downloading all the necessary files from github, usage documentation. But right now my goal is to produce a working install. I am as aware as you are of the high level complexity of even just the DbConfig.php file for the end user.

Yes, it's obvious how much work there is, but it's all something that will happen over time. it's a step by step, gradual progress. but first and foremost to attract the end user, the first step is a very simple installed system.
download as zip -> enter installation info -> forum is ready.
Once this part is done there will be plenty of time for other background work. new developments, additions and edits can already be done based on user comments and feedback.
Again, as I said, opening the website is a must :)

Demo is open now.
https://mentionbb.com/

Demo is open now.
https://mentionbb.com/

Hello again
We can actually do something like this, if you accept, I have prepared a portal introduction for Mentionbb, you can look at it here; https://github.com/slaweallx/mentionbb/blob/master/mention.zip
Mentionbb.com welcomed us here at the entrance, followed by the Forum (active for the real community) not the demo
Download directory (All versions)
You can divide it into sections like this, this is just an idea.

The idea is good but I guess this is a draft?
No need for a separate project, we can do this as a plugin. Let me set up the addon files, you can do it with HTML and send a pull-request. Do you know how to use Twig Template?

Aaa no no no, I made it as a portal just for the entrance, let the HTML stay, for example, if you look at smf mybb, I thought as a simple portal to the entrance like them, I thought we would welcome there. But it can also be done as an add-on, of course, it can be made active closed from the panel as a portal, but I have arranged and prepared it only for mentionbb specially, so it is not for everyone to use it.

It can be done as an add-on, but making it so that the end user can turn it on and off doesn't make sense right now.
Frankly, your idea is great, but the design is not worthy of Mention. You can make a design that is more Mention-friendly, or I can design it in the future.

Yes, yes, yes, I just gave an idea.

In the new version you can simply edit the .env.example file or create an automatic file with the command line. (Of course this will be included in the install)
php bin/console mention:install-db [--dbadapter DBADAPTER] [--user USER] [--password PASSWORD] [--dbname DBNAME] [--dbhost DBHOST]

*Excluding square brackets
OR
Rename:

.env.example to .env

Make your database settings in this file.
Or if you using Plesk, make it easy: https://github.com/mentionbb/mentionbb-plesk-setup

In the new version you can simply edit the .env.example file and rename to .env or create an automatic file with the command line. (Of course this will be included in the install)
php bin/console mention:install-db [--dbadapter DBADAPTER] [--user USER] [--password PASSWORD] [--dbname DBNAME] [--dbhost DBHOST]

*Excluding square brackets
OR Rename:

.env.example to .env

Make your database settings in this file.
Or if you using Plesk, make it easy: https://github.com/mentionbb/mentionbb-plesk-setup

I think it's better to share release notes for new developments separately. This way it makes more sense than writing it here because users should see it too.
Also the progress is going very fast and good :)

In the new version you can simply edit the .env.example file and rename to .env or create an automatic file with the command line. (Of course this will be included in the install)
php bin/console mention:install-db [--dbadapter DBADAPTER] [--user USER] [--password PASSWORD] [--dbname DBNAME] [--dbhost DBHOST]

*Excluding square brackets
OR Rename:

.env.example to .env

Make your database settings in this file.
Or if you using Plesk, make it easy: https://github.com/mentionbb/mentionbb-plesk-setup

I think it's better to share release notes for new developments separately. This way it makes more sense than writing it here because users should see it too.
Also the progress is going very fast and good :)

Actually, they do exist.
https://github.com/mentionbb/mentionbb/releases/tag/mentionbb-11020000-5


Job: Add support for runtime setup commands

This commit adds support for running commands inside the Job runtime environment before the actual node entrypoint. This allows users to run setup commands such as installing packages or missing libraries without having to modify the lambda base image.
We have this requirement on a project where we don't want to modify the lambda images but still need to run simple setup tasks.
Thanks.

Hmm I wonder if the better option here just might be to customize the docker image?

@jayair Thanks for the feedback!
Customizing the image is probably the standard path. However, there are scenarios where it's easier to just run setup commands during runtime, and having this functionality doesn't add a lot to the code base.

Yeah let me think about it. We might want to support Docker properly here.

It would be nice to have both options: setting up a custom Docker image and run pre/post commands. I have the same use case of @gslandtreter: installing browsers (Playwright) for screenshots/scrapping.

Docker we are working on soon!

Hey @gslandtreter, I just added container runtime support to Job in v2.21.1.
You can override CMD of the Docker image. Here's an example — https://docs.sst.dev/constructs/Job#creating-a-container-job
Does that work for you?

Hi @fwang, we heard the news!
Although we have not tested it yet, I'm confident it will work perfectly for our use case. Appreciate your support and the fact that you guys are hearing the community when it comes to adding new features.
Feel free to close this PR!
Thanks!


[BEAM-8472] Get default GCP region from gcloud (Java)

Same as #9868 but more verbose :smile:
As with the Python version, I've verified this works locally. This doesn't fit into a unit test because it requires special environment configuration and/or calling an external process, but nor does it seem to warrant an integration test.

Run Java PreCommit


[Build] Limit Ubuntu 18.04 LSAN CI to 12 link jobs

It's currently running out of memory, likely due to the machine's high core count and linking many jobs at once. Limit to 12 only.

@swift-ci please smoke test


Set metadata in ogg files

Hi everyone,
I'm newbie in TagLib but I'm discover so many new interesting things.
I'm working with Rails but I'm having problems with .ogg files.
It's my code:
def set_audio_tags
  TagLib::Ogg::Vorbis::File.open(self.audio.path, read_properties = true) do |file|
    tag = file.tag
    tag.title = 'file.title'
    tag.artist = 'abcd'
  end
end

and it's my trace of file:
 #<TagLib::Ogg::Vorbis::File:0x0000000a5d8180 @__swigtype__="_p_TagLib__Vorbis__File">

but I can't set 'tag' because it's nil
Can anyone explain me how to create new audio metadata fields in OGG::Vorbis fields?
Thanks for all ;)

Works fine here, just tried it using the first file from here: http://www.vorbis.com/music/
tag is nil, are you sure? Have you tried adding a file.save as the last line of the block? Otherwise the fields are not saved to the file.

hi @robinst thnaks for your answer. I sad to say that your answer doens't solve the problem.
def set_audio_tags
  TagLib::Ogg::Vorbis::File.open(self.audio.path, read_properties = true) do |file|
    #I write a binding.pry here and it say that file.tag = nil so if I try to write file.tag.title = 'somethig' it say
     that 'NoMethodError - undefined method `title=' for nil:NilClass:'

    tag = file.tag
    tag.title = 'file.title'
    tag.artist = 'abcd'
  end
end

What's self.audio.path?  I can't figure out what's wrong with it.

Hmm.. Is it possible for you to upload the audio file somewhere and send me the link?


Use updated DataDog/gopsutil for platform identification

What does this PR do?
Switches to using DataDog fork of gopsutil for host platform identification. The fork contains a fix so that we read VERSION_ID instead of VERSION from /etc/os-release.
Motivation
When we get host platform information from /etc/os-release, getting the platform version from the VERSION field can give us a string with the platform codename in it (ie "22.04.1 LTS (Jammy Jellyfish)"). We use the platform version to generate a file path for looking up BTFs, so we need to use the VERSION_ID field, which gives us the simplified version (ie "22.04").
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[x] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Wrote 2 comments on the gopsutil PR https://github.com/DataDog/gopsutil/pull/44, are we sure this does not break other code ?

I've double checked and gotten the 👍 from Processes that this doesn't break anything.


Working example

To review, follow the instructions in README.md and check that everything works as expected.

I have followed the build instructions from the above mentioned README on a DAS6 GPU node at our site with cmake version 3.20.2, gcc/9.4.0 and cuda/11.6.2. Build of both the cudawrappers library as well as the locally installed example from this repository seem to pass but upon running the example I get an error: cu::Error: the provided PTX was compiled with an unsupported toolchain.

Thanks for trying this out @svlugt!
I'm no expert on this, but I think that the cuda driver may be too old for the cuda toolkit that you're running. When I run nvidia-smi on the DAS6 astron cluster, it reports a driver version of 495.29.05. From the cuda toolkit release notes, it looks like this driver version is only supported up to cuda toolkit 11.5. 11.6.2 requires at least 510.47.03

@bouweandela there is indeed a mis match in my CUDA version and the driver, sorry for not picking this up!
I have re-build with an older version of CUDA, now I get another error though: cu::Error: named symbol not found. This still might be an issue from my side I'll look in to this when I have a bit more time and get back to you.

Could you post the commands that you're running here? Then I can try to reproduce the problem.

@bouweandela we have installed the new CUDA drivers on DAS6 and I am now using CUDA 11.6.2 with both the cudawrappers library and the example.
I have followed the build instructions from the cudawarappers repository and this repository.
I get different results depending on the location from where I'm running the vector_add example.
Running from my home directory:
~ $ vector_add_example
cu::Error: named symbol not found

Running from within the example directory:
~/usage-example-locally-installed $ vector_add_example
hurray! 2

So the example application works, but not in all situations.
The problem seems to be that the .cu file is not found when the example is run from another directory. Similarly to what is described in issue 116 .
As this is quite a common situation I do think that it's good to add to the example how to deal with this.
A solution might be to install the kernel sources along with the binary in the install directory and parse the kernel path with a cmake runtime variable to the application.

Also make sure that the CUDA source can be found when running through nv-nsight-cu.  Unfortunately, the profiler changes the current directory under some circumstances, so that the source file cannot be found while it would have been found when not running via nv-nsight-cu.  This was exactly the reason why I embed the CUDA code in an object file so that it is inside the executable and can always be found, but embedding CUDA code in object files significantly complicates the building processes, and may cause its own problems.

Thanks for trying this out @svlugt and @john-romein! I included the cuda code as a string now. Does this look like a good solution to you?
I will also have a look if I can improve the error message cu::Error: named symbol not found in the nvrtc library, something like file '/path/to/file.cu' not found would be much more clear.

A pull request with an improved error message is available here: https://github.com/nlesc-recruit/cudawrappers/pull/159

A follow-up MR is coming soon, so I will merge this to prepare for it.


Color animation issue

What's happening?
Depending on the color value you want to animate, some weird colors appear during the animation.
https://user-images.githubusercontent.com/6232522/125181736-815f8c00-e1cd-11eb-8a63-cb37b86f0d48.mov
Expected behaviour
The color animation is smooth, like any css animation would happen on web
Example:
https://snack.expo.io/@brandonma/color-bug-reanimated
Further information
This bug is actually happening in Reanimated 2 and not Moti itself, as I assume you are implementing the animation the same way I did in the example under the hood, the thing is, this bug doesn't happen if you use interpolateColor instead of this approach(I was building a library like Moti and had the same problem before but decided to use Moti as it seems better overall).
I created an issue a while ago but the Reanimated team did not pay much attention to it, I will create another one so if you can help getting some attention would be nice.
Also, if you are interested in more information about the workaround using interpolateColor let me know.

The issue on the Reanimated repo has been closed, basically using interpolateColor is not a workaround but the correct way to animate colors, here is the link of the issue:
https://github.com/software-mansion/react-native-reanimated/issues/2193

Interesting. So the idea is, you shouldn't animate between two color values, right? I actually thought I had it working, but maybe this will need some magic under the hood to add interpolate color...

Interesting. So the idea is, you shouldn't animate between two color values, right? I actually thought I had it working, but maybe this will need some magic under the hood to add interpolate color...

Yes! Actually what I used to do in the past to animate colors was to save the previous color on a reference, and then use that to build the new array, the trickiest part is to work with the animated value of 0 and 1, you would have to make it 0 again at some re render when the new array is generated for no weird color flickering, so a "magic" extra render would have to be introduced for color animations to work.
I am curious on why they decided this approach of animating colors tho, iOS doesn't need this to perform animations natively, not sure about Android but still seems like a bug for me on their side, that being said this is the best idea I had to fix it with interpolateColor.

Yeah, one other slightly hacky solution would be to have clone elements, one with a different color, and transition the opacity from one to the next haha.
This might be a use case that will be solved by https://github.com/nandorojo/moti/pull/54

Since this is a reanimated issue, I'm going to close here.

Since this is a reanimated issue, I'm going to close here.

Hello! Sorry for the late reply I have been really busy at school, but as I mentioned before, is not really a problem with Reanimated, at least according to them this occurs because color animations should be performed with interpolateColor(), I am not sure if you are already doing this, but just to be sure, if we do it that way colors animate as expected.
Reanimated issue:
https://github.com/software-mansion/react-native-reanimated/issues/2193

The thing is, that would require interpolating colors based on their previous value. Since moti is declarative, I’m not sure how this would work, since we don’t track the previous value, and there’s no shared value to interpolate. This use case might be best solved with using useAnimatedStyle and a shared value. I’m not sure how else it would work.

It is possible to implement this feature on a declarative way if you save the previous color value on a reference, and just performing the animation if the previous value is not undefined/null and is different from the current color, I implemented this before and it worked fine, I can create a small example on how to do it if you need it because I am not that familiar with Moti's codebase

Yeah, I’m just not totally sure if Moti should be tracking the previous value that way or not. Might fall outside of the simple scope I have working now. I can look at an example and consider it.

Just wanted to say that I ran into this issue as well. It would have been amazing if it were handled by Moti, but I cannot speak to how it should be implemented.


[Attachments] Extension appears instead of file name if to clear it

EPAM AI DIAL chat version
0.10.0
What happened?

Click on Clip icon on Conversation panel
Click on Upload from device button
Select any file -> Open it
Set cursor in file name field and clear the name -> note that extension is moved from it's position to file name position; cursor stays at the end of new file name-> bug

Click on backspace -> extension appears again, file names still has a part of extension

Note: if on the 4th step click on Upload button, the file with empty name is uploaded
Expected result: If to clear file name, extension stays on it's place, file name is empty. It's not allowed to upload the file with empty name.
Confidential information

[X] I confirm that do not share any confidential information

See also #1289 seems they have equal root cause

Also it's impossible to remove dot from the end


Terminal shows previous mangled buffer text when scrolling up after a resize action

Windows Terminal version
1.18.3181.0
Windows build number
10.0.19045.3693
Other Software
micro text editor 2.0.13 -- https://github.com/zyedidia/micro
nano for windows 7.2-22.1 -- https://github.com/okibcn/nano-for-windows

Steps to reproduce
Scrollbar visibility == Hidden, History size == 9001
Launch windows terminal, launch micro text editor. Write a few lines, resize terminal (larger->smaller or smaller->larger). Resizing unlocks ability to scroll up using keyboard bindings or using mouse select. Previous buffer shows with characters jumbled up / misaligned.
Same issue using windows terminal with nano editor.
Same issue using wezterm with both micro and nano.
Same issue using alacritty with both micro and nano.
Tested with nvim. No issue with nvim on all terminal emulators (cannot scroll back after resizing). This should be actual behavior for all editors.
Expected Behavior
Text should be redrawn normally and inability to scroll back should be maintained.
Actual Behavior
Resizing "unlocks" scrolling again, causing previous ghost text/buffers to show up. Particularly annoying when trying to drag-select with the mouse near the top of the file. Very confusing when seeing the previous buffers, especially when the text is all misaligned.

(I'm about to leave for the day, so just dropping some notes in here.)
This typically happens when a text editor is not properly using the "Xterm Alternate Screen Buffer". That's a mode that (1) restricts all scrolling and (2) ensures that nothing on the screen gets put into the scrollback buffer.
I wonder if applications are not aware that it exists on Windows, or perhaps their language runtimes don't allow them to use it.

Thanks for filing! Looks like this is being handled over in https://github.com/zyedidia/micro/issues/3073 so I'm going to close this as external.


  Sometimes you have 16 shades to work with. sometimes you have 2.  sometimes more.  My color theory is not good enough to know a good algorithm for this (I know you need to take RGB colors weighted by some factor of eye sensitivity etc).

True, well maybe the naming isn't ideal. While linear and sRGB are color spaces, this is really only useful for gamma-correction and not for grayscale conversion. I agree that such a conversion would be handy though, but it'd be another separate feature. I may rename the current color spaces to reflect their function better, any advice always welcome! What about "GammaCorrector"?

From my reading of what it does now; I think that is clearer.  That said I haven't don any serious colourspace stuff in 7 years (which wasn't much to begin with) and was print CYMK stuff; which is its own barrel of crazy.
For my use cases (which basically involves blitting changes to epaper), i may add the transaltion on the final stage on a per-device level.  Eg, if its 2 bit (BW), 3-bit (BW + Red or Yellow), 4 bit (16 grey) etc.  in these cases the exact 'color' representation is display specific, so it makes sense to happen right at the end while keeping the intermediate layering and composition in a consistent space.

That is absolutely a useful feature which I can implemented in two ways: a Filter struct between Context and a renderer that converts colors. You can draw to Canvas and then redraw to your final target passing through that filter. Otherwise, implement in the renderer itself if this is very renderer specific. What render target do you use? The rasterizer? You can use the rasterizer to draw to image.Image and then use another library to convert that to different color palettes, that might be the best way.

My plan was rasterizer -> image.Image -> change colour space -> low level outputs; in my case, its SPI (https://en.wikipedia.org/wiki/Serial_Peripheral_Interface).
I'm super happy you updated the docs because that article on gamma is really, really helpful.  I need to investigate the gamma corrections for the e-paper displays I'm targeting.  This is something I never considered.

To close this issue, I think that color space is the correct name, even though it only treats gamma correction color spaces. The specific requested functionality is best implemented as you said: rasterizer to an image and from there you can convert it to grayscale however you like. Happy to answer other questions!


adds remote maven repositories

Forgot to add them, sry!

Forget about the pull request, just c&p the single line in there. Probably easier then a merge.

Cool, done. I'm now getting this:
Building robotlegs-framework
Buildr aborted!
RuntimeError : Don't know how to build task '/Development/Projects/Robotlegs/robotlegs-framework/build/libs/SwiftSuspenders-v2.0.0b1.swc'
What to do? Sorry for all the questions!

Can you verify that the SwiftSuspenders swc file exists.

switching to IM

:)


Choose random mapping locations for single-end multimappers

This is an alternative to #360 that works by shuffling the top NAMs that have the same score as the best one.
See #359
samdiff.py outputs this when comparing to main:
       9643 reads were unmapped before and after
          0 reads became mapped
          0 reads became unmapped
      82741 reads were mapped to same locus before and after
*      7421 reads were multimapper before and after, same alignment score (AS)
*        17 reads were multimapper before and after, better alignment score (AS)
*         5 reads were multimapper before and after, worse alignment score (AS)
*       173 reads changed in another way
     100000 total reads

The high number of reads that get the same alignment score before and after is a sign that this works as intended. There are some reads that get a different score, but this is as expected and the price to pay for this faster variant that only shuffles the NAMs.
Accuracy changes in an unremarkable way (I ran this comparison only on a subset of the datasets):

dataset
main (8c74210)
this PR
difference

drosophila-50-se
82.5175
82.5108
-0.0067

drosophila-100-se
89.2626
89.255
-0.0076

drosophila-150-se
90.937
90.9448
+0.0078

drosophila-200-se
91.9463
91.9559
+0.0096

drosophila-300-se
93.2153
93.2187
+0.0034

maize-50-se
47.4622
47.4417
-0.0205

maize-100-se
70.5148
70.503
-0.0118

maize-200-se
86.7001
86.7083
+0.0082

maize-300-se
91.8516
91.8522
+0.0006

CHM13-50-se
81.7231
81.7321
+0.0090

CHM13-100-se
90.6468
90.6401
-0.0067

CHM13-150-se
92.4066
92.4081
+0.0015

CHM13-200-se
93.221
93.2167
-0.0043

CHM13-300-se
94.1508
94.1472
-0.0036

Average difference se: -0.0015
We don’t have a way at the moment to test whether this results in the intended outcome, that is, whether locations for multimappers are less biased.

Awesome! I will check on the 10 E. coli genomes dataset that I generated before. (also pinging @psj1997 @luispedro FYI)

Oh, I realize that this is currently only for SE. I will do the check in SE mode on the 10 E. colis.

One could do the exact same thing with the list of NAMs in paired-end mode. Do you think this could help a little bit until we have something smarter?

One could do the exact same thing with the list of NAMs in paired-end mode. Do you think this could help a little bit until we have something smarter?

I added that now. It seems to do something ...:
      17339 reads were unmapped before and after
*         5 reads became mapped
*         2 reads became unmapped
     171879 reads were mapped to same locus before and after
*      9761 reads were multimapper before and after, same alignment score (AS)
*        22 reads were multimapper before and after, better alignment score (AS)
*        11 reads were multimapper before and after, worse alignment score (AS)
*       981 reads changed in another way
     200000 total reads

I first compared a30d779 (strobealign-random) with v0.11.0 (strobealign-default). I accidentally ran in PE mode, the uniformity was still as biased as in v0.11.0 (as expected). However, I noticed a small change in accuracy, stats below. Is this expected?

Then you posted commit 57fe7df so I tried that too. Here are the accuracy stats:

The bump in accuracy is quite bizarre if you ask me. This is a subset of 10k reads so sample size is small, however it is nearly 1% improvement. I have to test on a larger dataset to see if it evens out.

Now to what I was actually gonna test (the randomness). The name of the game is to get close to 0.945-0.950 in the last column (based on bowtie2 and BWA's distributions). This column measures "fraction of bases on chromosome with depth equal to column 2 (i.e., 0 in our case)". The true numbers for each genome are unknown. However, since reads were simulated at random, I expect then to be the same if there is no mapping bias.
As we can see the random version improves the numbers a lot - although not as even as BWA/Bt2 (but we do obtain higher accuracy than both of them on this small dataset).
v0.11.0
$ bedtools genomecov -ibam /Users/ksahlin/tmp/STROBEALIGN_ECOLI/1/ecoli_10/150.strobealign_default.bam | grep -E "[0-9]+\t0\t"
0	0	4900606	5269732	0.929954
3	0	4878646	5217379	0.935076
5	0	5009761	5270094	0.950602
8	0	4955499	5255484	0.94292
11	0	5051905	5214647	0.968791
15	0	5370618	5703439	0.941646
18	0	5510964	5730399	0.961707
21	0	5139976	5536811	0.928328
26	0	5250803	5668289	0.926347
32	0	5674359	5770266	0.983379

commit 57fe7df
$ bedtools genomecov -ibam /Users/ksahlin/tmp/STROBEALIGN_ECOLI/1/ecoli_10/150.strobealign_random.bam | grep -E "[0-9]+\t0\t"
0	0	4968465	5269732	0.942831
3	0	4928918	5217379	0.944712
5	0	4989804	5270094	0.946815
8	0	4958396	5255484	0.943471
11	0	4946985	5214647	0.948671
15	0	5334311	5703439	0.93528
18	0	5464417	5730399	0.953584
21	0	5227056	5536811	0.944055
26	0	5396273	5668289	0.952011
32	0	5511444	5770266	0.955146

BWA-MEM
$ bedtools genomecov -ibam /Users/ksahlin/tmp/STROBEALIGN_ECOLI/1/ecoli_10/150.bwa_mem.bam | grep -E "[0-9]+\t0\t"
0	0	4979591	5269732	0.944942
3	0	4928776	5217379	0.944684
5	0	4981872	5270094	0.94531
8	0	4988722	5255484	0.949241
11	0	4930204	5214647	0.945453
15	0	5411526	5703439	0.948818
18	0	5431322	5730399	0.947809
21	0	5234316	5536811	0.945367
26	0	5364423	5668289	0.946392
32	0	5466397	5770266	0.947339

Bowtie2
$ bedtools genomecov -ibam /Users/ksahlin/tmp/STROBEALIGN_ECOLI/1/ecoli_10/150.bowtie2.bam | grep -E "[0-9]+\t0\t"
0	0	4981502	5269732	0.945305
3	0	4929527	5217379	0.944828
5	0	4988778	5270094	0.94662
8	0	4978964	5255484	0.947384
11	0	4927270	5214647	0.94489
15	0	5398555	5703439	0.946544
18	0	5431461	5730399	0.947833
21	0	5251857	5536811	0.948535
26	0	5360894	5668289	0.945769
32	0	5469114	5770266	0.94781

(I also checked the distribution in SE mode and it evend out the distribution substantially (I don't have the BWA/Bt2 comparison stats) but I beleive it works as expected.

I couldn't believe the accuracy increase, and rightfully so. I simulated three replicate instances with 100k PE reads in each instance, below are the PE alignment results. There seems to be no noticeable effect on accuracy, which we expect, I guess.
I will to the same analysis for 100 genomes (I want to check the effect as more repeated NAMs are introduced)

The read simulator complained about the 100 ecoli dataset, but here are the results for 50 ecoli genomes.

I also checked the read placement randomness (coverage uniformity across genomes). Looks much better - centered around 0.88-0.89 (below I dump the data for that for documentation - sry).
I think we can merge this. What do you say? We still have the potential improvement in PE NAM pairing that can be added before the random shuffle, but for now this version should be an improvement over baseline.
(devel) math172l2:OBJ_C ksahlin$ bedtools genomecov -ibam /Users/ksahlin/tmp/STROBEALIGN_ECOLI/1/ecoli_50/150.strobealign_default.bam | grep -E "[0-9]+\t0\t"
0	0	4425709	5269732	0.839836
3	0	4785792	5217379	0.917279
5	0	4729574	5270094	0.897436
8	0	4642790	5255484	0.883418
11	0	4801312	5214647	0.920736
15	0	4957996	5703439	0.869299
18	0	5266074	5730399	0.918972
21	0	5163049	5536811	0.932495
26	0	5181816	5668289	0.914176
32	0	5296838	5770266	0.917954
37	0	5288767	5767030	0.917069
41	0	5232706	5747479	0.910435
44	0	5150896	5770126	0.892683
47	0	5540245	5833706	0.949696
53	0	5184368	5722815	0.905912
58	0	4758577	5366488	0.886721
62	0	4831584	5226109	0.924509
65	0	4628852	5029720	0.9203
72	0	4629632	5061873	0.914608
75	0	4498989	5110834	0.880285
80	0	4732493	5132219	0.922114
85	0	4712882	5255408	0.896768
87	0	4542237	5111512	0.888629
89	0	4415368	5062632	0.872149
92	0	4291407	5079315	0.844879
95	0	4636143	5197709	0.891959
99	0	4407837	5041928	0.874236
102	0	4614709	5138060	0.898142
108	0	4492315	5138043	0.874324
114	0	4749347	5147947	0.922571
122	0	4628475	5029969	0.92018
126	0	4582979	5103692	0.897973
132	0	4076951	5098211	0.799683
133	0	4464208	5098635	0.875569
135	0	4519142	5255987	0.859808
137	0	4671210	5256017	0.888736
146	0	4708633	5099979	0.923265
147	0	4636300	5085570	0.911658
148	0	4624204	5099806	0.906741
149	0	4629665	5088968	0.909745
151	0	4668302	5089992	0.917153
153	0	4516693	5088968	0.887546
155	0	4759101	5285418	0.900421
159	0	4673995	5072410	0.921454
161	0	4717066	5060072	0.932213
163	0	4794753	5090382	0.941924
165	0	4679791	5088970	0.919595
167	0	4532985	5285512	0.857625
169	0	4851099	5285097	0.917883
171	0	4387097	5284087	0.830247
(devel) math172l2:OBJ_C ksahlin$ samtools view -b /Users/ksahlin/tmp/STROBEALIGN_ECOLI/1/ecoli_50/150.strobealign_random.sam | samtools sort -o /Users/ksahlin/tmp/STROBEALIGN_ECOLI/1/ecoli_50/150.strobealign_random.bam
(devel) math172l2:OBJ_C ksahlin$ bedtools genomecov -ibam /Users/ksahlin/tmp/STROBEALIGN_ECOLI/1/ecoli_50/150.strobealign_random.bam | grep -E "[0-9]+\t0\t"
0	0	4683567	5269732	0.888768
3	0	4702112	5217379	0.90124
5	0	4709841	5270094	0.893692
8	0	4632677	5255484	0.881494
11	0	4728239	5214647	0.906723
15	0	4962045	5703439	0.870009
18	0	5287916	5730399	0.922783
21	0	5061211	5536811	0.914102
26	0	5200693	5668289	0.917507
32	0	5280821	5770266	0.915178
37	0	5306395	5767030	0.920126
41	0	5262653	5747479	0.915645
44	0	5176862	5770126	0.897184
47	0	5383307	5833706	0.922794
53	0	5259823	5722815	0.919097
58	0	4761994	5366488	0.887358
62	0	4776485	5226109	0.913966
65	0	4493271	5029720	0.893344
72	0	4590887	5061873	0.906954
75	0	4497195	5110834	0.879934
80	0	4639345	5132219	0.903965
85	0	4705894	5255408	0.895438
87	0	4568653	5111512	0.893797
89	0	4399648	5062632	0.869044
92	0	4248203	5079315	0.836373
95	0	4609376	5197709	0.886809
99	0	4386183	5041928	0.869942
102	0	4571898	5138060	0.88981
108	0	4535183	5138043	0.882667
114	0	4673614	5147947	0.90786
122	0	4565770	5029969	0.907713
126	0	4563826	5103692	0.89422
132	0	4550035	5098211	0.892477
133	0	4472361	5098635	0.877168
135	0	4533234	5255987	0.86249
137	0	4624282	5256017	0.879807
146	0	4680840	5099979	0.917816
147	0	4626446	5085570	0.90972
148	0	4601493	5099806	0.902288
149	0	4630542	5088968	0.909918
151	0	4658783	5089992	0.915283
153	0	4560742	5088968	0.896202
155	0	4674017	5285418	0.884323
159	0	4635390	5072410	0.913844
161	0	4657656	5060072	0.920472
163	0	4684726	5090382	0.920309
165	0	4675791	5088970	0.918809
167	0	4610326	5285512	0.872257
169	0	4702754	5285097	0.889814
171	0	4606436	5284087	0.871756

Finally, I also checked that in mapping mode -x, the accuracy stayed the same (statistically) the randomness improved.
First, in the 50 genomes dataset the true number highest and lowest number of reads simulated from the genomes are and 4,530 and 3,680, respectively.
Counting the number of reads mapping to each of the 50 genomes, I observed 8699 and 1974 reads mapped to highest and lowest genome for v0.11.0, respectively.
For this PR I observed 6913 and 2788, respectively.
So a big improvement but still quite far of the true values. However, we need to be careful not to get too disappointed here because statistical variation comes into play (too tired to derive how large it is but upper bound - all reds completely random - probably follows some common distribution).

However, I noticed a small change in accuracy, stats below. Is this expected?

Yes, that change is from #336.
Here is the change in accuracy just from this PR.

Did you see any noticeable slowdown?
Would this approach work for PE?

Did you see any noticeable slowdown?

It did not become measurably slower.

Would this approach work for PE?

I think something similar could work; I will look into it.

So this PR now

adds NAM shuffling for single-end reads,
adds NAM shuffling for paired-end reads,
and adds random read selection (among those with the same score) for single-end reads.

I would like to suggest that we merge this PR with the three features above included and that I open a separate PR for adding random read pair selection for paired-end reads (so as not to make the discussion here too long).

Awesome! Approved to merge.


I can not find the weights after training

As described , I can not find the weights on my server after traing.

I fixed it by the following picture.


Scheduler Feature

这个 PR 实现什么功能/修复什么问题:

添加了instance group 和相关的 climc 命令
支持主机分散功能，在创建一个虚机的时候可以指定诺干个group，每个group有一个Granularity字段，表明每台宿主机最多课容纳的属于此Group的虚机的数量

是否需要 backport 到之前的 release 分支:

release/2.12

Codecov Report

Merging #3132 into master will decrease coverage by <.01%.
The diff coverage is 2.59%.

@@            Coverage Diff            @@
##           master   #3132      +/-   ##
=========================================
- Coverage    7.17%   7.16%   -0.01%
=========================================
  Files         478     478
  Lines       76895   76964      +69
=========================================
- Hits         5514    5512       -2
- Misses      70919   70989      +70
- Partials      462     463       +1

Flag
Coverage Δ

#aFlag
7.16% <2.59%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/mcclient/options/servers.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/guests.go
0.65% <0%> (-0.01%)
:arrow_down:

pkg/scheduler/models/pending_usage.go
22.04% <0%> (-1.96%)
:arrow_down:

pkg/compute/models/hosts.go
0.38% <0%> (-0.01%)
:arrow_down:

pkg/compute/models/groupguests.go
2.17% <0%> (-0.69%)
:arrow_down:

pkg/compute/models/groups.go
56.25% <100%> (ø)
:arrow_up:

pkg/util/seclib2/seclib.go
91.48% <0%> (-4.26%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db7880c...a2896aa. Read the comment docs.

/area region scheduler

/cc @zexi

/lgtm
/approve


How to use the default keyring using SecretServiceBackend?

Whatever I use for LibSecretCollectionName or ServiceName, the library attempts to create a new keyring. How can I use the default keyring? In seahorse it is listed as Default keyring:

Even if I name the collection the same way, the library will create a new keyring with the same name:
package main

import (
	"github.com/99designs/keyring"
)

func main() {
	ring, err := keyring.Open(keyring.Config{
		AllowedBackends: []keyring.BackendType{
			keyring.SecretServiceBackend,
		},
		LibSecretCollectionName: "Default keyring",
		ServiceName:             "myapp",
	})
	if err != nil {
		panic(err)
	}

	err = ring.Set(keyring.Item{
		Key:  "foo",
		Data: []byte("secret-bar"),
	})
	if err != nil {
		panic(err)
	}
}

With the https://github.com/zalando/go-keyring library, using their example from the README will write to the default keyring, so it should be possible somehow.

@benma I was able to work around this by not including a space in my LibSecretCollectionName.


Is there a way to configure apiUrl for both dev and prod?

In order for me to get apiUrl to work on dev, I currently have to set it as:
$authProvider.configure({ apiUrl: 'http://localhost:3000/' });
I would like to set it as:
$authProvider.configure({ apiUrl: '/' });
so that it would be independent of dev and prod. Unfortunately, this doesn't seem to work. Does anyone have suggestions for how to do that? Thanks!
Currently I have my rails route as follows:
Rails.application.routes.draw do

  root 'angular_app#index'
  mount_devise_token_auth_for 'User', at: 'auth'

  namespace :api, defaults: { format: 'json' } do

    # List a user's rooms, create and show a room (including the messages within it)
    resources :rooms, only: [:index, :create, :show] do
      # Create and retrieve a message
      resources :messages, only: [:create], shallow: true do
      end
    end

    # Join and leave a room
    resources :participations, only: [:create, :destroy] do
    end
  end

  get '*path', to: 'angular_app#index', constraints: { path: /(?:(?!omniauth\/).)*/ }
end

What is the issue when using this setting?
$authProvider.configure({ apiUrl: '/' });

When I use that setting, clicking on a simple "Sign in with Google" button goes to
http://auth/google_oauth2?auth_origin_url=http%3A%2F%2Flocalhost%3A3000%2Flogin
In other words, it doesn't seem to know to append urls to the root...

@davidtlee - did you get this sorted out?

nope, whoops, I must've accidentally closed it.

So I didn't fix it, but now the problem is no longer relevant since I just decided to change to:
$authProvider.configure({ apiUrl: 'api' });
and then change the rails route to:
mount_devise_token_auth_for 'User', at: 'api/auth'


Bug: system tray weird behavior

The system tray looks like this:

In production, that should not be the case.

Moving the app off the system tray automatically fixes this.
We will revisit having an applet on the system tray later.


Downloading screenshot/html-report if not installed #782

@cla-bot check if all the contributors of this pull request has signed our CLA.


Learning images not loading on iOS

Summary:
This morning it appears that the images for courses will not load on iOS or web.
Is this a static content issue?

Confirmed bug not working on local dev as well as ios. Definitely static content issue. This was introduced once we migrated to Amazon S3. I will investigate and fix.