Bug Text: display Menu?

Describe the bug
A clear and concise description of what the bug is.
Hello,
when I create a Text: Dispaly Menu I cannot exceed more than 6 letters
I left on the dialogue option.
To Reproduce
Steps to reproduce the behavior:
every time I create a Text Display Menu whatever the scene.

See error

Expected behavior
A clear and concise description of what you expected to happen.
I would like to be able to incorporate a variable after my text.
Screenshots
If applicable, add screenshots to help explain your problem.

Platform (please complete the following information):

OS: Win 10
App version Version 2.0.0 beta4

Additional context
Add any other context about the problem here.

I may be listing the obvious and I'm trying to find the Text: Display Menu in src. The Layout option when set to menu has a much smaller view.
However, Text: Display Multiple Choice allows more than 6.

I'll check some more later but this is in ScriptRunner_b.c; However looking at functionality in editor, when changing LAYOUT to MENU in Text: Display Menu, the smaller menu could be the reason behind the 6 char limit?
I'm inexperienced but I'll dig around more later and see if the limit from Text: Display Multiple Choice (which appears to be 17 char) could be applied there if layout = dialogue

TO CONFIRM: The original purpose is to make a menu? If not, Text: Display Dialogue or Text: Display Multiple Choice may be better.
After looking through this a little more, I would agree that this may not be a bug because Text: Display Menu has the two layout options: Dialogue and Menu.
When in Dialogue layout mode AND the inputs proceed past four (4) Options, a second column is made in that window (too many characters would "break" the menu).
If in Menu layout mode, the width of the menu and on the left of the screen, is shorter, thus less characters are required to prevent "breaking" the menu (as in too many characters would overflow from the menu).
Recommendation: Use Text: Display Menu but with shorter words/descriptions (see Pokemon for an example?)
Recommendation: I advise resolving this as a bug or marking as feedback for a new Display Menu function?


always show memory ftrace_events of linux.ftrace data_source

I used perffeto to trace my android app by config:
data_sources: {
config {
name: "linux.ftrace"
ftrace_config {
ftrace_events: "gpu_mem/gpu_mem_total"
ftrace_events: "mm_event/mm_event_record"
ftrace_events: "kmem/rss_stat"
ftrace_events: "kmem/ion_heap_grow"
ftrace_events: "kmem/ion_heap_shrink"
ftrace_events: "mm_event/mm_event_record"
ftrace_events: "dmabuf_heap/dma_heap_stat"
ftrace_events: "lowmemorykiller/lowmemory_kill"
ftrace_events: "oom/oom_score_adj_update"
}
}
}
after trace many times, I delete ftrace_config,but delete invalid still show mem related ftrace_events in my trace

mac pro trace in chrome

What does your config look like exactly? Do you also have process_stats enabled with proc_stats_poll_ms? If so, that'll be where those counters are coming from.


API enhancement

pyrtcm Pull Request Template
Description
It helps the programmer to use default arguments explicitly rather than kwarg.

completion is available in the editor
able to notice changes in arguments such as 34b8c6d ;-)

Testing
This changes does not include functional changes.
Coverage is now 98.2%, but this figure has not changed since the last commitment.
https://app.codecov.io/github/semuconsulting/pyrtcm/commit/d4b5a8afdf2061cf173060c70f4c6dc5a4af37fc
Checklist:

[x] I agree to abide by the code of conduct (see CODE_OF_CONDUCT.md).
[x] My code follows the style guidelines of this project (see CONTRIBUTING.md).
[x] I have performed a self-review of my own code.
[x] (if appropriate) I have cited my RTCM documentation source(s).
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] (if appropriate) I have added test cases to the tests/test_*.py unittest suite to maintain >= 99% code coverage.
[x] I have tested my code against the full tests/test_*.py unittest suite.
[x] My changes generate no new warnings.
[x] Any dependent changes have been merged and published in downstream modules.
[x] I have signed my commits.
[x] I understand and acknowledge that the code will be published under a BSD 3-Clause license.

