Lead Time currently only supports GitHub Repos

Because we are using the GitHub API to grab the commit timestamps, we can currently only support source code on GitHub. Right now the collector code skips repos that don't have github.com in the URL, but I would like to find a more generic way to handle grabbing the commit timestamp.
This is more difficult than it sounds, as the only generic way seems to be to clone each repo. I'd like to find some less expensive operation than a full repo clone.
Refactoring to a generic method would also remove a manual step in the install process, where the user has to go generate an API token, which is likely to have scalability problems anyway.

For this issue, would a git clone --bare be ok?
It would not be a full repo clone but would give access to the git log which could (probably) be parsed to pull out committer id, timestamps etc. Still not as fast as just a straight up api call though

@etsauer @springdo Yep, git clone --bare should work - that's the approach taken on PAM for project repositories.


Don't save state in new format as it won't work during deployment

What is the context of this PR?
As there is a chance of requests going to a new version of the app
and then back to an old version during deployment, we can't persist
any format changes to state in the next release. This release should
however have all the code required to understand the new state
formats, which are:

Questionnaire store compressed via snappy, and base64
encoding and json data wrapper removed
Removal of base64 encoding for session data
View submission data in answer map format

So, to be clear, anything written (questionnaire state, session or
view submission data) in this version of the app should be readable
by the currently released version (v2.73.0).

👍

Code looks good, apart from the unforgivable import order :)
I'll do some testing upgrading / downgrading.


Fix for IE.

My website was not working on IE because the styles were missing -- and the IE javascript console said that it could not read "cssText" property. I dig down a little bit and found that this simple two-line patch makes it working perfectly on IE 11 as well as Chrome and Firefox. I haven't tested this on other versions of IE.

Ensure rule.propertyInfo is correctly set in style-properties.html.  It was checking a wrong attribute: rule.properties instead of rule.propertyInfo before calling decorateRule().
Fix a condition check in custom-style.html when rule.propertyInfo is null or undefined.

@sorvell PTAL

LGTM... thx!

Thanks! :D


Fix Issue #63

Replaced json.parse with json.stringify at line 329 of generate-report.js

Coverage remained the same at 98.54% when pulling 4c9c74c680908217357db26c386e922c842adf7a on CTMarius:master into 68ca0bfc976d393167acd6414bbbfa38a728a1a9 on wswebcreation:master.

Merged it, tnx!


Do I need to use mapbox_access_token ?

Hello. I have download your project and run sample, but all pages where maps used are blank.
There are some changes in maps access or in project that not updated yet?

You need to provide your mapbox access token in order to load correctly


Fix syntax issue introduced in #1188 NPE bugfix

While manually writing the fix in #1188 using a text editor, I misread the return type of the GetNormalizedCHICompleted function as returning a bool rather than a pointer to a CHI. That broke the build of 0.21.3 once #1188 was merged.
Now, this looks more similar to the original state of the code (just using getters instead of direct field access):
https://github.com/Altinity/clickhouse-operator/pull/1188/files#diff-6dd1bf383889aa0b3dde98cf3579859d84b90dd638c4829390596f9237e62f65L295
cc @sunsingerus
Important items to consider before making a Pull Request
Please check items PR complies to:

[x] All commits in the PR are squashed. More info
[x] The PR is made into dedicated next-release branch, not into master branch1. More info
[x] The PR is signed. More info

Force pushed to fix conflict (DiscoveryWatchedCHIs had some changes on 0.21.3 after my unfortunately syntactically-broken first PR)

The problem is already fixed, so this PR is not needed anymore


DX: Deploy Error when function modules are missing

When a function with package.json + deps in code hasn't installed the node_modules a someone cryptic error message is thrown (see below)
Reproduce:

Clone https://github.com/netlify-labs/all-the-functions
netlify init
npm install
netlify deploy
See error

Error:
🏃  Hashing files... ›   Warning:
 ›   {}
 ›
🚶  Hashing netlify.tomlError: Could not find "encoding" module in file: /12-fetch-external-api/12-fetch-external-api.js.
Please ensure "encoding" is installed in the project.
    at handle (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/finders.js:76:17)
    at precinct.paperwork.forEach.dependency (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/finders.js:102:11)
    at Array.forEach (<anonymous>)
    at getDependencies (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/finders.js:95:8)
    at filesForFunctionZip (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/zip.js:57:7)
    at zipJs (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/zip.js:110:3)
    at zipFunction (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/zip.js:143:12)
    at ~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/zip.js:181:9
    at element (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/p-all/index.js:4:67)
    at Promise.resolve.then.element (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/p-map/index.js:47:21)

Next error:
⠂ Hashing asset-manifest.jsonError: Could not find "faker" module in file: /6-using-node-modules/6-using-node-modules.js.
Please ensure "faker" is installed in the project.
    at handle (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/finders.js:76:17)
    at precinct.paperwork.forEach.dependency (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/finders.js:102:11)
    at Array.forEach (<anonymous>)
    at getDependencies (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/finders.js:95:8)
    at filesForFunctionZip (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/zip.js:57:7)
    at zipJs (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/zip.js:110:3)
    at zipFunction (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/zip.js:143:12)
    at ~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/@netlify/zip-it-and-ship-it/src/zip.js:181:9
    at element (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/p-all/index.js:4:67)
    at Promise.resolve.then.element (~/.nvm/versions/node/v10.4.1/lib/node_modules/netlify-cli/node_modules/p-map/index.js:47:21)

This is happening because the modules aren't resolved in the folder or up the tree
Possible solutions

Make the "module not found" error nicer with a hint to "make sure you install your dependancies in that folder"
If package.json found in functions folder with dependencies listed AND no node_modules found, automatically install the node_modules and automatically retry the command. This should alleviate the problem unless the code deps are not listed in package.json (user error)
Other ? ¯_(ツ)_/¯

Closing this as stale. I believe this was fixed in https://github.com/netlify/zip-it-and-ship-it/issues/30 and probably other improvements to zisi.
If this still happens please comment.


do not error task loading if task was unable to load the answer

Description
Currently, if a task is unable to load an answer (for instance, wrong format), the task loading fails.
New behavior: it will load the task and let the task explain that the answer was not loaded
Test cases

[ ] Case 1: BEFORE

Given I am the usual user
When I go to this task which always fails on answer loading
Then I see the platform error for loading

[ ] Case 2: AFTER

Given I am the usual user
When I go to this task which always fails on answer loading
Then I see the task loads

[auto-generated message] E2E test report on: https://algorea-static.s3.eu-central-1.amazonaws.com/branch/donotfailontaskanswerloadfailure/playwright-report/index.html


Add HTML to Markdown converter

Is your feature request related to a problem? Please describe.
As converting html to markdown directly would be a great feature and helps user to write markdown code easily
Describe the solution you'd like
Adding and HTML to MD convertor
Additional context

Are you still working on this? @root-0101

please assign me this issue


Use YAML to store configuration instead of environment variables

I think there is no particular reason to use environment variables for configuration. YAML is easy to parse with the yaml library and I see a number of reasons to switch to it:

It will be handier to store the entire configuration in hap.yaml file and the application could load it from that file automatically so source foo && part becomes unnecessary and it's easier to call hap.

We can represent collections/lists of items properly. This is actually necessary for #54, because we would like to allow to specify arbitrary files to copy, and it should be possible to copy several files/directories. How to put this into environment variables? We could use : or something as separator like it's done in PATH, but this:
deploy_path: '/var/projects/foo'
host: myserver.com
port: 2222
repo: 'https://github.com/stackbuilders/hapistrano.git'
revision: origin/master
restart_command: echo "foo!"
build_script:
  - stack setup
  - stack build
copy_verbatim:
  - file-a.txt
  - file-b.txt

is much better.

As shown in the example above, this allows to put the build script directly into the configuration file instead of keeping it outside. And when we specify path to build script, it's also error-prone in a sense, because it's usually a relative path and resolved path will depend on current working directory (when playing with hap locally I was bitten by this a couple times). YAML also handles # comments automatically (although it's not a problem anymore, but anyway).

Also may be used later for #8 as a way to specify multiple hosts.


Add sections on WASI integration tests

This commit adds relevant sections on contributing new WASI integration tests to wasmtime.

Looks good!
FWIW, I still am thinking about whether and how to include precompiled copies of the tests in the repo, to avoid this requirement, but until we decide to go there, it's obviously good to document what we actually have. Thanks!

Agreed!


Change how post types display on search page

The post types displayed in gold colored font need to be changed.

Any hyphens need to be changed into spaces
The word "Page" needs to be removed

I am thinking regex may be the best way to do this, but a .find would work too.
Either way the changes can be made in the search.js component.

Jesse, I am trying to solve this issue. Gonna send you a pull request soon.

#153


Added additional gaming clients/tools

Added additional gaming clients/tools

Additional notes:
Main website links for each client/tool:

Ubisoft: https://ubisoftconnect.com/
Vortex: https://www.nexusmods.com/about/vortex/
Loot: https://loot.github.io/
Itch: https://itch.io/app
Humble: https://www.humblebundle.com/membership/collection
CurseForge: https://www.curseforge.com/
Amazon: https://www.amazongames.com/en-us/support/prime-gaming/articles/download-and-install-the-amazon-games-app
Paradox: https://www.paradoxinteractive.com/our-games/launcher

i don't think loot should be included, not many people even in the Skyrim modding community uses it, it would just make the list longer

i don't think loot should be included, not many people even in the Skyrim modding community uses it, it would just make the list longer

Not to play devils advocate with your suggestion, but if that is the case, then Vortex should go as well, as not every user will be modding their games.

Understandable, but people who use loot most probably use vortex as well, or another mod manager, but not everyone that uses vortex uses loot, but if you believe having loot there can be helpful, by all means, thank for the hard work 👍

Understandable, but people who use loot most probably use vortex as well, or another mod manager, but not everyone that uses vortex uses loot, but if you believe having loot there can be helpful, by all means, thank for the hard work 👍

Your scenario lines up with my use case for Vortex, as I don't heavily rely on it due to the built-in functionality of LOOT that is within Vortex already when it sorts out load orders.
However, I do still use it every once and a while just so I can see any warnings/errors that Vortex doesn't catch or cannot display through the UI they have.

Missing applications.json file.


Rename configs folder to instructs

