current behavior is to replace view with spend on import

What is the name of the abstraction that encapsulates a set of keys with a diminishing authority relation?  Shouldn't that abstraction naturally capture this case, with the two keys simply being elements of the authority cascade?

The closest thing we have to this is the placeholder name, not a good name WalletOKeyInner: https://github.com/zingolabs/zingolib/blob/99cbf19c025ceae0c9aa23399fdc36b6f4e44f58/lib/src/lightwallet/orchardkeys.rs#L37-L43


ResourceSet methods fetchMember and create do not set Resource to isLoaded

The ResourceSet methods fetchMember and create return a Resource object created from JSON obtained from the server within the same function call. However, they do not set the isLoaded flag in the returned Resource object to true. An examination of the code reveals the same issue with other methods, including sync and mergeSync (possibly others).
Subsequent method calls on the Resource object (notably toJSON) initiate a load call on the Resource object before returning a value. This creates an extraneous call to the web service and discards properties arguments passed to the original loading method. As a result, the user is unable to get a reduced-scope JSON object from Resource objects when using these methods, while also generating unexpected calls to the server.
Possible solutions:

Expose the isLoaded variable in Resource through package-private methods or constructors, allowing partner classes to create "pre-loaded" Resource objects.
Set isLoaded to true by default in the Resource constructors that take JsonObject parameters. Make the constructors package-private.

Neither solution exposes the isLoaded variable to the public interface, but Solution 2 seems superior. It is simpler to implement and requires no extra work within ResourceSet or other partner classes. The current library design offers no obvious reason for users to construct Resource objects directly. If that need arises, static factory methods could be added to enable controlled construction.

Thank you for your feedback, we are currently reviewing the issues. Will update the issue as soon as we have the explanation or fix ready to post.

Here is the code example for preload:
	//Functionality Test for pre-load
	int index=0;
	Resource preLoadResource = setHasTerms.member(index);
	while(preLoadResource != null)
	{
		if(index%2 != 0) {
			preLoadResource.setLoaded(true);
		}
		Util.jsonPrettyPrinter(preLoadResource.toJSON());
		preLoadResource = setHasTerms.member(++index);
	}
	Util.jsonPrettyPrinter(setHasTerms.count());

It is available in Git and Maven.


Headers of exported dimension table should be measure label (if present) not measure name

Added details in Notion

Here https://github.com/rilldata/rill/pull/3348


Feature request: Call Hierarchy

A call hierarchy of a function invocation is a very useful tool to quickly navigate in code. In VSCode I find my self using "Find Reference" repeatedly to do what I do with a call hierarchy in IDEs (Eclipse, IntelliJ).
UI-wise I think it could fit nicely with how "Find References" is done, but of course we would need to have a tree on the right hand side and the labels need to display the reference containing function definitions.
Conceptually it is close to a type hierarchy, which is requested in #15533.

Adding my vote to this one. This would be my number 1 most important missing feature and use it daily in eclipse to navigate and understand the structure of a large code base. I can live without find all references feature and use global text search with whole word option instead.

This feature is really necessary for people like me who need to browse source code of multiple open source projects.

It would be super beneficial especially for users coming from eclipse,intellij background.

@hualet @sandipanbiswas FYI we ignore comments in the sprit of "+1", "me too", "This is needed" etc but go for up votes (:+1:) on the descriptions of this item. Please refrain from adding these comments because it generates a lot of noise to the project and we spend time reading these comments instead of getting actual coding work done. Thanks

@jrieken Have you had any progress on this?

anynews

For those in interested in this, I think we can stop asking for news on this for a while as I found this comment from 2018-08-06:
https://github.com/Microsoft/vscode-cpptools/issues/16#issuecomment-

No news. We haven't started working on this yet. We plan to implement IntelliSense-based Go to Definition and Find All References before a Call Hierarchy, all of which requires implementing a multi-translation unit IntelliSense (our currently one is only one translation unit at a time).

@thernstig I think that comment is related to the c/c++ extension not generally to VS Code.

I need it desperately now - call graph or call hierarchy!

I agree, this is desperately needed

There now is a PR for this for the LSP
https://github.com/Microsoft/vscode-languageserver-node/pull/420

this feature will make our life easier.

I will explore this during the March milestone. Stay tuned.

UI-wise I think it could fit nicely with how "Find References" is done, but of course we would need to have a tree on the right hand side and the labels need to display the reference containing function definitions.

@svenefftinge The common UX for this seems to be a tree with a details pane on the side. Eclipse and IntelliJ are very similar, VS is a little different wrt calls/callers, e.g instead of the global direction toggle there is always nodes like 'Calls from "Foo"' and 'Calls to "Foo"'. That adds one more indent level for each symbol but it allows to see calls and callers at the same time. Something the Eclipse UI doesn't do:

Comments, ideas, preferences? Embedding this information as peek and/or viewlet should be doable. I was also thinking about a breadcrumb'ish approach, similar to the columns view of the Mac Finder, where a column represents all calls of a function and where expanding a call adds a new column.

@jrieken I think graph would be useful as well - for a quick glance when refactoring. Something like this extension has

@jrieken My preference would be for the Eclipse style of call hierarchy over the VS style.  The later, whilst more functional, I feel will get very noisy quickly; and in this situation I feel "less is more".
I Eclipse I am able to 'pin' a given call hierarchy such that I can have a few open at once, for me this deals with the uni-directional nature of the hierarchy.

My preference would be for the Eclipse style of call hierarchy over the VS style. The later, whilst more functional, I feel will get very noisy quickly; and in this situation I feel "less is more".

I was thinking along those lines but allowing all, e.g. "show calls",  "show callers", "show call & callers" (vs style). Think of the radio buttons in the Eclipse UI but in a way that both can be selected at the same time.

I was also thinking about a breadcrumb'ish approach, similar to the columns view of the Mac Finder, where a column represents all calls of a function and where expanding a call adds a new column.

Green Hills, an embeded systems compiler and IDE, implemented call hierarchy this way, in theory it sounds good but in practice it ends up being kinda clunky because it requires so much screen real estate, as it expands in both the X and Y directions, that it needed a separate window to navigate it. I much prefer Eclipse's implementation in this regard, as it can display below the code window as.
See this screen shot from the Green Hills IDE, illustrating this, (the blurred parts are from scrubbing the proprietary info)

UX: The plan is to have the call hierarchy as peek and as bottom panel. I have started with peek because it simpler. Two ideas:
A Tree & Preview like it's done in Eclipse, VS, et al

B Columns that shows calls with the actual source lines, where selecting a call drills into it

New UX proposal that stays closes to reference search peek. Have a tree on the right, have an editor on the left. The tree shows containers, e.g. functions calling the anchor function, with a hint of how often a function calls the anchor function. The editor itself highlight them.

Closing as we have a UI and proposed API for this. However, this time we have been faster than extension, e.g. no extension provides data yet and will take roughly one milestone until this feature lights up. Stay tuned

This is awesome.

Finally some hope for call hierarchy. Thank you all.

@zdog234 - Is this available for golang in vscode, i'm on the latest release and i'm unable to find this feature

@gandhiprachi I don't believe it's been implemented yet:
https://github.com/Microsoft/python-language-server/issues/888


When I build the environment, I encounter a error. Can you fix it?

× git clone --filter=blob:none --quiet git://github.com/erikwijmans/Pointnet2_PyTorch.git 'C:\Users\limaodaxia\AppData\Local\Temp\pip-install-uz5c90xe\pointnet2-ops_0
cea27ebf028479bb8eec4ff27e3a84f' did not run successfully.
│ exit code: 128
╰─> See above for output.

We didn't test the code on Windows platform, and we recommend you to use Linux system to try this project.
pointnet2-ops maybe not able to run on the Windows.

thank you，I solved the problem by changing the command. What I used is pip install "git+http://github.com/erikwijmans/Pointnet2_PyTorch.git#egg=pointnet2_ops&subdirectory=pointnet2_ops_lib"


Possible Sickrage improvements

[ ] Edit show modal (needs to be added to the api)
[ ] Status bar like sonarr has (needs to be added to api)

Also perhaps have shows be listed alpabetically the way it does in SR (e.g. The Big Bang Theory will be sorted under B, not T)

I can do that

And showing shows sorted A-Z by default as opposed to Z-A?

Isn't it sorted from A to Z today? I was sure it was. Anyway I'm working on some filter stuff.

@thejrose1984 the sorting is fixed in a branch in working on.


Use self-hosted runners

Attempt to use a "large" self-hosted runner for integration tests

Agreed, it was just a test to see what was blocking us from self-hosted runners.  thanks @bschimke95


Fixes a crash when allocating too much memory

We saw a crash in our application when using the collect operator related to either a overflow crash or a use-after-free memory corruption. After debugging the crash more I narrowed it down to the reserveCapacity line inside the collect implementation. When calling collect without any limit CombineX uses Int.max internally. This is somehow overflowing the array or causing the crash. Nevertheless, I think reserving this much memory upfront seems quite unreasonable.
@ddddxxx @luoxiu Let me know what you think about this change. This would be really important for us and it would be great if you can release a new version once this is merged :)

Confirmed Combine won't crash on pub.collect(.max) while CombineX will. I think it‘s worth releasing a new version for this. @luoxiu


invalid audio pts, packets dropped

I have been observing the playback of vdo files for quite a while. There seems to be quite a number of these invalid audio pts. Initially I thought it was caused by wifi problem or gpu problem. However, after pinging and adding gpu card, the issue remains. These vdo files will often get hiccups or stumbled from time to time. I took a sample from mpv playback of a file in this case. There seems to be quite a number received packets dropped while recording the video file.
Playing: rec/ch30_barn/2023-08-06/174024.mp4
(+) Video --vid=1 () (hevc 2560x1440 19.569fps)
(+) Audio --aid=1 () (aac 1ch 16000Hz)
AO: [pulse] 16000Hz mono 1ch float
VO: [gpu] 2560x1440 yuv420p
AV: 00:00:00 / 00:00:46 (1%) A-V:  0.000
Invalid audio PTS: 0.391937 -> 0.589500
AV: 00:00:00 / 00:00:46 (2%) A-V: -0.118 Dropped: 1
Invalid audio PTS: 1.190500 -> 1.832250
AV: 00:00:02 / 00:00:46 (6%) A-V: -0.107 ct: -0.075 Dropped: 3
Invalid audio PTS: 2.984688 -> 3.218313
AV: 00:00:03 / 00:00:46 (7%) A-V: -0.113 ct: -0.068 Dropped: 3
Invalid audio PTS: 3.466625 -> 3.574625
AV: 00:00:03 / 00:00:46 (8%) A-V: -0.093 ct: -0.084 Dropped: 3
Invalid audio PTS: 3.790812 -> 3.893375
AV: 00:00:05 / 00:00:46 (11%) A-V: -0.081 ct: -0.130 Dropped: 3
Invalid audio PTS: 5.357875 -> 5.549500
AV: 00:00:06 / 00:00:46 (13%) A-V: -0.014 ct: -0.147 Dropped: 3
Invalid audio PTS: 6.430438 -> 7.059000
AV: 00:00:06 / 00:00:46 (13%) A-V: -0.014 ct: -0.149 Dropped: 3

