email input selected on page load

Now the email field on the login screen is auto-selected when the user loads Portal.  It's a minor UX improvement so they don't have click on it to fill it out.
Preview:

Merged #476 into dev.


read blocked list from chrome storage

Todo

[x] make reddit respect the on/off switch and use the user's chosen blocked list (link to test)
[x] make instagram respect the on/off switch and use the user's chosen blocked list (link to test)
[x] make twitter respect the on/off switch and use the user's chosen blocked list (link to test)
[x] use the default list of banned words if no words are configured by the user

Implementation
The popup already saves user settings into chrome's storage API, which was added in #67 using chrome.storage.sync.set. This PR updates the extension workers to use chrome.storage.sync.get to read those preferences and adhere to them.
Since loading data from chrome's storage takes time, the blurring process is delayed until the user preferences are loaded. This was achieved by introducing window.onceReady, which is called by shared.js. The files for each social media platform can implement window.onceReady differently.
Test plan

test turning the switch on/off and then visiting each social media website (or reloading the page)
test configuring a custom list of banned words and see if this takes effect

Motivation
The user's settings weren't being respected, which is confusing and a bug. We want to fix all remaining bugs.
Attached GitHub issue
Closes #82
Documentation

[x] Updated the readme documents (README.md etc.)
[x] New functions and methods have additional comments added to document their usage

Local Build

[x] Ran all pre-commit hooks pre-commit run --all-files
[x] Extension has been tested to build correctly cd extension && yarn && yarn build
[x] Backend has been tested to build correctly cd backend && docker-compose build
[x] Extension test suite has been run locally cd extension && yarn && yarn test
[x] Backend test suite has been run locally cd backend && python3 -m unittest

GitHub

[x] Target branch has been set correctly
[x] PR has been rebased onto target branch
[x] PR has been assigned an owner
[x] Author has performed a self-review of the code
[x] Removed the WIP message from the top of this PR

Codecov Report

Merging #86 (9622286) into Release-1.0 (f13efe7) will decrease coverage by 0.37%.
The diff coverage is 43.75%.

@@               Coverage Diff               @@
##           Release-1.0      #86      +/-   ##
===============================================
- Coverage        70.27%   69.89%   -0.38%
===============================================
  Files               11       12       +1
  Lines              370      382      +12
  Branches            52       55       +3
===============================================
+ Hits               260      267       +7
- Misses              80       83       +3
- Partials            30       32       +2

Impacted Files
Coverage Δ

extension/public/popup.js
57.40% <0.00%> (-1.65%)
:arrow_down:

extension/public/worker/shared.js
77.77% <77.77%> (ø)

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

Good point, we could discuss this at our next meeting


message list width jumps around depending on window size

Run the app, login
Resize the window, making it narrower and wider

Expect: message list to either have a fixed width, or a minimum width that expands with the window width
Result: width goes through a number of gyrations

Bootstrap settings just needs some refinement here. They're overly aggressive in terms of what is considered "medium" and "small".

Should be better now.  @billba can you check and close?

Better. I'd call this above the bar for the Alpha.


Add travis-ci related files

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

Changes Unknown when pulling 87e40fead80f9dff5a546175d4e018bb5f04f121 on ci into ** on master**.


Request are printing only ipv6

I am running ipify-api on my Kubernetes cluster using an external GCP load balancer.
When I send a request to my ingress that point to the ipify workload, it returns the IPV6 instead of IPV4.
If I send a request from another computer/cellphones it returns IPV4 or IPV6.
I would like to know if there is a way to run it on my infra like the public official one:
https://api.ipify.org (Always return ipv4)
ttps://api64.ipify.org (Always return ipv6)
Thanks!

So you are seeing the IP of the load balancer?

So you are seeing the IP of the load balancer?

No, I am seeing my own IPv6.
The same one I am able to see if go here: https://api64.ipify.org


Smart folder can't be configured

I think an image shows more than 1000 words. when I try to configure a smart folder, when I add a secound criteria, the first one gets invisible. But it is still present, because the filter I configured first is working! See what I mean. I select 8 Filters, only 4 are visible but all 8 are used for the folder. Even after a save and edit of the folder only 4 of them are visible:

Fixed in version 3.5.4

Thanks for the Fix!


TensorFlow Android 1.2 - Adds READ_PHONE_STATE permission

I am building with:
compile 'org.tensorflow:tensorflow-android:1.2.0'
My app was rejected by Play Store due to having the READ_PHONE_STATE permission and no privacy policy. However I don't have this permission anywhere in my app, and tracked it down to TensorFlow adding it.
Judging by this thread, I am not alone:
https://stackoverflow.com/questions/44864081/tensorflow-android-library-asking-for-read-phone-state

@meavydev If you add targetSdkVersion = 23 to your build.gradle and/or minSdkVersion = 19, does this solve the issue? READ_PHONE_STATE is automatically requested by the application target/minSdkVersion are < 4. We set this in AndroidManifest.xml, but only compileSdkVersion in the build.gradle, in case you've copied that for your app.
If you only get this issue when including the TF AAR, then we should probably figure out how to add it to the AAR build as well.

I am only using the AAR, not building from source.
I am building with target SDK set to 24 and build tools on 24, due to a tool that only works with that setup.
My minSDK is set to 16.
Given it sounds like it's an AAR issue, I can try a build increasing the minSDK, but it was running at this level for some AR glasses.

It has been 14 days with no activity and this issue has an assignee.Please update the label and/or status accordingly.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @andrewharp: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @andrewharp: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @andrewharp: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @andrewharp: It has been 16 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @andrewharp: It has been 277 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @andrewharp: It has been 292 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @andrewharp: It has been 307 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I guess this just nees a min-sdk like #20454

Nagging Assignee @andrewharp: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@PaulWoitaschek I'm having this issue. And users are wondering why I need this permission (I don't). Any update on this?

My PR was merged and the next update should fix this.

This should be fixed by b148d228886e53f71401c332b8d48d45467dad47

Thanks for the update. The workaround for now is to add this to your manifest:
<uses-permission
        android:name="android.permission.READ_PHONE_STATE"
        tools:node="remove" />

Nagging Assignee @andrewharp: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

While checking why my app required READ_PHONE_STATE I found out some things I want to share:

crobertsbmw is absolutely right, the workaround is very helpful!
the fix mentioned by PaulWoitaschek was merged before release of 1.10.0 lib, which still does NOT contain the fix and asks for the permission in question. (I assume a merge readded the code snippet)
the actual master commit bf23e5f does again contain the fix and does not require the READ_PHONE_STATE permission

I hope the permission is fixed by next release of tensorflow-lite lib 1.11.0

I don't understand what happened here:

I don't understand what happened here:
https://github.com/tensorflow/tensorflow/commits/master/tensorflow/contrib/lite/java/AndroidManifest.xml
602b03d5fb521aef6561a5c131075d915907357e My commit from Jul 16, 2018
b148d228886e53f71401c332b8d48d45467dad47 jdduke authored and tensorflower-gardener committed 23 days ago
Two consecutive commits added the same thing to the same file? What am I missing?

That was a thing I ran into as well.
There are different folders with different manifests. Your commit downgraded the targetSdk for the Android TFLite sample app, as the jdduke commit added the sdk configuration to tensorflow/contrib/android/cmake/src/main/AndroidManifest.xml, which I assume to be the source for building the library.
The library manifest got fixed once, but lost the sdk configuration in a commit, somewhere before building version 1.10.0.
Somebody then noticed the mistake, as it is fixed for the mentioned master commit.

Nagging Assignee @andrewharp: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

1.11.0 is released but not to jcenter. Can you please push the update to jcenter too?

@PaulWoitaschek can you please check whether the update has been pushed to jcenter?

https://bintray.com/google/tensorflow/tensorflow-lite
Only 1.10.0

@wt-huang pingpong

@PaulWoitaschek thanks for the confirmation. Any ETA as to when the updated 1.11.0 will be pushed to jcenter?

I am not involved into that, I just contributed the fix.

@PaulWoitaschek @meavydev Closing this issue for now as the fix is in for 1.10.0 and 1.11.0 will be pushed to jcenter when the release is ready. Feel free to open a new issue if any error comes up.

But 1.10.1?


Fix: 1023 Misaligned shipping

Fixed 1023
Fixed this by fixing the price column to 100px, setting a max width on the free shipping column and clamping both to 4 to match the title/description box.
From @rmisio : Another optimization we could do in the future if we see a problem is progressively decrease the font of the price as the number of digits increase. So, just dummy numbers, but… if the price reaches 15 characters, we drop the font a little, if it reaches 20, we drop it some more. I’ve done that in some other places. Not sure it’s really necessary here at this point though. We could keep that card up our sleeve.

Don't forget to put
Closes #1023
in the first comment so the issue will be closed when the PR is merged.

@rmisio can you recheck your review and approve if the changes you requested look good?


Response header returned from getObject

Closes #1106
getObject returns the response headers which is exposed in Core.
func (c Core) GetObject(bucketName, objectName string, opts GetObjectOptions) (io.ReadCloser, ObjectInfo, http.Header, error) {
	return c.GetObjectWithContext(context.Background(), bucketName, objectName, opts)
}

The response headers are not exposed for client.
i.e. No change in the below code.
// GetObject - returns an seekable, readable object.
func (c Client) GetObject(bucketName, objectName string, opts GetObjectOptions) (*Object, error) {
	return c.getObjectWithContext(context.Background(), bucketName, objectName, opts)
}

The travis is failing here stating.
# github.com/minio/minio/cmd/gateway/s3
../../../../pkg/mod/github.com/minio/minio@v0.0.0-20190926060824-ff726969aa89/cmd/gateway/s3/gateway-s3.go:445:17: assignment mismatch: 3 variables but l.Client.GetObject returns 4 values
The command "go get github.com/minio/minio" failed and exited with 2 during .

The minio-go travis file does a - go get github.com/minio/minio.  Do i need to have a minio PR to fix this ? If yes then which PR minio or minio-go needs to be merged first .
@harshavardhana  please suggest.

what needs to be set is GO111MODULE=on in the .travis.yml such that MinIO server builds with the correct modules. That will work in this situation

