Gemspec permissions issue

I've started receiving this issue:
$ brew cask
Error: Permission denied - /Library/Ruby/Gems/2.0.0/specifications/bundler-1.10.6.gemspec

I'm sure its a system specific thing, but it only happens when running cask.
These are the only warnings I get for brew doctor:
$ brew doctor

Warning: Unbrewed header files were found in /usr/local/include.
Warning: Homebrew's sbin was not found in your PATH but you have installed
Warning: Some keg-only formula are linked into the Cellar.

Any help would be appreciated. Thanks.
$ brew update && brew upgrade brew-cask && brew cleanup && brew cask cleanup
Already up-to-date.
Error: caskroom/cask/brew-cask 0.56.0 already installed

What about brew cask doctor?

Can't even get that far, just get the gemspec error.

Have you tried uninstalling and reinstalling homebrew-cask?

Ha! Hmm, it didn't work, however it still had the issue while it was uninstalled??
$ brew uninstall --force brew-cask
Uninstalling brew-cask... (5528 files, 22M)

How would this bit be possible?
$ brew cask
Error: Permission denied - /Library/Ruby/Gems/2.0.0/specifications/bundler-1.10.6.gemspec

$ brew install caskroom/cask/brew-cask
==> Installing brew-cask from caskroom/homebrew-cask
==> Cloning https://github.com/caskroom/homebrew-cask.git
Updating /Library/Caches/Homebrew/brew-cask--git
==> Checking out tag v0.56.0
🍺  /usr/local/Cellar/brew-cask/0.56.0: 2802 files, 11M, built in 18 seconds

$ brew cask
Error: Permission denied - /Library/Ruby/Gems/2.0.0/specifications/bundler-1.10.6.gemspec

Issue is definitely in your setup. What’s the output of which brew-cask?

Issue is definitely in your setup. Uninstall homebrew-cask again. After that, what’s the output of which brew-cask?

I thought it would be, I just can't figure out why.
$ brew uninstall brew-cask
Uninstalling /usr/local/Cellar/brew-cask/0.56.0... (2802 files, 11M)

$ which brew-cask
brew-cask not found

$ brew cask
Error: Permission denied - /Library/Ruby/Gems/2.0.0/specifications/bundler-1.10.6.gemspec

$ brew install brew-cask
==> Installing brew-cask from caskroom/homebrew-cask
==> Cloning https://github.com/caskroom/homebrew-cask.git
Updating /Library/Caches/Homebrew/brew-cask--git
==> Checking out tag v0.56.0
🍺  /usr/local/Cellar/brew-cask/0.56.0: 2802 files, 11M, built in 14 seconds

$ which brew-cask
/usr/local/bin/brew-cask

$ which ruby bundler
$ ruby --version
$ bundler --version
$ ls -l /Library/Ruby/Gems/2.0.0/specifications/bundler-1.10.6.gemspec

Have you installed any of those (ruby, bundler, homebrew, homebrew-cask) as root, at any point.

I don't recall doing so, but this is probably where the issue is:
$ which ruby bundler
/usr/bin/ruby
bundler not found

$ ruby --version
ruby 2.0.0p481 (2014-05-08 revision 45883) [universal.x86_64-darwin14]

$ bundler --version
zsh: permission denied: bundler

$ ls -l /Library/Ruby/Gems/2.0.0/specifications/bundler-1.10.6.gemspec
-rw-------  1 root  wheel  1902  4 Aug 14:45 /Library/Ruby/Gems/2.0.0/specifications/bundler-1.10.6.gemspec

Have you experimented with trashing that problematic file temporarily?

The plot thickens...
$ sudo rm /Library/Ruby/Gems/2.0.0/specifications/bundler-1.10.6.gemspec
$ brew cask
Error: Permission denied - /Library/Ruby/Gems/2.0.0/specifications/github_api-0.12.4.gemspec
$ sudo rm /Library/Ruby/Gems/2.0.0/specifications/github_api-0.12.4.gemspec
$ brew cask
Error: Permission denied - /Library/Ruby/Gems/2.0.0/specifications/guard-2.13.0.gemspec

I said temporarily, as in moving it to another directory just to see if we could get another clue.
It’s clear you’re using the system ruby, though. That, together with sudo is surely what cause the problem — this is a permissions issue, after all. I searched a bit, and every problem similar to this one seems to have been solved by the asked installing RVM (or similar) and not using the system ruby.

I've switched to rvm and moved all the gemspecs in that folder, its started working again.
Thanks for your time, really appreciate it.

Glad it’s working.


Home-Assistant: Postgresql Nearing End of Support

Details
Home Assistant:
Per Home Assistant documentation, Postgres must be >= v12: https://www.home-assistant.io/blog/2021/11/03/release-202111/
Recorder / Databases

The Home Assistant project has now explicitly defined a set of databases that are supported by the recorder.

We now support the following databases:

    MariaDB ≥ 10.3
    MySQL ≥ 8.0
    PostgreSQL ≥ 12
    SQLite ≥ 3.32.1

All other databases and database versions are now deprecated.

Bitnami is tracking the v11 version and appears to be working on a v13 PR.
Per logs in hass container, recorder will no longer start with a non-supported version of Postgres starting with Home Assistant 2022.2.

It doesn't look like the referred issue is getting much traction. How would we best tackle this?
We could specify a default "custom" image version for the postgresql chart dependency in our values.yaml?
We should probably have renovate update those on a similar interval to the helm chart (every 1st of the month), to keep them up to date.


CCTV camera should be allowed on a vertex

CCTV cameras are mounted on walls.

Sounds good..
looks like you need to change:

https://github.com/openstreetmap/iD/blob/master/data/presets/presets/man_made/surveillance.json
https://github.com/openstreetmap/iD/blob/master/data/presets/presets/man_made/surveillance_camera.json
https://github.com/openstreetmap/iD/blob/master/data/presets/presets/highway/speed_camera.json


bep6plugin_il2cpp creates incorrect namespace

dotnet new bep6plugin_il2cpp -n bepinex-2018-test1

Creating a project with hyphens in -n, It's creates PluginInfo.cs as below
// PluginInfo.cs
namespace bepinex-2018-test1
{
    public static class PluginInfo
    {
        public const string PLUGIN_GUID = "com.example.MyPlugin";
        public const string PLUGIN_NAME = "MyPlugin";
        public const string PLUGIN_VERSION = "1.0.0";
    }
}

As expected, this fails the build.
Perhaps it should be replaced with an underscore like the namespace given to Plugin.cs?
// Plugin.cs
using BepInEx;
using BepInEx.IL2CPP;

namespace bepinex_2018_test1
{
    [BepInPlugin(PluginInfo.PLUGIN_GUID, PluginInfo.PLUGIN_NAME, PluginInfo.PLUGIN_VERSION)]
    public class Plugin : BasePlugin
    {
        public override void Load()
        {
            // Plugin startup logic
            Log.LogInfo($"Plugin {PluginInfo.PLUGIN_GUID} is loaded!");
        }
    }
}

I'm not a dotnet expert. If this is not a template issue, don't worry and close it.

Looks like this is no longer an issue in master.


Use OrdinaryDiffEqTsit5.jl and OrdinaryDiffEqLowStorageRK.jl

Since we only use Tsit5 and RDPK3SpFSAL35 in the examples, I thought it would be nice to also only use OrdinaryDiffEqTsit5.jl and OrdinaryDiffEqLowStorageRK.jl in the examples and recommend installing OrdinaryDiffEqTsit5.jl in the README and docs. This should significantly reduce precompile time and generally make things smoother and more lightweight. What do you think, @ranocha?

Sounds good to me! However, we need to wait for Trixi.jl to upgrade SciMLBase.jl etc. for this to work properly, don't we?

Sounds good to me! However, we need to wait for Trixi.jl to upgrade SciMLBase.jl etc. for this to work properly, don't we?

We only need that if we want to have Trixi.jl and DispersiveShallowWater.jl in the same project. But this currently fails already with the newest version of DispersiveShallowWater.jl, i.e. installing Trixi.jl in a project, where DispersiveShallowWater.jl is installed will downgrade DispersiveShallowWater.jl to an old version (something like v3... if I remember correctly) because the newer DispersiveShallowWater.jl version require versions of RecursiveArrayTools.jl, which are not compatible with Trixi.jl, cf. https://github.com/trixi-framework/Trixi.jl/pull/2150.

However, we would need to stop supporting Julia v1.9, I guess.

That's fine with me for this repo 👍

Ok, this required more version bumps than expected. The SciML repos have pretty high compat bounds sometimes... But Downgrade is finally happy. A couple of remarks:

The failing benchmark action is expected because it uses the main branch as reference, but the benchmark/Project.toml  from this PR. This means it still wants to use OrdinaryDiffEq.jl, but it cannot because it's not in the new Project.toml. IMHO, this can be ignored in this PR. Benchmark should not be influenced by this PR.

Codecov is failing probably because of some change in Julia v1.10 compared to v1.9 regarding the "lispy" tuple iteration we use in the AnalysisCallback, which means the final call to the empty tuple is not covered anymore. I already saw in previous PRs that these lines are only caught by the Julia v1.9 action, which is disabled now. But I think that's also fine.

Finally, as a replacement for the Julia v1.9 action I added a test using Julia v1.11. This currently fails for an allocation fix. Looks like Julia v1.11 allocates more for this example. Are you fine simply updating the allocations threshold, @ranocha or do you have an idea where this could come from?

Ok, this required more version bumps than expected. The SciML repos have pretty high compat bounds sometimes... But Downgrade is finally happy. A couple of remarks:

The failing benchmark action is expected because it uses the main branch as reference, but the benchmark/Project.toml  from this PR. This means it still wants to use OrdinaryDiffEq.jl, but it cannot because it's not in the new Project.toml. IMHO, this can be ignored in this PR. Benchmark should not be influenced by this PR.

👍

Codecov is failing probably because of some change in Julia v1.10 compared to v1.9 regarding the "lispy" tuple iteration we use in the AnalysisCallback, which means the final call to the empty tuple is not covered anymore. I already saw in previous PRs that these lines are only caught by the Julia v1.9 action, which is disabled now. But I think that's also fine.

👍

Finally, as a replacement for the Julia v1.9 action I added a test using Julia v1.11. This currently fails for an allocation fix. Looks like Julia v1.11 allocates more for this example. Are you fine simply updating the allocations threshold, @ranocha or do you have an idea where this could come from?

That's fine as well. Julia v1.11 introduced the Memory type so that array operations can lead to slightly more allocations tracked by Julia - but are often more efficient since the compiler can optimize the code better.


Suggestion: robots.txt denied event

