High level Vulnerability

Hello Flutter Community,
We are experiencing High level Vulnerability under the package that we are using local_auth. This local_auth package internally using the local_auth_android package that is causing the problem.
Security Scanning Orchestrator: Ostorlab
OS: Android
Problem:
Android provides mechanisms to enforce biometric authentication to protect sensitive information. Biometric authentication has evolved over time to provide improved user experience, developer experience and improved security.
Previous implementation using FingerprintManager is deprecated and must not be used. Proper implementation must useBiometricManager with BiometricPrompt and CryptoObject.
CryptoObject provides cryptographic primitives for encryption, decryption and signature validation.
In the example below, calling the authenticate method without cryptoObject is vulnerable to authentication bypass:
Proofs:
Attached the screenshots.
Would you please let me know whether we have an any open Vulnerability or do we need to create new. If yes, Would you please provide us the link where we can report about this.

Thanks,
Venkata.

Hi @stuartmorgan ,
Both the Biometric API on Android (BiometricPrompt.CryptoObject) and iOS (kSecAttrAccessibleWhenPasscodeSetThisDeviceOnly) exposes APIs to secure data relying on cryptographic primitives that are either generated or unlocked using the Biometric authentication.
local_auth does not expose those. The fact that these APIs are part of the native SDK should naturally be reflected in its Flutter counterpart.
local_auth without it is insecure for use and Biometric authentication with it is always bypassable [3]. The risks addressed with Biometric Authentication are not mitigated as nothing links them to the Biometric data of the user.
I hope the Flutter team would reconsider adding these to the package and exposing the SDK cryptographic/access primitive to Dart for secure usage.
[1] https://developer.android.com/reference/android/hardware/biometrics/BiometricPrompt.CryptoObject
[2] https://developer.apple.com/documentation/localauthentication/accessing_keychain_items_with_face_id_or_touch_id
[3] https://book.hacktricks.xyz/mobile-pentesting/android-app-pentesting/bypass-biometric-authentication-android

Both the Biometric API on Android (BiometricPrompt.CryptoObject)[1] and iOS (kSecAttrAccessibleWhenPasscodeSetThisDeviceOnly)[2] exposes APIs to secure data relying on cryptographic primitives that are either generated or unlocked using the Biometric authentication.

Yes, I'm aware that protecting access to a secret is one possible use of the same underlying systems that local_auth uses.

local_auth does not expose those.

This is because, as noted above, that's not within the scope of what local_auth is intended to do. Another plugin could certainly build, for example, a secure storage system based in part on those APIs.

The fact that these APIs are part of the native SDK should naturally be reflected in its Flutter counterpart.

local_auth is not intended to be a counterpart to the entire native SDK.

local_auth without it is insecure for use and Biometric authentication with it is always bypassable [3].

Based on the link, I assume when you say "always" you mean "when the device is rooted and an attacker has the ability to replace arbitrary code in the application". If you are attempting to design an application that is secure against a local attacker who can make arbitrary changes to that application, then you need something more complex than the use case local_auth is intended for (presumably either a secure local data storage system, or a passkey-like exchange with a server). Please feel free to file a new issue requesting a new plugin, describing the specific functionality and use case you are trying to address.

@stuartmorgan can we go ahead and raise the new plugin for above use case.


Simplify calc expression

0 - (a - b) = -(a - b) = (b - a)

Thanks @caillou! I don't have an iOS/macOS device to test on, can anyone confirm this will work? Just in reference to a different issue caused weird behaviours on iOS/macOS.

Safai iOS:

Safari MacOS:


Style updates

$ https://github.com/Expensify/Expensify/issues/334627
This updates the design to match @dubielzyk-expensify's requests.
@tgolen I couldn't reproduce the resetting of the filters. Might that have been fixed with your recent 1.3.6 update? It looks like these remain static even when the new data is fetched.

Yeah, I agree. I haven't been able to reproduce the problem with the filters again either.

updated!


Breaks the css in the Extra Networks tab

If you have your LORA / Checkpoints etc set up in subfolders to group them, adding this extension breaks the CSS in the Extra Networks Tab
This is what it looks like without the extension

while this is what it looks like as soon as you add the extension

This makes the extra network nearly unusable when you have a lot of files, because the navigation bar to swap between your subfolders gets pushed way down. For example, if I click LORA without filtering it by Meme, it shows ALL my Lora's and the subfolder navigation bar is like 5 screen heights down the page and I have to scroll all the way down just to filter back to the category I actually wanted to see

Try checking out the the fork. If you look on the extensions tab in auto1111, there seems to be a new maintainer.

That's the one I'm using, but there's no way to report an issue there that I can tell

Resolved, this branch fixes the issue https://github.com/SoCuteShibe/stable-diffusion-webui-two-shot


Return inner-Scan updates instead of setting default_updates

This PR replaces our use of .default_updates with the existing inner-function return value semantics, which is capable of handling updates.  These changes fixes some missing input issues caused by the hand-crafted .default_updates.

The current error is:
FAILED tests/test_hmc.py::test_nuts - assert 1.2154702717073333 == 1.0 ± 1.0e-01

@rlouf, do you think this is within an acceptable range, or a sign of something being off?

@rlouf, do you think this is within an acceptable range, or a sign of something being off?

So close to the target it's likely to be a problem with the parameters, most likely step_size.


Fix error json and tests

Fix error internal error: package "encoding/json" without types
Fix tests
Upgrade go 1.16

@cquilboss can you explain what's the issue we are trying to fix? This is a forked library so ideally we the least changes possible unless we are committed to maintaining it in which case I would have to test this on watson as well.

@vanillaricewraps do we pause the lint or do we try to rebase the fork?

@vanillaricewraps do we pause the lint or do we try to rebase the fork?

I don't think we ever used the linters here because we were never committed to maintain this repo so back to my original question: what is the bug that we are trying to fix? Ideally we don't make changes in this repo unless absolutely necessary but if there is a small change you need to make for it to work with listing-services then we can try to release a beta version so we can test it in watson.

@vanillaricewraps Basically it does not compile anymore. When I now use the library with go 1.18 we get the error: error: package "encoding/json" without types at compilation.
It's a fork from now an old version, since then the original package got updated. I simply used a chunk of the latest version to make it work again.
Yes you should test it against watson as well before merging.
Anyhow, if it's too much work to keep the library up to date, no worries I currently use this fork until we get this package stable.

Closing in favour of https://github.com/TripleMint/gqlgenc/pull/2


Add option to render a circle with a radius on DetailField

This adds the option to call ->circleRadius(1000) with a circle radius in meters as an attribute. This is very helpful for my project so I think others may want to use it as well :)
PS: I've fixed the indentation of the DetailField.vue file as my last step. Not sure if you like it but I think it's better.

@checkerschaf Thanks! I'll take a look at this over the next week or so.

@mikebronner Any updates on this one? :)

Thanks for the reminder! I got busy with Laravel 6 stuff these last two weeks. I'll try to look at it today.

This is now added, with additional functionality (see readme) in release 0.1.11. Let me know how it works for you! Thanks for adding this. :)


bug: Context selector overflows

Version
v0.17.1701702451
Describe the bug
When VSCode is half the screen size of my 16" MBP, this UI overflows.

Expected behavior
It does not overflow.
Additional context
No response

hey we actually work on bugs here @eseliger 😄
closed by https://github.com/sourcegraph/cody/pull/2102


Image not available?

Challenge Add Images to your Website has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.138 Safari/537.36 Vivaldi/1.8.770.56.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

CatPhotoApp

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.

The image is not being displayed in the output right?

@faizilfanie
Thanks for reporting this. But your code is incorrect.
GitHub Issues are for reporting bugs on the website only.
Please use the Help chat room for getting challenge related help.
Happy Coding.


