Running angular cli on cloud 9 IDE, ng serve does not work

Please provide us with the following information:

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

Ubuntu 14.04.3 LTS
Versions.
angular-cli: 1.0.0-beta.22-1
node: 6.9.2
os: linux x64
Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.

App is created with the CLU.
The log given by the failure.

Normally this include a stack trace and some more information.

No failure is evident, but when I go to the address linked below, there's no page found:
baruchkogan:~/workspace/my-cli-app (master) $ ng serve
** NG Live Development Server is running on http://localhost:8080. **
Hash: 9e648c8fd6cb8172535d
Time: 10998ms
chunk    {0} main.bundle.js, main.bundle.map (main) 4.66 kB {2} [initial] [rendered]
chunk    {1} styles.bundle.js, styles.bundle.map (styles) 9.99 kB {3} [initial] [rendered]
chunk    {2} vendor.bundle.js, vendor.bundle.map (vendor) 2.22 MB [initial] [rendered]
chunk    {3} inline.bundle.js, inline.bundle.map (inline) 0 bytes [entry] [rendered]
webpack: bundle is now VALID.
Mention any other details that might be useful.
I've tried ng serve --host <IP_ADDRESS> --port 8080 and updating NPM. Updating NPM helped last week, but this week it doesn't seem to work.

Thanks! We'll be in touch soon.

This seems more like a configuration issue with Cloud 9. We didn't release a new beta this week yet, so something changed on your Ubuntu machine between last week and today.
Closing this as there's nothing we can do on our side. Check that you don't have a blocking firewall, maybe?


[BUG] loadFile progressHandler doesn't include "loading" state for video files on iOS

Describe the bug
No matter how large a video is, PMRequestState.loading is never reported when a video asset is getting loaded on iOS.
  PMProgressHandler handler = PMProgressHandler();
  handler.stream.listen((event) {
    print(
        'loading $asset => ${event.progress}, ${event.state} -- ${DateTime.now().millisecondsSinceEpoch}');
  });
  File file = asset.loadFile(isOrigin: false, progressHandler: handler);

Is this issue reproducible for image? As I tested, the image type of assets seems working well.
flutter: false
PhotoManager info: cache path = /private/var/mobile/Containers/Data/Application/F4FABAE9-83D9-461B-A139-FD6D5DDFA301/tmp/.image
PhotoManager info: get title start
PhotoManager info: get title from kvo
flutter: progress state onChange: PMRequestState.prepare, progress: 0.0
flutter: progress state onChange: PMRequestState.loading, progress: 0.26
flutter: progress state onChange: PMRequestState.loading, progress: 0.99
flutter: progress state onChange: PMRequestState.success, progress: 1.0
flutter: file = File: '/private/var/mobile/Containers/Data/Application/F4FABAE9-83D9-461B-A139-FD6D5DDFA301/tmp/.image/1FCAD81D-DE67-453C-AFB7-5233EF037AD1_L0_001_origin.A4006DEE-B5B0-4026-B174-4F50F4FBE70F.PNG'

@AlexV525 I'm only seeing the issue with videos.

@AlexV525 to clarify, I'm not seeing the progress indicator for local video files either, which is what I'd be expecting.

@AlexV525 to clarify, I'm not seeing the progress indicator for local video files either, which is what I'd be expecting.

Any path that indicates "the progress indicator"? e.g. which pages you are using, what code you are calling.

@AlexV525 I'm simply using this code
  PMProgressHandler handler = PMProgressHandler();
  handler.stream.listen((event) {
    print(
        'loading $asset => ${event.progress}, ${event.state} -- ${DateTime.now().millisecondsSinceEpoch}');
  });
  File file = asset.loadFile(isOrigin: false, progressHandler: handler);

On an AssetEntity of type video that is being pulled locally (not iCloud) and it's not reporting back PMRequestState.loading

A local video should only reported prepare and success IIRC.

Tested with 1.2.7, iPad with iPad OS 14.6, videos from iCloud, didn't repro this issue.
flutter: isLocallyAvailable: false
PhotoManager info: The asset has 2 resources.
PhotoManager info: cache path = /private/var/mobile/Containers/Data/Application/D9313661-1323-4778-BCBA-40C07D2C4ACE/tmp/.video
PhotoManager info: get title start
PhotoManager info: get title from kvo
flutter: progress state onChange: PMRequestState.prepare, progress: 0.0
flutter: progress state onChange: PMRequestState.loading, progress: 0.14
flutter: progress state onChange: PMRequestState.loading, progress: 0.61
flutter: progress state onChange: PMRequestState.loading, progress: 0.98
flutter: progress state onChange: PMRequestState.loading, progress: 0.99
flutter: progress state onChange: PMRequestState.success, progress: 1.0
flutter: file = File: '/private/var/mobile/Containers/Data/Application/D9313661-1323-4778-BCBA-40C07D2C4ACE/tmp/.video/FE8AD7C2-1F54-4CD6-BEE0-BCA1BDCB0BE3_L0_001_origin.F0907BCE-7642-4DCE-82A4-B12AD854BF9A.MOV'

@AlexV525 thanks for the extra info. Closing out the issue.


Problem when generating audio for new epoch

Traceback (most recent call last):
File "train.py", line 335, in
main()
File "train.py", line 326, in main
verbose=0,
File "/usr/local/lib/python3.7/dist-packages/tensorflow/python/keras/engine/training.py", line 1229, in fit
callbacks.on_epoch_end(epoch, epoch_logs)
File "/usr/local/lib/python3.7/dist-packages/tensorflow/python/keras/callbacks.py", line 435, in on_epoch_end
callback.on_epoch_end(epoch, logs)
File "/content/gdrive/My Drive/prism-samplernn/callbacks.py", line 103, in on_epoch_end
self._generate(ckpt_path, self.generation_args)
File "/content/gdrive/My Drive/prism-samplernn/callbacks.py", line 109, in _generate
args['temperature'], args['seed'], args['seed_offset'])
File "/content/gdrive/My Drive/prism-samplernn/generate.py", line 135, in generate
file_name = f'{file_name}_t={temperature[i][0]}.wav'
IndexError: too many indices for array: array is 0-dimensional, but 1 were indexed

@deccolquitt Hi, looks like a problem with the value being passed to temperature parameter. Could you post how you're calling the train.py script, with the parameters? Thanks.

@deccolquitt OK I can see what's causing it - I'm assuming you weren't explicitly passing a temperature value. I've identified a bug in that circumstances, am fixing it now.

Hi, thanks for getting back so quickly, just in case you need it, here are the parameters I was using anyway:
!python train.py
--data_dir ./chunks
--num_epochs 100
--batch_size 64
--max_checkpoints 2
--checkpoint_every 1
--output_file_dur 5
--sample_rate 16000

Thanks, that's just as I expected. It would be using the default temperature if you don't explicitly set it (either as a list or as one float value), but the default value was not being passed down internally in the correct format
Should work fine now, just pushed the fix.

brilliant, will it try tomorrow

BTW, we're experimenting with replacing the default config params, you might want to try these:
{
    "seq_len": 512,
    "frame_sizes": [2,8],
    "dim": 1024,
    "rnn_type": "gru",
    "num_rnn_layers": 1,
    "q_type": "mu-law",
    "q_levels": 256,
    "emb_size": 256
}

These worked well for our Beethoven Piano Sonata model, though as always YMMV.


⚠ SceneTime has degraded performance

In 1581dce, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1508 ms

Resolved: SceneTime performance has improved in 7590362 after 6 minutes.


Multiple SubnetIds

Encountered this error testing multiple SubnetIds. The README does state that subnets need to be in different AZs (I had two in 1 AZ).  However, the error message below is not clear that "duplicate instance pools" refers to SubnetIds in the same AZ.
2020-12-05 19:27:20,108 - resume - ERROR - Failed to launch nodes for partition=aws and nodegroup=node - An error occurred
 (InvalidFleetConfig) when calling the CreateFleet operation: The fleet configuration contains duplicate instance pools.

Plugin-v2

Hello, thanks for your message. The README already included "List of subnets where EC2 instances can be launched for this node group. If you provide multiple subnets, they must be in different availability zones". However, I just appended "or the CreateFleet request may return the error message The fleet configuration contains duplicate instance pools" to make it clear.


Guess input type more carefully

Match on word boundaries only.
The word "hourly" triggers a url type validation, that's not great.

@feliperenan rebased!

@rafaelfranca updated for additional assert.

Hey @sringling, could you please rebase it again? 😅 after that we'll merge it.

@feliperenan re-rebased!


Use new get_unchecked methods in QuerySortedIter

Objective
In QuerySortedIter::fetch_next, Entites::get. Archetypes::get, Tables::get are all guaranteed to return correctly, but still do bounds checks/further checks.
Solution
Introduce get_unchecked methods on Entities, Archetypes, and Tables, and use those.

Should these methods be public?

It is the slice::get_unchecked that improves the perf here, plus avoiding the additional checks in Entities::get.
Not using debug_unchecked_unwrap is just a consequence of no longer handling options!

It is the slice::get_unchecked that improves the perf here

Right, but I would expect .get().debug_checked_unwrap() to compile to the same thing as .get_unchecked().  The optimizer will see the core::hint::unreachable_unchecked() on the None branch and eliminate any check that causes a None to be returned, reasoning that it knows the result of those checks without making them.
... Hmm, I bet I can check this with cargo-show-asm...
#[inline(never)]
pub fn asm_test(entities: &Entities, entity: Entity) -> EntityLocation {
    unsafe { entities.get(entity).debug_checked_unwrap() }
}

> cargo asm -p bevy_ecs --features bevy_reflect asm_test --lib --simplify

    Finished `release` profile [optimized] target(s) in 0.40s
bevy_ecs::entity::asm_test:
        mov rax, rcx
        mov ecx, r8d
        mov rdx, qword ptr [rdx + 8]
        lea rcx, [rcx + 4*rcx]
        movups xmm0, xmmword ptr [rdx + 4*rcx + 4]
        movups xmmword ptr [rax], xmm0
        ret

No branches!

ohhh, I see I misunderstood debug_checked_unwrap, and Entites::get only has one path to Some!


python3Packages.datasets: 1.0.1 -> 1.1.2

Motivation for this change
Changelog:
https://github.com/huggingface/datasets/releases/tag/1.1.2
https://github.com/huggingface/datasets/releases/tag/1.1.0
https://github.com/huggingface/datasets/releases/tag/1.0.2
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python3Packages.datasets

Result of nixpkgs-review pr 103116 run on x86_64-darwin 1

2 packages failed to build:

python37Packages.datasets
python38Packages.datasets

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


Change the wording to make it more clear that roles are installing things

