RuntimeError: "LayerNormKernelImpl" not implemented for 'Half

请问你是在GPU还是CPU上运行的？PyTorch的LayerNorm实现不支持CPU上的fp16计算，请把.half()换成.float()。
Are you using the model on GPU or CPU? The PyTorch's implementation of LayerNorm doesn't support fp16 computation on CPU. Please replace .half() with .float()

Thanks. And how to finetune ChatGLM for our own datasets? Could you give some examples for the forward function?

We will support that in the future. Currently, since the model implementation is already open on HuggingFace Hub，you can implement training with existing libraries like HF Trainer.

Is it similar to https://github.com/THUDM/GLM?
'''
Training
inputs = tokenizer(
["Tsinghua University is located in [MASK].", "One minus one equals zero, is it correct? Answer: [MASK]"],
return_tensors="pt", padding=True)
inputs = tokenizer.build_inputs_for_generation(inputs, targets=["Beijing", "No"], max_gen_length=8, padding=False)
inputs = inputs.to('cuda')
outputs = model(**inputs)
loss = outputs.loss
logits = outputs.logits
'''

Thanks. I have found it.

我在GPU上run 也遇到了这个问题，有什么解决方案吗

我在GPU上run 也遇到了这个问题，有什么解决方案吗

请问你解决了吗

我在GPU上run 也遇到了这个问题，有什么解决方案吗

我在GPU上run 也遇到了这个问题，有什么解决方案吗

请问你解决了吗

已解决，请修改web_demo.py中的这条语句：
Now sovled, please rewrite the following sentence in web_demo.py:
model = AutoModel.from_pretrained("THUDM/chatglm-6b", trust_remote_code=True).half().cuda()
如，使用INT8量化模型时语句为：
For instance, the sentence shall be as follow if you attempt to use INT8 Quantitative Module:
model = AutoModel.from_pretrained("THUDM/chatglm-6b", trust_remote_code=True).quantize(8).half().cuda()
Hope it helps :-)


Script updates

additional scripts needed for CRY digi/reco
tested to digi stage
Some more changes to come....

Hi @sophiemiddleton,
You have proposed changes to files in these packages:

JobConfig
CampaignConfig
Scripts

which require these tests: build.
@Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main.
:hourglass: The following tests have been triggered for 75026ca3d1a9be29b4d9ea2ee177fab4fdf19ffc: build  (Build queue is empty)
About FNALbuild. Code review on Mu2e/Offline.

:sunny: The build tests passed at 75026ca3d1a9be29b4d9ea2ee177fab4fdf19ffc.

Test
Result
Details

test with
:white_check_mark:
Command did not list any other PRs to include

merge
:white_check_mark:
Merged 75026ca3d1a9be29b4d9ea2ee177fab4fdf19ffc at 722e42d953d5a9a524f6a9be87582d155a21244b

build (prof)
:white_check_mark:
Log file. Build time: 17 min 13 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at 75026ca3d1a9be29b4d9ea2ee177fab4fdf19ffc after being merged into the base branch at 722e42d953d5a9a524f6a9be87582d155a21244b.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.

@sophiemiddleton are there any more expected commits on this PR; or is it ready to go?
I have done a cursory review and that's all that we can do at this time.

@FNALbuild run build test

:hourglass: The following tests have been triggered for 02df97a5c42c657af0155219841d3aae481384bb: build  (Build queue has 1 jobs)

:sunny: The build tests passed at 02df97a5c42c657af0155219841d3aae481384bb.

Test
Result
Details

test with
:white_check_mark:
Command did not list any other PRs to include

merge
:white_check_mark:
Merged 02df97a5c42c657af0155219841d3aae481384bb at 722e42d953d5a9a524f6a9be87582d155a21244b

build (prof)
:white_check_mark:
Log file. Build time: 22 min 16 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at 02df97a5c42c657af0155219841d3aae481384bb after being merged into the base branch at 722e42d953d5a9a524f6a9be87582d155a21244b.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.

nothing else to add for now, more will come separately


CalcNumRotatableBonds gives inconsistent results

Describe the bug
The results for CalcNumRotatableBonds() gives different results, depending if hydrogens are present or not.
For example, an amide may not be detected if the hydrogen on the C(=O) is missing.
An other example is, that methyl groups become a rotatable bond, when hydrogen is added.
While this behavior may - from a technical point of view - be correct, from a chemists point of view it will lead to a lot of confusion and a high chance of wrong decision making.
P.s. I just noticed, this bug only shows, when there is a ring present.
To Reproduce
from rdkit import Chem
from rdkit.Chem import AllChem
test_mol = Chem.MolFromSmiles('C2CC(C)(C)C(C)(C)C2')
print("Number of atoms:", test_mol.GetNumAtoms())
print(Chem.rdMolDescriptors.CalcNumRotatableBonds(test_mol))
test_mol= Chem.AddHs(test_mol)
print("Number of atoms:", test_mol.GetNumAtoms())
print(Chem.rdMolDescriptors.CalcNumRotatableBonds(test_mol))
test_mol = Chem.MolFromSmiles('O=CN1CCNCC1')
print("Number of atoms:", test_mol.GetNumAtoms())
print(Chem.rdMolDescriptors.CalcNumRotatableBonds(test_mol))
test_mol= Chem.AddHs(test_mol)
print("Number of atoms:", test_mol.GetNumAtoms())
print(Chem.rdMolDescriptors.CalcNumRotatableBonds(test_mol))
Expected behavior
Consistent results, no matter if hydrogens are present or not.
Configuration (please complete the following information):

RDKit version: 2019.03.4 and 2021.03.5

OS: [e.g. Ubuntu 20.04]

Python version (if relevant):

Are you using conda?

If you are using conda, which channel did you install the rdkit from? conda-forge

If you are not using conda: how did you install the RDKit?

I understand your point, and agree that this isn't the best behavior, but there are numerous 2D descriptors in the RDKit which will return different values for molecules with and without explicit Hs added.
To get correct values, it's almost always a good idea to use hydrogen-suppressed graphs (i.e. molecules which don't have Hs explicitly present in the graph).
I'm going to close this as "wont-fix" because the computational cost of handling detecting Hs with every call to the various descriptor values isn't reasonable.


Bug: scroll after closing the modal

I'm using vue-final-modal (4.4.6) and vue3-tabs-component (last). After closing the modal, it scrolls up or down depending on where you are on the screen.

Hello, the tabs don't have an ID attribute so such behaviour should not happen from the tabs component - could you post a minimum reproduction code to figure out the cause?
Thank you.

@Jacobs63 I created a project that shows the problem. If you remove tabs from App.vue and leave only the elements, the scroll will not jump.
Link: https://github.com/DarKsandr/vue-tabs-error
There is a problem:
 <tabs>
            <tab name="First tab">

Open

            </tab>
            <tab name="Second tab">
                Second tab content
            </tab>
        </tabs>

no problem:

Open

Currently investigating - if trying the native dialog api I'm not encountering such issue, only if using the dialog package.
Tried testing whether perhaps the first-tab-pane could be matched by the browser from the first-tab hash contained in URL, however that doesn't seem to be the case (as expected).
I'll continue investigating.

@Jacobs63 I found out why the scroll is jumping. if you remove tabindex="-1" in Tab.vue, then the scroll stops jumping

@Jacobs63 I found out why the scroll is jumping. if you remove tabindex="-1" in Tab.vue, then the scroll stops jumping. Maybe it's worth removing tabindex when the tab is active?

Hello, I've been testing this a bit during the weekend and currently believe this is caused by how the modal package is working with focus, likely caused by the useFocusTrap from VueUse, which it uses internally.. which uses focus-trap and jumps to previously focused element on trap deactivation.
I don't really see the reason to change tabindex dynamically here, it prevents it from being focusable, this would degrade usability for people with assistive technology.
This needs more thorough thinking, so I'll not accept the proposed solution at this time.

Note, reading from focus-trap docs:
When the focus trap is deactivated, this is what should happen:

Focus is passed to whichever element had focus when the trap was activated (e.g. the button that opened the modal or menu).
Tabbing and clicking behave normally everywhere.

So this means, after closing the modal, the focus is given back to the original button, meaning it, possibly, might make the browser scroll the the center of the container the button is in.

@Jacobs63 ok, until I solve the problem, I’ll rewrite the tabindex to an empty one in my project


Pan Pixel Offset and Swipe To Slide not working

I have just integrated this script into my project. I was able to get the Left Menu Controller to appear on button press but I can not seem to get the pixel offset or the "swipe to slide" working.
My project is setting this SlideNavigationController as the root view controller, and then within that, I am presenting a tab bar controller. Inside this tab bar controller , the menu slides in with the press of a button ( as stated above ) but enabling swipe controls and pixel offset changes does not seem to be doing anything.
Any ideas?
Thanks!

Yeah I am using the same set up with SlideNavigationController embedding UITabBarController and swipe gestures are not working. I heard JASidePanel can get this to work tho: https://github.com/gotosleep/JASidePanels. I might have to switch to this.

I Just got it to work. You simply had to implement below code into the UITabbarController custom class. Make sure you set that class as delegate.

(BOOL)slideNavigationControllerShouldDisplayLeftMenu
{
return YES;
}

(BOOL)slideNavigationControllerShouldDisplayRightMenu
{
return YES;
}


[ICU] build failure OSX

Host Environment

OS: macOS Big Sur 11
Compiler: Clang/LLVM 13.0.0

To Reproduce
Steps to reproduce the behavior:
./vcpkg install icu
Failure logs
Building package icu[core]:x64-osx...
-- Downloading https://github.com/unicode-org/icu/releases/download/release-69-1/icu4c-69_1-src.tgz -> icu4c-69_1-src.tgz...
-- Extracting source /Users/runner/work/vcpkg/vcpkg/downloads/icu4c-69_1-src.tgz
-- Applying patch disable-escapestr-tool.patch
-- Applying patch remove-MD-from-configure.patch
-- Applying patch fix_parallel_build_on_windows.patch
-- Applying patch fix-extra.patch
-- Applying patch mingw-dll-install.patch
-- Applying patch disable-static-prefix.patch
-- Applying patch fix-win-build.patch
-- Using source at /Users/runner/work/vcpkg/vcpkg/buildtrees/icu/src/c-69_1-src-b48be0c7ce.clean
-- Getting CMake variables for x64-osx-dbg
-- Getting CMake variables for x64-osx-rel
-- Generating configure for x64-osx
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:128 (message):
    Command failed: /usr/local/bin/autoreconf -vfi
    Working Directory: /Users/runner/work/vcpkg/vcpkg/buildtrees/icu/src/c-69_1-src-b48be0c7ce.clean/source
    Error code: 2
    See logs for more information:
      /Users/runner/work/vcpkg/vcpkg/buildtrees/icu/autoconf-x64-osx-err.log
Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_make.cmake:610 (vcpkg_execute_required_process)
  ports/icu/portfile.cmake:51 (vcpkg_configure_make)
  scripts/ports.cmake:145 (include)
