dynamic KITTI sequence mentioned in your paper seems to be inconsistent.

Hello, after visualizing the dynamic KITTI download link you gave, the qualitative result picture of the dynamic KITTI sequence mentioned in your paper seems to be inconsistent. Did you use other sequences when testing？

Dynamic KITTI is a strict subset of KITTI so all images you find in Dynamic KITTI are copied from KITTI. The image quality should be the same as KITTI as well as images shown in the paper. May I ask what "qualitative result picture" are you referring to?

Hello, attached are the pictures in your paper and the evaluation results on Dynamic KITTI using the evaluation code published on GitHub. The GT lines in KITTI in your paper are inconsistent with the 01,03,04,10 I drew on Dynamic KITTI. Meanwhile, In the given Dynamic KITTI, the sequence like 01 has two folders, 0 and 1. Which folder is used for evaluation?
逐梦
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "castacks/DytanVO"                                                                                @.>;
发送时间: 2024年1月23日(星期二) 上午10:43
@.>;
<EMAIL_ADDRESS>主题: Re: [castacks/DytanVO] dynamic KITTI sequence mentioned in your paper seems to be inconsistent. (Issue #18)
Dynamic KITTI is a strict subset of KITTI so all images you find in Dynamic KITTI are copied from KITTI. The image quality should be the same as KITTI as well as images shown in the paper. May I ask what "qualitative result picture" are you referring to?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Are the KITTI evaluation results in your paper obtained from the KITTI dynamic sequences you gave? I used the code and weights you published to evaluate and got inconsistent results, especially on the sequence of 04_0, it is best to get only 0.85

Have you used the correct model weights for KITTI? On 04_0 which has 36 frames in total, the ATE is 0.0180 using the finetuned model and 0.0206 if loading the flow model and pose model separately. Regarding sequence 01, we report the result of 01_1 in our paper, the one with 133 frames in total. For the other sequences, we use 00_1 (28 frames), 01_1 (133 frames), 02_1 (67 frames), 03_0 (31 frames), 04_0 (36 frames), 07_1 (136 frames), 08_0 (40 frames), 10_1 (59 frames).
If you still get inconsistent results, please provide instructions on how to reproduce the result you got.

Thank you very much

Thanks to @Xie-HIT, the reason might be found, which is on our end. Please refer to https://github.com/castacks/DytanVO/issues/20#issuecomment-1998504693


Improve dandiset description

Their other recent publication of the visual behavior dataset is a good example of what the final product should look like: https://dandiarchive.org/dandiset/000711/0.231121.1730
But will include (a) better AllenSDK documentationc (b) working links to white papers (those supposed to be att  have been down for a while...), (c) preferably DOI's for those papers, or for others related to this project

Saskia said they would take this one; just need to remind them in follow-up email

No response


implement fileupload

well it creates a new record but ignores the file, pocketbase doesnt throw any error or anything related to the file in the logs


Failed to set up: Access is denied for this resource

I have the latest stable version of HA and previously this year (or maybe the end of the last?) I had a working installation of this NissanConnect component by uploading the individual files.
Later I experienced something was broken and - details are unclear - after a while I reinstalled the component. When reconfiguring, it gave me the error: "Access is denied for this resource"
I believe the error came at about the same time as my personal phone needed an update of the NissanConnect Services app, but my personal Android is too old and no longer supported. A friend's iPhone runs the app however, where login and Car connection works.
Either way, I shook my head and waited a couple of months, to see if it magically solved itself. But here I am now, experiencing the same issue. App login works and has been re-tested before this post.
HA component login validation works, so I am using the correct credentials. I verify this by noting that the component refuses to set up if I supply an incorrect password.
With a successful login, and the component gives me a success message, it goes on to setting up. In this phase, I am informed about an error "Failed to set up" and I am informed I should check the logs.
Starting debug logs, I get the following:

2024-06-16 08:42:31.861 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration nissan_connect which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-06-16 08:54:53.356 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]
2024-06-16 08:55:04.402 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]
2024-06-16 08:55:42.964 INFO (MainThread) [custom_components.nissan_connect] Logging in to service
2024-06-16 08:55:43.535 DEBUG (MainThread) [custom_components.nissan_connect] Finding vehicles
2024-06-16 08:55:44.531 DEBUG (SyncWorker_29) [custom_components.nissan_connect.kamereon] Active features: [<Feature.MY_CAR_FINDER: '12'>, <Feature.MIL_ON_NOTIFICATION: '15'>, <Feature.VEHICLE_HEALTH_REPORT: '18'>, <Feature.APP_DOOR_LOCKING: '27'>, <Feature.FLEET_ASSET_INFORMATION: '29'>, <Feature.PAY_AS_YOU_DRIVE: '34'>, <Feature.APP_BASED_NAVIGATION: '35'>, <Feature.DESTINATION_SEND_TO_CAR: '37'>, <Feature.GOOGLE_PLACES_SEARCH: '40'>, <Feature.PREMIUM_TRAFFIC: '43'>, <Feature.THEFT_BURGLAR_NOTIFICATION_APP: '87'>, <Feature.HORN_AND_LIGHTS: '97'>, <Feature.CURFEW_ALERT: '98'>, <Feature.NAVIGATION_MAP_UPDATES: '217'>, <Feature.LAST_MILE_NAVIGATION: '227'>, <Feature.GOOGLE_STREET_VIEW: '229'>, <Feature.GOOGLE_SATELITE_VIEW: '230'>, <Feature.DRIVING_JOURNEY_HISTORY: '235'>, <Feature.ONLINE_PARKING_INFO: '243'>, <Feature.ONLINE_RESTAURANT_INFO: '244'>, <Feature.ONLINE_SPEED_RESTRICTION_INFO: '245'>, <Feature.WEATHER_INFO: '246'>, <Feature.ZONE_ALERT: '281'>, <Feature.SPEEDING_ALERT: '282'>, <Feature.CHARGING_START: '299'>, <Feature.CLIMATE_ON_OFF_NOTIFICATION: '311'>, <Feature.PLUG_IN_REMINDER: '314'>, <Feature.CHARGING_STOP_NOTIFICATION: '317'>, <Feature.BATTERY_STATUS: '319'>, <Feature.BATTERY_HEATING_NOTIFICATION: '320'>, <Feature.VEHICLE_STATE_OF_CHARGE_PERCENT: '322'>, <Feature.DRIVING_ANALYSIS: '340'>, <Feature.CO2_GAS_SAVINGS: '341'>, <Feature.ELECTRICITY_FEE_CALCULATION: '342'>, <Feature.CHARGING_CONSUMPTION_HISTORY: '344'>, <Feature.CHARGING_SPOT_UPDATES: '354'>, <Feature.CLIMATE_ON_OFF: '366'>, <Feature.LOCK_STATUS_CHECK: '2021'>, <Feature.TEMPERATURE: '2042'>]
2024-06-16 08:55:45.062 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]
2024-06-16 08:57:38.721 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]
2024-06-16 09:00:09.518 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]
2024-06-16 10:54:01.129 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]
2024-06-16 10:55:14.541 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]
2024-06-16 10:55:22.140 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]
2024-06-16 10:55:24.478 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]
2024-06-16 10:55:58.614 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]
2024-06-16 10:56:03.748 INFO (MainThread) [custom_components.nissan_connect] Logging in to service
2024-06-16 10:56:04.327 DEBUG (MainThread) [custom_components.nissan_connect] Finding vehicles
2024-06-16 10:56:05.283 DEBUG (SyncWorker_41) [custom_components.nissan_connect.kamereon] Active features: [<Feature.MY_CAR_FINDER: '12'>, <Feature.MIL_ON_NOTIFICATION: '15'>, <Feature.VEHICLE_HEALTH_REPORT: '18'>, <Feature.APP_DOOR_LOCKING: '27'>, <Feature.FLEET_ASSET_INFORMATION: '29'>, <Feature.PAY_AS_YOU_DRIVE: '34'>, <Feature.APP_BASED_NAVIGATION: '35'>, <Feature.DESTINATION_SEND_TO_CAR: '37'>, <Feature.GOOGLE_PLACES_SEARCH: '40'>, <Feature.PREMIUM_TRAFFIC: '43'>, <Feature.THEFT_BURGLAR_NOTIFICATION_APP: '87'>, <Feature.HORN_AND_LIGHTS: '97'>, <Feature.CURFEW_ALERT: '98'>, <Feature.NAVIGATION_MAP_UPDATES: '217'>, <Feature.LAST_MILE_NAVIGATION: '227'>, <Feature.GOOGLE_STREET_VIEW: '229'>, <Feature.GOOGLE_SATELITE_VIEW: '230'>, <Feature.DRIVING_JOURNEY_HISTORY: '235'>, <Feature.ONLINE_PARKING_INFO: '243'>, <Feature.ONLINE_RESTAURANT_INFO: '244'>, <Feature.ONLINE_SPEED_RESTRICTION_INFO: '245'>, <Feature.WEATHER_INFO: '246'>, <Feature.ZONE_ALERT: '281'>, <Feature.SPEEDING_ALERT: '282'>, <Feature.CHARGING_START: '299'>, <Feature.CLIMATE_ON_OFF_NOTIFICATION: '311'>, <Feature.PLUG_IN_REMINDER: '314'>, <Feature.CHARGING_STOP_NOTIFICATION: '317'>, <Feature.BATTERY_STATUS: '319'>, <Feature.BATTERY_HEATING_NOTIFICATION: '320'>, <Feature.VEHICLE_STATE_OF_CHARGE_PERCENT: '322'>, <Feature.DRIVING_ANALYSIS: '340'>, <Feature.CO2_GAS_SAVINGS: '341'>, <Feature.ELECTRICITY_FEE_CALCULATION: '342'>, <Feature.CHARGING_CONSUMPTION_HISTORY: '344'>, <Feature.CHARGING_SPOT_UPDATES: '354'>, <Feature.CLIMATE_ON_OFF: '366'>, <Feature.LOCK_STATUS_CHECK: '2021'>, <Feature.TEMPERATURE: '2042'>]
2024-06-16 10:56:05.845 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
result = await component.async_setup_entry(hass, self)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/init.py", line 54, in async_setup_entry
await hass.async_add_executor_job(vehicle.refresh)
File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/nissan_connect/kamereon.py", line 863, in refresh
self.fetch_all()
File "/config/custom_components/nissan_connect/kamereon.py", line 866, in fetch_all
self.fetch_cockpit()
File "/config/custom_components/nissan_connect/kamereon.py", line 1327, in fetch_cockpit
raise ValueError(body['errors'])
ValueError: [{'status': 'Forbidden', 'code': '403', 'title': 'security.access', 'detail': 'Access is denied for this resource'}]