To be compatible with documentation.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.157%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1329

Relevant Lines:
1658

💛  - Coveralls


rtl-sdr: fix cross compilation

This patch partially fixes #81985.
Cross-compiling rtl-sdr is possible using this patch.
Use nix build -f  -A pkgsCross.raspberryPi.rtl-sdr to build rtl-sdr.
The error that occured when trying to do this before was as follows:
/nix/store/ldb2v2z0198abkp2minr9a9vz5np4mn4-cmake-3.16.4-armv6l-unknown-linux-gnueabihf/nix-support/setup-hook: line 107: cmake: command not found

Motivation for this change
I want to be able to cross-compile rtl_433 using NixOS. This is a first step towards that goal.
Things done

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

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

[ ] Seems N/A Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Not sure how this works Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)

note: this fails because it's cross-compiled, but the cross-compiled binary is of the correct type (ELF 32-bit LSB executable, ARM, EABI5 version 1 (SYSV), dynamically linked, interpreter /nix/store/b996ssrajnrjkni0fz3k7ihrp0m77ahr-glibc-2.30-armv6l-unknown-linux-gnueabihf/lib/ld-linux-armhf.so.3, for GNU/Linux 2.6.32, not stripped )

[ ] Expect low impact Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] N/A Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Should this be merged to master or to another branch? I can't tell what's expected here.

I've moved it, thanks for the quick review.

Thanks!


Drop rustc-serialize dependency

It has been deprecated: announcement.
Is there any functionality you would require from another library before this would be possible?

Thanks a lot for reporting.


Fix #263 Implementation of torque layer for OpenLayers for 3.17 or later.

@javisantana removed generated files from previous pull request.

@mbforr @javisantana Any reason this pull requests isnt completed?

@donflopez could you please review and test this PR?

@donflopez Thanks!!!


Add thread-local randomized key mixing strategy. [HPPC-151]

Issue: HPPC-151 (migrated from JIRA), created by Dawid Weiss (@dweiss), resolved May 04 2015

Comment by Dawid Weiss (@dweiss) (migrated from JIRA)
Let's postpone until really needed to avoid confusion.


'%l' can fail silently, updated percent_l_supported test

Here is a test showing that %l in strftime can fail silently:
# python
Python 2.7.10 (default, Aug 13 2015, 12:27:27)
[GCC 4.9.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from datetime import datetime
>>> datetime.now().strftime("%a %l%p")
''
>>>

The existing test does not catch this case. This patch fixes that.

Good catch, thanks!


Moving file from workspace to downloads deletes file

Open a folder containing HTML doc with sample data in VsCode.
Open up finder on Mac
Once finder is open go to downloads
Grab the open HTML on the top tab
click and drag the HTML file to downloads
The file will have 0 bytes

/gifPlease


Oscyank also yank the content of previous yank

I am yanking texts from vim in a tmux session in a remote machine (Terminal emulator is Kitty)
Suppose I have two lines:
This is good
This is also good

When I first oscyank This is good It works. However, when I oscyank This is also good the content I got is two lines together, This is good This is also good

Hello!
This is because Kitty implements its own slightly different version of the OSC52 protocol: see here. By default it appends any yanked lines to the previous content of the clipboard. To disable this behavior add no-append to the clipboard_control option of kitty: see doc here.
I thought I had taken care of this behavior in the plugin but it seems not, I'll take a look.

Hello!
This is because Kitty implements its own slightly different version of the OSC52 protocol: see here. By default it appends any yanked lines to the previous content of the clipboard. To disable this behavior add no-append to the clipboard_control option of kitty: see doc here.
I thought I had taken care of this behavior in the plugin but it seems not, I'll take a look.

I'm closing this issue. Feel free to re-open it if your problem persists.

I'm closing this issue. Feel free to re-open it if your problem persists.

how was this solved? I'm using kitty locally, and tmux on a remote machine. I noticed the note about kitty's OSC52 implementation.
When I set let g:oscyank_term = 'kitty', the note about how many bytes were copied appears, but it doesn't actually end up in the paste board (OSX). If I do not assign g:oscyank_term, then it appends.

Tmux is also a special case that must be handled on its own. I suspect that the combination of kitty + tmux is what causes the issue.
Try removing let g:oscyank_term from your Vim config and instead put clipboard_control no-append in your kitty config in ~/.config/kitty/kitty.conf.

Most excellent. I should have looked into that myself. This could nearly have been a RTFM response. Thanks for being so helpful! clibpoard_control no-append overrides the default, so the actual solution is for my case of using tmux on a remote machine and kitty locally:
vimrc: leave let g:oscyank_term to do it's default
kitty.conf: clipboard_control no-append write-clipboard write-primary


Update ng2-smart-table,module.ts

It looks like it needs to add 'schemas: [ CUSTOM_ELEMENTS_SCHEMA ]' to each component where it is using custom HTML tags to avoid template parse errors from angular 2.0.0 final. This will fix "Can't bind to 'cell' since it isn't a known property of 'select-editor'" error

@macoto35, thanks for you PR.

@LexZhukov I've updated my angular version to v.2.4.6 and the error message ("Can't bind to 'cell' since it isn't a known property of 'select-editor'") has been disappeared without add 'CUSTOM_ELEMENTS_SCHEMA'. Close this PR.


Article Email Signup A/B Test

Test between a full modal and a banner inserted above articles. Should be a 50/50 test for all users (that are not signed up to the newsletter) that land on an article.
The modal should not display on landing but should fade in when a user begins scrolling.
When a user closes either it should not display for 30 days.
@kanaabe How long do you think we will need to run this test for significant results.
@wrgoldstein Could you confirm that we don't have any other tests running atm?

Sketch file: https://cl.ly/3E451M141N1F

We should run the test for a min of 10 days

Note if you run the test for 10 days and we show the modal again after 30 days we won't know how that reappearance will affect users before making a choice.  Probably an acceptable limit though.
The only other test I know of is the forced inquiry test, which shouldn't conflict with this one.  So you're 👍


V2 Upgrade Instructions

I am working with a customer that desires to upgrade Icebreaker to V2.  Is there any documentation detailing the upgrade steps?

We at our org also are trying to upgrade Icebreaker to version 2. We added the new manifest package on Teams, but it does not work. On running the Logic app manually, the status shows succeeded within 200ms but does not seem to pair up people in the Team on which Icebreaker is added. Please help.

Thank you for submitting this issue, as we work to ensure consistent responses with items logged in our issues list, we are working to address these items that did not get responses. Please let us know if you are still facing this issue and require assistance and we will be happy to discuss. If no further action is needed at moment and no response is received, we will close out the issue in 3 business days. You may also always open a new issue or respond here if something does come back up. Thank you!

I am not the original poster -- but I would like to see better documentation for this.

Hi @jeff29922 ,
Thank you for submitting this issue, as we work to ensure consistent responses with items logged in our issues list, we are working to close the long pending issues with no resolution/responses. You may always open a new issue if you are still facing the issue. We are closing this request. Thank you!


Some Unit Test Problems

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:  latest vscode.dev or Version: 1.78.0 (Universal)

Version: 1.78.0 (Universal)
Commit: 252e5463d60e63238250799aef7375787f68b4ee
Date: 2023-05-03T20:11:00.813Z
Electron: 22.4.8
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 21.3.0
Sandboxed: Yes

OS Version: 12.2.1 (21D62)

Steps to Reproduce:

Clone https://github.com/microsoft/vscode
Use Cmd + shift + p to build vscode.

Case 1:

Use yarn test-browser --browser chromium --glob vs/**/**.test.ts to run unit test.
Then You will find that the unit test executes all the test files instead of the glob match specified.

Case 2:

Use yarn test-browser --browser chromium --glob vs/**/**.test.ts --debug to run unit test.
There will be a browser flash, but I found that the loader.js is loaded repeatedly in renderer.html,  Ideally, what version of the file to load should be determined according to the parameters, not all of them, otherwise the browser will report an error.

Case 3:

Use yarn test-browser --browser chromium --glob src/vs/**/**.test.ts --debug to run unit test.
An error has occurred:  zsh: no matches found:  vs/**/**.test.ts. Because the path prefix and suffix are not processed in glob mode.

Fixed 1/2 in https://github.com/microsoft/vscode/pull/199979
3 is your shell trying to expand the glob pattern -- quote the pattern to avoid this


glb: bufferView can be undefined

Right now if bufferView of accessor is undefined the engine cannot parse glb.
As you can see here bufferview is not required. PR fixes it.
Project example: https://playcanvas.com/editor/project/1041648
The custom build of the engine:

I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

https://playcanvas.com/editor/project/1041648

You can find it in the project: https://playcanvas.com/editor/project/1041648

If you exclude playcanvas.dbg.js and set Loading time  == Asset, then


How is the data length distribution computed for LongBench-E?

Thanks for sharing your code and paper!
I'm trying to understand the distribution of several datasets in LongBench-E, as the numbers I get are not aligned with the description in task.md and Table 7 in the paper.
from datasets import load_dataset

subsets = ['qasper', 'hotpotqa', '2wikimqa', 'gov_report']

for subset in subsets:
    data = load_dataset('THUDM/LongBench', f"{subset}_e", split='test')
    context_lengths = [len(row['context']) for row in data]

    print(f'Dataset: {subset}_e')
    print(f'Number of Samples: {len(context_lengths)}')
    print(f'Number of Samples with Context Length < 4000: {len([x for x in context_lengths if x < 4000])}')
    print(f'Number of Samples with Context Length 4000-8000: {len([x for x in context_lengths if x >= 4000 and x < 8000])}')
    print(f'Number of Samples with Context Length 8000+: {len([x for x in context_lengths if x >= 8000])}')

Expected output:
qasper_e should have about 100, 100, 24 etc according to the table.
Output:
Dataset: qasper_e
Number of Samples: 224
Number of Samples with Context Length < 4000: 0
Number of Samples with Context Length 4000-8000: 0
Number of Samples with Context Length 8000+: 224
Dataset: hotpotqa_e
Number of Samples: 300
Number of Samples with Context Length < 4000: 2
Number of Samples with Context Length 4000-8000: 2
Number of Samples with Context Length 8000+: 296
Dataset: 2wikimqa_e
Number of Samples: 300
Number of Samples with Context Length < 4000: 0
Number of Samples with Context Length 4000-8000: 2
Number of Samples with Context Length 8000+: 298
Dataset: gov_report_e
Number of Samples: 300
Number of Samples with Context Length < 4000: 0
Number of Samples with Context Length 4000-8000: 1
Number of Samples with Context Length 8000+: 299

After adding prompt and tokenizing, I would imagine the context lengths be even longer, which is not aligned with having 100 samples with context length < 4000 as said in the paper.

Thanks for your attention! We count the number of words for English datasets, and number of characters for Chinese datasets. Make sure to change the code to:
context_lengths = [len(row['context'].split()+row['answers'][0].split()) for row in data] # for English datasets
context_lengths = [len(row['context']+row['answers'][0]) for row in data] # for Chinese datasets

Thank you so much! I couldn't believe I didn't find this bug in my code haha


R.3 - Chat view window UI improvements

The window that shows a chat history between one user and another needs UI improvements:

[x] It should be shown as a scrolling window where the most recent message is at the bottom
[x] The chat messages should be shown in reverse chronological order
[x] Each chat should be displayed as a bubble
[x] The logged in user's chat bubbles should be aligned right
[x] The other user's chat bubbles should be aligned left
[x] The username of the user you're chatting with should be shown prominently at the top of the window
[x] Bubbles should not display the username
[x] Bubbles should have a visible timestamp
[x] A text input bar shall be at the bottom of the window that let's users type a new message and send it

Closed by #59


Array :: Write Code in Python For one of  following tasks

Maximum and minimum of an array.
Find Kth smallest element of the array.
Move all negative numbers to beginning and positive to end with constant extra space.
Sort the given array
Smallest Positive missing number of given array.

assign me this

sure you can do this

@TechVine assign this to me. i want to work on it

@TechVine assign this to me. i want to work on it

sure you can work

Assign me any of the problem left and i can contribute.. can you do this please?

Assign me any of the problem left and i can contribute.. can you do this please?

you can do any task you want to do , a single task can be done by many users just make sure you do not use copied code.

@TechVine please assign this to me

sure you can work on this

hello sir, please assing me sorting of array.

hello sir, please assing me sorting of array.

ok you can do it , plz don"t forget to star mark this repository

@TechVine bhaiya please assign me..I can do it

@TechVine bhaiya please assign me..I can do it

sure

can u assign this to me ? want to contribute as i want to do all ur task in java

Sure you can , make sure you star mark this repo.

@TechVine bhaiya I've added the code for sorting the array..please merge it

#44

can u assign this to me?

Can you assign this issue?

I want to contribute to it.Please assign it to me.

Can you assign this issue to me??

Can you assign this issue ???

can you assign me to contribute?

@Zetahaxx    @Umang3072 @Kuntalpramanik @Satorusimpp @sabhisharma-ise  assigned........

Please assign this to me!!

Please assign this to me!!

sure......

Sir please assign issue to me....bhaiya please

Can you please assign this issue to me?

Can you please assign this issue to me ?

Can you please assign this issue to me ?

Please assign this to me!!

Hey @TechVine can you assign Maximum and minimum of an array to me?

can you assign me to contribute?

Hello sir, can you assign any two tasks to me for contributation?

Hey @TechVine  can you assign me this issue

Hai @TechVine, can you please assign this to me

@TechVine  can you assign this to me?

Can u assign me this?

Can u assign me this?

May I contribute to this issue?

Can you please assign this issue to me?

Hey bro! please assign this task to me. I can do all @TechVine


BUG: VSCode WebView scroll is much faster compared to the browser

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.88.1
OS Version: Windows 11

Steps to Reproduce:

Create a Webview with a scrollable List/Content
Scroll and compare to browser

You will notice its not possible to scroll slowly in the vscode webview compared to a browser. I already tried the scroll sensitivity setting.
I already tried to remove all vscode related css (default styles/from parent), but the behavior didn't change.
Example:
Scrolling in Browser (expected)
Expected behavior: slow scrolling is possible

Scrolling in VSCode

@mjbvz could you reproduce it?


How Debug android project?

Dear developers AvaloniaUI, is it possible to debug android project using launch.json?

Hello?

It looks like Microsoft DevKit plugin for VS Code doesn't support mobile projects yet.
You might need to create an issue there: https://github.com/microsoft/vscode-dotnettools/issues
Just to be clear, Avalonia doesn't use any special SDK or tooling for mobile, instead, it's based on standard .NET solutions. If standard .NET on VSCode doesn't allow you to run and debug projects, then Avalonia won't be able as well.

Actually, they already have an issue opened for this: https://github.com/microsoft/vscode-dotnettools/issues/470


Function missing return statement

Hi,
First, thanks for the lib! It has helped me immensely get the DAC working with zero effort :)
However, there is just one issue reported by the compiler, which would be nice if it were fixed:
MCP4728.h: In member function 'uint8_t MCP4728::analogWrite(MCP4728::DAC_CH, uint16_t, bool)':
MCP4728.h:44:5: warning: no return statement in function returning non-void [-Wreturn-type]
The warning is pretty self-explanatory, literally just the return keyword is missing in the method implementation: https://github.com/hideakitai/MCP4728/blob/master/MCP4728.h#L43
Thanks,
Doug.