Error: Building package icu:x64-osx failed with: BUILD_FAILED
Elapsed time for package icu:x64-osx: 5.831 s

Additional context
Environment is the GitHub actions host:  https://github.com/actions/virtual-environments/blob/main/images/macos/macos-11-Readme.md

Please provide /Users/runner/work/vcpkg/vcpkg/buildtrees/icu/autoconf-x64-osx-err.log first.
Thanks.

Should same with #22934.

@JackBoosY - FYI this was resolved with a
brew install automake libtool

(It was a bit of pain to work out as I don't have a mac and was relying on GitHub actions)

Yeah it needs those tools.

facing the same issue when trying to install qtbase.

facing the same issue when trying to install qtbase.

This also required 'brew install autoconf-archive' on my system (OSX 12.3.1 x86_64) for qtbase. HTH. :)

facing the same issue when trying to install qtbase.

This also required 'brew install autoconf-archive' on my system (OSX 12.3.1 x86_64) for qtbase. HTH. :)

Thank you! 🙏
How does one figure out this was the problem?


double ended slider

Hey,
Thanks so much for building this. I wondered if you might be able to help with sliders having two inputs so you can choose the minimum and maximum?
https://picturepan2.github.io/spectre/experimentals/sliders.html

@amlwwalker Not possible, the slider is one of the types of input elements, the slider will have min & max attributes that you can use to control the slider. If you want to control the min and max values use Javascript to do that.


feat(tuya-panel-lock-sdk): 新增 AesMediaPreview、RemoteManager、DPUtil

Owner

[ ] Nickname(Tuya):  乌迪
[ ] Intra-group review（组内审阅人员）：风眠

Description
Please provide a brief summary of the changes. If it is related to an issue, please bring the issue number, like fixes #${number}.
新增门锁音视频相关组件，组内review，视频相关单测先不写
Package
Please select the relevant package.

[ ] tuya-panel-animation-sdk
[ ] tuya-panel-api
[ ] tuya-panel-electrician-sdk
[ ] tuya-panel-ipc-sdk
[ ] tuya-panel-lamp-sdk
[ ] tuya-panel-lock-sdk
[ ] tuya-panel-robot-sdk
[ ] tuya-panel-outdoor-sdk

Type of change
Please select the relevant option.

[ ] Bug fix (non-breaking change which fixes an issue)
[☑ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Self CheckList
Please check all options.

[ ☑] Intra-group review
[ ☑] ESLint lint
[ ☑] Example Run
[ ☑] Jest Run
[ ☑] No Grammer Error
[ ☑] No Spell Error
[ ☑] No Merge Conflict

@U-DI-Page 单测有问题

Codecov Report

Merging #174 (feff4be) into main (1e9c794) will increase coverage by 0.05%.
The diff coverage is 63.82%.

@@            Coverage Diff             @@
##             main     #174      +/-   ##
==========================================
+ Coverage   61.57%   61.63%   +0.05%
==========================================
  Files         320      335      +15
  Lines       12100    12487     +387
  Branches     2818     2826       +8
==========================================
+ Hits         7451     7696     +245
- Misses       4617     4771     +154
+ Partials       32       20      -12

Impacted Files
Coverage Δ

...ck-sdk/src/AesMediaPreview/AesPictrue/AesImage.tsx
10.63% <10.63%> (ø)

packages/tuya-panel-lock-sdk/src/utils/index.tsx
73.07% <16.66%> (-26.93%)
:arrow_down:

...ck-sdk/src/AesMediaPreview/AesVideo/Controller.tsx
26.08% <26.08%> (ø)

...-panel-lock-sdk/src/AesMediaPreview/AesVideo/fn.ts
32.00% <32.00%> (ø)

...ck-sdk/src/AesMediaPreview/AesVideo/PlayButton.tsx
43.75% <43.75%> (ø)

...el-lock-sdk/src/AesMediaPreview/AesVideo/Video.tsx
51.31% <51.31%> (ø)

...ock-sdk/src/AesMediaPreview/AesPictrue/Picture.tsx
86.36% <86.36%> (ø)

...ages/tuya-panel-lock-sdk/src/utils/DPUtil/utils.ts
88.46% <88.46%> (ø)

...s/tuya-panel-lock-sdk/src/utils/DPUtil/observer.ts
91.11% <91.11%> (ø)

...l-lock-sdk/src/AesMediaPreview/AesVideo/Context.ts
100.00% <100.00%> (ø)

... and 16 more

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 1e9c794...feff4be. Read the comment docs.


Update README.md

Just working through setting this up.
Looks like you just started. I may have a few more changes as I get it working.

Just working through setting this up.
Looks like you just started. I may have a few more changes as I get it working.

Awesome, I appreciate any input! I do have some code in the project already that:

Enables Issue Estimation (with a type)
Adds Estimates to an issue

The problem though is that Pivotal allows you to create completely custom estimate values, so there's no guarantee that they will map perfectly to estimation values in linear. My idea so far is to create some sort of map / rounding algorithm to choose the closest option in the event there is no perfect match.


Check random_compat for PHP >= 7

Check: https://packagist.org/packages/paragonie/random_compat

Implemented as fallback.


Not a huge issue, more of a hey so you know. /usr/local/maven-3.1.1 directory created for maven-3.3.3

Not a huge issue, more of a hey so you know.
/usr/local/maven-3.1.1 directory created for maven-3.3.3
mvn -v shows it is indeed 3.3.3
CentOS 6.6
Chef_Zero
Test-Kitchen
Vagrant

also, sure would be nice to be able to give full version and you default to the Apache URL download and skip the ark checksum if don't care if sha256sum matches for a dev box to have maven blah.

@LevonBecker This should be resolved with how the version attribute is handled in the latest version.  Give it a try and feel free to open this back up if it's still an issue.


Make use of community driver and support of streams

Great, thanks very much
I'll have a look in a week, on vacation at the moment


Automated Greet Workflow

Is your feature request related to a problem? Please describe.
I propose to create a GitHub workflow named "Greetings". This workflow will automatically greet users who create new issues or pull requests in this repository.  The Greeting will look like "Hi there! Thanks for opening this issue. We appreciate your contribution to this open-source project. We aim to respond or assign your issue as soon as possible."
Please assign this issue to me
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@sanmarg
I am assigning this task to you. Please ensure you implement the feature in the best possible way.


[tool] Unable to install APK through flutter run when uninstalled once and installed a previous version

Steps to reproduce

Set the pubspec.yaml version to 1.0.0+1.
flutter build apk --release && adb install build/app/outputs/flutter-apk/app-release.apk
Run the app on the device, and ensure it is v1.0.0+1.
Set the pubspec.yaml version to 1.0.1+2.
flutter run -d the_android_device_id
Ensure it is v1.0.1+2.
Uninstall the current v1.0.1+2, then reinstall the previous version through adb install build/app/outputs/flutter-apk/app-release.apk.
Run the app on the device, and ensure it is v1.0.0+1.
flutter run -v -d the_android_device
The log shows Latest build already installed and launched the app, but the latest APK was not installed.

Expected results
flutter run should installs the latest APK.install
Actual results
When a temporary file of a previously installed version exists, the tool will recognize that the latest version is installed.
Code sample
N/A
Screenshots or Video
N/A
Logs
Logs
[   +2 ms] Stopping app 'app-debug.apk' on ONEPLUS A6010.
[        ] executing: /Users/alex/dev/android-sdk/platform-tools/adb -s f8e67ce3 shell am force-stop com.example.test
[  +95 ms] executing: /Users/alex/dev/android-sdk/platform-tools/adb -s f8e67ce3 shell pm list packages com.example.test
[ +109 ms] package:com.example.test
[        ] executing: /Users/alex/dev/android-sdk/platform-tools/adb -s f8e67ce3 shell cat /data/local/tmp/sky.com.example.test.sha1
[  +61 ms] a46ba751dfd13d0c0c4e153c8f8c52cc00b2a627
[        ] Latest build already installed.
[        ] executing: /Users/alex/dev/android-sdk/platform-tools/adb -s f8e67ce3 shell -x logcat -v time -t 1
[ +242 ms] --------- beginning of main
           12-17 09:53:38.172 I/cmd     (27942): oneway function results will be dropped but finished with status OK and parcel size 4
[   +3 ms] executing: /Users/alex/dev/android-sdk/platform-tools/adb -s f8e67ce3 shell am start -a android.intent.action.MAIN -c android.intent.category.LAUNCHER -f 0x20000000 --ez enable-dart-profiling true --ez enable-checked-mode true --ez verify-entry-points true --ez start-paused true com.example.test/com.example.test.MainActivity
[  +88 ms] Starting: Intent { act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x20000000 cmp=com.example.test/.MainActivity (has extras) }
[        ] Waiting for VM Service port to be available...

Flutter Doctor output
Doctor output
[!] Flutter (Channel [user-branch], 3.27.0-1.0.pre.753, on macOS 14.7.1 23H222 darwin-arm64, locale zh-Hans-CN)

https://github.com/flutter/flutter/blob/256a127da361188c08ff6e6fc314cd59a9ec7e71/packages/flutter_tools/lib/src/android/android_device.dart#L361-L363

Thanks for the detailed steps.
Reproducible using the steps outlined above.

flutter doctor -v
[✓] Flutter (Channel stable, 3.27.1, on macOS 15.1.1 24B91 darwin-arm64, locale en-US)
    • Flutter version 3.27.1 on channel stable at /Users/deanli/dev/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 17025dd882 (16 hours ago), 2024-12-17 03:23:09 +0900
    • Engine revision cb4b5fff73
    • Dart version 3.6.0
    • DevTools version 2.40.2

[!] Flutter (Channel master, 3.28.0-1.0.pre.157, on macOS 15.1.1 24B91 darwin-arm64, locale en-US)
    • Flutter version 3.28.0-1.0.pre.157 on channel master at /Users/deanli/dev/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 27ef1967b0 (6 hours ago), 2024-12-16 20:29:14 -0800
    • Engine revision 23731d6a96
    • Dart version 3.7.0 (build 3.7.0-256.0.dev)
    • DevTools version 2.41.0

I am curious, could you share a little about the workflow that caused you discover this? For example, did you make changes to the native config of your app (e.g. android/app/build.gradle.kts), re-run/install it, and notice that the changes did not make it to the device?

I am curious, could you share a little about the workflow that caused you discover this?

I was developing the new version for a while with the storage migration, in the end I want to double check whether it will cause a repression which may erase previous data, so I uninstalled the developing one and installed one of our previous release.

Im facing the same issue
In my case its in vs code
I created new peoject
I run the project and it stuck at

app installed in emulator but stuck
I did not change any settings
Created a project and was directly running it

@code-ashishkumar Your case seems variant. Please submit a new issue with the required contexts.

@AlexV525
Sure, thanks


Feature request: optionally allow empty files to be generated

Hi there :)
I am planning on using Scaffold to generate Terraform modules boilerplate. Hashicorps recommends using a standard layout with well-known file paths that should be used even if they're empty (main.tf, variables.tf, outputs.tf). This is not a hard requirement though, as Terraform itself does not care -- it's just a convention.
Currently (v0.0.91), an empty render will result in ErrTemplateIsEmpty, which is ignored but prevents the file from being written to (as specified in the docs). While I fully understand this behavior, I would like to be able to optionally bypass this check at the scaffold level. Now the issue as far as I can tell would be introducing options in the scaffold.yaml file, which you might want to avoid to prevent feature creep.
Thank you for your work, and for your time :)

