Add ability to customize cache path and error message when not found

Mainly so it can be reused in alfy, but others might have a need for this too. I'm thinking environment variables, but happy to consider better ideas.

// @mafredri

We could also use getopts, and provide this functionally through cli flags, e.g. ./run-node -c /cache/path -m 'Error message' index.js
Although, in this case, providing flags to index.js would be risky and should perhaps be separated with a --, e.g. ./run-node -c /cache/path -m 'Error message' index.js -- -c -v -l.

That would conflict with node though, and even if we did -- it could easily conflict some Node.js script reading those.

I don't think it will conflict, since all parsed options would be removed, which would include -c, -m and --. So essentially my example above would call node like so: node index.js -c -v -l.
But I think environment variables can also work, both have their own advantages / disadvantages. Environment variables are a bit harder, more hidden, to use and requires previous knowledge, whereas the cli options could also include -h and --help. Or actually, now that I think about it, a help in both cases would be ok.

Ok, let me illustrate what I meant:
// x.js
console.log(process.argv.slice(2));

❯ node x.js -- -c -v -l
[ '--', '-c', '-v', '-l' ]

That means x.js would see those flags and might try to pass them to some other script that parses and uses it.
Using flags would also complicate the "proxy" I'll use in Alfy to defer to this script, as it needs to take out the -- flags.
Just feels easier and safer with namespaced environment variables.

Thanks for the illustration, I do get what you mean, node inheriting the arguments to run-node. But this is only the case if we pass $@ along to the node binary. What I'm proposing is that we don't, this means that -c /cache/path, -m errmsg and -- will never be sent as arguments to the node binary. And as in my example, [ 'index.js', '-c', '-v', '-l' ] is exactly what we intended to send.

Just feels easier and safer with namespaced environment variables.

Like I said, I'm fine with that too :).

What I'm proposing is that we don't, this means that -c /cache/path, -m errmsg and -- will never be sent as arguments to the node binary.

Ok, let me turn it around then. What if the user actually wants to pass -- --foo to a module expecting it.

Regardless, I think environment variables has a clearer intent here.

So to the option naming:

RUN_NODE_CACHE
RUN_NODE_ERROR

?

Ok, let me turn it around then. What if the user actually wants to pass -- --foo to a module expecting it.

-- is common in unix and indicates that we stop processing args. So from there-on-out, everything will be as-is. This means that passing -- is possible with -- --.

So to the option naming:

Some minor suggestions for clarity:

RUN_NODE_CACHE_PATH (if we expect a directory, maybe FILE if we expect a file?)
RUN_NODE_ERROR_MSG (clearer)

Sounds good to me. Thanks for the feedback.

This is now up for grabs if anyone want to work on it :)

Greetings
Made an implementation for this issue - can be found here


added style guide v1

just adding the doc. suggest changes to style guide in this thread, and it shall be done so.

Are you planning to add style for classes?

please define or give ref for class style


Add constraints to properties

What this PR does / why we need it
To avoid simpe input mistakes, this PR adds constraints to numeric and string properties.
Checklist

[x] Update changelog in CHANGELOG.md.

What's the state of this PR @marians ? I see that it's still a draft.

I have parked this PR since I didn't find the time to work out which constraints would actually make sense. But I think it's still needed, as we have many strings where e. g. adding a blank space by mistake could break the whole thing and this would be easy to catch.

Let's close this since the diff is outdated. We'd have to redo the change once time allows and if we still find it useful.


Error when initializing neosnippet cache.

I using vim 7.4 in OS X, when opening any file and type the INSERT shortcut to edit the file, I'm getting the following error:
Error detected while processing function neosnippet#init#_initialize..
Cannot create directory: /Users/../.cache/neosnippet

I think that is a the neosnippet plugin does not have permissions to write in the .cache directory. How this could be fixed?

Try giving .cache directory full permission i.e. 777. It worked for me.

so what's the normal user directory on OSX?  /Users/UserName ?

Yes it is /Users/UserName. In my case the owner of the .cache directory was the root. To fix the problem I simply changed the owner of the directory to UserName.

@mvpgomes : your solution works.
sudo chown -R $USERNAME ~/.cache
@hgupta : Your solution gives ANY user in caching directory the ability to read / write / execute. Thats a problem.


systemd service doesn't works

I copied and enabled the service, when I close the lid of my laptop it suspends, but when I open the lid, I only see the lockscreen a few miliseconds before disappears, so the screen never locks (not at the desired way). I'm using Debian 10. What I can do? Thanks

I merged a change to the service, try it out now please

I tried, it doesn't works, now it doesn't lock the screen at all

As this issue is really old and there was some work done before and now I hope now everything will be working if give betterlockscreen another try :)


freezes computer on ie9 - fast fourier transform is incredibly slow

i am well aware that ie9 is not one of the newest browsers but many people running windows 7 are still using it. however, i understand that this is probably not a priority for anyone so i realize that this may get just closed
anyways, when trying to init the clmtrackr model (for example, a model from the examples), the IE9 in about a minute or two, while other browsers do it in a couple of miliseconds
i have done some thorough benchmarking and it seems that the problems lies when doing the 2d fast fourier transforms.
this is the bottleneck
the very root of the slow performance lies when the FFT.real_fft2d is being performed and fft1d called within it. it lasts for a couple of miliseconds but adds up massively, since it's done in a loop. google chrome, for example does it within 1/2 miliseconds
perhaps it would work with another FFT implementation faster on IE9?
note: in order to make the library work on IE9 you need to include a TypedArray from somewhere, like from here

There's not really a way to make it run fast on IE9 that I can think of. The reason IE9 is so slow, is because it doesn't natively support typed arrays, which speed up the fft calculations. I doubt there are any other fft implementations that are faster, since I was looking around for the fastest version I could find, and additionally optimized it even more to get it to run fast.
Unfortunately, I think the best option for windows 7 users is to install another browser :(

thank you for the answer! since i now know that the very root of the problem lies within TypedArrays now i know i can stop trying  :)


feat: Bouncer refactor, test streamlining, and vocabulary updates

🚀 Description
Update the SignatureBouncer to have the correct signature size and make constructing signatures way easier.

[x] 📘 I've reviewed the OpenZeppelin Contributor Guidelines
[x] ✅ I've added tests where applicable to test my new functionality.
[x] 📖 I've made sure that my contracts are well-documented.
[x] 🎨 I've run the JS/Solidity linters and fixed any issues (npm run lint:all:fix).

cc @PhABC and @frangio for comments on the latest commit

I dm'd these questions to @PhABC but asking @frangio and @nventuro as well:

so right now, Bouncer requires that messages are signed for a specific sender, which doesn't let anyone in an open market submit a transaction (i.e. a decentralized ethereum alarm clock). do you think it's ok to relax that requirement? Not sure what the security concerns would be off the top of my head. Since if you cared about, say, who's getting tokens in an airdrop, you'd put something like mint(address _to, uint256 _amount, bytes _ticket) onlyValidTicket(_ticket) and force whomever the sender is to use the original _to argument that was signed.

I also realized that the bouncer contract can/should be its own delegate so instead of requiring that contractAddress be the first 20 bytes that are signed, we can just put that into the method itself like onlyValidTicket(address _delegate, bytes _ticket) and then check _delegate. The Bouncer contract implements isValidSignature on behalf of all of the ethereum account delegates that it tracks.

But if I remove the address(this) from the hash, it requires that _delegate is passed into the function so it gets included in the hash; aka, only verifying the signature or only verifying the method ID won't work. I think I'm ok with that because I can't think of a ton of examples where I wouldn't want total control over the arguments being submitted to a contract on my behalf. And if someone does care about allowing someone to choose an argument after signing, that can be implemented separately.
So my plan is to

Remove the :contractAddress + :sender prefix from the hash, and remove the "only validate signature" and "only validate method ID" functions.
require it such that _delegate must be provided in the function's arguments as well as _ticket (this used to be _sig, but I'm unifying the language)

Another question: if we have two required arguments (_delegate, _ticket) would it make sense to require that they be compacted into a single bytes argument and then parsed out? I think I prefer it as two separate arguments like
function mint(address _to, uint256 _amount, address _delegate, bytes _ticket)
  onlyValidTicket(_delegate, _ticket)
  public
{
  MyToken.mind(_to, _amount);
}

Do you think it's ok to relax that requirement?

I think it's ok security-wise.
As for point 2, I'm gonna have to think about it a bit more and get back to you tomorrow. I'm not up to date on how Bouncer is being generalized for types of delegates other than ECDSA signers.

@frangio for things other than direct ECDSA sigs, my primary use-case is identity contract support; i.e. "my contract is my identity and because it can't directly sign things, it delegates that responsibility to some keypairs that it's decided should be able to sign on its behalf"

The PR has diverged from a refactor into a big new feature. Judging from @PhABC's review, it seems like we're not all in the same page with regards to the design of delegates, so I'm going to mark it on hold until we figure out the requirements and a good design in #1005. (I'll be posting a summary of my thoughts there later today!)

this PR depends on #1005 which depends on #1104 which depends on https://github.com/ethereum/EIPs/pull/1258 which is discussed in https://github.com/ethereum/EIPs/issues/1271
So: direct all discussion of validation standards to https://github.com/ethereum/EIPs/issues/1271 after which we'll implement that standard and use it as part of this Bouncer refactor PR.

closing in favor of https://github.com/OpenZeppelin/openzeppelin-solidity/issues/1272


Include error

Hello everyone,
I have just integrated your library into our software (Xmipp) and I am trying to replicate the example.cpp file for our code. The include I use is #include "libcifpp/include/cif++/Cif++.hpp", which is the equivalent of #include <cif++/Cif++.hpp> in the original example.cpp, but when I try to compile it, it throws the following error:
In the included library/data/pdb.cpp:41 file:
../libcifpp/include/cif++/Cif++.hpp:44:10: fatal error: cif++/CifUtils.hpp: No such file or directory
   #include "cif++/CifUtils.hpp"
            ^~~~~~~~~~~~~~~~~~~~~
compilation terminated.

So the line of code that rises the error is in the file libcifpp/include/cif++/Cif++.hpp, matching the line 44, #include "cif++/CifUtils.hpp", which seems strange to us. Do you know any reason for this or do you have any suggestion?
Looking forward to your response, thanks a lot.

