Status on store level is not filtered, only admin store

SELECT `e`.*, `link_table`.`parent_id`, `at_status`.`value` AS `status`, GROUP_CONCAT(parent_id SEPARATOR ',') AS parent_ids FROM `catalog_product_entity` AS `e`
    INNER JOIN `catalog_product_super_link` AS `link_table`
        ON link_table.product_id = e.entity_id
    INNER JOIN `catalog_product_entity_int` AS `at_status`
        ON (`at_status`.`entity_id` = `e`.`entity_id`) AND (`at_status`.`attribute_id` = '84') AND (`at_status`.`store_id` = 0)
    INNER JOIN `catalog_product_website` AS `product_website`
        ON product_website.product_id = e.entity_id AND product_website.website_id IN('3')
    WHERE (at_status.value = '1') AND (link_table.parent_id IN('71866'))
    GROUP BY `e`.`entity_id`

@Schrank Is this still current? If so, can you please add some more infomation? If it no longer is current, can you please close the issue?


Make landmark feature configurable (eg via env variables) (for different cities)

First landmark implementation: https://github.com/priobike/priobike-poi-service/commit/ce5434d7d2789d9a78bb7e0adfb81282dc550cbf


Create create fible1.toml

Add validator "fible1" to testnet 15.
Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[ ] Only one toml is added in this PR
[ ] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present
[ ] The email, discord, elements telegram, and twitter fields are present and valid

Whitespace found in filename. Found filename namada-public-testnet-15/create fible1.toml


unable to load images from  local files in nativescript angular

hear is the error log.
JS: Error in reading bitmap - java.io.FileNotFoundException: /data/data/org.nativescript.testingOne/files/app/images/chair.png (No such file or directory)
hear is the repo issue will recreate from the start itself
https://github.com/tahir-jamil/new-translator
hear is my list
`
<ActionItem (tap)="favourite()" backgroundColor="transparent" ios.position="right">

				<GridLayout rows="auto" columns="*,*" col="2" borderRadius="5">
					<GridLayout (tap)="saveData()" columns="auto,auto" horizontalAlignment="right">

					</GridLayout>
				</GridLayout>

				<GridLayout row="1" colSpan="3" class="image-size">
					<Image row="2" stretch="aspectFill" [src]="'~/images/' + item.img" ></Image>
				</GridLayout>

			</GridLayout>
		</ng-template>
	</RadListView>
</GridLayout>

data = [
{ english:"book", chinese: "chinese", img: "chair.png"},
{ english:"book", chinese: "chinese", img: "chair.png"},
{ english:"book", chinese: "chinese", img: "chair.png"},
{ english:"book", chinese: "chinese", img: "chair.png"},
{ english:"book", chinese: "chinese", img: "chair.png"},
{ english:"book", chinese: "chinese", img: "chair.png"},
{ english:"book", chinese: "chinese", img: "chair.png"},
{ english:"book", chinese: "chinese", img: "chair.png"}
]
`
and this is the data i am showing
this is the result i get

Hi @tahir-jamil,
The issue in the project is related to wrong image path. ~/ will points to the src folder. In this case, you should change the path from [src]="'~/images/' + item.img" to [src]="'~/app/images/' + item.img"

I had the same problem. Normally it wouldn't be a problem to figure out which directory is meant by "~" if there wasn't the confusing error message:

/data/data/org.nativescript.testingOne/files/app/images/chair.png (No such file or directory)

@tsonevn Thank you for the tip. Now I can even reference the assets folder ;-)

Thank You, Thank You, Thank You ..... :-)

Thank you....


add external  libbluray support

Hello @bugparty ,
Please retarget the PR to the development branch.

please review again, thanks

@bugparty So the Github Virtual Environments were updated to use Android NDK r24 as the 'latest' and the ffmpeg-android-maker isn't fully compatible with it. Will try to fix it first. Funny, in a month r25 is scheduled for the release.

@bugparty Please pull the latest Javernaut:development branch to your bugparty:add_libbluray.

ok

Thanks @bugparty for contribution


feat: introducing Image component

Hi! With the introduction of the Artwork Component I believe the use-cases here are covered, so I'm going to go ahead and close this. If you feel we're missing a feature please open an issue and we can try and tackle it. Thanks!


Release v0.80.9, via point-release

Tooling Release

In order to ship some minor improvements and bug fixes, let's prepare a v0.80.9 release, flushing out the current contents of the main branch.
Changes to include

[ ] Everything on current main
[ ] pindexer performance improvements in https://github.com/penumbra-zone/penumbra/pull/4913
[ ] pindexer bug fix for https://github.com/penumbra-zone/penumbra/issues/4916 which also landed in #4913
[ ] UIP-6, app version safeguard https://github.com/penumbra-zone/penumbra/pull/4919

Compatibility
As this is a point-release, all changes must be fully compatible for all nodes and clients.
Careful attention should be given to the delta between most recent tag on the main branch:
https://github.com/penumbra-zone/penumbra/compare/v0.80.8..main

Delaying release slightly due to a tooling error: running cargo release 0.80.9 --execute yields a commit message that doesn't interpolate the version string as usual:
chore: release version {{version}}

Worked fine for 0.80.8, so I'm not sure what changed. Will debug and follow up to unblock.

Quickly isolated the changes in #4930 as causing the problem: if I revert just the single commit from #4930, then the tool works correctly, and the version number is interpolated. Took longer to figure out just why that was, though. There are some discrepancies in the parameter-setup crate's Cargo.toml when compared against other crates, mostly omitted metadata like authors, homepage, license, and repository. Setting those doesn't resolve the problem, but what did resolve the problem was setting version = { workspace = true }.

Shipped: https://github.com/penumbra-zone/penumbra/releases/tag/v0.80.9


Work Mode set but not applied - ES 4.6kw

Hi, although the HA goodwe.set_work_mode service sets the work mode and you can actually see it as set in PV Master it does not take effect. When you reapply in PV Master it does take effect.

I also have this issue.
I can see the code is here to do it.
What I notice in the last part, the hex code is capitals where the set_ongrid_battery_dod is small letters in the hex and set_ongrid_battery_dod works.
But GIJOE mentions he can see it is set in pv master but not applied...I wonder.

I believe the work mode switching on ES/EM inverters was finally addressed, please verify.

Can confirm this now works for me in 0.9.7
After setting the mode I can see the inverter reboot, including the click sounds it make and then after the new mode as was set, is in effect.

Thank you gents, will only be upgrading by the end of next week when I am back but i am sure if it is working for David it will work on my side.

Hi, I decided to to upgrade today remotely and luckily everything worked fine. I can confirm that the mode setting is working thank you very much.
Unfortunately the new way of doing this without the services broke some of my automations. This were particularly helpful when Eskom load shedding picked that there were will be power outages and I used the service to set the discharge depth to 0 (Without restating the Inverter). This forced it basically to use Eskom power and any access solar power to charge batteries through automations with no human intervention.
The new way uses number.depth_of_discharge_on_grid value which I can set but it does not take effect, it even shows on the slide bar that it changes but has no effect until you use the slide bar itself.
Is there another way I can use the new method to achieve this? If not, is it possible to bring the services back?
Thanks for the help

@GIJOE-SA
You can still use it in automations. I manage my DoD multiple times a day and it still works as previously.
Instead of a service, just update the number.
This works for me:
      - alias: "Set DoD on inverter"
        service: number.set_value
        data:
          value: >
            {{ dod }}
        target:
          entity_id: number.depth_of_discharge_on_grid

My complete script looks (will work for automations too) look like this:
script:
  ######################################################################
  # Re-usable script to set the DoD via input
  ######################################################################
  # Example:
  ######################################################################
  # - service: script.goodwe_set_dod
  #   data:
  #     forecast: "50"
  ######################################################################
  goodwe_set_dod:
    sequence:
      - alias: "Set variables"
        variables:
          dod: >
            {% set forecast_int = forecast|int %}
            {% if forecast_int >= 28 %}70
            {% elif forecast_int >= 23 and forecast_int < 28 %}65
            {% elif forecast_int >= 20 and forecast_int < 23 %}60
            {% elif forecast_int >= 18 and forecast_int < 20 %}55
            {% elif forecast_int >= 16 and forecast_int < 18 %}50
            {% elif forecast_int >= 12 and forecast_int < 16 %}40
            {% elif forecast_int >= 10 and forecast_int < 12 %}30
            {% else %}16{% endif -%}
      - alias: "Set DoD on inverter"
        service: number.set_value
        data:
          value: >
            {{ dod }}
        target:
          entity_id: number.depth_of_discharge_on_grid
      - alias: "Log action taken"
        service: script.log
        data:
          icon: ':zap:'
          object: 'GoodWe'
          message: >
            Setting DoD to {{ dod }} based on {{ forecast }}kWh forecast

I know it works, because it takes affect as long as you are back in general mode.

@GIJOE-SA
You can still use it in automations. I manage my DoD multiple times a day and it still works as previously.
Instead of a service, just update the number.
This works for me:
      - alias: "Set DoD on inverter"
        service: number.set_value
        data:
          value: >
            {{ dod }}
        target:
          entity_id: number.depth_of_discharge_on_grid

My complete script looks (will work for automations too) look like this:
script:
  ######################################################################
  # Re-usable script to set the DoD via input
  ######################################################################
  # Example:
  ######################################################################
  # - service: script.goodwe_set_dod
  #   data:
  #     forecast: "50"
  ######################################################################
  goodwe_set_dod:
    sequence:
      - alias: "Set variables"
        variables:
          dod: >
            {% set forecast_int = forecast|int %}
            {% if forecast_int >= 28 %}70
            {% elif forecast_int >= 23 and forecast_int < 28 %}65
            {% elif forecast_int >= 20 and forecast_int < 23 %}60
            {% elif forecast_int >= 18 and forecast_int < 20 %}55
            {% elif forecast_int >= 16 and forecast_int < 18 %}50
            {% elif forecast_int >= 12 and forecast_int < 16 %}40
            {% elif forecast_int >= 10 and forecast_int < 12 %}30
            {% else %}16{% endif -%}
      - alias: "Set DoD on inverter"
        service: number.set_value
        data:
          value: >
            {{ dod }}
        target:
          entity_id: number.depth_of_discharge_on_grid
      - alias: "Log action taken"
        service: script.log
        data:
          icon: ':zap:'
          object: 'GoodWe'
          message: >
            Setting DoD to {{ dod }} based on {{ forecast }}kWh forecast

From this do I understand correctly if you set the DoD above the current DoD the invertor will start charging the battery until it reaches that DoD?
What do you use to determine the forecast value? Is it based on sol cast? Or do you have a way to include weather forecast etc as well?

@Jayksopp