WHY?
Similar to the yarn_install role that is getting changed in https://github.com/smashingboxes/taperole/pull/150. Some of these roles can be a little confusing
WHAT?

Changed roles to have install first to make it clear they are installing monit and puma rather than running a command called monit install or puma install

@dkniffin Sounds good. Feel free to merge whenever you want or let me know when to merge and I will 👍

@dkniffin FINALLY! 😂


allow using custom datetime format for inference and parsing csv file

The patch extends the current implementation to allow passing a custom
datetime_re and datetime_format to the ReaderBuilder.
datetime_re is used infer schema of the csv and then datetime_format is
used to parse the actual string to a Date64.
ofcourse  passing non-compatible datetime_re and datetime_format values
is going to fail the parsing or inference, however it is an expected but
hard-to-detect failure.
Which issue does this PR close?

Closes #.
Rationale for this change
Sometimes csv files follow non-standard datetime format. Arrow does not really care for the datetime format stored in the csv but merely guesses/assumes it to be of a certain fomrat. The change extends the csv reader to use a custom fromat.

What changes are included in this PR?

Are there any user-facing changes?
Any external libraries relying on ReaderBuilder and Reader structs will have to be updated to the new new API,

Codecov Report

Merging #1112 (af441b8) into master (2ad99ec) will decrease coverage by 0.03%.
The diff coverage is 67.50%.

@@            Coverage Diff             @@
##           master    #1112      +/-   ##
==========================================
- Coverage   82.34%   82.30%   -0.04%
==========================================
  Files         168      168
  Lines       49479    49546      +67
==========================================
+ Hits        40743    40779      +36
- Misses       8736     8767      +31

Impacted Files
Coverage Δ

arrow/src/csv/reader.rs
87.99% <67.22%> (-2.59%)
:arrow_down:

arrow/src/csv/writer.rs
72.13% <100.00%> (+0.10%)
:arrow_up:

parquet_derive/src/parquet_field.rs
65.98% <0.00%> (-0.23%)
:arrow_down:

arrow/src/array/transform/mod.rs
85.56% <0.00%> (ø)

arrow/src/datatypes/field.rs
53.68% <0.00%> (+0.30%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2ad99ec...af441b8. Read the comment docs.


Remove score_update_worst_case default impl

Do not ever default impl feature gated trait functions, unless you want very weird stuff to happen.
Reason being is that someone will forget to enable the feature and you use the default impl without noticing.
https://github.com/paritytech/substrate/pull/11976 should fix this permanently.

The CI pipeline was cancelled due to failure one of the required jobs.
The job name - cargo-check-benches
The job logs - https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/1749128

Actually i dont get what this fixes. if the feature flag is not there, the whole function wont be there? what does it matter if there is a default impl?

Actually i dont get what this fixes. if the feature flag is not there, the whole function wont be there? what does it matter if there is a default impl?

It happens since the feature of the crate which contains the trait is enabled, but the one which should implement is not.
So then it uses the default impl.

Now the messed up features make Polkadot not build :hankey:
I will merge this with #11976


Broken example

At least on Ubuntu 16.04 with g++-5, clang++-3.8, the example from readme seems to require some update:
mmm@mmm-U2442:~/devel/uvw/build$ clang++-3.8 -I ../src/ -std=c++14 -o socket_example example.cpp
In file included from example.cpp:1:
In file included from ../src/uvw.hpp:1:
In file included from ../src/uvw/async.hpp:8:
In file included from ../src/uvw/handle.hpp:9:
../src/uvw/util.hpp:188:28: error: use of undeclared identifier 'uv_passwd_t'
    Passwd(std::shared_ptr<uv_passwd_t> passwd): passwd{passwd} { }
                           ^
../src/uvw/util.hpp:188:28: error: use of undeclared identifier 'uv_passwd_t'
../src/uvw/util.hpp:197:21: error: use of undeclared identifier 'uv_passwd_t'
    std::shared_ptr<uv_passwd_t> passwd;
                    ^
../src/uvw/util.hpp:382:35: error: use of undeclared identifier 'uv_os_tmpdir'
            return details::path(&uv_os_tmpdir);
                                  ^
../src/uvw/util.hpp:398:31: error: unknown type name 'uv_passwd_t'
            auto deleter = [](uv_passwd_t *passwd){
                              ^
../src/uvw/util.hpp:403:29: error: use of undeclared identifier 'uv_passwd_t'
            std::shared_ptr<uv_passwd_t> ptr{new uv_passwd_t, std::move(deleter)};
                            ^
../src/uvw/util.hpp:403:42: error: C++ requires a type specifier for all declarations
            std::shared_ptr<uv_passwd_t> ptr{new uv_passwd_t, std::move(deleter)};
                                         ^
../src/uvw/util.hpp:403:50: error: unknown type name 'uv_passwd_t'
            std::shared_ptr<uv_passwd_t> ptr{new uv_passwd_t, std::move(deleter)};
                                                 ^
In file included from example.cpp:1:
In file included from ../src/uvw.hpp:12:
../src/uvw/poll.hpp:22:18: error: use of undeclared identifier 'UV_DISCONNECT'; did you mean 'UV_CONNECT'?
    DISCONNECT = UV_DISCONNECT
                 ^~~~~~~~~~~~~
                 UV_CONNECT
/usr/include/uv.h:192:19: note: 'UV_CONNECT' declared here
  UV_REQ_TYPE_MAP(XX)
                  ^
In file included from example.cpp:1:
In file included from ../src/uvw.hpp:1:
In file included from ../src/uvw/async.hpp:8:
In file included from ../src/uvw/handle.hpp:9:
../src/uvw/util.hpp:327:54: error: too few arguments to function call, expected 4, have 2
    auto err = std::forward<F>(f)(args..., buf, &size);
               ~~~~~~~~~~~~~~~~~~                    ^
../src/uvw/util.hpp:382:29: note: in instantiation of function template specialization 'uvw::details::path<int (*)(uv_loop_s *, uv_fs_s *, const char *, void (*)(uv_fs_s *))>'
      requested here
            return details::path(&uv_os_tmpdir);
                            ^
10 errors generated.

I'll fix it as soon as possible. It seems that you have an outdated version of libuv, can I ask you which one are you using to test it? Thank you.

Thank you! I have version libuv 1.8.0 from package libuv1-dev on Ubuntu 16.04

The example from the README is a reduced version of the test in test/main.cpp.
I've just tried both and they work like a charm. I suspect it's a problem due to the fact that you have an outdated version of libuv.
uvw has been written during June/July and we used the upstream of the library at that time.
Try to go in the build directory and launch the following commands:
cmake ..
make -j4
make test

If everything works fine, you have the right version of the library.
Otherwise you must update it if you want to use uvw.

See here: http://docs.libuv.org/en/v1.x/misc.html
uv_passwd_t is part of libuv since version 1.9.0 and I guess the same applies to UV_CONNECT.
The issue no longer stands, but I'm updating the README file to mention this. Thank you.

Thank you for updating the requirements in Readme and figuring it out!

Confirmed: http://docs.libuv.org/en/v1.x/poll.html
UV_DISCONNECT is part of libuv since version 1.9.0.
Thank you for having pointed it out.


increase acc

What does this PR do?
Fixes #2029. See https://github.com/PyTorchLightning/pytorch-lightning/pull/2029#discussion_r433296557
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

seems we have a hanging test for macOS and py3.8
https://github.com/PyTorchLightning/pytorch-lightning/pull/2044/checks?check_run_id=728728190
@williamFalcon it is already from master

@SkafteNicki this test is causing problem on macOS and py3.8
tests/loggers/test_all.py::test_logger_reset_correctly[Batch-size-Finder]
it is hanging there infinitely long and then the test is timeouted... any idea why?

@Borda not completly sure why it is failing specific on mac. Do you know how much RAM the machines have the test are running on? Since the test currently are evaluated on cpu, it is only when the RAM is filled up we get an OOM error and the search stops. Maybe it would help switching these tests to gpu, since the memory is smaller the test will be faster?

@SkafteNicki it is running on the very same servers, just different images...
also, it is failing on just one particular configuration...

@Borda am I reading the checks correct, that it works on mac with python 3.7 but not with python 3.8?

@Borda am I reading the checks correct, that it works on mac with python 3.7 but not with python 3.8?

exactly... :]

This solves the issue with invalid accuracy tests...


preLaunch task fails with: SSL connection could not be established

I'm experience an issue very similar to #7 (closed) in VS Code on mac. When running in debug mode, I receive the following error:
Built container image in 1s
Waiting for container...10s
The SSL connection could not be established, see inner exception.
The terminal process terminated with exit code: 1

Here is my launch.json
    {
        "name": ".NET Core Launch (AZDS)",
        "type": "coreclr",
        "request": "launch",
        "preLaunchTask": "azds: prelaunch",
        "pipeTransport": {
            "pipeCwd": "${workspaceFolder}",
            "pipeProgram": "azds",
            "pipeArgs": [
                "exec",
                "--interactive",
                "--on-exit-resume-up",
                "--"
            ],
            "debuggerPath": "/vsdbg/vsdbg",
            "quoteArgs": false
        },
        "cwd": "/src",
        "program": "/src/bin/Debug/netcoreapp2.2/dev-spaces-api.dll",
        "sourceFileMap": {
            "/src": "${workspaceFolder}/."
        }
    }

The same code works fine from a Windows 10 VM running Visual Studio 2017 so I don't believe this is a cluster or Azure infrastructure issue.

@travisnielsen
Can you share AZDS logs from your machine and CLI, VSCode extension version?
The logs should be present in $TMPDIR on mac under Azure Dev Spaces subdirectory, and you can run azds --version to get the CLI version.

@travisnielsen Closing this issue as we need the client logs to investigate further,


Add remark-shortcodes to plugins list

Hi,
Thanks for your work on Remark!
I've created a Markdown plugin which parses Wordpress/Hugo-like block-level shortcodes. It's a simple parser and just updates the AST with the appropriate details, the idea being that other plugins can then handle the transformation to something more meaningful like a template partial.
More here: https://www.npmjs.com/package/remark-shortcodes
Cheers!
Darian

Awesome! 👍
One suggestion for the readme: add a description of the new node type to Not a transformer in IDL format. Something like the footnoteReference description?

@wooorm Aha, I was looking for the right type syntax. Will do. Cheers.


func User() should return a pointer

To match other recent changes and keep consistency the restapi.go func User() should return a pointer to a User{} struct.
https://github.com/bwmarrin/discordgo/blob/develop/restapi.go#L155

Closed by SHA: 00cd5cb178d50980f68bd400bde37bd002b22b4d


SCBootstrappingTool genesisinfo command arguments currupted

genesisinfo command arguments can be too long for some platforms.
For example, on windows cmd max line length is 8192, but the command with arguments is about 8600, so the last ~400 symbols were cut from the command.
Currently bootstrapping tool accept only single-line commands. We need to add the support of multi line arguments.

Hi @i-Alex  i run into the same problem on Windows, this is how I was able to circumvent
created a txt file with the whole command  and passed the file as a input , give it a try
ex:
java -jar tools/sctool/target/Sidechains-SDK-ScBootstrappingTools-0.2.1.jar < genesis.txt

Solved in the PR https://github.com/HorizenOfficial/Sidechains-SDK/pull/172


Removing extra raycasts from eye gaze enabled objects

Overview
Some objects which respond to eye gaze make extra raycast calls that have already been made by the Eye Gaze Provider.
This PR removes those calls and cleans up the FocusProvider code to make the ways that it updates the GazeProvider more understandable.


Extra's

Good Day,
Is it possible to explain me how to add logo to login page?
As well as to add extra type applications?

Login page uses this template: https://github.com/eusonlito/Password-Manager/blob/master/resources/views/domains/user/auth-credentials.blade.php
To create a custom type:

Add the type here: https://github.com/eusonlito/Password-Manager/blob/master/app/Domains/App/Service/Type/Type.php
Use some Format class as base from here https://github.com/eusonlito/Password-Manager/blob/master/app/Domains/App/Service/Type/Format
Duplicate some type folder with the new format type https://github.com/eusonlito/Password-Manager/blob/master/resources/views/domains/app/create.blade.php
Add the image icon inside https://github.com/eusonlito/Password-Manager/blob/master/resources/views/assets/images/

After adding and making relevant changes Im getting a internal 500 error on the applications tab.

You can enable debug mode on .env file with APP_DEBUG=true (and composer deploy after the change).

Receiving following issue:

Seems like some s
App.zip
ort of
en.zip
email class is missing
email.zip
.

Let me know if there is something i'm missing. Thank you for your time.

You can create a public repository with your changes, then I will can compare sources. What's your new format?

I very new to the laraval environment. I only want 2 extra type of applications.

Email - In put field should only have a username(the user email) and Password(User email password)
idrive - Input field for username, password and Encryption key field)