What is the current behavior?
URLs denied by the robots.txt are silently swallowed.
What is the expected behavior?
When an URL is denied because of the robots.txt we should either have an event, a callback or something.
What is the motivation / use case for changing the behavior?
It could be useful to track all URLs that were discarded because of the robots.txt (and not because they were not found or whatever the reason)

I just saw that it was the purpose of "RequestSkipped".
Then maybe we just need to forward the reason so this event :)


Site goes extremely when the browser connection limit (6) is reached

What do you do on discourse if the user open 6 tabs with six pages that uses message_bus? That will leave any empty connection slot for another work on the website, as currently browsers (chrome/firefox) has a limit of 6 connections per domain.

http://tools.ietf.org/id/draft-loreto-http-bidirectional-07.html#connectionlimit

using the dom storage to share only one long polling connection is a possible solution: http://meta.stackexchange.com/a/85045

a library as a solution for that: https://bitbucket.org/balpha/bytheway.js/src.

but it also seems simple to do it manually: http://blog.fastmail.com/2012/11/26/inter-tab-communication-using-local-storage/ (demo at http://www.nmjenkins.com/intertab.html)

Is that necessary? When the tab isn't active messagebus will change from longpoll to polling every minute.

@jnstq and how for long will it keep the connection open?

When the tab is not active, it's just a normal requests, it's fast. I don't know how long the connection is open, I guess that is managed by the browser.

I do not think we cancel current requests when you open new tabs (need to
confirm by looking at code) , the long poll will expire within 29 seconds
and change to a poll.
it is conceivable that if you open 6 tabs in a row real fast you can cause
issues, but in practice that does not happen, regardless if we need to add
a .cancel we can its not a big deal.
On Fri, Oct 2, 2015 at 9:50 PM, Jon Stenqvist<EMAIL_ADDRESS>wrote:

When the tab is not active, it's just a normal requests, it's fast. I
don't know how long the connection is open, I guess that is managed by the
browser.
—
Reply to this email directly or view it on GitHub
https://github.com/SamSaffron/message_bus/issues/54#issuecomment-144997406
.

@SamSaffron it is not about cancelling, it is about reuse of one long polling by all tabs.

@riking has a work in progress PR that does that, it gets fairly tricky but
is doable
I do not want anything to break so it is going to take a while to test
On Saturday, 3 October 2015, Bráulio Bhavamitra<EMAIL_ADDRESS>wrote:

@SamSaffron https://github.com/SamSaffron it is not about cancelling,
it is about reuse of one long polling by all tabs.
—
Reply to this email directly or view it on GitHub
https://github.com/SamSaffron/message_bus/issues/54#issuecomment-145184112
.

The PR doesn't cancel any requests, though, due to whatwg/fetch#27.

I am closing this for now,
If you run HTTP/2 which is my recommendation the 6 connections thing is a non issue. If you are not, well you have to work really hard to make it an issue due to the fallback. I am open to adding web workers but it is hellishly complex and not fully supported across platforms.


Support Additional Plugins

It would be nice to see support for other plugins that handle "urls".
For example, KeePassRPC allows storing additional URLs in a json encoded string on the entry, which allows you to have multiple urls for a single entry.
Additional, the Keepass2Android application uses KP2A_URL and KP2A_URL_# (When KP2A_URL is already used) fields on the entry to store additional urls.
This could be handled with a "First in, First Out" approach, in the sense that once an icon has been found you can ignore the remaining urls.

Hey @Nikey646, do you this plugins that you have mentioned?
Can you attach a sample database with just a few entries?

Hello, you like a sample kdbx with KP2A_URL_* from Keepass2Android  ?

@4-FLOSS-Free-Libre-Open-Source-Software if you have one, that you are have sure that is working. Would be great ;)

I would also love this feature!
Also, another good setting could be to let YAFD try to "URLize" the title eventhough it has an URL.
I use Kee for communicating with firefox and use the url-field to save applinks on my phone like: androidapp://com.airbnb.android so I cant use url to get the icons on those.

It would be so nice, if you could support every "Custom field" to choose from. Like "Use title field, if URL field is empty".

I have some entries that have 20 custom fields.  He'd have to first be able to parse the field names, and then the data, which might be a bit hard for the ones that I have marked with "Enable in-memory protection".
Maybe just the title field?
Although, really, it needs a website to grab a favicon from - so is there really a need to look at other fields?

@4-FLOSS-Free-Libre-Open-Source-Software if you have one, that you are have sure that is working. Would be great ;)

I have a few working KP2A_URL that are working for me, if you still need this, man.

I would also love this feature!
Also, another good setting could be to let YAFD try to "URLize" the title eventhough it has an URL.
I use Kee for communicating with firefox and use the url-field to save applinks on my phone like: androidapp://com.airbnb.android so I cant use url to get the icons on those.

Not only storing URL of website but also of apps packe name.
GitHub app example for KeePass 2 Android:
AndroidApp1
androidapp://com.github.android
Bad luck trying to get favicon from something like reverse https://android.github.com/


Add yarn start:ssr command

Adds a new command yarn start:ssr as well as some docs that allows us to get around the yarn link issue + styled-components 4. This approach is pretty naive. (If anyone has any suggestions for a better approach please send them over!)

Actually, I have a suggestion that I think may work. What if we put a webpack alias in force for styled-components that always resolved to a single location? That would ensure that webpack always resolves that import to the proper place. A babel plugin could potentially be used too if we needed it to work for node. The ultimate idea being that it's always resolved to the version we want instead of potentially the version in reaction.

What if we put a webpack alias in force for styled-components that always resolved to a single location?

This would work if we didn't JIT compile server-side content but alas.

babel plugin to replace the import with an absolute path? styled-components -> /home/user/force/node_moduels/styled-components... where the path is actually correct, ha.

That could work! Trying to trace this stuff through deps and sub deps + how babel will pick up other .babelrc's is so mind melting.

For the record, this start:ssr command doesn't work for me. I still get a blank /collect page when running it. I'll follow up via slack to see if we can pair on it some time.

@pepopowitz - odd! I was testing this on /collect page (and other pages). Yeah let's SH when i'm back in office, wonder what's going on.


feat(AztecMacro): Function types and visibility

Update the aztec macros so that we have:

#[aztec(private)]: Private constrained functions
#[aztec(public)]: Public constrained functions
#[aztec(internal)]: Functions only callable via an aztec function call from the same contract (used with either of the two above)
#[aztec(query)]: Unconstrained query function (requires unconstrained)
unconstrained (no attribute): Unconstrained helper function that is inlined
no attribute: Constrained helper function that is inlined

Before implementing this, we still need agreement on the name aztec(query).
Full discussion in this forum thread.

I for one like query. It's distinct from other ethereum terms (e.g. it's not 'view'), so there'll be no confusion, and it nicely conveys the purpose.
We've also been toying with the idea of a fouth category, tentatively called 'provable', which would be functions whose execution you could easily prove off-chain (e.g. you'd submit a proof for NFT.owner_of(token_id) at some block number).


Update default parsers with is_new

Make use of the recent implemented parameter is_new that is passed to Parser.parse_fields
Functions like record.set_filepath, record.set_img_size, now only need to be called once, refer to this notebook for an example.
We need to update or default parsers to make use of this functionality:

[ ] coco_parser.py
[ ] voc_parser.py
[ ] via_parser.py

I will look this on the week end.

@aaodiall Thanks a lot! I just assigned the issue to you


Support pcap-ng format

Current pcap source input format type only supports classic PCAP and not PCAP-NG. This may entail a move to scapy from dpkt within the PcapFileMessageSource class.

interim workaround is to run tshark -F pcap -r {pcapng file} -w {pcap file} on pcap ng file to produce legacy pcap format supported by txcode


certificate error when connecting electrum

Since this week, my electrum can't connect to Raspiblitz any more. I updated to 1.10.0 the minimal installation with a fresh install, but that didn't help. Previously I had version 1.10.0. rc2 with worked smoothly until this week).
I work with the Electrum desktop app on windows 10.
In the Electrum log I found that the certificate of the local host has expired on the 22nd of September:
INFO | network | connecting to <IP_ADDRESS>:50002:s as new interface
20230929T181015.142126Z |     INFO | interface.[<IP_ADDRESS>:50002] | certificate has expired: Certificate (localhost) has expired (at 2023-09-22 07:47:17 UTC).
20230929T181015.156174Z |     INFO | interface.[<IP_ADDRESS>:50002] | succeeded in getting cert
20230929T181015.165157Z |  WARNING | interface.[<IP_ADDRESS>:50002] | Cannot connect to main server due to SSL error (maybe cert changed compared to "C:\Users\xxxxxx\AppData\Roaming\Electrum\certs\<IP_ADDRESS>"). Exc: ConnectError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: certificate has expired (_ssl.c:1129)'))
20230929T181015.166156Z |     INFO | network | couldn't launch iface <IP_ADDRESS>:50002:s -- CancelledError()
How and where can I change the certificate?
Just tried to connect Trezor Suite  to Raspiblitz and no issue there...
Tried to delete the certificate in Electrum in "certs", but that doesn't help

After thinking for a while, I changed the connection in my desktop Electrum from : 50002:s into 50001:t. That worked. I'm on my cabled home network.

I just downloaded Sparrow wallet and managed to connect it to Raspiblitz through 50002 and SSL enabled.
In de certs of Sparrow wallet on my windows pc, I found that my certificate there was dated at 22 -08-2022.
I deleted the Electrum cert and copied the sparrow cert into the Electrum cert directory, but unfortunately Electrum deleted that copied Sparrow cert and instead put the recent one that it finds and could not connect....
I also tried to open RTL in Firefox with https. Got the usual certificate error (LND autogenerated cert for local host). When I looked through the certificate, I found that the validation is from 28 July 2022 till 23 September 2023...So, outdated as described in the Electrum log. But no problem. After accepting, RTL starts.
Really don't know what to try else to solve this issue. I have seen two certificate files on Raspiblitz, but don't know how to read them nor how to change those (the date).

This is a client issue. Electrum uses the TOFU "trust on first use" principle. When a certificate from the server (here the Raspiblitz) expires, the client won't connect to that server even when you replace the certificate on the server with a new one, you need to issue a new certificate on the RaspiBlitz AND delete the old one on your client:
Replacing the server on the RaspiBlitz should be possible like this via SSH:

cd /home/electrumx/.electrumx/
openssl genrsa -out keyfile.key 2048
openssl req -new -key keyfile.key -out server.csr
openssl x509 -req -days 1825 -in server.csr -signkey server.key -out certfile.crt
service restart electrumx

On the client, delete the corresponding old cert from C:\Users\xxxxxx\AppData\Roaming\Electrum\certs\<IP_ADDRESS> or something like that.

