Feature request: Redirect to HTTPS

Current Behavior
We are running this excellent of oauth2_proxy on Heroku using this build-pack: https://github.com/cfra/heroku-buildpack-oauth2-proxy
oauth2_proxy runs as reverse proxy in front of an nginx which serves static files.
When I access the heroku application using plain HTTP, the "login with github" button will be shown and the cookie will be set, all without encryption.
After authenticating with GitHub, we get redirected to the oauth2 callback, but this time via HTTPS, because such is the configuration in GitHub. (And oauth2 spec recommends to only use HTTPS anyway)
This will lead to oauth2_proxy showing an error that a required cookie is missing, because the cookie set via HTTP is not taken into the HTTPS session.
After clicking "login" again and doing everything over HTTPS, it works fine.
Possible Solution
As the operation of oauth2_proxy as a user-facing reverse proxy seems to be a possible mode of deployment, it would be useful if there was an option to redirect all traffic to HTTPS.

I would love to fix this issue


Limit the number of images in a summary [feature request]

When working on dense prediction tasks it is very convenient to show the predictions of the network in TensorBoard to evaluate them qualitatively during training. Even if TensorBoard shows only a subset of these images, all the summaries that have been saved since the beginning of times are kept on disk. The occupied space grows with the size of the dataset, the sampling frequency and the number of experiments, often resulting in a huge waste of space. When the disk space is limited, this can heavily limit the number of experiments logs one can run simultaneously and/or keep stored.
It would be great to have a way to either limit the number of images saved on disk or to remove some of them (e.g., for old experiments). Here is what I suggest:

Add an argument to tf.Summary.Image to define how many images should be kept on disk (e.g., buffer_size or steps_to_retain). Also add an extra argument (e.g., retain_strategy) to select the strategy to define which images should be kept (e.g., keep latest, sample uniformly, ..).
Add a function to remove some of the stored images programmatically.

This feature request is partially related to https://github.com/tensorflow/tensorflow/issues/5039
Also related to this SO thread: https://stackoverflow.com/questions/41543954/remove-image-outputs-from-tensorboard/41690170

I don't believe we have any immediate plans for working on this, so I'm going to mark this as Contributions Welcome, with the expectation that this might be updated as things towards #5039 happen.
If anyone does want to assist with this, it may make sense to discuss some design here before investing too much time in implementation.
Thanks!

Hey guys, I have been working on TF for about 3 months now and would like to make a contribution. I can spare about 30 minutes every day. Please let me know what is the qualifying criteria for contribution.
Of the suggested approaches, I think the first approach works better, since there are future plans to display more than the current subset of images contained in the tensor, as per the SO thread.
Instead of adding new API parameters - can the max_output parameter be used to determine the summary to be stored on the disk ? Going by the documentation, max_outputs defines the max number of batch elements to generate images for.

@fvisin The summary doesn't really have any control or influence over how things are retained on disk. Basically, everything written to disk stays there forever until you manually remove it.
There's some work on the TensorBoard side to switch TB over to using a sql database on the backend, at which point it would be feasible to just delete the old event files directly because a subset of their data will have already been ingested into TensorBoard.
If you want to write code for sweeping and downsampling old event files, you're welcome to, but I think it would be better to wait and see how things play out with converting the TensorBoard backend to sql. Pinging @jart so she can chime in on this; also closing as infeasible.
@pbanavara, that's awesome that you want to contribute. I don't think this one would be the best one to implement. There's another TensorBoard issue here which a number of people thumbs-up'd and I think would be a good candidate for an external pull request.

Thanks @dandelionmane , I look forward to an sql backend for Tensorboard. Is there a PR or an issue I can subscribe to, to follow the advancement? I couldn't find any.

I would take a look here https://github.com/tensorflow/tensorboard/issues/92


WebSocket support?

This seems like it would probably be a pretty big feature to add, but have you thought through web socket support? I'm kinda fuzzy on how I would even want it to work, but I might be willing to build it if there was a clear path forward

I've thought about it, been waiting for someone to raise an issue before I gave it any more time. Are you looking to proxy the websocket connection itself, or is there some other use case?

Yeah, I have a server with both http test endpoints and a web socket live layer, and was trying to use tychus for it. I’ve instead been using tychus for the http endpoints but making the websocket connection directly to the underlying server.
Honestly, the only real downside is that I need to put two hosts into config instead of one

Should be possible, would need to use Hijacker (https://golang.org/pkg/net/http/#Hijacker) when the request is sending upgrade headers

I might take a stab at this sometime this week or next.


New model: eq3, HmIP-FSM

Device information

https://de.elv.com/p/homematic-ip-smart-home-schalt-mess-aktor-hmip-fsm-5-a-unterputz-P142721/?itemId=142721
Home Assistant Device information

Checklist

[x] I have created a single PR per device. When you have multiple submissions please create separate PR's.
[ ] For lights - I have only included the gzipped files (*.gz), not the raw CSV files.
[ ] For lights - I have provided a CSV file per supported color mode. Look that up in Developer Tools -> States

Additional info

Hey @dagobert, did you just take those values from the datasheet? Because one line says "manually measured", while the description says taken from manufacturer.
From my experience, the FSM has a normal relay, meaning the on-consumption was way higher than the 0.2W you specified here (same for the PSM). On the other hand the FSM16 actually has a bi-stable relay, meaning it's rly same off and on - 0.21W measured vs 0.25W spec/in your commit, so roughly correct.
Unfortunately I don't own them for this reason anymore, and can't provide actual values anymore, so that's why I'm asking.

I don't rly wanna argue over 0.21W vs 0.25W. Like yeah, the datasheet should be a worst case value, and this is an actual measurement, so I'd take it as better estimation of the "average" expected value (I always make sure to check the mains voltage is close to the nominal 230V, and I do have equipment to verify some of the values properly), but it's not a major difference.
But the ON consumption is off by a lot, like I recall the FSM16 and PSM to be in the order of like at least 0.5-0.6W. Yeah, one piece installed ppl won't notice, but powercalc is a great library, and we shouldn't spoil it bad values like this then. I'm certain the value is off by a factor of x2 at least.

Don't have time to provide an extensive answer currently, but just to give you my thought.
I agree with @folfy that the profiles in Powercalc database should be of high quality and at least provide 90% accurate calculation of estimated power consumption.
These normal relays have a more than duplicate power consumption when turned off vs turned on. That's also what we see for all the smart plugs which has been provided to the library and were measured, which have a normal relay.
In these circumstances power calculated by Powercalc will be off by a factor 2 so really render the whole sensor useless.
Do we have an overview which devices have a normal relay from the already submitted profiles. How many profiles potentially have this issue?
https://github.com/bramstroker/homeassistant-powercalc/tree/master/profile_library/eq-3
@dagobert do you have any possibility to measure one of these devices, or maybe contact the manufacturer if they have figures?

What I know from having bought and tested these 1-2y ago (unfortunately sold alle except the FSM16, due to the higher power consumption, and scraped my records of them):

HmIP-FSM - Normal, monostable NO relay
HmIP-FSM16 - Latched/bi-stable relay (so on-consumption = off/standby is correct)
HmIP-PSM - Normal, monostable NO relay

Unknown:

HM-ES-PMSw1-DR
HM-LC-Sw1-DR
HmIP-BDT
HmIP-BSM
HmIP-DRSI1
HmIP-FDT
HmIP-FROLL

I guess the question is about what you want to achieve. I took that supports a hand full of devices perfectly or close to all in the best way possible.
Of cause I want perfect but this would exclude most devices because there are too many out there.
I still am very sure of basically not more than a few people out there have reliable hardware for measurements. Some Shelly will not do the trick on sub 3W consumption if the measurements are off by +-20% at least at that value.
But anyway, I would go the zigbee2mqtt road and at least support a device somehow and not with all features instead of not at all. Why is it bad to support a eg. Simple smart light (no dimming and so on) by taking the 4W from the package print if there is boone around being able to perfectly measure it.
Let's do make a mark on devices that are not measured but values taken of specs or packaging.
People with proper equipment can than later take actual measurements.
To the question: I will get around measuring but at the moment I have my hands tied by the special time of the year and all that comes with it. And as you can imagine measuring these will take some effort because of the nature of where they are installed.
But I am on it...

@dagobert do you have any possibility to measure one of these devices, or maybe contact the manufacturer if they have figures?

Because it was the easiest to do, I measured the hmip-psm with the only measuring plug I had at hand ( nous a1z).
Standby off: 0.4
Standby on: 0.8 but jumping up to 1.2 every half minute or so.
Since the nous jumped up and down every time by 0.4 when only a dummy load was on it, I guess the internal compute cause this high jump.
Dummy load only:
7.3 and 7.7

Psm off:
7.7 and 8.1
Psm on:
8.1 and 8.5 and sometimes 8.9

At least this confirms the technical spec in regards to the standby off consumption. It is at 0.3. the standby on should be around 0.9 in average.
I suggest, if you really want to change something I can create a pull request and add 0.2 to the standby on change the measuring attributes to "from technical specs and estimated standby on" until I get around measuring them with a better tool.

I guess the question is about what you want to achieve. I took that supports a hand full of devices perfectly or close to all in the best way possible.

EQ-3 is one of the few vendors that I'd at least somewhat trust with their datasheets, but they still got to give the worst case value in their specs (i.e. <=0.25W, because they'll assume 230V + 10% as mains voltage, plus worst case tolerances of components inside), while powercalc tries to give you the somewhat average real-world value to expect. Also, while at least standby off consumption is specified, there is no spec for the ON-consumption at all (which is way higher for most of these devices I think - bistable relays are rare and expensive).
I have to say that powercalc is an amazing database of really high quality power consumption data. All devices I cross-checked the data for where within a few 10s of mW at worst, so I'd say it's even a rly good reference for buying decisions. Sure there could be other outliers in the DB, maybe someone taking measurements with a bad or defective plug, human error, etc., but we try our best. Thanks to @bramstroker for bringing this project to life and all the contributors.

Of cause I want perfect but this would exclude most devices because there are too many out there. I still am very sure of basically not more than a few people out there have reliable hardware for measurements. Some Shelly will not do the trick on sub 3W consumption if the measurements are off by +-20% at least at that value.