I would really appreciate it if you could assist me with this. I love the look and feel of your application and what to put time in to understand the laravel framework. But I need this passwordmanager asap running.

Please, create a public repository from this and I can help you with the changes.

Type.zip
Attached is the type folder with the new changes. Please amend these in the software.

Please @tunavis, create a public fork of this repository and I will can help you. I can't work with zip because it is very difficult to track the changes.

Like I mentioned I’m very new to laravel hardly know how it  works. If you could please just explain in more depth how to achieve this. Sorry for the inconvenience

Hi! I can see your fork https://github.com/tunavis/Password-Manager
I will try to work today to help you.
Regards!

Hi Thank you for the update I see changes are made but at soon as I select idrive type i get the following error

.

Same issue with Email type.

?

You have a new PR in your repository.


feat: add prometheus blackbox exporter dashboard

mostly copied from generic open source dashboard:
https://grafana.com/grafana/dashboards/7587
added availability over time visualizations

:tada: This PR is included in version 1.0.0-alpha.37 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Replace Reorderable list component

What
Replace reorderable component with the global version in govuk publishing gem
Why
This will clean up code and keep it consistent.
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
https://trello.com/c/9CQm5A0k/251-update-publishing-tools-to-use-new-reorderable-component

This looks good to me 👍
It looks like we still need to remove sortableJS from package.json and yarn.lock.

good spot

This looks good to me 👍
It looks like we still need to remove sortableJS from package.json and yarn.lock.

done

Super ⭐

Thank you!


QUANTITY fields in ExecutionPayloadV1 are too permissive

Description
Execution client's Engine API specifies baseFeePerGas as a QUANTITY field which has the following restrictions:

Values of a field of QUANTITY type MUST be encoded as a hexadecimal string with a 0x prefix and the leading 0s stripped (except for the case of encoding the value 0) matching the regular expression ^0x(?:0|(?:[a-fA-F1-9][a-fA-F0-9]*))$.
Note: Byte order of encoded value having QUANTITY type is big-endian.

Currently is possible to feed Teku the following spoofed values and they will end up in the Execution payload without issues:

Spoofing (node 1) blockNumber, 0x34 -> 0x00000000000000034 (Overflow 64bits)
Spoofing (node 2) blockNumber, 0x35 -> 0x035 (Leading zero)
Spoofing (node 1) gasLimit, 0x1c9c380 -> 0x00000000001c9c380 (Overflow 64bits)
Spoofing (node 2) gasLimit, 0x1c9c380 -> 0x01c9c380 (Leading zero)
Spoofing (node 1) gasUsed, 0x0 -> 0x00 (Leading zero)
Spoofing (node 1) gasUsed, 0x0 -> 0x (Empty hex)
Spoofing (node 1) timestamp, 0x62c5ec6f -> 0x00000000062c5ec6f (Overflow 64bits)
Spoofing (node 2) timestamp, 0x62c5ec75 -> 0x062c5ec75 (Leading zero)
Spoofing (node 2) baseFeePerGas, 0x50ef0 -> 0x050ef0 (Leading zero)

Steps to Reproduce (Bug)
Using hive branch https://github.com/marioevz/hive/tree/all-tests-plus-cl-clients, run:
./hive --client go-ethereum,teku-bn,teku-vc --sim eth2/engine --sim.loglevel 5

Expected behavior: The payload should be rejected
Actual behavior: The payload is accepted and incorporated into a beacon block.
Frequency: Always during this test case (invalid-quantity-fields)
Versions (Add all that apply)

Software version: teku/v22.6.1+29-gb2bc943/linux-x86_64/-eclipseadoptium-openjdk64bitservervm-java-17
Java version: openjdk 17.0.3 2022-04-19
OS Name & Version: Ubuntu 18.04.6 LTS
Docker Version: 20.10.17

Thanks for logging this. We'll take a look. In general Teku follows Postel's law "Be liberal in what you accept and conservative in what you send". There's no real ambiguity in what's meant with these examples, but given we have hive tests and the JSON-RPC spec is explicit in forbidding these things it's worth enforcing it in Teku as well.

@marioevz
I'm fixing this but I cant run hive tests since it brakes compiling the tests
INFO[07-19|10:26:52] building image                           image=hive/simulators/eth2/engine:latest dir=simulators/eth2/engine nocache=false pull=false
EROR[07-19|10:27:04] image build failed                       image=hive/simulators/eth2/engine:latest err="The command '/bin/sh -c go build -o ./sim .' returned a non-zero code: 2"
The command '/bin/sh -c go build -o ./sim .' returned a non-zero code: 2

I hit that today too - it doesn't seem to build on ARM architectures.


Support for Steam Deck

Hello. I tried to ways.
Downloading the latest .exe, then I extracted it and I was able to get the .bat file running with steam/proton. At first everything seemed to work. It opened the browser etc. But when I wanted to convert something it simply gave me an error.
Other way I did was the instructions for the linux install. It gave some errors in the terminal but at the end it said it was installed and I can run the sh file now. But that does nothing.
Could someone please offer support?

you have to follow the linux instructions


Rename "ClusterPolicy" to just "Policy"

Suggest renaming "ClusterPolicy" to just "Policy". It is supposed to support both Policy and ClusterPolicy custom resources.

We allow multiple policies so Policies would be the correct name.


Can’t change default login and password for Nakama Console

Versions: Nakama {3.13.1+2000e4b8}, {Docker Compose}
Hello everyone. I'm the new one of Nakama user. My Nakama Server is working. I could connect to the Nakama Console but I couldn't change default login and password for it.

This is my docker-compose.yml
...
  nakama:
    image: registry.heroiclabs.com/heroiclabs/nakama:3.13.1
    entrypoint:
      - "/bin/sh"
      - "-ecx"
      - >
          /nakama/nakama migrate up --database.address root@cockroachdb:26257 &&
          /nakama/nakama --config /nakama/data/config.yml
    restart: "always"
    links:
      - "cockroachdb:db"
    depends_on:
      cockroachdb:
        condition: service_healthy
      prometheus:
        condition: service_started
    volumes:
      - ./data:/nakama/data
...

This is my config.yml in data folder
...
 name: nakama-node-1
data_dir: "./data/"

logger:
    stdout: false
    level: "warn"
    file: "/nakama/data/logfile.log"

console:
    port: 7351
    username: "********"
    password: "*************"
...

"Admin" and "password" still work to entering in Nakama console.
Can somebody help me with it?

It looks like you aren't exposing the ports needed to access the console locally -- it's only running in your docker container. When you load the console you must be accessing a different instance of the console running on your host machine.
Adding the following port configuration from our sample docker compose to your docker compose file should fix this issue:
    expose:
      - "7349"
      - "7350"
      - "7351"
      - "9100"
    ports:
      - "7349:7349"
      - "7350:7350"
      - "7351:7351"

This is my compose config for nakama docker
version: '3' services:   cockroachdb:     image: cockroachdb/cockroach:latest-v20.2     command: start-single-node --insecure --store=attrs=ssd,path=/var/lib/cockroach/     restart: "always"     volumes:       - data:/var/lib/cockroach     expose:       - "8080"       - "26257"     ports:       - "26257:26257"       - "8080:8080"   nakama:     image: registry.heroiclabs.com/heroiclabs/nakama:3.13.1     entrypoint:       - "/bin/sh"       - "-ecx"       - >         /nakama/nakama migrate up --database.address root@cockroachdb:26257 &&         /nakama/nakama --config /nakama/data/config.yml         exec /nakama/nakama --name nakama1 --database.address root@cockroachdb:26257 --logger.level DEBUG --session.token_expiry_sec 7200 --metrics.prometheus_port 9100             restart: "always"     links:       - "cockroachdb:db"     depends_on:       - cockroachdb       - prometheus     volumes:       - ./data:/nakama/data     expose:       - "7349"       - "7350"       - "7351"       - "9100"     ports:       - "7349:7349"       - "7350:7350"       - "7351:7351"     healthcheck:       test: ["CMD", "curl", "-f", "http://localhost:7350/"]       interval: 10s       timeout: 5s       retries: 5   prometheus:     image: prom/prometheus     entrypoint: /bin/sh -c     command: |       'sh -s <<EOF         cat > ./prometheus.yml <<EON       global:         scrape_interval:     15s         evaluation_interval: 15s       scrape_configs:         - job_name: prometheus           static_configs:           - targets: ['localhost:9090']         - job_name: nakama           metrics_path: /           static_configs:           - targets: ['nakama:9100']       EON       prometheus --config.file=./prometheus.yml       EOF'           ports:       - '9090:9090' volumes:   data:
I made every steps one by one from this instruction: https://heroiclabs.com/docs/nakama/getting-started/install/docker/
Expose and ports is configured like you said.
I could go to the console with default log and pass.
Something wrong with me I suppose