electrs uses the nginx SSL proxy on port 5002 - if the SSL cert is outdated this needs to be handled by nginx
https://github.com/raspiblitz/raspiblitz/blob/624f8ca1a78d5935f94ea90af2136be088442c8a/home.admin/config.scripts/bonus.electrs.sh#L357
TODO: check nginx SSL certs - undo using the lnd one (system might not have lnd running)

After having installed Electrum on a Linux PC and connected to Electrs in my Raspiblitz, the same cert error popped up.
After SSH'd in, I recreated the LND Macaroons and TLS.cert from the Connect Menu.
That helped! I finally can connect through 50002:s instead of 50001:t

This is now also discussed here: https://github.com/raspiblitz/raspiblitz/issues/4431#issuecomment-2377624050 .. closing this issue as duplicate.


style system separation, clean up systems back to ECS

what this entails:

style system separation and back to main ECS
also https://github.com/Volumetrics-io/mrjs/issues/372
misc:

app --> App
moves old dom function from file --> mrjsUtils.CSS

need to do some remaining cleanup still for this pr
then next step would to be switch to an enum based update for the css sytem instead of all the separate methods - might do it in a separate pr instead depending on timing

As discussed on discord.

mrjsUtils.Name -> mrjsUtils.name
I'm seeing some z-fighting for UI that I'm not seeing on main
The flickering bug returns after entering/exiting XR along with a big drop in FPS and eventual crash

z-fighting
https://github.com/Volumetrics-io/mrjs/assets/19628643/24098589-8f09-44fd-9ae1-f4bbb0199b14

FPS drop, flickering, crash
https://github.com/Volumetrics-io/mrjs/assets/19628643/7c0ad831-51e6-406b-870c-e57e7053ea40

naming fix handled by #380 , looking into the other noted things on this pr ~

z-fighting
z-fighting.mp4

michael said the z-fighting mightve been introduced by recent mr-panel changes, he's looking into ~

FPS drop, flickering, crash
flicker-crash.mp4

i'm still looking into this atm

given #372 (comment) - flickering and z-fighting are not specific to this pr - going to clean up some remaining items
if crashing is specific to this pr then will solve here, otherwise will solve that// z-fighting // layout (ie #372) in a separate pr

solved and merged in from main now that those are in -- going to do final checks here

made new bug for the crash as it seems very specific to debug mode - did enter/exit vr a few different times in a release session version and it was happy


feat(lightshaft): add intensity and flashiness properties

Missing two properties in lightshaft.

you are on the right track, however you need to edit the cwxml ymap.py file as well. https://i.imgur.com/CicNJGa.png

you are on the right track, however you need to edit the cwxml ymap.py file as well. https://i.imgur.com/CicNJGa.png

Do you mean in the direction of the reading? or something else? an example in mind?

ignore my comment, this should work fine


ProtocolError: Protocol error (Network.setUserAgentOverride): Target closed.

Hello,
I have a problem with the cluster, if I go normally with puppeter, all pages are correctly scrapped.
But when I use "cluster" I get this error.
Sometimes it's okay, but many times it throws this error.
Can anyone have the same problem?
/mnt/puppeter/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:230
                error: new Errors_js_1.ProtocolError(),
                       ^

ProtocolError: Protocol error (Network.setUserAgentOverride): Target closed.
    at /mnt/puppeter/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:230:24
    at new Promise (<anonymous>)
    at CDPSession.send (/mnt/puppeter/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:226:16)
    at next (/mnt/puppeter/node_modules/puppeteer-extra-plugin-stealth/evasions/sourceurl/index.js:32:41)
    at CDPSession.send (/mnt/puppeter/node_modules/puppeteer-extra-plugin-stealth/evasions/sourceurl/index.js:57:18)
    at Plugin.onPageCreated (/mnt/puppeter/node_modules/puppeteer-extra-plugin-stealth/evasions/user-agent-override/index.js:180:18)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Plugin._onTargetCreated (/mnt/puppeter/node_modules/puppeteer-extra-plugin/dist/index.cjs.js:496:21) {
  originalMessage: undefined
}

Node.js v17.2.0

Can i retry the cluster queue on this error?

Hard to tell what the problem is without seeing any code.


Potential bug for SearchSpace length

https://github.com/apache/tvm/blob/08a69d4f92742c8c526d6a7c2a5805d00f5dc725/python/tvm/autotvm/task/space.py#L839
here, when _length is None, then initiate it to current search space length, it's almost good most of times, however if user/developer get len of search space when space is empty, then a unreasonable _length 1 will be produced( the result of empty list for np.prod is 1), so it's better for adding an extra condition such as " len(self.space_map.values()) != 0" . :)

Thanks for the proposal. Could you send a PR?

@comaniac No problem!

Closed via #7175


Nuxt Bridge TailwindCSS Issue

Also experiencing issues with this. What causes the issue for me is that the configuration syntax seems to have changed. You have to change this:
// nuxt2, no bridge:
export default {
  build: {
    postcss: {
      plugins: {
        tailwindcss: {},
        autoprefixer: {}
      }
    }
  },
}

to this:
// nuxt2, with bridge:
export default defineNuxtConfig({
  build: {
    postcss: {
      postcssOptions: {
        plugins: {
          tailwindcss: {},
          autoprefixer: {},
        },
      }
    },
  },
})

That's fine so far. Now the infinite refresh loop is gone. Issue now that postcss seems not to work quite right anymore. It doesn't pick up the .css file and therefore tailwind just doesn't work.
My question to the authors of nuxtjs: Where would the best place be to create issues for this? (I am not using nuxt-tailwind, so not there).

here in nuxt/framework?
in @nuxt/postcss8
or at tailwind?

Originally posted by @madebyfabian in https://github.com/nuxt/bridge/issues/29#issuecomment-1097947259
Has this issue been fixed yet or does it work on the beta for version 3?

Would you mind showing us your tailwind config?

`const colors = require('tailwindcss/colors')
module.exports = {
content: [
"./components//*.js",
"./components//.vue",
"./components/**/.ts",
"./layouts//*.vue",
"./pages//.vue",
"./plugins/**/.js",
"./plugins/**/*.ts",
"./nuxt.config.js",
"./nuxt.config.ts",
],
theme: {
extend: {
colors: {
primary: colors.blue,
secondary: colors.cyan,
plain: colors.gray
}
}
},
variants: {
extend: {
scale: ['active'],
textColor: ['active']
}
}
}`
Although I would try to fix the issue, instead I am in the process of migrating directly from Nuxt 2 to Nuxt 3 without using Nuxt bridge between them. I have made previous projects starting with Nuxt 3 where Tailwind worked so migrating directly to version 3 would probably work fine.


HIVE-26121: Hive transaction rollback should be thread-safe

What changes were proposed in this pull request?

Why are the changes needed?

Proper ACID rollback handling in case of operation interruption
Does this PR introduce any user-facing change?

No
How was this patch tested?

Itest

I have missed this before, but do we really need to synchronize DriverTxnHandler.endTransactionAndCleanup and DbTxnManager.java.stopHeartbeat too?
Otherwise LGTM

endTransactionAndCleanup

yes, we definitely need to synchronize DriverTxnHandler.endTransactionAndCleanup.  As for  DbTxnManager.java.stopHeartbeat it was already kinda synchronized and it's possible that concurrent execution might still occur when shutdownhook is invoked.


podman-pilot: no visible error message without terminal assignment

Description
If a container registration does not use -t messages including error messages are not transported back to the caller.
Debugging information on failed state is therefore not easy.
Idea
If PILOT_DEBUG=1 is set add -t to the call
I'm uncertain for any after effects of this implicit setting, thus not sure if it's a good idea

This was a false investigation. Problem was if the entry point is systemd and the actual application to run consists out of one or more systemd services, the output of those are logged to the systemd journal and not to the stdout/stderr channels by default. This is imho out of scope for flakes and needs to be addressed in the design of "how to call what inside of the container".


Upload / connect split

In this PR we split the Connect Dataset tab that was in Add dataset/Layer modal window into Upload and Connect tabs.
Related issue
https://github.com/CartoDB/cartodb/issues/15207
Screenshots

Acceptance
General use case

[ ] Access the modal from New dataset button in Dashboard

