Home Assistant 20223.5.0 now throwing exception

As of the latest version of HA, the status string returned from OTG is throwing an exception
Logger: homeassistant.components.mqtt.models
Source: components/sensor/init.py:583
Integration: MQTT (documentation, issues)
First occurred: 06:42:09 (2631 occurrences)
Last logged: 07:40:29
Exception raised when updating state of sensor.idealboiler_status_slave, topic: 'IdealBoiler/value/idealC40/status_slave' with payload: b'-C-F----'
Exception raised when updating state of sensor.idealboiler_status_slave, topic: 'IdealBoiler/value/idealC40' with payload: b'online'
Exception raised when updating state of sensor.idealboiler_status_master, topic: 'IdealBoiler/value/idealC40' with payload: b'online'
Exception raised when updating state of sensor.idealboiler_status_master, topic: 'IdealBoiler/value/idealC40/status_master' with payload: b'-D---W--'
Exception raised when updating state of sensor.idealboiler_status_slave, topic: 'IdealBoiler/value/idealC40/status_slave' with payload: b'--------'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 579, in state
numerical_value = int(value)
ValueError: invalid literal for int() with base 10: '-D---W--'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/mqtt/models.py", line 270, in process_write_state_requests
entity.async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 585, in async_write_ha_state
self._async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 649, in _async_write_ha_state
state = self._stringify_state(available)
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 591, in _stringify_state
if (state := self.state) is None:
File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 583, in state
raise ValueError(
ValueError: Sensor sensor.idealboiler_status_master has device class None, state class None unit  and suggested precision None thus indicating it has a numeric value; however, it has the non-numeric value: -D---W-- (<class 'str'>)

Done a change for merge to stop this

Can confirm and can also confirm that the changes made in #218 (at least on the lines 136 and 137) stop these errors.

@rvdbreemen Could you please release binaries with fix

Trying to fix CI pipeline... anyone any ideas what it fails, help would be welcome ;-)
@serrj-sv I want to fix the pipeline, but you can download mqttha.cfg from the data folder in the dev branch and upload it manually.

Ben,
That makes a lot of sense, I missed that completely.
Thanks for the PR,
Robert

Ben,
That makes a lot of sense, I missed that completely.
Thanks for the PR,
Robert

You're welcome!
Glad I have been able to give a little bit back :)

I looked into the issue myself last night. It turns out that the 2023.5.0 is breaking not just my integration, a lot of integrations broke of this change in HA Core. I have seen this before, this will also mean that 2023.5.x will most likely fix this in the Core side, but it's good to clean up the mqttha.cfg integration a bit.
Please keep upgrading HA yourself too, it could also be that it will fix itself in 2-3 weeks time because they fix core.
Seriously we have seen this many times before with HA, move fast break often and fix. The HA core project is really good at this ;-)

Not sure but this is in the logs. Didnt check logs for weeks:
`Logger: homeassistant.components.mqtt.models
Source: components/sensor/init.py:583
Integration: MQTT (documentation, issues)
First occurred: 10:40:21 (34 occurrences)
Last logged: 10:42:07
Exception raised when updating state of sensor.otgw_status_master, topic: 'OTGW/value/otgw-A4E57C95FAB1/status_master' with payload: b'-----W--'
Exception raised when updating state of sensor.otgw_status_slave, topic: 'OTGW/value/otgw-A4E57C95FAB1/status_slave' with payload: b'--------'
Exception raised when updating state of sensor.otgw_status_slave, topic: 'OTGW/value/otgw-A4E57C95FAB1' with payload: b'online'
Exception raised when updating state of sensor.otgw_status_master, topic: 'OTGW/value/otgw-A4E57C95FAB1' with payload: b'online'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 579, in state
numerical_value = int(value)
ValueError: invalid literal for int() with base 10: '-----W--'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/mqtt/models.py", line 270, in process_write_state_requests
entity.async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 585, in async_write_ha_state
self._async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 649, in _async_write_ha_state
state = self._stringify_state(available)
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 591, in _stringify_state
if (state := self.state) is None:
File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 583, in state
raise ValueError(
ValueError: Sensor sensor.otgw_status_master has device class None, state class None unit  and suggested precision None thus indicating it has a numeric value; however, it has the non-numeric value: -----W-- (<class 'str'>)
`

Same issue here, I'm running HA 2023.5.2 where it is not fixed yet. I don't understand the instructions referring to post #218 (https://github.com/rvdbreemen/OTGW-firmware/pull/218), Should I update the OTGW firmware, or manually change something in OTGW?
Sorry in case this is a beginner question, I just try to get rid of my HA error logs, and this one has count of 56269 occurences since 48 hours already

@rik1984 the fix needs to be validated, but you can download the mqttha.cfg from the dev branch on github.
You can find it right here: https://github.com/rvdbreemen/OTGW-firmware/blob/dev/data/mqttha.cfg
If you download it, you can manually to the ESP using the file explorer.
I have already download the beta build here:
https://github.com/rvdbreemen/OTGW-firmware/suites/12881434620/artifacts/695010907
If you have some time, then please test it for me. Worst thing that could happen is that it works, and then I can release it.

Thanks for your guidance. I have manually deleted mqttha.cfg from the file system and uploaded the new version. I can confirm that it solves the issue

@rik1984 the fix needs to be validated, but you can download the mqttha.cfg from the dev branch on github.
You can find it right here: https://github.com/rvdbreemen/OTGW-firmware/blob/dev/data/mqttha.cfg If you download it, you can manually to the ESP using the file explorer.
I have already download the beta build here: https://github.com/rvdbreemen/OTGW-firmware/suites/12881434620/artifacts/695010907
If you have some time, then please test it for me. Worst thing that could happen is that it works, and then I can release it.

Downloaded the mqttha.cfg and replaced it on LittleFS but issue persists on latest version of HA (Home Assistant 2023.8.3)

Tested with Home Assistant 2023.8.4 and it solves the error issue in the log. So far all sensors etc seem to work as normal.

Tested on HA 2023.9.2, works like a charm, otgw sensors look well and stable.

I still face the issue spamming logs like crazy. @rik1984 when deleting and upliading a new mqttha.cfg, did you changed anything in partiuclar?
I am using:
Home Assistant 2023.9.3
Supervisor 2023.09.2
Operating System 10.5
Frontend 20230911.0 - latest
`Logger: homeassistant.components.mqtt.models
Source: components/sensor/init.py:593
Integration: MQTT (documentation, issues)
First occurred: 26 September 2023 at 10:52:28 (73848 occurrences)
Last logged: 13:27:38
Exception raised when updating state of sensor.otgw_status_slave, topic: 'OTGW/value/otgw-485519150FBB/status_slave' with payload: b'-C------'
Exception raised when updating state of sensor.otgw_status_master, topic: 'OTGW/value/otgw-485519150FBB/status_master' with payload: b'-D---W--'
Exception raised when updating state of sensor.otgw_status_slave, topic: 'OTGW/value/otgw-485519150FBB/status_slave' with payload: b'--------'
Exception raised when updating state of sensor.otgw_status_slave, topic: 'OTGW/value/otgw-485519150FBB' with payload: b'online'
Exception raised when updating state of sensor.otgw_status_master, topic: 'OTGW/value/otgw-485519150FBB' with payload: b'online'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 586, in state
numerical_value = int(value)
^^^^^^^^^^
ValueError: invalid literal for int() with base 10: '-D---W--'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 589, in state
numerical_value = float(value)
^^^^^^^^^^^^
ValueError: could not convert string to float: '-D---W--'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/mqtt/models.py", line 305, in process_write_state_requests
entity.async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 730, in async_write_ha_state
self._async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 830, in _async_write_ha_state
state, attr = self._async_generate_attributes()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 771, in _async_generate_attributes
state = self._stringify_state(available)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 736, in _stringify_state
if (state := self.state) is None:
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/sensor/init.py", line 593, in state
raise ValueError(
ValueError: Sensor sensor.otgw_status_master has device class 'None', state class 'None' unit '' and suggested precision 'None' thus indicating it has a numeric value; however, it has the non-numeric value: '-D---W--' (<class 'str'>)`

I still face the issue spamming logs like crazy. @rik1984 when deleting and upliading a new mqttha.cfg, did you changed anything in particular?