@poornas @harshavardhana PTAL


it can't select all words in search box

Issue Type: Feature Request
When I edit an file, I use ctrl + f to search something, but i can't use ctrl + A or mouse to select all words in the search box
VS Code version: Code 1.43.2 (0ba0ca52957102ca3527cf479571617f0de6ed50, 2020-03-24T07:38:38.248Z)
OS version: Windows_NT x64 10.0.18363

Can't reproduce. Try adding screenshots

I can't reproduce this issue and since there is no more information provided, I'll close it for now.


quarterly_financials methods do not get same quarters of last year

Yahoo finance homepages shows 5 quarters of financial information.

But yfinance's financials methods show only 4 quarters information and didn't get same quarters of last year.

@jyh2378 if you haven't resolved this one already. Just wanted to flag that the TTM is Trailing Twelve Months, it's not quarterly information. You can use my pull request (https://github.com/ranaroussi/yfinance/pull/776) to grab this detail from the income statement. Note that the functions are slightly different in my PR, so just run the two lines below (it's all in the README):
msft = yf.Ticker("MSFT")
msft.income_statement
This will bring back those TTM figures. Furthermore, the PR also makes sure that no critical information from the income statement is missed. For example if you pull the income statement using the current version of yfinance for WFC you'll miss out on a lot of the detail (e.g. special charges), the PR I have raised ensures all of this is captured.

Fixed in yfinance 0.2


java.lang.Exception at IOClient.send(io_client.dart:62)

Steps to Reproduce

Run flutter create bug.
Update the files as follows:

import 'package:http/http.dart' as http;
String agent = "myApplication";
String code = "MySecretApiToken";
String username = "Username";
String password = "Password";

var headers = {
    'Content-Type': 'application/x-www-form-urlencoded; charset=utf-8',
    'User-Agent': '$agent',
};

var data = 'institute_code=$code&userName=$user&password=$pass&grant_type=password&client_id=xxxxxxxxx';
var response = await http.post(
              'https://$code.website.hu/idp/api/v1/Token',
              headers: headers,
              body: data);

Also add these to AndroidManifest:
<uses-permission android:name="android.permission.INTERNET" />
<uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED" />
<uses-permission android:name="android.permission.WAKE_LOCK" />
<service
    android:name="io.flutter.plugins.androidalarmmanager.AlarmService"
    android:permission="android.permission.BIND_JOB_SERVICE"
    android:exported="false"/>
<receiver
    android:name="io.flutter.plugins.androidalarmmanager.AlarmBroadcastReceiver"
    android:exported="false"/>
<receiver
    android:name="io.flutter.plugins.androidalarmmanager.RebootBroadcastReceiver"
    android:enabled="false">
    <intent-filter>
        <action android:name="android.intent.action.BOOT_COMPLETED"></action>
    </intent-filter>
</receiver>

Running this code works exactly as expected every time I run it, but fails to run when used in a background isolate with android_alarm manager nearly every time.

//Used in a background isolate like this
await AndroidAlarmManager.periodic(
        Duration(minutes: 60),
        0, //Id
        backgroundFetchHelper.backgroundFetch, //The function being called
        wakeup: true,
        rescheduleOnReboot: true,
 );

Expected results:
Get the data I'm trying to HTTP post.
Actual results:
Get the following error stack traces or similar ones:
Non-fatal Exception: java.lang.Exception
       at IOClient.send(io_client.dart:62)
       at BaseClient._sendUnstreamed(base_client.dart:176)
       at ._withClient(http.dart:166)
       at NetworkHelper.getToken(networkHelper.dart:45)
       at .backgroundFetch(backgroundFetchHelper.dart:113)

Error seen in crashlytics:
Non-fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: . Error thrown getToken.
       at IOClient.send(io_client.dart:62)
       at BaseClient._sendUnstreamed(base_client.dart:91)
       at NetworkHelper.getToken(networkHelper.dart:45)
       at .backgroundFetch(backgroundFetchHelper.dart:113)
       at .suspension>(<asynchronous)

Or sometimes with a call to _.withClient
Non-fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: . Error thrown getToken.
       at IOClient.send(io_client.dart:62)
       at BaseClient._sendUnstreamed(base_client.dart:91)
       at ._withClient(http.dart:166)
       at NetworkHelper.getToken(networkHelper.dart:45)
       at .backgroundFetch(backgroundFetchHelper.dart:113)
       at .suspension>(<asynchronous)

Logs

Couldn't reproduce the error in the emulator

D:\novyNaplo>flutter analyze
Analyzing novyNaplo...
No issues found! (ran in 5.1s)

[√] Flutter (Channel stable, 1.22.2, on Microsoft Windows [Version 10.0.19041.572], locale hu-HU)
    • Flutter version 1.22.2 at E:\FLUTTER\flutter
    • Framework revision 84f3d28555 (3 weeks ago), 2020-10-15 16:26:19 -0700
    • Engine revision b8752bbfff
    • Dart version 2.10.2

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at C:\Users\Novot\AppData\Local\Android\Sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = C:\Users\Novot\AppData\Local\Android\Sdk
    • ANDROID_SDK_ROOT = C:\Users\Novot\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Android Studio (version 4.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 48.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code (version 1.50.1)
    • VS Code at C:\Users\Novot\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.16.0

[!] Connected device
    ! No devices available

! Doctor found issues in 1 category.

@Legolaszstudio
Please provide a complete code sample that we can copy paste and run in IDE to see the issue. I used your code sample, but its complaining on $user

Sorry, I was in a hurry yesterday when I wrote the issue ticket.
So this is how the updated code should look like:
import 'package:http/http.dart' as http;
import 'package:android_alarm_manager/android_alarm_manager.dart';

String agent = "myApplication";
String code = "MySecretApiToken";
String user = "Username";
String pass = "Password";

void backgroundFetch() async {
  //this same as NetworkHelper.getToken
  var headers = {
    'Content-Type': 'application/x-www-form-urlencoded; charset=utf-8',
    'User-Agent': '$agent',
  };

  var data =
      'institute_code=$code&userName=$user&password=$pass&grant_type=password&client_id=xxxxxxxxx';
  //Error occours on the next line
  var response = await http.post('https://$code.website.hu/idp/api/v1/Token',
      headers: headers, body: data);
}

//How it looks when used with android_alarm_manager
void main() async {
  await AndroidAlarmManager.periodic(
    Duration(minutes: 60),
    0, //Id
    backgroundFetch, //The function being called
    wakeup: true,
    rescheduleOnReboot: true,
  );
}

I changed the API link because I do not want to expose my credentials if it is maybe a server-side error I can find a way to share it with you, if needed.
Also, these dependencies need to be added to pubspec.yaml:
dependencies:
  flutter:
    sdk: flutter
  http: any
  android_alarm_manager: any

And as I already mentioned this needs to be added to AndroidManifest.xml:
<uses-permission android:name="android.permission.INTERNET" />
<uses-permission android:name="android.permission.RECEIVE_BOOT_COMPLETED" />
<uses-permission android:name="android.permission.WAKE_LOCK" />
<service
    android:name="io.flutter.plugins.androidalarmmanager.AlarmService"
    android:permission="android.permission.BIND_JOB_SERVICE"
    android:exported="false"/>
<receiver
    android:name="io.flutter.plugins.androidalarmmanager.AlarmBroadcastReceiver"
    android:exported="false"/>
<receiver
    android:name="io.flutter.plugins.androidalarmmanager.RebootBroadcastReceiver"
    android:enabled="false">
    <intent-filter>
        <action android:name="android.intent.action.BOOT_COMPLETED"></action>
    </intent-filter>
</receiver>

I already reported the bug on the HTTP package site, but got redirected here:
https://github.com/dart-lang/http/issues/421

@Legolaszstudio
I ran the updated code sample but see a blank screen and nothing in the logs. How to replicate the issue ? Are there any specific steps ?

Strange, yes the code has no UI elements. But for me running my included code snippet is enough to occur every 60 minutes as I set it in the alarm manager, just simply rewrite the duration:
void main() async {
  await AndroidAlarmManager.periodic(
    Duration(minutes: 1), //Change this number to a lower value, for the code to run more frequently.....
    0, //Id
    backgroundFetch, //The function being called
    wakeup: true,
    rescheduleOnReboot: true,
  );
}

@Legolaszstudio
After 1 minute, I am getting below exception which is different from what you mentioned (although a crash still):
I/FlutterBackgroundExecutor(31914): Starting AlarmService...
I/AndroidAlarmManagerPlugin(31914): onAttachedToEngine
D/AndroidRuntime(31914): Shutting down VM
E/AndroidRuntime(31914): FATAL EXCEPTION: main
E/AndroidRuntime(31914): Process: com.example.webtest, PID: 31914
E/AndroidRuntime(31914): java.lang.RuntimeException: Unable to create service io.flutter.plugins.androidalarmmanager.AlarmService: java.lang.NullPointerException: Attempt to read from field 'java.lang.String io.flutter.view.FlutterCallbackInformation.callbackLibraryPath' on a null object reference
E/AndroidRuntime(31914): 	at android.app.ActivityThread.handleCreateService(ActivityThread.java:3365)
E/AndroidRuntime(31914): 	at android.app.ActivityThread.-wrap4(Unknown Source:0)
E/AndroidRuntime(31914): 	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1683)
E/AndroidRuntime(31914): 	at android.os.Handler.dispatchMessage(Handler.java:106)
E/AndroidRuntime(31914): 	at android.os.Looper.loop(Looper.java:164)
E/AndroidRuntime(31914): 	at android.app.ActivityThread.main(ActivityThread.java:6543)
E/AndroidRuntime(31914): 	at java.lang.reflect.Method.invoke(Native Method)
E/AndroidRuntime(31914): 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:440)
E/AndroidRuntime(31914): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:810)
E/AndroidRuntime(31914): Caused by: java.lang.NullPointerException: Attempt to read from field 'java.lang.String io.flutter.view.FlutterCallbackInformation.callbackLibraryPath' on a null object reference
E/AndroidRuntime(31914): 	at io.flutter.embedding.engine.dart.DartExecutor$DartCallback.toString(DartExecutor.java:336)
E/AndroidRuntime(31914): 	at java.lang.String.valueOf(String.java:2827)
E/AndroidRuntime(31914): 	at java.lang.StringBuilder.append(StringBuilder.java:132)
E/AndroidRuntime(31914): 	at io.flutter.embedding.engine.dart.DartExecutor.executeDartCallback(DartExecutor.java:142)
E/AndroidRuntime(31914): 	at io.flutter.plugins.androidalarmmanager.FlutterBackgroundExecutor.startBackgroundIsolate(FlutterBackgroundExecutor.java:173)
E/AndroidRuntime(31914): 	at io.flutter.plugins.androidalarmmanager.FlutterBackgroundExecutor.startBackgroundIsolate(FlutterBackgroundExecutor.java:126)
E/AndroidRuntime(31914): 	at io.flutter.plugins.androidalarmmanager.AlarmService.onCreate(AlarmService.java:349)
E/AndroidRuntime(31914): 	at android.app.ActivityThread.handleCreateService(ActivityThread.java:3355)
E/AndroidRuntime(31914): 	... 8 more