I am no expert on this but FFmpeg simply copies the audio from the stream so the issue is likely your camera.
What you can try is to set audio_codec under recorder to something like aac which will make FFmpeg re-encode the audio and maybe fix the pts issue

Every camera working now is aac audio and I have tried your suggestion and it made no difference.
I just found an issue where it discussed audio and video pts not synchronized. I am no expert on any subject here but what I can imagine is the synchronization if any will be at the time it created or it played. The other source of non-synchronization could be at the time the system received audio and video. However, when I put the rtsp on real-time stream I don't see any issue. Still this is a puzzle to me.
https://github.com/mpv-player/mpv/issues/9217

Both use_wallclock_as_timestamps and avoid_negative_ts are used in the default FFmpeg command when saving segments to disk.
When a recording is finished, Viseron concatenates these segments into a single mp4.
Maybe the problem occurs during the concatenation.
What you can do is to play around with the concatenation command by setting output_args under recorder.


S2 plus, can't download pit

Opensuse 13.1 x86_64
Heimdall 1.4.1
libusb 1.0.20
Samsung S2 plus GT-I9105P
Android 4.2.2
version de la bande base : I9105PXXUBNG2
version kernel : 3.0.31-2705991
version : JDQ39.I9105PXXUBNG1
the s2 plus is in downloading mode
i launch as root heimdall front end
then
i click on detect
result : "detected device"
smartphone is detected
then
i  indicate where to save the pit
then
i click on "download"
result :
pit file is empty
and
the log is

Heimdall v1.4.1
Copyright (c) 2010-2014 Benjamin Dobell, Glass Echidna
http://www.glassechidna.com.au/
This software is provided free of charge. Copying and redistribution is
encouraged.
If you appreciate this software and you would like to support future
development please consider donating:
http://www.glassechidna.com.au/donate/
Initialising connection...
Detecting device...
Claiming interface...
Attempt failed. Detaching driver...
Claiming interface again...
Setting up interface...
ERROR: Setting up interface failed!
Releasing device interface...
Re-attaching kernel driver...

with adb tool i can explore for example the / partition or the extsdcard of the phone

with Samsung Tab2 10.1 wifi
it does not detect the tablet . i get an error message :
ERROR: Failed to detect compatible download-mode device

good news
with heimdall and usb things from http://download.opensuse.org/repositories/hardware/openSUSE_13.1/x86_64
i can save pit of my S2 plus


Minor editorial changes to the landing page

Mostly punctuation, capitalization, rewording a few things to be more concise, and similar.

Just rebuild the locales and it worked like a charm. There is apparently a fuzzy matching against old msgid's which allows to mostly keep the translations correctly in place when updating the po files. All fuzzy matched msgid's will get marked and can be reviewed and updated, all non-matching msgid's are retained and can be reused to by the translator to update the msgtxt entries later. Really like the way gettext works in this.
However, if the po-files are not rebuild the msgid don't match anymore and the translation will be almost completely lost. I wonder whether we can automate this (the po=files have a timestamp, which makes it difficult to discover changes by running the gettext target and just rely on the existence of diffs). Have to think about something better.


How can I give different loss weight for each class?

Tku in advance.

If you have updated to the latest version, you can just modify the loss_weight in config files.


Implement hasPrefix and hasSuffix

Using the _compareDeterministicUnicodeCollation method, also used by the == operator.

I think this might be close enough, although inefficient (traverses both strings twice).
This change requires tests.  We have tests for the Objective-C side in test/1_stdlib/NSStringAPI.swift and validation-test/stdlib/String.swift, please port those to start.
It would make sense to put the tests into validation-test/stdlib/String.swift, using #if _runtime(ObjC) to separate Objective-C dependent parts and un-XFAIL the test on Linux.

On 11 déc. 2015, at 16:30, Dmitri Gribenko<EMAIL_ADDRESS>wrote:
I think this might be close enough, although inefficient (traverses both strings twice).

There’s no claim of efficiency implied, but it could be worse; I’m fairly sure no new memory is allocated.

This change requires tests. We have tests for the Objective-C side in test/1_stdlib/NSStringAPI.swift and validation-test/stdlib/String.swift, please port those to start.
It would make sense to put the tests into validation-test/stdlib/String.swift, using #if _runtime(ObjC) to separate Objective-C dependent parts and un-XFAIL the test on Linux.

Thanks for pointing me to those.
Guillaume Lessard

We should be careful with "close enough" implementations because the expectation is that it will behave the same as the Darwin version.

We have Foundation available on all platforms now. Can we make this use that implementation on all platforms as well?

@parkera That will create interesting circular dependencies in the build process.

If the build process is deficient then we should fix that instead of forcing an incompatibility onto clients of the API.

Or we do the obvious thing and put the implementation in an extension on String in Foundation.

@gribozavr The relevant test is ported from test/1_stdlib/NSStringAPI.swift.
Neither the original test nor the ported test are particularly satisfying, since ultimately the operation is tested with a different version of itself.

@parkera From what I ran into while doing this, I'd advocate to eliminate any way to reach Foundation-based comparisons from Swift.String. If a variable is being used as NSString or CFString, it's okay to use them, but otherwise there is at least one corner case where the Foundation-based comparison is not consistent with the Swift.String model. See https://bugs.swift.org/browse/SR-243

If we want to discuss dropping the NSString methods from String, that would be something to discuss on the evolution list. In the meantime, my primary concern is making sure developers don't get different answers for the same methods on Linux vs OS X.

On Dec 16, 2015, at 10:13 AM, Tony Parker<EMAIL_ADDRESS>wrote:
If we want to discuss dropping the NSString methods from String, that would be something to discuss on the evolution list. In the meantime, my primary concern is making sure developers don't get different answers for the same methods on Linux vs OS X.

+1
-Dave

After porting the corrected test from https://github.com/apple/swift/pull/629, the Foundation-based calls no longer pass, but the ICU-based calls do. Not sure where to go from here.

Would you like a consolidated PR with this?

