Polish Spring Boota auto-configuration

@garethahealy This issue can close because fix already via #663.


Fix the snackbar

Code Coverage

File
Coverage [0.00%]
:x:

app/src/main/java/com/example/myapplication/recipes/presentation/editrecipe/AddEditRecipeScreen.kt
0.00%
:x:

Total Project Coverage
3.31%
:x:


⚠ Bofrost.fr has degraded performance

In 7f12b61, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1448 ms

Resolved: Bofrost.fr performance has improved in 08e5767 after 8 minutes.


NumPy mathematical functions

There is a list at https://numpy.org/doc/stable/reference/routines.math.html

~Trigonometric functions~
~Hyperbolic functions~
[ ] Rounding (should be straightforward in many cases)

[ ] round, around, rint, fix, floor, ceil, trunc (should probably return NumPy arrays? Skip digitsfor (a)round? Or just an alias for cast for fixed-point?)

[ ] Sums, products, differences (most should be possible)

[ ] prod
[ ] sum
[ ] nanprod
[ ] nansum
[ ] cumprod
[ ] cumsum
[ ] nancumprod
[ ] nancumsum
[ ] diff
[ ] ediff1d
[ ] gradient ? Intermediate word length?
[ ] cross
[ ] trapz ? Intermediate word length?

~Exponents and logarithms~
~Other special functions~
[ ] Floating point routines (most)

[ ] signbit (return NumPy array)
[ ] copysign
[ ] frexp (return NumPy arrays of int32 and int64?)
[ ] ~ldexp~
[ ] nextafter (use next_up() and next_down())
[ ] spacing?

[ ] Rational routines (should be possible?)

[ ] lcm
[ ] gcd

[ ] Arithmetic operations (most can be implemented using +, -, etc)

[ ] add
[ ] reciprocal (Word length for fixed-point?)
[ ] positive
[ ] negative
[ ] multiply
[ ] divide
[ ] power
[ ] subtract
[ ] true_divide
[ ] floor_divide
[ ] ~float_power~
[ ] fmod ?
[ ] mod
[ ] modf (Return format?)
[ ] remainder
[ ] divmod