Are you trying to build the example.cpp file that comes with libcifpp?
The makefile in the example assumes you've installed libcifpp in a system wide location (e.g. /usr/local). But the default these days is in $HOME/.local. Perhaps you even installed it elsewhere. Anyway, if you now run pkg-config --cflags libcifpp it should print out  something containing -I/home/maarten/.local/include. (I installed libcifpp in my .local directory).
Now, if your pkg-config does not find libcifpp you either have to install it in e.g. /usr/local or you should specify the PKG_CONFIG_PATH in the environment.
First see if you can get pkg-config to locate libcifpp:
PKG_CONFIG_PATH=~/.local/lib/pkgconfig/ pkg-config --cflags libcifpp
And if that works, run make like this:
PKG_CONFIG_PATH=~/.local/lib/pkgconfig/ make
Hope this helps.

No, I basically copied the code inside the main method of the file example.cpp into our software (Xmipp), so I am compiling my code, instead of the file that you provide as part of the library.
In our case, all libraries are stored in this path ~/scipion3/xmipp-bundle/src/. We have tried to include it in the Xmipp environment like this PKG_CONFIG_PATH=~/scipion3/xmipp-bundle/src/, without success. Could you specify which environment you are referring to?
Thanks for your advice.

If you type:
pkg-config PKG_CONFIG_PATH=~/scipion3/xmipp-bundle/src/ --cflags libcifpp
Do you then get any output? If so, you should use that output when calling the compiler that compiles your code.
So, in my case e.g., if I run pkg-config like this:
$ PKG_CONFIG_PATH=$HOME/.local/lib/pkgconfig pkg-config --cflags libcifpp
It will reply with -pthread -I/home/maarten/.local/include
So I then have to call g++ with -pthread -I/home/maarten/.local/include

We found out what was causing the problem. If the include is #include <cif++/CifUtils.hpp> instead of #include "cif++/CifUtils.hpp", there are no compilation errors. Could you modify it? Because if we so do, any update in the library will bring back this issue.

Fixed, thanks for the feedback.


Bounding boxes

The poppler command line pdftotext to has a layout option allowing specific extraction based on bounding boxes. Can that feature be integrated into this?

I am also interested in this feature, and would help implement, but I'm not familiar enough with C. The poppler command line does have many possible arguments, and it would be nice to be able to implement them with this Python wrapper.
My usage case is to be able to extract text correctly from PDFs with multiple columns per page (ie like new papers), where layout is important.

Closing this one since it is discussed in more detail in #66


globally support config override flags in kgs

We want kgs to support config overrides such as --config and --context in general for all commands. Ideally this should be done at the root command.
Currently we only use clientcmd.ConfigAccess there. In order to support overrides we may need to work with clientcmd.ClientConfig.
Related:

https://github.com/giantswarm/roadmap/issues/1190
https://github.com/giantswarm/giantswarm/issues/22353
https://github.com/giantswarm/giantswarm/issues/22352

This is released with v2.19.0


[crashtracker] Move FFI to its own crate

What does this PR do?
Moves the crashtracker FFI to its own crate, seperate from Profiling.
Motivation
Consumers want to use crashtracking without profiling. This is a step in this direction.
Additional Notes

Building and shipping the crashtracking only part of the library is future work
As part of this refactor, I renamed a bunch of symbols which had unneeded prefixes.

How to test the change?
Run the existing test suite, plus the FFI examples
PROF-10245

Codecov Report
Attention: Patch coverage is 0% with 57 lines in your changes missing coverage. Please review.

