SqlServerDsc: Enabled PSSA rule violations to fail build

Pull Request (PR) description

Changes to SqlServerDsc

Enabled PSSA rule violations to fail build.

This Pull Request (PR) fixes the following issues
n/a
Task list

[x] Added an entry under the Unreleased section in the CHANGELOG.md? Entry
should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md?
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help?
[ ] Comment-based help added/updated?
[ ] Localization strings added/updated in all localization files as appropriate?
[ ] Examples appropriately added/updated?
[ ] Unit tests added/updated?
See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible)?
See DSC Resource Testing Guidelines.
[x] New/changed code adheres to
DSC Resource Style Guidelines
and Best Practices?

This change is

Codecov Report

Merging #1102 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff          @@
##            dev   #1102   +/-   ##
====================================
  Coverage    97%     97%
====================================
  Files        32      32
  Lines      3930    3930
====================================
  Hits       3844    3844
  Misses       86      86

@PlagueHO can you help me look at this PSSA rule warning/error? I have not seen this warning on this resource before! :s  Is this a false positive? 🤔
https://ci.appveyor.com/project/PowerShell/sqlserverdsc/build/9.0.264.0?fullLog=true#L1258
MSFT_SqlServiceAccount.psm1 (Line 409): Cmdlet 'Get-ChildItem' may be used incorrectly. Please check that all mandatory parameters are supplied.

I can't see any problem with the code. Am I missing something?
https://github.com/PowerShell/SqlServerDsc/blob/244634e783faca7c711112ff4b37ba5129a1a2e2/DSCResources/MSFT_SqlServiceAccount/MSFT_SqlServiceAccount.psm1#L408-L411

Reviewed 1 of 2 files at r1, 3 of 3 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


seekTo does not scroll waveform

Bug description
Using waveform.seekTo(.5) should scroll the waveform to the 50% position, but it doesn't :(
Environment

Browser: Chrome
Wavesurfer 7.3.0

Minimal code snippet
Add wavesurfer.seekTo(.5) to the end of this example: https://wavesurfer-js.org/examples/?zoom.js
I've also tried including autoScroll: true and autoCenter: true as options
Expected result
Zoom view should move to half way point in the waveform
Obtained result
It stays at zero
Screenshots

This method requires that the audio is already loaded. If you’d like it to jump to the middle as soon as possible, you could subscribe to the ready event. I’ve just tested and it does work if you add it not to the bottom of that example but on a button click.

Thanks, in my code I do have it firing after the 'ready' event, and it has been giving me trouble all morning, but retesting it now it's working for some reason. I recently moved to Cloudflare Pages and the Workers local dev setup feels like it has some oddities that cause wierd behavior.
Anyway, thanks for looking at it, I think we're good. :)

I've reproduced it, it's related to the width of the browser. If it's too narrow, it fires another scroll event:
Scroll 60.266982064470696s 65.53011707565867s
Scroll 60.266982064470696s 65.53011707565867s
Scroll 0.020002413642241448s 5.2831374248301985s

The third Scroll log only occurs with a narrow browser window..

Ok, it's reproducible using wavesurfer.on('ready', () => wavesurfer.seekTo(.5)) - if you resize the browser window by any amount, it goes back to zero.
So maybe this should be listening for a different event rather than ready?

Yep, this works perfectly: wavesurfer.on('redraw', () => wavesurfer.seekTo(.5))

@katspaugh
Before upgrading to v7 I could use seekTo without loading audio:
surfer.load('', wave, duration)

However now seekTo does not work anymore when audio is not loaded. Is it possible to restore the previous behaviour? I am not using wavesurfer for playback only for rendering.

@bernardwiesner as mentioned here, you can achieve that by adding media: new WebAudio() to the options.


Merge develop branch into master

This PR address the next merge of develop into master. The main objective of this PR is to restructure the install process to simplify management of multiple versions of gridlabd and generally simplify the default build/install process.
Current issues

Amazon EC2 build generates some warnings.
/usr/local/bin is not added to PATH in ~/.bash_profile on Linux machines.
It's not certain the gridlabd-version script always manipulates current correctly.
The html folder is not generation by make html yet. This should actually go to share/doc.
MySQL Connector C is not installed properly on all target systems.
Influx db is not properly installed on all target systems.

Code changes