Incidentally, it turns out that _compareDeterministicUnicodeCollation actually bridges into ObjC if _runtime(_ObjC). I don't know why it does this at all, unless _compareDeterministicUnicodeCollation doesn't actually match NSString behavior (but you'd think it would). This means that anything that uses _compareDeterministicUnicodeCollation ends up actually allocating a new _NSContiguousString if the string in question isn't already backed by a Cocoa string.

On Jan 3, 2016, at 4:00 PM, Kevin Ballard<EMAIL_ADDRESS>wrote:
Incidentally, it turns out that _compareDeterministicUnicodeCollation actually bridges into ObjC if _runtime(_ObjC). I don't know why it does this at all, unless _compareDeterministicUnicodeCollation doesn't actually match NSString behavior (but you'd think it would). This means that anything that uses _compareDeterministicUnicodeCollation ends up actually allocating a new _NSContiguousString if the string in question isn't already backed by a Cocoa string.

Except maybe on Linux, every string is backed by a Cocoa string.  The Swift-native String backing buffer is-a NSString.
-Dave

@dabrahams I don't see how that's possible. The Swift-native String buffer is a _StringBuffer, which uses a _HeapBuffer<_StringBufferIVars, UTF16.CodeUnit> as its storage, which in turn uses _HeapBufferStorage<_StringBufferIVars, UTF16.CodeUnit>, which is a class that inherits from NonObjectiveCBase. It doesn't seem possible for this to be layout-compatible with an NSString, because Swift classes always have the type info and retain counts at the beginning of their allocated storage (which is precisely where the isa pointer needs to go for Obj-C). Plus of course _StringBufferIVars doesn't even attempt to include an isa pointer.
You can also trivially demonstrate that a Swift string is not Cocoa-compatible with the following code:
var s = "test"
s += "!" // force it to copy to mutable memory
print(s._core.nativeBuffer)
print(s._core.cocoaBuffer)

If it was a Cocoa string, it would print nil and then the Cocoa string. But it's not, it's a native string, so it prints the _StringBuffer and then it prints nil. And since _stdlib_binary_bridgeToObjectiveCImpl only avoids creating a new _NSContiguousString if _core.cocoaBuffer is non-nil (and _swift_stdlib_CFStringGetLength reports the same length as _core.count, not entirely sure why), this should demonstrate that Swift-native strings always end up creating a new _NSContiguousString whenever they bridge into Obj-C.
So this all comes back to the question of why String bridges into Obj-C when it has a perfectly functional non-ObjC implementation available. It would make sense to bridge if it already has a Cocoa buffer, but it doesn't test that, it merely tests #if _runtime(_ObjC).
Incidentally, there's a bit of even more weirdness here. When comparing two strings with _runtime(_ObjC), if both strings are ASCII, it falls back to a separate non-ObjC comparison routine that's basically a wrapper around memcmp. The documentation says that Unicode actually defines the sort order differently for some punctuation than the ASCII value order, but that Foundation uses the ASCII value order. This is strange because it suggests that two Swift strings can actually compare differently depending on whether the ObjC runtime is present (e.g. "$" < "*" prints true with the Obj-C runtime but it should print false with the root collation order from Unicode). What's even odder about this is the Unicode Collation Algorithm can actually be tailored, and in fact the en_US_POSIX locale (also known as the C locale) already tailors the collation specifically to make ASCII characters sort in ASCII value order. So I'm surprised we don't just use an ICU collator with the collation table tailored to make ASCII characters sort in ASCII order the same way en_US_POSIX does (I assume en_US_POSIX inherits other collation tweaks from en_US or else I'd suggest just using en_US_POSIX directly, although I don't understand ICU well enough to know if that's actually true).

As far as I can tell, the OSX swift build doesn't link ICU at all. I attempted earlier to bypass the call to _stdlib_compareNSStringDeterministicUnicodeCollation within _compareDeterministicUnicodeCollation (by inserting #if false at line 371 of String.swift); that resulted in a link error.

@glessard, really nice implementation! 👌
I was just wondering which one is more Swift-y at the hasPrefix function?
if prefix.isEmpty { return false }

or
guard !prefix.isEmpty else { return false }

Any thoughts?
Same goes for hasSuffix.

@adrfer I'm sure it could be argued both ways, but I think it is a special condition rather than a precondition. If this eventually called into CFStringFind, the if statement could conceivably be removed; this eventually calls into CFStringCompare, and two empty strings would compare as equal, which is not the desired answer. (Plus, guard leads to a double-negative.)

@glessard indeed! I'm just trying to see what's more idiomatic. In this case guard's double negative isn't really cool, and its version is even longer than the if version. Anyways, Swift is evolving fast and looking forward to seeing what preferred style the community will come up with.

Rebased and re-submitted as https://github.com/apple/swift/pull/1031

You know, you can just force-push to your branch and GitHub will update the PR. That way all the previous discussion will be preserved.

@kballard I didn't know that could work. It doesn't look like I can reopen, though.

Rebased here. Thanks for letting me know this is possible, @kballard

@glessard Given the current implementation, do you think this pull request is still necessary/relevant?

@CodaFi Good question. I was considering re-implementing the tests part of this based on the newer work; it would at least illuminate some differences between an ICU-based implementation and the CFString-based one. I think the discussion is still of interest for the (hopefully) forthcoming String rethink.

Sounds good; we'll always take more tests.  To preserve the discussion here, I'm going to close this pull request.  Please open another if you decide to cherry pick the tests.


Wiki: Update IntelliJ setup Guide

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[x] I have checked the branches or the maintainers' PRs for upcoming features.

Feature Request
The wiki page about setting up a JDA project within IntelliJ is a bit outdated due to more recent changes in the 2019 version of IntelliJ.
Changes:
Step 5 (Setting group and artifact id) got changed. They are now in the very first page, where you set your project name and are hidden under "Artifact coordinates".
It is also noteworthy that the Artifact name may not be altered anymore, even if it would only be capitalization.
You are no longer able to enable auto-import during the creation of the project (at least I couldn't find any settings in the window) and have to do this after the project was created by clicking on the option in the notification on the bottom right.
This makes step 6 useless.
I would suggest making step 9 a bit more neutral by changing the shown version of JDA (Currently 3.8.3_464 in this case) to a placeholder text similar to that on the readme.
Step 10 is no longer needed as Gradle projects now have a src-directory created, containing the main/java, main/resources and test/java-directory created after the project was synced/indexed.
I would also give the tip, that you can select "New -> Java Class" and then f.e. type me.name.bot.Bot to create the path me.name.bot with the class Bot.java, essentially combining steps 11 and 12 (Small QOL improvement that removes an additional step to create a path first)
Screenshots:
New window for creating a project, after selecting "Gradle" in the previous one

The project with the src/main/java directory

Example Use-Case
-

you do know that changing the artifact name is as easy as modifying the pom.xml / build.gradle, right?
Other than that, ill take a look at it

I talk about changing it in the window itself, where you create the project.
Of course, you can change it after creating the project, but not before that.

Updated the page a bit. let me know what you think of the new one:
https://github.com/DV8FromTheWorld/JDA/wiki/2)-IntelliJ-IDEA-Setup

lgtm
The only thing that shouldn't be necessary is step 8 as mentioned above does IntelliJ now already create an src/main/java directory with a Gradle-project... Or are there still cases where this doesn't happen?

dunno
it did that way before updating for me already, but i left it in cuz idk why it was in to begin with

IntelliJ 2018 didn't create those folders for whatever reason so you had to create them yourself, which is why this step is most likely in this guide.

Looks good for me now.


.NetStandard - Interception works 1 level, but not 2 happen again

i think this issue happen again https://github.com/unitycontainer/interception/commit/e68493659b67e7c466bcb10f12fbdffdf6503a89 in <IP_ADDRESS> was rollback this code.

PR opened:
https://github.com/unitycontainer/interception/pull/25


⚠ Bofrost.lu has degraded performance

In f1b32bc, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2756 ms

Resolved: Bofrost.lu performance has improved in fe6d8bb after 6 minutes.


Bring directories into final order

seems like the submodules got lost by accident

Can you directly modify https://github.com/backstopmedia/deep-learning-browser/blob/master/.gitmodules and see if that works? If it doesn't work I'll look up the steps to move submodules and reinclude them myself.

What's up with this pull request ?


test_diff_actions fails for clang as part of make test

Noticed when testing different clean Docker CI configs based on Ubuntu 20.04; and repeatable across dozens of runs:

all unit tests pass with gcc
with clang, test_diff_actions repeatedly fails as part of make test (The following tests FAILED: 15 - test_diff_actions (Child aborted). When running the executable independently (./build/unittest/test_diff_actions), it passes and works fine.

I have observed this issue locally, but I don't what it's going on.
Could this be related with the linked boost unittest library?
I appreciate any feedback in this topic.

I found the source for part of this issue (test_diff_actions, haven't started debugging test_actions yet) and it's in
https://github.com/loco-3d/crocoddyl/blob/43502019fe5f744d0e20588815625467c69aab68/include/crocoddyl/multibody/contacts/multiple-contacts.hxx#L63-L84
In particular in how the active_ and inactive_ elements are updated. They are currently defined as std::vector<std::string>. I'd like to propose to change them to either std::set<std::string> if we care about order or std::unordered_set<std::string> to simplify the code and reduce the possibility of bugs in the logic. What do you think of this API change? The runtime complexity for inserting into/removing from a set is slower, but at the size of the number of contacts it shouldn't really matter

#1025 deactives the test_diff_action unittest when compiled with clang. It can pass locally but still fails on the buildserver. Since we are increasing coverage on the buildserver and it's tested with g++, let's keep it deactivated for now and this issue open until we can find a proper fix.


Find another tool for deduplication (that operates similarly to Clumpify)

Clumpify, the current tool that we use for deduplication, isn't able to tell us duplicate groups without changing the source code. We'd like to be able to find another deduplication tool that clusters reads then calls duplicates based on some percentage of mismatches (most current tools use exact matching which doesn't factor in sequencing errors). We're going to try to find other tools, and may consider making our own deduplication tool, however the latter would be a lot of work.

As of 11/13/2024, there is no better tool. I'm moving this down to p3.


Fix MS Store Broker Authentication setting failure

The CI Create and push the Application Payload to Package Flight on Microsoft store job uses Microsoft's StoreBroker to submit a package to Microsoft store's Alhpha Test Flight, but it fails at Set-StoreBrokerAuthentication now.
Here is the log:
##[debug]Evaluating: secrets.MSPARTNER_TENANTID
##[debug]Evaluating Index:
##[debug]..Evaluating secrets:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'MSPARTNER_TENANTID'
##[debug]=> '***'
##[debug]Result: '***'
##[debug]Evaluating: secrets.MSPARTNER_CLIENTID
##[debug]Evaluating Index:
##[debug]..Evaluating secrets:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'MSPARTNER_CLIENTID'
##[debug]=> '***'
##[debug]Result: '***'
##[debug]Evaluating: secrets.MSPARTNER_CLIENTPWD
##[debug]Evaluating Index:
##[debug]..Evaluating secrets:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'MSPARTNER_CLIENTPWD'
##[debug]=> '***'
##[debug]Result: '***'
##[debug]Evaluating condition for step: 'Create and push the Application Payload to Package Flight on Microsoft store'
##[debug]Evaluating: success()
##[debug]Evaluating success:
##[debug]=> true
##[debug]Result: true
##[debug]Starting: Create and push the Application Payload to Package Flight on Microsoft store
##[debug]Loading inputs
##[debug]Loading env
Run Set-PSRepository PSGallery -InstallationPolicy Trusted
  Set-PSRepository PSGallery -InstallationPolicy Trusted
  Install-Module -Name StoreBroker

  $APP_ID = "9PDLJFW5D4JR"
  $SubmissionRoot = "MSSubmission"

  $PWD = ConvertTo-SecureString $env:MS_CLIENT_PWD -AsPlainText -Force
  $Credentials = New-Object System.Management.Automation.PSCredential ($env:MS_CLIENT_ID, $PWD)
  Set-StoreBrokerAuthentication -TenantId $env:MS_TENANT_ID -Credential $Credentials -Verbose

  New-Item -Path "MSAppPayload" -ItemType Directory

  # Get current config and PDPs from Microsoft store
  New-StoreBrokerConfigFile -Path "MSAppPayload/SBConfig.json" -AppId $APP_ID
  $release_str = Get-Date -Format "yyyyMMdd"
  .\StoreBroker\Extensions\ConvertFrom-ExistingSubmission.ps1 -AppId $APP_ID -Release $release_str -OutPath "MSAppPayload/PDPs"

  # Wipe out the Release string in PDP.xml to avoid the release version folder
  [xml]$xmlDoc = Get-Content "MSAppPayload/PDPs/en-us/PDP.xml"
  $xmlDoc.ProductDescription.Release = ""
  $xmlDoc.Save( "MSAppPayload/PDPs/en-us/PDP.xml" )

  echo "Going to create dummy screenshots for Application Payload creation:"
  $xmlDoc.ProductDescription.ScreenshotCaptions.Caption | ForEach-Object {
    $dummy_screenshot = "MSAppPayload/PDPs/en-us/" + $_.DesktopImage
    echo $dummy_screenshot
    echo NULL > $dummy_screenshot
  }

  New-SubmissionPackage `
    -ConfigPath "MSAppPayload/SBConfig.json" `
    -PDPRootPath "MSAppPayload/PDPs" `
    -ImagesRootPath "MSAppPayload/PDPs" `
    -AppxPath packages/*.appx `
    -MediaFallbackLanguage en-US `
    -OutPath $SubmissionRoot `
    -OutName endlesskey `
    -Verbose

  $FLIGHT_NAME = "Alpha Test Flight"
  $flight = Get-ApplicationFlights -AppId $APP_ID | Where-Object { $_.friendlyName -eq $FLIGHT_NAME }

  Update-ApplicationFlightSubmission `
    -AppId $APP_ID `
    -FlightId $flight.flightId `
    -SubmissionDataPath "$SubmissionRoot/endlesskey.json" `
    -PackagePath "$SubmissionRoot/endlesskey.zip" `
    -ReplacePackages `
    -AutoCommit `
    -Force `
    -Verbose
  shell: C:\Program Files\PowerShell\7\pwsh.EXE -command ". '{0}'"
  env:
    MS_TENANT_ID: ***
    MS_CLIENT_ID: ***
    MS_CLIENT_PWD: ***

##[debug]C:\Program Files\PowerShell\7\pwsh.EXE -command ". 'D:\a\_temp\54358b45-5589-4f63-94e7-cec867[2](https://github.com/endlessm/endless-key-app/actions/runs/5049617230/jobs/9065651238#step:5:2)[3](https://github.com/endlessm/endless-key-app/actions/runs/5049617230/jobs/9065651238#step:5:3)8[4](https://github.com/endlessm/endless-key-app/actions/runs/5049617230/jobs/9065651238#step:5:4)1e.ps1'"
VERBOSE: 2023-0[5](https://github.com/endlessm/endless-key-app/actions/runs/5049617230/jobs/9065651238#step:5:5)-23 02:54:4[7](https://github.com/endlessm/endless-key-app/actions/runs/5049617230/jobs/9065651238#step:5:7) : runneradmin : [1.21.0] Executing: Set-StoreBrokerAuthentication -TenantId "***" -Credential {"UserName":"***","Password":{"Length":44}} -Verbose:$true
VERBOSE: Performing the operation "Cache tenantId" on target "***".
VERBOSE: Performing the operation "Cache credential" on target "System.Management.Automation.PSCredential".
VERBOSE: Performing the operation "Clear cached access token" on target "".

    Directory: D:\a\endless-key-app\endless-key-app

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d----           5/23/2023  2:54 AM                MSAppPayload

|  Elapsed: 0 second(s) [Getting access token]
/  Elapsed: 0 second(s) [Getting access token]
-  Elapsed: 0 second(s) [Getting access token]
\  Elapsed: 0 second(s) [Getting access token]
|  Elapsed: 0 second(s) [Getting access token]
/  Elapsed: 1 second(s) [Getting access token]
-  Elapsed: 1 second(s) [Getting access token]
DONE (FAILED) - Operation took 1 second(s) [Getting access token]
Write-Error: C:\Users\runneradmin\Documents\PowerShell\Modules\StoreBroker\1.21.0\StoreBroker\PackageTool.ps1:373
Line |
 373 |          Write-Log -Message "Encountered problems getting current appl …
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | 2023-05-23 02:54:4[9](https://github.com/endlessm/endless-key-app/actions/runs/5049617230/jobs/9065651238#step:5:9) : runneradmin : Encountered problems getting current application submission values:
     | Write-Log: [20](https://github.com/endlessm/endless-key-app/actions/runs/5049617230/jobs/9065651238#step:5:20)[23](https://github.com/endlessm/endless-key-app/actions/runs/5049617230/jobs/9065651238#step:5:23)-05-23 02:54:49 : runneradmin : Be sure to check that your client id/secret are valid. Response
     | status code does not indicate success: [40](https://github.com/endlessm/endless-key-app/actions/runs/5049617230/jobs/9065651238#step:5:40)1 (Unauthorized).  :

Error: Process completed with exit code 1.
##[debug]Finishing: Create and push the Application Payload to Package Flight on Microsoft store

Visit Automate uploading Endless Key app to Microsoft Store again!

Err, after test locally, the error comes from New-StoreBrokerConfigFile

Did we change "GitHub Actions"' password on MS store?  Or, it has a valid date?
@wjt Can you please help check the user "GitHub Actions"'s status on MS store?
In Microsoft Partner Center -> Account settings -> User management -> Azure AD application
Thanks!

Ah! It expired on Saturday, 13 May 2023.

I've generated a new key, updated it in Vault, and am redeploying…

Try again now!

Thanks!  The CI is happy now! https://github.com/endlessm/endless-key-app/actions/runs/5049617230

Added maintenance description of "GitHub actions" into Release to Microsoft's Alpha Test Flight.


enabled GPS on E290 -> PIN47/48 like on helltecV3

enable external GPS on E290
same GPIOS like in Helltec V3
tested it -> works

Please test again with original U0RXD (GPIO 44) and U0TXD (GPIO 43) Pins.

the GPS is using software-based serial and not the hardware UARTS...so i used "normal gpios"....like in helltecV3
So i kept the real UART (U0RXD and U0TXD) free.
Do you want to use the hardware-Uart and enable it?

OK, i follow you and we decide to use PIN 47/48.


Are we able to validate types using regex?

How can I validate a param type? Do I need to run the elixir Regex module separately for this or it's possible to check for existence and also against a format regex?

Yes, just define a module, use Params.Schema inside of it and define a changeset method (same as you do in your models) and make this method use Ecto.Changeset.validate_format


Dedicated ML/heavyweight container for remote processing

TL;DR
Suggestion: move machine learning and other heavyweight processing into a dedicated container. This will decouple those processes from the backend, allowing users to run them on a more powerful machine in cases where the media is stored on low-end hardware.
Similar to Immich's Remote Machine Learning.
Explanation
Let me preface this by saying that facial recognition and similar features are what I look for most in a self-hosted phot management software.
In that section, I think that Librephotos shines the brightest among other popular choices such as Immich or Photoprism.
There is one missing feature in Librephotos though that prevents me from using it for my large photo collection: Remote Machine Learning.
All my photos are stored on my NAS which has lots of storage space, but very low processing power. Besides that, I have a desktop PC with low storage, but very powerful hardware.
Now since my photo/video collection exceeds the storage space on my desktop PC by a large margin, I need to run it on the NAS. This means that indexing photos in Librephotos is very slow since I need to resort to the low-end NAS hardware.
Immich has a "feature" called "Remote Machine Learning" which is essentially just the machine learning container being decoupled from the backend.
This allows me to index my Immich photo collection on my desktop PC, finishing after about ~3h whereas the same collection takes ~12h in Librephotos, running on the NAS.
Yes, this has the downside that media files need to transferred away from the storage location for processing, but the higher the disparity between hardware capabilities in the storage and the processing machine, the more effective it becomes.
The complexity of implementing this change depends on how tightly backend and heavyweight processing code is coupled in Librephotos, so hopefully it won't cause too much trouble...

Thanks for the swift reply!
Running a full rescan now so I can give you accurate metrics. Will take some time though, I'll keep you posted.

In the meantime, I have a question:
You said:

Tasks, which should be externalized, are tasks which take longer than the network latency. This would currently be LLM generation, calculating clip_embeddings, calculating face embeddings and finding face locations, generating thumbnails for RAW images, image captioning and tags.
I do not think that generating thumbnails for regular images, would benefit from this approach, as the transfer of the image takes probably longer than calculating it on the slower machine. The same is true for extracting EXIF data and reverse geocoding images. Video processing could maybe benefit, but they have to be externalized as a separate service, which is currently not the case.

In my eyes, this isn't a question of which tasks should be externalized or not. This is a yes/no question.
If I configure my LibrePhotos installation so that it uses an external processor on a powerful machine, why would I only want to run the long-running tasks on that machine? The images have to be transferred over the network for the heavyweight tasks, sure, but once they're there, we can run all the tasks on that machine.
This seems like the most logical approach to me and also feels liek it's easier to implement and to maintain rather than deciding on which tasks to run where.

Unfortunately, I can't report proper runtimes due to execution errors in the jobs, an unrelated issue.
Can you support the feature request without those details?

If your goal is to run all processing tasks on one machine while saving the database and thumbnails on another, you might consider running the database and storing thumbnails on your NAS, while hosting LibrePhotos and the frontend on the desktop PC.
The immich feature only externalizes a couple of tasks e.g. smart search and face detection, but all other processing of the image are done on the original server.
Our backend has the following architecture: Django, with a background task system and API that communicates with the frontend and then flask services which run on different features e.g. LLM generation, clip embedding etc.
Currently, the flask services already communicate over a REST API and are decoupled, which makes it easy to put them on another machine and then call them for certain image processing tasks.
Processing the whole image and saving it to the database and NAS, would mean, that we have to support scaling horizontally with multiple machines, but I do not know if our background task system supports that.

Running the front- and backend on my desktop PC doesn't work well for me unfortunately.
I want the frontend to be available 24/7, without having to boot up my PC anytime I want to access LibrePhotos.
Since I only run the indexing jobs for LibrePhotos manually when I add new photos, in that case I can boot up my PC, let the indexing run and shut it down again.
All other components need to run permanently for LibrePhotos to work as far as I understand. The heavyprocess jobs are the only component I can decouple from the rest, that's where this feature request originated from.
If your backend doesn't support that, then I'm ok with that. Feel free to close this issue in that case.


Add i18n to app

Right now the app is only available in English. If we can reuse the po files generated for the main website, we can offer i18n pretty easily inside of the app.
Example: https://github.com/Glimesh/glimesh.tv/blob/dev/priv/gettext/pt_br/LC_MESSAGES/default.po

Looks like there's a library someone wrote for doing this: https://pub.dev/packages/gettext_i18n, been having a play around today and it's working nicely, however I'm not sure how we handle the (few) strings that are in the app and not in the main website or are different between the two?

however I'm not sure how we handle the (few) strings that are in the app and not in the main website or are different between the two?

I guess I was hoping we could modify all the text in the app to be the same as on the website. This solution wont scale forever, but for now it should work given the parallels between the two platforms.

yeah, there's a few places locally where I've done that, do you want me to push a branch so you can have a look?

Sounds good to me!

pushed to the ponkey-test-i18n branch

Looks super straightforward!

@ponkey364 want to turn this into a PR?

yeah sure, will tidy it up a bit and wrap everything (not just the strings that exist in the PO files) in context.t.
Would it be possible to get the translations synced between POEditor and the glimesh.tv repo so I can make sure we start with an up to date version?
Also, which languages should we include?
(will probably do this at the weekend)

Would it be possible to get the translations synced between POEditor and the glimesh.tv repo so I can make sure we start with an up to date version?

Wonder if we can submodule the i18n file from Glimesh? I would assume all languages we include on the regular website, but we can always start with English and Spanish if that makes sense to test.

We'll need some kind of selector unless we rely on system language too.

Wonder if we can submodule the i18n file from Glimesh? I would assume all languages we include on the regular website, but we can always start with English and Spanish if that makes sense to test.

I'll try the submoduling, I think that's the best idea. I'll set it up to be all the languages we currently support, although we have at least one that Flutter doesn't support, so I'll exclude that.

We'll need some kind of selector unless we rely on system language too.

Yeah, I was thinking we do both: use the system language, but allow the user to override it in our settings menu?


kmmo: load mei driver for media support

Load the mei driver which is required to support the media feature. But before loading make sure the in-tree media driver is unloaded. modprobing mei_gsc load all other necessary media drivers.

LGTM


CodingChallenges / Fluid Simulation / P5 (function renderV fix)

Description
This was probably not noticed before because the main loop does not use this function at all.
-removed leftover 3rd dimention variable "k" from line 88 and 89
-removed leftover "this.canvas." from line 90

Community Contribution
You can try the fix here

Sharing

[X] I would be happy to have my project shared on The Coding Train's social media!

Instagram:
@leithba
Twitter:
@leithba

This makes sense to me. What do you think @shiffman ?

I think this was forgotten 😅

@LeithBA it was indeed! Merged!


Custom package.json path support

I'm running a project where the Vue setup is inside a PHP project and therefore - the package.json file is not in the root of the workspace. VS Code works fine, since it supports scanning recursively for package.json, but this extension says it cannot detect it. So, I'd like to request either a setting, so I can set a custom path for package.json (or let's say a relative custom root would be maybe even better) or make the extension scan for package.json recursively, like VS Code does itself.

@lllopo Thanks for the advice! I would definitely consider that.
Before any implementation landed, you can set "vue-i18n-ally.forceEnabled": true in settings to make it works. (please be sure to update to the leasest version)

Yeah, I already tried this, but with no success. Maybe there are other stuff in my project, that are not compatible or something.


Primal Bluesky Brige IMETA incompatibility

client initiating action
Primal - no IMETA
client receiving or observing action (or lack thereof)
Bluesky Bridge
timestamp
https://github.com/PrimalHQ/primal-server/issues/6
what happens
suggestion
Add IMETA support: https://github.com/nostr-protocol/nips/blob/master/92.md

Thanks for the report @alexgleason
Let us know if there are other obstacles to emancipating bleu sky from their self-imposed thought prison.

@MarkoBraticevic

Thank you @alltheseas!
I think it would also be helpful to have a client matrix of NIP-92 support.

Thank you @alltheseas!
I think it would also be helpful to have a client matrix of NIP-92 support.

https://github.com/nostrability/nostrability/issues/133


Ubiquitin-dependent protein catabolic process via the destruction via C-end degron rule pathway

Hi GO,
I would need a new GO term for the C-end degron rule (DesCEND ) pathway, an equivalent of the N-end rule pathway but for the C-terminus of proteins
Thanks
Sylvain
Name: ubiquitin-dependent protein catabolic process via the destruction via C-end degron rule pathway
Synonym: DesCEND (exact)
Category: Biological Process
Definition: The chemical reactions and pathways resulting in the breakdown of a protein or peptide covalently tagged with ubiquitin, via the DesCEND (destruction via C-end degron) pathway. In the DesCEND pathway, C-terminal residues (C-end degrons) in substrates are recognized by Cul2-RING and Cul4-RING E3 ligases, whereupon the substrates are linked to ubiquitin and then delivered to the proteasome for degradation. C-end degrons can be present in full-length proteins, truncated proteins or proteolytically cleaved forms.
PMID:29779948
PMID:29775578
Child of GO:0043161 proteasome-mediated ubiquitin-dependent protein catabolic process

Are you sure we need 2 'via' in the term name?
how about just 'ubiquitin-dependent protein catabolic process via the C-end degron rule pathway', to model on the N-rule term?

+[Term]
+id: GO:0140627
+name: ubiquitin-dependent protein catabolic process via the C-end degron rule pathway
+namespace: biological_process
+def: "The chemical reactions and pathways resulting in the breakdown of a protein or peptide covalently tagged with ubiquitin, via the DesCEND (destruction via C-end degron) pathway. In the DesCEND pathway, C-terminal residues (C-end degrons) in substrates are recognized by Cul2-RING and Cul4-RING E3 ligases, whereupon the substrates are linked to ubiquitin and then delivered to the proteasome for degradation. C-end degrons can be present in full-length proteins, truncated proteins or proteolytically cleaved forms." [PMID:29775578, PMID:29779948]
+synonym: "DesCEND" EXACT []
+is_a: GO:0043161 ! proteasome-mediated ubiquitin-dependent protein catabolic process
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/21184 xsd:anyURI
+created_by: pg
+creation_date: 2021-03-24T07:30:47Z

Hi Pascale
thanks for correcting the name
Cheers
Sylvain


Remove session key of reaped accounts

Currently set_key is a public call that checks that the origin is a signed account using ensure_signed https://github.com/paritytech/substrate/blob/master/srml/session/src/lib.rs#L97, and then stores in NextKeyFor a mapping of their account id to the session key that they provided, which will be used as their validator session key in the next session.
But since we're using ensure_signed, the account id of the origin that we're storing in NextKeyFor may not necessarily be associated with a block author or a validator.
Shouldn't set_key instead be an inherent call that checks that the origin is a block author or a validator using ensure_inherent instead?

You may want to set your key prior to becoming a validator. It should, however, be removed once the account has been deleted.

Session module does indeed remove the key once the account balance is zero.


Cluster API cannot access the spark-operator webhook service endpoint on port 8080

Description
I utilized the Terraform module 'terraform-aws-eks' to provision an EKS cluster with version 1.23. However, I encountered a timeout issue when the API server attempted to access the spark-operator webhook service endpoint. What I found is that the ports between the cluster API and the nodes were not open within the cluster, which is not expected when you are inside the cluster.

[x ] ✋ I have searched the open/closed issues and my issue is not listed.

Versions

Module version: v19.12.0

Reproduction Code [Required]

examples/eks_managed_node_group/main.tf

Steps to reproduce the behavior:

Provision an EKS cluster using node groups.
Check the security groups and see if the Cluster API can access the service endpoints (1 to 65535)

Expected behavior
Cluster API must be able to access the service endpoints
Actual behavior
Cluster API is not able to access the service endpoints
API server logs
681315700000,I0412 16:08:20.134289      10 dialer.go:37] [6f5dcf0368464344: <IP_ADDRESS>:8080] Dialing...
1681315710000,E0412 16:08:30.125227      10 writers.go:117] apiserver was unable to write a JSON response: http: Handler timeout

port 8080 is not enabled by default, you would need to add a rule to enable that access https://github.com/terraform-aws-modules/terraform-aws-eks/blob/master/docs/network_connectivity.md#security-groups

Hi @bryantbiggs,
Is there a security reason for now allowing Cluster API to access that specific port?

did you read the linked docs?

Hi,
I read the document, is pretty clear, however from a k8s user point of view, the expectation was to have the required access when you are inside the cluster, and when I say inside, I refer to K8s components.
You can close the ticket, the documentation will help me to have a starting point.
Thanks

for a stronger security posture, and because its easier to open up access as opposed to remove access, the module defaults to starting with the bare minimum access required for the cluster to come up successfully. Any additional ports used by applications deployed onto the clusters should be added by users since this will vary by application and by use case.


Missing script to split BAM in long read mode

Hi,
I think the following script is missing: split_bam_by_bc_long.py it is required when running bam_processor.split_bam_files(). So far I modified the split_bc_long.sh script by changing split_bam_by_bc_long.py for split_bam_by_bc.py and it seems to work. I wanted to confirm that this is fine or get the corresponding script.
Best,
Mariela

Hi Mariela,
Thank you very much for your feedback. @RoKsaNne, can you check scripts again and make sure everything is uploaded?
Regards,
Yumin

Hi Mariela,
Thanks for pointing that out, your attempt is right!
The split_bc_long.sh seems to be an old version so the split_bam_by_bc_long.py didn't change to split_bam_by_bc.py, sorry for the inconvenience, I've updated the split_bc_long.sh now.
Best,
Ruohan

Hi Mariela,
Thanks for pointing that out, your attempt is right!
The split_bc_long.sh seems to be an old version so the split_bam_by_bc_long.py didn't change to split_bam_by_bc.py, sorry for the inconvenience, I've updated the split_bc_long.sh now.
Best,
Ruohan


A few fixes and improvements

Hi, pretty nice and complete jailbreak checklist that you have there :)
Please go through the commits to see what each one is doing.
The .modulemap file was causing compilation issues when used under CocoaPods so I had to remove. Is it really needed?

nice!


fix compiler warning

I'm not certain if this is correct - but "l" isn't used currently.

Ping @raffienficiaud

Thanks, on the tests queue


Coverage difference while targeting ES5 and ES6

@kwonoj could you, please, explain this strange behavior? When tsc targets ES6 we have this output in coverage.txt:
-----------|----------|----------|----------|----------|----------------|
File       |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
-----------|----------|----------|----------|----------|----------------|
 simple\   |    85.71 |    33.33 |       75 |    85.71 |                |
  Hello.ts |    85.71 |    33.33 |       75 |    85.71 |             20 |
-----------|----------|----------|----------|----------|----------------|
All files  |    85.71 |    33.33 |       75 |    85.71 |                |
-----------|----------|----------|----------|----------|----------------|

But when we targeting ES5 output is changed to this:
-----------|----------|----------|----------|----------|----------------|
File       |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
-----------|----------|----------|----------|----------|----------------|
 simple\   |    90.91 |       50 |       80 |    88.89 |                |
  Hello.ts |    90.91 |       50 |       80 |    88.89 |             20 |
-----------|----------|----------|----------|----------|----------------|
All files  |    90.91 |       50 |       80 |    88.89 |                |
-----------|----------|----------|----------|----------|----------------|

Is this expected and normal or something goes wrong with our remapping mechanism?

Interesting, I do not have good answer too. Coverage processor rely on remap-istanbul for remapping (https://github.com/kulshekhar/ts-jest/blob/master/src/coverageprocessor.ts#L32) and does not do anything by its own, not sure why this different comes in. Need to compare results and dig bit more.

I've digged into this bit more, and currently suspecting generated source map is different between es6 to es5 from typescript compiler. To isolate, I've used separate repo neither rely on ts-jest nor rely on coverage processer (https://github.com/kwonoj/rxjs-testscheduler-compat), which uses mocha and nyc for coverage remapping -  and below is result.
As you can see, left side (es5) to right side(es6) shows different coverage numbers and es5 result.
I can't exactly pinpoint which module causes this exact problem (typescript compiler / istanbul remapping, or something else) but seems current coverage remapping itself is working as expected.

@kwonoj ok, so we have to investigate and probably ask help from Istanbul or TS repo, because it seems to be a bug on their side?

That's my current guess. Either case I don't think it's issue of coverageprocessor. (I may be wrong, but given result comes out of nyc without ts-jest it seems so)

Ok, I'll leave this issue open for now and reference it when we'll create issues in TS or Istanbul repos.

Tried once again and got confused more - https://github.com/kwonoj/nyc_coverage_test this one works perfectly. It seems there's some difference generating source map, but can't pinpoint which one is responsible for this behavior.


Set POST method when CURLOPT_POSTFIELDS is set

According to the curl documentation, using CURLOPT_POSTFIELDS implies CURLOPT_POST. This patch ensures that when CURLOPT_POST is not set and CURLOPT_POSTFIELDS is set, the request method is POST.

That's awesome @webspectyler! Thank you for your contribution.
May I ask you to add a test to the CurlHelperTest to ensure this functionality will be verified automatically? Thanks a lot!

This is awesome. The integration test for SOAP is failing but this is not on you.
Thank you for your work. Will merge it after resolving the integration test.

Integration tests are fixed now! Thanks

This is a breaking change. It should warrant a 2.0.0.

Thanks for the heads up @TeslaDethray! Is this blocking you from updating?
Do you by any chance have a code example that we can use to make a unit test?

@adri Yes, I've had to pin at 1.3.1 because my the API my app hits uses PUT requests.
The unit test included in this PR actually tests the error pretty nicely - the 'GET' is being overridden just as other methods would be. Change 'GET' to 'PUT' in the extant test:
    public function testSetCurlOptionOnRequestDoesNotRuinMethods()
    {
        $method = 'ANYTHING BUT POST';
        $domain = 'example.com';
        $request = new Request($method, $domain);
        $headers = ["Host: $domain",];

        CurlHelper::setCurlOptionOnRequest($request, CURLOPT_HTTPHEADER, $headers);

        $this->assertEquals(['Host' => $domain,], $request->getHeaders());
        $this->assertNotEquals($request->getMethod(), $method);
    }

The array version of $args that CurlHook::__callStatic is getting from Guzzle includes an element [CURLOPT_POSTFIELDS] => 'PUT'.

I'm just expecting behavior as I understand it from the cURL documentation:
https://curl.haxx.se/libcurl/c/CURLOPT_POSTFIELDS.html
If I'm misinterpreting "Using CURLOPT_POSTFIELDS implies CURLOPT_POST", please disregard this issue.


SonarQube questions

@lewismc I just installed the SonarQube plugin of eclipse. The question I have is, why am I only seeing 60 bugs on my local SQ, but there are 200+ on the remote one. Is it because different setting are used in local and remote ones? I am also trying to New a SQ server, but I don't have the information below.

BTW, let's say I have another project, how am I supposed to connect it with SQ? Didn't find much document on this issue.

The document I am referencing is http://www.sonarlint.org/eclipse/

Hi @Yongyao thanks for trying this out. I can promise you it will make a HUGE difference.
OK so now to the questions

The question I have is, why am I only seeing 60 bugs on my local SQ, but there are 200+ on the remote one. Is it because different setting are used in local and remote ones?

In short yes, this is exactly the reason. The online SQ server has a 'policy' setup where types of issues are flagged as bugs, other issues are flagged as code smells, etc. I wouldn't worry too much about this though. We can just work through the issues online and you will see over time the difference that we make.

TW, let's say I have another project, how am I supposed to connect it with SQ? Didn't find much document on this issue.

Lets go through this next week at our meeting. You can then take some notes and we can put them on this project wiki or something. It takes quite a few steps (mostly configuration).

Great. Let's keep this issue open until next week.

@Yongyao did you ever fix this?


building error

Could not get unknown property 'bintray_user' for object of type com.jfrog.bintray.gradle.BintrayExtension. Open File

Why are you cloning the project when you can use the gradle installation? Are planning to contribute? The bintray_user is my bintray user name and bintray_key is my password. They are stored local on my computer and not supposed to be exported with the project.


[Feature]: [FRSM-04] Does the software include descriptive metadata which helps define its purpose?

D5.2 p14, p25
Detailed Description

Software requires descriptive metadata to support indexing, search and discoverability.
Domain-agnostic comments
There are several common places for descriptive metadata to be found, including intrinsic metadata that is part of the software source code such as README files, requirements files that describe dependencies, POM, CodeMeta or CFF files, or in the extrinsic metadata available through resolving the software identifier. It may also be directly embedded in software source code files. The implementation of this metric will depend on the coding standards for the programming language as well as community norms for which descriptive metadata is used. It is hard to check the relevance / correctness of unstructured metadata such as a text description, but it is possible to automatically check for existence.
CESSDA comments
CESSDA technical guidelines on CMA1: Documentation define what is required from end-user documentation, operational documentation, and development documentation but these are not machine-accessible. The CESSDA Software Requirements also demand that all tools and products have a comprehensive README.
Context

F2: Software is described with rich metadata.

R3: Software meets domain-relevant community standards.
Possible Implementation

domain-agnostic

requirements
software source code, software identifier

method
Check if the software and/or software identifier has machine-readable descriptive metadata associated with it that describe its purpose.

essential
The software includes a README or other file which includes the software title and description.

important
The software includes other descriptive metadata such as domain, funder, programming language, date created, and keywords.

useful
The metadata is contained in a format such as CodeMeta or ProjectObjectModel that enables full machine actionability.

CESSDA

requirements
software identifier (DOI) provided by Zenodo

method
Query the metadata provided by the Zenodo record for the software.

essential
Zenodo metadata includes the software name and description.

important
Zenodo metadata includes other descriptive metadata as recommended in CESSDA Software Requirements.

useful
n/a.

Expand evaluator for minimal metadata: https://github.com/softwaresaved/fuji/blob/1778e730c80852f691e135a1b288bc2796914c46/fuji_server/evaluators/fair_evaluator_minimal_metadata.py#L12


enable fast first sign-in

The text for this feature is referring to something else, please update the text for Enable First Sign-in feature to be correct, suggested if it is still in preview?

Why

Closes #[Issue Number]

Changes

@DeployWindowsCom, thank you for committing the suggested change, but I see that you then closed the PR. Was that intentional? If not, could you kindly open it again? Thanks.


semi-supervised learning

Hello,I read the paper and the source code.And it mentioned 'The last three hidden layers of the encoder as well as its output are concatenated to form a 8960-dimensional feature vector.' in section 4.3 of the paper.Could you please tell me how to compute the dimension?Thanks very much

Could you confirm if this was a typo in the paper?


Reggression tests do not work with plugin for IDA 7.x

Since update from IDA 6.x to IDA 7.x, plugin's regression tests have stopped working. This concerns only plugin developers, not regular users.

Fixed in https://github.com/avast-tl/retdec-idaplugin/commit/0984724ae6a47b5d586d0e05910fbe861ad3d5e9.


feat: add metadata to frameready signal

This emits a (currently-unstructured) metadata dict as the third value of frameReady.  This would be an important component for a proper writer spec (#29)
side-note: because both psygnal and qt let you connect a slot that accepts less values than the signal emits, we can always add positional arguments to signals without breaking any APIs.
closes #220

well shoot... looks like this does actually break napari-micromanager after all... need to look into it


App : Design for each menus fix needed

Needs to talk with for UI differential for app.

check and talk with designer

Received implemented Design from designer  design XD prototype
Design for menus all corrected


return dir entries from getfile in alphabetic order #199

readdir() returns directory items in an undefined order, determined by the underlying filesystem.
This replaces readdir() with scandir() so that the built-in command 'getfile' will return directory items in alphabetic order.  The main reason is so that auto-generated files that depend on directory traversal will be stable over time.  In particular the test output produced by vjqa (#199 ).  There is some extra overhead: every call to scandir() has to re-scan the entire directory, instead of being able to break out of the loop when 'index' has been reached - on avg twice as many calls to readdir().  Plus there's a call to qsort() every call to scandir().  These should both be pretty negligible on modern systems though.
I think it shouldn't break anything, since the old semantics had an undefined mapping of index to filename.
Haven't tested on macos, but looks good on centos6 & 7 ubuntu14 & 16.

The second commit (23bd410) is building here: https://circleci.com/gh/tesch1/OpenVnmrJ/164

I generally don't like to slow things down. I checked and it is approximately 10 times slower. What if we add an option.
getfile('directory', index, 'sort') or
getfile('directory','sort',index)

How are you testing the speed?  Maybe I can get the time down to something acceptable before resorting to added complexity.  I think it shouldn't be more than 3-4x slower, and I'd expect something on the order of 10us -> 40us on most directories.

Here is the macro I wrote. I used /vnmr/maclib as the argument. It took about 0.32 seconds with the readdir and 3.2 seconds with the scandir. We could just use getfile('directory',index,'sort') and then a simple test of (argc == 4) would select scandir rather than readdir.
"test macro"
$name=''
getfile($1):$num
write('line3','get %d files',$num)
$i=0
pushtime
while ($i < $num) do
$i= $i+1
getfile($1,$i):$name
endwhile
poptime

Committed an implementation with the sort flag.  I couldn't get it to show a slowdown (testing on linux in a VM) of more than 4x, but I only tried (so far) with directories with up to 4100 randomly named files (7.1 s and 2 s, sorted and unsorted, respectively).  With fewer than 1000 files I see about 2x slowdown, and under 500 it looks to be on the order of the measurement error (~10 ms).  ¯_(ツ)_/¯
https://circleci.com/gh/tesch1/OpenVnmrJ/175
#!/bin/bash
set -e
cd
rm -rf dirs
mkdir dirs
cd dirs
for i in $(seq 10 100 10000) ; do
  mkdir $i
  for j in $(seq 1 $i); do
    touch $i/$(cat /dev/urandom | tr -dc 'a-zA-Z0-9' | fold -w 32 | head -n 1)
  done
done

Even remembered the man page. I'm impressed.

Saw that in your 'add' commit, baby steps...


Possible speedup of _prominent_peaks method

Hi there, I have been using the _prominent_peaks method in peak.py, and it's running a bit slow for my application so I was looking to optimise it. I thought I would open an issue as opposed to a pull request because a) the underscore suggests this is for internal use only, and b) I'm not 100% sure my changes give identical behaviour.
I noted that than existing lines 395-400 are the bottleneck when using on my dataset. These lines label the data then use regionprops to find coordinate centroids, which ends up calling a cpu-based scipy find_objects function.
It seems to me these lines could be changed to avoid regionprops, simply finding the peak pixels in img_t and then sorting the coordinate list
    # Find array (x,y) indexes corresponding to max pixels
    peak_idxs = cp.argwhere(img_t)
    # Find corresponding maximum values
    peak_vals = img[peak_idxs[:, 0], peak_idxs[:, 1]]
    # Sort peak values low to high
    val_sort_idx = cp.argsort(peak_vals)[::-1]
    # Return (x,y) coordinates corresponding to sorted max pixels
    coords = peak_idxs[val_sort_idx]

This runs much faster (at least for me)! Our data is roughly (200 px, 1M px) so an unusual shape.
I also noticed in some cases, when there are a lot of peaks the for ycoords_idx, xcoords_idx in coords: loop is the bottleneck (or if I smoothed the data heavily beforehand). I'm still trying to get my head around what that loop is doing though 🤯.
Thanks!
Danny

Yes, regionprops is one area that is not currently well-accelerated on the GPU. I think it is somewhat difficult because often each region is relatively small in terms of the number of pixels and we currently launch separate CUDA kernels per region, so this can be pretty inefficient.
The form above does give a differing result in the following test case (taken from cupyimg/skimage/feature/tests/test_peak.py) where the peak is actually made up of three pixels of identical intensity. The implementation here (and in upstream scikit-image) will return the centroid of these 3 voxels which is at location (7, 7) while for the proposed code here I would get (8, 8) which is part of the peak, but is not the centroid. Whether or not this matters to the user is probably application-dependent.
image = cp.zeros((15, 15))
x0, y0, i0 = (8, 8, 1)
x1, y1, i1 = (7, 7, 1)  # prominent peak
x2, y2, i2 = (6, 6, 1)
image[y0, x0] = i0
image[y1, x1] = i1
image[y2, x2] = i2
out = peak._prominent_peaks(image, min_xdistance=3, min_ydistance=3)

I would suggest opening an issue at upstream scikit-image to discuss with other maintainers there who work with this peak detection module more than myself. Perhaps there can be a flag for a faster mode that does not guarantee returning the centroid of the peaks. If we settle on the API on the CPU side for core scikit-image then we are more willing to make a corresponding change for the GPU.
Also, the cupyimg.skimage functions in this repository have recently migrated over to a new home at https://github.com/rapidsai/cucim. I still need to update the main README file here to reflect that. There are some performance improvements made in the cuCIM repository vs. here, but I don't think these include the peak finding or regionprops functions, unfortunately. The functions in skimage.color conversions are one thing that is much faster in the new repository and a number of functions across the library do a better job of preserving single precision computations for single precision inputs.

Thanks @grlee77 for the quick and comprehensive response! I didn't consider identical intensities, so see why labelling + regionprops was used now.
For our needs, a small change in centroid doesn't matter. I will see what other speedups we can come up with and if it looks useful I'll get in touch with scikit-image folks. And will check out cucim. Thanks for cupyimg, it's been very helpful :).


Initial commit of updating pyo3 methods

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@alamb work in progress, but it shouldn't be too hard to wrap up any remaining issues. I want to test it with datafusion-python before I mark it ready

Ok, I pulled in these changes to datafusion-python and see no regression

@alamb since this targets your fork I can't merge, but I think this is ready for you.

Thanks @timsaucer  -- I plan to work on the arrow-rs release this weekend too so hopefully we'll be close with that


add support for ts config files

typewind generate fails when the tailwind config is a typescript file instead of a cjs file. Is there any way we can get support for writing the tailwind file as a cjs file, or does tailwind require specific behavior from cjs/require import syntax module loading?
Same issue with postcss.config.cjs.

Tailwind requires the config file to either be tailwind.config.js or tailwind.config.cjs and can't import a ts config file.
TS also doesn't have much advantage in the config as you already get autocompletion in js with jsdoc


Fixes url?

The link before gave me a 404 File not found.
It might be the administration_reference.md one?

I confirmed that this is now fixed, I am closing this one. Thank you for checking it!


Systemd unit fails to start when Type=notify and host is rebooted

Description
Containerd sits in "activating" for >15m when Systemd unit type is set to "notify".
I noticed that, with 1.4, it's advised we set the Systemd unit type to "notify" from "simple".  I beleive this is in order to allow separation of Containerd and Containerd-shims during outages of Containerd.
The problem is that, on reboot of the system, Containerd then takes a long time to come back up (and proportional to the amount of Containers).  By default, Systemd will wait 90s so most times the unit will just timeout and be killed and marked as failed (which is really undesirable on a host that's running a lot of containers, for example a Kubernetes worker.  I have set a 15m timeout on an example host and, when there are around 10 containers, the timeout is hit.

Describe the results you expected:
Containerd to come back up.
What version of containerd are you using:
1.4.4

I'm experiencing this problem in 1.5.2.
I guess it's relevant to #5597.

I'm experiencing this problem in 1.5.2.
I guess it's relevant to #5597.

Yes, it's the same problem according to your logs.
The reason why you get the start up timeout is that,  with Type=notify setting, systemd only consider a service ready after the service actively sending a Ready notification.
containerd takes hours to wait for cleaning up dead shims, and systemd didn't get Ready notification in time. There is no doubt that systemd will consider it failure. That's normal and expected.
When you changes it to Type=simple, as long as the containerd is running, systemd considers it up and healthy, but actually containerd is still struggle to startup and no service is provided.

@joedborg the state dir of containerd is not in the tmpfs?

@fuweid is this set with --state when starting containerd?  Is this behavior expected if that dir isn't blown away on reboot?

containerd sends the readiness signal from the below.
https://github.com/containerd/containerd/blob/3d7a93fed8f3153783ba1490f08b9368f8cf9de0/cmd/containerd/command/main.go#L273-L275
We may be able to send the signal before cleaning up dead shims, not I'm not really sure that is the right thing to do.


Anybody can help me? I just got a spinner on the main screen

What about the logs?
Please see Troubleshooting

@chopeta  I have similar problem but in my cases it thrown

Error: Invalid API-key, IP, or permissions for action.

@wuwen1030 As @chopeta mentioned, please check your log. Please close the issue if your issue is resolved.
@howtoautomateinth check your .env file.

@wuwen1030,  Also check the BINANCE_MODE (test vs live) environment variable in the docker-compose.yml file. If it is pointing to different mode then the frontend will show this behavior as well.

Thanks @chrisleekr but it still have the same issuse
what I did

recreate .env also added new API key

what should I do? or more How can I more investigate on it

@howtoautomateinth glad to hear your issue is resolved.
@wuwen1030 have you resolved your issue?

What about the logs?
Please see Troubleshooting

Thanks for your reply, I will have a try


Confusing Extensions userinterface

As a user
It is confusing to me when to press the add button
I now think that once I selected an extions I need to click the add button

Suggested solution, add a oke button for exiting the window and rename the add button to "add external plugin"
Background: this is what I saw multiple users making this mistake.

@JakobVogelsang do you have other suggestions for a solution?

I agree with both the suggestions


News?

Are you going to release any news that could impact the price in next few days? Just asking, because trading....

@Alentype I'm still waiting for any response from the team, not sure when they are going to release any news, but I'm pretty sure it's in their own interest too, since they hold some amount of MYX too.


PKG not finding .node object despite being added as an asset.

For some reason when building my node project as fully standalone it's not finding the module.

my package.json

The path to the tmp folder is too long.


Can not add game history without solving authentication

Google login not working.
Some users can not even see it
Some users getting bugs for things not being defined
Some users can not authenticate when pressing log in
Solution, move to account creations system and store in a SQL database.
Use flask to store the game details in the database.


Update README, bump version.

@lucaswiman
Bump TigerShark version to 0.3.0 for the Python 3 compatibility release.

LGTM!

Thanks @lucaswiman, I'll upload to pypi tomorrow.

Released!


chore: fix release process

Pull Request
📖 Description
Instead of building workarounds, we included a function in semantic-release-monorepo that allows it to check other folders to be relevant for a release. The change was included as a patch via pnpm.

🎫 Issues

👩💻 Reviewer Notes
I created a test commit in the react package.
Screenshot 1: Folder won't be analyzed → Works as intended, commit is not relevant.

Screenshot 2: Folder will be analyzed → Works as intended, commit is relevant.

📑 Test Plan

✅ DoD

[ ] I have tested my changes manually.
[ ] I have added automatic tests for my changes (unit- and visual regression tests).
[ ] I have updated the project documentation to reflect my changes (e.g. CHANGELOG, Storybook, ...).
[ ] I have made sure to follow the projects coding and contribution guides.
[ ] I have made sure that the bundle size has changed appropriately.
[ ] I have validated that there are no accessibility errors.
[ ] I have used design tokens instead of fix css values

Unfortunately, I get an error when running pnpm install -r. The CI is also complaining that checksums are not matching. Any Idea where this may come from:


SWARM-1085 - support vanilla json-p.

Motivation
JSON is usable without JAX-RS.
Modifications
New, :jsonp fraction, with autodetection.
Result
Lthon's example works.

[X] Have you followed the guidelines in our Contributing document?
[X] Have you created a JIRA and used it in the commit message?
[X] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[X] Have you built the project locally prior to submission with mvn clean install?

Please hold, local -examples not passing for weird unrelated reason, maybe.


WebView visible behind iOS statusbar when using SafeArea and no appBar set

In one of our screens with Material Scaffold no AppBar is set and the body is a ListView containing normal Flutter widgets and a WebView. The ListView is wrapped in a SafeArea.
When scrolling up, as expected, the normal Flutter widgets are not visible behind the statusbar. Unfortunately on iOS the WebView IS visible below the statusbar. It seems like the padding given by SafeArea is not working for WebViews/PlatformViews on iOS.
I created a simple version which you can see in attached gif.

Here is my output of flutter doctor -v:
[✓] Flutter (Channel beta, v1.0.0, on Mac OS X 10.14.1 18B75, locale de-DE)
    • Flutter version 1.0.0 at /Users/basti/Coding/SDKs/flutter
    • Framework revision 5391447fae (vor 12 Tagen), 2018-11-29 19:41:26 -0800
    • Engine revision 7375a0f414
    • Dart version 2.1.0 (build 2.1.0-dev.9.4 f9ebf21297)

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at /Users/basti/Coding/SDKs/android-sdk-macosx/
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /Users/basti/Coding/SDKs/android-sdk-macosx/
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 31.1.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

To investigate a minimal runnable reproduction would be helpful
as a single file so that we can just copy your code into lib/main.dart of a new project and run to reproduce.

Sure, here's the sample's main.dart
import 'package:flutter/material.dart';
import 'package:webview_flutter/webview_flutter.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: SafeArea(
        child: ListView(children: <Widget>[
          Container(
            padding: const EdgeInsets.all(8),
            alignment: Alignment.center,
            color: Colors.red,
            child: Text("This is a container widget with Text. "
                "\nBelow is a WebView.\n\nAs the Container the WebView should NOT show up behind the status bar.", textAlign: TextAlign.center,),
          ),
          Container(
              height: 1200,
              child: const WebView(
                initialUrl: 'https://flutter.io',
                javaScriptMode: JavaScriptMode.unrestricted,
              ))
        ],),

      ),
    );
  }
}

Using webview_flutter: ^0.0.1

Any specific reason you're not using 0.1.0+1?

It didn't solve the issue. Webview still bleeds outside the container.

same too


Show more details in PR tree view tooltip, closes #568.

Show the PR's author login name and creation date in the GitHub Pull Requests tree node tooltips.

Before:

After:

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: jerrymajewski sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

I like the idea but I'm a bit reluctant to display time in the PR tooltip, especially when it can be out of date very easily. cc @misolori for review , this is probably the first time we introduce multi line tooltip in this extension.

I agree that we should just display the author with no timestamp on a single line.

I updated the tooltip as follows:

We should just append the author to the end of the string after the parenthesis:
Closes BLOS-223 (#282) by katwal-dipak


Fixes for issue #76

Fixes https://github.com/serde-rs/aster/issues/76

Tests are passing locally.
r? @erickt

Oh, I guess we need to bump the version number...

ouch... wtf

Ok, I can reproduce when running |cargo build --features with-syntex|

I think syntex will need to be updated too with the changes made to libsyntax, right?

I think syntex will need to be updated too with the changes made to libsyntax, right?

Yes, I started doing that in https://github.com/serde-rs/syntex/pull/43
No idea so far why println!() ends up being broken...

@fabricedesre Any update on this? Is there anything else we need to do?

@KodrAus I haven't fixed the test failure yet. Any help appreciated!

Hmm, that println! error is really weird... Almost like it's not linking std? What's the point of that hello_world program?

Thanks! I'll fix up the issues on my side.

\o/ Thanks @erickt !


Bug Report

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform: 1.2.7
azure provider:  3.7
module: subscrtion
Description
Describe the bug
im trying to see if i am doing something wrong
i do not want to have the modules int he .terraform so i am using raw files
added a providers .tf with feature block as needed
added the variables that i could find int he variables.subscription.tf
wanted to make sure 1
the
/providers/Microsoft.Billing/billingAccounts/ ( IS THE SUBSCRITION ID)
and 2
after entering the required items from the comments i still get these errors?
i dont see where i am suppose to enter these.
╷
│ Error: ID was missing the billingProfiles element
│
│   with module.subscription[0].azurerm_subscription.this[0],
│   on modules/subscription/main.tf line 6, in resource "azurerm_subscription" "this":
│    6:   billing_scope_id  = var.subscription_billing_scope
│
╵
╷
│ Error: ID was missing the enrollmentAccounts element
│
│   with module.subscription[0].azurerm_subscription.this[0],
│   on modules/subscription/main.tf line 6, in resource "azurerm_subscription" "this":
│    6:   billing_scope_id  = var.subscription_billing_scope
│
╵
╷
│ Error: ID was missing the customers element
│
│   with module.subscription[0].azurerm_subscription.this[0],
│   on modules/subscription/main.tf line 6, in resource "azurerm_subscription" "this":
│    6:   billing_scope_id  = var.subscription_billing_scope
│
Steps to Reproduce

step 1
step 2
you get it...

Screenshots

Additional context

Hi @frankscalzo
The billing scope value is used when creating new subscriptions programatically and should be set as per this:
https://docs.microsoft.com/en-us/azure/templates/microsoft.subscription/aliases?pivots=deployment-language-bicep#putaliasrequestpropertiesorsubscriptionaliasresponseproperties
I will update the documentation for this variable accordingly https://github.com/Azure/terraform-azurerm-lz-vending#input_subscription_billing_scope
You need access to an EA, MPA or MCA.
However you can also use this module with existing subscriptions: https://github.com/Azure/terraform-azurerm-lz-vending/wiki/Example-5--Use-with-existing-subscriptions

i was able to solve the issue i found when using the cli to get the ID is that it came back with null and was incomplete.
i was only getting the
/providers/Microsoft.Billing/billingAccounts/12345
and not
/providers/Microsoft.Billing/billingAccounts/12345/enrollmentAccounts/DB4C-XXXX
i had to manually look the rest of the scope info up on the UI to get the id of the scope
i have made it past this issue and on to a new one regarding alias creation
ill push through but any insight is appreciated
module.subscription[0].azurerm_subscription.this[0]: Creating...
╷
│ Error: creating new Subscription (Alias "epiprod"): subscription.AliasClient#Create: Failure sending request: StatusCode=400 -- Original Error: Code="BadRequest" Message="Specified method is not supported."

@matt-FFFFFF  sorry i was able to solve i was reading a differnt post than you sent and it talked about enrollment account
which had got me past the error
it was not until i dug deeper and saw that with MSC you need to do
/billingAccounts/{billingAccountName}/billingProfiles/{billingProfileName}/invoiceSections/{invoiceSectionName}
thanks again ill keep running through

Glad I was able to help! Come back if you get stuck again.


Option for multiselect in table and excute an api with button

Hi Team,
Do we have any Option for multi select in table and excute an api with button.
Thank you

@vanimesh Buttons will be implemented in future versions: https://github.com/VolkovLabs/business-table/issues/2
We can add multi-select similar to Business Forms.

That's really great to have feature.can you point out to the feature in business forms? I am looking for a feature where the table populates with the data from the table and the user can perform operation on those rows

@vanimesh Business Forms works with one row at a time, it does not support multiple rows as in Business Table. This is the reason we created Business Table.

Understood that.. would be waiting for you to release the feature. Is there any timeline for me to know when it's released?


review: spv type lock

I use following tags to categorize review comments:

:thought_balloon: Discussing: No strong opinion, just for discussion.
:information_source: Informing: Information for reviewers.
:no_entry: Blocking: Issue requiring immediate attention.

There's no major issues found during my review, just some minor suggestions.

All resolved.


crash

Issue description
i cant even enter
oritech.txt
my world but the issue seems to be with the coal generator and oritech machines ticking
oritech.txt
Steps to reproduce
i just placed these oritech machines assembler,foundry,lava generator,centrifuge, and a pulvirizer
and 4 coal generator for power form the mod wormholes i dont know a good power option right now that would help.
Also when ever i exit fullscreen alt-tab the game crashes
Minecraft version
1.21.1 (Latest)
Fabric loader version
0.16.5
Fabric API version
fabric-api-0.102.0+1.21
Oritech version
oritech-0.9.0
Other relevant versions
the crash error said oritech,wormholes,fabric api lookup,and the conif and library needed for the wormhole mod wich are SuperMartijin642
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
https://pastebin.com/RCDpiZgy

How exactly did you achieve this? One of the machine cores seems to think it's connected to the coal generator, which can't ever happen without some weird worldfile manipulation or something. Oritech also has a basic generator which can burn coal btw.

Yeah i dont know how it happaned buy i just delted and reisntaled the mod that adds coal generator and it dwlted the generators so now i can llay

How exactly did you achieve this? One of the machine cores seems to think it's connected to the coal generator, which can't ever happen without some weird worldfile manipulation or something. Oritech also has a basic generator which can burn coal btw.

i think i know the issue the carryon mod lets u move the machines with multiblocks and it doesnt pick up the multiblocks since it a multiblock but the machine cores remain in their old place when you pickup the machine i can send a video to you with mediafire since the size is big even tho its 43 seconds long
https://www.mediafire.com/file/488wayuc2gu9nlw/Java-runtime-delta+2024.10.31+-+<IP_ADDRESS>.mp4/file


edited README.md for clarity and grammar

Description
Hi! I was one of the students from the class where we had to edit README files for an assignment. I went ahead and edited yours if that's okay. I found some grammatical errors and fixed those. Other than that, I changed a couple of words in sentences to make them flow nicer or to make them more clear. If I altered the original meaning of a sentence somehow, please let me know and I'll fix it!
Please approve this PR by Friday night because that's when my assignment is due. Thanks so much!

Thank you!


[SPARK-41699][CONNECT][BUILD] Upgrade buf to v1.11.0

What changes were proposed in this pull request?
Upgrade buf to v1.11.0
Why are the changes needed?
this upgrade doesn't change the generated codes now, so should be fine to upgrade
Does this PR introduce any user-facing change?
no
How was this patch tested?
existing UT

cc @HyukjinKwon

Thank you, @zhengruifeng .

@dongjoon-hyun thank you for merging!


Example not working anymore

Trying to run the example from here and I noticed it's not working anymore:
https://github.com/dfinity/stable-structures/blob/main/examples/src/custom_types_example/src/lib.rs
/// A trait indicating that a `Storable` element is bounded in size.
pub trait BoundedStorable: Storable {
    /// The maximum size, in bytes, of the type when serialized.
    fn max_size() -> u32;
}

This should be implemented and also I got stuck here:
the trait bound `u64: ic_stable_structures::Memory` is not satisfied
the following other types implement trait `ic_stable_structures::Memory`:
  FileMemory
  Rc<M>
  RefCell<std::vec::Vec<u8>>
  RestrictedMemory<M>
  VirtualMemory<M>

Hey @gabrielnic, can you share exactly how you're running the example, and are you using the latest code? I was just able to run this example from a clean master.

Closing this issue as there hasn't been any issues with the examples that I can notice. @gabrielnic feel free to reopen this if the problem persists on your side.


add activity stream as an api method

also adds ability to pass down options to collections

@miq-bot assign @bdunne @syncrou
@bdunne @syncrou please review

@syncrou extracted to  https://github.com/ansible/ansible_tower_client_ruby/pull/57

👍 LGTM


Fix list job -x

[ This is a placeholder PR for discussion. We need to discuss list job -x behavior wrt to history ]
Looking at the output of list job and list job -x after an updated pipeline showed a different list. Turns out history is set to none for -x
$ pc list job
ID                               SUBJOBS PROGRESS CREATED      MODIFIED
b7206e1f14e743b790c91797c8128394 1       ▇▇▇▇▇▇▇▇ 27 hours ago 27 hours ago
d2e7ac68e8134671abfa907dd9784b76 2       ▇▇▇▇▇▇▇▇ 27 hours ago 27 hours ago
31dfe367cc6448c390ea45cd155efc1a 2       ▇▇▇▇▇▇▇▇ 27 hours ago 27 hours ago
9d5a96e210ae47f3bd1edbf4709cb851 1       ▇▇▇▇▇▇▇▇ 27 hours ago 27 hours ago
45fe7aa3fdb840cabbade49775f6eede 1       ▇▇▇▇▇▇▇▇ 27 hours ago 27 hours ago
$ pc list job -x
ID                               PIPELINE STARTED      DURATION           RESTART PROGRESS  DL       UL       STATE
b7206e1f14e743b790c91797c8128394 montage  27 hours ago 4 seconds          0       1 + 0 / 1 371.9KiB 1.292MiB success
d2e7ac68e8134671abfa907dd9784b76 edges    27 hours ago 6 seconds          0       2 + 1 / 3 181.1KiB 111.4KiB success
31dfe367cc6448c390ea45cd155efc1a edges    27 hours ago 2 seconds          0       1 + 0 / 1 57.27KiB 22.22KiB success
45fe7aa3fdb840cabbade49775f6eede edges    27 hours ago Less than a second 0       0 + 0 / 0 0B       0B       success

As you can see, there are jobs missing in list job -x that are listed in list job
The output should be????
$ pc list job -x
ID                               PIPELINE STARTED      DURATION           RESTART PROGRESS  DL       UL       STATE
b7206e1f14e743b790c91797c8128394 montage  27 hours ago 4 seconds          0       1 + 0 / 1 371.9KiB 1.292MiB success
d2e7ac68e8134671abfa907dd9784b76 montage  27 hours ago 5 seconds          0       1 + 0 / 1 371.9KiB 1.292MiB success
d2e7ac68e8134671abfa907dd9784b76 edges    27 hours ago 6 seconds          0       2 + 1 / 3 181.1KiB 111.4KiB success
31dfe367cc6448c390ea45cd155efc1a edges    27 hours ago 2 seconds          0       1 + 0 / 1 57.27KiB 22.22KiB success
31dfe367cc6448c390ea45cd155efc1a montage  27 hours ago 2 seconds          0       1 + 0 / 1 79.49KiB 381.1KiB success
9d5a96e210ae47f3bd1edbf4709cb851 montage  27 hours ago Less than a second 0       0 + 0 / 0 0B       0B       success
45fe7aa3fdb840cabbade49775f6eede edges    27 hours ago Less than a second 0       0 + 0 / 0 0B       0B       success

The change here is to default history to all for list job --> I think this is the intended behavior of list job -x which is to expand on the list job output.
Side note... since history is used in multiple commands the default is taken from the last executed Flag statement. In this case it was in list pipeline. Hence the code change to disambiguate the defaults for both the commands.

@Tryneus In order to make things consistent, we should have "history" support for list job. From the implementation, seems like we explicitly prevent that today. Keeping this open to get your thoughts and reason.

Stale PR


Fix a typo

neet → need

Thanks!
I know this is silly for such a small change, but could you sign our cla? https://cla.developers.google.com/about/google-individual

Sure, I have signed it previously while contributing to other Google projects. Attached the screenshot of the same below.

CLA looks good, thanks!