This is an error regarding the embedding engine which is in the native kotlin/java code. If I remember correctly flutter already solved this issue, but for me, I had to change MainActivity.kt kotlin (android\app\src\main\kotlin) file to resolve the issue you are experiencing:
package YOURPACKAGENAME //Change it accordingly
import androidx.annotation.NonNull
import io.flutter.embedding.android.FlutterActivity
import io.flutter.embedding.engine.FlutterEngine
import io.flutter.plugins.GeneratedPluginRegistrant

//MainActivity automatically run by flutter
class MainActivity: FlutterActivity() {
  override fun configureFlutterEngine(@NonNull flutterEngine: FlutterEngine) {
    GeneratedPluginRegistrant.registerWith(flutterEngine);
  }
}

Also yo should have a android\app\src\main\java\io\flutter\plugins\GeneratedPluginRegistrant.java file if everything works normally

Also, I updated main.dart as it contained some mistakes:
import 'package:flutter/material.dart';
import 'package:http/http.dart' as http;
import 'package:android_alarm_manager/android_alarm_manager.dart';

String agent = "myApplication";
String code = "MySecretApiToken";
String user = "Username";
String pass = "Password";

void backgroundFetch() async {
  //this same as NetworkHelper.getToken
  var headers = {
    'Content-Type': 'application/x-www-form-urlencoded; charset=utf-8',
    'User-Agent': '$agent',
  };

  var data =
      'institute_code=$code&userName=$user&password=$pass&grant_type=password&client_id=xxxxxxxxx';
  //Error occours on the next line
  var response = await http.post('https://$code.website.hu/idp/api/v1/Token',
      headers: headers, body: data);
}

//How it looks when used with android_alarm_manager
void main() async {
  WidgetsFlutterBinding.ensureInitialized();
  await AndroidAlarmManager.initialize();
  await AndroidAlarmManager.periodic(
    Duration(seconds: 15),
    0, //Id
    backgroundFetch, //The function being called
    wakeup: true,
    rescheduleOnReboot: true,
  );
}

But you need to change the API link to something working.

@Legolaszstudio
What api link should I use to replicate the issue ?

I was able to replicate the issue with any API link, like http://icanhazip.com/ which should just return your IP address.

With above link as-is, I am getting below:
[+11291 ms] E/flutter (12769): [ERROR:flutter/lib/ui/ui_dart_state.cc(177)] Unhandled Exception: Bad state: Insecure HTTP is not allowed by platform: http://icanhazip.com/
[        ] E/flutter (12769): #0      _HttpClient._openUrl (dart:_http/http_impl.dart:2435:7)

with https://icanhazip.com/, I get following:
[ +138 ms] I/AlarmService(10617): AlarmService started!
[+13702 ms] D/ViewRootImpl@100ae37[MainActivity](10617): ViewPostIme pointer 0
[  +93 ms] D/ViewRootImpl@100ae37[MainActivity](10617): ViewPostIme pointer 1
[+2691 ms] Service protocol connection closed.
[        ] Lost connection to device.

Strange, why would you lose connection to your device?
For me like 1 out of 10 times, it throws the exception, I'm trying to figure out

Tested on latest master with another device, and this time I see below:
Running "flutter pub get" in webtest...
Launching lib/main.dart on SM A260G in debug mode...
Running Gradle task 'assembleDebug'...
✓ Built build/app/outputs/flutter-apk/app-debug.apk.
Installing build/app/outputs/flutter-apk/app.apk...
Waiting for SM A260G to report its views...
Debug service listening on ws://<IP_ADDRESS>:55082/Lmk5bIdeMvw=/ws
Syncing files to device SM A260G...
I/FlutterBackgroundExecutor(24643): Starting AlarmService...
I/AndroidAlarmManagerPlugin(24643): onAttachedToEngine
I/AlarmService(24643): AlarmService started!

After waiting a bit you should get an exception after AlarmService started!, when the alarm-manager runs

Waited for 10 mins after AlarmService started, but didn't notice the exception you mentioned originally. I'd suggest you to revisit your implementation and see where and how does it throw the exception, whereas using same implementation, we were unable to replicate it.
I/FlutterBackgroundExecutor(26021): Starting AlarmService...
I/AndroidAlarmManagerPlugin(26021): onAttachedToEngine
I/AlarmService(26021): AlarmService started!
W/libEGL  (26021): EGLNativeWindowType 0xc7073808 disconnect failed
W/libEGL  (26021): EGLNativeWindowType 0xc7066008 disconnect failed
I/zygote  (26021): Do partial code cache collection, code=27KB, data=30KB
I/zygote  (26021): After code cache collection, code=27KB, data=30KB
I/zygote  (26021): Increasing code cache capacity to 128KB
D/mali_winsys(26021): EGLint new_window_surface(egl_winsys_display *, void *, EGLSurface, EGLConfig, egl_winsys_surface **, egl_color_buffer_format *, EGLBoolean) returns 0x3000,  [540x960]-format:1
D/mali_winsys(26021): EGLint new_window_surface(egl_winsys_display *, void *, EGLSurface, EGLConfig, egl_winsys_surface **, egl_color_buffer_format *, EGLBoolean) returns 0x3000,  [540x888]-format:1
E/BpSurfaceComposerClient(26021): Failed to transact (-1)
E/BpSurfaceComposerClient(26021): Failed to transact (-1)

flutter doctor -v

[✓] Flutter (Channel stable, 1.22.3, on Mac OS X 10.15.4 19E2269, locale en-IN)
    • Flutter version 1.22.3 at /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 8874f21e79 (11 days ago), 2020-10-29 14:14:35 -0700
    • Engine revision a1440ca392
    • Dart version 2.10.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.50.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.15.1

[✓] Connected device (1 available)
    • SM A260G (mobile) • 5200763ebcfa861f • android-arm • Android 8.1.0 (API
      27)

• No issues found!

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Understanding Quantization results

💡 Your Question
Hi,
I am just checking, I see in the provided results that Yolo-NAS-L does not suffer much reduction in performance going to Yolo-NAS-INT8-L. Can I check what exactly is meant by Yolo-NAS-L and Yolo-NAS-INT8-L? Like is Yolo-NAS-L a model trained wtih quantization awareness, and then what happens is that it has some layers quantized and others not. Then Yolo-NAS-INT8-L is the results of converting that model through quantization, but the reduction in performance is negligible due to the quantization aware training that distributes the quantization in a certain way?
Also, what is the simplest way to do quantization aware training on my own dataset? Is it doable through a recipe or no?
Versions
No response

Yolo-NAS-INT8 variants are quantized with post-training-quantization.
We have added plenty of material on how to perform quantization in SG:
QAT and PTQ for YoloNAS
QAT and PTQ
Quantization aware training YoloNAS on custom dataset notebook

Also, can I clarify, suppose I go through PTQ and QAT and obtain:
/content/sg_checkpoints_dir/yolo_nas_s_soccer_players/yolo_nas_s_soccer_players_32x3x640x640_ptq.onnx and /content/sg_checkpoints_dir/yolo_nas_s_soccer_players/yolo_nas_s_soccer_players_32x3x640x640_qat.onnx".
When I convert these models to tensorrt, should I still be converting using --fp16 or --int8 flag? Like does that do anything, or does PTQ and QAT already implicitly do that.

Related to the above, when I quantize a model using PTQ and QAT as documented, the resulting ONNX model inputs and outputs are still float32. This inhibits use on NPUs that only support int based operations. Is it possible to get a resulting model that is full integer quantized? If so, can you please provide guidance on this process?


Pass Sales Header by reference in Event OnBeforeMessageIfSalesLinesExist - Table 36 - Sales Header

Describe the request
Current:
[IntegrationEvent(false, false)]
local procedure OnBeforeMessageIfSalesLinesExist(SalesHeader: Record "Sales Header"; ChangedFieldCaption: Text; var IsHandled: Boolean)
begin
end;

Should be:
[IntegrationEvent(false, false)]
local procedure OnBeforeMessageIfSalesLinesExist(var SalesHeader: Record "Sales Header"; ChangedFieldCaption: Text; var IsHandled: Boolean)
begin
end;

Additional context
Pass parameter SalesHeader: Record "Sales Header" as a reference in Event OnBeforeMessageIfSalesLinesExist - Table 36 - Sales Header It will allow modifying Sales Header by subscribing to the event. Since same subscriber in Purchase Header is already passed by reference.
Internal work item: AB#497695

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 15680.


README

README
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
User Acceptance
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

Created Readme providing user information about the project and usage to store lessons learned for the topics of HTML, CSS, Git and JavaScript.


[Suggestion] Show price for item

show item price with name on hover

I did say adding it to the item icon would make it too crowded, but i suppose it cant hurt on hover.
Need to know what it would look like with longer item names

Even with small text this doesn't really look very nice

Even with small text this doesn't really look very nice

try to reduce the distance between the lines

Added in dev


[PE-D][Tester B] add-patient command example

When the example command for add-patient is pasted, there is a error with the tags (should only contain alpha-numeric values)

