Cutting multiple folders causes the program to crash

2020-08-12 13:40:22.6151|ERROR|Files.App|Access is denied. (Exception from HRESULT: 0x80070005)|System.UnauthorizedAccessException: Access is denied. (Exception from HRESULT: 0x80070005)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
at Files.Interacts.Interaction.d__62.MoveNext() + 0x227
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
at Files.Interacts.Interaction.<>c__DisplayClass60_0.<b__0>d.MoveNext() + 0x816
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
at System.Runtime.CompilerServices.TaskAwaiter.GetResult() + 0xb
at Files.Interacts.Interaction.d__60.MoveNext() + 0xb1
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
at System.Runtime.CompilerServices.TaskAwaiter.GetResult() + 0xb
at Files.Interacts.Interaction.<PasteItem_ClickAsync>d__59.MoveNext() + 0xcf
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.AsyncMethodBuilderCore.<>c.b__7_0(Object) + 0x1e
at System.Action1.Invoke(T) + 0x28    at System.Threading.WinRTSynchronizationContext.Invoker.InvokeCore() + 0x33    at System.Runtime.InteropServices.McgMarshal.ThrowOnExternalCallFailed(Int32, RuntimeTypeHandle) + 0x21    at __Interop.ComCallHelpers.Call(__ComObject, RuntimeTypeHandle, Int32) + 0xb8    at __Interop.ForwardComStubs.Stub_10[TThis](__ComObject, Int32) + 0x24    at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object, UnhandledErrorDetectedEventArgs) + 0x3d --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21    at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object, UnhandledErrorDetectedEventArgs) + 0x75    at System.EventHandler1.Invoke(Object, TEventArgs) + 0x2e
at __Interop.Intrinsics.HasThisCall__23[TArg0](Object, IntPtr, Object, TArg0) + 0x36
at Files!+0x1279be0

Potential fix by #2269 ? @gave92

@duke7553 likely yes! Obviously#2269 will only avoids the app crash, if a file can't be accessed, #2269 won't change it


chore: Add missing executable permission for check-redis.sh

I added the missing executable permission by running chmod +x ./check-redis.sh
Before: -rw-r--r-- / 644
After:  -rwxr-xr-x / 755

@T-vK, please add a chart version bump with a proper ArtifacHub message.


What license?

I couldn't see a license agreement anywhere, what license is kube-aws release under?

Apologies, it's in CONTRIBUTING.md...

Could you reopen this and move it to LICENSE file, then there is a nice logo on the top right corner, like here: https://github.com/apprenda/kismatic?
Thanks!

Done 👍


Generating questions on summary and not source

  if qa_pairs_precomputed:
      qa_pairs_lists = qa_pairs_precomputed
  else:
      qa_pairs_lists = self._generate_qa_pairs(summaries)

Generating QA pairs from the summary. It should be source documents (reference summary)?

Hi! Thank you for your interest in our work.
In our approach we generate QA pairs from the model summaries as in the code and check their answerability using the source document.
Please check the below image from our paper.

Thanks for your reply. Yes, sorry for the confusion. I wanted to compare the method with qaeval which compares candidate summaries with reference summaries. I will close the issue.


Add more order of operation comparisons

As discussed with @rachitnigam
Previously all BinOps were grouped under a single precedence enum "Op."
Now, they are spread out and more fine grained. This should fix issues
regarding parentheses and order of operations.
 Can you separate them into a different PR or remove them entirely? Once done, I can re-review.

Sorry about that.
I reverted the formatting changes to all files without content-changes to them. But all of the the formatting changes in subset/pretty_print.rs were made through the rust-analyzers auto formatting with rustfmt. Is there a configuration I can use to properly reformat it? I could also revert those if needed

yeah, seems like this project doesn't have a .rustfmt file which describes the default formatting rules which is why rustfmt did this. I'm okay with committing some of them.

Made things nested, checks still fail due to other files' formatting from what I can tell, but should be good I think.

That's really weird. The commands run for the workflow are here: https://github.com/vegaluisjose/vast/blob/main/Makefile
Can you try running the rules in the Makefile and see if you can locally reproduce the errors? It might be the case that VSCode might be using a globally configured .rustfmt causing it to format things differently


Remove console.log calls

There are a few calls to console.log, presumably added during debugging when key rotation was introduced.
It didn't contain information which would be useful under normal circumstances (or any context to explain what the log entry was about), so I've removed them.

Landed here https://github.com/keycloak/keycloak-nodejs-auth-utils/commit/fd60e886d7b581fe87283a0a5644960c39898083, thanks!


Feature/320 camunda720

all in one PR for spring 6, camunda 7.20 and java 17

Codecov Report

Merging #338 (91b8bac) into master (b861e15) will increase coverage by 0.01%.
Report is 1 commits behind head on master.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@             Coverage Diff              @@
##             master     #338      +/-   ##
============================================
+ Coverage     61.88%   61.90%   +0.01%
  Complexity      548      548
============================================
  Files            94       94
  Lines          1994     1995       +1
  Branches         24       24
============================================
+ Hits           1234     1235       +1
  Misses          744      744
  Partials         16       16

Files
Coverage Δ

...da/community/mockito/process/CallActivityMock.java
91.52% <100.00%> (+0.14%)
:arrow_up:


If the map has been updated and that update moves buildings, a replay will crash the player to desktop

Need to better handle this error case. Maybe a pop-up prompt should be shown.

Was also fixed in 9942e15b0d745eed17ee805c395f579fc543fa03.


[Binance Chain] Add Staking API

Along launching with Binance Smart Chain(https://www.binance.org/en/smartChain), Binance chain will have 3 new messages/orders for staking

cosmos-sdk/MsgSideChainDelegate
cosmos-sdk/MsgSideChainRedelegate
cosmos-sdk/MsgSideChainUndelegate

We need to implement Staking api for the apps

endpoint: https://testnet-api.binance.org/
docs is not ready, but there is a swagger file


Migrate SDK utilities from FE repos

Migrate SDK utilities from the bridge, explorer, and wallet repos.

i.e getBlock, fee calculation on explorer

[!NOTE]

Incoming from https://github.com/FuelLabs/fuels-ts/discussions/1537

Should we migrate all utilities from FE repos?


Intellisense shows loading indefinitely

Issue Type: Bug

Open a project in visual studio code(React).
2.Try intellisense suggestion before doing npm install (Without node modules)
3.You can find the intellisense working perfectly.
4.Do npm install (add node modules) then try intellisense.
5.You can see it stuck at the loading text. no matter how much time you give.

This one have same effect even with all extensions deisabled.
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:58:56.166Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8365U CPU @ 1.60GHz (8 x 1896)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.82GB (23.46GB free)

Process Argv
D:\react-workspace\brand-ui-ups\brand-app-ui-ansal

Screen Reader
no

VM
0%

Extensions: none

The same issue, and version info:
Version: 1.41.1 (user setup)
Commit: 26076a4de974ead31f97692a0d32f90d735645c0
Date: 2019-12-18T14:58:56.166Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.17763

Reopen the vscode will work correctly for minutes, and then Intellisense break down and always show   loading text when you hover any code.

@creeperyang
I think the issue is with the version of vs code, and this is how I resolved it.
I removed vs code completely. even deleted the files in the following locations
%userprofile%.vscode
%userprofile%\AppData\Roaming\Code
Then uninstalled VSCode.
Finally installed VSCode v 1.40.2
Also disabled auto update.
Now it works fine.

@creeperyang
I think the issue is with the version of vs code, and this is how I resolved it.
I removed vs code completely. even deleted the files in the following locations
%userprofile%.vscode
%userprofile%\AppData\Roaming\Code
Then uninstalled VSCode.
Finally installed VSCode v 1.40.2
Also disabled auto update.
Now it works fine.

Thanks, I will try it.

I just uninstall v1.41.1 and turn back to v1.40.2, and everything get back to normal.

I have the same issue, is there any other ways to solving

Same issue here. IntelliSense auto imports stopped working randomly while working on a project.
Windows 10.0.18363
Visual Studio Code 1.41.1
Rolling back to 1.40.2 fixed the issue.

@Ansal-Khan Is your code stored somewhere besides the c: drive?

@Ansal-Khan Is your code stored somewhere besides the c: drive?

My code resides in a separate disk.

If you delete the node_modules file , it will return to normal. I think the @type file of node_modules is not loaded normally by vscode1.4.1

Hi, I'm having the same issue. My project folder is on a separate drive, but the path to it has a symlink back to my C: drive for convenience. Could this be the source of the problem?
To be clear, I made it so that C:\Users\Me\Source links to E:\Users\Me\Source, which is where I actually store my source code for all my projects. I open my project folder with the symlink path that starts with C:

Duplicate of https://github.com/microsoft/TypeScript/issues/35512
To workaround this, upgrade your workspace to use typescript@next by installing this extension


additional_security_group_rules parameter doesn't accept a list

Describe the Bug
Giving a list as a value to the additional_security_group_rules parameter causes a fail.
Expected Behavior
It should accept a list and the module should run smoothly.
Steps to Reproduce
Steps to reproduce the behavior:

Give a list as a parameter to the additional_security_group_rules parameter and run the module, such as

additional_security_group_rules  = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]

Error log
╷
│ Error: Invalid function argument
│
│   on .terraform/modules/aws_security_group/normalize.tf line 15, in locals:
│   15:     key         = coalesce(lookup(rule, "key", null), "${k}[${i}]")
│
│ Invalid value for "inputMap" parameter: lookup() requires a map as the
│ first argument.
╵

Environment (please complete the following information):

Terraform v1.1.3

Solution
Give the parameter as a list of objects, such as:

additional_security_group_rules  = [{type = "ingress", from_port = 0, to_port = 6379, protocol = "tcp", cidr_blocks =  ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24"]}]

IMHO this is not a bug. Parameter is named additional_security_group_rules which means you have to put there list of security group rules not list of CIDRs.
Works as designed (and expected).


Update IndividualDisplacements URL to post-rename repo

See https://github.com/JuliaRegistries/General/pull/119126

I do not understand why Merging is blocked.

I opened this PR because https://github.com/JuliaRegistries/General#readme says
if you rename the repository so it has a new URL, make a PR to edit the URL stored in the registry for the old package name to point to the new URL ([example](https://github.com/JuliaRegistries/General/pull/40190/files)). This allows the old versions of the package under the previous name to continue to work.
and it seems desirable that previous name continues to work.
Did I do something wrong?

I do not understand why Merging is blocked.

That's the same for all PRs to General unless you happen to have merge authorization.

Did I do something wrong?

This PR looks fine to me but since it is a manual PR, automerge doesn't handle it and it needs to be merged by someone who does have merge authorization.

I do not understand why Merging is blocked.

That's the same for all PRs to General unless you happen to have merge authorization.

Did I do something wrong?

This PR looks fine to me but since it is a manual PR, automerge doesn't handle it and it needs to be merged by someone who does have merge authorization.

Makes sense now. Thanks for the clarification

Hi. Could this get merged?


chore: box incoming connections on windows

🤖 Generated by Copilot at b5dab31
Fix compilation error on Windows by pinning NamedPipe stream. This change modifies crates/rpc/ipc/src/server/mod.rs to use Box::pin and #[cfg(windows)] to make the Incoming struct compatible with the FutureDriver.

Codecov Report

Merging #2805 (675fe8e) into main (5039122) will decrease coverage by 55.89%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##             main    #2805       +/-   ##
===========================================
- Coverage   71.38%   15.49%   -55.89%
===========================================
  Files         505      505
  Lines       65825    65828        +3
===========================================
- Hits        46986    10197    -36789
- Misses      18839    55631    +36792

Flag
Coverage Δ

integration-tests
15.49% <0.00%> (-1.96%)
:arrow_down:

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

crates/rpc/ipc/src/server/mod.rs
0.00% <0.00%> (-76.59%)
:arrow_down:

... and 378 files with indirect coverage changes


failed to download motion dataset

Hi,
I am trying to download the latest version of the motion dataset using:
gsutil -m cp -r "gs://waymo_open_dataset_motion_v_1_1_0/uncompressed/  .
But I got the error:
NotFoundException: 404 gs://waymo_open_dataset_motion_v_1_1_0/uncompressed/tf_example/training/training_tfexample.tfrecord-00153-of-01000 does not exist.
CommandException: Some components of ./uncompressed/tf_example/training/training_tfexample.tfrecord-00153-of-01000 were not downloaded successfully. Please retry this download.
Could you tell me how can I resolve this issue?
Thanks,
Sen

Hi @SenZh1234,
I've checked manually and that specific file is in the bucket. Are you able to download the rest of the dataset?

Thanks for your reply. @nicomon24
It seems to be a problem with my network. I've got the dataset from someone else.


30 minutes of waiting for shards to be ready, falcon-40b-instruct

System Info
ghcr.io/huggingface/text-generation-inference:latest
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction

Running this on A6000 x 4 machine

model=tiiuae/falcon-40b-instruct #bigscience/bloom-560m
num_shard=4
volume=$PWD/data # share a volume with the Docker container to avoid downloading weights every run

docker run --gpus '"device=4,5,6,7"' --net=host --cpus=".6"  -m 30G --shm-size 1g -p 8080:80 -v $volume:/data ghcr.io/huggingface/text-generation-inference:latest --model-id $model --num-shard $num_shard --revision 1e7fdcc9f45d13704f3826e99937917e007cd975

Expected behavior
I'd expect it to load in a few minutes? I'm not sure tbh. I can see this in nvidia-smi
|    4   N/A  N/A   1907904      C   /opt/conda/bin/python3.9        17000MiB |
|    5   N/A  N/A   1907905      C   /opt/conda/bin/python3.9        17112MiB |
|    6   N/A  N/A   1907908      C   /opt/conda/bin/python3.9        17192MiB |
|    7   N/A  N/A   1907907      C   /opt/conda/bin/python3.9        17112MiB |

It continues to climb so that's promising.. but I'd love to know from others if 48GBx4 is enough for falcon-40b-instruct and how long does it take to load?

What the model already downloaded?
Also, what is the speed of your disk?
We load falcon-40b in ~1 minute usually in our setup.

Can we please get some hardware details here? :)
Even with model already downloaded and converted to safetensors on good-quality disk, we see about 4-5 minutes of boot time

30 seconds actually on a p4de.24xlarge:

sharded on 2xA100 80GB
disk maximum read/write throughput of 2.7 GB/s


Add -v/--version, -h/--help description edit

Adds a -v/--version option, which shows pax's version. (Solves #63) The version is the same as the version in pax.nimble.
Note: Currently only implemented in spec, not each subcommand, so will only work if used with no subcommand.
Rework -h/--help option, change help description for this option from "Show help message" to "show help message" to be consistent with other help descriptions.

Not sure why CI is failing, could be because I'm testing on nim 1.6.2

No idea either why it thinks it's a void. Either way, I think reading the version from the latest git tag is a better method, as I don't need to manually keep the package version up-to-date.

Apologies for pushing directly to your pr, but would this work?

Yeah that looks good! Only concern is building outside of a git repo clone, eg from GitHub release zips


When the leader election is enabled, set true in leaderElectionReleaseOnCancel

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
When the manager is terminated, the leader role replica should voluntarily steps down from its own leader role
so that backup role can take the leader role as soon as possible.
Which issue(s) this PR fixes:

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

NONE

/assign @trasc

/lgtm

When the manager is terminated, the leader role replica should voluntarily steps down from its own leader role
so that backup role can take the leader role as soon as possible.

If we didn't do that it feels like a bug more than just cleanup. Do you know how much delay it was causing? Maybe we could consider this for cherry-picking. WDYT @tenzen-y ?

So, I think that cherry-picking this to release-0.8 could bring production env more stability.

/remove-kind-cleanup

/kind bug

/remove-kind cleanup

/release-note-edit
Fix a bug that could delay the election of a new leader in the Kueue with multiple replicas env.

/cherry-pick release-0.8


Add images for object store setup

Add here your WMS ID
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

Hi @garimasharma024,
Please solve the conflicts

Hi @garimasharma024,
Please solve the conflicts

Done


Move and Size Head Arrow of a multi-tail arrow

As a User, I want to move and resize the head of a multi-tail arrow so that I can accurately represent the product in my synthesis tree.
Acceptance Criteria:

When the user hovers over the arrow with select or arrow tools (not erase tool), the cursor should change accordingly: hover on head line - up/down arrow; hover on right point of head - left/right arrow.

When the user is hovering on the right corner of the head, clicking and dragging to the left or right changes the width of the head.
2.1. During and after moving, the whole arrow should be highlighted.
2.2. Dragging to the right is not limited.
2.3. Dragging to the left is limited to [minimum head length]

When the user is hovering over the line of the head (not the right corner), clicking and dragging up and down changes the position of the head.
3.1. During and after moving, the whole arrow should be highlighted.
3.2. The head can be moved along the spine, with one radius of the corner curve being the limitation on both sides.

3.3. Any movement above/bellow these lines results in no movement.

Selection and hover highlighting of the whole Multi-Tailed Arrow is out of scope for this task - it will be implemented here - https://github.com/epam/ketcher/issues/5057

Exploratory testing is completed on the RC Remote environment:

Ketcher [Version 2.25.0-rc.1] Build at 2024-09-03; 15:37:29
Indigo Version 1.24.0-rc.1.0-gb8f099851-x86_64-linux-gnu-11.2.1
Windows 11 Enterprise
Chrome Version 128.0.6613.120 (Official Build) (64-bit)

The following checks were performed, and tests are updated:

The head arrow of Multi-Tailed Arrow added from KET file or added by Tool can be moved up and down, but not closer than 0.15 from top and bottom tails:

The size of head arrow of Multi-Tailed Arrow added from KET file or added by Tool can be increased to right and decreased to left (0.4 from spine):

Checks above were performed for one/several Multi-Tailed Arrows on Canvas with/without chemical elements;
Undo/Redo actions for moving/resizing of head arrow of Multi-Tailed Arrows added by Tool or from KET file;
Copy-Paste and Cut-Paste actions for Multi-Tailed Arrows added by Tool or from KET file after moving/resizing of head;
Movement of Multi-Tailed Arrows added by Tool or from KET file after moving/resizing of head;


Dependency conflict around sinon, sinon-chai

RxJS version: 5.0.0-beta.7
Code to reproduce: npm install
Expected behavior: Install
Actual behavior:
Peer conflict between sinon, sinon-chai
Additional information:
Digging a bit, my system installed packages of these versions w/ the constraints below. It looks like RxJS wants sinon 2+ and sinon-chai 2.8+, but its sinon-chai 2.8 wants a peer dependency of sinon < 2.
In local testing, I downgraded sinon-chai to ^1.0.0, which seemed to have worked.
RxJS:
    "sinon": "^2.0.0-pre",
    "sinon-chai": "^2.8.0",

Sinon
     "version": "2.0.0-pre",

Sinon-chai
     "version": "2.8.0",
    "peerDependencies": {
        "sinon": ">=1.4.0 <2"
    devDependencies": {
        "sinon": "^1.10.3"

To overcome browser testing support, sinon 2.0 should be used. Once official release of sinon 2.0 will published sinon-chai will bump up its peerdependency as well(https://github.com/domenic/sinon-chai/issues/73).

Forgot to add last comment - meanwhile, please use npm@3 if available. npm 3 loosens requirement of peerdependency, works with current configuration in general.

@kwonoj We're in the process of updating these things (npm3, until about 2mo ago, had bugs that prevented it working for us). I think that was the issue -- we compiled w/ npm2, and so this broke on us.


Fixed default options not displaying for parameters without type hints.

Resolves #234
format_default() takes a new parameter is_annotated to better format the option.
_inject_types_to_docstring() now iterates through all signature parameters. Unchanged behaviour for parameters with type hints, but parameters without type hints now get their default value shown.
Also added tests with 4 functions trying out different variations with/without type hints, default values, and return types.
Works with both the "braces" and "commas" setting for typehints_defaults.
Before:

After:

Looks alright once the CI passes.

Any ideas on how to fix this?
tests/test_sphinx_autodoc_typehints.py:1131: error: "Config" has no attribute "typehints_defaults"
set_python_path()
app.config.master_doc = "without_complete_typehints"  # type: ignore # create flag
app.config.typehints_defaults = "comma"
app.build()

A similar syntax seems to work with the tests above this one...

You must add trailing # type: ignore on the line.

@gaborbernat does this look okay?


Negative prompt

Is there anyway for me to add a negative prompt for " "? Some models cause the the bot to end sentences with this.

Closing out, please reopen if it's still happening.


KNOX-1849 - Start the Java process with 'exec' when running the app in the foreground

What changes were proposed in this pull request?
Running the Knox Gateway and the LDAP server using exec in the foreground gives us the opportunity to execute the underlying Java command that completely replaces the current bash process (there won't be any parent process which makes graceful stops easier by clients).
Note: without this change CM will consider the Gateway stopped even if the JAva process is still running in the background.
How was this patch tested?
Running Knox Gateway and the LDAP server in the foreground after I built the project.

@risdenk @rlevas @moresandeep
Could you please review this change? Thanks!

Unit tests passed locally:
$ mvn clean -T1C verify -Prelease,package
...
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 20:27 min (Wall Clock)
[INFO] Finished at: 2019-04-05T10:03:42+02:00
[INFO] Final Memory: 359M/2471M
[INFO] ------------------------------------------------------------------------

@risdenk Could you please re-trigger the job?


:sparkles:Add dark mode logo option

This PR adds the option to specify a dark mode logo in the params.toml file as mentioned in the discussion. If no dark mode logo is specified, the default logo is used for both modes. If a dark mode logo is specified, the default logo is used for light mode and the dark mode logo is used for dark mode.
Thanks for creating such an amazing theme!

Thanks for the PR and sorry it's taken me so long to review it. I'm happy to include this feature as it's one that has been requested a few times. I'll make a few documentation additions and then merge it in.


Don't include Babel by default

This should be an option, and default to false.

:-1: I like it being default, I don't see why not.

:-1:. Similar to @arthurvr I don't see why.
For small modules i agree babel should not be the default, but, this generator is more for use in bigger modules. smaller few-lines modules just use generator-nm and you should assume something scalable by default.
The babel community is awesome, @thejameskyle was even keen enough to contribute a pull request, babel is really stable, and I didn't see anybody else who didn't like it.
Also, Babel just adds little overhead to the gulp setup and it's back-wards compatible with people that use ES5.

This should be an option,

It already is optional but just defaults to true. I don't really think it should've been an option eighter.

:-1: on not having it as default but it should be optional.

Don't take my -1 bad @SBoudrias. PS: have you had a chance to look at my environment pr yet?

I'm sorry to feel very strongly about having it off by default.
My main issue is that ES2015 is not currently the supported language for node. ES5 is the official language. I think the only reasonable thing to do if we want to be the Node.js module generator is to stick with the officially supported language by Node.js. When v8 is updated and Node support es2015 out of the box, then it'll be the right time to update our generator. But for now, let's stick with the official language - and what we know works well.
Babel might be a pretty stable tool, but honestly just look at the number of issues we've had this week. We can fix these issues, but truth is supporting a precompilation step before being able to run tests or deploy a module creates a more error prone workflow.
When I say not support it by default, I doesn't mean hide the feature behind a --babel flag. Let's add a prompt asking for the user decision. But let's not silently add it as the default option.
I still believe Babel and es2015 are not ready for global deployment. The tools and the platform ecosystem are not 100% there yet. And until it is, it is not reasonable for us to make it our default scaffolding.

Hey @SBoudrias really sorry you've been having a lot of issues related to Babel. I'd like to help out in any way that I can.

I still believe Babel and es2015 are not ready for global deployment. The tools and the platform ecosystem are not 100% there yet. And until it is, it is not reasonable for us to make it our default scaffolding.

This is something we've worked very hard on over the past year, and at this point I'm going to have to disagree that the ecosystem isn't there. Linting, testing, code coverage, deploying and all the rest are solved issues.
Certain things need a little bit of special attention, and in some cases out-of-date tools need to be swapped out for newer ones. But there's very little that has not already been considered. We just need to do a better job of communicating the work that's already been done.
I can contribute a couple of PRs related to this if you'd like. Even if you want to turn Babel off by default, I think there should be a rock solid setup here out of the box.

Babel might be a pretty stable tool, but honestly just look at the number of issues we've had this week.

None of the issues last week was related to the stability of babel and its tooling. the problem was that it wasn't integrated properly.

My main issue is that ES2015 is not currently the supported language for node. ES5 is the official language. I think the only reasonable thing to do if we want to be the Node.js module generator is to stick with the officially supported language by Node.js.

the official Node generator should stick to what the community likes, not to what the language supports. Just like the webapp generator adds Sass, which isn't supported by browsers. Node will support these features in the future.
This isn't about having a precompiler on (or at least not a classical case of that), because the it only adds features which will be native implemented in a few months.

Moved the option to a prompt. It is still on by default, but that choice is now clearly stated.
User won't end up with a es2015 project by surprise.


data.replaceAll is not a function

const replaceAttrMatcher = new RegExp("{=$}", "gi")
data = data.replaceAll(replaceAttrMatcher）
Regular is already a global mode,No need to use replaceAll，And replaceAll support is not good

Seems like that happens with NodeJS before version 15, due to not implemented .replaceAll() function.
Will use .replace() for broader support.


Fix SelectWidget vocabulary load on second component mount

See #2655

Ha found another bug within ArrayWidget :smile: video on the way.

https://user-images.githubusercontent.com/323385/133118481-47fb14d5-dd78-45de-942c-da80ef87061d.mp4

Ok, the search one seems deeper:
https://github.com/plone/volto/blob/ec386e1b276138e6be0730b2aea1f798af2a2a4c/src/components/manage/Widgets/ArrayWidget.jsx#L289-L294
@sneridagh I think we can merge this for now, and treat the search issue in a separate issue/PR

Mmmm I already experienced this on duplicated widgets for the same vocabulary, so great to have a fix! However, never experienced that last one... I even fix a search issue lately, I thought that is behaving well (eg. the languages selector in the control panel.
Might help to know exactly the type of field/vocab and the instantiation and the resultant widget.

@avoinea do you need this in 13?

@sneridagh Nope, already on 14 alpha. #yolo :)


Update visual tests.

The visual tests have been reformatted to conform with the style guide (and to an extent, my own personal aesthetic), and outdated components have been updated. The tests now match the relevant example in the docs as closely as possible.

<3


PathDet Dataset Curation - Comma2k19

Description
Parse the Comma2k19 Dataset to create a dataset comprising input images in PNG format and a single drivable path as the ground truth, derived from the recorded poses of the ego-vehicle projected into the image plane.
Please create a script called process_comma2k19.py in this folder (https://github.com/autowarefoundation/autoware.privately-owned-vehicles/tree/main/PathDet/create_path/Comma2k19) which is responsible for creating the ground truth and saving the data.
The ground truth drivable path information should be stored as a list keypoints defining the path in a JSON format. The drivable path keypoints should be sampled with a vertical (y image axis) displacement of 10 pixels to match the ground truth format used in other datasets.
Please ensure that keypoints are stored in relative coordinates, where the top left-most corner of the image is 0,0 and the bottom right-most corner of the image is 1,1, and all other coordinates are floating point values in the range (0,1) for x,y directions.
Please ensure ground truth images are stored in PNG format
Please also save a semantic drivable path mask which is drawn upon the input RGB image alongside the ground truth drivable area for data auditing purposes as well as a binary drivable path mask in PNG format, where drivable path pixels are assigned a value of 255, and non-drivable path pixels are assigned a value of 0.
Data Summary:
RGB image in PNG Format
Drivable path keypoints in JSON Format
Binary Drivable Path Mask in PNG format
Semantic Drivable Path Mask draw on top of RGB image in PNG format (not used during training, only for data auditing purposes)
Note
Please work on the path-det-dataset-curation branch

@siddas27 - I have created a new branch for this development called '21-pathdet-dataset-curation-comma2k19' - please fork this repo to your personal github, push your latest changes to the new branch on your github and then submit the pull request once completed for review. Thanks!


Pretty responses not being formatted properly

When I send requests the Pretty response isn't being properly formatted, there are no line breaks and no tabbing. Formatting is usually fine, but consistently broken today. I've restarted everything including the whole laptop.

VSCode details
Version: 1.95.0 (Universal)
Commit: 912bb683695358a54ae0c670461738984cbb5b95
Date: 2024-10-28T20:16:24.561Z
Electron: 32.2.1
ElectronBuildId: 10427718
Chromium: 128.0.6613.186
Node.js: 20.18.0
V8: 12.8.374.38-electron.0
OS: Darwin arm64 23.6.0

Postman plugin details

Please ignore, wrong repo! 🤦♀


Fix honeypot memory leak

The honeypot has a memory leak. That's not a pressing problem because it just gets restarted...
A brief investigation suggests it's related to some internal swarm things.

[ ] Open Issue with go-ipfs

The above screenshot depicts alloc_bytes.  However, there's no clear contender for a memory leak...


Replace GD_MONO_SCOPE_THREAD_ATTACH with GD_MONO_ENSURE_THREAD_ATTACH

It appears that mono can't currently handle a thread calling mono_thread_attach, mono_thread_detach and then mono_thread_attach again.
Doing so results in mono issuing a fatal error with the following log entry:

Assertion at debugger-agent.c:4144, condition `!tls' not met

I'm currently looking in to what it would take to fix that in mono -- tentative fix is posted here: https://github.com/tdaffin/mono/commit/fd8399a08b250461e9eb1d30959cb96547dced25
In order to handle the current state of mono I'm recommending switching to a strategy where godot does a single attach per thread and never attempts to detach.
To give this PR some context, I've been working on getting voxel_tools to work with the mono build (https://github.com/Zylann/godot_voxel/pull/193), and I ran into this problem while fixing the virtual bindings for VoxelGenerator and VoxelStream (https://github.com/tdaffin/godot_voxel/commit/2ce4b53ebdc7959a724d8cd0581c911f12deaeaf).
This PR was initially developed on the 3.2 stable branch with voxel_tools, but voxel_tools will not build on 4.0 so I can't test that the specific case is fixed on 4.0
In order to reproduce this problem on 3.2, use this project (https://github.com/tdaffin/voxel_tools_mono) with the fixed bindings for voxel_tools.

IINM, we actually patch this already in our Mono builds for the official Godot distribution (with a patch from Unity's Mono): https://github.com/godotengine/godot-mono-builds/blob/master/files/patches/mono-dbg-agent-clear-tls-instead-of-abort.diff
So if you build Mono using https://github.com/godotengine/godot-mono-builds, you might not need this workaround.

@akien-mga That patch does look like it would fix the issue, but I'm having trouble determining if the official godot distribution really uses that mono runtime.
I've downloaded the latest 3.2.2 for my work machine (windows), but I'm having trouble compiling the C# file using this version.
I do most of my development on linux, and I already have a working version of the official 3.2.2 on that machine so I will check on that platform after work.
Also, the main test case I have requires custom compilation due to use of voxel_tools so it is a little difficult to reproduce the behavior using the official distribution.
I might not be able to verify everything until next weekend...

We can't keep the threads attached, otherwise it will hang when unloading the domain during hotreload of assemblies.
At some point in the future we will switch to .NET Core version of Mono and use AssemblyLoadContext instead of domains. We won't need the patch after that.

I think I've seen that hang when returning to the editor after running the game when using this patch.
That probably means I should just go ahead and close this PR.
However, I'm still struggling with getting godot to run with the version of mono that I have compiled.
The problem is that when I build godot with:
scons -j6 target=release_debug tools=yes module_mono_enabled=yes copy_mono_root=yes mono_prefix="$HOME/mono-installs/desktop-linux-x86_64-release"
The GodotSharp/Mono/lib/mono/4.5 directory ends up empty, and if I try to run the version of godot I have built, I get this error:
tom@novo:~/dev/godot/godot$ ./bin/godot.x11.opt.tools.64.mono
Godot Engine v3.2.3.rc.mono.custom_build.c5fa7dcb6 - https://godotengine.org
OpenGL ES 3.0 Renderer: Mesa DRI Intel(R) HD Graphics 4600 (HSW GT2)

Mono: Log file is: '/home/tom/.local/share/godot/mono/mono_logs/2020_09_18 09.42.30 (10777).txt'
The assembly mscorlib.dll was not found or could not be loaded.
It should have been installed in the `/home/tom/dev/godot/godot/bin/GodotSharp/Mono/lib/mono/4.5/mscorlib.dll' directory.

I can see the reference dlls I built with python3 ./reference_assemblies.py install --mono-sources ~/dev/mono/ are there in my /home/tom/mono-installs/reference-assemblies/lib/mono/4.5-api directory, but even if I manually copy them over into godot I get this error when I run:
./bin/godot.x11.opt.tools.64.mono
Godot Engine v3.2.3.rc.mono.custom_build.c5fa7dcb6 - https://godotengine.org
OpenGL ES 3.0 Renderer: Mesa DRI Intel(R) HD Graphics 4600 (HSW GT2)

Mono: Log file is: '/home/tom/.local/share/godot/mono/mono_logs/2020_09_18 09.49.07 (12252).txt'
ERROR: _real_load_assembly_from: Failed to load assembly for image
   At: modules/mono/mono_gd/gd_mono_assembly.cpp:309.
The file /home/tom/dev/godot/godot/bin/GodotSharp/Mono/lib/mscorlib.dll is an invalid CIL image

I'm aware that this PR may be the wrong forum to discuss these problems, so let me know if that is the case.

Read my comment here about that error: https://github.com/godotengine/godot/issues/41652#issuecomment-694801736
You need to build the BCL rather than the reference assemblies. Then copy the contents of the folder where mscorlib.dll was created and paste them into $HOME/mono-installs/desktop-linux-x86_64-release/lib/mono/4.5/.

Thanks -- that worked!
Additionally, I also had to build godot with mono_static=yes, giving the full command line as:
scons -j6 target=release_debug tools=yes module_mono_enabled=yes copy_mono_root=yes mono_prefix="$HOME/mono-installs/desktop-linux-x86_64-release" mono_static=yes
Perhaps it would be a good idea to add the ability to specify '--target=x86_64' etc when running bcl.py, and have that automatically copy the dlls over?
I can look into implementing that and submit it a PR to voxel_tools_mono
I'm going to close this PR now though.


onLoad event not triggered

I wrote simply script based on select city example. I wanted dependent items list to be reloaded based on ajax query and next to select default value. This can be done by using onLoad method. But what i discovered is that dependent selectize object when using .load() method on it won't trigger onLoad event.
...
var xhr;
var select_state, $select_state;
var select_city, $select_city;
			$select_state = $('#select-state').selectize({
				onChange: function(value) {
					if (!value.length) return;
                    console.log('test');

                    select_city.disable();
					select_city.clearOptions();
					select_city.load(function(callback) {
						xhr && xhr.abort();
						xhr = $.ajax({
							url: 'http://www.corsproxy.com/api.sba.gov/geodata/primary_city_links_for_state_of/' + value + '.json',
							success: function(results) {
								select_city.enable();
								callback(results);
							},
							error: function() {
								callback();
							}
						})
					});
				}
			});

			$select_city = $('#select-city').selectize({
				valueField: 'name',
				labelField: 'name',
				searchField: ['name'],
                onLoad: function(value){
                    console.log(value);
                }
			});

			select_city  = $select_city[0].selectize;
			select_state = $select_state[0].selectize;

			select_city.disable();

...
If you read carefully I added to city example onLoad function definition to  $select_city which print value to console.
This won't work.

I think this may have been reintroduced with 0.11.2. I seem to not get any response from either onLoad or onInitialize

I have the same problem.
Had a quick look but am not sure why it happens. It needs more debugging.


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Closes: #1
Fixes #1

[x] Tests pass
[x] Appropriate changes to README are included in PR

@googlebot I signed it!

Closes: #1


Perform reads and modifications to Seed.Info in a concurrency safe way

How to categorize this PR?
/area control-plane
/kind bug
What this PR does / why we need it:

Ensures that patch operations on Seed.Info are always performed on a copy, and the result is written back to Seed.Info via a single pointer assignment.
Replaces usages of o.Seed.Info in shoot_control.go with the seed object that was used to initialize the operation. This is a cosmetic change that doesn't have a real functional impact (the 2 pointers are equal), but is important for consistency.
Replaces all naked usages of Seed.Info in production code that could be executed concurrently with the newly introduced methods GetInfo(), UpdateInfo(), and UpdateInfoStatus() that perform reading and modification in a concurrency safe way.
Introduces another new method SetInfo() and replaces all naked assignments to Seed.Info in test code with this method.
Renames Info to info, ensuring that it can only be read from and written to using the above methods.

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

This is a similar change for Seed.Info to what was introduced for Shoot.Info with #4459. You may want to a look at that PR as well for additional motivation, explanations, etc.
Unlike Shoot.Info, there are currently no concurrent reads / writes to Seed.Info that might lead to issues. So the purpose of this PR is mainly to avoid falling into the same trap with Seed.Info in the future, if for any reason such concurrent reads and writes are introduced. It also removes the inconsistency of having Seed.Info but Shoot.GetInfo().

Release note:
Improved handling of the seed resource in the seed controller to ensure that potential data races are avoided in the future.

/invite @timuthy

/squash

/assign


Magnitude constants representation type

As of today, mag_constant requires a value of long double (e.g., for pi). However, it might not be a good option for embedded projects. Should we change the requirement to just be a floating-point type? How to refactor pi to be embedded-friendly?

Make it possible for the mag_constant's value to be parametrized on Rep.
Have mp_units::pi carry std::numbers::pi_v rather than a floating-point value.

Why would those need to change?
There's no need to "expose Rep".
pi remains a constant in the magnitude expression,
and you only ever need to instantiate its value with a representation type
when its time to convert to another unit without pi.

Is long double available on all the embedded platforms? Said otherwise, is it possible to cross-compile the current pi definition to every platform?

Although, long double may appear due to common_magnitude_type being used in sudo_cast.

Good point @chiphogg. I forgot about get_value.
Is long double available on all the embedded platforms? Said otherwise, is it possible to cross-compile the current pi definition to every platform?

I don't actually know.  Empirically, I've never seen a bug report about long double causing problems, although we have had various other issue reports from embedded users, both Aurora-internal and external.  So I'm guessing we would have heard about it by now, but I can't be completely sure.


angular module implementation

I've spent a lot of time trying to figure out how to make this work on an angular module for an ionic app, any ideas?

same here.. does someone managed to implement this in an ionic ios app?

@romanrast i've got this working on an ionic android app:

is the code from the example. You don't need anything else.
but i think for troubleshooting on ios @jcesarmobile  is the right person to help you.

ok nice to here.
What did you do to implement the plugin?
I tried cordova plugin add https://github.com/macdonst/SpeechRecognitionPlugin on the root folder. But it doesn't seem to work for me.
Did you do anything else?

no, to install, open command line, navigate to your project's root folder and install the plugin:
cordova plugin add https://github.com/macdonst/SpeechRecognitionPlugin
Then include the example above in your index.html file
Note: in won't work on your browser, you have to run:
ionic platform add android
and
ionic build android
then install on your device (connected to any network)

This is what I have for just doing a basic recognition, It can be used as service but I just put this piece of code as a prototype of what I just started building:
angular.module('dct2.controllers', [])

.controller('DashCtrl', function($scope, $ionicPlatform, $timeout) {
    $scope.recognition = undefined;
    $scope.transcript = '';
    $scope.hyphenatedTranscript = '';
    $scope.h = new Hypher(HypherLanguages.spanish);

    $scope.hyphenate = function(transcript){
        //Check for multiple words (paragraph)
        transcript = transcript.split(' ');
        transcript = transcript.map(function(w){
            return $scope.h.hyphenate(w);
        });

        $scope.hyphenatedTranscript = transcript;
    };

    $ionicPlatform.ready(function(){
        $scope.recognition = new SpeechRecognition();
        $scope.recognition.lang = 'es-ES';//Set to spanish
        //TODO: Add multilingual support
        $scope.recognition.onresult = function(event) {
            if (event.results.length > 0) {
                $timeout(function(){
                    $scope.$apply(function(){
                        $scope.transcript = event.results[0][0].transcript;
                        $scope.hyphenate($scope.transcript);
                    });
                });
            }
        }
    });

});

I've read this blog post that uses this plugin on an ionic project, hope it helps
http://devgirl.org/2016/01/08/speaking-with-cordova/
BTW, I'm closing the issue as it's not a real issue with the plugin itself


[10.x] Fixes the validation issue in string validation when a numeric value is passed

This PR fixes the validation issue in string validation when a numeric value is passed. Issue details here

The current behaviour is the correct one.


Add Audit Event for MesSenMES3 added

Wait for @crystalmark to approve as well, but I'm ok with this.
Are you working on the assumption that the JSON conversion will be happening in another PR? (i.e. by @GeorgeColdham ?)

Yeah, I made the assumption the actual conversion would be done as part of the audit tickets.

I was told there would need to be a 3 amigos for this. I've done the same changes locally.

@GeorgeColdham Should I add anything or can I just wait for your PR to be merged?


Gertrude's Cat - Gertrude area needs to include outer yard area of house as well to account for Gertrude wandering

Currently the player attempts to talk to Gertrude, continues one time, then walks into the house, over and over while gertrude is outside.

Widened area to account for front and side yard - needs testing


Context Error

It is always displaying context error , dont know why
but the downloaded project CMPToast is running all fine.
java.lang.RuntimeException: Context Error
at multiplatform.network.cmptoast.AppContext.get$cmpToast_release(Platform.android.kt:14)
at multiplatform.network.cmptoast.Platform_androidKt.showToast-EKKmcho(Platform.android.kt:31)
at multiplatform.network.cmptoast.Platform_androidKt.showToast-EKKmcho$default(Platform.android.kt:18)
at screens.HomeScreenKt.HorizontalCardItem$lambda$8$lambda$7(HomeScreen.kt:219)

I faced the same issue and here is the temporary fix for that during wait for official fix.
For Application error, add: tools:replace="android:name" into your Android manifest at  level.
For Context error, add: multiplatform.network.cmptoast.AppContext.apply { set(applicationContext) } in to your Application class
Hope this help.

Thanks @onstonboy it solved the errors that I had on android device to run the app.
Thanks a lot.
Hope they bring an official fix for this soon.

@onstonboy @VISHNUONE8 Please try version 1.0.1

@mayur-ctn
The same error occurred even with version 1.0.1

@sobaya-0141 Ok I will add the fix for this in new version

I believe the issue will be resolved if you include the following explanation in the README file: add: multiplatform.network.cmptoast.AppContext.apply { set(applicationContext) } in to your Application class.

Added multiplatform.network.cmptoast.AppContext.apply { set(applicationContext) }  lin e in App class but still getting crash in Android 14

@puja55 Which version are you using?


opam files jump around when running opam monorepo pull

Hi,
When the duniverse folder is checked in, one can see the following interaction:
(clean git status)
% git rm -rf duniverse/
% opam monorepo pull
% git status
... some opam files are modified ...
% dune runtest
% git status
... all clean ...

It'd be great not to have to run tests before committing the duniverse folder again.
For workflows where the folder is not commited this is more innocuous, but it'd be interesting to see where this change comes from.

I'm not sure we should encourage workflows where we commit the duniverse, in particular since even RWO will probably move away from it.
My guess is that it comes from repos with opam file generation enabled in dune and out of sync opam files.


Using toothpick in libraries

I'm trying to use toothpick in a library project (no toothpick classes/objects exposed to the clients of the library). But I faced a problem that if a client of the library uses preventMultipleRootScopes configuration it will fail because the library opens new root scope. Also, we can't set isolated toothpick configuration in the library and in the client project because ConfigurationHolder.configuration field is static. How can I use multiple toothpicks with different configuration at the same time? Is it intended to use tootpick in this way?

TP is not intended to be running with multiple configuration, this doesn't make sense at all. TP uses static fields to ensure there is a single configuration.
It's not normal on Android to open multiple root scopes. The option is there to be more permissive in java, but for android the scope tree is unique and start with the app scope as a root. There is no use case for multiple root scopes on android and multiple root scopes are an indication of something going bad when opening scopes.
If you are ok with this answer, please close the issue. If you think there is a legitimate use case for opening multiple root scopes, let us know but it would be really really really surprising.

To be a little more precise on this:
when you create a scope you do:
TP.openScopes(app, activity);
//install modules, etc.

when you use a scope, not creating it but just using it, you do:
TP.openScope(activity);
//use the scope

openscope is a nice method has it returns a scope, creating it if needed. But it means that you can actually create scopes when you want to use scopes. A very common use case it to use openscope in a background thread, and this gets executed right after you activity was destroyed.
In this case, openscope will not return the activity scope that was created under the app scope (because the activity got destroyed and its scope got closed), but it will return a new root scope starting at the activity level.
To detect such issues, we have the configuration to detect multiple root scopes. This will help you to detect where your code is using TP wrong.
TP fully supports concurrent injection, but we have to take care on Android of not using an object after it died in its lifecycle.

Let's say I have a completely separated java library and I want to use toothpick to manage its internal dependencies. In the library I call Toothpick.setConfiguration and open scope using Toothpick.openScopes(LIBRARY_SCOPE). Then I want to use this library in an Android app. My app also calls Toothpick.setConfiguration with own configuration and opens own root Application scope.
As things stand now, we can't use this library if the client of the library (Android app) also use toothpick. It will work fine if Android app will not use toothpick. I can also face the same problem if I decide to use toothpick internally in the second library and use this two libraries simultaneously.
I can partially resolve the problem by disabling multiple root scopes check, but this check is really helpful and I don't want to accidendtely miss opening new root scope in the Android app.

@lukaville , your libs should not play with the configuration of TP. Only one point in the app should do that, and it should be done in App's onCreate method.
Regarding Toothpick.openScopes(LIBRARY_SCOPE), you are creating a root scope by doing this. I am wondering why you need this in your lib. If you lib components inject something, you could consider using the app scope, or passing them the scope they should use, or using a standard scope. For instance you can provide a component in your lib that is a @Singleton and then make it inject stuff. This will be injected using the app scope, and your app can provide the necessary bindings into the app scope when it creates it (in app's onCreate method as well).
I would really discourage you to open a scope in a lib. If you really really really want to do so, it means your app should create its appscope under the lib's scope. But this creates quite a semantic coupling indeed.

I want to use this library in two apps: one app use toothpick and the second one doesn't (it uses dagger2). In my library, I have a few internal components that I want to inject into each other and toothpick does a great job of dealing with it. I don't want to force the clients of my library to use toothpick. I think there should be a possibility to use multiple isolated toothpicks with different configurations and with own root scopes.

I would not use dependency injection in a standalone library, even less a standalone library. I don't know any lib doing that, and probably for good reasons... ;)

@stephanenicolas
Wait, but how is a library any different from an application? DI is there to help deal with complexity, so if the library is complex, it can benefit from DI.
It's not uncommon to extract a library with some common behavior or even a part of UI and reuse it across multiple applications. I know examples where these extracted libraries are very complex and using DI there makes a big difference.
Using TP in such a library should be an implementation detail, hidden from the clients of the library. E.g. the client should not have to somehow pass root scope name into the library in order to pass the multiple scope roots check (in case the client uses TP as well), let alone do any work on satisfying some dependencies used inside the library.
So I think this is a completely valid feature request.
I can imagine a solution could be to add Toothpick.createScopeTree() method and move the scope management including openScope method and multiple root checking from statics into ScopeTree objects. The scope trees would be completely independent.

A library is only different because you want developers to use it in any
app. And good libs to put too much constraint on your code, like
inheritance, or forcing you to use DI.
It's pretty simple for a lib that doesn't use DI to wrap it in DI, write
providers to get the new instances of the classes of the lib. That's
basically it.
I would probably never pick a lib that forces me to use a given DI
framework or another. I would simply discard it.
If it's completely hidden inside the lib, why not, but I don't think it's a
very great design for a lib. Usually you try to avoid this and have simple
libs, with few dependencies. It also avoids conflicts when people use the
same lib but with a different version.
2018-03-14 9:32 GMT-07:00 pshmakov<EMAIL_ADDRESS>
@stephanenicolas https://github.com/stephanenicolas
Wait, but how is a library any different from an application? DI is there
to help deal with complexity, so if the library is complex, it can benefit
from DI.
It's not uncommon to extract a library with some common behavior or even a
part of UI and reuse it across multiple applications. I know examples where
these extracted libraries are very complex and using DI there makes a big
difference.
Using TP in such a library should be an implementation detail, hidden from
the clients of the library. E.g. the client should not have to somehow pass
root scope name into the library in order to pass the multiple scope roots
check (in case the client uses TP as well), let alone do any work on
satisfying some dependencies used inside the library.
So I think this is a completely valid feature request.
I can imagine a solution could be to add Toothpick.createScopeTree()
method and move the scope management including openScope method and
multiple root checking from statics into ScopeTree objects. The scope trees
would be completely independent.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stephanenicolas/toothpick/issues/286#issuecomment-373087469,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABv33bf4V9vStyhKINeF03cEIBll5r03ks5teUYogaJpZM4SaDCo
.

@stephanenicolas

Usually you try to avoid this and have simple libs, with few dependencies.

But what if you can't?
Here's an example:
https://play.google.com/store/apps/details?id=com.yandex.browser
https://play.google.com/store/apps/details?id=com.yandex.zen
https://play.google.com/store/apps/details?id=com.yandex.launcher
In all 3 apps you can find a kind of news feed. It's more or less the same thing backed by the same library. And the library is very complicated, believe me :) It's essential to have in there a good level of modularity and testability, which is hard to achieve without DI. There is no option for the client apps to not use the library, and enforcing the same DI framework on all client apps is not an option either.
This situation is not exceptional in my experience, big companies tend to put effort in making features isolated and reusable across several apps they develop.
What would be your approach to such situation?

Any one knows why this completely valid feature request was closed? At my company we are now struggling because of this. We have a number of complex libraries that should be shared across the company and they are using TP. TP is not exposed outside of the libraries. Some projects use TP and there are no problems for them as they are just passing root scope to these libraries. But some projects use Dagger 2, and they also have to depend on TP so they can just open empty root scope. It's a very very big con of the TP so we are now considering migrating our TP projects to Dagger 2.

@arkivanov yes, let's just use dagger :)

Looks like TP 2.0 is still static 😑

Yeap, we didn't work on this at all in TP2. Our effort was mostly focused on build times and incremental annotation processing.
Personnally, I don't wanna be a blocker for features that I don't really want to see in TP. TP also belongs to his community and for sure as main contributors we don't see all the requirements and possible usages of "our" lib. So, feel free to submit a PR if you need such a feature.
One thing I was wondering when reading this thread is, why don't you just shade TP jars in your lib: make it so that all TP packages are renamed at build time. You would then get a private TP with it's private scope that wouldn't interfere with what the libs users would be doing ?


Safety Certification

Good morning!
I have worked with Zephyr before and it is hands down one of my favourite RTOSes out there.
I was looking to see if there was any sort of safety certification that you have for functional safety?
I saw this article that was written in 2019 that spoke briefly about the move that was being made to be certified (https://www.zephyrproject.org/zephyr-project-rtos-first-functional-safety-certification-submission-for-an-open-source-real-time-operating-system/) I was wondering if anything came of it, and where I can read more about your certification levels etc.
Thank you! You folks are amazing!

This article published on 09/11/2021 states:

The second long-term support version will be submitted for IEC 61508 certification in the next year or two.

@eden-desta, just an FYI, but we are working on establishing a Safety Working Group that shall be open to all interested individuals who wish to advance the Zephyr codebase to a safety auditable state. The scope of this working group is still being debated. I'll update this item after resolution.

Hi @AmberHibberd!!! Is the path to a qualification according to 26262 and/or 61508 still a hot WIP? Any advancement that can be publicly shared?

Hi @AmberHibberd & @dleach02, are there any updates regarding the advances toward ISO26262?


Update paragraph format to match style

This paragraphs has the wrong format when viewed in Microsoft Learn.

@jangelfdez - Thanks for the formatting fix!

@davidsmatlak - Is this content generated or can we merge this change? Thanks.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Court72 This change can be merged. Thanks for checking about whether it's generated content.

#sign-off


Add auto-build for commits

Which should create executables for windows, linux, mac os, (android)

I don't now how does that auto_apk works, sometimes works, sometimes not.


use relative header

so that it works in both cocoapods and other cases where source is copied e.g. submodules

SAMKeychain is designed to be used as a framework and framework umbrella headers are supposed to use full module imports. I don't think this is something we should change.

@calebd i think its resonable to be able to add this as a submodule. and this solution does work for a framework as well. also, the readme says otherwise "To manually add to your project:
Add Security.framework to your target
Add SAMKeychain.h, SAMKeychain.m, SAMKeychainQuery.h, and SAMKeychainQuery.m to your project."

Those instructions are outdated and should be removed. It is intended to be used as a framework. If you'd like to use it as a submodule, I'd recommend adding the SAMKeychain project as a subproject and then adding the target for your platform as a build dependency for your target.


set solmate settings with mqtt

          > Is there a possibility to add an adjustment for {client.set_min_injection(200)} , {client.set_max_injection(800)} and minimal battery load - sorry I am not capable to write this.

@lukashunziker99 I've running my custom code, because i want to dynamically set min, max and battery_percentage with mqtt.
the sol2mqtt.py script proxys the values to mqtt and subscribes from mqtt,
i'm using node-red integration in home-assistant for changeing the mqtt values.
config-data flows as follows:

node-red function (checks grid values)
** if power flows to grid => load battery (min, max = 0, battery percentage = 95)
** if power flows from grid => use battery (battery_percentage = 20, min, max depends on grid usage between 50-200 watts)

=> if something changed => write to "subscribe_topics" MQTT

sol2mqtt.py subscribed "subscribe_topics" from MQTT => writes back with official "solclient" (solmate_sdk)

It's working, long-time tested and robust although solmate api is not very stable and the script needs resets every 1-2 hours.
maybe @mmattel could integrate this and use some of my code after ensureing your coding standards.
i could also do some tests in my HAOS installation if you want?
just need the https://github.com/eet-energy/solmate-sdk running in your env, then you can use this script.
(
https://github.com/josy1024/eet2mqtt/blob/main/sol2mqtt.py
searchfor "subscribe_topics" and "mqtt2sol":

subscribe_topics = ["eet/solmate/0/set/user_maximum_injection", "eet/solmate/0/set/user_minimum_injection", "eet/solmate/0/set/user_minimum_battery_percentage"]

def mqtt2sol(topic, received_message):
    print(f"mqtt2sol: Received message on topic solclient.set {topic}: {received_message}")
    if "user_maximum_injection" in topic:
        solclient.set_max_injection(int(received_message))
    elif "user_minimum_injection" in topic:
        solclient.set_min_injection(int(received_message))
    elif "user_minimum_battery_percentage" in topic:
        solclient.set_min_battery_percentage(int(received_message))

Originally posted by @josy1024 in https://github.com/mmattel/EET-Solmate/issues/25#issuecomment-1968217511

@josy1024 I am working on integrating the three settings as changable values via
HA -> MQTT -> my-code --> Solmate. This takes a bit, pls be patient 😅

Done via: https://github.com/mmattel/EET-Solmate/releases/tag/v5.0.0 😃


lifecycle postStart hook may run before the service starts

What happened?
Kubernetes sends the postStart event immediately after a Container is started, so the postStart command may run even before the service is ready.
Pulling models before ollama serve will result in an error:
Error: could not connect to ollama app, is it running?

Chart version
0.58.0
Kubernetes version
1.27.16
Kubernetes distribution
k8s
Relevant log output
No response
Your values.yaml
No response
Additional context
No response

Hi @liangjs
Thanks for the issue, we will add a "loop check" to ensure Ollama is running before pulling models.


Workflow when testing plugins?

I was trying my hand at writing a plugin and found that testing it was rather tedious:
1.) Open Ghidra and Install Extension to load the plugin
2.) Reload Ghidra
3.) Open Ghidra and test stuff
4.) Uninstall plugin and close Ghidra
5.) Modify code, export as a "module extension" from Eclipse
6.) Go back to step 1
Is there some trick I've missed here? Ideally I would like to not have to reload Ghidra twice for every time I modify the plugin

You should only ever need to export to an extension when you are ready to distribute the final product.

watching a streamer developing a Ghidra extension in a non-Eclipse env, cmd line building, launching Ghidra, installing extension.... just to test a minor change was disheartening.

They shouldn't be developing an extension, they should be developing a module.  No need to go through the extension installation process during development.  We also now support running out of development mode when Eclipse isn't present (4e0fa990fb6a090937719250c67ab52e3e1ea829).  It relys on gradle having done the compilation (gradle jar)

watching a streamer developing a Ghidra extension in a non-Eclipse env, cmd line building, launching Ghidra, installing extension.... just to test a minor change was disheartening.

I have never used eclipse (its trash) and I still don't have to do this. As long as everything is in the classpath and the GhidraClassLoader are used everything should work.

Also nice of you to just watch them suffer :p

Heh.  I was the one doing the suffering, they were doing their thing, probably not knowing better.  Also, I'm guessing for streamers the journey is more important than the destination, and backseat driving is probably frowned on.  :smile:

I couldn't get it to play nice with Jython (maybe my PyDev is too new?) but after disabling Python I was able to get it to just run with the debug button, nice!

watching a streamer developing a Ghidra extension in a non-Eclipse env, cmd line building, launching Ghidra, installing extension.... just to test a minor change was disheartening.

You didn't see how many times I exported the extension and uninstalled/reinstalled it in Ghidra :)


fix checkstyle about usecase of favoriteRecipe

fix checkstyle about usecase of favoriteRecipe

The JavaDoc for the FavoriteRecipe use case is well-structured and provides a clear understanding of the functionality across all methods. For example, the JavaDoc for methods like getUsername clearly explains that it retrieves the username of the current user, providing a concise and precise description of its purpose. Similarly, the documentation for setFavoriteRecipes explicitly states that it assigns a list of favorite recipes to a user, ensuring that its functionality is well understood.


statistics: write qps in query stats is inconsistent with Prometheus

Bug Report

What version of TiKV are you using?
master
Steps to reproduce
sysbench test
What did you expect?
consistent
What did happened?

How about CPUs？

It's strange about apply CPU.


dismiss listener (Cancelable listener)

How to check if i cancel a dialog ?

when you call show() a Dialog object is returned, so you can call isShowing() to check if the dialog is still visible. If it returns false, then it is cancelled.
Example:
Dialog dialog = new Lovely*Dialog(context) ... . show();  if (dialog.isShowing){ dialog.dismiss() }else{ // It is already cancelled }


[Bug iOS] SetNativeControl Null Exception on Xamarin.forms 4.0

Hey,
We got a null exception when disposing the map.
The code is trying to reset a new map, when disposing the OldElement.
I'm on Xamarin.forms 4.0.

There is the PR #101

Thanks, makes sense.


Multiple completions support in ChatOpenAI

Feature request
Multiple Completions support would enable users to receive multiple responses or variations from the model for a given prompt. This feature would provide greater flexibility and allow users to explore different possibilities or perspectives in their conversations.
Allowing users to specify the number of completions they desire, it would enhance the richness and diversity of the generated responses. Users could gain a deeper understanding of different potential outcomes or receive alternative suggestions.
Multiple Completions support would be particularly valuable in scenarios where users are seeking creative ideas, exploring different options, or generating diverse responses for analysis. It would enable users to generate a range of potential answers, facilitating more comprehensive and robust conversations.
I believe that the implementation of Multiple Completions support in ChatOpenAI would greatly enhance the user experience and provide increased utility across a wide range of applications.
Please correct me and let me know if this feature is already available. Also in this case, please let me know how to access the same.
Motivation
I am building a framework that uses the multiple completions feature but I am not able to find this feature in ChatOpenAI.that
Your contribution
I will try to help the community to the best of my knowledge.

Thanks for the response. But this has not been resolved yet. Even after passing the n value greater than 1, I am getting only a single response from the LLM.
Please fix this.

Thanks for the response. But this has not been resolved yet. Even after passing the n value greater than 1, I am getting only a single response from the LLM.
Please fix this.

Thanks for the response. But this has not been resolved yet. Even after passing the n value greater than 1, I am getting only a single response from the LLM.
Please fix this.

Can please someone help me in this context?


DELETE /expenses/all doesnt update the account balance

add testcases too for the bug (include failing test)

fixed in https://github.com/gitsetgopack/MoneyManager/pull/42


Support error reporting via exceptions

Fixes issue #338

@lkorenc, @xlauko, I addressed the comments and simplified the implementation of error_stream whose only job is to throw the exception in dtor. Having template and trigger flag was probably making it look complex.


feature request: Support Github Container Registry

There are some other issues related to using a different container registry, or even self-hosting one:

https://github.com/basecamp/kamal/issues/1166
https://github.com/basecamp/kamal/issues/1058
https://github.com/basecamp/kamal/issues/69

This issue (or feature request) is more narrow, and may be easier to add on to existing functionality: Support github container registry (https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry).
I just tried using it, naively thinking it would be supported, but I get authentication issues like:
$ kamal deploy
Log into image registry...
  INFO [eacef5c7] Running docker login -u [REDACTED] -p [REDACTED] on localhost
  Finished all in 0.9 seconds
  ERROR (SSHKit::Command::Failed): docker exit status: 256
docker stdout: Nothing written
docker stderr: WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Error response from daemon: Get "https://registry-1.docker.io/v2/": unauthorized: incorrect username or password

even when my configuration states:
image: ghcr.io/stianlagstad/myapp
registry:
  username: stianlagstad
  password:
    - KAMAL_REGISTRY_PASSWORD

And https://kamal-deploy.org/docs/configuration/docker-registry/ does not mention github container registry, so my assumption now is that it's not supported yet.

kamal does support github container registry, the error your getting is telling you the issue, you need to configure it in your deploy file, the registry options are passed to the local docker cli to authenticate against the registry server heres an example using it
using github registry
# Name of the container image.
image: my-user/my-app

# Credentials for your image host.
registry:
  # Specify the registry server, if you're not using Docker Hub
  server: ghcr.io
  username: your-user

  # Always use an access token rather than real password when possible.
  password:
    - KAMAL_REGISTRY_PASSWORD

and here an example using a self hosted private registry
# Name of the container image.
image: my-user/my-app

# Credentials for your image host.
registry:
  # Specify the registry server, if you're not using Docker Hub
  server: private_registry:4443
  username: your-user
  password: your-pass

Thank you! I missed that. Sorry for the noise.


MQTT Discovered entities lost after ha restart

MQTT Discovered entities lost after ha restart because the retain flag was not set to true.
Fixed in git, to be added to 0.1.7 (TBA)
Docker fix to follow shortly...

fixed in 0.1.7


Error "Could not create the Java Virtual Machine" when trying init DB(Default conf,rockdb)

user@test:~/hugegraph/hugegraph-0.8.0$ ./bin/init-store.sh
Initing HugeGraph Store...
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.
user@test:~/hugegraph/hugegraph-0.8.0$ java -version
openjdk version "9-internal"
OpenJDK Runtime Environment (build 9-internal+0-2016-04-14-195246.buildd.src)
OpenJDK 64-Bit Server VM (build 9-internal+0-2016-04-14-195246.buildd.src, mixed mode)

@chrischeng021 We generally use java 8, of course java 9 should also be compatible, maybe you can try to use OpenJDK 9 GA version (please refer to https://github.com/elastic/elasticsearch/issues/28780 )

@javeme Hi, I've just updated my java version, but when I start the init-store.sh script, I still get the same error.... Here are the details:
bash-4.2# java -version
openjdk version "12" 2019-03-19
OpenJDK Runtime Environment (build 12+33)
OpenJDK 64-Bit Server VM (build 12+33, mixed mode, sharing)
bash-4.2# ./bin/init-store.sh
Initing HugeGraph Store...
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

@chrischeng021 It looks like an environmental issue. Could you try using Java 8 or other machines?

@javeme Actually I've tried 3 different servers and my local laptop, but all the same problem, and about the Java version, I've tried from openJDK 9ga to openJDK11, also the oracle jdk9, but none of them works.....

@chrischeng021 Is there something wrong during the operation? Please confirm that you are using the correct JDK via java -version after updating JDK.


Reference to a non-exported class RavenErrorHandler

Do you want to request a feature or report a bug?
report an encountered bug.
What is the current behavior?
In an Angular CLI project, I am trying to initialize raven on sentry.
app.module.ts
import * as Raven from 'raven-js';
import { BrowserModule } from '@angular/platform-browser';
import { NgModule, ErrorHandler } from '@angular/core';
import { AppComponent } from './app.component';

Raven
<EMAIL_ADDRESS>    .install();

class RavenErrorHandler implements ErrorHandler {
    handleError(err:any) : void {
        Raven.captureException(err.originalError);
    }
}

@NgModule({
    imports: [ BrowserModule ],
    declarations: [ AppComponent ],
    bootstrap: [ AppComponent ],
    providers: [ { provide: ErrorHandler, useClass: RavenErrorHandler } ]
})
export class AppModule { }

Throws the exception
ERROR in Error encountered resolving symbol values statically. Reference to a non-exported class RavenErrorHandler. Consider exporting the class (position 10:1 in the original .ts file), resolving symbol AppModule in /home/**/projects/**/src/app/app.module.ts
What is the expected behavior?
browser: Chrome
angular-cli: 1.0.0-beta.24
node: 6.9.0
os: linux x64
angular: 2.3.1
ravenjs : "raven-js": "^3.9.1",

I think you need to export RavenErrorHandler for aot / ngc.
This should solve your error. But using Raven-JS with Angular CLI is still blocked by https://github.com/getsentry/raven-js/issues/822

I was having the same issue.
Exporting solved it.
From:
class RavenErrorHandler implements ErrorHandler {   handleError(err:any) : void {     Raven.captureException(err.originalError);   } }
To:
export class RavenErrorHandler implements ErrorHandler {   handleError(err:any) : void {     Raven.captureException(err.originalError);   } }


Add xsuaa to the app

The app needs to have authentication switched on using xsuaa

Fixed via usage of SAP Workzone, standard edition


Ensure that shoot node network and seed service network do not overlap.

How to categorize this PR?

/area networking
/area robustness
/kind bug
What this PR does / why we need it:
Ensure that shoot node network and seed service network do not overlap.
The SNI implementation in Gardener using apiserver-proxy 'leaks' the cluster
ip of the kube-apiserver in the seed into all shoot nodes. As the ip is locally
assigned it can easily cause issues if the shoot node network overlaps with the
seed service network.
This change adds the corresponding validation and adapts the tests.
Which issue(s) this PR fixes:
None.
Special notes for your reviewer:
Release note:

The shoot node network is no longer allowed to overlap with the seed service network.

I don't know how many existing Shoots we have with their node network overlapping with the seed service network but I think similar change means that it is no longer possible to update similar Shoots. Shouldn't we think on how to make similar validation more backwards-compatible - for example allow existing Shoots to be updated/deleted?

@ialidzhikov How would you do it?

Perform the new validation only for new Shoots?


service_2/get_allocation_traces_test flaky and failing

There are new test failures on [cfe] Disallow subtyping Function with class modifiers enabled....[dds/dap] Ensure variable evaluateNames are stored for all lists/fields/getters.
The tests
service_2/get_allocation_traces_test/dds RuntimeError (expected Pass)

are failing on configurations
dartk-linux-release-arm64

  Expected: <2>
    Actual: <1>

  package:test_api/src/expect/expect.dart 134:31                               fail
  package:test_api/src/expect/expect.dart 129:3                                _expect
  package:test_api/src/expect/expect.dart 46:3                                 expect
  runtime/observatory_2/tests/service_2/get_allocation_traces_test.dart 127:5  tests.<fn>
  ===== asynchronous gap ===========================
  runtime/observatory_2/tests/service_2/test_helper.dart 495:17                _ServiceTesterRunner.run.runTest.<fn>.<fn>
  ===== asynchronous gap ===========================
  runtime/observatory_2/tests/service_2/test_helper.dart 374:9                 _ServiceTesterRunner.run.ignoreLateException
  ===== asynchronous gap ===========================
  package:test_api/src/backend/declarer.dart 215:9                             Declarer.test.<fn>.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/declarer.dart 213:7                             Declarer.test.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/invoker.dart 258:9                              Invoker._waitForOutstandingCallbacks.<fn>

full log
I don't see an obvious commit in the range that could have caused this.

56b303e64b7c7d651f2d3fbda865a2ac6fa1a938, but why would that only cause issues on one architecture?
41be8d54b55664543f8cc18683899bb4f0bfca99, but this doesn't touch the relevant test or code.

This test is flaky in service_2 on multiple other platforms: https://dart-ci.firebaseapp.com/current_results/#/filter=service_2/get_allocation_traces_test,service/get_allocation_traces_test&showAll
cc @bkonyi

happened again: https://ci.chromium.org/ui/p/dart/builders/ci.sandbox/vm-linux-release-arm64/1/overview

Once again: https://dart-ci.appspot.com/log/vm-linux-release-arm64/vm-linux-release-arm64/73/service/get_allocation_traces_test/dds

This test is meta flaking a lot: https://dart-ci.web.app/#showLatestFailures=false&test=service/get_allocation_traces_test/dds
It is flaking on multiple architectures, but on on ia32 it is meta flaking between failing and flaky.
@bkonyi can we either fix this test or disable it?

Still happening.

Closing in favor of https://github.com/dart-lang/sdk/issues/54487 since service_2/ no longer exists and service/ is in maintenance mode. However, we're still seeing flakes in the package:vm_service version of this test.


Update for Xcode 6.3

Xcode 6.3 shows a warning requesting to upgrade project settings. This patch does it.

Thanks @Dwarven for taking the time and effort to do a pull request!
Looks good and I would merge it, except that you should take out
buildSettings = {
+				ONLY_ACTIVE_ARCH = YES;
 			    };

and then rebase back to single commit.
The reasons given in #61.


I stop receiving DATA every day...

VWsFriends stops receiving DATA every day, restarting the container just gives me localhost not found.  Only restarting the PC helps... no idea why this is, started happening a few weeks ago....  every update had no effect

Same Probleme here

Can you provide the log output please?

HI
As soon as it stops receiving again, I'll copy the logs and get them to you
grts
Steve

@tillsteinbach : You've got mail

Thank you, will have a look tomorrow.

Hi. I seem to have the same issue with it stopping and nothing in the logs - although not every days tends to be every few days. Anything I can share to help or any advice? Thanks!

If you could have a look if you notice something special when it stops recording. Anything that you do at this time.

I add to the discussion.
I installed on Synology NAS the application (no Homekit) and after a while it stops updating.
In the attached log there is an error from 5th Nov and it has not updated since. Yesterday (9th Nov) between 1pm and 3pm I used the car, but no data is recorded.
vwsfriend-vwsfriend-1.html.txt

I started having the same problem for a while now. Restarting the VWsFriend container resolves it for a while, but usually after 1-3 days, there are no updates. The Volkswagen App on my phone is happily receiving updates though. I restarted just now, next time it stops updating I will check the logs.


Feature/toast component new

feat(toast): add new toast component

I would add bqClose event, that's triggered when the toaster is closed.

Close in favour of #301


docs: update example in readme

s/facility/metro so it matches the docs https://deploy.equinix.com/developers/docs/metal/deploy/on-demand/

I added the docs: Conventional Commits tag to the PR description to satisfy the bot.


SC quickstart and troubleshooting

Description

Edits based on Nancy's assessment of the SC Quickstart doc (see https://saucedev.atlassian.net/browse/TCD-2116).
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation fix (typos, incorrect content, missing content, etc.)

Checklist

[x] I have read the contributing document.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Just a question around how to upgrade.

@sweeneyskirt-sl Thank you for your feedback! Made some more changes -- can you please review when you have a moment?


nested gallery

Is it possible to do nested gallery?
If I do this, the plugin does not work with nested galleries ((.
Thank you!

Yes you can do nested galleries. Please use Stack Overflow for personal support requests.


Kill the engine

ET HIC DEFVNCTVS EST

It all seems to work just fine. The only remaining piece of work is to write a migration to munge the audit trail YAML yet again, to remove the Peoplefinder:: prefix from the class names.
Enjoy the weekend @jaikoo, @cfq, @Najaf, @fordie!

Sweet!! Best news all year!!!

I am incredibly happy about this. Laughing babies and rainbow unicorns frolicking the meadow happy. Shall I push the button? Because I'll push the button. The sooner we merge this, the better, so I vote for merging right now. Someone better push the button or I'm totes pushing the button.

Are GDS no longer using this engine then?

@ashb Not as far as we know. We don't have any ongoing contact, the people that were working on it aren't, and there hasn't been any work on their repository since early December.
Of course, they are still able to fork the repository at any point prior to this merge, or to fork this repository and make their own customisations.

As @ashb pointed out to me, this isn't a very helpful pull request: whilst everyone on the team knows the context, it's not expressed here. So here's an attempt to rectify that.
The original intention of the engine was to provide a core that both MOJ and GDS could use for their respective implementations, with differences in configuration, styling, and vocabulary. However, this benefit has not been realised because GDS have stopped working on it. Without any other consumers of the engine, it has, in fact, ended up growing MOJ-specific.
At the same time, developing against the engine has been a drag on productivity by adding a number of manual steps to development and deployment that were time-consuming and error-prone.
Finally, we believe that customisation needs can better be implemented using the environment, the database, feature flags and sub-locales.


Bytes filters doesn't work for '0'

The bytes filter works fine parsing numbers or strings
{{55 | bytes}}  //OK 55
{{'55' | bytes}}  //OK 55
{{0 | bytes}}  //OK 0
but fails for
{{'0' | bytes}}  //Error NaN
It would be nice to handle that case in the filter. And for other case don't show anything.

Thanks for reporting this issue. I've fixed it in the following commit: https://github.com/exceptionless/angular-filters/commit/fbc0c04d5122314f8ea203484775d62efc7dff61


This page must updated to include more details

The screenshots are not consistent and they talk about many different cases under the same context. Confusing!!
At least one screenshot is old and confusing. If you notice, the screenshot lists P1, P2, P3, EM1, EM2, and EM3. Where is P4 and P5 in the list? The same screenshot says 8 of 17 V-Cores are used, however the count from the drop down list is:
P1: 8, P2: 16, EM1: 1, EM2: 2, EM3: 4 (P3 is greyed out, so I presume it should not be in the calculation)
The math should have been: 8 + 16 + 1 + 2 + 4 = 31 which is 8 of 31 V-Cores are used.
I couldn't make up the number 17 available V-Core.
In this article, there is nothing with regards to memory allocation. It only talks about CPU.
Can I create a capacity with single V-Core? So, if I have P1, can I create 8 capacities in 8 different regions each with a single V-Core?
Will I get 25GB divided by 8 for each of the 8 capacities?

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

ID: 1f09b043-ca4f-8d06-16b9-19f03e89b344
Version Independent ID: 8166553d-853a-204d-b0bf-e7380d4042d6
Content: Configure and manage capacities in Power BI Premium - Power BI
Content Source: powerbi-docs/admin/service-admin-premium-manage.md
Service: powerbi
Sub-service: powerbi-premium
GitHub Login: @KesemSharabi
Microsoft Alias: kesharab

Hi @oussama70, thanks for your feedback. I am forwarding it along to the doc owner for consideration.

Hi @oussama70,
Sorry to here that you find this article confusing. Did you look at the Premium Gen2 articles. The original Power BI Premium capacities will shortly be phased out, see Plan your transition to Power BI Premium Gen2. Due to this reason, we'll not be updating the Premium articles.
I suggest that you look as some of the Gen2 articles such as What is Power BI Premium Gen2? and Premium Gen2 capacity load evaluation, and explore new Gen2 features such as autoscale.
Hope this helps,
Kesem


US 05.01.01

As an owner, I want help to specify the serial number for an item by automatically identifying the number from an image taken by the camera.
Requirements

Camera interface
Camera need to be able to scan the object.
Camera activity need to be able to pass the value back to the activity.

Checklist

[x] photo taking activity #136
[x] finding bar code or QR code from image

Barcode addition is done


Question: Is there any cache?

What if the user does not have internet connection?
Is there somewhere a cache or this feature will be finished in the future?

Yes the application caches the translations when they are loaded and it saves them locally so that it works offline and it is fast to load.
I am in the process of writing a script that you can put into your build phases that will pull an initial version for the app.


Remove Void Responses from being Eagerly Read

Description
Fixes #31555
Fixes #31849
Adds a new concept to SwaggerMethodParser which determines if the network response body will be consumed. This helps resolve an issue where before void-based responses would eagerly be read and could result in error cases. The Storage getProperties APIs are an example of this where the size of the blob or file is returned by the Content-Length header, then a buffer of that size would be created in preparation to read the response into memory eagerly. This would result in either excessive memory usage or OutOfMemoryErrors, depending on the size of the blob or file. Now void-based responses have special handling where the network response will be ignored and drained instead of attempting to read them.
This issue only began happening recently due to multiple, unrelated changes exposing this edge case.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Changed $timeout to $interval. Changing it only here an not in sessio…

…n_service.js allows protractor tests to run and also doesn't create a million stayloggedin modals as the interval is hit multiple times.

Discussed with Matt, and we're both good with the code. Merged.


As an author, when someone sends me a friends only-post I want to see the likes.

2

3/10

3/10

4

3

avg 3

15/5 = 3


Importing groups.

Having the possibility to import Group should be great.

isn't this done?

is it you're right


Add +project auto-completion

Hi,
Would it be possible to implement an auto-complete for project tags when one starts typing the first letters ? It would be very convenient for long project tags so that we don't have to write them entirely.
Thanks again for your great work !

I had been planning to make the projects/contexts  into checkboxes in the edit dialog. But that doesn't work well with the v2 changes.
This sounds like a good alternative.


There are no Libraries linked in iOS project of react-native

I have tried to create a new project in react-native, After creating when I have opened the iOS project, in libraries folder I could not any libraries and I am not able to run the iOS Project. I have tried react-native run-ios Got error like "error Failed to build iOS project. We ran "xcodebuild" command but it exited with error code 65. To debug build logs further, consider building your app with Xcode.app, by opening AppName.xcodeproj."
react-native info logs
System:
OS: macOS 10.14.5
Binaries:
Node: 11.3.0 - /usr/local/bin/node
Yarn: 1.5.1 - /usr/local/bin/yarn
npm: 6.11.3 - /usr/local/bin/npm
SDKs:
iOS SDK:
Platforms: iOS 12.2, macOS 10.14, tvOS 12.2, watchOS 5.2
IDEs:
Android Studio: 3.4 AI-183.6<IP_ADDRESS>92245
Xcode: 10.2.1/10E1001 - /usr/bin/xcodebuild
npmPackages:
react: 16.8.6 => 16.8.6
react-native: 0.60.5 => 0.60.5
npmGlobalPackages:
react-native-cli: 2.0.1
Steps To Reproduce

react-native init AppName
cd AppName
Open .xcodeproj & In Libraries there are no libraries found. I have tried to create multiple projects, but still no luck of getting libraries

My guess is that you need to install cocoapods:
Do gem install cocoapods in terminal, followed by cd ios && pod install in your project folder.

@samjt : Yeah you are right, After I have done those, it worked out.