Home Assistant Version
Home Assistant
Core 2024.6.3
Supervisor 2024.06.0
Operating System 12.3
Frontend 20240610.1
Integration Version
Zip download today. Release was made 3 days ago at time of writing:
@dan-r dan-r
 v0.5.2
https://github.com/dan-r/HomeAssistant-NissanConnect/commit/a09d16b6d955adf2c38d49d72ea7913f5f538ef7

Everything seems to be working with my NissanConnect account. Have you changed your password lately? If so you'll need to change this on the integration.
I'd also recommend installing HACS to keep your custom integrations up to date

Everything seems to be working with my NissanConnect account. Have you changed your password lately? If so you'll need to change this on the integration.

Hello, thanks for the quick response.
I have reset my password through the app with NissanConnect Services, removed and readded the entry in the integration configuration on /config/integrations/integration/nissan_connect
So I don't believe the issue is with the authentication credentials, as this is logged:

2024-06-16 08:55:42.964 INFO (MainThread) [custom_components.nissan_connect] Logging in to service
2024-06-16 08:55:43.535 DEBUG (MainThread) [custom_components.nissan_connect] Finding vehicles
2024-06-16 08:55:44.531 DEBUG (SyncWorker_29) [custom_components.nissan_connect.kamereon] Active features: [<Feature.MY_CAR_FINDER: '12'>, <Feature.MIL_ON_NOTIFICATION: '15'>, <Feature.VEHICLE_HEALTH_REPORT: '18'>, <Feature.APP_DOOR_LOCKING: '27'>, <Feature.FLEET_ASSET_INFORMATION: '29'>, <Feature.PAY_AS_YOU_DRIVE: '34'>, <Feature.APP_BASED_NAVIGATION: '35'>, <Feature.DESTINATION_SEND_TO_CAR: '37'>, <Feature.GOOGLE_PLACES_SEARCH: '40'>, <Feature.PREMIUM_TRAFFIC: '43'>, <Feature.THEFT_BURGLAR_NOTIFICATION_APP: '87'>, <Feature.HORN_AND_LIGHTS: '97'>, <Feature.CURFEW_ALERT: '98'>, <Feature.NAVIGATION_MAP_UPDATES: '217'>, <Feature.LAST_MILE_NAVIGATION: '227'>, <Feature.GOOGLE_STREET_VIEW: '229'>, <Feature.GOOGLE_SATELITE_VIEW: '230'>, <Feature.DRIVING_JOURNEY_HISTORY: '235'>, <Feature.ONLINE_PARKING_INFO: '243'>, <Feature.ONLINE_RESTAURANT_INFO: '244'>, <Feature.ONLINE_SPEED_RESTRICTION_INFO: '245'>, <Feature.WEATHER_INFO: '246'>, <Feature.ZONE_ALERT: '281'>, <Feature.SPEEDING_ALERT: '282'>, <Feature.CHARGING_START: '299'>, <Feature.CLIMATE_ON_OFF_NOTIFICATION: '311'>, <Feature.PLUG_IN_REMINDER: '314'>, <Feature.CHARGING_STOP_NOTIFICATION: '317'>, <Feature.BATTERY_STATUS: '319'>, <Feature.BATTERY_HEATING_NOTIFICATION: '320'>, <Feature.VEHICLE_STATE_OF_CHARGE_PERCENT: '322'>, <Feature.DRIVING_ANALYSIS: '340'>, <Feature.CO2_GAS_SAVINGS: '341'>, <Feature.ELECTRICITY_FEE_CALCULATION: '342'>, <Feature.CHARGING_CONSUMPTION_HISTORY: '344'>, <Feature.CHARGING_SPOT_UPDATES: '354'>, <Feature.CLIMATE_ON_OFF: '366'>, <Feature.LOCK_STATUS_CHECK: '2021'>, <Feature.TEMPERATURE: '2042'>]
2024-06-16 08:55:45.062 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry NissanConnect Account for nissan_connect
Traceback (most recent call last):
etc.

So it feels more like I make an unauthorized request after a successful login.

FWIW, there is this commit in dartnissanconnect. Could it be by chance related?

That's been implemented under this commit so shouldn't be the problem.
It seems weird because it's authenticated enough to get the list of features, but then seems to fail...

That's been implemented under this commit so shouldn't be the problem.
It seems weird because it's authenticated enough to get the list of features, but then seems to fail...

I can confirm that this commit is also on my HA platform.
I will setup HACS to avoid second-guessing the file versions, then return here with the results. :)

Did you have any luck with this?


Add link of this repository on the website

A button can be created which will link to this GitHub repository for visitors who want to contribute to the website.

@Maaz-Code That's a great idea. We could even add a projects page in the future. Do you want to add a link on the information page for now, then we can add a projects page issue later?

Yes, I can add the repo link on the information page for now and the projects page can be added later.

I am not able to run the project on local machine as the nvm is not installing the correct node version

@Maaz-Code Were you able to get the issue resolved?

I'm hoping this might be helpful to you.
https://github.com/coreybutler/nvm-windows/releases

I think you can just extract the link itself and place it below where the mapping function ends in the information.jsx file.
And you can use this to see how MUI handles links:
https://mui.com/components/links/#main-content


