Duplicate device ID because of truncated Panel Server component serial number

Describe the bug
Our PAS800L panel server has serial numbers with a length of 21 characters. When a device is created the integration turuncates the serial number to 20 characters. Devices that have a serial number with only the last character difference have a conflict.
I've been looking for a way to shorten the serial number in the panel server, but have not found.
Is it possible to increase the length of the device ID or to truncate the serial number at the front if it's to long?

Versions

HomeAssistant: 2024.10.3
Integration: v0.5.0

Logs
extract:
2024-10-28 09:09:03.701 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010wireless communication valid already exists - ignoring binary_sensor.hal_1_1_wireless_communication_valid 2024-10-28 09:09:03.701 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010radio communication valid already exists - ignoring binary_sensor.hal_1_1_radio_communication_valid 2024-10-28 09:09:03.701 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010alarm info already exists - ignoring binary_sensor.hal_1_1_alarm_info 2024-10-28 09:09:03.706 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010wireless communication valid already exists - ignoring binary_sensor.hal_1_1_wireless_communication_valid 2024-10-28 09:09:03.706 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010radio communication valid already exists - ignoring binary_sensor.hal_1_1_radio_communication_valid 2024-10-28 09:09:03.706 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010alarm info already exists - ignoring binary_sensor.hal_1_1_alarm_info 2024-10-28 09:09:03.707 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W10-7-GH-004wireless communication valid already exists - ignoring binary_sensor.ontkoppelbord_zp_wireless_communication_valid 2024-10-28 09:09:03.708 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W10-7-GH-004radio communication valid already exists - ignoring binary_sensor.ontkoppelbord_zp_radio_communication_valid 2024-10-28 09:09:03.708 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W10-7-GH-004alarm info already exists - ignoring binary_sensor.ontkoppelbord_zp_alarm_info 2024-10-28 09:09:03.725 WARNING (MainThread) [homeassistant.helpers.frame] Detected code that calls async_forward_entry_setup for integration powertag_gateway with title: EcoStruxure Panel Server and entry_id: 01JAYV9GJ313NNXT9KT79W8SYF, during setup without awaiting async_forward_entry_setup, which can cause the setup lock to be released before the setup is done. This will stop working in Home Assistant 2025.1. Please report this issue.

Seems to be an oversight on Schneider's end, so I put a message on their forum.
https://community.se.com/t5/Gateways-and-Energy-Servers/PowerTag-serial-numbers-can-be-longer-than-the-modbus-registers/m-p/489073#M4378
I can work around this though, but if I would do it in the codebase, everyone's entities would get duplicated, so I will propose a manual fix that's just for you:

Navigate to custom_components/powertag_gateway/entity_base.py
On line 126, find self._attr_unique_id = f"{TAG_DOMAIN}{serial}{entity_name}
Add {modbus_index} so that this line becomes self._attr_unique_id = f"{TAG_DOMAIN}{serial}{entity_name}{modbus_index}"
Save and restart

Please let me know if that helps!

I modified the entity_base.py, restarted HA, deleted the entry, restarted HA, reintegrated the entry. The result is the same with the exception that before I had 633 entities and now I have 810. Some are still mapped to the wrong device and not all devices are shown.
I checked if the change to entity_base.py was still there after the reintegration and that was the case.
I don't have the impression that new devices were created as my frontend views continue to show their data. So the existing entities are unchanged (except for the units that are set back from kW to W).

Enclosed the core log file:
home-assistant_2024-10-29T06-51-15.814Z blinded.txt

Hi Breina,
Sorry to insist on this subject, but I'm stuck with my project. Help from Schneider will be difficult. Even if they change their serial number format, it would mean that we physicaly will have to change the devices.
You indicate that you see a work around but that you don't want to jeperdize the entities of exisiting users. This is ofcoarse of the utmost importance.
Is it possible to include a "switch" in the software to activate the workaround?
Thanks in advance,
Mik

Hi Breina,
I removed Power Tag, deleted all entities and helpers that had a link with the integration and reinstalled 0.5.3 via HACS and reïntegrated (with all necessary restarts).
I started with the automatic DPWS discovery. That returned an error.
I then chose the manual option, but did not find the "flag". The interface was still the same as before.
After setup I still have 10 entities instead of 13. The only difference is that the 3 power tags that were not imported last time because of the truncated serail number are present now instead of the original ones I had before.
I checked the core log, but could not see any errors during the setup.
Do I have to change something manualy somewhere?