Project coverage is 70.51%. Comparing base (eb62a6d) to head (6a42ab9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #551      +/-   ##
==========================================
+ Coverage   70.49%   70.51%   +0.01%
==========================================
  Files         213      213
  Lines       28412    28406       -6
==========================================
  Hits        20030    20030
+ Misses       8382     8376       -6

Components
Coverage Δ

crashtracker
21.04% <0.00%> (-4.10%)
:arrow_down:

datadog-alloc
98.73% <ø> (ø)

data-pipeline
50.00% <ø> (ø)

data-pipeline-ffi
0.00% <ø> (ø)

ddcommon
86.23% <ø> (ø)

ddcommon-ffi
74.58% <ø> (ø)

ddtelemetry
58.95% <ø> (ø)

ipc
84.18% <ø> (ø)

profiling
83.96% <ø> (+5.86%)
:arrow_up:

profiling-ffi
76.10% <ø> (+19.33%)
:arrow_up:

serverless
0.00% <ø> (ø)

sidecar
35.42% <ø> (ø)

sidecar-ffi
0.00% <ø> (ø)

spawn-worker
54.98% <ø> (ø)

trace-mini-agent
71.14% <ø> (ø)

trace-normalization
98.24% <ø> (ø)

trace-obfuscation
95.73% <ø> (ø)

trace-protobuf
77.16% <ø> (ø)

trace-utils
91.63% <ø> (ø)


Remove unnecessary properties

Most notably in the Game module, properties are used to enforce read-only variables. This has a noticeable performance overhead for a feature that is not being used correctly. Instead, change the property calls to the "private" variables.

Moving this out of the 1.1.0 release. Reason: this assumes a performance penalty that may be a micro-optimisation. A much better solution would be to actually profile the code and see if this is a worthwhile endeavour. Issue will remain open just in case.


New converter for the hOCR format

Where text is being extracted from a variety of types of PDF within a business process, those PDFs where the text is only present in image form will need to be analysed using an OCR tool which will typically output hOCR. It would be good to have a PDFMiner converter that extracts the explicit text information from those PDFs that do have it and uses it to generate a basic hOCR representation that is designed to be used in conjunction with the image of the PDF in the same way as genuine OCR output would be, but without the inevitable OCR errors.
I have already developed a solution and am about to submit a pull request.

Duplicate of #265


GeoDNS

While having little knowledge on GeoDNS and Anycast, is there support for both these options in nsd?

Hi @katzeprior! Sorry for the overdue reply, unfortunately no, not at this time.


0xa2036f0538221a77A3937F1379699f44945018d0

[x] I understand that token listing is not required to use the Polygon Wallet with a token.

Please provide the following information for your token.
Token Name: Alshwrmany
Token Symbol:0xa2036f0538221a77A3937F1379699f44945018d0
Decimals:
Polygon Address:0x3d14ee7BFEeaa4FA5C1745cb69ac19B684b55cBd
Ethereum Address:0x3d14ee7BFEeaa4FA5C1745cb69ac19B684b55cBd
Logo URI (svg):0 MATIC
$0.99875
Polygon zkEVM Address
0xa203...18d0
Project Name
Matic Token
Ticker Name
MATIC
Token Decimal
18
Project Name:
Project Summary:
Project Contact:
Project Website:
Mainnet/Testnet:
Additonal Notes:

0x3d14ee7BFEeaa4FA5C1745cb69ac19B684b55cBd

0 MATIC
$0.99875
Polygon zkEVM Address
0xa203...18d0
Project Name
Matic Token
Ticker Name
MATIC
Token Decimal
18


What if the OS is 2019? will it support the Chef extension.

[ What if the OS is 2019? will it support ]

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

ID: d60bc290-92c2-da24-fe4c-dac3801dfe10
Version Independent ID: 0c3ce3ef-d9c9-e5ff-3c61-121f8178baeb
Content: Azure Extension Supported Operating Systems - Virtual Machines
Content Source: support/azure/virtual-machines/extension-supported-os.md
Service: virtual-machines
GitHub Login: @genlin
Microsoft Alias: genli

@prafulbusa, Thanks for the question. Windows server 2019 should support most of the Azure extensions.


require load from gdisk path directly

I think everytime  Upload pictures of the class would cost unnecessary caculating time on colab.
Can you made a way to load these on gdisk so that we can upload them first.

I will add that option


How i can to use select option in Cell property

I have a problem when i use Select in Cell property
i tried this code. in console.log loop work and have value in val but not render option to select
{
         Header: "last update",
         id : "appoint",
         accessor: "appoint",
         Cell: row => (
               <select>
                   {
                         row.row.appoint.map((val,key)=>{
                             <option value={val.work_start_date_full}>val.work_text</option>
                         })
                    }
                 </select>)
}

then i tried. it . loop work ( try in console.log ) but display val is not defined
{
       Header: "last update",
       id : "appoint",
       accessor: "appoint",
       Cell: row => (
              <select>
                     row.row.appoint.map((val,key)=>{
                           <option value={val.work_start_date_full}>val.work_text</option>
                      })
              </select>
        )
}

example data in appoint
appoint : [
             {work_no : xxxx, work_txt:"xxxxxxxxxxxx' },
             {work_no : yyyy, work_txt:"yyyyyyyyyyyy' },
]

You need to return it from the Cell function.

You are likely to get quicker answers on the #react-table Slack channel (badge at the top of the README in the doco) than as a github issue. Thanks.

{
	id: 'steps',
	Header: 'setps',
	accessor: (d) => d.all_steps,
	Cell: (props) => (
		<select>
			{props.value.map(function(val, key) {
				return <option value={val}>{val}</option>;
			})}
		</select>
	)
},


Add kmp, a safe wrapper for google/go-cmp

Returns an error instead of panicking.
Holds common options that we use.

See: https://github.com/knative/serving/issues/2622

@mattmoor this is what I had in mind. I think we'll eventually want to get rid of any non-test usage of cmp, but we can use this in the meantime.

Great naming suggestions from @tcnghia:

kmp instead of cmp
SafeDiff instead of Diff

Happy to 🚲 🏠 but I like those 😄

/lgtm

would it be useful to try to get these upstreamed?

I suspect they would be amenable to a non-panicky variant of Diff.
I just found this comment that proposes a new option: cmpopts.AllowUnexported(cmpopts.PackagePath("k8s.io")). If both of those would land, we could ditch this package.
@dsnet any thoughts on returning an error instead of panicking? Also 👍 to a PackagePath option.


Add WinINet impl.

Task: Add WinINet impl. to investigate client-cert support.
Doc:

https://learn.microsoft.com/en-us/windows/win32/wininet/about-wininet
https://stackoverflow.com/questions/55297953/client-authentication-certificat-private-key-using-wininet

Done in 9d26492de84d94554d00f0191fe69f015de3d2a8


Deploying more than 32 products causes Roblox to throttle creation of additional products

Describe the bug
Deploying more than 32 products causes Roblox to throttle creation of additional products. This leads to a lot of weird behavior and errors as Roblox hits race conditions and other things.
To Reproduce

Create a new mantle.yml
Put this list of 100 products in https://gist.github.com/Quenty/d729acce94131b512df9b57970072fc1
Run mantle deploy

Expected behavior
Upload occurs, if not slowly.
Screenshots
Errors occur and we are throttled.
Environment (please complete the following information):

OS: Windows
Mantle Version: Mantle 0.10.8

Additional context
You can upload at a rate of about 2 seconds safely. There's also another bug I saw which is Roblox would return a 500 error, but still succeed in creating the product.

What would you like to see here? Should Mantle automatically throttle itself to one request every 2 seconds after it has created 32 products?


[Off-canvas] Scrolls to top on open by large content

When I click on the "open left panel" button the page scrolls to top, which shouldn't. The "forceTop" option is already set to false so I don't know why it scrolls to top. Maybe a bug? At least it is very annoying, maybe someone knows how to prevent this scrolling?
http://codepen.io/anon/pen/ZQoVPr

I have got the same issue. As I look in the code of foundation.offcanvas.js I can see, that this option is not in use right now. Every line in the code that is relevant for "forceTop" is commented out.
OffCanvas.prototype.open = function(event, trigger) {
  if (this.$element.hasClass('is-open') || this.isRevealed){ return; }
  var _this = this,
      $body = $(document.body);
  $('body').scrollTop(0);
  // window.pageYOffset = 0;

  // if(!this.options.forceTop){
  //   var scrollPos = parseInt(window.pageYOffset);
  //   this.$element[0].style.transform = 'translate(0,' + scrollPos + 'px)';
  //   if(this.$exiter.length){
  //     this.$exiter[0].style.transform = 'translate(0,' + scrollPos + 'px)';
  //   }
  // }

This is a huge usability problem. Why this code isn´t in use?

Talk to the guys who make Safari. cough cough Seriously though, it works great in Chrome, Firefox, Edge, and even IE 11, but the code you are referring to breaks on Safari, which is why it's not in use. Off-Canvas as a concept is a tricky one, and very difficult to make work cross-browser. For a very similar experience to an off-canvas, I humbly recommend a solution like this codepen.

Hey chris,
what do you mean with it works great in Chrome? On my enviroment the page scroll up to the top in Google Chrome. And this is a hard coded feature in the plugin that can´t be disabled at the moment.
$('body').scrollTop(0);

As in, when I was developing it, (in Chrome), the default action was the Off-canvas was sticky relative to the viewport and would allow the page scroll to remain wherever it was on opening. This broke in browser testing, requiring the off-canvas to be positioned at the top of the document, that, coupled with auto-focus and focus trapping for accessibility reasons, forces the page to scroll to the top. There's not much I can do about it at this time, because we can't drop support for Safari, and as of yet, don't have another solution that works on all browsers.

Did you even take a look at my example? If you did and maked some tests in different browsers you will notice that it only works in Firefox. In other words, only in Firefox the off-canvas doesn't scroll to top on close/open panels, in all the other browsers it does.
Thanks for the other solution though, but still the mentioned issue with the off-canvas is a serious one. Not only me and Baedda complaining about it, so I hope this could be fixed in the next release.

That is exactly what is expected based on the current implementation @Silvester20, except the weird thing in Firefox. It should scroll to the top on all browsers if there's a focusable element contained in the off-canvas pane. Again, this is not how I wanted to build this thing, it's just I'm limited to what I can do with translates and transforms on the entire content section of a page and maintain browser support.

I understand the reason what the forceTop option is used for and I understand why this is nenecessary.
But in my case I want to disable this behavior. I set forceTop to false and autoFocus also to false. I just want the off-canvas to slide in without effecting the scrolling in any way.
But in Chrome it jumps to the top and this doesn´t happen in Firefox or IE.
@Silvester20 did the same thing. He used data-options="forceTop:false;"to set the option to false so I think that he doesn´t want to have that browserjump. But the jump happens every time the Off-Canvas is opened. But it should only happen if the Off-Canvas is revealed or the option is set to true.


bug: entering a buffer from oil silences any prints done by BufEnter autocmds

Did you check the docs and existing issues?

[X] I have read the docs
[X] I have searched the existing issues

Neovim version (nvim -v)
NVIM v0.9.5
Operating system/version
Void Linux x86_64 (musl)
Describe the bug
When opening a file from oil, no BufEnter autocmds are triggered. any prints by autocmds are silenced. Using netrw yields the exact opposite result, consistently triggering the BufEnter autocmds, prints from BufEnter autocmds. The same applies to other plugins (telescope.nvim) and the :edit command (even when oil is loaded). Trying to circumvent this issue by creating a custom mapping for the select action, using a callback that triggers the select action and then manually the BufEnter event, doesn't help; nothing from BufEnter autocmds is printed.
What is the severity of this bug?
minor (annoyance)
Steps To Reproduce

nvim -u repro.lua
:Oil
open any file

Expected Behavior
All BufEnter autocmds are ran
Directory structure
./foo.txt
./repro.lua
Repro
-- save as repro.lua
-- run with nvim -u repro.lua
-- DO NOT change the paths
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "runtime", "cache" }) do
    vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
    vim.fn.system({
        "git",
        "clone",
        "--filter=blob:none",
        "--single-branch",
        "https://github.com/folke/lazy.nvim.git",
        lazypath,
    })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
    "folke/tokyonight.nvim",
    {
        "stevearc/oil.nvim",
        opts = {
            default_file_explorer = true,
            columns = { "mtime", "permissions" },

            skip_confirm_for_simple_edits = true,
            use_default_keymaps = true,

            view_options = {
                show_hidden = true,
                is_always_hidden = function(name, _)
                    return name == ".git"
                        or name == "."
                        or name == ".."
                        or string.match(name, "cache")
                end
            }
        }
    }
}

require("lazy").setup(plugins, {
    root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")

-- test autocmd
vim.api.nvim_create_autocmd("BufEnter", {
    callback = function(ev)
        print("Triggered:", ev.event, ev.file)
    end
})

Did you check the bug with a clean config?

[X] I have confirmed that the bug reproduces with nvim -u repro.lua using the repro.lua file above.

It seems that the issue is of an entirely different kind... as it turns out, something silences all prints. Writing something to a global variable does in fact alter it's contents on BufEnter. Reading the code for all BufEnter autocmds created by oil didn't really reveal why

I have encountered this problem before (missing prints in autocmds), though not in relation to oil specifically. I wish I had made more of a note of it, but I think I just worked around it and moved on. This is almost certainly either a bug or a weird interaction in Neovim itself. The thing to do here would be to try to get a more minimal repro (without using oil) and file it upstream on Neovim.

May I know more about the workaround? There is an issue where the autocmd BufNewFile hook does not work when creating a new file in oil buffer, causing the inability to use skeleton files.

Dug into it more, and found a fix in #348


fix(docs): fix bunch of typos in different MD files

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Typos fixes in MD files
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
No testing needed here
Checklist:

[ ] The title of the PR states what changed and the related issues number (used for the release note).
[ ] Does this PR require documentation updates?
[ ] I've updated documentation as required by this PR.
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have tested it for all user roles

Hi There,
We Thankyou for your valuable contributions. Please follow the steps below for the Giveaways Entry -

Once the PR is raised you can join the discord community for the upcoming updates.
After joining the community, send your PR link on the #contrib channel with the tag @Hacktoberfest.
We will evaluate the PR and once it is accepted , we will send you the swags.


❓ Question: "next start" does not work with "output: standalone" configuration.
Mar 26 02:20:19 hostname npm[3467416]: ><EMAIL_ADDRESS>start
Mar 26 02:20:19 hostname npm[3467416]: > next start
Mar 26 02:20:20 hostname npm[3467437]:  ⚠ "next start" does not work with "output: standalone" configuration. Use "node .next/standalone/server.js" instead.
Mar 26 02:20:20 hostname npm[3467437]:   ▲ Next.js 13.5.6
Mar 26 02:20:20 hostname npm[3467437]:   - Local:        http://localhost:3030
Mar 26 02:20:20 hostname npm[3467437]:  ✓ Ready in 1418ms

Thanks. I've changed the start command to pm2 start --name="pingvin-share-frontend" .next/standalone/server.js in the Readme.

Mar 30 18:17:37 host node[1099394]:   ▲ Next.js 13.5.6
Mar 30 18:17:37 host node[1099394]:   - Local:        http://localhost:3030
Mar 30 18:17:37 host node[1099394]:   - Network:      http://<IP_ADDRESS>:3030
Mar 30 18:17:37 host node[1099394]:  ✓ Ready in 636ms


Running examples locally breaks

Do you want to request a feature or report a bug?
A bug
What's the current behavior?
When installing the project and running the examples as described in the README. I get an error when I go to http://localhost:3000/examples
basic.js:21 Uncaught TypeError: Cannot read property 'add' of undefined
    at Basic (basic.js:21)
    at renderWithHooks (react-dom.development.js:12938)
    at mountIndeterminateComponent (react-dom.development.js:15020)
    at beginWork (react-dom.development.js:15625)
    at performUnitOfWork (react-dom.development.js:19312)
    at workLoop (react-dom.development.js:19352)
    at HTMLUnknownElement.callCallback (react-dom.development.js:149)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:199)
    at invokeGuardedCallback (react-dom.development.js:256)
    at replayUnitOfWork (react-dom.development.js:18578)

It seems that there is something wrong with this dependency:
import * as dates from 'date-arithmetic'

I am wondering if this might be related to:
https://github.com/intljusticemission/react-big-calendar/pull/1376
What's the expected behavior?
I expect the examples to load.

I am curious if there is anyone with the same problem?

I see this has been fixed!


test: add SSL test

This adds a simple SSL test along with the framework for running
the test Zookeeper in a mode where it listens on both SSL and
non-SSL ports.
This is based on earlier work in #619.

nudge @jeblair

I rebased this PR but it is not working because ZK is expecting a keystore and a truststore (java stuff) instead of .pem files. Will try to get this working soon and add some "cert auto generation".

@jeffwidman @ceache @jeblair this one is finalized, would love a review to get it merged.

Thanks for updating this.  ZK can use a pem file that is just a concatenation of the cert and key as its "keystore" without any special jks functionality.  But since that seems to have bitrotted in this PR somehow, doing all the actual jks stuff is probably going to be more robust in the long run.  :)

Thank you again for the PR @jeblair and thank you for the updated information. I remember using a product that would "weirdly" accept this kind of .pem files as a "keystore" and you are certainly right that it was ZK. My comment about the reason why the tests were failing was probably wrong.


Automate build process

For the build process review we are expected to have an automated build process running.

One command to trigger and perform a full build
One command to start the software for demoing purposes
All of this independently of a particular person’s machine

Implementation Notes
A docker-compose file could prove very helpful for this task.
Done?

All bullet points above should hold true.
Ideally we will have a bit of training / support for team members that are not yet experienced with using Docker.

Optional: Maybe add an option to exit the application "elegantly"


Potentially incorrect type using browser.management.onUninstalled

Hi there, I think there may be an incorrect type on parameter for the browser.management.onUninstalled api. When I use the addListener method, the type is defined as a Management.ExtensionInfo type, however the parameter actually only returns a string which is the id of the uninstalled extension.
This is the code I am trying to run and if I were to use it in the intended way using the value info.id, it comes back undefined when I try to use it.
browser.management.onUninstalled.addListener(info => {
  if (info.id === browser.runtime.id) {
    // Code continues here
  }
});

Since the parameter returns the id of the uninstalled extension I tried running the following code block but ran into this error because of the mismatched types:
This comparison appears to be unintentional because the types 'ExtensionInfo' and 'string' have no overlap .ts(2367)
browser.management.onUninstalled.addListener(id => {
  if (id === browser.runtime.id) {
    // Code continues here
  }
});

The equivalent Chrome api parameter is typed as a string so I think existing type definition is incorrect.

Interesting, you seem to have found a difference between the chrome API and the webextension API:
https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/management/onUninstalled
https://developer.chrome.com/docs/extensions/reference/api/management
This seems like a problem for the polyfill itself though, since this project is only about the types.
Please try to open an issue there to see if they want to fix it. If they won't, I can only offer to make the type a string | ExtensionInfo and you'll have to check the type in the listener, since chrome supplies a string and firefox would supply an object. Not sure about other browsers.

For a quick fix, the following might also work, even though it's not entirely accurate without adjusting the types in this project:
browser.management.onUninstalled.addListener((info) => {
    const id = typeof info === "string" ? info : info.id;

Seems like this is just a difference between the browser APIs themselves then. I will definitely try to open an issue over on the main pollyfill repo. I guess alot of these more niche APIs are used much more infrequently than say the storage API so inconsistencies like this can go under the radar quite easily. Since I made this issue initially I ended up not using this API anyways but I appreciate the quick fix regardless. Thanks for the help!


Dev/ianhu/fetch variables impl

For #4290

[X] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[X] Title summarizes what is changing
[ ] Has a news entry file (remember to thank yourself!)
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[X] Unit tests & system/integration tests are added/updated
[ ] Test plan is updated as appropriate
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed)

Skipping news as this implementation is a first step for the variables view and is not yet customer visible.

@ericsnowcurrently Yup, you have the basic gist of the change. Sorry, I would have helped give you a rundown on the change, but I wasn't sure who was picking it up until you had already reviewed it. Appreciate you chipping in on this. I'll get your feedback fixed up first thing tomorrow AM.

@JakeRadMSFT for review

@ericsnowcurrently I just pushed a new change up with your review fixes. I took pretty much all of them, but left open the conversations for the ones that I didn't fully take. I closed the conversations on the items that I fixed but did leave comments on some of them. Just wanted them out of the way as I fixes stuff up. Thanks again for the review.


Does pc.cond support more func to process the comparison?

Describe the bug
I need to apply some simple func like str.split() and list index to firstly process the value transfered to it, and then do a comparsion. Or any other approach is also appreciated.
e.g: The pc.foreach sends a value to this func below,
def a_func(value):
return pc.cond(
value.split(",")[0] == "0",
pc.td(pc.icon(tag="arrow_up")),
pc.td(pc.icon(tag="download"))
)
To Reproduce
Steps to reproduce the behavior:

Code/Link to Repo:

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
** Specifics (please complete the following information):**

Python Version:
Pynecone Version: 0.1.32
OS:
Browser (Optional):

Additional context
Add any other context about the problem here.

If you have some complex logic for pc.cond, you should consider defining a computed var and pass this var as the condition to pc.cond

Case is:
I get some data(list[str], as a var) from database, and sent to pc.foreach which calls the func defined above, the func calls pc.cond to check if part of the value is equal to something.

Finally I found a work around. My use case is to check the value of first field and then show something different(as above, up arrow or download icon) and meanwhile, directly refer to second field value.
At the very beginning, I combine those values of 2 fields together hoping for splitting them which was confirmed not possible. Thanks to https://github.com/pynecone-io/pynecone/pull/1029 which inspired me, so I append second field value to the list of first field. Therefore, judgement on first element of this list, and use second one. Problem fixed.
Use list[dict[str,list[str]]] is more functional than list[list[list[str]]] as we can use item[0] for access to key(be aware of what the key is), for my case, instead of using index.

Finally I found a work around. My use case is to check the value of first field and then show something different(as above, up arrow or download icon) and meanwhile, directly refer to second field value.
At the very beginning, I combine those values of 2 fields together hoping for splitting them which was confirmed not possible. Thanks to https://github.com/pynecone-io/pynecone/pull/1029 which inspired me, so I append second field value to the list of first field. Therefore, judgement on first element of this list, and use second one. Problem fixed.
Use list[dict[str,list[str]]] is more functional than list[list[list[str]]] as we can use item[0] for access to key(be aware of what the key is), for my case, instead of using index.


Dynamically set height and width of widget based on widget content

Hi,
Is there any way so that height and width of widget can be set based on content of widget?
I am showing iFrames on each widget, and m not sure what will be the actual content, so we don't want to show any scroll bar.
How to achieve the same?

+1

+1

:+1:

+1

+1

1

+1

+1

+1


Philips PUS8601

Hello World of Cumulus TV
I've installed the app on my Philips PUS8601 and so far so good. Culumus TV appears in the Leanback Launcher to configure and appears under Sources when I scroll down to the bottom.
The issues:
Firstly, these issues could be down to Philips, Android or Culumus TV, but I guess this is a good place to start.
My main source is Satellite. When I set up Culumus TV from sources, it managed to mess up my satellite channel list. After a hard reboot, all the satellite channels were gone and I had to reinstall them through the manufacturer process. Edit...I've realised that if I disable the channels in Live Channels the native TV app cannot see them either. Once enabled they are there again.
After installation, the channels appear in Live Channels via Culumus TV but I can only hear audio on the Satellite channels, the screen is blank.
Once I chosen a satellite channel none of the HLS channels work. The screen is blank and there is no audio. If I go back to a satellite channel then I can hear only the audio of the channel. Edit..My workaround is to select a HLS channel and choose Sources on my remote and then choose back. The HLS channels will work again.
When I try and open Live Channels from the Culumus TV app itself, it pops up and asks is Live Channels installed?

Philips TVs are doing something unusual and I don't exactly know what I
should fix.
On Tue, May 31, 2016, 12:27 PM imbalancedzero<EMAIL_ADDRESS>wrote:

Hello World of Cumulus TV
I've installed the app on my Philips PUS8601 and so far so good. Culumus
TV appears in the Leanback Launcher to configure and appears under Sources
when I scroll down to the bottom.
The issues:
Firstly, these issues could be down to Philips, Android or Culumus TV, but
I guess this is a good place to start.
My main source is Satellite. When I set up Culumus TV from sources, it
managed to mess up my satellite channel list. After a hard reboot, all the
satellite channels were gone and I had to reinstall them through the
manufacturer process.
After installation, the channels appear in Live Channels via Culumus TV
but I can only hear audio, the screen is blank.
Once I chosen a satellite channel none of the HLS channels work. The
screen is blank and there is no audio. If I go back to a satellite channel
then I can hear only the audio of the channel.
When I try and open Live Channels from the Culumus TV app itself, it pops
up and asks is Live Channels installed?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Fleker/CumulusTV/issues/124, or mute the thread
https://github.com/notifications/unsubscribe/ADI58_W5uVkSkpmsRtB5JrAjAA4-f7jTks5qHIuVgaJpZM4Iq3x5
.

Confirm, I'm facing same issue with cable TV on Philips 7600. It's propably related with Google Live Channels list auto managment. What version of GLC you are using? Does it comes preloaded on your TV or you need to sideload it?

This should be fixed soon.


migrating tips when already migrated

Reticketed from https://github.com/gratipay/gratipay.com/issues/3592#issuecomment-118201780.

Why did we migrate tips for some teams that already had tips migrated? Shields, kivy ...

That's a bug, looks like it happens when there are no tips to migrate.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/24023586-migrating-tips-when-already-migrated?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

Are tips being migrated when new teams are approved?  Can you point me somewhere we mention this?

Are tips being migrated when new teams are approved? Can you point me somewhere we mention this?

Yes. It's mentioned in point three under "Getting Back on Track" on Gratipay 2.0:

If we approve your application, we will migrate the weekly payments from your Gratipay user account to your new Gratipay Team, and you’ll start receiving them as the owner of the team instead of receiving them directly. Your next payout will include any balance you’ve accumulated on Gratipay to date.

Recent paydays include the log output for this step on their respective tickets: https://github.com/gratipay/gratipay.com/search?q=run+gratipay&type=Issues&utf8=✓.

Is this possibly overwriting tips that have been said after the first migration?

Based on https://github.com/gratipay/gratipay.com/issues/3656#issuecomment-126440082, my hypothesis is that next week (166) we'll see "already migrated" for zed and gkz-open-source, and "0 tip(s)" for matlab-schemer.

Confirmed: https://github.com/gratipay/gratipay.com/issues/3669#issuecomment-128360874.


Java API to get the size of output list operations

This opens up access to the TF_OperationOutputListLength C API from java, for operations that return output lists.

Can one of the admins verify this patch?

great suggestions @asimshankar - thanks, and I've incorporated them wholesale.

@tensorflow-jenkins test this please


git clonning for apolloauto pytorch doesn't work

Hello,
"git clone --recursive --single-branch --branch apollo --depth 1 https://github.com/ApolloAuto/pytorch.git" fails with my github id and key code.
$ git clone --recursive --single-branch --branch apollo --depth 1 https://github.com/ApolloAuto/pytorch.git
Cloning into 'pytorch'...
Username for 'https://github.com': my-id
Password for<EMAIL_ADDRESS>remote: Repository not found.
fatal: repository 'https://github.com/ApolloAuto/pytorch.git/' not found

The command above is on here : https://github.com/ApolloAuto/apollo/blob/pre6/docker/build/installers/install_libtorch.sh
And the very url was mentioned here : https://github.com/ApolloAuto/apollo/blob/266afbf68d83fa6fac7a812ff8a950223f5ab2c0/docker/build/installers/install_libtorch.sh
The url was introduced for users to be able to install libtorch for jetson board.
I am now trying to setup apollo 7.0 on jetson Orin AGX devkit.
OS : Ubuntu 20.04
cuda : 11.04
Latest libtorch in apolloauto for arm64 uses cuda 10.2 but the GPU in Orin doesn't seem to work with cuda 10.2. So I am trying to rebuild libtorch myself. Isn't there a recommeded guide possible? Can't I compile libtorch myself?
Thanks

You can use the new install_libtorch.sh which already support arm and x86, the old one not supported for now!


Avalonia.Markup.Xaml.XamlLoadException

I don't know if this is the right place to ask, but I get the error Avalonia.Markup.Xaml.XamlLoadException: 'No precompiled XAML found for AvaloniaTest.App, make sure to specify x:Class and include your XAML file as AvaloniaResource' when I try to run any Avalonia application. This is code generated when I used the Visual Studio extension to create a project. Doesn't work for regular Avalonia applications or MVVM ones.

Hello, this one can help you https://github.com/AvaloniaUI/Avalonia/issues/4373

@Pxlop @mate1213 I'd be interested to know whether doing a build from the command line using dotnet build fixes this problem?

@grokys I tried it. The buid was without error.

But this is the same result when I'm running the build from VS19.

Only at the Avalonia Diagnostic segment showing some wierdness:
09:44:18.049 [Information]  Avalonia Package initialized
09:46:07.777 [Information] 0 Starting previewer process for '"C:\Users\VirtualUser\source\repos\test\test2\bin\Debug\netcoreapp2.0\test2.dll"'
09:46:07.800 [Information] 9612 Started previewer process for '"C:\Users\VirtualUser\source\repos\test\test2\bin\Debug\netcoreapp2.0\test2.dll"'. Waiting for connection to be initialized.
09:46:08.723 [Information] 9612 Connection initialized
09:46:12.985 [Error] 9612 <= "Unhandled Exception: Avalonia.Markup.Xaml.XamlLoadException: No precompiled XAML found for test2.App, make sure to specify x:Class and include your XAML file as AvaloniaResource"
09:46:13.026 [Error] 9612 <= "   at Avalonia.Markup.Xaml.AvaloniaXamlLoader.Load(Object obj) in D:\a\1\s\src\Markup\Avalonia.Markup.Xaml\AvaloniaXamlLoader.cs:line 26"
09:46:13.227 [Error] 9612 <= "   at test2.App.Initialize() in C:\Users\VirtualUser\source\repos\test\test2\App.axaml.cs:line 13"
09:46:13.862 [Error] 9612 <= "   at Avalonia.Controls.AppBuilderBase`1.Setup() in D:\a\1\s\src\Avalonia.Controls\AppBuilderBase.cs:line 294"
09:46:13.873 [Error] 9612 <= "   at Avalonia.DesignerSupport.Remote.RemoteDesignerEntryPoint.AppInitializer`1.GetConfiguredApp(IAvaloniaRemoteTransportConnection transport, CommandLineArgs args, Object obj) in D:\a\1\s\src\Avalonia.DesignerSupport\Remote\RemoteDesignerEntryPoint.cs:line 120"
09:46:13.904 [Error] 9612 <= "   at Avalonia.DesignerSupport.Remote.RemoteDesignerEntryPoint.Main(String[] cmdline) in D:\a\1\s\src\Avalonia.DesignerSupport\Remote\RemoteDesignerEntryPoint.cs:line 145"
09:46:13.921 [Error] 9612 <= "   at Avalonia.Designer.HostApp.Program.Main(String[] args)"
09:46:19.850 [Information]  Process exited
09:46:19.850 [Error]  Connection error
System.IO.IOException: Unable to read data from the transport connection: An existing connection was forcibly closed by the remote host. ---> System.Net.Sockets.SocketException: An existing connection was forcibly closed by the remote host
   at System.Net.Sockets.Socket.EndReceive(IAsyncResult asyncResult)
   at System.Net.Sockets.NetworkStream.EndRead(IAsyncResult asyncResult)
   --- End of inner exception stack trace ---
   at System.Net.Sockets.NetworkStream.EndRead(IAsyncResult asyncResult)
   at System.IO.Stream.<>c.<BeginEndReadAsync>b__43_1(Stream stream, IAsyncResult asyncResult)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncTrimPromise`1.Complete(TInstance thisRef, Func`3 endMethod, IAsyncResult asyncResult, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Avalonia.Remote.Protocol.BsonStreamTransportConnection.<ReadExact>d__15.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Avalonia.Remote.Protocol.BsonStreamTransportConnection.<Reader>d__16.MoveNext()
09:46:21.088 [Information]  Stopping previewer process
09:46:21.529 [Information]  Stopping previewer process
09:46:24.236 [Information]  Starting previewer process for '"C:\Users\VirtualUser\source\repos\test\test2\bin\Debug\netcoreapp2.0\test2.dll"'
09:46:24.257 [Information] 13364 Started previewer process for '"C:\Users\VirtualUser\source\repos\test\test2\bin\Debug\netcoreapp2.0\test2.dll"'. Waiting for connection to be initialized.
09:46:24.817 [Information] 13364 Connection initialized
09:46:27.209 [Error] 13364 <= "Unhandled Exception: Avalonia.Markup.Xaml.XamlLoadException: No precompiled XAML found for test2.App, make sure to specify x:Class and include your XAML file as AvaloniaResource"
09:46:27.221 [Error] 13364 <= "   at Avalonia.Markup.Xaml.AvaloniaXamlLoader.Load(Object obj) in D:\a\1\s\src\Markup\Avalonia.Markup.Xaml\AvaloniaXamlLoader.cs:line 26"
09:46:27.242 [Error] 13364 <= "   at test2.App.Initialize() in C:\Users\VirtualUser\source\repos\test\test2\App.axaml.cs:line 13"
09:46:27.245 [Error] 13364 <= "   at Avalonia.Controls.AppBuilderBase`1.Setup() in D:\a\1\s\src\Avalonia.Controls\AppBuilderBase.cs:line 294"
09:46:27.248 [Error] 13364 <= "   at Avalonia.DesignerSupport.Remote.RemoteDesignerEntryPoint.AppInitializer`1.GetConfiguredApp(IAvaloniaRemoteTransportConnection transport, CommandLineArgs args, Object obj) in D:\a\1\s\src\Avalonia.DesignerSupport\Remote\RemoteDesignerEntryPoint.cs:line 120"
09:46:27.251 [Error] 13364 <= "   at Avalonia.DesignerSupport.Remote.RemoteDesignerEntryPoint.Main(String[] cmdline) in D:\a\1\s\src\Avalonia.DesignerSupport\Remote\RemoteDesignerEntryPoint.cs:line 145"
09:46:27.254 [Error] 13364 <= "   at Avalonia.Designer.HostApp.Program.Main(String[] args)"
09:46:31.672 [Information]  Process exited
09:46:31.676 [Information]  Stopping previewer process
09:46:31.672 [Error]  Connection error
System.IO.IOException: Unable to read data from the transport connection: An existing connection was forcibly closed by the remote host. ---> System.Net.Sockets.SocketException: An existing connection was forcibly closed by the remote host
   at System.Net.Sockets.Socket.EndReceive(IAsyncResult asyncResult)
   at System.Net.Sockets.NetworkStream.EndRead(IAsyncResult asyncResult)
   --- End of inner exception stack trace ---
   at System.Net.Sockets.NetworkStream.EndRead(IAsyncResult asyncResult)
   at System.IO.Stream.<>c.<BeginEndReadAsync>b__43_1(Stream stream, IAsyncResult asyncResult)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncTrimPromise`1.Complete(TInstance thisRef, Func`3 endMethod, IAsyncResult asyncResult, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Avalonia.Remote.Protocol.BsonStreamTransportConnection.<ReadExact>d__15.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Avalonia.Remote.Protocol.BsonStreamTransportConnection.<Reader>d__16.MoveNext()

But I didn't tried yet the solution in the issue #4379
So 'til now thats seems to realy promesing. When I will have some resul about that, will let you know.
Thanks for Help!

@grokys I forget to mention before, after I build from console I build it from VS too and the pictures came from that build sequence.

Not an issue with "axaml" files.


MWPW-137510

Resolves: MWPW-137510

Codecov Report

Merging #412 (079cf5f) into stage (934d55e) will increase coverage by 0.03%.
Report is 1 commits behind head on stage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            stage     #412      +/-   ##
==========================================
+ Coverage   93.50%   93.53%   +0.03%
==========================================
  Files          26       26
  Lines        2987     2987
==========================================
+ Hits         2793     2794       +1
+ Misses        194      193       -1

see 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


SDN Info

Write a pox program that prints out basic information about every packet it receives, such as the ethernet port, src/dest IP/port, protocol, etc.

@huyang1988 @pradeep-k @wenhuizhang

Please assign it to me. While looking for port translator implementation I did reading on pox and I know the APIs which does this. It will be a good exercise for me to have hands-on using pox.


Attach android app to releases

Hi! Would it be difficult to attach the android app to releases? That way, it could be included on F-Droid through alternative repos, which will find it difficult to do so otherwise :)

Hi @MarionMoseby, thanks for opening an issue! We have no plans to attach the release APK to releases at this time.


Add pre-job-start job name validation for make train

see https://github.com/neuromation/cookiecutter-neuro-project/pull/210#pullrequestreview-336523474

the job name is generated by NF and validated by neuro CLI - not actual


IS update to v1.3 and expiration time

@gunnarif IS/1.3.0/REC.png contains "du": "0001-01-01" which is not a valid date. See spec:
"The last date on which the certificate is considered to be valid, assigned by the
certificate issuer. The date MUST NOT be after the date calculated as r/fr +
180 days.
The date MUST be provided in the format YYYY-MM-DD (complete date
without time). Other formats are not supported."

@mkubicek-dtc
fixed

Android 1.1.12 🆗

iOS <IP_ADDRESS> (7) 🆗

@gunnarif Just to be sure: Was removal of 1.0.0 intended and are there no 1.0.0 DCCs in the hands of citizens?


NIFI-11874: update process group configuration dialog UI, specific to nifi-1.x line

… nifi-1.x line

Summary
This PR is for merging to the support/nifi-1.x line. There is a separate PR for merging to main. See https://github.com/apache/nifi/pull/7591
NIFI-11874
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[ ] JDK 21
[X] JDK 8

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Reviewing...

Thanks @mcgilman for reviewing this 1.x-specific version of this MR too.


python not recognized properly after updating to 3.9

after updating python to 3.9 I can't jump to functions definition , get outline and code structure.
Python Version : 3.9
Extensions installed : Python Extension by Microsoft.
Thank's in advance.

Hi @AliMejbar, I'm a developer on the Python extension by Microsoft. We'd appreciate it if you could file this issue directly with us: https://github.com/microsoft/vscode-python.
When you file an issue with us, please also give us information on which Python language server you are using (Pylance, Microsoft, or Jedi) so we can better assist you! You can check this by looking at the python.languageServer value in your user settings.json file.
Thanks very much!

Hi @AliMejbar, I'm a developer on the Python extension by Microsoft. We'd appreciate it if you could file this issue directly with us: https://github.com/microsoft/vscode-python.
When you file an issue with us, please also give us information on which Python language server you are using (Pylance, Microsoft, or Jedi) so we can better assist you! You can check this by looking at the python.languageServer value in your user settings.json file.
Thanks very much!

hi thank's for your help , reinstalling the python extension worked , Thank you.


configurable parser encoding

Python3's open method opens the file using the systems encoding by default unless otherwise is specified.

In text mode, if encoding is not specified the encoding used is platform dependent: locale.getpreferredencoding(False) is called to get the current locale encoding.

In python2's open seems to always use byte str with US-ASCII encoding. I've bumped into quite a few obj files containing characters outside the python 2 str range.
It might be a good idea to make this a bit more deterministic by passing in utf-8 as the default encoding in the constructor and use codecs.open to read the files. This is slightly slower compared to reading byte strings, but will ensure the parser works in the vast majority of the cases. People who want to optimize can pass in 8 bit encodings if needed.

Fixed. Will make PR once my last ones are merged.

👍


Prevent editor spin slider from holding focus when not being edited

Fixes https://github.com/godotengine/godot/issues/95607 regression
Allow the main editor viewport to grab focus automatically when editor spin slider is not in text edit mode.
https://github.com/user-attachments/assets/c7db5d61-c29a-468d-a9c4-dcb283ab9b08
Reminder that all other inspector inputs except for text inputs allow the main viewport to regain focus on mouse hover.

CC @passivestar (regression fix from #93165) @KoBeWi

This reintroduces https://github.com/godotengine/godot/issues/88102, I'll open an alternative PR in a minute


operator [R] pulp-operator (0.10.0 0.10.1 0.11.0 0.11.1 0.12.0 0.13.0 0.14.0 0.14.1 0.5.0 0.6.0 0.6.1 0.7.0 0.7.1 0.8.0 0.9.0 1.0.0-alpha.1 1.0.0-alpha.2)

Signed-off-by: Fabricio Aguiar<EMAIL_ADDRESS>Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want to use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[ ] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description of the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/merge possible


netsim-ffi: consider sending void* instead of bytes

in #9 we introduced sending Box<[u8]> but this forces the users to serialise into bytes.
Consider using SimContext<* mut std::ffi::c_void>.

So it teems it's possible to pass void* instead of forcing the FFI callers to serialise into bytes.
We will need to make sure the constraints are well documented:

the rust code will take ownership of the pointer while it is in transition, i.e. no one should release the asset while passed through the send function
the rust code will release the ownership of the pointer when it is received, i.e. the caller is now responsible to releasing the asset when no longer used

Additionally the FFI will require a handle in order to hand back to the caller the released asset if/when dropped,

[ ] add context to have an optional on_drop function to call when a packet is being dropped (default None)
[ ] add FFI to register handle

void netsim_context_register_drop(Context* context, void (handle)(void*));


Out of date docs: Using Your Own Domain

https://www.kubeflow.org/docs/gke/custom-domain/
This page is very out of date.  I have not spent enough constructive time to figure this out on my own, but this does not seem consistent with the current project setup in 0.6 or 0.7.

It appears from some output in coordinator.go that the customization is done in app.yaml prior to kfctl generate.  Does that sound right?

/area docs
/priority p2

I'm taking this one!
/assign @fediazgon


Flexible Axpickvel with a 3x3 upgrade crashes the game when targeting the clear glass on a condenser multiblock

See https://github.com/davqvist/Exoria/issues/267
Might be because of the fluid or the clear glass, not sure

use latest version


Enable Azure Active Directory MFA authentication

This PR enables Azure Active Directory MFA authentication by adding a new authentication type, allowing the user to select a signed-in account, add a new account, and refresh credentials from the connection dialog.
This is for issue #13
This PR depends on https://github.com/Microsoft/sqltoolsservice/pull/727 so I'll merge it after that PR gets merged and we have a new version of SQL Tools Service.

I'll update this and merge it once I have a new Tools Service build


Update app.js and use arrow functions

Added morgan middleware for logging and debugging purpose and replaced normal callbacks with arrow functions.

Please have a look @theprover97 sir. Thanks.

Howdy...


Added Files for Automating Checking for Google Updates

Added the files needed for the Google Update Tracking automation. the following files were added:

update_checker.py
update_log.csv
updates.txt
check-updates.yml

This script scrapes updates from the Google Product blog, performs initial triage, checks the updates.txt file to see if it has already checked this update, creates Github issue, and logs update in update_log.csv. Finally, it sends email notifications to TCO, Technical Content Development Lead, and others.
This also creates a Github action to run this checker every hour on the hour.
DO NOT MERGE YET.  However, all the needed code should be here.

I am working on fixing the pylint errors.

I have fixed some of the errors, but the since all of them are styling, I don't think they will cause issues.

Some of the style changes would not be a practical use case in this context. For example using the following syntax:
with open (filename, operation):
< other code here>
when using the file throughout the code what create a large section of indented code, when using:
file = open(filename, operation)
would be a much more practical way to open the file for this context.

Closing PR. Found another solution.


CellRenderer alignment not supported

The cell renderer doesn't support alignments.
User: features
Test screen: CellProfile (LAST_NAME column)
The alignment isn't recognized at all.

this should already be fixed now by 99ccfb1f92fd60015fb74b00f1bfdffba665f765 -> last name is right aligned

the LAST_NAME is now right, but alignment doesn't work in general because:
If you set the alignment for the DOB column (CellProfileWorkScreen in Eclipse):
search for: mdbData.getRowDefinition().getColumnDefinition("LAST_NAME").getDataType().setCellEditor(tcedRight);
and add:
mdbData.getRowDefinition().getColumnDefinition("DOB").getDataType().setCellEditor(new UINumberCellEditor(IAlignmentConstants.ALIGN_RIGHT));
Alignment of DOB column works, but not formatted:

and if you set the alignment for a linked cell editor:
	UIEnumCellEditor eced = new UIEnumCellEditor(new Object[] {"A", "B", "C"});
	eced.setHorizontalAlignment(IAlignmentConstants.ALIGN_RIGHT);

the text is still left aligned.
Alignment should work for text, number, date, linked, image renderers.


Add CodeGen for Swift (URLSession)

Tell us about the task you want to perform and are unable to do so because the feature is not available
Add CodeGen for Swift
Library - URLSession

is anyone currently working on it, I would like to take it :)

@deepraj02 Sure, you can go ahead and work on this issue.

Thanks, also I see one closed PR for this issue can you guide me what's wrong with that, so I would avoid that.

@deepraj02 Please make sure you follow these guidelines https://github.com/foss42/apidash/discussions/80 while working on this issue.

Got it, Thanks


Is 1.1.x backward compatible with 1.0.x？

Hi，
We want to upgrade the snapp-java from 1.0.x to 1.1.10，but concern about the compatibility.
According to my understanding:

data compressed by 1.1.x could not be decompressed by 1.0.x (Non Framing APIs)
data compressed by 1.0.x could be decompressed by 1.1.x  (Non Framing APIs)
Is that right?

thanks

After some tests，it seems that the compressing apis are compatible between 1.0.5 and 1.1.10.
But not sure why someother report the compatibility issue: https://stackoverflow.com/questions/63185644/compress-and-uncompress-using-different-version-of-snappy-java-library

Snappy format itself has not been changed.
The framing format is a new addition to the snappy, and it's different from the regular snappy format. See https://github.com/xerial/snappy-java#compatibility-notes


Big 222 fix external bucket registration

Jira: BIG-222
KBC:  https://github.com/keboola/connection/pull/4846
Before asking for review make sure that:
Checklist

[x] New client method(s) has tests
[x] Apiary file is updated

Release

[x] I gave the PR a proper label:

major (BC break)
minor (new feature)
patch (backwards compatible fix)
no release (just test changes)

Este som upravil testo co testuje ze ta schema neexistuje pre snowflake, a pridal test pre BQ


Updated doctrine-orm-module package dependency version

to ^1.0 instead of 0.*

Master branch has been updated to include version 1.0+

Master branch has been updated to include version 1.0+

It'd be nice to have a tag version.

@marden, yes it would be nice to see that.
@Danielss89, could you add a new tag for dependency upgrade of the docctrine-orm-module? Instead of using the master branch.


Update Java to JDK 21

Changes:

Update JDK version to 21 in builder images

from #1758

/label px-approved
/label docs-approved
/label qe-approved

/cherrypick release-4.15

/lgtm

/lgtm


jit: support for and, or, xor, shl, shr, rotl and rotr operations

Signed-off-by: Takeshi Yoneda<EMAIL_ADDRESS>

Implemented Rotr and Rotl as well! https://github.com/tetratelabs/wazero/pull/118/commits/bbeaec5baa4578740593babfcb0b49fc63f5e658

Thanks for the look! 👯


Use of WebviewEditorInset

Hey there. For the last few days I have been experimenting with something called WebView. It essentially allows HTML rendering inside vscode.
There have been many requests over the years for vscode community to allow inline WebView render into editor.
Here is something similar to what we want: https://github.com/microsoft/vscode/issues/41775
VScode has added and merged a code inset feature here: https://github.com/microsoft/vscode/pull/66418
And here is their progress for WebviewEditorInset API: https://github.com/microsoft/vscode/issues/85682
There is however a drawback at this moment:

WebviewEditorInset is a proposed API, meaning that I don't know if this can be used is its not officially into vscode. It is like an experimental feature. But i'm pretty sure they will implement this after half a year or year (yeah, that a lot), but this has been a really requested feature and they also need it for julia lanuguage extension and jupyter notebooks.
Current solution requires to copy a vscode.proposed.d.ts file for this to work
VS Code insiders edition is also required for this

Heres a demo that I have managed to get working:

There are a lot of work to be done here, but I think it looks cool.
Also, you mentioned that you have to create a /tmp directory for this, check out what storagePath is in vscode.ExtensionContext in https://code.visualstudio.com/api/references/vscode-api#ExtensionContext
I am using this directory instead of /tmp, it looks like this: /home/maris/.config/Code - Insiders/User/workspaceStorage/RANDOM-HASH/johnguo.simply-view-image-for-python-opencv-debugging/
Is this the direction we can shift instead of opening images into a new pane? We can also try to combine them, or make this as a separate experimental branch.

Can you create a new branch called 'inline-render' so I can put the demo there?

The purpose of this extension is every using python-OpenCV vscode user could watch  image quickly and simply. It's for every vscode user not only experiment user.
About the temporary files, I think it's ok to use the storage path. But in my system, the temporary path is mapping to a memory driver, It's much faster than any other path. I will add an option to choose which path (temp or storage) to save later.
And last I will create a branch called 'inline-render'.


Allow swapable auth client

Allows you to swap the class used for the auth client, letting you do things like change the cache backend to django or redis, useful when using lambdas or other rapidly cycled services.
from django.core.cache import cache

from sp_api.auth import AccessTokenClient, AccessTokenResponse

class DjangoBackedClient(AccessTokenClient):
    def get_auth(self) -> AccessTokenResponse:
        cache_key = self._get_cache_key()
        try:
            access_token = cache[cache_key]
        except KeyError:
            request_url = self.scheme + self.host + self.path
            access_token = self._request(request_url, self.data, self.headers)
            # Expires after 1h, but lets be safe.
            cache.set(cache_key, access_token, 60 * 50)
        return AccessTokenResponse(**access_token)

Apologies, didn't mean to push this without a description! I've added one.

I was a bit confused at first, but quite easy to understand… 😀


Update request for PodSecurityPolicy Deprecation

The current plan is to remove PSP from Kubernetes in the 1.25 release.

@pratikpparikh thank you for raising this issue. I plan to update the chart to address all Kubernetes resource deprecation warnings in early October.

Thank you, @michaeljmarshall.


update_config function makes it unable to adjust learning rate by config

In utils/main_utils/update_config, it seems the config will be replaced by default args. So when I changed the learning rate in config, it may print 'overwrite config key -- lr: 0.005 -> 0.001'. And the actual learning rate is always 0.001
What's the purpose of update_config function?
def update_config(args, config, exclude_keys=list()):     """update the args dict with a new config"""     assert isinstance(args, dict) and isinstance(config, dict)     for k in config.keys():         if args.get(k, False):             if args[k] != config[k] and k not in exclude_keys:                 print(f'overwrite config key -- {k}: {config[k]} -> {args[k]}')             else:                 args[k] = config[k]         else:             args[k] = config[k]     print(args)     return args

I think the problem persists on the master branch. I wrote a pull request to change the parser behavior so that we can now follow a priority list:
1- Command line arguments
2- Arguments provided in a --config file
3- Arguments provided in the default .py config file provided with openSTL for that method.
4- Default arguments from parser.py.
https://github.com/chengtan9907/OpenSTL/pull/47

I've been testing it over the past weeks. While the new method is a bit weird to use, it works perfectly. I suggest closing this issue.


Auto performance monitoring for widgets

:scroll: Description
Adds a breadcrumb for taps
Starts an Idle transaction for taps
:bulb: Motivation and Context
Closes https://github.com/getsentry/sentry-dart/issues/706
:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

Codecov Report
Base: 90.09% // Head: 91.66% // Increases project coverage by +1.56% :tada:

Coverage data is based on head (d368f8d) compared to base (48e79fd).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1137      +/-   ##
==========================================
+ Coverage   90.09%   91.66%   +1.56%
==========================================
  Files         117        9     -108
  Lines        3605      192    -3413
==========================================
- Hits         3248      176    -3072
+ Misses        357       16     -341

Impacted Files
Coverage Δ

dart/lib/src/noop_sentry_span.dart

dart/lib/src/protocol/breadcrumb.dart

dart/lib/src/protocol/sentry_span.dart

dart/lib/src/sentry_span_interface.dart

dart/lib/src/sentry_tracer.dart

dart/lib/src/sentry.dart

dart/lib/src/hub_adapter.dart

...ib/src/environment/_web_environment_variables.dart

dart/lib/src/protocol/span_status.dart

dart/lib/src/protocol/sdk_info.dart

... and 98 more

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

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

This should be tested with platform views. I.e. it shouldn't prohibit interactions with platform views.

What do you mean by platform view, this?

This should be tested with platform views. I.e. it shouldn't prohibit interactions with platform views.

What do you mean by platform view, this?

Yeah, it can be tested by using one of the webview plugins, google maps or the camera plugin.

This should be tested with platform views. I.e. it shouldn't prohibit interactions with platform views.

What do you mean by platform view, this?

Yeah, it can be tested by using one of the webview plugins, google maps or the camera plugin. The reason is that I've seen global click listeners prohibiting interactions with those.

gotcha, I recall this from here https://github.com/getsentry/sentry-dart/issues/706#issuecomment-1031382401
Will do, thanks for the reminder.


how to build merkle_branch

now I need to do a siacoin mine pool,need to use (/transactionpool/transactions) api build merkle_branch，the latest version of the api did not provide, view the historical version(0.6.0) is yes, then the latest version of what should I do

Sorry, I am not sure what you are asking.  Maybe you can see if there is a Sia Discord channel in your native language and if they can answer your questions.
Recommendation to devs: try to interpret; else "close"


Update nightly build script to quietly publish public Parquet outputs.

Overview

Publish Parquet files to the public GCS & S3 buckets when the builds succeed.
Create a pudl_parquet.zip archive containing all the Parquet outputs that we can point Kaggle at.
Switch to using zip instead of gzip to compress SQLite files because Windows can only open zip archives and we don't want Windows users to have to download a 3rd party archive/compression utility.

Closes #3678
Testing

Is there an easy way to give this a test besides doing a whole deployment?

# To-do list
- [x] Review the PR yourself and call out any questions or issues you have
- [x] Update Zenodo data release process to only archive the zipped parquet files (due to file number limit)
- [ ] Ensure docs build, unit & integration tests, and test coverage pass locally with `make pytest-coverage`

A bunch of the deployment complexity stems from the different needs of the different deployment targets, combined with the fact that we're trying to use the same directory to prepare all of those different outputs.  Maybe it would be simpler to never touch the build outputs, and instead populate a temporary directory for each of the different places that data needs to be sent, such that each of them can be independently set up for the needs of that target?  Which would include:

Kaggle
Zenodo data release
Public GCS & AWS buckets
Private Parquet bucket (to be deprecated soon)

I'd love to refactor this more deeply, bring it into Python, make it testable (and avoid the need to install and depend on the gcloud and aws CLI tools) but at least for me I think that would take the better part of a work week and feels like more resources that we want to spend on just getting a basic version of the Parquet files out. Maybe we can put the refactor into our project menu for the summer? I think it would fall under the infrastructure portion of the POSE project if it comes through.

I'd love to refactor this more deeply, bring it into Python, make it testable (and avoid the need to install and depend on the gcloud and aws CLI tools) but at least for me I think that would take the better part of a work week and feels like more resources that we want to spend on just getting a basic version of the Parquet files out. Maybe we can put the refactor into our project menu for the summer? I think it would fall under the infrastructure portion of the POSE project if it comes through.

I think the reason to put work into testability now would be if we think that saves us time over "try running the nightly build and hope it works" - hence me suggesting the "incremental" (read "half-assed") way of getting some testability just for the bit of logic you're trying to add.
I totally trust you and your judgement on what would be the fastest expected path to shipping the dang Parquet, though - you're the one doing the development!

🙈 I will be pleasantly amazed if it actually works on the first try. 🙈


Guava v31.0.+ introducing @ParametricNullness breaks our builds

Nullaway Version: 0.8.0
Guava Version: v31.0
Build System: Bazel version 4.2.2

[x] If you think you found a bug, please include a code sample that reproduces the problem. A test case that reproduces the issue is preferred. A stack trace alone is ok but may not contain enough context for us to address the issue.

In the latest version of Google Guava, ParametricNullness is introduced. In v30.0 FutureCallback, for example, onSuccess is marked with @Nullable, however in v31.0 FutureCallback Guava changes this to @ParametricNullness. Since this annotation is unrecognized by Nullaway, it causes our builds to fail with the error:
pkg/NullawayParametricNullness.java:11: warning: [NullAway] passing @Nullable parameter 'result' where @NonNull is required
        futureCallback.onSuccess(result);
                                 ^
    (see http://t.uber.com/nullaway )
error: warnings found and -Werror specified

Source:
package pkg;

import com.google.common.util.concurrent.FutureCallback;
import javax.annotation.Nullable;

public class NullawayParametricNullness {
  public static <T> FutureCallback<T> wrapCallback(FutureCallback<T> futureCallback) {
    return new FutureCallback<T>() {
      @Override
      public void onSuccess(@Nullable T result) {
        futureCallback.onSuccess(result);
      }

      @Override
      public void onFailure(Throwable throwable) {
        futureCallback.onFailure(throwable);
      }
    };
  }
}

Thanks for the report!  My first thought as a quick fix is to just treat @ParametricNullness as if it were the same as @Nullable.  That should certainly fix this issue.  But I will try to dig and understand a bit better.

@holtherndon-stripe : Potential work around and our own proposed plan forward in this comment upstream.
tl;dr: -XepOpt:NullAway:CustomNullableAnnotations=com.google.common.util.concurrent.ParametricNullness should get you unblocked and we plan to hard-code support in the next NullAway version. It's not a perfect map for the semantics of @ParametricNullness (and I am still confirming that we got those right), but it might be the best we can do until we have full support for nullability of generics (type parameters).

Might be easier to wait for a new release of NullAway which handles them all (currently have a PR up for this: https://github.com/uber/NullAway/pull/629 )
OOC, roughly would this release be?

Might be easier to wait for a new release of NullAway which handles them all (currently have a PR up for this: #629 )

OOC, roughly when would this release be?

Aiming for the end of this week or early next week. However, if this is blocking people, I can also try and aim for tonight/tomorrow. There is some internal testing we'd like to do on an unrelated feature (unlikely to be enabled outside of Uber right now), but worst case it just means cutting two releases one soon after the other.

I don't believe it's blocking. We wound up suppressing a large number of warnings though for now, and those can be undone when we merge in the next version of NullAway.
It is somewhat time sensitive though, as I would like to avoid people too eagerly suppressing warnings and having to find them later.

It is somewhat time sensitive though, as I would like to avoid people too eagerly suppressing warnings and having to find them later.

Given this, we are releasing NullAway 0.9.9 with the fix today, please let us know if you run into any issues!
Note two things:

In general, said fix involves a trade-off where false positives are possible for Guava APIs (as opposed to false negatives). We are explicitly treating Foo<T> as having @Nullable T if that type is permitted for any instance of Foo.
In theory, this is no different than the level of support we had for nullability in Guava before 31.0, but the fact that they have more precise tracking of nullability depending on type parameters, means Guava has added @ParametricNullness to places were @Nullable wasn't present before (see this comment) and might do so more liberally in the future.

Short term, this is the best option we have for handling 31.0 in a manner that is compatible with how we handled previous versions of Guava.
Long term, if both Guava and NullAway implement JSpecify semantics (or even some more rudimentary compatible generics / type parameter support to begin with), this will lead to more precise tracking of nullness information through Guava APIs (e.g. FutureCallback<@Nullable String> will allow null, and FutureCallback<String> will not, and NullAway should be able to track which is which). This is definitely in our roadmap, just not something we can implement in a short timeframe!
We are exploring some medium-term solutions too. On the user side, right now, a medium effort workaround exists for any new false positives: 0.9.9's treatment of @ParametricNullness can be augmented with custom library models to force specific classes within Guava to default to disallowing null, even if Guava itself says it's up to the type parameter.
Will close this specific issue after a while or if we hear that NullAway 0.9.9 generally solves the issue for the time being :)

@holtherndon-stripe @rwinograd can this issue be closed now?

Yep! This solved our issue, I'll go ahead and close it!


chore: use PrimitiveArray::patch in bitpacking take

Resolves #213 (I think!).

would be good to run cargo bench --bench bitpacking_take on this branch vs develop just to confirm no regression, but lgtm

Hmm. there is a real slowdown:
group                                      develop                                pr
-----                                      -------                                --
patched_take_10K_adversarial               1.08    101.9±4.15µs        ? ?/sec    1.00     94.6±7.05µs        ? ?/sec
patched_take_10K_contiguous_not_patches    1.00     45.5±2.82µs        ? ?/sec    1.25     56.8±3.00µs        ? ?/sec
patched_take_10K_contiguous_patches        1.00     82.4±2.93µs        ? ?/sec    1.30   106.7±14.22µs        ? ?/sec
patched_take_10K_random                    1.00    157.2±6.15µs        ? ?/sec    1.03    161.5±6.88µs        ? ?/sec
patched_take_10_contiguous                 1.01    942.3±7.77ns        ? ?/sec    1.00   932.6±18.96ns        ? ?/sec
patched_take_10_stratified                 1.00    896.2±4.57ns        ? ?/sec    1.02   910.1±20.52ns        ? ?/sec
patched_take_200K_dispersed                1.00   496.6±50.84µs        ? ?/sec    1.02   506.6±41.16µs        ? ?/sec
patched_take_200K_first_chunk_only         1.08   388.0±25.74µs        ? ?/sec    1.00   360.3±19.99µs        ? ?/sec
take_10K_contiguous                        1.01     11.3±0.04µs        ? ?/sec    1.00     11.2±0.07µs        ? ?/sec
take_10K_random                            1.00     58.9±0.16µs        ? ?/sec    1.05     61.6±0.42µs        ? ?/sec
take_10_contiguous                         1.09    401.8±8.34ns        ? ?/sec    1.00    369.7±8.71ns        ? ?/sec
take_10_stratified                         1.02    365.1±2.30ns        ? ?/sec    1.00    356.9±3.47ns        ? ?/sec
take_200K_dispersed                        1.00    421.7±6.07µs        ? ?/sec    1.02   431.1±19.55µs        ? ?/sec
take_200K_first_chunk_only                 1.02   324.7±18.05µs        ? ?/sec    1.00   318.9±15.96µs        ? ?/sec

The profiles lack any clear issues. I ran take 1,000 times in a loop and also ran the pull request, then develop, then the pull request again.
group                                      develop               pr                      pr2
-----                                      -------               --                      ---
patched_take_10K_contiguous_not_patches    1.00     44.1±0.19ms  1.03     45.3±0.53ms    1.01     44.5±0.67ms
patched_take_10K_contiguous_patches        1.02     82.7±3.97ms  1.07     86.7±3.64ms    1.00     81.0±1.42ms

diff --git a/encodings/fastlanes/benches/bitpacking_take.rs b/encodings/fastlanes/benches/bitpacking_take.rs
index 6706c9564..caabb2431 100644
--- a/encodings/fastlanes/benches/bitpacking_take.rs
+++ b/encodings/fastlanes/benches/bitpacking_take.rs
@@ -114,7 +114,11 @@ fn bench_patched_take(c: &mut Criterion) {
         .collect_vec()
         .into();
     c.bench_function("patched_take_10K_contiguous_not_patches", |b| {
-        b.iter(|| black_box(take(packed.as_ref(), not_patch_indices.as_ref()).unwrap()));
+        b.iter(|| {
+            for _ in 0..1000 {
+                black_box(take(packed.as_ref(), not_patch_indices.as_ref()).unwrap());
+            }
+        });
     });

     let patch_indices: PrimitiveArray = (big_base2..big_base2 + num_exceptions)
@@ -123,7 +127,11 @@ fn bench_patched_take(c: &mut Criterion) {
         .collect_vec()
         .into();
     c.bench_function("patched_take_10K_contiguous_patches", |b| {
-        b.iter(|| black_box(take(packed.as_ref(), patch_indices.as_ref()).unwrap()));
+        b.iter(|| {
+            for _ in 0..1000 {
+                black_box(take(packed.as_ref(), patch_indices.as_ref()).unwrap());
+            }
+        });
     });

     let lots_of_indices: PrimitiveArray = (0..200_000)

For posterity, here's why this PR appeared to slow things down slightly. The latest changes actually notably reduce take time when the number of patches is large.
This PR modified BitPackedEncoding::take to use PrimitiveArray::patch(Patches) instead of implementing patching itself. PrimitiveArray::patch, although it took an owned Patches, called indices() and values() and then cloned them, thus incrementing the reference count to the underlying buffers. When we called into_maybe_null_slice, we (always) forced a copy of these buffers. This PR avoids that by using into_parts to get the indices and value of a Patches.
Critcmp shows patched_take_10K_contiguous_not_patches as a little slower, but if you look carefully the error bands are overlapping (develop could be as high as 56, pr could be as low as 53). In both develop and this PR, we call patches.take(&indices.to_array()). In both cases, for this test, take should return None and we should skip the part of the code that differs.
group                                      develop               pr
-----                                      -------               --
patched_take_10K_adversarial               1.13    101.8±3.21µs  1.00     90.2±7.68µs
patched_take_10K_contiguous_not_patches    1.00     50.0±6.08µs  1.07     53.7±3.79µs
patched_take_10K_contiguous_patches        1.02     98.3±2.48µs  1.00     96.1±6.77µs
patched_take_10K_random                    1.02    153.0±5.64µs  1.00    150.3±4.64µs
patched_take_10_contiguous                 1.06   983.4±18.63ns  1.00   928.3±17.56ns
patched_take_10_stratified                 1.00    919.9±7.37ns  1.01   928.8±10.46ns
patched_take_200K_dispersed                1.22   480.9±24.74µs  1.00   394.4±14.78µs
patched_take_200K_first_chunk_only         1.25   377.9±32.04µs  1.00   302.5±19.84µs
take_10K_contiguous                        1.03     11.3±0.02µs  1.00     11.0±0.17µs
take_10K_random                            1.00     58.9±0.27µs  1.00     59.0±0.49µs
take_10_contiguous                         1.00    396.0±8.07ns  1.02    405.1±4.54ns
take_10_stratified                         1.06   400.2±12.41ns  1.00   378.6±10.37ns
take_200K_dispersed                        1.25   435.3±22.05µs  1.00    348.7±7.17µs
take_200K_first_chunk_only                 1.32   321.2±12.22µs  1.00    242.6±6.01µs


Fix eclipse plugin bug which remove javabuilder from .project

The current eclipse plugin replace javabuilder with scalabuilder if scala-plugin is enabled.
This patch fix eclipse plugin not to replace but to add scalabuilder.
 In order to merge this change, we would need some test coverage to verify it's effect. Is that something you'd be interested in helping out with?
Here is some existing test coverage that could be extended:
https://github.com/gradle/gradle/blob/master/subprojects/ide/src/test/groovy/org/gradle/plugins/ide/eclipse/EclipsePluginTest.groovy#L43
https://github.com/gradle/gradle/blob/master/subprojects/ide/src/integTest/groovy/org/gradle/plugins/ide/eclipse/EclipseIntegrationTest.groovy#L60-L62

I think these tests don't have any appropriate test cases which for this pull request. So, I added some test cases.

Question:
When I was on a client project about 3 years ago which had a combination of traditional Java, Scala, and Groovy, I manually did something like this on the project then to try and get things working better in eclipse.
I believe at the time, Eclipse was the Indigo version.
However, at that time, I found that the Java and Scala builders did not "get along" at that time, almost always causing mass compilation failures which seemed to be a by product of one builder clobbering the other's output, with usually the javabuilder "winning"
Has the behavior in the builders changed to where they "get along" better now?
In this particular case of the client I was working with, Scala was eventually dropped to where the problem resolved itself in that fashion.

@kemuri-9
ummm... I tried Scala-IDE with this patch:

Did not clobber other builder
Occur scala code syntax error

I did not notice this problem until your question, because I write Scala code with Emacs and did not use Scala-IDE.
So, I want to drop this pull request.
I think the eclipse plugin problem is using Scala-IDE components even if it is not installed. Java code don't be built on a eclipse project which be created by current gradle and does not install Scala-IDE.