Labels: type.DocumentationBug severity.Medium
original: jack1e0/ped#1

Fix UG. Specify constraints with tag being alphanumeric


Volume Slide and volumeLimit not working

Hi, neighbours and friends...
I have already seen that is a known "issue" that slide doesn't work when an external source is connected (SoudBar), and that's why the switches are for (kind of annoying...). But is it normal that the volumeLimit also doesn't work?
My config is:
{
"accessory": "webostv",
"name": "La tele",
"ip": "192.168.x.x",
"mac": "xx:xx:xx:xx:xx:xx",
"keyFile": "/home/pi/.homebridge/lgtvKeyFile",
"volumeControl": true,
"mediaControl": true,
"channelControl": true,
"volumeLimit": 50,
"pollingInterval": 10,
"appSwitch": [
"com.webos.app.livetv",
"com.webos.app.tvguide",
"youtube.leanback.v4",
"com.webos.app.hdmi1",
"com.webos.app.hdmi2",
"com.webos.app.hdmi3",
"com.webos.app.hdmi4",
"com.webos.app.externalinput.component",
"netflix",
"amazon",
"canal.yomvi"
],
"channelButtons": [1,2,3,4,5,6,8,9,10,11,12,13,14,15,16,17,18,19,20],
"notificationButtons": [
"User A arrived",
"User B arrived"
]
}
When the volumeLimit is reached (50), if I press volume up in the remote or in the "Home Switch", the slide keeps on going up, until it reaches 100.
I guess is the same "issue" as the 'not-working-slide-for-external-devices'. Am I right?
Thank you!

I have a home theater (external speaker connect thru HDMI ARC) connected to my webostv, the volume slider doesn't work when the external speaker is selected, otherwise the volume buttons work normal with external speaker. Is there a fix for this?


There is no incremental change in the initial script associated with the upgrade of the soul version

When I plan to upgrade the version, I will find an error reminder for missing SQL fields. Can I provide incremental SQL for version change to guide quick upgrade?

we will add sql_version file to help upgrade..

may i ask a question? if we do this, what version of sql file can be depended? in other way, after what version to add sql_version file ?

pls see https://github.com/apache/incubator-shenyu/pull/2526


Modularisation