That's why the measurement device should be specified in the profile and there are recommended devices to use, see powercalc - contributing/measure:
"Supported smartplug which can accurately measure small currents (Shelly PM mini Gen3 recommended)"
Among other, more scientific and expensive devices, I use a Zhurui PR10-C, which others also tested for example to be scaringly accurate (only real weakness is it not reading <=0.13W and being "offline").  Yes, they're consumer devices, but as stated in the powercalc documentation, a device that can at least generally be considered somewhat accurate is necessary.
Your plug seems to only report at a resolution of 0.4W, while all recommended plugs will measure at 0.01W resolution. Yes, resolution doesn't necessarily mean accuracy, as not every high-resolution plug also is high accuracy, but a low-resolution plug can't be accurate at such small values, because it doesn't even give you the resolution to distinguish 0.7 and 0.8W.

But anyway, I would go the zigbee2mqtt road and at least support a device somehow and not with all features instead of not at all. Why is it bad to support a eg. Simple smart light (no dimming and so on) by taking the 4W from the package print if there is boone around being able to perfectly measure it. Let's do make a mark on devices that are not measured but values taken of specs or packaging. People with proper equipment can than later take actual measurements.

I'd say the meain reason is, that accuracy as well as credibility of those values varies a lot, if they're are even specififed, plus you can just configure the device in 30s manually with that value. Powercalc even has proper profiles for RGB LEDs, and they all were quite certainly taken from an actual measurement, because I haven't seen even one LED specify more than 100% brightness ON consumption and sometimes the standby consumption. So there are ppl around that can and do measure it, like this is what this project is all about.
The other reason is that if there is already a profile in the DB, we usually take it as is, assuming it's a decent measurement that someone actually took. I also only verify the profiles I use randomly as I got time and feel curious. If there's no profile, it's the first device I'll check and commit a profile back for here.

To the question: I will get around measuring but at the moment I have my hands tied by the special time of the year and all that comes with it. And as you can imagine measuring these will take some effort because of the nature of where they are installed. But I am on it...

Yeah, we understand that, I mean it's probably part of why nobody conributed data for them yet (I unfortunately also only got to know about powercalc after I had sold the PSM and normal FSM).

Standby off: 0.4 Standby on: 0.8 but jumping up to 1.2 every half minute or so.
Since the nous jumped up and down every time by 0.4 when only a dummy load was on it, I guess the internal compute cause this high jump.

Based on what I recall, the PSM was definitely within the spec of EQ-3 when off, somewhat similar to the FSM16, maybe something like 0.25W, and on was a quite steady value as well. I'd say the issue is your measurement plug not rly being accurate at all unfortunately.

At least this confirms the technical spec in regards to the standby off consumption. It is at 0.3. the standby on should be around 0.9 in average.
I suggest, if you really want to change something I can create a pull request and add 0.2 to the standby on change the measuring attributes to "from technical specs and estimated standby on" until I get around measuring them with a better tool.

The standby pretty sure should be at or below spec (0.3W), unless your PSM defective or something, so no need to increase that. The major issue is that you spec'ed all devices you've contributed with the standby consumption as ON consumption also, which is most likely way too low for most of them (as most devices should have a normal, monostable relay, unlike the FSM16).
-> I'd say it's up to @bramstroker to decide about using a very coarse "guesstimate" for the devices you can actually check for now, but given your plug can't even read at 0.1W resolution apparently, I'd vote against it. Either way, I'd say all the ones that are (probably) having completly wrong ON-consumption, should be measured&updated or removed.
I hope you agree @dagobert. that something like 0.6-1W (as you measured now, even if very coarse), for sure is way different than the ~0.3W you commited and shouldn't be used as a reference by others, and not just something like normal margin of error for a meassurement?

I have to say that powercalc is an amazing database of really high quality power consumption data. All devices I cross-checked the data for where within a few 10s of mW at worst, so I'd say it's even a rly good reference for buying decisions. Sure there could be other outliers in the DB, maybe someone taking measurements with a bad or defective plug, human error, etc., but we try our best. Thanks to @bramstroker for bringing this project to life and all the contributors.

Thanks for your kind words. And I agree, it's important for the project to have high quality power profiles. Can't be perfect and scientifically 100% correct, but we should strive to give a real good estimate within 95% accuracy ballpark.

Among other, more scientific and expensive devices, I use a Zhurui PR10-C, which others also tested for example to be scaringly accurate (only real weakness is it not reading <=0.13W and being "offline").  Yes, they're consumer devices, but as stated in the powercalc documentation, a device that can at least generally be considered somewhat accurate is necessary.

I also have a Zhurui PR10-C which I used to do < 1W measurements, it's really good.

Your plug seems to only report at a resolution of 0.4W, while all recommended plugs will measure at 0.01W resolution. Yes, resolution doesn't necessarily mean accuracy, as not every high-resolution plug also is high accuracy, but a low-resolution plug can't be accurate at such small values, because it doesn't even give you the resolution to distinguish 0.7 and 0.8W.

But anyway, I would go the zigbee2mqtt road and at least support a device somehow and not with all features instead of not at all. Why is it bad to support a eg. Simple smart light (no dimming and so on) by taking the 4W from the package print if there is boone around being able to perfectly measure it. Let's do make a mark on devices that are not measured but values taken of specs or packaging. People with proper equipment can than later take actual measurements.

No real expensive equipment is needed. No one contributing to powercalc library has expensive equipment. Just consumer devices which are more or less capable to get power values with some acceptable margin of error.
I check before merging if the graphs look good, and it's not to noisy and there are no outliers or 0 values.
New "Shelly PM mini Gen3" should be good (I don't have it myself yet, but it's recommended by @RubenKelevra). It costs ~$10. Another good one and probably the one for all the devices you are trying to get the self usage for is the Zhurui PR10-C. It's a littlebit more expensive but you can get it for <$50.
All power profiles (almost 500) in Powercalc are done by taking actual measurements, almost none of them have figures taken from manufacturer sheets.

To the question: I will get around measuring but at the moment I have my hands tied by the special time of the year and all that comes with it. And as you can imagine measuring these will take some effort because of the nature of where they are installed. But I am on it...

Thanks for the time and effort you put in it @dagobert. Because of people like you this project is what it is now.

At least this confirms the technical spec in regards to the standby off consumption. It is at 0.3. the standby on should be around 0.9 in average.
I suggest, if you really want to change something I can create a pull request and add 0.2 to the standby on change the measuring attributes to "from technical specs and estimated standby on" until I get around measuring them with a better tool.

The standby pretty sure should be at or below spec (0.3W), unless your PSM defective or something, so no need to increase that. The major issue is that you spec'ed all devices you've contributed with the standby consumption as ON consumption also, which is most likely way too low for most of them (as most devices should have a normal, monostable relay, unlike the FSM16). -> I'd say it's up to @bramstroker to decide about using a very coarse "guesstimate" for the devices you can actually check for now, but given your plug can't even read at 0.1W resolution apparently, I'd vote against it. Either way, I'd say all the ones that are (probably) having completly wrong ON-consumption, should be measured&updated or removed.

I would say to set the standby_power_on to 0.9 as it's a good estimate. We will update to new (more correct) figure when you get around to actually better measure it.
I prefer that to actually removing the profile completely, as people might have configured it already and removing the profile will cause errors in their installation.

New "Shelly PM mini Gen3" should be good (I don't have it myself yet, but it's recommended by @RubenKelevra). It costs ~$10.

Yeah, it's not only recommended by me, but also by Zerobrain on YouTube, who compared it to an "GW Instek GPM-8310" and found it's accurate by around 1% in all cases (even weird loads). The older Shelly Plug S many measurements were made before has an error of 10% for most cases in comparison.
The only issue is, as with most power measuring devices, with the Shelly PM mini Gen3 that it "only" measures in 0.1 W steps and seems to have a 0.1 W noise. So if you have a constant load, you see the value flicker around by 0.1 W, which can be averaged out for constant loads.
To measure switches, I connect a dummy load of around 12 W (just two ceramic resistors) and let them heat up for 2 hours. Then I measure the average power consumption of it over 5 minutes. Followed by an average measurement of the on and off state of the relay over 5 minutes, and then I subtract the dummy load.
This removes the 0.1 W noise and also makes it possible to measure values below the 0.1 step of the sensor.


bug: Ionic spins infinitely if there is a circular symink