No I didn't do anything special, only following the instructions. I'm now running HA9.1 ans will without issues (not sure whether the mqttha.cfg file has been updated along with my HA updates or whether it is still in the version I changed it to, to be honest.

well, the mqttha.cfg file is on the otgw right? (more specifically on the wifi microcontroller on the otgw), so I don't see how that would be affected by updating ha ;)

I still face the issue spamming logs like crazy. @rik1984 when deleting and upliading a new mqttha.cfg, did you changed anything in particular?

No I didn't do anything special, only following the instructions. I'm now running HA9.1 ans will without issues (not sure whether the mqttha.cfg file has been updated along with my HA updates or whether it is still in the version I changed it to, to be honest.

well, the mqttha.cfg file is on the otgw right? (more specifically on the wifi microcontroller on the otgw), so I don't see how that would be affected by updating ha ;)

You are right indeed ;) so no changes done there since

I will have a look into this soon. To see how to fix this.  It is most likely a fix to the mqttha.cfg templates that is needed.

There is a working mqttha.cfg in the devbranch and there are dev binaries candidate available.
Issue does not break HA integration.
But I do need to release this. 🙈


Duplicate Index

Index can never be 100%, index.redb from https://ordstuff.info/indexes/0.19/, I downloaded the latest one, 853446 to 853993 continuously updated in a loop
The latest block transactions can be queried locally, but runes and inscriptions cannot be found。
Ord.exe is 0.19.1, We have attempted to remove index.redb and reinstall it, but it did not work。

Hi JD - this is a known issue, #3804.  Please see that issue for workarounds.  If you continue to experience issues, please reach out to the tech-support channel on the Ordicord for troubleshooting assistance.


How to handle Sub-Grids in UCI? Need to handle "Add Record" etc. operations

There's an answer inside https://github.com/Microsoft/EasyRepro/issues/279, but it involves downloading the code from here and creating your own DLL. Follow the comment from remnam on Aug 11.
That said, this should be fixed here in the official version.


FR: Option to only enable in certain notes

Thanks for this plugin!
I was wondering whether it was possible to activate this plugin only in certain notes, e.g. based on file path and/or frontmatter. Reason being that I find a feature like this very useful for writing, but not so much for note-taking.

At the moment you can achieve this by manually editing the styles.css file and prepending e.g. .apply-sentence-focus  to all the selectors.
Then only notes with "cssclass: apply-sentence-focus" in their frontmatter will have the sentence highlighting applied to them.


Address k8s issue on shared core machines

GCP shared core machines are inaccurately reporting # of CPUs to k8s. This is creating problems in our data audit/tests.

This is complete-- we manually override the k8s-supplied data with the amount of cores per class.


Filtering suggestions further to trim results to a city

This is probably a noob question on my part so apologies in advance.
I am currently trying to add geo-filtering to my map to lock things down to a city but search suggestions still show areas outside the target zone which tells me I'm probably doing something wrong.
Here's my code:
    <MapGL
        ref={this.mapRef}
        {...viewport}
        onViewportChange={this.handleViewportChange}
        mapboxApiAccessToken={MAPBOX_TOKEN}
    >
      <Geocoder
          mapRef={this.mapRef}
          onResult={this.handleOnResult}
          containerRef={this.geocoderContainerRef}
          onViewportChange={this.handleGeocoderViewportChange}
          mapboxApiAccessToken={MAPBOX_TOKEN}
          reverseGeocode={true}
          bbox={[-0.489,51.28,0.236,51.686]}
          filter={event => {
            // returns true if item contains New South Wales region
            return event.context.map( i => {
              return (i.id.split('.').shift() === 'place' && i.text === 'London');
            }).reduce(function (acc, cur) {
              return acc || cur;
            });;
          }}
      />
      <DeckGL {...viewport} layers={[searchResultLayer]} />
    </MapGL>

I guess the question here would be; what should go in the reduce function arguments?

Hi @sabib0y, thanks for raising the issue. I'll look into this tonight.

You don't need the filter prop if you have the bounding box of the area that you are interested in. You can just use the bbox prop. Please see this example https://codesandbox.io/s/reactmapglgeocoder-bbox-example-zzlf1.
As for using the filter prop for filtering results, the function receives a single result in Carmen GeoJSON format. You can parse that result and return true or false depending on if you'd like to include that result in the suggestions shown.
Let me know if you have any questions or need any more help.

Let me know if you'd like me to provide an example of using the filter prop as well.

@sabib0y also whipped up a filter example for you https://codesandbox.io/s/reactmapglgeocoder-filter-example-602xq


Change the behavior of enter key when adding transactions

Thanks again! I moved the settings to the new Kango way. :+1:


Add feature(s) to use hyper-rustls and/or h3 in datacake-rpc

Initially I wanted to look into replacing TCP with QUIC, but since datacake-rpc uses hyper, it's probably better to wait for / experiment with https://github.com/hyperium/h3. For the time being, hyper-rustls would be a useful addition, though. Thoughts?

Do you have a particular use case for adding the TLS? It's not been something I've particularly looked at for a use case since generally, your inter-node RPC should be internal only rather than publically accessible where TLS becomes more significant.

The immediate use case is using datacake-rpc as a general purpose RPC framework, also for communicating with the cluster from outside (replace Axum with it in the sqlite example, for instance), and for sensitive data I don't want to rely on cloud-providers to secure their internal networks properly. Controlling cluster membership via mutual certificate authentication is also a good thing, so lots of reasons, really.

Seems reasonable, it is probably worth doing at some point then, unfortunately, I think the way the general connection system is handled in general could do with improvement as whole, since we have some interesting hacks in order to support turmoil simulation.

Yes, I was glad to see that. Presumably searching the places with #[cfg(feature = "simulation")] should help finding the relevant bits quickly.

Yes should be

I investigated a little, and surprisingly it seems hyper + tls is an largely an unsolved issue. The one crate I've found that

uses hyper with tls
supports client cert authentication
is flexible enough to access remote_addr and information on peer certificates in handlers
supports both clients and servers

is (hidden in) tonic
Others either don't have client support like tls-listener, or haven't found a way to offer the above mentioned flexibility https://github.com/rustls/hyper-rustls/issues/198 .

I second the idea of incorporating TLS.  Ultimately, it is critical to our application, meaning we cannot go to production without it.
Some industries require that any socket connections, even within an owned data-center, must use TLS.
@ousado, is this something you are taking on or just highlighting the need?

sorry, not sure why I haven't seen this notification earlier
I've looked a bit further, and the tonic client/server implementations already include and depend on some protobuf stuff. Then I found axum_server which allows to use a custom rustls configuration. For the client I'm back to using hyperrustls, but I ran out of time so I haven't tried to plug them in into datacake yet. I believe they'd be up to the task, though. It doesn't feel quite right to pull in two different dependencies for this, but that's what the ecosystem offers at this point, I'm afraid.
I've also thought about replacing https with a raw socket connection, tower and a simple codec, but I don't know which http features are used in datacake and would have to be reimplemented.

Apparently there have been some  changes to hyper-rustls with respect to the issues that made it impossible to access the remote address as well as certificates when accepting a connection, so it might indeed be sufficient now.
Do you also want to support nativetls?
https://github.com/rustls/hyper-rustls/pull/219
https://github.com/rustls/hyper-rustls/issues/198

@ChillFish8 did you see the direct email I sent you regarding this issue?

I did, I sent a reply back, not sure if you got my reply though :/
Anyway I can copy what I replied to:

What sort of application are you trying to use Datacake's RPC module with?

Is it behind a load balancer or similar? Or just for inter-node communication.

I ask this because I am currently working on the IO layer which will be primarily switching to QUIC which will support TLS v1.3 and be TLS by default.

That is really strange, I did not receive a response.  Double checked my spam folder as well.
It is a specialized key value store, using the datacake example as a reference.
In terms of TLS support, it needs to be for all connections; the connection with the clients and the inter-node connections.
Supporting TLS 1.3 is key!  Thank you!
I believe supporting TLS on the client side interface is straightforward.  It is the datacake inter-node communication that concerns me, which it sounds like you are addressing.
In our architecture, a load-balancer is not necessary.

Cool, sounds good then, like I said I'm working towards replacing the current IO layer using QUIC, but naturally due to QUIC's relatively young age few load balancers support QUIC in that way. But for your use case, it should be fine.


IPMI mc info aux version check

Currently, we don't check this in the code
https://github.com/openbmc/openbmc-test-automation/blob/master/tests/ipmi/test_general_ipmi.robot#L441
example:
     Aux Firmware Rev Info     :
         0x00
         0x00
         0x00
         0x00  <-- This will have value and check against the /etc/os-release

As per discussion, I understood, the basic pseudocode/algorithm step:
step1. Read  ‘/etc/os-release’
step2. Read r2 value / aux version from value found at step1.
step3. Compare  r2 value with [aux_firmware_rev_info][3]
step4. In step3, both value should be same, if not fail the test case.

From https://gerrit.openbmc-project.xyz/#/c/9481/8/meta-openbmc-machines/meta-openpower/meta-ibm/meta-witherspoon/recipes-phosphor/ipmi/phosphor-ipmi-host.bbappend
import re

def do_populate_aux_version(version_id):
     print "BMC Version:",version_id
     # count from the commit version
     count = re.findall("-(\d{1,4})-", version_id)
     print  "count:",count

     release = re.findall("-r(\d{1,4})", version_id)
     print "release:",release
     if release:
         auxVer = count[0] + "{0:0>4}".format(release[0])
     else:
         auxVer = count[0] + "0000"
     print "auxVer:",auxVer

do_populate_aux_version("xxx-v2.1-438-g0030304-r3-gfea8585")
do_populate_aux_version("v2.2-17-gabf205c")

o/p:
rango@ubuntu:~$ python test.py
BMC Version: xxx-v2.1-438-g0030304-r3-gfea8585
count: ['438']
release: ['3']
auxVer: 4380003
BMC Version: v2.2-17-gabf205c
count: ['17']
release: []
auxVer: 170000
rango@ubuntu:~$

From IPMI mc info for the above would look like
Aux Firmware Rev Info     :
    0x04
    0x38
    0x00
    0x03

Aux Firmware Rev Info     :
    0x00
    0x17
    0x00
    0x00


New version for egui-macroquad

I want to release new version of egui-macroquad, but there are conflicts between miniquad and macroquad versions, so I need latest miniquad in order to use latest macroquad.
Could you also please push new release to the crates after this PR?
@emilk

Shouldn't it suffice to do this:
egui = { version = "0.28", features = ["bytemuck"] }
miniquad = { version = "0.4" }

If I understand correctly cargo is capable of semantic versioning and this should be more flexible then the fixed versions you are suggesting.

@jbracker  see #69


feat: custom icons in tray

Waybar allows for customizing the output of most modules. It would be awesome to let users replace applet icons with their own. This would make the bar's style even more consistent.

I see two possible ways people would want to use this feature. The first being with an icon font like is common with other blocks. Here's an example of what that config might look like:
{
  "layer": "top",
  "modules-right": ["tray"],
  "tray": {
    "icons": {
      "discord": "icon-font-char-here"
    }
  }
}

The atlernative would be to use images similar to what is done today:
{
  "layer": "top",
  "modules-right": ["tray"],
  "tray": {
    "icons": {
      "discord": "/path/to/image.png"
    }
  }
}

I suppose it's also possible to support both options if we're comfortable adding multiple entries to the tray config (eg. fontIcons and imageIcons).

Of course, this will only be possible if there's something (like app id) that we can match on. I'm not very familiar with how applets or the system surrounding it work, so I'm not sure if this is possible. It seems like something that should be doable.

This is exactly what I was looking for.

Any Updates on this?

Hi @antidoid , not sure can you check whether it PR1978 solves current enhancement.

That would be sick.

I would love this feature.

I would love this feature.

This would be great to see, so I can make my tray icons integrate more with my system.

I want it! I need it!

This would be exactly what I was looking for.

Is this still not possible????

I think adding a tray shade/hue overlay would be good too.
like grayscale or sepia or a custom tint, where all icons are shaded the same.


fix(clerk-js): Include new redirect url params to sign in/up url building

Description

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

!snapshot

Closing since there will be more discussion before agreeing on the approach for this issue.


Improved UX/UI for moving for asserting locations

Hi Team,
With multiple hotspots in a wallet, the bluetooth pair is not necesarily the current hotspot selected. This can lead to the issue that when moving the hotspot, it is not the paired hotspot but the selected one.  Although this is a 100% user error this can be prevented by a simple measure:

Display Hotspot name in assert location screen and the final submit screen
Display OLD and New Location in final screen ( not sure how this can fit on mobile though )
Thanks!

pairing over bluetooth is not required to assert location, so users take an explicit action on a Hotspot they select, not one that is broadcasting over bluetooth.


foundry config error: invalid type: found string "ETHERSCAN_API_KEY", expected struct EtherscanConfig for setting etherscan.api_key

Component
Forge
Have you ensured that all of these are up to date?

[ ] Foundry
[ ] Foundryup

What version of Foundry are you on?
forge 0.2.0 (a79dfae 2024-11-16T00:21:53.964085000Z)
What command(s) is the bug in?
forge verify
Operating System
macOS (Apple Silicon)
Describe the bug
[etherscan]
api_key = "ETHERSCAN_API_KEY"
The writing style seems correct, but there might be an error in recognition, and I'm not sure why.

Hi @t-o-wayn3
See: https://book.getfoundry.sh/reference/config/etherscan#etherscan
The correct format  for the [etherscan] block is as follows:
[etherscan]
mainnet = { key = "${ETHERSCAN_MAINNET_KEY}" }
mainnet2 = { key = "ABCDEFG", chain = "mainnet" }
optimism = { key = "1234567" }
unknown_chain = { key = "ABCDEFG", url = "<etherscan api url for this chain>" }


Date Clarification > Salesforce Objects Connector Documentation > Troubleshooting Section

Under the Troubleshooting section of the page there is a reference to Enabling OAuth Authentication in Power BI Desktop "Starting in December 2021". Should that be "December 2020", or is the availability really delayed until December 2021?
"Enabling OAuth Authentication in Power BI Desktop
Starting in December 2021 Power BI Desktop:
-Install the new Edge Chromium browser, (at least beta) from https://www.microsoftedgeinsider.com/download.
-In your Environment Variables, set the System variable PQ_EnableEdgeChromiumOAuth to true . Once that is set it will use Edge Chromium for only QuickBooksOnline. For this release, only QuickBooksOnline is enabled by default.
-To set it for Salesforce, in System Environment Variables set PQ_ExtendEdgeChromiumOAuthAllowList to Salesforce"
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8c78a664-32c4-68c5-97ec-99d5fa4e5aab
Version Independent ID: 75152c2f-8bed-c5df-5fa3-2470920f6dde
Content: Power Query Salesforce Objects connector
Content Source: powerquery-docs/Connectors/SalesforceObjects.md
Service: powerquery
GitHub Login: @DougKlopfenstein
Microsoft Alias: bezhan

Hi @jkadilak -
The documentation has been corrected to December 2020.


eval from ui thread in Atom by require top level

Got errors from got using eval that fails when run on ui thread in Atom which will happen if require on top level.

https://github.com/AtomLinter/linter-eslint/issues/210

This doesn't seem to fail in Atom when running anymore, but it fails on ci:
https://travis-ci.org/ensime/ensime-atom/jobs/120055214:
ensime-server-update
  it should be able to download coursier
    EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".

      EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src 'self'".
      at createErrorClass (/home/travis/build/ensime/ensime-atom/node_modules/download/node_modules/got/node_modules/create-error-class/index.js:18:49)
      at Object.<anonymous> (/home/travis/build/ensime/ensime-atom/node_modules/download/node_modules/got/index.js:363:20)
      at Module._compile (/home/travis/atom/usr/share/atom/resources/app.asar/src/native-compile-cache.js:103:30)
      at Object.defineProperty.value [as .js] (/home/travis/atom/usr/share/atom/resources/app.asar/src/compile-cache.js:208:21)
      at Module.load (/home/travis/atom/usr/share/atom/resources/app.asar/node_modules/coffee-script/lib/coffee-script/register.js:45:36)
      at Function.Module._load (module.js:310:12)
      at Module.require (module.js:365:17)
      at require (/home/travis/atom/usr/share/atom/resources/app.asar/src/native-compile-cache.js:50:27)
      at Object.<anonymous> (/home/travis/build/ensime/ensime-atom/node_modules/download/index.js:9:11)
      at Module._compile (/home/travis/atom/usr/share/atom/resources/app.asar/src/native-compile-cache.js:103:30)
      at Object.defineProperty.value [as .js] (/home/travis/atom/usr/share/atom/resources/app.asar/src/compile-cache.js:208:21)
      at Module.load (/home/travis/atom/usr/share/atom/resources/app.asar/node_modules/coffee-script/lib/coffee-script/register.js:45:36)
      at Function.Module._load (module.js:310:12)
      at Module.require (module.js:365:17)
      at require (/home/travis/atom/usr/share/atom/resources/app.asar/src/native-compile-cache.js:50:27)
      at /home/travis/build/ensime/ensime-atom/node_modules/ensime-client/lib/ensime-server-update-coursier.js:70:20
      at [object Object].<anonymous> (/home/travis/build/ensime/ensime-atom/spec-integration/ensime-server-update-coursier-spec.coffee:28:5)

My uneducated guess is that it worked in my Atom when I delayed it to run on the chromium thread (https://github.com/ensime/ensime-node/commit/abd4ece521b184adfdb3a36f7dcd20822d37f889), but not in normal node which might run the init of all scripts?
Anyway, I think we should really use something else that doesn't use eval.

The eval has been removed at the end dep:
https://github.com/floatdrop/create-error-class/commit/c899536f5e1432a1b8846493fe8047978f5c5b00

Download is 4 months since last release and uses got ^5.0.0:
https://github.com/kevva/download/blob/master/package.json#L33
Got has updated long ago: https://github.com/sindresorhus/got/commits but obviously not in 5.x.x series. So download is the problem.

@jeckhart Do you know if we have any options to download in case we don't get any luck there?

@steelbrain Quicky question about eval: Is it possible to workaround in any way? You had similar problems with linter? https://github.com/AtomLinter/linter-eslint/issues/210

Oh, my fix did nothing but delay the error until download was required. I thought that maybe there was something about ui thread vs node or something, but this simply doesn't work in Atom. Download is broken in atom.

@hedefalk There are several workarounds for this, one is the loophole npm package, the other is to spawn a node worker using the Atom executable as the electron runtime. We chose the second option

@steelbrain Cool, thanks!
I actually started out doing npm shrinkwrap to fix the version of the transitive dependency to a later one without eval. But I misunderstood shrinkwrap. Basically I did like https://github.com/npm/npm/issues/7108#issue-53955766, but this failed on travis for some versions of node. I then tried it locally with those version and yes, I guess npm has changed on this. I thought I could just put a subset of stuff in shrinkwrap, but then stuff wasn't installed for older versions of npm:
https://travis-ci.org/hedefalk/ensime-node/builds/120089602
I guess I could just do a postinstall like suggested here:
https://github.com/npm/npm/issues/7108#issuecomment-69507982
find node_modules -type d -name got | xargs rm -fr
npm install<EMAIL_ADDRESS>
I am a good citizen though and put a PR on the upstream package: https://github.com/kevva/download/pull/95
Seems a bit abandon though :)


Scrolling through documents is extremely slow

Prerequisites

[ X ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Whenever I open a file, the scrolling is EXTREMELY slow. Like I have to move the mouse wheel 20 times to move the document by a single line. The weird thing is scrolling works just fine when I'm in the settings, it's just in files. I tried reinstalling the whole thing and nothing. All my settings are on default, including scroll sensitivity which remains at 40 (I did try turning it up to 200 and it did start scrolling a bit faster but it was still unacceptably slow). This never happens in any other software.
I've also noticed a general slowness when trying out Atom, there's often a lag between pressing a key and the letter appearing on the screen. I tried turning off all the programs I could and running essentially just Atom and the Windows Task Manager. CPU was usually below 10%, occasionally peaking at around 40-50%, RAM was stable at roughly 30%. Maybe it could somehow be connected to the scrolling issue, I don't know
(I'm not sure what to write into the other fields, the issue is there all the time so there aren't really any steps to reproduce it other than to just turn the editor on.)
Versions
Atom 1.23.3
apm 1.18.12
Windows 7, 64bit

Thanks for the report!

Whenever I open a file, the scrolling is EXTREMELY slow. Like I have to move the mouse wheel 20 times to move the document by a single line.

Do you have vertical scroll speed set to One screen at a time in the Windows control panel mouse settings? If so then this was fixed in https://github.com/atom/atom/pull/16114 which is currently scheduled to be released in Atom 1.24 (Currently in beta).

Thank you!! You were right, I did have that in my settings, I downloaded the beta and everything seems to work just fine now. If the problem somehow comes back even with the beta version, I'll update this issue thread


Pythran version of scipy.optimize._group_columns

@rgommers this one looks good. The CI issue seems unrelated. My local benchmarks give interesting speedup
$ python -m timeit -s 'from numpy import array; n = 200; m = n - 12; x = array(range(n)); y = array(range(12, 12 +n)); xy = array(range(n*n)).reshape((n,n)); from _group_columns import group_sparse as gs, group_dense as gd;' 'gd(n, m, xy)'

pythran: 10 loops, best of 3: 26.9 msec per loop
cython: 10 loops, best of 3: 79.2 msec per loop

@rgommers this one looks good. The CI issue seems unrelated. My local benchmarks give interesting speedup
$ python -m timeit -s 'from numpy import array; n = 200; m = n - 12; x = array(range(n)); y = array(range(12, 12 +n)); xy = array(range(n*n)).reshape((n,n)); from _group_columns import group_sparse as gs, group_dense as gd;' 'gd(n, m, xy)'

pythran: 10 loops, best of 3: 26.9 msec per loop
cython: 10 loops, best of 3: 79.2 msec per loop

@rgommers gentle ping ;-)

@rgommers gentle ping ;-)