I'd recommend trying it on a local machine first to make sure it works.

I checked it out with Godot client. It's working fine.
I could make new account and connect to it.
There is only one thing that make me sad. I can't change default login and password for the console

@NoizlessDog The problem is that your entrypoint command is not right. I'll assume you've mounted the volume correctly so your "config.yml" file can be found by the container fileystem, you should use this line:
/nakama/nakama migrate up --database.address root@cockroachdb:26257 && exec /nakama/nakama --name nakama1 --config /nakama/data/config.yml --database.address root@cockroachdb:26257 --logger.level DEBUG --session.token_expiry_sec 7200 --metrics.prometheus_port 9100

Whereas what you've done is attempt to start the nakama server in the container twice. I will lock this issue, please follow up with getting started questions on the community forums. Thanks!


Add google-spreadsheet 0.4.0 to stable

Please update my adapter ioBroker.google-spreadsheet to version 0.4.0.
This pull request was created by https://www.iobroker.dev c0726ff.

@mcm1957 Thanks for the review, I didn't notice this.
I created a new release v0.5.0 yesterday. Since there are a few other small changes (dependencies and connectionState) included, I will create a new Pull Request in one or two weeks, if there are now issues.


CIS 1.07 & 1.09: Separation of duties

@gkowalski-google can you please review and help Charlie get this merged. I think he has addressed Joe's questions/feedback.


Add option to get messages in an id interval

This is possible anyway by iterating through id's, creating an reply to them, inspecting the result and deleting this reply. Would be very nice that instead of this workaround we would be able to just get the message object by it's id or getting all messages from an interval of id's

You should store message contents locally. if you need them. Quering remote server is very inefficient, is subject to rate limiting and can affect rate limits of other legitimate requests.

Yes, I understand that this type of function should be somehow limited, to avoid problems.
My usecase is that i want to create a bot which takes all messages between start message and end message and transforms them into audio. start/end messages are specified by users, as reply to them with some commands. I thought that this will not be a problem, since there is an core telegram api for this usecase: https://core.telegram.org/method/messages.getMessages

No, it will be a problem. The bot can use direct replies to the messages instead.


use getpass to get user

os.getlogin() does not seem to work in all cases, however getpass.getuser() does.
As per https://docs.python.org/3/library/os.html#os.getlogin:

For most purposes, it is more useful to use getpass.getuser() since the latter checks the environment variables LOGNAME or USERNAME to find out who the user is, and falls back to pwd.getpwuid(os.getuid())[0] to get the login name of the current real user id.

Example:
$ sudo python
Python 3.11.3 (main, May 16 2023, 12:21:43) [GCC 12.2.1 20230428] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import os
>>> os.getlogin()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
FileNotFoundError: [Errno 2] No such file or directory
>>> import getpass
>>> getpass.getuser()
'root'

Thanks for the fix! Merged with b0e7fcdda8965858f5161d26e823e76405da0fac.


build: fix clang-cl exception flags

When building with clang-cl, we currently pass the /EHsc- flag in an attempt to disable exceptions, but this is equivalent to /EHs which still enables C++ exceptions and trips the #error in block.cpp. clang-cl actually disables exceptions by default, so we don't need to pass anything.
See https://msdn.microsoft.com/en-us/library/1deeycx5.aspx.

cc @compnerd

CC @madcoder

@swift-could please test

@swift-ci please test


configure a low metric for the TAP device, ensuring its non-filtered DNS servers are used

Unlike Unix systems, there's no "system-wide" DNS resolver on Windows. Instead, each network device has one or more DNS servers associated with it (actually two sets for each device: one for IPv4, another for IPv6!). Precisely which DNS server gets used for any query is a hazily-defined combination of the default route and network device metrics.
We've had reports - notably https://github.com/Jigsaw-Code/outline-client/issues/191 - that the DNS server(s) associated with the "default" network device (usually an ethernet card) were being used instead. If those DNS servers are filtered, then filtered websites may not be accessible. I didn't see this behaviour on my own system but I do believe it's possible.
This PR configures a low metric for the TAP device, ensuring that the (hopefully non-filtered) DNS servers associated with the TAP are used. I like this approach which doesn't make any modifications to the default gateway or network device. It looks like OpenVPN also does this.
Also a few fixes and tweaks to the Visual Studio project.
BTW, new binary is signed:
/cygdrive/c/Program\ Files\ \(x86\)/Windows\ Kits/10/bin/10.0.17134.0/x64/signtool.exe sign /tr http://timestamp.digicert.com /td sha256 /fd sha256 /n "Jigsaw Operations LLC" tools/OutlineService/OutlineService/bin/OutlineService.exe

@fortuna We keep the binary in the repository because it's a pain in the **** to build and signing additionally so.


Add a parser config/output-filter for parsing userAgent field in HTTP logs

There are two options to add userAgent parsing:

Write a parser entry in Logagent in the patterns.yaml
Create an input-filter that uses a userAgent parser

To adhere to the Logagent architecture guidelines, creating an input-filter is the better option.

After some fiddling, I've come to the conclusion that it's better to have this as an output filter. :ok_hand:


nixos/mailman: Fix mailman-settings.service configuration

Without this change, mailman-settings.service is not guaranteed to
complete before dependent services. This can lead to various errors
like:
mailman-web-setup.service: Changing to the requested working directory failed: No such file or directory

Motivation for this change
Fixes an error.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

cc: @lheckemann @alyssais

Thanks!


enable access to validation_level=min on data imports

To speed up import of very large data sets (e.g., collections of .fastq.gz files), we should enable users to change the format validation level on import. This functionality already exists on the formats, but isn't accessible through the existing import functionality.

I believe this was closed by #734

Confirmed, thanks @colinvwood.
qiime tools import --help
Usage: qiime tools import [OPTIONS]

  Import data to create a new QIIME 2 Artifact. See https://docs.qiime2.org/
  for usage examples and details on the file types and associated semantic
  types that can be imported.

Options:
  --type TEXT             The semantic type of the artifact that will be
                          created upon importing. Use `qiime tools list-types`
                          to see what importable semantic types are available
                          in the current deployment.                [required]
  --input-path PATH       Path to file or directory that should be imported.
                                                                    [required]
  --output-path ARTIFACT  Path where output artifact should be written.
                                                                    [required]
  --input-format TEXT     The format of the data to be imported. If not
                          provided, data must be in the format expected by the
                          semantic type provided via --type. Use `qiime tools
                          list-formats --importable` to see which formats of
                          input data are importable.
  --validate-level [min|max]
                          How much to validate the imported data before
                          creating the artifact. A value of "max" will
                          generally read the entire file or directory, whereas
                          "min" will not usually do so.       [default: "max"]
  --help                  Show this message and exit.


pip install -r requirements.txt stuck at "Building wheels for collected packages: grpcio"

When running pip install -r requirements.txt it is stuck at
"Building wheels for collected packages: grpcio"
Is this normal?
Building wheels for collected packages: grpcio
Building wheel for grpcio (setup.py) ... /
I am using the unclocked version, after an apt-get upgrade
Thanks

Which software package are you installing, containing the mentioned requirement.txt ?
I like to reproduce your issue.

Hi. For future references, it's not stuck.
There are times when the PyPi server is slow, or maybe your internet is unstable.
There are also times where the wheel takes too long to build, especially when C compiling is involved.
To check the full logs of the installation, try running with the verbose keyword:
pip install -vvv -r requirements.txt
I had a similar case when installing PyQt5 that took almost 2 hours.


Document Processing setup

This should be very simple and straightforward, but I want it to be done just for ease of use.
The target file for this issue is processing/README.md.

Working on this using the branch processing-readme.

Completed in the above-referenced merge.


Build Warnings - iOS 12 and above

Hi Drew...
I have a bunch of build warnings with Ensembles for iOS. My Deployment target is 12.0. Perhaps it shouldn't be?
Are these planned to be cleaned up?
What's the best way to get these addressed?
I can attempt to offer some assistance if that's the right thing to do?

If you want to add compiler directives to silence the warnings, you can. I am happy to include those via a PR.
Removing the warnings by changing the code is not really an option, because on older OSes the deprecated code is the only option there is. The warnings also don’t appear when you build Ensembles in isolation, because it targets early deployment targets.
Kind regards,
Drew

On 29 Dec 2020, at 04:22, David Wilson<EMAIL_ADDRESS>wrote:

Hi Drew...
I have a bunch of build warnings with Ensembles for iOS. My Deployment target is 12.0. Perhaps it shouldn't be?
Are these planned to be cleaned up?
What's the best way to get these addressed?
I can attempt to offer some assistance if that's the right thing to do?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

If you want to add compiler directives to silence the warnings, you can. I am happy to include those via a PR

Could you explain PR for me please.
Understood regarding changing the code because of support for older OSes.

David

On 29/12/2020, at 10:13 PM, Drew McCormack<EMAIL_ADDRESS>wrote:
If you want to add compiler directives to silence the warnings, you can. I am happy to include those via a PR.
Removing the warnings by changing the code is not really an option, because on older OSes the deprecated code is the only option there is. The warnings also don’t appear when you build Ensembles in isolation, because it targets early deployment targets.
Kind regards,
Drew

On 29 Dec 2020, at 04:22, David Wilson<EMAIL_ADDRESS>wrote:

Hi Drew...
I have a bunch of build warnings with Ensembles for iOS. My Deployment target is 12.0. Perhaps it shouldn't be?
Are these planned to be cleaned up?
What's the best way to get these addressed?
I can attempt to offer some assistance if that's the right thing to do?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/drewmccormack/ensembles/issues/293#issuecomment-752005700, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABVCN76LCGXMUN2UFX52Z3SXGMS3ANCNFSM4VMXQYCQ.

Pull Requests are how open source generally works with contributions. You make your own fork of the master branch at GitHub. You make the changes and commit them your branch. Then you make a pull request and I can accept the changes. You can probably Google how to do these steps.
Kind regards,
Drew

On 29 Dec 2020, at 10:22, David Wilson<EMAIL_ADDRESS>wrote:

If you want to add compiler directives to silence the warnings, you can. I am happy to include those via a PR

Could you explain PR for me please.
Understood regarding changing the code because of support for older OSes.

David

On 29/12/2020, at 10:13 PM, Drew McCormack<EMAIL_ADDRESS>wrote:
If you want to add compiler directives to silence the warnings, you can. I am happy to include those via a PR.
Removing the warnings by changing the code is not really an option, because on older OSes the deprecated code is the only option there is. The warnings also don’t appear when you build Ensembles in isolation, because it targets early deployment targets.
Kind regards,
Drew

