Support for dishwasher consumableSettings

In dishwasher config there is consumableSettings section. We should handle it
"consumableSettings": {
    "consumables": [
      {
        "consumableType": "DETERGENT",
        "forms": [
          {
            "consumableForm": "TABLET",
            "lastCycleConsumptionAmountDataArrayReadIndex": 69
          },
          {
            "consumableForm": "LIQUID",
            "lastCycleConsumptionAmountDataArrayReadIndex": 73,
            "lastCycleConsumptionAmountFactor": 1.6,
            "warningSetting": {
              "wifiArrayReadIndex": 61,
              "bitIndex": 7
            }
          }
        ]
      },
      {
        "consumableType": "RINSE_AID",
        "forms": [
          {
            "consumableForm": "LIQUID",
            "lastCycleConsumptionAmountDataArrayReadIndex": 72,
            "lastCycleConsumptionAmountFactor": 1.5,
            "warningSetting": {
              "wifiArrayReadIndex": 61,
              "bitIndex": 0
            },
            "autoDosingAmountSetting": {
              "wifiArrayIndex": 64,
              "boundedValues": [
                {
                  "strKey": "",
                  "lowerLimit": 0,
                  "upperLimit": 4,
                  "step": 1,
                  "factor": 1,
                  "unit": ""
                }
              ]
            }
          }
        ]
      },
      {
        "consumableType": "SALT",
        "forms": [
          {
            "consumableForm": "POWDER",
            "lastCycleConsumptionAmountDataArrayReadIndex": 71,
            "warningSetting": {
              "wifiArrayReadIndex": 61,
              "bitIndex": 1
            }
          }
        ]
      }
    ]
  }

Hello can I be some help? I have dishwasher. Also oficial app can notify you if you are low on detergent or rinser. That whould be cool to have in HA.

Hi, @Scorpio256 your help would be very welcome :)
Can you figure out how to map consumableSettings to Home Assistant entities? https://developers.home-assistant.io/docs/core/entity
For washing machines, I've implemented warnings as a Binary Sensor Entity
But what about lastCycleConsumptionAmountData is this a read-only information?
Also what's the deal with autoDosingAmountSetting?

Would it be possible to create new entities for this on the fly? Say you switch from a tablet to a liquid detergent and then a new warning / binary entity is created.

I don't know if removing entities from the integration is technically possible, but I don't think it is a good idea. I imagine HA automation complaining about missing entities. If we have binary sensor for warnings, we can just leave it "False".
Anyway in this case, consumableSettings.warningSetting is a duplicate of top-level warningSetting, so it should be already handled by #86

As discussed with @Scorpio256 over email:
The part
"lastCycleConsumptionAmountDataArrayReadIndex": 73,
"lastCycleConsumptionAmountFactor": 1.6,

Probably relates to the consumable amount section and should be represented as an sensor entity in HA

The part
"autoDosingAmountSetting": {
  "wifiArrayIndex": 64,
  "boundedValues": [
    {
      "strKey": "",
      "lowerLimit": 0,
      "upperLimit": 4,
      "step": 1,
      "factor": 1,
      "unit": ""
    }
  ]
}

Most certainly relates to "Amount per cycle" section and should be represented in HA as a select entity

@Scorpio256 can you confirm?

After updating to the latest version of the integration, the entities are starting to show data.
Unfortunately, no entities have been created for any of my dishwasher settings.
The dishwasher cannot be turned on via the integration, I have to turn it on manually and turn on the remote control. However, this is also how it behaves with the official app.
The following entities can be fully or partially controlled.
Extra Rinse - Can be turned on cannot be turned off
Fast+ - The dishwasher receives the signal but does not turn on (probably the correct washing program must be set)
Extra
Half Load - Can be switched on cannot be switched off
Program - Can be switched, you can see on the dishwasher that the programs are changing
State - The "running" state can be used to turn on the wash program completely but does not respond to other states and many of the states are more informative
Steamgloss - Can be switched on but cannot be switched off
Tray Wash - Can be turned on cannot be turned off

The sensors seem to show correctly the time remaining and how long the program is taking.
Sub-state - shows the current information what the dishwasher is doing once it starts washing.
However, notifications and information about refills are missing.
For example, now the app is reporting to me a lack of Detergent and the integration doesn't show that at all.

Is your device ID F999961730155826793828?
I just tested and you should have the following entities in HA:
            "STATE",
            "PROGRAM",
            "SUB_STATE",
            "DISHWASHER_TRAYWASH",
            "DISHWASHER_FAST+",
            "DISHWASHER_STEAMGLOSS",
            "DISHWASHER_EXTRARINSE",
            "DISHWASHER_HALFLOAD",
            "DISHWASHER_DELAY",
            "DISHWASHER_DURATION",
            "DISHWASHER_REMAINING",
            "REMOTE_CONTROL",
            "DISHWASHER_WARNING_NO_RINSE_AID",
            "DISHWASHER_WARNING_NO_SALT",
            "DISHWASHER_WARNING_NO_WATER",
            "DISHWASHER_WARNING_CHECK_THE_FILTER",
            "DISHWASHER_WARNING_DOOR_IS_OPEN",
            "DISHWASHER_WARNING_CALL_SERVICE",
            "DISHWASHER_LIQUID_DETERGENT_LOW",
            "SETTING_AUTO_DOOR",
            "SETTING_DETERGENT_TYPE",
            "DISHWASHER_RINSE_AID_LEVEL",
            "DISHWASHER_WATER_HARDNESS",
            "SETTING_INNER_ILLUMINATION",

Maybe try reconfiguring the integration?

I don't see any of the newly added entities in v0.3.0 on your screens. Maybe just restart the HA?

I removed the device, restarted HA, added the device and still don't see any more entities.

Can you share your login info at<EMAIL_ADDRESS>

Checked on two separate HA instances. All entities are there

Works fine with new version v0.3.0

@Scorpio256 Can this ticket then be closed or is there something else that doesn't work / is still missing?

I think all good

Entities described here are not yet implemented https://github.com/rowysock/home-assistant-HomeWhiz/issues/92#issuecomment-1369169776. @Scorpio256 please confirm that the described Entities match the original app

Hello!
I have a WiFi dishwasher (Beko BDIN39640A) and I can confirm that the Entities match the values from the dishwasher and the App.
And the issues I had:
data not refreshing
The data from dishwasher updates very irregularly. Mostly it happens when I open the DW in the app, and even then data may not update. Very rarely (maybe couple times per week) it updates by itself, but I can't get the idea what is triggering it.
In the logs I see one entry related to HomeWhiz:
2023-02-04 19:47:14.003 ERROR (MainThread) [homeassistant] Error doing job: Exception in callback async_track_point_in_utc_time.<locals>.run_action(<Job HassJobT...7f2bcde96710>>) at /usr/src/homeassistant/homeassistant/helpers/event.py:1310 Traceback (most recent call last):   File "/usr/local/lib/python3.10/asyncio/events.py", line 80, in _run     self._context.run(self._callback, *self._args)   File "/usr/src/homeassistant/homeassistant/helpers/event.py", line 1325, in run_action     hass.async_run_hass_job(job, utc_point_in_time)   File "/usr/src/homeassistant/homeassistant/core.py", line 580, in async_run_hass_job     hassjob.target(*args)   File "/usr/src/homeassistant/homeassistant/helpers/event.py", line 1285, in utc_converter     hass.async_run_hass_job(job, dt_util.as_local(utc_now))   File "/usr/src/homeassistant/homeassistant/core.py", line 583, in async_run_hass_job     return self.async_add_hass_job(hassjob, *args)   File "/usr/src/homeassistant/homeassistant/core.py", line 486, in async_add_hass_job     task = self.loop.create_task(hassjob.target(*args)) TypeError: HomewhizCloudUpdateCoordinator.refresh_connection() takes 1 positional argument but 2 were given
GB locale missing
I'm using the GB locale on HASS and adding the Dishwasher I had windows without any text. I solved the issue by copying translation files "en" to "gb". Now it works fine
incorrect instructions in the README.MD
I installed the extension manually (because it isn't available in HACS). First, I created the directory /custom_components/HomeWhiz as per README and it didn't work. The correct directory is /homewhiz (all lowercase)
I'm willing to commit to the project, but I'm totally new to HomeAssistant, GitHub and almost new to Python (experience limited to data analysis and such), so please don't have too much trust in me!


[xerces-c] Remove unnecessary duplication of import prefix

This fixes #5819 by simply removing the rewrite of the XercesCConfigInternal.cmake that appears to duplicate the _IMPORT_PREFIX. It is not immediately clear why this is being done in the first place but appears unnecessary and causes the package to not be found. Users are currently working around this by removing or manually editing the generated file.

/azp run

Please also bump the version in the CONTROL file

Hi @ruggin, here are some regressions from the current CI system:

x64-windows
test
notes

msix
Fail
Regression

x86-windows
test
notes

msix
Fail
Regression

failureLogs.zip


Document improvements to SV input in VEP (e110)

Related with https://github.com/Ensembl/ensembl-vep/pull/1370 and https://github.com/Ensembl/ensembl-vep/pull/1399

Standardise supported SV types across default VEP input, REST-style regions and VCF
Mention CNV support for <CN=3>-like format
Mention breakend support
Add more SV examples
Copy-edit documentation

Check changes in my sandbox: http://wp-np2-11.ebi.ac.uk:6070/info/docs/tools/vep/vep_formats.html

Hey @likhitha-surapaneni, just updated the documentation to improve on this.


tippecanoe: 2.9.1 -> 2.11.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/felt/tippecanoe/releases.
meta.description for tippecanoe is: "Build vector tilesets from large collections of GeoJSON features"
meta.homepage for tippecanoe is: "https://github.com/felt/tippecanoe"
meta.changelog for tippecanoe is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.11.0 with grep in /nix/store/5lrdn1v3bzw2mcaqgr9hsm7hmhpijfhw-tippecanoe-2.11.0
found 2.11.0 in filename of file in /nix/store/5lrdn1v3bzw2mcaqgr9hsm7hmhpijfhw-tippecanoe-2.11.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
tippecanoe

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5lrdn1v3bzw2mcaqgr9hsm7hmhpijfhw-tippecanoe-2.11.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A tippecanoe https://github.com/r-ryantm/nixpkgs/archive/1fa1b22364eff95154097914877e3e7b6bf151b6.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5lrdn1v3bzw2mcaqgr9hsm7hmhpijfhw-tippecanoe-2.11.0
ls -la /nix/store/5lrdn1v3bzw2mcaqgr9hsm7hmhpijfhw-tippecanoe-2.11.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

tippecanoe

Maintainer pings
cc @sikmir for testing.

Result of nixpkgs-review pr 200534 run on x86_64-darwin 1

1 package built:

tippecanoe


add time granularity to redshift

Coverage increased (+0.004%) to 70.166% when pulling a11095494f3fb73fe718c29634767363a25324c8 on CyrilLeMat:add-time-grains-to-redshift-engine into 17c7ca239aa2d17ac5d6abd601ecae78a343499e on apache:master.

yes it's exactly like the postgresql time grains. I added it because on the 0.20, I'm not able anymore to use time grains on redshift. If it's supposed to inherit from postgresl, then it's a bug. is it supposed to do so?

Did your patch here fix the "bug"? What are the symptoms, is the dropdown empty?


updated README.md , added hyperlink of official site to banner

updated README.md , added hyperlink of official site to banner, it might be beneficial ,please check
closes #19490

@joshdholtz  , thank you for giving time to review this PR  :)