From this do I understand correctly if you set the DoD above the current DoD the invertor will start charging the battery until it reaches that DoD?
No. If you set the DoD to lets say 60%, it means, when there is no sunlight and you are in general mode, that the inverter will use 60% of the battery, eg. it will go down to 40% charge left, before it switches the load over to grid.

What do you use to determine the forecast value? Is it based on sol cast? Or do you have a way to include weather forecast etc as well?
I do use solcast. You can see my solcast config here --> https://github.com/tinuva/home-assistant-config/blob/master/packages/energy/solcast_rest.yaml
My DoD automation are here --> https://github.com/tinuva/home-assistant-config/blob/master/packages/energy/goodwe_dod.yaml

I see I still need to remove the old input I used.

@tinuva

I know it works, because it takes affect as long as you are back in general mode.

Thank you very much David. The service: number.set_value is indeed working. I have tried yesterday setting the value using developers tools without using the number.set_value which obviously did not work.
Very nice script you wrote there.

@Jayksopp

From this do I understand correctly if you set the DoD above the current DoD the invertor will start charging the battery until it reaches that DoD?

I think I understood the question wrong first time. If you set DoD above the current DoD, inverter will not charge it, but will only use the battery if it goes even higher than the last DoD set which is now higher.

I struggled a week with the problem related to setting the DoD value, almost going crazy. After finding this thread, I checked again by setting the value of number.depth_of_discharge_on_grid entity. I found out, that decreasing the value works always, but you can increase the value only up to 89. If you set any higher value (90 -99) it is ignored by the inverter, however it looks just fine in the HA.


Fix bugs

In the current state, there are few bugs while loading incompatible files or adding tags. Let's fix them.

Codecov Report
Base: 50.26% // Head: 49.82% // Decreases project coverage by -0.43% :warning:

Coverage data is based on head (2d9db5b) compared to base (d160055).
Patch coverage: 26.08% of modified lines in pull request are covered.

:exclamation: Current head 2d9db5b differs from pull request most recent head d48f257. Consider uploading reports for the commit d48f257 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #145      +/-   ##
============================================
- Coverage     50.26%   49.82%   -0.44%
  Complexity      614      614
============================================
  Files           125      125
  Lines          2857     2870      +13
  Branches        318      317       -1
============================================
- Hits           1436     1430       -6
- Misses         1300     1316      +16
- Partials        121      124       +3

Impacted Files
Coverage Δ

src/main/java/seedu/clinkedin/MainApp.java
0.00% <0.00%> (ø)

.../seedu/clinkedin/logic/commands/AddTagCommand.java
28.12% <0.00%> (ø)

src/main/java/seedu/clinkedin/ui/ExportWindow.java
0.00% <0.00%> (ø)

...ava/seedu/clinkedin/storage/JsonAdaptedPerson.java
83.69% <50.00%> (-8.26%)
:arrow_down:

...eedu/clinkedin/storage/JsonAddressBookStorage.java
85.71% <0.00%> (-14.29%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add borrowing _withUnprotectedUnsafePointer

For withUnsafePointer we have both an inout and a borrowing form, allowing it to be used with immutable values. Add a parallel form for the unprotected variant.
Resolves: rdar://135889076

@swift-ci test

@swift-ci smoke test windows

@swift-ci test windows


Add a clean manifest.json

I'll add below a clean manifest.json file that you can try to include in the index.html page.
The header of the HTML page should contain
    <link rel="manifest" href="manifest.json">

{
  "short_name": "BetterOCaml",
  "name": "BetterOCaml: offline OCaml IDE",
  "icons": [
    {
      "src": "icon/android-chrome-192x192.png",
      "type": "image/png",
      "sizes": "192x192"
    },
    {
      "src": "icon/android-chrome-512x512.png",
      "type": "image/png",
      "sizes": "512x512"
    }
  ],
  "start_url": "/index.html",
  "background_color": "#263238",
  "display": "standalone",
  "scope": "/index.html",
  "theme_color": "#263238",
  "description": "A small but efficient, intuitive and responsive OCaml IDE right in your browser! Ships OCaml v4.08.1, interpreter by your browser (so it works offline!), compiled with js_of_ocaml. Open-source on GitHub.com/jbdo99/BetterOCaml",
  "screenshots": [
    {
      "src": "screenshots/screenshot1.png",
      "type": "image/png",
      "sizes": "549x943"
    }
  ]
}

Demo on my local Chromium:

:ok_hand: :iphone: It worked the other day on my Firefox on mobile, and i didn't change anything so it should work on mobiles as well!

Sorry my demo is a little bit fuzzy, but it shows that I can close and reopen the Progressive Web App, find it in my system's list of apps, etc.

This app https://app-manifest.firebaseapp.com/ could help in the future, if you have to write more app manifest!

This can be closed, @jbdo99 included this in main branch.


Not working with SQLite?

I'm trying to use this library in a project where I was previously using Doctrine's DBAL with an SQLite database. When I attempt to write to the database I get:
SQLSTATE[HY000]: General error: 8 attempt to write a readonly database

I was able to read from and write to it just fine previously, so I did some digging.
When creating my own Doctrine Connection, I must use the following code (with an actual path):
DriverManager::getConnection([
'path' => /path/to/database,
'driver' => 'pdo_sqlite'
]);
...while the SQL repository creates it via:
DriverManager::getConnection(['pdo' => $pdo])
It seems SQLite requires it to be done a bit differently.
Perhaps it would be best to allow the user to inject their own DBAL connection?

Never mind, getting the same error now whether I use my own DBAL connection or not.
I still would like to be able to inject an instance of Connection into SqlRepository instead of PDO, for when I get this working. :sweat_smile:


Error. Creative tab crashed while getting items [MC1.12.2]

Minecraft: 1.12.2
Forge: all (last <IP_ADDRESS>37)
Cyclic 1.14.6
JEI <IP_ADDRESS>
Lines: 13996 (JEI-Cyclic)
Debug Log - https://drive.google.com/open?id=17dCrv1l_taOd9Js8wWa6RZbId6PPamuF

Fix on the way, see the other linked issue for release details and other info


Fix bug with not display points close to given layer

Description
Partially solve a problem described in #2288 which comes from float == comparison
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

References

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

I think this implementation is fine for now. (eventually we'll want to do something more clever like cKDTree, but that's an existing problem.) @Czaki could you add a test with the failure that motivated this PR?

@Czaki could you add a test with the failure that motivated this PR?

I will do this tomorrow

One strong disclaimer. n current napari implementation the slider step is calculated basing only on scale parameters. This causes that event with this PR point with z coordinate 10.1 will not be displayed on any layer in 2D view.
I see two solutions:

Use np.round
Modify Points to calculate the real step base also on data coordinates. This may produce thousands of steps in slider.


fetchFromGitHub: fix compatibility issue with nix-prefetch

Description of changes
https://github.com/NixOS/nixpkgs/pull/279934#issuecomment-1916531889
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 284985 run on x86_64-linux 1

Result of nixpkgs-review pr 284985 run on x86_64-linux 1

@GaetanLepage have you tested this with the invidious script?

@GaetanLepage have you tested this with the invidious script?

Yes, it works fine :)

Cool :)


Any example with working remark and rehype.

I tested this plugin with using remark and rehype but i can't seem to make that work.
Is there any example for this that you can point me to?
I tried evaluating in the pre script but that doesn't seem to be working for me properly.
Thank you.

I didn't know this used the same method as it was in rest of them.
My mistake.


run pytest against nightly

[x] Closes #218
[ ] Tests added: Please use assert_type() to assert the type of any return value

Not sure whether it will succeed if pytest fails.

Might be good to file an issue with pyarrow to handle pandas nightly

ImportError: pyarrow requires pandas 0.23.0 or above, pandas 0+unknown is installed

Might be good to file an issue with pyarrow to handle pandas nightly

ImportError: pyarrow requires pandas 0.23.0 or above, pandas 0+unknown is installed

I guess only an issue on older pythons.....

Looking at this, I don't see where you install the nightly build, but maybe I'm missing something or you're still working on that part??

Looking at this, I don't see where you install the nightly build, but maybe I'm missing something or you're still working on that part??

This install (and uninstalls nightly):
https://github.com/pandas-dev/pandas-stubs/blob/cc6ea9b7d65d6c1e0d039742d2a7f399396c2bae/scripts/test/__init__.py#L52
The issue is that I don't yet know how to make github ignore errors.

Add "--upgrade","--use-deprecated=legacy-resolver" to the pip command that installs the nightly.

Looking at this, I don't see where you install the nightly build, but maybe I'm missing something or you're still working on that part??

This install (and uninstalls nightly):
https://github.com/pandas-dev/pandas-stubs/blob/cc6ea9b7d65d6c1e0d039742d2a7f399396c2bae/scripts/test/__init__.py#L52

I think it was here, which I didn't remember was in the code:
https://github.com/pandas-dev/pandas-stubs/blob/80f561fc73f387f81f3876eeef4fe3bae82dae18/scripts/test/run.py#L83

Add "--upgrade","--use-deprecated=legacy-resolver" to the pip command that installs the nightly.

Thank you! I simplify the install/uninstall. Why is --use-deprecated=legacy-resolver needed?

It has to do with how pip check versions. Legacy takes the latest. Without this it will download every whl file to get Metadata before deciding on the best fit.

I'm surprised we get these warnings:

/home/runner/work/pandas-stubs/pandas-stubs/tests/test_frame.py:290: FutureWarning: The 'inplace' keyword in DataFrame.set_index is deprecated and will be removed in a future version. Use df = df.set_index(..., copy=False) instead.
res6: None = df.set_index("col1", inplace=True)

I thought that change is already reverted on main

Reopen to trigger CI

So while we don't exit on failure if the test against nightly builds fails, if there was a failure, would it show up red in the CI report?

It will always be "green". As far as I know there is no "yellow".

There might be a way to mark the CI run as skipped if it fails. Then we would have a visual indicator.

There might be a way to mark the CI run as skipped if it fails. Then we would have a visual indicator.

Without any visual indicator, I'm not sure how we would ever notice that something was revealed via this test.  Can you investigate that?

I didn't find a way to visually indicate an error except by failing (failing now).
I also made pytest error on any warnings. This fails currently on nightly but #251 might fix that

So I like what the PR is doing, but it is raising some questions in how we want to manage the project.  Opinions wanted!

If we see a failure in the job due to the nightly build, should we expect users to fix the code before accepting the PR?
If the answer to (1) is yes, then it's easy to manage.
If the answer to (2) is no, then that brings up a couple of questions (4) and (5)
Should we create a separate job that only tests the nightly build, and then accept PR's where all the other tests pass?
Should we manage the project by having a "main_release" branch, corresponding to what we release weekly as we improve the stubs for the released version of pandas, and a "nightly" branch, corresponding to what is developed to work for a future version of pandas?  (Would have to decide which one of those is main)