Shit sorry, I did a typo. :(
Please try 0.5.4.
The flag is set for all new installations, If you don't see a warning Using older version of device's unique ID, may cause conflicts with duplicate serials. printed, it means that the fix is applied.

I have a PAS600, and this does not work with 0.5.4, I get no data from the PAS600.
If I go back to 0,5,3, everything is okay again

Sorry Breina,
Deleted, restarted, installed 0.5.4, restarted, added integration. Manual Panel Server. First went trough without error but also without creating entities.
Did a "add entry" again manual Panel server and got "failed to connect".
Core Log:
2024-11-13 10:42:49.401 INFO (MainThread) [custom_components.powertag_gateway.schneider_modbus] Connecting Modbus TCP to xx.xx.x.xxx:502
2024-11-13 10:42:50.447 ERROR (MainThread) [root] Flow aborted: already_configured
Traceback (most recent call last):
File "/config/custom_components/powertag_gateway/config_flow.py", line 170, in async_step_configure
return await self.async_step_connect()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/powertag_gateway/config_flow.py", line 240, in async_step_connect
self._abort_if_unique_id_configured()
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2434, in _abort_if_unique_id_configured
raise data_entry_flow.AbortFlow(error)
homeassistant.data_entry_flow.AbortFlow: Flow aborted: already_configured`

Hi Breina,
0.5.5 did not change anything for me, but I don't know if that was the intention of the release?
I still have 3 entries missing.

Hi Breina,
I plunged back into this and found that actually all Pannel parameters are present. The cabinets that have the same truncated serial number are grouped into 1 cabinet. As an example, cabinets called "Hal 1-1", "Hal 1-2" and "Hal 3" are all grouped into entity "Hal 3".
It takes some rearanging from my part, but I can live with it.

Hi Breina,
Ignore my previous post: I had to restart HA and now all entities are duplicated and the existing ones became unavailable.

Log extract:
2024-11-27 16:59:07.382 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'powertag_gateway' calls async_forward_entry_setup for integration, powertag_gateway with title: EcoStruxure Panel Server and entry_id: 01JCQKNJBXF2HNMPZPB3SGHPS9, which is deprecated and will stop working in Home Assistant 2025.6, await async_forward_entry_setups instead at custom_components/powertag_gateway/__init__.py, line 57: await hass.config_entries.async_forward_entry_setup(entry, platform), please create a bug report at https://github.com/Breina/PowerTagGateway/issues 2024-11-27 16:59:13.887 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010wireless communication valid already exists - ignoring binary_sensor.hal_1_1_wireless_communication_valid_2 2024-11-27 16:59:13.887 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010radio communication valid already exists - ignoring binary_sensor.hal_1_1_radio_communication_valid_2 2024-11-27 16:59:13.888 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010alarm info already exists - ignoring binary_sensor.hal_1_1_alarm_info_2 2024-11-27 16:59:13.922 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010wireless communication valid already exists - ignoring binary_sensor.hal_1_1_wireless_communication_valid_2 2024-11-27 16:59:13.923 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010radio communication valid already exists - ignoring binary_sensor.hal_1_1_radio_communication_valid_2 2024-11-27 16:59:13.923 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010alarm info already exists - ignoring binary_sensor.hal_1_1_alarm_info_2 2024-11-27 16:59:13.929 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W10-7-GH-004wireless communication valid already exists - ignoring binary_sensor.ontkoppelbord_zp_wireless_communication_valid_2 2024-11-27 16:59:13.930 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W10-7-GH-004radio communication valid already exists - ignoring binary_sensor.ontkoppelbord_zp_radio_communication_valid_2 2024-11-27 16:59:13.930 ERROR (MainThread) [homeassistant.components.binary_sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W10-7-GH-004alarm info already exists - ignoring binary_sensor.ontkoppelbord_zp_alarm_info_2 2024-11-27 16:59:18.945 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010reset active energy delivered already exists - ignoring button.hal_1_1_reset_active_energy_delivered_2 2024-11-27 16:59:18.945 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010reset active energy received already exists - ignoring button.hal_1_1_reset_active_energy_received_2 2024-11-27 16:59:18.945 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010reset reactive energy delivered already exists - ignoring button.hal_1_1_reset_reactive_energy_delivered_2 2024-11-27 16:59:18.945 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010reset reactive energy received already exists - ignoring button.hal_1_1_reset_reactive_energy_received_2 2024-11-27 16:59:18.955 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010reset active energy delivered already exists - ignoring button.hal_1_1_reset_active_energy_delivered_2 2024-11-27 16:59:18.955 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010reset active energy received already exists - ignoring button.hal_1_1_reset_active_energy_received_2 2024-11-27 16:59:18.956 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010reset reactive energy delivered already exists - ignoring button.hal_1_1_reset_reactive_energy_delivered_2 2024-11-27 16:59:18.956 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010reset reactive energy received already exists - ignoring button.hal_1_1_reset_reactive_energy_received_2 2024-11-27 16:59:18.959 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W10-7-GH-004reset active energy delivered already exists - ignoring button.ontkoppelbord_zp_reset_active_energy_delivered_2 2024-11-27 16:59:18.959 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W10-7-GH-004reset active energy received already exists - ignoring button.ontkoppelbord_zp_reset_active_energy_received_2 2024-11-27 16:59:18.960 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W10-7-GH-004reset reactive energy delivered already exists - ignoring button.ontkoppelbord_zp_reset_reactive_energy_delivered_2 2024-11-27 16:59:18.960 ERROR (MainThread) [homeassistant.components.button] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W10-7-GH-004reset reactive energy received already exists - ignoring button.ontkoppelbord_zp_reset_reactive_energy_received_2 2024-11-27 16:59:34.435 ERROR (MainThread) [homeassistant.components.sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010total active energy already exists - ignoring sensor.hal_1_1_total_active_energy_2 2024-11-27 16:59:34.435 ERROR (MainThread) [homeassistant.components.sensor] Platform powertag_gateway does not generate unique IDs. ID PowerTagRN-2023-W33-4-GH-010reactive power already exists - ignoring sensor.hal_1_1_reactive_power_2

Breina,
New development: I reinstalled version 0.5.3. All entities I need are present. They are not grouped under the right device (I still only have 10 instead of 13), but they are present and I can use them. I can live with this.
Sorry to have put you on the false track. I should have observed this before.

That's alright no worries. I didn't really know how to proceed with this, so I didn't update you on progress. I'll leave this open for when I get some inspiration and find your missing 3 entities.

Hey Breina,
I realy appreciate what you are doing! Don't worry about the lost entries for now. I can manage as all entities are there.
A real inconvenience is that everytime I restart HA all entities are duplicated. Everytime I have to delete the entry and add again to let HA use the original entities again.
I'm on 0.5.3.


Drag&drop

:tada: This PR is included in version 8.2.0-rc.1 :tada:
The release is available on:

GitHub release
npm package (@next dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 9.0.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Improve tvos support in deliver

Previously, finding builds on a universal app would not work.
cc #6480 #7648

Thanks for this PR @tmm1! Has this been tested for all of the supported platforms? 🐙

I only tested uploading a tvos app, but I have no reason to believe it wouldn't work everywhere.

@tmm1 I see other places in deliver where we don't pass the platform option.
E..g verify_available_languages!, UploadScreenshots#upload for edit_version and more for latest_version
Have you had a chance to test all those features? (I don't do tvos apps, so can't help there)

See:
$ git grep -n -E "(edit|latest)_version" deliver | grep -v ":platform"
deliver/lib/deliver/commands_generator.rb:150:          v = options[:app].latest_version
deliver/lib/deliver/download_screenshots.rb:13:      v = options[:app].latest_version
deliver/lib/deliver/html_generator.rb:41:      @languages ||= options[:app].latest_version.description.languages
deliver/lib/deliver/setup.rb:27:      v = options[:app].latest_version
deliver/lib/deliver/submit_for_review.rb:32:      v = app.edit_version
deliver/lib/deliver/submit_for_review.rb:58:        build = find_build(app.latest_version.candidate_builds)
deliver/lib/deliver/upload_metadata.rb:153:      v = options[:app].edit_version
deliver/lib/deliver/upload_screenshots.rb:37:        v = app.edit_version
deliver/spec/commands_generator_spec.rb:207:      expect(fake_app).to receive(:latest_version).and_return('1.0.0')
deliver/spec/setup_spec.rb:9:      # expect(app).to receive(:latest_version).and_return(version)

I've used those features in the past with a tvos-only app, but I imagine they would not work as expected with a universal app.
I actually don't have a universal app myself, so if someone else wants to take over this effort I would prefer that.

Hey @tmm1, just checking in to see if you had seen @giginet 's requested changes, also, have you tested this since the new changes that Apple rolled out to iTC? Would be good to know if it is still a working change.

Hey @tmm1 - Wanted to bump this PR again just to see if you had a chance to look at @giginet's comments? Since this is a pretty old PR we should try to get this in or closed in the near future 😄 🚀 🐙


bug in dropdown()

There appears to be a bug in dropdown().  If only $order, but not $attributes is passed in, the index 2 is incorrect for attributes and $attributes will end up having the integer value for $order.  Perhaps check $order to make sure it's an array if people leave $order out of the arguments?  Will fix and create pull request as soon as I can figure out how to set up my SSH keys.  Let me know if I am incorrect.
   public function dropdown($title, \Closure $callback, $order = null, array $attributes = array())
    {
        $properties = compact('title', 'order', 'attributes');

        if (func_num_args() == 3) {
            $arguments = func_get_args();

            $title = Arr::get($arguments, 0);

            // If only $order is passed in without attributes, 2 is the wrong index
            $attributes = Arr::get($arguments, 2);

            $properties = compact('title', 'attributes');
        }

        $item = MenuItem::make($properties);

        call_user_func($callback, $item);

        $this->items[] = $item;

        return $item;
    }

Can anyone help?  I'm trying to push a bugfix branch and getting:
remote: Permission to nWidart/laravel-menus.git denied to douglassdavis.
fatal: unable to access 'https://github.com/nWidart/laravel-menus.git/': The requested URL returned error: 403
What do I need to do to be able to fix a bug?


Lot/Sale state updates

OK, this should be the last of the architectural changes needed to drive the UI from state updates / socket events.
What
Previously, we had a lotStateSignal on a lot view model. This was really tricky, since the state of a lot depends on the current lot ID, which is a property of the sale. One option would be to iterate through every lot whenever the current lot ID changes and update them based on their state (which we would need to computer). Instead, we use a computed lot ID signal.
Why
Using a signal (returned from a function) is beneficial because the view controllers that are interested in the lot state can retain the signal, and when the VCs are deallocated, the work in updating the signal goes away too. I'm somewhat concerned about performance around signals whose subscriptions can't be disposed, and this approach helps mitigate that concern.
Issues
It's a bit weird to have the lot view model receive the auction view model as a function parameter, but the lot needs both the current lot ID and the list of lots in order to compute its state. It's awkward, but it works.
Also
The PR changes how the auction view model handles sale changes. I've made it just replace its model, and remove the start/end signals (since we don't have any UI that depends on those properties).
Also, it corrects the order in which we display the lots.

2 Warnings

:warning:
Needs testing on a Phone if change is non-trivial

:warning:
Found unstubbbed networking requests

ARFairFavoritesNetworkModelSpec in test_ignores_artworks_without_a_partner
ARFairFavoritesNetworkModelSpec in test_ignores_artworks_without_a_partner
ARFairFavoritesNetworkModelSpec in test_ignores_artworks_without_a_partner

  Generated by :no_entry_sign: danger

Cool, looks good.

:bow:


Error with handling baseURL

When pushcasette is put into a subfolder, for example /discography/, and the links in the pushcassette-menu are edited in cassette.json to send you to a page such as /about/, the menu sends you to /discography/about/, even if the baseURL is edited accrodingly in the index.html file. Also the logo sends you back to the "release page" instead of the (root) of the site.
I also tried to set the homePage value in cassette.json to / but it returns a broken pushcassette.
However, this is looking awesome! I love it so far!

Hmmm...does the site load at all (or do you get missing css/js)? Can you post what you have in your cassette.json? It's possible you might have some server side caching going on, try and view source on your index.html in the browser to confirm the baseURL actually changed.
I don't think you can set homePage to / as homePage is basically a redirect for /. The homePage path must exist as a JS route (not a server path). What were you expecting to happen by setting homePage to / ?

The site loads, no error (unless i change the homePage value, then it loads but no release to be seen)
Here is how the cassette.json looks:
{
  "lastBuild": {},
  "settings": {
    "homePage": "releases",
    "cleanURLs": false
  },
  "pages": {
     "releases" : {},
     "about" : {"location" : "/about/"},
     "radio" : {"location" : "/radio/"}
  },
  "releases": {
    "example-release": {
      "title" : "Cosmic Voyage",
      "playlist" : "releases/example-release/tracklist.jspf",
      "artwork" : "releases/example-release/artwork.jpg",
      "notes" : "releases/example-release/notes.md"
    },
    "example-release-two": {
      "title" : "Bird Life",
      "playlist" : "releases/example-release-two/tracklist.jspf",
      "artwork" : "releases/example-release-two/artwork.jpg",
      "notes" : "releases/example-release-two/notes.md"
    }
  }
}

Yes, the baseURL actualy changed, no server caching going on.
I want to have pushcassette in a subfolder, i want to visitor the be able to go back to the main site and the rest of the main sites sections via the pushcassette menu.

Aha, I think I see the problem. The issue is that the location path for pages expects to see a file, such as /pages/about.md. which gets loaded when you go to /about. If instead you'd like to redirect to /about/ you probably don't want to use the generated JS menu. In that case, you can just create a regular HTML menu in index.html with relative URLs. To get the logo and site name to link to / just remove the data-navigo attribute on them (the navigo JS routing library works by hijacking links to make them load using JS).

ah! awesome! :) thank you! :)

Thinking on this some more - it might be nice to be able to redirect to server paths within cassette.json...maybe a type="external" attribute might allow for more mixed / flexible menus.
Something like:
type="internal" (load markdown file via AJAX)
type="external" (treat as a normal path)

Potential for confusion there, but I can see wanting to add a link to Soundcloud or Bandcamp or something and an option like that might make life much easier.

@zirafa : i just need to big you up; your dedication and attention to detail is remarkable!


Duplication of pre-rendered pages in Pascal and Lower Case on 'generate' at production [DigitalOcean App Platform].

Environment
Nuxt project info:

Operating System: Windows_NT
Node Version:     v18.13.0
Nuxt Version:     3.0.0
Nitro Version:    1.0.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:     nitro, alias, modules, sitemap, runtimeConfig, twicpics, plugins, app
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>~/modules/sitemap
Build Modules:    -

Reproduction
Could be a potential issue with Digital Ocean as the 'generate' process seems to work fine on my local machine.

Here is the output from router.getRoutes() in dev mode, as you can see there is no pascal case here either.

Describe the bug
When I run 'generate' on my local system all routes are pre-rendered as expected. However, when I push to production the build process seems to pre-render duplicates of my non-dynamic pages in pascal case as well as lower case (should only be lower case). These duplicates also appear in my dynamically generated sitemap at prod on D.O but not in the output file on my local machine.
I should mention that I did initially have my .vue files for the pages named in pascal case (as per Vue conventions) but renamed them after seeing different conventions used by others in Nuxt.
Is there a file in .nuxt with router declarations where the pascal case might still be saved?
This is causing me some SEO issues and shouldn't be happening. Any ideas on how to debug the issue?
Additional context
No response
Logs
No response

Would you provide a reproduction? 🙏

Hey there! Please add a minimal reproduction so we can look into that properly.
Since there isn't any activity here for a longer time, I'd suggest closing this, but feel free to reopen. Thank you 🙏🏻💚

Sorry, I don't really know how to do that as the issue doesn't arise until I ship to Digital Ocean. Everything else looks entirely normal when I run 'generate' on my local system. It only adds the extra pages with Pascal Case when running 'generate' on the DO Server/App Platform.
If you can advise on how best to do it I will happily look at it. Or if you can suggest a place where I can find the config files for the router in nuxt to see if it has messed up the routes. (In the past I have renamed components and had to adjust the components.ts file when it hasn't updated properly. Is there an equivalent somewhere for the router? I cannot find it.

@driven-group Well, it sounds like DO has cached these routes somehow since you renamed them at some point. Otherwise I wouldn't have known why they would do something like that 😄.
A minimal reprod must not be a StackBlitz (mostly it's the best though), of course you can also create a minimal nuxt project with the CLI, add some example pages in it, and you sharing exactly the config/cli of DO and the preview urls.
I suppose on a fresh newly deployed project this should not happen.

@madebyfabian I thought this myself however I have forced a rebuild and deploy without using cache and this didn't change the output. I also (after reading someone's similar caching problem on StackOverflow) tried setting the node and npm engines in the config file for the ones that DO specifically uses. This forced it to re-parse the package.json and build from scratch but it still pre-renders duplicate pascal case files.
I imagine that a fresh install shouldn't do it. But,  when I get time, I will try to set one up with the CLI and test on DO.

Do you use the Preset from Nitro?

@madebyfabian yes. I have included it in my nuxt.config under nitro: { preset: 'digital-ocean' } and also as an environment variable [NITRO_PRESET]  in the App Platform.

Let's wait for the result of your freshly deployed project so we discuss that. I'm happy to help, but I think it's fine to let this issue closed right?

Yeh sure. it definitely sounds like a DO issue and not a Nuxt one. Am I able to reply to you here after testing though? Even if it is closed?

@driven-group sure, it's already closed and out comments work regardlessly 😊


Can this be used on server side with node?

Not support that.

Can i submit a pull request if i get it working? An api using your compression software would be awesome!

Why not just use server side solution as imagemin for node?


Tracking issue for release of DF 42

Tracking issue for release of DF 42. Please let us know if there's a particular issue you're waiting on!
PRs under review:

[x] #843
[x] #873
[x] #878

Issues to be resolved and included in release:

[x] #845
[x] #804
[x] #701
[x] #822

@Michael-J-Ward should we try to address #701 ?

Add it in.
I upgraded the maturin version here: https://github.com/apache/datafusion-python/pull/725. And that has allowed the rest of the datafusion-python versions to build for conda-forge.
The last remaining conda-forge PR mentioned in #701 is https://github.com/conda-forge/datafusion-feedstock/pull/46, which - from what I can tell - is blocked by something other than the maturin issue.

Removed #869 because I think we need some upstream support before we can do this. Only very simple cases currently encode right now. We could expose it as is, but I think that might not be helpful since it's not fully usable.

Released on 10/11/2024


Add alias for dotnet workload uninstall command

dotnet workload uninstall - Uninstalls a specified workload. Introduced with .NET 6 SDK.
The dotnet workload uninstall command uninstalls one or more workloads.
Docs: https://docs.microsoft.com/en-us/dotnet/core/tools/dotnet-workload-uninstall

/assign

Please assign it to me


Remove implementation details from Javadoc for Literal.hashCode and BindingSet.hashCode

Followup of #676 and #668 . The Javadoc in the interfaces for Literal and BindingSet contains too much implementation detail. This should be removed as it can falsely lead users to assume the implementation will never change.

We are not executing this change.  Quoting @ansell :

Users will virtually always be using Value objects from different ValueFactory's, both internally and from third-parties. If they add these Value objects to a HashSet or a LinkedHashModel, they want to be sure that they will obtain consistent results. Having the hashCode contract stated explicitly (whatever its contract turns out to be) is the only way to ensure that core promise of the Java Collections API (and out Model API) in the long term across different implementations. This partly relates to the removal of ValueFactory when we changed from Graph to Model, but users will expect to be able to use the hash-based Java Collections API in the same way.


Address bar broken

Release Version
ANY
Describe the bug
On High-DPI Displays, the address bar may appear smaller than it should be
To Reproduce
Steps to reproduce the behavior:

Open Simple-Browser on a High DPI Display

Desktop:

OS: Windows

This will be fixed in the next UI overhaul (Complete Redesign)


good malware

Describe the bug
A clear and concise description of what the bug is.
Steps to reproduce
Steps to reproduce the behavior.
Expected behavior
A clear and concise description of what you expected to happen.
Environment

OS: [e.g. Arch Linux]
Other details that you think may affect.

Additional context
Add any other context about the problem here.

good malware in release file don't download release file guys

download releases source and compile yourself, and you will get same application


Client process randomly terminates

Describe the bug
Client sometimes randomly closes for me. It happened once last week, and occured again today.
To Reproduce

Be logged in for a certain amount of time
Client randomly closes

Expected behavior
It shouldn't close randomly.
Environment (please complete the following information):

OS: Windows 10
RuneLite version: <IP_ADDRESS>
Launcher version: 2.0.2

Additional context
From the error log called jvm_crash_pid####, this is said at the top, can provide full file if needed:
A fatal error has been detected by the Java Runtime Environment:
EXCEPTION_ACCESS_VIOLATION (0xc0000005) at pc=0x00000213341463cb, pid=8692, tid=7472
JRE version: OpenJDK Runtime Environment (11.0.4+11) (build 11.0.4+11)
Java VM: OpenJDK 64-Bit Server VM (11.0.4+11, mixed mode, tiered, compressed oops, g1 gc, windows-amd64)
Problematic frame:
J 18717 c2 dv.([Ldv;I)V (1154 bytes) @ 0x00000213341463cb [0x0000021334143fc0+0x000000000000240b]
No core dump will be written. Minidumps are not enabled by default on client versions of Windows
If you would like to submit a bug report, please visit:
https://github.com/AdoptOpenJDK/openjdk-build/issues

Do you happen to be running navi based GPU like amd 5700 series? i'm trying to debug it a bit.
amd 3700x, 5700xt, windows 10 (1909).

Im running a ASUS Dual RTX2080, CPU im using is a AMD Ryzen 7, 8 Core 3.89 GHZ

Also having this issue. Sometimes it has happened when I right click -> lookup a person. Other times when opening the plugins menu. And sometimes while just playing the game.

I would say it is same boat as many other games and programs getting violation errors with ryzen CPUs. Googling "ryzen 3000 0xc0000005" will yield a lot of similar problems. So it would be a issue with chipset driver or with AGESA.
Cannot really point where it would come in code as it seems to rise at different places.
It's either EXCEPTION_ACCESS_VIOLATION or more specifically or while debugging:
java.lang.ArrayIndexOutOfBoundsException: Index 16777287 out of bounds for length 104
at hg.fz(hg.java:1209)
at el.zp(el.java)
at el.fv(el.java:16996)
at aj.gh(aj.java:4183)
at az.io(az.java:9062)
at az.io(az.java:9093)
at bb.iq(bb.java:8951)
at client.hp(client.java:3862)
at client.ar(client.java:1207)
at bo.lw(bo.java:385)
at bo.run(bo.java:339)
at java.base/java.lang.Thread.run(Thread.java:830)
Error: aj.gh() az.io() az.io() bb.iq() client.gs() client.ar() bo.q() | hg:1209 el.zp el:16996 aj:4183 az:9062 az:9093 bb:8951 client:3862 client:1207 bo:385 bo:339 Thread:830 | java.lang.ArrayIndexOutOfBoundsException: Index 16777287 out of bounds for length 104
error_game_crash

Does this happen on the latest release of RuneLite?

Does this happen on the latest release of RuneLite?


hairlineWidth import breaking styles and darkMode: class not being set

Describe the bug
import { hairlineWidth } from "nativewind/theme" instead of const { hairlineWidth } = require("nativewind/theme"); seems to be breaking styles.
also, related to #587, I am getting Unable to manually set color scheme without using darkMode: class
To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/trevorpfiz/create-t3-turbo-supabase/tree/05-15-nativewind_hairlinewidth_bug_and_darkmode_bug
Run pnpm run dev in apps/expo directory
Switch to Expo Go
Toggle dark mode in top right
See error

Expected behavior
I did not expect import vs require to break styles for hairlineWidth import.
I expected the dark mode to be toggled.
Screenshots

Additional context
I updated https://github.com/mrzachnugent/react-native-reusables/tree/main/apps/starter-base to the same dependencies, and things seemed to be working there. I used npm instead of pnpm. Don't know what else is different.

turns out this is from "react-native-css-interop": "~0.0.34"...
either remove it or use "react-native-css-interop": "~0.0.36"
the hairlineWidth import breaking styles issue I believe was related to needing to clear the cache with npx expo start --clear


Fixed typo in total budget amount for Volunteer Stipends

Signed-off-by: Mahfuza Humayra Mohona<EMAIL_ADDRESS>
Proposed changes
I noticed a typo in the total budget amount for Volunteer Stipends. Thought I would fix it as it might potentially lead to confusion.
Types of changes
What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices applies)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[ ] I have signed the commit for DCO to be passed.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)

Dependency

Please add the links to the dependent PR need to be merged before this (if any).

Special notes for your reviewer:

It was running total, not only cost for volunteers. 😅
Sorry for my misunderstanding.


try to avoid cloning

A way to avoid cloning strings.

@pluehne I'm not sure how to improve this. I think there must be some kind of memory copying here because the borrowed puzzle is being modified. I'm just trying to avoid cloning a string.


upstream regression in kitty 0.24.4 (ValueError: invalid literal for int() with base 10: '')

Hi,
I'm getting the following error trying to use the package: ValueError: invalid literal for int() with base 10: ''
Here is the full stack trace
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/pandas/plotting/_core.py", line 1758, in hexbin
    return self(kind="hexbin", x=x, y=y, C=C, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/pandas/plotting/_core.py", line 917, in __call__
    return plot_backend.plot(data, x=x, y=y, kind=kind, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/pandas/plotting/_matplotlib/__init__.py", line 72, in plot
    plot_obj.draw()
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/pandas/plotting/_matplotlib/core.py", line 323, in draw
    self.plt.draw_if_interactive()
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/matplotlib/pyplot.py", line 319, in draw_if_interactive
    return _backend_mod.draw_if_interactive(*args, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/matplotlib-backend-kitty/__init__.py", line 79, in draw_if_interactive
    cls.show()
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/matplotlib-backend-kitty/__init__.py", line 83, in show
    _Backend.show(*args, **kwargs)
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/matplotlib/backend_bases.py", line 3524, in show
    manager.show()  # Emits a warning for non-interactive backend.
  File "/opt/homebrew/Caskroom/miniconda/base/lib/python3.8/site-packages/matplotlib-backend-kitty/__init__.py", line 44, in show
    px = list(map(int, px.split('x')))
ValueError: invalid literal for int() with base 10: ''

I am using Kitty 0.24.4 under MacOS 12.3 with Python 3.8.13 installed with Conda and Matplotlib 3.5.1

Actually, this seems to have already been fixed on kitty master: https://github.com/kovidgoyal/kitty/pull/4818
So you could wait for 0.24.5 to be released, run kitty nightly in the meantime, or downgrade.

Interestingly, with kitty 0.25.0 it works when using iPython but using a Jupyter console it still gives the same error.

kitty 0.25.0 and 0.25.1 have been released and are likely available in most distros' repos by now; both of them seem to work fine on my end. My thanks go out to @nertro for fixing this issue upstream.

Hi @julianfritzsch , I just want to ask if you have found any solution for this error on Jupyter Console?

Hi, unfortunately it does not work with the Jupyter console for me but I get an error
Error: Failed to open controlling terminal with error: open /dev/tty: device not configured

However, it works perfectly fine with iPython for me


How to add Numbering Column at first column?

Does somebody can tell me the best way to add the numbering column (1,2,3,4 ... ) at the first column?

Not sure whether it is best way or not. I did something like below and it's work for me!
var i = 1;
        const renderNumber =
        (val, row) =>
        (
            i++
        );

const tableColumns = [
{ title: 'No', render: renderNumber, className: 'text-center'},
{ title: 'Region Code', prop: 'region_code' },
{ title: 'Region Name', prop: 'region_name' }
];


Render only specified element

[x] Store original id's
[x] Detect item bounding box #33
[x] Add method to query id's
[x] Add option to CLI

Hi. I am a GSoC student working on the project "Adding OpenType SVG font support to FreeType". For this project, we are evaluating different SVG rendering libraries to ultimately seek out what works best for us. Resvg is one of the candidates.
An important feature we need is the ability to render a <g> element that has an specific id. Why we need this feature and how we plan to use it is described in the section Glyph Identifiers of the OpenType SVG specs. You can see that here.
I was happy to see that resvg provides a function named resvg_cairo_render_to_canvas_by_id. The SVG document that I tried it is attached below. However, when used, the method returns an error, "A node with glyph8 ID was not found". So I started looking and it looks like that when convert_doc is called, the new simplified tree doesn't have those Group elements whose ID is being searched for.
What I want to ask is, am I doing it wrong? or is it expected to behave this way? if yes, do you plan to fix this issue? I am attaching my main file and the svg document for your convenience. The program is written in C.
svg-file.txt
prog.txt

You should enable keep_named_groups.

Oops, missed that. Thank you very much, that works. :+1:


STG88 Beta Release

Description
Beta release for STG88 features.

API change check
API changes are not detected in this pull request.


Landingpage

Feat/landingpage
TL;DR

Updated landing page design
Changed back button design, created pagination button, deleted old buttons
Updated cards, sections and grid components
General update and clean up -> v2

Details
TODO:

[ ] N/A

Checklist / Reminders:

[ ] Documentation is created / adapted
[ ] Cypress Tests

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


Re-enable default tcp_keepalive

Re-enable default tcp_keepalive.
Fixes seanmonstar/reqwest#1184

Hello ! Any reason why it is not pulled yet ?
Thank you for the library anyway :smiley:

@seanmonstar is there anything blocking this PR?


Run tests from cli

Hi, I've been using robospock with android studio and it's working quite well. I can run my tests from the IDE with no problems. However, I'm now attempting to set up a CI server, and I'm having difficulty executing the tests from command line.
When running ./gradlew clean assemble test I can see the test tasks are being executed, but my tests are not actually running. The build always succeeds even when I add tests that should always fail:
public class FailureTest extends Specification {
   def "Failing Test"() {
      expect:
      false
   }
}

I expect I'm just missing something simple like a config option. Any help would be appreciated :)

Which version of gradle groovy plugin are you using?

'org.codehaus.groovy:gradle-groovy-android-plugin:0.3.6'

It depends on Google Android plugin. Versions lower than 1.3.0 are compatible with 0.3.6 plugin. However newer not. Unfortunately authors of didn't release a fix to central repository.
In my project I use jitpack to make it work:
buildscript {
  repositories {
    maven {
      url "https://jitpack.io"
    }

  dependencies {
    classpath 'com.github.groovy:groovy-android-gradle-plugin:570c650ee00622120ad0d0653f8329b64321f37d'
  }
}

Sweet, using jitpack with that commit seems to have fixed the issue. Cheers!


Python SDK: Support pydantic 1.x + 2.x environments?

This is an adaptation of the concept used by pydantic/bump-pydantic - uses libCST to replace pydantic imports with:
try:
    from pydantic.v1 import X
except ImportError:
    from pydantic import X

Where X is a member of pydantic v1.
This should work with both pydantic 1.x and pydantic 2.x given their backwards compatibility.
We can think about introducing this as a post processing step after code generation with the current generator.
(context: possible solution for #6903)

@ozkatz  I'm already working on a solution for this...

No worries! Feel free to close this one

superseded by #7672


[language][move] Add parse_comma_list function to the parser

This is the move-lang equivalent of https://github.com/libra/libra/pull/1931. In the same way that the IR compiler change made some commas mandatory, this does the same for function parameters, where all commas were previously optional.
This PR also changes to consistently allow optional trailing commas in all comma-separated lists.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
I updated the test for function parameter parsing since commas are no longer optional.

OK. As I commented in the initial description of this PR, I was planning to tackle any language changes separately, but if you'd prefer we can include that here. I suspect there may be some bikeshedding involved....
I did some reading about where and when to allow/use trailing commas. Here are a few highlights:

People have strong preferences here (in both directions) so giving them the option of using trailing commas may be valuable.
When code is generated automatically, it is somewhat easier to just emit a comma after each item in a list. I don't expect this to be very relevant for Move, and it's not a very strong argument anyway.
When items are on separate lines, trailing commas make it easier to modify the list (i.e., without having to add a comma to the previously final entry). Most arguments about this topic focus on the multi-line situation. I don't remember seeing anyone arguing for using trailing commas when a list is on one source line.
Some languages and/or code formatting tools look at whitespace and distinguish the case where there is a newline after a list item, requiring/using a trailing comma then but not otherwise. For the sake of simplicity, I'd rather not go that route, but it is an option.

I reviewed the different kinds of comma-separated lists in Move:

Fields (in struct definitions, bindings, and pack operations)
Function arguments (both definitions and calls)
Type parameters for generics
Things related to multiple return values: Expressions (in expression lists), Types (in tuple-like return types), and Bindings (when binding multiple return values)
Acquires

@tnowacki suggests here that we change to ban trailing commas for everything except field lists and possibly function arguments. IMO, the function arguments should be consistent with generic type parameters, so if we're not going to allow trailing commas for type parameters, I propose that we also ban them for function arguments.
At the other extreme, we could choose to allow optional trailing commas for all of these. (It would be a minor hassle to parse that for the "acquires" lists, but not a serious problem.) That would be very consistent and leave it up to individual developers to decide which style they like. I don't necessarily want to advocate for that, but it is something we should at least consider.
Personally, I would be OK to say that trailing commas are only allowed for field lists.

the function arguments should be consistent with generic type parameters

I'm not sure if I follow why this is true. For me its more that function arguments need to be consistent with ExpList. I don't see it being necessary to be consistent with types.
Definitely though, type parameters need to be consistent with type arguments

I'm not sure if I follow why this is true. For me its more that function arguments need to be consistent with ExpList. I don't see it being necessary to be consistent with types.

Just that when you see a generic function there are 2 comma-separated lists of parameters, one for the types and the other for the arguments. It would be inconsistent and confusing if the rules for trailing commas would be different between those 2 lists.

It would be inconsistent and confusing if the rules for trailing commas would be different between those 2 lists.

I don't see how its inconsistent or confusing. No more confusing than allowing trailing commas on pack but not on call

These are all different cases, I don't think it is confusing or inconsistent to have trailing commas one thing and and not on another, even if those things are next to each other in code.
Keep in mind type arguments also exist at pack and unpack, alongside call and call. I think its totally reasonable for types to have a different rule than values/expressions

:umbrella: The latest upstream changes (presumably b77f031149157a97b94e8c0b7852031720c817af) made this pull request unmergeable. Please resolve the merge conflicts.

Todd and I discussed this in person. We agreed that the decision of where to use trailing commas is best left to formatting tools, where it is more natural to consider whitespace, e.g., whether a list item is followed by a newline. Otherwise, it is best for the compiler to be consistent and permissive, so that it doesn't frustrate advanced users who have strong opinions about using trailing commas. I've gone ahead and added changes here to allow trailing commas in all comma-separated lists.

@bors-libra r=tnowacki

:pushpin: Commit 1186122 has been approved by tnowacki

:hourglass: Testing commit 118612214eaf2d72e8f330a291f3df075ae150f2 with merge fc084518a05d209f517ca813d85d50c865b71b9e...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: tnowacki
Pushing fc084518a05d209f517ca813d85d50c865b71b9e to master...


Pagination Bug

No longer getting console error expecting 'number'.

Fixes pagination so it no longer makes it a string. Number only.

closes #46


[Bug] The main method caused an error: StreamTableEnvironment can not run in batch mode for now, please use TableEnvironment.

Search before asking

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

What happened
After I changed the mode of the built-in sqldemo from streaming to batch, I reported an error
StreamPark Version
1.2.4
Java Version
No response
Flink Version
1.13.5
Scala Version of Flink
2.12
Error Exception
After I changed the mode of the built-in sqldemo from streaming to batch, I reported an error

Screenshots

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Thanks for your feedback,  It's a bug, we'll fix it quickly


Feature/59 create test cases

Description
Add a sample project files to test the build after a change.
npm run build:test

Related Issue
Fixes #59
Types of changes

[x] Test (non-breaking change which adds tests)

Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All new and existing tests passed.

@quicoto @easingthemes I was thinking of adding git hook to run the test (build the sample project) before any commit.
What do you think?

@raohmaru yeah big fan of pre-commit hooks or pre-push. Go for it :)


Docker Compose - ElasticSearch / Logstash Communication issue

Trying to use docker-compose up in order to create the docker containers.
I have used the master branch before, and it works beautifully, but trying the searchguard branch out, I have an error where logstash will repeatedly output:
logstash_1       | [2018-08-10T20:00:31,869][INFO ][logstash.outputs.elasticsearch] Running health check to see if an Elasticsearch connection is working {:healthcheck_url=>http://logstash:xxxxxx@elasticsearch:9200/, :path=>"/"}
logstash_1       | [2018-08-10T20:00:31,873][WARN ][logstash.outputs.elasticsearch] Attempted to resurrect connection to dead ES instance, but got an error. {:url=>"http://logstash:xxxxxx@elasticsearch:9200/", :error_type=>LogStash::Outputs::ElasticSearch::HttpClient::Pool::HostUnreachableError, :error=>"Elasticsearch Unreachable: [http://logstash:xxxxxx@elasticsearch:9200/][Manticore::ResolutionFailure] elasticsearch"}

Any known reason on why this might be happening?
No other docker containers are running.

Did you run sg_admin as suggested in the README?

Yes, but that doesn't seem to work either; it just gives me this error:

root@ubuntu:~/docker-elk# docker-compose exec -T elasticsearch bin/init_sg.sh
OCI runtime exec failed: exec failed: container_linux.go:348: starting container process caused "exec: "bin/init_sg.sh": stat bin/init_sg.sh: no such file or directory": unknown

It doesn't even get to a point where all of the containers can successfully run.. Only dockerelk_logstash_1 will be running when doing a  docker ps, and if I try to start up dockerelk_elasticsearch_1, it will just stop itself right away..

Seems like you forgot to run docker-compose build after switching branches.

D'oh! Didn't see that note in there. That is absolutely correct. That was my issue. Thanks for your quick response!


[WFCORE-1611] fix CLI stuck if SIGINT is send during authentication dialogue.

https://issues.jboss.org/browse/WFCORE-1611
Fix a potential deadlock on handlers in issue reported scenario.

Full integration - Windows Build 1428 outcome was FAILURE using a merge of 4e193b06d745eb1978175f3a598bdfe02d9bc8a8
Summary: Tests failed: 1, passed: 3381, ignored: 194 Build time: 01:15:27
Failed tests
org.jboss.as.test.integration.security.loginmodules.LdapExtPasswordCachingTestCase.test3: java.lang.AssertionError: Password from cache 1 expected:<1> but was:<2>
	at org.jboss.as.test.integration.security.loginmodules.LdapExtPasswordCachingTestCase.test3(LdapExtPasswordCachingTestCase.java:182)
------- Stdout: -------
reseting file writer
counterFile = C:\BuildAgent\work\38e1ea00f932992d\full\testsuite\integration\basic\target\tmp.password
callsCounter = -1
reading counter file: C:\BuildAgent\work\38e1ea00f932992d\full\testsuite\integration\basic\target\tmp.password
10:20:19,227 INFO  [org.jboss.as.test.integration.security.common.Utils] (main) Requesting URL http://[::1]:8080/test-DEP3/printPrincipal
10:20:22,184 WARN  [org.apache.directory.server.ldap.LdapSession] (pool-29-thread-1) AbandonableRequest with messageId 2 not found in outstandingRequests.
callsCounter = -1
reading counter file: C:\BuildAgent\work\38e1ea00f932992d\full\testsuite\integration\basic\target\tmp.password
10:20:22,730 INFO  [org.jboss.as.test.integration.security.common.Utils] (main) Requesting URL http://[::1]:8080/test-DEP3/printPrincipal
callsCounter = -1
reading counter file: C:\BuildAgent\work\38e1ea00f932992d\full\testsuite\integration\basic\target\tmp.password

@aloubyansky @jfdenise Please approve

This is good to merge. Thanks!


merge in the upstream

This merges in the upstream in to the zephyr "fork"

closing as I see 0.8.0 was pushed


Fix tqdm encoding error on python2.6

Signed-off-by: Xuanwo<EMAIL_ADDRESS>

Add test for py26 would be better.

LGTM, I will add it.


Refreshing calendar when i try to render a new json file

Hi everyone,
Is any ways to refresh the calendar when i try to render a new json file ?
For example i load a json file and i render it on the calendar, i make a modification on my groups, i delete one group, how can i re-render the calendar without refreshing the page ?

you need to provide a new instance of groups.

NB! All props need to be immutable. For example, this means if you wish to change the title of one of your items, please pass in a whole new items array instead of changing the title in the old array. Here's more info.

you can find an example of this here. every time you open a node you will get new groups instance

I can't make it work :(
there is my code : consider this like the GenerateFakeData function :
        let newGroups = []
        for (var i = 0; i < selectedOption.length; i++) {
            newGroups.push({id: selectedOption[i].value, title: selectedOption[i].label})
        }

        groups = newGroups

when i do groups = newGroups there is no refreshing of group hapenning


e2e tests: Reuse VMIs in multiple tests

What this PR does / why we need it:
In Kubevirt e2e test suite, there are test suites/tests-batches where the exact same VMI is created before each test but each test asserts different things regardless of the VMI spec.
In other words, we could use a single VMI for all the tests in the batch.
Since Ginkgo v2.8.0, a new decorator was introduced ContinueOnFailure which enables an Ordered tests container (e.g: Context/Describeclouse that uses BeforeAll/AfterAll) to tolerate failing tests and continue.
(The default behavior for Ordered tests-containers was to exist early as soon as a test fails).
The new  ContinueOnFailuredecorator was the missing part in order to introduce tests that reuse VMIs.
Naturally reusing VMs enables cutting down the overall time of tests as we save the boot time each VM takes.
For example:
Assuming the global test suite setup and teardown takes 1 minute and VMIs boot time is 1 minute.
Given a batch of four tests that takes 5 minutes to finish, by reusing the same VMI we can cut down the time to 2 minutes.
This PR PoCs resuing VMIs in multiple tests and takes the network Expose tests suite as an example.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
This PR:

Bumps Ginkgo to v2.8.3 and Gomega to v1.27.1.
Introduces additional test-containers decorators in order to make the global-test-suite-clean-up skip VMIs and virt-launcher pods deletion (and enable reusing them for other tests).
Changes network Expose tests suite to reuse VMIs.

By having the combination of Ordered, ContinueOnFailure, retain-VMIs decorators and setting a VMI creation in a BeforeAll clause, its possible to reuse VMI in multiple tests:
Context("running virtual machine", Ordered, ContinueOnFailure, RetainVirtualMachineInstances, func(){
    var vmi *v1.VirtualMachineInstance
    BeforeAll(func() {
        vmi = createVMI()
    })
    It("case1", func(){
        Expect(vmi.Annotations).To(...)
    })
    It("case2", func(){
        Expect(vmi.Status).To(...)
    })
})

The last commit verifies:

The new ContinueOnFailure actually works and a falling test doesn't cause the whole suite to exit.
Failing test artifacts in an ordered test container are collected
It's for debugging and will be removed soon.

Release note:

/test pull-kubevirt-e2e-k8s-1.26-sig-network

/cc @EdDev @phoracek

/test pull-kubevirt-e2e-k8s-1.26-sig-network

/cc

/test pull-kubevirt-e2e-k8s-1.26-sig-network

/test pull-kubevirt-e2e-k8s-1.26-sig-network


CLOUD-61427 add blueprint status to response

@doktoric plz check

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/322/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2831/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/324/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2832/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/325/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2833/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/326/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2834/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/327/


Mosh Implementation

I have absolutely no expectation this will get merged. For anyone that wants it however, and for my own future reference, here's a patch I recreated after I lost it two years ago.
Embedded Mosh on api29+

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

Oh, this is interesting actually. I wonder if there is a way to split this out so there is no requirement to have a certain ABI.

AFAIK those are all the ABIs possible to use on android anyway. It does have to be compiled per-architecture, unless someone is enthusiastic about reimplementing mosh on the jvm... (llvm magic...?)

Right now you can do a split per ABI, but I don't know if it allows you to just not have the native libraries. The ideal situation is that Android supports wasm! 🥴


[PE-D][Tester D] Person name can contain digits

Current Behaviour
A person name can contain digits.
Expected Behaviour
A person name should not contain digits.
Steps to reproduce

We can edit a person's name to contain digits.
Enter edit S1234567 n\12345

The person has digits in their name.

Why this is an issue
A person name should technically not contain digits, but digits could be contained in the name.
From the user guide, it only says special characters are allowed, but digits are not special characters.

Suggested Change
Apply validation to names to prevent having numbers in a person's name.
Or update the documentation to reflect that digits are also allowed.

Labels: severity.VeryLow type.FeatureFlaw
original: tituschewxj/ped#2

duplicate to #202


Add current_humidity attribute for dehumidifiers

Is your feature request related to a problem? Please describe.
The currenty humidity attribute was added as part of Home Assistant 2023.7 core but it has yet to be implemented with this integration:
https://www.home-assistant.io/blog/2023/07/05/release-20237/#other-noteworthy-changes

The current humidity is currently an entity of it's own, but it should perhaps be included in the humidifier's attribute so that the current versions of the humidifier cards would display it.
Describe the solution you'd like
Currently the default humidifier card does not show the current humidity:

However, if the current_humidity attribute was added, the built in humidifier card would work perfectly

Describe alternatives you've considered
I've tried using a python script to update the current_humidity attribute but it always gets blanked out after a few seconds.

I made a PR (#128 ) for this issue, it`s only one line in the humidifier.py

Awesome. Thank you! I copied that line of code into my humidifier.py. Hope this will get merged at some point.

Whoops... accidentally closed the issue.

Integrated


TAS: introduce scheduled-level tests

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
To do not work-around accounting for TAS pods and workloads.
Which issue(s) this PR fixes:
Part of #2724
Special notes for your reviewer:
This was ticked as "Account for usage from TAS workloads in the unit tests for cache" in the spreadsheet.
I'm going to expand the tests in follow up tasks.
I moved already the most "workaround"-ish test, but I could move tests in the future, let me know.
Does this PR introduce a user-facing change?
NONE

/cc @PBundyra @tenzen-y

/cc @mbobrovskyi

/lgtm
Thanks!


BUG: Dark mode is not working properly

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When a user enables dark mode some of the text on the settings page , it does not appears properly . The font color of these text should be changed .
@hereisSwapnil kindly assign me this issue I can handle this bug .

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'24 contributor
[X] I want to work on this issue

Hello, @hereisSwapnil and @Apoorva-Rajput, I'm new to open source contributions. I would like to work on this issue or help if someone working on it is struck somewhere. I kindly request you to consider this.

https://github.com/akshitagupta15june/Moksh/issues

Hey, allow me to work on this under GSSOC' 2024


Interchange KeyError due to too many heartbeat missed

Running this code snipset on Theta always fails.
import parsl
from parsl.providers import LocalProvider, CobaltProvider
from parsl.channels import LocalChannel
from parsl.launchers import SingleNodeLauncher, AprunLauncher
from parsl.config import Config
from parsl.executors import HighThroughputExecutor
from parsl.app.app import python_app  # , bash_app
from parsl.addresses import address_by_hostname
import time

theta_config = Config(
    executors=[
        HighThroughputExecutor(
            label="theta_htex",
            max_workers=1,
            address=address_by_hostname(),
            heartbeat_threshold=6,
            heartbeat_period=2,
            provider=CobaltProvider(
                queue='debug-flat-quad',
                account='CSC249ADCD01',
                launcher=AprunLauncher(overrides="-d 64"),
                worker_init='source activate parsl-issues',
                init_blocks=1,
                max_blocks=1,
                min_blocks=0,
                nodes_per_block=1,
                cmd_timeout=60,
                walltime='01:00:00',
           ),
        )
    ],
    strategy='simple',
)

dfk = parsl.load(theta_config)

@python_app
def double(x):
    return x * 2

if __name__ == '__main__':

    fut1 = [double(i) for i in range(10)]
    res1 = [f.result() for f in fut1]
    print("Task 1 result is {}".format(res1))
    print("Sleeping for 4 minutes")
    time.sleep(240)
    print("Finished sleeping")
    fut2 = [double(1) for i in range(10, 20)]
    res2 = [f.result() for f in fut2]
    print("Task 2 result is {}".format(res2))
    print("Done")

It would raise the exception below
Process Process-1:
Traceback (most recent call last):
  File "/home/zzli/anaconda3/envs/funcx/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap
    self.run()
  File "/home/zzli/anaconda3/envs/funcx/lib/python3.6/multiprocessing/process.py", line 93, in run
    self._target(*self._args, **self._kwargs)
  File "/home/zzli/anaconda3/envs/funcx/lib/python3.6/site-packages/parsl-0.8.0-py3.6.egg/parsl/executors/high_throughput/interchange.py", line 563, in starter
    ic.start()
  File "/home/zzli/anaconda3/envs/funcx/lib/python3.6/site-packages/parsl-0.8.0-py3.6.egg/parsl/executors/high_throughput/interchange.py", line 458, in start
    tasks_inflight = len(self._ready_manager_queue[manager]['tasks'])
KeyError: b'7c51d275c32f'

Took an initial scan of the code. It seems like this is because the manager is removed from the ready_manager_queue due to too many heartbeats missed in https://github.com/Parsl/parsl/blob/4cea89bdea96f44b74d5c6eb4030e6c6679a31f1/parsl/executors/high_throughput/interchange.py#L514
but is not removed from interesting_managers.

Is this a situation where managers are failing and the failing manager is being handled badly? Or do you think this should work if the pasted exception was fixed?

Is this a situation where managers are failing and the failing manager is being handled badly? Or do you think this should work if the pasted exception was fixed?

I think it should work if we properly fix the exception.
I can confirm that changing the line https://github.com/Parsl/parsl/blob/4cea89bdea96f44b74d5c6eb4030e6c6679a31f1/parsl/executors/high_throughput/interchange.py#L514
to self._ready_manager_queue[manager]['active'] = False works.
I am testing now with a newly added line interesting_managers.remove(manager) after https://github.com/Parsl/parsl/blob/4cea89bdea96f44b74d5c6eb4030e6c6679a31f1/parsl/executors/high_throughput/interchange.py#L514

Is this a situation where managers are failing and the failing manager is being handled badly? Or do you think this should work if the pasted exception was fixed?

The manager is killed by the strategy because there is no task for 2 minutes.


Measure bunch of pages

I think it can be useful to have possibility measure bunch of pages with running one command.
Right now we should do
pwmetrics http://example.com --runs=3
pwmetrics http://example.com/page1 --runs=3
pwmetrics http://example.com/page2 --runs=3

I think we should do smth either like
pwmetrics http://example.com http://example.com/page2 --runs=3
or use some config property
module.exports = {
  urls: ['http://example.com', 'http://example.com/page1'. 'http://example.com/page2']
}

i guess it depends on the usecase.
if you want to measure a BUNCH then you probably want to save the results as JSON or submit all to sheets?
dunno if you'd want to do a bunch of diff URLs and view the charts in the CLI.

Yeap, it's really just for json or sheets cases. We should postpone this a bit...
I'm just creating issues when smth pops out... 💡

Lets close it.


Compile for glibc 2.17

Great implementation and solved my problem locally! When trying to run on a Centos based cluster, I found that the library was compiled at minimum for GLIBC_2.18. Would it be feasible to compile against GLIBC_2.17?

Thank you for your report! I'll try to solve this problem.
Could you share the environment information?

I repro it in centos:7 docker image

@AndriyMulyar I've published a new version of pytokenizations which is able to be installed in old centos (#7) . Please try it!

Thank you!
On Mon, Mar 9, 2020, 5:49 AM Yohei Tamura<EMAIL_ADDRESS>wrote:

@AndriyMulyar https://github.com/AndriyMulyar I've published a new
version of pytokenizations which is able to be installed in old centos (#7
https://github.com/tamuhey/tokenizations/pull/7) . Please try it!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tamuhey/tokenizations/issues/6?email_source=notifications&email_token=ADJ4TBXSFWAOQF2UIPAJDADRGRYPBA5CNFSM4K2UFXF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOFUMII#issuecomment-596330017,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADJ4TBQLOAKPY7XK7N2VNUTRGRYPBANCNFSM4K2UFXFQ
.


[SPARK-50282][ML] Simplify TargetEncoderModel.transform

What changes were proposed in this pull request?
Simplify TargetEncoderModel.transform
Why are the changes needed?
existing implementation builds the lookup logic by a if-else approach of many branches, actually we can use try_element_at for this purpose
Does this PR introduce any user-facing change?
no
How was this patch tested?
ci
Was this patch authored or co-authored using generative AI tooling?
no

Merged to master for Apache Spark 4.0 on February 2025.
Thank you, @zhengruifeng .


Update setting-up-databricks.md

SQL endpoints are now called SQL Warehouses in Databricks. The image on line 98 is outdated and no longer relevant. Line 96 still says "SQL endpoint", not sure if this should also be changed or not.
Description & motivation

To-do before merge

Prerelease docs
If this change is related to functionality in a prerelease version of dbt (delete if not applicable):

[ ] I've added versioning components, as described in "Versioning Docs"
[ ] I've added a note to the prerelease version's Migration Guide

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects
[ ] Run link testing to update the links that point to the deleted page

commented out image code and deploy preview looks good. merging now!


Enabling XMRig as the active mining plugin.

Taking a shortcut from the full fallback PR to try and fix the present NiceHash issues.

Superseded by #258.


Invalid value for requires_dist. Error: Can't have direct dependency on nuls

Invalid value for requires_dist. Error: Can't have direct dependency: "nuls2-python @ git+https://github.com/odesenfans/nuls2-python.git@fda2ba4a5f9397f4f84cfee738942c4a15f88840 ; extra == 'nuls2'"

PyPI provided binary builds for eciespy 17 but not with Python 3.11


How to change the User-Agent for gRPC HealthCheck

Title:
Is there a way to change the User-Agent for gRPC HealthCheck?
Description:
The User-Agent for gRPC HealthCheck is "Envoy/HC". gRPC HealthCheck uses the following service_name option You can change the service in grpc.health.v1.HealthCheckRequest, but not the UserAgent.

https://www.envoyproxy.io/docs/envoy/v1.21.0/api-v3/config/core/v3/health_check.proto.html#config-core-v3-healthcheck-grpchealthcheck

HTTP Health Checks can be changed to any UserAgent using the request_headers_to_add option below.

https://www.envoyproxy.io/docs/envoy/v1.21.0/api-v3/config/core/v3/health_check.proto.html#config-core-v3-healthcheck-httphealthcheck

Is there any way to change the UserAgent for gRPC HealthCheck?

https://github.com/envoyproxy/envoy/pull/20071 does initial_metadata work here? CC @wanlill

I think so, we actually have a unit test which exercise this: https://github.com/envoyproxy/envoy/blob/main/test/common/upstream/health_checker_impl_test.cc#L4503


how do you call qtip without hover

I have a field like this
<input name="name" type=""text" id="name>

I want to call the tip inside a function like this:
function exe(){
1. show tip next to the right of input
2. add custom content "hello world" to that tip
3. this tip will only disappear on demand, not blur or hover out.
}

How would you config that?

@craigcosmo your image or code does not show please recheck.

thanks, I made change to the comment. I think you can see the issue now

$('.qtip qtip-default ').show(); like something

Hi raj, thanks. Does it open on demand? like I call a function and it show. then I call close() it close.

I figure out. Just add a custom event to the tip. Then trigger the host element with trigger('my_custom_e')
$('#space').qtip({
					// prerender: true,
					content: {
						text: i.space
				    },
				    position: {
				        my: 'left center',
				        at: 'right center'
				    },
				    style: {
				        classes: 'qtip-busy qtip-shadow'
				    },
				    show: 'my_custome_e',
				    hide: 'my_other_e'
				});

it opens on click on input #name and #pass so outside click it will hide qtip :smile:


*: Cut 0.15.1

Cherry-picking the following bug fixes:

[BUGFIX] Fix email template typo in alert-warning style (#1421)
[BUGFIX] Fix regression in Pager Duty config (#1455)
[BUGFIX] Catch templating errors in Wechat Notify (#1436)
[BUGFIX] Fail when no private address can be found for cluster (#1437)
[BUGFIX] Make sure we don't miss the first pushPull when joining cluster (#1456)
[BUGFIX] Fix concurrent read and wirte group error in dispatch (#1447)

Including version bump and CHANGELOG.md entry.
//CC @stuartnelson3 @simonpasquier @brancz

Mhh, DCO check is failing with Expected "Guido García<EMAIL_ADDRESS>but got "Guido García<EMAIL_ADDRESS>

I ran our Prometheus Operator test suit with this patch release. Results can be found here.
It is failing. I would prefer to investigate the failures tomorrow morning before continuing here. What do you think?

Update: The first Alertmanager instance in a Kubernetes Statefulset of 3 instances is unable to resolve the DNS entries of its peers via a Kubernetes Service at startup, and crash loops. I am further looking into it. I am not sure if this is related to this Alertmanager patch release or just our Prometheus Operator.

Got the issue. I will propose a PR in a little bit.

Fix: https://github.com/prometheus/alertmanager/pull/1465

@mxinden I think I signed the commit with a different email and DCO is complaining. Can I do something to help?

Agreed

I agree with using my commit here


Question about PIP, what libraries are included as standard with regards to Okta Device Trust

Okta is a tool for managing Identity and Access Management. You can think of it as a cloud based alternative to Microsoft Active Directory although not identical to that. Being that it does not require Microsoft services or servers it is popular in the Mac community so likely most people here will be familiar with it.
One of the things Okta can do is 'Device Trust' whereby only enrolled and trusted devices with their own client certificate are 'trusted' to have access to systems. Again this is similar to Microsoft AD which does this when you 'bind' to AD.
Okta implement this via a Python script and this requires that you -

Have a copy of Python3 installed
Have PIP installed
and in order to install PIP libraries also have Xcode Command Line Tools installed
and of course the Okta provided script(s)

My question relates to determining whether the pre-built Managed-Python3 provided here already includes the sole PIP library that Okta requires. To hopefully make this easier for people to answer I am including the PIP configuration script Okta provide below.
#!/bin/sh
echo "Running pip3 install --upgrade pip"
sudo pip3 install --upgrade pip
echo "Running pip3 install pyobjc-framework-SystemConfiguration"
sudo pip3 install pyobjc-framework-SystemConfiguration
exit

As you will see the sole PIP library installed is pyobjc-framework-SystemConfiguration
If this library is included as standard then I can skip this process completely and even avoid having to install Xcode Command Line Tools at all.
If it is not included currently as standard would there be any likelihood that it would be added to the standard list?
On a second related topic, if I do install Managed-Python3 first - from here, how would I modify the above script to ensure PIP adds this library to the Managed-Python3 copy and not to the copy included with Xcode Command Line Tools? (I know how to get a Python script to use a specific Python installation this question is about getting PIP to do the same.)

I am able to answer and solve this myself.
It turns out the following command ensures using PIP is targeted to the managed_python3 provided here.
/usr/local/bin/managed_python3 -m pip

You can also ensure a python script uses this copy of python by having the following as the shebang line
#!/usr/local/bin/managed_python3

The following command will list the packages that are included as standard in managed_python3
/usr/local/bin/managed_python3 -m pip list

This was therefore able to confirm that along with pyobjc (and many others) pyobjc-framework-SystemConfiguration is indeed included as standard in managed_python3
Therefore I can should I choose no both to install Xcode Command Line Tools and homebrew as everything required for Okta Device Trust is included.


Allow fix-permissions to fail without failing a build

If some files can't be updated because the current user doesn't own them or some other reason, the entire build should not fail. This will allow secrets to be injected inside the source tree.

Can one of the admins verify this patch?

@openshift/dev-ex ptal

[test]


Why you don't use Application Gateway instead of Azure LB?

Hi guys!
Any reason why you don't use Azure Application Gateway(AAG) instead of Azure Load Balancer (ALB)?
Or at least combination of both ALB & AAG, if you need L4.
AAG has support for SSL termination and other L7 securities.
Which makes life easier in Azure environment.

Hi @kamoljan, good speaking to you last week. Ops Manager will start supporting Azure Application Gateway with PCF 2.1. The pipelines can review adding this functionality then. Please let us know if any questions.

@abbyachau Thank you Abby!


Return the I2C instance when initialization fails

Otherwise the user loses their I2C instance forever. I had to avoid using the ? operator because otherwise Rust couldn't figure out the ownership correctly.

LGTM


How to Host this on server (base model ) what should I do . I am a begginer to hosting

If I want to host this to a server which type of server I should use , for CPU and GPU .
I researched a little for CPU i came across linode .
I have no experience hosting thinks like this , appreciate if anyone can answer .

If I want to host this to a server which type of server I should use , for CPU and GPU .
I researched a little for CPU i came across linode .
I have no experience hosting thinks like this , appreciate if anyone can answer .

Not the library maintainer but we currently have this deployed on AWS GPUs. Any machine will work honestly. I prefer DigitalOcean (https://www.digitalocean.com/) for my machines, but anything will work. For CPU, you'll probably want a few cores and 8/16GB of memory depending on how big your files are. You can check out the README for examples of memory usage.
Pricing here: https://www.digitalocean.com/pricing/droplets

Thank you for replying . Which aws instance are you using and how much does it cost for 1 month .

Thank you for replying . Which aws instance are you using and how much does it cost for 1 month .

The cheapest GPU option is g4dn.xlarge (there are cheaper ARM options but that is a pain to set up) and it costs roughly $400 a month to run. We don't have this setup to use CPUs. It would be cheaper but take much longer.

Hi! I am also trying to figure out how to host faster-whisper. Do you have any advice? Which service did you end up using?


fix updating apps

currently when I want to update an environment variable with a value containing some special characters e.g. !, it updates with escaped characters e.g. %21 so we need to just send request body

@jabrena
could you review this?


LHN - Chat that contains only task disappears from LHN

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.46-3
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Login with expensifail account
Start a new chat
Create a task on the newly created chat
Go to another chat

Expected Result:
Chat that contains only task does not disappear from from LHN
Actual Result:
Chat that contains only task disappears from LHN when navigating to other chats
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/23d08237-3d86-46c6-91cf-32048c5b7ead
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.

LHN - Chat that contains only task disappears from LHN

What is the root cause of that problem?

When creating a task, we don't set the notificationPreference to always:

https://github.com/Expensify/App/blob/8db63476c908f490b808fb987e9f063653af2614/src/libs/actions/Task.ts#L220-L233
like we did when creating a new comment:
https://github.com/Expensify/App/blob/8db63476c908f490b808fb987e9f063653af2614/src/libs/actions/Report.ts#L492-L498
so the chat disappears from LHN:
https://github.com/Expensify/App/blob/8db63476c908f490b808fb987e9f063653af2614/src/libs/SidebarUtils.ts#L141
What changes do you think we should make in order to solve the problem?
Add a logic to update notificationPreference to place:
    const shouldUpdateNotificationPrefernece = !isEmptyObject(parentReport) && ReportUtils.getReportNotificationPreference(parent) === CONST.REPORT.NOTIFICATION_PREFERENCE.HIDDEN;

    if (shouldUpdateNotificationPrefernece) {
        optimisticData.push({
            onyxMethod: Onyx.METHOD.MERGE,
            key: `${ONYXKEYS.COLLECTION.REPORT}${parentReportID}`,
            value: {
                participants: {
                    [currentUserAccountID]: {notificationPreference: CONST.REPORT.NOTIFICATION_PREFERENCE.ALWAYS},
                },
            },
        });
    }

What alternative solutions did you explore? (Optional)
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

Proposal
Please re-state the problem that we are trying to solve in this issue.

LHN - Chat that contains only task disappears from LHN

What is the root cause of that problem?

When we start a new chat, the notificaitonPreference is hidden. When creating a task, we don't set the notificationPreference to always:

https://github.com/Expensify/App/blob/8db63476c908f490b808fb987e9f063653af2614/src/libs/actions/Task.ts#L220-L233
like we did when creating a new comment:
https://github.com/Expensify/App/blob/8db63476c908f490b808fb987e9f063653af2614/src/libs/actions/Report.ts#L492-L498
so the chat disappears from LHN:
https://github.com/Expensify/App/blob/8db63476c908f490b808fb987e9f063653af2614/src/libs/SidebarUtils.ts#L141
until the API CreateTask is called succesfully.
What changes do you think we should make in order to solve the problem?
Add a logic to update notificationPreference to place:
    const shouldUpdateNotificationPrefernece = !isEmptyObject(parentReport) && ReportUtils.getReportNotificationPreference(parent) === CONST.REPORT.NOTIFICATION_PREFERENCE.HIDDEN;

    if (shouldUpdateNotificationPrefernece) {
        optimisticData.push({
            onyxMethod: Onyx.METHOD.MERGE,
            key: `${ONYXKEYS.COLLECTION.REPORT}${parentReportID}`,
            value: {
                participants: {
                    [currentUserAccountID]: {notificationPreference: CONST.REPORT.NOTIFICATION_PREFERENCE.ALWAYS},
                },
                notificationPreference: CONST.REPORT.NOTIFICATION_PREFERENCE.ALWAYS // Optional
            },
        });
    }

What alternative solutions did you explore? (Optional)
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

Thanks for the review @hoangzinh, assigning @mkzie2 🚀

Hi @mkzie2 - can you please accept this offer so I can process your payment - https://www.upwork.com/nx/wm/offer/104636304
@hoangzinh can you please complete the checklist
Thank you!

@VictoriaExpensify I accepted offer, thanks

Thanks @hoangzinh - raised request to add ^^ to testrail - https://github.com/Expensify/Expensify/issues/440023

Payment summary:
Contributor: @mkzie2 paid $250 via Upwork
Contributor+: @hoangzinh paid $250 via Upwork


feat: add @apify/docs-search-modal

Adds the new search modal with the result preview.

~The modal is definitely still wip - mostly small CSS things (dark mode is not dark), responsiveness / styling of the search "button" etc.~
Closes #655

The PR is still missing the search in versioned files and the dynamic import of the modal.
btw did the dynamic import improve anything (considering it all gets webpack-ed into one bundle either way?)

Video of the current state:

We don't have stagings for docs, right? :) I would like to take a peek if everything is okay design-wise. Do I need to run this locally?

Yes, there is no staging for docs I'm afraid. If you're in the office, I can run it from my server and share the address on Slack :)

I'm in the office and I like the idea.

The fact that the first element in the list not visually pointing to the preview window on the right is weird.

I suggest applying our color tokens: For the text highlight, you can apply the Bamboo color from Decorative palette

The breadcrumb in the preview has low line-height, resulting in certain letters being cut at the bottom. I suggest increasing the line-height.

Please replace the > with the respective icon. Both in the list and the preview.

This is how it looks in the Docs.

Let's keep this consistent.

The whole preview is not clickable, just the links in the preview.

All the mentioned issues have been taken care of, see the recording below. I'm also now injecting the "React Router" (client JS) navigation to the modal for faster load times and better UX.
Feel free to leave any further comments if you find anything in the video.
https://github.com/apify/apify-docs/assets/61918049/8da3f45c-56ca-466d-b56e-59a28a01f3c6

The new @apify/docs-search-modal version (git repo here) features fixed fonts, same document icons as the current docsearch modal and a keyboard tooltip in the footer.
@B4nan feel free to try out locally.

I have addressed all of the (rendering | logic) issues in<EMAIL_ADDRESS>You can pull the latest commit here, reinstall deps and try it out.
Regarding the last thing - I've updated the index so that it contains the name of the class (interface, enum...), so the results hopefully make more sense. I think it's kind of nifty to have the other results there too, especially if they are rendered in the hierarchy like this.

Following our conversation, hierarchy-only hits (where the result is considered relevant for the query only based on its parents) are now rendered only if the parent itself is one of the results. This is best shown on results from the reference pages - when searching for a class name, you will get the main document as well as the class's methods etc.
Otherwise, those hierarchy-only matches are omitted from the results.


Add examples for Notifications from async operations

Provides reproducable examples for the exceptions mentioned in #221 on the NotificationPage in the Gallery.
It also includes some questions regarding the recent changes towards the concept of CmdMsg, which I don't understand, seemingly replacing Cmd<'msg>, which I think I understand by now.

Thanks for this PR.  I will go through it during the week and make some notes where we can add more docs or samples.

@edgarfgp Thank you for taking the time to respond :) Your suggestion worked for one of the added scenarios examples. I have converted your feedback into hopefully helpful comments and updated my questions for you to have a look at when you get to it.


Horizontal with items starting on the left

I have been going crazy trying to adjust the slider so that the items start on the left side of the container. No mater what I try the items start centred. Here is a screenshot:
http://d.pr/i/F5lWoV
Here are the styles:
import { Dimensions } from 'react-native';

const { width: viewportWidth, height: viewportHeight } = Dimensions.get('window');

function wp(percentage) {
    const value = (percentage * viewportWidth) / 100;
    return Math.round(value);
}

const slideHeight = viewportHeight * 0.4;
const slideWidth = wp(36);
const itemHorizontalMargin = wp(2);

export const sliderWidth = viewportWidth;
export const itemWidth = slideWidth + itemHorizontalMargin * 2;

export default {
    container: {
        flex: 3,
        paddingLeft: 0,
        paddingBottom: 20,
    },
    slide: {
        flex: 1,
        height: slideHeight,
        width: itemWidth,
        justifyContent: 'center',
        paddingHorizontal: itemHorizontalMargin,
    },
    title: {
        fontSize: 13,
        marginBottom: 10,
        marginLeft: 15,
    },
    slideText: {
        color: 'white',
        fontSize: 13,
        alignSelf: 'center',
    },
};

I have tried aligning the items with containerCustomStyle and contentContainerCustomStyle but also no luck there.
Would really appreciate any guidance on this.

Hi @stueynet,
I think you'll be very pleased to know that the flatlist branch contains just what you need (check prop activeSlideAlignment).
Note that the updated example also features a left-aligned slider, as you can see below.

Thanks so much! When were you thinking of merging?

My idea is to implement a proper loop mode before merging the branch, but if it takes me too long to figure out how I might decide on merging pretty soon.
Anyway, you can start using it already by referencing branch's latest commit:
"react-native-snap-carousel": "https://github.com/archriss/react-native-snap-carousel#fbdb671"

Don't forget to check the updated documentation as well as the migration guide.

Ah that's easy I'll try that. Thanks again!

On Aug 4, 2017, at 3:59 AM, Benoît Delmaire<EMAIL_ADDRESS>wrote:
My idea is to implement a proper loop mode before merging the branch, but if it takes me too long to figure out how I might decide on merging pretty soon.
Anyway, you can start using it already by referencing branch's latest commit:
"react-native-snap-carousel": "https://github.com/archriss/react-native-snap-carousel#fbdb671"
Don't forget to check the updated documentation as well as the migration guide.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@stueynet , @bd-arc can you please help out here, having the same issue and links aren't working to the guides

@hjhimanshu01 carousel component contains the activeSlideAlignment property.
Try to set that to left and you might get what you want :)
<Carousel activeSlideAlignment='left' ... />

@Naturalclar , displays invalid prop type error, probably a bug?

<Carousel activeSlideAlignment='center' ... />


fix: change highlight only on zoom and mouse press

References and relevant issues
Closes: #7074
Description
Instead of updating the highlight on every draw, just do it when the mouse moves or press.

I found it weird that we change highlight on zoom? Is that common in other pieces of software? It happens on main too so I don't think it should block this PR but I was surprised.

This event should be a private event as it is created for vispy to inform when repaint the highlight on image.
It is because highlight width is in screen units, not data, so after zoom changed its width needs to be updated.
Also, we highlight an object when the mouse is over it during zoom-in/out. I do not remember why by @psobolewskiPhD should remember.


Add option to make o and O insert bullets and behave more like the enhanced "enter"

Fixes https://github.com/vslinko/obsidian-outliner/issues/460
Demo:
https://github.com/vslinko/obsidian-outliner/assets/903365/333e49a1-8e9d-452c-bc3b-409dbc059c9a
I went ahead and added an option to toggle whether this should be enabled, but I suspect if you're using the vim keymap you will want to have this enabled so I can remove the setting and enable it for all if you'd prefer to keep the settings simple.

I would love this feature 🙌

I cannot replicate this issue. I am using version 4.8.1.

@DivineMK  Are you saying you aren't able to reproduce https://github.com/vslinko/obsidian-outliner/issues/460 or that you're having an issue with my patch?

I can't reproduce #460.

Can we move the conversation to that issue so the conversation here can focus on whether this is the best way to fix the issue? It's a behavior that's been seen by many users, so I'm sure we can figure out what's missing from your reproduction so you can see the bug for yourself.

I agree.


Fix initial commit page & binary munching problem

This PR fixes two issues:

Missing diff following binary files

There was a missing check to break out of the current file loop in ParsePatch
Fix #13248

404 Errors on viewing diff for parentless commits

Unfortunately as a result of properly fixing ParsePatch the hack that
used git show <initial_commit_id> to get the diff for this failed.
This PR fixes this using the "super-secret" empty tree ref to make the
diff against.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Codecov Report

Merging #13249 into master will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #13249      +/-   ##
==========================================
+ Coverage   42.21%   42.22%   +0.01%
==========================================
  Files         683      683
  Lines       75458    75463       +5
==========================================
+ Hits        31851    31866      +15
+ Misses      38382    38375       -7
+ Partials     5225     5222       -3

Impacted Files
Coverage Δ

services/gitdiff/gitdiff.go
67.77% <0.00%> (-0.48%)
:arrow_down:

modules/indexer/stats/db.go
52.17% <0.00%> (-17.40%)
:arrow_down:

modules/git/utils.go
73.77% <0.00%> (-3.28%)
:arrow_down:

modules/git/repo.go
46.19% <0.00%> (-0.51%)
:arrow_down:

services/pull/pull.go
41.27% <0.00%> (+0.49%)
:arrow_up:

modules/queue/unique_queue_disk_channel.go
55.38% <0.00%> (+1.53%)
:arrow_up:

services/pull/patch.go
55.68% <0.00%> (+1.70%)
:arrow_up:

services/pull/check.go
52.55% <0.00%> (+2.91%)
:arrow_up:

services/pull/temp_repo.go
29.78% <0.00%> (+3.19%)
:arrow_up:

modules/charset/charset.go
73.03% <0.00%> (+4.49%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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

sorry about that ^, will not reference PRs in this manner anymore, unless necessary ofc 🚀

please send backports :)