Changed input parameter from userID to patientID (resolve #15 ).
Sensors end-points support the Time-Frame parameter (resolve #17 ).
Define the concept of Device Catalog using the configuration file device-catalog.yml (resolve #20 ).

Could we use userId or subjectId instead of patientId? Since it is an ID of user in our system, but patientId implies that it is a patient, which in many research studies is not the case. In OpenMHealth header, user_id is used. subjectId would also be fine in the context of a research study, although it would be less appropriate for self-organized data gathering.

This was discussed during a call with the Front-end meeting a few moths ago. I see your point: userId is more general and for this reason it was initially defined as userId.
@herkulano asked for this change.

@fnobilia @blootsvoets I agree with subjectId. userId can easily be mistaken or confused with the user of the applications, either in the dashboard or the questionnaire.
//cc @kerzmaximilian

Changed here RADAR-CNS/RADAR-RestApi@254acef


[Proposal] Update UseCases and Landscape document to include section on "Testing" workload specific challenges

Description
There is a specific type of workload that is related to the testing of features/applications that is separate to the workload of the feature/applications in production. The environmental foot print of this testing load could be as substantial as the application in production. The suite of possible actions to lessen the impacts from the testing load are different to the application in production and needs a separate discussion.
...
Impact
Update https://github.com/cncf/tag-env-sustainability/blob/main/website/content/about/SustainabilityUseCasesAndLandscape2023.md to discussthe impact of testing
...
Scope
1 day to make initial proposal

how long is a piece of string to get through review but the review discussion is a big reason to try make the update.
...

Additional info and comments
Mentioned this a long time ago on Slack and finally getting around to it
TODO

[ ] TAG Environmental Sustainability lead representative:
[ ] Project leader(s):
[ ] Project Members:
[ ] Fill in addition TODO items here so the project team and community can see progress!
[ ] Scope
[ ] Deliverable(s)
[ ] Project Schedule
[ ] Slack Channel (as needed)
[ ] Meeting Time & Day:
[ ] Meeting Notes (link)
[ ] Meeting Details (zoom or hangouts link)
[ ] Retrospective

cc @catblade

I'm all for it.  I'll support this!

@by-d-sign if you find a min could you take a look at this issue since its connected to your recent work rewriting the landscape document. @andrew-est would be great to get your input here https://github.com/cncf/tag-env-sustainability/issues/302 🙌

@leonardpahlke  can we split these out please into 2, and put one for use cases to "in progress".
Separate one for landscape.

@leonardpahlke can we split these out please into 2,

I get from the issue description "Usecases and landscape document" refers to this https://tag-env-sustainability.cncf.io/landscape/ - so this issue talks about just this doc.


Support for self hosted gitlab

Add ability to specify url to gitlab instance for self hosted gitlab.

Add ability to specify url to gitlab instance for self hosted gitlab.

environment variable "GIT_HOST" added for self hosted gitlab. it takes a valid URL. please refer readme.


Pagination in StarWars API

You made a nice sample using MVI. The Search history feature is quite interesting. Thank you for that, But as far as I know StarWars API supports pagination. Did you use Paging library to display search items?

Hi, thank you 🙏🏾.
I didn't implement lazy loading kind of pagination like the paging library does.
I used recursion to pull all the data in one call. Yea it doesn't really scale but was sorta fun

You made a nice sample using MVI. The Search history feature is quite interesting. Thank you for that, But as far as I know StarWars API supports pagination. Did you use Paging library to display search items?

Hi, thank you 🙏🏾.
I didn't implement lazy loading kind of pagination like the paging library does.
I used recursion to pull all the data in one call. Yea it doesn't scale but it was sorta fun 😸

Considering that StarWars API does not need to load many items and it is always limited to a few characters, I agree that you had a nice idea here. Thank you for sharing.


Assignment 5 - Redo Pull Request (headSimulatorTwo)

We have:

Updated the Blackboard
Moved some of the used classes into the library (we created our subdirectory within the library module, per your recommendation)
Removed Websockets
Cleaned up comments
Refactored other files

Do not commit the target folder (class files).

Maybe the README.md that you are uploading should be inside your Module otherwise explain the update

As of the latest commits, there is no 'target' folder (they've been deleted)
The README changes were accidentally applied when using IntelliJ's refactoring tool. The names of some classes changed in the README file, which has since been reset.

Double check. I still see them. I just deleted the README.md from your submission and a *.lck.
But there are several others that should not be there.


Update carfiles to point to mm forks

Forked all dependencies to modmed space. Update cartfile dependecies

We won't update the dependencies to point to modmed forked versions for now


CSS Containment Firefox 69

See https://trello.com/c/AQOpkXpT/3-css-containment-fx-69 for original trello card
Acceptance criteria

[ ] Enable pref for CSS Containment (layout.css.contain.enabled)

[ ] Investigate the other items on the trello card to see if anything else needs doing

[ ] Write short tutorial covering using containment?

Slightly higher points estimate here thhan you might expect, because I think maybe we need to investigate the oter bugs on the trello card, see if anything else needs doing, maybe write a short tutorial about containment?

@chrisdavidmills I've been poking around this to try and figure out the status  and I note that here it is listed as being Firefox 70 https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Experimental_features#contain-property.
There is also another ticket for the work open #1555
Other things on the Trello card which relate to containment are as yet open, and not flagged for Fx 69 or 70.
So, I'm a bit confused as to what is actually shipping. Is there some more information anywhere I'm missing?

Ah, looks like this one slipped to 70 since I put it in our sprint system!
I'll close this one, and re-add it when we get round to Fx70


update ADOPTERS.md

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

thanks @jazzsir !
/lgtm
/approve


Fix map wrapping

Description

This fixes map wrapping to be dynamic for any map size instead of being a hard-coded value. This update has an updated version of the Core dll which includes an update that makes the RftVector.Map variable a static value, as well as updates the MapGeneration class to calculate the generated map size.
NOTE: This update includes the updated networking PR within the dll from the core repository. Recommend reviewing the networking udpate and approving it first.
Versioning
Current Version Number: N/A
Next Version Number: N/A

[ ] I have updated the README with the next version number
[ ] I have updated appropraite strings in the app with the new version number.

Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] New dependencies/packages

Testing

[ ] I have created automated tests for any new features
[ ] All automated tests pass
[X] I have performed manual tests

The map smoothly updates based on any number of players within the game. We may update map generation again in the future to not generate based on a rectangle and that is fine, this just fixes the wrapping to remove hard coded values.
Checklist:

[X] I have performed a self-review of my own code
[X] I have commented my code
[ ] I have made corresponding changes to the README (if required)
[X] My changes generate no new warnings and do not break existing endpoints

Questions / Discussion

This PR now includes the PR https://github.com/Subterfuge-Revived/Remake-Core/pull/25 from Core.

Map wrapping has been merged into the core repository's master and building the dll in master of core is throwing errors in the master branch of the unity repo. I'm merging this so that they are both in sync.


Use dotnet SDK 4.8

In this way, we can create the images on Windows server updated with the February 2020 patch.
Fixes https://github.com/adamrehn/ue4-docker/issues/73

Have you tested this change for both source builds (ue4-engine) and Installed Builds (ue4-full) of all of the versions of the Unreal Engine that ue4-docker is compatible with (4.19.2, 4.20.3, 4.21.2, 4.22.3, 4.23.1, 4.24.3) to ensure that it works correctly for all of them and does not break backwards compatibility with the older Engine versions?

I don't have the required hardware to test all of them (it takes really a lot of time and storage building windows containers), I'm sorry. I tested with 4.23.1 and 4.24.3 and in both cases, I was able to build ue4-engine and ue4-full images.

Yes, it certainly takes quite a lot of CPU horsepower and disk space, which is why updates to the Windows container images always take me much longer to validate than the Linux images. I'll test out each of those configurations myself over the coming week or so to make sure everything works before I merge this pull request.

Thank you very much for your efforts in maintaining this project :+1:

After running test builds pretty much around the clock for the past few days, I've managed to verify that this change is compatible with all six of the currently supported versions of the Unreal Engine and that everything builds happily.


Fixed #32265, Refs #32355 -- Removed unnecessary ServerHandler.handle_error().

ConnectionAbortedError, BrokenPipeError, ConnectionResetError raised from SocketServer.BaseServer.finish_request() are already suppressed by wsgiref.handlers.BaseHandler.run() in Python 3.7+, see https://github.com/python/cpython/commit/47ffc1a9f6fab1c17cdcc325d4af066317369ed7.
ticket-32265

Thanks for checking :+1:

Thanks for checking :+1:


Data descrepancies

Invalid release year 3003 for Lost in Translation it should be 2003

Yes it has been fixed in release 1.0.0


two factor doc

Need to move these to the new content folder. I'll merge development into the PR

@jtwaleson please approve review

@jtwaleson done ready for merge


refactor(pure_pursuit): rework parameters

Description
Implement the ROS Node configuration layout described in https://github.com/orgs/autowarefoundation/discussions/3371
Tests performed
Package built and launch locally.
Effects on system behavior
More reliable and faster parameter configuration file creation.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[x] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

I was testing this locally by running the local launch file, but there seems to be a issue with package names.
They are fixed in this PR.
https://github.com/autowarefoundation/autoware.universe/pull/8687/files

I was testing this locally by running the local launch file, but there seems to be a issue with package names. They are fixed in this PR. https://github.com/autowarefoundation/autoware.universe/pull/8687/files

Thank you for your feedback


Allow more types for "picks"

Describe the new feature or enhancement
I'm not exactly sure if a bug or feature request, so I'll go with my gut:
Most methods that have picks as argument accept a list. They also accept a tuple even if not listed in the docs (bug?). However, most of the times I want to provide a set or a frozenset, since I want to make sure that there are no repetitions at any point (e.g., no two "Pz" by mistake), and their order is irrelevant. For such case, a set might be the most correct native data structure. Giving a set or a frozenset as picks will raise an exception TypeError: picks must be a list of int or list of str, got a data type of object
Describe your proposed implementation
Relax current type restrictions for picks to at least allow sets and frozensets.
Describe possible alternatives
Document better why this is not possible even if a priori intuitive.
Thanks!

However, most of the times I want to provide a set or a frozenset, since I want to make sure that there are no repetitions at any point (e.g., no two "Pz" by mistake), and their order is irrelevant.

Careful, the order does matter, assuming you want to preserve the original order in the data:
>>> raw = mne.io.read_raw_fif(mne.datasets.sample.data_path() / 'MEG' / 'sample' / 'sample_audvis_raw.fif')
>>> raw.copy().pick([0, 1]).ch_names[0]
'MEG 0113'
>>> raw.copy().pick([1, 0]).ch_names[0]
'MEG 0112'

set in 3.7+ does preserve order, though, so this might be less important nowadays.

Relax current type restrictions for picks to at least allow sets and frozensets.

I would rather have users pass list(picks) instead. I think it makes clearer that it's really supposed to be array-like

Thanks for the reply. True, there are cases in which the order does matter. I was looking mainly at Epochs.average. Then I see several ways to proceed:

Allow to pass a set and call list(picks) internally, since there's no reason a priori reason why a set could not be passed.
Allow to pass a set and call list(picks) internally, but issue a runtime warning about the ordering issues.

I still think that sets should be allowed and that 1. is the better option. It still sounds a bit weird to me to impose this constraint. If I'm implementing e.g, logic to select channels via set intersections, among others, and I pass the set to MNE, it's an unexpected behavior that it raises an error. Sure someone can make a mistake regarding order in <3.7, but I think it's reasonable to think that it's the user the one that should be careful in this case.
Also, independently of above, should the documentation say array-like instead of list then?

It still sounds a bit weird to me to impose this constraint. If I'm implementing e.g, logic to select channels via set intersections, among others, and I pass the set to MNE, it's an unexpected behavior that it raises an error.

One problem is that we actually do treat set-like differently in MNE. In pick_channels, we treat it as a set, and the order there does not matter, at least by default.
Another problem is that a set is not array-like in the same way as list and tuple:
>>> np.array(list([1, 2]))
array([1, 2])
>>> np.array(tuple([1, 2]))
array([1, 2])
>>> np.array(set([1, 2]))
array({1, 2}, dtype=object)

So I think we shouldn't change behavior, and instead just update it to say array-like of int

Interesting, so for methods such as pick_channels and Epochs.average, should they accept both "array-like" and a "set-like"? Then update the docs to reflect these changes.

OK that can work then. Maybe this can be closed then. Thanks!

it seems we should close after amending the docstrings as suggested by
@larsoner
Message ID: @.***>

I'll have a look at changing the docstrings as suggested above.


non-ipynotebook version

Need instrucion to install native Linux OS

bash script should be able to specify where the file is.
This wold be difficult from inside the docker.  But it should work if (1) no docker at all or (2) use docker, but not from inside the container, use it as as a postgis server.

This is pretty much done, but i made changes here and there in the main code in code_anaconda.  So need to make sure that notebook part is not broken.  Once that's that, this nrt branch will be marged into master.

#36


Don't estimate sketch batch size when rmm is used.

Codecov Report

Merging #6807 (1ca0dc3) into master (a59c732) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6807      +/-   ##
==========================================
- Coverage   81.78%   81.75%   -0.03%
==========================================
  Files          13       13
  Lines        3848     3848
==========================================
- Hits         3147     3146       -1
- Misses        701      702       +1

Impacted Files
Coverage Δ

python-package/xgboost/dask.py
81.76% <0.00%> (-0.14%)
: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 a59c732...1ca0dc3. Read the comment docs.


Allow more control of the status bar text for a LanguageStatusItem

I'm trying to adopt the LanguageStatusItem API for some things that were previously just standard status bar entries, however the way the text is rendering makes it a much worse experience than before even if when items are pinned.
For example, in Flutter we currently show the selected device on the status bar:

There's a tooltip that tells you how many devices are connected. It isn't prefixed with "Flutter" because that consumers more space.
Using the new API, it's natural to set text to something like "Flutter Device" and detail to the device name:

However if you want the original behaviour and you pin the item, you just get "Flutter Device" and not what you really wanted:

And if we just set text to the device name, then the main flyout doesn't look so good because it doesn't have a description of what it is.
It would be nice if we could control the status bar text separately, otherwise we have to decide between a better experience, or users being able to collapse the item into the flyout. Right now for Flutter devices it feels like status bars are better (even though I'd like for users to be able to move it into the flyout).
Slightly related (and I can raise separate issues if worthy), things I'd also love:

Add tooltips
The ability for text/detail to be display like columns rather than just joined with a hyphen. The current layout in my screenshot above looks a bit messy and I think would look better if the detail fields were all lined up.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

While talking to extension authors that have adopted to the language status the feedback that I usually got is that some functionality is missing. Since most of the extensions have now shipped the language status bar, I would like us to revisit this, and see if you all feel like something is missing still.
So can you please share what exactly you are trying to achieve that you can not with the current language status bar? Based on each scenario we can decide if that truly belongs to the status bar and if we need to add functionality so you can all achieve what you want. Thanks all 🙏
@DanTup @bobbrow @AlexandraKemperMS @luabud @brettcannon @akaroml @maddymontaquila

Thanks all. This is very helpful. Let's assume we add these two capabilities from @bobbrow comment above.

What is the exact text that you would display when pinned vs what text when not pinned?
What items would you pin by default?

Specific examples would really help!

What is the exact text that you would display when pinned vs what text when not pinned?

For me, I want to show the name of the current device when pinned ("Pixel 4a (android)") and show a label alongside it when in the popup menu ("Flutter Device - Pixel 4a (android) - [Change]"). Today, the first label is used in the status bar so we either don't get "Flutter Device" in the popup menu, or we get "Flutter Device" in the pinned status bar).

What items would you pin by default?

Just the Flutter device for me. Everything else is already in the popup. It's very convenient to be able to glance and see what device is selected before you hit F5 to run.

For us, we generally want descriptions in the flyout and icons or something less verbose pinned in the status bar.  Right now we have to put both in there because we don't know what the context is.

We would pin the "Win32" (active configuration) item in the status bar.  In the flyout we would be able to have the description visible as well.

@isidorn from the perspective of Java extension, the main issue for us is that it only shows the status view when the relevant document is opened in the editor (e.g., a Java file or a build file such as pom.xml/build.gradle is opened). If no relevant file is open, the status view is hidden. This can make users unaware of the Java extension’s activity in some scenarios. For instance, when users open a Java project, Java extension automatically imports the project, downloads the dependencies and builds the workspace. However, if users do not open any Java file, they cannot see the background activity of the Java extension.
To enhance the user experience, we have two requests:

Allow extension to set the status to be pinned or unpinned by default.
The pinned status should always be visible in the status bar, regardless of the current open editor.

We have a general icon 👍 to indicate whether the Java language server is running. We would like to make it pinned by default if the requests above are supported.

If no relevant file is open, the status view is hidden

This bugs me slightly too. I think in part it's because the design is that this is "language status" so it's attached to the language status item, but some of the things we end up putting in it are "project-wide". For example in Dart we show the language server status and for Flutter we show the select device. These are really Dart/Flutter project items, not specific to a file or the language of an open file. It seems odd that they become inaccessible if you don't have any files open (even though you have a Dart/Flutter project open and things like the LSP server are running even without any open files).

the main issue for us is that it only shows the status view when the relevant document is opened in the editor (e.g., a Java file or a build file such as pom.xml/build.gradle is opened). If no relevant file is open, the status view is hidden.

That's the major objective for introducing the language status API and something we have not planned to change. The rule is that all project specific things should go left and editor specific things go right. Long running init-operations should use the Window progress location.

Using the new API, it's natural to set text to something like "Flutter Device" and detail to the device name:

We are listening but I have trouble understanding why it is such a big deal to then use the API the "other way" around, e.g use device name for text and detail for the descriptive part. The hyphen might not be the best UI but the arrangement of "<primary_label> <secondary_label>" isn't novel, e.g quick pick puts the "important" part before the detail part. So, we could experiment with other arrangements/layouts here but I am a bit reluctant to add a new API for 'pinned text' because 'text' is kinda meant for that.

The hyphen might not be the best UI but the arrangement of "<primary_label> <secondary_label>" isn't novel, e.g quick pick puts the "important" part before the detail part.

Ah sorry, I never realised it was intended to be this way. The names text and detail felt to me like "label" and "value" so I had assumed it was intended to be the way I had it. However, I now see that TypeScript has it the other way:

So if this is how it's expected to be used, I will swap Dart around for consistency (although FWIW I do think it looked better the other way).
(apologies for not seeing and responding to this before https://github.com/microsoft/vscode/issues/192880 - the proposal there makes more sense now I've seen this)

the main issue for us is that it only shows the status view when the relevant document is opened in the editor (e.g., a Java file or a build file such as pom.xml/build.gradle is opened). If no relevant file is open, the status view is hidden.

That's the major objective for introducing the language status API and something we have not planned to change. The rule is that all project specific things should go left and editor specific things go right. Long running init-operations should use the Window progress location.

Great point. It's super helpful to clarify what fits and doesn't. This picture shows three use cases.

Show Build Status is the visualization of the long-running backend build task status. The status bar is the ideal place because we want to help users understand that the server is busy building and set the right expectations that it may not respond to requests timely.
Open Config File is the project-level info (not specific to the active editor). So it's supposed to be pinned to the status bar by default. But the same file is accessible from many entries (File Explore, Maven/Gradle Explorer, Project Viewer). So maybe we don't need another entry.
Configure Java Runtime also looks like project-level info, but it also has a close relationship with the active editor. So to me, it's a mix. The JDK configuration has been one of the most complained areas and deserves high visibility.

Another thought is that the physical appearances of different visualizations result in different visibilities. Something that appears straight in the status bar gets more attention than the ones folded into the language status. So we can consider the attention level we want to have when deciding. @testforstephen


cargo install cargo-bpf fails with linking issue

After installing llvm 9 and associated deps on ubuntu 18 I cannot seem to get cargo install to successfully install cargo-bpf. I've tried both default and gcc and both result in the same error of:
error: linking with `gcc` failed: exit code: 1

with a note about ld -lz not being found
  = note: /usr/bin/ld: cannot find -lz
          collect2: error: ld returned 1 exit status

Any ideas? Not sure if there is specific version information which might be helpful but am running latest stable rust / cargo.

Running install with --verbose there is a bit more detail
Caused by:
  could not compile `cargo-bpf`.

Caused by:
  process didn't exit successfully: `rustc --edition=2018 --crate-name cargo_bpf /root/.cargo/registry/src/github.com-1ecc6299db9ec823/cargo-bpf-0.9.10/src/main.rs --error-format=json --json=diagnostic-rendered-ansi --crate-type bin --emit=dep-info,link -C opt-level=3 -C metadata=f5152c83bb164c5e -C extra-filename=-f5152c83bb164c5e --out-dir /tmp/cargo-installupv74r/release/deps -C linker=gcc -L dependency=/tmp/cargo-installupv74r/release/deps --extern bindgen=/tmp/cargo-installupv74r/release/deps/libbindgen-8b243bb3148b2916.rlib --extern cargo_bpf_lib=/tmp/cargo-installupv74r/release/deps/libcargo_bpf_lib-223048ee68e7b113.rlib --extern clap=/tmp/cargo-installupv74r/release/deps/libclap-e4afbfeff4c6e519.rlib --extern futures=/tmp/cargo-installupv74r/release/deps/libfutures-e602da79ab17c334.rlib --extern hexdump=/tmp/cargo-installupv74r/release/deps/libhexdump-13abeabd41398d37.rlib --extern libc=/tmp/cargo-installupv74r/release/deps/liblibc-5cc997263ec9d596.rlib --extern llvm_sys=/tmp/cargo-installupv74r/release/deps/libllvm_sys-221f1fc634fc51c1.rlib --extern proc_macro2=/tmp/cargo-installupv74r/release/deps/libproc_macro2-ce62abe820ec95ab.rlib --extern quote=/tmp/cargo-installupv74r/release/deps/libquote-5dd3b63b3c37ba50.rlib --extern redbpf=/tmp/cargo-installupv74r/release/deps/libredbpf-f11b01db69467f4c.rlib --extern syn=/tmp/cargo-installupv74r/release/deps/libsyn-0a3b9c6a54a8d527.rlib --extern tempfile=/tmp/cargo-installupv74r/release/deps/libtempfile-c2c3b1805ee83895.rlib --extern tokio=/tmp/cargo-installupv74r/release/deps/libtokio-4f81a95db43fc3b0.rlib --extern toml_edit=/tmp/cargo-installupv74r/release/deps/libtoml_edit-d3bd8a7c56c14f58.rlib --cap-lints allow -L native=/tmp/cargo-installupv74r/release/build/libloading-8059eb414c249b39/out -L native=/tmp/cargo-installupv74r/release/build/llvm-sys-2fce57eea06f2a74/out -L native=/usr/lib/llvm-9/lib -L native=/tmp/cargo-installupv74r/release/build/bpf-sys-5671ce2c156628b4/out -L native=/tmp/cargo-installupv74r/release/build/ring-5c5617bacb63ddf4/out` (exit code: 1)

Installing the libz-dev package should fix it

Well, that makes sense doesn't it 🤦♂- thanks, that addressed my issue.


Extend Handler interface with generic handler method

This PR extends Handler interface in the server package to have a method for handling commands not yet handled by the library.
This allows users of this library to implement extra functionality without extending server package. In my case I needed to handle COM_REGISTER_SLAVE and COM_BINLOG_DUMP commands which can not be easily handled by the server package itself.
This is straight forward implementation without caring about package backwards compatibility. If it is merged it will cause breakage for all existing server package users until they implement new method.
If breaking old clients is not acceptable we could:

Create new interface GenericHandler with the new method, and try detect if given handler implements additional interface at runtime in NewConn().
Extend server package to have new constructor dedicated for handlers that implement extended interface.
something else?

Rest LGTM


Heavy performance lag on note's editing

I have a big vault with a lot of tags. And if I enable metadata menu I have a lot of lags during typing in my notes.
According to performance profiling the issue comes from the plugin, and it can take up to 500ms every time.
Here is the report itself (every pink column is metadata function):

Profile-20221214T203157.zip
The source code that is called located here
Plugin version: 0.3.9

Are you using Lookup and Formulas type fields?
If so. I would start by changing their setting to disable the automatic update of those fields and see if it improves the performance

Are you using Lookup and Formulas type fields? If so. I would start by changing their setting to disable the automatic update of those fields and see if it improves the performance

No. I use only fields with predefined values from list that I've specified in settings

I've made some changes in 0.4.0 in the way fields get indexed and updated. In some cases it reduces the number of read/write operations and therefore improves the performance. Could you give it a shot?

It didn't help much, the lags are pretty the same. I'm attaching performance profiles that you can open with Chrome DevTools on Performance tab and explore the things that taking too much time inside your plugin.
Profile.zip
At quick view it seems that function getFilesFieldsExists executes some heavy computation. Looks like the function traverses all of the notes inside vault (over 5k notes in my case) to check that some of them have predefined fields. I'm not sure that computations are even required when I updating single note and in most cases such computations should be cached to prevent high cpu usage.

Thanks a lot! I didn't know about the performance profile. I'll dig into it.

I've made some optimization. Can you try again with 0.4.10?

With 0.4.10 I see big improvement and now it feels almost as fluent as without the plugin. Thank you
Although, it seems like there are still some parts that should be considered to optimize.

I'll close this one at this stage and see if the new datacore plugin from dataview developer will provide improvements on fields indexing that I can leverage to improve the speed of metadata-menu


[Bug] Not Found schedmd-slurm-gcp-v7-partition

While following the steps from the official documentation Llama 2 Fine-tuning with Slurm, I encountered a failure with the error message:
community/modules/compute/schedmd-slurm-gcp-v7-partition not found.

I checked the repository at GoogleCloudPlatform/cluster-toolkit, and it seems that the v7 version is not available yet.
It looks like this hasn’t been updated.
Possible Solutions

Create the v7 module.
Roll back the hpc-slurm-llama2.yaml to use the v6 version.

This should be corrected now.


API has been updated and is not in Beta anymore

API has been updated and is not in Beta anymore so the endpoints have changed.
I.e. https://graph.microsoft.com/beta/me is now https://graph.microsoft.com/v1.0/me

Thanks for the feedback. We're using the /beta endpoint at this point because in this sample we want to use a capability that is still available only on beta: the ability to fetch user profile photos from non-work and non-school Microsoft accounts (live.com). Everything else that this sample does should work on the v1.0 endpoint, though.


Update .NET Core hosting docs to highlight CoreClrHost.h

CoreCLR can be hosted using APIs in mscoree.h or coreclrhost.h. Our existing docs focus on mscoree.h even though coreclrhost.h is now preferred in many cases.
This update revises the hosting tutorial to walk through how to use both mscoree.h and coreclrhost.h (and puts coreclrhost.h first in this list). A corresponding change to host samples is included in PR dotnet/samples#538.

@morganbr, this is the docs update to go along with the samples update I tagged you in. As with that one, I'd appreciate your technical review here. Thanks!

I don't know much about hosting, but @AaronRobinsonMSFT or @vitek-karas might be able to help.


Scheduling

Firstly, thanks for taking the time to develop this.
I have rsync working, but can't seem to find how to run it on a schedule, ie daily.
I have looked through the documentation & readme but just cant see to find anything related to this ?
It does execute successfully, when I start the service, but it appears only to be a one-shot ?
Thanks again

Hi there, the addon is, as you already guessed it, designed as a one-shot.
If you want to execute the sync on a schedule you want to automate the addon start with Home Assistant Automation or some other System like Node Red.
Running it no a schedule is out of scope of the addon.


For for Issue #713 Unavailable Date phrase not reading on Screen Readers

Fix for Issue #713 by updating the way in which modifiers is checked for existence of the BLOCKED_MODIFIER now that modifiers is a Set and no longer an object.

Coverage increased (+0.05%) to 86.784% when pulling 1b36e852b167e26841bc95d9229282ecaa720623 on erin-doyle:Issue_713_Fix_Unavailable_Days into 261a477f5874c9a6d4d84a7c68e4eb7a3f46ad08 on airbnb:master.

No problem, I'm happy I was able to find it!  This was one of those sneaky bugs where for the case of available dates it was coincidentally working fine, where the check for the blocking modifier was returning false, it just happened to be for the wrong reason.  So it's easy to see how it was able to fly under the radar. 😉


Allow arbitrary base conversions

This allows arbitrary base conversion for the toString method

Coverage decreased (-0.8%) to 98.679% when pulling 04eb227c31ed79e2a97a1d403f5cf160b8706f92 on Auxolust:master into b7e949acdc534cc77e511a707b16f0c5e2ccfe7c on peterolson:master.

Failed tests.


[Bug] Function fails to load when another class file is in the function directory

Expected Behavior
I want to be able to split my logic into files within the function directory, or at least have it documented that I cannot.
Actual Behavior
When I added a second file to my function directory, if that file has a class the function will fail to load when deployed to an Azure Function App. Instead the logs will say:
2024-10-11T03:49:21Z   [Verbose]   Initiating background SyncTriggers operation
2024-10-11T03:49:21Z   [Information]   Loading functions metadata
2024-10-11T03:49:21Z   [Information]   Reading functions metadata (Custom)
2024-10-11T03:49:21Z   [Information]   1 functions found (Custom)
2024-10-11T03:49:21Z   [Information]   0 functions loaded
2024-10-11T03:49:21Z   [Information]   Loading functions metadata
2024-10-11T03:49:21Z   [Information]   Reading functions metadata (Custom)
2024-10-11T03:49:21Z   [Information]   1 functions found (Custom)
2024-10-11T03:49:21Z   [Information]   0 functions loaded

If that file has only function definitions than everything works as expected. I have also moved the helper file with a class definition to a sub directory and it worked again.
There appears to be some form of reflection that is preventing the function entry from being detected. If this is fundamental to how the worker functions, then adding clear guidance to the documentation that this can't be done would be helpful.
Steps to Reproduce

Create a function app with the code below. This is pulled from the timer trigger documentation, but adds the import of the Helper class
Add the following code to helpers.py in the function directory:

class Helper:
    def add(a: int, b: int) -> int:
        return a + b

Deploy the function app. I used the VS Code extension
Look at the logs of the Azure Function to see that the function never gets started. Logs included below

Relevant code being tried
import datetime
import logging
import azure.functions as func

from helpers import Helper

app = func.FunctionApp()

@app.function_name(name="my_timer")
@app.timer_trigger(schedule="0 */5 * * * *", arg_name="my_timer", run_on_startup=True)
def test_function(my_timer: func.TimerRequest) -> None:
    utc_timestamp = datetime.datetime.now()
    if my_timer.past_due:
        logging.info("The timer is past due!")
    logging.info("Python timer trigger function ran at %s", utc_timestamp)
    helper = Helper()
    logging.info(f"Result of add(1, 2): {helper.add(1, 2)}")

Relevant log output
...
2024-10-11T03:49:21Z   [Verbose]   Initiating background SyncTriggers operation
2024-10-11T03:49:21Z   [Information]   Loading functions metadata
2024-10-11T03:49:21Z   [Information]   Reading functions metadata (Custom)
2024-10-11T03:49:21Z   [Information]   1 functions found (Custom)
2024-10-11T03:49:21Z   [Information]   0 functions loaded
2024-10-11T03:49:21Z   [Information]   Loading functions metadata
2024-10-11T03:49:21Z   [Information]   Reading functions metadata (Custom)
2024-10-11T03:49:21Z   [Information]   1 functions found (Custom)
2024-10-11T03:49:21Z   [Information]   0 functions loaded
2024-10-11T03:50:01Z   [Verbose]   Received request to drain the host
2024-10-11T03:50:01Z   [Information]   DrainMode mode enabled
2024-10-11T03:50:01Z   [Information]   Calling StopAsync on the registered listeners
2024-10-11T03:50:01Z   [Information]   Call to StopAsync complete, registered listeners are now stopped
2024-10-11T03:50:01Z   [Verbose]   Received request to drain the host
2024-10-11T03:50:01Z   [Information]   DrainMode mode enabled
2024-10-11T03:50:01Z   [Information]   Calling StopAsync on the registered listeners
2024-10-11T03:50:01Z   [Information]   Call to StopAsync complete, registered listeners are now stopped
2024-10-11T03:51:06Z   [Verbose]   Received request to drain the host
2024-10-11T03:51:06Z   [Information]   DrainMode mode enabled
2024-10-11T03:51:06Z   [Information]   Calling StopAsync on the registered listeners
2024-10-11T03:51:06Z   [Information]   Call to StopAsync complete, registered listeners are now stopped
2024-10-11T03:51:06Z   [Verbose]   Received request to drain the host
2024-10-11T03:51:06Z   [Information]   DrainMode mode enabled
2024-10-11T03:51:06Z   [Information]   Calling StopAsync on the registered listeners
2024-10-11T03:51:06Z   [Information]   Call to StopAsync complete, registered listeners are now stopped
...

requirements.txt file
No response
Where are you facing this problem?
Production Environment (explain below)
Function app name
No response
Additional Information
No response

Related #1315

Does this function also work and load when running locally?
I tried this as well, and it worked as expected.
function_app.py:
import logging
import azure.functions as func

from helpers import Helper

app = func.FunctionApp()

@app.timer_trigger(schedule="0 * * * * *", arg_name="myTimer", run_on_startup=False,
              use_monitor=False)
def timer_trigger(myTimer: func.TimerRequest) -> None:
    if myTimer.past_due:
        logging.info('The timer is past due!')

    logging.info('Python timer trigger function executed.')
    #helper = Helper()
    logging.info(f"Result of add(1, 2): {Helper.add(1, 2)}")

helpers.py:
class Helper:
    def add(a: int, b: int) -> int:
        return a + b

directory structure:
- timer-trigger/
   - function_app.py
   - helpers.py
   - ...

One other way to make this work would be to create a subdirectory and add an init.py file, if it's easier to separate logic like that.
- timer-trigger/
   - function_app.py
   - utils/
      - __init__.py
      - helpers.py

Thank you for your example! This helped me narrow in a bit further on what seems to be going on. If there is a runtime error in the code before the function is defined (missing import, exception thrown in loading), then the function does not get loaded and there is no error messaging displayed to the user.  However, the function deploys successfully.
Is there any way to get clearer messaging that there was an error in the function code that is preventing it from being loaded? Potentially improving the log message

Yes, surfacing these types of errors to App Insights and other logging locations has been a priority for us. We've made several improvements for this that will be available in the Functions Runtime version 4.37, which will be available around late November / early December.


Parse scripts into an AST-like structure before interpreting their contents

toplevel is either:

Include

name. parser doesn't read the file

Command

filename & line number
argv
stdin & stdout names ("-" is implicit pipe)

Assignment

name
string value

Heredoc

name & string literal

Barrier
Tmpfile declaration

name

String is an array of pieces, which are:

Literal piece
Variable reference
getenv

Arg is either:

String
Different kinds of variable references

type & var name


 Is there a particular reason Go doesn't do this?

Thanks for doing this! We had some of these on our TODO list but it would've taken us a long time to get to them. :)
cc @enocom for context.

The other languages' start commands call releasetool.common.setup_github_context. Is there a particular reason Go doesn't do this?

We use Gerrit instead of Github, so I suspect that's why.


Improve Minify_Lines algorithm and add tests

Fixes #531
Fixes #534

Updated to make it cleaner to match tokens without false positives. Adding more exceptions adds more risk, but we can't guarantee perfection without a real tokenizer.


Obtaining GITHUB_PULL_REQUEST_ID for TeamCity – impossible?

Hello! Danger supposedly supports TeamCity, but we can't for the life of us figure out one critical part, which is how to get the corresponding PR number for a build. We've tried using an API to query GitHub with, but a major pain point has been that branches have a one-to-many relationship with pull requests.
Is anybody else using TeamCity with Danger, and if so, how are you obtaining the GITHUB_PULL_REQUEST_ID variable?

Good Q, I assume that @rbsgn was using something like https://github.com/lawliet89/pr_demon or the script at the end of this https://youtrack.jetbrains.com/issue/TW-43759

I just went through setting up Danger with TeamCity and was able to do the following for pull requests:
branch="%teamcity.build.branch%"
GITHUB_PULL_REQUEST_ID=(${branch//\// })

@atelic is there some way we can grab that ID from inside Danger?

@orta You might be able to ask users to add the following to their Simple Command Runner:
export TEAMCITY_BRANCH="%teamcity.build.branch%"

then in Danger
branch_name = ENV["TEAMCITY_BRANCH"]
GITHUB_PULL_REQUEST_ID = branch.split('/')[0]

I don't believe that %teamcity.build.branch% is available in ENV without exporting it though

Yep, I've configured this manually the same way

Alright - we've got improved docs in master now, so this can be closed - thanks

I'm confused. %teamcity.build.branch% only has the branch name (in my case refs/heads/tc-pr-test). Where, when or how does that include the PR number? Any help would be very much appreciated. I feel like I've been at this forever...

Hey, will issue a PR for the docs but incase anyone else comes here to find this, the best way to do this nowdays is using the Pull Request feature in TeamCity which gives you the following variable teamcity.pullRequest.number you simply set the env var to this in the parameters and done.


Normalize glob matching between Linux and Windows during TSLint build. Fix lint error.

PR checklist

[ ] Addresses an existing issue: #0000
[X] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update

What changes did you make?
Currently builds on AppVeyor fails due to lint errors however Circle CI builds are fine which is strange
Keeping in mind discussion on bug #1639 I've checked it on Debian "Jessie" (Docker image for node:6) and looks like glob src/**/*.ts passed without quotes is expanded and matches files only in nested folders (e.g. src/configs/latest.ts), but not directly in src folder (e.g. src/tslintMulti.ts ).
Since Windows doesn't expand glob patterns by itself only one glob string with patters is passed to glob module which correctly matches all files and leads to fail on lint error.
This PR normalizes behavior between platforms by preventing glob expand and delegates in to glob module.
Is there anything you'd like reviewers to focus on?
next branch has same root problem, however there are other lint errors. Should I create second PR to next branch or it should be done by team members only?

Thanks @IllusionMH! This PR is fine, we'll fix those other lint errors when we merge things next. We should recommend that people put double quotes around their globs, it seems to be a source of confusion for many (including us haha).


Issue in upload APIs.

Non-seekable stream cannot be used as input stream for upload APIs, otherwise it would result in a authentication failure.
Input stream's length cannot be 0, otherwise a authentication will fail.

2. Input stream's length cannot be 0, otherwise a authentication will fail.

Not exists anymore in latest commit, seems it's been fixed sometime.


freegeoip.net API endpoint is deprecated and will stop working on July 1st, 2018

Hey there, not sure if you are still maintaining this, but looks like it's about to break!
Will require a API key and a new endpoint.
curl -s http://freegeoip.net/json/<IP_ADDRESS>
{"deprecation_message":"This API endpoint is deprecated and will stop working on July 1st, 2018. For more information please visit: https://github.com/apilayer/freegeoip#readme","ip":"<IP_ADDRESS>","country_code":"AU","country_name":"Australia","region_code":"NSW","region_name":"New South Wales","city":"Marrickville","zip_code":"2204","time_zone":"Australia/Sydney","latitude":-33.9032,"longitude":151.1518,"metro_code":0}

Changing the two lines in FreegeoipProvider.java from
  private static final String SERVICE_URL_INDEX = "http://freegeoip.net/json/";
  private static final String RESPONSE_NOT_FOUND = "not found";

To
    private static final String SERVICE_URL_INDEX = "https://freegeoip.app/json/";
    private static final String RESPONSE_NOT_FOUND = "404 page not found";

Fixes the problem, as freegeoip.net is now dead, and the .app is a great replacement (free and 15,000 requests per hour allowed with no account rego crap).
I've implemented as a custom provider (nice docs and interface on that!), to make us more agile in the future, but thought you might like to know this can be rescued easily :)
Cheers, and thanks for a very nicely written plugin!

Changing the two lines in FreegeoipProvider.java from
  private static final String SERVICE_URL_INDEX = "http://freegeoip.net/json/";
  private static final String RESPONSE_NOT_FOUND = "not found";

To
    private static final String SERVICE_URL_INDEX = "https://freegeoip.app/json/";
    private static final String RESPONSE_NOT_FOUND = "404 page not found";

Fixes the problem, as freegeoip.net is now dead, and the .app is a great replacement (free and 15,000 requests per hour allowed with no account rego crap).
I've implemented as a custom provider (nice docs and interface on that!), to make us more agile in the future, but thought you might like to know this can be rescued easily :)
Cheers, and thanks for a very nicely written plugin!


Add ipynb to try Synbols

Summary
Add an ipynb that shows how to use Synbols and how to train something with it (using Pytorch Lightning).
Questions:

Do we want this ipynb?
Do we add a new step in the docker to install pytorch_lightning/jupyter
Do we make a new cli command to launch this notebook :)

Awesome! 🥇
Maybe it's worth adding a comment about how people should either run that notebook inside the docker image or copy the code to files and use the synbols command to run it.

I edited the readme and added a command to the makefile.

Installing synbols inside the docker image will likely break the synbols command. The reason is that the user's system installation of synbols is injected into the image at runtime. What I would suggest is to add an endpoint to the setup.py. I'll try to make an example of what I mean.

Also, it creates a chicken and egg problem since the docker needs to be built before we release the new version to pypi.

I pushed a proposal @Dref360. Let me know what you think. The only thing left to do would be to add a port argument to the synbols command and to include it in the call to docker run.

That command would allow users to run any jupyter notebook with their installation of synbols available.

I had to downgrade some packages to really old versions to get jupyter to work. I see many examples where people get up and running by just pip install jupyterlab. Maybe we would be able to do the same. If this works, I don't see the point of limiting the installation to jupyter notebooks only.


Added transition effect to product detail

Refer Issue: #97
Added animation and transition files under res/anim and res/transition section which can be used later anywhere in the app.
Used slide right and fade transition effect into product details while entering and exiting the fragment respectively.

It would be great if you add screenshort or video of the result

https://user-images.githubusercontent.com/85061899/193420554-3beb8d86-6efc-455a-adb8-5a6a2558ac6a.mp4

@hieuwu Please reopen the PR, it was accidently closed by me !!

HI I checked again. All the commits of this PR ahs been merged to main branch. So that we can not reopen this PR. Still marked as accepted but not sure Hacktofest will accept this


Adding underscores to test cases.

Pull Request Test Coverage Report for Build 768

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.366%

Totals

Change from base Build 762:
0.0%

Covered Lines:
2966

Relevant Lines:
3601

💛  - Coveralls


Adding options to compeletly disable creating userdb and sync_dir in building process

Is your feature request related to a problem? Please describe.
For privacy reasons, I don't want any of my typing history being logged into xxx.userdb dir and sync_dir. However, I could not find any ways to disable  them completely and easily. Right now I have to use chatter +i to make the xxx.userdb dir and sync_dir read-only. Although this just works, the librime may still attempt to log my typing history to those folders above (this might lead to some errors, but I'm not confirmed). So I tried to modify the source code to make the librime never do these things. But this gonna to be pretty tricky for me because I'm not familiar with the code and even if I am succeeded I have to manually cherry pick my patch every time the mainline updates(because this breaks the main function and will be never merged into mainline I guess)
Describe the solution you'd like
Make some options in the CMakeLists to control the behavior of the librime of creating userdb and sync_dir that can be simply triggered with ON or OFF

There is an easier way to disable userdb than chattr. Set enable_user_dict: false for every translator in your schema, and librime will not generate any userdb any more. Would this be enough for you?

I would like to add a few words: disabling userdb from the level of code logic, IMHO, it is just throwing the baby out with the bath water. The input experience of the local input method has to fight hard to compete with the Internet input method, and if you have to completely disable userdb, you are asking for trouble.

The option exists in YAML configuration.
Don't use the default YAML files.
Write your own *.schema.yaml file with these lines:
dictionary:
  enable_user_dict: false

Perhaps the approach I just described of adding an option during the build process to disable personalized word recommendations based on the user's input history is a bit too radical. But I don't think it's completely a case of throwing the baby out with the bathwater, especially if you need to temporarily stop recording input history and making personalized recommendations based on input history for some reason. And one of the reasons why I put the issue just now is because when I use the mozc input method, it has the function of setting the above behavior through the GUI (of course, maybe I should put this issue to fcitx5-rime)

Rime doesn't have a GUI, so the equivalent here is YAML config


Full Translated German

de.json

Could you create a proper PR? The current german translation was created from the community with the help of my weblate instance (See #5). Feel free to use my weblate instance to adjust or extend the translation. Otherwise, I will look at your changes as soon as I got time for it. I also want to improve the current translation to the latest changes, but I don't have the time currently.


geojson_hideout example doesn't work with debugger on

Hey guys, when I copy & paste and run example from doc site and enable debugger:
if __name__ == '__main__':
    app.run_server(debug = True)

I've got this error:

Any ideas how to fix it?
Thanks

It looks like the type validation is a bit too strict. I’ll relax it in the next update. For now, you can ignore it - or wrap the array in a dict
\emher
Sendt fra min iPhone

Den 10. jul. 2021 kl. 23.42 skrev Josef Heidler @.***>:

Hey guys, when I copy & paste and run example from doc site and enable debugger:
if name == 'main':
app.run_server(debug = True)
I've got this error:
Any ideas how to fix it?
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


kile: 2017-02-09 -> 2.9.92

No need to use an old git snapshot, since upstream has a proper beta version.
Motivation for this change
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

Any reason not to use the stable 2.1 release instead? Was 2017-02-09 newer than 2.1? Can't find release dates.

@GrahamcOfBorg build kile

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

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

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

Any reason not to use the stable 2.1 release instead? Was 2017-02-09 newer than 2.1? Can't find release dates.
2.1 is a Qt4/KDE4 version, so 2017-02-09 is definitely newer than 2.1.

Alright. Thank you!


[PE-D][Tester E] Lack of information on usage

What will the notification look like and how do I know if it is going to work?

Labels: type.DocumentationBug severity.Low
original: wolffe88/ped#13

move to future implementation / remove


Update request: zrythm 1.0.0-beta.4.9.1 → 1.0.0-beta.4.12.34

Package name: zrythm
Latest released version: v1.0.0-beta.4.12.34

Current version on the unstable channel:  1.0.0-beta.4.9.1
Current version on the stable/release channel: 1.0.0-beta.4.6.3

[ X] Checked the nixpkgs pull requests

Notify maintainers
@yuuyins
@magnetophon
@tshaynik

Note for maintainers: Please tag this issue in your PR.

1.0 is about to release 🥳

They are now packaging it, see the last entry in this issue :)
You can close it

Superseded by #306959 (Merged)


Invalid yaml causes GUI to freeze

If yaml in deepkit.yml contains syntax errors, the GUI freezes when trying to start a new experiment and displays "Loading.." while becoming unresponsive. Closing and restarting fixes this, but it would be nice to get a more subtle error message that was less of an interruption to workflows.

What version do you use?
Is this a project linked with Git or is it linked with a local folder?

With the current version this is not reproducible with projects linked to a local folder. Invalid configuration files are currently simply hidden.
I found an issue that no configurations are showed when at least one has errors in it, but only for projects linked with Git. I've fixed that in https://github.com/deepkit/deepkit/commit/53f6f5753cae1ba3c6f2bd83e4e989437df19470


Manage Users displays in dashboard sidebar for non-admins

To be done post #482

This was fixed somewhere along the line.


contributing.md

created a CONTRIBUTING.md file
closes https://github.com/origranot/reduced.to/issues/543

Hey @mzsrtgzr2,
Do you mind run the formatter?
Thank you for your contribution!