Wokring Dropdown 1 added

For Issue #965, I have improved the component and made working dropdown on hover

Amazing work @RavindraP04


Adding Ruff and Black CI Jobs

Adds Ruff and Black CI Jobs to be run on push and pull request, fixes files that don't pass ruff and black formatting, and updates CONTRIBUTING.md to reflect Ruff addition

 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.


[2842] - Public API fixes

Context

Currently it is not possible to retrieve locations (known as Sites) from the Public API via a Course
It's not possible to search for Providers - this breaks the 'search by provider' functionality on Find Teacher Training.

Changes proposed in this pull request

Add missing relationships (location statuses and locations) to serializers
Implement missing 'search by provider' functionality

Trello
https://trello.com/c/ftLKFssd/2842-add-missing-serializer-relationship-definition-to-public-api
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

https://api.publish-teacher-training-courses.service.gov.uk/api-reference.html#recruitment_cycles-year-providers-provider_code-courses-course_code-locations-get

@defong
I guess so but seems a little efficient as we would need to retrieve the Provider first, and filter the courses again.
This is where the API call is happening in Find. Some filtering of SiteStatuses required also - https://github.com/DFE-Digital/find-teacher-training/blob/master/app/view_objects/results_view.rb#L283
Happy to take your steer on this though 👍

https://api.publish-teacher-training-courses.service.gov.uk/api-reference.html#recruitment_cycles-year-providers-provider_code-courses-course_code-locations-get

@defong
I guess so but seems a little efficient as we would need to retrieve the Provider first, and filter the courses again.
This is where the API call is happening in Find. Some filtering of SiteStatuses required also - https://github.com/DFE-Digital/find-teacher-training/blob/master/app/view_objects/results_view.rb#L283
Happy to take your steer on this though 👍

https://api.publish-teacher-training-courses.service.gov.uk/api-reference.html#recruitment_cycles-year-providers-provider_code-courses-course_code-locations-get

@defong
I guess so but seems a little efficient as we would need to retrieve the Provider first, and filter the courses again.
This is where the API call is happening in Find. Some filtering of SiteStatuses required also - https://github.com/DFE-Digital/find-teacher-training/blob/master/app/view_objects/results_view.rb#L283
Happy to take your steer on this though +1

@misaka @gpeng to steer this forward, as this affects the overall architectural design profoundly

https://api.publish-teacher-training-courses.service.gov.uk/api-reference.html#recruitment_cycles-year-providers-provider_code-courses-course_code-locations-get

@defong
I guess so but seems a little efficient as we would need to retrieve the Provider first, and filter the courses again.
This is where the API call is happening in Find. Some filtering of SiteStatuses required also - https://github.com/DFE-Digital/find-teacher-training/blob/master/app/view_objects/results_view.rb#L283
Happy to take your steer on this though +1

@misaka @gpeng to steer this forward, as this affects the overall architectural design profoundly

api.publish-teacher-training-courses.service.gov.uk/api-reference.html#recruitment_cycles-year-providers-provider_code-courses-course_code-locations-get

@defong
I guess so but seems a little efficient as we would need to retrieve the Provider first, and filter the courses again.

Inefficient I guess you mean? Ya, but it's not that bad really since the request will be filtered. As discussed in Slack, this is our attempt at avoiding accidental N+1s which we were getting regularly with API v3 ... we could've spent more time to sort out all the includes to avoid them, but that was taking a lot of time and it seemed more efficient to change the API shape to stop them. Also, this rebalances the API a little allowing for more requests to come in, which means they can be balanced across servers better ... big long requests (think about serialisation, etc) make it harder to spread the load.

This is where the API call is happening in Find. Some filtering of SiteStatuses required also - https://github.com/DFE-Digital/find-teacher-training/blob/master/app/view_objects/results_view.rb#L283

If you're having to do filtering on the client side then we should consider, where it makes sense, adding server-side filtering. Is that the case?

api.publish-teacher-training-courses.service.gov.uk/api-reference.html#recruitment_cycles-year-providers-provider_code-courses-course_code-locations-get

@defong
I guess so but seems a little efficient as we would need to retrieve the Provider first, and filter the courses again.

Inefficient I guess you mean? Ya, but it's not that bad really since the request will be filtered. As discussed in Slack, this is our attempt at avoiding accidental N+1s which we were getting regularly with API v3 ... we could've spent more time to sort out all the includes to avoid them, but that was taking a lot of time and it seemed more efficient to change the API shape to stop them. Also, this rebalances the API a little allowing for more requests to come in, which means they can be balanced across servers better ... big long requests (think about serialisation, etc) make it harder to spread the load.

This is where the API call is happening in Find. Some filtering of SiteStatuses required also - https://github.com/DFE-Digital/find-teacher-training/blob/master/app/view_objects/results_view.rb#L283

If you're having to do filtering on the client side then we should consider, where it makes sense, adding server-side filtering. Is that the case?

Closing PR as changes no longer required

Closing PR as changes no longer required


fixed issue with lighthouse

Pre-Flight checklist

[X] Testing

[X] Code is tested

What are the relevant tickets?
No tickets
What's this PR do?
Fixes url for lighthouse
How should this be manually tested?
verify lighthouse stats are shown

There seems to be an unrelated change in this PR, might be best to rebase it off of main:

Yeah it had some unwanted commits I have removed those and is ready to merge


Does bridge support cocurrent jdbcfunction?

I hava exucte a very long sql such as "insert into table select * from jdbc(.....)" where jdbc(....) has about 1 blilion rows.
It takes a very long time to finish the sql.
But I found that when the sql excuting , the other sql that use jdbc fuction and jdbcTable will block until the sql above has been done.
Every jdbcfunciton must be excueted one by one? sync??
I have change the default timeout to a very long time. inlude the config in server.json 、vertx.json、datasource.json.
The work.num of bridge use the default value
If I missed something??
Thank U for your Reply.

I'd suggest you to consider partition(query by date range etc.) when there's lots of data. This will break one long-running query into smaller pieces each will take less time, so that you don't have to increase timeout to an unreasonable number like an hour.
What's the observation from show processlist? I think it's controlled by clickhouse, but the connection pool size configured in JDBC bridge may affect concurrency as well. I believe you're using the latest JDBC bridge, but what's the version of ClickHouse you're using?

I'd suggest you to consider partition(query by date range etc.) when there's lots of data. This will break one long-running query into smaller pieces each will take less time, so that you don't have to increase timeout to an unreasonable number like an hour.
What's the observation from show processlist? I think it's controlled by clickhouse, but the connection pool size configured in JDBC bridge may affect concurrency as well. I believe you're using the latest JDBC bridge, but what's the version of ClickHouse you're using?

In our scenario, there are some dimension tables that change all the time, and we hope to query the latest data in the dimension tables through JDBC.
But the problem is that once jdbcfunction is serial in the global scope, all statements that represent jdbcfunction are serial.Once one sql delay,the other will delay too.
Should I make new issue to clickhouse?not bridge?
My bridge version is 2.0.3, clickhouse version is <IP_ADDRESS>59

I wasn't able to produce your issue. I'd suggest you to double check your environment before creating an issue:

run show processlist on clickhouse to confirm if jdbc queries are running in parallel or not
check connection pool size defined in JDBC bridge
check if there's any restriction on source database limiting connection from JDBC bridge
try if you can reproduce the issue on a LTS/stable version like 21.8

Taking my test environment as an example:

on jdbc bridge I have below self datasource defined as below{
  "self": {
    "aliases": ["_", "current"],
    "driverClassName": "ru.yandex.clickhouse.ClickHouseDriver",
    "jdbcUrl": "jdbc:clickhouse://my-ch-server1:8123/system?ssl=false",
    "username": "readonly",
    "password": "***",
    "maximumPoolSize": 5
  }
}

repeat below queries a few times, and then run show processlist on clickhouse, you'll see up to 5 processes(because max connection pool size is 5)select * from jdbc('self', 'select * from numbers(10000000)')

I wasn't able to produce your issue. I'd suggest you to double check your environment before creating an issue:

run show processlist on clickhouse to confirm if jdbc queries are running in parallel or not
check connection pool size defined in JDBC bridge
check if there's any restriction on source database limiting connection from JDBC bridge
try if you can reproduce the issue on a LTS/stable version like 21.8

Taking my test environment as an example:

on jdbc bridge I have below self datasource defined as below{
  "self": {
    "aliases": ["_", "current"],
    "driverClassName": "ru.yandex.clickhouse.ClickHouseDriver",
    "jdbcUrl": "jdbc:clickhouse://my-ch-server1:8123/system?ssl=false",
    "username": "readonly",
    "password": "***",
    "maximumPoolSize": 5
  }
}

repeat below queries a few times, and then run show processlist on clickhouse, you'll see up to 5 processes(because max connection pool size is 5)select * from jdbc('self', 'select * from numbers(10000000)')

There would be 5 process running,but the sql will be excuted one by one.
if one sql coast 10sencods the last one will coast 50s.

There would be 5 process running,but the sql will be excuted one by one.
if one sql coast 10sencods the last one will coast 50s.

OK, I'm not aware of this. Then I guess you can raise an issue in clickhouse repo for help.

SELECT
    query_id,
    query_start_time,
    query,
    query_duration_ms
FROM system.query_log
WHERE (query LIKE '%numbers(10000000)%') AND (type = 'QueryFinish')
ORDER BY event_time DESC
LIMIT 10

┌─query_id─────────────────────────────┬────query_start_time─┬─query───────────────────────────────────────────────────────────────┬─query_duration_ms─┐
│ 3ada22f9-16b2-40b2-9319-73cf29142c12 │ 2021-08-25 08:22:41 │ select * from jdbc('self', 'select * from numbers(10000000)')       │             37077 │
│ ca41ab6a-440c-486a-bba8-73de0923199a │ 2021-08-25 08:23:15 │ select * from numbers(10000000)FORMAT TabSeparatedWithNamesAndTypes │              3007 │
│ de7b4565-8f83-42ca-8f36-23b9e992dd0d │ 2021-08-25 08:22:40 │ select * from jdbc('self', 'select * from numbers(10000000)')       │             34160 │
│ 8201ba9d-ba0c-455d-bb82-418a84c17290 │ 2021-08-25 08:23:11 │ select * from numbers(10000000)FORMAT TabSeparatedWithNamesAndTypes │              2970 │
│ ed6d7274-16f4-41f1-9a53-399f2f778fe1 │ 2021-08-25 08:22:40 │ select * from jdbc('self', 'select * from numbers(10000000)')       │             31256 │
│ 05e73f8b-b170-4c53-a4b4-461c04ae03cc │ 2021-08-25 08:23:07 │ select * from numbers(10000000)FORMAT TabSeparatedWithNamesAndTypes │              3605 │
│ b4bc8db3-72c5-4650-a0bf-40b8c64432f9 │ 2021-08-25 08:22:39 │ select * from jdbc('self', 'select * from numbers(10000000)')       │             27712 │
│ 194164fc-5517-4244-a28f-59ab4a52e79e │ 2021-08-25 08:23:02 │ select * from numbers(10000000)FORMAT TabSeparatedWithNamesAndTypes │              3446 │
│ 4424095b-bd45-42b0-b1bf-9be08c82a002 │ 2021-08-25 08:22:38 │ select * from jdbc('self', 'select * from numbers(10000000)')       │             24501 │
│ 17a4f296-3c1b-46d5-965b-6acf34e41584 │ 2021-08-25 08:22:57 │ select * from numbers(10000000)FORMAT TabSeparatedWithNamesAndTypes │              4416 │
└──────────────────────────────────────┴─────────────────────┴─────────────────────────────────────────────────────────────────────┴───────────────────┘