Thanks for the ping, and sorry for the delay @serge-sans-paille. I'm kind of distracted by a proposal deadline until the 19th.
That's a massive speedup, guess there's a serious problem in the Cython code somehow. The code looks like a correct line-by-line translation, it's not clear to me why there should be a ~700x performance difference.

Thanks for the ping, and sorry for the delay @serge-sans-paille. I'm kind of distracted by a proposal deadline until the 19th.
That's a massive speedup, guess there's a serious problem in the Cython code somehow. The code looks like a correct line-by-line translation, it's not clear to me why there should be a ~700x performance difference.

I can't reproduce that. I get the same result for Pythran:
>>> n = 200
>>> m = n - 12
>>> xy = np.arange(n**2).reshape((n, n))
>>> n = 200
>>> m = n - 12
>>> x = np.arange(200)
>>> y = np.arange(12, 12 + n)
>>> xy = np.arange(n**2).reshape((n, n))
>>> from scipy.optimize._group_columns import group_dense
>>> %timeit group_dense(n, m, xy)
115 µs ± 444 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

Cython code gives an exception for that, it only works with int32 xy:
>>> n = 200
>>> m = n - 12
>>> x = np.arange(200)
>>> y = np.arange(12, 12 + n)
>>> xy = np.arange(n**2).reshape((n, n)).astype(np.int32)
>>> from scipy.optimize._group_columns import group_dense
>>> %timeit group_dense(n, m, xy)
135 µs ± 1.43 µs per loop (mean ± std. dev. of 7 runs, 10000 loops each)

Then if I change Pythran to int32:
>>> xy = np.arange(n**2).reshape((n, n)).astype(np.int32)
>>> %timeit group_dense(n, m, xy)
75.6 µs ± 429 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

So the performance gain is close to 2x. Looks like you swapped the numbers and wrote msec instead of usec?

I can't reproduce that. I get the same result for Pythran:
>>> n = 200
>>> m = n - 12
>>> xy = np.arange(n**2).reshape((n, n))
>>> n = 200
>>> m = n - 12
>>> x = np.arange(200)
>>> y = np.arange(12, 12 + n)
>>> xy = np.arange(n**2).reshape((n, n))
>>> from scipy.optimize._group_columns import group_dense
>>> %timeit group_dense(n, m, xy)
115 µs ± 444 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