Error at install steroids (Node 5 and NPM 2)

I am unable to install steroids with Node 5 and NPM 2. Could you please help?
Yeoman Doctor
Running sanity checks on your system
✔ Global configuration file is valid
✔ NODE_PATH matches the npm root
✔ Node.js version
✔ No .bowerrc file in home directory
✔ No .yo-rc.json file in home directory
✖ npm version
Your npm version is outdated.
Upgrade to the latest version by running:
npm install -g npm
Found potential issues on your machine :(
/usr/local/bin/steroids -> /usr/local/lib/node_modules/steroids/bin/steroids

Even though the error message is given by Yeoman, Steroids should still work fine. If you run the steroids command, what happens?

Hi Hari -
Thanks for your response. I fixed the error.
Cheers,
Ali

On 14 Dec 2015, at 11:35 PM, Harri Sarsa<EMAIL_ADDRESS>wrote:
Even though the error message is given by Yeoman, Steroids should still work fine. If you run the steroids command, what happens?
—
Reply to this email directly or view it on GitHub https://github.com/AppGyver/steroids/issues/1083#issuecomment-164426130.


body creation not working as expected

Currently, the document's <body> element doesn't get created by the lines around 48-49 as expected. No body is appended to the document until the XHR resolves happily in ulUtilGet, resulting in a failure to create a body element for pages that use no partials.

doesn't reproduce in the mwe in gh-pages; seems to be an upstream problem with beam companion explorer.


[Stream Deck+] Custom background is covered by HASS entity

Maybe there's a way to use a transparent background?

Easy fix :)
But may look ugly depending on the brightness of the background image:


Please report more info when not like the target database errors are given

When errors like the following are shown:
WARNING: The database does not match the target database:
 - Table "account" is not like the target database.

it would be good to be told at least one way in which the database schema doesn't match the expected schema.

Hey 👋
Makes sense to me, good catch!

Same here, just don't have much to go on and the proposed fix are not really helping. thanks


[bug]: error while loading shared libraries: libc++.so.1: cannot open shared object file: No such file or directory

Describe the bug
failed to build the tgt
Reproduction steps

git clone git@github.com:FedericoBruzzone/tgt.git
cd tgt
cargo build --release --features download-tdlib

Expected behavior
Install and work.
Logs and screenshots
Compiling tgt v1.0.0 (/home/celly/tgt)
error: failed to run custom build command for `tgt v1.0.0 (/home/celly/tgt)`

Caused by:
  process didn't exit successfully: `/home/celly/tgt/target/release/build/tgt-fb0c1f67b870d63c/build-script-build` (exit status: 127)
  --- stderr
  /home/celly/tgt/target/release/build/tgt-fb0c1f67b870d63c/build-script-build: error while loading shared libraries: libc++.so.1: cannot open shared object file: No such file or directory

Method of Installation
Build from source
tgt version
git latest
Operating System (including version and architecture)
Arch Linux x86_64, 6.10.3-arch1-2
Terminal and Command Line Interface (CLI) (including versions)
fish, version 3.7.1
rustc and rustup target (including versions)
rustc 1.80.0 (051478957 2024-07-21)

I installed libc++ and libc++abi from extra arch's repository and worked, later i checked https://github.com/FedericoBruzzone/tgt/issues/32 and fixed the remaining error.

Perfect, thank you for reporting it! 🫶


Don't remove launchctl services when upgrading cask

Verification