SELECT
    query_id,
    query_start_time,
    query,
    query_duration_ms
FROM system.query_log
WHERE (query LIKE '%numbers(10000000)%') AND (type = 'QueryFinish')
ORDER BY event_time DESC
LIMIT 10

┌─query_id─────────────────────────────┬────query_start_time─┬─query───────────────────────────────────────────────────────────────┬─query_duration_ms─┐
│ 3ada22f9-16b2-40b2-9319-73cf29142c12 │ 2021-08-25 08:22:41 │ select * from jdbc('self', 'select * from numbers(10000000)')       │             37077 │
│ ca41ab6a-440c-486a-bba8-73de0923199a │ 2021-08-25 08:23:15 │ select * from numbers(10000000)FORMAT TabSeparatedWithNamesAndTypes │              3007 │
│ de7b4565-8f83-42ca-8f36-23b9e992dd0d │ 2021-08-25 08:22:40 │ select * from jdbc('self', 'select * from numbers(10000000)')       │             34160 │
│ 8201ba9d-ba0c-455d-bb82-418a84c17290 │ 2021-08-25 08:23:11 │ select * from numbers(10000000)FORMAT TabSeparatedWithNamesAndTypes │              2970 │
│ ed6d7274-16f4-41f1-9a53-399f2f778fe1 │ 2021-08-25 08:22:40 │ select * from jdbc('self', 'select * from numbers(10000000)')       │             31256 │
│ 05e73f8b-b170-4c53-a4b4-461c04ae03cc │ 2021-08-25 08:23:07 │ select * from numbers(10000000)FORMAT TabSeparatedWithNamesAndTypes │              3605 │
│ b4bc8db3-72c5-4650-a0bf-40b8c64432f9 │ 2021-08-25 08:22:39 │ select * from jdbc('self', 'select * from numbers(10000000)')       │             27712 │
│ 194164fc-5517-4244-a28f-59ab4a52e79e │ 2021-08-25 08:23:02 │ select * from numbers(10000000)FORMAT TabSeparatedWithNamesAndTypes │              3446 │
│ 4424095b-bd45-42b0-b1bf-9be08c82a002 │ 2021-08-25 08:22:38 │ select * from jdbc('self', 'select * from numbers(10000000)')       │             24501 │
│ 17a4f296-3c1b-46d5-965b-6acf34e41584 │ 2021-08-25 08:22:57 │ select * from numbers(10000000)FORMAT TabSeparatedWithNamesAndTypes │              4416 │
└──────────────────────────────────────┴─────────────────────┴─────────────────────────────────────────────────────────────────────┴───────────────────┘

I'm really consufed....

This has nothing to do with clickhouse.  I also took your advice by catching throwable when setting client info so it should work well with jTDS driver as well.

Thank you very much. It was a great help


OPS or WHITELIST doesn't work in offline mode

Im working with java version
I installed the server with the helm chart, added player usernames to the ops and whitelist fields, like this:
whitelist: User1,User2,User3,User4,klucsik
ops: User1,User2,User3,User4,klucsik
overrideServerProperties: true
The server is freshly installed, its in offline mode, but I got rejected with the klucsik username.
[15:09:13] [Server thread/INFO]: Disconnecting com.mojang.authlib.GameProfile@6b639c5c[id=09cfa1dc-dd17-3c85-bb06-5e391a0f199e,name=klucsik,properties={},legacy=false] (/<IP_ADDRESS>:38299): You are not white-listed on this server!
Is it possible to use whitelist/ops in offline mode?

Seems to be related to https://github.com/itzg/docker-minecraft-server/issues/1436

I looked into the volume, and the uuid-s are different than the ones in the usercache.json. If I populate the whitelist/ops from the usercache, it works.


Unable to start service Intent on calling BackgroundGeolocation.start()

Your Environment

Plugin version: 3.0.0-alpha.50
Platform: Android
OS version: 8.1 (API 27)
Device manufacturer and model: Nexus 5X

Running in Simulator: Yes
Cordova version: 8.1.2
Cordova platform version: 7.1.4 (Android)

Context

Using the example code provided, the GPS tracking doesn't initialize properly.
Expected Behavior

The GPS tracking service should start.
Actual Behavior

Nothing happens, no events are being triggered, the service isn't started. Background & foreground mode still works and triggers events.
Possible Fix

It is the exact same problem as has been posted (and fixed?) in the react-native project:
https://github.com/mauron85/react-native-background-geolocation/issues/172
Debug logs

02-04 09:04:57.034  5194  5270 I com.marianhello.bgloc.BackgroundGeolocationFacade: Initializing plugin 02-04 09:04:57.082  5194  5270 D com.tenforwardconsulting.bgloc.cordova.BackgroundGeolocationPlugin: Registering event listeners 02-04 09:04:57.109  5194  5270 W PluginManager: THREAD WARNING: exec() call to BackgroundGeolocation.addEventListener blocked the main thread for 27ms. Plugin should use CordovaInterface.getThreadPool(). 02-04 09:04:57.117  5194  5230 D NetworkSecurityConfig: No Network Security Config specified, using platform default 02-04 09:04:57.362  5194  5194 D SystemWebChromeClient: file:///android_asset/www/plugins/cordova-plugin-android-softinputmode/www/interface.js: Line 3 : Success 02-04 09:04:57.362  5194  5194 I chromium: [INFO:CONSOLE(3)] "Success", source: file:///android_asset/www/plugins/cordova-plugin-android-softinputmode/www/interface.js (3) 02-04 09:04:57.462  5194  5272 D com.marianhello.bgloc.data.sqlite.SQLiteConfigurationDAO: Configuration persisted with rowId = 1 02-04 09:04:57.466  5194  5272 D com.marianhello.bgloc.BackgroundGeolocationFacade: Service configured with: Config[distanceFilter=50 stationaryRadius=100.0 desiredAccuracy=0 interval=3000 fastestInterval=5000 activitiesInterval=10000 isDebugging=true stopOnTerminate=true stopOnStillActivity=true startOnBoot=false startForeground=true notificationsEnabled=true locationProvider=1 nTitle=PolarFleet GPS tracking nText=enabled nIconLarge= nIconSmall= nIconColor= url=myUrl Threshold=100 httpHeaders={Content-Type=application/json} maxLocations=1 postTemplate={"heading":"@bearing","foo":"bar","lon":"@longitude","id":"ADG198","lat":"@latitude","speed":"@speed","timestamp":""}] 02-04 09:04:57.490  5194  5251 D EGL_emulation: eglMakeCurrent: 0xa9205d80: ver 3 0 (tinfo 0x9563a050) 02-04 09:04:57.508  1638  1941 W ActivityManager: Unable to start service Intent { cmp=myApp/com.marianhello.bgloc.service.LocationServiceImpl (has extras) } U=0: not found

It is the exact same problem as has been posted and fixed in the react-native project:
mauron85/react-native-background-geolocation#172

if it's exactly same problem, then it was already merged here somewhere before alpha.50.

That's not very helpful :)

That doesn't give me much to go on in trying to solve the problem. I am using the latest version of this plugin and I can't get it to work.

I have the exact same issue, my cordova and android version is the exact same as @Amsvartner but I am using ionic (shouldn't matter). The issue isn't with the .start() but .configure() anything below it is not getting executed like there were an error or something but the app don't crash.
import { Component } from '@angular/core';
import { Platform, AlertController } from 'ionic-angular';
import { StatusBar } from '@ionic-native/status-bar';
import { SplashScreen } from '@ionic-native/splash-screen';

import { HomePage } from '../pages/home/home';
import { BackgroundGeolocation, BackgroundGeolocationConfig, BackgroundGeolocationResponse } from '@ionic-native/background-geolocation/ngx';
import { LocalNotifications } from '@ionic-native/local-notifications';

@Component({
  templateUrl: 'app.html'
})
export class MyApp {
  rootPage:any = HomePage;

  constructor(public platform: Platform,
              public statusBar: StatusBar,
              public splashScreen: SplashScreen,
              protected backgroundGeolocation: BackgroundGeolocation,
              protected alert: AlertController,
              private localNotifications: LocalNotifications) {
    platform.ready().then(() => {
      // Okay, so the platform is ready and our plugins are available.
      // Here you can do any higher level native things you might need.
      statusBar.styleDefault();
      splashScreen.hide();

      console.log('pre-initializing everything! <--------------------------------------------------------------');

      const config: BackgroundGeolocationConfig = {
        desiredAccuracy: 1000,
        locationProvider: 1,
        stationaryRadius: 50,
        distanceFilter: 50,
        notificationTitle: 'Background tracking',
        notificationText: 'enabled',
        debug: true,
        interval: 10000,
        fastestInterval: 5000,
        activitiesInterval: 10000,
      };

      this.backgroundGeolocation.configure(config)
        .subscribe((location: BackgroundGeolocationResponse) => {
          console.log(location);

          // IMPORTANT:  You must execute the finish method here to inform the native plugin that you're finished,
          // and the background-task may be completed.  You must do this regardless if your HTTP request is successful or not.
          // IF YOU DON'T, ios will CRASH YOUR APP for spending too much time in the background.
          this.backgroundGeolocation.finish(); // FOR IOS ONLY
        }, (error) => {
          console.log('hey something bad happened!');
          console.log(error);
        });

      console.log('hey before we start anything did configure work?');

      // start recording location
      this.backgroundGeolocation.start();

      console.log('hey everything should be initialized!');

      // If you wish to turn OFF background-tracking, call the #stop method.
      //this.backgroundGeolocation.stop();
    });
  }
}

D/SystemWebChromeClient: http://localhost/build/vendor.js: Line 134709 : Ionic Native: deviceready event fired after 1433 ms
I/chromium: [INFO:CONSOLE(134709)] "Ionic Native: deviceready event fired after 1433 ms", source: http://localhost/build/vendor.js (134709)
D/SystemWebChromeClient: http://localhost/build/main.js: Line 218 : pre-initializing everything! <--------------------------------------------------------------
I/chromium: [INFO:CONSOLE(218)] "pre-initializing everything! <--------------------------------------------------------------", source: http://localhost/build/main.js (218)
D/SystemWebChromeClient: http://localhost/build/vendor.js: Line 2085 : ERROR
I/chromium: [INFO:CONSOLE(2085)] "ERROR", source: http://localhost/build/vendor.js (2085)
D/EGL_emulation: eglMakeCurrent: 0x9079a120: ver 3 0 (tinfo 0x93f07980)

I suspect that we may not have the same problem, because everything else works for me (background/foreground mode, app settings etc) but nothing happens when I call start() in the checkStatus callback:
  BackgroundGeolocation.checkStatus(function(status) {
    console.log('[INFO] BackgroundGeolocation service is running', status.isRunning);
    console.log('[INFO] BackgroundGeolocation services enabled', status.locationServicesEnabled);
    console.log('[INFO] BackgroundGeolocation auth status: ' + status.authorization);

    if (!status.isRunning) {
      BackgroundGeolocation.start(); //triggers start on start event
    }
  });

Thanks for your inputs, I've just got it working, apparently the issue was in the @ionic-native/background-geolocation/ngx
Using the following work properly, I will create my own ionic provider at that point.
declare var BackgroundGeolocation: any;

I have skimmed through a bug report sent in from one of the app users. Here's a couple of lines of it, that might be relevant to the case.
 I replaced line 242 with the following code:
// Build a Notification required for running service in foreground.
String channel;

if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
    NotificationManager mNotificationManager = (NotificationManager)  this.getSystemService(Context.NOTIFICATION_SERVICE);

    String name = "LocationService ";
    int importance = NotificationManager.IMPORTANCE_LOW;

    NotificationChannel mChannel = new NotificationChannel("LocationService channel", name, importance);

    mChannel.enableLights(true);
    mChannel.setLightColor(Color.BLUE);

    if (mNotificationManager != null) {
        mNotificationManager.createNotificationChannel(mChannel);
    } else {
        stopSelf();
     }

    channel = "LocationService channel";

} else {
    channel = "";
}