[ ] Upload a valid file. A new dataset should be created. No new map.
[ ] Test URL upload. A new dataset should be created. No new map.
(http://data.insideairbnb.com/spain/catalonia/barcelona/2019-09-17/data/listings.csv.gz)
[ ] CTA in Upload tab should be Upload dataset
[ ] CTA in Connect tab should be Connect dataset

[ ] Access the modal from New map button in Dashboard

[ ] Upload a valid file. A new dataset and a new map should be created.
[ ] Test URL upload. A new dataset and a new map should be created. (http://data.insideairbnb.com/spain/catalonia/barcelona/2019-09-17/data/listings.csv.gz)
[ ] CTA in Upload tab should be Upload dataset
[ ] CTA in Connect tab should be Connect dataset

[ ] Access the modal from Add new layer button in a Builder map

[ ] Upload a valid file. A new layer (new dataset) in our current map should be created. The dataaset should be in Datasets list
[ ] Test URL upload.  A new layer (new dataset) in our current map should be created. The dataaset should be in Datasets list (http://data.insideairbnb.com/spain/catalonia/barcelona/2019-09-17/data/listings.csv.gz)
[ ] CTA in Upload tab should be Add layer
[ ] CTA in Connect tab should be Add layer

No dataset quota use case

[ ] Change user’s limits to current dataset usage.
[ ] Access the modal from New dataset button in Dashboard

[ ] Upload and Import tabs should be disabled.

[ ] Access the modal from New map button in Dashboard

[ ] Upload and Import tabs should be disabled.

[ ] Access the modal from Add new layer button in a Builder map

[ ] Upload and Import tabs should be disabled.

Retest this please

Acceptance done following the steps mentioned in the description -> 🆗 👌


tigervnc: unbreak the build (and add tests)

Motivation for this change
Right now, the vncserver of tigervnc doesn't start because xauth is missing from $PATH. This pull request fixes this and also adds a basic test, both for tigervnc and for tightvnc.
Note that tightvnc in principle has the same xauth problem, but there it has already been solved by patching the start script (by replacing xauth with ${xauth}). Here we simply add it to the path with makeBinPath. I don't know which procedure is best practice.
I was not able to actually run the test I added since the computers I have access to lack kvm support.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test vnc
(I don't actually know whether I have permissions to ask our bot for this favor, but thought that I might just try it.)

@GrahamcOfBorg test vnc

@GrahamcOfBorg test vnc

I cherry-picked the vncserver fix itself into 4fb8fab4d6949fd1f499d0e5a312f8827dadda98
and backported it to 20.03 in 0321ceaf44fb4cee884f99a0f514991b363db5aa

@iblech AFAIU @Mic92 merged the fix, meaning that this PR only adds tests now. Are you still interested in rebasing on a recent master and pushing this to completion?
On that topic @Mic92, I can't see in the comments any reason for you not to merge the tests too. Or maybe you just didn't review them?

Closing due to inactivity from author.


Verify if GhostPad should be sealed

Verify if GhostPad class should be "sealed" / "final" as the constructors contain code which must be executed and a derived class uses the generic constructor which does not execute the custom code.
It does not set the "final" gir attribute (#532). Is this an upstream bug?
Our Code
Upstream code
Explanation from ebassi: https://www.bassi.io/articles/2021/07/27/final-types/
@firox263 As you did add the comment regarding the ghost pad constructor can you confirm this again as the code is pretty old? And if so: should the class be marked as final upstream or do we add the sealed attribute ourself or do we ignore this?

This is not so relevant now that we're generating "factory"-style constructors. Users who wish to use the GStreamer API directly will just use the constructor method. Subclasses would need to reimplement that logic anyway, so I think it's fine to leave it as-is.
Also, GstGhostPad is (afaik) meant to be derivable, so making it sealed would be an error.


Add IE stylesheets

Using conditional comments and respond-min mixin
In the head of the document
<link rel="stylesheet" href="stylesheets/style.css" />
<!--[if (lt IE 9)&(!IEMobile)]><link rel="stylesheet" href="stylesheets/ie.css" /> <![endif]-->

In ie.scss
$old-ie: true;
$fix-mqs: 65em;

@import "style";

in _mixins.scss
// https://github.com/jakearchibald/sass-ie/

$fix-mqs: false !default;

@mixin respond-min($width) {
	@if $fix-mqs {
		@if $fix-mqs >= $width {
			@content;
		}
	}
	@else {
		@media screen and (min-width: $width) {
			@content;
		}
	}
}

Given that versions of IE < 9 worldwide and locally are into the single digits (around 2% as of July 2015), and that other non-MQ-supporting browsers locally are mobile (and get served our base css) this can be closed.


scaling the model will continuously grow the model, incomplete model display, etc;Mobile system incompatibility

If the setting of model color is called, there will be problems such as scaling the model will continuously grow the model, incomplete model display, etc
    updateRepProperty(sourceId, fieldName, ...args) {
      // console.log('updateRepProperty', sourceId);
      const methodName = `set${macro.capitalize(fieldName)}`;
      const source = this.$proxyManager.getProxyById(sourceId);
      const myRepresentations = this.$proxyManager
        .getRepresentations()
        .filter((r) => r.getInput() === source);
      for (let i = 0; i < myRepresentations.length; i++) {
        // console.log(myRepresentations[i]);
        if (myRepresentations[i][methodName]) {
          myRepresentations[i][methodName](...args);
        }
      }
      this.$proxyManager.renderAllViews();
    },

https://github.com/user-attachments/assets/31d883a6-ae56-4c2e-89f2-8f599bd7f648


Wrong with complex if/else condition

Thanks your for very fast bug correction, but this is other error.
Below code will failed to decompile:
public class Main {
	static boolean cond1 = true;
	static boolean cond2 = false;
	static boolean backPlay = false;
	static boolean soundPlay = false;
	static boolean cond1_1 = false;
	static boolean result = false;
	static boolean vibration = false;

	public static void main(String[] args) {
		takeAction(1, 2);
	}

	public static void takeAction(int x, int y) {

		if (cond1) {

			if (cond1_1) {
				action(x);
				backPlay = true;
				soundPlay = true;
				action(y);
			} else {
				action(1);
				backPlay = false;
				soundPlay = false;
			}
		} else if (cond2) {
			action(x);
			if (vibration) {
				result = false;
			} else {
				result = true;
			}

			vibration = result;
		}
	}

	public static void action(int id) {

	}
}

This type of issue have been report many time and you fixed it, but not correct in general, just case by case.
I see many output code with "return;" nearly all of them are wrong code.
Can you take more time to study it and fix completely?

Sorry, the produce code not very nice but correct.


App Crash iOS Release Mode

Sometimes I'm getting EXC_BAD_ACCESS (code=1, address=0x0).
This only happens in release mode

run first flutter build ios --release, then archive


Incorrect code completion results from text super

Consider this similar case:
class Thing {
  String get superDuper;
}

Create an instance of thing, and type thing.super and then try to code complete.
Expected: see the superDuper option.
Actual: nothing.
See screenshots:

Hope that helps!

cc @devoncarew but I suspect this is an AS issue, but maybe an Atom issue.

If you turn the logging up to finer, you can see the data that the AS sends atom. Fair warning, it can be pretty verbose.

cc @lukechurch do you see this in WebStorm?

Duplicate of #23882


Carsh on cancel on iOS because tcs is null

Expected Behavior
don't crash on cancel
Actual Behavior
crash on cancel in tcs.SetResult(null); because tcs is null.
I have not determined the conditions that leads to this problem.
But it just happened while debugging on a xr device.
   public void DocumentPicker_WasCancelled(object sender, EventArgs args)
        {
            var tcs = Interlocked.Exchange(ref this.completionSource, null);
            tcs.SetResult(null);
        }

Specifications

NuGet package version: 2.1.41
Platform(s): iOS 13

Closing this since this project will retire, please check out Xamarin.Essentials which now has a FilePicker based on this one.


[Conda] Specify python version in build-environment

The current build-environment.yaml doesn't specify the python version for tvm-build conda environment and as a result, any python version might be selected depending on the available base conda. In my case, the base conda is 3.10 so tvm won't work given tvm is unsupported for python >= 3.9.
To validate run on tvm-build env
conda env update -f conda/build-environment.yaml

@junrushao

@driazati


Using marshmallow and automatically generate definitions

TL;DR:
Is it possible to generate a spec definition automatically from a marshmallow schema and add it to my spec defined when I do swag = Flasgger(app) ?

Currently, i'm writting a Flask app, using :

Flask-SQLAlchemy
Flask-Marshmallow
Marshmallow-SQLAlchemy
Flasgger

I have built the application the following way :
inside a server.py file:
from flask import Flask
from flasgger import Swagger
from flask_sqlalchemy import SQLAlchemy
from flask_marshmallow import Marshmallow
# [...]
app = Flask(__name__)
swag = Swagger(app)
db = SQLAlchemy(app)
ma = Marshmallow(app)
# [...]
from submodule.routes import *

and the routes (defined with the @app.route decorator, inside a submodule).
These routes extend the "db.Model" SQLAlchemy class, and in order to add definitions to my spec, I'm currently using the @Flasgger::definition decorator (and writting a YAML file in the docstring of the decorated class, re-declaring all of the properties). This means that I'm rewritting the information that already exists within the marshmallow.ModelShema object.
I'd like to know if it is possible to generate the same spec definitions, automatically from the marshmallow schemas, and add them to the spec.
(NB: The "template from apidocs" pattern described in here can't be used in my case as I define the Flasgger object (swag = Flasgger(app))  before my routes (and my schemas) are imported.

Ideally, it would be nice to have a decorator that decorates Marshmallow classes, and adds them to a spec as definitions:
something like
@swag.definition_from_marshmallow(definition_name='User', endpoint='myendpoint')
class UserSchema(ma.ModelSchema):
    class Meta:
         model = User

that would result in a User definition, that can be then referenced.

If all you're interested in is referencing the marshmallow definitions from within a SwaggerView method's docstring (as I am), this seems to do the trick:
from flasgger import Schema, SwaggerView

class UserSchema(Schema):
    ... (fields go here) ...

class UserView(SwaggerView):
    methods = ['GET']
    definitions = {'UserSchema': UserSchema}

    def get(self):
        """Get users.
        ---
        responses:
          200:
            description:
            schema:
              $ref: '#/definitions/UserSchema'
        """
        ... (rest of get method's code goes here) ...

    ... (rest of UserView class's code goes here) ...

You might be able to use this approach to get what you want. For example, you could write a SwaggerView class that does nothing but reference all of your marshmallow schemas (you'd need to reference each schema in a docstring, however).

Based on #341 and starting with release 0.9.4.dev2 (or later), you can add your marshmallow schema using the @swag_from decorator:
from flasgger import Schema
from http import HTTPStatus

class TestSchema(Schema):
    number = fields.Int()

@app.route('/test', methods=['POST'])
@swag_from({
    'responses': {
        HTTPStatus.OK.value: {
            'schema': TestSchema
        }
    }
}, validation=True)
def test():
    """
    Test POST
    ---
    """
    data = request.json
    # TODO

#341 feature works for OAS v2. For version 3+, the schema is not being rendered :(.

I can confirm on my codebase that using
@swag_from({
    'responses': {
        HTTPStatus.OK.value: {
            'schema': TestSchema
        }
    }
})

Does not render the schema if OAS3 is enabled.

Hi, I am create new PR https://github.com/flasgger/flasgger/pull/396 for easy usage Marshmallow Schema in flasgger


⚠ Bofrost.nl has degraded performance

In c8ba42c, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1335 ms

Resolved: Bofrost.nl performance has improved in 7a36451.


Support initial value for metadata

It would be interesting to be able to set a default value for metadata in the yaml file

In GitLab by @ksiero on Jun 22, 2020, 08:31
closed via commit 178e7010045a76b143de3973319439df8a0712bd

In GitLab by @ksiero on Jun 20, 2020, 12:43
mentioned in commit 178e7010045a76b143de3973319439df8a0712bd

In GitLab by @ksiero on Feb 6, 2020, 09:43
Closes with MR !221

removed milestone

In GitLab by @ksiero on Jun 19, 2020, 14:02
mentioned in commit ff6612ed07a7e4c688cfa7ccd4337d8cd289bf9d

In GitLab by @ksiero on Jun 22, 2020, 08:31
closed via merge request !240


Fix deprecated setting in demo profile

Partial backport of a massive PR in master. This fixes the deprecation
warning when installing with demo profile, and adds a regression test.
this has no impact on the generated manifests; the option does nothing.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio_release-1.7
7baa8266fa8b7e155fcddbfbe950186ca2046f46
link
/test release-notes_istio_release-1.7

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Add actual working remote user authentication

The existing teletype used GitHub as Authentication against the API-Server. As such, you had to trust that the API-Server does not do anything malicious or simply wrong. If the API-Server would ever be compromised, anyone with a valid GitHub-Account could connect to your instance.
In a pure P2P implementation this would not work at all, as there would not be a single centralized secure/secret source of truth.
This issue collects ideas on how to enable meaningful authentication to the connected parties.

One idea, could be an extension to what I suggested here:
https://github.com/pulsar-edit/pulsar/issues/536#issuecomment-1939189050
Currently, the invite link looks like this:
atom://teletype/portal/9ab41a56-11c3-4ed5-a346-c00830d67d0b
It uses a single UUID to identity the portal, which then gets resolved by the API-Server to get the actual portal name. The client then opens the portal, and can connect.
If we instead extend the invite link to look like this:
pulsar://teletype/portal/{secret}/{portal_name}
The secret would never leave the local machine. Only the user would send the invite link via their chosen communication link. Under the assumption, that said link is secure, would in turn ensure, that only the intended partner has access to the secret.
This would enable the connection partners to authenticate each other.

There is more research needed to understand how the connection really works. Clearly I do not fully understand it, as this experiment shows:
I pressed share once, this created a invite link. I copied it and pressed share once again. When I tried to connect with the 'old' link, it did not work. If it works like I described above, it should have, as the portal name/peer_id whatever you want to call it, did not change.

Should be mostly accomplished by https://github.com/teletype-diy/teletype-diy-client/commit/82ea20a76348873e5fe5c85d0ab2781bcb6b4b42 and https://github.com/teletype-diy/teletype-diy/commit/92e919f6238b6adbae1a9fd85bd86e8cb43a0099
For completeness, I should really test if the psk/secret is really not transmitted anywhere else, #4 would also ensure, that this is the case.


Logging to stderr (usage of debug)

I've started to quickly skim through the code, and looking here https://github.com/vgno/roc-web/blob/master/src/app/server/index.js#L72 I see that you use debug for logging.
By default debug uses stderr (console.error) as the output source. The message in the example above is not an error but information, so it should be outputted to stdout.
You can either fix this by following this explanation or by using a more fully featured logging library like bunyan if you're going to do a lot of logging.

Thanks a lot for the feedback. We'll be looking at this.
Do you have any experience with bunyan in isomorphic/universal apps btw? I believe it can probably be bundled for clients, but it may be a bit bloated. Experiences welcome.

I've never tried it on the frontend, but according to https://github.com/trentm/node-bunyan it supports browserify/webpack. So the question then is the code size and if there is a need for it on the frontend. Looking at https://github.com/trentm/node-bunyan/blob/master/lib/bunyan.js it doesn't seem to be much if its minified.

I think the advisable thing to do here is to use both since some of the packages that we use support debug internally. Need to look more into this, it might be the case that bunyan can manage this for us as well.
The size of bunyan should not be a problem since we only want to log in development on the client and can easily remove it from the production build.


Remove the 'm' from the readme

Removed m from the main readme. Removal from the math readme is still required


Fixed navbar overflowing to next line

Description
Closes #2452
The right navigation bar would wrap around to the next line when on mobile devices. This would prevent users from properly selected the playground to code. Fixed it by changing the navbar component to be absolute so wrapping would not occur.
Type of change

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

How to test

Go to Playground on Mobile
Reduce the screen width till right navbar wraps over

Checklist

[x] I have tested this code

Hi, thanks for working on this.
Unfortunately, I don't think this is the right approach. The navbar should not be removed from the document flow (which is what position absolute does). The z-index is also very hacky.
Ideally, the right navbar shouldn't even have language selector (see #2491). It was a hacky and rushed implementation when multiple languages were first added. The whole design needs to be re-thought out.
Would you be keen to think of this?

I think the language selection should remain on the top right corner. It allows the user to fix the programming language to be used throughout sourceacademy.org. (In the future, we might also have a natural language selection there, i.e. you can use Source Academy to study programming using Scheme in Hindi, etc.)
We all agree that the design of the language selection was done in a rush, which leads to all kinds of inconsistencies. We can't fix the design during this semester (till May 2024), so will need to live with the inconsistencies as far as the business logic is concerned.
I hope that this situation doesn't lead to a deadlock regarding this PR. As far as I can tell, the PR addresses a UI aspect of the language selection.


Bible version SelectControl doesn't have optgroups

In all other BibleGet implementations (Google Docs, Microsoft Word, OpenOffice, LibreOffice ...) the Bible version multiselect is organized by language using optgroups.
However, the select component in the WordPress Customizer did not allow for optgroups. An issue for this was raised in the WordPress Core trac: https://core.trac.wordpress.org/ticket/47692 .
And a patch was submitted on the WordPress repo :  https://github.com/WordPress/wordpress-develop/pull/430 .
Also the Gutenberg SelectControl component does not allow for optgroups. An issue for this was raised in the Gutenberg development repository: https://github.com/WordPress/gutenberg/issues/26505 .
And a provisional patch was submitted : https://github.com/JohnRDOrazio/gutenberg/pull/4/files .
However it's hard to tell if and when the submitted patches will be accepted and integrated into the codebase. Until that happens, perhaps alternative components should be created which allow for optgroups.

I found my way here from the Gutenberg repo. Just wanted to suggest that maybe instead of (or while) waiting on that improved SelectControl component from there, you could create the component you need without much code (or build steps).
I made an example of doing so that you can see here:
https://codesandbox.io/s/createelement-select-es5-ukllr?file=/src/index.js
It's written to use createElement and ES5 so you could drop the three functions Select, OptionGroup, Option into the gutenberg.js here and it should work to use like so:
createElement( Select, {
	className: 'bibleVersionSelect',
	value: attributes.VERSION,
	onChange: changeVersion,
	multiple: true,
	options: bibleVersionsSelectOptions,
})

Only I didn't put in support for the value prop so that's something you'd probably need to add but it shouldn't be too tricky. Besides that, it doesn't support the label and help props that SelectControl does but adding those or just creating them around the component as separate elements wouldn't be difficult.
And after all that I finally see:

Until that happens, perhaps alternative components should be created which allow for optgroups.

So yeah, I think that's a good idea.

Thanks so much @stokesman ! I look into this kind of implementation.

Fixed in PR #11  thanks to input from @stokesman

@stokesman the component seems to be working fine, however I notice that when I refresh the page that I'm editing, if it has a block with this custom OptGroupControl, I see this error popup in the console after the page is refreshed:

react_devtools_backend.js:2430 Warning: A component is changing an uncontrolled input of type undefined to be controlled. Input elements should not switch from uncontrolled to controlled (or vice versa). Decide between using a controlled or uncontrolled input element for the lifetime of the component. More info: https://fb.me/react-controlled-components

Do you know what could be causing this?
The code, based mostly on your example, is here: https://github.com/BibleGet-I-O/bibleget-wordpress/blob/da254c75347da1ad8c53ef542375e9f4e717ec79/js/gutenberg.js#L48-L102

@JohnRDOrazio, glad to see you got this going! I like that you thought to wrap it with BaseControl 👍
I just cloned this repo and installed it in my local WP 5.6.2 and have not been able to reproduce the issue. I tried with the built-in version of Gutenberg and a recent build of the plugin.  I know there was something in something in recent versions of Gutenberg that caused the same type of error upon page load but I can't remember what it was coming from specifically.


Correct workflow for running compass

Hi there, how should I be running compass if I have 2 phenotype?
Should I be running compass on each phenotype separately., then merge the results to compare? Another question is what if I have many different celltypes. Should I be running compass on each cluster separately, since I see that there is knn smoothing over the output matrix. Thanks!

For two phenotypes, the method we used in the paper was running Compass on both phenotypes combined with the knn smoothing, so gene expressions could potentially have been averaged across the phenotypes though in this case the phenotypes nearest neighbors tended to be of the same phenotype anyways. The averaging across k nearest neighbors helps mitigate dropout, which can results in lower predicted activity across all cells, but it could reduce the effect sizes across cell types.
I'll let Allon comment more on the tradeoffs of the smoothing as well.

Hi @schel337 ,
thanks for the reply.

By default Compass does not smooth across samples though, in which case you can run the samples together or separately and get the same results.

I thought the default settings is 30 neighbor smoothing with 0.25 weight sharing though

The weight sharing parameter (--lambda which may not be the best name) is set to 0 by default, though if you want to enable it we suggest 0.25 or 0.2 as values.

Yes, I don't have much to add beyond Brandon's response. If you set lambda=0 (which is the default), namely no smoothing, then running them separately or together will produce the same results.
In the Compass paper we used information sharing and ran the two phenotypes together because we were interested in the overlap and continuous gradient that existed between two seemingly disparate cell types. In other scenarios, where you have distinct phenotype and you're interested only in comparing between them, it may make more sense to run them separately, so that the kNN is computed separately on each phenotype and cells of phenotype A will not affect phenotype B.

alright thanks! I think I skim through the docs and saw the 0,25 and mistakenly thought that it is 0.25 by default.


Ecommerce SmartContract

Hello, I want to add a Smart-Contract for an Ecommerce Site. That will have buy, delivery, register-Products function that will ensure that only after the delivery of the Product is successful, money is deducted from the buyer account and sent to seller account.

Yup , now u can start working on this . @Satyam5665

Are u still working on this issue ?

Hello, @Vikash-8090-Yadav as I could see there is no response from the assignee for a long time. Can I work on this issue?

Don't forget to introduce yourself in the community !

Yeah I have already introduced myself in the introduction channel in SWOC Discord server ! If there is any dedicated channel for Solidity-Pathsala can you please join me in.
Thanks

@Debopom-Banerjee introduce yourself  here !
https://github.com/Vikash-8090-Yadav/Solidity-Pathshala/discussions/175

And please give me your discord id also

Hello Myself Debopom Banerjee, CSE undergrad from West Bengal, India. I am a full stack blockchain developer and also a MERN stack Developer. I am an Open-source enthusiast and also learning DSA in recent times.
Discord id : Debopom Banerjee#4862

Click in this ->  https://github.com/Vikash-8090-Yadav/Solidity-Pathshala/discussions/175

ohh I am sorry didn't noticed that


Lots of "attribute names must match the format: lowercase (E002)" warnings

Using the Brackets extension, updated from htmllint  0.2.1  →  0.2.7 and instead of 2 warnings I get 152 warnings, most being (E002) warnings.
For example this produces an warning:
<a class="dimmable image" data-modal="a">

The attribute name here (data-modal) does not match the lowercase style, which requires that the name be composed entirely of lowercase letters and digits. You should set the option attr-name-style to "dash" instead.
Documentation was unclear in defining the lowercase format; I've fixed it.
I'm not sure why you didn't get these warnings before updating, as neither the attr-name-style nor the default setting for the corresponding option have changed. In any case, the current behavior is reasonable (although perhaps dash would be a better default option).

The reason why I got 152 warnings instead of 2 warnings was that I had edited the options_manifest.json to change the default settings in 0.2.1, and when updating to 0.2.7, the modified settings were lost. This issue can be closed.


Cannot detect env when I run from android studio directly

I am trying to set up environment and only have buildtype like this.
  buildTypes {
    debug {
      debuggable true
      ext.enableCrashlytics = false
    }
    release {
      minifyEnabled true
      proguardFiles getDefaultProguardFile('proguard-android.txt'), 'proguard-rules.pro'
      consumerProguardFiles 'proguard-rules.pro'
      signingConfig signingConfigs.release
    }
  }

As a result, I write like this.
project.ext.envConfigFiles = [
        debug: ".env.dev",
        release: ".env.prod",
]

apply from: project(':react-native-config').projectDir.getPath() + "/dotenv.gradle"

I have already created .env.dev and .env.prod in root project directory.
ENV=dev
For testing, in my react native, I wrote like this as well.
<Button >{"Test " + Config.ENV}</Button>

Problem is it show as 'undefinied' and cannot detect Config.ENV. For iOS, it is okay. It means my gradle got issue I guess or react native version.
I am using expo eject as well (ejected expo project)
Expo version: 35.0.0
React Native: 0.59.8
How shall I do?
For reference, this is my dotenv.gradle. I haven't changed anything.
def loadDotEnv(flavor = getCurrentFlavor()) {
    def envFile = ".env"

    if (System.env['ENVFILE']) {
        envFile = System.env['ENVFILE']
    } else if (System.getProperty('ENVFILE')) {
        envFile = System.getProperty('ENVFILE')
    } else if (project.hasProperty("envConfigFiles")) {
        // use startsWith because sometimes the task is "generateDebugSources", so we want to match "debug"
        project.ext.envConfigFiles.any { pair ->
            if (flavor.startsWith(pair.key)) {
                envFile = pair.value
                return true
            }
        }
    } else if (project.hasProperty("defaultEnvFile")) {
        envFile = project.defaultEnvFile
    }

    def env = [:]
    println("Reading env from: $envFile")

    File f = new File("$project.rootDir/../$envFile");
    if (!f.exists()) {
        f = new File("$envFile");
    }

    if (f.exists()) {
        f.eachLine { line ->
            def matcher = (line =~ /^\s*(?:export\s+|)([\w\d\.\-_]+)\s*=\s*['"]?(.*?)?['"]?\s*$/)
            if (matcher.getCount() == 1 && matcher[0].size() == 3) {
                env.put(matcher[0][1], matcher[0][2].replace('"', '\\"'))
            }
        }
    } else {
        println("**************************")
        println("*** Missing .env file ****")
        println("**************************")
    }

    project.ext.set("env", env)
}

I'm also facing this issue but iOS work well.

I'm also facing this issue but iOS work well.

1

I'm also facing this issue but iOS work well.

1

+1

+1

Adding proguard rule solved for me
https://github.com/luggit/react-native-config#problems-with-proguard

I think I understand this issue now as it was affecting me with appcenter, I think perhaps the problem is that the folder this is getting executed from is different to what react-native-config expects.
To resolve this I set
buildscript {
    ext {
        defaultEnvFile = file('.env').absolutePath.replace("/android", "")

I think I understand this issue now as it was affecting me with appcenter, I think perhaps the problem is that the folder this is getting executed from is different to what react-native-config expects.
To resolve this I set
buildscript {
    ext {
        defaultEnvFile = file('.env').absolutePath.replace("/android", "")

I guess it should be documented.
 All of it because a line of conf not documented.
This lib always expect to use a .env as default. Even if we use something like:
project.ext.envConfigFiles = [   debug: ".env.dev",   release: ".env" ]
Is this the right way @erickreutz @jordanbrown ? We need to set defaultEnvFile to use something like .env.dev as custom env?


Add logic adapter that returns the current time when asked

Since we are using logic adapters for many of these additional functions, should we remove plugin adapters? Or are there still functions that we should implement as plugins?
I would believe that functions that are deeply integrated into the chatbot, such as time, date, location, etc, should be added as logic adapters while functions that are "extra", or not core to chatterbot be added as plugins (which are essentially logic adapters but more complex). What do you think @gunthercox ?

@DarkmatterVale Logic adapters and plugin adapters both accomplish the same task. I think it would be beneficial to make a move to migrate the existing selection of plugin adapters to become logic adapters (this should not require a huge amount of modification). Apart from sharing a more unified code base, the other benefit is that there will be no restrictions to define what parts of the program are "deeply integrated" and what parts are not.


list index out of range errors

:Grepper
ag --vimgrep> lass

and then I get
Error detected while processing function <SNR>83_on_stdout_nvim:
line    5:
E684: list index out of range: -1

After pressing q, the results show up.
I don't have any options enabled in vim-grepper.

Oops. Should be fixed now!


Has this repository compiled on Mac?

Has this repository compiled on Mac? I'm using Mac to compile this repository and I got following error message:clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [../lib/libinekf.dylib] Error 1
make[1]: *** [CMakeFiles/inekf.dir/all] Error 2
make: *** [all] Error 2
I have installed the requirement package already.
Thanks in advance!

I have not tried compiling on Mac. This was only tested on Ubuntu 16.04.

Thank you so much. It runs perfectly on my Ubuntu 18.


fixed

"fixed" what?  PR's should have some context in the title and description so the rest of us know what's going on.

Sorry, the branch has the more descriptive title. I fixed one of the grid queries in the Kubelet workbook


undefined: gogs.Payloader

I upgraded my gogits and after deleting the gogs folder and:
git clone --depth=1 -b develop https://github.com/gogits/gogs.git
cd gogs
go get -u -tags "sqlite" ./...
go build -tags "sqlite"

there was the following error messange:
# github.com/gogits/gogs/models
models/webhook.go:288: undefined: gogs.Payloader

You have the old version of github.com/gogits/go-gogs-client

Thanks. I updatet go-gogs-client and now gogits 0.6.7 beta works.

@peppage thanks for your answer!
Actually go get -u ./... should update your reps to latest version if they were downloaded by go get command. Not sure it's not working sometimes, but glad you solved the problem, and I'm changing APIs a lot these days so you would expect similar errors again. :)

I downloaded the latest version via git clone, not go get

@focmb Yes, but i see in your comment you use go get -u ./... for dependencies.


Information

What exactly is data loss?

It is the name of the loss function (for example, Dice loss).

@Zach-ER could you please elaborate with the mathematical form of this loss function?


Fix cgroup_path builtin

bpftrace -e '{ print(cgroup_path(cgroup)); exit(); }'
The addrspace is not set and bpf code would resolve it  to
bpf_probe_read, which is invalid on s390.
As cgroup_path is already allocated on the bpf stack, a normal copy
should be enough. No need to set the addrspace or use the
CreateProbeRead variant.

Nice catch, it's a structure built on the stack in codegen, so there's no reason to proberead here :) I probably just forgot to add it there, since my original implementation was different.


Attempting to upgrade from Aggregate 1.4.2 to 1.4.4 and I am getting errors "Form definition incomplete or missing"

Originally reported on Google Code with ID 1093

What steps will reproduce the problem?
1.Using tomcat/mysql setup with 1.4.2
2.Swap out the old 1.4.2 war file with new generated 1.4.4 war file

What is the expected output? What do you see instead?
No old data is viewable and the main submission page is not viewable for old surveys.
New ones subsequent to the upgrade appear fine. I keep getting the error  "Form definition
incomplete or missing". From the mysql query log it appears aggregate is trying to
recreate the core data tables that already exist and this causes an error. Why is it
trying to recreate the data tables?

I noted in the upgrade instructions that you should "delete all ODK Tables data tables".
If this means I need to clear out all the old data -- that is not a tenable solution
because the data is used. I looked through the schemas for the core tables and did
not see any differences out side some new tables which get created. How can I maintain
my old data after updating to 1.4.4? If need be I can write transformation scripts
for the old data so that it can be transferred -- bu I am hoping I do not need to do
that.

Thank you,
Daniel

Reported by dalasaga14 on 2014-12-02 00:23:25

Please ask any questions such as this in the Google group.
Thanks,
Dev Team


Fix bug report issue

A working demo - https://github.com/Saransh-cpp/.github-FluxML/issues/new/choose
See https://github.com/FluxML/.github/pull/4#issuecomment-1345499741
PR Checklist

[ ] Tests are added
[ ] Documentation, if applicable

nice, thanks


Listen for messages from specific group or user?

How would I go about creating an automation that would be triggered by a specific message to a specific group?
I've been able to trigger with a specific message, but anyone could send that message.
Thanks!

Why not trigger on the new message event and use a template condition to test if it is the correct group?

Had a play around with it and realised I could utilise more of the data from the new_whatsapp_message event.
Came up with an automation that checks for a specific message of 'Unlock' and in a specific group using 'remoteJid'
It even announces who has opened the gate by their whatsapp username and sends a reply acknowledging the request.
alias: Whatsapp Unlock Gate
description: ""
trigger:

platform: event
event_type: new_whatsapp_message
condition:
condition: template
value_template: "{{ trigger.event.data.message.conversation == 'Unlock' }}"
condition: template
value_template: "{{ trigger.event.data.key.remoteJid ==<EMAIL_ADDRESS>}}"
action:
type: turn_on
device_id: a8066d26bc0d43c2989e967cae*****
entity_id: switch.gate_beep
domain: switch
data:
message: Back Gate Unlocked by {{trigger.event.data.pushName}}
entity_id: media_player.google_assistants
service: tts.google_say
service: whatsapp.send_message
data:
clientId: default
to<EMAIL_ADDRESS>body:
text: Gate Released
type: turn_on
device_id: a8066d26bc0d43c2989e967cae******
entity_id: switch.gate_release
domain: switch
mode: single

Thank you for the suggestion!


jump-to-char: Case sensitivity toggle

yazi --debug output
Yazi    Version: 0.3.3 (Nixpkgs 2024-09-04)    Debug  : false    OS     : linux-x86_64 (unix)Ya    Version: 0.3.3 (Nixpkgs 2024-09-04)Emulator    Emulator.via_env: ("xterm-kitty", "")    Emulator.via_csi: Ok(Kitty)    Emulator.detect : KittyAdapter    Adapter.matches: KittyDesktop    XDG_SESSION_TYPE           : Some("wayland")    WAYLAND_DISPLAY            : Some("wayland-1")    DISPLAY                    : Some(":0")    SWAYSOCK                   : None    HYPRLAND_INSTANCE_SIGNATURE: Some("a425fbebe4cf4238e48a42f724ef2208959d66cf_1732546057_78323281")    WAYFIRE_SOCKET             : NoneSSH    shared.in_ssh_connection: falseWSL    WSL: falseVariables    SHELL              : Some("/run/current-system/sw/bin/zsh")    EDITOR             : Some("nvim")    VISUAL             : Some("nvim")    YAZI_FILE_ONE      : None    YAZI_CONFIG_HOME   : NoneText Opener    default: Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })    block  : Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })Multiplexers    TMUX               : false    tmux version       : tmux 3.5a    ZELLIJ_SESSION_NAME: None    Zellij version     : No such file or directory (os error 2)Dependencies    file             : 5.45    ueberzugpp       : 2.9.6    ffmpegthumbnailer: 2.2.2    magick           : 7.1.1-40    fzf              : 0.56.2    fd               : 10.2.0    rg               : 14.1.1    chafa            : 1.14.5    zoxide           : 0.9.6    7z               : No such file or directory (os error 2)    7zz              : 24.08    jq               : 1.7.1--------------------------------------------------When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.You can find it in the "/home/repparw/.local/state/yazi" directory.

Please describe the problem you're trying to solve
Jump not grabbing uppercase files/directories on lowercase letter
Would you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
Add a case sensitivity toggle/option, maybe make it ignore by default.
so you can use fd to get to Downloads/Documents, not only fD.
(Probably ignore on lowercase and only upper on uppercase?, something similar to a search in nvim)
Additional context
No response
Validations

[X] I have searched the existing issues/discussions

Not for f, I don't think so. but this is more akin to a search since this implementation doesn't jump to next char only to firsts.
You could bypass jumping to Documents in nvim by jumping to the o if it's easier than jumping to D, or if there are other D before.
You also don't usually mix and match many uppercase words in nvim, while some directories are created in uppercase (Unix default I think? see most dirs inside home)


Supporting renderers that use comms

I have a lab extension that adds a renderer that uses comms. It displays interactive vega charts that call back to the backend to recompile expressions as the user interacts with the data (https://github.com/Quansight/ibis-vega-transform).
Currently it uses the notebook tracker to get the kernel for the current kernel and register a comm handler
https://github.com/Quansight/ibis-vega-transform/blob/c59d36ab575e0f0eb5d6fbee5e2aa8b420cde6d9/src/index.ts#L28-L40
This was inspired by @gnestor's PR to add even handling support to the vdom extension which was in turn inspired by @jasongrout
What would be the best way to support this in phoila? One option would be to add the notebook tracker extension and keep the extensions the same. Another option would be for JupyterLab to expose a higher level way of creating a comm in a mime render extension.

This is a good question! Currently, ipywidgets gets special cased, but we should ideally find a solution that works for all Comm-based mime-renderers with all contexts that have a kernel associated with it.
While this is being solved, a workaround is to require the TVoilaTracker tracker, and use this to get the kernel:
tracker: TVoilaTracker;

tracker.widgetAdded.connect((tracker_, widget) => {
  (await widget.content.session.connected).kernel
});

Thank you vidar for the workaround, that's great! So I guess for an extension that supports both lab and phoila it could require the voila tracker notebook tracker and make them both optional

So I guess for an extension that supports both lab and phoila it could require the voila tracker notebook tracker and make them both optional

Yup, but I still think we should try to come up with a more generic solution to this. It could help clean up some of the logic in the ipywidgets lab manager as well! cc @jasongrout

@vidartf unfortunately I don't see the content attribute on the widget, either in TS or at runtime:
    voila.widgetAdded.connect((_, widget) => {
      console.log(widget);
    });

Ah, it might not have made it to a released version yet. A new version is right around the corner though.

Thank you Vidar. Let me know if I can help with anything.


listen for window ready or displayed.

Hi, I am writing a PowerShell UI and want to run some powershell comes as soon as the Window is "ready", ie, before the user starts interacting with the console, however i don't want the background task to delay the window from running until its finished.
Im doing something like this, where i am trying to listen for a Ready event then run this code.
...
$Top.add_Ready{
    $UIDs = Get-ADUser -Properties uidnumber,gidnumber -LDAPFilter '(|(uidnumber=*)(gidnumber=*))'
    $UIDs.count
}

$Top = [Application]::Top
$Top.Add($Window)
[Application]::Run()
[Application]::Shutdown()

Is there a reason not to just start a normal Thread right before calling Run()?

That sounds logical, can you point me to an example of what you mean?

You can use the Application.Top.Ready action which is fired on the first instance, before interacting with the console.
.```
..
$Top = [Application]::Top
$Top.Ready += {
$UIDs = Get-ADUser -Properties uidnumber,gidnumber -LDAPFilter '(|(uidnumber=)(gidnumber=))'
$UIDs.count
}
$Top.Add($Window)
[Application]::Run()
[Application]::Shutdown()

Another option is the Loaded event which is fired before the Ready event.

however i don't want the background task to delay the window from running until its finished.

I'm not used to the syntax of powershell but it looks like you can create a new thread/task using these commands (see below).  That would prevent the callback from blocking user input while it completes.
https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/start-job?view=powershell-7.1
https://docs.microsoft.com/en-us/powershell/module/threadjob/start-threadjob?view=powershell-7.1


Private bot backpack throws useless exception

From #701
At the very least, we should be explicitly throwing a meaningful exception.

Indeed, that was my thought. I can look at this tomorrow if you want me to
take care of it?
On 28 Jan 2015 19:44, "BlueRaja"<EMAIL_ADDRESS>wrote:

From #701 https://github.com/Jessecar96/SteamBot/issues/701
At the very least, we should be explicitly throwing a meaningful exception.
—
Reply to this email directly or view it on GitHub
https://github.com/Jessecar96/SteamBot/issues/704.

That'd be awesome - thanks!

#706 #718


[Bug]Do Table Cells support adding images?

I just implemented image frames as a 2x1 table with the top containing an image and the bottom containing some text as a caption. The method works on a blank document for appending an image to it and does not work for this purpose. I can supply examples for what I mean.
Code can be found at https://sourceforge.net/p/f-docx/code/ci/master/tree/Program.fs
The test script throws a null reference exception in the final call to save. https://sourceforge.net/p/f-docx/code/ci/master/tree/bin/Debug/net7.0/publish/test.fsx
Each file is short, if this feature is currently supported then do you know what I need to be doing instead?
Thanks

Seen above the documentation implies the result is going to be a paragraph. Seen below there is a failure
since the parent is null but the method used to create the parent does not return nullable.

The code in question is below.
((addParagraph "").AddImage (image.path, image.width, image.height, wrap)).AddParagraphAfterSelf() |> ignore
I got this from adding the call to addparagraphafterself to try checking if it was due to the requirement to end a cell with a paragraph. This tells me that AddImage is returning null, I can't say more than this as I don't know the codebase.

    public WordParagraph AddImage(string filePathImage, double? width = null, double? height = null, WrapTextImage wrapImageText = WrapTextImage.InLineWithText, string description = "") {
        var wordImage = new WordImage(_document, this, filePathImage, width, height, wrapImageText, description);
        // Why was this change made?
        //var wordImage = new WordImage(_document, filePathImage, width, height);
        var paragraph = new WordParagraph(_document);
        VerifyRun();
        _run.Append(wordImage._Image);
        return paragraph;

I've looked around the code a little, I see that the _document attributes seem to form a tree. Do you think the issue might be that somehow _document of something is set to null when I add the image in the table? The same code for adding an image works just fine outside in a normal paragraph, and it is unrelated to any requirement for ending a cell in a paragraph (I tried it).

@PrzemyslawKlys
I believe that I found the problem.
WordTableCell creates a brand new WordParagraph via call to the default constructor with no arguments using new.
This causes the paragraph not to have it's own reference to _document, the field will be left null, and so when addImage
is called on this paragraph, there is an error because _wordimage is passed the null reference to the document from
the paragraph so the image resource cannot be added to the root of the document.
WordParagraph public methods
WordParagraph constructor definition
WordTableCell
Should I submit a PR adding an optional argument for the _document reference to the addImage method on WordParagraph?

The alternative short term fix for adding an image into a table looks like the user needs to explicitly create a WordParagraph from the document or directly and then add this paragraph into the table cell.

Just looked over the existing PR for improving the table class and this fix is not yet present there, maybe better to fix there idk.

You can always try fixing it, I will need to look at code to be able to tell whether that's the simplest way to fix it. for me if you have optional parameter that requires you to "fix" this or it throws an error - it's not good design. I would prefer something automatic, so when user does AddImage it just works without thinking on "paragraphs", but maybe there's no other way. I'll take a look when I get some time, but if you want you can always play with it.


thumbnail should match editor.minimap.enabled

Issue Type: Settings Search Issue
in settings search i searched for "thumbnail", which is a term used for the scrollbar/minimap.  editor.minimap.renderCharacters comes up first, but editor.minimap.enabled  comes up in the 16th position. i would expect them to be closer together.
VS Code version: Code - Insiders 1.21.0-insider (3fec61014742dec894b3a31cdf8556bf533cb0e1, 2018-02-28T05:17:16.839Z)
OS version: Windows_NT x64 10.0.16299
Results

Setting
Extension
Score

editor.minimap.renderCharacters
core
2.003

editor.overviewRulerBorder
core
1.512

editor.overviewRulerLanes
core
1.475

markdown.preview.scrollPreviewWithEditor
core
1.447

markdown.preview.fontSize
core
1.435

markdown.preview.scrollEditorWithPreview
core
1.322

markdown.preview.lineHeight
core
1.305

editor.minimap.side
core
1.280

markdown.preview.linkify
core
1.268

markdown.preview.doubleClickToSwitchToEditor
core
1.267

markdown.preview.fontFamily
core
1.264

markdown.preview.breaks
core
1.200

editor.minimap.showSlider
core
1.175

markdown.preview.markEditorSelection
core
1.144

editor.minimap.maxColumn
core
1.089

editor.minimap.enabled
core
1.083

workbench.editor.enablePreview
core
0.889

workbench.editor.enablePreviewFromQuickOpen
core
0.826

editor.hideCursorInOverviewRuler
core
0.331

markdown.previewFrontMatter
core
0.239

Query: thumbnail
Literal matches: 0

Filed upstream.
@chrisdias Do you think .enabled settings should generally rank higher than others? Could add a general boost for settings that end in .enabled.l

today nothing comes up when searching for thumbnail.

Not sure we even do the Bing search anymore, and I assume something like this (very fuzzy search) would be subsumed by Copilot.
So, this issue can probably be closed.


Exception when closing the terminal

Repro:

Open a terminal
Hit the trash icon

Upstream: https://github.com/xtermjs/xterm.js/issues/2933


Extend existing tip about PointerEvents,  use disabled flag on button

Thanks for the suggestion. I don't think there's any need to extend it to the pseudo-class because it's doing what the CSS class example is already doing, but what I may end up doing it updating the ruleset to use pseudo-classes instead of a generic classs.


fix(error): adding type to error in string validator in introspect

Closes https://github.com/accordproject/concerto/issues/772

Changes

Adds type to validator error in introspect validator.

created a new custom error class StringValidatorException in introspect which can take errorType as well as errorMessage which helps to determine what kind of error is getting thrown from the validator itself.

Author Checklist

[x] Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
[ ] Vital features and changes captured in unit and/or integration tests
[x] Commits messages follow AP format
[ ] Extend the documentation, if necessary
[x] Merging to main from fork:branchname

Should we have a new folder for error classes and not keep it in introspect?

Hi @dselman , as you requested I have changed the structure by doing the following changes to the structure:

I no longer am using the StringValidatorException class instead I have added the errorType as an optional param in BaseException and using that.
added an errorCodes.js file where I am storing the error type constants and using them directly , these are exposed so the client will also be able to use these , as discussed in the PR .

requesting your approval on this PR


added bootstrap and re-formatted calculator pages

renamed dilution calculator html file
rewrote html for the calculator main pages
updated form labels
redid the header

Hopefully resolved the merge conflicts correctly:

added the drop-down menu to be the correct version
approved most if not all of the front end changes
resolved conflict codes in the view.py file
added a few changes from the cutting reaction page

We will keep this page open for now even though I will go ahead and merge the pull request in case any future changes happen.


kubernetes-helmPlugins.helm-s3: 0.16.0 -> 0.16.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/hypnoglow/helm-s3/releases.
meta.description for kubernetes-helmPlugins.helm-s3 is: Helm plugin that allows to set up a chart repository using AWS S3
meta.homepage for kubernetes-helmPlugins.helm-s3 is: https://github.com/hypnoglow/helm-s3
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.16.1 with grep in /nix/store/190sgdplcbxmb6icw4n9nb851ldx4apb-helm-s3-0.16.1
found 0.16.1 in filename of file in /nix/store/190sgdplcbxmb6icw4n9nb851ldx4apb-helm-s3-0.16.1

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

kubernetes-helmPlugins.helm-s3

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/190sgdplcbxmb6icw4n9nb851ldx4apb-helm-s3-0.16.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/45c32c574c90853fabf08e14ec69ec98a6898ebc#kubernetes-helmPlugins.helm-s3

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/190sgdplcbxmb6icw4n9nb851ldx4apb-helm-s3-0.16.1
ls -la /nix/store/190sgdplcbxmb6icw4n9nb851ldx4apb-helm-s3-0.16.1/bin

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

1 package built:

kubernetes-helmPlugins.helm-s3

Maintainer pings
cc @yurrriq for testing.

Unfortunately I haven't used, helm-s3 for a while now, so am not in the best position to test. It builds and runs on x86_64-linux, at least.

The changelog seems innocuous enough. :shrug:

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/hypnoglow/helm-s3/releases.
meta.description for kubernetes-helmPlugins.helm-s3 is: Helm plugin that allows to set up a chart repository using AWS S3
meta.homepage for kubernetes-helmPlugins.helm-s3 is: https://github.com/hypnoglow/helm-s3
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.16.2 with grep in /nix/store/iky0f0rpnsg5q0245lcajz6pab4i08qi-helm-s3-0.16.2
found 0.16.2 in filename of file in /nix/store/iky0f0rpnsg5q0245lcajz6pab4i08qi-helm-s3-0.16.2

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

kubernetes-helmPlugins.helm-s3

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/ef9ba5e62a0e0b4fc4ed02892e6d64da229f7a83#kubernetes-helmPlugins.helm-s3

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/iky0f0rpnsg5q0245lcajz6pab4i08qi-helm-s3-0.16.2
ls -la /nix/store/iky0f0rpnsg5q0245lcajz6pab4i08qi-helm-s3-0.16.2/bin

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

1 package built:

kubernetes-helmPlugins.helm-s3

Maintainer pings
cc @yurrriq for testing.


[FLINK-13131] [Deployment / YARN] add a custom directory for storing configuration files and jar files for user jobs

The user-related jars and configuration files can only be placed under fileSystem.getHomeDirectory(/home/xxx), which is inflexible and increases the configuration so that users can customize the directory.

@Myasuka  Thank you very much for your review and Suggestions

In hadoop 2.7.2, if dfs.user.home.dir.prefix=/home is set, but the value returned by getHomeDirectory is /user/xxx, the actual online environment has speciality, so I think flink should provide a flexible configuration. So that the user can set a directory, if not set, use the default (/user/xxx).
I have modified the problem of import *
Thank you!

@zhangbinzaifendou Please correct me if I am wrong, if dfs.user.home.dir.prefix has been configured to another value, e.g. "/home". From Hadoop latest code, DistributedFileSystem would return the home directory as "/home/userxxx" via DFSUtilClient. In other words, there is no need for flink user to configure the staging folder since Hadoop could return the wanted home directory even dfs.user.home.dir.prefix changed.

@Myasuka Sorry for not being precise.
The apache community version of hadoop is fine, there is a problem in our hadoop version, I also said that our online environment is very special, for our online environment, so we need to add a configuration that can meet our needs very simply. It also adds flexibility to flink.If the user wants to redefine a new directory, flink can't do it now. So the benefits are obvious.
Thank you!

@zhangbinzaifendou as you explained, the motivation to create this PR is due to your special online environment and your Hadoop. However, this is not the reason that you have to modify the Flink source code. I think the real solution is to modify your special Hadoop which caused this problem.
As an open source software, Flink should adapt to the standard or widely-used ecosystems not some special customized ones. There would be no problems if we follow the correct logic of FileSystem.getHomeDirectory() even dfs.user.home.dir.prefix has been configured. And Flink use FileSystem.getHomeDirectory() to store its internal jars and files which is totally transparent to users. Exposing this internal implementation to users by adding such a configuration would never be a good idea.

@Myasuka Thanks for your reply.
Our design is based on the needs of the company, the changes are simple, but the impact is huge. This is obviously very flexible if Flink supports user-defined directories. I believe that many open source software must have a lot of unreasonable design from the beginning, and it is gradually matured based on the continuous improvement of many companies' real scenes. It is because we have such a scenario that we find that Flink design is not flexible enough, so we need to increase this feature, so that it will not affect the original use, but also ensure flexibility, is it better?

Hi @zhangbinzaifendou, thanks for your contribution.
I agree with @Myasuka. Your private hadoop version should be updated to support this scenario. Maybe you could build your own flink-shaded-hadoop uber jar.

@wangyang0918  Thank you for your answer.
I'm afraid I don't quite agree with you. User scenarios are very complex, shouldn't there be a flexible configuration? Providing flexible configuration is a principle of good architectural design.


Feature mouse events

Hi guys,
is is possible any mouse event in Feature itself?
I need to have mouse click (double click).
For example:
I have layer with one feature and when I click on it I need to do some action. When I do double click on the feature I want to do layer.Clear() (remove clicked feature)
The map has mouse events but how can I know the click was on a specific feature?
Regards,
Łukasz

Use the event Map.Info. This has a MapInfo property and a NumTaps. For a double click, NumTaps should be 2. MapInfo contains the Feature, which is clicked.
A short view into one of the sample should have delivered that.

An alternative. The Wpf MapControl has a GetMapInfo method that you can call from your own code. You you could bind to any event in your platform and call GetMapInfo from there.

But how can I find which feature is clicked?

The MapInfo return the feature, that is clicked. See comment above.


Have AbstractInstantAssertions implement ComparableAssert

Using an older assert/j, we have a bunch of assertions that
work using ComparableAssertions on Instant objects.  With the
addition of native Instant assertions, these currently fail to
compile.

Yeah, we should do the same for any Comparable type we provide assertions for.

@jbarnettwomply how much work is it to migrate the Comparable assertions to Instant ? I feel that adding isGreaterThan to Instant assertion is actually polluting the API so I would rather see people using the time related assertions instead of the number related ones.
Is there any assertion in Comparable that does not have an equivalent in Instant ?

it's not much work to change each Comparable assertion to the appropriate Instant/Temporal assertion, just a number of small changes scattered throughout our codebases and teams.  I agree it'd be better to be using time related assertions, the main reason for requesting this change is for backwards compatibility moreso than anything else.

I think I'm going to pass on this one and https://github.com/joel-costigliola/assertj-core/issues/1112 because of API pollution. Feel free to come with additional to change my mind.


Configure CI to correctly set isRelease

See https://github.com/ansys/grantami-serverapi-openapi/pull/122 and https://github.com/ansys/grantami-serverapi-openapi/pull/124
Also add the docs url

Also, assuming https://github.com/ansys/grantami-serverapi-openapi/pull/125 is merged, something similar should be done here as well.


Some way to know when drawer is open and closed

How about a this.props.navigation.isDrawerOpen() method?

Hi,
We have a withNavigationFocus hoc, maybe it's worth adding withDrawerOpen? It would permit to re-render on change without needing to create subscriptions.
Also, with hooks it would be more convenient to expose this as a hook too, like useNavigationFocused() or useDrawerOpened() (with better names if possible)

There is already a hook for focus actually: useFocusState(), so it may be worth creating a useDrawerState().
https://github.com/react-navigation/react-navigation-hooks#usefocusstate
Also, the navigation prop has a isFocused() func so it's probbably still worth to also add a isDrawerOpen fn here

We should trigger drawerOpen and drawerClose events like we trigger refocus event in tabs. This will need change in core to remove the restriction that only refocus can be triggered.

@satya164 I can help take care of this issue. If there is any further information you think I would need to achieve this task, feel free to let me know.

@michizhou we'd need to emit the events with navigation.emit (like we emit refocus here: https://github.com/react-navigation/tabs/blob/f27327cec5089b72796b393f55227e6719e029a1/src/utils/createTabNavigator.tsx#L151). There's also code in core which prevents any other event from being dispatched which needs to be removed: https://github.com/react-navigation/core/blob/cdc4d9192d5d3add1739cde47034050e52df70b4/src/getChildEventSubscriber.js#L187

@satya164 I have made new commits for my PRs based on your suggestions and feedback. Whenever you have the chance, feel free to look them over and let me know if additional changes are warranted. Thank you again for your time.

@michizhou thank you!


better-monadic-for should runsRightAfter the parser phase instead of runsAfter the parser phase

This commit fixes https://github.com/ThoughtWorksInc/Dsl.scala/issues/113

@Atry thanks, I'll check it and publish a new version soon


fixed "NativeScript CLI not found" error. That because execSync & s…

…pawn by default will run under '/bin/sh' without correct setup $PATH, instead we should use process.env.SHELL

@paynechu You are absolutely right.
Thanks for your contribution!


add retrolambda support again

I believe we can clean this up a lot. Ill commit on top of this branch. thanks for the initial idea!