Cython code gives an exception for that, it only works with int32 xy:
>>> n = 200
>>> m = n - 12
>>> x = np.arange(200)
>>> y = np.arange(12, 12 + n)
>>> xy = np.arange(n**2).reshape((n, n)).astype(np.int32)
>>> from scipy.optimize._group_columns import group_dense
>>> %timeit group_dense(n, m, xy)
135 µs ± 1.43 µs per loop (mean ± std. dev. of 7 runs, 10000 loops each)

Then if I change Pythran to int32:
>>> xy = np.arange(n**2).reshape((n, n)).astype(np.int32)
>>> %timeit group_dense(n, m, xy)
75.6 µs ± 429 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

So the performance gain is close to 2x. Looks like you swapped the numbers and wrote msec instead of usec?

Oh, got it, I was using Python Int in my setup, which probably caused my Cython measurement to be meaningless. I've updated the Pythran export line to only accept C int, but it fails on windows I need to have both intc and int. I guess a speedup of x2 is already good ;-)

Oh, got it, I was using Python Int in my setup, which probably caused my Cython measurement to be meaningless. I've updated the Pythran export line to only accept C int, but it fails on windows I need to have both intc and int. I guess a speedup of x2 is already good ;-)

@rgommers is there anything I should do for that one?

@rgommers is there anything I should do for that one?


Build fork for "AWA - Affenpocken Warn App"

Feature description
To be prepared for the upcoming Monkeypox pandemic I would suggest to build a fork of the CWA, maybe already with the pandemic as a "variable" to save some time, effort & costs in the future.
Problem and motivation
Problem: Monkeypox pandemic
Motivation: Want to make sure we are better prepared then last time.
Is this something you're interested in working on
Yes

Related issue: https://github.com/corona-warn-app/cwa-wishlist/issues/827

Just a small suggestion from my side: I wouldn't overuse the word pandemic (which is defined as en epidemic spreading across a large region, continents or even worldwide, see Wikipedia), because it causes people to either overreact or people will say "Yeah, the next pandemic, whatsoever".
The WHO currently does not declare monkeypox neither as a pandemic nor as a epidemic.
But these are only my 2 cents on this.

CWA would have to be completely changed from the basis, as according to the RKI monkeypox spreads very different than Sara-Cov-2:

Menschen können sich vor allem durch Kontakt mit den Hauteffloreszenzen, Blut, Gewebe oder Ausscheidungen infizierter Tiere (in erster Linie verschiedener Nagetiere) und beim Umgang mit dem Fleisch erkrankter Tiere infizieren. Eine Übertragung von Mensch zu Mensch ist selten und nur bei engem Kontakt möglich, kann aber durch Kontakt mit Körperflüssigkeiten oder Schorf der Affenpocken-Infizierten auftreten, vermutlich auch im Rahmen von sexuellen Handlungen. Eine Übertragung bereits in der Prodromalphase ist bei Face-to-Face-Kontakt durch ausgeschiedene Atemwegssekrete möglich. […]