Could also make pytest-nightly run in its own workflow, then it is clear that ubuntu+3.10 did not fail for any other reason.

1. If we see a failure in the job due to the nightly build, should we expect users to fix the code before accepting the PR?

If pandas nightly causes all new PRs to have errors - I would not expect people to fix that as part of their PR. But if people want to add new annotations that trigger future/deprecation warnings, I would be inclined to request changes :)

4. Should we create a separate job that only tests the nightly build, and then accept PR's where all the other tests pass?

In either case, it probably makes sense to have a separate workflow just for nightly.

5. Should we manage the project by having a "main_release" branch, corresponding to what we release weekly as we improve the stubs for the released version of pandas, and a "nightly" branch, corresponding to what is developed to work for a future version of pandas?  (Would have to decide which one of those is `main`)

I really like the current simplicity of pandas-stubs. If there are cases where we do not (yet) want to be in-line with nightly, we could simply have a pytest skip decorator based on the pandas version.

In either case, it probably makes sense to have a separate workflow just for nightly.

So let's do that, and we'll operate on the principle that if the nightly fails, and everything else passes, then we'll accept the PR.

I really like the current simplicity of pandas-stubs.

Me too!!!

If there are cases where we do not (yet) want to be in-line with nightly, we could simply have a pytest skip decorator based on the pandas version.

Good idea

I will mark this PR as draft and then debug sharing steps between the general checks and pytest nightly.

Might be worth waiting for #251 to keep the nice green check mark :)

Can you make test/nightly orange for failure rather than red?

Can you make test/nightly orange for failure rather than red?

I did not find an option for that.

statsmodels has it on their pip pre run if pytest fails. If you merge this I'll see if I can figure out what is needed in a later PR.

I think there is a way to mark a run as "allowed to fail".  At least I remember seeing it somewhere.
Hopefully @bashtage can figure it out!

If it is not possible to make it warn-only but not fail, an option could be to run it only on main (after PRs are merged) or the reverse (only for PRs but not on main): then we have some more green :)

If it is not possible to make it warn-only but not fail, an option could be to run it only on main (after PRs are merged) or the reverse (only for PRs but not on main): then we have some more green :)

Looks pretty hopeless for now.  Seems GHA doesn't have a yellow/warn mode despite the similarities to Azure (the I suspect that it runs on Azure).

I would always pass the run and have it there in CI so that interested parties could easily see what doing on.
The alternative would be to add 1 Azure run which could be configured to be yellow.  If you want the Azure run, I can set it up.

Is there a way to customize the failing/successful after n seconds message:

We could then let it always succeed but at least change the message to failed :)


Add username to /etc/passwd inside of container if --userns keep-id

If I enter a continer with --userns keep-id, my UID will be present
inside of the container, but most likely my user will not be defined.
This patch will take information about the user and stick it into the
container.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

Looks like a bug in the tests. I'll check out the PR and have a look.

/lgtm

Shouldn't it also have created the group? :)


[BUG] Timeline

Description
There are several issues we have to investigate for beta testing-user.
Acceptance Criteria