On 29 Dec 2020, at 04:22, David Wilson<EMAIL_ADDRESS>wrote:

Hi Drew...
I have a bunch of build warnings with Ensembles for iOS. My Deployment target is 12.0. Perhaps it shouldn't be?
Are these planned to be cleaned up?
What's the best way to get these addressed?
I can attempt to offer some assistance if that's the right thing to do?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/drewmccormack/ensembles/issues/293#issuecomment-752005700, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABVCN76LCGXMUN2UFX52Z3SXGMS3ANCNFSM4VMXQYCQ.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

PR=Pull Request. Doh.
Not doing this often enough to make the translation.
Thank you. I’ll see how I get on in the next couple of days (my day may be spoken for tomorrow).

David

On 29/12/2020, at 10:28 PM, Drew McCormack<EMAIL_ADDRESS>wrote:
Pull Requests are how open source generally works with contributions. You make your own fork of the master branch at GitHub. You make the changes and commit them your branch. Then you make a pull request and I can accept the changes. You can probably Google how to do these steps.
Kind regards,
Drew

On 29 Dec 2020, at 10:22, David Wilson<EMAIL_ADDRESS>wrote:

If you want to add compiler directives to silence the warnings, you can. I am happy to include those via a PR

Could you explain PR for me please.
Understood regarding changing the code because of support for older OSes.

David

On 29/12/2020, at 10:13 PM, Drew McCormack<EMAIL_ADDRESS>wrote:
If you want to add compiler directives to silence the warnings, you can. I am happy to include those via a PR.
Removing the warnings by changing the code is not really an option, because on older OSes the deprecated code is the only option there is. The warnings also don’t appear when you build Ensembles in isolation, because it targets early deployment targets.
Kind regards,
Drew

On 29 Dec 2020, at 04:22, David Wilson<EMAIL_ADDRESS>wrote:

Hi Drew...
I have a bunch of build warnings with Ensembles for iOS. My Deployment target is 12.0. Perhaps it shouldn't be?
Are these planned to be cleaned up?
What's the best way to get these addressed?
I can attempt to offer some assistance if that's the right thing to do?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/drewmccormack/ensembles/issues/293#issuecomment-752005700, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABVCN76LCGXMUN2UFX52Z3SXGMS3ANCNFSM4VMXQYCQ.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/drewmccormack/ensembles/issues/293#issuecomment-752010199, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABVCN2LMVTOP5A5KIID4OTSXGOLDANCNFSM4VMXQYCQ.

I have already used directives to silence warnings in other places. You can search for “push” or “pop” to find these places, and how best to do it.
Kind regards,
Drew

On 29 Dec 2020, at 10:31, David Wilson<EMAIL_ADDRESS>wrote:

PR=Pull Request. Doh.
Not doing this often enough to make the translation.
Thank you. I’ll see how I get on in the next couple of days (my day may be spoken for tomorrow).

David

On 29/12/2020, at 10:28 PM, Drew McCormack<EMAIL_ADDRESS>wrote:
Pull Requests are how open source generally works with contributions. You make your own fork of the master branch at GitHub. You make the changes and commit them your branch. Then you make a pull request and I can accept the changes. You can probably Google how to do these steps.
Kind regards,
Drew

On 29 Dec 2020, at 10:22, David Wilson<EMAIL_ADDRESS>wrote:

If you want to add compiler directives to silence the warnings, you can. I am happy to include those via a PR

Could you explain PR for me please.
Understood regarding changing the code because of support for older OSes.

David

On 29/12/2020, at 10:13 PM, Drew McCormack<EMAIL_ADDRESS>wrote:
If you want to add compiler directives to silence the warnings, you can. I am happy to include those via a PR.
Removing the warnings by changing the code is not really an option, because on older OSes the deprecated code is the only option there is. The warnings also don’t appear when you build Ensembles in isolation, because it targets early deployment targets.
Kind regards,
Drew

On 29 Dec 2020, at 04:22, David Wilson<EMAIL_ADDRESS>wrote:

Hi Drew...
I have a bunch of build warnings with Ensembles for iOS. My Deployment target is 12.0. Perhaps it shouldn't be?
Are these planned to be cleaned up?
What's the best way to get these addressed?
I can attempt to offer some assistance if that's the right thing to do?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/drewmccormack/ensembles/issues/293#issuecomment-752005700, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABVCN76LCGXMUN2UFX52Z3SXGMS3ANCNFSM4VMXQYCQ.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/drewmccormack/ensembles/issues/293#issuecomment-752010199, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABVCN2LMVTOP5A5KIID4OTSXGOLDANCNFSM4VMXQYCQ.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

This is weird. I've performed the clone from githib (as previously I'd been downloading... unzipping ...)... deleted the iOS.xcodeproj from my project and dragged in the new one (from the now cloned download)... and deleted and added the ensembles.framework ... and I don't have the warnings I had before. I've check the deployment targets and they are set to iOS 12 for my project and ensembles.
I don't understand why I'm no longer getting the warnings. It's good not to have them though. :-)
This is new...
When I run my app... I'm getting a ton of these messages...
[general] 'NSKeyedUnarchiveFromData' should not be used to for un-archiving and will be removed in a future release
And if I've done my breakpoints properly they're coming from CDESaveMonitor.m - line 155.
if (![eventContext save:&error]) CDELog(CDELoggingLevelError, @"Error saving event store: %@", error);

David

OK, I've ignored the warnings for deprecated code.
One warning left to track down.
The compiler is not pointing me to any line of code. I'm assuming the issue is in "CDEPropertyChangeValue" though I can't see what the issue might be.
CDEObjectChange.propertyChangeValues is using a nil or insecure value transformer. Please switch to NSSecureUnarchiveFromDataTransformerName or a custom NSValueTransformer subclass of NSSecureUnarchiveFromDataTransformer [2]

David

Pull Request created.
I've never done that before... hopefully I've been through and done the right things.
I have not addressed "CDEObjectChange.propertyChangeValues is using a nil or insecure value transformer." referenced in the last message.


Feature: Dark mode and user styles

Initial implementation for comment.

I'm expecting that the colors I've picked for dark mode (especially the accent colors) may need some feedback, but it's a starting point.
Thanks again for your fantastic frontend project!

A few example screenshots:

I'm closing this in favor of current develop branch changes.


Unclear how to add config for multiple backends

Currently I am using vault injector plugin for helm and I would like to be able to inject secrets via kustomization as well.
I am using an init sidecar for helm and my values for it look like this:
initContainers:
   - name: download-tools
     image: registry.access.redhat.com/ubi8
     env:
       - name: AVP_VERSION
         value: 1.11.0
     command: [sh, -c]
     args:
       - >-
         curl -L https://github.com/argoproj-labs/argocd-vault-plugin/releases/download/v$(AVP_VERSION)/argocd-vault-plugin_$(AVP_VERSION)_linux_amd64 -o argocd-vault-plugin &&
         chmod +x argocd-vault-plugin &&
         mv argocd-vault-plugin /custom-tools/
     volumeMounts:
       - mountPath: /custom-tools
         name: custom-tools
 extraContainers:
   - name: avp-helm
     command: [/var/run/argocd/argocd-cmp-server]
     image: quay.io/argoproj/argocd:v2.4.8
     securityContext:
       runAsNonRoot: true
       runAsUser: 999
     volumeMounts:
       - mountPath: /var/run/argocd
         name: var-files
       - mountPath: /home/argocd/cmp-server/plugins
         name: plugins
       - mountPath: /tmp
         name: tmp-dir
       - mountPath: /home/argocd/cmp-server/config
         name: cmp-plugin
       - name: custom-tools
         subPath: argocd-vault-plugin
         mountPath: /usr/local/bin/argocd-vault-plugin

The configmap loolks like this:
apiVersion: v1
kind: ConfigMap
metadata:
  name: cmp-plugin
data:
  plugin.yaml: |
    ---
    apiVersion: argoproj.io/v1alpha1
    kind: ConfigManagementPlugin
    metadata:
      name: argocd-vault-plugin-helm
    spec:
      allowConcurrency: true
      discover:
        find:
          command:
            - sh
            - "-c"
            - "find . -name 'Chart.yaml' && find . -name 'values.yaml'"
      generate:
        command:
          - bash
          - "-c"
          - |
            helm template $ARGOCD_APP_NAME -n $ARGOCD_APP_NAMESPACE -f <(echo "$ARGOCD_ENV_HELM_VALUES") . |
            argocd-vault-plugin generate -s vault-configuration -
      lockRepo: false

My question is, how can I add Kustomize as well? Do I need to run another side car and add to the CMP? Can I just add config to the CMP and then add another mountpath in my extra container?
Please advise

I decided to just go with installing 3 side cars and it appears to be working.


Allow rag file paths to be read from file

Is your feature request related to a problem? Please describe.
Imagine you have a directory that contains both *.md files as well as  *.excalidraw.md files. You want to rag the *.md but not the *.excalidraw.md files. Currently, it is not possible to include just the *.md files with the
add_documents can load complex document paths as there is no way to exclude files:

This problem extends to the case where we have a directory with many subdirectories, where we want to include some, but not others.
Typing out all of the paths at the CLI is impractical.
Describe the solution you'd like
Load paths from a pre-generated file e.g. paths.txt:
/home/gandalf/notes.md
/home/gandalf/spells.md
...

That can be generated e.g. with a command like
find . -type f -not -path '*/.obsidian' -not -path '*/.git' -not -path '*/_media' -iname '*.md' > paths.txt

We will not support this feature.
Most well-designed document directories can be loaded directly with format dir1/ or dir1/**/*.{ext1,ext2}.
If there is really no other way, add_documents allows input of arbitrary paths.
paths.txt  doesn't fit well into the current workflow.
The solution
1. Collect paths
find . -type f -not -path '*/.obsidian' -not -path '*/.git' -not -path '*/_media' -iname '*.md' | tr '\n' ';'

2. Copy the text and paste it into the add_documents

There is a minor issue with a trailing semicolon, but we will fix it. Now, simply remove the trailing semicolon.

How to handle a project directory which usually contains .git/?
It looks reasonable to provide a way to define excluded globs.
Maybe even add .gitignore support, to avoid processing files with secrets, binaries, caches, etc.


Cannot install on mac using brew

The brew install thefuckis not working here.
There isn't any formula found for thefuck (brew search thefuck).

Make sure your Homebrew is up-to-date.

brew update && brew install thefuck

Should help you

@jonibatista ping, any update on this?

It's working fine for me now :)
Sorry for taking so long to replay @scorphus

@jonibatista If your issue has been resolved, you can close this issue. :)