fix(ssr): watchEffect onInvalidate runner initialization (#3322)

Hey, thank you for your PR, but I think there is a better way. In SSR, we do not need to initialize runner.options.onStop, just change a little bit:
  const onInvalidate: InvalidateCbRegistrator = (fn: () => void) => {
    cleanup = () => {
      callWithErrorHandling(fn, instance, ErrorCodes.WATCH_CLEANUP)
    }
    if (!(__NODE_JS__ && isInSSRComponentSetup)) {
      runner.options.onStop = cleanup
    }
  }

I rethink about that, the callback registered with onInvalidate will not be executed in SSR at all, so we only need to provide a NOOP instead of onInvalidate.

@HcySunYang thanks for the feedback, that actually makes sense! I'll do another pass. Do you have a suggestion for improving the test?

I think it would be better if we write test case in server-renderer, as a special case, although it is related to watch, but it is also related to the rendering behavior of SSR.

@HcySunYang thanks for that pointer... much better place for it! Let me know if anything else should be touched up.

@tjk , great work, thanks


StorageUIController needs refactoring

My issues with it:

I needed to bugfix something to do with BUIs but the control flow is all over the shop it's hard to tell what I even need to do because sometimes the container is null and sometimes it isn't.
Static UI shouldn't be baked in as much as it is, it should just always use window support. Attaching it to the hotbar makes it significantly harder to keep track of because there's a bunch of hotbar handling over the class to make sure the window doesn't just erroneously get stuck orphaned and it now makes it much more prone to just breaking. And I can't just add it to the hotbar because it may or may not be static and this is handled in 4 different methods.
It functions like a stack but it doesn't even use a stack to do this.
It's possible for dragging ghosts to just get orphaned and stick around forever.
Sometimes it just breaks and is unresponsive to keybinds https://github.com/space-wizards/space-station-14/issues/26733

Really it comes down to this is the hardest thing by far I have worked with to add BUI prediction support to compared to something like gas analyzer where it just works. It's not even the keybinds or any of the custom handling it's just basic opening and closing support.
What I would like to see:

A storage container that actually persists and doesn't just shuffle the entire UI first time it shows up.
Make StorageContainer a window that's just fixed position PLEASE, the custom handling breaks so easily.
The UI to not just break upon calling open during state handling.
Remove the staticstorage handling everywhere, same with hotbar.
Remove the control flow split across UIController + System + StorageContainer.
Obviously StorageContainer needs optimising because it's extremely slow to build a BoH.

I've experienced the drag ghost getting stuck to my mouse cursor, but I also don't have replication steps for it

Also had another issue popup due to lack of networking for it when I added deferred disposing which has been bandaided.


add map function to tree data structure

Given a tree, return a tree of the same structure made up of the objects returned by the callback which is executed at each node. Similar to Array.map or _,map (underscore library).
Example : node.map(function (node, parent_node, index, root_node) {/*SOMETHING*/}) :: tree
From the walk method, it should be relatively straightforwardly derived. It is still good I think to have in the API.

Thank you for the suggestion. I'm closing this issue for now because I have other priorities for this library besides making it more functional (as in functional programming). Maybe this library is more suited to your needs http://aaronstacy.com/t-js/ as it already implements the traversal methods you are requesting. While I might consider this in the future, I don't currently have use cases for these kind of tree transformation functions.

Long time has passed, I did my own functional library as we discussed at some point : https://github.com/brucou/functional-rose-tree

Great work, congrats! Seems solid.

Thanks!!


Add missing RegExp case in the token_type_to_string

JerryScript-DCO-1.0-Signed-off-by: Szilard Ledan<EMAIL_ADDRESS>

Fix for #284

Looks good to me.

Please add a test too.

LGTM

Good to me

I'll push it

Merged at d2457b1fc61c8eeced239662e9eede5752791dd8


Scripts not saving or running 2nd time Error, Failed to run script... is shown

Problem
A script error is appearing when running a script twice: "Failed to run the script; refresh or try again later"

Scope
Identified on 2 stores
Examples
https://shinesty.myshopify.com/admin/apps/d25e45407e508f96409c2dd796e9bd95/scripts/6848527
https://shinesty.myshopify.com/admin/apps/d25e45407e508f96409c2dd796e9bd95/scripts/46923845
Screencast:
https://www.loom.com/share/25a6d9c71c214805a22c8fc306f56900
Expected vs Actual
Expected: Should save and run without error
Actual: Save shows error, 2nd run script also shows error. (workaround steps at bottom)
Replication Steps

Adding a product to the cart input
Clicking "Run Script"
Clicking "Run Script" again will show the error above

This happens even on a script with nothing except Output.cart = Input.cart in the code
More Information
Workaround for now is:
When adding a product to the cart input, save the script first. Once it's saved, running the script multiple times will work
Zendesk ticket 1
Zendesk ticket 2
@Shopify/scripts

I tophatted in my dev shop and seems to be fixed.


Migrate from tui-rs to ratatui

Ratatui is a fork of tui-rs, that is meant to be its successor, since tui-rs is unmaintained.

Let's do it, thanks for making the migration happen :)


Expose Rye as a Library

Having python dependencies is hard to manage when called from other languages. If Rye could install those dependencies through a library, that would make it a lot easier to integrate python dependencies within Rust or other languages.
Issues that could be tackled:

Simplify install scripts.
Make the installation of the right version of python easier.
Make dependencies more isolated and also more programmable.
Less libpython, PYTHONPATH, LD_LIBRARY_PATH shenanigans as we could have a more reliable way to know where to find each dependencies.

This is tangential to the current vision of Rye, but would make life on planet Rust-Python a lot easier.
I can make contribute those change myself as part of rye or as a third-party lib, but just wanted to know first if that was something rye would be interested in.

I think this would be interesting but it's not likely that I will be working on this.


Test push to look at the linux path

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Closing this PR out.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Killing this PR, it wasn't meant to do anything other than print out a couple of env variables for information gathering so I could write an ICM ticket.


terminate called after throwing an instance of 'util::CompressedException'

通过docker配置的环境，执行到sh run_infer.sh 报错，模型下载路径已经修改了
English LM
LibriSpeech Model
root@6c5480fbb525:/DeepSpeech/examples/tiny# sh run_infer.sh
Download language model ...
download_lm_en.sh: 8: [: 099a601759d467cd0a8523ff939819c5: unexpected operator
--2019-05-08 10:25:03--  https://deepspeech.bj.bcebos.com/en_lm/common_crawl_00.prune01111.trie.klm
Resolving deepspeech.bj.bcebos.com (deepspeech.bj.bcebos.com)... <IP_ADDRESS>, <IP_ADDRESS>
Connecting to deepspeech.bj.bcebos.com (deepspeech.bj.bcebos.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 416 Requested Range Not Satisfiable
The file is already fully retrieved; nothing to do.

download_lm_en.sh: 20: [: 099a601759d467cd0a8523ff939819c5: unexpected operator
-----------  Configuration Arguments -----------
alpha: 2.5
beam_size: 500
beta: 0.3
cutoff_prob: 1.0
cutoff_top_n: 40
decoding_method: ctc_beam_search
error_rate_type: wer
infer_manifest: data/tiny/manifest.tiny
lang_model_path: models/lm/common_crawl_00.prune01111.trie.klm
mean_std_path: data/tiny/mean_std.npz
model_path: checkpoints/tiny/params.pass-19.tar.gz
num_conv_layers: 2
num_proc_bsearch: 8
num_rnn_layers: 3
num_samples: 10
rnn_layer_size: 2048
share_rnn_weights: 1
specgram_type: linear
trainer_count: 1
use_gpu: 1
use_gru: 0
vocab_path: data/tiny/vocab.txt
I0508 10:25:22.147784    89 Util.cpp:166] commandline:  --use_gpu=1 --rnn_use_batch=True --trainer_count=1
[INFO 2019-05-08 10:25:26,183 layers.py:2606] output for conv_0: c = 32, h = 81, w = 54, size = 139968
[INFO 2019-05-08 10:25:26,184 layers.py:3133] output for batch_norm_0: c = 32, h = 81, w = 54, size = 139968
[INFO 2019-05-08 10:25:26,185 layers.py:7224] output for scale_sub_region_0: c = 32, h = 81, w = 54, size = 139968
[INFO 2019-05-08 10:25:26,186 layers.py:2606] output for conv_1: c = 32, h = 41, w = 54, size = 70848
[INFO 2019-05-08 10:25:26,187 layers.py:3133] output for batch_norm_1: c = 32, h = 41, w = 54, size = 70848
[INFO 2019-05-08 10:25:26,187 layers.py:7224] output for scale_sub_region_1: c = 32, h = 41, w = 54, size = 70848
[INFO 2019-05-08 10:25:30,907 model.py:243] begin to initialize the external scorer for decoding
Loading the LM will be faster if you build a binary file.
Reading models/lm/common_crawl_00.prune01111.trie.klm
----5---10---15---20---25---30---35---40---45---50---55---60---65---70---75---80---85---90---95--100
terminate called after throwing an instance of 'util::CompressedException'
what():  kenlm/util/read_compressed.cc:376 in util::ReadBase* util::{anonymous}::ReadFactory(int, uint64_t&, const void*, std::size_t, bool) threw CompressedException.

把models/lm/common_crawl_00.prune01111.trie.klm 删除了，再运行一遍就好了。。。好神奇的bug。


Accessibility Standards

Accessibility Standards
User Story
AS A marketing agency
I WANT a codebase that follows accessibility standards
SO THAT our own site is optimized for search engines
Acceptance Criteria
GIVEN a webpage meets accessibility standards
WHEN I view the source code
THEN I find semantic HTML elements
WHEN I view the structure of the HTML elements
THEN I find that the elements follow a logical structure independent of styling and positioning
WHEN I view the image elements
THEN I find accessible alt attributes
WHEN I view the heading attributes
THEN they fall in sequential order
WHEN I view the title element
THEN I find a concise, descriptive title

The issues in the acceptance criteria have been resolved.


XAxis with showTickLabel false left white space and conflicting with other charts

I'm submitting a...

[x] bug
[ ] feature
[ ] chore

What is the current behavior
XAxis blocking mouse interaction on the top of separate indicator.

Link: https://reactivemarkets.github.io/react-financial-charts/?path=/docs/features-full-screen--daily
Please tell us about your environment

Version: 1.1.0
Browser: [ Chrome 88.0.4324.182  ]

Other information
Its quite easy to fix by removing XAxis component on chart above.
Not sure if "showTickLabel={false}" should remove xAxis because now its showing empty bar.

I post it because StockChart is a main example of library and probably should not have any issues at all.

showTicks={false} is missing from the xaxis, i'll update.


Separate posts with horizontal rule

As a user I want to have clear distinction between different posts on Help Board so that I they are readable.
Currently, there is no separation between different posts on helpboard.

Hi @ppagarawal, in Zepplin I found out this one feed-help-m, How is it look to you?

@OzkanAbdullahoglu the changes you have in the pull request is good. May I know what exactly do you mean to change here in the image you've attached?

@OzkanAbdullahoglu the changes you have in the pull request is good. May I know what exactly do you mean to change here in the image you've attached?

I meant I tried to make similar changes in PR up to Zepplin. Glad to hear that you like it :)


Swith from cargoSha256 to cargoLocks

Use cargoLocks for easier maintenance.

Moved to the create-pull-request/patch branch.


CVE-2023-44981 for zookeeper version

In what version(s) of Spring for Apache Kafka are you seeing this issue?
3.0.12
Describe the bug
Blackduck is reporting https://nvd.nist.gov/vuln/detail/CVE-2023-44981
It looks like this is a result of us pulling in an older ZooKeeper org.apache.zookeeper:zookeeper:3.6.3
Could this ZooKeeper version be updated to remove the CVE?

Spring for Apache Kafka manages org.apache.zookeeper:zookeeper only for spring-kafka-test module:
project ('spring-kafka-test') {
...
		api ("org.apache.zookeeper:zookeeper:$zookeeperVersion") {
			exclude group: 'org.slf4j', module: 'slf4j-log4j12'
			exclude group: 'log4j'
		}

You vulnerability report is false positive since test dependencies cannot affect a production code where exploit is possible.
Sure! We can look into upgrading even that test dependency in the latest our version.
But that 3.0.x branch cannot be upgraded since, according to the CVE you share, they didn't fix their 3.6.x generation.

We should probably make zookeeper an optionalApi in 3.1.x because KRaft mode is now the default for the embedded broker.


Autoformer_Space2nni

Description
We build the search space of Autoformer in nni/examples/nas/hub/searchspace/autoformer.py.
There are 4 searchable variables, depth, embedding dimension, heads number and MLP ratio.

Depth, the number of layers of the architecture.
Embedding dimension, the embedding dimension in each layer. Note that it is shared in each layer which means the architecture is non-hierarchical.
Heads number, the number of heads in self-attn in each layer. The heads number in each layer is independent, like the kernel size in traditional CNN search space.
MLP ratio, the ratio in FFN in each layer. Like the heads number, the MLP ratio is also independent in each layer.

For depth and embedding dimension, we use nn.ValueChoice() to select the values and pass the embedding dimension as a parameter to each layer to keep the dimension consistent.
For the heads number and MLP ratio, we use nn.LayerChoice() to select from the candidates in each layer.
The weight entanglement technique is not supported in the current version. We will follow the latest NNI framework and update the search space in time.
Checklist

[x] test case
[x] doc

How to test
We used the coming features of nn.ValueChoice(), such as multiplying the value with a constant or passing the value to nn.Parameter(). We will test the search space once the new features are supported. I look forward to more discussion about Autoformer_Space2NNI.

Hi there,
I briefly go through the code, the only thing that is troublesome here is that you are using ValueChoice in nn.Parameter(torch.zeros(xxx, xxx)). I haven't given much thought about it, but at first glance, that might be a little bit tricky to support.
The other parts look just fine.
A few minor advices / questions here:

ValueChoice can NOT be used in a for-loop. You need to use the nn.Repeat API to represent a mutable depth.
set_sample_config is not needed. NNI is born for that.
I notice calc_dropout. Do you have a dynamic dropout based on the sampled embed_dim?
What's the difference between AttentionSuper and PyTorch's official MultiHeadAttention? Since we are working on an official super-kernel support for PyTorch's built-in operators (including MultiHeadAttention), it would be better to use the PyTorch's official implementation, so that we can leverage each other's efforts to implement weight entanglement later.

Hi @ultmaster ,
We are extremely sorry for the late reply. Thanks for your careful review and valuable suggestions. The following are our responses

The dimension of absolute positional embedding and cls_token are both associated to the sampled embedding dimension, so I used ValueChoice in nn.Parameter to generate they two. We can discuss how to avoid the trouble.

I have used the nn.Repeat API for ValueChoice, instead of using ValueChoice in a for-loop.

The set_sample_config has been moved. Sorry for the uncleaned code.

The dynamic calc_dropout has been moved. Since drop_out (NOT drop_path_rate) and attn_drop_out have been set to 0 in most vision transformer works up to now, dynamic calc_dropout has not been fully tested, so we decide to remove this feature first.

The AttentionSuper was designed to support optional relative positional embedding and some tricky implementation of the Q,K,V dimension. However, as your suggestion, let us build a general search space first, so I replace the AttentionSuper with Pytorch's official MultiHeadAttention.

Thanks for your suggestions. I have made the following modifications:

Re-correct the variable name to avoid confusion, such as removing prefix sample_ and super_.
Adding type-hints for all arguments.
Adding some comments for AutoformerSpace.

Please let me know if I can be of any further help.

We will have a discussion on #4524 very soon. Get back to you after then.

@nbl97 thanks for your contribution! could you resolve the comments?

Hi @QuanluZhang, I have resolved all the comments, pls check it. Thanks~

Hi @ultmaster, I have resolved the comments. Thanks for your review~


Topbar overlaps popover

Can't replicate... fixed on accident? Closing as this won't block launch.


Why convert single quotes to double?

I try to make bookmarklet, I don't need " in output.

@mevsme we may want to add options menu to the minifier but we're not going to change the default behavior. Thank you.


Ignore randomDelay when calling -version

If you have randomDelay configured (e.g. in a profile), then that gets applied even when using flags such as -version, where it doesn't really make much sense.  (However, I note that -print-profile-config and -print-json-config don't have the same issue.)
It would be great if -version ignored the randomDelay, or even if we could have a -noRandomDelay flag that we could pass for those situations where it is not needed.

I believe with the pre release version of v1.1.14 this is fixed. Can you please test?

In my testing with randomDelay configured in a profile, version <IP_ADDRESS>536 does not delay in properly giving you version information when the command /Applications/Utilities/Nudge.app/Contents/MacOS/Nudge -version is given.

Thanks for confirming.


[requires.io] dependency update on master branch

Coverage decreased (-0.8%) to 97.403% when pulling 2dcf34d1fad6b518f945394b3a043718a665c26e on requires-io-master into 435ab3e450d160919c7790491a5ef94b0dbd1597 on master.


⚠ Bofrost.com has degraded performance

In 866abf6, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 860 ms

Resolved: Bofrost.com performance has improved in eb5412c after 6 minutes.


Difference in animation speed of animated 3d model between  and

I tried with this simple example of imported animated model from both a-scene and ar-scene
https://github.com/donmccurdy/aframe-extras/blob/master/examples/animation/index.html
The animation runs unreasonably fast in  compared to
I found the reasons for that is the timeDelta in argonUpdate function
    argonUpdate: {
        value: function (frame) {
            var time = frame.systemTime;
            var timeDelta = frame.deltaTime;

            if (this.isPlaying) {
                this.tick(time, timeDelta);
            }

            this.time = time;
        },
        writable: true
    },

The variable timeDelta is much higher in ar-scene compared to a-scene (300 to 16) that makes the animation skips the frame. If I change the code to
 var timeDelta = time - this.time;
(which is similar to how aframe defines timeDelta), the animation would run normally.

I think this is actually a bug in argon.js.  This is the only place we have used deltaTime, which I modified argon to compute from it's time values (since it already had these values).  But, I just found a typo in the code.  I'll test it.

BTW, the reason I didn't notice it before is the only animation we were doing was using the Tween library, and it takes only time, not deltaTime.

ok, update is checked in.  Please try it.


Can not use spark-solr in intellij

I have created a small project in Intellij with built.sbt below. I would like to run all spark-solr applications coming with https://github.com/lucidworks/fusion-spark-bootcamp.All those are using spark-solr library.
//built.sbt
name := "SparkScala"
version := "0.1"
libraryDependencies += "org.scala-lang" % "scala-library" % "2.12.6"
libraryDependencies += "com.lucidworks.spark" % "spark-solr" % "3.4.4"
But sbt is not initializing  and throwing error:
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]  ::          UNRESOLVED DEPENDENCIES         ::
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]  :: org.restlet.jee#org.restlet;2.3.0: not found
[warn]  :: org.restlet.jee#org.restlet.ext.servlet;2.3.0: not found
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]
[warn]  Note: Unresolved dependencies path:
[warn]          org.restlet.jee:org.restlet:2.3.0
[warn]            +- org.apache.solr:solr-core:7.2.1
[warn]            +- com.lucidworks.spark:spark-solr:3.4.4 (C:\InterviewPreps\react\SparkScala\build.sbt#L7-8)
[warn]            +- default:sparkscala_2.12:0.1
[warn]          org.restlet.jee:org.restlet.ext.servlet:2.3.0
[warn]            +- org.apache.solr:solr-core:7.2.1
[warn]            +- com.lucidworks.spark:spark-solr:3.4.4 (C:\InterviewPreps\react\SparkScala\build.sbt#L7-8)
[warn]            +- default:sparkscala_2.12:0.1
[error] sbt.librarymanagement.ResolveException: unresolved dependency: org.restlet.jee#org.restlet;2.3.0: not found
[error] unresolved dependency: org.restlet.jee#org.restlet.ext.servlet;2.3.0: not found
[error]         at sbt.internal.librarymanagement.IvyActions$.resolveAndRetrieve(IvyActions.scala:334)
[error]         at sbt.internal.librarymanagement.IvyActions$.$anonfun$updateEither$1(IvyActions.scala:208)

I think you need to add this to the repositories/resolvers section in the sbt file https://github.com/LucidWorks/spark-solr/blob/b6defa4839a40a5101cdcaf2237f73ae0ac32574/pom.xml#L50. That should resolve those two dependencies.

Check #195  too

Thanks for instant help
I converted sbt to pom.xml and added your repositories and not getting error.
But now, I am getting compile error when I compile (https://github.com/lucidworks/fusion-spark-bootcamp/blob/master/labs/movielens/load_solr.scala)
For example, compiler is not resolving SparkShellSupport in first line  and udf in the second
val dataDir = com.lucidworks.spark.util.SparkShellSupport.prop("dataDir").get
spark.udf.register("toInt", (str: String) => str.toInt)
Any thing I missed?
Thanks

ah, looks like you are using Fusion. Instead of adding a dependency on spark-solr, you should add the lib folder as deps ${FUSION_HOME}/apps/libs. SparkShellSupport comes from Fusion libs

That is my fault. I need to create Spark session manually. That issue has gone. But I am getting different problem. Fusion is running in local vagrant machine and try to connect from host machine. I have set up "zkhost" but getting this problem
var writeToSolrOpts = Map("collection" -> "users", "soft_commit_secs" -> "10", "zkhost" -> "<IP_ADDRESS>:9983")
userDF.write.format("solr").options(writeToSolrOpts).save
Exception in thread "main" com.google.common.util.concurrent.UncheckedExecutionException: org.apache.solr.common.SolrException: Cannot connect to cluster at <IP_ADDRESS>:9983: cluster not found/not ready
at com.google.common.cache.LocalCache$Segment.get(LocalCache.java:2263)
at com.google.common.cache.LocalCache.get(LocalCache.java:4000)
at com.google.common.cache.LocalCache.getOrLoad(LocalCache.java:4004)
at com.google.common.cache.LocalCache$LocalLoadingCache.get(LocalCache.java:4874)
at com.lucidworks.spark.util.SolrSupport$.getCachedCloudClient(SolrSupport.scala:246)
at com.lucidworks.spark.util.SolrSupport$.getSolrBaseUrl(SolrSupport.scala:250)
at com.lucidworks.spark.SolrRelation.insert(SolrRelation.scala:630)
at solr.DefaultSource.createRelation(DefaultSource.scala:27)

We should move to fusion-spark-bootcamp. I don't think this issue is related to spark-solr anymore


IFE eating piggyback bond on the wrong side

Issue Type

[x] bug report
[ ] feature request

Kudo @pdobacz for realizing this as an real issue and not MoreVP design.
Slack discussion: link
Current Behavior
Once piggybacked on the wrong side of canonicity, user would not get the piggyback bond back. The bond would be lock on the contract forever.
Expected Behavior
Return bond even on the wrong side of canonicity.
Steps to Reproduce
Alice creates tx1 spending utxo1 to Bob
Alice submits, tx1 withheld, Alice IFEs
Alice piggybacks on utxo1
Bob piggybacks on his output
Ife finalizes canonically
Alice doesn't get pb bond back

Alice and Carol create tx1 spending utxo1 and `utxo2` respectively to Bob
Alice submits, tx1 withheld, Alice IFEs
Alice piggybacks on utxo1 , Carol piggybacks on utxo2
Bob piggybacks on his output
Ife finalizes canonically
Alice & Carol don't get pb bond back,

Suggested Fix
Implement return piggyback bond not only on one side.
Motivation for Change
protocol correctness

I'd suggest adding those follow-up items to this effort:

generate more events to let consumers know in the cases, where there are only input piggybacks and the IFE finalizes canonically (and vice versa). Currently, there would be no events I think, and also deleteIFE facility would not work
update the morevp.md to be more explicit about the scenarios, where the pb bond must be returned. This could be the 2 scenarios posted in the description :point_up:
when all of that is done, I think elixir-omg must also recognize the scenarios where there are piggybacks "on the other end", i.e. input piggybacks for canonical and output piggybacks for non-canonical. I'm afraid this is not catered for and would require the said events to fully work.

also :heart: the issue title :joy:, much jargon


Error while trying to sync Knative Service

Steps to recreate the error:

Setup Knative serving on physical clusters.
Run startKcpAndClusterController.sh script after making the following update in the script (as we want to sync knative service):

${KCP_ROOT}/bin/cluster-controller -push_mode=true -pull_mode=false -kubeconfig=${KUBECONFIG} services.serving.knative.dev &> cluster-controller.log &
CC_PID=$!

Error received in the cluster-controller logs:

I0819 16:36:36.719458  254618 discovery.go:157] processing discovery for resource services (services.serving.knative.dev)
I0819 16:36:36.726736  254618 discovery.go:157] processing discovery for resource services (services.serving.knative.dev)
E0819 16:36:36.740409  254618 apiimporter.go:178] error creating APIResourceImport services.us-west1.v1.serving.knative.dev: APIResourceImport.apiresource.kcp.dev "services.us-west1.v1.serving.knative.dev" is invalid: [spec.columnDefinitions.format: Invalid value: "null": spec.columnDefinitions.format in body must be of type string: "null", spec.columnDefinitions.description: Invalid value: "null": spec.columnDefinitions.description in body must be of type string: "null"]
E0819 16:36:36.746479  254618 apiimporter.go:178] error creating APIResourceImport services.us-east1.v1.serving.knative.dev: APIResourceImport.apiresource.kcp.dev "services.us-east1.v1.serving.knative.dev" is invalid: [spec.columnDefinitions.description: Invalid value: "null": spec.columnDefinitions.description in body must be of type string: "null", spec.columnDefinitions.format: Invalid value: "null": spec.columnDefinitions.format in body must be of type string: "null"]