[ ] Investigate why there is a huge gap between the AutoComplete View and the Timer Bar (https://toggl.slack.com/archives/CH7DV7VSR/p1569480096001900)
[ ] Bug in Editing: The Editor in Timeline always lost focus (https://toggl.slack.com/archives/CH7DV7VSR/p1569480257002200)
[ ] Why Record Activity doesn't show up in some scenario (https://toggl.slack.com/archives/CH7DV7VSR/p1569480069001800)

All done at https://github.com/toggl/toggldesktop/pull/3333#issuecomment-535855913


AAP-20333: Adding Ansible as a Managed Service to the navigation bar

Jira: https://issues.redhat.com/browse/AAP-20333
Not to be added to production yet.
Configures the current list of URL structures for stable/beta.
Currently a draft.
Please feel free to leave comments/questions, especially if something seems strange below

Maybe a better way to do this would be to check the status of the flux kustomize component because they have a status API that will only be healthy if all contained components are healthy according to their published health checks


Added Rust page

Fixes Issue
This PR fixes the following issues:
#example

Changes proposed
Here comes all the changes proposed through this PR

Check List (Check all the boxes which are applicable)

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots and video
Add all the screenshots and a video which support your changes

hi @piyushw0203 actually we do not want an individual rust card on the main page of resources. it would be better if you could create a card in the blockchain page for rust and then connect the rust page to it. and take care of the file locations appropriately.

Alright.


switch to Chainguard's netcat image

Description

CG's netcat is multi-arch (amd64 and arm64) so we can remove nodeSelector
https://github.com/sigstore/helm-charts/blob/main/charts/trillian/values.yaml#L138
https://github.com/chainguard-images/images/tree/main/images/netcat

I'm not very deep into this area yet, but I'd be happy to collaborate on a PR with you. What do you think?

the nc image from Chainguard is lacking sh
we use sh and nc in initContainer
https://github.com/sigstore/helm-charts/blob/55e320ec47fea672bc0ad2472c73f56f4c1f0684/charts/trillian/templates/trillian-log-server/deployment.yaml#L43
so I think we need to open a PR to Chainguard's images repo first https://github.com/chainguard-images/images/blob/main/images/netcat/configs/latest.apko.yaml


Release v0.1.7

If the fatal issue is not reported on v0.1.7-alpha, we will release it on October 29.
refs: #16 #18 #21 #25 #28

done


pytest-watch with debugger

Quite often I invoke pytest with the --ipdb option so I could see what is wrong with my code on first failure.
So I invkoed pytest-watch with ptw -- --ipdb. The problem is that if after droping into ipdb I modify a test or a file it starts a new session and the one with ipdb console goes to the background. In the end I end up with dozens of ipdb instances running.
Is it possible to block ptw if current session has dropped to the ipdb?

That's an interesting case. I think that'd be a good thing to handle.
Any idea how pytest works with --ipdb? Namely, does the pytest process remain running until the debug session is over? (I would think it does.) If so, it might be worth adding an --ipdb argument that gets forwarded to pytest and also signals to ptw to wait for the process to finish.
What do you think, @Glueon?

Yee, pytest keeps running when it is in the ipdb.
I do not know if a forwarded --ipdb option is a good solution, because py.test can be dropped into  debugger from code with something like import ipdb; ipdb.set_trace().
But I think you can just block ptw in all cases untill pytest finishes. Then --ipdb forwarding will be redundant.

Yes, pytest keeps running when it is in the ipdb

Excellent.

because py.test can be dropped into debugger from code with something like import ipdb; ipdb.set_trace()

Ah, very true. Good point.

But I think you can just block ptw in all cases untill pytest finishes

I thought about that initially before replying. The downside is then you sacrifice the feature of cancelling a long-running testsuite when you save a new file. It's a nice feature of ptw so you don't have to wait for the tests to finish before running them again with the latest code.
Another approach would be to add a --wait / -w argument to tell ptw to wait for cases like this. (I'm sure --ipbd won't be the only one.)

I think -w approach is better. If someone relies on cancellation fucntionallity you won't break anthing introducing it.

It may also make sense to suspend any pytest re-runs until the currently running pytest process has finished (although the events can still be collected / spooled). Maybe just make that a default option to avoid yet another command line arg.

It may also make sense to suspend any pytest re-runs until the currently running pytest process has finished

@aldanor I personall work on some bigger projects with long-running test suites. The cancelling behavior is really nice in these cases since it nearly doubles the time running tests, which can be significant. But you're right, it'd be best to not introduce an option if we can avoid it. I wonder if there's a simple cross-platform way to detect whether py.test is running vs ipdb / something else. That would make running tests cancellable, but wait on child / spawned processes.
If there's no other option, I think I'd still prefer cancel-by-default over a new CLI argument. Have any other ideas?

Question for you since I don't use ipdb. If signal.SIGTERM is sent to a Python process in a debug session, will it terminate or keep running until you explicitly exit with a command?

If signal.SIGTERM is sent to a Python process in a debug session, will it terminate or keep running until you explicitly exit with a command?

For reference, with --pdb, it terminates your debugging session after you enter a command.

Fixed with 18a99b8c8bb7451ee27a7fa9a32a538e2faac8e2, which adds --wait. (This was enabled by 0cf6b93966320847600c9b50699f00adeb63d307, which ensures only one instance of pytest is running, and 6e07f80097d677a7fe85ddd884f41364bc2233dd, which gracefully terminates a test run with KeyboardInterrupt which --wait overrides.)
Just published v4.0, which includes these changes. Feel free to re-open if there's still an issue!


[backport] Simplify ROCm install guide

Backport of #4048

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 6323abcacee72ab2dfdd84a315a39ee9a54c09eb, target branch v8) succeeded!

Thanks, LGTM!


20538 Update FastDDSGen tests for indirect hashes

This PR includes updates to the tests for indirect hash TypeIdentifiers. Now, maps consider the key when determining equiv_kind.
It updates the union tests to accommodate the member_id change. See more details at https://github.com/eProsima/IDL-Parser/pull/123.

https://github.com/eProsima/Fast-DDS-Gen/actions/runs/8092202026

Green CI: https://github.com/eProsima/Fast-DDS-Gen/actions/runs/8140925484


Addition of splide js to show the reviews in slideshow and change of color of text in "frequently asked question"

Showing the reviews with slideshow.

Questions are not visible due to the white color . Want to change the color of text under "frequently asked questions"

i want to work on this issues
@Vimall03 assign me this work with the labels gssoc-ext, hacktoberfest-accepted and level according to the issue.

Hi @Shirisha-16, There was a recent UI update that was made to the landing page. Please update your fork and check the site again. Let me know if you would like to make any changes to the reviews section of the new landing page.

@Vimall03 is there any problem with index.html page . I am unable to see that page

@Shirisha-16 In the recent UI update, the index.html file is changed to index.php
You will need a xampp / wampp server to run the project.

@Shirisha-16 please let me know if you are still interested in contributing


OCPBUGS-28597: Disable PO

Temporarily disable this component while upgrading rukpak with a breaking change

re-title commit in the form "UPSTREAM: :" ?

/retest

/retest

/label qe-approved

/remove-label qe-approved

wait @jianzhangbjz to add qe-approved

Seeing if a comment will kick tide

/approve


fix lz4_flex version

Codecov Report

Merging #1209 (cf92be3) into main (72cef12) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1209      +/-   ##
==========================================
- Coverage   94.07%   94.04%   -0.03%
==========================================
  Files         205      205
  Lines       34514    34514
==========================================
- Hits        32469    32460       -9
- Misses       2045     2054       +9

Impacted Files
Coverage Δ

src/directory/file_watcher.rs
96.69% <0.00%> (-3.31%)
:arrow_down:

src/indexer/segment_updater.rs
93.94% <0.00%> (-1.04%)
:arrow_down:

src/directory/watch_event_router.rs
95.41% <0.00%> (-0.77%)
:arrow_down:

src/postings/stacker/expull.rs
93.44% <0.00%> (-0.44%)
:arrow_down:

src/query/union.rs
99.39% <0.00%> (+0.60%)
:arrow_up:

src/fastfield/reader.rs
94.91% <0.00%> (+0.84%)
:arrow_up:

src/fastfield/multivalued/mod.rs
96.96% <0.00%> (+3.03%)
:arrow_up:

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 72cef12...cf92be3. Read the comment docs.


Steps and information not clear (or up-to-date) in article

Hi there,
I have some feedback on the article that hopefully helps in making it more clear:

Key Vault access configuration is still shown with an old screenshot that shows "Access policies" where nowadays this is "Access configuration" [EDIT: I see it now, the setting under Settings is called Access configuration, where the Access policies option is under the overall-options. Might be practical to review to one of these explicitly]
[EDIT2: I noticed the Key Vault creation says "create a Key Vault", might be good to change to "create a Key Vault that uses Access policies, not role-based access control"]
I suggest adding an overview of how things work in configuration in more detail, so something like:

You will create a Key Vault and allow access for the administrator-account that you will be setting up Verified ID with
You will configure Verified ID in the portal-page (cannot be done from the ENTRA-portal as I found the Setup-blade is not available there)
In the creation, the proper permissions will be configured on the Key Vault to be able to write secrets by the Verified ID back-end services
You will create an App Registration that will be... [fill in here, not sure to describe]
You will configure permissions for the created App Registration to be able write secrets to the Key Vault

Describe that, if you want to use this configuration to be able to hand out Verified ID's, you will a kind of a "front-end" application (web app or something), and refer to the page that describes this (like a reference to the example app on GitHub). The article describes the domain/URL, but nothing on how to "use" that
For the Key Vault permissions this is described "For Key permissions, verify that the following permissions are selected: Create, Delete, and Sign. By default, Create and Delete are already enabled. Sign should be the only key permission you need to update.". That is not the case, to my knowledge, nothing is selected by default. Additionally, your account also needs List permissions, which are not described (although the screenshot shows all are checked).

Hope it helps, and please note I am adding the feedback from a constructive point of view. A lot is described well, but for me it did not give me a complete picture.
Best regards, Jop

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

ID: e85be67f-8375-6229-18aa-d687463991f9
Version Independent ID: c494773d-9fcc-d733-6ddb-935a8e545abd
Content: Tutorial - Issue Microsoft Entra Verified ID credentials from an application - Microsoft Entra
Content Source: articles/active-directory/verifiable-credentials/verifiable-credentials-configure-issuer.md
Service: decentralized-identity
Sub-service: verifiable-credentials
GitHub Login: @barclayn
Microsoft Alias: barclayn

The sample app has been updated to ngrok, so the Key Vault example no longer applied. No change is needed now.
#please-close


Unable to set value on Object

Describe the bug
When trying to set a value on a variable of type Object an exception is thrown. E.g.:
java.lang.IllegalArgumentException: The value is not of the same type as the property.
parent node type : com.example.bug.ObjectTest node name: object, node type: java.lang.Object, value type: java.lang.String

This issue was introduced on 1.0.9
Your environment

1.0.9
Java 17

Steps to reproduce
https://github.com/esfomeado/fixture-monkey-bug/blob/master/src/test/java/com/example/bug/FixtureMonkeyTest.java#L123
Expected behaviour
I should be able to set a string in a object type like in previous versions.
Actual behaviour
It's not possible to set a string on a object type.

@esfomeado
Hello, thank you for reporting an issue.
It is a bug, It'll be fixed in 1.0.10!
I'll let you know when it is published.

@esfomeado
1.0.10 is released.
Please check it out!

Fixed thanks


feat(#136): Handle AASTORE Bytecode Instruction Correctly

Handle AASTORE instruction correctly. Now we handle stack correctly.
Closes: #136.

History:

feat(#136): use array references
feat(#136): remove the puzzle for 136 issue
feat(#136): fix all qulice suggestions
feat(#136): add one more puzzle
feat(#136): fix compilation errors

PR-Codex overview
Detailed summary

Added a new method object() to the Reference class that returns the object.
Added a new instruction DUP to the DecompilerMachineTest class.
Modified the variable() method in the LocalVariables class to fix a typo in the return type.
Added a new comment in the DecompilerMachine class to refactor and simplify the code.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@rultor merge

@rultor merge

@rultor status

@rultor status

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 3min)

@rultor merge

@volodya-lombrozo The pull request is closed already, so I can't merge it

@rultor merge

@volodya-lombrozo I'm sorry, I don't understand you :( Check this page and try again please

@rultor merge

@volodya-lombrozo The pull request is closed already, so I can't merge it


Symfonisk speakers are nor discovered by AirPlay

What version of Music Assistant has the issue?
2.3.3
What version of the Home Assistant Integration have you got installed?
2024.12.4
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
I’m using Music Assistant with a combination of HomePods (AirPlay) and Symfonisk (Sonos) speakers (also AirPlay-capable). The issue is that Music Assistant does not detect the Sonos AirPlay speakers, which means I cannot combine Symfonisk and HomePods in a multi-room group.
I expected both sets of AirPlay speakers (HomePod and Symfonisk) to be recognized and available for grouping within Music Assistant, but currently only the HomePods are visible.
How to reproduce
Attempt to discover and group all AirPlay speakers using Music Assistant.
The Sonos (Symfonisk) AirPlay speakers are not detected and cannot be grouped with other AirPlay speakers.
Music Providers
Not relevant
Player Providers
AirPlay
Full log output
No response
Additional information
I'm able discover and use Symfonisk speakers with Sonos provider. But than they cann't be grouped with other airplay speakers.
What version of Home Assistant Core are your running
2024.12.4
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
ODROID

Attach the full log as required by the issue template.

@OzGav here. Garage is Apple TV.
2024-12-20 10:59:05.813 INFO (MainThread) [music_assistant] Starting Music Assistant Server (e13e95b931b74d469a00eb1c0ed89ac5) version 2.3.3 - HA add-on: True - Safe mode: False
2024-12-20 10:59:05.828 INFO (MainThread) [music_assistant.cache] Initializing cache controller...
2024-12-20 10:59:06.283 INFO (MainThread) [music_assistant.music] Using a sync interval of 180 minutes.
2024-12-20 10:59:06.804 INFO (MainThread) [music_assistant.streams] Detected ffmpeg version 6.1.1 with libsoxr support
2024-12-20 10:59:06.804 INFO (MainThread) [music_assistant.streams] Starting server on  <IP_ADDRESS>:8097 - base url: http://<IP_ADDRESS>:8097
2024-12-20 10:59:06.817 INFO (MainThread) [music_assistant.webserver] Starting server on  <IP_ADDRESS>:8095 - base url: http://<IP_ADDRESS>:8095
2024-12-20 10:59:07.431 INFO (MainThread) [music_assistant] Loaded metadata provider fanart.tv
2024-12-20 10:59:07.432 INFO (MainThread) [music_assistant] Loaded music provider Music Assistant
2024-12-20 10:59:07.433 INFO (MainThread) [music_assistant] Loaded player provider Playergroup
2024-12-20 10:59:07.434 INFO (MainThread) [music_assistant] Loaded metadata provider The Audio DB
2024-12-20 10:59:07.435 INFO (MainThread) [music_assistant] Loaded metadata provider MusicBrainz
2024-12-20 10:59:07.639 INFO (MainThread) [music_assistant.music] Sync task for Music Assistant completed
2024-12-20 11:44:12.974 INFO (MainThread) [music_assistant] Loaded player provider Airplay
2024-12-20 11:44:13.000 INFO (MainThread) [music_assistant.players] Player registered: ap1aac55018808/bathroom
2024-12-20 11:44:13.038 INFO (MainThread) [music_assistant.players] Player registered: apa64633a36bfd/office
2024-12-20 11:44:13.051 INFO (MainThread) [music_assistant.players] Player registered: apb275f38a9b8e/martinâ€™s macbook pro
2024-12-20 11:44:13.077 INFO (MainThread) [music_assistant.players] Player registered: ape2d3616fbde8/edit's room
2024-12-20 11:44:13.086 INFO (MainThread) [music_assistant.players] Player registered: ap0e03af0e124e/bedroom
2024-12-20 11:44:13.093 INFO (MainThread) [music_assistant.players] Player registered: apf60dfef835be/sitting room
2024-12-20 11:44:13.099 INFO (MainThread) [music_assistant.players] Player registered: apaa08f586967b/sauna
2024-12-20 11:44:13.172 INFO (MainThread) [music_assistant.airplay] Ignoring garage in discovery because it is not yet supported.
2024-12-20 11:56:13.484 INFO (MainThread) [music_assistant.spotify] Successfully logged in to Spotify as martin.dybal
2024-12-20 11:56:13.484 INFO (MainThread) [music_assistant] Loaded music provider Spotify
2024-12-20 11:59:23.840 INFO (MainThread) [music_assistant.music] Sync task for Spotify: martin.dybal completed
2024-12-20 12:14:30.948 INFO (MainThread) [music_assistant.players] Playback announcement to player Pracovna HomePod (with pre-announce: True): http://<IP_ADDRESS>:8123/api/tts_proxy/QDpSUxycdE6tJY1EMmTncw.mp3
2024-12-20 12:15:07.869 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue Pracovna HomePod - crossfade: False
2024-12-20 12:15:14.005 INFO (MainThread) [music_assistant.players] Playback announcement to player Pracovna HomePod (with pre-announce: True): http://<IP_ADDRESS>:8123/api/tts_proxy/QDpSUxycdE6tJY1EMmTncw.mp3
2024-12-20 12:15:22.170 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue Pracovna HomePod - crossfade: False
2024-12-20 12:17:39.373 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue Pracovna HomePod - crossfade: False
2024-12-20 12:33:54.514 INFO (MainThread) [music_assistant.airplay] Ignoring Garage in discovery because it is not yet supported.
2024-12-20 12:33:54.517 INFO (MainThread) [music_assistant.airplay] Ignoring Garage in discovery because it is not yet supported.
2024-12-20 12:34:04.323 INFO (MainThread) [music_assistant.airplay] Ignoring Garage in discovery because it is not yet supported.
2024-12-20 12:34:06.577 INFO (MainThread) [music_assistant.airplay] Ignoring Garage in discovery because it is not yet supported.
2024-12-20 12:34:14.857 INFO (MainThread) [music_assistant.airplay] Ignoring Garage in discovery because it is not yet supported.
2024-12-20 12:34:26.010 INFO (MainThread) [music_assistant.airplay] Ignoring Garage in discovery because it is not yet supported.
2024-12-20 12:47:57.503 WARNING (MainThread) [music_assistant.webserver] Disconnected: Received non-Text message.
2024-12-20 12:57:13.125 INFO (MainThread) [music_assistant.streams] Finished Queue Flow stream for Queue Pracovna HomePod
2024-12-20 12:57:27.980 INFO (MainThread) [music_assistant.player_queues] End of queue reached, clearing items
2024-12-20 13:23:01.048 INFO (MainThread) [music_assistant.players] Playback announcement to player Pracovna HomePod (with pre-announce: True): http://<IP_ADDRESS>:8123/api/tts_proxy/QDpSUxycdE6tJY1EMmTncw.mp3
2024-12-20 13:23:53.594 INFO (MainThread) [music_assistant.players] Playback announcement to player Pracovna HomePod (with pre-announce: True): http://<IP_ADDRESS>:8123/api/tts_proxy/QDpSUxycdE6tJY1EMmTncw.mp3
2024-12-20 13:24:10.758 INFO (MainThread) [music_assistant.players] Playback announcement to player Pracovna HomePod (with pre-announce: True): http://<IP_ADDRESS>:8123/api/tts_proxy/QDpSUxycdE6tJY1EMmTncw.mp3
2024-12-20 13:24:23.600 INFO (MainThread) [music_assistant.players] Playback announcement to player Pracovna HomePod (with pre-announce: True): http://<IP_ADDRESS>:8123/api/tts_proxy/PGf8HW1hswrZ6fjsb5ajWg.mp3
2024-12-20 13:24:52.011 INFO (MainThread) [music_assistant.players] Playback announcement to player Pracovna HomePod (with pre-announce: True): http://<IP_ADDRESS>:8123/api/tts_proxy/n-qQkfwjANA70KLaFMJtzw.mp3
2024-12-20 13:35:37.859 INFO (MainThread) [music_assistant.players] Playback announcement to player Pracovna HomePod (with pre-announce: True): http://<IP_ADDRESS>:8123/api/tts_proxy/_dQ6uoYkw3OazBChBGiThQ.mp3
2024-12-20 13:40:03.856 INFO (MainThread) [music_assistant.players] Playback announcement to player Pracovna HomePod (with pre-announce: True): http://<IP_ADDRESS>:8123/api/tts_proxy/_dQ6uoYkw3OazBChBGiThQ.mp3
2024-12-20 13:41:05.064 WARNING (MainThread) [music_assistant.streams] Received unhandled GET request to /sonos_queue/v2.3/version from <IP_ADDRESS>
headers: <CIMultiDictProxy('Connection': 'close', 'Accept': '*/*', 'Accept-Encoding': 'gzip', 'Host': '<IP_ADDRESS>:8097', 'User-Agent': 'Linux UPnP/1.0 Sonos/82.2-59204 (ZPS33)', 'X-Sonos-Playback-Id': 'RINCON_542A1B52090801400:2093436828', 'X-Sonos-Corr-Id': '9743609b-005e-4058-a37b-db1b4d5e3536', 'X-Sonos-SWGen': '2')>

2024-12-20 13:42:02.580 INFO (MainThread) [music_assistant.players] Playback announcement to player Pracovna HomePod (with pre-announce: True): http://<IP_ADDRESS>:8123/api/tts_proxy/NSjaK2pOTjfGgrMLOandpw.mp3
2024-12-20 13:51:06.638 WARNING (MainThread) [music_assistant.webserver] Disconnected: Received non-Text message.

Please don’t paste log entries. Additionally this is a duplicate please review and follow https://github.com/music-assistant/hass-music-assistant/issues/3170


Either left or right joycon stops working and disconnects after a minute of opening joycon-driver.exe

I'm so excited to use your driver, please answer if you can >_<
To give some background, both of the JoyCons work fine by themselves, but the moment I open joycon-driver.exe, it all works fine, and after 30 - 60 seconds either the left or the right joycon just stops responding and usually disconnects it from the bluetooth as well.
This is what i'm getting from the CMD.

vJoy device 1 is free
Acquired device number 1 - OK
vJoy device 2 is free
Acquired device number 2 - OK
Found joycon L 0: UnKnownFound joycon R 1: UnKnownEnabling vibration...
Enabling IMU data...
Increase data rate for Bluetooth...
Pairing1?...
Pairing2?...
Pairing3?...
Successfully initialized Joy-Con (L) !
Enabling vibration...
Enabling IMU data...
Increase data rate for Bluetooth...
Pairing1?...
Pairing2?...
Pairing3?...
Successfully initialized Joy-Con (R) !
setting LEDs...
vibrating JoyCon(s).
Done.
0.0000
0.0617
0.0000
0.0000
0.0000
Then i'm just endlessly bombarded with 0.0000 messages.
Do you think this could be a problem with my bluetooth? I don't really think so because of what I mentioned earlier.. I was playing with Dolphin really nicely, and when I turned on joycon-driver.exe it was way more fun obviously, but one JoyCon just randomly stops working..
More info, when I connect my JoyCons to the Bluetooth, they start flashing up and down like they should. They keep on doing this for pretty much forever, which is definitely normal. As soon as I open joycon-driver.exe the lights stop, by that I mean its still lit but it isn't moving up and down. Only 1 cube is lit.  When the error occurs and one of the JoyCons stop working, the lights either start going up and down again or just turn off completely. Then if I check my Bluetooth, that joycon is disconnected and I have to connect it all over again.

I also have this problem. Same issue, same situation. Just to note sometimes when one disconnects I have to do the pairing all over again.

Exactly, already said that lol. Glad to see i'm not the only one xD

Same for me

Anybody has work around for this yet? Now I got workaround for everything only except this one which make us can't stably play the game.

@jabujabu I don't know why this happens for some people, my guess is bluetooth connectivity issues, it's hard to debug since it doesn't happen to me;
If I were you I'd probably get an external bluetooth adapter (if you are using an internal one like on a laptop) or if you are already using an external adapter maybe get a different one, this is the one that I use: https://www.amazon.com/Plugable-Bluetooth-Adapter-Raspberry-Compatible/dp/B009ZIILLI/ref=sr_1_1/146-0027820-3965552?ie=UTF8&qid=1504102371&sr=8-1&keywords=plugable+bluetooth+adapter
please let me know if changing the adapter fixes your issue / list your current adapter

thanks for the answer and it worked!! no more disconnect. My model is "NEXT-104BT". I had used intergrated bluetooth in my window tablet before. But there is still input delay on master build even with new bluetooth dongle.(holleyjean's build is OK on both bluetooth). at least I have all solution for almost issues, Thanks!!!


Bug: quick-test npm task fails

See stacktrace:

pack "after each" hook:
Error: Command failed: /bin/sh -c cd /Users/pauljensen/Work/socketstream/socketstream/test/fixtures && git checkout project/client/static/assets
fatal: Not a git repository: '.git'
at ChildProcess.exithandler (child_process.js:197:12)
at emitTwo (events.js:87:13)
at ChildProcess.emit (events.js:172:7)
at maybeClose (internal/child_process.js:764:16)
at Socket. (internal/child_process.js:319:11)
at emitOne (events.js:77:13)
at Socket.emit (events.js:169:7)
at Pipe._onclose (net.js:469:12)

For me it passes, but when run in a commit hook in Tower it fails due to colors being output. Some test needs a logHook to catch the build dump

ah you mean if you run it as a download, not a git repo?

removed the git checkout


Begin Encounter doesn't take into account with Eidolons

So I have Pf2e Perception as well as Pf2e Avoid Notice. Because Eidolons are excluded from initiative when combat begins and the tokens are revealed, the eidolon can still see everything. Is there any way to implement that or a workaround?

I'll have a look at it - I was able to get familiars to behave properly, so in theory I should be able to make these work as well...


i18n Translation Editor drops content

When I iterate with http://codecombat.com/i18n (over days) I can add translations (in danish) to the game, but when I reload the same translations later they are missing bits and pieces.
At first I was pretty sure it was a problem with multi-line entries, but now I am also missing a 1-liner.
Example:
For language danish I have added all javascript entries in http://codecombat.com/i18n/component/usessnippets .
Currently when I load the game I can see my danish translation of the Loop snippet: .
However, in Translation Editor the entry for 'loop description (javascript)' is an empty line:

.

Is it still sitting in the Pending Patches list for that Component?

I don't think I know where the Pending Patches list is :(

My translations are in the public game right now, but the Translation Editor can't load them, so I am pretty concerned whether they will be removed at next submit from the Translation Editor.

I found this Pending Patches list: https://codecombat.com/admin/pending-patches
Currently component UsesSnippet is only on that list for another language.
If I look at other languages like french and spanish none of them have translations for entries 'if/else description' and 'loop description', so I gather it might just be this component that has a bug.

I reworked the I18N diplomat submission system to be more transparent and reliable. It should be deployed soon; you'll know you've got it if there's a list of patches on the right side of the screen.


multiple allocations for lookahead intra costs

It says here that lookahead intra costs are allocated once per frame.
https://github.com/xiph/rav1e/blob/master/src/encoder.rs#L737
But the only place they are set allocates.
https://github.com/xiph/rav1e/blob/master/src/api/internal.rs#L714
https://github.com/xiph/rav1e/blob/master/src/api/lookahead.rs#L29

It's small, but it's something. PR incoming.


PR: Run files without saving them

Description of Changes
Add save_new_files option to the save() function to allow running files that are not saved.
A known limitation is that the code doesn't allow to set breakpoints in non-existing files, but this can be address in a later PR.  Works with https://github.com/spyder-ide/spyder-kernels/pull/143

[x] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #1643
(Only for python3)
(because of https://github.com/spyder-ide/spyder-kernels/pull/143)
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

By the way, the test failure on Linux and Python 2 look legit (the same test fails every time). So please take a look at that.

By the way, the test failure on Linux and Python 2 look legit (the same test fails every time). So please take a look at that.

I think this is fixed in qtconsole#375

Maybe not. The problem seems to be similar though: In [3]: !import os; os.getcwd()\n   ...:  somehow qtbot.keyClick(control, Qt.Key_Enter) doesn't execute the line but adds a continuation prompt. (this should have been fixed in jupyter/qtconsole#375). Could someone with linux have a look at this? An easy fix would be to add a shift modifier, but this wouldn't solve the underlying issue.

TODO: add tests

The tests are added in #10428

Hello @impact27! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file runtests.py:

Line 35:80: E501 line too long (82 > 79 characters)

@impact27, please update this PR and also post a screenshot in the description of how things looks while debugging an unsaved file.

@impact27, please update this PR and also post a screenshot in the description of how things looks while debugging an unsaved file.

I am not sure what you want me to update. Also this PR doesn't add anything to the GUI. It just avoids a pop-up when you run a file.


HDRI image preview is glitched in Editor

Godot version
4.0 beta 3
System information
Windows 10, Vulkan, Radeon RT 6800XT
Issue description
Preview of "Panorama" of PanoramaSkyMaterial is glitched.

Steps to reproduce

Create a HDRI image (I'm using blender).
Create PanoramaSkyMaterial.
Set Panorama with the HDRI image.

Minimal reproduction project
Glitched.zip

Related to https://github.com/godotengine/godot/issues/62095.


Make sure not to show Error on production environment

I got the error display on production environment.
A Database Error Occurred

Error Number: HY000/1
no such table: project
SELECT * FROM "project" ORDER BY "score" DESC, "url" ASC LIMIT 100
Filename: models/ProjectRepository.php
Line Number: 70

This is controlled by configuration.

@narfbg What do you mean? How do I configure not to show error?

For a database error? By setting 'db_debug' to FALSE.

@narfbg Thank you!
User Guide:

Conversely, setting the constant to ‘production’ will disable all error output.

The description is not a bit correct.  production and db_debug to FALSE will disable all error output. Right?

Well, it's not incorrect, but rather 'db_debug' is overriding this ...
I see how it can be tricky, I'll think about the best way to solve that.


Fix margin between nodes on palette

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
I fixed #2946.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage increased (+0.03%) to 67.41% when pulling 372122037fa0ff514d05152c27d60d96110223f5 on kazuhitoyokoi:master into 23a5cb1917fa6f7fd80e8cae8d04c753c76eecbb on node-red:master.

@knolleary Thank you!


Exception Checking for Invalid Token/Query

At some point, it's not worth it, as github's servers will tell us if the token or query is bad, and give better diagnostic info than we probably could.
Maybe checking to make sure they aren't nil or "", but nothing beyond that.

On the other hand, it does offer better performance to do it client side instead of on the server.


Shouldn't Resolver#resolve return the format of the source image as well?

@no-reply Resolver#resolve is supposed to return the local path to an image, and its format (jp2, jpg, tif, etc.), right?

Nevermind, dupe(ish) of #17.


Custom icon rendering to support SVG icons

Hi,
i was looking into a solution to support SVG icons along with the font icons used by Radzen.
I made a rapid prototype that uses an interface for icons that offers one method to return a RenderFragment for the icon.
CustomIconRendererPrototypeDiff
The idea is to replace all the string Icon properties with IRadzenIcon Icon.
Instead of having the hardcoded <i> element, the interface implementation provides the RenderFragment to render.
My goal was to make it code compatible by implementing an implicit cast operator from string to IRadzenIcon, but that failed because the razor files do not recognize the icon names as string anymore once the property type is not string.
This is why for the prototype I added the IconX property to the RadzenIcon. If it is set it will be rendered instead of the <i> element.
I made an implementation for the FontIcon. The MaterialIcons class offers all known standard icons (just a subset in the prototype). This class would best be generated with a source generator. Using this class instead of just the string for the icon name has the advantage to offer auto completion in the editor.
I also added a SvgIcon implementation that loads the SVG content from an embedded resource and makes some adjustments to it for sizing and tinting.
This solution supports sizing and coloring of the icon just like the font icons. Using the <img> element and load the file from wwwroot would not allow coloring and probably would have sizing issues as well.
The only downside I can see here is that if the string Icon in all components is replaced with IRadzenIcon Icon, all existing code out there would break. My solution in the prototype with the additional property would make it a non-breaking change but might be a bit confusing to use. It also requires more changes to all components with icons.
This prototype is by no means something to merge into the Radzen codebase. It is just a prototype/prove of concept.
If you say you would like to have this in Radzen, and maybe have some more ideas as well, I would happily continue development to a releasable state.

Hi @osre77,
I like the idea in general but I am not sure why this is a feature of RadzenIcon. To be honest it doesn't use anything from it and seems to just complicate things.

I just used RadzenIcon for my prototype. The idea is to use IRadzenIcon everywhere where icons are currently used, like RadzenButton, ... If it were just for standalone SVG icons a SvgIcon component would be the easier solution.
But if you want custom icons on all the other components you currently must not use the icon and put it in the content if the component supports templated content.

How would this be used in RadzenButton for example? Wouldn't it require a new property? The diff doesn't show such usage at the moment.

If implemented as a braking change then the RadzenButton.Icon property would be of type IRadzenIcon.
And you would use it like that:
<RadzenButton<EMAIL_ADDRESS>... />
or
<RadzenButton<EMAIL_ADDRESS>...>

where MaterialIcons.Close and MySvgIcons.SomeAwsomeIcon are IRadzenIcon implementations.
but the old
<RadzenButton Icon="close" ... />

Would cause a compilation error.
I tried to solve that win an implicit cast, but razor files don't know that "close" is a sting if the property is not a string.
As a non-breaking change a new property like IRadzenIcon IconXwould be added to RadzenButton:
<RadzenButton<EMAIL_ADDRESS>... />
or
<RadzenButton<EMAIL_ADDRESS>...>

Using both Icon and IconX would favor one over the other.

I know that all components with icons would have to be touched: I would offer to do that, either as a breaking change or with the alternative property for the icon.
And I'm also aware that the SVG icon could simply be put into the content.
But I would prefer if the icons would just work as is with the property but is not locked to font icons.
But if you don't want that big change (especially the breaking one) I can understand that.
This is why I wanted to discuss this before I put in the effort.

Honestly I think this is too much of an effort and quite a big of a change. The workaround is simple enough - to use the content of the component. So let's not do that but use the child content instead.

OK, thank you for your input.
I'll scrap my fork then.


Cv prefit

Description

Allow the cv parameter to take the value "prefit"

Type of change

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

How Has This Been Tested?
Linting, typing, unit test, doc generation
Checklist:

[x] I have read the contributing guidelines
[x] I have updated the HISTORY.rst and AUTHORS.rst files
[x] Linting passes successfully : flake8 . --exclude=doc
[x] Typing passes successfully : mypy mapie examples --strict --config-file mypy.ini
[x] Unit tests pass successfully : pytest -vs --doctest-modules mapie
[x] Coverage is 100% : pytest -vs --doctest-modules --cov-branch --cov=mapie --pyargs mapie
[x] Documentation builds successfully : cd doc; make clean; make html

Hello @gmartinonQM! Thanks for opening this PR.

In the file examples/plot_toy_model.py:

Line 35:112: W504 line break after binary operator

In the file mapie/estimators.py:

Line 472:30: E241 multiple spaces after ','

In the file mapie/tests/test_estimators.py:

Line 168:27: E241 multiple spaces after ','
Line 238:27: E241 multiple spaces after ','
Line 321:27: E241 multiple spaces after ','

Do see the Hitchhiker's guide to code style


[DO NOT MERGE] Convert user manual to be generated by Asciidoctor only

This allows a single page HTML and PDF user manual to be
generated via asciidoctor, and includes what's necessary to
convert the user manual totally to asciidoctor syntax.
Supersedes #5837
Companion #6021
This cannot be merged to master without other PRs that actually convert the docbook stuff to asciidoc included. The intent is to review this automation separately.
Here's what one can to test now, though:
./gradlew :docs:userguide then open subprojects/docs/build/userguidePdf/userguide.pdf. This should show you the asciidoctor infrastructure working, however, the samples will be missing. Same for userguide_single.html.
Context

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

I hope this isn't a big additional burden, but @wolfs' comment made me decide to remove all the other now-unused docbook bits. I have added commit e13b37f to this PR for this.

@eriwen How are the samples tested now?

@pledbrook Samples are tested using Exemplar which we'll release publicly not long after the Kotlin DSL work is done.

Merged with 135ac7bd3cfe91b57d2a2e4ed759e2af29b6ea45


CMake build fails with: Checksum invalid: asm-nseel-x64-sse.asm

I suppose maybe I am supposed to regenerate the SHA or something?
Unsure why this would happen on first build, as I haven't touched anything though.
Compiled with MinGW GCC 11 on Windows 11
====================[ Build | eel2 | RelWithDebInfo-MinGW ]=====================
CLion\ch-0\213.5744.123\bin\cmake\win\bin\cmake.exe --build Projects\tmp\ysfx\cmake-build-relwithdebinfo-mingw --target eel2
[0/2] CLion\ch-0\213.5744.123\bin\cmake\win\bin\cmake.exe -P Projects\tmp\ysfx\cmake-build-relwithdebinfo-mingw\CMakeFiles\VerifyGlobs.cmake
[1/12] cmd.exe /C "cd /D Projects\tmp\ysfx\cmake-build-relwithdebinfo-mingw && CLion\ch-0\213.5744.123\bin\cmake\win\bin\cmake.exe -DSOURCE_FILE=Projects/tmp/ysfx/thirdparty/WDL/source/WDL/eel2/asm-nseel-x64-sse.asm -DCHECKSUM_FILE=Projects/tmp/ysfx/sources/eel2-gas/sources/ref-hash-sha512.txt -P Projects/tmp/ysfx/cmake/ValidateSHA512.cmake && CLion\ch-0\213.5744.123\bin\cmake\win\bin\cmake.exe -E touch Projects/tmp/ysfx/cmake-build-relwithdebinfo-mingw/eel2-gas-validate-stamp.txt"
FAILED: eel2-gas-validate-stamp.txt Projects/tmp/ysfx/cmake-build-relwithdebinfo-mingw/eel2-gas-validate-stamp.txt
cmd.exe /C "cd /D Projects\tmp\ysfx\cmake-build-relwithdebinfo-mingw && CLion\ch-0\213.5744.123\bin\cmake\win\bin\cmake.exe -DSOURCE_FILE=Projects/tmp/ysfx/thirdparty/WDL/source/WDL/eel2/asm-nseel-x64-sse.asm -DCHECKSUM_FILE=Projects/tmp/ysfx/sources/eel2-gas/sources/ref-hash-sha512.txt -P Projects/tmp/ysfx/cmake/ValidateSHA512.cmake && CLion\ch-0\213.5744.123\bin\cmake\win\bin\cmake.exe -E touch Projects/tmp/ysfx/cmake-build-relwithdebinfo-mingw/eel2-gas-validate-stamp.txt"
CMake Error at Projects/tmp/ysfx/cmake/ValidateSHA512.cmake:31 (message):
  Checksum invalid: asm-nseel-x64-sse.asm
  ('aa5896ad4860cba62ec3450b55221f4985db4f79d83fc14dadd2f2aceae87cdfd8645f35160bf47bb7700046e3ecdd892aa74639130b910598ddb70515787ed8'
  !=
  'd4a4d8bf4d906f6a6cdbe54bc8a2699e01a11e1109a716c696fc3df96046422a71aea29ff643db4b23cf2b29a3ccd8dbb309a14bab327f13d7f2be461463bc02')

ninja: build stopped: subcommand failed.

This is a cmake bug.. it doesn't compute the checksum right in some windows environments.
I'd guess it's because it reads the files in text mode instead of binary.
I've not checked yet if cmake has this problem reported upstream.
For now, pass cmake the option -DYSFX_SKIP_CHECKSUM=ON

That's what I get for using Windows. Terrible OS, should just re-clone the project to WSL2.
Thanks for the swift reply! 🙏

Issue made at cmake bug tracker
https://gitlab.kitware.com/cmake/cmake/-/issues/22936

It turns out the problem is Git's line ending conversions.
Problem solved


Select doesn't work when inside a Collapsible

Hi there,
I'm with a problem similar to issue 105 (https://github.com/sherweb/ng2-materialize/issues/105)
When I put Input elements inside a Collapsible, the Input is not rendered at all.
Select elements are rendered, but the functionality is gone (the options don't pop up).
Running a few tests I found out that even normal HTML elements with (click) do not trigger events from inside the Collapsible. Is this an expected behaviour?

Hi, it's a duplicate with #116 . They work on this issue.

I'll reopen this issue since it's related to elements inside the Collapsible that don't work and the linked issue was about preselected behaviour.

Hi @renadlerossi,
I'm am not able to reproduce your issue, could you please confirm if you're still having problems regarding inputs inside mz-collapsible component?
Here is the result of my test and some code sample I used.
Let me know if I can be of any help,

html
<mz-collapsible [mode]="'accordion'">

  <mz-collapsible-item>
    <mz-collapsible-item-header>
        <i mz-icon-mdi [icon]="'heart'"></i>Select in collapsible
    </mz-collapsible-item-header>

    <mz-collapsible-item-body>
      <mz-input-container>
        <input mz-input
          [label]="'Label as placeholder'"
          id="basic-input"
          type="text">
      </mz-input-container>

      <mz-select-container>
        <select mz-select
          id="select"
          [label]="selectLabel"
          [placeholder]="selectPlaceholder"
          [disabled]="selectDisabled"
          [(ngModel)]="selectValue">
          <option *ngFor="let selectOption of selectOptions" [selected]="selectOption === selectValue">{{ selectOption }}</option>
        </select>
      </mz-select-container>
    </mz-collapsible-item-body>
  </mz-collapsible-item>

</mz-collapsible>

typescript
selectOptions: string[];
selectValue: string;

ngOnInit() {
  // simulate 1000ms latency from server
  setTimeout(() => {
    this.selectOptions = ['Option 1', 'Option 2', 'Option 3'];
    this.selectValue = this.selectOptions[1];
  }, 1000);
}

I am closing the issue as I am unable to reproduce the problem and got no response.
Feel free to reopen if there is anything!


SauceConnect failing?

Opening local tunnel using Sauce Connect
Starting sc with args: -u XXXXXXXX -k XXXXXXXX --verbose -vv -P 4445 --log-stats 5 --readyfile /var/folders/8m/y7txsryn7ms7sfvtz11tjfyw0000gn/T/sc-launcher-readyfile
A service failed in the 'onPrepare' hook
Error: spawn EACCES
    at exports._errnoException (util.js:1050:11)
    at ChildProcess.spawn (internal/child_process.js:319:11)
    at exports.spawn (child_process.js:378:9)
    at connect (/Users/nathan/Projects/symbiont/wdio-cuke/uat/node_modules/sauce-connect-launcher/lib/sauce-connect-launcher.js:494:11)
    at /Users/nathan/Projects/symbiont/wdio-cuke/uat/node_modules/sauce-connect-launcher/lib/sauce-connect-launcher.js:599:12
    at tryRun (/Users/nathan/Projects/symbiont/wdio-cuke/uat/node_modules/sauce-connect-launcher/lib/try_run.js:4:3)
    at run (/Users/nathan/Projects/symbiont/wdio-cuke/uat/node_modules/sauce-connect-launcher/lib/sauce-connect-launcher.js:594:3)
    at /Users/nathan/Projects/symbiont/wdio-cuke/uat/node_modules/sauce-connect-launcher/lib/sauce-connect-launcher.js:623:14
    at nextTask (/Users/nathan/Projects/symbiont/wdio-cuke/uat/node_modules/async/dist/async.js:5210:14)
    at /Users/nathan/Projects/symbiont/wdio-cuke/uat/node_modules/async/dist/async.js:5204:13

@nwwells I guess this is due to the missing SC binary as reported in your last issue. This is nothing that can be fixed within this project. I had no issues using SC with default params.


Error when convert XGBClassifier to onnx

When I convert XGBClassifier to onnx an error appears: "AssertionError: Modality b'BRANCH_LT' not supported"
import xgboost as xgb
from hummingbird.ml import convert
from sklearn.metrics import accuracy_score
from onnxconverter_common.data_types import FloatTensorType
from onnxmltools.convert import convert_xgboost
import onnxruntime as ort

num_classes = 2
X = np.array(np.random.rand(10000, 28), dtype=np.float32)
y = np.random.randint(num_classes, size=10000)
X_test = np.array(np.random.rand(10000, 28), dtype=np.float32)
y_test = np.random.randint(num_classes, size=10000)

model = xgb.XGBClassifier()
model.fit(X, y)

predict = model.predict(X_test)
print('accuracy', accuracy_score(y_test, predict))

initial_types = [("input", FloatTensorType([X.shape[0], X.shape[1]]))] # Define the inputs for the ONNX
onnx_ml_model = convert_xgboost(model, initial_types=initial_types, target_opset=9)

onnx_model = convert(onnx_ml_model, "onnx", X)

session = ort.InferenceSession(onnx_model.SerializeToString())
output_names = [session.get_outputs()[i].name for i in range(len(session.get_outputs()))]
inputs = {session.get_inputs()[0].name: X_test}

predict_2 = session.run(output_names, inputs)[0]
print('accuracy after', accuracy_score(y_test, predict_2))

Codecov Report

Merging #226 into master will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #226      +/-   ##
==========================================
- Coverage   94.27%   94.27%   -0.01%
==========================================
  Files          39       39
  Lines        1940     1939       -1
  Branches      348      348
==========================================
- Hits         1829     1828       -1
  Misses         55       55
  Partials       56       56

Flag
Coverage Δ

#unittests
94.27% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...l/operator_converters/onnx/onnxml_tree_ensemble.py
91.39% <0.00%> (ø)

...rd/ml/operator_converters/_tree_implementations.py
98.14% <0.00%> (-0.01%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks @arfangeta for the contribution! Unfortunately this is not enough to support LT. We will have to add < to the tree implementations, probably by setting the comparison operator (LE or LEQ) from configuration. Do you want to give it a shot? Otherwise I can work on it.

Closed by #631.


Fix invisible text in Readme

Fixed some JSX tags that are invisible due to <> elements by converting to code blocks

 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.

Fixed up - please do the CLA and offer more PRs! Thanks


PHP SDK proposal

Summary
Describes the PHP SDK for Temporal based on RoadRunner's long-running workers.
Status
The draft proposal is ready but at this stage, I require Temporal team and community feedback in order to continue.
Basic example
class ServiceUsageWorkflow extends Workflow
{
    private int $points = 0;

    /** @Workflow\QueryMethod(name="points") */
    public function getPoints(): int
    {
        return $this->points;
    }

    /** @Workflow\SignalMethod(name="points") */
    public function addPoints(int $points): void
    {
        $this->points += $points;
    }

    /** @Workflow(name="serviceUsage") */
    public function run(string $customerID)
    {
        try {
            while(true) {
                 yield $this->sleep(Time::DAY);

                if ($this->points > 0) {
                    $this->points--;
                } else {
                    yield $this->activities->chargeDailyUsage($customerID);
                    yield $this->activities->prolongAccess($customerID);
                }
            }
        } catch (CancellationException $e) {
            yield $this->activities->cancelAccess($customerID);
            yield $this->activities->cancelSubscription($customerID);
        }
    }
}

Annotations format TBD.

Motivation
Not applicable.
Detailed design
Please see the design in the attached MD document.

Please note the RR based approach will make it possible to implement IPC based clients in any other language while utilizing native Golang API.

Drawbacks
IPC based solution, slower than in-memory communication. However, the local pipe latency should not cause any issues since all the events pipeв from a remote server over TCP anyway.
Alternatives
None.
Adoption strategy
It can be handled by the PHP community.
How we teach this
We can convert proposals to docs at a later stage.
Unresolved questions
Syntax approach.

Time via static global variable which switches when context changes

unit-testing is out of scope

REMOVE LOGGING

replace yield for DECLARATION (non-blocking) with local queue via global context

support previous exception in php

out of cache ex: destroy exeption

I'm going to do my best to look this over thoroughly this week (def not fluent in PHP). Looked things over at a high level already and it looks really high quality work. Seriously appreciate the amount of time and effort you put in here.

Thank you. I left comments after talking to Max, will address them next week.

@mfateev @rylandg I did some adjustments. Let me know if I missed something.


how to get Results when use pop or beamBack

like this:
var result = await router.push(LoginRoute());
router.pop(true);

Refer to auto_route#returning-result

Hi, I had this similar question also here https://github.com/slovnicki/beamer/issues/525#issue-1251695691

Hey @lemos1235 :wave:
Thanks for creating an issue!
Indeed, as @arkanmgerges said, we can refer to #525 for that question and I will post updates there as I'm developing more features for this scenario. Therefore, I will close this one :slightly_smiling_face:


HDDS-11869. Enable OM Ratis in TestOzoneDelegationTokenSecretManager

What changes were proposed in this pull request?
Fix tests to renew/cancel TestOzoneDelegationTokenSecretManager with OM Ratis enabled.
In ratis, create/renew token doesn't add token to store until validateAndUpdateCache.
Add addToTokenStore to the tests.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-11869
How was this patch tested?
CI:
https://github.com/chungen0126/ozone/actions/runs/12396367209

The OzoneDelegationTokenSecretManager.renewToken() has this code snippet:
I believe this snippet should be removed as part of this PR.
There are other places in this class where non-HA code is expected to be removed too.

Originally prod code was planned to be removed in HDDS-11866.

Sorry for nitpicking, but I think we should restrict this change to the test.  If we merge it as is, OzoneDelegationTokenSecretManager will ignore the config, while other parts of the code continue to use it.  OzoneDelegationTokenSecretManager will stop working with OM Ratis disabled.
I think we should drop support in the following steps:

ignore config, use hard-coded value (HDDS-11864)
remove production code (HDDS-11866)
remove tests (HDDS-11865 + couple of tickets for specific tests, like this)

@adoroszlai I agree. I'll revert those changes in the production code.

Thanks @chungen0126 for the patch, @ChenSammi, @jojochuang for the review.


Hash function in Python 3.2.3 returns different results between sessions

I am using the last stable python version (3.10.4) for Mac (M1).
Every new execution (or session) generates a different hash in this part:
https://github.com/python-cachier/cachier/blob/56cf8c7f74e1e65ffd83e49aa21e7baa62dcb207/cachier/base_core.py#L14-L17
The problem is related to this new feature:

PYTHONHASHSEED
If this variable is not set or set to random, a random value is used to seed the hashes of str and bytes objects.
If PYTHONHASHSEED is set to an integer value, > it is used as a fixed seed for generating the hash() of the types covered by the hash randomization.
Its purpose is to allow repeatable hashing, such as for selftests for the interpreter itself, or to allow a cluster of python processes to share hash values.
The integer must be a decimal number in the range [0,4294967295]. Specifying the value 0 will disable hash randomization.
New in version 3.2.3.

References:

https://stackoverflow.com/q/27522626/511514
https://stackoverflow.com/q/30585108/511514

Yes. It is not good for those who are using this library and updated the Python version.
After updating the Python, the code will work properly, and the developer may not see the issue.
For now, I am setting the environment variable PYTHONHASHSEED in my environment as 0.


Update libs

data_platform_libs/data_interface to v1.12
operator_libs_linux/snap from v1 to v2.1

Moved the work on this to this PR.


canonise vs canonize vs canonicalize

According to wikipedia, the verb " to put something in canonical form" is "to canonicalize",  https://en.wikipedia.org/wiki/Canonical_form
However, it seems that in other contexts "to canonize" is used
https://en.wikipedia.org/wiki/Graph_canonization
Schollwock also uses "to canonize".
Any suggestions on which convention to choose? It would be better to choose now, rather than later.

I like the shorter canonise/ze. Canonise vs canonize is a question of British/Aussie English vs American. I usually try to write American English myself, but I don't have any strong preference.

OK I'll keep it as canonise for now.


[TINY] Fix copy command in README

Removed the $ sign to allow users to take advantage of the handy copy snippet functionality offered by the GitHub UI

@ilmax Could you rebase this MR with signed commits? This is required for us to merge these changes in


Add in another name combo for libcrypto that matches libssl.

libcrypto should have matching find pattern that libssl has for builds without the prefix on the front.

just read this isn't the right place to submit patches.


Update and rename options.example.js to options.js

@lisa262 - if these are your real details (username, ip address, etc), then you should change the auth info.  You don't want this stuff sitting out in the open

Thanks; multitasking and didn't notice I'd left all that in there...

On Nov 6, 2016, at 11:54 PM, Ryan Graciano<EMAIL_ADDRESS>wrote:
@lisa262 - if these are your real details (username, ip address, etc), then you should change the auth info. You don't want this stuff sitting out in the open
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


CAMEL-11008: Consumer/Producer templates are not stopped when auto-configured in Spring Boot

In [CAMEL-9431|https://issues.apache.org/jira/browse/CAMEL-9431] producer/consumer templates did not receive their own destroyMethod methods. I've changed those to use stop for destroyMethod, a proper solution might be that DefaultCamelContext tracks all templates and stops them on shutdown, or make them CamelContextAware and addService them to CamelContext, or addService them in createProducerTemplate createConsumerTemplate.
I've decided to do a simple fix here and provide a unit test that can be used to validate the best solution to the problem.

I recall something about this. Did you check the git blame of those code lines, it may refer to a good CAMEL ticket about this.

@davsclaus yeah, I think its this CAMEL-9431 i.e. https://github.com/apache/camel/commit/4af2c3c34bfee00e58632a3b1dc6e299cfd43661

I think adding them as service to camel context is better. It does not make sense to have a template that is tied to a camel context which has been stopped which means your template is no longer usable either.
So in your unit test when camel context is stopped those templates should be stopped also.

@davsclaus yeah that was on purpose to demonstrate the issue, will rework and addService them, thanks :+1:

@davsclaus what do you think can I merge this?

Oh you should addService from spring-boot and not in camel-core, that is a big change.

Yeah, that would be easier, but who stops these if I get them from CamelContext without Spring Boot? For instance if I do:
final DefaultCamelContext context = new DefaultCamelContext();
context.start();

try {
    context.createProducerTemplate().requestBody("undertow:http://www.google.com", null, String.class);
} finally {
    context.shutdown();
}

JVM does not stop because the UndertowProducer's XnioWorker starts daemon threads, and the UndertowProducer's doStop is never called to stop them :worried:

Read the contract on the camel context api, it says YOU must call stop.

@davsclaus are you referring to the big "Important" text in the javadoc, yeah totally glossed over that :smile:, will rework to add addService on the Spring Boot side.

@davsclaus can you have another look, thanks :+1:

the spring boot afair is also a non singleton bean so you get a shared template if you get it multiple times.
Where as with createXXX it create a new instance.
For Camel 3.0 we can consider an api on camel context that creates and add as service to control its lifecycle OOTB. But not sure what a good method name would be. If changing the current default then users that want a temporary template would need to not use CamelContext anymore but have to create it manually using the new DefaultProducerTemplate ...
Also the current behavior is also how it has been working for years, so its better to keep 2.x as-is.

+1 lgtm

Merged into master (2.19.x), 2.18.x and 2.17.x branches


toString methods should return something that's more readable

Project should be its title for example.

I happened to just push a change to master.
Based on what's being deployed, let's see what more we need to do on this issue.

You can code for shortTitle and if its not there use identifier.

That's what I did already.
https://aide.md.kth.se/sysml_oslc_server/services/projects/585a5a94-61a0-4a60-9731-c2be32b06f89/commits/7cbc0568-1f44-4725-ace0-e7d4e1c046c6/service3/sysmlClasss/query


Item Equipping issue

Steps to recreate:
Have a rusty sword equipped, you can't remove it as the game doesn't allow it to be removed (it says you are not wearing that item but it's shown in EQ list)
when you type equip long sword then it equips it and just deletes the rusty as it can't remove it.
 Potentially item location status is not being set correctly

fixed


Index logging and counter cleanup

Tweaked logging output during indexing to make less noisy.
Documentation and clarification of what each counter means.
Better logging of final counter states.

@Peilin-Yang Addressed CR. Running from scratch:
nohup python src/main/python/run_regression_tweets2013.py >& log.tweets2013 &

@Peilin-Yang Actually, I just realized run_regression_tweets2013.py is likely to fail... let me rethink how to handle.

@Peilin-Yang Actually, you were correct. For Tweets2013 the indexer did throw an exception because of the duplicate docs issue. However, since by that time the index had already been built, the exception doesn't really do anything and the regression completes as usual.
I've changed to logging a warning instead of throwing an exception. Running one final time and then I'll merge.

I am a bit confused. Is this related to #282 ?

@Peilin-Yang This is related to your -uniqueDocid comment.
Regardless, tweets2011 and tweets2013 passed perfectly.


Event at launch of animation

This pull request addresses issue #112 . Adding events when the flip or unflip start, could be an interesting feature in the case that we don't fully control the HTML markup. Therefore, we could control the content of the back and front sides before we flip them.
Besides the flip:start event, two more events were added: unflip:start and unflip:done.
The function flipDone was refactored to accept a list of events that fire after the flip is done.

Why was this not merged yet?


Error loading ImageBitmap for Cesium3DTileset

No problem at 1.55;
Error Message:
A 3D tile failed to load: http://../**.b3dm
Cesium.js:507 Error: Failed to load image: id: 3419, bufferView: 18
The ImageBitmap could not be allocated.
Sandcastle example:
var modelMatrix = Cesium.Transforms.eastNorthUpToFixedFrame(Cesium.Cartesian3.fromDegrees(longitude, latitude, height));
var tileset = viewer.scene.primitives.add(new Cesium.Cesium3DTileset({
url : 'http://..../tileset.json',
modelMatrix: modelMatrix
}));
Browser: Google Chrome 版本 74.0.3729.131（正式版本） （32 位）
Cesium:1.57
Operating System: win10

Thanks for reporting this @xiaobaogeit ! Are you able to share your tileset so we can reproduce and fix this issue?

This tileset will work fine in the latest version of firefox, but it will be very memory-intensive and will have poor performance.
Tilset address is as follows:
https://github.com/xiaobaogeit/model/blob/master/tiles.zip

@xiaobaogeit I'm not able to replicate a crash a with this tileset on the latest CesiumJS. Are you saying this tileset has poor performance in 1.57 but has good performance in 1.55?
Can you test the crash in Firefox and copy/paste the error you get from there? Or does this only happen in Chrome?

@OmarShehata
Operating System: win10 Pro (x64)
1、Firefox latest version of the rendering is no problem, but eat memory.

2、In version 1.57, Google renders an error and eventually the page crashes.
Error Message:
A 3D tile failed to load: http://../tiles/he_1_0.b3dm
Cesium.js:507 Error: Failed to load image: id: 3319, bufferView: 16
The ImageBitmap could not be allocated.
Browser: Google Chrome 版本 74.0.3729.131（正式版本） （32 位）
3、For version 1.55, after multi-machine test, whether can render out depends on the performance of the computer, but 1.57 are not rendered out.
4、I only tested Google and firefox


DOCS-1950-ebpf-host-ports

Delete reference to host ports not supported.
Product Version(s):
CC, CE, OSS versions are TBD
Issue:
https://tigera.slack.com/archives/C03NP7QJSC8/p1698698095538919
Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

Here is a confirmation from the field that it works https://github.com/projectcalico/calico/issues/6522#issuecomment-1795333492

@tomastigera How many versions back should I fix this? OSS goes back to 3.24, and CE goes back to 3.15.


Result cannot be instatiated deserializing with jackson

how to solve this?:
com.fasterxml.jackson.databind.exc.InvalidDefinitionException: Cannot construct instance of io.requery.query.Result (no Creators, like default construct, exist): abstract types either need to be mapped to concrete types, have custom deserializer, or contain additional type information`

You would need to ignore the Result from Serialization or have an adapter that fetches the results into the object and serializes them. The Result class itself is not mean't to be serialized.