Fixed typo described on issue #45

I've updated podspec to correct a little typo when 's' is used instead of 'spec' as described on issue #45

Thanks @vbergae 👍


Direct API for nocopy operations on sender-side and receiver-side

Original issue: https://charm.cs.illinois.edu/redmine/issues/1667

Since the receiver side API (https://charm.cs.illinois.edu/redmine/issues/1236) has restrictions which demand the user to name a generated entry method in a specific manner in order to overload the generated method, it will be a worthwhile effort to have an alternative API which is more direct as compared to entry method overloading and gives user the control to perform GET/PUT calls directly.

Original date: 2017-08-29 20:45:06

Initial commit: ~~https://charm.cs.illinois.edu/gerrit/#/c/2967/~~ https://github.com/UIUC-PPL/charm/commit/504bc22b81afa75788cb0e38974f0268e52ecec0

Original date: 2017-08-29 21:32:32

We may want to avoid calling this "direct" since there is already something else called CmiDirect. I think in general the recv-side stuff is the completion of the Zero Copy API, but the fact that we will eventually have 2 APIs for the recv-side stuff will make it a bit confusing (I think of this API as the lower-level one, and the RdmaPost one as the higher-level abstraction).

Original date: 2017-08-29 21:48:34

If the generic layer gets done, possibly even with a hacky implementation that only really provides one of GET or PUT and uses that for the other, then we can implement the other layers piecemeal after the basic thing is integrated.

Original date: 2017-08-29 21:49:29

And similarly, that one-of-GET-or-PUT design can get us up and running on the other layers quickly, though, with an extra round trip, and then be optimized later.

Original date: 2018-04-04 19:26:04

Except for documentation for the Direct API, which is targeted for 6.9, all the other tasks are for later releases.

Original date: 2018-09-20 18:28:46

Netlrts will not have a true "zero copy" implementation, it'll use the generic copy based implementation.

Original date: 2018-09-20 18:33:02

We could potentially use sendmsg and WSASend on netlrts-tcp as Juan did here, though I don't think it should be a priority right now:
~~https://charm.cs.illinois.edu/gerrit/#/c/charm/+/4357/~~ https://github.com/UIUC-PPL/charm/commit/4af1d868a045935c5401856e47df919b0cd5ce95
~~https://charm.cs.illinois.edu/gerrit/#/c/charm/+/4535/~~ https://github.com/UIUC-PPL/charm/commit/6c657aa5121990a40f56d7cd963f6924ab487c18


If we have two modules:

Module A provides custom backbone.js models
Module B provides custom backbone.js collections

Then we have our app which:

Requires Module A
Requires Module B
Creates a model instance using a custom model from Module A
Creates a collection instance using a custom collection from Module B
Adds the model instance to the collection instance

Then the sky falls down and burns all the horrible plagues at once. Why? I know the answer, but I'll give you a gold star if you can guess it.
Here's the answer: because our lovely model instanceof Model inside Backbone.Model.prototype._prepareModel fails, as the models are using one backbone.js and the collections are using another!!!! Despite them being the same backbone.js version.
ARGH! That line needs to be better, perhaps check for the presence of a this.isBackboneModel variable defined by the Backbone.Model.prototype.
ARGHHHH. For now, it seems the only solution is to pass the Modules the Backbone.js module that we want them to use, rather than having them use their own local copies. Which is a real PIA. Which imho, the flag idea is a much better one.
Thoughts?

For anybody that is using webpack. I had a problem related to the instanceof fail using with-backbone. I could only solve by forcing Backbone globally in the webpack config.
    resolve: {
      alias: {
        backbone: path.resolve(
          path.join(__dirname, "node_modules", "backbone")
        ),
        Backbone: path.resolve(
          path.join(__dirname, "node_modules", "backbone")
        ),
...
      },
    }


Multidimensional array in Illuminate\Support\MessageBag

Laravel Version: 5.7.x-7.x
PHP Version: 4.0.1-7.4

Description:
Illuminate\Support\MessageBag does not support multidimensional errors. I guess that because of using PHP array_unique in the constructor method.
Steps To Reproduce:
    $errors = [
        'sigup' => [
            "country" => [
                "The country must be a string."
            ],
            "zip" => [
                "Zip code is required"
            ]
        ]
    ];

   new \Illuminate\Support\MessageBag($errors);

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

Not sure why they would close this, but what you are looking to achieve is simple, just use
$errors->getMessages()


Typo mistake in the readme

Thank you for taking the time to file a Bug report.
Describe the bug
In the readme, it is written Journal Policy Tracker Backend.
To Reproduce
Edit the readme file and change the text to Journal Policy Tracker Frontend.
Screenshots

Please go ahead and make a PR to fix it! :)


Add list of related software.

Closes #67.

Maybe split this up into sections of software for demes (demes-python, demes-draw, etc) and software that uses demes?

(maybe we add a note at the bottom saying "if you'd like to add your software to this list, please open a Pull Request"?)

There's a "click here" link at the top of the section which takes the user directly to a "edit this page in github, which has a markdown preview" (ie. https://github.com/popsim-consortium/demes-spec/edit/main/docs/introduction.md). Would you prefer it was linked under the list?

It's not so much the "how" I have in mind, it's just encouraging people to actually do it. I think we're a lot more used to open development that others, and they might not realise that we actually want them up update the list.


Ears are never getting detected.

LEFT_EAR_TIP, LEFT_EAR, RIGHT_EAR_TIP, RIGHT_EAR are never getting detected.

Are other landmarks being detected?
What is the image resolution and face size within the image?

The resolution is same as the one in "FaceTracker" example.

Yes, Ear landmark is never getting detected.

Other 8 landmarks are coming but not able to detect ear landmarks from getlandmark() method.

For IOS ,they are able to detect the ear position properly but for android it is not .
Please answer for this, i have spend around 2 months for this issue.

Can you provide the sample image you're trying to detect the face with ?

These are all the images am trying to detect the ear position , for ios the ear position coming properly but for android it is not coming .Please answer for this orelse we can't complete this project.
and one more thing is Other 8 landmarks are coming but not able to detect ear landmarks from getlandmark() method in android.

Can you also provide your FaceDetector configuration?
Can you also try the following configuration?
FaceDetector detector = new FaceDetector.Builder(context)
.setTrackingEnabled(true)
.setMode(FaceDetector.ACCURATE_MODE)
.setLandmarkType(FaceDetector.ALL_LANDMARKS)
.build();

duplicate: https://github.com/googlesamples/android-vision/issues/174
Really sorry for the inconvenience, Android Face API doesn't support ear detection for now.

I have tried your above code for my image
plz look at the green circle .only these things are coming . but i want that for ear.
here i have attached the sample image and code too.
plz take a look.
FaceDetector detector = new FaceDetector.Builder(getApplicationContext())
.setTrackingEnabled(true)
.setMode(FaceDetector.ACCURATE_MODE)
.setLandmarkType(FaceDetector.ALL_LANDMARKS)
.build();

That image works fine in my test App.
Can you output all landmark types, and check if LEFT_EAR is detected?
The top right landmark looks like a wrongly positioned LEFT_EAR to me.

Could you plz send the exact code that you have used to detect the ear position?
Thanks..

@PremMayilai Did you ever figure out how to get the scale to properly place the landmarks in real time video? I have tried multiple things but no luck.

@PremMayilai Did you ever figure out how to get the scale to properly place the landmarks in real time video? I have tried multiple things but no luck.

Ears are not detected till now. Using play-services-vision: 11.8.0

Is there any way to detect the ears. Same as above images.

I am also facing the same issue, is there any solution yet?


Changed page_content padding to 15px in _markdown.scss

@palewire as you suggest i have changed the padding in scss file.

This is the right spot to find this kind of padding, no doubt. But I think an issue is that our tangled CSS means more fixes will be necessary. I am going to do a refactoring all the container code shortly, which should make this fixes more clear and direct. Once I get it to that point I'll flag you so you can make that patc.


ruff

ruff all the things.
Solves

#30

You got rid of isort, but you also have to purge all things flake8, We don't need that if we have ruff.

but you also have to purge all things flake8

😊 someone didn't read the issue title very carefully.


Expected a conditional expression and instead saw an assignment.

Codacy detected an issue:
Message: Expected a conditional expression and instead saw an assignment.
Currently on:

Commit:  278bdc471b899bb62c99b57a7f74237328223e0e
File:    doc/integrations/myDrive/public/fontawesome/js/fontawesome.min.js
LineNum: 5

For the convenience of the Seagate development team, this issue has been mirrored in a private Seagate Jira Server: https://jts.seagate.com/browse/CORTX-31577. Note that community members will not be able to access that Jira server but that is not a problem since all activity in that Jira mirror will be copied into this GitHub issue.

Amnuay Boottrakoat commented in Jira Server:

Inspect by Stylelint no issue and merged to main.

Amnuay Boottrakoat commented in Jira Server:

Merged to main per PR#1535

Amnuay Boottrakoat commented in Jira Server:

closed


Allow to specify if date is UTC

Currently everywhere in your code you have moment(value). It will cause issues if I don't want to use local time.  The solution would be to add isUTC props and change moment() calls into moment.UTC() if isUTC is set to true. E.g. here
getValidDateOrCurrent = () => {
    const date = moment(this.props.value);

    return date.isValid() ? date : moment();
  }

Yep, pickers would work with your local time. And from where I stand they should. If you wont to create something like timezone control you should create wrapper of Picker, smth like
const DateTimePickerWithTimeZone = props => {
  const { timezone, input: { onChange, value }, meta: { touched, error }, dispatch, ...other } = props
  const valueWithTimeZone = momentTz.tz(value, timezone)

  return (
    <DateTimePicker
      error={!!(touched && error)}
      helperText={touched && error}
      value={valueWithTimeZone}
      onChange={onChange}
      {...other}
    />
  )
}


API documentation link in Readme / Shoppe Website broken

When I click the API documentation link it sends me here: https://s.adamcooke.io/files/
And given that this file contains the following:
Password: dmMtAmyqbVkutifMcgng8hQmniU6VB

It probably shouldn't be public.
Are there real api docs somewhere I'm not seeing?

The API site is working fine for me - http://api.tryshoppe.com

Oh weird, when I opened the link on my home laptop, I got redirected, but on a different machine, it sends me to the correct link.  Probably something wrong on my end.

Not sure, maybe @adamcooke quickly fixed it!


Bilingual jargon

Jargon: using my own private jargon will be viewed as excentric by others.
To be more courteous, always use parens to allow others to understand what the heck I'm talking about:  "kinematic rotation (Wigner rotation)".

Done.


Instability Stops working