This issue is fairly old and there has been quite a few changes as we've worked through prototypes.  Mind confirming that you're still seeing this? Thanks!

I am not able to reproduce:

Start kcp as a "physical cluster"
Start kcp (which now runs the cluster controller by default) configured to sync services.serving.knative.dev
Register the "physical cluster" kcp as a Cluster with the regular kcp

$ kubectl get apiresourceimports.apiresource.kcp.dev -o wide
NAME                                   LOCATION   SCHEMA UPDATE STRATEGY   API VERSION              API RESOURCE   COMPATIBLE   AVAILABLE
services.kcp2.v1.serving.knative.dev   kcp2       UpdateUnpublished        serving.knative.dev/v1   services       True

If you are able to reproduce, please reopen and we'll investigate!


When pann extracts audio features with much smaller dimensions than video features, the code runs incorrectly.

Hi，thanks for your work! I'm running into some problems and I'd like to ask you some questions.

I have a video which duration is 116s，saliency scores(ground truth) size(58,3)，relevant_clip_ids size(58)， audio features size(33,2048)，video features size(59, 2816) ，ctx_l=33, and then I got an error :
cg_detr_dataset.py", line 301, in get_saliency_labels_all
score_array_new[:ctx_l] = score_array
could not broadcast input array from shape (34,) into shape (33,)
def get_saliency_labels_all(self, rel_clip_ids, scores, ctx_l, max_n=1, add_easy_negative=True):
 ...

 # score_array = [min(agg_scores[idx], ctx_l-1) for idx in range(ctx_l)]
 score_array = np.zeros(ctx_l)
 for idx in range(len(rel_clip_ids)):
     if rel_clip_ids[idx] >= ctx_l:
         score_array_new = np.zeros(ctx_l + 1)
         score_array_new[:ctx_l] = score_array
         score_array = score_array_new
     score_array[rel_clip_ids[idx]] = agg_scores[idx]