Type: bug
Platform: all
If I have a symlink in www/source which links to . then when I run any ionic command it just spins forever. (The symlink was there so that my sourcemaps which refer to node_modules and bower_components are accessible by the browser)
Simple repro steps:
$ ionic run
... works great ...
$ ln -s .. www/source
$ ionic run
... hung ...
``
`

Greetings @ptarjan!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Short answer: Don't do that :grin:
Long answer: ln -s is little tricky here, as it does create a symlink, but just not in the way that cordova can handle. It's probably better to use a node based option here, like gulp-symlink

Uh, @mhartington you understand that the gulp-symlink you sent is just delegating to https://nodejs.org/api/fs.html#fs_fs_symlink_destination_path_type_callback which uses the same syscall symlink (2) as ln uses? It isn't how the symlink is made, it is the fact that ionic is recursively trying to walk the tree and not memorizing what it is has seen in the past.

Hmm, maybe I'm not understanding the problem.
I don't get why you need to have a symlink when the contents of www/ gets copied to the platforms folder. Could you explain your process here?

Sure, thanks. @mhartington during development I use https://www.npmjs.com/package/gulp-sourcemaps which builds a sourcemap of my js (since I'm compiling it via various transpilers, most notably browserify). This sourcemap references all the original files in their correct location (./www/js/whatever.js) but for node_modules it says they live in (./source/node_modules/whatever) so I make the symlink so all the sourcemap magic works.

Hmm, do you see this when running cordova run?


Feature/allow deserialize string as number

関連Issue
NA
概要
I was having an issue with serialization from bigint to JSON as it doesn't seem well supported. The solution I am using for now is to convert to string, and then convert back from the string to (u)int64_t. This PR allows deserialization of string as a number when the string is composed only of digits.
使い方
NA
NA
確認項目

[ ] チェックスクリプトでチェックを実施した
[X] 正常にビルドできた
[ ] 関連チケットに実績をつけた
[ ] ZenHubでPRとIssueを関連付けた
[ ] ZenHubのIssueを Review/QA Pipelineに移した

レビューする人

[ ] 関連チケットにレビュー実績をつけた

備考
There might be a better way to achieve this.
Also npm run check is giving me some error.

I also checked with cfd-js, but there was no problem if it was within the range of the value currently used.

Do you mean there was no issue with using BigInt? Maybe I am doing something wrong and this might not be needed then.

I also checked with cfd-js, but there was no problem if it was within the range of the value currently used.

Do you mean there was no issue with using BigInt? Maybe I am doing something wrong and this might not be needed then.

I think this is necessary. Since cfd-js is currently mostly javascript-only, all I did was check to see if it's update affected cfd-js.


'EntryPoint' object is not callable error when using PyBaMM

Description
What steps will reproduce the problem?

I'm running pybamm using DFN model but I get this error'EntryPoint' object is not callable, see below script;
import pybamm
experiment = pybamm.Experiment(
[
"Discharge at C/20 for 1 hour",
"Rest for 1 hour"
] * 20,
)
model = pybamm.lithium_ion.DFN()
sim = pybamm.Simulation(model, experiment=experiment)
sim.solve()
sim.plot()
Versions

Spyder version: 5.3.3  (conda)
Python version: 3.9.13 64-bit
Qt version: 5.15.2
PyQt5 version: 5.15.7
Operating System: Windows 10

Dependencies
# Mandatory:
atomicwrites >=1.2.0          :  1.4.0 (OK)
chardet >=2.0.0               :  4.0.0 (OK)
cloudpickle >=0.5.0           :  2.0.0 (OK)
cookiecutter >=1.6.0          :  1.7.3 (OK)
diff_match_patch >=20181111   :  20200713 (OK)
intervaltree >=3.0.2          :  3.1.0 (OK)
IPython >=7.31.1;<8.0.0       :  7.31.1 (OK)
jedi >=0.17.2;<0.19.0         :  0.18.1 (OK)
jellyfish >=0.7               :  0.9.0 (OK)
jsonschema >=3.2.0            :  4.16.0 (OK)
keyring >=17.0.0              :  23.4.0 (OK)
nbconvert >=4.0               :  6.4.4 (OK)
numpydoc >=0.6.0              :  1.5.0 (OK)
paramiko >=2.4.0              :  2.8.1 (OK)
parso >=0.7.0;<0.9.0          :  0.8.3 (OK)
pexpect >=4.4.0               :  4.8.0 (OK)
pickleshare >=0.4             :  0.7.5 (OK)
psutil >=5.3                  :  5.9.0 (OK)
pygments >=2.0                :  2.11.2 (OK)
pylint >=2.5.0;<3.0           :  2.14.5 (OK)
pyls_spyder >=0.4.0           :  0.4.0 (OK)
pylsp >=1.5.0;<1.6.0          :  1.5.0 (OK)
pylsp_black >=1.2.0           :  1.2.1 (OK)
qdarkstyle >=3.0.2;<3.1.0     :  3.0.2 (OK)
qstylizer >=0.1.10            :  0.1.10 (OK)
qtawesome >=1.0.2             :  1.0.3 (OK)
qtconsole >=5.3.2;<5.4.0      :  5.3.2 (OK)
qtpy >=2.1.0                  :  2.2.0 (OK)
rtree >=0.9.7                 :  0.9.7 (OK)
setuptools >=49.6.0           :  65.5.0 (OK)
sphinx >=0.6.6                :  5.0.2 (OK)
spyder_kernels >=2.3.3;<2.4.0 :  2.3.3 (OK)
textdistance >=4.2.0          :  4.2.1 (OK)
three_merge >=0.1.1           :  0.1.1 (OK)
watchdog >=0.10.3             :  2.1.6 (OK)
zmq >=22.1.0                  :  23.2.0 (OK)

# Optional:
cython >=0.21                 :  0.29.32 (OK)
matplotlib >=3.0.0            :  3.6.2 (OK)
numpy >=1.7                   :  1.23.4 (OK)
pandas >=1.1.1                :  1.4.4 (OK)
scipy >=0.17.0                :  1.9.3 (OK)
sympy >=0.7.3                 :  1.11.1 (OK)

Hi @minatouma seems like this is an error with pybamm https://github.com/pybamm-team/PyBaMM/issues/2534 ? Just in case, could you try to run your code using IPython and check if the same issue appears? For that open an Anaconda prompt, activate the env Spyder is installed and then run from there ipython. Let us know!

@dalthviz this comment shows the problem: https://github.com/pybamm-team/PyBaMM/issues/2534#issuecomment-1363815564
The entry points are created here: https://github.com/pybamm-team/PyBaMM/blob/c4baa51081f7ba38eea6ccb21a9e9fe89b71c934/pybamm/parameters/parameter_sets.py#L39-L42
for entry_point in importlib_metadata.entry_points(
            group="pybamm_parameter_sets"
        ):
            self.__all_parameter_sets[entry_point.name] = entry_point

which in most environments returns an imporlib_metdata.EntryPoint object that we then load here:
https://github.com/pybamm-team/PyBaMM/blob/c4baa51081f7ba38eea6ccb21a9e9fe89b71c934/pybamm/parameters/parameter_sets.py#L60-L61
if isinstance(ps, importlib_metadata.EntryPoint):
            ps = self.__all_parameter_sets[key] = ps.load()

But in spyder the object returned in the first step is a setuptools._vendor.importlib_metadata.EntryPoint, so it doesn't get caught by the second if statement
How would you suggest we resolve this? Add a second isintance() check for the spyder type or something different?

Hi @tinosulzer thank you for the explanation! Adding another isinstance sounds good to me but just in case, what do you think @ccordoba12 ?

Yeah, sounds good to me too. And thanks @tinosulzer for chiming in and be interested in fixing this issue on behalf of Spyder users.

Thanks so much for your support. However, I'm still getting the same error with Spyder, may be I'm not sure how to incorporate the code with pybamm script above to fix the issue

@minatouma, this needs to be fixed in pybamm and released in a new version of it. Until that happens, I'm afraid there's nothing you can do about it.

Working on the fix @minatouma but until then just use v22.10 or earlier

Closing since the fix was released some time ago. See https://github.com/pybamm-team/PyBaMM/releases/tag/v22.12


Adding comments to generated OpenAPI file does not work on macos

Using nswag MSBuild tool's aspnetcore2openapi command does not correctly resolve the XML documentation on macos.
See this repro: https://github.com/dasMulli/nswag-macos-issue
After building the project on macos, the generated openapi.json file in the root directory does not contain the summary of the WeatherForecastController/Get method.
It seems that it can be fixed with a patch to Namotion.Reflection, but I'm not sure if that's the necessary fix here (provided that it seems to work on linux):

XmlDocsExtensions.cs
--- a/src/Namotion.Reflection/XmlDocsExtensions.cs
+++ b/src/Namotion.Reflection/XmlDocsExtensions.cs
@@ -771,7 +771,7 @@ namespace Namotion.Reflection
                         if (!string.IsNullOrEmpty(codeBase))
                         {
                             path = DynamicApis.PathCombine(DynamicApis.PathGetDirectoryName(codeBase
-                                .Replace("file:///", string.Empty)), assemblyName.Name + ".xml")
+                                .Replace("file://", string.Empty)), assemblyName.Name + ".xml")
                                 .Replace("file:\\", string.Empty);

                             if (DynamicApis.FileExists(path))

This issue is preventing me to work on my project on MacBook. @RicoSuter please help.

Fixed by not using aspnetcore2openapi command. Used nswag with config instead.

Fixed by not using aspnetcore2openapi command. Used nswag with config instead.

How did you use nswag before and what did you do to resolve (overcome) the issue?
I'm having the same issue with API comments missing when I run it on MacOS.

Fixed by not using aspnetcore2openapi command. Used nswag with config instead.

@Fennixx How did you use nswag before and what did you do to resolve (overcome) the issue? I'm having the same issue with API comments missing when I run it on MacOS.

Unfortunatelly, I dont have access to code base where I had the issue any more, but as an example, you need to create the nswag.json file with the configuration what you need.
Basic example:
{
  "runtime": "NetCore31",
  "defaultVariables": "Configuration=Release",
  "documentGenerator": {
    "aspNetCoreToOpenApi": {
      "project": "./path/to/YourProject.csproj",
      "configuration": "Release",
      "output": "./output/swagger.json",
      "verbose": true
    }
  },
  "codeGenerators": {
    "openApiToTypeScriptClient": {
      "className": "ApiClient",
      "output": "./output/api-client.ts"
    }
  }
}

then you will call instead of aspnetcore2openapi something like this:
nswag run /runtime:NetCore31 /variables:Configuration=Release /input:nswag.json

I'm currently using the webApiToOpenApi generator, worth trying the aspNetCoreToOpenApi instead. Thanks.


illegal hardware instruction (Installation error)

I followed the install steps, but when I try to check my install with this following line, I get an "illegal hardware instruction" error.
python -m spinup.run ppo --hid "[32,32]" --env LunarLander-v2 --exp_name installtest --gamma 0.999

Hi, @mei-chen , I think you'll have to give some more information if you want somebody to help you here. For example, a description of your environment (OS, versions of libraries installed, etc.) and, most important, the full error you receive.
Best of luck!

I received the same here:
call:
python -m spinup.run ppo --hid "[32,32]" --env LunarLander-v2 --exp_name installtest --gamma 0.999
reply:
zsh: illegal hardware instruction  python -m spinup.run ppo --hid "[32,32]" --env LunarLander-v2 --exp_name
I use M1 mac

I also received this error in M1 Mac

I also have received this error on an M1 Mac running MacOS Monterrey 12.6
Has anyone found a solution for this?

Which python version you are running?
Python should (or must) be an arm64-version.
Execute python in terminal/iterm:
python

import platform
platform.platform()
'macOS-13.3-arm64-arm-64bit'

Note: I've had the illegal instruction error with numpy-library. Turns out the python version runs through Rosetta2 and that was wrong.
Hope this might help you,
Peter

I have the same issue.
My python version is: Python 3.6.12
On Apple M1 Max.
MacOS 13.4

I have the same problem:
zsh: illegal hardware instruction  python -m spinup.run ppo --hid "[32,32]" --env LunarLander-v2 --exp_name
zsh: illegal hardware instruction  python -m spinup.run test_policy data/installtest/installtest_s0
zsh: illegal hardware instruction  python -m spinup.run plot data/installtest/installtest_s0

No resolution for this yet?

I tried but still cannot solve it
Sivakumar @.***> 于2024年4月22日周一 18:39写道：

No resolution for this yet?
—
Reply to this email directly, view it on GitHub
https://github.com/openai/spinningup/issues/360#issuecomment-2069060076,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQO2BANQ7KQR534PU2DONMLY6TSG3AVCNFSM5QFJSG62U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMBWHEYDMMBQG43A
.
You are receiving this because you commented.Message ID:
@.***>


TextBox menu does not follow theme change

Describe the bug
This is bugging UWP & WinUI3. And lot of Windows's built-in apps, even the Windows built-in search.
How is this not get solved after 3 years of Windows 11 release?
Steps to reproduce the bug
For WinUI3:

Open WinUI 3 Gallery, type random stuff in the search TextBox.
Change theme from white->dark / dark->white in Windows Settings

For UWP, do the same thing with the windows settings app.
Expected behavior
No response
Screenshots
WinUI 3:

UWP:

NuGet package version
WinUI 3 - Windows App SDK 1.5.3: 1.5.240428000
Windows version
No response
Additional context
No response

Reopen #7701 and mark this as dup, or keep this open


Error when deploying

When using npm run deploy I receive the error

The serverless deployment bucket "chromda-dev-serverlessdeploymentbucket-xs9t4b58553s" does not exist. Create it manually if you want to reuse the CloudFormation stack "chromda-dev", or delete the stack if it is no longer required.

The bucket can not be created manually. Could you possibly assist?

#6 worked now after multiple attempts.
Could you now assist, how to call the function via REST?


New "paste from clipboard" warning dialog when opened freshly installed app for the first time (iOS 16)

[REQUIRED] Step 1: Describe your environment

Xcode version: 14
Firebase SDK version: 9.1.0
Installation method: CocoaPods
Firebase Component: DeepLinks
Target platform(s): iOS

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:

update iPhone to iOS 16 beta
make sure that app is not installed on the device
install app with firebase deep links enabled using App Store (or Testflight)
when freshly installed app opened for the first time, the warning dialog "App would like to paste from 'Safari'" is displayed.

I understand, that this is new iOS 16 privacy restriction and it is used instead of not so disruptive small warning that was presented in iOS 15. However, looks like we have to found workaround for this functionality, because showing this dialog to every new user is the problem.

Thanks for reporting, @sarsonj. As a workaround, you may try to follow step 3 here by setting the FirebaseDeepLinkPasteboardRetrievalEnabled key to NO in your app's Info.plist file.

Thanks for reporting, @sarsonj. As a workaround, you may try to follow step 3 here by setting the FirebaseDeepLinkPasteboardRetrievalEnabled key to NO in your app's Info.plist file.

@rizafran , won't that have the side effect of reducing FBDL's ability to match the install and retrieve the universal link data on first launch?

Yes, it will. Currently with FDL you can have either exact matching and pasteboard popups or fuzzier matching and no popups.

Yes, it will. Currently with FDL you can have either exact matching and pasteboard popups or fuzzier matching and no popups.

@morganchen12 , It seems like there may be a third way according to https://developer.apple.com/wwdc22/10096:

In iOS 15, when an application programmatically accessed the pasteboard without using system provided Paste interfaces, a banner would appear to indicate that the pasteboard was accessed.
New to iOS 16, the system behavior has changed. Now, instead of a banner, we will display an alert that asks for permission to use the pasteboard.
System paste interfaces that the users interact with provide implicit access to the pasteboard and will avoid the alert.
If you have custom paste controls, you can replace them with the new UIPasteControl that looks and behaves like a filled UIButton.
It is enabled whenever the pasteboard gets content compatible with the control's paste target.

It seems like developers may be able to create a paste button (that Apple is looking for interactions with at runtime?), and some UI around it, we can avoid the popup. For now, what I'll be proposing to my UX team is we build a consent page in app containing this paste button.
Big picture: what I'm wondering is can the workflow of FDL change such that the copy/pate consent of the preview page move its way from before app install to within the app itself. This would be a big change in how we work with FDL today, but maybe a necessary one to preserve everyone's desired UX?

It seems like if developers create a paste button (that Apple are looking for interactions with at runtime?), and some CTA UI around it explaining why to tap it, we can avoid the popup. For now, this is what I'll be proposing to my UX team.

Well, we can even use UIPasteControl instead of UIButton with label "Next" in our onboarding wizard, but I am not sure that it is what Apple intended this new control to be used for.
I am worried that any use that will not be close to "paste" usage will be rejected sooner or later during App Review.

But is'nt there any other way to attribute link after app installation? Don't use AppsFlyer or others different technique?

@sarsonj

Well, we can even use UIPasteControl instead of UIButton with label "Next" in our onboarding wizard, but I am not sure that it is what Apple intended this new control to be used for.

Yes, and we shouldn't be tricking users to paste here. We should be showing some info around the paste button asking them something along the lines of "tap paste to continue where you left off" or whatever is appropriate for your use-case. From the WWDC what's new in privacy talk, It sounded like a button with a paste icon was sufficient.

But is'nt there any other way to attribute link after app installation? Don't use AppsFlyer or others different technique?

Yes, and in fact FDL already does this with the pasting option turned off. It's called something like "probabilistic matching" depending on what the vendor wants to call it (Adjust also does this), where it tries to build a fingerprint of your device based on time, ip, screen-size, OS version, etc. It works, but it's not 100% guaranteed to work like the pasting method does.

@sarsonj would you be up for a short call regarding your use case w/ FDL?

This can come earlier, but marking the milestone to make sure we review before the Firebase 10 release.

We finally do tests in our app and we can live with "weak" match type in most cases and in the case where not, the processing will not be as automatic as now.
However, we realised, that now we can enable or disable pasteboard usage globally, not only for iOS 16 users - the property is taken from Info.plist. Or did I miss something, is it possible to decide it on application level?

We'll need at least a release note documenting the iOS 16 usage.

We just hit this issue and it is going to be very problematic for our solution as we found that by using the weak matching in the past there was about a 20% failure rate.

My team is experimenting with a mixture of using Firebase Dynamic Links and App Clips to achieve 100% attribution. Will update here if it proves successful.

@asowers1 Hey, we are about to start the same experiment as above. Just want to know any update on this.

Following. That extra paste warning on app install is sure to confuse a lot of users - and many will say no, as well.
Apple does not seem to realize outside of California, people care way more about being actually able to use an app vs. privacy protection.
Wonder if there's a solution in Flutter with a custom paste / next button - that would be extremely appropriate for our use case.

@abhi0987 , we found that using App Clips in conjunction with dynamic links worked, although our product team was not happy with the number of popups that the user needed to go through to achieve the desired effect.

@asowers1 Are you able to track the attributions in the dynamic link using the appclip post installation?

@abhi0987 , what we did was configure the Firebase dynamic link to route to a specific path on our website; not our app.  When the browser hits this path on our website, iOS presents an App Clip to the user.  When the App Clip launches, it will need to write data to a shared preference group or keychain so that the data you expect is available in your app bundle when users convert to using your full app.  One thing we did not need to do for our use-case was inject data into the App Clip from our site; all we needed was the context of launching the App Clip from a specific route, so I don't have a solution for that if that's what you're looking for, although there may be one and I'd be curious to hear if you find a way.  You can learn more about configuring App Clips on your site here: https://developer.apple.com/documentation/app_clips/associating_your_app_clip_with_your_website

@asowers1 Could you explain me what is the need for FDLs then in your use case? Cause the way I see it, you are not able to track attribution neither serve dynamic content using FDL.

It would be great to have this issue fixed. Having this copy paste popup is not a viable solution for most of the usecases.

Up

@eeoom , https://developer.apple.com/videos/play/wwdc2021/10012/ 3:33

Is there anyone who tried with having FDL and PasteboardButton to avoid the popup. I tried having a pasteboard button on the initial screen, but still i get the dialog.
                    PasteButton(payloadType: String.self) { strings in                         print(strings[0])                     }                     .labelStyle(.iconOnly)                     .circleCrop()                     .tint(Colors.primaryBackground)                     .frame(width: 50, height: 50)
Is there anyone used pasteboardbutton and avoided the dialog ? Branch also using this approach but doesn't work for me

It seems like if developers create a paste button (that Apple are looking for interactions with at runtime?), and some CTA UI around it explaining why to tap it, we can avoid the popup. For now, this is what I'll be proposing to my UX team.

@asowers1  did you succeed with this approach or have example of this approach . I tried the same but i still get the dialog

Yes

I have some troubling news there.

Firebase Dynamic Links is no longer recommended for new projects. In the future, the Dynamic Links service will shut down, but you will have at least 12 months from the announcement date to migrate. Around the end of Q2-2023, we will announce more information.

Yes I am aware of that too. But we have got time to change to another provider.. so meantime trying to find a solution with FDL :(

The Firebase Dynamic Links service will be shutdown on August 25, 2025.  In the meantime, only critical or security issues will be fixed in the SDK.
More at https://firebase.google.com/support/dynamic-links-faq


Methods supported for error type values are not suggested for a error value belonging to a union of error types in a different module

Description:

https://github.com/ballerina-platform/module-ballerina-grpc/blob/master/grpc-ballerina/grpc_errors.bal#L84
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

You can find the sample project in https://github.com/lasinicl/order-management-system
Please check
https://github.com/lasinicl/order-management-system/blob/main/order_service/OrderMgt_sample_service.bal#L24

The issue has been fixed. Tested on the latest distribution.


How to custumize existing fields.

Hi, i have a field of type "redactor" that contains text. How can i encrypt data before submit in database and decrypt when i edit it.
Thank you.

Use mutators and accessors https://laravel.com/docs/master/eloquent-mutators#accessors-and-mutators


[QUESTION] Find if database exists and seed data after being created

When using the following:
using(var db = new LiteDatabase(@"MyData.db"))
}

LiteDB creates a new database if it does not exist.
Is there a way to determine if a database exists or not? Or if it is empty?
I would like to seed the database with data but only after its creation.

@mdmoura I think something like this would work for you:
using(var db = new LiteDatabase(@"MyData.db"))
{
    int colCount = db.GetCollectionNames().Count();
    if (colCount == 0)
    {
        //initialization code here
    }
}

@lbnascimento I have been reading your docs and I found different information.
For example, in here there is an OnVersionUpdate method and a version can be included in the connection string.
But in source code and in here there is no such option.
I am trying the following to seed initial data:
public class Database : LiteDatabase {

   public Database(String connectionString, DatabaseMapper databaseMapper) : base (connectionString, databaseMapper) {

     if (base.UserVersion == 0) {
       // seed initial data
      base.UserVersion = base.UserVersion + 1;
     }

     if (base.UserVersion == 1) {
       // do something else
     }

   }

 }

Does this make sense?
The class DatabaseMapper is the following:
  public class DatabaseMapper : BsonMapper {
    public DatabaseMapper() {
         base.Entity<Post>()
           .DbRef(x => x.Tags, "tags");
    }
  }

Then I created an extension to inject Database class in ASP.NET Core:
    public static void AddLiteDatabase<TDatabase, TDatabaseMapper>(this IServiceCollection services, String connectionString) where TDatabase : LiteDatabase where TDatabaseMapper : BsonMapper {

      TDatabaseMapper contextMapper = (TDatabaseMapper)Activator.CreateInstance(typeof(TDatabaseMapper));

      TDatabase context = (TDatabase)Activator.CreateInstance(typeof(TDatabase), new Object[] { connectionString, contextMapper });

      services.AddScoped<TDatabase>(x => context);

    }

Does this make sense?

@mdmoura Regarding the documentation you linked, that's for v1, a very old version. Other than that, your solution seems good to me.


⚠ Bofrost.be has degraded performance

In 9ed5266, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 982 ms

Resolved: Bofrost.be performance has improved in 670c4ad after 9 minutes.


any way to disable new tab?

When chromedp.click a link <target="_blank">,the targetID will be changed, is there a way to disable new tab open the link?

Please see the chromedp/examples repository for some examples to get you started with your project. Specifically, checkout the click example. You will likely need to do network intercept requests, which will require having a strong understanding of the way Chromium works. Good luck!

Please see the chromedp/examples repository for some examples to get you started with your project. Specifically, checkout the click example. You will likely need to do network intercept requests, which will require having a strong understanding of the way Chromium works. Good luck!


High CPU usage across multiple cores during play

Version 1.5.0 uses 2 threads on my PC. Unfortunately I noticed that only being after 240 games in my gauntlet.

So the issue was the "poling" thread poling way too often Adding a sleep so the poling only happens intermittently drastically reduced the usage on the second core.
I have a hex core so you would expect using a core fully to be ~16% usage. In version 1.5.1 you can see full usage of one core by the search thread but the second thread has barely any usage at all, compare this to 1.5.0 where the second thread it using another full 16%.

1.5.1
1.5.0

Let me know if version 1.5.1 fixes the issue for you

Unfortunately this executable doesn't work in my pc, it should be an AVX2/512 or BMI; I support only AVX and SSE.
Anyway, your screenshot shows it's ok now. I suppose Gabor will be satisfied, since the polling thread will run in his "free" thread unnoticeably.
Since you didn't change anything else in the engine, I don't need to download it, I've already tested 1.5.0: I usually keep two free threads to overcome this situation.
Looking forward to the next improvement. Keep up the good job!
Alex

"I suppose Gabor will be satisfied, since the polling thread will run in his "free" thread unnoticeably."
Yes, he is. I can confirm that CPU usage is now 25 %.


Add AWS CodeCommit git provider entry

https://github.com/daytonaio/daytona/pull/797 added support for AWS CodeCommit to Daytona.
We should add a section to https://www.daytona.io/docs/configuration/git-providers/ outlining how to add it.

Also i note to @stefanicjuraj i would rewrite GIT Providers similar to installation. More specifically that the urls are https://www.daytona.io/docs/configuration/git-providers#github
https://www.daytona.io/docs/configuration/git-providers#bitbucket
and so on.
And on those places you fined end to end the instruction on how to install each.
BTW I am aware there will be a lot of duplicate text, but as a user its is much more straight forward as all I need is on one place.

Moved https://github.com/daytonaio/docs/issues/109#issuecomment-2273388599 to a separate issue


Any plans for arm64 official images?

Given that arm64 images are available https://www.docker.elastic.co/r/kibana are there any plans to add arm64 to the docker official images?

@tianon fixed in 094d900939e2a66300d48cbf2f3a8e0fc1d51abc


License?

Hi @Sleepychord,
When looking at this project, I wasn't able to find a license. Is it under MIT like https://github.com/Sleepychord/cogdata, or is there a different license?

That's Okay. I will add a MIT.

Thanks @Sleepychord!
As I'm causing you effort, I went ahead and created a PR to hopefully mean you have less to do :)

Hi @Sleepychord. Any update on the license? Thanks!

Closing :) Thank you @Sleepychord


Can't upload existing project.

Describe the bug
I worked on a new front end for my start up using bolt.new which was great. However, I downloaded the code locally, made some change and wanted to upload it again and I can't. I paid 100 USD for this feature and it is set me back. I am meant to be presenting my MVP in a few weeks and this is critical! Is there anyway around this???
Link to the Bolt URL that caused the error
bolt.new
Steps to reproduce
Try adding in a local repo or a private repo from github.
Expected behavior
Being able to add in a local repo.
Screen Recording / Screenshot
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Yes! Drag-n-drop the code into a blank StackBlitz project (you can create one by going to node.new) and then use the Open In Bolt button to open it in Bolt!


addPage() method does not work properly with "p" | "l" orientations

Hey!
Thanks for the jsPDF !
Recently I started working with this library and I came across the issue with addPage() method.
The issue is that the method does not flip the sizes of a page, so I dug into the addPage() src code and found these lines
 //  part of my code
const orientation = data.width > data.height ? "l" : "p";
doc.addPage([PDF_FILE_WIDTH, PDF_FILE_HEIGHT], orientation);
...

// in the library
 var _addPage = function (parmFormat, parmOrientation) {

    // seems ok, the first param is  width
    if (Array.isArray(parmFormat)) {
      width = parmFormat[0] * scaleFactor;
      height = parmFormat[1] * scaleFactor;
    }

   // some code

    // BTW these case statements with nested if statements makes code-reading harder
    switch (orientation.substr(0, 1)) {
      case "l":
        if (height > width) {
          format = [height, width]; //  this line is the same as in the next case
        }
        break;
      case "p":
        if (width > height) {
          format = [height, width];
        }
        break;
    }

I am using the library as a package
  "dependencies": {
    "jspdf": "^2.3.0",
    }

It seems to be a bug, isn't it?
Thanks for helping!

The code looks correct to me. If the dimensions do not match the orientation, the dimensions are flipped. In your case, the dimensions and orientation are already correct, because of this: data.width > data.height ? "l" : "p". This is why they are not flipped. Correct me if I'm wrong.

Thank you for a quick response!
If the dimensions do not match the orientation, the dimensions are flipped.
This sentence makes it clearer for me and why it I had  this issue.
Thanks!


modal jump

modal jump up on keyboard input

Hello, can you please describe the problem better? A video would help

Can we revisit this issue? Basically, whenever a keyboard is opened, the modal jumps up. I'm attaching a video. As seen in the video, the black bar at the bottom (my padding) exists as it should, but never the less this random jump-up behavior occurs. It seems like the component is automatically taking into account the keyboard height, but this is problematic when users (like myself) need to handle it on a custom basis.
In my code below, the <Portal>...</Portal> just renders everything in the absolute root, so there would be no styling conflicts with any parent element. I have no other issues with any other components, keyboards, and children of Portal.
https://github.com/birdwingo/react-native-swipe-modal/assets/127700939/d7111e2a-b402-4dc5-a23a-bfaa1e892eb7
import SwipeModal from "@birdwingo/react-native-swipe-modal/src/components/SwipeModal";
import { Portal, Surface, TextInput, useTheme } from "react-native-paper";
import { ReactNode, useEffect, useRef } from "react";
import { SwipeModalPublicMethods } from "@birdwingo/react-native-swipe-modal";
import { KeyboardAvoidingView, ScrollView, useColorScheme, View } from "react-native";
import { useSafeAreaInsets } from "react-native-safe-area-context";
import { useKeyboard } from "@react-native-community/hooks";

export const Modal = ({ children, open, setOpen }: ModalProps & {
  children: ReactNode;
}) => {

  const modalRef = useRef<SwipeModalPublicMethods>(null);
  const keyboard = useKeyboard();
  const { top, bottom } = useSafeAreaInsets();
  const scheme = useColorScheme();

  const topOffset = top + 50;
  const barHeight = 40;

  useEffect(() => {
    open ? modalRef.current?.show() : modalRef.current?.hide()
  }, [ open ]);

  return (
    <Portal>
      <KeyboardAvoidingView behavior={"height"} style={{ flexGrow: open ? 1 : 0 }}>
        <SwipeModal
          ref={modalRef}
          fixedHeight={true}
          topOffset={topOffset}
          visible={open}
          onShow={() => setOpen(true)}
          onHide={() => setOpen(false)}
          bg={"rgba(0,0,0,0)"}
          style={{
            overflow: "scroll",
          }}
          barContainerStyle={{
            backgroundColor: scheme === "dark" ? "#1c1d2b" : "lightgrey",
            height: barHeight,
            borderTopRightRadius: 10,
            borderTopLeftRadius: 10,
          }}
        >
          <View style={{ width: "100%", flexGrow: 1, paddingBottom: bottom }}>
            <Surface elevation={1} style={{ width: "100%", height: "100%", paddingBottom: keyboard.keyboardShown ? keyboard.keyboardHeight : bottom }}>
              {children}
            </Surface>
          </View>
        </SwipeModal>
      </KeyboardAvoidingView>
    </Portal>
  );
};


Re-rendering improvements for like/unlike

Scoping this update a bit tighter to what it actually affects.
Before
https://github.com/bluesky-social/social-app/assets/810438/8335e802-289d-412b-946b-7789e2dcca30
After
https://github.com/bluesky-social/social-app/assets/810438/11eb21b0-2a16-49d2-8e88-1fb5f9c4ddde

Woooow @gaearon


[1.14.4] Stacktraces on load NPE -  Failed to load data for block entity uselessmod:energy_cable [537/1940] java.lang.NullPointerException: null

Describe the bug
A clear and concise description of what the bug is.
Forge Version forge-1.14.4-28.1.39
Mod Version UselessMod-1.14.4-2.2.4
https://gist.github.com/ProsperCraft/191e1e5329c60c935e321235ba53d6a6

I have rewritten the entire energy cable code.
It still does not work 100% correctly, but it will be enough for the next update.
This bug should be fixed in 1.14.4-2.2.5


Add cgroup cpu/mem usage metrics

Description
This PRs expands the existing metrics reported by the Cgroup monitors. Specifically adds the usage for cpu and memory.
Release note
Introduces new usage metrics for cpu and memory cgroups

Key changed/added classes in this PR

CgroupCpuMonitor.java
CgroupMemoryMonitor.java

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[x] added documentation for new or modified features or behaviors.
[x] a release note entry in the PR description.
[x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Added a few comments but overall, the change looks good to me. Were you able to validate that you were getting the right values for these? One way would be to test your assumptions in Docker and see if your numbers match with what Docker sees.

Were you able to validate that you were getting the right values for these? One way would be to test your assumptions in Docker and see if your numbers match with what Docker sees.

Yeah tested in docker to see the formats match. But for cpu usage percentage, where do you want me to confirm from ?

The Docker desktop UI displays CPU usage for a container, but I'm not 100% sure if even Docker does it right. But if it does, it'll give you additional confidence.

I have tested with Datadog, even though the %s aren't exactly matching up, the spikes are matching. So I think it is safe to assume working fine.


OCPBUGS-8276: Bugfix check imagesetconfig for valid oci protocol when oci feature i…

…s used
Description
This fixes the bug when using the --use-oci-feature flag and no oci protocol is used in the imagesetconfig.
Fixes OCPBUGS-8276
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Added unit test to verify this bug is addressed.

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

/jira refresh

/jira refresh

/jira refresh

/retest

/test images

/assign @soltysh
PTAL when you get a chance - thanks

/label qe-approved

Done pre-merge test , no issue found

/cherrypick release-4.13

/cherrypick release-4.13

/cherrypick release-4.13


MATH table

ttx is unable to read more than 71 stretchy delimiters from a .ttx file. It will read an .otf with more than 71 delimiters, but barfs when round tripping. I typically export the .ttx file from an .otf file, replace the MATH table with one using glyph names, and then create a new .otf file (and then import the compiled MATH table into FontLab which does not support glyph names at this time). It does not matter which delimiter has index='71", or the order. It simply fails when it sees index="71". I have appended the error trace at the end. If you would like a failing source file, I will be happy to provide one. I am using python 3.8.3 and ttx version 4.33.3.
Thank you.
R:\ctan\logix\source\xxx>ttx logix.ttx
Compiling "logix.ttx" to "logix#3.otf"...
Parsing 'GlyphOrder' table...
Parsing 'head' table...
Parsing 'hhea' table...
Parsing 'maxp' table...
Parsing 'OS/2' table...
Parsing 'name' table...
Parsing 'cmap' table...
Parsing 'post' table...
Parsing 'CFF ' table...
Parsing 'GDEF' table...
Parsing 'GPOS' table...
Parsing 'GSUB' table...
Parsing 'MATH' table...
Parsing 'hmtx' table...
Parsing 'DSIG' table...
ERROR: Unhandled exception has occurred
Traceback (most recent call last):
File "c:\bin\python\lib\site-packages\fontTools\ttx.py", line 405, in main
process(jobs, options)
File "c:\bin\python\lib\site-packages\fontTools\ttx.py", line 387, in process
action(input, output, options)
File "c:\bin\python\lib\site-packages\fontTools\misc\loggingTools.py", line 372, in wrapper
return func(*args, **kwds)
File "c:\bin\python\lib\site-packages\fontTools\ttx.py", line 305, in ttCompile
ttf.save(output)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\ttFont.py", line 184, in save
writer_reordersTables = self._save(tmp)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\ttFont.py", line 224, in _save
self._writeTable(tag, writer, done, tableCache)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\ttFont.py", line 647, in _writeTable
tabledata = self.getTableData(tag)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\ttFont.py", line 669, in getTableData
return self.tables[tag].compile(self)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 103, in compile
self.table.compile(writer, font)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 940, in compile
conv.write(writer, font, table, value)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otConverters.py", line 631, in write
value.compile(subWriter, font)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 904, in compile
conv.writeArray(writer, font, table, value)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otConverters.py", line 197, in writeArray
self.write(writer, font, tableDict, value, i)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otConverters.py", line 631, in write
value.compile(subWriter, font)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 940, in compile
conv.write(writer, font, table, value)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otConverters.py", line 631, in write
value.compile(subWriter, font)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 904, in compile
conv.writeArray(writer, font, table, value)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otConverters.py", line 197, in writeArray
self.write(writer, font, tableDict, value, i)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otConverters.py", line 510, in write
value.compile(writer, font)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 940, in compile
conv.write(writer, font, table, value)
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otConverters.py", line 356, in write
writer.writeValue(self.typecode, font.getGlyphID(value))
File "c:\bin\python\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 601, in writeValue
self.items.append(struct.pack(f">{typecode}", value))
struct.error: ('required argument is not an integer', 'str', 0, 'PartRecords[]', 'GlyphAssembly', 71, 'VertGlyphConstruction[]', 'MathVariants')

Clarification: The indices are in order. It doesn't matter which delimiter is 71. The order of the delimiters doesn't matter. The first 71 are processed ok, and it fails on 72 (index 71).

Please attach or points us to a sample font file that fails.

Here you go. Note that I had to rename this from logix.ttx to logix.txt to allow the upload to work.
logix.txt

It's because your record 71 references a nonexistent glyph "ClsDblFloorBtm". Sure our error message sucks, but that's the reason. No superstitous 71 or other magic number.

Error message will be improved by https://github.com/fonttools/fonttools/issues/3030


PendingDeprecationWarning: Use  'tree.iter()' or 'list(tree.iter())' instead

PendingDeprecationWarning: This method will be removed in future versions.  Use
'tree.iter()' or 'list(tree.iter())' instead.
  for elem in self.tree.iter() if Element_has_iter else self.tree.getiterator():

This warning comes up during testing. Similar to #179 I can't find where this occurs during debugging. Any help would be appreciated.

one function that returns this is:
all_localisations()

@sadeghs how does one go about chasing the functions that result in the above warning? I can't figure what the best debug method is when the call stack doesn't trace this warning

If it's not in your solution then it's probably in your dependencies. If you can search your deps, if not then it's not the end of the world, when you update your deps the warning should go away hopefully.

 maybe a year and a half ago we only had xlrd as a reader, so it was a non-issue. But we've since added openpyxl, pyxlsb, and odfpy as readers, I think without using machinery similar to the writer side for respecting options.

from
https://github.com/pandas-dev/pandas/issues/34252
and
https://stackoverflow.com/questions/60288732/pandas-read-excel-returns-pendingdeprecationwarning


refactor(crosswalk_traffic_light_estimator): rework parameters

Description

Refactor the ROS Node configuration layout for the crosswalk_traffic_light_estimator package according to the guideline.

Create the schema and parameter files.
Remove the default values from the source code to avoid duplication.
Add a parameter last_detect_color_hold_time in the README.md file.

Related links

None
Tests performed

The refactored node was successfully built and tested by launching this node individually.
The running result is shown below:

Notes for reviewers

None
Interface changes

Create two configuration files perception/crosswalk_traffic_light_estimator/config/crosswalk_traffic_light_estimator.param.yaml and perception/crosswalk_traffic_light_estimator/schema/crosswalk_traffic_light_estimator.schema.json
Parameter values are now required to be passed to the node when launched.

Effects on system behavior

None
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.
[x] The PR has been properly tested.
[x] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Oh. Sorry. I think it needs to add parameter config path in launcher file since default parameters are removed.

I think it's already added in the launch.xml file.

Oh. Sorry. I think it needs to add parameter config path in launcher file since default parameters are removed.

I think it's already added in the launch.xml file. <arg name="crosswalk_traffic_light_estimator_param_file" default="$(find-pkg-share crosswalk_traffic_light_estimator)/config/crosswalk_traffic_light_estimator.param.yaml"/>

Sorry I missed it. LGTM.

hmm... It seems that this node called in tier4_perception_launch.launch.xml directly. (not via the one of this package.)
https://github.com/autowarefoundation/autoware.universe/blob/b09f3096f6e9af7d808174e80c2b492f6a5306ec/launch/tier4_perception_launch/launch/traffic_light_recognition/traffic_light.launch.xml#L140-L147

@yuntianyi-chen
Can I add some modification to tier4_perception_launch.launch.xml to launch correctly?

@yuntianyi-chen Can I add some modification to tier4_perception_launch.launch.xml to launch correctly?

@miursh Sure! Please do it.

@yuntianyi-chen Done. 5cb0f3f408048f869a5c60148e90083a4ae696e1


Convert hostname to IP when getting distance

What does this pull request do? Explain your changes. (required)
Allows hostnames to be used as orchAddr with geo routing.
Specific updates (required)

When getting distance from O to B, attempts to resolve hostname to an IP when IP parsing fails. Only uses IP for geo distance lookup

How did you test each of these updates (required)

Added log message and deployed with a hostname in orchAddr flag
Does this pull request close any open issues?

Checklist:

[ ] Read the contribution guide
[ ] make runs successfully
[ ] All tests in ./test.sh pass
[ ] README and other documentation updated
[ ] Pending changelog updated

Closing in favor of https://github.com/ad-astra-video/go-livepeer/pull/30


Make Work for RSocrata

Get and clean Atlanta Police Department Data get.apd.data <- function(selector.text, save.location){      require(rvest)      require(dplyr)      temp <- tempfile()      download.file(           read_html("http://www.atlantapd.org/crimedatadownloads.aspx") %>%                 html_nodes(selector.text) %>%                 html_attr("href") %>%                 paste0("http://www.atlantapd.org/", .),           destfile = temp)      # Save filename of csv crime data raw file, for later use      crime.data.filename <- unzip(temp, list = TRUE)$Name      # Unzip crime data to "data" folder and close temp file placeholder      unzip(temp, exdir = save.location)      unlink(temp)      crime.data.filename
Was there a reason why you did it this way instead of using SODA? Other PD's could definitely take advantage of this if it was using RSocrata from Chicago.

Good point.  That will probably be the best route; I think we need to clean up the data in Socrata, though.  Looks like there's a difference starting in 2015:
Socrata Data:

COBRA051916.csv file:

Possible Cause: import tool is duplicating rows

@stuagano would it make more sense to do the ETL via https://dev.socrata.com/connectors/pentaho-kettle.html instead of the R script in this repo?  Maybe if there was a way to schedule an R ETL job it could work.  What do you think?
Here's the ETL current process (it would be nice to automate the last step of publishing cleaned data to Socrata):


🐛 Bug Report: Unable to compile the application after installing opencost plugin

📜 Description
Backstage (v.1.18.0) application fails to compile after installing opencost plugin, with error of unable to find image pig.png in the required folder within node_modules
Application is able to compile and build  properly once image(any image) is placed in folder<EMAIL_ADDRESS>👍 Expected behavior
Application should compile without any issues with standard opencost plugin (getting necessary assets into node_modules as per the import)
👎 Actual Behavior with Screenshots
When run locally I get following message after compiling the application with yarn: build
Error Message :
Failed to compile.
Module not found: Can't resolve '../../images/pig.png' in '../backstage/node_modules/@backstage/plugin-opencost/dist/esm'
<EMAIL_ADDRESS>👟 Reproduction steps

Followed all steps in the document,
Compile the application.

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Yep this doesn't work: https://github.com/backstage/backstage/blob/7accd01a2cde796a4ee474e56dd2b7057cd404e0/plugins/opencost/src/components/OpenCostPage/OpenCostPage.tsx#L31
Something you could look into @mattray?

Assign it to me and I'll see about getting a patch ASAP.

I wrote the plugin using yarn install - yarn dev without the npx @backstage/create-app@latest step, so I never ran into this. I'll likely need to update the instructions as well as provide a patch.

I believe this fixes it, the backstage-cli package build has the png file in the right place and it renders.

I believe this fixes it, the backstage-cli package build has the png file in the right place and it renders.

Hi guys! any update how to integrate this plugin inside a project created with npx as a common plugin? not sure the best way to run this 'backstage-cli package build' without a rg role.
ta!


mdh should "secondary-output", not "first-output"

Mod source
Factorio Mod Portal
Which mod are you having an issue with?

[ ] pyalienlife
[X] pyalternativeenergy
[ ] pycoalprocessing
[ ] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[ ] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[ ] Graphical
[ ] Crash
[ ] Progression
[X] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
mdh should "secondary-output", not "first-output"
Steps to reproduce
No response
Additional context
No response
Log file
No response

Fixed. Thank you.
https://github.com/pyanodon/pyalternativeenergy/commit/c26fbbd79f455894c838626c720fbc55b9972700


Refactoring blog schema and fixing pic function

This is some preliminary cleanup so that I can extend the schema to support mutations.
There was also a bug in the schema when retrieving pic data, this fixes that as well.

Instead of having a hard coded list of articles, a new article will be generated every time a new article ID is passed for. This is a stepping stone to getting mutation support around. This is all accomplished by maintaining global state inside of an agent and referencing that article map when requesting data.


Auto install fails for @vitest/coverage-v8 when using bun

Describe the bug
I added vitest as a script called test, then ran bun run test --coverage. Since I didn't have the coverage package installed I was prompted to install it. Bun then crashes.
Happy to open this under bun's issue tracker if it's more of a issue on their end.
Failed to parse argument due to unexpected single dash

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Unhandled Error ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
Error: Command failed with exit code 1: bun install -D @vitest/coverage-v8
    at makeError$1 (file:///Users/xo/code/ImLunaHey/vote/node_modules/vitest/dist/chunk-install-pkg.dd40cbef.js:978:11)
    at handlePromise (file:///Users/xo/code/ImLunaHey/vote/node_modules/vitest/dist/chunk-install-pkg.dd40cbef.js:1514:26)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async ensurePackageInstalled (file:///Users/xo/code/ImLunaHey/vote/node_modules/vitest/dist/vendor-node.caa511fc.js:83:5)
    at async startVitest (file:///Users/xo/code/ImLunaHey/vote/node_modules/vitest/dist/vendor-node.caa511fc.js:20493:12)
    at async start (file:///Users/xo/code/ImLunaHey/vote/node_modules/vitest/dist/cli.js:131:17) {
  shortMessage: 'Command failed with exit code 1: bun install -D @vitest/coverage-v8',
  command: 'bun install -D @vitest/coverage-v8',
  escapedCommand: 'bun install -D "@vitest/coverage-v8"',
  exitCode: 1,
  signal: undefined,
  signalDescription: undefined,
  stdout: undefined,
  stderr: undefined,
  failed: true,
  timedOut: false,
  isCanceled: false,
  killed: false
}

error: script "test" exited with code 1 (SIGHUP)

Reproduction
N/A
System Info
System:
    OS: macOS 12.6.6
    CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
    Memory: 364.80 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.3.0 - ~/.nvm/versions/node/v18.3.0/bin/node
    npm: 8.19.3 - ~/.nvm/versions/node/v18.3.0/bin/npm
    pnpm: 8.5.1 - ~/.nvm/versions/node/v18.3.0/bin/pnpm
  Browsers:
    Chrome: 115.0.5790.170
    Safari: 16.5
    Safari Technology Preview: 16.4
  npmPackages:
    vitest: ^0.34.1 => 0.34.1

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

If the command failed, then the command failed. Looks like it cannot install @vitest/coverage-v8 for some reason. I would recommend opening an issue in their repo.


Use MVVM architecture using hilt as DI

Is your feature request related to a problem? Please describe.
Currently the app dose follow MVVM architecture or any DI framework. Without out using the MVVM the room DB every time fetch  all data whenever the activity recreate like rotating phone. If there is more data it leads to a performance issue. Implementing MVVM leads the code more maintainable.
Describe the solution you'd like
I would like to move all the data related part to the view models that is lifecycle aware component. And use Hilt for Dependency Injection.
Describe alternatives you've considered
NA
Additional context
NA
Code of Conduct

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

Hi I'm from GSSoC'24. I would like to implement DI and MVVM to the project please assign it to me.

@Rajkumarbhakta
I am assigning this task to you. Please ensure you implement the feature in the best possible way. Also, star ⭐ the repo for better reach. Connect with me on:
GitHub: https://github.com/Suryansh1720001
LinkedIn: https://www.linkedin.com/in/itssuryansh/

@Suryansh1720001 Please add a lavel to this like level 1 ,2,3


superset怎么关闭每次查询的缓存，是在配置文件中配置嘛，请教一下在什么文件里面，怎么配置？

Screenshot
[drag & drop image(s) here!]
Description
[describe the issue here!]
Design input
[describe any input/collaboration you'd like from designers, and
tag accordingly. For design review, add the
label design:review. If this includes a design proposal,
include the label design:suggest]

This seems like a Q&A thread rather than a bug report. Hopefully, the answer above is sufficient to close this thread. I'll do so anyway since it's been so long since there was a comment here. If this needs revisiting/reopening, just say the word.


Adding edge encoding utility

If wanted, here is a utility function for remapping the species edges onto the gene edges after reading in the gene tree.
Some of the stuff I've been working on uses the edge mappings and these mappings are lost when reading/writing the gene trees.
Let me know if  this seems useful but you want more tests, documentation, clarification, etc.

That looks like a very useful and even necessary utility indeed. Thank you so much! I will review asap.
My first high-level thoughts are:

The test would ideally use a gene tree read from file, not a gene tree simulated during the tests, to show the main reason why this function exists and also to have an input gene tree with the default (empty) internal attributes, especially those modified by the function.
Given how useful this function could be to others, it would be great to export it and to add a section or subsection or page showing it in the documentation manual.

@jjustison : could you please place this pull request on a branch other than your main branch? that will make it possible for all developers here to make edits to your PR. No rush, as you know!

I created a new branch called edge_encoding. I am not sure if the the proper git flow but I will close this PR and open one from that branch.


Option -k <key_name> no longer exists

Hi,
When I tried to use option -k <key_name_other_than_credstash> to put some credential and I got error.
credstash -p dpe_non_prod --key cloudwatch_feeder  put testme testvalue
usage: credstash [-h] [-r REGION] [-t TABLE] [-p PROFILE | -n ARN]
{delete,get,getall,keys,list,put,putall,setup} ...
credstash: error: invalid choice: 'cloudwatch_feeder' (choose from 'delete', 'get', 'getall', 'keys', 'list', 'put', 'putall', 'setup')
Same if I use short opt  -k
Is it a bug or the support for that option is dropped?
Thanks

--key is an option for the put command. you need to do
credstash -p dpe_non_prod put --key cloudwatch_feeder  testme testvalue

Thanks looks like it works


Fix cross-bank PBR abs ops in 64tass output

The regression tests don't work with the last few releases of 64tass, because the assembler became more strict about things like doing a JSR to a label in bank zero from a different bank.  (This isn't a sensible thing to do, but SourceGen has to deal with it somehow if that's what the project says to do.  Other assemblers see a 16-bit value for a 16-bit operand and just roll with it.)
64tass allows ",k" to be appended to instructions to force them to be handled relative to the current bank.
Longer discussion: https://sourceforge.net/p/tass64/bugs/58/

Fixed in https://github.com/fadden/6502bench/releases/tag/v1.7.5-dev4


MNE - GPU Installation Fail

'MNE_USE_CUDA' is not recognized as an internal or external command,
operable program or batch file.

'mne.utils.set_config' is not recognized as an internal or external command,
operable program or batch file.

Note:
conda install cupy; was successful
MNE-python installation successful
My notebook has Nvidea RTX 2060

The first one didn't work because Bash died not allow spaces around the equals sign when assigning values to environment variables. The second one didn't work because you're trying to run a python command in the Bash shell.

Issue still prevails

The first one didn't work because Bash does not allow spaces around the equals sign when assigning values to environment variables. The second one didn't work because you're trying to run a python command in the Bash shell.

Issue still prevails

Is that actually even a Bash shell? It looks like a windows command prompt based on the > end-line character

It is anaconda prompt snip, I thought the given code snippet was even applicable for Windows machine

All of our code samples are Bash, for windows command prompt shell users -- or users of other shells generally -- you'll need to look up how to set environment variables in your given shell

I am still a newbie, it will be very helpful if you could guide me in a simple way

I am still a newbie, it will be very helpful if you could guide me in a simple way

Can you direct me to the correct syntax to create a new environment variable(MNE_USE_CUDA)?
If that's what you are trying to say in your last message to create the environment variable first and then implement the above code.

maybe this can help http://www.dowdandassociates.com/blog/content/howto-set-an-environment-variable-in-windows-command-line-and-registry/
none of the mne dev work on windows sorry

Windows is not my main OS for dev but here is what I do to set variables on Windows 10:
On Settings:

In the search fileld: Edit environment variables:

Then New:

And finally OK:

Hoping it could be useful.

@Vatsal-Malaviya what @GuillaumeFavelier suggested is one way to set environment variables in Windows.  But perhaps easier in your case is to run the second command you tried: mne.set_config('MNE_USE_CUDA', 'true'), but run it in a python interpreter instead of in the windows command shell.  In other words:
(mne) C:\Users\Vatsal\Desktop> python
>>> import mne
>>> mne.set_config('MNE_USE_CUDA', 'true')


Adding additional aggregate benchmarks for tsm1

This will help us address performance problems in the underlying tsm1
implementations of the aggregate iterators.

During a run of megacheck the following issues were discovered:
/tmp/587897810/src/github.com/influxdata/influxdb/tsdb/engine/tsm1/engine_test.go:1753:6: func benchmarkIterator is unused (U1000)


Minor: in gen_model.learned_lf_stats, call "acc" -> "est. acc" to be clear?

Think this is actually clear enough


[icu]Build error

Host Environment

OS: [Windows]
Compiler: vs2019

To Reproduce
Steps to reproduce the behavior:
vcpkg install qtbase
Failure logs
C:\vcpkg>vcpkg.exe install qtbase
warning: Starting with the September 2023 release, the default triplet for vcpkg libraries will change from x86-windows to the detected host triplet (x64-windows). To resolve this message, add --triplet x86-windows to keep the same behavior.
Computing installation plan...
The following packages will be built and installed:

icu[core,tools]:x64-windows -> 72.1#3
icu[core]:x86-windows -> 72.1#3
libjpeg-turbo[core]:x64-windows -> <IP_ADDRESS>
libjpeg-turbo[core]:x86-windows -> <IP_ADDRESS>
libpq[core,lz4,openssl,zlib]:x64-windows -> 15.2#1
libpq[core,lz4,openssl,zlib]:x86-windows -> 15.2#1
lz4[core]:x64-windows -> 1.9.4#1
lz4[core]:x86-windows -> 1.9.4#1
opengl[core]:x64-windows -> 2022-12-04#3
opengl[core]:x86-windows -> 2022-12-04#3
opengl-registry[core]:x86-windows -> 2022-09-29#1
pcre2[core]:x64-windows -> 10.40#1
pcre2[core]:x86-windows -> 10.40#1
qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,opengl,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,thread,widgets,zstd]:x64-windows -> 6.4.3#1
qtbase[brotli,concurrent,core,dbus,default-features,doubleconversion,freetype,gui,harfbuzz,icu,jpeg,network,opengl,openssl,pcre2,png,sql,sql-psql,sql-sqlite,testlib,thread,widgets,zstd]:x86-windows -> 6.4.3#1
sqlite3[core]:x64-windows -> 3.40.1#1
sqlite3[core]:x86-windows -> 3.40.1#1
zstd[core]:x64-windows -> 1.5.4#1
zstd[core]:x86-windows -> 1.5.4#1
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
Detecting compiler hash for triplet x86-windows...
Restored 0 package(s) from C:\Users\gwang1\AppData\Local\vcpkg\archives in 547.9 us. Use --debug to see more details.
Installing 1/19 icu:x64-windows...
Building icu[core,tools]:x64-windows...
-- Using cached icu4c-72_1-src.tgz.
-- Cleaning sources at C:/vcpkg/buildtrees/icu/src/c-72_1-src-deba61ed52.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/vcpkg/downloads/icu4c-72_1-src.tgz
-- Applying patch disable-escapestr-tool.patch
-- Applying patch remove-MD-from-configure.patch
-- Applying patch fix_parallel_build_on_windows.patch
-- Applying patch fix-extra.patch
-- Applying patch mingw-dll-install.patch
-- Applying patch disable-static-prefix.patch
-- Applying patch fix-win-build.patch
-- Using source at C:/vcpkg/buildtrees/icu/src/c-72_1-src-deba61ed52.clean
-- Found external ninja('1.10.2').
-- Getting CMake variables for x64-windows
-- Using cached msys-gzip-1.11-1-x86_64.pkg.tar.zst.
-- Using cached msys-bash-5.1.008-1-x86_64.pkg.tar.zst.
-- Using cached msys-autoconf-2.71-3-any.pkg.tar.zst.
-- Using cached msys-autoconf-archive-2019.01.06-1-any.pkg.tar.xz.
-- Using cached msys-diffutils-3.8-2-x86_64.pkg.tar.zst.
-- Using cached msys-binutils-2.37-5-x86_64.pkg.tar.zst.
-- Using cached msys-libtool-2.4.7-3-x86_64.pkg.tar.zst.
-- Using cached msys-file-5.41-2-x86_64.pkg.tar.zst.
-- Using cached msys-zlib-1.2.11-1-x86_64.pkg.tar.xz.
-- Using cached msys-libbz2-1.0.8-3-x86_64.pkg.tar.zst.
-- Using cached msys-coreutils-8.32-2-x86_64.pkg.tar.zst.
-- Using cached msys-grep-3.0-2-x86_64.pkg.tar.xz.
-- Using cached msys-sed-4.8-2-x86_64.pkg.tar.zst.
-- Using cached msys-libpcre-8.45-1-x86_64.pkg.tar.zst.
-- Using cached msys-m4-1.4.19-2-x86_64.pkg.tar.zst.
-- Using cached msys-automake-wrapper-11-4-any.pkg.tar.zst.
-- Using cached msys-gawk-5.1.0-2-x86_64.pkg.tar.zst.
-- Using cached msys-mpfr-4.1.0-1-x86_64.pkg.tar.zst.
-- Using cached msys-gmp-6.2.1-1-x86_64.pkg.tar.zst.
-- Using cached msys-libreadline-8.1.001-1-x86_64.pkg.tar.zst.
-- Using cached msys-ncurses-6.2-2-x86_64.pkg.tar.zst.
-- Using cached msys-automake1.16-1.16.3-3-any.pkg.tar.zst.
-- Using cached msys-perl-5.32.1-2-x86_64.pkg.tar.zst.
-- Using cached msys-libcrypt-2.1-3-x86_64.pkg.tar.zst.
-- Using cached msys-pkg-config-0.29.2-4-x86_64.pkg.tar.zst.
-- Using cached msys-make-4.3-3-x86_64.pkg.tar.zst.
-- Using cached msys-findutils-4.8.0-1-x86_64.pkg.tar.zst.
-- Using cached msys-libintl-0.21-1-x86_64.pkg.tar.zst.
-- Using cached msys-libiconv-1.16-2-x86_64.pkg.tar.zst.
-- Using cached msys-gcc-libs-11.2.0-3-x86_64.pkg.tar.zst.
-- Downloading https://repo.msys2.org/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst;https://www2.futureware.at/~nickoe/msys2-mirror/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst;https://mirror.yandex.ru/mirrors/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst;https://mirrors.tuna.tsinghua.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst;https://mirrors.ustc.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst;https://mirror.bit.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst;https://mirror.selfnet.de/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst;https://mirrors.sjtug.sjtu.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst -> msys-msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst...
[DEBUG] To include the environment variables in debug output, pass --debug-env
[DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Feature flag 'registries' unset
[DEBUG] Feature flag 'versions' unset
Downloading https://repo.msys2.org/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst
Downloading https://www2.futureware.at/~nickoe/msys2-mirror/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst
Downloading https://mirror.yandex.ru/mirrors/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst
Downloading https://mirrors.tuna.tsinghua.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst
Downloading https://mirrors.ustc.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst
Downloading https://mirror.bit.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst
warning: Download failed -- retrying after 1000ms
warning: Download failed -- retrying after 2000ms
warning: Download failed -- retrying after 4000ms
Downloading https://mirror.selfnet.de/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst
Downloading https://mirrors.sjtug.sjtu.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst
error: Failed to download from mirror set
error: https://repo.msys2.org/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: failed: status code 404
error: https://www2.futureware.at/~nickoe/msys2-mirror/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: failed: status code 404
error: https://mirror.yandex.ru/mirrors/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: failed: status code 404
error: https://mirrors.tuna.tsinghua.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: failed: status code 404
error: https://mirrors.ustc.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: failed: status code 404
error: https://mirror.bit.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: WinHttpSendRequest failed with exit code 12007
error: https://mirror.bit.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: WinHttpSendRequest failed with exit code 12007
error: https://mirror.bit.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: WinHttpSendRequest failed with exit code 12007
error: https://mirror.bit.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: WinHttpSendRequest failed with exit code 12007
error: https://mirror.selfnet.de/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: failed: status code 404
error: https://mirrors.sjtug.sjtu.edu.cn/msys2/msys/x86_64/msys2-runtime-3.2.0-8-x86_64.pkg.tar.zst: failed: status code 404
[DEBUG] D:\a_work\1\s\src\vcpkg\base\downloads.cpp(956):
[DEBUG] Time in subprocesses: 0 us
[DEBUG] Time in parsing JSON: 3 us
[DEBUG] Time in JSON reader: 0 us
[DEBUG] Time in filesystem: 13186 us
[DEBUG] Time in loading ports: 0 us
[DEBUG] Exiting after 14.71 s (14674831 us)

CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:32 (message):
  Failed to download file with error: 1
  If you are using a proxy, please check your proxy setting. Possible causes are:

  1. You are actually using an HTTP proxy, but setting HTTPS_PROXY variable
     to `https://address:port`. This is not correct, because `https://` prefix
     claims the proxy is an HTTPS proxy, while your proxy (v2ray, shadowsocksr
     , etc..) is an HTTP proxy. Try setting `http://address:port` to both
     HTTP_PROXY and HTTPS_PROXY instead.

  2. If you are using Windows, vcpkg will automatically use your Windows IE Proxy Settings
     set by your proxy software. See https://github.com/microsoft/vcpkg-tool/pull/77
     The value set by your proxy might be wrong, or have same `https://` prefix issue.

  3. Your proxy's remote server is out of service.

  If you've tried directly download the link, and believe this is not a temporary
  download server failure, please submit an issue at https://github.com/Microsoft/vcpkg/issues
  to report this upstream download server failure.