Hi @zakkie
Thanks for this. I think you're right in saying this make the API clearer to the developer, but losing the **kwargs argument altogether causes downstream errors e.g. in PyGPSClient, which invokes pyrtcm in its NTRIP client (see traceback below). The error is in fact spurious in this case (an earlier oversight on my part :-( ) - the method in question doesn't actually use the quitonerror argument - but it's something that needs to be taken into account.
I'll need to check if there are likely to be any other spurious downstream errors before merging (and maybe do some updating in other modules!!!), but appreciate the contribution.
Out of interest, what brought you here?
➜  pyrtcm git:(API_Enhancement) pygpsclient
Exception in thread Thread-4 (_read_thread):
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/threading.py", line 1038, in _bootstrap_inner
    self.run()
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/threading.py", line 975, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/steve/Library/Python/3.11/lib/python/site-packages/pygnssutils/gnssntripclient.py", line 449, in _read_thread
    self._do_data(self._socket, stopevent, ggainterval, output)
  File "/Users/steve/Library/Python/3.11/lib/python/site-packages/pygnssutils/gnssntripclient.py", line 537, in _do_data
    raw_data, parsed_data = ubr.read()
                            ^^^^^^^^^^
  File "/Users/steve/Library/Python/3.11/lib/python/site-packages/pyubx2/ubxreader.py", line 161, in read
    (raw_data, parsed_data) = self._parse_rtcm3(bytehdr)
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/steve/Library/Python/3.11/lib/python/site-packages/pyubx2/ubxreader.py", line 276, in _parse_rtcm3
    parsed_data = RTCMReader.parse(
                  ^^^^^^^^^^^^^^^^^
TypeError: RTCMReader.parse() got an unexpected keyword argument 'quitonerror'

Hi @zakkie
Many thanks for this contribution - it'll be merged in v1.0.11

Hi @semuadmin
Thanks for your quick review!

losing the **kwargs argument altogether causes downstream errors
Sure. I tried to add deprecated warnings. Do you think about it?
(if you don't need it, please revert it after marge).

@semuadmin

Out of interest, what brought you here?

I'm developing a Ntrip client for use with narrow bandwidth.
This pyrtcm is very helpful for filtering RTCM messages. Thanks a lot.

Hi @semuadmin
Thanks for your quick review!

losing the **kwargs argument altogether causes downstream error

Sure. I tried to add deprecated warnings. What do you think about it? (if you don't need it, please revert it after marge).

I don't think this is needed - any chance you can simply back out 6442131 at your end? I've already merged the previous changes into RC 1.0.11.

@semuadmin

Out of interest, what brought you here?

I'm developing a Ntrip client for use with narrow bandwidth. This pyrtcm is very helpful for filtering RTCM messages. Thanks a lot.

Sounds interesting - FYI there is already an NTRIP client in my pygnssutils library.

I don't think this is needed

ok, done!

Sounds interesting - FYI there is already an NTRIP client in my pygnssutils library.

Thanks! But we develop apps with asyncio, and it is incompatible with asyncio.

I don't think this is needed

ok, done!

Sounds interesting - FYI there is already an NTRIP client in my pygnssutils library.

Thanks! But we develop apps with asyncio, and it is incompatible with asyncio.

Fair enough. When I find the time, my intention is to rewrite pygnssutils to use the the asynchio streams API rather than raw sockets. Main challenge is getting it to play nice with tkinter, as these libraries are used within the PyGPSClient app.

merged via https://github.com/semuconsulting/pyrtcm/commit/fcdfd34619c53d1b2cb56ab64c043bf67e5b035e


fix: fix typo on page-limit docs

Fixed a typo on page limit docs, we should add next_jsonapi.size_max to parameters in sites/default/services.yml.

@alex-ahumada Thanks for the PR. I merge #439 with the fix.


sum-of-multiples: add clarifying comment

Clarify the discrepancy between the Go implementation and the current README.
This fixes #256.

Yeah looks good, so I'm :+1: on it. Will probably leave you to hit the merge button since I'll occupied for a bit.

I appreciate having a second pair of eyes on this. Thanks!


Create the landing page using React and CSS

[ ] Set up the necessary JSX and CSS files in the project, ensuring they are properly linked and organized.

[ ] Customize the UI elements, layout, and styling according to your application's design requirements.

#11


Update and improve discovery service

This service hasn't seen much attention in recent years, and it's time to fix that.
After talking with @hexfusion about https://github.com/coreos/etcd/issues/9978, we decided on an approach that we think should let us update the service to use the v3 backend.

[x] Add tests to ensure that improvements/refactoring don't break anything
[x] Map out current API calls and endpoints
[x] Add a true handler for each endpoint, rather than the current proxy
[x] Convert the current logic to a v2 "backend"
[x] Add a v3 backend, and allow for reading from the v2 backend first
[ ] Add a TTL to clusters to prevent data from growing unbounded over time

At that point, we will probably call the discovery service "version 3", and begin strongly encouraging people to run their own discovery services, and not rely directly on the publicly hosted instance.

I want to add the TTL to https://github.com/etcd-io/etcd/pull/10627 before deploying it. I think I am just going to add a secondary index with timestamps for a deletion date.


protobuf: refactor proto visitor pattern.

Move the proto traversal handling in version_converter.cc to a
standalone library. This lets us replace existing proto visitor patterns in
common/protobuf/utility.cc for unexpected field checks.
The redaction code is actually a bit more involved, so I'm not
refactoring this; it needs to recurse through Any/TypedStruct.
Ultimately we might want something like this, but it doesn't seem super
helpful given we only have a the single instance of this right now.
Risk level: Low
Testing: Existing tests continue to pass.
Signed-off-by: Harvey Tuch<EMAIL_ADDRESS>

@alyssawilk this continues https://github.com/envoyproxy/envoy/pull/9807; I didn't seem to be able to reopen that PR.


Update pre-request.js

Modified code to inject in authorization header without need of global variable

@dmedeiros-veracode Veracode Example.postman_collection.json should also be updated because that's the thing that will be imported into postman. The .js file is just there for reference I think.

@dmedeiros-veracode and @relaxnow -- I think these changes look good as far as the code goes. You've added the relevant API creds fields into the environment file and added error handling, as far as I can tell. Can one of you please also update the README.md with any appropriate changes for setup?

@tjarrettveracode there seems this pull request is waiting on your review to move forward

@dmedeiros-veracode -- did you address the comments regarding the need to update the README file? I don't see that in the list of changes on this merge request.

Providing a different pull request with Updated Readme and confirmed fixes


[AppConfig] Adding new parameter 'PublicNetworkAccess' for ConfigurationStoreUpdateParameters

This PR is for updating management plane SDK for AppConfig after we made some changes to the REST API specs here.

/azp run net - appconfiguration - tests

/azp run net - mgmt - ci

/azp run net - appconfiguration - ci

Please see failure details and make correction. We will revisit this PR once checks are passing.

Hi @allenjzhang, the PR is ready for review.


Implement drive webhook notification events

Reference: https://docs.microsoft.com/en-us/onedrive/developer/rest-api/concepts/using-webhooks?view=odsp-graph-online

Done. Provided by the subscriptions request client.


Revert "Alert on pipeline failure."

Reverts BlueConduit/open-data-platform#143
The code is failing to build.

@justinebreuch Yeah, I'll look for a fix and send a new PR.


fix: add Pericles001

resolves #1211
Checklist for profile addition

[x] Name the PR fix: add <your-username>.
[x] You've added file named your-username.json in the profiles directory.
[x] You've linked the PR to the correct issue.

Other additions/changes

[x] Changes made in this PR are mentioned below
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.
[x] I've read the contributing guidelines

This PR adds/fixes

Notes

Review suggestions has been applied.

@schmelto, your suggestion has been applied.


fix: fixed the problem of deprecated widgets. Used TextButton instead…

Replaced the FlatButton with TextButton

@kunal-gg  Please make a PR to the develop branch. Once we have enough changes and testing done we'll merge with main. Till then, I am closing this PR.


Building flutter tool is broken after switching lanes

How it started
After switching lanes from stable to beta, it started crash my flutter tool when it was being build.
(I was at latest stable version and switched lanes)
I have reinstalled it from scratch, but that doesn't fix anything.
I am unable to use flutter in the meanwhile.
Steps to Reproduce

... Be on Linux (My distro is: Ubuntu 20.04 LTS)
... Download flutter from git - stable version (git clone and export PATH)
... Update to latest version of stable
... Switch to beta and try to run upgrade

Logs
 flutter precache                      ✔  38s    09:41:07   
Downloading Dart SDK from Flutter engine 20caf549695193a9920ca37752eca4f141bbef31...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  176M  100  176M    0     0  12.2M      0  0:00:14  0:00:14 --:--:-- 13.9M
Building flutter tool...
kernel-service: Error: Unhandled exception:
Invalid argument(s): Exception when reading 'file://home/venturus/flutter/packages/flutter_tools/.dart_tool/package_config.json': Unsupported operation: Cannot extract a non-Windows file path from a file URI with an authority
#0      ProcessedOptions._readFile (package:front_end/src/base/processed_options.dart:569:7)
<asynchronous suspension>
#1      ProcessedOptions._createPackagesFromFile (package:front_end/src/base/processed_options.dart:583:26)
<asynchronous suspension>
#2      ProcessedOptions.createPackagesFromFile (package:front_end/src/base/processed_options.dart:648:11)
<asynchronous suspension>
#3      ProcessedOptions._getPackages (package:front_end/src/base/processed_options.dart:517:26)
<asynchronous suspension>
#4      ProcessedOptions.getUriTranslator (package:front_end/src/base/processed_options.dart:471:32)
<asynchronous suspension>
#5      languageVersionForUri.<anonymous closure> (package:front_end/src/api_prototype/language_version.dart:58:35)
<asynchronous suspension>
#6      languageVersionForUri (package:front_end/src/api_prototype/language_version.dart:38:10)
<asynchronous suspension>
#7      uriUsesLegacyLanguageVersion (package:front_end/src/api_prototype/language_version.dart:147:7)
<asynchronous suspension>
#8      autoDetectNullSafetyMode (package:vm/kernel_front_end.dart:557:18)
<asynchronous suspension>
#9      _processLoadRequest (file:///b/s/w/ir/cache/builder/src/third_party/dart/pkg/vm/bin/kernel_service.dart:801:5)
<asynchronous suspension>

Unhandled exception:
Invalid argument(s): Exception when reading 'file://home/venturus/flutter/packages/flutter_tools/.dart_tool/package_config.json': Unsupported operation: Cannot extract a non-Windows file path from a file URI with an authority
#0      ProcessedOptions._readFile (package:front_end/src/base/processed_options.dart:569:7)
<asynchronous suspension>
#1      ProcessedOptions._createPackagesFromFile (package:front_end/src/base/processed_options.dart:583:26)
<asynchronous suspension>
#2      ProcessedOptions.createPackagesFromFile (package:front_end/src/base/processed_options.dart:648:11)
<asynchronous suspension>
#3      ProcessedOptions._getPackages (package:front_end/src/base/processed_options.dart:517:26)
<asynchronous suspension>
#4      ProcessedOptions.getUriTranslator (package:front_end/src/base/processed_options.dart:471:32)
<asynchronous suspension>
#5      languageVersionForUri.<anonymous closure> (package:front_end/src/api_prototype/language_version.dart:58:35)
<asynchronous suspension>
#6      languageVersionForUri (package:front_end/src/api_prototype/language_version.dart:38:10)
<asynchronous suspension>
#7      uriUsesLegacyLanguageVersion (package:front_end/src/api_prototype/language_version.dart:147:7)
<asynchronous suspension>
#8      autoDetectNullSafetyMode (package:vm/kernel_front_end.dart:557:18)
<asynchronous suspension>
#9      _processLoadRequest (file:///b/s/w/ir/cache/builder/src/third_party/dart/pkg/vm/bin/kernel_service.dart:801:5)
<asynchronous suspension>

/cc @jensjoha

Could you provide us with the content of file://home/venturus/flutter/packages/flutter_tools/.dart_tool/package_config.json?

This is likely related to https://github.com/flutter/flutter/issues/63579

From that issue --- an interesting point would be what which flutter dart returns.

Yup --- I think that's the reproduction:
$ which flutter
# (empty output --- there's no flutter)

$ git clone https://github.com/flutter/flutter.git
# (...)

$ export PATH="//usr/local/google/home/jensj/code/nobackup/flutter/issue71274/flutter/bin:$PATH"
# (notice that the path is basically correct except that it starts with two slashes instead of one!)

$ which flutter
//usr/local/google/home/jensj/code/nobackup/flutter/issue71274/flutter/bin/flutter

$ flutter channel beta
Building flutter tool...
kernel-service: Error: Unhandled exception:
Invalid argument(s): Exception when reading 'file://usr/local/google/home/jensj/code/nobackup/flutter/issue71274/flutter/packages/flutter_tools/.dart_tool/package_config.json': Unsupported operation: Cannot extract a non-Windows file path from a file URI with an authority
#0      ProcessedOptions._readFile (package:front_end/src/base/processed_options.dart:569:7)
<asynchronous suspension>
#1      ProcessedOptions._createPackagesFromFile (package:front_end/src/base/processed_options.dart:583:26)
<asynchronous suspension>
[...]

Which means that the workaround here is probably to fix the path. Change the path to flutter to only include one slash and it seems to build just fine.

Could you provide us with the content of file://home/venturus/flutter/packages/flutter_tools/.dart_tool/package_config.json?

Sure, Here it is:
https://paste.ofcode.org/3bwB62dEfaVtS54HYstfMzS

which flutter dart

Result:
//home/venturus/flutter/bin/flutter
//home/venturus/flutter/bin/dart

Solved this issue reseting my $PATH
Step By Step Fix:
1) Check your Path

echo $PATH                                  # Check the double slash in the flutter import

2) Clear your Path Variable

export PATH=""                             # Clear PATH variable

3) Restore environments config

source /etc/environment              # Restore Environments PATH

3) Restore your bash config

source ~/.zshrc                            # use  your bash here: here are some .bashrc or .zshrc

Then just run flutter again.
For some strange reason it was added when I switch to the Beta lane. Really odd.

After my reseting my $PATH:
which flutter dart
/home/venturus/flutter/bin/flutter
/home/venturus/flutter/bin/dart

@lcsvcn
Is this safe to close or still need more investigation ?

We can close this issue


Feature request: Get ID of current session

I often have multiple sessions running in separate code files; is there a way to figure out the identifier for the current session in order to connect with that session in a new file?
I.e. It would be convenient if there were a way to figure out from a file that's connected to one of these two sessions what the ID is, so that I can connect to the right one from a new document.

Looks like I'm not the only one running into this. This has actually been bothering me a lot ever since I upgraded Hydrogen around September.
I want to point out that from my perspective (as the person who implemented the first version of the remote kernel connections), the screenshot above is a bug. The session list was designed to always use filenames instead of IDs.
If you click on the "Python 3 | idle" text in the status bar, there's an option called "Rename session". The name given there should be the one displayed in the kernel selector -- with the "rename" command providing additional management flexibility. Also if you open up the notebook in a web browser, it will also always list a meaningful name and not "Session ..."
Unfortunately, there's been a lot of rapid change in 3 distinct pieces of software: Jupyter, the Jupyter services library, and Hydrogen. It now seems like the screenshot above happens for the majority of version combinations, even though originally it was originally designed as fallback code that would only trigger in less than 0.1% of situations.
I'm not really sure why this is the case. I probably won't be able to investigate in detail in the near future, since the precise behavior depends on version combinations. I swear there is at least one version of Jupyter notebook out there that will correctly display filepaths.

It seems like "rename session" doesn't transfer information back to the Jupyter kernel.
When I click on rename session, it does have filled in
filepath-6d45bdb3-d114-49f8-ba04-082d7990fe0b

, but that UUID isn't the same as any of the UUID's listed when I try to connect to a remote session. It seems like the names aren't the same. And when I do rename the session, it doesn't persist for when I try to connect from a new file.
Thanks still, for the info.