[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature
Don't remove launchctl services when upgrading a cask.
What is the motivation for the feature?
Upgrading a cask is not the same as removing and reinstalling it. If the services are removed, the app will try/prompt to set them up again after upgrade, which is annoying.
How will the feature be relevant to at least 90% of Homebrew users?
Upgrading a cask with launchctl services will be much smoother.
What alternatives to the feature have been considered?
N/A

CC @Homebrew/cask for thoughts.

I guess a big issue I see is how to handle situations where services change between versions.

This would also be my primary concern. What happens when the launchctl changes on upgrade.

Passing on this for now given a bove, sorry @stevenlele.


Nucleus causes errors on some of its event listener registration

I am not sure where this problem sprouts from or if it is even problematic, but it does spit out quite a few errors to the console. I couldn't really follow the stacktrace well into what is occurring in Nucleus.
https://gist.github.com/Meronat/10afccb29517684afcdc8ed38f7b2ae5

@dualspiral may have some input here.

When did it crash? Startup or reload?

@dualspiral So it does not crash, Lantern and Nucleus both continue to work. This is on start-up though as Nucleus is registering its listeners initially.
It seems Nucleus may be registering these listeners more than once somehow or Lantern may be doing something wrong. See the registration code:
https://github.com/LanternPowered/LanternServer/blob/master/src/main/java/org/lanternpowered/server/event/LanternEventManager.java#L172

If anything, it's unregistrstion that's the problem - it may not be unregistering.


Fix unexpected token error

When trying to use this plugin, I ran into the following error:
                if (!ctx?.bundle)
                         ^

SyntaxError: Unexpected token '.'
    at wrapSafe (internal/modules/cjs/loader.js:931:16)
    at Module._compile (internal/modules/cjs/loader.js:979:27)
    at Module._extensions..js (internal/modules/cjs/loader.js:1035:10)

This can be fixed by removing the ? operator and replacing with a more explicit check

Hi @paintedbicycle,
Thanks for the PR!
Optional chaining is still in the TC39 proposal stage, but it's pretty much universally supported.
Node added support for it in version 14, which came out a little under a year ago. Can you confirm if you're still using an older version than that, which would explain the error?
I've accepted the pull request, but if you're using pre-14.0, is that common? I only deal with node for builds, not on the server side, so if there's a strong reason for keeping this compatible with an older version of node, I'll need to make some notes so I don't trip into newer unsupported features in future versions.

Yes, I do use it in my apps, but for some reason I hit an error here. My local dev environment is indeed pinned to node 12 because I have an app that uses the Google Cloud environment. Their cloud functions use node 12. About a month or so ago they released node 14 support but it's still in beta for them so don't want to move my product app to it yet

Ah! Makes sense. v0.5.1 has been published with this fix.


[Bug] GrafanaDashboard import does not work when oauth is enabled

Describe the bug
When we create a grafana dashboard custom resource, it is not able to import dashboard and I get an error in events stating "error getting folders, expected status 200 but got 401". I believe it is not able to hit /api/folders after enabling oauth.
here's the full log of controller manager
2022-04-21T14:34:18.339Z        ERROR   failed to get or create namespace folder for dashboard  {"folder": "default", "dashboard": "", "error": "error getting folders, expected status 200 but got 401"}
github.com/go-logr/zapr.(*zapLogger).Error
        /go/pkg/mod/github.com/go-logr/zapr@v0.2.0/zapr.go:132
sigs.k8s.io/controller-runtime/pkg/log.(*DelegatingLogger).Error
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.8.3/pkg/log/deleg.go:144
github.com/grafana-operator/grafana-operator/v4/controllers/grafanadashboard.(*GrafanaDashboardReconciler).reconcileDashboards
        /workspace/controllers/grafanadashboard/grafanadashboard_controller.go:283
github.com/grafana-operator/grafana-operator/v4/controllers/grafanadashboard.(*GrafanaDashboardReconciler).Reconcile
        /workspace/controllers/grafanadashboard/grafanadashboard_controller.go:101
github.com/grafana-operator/grafana-operator/v4/controllers/grafanadashboard.SetupWithManager.func1
        /workspace/controllers/grafanadashboard/grafanadashboard_controller.go:184
github.com/grafana-operator/grafana-operator/v4/controllers/grafanadashboard.SetupWithManager.func2
        /workspace/controllers/grafanadashboard/grafanadashboard_controller.go:193
2022-04-21T14:34:18.339Z        ERROR   error updating dashboard        {"error": "error getting folders, expected status 200 but got 401"}
github.com/go-logr/zapr.(*zapLogger).Error
        /go/pkg/mod/github.com/go-logr/zapr@v0.2.0/zapr.go:132
sigs.k8s.io/controller-runtime/pkg/log.(*DelegatingLogger).Error
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.8.3/pkg/log/deleg.go:144
github.com/grafana-operator/grafana-operator/v4/controllers/grafanadashboard.(*GrafanaDashboardReconciler).manageError
        /workspace/controllers/grafanadashboard/grafanadashboard_controller.go:468
github.com/grafana-operator/grafana-operator/v4/controllers/grafanadashboard.(*GrafanaDashboardReconciler).reconcileDashboards
        /workspace/controllers/grafanadashboard/grafanadashboard_controller.go:284
github.com/grafana-operator/grafana-operator/v4/controllers/grafanadashboard.(*GrafanaDashboardReconciler).Reconcile
        /workspace/controllers/grafanadashboard/grafanadashboard_controller.go:101
github.com/grafana-operator/grafana-operator/v4/controllers/grafanadashboard.SetupWithManager.func1
        /workspace/controllers/grafanadashboard/grafanadashboard_controller.go:184
github.com/grafana-operator/grafana-operator/v4/controllers/grafanadashboard.SetupWithManager.func2
        /workspace/controllers/grafanadashboard/grafanadashboard_controller.go:193

Version
Full semver version of the operator being used v4.0.1
Expected behavior
Dashboard should be imported in grafana
Suspect component/Location where the bug might be occuring
Please provide this if you know where this bug might occur otherwise leave as unknown
Screenshots

Runtime (please complete the following information):

Grafana Operator Version - 4
Environment: Kubernetes
Deployment type: AWS, local

@manvindar can you update to the latest version of the operator and try again? Allot have changed since v4.0.0.

@NissesSenap  I tried the latest version and got the same error

@manvindar can you share how your yaml of your grafana and your dashboard looks like?
We need a way of reproducing this issue.
I think it sounds strange that the admin account that we use to talk to the API would get some issues when using oauth since it's a bearer token. So unless you have enforced something in grafana that makes that impossible to use I think shouldn't be a problem.

hi @NissesSenap
Please find the yamls below
###################### Creating GRAFANA for Omara cluster   ###########################################
apiVersion: integreatly.org/v1alpha1
kind: Grafana
metadata:
  name: basic
  labels:
    grafana: omara-monitoring-grafana-basic
spec:
  config:
    server:
      domain: "grafana.DOMAIN_NOT_DEFINED"
      root_url: "https://grafana.DOMAIN_NOT_DEFINED"
      enforce_domain: true
    auth:
      disable_login_form: true
      oauth_auto_login: true
      signout_redirect_url: https://id.DOMAIN_NOT_DEFINED/auth/realms/omara-core/protocol/openid-connect/logout?redirect_uri=https://grafana.DOMAIN_NOT_DEFINED/login
    auth.generic_oauth:
      enabled: true
      allow_sign_up: true
      client_id: "omara-monitoring-grafana-basic"
      client_secret: "SECRET_NOT_DEFINED"
      scopes: "openid profile email"
      role_attribute_path: "realm_access | contains(roles[*], 'platform-administrators') && 'Admin' || 'Viewer'"
      auth_url: "https://id.DOMAIN_NOT_DEFINED/auth/realms/omara-core/protocol/openid-connect/auth"
      token_url: "https://id.DOMAIN_NOT_DEFINED/auth/realms/omara-core/protocol/openid-connect/token"
      api_url: "https://id.DOMAIN_NOT_DEFINED/auth/realms/omara-core/protocol/openid-connect/userinfo"
    security:
      admin_user: "admin"
      admin_password: "admin"
      allow_embedding: true
      cookie_samesite: none
      cookie_secure: true
    smtp:
      enabled: true
      host: "mail-service:25"
      skip_verify: true
      from_address<EMAIL_ADDRESS>      from_name: Grafana
  dataStorage:
    class: hostpath
    accessModes:
      - ReadWriteOnce
    size: 10Gi
  deployment:
    securityContext:
      fsGroup: 472
      runAsUser: 472
      runAsGroup: 472
    replicas: 1
    labels:
      app: "omara-monitoring-grafana"
  dashboardLabelSelector:
    - matchExpressions:
        - { key: app, operator: In, values: [ grafana ] }
  resources:
    requests:
      cpu: 500m
      memory: 50Mi

apiVersion: integreatly.org/v1alpha1
kind: GrafanaDashboard
metadata:
  labels:
    app: grafana
  name: omara-infrastructure
spec:
  json: |
        {
          ...
        }

Hi @NissesSenap
do you have any update on this?

@manvindar the operator is rather simple, it works by using the admin account, and grabs the credentials.
Then it just goes towards the API.
I don't know enough about grafana generic oauth solution but it wouldn't surprise me if it's possible to disable basic auth towards the API and enforce oauth 100%.
If this would happen you would break the operator.
Could you please verify that you can reach the API using the basic auth solution?
Below you can find a simple example on how to do it taken from our e2e tests.
NAMESPACE="grafana-operator-system"
HEADER='-H Accept:application/json -H Content-Type:application/json'
PASSWORD=$(kubectl -n $NAMESPACE get secrets grafana-admin-credentials --template={{.data.GF_SECURITY_ADMIN_PASSWORD}} | base64 -d)
kubectl port-forward -n $NAMESPACE service/grafana-service 3000:3000 &

# Check to see that your grafana instance is up:
curl localhost:3000/api/health
# Check to see if we can talk to the folders grafana API
curl $HEADER "http://admin:$PASSWORD@localhost:3000/api/search?folderIds=0"

hi @NissesSenap I get the below error, it is trying to redirect to public endpoint

@HubertStefanski @pb82 do we have a way of forcing internal traffic?

@manvindar please update the client config to prefer the service.
grafana
  spec:
    client::
      preferService: true

This should probably be more highlighted in the docs. Would love to get a PR with this information if it solves the issue for you.

I tried that also, but it doesn't seem to affect anything

@manvindar Did you check this already? https://github.com/grafana-operator/grafana-operator/issues/659#issuecomment-1009923994

yes, I tried that too, but it's not working, also in that example, they are using anonymous auth which works anyways.
If I curl API after enabling OAuth using the service name then it redirects me to the ingress url as I commented above.

Did you delete the original admin account created by the operator? I think you need to keep it in the Grafana instance although you may not be using it manually.

@manvindar can you confirm that the admin credential secret is still available and that the operator is using those credentials?

@manvindar I've had an oidc-enabled grafana for over a year, and it all worked just fine. I think the issue here is due to two parameters that you set in the server section:
spec:
  config:
    server:
      domain: "grafana.DOMAIN_NOT_DEFINED" # <---- remove it
      root_url: "https://grafana.DOMAIN_NOT_DEFINED"
      enforce_domain: true # <---- remove it

If you remove those two, grafana should not redirect you to the main domain anymore, and everything should work just fine.
Better to combine it with @NissesSenap recommendation:
spec:
  client:
    preferService: true

@manvindar would the solution from @weisdd work for you?


[$250] Update the behavior of the yellow thread highlight after exiting a thread

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

Version Number:
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail: n/a
Email or phone of affected tester (no customers): everyone
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @JmillsExpensify
Slack conversation: https://expensify.slack.com/archives/C03U7DCU4/p1714062455602839
Action Performed:

Click into a thread
Click back into the main conversation by clicking the the room title in the top left of the thread window
Observe that the thread is highlighted in yellow after you exit it

Expected Result:
The yellow highlight should fade after

Clicking out of the chat into another chat
Clicking into another thread
Refreshing the page
Sending a new comment in the same chat (but not receiving a comment from another user)
Taking any action from the + menu while in the chat

Actual Result:
Highlight remains until you click out of the chat into another chat. It does not clear after any of the other cases mentioned above.
Workaround:
Usable but many have found the highlight confusing and distracting
Platforms:

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

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

Screenshots/Videos
https://github.com/Expensify/App/assets/17131195/aa847e2d-cda1-40ba-a825-f3e1affc5129

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019325e864a40fa38f
Upwork Job ID:<PHONE_NUMBER>916172800
Last Price Increase: 2024-05-14
Automatic offers:

brunovjk | Contributor | 102439671

Proposal
Please re-state the problem that we are trying to solve in this issue.
The yellow thread highlight in the Expensify app persists even after the user has exited the thread and performed other actions within the same chat.
What is the root cause of that problem?
The root cause lies in the highlightedBackgroundColorIfNeeded logic within ReportActionItem.tsx. Currently, the highlight is only cleared when the user navigates to a different chat. It does not consider other actions that should logically dismiss the highlight.
What changes do you think we should make in order to solve the problem?
We should modify the highlightedBackgroundColorIfNeeded logic to include additional conditions for clearing the highlight. Specifically, the highlight should be cleared when:

The user clicks out of the chat into another chat.
The user clicks into another thread within the same chat.
The user refreshes the page.
The user sends a new comment in the same chat.
The user takes any action from the "+" menu while in the chat.

To achieve this, we can introduce a state variable (e.g., isThreadHighlighted) and a useEffect hook to manage the highlight state. The hook would listen for the relevant events (e.g., navigation changes, new comments, "+" menu actions) and update the isThreadHighlighted state accordingly. The highlightedBackgroundColorIfNeeded logic would then depend on this state variable to determine whether to apply the highlight.
What alternative solutions did you explore? (Optional)
One alternative solution could be to use a timer to automatically clear the highlight after a certain delay. However, this approach might not be ideal as it could lead to inconsistent behavior depending on the user's actions. Another alternative is to clear the highlight whenever the user interacts with any element outside the highlighted thread. This might be too broad and could unintentionally clear the highlight in some cases.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Highlight remains until you click out of the chat into another chat. It does not clear after any of the other cases mentioned above.
What is the root cause of that problem?
The report action is always highlighted if it's the linked action.
https://github.com/Expensify/App/blob/edd1719f62d74663d52f4168db2fd7adfe0cdda3/src/pages/home/report/ReportActionItem.tsx#L215
What changes do you think we should make in order to solve the problem?
I think we can do the same behavior in Slack that only highlights the link action in a few seconds so we don't need to take care what action we should clear the highlighted action
We can do this by using animated view here
https://github.com/Expensify/App/blob/edd1719f62d74663d52f4168db2fd7adfe0cdda3/src/pages/home/report/ReportActionItem.tsx#L951

Create a shared value

const bgColor = useSharedValue<string>(isReportActionLinked ? theme.messageHighlightBG : 'inherit');

Create a useEffect to change the value to inherit

useEffect(() => {
    if (!isReportActionLinked) {
        return;
    }

    bgColor.value = withTiming('inherit', {
        duration: 3000,
        easing: Easing.inOut(Easing.ease),
    })
}, [isReportActionLinked, theme.messageHighlightBG])

Create a useEffect to do the animation again if the page is changed from un-focused to focused

const isFocused = useIsFocused();
const prevIsFocused = usePrevious(isFocused);
useEffect(() => {
    if (!isFocused || prevIsFocused || !isReportActionLinked) {
        return;
    }

    bgColor.value = theme.messageHighlightBG;
    bgColor.value = withTiming('inherit', {
        duration: 3000,
        easing: Easing.inOut(Easing.ease),
    })
}, [isFocused, prevIsFocused, isReportActionLinked])

Change the style here to use animated style

const highlightedBackgroundColorIfNeeded = useAnimatedStyle(() =>
    StyleUtils.getBackgroundColorStyle(bgColor.value)
);

https://github.com/nkdengineer/App/blob/edd1719f62d74663d52f4168db2fd7adfe0cdda3/src/pages/home/report/ReportActionItem.tsx#L215

Use animated view here

<Animated.View style={highlightedBackgroundColorIfNeeded}>

https://github.com/Expensify/App/blob/edd1719f62d74663d52f4168db2fd7adfe0cdda3/src/pages/home/report/ReportActionItem.tsx#L951
What alternative solutions did you explore? (Optional)
NA
Result
https://github.com/Expensify/App/assets/161821005/203242b6-ba90-498d-b35f-697a96a75969

Hi @sobitneupane - Can you take a look at these proposals?

Thanks for the proposal @brunovjk

The hook would listen for the relevant events (e.g., navigation changes, new comments, "+" menu actions) and update the isThreadHighlighted state accordingly.

Can you please add more details about each events that hook would listen to and how will you handle it?

Thanks for the review @sobitneupane
Proposal
Updated
Added more detailed implementation steps for handling events that should clear the thread highlight

Thanks for the update @brunovjk
Proposal from @brunovjk looks good to me.
🎀 👀 🎀 C+ reviewed

Clicking out of the chat into another chat
Clicking into another thread
Refreshing the page
Sending a new comment in the same chat (but not receiving a comment from another user)
Taking any action from the + menu while in the chat

@sobitneupane I think we shouldn't removing the highlight of a message when we do these action because in addition to those, we have many other actions that should also remove the highlight like edit another comment, react, ... Instead, we should only have an animation for linked report action like Slack so we don't need to care what action we should clear the highlight. What do you think?
cc @yuwenmemon

I think we shouldn't removing the highlight of a message when we do these action because in addition to those, we have many other actions that should also remove the highlight like edit another comment, react, ... Instead, we should only have an animation for linked report action like Slack so we don't need to care what action we should clear the highlight.

That's a good point @nkdengineer. I kind of agree with you. @yuwenmemon What's your take on highlighting just for few seconds and fading it out?

I also think it makes sense, I even pointed out this solution in the "alternative solutions" section before

I also think it makes sense, I even pointed out this solution in the "alternative solutions" section before. We just need to test it carefully, but I think it will be good too.

What do you think about using both options together? It disappeared after a while and/or when the user performed some action.

Hmmm... I'd defer to @Expensify/design on this question.
The question is: Should the "yellow highlight" that highlights a linked message disappear a few seconds after showing?

I think we want to do what is outlined in the original comment:

The yellow highlight should fade after
Clicking out of the chat into another chat
Clicking into another thread
Refreshing the page
Sending a new comment in the same chat (but not receiving a comment from another user)
Taking any action from the + menu while in the chat

cc @sonialiap for confirmation though. I know this one was a long discussion, and I'm fairly certain we landed on the scenarios listed above.

The question is: Should the "yellow highlight" that highlights a linked message disappear a few seconds after showing?

This has come up several times in Slack and there's been quite a bit of discussion about it. We've consistently received push-back about the highlight fading after a set period of time (some people are for it, some people are against it), and I think the consensus was to rely on actions to dismiss it rather than time. This seemed to be the best compromise amongst the entire team, so I think we should continue to pursue that path.

Exactly ^ 🤣

Awesome! @yuwenmemon  Can I move forward with my proposal?

Yep

@yuwenmemon I can complete the task within a week. Is there a target deadline, is it needed sooner?
Thank you!

ETA 05/30

Not Overdue. Continue working in PR

Go away Melv - https://github.com/Expensify/App/issues/42165#issuecomment-2135829854

@lschurr, @yuwenmemon and @sobitneupane I found the task more challenging than expected. ETA might need to extend. Will keep working and raise the PR ASAP. Is this okay? Thank you.

All good

Update: Keep working on the PR, I will raise it ASAP

Any update @brunovjk?

Sorry for the delay, I had to focus on another issue but I'm back here, I just need to do a few tests and we're ready :D

Update: Keep working on the PR

Update: Keep working on the PR

Update: Same as above

Update: Same as above

Update: I got almost all cover:

[x] - Clicking out of the chat into another chat
[x] - Clicking into another thread
[ ] - Refreshing the page
[x] - Sending a new comment in the same chat (but not receiving a comment from another user)
[x] - Taking any action from the + menu while in the chat

I need to confirm the expected behavior for the highlight when refreshing the page. According to the previous PR that introduced the highlight for a linked report, a report item should be highlighted when a user sends a link (e.g.,<PHONE_NUMBER>56/r/465456456464) to another user. However, if the user refreshes the page using the same link, should the highlight still be removed?

However, if the user refreshes the page using the same link, should the highlight still be removed?

I believe this is the desired behavior—remove the highlight if you refresh the page. I'll let others chime in too to make sure I've followed the conversation correctly.

Melvin PR is almost ready

Update: Still need to safely implement the dismissal when the user refreshes the page, without crashing the new 'Comment Linking' feature.

I'm stuck on clearing the highlight comment (linkedReportActionID) during a page refresh. We need to call resetLinkedReportActionID() (created in our PR) only on page refresh, not on deep link navigation or any other way.
@sobitneupane any suggestions on differentiating between a page refresh and accessing via a link? Thanks!

Update: Testing on multiple platforms.
Next: Record videos and raise PR.

Update: Same as above

Update: Continue working on PR.

Update: Continue working on PR. Catching possible regressions.

Update: Same as above

Update: The feat to reset the yellow highlight when refreshing a page is taking a little more work than expected, but going well, get back with the PR to review ASAP

Update: Same as above

Update: Continue working on PR. Catching possible regressions.

@shawnborton @yuwenmemon Should we remove the yellow highlight as soon as we begin taking any action (after clicking Submit expense from the + menu), or after the action is complete?

Probably as soon as we begin taking any action? Curious what @Expensify/design and @yuwenmemon thinks though.

Probably as soon as we begin taking any action?

Makes sense to me.

Agree with the designers

Update: PR in review

Update: Continue working on PR. Catching possible regressions.

Same as above.

Hi team @yuwenmemon @sobitneupane,
I've been working on the yellow highlight behavior update and initially pursued a more robust solution by resetting linkedReportActionID using Navigation.setParams({reportActionID: ''}), since the highlight item depends on the linkedReportActionID.
This successfully removes the highlight but causes unwanted scrolling when the highlighted message is above the chat's end. After several attempts, including different strategies, I reverted to the original proposal of managing the highlight via state. However, this approach introduced regressions, especially with refreshing the page. I faced significant challenges in detecting page refreshes, specifically on web, and implementing a reliable system to clear the highlight without affecting other functionalities.
Despite daily efforts over months, I couldn't find a solution that avoids breaking the current behavior. Hence, I request to be unassigned from this task, as I believe someone else might have better success. Is there a problem?
I sincerely apologize for any delays and remain available to assist during the transition until this feature works flawlessly.
Thank you for your understanding.

Yes, no worries at all @brunovjk. Let's reopen.

Thank you all for your understanding.

Thanks for your work so far, @ishpaul777. I'm open to the idea and look forward to seeing your proposal.

Is the expected behavior only when exiting a thread?
What if the user deeplink from report action copy link? should it behave the same?

What's next for this one @sobitneupane? Are we waiting on a proposal from @ishpaul777?

Proposal
Please re-state the problem that we are trying to solve in this issue.
Update the yellow thread highlight behavior upon exiting a thread.
What is the root cause of that problem?
New feature
What changes do you think we should make in order to solve the problem?
Since there should be specific logic when re-load we'll need to store highlighted action in persist storage, e.g. onyx.
Introduce new onyx key ACTION_HIGHLIGHT

type ActionHighlight = {
    reportActionID: string;
    isHighlighted: boolean;
    isVisited: boolean;
};

   [ONYXKEYS.ACTION_HIGHLIGHT]: ActionHighlight;

Then we would need function to dismiss and reset the highlight, it can be in ReportActionUtils or a separate file:
dismiss and reset function

function dismissHighlight(shouldUseSet = false)
{
    if(shouldUseSet)
    {
        if (!currentActionHighlight || !currentActionHighlight.reportActionID || !currentActionHighlight.isHighlighted)
        {
            return;
        }
        Onyx.set(ONYXKEYS.ACTION_HIGHLIGHT, {reportActionID: currentActionHighlight?.reportActionID, isVisited: currentActionHighlight?.isVisited, isHighlighted: false});
        return;
    }
    // use merge by default to avoid unnecessary render in useEffect
    Onyx.merge(ONYXKEYS.ACTION_HIGHLIGHT, {isHighlighted: false});
}

function resetHighlight()
{
    Onyx.set(ONYXKEYS.ACTION_HIGHLIGHT, null);
}

Then we have to modify highlight logic:

isFocused is used for re-highlight the action when we navigate to parent report thread
actionHighlight.isVisited is to track if the highlight already visited and we'll dismiss highlight if we reload the page and isVisited is true.
actionHighlight.reportActionID is to know which report action is currently active

Modify ReportAction logic useEffect

    const isFocused = useIsFocused();

    useEffect(() => {
        if (!linkedReportActionID || linkedReportActionID === '-1') {
            console.log("[wildebug] linkedReportActionID is not valid:", linkedReportActionID);
            return;
        }

        if (!action.reportActionID || !isFocused) {
            console.log("[wildebug] action.reportActionID or isFocused is not valid");
            return;
        }

        const isReportActionLinked = linkedReportActionID === action.reportActionID;
        console.log("[wildebug] isReportActionLinked:", isReportActionLinked);

        if (!isReportActionLinked) {
            console.log("[wildebug] isReportActionLinked is false");
            return;
        }

        if (actionHighlight?.reportActionID === action.reportActionID) {
            console.log("[wildebug] actionHighlight matches action.reportActionID");
            if (actionHighlight.isVisited) {
                console.log("[wildebug] actionHighlight isVisited is true, setting isHighlighted to false");
                Onyx.merge(ONYXKEYS.ACTION_HIGHLIGHT, { isHighlighted: false });
            } else {
                console.log("[wildebug] actionHighlight isVisited is false, setting isVisited to true");
                Onyx.merge(ONYXKEYS.ACTION_HIGHLIGHT, { isVisited: true });
            }
            return;
        }

        console.log("[wildebug] Initializing or replacing actionHighlight");
        Onyx.merge(ONYXKEYS.ACTION_HIGHLIGHT, { reportActionID: action.reportActionID, isHighlighted: true, isVisited: false });
    }, [actionHighlight?.reportActionID, linkedReportActionID, action.reportActionID, isFocused]);

Then Modify highlightedBackgroundColorIfNeeded logic to use our persisted highlight as main source of value to determine the highlighting.
Modify highlightedBackgroundColorIfNeeded logic

    const isReportActionLinked = linkedReportActionID && action.reportActionID && linkedReportActionID === action.reportActionID;
    const highlightedBackgroundColorIfNeeded = useMemo(() => {
        const isHighlighted = actionHighlight?.reportActionID && action.reportActionID && actionHighlight?.isHighlighted && actionHighlight.reportActionID === action.reportActionID;
        if ((isHighlighted) {
            return StyleUtils.getBackgroundColorStyle(theme.messageHighlightBG);
        }
        return {};
    }, [StyleUtils, actionHighlight?.reportActionID, actionHighlight?.isHighlighted, action.reportActionID, theme.messageHighlightBG]);

To avoid highlight flickering when highlighting last report action of a chat we can listen beforeunload to dismiss highlight before we reload. We should use Onyx.set instead of merge since the page reloading can cancel Onyx.merge
Resolve highlight flickering
ReportActionsList.tsx

    useEffect(() => {
        if(linkedReportActionID === '-1')
        {
            // no report action id defined in param, reset highlight
            ReportActionsUtils.resetHighlight();
            return;
        }

        const dismissActionHighlight = () => {
            ReportActionsUtils.dismissHighlight(true);
        };

        window.addEventListener('beforeunload', dismissActionHighlight);

        return () => {
            window.removeEventListener('beforeunload', dismissActionHighlight);
        };
    }, [linkedReportActionID]);

Optional
If linking from Copy Link context menu should behave differently (e.g. it should persist highlight and not affected by user action dismissal), then we can distinguish the behavior by using query string. We can use existing referrer param or use separate param to determine highlight-mode
CONST.TS

    REFERRER: {
        NOTIFICATION: 'notification',
        COPY_LINK: 'copy-link',  < --- add new referer
    },

Then we can modify the highlight logic to not affected by the user action

    const isFromCopyLink = route?.params?.referrer === CONST.REFERRER.COPY_LINK;
    const highlightedBackgroundColorIfNeeded = useMemo(() => {
        const isHighlighted = actionHighlight?.reportActionID && action.reportActionID && actionHighlight?.isHighlighted && actionHighlight.reportActionID === action.reportActionID;
        if ((isHighlighted || (isReportActionLinked && isFromCopyLink))) {
            return StyleUtils.getBackgroundColorStyle(theme.messageHighlightBG);
        }
        return {};
    }, [StyleUtils, actionHighlight?.reportActionID, actionHighlight?.isHighlighted, action.reportActionID, theme.messageHighlightBG]);

Summary

Clicking out of the chat into another chat (Call dimsissHighlight in ReportActionItem onPress)
Clicking into another thread (Call resetHighlight when navigating to parent report)
Refreshing the page (Utilize isVisited to determine isHighlighted, dismiss the highlight beforeunload)
Sending a new comment in the same chat  (Call dimsissHighlight in Composer submission events)
Taking any action from the + menu while in the chat (Call dimsissHighlight in menu onSelect)

Branch for this solution
What alternative solutions did you explore? (Optional)
N/A

Could you review @sobitneupane?

@wildan-m @ishpaul777 Could you please confirm that the highlight gets removed on taking all of the following actions?

Clicking out of the chat into another chat
Clicking into another thread
Refreshing the page
Sending a new comment in the same chat (but not receiving a comment from another user)
Taking any action from the + menu while in the chat

Thanks for your proposal @wildan-m  I think persistent storage might not be ideal since the highlight is temporary and should reset when users switch chats, refresh, or take these actions. We only want this behavior for the current session.

I can confirm that my solution can resolve all cases, but not implemented all action in the POC, especially the last requirement
without persistent storage how can we distinguish when user re-visit the same report linking or a different one, AFAIK global var / context will reset on each browser refresh

@ishpaul777 We would like the highlight to be removed in each of the cases mentioned here.

Honestly I think if we've taken care of every other case, I don't mind leaving the highlight on refresh. It's the least weird / lowest priority IMO.

Thanks @dannymcclain for chiming in.

Honestly I think if we've taken care of every other case, I don't mind leaving the highlight on refresh. It's the least weird / lowest priority IMO.

@brunovjk I believe your PR handled every other case except refreshing. I believe we can proceed with your PR in that case. Can you please confirm?

Hi @sobitneupane, thanks for asking! And yes! In our PR, we address:

Sending a new comment in the same chat (but not receiving a comment from another user)
Taking any action from the + menu while in the chat

Note: Clicking out of the chat into another chat and Clicking into another thread already dismiss/update the highlight, since in both scenarios we update the linkedReportActionID.
My solution offers simplicity and component-level control. However @ishpaul777's proposal could be more robust in a global context, but it might be slightly outside the original issue’s scope. I’m happy to move forward with my solution if it fits the issue’s needs, but if you decide to go with @ishpaul777's proposal I also think it's a good idea and I’m available to help with testing and implementation, regardless of compensation.

Thanks @brunovjk . I appreciate your understanding.
Proposal from @ishpaul777 looks good to me.  I believe it will make the thread highlight behavior more consistent and intuitive.
cc: @yuwenmemon

Relabelling as Lauren is OOO now.

PR in review.


didBegin didEnd methods do not need enumeration

The scene callbacks already have the node that is involved so instead of enumerating all element node attachments just look up the ones needed.

This has been resolved. The related elements are now looked up directly.


fix: live preview: added variant support in data cslp tag function

fix: live preview: added variant support in data cslp tag function

@contentstackMridul Reverting the PR as its hampering the current release. The branch will be kept intact. Will merge this again later
cc: @KANE-99 @cs-raj @netrajpatel


Add This type to lib

Sync TSJS repo with master.
Address https://github.com/Microsoft/TypeScript/issues/9374

:+1:


Release 3.1

Fixes #

 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.:x: seshapu sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


[feat] Check subtrees in CI

Originally posted by @daank-c in https://github.com/dogecoin/dogecoin/pull/3525#discussion_r1623689225:
do you think it would be useful/appropriate to add subtree checks to the linter?

We could definitely add a check process that ensures the integrity of the subtrees, perhaps in a similar way to the scripted commit checker.

I tested the commit script checker and it runs fine with no issues.
I also ran the subtree checker on all subtree directories I could find (by examining the Git log). It reports multiple pending updates:
+ bash contrib/devtools/git-subtree-check.sh src/univalue
ERROR: subtree commit a44caf65fe55b9dd8ddb08f04c0f70409efd53b3 unavailable. Fetch/update the subtree repository
+ bash contrib/devtools/git-subtree-check.sh src/secp256k1
ERROR: subtree commit 44c2452fd387f7ca604ab42d73746e7d3a44d8a2 unavailable. Fetch/update the subtree repository
+ bash contrib/devtools/git-subtree-check.sh src/crypto/ctaes
ERROR: subtree commit 003a4acfc273932ab8c2e276cde3b4f3541012dd unavailable. Fetch/update the subtree repository
+ bash contrib/devtools/git-subtree-check.sh src/leveldb
ERROR: subtree commit a31c8aa408d5594830f7cb20ead1ef1dff51b79e unavailable. Fetch/update the subtree repository
+ bash contrib/devtools/git-subtree-check.sh qa/rpc-tests/python-bitcoinrpc
ERROR: subtree commit e48474322de165212c5aa29574297e78b09df8d6 unavailable. Fetch/update the subtree repository

Does that look expected?


Rework algorithm to place additional elements

While the previous algorithm was better in saturating articles with additional elements it did not perform well when placing a single additional element into an article, this was always placed at the end.
Considering the known use-cases of this algorithm (placing ads or CTA forms within an article), it seems more common that single elements are placed than that the article is being saturated with additional elements.
For this, also add #length methods on Text and Paragraph elements: Similar to the String#length method these methods return the number of characters contained by the element. For compatibility with the String methods, also add the alias #size.
Outline of the new algorithm:

It calculates every how many characters (only considering paragraphs) an additional element should be inserted, given the number of additional elements provided
It then iterates over the existing elements within the article, inserting them into the resulting, merged article
As soon as it has passed the right amount characters, it checks if the current position is eligible for inserting an additional element (the previous element needs to be a paragraph and the following needs to be either a paragraph or a headline)
a. If so, it inserts the additional element and recalculates in how many characters the next element should be inserted, based on the remaining number of characters and remaining additional elements
b. If not, it keeps iterating over the article until it finds an eligible position and only then inserts the additional element and recalculates the number of characters until the next insertion
If there are still additional elements remaining which couldn't be inserted (due to not finding enough eligible positions), they are appended to the end of the article

+1, Nice!

Thanks for the reviews 🎉


Out of tree soc sample

Is your enhancement proposal related to a problem? Please describe.
I have been unable to pull a riscv-privilege soc out of tree. Placing it in an application/soc/riscv/riscv-privilege folder leads to the Kconfigs seemingly not being discovered. Placing it in an application/soc/riscv folder results in unresolved dependencies.
Describe the solution you'd like
I would love to see a sample of an out of tree soc, particularly that is part of an soc family.
Describe alternatives you've considered
I've tried getting this working on my own without success. I haven't been able to find examples of others doing so successfully. I've tried this with my own custom soc that works in tree as well as with the sifive-freedom soc as a sanity check (and to attempt to show what an unsuccessful example looks like).
Additional context
I'm happy to provide any additional context that would be helpful. My end goal is just to get this working but I think a sample would more useful to others going forward.

I'm happy to provide any additional context that would be helpful.

A non-working example repo along with error messages would be good.

Did you find a solution for out-of-tree soc porting?


setCookie express middleware only works inside hooks

Apparently, the middleware checks if it's being called by a hook though I think that it should be possible to pass it to app.get() too.
Example: https://gist.github.com/HenriBeck/fa1fa401b8193da765e207fa4e4ca750
https://github.com/feathersjs/feathers-authentication/blob/master/src/express/set-cookie.js#L27
This block is never run because res.hook is undefined and no cookie is set.

Can you help clarify what the goal is in this issue?  We want cookies to be ignored by the API server.  It can set cookies, because that helps with SSR, but we don't want a cookie to get consumed directly by any Feathers service.  That opens up the service to CSRF attacks.

This line does nothing. It doesn't set the cookie as you would expect.
That's because in the middleware it checks if it has been called after a service method.

Closing as it's the same as #389


Empty video

All final videos weigh 1kb. I waited 30 minutes for video generation, while observing intermediate results. In the end I have nothing but the inscription "Done".
No errors in the console. Version 0.8 is the latest.
Maybe I should export to jpg pictures just in case? And generate the video with a separate button? Maybe you can get some useful tricks from Deforum? (https://github.com/deforum-art/deforum-stable-diffusion)

Same issue on 0.8 version

The same issue，after 20-30minutes,I only got the 258 Bytes files.........it is "Bytes",not "MBytes"

@PandaBalls Images are not saved for now, only the videos. I'm gonna add a special option in extension settings in the next version that will allow me to save images. Regarding the issue with video, I have a rough guess that it might be related to how cv2.VideoWriter works, but I'm not sure how to fix it yet.

@volotat
Well. The problem remains on version 0.9. The checkbox "Save frames into a folder nearby a video (check it before running the generation if you also want to save frames separately)" is on. No video, no images either. Although, when working, all frames are shown in preview.


investigate how to migrate the repo from ES5 to Typescript

investigate how to migrate the repo from ES5 to Typescript

Migration plan from ES5 to TS:

Install TypeScript and Type Definitions
Create and configure tsconfig.json file
Port existing JS rules to use the new typescript packages and add type annotations wherever required.
Port existing test to use the new typescript packages
Check and update package.json file to ensure dependencies are managed properly.

[like]  Aubrey Quinn reacted to your message:

From: Ajay Kumar Agarwal @.>
Sent: Monday, September 16, 2024 10:44:14 AM
To: microsoft/eslint-plugin-fluentui-jsx-a11y @.>
Cc: Author @.***>
Subject: Re: [microsoft/eslint-plugin-fluentui-jsx-a11y] investigate how to migrate the repo from ES5 to Typescript (Issue #86)
Migration plan from ES5 to TS:

Install TypeScript and Type Definitions
Create and configure tsconfig.json file
Port existing JS rules to use the new typescript packages and add type annotations wherever required.
Port existing test to use the new typescript packages
Check and update package.json file to ensure dependencies are managed properly.

—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/eslint-plugin-fluentui-jsx-a11y/issues/86#issuecomment-2352570592 or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATJUBIKPFU6DRHFR5XH4VCTZW2Y77BFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVE2TSNZQGAZDSNZRQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRSGUZDKMJVGYZTON5HORZGSZ3HMVZKMY3SMVQXIZI.
You are receiving this email because you authored the thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


chore: Removes react-bootstrap and react-bootstrap-slider

SUMMARY
Removes react-bootstrap and react-bootstrap-slider.
🥳 🕺🏼 ✅ 💯 🎊 💪🏼 💥 🔥
@junlincc @rusackas
Depends on: https://github.com/apache-superset/superset-ui/pull/1108
TEST PLAN
1 - Execute all tests
2 - All tests should pass
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thank you so much for completing 💯  the project(all the dangerous work), your work is much appreciated. @michael-s-molina 🙏
cc @mistercrunch


Support arm64 runners

if you're looking for a cheap arm64 runner, use build jet.
runner is buildjet-2vcpu-ubuntu-2204-arm
It's brand new and not even on their website yet... but the pricing is stupidly cheap and it runs very fast.


Training failed how to restart training?

So training failed, how to re-train it due to a server error or something?
autonlp is latest version, python is 3.9
gorkemgoknar@Gorkem-MacBook-Pro:~/Desktop/autonlptest$ autonlp project_info --name sentiment_turkish
2021-03-11 16:35:20.687 | INFO     | autonlp.cli.project_info:run:28 - Fetching info for project: sentiment_turkish
2021-03-11 16:35:20.688 | INFO     | autonlp.autonlp:_login_from_conf:66 - 🗝 Retrieving credentials from config...
2021-03-11 16:35:20.688 | INFO     | autonlp.autonlp:get_project:109 - ☁ Retrieving project 'sentiment_turkish' from AutoNLP...
2021-03-11 16:35:21.205 | INFO     | autonlp.project:refresh:195 - 🔄 Refreshing uploaded files information...
2021-03-11 16:35:21.700 | INFO     | autonlp.project:refresh:200 - 🔄 Refreshing models information...
2021-03-11 16:35:22.206 | INFO     | autonlp.autonlp:get_project:121 - ✅ Successfully loaded project: 'sentiment_turkish'!
AutoNLP Project (id # 29)
 • Name:        sentiment_turkish
 • Owner:       gorkemgoknar
 • Status:      ❌ Failed to download data files from the huggingface hub
 • Task:        Binary Classification
 • Created at:  2021-03-11 12:51 Z
 • Last update: 2021-03-11 13:34 Z

~~~~~~~~~~~~~~ Files ~~~~~~~~~~~~~~

Dataset ID:
gorkemgoknar/autonlp-data-sentiment_turkish

📁 turkish_movie_train.csv (id # 27)
   • Split:             train
   • Processing status: ❌ Failed: server error
   • Last update:       2021-03-11 13:20 Z
📁 turkish_movie_valid.csv (id # 28)
   • Split:             valid
   • Processing status: ❓ Unhandled status! Please update autonlp
   • Last update:       2021-03-11 13:32 Z

~~~~~~~~~~~~ Models ~~~~~~~~~~~

🤷 No train jobs started yet!

Hello @gorkemgoknar !
It looks like you're not running the latest version of AutoNLP. You can run: pip install --upgrade autonlp to update!
You selected a binary_classification task for your project, but the validation file you've uploaded contains more 2 categories: as updating the task of a project is not supported in AutoNLP yet, you will need to create another project and select multi_class_classification as task to proceed.
Plus, we are having trouble decoding your files. Could you run this before uploading the files, please?
import pandas as pd
pd.read_csv("turkish_movie_train.csv").to_csv("turkish_movie_train.csv")
pd.read_csv("turkish_movie_valid.csv").to_csv("turkish_movie_valid.csv")

Thanks @SBrandeis.
I will retry after cleaning up csv file.
My installed autonlp is 0.1.9 (which I can found out by upgrading it), maybe you should add version info to package version also while upgrading.

autonlp.version
'0.0.1'

For information file format is like this (language is Turkish plain-text, characters are iso-8859-9 encoded usually)
comment,film_name,point
"     Jean Reno denince zaten leon filmi gelir akla izlemeyen kalmamıştır ama kaldıysada ee ne duruyorsun hemen izle :)
",Sevginin Gücü,"5,0"\n
Classes are 1 to 5 so it is better to make it as a regression problem maybe, or do a preprocessing to make it <2.5 0 and upper 1.
I wanted to directly test it out with a common dataset but it seems we need to do additional preprocessing for training and validation dat and take care of non-ascii character texts.
Prediction is as good as its data anyway..

I have a same issue. Could you help me for this ?

Hi @halilyesilceng!
Please make sure your files can be loaded with the datasets library:
from datasets import load_dataset
dset = load_dataset("json", data_files=["/path/to/your/file.json"], split="train")

Let me know if this helps!


proc/native/linux: ignore ENODEV when retrieving fp registers

proc/native/linux: ignore ENODEV when retrieving fp registers

Either the CPU or the kernel may not support the calls we do when
retrieving floating point registers, this isn't an error we should
propagate.
Also improve the error reporint of pkg/proc/native.fpRegisters.

Fixes #1022

I think we're running into https://github.com/mozilla/rr/issues/1282.
@mtds can you cat /proc/cpuinfo on the affected machine? I suspect the hardware does not support xsave as in the RR issue linked above. If so, we can implement a similar workaround and not use PT_GETREGSET.

@aarzilli I'm fine with that. I'm still going to open an issue to track if we ever want to implement the workaround.

@derekparker: I am not sure if it still useful but the following is a printout of cat /proc/cpuinfo
on my Linux workstation (8 cores). As you suspected, xsave is not supported (at least is
nowhere to be found on the flags line).
My second attempt was on a Linux VM (on KVM) where the CPU is marked as Intel and the model
is as follows: QEMU Virtual CPU version 2.5+. Also in this case, xsave does not appear in
the flags line of cpuinfo. Anyway, this pull request from @aarzilli solved the problem in both
cases.
processor       : 0
vendor_id       : GenuineIntel
cpu family      : 6
model           : 26
model name      : Intel(R) Xeon(R) CPU           E5520  @ 2.27GHz
stepping        : 5
microcode       : 0x11
cpu MHz         : 1596.000
cache size      : 8192 KB
physical id     : 0
siblings        : 8
core id         : 0
cpu cores       : 4
apicid          : 0
initial apicid  : 0
fpu             : yes
fpu_exception   : yes
cpuid level     : 11
wp              : yes
flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx
rdtscp lm constant_tsc arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc aperfmperf eagerfpu pni dtes64 monitor ds_cpl vmx est tm2 ssse3 cx16
 xtpr pdcm dca sse4_1 sse4_2 popcnt lahf_lm tpr_shadow vnmi flexpriority ept vpid dtherm ida
bugs            :
bogomips        : 4533.46
clflush size    : 64
cache_alignment : 64
address sizes   : 40 bits physical, 48 bits virtual
power management:


feat:docs for bits

Added Documentation for the Bit mainpulation

Will you merge this pull request or wanted any other modification changes to this file.


can we use plain class names with .scss

for external .scss files i am using the below syntax
import styles from './main.scss'
....
render {
 return (

) }
it will assign  a class name like src-layouts-component-index--myClassName--1K6rL
instead of this is there any way to import .scss globally so that we can write classnames normally
like below
import ./main.scss
....
<div className="myClassName nav nav-default"

You have to tweak your webpack config for this. This is more a generic webpack problem. And I think default boilerplate have some comments on how to handle this. You can alternatively use *.global.css files to do that. Just read the webpack config full of comments please.


[SPARK-45077][UI] Upgrade dagre-d3.js from 0.4.3 to 0.6.4

What changes were proposed in this pull request?

This PR upgrades dagre-d3.js from 0.4.3 to 0.6.4, a.k.a. changes the dist build by https://github.com/andrewor14/dagre-d3 to https://github.com/yaooqinn/dagre-d3/releases/tag/v0.6.4-patch
The full list of commits can be found at:
https://github.com/yaooqinn/dagre-d3/compare/v0.6.4...v0.6.4-patch
BTW, I have tried to use the official release of 0.6.4 and applied the changes we want in a post-hoc way, but it didn't work well.
Why are the changes needed?

0.4.3 is pretty old
Does this PR introduce any user-facing change?

no
How was this patch tested?

Tested with the case bellow and verify the SVG objects on both SQL and stage pages
spark-sql (default)> select avg(id) from range(1E10) group by id % 5;

Was this patch authored or co-authored using generative AI tooling?

no

Are you saying 0.6.4 doesn't work well? is this just for your testing then and not to merge?

cc @gengliangwang @jasonli-db

Are you saying 0.6.4 doesn't work well? is this just for your testing then and not to merge?

Hi @srowen, I apologize for any confusion caused. This pull request is for merging purposes not testing. Neither version 0.6.4 nor 0.4.3 are functional, except for the custom changes applied.

also cc @sarutak @HyukjinKwon and @dongjoon-hyun

I'm concerned about this, what does it mean?
BTW, I have tried to use the official release of 0.6.4 and applied the changes we want in a post-hoc way, but it didn't work well.

@srowen We need to add some custom features during the creation cluster/node/label of DAG

Hi @srowen, I refined the PR description and removed the worrisome comments from my POC phase

Merged to master


Transfer jit-open/CsvToDmnConverter into the Camunda Community Hub

As described in the instructions on https://github.com/camunda-community-hub/community I am creating this issue to transfer the repository to the Camunda Community Hub.

Hi @MiguelitoEnrique! Thank you so much for transferring this repository to the Camunda Community Hub. If you would be so kind as to review the extension lifecycle documentation and determine where this extension is in its current lifecycle, that would be great!
Once you've done that, you can feel free to add the corresponding Shields.io badge and a Community Extension badge to the extension's README. If you have any questions, please let me know. I am happy to help!


Calling the whole module with 'this' does not work with browserify.

I have a problem using Split.js with my browserify-based project.
Browserify packages every module into a function call, so when Split.js is imported, the .call(this) at the end of the module won't refer anymore to the global window object, but it will be undefined.
Do you have a suggestion to this problem?

Ah, I see. I'll look into this.

Thanks!

I just passed temporarily 'window' instead of this, but maybe it would limit the possible usage environments.

So that last line would be }).call(window);?
Split.js isn't useful outside of the browser, so I'm fine with that.

Yes, that would solve the problem, thanks!


Added reference to Arxiv for porous media article

Description
The recent article on CFD in porous media is now available on Arxiv. This PR adds the link in the referencing section of the documentation.

Miscellaneous (will be removed when merged)

Checklist (will be removed when merged)
See this page for more information about the pull request process.
Code related list:

[x] All in-code documentation related to this PR is up to date (Doxygen format)
[x] Lethe documentation is up to date
[x] The branch is rebased onto master
[x] Code is indented with indent-all and .prm files (examples and tests) with prm-indent

Pull request related list:

[x] Labels are applied
[x] There are at least 2 reviewers (or 1 if small feature) excluding the responsible for the merge
[x] If this PR closes an issue or is related to a project, it is linked in the "Projects" or "Development" section
[x] If any future works is planed, an issue is opened
[ ] The PR description is cleaned and ready for merge

Can you also add it in the publication list under year 2024


update<EMAIL_ADDRESS>

Description of the change
This updates apm to 2.5.2-atomic.1
Verification
The ci passes
Drawbacks
none
Release Notes

This updates apm to 2.5.2-atomic.1

Sweet! Everything passed in the first try.

I don't see Linux failures after merging this anymore.

That's great news. Just hope it keeps up... hopefully it will. 🤞


tooltip delay

Currently tooltips always display instantly which can be quite annoying in some UI contexts where you'd only like the tooltip to appear on a lingering hover.
The MD standard doesn't seem to mention delay at all (nothing for or against). It seems common to have a default, but configurable delay of 500ms in other tooltip implementations (polymer uses 500ms by default for example), and it would be great if MDL added a configurable delay.

@jestelle @shyndman Are you aware of a sensible set of defaults to prescribe for tooltip delays? If not, we might ask Zach if there's guidance available but atm it seems other implementations have arbitrarily gone for what they think makes sense.

I'd rather we not do a "configurable" delay, but either introduce a set delay or keep it as-is. Having tooltips display in 500ms in one area but after 250ms in another, and instant in another... Gets very confusing in terms of user expectations.

For my personal use case the issue is not that I'd want to have 500ms in one place, and 250ms in another, but I'd like to have 500ms in some places, but would happy to have the current 0 delay in others. 0 delays are nice on low volume action sites, but can get very annoying if the tooltip target site is one used very often, but may still need delayed hinting for new users to get hints.
I figured making it configurable would achieve that, but I can also that the current strategy to prefer to use CSS classes to describe how things should behave instead of JS api would be easier if a hard coded 500ms versus 0ms was used, given it is something that is easy to define using classes, with two classes, one for the delayed case, and another for the non-delayed.

I'd rather we just provide one tooltip. Introducing another setup for delay vs no delay means needing to also explain when to use one vs the other to everyday developers. That is difficult and will lead to many "X situation" questions getting asked. Also, keep the provided components lite and  simple. In my opinion, we get with the MD team to see if there should be a delay. Then, follow their guidance with what we provide and if you need something else then handle it in your application.

@Garbee how did it go, did you get from the MD team their opinion about delay/no delay?

I don't talk with the spec team. Things get tagged then the Googler's will go through the guidance needed tags and any new ones they send internal emails for. From there, it can take a long time for things to come out. The MD team has numerous other teams asking questions and trying to get things clarified. It takes some time for things to be worked through and specified accordingly.
As information comes back, the issues will get updated.


Add a just docs command

Consider adding a just command to build the docs. Can check in the run_task script for how we do it or base it off of:
RUSTDOCFLAGS="--cfg docsrs" cargo +nightly rustdoc --all-features -- -D rustdoc::broken-intra-doc-links

(Will likely want to use the nightly config file like the rest of the commands though.)

concept ACK but maybe it should be called just docsrs to emphasize that it's creating the docs as they appear on docsrs. Otherwise it is hard to distinguish from cargo doc which is just as easy to type and doesn't require nightly.

Can just pass in the arguments? Being able to type just docsrs --open would be nice.


Create wpsstudent.txt

Domain for students at Winchester High School pull request.
website: http://www.winchesterps.org/
email domain: @wpsstudent.com

Which domain was registered? @wpsstudent.com is currently not working still (unsure if edited correctly, apologies)

@Mobkinz78 I am sorry, it seems to be messed up with all the typos. There is a file lib/domains/org/wpsstudent.txt in this request. It represents a domain wpsstudent.org. If the actual domain is wpsstudent.com, this file must be renamed to lib/domains/com/wpsstudent.txt. You can make a new request to delete lib/domains/org/wpsstudent.txt and add lib/domains/com/wpsstudent.txt.

Oh I see, that was fairly silly of me. Sorry about that. Thanks you.


src: reformat codes to fix ci

This PR fixes failures at lint stage by reformatting several codes under src.
We found the issue in https://github.com/oopt-goldstone/goldstone-mgmt/pull/74.

@KazuyaAnazawa Thank you.


Wrapping text

This is more of a question than an issue. Is it possible to get the text to wrap within the size of the input?
Pretty new to Framer and Cofeescript, but if not available, I may take a try at it.
Thanks.

Hey @KingWalrus an input is not designed for this purpose. If you want to have the text wrap on it, you should use a textarea (and this module currently doesn't support textarea).
But you can "hack" a little bit your input to wrap the text around it. As I said is not very good to do as the input was not created for that purpose, but if you like to hack it, here is the code :)
{Input} = require "input"

someInput = new Input
	setup: true
	height: 300
someInput.style =
	"word-wrap": "break-word"
	"word-break": "break-word"

If you feel adventurous and you want to adapt the code of this module to allow textareas too, go ahead and make a pull request, I'll be happy to review it and merge it :)

Awesome. Thanks for getting back to me. I actually found another module that does text area. I might propose a PR that makes it available as a type of input here. We'll see if I have enough time.


IOException Error: The supplied user buffer is not valid for the requested operation.

Hi, I know it's be awhile this repo was updated and I don't know if anyone is still managing this, but I need help.
I followed the YT tutorial for this project but I keep getting this error on Unity.
`IOException: The supplied user buffer is not valid for the requested operation.
System.IO.Ports.WinSerialStream.ReportIOError (System.String optional_arg) (at <36d5d97f0e39429283d80156f1c7f1fc>:0)
System.IO.Ports.WinSerialStream.Read (System.Byte[] buffer, System.Int32 offset, System.Int32 count) (at <36d5d97f0e39429283d80156f1c7f1fc>:0)
System.IO.Ports.SerialPort.ReadExisting () (at <36d5d97f0e39429283d80156f1c7f1fc>:0)
(wrapper remoting-invoke-with-check) System.IO.Ports.SerialPort.ReadExisting()
Controller.DataThread () (at Assets/Scripts/Controller.cs:41)`
More Info:
Serial monitor is closed.
No application is using COM3 (Arduino UNO on my system is connected to COM3). I made sure to run only Unity.
Unity version - Unity 2022.3.12f1
.NET - .NET Framework
OS - Windows

Hi! Whilst you are correct the repo has not been updated in a while, I'm still here!  I am currently at work so I shall have a look a little later, thanks for supplying version numbers, can I confirm if it is Windows 10 or 11 you are currently using?

Windows 11.
Thank you for responding!


starlite 1.51.9

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Changes:

remove patch introduced in #43

@conda-forge-admin please rerender


[Mention The Feature]: Add a user login option

Is your feature request related to a problem? Please describe.
Adding a login feature will enhance the website. I want to add a user login icon in nav bar and add a functionality to it where user can login with his/her details. So please let me work on this issue.
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

Can you please assign me this task I would like to do it.

this issue has been solved, so we are closing this issue.


tests: openflow switched to use gremlin builder

just verify that openflow tests work as before

moved to tests-all-gremlin PR


[Feature Request] Is there a way to disable/delete Foreground Service in Android?

What is the need and use case of this feature?
My app release was rejected by the Play Store due to having FOREGROUND_SERVICE_MEDIA_PLAYBACK configured. However, my app only plays sounds while it is actively in use. Is there a way to remove or disable the FOREGROUND_SERVICE_MEDIA_PLAYBACK setup in this case? Your package has been very helpful, and I'd prefer not to switch to a different one.
I tried to remove the FOREGROUND_SERVICE_MEDIA_PLAYBACK but the app would be crashed.
Describe the ideal solution
Having the ability to disable FOREGROUND_SERVICE_MEDIA_PLAYBACK user permission.

nope. u should seek other alternatives like expo-audio if u dont need a foreground service.

Got it, thanks for confirming!


Added '+' button in product fragment, allowing addition of product…

Description

Added '+' button in product fragment, allowing the addition of the product to lists.
Replaced new Handler variable with global handler variable

Related issues and discussion
#fixes #2266
Screen-shots

@prashantkh19 can you hide it in the collaspsed view ? This is too much. Or perhaps somewhere else as a side bubble ?

@prashantkh19 can you hide it in the collaspsed view ? This is too much. Or perhaps somewhere else as a side bubble ?

Looks good, thanks @prashantkh19


No module named 'serverless_wsgi' on Azure

I get the following error when using serverless-wsgi on Azure.

Exception while executing function: Functions.app Result: Failure
Exception: ModuleNotFoundError: No module named 'serverless_wsgi'
Stack:   File "/azure-functions-host/workers/python/3.8/LINUX/X64/azure_functions_worker/dispatcher.py", line 238, in _handle__function_load_request
func = loader.load_function(
File "/azure-functions-host/workers/python/3.8/LINUX/X64/azure_functions_worker/loader.py", line 66, in load_function
mod = importlib.import_module(fullmodname)
File "/usr/local/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "/home/site/wwwroot/wsgi_handler.py", line 22, in
import serverless_wsgi

When reading up on the python developer reference here, you can see that referring to a local module only works with explicit relative or absolute paths, but not implicit relative.
A fix might be to change line 22 of wsgi_handler.py to
from . import serverless_wsgi
I'm using serverless_wsgi version 1.7.5.
Note that running serverless locally with 'sls wsgi serve' works like a charm :)

Hi @MattiasDC,
Yeah, I've not tested on Azure personally - on AWS, the package root is PYTHONPATH, so it's not importing from a relative path and there's no issue.
Did you test that from . import serverless_wsgi works on Azure? And I assume you had to add __init__.py to the package root as well?
In that case, I suppose we catch the exception when importing from PYTHONPATH and fall back to the relative import. Another option would be to check whether PYTHONPATH is where wsgi_handler.py is and whether there's an __init__.py (assuming it's required), and import accordingly. I don't think we should add __init__.py automatically, but we should probably emit a warning if it's required on Azure and is not present in the project.

Hi,
Putting the following code before the import fixes that problem (and the importing of the module containing the app).
root = os.path.abspath(os.path.dirname(__file__)) sys.path.insert(0, root)
There seem to be some other issues though, it seems Azure expects other function parameters.
An example Azure Function with WSGI can be found here.

Hi @logandk,
I created a fork and got it working on Azure. In a nutshell:

serverless_wsgi.py may be removed as azure provides a library to do this
Re-implement wsgi_handler.handler.

If you were to add support for Azure, feel free to use my changes.
Make sure to let me know in that case.


Improve reliability and build process.

So, big release ahead. I have the time to do some gardening over issues and pull requests. Merged the one that looked solid, checked out from there and solved 3 or 4 issues plus a list of improvements:

Element.scroll and window.scroll now support all possible object and arguments combination plus errors and fallbacks matching current Firefox implementation
Element.scroll and Element.scrollBy support for strings
Improved Element.scroll and Element.scrollBy examples in the page
Added __forceSmoothScrollPolyfill__ global so both module and sites including the polyfill on script tag can override native implementation, something that was asked a lot
Added __smoothScrollPolyfillSpeed__ global to modify scrolling speed (though it's recommended it not to on the README)
Switch to explicit conditions whenever possible to improve performance a little bit
Minifed version
CI builds now will check for linting and minifying to prevent unnecessary back and forths in future PRs
Build is now handle by rollup 🎉
ESLint version and rules updated

@iamdustan please, take a look as soon as you can, I have some free time this week and would be awesome to merge it and then go over PRs and Issues that could be closed. Also, it would be good to make the gh-pages point to master so the polyfill site gets properly updated without extra merges needed.
@GioSensation and @robertmorgan, feel free to take a look at the src file to make sure your code hasn't been compromised, it would also be lovely if you could try the dist version on this PR to see if you can spot bugs.
Thanks.

@jeremenichelli Looks solid to me. I have just pointed out a tiny tiny typo in a comment, so rather irrelevant. Tests run on my sites are working beautifully.

@GioSensation thanks for catching that one and for running some tests for me. Glad to hear it's working well on your end.

Seriously, @jeremenichelli. You’re amazing. Will get to looking at this this week.

I’d vote for a minor. 👍 🚢 it!

Just turned on travis at https://travis-ci.org/iamdustan/smoothscroll though we may need to rename _travis.yml to .travis.yml

Yay! Look at that, travis CI working already.
Merging this and shipping it...
Thanks @iamdustan and @GioSensation!

:muscle: niiice


After 0.23.0 version app.component, there is no prompt.

0.29.0 version

0.30.0 Later version

This is expected, see https://github.com/johnsoncodehk/volar/blob/master/CHANGELOG.md#0230
You need to use interface GlobalComponents after 0.23.0.
import HelloWorld from './components/HelloWorld.vue'

declare module 'vue' {
	export interface GlobalComponents {
		HelloWorld: typeof HelloWorld
	}
}

I think this must be a good idea to compile templates faster, but what I might care more about is saving this job. I wonder if it can be an option, let’s choose whether to enable it. What do you think of this feature?

Yes, this feature can be convenient for some users, but it was decided to remove it for many reasons.
The two most important reasons are:

This is a Volar-specific feature, and users cannot smoothly transition to other IDEs such as webstorm.
This feature prevents users learning Vue 3 TS on the correct path.


Can't run asciidoctor-pdf 1.5.0.beta.3 in asciidoctorj-pdf

I am currently trying to bring asciidoctor-pdf 1.5.0.beta.3 to asciidoctorj-pdf.
However the tests currently fail here in Converter#register_fonts:
https://github.com/asciidoctor/asciidoctor-pdf/blob/946339c1fea1d2b1c97158993fddfdc5fb97ab7f/lib/asciidoctor-pdf/converter.rb#L3333
This seems to be because fonts_dir is a URI to an internal resource that is retrieved via the Ruby classloader here:
https://github.com/asciidoctor/asciidoctor-pdf/blob/946339c1fea1d2b1c97158993fddfdc5fb97ab7f/lib/asciidoctor-pdf/converter.rb#L3321-L3323
Using my puts debugging superpowers I figured out that dirs resolves to this:
["uri", "classloader", "/gems/asciidoctor-pdf-1.5.0.beta.3/data/fonts"]

(As it splits on the system path separator – which is : on my machine – I can imagine that this even runs on Windows :-D)

As it splits on the system path separator – which is : on my machine – I can imagine that this even runs on Windows :-D

Dang it. Windows is suppose to be using ; as the path separator (not the file separator). But on JRuby, this doesn't seem to be the case. And then there are JAR paths, which further complicate the matter.

I verified that on CRuby on Windows, File::PATH_SEPARATOR is ;. So JRuby is incorrect here.
However, there is still the large issue of the JAR filename in JRuby, which would certainly break JRuby on non-Windows.

My last comment was probably misleading, sorry about this ;-)
It doesn't run on my Mac, and I suppose it will also fail on Linux, because without any further customization fonts-dir is uri:classloader:/gems/asciidoctor-pdf-1.5.0.beta.3/data/fonts.
This is split to the 3 elements of dirs usr, class loader and /gems/asciidoctor-pdf-1.5.0.beta.3/data/fonts.
And neither of these 3 elements is a path that exists in the filesystem,.

Yes, that's what I suspected. The JAR paths are interfering with the assumption that we can separate entries using the path separator. Back to the drawing board.

We'll just change it to always be semicolon. That seems safe.

Another option is to detect uri:classloader: and skip it. I'm going to try it both ways and see which one feels better.

What's interesting is that now we see how to load fonts out of a JAR.
-a pdf-fontsdir=uri:classloader:/gems/asciidoctor-epub3-1.5.0.alpha.9/data/fonts

That makes me think that just standardizing on the ; as a separator is the way to go.

:+1:
I like the approach of treating  uri:classloader:, but then I was thinking that it might also be possible to load fonts from URLs. (If I understood correctly what the open-uri gem does).
So a ; is probably better.

Yeah. And it seems like Java uses ; in a lot of places for this very purpose...so it seems like a reasonable and safe thing to do.

Thanks for the heads up on this and good detective work. :male_detective:

Thanks! ☺

Keep in mind that when specifying multiple font dirs on the CLI, it's necessary to either escape the semi-colon or enclose the value in double quotes. Otherwise, the shell will interpret it as a command separator.


Title should be fixed

The title of this documentation page doesn't seem correct. it should be reviewed and fixed.
Thanks,
Luís.

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

ID: 19da7a75-ffd3-adb4-1abf-084c3dff1aeb
Version Independent ID: 6e99fd96-00f9-abfa-ba3b-cbb424ed6cc4
Content: Manage job logins for databases in an availability group - SQL Server Always On
Content Source: docs/database-engine/availability-groups/windows/logins-and-jobs-for-availability-group-databases.md
Product: sql
Technology: high-availability
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

@lcanastreiro -- Luís, thank you for your feedback.  Please clarify what may be misinterpreted in the title, or the source of confusion in the article.

@WilliamAntonRohm the HTML title of this page is - "Manage job logins for databases in an availability group" but the Title on the page is "Manage logins for jobs using databases in an Always On availability group". But what it is covered in this page is what a SQL Admin should have in consideration to synchronize SQL Server Agent Jobs and SQL Server Logins between AG replicas. Having this in consideration I consider that the TITLE of the page (and also the HTML one) should be rephrased or changed to something more intuitive easy to understand,

@lcanastreiro -- thank you for clarifying.
@MashaMSFT -- please look into this issue.

Hi @lcanastreiro thanks for your feedback! I am seeing a number of inconsistencies with this article, along with the title :) I have created an internal backlog item (1771657) to address this in a future iteration.
I'll close out this Git Issue for now but please feel free to comment should you need anything further.
Thanks!
Masha from the SQL Docs team

#please-close


Program runs but DNS not bypassed

Operating system / операционная система
Windows 11 22H2
Running as service / Запуск программы как сервис
I run it as a regular program / Запускаю программу обычным образом
Describe the bug / Опишите ошибку программы
Hey Vlad, using goodbyedpi for months now without any issue whatsoever, and it works like a charm. I recently purchased a laptop and can't get goodbyedpi to work on it for some reason. Both my PC(Where i have been using goodbyeDPI for over a year) & my laptop(Newly purchased) are on the same ISP and connected to the same router. Any leads to what might be causing this issue?
Additional information / Дополнительная информация
.

See this
https://github.com/ValdikSS/GoodbyeDPI/issues/313
Sep 10, 2023 12:15:10 Sayangd095 @.***>:

Operating system / операционная система
Windows 11 22H2
Running as service / Запуск программы как сервис
I run it as a regular program / Запускаю программу обычным образом
Describe the bug / Опишите ошибку программы
Hey Vlad, using goodbyedpi for months now without any issue whatsoever, and it works like a charm. I recently purchased a laptop and can't get goodbyedpi to work on it for some reason. Both my PC(Where i have been using goodbyeDPI for over a year) & my laptop(Newly purchased) are on the same ISP and connected to the same router. Any leads to what might be causing this issue?
Additional information / Дополнительная информация
.
—
Reply to this email directly, view it on GitHub[https://github.com/ValdikSS/GoodbyeDPI/issues/317], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AAXJZCLEPV7UY7P2B6HBFELXZWAG7ANCNFSM6AAAAAA4SBHVEA].
You are receiving this because you are subscribed to this thread.
[Tracking image][https://github.com/notifications/beacon/AAXJZCNB647XXFYXS5L67BTXZWAG7A5CNFSM6AAAAAA4SBHVECWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHHBGGYOQ.gif]

Hey,
Before trying that I just upgraded to WIndows 11 Pro from the Home Edition and that fixed the issue. It's running again.
Thank You


Restore the defaults for targetPort when not explicitly set

The original intent with having separate httpPorts and httpsPorts was to be able to specify multiple ports of each kind, mapped to the http and https ports, respectively, on the haproxy-ingress pod. When the ability to set the targetPort was added by https://github.com/haproxy-ingress/charts/commit/00ce05399db68257bd9906c397760ed053904f52, backwards compatibility was broken by removing the defaults.
Fixes #25

Lgtm, thanks! Merging.


Fixed Scope of MutationObserver

Yo since the MutationObservers aren't in the global scope, they don't continue to observe after the walkAndObserve function executes. So now the function returns them things as one object and I keep it in global scope.

Wrong goddamned repo, sorry!


Vitejs 3.0

Hi, thank you for this awesome plugin.
Do you plan to make it compatible with Vitejs 3 ?
Regards.

I just went to install Vite 3 in my project and also realized that the peerDependency is set to ^2 and thus causing issues with a 3+ install. Would be great to update the peer deps to support 3 if possible. I know I can override the npm install to force the update but I'd rather not.

Yes I think the only thing to fix is the peerDependency range, I will publish a version later today

Available in 1.4.1 🎉

@ArnaudBarre Thank you for the quick turnaround on this, much appreciated!