Here is a video of it stopping: https://youtu.be/EtazaEPGQzk
It will even fail to relaunch. I am using a screen recorder Open Broadcast Streamer https://obsproject.com/
While the screen is being recorded it works for about 45 seconds at a time then freezes.

I have fixed some bugs and added new features, but that special bug did only occured once during testing and i could not reproduce it.
Maybe it is fixed, maybe not.

hopefully fixed with v1.0.0


Lower the length of timestamp column

If a database has a limit of 1000 bytes for indexes, the timestamp column will in utf8 be too long and throw an error when trying to create the migration table. This suggested change lowers the limit to 64, which should be more than enough for this field.

This should perhaps even be changed to an int field.

Will you be interested to be a collaborator of this repository? Pls give your email address if you are interested.

Possibly, what would that entail exactly?

You will have ownership rights for this repository. You can merge code, update the package, add new features. I will also provide npm access to update the npm package as well.

I think I'll pass. I don't have the time I'm afraid. I'll keep doing PR's when I find something.

Is there a reason why this has not been merged yet?


chore: migrate QueryTable component from jsx to tsx

SUMMARY

updated Query type at superset-frontend/src/SqlLab/types.ts/Query and extended it into QueryTable
updated defaultQueryLimit to be optional at ResultSetProps of superset-frontend/src/SqlLab/components/ResultSet/index.tsx
moved statusAttributes into useMemo hook so that it will not make the dependencies of useMemo Hook change on every render and statusAttributes is only used within useMomo hook
QueryTable is used in the following places

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up

@yousoph and @rosemarie-chiu Meitong added screenshots for qa testing


exclude not used infinispan transitive dependencies no longer availab…

Following suggestions of infinispan team I replaced this PR with https://github.com/kiegroup/kogito-runtimes/pull/669


Auto-complete annoyance

If I try and type the following
select *
from Table1 t1
join Table2 t2 on t1.Id = t2.Table1Id

When I type the alias t1 for Table1, the autocomplete list includes an entry "Table1" which is selected so as soon as I hit enter t1 is replaced with Table1 leaving
select *
from Table1 Table1

It should recognise the t1 is an alias and not try and autocomplete

I don't see Table1 in the autocomplete list when typing the above snippet on the latest build.  Please let us know if this issue is still impacting you.


Added PD CSI Driver Addon support