Will definitely need to think about this some more. I know I've been wanting to make some changes that could possibly break some scaffolds, so having some way to opt-into behavior may be necessary.
In the mean time, adding comments to the files so they always have content or if they aren't templates, adding them to the skip list would always copy them (I think)

Adding Terraform comments is what I have been doing in the meantime.
By the way, there is a strange output if the template only contains only gotemplate comment and whitespace (e.g if it's only {{/* ... */}}): the resulting file contains its own path.

.scaffold/terraform-module/scaffold.yaml
questions:
  - name: module_name
    required: true
    prompt:
      message: name

computed:
  module_dir: "infrastructure/modules/terraform-aws-{{ .Scaffold.module_name }}"

rewrites:
  - from: templates/main.tf
    to: "{{ .Computed.module_dir }}/main.tf"

.scaffold/terraform-module/templates/main.tf
{{/* test */}}

infrastructure/modules/terraform-aws-example/main.tf
infrastructure/modules/terraform-aws-example/main.tf

Super weird, found the issue there - fixed in #138. Thank you!

Sounds good, thanks for your work on this one!


Test on py 3.8

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 50.28%. Comparing base (369c9fa) to head (f220738).
Report is 35 commits behind head on main.

:exclamation:  There is a different number of reports uploaded between BASE (369c9fa) and HEAD (f220738). Click for more details.
HEAD has 11 uploads less than BASE

Flag
BASE (369c9fa)
HEAD (f220738)

pytest
12
1

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #157       +/-   ##
===========================================
- Coverage   75.03%   50.28%   -24.75%
===========================================
  Files           8        9        +1
  Lines         833      871       +38
===========================================
- Hits          625      438      -187
- Misses        208      433      +225

Flag
Coverage Δ

pytest
50.28% <ø> (-24.75%)
:arrow_down:

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

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


Add "Note" Feature to friends list

I'm thinking along the lines of a right click option, and it would bring up a box which would allow a person to input a note for a friend. This could be used for remembering people better. OSbuddy has an option like this, and Id love to see it implemented into here. When u hover over the persons name u just created a note for, it would display a cursor-dialogue box.

Duplicate of #1877


FluidSynth

SteamDB app page links to a few games using this
https://steamdb.info/app/206440/
https://steamdb.info/app/292300/
https://steamdb.info/app/292840/
https://steamdb.info/app/294540/
https://steamdb.info/app/347400/
https://steamdb.info/app/434520/
https://steamdb.info/app/666910/
https://steamdb.info/app/903910/
https://steamdb.info/app/1010750/
https://steamdb.info/app/1208460/
https://steamdb.info/app/1239410/
https://steamdb.info/app/1255460/
https://steamdb.info/app/1272200/
https://steamdb.info/app/1482520/
https://steamdb.info/app/1536610/
https://steamdb.info/app/1572320/
https://steamdb.info/app/2125300/
https://steamdb.info/app/2163890/
https://steamdb.info/app/2262930/
https://steamdb.info/app/2395450/
https://steamdb.info/app/2435900/
Brief explanation of the change
Detect FluidSynth by it's libraries.

would have been faster if i didn't have to touch your pr :D


Unblock CI check for unacceptable language

Rewords documentation and APIs that don't meet Apple's unacceptable language standards. Unblocks the CI check that runs the check-unacceptable-language.sh script.
Issue: #1161

@swift-server-bot test this please


[order] pathGroup doesn't appear to work

Problem
I am configuring pathGroups, but they do not appear to be working.
No matter the group I set for the pattern, it always requires that it appear at the top of the external imports.
I've manually verified that minimatch is returning the correct output for some of the test paths that I'm working with.
tsconfig.json
"compilerOptions": {
   ...
   "paths": {
     "@/*": ["./*"]
   }
}

.eslintrc.js
  'import/order': [
    'error',
    {
      groups: ['builtin', 'external', ['sibling', 'parent'], 'index', 'internal', 'object'],
      pathGroups: [
        {
          pattern: '@/**',
          group: 'internal',
        },
      ],
      'newlines-between': 'always',
      alphabetize: {
        order: 'asc',
        caseInsensitive: true,
      },
    },
  ],

Expected
import { StatusCodes } from 'http-status-codes';
import { Route, useRouter } from 'next-router-provider';
import React, { useCallback, useEffect, useRef, useState } from 'react';
import styled from 'styled-components';

import { useSession } from '@/client/providers/session';
import { pageRoutes } from '@/client/routes';

Actual
import { useSession } from '@/client/providers/session';
import { pageRoutes } from '@/client/routes';
import { StatusCodes } from 'http-status-codes';
import { Route, useRouter } from 'next-router-provider';
import React, { useCallback, useEffect, useRef, useState } from 'react';
import styled from 'styled-components';

versions
eslint 7.22.0
eslint-plugin-import 2.22.1
typescript 4.2.3

minimatch outputs (both of which are correct)
> minimatch('@/client/types', '@/**')
true
> minimatch('@external-package/foo-package', '@/**')
false

It would be nice if this could be supported; Vue CLI projects use the @/ prefix to point to source root by default and it would make sense for paths with a slash after the @ to be differentiated between paths without a slash, @/my-internal-module vs.  @external-module

Ah yes, that seems to have done it @Jack-Barry - however that was not obvious! Is there any lingering effects of setting pathGroupsExcludedImportTypes to empty array? Like ideally it would be nice to get the defaults for that minus the alias check.

@nickzelei Indeed that does do it, and is cleaner - thanks for taking some time to look into it. I'm adding a second edit in my previous comment to point others to your answer.

You can also use an eslint setting in order to interpret specific path patterns as internal:
# .eslintrc.yml
settings:
  import/internal-regex: ^@/

https://github.com/benmosher/eslint-plugin-import/blob/master/README.md#importinternal-regex

This seems resolved.
PRs to improve the docs are always welcome!


Fix PivotItem/SecuredByRole render and workspace cost reloading

Fixes #3738
What is being addressed

Cannot surround PivotItem by SecuredByRole element so using error message
Costs relead when roles update

/test-force-approve
UI Only


Update integrations

Updates the integrations in the README to be a bit more general allowing for us to include high visibility repos utilizing the library. Since the integration zoo is based on python frameworks with examples using them, instead of very, very high level frameworks (like Invoke and Open Assistant, where it's mostly through a web interface things happen) they were not added to this list.
This list was generated looking at the most forked and utilized repos based on this dataset for February 12th, 2023

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


go to top modified

I have modified the go to top. You asked me to beautify the button more. I did that. Kindly check..

Hey @mandiraghosh783, there are some issues with the changes you made on the website. Due to the addition of the Back to Top button, Read more button on home page is getting wrongly aligned and wrong format. Kindly look over it.
Preview of Error

Expected Result

But, I have made no changes in index.html

@mandiraghosh783 you have made changes in style.css file which is linked with index.html file as well. You may have changed the formatting in some classes which are linked to index.html file as well. So it will affect that file as well.

ok, i will look into it once again.

its working perfectly fine for me now..

Yepp @mandiraghosh783, I have reviewed the changes and the current website is looking good. So, we will be soon merging this PR after resolving conflicts. @ghulamyazdani will resolve the conflicts on this PR.


Get started with deCONZ API

Using Curl commands get API up and running

API is working. Important to first generate api key, when doing so, the gateway has to be opened from webapp


try to make chrome ext id constant during release for authentication

should be constant now....otherwise no option

https://github.com/amitsingh-007/bypass-links/commit/6e0fc12613195e127183d8b2f748ea879155c728


Adjust for Entry unification

This PR adjust for changes in intake that will be introduced in
intake/intake#490. This branch has been tested against that PR branch, as well
as intake master and the most recent intake release, 0.5.5.
This is how it looks interactively against that PR branch:
In [1]: import databroker

In [3]: list(databroker.catalog)
Out[3]:
['dev1',
 'dev2',
 'fxi_feb',
 'test1',
 'test_unpack1',
 'test_unpack3',
 'dev',
 'csx',
 'example']

In [4]: databroker.catalog.dev1
Out[4]: <Intake catalog: dev1>  # <--- NOT AN ENTRY ANYMORE

In [5]: type(databroker.catalog.dev1)
Out[5]: databroker._drivers.msgpack.BlueskyMsgpackCatalog

Restarted CI to run against current intake master just to be sure.

Confirming this works on my side. Excellent work both 👍

The __dask_tokenize__ methods are left there as a long-form TODOs / skeletons.
We played with them while sorting out how to make this fast.  One of the effects of the changes here is to defer actually computing the tokenization until actually needed (which for most cases for use is "never" because we turned off all of the persist functionality), but if we ever do need the tokenization it will be slow again.  Finishing the implementation of __dask_tokenize__ to not convert the kwargs to a string and then hash it is the way to do that (now that upstream is changed to respect our implementation).

If we use the intake server do we need to tokenize?

I think so, yes. To summarize the state of things:
Previously --- Tokenizaiton happened "greedily" when an instance was first created, so we always paid the price for it, whether the token was actually accessed or not.
In this PR --- Tokenization happens only if and when the token is first accessed. And it is cached, so any repeated access will be fast. Often, in the local use case, it is never accessed, so now things are a lot faster.
In the future --- When the __dask_tokenize__ sketches are finished, we can make tokenization faster across the board.


TestHunspell: reduce the flakiness probability

We need to check how the timeout interacts with custom exception-throwing checkCanceled. The default timeout seems not enough for some CI agents, so let's increase it.

Thanks @donnerpeter !


cannot find adalomo class

In adalomo/instruction-tuning/train.py, AdaLomo is imported from collie.optim but I cannot such optimizer from collie.optim. Would it be possible to help me with finding the implementation of AdaLomo? Thanks!

Hi, AdaLomo now is implemented in the dev branch of collie. It may take some time to merge into the main branch. You can install the dev branch and have a try :)

oh I see. Thank you for your kind reply!


Add Data Type LOCATION

Add location data
Longitude, Latitude, Altitude and location accuracy (preferably as doubles).
This will avoid having  to use another package like geolocation. Health Services already provides this data.

Data types for Health Services can be found here:
https://developer.android.com/reference/kotlin/androidx/health/services/client/data/DataType


Uploaded videos don't load on Safari iOS

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
See title
Screenshots
Safari PWA on iOS:

Brave on Desktop:

Steps To Reproduce

Upload video from other device because of #1568
View item on Safari iOS

(not confirmed)
Expected behavior
Video is visible on Safari iOS
Logs
No response
Device information
No response
Additional context
No response

It's likely a CSS issue. Safari is uniquely glitchy when it comes to video.

It's likely a CSS issue. Safari is uniquely glitchy when it comes to video.

I'm exploring also this issue, you're right it is indeed a CSS issue, specifically text.module.css at line 187
.p:has(> .mediaContainer) .mediaContainer
{
    display: flex;
    width: min-content; /* Safari doesn't like this */
    max-width: 100%;
}

This actually works on Safari for Mac

But on Safari for iOS (or PWA) the story is much different, it tries to render all the possible resolutions of the image that actually is a video, in fact media-or-link.js tries to load the video as img before creating a video element that only then will be populated with the video data.
I'm still investigating, for now I'm battling hydration errors as testing PWAs is its own hell


Remove Minor Version Constraints

Several dependencies are marked upToNextMinor even though they no longer point at 0‐prefixed versions. They are not kept in sync across toolchain packages very well over time, and it causes issues for clients attempting to use them together.

Closing for same reason as apple/swift-package-manager#5769.


listener: deprecate use_proxy_proto

Title: Check all filter chains before injecting proxy proto listener filter deprecate use_proxy_proto
Description: See discussion here

Yes, we should deprecate it, but I'm more concerned about whether we block it or not right now. Do we have a check that it's all or none? If not can we add it behind a runtime guard?

Incidentally, it seems this behavior has been around for a while, it looks like since #1471 if I'm interpreting the code right

+1 for runtime guard and add warning message

I'd say don't change the behavior, deprecate it, and make sure docs show how to add it to the listener filters explicitly.

My concern is the existing behavior makes absolutely no sense. How can it work at all? I'm fine with whatever approach we want, definitely deprecation, but I'm also potentially in favor of warn/fail on mismatch behind a runtime flag.

I agree it makes no sense, but changing it has the risk of breaking someone's config that currently works (even if it doesn't make sense). And if we're deprecating it, the risk doesn't seem worth it. If we weren't going to deprecate it, I'd say definitely fix it (with runtime guard).