NotificationCompat.Builder builder = new NotificationCompat.Builder(this, channel);

Don't forget to update your imports.
import android.app.Notification;
import android.app.NotificationChannel;
import android.app.NotificationManager;


"cannot import name '_typeconv' from 'numba.core.typeconv'"

Im using Anaconda Navigator and Spyder "inside" that. The "base" environment comes with numba installed, however, failes multiple ways no matter what I do. So I made a new environment "Test" and installed numba via cmd promt as explained here: https://anaconda.org/numba/numba
conda install -c numba numba
Then in Spyder try to run:
import numba
I tryed updating (anaconda navigator, conda, spyder, numba), reinstalling (anaconda navigator, numba, updates this and that way, uninstall->install, force install; so a buch of stuff found via google)
I still get the error below. In the "base" environment I had a few other errors (like "OSError: Could not load shared object file: llvmlite.dll") over the course of trying to get numba running, but its the same import _typeconv now.
Oh and I did find a at least 2 issues like this, closed, but with no solution (that works for me).
C:\ProgramData\**Anaconda3**\envs\Test\python.exe
**Python version 3.8.8** | packaged by **conda-forge** | (default, Feb 20 2021, 15:50:08) [MSC v.1916 64 bit (AMD64)]
sys.version_info(major=3, minor=8, micro=8, releaselevel='final', serial=0)
Traceback (most recent call last):

  File "C:\Users\***\AppData\Roaming\Python\Python38\site-packages\numba\core\typeconv\typeconv.py", line 4, in <module>
    from numba.core.typeconv import _typeconv

ImportError: cannot import name '_typeconv' from 'numba.core.typeconv' (C:\Users\***\AppData\Roaming\Python\Python38\site-packages\numba\core\typeconv\__init__.py)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

  File "C:\Users\***\.spyder-py3\temp.py", line 14, in <module>
    import numba

  File "C:\Users\***\AppData\Roaming\Python\Python38\site-packages\numba\__init__.py", line 24, in <module>
    from numba.misc.special import (

  File "C:\Users\***\AppData\Roaming\Python\Python38\site-packages\numba\misc\special.py", line 3, in <module>
    from numba.core.typing.typeof import typeof

  File "C:\Users\***\AppData\Roaming\Python\Python38\site-packages\numba\core\typing\__init__.py", line 1, in <module>
    from .context import BaseContext, Context

  File "C:\Users\***\AppData\Roaming\Python\Python38\site-packages\numba\core\typing\context.py", line 11, in <module>
    from numba.core.typeconv import Conversion, rules

  File "C:\Users\***\AppData\Roaming\Python\Python38\site-packages\numba\core\typeconv\rules.py", line 2, in <module>
    from .typeconv import TypeManager, TypeCastingRules

  File "C:\Users\***\AppData\Roaming\Python\Python38\site-packages\numba\core\typeconv\typeconv.py", line 17, in <module>
    raise ImportError(msg % (url, reportme, str(e), sys.executable))

ImportError: Numba could not be imported.

[not development, blabla cut out]

The original error was: 'cannot import name '_typeconv' from 'numba.core.typeconv' (C:\Users\***\AppData\Roaming\Python\Python38\site-packages\numba\core\typeconv\__init__.py)'
--------------------------------------------------------------------------------
If possible please include the following in your error report:

sys.executable: C:\ProgramData\Anaconda3\envs\Test\python.exe

Thanks for the report. To try and isolate the problem, have you tried using the windows command prompt to create a new environment and then installing Numba into that and seeing if it works?

I'm hitting this issue as well on Windows 10 with python3.7 installing numba (latest version 0.53.1) in a virtual environment with pip. See steps below in PowerShell. Note that I have installed the latest Visual C++ Redistributable as suggested in #5600 and I also attemped rolling back to 0.49.0 (also suggested in #5600) neither of which helped.
PS C:\Users\cknight\src> python -m venv .venv
PS C:\Users\cknight\src> .venv\Scripts\activate
(.venv) PS C:\Users\cknight\src> python -m pip install -U pip
Collecting pip
  Using cached pip-21.1.1-py3-none-any.whl (1.5 MB)
Installing collected packages: pip
  Attempting uninstall: pip
    Found existing installation: pip 20.1.1
    Uninstalling pip-20.1.1:
      Successfully uninstalled pip-20.1.1
Successfully installed pip-21.1.1
(.venv) PS C:\Users\cknight\src> python -m pip install numba
Collecting numba
  Downloading numba-0.53.1-cp37-cp37m-win_amd64.whl (2.3 MB)
     |████████████████████████████████| 2.3 MB 2.2 MB/s
Collecting numpy>=1.15
  Downloading numpy-1.20.2-cp37-cp37m-win_amd64.whl (13.6 MB)
     |████████████████████████████████| 13.6 MB ...
Requirement already satisfied: setuptools in c:\users\cknight\src\.venv\lib\site-packages (from numba) (47.1.0)
Collecting llvmlite<0.37,>=0.36.0rc1
  Downloading llvmlite-0.36.0-cp37-cp37m-win_amd64.whl (16.0 MB)
     |████████████████████████████████| 16.0 MB 6.4 MB/s
Installing collected packages: numpy, llvmlite, numba
Successfully installed llvmlite-0.36.0 numba-0.53.1 numpy-1.20.2
(.venv) PS C:\Users\cknight\src> python
Python 3.7.9 (tags/v3.7.9:13c94747c7, Aug 17 2020, 18:58:18) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import numba
Traceback (most recent call last):
  File "C:\Users\cknight\src\.venv\lib\site-packages\numba\core\typeconv\typeconv.py", line 4, in <module>
    from numba.core.typeconv import _typeconv
ImportError: DLL load failed: The specified module could not be found.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\cknight\src\.venv\lib\site-packages\numba\__init__.py", line 24, in <module>
    from numba.misc.special import (
  File "C:\Users\cknight\src\.venv\lib\site-packages\numba\misc\special.py", line 3, in <module>
    from numba.core.typing.typeof import typeof
  File "C:\Users\cknight\src\.venv\lib\site-packages\numba\core\typing\__init__.py", line 1, in <module>
    from .context import BaseContext, Context
  File "C:\Users\cknight\src\.venv\lib\site-packages\numba\core\typing\context.py", line 11, in <module>
    from numba.core.typeconv import Conversion, rules
  File "C:\Users\cknight\src\.venv\lib\site-packages\numba\core\typeconv\rules.py", line 2, in <module>
    from .typeconv import TypeManager, TypeCastingRules
  File "C:\Users\cknight\src\.venv\lib\site-packages\numba\core\typeconv\typeconv.py", line 17, in <module>
    raise ImportError(msg % (url, reportme, str(e), sys.executable))
ImportError: Numba could not be imported.
If you are seeing this message and are undertaking Numba development work, you may need to re-run:

python setup.py build_ext --inplace

(Also, please check the development set up guide https://numba.pydata.org/numba-doc/latest/developer/contributing.html.)

If you are not working on Numba development:

Please report the error message and traceback, along with a minimal reproducer
at: https://github.com/numba/numba/issues/new

If more help is needed please feel free to speak to the Numba core developers
directly at: https://gitter.im/numba/numba

Thanks in advance for your help in improving Numba!

The original error was: 'DLL load failed: The specified module could not be found.'
--------------------------------------------------------------------------------
If possible please include the following in your error report:

sys.executable: C:\Users\cknight\src\.venv\Scripts\python.exe

>>>

I have temporarily fixed my issue by pinning dependencies as follows: llvmlite==0.32.1, numba==0.47.0.

python -m pip uninstall numba
python -m pip install numba==0.47.0

Thank you everyone for commenting on this issue. Since the OP @Eheran1 never replied back I am assuming this issue is resolved and will now close it.
As a remark: the error ImportError: cannot import name '_typeconv' from 'numba.core.typeconv' is usually the result of a failed update procedure by pip. Numba was changed significantly from 0.47 to 0.48 with a large number of internal refactoring. Essentially what happened, was that the _typeconv was moved to numba.core. We had found that during the update pip would not cleanly deinstall older versions, so two very different versions of Numba had become conflated leading to this specific error.  We had seen this error very frequently after the refactoring and the fix for almost all the occurrences was to setup a fresh virtualenv and install numba again. This is probably also why the recommendation to install or re-install version 0.47.0 seems to help some users.
Hope this helps!

I have not tried to use numba since, thus no update from me since I have nothing new to add.

I have not tried to use numba since, thus no update from me since I have nothing new to add.

OK, well, do feel free to re-open, or make a new issue, in case the issue does persist.

Thanks for the report. To try and isolate the problem, have you tried using the windows command prompt to create a new environment and then installing Numba into that and seeing if it works?

I have the same problem, tried creating a completely new environment with required dependencies, still throws same error

Thanks for the report. To try and isolate the problem, have you tried using the windows command prompt to create a new environment and then installing Numba into that and seeing if it works?

I have the same problem, tried creating a completely new environment with required dependencies, still throws same error

Thank you. As stated previously, this error message is almost certainly caused by an older release of Numba "sticking" around somehow (probably a bug).
We will be more than happy to debug this for you. Please open a new issue for your case. In order to debug, we will need:
a) The commands you entered and what they output.
b) A description of what you expected to happen
c) What did happen instead
Thank you.


Question on output dimension

Hi, great work there!
I noticed that the dimension of the output is not fixed and depends on video length(eg. some are 33x2048, some are 12x2048, etc)
What's the best way to get them to a single dim(eg. 1x2048)?
Thanks!