Google supports CSI Driver for PD (https://cloud.google.com/kubernetes-engine/docs/how-to/gce-pd-csi-driver) which is also supported by TF Google Beta Provider (https://www.terraform.io/docs/providers/google/r/container_cluster.html#gce_persistent_disk_csi_driver_config)

Thanks for this PR!


Add a jason data to an existing sheet.

Hi guys,
From previous example, we know that after reading a file, we can add new data into that particular sheet:
var XLSX = require('xlsx');
var wb = XLSX.readFile('sheetjs.xlsx');
add_cell_to_sheet(wb.Sheets.SheetJS, "F6", 12345);
XLSX.writeFile('sheetjs-new.xlsx', wb);

But how to do if my data is not just a single cell? I have a json data that I would like to add into the file that I read. How to add the json data?

XLSX.utils.sheet_add_aoa takes an array of array and XLSX.utils.sheet_add_json takes an array of objects.

After we read a file, it is under a workbook right?
But the sheet_add_json doesn't specify the workbook name:
sheet_add_json(ws: WorkSheet, data: any[], opts?: SheetJSONOpts): WorkSheet;
It only specify the worksheet name.

A literal conversion of the first example would be:
var XLSX = require('xlsx');
var wb = XLSX.readFile('sheetjs.xlsx');
XLSX.utils.sheet_add_json(
  wb.Sheets.SheetJS, // worksheet
  [{number: 12345}], // data
  { // options
    header: ["number"], // properties to read from each object
    skipHeader:true, // do not include header row
    origin: "F6" // start from cell F6
  }
);
XLSX.writeFile('sheetjs-new.xlsx', wb);

This example writes F6="a", F7="b", G6=1, G7=2:
XLSX.utils.sheet_add_json(
  wb.Sheets.SheetJS,
  [
    {text: "a", number: 1},
    {number:2, text: "b"}
  ],
  {
    header: ["text", "number"],
    skipHeader:true,
    origin: "F6"
  }
);

Can explain "wb.Sheets.SheetJS"?
wb=workbook that we read
How bout Sheets.SheetJs? Is it the sheet name?

Ok. I can add a json file into my read workbook already.
But the workbook that I read did not preserved the styles like bold word or underline. How to preserved the styles?

You would need the SheetJS Pro version in order to access the styles.
https://sheetjs.com/pro

@harbhub  and what is the price of the pro version ?

It varies based on the use case.  You'd have to email their support team at<EMAIL_ADDRESS>to find out.

@NikicaCh ^

@harbhub, I emailed them this morning, but still waiting for the reply, and was wondering if it is to expensive.I don't need it for something big, just for populating some daily report templates and maybe invoices and nothing more.

@NikicaCh unfortunately I don't know how quickly you can expect a response, but you can try to expedite the process by joining the IRC webchat.  Sometimes a representative is present there, in which case you can ask about your email and have it attended to more quickly.  The official channel is #SheetJS

@harbhub thanks for the reply again.They responded later that day, but I am not ready to pay the sum they ask since it is a simple project I build for myself and not a client.
Do you know a alternative to this package, something that will do the job for me ?

@NikicaCh I'm sorry to hear that.  Unfortunately, I do not know of any comparable library for handling spreadsheets in JavaScript.  SheetJS is the most robust solution to my knowledge.
Did you specify that this was for personal use in your email to their support team?

this is what working with me..after getting error
var ext = o.file.slice(o.file.lastIndexOf(".")).toLowerCase();                                       ^ TypeError: o.file.lastIndexOf is not a function
`
var wb = XLSX.readFile('./data.xlsx');
const ws = XLSX.utils.sheet_add_json(
wb.Sheets.Sheet1, // worksheet
[{ number: 12345 }], // data
{ // options
header: ["number"], // properties to read from each object
skipHeader: true, // do not include header row
origin: "A3" // start from cell F6
}
);
XLSX.utils.book_append_sheet(wb, ws)
XLSX.writeFile(wb, 'data.xlsx');

`


ci: pass race-detection env vars to vagrant boxes

test-race-runtime

test-race-upstream

test-race-upstream-k8s

test-race-runtime

test-race-runtime

test-race-upstream-k8s

test-race-upstream-k8s

test-race-upstream-k8s

test-race-upstream-k8s

test-net-next

test-race-runtime


Feature Request: Weapon Dimensions

It would be convenient to see how large weapon dimensions are for research away from my PC. It would fit right into the weapon description, and would save time for users.

To clarify, are you asking about how many blocks of space it takes over from your inventory?
Or literal dimensions ( I assume this isn't what you asked for but I just wanted to ask anyways)


$state.params and $stateParams are different during resolve phase of ui-router.

I made this demo to demonstrate the issue; while resolving, $state.params is not populated, while $stateParams is populated. As far as I can tell, they should be the same thing.

I agree they should be. Although if they are, what is the point of having two things that are the same?

$stateParams is a "locals object" injected into states during a transition using $injector.invoke().  It is specially filtered to only those params that are declared on the state it is being injected into.
$state.params is updated only once the transition promise has successfully resolved, like $state.current
Confusing, yes, but this is how it is supposed to work.


Merge branch '4.10.x' into 5.0.x

* 4.10.x:
  (PUP-7742) Speed up iteration by resolving method only once
  (PUP-7742) Ensure getpwent and getgrent iterators are properly closed
  (PUP-7752) amazon rel ver to service test

👍

I merged 4.10.x to 5.0.x in 1481600, closing this.

Follow up from 5.0.x to master in PR #6064


ARTEMIS-2415 JDBCJournal miss pending tasks during shutdown

Wait deleting large message tasks during stop.

@franz1981 wdyt to also add a timeout for deletingLargeMessageTasks.wait?

-1 adding any form of wait blocking or locking on shutdown

@clebert why did you merge this?

Im tempted just to revert on master

This is dangerous blocking stop. Also it doesnt solve the problem if the broker is killed anyhow. As mentioned really the broker should clear up lose ends on start up. This way it doesnt block stop and also fixes the case if its killed

I did not realize your -1, and I confused this wait with a similar change on FDs.
I'm reverting it.
@brusdev  can you address the issue with a new PR? a restart of the broker should remove any previous messages that were not deleted.


Use alexandria

We use cffi, cffi uses alexandria. So this is a no cost decision

we do. Arguably we need to replace more of our utils with it, but that'll happen naturally


Perform regexp escape on individual words before joining them

To complete changes discussed in #8, we need to change Regexp escaping a little.
Specifically, current implementation performs escape after joining all query argument into a string. For example, while running pod search --full network library command, a string like "network\ library" will be generated (they joined with space character, then whole string is escaped). This cause a problem inside SourcesManager. Because it needs to search for each query word individually and then return an intersection of individual results, it needs to split given query string to get individual words.
Proposed implementation performs regexp escape on individual query arguments first, then joins escaped strings with space character.

👍🏻 with specs and CHANGELOG

Good to go from me :rocket:


Added info about calling generic methods to the Razor syntax topic

Most of the necessary background on this PR is provided in an Issue that I created, linked below:
Fixes #4575
I wasn't quite sure on the syntax for relative links to sections within the same topic, so someone will want to review those.
Suggested Reviewers

@Rick-Anderson
@guardrex
@NTaylorMullen

@Rabadash8820,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@Rabadash8820, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

Okay I'm still a little new to PRs.  I accidentally deleted the branch on the PR above too soon... reopening it now!

@Rabadash8820,
Thanks for having already signed the Contribution License Agreement. Your agreement has not been validated yet. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@guardrex No problem!  So this is one of the more complicated PRs I've ever made on GitHub.  Do I approve your suggested edits through GitHub or do I commit/push the edits manually?

Make the updates to your local branch ... then push those.
What we usually do on these is that for each comment the reviewer makes, just tick it off with a :+1: if you agree and made the change on your branch as you go. If you come to one you don't like (i.e., you don't want to make the change because you think the green dinosaur :smile: is wrong, then just mark it with a :-1: and reply on that line with your concerns about the reviewer's comment. Usually, the reviewer and the author just work it out right there.
When ur done and you commit your local changes to your branch, it will ping the reviewer to take another look at the PR. Then, they can approve the changes or add additional comments (i.e., a 2nd round of review).

@Rabadash8820  Thanks for responding to @guardrex . Can you hold off making more edits for the next hour? I'm going to make some edits.

Yes, sorry @Rick-Anderson. Didn't mean to jump in tooooo fast.

@guardrex A couple other questions about PRs for this repo:

Can you provide a link to the MS Style manual?
When making suggested edits, is it ok to force-push over the original commit?  I didn't do that here, but I could see a more complicated PR having multiple little micro-commits cluttering up its history, and I like squash those together in my own projects 😛
Is it better to put the "fixes #issue" syntax in the commit message, PR message, or both?

Can you provide a link to the MS Style manual

It's not out yet. They're working on a public version the last I heard.

I like squash those together in my own projects

All of the commits are squashed on merge, so you don't need to take any action on your end as you make multiple commits. It will all clean up on merge.

Is it better to put the "fixes #issue" syntax in the commit message, PR message, or both?

Put "Fixes #issue" for each issue (separately) that the PR fixes in the PR message (the first comment of the PR). If you need to merely "address" (i.e., not close an issue when the PR merges), then use "Addresses #issue"

btw ---- For some items not in the contribution guidelines for the repo at ...
https://github.com/aspnet/Docs/blob/master/CONTRIBUTING.md
... I touched on them in https://github.com/aspnet/Docs/issues/2617. I think that's mostly correct. I put it up before I had access to the official style manual, so it might need a touch here and there. These were things that weren't in the contribution guidelines but that I needed to clarify for working on topics.

Put "Fixes #issue" for each issue (separately) that the PR fixes in the PR message (the first comment of the PR).

Okay, so we don't need it in the commit message.  That makes sense; I've definitely seen large PRs in other repos where it doesn't make sense to say that any single commit closed the issue.

we plan to link to the MS style guide which should be coming out soon.

commented @Rick-Anderson on Jan 31.  Lol clearly public style guides are not high-priority items

clearly public style guides are not high-priority items

no comment :grinning: lol

@Rabadash8820  really great work. Can you check my edits?

@Rick-Anderson I think you still need to distinguish that there are two separate compiler errors when using generic methods in implicit expressions (see my comment on your previous commit).  I sometimes saw either error or both.  You also set them apart with bullet in the implicit expression section, but only with a newline in the explicit section.  Seems like you should have two separate bullets in both sections.

imo single bullets (single bullet lists) look strange ... a "list" of one item? lol Mrs. Calvary, my dearly departed 7th grade English teacher will rise and come after us this 🎃 Halloween :skull: lol
Also, this content isn't truly a list. In this case, we have quoted exception messages. I suggest blockquotes for quoted content.

@Rabadash8820  check now. I thought it was one big ugly error.

@Rick-Anderson  Looks good!  The only things that might still need addressed are these:

As per @guardrex 's and my conversation earlier, maybe say:

An implicit expression cannot contain C# generics, as the characters with the brackets (<>) is interpreted as an HTML tag.

Whether those compiler errors should be placed in blockquotes or codified, rather than (or in addition to) being in bullets
Maybe use that fancy > [!WARNING] syntax to make the warning at the end of the Code Blocks section stand out more.

As per @guardrex 's and my conversation earlier, maybe say:

An implicit expression cannot contain C# generics, as the characters with the brackets (<>) is interpreted as an HTML tag.

That's in there.

As per @guardrex's previous comment, whether those compiler errors should be placed in blockquotes or codified, rather than (or in addition to) being in bullets

That makes them hard to read on small screens.

Maybe use that fancy > [!WARNING] syntax to make the warning at the end of the Code Blocks section stand out more.

We like to avoid fancy notes, it turns out so do readers.
Thanks for your valuable input and review of my edits.


VD-42 removing external links for mapveu

this is in part related to:
https://redmine.apidb.org/issues/39536

Tests can be made at:
https://dkwon.vectorbase.org/popbio-map/web/

implemented Bob's suggestion


Add storage directory to release packages

Proposed Change

Stateful receivers will likely make use of the storage extension. For example, reading log files (saving state and buffering).
Linux

Added directory /opt/observiq-otel-collector/storage
Added environment OIQ_OTEL_COLLECTOR_STORAGE pointing to the directory, allowing configs to read $OIQ_OTEL_COLLECTOR_STORAGE instead of hard coding the value.

Windows

Added storage directory to install directory. The .include file is required for WIX to copy the directory, afaik.

Windows test output
  File C:/Program Files/observIQ OpenTelemetry Collector/plugins
     [PASS]  should exist
     [PASS]  should be directory
  File C:/Program Files/observIQ OpenTelemetry Collector/storage
     [PASS]  should exist
     [PASS]  should be directory
  File C:/Program Files/observIQ OpenTelemetry Collector/observiq-otel-collector.exe
     [PASS]  should exist
     [PASS]  should be file
  File C:/Program Files/observIQ OpenTelemetry Collector/config.yaml
     [PASS]  should exist
     [PASS]  should be file
  File C:/Program Files/observIQ OpenTelemetry Collector/plugins/aerospike.yaml
     [PASS]  should exist
     [PASS]  should be file
  File C:/Program Files/observIQ OpenTelemetry Collector/plugins/microsoft_iis.yaml
     [PASS]  should exist
     [PASS]  should be file
  File C:/Program Files/observIQ OpenTelemetry Collector/plugins/zookeeper.yaml
     [PASS]  should exist
     [PASS]  should be file
  Service observiq-otel-collector
     [PASS]  should be installed
     [PASS]  should be enabled
     [PASS]  should be running

Test Summary: 17 successful, 0 failures, 0 skipped

Other
Also updated make release-test to build with --snapshot, allowing the package to be installed over the previous tagged release. Useful when testing locally.
Checklist

[ ] Changes are tested
[ ] CI has passed

Does this need to be added to homebrew also?

I pushed a commit, not sure how I feel about it. We need to bundle storage/.include with all archives to pull this off.

@cpheps I decided to revert the homebrew change for a couple reasons.
I could not successfully create the directory (probably ignorance on my part, with Ruby and Homebrew)
If created in the homebrew install directory, the user will need to identify that directory when configuring the storage extension. I think it is reasonable to expect the user to create their own storage directory instead. Ideally, the directory would live outside of homebrew's scope, allowing upgrades to occur without modifying the storage directory. Further, macOS is generally not used as a long term solution, while Linux and Windows are more production oriented. I think it is expected that we provide a storage directory on those platforms.
I did test the homebrew install directory when configuring a logging pipeline with storage, it works well in the root of the install directory if the user really wants to avoid creating a directory.


show snapshots in datetime-tree in fuse-mount

Output of restic version
restic 0.9.6 (v0.9.6-0-gb723ca3) compiled with go1.13.8 on linux/386

What should restic do differently? Which functionality do you think we should add?

Restic should either represent the snapshots in a directory tree with date and time as separate path elements or show tags with slash as subdirectories under tags directory in fuse mount.
By separate date and time path elements I mean eg. snapshots-by-date/2020/08/31/08/15 for a snapshot which was created at 2020-08-31 08:15:ss. It may be configured, because everyone has different snapshotting frequency: who does snapshots hourly he may want this 5 level depth, who does it daily he may want only 3 levels, etc. The last directory (before the snapshot's content) can be the snapshot ID to distinguish snapshots with the same date/time up to the indicated precision: snapshots-by-date/2020/08/31/SNAPSHOTID/home/joe/backup/....
But IMO it'd be easier to show tags with slash in their name as directory structure. Currently tags with slash are not shown in fuse mount. In this way users could organize huge number of snapshots simply by tags.
What are you trying to do? What problem would this solve?
I have many snapshots (eg. for every hour in every day kept for years) and directory listing in fuse mount loads slowly and navigating through it by file managers are also challenging. I like to have directory structure which I can browse easier, ie. less entries per directory. IMO restic should show tags in dir structure (eg. tags/2020/aug-31/... for tag "2020/aug-31") in fuse mount.

Did restic help you today? Did it make you happy in any way?

yes, absolutely, i can not emphasize more.

I've thought about having a date path too, it would be very useful and effective. I think it should start with /dates/.. though to keep it consistent with the other directory names though (tags, hosts, snapshots, etc).

One reason of restic mount being slow on showing snapshots might be that restic always needs to read all snapshot files. A different directory structure would not solve this problem.
About the directory structure:
Actually there is already --snapshot-template which allows to specify how snapshots are listed under /snapshots/. It has the same problem as tags with slashes: It doesn't work (and here also prints an error message). So I would propose to change mount such that slashes in names (tags and in --snapshot-template) simply work.
I can work on a PR if we agree that this is the way to go..

I realized that implementing this also allows a lot of simplifications on the fuse code. The current state is in a draft PR now. I'll refine this and add the missing latest link soon...

PR #2913 is now in a state that it can be tested.
@bAndie91 Your requirements can be fulfilled using #2913 with
--dir-template "snapshots-by-date/%T" --time-template "2006/01/02/15/04"

and
--dir-template "snapshots-by-date/%T/%i" --time-template "2006/01/02"

Looking forward to getting feedback!

does it reduce the load time?

$ restic version
restic 0.9.6 (v0.9.6-0-gb723ca3) compiled with go1.13.8 on linux/386
$ time ls ~/mnt/restic/snapshots/
real 254.645    user 0.204      sys 0.268       cpu 0.18%
$ time ls ~/mnt/restic/snapshots/
real 268.972    user 0.232      sys 0.404       cpu 0.23%

$ restic version
restic 0.9.6 (v0.9.6-395-gc670046) compiled with go1.13.8 on linux/386
$ restic mount ~/mnt/restic/ --path-template snapshots/%T --time-template 2006/01/02 &
$ time ls ~/mnt/restic/snapshots/
real 0.134      user 0.000      sys 0.000       cpu 0.00%
$ time ls ~/mnt/restic/snapshots/
real 0.072      user 0.004      sys 0.000       cpu 5.56%
$ time find ~/mnt/restic/snapshots/ -mindepth 3 -maxdepth 3
real 4.604      user 0.044      sys 0.148       cpu 4.17%
$ time find ~/mnt/restic/snapshots/ -mindepth 3 -maxdepth 3
real 2.556      user 0.056      sys 0.172       cpu 8.92%

it is faster. excellent work. thanks.
when will it be stable? when a version tag is added?

i haven't found docs on the macros but found "%i", "%I", "%u", "%h", "%t", "%T" in the code for short and long snapshot IDs, username, hostname, tag, and time respectively.

(ok it's in  #2913 's description)

Thanks for trying out #2913 @bAndie91 :+1:
About your measurements: It seems to me you compared #2913 (on top of the current master branch) with version 0.9.6. Since version 0.9.6. there has been quite some improvements to the FUSE mount which already made it into master. I think the improved speed is mainly due to these changes.
To get #2913 stable, it must be reviewed by a project maintainer and get merged into the master branch first.
Then it will be included in the next release to come after the merge.
So I would advise you to leave this issue open until #2913 is merged. (Usually the issues are closed by maintainers during the merge)