OK sounds good. I'm fine either way, but let's definitely do the deprecation.

I have a few questions about how to deprecate something (referring to the breaking change policy)

Deprecators must implement a conversion from the deprecated configuration to the latest vNalpha (with the deprecated field) that Envoy uses internally.

What does this mean in this case? For examples that use use_proxy_proto like  this it seems like I just need to update them to instead use
listener_filters:
- name: envoy.filters.listener.proxy_protocol
  typed_config:
    "@type": type.googleapis.com/envoy.extensions.filters.listener.proxy_protocol.v3.ProxyProtocol

But must there also be some code/tool that manipulates configuration for people?

The PR author deprecating the old configuration is responsible for updating all tests and canonical configuration, or guarding them with the DEPRECATED_FEATURE_TEST() macro.

Is this true for fuzzing tests as well? Referring to this test case
For adding [deprecated = true] to proto definitions, would this only be for v4alpha (there's also v3 and v2)?

For adding [deprecated = true] to proto definitions, would this only be for v4alpha (there's also v3 and v2)?
You only need to add for v3 and run  tools/proto_format/proto_format.sh fix to sync to other versions.

Not fully sure about the fuzzer CC @asraa

+1 on deprecate. The less magic in the filter chains the better IMHO (as well as existing behavior being broken).

I think we should a) deprecate and also b) warn on mismatch (not all or nothing).
I think this will give us good coverage.

This is fixed.

This is fixed.


IE 11 "path" not found error with the v0.5.0-beta4

Hi we've implemented html2canvas and image can be downloaded on chrome fine (with some tweeks still needed), but the same code on IE11/Edge shows the error:
"path" argument is required
could you explain more in detail what does this error mean exactly and if there are typical mistakes that would cause this error.
we are also using bluebird as our promise library. Added in window.Promise = bluebird in our code.
here is how we are currently calling html2canvas
var elem = $([SELECTOR])[0];
var height, width = from elem.height (width)
html2canvas(elem, {
height: height,
width: width,
scale: 2,
logging: true,
onclone: [CUSTOM FUNCTION HERE]
}).then(function (canvas) { ... })

Please share an example on jsfiddle

Hi sorry for the late reply. I've been spending days to recreate it on the fiddle. In the process I figured out what was wrong with mine. So feel free to look at what I did and close the ticket or add more explanation as you will.
Here are what I found:
PLEASE note that my lead decided to use this particular commit version of library
149244d0f86a16097c0cb4c74578d2f61de7b180
so anything I've found pertains to this commit history. Some thing may have changed.
1.) IE11 does not pass the is browser native svg logic in the library, this.support.svg is false.
2.) IE11 gets into the .hasFabric prototype as result of #1
3.) .hasFabric checks for window.html2canvas and window.html2canvas.svg
4.) only window.html2canvas is required for it to work, not window.html2canvas.svg also.
(jsFiddle automatically adds window.html2canvas that's why it worked on the fiddle)
5.) Adding window.html2canvas.svg should produce the error of this ticket (unconfirmed, but it's the logical conclusion from my search).
6.) once the canvas object comes back, you can use canvas.msToBlob() to create the blob object and then save it via navigator.msSaveBlob(blob, filename)
https://jsfiddle.net/kevin074/fd0zszhb/15/
you can open the console once you clicked the download button in the JsFiddle. I've added console logs into the library while searching. The fiddle does not contain #5 or #6 of my findings above.
A successful IE download will be seeing the canvas object coming back.