Depending on the length of the video, I guess temporally averaging the descriptors is a good choice for short video clip (ie less than 100). Otherwise you will need to aggregate the features using neural network approaches such as NetVLAD for example.

Depending on the length of the video, I guess temporally averaging the descriptors is a good choice for short video clip (ie less than 100). Otherwise you will need to aggregate the features using neural network approaches such as NetVLAD for example.

Hey thanks for the idea!


fluid.dataset~ help file says wrong outlet

At the bottom of the fluid.dataset~ help file it says the phrase "right outlet" which (as of beta7) is now the left outlet.

Solved in 01abf417f03bed27347e3e947da6524f6b4ef91d


added more fields for user information

I've added new entries that are returned when the user interacts with the API. Lemme know if I've broken any canons when parsing the was_online field.

Thanks for the contribution @BDFL669 - all merged 🚀

Always happy to help!


WIP: Installing Pelican/OSDF Origin via RPM (SOFTWARE-5962)

Things that are missing:

Some of the configuration is still up in the air as we learn more from
supporting JLab/GLBRC/others, and as we revamp the config we ship

Registration instructions could use improvement

No validation instructions yet

I have a second take incorporating your comments and also things from the cache installation via RPM docs, ready for review.
One thing that's missing from the doc is picking a namespace. Since that's going to be common with the 'installation via container' docs, I think that should go into a separate doc (some Pelican/OSDF equivalent of the Getting VO Data into the OSDF doc)

Made public in #222


Reactive properties seems doesn't work

Hello!
Your framework is just wonderful.
However, I found a problem. I need ReactiveProperty for my project, but it looks like even the example from the readme doesn't work as intended.
Steps to reproduce:

Make new project.
Download and enable plugin.
Create test scene based on Control.
Add script and place this code (from readme):

var _hp : int = 100
var _stamina : float = 1.0
var _attack_damage : int = 100
func _ready():
	# Create ReactiveProperty from member
	var _Hp : ReactiveProperty = ReactiveProperty.FromMember(self, "_hp")
	var __ = _Hp.subscribe(func(i): print("Changed Hp ", i))
	_Hp.Value += 10
	print("Reflected: ", self._hp)
I expect "Reflected: 110" but _hp still contains 100
It needs fix or I misunderstand?
Thank you in advance for your help!

G'Day and thank you for your heads-up on this! You are absolutely correct.
This bug originates from disposables now disposing themselves when going out of scope. The behavior was correct on the 4.1-branch (This shows, why a test-suit would be important...)
I will fix this as soon as possible :)

Fixed bug and verified behavior in 954a40f45fbad29a9b760678979d1a019722bd00


Remove proning feature

📝 Description
This PR aims to remove everything relevant to the player proning feature.
🎯 Goals
Remove the ability for players to prone and remove all trace of proning ever existing.
Okay maybe not completely, but for the most part.

[x] Remove @ deprecated methods for gun factory builders
[x] Remove the PlayerAnimations.java class
[x] Clean up some of the surrounding code
[x] Remove proning keybind
[x] Remove proning lang
[x] Remove proning packets
[x] Remove the 4 different isProning() methods

🚦 Testing
Tested this by running within dev-env and running around, trying to force myself to prone (external cheat plugin I made before) and done all sorts of actions to make sure no random null pointers exist.
Could not produce any issues other than commit 6a6203b.
⏮ Backwards Compatibility
Fully backwards compatible upon updating. Players will just be unable to lay down.
📖 Added to documentation?

[ ] 📜 README.md
[ ] 📑 Documentation
[ ] 📓 Javadoc
[ ] 🍕 Comments
[x] 🙅 No documentation needed

Summary by CodeRabbit
Summary by CodeRabbit

Chores

Removed the PRONE feature and various related elements like the ability to disable shaders with F4 and the WIP status of the flashbang.

Bug Fixes

Simplified the logic for adjusting the Y position of smoke particles to improve consistency.

Refactor

Improved code readability and maintainability in shell casing positioning logic.

New Features

Added key.unload functionality for enhanced user interaction.

Documentation

Updated the changelog to reflect recent changes.

Style

Refactored conditional statements in WeaponRenderer for better readability.

Seems good to me

When a player is proning, mod will write an NBT with the content player_entity_flags to the player. Personally, I think it is necessary to delete CompatibleExtraEntityFlags, EntityControlServerMessageHandler and EntityControlClientMessageHandler

When a player is proning, mod will write an NBT with the content player_entity_flags to the player. Personally, I think it is necessary to delete CompatibleExtraEntityFlags, EntityControlServerMessageHandler and EntityControlClientMessageHandler

Definitely overlooked those. Thanks for the heads up.


Update laravel-mix to 6.0.19

laravel/framework : "^8.12",

Laravel Mix Version: 5.0.9
Node Version 17.7.0
NPM Version 7.4.3
OS: nginx

Description:
/var/www/html/erp/node_modules/webpack-cli/bin/cli.js:93
throw err;
^
AssertionError [ERR_ASSERTION]: mix.js() is missing required parameter 1: entry
at Function.js (/var/www/html/erp/node_modules/laravel-mix/src/Assert.js:13:9)
at JavaScript.register (/var/www/html/erp/node_modules/laravel-mix/src/components/JavaScript.js:40:16)
at Api.mix. [as vue] (/var/www/html/erp/node_modules/laravel-mix/src/components/ComponentFactory.js:98:53)
at Object. (/var/www/html/erp/webpack.mix.js:18:6)
at Module._compile (/var/www/html/erp/node_modules/v8-compile-cache/v8-compile-cache.js:192:30)
Steps To Reproduce:
run  npm install vue2-datepicker
return:
42 vulnerabilities (37 moderate, 5 high)
To address issues that do not require attention, run:
npm audit fix
run  npm audit fix
return:
npm WARN using --force Recommended protections disabled.
npm WARN audit Updating resolve-url-loader to 2.3.2,which is a SemVer major change.
npm WARN audit Updating laravel-mix to 5.0.9,which is a SemVer major change.
npm WARN audit Updating vue-loader to 16.2.0,which is a SemVer major change.
run  npm audit fix --force
Will install<EMAIL_ADDRESS>which is a breaking change
43 vulnerabilities (35 moderate, 8 high)

Laravel Mix Version: 6.0.19

Here starts the problem!
run  npm run dev

dev
npm run development

development
cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --progress --config=node_modules/laravel-mix/setup/webpack.config.js

/var/www/html/erp/node_modules/webpack-cli/bin/cli.js:93
throw err;
^
AssertionError [ERR_ASSERTION]: mix.js() is missing required parameter 1: entry
at Function.js (/var/www/html/erp/node_modules/laravel-mix/src/Assert.js:13:9)
at JavaScript.register (/var/www/html/erp/node_modules/laravel-mix/src/components/JavaScript.js:40:16)
at Api.mix. [as vue] (/var/www/html/erp/node_modules/laravel-mix/src/components/ComponentFactory.js:98:53)
at Object. (/var/www/html/erp/webpack.mix.js:18:6)
at Module._compile (/var/www/html/erp/node_modules/v8-compile-cache/v8-compile-cache.js:192:30)
run npm i --legacy-peer-deps
It doesn't fixed for me, even if it took me to the previus laravel-mix version.

try to use node 15?

What's your webpack.mix.js file look like? Is that assertion error wrong?
AssertionError [ERR_ASSERTION]: mix.js() is missing required parameter 1: entry

Did you update your scripts?
I noticed the development command included cross-env which is used in laravel-mix 5 scripts but have been changed to:
  "scripts": {
    "dev": "npm run development",
    "development": "mix",
    "watch": "mix watch",
    "watch-poll": "mix watch -- --watch-options-poll=1000",
    "hot": "mix watch --hot",
    "prod": "npm run production",
    "production": "mix --production"
  },


using CamelCase #11

Images TheGreatWar.jpg changes name in camel case notation

It is ok. You main merge it to master.


Added geminate consonants to Norwegian Bokmal phonelist

Added long consonants Norwegian Bokmal (nob) phonelist, as they were missing.

The change to the phonelist itself looks good. Please follow steps 4-7 here and the tests you see above ought to pass.
I am assuming that segments like /h/ are excluded on the grounds they are (never?) geminates in this language.

I am assuming that segments like /h/ are excluded on the grounds they are (never?) geminates in this language.

That’s right, I only considered consonants for which I found evidence, either online or in the data itself, that they can be geminates. /h/ was not one of these. Unfortunately, there wasn't a lot of detailed information online. Many sources simply mentioned the possibility of "long consonants” and their relationship to short vowels.

That’s right, I only considered consonants for which I found evidence, either online or in the data itself, that they can be geminates. /h/ was not one of these.

That’s fine, proceed with that.

Reverting those two files (as suggested above) and then rerunning ./postprocess in both the scrape and phones directory ought to get the tests passing again---try it out.

I realize I mispoke. I think that git checkout won't undo a change you've already committed, it will just undo an uncommitted change? That is why languages.json and unscraped.json are still in this PR. I had to research how to do this. The internet says to try the newly-added (as of late 2019) verb restore. That is: git restore myfile will restore myfile to its state at HEAD. After committing and pushing, you'll know it worked if those files are no longer listed in the PR on GitHub.

Restore didn’t seem to do anything, but I was able to remove the two files from the PR by reverting the files to the commit before I made the changes using git checkout <commit-hash> <file-name> and then committing those changes

There is now a different, later error that the tests are failing on, but it's one completely unrelated to what you did so I give you a dispensation to submit; I'll resolve that issue separately. Let me know if you're ready for me to merge.

Yep, ready!

Before I merge this I'm going to edit your PR description to include the string "Closes #532". This tells GitHub that the PR, once merged, should be linked with that issue, and should mark it as closed.


Expand documentation on local encryption of unlinked data

I was only able to figure out how to do this by trial and error. Maybe we could add a specific example for local encryption of unlinked data to the documentation.
This topic is currently not well documented and users might be confused as how to achieve local encryption uf unlinked data. The documentation should cover fernet keys and key generation, as well as the appropriate alfred3 config settings in the secrets.conf.

Before this is tackled, @ctreffe will refine the issue - because, currently, it is not entirely clear what should be written into the docs.


issues with understanding errors (might require client work)