Hi, thank you for using and reporting issue!
I've fixed it in b2462e0e867b733ec51de99e3ff040c22e86780d, so check it out and let me know if you have another problem :) Thanks!


Fix for Documentation end point list empty #177

needs to be rebased maybe?

Hi Andrew, trying to clean up old PR. What was the issue your where trying to fix with this change. It a little unclear from the title and code change. Does it exist?

Hi Glenn
It has been a while since I suggested this.  If I recall one of my route definitions was  expecting an array and for some reason the generated swagger json was not validating on the swagger site. So I made the suggested change.
If it makes no sense ignore the pr and if it pops up again as an issue I will let you know
Andrew

On 6 Feb 2016, at 12:40 AM, Glenn Jones<EMAIL_ADDRESS>wrote:
Hi Andrew, trying to clean up old PR. What was the issue your where trying to fix with this change. It a little unclear from the title and code change. Does it exist?
—
Reply to this email directly or view it on GitHub.


Switch to libbpf_1 dependency in nix

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
We now have an official libbpf version in upstream nix after the merge of: https://github.com/NixOS/nixpkgs/pull/193591
Which issue(s) this PR fixes:

None
Does this PR have test?
None

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

Updated libbpf to v1.0.1.

Codecov Report

Merging #1205 (f7b6b65) into main (87eab44) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head f7b6b65 differs from pull request most recent head a6e3a7d. Consider uploading reports for the commit a6e3a7d to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1205   +/-   ##
=======================================
  Coverage   47.49%   47.49%
=======================================
  Files          44       44
  Lines        4925     4925
=======================================
  Hits         2339     2339
  Misses       2487     2487
  Partials       99       99


apache age

@sherwan777 Thank you for your PR. However, your PR is missing the following required items and therefore cannot be processed and will be closed.

A Github issue that the PR is based off of. This is generally required for all PRs and should exist before the PR is created.
A valid branch to apply the PR to. Release branches are not valid targets. A Github issue will usually have this information.
A valid title, briefly explaining the issue that the PR resolves. This generally will point to the Github issue.
A description (and extended description), fully explaining the PR.
A PR without any conflicts.
A PR that only modifies the necessary files. Generally speaking, modifying more than a few files is frowned upon.

If you need assistance, please create a Github issue/question or provide all of the required information above.


Add an info pop-up after successful submission of transaction

As a visitor I want to see an info pop-up after I have successfully submitted a transaction.

After successfully submitted a transaction we put this one into the transaction list with pending status... so do we need duplicate this info with a pop-up?

@mateleshkavo @a5kold Do we really need this pop-up?

After you submit a transaction transaction is added to the list, however it's not visible if you don't scroll down. So it feels like nothing happened. For consistent UX I would suggest to add some pop-up or at least info message that transaction was sent.

Please, no popup. A message is ok. A thing that will cover entire screen — no no.
We need Alice also for ourselves and we need to send a lot of transactions sometimes. Popup will add unnecessary action during the test

@mateleshkavo Transaction is added to the top of the list, as the first row

As an option. We can scroll the page to place at the bottom of the window. So we'll have form and transactions list in the viewport