Forgot to comment. I've found the solution.
I used canvg to change SVG -> canvas, and then use html2canvas to generate the rest of the picture. Here is the code i used to integrate the svg
 $svgElements.each(function () {
        var canvas, xml;
        canvas = document.createElement("canvas");
        canvas.className = "screenShotTempCanvas";
        xml = (new XMLSerializer()).serializeToString(this);

        // Removing the name space as IE throws an error
        xml = xml.replace(/xmlns=\"http:\/\/www\.w3\.org\/2000\/svg\"/, '');
        window.canvg(canvas, xml);
        $(canvas).insertAfter(this);

})
You'd need to re-show the original svg and remove the canvas element after you are done with it.


Combine ibm and sbm

This PR is created in order to include shifted boundary method into immersed boundary method based on latest Master branch and add_sb branch.

[x] combine sbm method
[x] add cylinder2D test case
[x] check drag/lift coefficient and add ipynb file
[x] add pytest file, README.md

@cekees The test for sbm has been passed. The only error now we have is the job exceeded the maximum time limit for jobs and has been terminated as shown in Travis. Do you have some suggestions to solve the problem? After solving this problem, we can merge this branch.

The builds time out sometimes due to loads on travis.


clh: Rely on the PCI BDF information to locate the hotplugged block devices in the geust

We now relies on the predicted block device path (VmPath) from host to locate hotplugged block devices in the guest, which can be not reliable. As the updated version of clh v0.9.0, we now have the BDF information for all hotplugged devices, and we should use this information to locate devices in the guest.

Close the issue in favor of #3002


Fix Hanging CI

This PR:
Finally fix the test hanging issue.  The main issue was that the test task implementation could infinite loop.  The issue was exposed one of the test tasks takes the internal event streams.  The task is just a while loop that selects the next event from any event receiver it has.  Because in the case of the restart test we create new internal senders for the new nodes and kill all of senders of the of the old nodes, the receivers held by the test tasks are all closed.  This means that each time we select one of the receivers next event we immediately get an error and the loop just becomes a hot loop that constantly is processing the error and never awaiting.  This spin loop then starves the executor from running so none of the async code can run.   It was easy to reproduce by limiting the async executor threads to 1 on tokio.
In the process of debugging this I found a few other issues that I fixed:

In the spinning test task I now spawn all the new nodes in parallel so they should start a roughly the same time
I noticed that when we create consensus and before we call start consensus we spawn the first timeout tasks, if it takes awhile to call start_consensus we could get a timeout before even getting the event for genesis.  I fixed that by starting the first timeout task when we start consensus
We remove a failed view from our failed view list if all nodes who failed it eventually succeed it, but in some tests the nodes might go down or not decided it before the test completes so I made it so that we remove the failed view if a quorum of nodes succeed it.  (this scenario should not be possible but I think it's something that might have been fixed with the timeout change
The completion task was trying to shutdown nodes but that logic is already handled when the test finishes so it is duplicated.
Rebalances the time of the CI jobs, they should be all about 10 minutes now
Update the justfile to log all logs at info.  That's the level we log in production so it shouldn't be too spammy.  Also the previous justfile would only show logs of consensus starting up and was not usually helpful.

This PR does not:
Fix the underlying issue that some test tasks are holding dead receivers when all nodes are restarted.  I think it's just the view sync task and we should probably not spawn it for the restart tests.
Key places to review:
Changes to spinning_task, the new initial timeout logic, changes to test_task

This has gone through at least 5 or 6 consecutive CI successes without a failure, and more like 10 if you ignore the marketplace test failures which I believe were not related


ha bridge not paired with my Google assistance"OK GOOGLE "home control of updated google assitance Application on android cell phone

Hello ,
A Warm hello from India
Thank you very much for HA bridge , Its very usefull for me and for any one .
I'm writing here something because of that , as people always want more and more .
I have Google home which is easily connect with  ha-bridge , and now i'm able to use its TCP Device to control my home automation system.
Here Comes More ,
My Android Phone also  have OK Google Feature (Attaching screenshot-
 )  with Home control , tab on home control , add the philips hue device  but its asking

me for login as both i.e. my cell phone and ha-bridge are on same network (Which is not happening with Google home App i.e. ha-bridge directly get connected with Google home and populated all the device )
Please help to add habrodge to my google assitance
What should need to be done please let me know
Eagerly waiting for your reply  ...
My Email-Id -<EMAIL_ADDRESS>Skype Id-avaneesh.pathak
Please , Help me on that ...

@vngic the help section is to answer questions, not tutorials.  you need to watch this video to understand how to install it and get it started.  https://www.youtube.com/watch?v=93PE8avLq5M

https://github.com/bwssytems/ha-bridge#google-assistant

@vngic Unfortunately, the phone version of the Google Assistant can not connect to local network devices. Only the Google Home appliance can do that.

Hey ..
Is there anyway to connect phone version to connect with google home..??

A: The ha-bridge requires a physical Amazon Echo, Dot, Tap or Google Home device and does not work with prototype devices built using the Alexa Voice Service e.g. Amazon’s Alexa AVS Sample App and Sam Machin’s AlexaPi. The AVS version does not have any capability for Hue Bridge discovery!

sorry for my previous comment , i just need to ask about is there any way to connect phone version of google assitance with habridge  .
any way ...?

Or should i add harbridge to my account which is created on hue website .. i think on that way i can connect habridge to home assistance

Please suggest me something , some code changes some extra device anything ..

I'm sure if there is option for home control in phone version ,there is way to connect with it .

you can build your own.  all the ha-bridges need a physical device

HA-Bridge is local lan.
As far as I'm aware the Google stuff goes out to the cloud?
On 9 Mar 2017 2:15 pm, "vngic"<EMAIL_ADDRESS>wrote:

Please suggest me something , some code changes some extra device anything
..
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bwssytems/ha-bridge/issues/524#issuecomment-285362053,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACkXzeFoDZzKJzq9TxtArNibolSlGHbKks5rkAl1gaJpZM4MWeGh
.

hey Owner ..
Any suggestions...

@Matt8119  I love to build my own , but need your help .. can you help me out?

you can use autovoice with tasker.

https://ifttt.com

Great ..you have something on that?
I mean where i can start with ..

'google search" is your friend.

hahaha...
You are also my friend ..

@vgnic You will need to do your homework, this is a contribute what you know project and we can help along the way if you have understanding. If you do not want to take the time to investigate and to understand yourself, it would be difficult to spend days with you. We do this in our spare time.

Yeah ..Understood ..I'll do my homework ..For sure ..if I stuck somewhere , i need your help guys..

Closign for now


obsured? method is not waiting for element to be present before it performs it's check

I have written this code in my project

'' while @b.link(loc.first => loc.last).obscured?

This line immediately fails before element to be present but it's reporting the error by saying that it has waited for 30 seconds in FIREFOX
Error
Watir::Exception::UnknownObjectException: element located, but timed out after 30 seconds, waiting for #<Watir::Anchor: located: true; {:xpath=>"//a[text()="Workers' Compensation"]/../../following-sibling::tr[2]/td/a", :tag_name=>"a"}> to be present

AGCSquoteCreation Test1
?[31mFailure/Error: raise unknown_exception, msg?[0m
?[31m?[0m
?[31mWatir::Exception::UnknownObjectException:?[0m
?[31m  element located, but timed out after 60 seconds, waiting for #<Watir::Anchor: located: true; {:xpath=>"//a[text()="Workers' Compensation"]/../../following-sibling::tr[2]/td/a", :tag_name=>"a"}> to be present?[0m
?[36m# ./Source/FrameWorkModules/Elements/Button.rb:50:in clickLink'?[0m ?[36m# ./Source/FrameWorkModules/Browser/Framework.rb:12:in block in execute'?[0m

It's intentional that #obscured? does not wait for the element to be present - see https://github.com/watir/watir/pull/804#issuecomment-429669241
You highlighted Firefox. Does that mean you only have the problem in Firefox? It doesn't occur in other browsers?

Yes it's not happening in chrome.
On Fri 9 Nov, 2018, 9:18 PM Justin Ko<EMAIL_ADDRESS>wrote:

It's intentional that #obscured? does not wait for the element to be
present - see #804 (comment)
https://github.com/watir/watir/pull/804#issuecomment-429669241
You highlighted Firefox. Does that mean you only have the problem in
Firefox? It doesn't occur in other browsers?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/watir/watir/issues/818#issuecomment-437400603, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AeOm7uFSk5YBahp6KWIyVa55f6SjBMnmks5utaOwgaJpZM4YWtIb
.

It'd help if you can isolate the markup needed to reproduce the problem.

As an aside, if the element remains obscured, won't that loop run forever? Might be safer to do:
@b.link(loc.first => loc.last).wait_until(&:present?).wait_while(&:obscured?).click

This seems to be good idea and I am actually expecting something like this.
But here wait_until and wait_whike only waits for 30 sec , can I set my own
time?
On Fri 9 Nov, 2018, 9:26 PM Justin Ko<EMAIL_ADDRESS>wrote:

As an aside, if the element remains obscured, won't that loop run forever?
Might be safer to do:
@b.link(loc.first => loc.last).wait_until(&:present?).wait_while(&:obscured?).click
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/watir/watir/issues/818#issuecomment-437403681, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AeOm7quKxUtu1osAzv1WWyZc5mDWyQCUks5utaW_gaJpZM4YWtIb
.

Hi, no, this solution is not working for me. It throws the same error
I have written this line
@b.link(loc.first => loc.last).wait_until(&:present?).wait_while(&:obscured?).click

Watir::Exception::UnknownObjectException: element located, but timed out after 60 seconds, waiting for #<Watir::Anchor: located: true; {:xpath=>"//a[text()="Workers' Compensation"]/../../following-sibling::tr[2]/td/a", :tag_name=>"a"}> to be present

AGCSquoteCreation Test1
?[31mFailure/Error: raise unknown_exception, msg?[0m
?[31m?[0m
?[31mWatir::Exception::UnknownObjectException:?[0m
?[31m  element located, but timed out after 60 seconds, waiting for #<Watir::Anchor: located: true; {:xpath=>"//a[text()="Workers' Compensation"]/../../following-sibling::tr[2]/td/a", :tag_name=>"a"}> to be present?[0m
?[36m# ./Source/FrameWorkModules/Elements/Button.rb:45:in `clickLink'?[0m
?[36m# ./Source/FrameWorkModules/Browser/Framework.rb:12:in `block in execute'?[0m
?[36m# ./Source/FrameWorkModules/Browser/Framework.rb:11:in `each'?[0m

You can set your own time by passing in the timeout parameter:
browser.element.wait_until(timeout: 5, &:present?)

There was a chance that #obscured? wouldn't be a perfect approximation. We'll need the markup/CSS to help address the problem.

Actually its not working. It is not even waiting it immediately throws the
error .
On Sat 10 Nov, 2018, 12:02 AM Justin Ko<EMAIL_ADDRESS>wrote:

You can set your own time by passing in the timeout parameter:
browser.element.wait_until(timeout: 5, &:present?)
There was a chance that #obscured? wouldn't be a perfect approximation.
We'll need the markup/CSS to help address the problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/watir/watir/issues/818#issuecomment-437452537, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AeOm7vzCwk0GKKezDryuWZVpO0E9qYHWks5utco3gaJpZM4YWtIb
.

@jkotests
Actually now I have noticed that normal click method is not even waiting for element to present. That's why wait_until(&:present?) is not working I think.

@jkotests  Hi, I noticed that it works(waiting for element to be visible) for other sites. But it's not waiting for the site I am automating,it immediately throws the error. What might be the reason?

Hey @jkotests, I am able to reproduce this in Internet Explorer 11 as well. Here is a quick test:
b = Watir::Browser.new :ie
b.goto 'http://watir.com'

b.div(class: 'intro').present? #=> true

b.div(class: 'intro').obscured? # raises Watir::Exception::UnknownObjectException

Can you run this with Selenium::WebDriver.logger.level = :debug and get me the output?

Yes, here it is for IE - https://gist.github.com/kapoorlakshya/f622b9be8395678b2b80196a3119f78d

Hmm, it looks like it is ending up in the loop for ElementNotFound, which is why it is checking for an iframe. Can you run it again also adding Watir.logger.level = :debug? Thanks!

I think the problem is in Element#scroll_into_view, which is called by #obscured?.
b.div(class: 'intro').scroll_into_view
#-> C:/Ruby23/lib/ruby/gems/2.3.0/gems/watir-6.15.0/lib/watir/elements/element.rb:739:in `raise_present': element located, but timed out after 30 seconds, waiting for #<Watir::Div: located: true; {:class=>"intro", :tag_name=>"div"}> to be present (Watir::Exception::UnknownObjectException)
#-> 	from C:/Ruby23/lib/ruby/gems/2.3.0/gems/watir-6.15.0/lib/watir/elements/element.rb:791:in `rescue in element_call'
#-> 	from C:/Ruby23/lib/ruby/gems/2.3.0/gems/watir-6.15.0/lib/watir/elements/element.rb:802:in `element_call'
#-> 	from C:/Ruby23/lib/ruby/gems/2.3.0/gems/watir-6.15.0/lib/watir/elements/element.rb:352:in `scroll_into_view'
#-> 	from webdrivers.rb:11:in `<main>'

IE and Firefox do not like scrolling to the div - both raise the Selenium::WebDriver::Error::ElementNotInteractableError. Message says that the element is not reachable by keyboard. Seems like a weird expectation.
We seem to have 2 problems:

#scroll_into_view does not have specify a precondition for #element_call. As result, there is no retry.
Some drivers do not like scrolling to non-keyboard focusable elements.

@p0deje what do you think of pulling watir-scroll code directly into Watir? I was going to ask about doing this for Watir 7, but no reason not to do it ahead of time. It is technically currently a Monkey Patch since it puts additional methods on Watir objects rather than composing new objects. Unlike WatirAngular, etc, this code is universally applicable to users, and there isn't an alternative implementation that this will prevent people from using. Plus the code seems super stable at this point. We discussed at TPAC adding this functionality to w3c specification directly in future versions, so Watir will eventually be updating it directly anyway.
It would be useful right now because we can avoid using #scroll_into_view here.

Oh great, problem is diagnosed now
On Thu 29 Nov, 2018, 3:21 AM Titus<EMAIL_ADDRESS>wrote:

@p0deje https://github.com/p0deje what do you think of pulling
watir-scroll code directly into Watir? I was going to ask about doing
this for Watir 7, but no reason not to do it ahead of time. It is
technically currently a Monkey Patch since it puts additional methods on
Watir objects rather than composing new objects. Unlike WatirAngular, etc,
this code is universally applicable to users, and there isn't an
alternative implementation that this will prevent people from using. Plus
the code seems super stable at this point. We discussed at TPAC adding this
functionality to w3c specification directly in future versions, so Watir
will eventually be updating it directly anyway.
It would be useful right now because we can avoid using #scroll_into_view
here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/watir/watir/issues/818#issuecomment-442618363, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AeOm7koXQQ_KV4y0a9yN1ebbn3CLgJc_ks5uzwVagaJpZM4YWtIb
.

@titusfortner Some of APIs watir-scroll uses are not part of any standard (e.g. window.scrollTo), but quick search shows that all of them are supported by major browsers, so I have nothing against pulling it into Watir codebase. Let's only keep its API the same.

@jkotests
Can you please help me here
https://github.com/SeleniumHQ/selenium/issues/7917


WIP | Fix Sonarqube pipelines (TS Lint issues)

What's the purpose of this pull request?
Our Sonarqube pipeline is triggering some errors and warnings about Typescript ESLint issues, those changes will fix them.
How to test it?
Those pipelines happens in our store projects. So, we can check if the pipeline will fail in the PRs below.
Store Components
Marin Brasil

@Gmantiqueira can u make a rebase to fix all conflicts?

@Gmantiqueira can u make a rebase to fix all conflicts?

@Gmantiqueira can u make a rebase to fix all conflicts?

@emersonlaurentino I'm gonna make another PR, did some mistakes here.

@Gmantiqueira can u make a rebase to fix all conflicts?

@emersonlaurentino I'm gonna make another PR, did some mistakes here.


Add possibility to edit embargo on bitstream of archived Item

References

Fixes #1129

Description
This PR add the possibility to edit embargo on bitstream of archived Item by means use of the manage resource policies functionality.
Instructions for Reviewers

Login as someone with Admin level permissions (full Admin, Comm/Coll Admin)
Edit an Item
Go to Bitstreams tab
Edit a Bitstream
try the link to edit the resource policies
Add a new policy to the bitstream

List of changes in this PR:

Added an authorization page for managing bitstream's resource policies, like for community, collection and item
Removed embargo field from the edit bitstream form
Added a link to the manage bitstream's resource policies

Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes TSLint validation using yarn run lint
[x] My PR doesn't introduce circular dependencies
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[x] If my PR includes new, third-party dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.

@tdonohue the issue was due to an issue with date utils and it's been already resolved within the https://github.com/DSpace/dspace-angular/pull/1201 so I've merged into this PR and it works properly. Now this PR depends on https://github.com/DSpace/dspace-angular/pull/1201

As discussed today, merging this as-is, since the reported embargo date issue was moved to a separate ticket: #1256


Azure digital twins explorer - authentication fails

Hi,
I've been trying unsuccessfully to login with this sample:
https://docs.microsoft.com/en-us/samples/azure-samples/digital-twins-explorer/digital-twins-explorer/
-just keep getting "Authentication failed... "
I've tried from multiple local machines, Windows and Linux, using different Azure accounts to login first (MSDN subscription, two factor internal...) to different DT instances... all fail. My user is a Digital Twins owner.
Wondering if I'm missing something
thanks

@pietronromano Thanks for the feedback! We are currently investigating and will update you shortly.

@pietronromano Thanks for the feedback! We are currently investigating and will update you shortly.

@pietronromano I am able to repro your issue. Getting the same issue.

I will check this issue internally and update you as earliest. Appreciate your time and patience.

@pietronromano I am able to repro your issue. Getting the same issue.

I will check this issue internally and update you as earliest. Appreciate your time and patience.

@pietronromano I hope you won't mind me asking but can you please confirm you didn't forget to add "https://" at the beginning of the ADT URL when adding it in the UI?

@pietronromano I hope you won't mind me asking but can you please confirm you didn't forget to add "https://" at the beginning of the ADT URL when adding it in the UI?

@kartben Thank you for the suggestions.
@pietronromano Issue with the role assignment.
While adding role on on the role assignment page, fill in the values (must be completed by a user with sufficient permissions in the Azure subscription):
Role: Select Azure Digital Twins Data Owner from the dropdown menu
Assign access to: Use User, group or service principal
Select: Search for the name or email address of the user to assign. When you select the result, the user will show up in a Selected members section.

This will resolve the issue. Please add and update if you are still blocked.

@kartben Thank you for the suggestions.
@pietronromano Issue with the role assignment.
While adding role on on the role assignment page, fill in the values (must be completed by a user with sufficient permissions in the Azure subscription):
Role: Select Azure Digital Twins Data Owner from the dropdown menu
Assign access to: Use User, group or service principal
Select: Search for the name or email address of the user to assign. When you select the result, the user will show up in a Selected members section.

This will resolve the issue. Please add and update if you are still blocked.

thanks - I already had that role and was using https:// of course
As a general feedback, I'd suggest adding https:// to the ADT URL if the user forgets it... it would remove a common source of error

thanks - I already had that role and was using https:// of course
As a general feedback, I'd suggest adding https:// to the ADT URL if the user forgets it... it would remove a common source of error

@pietronromano We hope your issue is resolved now. Please let me know if you face any issues.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@pietronromano We hope your issue is resolved now. Please let me know if you face any issues.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Thanks a lot to the team for your help
Turns out I was simply missing a  az account set --subscription  to get the tool to select between a number of available subscriptions

Thanks a lot to the team for your help
Turns out I was simply missing a  az account set --subscription  to get the tool to select between a number of available subscriptions


Add override image, ruby version and upgrade jenkins lib

Signed-off-by: Sayali Gaikawad<EMAIL_ADDRESS>Description
The current docker image did not have jruby installed which is used by this project. Hence we added it and published a new version of docker image. Also the new lib adds the support to specify the jruby version which is used to install gem and verify if the gem is signed or not.
Related PRs: https://github.com/opensearch-project/opensearch-build/pull/3169
https://github.com/opensearch-project/opensearch-build-libraries/pull/119
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has documentation added

[ ] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

We've been using JRuby 9.2.9 for previous builds. I assume that 9.3 should be fine as well, right?

Yes we are only using ruby to install the gem and verify the signature.


LibWeb/CSS: Wrong width of flexbox items with width: fit-content -> width: 100%

Reduced from audio attachment control on Discord.

test

Firefox:

Ladybird:

(note also the wrong body height / vertical overflow)
I couldn't reduce this further, removing any of the nesting or styles makes this look OK (except the vertical overflow).

Probably(?) related: https://github.com/SerenityOS/serenity/issues/21706


support for single page extraction

Simply added an argument as I need to extract a specific page

LGTM

new package published as 1.4.0


Ruby rails build failure

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
When I tried to build a Ruby app:
❯ nixpacks build app --name app-nix --verbose

╔══════════════════════════════ Nixpacks v0.15.0 ══════════════════════════════╗
║ setup      │ pkgs: imagemagick, procps, libpq-dev, libmagickwand-dev         ║
║            │ cmds: curl -sSL https://get.rvm.io | bash -s stable && .        ║
║            │ /etc/profile.d/rvm.sh && rvm install 3.1.2 && rvm --default use ║
║            │ 3.1.2 && gem install bundler:2.3.23 && rm -rf                   ║
║            │ /usr/local/rvm/src                                              ║
║            │ echo 'source /usr/local/rvm/scripts/rvm' >> /root/.profile      ║
║──────────────────────────────────────────────────────────────────────────────║
║ install    │ bundle install                                                  ║
║──────────────────────────────────────────────────────────────────────────────║
║ build      │ bundle exec rake assets:precompile                              ║
║──────────────────────────────────────────────────────────────────────────────║
║ start      │ bundle exec puma                                                ║
╚══════════════════════════════════════════════════════════════════════════════╝

Error: Writing app

Caused by:
    Permission denied (os error 13)

If I use sudo, it starts building but fails:
 > [internal] load metadata for ghcr.io/railwayapp/nixpacks:debian-1669098887:
------
failed to solve with frontend dockerfile.v0: failed to create LLB definition: rpc error: code = Unknown desc = error getting credentials - err: exit status 1, out: ``
Error: Docker build failed

To reproduce
On an Apple M1 try to build a Ruby app.
Expected behavior
The app to build and be runnable.
Environment
nixpacks 0.15.0
Ventura 13.0.1
Apple Silicon

This looks like a Docker issue on your machine. Can you please provide a repo to reproduce this issue on. Closing for now, but will reopen if it is a Nixpacks issue.

I stumbled on this error today as well for a Remix app using a fresh install of Nixpacks. I was able to resolve it by pulling the docker image manually using docker and rerunning the command.
docker pull ghcr.io/railwayapp/nixpacks:debian-1670285109
sudo nixpacks build . --name tabello-app

Environment
nixpacks 0.16.0
Ventura 13.0.1
Apple Silicon M1

I'm having the same issue and resolved it the same way. To be more precise;

Getting this error without sudo: Error: Writing app Caused by: Permission denied (os error 13)
Unable to pull with sudo
Manually running docker pull
It now works with sudo (but not without sudo)

I'm under the impression that it should work without sudo.
Environment
nixpacks 1.0.3
Apple Silicon M2
Ventura 13.0.1
Docker Desktop 4.14.1

For a newer Rails project I am not seeing this
But for an older ruby 2.6 I am seeing this and running as sudo is not really a fix. Is there a way to debug this? --verbose isn't given any more information than permission denied.
I am on a M1 chip.
cc @coffee-cup

@frontsideair
I figured it out. In our Rails application locally we are mounting something shared in the container for development. These a .bash_history or .viminfo
They are created with low file permissions
-rw-------  1 Henrik  staff   5718 Sep 29 15:44 .viminfo

After removing those files I am able to build our rails application without any issues.
@coffee-cup Would it be an idea to have something like .nixpacksignore or something?

In my case, deleting all untracked files with git clean -fdx solved the issue. Most likely the problem was node_modules, which I installed with pnpm and linked from a global cache.

This!!! Thank you 🙏

This problem can manifest in a multitude of ways, as is obvious by reading other users' experiences here.
Excuse my lack of nixpacks-specific knowledge here; I may use some terminology incorrectly.
For Go projects, this can be a file permission issue. To be even more specific, for Go projects using devenv, your go modules are added to .devenv/state/go/pkg. When nixpacks copies this directory into its temporary directory during nixpacks build ..., it copies the contents of .devenv/state/go/pkg with existing file permissions. These permissions are 444 for regular files, which is read-only, even by the owning user.
During most build processes, read-only dependencies are fine, but nixpacks opens files with O_WRONLY|O_CREAT|O_TRUNC|O_CLOEXEC. Which means Go dependencies must be writable by the user running nixpacks build ....
My solution was:
chmod 600 .denvenv/state/go/pkg

It would be great if it was easy to understand which file is causing this error. Right now it's not possible, especially on macos which doesn't have strace and friends.


zfs: enable volume.size property

Signed-off-by: Christian Brauner<EMAIL_ADDRESS>

@stgraber, I'd call this a bugfix not an api extension. Objections?

I'm fine with this as a bugfix, but the same should also apply to btrfs for consistency and we need tests for this stuff.

@stgraber, sure my plan was to do the same for btrfs as well. But whether we do this as a separate branch or here doesn't really matter too much. :)

jenkins: test this please

jenkins: test this please

Closing this for now. Can re-open when we have an implementation of the above plan.


[Feature]: Decrease repo size with the robot.

Checklist

[X] I've searched for similar issues and couldn't find anything matching
[X] I've discussed this feature request in the K8sGPT Slack and got positive feedback

Is this feature request related to a problem?
No
Problem Description
No response
Solution Description
We can add some useful bots to help me manage the repo size. Here is an example for reducing the repo size, please take a look: https://github.com/Aisuko/k8sgpt-operator/pull/1
Benefits
Automatically help us to decrease the repo size.
Potential Drawbacks
Of course drawbacks forward.
Additional Information
I believe it is free to plan is ok for covering the public project.

Thanks @Aisuko great suggestion. @thschue any remarks? Otherwise, I'll go ahead and add it.


Canvas GetPixelColor Removes Container Positioning

Describe the bug
Screen set to Landscape in Designer (same issue when Portrait)
A vertical arrangement Centre/Centre
A canvas with dimensions larger than the vertical arrangement, inside the vertical arrangement
An imagesprite, centred to the canvas

If the GetPixelColor block is run on any x,y coordinates, the color is returned, but the vertical arrangement loses its centre/centre setting and the canvas is set to 0,0 in the vertical arrangement. I have found that if the imageSprite is set to larger than the vertical arrangement, the same issue occurs but drag functionality on the imageSprite is also lost.

Affects

[ ] Designer
[ ] Blocks editor
[x ] Companion
[x ] Compiled apps
[ ] Buildserver
[ ] Debugging
[ ] Other... (please describe)

Expected behavior
Run the GetPixelColor block, which returns the colour, and all elements and positioning remains unchanged.
Steps to reproduce
Screen set to Landscape in Designer (same issue when Portrait)
A vertical arrangement Centre/Centre
A canvas with dimensions larger than the vertical arrangement, inside the vertical arrangement
An imagesprite, centred to the canvas
Request getPixelColor of the middle pixel of the canvas (or any other position)
Canvas moves to 0,0 of vertical arrangement, is no longer centred, and dragging of imageSprite can also be affected.
(This doesn't seem to be related to the "really large canvas bug, #2404", canvas only set to 600x800 in a 300x400 VA.)
Example aia
dragImagebug.aia.zip

The example aia file doesn't seem to be a real aia file. Could you re upload in aia file format? Thanks

@daki7711 I think GitHub doesn't support .aia file format so maybe you need to rename the file after download.

@TimCarterLinux would it be possible for us to know which android version you used for this? Thanks

@TimCarterLinux thanks, does this problem still happen? Cause I wasn't able to replicate this problem with my device. Trying to figure out where the problem is stemming from

Just retested on devices as above, and the problem still exists on both
A short webm video attached to demonstrate (remove the .zip)
spritepixel.webm.zip

I tried re-uploaded the hland.jpg image, I even tried with a completely different image (and removing hland.jpg from the project), the same thing happens

@TimCarterLinux I think the first thing for me is to find a device that has the same problem. Until now all the devices (three chromebooks, two android tablet, one ipad) I tested works fine with this aia. Have you ever experience something similar with a different aia?

I just tried it on my chromebook, and like you say no problem. Could possibly be an issue with My computer (Linux Xubuntu 20.04) and Chromium browser ? Just tested with Firefox and the problem persists.

@daki7711 please look at this method I am sure you will definitely find something realted this here. Especially on this line...
https://github.com/mit-cml/appinventor-sources/blob/77882bbd799214ddc44dc84d0d7fdae9d5e3b42b/appinventor/components/src/com/google/appinventor/components/runtime/Canvas.java#L448

@ewpatton Do you think it is worth investigating different browsers?

@preetvadaliya @ewpatton is this still open, I will like to work on this

@preetvadaliya @ewpatton, how can i view the Logs mad by Log.e() while using "Do it" Feature in blockly editor. I connected my phone using usb cable. App is running but not able to find the logs.

To see logs, you need to use adb logcat. adb is provided as part of the Android SDK.

Already tried it but when adb logcat running it is killing the companion mode and when companion mode is running it is killing adb logcat process.

Are you running in the emulator?

No I am using real device, But i have separately downloaded the adb than running it from terminal

i want to contribute in issue please assign me

@Vickyaryan33 @arinmodi has already submitted a pull request that may address this issue. Please look for other issues marked help wanted for potential first contributions.


MODCHARTS

What feature do you want to get added on the base engine?
modcharts.
ik basicly you can but not like from KE
ik their working on it but i think it will be a good thing cause i am tired of using noteTweenX/noteTweenY
i am not sure if this actually gonna be confirmed as a feature but i will like to see this :)
To test your sight, and reliability, please select the option of what should NOT be requested.


Feature: AMQP queue headers

Feature: roadrunner-php/issues#22
Declare AMQP Queue headers
// ...
'connector' => new AMQPCreateInfo(
    // ...
    queueHeaders: [
        'x-queue-type' => 'quorum',
        'x-dead-letter-exchange' => 'jobs-dlx',
    ],
),
// ...

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (9185d6c) 86.52% compared to head (7043283) 86.56%.
Report is 2 commits behind head on 4.x.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##                4.x      #59      +/-   ##
============================================
+ Coverage     86.52%   86.56%   +0.03%
  Complexity      191      191
============================================
  Files            29       30       +1
  Lines           579      588       +9
============================================
+ Hits            501      509       +8
- Misses           78       79       +1

Files
Coverage Δ

src/Jobs.php
96.66% <100.00%> (+0.44%)
:arrow_up:

src/Queue/AMQPCreateInfo.php
100.00% <100.00%> (ø)

src/Queue/AMQP/Header.php
0.00% <0.00%> (ø)

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


[MGDOBR-875] - Adding submitted and received time check to the perf test along with some minor adjustments

MGDOBR-875
Depends on https://github.com/afalhambra/webhook-perf-test/pull/1
PR owner checklist
Please make sure that your PR meets the following requirements:

[x] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[x] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[x] The layers in the kustomize folder have been updated accordingly.
[x] All new functionality is tested
[x] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Deploy BOT
Comment with `/deploy <dev,stable> when the PR has been merged to deploy to a target environment.

@sutaakar it should be ready to review now - thanks!

@afalhambra The imports are not sorted properly (see PR check). Please fix it once back from PTO.


Updated French translation

Updated French translation (native french speaker here)

Thank you for this. I randomly picked french to do a poc :)


Inject Error

use tag1.4 in visual studio 2017. the app can inject and capture in one of my android device, but can't inject in other device.  there is the log.
renderdoc_errorLog.txt

Does this happen with the official v1.4 build? I can't support any problems you encounter on Android that aren't reproducible on an official stable build or nightly build due both to the requirement for interceptor-lib, and how unreliable the platform is in general leading to build variation and unpredictability.
I also can't investigate this bug unless you provide steps to reproduce and give the details of your environment. Since you say it works on one device the chances I'll be able to reproduce it on one of my available devices is very low, and in fact that indicates that the bug is likely not in RenderDoc but in the version of Android on the device itself.
The log indicates that android.app.ApplicationLoaders.getClassLoader was found but then never called. To debug this you'd need to find out why that isn't called on your version of Android but I don't know how you would do that. It's very likely that this is a bug in your device or Android version.
If you can show that this could be a RenderDoc bug, or provide steps to reproduce it that I can follow, I'll look into it. My expectation is that this is an Android bug though especially when it is device-specific issue.

I have root the device.Maybe  the   error caused because this.I want to fix it, but dont know how to start, and what reason the error become.

If you've rooted the device I can't predict what could be going wrong, something is breaking the normal call to that function getClassLoader but whether it can be fixed I don't know.


[ci] Increase timeout for Fast Verilated Earl Grey Tests

Sometimes CI fails due to the timeout of
watchdog_functest_sim_verilator.
This commit increases timeout from 3600 s to 5400 s.
Signed-off-by: Vladimir Rozic<EMAIL_ADDRESS>

FWIW it might be better to trim/subdivide the test so it won't take as long. 1Hr is already a long time to wait for CI results.

@vrozic Can you link to recent test logs where this times out? This would be useful to (1) asses how frequently this happens, (2) reference back if something similar happens in the future, and (3) guide trimming / dividing the test as @drewmacrae suggests (in a separate issue / PR). TIA!

It seems that the timeouts are no longer happening, so the problem must have been resolved some other way. I am therefore closing this PR.


Read from Workers KV for all gallery pages (as composites)

Not necessary: we have changed the tickets so they are instead read from concrete Redis queries.


Feature/tuya panel szos sdk

Owner

[x] Nickname(Tuya): tuya-panel-szos-sdk
[x] Intra-group review（组内审阅人员）：羲之(祝泽森)

Description
Please provide a brief summary of the changes. If it is related to an issue, please bring the issue number, like fixes #${number}.
Package
Please select the relevant package.

[ ] tuya-panel-animation-sdk
[ ] tuya-panel-api
[ ] tuya-panel-electrician-sdk
[ ] tuya-panel-ipc-sdk
[ ] tuya-panel-lamp-sdk
[ ] tuya-panel-lock-sdk
[ ] tuya-panel-robot-sdk
[ ] tuya-panel-outdoor-sdk
[x] tuya-panel-szos-sdk

Type of change
Please select the relevant option.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Self CheckList
Please check all options.

[x] Intra-group review
[x] ESLint lint
[x] Example Run
[x] Jest Run
[x] No Grammer Error
[x] No Spell Error
[x] No Merge Conflict

@zesenzhu 组内 review & 修改文件冲突

Codecov Report

Merging #164 (2551336) into main (0050bff) will increase coverage by 0.43%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##             main     #164      +/-   ##
==========================================
+ Coverage   61.47%   61.91%   +0.43%
==========================================
  Files         316      322       +6
  Lines       12009    12272     +263
  Branches     2775     2831      +56
==========================================
+ Hits         7383     7598     +215
- Misses       4597     4645      +48
  Partials       29       29

Impacted Files
Coverage Δ

...l-szos-sdk/src/components/steering-wheel/index.tsx
70.94% <70.94%> (ø)

...zos-sdk/src/components/break-point-input/index.tsx
80.00% <80.00%> (ø)

...ents/break-point-input/number-area-input/index.tsx
80.00% <80.00%> (ø)

...l-szos-sdk/src/components/gesture-slider/index.tsx
97.29% <97.29%> (ø)

...l-szos-sdk/src/components/simple-top-bar/index.tsx
100.00% <100.00%> (ø)

...zos-sdk/src/components/steering-wheel/res/index.ts
100.00% <100.00%> (ø)

...panel-szos-sdk/src/components/turn-plate/index.tsx
70.83% <100.00%> (ø)

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 0050bff...2551336. Read the comment docs.


error when parse env

echo $allowips
<IP_ADDRESS> <IP_ADDRESS>

Caddyfile

...
 ipfilter / {
        rule allow
        ip {$allowips}  //not work
        ip <IP_ADDRESS> <IP_ADDRESS> //work
    }
...

thank you.

ipfilter doesn't support env vars.


style(util): fix typo re: imagePath

Have you read the section in CONTRIBUTING.md about pull requests?
Yes
Are your changes appropriately documented?
Yes
Do your changes have sufficient test coverage?
Yes
Does the testsuite pass successfully on your local machine?
Yes
Summarize your changes:
Fix a typo from iamgePath to imagePath

Didn't seem to be in any tests / affect anything downstream; at least according to test suite results + a cursory grep

Yeah, this property is never used anywhere else 😆 It's just there for some reason 👍

Yeah, this property is never used anywhere else 😆 It's just there for some reason 👍

Yeah, that's comforting 😰


Can't build new react-native project on android

Hi! I created new react native project and I can't build on android
React Native version:
System:
OS: macOS 10.15
CPU: (4) x64 Intel(R) Core(TM) i5-5350U CPU @ 1.80GHz
Memory: 60.53 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 10.2.1 - /usr/local/bin/node
Yarn: 1.16.0 - ~/.yarn/bin/yarn
npm: 6.4.1 - /usr/local/bin/npm
IDEs:
Android Studio: 3.5 AI-191.8<IP_ADDRESS>91312
Xcode: /undefined - /usr/bin/xcodebuild
npmPackages:
react: 16.9.0 => 16.9.0
react-native: 0.61.1 => 0.61.1
npmGlobalPackages:
react-native-cli: 2.0.1
When I try to run my project using react-native run-android it shows me this error:error Failed to install the app. Make sure you have the Android development environment set up: https://facebook.github.io/react-native/docs/getting-started.html#android-development-environment.
Also, when I run from Android Studio it shows this error:ERROR: Android dependency 'org.webkit:android-JSC:r245459' is set to compileOnly/provided which is not supported</code>
This is my package.json file:

{
"name": "churchmap",
"version": "0.0.1",
"private": true,
"scripts": {
"android": "react-native run-android",
"ios": "react-native run-ios",
"start": "react-native start",
"test": "jest"
},
"dependencies": {
"react": "16.9.0",
"react-native": "0.61.1"
},
"devDependencies": {
"@babel/core": "^7.5.0",
"@babel/runtime": "^7.5.0",
"@react-native-community/eslint-config": "^0.0.3",
"babel-jest": "^24.1.0",
"jest": "^24.1.0",
"metro-react-native-babel-preset": "^0.51.1",
"react-test-renderer": "16.8.1"
},
"jest": {
"preset": "react-native"
}
}

I encountered the same problem yesterday. What I did is that inside my android/build.gradle, I replaced  maven { url 'https://jitpack.io' } with  maven { url 'https://www.jitpack.io' }, then I downloaded jetify and ran npx jetify. That solved my problem :)

npx jetifier

It works thank you very much

I encountered the same problem yesterday. What I did is that inside my android/build.gradle, I replaced  maven { url 'https://jitpack.io' } with  maven { url 'https://www.jitpack.io' }, then I downloaded jetifier npm install --save-dev jetifier and ran npx jetifier. That solved my problem :)

For me it works just on change the link, 'https://jitpack.io' to 'https://www.jitpack.io' in android studio and make sync.
Thanks


Type spec generation

I took a look at whether it would be possible and I think it is. Macros don't get expanded inside @type for some reason so I didn't get this working:
require NimbleOptions.Type
@type opts :: NimbleOptions.Type.generate(schema)

But I think this could work:
require NimbleOptions.Type
NimbleOptions.Type.generate(opts, schema)

Is there a plan to add ability to generate a @type spec from the schema? Would that be a welcome addition in this project? I could publish a separate package instead

Currently no plans. Thanks for checking though!

Can I use nimble_options in the package name? Something like nimble_options_type

Sure!

For anyone interested, I have a WIP project, tests show what it generates. Overall seems fine, I am getting the usual fast-enough feedback cycle from dialyzer for catching option keys typos etc. Because NimbleOptions works on keyword lists (and not maps), Dialyzer cannot enforce requiredness per field.
I may add some escape hatch option to pass custom type definitions, it could also allow to support recursive options I think.


Add solution for 875 - Koko Eating Bananas in C

File(s) Modified: c/875-Koko-Eating-Bananas.c
Language(s) Used: C
Submission URL: https://leetcode.com/submissions/detail/792157405/

Thanks, @ritesh-dt!


hidesBottomBarWhenPushed does not hide when using ECSlidingViewController

I have a very simple example application (Tabbed Application) created.
To describe the Storyboard, we obviously have the TabBarController first, with two tabs "First" and "Second". Second points to "SecondViewController" that I have embedded in a Navigation Controller. I have added a UIBarButtonItem 'Add' which pushes a ViewController with the label "Without ECSlide" on it.
As for "First", the first tab relationship segue goes to an ECSlidingViewController. Disconnected from this we have a "MenuTableViewController" with custom sliding segues to "FirstNavigationController" and its relationship root view controller to our "FirstViewController". Similar to above, I have added a UIBarButtonItem 'Add' which pushes a ViewController with the label "With ECSlide" on it.
Both the "Without ECSlide" and "With ECSlide" ViewControllers have their "Hide Bottom Bar on Push" selected in the View Controller Properties of the Attributes Inspector for the VC.
Now, when we run, we tap the 'Second' tab. We tap the 'Add' button and the "Without ECSlide" view appears, and the TabBar at the bottom disappears. We go "< Back" and select the 'First' tab. We tap the 'Add' button and the "With ECSlide" view appears, however, the TabBar at the bottom is still showing.

I have the same question.how to solve.any body can help?Thank you.

Not a true solution, but may work in some circumstances.
Search for "UITabBarController+ShowHideBar.m", you need to use it's "setHidden" method. It's a simple category class. Find the code and use it.
If you are pushing A -> B, then in A's viewWillAppear you will want to do [self.tabBarController setHidden:NO]; //setHidden is from the +ShowHideBar category.
In B's viewWillAppear, you'll want something like this...
if (CGRectGetMaxY(self.tabBarController.tabBar.frame) - (CGRectGetMaxY(self.view.frame)) == CGRectGetHeight(self.tabBarController.tabBar.frame)) {
    [self.tabBarController setHidden:YES];
    self.constraintTableViewBottomToSuperViewBottom.constant -= CGRectGetHeight(self.tabBarController.tabBar.frame);
}


Desktop shortcut created despite checkbox not checked during installation

During the installation, I did not enable desktop shortcut creation, and it was created anyway. This has been happening for a few months now.

Fixed in 2.8.2. Thanks for reporting!

Just installed 2.8.2 (updated from the previous version) -- still got a desktop shortcut :-(

That’s weird. Could you try to do a clean install, and provide the exact steps/options you choose within the installer? Thanks!

I've recorded a GIF (under 20 seconds). Please let me know if there are any other details you'd find helpful.

I'm not able to reproduce this behaviour on my end. Could you try with the newest 2.8.3 installer, please?

It creates a shortcut, which disappears after some time, ha-ha. Happy with it as is, thank you!