RKI does not mention spread via Aerosole (main way of spread of Corona) currently.
You're definitely right that we should be prepared and maybe even start working on a AWA app, however, RKI would have to give the contract for this and I honestly don't see them doing this at the current stage.


        Parameters
        ----------
        start : parse_time compatible
        end : parse_time compatible
        instruments : sequence
            Sequence of instruments whose data is requested.
        url : str
            Base URL for the request.
        """
        day = datetime.datetime(start.year, start.month, start.day)
        while day <= end:
            directory = url + day.strftime('%Y/%m/%d/')
            opn = urllib2.urlopen(directory)
            try:
                soup = BeautifulSoup(opn)
                for link in soup.find_all("a"):
                    href = link.get("href")
                    for prefix, parser in PARSERS:
>                       if href.startswith(prefix):
E                       AttributeError: 'NoneType' object has no attribute 'startswith'

Copied from original issue: sunpy/sunpy#1376

From @dumindux on April 7, 2015 1:35
@Cadair @ehsteve I have run this test in my local environment(master branch) and it seemed to pass. Are there any specific conditions for this to fail?

From @ayshih on April 8, 2015 14:32
I can't reproduce it locally either.  Has this test ever succeeded on Travis?  From a cursory look, it appears as if BeautifulSoup is finding an anchor tag (<a>) that doesn't contain an href URL.  This suggests that the server is providing a different page to Travis, possibly a 30x or 40x return page?
Loosely related code comments from looking at the traceback:

The docstring claims that query() accepts parse_time-compatible inputs, but the code immediately assumes that they are already datetime objects.
Shouldn't urlparse.urljoin() be used to build URLs rather than just concatenating strings?

From @ehsteve on April 9, 2015 20:14
I concur. Let's just xfail for this 0.6. Cool?

From @ehsteve on April 10, 2015 14:41
Created a PR to ignore this for now to enable the release of 0.6 (#1380).

From @ayshih on April 13, 2015 13:57
Ugh, a different Callisto test (test_extend) failed on a recent Travis run (https://travis-ci.org/sunpy/sunpy/jobs/58267452).  I'm guessing that the Callisto webserver sometimes times out or returns garbage.  The Callisto query() function needs to have some form of error-catching!

From @Cadair on April 30, 2015 11:53
I am going to leave this open, to promote a long-term solution.

Deleted Spectrogram class and started again.


Task not replaceing existing archive if required

Task name: ArchiveFiles
Description: Added remove action tl.rmRF(archiveFile); if the option replaceExistingArchive is set.
Documentation changes required: N
Added unit tests: (Y/N) N
Attached related issue: Y https://github.com/microsoft/azure-pipelines-tasks/issues/14515#L340
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

Btw this task should remove original archive by default, so I'm not sure if these changes are actual... Do you have some specific case when it doesn't work?

Btw this task should remove already existing archive by default, so I'm not sure if these changes are actual... Do you have some specific case when it doesn't work?

I used this task and the archive is not removed by default or by requesting the replaceExistingArchive option.
As a workaround, I had to use the DeleteFilesV1 task to ensure the existing archive is removed before I can use the same archive name. I have to check again the exact issue but:

either the new data is not archived and the archive has the existing data;
or the new data is merged with the data of the old archive.

Btw this task should remove already existing archive by default, so I'm not sure if these changes are actual... Do you have some specific case when it doesn't work?

I used this task and the archive is not removed by default or by requesting the replaceExistingArchive option.
As a workaround, I had to use the DeleteFilesV1 task to ensure the existing archive is removed before I can use the same archive name. I have to check again the exact issue but:

either the new data is not archived and the archive has the existing data;
or the new data is merged with the data of the old archive.

Got it, thanks! Anyway I think it would be a good way to remove it explicitly instead of relying on default tool behavior as it was before.

Btw this task should remove already existing archive by default, so I'm not sure if these changes are actual... Do you have some specific case when it doesn't work?

I used this task and the archive is not removed by default or by requesting the replaceExistingArchive option.
As a workaround, I had to use the DeleteFilesV1 task to ensure the existing archive is removed before I can use the same archive name. I have to check again the exact issue but:

either the new data is not archived and the archive has the existing data;
or the new data is merged with the data of the old archive.

Got it, thanks! Anyway I think it would be a good way to remove it explicitly instead of relying on default tool behavior as it was before.

 I'm not sure what is the definition and the idea of the default behavior for this task and therefor is it a good way to remove archive explicitly. There could be a scenario where you have un existing archive and you would like to add new files to it. Then it is probably not a good way to remove archive explicitly. Just a supposition :)


Crazy idea: Display File renderer?

I wonder what people might think of a display file renderer in a webview? It's a crazy idea. Would it even be worth it?
Would love to hear some thoughts.

Hi Liam, I think this could be a good idea, many directors dislike IBM I for the green screen, If you invent some strategies to wrap the millions of display file to something more web friendly this could be great. But please something more efficient then Hats...

Am I right in assuming you are thinking of emulating the 5250 data stream?  If so, I'm not sure how it fits into this extension.  And I believe it might be quite challenging to do.  Personally, I'd rather see work on content assist (if that is even feasible and maintainable), or integrating a debugger.

RDi has this feature... so not so crazy!

@priceaj I was thinking we could use something like canvas to build it? I've personally used Konva before, which is just a nice wrapper.
Have you any ideas?

The only canvas tool I know of is paper.js, but it's not something I'm very familiar with

@priceaj paper.js is awesome, but maybe too complex for a simple text and line renderer? Maybe one day if we want to make it do funny things it could be neat though.


Majority Element(Leetcode)

AIM
I will be adding a question in the Leetcode folder Majority Element.
Details

Problem Statement:-

Given an array nums of size n, return the majority element.
The majority element is the element that appears more than ⌊n / 2⌋ times. You may assume that the majority element always exists in the array.

Test Cases:-

Input: nums = [3,2,3]
Output: 3
Input: nums = [2,2,1,1,1,2,2]
Output: 2
Please Assign me this issue @Amit366  @ravikr126  @Saurabh4626
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

@Ayush7614 I'd like to work on this issue in C language under HCSSoC. Kindly assign me this issue.

@ravikr126  I like to work in this issue in Python under LGMSOC21


chore(docs): remove leading dollar sign

See https://github.com/reactphp/http/pull/458
Reopened because I messed https://github.com/reactphp/cache/pull/50

Couldn't find a way to reopen, possibly because I don't have the rights to. I'll ask your next time this arises.


Wait for event

Hi!
How can i wait for an event, and then continue execution? If I just  connect a host event source to a script handler function, the script execution finished before it is raised. How can I solve it?
I would like something like this:
var obj = new StatedObject();

//...

//do process...

//wait obj.Status changed to "Stopped"

//continue process...

Thanks!

@balazsszekely Yes, the script that set up the event connection will terminate, but the script handler function will be invoked when the event is raised. Is that not the expected behavior?

Yes, the problem was that I disposed the ScriptEngine before event was raised. Thanks.


feat: Update Karpenter to to 0.21.1

What does this PR do?
pdate karpenter to v0.21.1
Motivation
Improvements and fixes. Drift feature https://karpenter.sh/preview/concepts/settings/#feature-gates which is hard to test because of https://github.com/aws-ia/terraform-aws-eks-blueprints/issues/512 (pass helm values without overriding/deleting default values)
More

[X] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[ ] Yes, I have added a new example under examples to support my PR
[ ] Yes, I have created another PR for add-ons under add-ons repo (if applicable)
[ ] Yes, I have updated the docs for this feature
[ ] Yes, I ran pre-commit run -a with this PR

Note: Not all the PRs require a new example and/or doc page. In general:

Use an existing example when possible to demonstrate a new addons usage
A new docs page under docs/add-ons/* is required for new a new addon

For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes
Karpenter example deployment:
tested karpenter example with terraform v1.3.6 using sample_deployment_lt.yaml and sample_deployment.yaml
and all 6 pods were successfully scheduled from both deployments.

@armujahid we have no problem setting values in karpenter with
module "eks_blueprints_kubernetes_addons_karpenter" {
  source = "git::ssh://git@github.com/aws-ia/terraform-aws-eks-blueprints.git//modules/kubernetes-addons/karpenter?ref=v4.20.0"

  helm_config = {
    create_namespace      = true
    version               = var.karpenter_helm_config_version
    wait                  = true
    repository_username   = var.ecrpublic_token.user_name
    repository_password   = var.ecrpublic_token.password

    set = [
      # {
      #   name  = "logLevel"
      #   value = "debug"
      #  type  = "string"
      # },
      {
        name  = "settings.batchMaxDuration"
        value = "10s"
        type  = "string"
      },
      {
        name  = "settings.batchIdleDuration"
        value = "5s"
        type  = "string"
      },
      {
        name  = "hostNetwork"
        value = "false"
        type  = "string"
      },
      {
        name  = "settings.aws.interruptionQueueName"
        value = aws_sqs_queue.this.name
        type  = "string"
      },
      # {
      #   name  = "settings.aws.enablePodENI"
      #   value = "true"
      #   type  = "string"
      # },
      {
        name  = "settings.aws.nodeNameConvention"
        value = "resource-name"
        type  = "string"
      },
      {
        name  = "settings.featureGates.driftEnabled"
        value = "true"
        type  = "string"
      },
    ]
  }
  addon_context = var.addon_context
  sqs_queue_arn = aws_sqs_queue.this.arn

}

@FernandoMiguel Nice. Sorry for my limited terraform knowledge. I hope this will also work by setting karpenter_helm_config like
  enable_karpenter = true
  # # Optional  karpenter_helm_config
  karpenter_helm_config = {
    set = [
       {
        name  = "settings.featureGates.driftEnabled"
        value = "true"
        type  = "string"
      },
    ]
  }

Not sure if this will delete helm config values of module (clusterName, clusterEndpoint, defaultInstanceProfile, interruptionQueueName}) https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/73e17edb5014fe022c3295b125f29234f864677f/modules/kubernetes-addons/karpenter/locals.tf#L24

that is our helm_config and we have no issues with overrides


Feature/pagination

Added pagitation to the public, private and user timelines.

https://github.com/DevOps-2024-group-p/maxitwit/actions/runs/8606189981/job/23584080668


".mesh" mesh with GMSH

Hello,
I would like to make my own mesh with GMSH. After exporting my mesh to ".mesh", I notice that the mesh file obtained doesnt' look like the standard format used by MFEM and GlVIS. How can I use a good mesh format with GMSH?
Best regards,
Leïla

The MFEM Mesh class will try to detect various formats it supports. The simplest way is to just try to view the mesh in GLVis: ./glvis -m path/to/file.mesh. MFEM (and therefore GLVis) should be able to recognize and read linear GMSH meshes.

Hi Leïla,
To add to @v-dobrev's answer -- the Gmsh support was added in this PR: https://github.com/mfem/mfem/pull/51 by @martemyev. For the mesh types that MFEM supports (tri/quad/tet/hex), you should be able to use a Gmsh mesh in ASCII or Binary format. Note that mixed meshes may not be fully supported, so it is probably best to stick with pure tet meshes in 3D.
Let us know if you have any issues.
Tzanio

After generating a mesh in GMSH, what format does the mesh save in to be recognized by the library?
The ".mesh" format is not recognized by GLVIS.

Normally, GMSH generate .msh (not .mesh) and GLVIS is supposed
to load it.

On Jun 28, 2018, at 9:33 AM, LeilaSalomon<EMAIL_ADDRESS>wrote:
After generating a mesh in GMSH, what format does the mesh save in to be recognized by the library?
The ".mesh" format is not recognized by GLVIS.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@LeilaSalomon To add to what @sshiraiwa said, GMSH does not output in the MFEM .mesh format. But the native GMSH format .msh is readable in MFEM. I have actually used mixed meshes as well in MFEM and it seems to be fine, but maybe try what @tzanio suggested first.

@LeilaSalomon would you mind sharing the mesh file you're having issues with?

I am not sure what is going on here, but this format doesn't look like the usual GMSH file I am used to. GMSH in my environment uses $Nodes - $EndNodes instead of $NOD-$ENDNODE. After quick google search, I found $NOD-$ENDNODE might have been used in the legacy file format, but still the  node data in $NOD segment does not make sense. Should it be like this?
$NOD
number-of-nodes
node-number x-coord y-coord z-coord
…
$ENDNOD

This looks like an old Gmsh format, which we do not support: http://gmsh.info/doc/texinfo/gmsh.html#Legacy-formats
Can you use a newer version of Gmsh?

What is the last version? I thought I had it

According to http://gmsh.info/ the latest version is 3.0.6.
You probably exported by accident into "Version 1" format -- this choice is in the "MSH Options" dialog that appears after you choose the filename -- try switching to "Version 2 ASCII".
This applies to version 2.12 of Gmsh but I assume it is not that different in newer versions.

Can you tell me which command line should I use in GLVis to visualize the "Version 2 ASCII" mesh?

glvis -m your-file.gmsh

@LeilaSalomon It reads only linear meshes. But I hope at some point there will be one built in.

@vikramsg Ok, so how can I make a mesh with elements of second order can be read by GLVis ?

@LeilaSalomon, for second order meshes, try exporting to VTK format. MFEM and GLVis can read second order unstructured VTK meshes.


Add other solutions section

I put a section like this on all my repositories because I think it's pretty handy for people trying to make a decision about what solution fits their use cases and preferences best.

A simple question aside. What's the benefit of a review request? Is there a special page where I can see all PR I got requested to review? This would be really handy. But as far as I know it's only another notification isn't it?

as far as I know it's only another notification

That's me as well :man_shrugging:

There's https://github.com/pulls/review-requested

Also, here

ping?


NEXT-20638 - Fix line item children smooth collapse

1. Why is this change necessary?
The collapse for the line item children currently isn't smooth because of wrong padding

2. What does this change do, exactly?
Refactors the styling so the collapse is smooth
3. Describe each step to reproduce the issue or behaviour.
Add line item with children to cart. When toggling the collapse you can see that it currently isn't smooth because of wrong padding (set on the collapse itself, instead of first div)
4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/NEXT-20638
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-20647
Please use this issue to track the state of your pull request.

hey @MelvinAchterhuis we've merged your changes, thanks for your contribution! :tada: :blue_heart:


Invalid characters in the request URL causes an ArgumentException to be raised

A request with invalid characters (e.g. /%3C [edit] which is URL decoded to <) causes an ArgumentException to be raised. This produces an HTTP 500 error to be returned to the client, rather than the expected  HTTP 404.
Below if the stack trace from the exception:
ArgumentException: Illegal characters in path.
System.IO.Path.CheckInvalidPathChars(String path, Boolean checkAdditional)
System.IO.Path.GetExtension(String path)
Microsoft.AspNet.StaticFiles.FileExtensionContentTypeProvider.TryGetContentType(String subpath, String& contentType)
Microsoft.AspNet.StaticFiles.StaticFileContext.LookupContentType()
Microsoft.AspNet.StaticFiles.StaticFileMiddleware.Invoke(HttpContext context)

Wondering if we should check for invalid path characters as part of the StaticFileContext.ValidatePath() method call in StaticFileMiddleware.Invoke(HttpContext context)? I would expect a method called ValidatePath to do something appropriate with invalid characters in the path, i.e. return false in this case.
The odd thing is the log output would be "The request path {Path} does not match the path filter". I have an feeling I might have misunderstood the purpose of the method due to the log output if it returns false and the comment // Check if the URL matches any expected paths

I wanted to take a whack at fixing this issue, but I think the solution might warrant some discussion.
For example, do we strip invalid characters from the request in the static file middleware? Do we check for invalid characters in ValidatePath(), return false if we find any, and change the error message outputted by the middleware after the call? Do we add a new method which checks for invalid characters, which the static file middleware calls, outputting an appropriate log entry?
I'd want to try and avoid sticking try/catches around the methods as we get a lot of first chance exceptions happening, potentially pausing the debugger, etc.  Invalid characters aren’t exceptional, we should try to detect the problem and handle it gracefully.

Let me know if that potential solution is good enough for a pull request. I've renamed the existing ValidatePath() method to MatchPath() as that's what it's doing. Then added a new ValidatePath() method that will check for invalid characters (and potentially other path validation in the future).

Related: https://github.com/aspnet/FileSystem/pull/147

Just looked through that issue @Tratcher, and I agree with @davidfowl; they're kinda related in that they're both trying to deal with odd characters, but I think both issues need their separate fixes. Let me know if https://github.com/brentnewbury/StaticFiles/commit/11e02d1a513f41a8dc704027dd6a50e25f64054e seems like a good candidate for a pull request and I'll create one.

I'd want to try and avoid sticking try/catches around the methods as we get a lot of first chance exceptions happening, potentially pausing the debugger, etc. Invalid characters aren’t exceptional, we should try to detect the problem and handle it gracefully.

Yes!
@brentnewbury do send a pull request for your change. The only thing I'm not sure about is if the static file handler should pass through to the next middleware. I could see that being ok though.

I let it pass through so that other middleware can try to handle it, better to passthrough than to swallow it up. Maybe in the future, if it's asked for, we could make it optional. I'll submit that pr now.


Add support for docker healthcheck 👼

This add the simplest support for HEALTHCHECK for the docker provider.

React to health_status events
Filter container that have a health status and that are not healthy

It works like the other provider that support healthchecks (marathon for example) — if healthcheck are not supported, then it work as usual and if a container supports it, it filters the container if it's not healthy.

[x] Update documentation (it's actually not documented on other providers, so…)
[x] Add integration tests (too much extra work on that, I need to update libkermit and add support for a testRequire thingy with a check on docker >= 1.12 — will do in a follow up)

/cc @containous/traefik
🐸
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

Tested, it works nicely, thanks! 💯
Since the web UI shows all backend containers and not only the filteredContainers, I added some logging in the build I tested - this way I could see what was happening (otherwise my unhealthy containers were showing up along the healthy ones).
A nice addition would be to display the health status in the web UI. I think this PR is fine without this feature and I don't want to hijack it, but I might try to add this as a separate PR.

Fixes #666


Shoal Client Issue - Prevents CERNVM from Booting

The shoal client on the CERNVM seems to have an issue if the shoal service is down and prevent the cernvm from booting further

There are fallback mechanisms that should prevent this from happening. I guess we should test them.
For the application repositories, we have two levels of fallback in case shoal doesn't work:
in shoal_client.conf:
default_squid_proxy = http://kraken01.westgrid.ca:3128;http://cernvm-webfs.atlas-canada.ca:3128
and in /etc/cvmfs/default.local:
CVMFS_HTTP_PROXY: kraken01.westgrid.ca:3128
For the OS repository, I think there is just one level of fallback. The Shoal client does not come into play here.
CVMFS_PAC_URLS=http://shoal.heprc.uvic.ca/wpad.dat
CVMFS_HTTP_PROXY=auto;http://kraken01.westgrid.ca:3128;DIRECT
Shoal client is specifically designed not to fail in this way, but I guess we should test it.
For the OS repository I would expect it to fallback to kraken01 after a timeout; again this would be good to test.
It seems more likely that the incident can be explained by a general network outage affecting the cloud. Any general network problem on a cloud that blocks access to an external squid server will also necessarily block access to the external Shoal server (as well as disconnecting VMs from the condor server). In this situation, loss of access to the squid is the real problem, and loss of access to Shoal is just coincidental.
In this case it's not definitive whether there was a network problem on datacentred, but it has happened before and it would seem to explain the observations.

So you did some testing that seemed to confirm this was not an issue?

I don't think this is a (confirmed) problem...


populate_db: While populating test database, add a realm emoji which can be used for testing.

Since a number of tests relating to realm emojis require a realm emoji to be present, one needs to add code for ensuring atleast a realm emoji is present every time a test related to realm emoji is added. This makes testing code repetitive. This issue aims at adding a default realm emoji in test database while populating it and removing the duplicate code for ensuring the presence of atleast one realm emoji in tests. It is follow-up task of PR #5728.

@zulipbot label "area: testing-infrastructure"
@zulipbot claim

@timabbott Would it make more sense to populate a default realm emoji to both automated as well as manual testing database or to only automated testing database(I am in favor of adding it to both). If we go for adding it to both the databases then we will need a test image for the realm emoji since the image file we have been using till now in the tests is the zulip logo which will collide with :zulip: emoji.

Both is the right answer.  For a test image, we can probably do something already in the repo like static/images/checkbox-valid.svg?


Popover does not disappear in Playground UI after latest Chrome update

Hello,
We have recently encountered an issue where the UI pop-up hints in the Playground never disappear while using the interface. From what we have gathered, this appears to be caused by a change in the latest version of Chrome.
Here is a screenshot of an example, pardon the blanked out field names :)

Note the Date and the BigInt popups never disappear until refreshing the page.
We have noticed that the following error message is now appearing in the developer console:

We are running Chrome Version 127.0.6533.73 (Official Build) (64-bit) on Windows 11.
Currently our stack is on graphql-go-tools version 1.64.0, but I can confirm that upgrading to the latest version of V1 does not resolve the issue.

Hi @wcwright, the playground is outdated and no longer maintained. You can create a PR to upgrade the playground v1 or migrate to V2 which uses CDN.

Ah, OK. Thank you for the follow-up, we will take a look!


limit of bluetooth device can be connected simultaneous

Hello Team, i tried to connect heart rate monitor with your library that display the bpm in android tablet or UWP
When Device Discovered is up to seven devices in android then the eighth can't connect.
In UWP i had the same problem but the problem is up to eight device and can't connect to nineth
I made a reasearch in internet how can connect up to 15 devices ,but everyone said that reasons make that problem.

The library of bluetooth has limit.
The Bluetooth adapter has limit.

First of all i want to know if we have some limit in your library that is my first question.
My second question is how can i search for a bluetooth adapter that can connect up to 15 devices. Which is the search condition that i must search for it.
Can you suggest me some adapters that can connect up to 15 devices simultaneous and is compatible with your library
Thank you

Hi @Sujiyama,
I'm the friendly issue checker.
It seems like (100.00 %) you haven't used our issue template :cry: I think it is very frustrating for the repository owners, if you ignore them.
If you think it's fine to make an exception, just ignore this message.
But if you think it was a mistake to delete the template, please close the issue and create a new one.
Thanks!

Hello AskBojesen,
Thanks for your response.

Great then! I suggest to close this issue!?

Android connection limit is even more confusing and even depends on Manufacturer

Thank you all for your responses, i try to understand how other developers can connect more than 7 devices.
Some of them present that can connect up to 10 and more with bluetooth.
I think the limit is not coming from OS but from adapter
If you research for Chileaf BLE Data Receiving Portable Long Range Wireless USB Stick Dongle they said that you can connect up to 60 members’movement data can be collected via Bluetooth.
I don't want to send you link because i don't want to to promote this bluetooth adapter. Only i try to find a solution of my problem.
If you allow me i will send you some links.
Also many companies develop softwares that work with bluetooth heart rate monitors and receive data with more than 7 devices.
I try to find a solution because your library is so great and i don't want to work with ANT+ solution.
Our project is for customer that have GYM and we develop a software that show the heart rate of BLE devices while training in class.
The maximum people in class will be 10-14 people, with that limit we discovered, we stucked and we don't know how to find a solution.
That is the reason that i open ticket to you.
Do you know some trick with your library that can we connect 4 more and avoid the limit?
Thank you

I think this is more a question regarding OS or adapter limits, rather than a bug report. I'm transferring it from an 'issue' into a 'discussion'.


Should return 404 for non-existent players

Valid (i.e. existing) players without a skin should return 200, See #111 but players that do not exist should return a 404 IMO.
We need to implement #106 first to detect the existence of players by name.
/cc: @confuser

I thought we could implement this without #106 because Mojang's API doesn't return 200 OK when the user does not exist. The problem is that it returns the same when the user does exist, but has no skin, which is quite a different thing.


Problem with remote collection since toplevel was added

          So, all powa-archivist extensions were updated to 4.2.0. But now I came to another issue with the collector :

2023-11-09 09:47:15,815 hostname:5432 DEBUG : Calling public.powa_statements_src(0)...
2023-11-09 09:47:15,839 hostname:5432 WARNING: Error while inserting data:
invalid input syntax for type bigint: "t"
CONTEXT:  COPY powa_statements_src_tmp, line 1, column queryid: "t"

In the repository DB log :
2023-11-09 09:47:15 CET pid=2545344 : <IP_ADDRESS>(35058) - powa CONTEXT:  COPY powa_statements_src_tmp, line 1, column queryid: "t"
2023-11-09 09:47:15 CET pid=2545344 : <IP_ADDRESS>(35058) - powa STATEMENT:  COPY powa_statements_src_tmp FROM stdin

To my surprise, this happens alos on PostgreSQL 15 instances.
Function powa_statements_src outputs the toplevel column before queryid, and in powa_statements_src_tmp, toplevel is the last column of the table. Isn't there a mismatch in the column order between the data output function and the integration in the powa repo ?
See :
https://github.com/powa-team/powa-archivist/blob/b5835fe0e5cc122b80309c0d0bcdd07e55621838/powa--4.2.0.sql#L2419C1-L2419C1
And :
https://github.com/powa-team/powa-archivist/blob/b5835fe0e5cc122b80309c0d0bcdd07e55621838/powa--4.2.0.sql#L756
Originally posted by @frost242 in https://github.com/powa-team/powa-collector/issues/18#issuecomment-1803445516

          Just did a quick hack on the repository server and recreated the extension (yep, history is lost, but it's a non-production system, it's OK). The toplevel was physically moved at the same place as in pg_stat_statements :

root@powahp1:~# diff /usr/share/postgresql/16/extension/powa--4.2.0.sql  ~postgres/powa--4.2.0.sql
761d760
<     toplevel boolean NOT NULL,
783c782,783
<     wal_bytes numeric NOT NULL
---
>     wal_bytes numeric NOT NULL,
>     toplevel boolean NOT NULL

Then a quick test with one remote instance works OK :
2023-11-09 10:39:22,895 hostname:5432 DEBUG : Connecting on repository...
2023-11-09 10:39:22,910 hostname:5432 DEBUG : Connected.
2023-11-09 10:39:22,915 hostname:5432 DEBUG : Working on module pg_stat_kcache
2023-11-09 10:39:22,915 hostname:5432 DEBUG : Calling public.powa_kcache_src(0)...
2023-11-09 10:39:22,921 hostname:5432 DEBUG : Working on module pg_stat_statements
2023-11-09 10:39:22,921 hostname:5432 DEBUG : Calling public.powa_databases_src(0)...
2023-11-09 10:39:22,923 hostname:5432 DEBUG : Working on module powa_stat_user_functions
2023-11-09 10:39:22,923 hostname:5432 DEBUG : Calling public.powa_user_functions_src(0)...
2023-11-09 10:39:22,924 hostname:5432 DEBUG : Working on module pg_stat_bgwriter
2023-11-09 10:39:22,924 hostname:5432 DEBUG : Calling public.powa_stat_bgwriter_src(0)...
2023-11-09 10:39:22,925 hostname:5432 DEBUG : Working on module pg_qualstats
2023-11-09 10:39:22,925 hostname:5432 DEBUG : Calling public.powa_qualstats_src(0)...
2023-11-09 10:39:22,932 hostname:5432 DEBUG : Calling SELECT pg_qualstats_reset()...
2023-11-09 10:39:22,933 hostname:5432 DEBUG : Working on module powa_stat_all_relations
2023-11-09 10:39:22,933 hostname:5432 DEBUG : Calling public.powa_all_relations_src(0)...
2023-11-09 10:39:22,935 hostname:5432 DEBUG : Working on module pg_stat_statements
2023-11-09 10:39:22,935 hostname:5432 DEBUG : Calling public.powa_statements_src(0)...
2023-11-09 10:39:22,949 hostname:5432 DEBUG : Calling powa_take_snapshot(1)...
2023-11-09 10:39:22,977 hostname:5432 DEBUG : Committing transaction

The UI seems to behave correctly, but I didn't do an extended test.
Originally posted by @frost242 in https://github.com/powa-team/powa-collector/issues/18#issuecomment-1803487443

You're right, there's another problem here :(. Unfortunately I missed it as I'm mostly working on the v5 for now, where this problem has been fixed in some other refactoring.

Do you need me to provide a PR to update the extension to version 4.2.2 ?

I don't think that your approach is possible unfortunately.  The new version should be both installable and upgradable, and should give the same result each time (I have a tool to automatically check it).   It means that new column has to be added at the end of the table.  I don't how I missed it, I'm pretty sure that I tested a full remote setup with the 4.2.x version.
I will work on that, and do more thorough testing to avoid yet another big bug like that on the 4.2.x series.

Thanks. Please let me know here, as I've hacked the extension to use the correct column order and I want to put the extension in good shape for the next upgrade.

I just pushed d61e829b7af37b234766585e3a2db615c97870cb which should fix the problem and work for both installed and upgrade extensions.  Do you have some non-production system where you can double check it?

One of our clients tested that commit because they have the same problem after upgrading to 4.2.0, but they have got an error:
ERROR:  syntax error at or near "timestamp"
LINE 2: ￼    OUT ts timestamp with time zone,

We will try to reproduce it on our lab to give you more info.

Thanks for the report @JosefMachytkaNetApp .    INTO STRICT v_pgss
        FROM pg_extension
        WHERE extname = 'pg_stat_statements';

        IF (v_pgss[1] = 1 AND v_pgss[2] >= 10) THEN
            RETURN QUERY SELECT now(),
                pgss.userid, pgss.dbid, pgss.queryid, pgss.query,
                pgss.calls, pgss.total_exec_time,
                pgss.rows, pgss.shared_blks_hit,
                pgss.shared_blks_read, pgss.shared_blks_dirtied,
                pgss.shared_blks_written, pgss.local_blks_hit,
                pgss.local_blks_read, pgss.local_blks_dirtied,
                pgss.local_blks_written, pgss.temp_blks_read,
                pgss.temp_blks_written, pgss.blk_read_time, pgss.blk_write_time,
                pgss.plans, pgss.total_plan_time,
                pgss.wal_records, pgss.wal_fpi, pgss.wal_bytes, pgss.toplevel
$PROC$ LANGUAGE plpgsql;;nts_src_tmp pgss WHERE srvid = _srvid;s.toplevelme,NSACTION|COMMIT PREPARED|ROLLBACK PREPARED)'
CREATE FUNCTION

psql displays a bit of a mess, but in any case it doesnet complain with the function prototype.

I noticed that your error message says:
LINE 2: ￼ OUT ts timestamp with time zone,

with a weird character before the OUT.  Maybe you have a non-breakable space or something like that that leads to the problem?

I've been having the same problem, and tested the 4.2.2 git version. It solves the problem. Note: I only did a create extension, not an update, so I didn't test the update path

We had pinned powa to 4.1.4.
The patch wors fine on 4.2.1.

Thanks @marco44 and @jw1u1 for testing!
I just released the version 4.2.2 with that fix.  This will hopefully be the last problem on the 4.2.x versions.


RunSequence with callback

Hi, is it possible to set a callback for when $.Velocity.RunSequence(queue) is complete?  If so, how do you do it?  I can't seem to get it going.
I've tried
$.Velocity.RunSequence(queue).then(function() {
    console.log('test');
});

with no success.

+1 for making RunSequence thennable.
I couldn't find an example of inserting a function in a sequence as you suggest — is this still forthcoming? I've just bought your book & although it looks great & am excited to read it I haven't yet found any info on a good strategy for this (it seems to be missing info about promises in general).
Thanks for the great library.

+1 RunSequence.then()

+1 RunSequence.then()

+1 RunSequence.then()

+1 RunSequnce.then()

+1 RunSequence.then()

+1 RunSequence.then()

@julianshapiro was the ability to include a function as an array item ever added? Better yet, support for RunSequence.then() added?

Anything at all?

Might get this in v1 - depends if I have time, not quite a bug, but close enough to be worth it ;-)

@Rycochet my hero

My issue is purely time, got about three half evenings free for the next two weeks - but if anyone else can fix it and send in a pull request I'd get enough time to check it ;-)

I wrote a quick wrapper because I needed this. Including here for anyone that may find it useful and for @Rycochet to look at in case you want to add this is in some format.
/***************************
 Velocity UI Pack extension
 ***************************/
(function(factory) {
	"use strict";
	/* CommonJS module. */
	if (typeof require === "function" && typeof exports === "object") {
		module.exports = factory();
		/* AMD module. */
	} else if (typeof define === "function" && define.amd) {
		define(["velocity"], factory);
		/* Browser globals. */
	} else {
		factory();
	}
}(function() {
	"use strict";
	return function(global, window, document, undefined) {

        var Velocity = global.Velocity;
        var $ = Velocity.Utilities;

        /**
         * RunSequence utility function to wrap the standard
         * Velocity RunSequence so that a promise can be returned that
         * resolves when all animations have completed.
         *
         * @param Array sequence The sequence of animations
         * @return Promise a promise that completes when all animations have completed
         * @uses Velicity.RunSequence
         *
         */
        Velocity.RunSequencePromise = function( sequence ){
            var deferred = $.Deferred();
            var total = 0;
            var done = 0;

            $.each( sequence, function( i, animation ){
                total++;
                if( !animation.o ) animation.o = {};
                var old = animation.o.complete;
                animation.o.complete = function(){

                    if( old && old instanceof Function ){
                        old.apply( this, arguments );
                    }

                    if( ++done === total ){
                        deferred.resolve();
                    }
                };
            });

            Velocity.RunSequence( sequence );
            return deferred.promise();
        };
    }((window.jQuery || window.Zepto || window), window, (window ? window.document : undefined));
}));

RunSequence has been removed in favour of css animation-like sequences


[bug] - Optimize DiskBufferReader's Seek method to reduce memory usage

This PR introduces an optimization to the Seek method of the DiskBufferReader. Previously, when seeking beyond the current read position, the method was inefficient in terms of memory usage, as it required allocating and managing large byte slices to skip unwanted data. This could lead to excessive memory consumption, especially when large skips were involved.
Key Changes:

Elimination of Large Buffer Allocations: The new implementation removes the need for allocating large byte slices (trashBytes). Instead of reading data into a buffer and then discarding it, we now directly discard the data using io.Discard.
Use of io.CopyN with io.Discard: To efficiently skip over bytes, io.CopyN is used in conjunction with io.Discard. This method allows us to precisely skip the desired amount of data without actually loading it into memory.

Bechmarks:

I feel like I must be missing something obvious because it reads to me like you used to throw away data and now you don't, which seems like a pretty big change to not be called out in the PR description. Where have I gone wrong?

Whoops, yea my bad.

why aren't you using io.Discard? is it just beacuse we'll eventually want to read those bytes?

That's what I initially thought, but now that I look at it again, i'm not so sure. I think we could use io.Discard.

@bill-rich any reason why we can't use io.Discard here with io.CopyN vs a Read?

why aren't you using io.Discard? is it just beacuse we'll eventually want to read those bytes?

That's what I initially thought, but now that I look at it again, i'm not so sure. I think we could use io.Discard. I think the re-reads would take place from the tempFile. I'll confirm with Bill.

Well... looks like even using discard does a read from the reader 🤣 .


VoIP: Fallback to matrix.org STUN server with a confirmation dialog

We do not fallback anymore to the Google STUN server but users cannot make call anymore if their HS admin has not set up a STUN server.
We should fallback them to the STUN server available on matrix.org but with their permission.
We should do it when accepting or creating a call. If not possible, that will be another popup at app startup :/
The copy for the permission will come from riot-web: https://github.com/vector-im/riot-web/issues/10173#issuecomment-519874468 and next.

Kamino cloned this issue to vector-im/riot-ios


There is only a Dark mode, no back button, no responsive.

Some issues I'd like to fix:

The theme is dark only - users who don't like to read white on black may not enjoy using the app.
The example pages don't have a link back to the homepage. If users come from Google to an example page, they may never know that there are other examples to view.
The site is not responsive for mobile.

hello sir, can u assign this issue to me?

Hi there, it's already done in this PR.


JetBrains: Opening some specific older chats from chat history redirects user to /cody/subscription page

IDE Information
PyCharm 2023.3.4 (Community Edition)
Build #PC-233.14475.56, built on February 26, 2024
Runtime version: 17.0.10+1-b1087.17 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 11.0
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 12
Registry:
ide.experimental.ui=true
Non-Bundled Plugins:
com.sourcegraph.jetbrains (6.0.10-nightly)
Bug Description
Platform: Windows
Steps to reproduce:

Have Cody plugin installed in your JB IntelliJ editor
Login to Cody with a Pro account
Go to Chat History and open chats one by one

Observed behavior:
While opening some chats, user will be redirected to /cody/subscription page.
Expected behavior:
User should not be redirected to  /cody/subscription page
Additional context
Video link:
https://drive.google.com/file/d/1ajR7laFq38sVZmLzLs5xnGieGD5FeYH0/view?usp=drive_link

@jay-fibi going to close this as we're using webviews now, please re-file if its still a problem


Issue with Magento 2.2.6

This error occurs when trying to enable this extension via the command line when using Magento 2.2.6.
Possibly related to https://github.com/olegkoval/magento2-regenerate_url_rewrites/issues/32?
PHP Fatal error:  Declaration of Magento\CatalogUrlRewrite\Observer\UrlRewriteHandler\Proxy::generateProductUrlRewrites(Magento\Catalog\Model\Category $category) must be compatible with Magento\CatalogUrlRewrite\Observer\UrlRewriteHandler::generateProductUrlRewrites(Magento\Catalog\Model\Category $category): array in generated/code/Magento/CatalogUrlRewrite/Observer/UrlRewriteHandler/Proxy.php on line 7

This is an issue with Magento: https://github.com/magento/magento2/pull/17768

Same here on Magento 2.2.6

@ctadlock , @mageho Thank you for your help with this issue.
Closed.


Real project example

lets add an example of tracehub in case of software development project for some pizza shop

duplicate of #163


MultiThread with synchronized

MultiThreading with synchronization to avoid race condition.

@Prakash-sa please add any document related to this code so that it can be easy to understand. A link as a comment in the code will work too.


Remove "manage social accounts"?

Just thinking this will be confusing if people explore their profile..... can we remove or hide?

good idea, will hide

thanks!


FIX: Raise exception for default value of baseline and tmin=0 in Epochs

Closes #6616
Raise an exception when using the default value of baseline=(None, 0) with tmin=0 in Epochs. The result would be one-sample baseline which is not desired in most cases. One should use baseline=(0, 0) if this is desired.

thx @milan-rybar
can you see why CIs are failing?

The following tests use the default value of baseline with tmin=0. I am not able to tell whether that's the correct approach or not. Could someone have a look?

tests for beamformer
mne.minimum_norm.time_frequency.compute_source_psd implementation
Some tests in mne/tests/test_epochs.py use baseline=(None, 0) as 1-sample baseline (which could be easily changed to baseline=(0, 0).

Pushed a commit

@milan-rybar CIs are happy (other than AppVeyor which is a spurious failure), can you push a commit with a whats_new.rst update in the BUG section?

@larsoner Thank you for so quickly updating the tests.
I added a commit with the change to whats_new.rst.

Thanks @milan-rybar, merged after changing mne.epochs.BaseEpochs to mne.BaseEpochs in the whats_new.rst (we need to link using the same namespace where things are listed in python_refreence.rst, and BaseEpochs is in the mne namespace)


Support for cimfs

Includes cimfs in containerd main.go, vendors latest go-winio & golang.org/x/sys
Signed-off-by: Amit Barve<EMAIL_ADDRESS>

We need to merge corresponding cimfs PRs in hcsshim & containerd first and the update this PR to revendor those commits and only then merge this.

Opening another PR in new cri clone at github.com/kevpar/cri


lvgl approach "event driven" instead of "polling"?

Hi!
As far as I understood, the operation of the LvGL library is the following:

perodic call of lv_task_handler()
perodic call of lv_tick_inc()

The device I am working on will have a very limited LCD display and the only input device will be a rotary encoder with push button. Moreover, it can be expected that this interface is rarely used as the device is meant to be controlled via ethernet.
So I was thinking if it is possible to change this basic operation principle from polling to a event driven approach. That is the lv_task_handler() needs only be called on an event from input device or an update of the displayed data.
Or is there anything like a sleep mode?
Thanks in advance for your help.
BR,
Stephan

Hi,
Theoretically, it's possible. So there two main things as you mentioned:

Input device (encoder) read
Display refresh

If the GUI will change only if you change something by the encoder then it seems okay to call lv_task_handler() if the encoder is changed. It will call your indev_read function to handle the encoder.
However, you should call lv_task_handler() twice because it might happen that the display refresher task runs first, sees nothing happened (yet), and then comes the input device handling which changes the GUI. But in the next call of lv_task_handler() the refresher task will be called.
Keep in mind the features which have an effect as time passes (e.g. animation) won't work. To handle long press with the encoder lv_task_handler() should be called during pressing.
It might be required to lower the values of LV_INDEV_READ_PERIOD (in lv_conf.h) to surely not miss a fast click. If LV_INDEV_READ_PERIOD is not elapsed since the last call of lv_task_handler then the input device handler task won't be called.
If the data from Ethernet can change the GUI then lv_task_handler() should be called after you refreshed the content.
Independently from this lv_tick_inc(x) still need to be called periodically.

Could you elaborate on this.

Why is lv_tick_inc(x) necessary?
How much control do we have over when the "display refresher" runs?
Can we sleep for a long time, detect a button press in non-lvgl code, only then run the display refresher, then run lv_task_handler which is told of the button press (which was saved to a queue for instance). and only then run the display refresher again?
Is there a way to know whether all animations have ended, so that we can re-enter sleep?

Why is lv_tick_inc(x) necessary?
The library runs task periodically (screen refresh, input device read, animations etc) These needs to know how much time has elapsed.

How much control do we have over when the "display refresher" runs?
In lv_conf.h LV_REFR_PERIOD sets its period time

Can we sleep for a long time, detect a button press in non-lvgl code, only then run the display refresher, then run lv_task_handler which is told of the button press (which was saved to a queue for instance). and only then run the display refresher again?

If you don't have animations it seems possible. If the GUI is idle then it only needs to read the input devices to see if something happens. So if you call lv_task_handler only while user actions are happening then it could work, but need to be tried out.

Is there a way to know whether all animations have ended, so that we can re-enter sleep?
Not now, but I can add a function to get it. It's quite simple.

In the practice, you don't need to weak up from sleep to call lv_tick_inc() periodically. It's enough to call it on wake up with big value to be sure all the task will run once on the first call of lv_task_handler(). So for example lv_tick_inc(1000) should be fine on wake up.
However, as you mentioned while the button or touchpad press lasts you need to call lv_tick_inc() periodically.
If "officially" guaranteed" means that this opportunity will be stated in the documentation and the upcoming features won't break it then yes, I can guarantee that for the foreseeable future. But a confirmation from your side is required to state this responsibly.

Yeah, I am sorry, my phrasing was quite poor. I did indeed just wanted to be able to see this behavior be documented and not break in the upcoming versions.

Okay, I will update the documentation!
I was able to test in the PC simulator.
I modified the mouse_handler() in mouse.c:
void mouse_handler(SDL_Event *event)
{
    switch (event->type) {
        case SDL_MOUSEBUTTONUP:
            if (event->button.button == SDL_BUTTON_LEFT)
                left_button_down = false;
            break;
        case SDL_MOUSEBUTTONDOWN:
            if (event->button.button == SDL_BUTTON_LEFT) {
                left_button_down = true;
                last_x = event->motion.x;
                last_y = event->motion.y;

            	lv_tick_inc(1000);                        /*Execute tasks on button press*/
            	lv_task_handler();
            }
            break;
        case SDL_MOUSEMOTION:
            last_x = event->motion.x;
            last_y = event->motion.y;

            break;
    }
}

And I modified the while(1)  like this:
	bool print_sleep = true;
	bool print_wake_up = true;
	while(1) {

		if(lv_indev_get_inactive_time(NULL) < 1000) {
			lv_task_handler();
			lv_tick_inc(5);
			print_sleep = true;
			if(print_wake_up) {
				printf("Wake up\n");
				print_wake_up = false;
			}
		} else {
			print_wake_up = true;
			if(print_sleep) {
				printf("Sleep\n");
				print_sleep = false;
			}
		}
		usleep(5000);
	}

And of course, I removed the tick increment task.
So now the system wakes up on button press, and go sleep after 1000 ms inactivity.

MEanwhile I found that it's better to use lv_tick_inc(LV_REFR_PERIOD); in mouse_handler because the too big number can break some animations.

Hi again,
Thanks for your answers so far.
Is there a possibility to check if there are animations currently running on the screen? Or that lv_task_handler had actually something to do?

I added the lv_anim_count_running function to dev-5.3.
There is no explicit way to know that the library has nothing to do but by default there is only 3 tasks:

screen refresh: do something only if something happens
indev read: can be checked with lv_indev_get_inactive_time
animation: can be checked with the new function: lv_anim_count_running

I updated the documentation:
https://github.com/littlevgl/lvgl/wiki/Porting#task-handling

@StephanSy It's a bit ugly but maybe you could add some timestamp variable to your driver?

I updated the documentation as you suggested.
The suggestion of @embeddedt  seems quite flexible.

As this use case is added to the documentation I close this issue.


CallKeepDidDisplayIncomingCall doesn't trigger

Application state - foreground
This is my configuration:
_callKeep.on(CallKeepPerformAnswerCallAction(), answerCall);
_callKeep.on(CallKeepDidPerformDTMFAction(), didPerformDTMFAction);
_callKeep.on(CallKeepDidReceiveStartCallAction(), didReceiveStartCallAction);
_callKeep.on(CallKeepDidToggleHoldAction(), didToggleHoldCallAction);
_callKeep.on(CallKeepDidDisplayIncomingCall(), didDisplayIncomingCall);
_callKeep.on(CallKeepDidPerformSetMutedCallAction(), didPerformSetMutedCallAction);
_callKeep.on(CallKeepPerformEndCallAction(), endCall);
_callKeep.setup(<String, dynamic>{
  'ios': {
    'appName': 'Example',
  },
  'android': {
    'alertTitle': 'Permissions required',
    'alertDescription':
        'This application needs to access your phone accounts',
    'cancelButton': 'Cancel',
    'okButton': 'ok',
  },
});

My didDisplayIncomingCall method
Future didDisplayedIncomingCall(
CallKeepDidDisplayIncomingCall event) async {
print("didDisplayedIncomingCall");
}
My logs:
INFO: received event "CallKeepDidDisplayIncomingCall" {handle:<PHONE_NUMBER>7, hasVideo: 0, localizedCallerName:<PHONE_NUMBER>7_509175, payload: , callUUID: 78A32447-DF18-4399-AB72-09EFFF5FB039, fromPushKit: 1, error: }
received event "CallKeepPerformAnswerCallAction" {callUUID: 78a32447-df18-4399-ab72-09efff5fb039}
CallKeepPerformAnswerCallAction - triggered as well on dart side, but CallKeepDidDisplayIncomingCall not triggered at all

I have same issue. Don't know why it can't receive payload in Flutter

Did yo set the calling account, manifest and setup correctly? Does the
example app work for you?
On Sun, 18 Oct 2020, 13:58 Barry0501<EMAIL_ADDRESS>wrote:

I have same issue. Don't know why it can't receive payload in Flutter
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter-webrtc/callkeep/issues/17#issuecomment-711164398,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG66EAISTMBAOZTMLJG53SLLQ7HANCNFSM4SHMERFA
.

I am working on IOS, android works perfectly

So what's different between your apk and the example app?

I register to listen event CallKeepDidDisplayIncomingCall, I notice it can log in debug console but it cannot callback in Dart. Please tell me if I do wrong somewhere.

How can I show incoming call when app is closed ! I have implemented FCM , but when app is in background I got Not implemented platform exception !


InsertEnd operator

just sharing an experimenting

I see you have CountDown embedded. 😆

📝 This is for #460.

@atifaziz I adjust the documentation. IMHO this PR is ready for merge. Some more in your opinion?

@atifaziz I adjusted the documentation, renamed operator to Backsert and updated README.
Something before merge in your opinion?

All tests should use a TestingSequence to make sure Backsert disposes and doesn't iterate the source sequence more than once.

done

@atifaziz is it ready for merge?


Greak work！Can CentOS be supported?

Greak work！Can CentOS be supported?

Hi @huapohen , we are currently testing deepdataspace on Ubuntu, Windows and macOS only.
But you can try our docker compose solution:
# clone the source code
git clone https://github.com/IDEA-Research/deepdataspace.git

# prepare dataset directory(where you put all your datasets inside)
mkdir -p datasets
export DDS_DATASET_DIR=$PWD/datasets

# choose a visiting port for DDS
export DDS_PORT=8765

# start DDS with docker compose
cd deepdataspace
docker compose up

If everything goes well, you can start visiting DDS at http://<IP_ADDRESS>:8765.
I hope this helps and feel free to reach out for any further problems.

Hi @huapohen , we're temporarily closing this issue due to inactivity.


Errors Validating and Uploading to Appstore

By upgrading to Swift 3 and XCode 9, the library causes this error while uploading or validating with the app store:
Unexpected CFBundleExecutable Key. The bundle at 'Payload/Europiel.app/Frameworks/JGProgressHUD.framework/JGProgressHUD Resources.bundle' does not contain a bundle executable. If this bundle intentionally does not contain an executable, consider removing the CFBundleExecutable key from its Info.plist and using a CFBundlePackageType of BNDL. If this bundle is part of a third-party framework, consider contacting the developer of the framework for an update to address this issue.

After removing the CFBundleExecutable Key and using CFBundlePackageType of BNDL, the new error is:

invalid Bundle Structure - The binary file 'Europiel.app/Frameworks/JGProgressHUD.framework/JGProgressHUD' is not permitted. Your app can’t contain standalone executables or libraries, other than a valid CFBundleExecutable of supported bundles. Refer to the Bundle Programming Guide at https://developer.apple.com/go/?id=bundle-structure for information on the iOS app bundle structure.
** Any workaround to this?

Are you using release version 2.0? Because 2.0 no longer contains JGProgressHUD Resources.bundle.

I was using an old version of JGProgressHUD, now testing with the newest one. I'll let you know about the outcome. Thnkx

Ok, using the newest version, and adding it manually as a framework seems to have fixed the problem. Thank you for your insight.