@maxkudla please open the page, don't scroll to the bottom, send the tx -> you don't see any transactions or any movement (or at least I don't see on my screen resolution). @sunify agree, message will be fine

@a5kold Could u provide design pls?

Can we add the same "success message" as when you copy the address? Above the send button.

@a5kold yes, but it'll be gr8 to show tx info (from, to, value)... in the message. What do u think?


哈喽，经常出现执行到连接到action地方时，没有连接日志信息

同问！

同问啊

The Actions Log seems to load a bit late for other action scripts too, not just this one.
So, it's a normal internal problem that we all have to face.


chore: automated PR to update VHDVersion for 202312.06.0 VHD

This is an automated PR to bump VHDVersion for the VHD release with image version 202312.06.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.095%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2255

Relevant Lines:
2851

💛  - Coveralls


Persistence: Test Restructuring and Bug Fixes for Bitemporal Delta

What type of PR is this?
Test Restructuring and Bug Fix
What does this PR do / why is it needed ?

Test Restructuring for Bitemporal Delta
Fix the bug about automatic case conversion for aliases

Which issue(s) this PR fixes:

Add validation while loading test data in H2
Define test scenarios and test cases for Bitemporal Delta
Add Bitemporal Delta tests in Memsql and Snowflake Modules
Fix the alias converting to upper case issue for Bitemporal Delta
Support upper and lower case optimisation for alias

Other notes for reviewers:
Does this PR introduce a user-facing change?
NO

Besides that duplication I highlighted on the prev comment, the rest looks good.

This UpperCaseOptimizer and LowerCaseOptimizer seems to be duplicated code with the different of the operation on the string.
Can we create an abstract class StringCaseOptimizer, where it receive in the constructor the strategy to apply, and then Upper and Lower just set the right strategy.
public StringCaseOptimizer(Funtion<String, String> strategy){
  this.strategy = strategy;
}

Then, operations will look like this:
field.setAlias(this.strategy.apply(fielld.getAlias()));
The upper and lower class will only need a constructor:
public UpperCaseOtmizer()
{
   super(String::toUpperCase);
}

This UpperCaseOptimizer and LowerCaseOptimizer seems to be duplicated code with the different of the operation on the string.
Can we create an abstract class StringCaseOptimizer, where it receive in the constructor the strategy to apply, and then Upper and Lower just set the right strategy.
public StringCaseOptimizer(Funtion<String, String> strategy){
  this.strategy = strategy;
}

Then, operations will look like this:
field.setAlias(this.strategy.apply(fielld.getAlias()));
The upper and lower class will only need a constructor:
public UpperCaseOtmizer()
{
   super(String::toUpperCase);
}

Thanks's Rafael.
That's a great suggestion.
Made the code changes using the suggested approach.
Also added applyCase method to reduce some more duplication.


Cannot delete Auth Resource

I am receiving this error when trying to delete the SST Auth resource:
Received response status [FAILED] from custom resource. Message returned: User: arn:aws:sts::008193302444:assumed-role/...-Auth-CustomResourceHandlerServ-LC2ZFlSsHCBx/... is not authorized to perform: ssm:DeleteParameters on resource: arn:aws:ssm:us-east-1:008193302444:parameter/sst/.../ip-dev/Auth/auth/publicKey because no identity-based policy allows the ssm:DeleteParameters action
I have to manually go into cloud formation and delete the resource.

If you've already posted your issue on Discord, make sure to leave a link to it here.

Are you using the latest version of sst? I believe this has been fixed in #3417

npx sst update? I am still having the issue, even after updating the sst package.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Threading layer: Race condition in the layer causes incorrect "object simultaneously used in thread .. and thread .." errors to be reported

Consider the following scenario:

Thread A calls vkDestroyBuffer()
Thread B calls vkCreateBuffer()

It is legitimate for the driver to return the same handle if the calls are executed in a serialized manner, in order: A followed by B.
The validation layer chain does some additional chores after VK impl returns execution flow from vkDestroyBuffer(), before passing control back to the app. However, this additional housekeeping work does not seem to be protected by any sort of a critical section. This can lead to a situation where the threading layer notices the same handle is being used in two threads at the same time, even though that very handle has already had ownership released from the original object and passed over to a new object.

I have a repro for this if that would be helpful.

Forgot to mention: if the error is ignored, the layer first does an assert(0) and then crashes afterward due to a NPE.

How are you stacking up the layers for this? CV etc relies on unique_objects to guarantee that we don't see reuse of nondispatchable handles.

Fair point. IIRC, I used alphabetic ordering when typing all the custom paths to json files in the registry so that would mean unique_objects is loaded before core_validation. I'll get back to you on this tomorrow, I'm OoO already.
As a side point, are you considering adding some sort of a requirements chain to the layers? Issues like these are pretty hardcore to nail down if you're not knee-deep into VL project's guts :-/ A warning along the lines of "layer X should come before layer Y or else!" would suffice just fine.

You should already be getting such a warning, at least for CV misordered wrt UO.
The standard_validation pseudo-layer enables the layers in the order that is known to work.


Java 5.4: Assignment and ternary Operator #1806

Added google drive video link.

Description 📜
Added video link on the topic Assignment and Ternary Operator in Java.
Issue #1806

Type of change 📝

[x] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilities etc)

Domain of Contribution 📊

[x] Java_Domain

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] My changes generates no new warnings.
[x] I'm a GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@AkMo3  Please review the content.

@garvitraj


"Open In Terminal" in Source Control opens a separate terminal instance outside VS Code in OSX

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:  1.78.0 (Universal)
OS Version: OSX 13.3.1 (a) (22E772610a)
Hardware: Apple M1 Pro (32GB)

Opening a terminal for a given repository in the Source Control panel no longer opens said terminal in the integrated terminal panel but in a new external terminal window. This issue started to occur for unknown reasons a day prior to this issue being opened. I have a colleague for whom this issue has also started occurring, and neither of us know what has happened to trigger this unexpected behaviour.
Steps to Reproduce:

Open VS in a folder with multiple git repositories in the directory structure.
In the Source Control panel, right-click a repository and select Open In Terminal.

Expected Result:
A new terminal tab is opened in the integrated terminal panel in VS Code.
Actual Result:
An external terminal instance outside of VS Code is opened.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.78.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Apparently I am on the latest version, as downloaded from the MS site:
Version: 1.78.0 (Universal)
Commit: 252e5463d60e63238250799aef7375787f68b4ee
Date: 2023-05-03T20:11:00.813Z
Electron: 22.4.8
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.4.0
Sandboxed: Yes

/duplicate of #182602 for which the fix is already merged.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


User.archive does not remove entries in the identifiers table

The User.archive method was introduced several years ago and has not kept up with the recent changes. It should be updated to clear out any entries in the identifiers table (Shibboleth user ids and ORCIDs) that are related to the user.
A script would then need to be run to purge these records for any user accounts that have already been archived.

Also see issue #2680


Enable raven JS, log upload json error

This enabled ravenJS and manually logs the error that causes "There was a problem contacting the server."
This caused a CSP error locally:
Content Security Policy: The page's settings observed the loading of a resource at https://sentry.prod.mozaws.net/api/75/store/?sentry_version=4&sentry_client=raven-js/1.1.15&sentry_key=5686e2a8f14446a3940c651c6a14dc73&sentry_data=%7B%22project%22%3A%2275%22%2C%22logger%22%3A%22javascript%22%2C%22platform%22%3A%22javascript%22%2C%22request%22%3A%7B%22url%22%3A%22http%3A%2F%2Folympia.dev%2Fen-US%2Ffirefox%2F%22%2C%22headers%22%3A%7B%22User-Agent%22%3A%22Mozilla%2F5.0%20(Macintosh%3B%20Intel%20Mac%20OS%20X%2010.11%3B%20rv%3A45.0)%20Gecko%2F20100101%20Firefox%2F45.0%22%7D%7D%2C%22message%22%3A%22Test%20message%22%2C%22event_id%22%3A%229182176ae8e74e85bbfa1f6be8a2de52%22%7D ("img-src http://olympia.dev https://ssl.google-analytics.com https://addons.cdn.mozilla.net http://www.google-analytics.com"). A CSP report is being sent.

Can you list the hosts that raven needs to send to? Presumably it uses XHR? In which case adding the hosts to CSP_CONNECT_SRC  [1] should fix it.
[1] https://github.com/mozilla/olympia/blob/master/lib/settings_base.py#L1256

Looks like uglify is choking on something.

This is why you shouldn't use cat to combine your assets. raven.min.js didn't have a newline at the end and it ended with a comment so it was commenting out the first line of raven-config.js.


Error on freebsd with $PWD != $(realpath $PWD)

On FreeBSD /home is often a symlink to /usr/home. This seems to not agree with gb for some reason, and I have to specify the -R path directly.
$ pwd
/home/user/projects/rollcage

$ realpath $(pwd)
/usr/home/user/projects/rollcage

$ gb build -v
DEBUG project root "/usr/home/user/projects/rollcage"
DEBUG matchPackages: /usr/home/user/projects/rollcage/src ../../../../../../home/user/projects/rollcage/...
DEBUG args: []
DEBUG removing work directory: /tmp/gb148345295
FATAL command "build" failed: no packages supplied

$ gb build -R $(realpath $(pwd)) -v
DEBUG project root "/usr/home/user/projects/rollcage"
DEBUG matchPackages: /usr/home/user/projects/rollcage/src ...
DEBUG args: [rollcage rollcage/commands]
DEBUG build duration: 15.232µs map[]
DEBUG removing work directory: /tmp/gb435798870

Using gb fetched via go get today (Oct  6).

Thanks for the report. I'll look into it soon.
On Wed, 7 Oct 2015 09:11 eli<EMAIL_ADDRESS>wrote:

On FreeBSD /home is often a symlink to /usr/home. This seems to not agree
with gb for some reason, and I have to specify the -R path directly.
$ pwd
/home/user/projects/rollcage
$ realpath $(pwd)
/usr/home/user/projects/rollcage
$ gb build -v
DEBUG project root "/usr/home/user/projects/rollcage"
DEBUG matchPackages: /usr/home/user/projects/rollcage/src ../../../../../../home/user/projects/rollcage/...
DEBUG args: []
DEBUG removing work directory: /tmp/gb148345295
FATAL command "build" failed: no packages supplied
$ gb build -R $(realpath $(pwd)) -v
DEBUG project root "/usr/home/user/projects/rollcage"
DEBUG matchPackages: /usr/home/user/projects/rollcage/src ...
DEBUG args: [rollcage rollcage/commands]
DEBUG build duration: 15.232µs map[]
DEBUG removing work directory: /tmp/gb435798870
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/422.

@davecheney I think I found the root cause.
In cmd/main.go
	cwd, err := filepath.Abs(cwd) // if cwd was passed in via -R, make sure it is absolute