Yes, currently session naming is quite broken.
After taking a quick look, it looks like the fix might actually be quite simple (see #1247).

Closed by #1247

@kylebarron I know that this bugfix isn't precisely the feature you were requesting, so please let me know if there's some aspect of your workflow that hasn't been addressed with the latest release.

That works great! Thanks!


feat(cache): add support for IO caching in RequestContext

Summary:
Eliminates duplicate IO calls made in the same request by saving the results in RequestContext
Issue Reference(s):
Fixes #1445
Build & Testing:

[x] I ran cargo test successfully.
[x] I have run ./lint.sh --mode=fix to fix all linting issues raised by ./lint.sh --mode=check.

Checklist:

[ ] I have added relevant unit & integration tests.
[ ] I have updated the documentation accordingly.
[ ] I have performed a self-review of my code.
[x] PR follows the naming convention of <type>(<optional scope>): <title>

Summary by CodeRabbit

New Features

Introduced a new caching mechanism across the application to improve performance.
Added an AsyncCache structure for asynchronous value retrieval and caching.

Enhancements

Expanded RequestContext to include caching capabilities using a combination of HashMap and broadcast channels.
Enhanced IO operations to support caching, improving efficiency in data handling.

Refactor

Reorganized module imports for better code structure and readability.

Documentation

Updated public exports to include CacheValue, making it accessible for further development and integration.

closing in favour of https://github.com/tailcallhq/tailcall/pull/1468


scss-lint:disable is leaking into other files.

I started to integrate scss-lint into one of my pre-existing projects and marked several of my files with //scss-lint:disable all so I could tackle them later.
I run it with a rather large list of files, and I noticed that some files are not being flagged for lint errors even when they should be. I realized when I removed scss-lint:disable from a different file, it then flagged the errors in the other file correctly.
I have tried adding //scss-lint:enable all to that file but it still does nothing. The strange part is that when I remove all of the styles from the file that contains the disable comment (but leave the disable comment in), it then correctly flags the other file for errors.
I thought this might indicate a race condition (some shared flag is set but gets applied to a different file) but I don't see any multi-threading going on in the engine.
Any ideas or workarounds? If you have any suggestions about why this might be happening i can try to fix it.

See #429.

Thank you! Missed that issue!


feat: add xa

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

fix  Conflict

Codecov Report

Merging #467 (3a9e95f) into master (46da59f) will decrease coverage by 1.56%.
The diff coverage is 29.71%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #467      +/-   ##
==========================================
- Coverage   38.38%   36.83%   -1.56%
==========================================
  Files         155      159       +4
  Lines       10319    11020     +701
==========================================
+ Hits         3961     4059      +98
- Misses       6032     6615     +583
- Partials      326      346      +20

Impacted Files
Coverage Δ

pkg/client/client.go
0.00% <0.00%> (ø)

pkg/datasource/sql/at_resource_manager.go
0.00% <ø> (ø)

pkg/datasource/sql/conn_at.go
49.20% <ø> (ø)

pkg/datasource/sql/exec/executor.go
0.00% <ø> (ø)

.../datasource/sql/exec/select_for_update_executor.go
0.00% <ø> (ø)

pkg/datasource/sql/plugin.go
100.00% <ø> (ø)

pkg/datasource/sql/tx_at.go
23.52% <ø> (ø)

pkg/datasource/sql/tx_xa.go
45.45% <0.00%> (+12.12%)
:arrow_up:

pkg/datasource/sql/types/types.go
12.82% <ø> (ø)

pkg/datasource/sql/xa/oracle_xa_connection.go
0.00% <ø> (ø)

... and 14 more

... and 3 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[PROJECT PROPOSAL] Brain Tumor Classification using machine learning

Project Request
Creating multiple machine learning models to classify brain tumors given images of brain scans.

Field
Description

About
Models that help predict the brain tumor a person has

Github
joeadham

Email
<EMAIL_ADDRESS>

Label
GSSOC'23

https://github.com/joeadham

Define You

[x] GSSOC Participant
[x] Contributor

Project Name
Brain Tumor Classification
Description

Project: Brain tumour classification using machine learning for accurate diagnosis and early detection.
Goals: Develop a robust model for precise classification, reducing diagnostic burden and enabling timely treatments.
Outcomes: Improved accuracy in tumour classification, streamlined diagnostic workflow, and potential integration into clinical practice.

Scope

Objectives: Develop a machine learning model for brain tumour classification, achieve accurate and early detection, reduce diagnostic burden, and improve decision-making for medical professionals.

Deliverables: A robust machine learning model capable of accurately classifying brain tumours, streamlined diagnostic workflow, and potential integration into existing clinical systems.

Constraints: Limited availability of annotated medical imaging data, computational resources, and adherence to ethical guidelines and privacy regulations in handling patient data.

Timeline
I have already worked on this project in my personal time, I can deliver it as soon as the issue is assigned to me
Video Links or Support Links
Dataset used: https://www.kaggle.com/datasets/masoudnickparvar/brain-tumor-mri-dataset

Hi @miraj0507 @adithya-s-k! I have raised this issue. Kindly assign this to me.
Thank you.


fix: issue #5752 Field should be unique but is not

causing unnecessary ALTER TABLE executions

[X] Do only one thing
[X] Non breaking API changes
[X] Tested

What did this pull request do?
On cockroachdb, auto migrating a table with a field like:
PublicKey string      `gorm:"uniqueIndex" json:"public_key"`

Will create the table and then create the unique index, and then do an alter to create the unique index again which causes an error.  This change fixes that.
User Case Description
auto migrating a table with a field with a uniqueIndex tag.

Thank you for your PR, Going to close this PR for now as tests failed.


kvdb-rocksdb overlay cache is not effective

Seems that cache in kvdb itself (memorizing raw key-value pairs on write), despite showing effectiveness in synthetic benchmarks, offers no performance advantage for sync speed, sometimes even negative (anybody can try run polkadot it with parity-common replaced from this branch https://github.com/paritytech/parity-common/tree/no-overlay). Not counting increased memory footprint.
This is probably due to:

There are a higher-level cache-s with good hit ratio.
RocksDb has its own raw key-value cache, and quite configurable.

Given that, and also the fact that this cache prevents other possible optimisations (such as zero(little)-copy database transactions), we might want to remove it.
There are probably considerations other than sync speed, which would be great to hear about.

In fact, it is not used at all probably
So I might draft a version of kvdb with no overlay cache + significantly reduced copy of transaction data (now any key and any value must be put into dedicated Vec<u8>, and this totally can be avoided)

This resolved in https://github.com/paritytech/parity-common/pull/313 I believe.


"C API for WASI": Segmentation fault when calling "wasi_config_delete"

When calling wasi_config_delete() on a configuration that was created with wasi_config_new(), I get a Segmentation fault.
Test Case
I used the official WASI example of wasmtime as a template:
main.c
I added wasi_config_delete(wasi_config); in the line before return 0 of the main function. I did this because the documentation of wasi_config_new() reads:

The caller is expected to deallocate the returned configuration

Steps to Reproduce
Running the program with this one additional line containing wasi_config_delete causes the process to crash with a segmentation fault.
I discovered that wasi_config_delete can be called without problems up to the point when wasi_instance_new is called. It seems that wasi_instance_new takes ownership of parts of the data that is stored in wasi_config_t.
Expected Results
wasi_config_delete should delete the wasi_config_t structure without any negative side effects.
Actual Results
Segfault!
Versions and Environment
Wasmtime version or commit: Fresh pull of wasmtime on April 19th (200d7f1df668db31baf14f1453ab8cee54724292)
Operating system: Ubuntu 20.04
Architecture: x86_64 GNU/Linux

Are you using wasm_engine_new_with_config? If so that function is documented as taking ownership of the wasm_config_t so you don't need to free it.

Are you using wasm_engine_new_with_config? If so that function is documented as taking ownership of the wasm_config_t so you don't need to free it.

@alexcrichton Thanks for pointing this out. Actually I'm using wasi_instance_new , and this function has the same semantics of taking ownership of the config object. I should have read the f***ing manual in more detail ;-)
I'll close this issue because this information resolves my problem. Thanks for looking into this.


NIFI-7588: Handling dynamic properties on termination

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Sets the new properties of a processor from the existing processor upon termination, allowing onPropertyModified() to fire.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

@markap14 This looks good, can you take look to confirm the approach?

Thanks @gresockj changes look good to me. +1 will merge.


DRA1-UCA-EP02-US06 eContent Maintenance and Ownership over Life Cycle

As a MAH
I want to always have clear ownership of the structured eContent, and be able to transfer it to another MAH in case of divestment
So That the Patient always gets the ‘right’ and ‘latest’eContent from the me, because I am liable and accountable for it unless I divest it.  If I divest a product to a different MAH, I want to transfer the structured eContent to them, because the new MAH will be  responsible to give the ‘right’ and ‘latest’ eContent.  The onus is on both parties to provide this In the case of acquisition of a product from another MAH, I am responsible to give the ‘right’ and ‘latest’ eContent.
User Story Rationale/ Examples:
Rationale:   The MAH is responsible and liable for a product that it places on the market, including  the provision of correct contents for product inserts.
Consider the digital eLeaflet as an integral part of the product… a digital shadow of the physical product where the 2 are tied together.
**
It follows that a transfer of marketing authorization of a product infers a transfer in ownership of the digital shadow and the eContent Asset.
**
Parking Lot item:  Internal Pharma co Quality Release Processes will need to change from verifying the paper insert is correct, to verifying that the eContent is correct prior to releasing product to the market so a digital leaflet is always available.
Acceptance Tests
eContent can be transferred from one MAH to another and retrieved by a Patient using the new and old GTIN.
All batch-specific eContent is transferred and provides the correct eLeaflet for batches prior to transfer.
1- Scan GTIN-A/Batch-1 and get Company A resolved (see attachment scenario 1.a.)
-> Company A shown on landing page and content for that GTIN/Batch, maintained by Company A shown
2-Transfer GTIN-A to Company B
3-After GTIN-A was transferred to Company B, update leaflet
4-Create new Batch-2 for GTIN-A
5-Scan GTIN-A/Batch-1 and still get Company A resolved (see attachment scenario 2.a.)
6-Scan GTIN-A/Batch-2 and get Company B resolved (see attachment scenario 5.b.)
7-After GTIN-A was transferred to Company B, Company-A tries to update leaflet, but this is not possible
Open for MVP:

shall we add scenarios where all existing batches will be transferred to company B as part of divestment?
shall we include scenarios where the leaflet shows new owner, but batch, SN etc is still checked by original owner? (see attachment scenario 3.a.)

More clarification will be useful:
Ken/Patrick? to create clear rules of divestment scenarios and what is expected from the platform.
Ken/Patrick? to add clear acceptance criteria for each scenario.

More clarification will be useful:
Ken/Patrick? to create clear rules of divestment scenarios and what is expected from the platform.
Ken/Patrick? to add clear acceptance criteria for each scenario.

The divestment scenarios have been drafted. To be shared with the Tech team.
@nhrishi please upload and share information.

Please see attached divestment scenarios drafted for EPI.
DivestmentScenarios_v0.5.xlsx

Next Actions decided -

Schedule a workshop to finalize all the scenarios [Patrick] -
Identity obvious scenarios and create user stories [Sonja] -

@maherpa2  @steinso2


python311Packages.pytest-examples: don't propagate ruff

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.05 Release Notes (or backporting 23.05 and 23.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

[x] Fits CONTRIBUTING.md.

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

Isn't ruff propagated via --replace "'ruff'" "'${ruff}/bin/ruff'" anyway?

Isn't ruff propagated via --replace "'ruff'" "'${ruff}/bin/ruff'" anyway?

That's not the same though, is it?

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

2 packages marked as broken and skipped:

python312Packages.pytest-examples
python312Packages.pytest-examples.dist

2 packages built:

python311Packages.pytest-examples
python311Packages.pytest-examples.dist


How do you get policy details from a quarantined message?

We should include details about how to expose the policy details for a particular message. This will help admins troubleshoot which policy applied to a message in quarantine. This is available by modifying the columns in quarantine and choosing to add Policy Name (you have to remove a column to add one as the maximum is seven). An admin can also use the Get-QuarantineMessage PowerShell cmdlet.

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

ID: 2cfc15d1-833b-3139-6bc0-1acb97d507c8
Version Independent ID: 31e24618-c2e6-4fe4-9651-04d6007548a4
Content: Find and release quarantined messages as a user in Office 365 - Office 365
Content Source: microsoft-365/security/office-365-security/find-and-release-quarantined-messages-as-a-user.md
Service: o365-seccomp
GitHub Login: @chrisda
Microsoft Alias: chrisda

OK. Note added.
Thanks for the feedback. @kellybos, in the future, you, too can add information like this directly to the topic; no need to file a GitHub issue first. For instructions (works for any repo), see https://github.com/MicrosoftDocs/office-docs-powershell/blob/master/README.md.
Closing


Prevent start script code execution in env vars using bash

Fixes https://github.com/gradle/gradle-private/issues/3386
Start scripts generated by the application plugin accept two environment variables from the user - JAVA_OPTS and <APP_NAME>_OPTS whose values get passed to the java process that runs the application jar.
Those variables can have argument lists, delimited by spaces:
APP_OPTS="-Dfoo=bar -Dfizz=buzz"

and the arguments themselves might contain spaces that are either quoted or escaped:
APP_OPTS="-DtestValue=value -DtestValue2='some value'"

To support the latter case, the generated start scripts use eval set -- $APP_OPTS that correctly tokenizes the quoted arguments and in the above case sets the two separate positional arguments with values -DtestValue=value, -DtestValue2=some value respectively.
The problem with the above is that eval will also interpret and execute any code passed in that environment variable.
This change attempts to patch the possibility to inject and execute code this way by tokenizing the supplied environment variables into bash arrays without evaluating their content.
This, however, is a breaking change that is not POSIX compliant and requires bash shell to be available.
It affects both the application start scripts and the Gradle wrapper script.

This would also need to be verified with cygwin (which should be using bash) as well as MinGW and any other unix-like terminals under Windows.

Tried this out on Windows with cygwin, MSYS2 and git-bash - looked ok.

@bot-gradle test and merge

OK, I've already triggered a build for you.

@bot-gradle cancel

@bot-gradle test and merge

OK, I've already cancelled the pre-tested commit build.

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Fix two unavailable menu dialogs from being open

Closes
Found in testing. This prevents interaction with the trigger that opened the sub dialog in the first place. This also has the benefit of keeping the menu open when clicking the item which was previously noted as an oddity that the dialog would close and then reopen.
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

Still able to reproduce the error of two open context menus.

how? I can't anymore. steps I'm using are

Open menu
mouse over an unavailable item
click that item
arrow key down

I'm not seeing anything happen when I click the item and it's not giving focus to the trigger item, so arrow keys do nothing as well.

I can still reproduce as well:

Open the first submenu via keyboard
Mouse hover over that same menu item
Arrow down and open the other one via keyboard


API responding with an error wrongly.

Hi,
What could be wrong with this HTTP request, I am trying to register a user, but as per the error message I am missing something.
curl 'https://conduit.productionready.io/api/users' -H 'Accept: application/json, text/plain, */*' -H 'Referer: http://localhost:8080/register' -H 'Origin: http://localhost:8080' -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.100 Safari/537.36' -H 'Content-Type: application/json;charset=UTF-8' --data-binary '{"email":"muhammadsackib@gmail.com","password":"mary","username":"saqib"}' --compressed

My bad, I am posting with wrong body.


Be Quiet by Default

Typical Unix programs should be quiet by default and print only if there are failures. GNU Make is actually very, very, very bad at this because it is super verbose. It would be great if just would follow the Unix tradition here and only starts being talkative if -v or --verbose is passed to it.

Thanks for opening this issue!
I personally like it the way it is, since it allows following along with what commands are being run. For something like just or make, I think this is reasonable, since many commands are being run, and when something fails we want to know what command failed.
If most users would prefer it to be quiet, I would be open to changing it.

You are basically already stating what is of interest: what failed? Having overly verbose output by default means that the user must scan the output manually. An absurdity we see with many modern tools (I’m looking at you npm). Our QA staff is often misinterpreting build output because the programs print a ton of warnings, and the actual error is overlooked.
# Justfile
build:
    cargo +nightly build --release

$ just build
error: recipe 'build' failed `cargo +nightly build --release` at /usr/opt/proj/Justfile:3
# stderr output

I wrote stderr in the previous comment but actually, all output should be buffered and printed in case of error. I thought a little longer, and the perfect way for this would be imho as follows:

Default: no option was passed to just, nothing gets printed and output of the called programs is buffered. In case of error, the output buffer gets flushed and a banner is displayed.
-v or --verbose: nothing gets printed and output of the called programs is buffered. In case of success, a banner is displayed with the name of the target(s) that succeeded. Error case is handled as it was previously. This mode would be the default if a CI environment variable is defined.
-vv: executed target name(s) get printed and the output of the programs is piped through. Plus success and error banner. This mode would be the default if a VERBOSE environment variable is defined.
-vvv: like -vv but incl. just informational stuff. This mode would be the default if a DEBUG environment variable is defined.
--debug: like -vvv but incl. just debug stuff.

This allows users fine-grained control over the output that is being produced and follows plus enforces the Unix rule of silence.

Command stderr and stdout should still be printed by default. Since, for example, a compiler might produce warnings but still signal success, and just shouldn't hide those warnings from the programmer.

You’re probably right here and a --silent mode to actually enable this buffering is more sensible.

I think I'm going to close this. I'm actually highly sympathetic to the argument that programs should be quiet by default, but I personally find just's current behavior to be useful. I would definitely be open to some kind of argument that enabled buffering, and only showed buffered output if there was an error, if someone wanted to contribute a patch.


IdGen function should work per call.

Refactor IdGen to assign IDs on a per-call basis for streamed RPC responses, ensuring that all responses in a stream share the same ID as the originating request.
Description
In the existing setup, IdGen assigns unique IDs to each individual RPC request. This becomes problematic when streaming multiple responses, as each response should have an ID identical to that of the original request, according to the JSON RPC Spec.
Tasks

[ ] 1. Refactor IdGen so that it applies ID on a per-call basis.

Draw up the unary, client streaming, server streaming, duplex streaming examples as I had on the whiteboard.

I had a thought that ID gen could also generate subids for each serial message in a call. But that would be something for the future.

If you want to avoid using fix in the commit message I suggest formatting it like this.
fix: fixes problem with id generation

* Related: #11

[ci skip]


Provide option to archive JSON-LD repr for speed

Provide option to archive the JSON-LD representation as a str, rather than generating the JSON each time. This improves speed substantially (4X improvement) at cost of memory (storing the string).

Fixed in #80


Add watch mode for the codegen

Closes: #326
Also bring back out flag for codegen

I agree it's a greedy operation right now, with proper backend support we could have something better.
In any case even if we don't have schema files locally anymore, I see it useful if you're interacting with the schema on the webpage and you receive changes in the moment to your code.
Some kind of watch mode will be needed in the CLI, and the VSCode extension will definitely have it too.
We can raise it up maybe in the next frontenders sync.


Parallel audio playback method

Looks like parallel audio playback is Phase 6 here https://wiki.asterisk.org/wiki/display/AST/Asterisk+14+Project+-+URI+Media+Playback .
Any way to do this without text/uri-list support?

Not natively, anyway, no.  We can, I believe, manually and separately play each... but this isn't a high priority feature.


JRE bundled with our software gets corrupted while being installed by brew

brew gist-logs <formula> link OR brew config AND brew doctor output
https://gist.github.com/Ahmed3lmallah/9c913954578753cc1c2137dea733b92a

Verification

[X] My "brew doctor output" says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.
[X] I searched for recent similar issues at https://github.com/Homebrew/homebrew-core/issues?q=is%3Aissue and found no duplicates.

What were you trying to do (and why)?
Install a binary package that exists at tar file using the following tap and have it run successfully using: srcclr --version
To eliminate the requirements for users to separately install java runtime environment, we bundle it with the package inside the 'jre' folder.
It seems that the 'jre' folder is getting corrupted or modified somehow when brew install srcclr is executed.
What happened (include all command output)?
The following error was presented:
➜  ~ srcclr --version
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x0000000000000000, pid=90912, tid=11523
#
# JRE version:  (17.0.6+10) (build )
# Java VM: OpenJDK 64-Bit Server VM (17.0.6+10-LTS, mixed mode, tiered, compressed oops, compressed class ptrs, g1 gc, bsd-amd64)
# Problematic frame:
# C  0x0000000000000000
#
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /Users/aelmallah/hs_err_pid90912.log
#
#
[1]    90912 abort      srcclr --version

The same error is presented when attempting to run the JRE using the java binary:
➜  bin git:(stable) pwd
/opt/homebrew/Cellar/srcclr/3.8.27/libexec/jre/bin
➜  bin git:(stable) ./java --version
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x0000000000000000, pid=94174, tid=11267
#
# JRE version:  (17.0.6+10) (build )
# Java VM: OpenJDK 64-Bit Server VM (17.0.6+10-LTS, mixed mode, tiered, compressed oops, compressed class ptrs, g1 gc, bsd-amd64)
# Problematic frame:
# C  0x0000000000000000
#
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /opt/homebrew/Cellar/srcclr/3.8.27/libexec/jre/bin/hs_err_pid94174.log
#
#
[1]    94174 abort      ./java --version

What did you expect to happen?
The JRE was expected to work correctly. In fact, deleting the jre folder and replacing it with the folder from the download link works and gives the following result:
➜  bin git:(stable) pwd
/opt/homebrew/Cellar/srcclr/3.8.27/libexec/jre/bin
➜  bin git:(stable) ./java --version
openjdk 17.0.6 2023-01-17 LTS
OpenJDK Runtime Environment Zulu17.40+19-CA (build 17.0.6+10-LTS)
OpenJDK 64-Bit Server VM Zulu17.40+19-CA (build 17.0.6+10-LTS, mixed mode)
➜  bin git:(stable) srcclr --version
Version: 3.8.27

Step-by-step reproduction instructions (by running brew commands)
brew tap veracode/srcclr
brew install srcclr
srcclr --version

To confirm that the 'jre' folder was corrupted/modified and the problem is not from the source:

delete the 'jre' folder from /opt/homebrew/Cellar/srcclr/3.8.27/libexec
download the package from https://download.srcclr.com/srcclr-3.8.27-macosx.tgz
unzip the package
copy the 'jre' folder from /Users/***/Downloads/srcclr-3.8.27 to /opt/homebrew/Cellar/srcclr/3.8.27/libexec
run srcclr --version
you should get back the correct version

Formulae were designed to build things from source, so it's not really unexpected to end up with a broken install when you use them for pre-built binaries.
Have you tried packaging your software as a cask instead?

Have you tried packaging your software as a cask instead?

Thank you @carlocab for pointing me in the right direction. It seems that using a cask instead of a formula is what we are looking for.


"Burn After" first read burns immediately after preview

Describe the bug
When set a text to burn after first read and hit save, the preview of the text will also be counted as a read.
To Reproduce
Steps to reproduce the behavior:

Change the Burn After parameter to Burn after 1st read
Enter some text, and hit Save to submit
Browser redirects to the text to "preview" the pasta
The pasta is now gone and unable to share

Expected behavior
The "burn" count should not count the "preview" of the pasta.
For instance, I want to share a password to a friend, and set the pasta to burn right after their first read. But when I try to copy the link, the link has already expired.
Screenshots
N/A
Desktop (please complete the following information):

OS: Windows 11 22H2
Browser: Microsoft Edge x64 114.0.1823.82
Version Microbin 2.0.4

Additional context
The issue didn't exist in MicroBin 1.x. The bug is also reproducible on Safari on iPadOS 17 Beta 3, assuming it's a cross-platform cross-browser issue.

Closing due to a duplication of #182


cmd/cue: hidden values cannot be referenced by tools defined in parent directory

What version of CUE are you using (cue version)?

$ cue version
cue version 0.4.0 linux/amd64

What did you do?

$ cat dir/ca.cue
package abc

_name: "cue"

$ cat hello_tool.cue
package abc
import (
	"tool/cli"
)

command: hello: {
	print: cli.Print & {
		text: "Hello \(_name)!"
	}
}

$ cue cmd hello ./...

What did you expect to see?

Hello cue!

What did you see instead?

command.hello.print.text: reference "_name" not found:
    ./echo_tool.cue:9:18

This seems to be related to #924. In this case, the cue file resides in a leaf directory. Moving the ca.cue file next to the hello_tool.cue file works as expected.

@timspeetjens thanks for the reproducer above. Yes, definitely related to #924 and again seems like a valid request.

(sorry, that last message sent too quickly)
@timspeetjens purely as an FYI (because your reproducer above is great) you can specify a reproducer as a single file that can be copy-pasted as a test case:
https://github.com/cuelang/cue/wiki/Creating-test-or-performance-reproducers
Not critical, but does make the job of fixing things that bit easier.
Here's such a txtar archive for this example:
exec cue cmd hello ./...
stdout 'Hello cue!'

-- dir/ca.cue --
package abc

_name: "cue"
-- hello_tool.cue --
package abc

import (
	"tool/cli"
)

command: hello: {
	print: cli.Print & {
		text: "Hello \(_name)!"
	}
}

For completeness, the following also fails, but might be the same issue:
exec cue cmd hello ./...
stdout 'Hello cue!'

-- dir/ca.cue --
package abc

p: {
	_name: "cue"
}
-- hello_tool.cue --
package abc

import (
	"tool/cli"
)

command: hello: {
	print: cli.Print & {
		text: "Hello \(p._name)!"
	}
}

The output differs slightly:
command.hello.print.text: invalid string argument: invalid interpolation: undefined field: _name:
    ./hello_tool.cue:8:2
    ./hello_tool.cue:9:3
    ./hello_tool.cue:9:9
    ./hello_tool.cue:9:20
    tool/cli:4:3

but might be the same issue:

Yes, it's the same issue.

This issue has been migrated to https://github.com/cue-lang/cue/issues/1032.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Module file path incorrect when nested under mod block

When I try to use cargo-mutants in my crate, it reports that some modules are not found.   It seems when a mod (no block) is defined within a mod block, the relative path is not fully tracked.
The description below is based on a minimal reproduction.
Desired behavior
$ cargo mutants
Found 4 mutants to test
ok       Unmutated baseline in 0.2s build + 0.1s test
 INFO Auto-set test timeout to 20s
MISSED   src/toplevel/nested.rs:2:10: replace + with * in add in 0.2s build + 0.1s test
4 mutants tested in 1s: 1 missed, 3 caught

Failure behavior
$ cargo mutants
 WARN referent of mod not found parent_path="src/lib.rs" mod_name=nested tried_paths=["/home/ned/git/cargo-mutants-testcase/src/nested.rs", "/home/ned/git/cargo-mutants-testcase/src/nested/mod.rs"]
Found 0 mutants to test
 WARN No mutants found under the active filters

$ cargo mutants --version
cargo-mutants 24.3.0

Details of the test case
Details of the change from "desired" to "failure" that triggers the issue

src/lib.rs-pub mod toplevel;
+pub mod toplevel {
+    pub mod nested;
+}

 #[cfg(test)]
 mod tests {
    use super::toplevel::nested::*;

    #[test]
    fn it_works() {
        let result = add(2, 2);
        assert_eq!(result, 4);
    }
}

src/toplevel.rs (deleted)-pub mod nested;

tree summary $ tree .
 .
 ├── Cargo.lock
 ├── Cargo.toml
 └── src
     ├── lib.rs
-    ├── toplevel.rs
     └── toplevel
         └── nested.rs

Proposed solution
Looking over DESIGN.md, I see that module discovery ignores mod statements with a block.
Would you be open to a PR to track the nesting of mod blocks, to calculate the path to mod files (with no block) similar to rustc?  I don't have much experience with the syn crate, but would be happy to see how this might work for cargo-mutants.
Metadata

Tested using branch `main` at 442412650fc9770f3bc84467e1b104f5ef51b7a6
$ git clone https://github.com/sourcefrog/cargo-mutants.git

$ cd cargo-mutants

$ git rev-parse HEAD
442412650fc9770f3bc84467e1b104f5ef51b7a6

$ cargo build

$ ./target/debug/cargo-mutants mutants --version
cargo-mutants 24.3.0

[setup minimal reproduction case]
$ cd ../cargo-mutants-minimal-reproduction

$ ../cargo-mutants/target/debug/cargo-mutants mutants

Thanks very much for the clear report and reproduction! I had not realized that you could have mod statements that correspond to multiple directories but it makes sense.
I think visit_item_mod should already be tracking the nested namespaces. https://github.com/sourcefrog/cargo-mutants/blob/442412650fc9770f3bc84467e1b104f5ef51b7a6/src/visit.rs#L341-L356
The problem is that the accumulated list of external_mods only takes the last component of the path. Maybe it should use the whole stack... although, we need to probably be careful to get this right if the mod we're already looking at is some layers down.
The interaction of the stack with the existing filenames suggests to me that maybe a better test is something with a mix of a file that is itself 2 levels down, and that then has an external mod that is then some further levels down.
A PR would be very welcome.
Some ideas on how to start:

[ ] Copy your tree under testdata with an appropriate directory and package name
[ ] Add it to the workspace exclusions in /Cargo.toml
[ ] Add the namespaces to the stack in visit.rs
[ ] Add a test in visit.rs for reading the testdata, like in https://github.com/sourcefrog/cargo-mutants/blob/442412650fc9770f3bc84467e1b104f5ef51b7a6/src/visit.rs#L589-L599

Also, not necessarily in the same PR: at the moment https://github.com/sourcefrog/cargo-mutants/blob/442412650fc9770f3bc84467e1b104f5ef51b7a6/src/visit.rs#L65-L74 only warns if it can't find a mod, but at least for tests we should probably have an option to make this an error.
https://github.com/sourcefrog/cargo-mutants/blob/442412650fc9770f3bc84467e1b104f5ef51b7a6/src/visit.rs#L457-L468


Fix ordering so that Openldap::Server::Globalconf resources will come after the openldap service

Without this change, using openldap::server::globalconf results in a fresh system trying to set global config items before the slapd service is running.

LGTM


Add pattern matcher validation

Address issue #29 by adding the pattern matcher validation and corresponding tests
Update version to 2.0.4

Looking great! One bit we're missing here is an update to src/constants.ts.  We currently define the default error messages there. They're override-able, but the current error messaging scheme uses the validation name as a lookup.
How do you feel an error message for pattern should be laid out. Maybe something generic as some form of "invalid pattern" error message, and then instances of the error messages can be overriden as necessary?
Open to suggestions!

Sure. I'm happy to add something somewhat generic to constants.ts.
As it is written right now this validator is actually essentially  an extension of  the contains validator unless a user specifically wrote the regex with a ^ at the start and a $ at the end. I'm wondering if this or the double \\ for literal slash character should be documented somehow.

@JenkinsDev Added the default message to constants.

Ah good catch. Right now I'm compiling most of the documentation into the README, which also gets used in the root of the API site. Perhaps this is a good place to document the addition of pattern, plus the nuance above.
I have plans to work out some form of library/documentation site (beyond https://jenkinsdev.github.io/Validatinator/).

I've added the pattern validation to the README along with some notes about it and the difference and same validations. I found the meaning of the second argument for those validations difficult to find, so I included it here as well.

I think we're all set here!
Good catch on same, contains, and them->then, in the readme!
Going to go ahead and merge this in now

thanks for merging this and maintaining this useful library 👍


chore: remove gh-pages links and less clunky telemetry opt-in

Removes explicit links to gh-pages and re-map them to getsynth.com at the right place
Reformat the displaying of the telemetry message, add a TL;DR
Also removes cloud shell related utils.

LGTM


Adding/removing a metric resets skyplot to home zoom level/pan position

A common use case might be to zoom in on a region of the sky plot and then add a metric, wishing to see it at that same location.  As it is now, adding or removing a metric fully resets the sky plot.  Perhaps this can be addressed by independently keeping intentional track of the x_range and y_range of the skyplots, such that new plots are loaded with those ranges.

This might relate to #28 as well.

Perhaps this can be addressed by independently keeping intentional track of the x_range and y_range of the skyplots, such that new plots are loaded with those ranges.

I'd create RangeXY streams once on initialization and then reuse them and initialize the x_range and y_range parameters on the rasterize call with their values.


configure_make: use CMAKE_C_COMPILER_TARGET to detect cross-compilation

Currently, the only way to cross-compile using vcpkg_configure_make is
to use a gcc cross toolchain with the target as the prefix.
This is not how you cross compile with clang--with clang you set
CMAKE_<LANG>_COMPILER_TARGET. This detects this from the toolchain file
and passes it along.
This is tested & works with an internal toolchain file. libbacktrace failed to cross-compile without this change.

Don't expect changes to vcpkg_configure_make: Port vcpkg-make is coming in #39050. And there is a triplet variable which you can already use.

Not sure what you mean by "there is a triplet variable which I can already use" which variable are you referring to, and why would it be better than CMAKE_C_COMPILER_TARGET?
Should I wait for that PR to land and update ports?

The vcpkg owners don't want make changes to the unversioned script. Too much risk. So we have to wait for that PR.
The variable is VCPKG_MAKE_BUILD_TRIPLET,
https://learn.microsoft.com/en-us/vcpkg/maintainers/functions/vcpkg_configure_make#build_triplet
I don't say it is better asking the compiler, but at least you can control which options are passed to configure.

Closing for both those reasons.


feature: Added onError callback to handle Errors on mounting/starting the Camera

As the title says, this PR implements the missing callback for errors which are thrown by the useCamera hook.
Issue: #54

@yudielcurbelo This merge request is crucial for our project. Please approve it as soon as possible. Without this update, we won’t be able to manage unauthorized user actions effectively.


[Scope] codeunit 8 AccSchedManagement - DrillDownOnGLAccount, DrillDownOnCFAccount

Please could you change the scope to public for these methods?

DrillDownOnGLAccount
DrillDownOnCFAccount

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


1.8.6 is not generating the widgets

In my nox environment the version 1.8.6 is not generating the widget upon installation. Is it something you experienced as well ?

Thank you for reporting, I yanked the release. We will look into it!


⚠ Bofrost.nl has degraded performance

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

HTTP code: 200
Response time: 1280 ms

Resolved: Bofrost.nl performance has improved in 55516bc.


Improve message broker function by adopting nestjs

Problem description
Currently Guardian system works directly with NATS message broker. This is not ideal as it requires implementation of message handling logic which already exists in other scalable broker management frameworks including nestjs.
Requirements
Introduce nest.js  framework into Guardian architecture to achieve better scalability, stability and improve code reuse. This should also enable a switch to another message broker in the future should we need to.
Definition of done

Guardian architecture relies on nest.js for messaging.
No clients customisations are affected by the above.

Acceptance criteria

See DOD above

NestJS is  great, we using NestJS for everything at Tymlez (includes microservice architecture)


Concept for Project Documentation

Summary
The goal of this issue is to propose a first documentation concept to iterate on and start a discussion around this topic.
Motivation
As the Project grows it would be helpful to have a good documentation, especially for new contributors and users.
The documentation has to contain all neccessary information while still beeing easy to use and maintain.
Technical Design
TL;DR
Create a seperate repository "OpenDevin/OpenDevinDocs" for documentation, containing autogenerated code documentation and manually created parts (Overview, architecture, examples)
Generation and content
The documentation should at least contain the following contents:

High-level project overview
Installation instructions, How-To Guide, Examples
Architectural diagrams
API documentation
Generated code documentation

This means documentation will consist of two parts: An autogenerated code documentation Part which is generated by GitHub actions on every commit/PR and a part for which has to be maintained manually, but less frequently.
Versioning / Repository
The documentation should be stored in a seperate repository (e.g. OpenDevin/OpenDevinDocs)
This enables tracking documentation-related issues separately, keeping the main repository focused on development.
It would also be possible to implement the same branch concept as in the code repository by automatically creating a docs branch for each code branch.
Separation between Frontend and Backend
Pro seperation: Frontend and backend use very different technology stacks. A seperation allows it to use better suited tools for each part.
Con seperation: Frontend and Backend are part of the same project, so the documentation should also consider both parts for a better understanding of the project as a whole.
Proposal: Store frontend and backend docs in the same repository with a root-level separation (similar to the code repository). Then docs for frontend and backend can be generated separately and maybe can be tied together with an index site for navigation between both parts.
Tooling
For Backend: Use Sphinx due to its wide usage and extensive configurability.
For Frontend: TBD.
Documentation Format
HTML for best readability/design and compatibility for hosting on various platforms.
Hosting
Sphinx generates HTML files which can be hosted for example on GitHub Pages or readthedocs.
Alternatives to Consider
Additional Context
This page provides a good guideline on the content of a project documentation
https://coderefinery.github.io/documentation/wishlist/

@huybery we can close this

We will definitely need a docs site at some point, so I think it makes sense to keep it open.
@jannikwinghart IMO we should keep the docs within this repo for now. It's much simpler to manage everything in one place. I could be convinced otherwise though.

In my opinion the main constraints regarding docs generation and storage should be:

Don't flood the source repo with autogenerated docs html files (increases repo size, leads to many "irrelevant" changes in each PR, etc.)
Don't build the docs locally, build process has to happen in the CI-Pipeline (no time consuming pre-commit hooks, don't introduce additional toolchain requirements for contributors)

In case we want to use github pages for hosting (or similar static html hosting sites), we would need to generate all html files and push them to a repo which then can be hosted.
In this case I would prefer to use a seperate repo to fulfill the constraints above.
The other option (which I also like after reading into it) is to not generate and store the html files on our end, but instead use a service like readthedocs for generation and hosting of the docs files. readthedocs builds and stores the html files on their servers completely, so our repo would only contain the documentation "source" files (sphinx rst, etc.), and no autogenerated html files.
At least for the python part this would be a clean solution, even though I'm not quite sure yet if docs for the tsx/frontend part can be integrated easily.
Possible downside on this option is that the docs build process (and possible errors) is not visible for everybody, only for the readthedocs project maintainers (unlike github actions).


Fix leak in example.c test_deflate_copy()

In test_deflate_copy(), we free the original struct but forget to free the copy stuct.
Found by msan.

Looks good.


add prefix-variable to version-number wrapper

Add support for prefix-variable in version-number wrapper.

wrong process


Add "wedge15" to topological_dimension dictionary

The topological_dimension dictionary in version 5.2.2 is missing information from the cell type "wedge15". This pull request is intended to fix #1247.

This got fixed in a different way now. Thanks anyway!


Use myadd.ir for get IP address

The ifconfig.co is behind Cloudflare's CDN and blocks some IPs to access it and give a 403 HTTP error. I propose myadd.ir to fetch the IP address.

myadd.ir

Thank you for your kind contribution.
Yes it seems some ips are blocked by ifconfig.me, I have already replace ifconfig.io with it and it seems this site is working without limitation, so the issue is resolved.
Thanks again!


AZ cli is fialed to install on ubuntu for xenail

gnutls_handshake() failed: Error in the pull function.
W: The repository 'https://packages.microsoft.com/repos/azure-cli xenial Release' does not have a Release file.
N: Data from such a repository can't be authenticated and is therefore potentially dangerous to use.
N: See apt-secure(8) manpage for repository creation and user configuration details.
E: Failed to fetch https://packages.microsoft.com/repos/azure-cli/dists/xenial/main/binary-amd64/Packages  gnutls_handshake() failed: Error in the pull function.
E: Some index files failed to download. They have been ignored, or old ones used instead.---
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) / CLI version (az --version) / OS version / Shell Type (e.g. bash, cmd.exe, Bash on Windows)

Please try installing again. The Xenial package of CLI is available.


sdk_lib/sdk_entry.sh: use a login shell to source /etc/profile

For execution of the compiled binaries in /build/arm64-usr we rely on
qemu-user binfmt emulation and have to tell it where the root is with
QEMU_LD_PREFIX because build systems don't chroot into /build/arm64-usr
themselves (which also works just by chance on amd64 because we have
similar glibc versions and so on). The env var setup was done in
/etc/profile.d/qemu-aarch64.sh but is now not read anymore since the
container runs the shell not as login shell.

Add the login options to the bash and su calls when starting the
container.

How to use
Check that env really prints out QEMU_LD_PREFIX=/build/arm64-usr/
Backport to all active branches
Testing done
Manual edit of the file

@pothos please don't forget to cherry-pick into flatcar-3033 and flatcar-3066 (we can skip flatcar-3087 I guess since it never got released and we're now planning to release an entirely new Alpha).

Thanks, I did it but hadn't pushed it until now


add RectsPulsarCalculator and NormalizedRectsPulsarCalculator

This calculator turns a vector<T,N> packet into N packets of T.
Tests are coming.

Actually, today's SplitVectorCalculator is enough for my needs.
I am still curious about how you think a MapCalculator could be done (applying some calculator to N streams of inputs, N being known only at runtime. With a side-packet concurrency granularity parameter).
Can one add input/output streams and calculators instances to an already Initialized graph?


feature request: Progress Bar option

Some operations can become extremely long when converting or hadd'ing (usually too many) histograms, progress bars can be great to have around.
rich is part of pip for some time, and has options for sub-progress bars where appropriate.
https://rich.readthedocs.io/en/stable/progress.html#advanced-usage

In scientific contexts, I more often see tqdm used as the default progress bar. I had hoped that it would have the same interface as rich.progress.Progress, but it doesn't. It does seem to be the more popular of the two. Oh wait—there's a backend to use rich progress bars with a tqdm interface, so it's not an exclusive choice.
tqdm's interface comes in two steps:

construct with progress_bar = tqdm.tqdm(total=number_of_items) or reassign an already-constructed object with the number of items using progress_bar.reset(total=number_of_items).
update with progress_bar.update() "number of items" times.

It has lots of bells and whistles, but rather than make hepconvert choose what you hope is the user's favorite display style, hepconvert functions could take a progress_bar argument and (1) immediately reset it to tell it the number of items and (2) call update in hepconvert's loop.
That way, users can set up the progress bar however they want and you just control its filling. Also, hepconvert wouldn't need to depend on either rich or tqdm (though it might, for other reasons).
Here's what a minimal use would look like to a user:
hepconvert.merge_root("out.root", ["in1.root", "in2.root"], progress_bar=tqdm.tqdm())

and the implementation inside hepconvert would look like:
def merge_root(output, inputs, progress_bar=None):
    number_of_items = ...  # calculate this from the number of inputs somehow

    if progress_bar:   # not None and not False
        if progress_bar is True:  # progress_bar=True should provide a default
            import tqdm           # which strictly depends on the tqdm library
            progress_bar = tqdm.tqdm()
        progress_bar.reset(number_of_items)

    for item in data:
        ...
        if progress_bar:
            progress_bar.update()

See uproot/src/uproot/extras.py for ways of making the "failure due to missing optional dependency" more helpful, by providing copy-pasteable command lines to install the missing package.


gulp-eslint errors out on it's own arrow function

Using latest gulp-eslint with gulp 4.0. On a couple of OSX machines, no problems. On another with identical setup and same package.json, getting the following:
/node_modules/gulp-eslint/index.js:32
return util.transform((file, enc, cb) => {
                                      ^^
SyntaxError: Unexpected token =>
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:427:25)
    at Object.Module._extensions..js (module.js:462:10)
    at Module.load (module.js:339:32)
    at Function.Module._load (module.js:294:12)
    at Module.require (module.js:349:17)
    at require (module.js:368:17)
    at requireFn

Looking at mismatch version of gulp-cli, but that doesn't seem to be the case. Any thoughts?
Thanks!

http://www.catb.org/esr/faqs/smart-questions.html#forum

@shinnn
Could you elaborate? There's an ES6 arrow function on line 32 of gulp-eslint/index.js:32. Why would one instance compile fine and another not? There's no Babel or other transpiler in the project. According to http://node.green/ arrow funcs are supported near 100% in Node going back to 4.4.6. One of the machines not throwing an err has 4.2.2, lower than the machine throwing the error.
I must be missing something, since you closed it w/o reply, but I'm not sure what. Any guidance would be appreciated.
package.json
{
  "name": "redacted",
  "title": "redacted",
  "version": "2.0.0",
  "description": "front end build tools, including legacy and current",
  "main": "",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "gulp": "./node_modules/gulp-cli/bin/gulp.js",
    "generate": "./node_modules/generator/bin/index.js"
  },
  "author": "",
  "devDependencies": {
    "browser-sync": "^2.13.0",
    "camelcase": "^3.0.0",
    "eslint-formatter-pretty": "^0.3.1",
    "fs-extra-promise": "^0.4.0",
    "glob": "^7.0.5",
    "gulp": "github:gulpjs/gulp#4.0",
    "gulp-angular-templatecache": "^1.8.0",
    "gulp-cli": "latest",
    "gulp-concat": "^2.6.0",
    "gulp-eslint": "^3.0.1",
    "gulp-imagemin": "^2.4.0",
    "gulp-less": "^3.0.5",
    "gulp-load-plugins": "^1.1.0",
    "gulp-ng-annotate": "^1.1.0",
    "gulp-rename": "^1.2.2",
    "gulp-sourcemaps": "^1.6.0",
    "gulp-svg-sprite": "^1.3.3",
    "gulp-template": "^4.0.0",
    "gulp-uglify": "^1.5.1",
    "gulp-util": "^3.0.7",
    "gulp.spritesmith": "^6.2.1",
    "imagemin-jpeg-recompress": "^4.3.0",
    "less-plugin-glob": "^1.1.1",
    "lodash": "^4.14.0",
    "node-notifier": "^4.3.1",
    "request": "^2.72.0",
    "require-dir": "^0.3.0",
    "yargs": "^3.30.0"
  }
}

eslint config
{
  "extends": "eslint:recommended",
  "rules": {
    "no-alert": 1,
    "no-bitwise": 0,
    "camelcase": 1,
    "curly": 0,
    "eqeqeq": 0,
    "no-eq-null": 0,
    "guard-for-in": 1,
    "no-empty": 1,
    "no-unused-vars": 1,
    "no-console": 1,
    "no-mixed-spaces-and-tabs": 0
  },
  "globals": {
    "$": true,
    "angular": true,
    "App": true,
    "MainController": true,
    "Modernizr": true
  },
  "env": {
    "browser": true,
    "node": false
  }
}

you closed it w/o reply

You're missing https://github.com/adametry/gulp-eslint/issues/175#issuecomment-244254081.
===========
We can dig into the problem further by spending our precious lifetime, if:

You create a public Github repository.
You build the project on Travis CI with macOS environment and show us the log of failing build.

However, you still need to read and understand https://github.com/adametry/gulp-eslint/issues/175#issuecomment-244254081.

Ouch. Well, based on your comment, you're misunderstanding the question. No worries, time to move on.

@samelwitt Again, just read https://github.com/adametry/gulp-eslint/issues/175#issuecomment-244254081. Do you really think this issue should have been posted here?


improvements on providerID logic

During the work on the new providerID format, there were some improvement ideas discussed.
This issue tracks these issues. For each suggestion, a github issue is created, listed below.
https://github.com/metal3-io/cluster-api-provider-metal3/issues/588
https://github.com/metal3-io/cluster-api-provider-metal3/issues/589
https://github.com/metal3-io/cluster-api-provider-metal3/issues/590

/triage accepted
/kind feature

ProviderID logic has changed recently. Due to that, the suggested improvements are no longer relevant. closing this and the sub-issues.


Frontend deploy cleanup

Related Issue or Background Info

env vars set in package.json are not human readable
deploys should be managed through github actions
deploys run yarn build twice but only use the output of one

Changes Proposed

remove old maintenance page
remove duplicate ENV VARS (REACT_APP_OKTA_URL)
remove deploy scripts from package.json

Don't see why it wouldn't work, but maybe deploy test from this branch?

Don't see why it wouldn't work, but maybe deploy test from this branch?

Works on test :D


User Link Disappears After Navigating Back to Security Group from Linked Users Page

Current behavior
Upon returning to the Permissions tab and reopening the same security group, the linked users link is missing unless the page is refreshed.
Expected behavior
The linked users count (e.g., "6 users") should consistently display as an active link every time the security group is accessed, regardless of prior navigation actions.
Steps to reproduce the issue
Navigate to the Permissions tab in the application.
Open a Security Group that displays a linked users count (e.g., "6 users") with an active link.
Click on the linked users link to be redirected to the Users page showing the associated users.
Use the application's navigation to return to the Permissions tab.
Reopen the same Security Group.
Environment details
App version : v1.10.2
App environment : UAT
Additional Information
https://jam.dev/c/434b3544-1ea1-446b-b98f-6ef3a9eda2c1

Issue verified and working fine in release v1.11.0. Hence, closing this issue.


Do not cache prep calls.

This fixes the bug that the master password that was temporarily set when checking for valid configs are cached. The master password was temporarily set when performing checks to avoid typing the master password twice.

@MoSHad91 this should do it. After you've had a look we create a new release.


Update custom auth documentation with better challenge example

Describe the bug
Current custom auth documentation in auth category uses a hard coded value as the challenge. We should provide a better example similar to this blog where it generates a random code for the custom challenge.
Steps To Reproduce
Steps to reproduce the behavior:
https://docs.amplify.aws/lib/auth/signin_with_custom_flow/q/platform/ios

Expected behavior
Document should have better steps
Amplify Framework Version
All
Amplify Categories
Auth
Dependency manager
Swift PM
Swift version
NA
CLI version
NA
Xcode version
NA
Relevant log output
No response
Is this a regression? (i.e. was this working before a version upgrade)
No response
Device
NA
iOS Version
NA
Specific to simulators
No response
Additional context
No response

Waiting for CLI to remove the prompt asking for the hard coded value. After that is implemented, we need to update the doc.

PR added - https://github.com/aws-amplify/docs/pull/3620

PR merged


[THREESCALE-1692] Change schema of IP check policy so it renders correctly in the UI

Closes #970 and THREESCALE-1692

Rebased to solve the conflicts in the changelog.


feat(dashboard): add functions to logs

Codecov Report

Merging #1426 (9a1aa7b) into main (6ef340d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1426   +/-   ##
=======================================
  Coverage   81.39%   81.39%
=======================================
  Files          83       83
  Lines        9236     9236
  Branches      473      473
=======================================
  Hits         7518     7518
  Misses       1718     1718

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

@guicurcio I am testing the backend changes now required to have this merged.


Some sound files do not play

Example: kämpar https://lexin.mobi/?lang=english&query=kämpar
The "lyssna" link points to https://lexin.nada.kth.se/sound/k%C3%A4mpar.mp3 which is not available.
In the third-party mobile application Lexin this word works fine.
Screenshot

(Semi-accidentally) Fixed in #32


Support new quantization formats for quantizer

At present, the quantizer only supports q4_0 and q4_1 output. We should support all the formats that GGML supports for quantization at the time this issue is written.
(Note: the formats can change or be removed, so always check what the currently supported formats are)

It's been merged: https://github.com/ggerganov/llama.cpp/pull/1405
There doesn't seem to be a migration path at present, so let's wait a bit: https://github.com/ggerganov/llama.cpp/issues/1408


Bugfix: Use HTML from content API for guest frame includes

Fixes #5 by using the generated HTML from Neos itself.
Additionally fixes #3 by adding an effect to dispatch load/unload events on route changes.
Note: needs https://github.com/networkteam/Networkteam.Neos.ContentApi/pull/2 to work correctly.

I only tested this with Neos 8.1 so far.


prevent extension from splitting inline-code and link marks

Currently, the extension will split around the inline code block or a link for the final markup as follows:
 and the markup looks like this:
prevent extension from splitting
code
 blocks and links

I'd prefer it as follows:

prevent extension from splitting code blocks and links

Code.extend({
  excludes: "",
})

should allow comments on code mark.
don't think it'll render code inside comment span, for prosemirror has a special way of rendering multiple marks on the same text & it doesn't nest one inside another.


BUG: {CSR, CSC}.has_canonical_format not updated after .sort_index()

Describe your issue.
The has_canonical_format property does not accurately update after calling sort_indices on a csr or csc matrix that doesn't have duplicated values.
The cached value should instead be (at least) unset when the value of has_sorted_indices changes.
Reproducing Code Example
from scipy import sparse

x = sparse.csr_matrix((list(range(5)), [3, 1, 2, 0, 1], [0, 3, 5]))
assert not x.has_sorted_indices # False
assert not x.has_canonical_format # False
x.sort_indices()
assert x.has_sorted_indices # True
assert x.has_canonical_format # Raises

assert sparse.csr_matrix((x.data, x.indices, x.indptr), shape=x.shape, copy=True).has_canonical_format # True

Error message
AssertionError:

SciPy/NumPy/Python version and system information
1.11.2 1.25.2 sys.version_info(major=3, minor=10, micro=12, releaselevel='final', serial=0)
Build Dependencies:
  blas:
    detection method: pkgconfig
    found: true
    include directory: /usr/local/include
    lib directory: /usr/local/lib
    name: openblas
    openblas configuration: USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= SANDYBRIDGE MAX_THREADS=3
    pc file directory: /usr/local/lib/pkgconfig
    version: 0.3.21.dev
  lapack:
    detection method: pkgconfig
    found: true
    include directory: /usr/local/include
    lib directory: /usr/local/lib
    name: openblas
    openblas configuration: USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= SANDYBRIDGE MAX_THREADS=3
    pc file directory: /usr/local/lib/pkgconfig
    version: 0.3.21.dev
  pybind11:
    detection method: config-tool
    include directory: unknown
    name: pybind11
    version: 2.10.4
Compilers:
  c:
    commands: cc
    linker: ld64
    name: clang
    version: 13.0.0
  c++:
    commands: c++
    linker: ld64
    name: clang
    version: 13.0.0
  cython:
    commands: cython
    linker: cython
    name: cython
    version: 0.29.36
  fortran:
    commands: gfortran
    linker: ld64
    name: gcc
    version: 11.3.0
  pythran:
    include directory: ../../../../../../private/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/pip-build-env-mxdji_nf/overlay/lib/python3.10/site-packages/pythran
    version: 0.13.1
Machine Information:
  build:
    cpu: x86_64
    endian: little
    family: x86_64
    system: darwin
  cross-compiled: false
  host:
    cpu: x86_64
    endian: little
    family: x86_64
    system: darwin
Python Information:
  path: /private/var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/cibw-run-6np22kwu/cp310-macosx_x86_64/build/venv/bin/python
  version: '3.10'

this issue is now manifesting in our tests for pandapower. Even after doing the following:
## build Ybus
Ybus = Cf.T * Yf + Ct.T * Yt + \
       csr_matrix((Ysh, (range(nb), range(nb))), (nb, nb))
Ybus.eliminate_zeros()
Ybus.sum_duplicates()
Ybus.sort_indices()

the test for canonical format is failing (used to pass with the previous scipy release)

@rbolgaryn can you be more specific about which versions are "previous" and "now"? Do you mean 1.13 makes it change or 1.12 makes it change?
Also, I don't understand your example, it adds a shape (nb,nb) csr_matrix to an expression, which should create a square matrix shape=(nb, nb) but the error report is a 5x6 matrix. Am I missing something?


How to automatically scroll to the specified header item?

If I understand correctly, getGlobalPositionOf() only searches in the list of items provided in the constructor. How to get global position of a header(based on it's instance) so it can be used in recyclerView.scrollToPosition() ?
edit: i have already correctly implemented equals() nad hashCode() methods

@shymmq, getGlobalPositionOf returns the position of any item at any time, also for Headers if they are visible! So you can scrollTo using that method.
Good that you implement equals method, because List makes use of it.

Hmm, for some reason getGlobalPositionOf() always returns 0. Can you quickly check the following code?
//in onCreateView:
RecyclerView recyclerView = (RecyclerView) root.findViewById(R.id.fragment_timetable_recycler);
Collections.sort(schoolWeeks);
for (SchoolWeek schoolWeek : schoolWeeks) addSchoolWeek(schoolWeek); //listElement items are being added here, with respective header items.
final FlexibleAdapter<IFlexible> adapter = new FlexibleAdapter<>(listElements);
adapter.setDisplayHeadersAtStartUp(true);
LinearLayoutManager layoutManager = new LinearLayoutManager(getContext());
recyclerView.setLayoutManager(layoutManager);
recyclerView.setAdapter(adapter);
int sectionIndex = adapter.getGlobalPositionOf(todayHeader);
Log.d(TAG, "onCreateView: Scrolling to position " + sectionIndex);
layoutManager.scrollToPositionWithOffset(sectionIndex, 0);

Using debugger i checked that todayHeader is in fact being correctly set in addSchoolWeek()
This is equals() from my header item class:
@Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (o == null || getClass() != o.getClass()) return false;

        LessonHeaderItem that = (LessonHeaderItem) o;

        return date.equals(that.date);
    }

Does the getGlobalPositionOf use equals(), or == to compare the items?

I said at any time, but it is not really true at startup, sorry:
scrollTo is strictly linked to the layout manager, as Android team says, layout items might not match with adapter items due to layout operations. So we should also invoke scrollTo after the layout manager complete the layout. Use a delayed post of 50 or 100 ms.
But also, you should use scrollTo from the RV and not from the layout manager.
0 is a valid position, it means that the adapter found the item at first position. If so, scrollTo doesn't produce any effect because the first visible item is already at that position. Which position did you expect instead?
Your equals() seems ok, it is used by the Java collections (in this case List), not by the Adapter directly.

Thnak you for your support. postDelayed() with delay of 50ms works OK, but i will try to find a better way to do this as the delay is noticeable.

50-100ms is not noticeable. You can try even less, the importance is the layout manager completes the layout.


Visibility: Query with customized filter

Currently with Visibility on Cassandra, filter by fields is built on secondary index, so it hard to support multiple field search in combination, such as use both workflow_type and status as filter.
With Visibility on ElasticSearch, we can support multiple fields search.
The plan is to create a new API ListWorkflow, which takes a "SQL where clause string" as query, such as "(WorkflowType = wt1 AND Status = 0) OR WorkflowID = wid1"

https://github.com/uber/cadence/pull/1744

New API will takes query string like SQL, such as (WorkflowID = 'id1' and WorkflowType = 'type1') or WorkflowType = 'type2'.
Add time filter like StartTime between t1 and t2 and CloseTime >= t3 (note time value t1 t2 t3 is epoch time in nanoseconds)
List only open workflow, add CloseTime = missing in query string. Default will list all workflows according to query.

New API will takes query string like SQL, such as (WorkflowID = 'id1' and WorkflowType = 'type1') or WorkflowType = 'type2'.
Add time filter like StartTime between t1 and t2 and CloseTime >= t3 (note time value t1 t2 t3 is epoch time in nanoseconds)
List only open workflow, add CloseTime = missing in query string. Default will list all workflows according to query.


Add withAtLeastOne() to Parameter* for all-optional members

When working with PATCH HTTP Controllers I did notice how often parameters could be all optional, and that is desired to assert that provided payload comes with at least one of these optional values set. This happened to me when working with some UPDATE query as when passing [] I got a malformed SQL. To avoid calling the database the system requires the following boilerplate:
$binds = $this->query()->toArray()
    ?: throw new ControllerException(code: 400);

This code will get expensive as for each controller accepting all-optional parameters I will require to test for empty array, which can be avoided by introducing a new rule on the accept* methods, here an example on acceptQuery:
    public static function acceptQuery(): ArrayStringParameterInterface
    {
        return arrayString()
            ->withOptional(
                company_id: string(),
                ranch_id: string(),
            )
            ->withAtLeastOne();
    }

The final code at Controller doesn't need the check, neither the test:
$binds = $this->query()->toArray();

This is because on withQuery the system will process the rule considering withAtLeastOne().

Requires implementation at Parameters, ArrayType* and at Arguments for validation.

[x] Parameters
[ ] ArrayType*
[ ] Arguments


<EMAIL_ADDRESS>hash check failed

App:         naderi/cpdf
URL:         https://github.com/coherentgraphics/cpdf-binaries/raw/master/Windows64bit/cpdf.exe
First bytes: 4D 5A 90 00 03 00 00 00
Expected:    d9951f5b6c8a17835c5fd10b41cfd4c5915291fde171bc050b5a14900426dd96
Actual:      0160491b8a6108ebde415b58b398e5a7449c45de9643e9d99f6e030e797dec3e

@NSP-0123456 Thank you for the report. Fixed!


Wrong hash calculated for some PE files

I've found that some files - especially installers - fail to verify because the calculated hash doesn't match.
Here's the latest Firefox installer, for example:
https://ftp.mozilla.org/pub/firefox/releases/101.0/win64/en-US/Firefox Setup 101.0.exe
Its embedded SHA-256 hash is DEF2F92C51E7DD222F480B9459C1E2E0A35C598453FC85EF74D9439D105AC853, but uthenticode::calculate_checksum returns 9338FBD1F37B5544CD759681C23A8343CBE4043CDB3A89A1818C909C81AFC6AC.
The problem is that pe_bits only contains 134,644 bytes at the time it's hashed, a tiny fraction of the EXE's total 56,014,584 bytes. It seems that enumerating the PE sections is not a reliable way to collect the data we need to hash, regardless of what Microsoft's documentation says.
The simpler approach of just hashing the file in order, skipping a few pieces, gives the correct result. For example, PeNet (Apache licensed) has an implementation like this:
https://github.com/secana/PeNet/blob/ec0bb60d62c9d44dc7d5648ac742f96de3a0a166/src/PeNet/Header/Authenticode/AuthenticodeInfo.cs#L155
If you want, I'd be happy to submit a PR along those lines, since it's work I need to do anyway.
(I'd also like to get uthenticode working with .msi and .cat files, but that's a separate issue)

Thanks for the report!
Yes, a PR would be greatly appreciated. I'd also be happy to review changes for MSI and CAT files in the future.

(I'd also like to get uthenticode working with .msi and .cat files, but that's a separate issue)

Why not throw in .cab files while at it? 😜
mtrojnar/osslsigncode has an msi.c and refers to microsoft/compoundfilereader for the heavy lifting of the compound file format. Since it's also MIT license, perhaps it's something that could be included as a compile time option all the while relying on the code from MS?
PS: for checking validity of assumptions etc, it's also worth looking into avast/authenticode-parser at times. It's been only around since 2021 and clearly has a different focus; and it's in C instead of C++. But it's also liberally licensed like µthenticode.

CAB support would also be welcome 🙂

Since it's also MIT license, perhaps it's something that could be included as a compile time option all the while relying on the code from MS?

Given that it's MIT licensed, I'd be okay with us vendoring it here, assuming it doesn't cause any regressions with our current ASan/valgrind/etc. checks in the testsuite.

I'm interested in making it work with .cat files.
Does any of you know some documentation where I can start with?

I'm only familiar with the stuff MS has here: https://learn.microsoft.com/en-us/windows-hardware/drivers/install/catalog-files
It sounds like they're essentially a detached signature over a bunch of digests of individual files. So the verification process should be pretty similar to what's already in the codebase, but with a detached signature rather than one baked into the PKCS#7 blob.