Handling complex numbers (depends on #392)

[ ] angle Return NumPy float array?
[ ] real
[ ] imag
[ ] conj / conjugate

[ ] Extrema Finding (most)

[ ] maximum
[ ] max/amax
[ ] fmax
[ ] nanmax
[ ] minimum
[ ] min/amin
[ ] fmin
[ ] nanmin

[ ] Miscellaneous (some)

[x] convolve
[ ] clip
[ ] ~sqrt~ ?
[ ] ~qbrt~ ?
[ ] square
[ ] absolute / abs (word length for complex numbers?, sqrt needed then)
[ ] fabs
[ ] sign (return NumPy array)
[ ] heaviside
[ ] nan_to_num
[ ] ~real_if_close~ ?
[ ] ~interp~ ? (Word length?)

At least a trivial version should be possible for many, like square => *this * (*this) etc.
One may even consider if it is possible to have a common Array parent class for these and some other ones?
I think that there maybe should be both apytypes.square(a) and a.square(), where possibly the former is just a Python wrapper for the latter.

Support for convolutions was added in #462.


Signing in to mastadon.social error

When you try to sign in to mastadon.social, it has suddenly started throwing an error:
[TootSDK bug] There was an error decoding incoming data: context: Context(codingPath: [], debugDescription: "The given data was not valid JSON.", underlyingError: Optional(Error Domain=NSCocoaErrorDomain Code=3840 "Invalid value around line 1, column 0." UserInfo={NSDebugDescription=Invalid value around line 1, column 0., NSJSONSerializationErrorIndex=0})).
Other servers don't seem to have this problem.

Looks like this was temporary so presumably a server issue.
Perhaps these could pop an error message, currently it just silently fails.

Thank you for reporting this!
The message [TootSDK bug] There was an error decoding incoming data... is printed when a network request succeeds (HTTP status code is in the 2xx range) but the response body is not formatted correctly/does not contain a correct payload.
We will need additional information about the server response to improve the error message. If you encounter this again, could you please include the traces from TootClient after calling client.debugOn()? This will include several lines starting with the ⬅ emojis that include all headers and the raw body response from the server. Thank you 🙌!


uiux(client): add user avatar next to user name in lab cards

Add the small avatar image next to the name of the author of a lab.

Oh and one more thing @andrei-antal
Can you please update the commit message to also say
Fixes: #744
This will then automatically close the issue once this is merged.

@PascalPrecht I managed to add unit tests for the component to for the new behaviour. So, no more nested <a>s, the redirection works the correct way and card area is (mostly) clickable and routing. When you have a bit of time, have a look and let me know if everything is ok. Cheers!

Okay I think this looks good. There's only one tiny thing I'd ask you to do. Right now this PR introduces 6 commits, of which some are fixing things of previous commits of the same PR.
At the end what we want is a commit that introduces the avatar header (including tests). And another one that fixes the link situation.
Can you rebase this PR? I'm happy to help with this.


Spring Controls

This control mixes a graph element with two+ sliders for spring parameters.

From Principle

https://codesandbox.io/s/young-currying-zs4w1


use vfork() instead of fork()

Hi, I noticed this uses fork() (through tiny-process-library) and thought I'd let you know about this:
https://github.com/surge-synthesizer/surge/issues/657#issuecomment-466800504
It's not recommended to use fork() for plugins

Ohh, I hadn't thought about that. Yeah child processes (especially with the simplified utility library I make use of) would definitely involve fork() and that'd be problematic. Thanks, that issue is going to be an infinite but fun reading.
Since it is called as part of the dependency I would have to fork and rewrite that part. I will work on my spare time.
Let me shoot a quick question though; was there any useful tool to detect problematic plugin behavior that lets you find this issue?

awesome!

was there any useful tool to detect problematic plugin behavior that lets you find this issue?

There's lv2lint that detects a bunch of problems but I'm not sure if it also detects use of fork().

Ah, I thought you used some tool to find the issue. Maybe it was just caught by your debugger I guess then. Thanks for the hint anyways!

I have made a dirty hack to locally patch tiny-process-library instead of creating a fork for this vfork thing. 0255c8b contains a patching step in build.sh.
The resulting binary seems to load on zrythm and qtractor, but (1) I have trouble on enabling jackd on my Ubuntu 20.04 laptop so any DAWs that requires jackd can verify only UI part, and (2) zrythm on develop branch somehow goes into infinite crash report dialog loop with or without this fix. The stack trace is here. Maybe it is debuggable locally, but that's going to be another investigation.
Anyhow I'm closing this issue now. Thanks for the report!

:+1: thanks for the stack trace, will fix it soon. I made some internal changes and there's many regressions at the moment

Oh you had an idea even from that log? That's cool. Let me know if you would like further details on it :)

yes those are very helpful! please submit them whenever they pop up (no registration needed) :slightly_smiling_face:


fix: event naming to prefix subi connect

#11  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @keeganpotgieter and the rest of your teammates on  Graphite


Run Adbalance experiment(s) again

Run when more behind-the-adverts are designed and in place
Needs #395

Started 1/4/2020 - review in 2 weeks.

Autochose 63%, but that only gets 97% of income now, so have reset experiment to 75% - predicts 100% income.

Google has stopped AdBalance so nothing to experiement with


Update pytest and pytest-cov

This was supposed to remove py as a dependency and avoid a security alert that isn't actually a problem, but locust still depends on it through pyzmq.

I'm not concerned about the issue (we don't use svn, which means we'll never hit the problematic code), but updating pytest is an easy way to make it go away... unless something else also depends on it. We can tell dependabot to ignore it in this repo.


Joe Cosenza Homework 3 MuteIds

I added two files, the app, get_mute_ids.rb and the request MuteIDs which returns an array of ID's of users muted for the Twitter account.

Looks good. Thank you!


Support FlexBox Layout

Original discussion: https://github.com/eclipsesource/tabris-js/issues/2101

We will not remove the flex  prefixes for the container properties after all.

width, height and flexBasis will be rendered as tuple [percent, 0] only if it's percent in layoutData, otherwise still number or 'auto'.


Support CSS pseudo classes without selector

Thanks a lot for your minifier!
If I am correctly informed using pseudo classes without selectors is valid CSS code, e.g. :is().
Yet, minify-html produces wrong code breaking the whole CSS file if the asterisk is left out.
/* Original CSS (without `*` selector) */
:is(nav, .posts) a {
	text-decoration: none;
}

/* Minified CSS (error) */
:is(nav{text-decoration:none}

/* Original CSS (with `*` selector) */
:is(nav, .posts) a {
	text-decoration: none;
}

/* Generated CSS (correct) */
*:is(nav,.posts) a{text-decoration:none}

Thanks for raising, minify-html uses css-minify, I haven't investigated the example yet but it might be worth raising with the upstream as well.

It looks like the issue was resolved upstream, so I've bumped the version of css-minify, and the issue should be resolved in the next release.


The development environment is normal, the production environment is blank.

What version of Hugo are you using (hugo version)?
When can the official Docker image be released? Currently, the Kalkegg's Docker Hub image is one year old, and I am using the one in my NAS; however, the image is at version 0.129.0 in Docker Hub. Since using the Docker Hub image:
Windows 11: Running hugo server through the command on the taskbar and going to localhost:1313 works normally.
My NAS: Going to the IP address of the NAS through 1313: and it shows a blank screen with nothing (Every time I update the content on the NAS, the hugo Docker container is restarted.)
Problem:
Running hugo server through the command on the taskbar works on Windows 11.
When going to localhost:1313 on the NAS, it shows a blank screen with nothing (Every time I update the content on the NAS, the hugo Docker container is restarted.)

Please use https://discourse.gohugo.io/ for questions/troubleshooting. Also see  Hugo Documentation.


Set an Optimizely dimension for auth link clicks.

What's this PR do?
This pull request adds a lil' snippet of code to set a custom Optimizely Dimension whenever users click on either the "Log In" or "Sign Up" links on the page.
This will allow us to better track differences in the conversion rate of the new auth flow, since otherwise we can only track the overall conversion rate of all users that see the modified links, rather than just those that click them.
How should this be reviewed?
👀
Any background context you want to provide?
📊
Relevant tickets
Fixes 🍃.
Checklist

[ ] Documentation added for new features/changed endpoints.
[ ] Tested on staging.
[ ] Pinged a PM if this is a larger PR that would benefit from some additional testing love
[ ] Post a message in #phoenix if this includes something that causes a rebuild!

 (2) Add a custom dimension to better segment Optimizely login test results.


BUG: mne.make_field_map indicate wrong origin parameter

There's a mismatch between docs and actual default parameter.
I'm not sure what the param should be.
cc @Eric89GXL

We usually try to use 'auto', so we should probably change it to that as a bugfix

does auto take the center of the electrodes or of a sphere fit to the head ?
On 16 May 2017 at 09:29, Eric Larson<EMAIL_ADDRESS>wrote:

We usually try to use 'auto', so we should probably change it to that as
a bugfix
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/4268#issuecomment-301782110,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEp7DHSgNE1IZVBWeS2EpwqM2Tw7I25Yks5r6aScgaJpZM4NbxI1
.

sphere fit to the head digitization

I started to make a fix by changing the default parameter value, but it
creates a series of error in the tutorials and in the tests, so I'll let
you guys deal with it
On 16 May 2017 at 09:38, Eric Larson<EMAIL_ADDRESS>wrote:

sphere fit to the head digitization
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/4268#issuecomment-301784704,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEp7DGSYWMIxPWVRxzz2JuBxtkCn7_vTks5r6abAgaJpZM4NbxI1
.

For now we can just fix the documentation to reflect the actual default then.

(0, 0, 0.04) is a very weird default parameter IMO
On 16 May 2017 at 11:02, Eric Larson<EMAIL_ADDRESS>wrote:

For now we can just fix the documentation to reflect the actual default
then.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/4268#issuecomment-301810728,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEp7DCaFoJm1xPjASrBhSXxykMhllTHxks5r6bp2gaJpZM4NbxI1
.

it is approximately where the origin fit usually ends up for adult subjects

yes, I had guessed, but this kind of numerical guess can lead to very
weird-looking results, without a clear warning, so I would recommend doing
either a real fix, or just explicitly use these numbers in the tests and
tutorials that crash with the 'auto' parameter.
On 16 May 2017 at 11:24, Eric Larson<EMAIL_ADDRESS>wrote:

it is approximately where the origin fit usually ends up for adult subjects
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/4268#issuecomment-301817969,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEp7DEegGQz-TK6Xd6fPte-SCZX_i6Leks5r6b-ugaJpZM4NbxI1
.


Update app version to 1908

Update the version of the app to 1908.

Is there an appx? ☺

Is there an appx? ☺

There's an appxbundle available with every PR and every merge into master.  To find it, just click on the "Checks" tab of the PR and then click on "View more details on Azure Pipelines".  From there, click on the blue "Artifacts" button in the top-right and choose "appxbundle".
For this PR, here's the direct link.

Thank you so much Howard!

@Poopooracoocoo Instructions from https://github.com/microsoft/calculator/issues/543#issuecomment-505056530 should still work.


Compression

Some very quick benchmarks on Production data using Facebook's zstd compression suggests that disk space used by chunks can be reduced by a factor of 2.5x and index files reduced by at least 3.x.
When storing petabytes of TSDB data (using 3x replication or erasure coding) that may then be backed up, these kinds of savings are worth pursuing. Even if the object store or the filesystem it uses supports compression, the network bandwidth saved is also significant.
Ideally all compression, depending on the overhead, would be in the TSDB library itself (e.g. https://github.com/prometheus/tsdb/issues/249), though Thanos use case is distinct enough that we might consider compressing blocks on upload/download to and from the object store. Opening this ticket to track potential options.

The question is do we need this. Petabytes of TSDB data is... a lot (: If I am counting correctly,
even terabyte is not achievable for most companies - even for over 1yr data, including redundancy of downsampled data and HA scrapers

Thanks for investigating this!
I noticed that by its nature our chunk compression has surprisingly many reptitive patterns a while ago. But I didn't expect another layer of compression being so effective.
The problem is of course, that once we compress entire files, we can no longer randomly access them, which is what Thanos is build on and how it can achieve good query latencies even against object storage.
So for pure archival, this would all be fine. But I don't see a straightforward way to do both. One could break up files into smaller chunks (not the same our normal chunks) again that are then compressed and always fetched if a requested range falls into them.
But we'd need to keep a mapping of what actual range map to which compressed chunks.
For the index file specifically it would get extra hairy.
While the theoretical improvement is great, I believe overall we are doing extremely well in terms of data size compared to existing alternatives.
So I doubt it's worth pursuing something so complex at this point – if it is possible without deal breaker tradeoffs at all.

I think another main problem for pursing something like this is, that we have no clear query performance profile of TSDBs current state even. We know it's generally doing well, but lack data on what impacts ultimate query latency.
That's owed one the one hand to the fully deferred evaluation and OTOH on mmap, which also causes actual reads to be deferred.
So if we added something like the above, we'd have a hard time judging how it shifts performance around.

I took the first chunk (000001) and the index file from a compacted block (spanning 18 hours) written by Prometheus 2.1.2.
I'm not suggesting zstd is the best option here, it's just the first thing I tried.
Block meta:
{
	"ulid": "01C9QVCDKFW2070XRCMAMTYA9T",
	"minTime":<PHONE_NUMBER>000,
	"maxTime":<PHONE_NUMBER>000,
	"stats": {
		"numSamples":<PHONE_NUMBER>,
		"numSeries": 3958306,
		"numChunks": 31660562
	},
	"compaction": {
		"level": 3,
		"sources": [
			"01C9NXBPM8NDJZZDHZY4EXXHP5",
			"01C9P47DW4QEXHX3RXAKT8GD9D",
			"01C9PB35489N3W251ZHA7TN0QH",
			"01C9PHYWC4EBH5HAFRM7EHEMQ7",
			"01C9PRTKM5KT51535RYHQVPZG1",
			"01C9PZPAW5J0R7KNDV0RWRZV9W",
			"01C9Q6J247MFAHRV6RTQ1SFGA1",
			"01C9QDDSC592YWVJYY8PJND9BD",
			"01C9QM9GM69VDBDXJJXX6X7JSA"
		]
	},
	"version": 1
}

Chunk
$ ls -alh 000001
-rw-r--r--  1 matt  staff   512M 12 Apr 11:46 000001

# Test ran on my laptop (Intel(R) Core(TM) i7-5557U CPU @ 3.10GHz), while doing other things
# Compression ratio is in parentheses, followed by compression rate, then decompression rate
$ zstd -b1 -e22 000001 -o /dev/null
Benchmarking levels from 1 to 22
 1#000001            : 536864556 -> 275802287 (1.947), 249.4 MB/s , 601.9 MB/s
 2#000001            : 536864556 -> 244681820 (2.194), 174.0 MB/s , 563.4 MB/s
 3#000001            : 536864556 -> 230086353 (2.333), 127.3 MB/s , 581.1 MB/s
 4#000001            : 536864556 -> 225878032 (2.377), 113.8 MB/s , 573.9 MB/s
 5#000001            : 536864556 -> 224305136 (2.393),  56.6 MB/s , 541.6 MB/s
 6#000001            : 536864556 -> 217657382 (2.467),  31.5 MB/s , 570.8 MB/s
 7#000001            : 536864556 -> 213948842 (2.509),  21.8 MB/s , 538.7 MB/s
 8#000001            : 536864556 -> 211854560 (2.534),  19.2 MB/s , 534.8 MB/s
 9#000001            : 536864556 -> 210459603 (2.551),  15.2 MB/s , 568.1 MB/s
10#000001            : 536864556 -> 208035011 (2.581),  11.6 MB/s , 566.7 MB/s
11#000001            : 536864556 -> 204515614 (2.625),  9.46 MB/s , 474.7 MB/s
12#000001            : 536864556 -> 202896105 (2.646),  7.23 MB/s , 551.1 MB/s
13#000001            : 536864556 -> 199168307 (2.696),  5.57 MB/s , 508.0 MB/s
14#000001            : 536864556 -> 197814090 (2.714),  4.80 MB/s , 529.0 MB/s
15#000001            : 536864556 -> 194967240 (2.754),  3.38 MB/s , 489.1 MB/s
16#000001            : 536864556 -> 190168655 (2.823),  3.85 MB/s , 601.0 MB/s
17#000001            : 536864556 -> 186347635 (2.881),  3.20 MB/s , 580.0 MB/s
18#000001            : 536864556 -> 183627299 (2.924),  2.71 MB/s , 589.3 MB/s
19#000001            : 536864556 -> 181573064 (2.957),  2.37 MB/s , 550.7 MB/s
20#000001            : 536864556 -> 174809487 (3.071),  1.74 MB/s , 426.4 MB/s
21#000001            : 536864556 -> 173167400 (3.100),  1.57 MB/s , 416.0 MB/s
22#000001            : 536864556 -> 171953306 (3.122),  1.44 MB/s , 413.0 MB/s

Index
$ ls -alh index
-rw-r--r--  1 matt  staff   1.5G 12 Apr 11:06 index

$ zstd -b1 -e22 index -o /dev/null
Benchmarking levels from 1 to 22
 1#index             :1579860816 -> 580201924 (2.723), 296.5 MB/s , 657.5 MB/s
 2#index             :1579860816 -> 578631639 (2.730), 292.6 MB/s , 615.9 MB/s
 3#index             :1579860816 -> 562291573 (2.810), 211.4 MB/s , 619.5 MB/s
 4#index             :1579860816 -> 557336759 (2.835), 145.9 MB/s , 504.5 MB/s
 5#index             :1579860816 -> 552620476 (2.859), 103.4 MB/s , 510.4 MB/s
 6#index             :1579860816 -> 529761565 (2.982),  67.8 MB/s , 677.3 MB/s
 7#index             :1579860816 -> 527049052 (2.998),  47.3 MB/s , 514.3 MB/s
 8#index             :1579860816 -> 518433934 (3.047),  41.2 MB/s , 734.3 MB/s
 9#index             :1579860816 -> 517052328 (3.056),  36.8 MB/s , 682.3 MB/s
10#index             :1579860816 -> 514818517 (3.069),  31.2 MB/s , 764.1 MB/s
11#index             :1579860816 -> 502642881 (3.143),  25.9 MB/s , 748.6 MB/s
12#index             :1579860816 -> 502087438 (3.147),  14.5 MB/s , 720.8 MB/s
13#index             :1579860816 -> 501217631 (3.152),  11.3 MB/s , 740.6 MB/s
14#index             :1579860816 -> 500456004 (3.157),  7.63 MB/s , 569.1 MB/s
15#index             :1579860816 -> 498248964 (3.171),  5.82 MB/s , 644.5 MB/s
16#index             :1579860816 -> 478442609 (3.302),  4.17 MB/s , 723.8 MB/s
^C-index             :1579860816 ->

Sorry @fabxc I should have clarified in my last comment, I'm not clear as to why the index file is more difficult to compress.
For example, I'm thinking the Thanos shipper compresses the index file on upload, and the store instance decompresses it on download. As far as I know, the index file is not subject to range requests on the object storage.
I suspect the index file compresses so well due to the label names and values. As a side note, I'm seeing up to 5.6x compression ratios on this same index file using xz, though that's an extreme example and may be too slow to be practical (I'm thinking more about backups where xz is concerned).

My mistake, Thanos does perform range queries on index files.

I suspect the index file compresses so well due to the label names and values.

Strings are actually fully deduplicated already, i.e. only one copy exists of each. They may have partial overlaps of course (e.g. instance names) but in total only account for a few dozen kilobytes of the file.
Of course elsewhere we reference those strings. So the references are quite repetitive. There's an upstream issue to give smaller reference values to more frequent strings.

One way round this would be to pre-fetch the index files in full, but that would make restarting the store nodes an expensive operation (if the files were not persisted locally) and would delay queries until the indices are fully loaded.

Yes, that was actually the initial design because intuitively range queries against index files seemed to complex. Turned out it's actually pretty okay and fast.
Storing the whole index file indeed makes restarts (with a clean disk very slow) but also would mean you'd need a lot of store nodes or really huge disks for them pretty quickly. It feels much more preferrable to have store nodes be bounded by query load they experience rather than amount of data they provide access to.

Should we close this? It seems any further file sizes gains would be best implemented in the TSDB library directly (and better tracked in that repository).

SGTM. Adding more custom complexity to Thanos will probably make it hard to stay compatible in the long run.


Adds vendored Nix packaging.

Adds Nix(https://nixos.org) packaging.
This enables various approaches related to reproducibility, deployment, ease of use - see for instance this one-liner for running one of your provided notebooks from any machine.
nix-shell https://github.com/freuk/SMPyBandits/archive/master.tar.gz --run 'jupyter-notebook $EXAMPLE_NOTEBOOKS/Example_of_a_small_Multi-Player_Simulation__with_Centralized_Algorithms.ipynb'

This one runs fine for me, provided I start execution at cell number 2.

Could you add a little bit of documentation, somewhere in the Markdown files, maybe in How_to_run_the_code.md and this section of the README.md ?
Otherwise this new feature will be left undocumented, and nobody will be able to take advantage of it!

done, this was tested locally using:

Notebook Example_of_a_small_Multi-Player_Simulation__with_Centralized_Algorithms.ipynb cells 2 to end.
Makefile target N=100 T=10000 K=9 N_JOBS=4 make single.

FYI the more makefile target does not exist on the latest master branch. a+


Particle World does not render properly

Hi, When I try to use the Particle World function in AE, the animation does not render properly, only the reference layer is shown. Static layers are fine though. Can Bodymovin render particles?

Yes, effects are not supported by the plugin.
Particles are too expensive on performance for real time rendering.
sorry :/


Besu Operator

Short Description
Kubernetes operator to provide an easier way of packaging, deploying, and managing Hyperledger Besu network.
Part of Hyperledger mentorship program : Wiki Page
Signed-off-by: Sumaid Syed<EMAIL_ADDRESS>

+1 proposal lgtm
Can you ask Mark Wagner to confirm sponsorship here ?

I approve this as a sponsor.

Moved besu-operator.md to labs folder in #140
Created Repo: https://github.com/hyperledger-labs/besu-operator
Sent invite to @Sumaid and @n1zyz


Add hooks for Recorded test Request/Response Body manipulation

Most of the Storage tests upload random data and we don't want to add 1GB of noise in the recordings that everyone has to sync.  It would be great if we could selectively exclude/generate/etc. the request/response bodies of those recordings.

Hi @tg-msft, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.


Experimental Alfresco Docker Extension

Deploy Alfresco using a single "Run" button.

Thanks for your contribution @aborroy


Question: Is it possble to pass arguments when starting the process?

I am trying out pywinauto (coming from another ui automation framework https://github.com/Roemer/FlaUI).
I am trying to automate a windows program that accepts some arguments (from the command line).
Does pywinauto support passing arguments when starting the process?
If answer is no, is there a reason or it?
Regards Fredrik

You can specify command line arguments in the cmd_line parameter of Application.start() like so:
app = pywinauto.application.Application()
app.start(cmd_line=r"c:\myprogram.exe -args")

Thanks!


let g:indentLine_fileType  not work in macvim8.0

I have set let g:indentLine_fileType =['js', 'ts']  ,but the correspond type file does not have indentline , the set is not work. I don't know why. Does anyboy have the same problem?

Please refer to issue #151 and #59 .


[Bug]: Doesn't seem to work?

Neovim version
0.9.1
Language affected
all I guess
Query
No response
Strategy
No response
Description
- Install [kickstart.nvim](https://github.com/nvim-lua/kickstart.nvim)
- Add `rainbow-delimiters.nvim` to `nvim-treesitter`'s dependencies like:

 {
    -- Highlight, edit, and navigate code
    'nvim-treesitter/nvim-treesitter',
    dependencies = {
      'nvim-treesitter/nvim-treesitter-textobjects',
      'HiPhish/rainbow-delimiters.nvim'
    },
    build = ':TSUpdate',
  },

- See if it works ... which it doesn't in my case

The health checks also says:

`- ERROR The healthcheck report for "rainbow-delimiters" plugin is empty.`

You could be experiencing the same problem as #2.  Does highlighting work if you :write and then :edit the file?
As for the health check, if you don't have any custom configuration (which you don't need to have) there is nothing to check, so the health report will be empty. Perhaps I should add an explicit message for that case.

#2 seems like a different problem as it was present in nvim-ts-rainbow2 too (which worked totally fine here). I recently decided to switch to rainbow-delimiters as I was getting some query errors in nvim-ts-rainbow2 (maybe because of outdated API calls?).
Does the repro works on your machine? Or it's working fine on yours?

Can you please update to the current master, then set the log level to TRACE in your settings, then post the contents of the log file here? See :h rb-delimiters-logging for information on how to adjust the log level.
vim.g.rainbow_delimiters = {
    log = {
        level = vim.log.levels.TRACE
    }
}

For tsx file:
2023-07-22T21:05+0530	ERROR		Error attaching strategy to buffer 1: /usr/share/nvim/runtime/lua/vim/treesitter/query.lua:259: query: invalid structure at position 1394 for language tsx
2023-07-22T21:05+0530	ERROR		Error attaching strategy to buffer 1: /usr/share/nvim/runtime/lua/vim/treesitter/query.lua:259: query: invalid structure at position 1394 for language tsx
2023-07-22T21:05+0530	ERROR		Error attaching strategy to buffer 1: /usr/share/nvim/runtime/lua/vim/treesitter/query.lua:259: query: invalid structure at position 1394 for language tsx

For jsx file:
2023-07-22T21:09+0530	ERROR		Error attaching strategy to buffer 1: /usr/share/nvim/runtime/lua/vim/treesitter/query.lua:259: query: invalid structure at position 1065 for language javascript
2023-07-22T21:09+0530	ERROR		Error attaching strategy to buffer 1: /usr/share/nvim/runtime/lua/vim/treesitter/query.lua:259: query: invalid structure at position 1065 for language javascript

Interestingly, it started working for ts, html (and maybe other) files, but not js files.

Oops, I had a typo in my logger logic. Can you please update again and post the news logs?
The error message indicates that something is wrong with the query. Please run :TSUpdate to updated all parsers, then try running the following code:
local q = vim.fn.join(vim.fn.readfile('path/to/plugins/rainbow-delimiters.nvim/queries/javascript/rainbow-delimiters-react.scm'), '\n')
local query = vim._ts_parse_query('javascript', s)
print(query)

The first line loads the text of the query from file. The second line call the function which fails in your error message. If everything went correctly the last line should print something like <userdata 1>.

Sorry for the delay, was busy with work.
When running those commands it fails at the vim._ts_parse_query('javascript', q) function with error:
E5108: Error executing lua [string ":lua"]:1: no such language: javascript
stack traceback:
        [C]: in function '_ts_parse_query'
        [string ":lua"]:1: in main chunk

Which is weird as I made sure that the parser for javascript, tsx and typescript are properly installed (and yes I even re-installed them just to be sure).
Running the commands with tsx as the language gives me previously reported error:
E5108: Error executing lua [string ":lua"]:1: query: invalid structure at posi
tion 1065 for language tsx
stack traceback:
        [C]: in function '_ts_parse_query'
        [string ":lua"]:1: in main chunk

And here are the logs:
2023-07-24T19:34+0530	TRACE		Define highlight groups
2023-07-24T19:34+0530	TRACE		Attaching to buffer 1 with language tsx.
2023-07-24T19:34+0530	TRACE	strategy.global	global strategy on_attach
2023-07-24T19:34+0530	TRACE	strategy.global	Set up parser for buffer 1
2023-07-24T19:34+0530	TRACE	strategy.global	Set up parser for tsx
2023-07-24T19:34+0530	TRACE		Cannot attach to buffer 2, no parser for nil
2023-07-24T19:34+0530	TRACE		Cannot attach to buffer 3, no parser for nil
2023-07-24T19:34+0530	TRACE		Detaching from buffer 1.
2023-07-24T19:34+0530	TRACE		Attaching to buffer 1 with language tsx.
2023-07-24T19:34+0530	TRACE	strategy.global	global strategy on_attach
2023-07-24T19:34+0530	TRACE	strategy.global	Set up parser for buffer 1
2023-07-24T19:34+0530	TRACE	strategy.global	Set up parser for tsx
2023-07-24T19:34+0530	TRACE		Detaching from buffer 1.
2023-07-24T19:34+0530	TRACE		Attaching to buffer 1 with language tsx.
2023-07-24T19:34+0530	TRACE	strategy.global	global strategy on_attach
2023-07-24T19:34+0530	TRACE	strategy.global	Set up parser for buffer 1
2023-07-24T19:34+0530	TRACE	strategy.global	Set up parser for tsx
2023-07-24T19:35+0530	TRACE		Detaching from buffer 3.
2023-07-24T19:35+0530	TRACE		Detaching from buffer 2.
2023-07-24T19:42+0530	TRACE		Detaching from buffer 1.

What do you get if you try a language that does work for you? (Remember that you have to adjust the path to the query as well, otherwise you will get an error.)

for css, running:
local q = vim.fn.join(vim.fn.readfile('/home/schardev/.local/share/nvim/lazy/rai
nbow-delimiters.nvim/queries/css/rainbow-delimiters.scm'), '\n')
local query = vim._ts_parse_query('css', q)
print(query) -- returns `<query>`

and the logs says:
2023-07-25T16:44+0530	TRACE		Detaching from buffer 1.
2023-07-25T16:44+0530	TRACE		Attaching to buffer 1 with language scss.
2023-07-25T16:44+0530	TRACE	strategy.global	global strategy on_attach
2023-07-25T16:44+0530	TRACE	strategy.global	Set up parser for buffer 1
2023-07-25T16:44+0530	TRACE	strategy.global	Set up parser for scss
2023-07-25T16:44+0530	TRACE	lib	Query rainbow-delimiters found for scss
2023-07-25T16:44+0530	TRACE	lib	Query rainbow-delimiters found for scss

and the same for json too:
2023-07-25T16:51+0530	TRACE		Define highlight groups
2023-07-25T16:51+0530	TRACE		Detaching from buffer 1.
2023-07-25T16:51+0530	TRACE		Attaching to buffer 1 with language json.
2023-07-25T16:51+0530	TRACE	strategy.global	global strategy on_attach
2023-07-25T16:51+0530	TRACE	strategy.global	Set up parser for buffer 1
2023-07-25T16:51+0530	TRACE	strategy.global	Set up parser for json
2023-07-25T16:51+0530	TRACE	lib	Query rainbow-delimiters found for json
2023-07-25T16:51+0530	TRACE	lib	Query rainbow-delimiters found for json
2023-07-25T16:52+0530	TRACE		Detaching from buffer 2.
2023-07-25T16:52+0530	TRACE		Detaching from buffer 1.

So CSS and JSON are working correctly. Then the problem must be either with my query or with your Neovim, because by calling vim._ts_parse_query directly we are bypassing any of my logic. Here is an idea: open up the offending query file, comment everything out and then re-enable one pattern at a time until you find the offending pattern.

Please try the current master branch. It looks like I had an older version of the parser installed and at some point the tree structure changed. There is no way of supporting both patterns, so I'm going with the more recent one.

It works but the highlighting is a little bit off

Yes, that is unfortunate. Currently Neovim can only have one match per capture, so I can have only one opening and one closing node. The three nodes are <, NavArrowLeft and />. The closing one is obvious, but which one should be the opening node? I went with the tag name, it seemed the lest bad option to me.
Ideally both < and NavArrowLeft should be opening nodes, but I cannot do that yet.

Updating to master obviously fixed it. Just hoping that https://github.com/neovim/neovim/pull/17099 get's merged so we can have proper highlighting.
You can close the issue now. Thanks for the support :)


numpy fft not yet implemented

As a fairly fundamental operation I was surprised to find out this wasn't implemented.
NotImplementedError: Numpy function <function fft at 0x7fdb56275510> not yet implemented
In the short term is there a suggested workaround? I suppose I could use a full dft matrix.

We haven't implemented any FFT functionality yet. It's probably best that we just implement it. We're trying to be fairly user-driven in the things that we add.
Do you just need to compute FFTs (easy enough), or do you also need to differentiate through them?

Being able to differentiate through the fft and ifft would be amazing. In particular for end-to-end optimization of frequency-domain audio processing algorithms.

I would like to second this functionality, it will make a lot of signal processing users happy.

I started working on this earlier and got distracted by other things, I'll try to get something up for review soon!

Hi, sorry this took so long! I just added np.fft.fftn support, with some limitations (see https://github.com/google/jax/commit/5d1c014509d429001181ff7ffcfcb54bd6aa907d for details). Please comment here or open a new issue if you need extra functionality! I committed this as-is to get the basics in, but any follow-up work will be much easier.

Looks not too hard to add ifftn?

Do we have a guide or tutorial showing how to use FFTs with JAX? If no, I'll add a feature request - it would be a great add. 🙂

No tutorial yet AFAIK.

I added https://github.com/google/jax/issues/1877 for keeping track of all the other FFT functions.


Add feature gate WatchFromStorageWithoutResourceVersion for v1.30

FYI: Here is the corresponding upstream PR: https://github.com/kubernetes/kubernetes/pull/123935
/sig spi-machinery

FYI: Here is the corresponding upstream PR: https://github.com/kubernetes/kubernetes/pull/123935
/sig api-machinery

thank @tengqm for the PR.
/lgtm

/approve


🐛 BUG: native node bindings not correctly resolved

Which Cloudflare product(s) does this pertain to?
Wrangler core
What version(s) of the tool(s) are you using?
3.26.0
What version of Node are you using?
20.9.0
What operating system and version are you using?
MacOS Sonoma 14.2.1
Describe the Bug
Observed behavior
When installing a NPM package which includes native node bindings (for example bundled using https://napi.rs/), the package is not resolved correctly from the bundler used by Wrangler.
Expected behavior
The tooling is able to correctly resolve .node files from the bundler, to be used in the Workers runtime.
Steps to reproduce
I've pushed a sample repository which includes this issue: https://github.com/lorenzodejong/cloudflare-worker-native-binding. The specific package which is failing is @napi-rs/image.
Steps:

Clone the repository
Install the repository using pnpm install (or your preferred package manager)
Run wranger deploy
--> Wrangler throws an error (see below)

Please provide a link to a minimal reproduction
https://github.com/lorenzodejong/cloudflare-worker-native-binding
Please provide any relevant error logs
✘ [ERROR] No loader is configured for ".node" files: node_modules/.pnpm/@napi-rs+image-darwin-arm64<EMAIL_ADDRESS>node_modules/.pnpm/@napi-rs+image@1.7.0/node_modules/@napi-rs/image/index.js:141:36:
  141 │             nativeBinding = require('@napi-rs/image-darwin-arm64')
      ╵                                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

✘ [ERROR] Build failed with 1 error:
node_modules/.pnpm/@napi-rs+image@1.7.0/node_modules/@napi-rs/image/index.js:141:36: ERROR: No
loader is configured for ".node" files:
node_modules/.pnpm/@napi-rs+image-darwin-arm64<EMAIL_ADDRESS>

@lorenzodejong the best path forward here would be to ask the @napi-rs/image library authors to provide a Wasm version of that image as well as a node version. Then this should be usable by our runtime.
I'm going to close this as we do not have plans to support .node files at this time, and this isn't something we can fix at the Wrangler level.


Calendar view can not be scrolled when enable 2 pages

Thanks for giving the good library for ios.
I have found out some issue when i start using it,
As I have defined there are only the current month and next month page enable in the delegate method, but somehow the page can not be scrolled from the current page to the next.
 _minDate = [_calendarManager.dateHelper addToDate:_todayDate months:0];

_maxDate = [_calendarManager.dateHelper addToDate:_todayDate months:1];

But, it works fine when i enable 3 pages.
Can you help me found out whats the issue or give some suggestion how to handle my requirement.

Thanks, it's fixed.
But it's highly probable there is the same bug with different configuration (when you have only 2 pages).

Hi Jonathan
Thanks for your kindly help, it solve on my project.


Sign in sign out feature

Acceptance Criteria:
Add a sign in page to the app entry point which allows users to sign in with email + password, and add a sign out button in the home page.


Are xintstatus registers writable?

The numbers given to the xintstatus registers, 0x345, 0x145, and 0x045 are writable registers, but the description says all the fields are read-only.  Could that be made clearer in the spec?

These should be read only. - will change.

draft 20200423 lists xintstatus as:
(NEW) 0xm??   xintstatus   Current interrupt levels
This issue can be closed and tracked by issue #42: Address of xintstatus CSR
The address for xintstatus is listed as 0xm?? at the top of section 4. Can you please update this to reflect the correct address?

Please refer to issue #88.


Terminal split button should un-split back when clicked again

The issue:
When we click on the terminal split button " [ | ] " it splits the terminal window, but there is no way to unsplit it back to a single full window.
The feature I would like to have:
click on terminal split button => split the window.
click on it again => un-split the window back to full size.

That button does not split the window so much as it create two terminals in the same view? You can close out of any of these spawned terminals by clicking the trash can icon while the terminal is active.
What is the experience you were expecting?

mdgossa Yeah I see, it works great. Thanks for pointing this out.


Is it meant to be possible to etch without an inscription, or even a valid "ord" envelope?

I see several etchings on testnet which don't have associated inscriptions. For example:
https://testnet.ordinals.com/rune/ABANG•ABANG•ABANG
Instead of the usual:
FALSE
IF
"ord"
RUNE_TAG <rune_id>
...
ENDIF

envelope, it just has the following:
FALSE
IF
<rune_id>
ENDIF

Is that meant to be a valid etching? It doesn't have the "ord" magic string, but because the commitment pushes the rune_id it is accepted.

Yup, that's definitely allowed. The rune updater looks through all input witness instructions for a valid commitment, so it doesn't need to be in an envelope or inscription.
And the cheapest way include a commitment is PUSHDATA <rune_commitment> OP_DROP, since it saves two instructions over FALSE IF <rune_commitment> ENDIF.


Move content to https://github.com/EGI-Foundation/documentation

In order to ease documentation discovery content should be moved to https://github.com/EGI-Foundation/documentation.

Done!


[SPARK-25719] : Search functionality in datatables in stages page should search over formatted data rather than the raw data

The Pull Request to add datatables to stage page SPARK-21809 got merged. The search functionality in those datatables being a great improvement for searching through a large number of tasks, also performs search over the raw data rather than the formatted data displayed in the tables. It would be great if the search can happen for the formatted data as well.
What changes were proposed in this pull request?
Added code to enable searching over displayed data in tables e.g. searching on "165.7 MiB" or "0.3 ms" will now return the search results. Also, earlier we were missing search for two columns in the task table "Shuffle Read Bytes" as well as "Shuffle Remote Reads", which I have added here.
How was this patch tested?
Manual Tests

Test build #104762 has started for PR 24419 at commit a1e72f7.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

@abellina @vanzin @shahidki31

Have also added searching on Shuffle Read Bytes as well as Shuffle Remote Reads columns which was somehow missed out earlier.

Test build #104762 has finished for PR 24419 at commit a1e72f7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104762/
Test FAILed.

test this please

Test build #104764 has started for PR 24419 at commit a1e72f7.

Test build #104764 has finished for PR 24419 at commit a1e72f7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104764/
Test PASSed.

Test build #104813 has started for PR 24419 at commit a3a74f7.

Test build #104813 has finished for PR 24419 at commit a3a74f7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104813/
Test PASSed.

Test build #105294 has started for PR 24419 at commit 8822c82.

Test build #105294 has finished for PR 24419 at commit 8822c82.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105294/
Test PASSed.

Test build #105315 has started for PR 24419 at commit 35f78c2.

Test build #105315 has finished for PR 24419 at commit 35f78c2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105315/
Test PASSed.

Test build #105343 has started for PR 24419 at commit 35f78c2.

Test build #105343 has finished for PR 24419 at commit 35f78c2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105343/
Test PASSed.

Merged to master

Thank you @srowen and @abellina .


CHAOSS metrics

Have listed in wiki. Anything we want to have a final look at?

Thoughts:

Code change lines: could be an interesting marker to gain a better understanding of activity of the project. Our analysis so far has been more focused on how people interact with the project as contributors, disregarding information about the volume of their contributions.
Project popularity: implemented stars, forks, contributors, issues,... Looking at PRs specifically might be interesting, too (see #52).
Technical fork: related to #34.
Time to first response: would be interesting to test hypothesis about attracting new contributors. Would need to gather substantially more data though.

None of these seem very pressing at the moment. I created backlog issues for the two that we hadn't considered yet (#54 and #55).


Implement Admin Set Managers

When a user or group is added to the list of managers for an Admin Set, they should be able to:

See the Admin Sets UI with the Admin Sets to which they have access
Edit an Admin Set's descriptions, participants, visibility, and workflow
Edit all works in an Admin Set
See all works in an Admin Set even if suppressed and advance the works through workflow

@hannahfrost Can you QA this to see if we can close the issue? (I suspect we can but I don't want to be hasty.)
cc: @jcoyne

@mjgiarlo Need to leave this open. I'm finding that the functionality described in Hyku#1110 is not in Hyku based on testing today.

@hannahfrost I'm not sure we've updated Hyku in a while. Worth testing again once we do that.

Closing.


Windows Terminal - Use Arch Icon instead of the generic Linux icon

Requirements

[X] Yes
[ ] No

CPU Architecture

[ ] arm64
[X] x64

WSL Version

[X] WSL 2
[ ] WSL 1

Version
10.0.26100.2605
Kernel Version
<IP_ADDRESS>-1
Repro Steps

Install Arch.exe
Open Terminal
See the entry added as Arch with the generic Linux icon

Expected Behavior

Install Arch.exe
Open Terminal
See the entry added as Arch with the Arch Linux icon

Actual Behavior
The entry in the Terminal uses the generic Linux icon

known issue. have to set manually

Thanks for the kind reply. I am currently doing so, but isn't it possible to include the PNG icon in the ZIP release and use it when adding the entry on the Terminal?
I wanted to contribute with a PR, but I'm not able to find the source code behind Arch.exe, I saw it is based on wsldl however I'm not able to relate when that binary code becomes Arch.exe. I poked a bit in the Makefile as well but I wasn't able to find any logical reference to this as well.
Thanks in advance nevertheless.

It has to be done in windows terminal.
https://github.com/microsoft/terminal/issues/8604

I'm pretty sure that when I run wsl --install on a clean Win 11 24H2 installation, the Ubuntu icon did show up correctly in the terminal without me doing anything. Now, if it's them doing some magic behind the scene, might be a different story. But how do we add the entry on the Terminal app for Arch.exe? Where can the source code be seen for this action?

Windows terminal-> Settings-> Settings icon at the bottom to open the settings.json file. Edit it to add the icon. The link i gave you above has the details on how to do it.

@sileshn I know how to change it on the Windows Terminal app, and I have already done so. What I am trying to understand is how can we bake this into your Arch.exe so next time I don't have to change it manually. I am trying to understand where the source code of that binary is, so I can create a PR for you if you're interested, If you are not, then it's fine and we can close this issue. All I am trying to do here is trying to help :) Hope you do understand, thank you!

I can't do that. You'll have to contact @yuk7 to see if it can be incorporated into wsldl.

Thanks, that helps :) Appreciated! I'll keep you posted


DOMException when leaving from url including hash that is not valid as an ID.

below code causes DOMException when leaving from a url including hash that is not valid as an ID.
https://github.com/nuxt/nuxt.js/blob/358e2f32fdc01adb2631b68c027ba2d316dfffaf/lib/app/router.js#L56
A hash has no need to be valid as an ID.
For example, below url is valid. but causes DOMException.
http://localhost:4000/hoge#a*

sorry, I didn't understand html5 and querySelector specification well.
Please forget above explanation.
I'll send PR to fix this problem.


#11691: Move tensor module to ttnn

Ticket
Link to Github Issue #11691
Problem description
Move tensor module to ttnn
What's changed

Move tensor module bindings from ttnn/deprecated to ttnn
all references to ttnn.experimental.tensor.* no longer exist
moved convert_conv_weight_tensor* apis from tensor module to convolution (conv2d)
removed duplicated bindings of allocate_tensor_on_device, write_tensor
make Core* classes available in tt::tt_metal namespace

Checklist

[ ] Post commit CI https://github.com/tenstorrent/tt-metal/actions/runs/10670076608
https://github.com/tenstorrent/tt-metal/actions/runs/10796233975
[ ] Demo tests https://github.com/tenstorrent/tt-metal/actions/runs/10670082858
as main - https://github.com/tenstorrent/tt-metal/actions/runs/10775974676
[ ] t3k demos https://github.com/tenstorrent/tt-metal/actions/runs/10670095656
https://github.com/tenstorrent/tt-metal/actions/runs/10775981583
[ ] t3k frequent https://github.com/tenstorrent/tt-metal/actions/runs/10670080283
https://github.com/tenstorrent/tt-metal/actions/runs/10775970710
[ ] t3k unit https://github.com/tenstorrent/tt-metal/actions/runs/10670091567
https://github.com/tenstorrent/tt-metal/actions/runs/10775991646
[ ] t3k model perf https://github.com/tenstorrent/tt-metal/actions/runs/10670086480
[ ] nightly FD https://github.com/tenstorrent/tt-metal/actions/runs/10670088963
https://github.com/tenstorrent/tt-metal/actions/runs/10775985909
[ ] device perf https://github.com/tenstorrent/tt-metal/actions/runs/10670084135
https://github.com/tenstorrent/tt-metal/actions/runs/10775977596
[ ] pc model
[ ] model perf https://github.com/tenstorrent/tt-metal/actions/runs/10670078386
https://github.com/tenstorrent/tt-metal/actions/runs/10777550272
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

Hi @razorback3, @dongjin-na,
Could you please review this PR ?


Use single thread to refresh kerberos

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #13456 (934bc40) into dev (23c7363) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev   #13456      +/-   ##
============================================
- Coverage     39.54%   39.51%   -0.03%
- Complexity     4332     4345      +13
============================================
  Files          1087     1095       +8
  Lines         40985    41125     +140
  Branches       4701     4698       -3
============================================
+ Hits          16206    16251      +45
- Misses        22973    23070      +97
+ Partials       1806     1804       -2

Impacted Files
Coverage Δ

...he/dolphinscheduler/common/thread/ThreadUtils.java
0.00% <0.00%> (ø)

...r/plugin/datasource/hive/HiveDataSourceClient.java
2.50% <0.00%> (+1.14%)
:arrow_up:

...rce/hive/security/UserGroupInformationFactory.java
0.00% <0.00%> (ø)

...erver/master/processor/queue/TaskEventService.java
75.00% <0.00%> (-5.36%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
42.55% <0.00%> (-2.13%)
:arrow_down:

...hinscheduler/plugin/alert/script/ScriptSender.java
70.58% <0.00%> (-1.64%)
:arrow_down:

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
51.38% <0.00%> (-0.70%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...inscheduler/plugin/registry/etcd/EtcdRegistry.java
50.69% <0.00%> (-0.35%)
:arrow_down:

... and 30 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Bug] FileNotFound [Errno 2] For ideas.json

After updating to the image you published yesterday, I wasn't able to view my own ideas. I erased all of my config information and went back in with the default demo:password user, but see the same error when trying to look at /my_ideas or /user_gift_ideas/user2
The full error is below:
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1498, in __call__
    return self.wsgi_app(environ, start_response)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1476, in wsgi_app
    response = self.handle_exception(e)
               ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1473, in wsgi_app
    response = self.full_dispatch_request()
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 882, in full_dispatch_request
    rv = self.handle_user_exception(e)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 880, in full_dispatch_request
    rv = self.dispatch_request()
         ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 865, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)  # type: ignore[no-any-return]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/app.py", line 99, in decorated_function
    return f(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^
  File "/app/app.py", line 608, in user_gift_ideas
    gift_ideas_data = read_gift_ideas()
                      ^^^^^^^^^^^^^^^^^
  File "/app/app.py", line 624, in read_gift_ideas
    with open('ideas.json', 'r') as file:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: 'ideas.json'

I checked and the container did create both users.json and ideas.json in /app/data

Thanks, I forgot to change the reading function for docker. Will be fixed in 2 hours.

Now fixed, the image is built and pushed.
Was simply on line 624 i added a reading function last time and forgot to edit the docker version since it uses volumes.
Thanks


charged_kwh & charged_total disappeared after upgrading to 1.4

I've upgraded earlier this week from 1.3 to 1.4 in order to get rid of the wifi issues. But since the upgrade the charged_kwh & charged_total are not present anymore in the API (and also not visible on the webpage dashboard).
Do I need to do something after the upgrade or is this a bug?

Did you take the manual build of dingo35? That is not yet the full build, hopefully this week we can finally merge 1.4 with the master and have the final build ready but I need to resolve some merge conflicts first

Maybe related; I thought in earlier releases I had more info in the raw data, like the Mainsmet address.
With the @dingo35 release I get:
{"version":"v3serkri-0.00","mode":"SMART","mode_id":3,"car_connected":"false","evse":{"temp":34,"connected":"false","access":true,"mode":1,"solar_stop_timer":0,"state":"Ready to Charge","state_id":0,"error":"None","error_id":0,"rfid":"Not Installed"},"settings":{"charge_current":0,"override_current":0,"current_min":6,"current_max":13,"current_main":25,"solar_max_import":0,"solar_start_current":4,"solar_stop_time":10,"3phases_enabled":true,"mains_meter":"Eastron"},"home_battery":{"current":0,"last_update":0},"ev_meter":{"EVMeter_import_active_energy":718},"mains_meter":{"MainsMeter_import_active_energy":1755.9,"MainsMeter_export_active_energy":0},"phase_currents":{"TOTAL":30,"L1":8,"L2":16,"L3":6,"last_data_update":1661411985,"original_data":{"TOTAL":30,"L1":8,"L2":16,"L3":6}},"backlight":{"timer":0,"status":"OFF"}}

Below you'll see what I get in my last 1.4 build, this seems to contain everything?
{ "version": "v3serkri-1.4.0", "mode": "SOLAR", "mode_id": 2, "car_connected": true, "evse": { "temp": 38, "temp_max": 70, "connected": true, "access": true, "mode": 2, "solar_stop_timer": 0, "state": "Connected to EV", "state_id": 1, "error": "None", "error_id": 0, "rfid": "Not Installed" }, "settings": { "charge_current": 160, "override_current": 0, "current_min": 6, "current_max": 16, "current_main": 40, "solar_max_import": 0, "solar_start_current": 6, "solar_stop_time": 2, "3phases_enabled": false, "mains_meter": "Sensorbox" }, "home_battery": { "current": -12, "last_update":<PHONE_NUMBER> }, "ev_meter": { "description": "Disabled", "address": 12, "import_active_energy": 0, "total_kwh": 0, "charged_kwh": 0 }, "mains_meter": { "import_active_energy": 0, "export_active_energy": 0 }, "phase_currents": { "TOTAL": 11, "L1": 9, "L2": 6, "L3": -4, "last_data_update":<PHONE_NUMBER>, "original_data": { "TOTAL": -1, "L1": 5, "L2": 2, "L3": -8 } }, "backlight": { "timer": 0, "status": "OFF" } }

In my output I see at least the Mainsmet address missing (should be 101), also the EVMeter info is incomplete (I have the same Eastron there at address 102.
So your version looks more complete. Will have another look at the raw data when you release 1.4.0

@k-janssens everything is in there; AFAIK mainsmeter address has never been in there; the last 1.4.0 build has some extra evmeter info as compared to "my" earlier 1.4.0 build, which didn't have some of your / Koens changes since you did them in some other branch.
So new finally everything is converging into one branch. Again, AFAIK the merge went fine!

Did you take the manual build of dingo35? That is not yet the full build, hopefully this week we can finally merge 1.4 with the master and have the final build ready but I need to resolve some merge conflicts first

I used this one: https://github.com/serkri/SmartEVSE-3/issues/53#issuecomment-1221605205
So that is probably the issue... thanks :)
This is my output:
{
   "version":"SERKRI-1.4.0",
   "mode":"SMART",
   "mode_id":3,
   "car_connected":"true",
   "evse":{
      "temp":37,
      "connected":"true",
      "access":true,
      "mode":1,
      "charge_timer":0,
      "solar_stop_timer":0,
      "state":"Connected to EV",
      "state_id":1,
      "error":"None",
      "error_id":0,
      "rfid":"Not Installed"
   },
   "settings":{
      "charge_current":0,
      "override_current":0,
      "current_min":6,
      "current_max":16,
      "current_main":20,
      "solar_max_import":0,
      "solar_start_current":4,
      "solar_stop_time":10,
      "3phases_enabled":false,
      "mains_meter":"Sensorbox"
   },
   "home_battery":{
      "current":0,
      "last_update":0
   },
   "phase_currents":{
      "TOTAL":-7,
      "L1":22,
      "L2":-18,
      "L3":-11,
      "last_data_update":1661415341,
      "original_data":{
         "TOTAL":-7,
         "L1":22,
         "L2":-18,
         "L3":-11
      }
   },
   "backlight":{
      "timer":0,
      "status":"OFF"
   }
}

@k-janssens everything is in there; AFAIK mainsmeter address has never been in there;

Ok, I went back to 1.3.8 to confirm what I saw. Here is the raw data and it does show the Mainsmet address.
{"version":"SERKRI-1.3.8","mode":"SMART","mode_id":3,"car_connected":"false","evse":{"temp":35,"temp_max":65,"connected":"false","access":true,"mode":1,"charge_timer":0,"solar_stop_timer":0,"state":"Ready to Charge","state_id":0,"error":"None","error_id":0,"rfid":"Not Installed"},"settings":{"charge_current":0,"override_current":0,"current_min":6,"current_max":13,"current_main":25,"solar_max_import":0,"solar_start_current":4,"solar_stop_time":10,"3phases_enabled":true,"mains_meter":"Eastron"},"home_battery":{"current":0,"last_update":0},"ev_meter":{"description":"Eastron","address":102,"power":0,"total_kwh":718054,"charged_kwh":0},"phase_currents":{"TOTAL":30,"L1":8,"L2":16,"L3":6,"last_data_update":1661415477,"original_data":{"TOTAL":30,"L1":8,"L2":16,"L3":6}},"backlight":{"timer":113,"status":"ON"}}

No I did not check correct. It is the other Eastron. The mainsmet has address 101, and is indeed missing.

The new release has been build:
https://github.com/serkri/SmartEVSE-3/releases/tag/v1.4.1-serkri-v3

The new release has been build: https://github.com/serkri/SmartEVSE-3/releases/tag/v1.4.1-serkri-v3

Great. Issue is resolved!


Fix imaging extensions

Add Imaging class to replace original OmeTif class.

I think that’s OK for now. But I do see the limitation.
On Thu, 9 Jun 2022 at 12:38, Thomas Yu @.***> wrote:

@.**** approved this pull request.
LGTM. Another downside of this is that the validation status table won't
have granular file types . It will all display as "Imaging"
—
Reply to this email directly, view it on GitHub
https://github.com/Sage-Bionetworks/synapsegenie-htan-registry/pull/7#pullrequestreview-1001174838,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSA366EJDZDFNLXICEYZ33VOHJUVANCNFSM5YGS4MRA
.
You are receiving this because you authored the thread.Message ID:
<Sage-Bionetworks/synapsegenie-htan-registry/pull/7/review/1001174838@
github.com>

--
[image: photo]
Dr Adam J Taylor
Senior Research Scientist, Sage Bionetworks
@.***
sagebionetworks.org
London based. GMT = PT+8/ET+5


update description metadata - batch 8

@billwagner @tdykstra
Updates for missing/incorrect description metadata.

OK, all the missing single quotes fixed on this one (that I could find)


Marine CBS UI enhancements

-Designed 4 cost pages with options for varying level of cost definitions for various aspects of Marine Energy Converter projects (both wave and tidal)
-Summary page enhancements that allow for multiple tables, multiple columns per table
-Linked to ssc pull request with same branch name.
-Also includes wex changes
-Closes #389
-Closes #390

@cpaulgilman Would it be possible to change the documentation to reflect changing labels for the itemized costs? Cost breakdown is more consistent with the Marine Energy terminology so far. If that is more confusing that way I can revert.

@mjprilliman Yes. I'm happy to make those changes. I'll do that work in this branch so you can review. If that causes conflicts with Help files when you merge this branch into Develop, feel free to overwrite the ones in Develop. I will build a new Help system for the Develop branch right before we build the beta release version.
One question: Your screenshots above use "cost breakdown" but what I see in the version I built this morning is "itemized costs". Either is fine, I just want to make sure use the correct terms in Help.

@cpaulgilman I forgot to push my changes last night. Just pushed now. Cost breakdown is what I am going with now

Help revisions done in https://github.com/NREL/SAM/commit/c89fa6f6b78175afaa0cff18d700ec7b7224b881.

@sjanzou @cpaulgilman merging this in for the beta.


Combine openshift-tests and openshift-tests-upgrade suite

We differentiate tests on suite and name, because I imagined a scenario where two entirely different test suites create a generically named test like "TestClusterCreate" and we don't want to conflate them.  However, openshift-tests and  openshift-tests-upgrade are definitely the same tests, and it makes sense to combine them. We already get the "upgrade" view from the variants.

After talking on slack, it's best to leave them separate for a while


Text does not load when running from local filesystem

None of the text gets loaded when running from the local filesystem.
Looking at the console this seems to be the error
Access to XMLHttpRequest at 'file:///D:/Download/ac-nh-turnip-prices-master/ac-nh-turnip-prices-master/locales/en.json' from origin 'null' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, https.

The best way to develop/run locally would be running a webserver locally. For example, running python -m SimpleHTTPServer in the folder.
Many thing don't work with file:// protocol because of security restrictions by browser, so I don't think we can do anything to solve this.

I'm actually running google-chrome --allow-file-access-from-files to develop locally from the filesystem.

I've been using webpack-dev-server to host the app locally for dev.
It might be worth including a section in the README that describes a couple options for running the site in a lightweight development environment.

python -m SimpleHTTPServer and python3 -m http.server will also work (depending on your version of python).

I've switched to using
npx serve
It's a very simple zero config web server that only requires you have NodeJS on your machine.


Plans for a stable version release of 1.0

wanted to check a couple of things
Any plans for releasing a stable 1.0 version?
want to update the go version as the one here is old.
finch is built on top of nerdctl and nerdctl depends on this package for network namespacing. so wanted to check if we can move to a stable version.

I'm personally not in a hurry to go 1.0 because I'm not super familiar with netns deep intricacies and work at a different day job that doesn't actually use this code. So I don't know enough to know if this API is solid or if it still needs a few tweaks.
That said, looking at the commit history, this library is pretty stable so I'd feel extremely comfortable using it in production (and did at a former employer).
As far as the go version, that's the "minimal" version of go that's required in order to import the library... unless/until we have a need, I don't see a point in bumping it...
To be clear, I'm totally fine with saying "practically as open source maintainers/bug fixers, we only promise to support the same versions of go that the core team supports" but there's no reason we need to enforce that at the code level until there's actually a change that we want to pull in that depends on new/changed behavior in newer versions of go.

As far as the go version, that's the "minimal" version of go that's required in order to import the library...

I don't see a point in bumping it...
:+1:


Match MQ-E and CE-E

I ported the existing changes here, this is a bit ugly but I'm out of ideas at this point lol
european versions are changing a lot (by that I mean more then the north-american versions did), mainly due to the language support, MQ-E adds german and french and CE-E adds spanish and italian (+ the MQ ones obviously), both european versions adds extra files: xlText.c and xlFile.c, the latter brings symbol oddities with xlFileGCN.c (some things were moved between these files)
I decided to move the offset tables from rom.c to its own header, it became way too large it's a bit annoying, and the worst function is systemSetupGameAll because it's a big one and there's lots of changes lol
the only file I didn't match is simGCN.c, I did some changes there but it's very wip (I just don't want to complete it, maybe someday if it's not done until then), otherwise everything else is matched (except libraries/sdk but I'll do that in another PR)
feel free to make suggestions, btw the PR looks big because of the config files, actual code changes aren't that large

I'm not sure about incbin for rom arrays tbh, it makes it harder to actually view the data, and as you pointed out the type is wrong. Maybe it doesn't belong in this PR either way though


Filter text colour looks greyed out

When I write a filter, it looks greyed out:

Compare with the placeholder text here:

I bet this is related to redesign that spurred #204
Thanks :)
This is low-priority

Fixed in version 4.17.3 (I had originally planned to fix this at the same time as implementing Manifest v3 compatibility, hence the delay, but Duo had other ideas it seems!)


Issue with a big set of Ids in a where clause

Hi @jongpie and @jamessimone
Similar to the issue mentioned over here: https://github.com/jamessimone/apex-dml-mocking/issues/2 (which has been fixed by @jamessimone ) I discover the same issue also in this framework (which is pretty cool by the way)
So, in my scenario I do a query on account team member by providing a huge list of account ids
I've created a small Repository which has a single method:
public with sharing class AccountTeamMemberRepository extends SObjectRepository {

    public override SObjectType getSObjectType() {
        return Schema.AccountTeamMember.SObjectType;
    }

    public List<AccountTeamMember> getAccountTeamMembersByAccountIds(Set<Id> accountIds) {
        return new SObjectQueryBuilder(this.getSObjectType())
                .addAllFields()
                .filterBy(new QueryFilter().filterByField(new QueryField(Schema.AccountTeamMember.AccountId), QueryOperator.IS_IN, accountIds))
                .getQueryResults();
    }
}

When I run this in Anonymous Apex with a set of more than 12 account Ids, I get an error
//anonymous apex
AccountTeamMemberRepository repo = new AccountTeamMemberRepository();
repo.getAccountTeamMembersByAccountIds(new Set<Id>{
        '0010O0000265ZliQAE', '0010O000028mXFFQA2', '0010O000028peYlQAI', '0010O000028pexAQAQ', '0010O00002B6sgBQAR', '0010O00002B78G6QAJ', '0010O00002B7AceQAF', '0010O00002B7AgWQAV', '0010O00002B7FilQAF', '0010O00002ENWYuQAP', '0010O00002ENXJDQA5', '0010O00002Md4QBQAZ', '0010O00002MdnnFQAR', '0010O00002OVySHQA1'
});

Error on line 126, column 1: System.QueryException: invalid ID field: ...
Class.QueryBuilder.executeQuery: line 126, column 1
Class.QueryBuilder.doGetQueryResults: line 97, column 1
Class.SObjectQueryBuilder.getQueryResults: line 210, column 1

When I debug the query right before execution we can clearly see, that the last "entry" is just some dots, and most of the Ids are missing
SELECT account.name, accountaccesslevel, accountid, automaticallyadded__c, caseaccesslevel, contactaccesslevel, createdby.name, createdbyid, createddate, id, isdeleted, lastmodifiedby.name, lastmodifiedbyid, lastmodifieddate, opportunityaccesslevel, photourl, projectaccesslevel__c, systemmodstamp, teammemberrole, title, user.name, userid
FROM AccountTeamMember
WHERE AccountId IN ('0010O0000265ZliQAE', '0010O000028mXFFQA2', '0010O000028peYlQAI', '0010O000028pexAQAQ', '0010O00002B6sgBQAR', '0010O00002B78G6QAJ', '0010O00002B7AceQAF', '0010O00002B7AgWQAV', '0010O00002B7FilQAF', '0010O00002ENWYuQAP', '...')

mhh... ?! :-(

Fun fact: when I change the parameter from Set to List it works smoothly.
Comments
@derroman

derroman commented 16 minutes ago
Hi @jongpie and @jamessimone

Similar to the issue mentioned over here: jamessimone/apex-dml-mocking#2 (which has been fixed by @jamessimone ) I discover the same issue also in this framework (which is pretty cool by the way)

So, in my scenario I do a query on account team member by providing a huge list of account ids

I've created a small Repository which has a single method:

public with sharing class AccountTeamMemberRepository extends SObjectRepository {

    public override SObjectType getSObjectType() {
        return Schema.AccountTeamMember.SObjectType;
    }
    //changed from Set to List
    public List<AccountTeamMember> getAccountTeamMembersByAccountIds(List<Id> accountIds) {
        return new SObjectQueryBuilder(this.getSObjectType())
                .addAllFields()
                .filterBy(new QueryFilter().filterByField(new QueryField(Schema.AccountTeamMember.AccountId), QueryOperator.IS_IN, accountIds))
                .getQueryResults();
    }
}

Based on this comment by @jamessimone here https://github.com/jamessimone/apex-dml-mocking/issues/1#issuecomment-764743723 and here https://github.com/jamessimone/apex-dml-mocking/blob/384ce721fc1ef46c251e590b7b2fd521dc81d9d6/force-app/repository/Query.cls#L100  I assume Sets in apex are kind of difficult to handle?! ;-)

Fun fact: when I change the parameter from Set to List it works smoothly.
Comments
@derroman

derroman commented 16 minutes ago
Hi @jongpie and @jamessimone

Similar to the issue mentioned over here: jamessimone/apex-dml-mocking#2 (which has been fixed by @jamessimone ) I discover the same issue also in this framework (which is pretty cool by the way)

So, in my scenario I do a query on account team member by providing a huge list of account ids

I've created a small Repository which has a single method:

public with sharing class AccountTeamMemberRepository extends SObjectRepository {

    public override SObjectType getSObjectType() {
        return Schema.AccountTeamMember.SObjectType;
    }
    //changed from Set to List
    public List<AccountTeamMember> getAccountTeamMembersByAccountIds(List<Id> accountIds) {
        return new SObjectQueryBuilder(this.getSObjectType())
                .addAllFields()
                .filterBy(new QueryFilter().filterByField(new QueryField(Schema.AccountTeamMember.AccountId), QueryOperator.IS_IN, accountIds))
                .getQueryResults();
    }
}

Based on this comment by @jamessimone here https://github.com/jamessimone/apex-dml-mocking/issues/1#issuecomment-764743723 and here https://github.com/jamessimone/apex-dml-mocking/blob/384ce721fc1ef46c251e590b7b2fd521dc81d9d6/force-app/repository/Query.cls#L100  I assume Sets in apex are kind of difficult to handle?! ;-)


Consider adding this copy/paste extension

https://predictablynoisy.com/sphinx-copy-buttons

Closed by #15


[Runtime] Fix warnings for inconsistent nullability annotation on _stdlib_getEnviron.

rdar://problem/39739801

@swift-ci please test

PR#13254 is related to this.  @jrose-apple and I discussed this on the original PR, and fixing that seemed to uncover something which needed to go through evolution :-(.

I'm in favor of _Null_unspecified as well.

Given that there seems to be disagreement (between standard and implementation, not just between us!) over whether it is in fact nullable, _Null_unspecified actually seems like an accurate annotation for once.

@rjmccall No good reason! I'm going to separate that into another PR now.

Switched to _Null_unspecified and cut out the unrelated offsetof fix.

@swift-ci please smoke test

@swift-ci please smoke test


make progress based on upload API

Make progress upload based on upload, not need backend compatible

Thanks, now its pointing to develop 😄


Can you call stored procedures?

Hi,
I've been researching Rezoom because I want to access some SQL Server stored procedures but I'm not able to find anything on the documentation for it. Is it supported?
Thanks!
Erlis

Hi,
You can use a vendor statement to call stored procs. Since this bypasses the typechecker, and requires you to manually write an imagine clause if you want to read data coming from the sproc, rzsql's probably not a good fit if your project is mostly stored procedures. On the other hand if you're doing 95% plain queries and only need to run a stored procedure occasionally, the cost/benefit probably works out and it's worth the hassle.
Regards,
Robert


[MRG] Reduce linewidth of contours in plot_topomap

The contours should not have the same width as the head geometry - using thinner lines separates the contours better from the head IMO.
Before:

After:

please put the 2.
it's a good practice anyway for readability.

I disagree, I think 2. reduces readability. But anyway, no problem!

it makes it explicit that you work with floats
but anyway yes.

In Python 3 it doesn't matter if you divide floats or ints, the result will always be an int. I've updated it anyway. Pretty long discussion for such a tiny change 😄.

thx @cbrnr
just keep some love for our python 2 friends...

Merging before the CIs come back happy, bold move @agramfort !

I'm sure you ran unit test locally like in 2012 :)

sure :)

Sorry, my Python 2 love is long gone - but I did it for MNE anyway 😆.


linked services missing /etc/hosts entry

Hello,
I'm testing out the linked services and i've noticed that the environment variables are there, but the entry in /etc/hosts is not.
For example, i have this in my docker-compose.yml file
db:
  labels:
    io.rancher.scheduler.affinity:host_label: name=bbb
  ports:
  - 3306/tcp
  environment:
    MYSQL_DATABASE: aaa
    MYSQL_PASSWORD: aaa
    MYSQL_ROOT_PASSWORD: aaaa
    MYSQL_USER: aaa
  image: mysql:5.6
redis:
  labels:
    io.rancher.scheduler.affinity:host_label: name=bbb
  ports:
  - 6379/tcp
  image: redis
aaa:
  labels:
    io.rancher.scheduler.affinity:host_label: name=bbb
  ports:
  - 443/tcp
  - 80/tcp
  environment:
    DB_NAME: aaa
    DB_PASS: aaa
    DB_USER: aaa
  image: my-image/test:latest
  links:
    db: mysql
    redis: redis

but when I cat the /etc/hosts/ file on a container in the aaa service, here is what it contains
<IP_ADDRESS>     2943fc46c73a
<IP_ADDRESS>       localhost
::1     localhost ip6-localhost ip6-loopback
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters

this is on a php-apache instance from docker.  With out the host entry there, it causes PHP to throw php network getaddresses getaddrinfo failed name or service not known because it's not in the /etc/hosts file.

however, when i get a terminal into the box, i can ping the db and redis links just fine.

Service links can be dynamically added and removed.  The containers are configured to point at an internal DNS server on the network agent and that provides the resolution.  They are not put into /etc/hosts because we can't update that once a container is running.

@vincent99 thanks for the info.  Something strange is going on still, because if the redis server is on the same host it works fine, however when they are on different hosts, that error comes up.  I'm sure it's just internals of php being weird when using stream_socket_client

@vincent99, so i played around with it this am, and turned off the php service and then brought it back up and it started working fine.  I wonder if it was just a procedure thing.  I'll close this for now since it seems like it was my issue and not rancher.

It's been a while since I did PHP, does it have its own DNS cache?  Or does your app only try to look the name up once on startup maybe?
It's possible there's a race condition and the name is not available in the DNS server yet by the time your container is looking for it.


io/fs: Provide Package-Level Documentation for the Package

Proposal Details
Related to #64373, I would like to recommend that package fs gets some more top-level package documentation.  At present, all that it states is:

Package fs defines basic interfaces to a file system. A file system can be provided by the host operating system but also by other packages.

This provides no information about the package's use and philosophy.  There is a proposal document for package fs's design, but that would not be a suitable drop-in replacement to describe use.  A couple of things come to mind that are worth mentioning that are not obvious:

For casual use, the a user of the package should primarily rely upon the package's top-level functions when they exist versus specialized interface methods.  For instance, use function fs.ReadFile on a fs.FS instead of caring whether a fs.FS implements fs.ReadFileFS (runtime type assertion) and then calling (fs.ReadFileFS).ReadFile directly on the asserted value.

The package's documentation should suggest/tease a couple of common entrypoints for file system access and creation:

os.DirFS creates a filesystem around an ordinary POSIX file system.
package embed can provide embedded file systems in the binary.

Without this information, it leaves a reader to wonder whether there are any batteries included, which is not a good developer experience.

The package's top-level documentation should demonstrate some common workflows, too.  That would help make many of these points non-issues.  There is one package example in the documentation (for WalkDir).  If a user's new to this package, the WalkDir example may not be the best foray to demonstrate it.

There are enough symbols floating around in this package's identifier space that it's rather hard to tease apart critical API versus secondary API, which hinders overall comprehension of the package.

removing from proposal process, it should be fine to just submit CLs.

...and as a counterpart to point 1, a section for how to properly implement a file system would help as well.  After all, those are also outside users of the io/fs package.
There are some subtleties especially when wrapping and composition of filesystems is involved where the results might not implement the same "extension interfaces" any more as the input.  (e.g. if a file system does not implement Sub(), using fs.Sub() can result in a file system that implements fewer extension interfaces than the input file system)
Most of this is probably already explained in the resources that surrounded the creation of this package, but that is not obvious from the existing documentation:

https://go.googlesource.com/proposal/+/master/design/draft-iofs.md
https://www.youtube.com/watch?v=yx7lmuwUNv8


Encrypt Using Public Key - Example?

Hello. I've been reading through the tests to understand how to use your library to encrypt using only a public key but have had a hard time understanding how.
Here's the function I've been looking at to understand:
https://github.com/rpgp/rpgp/blob/202e7c1059393ae6540d3376348924e12e8c8c17/tests/key_test.rs#L156
Would you be able to point me to docs or other code that could help illustrate how to encrypt with a public key?

I am not sure I understand the question. The test you are pointing at, does exactly that, using a public key to encrypt and decrypt using a private key:

encrypt: https://github.com/rpgp/rpgp/blob/master/tests/key_test.rs#L192-L193
decrypt: https://github.com/rpgp/rpgp/blob/master/tests/key_test.rs#L201-L202


getbootstrap.com mixed content

Carbonads is currently setup to use http for all requests instead of https meaning there's a mixed content warning shown on getbootstrap.com
I've already checked and they serve the https version of all the files currently being served over http.

This is already fixed in v4 (4a63dc6). @mdo, do we care about this enough to backport to v4?

Hmm, I thought I updated the hosted docs to the new ads code. 🤔
For stuff like this, I think I'm okay to update the docs. I'd hate to have folks feel it's not usable for some reason because of console warnings like this.

@mdo, you did update the hosted docs, but the v4 ones, not v3.

@mdo You porting back 4a63dc6 or should we do the easy fix for v3 and just update the script URL to be HTTPS?

@mdo You porting back 4a63dc6 or should we do the easy fix for v3 and just update the script URL to be HTTPS?

Punting on this.


[NCL-8685] More strict url validation for gitlab

If we are using gitlab as the internal url during build-config creation, we need to enforce scp format for the internal url for consistency.
Special thanks to @DnsZhou for his huge help on this one!

Ready for review after comment fixes.


Error fat file

#I am using v3.0.0, it's work in IDE, but have an error while running in single jar (fat) file with dependencies.
OS. Linux ubuntu 18.04 LTS
jdk 1.8
I think the error cause of missing in the  loading  of .so file.
====== Error ===========
java -jar my-project-name-jar-with-dependencies.jar
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.codebrig.journey.JourneyLoader (file:/home/khalegh/Downloads/8/projectJavaChromeBrowserJavaEight/WEB/target/my-project-name-jar-with-dependencies.jar) to field java.lang.ClassLoader.sys_paths
WARNING: Please consider reporting this to the maintainers of com.codebrig.journey.JourneyLoader
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
Exception in thread "main" java.lang.ExceptionInInitializerError
at net.khalegh.ShowBrowserDemo.main(ShowBrowserDemo.java:12)
Caused by: java.lang.RuntimeException: java.lang.NullPointerException
at com.codebrig.journey.JourneyLoader.setup(JourneyLoader.java:204)
at com.codebrig.journey.JourneyBrowserView.(JourneyBrowserView.java:33)
... 1 more
Caused by: java.lang.NullPointerException
at java.base/java.lang.ClassLoader.loadLibrary(ClassLoader.java:2646)
at java.base/java.lang.Runtime.loadLibrary0(Runtime.java:830)
at java.base/java.lang.System.loadLibrary(System.java:1873)
at com.codebrig.journey.JourneyLoader.loadLinux(JourneyLoader.java:338)
at com.codebrig.journey.JourneyLoader.setup(JourneyLoader.java:173)
... 2 more

done. it's work.


[HOLD for payment 2024-11-28] [$250] Search - Search icon is not responsive after sharing expense with accountant and return to LHN

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

Version Number: 9.0.59-0
Reproducible in staging?: Y
Reproducible in production?: N/A - new feature, doesn't exist in prod
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Launch ND or hybrid app.
Go to self DM.
Track a manual expense.
Go to Search.
Tap on the track expense in Step 3.
Open report header.
Tap Share it with my accountant.
Select a workspace, enter merchant and submit.
Swipe to the right (iOS navigation, not app back button) twice to return to LHN.
Tap on the search icon.

Expected Result:
Search icon will be responsive after sharing expense with accountant from Search and returning to LHN.
Actual Result:
Search icon is not responsive after sharing expense with accountant from Search and returning to LHN.
Workaround:
Unknown
Platforms:

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

Screenshots/Videos
https://github.com/user-attachments/assets/fb7c98ba-1904-4de2-9375-0b03335f93c8

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

Upwork Job URL: https://www.upwork.com/jobs/~021855821019880549411
Upwork Job ID:<PHONE_NUMBER>880549411
Last Price Increase: 2024-11-11
Automatic offers:

mkzie2 | Contributor | 104839852

Issue OwnerCurrent Issue Owner: @zanyrenney

Hey Vivek, how is this one going? Let me know if I can help at all or if you need anything tested.

Reproducible in production?: N/A - new feature, doesn't exist in prod

Not a blocker. Checking off deploy list

Proposal
Please re-state the problem that we are trying to solve in this issue.
Search icon is not responsive after sharing expense with accountant from Search and returning to LHN.
What is the root cause of that problem?
The problem is that when we share expenses with the accountant from the RHP report, we dismiss the modal, which also dismisses the RHP report, but the search screen still exists. Additionally, we navigate to the room invite page which leads to a report screen being added.
https://github.com/Expensify/App/blob/ba543d742e112f7f0879da76b1f76505c97c87ff/src/libs/actions/IOU.ts#L3960
After we click on the device's back button twice, LHN is displayed, but the current central pane is the search screen. Then nothing happens when we click on the search icon because the current central pane is the search page.
https://github.com/Expensify/App/blob/ba543d742e112f7f0879da76b1f76505c97c87ff/src/libs/Navigation/linkTo/index.ts#L95
What changes do you think we should make in order to solve the problem?
If the action is share and the topmost central pane is search, we should goBack first to remove the search central pane and navigate to the activeReportID
if (action === CONST.IOU.ACTION.SHARE) {
    if (isSearchTopmostCentralPane() && activeReportID) {
        // Navigation.navigate(ROUTES.SEARCH_REPORT.getRoute({reportID: transactionThreadReportID}));
        Navigation.goBack();
        Navigation.navigate(ROUTES.REPORT_WITH_ID.getRoute(activeReportID));
    }
    Navigation.setNavigationActionToMicrotaskQueue(() => Navigation.navigate(ROUTES.ROOM_INVITE.getRoute(activeReportID ?? '-1', CONST.IOU.SHARE.ROLE.ACCOUNTANT)));
}

https://github.com/Expensify/App/blob/ba543d742e112f7f0879da76b1f76505c97c87ff/src/libs/actions/IOU.ts#L3962
What alternative solutions did you explore? (Optional)
Another solution is we will back to the RHP transaction thread report after we share expense
if (isSearchTopmostCentralPane() && (action === CONST.IOU.ACTION.SHARE) && activeReportID) {
    Navigation.dismissModal();
    Navigation.navigate(ROUTES.SEARCH_REPORT.getRoute({reportID: activeReportID}));
} else {
    Navigation.dismissModal(isSearchTopmostCentralPane() ? undefined : activeReportID);
}

https://github.com/Expensify/App/blob/ba543d742e112f7f0879da76b1f76505c97c87ff/src/libs/actions/IOU.ts#L3960

@allroundexperts Can you take a look at my proposal?
cc @techievivek

@mkzie2 Can you also please point out the PR which caused this?

@allroundexperts It comes from https://github.com/Expensify/App/pull/51951.

@mkzie2's proposal looks good to me. It has the correct RCA and the solution makes sense as well. Let's go with them.
🎀 👀 🎀 C+ reviewed

payment summary
@allroundexperts requires payment through NewDot Manual Requests - please request $250
@mkzie2 requires payment automatic offer (Contributor) - paid $250 via Upwork.

@allroundexperts please can you complete the checklist before getting the ND payment? Thanks!

Checklist

https://github.com/Expensify/App/pull/51951
https://github.com/Expensify/App/pull/51951/files#r1862758035
N/A
A regression test would be helpful.

Regression test

Launch the app and Go to self DM.
Track a manual expense.
Go to Search and Tap on the expense created earlier.
Open report header.
Tap Share it with my accountant.
Select a workspace, enter merchant and submit.
Press the physical back button (on Android) or Swipe right (on iOS) twice to return to LHN.
Tap on the search icon.

Verify that the search icon will be responsive after sharing expense with accountant from Search and returning to LHN.
Do we 👍 or 👎 ?

Regression tests looks good to me.

https://github.com/Expensify/Expensify/issues/448689

Complete!


[BEAM-14350] - beam_PerformanceTests_ManyFiles_TextIOIT_HDFS missing kubeconfig

Moved up get-credentials instruction for getting the kubeconfig file before copy it to the given path.
Note: In PR#17396 was updated metrics test to get credentials every time the job is triggered and, as a best practice, a cleanup was introduced as well, this may be causing the issue in this task.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java ManyFilesTextIO Performance Test

Run Java ManyFilesTextIO Performance Test HDFS

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Run Java ManyFilesTextIO Performance Test HDFS

Run Java HadoopFormatIO Performance Test

Hi @apilloud , could you help me running a Run Java ManyFilesTextIO Performance Test HDFS  seed job for this PR please?
Thanks

Run Seed Job

Seed job running: https://ci-beam.apache.org/job/beam_SeedJob/9565/console

Thanks @apilloud

Run Java ManyFilesTextIO Performance Test HDFS


bower install warning message

bower install ekko-lightbox --save bower not-cached    https://github.com/ashleydw/lightbox.git#* bower resolve       https://github.com/ashleydw/lightbox.git#* bower download      https://github.com/ashleydw/lightbox/archive/v5.0.0.tar.gz bower extract       ekko-lightbox#* archive.tar.gz bower invalid-meta  for:/var/folders/lw/4bpgcsqd4l9_5vnwmhsdffy80000gn/T/artem/bower/c018eb69d62130b52e552e8e3761a4db-54041-LyuYq0/bower.json bower invalid-meta  The "main" field cannot contain minified files bower mismatch      Version declared in the json (4.1.0) is different than the resolved one (5.0.0) bower resolved      https://github.com/ashleydw/lightbox.git#5.0.0 bower install       ekko-lightbox#5.0.0

hm i thought i'd removed the bower.json file but anyway, i've changed the version in it to 5.0.0, try again?


[Plugin] Comments

Name: Comments or some fancy name
Idea:
Create a file for each page to store the comments.

filename: the same as the key page
the content of the file could be an array serialized to JSON.
the new comments are inserted at the end or at the beginner? We need to think about if going be displayed from newer to old or from old to newer and try to focus en the best to keep performance.

Hooks

afterPageCreate create the file with empty structure
-afterPageModify check if not was edited the key
afterPageDelete delete the file with the comment for the page deleted
pageEnd load the file for the comments

Settings

Template for comments
Order by older to newer / newer to older
Date format

To be GDPR compliant is has to be possible to delete on request the comments of a user (name and e-mail are personal data).

That is going to be a problem, I mean is going to be hard to develop, but for the moment maybe is possible to request a nickname and the comment, or use some authentication via GMail or Facebook, so the data is not stored in Bludit.

This would be very useful!!

Howdy guys
Just popped up to ask how are things going with implementing discussed commenting plugin?
Found bludit couple days ago and after testing several dozen flat file CMS-es, this one suits perfectly for a personal blog I'd want to launch. And it lacks only an internal commenting system.

@Joduai Currently there are three alternatives I've found:

Using Disquz with the Disquz plug-in (already included with the base package)
Using Vanilla Forums as comment system via https://github.com/clickwork-git/vanilla
Using ISSO via https://git.mal-richtig.de/maxmalrichtig/bludit-isso-client (hasn't been updated in a year, though)

@ltGuillaume

Currently there are three alternatives I've found:

Thanks for mentioning, but I'm aware of all three of them, as I was searching if something else than disqus plugin exist.
Unfortunately all available commenting solutions don't come as working out of box with bludit.
Kludge raised this issue on bludit's forum: https://forum.bludit.org/viewtopic.php?t=774 long time ago so I won't duplicate the reasons why a blogging platform needs an internal comment system.
Maintaining additional software which vanilla only for a comment purpose doesn't make sense for a simple blog. Not to mention it's a database driven piece of software, thus a flat file blog platform loose its sense.
As it goes to isso, bludit has only a connector_like plugin to isso server. This solution won't work on a shared hosting environment, and even with a shell access I'm afraid the restrictions would be too high.

Maintaining additional software which vanilla only for a comment purpose doesn't make sense for a simple blog. Not to mention it's a database driven piece of software, thus a flat file blog platform loose its sense.

@Joduai I completely agree

As it goes to isso, bludit has only a connector_like plugin to isso server. This solution won't work on a shared hosting environment, and even with a shell access I'm afraid the restrictions would be too high.

Yeah I suspected this when I had a quick peek at it earlier today. Pretty useless then, indeed.

Oh, the built-in commenting system is my dream.
Disqus has some problem, it banned in some country just like in my country, it's LTR, a lot of people use RTL language, and the main problem is it slows down the Bludit.
As @Joduai said using vanilla forum just for the commenting system doesn't make sense.


adds vault chart

This will replace the old vaultlab.

I created it and it got already pushed into it.