Add source_provider setting for edit link (#35 proposal)

This patch allow to use source_provider to handle the case where source_directory is not a well-known url (https://gitlab.com, https://github.com, https://bitbucket.org).
This patch implies that project using spinx_basic_ng update their theme.conf file to add a source_directory = default setting (else this patch trigger errors at runtime). Not sure if it is possible and/or desired to handle the case where theme.conf is not updated.

Would this be better served by https://github.com/pradyunsg/sphinx-basic-ng/pull/34?

My bad, #34 is just fine; I just do not see it before posting :-/


Updated js dependency

Make allow to use package JS 0.7.x version

@balvinderg please take a look

On vacation till 16th march. Will check after that

Thanks for contributing @dkrutskikh  :D

Published in 1.1.1.


Use the mobile phone number and verification code to log in

Expected Behavior

It is hoped that mobile phone number and verification code can be used for login in the future
Current Behavior

Most websites in China support the system login by using the mobile phone number and the verification code sent by the service provider. I hope it can be supported in the native framework. Before this, we need to extend the original framework in the development process to support it
Context

之前用老项目扩展了下手机验证码登录，真的是好啰嗦啊。希望可以直接提供下。

This project aims to provide oauth support, basic authentication workflow is out of the project's scope.
You should open this issue to spring security

I am not expert of this project, but in spring you can customize many things, here you need custom login form and custom filter and authentication provider?

Thanks for the suggestion @robintian-bud. However, @edwardzjl is correct, this project strictly focuses on OAuth and OpenID Connect support only.
Please see the following for further information:

spring-security#2603
spring-security#6842
spring-security-samples#7


Use carbon to generate the image

https://github.com/carbon-app/carbon

We mustn't forget that the code image is also used as the social image for sharing snippet.

I am closing this for now, it's far easier to copy/paste the image in the project resources.


feat: table support

Describe the feature you'd like to request
table support
Describe the solution you'd like to see
table support
Additional information
No response

yes

I've tried to use the tiptap tables extensions but I fail :(

+1


Some text need to manual reload the translation to be translated

like the title say. Only some text are require to manual pressing reload not all of them.

Have you tried changing the option:
"ReloadTranslationsOnFileChange=False" to
"ReloadTranslationsOnFileChange=True"


no nodes

I have done installation correct, but cascade nodes not existing inside the ComfyUI

This issue would go into the comfyui repo.


Support for Windows PowerToys (Specifically Peek)

What feature or improvement do you think would benefit Files?
As a previous Mac user, I used to use the peek feature constantly. On Windows, I use it through PowerToys but would love to combine it with column view
Requirements
Holding down Ctrl + Space to show a peek of the desired content, as well as other PowerToys content
Files Version
<IP_ADDRESS>
Windows Version
Edition	Windows 11 Pro Version	22H2 Installed on	14/11/23 OS build	22621.2715 Experience	Windows Feature Experience Pack 1000.22677.1000.0
Comments
No response

Thanks for the feedback, Peek doesn't have an API to allow Files to use it. This is being tracked here https://github.com/microsoft/PowerToys/issues/26422


JSS URL: Unable to Connect with trailing slash

When entering the JSS URL in the utility, if you use the trailing slash such as "https://jss.acme.com:8443/" the utility will be unable to connect to the jss with an error of "Unable to connect or login to the JSS (see attached screenshot).

But if you remove the slash, such as "https://jss.acme.com:8443" the utility will connect as expected.

This has been fixed, and will be included in the next beta. Thanks for the feedback!


One-way bindings of arrays behave inconsistently when a default value is set in the constructor

Do you want to request a feature or report a bug?
Bug.
What is the current behavior?
One-way bindings of arrays behave inconsistently when a default value is set in the constructor.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar (template: http://plnkr.co/edit/tpl:yBpEi4).
Minimal JSFiddle example.
childCtrl.list2 is set but childCtrl.list1 never is.  The changes argument to $onChanges(changes) contains both values, however.
What is the expected behavior?
Perhaps I made a terrible mistake by trying to set default values (list1=[] and list2=[]) in my child controller's constructor. Probably that's my fault, and list1 being empty is expected. But the problem did take me longer to track down because some previous code (like list2) had worked just fine.
What is the motivation / use case for changing the behavior?
I'd have expected list1 and list2 to behave the same way here. If setting a default in the constructor is bad and should never be done, then maybe no change is needed.
Which versions of Angular, and which browser / OS are affected by this issue? Did this work in previous versions of Angular? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
v1.5.8, v1.5.9
Other information (e.g. stacktraces, related issues, suggestions how to fix)
Thanks!

The current implementation in Angular has been assigning bindings to the controller prior to your constructor being called;  you were overwriting the binding that was passed in.
Once https://github.com/angular/angular.js/pull/15095, this behavior will be optionalized and you can opt-in to having the bindings initialized later, so you can safely set defaults in the constructor.  That and since you weren't specifying your one-way bindings as optional, any defaults you did set would be overwritten even if no binding was provided.

@dcherman, I think this is a different problem (which is also highlighted by the difference in behavior between list1 and list2. I will add a more detailed answer soon.

Actually, I meant there is not noly that. What @dcherman mentioned above explains why the list1 binding is overwritten by the default value. But there is more going on (that causes list2 to be set to the "correct" value).

So, the fact that list2 shows correctly is (kind of) a bug :smiley:
The main takeaway (for now) should be that if you want to set default values, you should set them either in the first call of $onChanges or in the $onInit hook (in both cases only if there are no values present):
// In child
this.$onInit = function $onInit() {
  if (!this.list1) this.list1 = [];
  if (!this.list2) this.list2 = [];
};

--
For (self) reference and the curious:
When we initialize the bindings on the child controller (which happens before actually instantiating the controller), we assign the initial values for list1/list2 to the controller instance (here) - yes, there is an instance even if we haven't instantiated the controller yet (don't ask :smiley:) - collect the initial changes for list1/list2 (here) and set up a watch on them (here). The collected initial changes will be used to fire an initial $onChanges call.
At this point, child.list1 == child.list2 == [1, 2, 3].
Then, we instantiate the controller (i.e. run the constructor in the context of the instance), so the default values overwrite the alreayd set bindings.
At this point, child.list1 == child.list2 == [].
Later, when the watcher fires for the first time, it will compare the current values with those initial values:

If they are the same (in the === sense), it will do nothing. This is what happens with list1. It remains at [].
If they have changed, it will assign the new values to the child controller (and schedule an $onChanges). This is what happens to list2. It gets re-assigned to [1, 2, 3].

The reason why initialList1BindingValue === list1FromFirstWatch is that it refers to parent.list, which hasn't changed.
The reason why initialList2BindingValue !== list2FromFirstWatch is that the list2 binding is a literal expression, which means that a new array will be created with every digest. (The new array will contain the exact same elements, but it wil be a different array object.) Thus the === comparison fails.

Note, subsequent watch evaluations will not run into the same issue, because we set up a deep watcher for literals, but it seems that we missed that on the comparison here. <-- This is the bug I referred to earlier.

We should probably change this line like this:
-if (oldValue === initialValue) return;
+if (parentGet.literal ? equals(oldValue, initialValue) : oldValue === initialValue) return;

Thanks for looking into this! I appreciate the detailed answer. The bug was harmless enough, so leaving it alone seems fine to me.

There is no such thing as "harmless bug" :stuck_out_tongue:
(Submitted a fix in #15123.)

Awesome, thanks! 😄


Return static 500.30 file on startup fails for in-process

For #556 .
There was a duplicated check for validating the status of the application in the shim. I removed this as the application should be the one handling setting the http message and status.
This does create a drawback of us needing to allocate an INPROCESS_HANDLER before returning the error page.

🆙 📅 . Instead of returning the exception in the inprocess application, we create a new application that constantly returns 500.30s.

Can we fix the flaky CI  build /tests?   A lot of time,  I don't know the Whether  PR had issue or not and should I look at a PR which has CI failure.

This is my failure. Investigating.

Can we fix the flaky CI build /tests? A lot of time, I don't know the Whether PR had issue or not and should I look at a PR which has CI failure.

Tests don't seem to flaky, it's just PRs are breaking them.

@pakrym, @pan-wang: We have seen some flaky tests in IISIntegration: https://github.com/aspnet/iisintegration/issues?utf8=✓&q=is%3Aissue+is%3Aopen+flaky.  We do need to fix them eventually but they may not currently be the highest priority.

@dotnet-bot test this please

Current code path works fine on IISExpress and IIS during recycle.

I'll check appoffline scenarios too.

There seems to be larger issues that should be handled in other PRs, including #950 and #949 . This handles all scenarios we expect it to.


Chacha20 problem

The following snippet reads a file, encrypts and writes it back with .cha20 extension. Given a file for encryption, and the result back again for decryption, the original file content is truncated.
chacha20 = require "chacha20"
sha3 = require "sha3"

f = io.open(arg[1], "r")
plaintext = f:read("*all")
f:close()

key = sha3.hash256("Super Password 123 XXX")
nonce = "\x00\x00\x00\x00\x00\x00\x00\x4a\x00\x00\x00\x00"
assert(#key == chacha20.key_size)
assert(#nonce == chacha20.nonce_size)

counter = 1
ciphertext = chacha20.encrypt(key, counter, nonce, plaintext)

f = io.open(arg[1] .. ".cha20", "w")
f:write(ciphertext)
f:close()

LICENSE - 1070 bytes
LICENSE.cha20 - 1071 bytes
LICENSE.cha20.cha20 - 234 bytes

Solved.


Feature request - Allow Admins To Lockdown Site If Hacked

There should be a way to lockdown the site if currently being hacked. Maybe through the django admin dashboard where only admin staff/superuser users could login and make changes to it. Perhaps with a database 1 (on) or 0 (off) setting that updates the same row every time without adding new rows. Also make a setting to turn this feature and the database on or off. Do this without SSH into the server to stop gunicorn or the runserver command. This way, your site could still be running on the server and could help you further mitigate/understand what the hack is to better help you to try and stop it.
This is similar to if someone steals your credit card and tries to buy something, lock them out and prevent them from going on a buying spree.
LOCKDOWN_URL_EXCEPTIONS = (
    r'^/admin/.*$',
)

Or maybe it is best to just SSH and change LOCKDOWN_ENABLED to False in settings.py? Maybe there is a way to do it without a database table? Also wondering if it would be worth using a database or not for other settings to update/add to them?

Thanks for your request. After thinking a bit about it I wonder how useful that would be, as an attacker with write access to a website might just replace arbitrary parts of it. Therefore he'd be able to disable the lock down or replace the Django installation completely as well. What are your thoughts about that?

Write access to the linux server itself do you mean? I see your point if that is the case. It all really depends on if the system admin takes security seriously in the first place. If they don't have good security defenses setup, and they are somehow able to gain root access, they could very well cause some damage to the django installation and do whatever they want and disable the lockdown program as an example.
One way to protect the admin login page is to use django-otp to enable two factor authentication where only true admin users can login. Even if a hacker has the admin's username and password, they still won't be able to login because they need the 2FA device. But if they have root access, they could still disable the django-otp 2FA for the admin dashboard if they edit settings.py. Then even a feature like this would be useless.
However, if a system admin takes security seriously, then something like this will only add to their toolbelt. Things like Docker isolation and following something like https://www.amazon.com/Mastering-Linux-Security-Hardening-intruders/dp/1838981772/ref=sr_1_1?dchild=1&qid=1597444083&refinements=p_27%3ADonald+A.+Tevault&s=books&sr=1-1&text=Donald+A.+Tevault come to mind.
Another use case for this type of feature is to lockdown your whole site if you are undergoing maintenance mode.

One of the core principles of django-lockdown has always been that the configuration for locked pages is done in a configuration file on disk. That has to stay available to keep backwards compatibility. What your feature request is essentially about is providing the ability to configure django-lockdown using the Django admin interface as well. However I currently don't see a way to have both, configuration file and admin interface, as I see no good way to keep settings made by either of them in sync with the other, as changes in the admin interface shouldn't modify with Django's configuration file. Do you have an idea how to solve that?

I'm closing this now, as there hasn't been further discussion in a while.


1.21 OST issue?

If I play a level from OST V or the FOB pack, notes become fake but damaging after a little bit. Causing me to fail.
If I fail on an OST level, the game locks and I have to close it. (Saw with Magic, Thnks fr th Mmrs and Escape)
Here's the log if it helps at all _latest.log
Mod version: 2.1.11
https://user-images.githubusercontent.com/57236015/161568017-882c617b-fbb8-4a94-b3da-311ed7f581ab.mp4

Experienced the same problem.

Should be fixed in 2.1.12.


[JN-1298] Ability to add roles to users via UX

DESCRIPTION (include screenshots, and mobile screenshots for participant UX)
This has the main purpose of allowing users to specify roles when creating new users.  However, in order to get there, we needed to start auditing admin user stuff.
We'll need to put some thought into role visibility, and how we let users add/remove roles.  Right now, the main concern is not letting users add roles they don't already have, since we'll be using roles to hide features we don't want to expose to certain customers yet.  But they can still see all the roles since we don't mind (and in fact welcome) questions about them.  Later, we might want to add logic hide roles that users/portals don't have access to.
Setting roles for new users

Updating roles of an existing user can be done from either the edit button on the list page, or an "edit roles" button in the detail view.  Note that you can only access the update function from the Portal-specific users list.  The superuser users list (accessed from the "superuser functions" nav list) does not support this function yet since it's operating at a different scope.

TO TEST:  (simple manual steps for confirming core behavior -- used for pre-release checks)

redeploy ApiAdminApp
go to  https://localhost:3000/demo/studies/heartdemo/users
click the 'edit' icon next to one of the users
add/remove roles and click update
confirm roles are saved
logout and log back in as<EMAIL_ADDRESS>go to https://localhost:3000/demo/studies/heartdemo/users
Edit<EMAIL_ADDRESS>Try to add the 'publish' role to yourself.  It should error -- you can't add roles you don't have
Generally click around the user admin UX and do stuff (add/delete users, etc...) -- this PR changed a lot of the underlying mechanics

thanks for the great feedback, Matt


Fix cut-off session titles

Session titles were being cut off due to the padding for the buttons that appear when you're signed in and can express your interest.

Coverage remained the same at 52.482% when pulling 2dd72dfc241c3644f86047e4156fcf735bd1da47 on tonyc:session_title_css_tweak into c4e238012e750345a7f7190eb8dd4f0c5a631c44 on minnestar:master.


Win10 64bit Read timeout

When try to add a new device, type Passphrase, click continue button but System Crash displaying a message: Keybase: Read timeout. Then at new device error displayed: Unknown error: kex canceled by caller. @zanderz keybase log sent

If the crashed system is still in this state, can you issue "keybase log send" and attach the resulting ID? Thanks.

Keybase Log screenshot


[ASV] performance regression in value_counts operation

Pandas_on_ray engine - 44 cpus

Benchmark
Commit
Factor
Before
After

benchmarks.TimeValueCountsSeries.time_value_counts((1000000, 10), 100, None)
09d7c180
2.18x
138.171ms
300.928ms

benchmarks.TimeValueCountsSeries.time_value_counts((5000, 5000), 100, None)
09d7c180
1.99x
138.166ms
275.188ms

Feel free to reassign.

Hi @anmyachev - do these performance regressions still appear on master?

Results are not actual.


Can we get you pretrained model?

Hello guys,
I had requested your pre-trained model from the google drive access link. However, I got no responses.
Is it possible to get pre-trained model which was used to trained this pipeline?
Thanks

The link to the PyTorch model can be found in the pytorch folder README. https://drive.google.com/open?id=1e5c7308fdoCMRv0w-XduWqyjYPV4JWHS


Change the config structure

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

I see that this structure is confusing, as folders are repeated within the configurations.

App folder for bootstrap settings and app events.
Definition folder for setting up di, adding delegates and providers.
Folders in the console and web for settings for their groups.
array_merge(common/application, console/application or web/application)
array_merge(common/defition, console/definition or web/definition)

The old structure was also confusing, this is my attempt to fix it


provide more context when sending money to a username

There has been cases when money has been sent to the wrong account. I guess this is the convenience of instant e-money is that now it doesn't take long to send a payment to the wrong account that has a similar name.
One way we can help to prevent that is by providing a bit more context when sending money to a username. One idea: check if the user has already send something to that username or not? if not, we should have maybe a popup saying something like "this is the first time you're sending money to this account. make sure you verify this the right account before proceeding".

For now this can be done by cross-referencing with previous transactions, but once we paginate transactions we may need a new graphql endpoint to make the query.


Bus factor / Long-term sustainability and maintainability

Are there any concrete plans in case Vaxry steps away from the project?
https://en.wikipedia.org/wiki/Bus_factor
I really love Hyprland and I don't know if I will still have the will to live if it gets abandoned.

I will still have the will to live
Woah, no need to be so dramatic about a piece of software, even a good one.

no plans atm
Hope that answers the question


Jiahua

9.21 11am

This isn't essential for reproducing the paper results, but the file eda.ipynb is kept elsewhere for posterity


Fix tox -e upload by separating build and upload steps

For some reason, tox -e build -e upload started failing because the upload step would run before the build step. As it turns out, you can't upload something that doesn't exist yet. Fix this by separating the build and upload steps.

Tests performed here: https://github.com/michael-the1/diepvries/actions/runs/4303167950/jobs/7502564342 which succesfully uploaded to here https://test.pypi.org/project/deeperfreezer/0.6.4/#history


Add flag to enable Socks proxy configuration in the cluster section of the kubeconfig

What does this PR do?
Add support for socks proxy configuration in the cluster section of a kubeconfig created by the login command
What is the effect of this change to users?
When the flag --proxy is specified a config like like
proxy-url: socks5://localhost:9000
will be added to the to the cluster configuration for the cluster that is being logged in
What does it look like?
clusters:
- cluster:
    certificate-authority-data: DATA+OMITTED
    proxy-url: socks5://localhost:9000
    server: https://apiserver.cluster1.example.com:6443
  name: cluster1

Any background context you can provide?
In some cases we need to access apiserver through an ssh bastion host, using an SSHD socks proxy is a quick and convenient way to do this
What is needed from the reviewers?
Do the docs need to be updated?
Should this change be mentioned in the release notes?

[x] CHANGELOG.md has been updated

Is this a breaking change?
(Breaking changes are, for example, removal of commnds/flags or substantial changes in the meaning of a flag. If yes, please add the breaking-change label to the PR.)

@anvddriesch

However, would be good to specify that this only affects WC client cert creation :)

Do we support other authentication types ? i can add the options to those as well , i did not know :+1:

@anvddriesch added proxy support to MC Oidc contexts as well, please review :pray:

yes, there is an issue though
It works as expected for

normal MC login

➜ ./kubectl-gs login https://g8s.gauss --cluster-admin

➜ kubectl config view --minify | yq .clusters
- cluster:
    certificate-authority-data: DATA+OMITTED
    server: https://g8s.gauss:443
  name: gs-gauss

MC with proxy

➜ ./kubectl-gs login https://g8s.gauss --cluster-admin --proxy

➜ kubectl config view --minify | yq .clusters
- cluster:
    certificate-authority-data: DATA+OMITTED
    proxy-url: socks5://localhost:9000
    server: https://g8s.gauss:443
  name: gs-gauss

WC normal

➜ ./kubectl-gs login https://g8s.gauss  --cluster-admin --workload-cluster qh64q

➜ kubectl config view --minify | yq .clusters
- cluster:
    certificate-authority-data: DATA+OMITTED
    server: https://api.qh64q:443
  name: gs-gauss-qh64q-clientcert

WC with proxy
It fails when using the proxy flag now that is also added for the MC ( it worked when it was only for clientcert ) because the same --proxy flag is used for both mc and wc , meaning that the MC context is updated to use the proxy which in 99.9% of cases won't make sense

➜ ./kubectl-gs login https://g8s.gauss --cluster-admin --workload-cluster qh64q --proxy

Error: Get "https://g8s.gauss:443/api?timeout=32s": proxyconnect tcp: dial tcp [::1]:9000: connect: connection refused

I can

Have a different flag for mc and wc proxy , possibly keep the --proxy as a common catch for both
revert the support for the proxy in the MC authentication

what do you think @anvddriesch  ?

Hmmm I guess for the WC client-cert creation (if --workload-cluster flag is used) we now need to consider the following cases:

MC with proxy, WC without
MC without proxy, WC with proxy
MC and WC with proxy
Unless some of these cases simply don't happen, we are going to need two flags.
Is there a use case for the proxy flag with OIDC? Because if not, I would suggest to indicate that the flag is for WC client cert only and remove it from the OIDC part for the time being.

Is there a use case for the proxy flag with OIDC? Because if not, I would suggest to indicate that the flag is for WC client cert only and remove it from the OIDC part for the time being.

i would also prefer antonia's suggestion

yeah sounds good to me

reverted to only work on WC when using clientcert


Feat/default bench

Coverage decreased (-9.4%) to 77.143% when pulling 108e730d19a0697962c384b18e98acfca84aaf9d on feat/default-bench into 36016ee532feeaf87adea94b1573c8a76d98db93 on master.

Coverage decreased (-10.4%) to 76.228% when pulling d88743853100fe7b5d722fc86017e4b186e901f4 on feat/default-bench into 0ff4f97e6c1bdd028e7e64772ea51c95b5e8d18d on master.

Coverage decreased (-0.5%) to 86.154% when pulling fe8895a26112051f0f30ddfc8e93300c94126eb7 on feat/default-bench into 0ff4f97e6c1bdd028e7e64772ea51c95b5e8d18d on master.

Coverage decreased (-0.5%) to 86.154% when pulling fe8895a26112051f0f30ddfc8e93300c94126eb7 on feat/default-bench into 0ff4f97e6c1bdd028e7e64772ea51c95b5e8d18d on master.

Coverage decreased (-0.4%) to 86.105% when pulling 7246c8e00748645e12a2b29934b91b7414d0027c on feat/default-bench into acca633a037d2354413565bd6d1593377c9c14d5 on master.


Apply HostFilter for fetch_primary

When use capistrano with rails migration task, it will fetch the primary db server to run the migration. If I use HostFilter which does not include the primary db server. A "Directory Not Exists" error will occure.
So I think add the HostFilter to the fetch_primary method will help, and it does(for my situation).
How to reproduce
# deploy.rb
server "server1", roles: %w/app db web/
server "server2", roles: %w/app web/

Deploy with hosts filter
cap production deploy --hosts=server2

err logs
** Invoke deploy:migrate (first_time)
** Invoke deploy:set_rails_env
** Execute deploy:migrate
DEBUG [59ff49e4] Running /usr/bin/env if test ! -d /home/oob/apps/alioob/releases/20140528095735; then echo "Directory does not exist '/home/oob/apps/alioob/
releases/20140528095735'" 1>&2; false; fi on server1
DEBUG [59ff49e4] Command: if test ! -d /home/oob/apps/alioob/releases/20140528095735; then echo "Directory does not exist '/home/oob/apps/alioob/releases/201
40528095735'" 1>&2; false; fi
DEBUG [59ff49e4]        Directory does not exist '/home/oob/apps/alioob/releases/20140528095735'

The key point is db migration task on server1 is triggered, which is not as expected with the hostfilter.

@leehambley rebased

but wait....the HostFilter code has changed a lot.... I'm fixing it

It's odd, the thing I want to prevent is the thing that @townsen think is correct. (Referencing the commit https://github.com/capistrano/capistrano/commit/ede09c93e80155716ebf2ef0f2e7084a1521b55e)
What do you think for this problem? @townsen
What I'm going to do is to filter out the server that dose not have the primary flag. Aiming to prevent database migration in rails deployment. (In the case that deploy a new server, so I the primary db server should be filtered out)

hummmm, It seems like problem is fixed by @townsen at https://github.com/capistrano/capistrano/commit/ede09c93e80155716ebf2ef0f2e7084a1521b55e
Is this problem really exist? @joshmyers please check the spec I add in this PR


PLT-7169 Read replicas for mysql

This changes mattermost-kubernetes to have read replicas for mysql. It's a copy of a pull request for a helm incubator repo (https://github.com/kubernetes/charts/pull/766). Which in turn is based on https://kubernetes.io/docs/tasks/run-application/run-replicated-stateful-application/
I removed mattermost-db in favour of this, as once the PR is merged into helm, this code can be removed, reducing maintenance problems. This is the reason it's called mysqlha.
I modified how the values for the read replica are passed in so that the chart is able to be used as it is in the pr without modification (pending a pr for that chart https://github.com/jpoon/charts/pull/3). values are set in the mysqlha section of the root values.yaml, and will be hoisted out into the global section via a section in requirements.yaml. This is a little more portable, and provides a way to use the password generation in the mysql chart to have a more secure secret.
replica count is controlled by mysqlha.mysqlha.replicaCount, which defaults to 3 (this includes the mysql master)
This change also includes using persistent volumes, which are also configurable.
Should resolve https://github.com/mattermost/platform/issues/6959

So this new chart is not in the official repository yet. But we should be able to grab it when it is by using helm dependency update right?

Not quite. Right now there is a copy in the repo that is identical to the outstanding PR. When it merges, you can delete the code in this repo, add the incubator dependency, then do helm dependency update. For now you don't need to do any dependency updates.
I looked into hosting the dependency in such a way that you would only have to change what repository the chart is coming from, but it involved some things I felt were more problematic than just keeping the code here.

Oh alright. I misread, thought it was to make it official. Strange that kubernetes doesn't already support RRs for mysql.

@crspeller can we run a load test with this PR to see if it's good?  I guess we might need to check the fork or locally? not sure how that works with the build.

@escardin Can you do a rebase?

done.

Closing as this functionality has been resubmitted in https://github.com/mattermost/mattermost-kubernetes/pull/10.


Jesse: Watch btreesystem.com

Jesse requests the watch of the watch_keyword btreesystem\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

btreesystem\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by cigien in Charcoal HQ


Registration page - Data Save

Registration page is not giving proper message if data is not saved properly due to any application errors. Application has to handle those and properly convey message to the users.

Updated the code to provide a proper failure message to the user when there is any issue with saving the Registration information.


Improved GraphViz support

Is your feature request related to a problem? Please describe.
Per #326, there's scope for improvement in the GraphViz code. For example:

Use a GraphViz-specific abstraction, rather than fmt::Display.
Use, e.g., colour to indicate a node's status (e.g., red for error nodes, etc.).
Embed other metadata (e.g., position) into the node, if possible.

Describe the solution you'd like
If a crate doesn't already exist for this -- it probably does -- a more generic GraphViz abstraction could both satisfy this and be useful outside of Topiary.
Describe alternatives you've considered
A Serde serialiser for GraphViz's DOT format. DOT is not really a serialisation format, so I'm not sure this is possible or, at least, the "right thing to do".
Additional context
Some preliminary/experimental work exists in the chris/experiment/graphviz-trait branch. It has not been pursued, as it doesn't add anything to the basic GraphViz output at the expense of complexity, but may serve as a useful starting point.

Note: Tree-Sitter exposes Parser::print_dot_graphs. Its output is more verbose than our current GraphViz output implementation (which is good), but doesn't lend itself to customisability (which is less good).


added user option of viewing their balance function

Trying committing more when you accomplish a piece of code, @mrsantons


Update generated types to reflect endpoint removals

We have recently removed some API endpoints on the server side; update the
generated types on the client side to reflect those removals.

#2822  👈
main

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

Join @sgrimm and the rest of your teammates on  Graphite

Merge activity

Jul 8, 12:09 PM EDT: @sgrimm merged this pull request with Graphite.


failed to get active systems err context deadline exceeded; failed query: SELECT systems.* FROM systems WHERE status = 'up'

May I know how to fix this issue?
In dashboard page, here is error:

Please check the logs page /_/#/logs for more information.
Is this a new installation, or was it working previously? Was the problem introduced after an update?
My guess is that it's an authentication problem. Try logging out and logging back in. If you can't access the interface to log out, delete the pocketbase_auth entry in localstorage using devtools.


get the ticketdevil design going (need A)

The UI needs some help - -and removing any ticket should only be able to be done by a super user.

I think adding "ticket since" e.g. Jan 2022 or e.g. Ticket since April 2011 would be  good addition -- maybe located where "show ticket" is now.

I was doing some brainstorming on this and I have a couple of cool ideas.
First, I think it would be good to have a Browse page, and possible mix that page with "New Ticket."
I also think it would be cool to have a large menu where Browse, Search Tickets, New Ticket, and Log In are shows as really large options instead of defaulting to browse tickets.  Then do an animation and show the options smaller either above, or on the left.

Also, it may be good to have a "popular tickets' page as well.

Also thinking there needs to be a social is work worker app that integrates with this functionality.  Also, an alternative form of payment would be good to integrate with this as well.
Social is work worker priorities.

Time enough with loved ones?
time enough with friends?
Basic Necessities met
Work/school home back ups
Vacation time
Charity and business goals

It may be good to look at dreambop again.

created a ticket outline image to make it look a little more fun

This is a proposed design.  Looks good, but it still needs some finishing touches.

Added a subtle wallpaper to one side.

It looks more enticing, but still kind of a 90's minimalist feel.  Needs something more relevant, and recent.

currently the design is looking better, but a few things need to be fixed.  The log out button needs to be a link.  The will paginate plugin needs to be centered.  The menu "new ticket" needs to spaced out better.   And the comments need to be colored in the background to pop more.


Started adding boosty stuff and fixed un peu des commenting issues.

I started getting rid of some of the goofy over-commenting and I started to implement boost.

Didn't say anything about adding ResourceManager in title.


Always use default template

I have the following settings with nodemailer
this.mailer.use('compile', hbs({
        viewEngine: {
            extname: '.hbs',
            layoutsDir: 'src/emails/',
            partialsDir: 'src/emails/',
            defaultLayout: 'new_message_from_shooter.hbs',
        },
        viewPath: 'src/emails/',
        extName: '.hbs'
    }));

When I call this
this.mailer.sendMail({
                from<EMAIL_ADDRESS>                to<EMAIL_ADDRESS>                subject: `Test message user`, // Subject line
                template: 'new_message_from_user',
                context: {
                    a: 1
                }
            });

or this
this.mailer.sendMail({
                from<EMAIL_ADDRESS>                to<EMAIL_ADDRESS>                subject: `Test message shooter`, // Subject line
                template: 'new_message_from_shooter',
                context: {
                    a: 1
                }
            });

I always get the same template.... "new_message_from_shooter"
Any ideas?

layout and template are two different things. Think of the layout as the common parts of all of the templates. (header, footer, etc.)
express-handlebars renders the template then renders the layout.
In your case it is always rendering new_message_from_shooter.hbs last so that is what is rendered.


Use proper types for addSnapshotSerializer

Replaces #957

@mikewoudenberg Does this make sense to you?

Awesome, looks like something we can work with!


Examples CMakeLists file missing a dependancy?

Had an issue building the feature_layer example due to a missing dependency.
When I edited the following code at line 45:
target_link_libraries(feature_layers sc2api sc2utils sc2renderer) to target_link_libraries(feature_layers sc2api sc2utils sc2renderer libvcruntime)
It resolved the issue.

What was the linker error you were seeing? Was it just feature_layers failing?

This was the linker error I had.
MSVCRTD.lib(_chandler4gs_.obj) : error LNK2019: unresolved external symbol __except_handler4_common referenced in function __except_handler4

I've seen that error when you build it as a 32-bit target. Is this was you are doing?
Our docs say to build it as 64-bit as that is all we have been testing:
cmake ../ -G "Visual Studio 15 Win64"
Does it still occur when you use that cmake command line? Do you need 32-bit support?

Thank you, very much. That resolved the issue.


xbar not able to record screen

I don't think this is a plugin problem but an xbar one, since the plugin only calls screencapture command
When using ScreenShooter plugin, xbar asks for Screen Recording permissions as expected.
But the permission is not honored and you can see it will ask again after a restart.
I'm using 2.0.35beta version.

@leaanthony Do you know if the app requires specific entitlements to access special features like this?

I'll need to research this. I would have thought this would be between the plugin and the OS

Yeah me too - but I wonder if they share 'privileges' or 'entitlements' with the launching app.

I'll look it up but as we're spawning the process, you'd think it would be out of your hands

I've found this to be the case with any of these privileges I encounter in shell scripts; the privilege has to be given to the parent application, whether that be Terminal or xbar.

The tricky thing about that is knowing what privileges scripts will require at sign time!

maybe this info helps:
if i run the exact same command xbar would run itself from a terminal, it does the job.
but if i try to do it using xbar, it will ask me again to grant screen recording permissions to xbar
despite both Terminal and xbar being listed on allowed apps to screen record.
using v2.0.42beta

It must be related to the fact that the script is being launched from xbar. Does it work if you set terminal=true?

no, it doesn't.
I have duplicated the menu entry to use one command with terminal=true and the other with terminal=false (as the original)
The terminal=true don't do anything at all.
terminal=false option ask again for permissions to be granted.

Somehow xbar asked me to allow control "System Events.app".
After accepting, xbar is not asking to grant screen recording permissions again and the screencapture command works as expected.
I don't think this is an issue anymore.


Remove extra argument passed to FileHandle.init

Fixes chapmanb/bcbio-nextgen-v#92

Thanks so much for this quick fix.


feature-collection w/ one feature

why is examples/problematic_geometries/problematic_multitype_geometry_with_just_one_geometry.geojson problematic?
The geojson spec says a feature collection can have 0+ features

Hey, yes you are correct. It's not invalid (not in the invalid category), but listed here as a "problematic" example. As a reminder that this case should be handled in your code/tool, which some don't and sometimes fail when feeding in an empty collection.

@chrieke thanks for your reply!
I found 2 more problems.

examples/invalid_structure/invalid_geometry_geometrycollection_nested.geojson is totally valid -- you can have nested geometry collections
examples/invalid_structure/invalid_geometry_geometrycollection_single.geojson is also valid -- you can have just one geometry w/ in a geom-collection

structurally invalid

examples/invalid_geometries/invalid_less_three_unique_nodes.geojson should be moved to invalid-structure as it is structurally invalid -- polygon needs at least 4 points regardless of uniqueness
`examples/invalid_geometries/invalid_incorrect_geometry_data_type.geojson should also be moved invalid-structural -- structure is invalid for line string

mixed feelings

examples/problematic_structure/invalid_crs_defined.geojson I have mixed feelings on crs in geojson as it is technically deprecated, however it is extremely useful when working with other crs-s (no idea how to pluralize that)

Thanks for checking it out and the comments and suggestions! Will have a look on the weekend and get back!

@chrieke no problemo! lmk if there is anything I can do to help! thanks for putting this repo together!

Hi @jessekrubin finally got to this, thanks for your comments.

examples/invalid_structure/invalid_geometry_geometrycollection_nested.geojson is totally valid -- you can have nested geometry collections
examples/invalid_structure/invalid_geometry_geometrycollection_single.geojson is also valid -- you can have just one geometry w/ in a geom-collection

Moved this to problematic section. While they are allowed, the geojson specification recommends against it.

To maximize interoperability, implementations SHOULD avoid nested
GeometryCollections.  Furthermore, GeometryCollections composed of a
single part or a number of parts of a single type SHOULD be avoided
when that single part or a single object of multipart type
(MultiPoint, MultiLineString, or MultiPolygon) could be used instead.

examples/problematic_structure/invalid_crs_defined.geojson I have mixed feelings on crs in geojson as it is technically deprecated, however it is extremely useful when working with other crs-s (no idea how to pluralize that)

Renamed to "problematic_crs_defined.geojson". Also moved the section up to the top of the "problematic" section in the Readme, it is important to explain.

examples/invalid_geometries/invalid_less_three_unique_nodes.geojson should be moved to invalid-structure as it is structurally invalid -- polygon needs at least 4 points regardless of uniqueness
examples/invalid_geometries/invalid_incorrect_geometry_data_type.geojson should also be moved invalid-structural -- structure is invalid for line string

Not sure, guess semantically you could argue both ways. I'd say a specific geometry type is defined in the structure, and the geometry coordinates don't fit that, so for me it makes more sense to put it in the "geometry" category. To also ensure that the structure category only contains obvious "structural" issues. Guess argument works both ways, so you might be right, but I think I prefer it this way.

Thank you very much for the comments and thoughts @jessekrubin some important changes! Will close this then, but feel free to reopen or discuss further! Have a good 2025!


International Day Event

Hello guys, @feross  signed you up for the international day but it seems you havn't come around to setup an event site yet! See: nodeschool/international-day#22

is this event still happening?

I didn't sign anyone up for international day.

lol, it's @forres

Sorry for my typo @feross

no problem :-)

this can only mean one thing...you should visit Chile @feross

Haha, would love to! Lmk if there's a good conference and I'm happy to come!
On Tue, May 19, 2015 at 6:53 AM Alejandro Oviedo<EMAIL_ADDRESS>wrote:

this can only mean one thing...you should visit Chile @feross
https://github.com/feross
—
Reply to this email directly or view it on GitHub
https://github.com/nodeschool/Santiago/issues/2#issuecomment-103504277.


Tag job failing for dotnet-spanner-nhibernate

Failure here: https://fusion2.corp.google.com/invocations/b34e2559-f5dc-4af8-a397-159dd8491903/targets/cloud-devrel%2Fclient-libraries%2Fautorelease%2Ftag;config=default/log
Only thing that seems out of the ordinary is that this repo has had no releases before.

FYI: @olavloite

@bcoe Friendly ping. Any update on this?

Is using release-please an option? I think that is the much better supported tool at this point in time.
https://github.com/googleapis/repo-automation-bots/tree/main/packages/release-please
https://github.com/googleapis/release-please

@busunkim96 Jeff and Jon are working on a plan to move all .NET repos to release-please, but for the time being we are still using autorelease.
Once we have moved, we'll move the Spanner ORM repos as well.

I'm going to mark this as P2 as there is a workaround to use release-please

That's not a workaround, that's a whole project on its own that we are looking into with @chingor13 . For us to use release-please there's work to be done on both the .NET repos and release please.
We were able to release mmanually last time but it'd be great if we could use our normal release process on dotnet-spanner-nhibernate the same way we do for the rest of the .NET repos.

@bcoe @chingor13 Do you expect that it would be quicker to fix this issue or migrate to release-please?

Added to agenda for next week, sorry for missing this ping.

@bcoe possibly sync-up with @jskeet as well as he'll have a better idea of the work on the .NET side of thing than I have.

This issue will fall OoSLO in 27 days.

Not sure why this was assigned back to me. Assigning back to @SurferJeffAtGoogle for triage.

The .NET repos are still using autorelease and I'm not sure how the work to migrate them to release-please is going. Possibly @chingor13 may know more.
I think this might still be an issues in the dotnet-spanner-nhibernate repo but @olavloite can confirm that.

Still planning on migrating workloads off of releasetool onto release-please, it's just not the highest priority

Only gax and google-cloud-dotnet remain on autorelease.


chore: enhance extrinsic testing 🚀

Pull Request type
Please add the labels corresponding to the type of changes your PR introduces:

Testing

What is the current behavior?
The current extrinsic testing just depends upon event emission, the testing should be more dependent on the state changes that calls to these extrinsic calls should result in { have been elaborated in #617  }.
The current implementation of deployTokenContractUDC also has a bug, where the wrong value for unique is provided { have been described in #630  }
Resolves: #617 , #630
What is the new behavior?
Extrinsic testing involves checking for real state transitions that should result from calls to these extrinsic.
Example:

check for a change in equivalent balance in case of ERC20/ERC721 transfer/mint
check for contract deployment in case of deployment { by checking class hash at address, and checking ERC20 balance of the initial amount of recipient, which is assigned in constructor }

#630 is resolved, and correct value for unique is assigned.
Does this introduce a breaking change?
NO

NOTE:
For now, to limit the scope of this PR, I haven't touched the declare test, it remains skipped and I have marked it TODO.

I think fixing declare should be a different issue, and should be addressed in a different PR.
While navigating the codebase inconsistency in using constants, where in some places rather than using the constant, value was hardcoded again. This also needs to be addressed in a different issue and a different PR!

I will make issues for them if everyone agrees.
cc: @abdelhamidbakhta


Working set size reduction via HLO broadcast/indexing code motion

To reduce the average working set size (concurrent total live buffer size, basically) we should look for canonicalizations that favor operating on more compact tensors.
For example, the following sequence is non-optimal:
%0 = xla_hlo.broadcast(%arg0) // tensor<4xi32> -> tensor<4x1024xi32>
%1 = xla_hlo.broadcast(%arg1) // tensor<4xi32> -> tensor<4x1024xi32>
%2 = xla_hlo.add %0, %2 : tensor<4x1024xi32>

Instead, we should try to move broadcasts as far down in the IR tree as possible:
%0 = xla_hlo.add %arg0, %arg1 : tensor<4xi32>
%1 = xla_hlo.broadcast(%0) // tensor<4xi32> -> tensor<4x1024xi32>

There are reverse situations as well, where for example xla_hlo.slice should be moved as early as possible in the IR tree:
%0 = xla_hlo.mul %arg0, %arg1 : tensor<4x1024xi32>
%1 = xla_hlo.slice(%0) // tensor<4x1024xi32> -> tensor<4xi32>

which should instead be:
%0 = xla_hlo.slice(%arg0) // tensor<4x1024xi32> -> tensor<4xi32>
%1 = xla_hlo.slice(%arg1) // tensor<4x1024xi32> -> tensor<4xi32>
%2 = xla_hlo.mul %0, %1 : tensor<4xi32>

There are a great deal of these kind of transformations that could be done with the goal being to introduce additional broadcast/indexing operations in order to reduce the total arithmetic required and memory consumed.
This pairs with other optimization work to rematerialize CSE'd IR into dispatch regions that is cheap to perform.

It'd still be nice to do this but will be less important with linalg-on-tensors (which should hopefully fold all these things together). Similar concepts applied to global variables/constants are still valid, though.


feat: add app deletion mutation

This works, but couple of comments/questions :

Currently using --force destroy which seems fine as we can handle more validations on client side, for example popup "to confirm enter name of the app".

Do we need queue for app deletion? If so then it will be a fun tech task to build one :)

I didn't parse stdout because it didn't seem to provide any valuable log that we wouldn't be able to make ourselves. Here is now it looked :
stdout: '-----> Destroying node14 (including all add-ons)\n-----> Cleaning up...',"

Yup good idea :+1:
Well depends how long it is to delete an app?
Will depends of the response to the énd question :D


Make modifiersCache of limited size

At the moment "modifiersCache" in NodeViewHolder can grow indefinitely. Configurable cache size & strategy to clean the cache are needed to be implemented.

Should we consider using a caching library (e.g. scaffeine, scalacache)?


Implement phase 2

Adding major data groups related to HVAC in Phase 2 of development based on new EnergyPlus tabular output reports.

@Myoldmopar I have the script to a good point supporting a bunch of new data groups in the RMD file. Please take a look and let me know what you think. I still need to add unit tests for the new code, hopefully next week.

This is giving code coverage of 95%, which is really high and great.  I'm not sure what develop is at because the last commit was too long ago to see in the GHA logs.  If develop is already 100%, then it might be worth looking at those last few missing coverage spots (https://github.com/JasonGlazer/createRulesetModelDescription/actions/runs/7575402982/job/20631931543?pr=17#step:5:27) in this branch and see if just a tweak of a test or two could cover them meaningfully.  If not, no big deal.

This is giving code coverage of 95%, which is really high and great. I'm not sure what develop is at because the last commit was too long ago to see in the GHA logs. If develop is already 100%, then it might be worth looking at those last few missing coverage spots (https://github.com/JasonGlazer/createRulesetModelDescription/actions/runs/7575402982/job/20631931543?pr=17#step:5:27) in this branch and see if just a tweak of a test or two could cover them meaningfully. If not, no big deal.

We were at 91% before. So I thought 95% was a good improvement.

OK, thanks @JasonGlazer.  I'm good with merging this in.  The other PRs don't need to go in first or anything, do they?

Merge away. I removed one of the pull requests that was already part of this branch. The other two I need to circle back on and update.


502 Bad Gateway

I am getting a 502 Gateway when following the instructions.
Here are some of the details:
npm install
npm WARN<EMAIL_ADDRESS>No repository field.

added 269 packages from 282 contributors and audited 717 packages in 3.139s
found 19 vulnerabilities (4 low, 3 moderate, 11 high, 1 critical)
  run `npm audit fix` to fix them, or `npm audit` for details
peterdunham@Peters-MacBook-Pro formio-webhook-receiver % node basic
Basic Webhook Receiver running on undefined

Is "running on undefined" correct?
Then:
peterdunham@Peters-MacBook-Pro ~ % ./ngrok http 4001

ngrok by @inconshreveable                                       (Ctrl+C to quit)

Session Status                online
Session Expires               7 hours, 53 minutes
Version                       2.3.35
Region                        United States (us)
Web Interface                 http://<IP_ADDRESS>:4040
Forwarding                    http://d457a788.ngrok.io -> http://localhost:4001
Forwarding                    https://d457a788.ngrok.io -> http://localhost:4001

Connections                   ttl     opn     rt1     rt5     p50     p90
                              1       0       0.00    0.00    0.00    0.00

HTTP Requests
-------------

GET /                          502 Bad Gateway

Here is my webhook from GUI - I have tried a POST and a GET with same result

Any help is greatly appreciated

Here are some more details form the ngrok browser GUI:
<!doctype html5>

Failed to complete tunnel connection

    The connection to https://d457a788.ngrok.io
    was successfully tunneled to your ngrok client,
    but the client failed to establish a connection to
    the local address localhost:4001.

    Make sure that a web service is running on
    localhost:4001 and that it is a valid address.

    The error encountered was: dial tcp [::1]:4001: connect: connection refused

This issue was resolved by running
PORT=4001 node basic

However, I am now receiving at 401 error authorization error when using
test
password123

So issue was resolved by running
PORT=4001 USER=test PASS=password123 node basic

it appears that .env.example
needs to be made not example.  This is not clear in the ReadMe

We're currently addressing a backlog of GitHub issues. Closing this thread as it is outdated. Please re-open if it is still relevant. Thank you for your contribution!


Carousel: exposing current slide index

Should fix #11.

[Current coverage][cc-pull] is 81.54% (diff: 100%)

Merging [#769][cc-pull] into [development][cc-base-branch] will increase coverage by 0.19%

@@           development       #769   diff @@
=============================================
  Files               17         17
  Lines              531        542    +11
  Methods            103        105     +2
  Messages             0          0
  Branches            67         69     +2
=============================================
+ Hits               432        442    +10
- Misses              99        100     +1
  Partials             0          0

Powered by Codecov. Last update [bd4cc96...e816022][cc-compare]
[cc-base-branch]: https://codecov.io/gh/valor-software/ng2-bootstrap/branch/development?src=pr
[cc-compare]: https://codecov.io/gh/valor-software/ng2-bootstrap/compare/bd4cc96cb4d2c0931824eb1a1de66dbdf3e918a6...e8160226be11236e57697ed40f749ed4891b2b2a?src=pr
[cc-pull]: https://codecov.io/gh/valor-software/ng2-bootstrap/pull/769?src=pr

Thanks for adding this featrue. I see this PR was closed a month ago, but it doesn't look like it's merged into master yet. Any idea when that will happen?

There already was a change implemented to fix this. It will probably be merged from #760.


Deprecate external_url in Invoice in favour of url

I want to merge this change because it's closing #11506
Invoice now has only one url field instead of external_url and url. The underlying functionality already returned one or the other when using url field so the PR is only deprecating a GraphQL field.
TODO: add docs PR and link

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[x] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations

Docs

[ ] Link to documentation:

Pull Request Checklist

[ ] Privileged queries and mutations are either absent or guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migrations are either absent or optimized for zero downtime
[ ] The changes are covered by test cases

@fowczarek I'll leave this PR for you for any potential fixes as you were the original reporter of the issue and the change seems to be straightforward. If it proves to need a lot of additional work, feel free to leave it and assign me back.


Should mypy_primer default to --strict and --install-types ?

I was adding a project with a ton of types-* stubs when I thought: why not just auto-install all necessary stubs? Wouldn't that remove some configuration burden?

I was also thinking, what if the primer ran with --strict? It would result in more errors for projects that didn't turn it on. Which doesn't really matter when we're interested in the diff. And it would be beneficial to catch more of the changes' effects (whether they're negative, or positive: allowing libraries to turn on strict checks that would otherwise fail).
At least for those configured with expected_success=False

Mainly because --install-types is super duper slow and I dislike it :-P Also worth noting that in typeshed CI, we'll still type check against typeshed stubs even if we don't specify types-xyz in the pip_cmd, so it's not super critical if one gets missed. We will get slightly less accurate typing in mypy CI, but my sense is that if you've got the main dependencies covered it's not too bad. Would definitely be useful to have a mode to update configuration if we got something like https://github.com/python/mypy/issues/10600 though!
Running with --strict is an interesting idea. In mypy CI we definitely do not want to run with --strict — we want to run mypy similar to how the projects themselves run it — but it's a more compelling idea for typeshed CI. I'm genuinely unsure how helpful it would be... you get some amount of value from --check-untyped-defs, but for the most part everything unannotated is still Any, so feels like if it catches something it'd probably just catch the same kind of shape errors that stubtest can catch. That said, mypy_primer already supports a --additional-flags argument, so this is a thing typeshed could just try today! Maybe make a PR to typeshed and see what people think.

Akuli brought this up today as well in https://github.com/hauntsaninja/mypy_primer/issues/66 ... Sounds like we should try it in typeshed CI!

Unless you see value in leaving this open, I think you can close in favor of the other issue. Plus "mypy_primer already supports a --additional-flags argument" it's something we'd change in typeshed.

Yup, agreed that using --additional-flags to pass --strict in typeshed is the way to go.


SARAALERT-477 Bulk Update Assigned User

Description
Jira Ticket: SARAALERT-477
Ability to bulk update assigned user through the main dashboard.
Demo/Screenshots
View from Dashboard:

Modal that appears:

Important Changes
app/controllers/patients_controller.rb

Updated function name to reflect its usage with this PR

app/javascript/components/public_health/PatientsTable.js

Included "Update Assigned User" option in Bulk Actions drop down

app/javascript/components/public_health/actions/UpdateAssignedUser.js

New file for the Update Assigned User model that appears when that bulk action is selected

config/routes.rb

New route for bulk action for updating assigned user
Updated status update route to refer to the renamed function in patients_controller.rb

test/controllers/patients_controller_test.rb

Updated tests to call renamed function & included assigned user as a part of the tests

Testing
This fix was tested on the following browsers (submitter must check all those that apply):

[x] Chrome
[ ] Firefox
[ ] Safari
[ ] IE11

Test using the bulk action for Updating Assigned User in the following situations and make sure it saves properly:

[ ] Attempt with one monitoree selected
[ ] Attempt with multiple monitorees selected
[ ] Attempt with a Head of Household selected and don't apply change to rest of household
[ ] Attempt with a Head of Household selected and do apply change to rest of household

Test this other aspect:

[ ] Select a single monitoree with an Assigned User already set. When the modal to update assigned user appears, verify that the text field is already populated with the monitoree's existing Assigned User.

I need to investigate the failing tests for the public-health-bulk-edit file. I will re-assigned reviewers once the tests have been updated.

I need to investigate the failing tests for the public-health-bulk-edit file. I will re-assigned reviewers once the tests have been updated.

The tests were updated and new ones have been created. This PR should be good for review now.


Raspberry Pi  unable to compile (kernel 5.10.63-v8+)

Hi everyone,
SInce I have a cron updating my pi every week, I've found out the kernel has been updated recently and the raspbee2 was not streaming data anymore.
When trying to compile the master (after getting the kernel source), heres the message i got :
Linux raspberrypi 5.10.63-v8+ #1459 SMP PREEMPT

checking deb package dependencies...
i2c-tools
build-essential
raspberrypi-kernel-headers
build rtc module...
make -C /lib/modules/5.10.63-v8+/build M=/tmp/raspbee2-rtc-master modules
make[1]: Entering directory '/home/pi/linux-abff5012ab7ccbe1227b9d0af69697cc592df44c'
arch/arm64/Makefile:25: ld does not support --fix-cortex-a53-843419; kernel may be susceptible to erratum
arch/arm64/Makefile:33: LSE atomics not supported by binutils
  CC [M]  /tmp/raspbee2-rtc-master/rtc-pcf85063.o
gcc: error: unrecognized command line option ‘-mgeneral-regs-only’
make[2]: *** [scripts/Makefile.build:280: /tmp/raspbee2-rtc-master/rtc-pcf85063.o] Error 1
make[1]: *** [Makefile:1825: /tmp/raspbee2-rtc-master] Error 2
make[1]: Leaving directory '/home/pi/linux-abff5012ab7ccbe1227b9d0af69697cc592df44c'
make: *** [Makefile:17: build] Error 2

Since then, I am unable to get the raspbee2 working anymore. You're help is much appreciated.

Installing deCONZ seems to work, but the application keeps rebooting constantly.

besides the compilation error you got your raspbee has another problem. The functionality of the RaspBee2 does not depend on the compilation of the source  code. This is only needed for the RTC and I think when you have compiled and installed it one time it is not needed to do anymore.
Also updating the OS should not compromise the functionality of the RaspBee2.
So if you can't connect to deCONZ or deCONZ is loosing the connection repeatedly I would suggest the following:
try to install the newest firmware with GCFFlasher
https://github.com/dresden-elektronik/deconz-rest-plugin/wiki/Update-deCONZ-manually#update-in-raspbian
Disconnect other devices connected to the USB port because they could produce interferences.


Incompatible version

Plugin not usable in EAP. Which, fair enough if you're waiting for the release first.
Using GoLand version Build #GO-241.8102.131 EAP

I just noticed this to. The compatibility range on the the plugin page is showing that that the supported version is going down?

Latest version should have support. Please reopen if the issue persists!

Confirmed with 2023.3.3

Epic. Thanks so much!


Module Manager

First pass of a Module Manager for #17
Probably good for a PR as is but I would like to finish it and have good unit test coverage before then.
Just creating the PR so you can have a look 👍

load_library integration added, usage example:
with open("hello-world-x64.dll", "rb") as dll:
    module_base_addr = dp.modules.load_library("hello-world-x64.dll", dll.read(), 1) # DONT_RESOLVE_DLL_REFERENCES
    print(f"hello-world-x64.dll: 0x{module_base_addr:x}")
    module = dp.modules["hello-world-x64.dll"]
    print(f"hello-world-x64.dll: 0x{module.base_address:x}")
    assert module_base_addr == module.base_address

This will then get loaded into the module manager

Closed ref https://github.com/mrexodia/dumpulator/pull/23#discussion_r990846562


feat: Improve logging output

I decided to increase the amount of information you get the more verbose you set the logs to be. Here are the formats:

Error, Warn, and Info level

Debug Level

Trace Level

What's nice is, now the trace logging will display the line number in the code which will perfect for debugging!

Thinking about it, probably don't want to have white for the time in case users use a white background

Here's standard fg color for time

And now with some proper padding

Actually, uhh, why is most of the text orange in your screenshot? Shouldn't color be used as a highlight and not just added to everything for the sake of it. Or is this a setting of your terminal?

Orange color is usually used to indicate warning or attention, along with yellow for caution.
I think that terminals automatically switch colors from black to white & vice-versa when switching the theme.
So I'm not sure if there is a need to make everything orange to comply with most themes.

Actually, uhh, why is most of the text orange in your screenshot? Shouldn't color be used as a highlight and not just added to everything for the sake of it. Or is this a setting of your terminal?

The orange is my personal color scheme. The parts of the log that aren't orange are the colors that are added.


Test 6

@Zolix-ai, Seems good +1

@nuilewis Great job! CircleCI Build Veried✅

@nuilewis Great job! CircleCI Build Veried✅

Thank you @Redeem-Grimm-Satoshi 🙏


Update Cory Dolphin's Github username in AUTHORS.

Cory Dolphin changed Github usernames from "wcdolphin" to "corydolphin".
Github does not automatically redirect profile links when you change
usernames so his link in the AUTHORS file went to a 404.
https://help.github.com/articles/what-happens-when-i-change-my-username/#changes-that-arent-automatic

Thanks!


How to prepare my own data set

In your code, you say follow the scripts folder to learn how to prepare the data set.But there are four files in the folder, what steps should be followed to prepare.
Also, how do I set the num_points parameter when I do the prepare_h5.py file:
batch = room_to_blocks(fname, **num_points,** size=1.0, stride=0.5)
I would also like to ask which of the following data sets is appropriate for processing
Stanford3dDataset_v1.2_Aligned_Version  or  Stanford3dDataset_v1.2

Assume that you use the aligned version, you only need to run collect_annotations.py and then prepare_h5.py to prepare the data, the others are standalone scripts to calculate some dataset statistics. num_points is set to 4096 in my experiments.

Thanks a lot, I will try it now to prepare my dataset


Changing channel in About view doesn't trigger update check

cannot reproduce in master, request is visible in log after changing the channel

ok, it took me a while to reproduce the issue. should be better now.


Fixed deprecation warning ":public is no longer used to avoid overloading Module#public, use :public_folder instead"

Fixed deprecation warning ":public is no longer used to avoid overloading Module#public, use :public_folder instead"

I'll pull in the fix from original Resque branch eventually. Thanks though.


Having Error after solver.click_recaptcha_v2(iframe=recaptcha_iframe)

Hello，
Need your help , I having error after solver.click_recaptcha_v2(iframe=recaptcha_iframe) , as below  :
solver.click_recaptcha_v2(iframe=recaptcha_iframe)

site-packages\selenium_recaptcha_solver\solver.py", line 81, in click_recaptcha_v2
captcha_challenge = self._wait_for_element(
site-packages\selenium_recaptcha_solver\solver.py", line 230, in _wait_for_element
return WebDriverWait(self._driver, timeout).until(ec.visibility_of_element_located((by, locator)))
site-packages\selenium\webdriver\support\wait.py", line 95, in until
raise TimeoutException(message, screen, stacktrace)
**I noticed that it only work on if have popup press PLAY to listen popout (multi-step audio) , if is single step then will have issue**

Hello,
Usually, when there's a timeout exception, it's because either your internet was too slow or something went wrong in the solving process like Google blocking your IP. However, I see you commented it doesn't work if it's a single step, so can you elaborate on that and include some examples?
I appreciate you making this issue!

Hello,
Thank for the reply.
The issue only happen on single step , example code :

recaptcha_iframe = driver.find_element(By.XPATH, '//iframe[@title="reCAPTCHA"]')
solver.click_recaptcha_v2(recaptcha_iframe)  // After resolve the recaptcha , the website will automatically redirect to another page , I can see the resolve recaptcha is successfully and the website is redirect completed.

so I'm thinking maybe because the website redirect too fast ?

//other code......

Hello,
I had some time today, so I implemented a fix for your case.
If you have any more issues or suggestions let me know!

Hello,
I had some time today, so I implemented a fix for your case.
If you have any more issues or suggestions let me know!

Hello
Sorry , I have update the plugin to 1.8.1
The issue still there , do I need clear any cache?
Error message same :
site-packages\selenium_recaptcha_solver\solver.py", line 81, in click_recaptcha_v2
captcha_challenge = self._wait_for_element(
site-packages\selenium_recaptcha_solver\solver.py", line 230, in _wait_for_element
return WebDriverWait(self._driver, timeout).until(ec.visibility_of_element_located((by, locator)))
site-packages\selenium\webdriver\support\wait.py", line 95, in until
raise TimeoutException(message, screen, stacktrace)

Hello,
Could you send me the website you're trying to automate (Or send me the script you're using if it doesn't have sensitive data like passwords or secrets)? I can understand what's going on better that way.

Sure, here you go<EMAIL_ADDRESS>

Sure, here you go<EMAIL_ADDRESS>
Hello
I have send you an email , please check.
Thank you for your help.

Hello,
I'll be checking the issue out soon. Thank you for your cooperation!

Hey @thicccat688 , how are you? Happens the same to me. It says "Google has detected automated queries. Try again later" but if I click manually to the audio button I can download it.
The captcha Im trying to resolve opens automatically so instead of using click_recaptcha_v2 I directly use solve_recaptcha_v2_challenge because it is already opened with the pictures to choose:

Thanks for the amazing work!

hello
we can by pass the error for solve the issue , let the process continue run.
try :
solver.click_recaptcha_v2(recaptcha_iframe)
except Exception :
pass
......


Enhanced Item Information

I really don't want to be that idiot who uses issues for requests, but this is one feature that's been missing for a very long time in the mod world. I would be forever grateful if you find time to look into this:

Thank you Cursey!

I support this, it would be a nice feature to have.

I'll see what I can do.

If it helps at all, here's some offsets in the item data structure (in hex of course).
+C = Item ID (4 byte)
+10 = Color 1 (4 byte)
+14 = Color 2 (4 byte)
+18 = Color 3 (4 byte)
+44 = Purchase Price (4 byte)
+48 = Sale Price (4 byte)
+50 = Durability (2 byte)
+54 = Max Durability (2 byte)
+6E = Effective Range (2 byte)

I just have no idea how to go about this myself.

Bump, I have a feeling this has been forgotten. Would do it myself but I'm not familiar enough with the structure of Kanan and am unable to get IDA to co-operate with Mabi, not much luck with CE either.


[SSO] Separate settings for local and social signup

The "self registration" setting controls self-signup of both local (username/password) and social (SSO) accounts.
Ideally, we would have separate control for allowing (or preventing) self-reg of local accounts and / or SSO accounts.
For example, prevent users from self-registration of local accounts, but allow self-registration of SSO accounts through an external app which already restricts which user accounts have access.
Currently, enabling "self registration" (required for SSO self reg) also lets local accounts be created without restriction.

@matmair what do you think about this one? Can we implement with some custom django-allauth classes?

@SchrodingersGat this is rather simple, I just thought it might make the settings for SSO too complex.
I will program this in the next few days. If I touch this code I can just add the scoping checks asked for in discussions also: https://github.com/inventree/InvenTree/discussions/2619

Seems 0.10.0 is going to have a bunch of security things in it.

Nice, thanks @matmair - please also see the changes in https://github.com/inventree/InvenTree/pull/4169

See https://github.com/inventree/InvenTree/issues/4168