Restructure install (#417)
Develop update docs (#418)
Update branch 'fix_manual_subcommand' from develop (#422) (#424)
Add framework to support ML power flow solver (#423)
Add aws subcommand (#425)
Add support for validate subcommand (#431)
Update gridlabd-version (#433)
Fix manual command (#434)
Develop update docs (#435)
Add help subcommand (#436)
Develop update docs (#441)
Changed example option to output defaults (#445)
Add support for RANGE global (#446)
Add #for macro (#447)
Add support for PYTHON global variable expansion (#448)
Add support for SHELL global variable expansion (#449)
Add status website (#450)
Add support for correlated randomvar (#451)
Create create_docs script (#458)
Change PATCH number and PACKAGE_NAME (#462)
Added missing docs/GLM/Macros MD files (#464)
Add global docs MD files (#466)
Merge branch 'doc_master_fix' (#468)
Develop start usecases (#477)
Fix installs for branch names that contain one or more / characters (#483)
Add subcommand macros (#486)
Add inline interpreter macro (#487)
Add subcommand pipes (#490)
Add commercial parking structure model (#492)
Change primary repo from dchassin to slacgismo (#496)
Change license to GPL3 (#500)
Develop fix websites (#501)
Update README.md (#503)
Create Schedule.md (#504)
Fixing references related to DockerHub (#509)
Fix missing returns (#510)
Fixing issue with Dockerhub tests running master only (#512)
Add initialize-only command-line option (#515)
Add command line option documentation (#516)
Update json2png.py (#517)
Adding tag to dockerhub build hook (#519)
Add industrial loads module (#522)
Develop list converters (#524)
Addressing the issue with kmlhost variable after json2glm (#528)
Add support for install on Darwin 19 (#530)
Add support for generic converters (#531)
Build docs pdf (#532)
Develop merge master (#537)
Update Weather.md (#538)

Documentation changes

The project wikis are retired for user and develop docs. The online documentation source is now located in the docs/ folder and generated for each branch so that documentation changes can be tracked with revisions to the code itself. The only documentation that remains on the wiki relates to project adminstration. The docs are now readable online at http://docs.gridlabd.us/. The development docs will continue to be available at http://docs-dev.gridlabd.us/.

Test and Validation Notes

The subcommand version is used to manage which version is running.  Use gridlabd version help for details.
The subcommand manual is used to view the documentation.  Use gridlabd manual to open the current version's documentation.

@aivanova5, can you address the conflict on gldcore/rt/json2glm.py.  I think it's because the file has moved to gldcore/converters.


[Feature request]: Never make code transparent.

Maintaining code visibility and quality while keeping the VSCODE form transparent sounds like it could look really nice :)

Now that I looked at the method you used, it seems like it's not possible.

What method exactly?

Since it is done with the "User32.SetLayeredWindowAttributes" library, it makes the entire window transparent, including the images and texts inside it. including the images and texts in it. It is not possible to exclude them from the equation.


Added a reminder about the done() callback to test timeout error message

You are missing another error message at line 206.

Woops. I've updated all the messages and pushed.

No problemo! Now you need to squash commits, though.

Okay, here we go. Let's hope my sparse rebasing skills are up to it.

There are still two commits. If you'd like any help, I'm at mocha's gitter room.

I've squashed them into a single commit now.

Oh, sorry, I thought you meant you already did it.
PR looks good now, thanks! :)

Thanks. No, I was saying I'd be a while making sure I got the rebase working properly. The only time I ever really do it is in PRs so I don't have much practice.

Same here. Though I should do it more often, specially for commits that are just for saving work in progress.


arch/sparc: add initial support for S698PM chip and SMP

Summary
After several weeks of after hours hard work,  I completed initial support of porting nuttx to S698PM,which is a quad-core SPARC-V8 architecture processor,  the maximum frequency is up to 600 MHz. The S698PM is compatible with LEON4, more people can port NuttX to other LEON3/LEON4 chip, such as Dual-Core LEON3FT GR712RC and Quad Core LEON4 GR740. The journey of debug is long and tortuous, but finally succeeded, glory to Lord in the highest！
Impact
sparc
Testing
Tested nsh and smp on S698PM-DKIT development board


Break bash_functions into multiple files

bash_functions file currently currently has 9 functions in it the cover a variety of things
I want to break this up into several scripts - each focusing on a different aspect of functions
global_procs : this should handle the cheat, remove snaps, and dir_list functions go and add in the messenger function I've been using in other various apps
dev_procs : this should be where the git_pull_all, update_ytdl functions go as well as other various dev procedures
archive_procs : this should be where the backup functions should go
THESE SHOULD ALSO BE REWRITTEN TO NOT BE HARD-CODED for user and computer names as far as coming from and validate that the NAS folders do exist
In turn - once these are changed - the bash_rc file needs to be updated to reflect these new procedure files

also redesigning the git_pull_all to use directory arrays and the same functionality

this needs tested before closing - but otherwise the code has been merged

the actual archive functions need tested - but the aliases work and the linking task works as well


Disable unneeded assertion

Description
This PR fixes #3778 and #3748. It just disables an unneeded assertion that checks if the peer that is getting promoted is a member of the KnownPeer set. The reason this assertion is unneeded is because it is possible to reach that point of the code and the peer is not a member of the set due to race conditions. That's why there's an if guard to cover for that case.
Checklist

Branch

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] If this branch changes Consensus and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If this branch changes Network and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
[ ] If serialization changes, user-facing consequences (e.g. replay from genesis) are confirmed to be intentional.

Pull Request

[x] Self-reviewed the diff
[x] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[x] Reviewer requested

bors merge


FR: Numbers for quicker result selection

some apps offer shortcuts to select specific search results, to save one from repeatedly clicking arrow-down or cmd/ctrl+j to get to a certain search result. Maybe something in that regard could also be a nice feature for AQS?

Does it satisfy your requirements by changing from
[2]

to
2

?

sure, that would also work!
You can also simply add these to the styles.css, which would result in the same [2], but woudl be more easier for theme devs to style
.another-quick-switcher__item__hot-key-guide::before {
	content: "[";
}
.another-quick-switcher__item__hot-key-guide::after {
	content: "]";
}

@chrisgrieser
Released in v4.13.0-beta2 🚀

Thanks! Now I could easily style the hotkey numbers the same way I style other hotkey information in my theme 😌

Can't think of anything else to improve here 🙂

Released v4.14.0 🚀


satty: 0.13.0 -> 0.14.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/gabm/Satty/releases.
meta.description for satty is: Screenshot annotation tool inspired by Swappy and Flameshot
meta.homepage for satty is: https://github.com/gabm/Satty
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.14.0 with grep in /nix/store/48gszkw6r6i6abnn16vr26iwjcszm9c6-satty-0.14.0
found 0.14.0 in filename of file in /nix/store/48gszkw6r6i6abnn16vr26iwjcszm9c6-satty-0.14.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

satty

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/48gszkw6r6i6abnn16vr26iwjcszm9c6-satty-0.14.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A satty https://github.com/r-ryantm/nixpkgs/archive/5852d9f458752171d705a561049ead06f715555b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/5852d9f458752171d705a561049ead06f715555b#satty

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/48gszkw6r6i6abnn16vr26iwjcszm9c6-satty-0.14.0
ls -la /nix/store/48gszkw6r6i6abnn16vr26iwjcszm9c6-satty-0.14.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

satty

Maintainer pings
cc @pinpox @donovanglover for testing.

@NixOS/nixpkgs-merge-bot merge


Difference in versions specified

In CHANGELOG.md, the latest version specified is 1.1.3.
However, according to package.json and package-lock.json, the current version is 1.1.2.
When I run 'npm version patch', it creates the version 1.1.3. Just wanted to clarify if the current version needs to be 1.1.3 as I am creating a pull request for an updated version.

Good catch. I think this happened when I hit #72. Fixing now.


About pose optimizer

Hi,
Thanks for your great work. I looked at your code and found the implementation uses a shared Adam optimizer for all poses in global BA.
My question is that if some poses are not selected in a global BA iteration, do they get updated in the optimizer ?

My expectation is that if the randomly selected rays don't have pose A, pose A shouldn't be updated.
However, the Adam optimizer may use history gradient to update all parameters even if some parameters have zero gradient.

Do you think this is a valid concern ?
Thanks.

Hi @jinmaodaomaye2021, thank you for your question.
It seems you are right, this is a valid concern. The Adam optimizer uses the history gradient to update all parameters. Currently, for each global BA, we initialize a new optimizer and update the pose parameters only twice, so the impact of this will not be significant. However, I will try to do some experiments to see if this will indeed have a negative impact on pose optimization when I have time. If so, I will switch to SparseAdam for global BA.


我用A40微调出现了下列问题，推理没有问题，请问有大佬知道是什么原因吗？

(huatuo) root@autodl-container-cec311b53c-c2dea304:~/Huatuo-Llama-Med-Chinese-main# bash ./scripts/finetune.sh
===================================BUG REPORT===================================
Welcome to bitsandbytes. For bug reports, please submit your error trace to: https://github.com/TimDettmers/bitsandbytes/issues
/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/bitsandbytes/cuda_setup/main.py:136: UserWarning: /root/miniconda3/envs/huatuo did not contain libcudart.so as expected! Searching further paths...
warn(msg)
/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/bitsandbytes/cuda_setup/main.py:136: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/usr/local/nvidia/lib'), PosixPath('/usr/local/nvidia/lib64')}
warn(msg)
/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/bitsandbytes/cuda_setup/main.py:136: UserWarning: /usr/local/nvidia/lib:/usr/local/nvidia/lib64 did not contain libcudart.so as expected! Searching further paths...
warn(msg)
/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/bitsandbytes/cuda_setup/main.py:136: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('Asia/Shanghai')}
warn(msg)
/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/bitsandbytes/cuda_setup/main.py:136: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('http'), PosixPath('8888/jupyter'), PosixPath('//autodl-container-cec311b53c-c2dea304')}
warn(msg)
/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/bitsandbytes/cuda_setup/main.py:136: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('http'), PosixPath('37135'), PosixPath('//<IP_ADDRESS>')}
warn(msg)
CUDA_SETUP: WARNING! libcudart.so not found in any environmental path. Searching /usr/local/cuda/lib64...
CUDA SETUP: CUDA runtime path found: /usr/local/cuda/lib64/libcudart.so
CUDA SETUP: Highest compute capability among GPUs detected: 8.6
CUDA SETUP: Detected CUDA version 111
CUDA SETUP: Loading binary /root/miniconda3/envs/huatuo/lib/python3.10/site-packages/bitsandbytes/libbitsandbytes_cuda111.so...
Training Alpaca-LoRA model with params:
base_model: /root/autodl-tmp/llama-7b
data_path: ./data/llama_data.json
output_dir: ./lora-llama-med-e1
batch_size: 128
micro_batch_size: 128
num_epochs: 10
learning_rate: 0.0003
cutoff_len: 256
val_set_size: 500
lora_r: 8
lora_alpha: 16
lora_dropout: 0.05
lora_target_modules: ['q_proj', 'v_proj']
train_on_inputs: False
group_by_length: False
wandb_project: llama_med
wandb_run_name: e1
wandb_watch:
wandb_log_model:
resume_from_checkpoint: False
prompt template: med_template
The model weights are not tied. Please use the tie_weights method before using the infer_auto_device function.
Loading checkpoint shards: 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 33/33 [00:11<00:00,  2.86it/s]
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Found cached dataset json (file:///root/.cache/huggingface/datasets/json/default-081224d68fbae22b/0.0.0/fe5dd6ea2639a6df622901539cb550cf8797e5a6b2dd7af1cf934bed8e233e6e)
Traceback (most recent call last):
File "/root/Huatuo-Llama-Med-Chinese-main/finetune.py", line 289, in
fire.Fire(train)
File "/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/fire/core.py", line 141, in Fire
component_trace = _Fire(component, args, parsed_flag_args, context, name)
File "/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/fire/core.py", line 475, in _Fire
component, remaining_args = _CallAndUpdateTrace(
File "/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/fire/core.py", line 691, in _CallAndUpdateTrace
component = fn(*varargs, **kwargs)
File "/root/Huatuo-Llama-Med-Chinese-main/finetune.py", line 184, in train
data = load_dataset("json", data_files=data_path)
File "/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/datasets/load.py", line 1804, in load_dataset
ds = builder_instance.as_dataset(split=split, verification_mode=verification_mode, in_memory=keep_in_memory)
File "/root/miniconda3/envs/huatuo/lib/python3.10/site-packages/datasets/builder.py", line 1108, in as_dataset
raise NotImplementedError(f"Loading a dataset cached in a {type(self._fs).name} is not supported.")
NotImplementedError: Loading a dataset cached in a LocalFileSystem is not supported.

解决了，升级了transformers、atasets这两个包


Browser blueprint crashes in Safari during plugin installlation

Bug Description:
Current theia blueprint on 1.37.0 crashes in Safari.

Steps to Reproduce:

Clone Theia-Blueprint git repo.
Run Docker commands for browser in the README.md

docker build -t theia-blueprint -f browser.Dockerfile .
docker run -p=3000:3000 --rm theia-blueprint

Open Safari and view Theia. Attempting to open any file crashes Theia.

Additional Information

Operating System: MacOS Ventura: 13.2.1

Theia Version: theia-blueprint version (1.37.0).

Hi @richardtrgao  ,
Can you provide crash logs? Which files are you attempting to open? They reside in the container? Maybe there is a permission issue.

Hey @marcdumais-work -
The files were ones that resided in the container. After taking a look around, it seems that addressing #265 was able to solve this crash. Closing this issue.


Drop legacy helper functions in Zend namespace

Q
A

Documentation
no

Bugfix
no

BC Break
yes

New Feature
no

RFC
no

QA
no

Description
Legacy helper functions in Zend namespace were left in place to facilitate migration but no longer needed.

Psalm issue: UndefinedAttributeClass can be suppressed with issue handler using targeted referencedClass.
When running on 8.2 psalm targets 8.0 and complains with
ERROR: InvalidAttribute - src/Uri.php:246:11 - The class SensitiveParameter doesn't have the Attribute attribute (see https://psalm.dev/242)
        #[SensitiveParameter]

This issue does not have referencedClass to make targeted issue handler and it can't be baselined either.

Thanks @Xerkus!


IAR compiler warnings in bignum.c

Description

Type: Compiler warning
Priority: Minor

Hey folks!
The IAR Compiler (IAR ANSI C/C++ Compiler V<IP_ADDRESS>/W32 for ARM) has some remarks in bignum.c.

Remark: the unary minus operator is applied to an unsigned expression

https://github.com/ARMmbed/mbedtls/blob/935b4f96f9fb9d47158c988a1acaf9e1fa042e67/library/bignum.c#L269
https://github.com/ARMmbed/mbedtls/blob/935b4f96f9fb9d47158c988a1acaf9e1fa042e67/library/bignum.c#L302

Remark: integer conversion resulted in a change of sign

https://github.com/ARMmbed/mbedtls/blob/935b4f96f9fb9d47158c988a1acaf9e1fa042e67/library/bignum.c#L1683-L1685
https://github.com/ARMmbed/mbedtls/blob/935b4f96f9fb9d47158c988a1acaf9e1fa042e67/library/bignum.c#L1824
One possibility would be to add additional type casts to the lines.

The first two instances in "1." above no longer appear in the library.
The items for "2." can portable set all bits with a bitwise complement after casting to the appropriately-sized type.
(The u on the 0u is not strictly required)
--- a/library/bignum.c
+++ b/library/bignum.c
@@ -1563,9 +1563,9 @@ static mbedtls_mpi_uint mbedtls_int_div_int( mbedtls_mpi_uint u1,
      */
     if( 0 == d || u1 >= d )
     {
-        if (r != NULL) *r = ~0;
+        if (r != NULL) *r = ~(mbedtls_mpi_uint)0u;

-        return ( ~0 );
+        return ( ~(mbedtls_mpi_uint)0u );
     }

 #if defined(MBEDTLS_HAVE_UDBL)
@@ -1704,7 +1704,7 @@ int mbedtls_mpi_div_mpi( mbedtls_mpi *Q, mbedtls_mpi *R, const mbedtls_mpi *A,
     for( i = n; i > t ; i-- )
     {
         if( X.p[i] >= Y.p[t] )
-            Z.p[i - t - 1] = ~0;
+            Z.p[i - t - 1] = ~(mbedtls_mpi_uint)0u;
         else
         {
             Z.p[i - t - 1] = mbedtls_int_div_int( X.p[i], X.p[i - 1],


removed pip parsing requirement

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/939/
Test PASSed.

Coverage remained the same at 80.27% when pulling 818bbed8cff8470285a70c434a7bafaec55b48be on akmorrow13:pip_parse into 68bba474e43f9ad1ece4ce8d4172d8aad0f75b28 on bigdatagenomics:master.


Reading a single worksheet

Is there a way to read a single worksheet without having to read everything out of the workbook?

If you use the IDataReader interface and NextResultSet you can

The method is called NextResult.


Feature/cu 1e13vc4 update floor is lava script

Created new class TilesPattern
Now there is a new option to have sequential pattern iteration instead of only random.

If you want to test these changes, when loading Level2.scene, you need to check the exposed member checkbox sequential in the script FloorIsLava of the object Fire Bridge.

Task linked: CU-1e13vc4 Update FloorIsLava script to work with sequential or random patterns


Remove EpochSlot and RelativeSlot from Immutable DB interface

In the original design for the immutable DB, the DB was not aware of epoch sizes; since it did want to store blocks per epoch, its interface was in terms of EpochSlots: a pair of an epoch and a relative slot index within the epoch. In later iterations however it turned out that the DB did need to know the sizes of epochs. The current design is therefore a bit of a historical accident: the API is still in terms of EpochSlots, even though the DB could do the translation from a Slot to an EpochSlot entirely internally.
This ticket is about changing that: the API will just be in terms of Slots. This makes it more uniform also with the volatile store (which doesn't need to know anything about epochs), and it makes the decision to store files per epoch an internal detail.

/cc @dcoutts , @avieth .

Just in terms of slots, with the large files not being aligned to epoch boundaries is a good simplification.
I'd suggest picking a size that makes sense for the expected size of data, and avoid picking the current epoch length (21600) so as to avoid confusing people reading the code.
Note that the size we pick here has to work for both Ouroboros Classic and Praos, which have a 10x difference in slot occupancy. On the other hand we expect the average block size to grow in future.

Completed in #341.


Unable to login on web (https://app.insomnia.rest/app/authorize) with CORS issue

Expected Behavior

Pressed Login button on Insomnia Mac OS application (2023.2.2)
Chrome has been opened and redirected to https://app.insomnia.rest/app/authorize
Login via Google and succeeded
I have entered my correct passphrase, and it should be continued.

Actual Behavior
If I enter my passphrase and continue,

'Invalid passphrase, please try again' error is appearing.

It is the correct passphrase, and I did reset it several times
Access to fetch at 'https://api.insomnia.rest/auth/web-login-s' from origin 'https://app.insomnia.rest' has been blocked by CORS policy: The value of the 'Access-Control-Allow-Origin' header in the response must not be the wildcard '*' when the request's credentials mode is 'include'.

Failed to load resource: net::ERR_FAILED

Reproduction Steps
You can see expected behavior
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
2023.2.2
What operating system are you using?
macOS
Operating System Version
13.2.1 (22D68)
Installation method
download from insomnia.rest
Last Known Working Insomnia version
No response

https://github.com/Kong/insomnia/assets/27001666/02d4d03e-d06e-45bc-b5f7-b341d8e18577

@2hyjun are you by any chance using a Proxy or a VPN?

@filfreire yes, I have a proxy for corporate security 😢.
Is there any ways to login with the Proxy?

@2hyjun I encourage you to try the latest beta version https://github.com/Kong/insomnia/releases/tag/core%409.3.3-beta.0
There is a new setting there that should help:

Let us know if it helped

Unfortunately, It doesn't work.
Can the settings you mentioned above resolve my CORS error on the web?
https://github.com/user-attachments/assets/f175b800-52e3-463c-b507-2810249780e6

Same here.
Access to fetch at 'https://api.insomnia.rest/auth/web-login-s' from origin 'https://app.insomnia.rest' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: The value of the 'Access-Control-Allow-Origin' header in the response must not be the wildcard '*' when the request's credentials mode is 'include'.Understand this error
api.insomnia.rest/auth/web-login-s:1

       Failed to load resource: net::ERR_FAILED

@monecchi @2hyjun I wonder if you enable any CORS plugin that blocks this request?

Hi there. Thanks for replying. No, I haven't any Insomnia plugins that
might be blocking those requests. Actually I had to enable a Chrome
extension in order to enable a rule which allows *.insomnia.rest domain to
successfully pass the browser CORS issue. If I remove the rule I created,
the requests to  insomnia.rest are blocked again. I'm using Chrome's latest
version (128.0.6613.114) by the way.
Att.
Adriano Monecchi
@.***
Tel: +55 31 99679-5773
Em sex., 30 de ago. de 2024 às 12:03, Jay Wu @.***>
escreveu:

@monecchi https://github.com/monecchi @2hyjun
https://github.com/2hyjun I wonder if you enable any CORS plugin that
blocks this request?
—
Reply to this email directly, view it on GitHub
https://github.com/Kong/insomnia/issues/7647#issuecomment-2321541876,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABICLJ2LA2VWMRJN2EZI6ATZUCCUXAVCNFSM6AAAAABKI3MRKGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRRGU2DCOBXGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@monecchi Can you review if any Chrome plugin or security policy will remove the Origin header from the preflight request? I'd suggest you use another browser.

Hey 👋 everyone! For anyone having this issue because of a proxy config in their machine,
we have an update in v10.1.0-beta where Insomnia now respects the System Proxy config if it's there.
Please try the latest beta and let us know if it works as expected for you!


⚠ Bofrost.de has degraded performance

In 67a651d, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1304 ms

Resolved: Bofrost.de performance has improved in 08d204c after 14 minutes.


Cannot translate multiple languages withTranslation()

I try to translate multiple languages for display in a table. My problem is too many queries requested, so I want to reduce number of queries by using withTranslation() but it's not working. Please check my source code below.
    App::setLocale('en');

    // working but many queries
    $locations = Location::get();
    foreach ($locations as $row) {
      echo $row->name.' => '.$row->{'name:km'}.'
';
    }

    $locations = Location::get();
    foreach ($locations as $row) {
      echo $row->name.' => '.$row->translate('km')->name.'
';
    }

    // not working
    $locations = Location::withTranslation()->get();
    foreach ($locations as $row) {
      echo $row->name.' => '.$row->{'name:km'}.'
';
    }

    $locations = Location::withTranslation()->get();
    foreach ($locations as $row) {
      echo $row->name.' => '.$row->translate('km')->name.'
';
    }

The tests indicate this works. Please check again your configuration.

@dimsav thank for your reply. Your test is one record but could try with many records please?

Please make a pull request

@chundany did you find a solution ? i have same problem :(

same here,
Detail:
laravel 5.5
laravel-translatable ^8.0

User model:
public function pet(){
        return $this->hasOne(Pet::class)->withTranslation();
}

Pet model:
    use Translatable;
    public $translatedAttributes = [
        'name','nickname'
    ];

PetTranslation model:
    public $timestamps = false;
    protected $fillable = [
        'name','nickname'
    ];

my config is
'locales' => [
        'en' => [
            'GB',
            'US',
        ],
    ],
'locale_separator' => '_',
'locale' => 'en_US',
'use_fallback' => true,
'use_property_fallback' => true,
'fallback_locale' => 'en_US',
'translation_suffix' => 'Translation',
'locale_key' => 'locale',
'to_array_always_loads_translations' => true,

Setting above if not wrong is en_GB and en_US right?
below is how i create new pet (Example):
$user = User::find(1);
$user->pet()->updateOrCreate(['user_id'=>$user->id],
[
'en_US' => ['name'=>'Jacky','nickname'=>'Doggy'],
'en_GB' => ['name'=>'Jackie','nickname'=>'Doggie'],
]);

in DB Record:
| id | user_id | name | nickname | locale |
| 1 | 1 | Jacky | Doggy | en_US |
| 2 | 1 | Jackie | Doggie | en_GB |

how i get the result:
$user = User::find(1);
// $petname = $user->pet->translate('en_GB')->name; // < get error `Trying to get property of non-object`
// $petData = $user->pet()->withTranslate()->get(); // < get only en_US without en_GB
$petname = $user->pet->translate('en_US')->name // < It work

may i know what is the problem or i missing something?
Thanks

closed due to inactivity


Unhandled exception occurs when you click to open an expired activity

Here is the error message that came up:
An unhandled exception occurred while processing the request.
JsonSerializationException: Self referencing loop detected for property 'ManagingOrganization' with type 'AllReady.Models.Organization'. Path 'tasks[1].RequiredSkillObjects[0].OwningOrganization.Campaigns[0]'.
Newtonsoft.Json.Serialization.JsonSerializerInternalWriter.CheckForCircularReference(JsonWriter writer, Object value, JsonProperty property, JsonContract contract, JsonContainerContract containerContract, JsonProperty containerProperty)
Microsoft.AspNet.Mvc.ViewFeatures.JsonHelper.Serialize(Object value)
Asp.ASPV__Views_Activity__ActivityScripts_cshtml.d__30.MoveNext() in _ActivityScripts.cshtml
var modelStuff = @Json.Serialize(new

Please assign this one to me.

@palittle can you please provide more specifics on this one? Steps to reproduce, data used when error occurred, etc.? I'm having trouble reproducing this on my machine right now.

I am not able to recreate this issue either.  I am not able to locate the file _ActivityScripts.cshtml as referenced in the original issue.

_ActivityScripts.cshtml may have been renamed to _EventScripts.cshtml
See \AllReadyApp\Web-App\AllReady\Views\Event\_EventScripts.cshtml

http://allready-d.azurewebsites.net/Campaign/9 -> click on "Installs" -> blows up loading http://allready-d.azurewebsites.net/Event/1.  I'm not able to reproduce this locally either.

Hmmm so only happens on the deployed site?

The exception is thrown if a Task in the Event has a RequiredSkill owned by an Organisation.
The code is currently serialising the Skill's Entity so I've introduced a SkillViewModel without navigation properties that will get serialised instead.
See PR #807

https://github.com/HTBox/allReady/pull/807 solves this ... together with @tonysurma flashing the database.  Well done.  We can close this.


Chat doesn't correctly scroll to bottom after receiving a new message on iOS 16

On iOS 16, the collection view doesn't correctly scroll to the bottom after receiving a new message.
Affected: All beta versions of iOS 16.
Expected Behavior

If the user is scrolled to the bottom of the collection view.
When they receive a new message
They collection view should scroll down to show the new message
Current Behavior

If the user is scrolled to the bottom of the collection view.
When they receive a new message
The collection view doesn't scroll
https://user-images.githubusercontent.com/52077255/185611149-350893be-9542-41d7-871c-f7d9892f058f.mov
Possible Solution

Investigating, but no suggested fixes as of yet.
Steps to Reproduce

Open the ChatLayout example on an iOS 16 simulator.
Scroll to the bottom.
Wait for new messages to arrive.
Observe that they do not scroll to the correct item.

Context (Environment)

This leads to a broken experience on iOS 16.

@IAmConorNolan Hi, Thank you for the issue.
Yep, I also noticed it in the latest IOS 16 beta. I remember testing the layout in one of the first betas and it was allright. Can you please test latest version 1.2.4 and confirm that the bug is gone.
Regards,
Eugene

@IAmConorNolan I am closing the issue due to incativity. If the new version did not solve your issue - feel free to reopen it

@ekazaev I can confirm this version resolved the issue. Thank you!


This testing approach used by this project is mostly known as "approval testing"

For example, see https://github.com/approvals.
Please consider changing terminology in the project.

Hi @dkandalov,
We appreciate your interest on the project but we are not going to change its name for two reasons:

There is no standard name for these sort of tests. They are called approval tests, golden standard tests, characterization tests or snapshot tests depending on where you read it.
The project forks GAumala's project who called it KotlinSnapshot, we wanted to honor that.
In mobile development, which is our context, it's called snapshot testing. See the most popular Swift counterpart: https://github.com/pointfreeco/swift-snapshot-testing

Thank you again for pointing it out, we are well aware of the issues it might introduce but we don't have any plans to change it.


Javascript packages not published to npm?

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
Fury 0.9.0, MacOS 15.1.1, NodeJS 21.6.1
Component(s)
JavaScript
Minimal reproduce step
Check the latest version of https://www.npmjs.com/package/@furyjs/fury and https://www.npmjs.com/package/@furyjs/hps
What did you expect to see?
Version 0.9.0 as in released versions fury and hps
What did you see instead?
@furyjs/fury at version 0.0.18 and  @furyjs/hps at version 0.0.6
Anything Else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Hi @ggecy , we are refactoring fury xlang serialization protocol, furyjs will not be published to npm until we finished the reafctor. If everything goes right,  I think we can make a release in next month and publish it to npm


Screen upside down iOS

Godot version
3.4.4
System information
Ipad 8th generation with OS 14.6
Issue description
The screen is upside down in iOS.
Steps to reproduce
Run the Game of Life (Shader) sample (see below)
Minimal reproduction project
https://godotengine.org/asset-library/asset/842

I've tried to compile iOS godot export with some other parameter at supportedInterfaceOrientations and it only goes to portrait.

Ok, just had to follow https://godotengine.org/qa/91264/ios-landscape-locked-to-only-one-orientation, enabling sensor_landscape.


Some files appear to be missing

FileNotFoundError: [Errno 2] No such file or directory: '/mnt/ufs18/nodr/home/lichenni/projects/FedScale/dataset/data/device_info/client_behave_trace'
how to fix that

This is the path to the file on my server.
Please find these files (from FedScale) and change the path accordingly.
Thanks,


Feature request - Pause reading

Thanks for your library, I'm trying to implement application to verify vouchers and it fits perfectly. But I'm missing one feature - pause reading.
Imagine scenario:

Person at the entrance reads QR code and I send request to server. - Here I would perform pause reading
Waiting for server response about validity and details. - show details.
Show popup for lets say 5 seconds with details.
Unpause reading

I can do videoElement.pause() but which freeze the video. But then your app is constantly performing reading on that element. I know I can increase the scanDelay. But I would like to freeze it until timeout is gone or some click happen etc. And have possibility to pause reading instead of playing with scanDelay would be easier.
I want to perform pause to not send more request to server.
Thanks

I just published v1.2.3. Now you have the option to pass a prop stopDecoding that is a boolean. Take a look at the demo. Hope this helps.

Thanks for very quick fix. It helps a lot.


piwik: 3.1.1 -> 3.2.0

Motivation for this change
Some new features and bugfixes: https://piwik.org/changelog/piwik-3-2-0/
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
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 nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Thanks!


Can't Open

[Error] Downloading stash for Standard, details logged to DebugInfo.log, please open a ticket at https://github.com/Stickymaddness/Procurement/issues
Log File
2015/10/02 21:41:16 ***** LOG FILE OPENING *****
2015/10/02 21:41:16 25213099 14c [INFO Client 2036] Send protocol version
2015/10/02 21:41:16 25213162 1b6 [INFO Client 2036] Web root: http://patchcdn.pathofexile.com/<IP_ADDRESS>/
2015/10/02 21:41:16 25213162 1b7 [INFO Client 2036] Backup Web root: http://patch527.pathofexile.com:8095/patch/<IP_ADDRESS>/
2015/10/02 21:41:16 25213162 1c2 [INFO Client 2036] Requesting root contents 1
2015/10/02 21:41:16 25213224 1ea [INFO Client 2036] Got file list for "" 0
2015/10/02 21:41:17 25213677 222 [INFO Client 2036] Requesting folder Data 1
2015/10/02 21:41:17 25213677 222 [INFO Client 2036] Requesting folder Metadata 2
2015/10/02 21:41:17 25213801 1ea [INFO Client 2036] Got file list for "Data" 1
2015/10/02 21:41:21 25217577 230 [INFO Client 2036] Queue file to download: Data/PassiveJewelDistanceList.pjd
2015/10/02 21:41:21 25217592 1ea [INFO Client 2036] Got file list for "Metadata" 1
2015/10/02 21:41:22 25218747 222 [INFO Client 2036] Requesting folder Metadata/Terrain 2
2015/10/02 21:41:22 25218872 1ea [INFO Client 2036] Got file list for "Metadata/Terrain" 0
2015/10/02 21:41:22 25219449 222 [INFO Client 2036] Requesting folder Metadata/Terrain/Missions 1
2015/10/02 21:41:23 25219527 1ea [INFO Client 2036] Got file list for "Metadata/Terrain/Missions" 0
2015/10/02 21:41:23 25219714 222 [INFO Client 2036] Requesting folder Metadata/Terrain/Missions/Ranger 1
2015/10/02 21:41:23 25219776 1ea [INFO Client 2036] Got file list for "Metadata/Terrain/Missions/Ranger" 0
2015/10/02 21:41:23 25219870 248 [INFO Client 2036] Finished checking files
2015/10/02 21:46:21 25517707 ee [INFO Client 2036] Changing to OpenAL device "Generic Software on Speakers (Realtek High Definition Audio)"
2015/10/02 21:50:47 25784063 63b [DEBUG Client 2036] Got Instance Details from login server
2015/10/02 21:50:47 25784266 657 [INFO Client 2036] Just before calling client instance session
2015/10/02 21:50:48 25784531 b3 [INFO Client 2036] Connecting to instance server at <IP_ADDRESS>:6112
2015/10/02 21:50:48 25784609 f3 [DEBUG Client 2036] Connect time to instance server was 63ms
2015/10/02 21:51:01 25797978 5d [INFO Client 2036] Tile hash: 750952238
2015/10/02 21:51:01 25797978 5e [INFO Client 2036] Doodad hash: 0

The new version of Procurement is out, which fixes this issue: https://github.com/Stickymaddness/Procurement/releases/tag/1.9.6
Unfortunately until GGG decide to support the dev community, these type of issues will continue to happen as it is out of our hands. http://i.imgur.com/yrPtajM.jpg


Port 443 is opened with aws cert manager even only TCP is used

From rohanh in Slack:
When upgrading from 3.0.0 to 3.2.2 ingress validation fails with:
Reason: Failed to open port 443 on service for AWS cert manager for Ingress ....

Our Ingress has SSL into ELB then TCP to backend. ELB has an AWS Cert. They are non-standard ports carrying TCP traffic only ( No HTTP(s) ).
apiVersion: voyager.appscode.com/v1beta1
kind: Ingress
metadata:
  annotations:
    ingress.appscode.com/annotations-service: |
      {
        "service.beta.kubernetes.io/aws-load-balancer-ssl-cert": "arn:aws:...",
        "service.beta.kubernetes.io/aws-load-balancer-backend-protocol": "tcp",
        "service.beta.kubernetes.io/aws-load-balancer-ssl-ports": "*"
      }
    ingress.appscode.com/type: LoadBalancer
    kubernetes.io/ingress.class: voyager
  labels:
    app: test-collectors-ingress
    chart: test-collectors-ingress-0.1.0
    heritage: Tiller
    release: test-collectors-ingress
  name: test-collectors-ingress
  namespace: test
spec:
  loadBalancerSourceRanges:
  - x.x.x.x/32
  rules:
  - host: collectors.test.blah
    tcp:
      backend:
        serviceName: test-collectors-10001
        servicePort: 10001
      port: 10001
status:
  loadBalancer:
    ingress:
    - hostname: 88585647.eu-west-1.elb.amazonaws.com

https://github.com/appscode/voyager/blob/master/apis/voyager/v1beta1/port.go#L55:36
Looking at the code, if http://service.beta.kubernetes.io/aws-load-balancer-ssl-cert annotation is present for cloud provider aws and also port 80 is used, then 443 -> 80 is opened. I wonder, why code thinks port 80 is used here.

Closing old issue.


[SPARK-45761][K8S][INFRA][DOCS] Upgrade Volcano to 1.8.1

What changes were proposed in this pull request?
This PR aims to upgrade Volcano to 1.8.1 in K8s integration test document and GitHub Action job.
Why are the changes needed?
To bring the latest feature and bug fixes in addition to the test coverage for Volcano scheduler 1.8.1.

https://github.com/volcano-sh/volcano/releases/tag/v1.8.1

https://github.com/volcano-sh/volcano/pull/3101

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs.
Was this patch authored or co-authored using generative AI tooling?
No.

K8s integration test passed.

Merged to master.

Thank you, @HyukjinKwon !


⚠ GitHub has degraded performance

In 951e947, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 129 ms

Resolved: GitHub performance has improved in d228bd8.


Cross Repo view like "Pull Requests" view on github.com

Currently, I can see PRs only from one repo.Most companies use tracks across multiple repos, so it would be better to simulate the list like the one on "Pull Requests" tab of github.com

This looks like a duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/1583, let's track it there


💡[Request] Can we get litewallet APK and/or fdroid repo made available please?

Would be nice for those that don't trust or want to go through Google Play Store

Add in guideline docs

Looking into it: @Final-Phoenix

Update: @Final-Phoenix @TheeTw23
Fdroid is not going to be a possiblity because:

Fdroid doesnt allow Firebase which is integral to our management of Litewallet

No Fdroid...looking of posting APK in litewallet.io


Arch4 Beta crash on launch

Running Windows 10 Pro 64bit version 1511 on a Lenovo E540 i7-4712MQ 2.3 GHz with 8 GB RAM
Error log contents:
!SESSION 2016-11-03 12:36:46.437 -----------------------------------------------
eclipse.buildId=unknown
java.version=1.8.0_101
java.vendor=Oracle Corporation
BootLoader constants: OS=win32, ARCH=x86, WS=win32, NL=en_ZA
Command-line arguments:  -os win32 -ws win32 -arch x86 -clean
!ENTRY org.eclipse.osgi 2 0 2016-11-03 12:36:48.000
!MESSAGE com.archimatetool.editor.Application cannot be found by com.archimatetool.editor_<IP_ADDRESS>611021627
!STACK 0
java.lang.ClassNotFoundException: com.archimatetool.editor.Application cannot be found by com.archimatetool.editor_<IP_ADDRESS>611021627
at org.eclipse.osgi.internal.loader.BundleLoader.findClassInternal(BundleLoader.java:461)
at org.eclipse.osgi.internal.loader.BundleLoader.findClass(BundleLoader.java:372)
at org.eclipse.osgi.internal.loader.BundleLoader.findClass(BundleLoader.java:364)
at org.eclipse.osgi.internal.loader.ModuleClassLoader.loadClass(ModuleClassLoader.java:161)
at java.lang.ClassLoader.loadClass(Unknown Source)
at org.eclipse.osgi.internal.framework.EquinoxBundle.loadClass(EquinoxBundle.java:564)
at org.eclipse.core.internal.registry.osgi.RegistryStrategyOSGI.createExecutableExtension(RegistryStrategyOSGI.java:174)
at org.eclipse.core.internal.registry.ExtensionRegistry.createExecutableExtension(ExtensionRegistry.java:905)
at org.eclipse.core.internal.registry.ConfigurationElement.createExecutableExtension(ConfigurationElement.java:243)
at org.eclipse.core.internal.registry.ConfigurationElementHandle.createExecutableExtension(ConfigurationElementHandle.java:55)
at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:191)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:134)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:104)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:388)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:243)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:673)
at org.eclipse.equinox.launcher.Main.basicRun(Main.java:610)
at org.eclipse.equinox.launcher.Main.run(Main.java:1519)
!ENTRY com.archimatetool.editor.fullscreen.macos 2 0 2016-11-03 12:36:48.000
!MESSAGE Could not resolve module: com.archimatetool.editor.fullscreen.macos [8]
Unresolved requirement: Require-Capability: eclipse.platform; filter:="(osgi.ws=cocoa)"
!ENTRY org.eclipse.osgi 4 0 2016-11-03 12:36:48.000
!MESSAGE Application error
!STACK 1
org.eclipse.core.runtime.CoreException: Plug-in com.archimatetool.editor was unable to load class com.archimatetool.editor.Application.
at org.eclipse.core.internal.registry.osgi.RegistryStrategyOSGI.throwException(RegistryStrategyOSGI.java:194)
at org.eclipse.core.internal.registry.osgi.RegistryStrategyOSGI.createExecutableExtension(RegistryStrategyOSGI.java:176)
at org.eclipse.core.internal.registry.ExtensionRegistry.createExecutableExtension(ExtensionRegistry.java:905)
at org.eclipse.core.internal.registry.ConfigurationElement.createExecutableExtension(ConfigurationElement.java:243)
at org.eclipse.core.internal.registry.ConfigurationElementHandle.createExecutableExtension(ConfigurationElementHandle.java:55)
at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:191)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:134)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:104)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:388)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:243)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:673)
at org.eclipse.equinox.launcher.Main.basicRun(Main.java:610)
at org.eclipse.equinox.launcher.Main.run(Main.java:1519)
Caused by: java.lang.ClassNotFoundException: com.archimatetool.editor.Application cannot be found by com.archimatetool.editor_<IP_ADDRESS>611021627
at org.eclipse.osgi.internal.loader.BundleLoader.findClassInternal(BundleLoader.java:461)
at org.eclipse.osgi.internal.loader.BundleLoader.findClass(BundleLoader.java:372)
at org.eclipse.osgi.internal.loader.BundleLoader.findClass(BundleLoader.java:364)
at org.eclipse.osgi.internal.loader.ModuleClassLoader.loadClass(ModuleClassLoader.java:161)
at java.lang.ClassLoader.loadClass(Unknown Source)
at org.eclipse.osgi.internal.framework.EquinoxBundle.loadClass(EquinoxBundle.java:564)
at org.eclipse.core.internal.registry.osgi.RegistryStrategyOSGI.createExecutableExtension(RegistryStrategyOSGI.java:174)
... 15 more

Hi,
I can't reproduce this on my 64-bit Windows machine. Could you try:

Download again
Unzip again
Different machine

Any more info?

Hi Phillipus,
Tried it again and extracted it in the download folder and ran it from there and it worked...
One thing I did do before which I probably shouldn't have, was to rename the folder from Archi to Archi4.
Anyways, got the beta to run, will begin testing it...

Thanks :)

Good to hear. Renaming the folder shouldn't cause this.

I did it INSIDE the zip before extracting to my Program Files (x86)
folder... perhaps its a Windows permission issue in this folder?
Don't know... will try moving the extracted folder to there and test...
Thanks Phil...
Regards,
Theuns
On 4 November 2016 at 11:54, Phil Beauvoir<EMAIL_ADDRESS>wrote:

Good to hear. Renaming the folder shouldn't cause this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/archimatetool/archi/issues/159#issuecomment-258386610,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2-pc8f_K-voLGMB9eS9jGZMBF7y30lks5q6wC5gaJpZM4KpPXU
.

Yes, I think if you unzip to Program Files (x86) it could be a permissions issue. You'd need to do it as Administrator I think.

Yes, I think if you unzip to Program Files (x86) it could be a permissions issue. You'd need to do it as Administrator I think.

Yes, I think if you unzip to Program Files (x86) it could be a permissions issue. You'd need to do it as Administrator I think.

Yes, I think if you unzip to Program Files (x86) it could be a permissions issue. You'd need to do it as Administrator I think.

Yes, I think if you unzip to Program Files (x86) it could be a permissions issue. You'd need to do it as Administrator I think.


Add required LABEL to Dockerfile.konflux

@dhirajsb Could you please review the PR?

@MohammadiIram I'm not sure what that label is used for tbh.


Add kokoro build scripts for windows platform.

compiled buildscripts/kokoro/windows.bat on Windows instance.

windows_build.log
Attaching build logs from windows platform


Has Signature

How to check if the signature is present now? The method has been removed. Also the touch delegates are not intercept able in the view controller, by which I can know that something happened. P.S.: I'm bridging to Objective-C.

Calculating the content of the produced UIImage was never the scope of this view. It is not good OOP to put some kind of validation within the display mechanism. There would also be a lot of different needs regarding a validator, some would calculate a mathematical hash as a result of the signature, and run some algorithm to compare it with previously stored signatures, others merely checking if the signature field is not blank. Thats why making a validator is not something I will do.
I would recommend you to check out similar projects made in Objective-C. MacHeads101 has a nice tutorial that inspired me to make this class.

Thanks very much for the help and for the control!!!

For a quick and dirty validating routine, I would set up the controller where you have the view like this:
`@IBOutlet weak var drawSignatureView: YPDrawSignatureView!
@IBAction func save(sender: AnyObject) {
let img = drawSignatureView.getSignature()
let imgData: NSData = UIImageJPEGRepresentation(img, 0)!
print("Size of Image: (imgData.length) bytes")
}`
Use the size in bytes you get for your signature image when its empty, and do an if-else on the returned size, if someone has signed, the size in bytes will be larger than the empty signature image.

You got your wish granted!
Before getting the UIImage from the signature view, check if its new calculated property containsSignature is true.
Usage:
`@IBOutlet weak var drawSignatureView: YPDrawSignatureView!
@IBAction func save(sender: AnyObject) {
if drawSignatureView.containsSignature {
let img = drawSignatureView.getSignature()
// Do something with img
} else {
// Alert the user or do something else
}
}`

Thanks!

Just remember to download the latest source file :+1:

Done! :)


New version: Downloads v1.4.1

Registering package: Downloads
Repository: https://github.com/JuliaLang/Downloads.jl
Created by: @KristofferC
Version: v1.4.1
Commit: 7d0a469651009839866846b51f8483fc5dd326cd
Reviewed by: @KristofferC
Reference: https://github.com/JuliaLang/Downloads.jl/commit/7d0a469651009839866846b51f8483fc5dd326cd#commitcomment-48390425

@KristofferC
[noblock]

https://github.com/JuliaRegistries/General/pull/32277


Is it possible to send error out record in separate pipline while using custom processor

Hi,
I am new for StreamSets development.  I implemented custom processor successfully. But I am stuck  with to  send custom processor error records to other destination pipeline.
Thanks,
Ajit

You can send error records to another pipeline via RPC or Kafka - just set the Error Records pipeline config and use the appropriate origin in your error handling pipeline with data format SDC Record.
Feel free to ask follow-up questions via the Slack channel or Google Group - details at https://streamsets.com/community/


Similarity search threshold can be a float instead of a double

public Builder similarityThreshold(double threshold)
results in having to pass in a d to the method that is inconvenient.  There isn't a need for such fine grained values of the threshold.
SearchRequest.builder().similarityThreshold(0.99d).topK(6).build()

@markpollack then on the caller sites we need to update it to use the f qualifier right? The advantage is that the float takes less space than the double for the same effect right?
I haven’t investigated yet but then at the DB schema level the value will be automatically converted right?


Use 3.12 as the canonical environment

Follow-up: #10893

Stepped on your toes a bit, https://github.com/dask/dask/pull/10890. (sorry)

Looks like pandas nightly builds are broken on Python 3.12. Shelving this PR until later.

This has already been merged as part of #10939


containerCSS backgroundColor: 'transparent'

A white backgroundColor overwrites the backgroundColor of html2canvas. changed to transparent and added an opt.backgroundColor as well (optional)

this may fix issue #133 and #182.

Does setting it to transparent reduce the generated PDF size?

@hakimio Looks like changing from white to transparent doesn't reduce the file size. I mainly used window.open to view the PDF file, so I might missed something.

Hi @kakugiki, thanks for the PR! In fact I'm getting rid of the "container" element entirely, which should be an effective fix for this problem and others. You can see the proposed fix here:
html2pdf.js/bugfix/clone-nodes-BUILD
I'm busy working on unit tests for all of html2pdf.js, so I can make sure I'm not breaking things each time I try to fix them. I'll keep you posted!


Update all roll templates to match

Use the style @johnackley created, the purple one which STR and Luck are using, as a base.
Needs minor tweaking of the layout.

First pass of the basic roll template (with Critical/Hard/Success/Failure/Fumble) for all attributes and skills.
and with a simple Success/Failure roll template for Luck.


'too much recursion' hit when injecting Vue-based mixins

Version
2.4.1
Reproduction link
https://jsfiddle.net/razvansbsb/2z359v28/
Steps to reproduce

Define a component A that uses 2 mixins that each define their own provide properties wrapped in a Vue component, e.g. foo and bar.
Define a component B inside A that uses inject to add foo and bar properties

What is expected?
Both foo and bar should be available in component B.
What is actually happening?
Get 'too much recursion' exception

You have to return plain objects in provide: https://jsfiddle.net/posva/nvbyv9wc/
I recommend you taking a look to provide inject: https://vuejs.org/v2/api/#provide-inject

This used to work in 2.3.4. It stopped working in 2.4.0.
The problem occurs when trying to recursively merge the objects in mergeData. Can we add some smarts and if you detect a Vue object only clone the data properties?
Thanks!

@posva what's the meaning behind this note here https://vuejs.org/v2/api/#provide-inject:

Note: the provide and inject bindings are NOT reactive. This is intentional. However, if you pass down an observed object, properties on that object do remain reactive.

I took this to mean that reactive objects were not the default behavior but could be used if implemented.

@jasonsanjose It means that if you pass down objects that are being observed the injected properties can become reactive: https://jsfiddle.net/4yycfnfm/

@posva thanks for the clarification. We'll give that a try.

I would also like to chime in that I received this issue trying to upgrade to 2.4.x. The regression from 2.3.x with an unhelpful error message made it really hard to track down.
I was also not doing anything weird, just copy-pasting this mixin from the vue-hackernews-2.0 project:
https://github.com/vuejs/vue-hackernews-2.0/blob/master/src/util/title.js

i use the element-ui's el-upload compoent with vee-validate, it happens
wrong info is:Maximum call stack size exceeded at mergeData,so now ,what that i can do.

@posva we're still looking for a solution. If you have time to chime in we would really appreciate it. If not, hopefully someone else here might have some ideas or at least be in the same situation.
Here's my updated example based on your last fiddle: https://jsfiddle.net/jason_sanjose/4yycfnfm/1/. The new GrandChild isn't getting reactive updates to bar.stuff.
Our goal is to mixin properties into a component tree and let them be inherited in some specific way. Here's a contrived example:
Let's say we're building a library of form components that can be nested. We want to mixin disabled state into all our form components and we also want that state to propagate down a sub tree.
<MegaForm> // disabled=false (default)
  <OtherForm disabled="true">
    <SubForm> // desired behavior: "inherits" disabled=true

For example, say MegaForm has some property that requires OtherForm and it's subtree to become disabled. We want SubForm to be able to get a reactive update when OtherForm disabled changes.

@jasonsanjose it's the same problem, the injected computed-stuff at Child is not reactive because it's a whole new object every time. It's the same as setting this.bar = { stuff: 'Completely new object' } (https://jsfiddle.net/uy757cu8/)
I really think you're twisting this feature too much 😅 . It looks like you need a way of sharing state between components, but inject/provide is not meant for that

@posva Thanks for the feedback. We found a "creative" solution: https://jsfiddle.net/jason_sanjose/yc398k68/2/.
It involves some watchers on injected properties and a bit of indirection. It's not pretty nor easy to follow but it seems to work in 2.4.2. Hopefully we're still playing within the rules or at least not going to trip over any future behavior changes.

Here's a better example with cascading breadcrumbs: https://jsfiddle.net/jason_sanjose/sonurrof/2/

I still have this problem. mergeData infinitely recurses with empty objects.


typed-list: expose a view for numpy arrays

As suggested in:
https://github.com/numba/numba/issues/4347

Would it be possible for lists where the element type maps to a NumPy dtype to be able to create a NumPy array view on the internal list data? This view would become invalid if the list length is changed, but it would be useful in these sorts of cases to speed import of lists into things that expect numpy arrays.

This would likely solve the performance difference between list(TypedList) and np.array(TypedList): https://github.com/pandas-dev/pandas/issues/27539#issuecomment-514153765
In [18]: %timeit np.array(y)
520 ms ± 11.8 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [19]: %timeit list(y)
171 ms ± 9.97 ms per loop (mean ± std. dev. of 7 runs, 10 loops each

I second this. It should be easy to add a buffer interface for TypedList, so that numpy array views can be created. I am surprised that this is not possible out of the box already, since numba cares a lot about numpy compatibility otherwise.

I've added this to my Q2 list

Another thing this would help with is the Cython compatibility. For code using Cython, it would "pass over" 1-D lists into Cython variables annotated with type[:].

This is feasible, I think the first step would be to expose the base-pointer for the underlying data, a start on that is in this PR: https://github.com/numba/numba/pull/5184


Renamed getDevicePose to getViewerPose according to update in spec

I've renamed the single use of frame.getDevicePose to frame.getViewerPost which fixes #286.

I signed it!


curl -m not working due to connect call hangs

Your Windows build number:  (Type ver at a Windows Command Prompt)

Microsoft Windows [Version 10.0.15063]

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)

curl -m 1 did not end in 1s, but a long time

What's wrong / what should be happening instead:

seems connect call takes a long time (see strace log)

Strace of the failing command, if applicable:  (If <cmd> is failing, then run strace -o strace.txt -ff <cmd>, and post the strace.txt output here)

strace -T curl <IP_ADDRESS>:5678 -m 1
shows connect cost 21s
socket(PF_INET, SOCK_STREAM, IPPROTO_IP) = 4 <0.000104>
setsockopt(4, SOL_SOCKET, SO_KEEPALIVE, [1], 4) = 0 <0.000028>
setsockopt(4, SOL_TCP, TCP_KEEPIDLE, [60], 4) = 0 <0.000019>
setsockopt(4, SOL_TCP, TCP_KEEPINTVL, [60], 4) = 0 <0.000047>
fcntl(4, F_GETFL)                       = 0x2 (flags O_RDWR) <0.000025>
fcntl(4, F_SETFL, O_RDWR|O_NONBLOCK)    = 0 <0.000032>
clock_gettime(CLOCK_MONOTONIC, {166492, 668494000}) = 0 <0.000045>
connect(4, {sa_family=AF_INET, sin_port=htons(5678), sin_addr=inet_addr("<IP_ADDRESS>")}, 16) = -1 EAGAIN (Resource temporarily unavailable) <21.002000>
clock_gettime(CLOCK_MONOTONIC, {166513, 672789000}) = 0 <0.000054>

vs strace on Ubuntu 16.04.2
socket(PF_INET, SOCK_STREAM, IPPROTO_IP) = 4 <0.000024>
setsockopt(4, SOL_SOCKET, SO_KEEPALIVE, [1], 4) = 0 <0.000018>
setsockopt(4, SOL_TCP, TCP_KEEPIDLE, [60], 4) = 0 <0.000012>
setsockopt(4, SOL_TCP, TCP_KEEPINTVL, [60], 4) = 0 <0.000017>
fcntl(4, F_GETFL)                       = 0x2 (flags O_RDWR) <0.000010>
fcntl(4, F_SETFL, O_RDWR|O_NONBLOCK)    = 0 <0.000018>
clock_gettime(CLOCK_MONOTONIC, {1282779, 886594029}) = 0 <0.000010>
connect(4, {sa_family=AF_INET, sin_port=htons(5678), sin_addr=inet_addr("<IP_ADDRESS>")}, 16) = -1 EINPROGRESS (Operation now in progress) <0.000048>

See our contributing instructions for assistance.

shows connect cost 21s

21 seconds. Surly the same as #1829 (message) #903 #1584. Non blocking connect getting popular as a feature request lately.

@tg123 - Thanks for your post and the trace. Very helpful.
Yes, non-blocking connect seems to be coming up more frequently now. Note taken. Marking this one as dupe.


UpdateApplication returns DeploymentID.

According to https://mesosphere.github.io/marathon/docs/rest-api.html#put-v2-apps-appid marathon returns deployment information after updating an app.
Additionally, duplicate put endpoint was removed from methods.yml.

:+1: ... merging now ..


docs: Simplify OpenAPI docs

Simplifies OpenAPI docs and adds a test for Python client that can be generated by following docs.

Something unrelated to the change failed.

@artemrys I changed one command (python setup.py install to pip install .), could you re-approve?


v0.6.0-pending

TODO list before releasing v0.6.0

[ ] Update revision history
[ ] Review documentation for changes
[ ] Update library version in project
[ ] Link test project to new version
[ ] Merge branch to main and close this issue

Should Collection even exist? It just wraps a LinkedList. Since I_LinkedList implements the I_Collection interface it can be passed where an I_Collection is requested.


Make explorative Tests

User story

As a User
I want / need to use an app that runs without bugs, regardless of my prior knowledge
So that I enjoy using the app and are not annoyed by occurring bugs

Further description

We want to make 100 % sure, that the final app runs without any bugs. Keep Marcus´ words in mind: "Stop Starting - Start Finishing". That´s why we want to take care of this concern early
Some edge cases already mentioned in initial presentation sent to us by Markus

Acceptance criteria

[ ] Try to find out how the software actually works and ask questions about how it handles difficult and simple cases
[ ] Be creative in what values to enter, which paths to take, ...
[ ] Prepare a summary in a format of your preference where you let the whole team know, which cases you considered, what the expected behavior was and which improvements you would suggest

Definition of done (DoD)

It builds without errors
code has been reviewed
Build has been made and deployed on a testing environment
Code coverage >75%
All acceptance criteria were met
Documentation updated whenever needed within planning documents (Tabs: Bill of materials, Documentations)
Branch/feature was merged into dev and then main

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

Current status: https://github.com/amosproj/amos2022ss05-find-my-hearing-aid/blob/explorative-tests/Documentation/explorative_tests.md


Add TxId and inputs' amount

This PR builds on top of https://github.com/zkSNACKs/WalletWasabi/pull/10407 to add the (coinjoin) transaction hash and inputs' amount to the coinjoin notification request.

Yes, I cannot use the tor that comes with Wasabi and that's why I replace it with the one that works in my OS but the mistake is for using Rider git integration instead of CLI

Done in https://github.com/zkSNACKs/WalletWasabi/pull/10442.


[SPARK-36419][CORE] Optionally move final aggregation in RDD.treeAggregate to executor

What changes were proposed in this pull request?
Move final iteration of aggregation of RDD.treeAggregate to an executor with one partition and fetch that result to the driver
Why are the changes needed?

RDD.fold pulls all shuffle partitions to the driver to merge the result
a. Driver becomes a single point of failure in the case that there are a lot of partitions to do the final aggregation on
Shuffle machinery at executors is much more robust/fault tolerant compared to fetching results to driver.

Does this PR introduce any user-facing change?
The previous behavior always did the final aggregation in the driver. The user can now (optionally) provide a boolean config (default = false) ENABLE_EXECUTOR_TREE_AGGREGATE to do that final aggregation in a single partition executor before fetching the results to the driver. The only additional cost is that the user will see an extra stage in their job.
How was this patch tested?
This patch was tested via unit tests, and also tested on a cluster.
The screenshots showing the extra stage on a cluster are attached below (before vs after).

Can one of the admins verify this patch?

cc @mridulm @xkrogen @Ngone51  to take a look.

retest this please

+CC @srowen, @dongjoon-hyun
The original folks who worked on the api dont seem to be active anymore.

@srowen You are right that the total computation is the same (it is slightly higher actually given the additional fold with zero value at driver and ofcourse the extra shuffle).
The main issue is what @venkata91 mentioned - at executor, it is done in context of shuffle, where memory is better managed (not necessarily because executor has more memory).
At driver, it is essentially fetching all partitions (single value per partition) as part of the fold's runJob : for nontrivial values, this causes OOM at driver, failing the application (memory usage for io + N serde in addition to the fold).

Hm, I wonder if this is better solved with a slightly higher tree depth - or is it not controllable in the context in question?

@srowen That is controlled by scale - which is unfortunately not customizable (val scale = math.max(math.ceil(math.pow(numPartitions, 1.0 / depth)).toInt, 2)).
The potential for increased cost was the reason I guess @akpatnam25 had the config (in my impl from quite a while back, I always did the final fold at executors when partitions > 1  :-) ).

Hm. Still feels weird. What's the overhead of the extra hop in 'normal' cases? This case seems solvable with sufficient driver memory

Extra shuffle and the additional scheduling overhead for the 1 task foldByKey operation. This can be slightly high for trivial treeAggregate operations, in that case it makes sense to control this behavior under a flag. Thoughts?

Can it be quantified at all? Trying to get a sense of whether it affects most normal sized cases non trivially or really adds very little except for trivial cases. Users are not going to find the flag in general so question is how much this impacts users vs solves a problem that already has a solution of sorts

for example, below is a snippet of a case that we would want to safeguard against:
def createValue() = new Array[Byte](180 * 1024 * 1024)
def createRdd(n: Int) = sc.parallelize(0 until n, n).map(_ => createValue())
val rdd = createRdd(13)
rdd.treeAggregate(createValue())((v1: Array[Byte], v2: Array[Byte]) => v1, (v1: Array[Byte], v2: Array[Byte]) => v1, 4)

For the above snippet, we are generating synthetic data to maximize the partitions being pulled into the driver. This is just to illustrate an example of what "bad behavior" might look like. With optimal resources on the driver side, this succeeds, but does not succeed when the driver memory is not set high enough.  The applications that have encountered this issue have obviously dealt with this by increasing driver memory, etc.
Given that, the overhead for most normal sized cases would just be the extra stage.

for example, below is a snippet of a case that we would want to safeguard against:
def createValue() = new Array[Byte](180 * 1024 * 1024)
def createRdd(n: Int) = sc.parallelize(0 until n, n).map(_ => createValue())
val rdd = createRdd(13)
rdd.treeAggregate(createValue())((v1: Array[Byte], v2: Array[Byte]) => v1, (v1: Array[Byte], v2: Array[Byte]) => v1, 4)

For the above snippet, we are generating synthetic data to maximize the partitions being pulled into the driver. This is just to illustrate an example of what "bad behavior" might look like. With optimal resources on the driver side, this succeeds, but does not succeed when the driver memory is not set high enough.  The applications that have encountered this issue have obviously dealt with this by increasing driver memory, etc.
Given that, the overhead for most normal sized cases would just be the extra stage. In our above synthetic workload, there was no noticeable difference in time to complete despite the number of partitions we are pulling into the last stage. This last reduce step would be happening on the driver side anyways, so the only real overhead is the shuffle and the scheduling.

I end up neutral on this - that the driver may not have enough memory doesn't seem quite like justification here, but possibly the argument about better aggregation on the executor (really?). I am concerned there is no quantified argument about overhead yet here. I wouldn't veto it if others feel strongly but might ask for a third (fourth?) opinion to be cautious

Makes sense @srowen, thanks for taking a look !
+CC @mengxr, @HyukjinKwon, @cloud-fan, @dongjoon-hyun - would be great if you can take a look (Tagging based on history).

I am okay with this in general. We have some places like this e.g., listing files or single partition when operating a global window in Spark SQL. Not sure if the configuration is the best way though. Let me add some more people who might be interested in here: @jiangxb1987 @Ngone51

I have no problem with adding it in as a parameter defaulted to false (and removing it as a config), if @venkata91  and @mridulm are fine with it. I'll wait for some of the others to chime in before I go ahead and make the change to the pr.
Thanks for reviewing @HyukjinKwon .

@HyukjinKwon Adding as parameter instead of config makes sense, but that would require application code change - Not sure if users would adopt it this way. Thoughts?

This is a new behaviour and this change won't be beneficial always given the discussion above so I would think it might be better for users to selectively choose the behaviour. But I really don't have a strong preference. I see no objection on this change so I would defer to @mridulm on how to proceed further - he has a much better insight in core than me.

I agree with @HyukjinKwon, let us make it a method param disabled by default.
Even with the config option, given it is disabled by default, it will require app config change in order to leverage it anyway - and we have finer control with this than doing it app wide as a config option.

Ok to test

Test build #142347 has started for PR 33644 at commit 811eb3a.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46855/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46855/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46855/

Test build #142347 has finished for PR 33644 at commit 811eb3a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142347/

The idea looks fine. In Spark SQL, global aggregate (no grouping key) also launches a single reduce task at the executor side to do the aggregation work, and send back the result to the driver later. It's useful when you have many concurrent SQL queries, as you have many reduce tasks to do the computation work instead of a single driver node.
And also +1 to use parameters instead of configs.

OK I find that argument convincing BTW, thanks

i just pushed up a new change that reverts back to the older config approach. I also changed the config as per HyukjinKwon's guidance on the naming.

Test build #142445 has started for PR 33644 at commit 722cbed.

Test build #142445 has finished for PR 33644 at commit 722cbed.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142445/

Test build #142448 has started for PR 33644 at commit 3542781.

Test build #142448 has finished for PR 33644 at commit 3542781.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142448/

Test build #142450 has started for PR 33644 at commit 2ab08c4.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142450/

Test build #142451 has started for PR 33644 at commit 75945bc.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46951/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46951/

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46954/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46954/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46956/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46956/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46956/

Test build #142451 has finished for PR 33644 at commit 75945bc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142451/

can you take one more look at this pr so that we can merge? @HyukjinKwon @cloud-fan @srowen

I am okay w this change. Seems like @cloud-fan and @srowen are fine either.
I think we agreed upon having it as a parameter instead of a configuration which seems not addressed yet.
Otherwise, I would defer to @mridulm

I am good with conf approach (see https://github.com/apache/spark/pull/33644#discussion_r690888342)

@HyukjinKwon can we merge this then now? seems like we have consensus from everyone.

We can - smems like the test failure is by linter which is unrelated. But I haven't taken a close look for the codes. TBH. I'll leave it to @mridulm

@akpatnam25 Also update the PR with the latest changes from master, so that GA would run again probably that can fix the linters issue.

updated from latest changes from master

Test build #142621 has started for PR 33644 at commit d50601d.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47122/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47122/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/47122/

Test build #142621 has finished for PR 33644 at commit d50601d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142621/

New approach looks good to me. I think I didn't pay attention to @HyukjinKwon's suggestion earlier 😞 .

Test build #142650 has started for PR 33644 at commit 3d49c1d.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47151/

Test build #142651 has started for PR 33644 at commit f728a02.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47151/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/47151/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47152/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47152/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/47152/

Test build #142650 has finished for PR 33644 at commit 3d49c1d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142650/

Test build #142651 has finished for PR 33644 at commit f728a02.

This patch fails from timeout after a configured wait of 500m.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142651/

can this be merged now? We now have consensus on both the reason for doing this change and also have implemented a parameter based approach. @mridulm @venkata91 @HyukjinKwon

thanks, merging to master!


[BugFix] BE crash when table use snappy compression (#10883)

What type of PR is this：

[x] bugfix
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：
Fixes https://github.com/StarRocks/starrocks/issues/10881
Problem Summary(Required) ：
Problem:
When we use snappy for page compression, be may crash. The reason is that the original snappy lib does not support multiple discontinue memory for compression. So we reform the input parameter for it, which is SnappySlicesSource. But the member function Peek may return the wrong pointer which causes this problem.
Solution:
Make SnappySlicesSource.Peek return the correct pointer.

better add a unit test

run starrocks_admit_test


How to make delegates for JS

Hi, how to let it to learn what delegeates (or System.Action) are?
I have this code:
engineInterpreter.AddGlobalFunction(
    "Gameloop",
    new System.Action<Delegate>((code) => Debug.Log(code)),
    typeof(JSScriptRunner));

which calls in the background with some conditions engine.SetGlobalFunction(name, value)
I get the error

NotImplementedException: Unsupported parameter type 'System.Delegate'.

What I want to achieve is this code in js
Gameloop(function()
{
  print("test");
});

The Debug.Log will be changed later.

Ok I just changed delegate to object and find out it converts to "UserDefinedFunction". So I could call there the code which is ok.

You should use FunctionInstance rather than UserDefinedFunction (UserDefinedFunction
inherits from FunctionInstance).

Thanks, I tried it and it works as well. But I still needed to change the whole idea.
I have multiple instances of ScriptFile (one instance per engine). The Gameloop content should be get called every frame in Unity but the problem was I couldn't get what instance actually is calling the function.
So I changed to function Gameloop() { // content }
and I use just now
if (jsScriptRunner.engine.HasGlobalValue("Gameloop"))
    jsScriptRunner.engine.CallGlobalFunction("Gameloop");

I am still testing around which way is better.


Docs overhaul

Reference issue for docs refactor
We are planning on overhauling the documentation for vue-cli soon. There will be two docs sites:

User Manual (for the users of vue-cli and vue-cli ui)
Plugin developer docs

Checklist:

Improve docs (obviously)
Move the plugin development docs to a separate website (for example vue-cli-dev.vuejs.org)
Create a complete plugin dev guide
Create a full API reference

cc @NataliaTepluhina

Working on plugin dev guide + API reference

@Akryum same as @JonathanDn
Would love to help if there's a path or somewhere you would love us to get started from.
Thanks

Bumping the Docs overhaul issue!
I think there's an opportunity to improve the docs about Build Targets and the --target lib. Specifically, about useBuiltIns config for lib authors. @vue/babel-preset-app docs already explain it, but I think it's not easy to find this information and a number of projects ship dist files sometimes twice as large, just because of the polyfills.
Here's a fresh example of how just using the Number prop validator like this:
props: {
  limit: {
    type: Number, // <- es6.number.constructor polyfill is added
    default: 0
  }
}

increases the (unminified) bundle size from 17.33 KiB to 39.42 KiB.
Would you be interested in a Pull Request for the docs?
cc: @Akryum @NataliaTepluhina

With the release of vue cli v4, it would be really nice to have a guide on migrating plugins. It also appears as if the new migrate feature is undocumented.

It would be great if looking at older versions of the documentation was added here as well.  It is confusing for me what works with Vue 2, 3, and 4.

It would be great if looking at older versions of the documentation was added here as well.  It is confusing for me what works with Vue 2, 3, and 4.


extra files excluded from package

LITE-19686

Using setuptools_scm forces all files to be packaged. The most common way to avoid it - list all files in MANIFEST.in
Discussion https://github.com/pypa/setuptools_scm/issues/190


Fix back navigation when tabBarVisible is false ...

Workaround that fix back navigation when tabBarVisible is false and tabBarPosition is bottom. The issue is in react-native-tab-view because doesn't support hide the children elements.
Related issues: #1398 and #1584
Render renderTabBar with display: 'none' instead of nothing.

@matthamil the test fails because of warning  Unexpected use of weak type "any"  flowtype/no-weak-types maybe it should be better to disable the eslint rule? or is used when a new version is released?

@cooperka I will test all the PR during the weekend.

@matthamil could you check it?

+1

I tried this patch as is and got errors (display: 'none' isn't a valid property).
I think this fix is the right idea, but needs more attention.

Assigning myself so I can take a look at this in the next couple of days.

@etaiklein what version of react-native are you using? Also, I will check it again with the last merge from master.
@matthamil great roadmap!

Confirming that this will fix navigation FROM a tab whose tab bar is hidden (original issue in #1584).
If so, I'll wait for the merge... thanks!

@dchersey I tested with the last version of master a few minutes ago and works fine. Please test it with this version https://github.com/gusgard/react-navigation#1f411bc1f45cebae0c6d7961bbe8f70420495eac

Thank you @gusgard, :thumbsup: that version does the trick!

@matthamil I merged with 1.0.0-beta.16 and works correctly. Could you take a look?

@gusgard: hey, thanks for this PR, I'm using it with react-navigation: "v1.0.0-beta.19" and tab hides fine but doesn't show the images of the icon of each tab when it appears back again, is there something that I can do?

@lucasbento could you share the code? I never tested with images but with components work fine.

Codecov Report

Merging #2101 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2101      +/-   ##
==========================================
+ Coverage   71.89%   71.94%   +0.04%
==========================================
  Files          52       52
  Lines        1701     1704       +3
==========================================
+ Hits         1223     1226       +3
  Misses        478      478

Impacted Files
Coverage Δ

src/views/TabView/TabView.js
88.09% <100%> (+0.91%)
: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 7385c24...629f272. Read the comment docs.

Can you file a bug report for this in https://github.com/react-native-community/react-native-tab-view with a repro? This is weird. Also, does this issue still exist? There have been a lot of changes to react-native-tab-view since the PR was opened. Just need to ensure that this is still an issue.

Yes, still happen I will try to have a working example for this weekend.

NavigationPlayground for 64f51cff9834c2906df4f5b8408841e8552c67bf has been deployed.

https://expo.io/@react-navigation-ci/NavigationPlayground?release-channel=64f51cff9834c2906df4f5b8408841e8552c67bf

Any update?

Version 1.5.9 and issue still exist. Even workaround which was working on older version is not working now.

@ronal2do @AbhishekCode What version of RN are you using it?

I am using 0.51.0.
By the way i have found a workaround for now https://github.com/react-navigation/react-navigation/issues/1584#issuecomment-377938796

"react": "16.0.0",
"react-native": "0.49.3",

@gusgard But, If possible to pass tabBarStyles can be solve my problem, I have several internal levels of tabbar hide/show, only 1 has this problem

@ronal2do You can create your own tabbarcomponent, then inside that you can check this.props.navigation which contains whole navigator props including styles. Then you can overwrite style depending on your tab.

this belongs on https://github.com/react-navigation/react-navigation-tabs now :)


No Release task

Hi,
I am using the Visual Studio Team Services and added your extension. Unfortunately I am not able to find the task when creating a new build sequence. Did I miss something ?
Thanks,
Fabian
Update:
BTW: Just saw that its clearly writtenin the description, shame on me...

Unfortunately the release task is also not there.

Fabian,
It's a task for Release Definitions, not Build Definitions.

Thanks for your fast reply, that make sense. But should I not see it then here ?

(Sorry for those dumb questions, just startet to take a look at the whole Team Services topic today)

Yes you should see it here if the extension is properly installed (the "Deploy" category):

Hmm unfortunately it does not. Any idea what I can do to resolve it ? I already uninstalled and installed it again without luck.

After uninstall/install again, the tasks now show up. Strange, but luckily sovled.

We noticed some temporary issue on VSTS or Azure on the extension page. I guess that was it..


Merge timeseries when send triggered

Merge time series before periodic push. This is to accommodate case where traffic is not heavy enough to trigger buffer full merging.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: bianpengyuan
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: kyessenov
If they are not already assigned, you can assign the PR to them by writing /assign @kyessenov in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
34aed0b3677bdcf7ca2a55e502382d8c83db35f5
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
34aed0b3677bdcf7ca2a55e502382d8c83db35f5
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
34aed0b3677bdcf7ca2a55e502382d8c83db35f5
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bianpengyuan: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
34aed0b3677bdcf7ca2a55e502382d8c83db35f5
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
34aed0b3677bdcf7ca2a55e502382d8c83db35f5
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
34aed0b3677bdcf7ca2a55e502382d8c83db35f5
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bianpengyuan, douglas-reid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [douglas-reid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

this will go into 1.1 rc1


prettier-plugin-organize-imports breaking this plugin

I've read the section on configuring the plugin order to get both plugins to work together, but they just won't.
    plugins: [
        "./node_modules/prettier-plugin-multiline-arrays",
        "./node_modules/prettier-plugin-organize-imports",
    ],

Have tried the other order and no difference. If I npm remove prettier-plugin-organize-imports then this plugin works just fine. Any ideas?

Actually I suspect this is more of a problem with prettier-plugin-organize-imports. It's readme states:

Caveat

This plugin inherits, extends and overrides the built-in Prettier parsers for babel, babel-ts, typescript and vue, i. e., it's incompatible with other plugins that do the same... so only the last loaded plugin that exports one of those parsers will function.