I find that, when the element in relevant_clip_ids is greater than ctx_l, it will report an error here： score_array_new[:ctx_l] = score_array. How can I change this? And I'm wondering that why  audio features dimensions could be smaller so much than video feathures，will such data bring negative impact in training process?

I was hoping to make some tweaks based on that work that would make it possible for the model to input just the video, not the prompt, and for the model to give a score for each 2 second clip.My dataset was labeled with a score for every 2 seconds clip without mr windows，can I just set model_inputs["span_labels"] = torch.tensor([[0., 0.]]) and q_feat = np.zeros([32,512]) in dataloader?

@ffiioonnaa Thank you for your feedback!

I have a video which duration is 116s，saliency scores(ground truth) size(58,3)，relevant_clip_ids size(58)， audio features size(33,2048)，video features size(59, 2816) ，ctx_l=33, and then I got an error :
cg_detr_dataset.py", line 301, in get_saliency_labels_all
score_array_new[:ctx_l] = score_array
could not broadcast input array from shape (34,) into shape (33,). I find that, when the element in relevant_clip_ids is greater than ctx_l, it will report an error here： score_array_new[:ctx_l] = score_array. How can I change this?

Is this your own dataset, not QVHighlights? The current training code supports only the benchmarks, so I didn't meet this issue. Let me take some time to reproduce this issue for one week.

And I'm wondering that why audio features dimensions could be smaller so much than video feathures，will such data bring negative impact in training process?

Yes, I think that the length of audio features should be equal to video features. For example, in QVHighlights, the video length is 75 (0.5 fps, 150s videos, 75 x video_dim) and the audio feature has also 75 length (75 x audio_dim).

I was hoping to make some tweaks based on that work that would make it possible for the model to input just the video, not the prompt, and for the model to give a score for each 2 second clip.My dataset was labeled with a score for every 2 seconds clip without mr windows，can I just set model_inputs["span_labels"] = torch.tensor([[0., 0.]]) and q_feat = np.zeros([32,512]) in dataloader?

Let me take 1 week to investigate it. If you can, could you share one example video-query pair with me? My mail address is<EMAIL_ADDRESS>

I conclude that this error occurs due to the difference of video and audio feature length. I close this issue. In the future version, I will prepare for the custom dataset training.


Russian language

Let's teach it russian

@salikjan Think you could have a look over it when you have time. See how the Language class is used and those properties files. It's a nice chance to learn this part of the project :)

@salikjan Think you could have a look over it when you have time. See how the Language class is used and those properties files. It's a nice chance to learn this part of the project :)

@salikjan here, for starters, just have a look over the 2 properties files and see if they could be translated into russian, while respecting the same regexes. Let me know what you think :D

When i break down an issie into a few puzzles shoud i mark this somewhere on git or just in mind?
Files can be easily tranlsated to russian.
As i understand files shoud be commands_ru.properties, responses_en.properties?

@salikjan yes exactly, the 2 new files should be commands_ru.properties and responses_ru.properties
You add each puzzle in the code as a @todo. Such a todo will usually be located in a javadoc (it could also be a comment in an xml file for instance). You should put the todo in the code, closest to where the change should happen.
Here is an example
The format of a todo is:
@todo #parent_issue_number:30m/DEV Explanation of the puzzle here which will most likely
 be on multiple lines. Should be explanatory enough to get the job done, or point out an issue
 where the assignee can find more details.

For the puzzles that you will leave in the code, parent_issue_number is 174 (this issue).
Estimation can also be 1h if 30m is not enough. But usually, 30m is ok, since that puzzle can as well be broken down in multiple puzzles.

Ok. But where in code i should place @todo in the case of properties files which are not exist?

@salikjan In this case I would say above the Language class. More puzzles can be in the same place, one under the other. I say this because in that place there will also be the todo for implementing Russian class.

@salikjan can you pls make another PR, adding spaces at the beginning of each new line in the todos from Language? Also, put back the puzzle for translating responses.properties as well. A task should be opened, and I'm going to assign it to you :D

@salikjan Also, it seems that character encoding will be fun, but we'll see :D

@amihaiemil i thought about it )) may be we should negotiate the codepage? koi8-r, win-1251 or unicode so i could set it in my ide or external editor as like as notepad++

@amihaiemil now utf-8 is set in idea

@salikjan well, for now, just make that PR when you have time (to fix the puzzles) and let's close this issue. We'll talk later. I think UTF-8 should do the trick? Not sure, I'll have to study it a bit

@salikjan koi8-r worked for me, in Eclipse. I set the file encoding to that and now they seem to be displayed alright. I mean, I guess. They seem like paitings to me, haha :))

I have translated 1/3 of responses properties file. To be continued tomorrow ))

@salikjan Sure, thank you :D
Just a note: a puzzle is intended to delegate the task to someone else. So you add a puzzle in the code not for yourself to keep track of your work, but for someone else to take it up later. Well, in this particular case, you would have done it anyway, since you're the only russian speaker. But if there was someone else, the puzzle might have been taken up by them.
I added this, because it gave me the impression you added the puzzle in order for you to track your work. You also said something about "keeping the puzzles in mind" :D

What is the attitude in responses should be when I see you. You? or thou? Beacuse in russian two words used for this. Polite You and not polite Thou.

@salikjan casual, so not necessarily polite, is ok. "you" should be fine. I doubt anyone will ever get upset about this. Those people do not use Github, for sure :))

ok )) what about delegating? i should tell you that I've finished working on issue and you can delegate issue to someone else?

@salikjan You simply leave the puzzles in the code, and when the branch is merged, one issue for each puzzle should be opened automatically (by 0pdd). Now one issue should have been opened, since you let a todo, but I think it didn't work because it misses the space at the beginning of the new sentence (I'll add it).
So, when you add a puzzle, you're basically saying "I let this part of the task to someone else".
When the issue gets created, yes, I'll assign the task to someone or maybe do it myself.

@salikjan I'm closing this since the property files are done. Btw, you are now listed as a contributor on the site as well: http://charles.amihaiemil.com/


terminate peerconnection when we believe it has reached an unusable state

If we think the peerconnection has become unuseable, we should just pro-actively terminate it. This should give a better user experience when users run into the out-of-order close events issue which, sadly, seems quite easy to trigger on medium to high latency connections.
Related to this issue:
https://github.com/uProxy/uproxy/issues/1289

Attention, @mollyling and @bemasc.

Worth looking into whether we can integrate reporting of these events into the rappor metrics system.

Yes, worth having success/failure metrics in there.

LGTM

Not clear that we still want this with pool management.

Right! Dropping this.


Transport websocket send test

PR summary
This PR fixes several remaining problems from  #118:

sets non_blocking to true on the tcp acceptor socket (fixes blocking issue)
adds a static counter to both the memory and websocket id generator to reduce id collisions between transports
generalizes the transport::tests::send_test to support websockets

process loops are executed a constant number of times rather than just once to complete the tls hand shaking
both nodes A and B are now processed in the loop rather than just one or the other
the uri equal to connection id assertions instead now assert existance of a uri given a connection id but no longer verify exact values (might require some discussion)

to support similar transport::get_uri functionality in the memory trait that the websocket trait already has, the memory trait's get_uri function now checks if its bound server has the connection id- if it does it returns its own uri.
uncomment out the wss_send_test, which now passes

Review checklist

[x] The story has unit or integration tests

uncommented out the transport wss_send_test which tests a sending of a payload to and from a pair of nodes

[x] No new bugs, and any tech-debt is identified and justified

need to discuss semantics of get_uri again to make sure it's legitimate