Call Stack (most recent call first):
scripts/cmake/vcpkg_download_distfile.cmake:270 (z_vcpkg_download_distfile_show_proxy_and_fail)
scripts/cmake/vcpkg_acquire_msys.cmake:26 (vcpkg_download_distfile)
scripts/cmake/vcpkg_acquire_msys.cmake:67 (z_vcpkg_acquire_msys_download_package)
scripts/cmake/vcpkg_acquire_msys.cmake:345 (z_vcpkg_acquire_msys_declare_package)
scripts/cmake/vcpkg_configure_make.cmake:224 (vcpkg_acquire_msys)
ports/icu/portfile.cmake:55 (vcpkg_configure_make)
scripts/ports.cmake:147 (include)
error: building icu:x64-windows failed with: BUILD_FAILED
error: Please ensure you're using the latest port files with git pull and vcpkg update.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+icu
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[icu]+Build+error
Include '[icu] Build error' in your bug report title, the following version information in your bug description, and attach any relevant failure logs from above.
vcpkg-tool version: 2023-02-16-12e657924d99511514c0287ca5ce46882d3657c7
vcpkg-scripts version: 5b1214315 2023-03-30 (6 hours ago)

it seems the package version defined in scripts/cmake/vcpkg_acquire_msys.cmake needs to be updated

I got this issue as well. It seems that MSYS is deleting old ports. Is there any archive for them?

Updating the msys version is a patching of current problems. The problem is deeper.
I caught a similar error when trying to rebuild a project that hasn't changed in a year. It turns out that using vcpkg I cannot guarantee the possibility of rebuilding in a month / year.
The solution to the problem is that vcpkg must download dependencies from its own mirror, on which files are stored for a long time.

Duplicate of https://github.com/microsoft/vcpkg/issues/30542

@yanggaome, This issue is fixed by PR https://github.com/microsoft/vcpkg/pull/30546, please rerun your pipeline.