It appears that filepath.Abs does not resolve symlinks on its own.
The following gets it to work -- but I admit to not knowing how portable it would end up being.
diff --git a/cmd/gb/main.go b/cmd/gb/main.go
index 4a5a311..469311f 100644
--- a/cmd/gb/main.go
+++ b/cmd/gb/main.go
@@ -110,6 +110,10 @@ func main() {
 	if err != nil {
 		log.Fatalf("could not make project root absolute: %v", err)
 	}
+	cwd, err = filepath.EvalSymlinks(cwd)
+	if err != nil {
+		log.Fatalf("could not make project root absolute: %v", err)
+	}

 	ctx, err := cmd.NewContext(
 		cwd, // project root

Thanks. I agree with your diagnosis. That check was added a while back. The
details escape me, but I believe it was also symlinks related, before
changing that path, I'd like to review the reason that line was added.
On Wed, 7 Oct 2015 12:56 eli<EMAIL_ADDRESS>wrote:

@davecheney https://github.com/davecheney I think I found the root
cause.
In cmd/main.go
cwd, err := filepath.Abs(cwd) // if cwd was passed in via -R, make sure it is absolute

It appears that filepath.Abs does not resolve symlinks on its own.
The following gets it to work -- but I admit to not knowing how portable
it would end up being.
diff --git a/cmd/gb/main.go b/cmd/gb/main.go
index 4a5a311..469311f 100644
--- a/cmd/gb/main.go
+++ b/cmd/gb/main.go
@@ -110,6 +110,10 @@ func main() {
if err != nil {
log.Fatalf("could not make project root absolute: %v", err)
}

cwd, err = filepath.EvalSymlinks(cwd)

if err != nil {

  log.Fatalf("could not make project root absolute: %v", err)

}
ctx, err := cmd.NewContext(
cwd, // project root

—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/422#issuecomment-146056216.

I updated the diff to show a bit more context, and to make it clear that resolving symlinks would be additive, not intended to replace the existing absolutification (that isn't a real word is it?!) of the path.

Forgot to mention that I am in no hurry, as I do have a valid workaround (via -R).
Thanks!

@davecheney any further thoughts on this?

Awesome. Thanks @davecheney.
Just tested on my FreeBSD box and it indeed appears to be fixed!

Thanks for confirming.

On 15 Dec 2015, at 10:54, eli<EMAIL_ADDRESS>wrote:
Awesome. Thanks @davecheney.
Just tested on my FreeBSD box and it indeed appears to be fixed!
—
Reply to this email directly or view it on GitHub.


duplicate entry: public/main.css

I see this message from the front-end module when doing docker:publishLocal on master

hmmm, no indication of anything within the file - it just tells me it's public/main.css and i can't find another file with that name so assumed it was that one.

Interesting, clean didn't help but manually deleting the /target directories and rebuilding DID fix it - thanks for the suggestion


sad: init at 0.4.14

Motivation for this change
CLI tool to search and replace
https://github.com/ms-jpq/sad
Fixes #136604
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

1 package built:

sad

Builds & runs, macos 10.15 & nixos x86_64


Schema Validator counter inconsistent

http://bpmn-miwg.github.io/bpmn-miwg-tools/
In the result table, Software AG has 10/10, Trisotech has 12/12... we are not certain why Software AG are missing two results.
Also, C.5 and C.6 seems to not be checked.

@tstephen could you have a look into this? It seems like not all test results are passed into the schema validation.

Software AG has C.1.0 and C.1.1 submitted but for some reason they don't get picked up, neither by the Schema Validation Tool nor any of the other XML-based tools.
C.5 and C.6 are not showing up for any of the submitting tools.

Still not showing: http://bpmn-miwg.github.io/bpmn-miwg-tools/overview.html#ARIS Architect 10.0.10 =(

@tstephen any update on this?

Has the site already been rebuilt? It is still not showing up on the site.

It will rebuild during the night. Per our conversation on Friday, this 'fix' is speculative rather than proven.

Okay, thanks for trying. Let's leave the issue open until we see the results tomorrow.

Still not showing. =(

I have fixed the generation of submissions.json but that was https://github.com/bpmn-miwg/bpmn-miwg-test-suite/issues/885 rather than this one.
The reason that the max schema validations varies for different tools lies in the logic of this piece of JavaScript: https://github.com/bpmn-miwg/bpmn-miwg-tools/blob/c6c5e9ec56b0ab6d20887d1b97daf3d703d8738f/index.html#L65 Basically it is calculating the max per tool so it is impossible to get a consistent max across all tools. What is coded here should be regarded as a boolean: If count == max then all submissions are schema valid. I'll look at converting it to a tick

@tstephen thanks for the fix, maybe not related but the image on the miwg page that links to the table seems to be broken:
https://github.com/bpmn-miwg/bpmn-miwg-test-suite under Test results of BPMN 2.0 tools that participated in the tests

Clarification at meeting 05 Jan 23: It is by design that the Schema Validation column shows the number of files for the vendor (not the reference) as denominator. The issue is that some (eg Aris) appear to have complete submissions judging by other columns but fewer than the maximum 14 in the schema validation column. Hence something to be investigated

Investigated the errors with findings as follows (click the Schema Valid link in the table to see for yourself):

ADONIS: B.1.0: roundtrip, B.2.0: roundtrip, C.1.0: roundtrip and C.6.0: roundtrip all correctly flagged as failing schema validation
ARIS is missing C.1.0: roundtrip and C.1.1: roundtrip submissions hence 12/12 not 14/14. The files are present but the encoding of the umlaut on line 433 and 131 respectively makes them unparseable using the validating parser. xmllint explains:$ xmllint --schema ~/git/bpmn-miwg-tools-gh-pages/./schema-tool/src/main/resources/schema/BPMN20.xsd ARIS\ 10.0.20/C.1.0-roundtrip.bpmn --noout
ARIS 10.0.20/C.1.0-roundtrip.bpmn:433: parser error : Input is not proper UTF-8, indicate encoding !
Bytes: 0xE4 0x72 0x65 0x6E
   <semantic:userTask xmlns:camunda="http://activiti.org/bpmn" name="Rechnung kl

Signavio has 2 schema invalid files: B.1.0: roundtrip and B.2.0: roundtrip though for some reason B.1.0 reports it to be ok. Again xmllint explains:$ xmllint --schema ~/git/bpmn-miwg-tools-gh-pages/./schema-tool/src/main/resources/schema/BPMN20.xsd Signavio\ Process\ Editor\ 10.0.0/B.1.0-roundtrip.bpmn --noout
Signavio%20Process%20Editor%2010.0.0/B.1.0-roundtrip.bpmn:573: element process: Schemas validity error : Element '{http://www.omg.org/spec/BPMN/20100524/MODEL}process', attribute 'id': 'Process_ba16239e-181e-4b9f-bc5b-0bb2ee973450' is not a valid value of the atomic type 'xs:ID'.
Signavio Process Editor 10.0.0/B.1.0-roundtrip.bpmn fails to validate

Models after C.3.0 are missing which explains the lower number
Enterprise explorer produced invalid B.1.0: roundtrip and B.2.0: roundtrip too and additionally C.1.1: roundtrip

After that I stopped content that the system is in fact behaving.

With latest test result of ARIS the encoding issue is fixed, the schema validation counter is as expected (14/14).


[git_pull] Multi-line deployment keys are not allowed

I cannot save my git_pull-configuration because a multi-line deployment key is not allowed. The text turns red and the "SAVE" button will be disabled.

@janpieper put each line of your key in quotes and separate each line with a comma. E.g.
"--------BEGIN RSA PRIVATE KEY ----------",
"why-is-this-not-allowed?",
"actually-it-is!",
"--------END RSA PRIVATE KEY------------"

Yep, @TravelinMax is correct.
👍


GIF animation for .ZIP drop

As the .zip drag-and-drop-and-expand thing is quite magnificent to look at, let's also have an animated GIF telling the story.
Should go here https://github.com/projectkudu/kudu/wiki/Kudu-console
Under this section: Upload and expand zip file
GIF here:

No image attribution necessary. I'm happy to just contribute to the Kudu project.

Done, thanks for that!


The feed animation is broken

Pre-condition: User is logged in wallet
Steps to reproduce:

Open the Dashboard
Refresh the page
Pay attention how feed animation works. Also, pay attention on "Claim" button animation

Actual result: The feed animation is broken. The real user's feed cards aren't displayed after animations. There is no animation for "Claim" button
Expected result: The feed animation displays correctly - there are user's feed cards dispayes after. The "Claim" button has animation
Environment QA (current ver0.14.1-2)
Mnemonic: cheese barrel brave seed auction seek language order cruel direct monster grit
Devices used:
Xiaomi Redmi 7 (Android 9.0) - reproduced 5/5
Samsung Galaxy S6 (Android 7.0) - reproduced 5/5
iPhone 7 (iOS 13.2 beta) - reproduced 2/5
Note: The issue reproduced only on mobile
Video: https://www.dropbox.com/s/pizl76wb5psg8sr/feed anim.mp4?dl=0

@AnastasiiaOdnoshevna it is not the issue with animations. It is something wrong with your account that feeds are not loading at all. I am going to investigate what is the problem.

@AnastasiiaOdnoshevna please check it now.

Check it on the Dev environment.
The issue isn't reproduce.
The scrolling is displayed correctly


feature: Data Model management not allowed to clilck "Next" if information not provided

Feature Request
Describe the Feature Request

1.- User should not be allowed to click on Next in step 2 if he didn't select a dataset.
2.-  User should not be allowed to click on "Next" if he didn't select at least one algorithm or provide training size&test size
Is your feature request related to a problem? Please describe

Describe Preferred Solution

Additional Context

To block the navigation by header, we must to modify the style sheet:
::ng-deep .mat-horizontal-stepper-header{
  pointer-events: none !important;
}


Block Grid Demo Page unequal blocks

Advanced section on the link.
http://foundation.zurb.com/docs/components/block_grid.html
I don't know if this intended but it was working ok on my works.
Here screenshot

Tested on
Windows 7 / Chrome / Firefox / Opera

This seems to be working as intended as the images you used are of different sizes. Howe did you want it to look?

So, according to image size. I checked sizes and understood. Thanks.


Unhide code comments in generated HTML

I disagree with the rationale for #38, and the effects of the associated code changes.
Sometimes I put in code comments that are very specific to a particular line; sometimes I adopt code that was not originally in literate.  In both cases it would be useful to me to see the comments in the preview; it was surprising when they weren't there.
Perhaps an intermediate solution would be to add an option that controlled whether or not code comments are tossed for HTML.
If you decide to retain the feature anyway, it might be better to eat the comments completely.  The whitespace associated with them remains in the HTML output, which is a little weird.  I haven't been able to trace exactly how that happens.  Perhaps the data-line value in, e.g., <code data-line="233"... is actually controlling the vertical position, and is set before the comment is removed.

An option would be useful here indeed.


write coverage report to file

Is your feature request related to a problem? Please describe.
It would be great if Specmatic could produce a dedicated report file with API coverage report - so that we could surface it as a build artifact in CI.
Describe the solution you'd like
Add a new report formatter  and corresponding config options to enable it, for example:
"report": {
    "formatters": [
      {
        "type": "text",
        "layout": "table"
      },
    {
        "type": "file",
        "layout": "table",
       "name" : "my_coverage_report.txt"
      }
    ],

Describe alternatives you've considered
We could extract the report section out of the test stdout, but its cumbersome.

@yntelectual Thanks again. This is already on our backlog. Will keep you posted.

Hi @yntelectual, Specmatic publishes the API Coverage Report to a file (build/reports/specmatic/coverage_report.json). We have documented this here.
Closing this for now. But please reopen it if you have followup questions.


AFHTTPSessionManager Timeout Issue

I want to reduce the request timeout be default it is taking 60 seconds for timout.
I have used:
manager.requestSerializer.timeoutInterval = 10.0f;
For request timeout but it is not working.
Give me an idea to make the request timeout.
Please Help me out to solve this.

From our Contribution Guidelines

Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.

Waiting for you response.

@abhibansal20 you won't get one here, jump over to Stack Overflow and ask the same question. This is all in the answer Matt already gave you.


popover position issues

I'm submitting a...

[ ] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Support request => Please do not submit support request here

Current behavior

popover position issues
Expected behavior

popover should follow button when scroll
Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

Environment

Angular version: X.Y.Z

ng-zorro-antd version: X.Y.Z

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] IE version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

pic:

please provide your env

Angular version: X.Y.Z

ng-zorro-antd version: X.Y.Z

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] IE version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

should put in icebox. should not close issue is not intended

it seems that popup is not following the scrollbar.
or should close popup when scroll change.
@fjh352


 Resulting image is correct, but takes longer to burn.
I am not expecting this to be fixed in 0.7 - I am reporting it so that this can be corrected in 0.8 if it still exists. (My limited experience with 0.8 is that it didn't happen.)
Also, some but not all raster wizards also treat these same pixels as black.

I need more details and maybe an example of this. Fixes in 0.7.x are still permitted and will probably be done. They just have to be done with an extremely narrow scope that modifies as little code as possible to fix a very specific error. We actually have a few things in 0.7.x that need fixing. If the fix isn't difficult and will not create any additional bugs that's absolutely a thing that goes into maintenance mode.
Secondly, please actually recheck that the laser was entirely off. The only reason it would go to a place is to laser it in raster mode. Get the example bit you checked and see that it didn't actually fire a single laser pulse or two in that area. You might need to look at an angle or whatnot. But, the code doesn't really allow the behavior you're describing. If it's going to that location in a raster, it's going there to fire at least one pulse.

Ruidacontrol RunStatus reply is now known.
Corrections for wx4.1 / py3.10. (There's crashing if they run this combo from source)*
Correctiions for wx4.2 / py 3.10. (This actually mostly works and a few windows need fixing)*

* - python 3.10 changed the binding rules to no longer silently permit floats to be cast to ints when going to C++. These need to be overt, wxPython is entirely made of those things so we have a special test case of wx.Size(0.5) and if that crashes we know the python and wxPython aren't compatible, the exception being wxPython 4.2 which mostly fixed everything internal but we'll need to cast a few values to int in MK source.

I checked the png - they were semi transparent white pixels.
I have no idea whether the laser was firing because it was burning faintly om paper i.e. 100ppi and low power so beam was not visible.
The PNG file is the one on the 0.8beta13 bugs issue.
You can see the artefacts if you use the raster wizard on this PNG - you get black dashes around the edge.


Include property that is missing in transaction.js

Relevant issue: #5269

@ConnorDuvalS Could you please add integration test making use of this field in some way?

Coverage increased (+0.0003%) to 92.272% when pulling ca0d49245b5affa8d0e920b5a77a9ac969a7f7b2 on ConnorDuvalS:master into 72065c179772c18dc4707a1dfd5990ec8b09fd46 on knex:master.


Base animations from TailwindCSS are gone

This package doesn't quite function how a normal TailwindCSS plugin does (you can't just register it inside the plugins array in your config), instead you have to wrap your own config with it's withAnimations function and that causes a problem. It gets rid of the default animations, such as animate-spin, animate-pulse, etc.
I believe this should be patched, maybe whenever this starts following Tailwind's conventions for plugins, but for now if you have the same issue, you can yoink the data injected by their withAnimations function and extend the config yourself.
Here's a sample that worked for me, basically don't wrap their function (it doesn't require a config) and just use the response on your own config:
const animatedTailwindCSS = require('animated-tailwindcss')();

module.exports = {
  theme: {
    extend: {
      animation: { ...animatedTailwindCSS.theme.animation },
      keyframes: { ...animatedTailwindCSS.theme.keyframes }
    },
  },
  plugins: [ ...animatedTailwindCSS.plugins ]
};

It's a bit dirty but it does the job for now.

This is actually mentioned explicitly in the docs:

For compatibility reasons, the built-in animations (spin, ping, pulse, bounce) will be removed (or replaced by their Animate.css counterparts).

Earlier I was (internally) extending the config too, but that caused weird merging with the built-in ones like bounce as both TailwindCSS and Animate.css have it. IIRC, the keyframes were getting properties of both instead of Animate.css overriding TailwindCSS. Not sure if that issue still persists.
Regarding your hack, you can improve its readability by using destructuring assignment:
const { theme, plugins: animationPlugins } = require('animated-tailwindcss')()

module.exports = {
  theme: { extend: { ...theme } },
  plugins: [ ...animationPlugins ]
}

We probably can have tw- prefixed default animations to avoid conflicts. I'll look into that. Thanks.

This is actually mentioned explicitly in the docs:

That is true! Although there isn't any proposed workaround for the time being, hence why I posted this issue, in case someone else comes up with a need to access the default animations.

the keyframes were getting properties of both instead of Animate.css overriding TailwindCSS

That still persists, unfortunately, but for the built-in ones that I use I don't have any issues luckily!

We probably can have tw- prefixed default animations to avoid conflicts.

One idea that will be less work and not having to bother transforming the defaults:

Since this package isn't actually importing the animate.css package as a dependency, you could have a different name for the conflicting ones declared here, just document the animations in a playground style (like animate.css own website). It's already changing the default prefix animate__ to animate- anyways.

just document the animations in a playground style (like animate.css own website

This is already there: https://ikcb.org/animated-tailwindcss/docs/animations

Fixed in v3.3.0. Check the updated docs.


 I'm sick of looking at it!

Fixed in a797397


Update CPU manager feature enabling

With CPUManager feature graduating to beta. No explicit enabling is
required starting v1.10.
References: kubernetes/kubernetes#55977

This change is

/hold
/assign

@heckj ping

/hold cancel
/approve
(Thanks for the reminder @tengqm )

/lgtm


Foundation ZURB Template error - sassBuild error after running 'yarn start'

What should happen?
... Should build the template and be viewable at localhost:8000
What happens instead?
...[17:02:17] 'sassBuild' errored after 1.54 ms
[17:02:17] TypeError: The "listener" argument must be of type function. Received undefined
at checkListener (node:events:259:3)
at _addListener (node:events:560:3)
at Transform.addListener (node:events:619:10)
at Transform.on (node:internal/streams/readable:875:35)
at sassBuild (/Users/%username%/Documents/GitHub/%folder%/projectname/gulpfile.babel.js:108:6)
at bound (node:domain:421:15)
at runBound (node:domain:432:12)
at asyncRunner (/Users/%username%/Documents/GitHub/%folder%/projectname/node_modules/async-done/index.js:55:18)
at processTicksAndRejections (node:internal/process/task_queues:78:11)
[17:02:17] 'build' errored after 1.67 s
[17:02:17] 'default' errored after 1.67 s
error Command failed with exit code 1.
I changed gulp-sass from 4.1.1 to 5.1.0, if I have version 4.1.1 installed I get the following error:
error /Users/$username%/Documents/GitHub/%folder%/projectname/node_modules/gulp-sass/node_modules/node-sass: Command failed.
Exit code: 1
Command: node scripts/build.js
Arguments:
Directory: /Users/$username%/Documents/GitHub/%folder%/projectname/node_modules/gulp-sass/node_modules/node-sass
Output:
Building: /usr/local/bin/node /Users/$username%/Documents/GitHub/%folder%/projectname/node_modules/node-gyp/bin/node-gyp.js rebuild --verbose --libsass_ext= --libsass_cflags= --libsass_ldflags= --libsass_library=
Possible Solution

...
Test Case and/or Steps to Reproduce (for bugs)

Test Case:

How to reproduce:

git clone https://github.com/foundation/foundation-zurb-template projectname
2 yarn
3 cd projectname
4 yarn start

Context

...
Your Environment

Foundation version(s) used: 6.7.3
Browser(s) name and version(s):
Device, Operating System and version: MacBook Pro M1, Monterey 12.4
Link to your project:

Checklist

[x ] I have read and follow the CONTRIBUTING.md document.
[x ] There are no other issues similar to this one.
[x ] The issue title and template are correctly filled.

Your order is a bit off...
git clone https://github.com/foundation/foundation-zurb-template projectname
cd projectname
yarn install
yarn start

I have exactly same errors.


Password Strength Meter

Project title
Password Strength Meter
Project Description
Lets the user check how strong their password is. [Strong,medium,weak]
Stack:
[x] Html
[x] CSS
[x] JavaScript

Please assign it to me!


DatabaseCleaner not cleaning between otherwise-properly-running tests in spec/lib directory

I have a PORO class EventInterface which creates Event instances (Event is an ActiveRecord-backed class).  I have the EventInterface class in my app/lib directory, and configured Rails to load it.  All functions just fine.
For the sake of consistency, I created a corresponding spec/lib directory, and an event_interface_spec.rb file for my tests.
RSpec is running those tests in spec/lib just fine, and DatabaseCleaner is (and has been) performing dutifully across everywhere else my test suite.  But I ran into a problem where Event records created in event_interface_spec.rb are not cleaned out, and are affecting other tests (those other tests pass just fine when run in isolation).
I confirmed that lingering records were the culprit for the failures.  I then confirmed that DatabaseCleaner was not running between the tests in my spec/lib directory by moving event_interface_spec.rb (unchanged) from spec/lib to spec/models.  After the move, my entire suite passed.
I haven't found a way to configure DatabaseCleaner to explicitly include/exclude test files or directories — does one exist?  If not, is there a reason DatabaseCleaner wouldn't run between tests in my spec/lib directory, despite RSpec running those tests and despite DatabaseCleaner running properly everywhere else in my suite?
Here is my DatabaseCleaner-relevant config, for reference:
RSpec.configure do |config|

  config.before(:suite) do
    DatabaseCleaner.strategy = :transaction
    DatabaseCleaner.clean_with(:truncation)
    load "#{Rails.root}/db/seeds.rb"
  end

  config.before(:each, js: true) do
    DatabaseCleaner.strategy = :truncation
  end

  config.before(:each) do
    DatabaseCleaner.start
  end

  config.after(:each) do
    DatabaseCleaner.clean
  end
end

Are you sure that the cleaner is not being ran at all when the files are located inside spec/lib?
It could be an dependency order bug since moving the files to another folder would change the run order of the specs

@jasonmklug What does event_interface_spec.rb look like?

Apologies for the slow response.
@karlingen nailed it — I was creating an Event record inside a before(:all) block, so DatabaseCleaner (rightly) didn't roll back the creation of that record between tests.
Moving the problematic spec file to spec/models still resulted in lingering Event records, but the move changed it's place in the order in which RSpec runs my spec files, such that existence of the lingering records didn't affect the few existing-record-sensitive tests in my suite.
Thanks.  Consider this one closed.


Mouse click has a weird behaviour ?

Operating system or device - Godot version:

Windows 7 Pro, Service Pack 1
64 bits
Godot_v2.1.3-stable_win64

Issue description:
I noticed a problem with the buttons (and scrollbars) and the mouse inputs.
Actually, when you hold another mouse button than the left button (so, right button, wheel button, etc), then you press the left click, then you release the other button, and finally you release the left click, the button appears pressed but is not considered as pressed.
Same problem for the scrollbar buttons.
(Note that the problem happens also when you hold two other buttons than the left)
It's something which happens very rarely (I discovered it by accident), but it can happen with all Godot's buttons (including these in the editor).
Steps to reproduce:

Press Right-click with mouse over a button and hold it
Press Left-click and hold it, with the previous right-click holded.
Release Right-Click
Release Left-Click
(It also works with the wheel-click instead of Right-Click)

Thanks !

Reproduced in the project manager of Godot 2.1.3. Works with almost anything: scrollbars, buttons, drop downs.

Does this happens in other OS?

I just tried this in Windows 10 with Godot 3.0 pre-alpha. Can confirm it's happening here as well.
Also, the button_down event fires, but not the button_up, button_pressed, or button_toggled events.

Can confirm in macOS, same behavior

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the current stable version, and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about its current status here?
For bug reports, please also make sure that the issue contains detailed steps to reproduce the bug and, if possible, a zipped project that can be used to reproduce it right away. This greatly speeds up debugging and bugfixing tasks for our contributors.
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.

This clicking repro is a similar story to this one: https://github.com/godotengine/godot/issues/14373#issuecomment-350515100

#14373 and related issues were fixed recently in the master branch. Is this one still valid, or fixed too?

Can reproduce on 3.0.5's project manager, Import button, following steps in the OP. Cannot reproduce on the Tabs.

Couldn't reproduce the issue on master eeee1dc73e6344a5541ee323b6ae61744b3348b6 and 3.1.1 stable.

Indeed, I can reproduce it with a current build from the 3.0 branch, but it's fixed in the 3.1 branch and master.


Feature Request: Add Playback Controls via Thumbnail Buttons similiar to Windows Media Player

I'm not sure if this is possible but would be neat if it could be implemented somehow.
https://docs.microsoft.com/en-us/windows/win32/shell/samples-taskbarthumbnailtoolbar
https://docs.microsoft.com/en-us/windows/win32/shell/taskbar-extensions?redirectedfrom=MSDN#thumbnail-toolbars
Examples:

Edit: updated some links

It was implemented by this script so no need for this anymore
https://github.com/qwerty12/mpv-taskbar-buttons
will close this issue :>


Redisson does not recover from DNS failures on master resolution for single-shard clusters

Affected Versions
3.32.0 onwards. Validated with 3.31.0 and it was not present; 3.32.0, 3.33.0 and current master as of now are affected.
Redis Cluster Details

AWS ElastiCache
Redis 6.2.6
Instance t3.micro
Cluster Configuration

1 Shard (aka 1 Master)
2 Slaves

Issue Isolation

Tried with different instances size; behaviour did not change.
Tried with different amount of slaves; behaviour did not change.
Tried with different amount of shards; behaviour DID change.

The issue is only reproducible on single-shard clusters.
Issue Description
If Redisson detects a DNS resolution failure on the master node for a cluster, and there is only a single master (single shard), it never connects back again despite the DNS issue being fixed.
To reproduce, the following code was introduced into ServiceManager.resolveAll promise:
            String host = uri.getHost();
            if (host.contains("-001.")) {  // Target only masters
                if (!attemptCounter.containsKey(host)) {
                    attemptCounter.put(host, 0);
                }

                Integer attempt = attemptCounter.get(host);
                log.info("ATTEMPT {} for {}", attempt, host);
                attemptCounter.put(host, attempt + 1);

                if (attempt > 1 && attempt < 4) {
                    log.error("FORCING DNS FAILURE for {}", uri, f.cause());
                    result.completeExceptionally(new RuntimeException("Dummy"));
                    return;
                }
            }

The following was observed:

On DNS success, everything works fine.
When DNS failure occurs, it fails all connections.
When DNS issue is mitigated, it resolves correctly the IP, but it does not connect back and from a Redisson client perspective, you cannot fetch any keys anymore.

I did try to replicate failure for ALL masters in multi-shard, but it connected back again correctly.
The initialization of Redisson is as follows:
        Config config = new Config();

        // Only one node of cluster configuration endpoint need to be specified - all other nodes are auto detected
        // SLAVE - Read from slave nodes, uses MASTER if no SLAVES are available.
        // MASTER - Read from master node,
        // MASTER_SLAVE - Read from master and slave nodes
        config.useClusterServers()
                .setReadMode(ReadMode.MASTER_SLAVE)
                .addNodeAddress(REDIS_DATABASE_URL)
                .setTimeout(REDIS_TIMEOUT)
                .setRetryAttempts(REDIS_RETRY_ATTEMPT)
                .setRetryInterval(REDIS_RETRY_INTERVAL)
                .setConnectTimeout(REDIS_CONNECTION_TIMEOUT);

        return Redisson.create(config);

Thanks for report. Are there any errors in logs?

@mrniko here are the logs: https://pastebin.com/xARR04A3 (sent in Pastebin because they are extensive and it would overload the comments here)
TestClient is just a dummy Java program with an infinite loop:
        RedissonClient client = testClient.getClient();

        while (true) {
            try {
                RMap<String, String> map1 = client.getMap("test");
                map1.put("a", "b");

                RMap<String, String> map2 = client.getMap("test");
                String value = map2.get("a");
                log.info("Retrieved from cache: {}", value);

                RMap<String, String> map3 = client.getMap("test");
                map3.remove("a");

            } catch (Exception e) {
                log.error("Exception");
                e.printStackTrace();
                // break;
            }

            Thread.sleep(2000);
        }

Can you try attached version or build from master branch?
redisson-3.33.1-SNAPSHOT.jar.zip

@mrniko thanks, but would it be possible to share the git patch instead? I am simulating the master nodes DNS resolution failure by directly modifying Radisson source code, so I cannot mimic the failure if I have the compiled JAR file.

Here is the commit - https://github.com/redisson/redisson/commit/7d9b899fa139053134057cca7d18be741bb36b01

@mrniko thank you, just confirmed that this commit has fixed the issue:

On single-shard, a DNS failure on master nodes does not prevent Redisson from fully fail to access Redis:

During DNS failure, Redisson falls back to previously known nodes (not sure if it is going to slaves or using the "previously known IPs").
After DNS failure, Redisson recovers.

Multi-shard cluster behaves the same way:

Same as single shard...
The NPE fix seems to also have worked, because on multi-shard we were getting NPE even after DNS recovery (it was not affecting reading or writing though).

It looks like the behaviour is restored to the previous version. Thank you again for the quick fix!

Thanks much for testing!


Test suite websocket

I was using client.transition to simulate incomings of errors & connection state change and it is wrong because some of the logic is missed.
To change the connection states:

client.onConnected(message: ARTProtocolMessage)
client.onDisconnected()
client.onSuspended()
client.onError(AblyTests.simulateError())

With this PR we have:

simulateIncomingNormalClose
simulateIncomingAbruptlyClose
simulateIncomingError
simulateLostConnection

@tcard What do you think?
@mattheworiordan I really needed this. There were several solutions around the test suite.

Looks fine, few comments re: naming

LGTM


Add files via upload

Hello. I may be 2 years late but I am just taking a matlab class and I discovered this...so I wanted to give it a go? Thanks.

You're not too late at all. However, you've not issued the pull request correctly. At the moment you are requesting to merge from your branch, klduran22/klduran22-patch-1, into your other branch, klduran22/master. Instead, you need to request a merge into the upstream repository (my repository) scottclowe/matlab-schemes.
You should follow the instructions here: https://help.github.com/articles/creating-a-pull-request-from-a-fork/
Basically you need to click the New Pull Request button on this page here: https://github.com/scottclowe/matlab-schemes/pulls, and then select your klduran22-patch-1 branch.
(You're actually rather lucky I happened to spot this!)

Oops my bad, I do not know how to work GitHub.


Benchmark statistics

Record more benchmark samples, and compute, display, and use statistics based on them.

Change default settings to record more (but shorter) samples. Changes meaning of goal_time Makes determination of goal_time more accurate; however, no big changes to methodology --- there's room to improve here.
Do warmup before benchmark --- this seems to matter also on CPython (although possibly not due to CPython itself but some OS/CPU effects)
Display spread of measurements (median +/- half of interquartile range)
Estimate confidence interval and use that to decide what is not significant in asv compare.
Optionally, save samples to the json files.
Switch to gzipped files.

The statistical confidence estimates are a somewhat tricky point, because timing samples usually have strong autocorrelation (multimodality, stepwise changes in location, etc.), which makes simple stuff often misleading. There's some alleviation for this currently there in that it tries to regress the timing sample time series looking for steps, and adds those to the CI. Not rigorous, but probably better than nothing.
The problem is that there's low-frequency noise in the measurement, so measuring for a couple of second does not give a good idea of the full distribution.
todo:

[x] more tests, e.g. printing and formatting is mostly untested
[x] do we really need to gzip?
[x] update documentation
[x] tuning for pypy

Coverage increased (+0.7%) to 72.988% when pulling f953513a4e9166a66e39f9974254b336f300e967 on pv:timing-estimator6 into cfe3d3e41cdc9ad44973ce0012e3ac021bb40369 on spacetelescope:master.

Coverage increased (+0.6%) to 72.971% when pulling bd4ddcac1b545f40e0f5460eab7e6a161f72e460 on pv:timing-estimator6 into cfe3d3e41cdc9ad44973ce0012e3ac021bb40369 on spacetelescope:master.

Coverage increased (+0.6%) to 72.971% when pulling bd4ddcac1b545f40e0f5460eab7e6a161f72e460 on pv:timing-estimator6 into cfe3d3e41cdc9ad44973ce0012e3ac021bb40369 on spacetelescope:master.

Coverage increased (+0.6%) to 72.941% when pulling 364e0edd37cc1a13c171a89db9e9bed8e863e96f on pv:timing-estimator6 into cfe3d3e41cdc9ad44973ce0012e3ac021bb40369 on spacetelescope:master.

Changing goal_time to mark a single repeat time is nice, but do I read it right that setting repeat = 5 will take 5 samples if number is fixed and 50 samples if it is left at zero? Why not make the number of samples equal to repeat in the adaptive case too (with a single setup per sample and a higher default)?

Yes, I've got something like that in the works. This is partly
orthogonal to this PR, however.
.
The benchmark methodology may still need tuning. One issue faced here
not faced by perf is that the benchmarks that run later, run under
different CPU/systemload/heating conditions than the benchmarks that run
earlier. So the sampling probably should use multiple processes, and be
done in A B C ... A' B' C' ... A'' B'' C'' ... order.

Interleaving benchmarks would surely be good, comparability across separate import benchmarks was something I was aiming for. The whole samples/repeats logic would probably need to be moved up to realize that though (with a "sample" rather than a "benchmark" script?).

Coverage increased (+0.7%) to 73.009% when pulling 8c41b730470235fe4c2b098c843305f22447127d on pv:timing-estimator6 into 390145a57b8c6644e570af844ef28bcfd32e460d on spacetelescope:master.

Coverage increased (+0.7%) to 72.991% when pulling 8c8fbe031c90a5c8781a3cefe598a970ed732ec3 on pv:timing-estimator6 into 390145a57b8c6644e570af844ef28bcfd32e460d on spacetelescope:master.

Coverage increased (+0.9%) to 73.191% when pulling 2c5b78ec726cfb1f90f30457dfd4bf2788e8cf1e on pv:timing-estimator6 into 390145a57b8c6644e570af844ef28bcfd32e460d on spacetelescope:master.

I think this is more or less finished now. Some fine-tuning may be useful to do later on.
The changes in benchmarking parameter default values may be annoying, but these should give better accuracy. The default runtime also becomes shorter. Big improvements on the accuracy probably will need stuff such as splitting the benchmark runs into several parts, to be run in an interleaved order to ensure long sampling time spans to capture the low-frequency noise properly. https://github.com/pv/asv/tree/many-proc

Coverage increased (+0.9%) to 73.203% when pulling 26a42ba1cf0e788f6d0ee6f0139d080b2e38f186 on pv:timing-estimator6 into 390145a57b8c6644e570af844ef28bcfd32e460d on spacetelescope:master.

Coverage increased (+0.9%) to 73.187% when pulling fc2045de00c174caa3b4af53a11654717dd462cb on pv:timing-estimator6 into 390145a57b8c6644e570af844ef28bcfd32e460d on spacetelescope:master.

Coverage increased (+0.9%) to 73.233% when pulling 347bb535724a318a5c93c95028c82328aae6caaa on pv:timing-estimator6 into e581104883574a92e277f274cd784675d87cb807 on spacetelescope:master.

Coverage increased (+0.9%) to 73.253% when pulling 50ddeb8edfdc830948d3c4cd499ab4b00ed3806c on pv:timing-estimator6 into e581104883574a92e277f274cd784675d87cb807 on spacetelescope:master.

Coverage increased (+0.9%) to 73.268% when pulling 10fdd49944b353627e0366697c78572ce3aeacbb on pv:timing-estimator6 into e581104883574a92e277f274cd784675d87cb807 on spacetelescope:master.

After some dogfooding, I think this is ready to go.


Write an a example how to use without formtastic or Active Admin

I set on text area fields class 'rich_editor/edtior/rich' but CKEditor doesn't load. When i put custom id 'editor' on text area field and do this CKEDITOR.replace( 'editor' ) form firebug CKEditora is loaded.  Can you write some example, how to use it without formtastic, please.

usage without either formtastic or Rails Admin is not supported. If you'd like to add that functionality, you're more than welcome.


Zero-dollar invoices

[Enter feedback here]
How do I get you to stop sending zero-dollar invoices? There's no mention of an option here.

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

ID: 925122d4-0d3a-f271-02d8-db773c647cbe
Version Independent ID: c5d17dc9-54c4-3092-85a9-3710e15983cb
Content: Understand your invoice for your Microsoft MOSA billing account
Content Source: microsoft-365/commerce/billing-and-payments/understand-your-invoice2.md
Service: microsoft-365-business
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@dmore54 Thank you for your feedback. Sorry about that, I'd recommend you contact our billing team via the need help section on the admin center, so that our engineer can assist you with the query.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @dmore54 Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. You can contact Microsoft support via the need help section on the admin center. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Specify Dart liveness guarantees

Currently Dart2JS and the Dart VM diverge in what variables are captured by a closure.
The Dart2JS behavior better matches user intuition on what a closure should capture and is less likely to introduce surprising memory leaks. The DartVM behavior leads to fewer context allocations and may have significant runtime performance benefits. Very few Dart users understand what closures capture as shown by https://github.com/flutter/flutter/issues/79605. If the DartVM behavior can be made to match the Dart2JS behavior without a regression of Flutter app performance, that would greatly reduce hard to debug Flutter memory leaks. If not, it might be beneficial to align Dart2JS with DartVM to improve consistency of Flutter Web and Flutter Native.
As mentioned by @rmacnak-google in https://github.com/dart-lang/sdk/issues/45455#issuecomment-831474391_, debuggers would prefer a solution that kept even more variables alive than the Dart VM. Such a solution would introduce significant memory leaks in Flutter apps and would only be interesting for users debugging logic not performance issues. An intermediate solution for debuggers would be to to make debuggers more clearly surface which variables are live.
The lack of specification is more of an issue now that Flutter code can run on the VM and Dart2JS and Dart is looking to add WeakRef support which makes finalization behavior more visible to user code . Flutter code that leaks badly on the VM may not leak on Dart2JS.

cc @mraleph @sigmundch


Create the initial runtime-watcher release

Create manual 0.1.0 release of the runtime-watcher.
Purpose:
We need to start releasing. Once the initial release is done, the automatic release workflow may be used to create subsequent releases.
Acceptance Criterias:

[ ] Release runtime-watcher 0.1.0
[ ] Create initial Release-Notes

https://github.com/kyma-project/runtime-watcher/releases/tag/0.1.0


[C 05:11:05.325 NotebookApp] Bad config encountered during initialization: #12751

When I try to run IJulia since julia

using IJulia
notebook()

[C 05:11:05.325 NotebookApp] Bad config encountered during initialization:
[C 05:11:05.325 NotebookApp] Unrecognized flag: '--profile'
ERROR: failed process: Process(ipython notebook --profile julia, ProcessExited(1)) [1]
in pipeline_error at process.jl:502
in notebook at /home/vikohtb/.julia/v0.3/IJulia/src/IJulia.jl:177
in notebook at /home/vikohtb/.julia/v0.3/IJulia/src/IJulia.jl:176
And the directory src does not exist.

I'm guessing you have a recent version of IPython?  Try Pkg.update(); Pkg.checkout("IJulia"); Pkg.build("IJulia") to get the latest git master of IJulia, which no longer uses --profile (and consequently requires IPython 3 or later).

Thanks! that works, the error is done. But in the page open says Jupyter no IJulia, that means something?

The Jupyter project is the new name for IPython, to highlight its support for multiple programming languages.

Technically, IPython still exists: it is the Python kernel (back-end) for the Jupyter front-end.  IJulia is the Julia back-end for the Jupyter front-end.


deploy-test deploys bm-invetory image to minikube to ease testing

Instead of updating quay.io user repository every time
Allowing make deploy-test to deploy bm-inventory local code to minikube to be tested

Rebased @oshercc and changed the default behavior to skipper make ...


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3037

Questions ?
Please refer to the Backport tool documentation


Rename catalogue-item to work

Type
🚑 Health
Value
Updating nomenclature to match platform team.
Checklist
All

[x] Demoed to the relevant people
[x] PR labelled and assigned
[x] Any introduced code complexity has been flagged

Conflicts...


Add neighbors feature, closes #118

For #118.
I just started to write a neighbors function in my code again and thought: this has to stop, this should be upstreamed! :no_good:
This changeset adds a basic mercantile.neighbors function (& command) returning the adjacent tiles for an input tiles.
The feature only returns direct neighbors and in no guaranteed order.
I've added a few more commits for things I ran into; have a look please! :bow:

Coverage increased (+0.02%) to 99.619% when pulling 791bfdc70231329800ab3bcbf7ba80db7c723ca2 on daniel-j-h:add-neighbors into d00e29330663d2ca8dc1f4056f92194286250b38 on mapbox:master.

Thank you @daniel-j-h !


Add changelog for auto snapshot

closes #1670
Description
Adds a changelog for Device Auto Snapshots
Related Issue(s)
#1670
https://github.com/FlowFuse/flowfuse/issues/3358
Checklist

[x] I have read the contribution guidelines
[ ] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

Waiting for the tests

Waiting for the tests

and maybe a little longer too (I spotted the word "bases" - not a typo, just a wrong word 🤦 Now corrected to "basis" in db70be4

Odd error (I never touched anything near that)
Error: bad anchors:
  docs/install/kubernetes#upgrading-to-2.0

Found 0 bad links
Found 1 bad anchors
Error: Process completed with exit code 2.

@ppawlowski https://github.com/FlowFuse/flowfuse/pull/3435 removed a section named #upgrading-to-2.0 and is causing an error in the website tests.
I don't know what the doc is supposed to be or if it is even relevant now?
Could you take a look?

@MarianRaphael I propose we proceed with this merge and a separate issue/PR is raised to fix bad links