[ ] There is enough API documentation (how to use)
[ ] There is enough code documentation (how the code works)

and thanks for adding the counter to TransportMemory! 🙏


remove useless parenthesis and ellipse

I like some of the changes here, but removal of parens in argument pattern matching of records does not work for me. If you would be willing to revert those, I’ll merge this one.

Should be fixed now.

Thank you!


afpacket: fix NewPacketSource params GOOS!=linux

The signature of afpacket.NewPacketSource was updated in readwriter.go (29ca59d5) for GOOS=linux, but the signature for that same function in readwriter_other.go for GOOS!=linux was not updated to match. This causes a build failure when GOOS!=linux. The build failure is not critical since only GOOS=linux is supported .

You are absolutely right. Thanks for your help.


[BUG] -

External   otherwise.
Area
Other   node 1.33.0 building
Summary
I am updating node software for a stake pool operation. The cabal build ends with the following error:
Configuring library for katip-<IP_ADDRESS>..
Preprocessing library for katip-<IP_ADDRESS>..
Building library for katip-<IP_ADDRESS>..
[1 of 5] Compiling Katip.Core       ( src/Katip/Core.hs, dist/build/Katip/Core.o, dist/build/Katip/Core.dyn_o )
src/Katip/Core.hs:114:74: error:
• No instance for (TH.Lift Text)
arising from the first field of ‘Namespace’ (type ‘[Text]’)
Possible fix:
use a standalone 'deriving instance' declaration,
so you can specify the instance context yourself
• When deriving the instance for (TH.Lift Namespace)
|
114 |   deriving (Eq,Show,Read,Ord,Generic,ToJSON,FromJSON,SG.Semigroup,Monoid,TH.Lift)
|                                                                          ^^^^^^^
cabal: Failed to build katip-<IP_ADDRESS> (which is required by exe:tx-generator
from tx-generator-1.32, test:cardano-testnet-tests from cardano-testnet-1.33.0
and others). See the build log above for details.
Steps to reproduce
Steps to reproduce the behavior:

I am running node version 1.32.1 on all relays and BP
compiling with ghc 8.10.2
cabal update
cabal build all

Expected behavior
I have never had the build fail in well over a year of updating/upgrading nodes to new releases. Could it be that I need a newer version of GHC?
System info (please complete the following information):

Ubuntu
20.04
currently running 1.32.1, attempting to build and update to 1.33.0 following new release
as above

Thank you, Stef (SPO RABIT)

You're using old ghc, use 8.10.4/7 instead

thank you, I am trying. Will post result here and close issue when resolved.

@rdlrt,
I have tried several combinations of GHC/CABAL and the build works with the following:
-GHC 8.10.4 / CABAL 3.2.0
-GHC 8.10.7 / CABAL 3.2.0
-GHC 8.10.7 / CABAL 3.4.0
Thus far I can confirm a slight performance increase on the RAM side of things (approximately 2.5GiB less), but the boot time (after the initial replay from genesis) is still way slower than it used to be before 1.32.1 (roughly 5 minutes).
Is there a way to explicitly force the usage of multiple threads on startup?
Thank you for your time and help,
S


Don't monkey-patch any prototypes

The code here modifies String.prototype, adding some defunct regexp-based methods. My code was using this more robust method, and I had to switch from a passive ?= to a more aggressive = to override it.
String::startsWith ?= (str)->
    @substring(0, str.length) is str

The rest of the code doesn't seem to be using these methods anyways.

Thanks, I'll take a look.

Thanks!

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Why can't we determine the primitive type first?

Most of the js executed in our project get some primitive types, such as string, number, etc. Is there a way to judge in advance

Getting primitive properties from a V8 object doesn't hit this code path. These calls are optimized with direct type conversion without if-else.
Some reference types can also be primitive types so that they must go through the reference type checks firstly.

If there are too many invalid judgments, will it affect performance? Can you give an example to illustrate whether the reference type includes the original type? Are some complex return types really used in actual applications?

I consult GPT and conclude that there will be no problem if you first determine the original type.


minor tweaks on AATree Node methods

This looks like a "how did this ever work?" kind of thing.
That's exactly what I thought. Scary. The old code definitely looks wrong.

@mdittmer yes I'll try to work on tests for the whole AATree.

Goes all the way back to FOAM1: https://github.com/foam-framework/foam/blob/c321d7d5ffc621e84a67fab85cf45155d1af947c/core/index.js#L446
LT case looks to be the correct way, so we may just have never (throroughly) tried an LTE.


nixos/{hyprland, wayland-common}: disable wlr portal for hyprland, enable xdg autostart for all wayland compositors

Description of changes
Hyprland has its own portal xdg-desktop-portal-hyprland and does not need the wlr portal.
As a follow up to #240989,

Adds wlr-portal override of wayland-session module (enabled by default)
Disable it for hyprland module

Additionally, enable xdg autostart service for Wayland Window compositors as unlike DEs, they do not handle it themselves.
Things done

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

CC: @fufexan @SuperSandro2000

Add a :+1: reaction to pull requests you find important.

@fufexan is there a reason you are not in meta.maintainers of nixos/hyprland module? :)

AFAIK it's not required to have a maintainer for a module, so I didn't add myself when I created it.
Feel free to add me though, it may allow for greater visibility in issues.

Feel free to add me though, it may allow for greater visibility in issues.

Exactly my point. Done

I plan to backport this to stable as well, but #314978 should be merged for backport to work.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4014

@fufexan perhaps also add services.xserver.desktopManager.runXdgAutostartIfNone = true to this?
Description

@fufexan perhaps also add services.xserver.desktopManager.runXdgAutostartIfNone = true to this?
From the description:

Whether to run XDG autostart files for sessions without a desktop manager (with only a window manager), these sessions usually don’t handle XDG autostart files by default.

Might make sense. Would it conflict with the HM option? It is being set to true by default in the HM module for Hyprland.

The mechanism behind the option on Nixpkgs is just a systemd service.
I feel like Home Manager, by itself, has a nice mechanism for avoiding conflicts.
For example, some bash and zsh options are enabled by default in nixos configuration, but if their counterpart options are disabled in home manager module, NixOS respects and prioritises Home manager modules.
I believe if any actions are to be needed, they need to be performed on the Home Manager side. Home manager bases its development off Nixpkgs master, not the other way around.

I added it to the common wayland-session module, so that other WMs can utilise this as well.

@fufexan is this pull request good to go?

Is there a particular reason for which this change isn't flagged with a lib.mkDefault? In my configuration, I set it to true (without mkForce or mkDefault) but it now has conflicting definitions since my last update.
error: The option `xdg.portal.wlr.enable' has conflicting definition values:
- In `/nix/store/aacfsg6r005hi7gna3s1vlpzrg5v5264-source/hosts/shared/pc/hyprland.nix': true
- In `/nix/store/8s55w0927lh3mdbkxf434zb0c5hqsz8z-source/nixos/modules/programs/wayland/hyprland.nix': false
Use `lib.mkForce value` or `lib.mkDefault value` to change the priority on any of these definitions.

FYI you don't need the wlr portal if you use hyprland. The hyprland portal was forked from thn wlroots one.

Is there a particular reason for which this change isn't flagged with a lib.mkDefault?

That people will notice it and hopefully remove it.

? ```
error:
Failed assertions:
- To use Flatpak you must enable XDG Desktop Portals with xdg.portal.enable.

       error:
       Failed assertions:
       - To use Flatpak you must enable XDG Desktop Portals with xdg.portal.enable.

Please create a new issue with config details @wxlyyy


Preview PSD in Browser

Hi,
I am trying to view the file example.psd in chrome, but without success.
Please tell me what I'm doing wrong.
Thank you.
This is my code:
<!DOCTYPE html>

Are you loading the webpage through a server or are you just opening the HTML directly in the browser? Due to browser security restrictions, the files have to be served through a server.

I tried to open the PSD directly in the browser.
Thank you.


release HMI_SDK 1.0

@liukang325
Please add description for your pull request.
Please '@' other people to review your codes.

release HMI_SDK 1.0 code. Each platform to compile successed, include win32, linux, android, wince.
@VENGEL  please review codes

@smartdevice475 @ilikegithubcom @bahamut65536 @VENGEL
Please join to review

@xuwenlong02 please also join to review

reviewed

@henry-fan @yifili09 Please kindly review if you have any comment.

Reviewed


feat(logging): abstract logging methods to macros

Component
Other (please specify)
Describe the feature you would like
Instead of instantiating new Loggers in each method and passing references around, allLogger methods should be converted to macros that:

get verbosity from the RUST_LOG environment variable (this is set by heimdall-cli, so it's accurate)
check if verbosity is greater minimum verbosity for that log. (error >= 0, warn >= 1, info >= 2, etc.)
a. if it is, send the log
b. if not, return!

Ex: https://github.com/Jon-Becker/heimdall-rs/blob/main/common/src/utils/io/macros.rs
Then, the PR should delete all instances of Logger. Trace TraceFactory still make sense in their current implementations, so these structs should be preserved.

I'll take this one 😄 🏅


For Published Apps, URL Filter only works for IE 11

Using URL parameters to filter a report works for reports in workspaces--where the URL is in the form of:
https://app.powerbi.com/groups/___/reports/___/ReportSection?filter=____
However, when the URL is for a published app, the filtered URL only works in the IE 11 browser. It does not work in Chrome or Microsoft Edge. That is, when the URL looks like:
https://app.powerbi.com/groups/me/apps/___/reports/___/ReportSection?filter=___
This documentation should note that reports in workspaces and reports in published apps do not have the same behavior.

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

ID: 70b360f3-21a7-adc2-896a-b1dfd5d9e384
Version Independent ID: 15e8c09f-46ad-68f9-75e6-c3d5c2c214e5
Content: Filter a report using query string parameters in the URL - Power BI
Content Source: powerbi-docs/service-url-filters.md
Service: powerbi
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @dejadee -- I've checked with the product team. They point out a clarification I need to add to the article:
Links to reports in an app have a query parameter (ctid) that gets added to the URL. Query parameters need to be separated by &’s. So you need to append the query with “&filter=” (after the ctid parameter) instead of “?filter=”. Like this:
app.powerbi.com/groups/me/apps//reports//ReportSection
?ctid=xxxxxxxxxxx&filter=Table~2FName%20eq%20%27A%27

Hi Maggie,
After about an hour of trying to get this to work, I have failed!  I do not see any "ctid" generated when I look at a link to a Power BI app.  Thus, I don't know where / how to append the code you discuss above.  We are not running Premium.  Can you please advise how / where to find the CTID?  Or, if it's not visible, how it can be found?  Here is the link when the app loads:
https://app.powerbi.com/groups/me/apps/c1d03b08-8706-47a7-8da6-a5294697d87a/reports/4f336567-1c3f-44b1-bf6e-c0e5a7acc419/ReportSection00b3a19702d303247519

Hi, @jburns157 -- Hmmm, agreed, that's a puzzle. I just tried creating a query string for a report in an app. The URL didn't have 'ctid' in it either. So I created the URL without the ctid and it worked fine. Try that.
Meanwhile I'll check some more.
thx!

Hi!  Thanks for helping.  When you say that you built the URL without the ctid, does that mean (for the below link), that you chopped off the last section that happens after "ReportSection"?  And are you using the "?" or the "&"?
https://app.powerbi.com/groups/me/apps/c1d03b08-8706-47a7-8da6-a5294697d87a/reports/4f336567-1c3f-44b1-bf6e-c0e5a7acc419/ReportSection00b3a19702d303247519
Thanks!

Hi, @jburns157  -- I used the ? because you only need the & to separate parameters, and I had only one. So it was:
ReportSection5?filter=/ eq 'value'
But now I'm wondering. I created the app myself from an app workspace, so even though I'm viewing it in the app, it may not be acting as it would if the app were shared with me. Need to check.

Oh! Git very helpfully removed the text because I put it in brackets. Here it is:
ReportSection5?filter=table-name/column-name eq 'value'
see if that shows up better

Hi Maggie,
I'm sorry for the delayed response!  I just got back to my desk.  I have tried this and it does not work, sadly.  The browser just routes back to app.powerBI.com.
I have pondered if the random string in the URL that comes after "ReportSection" might be the culprit, but I have not been able to figure it out.

Hi, @jburns157 -- yup, it was because I was the author. When I tried same with a report in an app I couldn't edit, I did indeed get the ctid. And when I added it after the ?filter = part, it worked. Like this:
https://app.powerbi.com/groups/me/apps/_app-id_/reports/_report-id_/ReportSection3?filter=_Table/field_ eq 'value'&ctid=ctid

Hi @maggiesMSFT , I just tested this extensively.  Very strange - it worked a single time for me.  After the initial success, it repeatedly fails now.  When I click the link now, it shows the proper URL, thinks for a millisecond, and then replaces the URL with the "?filter" logic with just app.powerbi.com, and it loads a blank page.  I took my test user out of all workspaces - all the user has access to is the app itself.

Hi, @jburns157 -- hmmm. Sounds like time to hit up the community and/or support sites, I'm afraid.
https://community.powerbi.com/ and https://powerbi.microsoft.com/en-us/support/

Has this been updated to work in Chrome or Firefox?

Hi there mcSmith3, I am not sure about Firefox... but I can tell you that it is working great for me on Chrome.  Seems like the problem just went away for me.


Update readme to reflect getUserMedia support on iOS

Small README PR to update the support table to show that getUserMedia will work in some situations on iOS 11.

This looks great. One suggestion: is there a way to make it clear that other browsers like Chrome iOS are using Webview, and therefore are not supported?

Good call, thanks @michaeltrollan


[OpenMP][OMPT] Add cmake-based libomptest example

For now this is primarily intended for demonstration purposes, e.g. how to use find_package in conjunction with libomptest.

Removed related change within Makefile.defs into its own PR: #730

Yes, good point. Now that I think about all of this, there are three questions on my mind:

How do we want to connect the Makefile with the suggested bash script + CMakeLists.txt?
(Just call the script which handles the build and run?)
Where do we want to put the user-defined information?
How much customization do we want to allow through possible parameters / options of the script?

Yes, good point. Now that I think about all of this, there are three questions on my mind:

How do we want to connect the Makefile with the suggested bash script + CMakeLists.txt?
(Just call the script which handles the build and run?)

I would just call it (w/ necessary parameters)

Where do we want to put the user-defined information?

Not sure I understand this. I have imagined that we can call the script with ./run_the_test.sh $(AOMP) or something similar in the RUNCMD of the smoke Makefile and have all other parts be relative.

How much customization do we want to allow through possible parameters / options of the script?

For our smoke tests?  None.

edit: 2. -> Tend to CMakeLists.txt. 3. -> Background: Since we allow to change the project name, my apprehension is how to handle (multiple) source files which are not named like the project itself. (That example could be handled by leaving this on default / alone and putting the info in CMakeLists.txt)

I'm not sure which project name you are referring to, but if it's the smoke test project name, I would simply not make that configurable, but give it the test name.
While this test is using the omptest machinery, I don't see this as an example (for teaching / example puposes) or "How-To" of what could be done.


Dark toolbar text color on smartphones when using light theme

When using the library's default light theme, the text color of a PreferenceActivity's toolbar is dark on smartphones (see attached screenshot). On tablets the text color is white as expected.

Fixed in version 4.2.5.


[PR][CLOSED] Update README.md - @open sesame 06/25 14:39

 Issue by geunsik-lim
Monday Jun 25, 2018 at 05:37 GMT
Originally opened as https://github.sec.samsung.net/STAR/TAOS-CI/pull/55

It's trivial. Updated usage cibot_comment() and cibot_pr_report()
Signed-off-by: Geunsik Lim<EMAIL_ADDRESS>
geunsik-lim included the following code: https://github.sec.samsung.net/STAR/TAOS-CI/pull/55/commits

:octocat: cibot: Thank you for posting issue #79. The person in charge will reply soon.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:37 GMT

If you want to monitor the current build status of your PR, Please refer to http://aaci.mooo.com//TAOS-CI/ci/standalone/ci-server/monitor.php

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:37 GMT

To contributor, We have used 'Signed-off-by:' notation by default to handle the license issues, that result from contributors. Note that 'Is there a Signed-off-by line?' is important because lawyers tell us we must have to it to cleanly maintain the license issues such as GPL and LGPL even though it has nothing to do with the code itself.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:37 GMT

:dart: cibot: @STAR has updated the pull request.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:37 GMT

:octocat: cibot: @STAR, One of the format checkers is failed. If you want to get a hint to fix this issue, please go to https://github.sec.samsung.net/STAR/TAOS-CI/wiki/.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:37 GMT

To contributor, We have used 'Signed-off-by:' notation by default to handle the license issues, that result from contributors. Note that 'Is there a Signed-off-by line?' is important because lawyers tell us we must have to it to cleanly maintain the license issues such as GPL and LGPL even though it has nothing to do with the code itself.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:38 GMT

To contributor, We have used 'Signed-off-by:' notation by default to handle the license issues, that result from contributors. Note that 'Is there a Signed-off-by line?' is important because lawyers tell us we must have to it to cleanly maintain the license issues such as GPL and LGPL even though it has nothing to do with the code itself.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:38 GMT

:dart: cibot: @STAR has updated the pull request.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:38 GMT

:octocat: cibot: @STAR, One of the format checkers is failed. If you want to get a hint to fix this issue, please go to https://github.sec.samsung.net/STAR/TAOS-CI/wiki/.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:38 GMT

:octocat: cibot: @STAR, One of the format checkers is failed. If you want to get a hint to fix this issue, please go to https://github.sec.samsung.net/STAR/TAOS-CI/wiki/.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:38 GMT

:dart: cibot: @STAR has updated the pull request.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:38 GMT

:dart: cibot: @STAR has updated the pull request.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:38 GMT

:octocat: cibot: @STAR, One of the format checkers is failed. If you want to get a hint to fix this issue, please go to https://github.sec.samsung.net/STAR/TAOS-CI/wiki/.

 Comment by git-bot-sec
Monday Jun 25, 2018 at 05:44 GMT

:dart: cibot: @STAR has updated the pull request.


File API support?

Hi!
Any plans to allow parse shp-files uploaded via File API?
Thanks!

you can use fileReader.readAsArrayBuffer to get the file as an arrayBuffer which you can just pass in here

Thanks for reply!
I try to load SHP via following block but get Error: Corrupted zip : can't find end of central directory. Any advise?
var reader = new FileReader();
reader.onload = function() {
    shp(this.result).then(function(geoJSON) {
        console.debug(geoJSON);
    }, function(e) {
        console.debug(e);
    });
};
reader.readAsArrayBuffer(files[0]);

double check that it is a valid zipfile from the command line?

Hm. It was not zip-file - just simple shp-file.
Is it possible to convert just shp-data without other parts (dbf and others)?

shp.parseShp
On Wed, Jun 10, 2015 at 9:01 AM Alexander Kuznetsov <
<EMAIL_ADDRESS>wrote:

Hm. It was not zip-file - just simple shp-file.
Is it possible to convert just shp-data without other parts (dbf and
others)?
—
Reply to this email directly or view it on GitHub
https://github.com/calvinmetcalf/shapefile-js/issues/42#issuecomment-110742631
.

I got it. Thanks!


missing login instructions, get JSON instead

Ubuntu24.04 AMD64, Following the "Setting up the UI" instructions, I basically did
sh -c 'cat <<EOF > /etc/apt/sources.list.d/zabbly-incus-stable.sources
Enabled: yes
Types: deb
URIs: https://pkgs.zabbly.com/incus/stable
Suites: $(. /etc/os-release && echo ${VERSION_CODENAME})
Components: main
Architectures: $(dpkg --print-architecture)
Signed-By: /etc/apt/keyrings/zabbly.asc

EOF';
apt update;
apt full-upgrade;
apt-get install incus incus-ui-canonical;
incus admin init; (using mostly just the default settings, no cluster)
incus config set core.https_address :8443;

and pointed my browser to https://localhost:8443/ui/login , after accepting the certificate, I just get this JSON:
{"type":"sync","status":"Success","status_code":200,"operation":"","error_code":0,"error":"","metadata":["/1.0"]}

and if I try to enter https://localhost:8443/ui/login I get
{"type":"error","status":"","status_code":0,"operation":"","error_code":404,"error":"not found","metadata":null}

Please advice

oops wrong repository i guess


python310Packages.griffe: 0.26.0 -> 0.27.0

Diff: https://github.com/mkdocstrings/griffe/compare/refs/tags/0.26.0...0.27.0
Changelog: https://github.com/mkdocstrings/griffe/blob/0.27.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 225849 run on x86_64-linux 1

1 package marked as broken and skipped:

netbox_3_3

1 package failed to build:

netbox

8 packages built:

griffe (python310Packages.griffe)
griffe.dist (python310Packages.griffe.dist)
python310Packages.mkdocstrings-python
python310Packages.mkdocstrings-python.dist
python311Packages.griffe
python311Packages.griffe.dist
python311Packages.mkdocstrings-python
python311Packages.mkdocstrings-python.dist

@ofborg build netbox

Result of nixpkgs-review pr 225849 run on x86_64-linux 1

9 packages built:

griffe (python310Packages.griffe)
griffe.dist (python310Packages.griffe.dist)
netbox
python310Packages.mkdocstrings-python
python310Packages.mkdocstrings-python.dist
python311Packages.griffe
python311Packages.griffe.dist
python311Packages.mkdocstrings-python
python311Packages.mkdocstrings-python.dist

is there a reason you are updating netbox too here?

is there a reason you are updating netbox too here?

nix-review and CI were failing.

Result of nixpkgs-review pr 225849 run on x86_64-linux 1

9 packages built:

griffe (python310Packages.griffe)
griffe.dist (python310Packages.griffe.dist)
netbox
python310Packages.mkdocstrings-python
python310Packages.mkdocstrings-python.dist
python311Packages.griffe
python311Packages.griffe.dist
python311Packages.mkdocstrings-python
python311Packages.mkdocstrings-python.dist

netbox update removed as with the latest griffe the build works.

@ofborg build python311Packages.griffe netbox


Adding --enable-managed-identity true fails on create

Describe the bug
Command Name
az apim create --name $name --publisher-email<EMAIL_ADDRESS>--publisher-name "bu" -g $resourceGroup --enable-managed-identity true
Errors:
'ApiManagementServiceResource' object does not support item assignment
Traceback (most recent call last):
Temp\pip-install-9101vebg\knack\knack\cli.py, ln 206, in invoke
azure\cli\core\commands\__init__.py, ln 608, in execute
azure\cli\core\commands\__init__.py, ln 666, in _run_jobs_serially
azure\cli\core\commands\__init__.py, ln 659, in _run_job
Local\Temp\pip-install-9101vebg\six\six.py, ln 703, in reraise
azure\cli\core\commands\__init__.py, ln 636, in _run_job
azure\cli\core\commands\__init__.py, ln 306, in __call__
azure-cli-core\azure\cli\core\__init__.py, ln 493, in default_command_handler
azure\cli\command_modules\apim\custom.py, ln 31, in create_apim
TypeError: 'ApiManagementServiceResource' object does not support item assignment

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az apim create --name {} --publisher-email {} --publisher-name {} -g {} --enable-managed-identity {} --no-wait

Expected Behavior
Environment Summary
Windows-10-10.0.18362-SP0
Python 3.6.6

azure-cli 2.2.0 *

Extensions:
application-insights 0.1.1

Additional Context

hi @qwordy could you pls have a look? thanks

@kevinhillinger @jonlester Are you still the code owner of apim?

Looking into this.


After upgrading from 1.6.3 to 1.7.0-rc.0 fonts are corrupted when project build is executed

Versions
Angular CLI: 1.7.0-rc.0
Node: 9.3.0
OS: darwin x64
Angular: 5.2.4
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 1.7.0-rc.0
@angular-devkit/build-optimizer: 0.3.1
@angular-devkit/core: 0.3.1
@angular-devkit/schematics: 0.3.1
@ngtools/json-schema: 1.2.0
@ngtools/webpack: 1.10.0-rc.0
@schematics/angular: 0.3.1
@schematics/package-update: 0.3.1
typescript: 2.5.3
webpack: 3.10.0

Repro steps
It is enough to create vanilla project and to add usage of fonts, for example, you can add dependencies "font-awesome" and "angular-font-awesome", then make the changes to the "app.component.html" by adding <fa name="cog" animation="spin">Sorry, but something went wrong with my fonts</fa>
then execute the following commands

yarn
ng serve

A link to a sample repo
contains all mentioned above changes
Observed behavior
When you run the app and open the console you see the following messages
Failed to decode downloaded font: http://localhost:4200/fontawesome-webfont.68560c510686599ffbc9.woff2?v=4.7.0
localhost/:1 OTS parsing error: Failed to convert WOFF 2.0 font to SFNT
localhost/:1 Failed to decode downloaded font: http://localhost:4200/fontawesome-webfont.a7bbeb589ae63e018872.woff?v=4.7.0
localhost/:1 OTS parsing error: incorrect file size in WOFF header
localhost/:1 Failed to decode downloaded font: http://localhost:4200/fontawesome-webfont.e1dd9adb4f69bb9a0bdc.ttf?v=4.7.0
localhost/:1 OTS parsing error: incorrect entrySelector for table directory
localhost/:1 Failed to decode downloaded font: http://localhost:4200/fontawesome-webfont.912ec66d7572ff821749.svg?v=4.7.0#fontawesomeregular
localhost/:1 OTS parsing error: invalid version tag

And those HTML elements, which use fonts from corrupted files, do not have font symbols
Desired behavior
HTML elements, which use font symbols, should have them
Mention any other details that might be useful (optional)

All works fine if to use angular-cli 1.6.3.
Project was created using the command ng new <projectName> --style=scss
The checking of mentioned before fonts shows that they are really corrupted and have different size than original fonts from the catalog node_modules/font-awesome/fonts/

Same here, can build but my browser can't read the served fonts

Sorry, noticed that bug was reported before https://github.com/angular/angular-cli/issues/9557


Use a correct process name when start service from /etc/rc3.d/S87xxxx

$proc in /etc/init.d/xxxxxx is set different value by start method, because current template generate $proc from $0.
# echo `basename /etc/init.d/xxxxxxxx`
xxxxxxxx
# echo `basename /etc/rc3.d/S87xxxxxxxx`
S87xxxxxxxx

As a result of above difference, /var/run/xxxx.pid is confused.
Reproduce methods

Start service from /etc/rc3.d/S87xxxxxxxx while launch instance.
SSH login to it instance. and execute follow command.

# service xxxxxxxx status
xxxxxxxx is stopped
# ps aux | grep xxxxxxxx
root      1753  0.0  0.2 179124  5192 pts/1    Sl   11:41   0:00 /opt/xxxxxxxx
root      1770  0.0  0.0 103240   868 pts/1    R+   11:49   0:00 grep xxxxxxxx

Great! Thank you Sheile!


Plugin fails to resolve dependency from bom

Describe the bug
When using a platform dependency on the compileOnly configuration and using one of its dependencies in the api configuration, the plugin fails to resolve the dependency for the apiHelper configuration.
To Reproduce
Steps to reproduce the behavior:

configure the plugin in the following way

plugins {
    `java-library`
    id("ca.cutterslade.analyze") version "1.9.0"
}

dependencies {
    compileOnly(platform("com.squareup.okhttp3:okhttp-bom:4.9.3"))
    compileOnly("com.squareup.okhttp3:okhttp")
    api("org.jetbrains.kotlin:kotlin-stdlib")
}

execute command gradlew check
Output:

> Task :analyzeClassesDependencies FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':analyzeClassesDependencies'.
> Could not resolve all files for configuration ':apiHelper'.
   > Could not find org.jetbrains.kotlin:kotlin-stdlib:.
     Required by:
         project :

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 2s
2 actionable tasks: 1 executed, 1 up-to-date

Repository containing a sample project
URL: https://github.com/baardsen/gradle-dependency-analyze-platform
Versions
Java: 17
Gradle: 7.4.2
Plugin: 1.9.0
Expected behavior
Build should succeed.
If a dependency is resolvable in the api configuration it should be resolvable in the apiHelper configuration.

Is this a kotlin project?

No, not a kotlin project. That dependency is just a stand in for a dependency that we might want to expose in a library. The version comes from the okhttp bom (or possibly one of its transitive dependencies).
The problem is that while the api configuration is able to use the compileOnly configuration to resolve its dependencies, apiHelper is not. Setting that apiHelper extends from compileOnly does fix it though IMHO that requires more knowledge about the innards of the plugin than should be expected of a user.
I played around a bit and if I add the bom to the implementation configuration instead then I get that both apiHelper and compileOnlyHelper are unable to resolve the dependency. I suspect that there are some relationships between the "standard" configurations that aren't, but maybe should be, reflected in the *Helper configurations.

For the moment I can not really follow what Gradle is doing here. When I would convert this to an equivalent Maven config it dies not work as I would expect it.
Need to study the Gradle implementation to understand why extending API does not consider the platform but when used it is. Too many question marks right now.
Need also to check what will be in the Gradle descriptor file when such an artifact is published. Can not really think of a version there.
For the moment pls do not expect a quick fix

I looked at how the configurations are connected and I think it works because the api configuration is never resolved directly, only the compileClasspath is resolved which extends both api (via implementation) and compileOnly. But apiHelper is resolved directly and so it needs some extra help. For now I've set that it (and compileOnlyHelperTest) extends from compileOnly but I'm not sure if it has any unintended consequences.
You're correct that by default this setup does not include any versions in the published file. And since we don't want to expose the bom file we use we've configured the build to use the resolved version numbers when generating the pom file.

Good morning, we have a similar issue with Kotlin in a multi-module project (with Groovy for the Build-Scripts) where all our dependency versions (including kotlin-stdlib) are defined as api constraints in a custom platform module.
Our symptoms are:

a regular module that uses Kotlin is falsely reported to have unused declered dependencies
another regular module that depends on the first module is also falsely reported to have unused declared dependencies

Here is a minimal Kotlin-based sample to reproduce the issue
GdaMinimalReproduce-299.tar.gz

Still trying to add/improve kotlin support for the plugin but currently there are some major issues related to figuring out what classes are in use.

I run into a similar problem, but with moneta.
My build script:
plugins {
    java
    id("ca.cutterslade.analyze") version "1.10.0"
}

repositories {
    mavenCentral()
}

dependencies {
    implementation(group = "org.javamoney", name = "moneta", version = "1.4.4", ext = "pom")
}

And then the dependency analysis fails with:
Unsupported file for collecting classes: .../.gradle/caches/modules-2/files-2.1/org.javamoney/moneta/1.4.4/d79a8aa43a55e9d2341ffa956783327cb730037a/moneta-1.4.4.pom

Here is a codesandbox: https://codesandbox.io/p/devbox/l96s8m
Not sure if this is the same problem, but seems related.


[New game]: PRONTO 👾

🎮 Game Request
Game logic and basic description

A very simple reaction game about not getting off the line.

Point down the features
Game points
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @0xabdulkhalid !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign


Release Version 2.2.0

Release Version 2.2.0
This is a component issue for 2.2.0.
Coming from opensearch-build#2271. Please follow the following checklist.
Please refer to the DATES in that post.
How to use this issue

This Component Release Issue
This issue captures the state of the OpenSearch release, on component/plugin level; its assignee is responsible for driving the release. Please contact them or @mention them on this issue for help.
Any release related work can be linked to this issue or added as comments to create visiblity into the release status.
Release Steps
There are several steps to the release process; these steps are completed as the whole component release and components that are behind present risk to the release. The component owner resolves the tasks in this issue and communicate with the overall release owner to make sure each component are moving along as expected.
Steps have completion dates for coordinating efforts between the components of a release; components can start as soon as they are ready far in advance of a future release. The most current set of dates is on the overall release issue linked at the top of this issue.
The Overall Release Issue
Linked at the top of this issue, the overall release issue captures the state of the entire OpenSearch release including references to this issue, the release owner which is the assignee is responsible for communicating the release status broadly.  Please contact them or @mention them on that issue for help.
What should I do if my plugin isn't making any changes?
If including changes in this release, increment the version on 2.0 branch to 2.2.0 for Min/Core, and <IP_ADDRESS> for components. Otherwise, keep the version number unchanged for both.

Preparation

[x] Assign this issue to a release owner.
[x] Update Apache Lucene to 9.3.0
[x] Update OpenJDK (July 2022 Patch releases)
[ ] Finalize scope and feature set and update the Public Roadmap.
[ ] All the tasks in this issue have been reviewed by the release owner.
[ ] Create, update, triage and label all features and issues targeted for this release with v2.2.0.
[ ] Cut 2.2 branch
[ ] Bump 2.x to version 2.3.0 and update BWC versions
[ ] Update BWC version in main 3.0.0

CI/CD

[ ] All code changes for 2.2.0 are complete.
[ ] Ensure working and passing CI.
[ ] Check that this repo is included in the distribution manifest.

Pre-Release

[ ] Update to the 2.2.0 release branch in the distribution manifest.
[ ] Increment the version on the parent branch to the next development iteration.
[ ] Gather, review and publish release notes following the rules and back port it to the release branch.git-release-notes may be used to generate release notes from your commit history.
[ ] Confirm that all changes for 2.2.0 have been merged.
[ ] Add this repo to the manifest for the next developer iteration.

Release Testing

[ ] Find/fix bugs using latest tarball and docker image provided in parent release issue and update the release notes if necessary.
[ ] Code Complete: Test within the distribution, ensuring integration, backwards compatibility, and performance tests pass.
[ ] Sanity Testing: Sanity testing and fixing of critical issues found.
[ ] File issues for all intermittent test failures.

Release

[ ] Complete documentation.
[ ] Verify all issued labeled for this release are closed or labelled for the next release.

Post Release

[ ] Prepare for an eventual security fix development iteration by incrementing the version on the release branch to the next eventual patch version.
[ ] Add this repo to the manifest of the next eventual security patch version.
[ ] Suggest improvements to this template.
[ ] Conduct a retrospective, and publish its results.

Apache Lucene has been updated to 9.3.0 - https://github.com/opensearch-project/OpenSearch/pull/4088
OpenJDK has been updated with backports to 2.x, 2.0 and 2.1 - https://github.com/opensearch-project/OpenSearch/pull/4023

All thanks to @reta 🏅

2.2 branch created - https://github.com/opensearch-project/OpenSearch/tree/2.2
I'll work on bumping 2.x to 2.3.0 next

2.x has been bumped to 2.3.0 and BWC versions have been updated in main.
The manifest file in opensearch-build has been updated to point to the 2.2 branch rather than 2.x
I'm currently triaging all open issues tagged with v2.2.0

Aside from the issues above, https://github.com/opensearch-project/OpenSearch/pull/2293 is also aiming to make it to 2.2.0. This was identified via the project roadmap

There are no more open issues tagged with v2.2.0. #2293 has also been backported to 2.2 - https://github.com/opensearch-project/OpenSearch/pull/4141

Generating release notes as outlined here - https://github.com/opensearch-project/OpenSearch/issues/789#issuecomment-1179436917

2.x was bumped to 2.2.0 on Jun 24 - https://github.com/opensearch-project/OpenSearch/commit/718ea66c2c3f28ad1d432ec4943dea3ae651efec
Generated rough draft of release notes, with an in-line sed command to replace PR ids with the full Github URL:

git log --pretty=format:"* %s" --since=6-24-2022 | sed -E "s|(\(#([0-9]*)\))|([#\2](https://github.com/opensearch-project/opensearch/pull/\2))|g"

Note that I'm generating these notes off the 2.2 branch and not main. I'll eventually cherry-pick this change to main
Manual organize the release notes into the categories defined here.

@kartg any chance to get https://github.com/opensearch-project/OpenSearch/pull/4145 in?

@kartg any chance to get #4145 in? sorry for dragging the release ...

Hi @reta as shown in the schedule of this post, it is code freeze today so normally we would against merging any code to 2.2 anymore unless it is critical function breaking or security related.
However I was resolving a bug earlier as the build pipeline is not pass yet, so I will accept this change this time. 😄
Thanks.

@kartg @anasalkouz  I am closing this issue since we released 2.2.0 already but I would request you to keep this issue up to date during release and close it on time going forward. Make sure to add any retrospective learning on this issue as well.


Remove tags from generated artifacts

I.e. the changes in https://github.com/SuffolkLITLab/ALKiln/pull/561/files#diff-4b632aedc4c91285de358dd7cbcc91a52724745c5c759b81b3a6d80e8d6d7abcL187-R195.
Adding all of the tags to every output file makes the file names much too long, to the point where it's difficult to see more important information about an artifact. IMO tags are much less useful that the scenario id, which serves the same purpose.

Sounds good. I was on the fence about having them around anyway.


include non-distributable layers

This is an epic placeholder. The context for this work can be found in https://github.com/vmware-tanzu/carvel-imgpkg/issues/8

This epic was completed as a part of v0.6.0.


[2085] Create endpoint for triggering bulk sync

Context
We had this functionality in the C# version via WebJobs as Azure allowed us to manually trigger it. We would like to replicate this functionality by creating an endpoint that when sent a message will trigger the jobs.
Changes proposed in this pull request
Add the /api/v2/sync endpoint that when pinged will queue another sync
Guidance to review
Checklist

[X] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[ ] Rebased master
[X] Cleaned commit history
[X] Tested by running locally

 Create an endpoint for triggering bulk sync jobs

My understanding was that API v1 was legacy, rather than about the actor. If your concern is MCB we already have a way to generate v2 tokens:
https://github.com/DFE-Digital/manage-courses-backend/blob/master/lib/mcb.rb#L103

I read @defong's suggestion as not to add this to api v1, but to change the authentication mechanism to not be based on a specific user, but some kind of a shared secret, like in api v1. Good suggestion. 👍


datastore.js does a  "throw err" and just says "Object"

from 4eeb9b0 on LocalDB
I cant start gekko how can I do a better debug?

2014-01-15 20:11:00 (INFO):     I'm gonna make you rich, Bud Fox.
2014-01-15 20:11:00 (INFO):     Setting up Gekko in realtime mode
2014-01-15 20:11:00 (INFO):     Setting up:
2014-01-15 20:11:00 (INFO):              Trading Advisor
2014-01-15 20:11:00 (INFO):              Calculate trading advice
2014-01-15 20:11:00 (INFO):              Using the trading method:EMA
2014-01-15 20:11:00 (INFO):     Setting up:
2014-01-15 20:11:00 (INFO):              IRC bot
2014-01-15 20:11:00 (INFO):              IRC module lets you communicate with Gekko on IRC.
2014-01-15 20:11:00 (INFO):     Setting up:
2014-01-15 20:11:00 (INFO):              Mailer
2014-01-15 20:11:00 (INFO):              Mail module lets sends you email yourself everytime Gekko has new advice
2014-01-15 20:11:00 (DEBUG):    Setup email adviser.
2014-01-15 20:11:03 (INFO):     Send advice via email.
2014-01-15 20:11:03 (INFO):     Setting up:
2014-01-15 20:11:03 (INFO):              Profit Simulator
2014-01-15 20:11:03 (INFO):              Paper trader that logs fake profits.
2014-01-15 20:11:03 (INFO):     Starting to watch the market: BTC-e BTC/USD
2014-01-15 20:11:03 (DEBUG):    ~market start
2014-01-15 20:11:03 (DEBUG):    checking history
/home/ncesar/gekko/node_modules/nedb/lib/datastore.js:60
if (err) { throw err; }
^
[object Object]

fixed.


v1.8.4 hotfix release

This PR squashes the cherry picked commits on the release_1.8.4 branch (a temporary branch that will be deleted) in order to make a 1.8.4 hotfix release.

As this PR represents a new workflow for making hotfix release, I think it is worth summarizing the process:

Each hotfix release branches should be branched off the previous one (e.g 1.8.4 is branched off 1.8.3)
The bug fixes are cherry picked onto a temporary hotfix branch (e.g release_1.8.4)
Once we are happy with that, we can make a single diff between the head of that branch and the previous hotfix release tag to get a single diff file. In this case I used git diff v1.8.3.
That diff can be applied on a second branch, also off the previous hotfix. In this case this is the hotfix_1.8.4 branch used in this PR. The goal is to have a single commit with all the bug fixes with an annotated tag (here v1.8.4).
The release can be made off that commit, then this PR can be merged into master with an appropriate merge commit (NOTE: DO NOT SQUASH MERGE!) and the temporary branch can be deleted.

Tests are passing. Merging.


[jvm-packages] add support of json-formatted dump model

A follow-up of merging of https://github.com/dmlc/xgboost/pull/1726

Is there a common specification for the JSON output? Or if the JSON-formatted in the referred PR will be the de facto format?

Never mind, the JSON dump was added to the C++ native code. I thought it was only added to the Python bindings.

@CodingCat could you confirm that this is already supported?


fix: Tier module fixes and improvements (part 3)

Description
This PR aims to add a number of fixes and additional improvements on top of work done in https://github.com/sourcenetwork/sourcehub/pull/8
Tasks

[x] Improve CancelUnlocking logic to support partial lockups.
[x] Improve SubtractLockup to remove existing records if subtracting whole amount.
[x] Add GetLockups to fetch all lockups for a given delAddr.
[x] Add SubtractUnlockingLockup to handle partial unlocking lockups.
[x] Add SaveLockup to be used in  SubtractUnlockingLockup and InitGenesis.
[x] Emit corresponding events in Lock, Unlock, Redelegate, and CompleteUnlocking.
[x] Fix bugs in TestTotalAmountByAddr, GetLockup, and removeUnlockingLockup.
[x] Add more keeper and lockup tests, fix and improve existing tests.

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

Project coverage is 27.45%. Comparing base (9a8ec65) to head (c435066).

Files with missing lines
Patch %
Lines

x/tier/keeper/keeper.go
93.02%
6 Missing :warning:

x/tier/keeper/lockup.go
92.94%
4 Missing and 2 partials :warning:

x/tier/keeper/credit.go
50.00%
1 Missing :warning:

x/tier/keeper/msg_server.go
0.00%
1 Missing :warning:

Additional details and impacted files
@@                         Coverage Diff                          @@
##           iverc/tier-module-improvements-2      #10      +/-   ##
====================================================================
+ Coverage                             25.54%   27.45%   +1.91%
====================================================================
  Files                                   147      148       +1
  Lines                                  6088     6224     +136
====================================================================
+ Hits                                   1555     1709     +154
+ Misses                                 4417     4400      -17
+ Partials                                116      115       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.