(validation in https://github.com/VEuPathDB/service-multi-blast/tree/dev/lib-blast-query-parser/src/main/kotlin/mblast/query/validate)
1- blast 500 usually due to sequence issues , we dont tell user, and dont give option to fix
latest issue: initial dashes  => ellie will add to validation   (to the errors we show on UI about sequence validation are 422 before runnig blast)
2- param sequence regex: remove? search run breaks when defline starts with >> which is allowed by ncbi blast
3-formatter exit status 255:  we show a 400 unclear json description missing message   (opened issue VEuPathDB/EbrcWebSvcCommon#4)
4- service throws 400 for unclear reason since it doesnt run and it is not in the workspace (job had ben run days earlier and workspace got deleted, so it is in user database)  fungi e0f0861e0be42c1b88dcfeb7f3dedb28 dec 13 6:48am

For clarification #1, the 500 is not passed to the user, the 500 is returned by the internal blast CLI container's micro-http server because the blast CLI command returned a non-zero exit code.
One of the major issues that causes the blast CLI command to fail is the fact that we previously didn't apply any validation rules to dashes in sequences, however the blast CLI tools do validate the dash count, specifically on the first line of a sequence, up to a max of 70 characters.
This has been patched on the old service by #211 and patched in the rewrite by #210

Point #2 is an issue in one or more of the site projects, the multiblast service applies no validation rules to deflines and has no regex used when parsing sequence inputs.

Point #3 is/was an issue with the multiblast plugin in EbrcWebSvcCommon.  When receiving an errored job response, the site code is attempting to get the JSON property "description" from the response object, but the response object does have not and never has had a "description" property, so there must have been some confusion around the MBlast API responses when developing this plugin.
Either way, the issue has been patched on the master branch and is tracked by issue: https://github.com/VEuPathDB/EbrcWebSvcCommon/issues/4

Point #4 I was unable to reproduce, I am able to access that job's results both as a guest and a logged in user.

For point # 4
The log output from the old mblast service for the request that returned a 400 is (with fgputil lines trimmed out):
2022-12-13 06:42:48.578 [rid:yRSgm] DEBUG RequestLogger:33 - Request start: POST /reports
2022-12-13 06:42:48.579 [rid:yRSgm] DEBUG AuthFilter:99 - Authenticating request
2022-12-13 06:42:48.579 [rid:yRSgm] DEBUG AuthFilter:116 - Endpoint allows guest logins.
2022-12-13 06:42:48.579 [rid:yRSgm] DEBUG AuthFilter:137 - Auth token is not a user id.
2022-12-13 06:42:48.584 [rid:yRSgm] DEBUG AuthFilter:169 - Request authenticated as a registered user
2022-12-13 06:42:48.585 [rid:yRSgm] DEBUG BlastDBManager:15 - Opening connection org.gusdb.fgputil.db.wrapper.ConnectionWrapper@5a23a35a
2022-12-13 06:42:48.852 [rid:yRSgm] DEBUG BlastManager:279 - Blast workspace e0f0861e0be42c1b88dcfeb7f3dedb28 does not exist.
2022-12-13 06:42:48.853 [rid:yRSgm] DEBUG BlastDBManager:19 - Closing connection org.gusdb.fgputil.db.wrapper.ConnectionWrapper@5a23a35a
2022-12-13 06:42:48.854 [rid:yRSgm] DEBUG ErrorMapper:81 - Caught Exception: HTTP 400 Bad Request
2022-12-13 06:42:48.854 [rid:yRSgm] DEBUG RequestLogger:42 - Request end: POST /reports 400


The debugging disconnects automatically for JavaScript/TypeScript project on Linux

OS: Ubuntu18.04
Build Version: 20220228.3
Regression: Not a Regression
Repro Steps:

Create a JavaScript project in VSCode.
Debug with "Attach to Node Functions“.
Check whether succeeds to debug.

Expect:
Succeed to debug.
Actual:
The debugging disconnects automatically.

More Info:

This issue also reproduces for TypeScript project.
This issue doesn't reproduce for Python, PowerShell and .NET projects.
This issue doesn't reproduce on Windows and Mac.

What version of node are you running on that Linux machine?  Based off the gif, it looks like it might be v10.  Could you try upgrading node and trying again?

The debugging doesn't disconnect automatically after upgrading nodejs to v16.14.0. So I close this issue.


[New Game]: Added Rock_Paper_Scissor Game

PR Description 📜
Added Rock Paper Scissor Game
Fixes # 227

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Thank you soo much for contributing to our repository 💗

Thank you @utkarshjaulkar ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @utkarshjaulkar 👋!
@kunjgit here,
We have this annoucement to make 📢!!
During the time of PR don't alter main index file ! 🙂
In main Readme file just use * as a game number during the time we are merging we will add number !! 👀
Make sure you join our Discord🕹 if you haven't 😇
Thank you 💗!

Thank you  @utkarshjaulkar , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Provide a flag to disable parallel configuration for IP mode in favor of configure-on-demand

Can you provide some details why parallel CC with IP and configure on demand can't be enabled simultaneously?

Can you provide some details why parallel CC with IP and configure on demand can't be enabled simultaneously?

As a note for the context: parallel CC != parallel configuration.  Parallel CC is mostly about parallel CC store/load(https://github.com/gradle/gradle/pull/30132), and parallel configuration is about running projects configuration in parallel. Given that, I'd rather answer on question "why parallel configuration and configure on demand can't be enabled simultaneously"
Eventually, it's definitely a goal to make configuration phase parallel AND incremental in IP mode.
Enabling parallelism in IP mode without incrementality(aka configure-on-demand) support is a step in this direction. It will allow to observe performance improvements been brought by parallelism during IDE project sync and tasks running from CLI.
However, for the builds of the extreme size (in terms of projects count), benefits from configure-on-demand could be more significant than benefits from parallel configuration. That's why the flag to control is going to be introduced.


Can you alias the controller to another name?

This is more of a question - the controllerAs syntax lets you rename the controller 'this' instance to something else, and you can do that with ng-controller, but in the router you get an automatic rename to the component name.
I'm not a huge fan of this, as I'd like to use vm as my view model name if I want to, regardless of the component name.  That kind of follows what I've seen in Angular 2.0 as a naming convention as well as using controllerAs with John Papa's naming conventions.
For example, I have a galleries use case that lists all galleries, and I'd prefer to have the controller instance name be vm, so I can inject 'galleries' as a model element. such as:
vm.galleries
rather than
galleries.galleries
Any way to do this? I'd like to see it as an 'as' property in the route instructions
Ken

I would +1 this.
If it is best practice to minimize the variables on scope this is a great way to achieve this.
This issue impacts existing directives I have created where I could go,
using controllerAs methodology, scope.vm.property, now, with the new router it is scope.component.property. If this directive is used in multiple components the issue is self evident.
To implement the new router and migrate to 1.4 I would need to return to assigning variables to scope again.
I would call this an  alias instead of as.
I am certain this issue will continue to arise as users who have been following 'best practices' look at migrating to the new router, it would also aid in scenarios where users generate html from metadata and use vm or some other alias as a consistent controllerAs view model name

It does look like the code supports an 'as' token, and that it creates
aliases for it. However, it looks like with further research, it also
renames the route instruction. So if you alias the controller as 'vm', then
your route function is 'vm()' - not a great plan.
I'd be happy seeing them support contollerAs type renaming in the view but
then keeping the route name the name of the component. Or maybe having a
name= in the route path?
Ken Rimple
Training and Mentoring Services
Chariot Solutions
610-608-3935
@krimple
On Sun, Mar 15, 2015 at 10:50 AM, Michael<EMAIL_ADDRESS>wrote:

I would +1 this.
If it is best practice to minimize the variables on scope this is a great
way to achieve this.
This issue impacts existing directives I have created where I could go,
using controllerAs methodology, scope.vm.property, now, with the new
router it is scope.component.property. If this directive is used in
multiple components the issue is self evident.
To implement the new router and migrate to 1.4 I would need to return to
assigning variables to scope again.
I would call this an alias instead of as.
I am certain this issue will continue to arise as users who have been
following 'best practices' look at migrating to the new router, it would
also aid in scenarios where users generate html from metadata and use vm or
some other alias as a consistent controllerAs view model name
--
Reply to this email directly or view it on GitHub
https://github.com/angular/router/issues/179#issuecomment-81057320.

Seems like this could be another option on $componentLoaderProvider because its another way to control the 'conventions' of the new router. I'd consider this a 'convention'.

Tim, I'd agree. Would be nice if we could set a viewModelName property or something. like $componentLoaderProvider.setViewModelName('vm');

I think it would be a function that takes the component name and returns the viewModelName as a string. This would be in line with the other methods.
$componentLoaderProvider.setViewCtrlNameMapping(function(componentName){
    return 'vm'; // Ignore incoming component name and just set them all to 'vm'
});

Agreed
On Mar 15, 2015 10:54 PM, "Tim Kindberg"<EMAIL_ADDRESS>wrote:

I think it would be a function that takes the component name and returns
the viewModelName as a string. This would be in line with the other methods.
$componentLoaderProvider.setViewCtrlNameMapping(function(componentName){
return 'vm'; // Ignore incoming component name and just set them all to 'vm'
});
--
Reply to this email directly or view it on GitHub
https://github.com/angular/router/issues/179#issuecomment-81362947.

The as method works for aliasing the component, I keep having to remember to read the code and not the documentation. This will make migration simpler to 1.4 and beyond. I still would recommend a means for aliasing the controller as originally requested.

So I'm fine with either one, although the more I think of it as: is better
as it's in the routing instruction, and therefore is one place to look as
opposed to searching around for the prefix. Plus you won't have a second
method call with the equivalent of case statements everywhere if you wanted
to customize each one differently.
You could argue that you should be able to change the default, and that the
as: would override the default.
Ken
Ken Rimple
Training and Mentoring Services
Chariot Solutions
610-608-3935
@krimple
On Mon, Mar 16, 2015 at 9:46 AM, Michael<EMAIL_ADDRESS>wrote:

The as method works for aliasing the component, I keep having to remember
to read the code and not the documentation. This will make migration
simpler to 1.4 and beyond. I still would recommend a means for aliasing the
controller as originally requested.
--
Reply to this email directly or view it on GitHub
https://github.com/angular/router/issues/179#issuecomment-81675210.

@timkindberg – thanks for the suggestion. I'm willing to try something like this, but I think it needs to be scoped so that components written with different conventions can still be used in the same application.
This may be related to other issues that have recently been brought up with regards to configuring component conventions.

@btford the 'as' solution is great too, if I understand it correctly. I hadn't learned of it yet when I made my suggestion above. So 'as' might be good enough on its own.

No worries. I think it was still in the pipeline at that time. I'm going to update the docs with more info.

The ideal solution migration-wise would be to get the ability to scope the template to the controller, not scope (that would simplify the module.component helper as we discussed recently). This won't happen before 1.5, though and people will most likely still need an override for situations where they need the scope.

+1 for controllerAs allowance. For various reasons we use some conventions in out template files, weather it's 'vm', or we subclass our controllers which use the same template file, and thus the same alias for the controller.
I'm not sure 'as:' syntax is what we need, but will wait for updated docs...

I, for one, feel this thread is as clear as mud
I thought the "as" syntax aliased the route.  Are we saying it winds up being the "controllerAs" type override too?
Are there updated docs that clarify this?

Not sure where things landed on this discussion, but I'd like to vote for a "controllerAs" setting in the $routeConfig in addition to the "as" setting.  I like the ability of the "as" setting to alias the route for use with ng-link.  But I'd like to be able to scope all of my route templates to "vm" just as if the I had setup the view with ng-controller="myController as vm".

@chrisaswain, That's easy, altough kinda ugly to do now:
   MyController.$inject=['$scope']
   function MyController($scope) {
     var vm = this;
     $scope.vm = vm;
     ...
   }

I don't like the fact you have to inject $scope at all, but it will get you going now.

Thanks @SanderElias . However, when I try to do that with my "AllergensController" I get the following error:
[$injector:unpr] Unknown provider: $scopeProvider <- $scope <- AllergensController

Here is my code:
angular.module("app").controller("AllergensController", AllergensController);

    AllergensController.$inject = ["$scope", "$location", "azureMobileClient"];

    function AllergensController($scope, $location, azureMobileClient) {
        var vm = this;
        $scope.vm = vm;

@chrisaswain, you are right. current version doesn't let you inject scope. (I just tried in a project of mine, apparently I'm not using $scope at all anymore... ) But there still is a solution, I might even prefer slightly above my earlier one.
    SomeController.prototype.activate = function ($scope) {
        $scope.vm = this;
    }
    SomeController.prototype.activate.$inject = ['$scope'];

Sorry, I missed something! :wink:

Ah, I see.  Here's an interesting question... I like to use the styles defined by the popular [style guide] (https://github.com/johnpapa/angular-styleguide) originated by John Papa.  In it, he encourages the use of an activate() function in your controllers.  So I always have an activate function in my controller that is called by default in the controller is instantiated.  Will this interfere with the activate function you mention above that seems to be part of the component lifecycle hooks?


Update team.md

I want to be a part of your great team.
I am also a student of your PHP Basic Course.
Please, accept me.
Insha Allah, I will try my best to help you in every possible way possible for me. Thank you

I want to be a part of your great team.
I am also a student of your PHP Basic Course.
Please, accept me.
Insha Allah, I will try my best to help you in every possible way possible for me. Thank you


FormatCustomText not updated

Description
I've modified the format_custom_text example to use a with... as... context. The progress bar does not update when I do that, and I can't quite figure out why
Code
def format_custom_text():
    format_custom_text = progressbar.FormatCustomText(
        'Spam: %(spam).1f kg, eggs: %(eggs)d',
        dict(
            spam=0.25,
            eggs=3,
        ),
    )

    with progressbar.ProgressBar(widgets=[
        format_custom_text,
        ' :: ',
        progressbar.Percentage(),
    ]).start() as bar:
        for i in range(25):
            format_custom_text.update_mapping(eggs=i * 2)
            time.sleep(0.1)

The output only updates at the end, to show:
Spam: 0.2 kg, eggs: 48 :: N/A%

I also tried with adding a bar.update() after the last line within the loop, but with no success
Versions

Python version: 3.9
Operating System: Mac OS X
Package version: 4.0.0

I've also tried:
    format_custom_text = progressbar.FormatCustomText(
        'Spam: %(spam).1f kg, eggs: %(eggs)d',
        dict(
            spam=0.25,
            eggs=3,
        ),
    )

    bar = progressbar.ProgressBar(widgets=[format_custom_text])
    bar.start()
    for i in range(25):
        format_custom_text.update_mapping(eggs=i * 2)
        time.sleep(0.1)
        bar.update(i)


autoregister method

Fixes #933
Changes -

We use register_method (for torch.Tensor.*) to register method on TensorProxy.
We had conflicting registration for torch.Tensor.size and couple other methods. This was because they are directly registered as a method on TensorProxy (and not via torchsymbol. so they don't show up in _torch_to_thunder_function_map). Have updated to check_overlap_ops to also check for registered methods.
https://github.com/Lightning-AI/lightning-thunder/blob/36f46dde9b280784d8ff2a3a24e3a975fc67ffe1/thunder/torch/__init__.py#L245-L252

also thank you @kiya00


End-effector collision geometry being ignored in collision check inside local planner

Issue:

Contacts are added to the ContactManager into a map with keys as the name of the parent link of the colliding geometry.
If there are two contacts on the same link, only the first one will be added, because std::map cannot have repeat keys.
ContactManager::reduceContacts() will not see the other contacts as a result, and will not do anything.
This is why sometimes there is collision and sometimes not, since it is just the first contact that is added.

Fix:

Use some other sort of associative container besides map to hold the contact points.


Changed curvature method to that of Jibben et al. 2017

The dynamic memory allocation adds ~2% to the curvature cost.


pull request for e2e test at 01/02/2017 22:45:08

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Can't use Void with EventProducer.

I tried to make EventProducer<Void>, Observable<()> and something like above, but in all cases XCode will hang.
Anyone know in detail this problem? or Bond doesn't support like this? 🙏

https://github.com/SwiftBond/Bond/blob/master/Bond/Extensions/iOS/UIButton%2BBond.swift#L35
EventProducer<Void> works, I have used it many times in my own project.


Update README.md

Removed historical parts with not really published roles.

I think we do not need this change any more, after integrating pull request #63 .
Can you please check?

I made several changes to README.md, so I hope all your issues are fixed. If not, please create a new issue or pull request or reopen this one.


Upgrade dev-lang/rust and virtual/rust in main from 1.73.0 to 1.74.1

Sync with Gentoo, based on commit f679cca3b5cb.
Apply Flatcar patches, based on commit ec8e7e8f8710.
Note: it is necessary to manually reset and sync with Gentoo due to a build issue in the Gentoo ebuild that started to happen since Rust 1.74+.
/update-sdk
CI: http://jenkins.infra.kinvolk.io:8080/job/container/job/sdk/1223/cldsv/

CI passed: http://jenkins.infra.kinvolk.io:8080/job/container/job/sdk/1223/cldsv/

Thanks for bringing this in


OSX Firefox: router-outlet component is not displayed if several times click another tab and refresh browser

Click tab, refresh browser, repeat this several times and it shows only app component without  router-outlet content. There are no any related warnings/errors in console.

Would you clean the cache of the browser and try again? I had the same issue before I did this.
The error you get from livereload is caused by reload of the page. The warning is caused by the es6-shim module.

Should be fixed by https://github.com/mgechev/angular2-seed/commit/090bace6a36ec86881927c2e6c4a85e6fdc3bdd4 Let me know if you still can reproduce the issue.

I have tested, I can't reproduce the issue.


Revert "Remove ArticleDesign.PrintShop"

Reverts guardian/csnx#1683

closing in favour of looking at removing format-based stuff in another PR


Improving dark mode for Bootstrap elements

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Some elements do not switch to "dark mode", like outline buttons, cards and more. The ideal solution would be to switch all elements, although I understand this would take considerable time.
Describe alternatives you've considered
Provide a documentation how the user can fix/improve this himself.
Thanks

Bootstrap twbs/bootstrap#35857 add dark mode support officially. I'll handle it after that.

Sounds great, can't wait!


Stabilize MessageDrivenBeanInterceptorInvocationTest.

Application of [pull/105] (https://github.com/cdi-spec/cdi-tck/pull/105) to 1.2 branch.

Merged. Thank you.


Training throws key error

When Label present in validation set and not present in training set , the train method throws KeyError

Thanks @oneraghavan .
I will fix the problem.


Tried to upgrade and got dependency error

NPM error:  deprecated<EMAIL_ADDRESS>pngjs2 has now taken over the original pngjs package on npm. Please change to use pngjs dependency, version 2+.

Strange, that should be a warning not an error since it's a deprecation. The underlying issue is save-pixels in pixelsmith using an out of date version (determined via npm ls).
There's an open issue here: https://github.com/scijs/save-pixels/issues/17
Would you be interested in submitting a PR to patch it on their repo? They are usually pretty responsive

Sure thanks.

@CoralSilver I'm confused why you never submitted a PR to patch the repo despite saying you were going to =/
It looks like Mikola finally patched it recently in a patch release so this issue should be resolved now.


proposal: unpadded base32 encodings

The encoding/base64 package has unpadded encodings as of 2e0a1a757. I'd find it useful to have padding-free base32 encodings.

I don't see any reason they should be inconsistent.

CL welcome.


Delegate moment module to ember-cli-moment-shim

I've opened a PR against ember-cli-moment-shim to move ember-moment's moment module code to ember-cli-moment-shim.
Could you look over that PR?
While testing locally, I'm trying to import moment from 'moment' to no avail.

While testing locally, I'm trying to import moment from 'moment' to no avail.

While testing ember-moment or ember-moment-shim?

@jasonmit while testing an ember-cli app that is consuming ember-cli-moment-shim.
@stefanpenner I'm looping you into this both because:

I've hoisted the code from this repo, and plan on making this addon delegate to ember-cli-moment-shim once that PR is merged
I largely don't know what I'm doing when it comes to bower and ember-cli addons interact

FYI: https://github.com/stefanpenner/ember-moment/blob/4b64dbe6c23a8ef45552392cbe24fd773cac1328/index.js#L40-L42 does basically 0 things now, it did in the past and it may in the future.

This is resolved on both ember-moment and ember-cli-moment-shim

:clap: nice work you two.

its all @jasonmit :)


[Bug]: [CanvasLayout] Fix scrollbar positioning.

Objective:

Remove white space to the right of the canvas.
Scroll bar should be aligned along the right edge.

The scrollbar is now positioned within the canvas, working as expected in the PR #20836 , Moving to done.


PrettyPrinter reports wrong line LineNumbersTests

This is to reproduce issue #882.
I am not sure if the fix is done in the right place. With my change every string is checked for new lines, which might be not necessary and comes at a slight cost. Maybe there is another, better place to fix this.

@bkolb Could you run swift-format on your changes? The Windows failures will be fixed by https://github.com/swiftlang/github-workflows/pull/70.

@bkolb Could you run swift-format on your changes? The Windows failures will be fixed by swiftlang/github-workflows#70.

Done

No problem :-) Thx for your reviews.
I don't understand the build failure on windows. Can you give me a hint?

The Windows test failure is an infrastructure issue and will be fixed by https://github.com/swiftlang/github-workflows/pull/70.

@bkolb This PR regressed swift-format’s performance by ~7% (https://github.com/swiftlang/swift-format/issues/894). Could you check if there’s a way to avoid that performance regression?

That split call is doing a lot of work that gets thrown away. We don't actually use the substrings themselves, just the number of them and the length of the last one, so we have allocations on the critical printing path that we don't need. Replacing that with something that just counts the number of \ns and the length of the final segment would probably yield a good improvement.

https://github.com/swiftlang/swift-format/issues/894 contains instruction how to reproduce the measurement.

Changing the code to:
let lines = text.count { $0 == "\n" }
    lineNumber += lines
    guard lines > 0, let range = text.range(of: "\n", options: .backwards) else {
      // Handle case where no newline exists
      column += text.count
      return
    }
    let lastLine = text[range.upperBound...]
    column = lastLine.count

Brings the performance measurement to be much closer to the original. There may be a better way to get the last line though.
@bkolb I can open a PR for #894 if you are busy.

Thank you! I was just about to take a look. You have been faster :-)


[Feature] Support Flink 1.18

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Flink 1.18 official version is about to be released, and StreamPark needs to support the latest Flink 1.18.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

i can support flink1.18 on k8s

I add streampark-flink-shims_flink-1.18 module

@wolfboys  Hello, when flink-kubernetes-v2 supports flink1.18, you only need to upgrade the crd


Updated protoc-jar to 3.7.1

The tests pass using https://github.com/scalapb/protoc-bridge 0.7.5 but we may want to release a new version of that before merging this PR. The new version of bridge uses this protoc-jar 3.7.1

Thanks for the PR. Committed via bf271de


autoSkip doesn't properly hide labels that overlap with explicit min/max values

I am trying to implement a smoothly updating stripchart by using a scatter plot in Chart.js. My x-axis data is a relative time value, in seconds. The values are not perfectly evenly spaced; the interval can be different by many milliseconds.  This results in the stripchart animation being jumpy and hard to watch.
I can solve this by explicitly setting these to the first and last X values that I want to display, but when I do that, the x-axis labels sometimes overlap when the min/max values are close to a tick label. Setting autoHide=true is supposed to prevent this, but it only considers the evenly spaced ticks in between, ignoring the explicitly set minimum and maximum values.
A simple implementation showing the issue is here:  https://jsfiddle.net/5fok05jd/. Set LIMITS=false for the jumpy updating behavior; set it true for the overlapping labels behavior.

@jorenko I had a look at your fiddle but I don't see a variable called LIMITS. Did I miss something?

Oops, that is an old version. Check out https://jsfiddle.net/5fok05jd/4/ instead.

This is the same issue as #1968. Closing as a duplicate


Drawing svg during scrolling

Dear Maxwellito,
first of all, thanks for your script and sorry for my bad English,
I'm an Italian graphic designer, not a developer, so I ask you sorry also for what I will to submit.
I'm working with your library for a webpage, playing some svgs, in different ways, timings and delays.
After playing successfully the first ones, I would like to draw an svg during scrolling.
The issue is that works locally only with Safari,
nothing happens with Firefox and Chrome,
while it does not work on the network with any browser,
I'm quite sure the problem is my script, I'm not a developer
and I would like to ask you an help.
Here's the link to a simplified part of my project.
https://jsfiddle.net/rotella/Lfqrzpod/
Thanks in advance for your help,
alberto.

Hey there,
Sorry but I got a pretty busy weekend. I think I found a temporary solution
to your problem. I'll come back with some details later tomorrow
https://jsfiddle.net/r9wyxofv/
Le sam. 8 déc. 2018 à 13:07, rotellanomala<EMAIL_ADDRESS>a
écrit :

Dear Maxwellito,
first of all, thanks for your script and sorry for my bad English,
I'm an Italian graphic designer, not a developer, so I ask you sorry also
for what I will to submit.
I'm working with your library for a webpage, playing some svgs, in
different ways, timings and delays.
After playing successfully the first ones, I would like to draw an svg
during scrolling.
The issue is that works locally only with Safari,
nothing happens with Firefox and Chrome,
while it does not work on the network with any browser,
I'm quite sure the problem is my script, I'm not a developer
and I would like to ask you an help.
Here's the link to a simplified part of my project.
https://jsfiddle.net/rotella/Lfqrzpod/
Thanks in advance for your help,
alberto.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/maxwellito/vivus/issues/211, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABC-EOE6mrsr1fmPFN1HyE-2NnoYTaJJks5u27mZgaJpZM4ZJoTP
.

Dear Maxwellito,
thanks a lot for your fast reply and "temporary" solution,
for me works just great!
I've also tried with progress,
something like this
var drawLogo = new Vivus ('logoLettering', {type: 'sync', start:'manual'});

				function resetta(){
					drawLogo.reset();
				}

		window.addEventListener('scroll', scrolling);

				function scrolling(){

					if (document.body.scrollTop > 0 || document.documentElement.scrollTop > 0){

						progress = window.scrollY/100;
						console.log(progress);
						drawLogo.setFrameProgress(progress);
						drawLogo.stop();

					}

				}

but having same type of issues.
So your solution is perfect!
Don't worry and take your time, I'm not in a hurry.
Thanks again and have a nice weekend.
alberto

Brilliant then :) You were pretty close, you were listening on the wrong
event. The event 'scroll' is triggered when the target is scrolled, but the
body of your page is not big enough for scrolling. But the event 'wheel' is
used when the user use the wheel on the target.
Enjoy then :)
If you have more questions, don't hesitate

The CSS change I did was something I forgot to remove. It's not even valid. You can get rid of it. Only the event type is important :)
So, all good now?

Hi there,
sorry for my delay,
regarding the CSS changes, I've noticed that
with the #testLogo #logo position property (of the jsFiddle sample) set to "absolute", doesn't work,
while with the "relative" position works great,
so I thought that, besides the event type, was also important the positioning.
Anyway, as I said, I'm not a developer, so I would not say nonsense,
now I'm working with your solution and it's ok!
Thanks a lot for your valuable help!!!
Thanks for your awesome work!!!
Have a nice day,
my best regards,
alberto.

Dear Maxwellito,
sorry if I write a new comment,
I would only like to thank you once again
and let you know that your solution with deltay wheel event works great!
I've just finished redoing my page, implementing your precious code,
at first, it was fine for Firefox and Chrome but not in Safari :confounded:,
the opposite status of my original issue,
even more clear then that it was my code full of bugs!
I've found that, within my page, deltay wheel event wasn't work properly on Safari due to my html/css "confusion", by adding a simply overflow:hidden property to the body all works great!
So, thanks for your great work and precious help!
Happy holidays,
alberto.


Update changelog for 0.30.0 release

What are you trying to accomplish with this PR?
Release a new version of the gem
How is this accomplished?
Bump the gem to 0.30.0 and update changelog
What could go wrong?
Breaking changes or important changes could have been missed from changelog

LGTY @KnVerey ?


feature: Shareable link (URL shortner)

Type of feature
Select the type of feature request, the lowercase should also be the PR prefix.

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Current behavior
Is your feature request related to a problem? Please describe.
As a user, I should be able to copy a shortened URL by clicking the share button on the insights. My recommendation is that this URL is shortened, which will require us to use our own custom URL shortener.
We could potential use this path outline by Netlify. Open to suggestions

Suggested solution
URL shortener URL options.

oss.fyi
that's it 😅

Code of Conduct
By submitting this issue, you agree to follow our Code of Conduct.

[ ] I agree to follow this project's Code of Conduct

Contributing Docs
If you plan on contributing code please read - Contribution Guide

[ ]  I agree to follow this project's Contribution Docs

I'd seriously like to see this work

This is actually blocked until dub.sh ships their API integration, expected this fall. WE could potentially self-hosted, but I would rather prioritize that since we have some other pressing things that need to ship.
Checkout the new URL though https://oss.fy/bdougie


Not possible to merge set calls with Firestore transactions

Merging set calls using transactions is currently not possible in Flutter because we don't have a SetOptions class and parameter for the set() method.
However, the docs stay otherwise:
    /// Writes to the document referred to by the provided [DocumentReference].
    /// If the document does not exist yet, it will be created. If you pass
    /// SetOptions, the provided data can be merged into the existing document.
      Future<void> set(
      ///.....

Issue was also posted on StackOverflow: https://stackoverflow.com/questions/57202980/unable-to-merge-a-transaction-set-with-setoptions
Doctor
[√] Flutter (Channel stable, v1.7.8+hotfix.3, on Microsoft Windows [Version 10.0.17134.885], locale en-US)
    • Flutter version 1.7.8+hotfix.3 at C:\flutter
    • Framework revision b712a172f9 (2 weeks ago), 2019-07-09 13:14:38 -0700
    • Engine revision 54ad777fd2
    • Dart version 2.4.0

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\milen\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[√] Android Studio (version 3.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 32.0.1
    • Dart plugin version 182.5124
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[√] VS Code (version 1.36.1)
    • VS Code at C:\Users\milen\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.2.0

[√] VS Code, 64-bit edition (version 1.23.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.2.0

[√] Connected device (1 available)
    • INE LX1 • 5MG7N18829009925 • android-arm64 • Android 9 (API 28)

• No issues found!

Any update on this?

I have the same problem, would be happy to have a solution...

As workaround right know you can use the get function first, then check if the document exists and if it exists use "set" otherwise "update". But this seems very heavy just to merge a document.

yeah I did the same workaround but this needs to be fixed

Waiting a long time for implementation of SetOptions in Set Transactions in Flutter.

Still waiting for some one from Flutter team to implement this.

Same here, waiting for this feature to be implemented

@mpivchev
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/1212. Any further collaboration will be done there.


Extend dbsync to handle full update/upgrade

Move dbsync to reconcileUpgrade
Move dbsync to a dedicated script
Run upgrade check before dbsync
Use conductor container image for dbsync
Wait 5 seconds between dbsync reconciles
Run online_data_migrations after default dbsync

Jira: OSP-22712

I successfully did a zed->antelope upgrade with this code


Make ESM the default format

Also:

update CI
update deps
remove a build script
move static properties and isTonicComponent getter to the Tonic class
set the version of Tonic on a build step instead of require('package').version in the runtime (which also breaks old esm version)

@chicoxyzzy can you rebase and merge?

@heapwolf this PR removes minified version. Is it ok or we still want to have it? I think I will bring it back to be able to use Tonic via unpkg.com, etc

should this be a major version or minor? technically it has breaking changes

End of an era, no more commonJS


Fix window hide and quit logic

App's hide and quit logic was not properly handled. Attempts to call a function in a renderer window that has been closed or released. The bug can be reproduced on macOS 10.12.5, with Zulip-Electron 0.5.10 (a most recent release).

@zulipbot

Fixed in c2e6e96 by @akashnimare . 🎱


yarn install dies on v8-profiler

Console dump:
╭─jamied@hyperion  ~/dev/shopify-node-app  ‹master›
╰─$ yarn install                                                                     1 ↵
yarn install v1.3.2
[1/5] 🔍  Validating package.json...
[2/5] 🔍  Resolving packages...
[3/5] 🚚  Fetching packages...
[4/5] 🔗  Linking dependencies...
warning "webpack ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>[5/5] 📃  Building fresh packages...
[-/4] ⠐ waiting...
[-/4] ⠐ waiting...
[3/4] ⠐ v8-debug: http GET https://node-inspector.s3.amazonaws.com/debug/v1.0.1/
[4/4] ⠐ v8-profiler: http GET https://node-inspector.s3.amazonaws.com/profiler/v
error /Users/jamied/dev/shopify-node-app/node_modules/v8-profiler: Command failed.
Exit code: 1
Command: node-pre-gyp install --fallback-to-build
Arguments:
Directory: /Users/jamied/dev/shopify-node-app/node_modules/v8-profiler
Output:
node-pre-gyp info it worked if it ends with ok
node-pre-gyp info using<EMAIL_ADDRESS>node-pre-gyp info using<EMAIL_ADDRESS>| darwin | x64
node-pre-gyp info check checked for "/Users/jamied/dev/shopify-node-app/node_modules/v8-profiler/build/profiler/v5.7.0/node-v57-darwin-x64/profiler.node" (not found)
node-pre-gyp http GET https://node-inspector.s3.amazonaws.com/profiler/v5.7.0/node-v57-darwin-x64.tar.gz
node-pre-gyp http 404 https://node-inspector.s3.amazonaws.com/profiler/v5.7.0/node-v57-darwin-x64.tar.gz
node-pre-gyp ERR! Tried to download(404): https://node-inspector.s3.amazonaws.com/profiler/v5.7.0/node-v57-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v57 ABI) (falling back to source compile with node-gyp)
node-pre-gyp http 404 status code downloading tarball https://node-inspector.s3.amazonaws.com/profiler/v5.7.0/node-v57-darwin-x64.tar.gz
node-pre-gyp ERR! build error
node-pre-gyp ERR! stack Error: Failed to execute 'node-gyp clean' (Error: spawn node-gyp ENOENT)
node-pre-gyp ERR! stack     at ChildProcess.<anonymous> (/Users/jamied/dev/shopify-node-app/node_modules/node-pre-gyp/lib/util/compile.js:77:29)
node-pre-gyp ERR! stack     at emitOne (events.js:116:13)
node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:211:7)
node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:196:12)
node-pre-gyp ERR! stack     at onErrorNT (internal/child_process.js:372:16)
node-pre-gyp ERR! stack     at _combinedTickCallback (internal/process/next_tick.js:138:11)
node-pre-gyp ERR! stack     at process._tickCallback (internal/process/next_tick.js:180:9)
node-pre-gyp ERR! System Darwin 16.7.0
node-pre-gyp ERR! command "/usr/local/Cellar/node/8.9.1/bin/node" "/Users/jamied/dev/shopify-node-app/node_modules/v8-profiler/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build"
node-pre-gyp ERR! cwd /Users/jamied/dev/shopify-node-app/node_modules/v8-profiler
node-pre-gyp ERR! node -v v8.9.1

yarn cache clean didn't fix for me.
Removing node-inspector from package.json was the only thing that worked for me.

Looks like node v6.x is the solution to this issue on OS X.
https://github.com/node-inspector/node-inspector/issues/960
But then you cannot install because of error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">= 8.1.0".

Looks like node v6.x is the solution to this issue on OS X.
node-inspector/node-inspector#960
But then you cannot install because of error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">= 8.1.0".


build: refactor repository_locations.bzl

Per slack/issue convo. Opening here to track. The goal here is to make it less confusing to see what versions of things we are using and to update them.
See https://github.com/envoyproxy/envoy/blob/master/bazel/repository_locations.bzl
cc @jmillikin-stripe

Thanks! This'll make it easier to track.
I think it'll be practical to represent the (url, version) for all currently present repos as a dict, since there's only a few combinations. Then it can be applied within repositories.bzl as kwargs. Something like this?
EXTERNAL_DEPENDENCIES = {
  "backward": dict(
    bazel_name = "com_github_bombela_backward",
    remote = "https://github.com/bombela/backward-cpp",
    commit = "cd1c4bd9e48afe812a0e996d335298c455afcd92", # v1.3
  ),
  "fmtlib": dict(
    bazel_name = "com_github_fmtlib_fmt",
    urls = ["https://github.com/fmtlib/fmt/releases/download/4.0.0/fmt-4.0.0.zip"],
    sha256 = "10a9f184d4d66f135093a08396d3b0a0ebe8d97b79f8b3ddb8559f75fe4fcbc3",
    strip_prefix = "fmt-4.0.0",
  ),
  # ...
}

Then a lot of the repetitive boilerplate in repositories.bzl can be eliminated.

@jmillikin-stripe yes +100. My goal is to basically hide the depths of bazel from anyone that is not you or @htuch 😉


fix: expose IamAuthenticator.requestToken as a public method

Fixes: https://github.ibm.com/arf/planning-sdk-squad/issues/2215
This PR exposes IamAuthenticator.requestToken() as a public method, along with the IamTokenServerResponse struct which is returned by requestToken().

Validating these changes based on request in https://github.ibm.com/arf/planning-sdk-squad/issues/2215#issuecomment-24202267 now

:tada: This PR is included in version 4.6.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Use TypedDict for type safety of request data

Ticket
Resolves #52
Resolves #61
Resolves #60
Changes

Add generic PatchParams class
Define CreateUserParams and PatchUserParams TypedDict types to represent
parameters to create_user and patch_user service methods
Remove now unused module api.route.models
Fix issue in apiflask where generated openapi schema fields had nondeterministic order

Context for reviewers
This came out of the todos that were left from PR #51. @zelgadis you and I both tackled this (his PR is #56) and we ended up with a similar approach. I incorporated some of the changes from your PR into this PR, but happy to go with your PR or mine. This PR also fixes a bug in the openapi.yaml generation, where the order of fields isn't deterministic – in a future PR we should just be able to update apiflask and then we can remove the workaround.
Testing
CI

I pulled out the openapi order fix into a separate PR so that this PR can be easier to review once that's merged in: https://github.com/navapbc/template-application-flask/pull/60

ready for a re-review. also, i put up a PR to document this in an ADR: https://github.com/navapbc/template-application-flask/pull/64


SegmentedControl onChange acts differently on iOS vs Android/Web

Hi,
We noticed that pressing the selected item in the segmented control triggers the onChange and onValueChange every time on android or web but it doesn't on iOS. Only pressing an unselected item triggers the onChange.
This behaviour should be the same on both android & iOS.
Tapping an active segment should never call the onChange orValueChange handlers, since there is no change in the state. Perhaps an onPress can be added that triggers the event on every press?
Reproducible example
https://snack.expo.dev/gfEAxSb7j4Pd5uGSENPVw
Notice the difference when running the snack on iOS vs on android

diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index 99bdc2e..488e1b4 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -44,6 +44,7 @@ const SegmentedControl = ({
   const animation = React.useRef(new Animated.Value(0)).current;

   const handleChange = (index: number) => {
+    if(index===selectedIndex) return;
     // mocks iOS's nativeEvent
     const event: any = {
       nativeEvent: {
``` /


Image not immediately displayed on UITableViewCell

Check List

[X] I have read the wiki page and cheat sheet, but there is no information I need.
[X] I have searched in existing issues, but did not find a same one.
[X] I want to reoprt a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
When I use Kingfisher to set images on default UITableViewCells, the images are not immediately displayed after being loaded. Instead, they are displayed the next time the cells are updated in some way (e.g. when you tap a cell). This is especially true for the first launch, i.e. the first time the images are loaded.
I tested it with Xcode 9.2 and iPhone 8 (iOS 11) simulator.
Reproduce
I have created a sample project you can use to reproduce it. Open the workspace and run the app. The images are not immediately displayed. When you tap cells, images are displayed.
Other Comment
Am I missing something?

This seems to be a general iOS issue. I can also reproduce it using AlamofireImage.
However, there is a workaround. It works when you replace
cell.imageView?.kf.setImage(with: url)

with
cell.imageView?.kf.setImage(with: url, placeholder: nil, options: nil, progressBlock: nil, completionHandler: { (image, error, cacheType, URL) in
    cell.setNeedsLayout()
})

But still, isn't it supposed to work without this?

Thank you this solves my problem!

In New Kingfisher use this code:
cell.imageView?.kf.setImage(with: url) { result in
cell.setNeedsLayout()
}


Create a new [Https] attribute that enforces HTTPS on a given endpoint

This is useful for libraries that expose endpoints to enforce that HTTPS is used on the environments they are consumed. For example libraries with UI for authentication flows. The attribute (and associated filter) will send back a 400 on incoming HTTP calls and will possibly allow you to configure a callback to know that this is happening (for example to revoke/blacklist disclosed credentials).

EnforceHttpsAttribute will be an option...

Closing this at the moment, as there doesn't seem yet any customer need. We'll reconsider this, if we'll see customers asking for it.


Try updating Github action to v3, since v2 was long-since deprecated

Changes

Switch from checkout@v2 to checkout@v3

Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows project standards (link if applicable)
[x] Passes all existing automated tests
[x] Any change in functionality is tested
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged / future todos are captured in comments
[x] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[x] Reviewers requested with the Reviewers tool :arrow_right:

Before this PR
https://github.com/NOAA-OWP/ngen/actions/runs/5789781100?pr=606

And no warnings with it


HideOobe for desktop

In Win 10 1803 we are finding when HideOobe for desktop is setup in our provision package, Reset PC will render the device unusable. The document here states a user account must be created - we already have a local Support, and domain users at this point in Reset PC. Removing HideOobe for desktop in the provision package, and adding some hide Oobe nodes in the unattend does provide a workaround, but we did not have this issue when deploying 1709. We are deploying 1803 to all our 1709 devices as an upgrade patch, but this will break Reset PC as these devices have our previous provision package installed with HideOobe for desktop enabled. Any suggestions?

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

ID: 5c8e6f62-b786-e077-8fd9-5c34821a67b0
Version Independent ID: 0e01f920-91e5-0bef-bacf-0b87dfbc4a99
Content: OOBE (Windows 10)
Content Source: windows/configuration/wcd/wcd-oobe.md
Product: w10
GitHub Login: @jdeckerMS
Microsoft Alias: jdecker

@officedocsbot assign @jvsam

Hi @NigelbrownIBM were you able to resolve the conflict between HideOobe and Reset PC? Would you mind letting us know of the status? Please keep us posted here on the resolution or any relevant information that can be called out in the docs. We want users to have the best Windows 10 experience by ensuring the contents of the Microsoft Docs are useful, accurate and up-to-date.
If you are still experiencing the issue—and since this Github site is intended for product documentation—I would recommend contacting support and opening a service ticket in your tenant to get immediate assistance.
Thank you for engaging with the community!

Hi @NigelbrownIBM. We haven't received any response from you. Hopefully, you were able to contact Windows 10 support for assistance. For now, we will close this issue, however, if you believe the documentation needs improvement, please re-open and provide recommendations on what needs to be changed or updated. Again, our main goal here is to ensure users have the best Windows 10 experience by providing useful, accurate and up-to-date documentation.
Thank you for engaging with the community!

@officedocsbot close


Installing Tippecanoe with no access to hosted linux machine

I'm attempting to clone and install Tippecanoe on shared Linux cluster with restricted access to /usr/local/bin and the only option I have is to have it installed on home directory. Do you have any suggestion on how I might be able to use a --prefix option to redirect and compile the package?
g++ -L/usr/local/lib -g -Wall -Wshadow -Wsign-compare -Wextra -Wunreachable-code -Wuninitialized -Wshadow -O3 -DNDEBUG -std=c++11 -o tippecanoe geojson.o jsonpull/jsonpull.o tile.o pool.o mbtiles.o geometry.o projection.o memfile.o mvt.o serial.o main.o text.o dirtiles.o plugin.o read_json.o write_json.o geobuf.o evaluator.o geocsv.o csv.o -lm -lz -lsqlite3 -lpthread
mkdir -p /usr/local/bin
mkdir -p /usr/local/share/man/man1/
cp tippecanoe /usr/local/bin/tippecanoe
cp: cannot create regular file `/usr/local/bin/tippecanoe’: Permission denied
make: * [install] Error 1

FYI, I installed Tippecanoe on Ubuntu on a pc and there was no error.

If you do PREFIX=/home/bin make install it should install to that prefix instead of the default /usr/local.

awsome @ericfisher! It worked.